alecbcs commented on issue spack/spack-packages#1028.
Unfortunately the CI failures are related. I’m still figuring out a fix. …
jeremylt commented on issue freeCodeCamp/freeCodeCamp#61453.
I recommend reading the contributing guide. Issues are not assigned and the first valid PR is accepted….
grondo commented on issue flux-framework/pr-validator#9.
Tested and seems to be working, thanks!…
garlick open issue flux-framework/flux-core#6960.
rc1.0: /etc/flux/rc1.old: 91: shopt: not found
Problem: after updating to 0.77, a flux system instance on debian-12 won’t start due to the shopt command in rc1.old….View Comment
grondo commented on issue flux-framework/flux-core#6959.
Ok, that makes sense. It only came up because modprobe wants a list of all modules, but has to treat connector-local differently since it is auto-loaded and should not be unloaded, etc. Easy enough to work around though….
grondo commented on issue flux-framework/rfc#464.
Yes, please merge….
wihobbs open issue flux-framework/pr-validator#8.
refactor script to explicitly define truthy return values
In the pr-validator, because of the naming convention, a problem in the commit is a truthy return value in the existing functions (e.g. for is_body_empty(), body_line_length_exceeds() etc.).
…View Comment
JustinPrivitera open issue Alpine-DAV/ascent#1598.
Make Rover Metadata Match VisIt
It occurs to me that we are writing these values out from Rover different than VisIt: https://visit-sphinx-github-user-manual.readthedocs.io/en/develop/using_visit/Quantitative/XRayImageQuery.html#metadata …View Comment
cyrush open issue Alpine-DAV/ascent#1597.
caliper annotation mismatch
Example:…View Comment
JustinPrivitera commented on issue visit-dav/visit#20520.
Thank you! Let’s leave this ticket open then until we get the dmg file….
tzanio commented on issue mfem/mfem#4970.
ping: @jandrej, @sohailreddy, @alexanderianblair …
wihobbs commented on issue flux-framework/rfc#414.
This is long out of date, and a lot of the proposed components are unnecessary now due to improvements in the python-based job-manager JournalConsumer class and the ability to extend the command line via plugins. Maybe we’ll still need an RFC for notifications, but it will look very different from this one. Closing….
chu11 commented on issue flux-framework/flux-core#6953.
> Seems like repair should be the default …
trws commented on issue flux-framework/flux-core#6954.
Yup, sounds right. Feel free to hit it, or I can if you prefer….
trws commented on issue flux-framework/flux-accounting#714.
Yes, definitely. It’s useful to have the requested time because it has a significant impact on queue policy outcomes, especially with backfill policies, and can help us determine what queue policies and settings to use in the future….
cmoussa1 commented on issue flux-framework/flux-accounting#714.
OK, wrote up a small test and it looks like flux-accounting is fetching the updated duration, so this is something that flux-accounting could store in a separate column in its job-archive if that was to be desired?…
benzwick commented on issue Slicer/Slicer#6388.
Is there any documentation that explains how to compile Slicer with Qt6 so we can test this on Linux (and later on Mac and Windows)? And which branch should be used to test the latest progress on this issue?…
jeremylt commented on issue CEED/libCEED#842.
Given that XSMM targets this hardware and already gives us better performance on AVX hardware, I propose we close this. Any objections?…
alecbcs commented on issue spack/spack-tutorial#477.
Merging without waiting for a review since this exact file is already in spack/spack….
vladotomov commented on issue mfem/mfem#4289.
I might be missing something, but I’m not seeing the value of this example. Remhos already handles advection, offers more sophisticated limiting options, computes local bounds correctly based on the stencil, and supports general meshes. I realize this showcases the new bounds calculation, but there are already dedicated examples for that….
garlick commented on issue flux-framework/flux-sched#1380.
With FLUX_RC_USE_MODPROBE=t set for tests, flux-core 0.77 is required, but looking at one of the bookworm test failures, it appears we’re picking up something less than: …
jameshcorbett commented on issue flux-framework/flux-coral2#383.
I’d like to read through the core and sched modprobe PRs first so I can understand what’s going on. I’m doing that now. So far looks totally fine though….
garlick commented on issue flux-framework/flux-coral2#383.
NP if we want to hold off. I guess I am enthusiastic about moving fowrard. :-)…
wihobbs open issue flux-framework/flux-accounting#714.
archive the amount of time a user _requested_ for their job
In the fluxion meeting today, @trws brought up that at some point system admins might want to penalize people for requesting far more time than their job actually took. …View Comment
adayton1 commented on issue LLNL/CHAI#298.
Could you add a new CMake configuration variable? Something like CHAI_ENABLE_EXPERIMENTAL? Then in CMakeLists.txt only add the new files if CHAI_ENABLE_EXPERIMENTAL is on. …
v-dobrev commented on issue mfem/mfem#4923.
@lindsayad, if it is not too much trouble, can you try this branch to see if it solves the issues you encountered in #4959?…
wihobbs commented on issue flux-framework/flux-sched#1285.
> How did you test this? I added the change in this commit to t1020 in current master and the test still passes. …
milroy commented on issue flux-framework/flux-sched#1285.
> Are we still OK with leaving in the uncovered code in resource_match.cpp:select_subsystems? …
bgunnar5 commented on issue LLNL/merlin#524.
TODO: add documentation…
tgamblin commented on issue spack/spack#51061.
@haampie We discussed a number of solutions for this, and I don’t see an easy one. The proposal is not to ignore the provenance – it’s to keep it but build one version as another. As you say, when you tell Spack to treat sources as a particular version, you are doing the same thing – assigning a version. In fact, you could already do what is in this PR if you had the git repo for every version. We’re simply extending it to work for releases/tarballs instead of just git versions….
bwpriest commented on issue LLNL/ygm#368.
Ok, good to know. Should we cancel this PR?…
tgamblin commented on issue spack/spack-packages#924.
fixed comments…
tgamblin commented on issue spack/spack-packages#739.
Finally
tzanio commented on issue mfem/mfem#4955.
Merged in next for testing……
cjvogl commented on issue E3SM-Project/E3SM#7481.
@tcclevenger , can you please add the CI: Approved flag to run the tests?…
JustinPrivitera commented on issue Alpine-DAV/ascent#1587.
…
JustinPrivitera commented on issue visit-dav/visit#19488.
I think this is a good idea and a good feature to add but it would be nice to also fix the underlying issue causing things to be so slow….
hppritcha commented on issue open-mpi/ompi#13351.
@edgargabriel could you check what’s going on with rocm ci?…
v-dobrev commented on issue mfem/mfem#4944.
Unfortunately, I don’t think there’s a way to automate the synchronization of the validity flags, at least not with the current interface….
mplegendre commented on issue hpc/Spindle#4.
Closing old PRs…
trws commented on issue flux-framework/flux-sched#1385.
@mhyatt000, what version of spack are you using? The dependency is listed, and has the correct version. Is it not pulled in as one of the dependencies of your sched build? Here’s the relevant part of the package….
milroy commented on issue flux-framework/flux-sched#1382.
Thanks for the review @trws! Adding MWP….
chu11 commented on issue flux-framework/flux-core#6589.
Wanted to start a design discussion on what we would like flux-fsck to look like. First, summarizing several PRs I submitted….
jameshcorbett commented on issue flux-framework/flux-coral2#379.
I think I understand now. As can be seen in the eventlog, Flux was restarted while the job was in the queue, so the plugin was reloaded. The exception callback in the jobtap plugin is only triggered if the plugin has subscribed to events for the job, but the call to flux_jobtap_job_subscribe() is only done in the callback for the SUBMIT state. (See here.) So the exception callback was never invoked and the prolog was never removed….
vladotomov commented on issue CEED/Remhos#53.
@pearce8 it’s merged, thank you, sorry for the delay….
cyrush open issue Alpine-DAV/ascent#1591.
replay: passing dir instead of root file crashes
need to make this an error message, not a crash…View Comment
cmelone commented on issue spack/spack-tutorial#467.
@scheibelp …
becker33 commented on issue spack/spack-packages#885.
I think we should we backport this to v1.0.1. It may come up in release CI for future 1.0.x releases…
becker33 closed issue spack/spack#51054.
Non path-based package repositories in environments broken
### Steps to reproduce…View Comment
garlick commented on issue flux-framework/flux-core#6260.
Oh, I thought that one was already in! Sorry, will look at that now….
tgamblin commented on issue curl/curl#18104.
Thanks for this!…
tpatki commented on issue LLNL/variorum#587.
Thanks for testing this on Tuo, @kulnaman!…
jeremylt commented on issue CEED/libCEED#1881.
Ok, the blocking issue here seems to be around permissions to create files/folders in the runner…
jedbrown commented on issue rust-lang/rustup#3397.
It would be very useful if we could rustup which llvm-link or perhaps rustup which --component llvm-tools llvm-link. If you agree this would be useful, do you have a preference on the two syntaxes, or pointers to where you think it should be implemented?…
tomstitt commented on issue mfem/mfem#4028.
> Thanks for checking @tomstitt! Can you give it another try with the latest commit? …
chu11 commented on issue flux-framework/rfc#459.
i will simply merge this manually until we can figure out what happened (or perhaps mergify updated some check and we need to update our mergify config)…
wihobbs commented on issue flux-framework/Tutorials#55.
Smart idea, my only worry is that we have sent and posted links that reference the existing directory structure. Is there a way to invisibly symlink everything so that the current links continue to work? (probably not :/)…
cyrush commented on issue LLNL/conduit#1453.
@Arlie-Capps since there were changes to the amr bent grid example as part of the PR – this has diverged. …
cyrush commented on issue visit-dav/visit#20509.
This is currently running tests using test data from an http mirror, w/o baselines (which of course will fail). …
alecbcs commented on issue spack/spack-packages#571.
@adamjstewart this is a bit our of my wheelhouse. Can you speak to this situation a bit more? …
wihobbs commented on issue flux-framework/flux-core#5531.
I wanted to update this issue with the outstanding to-dos as I see them….
gardner48 commented on issue LLNL/sundials#753.
Hi @mottelet, between versions 6.6 and 7.4 there have not been any changes to the projection capability in CVODE. The error handling across SUNDIALS was updated in v7 however, this should not have changed any of the return codes from CVODE. There were some fixes related to the stop time in versions v6.6.1 and v6.7.0 that could be related to what you are seeing. Before these fixes, it was possible that CVODE would reach a stop time but return CV_SUCCESS instead of CV_TSTOP_RETURN when the output time and the stop time are the same. …
adayton1 commented on issue LLNL/RAJA#1754.
I’m encountering this issue when I configure RAJA with ENABLE_CUDA=ON and ENABLE_OPENMP=ON and a downstream library (LEOS) with ENABLE_CUDA=ON and ENABLE_OPENMP=OFF (since LEOS only supports one backend at a time)….
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thanks @warrickball we have merged your changes. …
jedbrown commented on issue openjournals/joss-reviews#7716.
Hi, @jfe-dkrz.
trws commented on issue flux-framework/flux-core#6925.
Thanks for the tmate work, going to try that now that I have the source cleaned up. Also going to see if I can get the annotate failures script (or part of it) to go so we get the debug info more reliably….
grondo commented on issue flux-framework/flux-core#6925.
> It appears in my workflow output but doesn’t look like it executes. (gray checkbox, 0s elapsed, workflow continues on) …
samuelkgutierrez commented on issue hpc/quo-vadis#354.
> Question: I issued a git rebase --whitespace=fix HEAD before pushing into my repo and the whitespace test still failed. …
JustinPrivitera commented on issue visit-dav/visit#20504.
nope, nothing. I’ll pick this up again on monday….
tzanio commented on issue mfem/mfem#4957.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
trws commented on issue flux-framework/flux-sched#1384.
They are not meant to be, but some of them operate on timeouts and can have failures like that especially if the resource is overloaded. The first of those tests is on a ten second timeout, so I wouldn’t expect a spurious failure there unless either launching the subinstance is very delayed (say a very, very slow filesystem) or running too many tests in parallel. …
cjekel commented on issue cjekel/piecewise_linear_fit_py#131.
Thank you for this!…
v-dobrev commented on issue xsdk-project/xsdk-issues#277.
@balay, By the way, now that the Spack packages are in a separate repo, the xSDK pipelines may need to be updated to pull the latest spack/spack-packages repository, e.g. by calling spack repo update along with some additional info like spack repo list and some command(s) to show what commit of spack/spack-packages is being used….
tgamblin commented on issue spack/spack-packages#739.
Ok bailing on this as a back port – tried something less disruptive in #792, which should go into 2025.07.1 instead of this. …
cyrush commented on issue spack/spack-packages#744.
this looks great thank you @vicentebolea !…
grondo commented on issue flux-framework/pr-validator#7.
I just noticed that crate-ci/typos publishes pre-built binaries in their releases. …
grondo commented on issue flux-framework/flux-core#6908.
> I don’t know if it is relevant but I did find a single remaining case of using a hardcoded path to lua:…
wihobbs commented on issue flux-framework/flux-core#6906.
oops y’all saw nothing
trws commented on issue flux-framework/flux-core#6926.
Oh and it can definitely wait, there’s a heisenbug in fileref of all things on GHA right now that’s going to take a bit to work out. Thanks for looking it over!…
garlick commented on issue flux-framework/flux-core#6925.
Apologies, I made a comment on this stuff in the other PR that is based on top of it. Just that I’m traveling today and wanted to have a closer look at this before it goes in. Great work @trws….
cyrush commented on issue conda-forge/conduit-feedstock#52.
@ h-vetinari thank you for trying to solve this mystery. …
artv3 commented on issue LLNL/raja-suite-tutorial#67.
I think is good to now, @tdrwenski are you able to hit merge? or should I merge?…
jeremylt commented on issue Eudicods/outworlds-wastes#134.
In #135…
cyrush commented on issue AMReX-Codes/amrex#4572.
@ax3l our current ascent containers (0.9.4) are using CMake 3.28. The internal layout changed a bit, but I can address that. …
becker33 closed issue spack/spack#51019.
Specifying un-needed dependency turns on default-False variant
### Steps to reproduce…View Comment
hppritcha commented on issue open-mpi/ompi#13343.
this looks good to me…
wihobbs commented on issue flux-framework/flux-sched#1375.
Hi @BagchiS6, sorry we missed this issue, but I looked back and it seems like we made some progress on Slack a few days after you opened this….
grondo commented on issue flux-framework/flux-sched#1383.
Or the scheduling loop isn’t being triggered to consider the next pending job….
milroy commented on issue flux-framework/flux-sched#963.
We discussed this issue in the Fluxion group meeting today. I mentioned one more disadvantage of modeling a user (as opposed to an admin DAT) advanced reservation as a reservation rather than an allocation: re-reserving the reservation during each scheduling loop can cause the start time to change, necessitating feedback to the user submitting the request….
adrienbernede open issue LLNL/RAJAPerf#531.
Unused CMake arguments ?
https://github.com/LLNL/RAJAPerf/blob/cc6a8c5c554da8576f9c31143bfd8880e1c5f56c/azure-pipelines.yml#L51…View Comment
jedbrown commented on issue CEED/libCEED#1875.
This seems like only a naming convention thing, and like it would be possible to preserve deprecated stubs for the old names. That would ease migration at what looks to be minimal cost….
gardner48 commented on issue spack/spack-packages#496.
> I would also like to hear from at least one maintainer per package. …
vsoch commented on issue kubernetes-sigs/node-feature-discovery#2183.
Ping @adrianchiris @zvonkok @ArangoGutierrez @marquiz we’ve gone through several rounds of review and have LGTM from the compatibility working group. Can any of you review this?…
samuelkgutierrez commented on issue hpc/quo-vadis#349.
Done in #351. Thank you again for the reminder, @GuillaumeMercier….
cmoussa1 commented on issue flux-framework/flux-accounting#703.
thanks @jameshcorbett! Setting MWP here…
cyrush commented on issue Alpine-DAV/ascent#1585.
More build tweaks for mi300a:…
v-dobrev commented on issue spack/spack-packages#496.
@balay, can you please take a look at the petsc addition?
…
jandrej commented on issue mfem/mfem#4951.
Probably because the algoim integration in mfem is outdated and algoim doesn’t use versioning. ping @bslazarov …
v-dobrev commented on issue mfem/mfem#4827.
@mlstowell, I merge this PR in next since we have two approvals, however, you can still review it before merging into master if you want – it is a very small change….
chu11 commented on issue flux-framework/flux-core#6928.
thanks, tweaked the commit message a bit based on your suggestion and removed the setting of $jq in sharness….