JustinPrivitera commented on issue visit-dav/visit#20288.
I think it is any change made to it gets reverted after clicking apply….
markcmiller86 commented on issue visit-dav/visit#20288.
> Does it occur when you type to change the % or when you use the arrows?…
chu11 commented on issue flux-framework/flux-docs#295.
ooops created in wrong tab…
chu11 commented on issue flux-framework/flux-core#6694.
note for myself in current kvs-watch code…
cyrush commented on issue Alpine-DAV/ascent#1369.
resolved in https://github.com/Alpine-DAV/ascent/pull/1370…
jedbrown commented on issue openjournals/joss-reviews#7845.
Hi, @rreischke! At first glance, I’m not finding automated tests or community/contributing guidelines. Can you please check the review checklist and make sure you’ve hit all items before we recruit reviewers?…
grondo commented on issue flux-framework/flux-docs#294.
Thanks @wihobbs! I fixed your commit message nit and also a couple other issues preventing CI from passing and now I’ll set MWP….
wihobbs commented on issue flux-framework/flux-core#6691.
Thanks for taking the time to dissect all of this @grondo! It was getting difficult in the thick of this project to figure out what exactly to do next, so thanks for getting me going. I’ll note a few things: …
garlick commented on issue flux-framework/flux-coral2#310.
Thanks!…
adrienbernede commented on issue LLNL/radiuss-shared-ci#43.
markcmiller86 commented on issue LLNL/Silo#439.
The failing test is rocket.cxx. I just removed that test from the test suite. But apparently did so directly on 4.11RC. So, I am merging this to 4.11RC and then will cherry-pick to main….
jeremylt commented on issue CEED/libCEED#1762.
ugh, T_1D is wrong for this strategy. Pondering…
adrienbernede commented on issue spack/spack#49345.
Closed for security concerns….
hppritcha commented on issue open-mpi/ompi#13108.
closed via https://github.com/open-mpi/ompi/pull/13109…
cmoussa1 commented on issue flux-framework/flux-accounting#592.
Thanks @wihobbs! I added a small note about delete-* commands. Setting MWP here…
jameshcorbett commented on issue NearNodeFlash/NearNodeFlash.github.io#176.
Closing because it is a Flux issue….
artv3 commented on issue LLNL/RAJA#1760.
> > I like this idea: it makes permutations and layouts much easier to use. However, I think it should be called something like ViewMaker, or something else. When I see Reshape, I think that this will take an existing View I’ve already defined, and will alter its dimensions. Instead, this PR is creating a wrapper that returns a new View, and can be used to construct Views from scratch. I realize that ReShape is used in a similar way in MFEM as well, so if everyone else is on board with this then that’s fine with me too.
…
camierjs commented on issue mfem/mfem#3658.
> Is there a convenient way to modify T_MAX in the kernels?
…
jedbrown commented on issue coreylowman/cudarc#340.
I’m not sure what’s being proposed here, as memcpy_dtoh_async is unsafe, and any safe dtoh_copy_pinned would take a &mut PinnedHostSlice<T>. But it’s true that it’s necessary to prevent passing the same slice as destination for multiple copies (or as immutable arguments) on different streams….
cmoussa1 commented on issue LLNL/cmvl#8.
thanks! merging this…
jedbrown commented on issue EnzymeAD/Enzyme#2025.
Is it appropriate to add a test that pulls in rsmpi? That would require building a Rust toolchain using this Enzyme and then cargo test (which would build rsmpi, assuming an MPI implementation is available somewhere in the CI environment). I’m supposing that “soon”, people will be getting their Enzyme-enable Rust from rustup, and that will increase interest in support for differentiable rsmpi, etc.
…
jedbrown commented on issue CEED/libCEED#281.
That’s a different sort of test harness. It would be a good project to create it, but it’s going to be more intrusive than something we already know how to measure. Also, constants matter, so having a test that enforces “no algorithm in PETSc should exhibit $\omega(n \log^k n)$ complexity” ($\omega$ notation is an open lower bound) isn’t assurance that code is efficient. This issue is about something practical for performance, not that other thing (which would be useful, but not a replacement for this)….
jedbrown commented on issue openjournals/joss#1411.
I agree with @sneakers-the-rat, and would go further, that people should know (and thus be defensive toward) the use of Copilot and similar products can automate what would assuredly be copyright violation if a human did it. For example, it reproduces substantial portions of SuiteSparse (an LGPL library) with copyright stripped and only trivial cosmetic changes (article from 2022), and this question of whether slight obfuscation is defense against copyright infringement is the subject of ongoing class-action litigation. (Obfuscation is not a defense against copyright infringement by a human, which is why clean-room design exists.) Companies are banking on use of such products being sufficiently ubiquitous they can obtain a legislative solution if courts rule against them. They may be successful, and yet I object on ethical grounds. …
correaa commented on issue mull-project/mull#1090.
right, I think in my case it combines with the problem of having to run the whole ctest for each mutation (for some reason that I can’t understand completely, perhaps because my code it heavily templated and different test instantiate different functions)….
hppritcha commented on issue mpi-forum/mpi-issues#916.
I will see about reorganizing the chapter without using items….
tzanio commented on issue mfem/mfem#4730.
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….
brendankeith commented on issue mfem/mfem#4702.
@tzanio, should I remove the modified file ex40.cpp to prepare for merging? …
milroy commented on issue flux-framework/flux-sched#1353.
Thanks for the review and feedback @jameshcorbett and @trws! Setting MWP….
alecbcs commented on issue flux-framework/flux-core#6667.
From the user perspective, I think any of these proposed solutions work for me as long as we can enforce exclusivity between users at the admin level to meet the security requirements to turn this on. (E.g. so users can’t disable the policy and get on a node with another user.)…
tpatki commented on issue flux-framework/PerfFlowAspect#182.
- @loudsun1997 : Please run the check-c-code-format.sh in the scripts directory to format the code.
…
jandrej commented on issue mfem/mfem#4730.
@mfem/editors whoever picks this up, this can be considered reviewed and ready to merge as long as CI passes. …
wihobbs commented on issue flux-framework/rfc#450.
I’m livid that the Oxford English Dictionary is behind a paywall now
trws commented on issue flux-framework/flux-core#6667.
That’s really good to know. If we go with the property approach, that should be pretty trivial (as would arbitrary other ones) if we plan it as a more general mechanism. Our label implementation is a generic string=>string map, and we can do matches on labels and their values, so that would be a pretty good way to start (I could see reasons not to do it that way longer term but I think it would get us pretty far). Maybe we could add something to jobspec or in the generic attributes for “ephemeral labels” that are associated with a job and apply to a resource or the node? Then it’s just two additions to fluxion:…
milroy commented on issue flux-framework/flux-sched#1352.
@trws can you give this a look when you get a chance? I think it’s near merge-able but it would be good to get your feedback….
grondo commented on issue flux-framework/flux-core#6670.
Thanks! setting MWP….
garlick commented on issue flux-framework/flux-core#6616.
Well, I’ll go ahead and set MWP here to keep things moving along. Thanks!…
markcmiller86 commented on issue visit-dav/visit#20263.
By the way, the “current solution” I refer to is available only via adjusting plugin code. That is problematic in two ways. First, it doesn’t help if a user just happens to open a database of a given plugin type with a baziilion variables. Second, every plugin needs to be adjusted in the same way to handle the bzillion variable cases….
jedbrown commented on issue rsmpi/rsmpi#204.
This would be great to support, especially since MSMPI appears to be unmaintained. I don’t have a test environment, but it should be doable. You might need to start with build-probe-mpi to detect Intel MPI….
jandrej commented on issue mfem/mfem#4728.
If you want to keep this branch as a testbed, sure….
vsoch commented on issue flux-framework/spack#303.
This will be closed by https://github.com/spack/spack/pull/49230…
grondo commented on issue flux-framework/flux-sched#1347.
Thanks! I’ll set MWP….
trws commented on issue flux-framework/flux-core#6667.
It depends on what you mean by policy in this context. We don’t support different match policies, currently, but we do support different queueing policies. We’ve given some thought to how to do something like this in the past, because it would be an efficiently improvement certainly. It ended up on the back burner largely because we get a somewhat similar effect by running a user-level flux instance that isn’t node-exclusive under the node-exclusive system scheduler. This would let us be much more flexible though….
chu11 commented on issue flux-framework/flux-core#6581.
saw this today …
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2225.
@rinkug as per phone conversation, I followed your lead and adjusted example to encryption and softened ending remarks a tad. I think I’ve captured the spirit of what you asked. Have a look….
jeremylt commented on issue CEED/libCEED#1734.
I think we want to merge this before the review so we can have libCEED 0.13 and Ratel 0.4 with this? Is the question of the sync call the big blocker right now?…
vsoch commented on issue singularityhub/shpc-registry#301.
I think we would want to make sure that the path is derived as simply the digest with sif. If you want to open a PR to work on it I’d be happy to review….
jedbrown commented on issue openjournals/joss-reviews#7484.
@editorialbot recommend-accept…
tzanio commented on issue mfem/mfem#4452.
Re-merged in next for testing……
garlick commented on issue flux-framework/flux-security#203.
Also need reapproval on this one :-)…
trws commented on issue flux-framework/flux-sched#1348.
Looks like it, I’ll reopen then….
jameshcorbett commented on issue flux-framework/flux-sched#1348.
> Bah, just noticed you added a commit for this to #1346, I’ll close this one. …
wihobbs commented on issue flux-framework/flux-core#6667.
Ah, my apologies @alecbcs, I might have misled you yesterday – I thought we supported different match policies per queue. Although, as @grondo said, it might be possible to overcome that limitation if the only policy difference between the ci queue and batch or debug is node exclusivity….
jameshcorbett commented on issue flux-framework/flux-coral2#298.
Thanks for checking on our standard formats! I would never have thought of that….
wihobbs commented on issue flux-framework/flux-coral2#298.
Thanks @jameshcorbett!…
cyrush commented on issue LLNL/conduit#1377.
Connectivity is local for each processor, actually local to each domain….
JustinPrivitera commented on issue LLNL/conduit#1377.
Hello @uturuncoglu! We provide example meshes within the Blueprint library. You can see how to create them and view examples with labeled renders for unstructured meshes in our docs (for triangles) (for quads)….
trws commented on issue spack/spack#46416.
I’m not really fond of this as an idea, and especially not as a name for the variant. It’s building LLVM the package, but doesn’t provide LLVM, which prevents the use of LLVM in the same package DAG if this variant is selected. It would be less disruptive if it were a separate package, though I do understand that’s a harder rework. If this needs to go in this way, please make the variant self-explanatory. The current name standalone implies to me that it builds a standalone LLVM (I would expect that to mean self-hosted on the llvm libc or similar). Perhaps openmp_libraries_only?…
hppritcha commented on issue open-mpi/ompi#12226.
@jsquyres don’t forget to add this to your queue
chakshinglee commented on issue mfem/mfem#4704.
yes, changing the curvature works…
camierjs commented on issue mfem/mfem#4723.
> * I am trying to recreate the results seen in this paper, but I’m having issues running the benchmarks in bench_ceed.cpp. In particular, in BP2/1, I get an assertion error …
wihobbs commented on issue flux-framework/rfc#448.
> Sometimes it is impossible to restrict a commit message title to 50 characters, I think the pr-validator does set a hard limit at some point (I think it is 70 characters). That is why this one’s a SHOULD not a MUST. Same with the body line length….
cmoussa1 commented on issue flux-framework/flux-accounting#498.
I’m going to close this in favor of re-opening a more specific in the future if it pops up again. I’ve just made some changes in #590 to add more specific error messages to jobspec-updates in the case where an exception is raised….
hppritcha commented on issue mpi-forum/mpi-issues#892.
I am also going to leave the split example problem for any final changes to the doc since the break may move if we do additional edits….
brendankeith commented on issue mfem/mfem#4460.
> @brendankeith, a friendly reminder to please take another look! …
gardner48 commented on issue erf-model/ERF#2143.
@skang67 this will fix your build issues with FFTW on Mac…
garlick commented on issue flux-framework/flux-coral2#283.
Thanks! Setting MWP….
cjekel commented on issue cjekel/piecewise_linear_fit_py#126.
Thanks for the report! I believe i just fixed this, but let me know if you still have issues!…
alecbcs commented on issue spack/spack#48507.
@nhanford looks like the mixins have been added. If you’ve got a moment, it’d be great to get a final review….
vsoch commented on issue rootless-containers/usernetes#242.
I got this fully working in rootless mode - I’ll put together a quick writeup soon….
tzanio commented on issue mfem/mfem#4689.
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….
acfisher commented on issue mfem/mfem#4689.
> The CI is failing, @acfisher can you take a look? …
milroy commented on issue flux-framework/flux-sched#1337.
After further thought, I don’t think the above observation is related to the incident on tioga. In the off chance it is I’ll leave the comment posted….
grondo commented on issue flux-framework/flux-sched#1219.
Oh, wow! Thanks for that extra detail @trws!…
trws commented on issue flux-framework/flux-sched#1345.
I’m surprised to say I’m on the fence. If we can do it with existing facilities and without having to add a new state to track, that would be nice, and it sounds like the partial cancel plus remove should do that. That said, it seems like the lost state might be useful for long-term down nodes. Maybe worth getting some feedback from Ryan or someone on if that would be a thing admins would want?…
garlick commented on issue flux-framework/flux-core#6656.
OK I did the latter quick test and verified that sending STOPPING=1 transitions the unit state to “deactivating”….
vsoch commented on issue converged-computing/flux-usernetes#21.
Note that we will need to update GKE to use n1-standard-8 with 1 GPU node, I just don’t see that we are going to get enough capacity of GPU to get 4/node….
trws commented on issue spack/spack#49065.
I would be ok with this, but is there a reason not to add it back to the base? Does any other package use the result?…
jedbrown commented on issue openjournals/joss-reviews#7484.
@editorialbot generate pdf…
hppritcha commented on issue open-mpi/ompi#13076.
its a problem in the prte data server code. looking at a way to best address the issue….
v-dobrev commented on issue mfem/mfem#4403.
This issue is addressed in #4691. Please use that PR for further discussion….
cjvogl commented on issue mfem/mfem#4673.
@nmnobre , thanks for the improvements! I’ve added some finishing touches to the defaults and documentation… @jandrej and @helloworld922 , I believe this PR is ready for another look….
wihobbs commented on issue flux-framework/flux-core#6655.
> It is interesting we don’t see this on other clusters….
grondo commented on issue flux-framework/flux-coral2#285.
Conceptually, it seems cleanest to set the aux flag at the same time the prolog-start event is emitted. Perhaps it would be possible to add an interface to jobtap to allow a plugin to query if there is an outstanding prolog-start or epilog-start event with a given description if that would be easiest in this case….
jameshcorbett commented on issue flux-framework/flux-coral2#285.
After discussing in today’s coffee call, it sounds like setting the aux in the RUN callback should be satisfactory for now. However, in the future there may be improvements that let the plugin more easily determine whether its prolog is still running….
vsoch commented on issue NVIDIA/nvidia-container-toolkit#85.
@elezar is CDI supported for Docker 28.0.0 now? I am having this specific issue (where I can’t use no-cgroups) and would like to test CDI - my setup is using rootless docker and docker compose….
bgunnar5 commented on issue LLNL/merlin#494.
Updated docs for this PR can be found here: https://merlin–494.org.readthedocs.build/en/494/. The new monitor stuff (including a flow chart) is at: https://merlin–494.org.readthedocs.build/en/494/user_guide/monitoring/monitor_for_allocation/. Additionally, documentation on the new merlin database command can be found here: https://merlin–494.org.readthedocs.build/en/494/user_guide/monitoring/database/.
…
cyrush commented on issue LLNL/conduit#1364.
@mlohry yes, I can setup HDF5 to build with mpi support. …
markcmiller86 commented on issue LLNL/H5Z-ZFP#153.
> Actually, let me make one more point about inner vs. outer dimensions. It would be possible to always use x, y, … as the domain dimensions, even when the fastest varying dimensions are part of the range. This is because zfp supports strided layouts, where the leftmost dimension does not necessarily vary fastest. So instead of a default stride of 1 for x and 4 for y in the tensor field example above, you could set them to 6 and 24, respectively, with the strides for z and w set to 1 and 2. There would possibly be a performance penalty, however, as zfp loops over x in the innermost loop, and the data would be gathered and scattered non-contiguously. …
markcmiller86 commented on issue visit-dav/visit#20239.
What troubles me most about experiences like this is that I have an uneasy feeling it is often attributable to changes in parts of the system and its software that are a) wholly outside our control and worse, b) not fully tested or made known to us or the users….
v-dobrev commented on issue spack/spack#47642.
ping: @tgamblin, what should be done about the llvm failures caused by other PRs, see https://github.com/spack/spack/pull/47642#issuecomment-2585888138, that are blocking this PR?…
artv3 commented on issue mfem/mfem#4571.
Can we do 1 more update with the base branch? …
chu11 commented on issue chu11/freeipmi-mirror#75.
redhat just sent me a patch to fix this issue and it has been pushed to master & the 1.6.0 stable branch. PLMK if you think a release is urgent/necessary….
balos1 commented on issue LLNL/sundials#49.
Thanks for the reproducer @dweindl. I think adding a check to gracefully fail with an error message is reasonable. …
adrienbernede commented on issue LLNL/radiuss-spack-configs#122.
@tdrwenski This basically leverage my work in radiuss-spack-testing to implement a CI for radiuss-spack-configs. …
corbett5 commented on issue GEOS-DEV/GEOS#3538.
@sframba @rrsettgast I just ran testArray1D.cpp through memcheck on develop and there are no leaks. Even when I change from using my TestString object which provides a convenient wrapper to using std::string directly there are no leaks. And this is without using the suppression file to boot. testArray1DOfArray1D is similarly clean….
cyrush commented on issue Alpine-DAV/ascent#1463.
thank you, I fixed this for the cmake case, but missed the config.mk case! …
tpatki commented on issue ovis-hpc/ldms#1447.
Hi @morrone Noted the above, we will take a look in early March once @slabasan and I are back in office, update and then and re-open the PR like you suggested….
hppritcha commented on issue openpmix/prrte#2139.
are you using –enable-debug option? …
samuelkgutierrez commented on issue hpc/quo-vadis#308.
Fixed by #309….
garlick commented on issue flux-framework/flux-security#200.
@mergifyio requeue…
chu11 commented on issue flux-framework/flux-core#6581.
Some small set of transaction stats …
cmoussa1 commented on issue flux-framework/flux-accounting#587.
Fixed now by #588….
wihobbs commented on issue codecov/codecov-action#1599.
Why was this closed? Is it fixed?…
garlick commented on issue chaos/genders#70.
> This is more brainstorming for next generation genders if there is a 100k cluster out there in the future with 1000s of attrs….
tpatki commented on issue LLNL/variorum#579.
Hi @morrone Thanks for opening this issue on the Variorum end. I am on leave this week, and @slabasan is out until March 7, but we will take a look as soon as we can. Will update and open a PR on the LDMS end once we address this. Thank you for reviewing and helping!…
bgunnar5 commented on issue LLNL/merlin-spellbook#45.
closed this PR as it was implemented in https://github.com/LLNL/merlin-spellbook/pull/46…
jeremylt commented on issue CEED/libCEED#1758.
GPU-shared doesn’t have its own operators. GPU-ref holds the operator implementation. The GPU-ref operators and GPU-gen operators don’t really play nice together so I don’t think we’ll be able to easily launch both at the same time with how the code is currently designed….
cyrush commented on issue visit-dav/visit#20176.
@markcmiller86 at the highest level expressions can use MPI, however in our convenience class hierarchy that implements many of them we limit those paths when we provide an interface to in terms of a only single domain or a part of a single domain (like a data array). Justin already had to plumb some info, we may need to peel back a bit higher. …
garlick commented on issue flux-framework/rfc#447.
> This would allow a scheduler that doesn’t yet support shrink to continue to work with the updated protocol …
brendankeith commented on issue METHODS-Group/ProximalGalerkin#11.
Some comments: …
adrienbernede commented on issue mfem/mfem#4684.
@v-dobrev as I said before, I think the handling of the --spec option in the tests/gitlab/build_and_test script was lacking robustness: It not trivial in bash to ensure that a “quoted string with white-spaces” is treated in a consistent way through several layer of scripts and/or variable resolutions.
…
IanLee1521 commented on issue elastic/kibana#198887.
Thank you!! This has been bugging me for a while!
markcmiller86 commented on issue visit-dav/visit#20176.
The more I think about it…isn’t a Query most of the time just an Expression + Parallel Reduction?…
JustinPrivitera commented on issue visit-dav/visit#20176.
The global mesh expressions are incorrectly calculated on a per-domain basis. Fixing this will require some retooling of the expression infrastructure….
milroy commented on issue flux-framework/flux-sched#1342.
> The RV1 test that’s blowing up is t3027-resource-RV.t, but it looks like it might be an out of range error …
grondo commented on issue flux-framework/flux-core#6648.
This issue is quite obvious, and I think it’s been present in the code for quite awhile….
chu11 commented on issue flux-framework/flux-core#6648.
I can’t help but think something like https://github.com/flux-framework/flux-core/issues/2547 or https://github.com/flux-framework/flux-core/pull/6210 is in play here. But that should all be shored up….
vsoch commented on issue eksctl-io/eksctl#8220.
Sounds good - thanks @bryantbiggs ! And it could be this allocation was special for us (I don’t know the details) and this issue is extremely unlikely to happen for anyone else. If that’s the case, then the custom build is probably our best bet, and we can pray to the GPU gods that circumstances change in the future to make them easier to get! :laughing: …
chu11 commented on issue chaos/genders#70.
hmmmm code review in _attr_node_processing(), there seems to be a duplicate check…
garlick commented on issue chaos/genders#70.
If there’s really a hash per node, that seems like a problem!…
cyrush commented on issue visit-dav/visit#20238.
One idea: …
garlick commented on issue flux-framework/flux-coral2#278.
flux-getthewabbit(1)?…
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2251.
@vijaysm and @iulian787 I have started writing and will plod along here periodically. My practice is typically to write often a lot more than necessary and then pare down to get to the final product. So, don’t worry too much about length at this point. We can adjust to ensure we emphasize the desired items and don’t over saturate with irrelevant details….
cyrush commented on issue Alpine-DAV/ascent#1360.
This should be another case in the transform filter….
trws commented on issue flux-framework/rfc#444.
I agree, this looks good to me. If we decide to also allow an idset as a count, we’ll need to adjust it, but I would expect that to be mutually exclusive with these anyway so they’re pretty orthogonal. Thanks for the contribution @sam-maloney!…
IanLee1521 commented on issue elastic/package-registry#1270.
I cam upon this issue after finding that the https://www.docker.elastic.co/r/package-registry/distribution isn’t uploaded yet for 8.17.2 (AFAICT) where other parts of the stack appear to be published….
adrienbernede commented on issue LLNL/uberenv#138.
I don’t need the PR anymore, and pushed a tag for reference of the commit if necessary….
jedbrown commented on issue CEED/libCEED#1750.
Looks good. We can update CUDA on Noether too (Debian testing is currently on cuda-12.2). Should do that in the next few months regardless….
hppritcha commented on issue open-mpi/ompi#13086.
i’ll add man pages to this PR….
acfisher commented on issue mfem/mfem#4689.
> The CI is failing, @acfisher can you take a look? …
tzanio commented on issue mfem/mfem#4675.
Merged in next for testing……
jandrej commented on issue mfem/mfem#4483.
Added changes requested in comments and meeting. I did not change the norm computation as I’m not confident to know how different solvers are supposed to report their norms to be consistent. Can someone else propose those changes please?…
tomstitt commented on issue Olympus-HPC/proteus#152.
> Good catch! I wonder if we should use regex for string matching …
cjvogl commented on issue LLNL/sundials#664.
@gardner48 , apologies that I keep forgetting to branch off develop instead of main …
v-dobrev commented on issue mfem/mfem#4690.
This change is already in master, so I’m not sure what the goal of the PR is because it also targets master.
…
alecbcs commented on issue spack/spack-gantry#132.
This PR looks good to me. Do we need to store the cost of the node in the job? Or could we store the cost of the node in the node object and then calculate that later on for jobs based on their duration?…
alecbcs commented on issue spack/spack#48955.
These look good to me! Thanks @paulbry!…
cjvogl commented on issue mfem/mfem#4687.
It’s worthwhile to note that the weak formulation (before imposing any boundary conditions) on $\xi \in (0,L)$ is…
trws commented on issue flux-framework/flux-sched#1335.
Oh god, of course it does, because of the upwalk.
artv3 commented on issue LLNL/RAJA#1789.
This PR has a large number of small changes, notably the date changes. I thought dates had been previously updated?…
v-dobrev commented on issue mfem/mfem-uberenv#2.
@adrienbernede, I think this is ready to be merged. If you agree, go ahead and do it. Thanks! …
tzanio commented on issue mfem/mfem#4690.
Thanks for working on this @Chris00 and @mkovaxx! …
tgamblin commented on issue hpsfoundation/tac#29.
Needs DCO…
vsoch commented on issue flux-framework/flux-framework.github.io#139.
Nice job flux-sched developers! :partying_face: …
correaa commented on issue compiler-explorer/compiler-explorer#7316.
I don’t understand, this used to work well in the recent past. …
tzanio commented on issue GLVis/glvis#325.
Time navigation will be a useful extension, but it will require new infrastructure and significantly more memory than the current approach (which essentially holds a single instance at a time). …
v-dobrev commented on issue GLVis/glvis#325.
The data from previous times steps is discarded when a new time step is received, so going back is not supported at this point….
JustinPrivitera commented on issue visit-dav/visit#20220.
I don’t have PR approval powers since I created it, but I approve of all the changes in it….
markcmiller86 commented on issue visit-dav/visit#1532.
We should do this. I am taking out of review to discuss. Its almost as simply as adding just another option to the Label display format though I might opt to rephrase that to label type or maybe Label display mode….
brendankeith commented on issue mfem/mfem#4670.
> There are number of differences in our nightly autotest runs from this PR. …
vsoch commented on issue hpc-social/jobs#28.
It should be OK to run locally - the Google sheet that is used for the update is grabbed progammatically via a csv url from Google sheets. What you might want to do is make a copy of the current _data/jobs.yaml in case you want to restore or redo tests. Although the script (if I remember) will generate a previous jobs file in the same directory. The update will write to tmp and then use that file to replace the current jobs, so it will update….
garlick commented on issue flux-framework/flux-core#6616.
> (I guess one thought is that as state is able to be recaptured, this would reduce the list of things that prevent a broker from joining sdmod.idle)
…
grondo commented on issue flux-framework/flux-core#6614.
Thanks! I’ve rebased and fixed that typo and will set MWP now….
jedbrown commented on issue rsmpi/rsmpi#197.
@oxcrow Try building with `RUSTFLAGS=’-C panic=abort’ (or put it in your profile). If that works like the C program, then this is the issue. Panics generally interact poorly with collective semantics so it’s best to minimize when that can happen….
vsoch commented on issue pydicom/deid#240.
Thanks for the update!…
hppritcha commented on issue open-mpi/ompi#13086.
related to #8057 …
grondo commented on issue flux-framework/rfc#442.
Good idea! I think @trws should comment since this would impact Fluxion mostly….
jameshcorbett commented on issue flux-framework/flux-sched#1332.
Thanks! force-pushed to add some additional tests, will set MWP….
chu11 commented on issue flux-framework/flux-core#6615.
rebased now that #6604 is in…
trws commented on issue flux-framework/flux-core#6575.
> Looks like that did it, though I can’t think of why?? …
markcmiller86 commented on issue HDFGroup/hdf5#5290.
To summarize then……
emily-howell commented on issue Alpine-DAV/ascent#1442.
@cyrush I circled back to this and updated the MPI thresholding to be Debug for rank 0 and Warnings for all other ranks. Does that work?…
tgamblin commented on issue spack/spack#48893.
If they define it already it would not matter – they could still use it….
correaa commented on issue mull-project/mull#1085.
It doesn’t seem to be deterministic even. I ran the CI again an it passed this time….
v-dobrev commented on issue mfem/mfem-uberenv#3.
@adrienbernede, I think I addressed all your suggestions. Let me know if I missed something. Thanks!…
wihobbs commented on issue flux-framework/flux-core#6609.
> It’s easy enough to relax the checks that occur during broker wireup. It will be a little more challenging to figure out how to check this in CI….
cmoussa1 commented on issue flux-framework/flux-accounting#579.
I’ll go ahead and leave them there, then. For the sake of solidarity
JustinPrivitera commented on issue LLNL/conduit#1151.
For future generations:…
cmoussa1 commented on issue LLNL/cmvl#6.
We talked about this offline, but there might be a way for the logging script to handle this in case of a failure. Since the script needs to establish a connection to the main Flux broker via handle object, if this fails, the script should append a failure message in a separate log file. Perhaps something like the following logic:…
vsoch commented on issue GoogleCloudPlatform/ramble#869.
I’m also wondering if we can support a generic application executor. For example, I don’t need to hard code every specific argument for some application, I could just define a container, an input line with parameters, and then use ramble to programatically (and easily) run a lot of experiments without having to write an application.py for each. It’s a more flexible approach that is less prone to not having, for example, an updated download of data, and it would allow for using ramble to reproduce experiments without needing to write a lot of those complex files. What do you think?…
cyrush commented on issue visit-dav/visit#20059.
We may be able to create auto scroll bars when things are off screen…
bhatele commented on issue NVIDIA/nccl#1594.
Also, if there are two available options, how does NCCL choose which one to actually use?…
adrienbernede commented on issue LLNL/uberenv#138.
@chapman39 I need for these branch to exist a little longer. …
trws commented on issue spack/spack#48865.
The TargetConditionals.h include is hidden behind including sanitizer_platform.h, which pulls it in as long as __APPLE__ is defined. I’m really not sure how this could happen, especially with the apple clang used to do the compile, maybe somehow the target triple isn’t what it should be?…
alecbcs commented on issue spack/spack#48507.
@MatthewLieber thanks for the quick turn around! This PR looks good to me and I’ve asked @nhanford to quick take a look since he’s more familiar with how we package mvapich….
milroy commented on issue flux-framework/flux-sched#1322.
Thanks for the reviews @jameshcorbett and @trws! Setting MWP….
trws commented on issue flux-framework/flux-sched#1322.
Just glanced again, and the generic predicates rather than the explicit pair look better to me.
wihobbs commented on issue flux-framework/flux-sched#1268.
Thanks @trws!…
alecbcs commented on issue eldertek/duplicatefinder#133.
I can confirm I’m also seeing the same issue as @Ludo-code in my Nextcloud logs….
brendankeith commented on issue mfem/web#308.
@tzanio I will get back to https://github.com/mfem/web/pull/305 (as well as https://github.com/mfem/mfem/pull/4460) at the end of the week…
tpatki commented on issue flux-framework/flux-multi-cluster-utilities#4.
Duplicate issue (see #6)….
v-dobrev commented on issue mfem/mfem#4651.
Merged in next for testing……
JustinPrivitera commented on issue visit-dav/visit#17269.
I’m glad you got up and running! It’s not super easy to get set up to develop VisIt….
tgamblin commented on issue spack/spack#48689.
> Yeah, it was a bit ad-hoc. But this PR is a very similar issue, so all I’m asking is consistent behavior :) …
hppritcha commented on issue open-mpi/ompi#13073.
This patch isn’t going to work. The problem is by the time the ompi_mpi_instance_cleanup_pml is invoked the PML framework has already been closed. For some reason this happens to work for OB1 PML. Not sure exactly why that is yet. …
tzanio commented on issue mfem/mfem#4670.
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….
garlick commented on issue flux-framework/flux-core#6596.
Looking at logs, this job (f2DBk4ARinuV) ran Jan31 18:56 thru Jan31 19:10 on rzadams[1028,1030] and was successful….
milroy commented on issue flux-framework/flux-core#6596.
The "final": false value triggers the partial cancel logic (https://github.com/flux-framework/flux-sched/blob/471da50d167bdaa12a3dc1429c5b6b35674f1aa5/qmanager/policies/base/queue_policy_base.hpp#L649-L658): …
correaa commented on issue compiler-explorer/compiler-explorer#7316.
That is right, when I remove the nvc++ compilers the libraries can be added again. It used to work well until recently….
jedbrown commented on issue rsmpi/rsmpi#202.
Ah, fair point. What about writing this (in the top-level application) and letting feature unification apply it to what’s used by rsmpi? …
vsoch commented on issue pydicom/deid#268.
Thank you for this work @sammaxwellxyz - I have finished up the PR in #273 with a slightly different design to import the pydicom.dcrmread in one place that can be edited in the future. Please let me know if you run into any issues, and have a good weekend!…
astrophpeter commented on issue openjournals/joss-reviews#7699.
@editorialbot generate pdf…
vsoch commented on issue flux-framework/fluxion-go#16.
OK I went ahead and built with the grow-api branch, and made sure I had a fresh (not used yet) graph, and first I’m testing creating an allocation and then issuing a partial cancel with the entire graph (which I think should work)? …
wihobbs commented on issue flux-framework/flux-sched#1268.
Thanks @cmoussa1! Fixed and pushed….
chu11 commented on issue flux-framework/flux-core#6255.
> So what happens if you call flux_kvs_commit(FLUX_KVS_SYNC) or kvs_checkpoint_commit() without a backing store? …
grondo commented on issue flux-framework/flux-core#6593.
Thanks for your attention to detail on this one @chu11. I’ll set MWP here now….
cmoussa1 commented on issue flux-framework/flux-accounting#568.
OK, I believe I have a working test file now to reproduce the behavior I was describing in #575, so I’ve dropped [WIP] from that commit….
grondo commented on issue flux-framework/flux-accounting#568.
You can take a look at t3200-instance-restart.t which uses a series of flux start commands to simulate restarts.
…
corbett5 commented on issue GEOS-DEV/GEOS#3529.
> …
jeremylt commented on issue CEED/libCEED#1738.
Confirmed this fixed it…
markcmiller86 commented on issue visit-dav/visit#20209.
Ok, yes, I had two issues that. testing revealed and I fixed. I wasn’t actually calling the lambdas in the loop due to missing () and I wasn’t indexing the cases correctly. Working now….
alecbcs commented on issue spack/spack#48507.
I’m still seeing a few of the typos pointed out on the latest commit….
adayton1 commented on issue LLNL/camp#165.
I would add a comment in LLVM_LICENSE saying that derivative works from LLVM have the LLVM license info in the specific file or something like that….
trws commented on issue LLNL/camp#163.
We may have a file that we redistribute from LLVM that has that, but the SPDX identifier in all the files I wrote should be BSD-3-Clause. I don’t mind the format, though It’s more verbose than we need. Will help get this straightened out if I can. Really sorry I didn’t get to this sooner, strep completely laid me flat last week….
cyrush commented on issue visit-dav/visit#20205.
Thanks for these fixes @markcmiller86 !…
jandrej commented on issue mfem/mfem#4483.
Please review the current changes and refrain from commits/merges that aren’t moderated….
grondo commented on issue grondo/edac-utils#19.
We’ll there’s this, but my guess is that all of this stuff is woefully out of date. I think the kernel edac module(s) has long been replaced by MCE events and mcelog. …
grondo commented on issue flux-framework/flux-coral2#256.
Just discussed this issue a bit offline with @garlick and here’s a summary of our conclusions:…
jameshcorbett commented on issue flux-framework/flux-coral2#256.
> > Are these secrets going to be the same for every job, or will a new secret be created per job, reducing the impact of a potential compromise?…
garlick commented on issue chaos/diod#133.
Thanks!…
gardner48 commented on issue LLNL/sundials#657.
> Any reason we can’t remove limits like this: …
garlick commented on issue flux-framework/rfc#440.
OK, I’ll go ahead and set MWP then!…
grondo commented on issue flux-framework/rfc#440.
> Any objections? …
wihobbs commented on issue flux-framework/flux-docs#292.
Done. Thanks @jameshcorbett! …
balos1 commented on issue LLNL/sundials#652.
>It looks like as part of the refactoring around logging/error handling, the option of being able to specify a function to control where logs go has disappeared….
adayton1 commented on issue LLNL/Umpire#927.
> @adayton1 did you want to test drive this PR somewhere? …
jedbrown commented on issue CEED/libCEED#1735.
Cool! Any preliminary performance comparison?…
trws commented on issue flux-framework/flux-core#6575.
That might be reasonable, especially since it
jedbrown commented on issue openjournals/joss-reviews#7716.
Hi, @dostuffthatmatters!
jameshcorbett commented on issue flux-framework/flux-docs#292.
Can you merge it manually @wihobbs ?…
wihobbs commented on issue flux-framework/flux-docs#292.
This’ll need a rebase but then it’s good to go!…
grondo commented on issue flux-framework/flux-docs#293.
Maybe use commit title: github: add –ignore-installed to pip install of requirements?
…
grondo commented on issue flux-framework/flux-core#6575.
I should try to reproduce locally as well. Maybe the answer is to disable the faketime tests where they’re known to be flaky. I admit I’ve looked into these issues before and didn’t get much further than baffled as well….
alecbcs commented on issue spack/spack#48507.
Nope! I periodically got through and rename PRs to help those of us on the core team better understand what they’re adding, updating, etc… …
jeremylt commented on issue freeCodeCamp/fcc-express-bground-pkg#14.
Please put the full checklist back and follow it…
trws commented on issue flux-framework/flux-sched#1328.
Ok, I think this is all cleaned up. Setting MWP….
trws commented on issue flux-framework/flux-core#6575.
@garlick I might need some of your expertise on the cron/faketime issue. Somehow I’m getting correct faketime for everything but flux cron tab in the cron test, but if I try to use faketime the executable in that spot to check it, it correctly reports being in a faketime environment. I have a vague recollection we had some issues with it before, but it’s striking me as absolutely baffling. Might just be my brains still not all here, but I’m lost. Still looking into bookworm.
…
chu11 commented on issue flux-framework/flux-core#6573.
@grondo yup, i think that’s a bit more clear saying “only do this”…
garlick commented on issue flux-framework/flux-core#6576.
I haven’t looked at the specific error you are talking about yet but FYI, shell logs are mixed in with stdout and stderr in the output eventlog that you can access directly with e.g….
garlick commented on issue flux-framework/flux-coral2#222.
Sorry that was more painful than expected!…
grondo commented on issue flux-framework/flux-accounting#565.
FYI @cmoussa1 - you may find flux.util.parse_datetime() useful here….
jedbrown commented on issue Tehforsch/diman#87.
cargo test --doc is still ICE’ing on Diman main (46ff2201d8ac764e0e6215e064eb9bf722491bad) with today’s nightly. Do you also see the same? Have you done any reduction or know if there is already an upstream issue?
…
adrienbernede commented on issue LLNL/RAJAPerf#496.
@rhornung67 That worked seamlessly!…
adrienbernede commented on issue spack/spack#48689.
> Notice there is already a flag --keep-relative so look for consistent behavior with that?
…
tzanio commented on issue mfem/mfem#4673.
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….
samuelkgutierrez commented on issue hpc/quo-vadis#73.
I’m writing down a half-baked idea as a note to myself, so I don’t forget:…
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2225.
@betterscientificsoftware/bssw-maint this is ready to publish. That said, it does currently include the inclusivity metadata and I will expect anyone carrying it forward to replace that as appropriate….
JustinPrivitera commented on issue LLNL/conduit#1322.
I am adding the set methods in my device support branch: https://github.com/LLNL/conduit/tree/task/JustinPrivitera/05_01_24/device_support…
corbett5 commented on issue GEOS-DEV/LvArray#342.
I’m not familiar with Kokkos. but no there really isn’t any unordered map or set in LvArray. If this data doesn’t need to go on the GPU then just use the standard libraries implementation,…
JustinPrivitera commented on issue visit-dav/visit#5770.
Thanks @BenWibking for working on this. We really appreciate your efforts! We will do what we can to help you get your database reader into VisIt when you think it is ready….
cjvogl commented on issue mfem/mfem#4673.
I implemented the ExternalProject approach, which should isolated the TPL builds from the MFEM build (namely, not bringing all the TPL cache variables into the MFEM build). Even though a corresponding library target has to be created (along with the any interface include directories), I think this approach is cleaner. Thanks @jandrej for the input!…
v-dobrev commented on issue mfem/mfem#4630.
This branch has the changes in #4484 but it is not based on that branch. Since #4484 is already merged in next, we should probably merge both PRs together in both next and in master….
jandrej commented on issue mfem/mfem#4673.
>For example, all the cache variables set by hypre’s CMake appear in the MFEM build directory cache (although maybe this isn’t what you are concerned about?). …
jeremylt commented on issue freeCodeCamp/freeCodeCamp#58303.
I agree…
wihobbs commented on issue flux-framework/flux-sched#1268.
Once the automated tests pass I’ll drop draft from this PR….
adrienbernede commented on issue LLNL/uberenv#138.
@becker33 Please let us know when the fix is pushed. I’ll be happy to use the following snapshot release….
jedbrown commented on issue spack/spack#48668.
Thanks! These variants are fine. We test the libxsmm and magma variants in our CI, but I don’t think it’s important for E4S to add those. We also run on Intel GPUs and are setting up CI for it now (will add the variant to Spack with our next release)….
hppritcha commented on issue spack/spack#44654.
I think the problem with checking for FI_HMEM_ROCR is that this AMD specific flavor of cap was introduced recently into libfabric. But since your proposed change already uses the slingshot_network I’m okay with the PR now. Maybe replace the is_CrayXE with a method which just checks if fi_info –caps FI_HMEM_ROCR returns at least one provider?
…
jedbrown commented on issue rsmpi/rsmpi#200.
Do we need to bump dependencies or is it safe to say people will either get the old version or the latest version?…
hppritcha commented on issue ofiwg/libfabric#10712.
yes please open an issue and I’ll assign it to the author of 12290….
vladotomov commented on issue mfem/mfem#4671.
Hi @chunyu79,…
wihobbs commented on issue flux-framework/flux-core#6392.
This came up again today; apparently, some errors show up under flux dmesg as just:…
jameshcorbett commented on issue flux-framework/flux-coral2#222.
> I could be mistaken but I wouldn’t this would be related to how the flux instance is started. It more likely has to do with how the MPI job’s resources are translated to apinfo….
tgamblin commented on issue spack/spack#48612.
I think this is correct. I think technically, code generated by amdclang should have a link dependency on some piece of the rocm runtime, but we should model that with runtime injection (like gcc and the synthetic gcc-runtime package), not with a link dependency on llvm-amdgpu (which includes way too much stuff)….
astrophpeter commented on issue openjournals/joss-reviews#7342.
Thank you!…
jedbrown commented on issue mpi-forum/mpi-issues#846.
Reading through this today, it seems there has been a lot of talking past each other. I’d hope everyone here agrees that the last three in George’s four examples https://github.com/mpi-forum/mpi-issues/issues/846#issuecomment-2035945121 are invalid. The first case could be made to work, but it’s really dangerous for users to work with that semantic and what is the value? If someone is advocating for that case to be valid, it would be useful for them to suggest a code sample indicating how it would be useful….
jedbrown commented on issue E4S-Project/E4S-Project.github.io#9.
This remains obtuse in that it just goes to the generic “contact us” page (presumably this is an email to the technical lead) and the Level 0/1/2 don’t define what should be done by the submitter (they can’t unilaterally add to these registries) before emailing. A form or GitHub issue template would streamline that….
brendankeith commented on issue mfem/mfem#4460.
> @brendankeith, @koning and @psocratis – Reminder to please review this when you get a chance. We would like to merge it soon. Thanks in advance for your help! …
markcmiller86 commented on issue visit-dav/visit#19651.
@brugger1 I know you are doing stuff regarding xcb. I wonder if this feature request relates in any way?…
hppritcha commented on issue spack/spack#44654.
I wonder if you could use something like this from the chapel spackage to qualify the depends on UCX at 629? …
tzanio commented on issue mfem/web#291.
ping: @jandrej and @v-dobrev …
v-dobrev commented on issue mfem/mfem#4651.
@psocratis, I see that you gave thunbs-up to @thartland’s comment. Does that mean you also approve the PR? If so, can you please hit the approve button?…
cjvogl commented on issue mfem/mfem#4496.
> I tried out the current state of this branch (23edd75), and I noticed a few things: …
grondo commented on issue flux-framework/flux-core#6336.
This has be reproducing lately and we now get the extra information about what signal terminated the broker:…
garlick commented on issue flux-framework/flux-core#6501.
While it’s disturbing that the systemd unit will complete with processes still running in its cgroup, I think we’ve addressed the fundamental problem with the changes in 0.68 to the way the IMP forwards signals….
chu11 commented on issue flux-framework/flux-core#6556.
oops, i totally forgot about stats clearing. Tweaked and added a test….
garlick commented on issue chaos/diod#129.
Thanks!…
cyrush commented on issue LLNL/conduit#1352.
It turns out the future is already here….
trws commented on issue spack/spack#20551.
It’s failing because you don’t have bash-style completion loaded. If you source the setup-env.sh in zsh rather than in bash, or in addition to I suppose, and then launching zsh it should work correctly for you @ChrisPWelsh. …
jedbrown commented on issue rust-lang/rust#135516.
It’s nigh impossible to use shared memory (or write any useful kernel) without the concept of thread indices, which are in core::arch::nvptx. I had in mind that there would be vendor-specific intrinsics using that vendor’s preferred terminology (thus my initial suggestion that the feature gate would be stdarch_nvptx, same as #111199) and we could later define a common subset using a single consistent nomenclature. I think it would be confusing if users of vendor-specific features had to deal with two nomenclatures, so I’d be in favor of the the “re-expose” strategy….
vsoch commented on issue oras-project/oras-py#180.
If you found a way that works, why would you be doing something wrong?…
mplegendre commented on issue hpc/Spindle#59.
JSC = Juelich Supercomputing Center. They submitted the #52 PR….
trws commented on issue flux-framework/flux-sched#1326.
They’re the fastest runners, by almost half… This is crazy….
cmoussa1 commented on issue flux-framework/flux-accounting#562.
Thanks @ryanday36. The cur_nodes and cur_cores for a job will get updated when the job enters RUN state and INACTIVE state. Sorry that wasn’t made immediately clear - I added that support in #561….
vsoch commented on issue apptainer/apptainer#2706.
> I don’t know how to get the container you reference to get a python tensorflow installation….
adayton1 commented on issue LLNL/Umpire#795.
What’s the current status of this merge request? I’m testing out asan and finding it works for simple tests. I’m working on enabling it in my application, but until this goes in, I’ll have to disable memory pools….
vsoch commented on issue vsoch/citelang#54.
That sounds like a bug! Do you want to take a shot at a PR to fix it?…
acfisher commented on issue mfem/mfem#4641.
After some discussion, we confirmed that this work is fully superseded by the arbitrary order version so I will close this PR….
correaa commented on issue compiler-explorer/compiler-explorer#7279.
Well, thankfully (to you and others), most libraries already work across different GB compilers already….
jeremylt commented on issue CEED/libCEED#1728.
The user facing interface is correct - my impl is bad. I am fixing it…
jedbrown commented on issue CEED/libCEED#1728.
I’m not really following the need for vec->CopyArray. Where would that memory come from? Is there a reason not to wrap it in a CeedVector and then CeedVectorCopy instead of introducing a new way to use a raw pointer (without specifying its length, but tacitly promising that it’s long enough)?…
tgamblin commented on issue spack/spack#48053.
> how crucial is your suggested enhancement …
hppritcha commented on issue open-mpi/ompi#12307.
@janjust could you check the VERSION file situation on v5.0x ?…
vsoch commented on issue flux-framework/flux-sched#1316.
This is tested (and the basics are working in fluxion-go)! :partying_face: …
vsoch commented on issue expfactory/expfactory#179.
Thanks! You will need to make sure tests pass, and please ping me if you need me to look at something….
adrienbernede commented on issue LLNL/uberenv#138.
@becker33 I tried you suggested change. It turned a failure to find included files into a failure to find the compilers. …
BradWhitlock commented on issue LLNL/axom#1438.
Blending takes tuples of ids and weights applies them to an input coordset or field to make a new coordset or field. The inputs are blended according to the weights to make the new item. https://github.com/LLNL/axom/pull/1438/files#diff-fb605bba545a37f0257de0b2acdc23ec71f15af3ab1cf907ba5a580e635cb754…
hppritcha commented on issue HewlettPackard/shs-libcxi#3.
thanks that should do @boeschf . …
emily-howell commented on issue Alpine-DAV/ascent#1442.
Yes, I can add a note to the change log. …
JustinPrivitera commented on issue visit-dav/visit#19952.
@markcmiller86 mentioned that “Sample Statistics Query” might mean that it is taking a representative sample of the dataset to perform the statistical queries. If that is the case, that might explain away some of the differences with the “Sample Statistics Query”. We should investigate if that is the case. …
cyrush commented on issue visit-dav/visit#20189.
Do you have libblas installed on your system? I don’t think we bundle blas. …
alecbcs commented on issue spack/spack#47921.
@citibeth merging after waiting >3 days for a response. Looks good to me….
jedbrown commented on issue openjournals/joss-reviews#6099.
Yay. I saw the release and figured I’d wait a couple days before pinging about the paper. Great to hear it’s being wrapped up….
milroy commented on issue flux-framework/flux-sched#1322.
I updated the PR to include an alternative method of extracting (p, x) pairs of strings (in the spirit of what you suggested, @trws, and included that approach as a series of fixup commits. Those fixups can be squashed depending on which approach is better.
…
wihobbs commented on issue flux-framework/flux-core#6549.
@grondo and I chatted this morning about the possibility of leveraging fuse to help with some of this work. We found some docs about a port of 9P to fuse that could be helpful. …
aperloff commented on issue LLNL/SSAPy#42.
Please resolve conflicts. I’m satisfied with this PR and will merge it in once the conflicts are resolved….
adrienbernede commented on issue spack/spack#48553.
I don’t know much of the infrastructure you are using, but here are some thoughts from my experience running CI job (including Spack CI) on an HPC cluster: …
jameshcorbett commented on issue flux-framework/flux-coral2#249.
Done thanks! Setting MWP….
adrienbernede commented on issue LLNL/CARE#307.
@adayton1 I can confirm that by turning off tests I get the CI to pass….
v-dobrev commented on issue spack/spack#47642.
For some reason the ci/gitlab-ci pipeline does not test parts that depend on llvm when just the llvm package is updated (see e.g. #48162), so pipeline errors from llvm updates show up in other PRs like this PR.
…
vsoch commented on issue singularityhub/singularity-hpc#663.
This was an issue with Python versioning, going to close because the version requirements have been made clear. …
grondo commented on issue flux-framework/flux-core#6539.
Thanks! I’m fine leaving the commits as is. I pushed the suggested changes and will remove WIP. There are already tests and documentation, so I’ll go ahead and set MWP as well….
chu11 commented on issue flux-framework/flux-core#6544.
hmmm inception had a failure with …
correaa commented on issue conan-io/conan-center-index#21179.
I though the next step was to merge this. Do I need to do something before that?…
btalamini commented on issue LLNL/serac#1310.
@white238, requesting help with a test. I had to disable a test that verifies an error check works as expected. It uses the EXPECT_DEATH gtest macro. It works fine when I launch the executable myself, but fails when the ctest launcher is executing the entire suite. Is there a way to get this unit test of the error check to operate?…
cyrush commented on issue LLNL/conduit#1036.
I ran these to test this and didn’t hit an issue….
markcmiller86 commented on issue visit-dav/visit#20182.
It should really skip when platform is not the tier1 platform. But, am busy with other items and don’t have time to provide precise logic. I think you get the idea though….
alecbcs commented on issue spack/spack#47785.
Looks good! …
hppritcha commented on issue open-mpi/ompi#13030.
nope, that’s part of mega PR https://github.com/open-mpi/ompi/pull/12226…
v-dobrev commented on issue mfem/mfem#4296.
Superseded by #4661….
milroy commented on issue flux-framework/flux-sched#1323.
> I’m looking at the test cases to see if there’s a straightforward way to check that the message has been cleared. …
IanLee1521 commented on issue elastic/kibana#194240.
I’m going to close this since I can confirm now that in at least 8.16.1 the widget sized to the full panel addresses the issue. Thanks!…
IanLee1521 commented on issue elastic/integrations#11656.
Thank you yes, 3kt pointed that out separately that I’d missed that line in their message. Looking forward to reporting back once that is out!…
adrienbernede commented on issue LLNL/radiuss-spack-configs#117.
Replaced by #116 (next release will have split spack configuration files so users can pick e.g. only shared compilers configuration)….
markcmiller86 commented on issue LLNL/H5Z-ZFP#137.
Ok, so boiling down further, if HDF5 install used to build H5Z-ZFP is shared-only or static-only, then everything is fine because both the HDF5 tools and the H5Z-ZFP plugin will be built against the only available HDF5 install….
adrienbernede commented on issue LLNL/CARE#307.
> The below error should be fixed by setting CARE_ENABLE_IMPLICIT_CONVERSIONS and ENABLE_IMPLICIT_CONVERSIONS to OFF (or the corresponding variant in spack https://github.com/LLNL/radiuss-spack-configs/blob/main/packages/care/package.py#L75). Those options are removed in the develop branch of CHAI and CARE. …
jedbrown commented on issue FreeCAD/FreeCAD#14667.
Sorry, been afk. I’m not sure what action here is expected from me, besides making a new rebased branch and opening a new PR. The forum thread seems targeted at users, not avoiding replication between different plugins (that is, out-of-tree solvers using the FEM Workbench). Another thing to consider there is desire to compare solvers without needing to set up all boundary conditions, etc., separately for each solver. …
cyrush commented on issue Alpine-DAV/ascent#1438.
dataset_bounds provides this already for normal renders.
…
tgamblin commented on issue spack/spack#26397.
It’s a good feature but I think we need to prioritize more pertinent things for 1.0…
trws commented on issue flux-framework/flux-sched#1323.
> I just remembered that the error message is actually member data of dfu_impl_t and dfu_traverser_t calls methods that change the error message.
…
jameshcorbett commented on issue flux-framework/flux-coral2#245.
@bdevcich how come you closed this?…
cmoussa1 commented on issue flux-framework/flux-accounting#561.
Thank you for the clarification! Yes, the latter case is absolutely something to consider - I think at this point, just to make some progress on this front, maybe using jobspec (and/or making an estimation as best we can) should be okay. …
garlick commented on issue flux-framework/flux-accounting#561.
Yes, R is only available at RUN state. …
astrophpeter commented on issue LLNL/SSAPy#47.
On the LFS of binaries / commit history, @aperloff I’ll let you deal with that if it’s fast for you?…
jeremylt commented on issue CEED/libCEED#1723.
Note: Rust failure is an unrelated issue in their nightly build that they are working on…
gauthier12 commented on issue wendlers/mpfshell#114.
Hi, …
balos1 commented on issue spack/spack#48389.
@spackbot re-run pipeline …
btalamini commented on issue sandialabs/optimism#105.
@ralberd , hod on, don’t merge - Mike has a more fundamental fix. See comments above….
mej commented on issue mej/nhc#129.
This has now been addressed in 2b6ea0f; closing Issue. …
jameshcorbett commented on issue flux-framework/flux-accounting#560.
Got it that makes sense!…
gardner48 commented on issue LLNL/sundials#635.
Looks like the Swig test is failing because of ubuntu-latest switching from ubuntu-22.04 to ubuntu-24.04. It seems pcre is not installed by default now….
adayton1 commented on issue LLNL/Umpire#927.
How is synchronization handled? i.e. after allocating some shared memory, say I want to fill it in on rank 0 and then make it available to all other ranks?…
adayton1 commented on issue LLNL/CARE#307.
I can’t reproduce this error locally: …
hppritcha commented on issue HewlettPackard/shs-libcxi#3.
this is an issue on our grace hopper system….
astrophpeter commented on issue openjournals/joss-reviews#7342.
Hi @ivastar, are there any updates on this?…
jandrej commented on issue mfem/mfem#4656.
You can achieve this by using the SubMesh functionality. You have to define each problem on a SubMesh and define the interfaces for each problem in a way that they communicate the correct quantity that you want to transfer….
artv3 commented on issue LLNL/RAJA#1784.
The future is now…
tzanio commented on issue mfem/mfem#4619.
Merged in next for testing……
tzanio commented on issue mfem/web#291.
@jandrej and @v-dobrev, can you take another look when you get a chance?…
v-dobrev commented on issue mfem/mfem#4651.
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….
CRobeck commented on issue CRobeck/instrument-amdgpu-kernels#6.
@coleramos425 can you re-land this when you figure out the internal build issues?…
CRobeck commented on issue triton-lang/triton#5520.
@antiagainst …
tgamblin commented on issue spack/spack#48353.
> For example if I replace class EdgeAndDepth with a tuple …
alecbcs commented on issue spack/spack#48373.
@spackbot run pipeline…
hppritcha commented on issue open-mpi/ompi#12954.
fixed via #12956 and #12959…
gauthier12 commented on issue michaelrsweet/lprint#130.
Ok, my bad, it seems that I was using the debian package executable and not the one compiled by myself. …
vsoch commented on issue converged-computing/fluxqueue#1.
TODO: I don’t like this design - I want to instead try using https://github.com/converged-computing/fluxion. I’ll make a branch off of this one to test in case it doesn’t work. It will simplify things hugely….
balos1 commented on issue LLNL/sundials#634.
It looks like you may just need to add sundials_core to the target_link_libraries command….
vsoch commented on issue spack/spack#48341.
> Unfortunately that also means that files that were added in the middle of 2024 with a copyright header year of 2023 were not flagged until today… …
vsoch commented on issue singularityhub/singularity-hpc#679.
@Amjadhpc could you please add a test that enables the new setting and ensures it produces the right output? And @muffato I’m interested in your feedback here since you are correct views could be an alternative solution….
acfisher commented on issue mfem/mfem#4641.
@mlstowell Is this fully superseded by the arbitrary order version? …
chu11 commented on issue flux-framework/flux-core#6518.
``` …
garlick commented on issue chaos/diod#116.
Thanks!…
cmoussa1 commented on issue LLNL/cmvl#1.
This looks good on my end!…
cjekel commented on issue cjekel/piecewise_linear_fit_py#120.
Wheel is live …
trws commented on issue spack/spack#48073.
@haampie, given that #48065 builds its own gcc and libstdc++, is this not also solved by that?…
trws commented on issue flux-framework/spack#245.
@vsoch, I really would like to get this fixed, but I keep having no luck actually reproing it locally. Have you been able to do that, and if so could you send me a script/command? Debugging it in GHA is proving deeply challenging….
grondo commented on issue flux-framework/flux-core#6536.
Thanks! I’ve set MWP….
trws commented on issue flux-framework/flux-core#2875.
That looks pretty good. I’m working back through older emails today and saw the previous few all together. If it makes it easier, we could also use something like a prefix argument, think -Wl, so the flags would be normal in the plugin but would only be valid either immediately after a --plugin or --ext or similar or even as a comma-separated string. The prefixes work fine too, but it’s easier to transition them later and keep the old support if the actual name the code handles isn’t changing.
…
SteVwonder commented on issue cowboy/dotfiles#42.
FWIW, in my fork I added a check after the backup for the authorized_keys file in the backup directory. If it exists, then copy the authorized_keys back into ~/.ssh: https://github.com/SteVwonder/dotfiles/blob/master/bin/dotfiles#L272
…
IanLee1521 commented on issue LLNL/cmvl#1.
@cdefrates / @cmoussa1 – Anything more we need to do on this before we merge it? …
correaa commented on issue mull-project/mull#794.
ups, I shared the CI link before it finished. …
cmoussa1 commented on issue flux-framework/flux-accounting#548.
Thanks for the review @wihobbs! Force-pushed a fix for the spelling mistake and rebased to catch up after #527; setting MWP here…
grondo commented on issue chaos/diod#110.
Hmm, one comment got dropped. typo ‘reqst’ in df07c796fe620e5f0e2d8ed47678efea0982230…
IanLee1521 commented on issue LLNL/benchpark#481.
@pearce8 / anyone – is there a good place you think that the schema should be documented? At the moment I’m considering either https://github.com/llnl/cmvl or https://github.com/LLNL/toss-configs which each have pros and cons. But certainly could be elsewhere too….
artv3 commented on issue mfem/mfem#4647.
YES!…
CRobeck commented on issue triton-lang/triton#5506.
cc: @fywkevin …
correaa commented on issue mull-project/mull#1077.
Is this good to replicate the problem? …
v-dobrev commented on issue mfem/mfem#4648.
I’m not sure what Caliper config will give you the details you are looking for. Maybe try using event-trace,mem.pages or hatchet-region-profile,mem.pages,output.format=json. You can see the various Caliper config options with <caliper prefix>/bin/cali-query --help=configs and then for specific recipes you can see their options with e.g. <caliper prefix>/bin/cali-query --help=event-trace….
artv3 commented on issue LLNL/RAJA#1773.
This isn’t quite done, but I do want to trigger the CI to double check that we are passing test across the board. …
vsoch commented on issue singularityhub/singularity-hpc#678.
@Amjadhpc that is a very reasonable feature request! Are you able to develop in Python or would you need help? You’d want to add a setting to settings.yaml that is something akin to version_naming and then a boolean that defaults to false (to preserve current behavior) and if true, you’d want to tweak the install path to instead of creating a directory with the version, to have it write a file with the module extension to that path instead. Let me know if you would like to give that a try and open a PR. Happy Holidays!…
imguoguo commented on issue imguoguo/Comments#12.
> @qilianshuo…
vsoch commented on issue flux-framework/flux-core#5326.
Worked! …
vsoch commented on issue converged-computing/flux-tutorials#1.
@wkharold this is fully working to build and deploy, and (if/when you have bandwidth) the cleanup (without my custom delete.sh script) doesn’t fully work - there looks like something circular going on with foundation vs. the instances subnetwork….
jedbrown commented on issue FreeCAD/FreeCAD#14667.
Hi, @maxwxyz. I’ll rebase and create a new PR (as noted https://github.com/FreeCAD/FreeCAD/pull/14667#issuecomment-2484275151). What would be a better place to discuss these questions – in particular I can’t tell what is aspirational versus meant to be ready now without heavy refactoring of the FEM workbench?…
hppritcha commented on issue open-mpi/ompi#12998.
we should add this to 5.0.7 to avoid embarrassment….
vladotomov commented on issue mfem/mfem#4642.
Hi @thorvath12, take a look at the implementation of …
acfisher commented on issue mfem/mfem#4641.
@mlstowell I think I managed to extract a coherent piece of work from that branch. I believe you put pretty much all of this is your work. It compiles and passes all of the tests except sedov have a look and let me know if you think this is ready for review….
JustinPrivitera commented on issue visit-dav/visit#20166.
See the linked discussion for images and additional context….
garlick commented on issue flux-framework/flux-core#6534.
Thanks! Doing that!…
grondo commented on issue flux-framework/flux-core#6534.
Already approved, so feel free to set MWP!…
tzanio commented on issue mfem/mfem#4637.
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….
vsoch commented on issue converged-computing/performance-study#82.
Note that most of the paper plots are not added here, these are earlier versions (still with details that are since fixed). I’m not adding any paper plots until the data review is done….
adrienbernede commented on issue LLNL/RAJA#1771.
Replaced by #1774 …
JustinPrivitera commented on issue visit-dav/visit#19851.
we’ll do it for 3.5..0 :-)…
markcmiller86 commented on issue visit-dav/visit#20160.
Any chance you can attach the associated session file. The # in the format string is odd to me and probably defeating the logic I added to try first what is specified in the format string before using anything embedded in the actual string itself….
correaa commented on issue mull-project/mull#794.
The diagnosis is fairly simple, with my library, if I run this naive line I get surviving mutants, …
brendankeith commented on issue mfem/mfem#4460.
> @brendankeith – friendly reminder
v-dobrev commented on issue mfem/mfem#4578.
garlick commented on issue flux-framework/flux-sched#1321.
I did just, plus reordered the commits in the PR to avoid breaking git bisect. I’ll drop the WIP from the PR title since this seems complete now.
…
milroy commented on issue flux-framework/flux-sched#1321.
> When working with someone else on a PR I usually add their fork as a remote, pull down their working branch, do some work, then push it to my own repo. Like …
grondo commented on issue flux-framework/flux-core#6533.
Thanks!…
tgamblin commented on issue cass-community/new#6.
@jwillenbring thanks! I’ve updated this with more information….
cyrush commented on issue visit-dav/visit#20152.
We still don’t have a good way to create this list. …
BradWhitlock commented on issue visit-dav/visit#20143.
It’s not intuitive but a middle mouse click will reset the lighting window view in preview mode….
tgamblin commented on issue glotzerlab/hoomd-blue#1973.
Note that this is already in Spack: https://packages.spack.io/package.html?name=hoomd-blue …
chu11 commented on issue flux-framework/flux-core#6531.
thanks, setting MWP…
gardner48 commented on issue LLNL/sundials#632.
@blaos can you take a look at the error handling and SuperLU_DIST tests on Dane? …
artv3 commented on issue LLNL/RAJA#1773.
TODO: Compare the Caliper results with standard timers. …
tgamblin commented on issue spack/spack#48100.
Personally, I’d rather just do {name}-{version}-{hash} for the default projection. nix is similar – they do /nix/{hash}-{name}-{version}. I think the extra metadata in the PATH` will eventually become a support burden, or confusing in a weird way.
…
cmoussa1 commented on issue flux-framework/flux-accounting#556.
@ryanday36 when you have some time, could you give this a look and let me know if this looks like something you’d be good with in terms of managing banks in the flux-accounting database? Note that this doesn’t actually enforce the limit in the priority plugin (yet); it’s just the first part in being able to define and set this limit in the DB….
adrienbernede commented on issue LLNL/radiuss-shared-ci#38.
@balos1 sorry I forgot about this, I’ll get it merge soon….
JustinPrivitera commented on issue Alpine-DAV/ascent#1435.
num_domains is wrong in different places in the ascent info node…
jedbrown commented on issue rsmpi/rsmpi#196.
I’m not following the question, but this makes use of a lot of unsafe for what derive(Equivalence) would presumably make safe. PRs welcome, and feel free to reopen….
hppritcha commented on issue open-mpi/ompi#12226.
uh oh. nvidia needs to be more disks!…
dylan-copeland commented on issue mfem/mfem#4371.
> Some modern art thanks to the new ref321 miniapp
…
grondo commented on issue flux-framework/flux-sched#1321.
I was thinking it was a delay in available of the switch to preprocessor defines for schedutil flags (flux-framework/flux-core#6520), but since the +partial-ok message was present in the logs, I’m also mystified.
…
hppritcha commented on issue open-mpi/mtt#950.
@jsquyres @bwbarrett …
vladotomov commented on issue mfem/mfem#4467.
The issues were resolved by updating the ROCm version, thank you @camierjs and @mstees!…
IanLee1521 commented on issue elastic/integrations#8793.
I am wondering if it could be possible for the integration to do something like what the cisco-ios integration or panw integration do to start a syslog listener that can receive and process the logs….
tgamblin commented on issue actions/runner-images#10636.
In case this helps anyone, it looks like kcov is not available in Ubuntu 24.04 (it was removed from upstream Debian, and then added back – so it may come back in Ubuntu 25). For now, though, tests using it will break. …
adrienbernede commented on issue LLNL/CARE#307.
@adayton1 @davidbeckingsale Any idea about what could be causing the error we see everywhere in the CI ?…
jandrej commented on issue mfem/mfem#4610.
Can we get a simple unit test with a test mesh for this? We have the data repo for binary mesh files https://github.com/mfem/data/tree/master/exodusii…
tzanio commented on issue mfem/mfem#4626.
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….
markcmiller86 commented on issue visit-dav/visit#20113.
This is ready for review. There is one remaining issue after this…to unwind python eggs from pillow but that will be done in a different PR….
v-dobrev commented on issue mfem/mfem#4583.
Here’s the patch I came up with to address the issues noted by @nmnobre. I also added some documentation in the INSTALL file about the fine-tuning of the installation directories with CMake.
…
garlick commented on issue flux-framework/flux-pmix#107.
Thanks! Setting MWP….
garlick commented on issue flux-framework/flux-core#6485.
Both of these problems seem to stem from libev using the epoll-shim, when it actually has support for kqueue. At least building flux-core against the brew version of libev appears to get around them. …
JustinPrivitera commented on issue visit-dav/visit#20131.
merge w/ develop in #20132…
cyrush commented on issue visit-dav/visit#20124.
Looks good, feel free to merge….
CRobeck commented on issue triton-lang/triton#5127.
@antiagainst don’t merge this yet. @AlexAUT is looking into why this seems to sometimes hang the MI300 CI. So far it appears random so we need to look into it a bit….
adrienbernede commented on issue spack/spack#48067.
@haampie if you have the bandwith to review it today, that’s awesome cause then I can get it in the next snapshot. Otherwise, so be it. Thank you….
jedbrown commented on issue openjournals/joss-reviews#7484.
Thanks, @slamont1! …
hppritcha commented on issue open-mpi/ompi#12925.
please rebase to clear out the mpi4py error….
jandrej commented on issue mfem/mfem#4586.
> FWIW, std::array and std::tuple aren’t used in dtensor.hpp.
…
vladotomov commented on issue mfem/mfem#4467.
@artv3 we’re still investigating the errors reported by @mstees …
grondo commented on issue flux-framework/flux-core#2875.
I did some prototyping on this issue recently (see my cli-plugins branch) …
chu11 commented on issue flux-framework/flux-core#6507.
> That would probably work although I think it’s pretty clear what’s going on in these tests. I’d be inclined to not continue to hard code a path - it feels slightly wrong. …
jeremylt commented on issue CEED/libCEED#1715.
```…
trws commented on issue spack/spack#47739.
> We can run the binaries on any machine since we ship libstdc++.so. …
artv3 commented on issue mfem/mfem#4467.
Hi all, just wanted to follow up here. Is this PR ready to be merged?…
tgamblin commented on issue hpsfoundation/tac#17.
Viskores was voted in at the sandbox level by the TAC!…
artv3 commented on issue LLNL/RAJA#1759.
This omp target test keeps failing: …
alecbcs commented on issue systemd/systemd#26498.
I’m also experiencing this behavior with v257. I set HibernateDelaySec=24h to have my laptop hibernate after being suspended for 24h but this seems to inhibit the laptop from checking the battery and suspending when < 5%. My laptop (Framework 13) has relatively high battery drain meaning I end up regularly finding my laptop with dead (0%) after leaving it for half a day if it wasn’t fully charged when I first put it to sleep….
hppritcha commented on issue spack/spack#47705.
this PR works without any special workarounds for me now on my ss11 system….
hppritcha commented on issue open-mpi/ompi-www#519.
please try again. the NMC folks fixed the issue. closing….
chakshinglee commented on issue mfem/mfem#4572.
I agree that the result of the bilinear form is a scalar, so terms like e_i e_i^{\mathrm{T}} should/could be removed. I was just following the notation/convention in the original documentation….
balos1 commented on issue LLNL/sundials#602.
/autofix…
trws commented on issue flux-framework/flux-sched#1319.
That’s pretty much what I would have expected to happen. I’m sure we can fix this, but as it sits we don’t just validate it, the queue is the actual object in qmanager that defines behavior for various transitions, there’s no notion of a job that doesn’t have a queue so there are a number of places where we unconditionally invoke something on the queue of a job. …
trws commented on issue flux-framework/flux-core#6467.
> In the previous iteration I had introduced a signed arithmetic error that caused a -1 error return not to be detected in the content stack. …
jameshcorbett commented on issue flux-framework/flux-coral2#244.
@grondo do you have any thoughts on this / whether it sounds like a good idea?…
correaa commented on issue conan-io/conan-center-index#21179.
All this is great. Thank you. …
emily-howell commented on issue Alpine-DAV/ascent#1400.
This does hit a Conduit warning in the ascent_png_encoder.cpp (https://github.com/Alpine-DAV/ascent/blob/4b9321285ce290aa51ecb7eb91acbb73b701d7b7/src/libs/png_utils/ascent_png_encoder.cpp#L336) …
astrophpeter commented on issue openjournals/joss-reviews#7342.
Hi! Thanks, maybe aneeshnaik, or sterinaldi, or dgerosa. …
hppritcha commented on issue mpi-forum/mpi-forum.github.io#22.
@wesbland …
garlick commented on issue flux-framework/flux-sched#1318.
This appeared in flux-core 0.61. If we remove these calls then sched will need to require a minimum of that version….
vsoch commented on issue flux-framework/flux-k8s#87.
Note that cert-manager is failing in three different projects of mine (this one included) so there was some change, either to it or GitHub. I’m going to try updating first and then try to figure out the error if necessary….
tgamblin commented on issue spack/spack#47947.
Can we add something to the importer to warn that the import is needed?…
hppritcha commented on issue open-mpi/ompi-www#519.
we’ve been having problems with the Open MPI project mail lists and this is probably an associated problem. I’ll look in to fixing this….
v-dobrev commented on issue mfem/mfem#4504.
Here’s a patch (for commit 9bee2fca) that works for me: …
tzanio commented on issue mfem/mfem#4602.
Discussion continued in #4610 …
grondo commented on issue flux-framework/flux-core#6491.
> else If there is an active job for user flux on the current node and this is rank 0 of that job: permit access …
gardner48 commented on issue LLNL/sundials#611.
This might fix the random CI failures we’ve been seeing with the Fortran interface to the dense linear solver on some platforms…
markcmiller86 commented on issue visit-dav/visit#20113.
Wait a minute, I don’t have a /usr/local/opt on my system…
hppritcha commented on issue uofl-capstone-open-mpi/ompi#2.
okay now just getting known comm create from group thing in mpi4py. rerunning….
alecbcs commented on issue spack/spack#47953.
@dependabot recreate…
vsoch commented on issue singularityhub/docker2singularity#136.
Pushed! …
grondo commented on issue flux-framework/flux-pam#7.
If possible, for extra validation, the PAM module could check if the local instance has access.allow-guest-user enabled….
chu11 commented on issue flux-framework/flux-core#6456.
While this issue still exists, it appears that the larger performance issue is in job-info, where an entire lookup on the entire key is done if the job has completed. That should probably similarly be “windowed” as well….
garlick commented on issue flux-framework/flux-core#6467.
This has been reworked a bit but the API facing changes are as originally proposed. …
jameshcorbett commented on issue flux-framework/flux-core#6477.
This flux-coral2 issue https://github.com/flux-framework/flux-coral2/issues/229 is relevant. …
cyrush commented on issue Alpine-DAV/ascent#1418.
@dpugmire circling back, it looks like the issue is that the c++ compiler is seeing these flags and does not know what do to with them. …
gauthier12 commented on issue justanhduc/task-spooler#48.
Thanks for your response. …
wihobbs commented on issue flux-framework/flux-core#6483.
I forgot to mention, in addition to the python file above, you need to copy the header file from src/common/libflux/plugin.h, with the function declaration prepended before the final #endif:
…
cmoussa1 commented on issue flux-framework/flux-accounting#546.
Thanks for opening this up!! I’m assuming no, but should I tag a new release of flux-accounting with this PR now merged? I don’t mind either way. …
artv3 commented on issue LLNL/RAJA#1759.
To double check, Is this ready for merging? It sounds like we are keeping the expt namespace on KernelNaming. …
vladotomov commented on issue CEED/Remhos#48.
> I guess I know why it was not passing the regression, we turned off the clipping of the values here to see the undershoots
hppritcha commented on issue uofl-capstone-open-mpi/prrte#9.
pushing a small fix to this branch shortly. Please squash it before merging….
hppritcha commented on issue openpmix/prrte#2087.
this sort of change broke running autogen.pl for me on my rhel8 system…
hppritcha commented on issue open-mpi/ompi#12906.
trying to beef up param checking. i’ll probably do that in a separate pr….
gauthier12 commented on issue fairecasoimeme/Zlinky_TIC#290.
La proc
jeremylt commented on issue mfem/mfem#4609.
Not quite - we need to finalize libCEED v0.13, which I hope to do before we all break for Christmas…
trws commented on issue flux-framework/flux-core#6466.
Looking it over, if the calls from job-manager can become a jobtap plugin (not sure but it looks like it at first glance), that would definitely help. The main thing that might need some thought is how to define “busy” and “quiescent” callbacks in fluxion after everything went asynchronous. It’s not quite as easy as it used to be (we used to just process everything in-order, so when the callback ran, it was time) but now it will have to detect when the sched loop has no further work to do. That’s actually possible, it puts the event loop to sleep in that case until something comes in, but it’s a bit more work. There are also some strange states we didn’t really have before, like there being jobs that are satisfiable but not reservable….
jedbrown commented on issue EnzymeAD/rust#189.
I think it’ll be an issue if allocation is needed. We should eventually expose a pre-allocated tape as can be done using C-Enzyme. …
trws commented on issue spack/spack#47470.
On the libselinux.so dependence, if it’s removed from DT_NEEDED do the binaries fail? It seems like an odd one to be pulled in that way rather than by dlopen or similar….
chakshinglee commented on issue mfem/mfem#4572.
more specifically these lines…
jandrej commented on issue mfem/mfem#4572.
Quoting the documentation for the Coefficient overload
…
vladotomov commented on issue mfem/mfem#4572.
@chakshinglee are you saying to fix the code or fix the documentation?…
trws commented on issue flux-framework/flux-sched#1315.
That begs the question of why it isn’t being added at the end, which would have left these unchanged. Is there a common format being mimicked?…
balos1 commented on issue LLNL/sundials#603.
@Steven-Roberts I think core is missing from here. …
cyrush commented on issue LLNL/conduit#1326.
See: …
trws commented on issue LLNL/camp#160.
> * Use of camp::list to hold a set of numerical values in addition to types …
vsoch commented on issue flux-framework/flux-core#6463.
I did 3x on kind, did not reproduce….
markcmiller86 commented on issue visit-dav/visit#20080.
I will have a few options for fixing monday morning. Handling for ALL possible cases is slightly more involved. …
tgamblin commented on issue spack/spack#47781.
@alalazo I think this is ready. I deepened the annotated types by adding defs for all the pieces of the internal Tuple representation, added some docs, and realized we could speed up _next_version by making string lazy like __str__ already was….
garlick commented on issue flux-framework/flux-core#6463.
Is there a way we could create a reproducer for just the flux archive failure? It seems like that ought to be reproducible with a size=1 instance since it’s just flux archive create writing to the KVS, correct? Is it maybe something to do with content of the files you are putting in the archive? In my unsuccessful attempts, I was just getting random bytes from /dev/urandom and trying to create an archive with one big “file” in it (up to 10G). There were hundreds of G available in /tmp….
grondo commented on issue flux-framework/flux-core#6461.
> Also - I know that binomial == kary:2, I’m mostly doing both to sanity check flux knows that too - seems like something is funny! :) …
JustinPrivitera commented on issue visit-dav/visit#17276.
I’m not up-to-date on this conversation, but the current version of VisIt is 3.4.1….
v-dobrev commented on issue mfem/mfem#4565.
Thanks, @tepperly! I marked #4601 as ready-for-review to initiate our review process. If you have some time to test the PR in LiDO, that will be great….
v-dobrev commented on issue spack/spack#47780.
The following issue is probably an issue with hypre’s build system, not hypre’s Spack package – I just noticed it when testing this PR: looking at the hypre build log from Spack, I see that the hypre-internal blas and lapack directories are built and linked into libHYPRE.a even when the +lapack variant is used. I think hypre functions do not actually call these hypre-internal blas and lapack functions, so the behavior is correct – there’s just redundant compilation and linking of the hypre-internal blas and lapack….
hppritcha commented on issue spack/spack#47800.
hmmm not sure what that CI failure is about….
hppritcha commented on issue open-mpi/ompi#12226.
thanks Jeff. I think most all of the warnings are due to lack of embiggening of the internal ompi/opal datatype interfaces. but i’ll double check to see if we missed something. …
cmoussa1 commented on issue flux-framework/flux-docs#290.
Thanks for taking a look and reviewing this @garlick! I’ve force-pushed some fixes based on your suggestions above. Setting MWP here…
hppritcha commented on issue conda-forge/openmpi-feedstock#188.
For Open MPI 5.0.x releases, we support use of an external PRRte against which Open MPI 5.0.x releases can be built. …
JustinPrivitera commented on issue LLNL/conduit#1333.
I think a lot of code used in relay mpi for python can be reused here. Will discuss w/ @cyrush the best way to tackle….
artv3 commented on issue LLNL/RAJA#1775.
Thanks for working on this, on my side I need a small modification on the PluginContext as its where I am storing the kernel name for the caliper integration. See here: https://github.com/LLNL/RAJA/pull/1773….
alecbcs commented on issue spack/spack#47691.
I’m seeing similar broken behavior with git coming from the darwin-devtools stack on the main Spack binary buildcache….
imguoguo commented on issue pjsip/pjproject#4173.
> The flag -mfma is used when the instruction set is sse2: https://github.com/pjsip/pjproject/blob/master/third_party/build/os-auto.mak.in#L145
…
tzanio commented on issue mfem/mfem#4600.
Some of the CI runs are failing, can you take a look?…
dylan-copeland commented on issue mfem/mfem#4038.
@adam-sim-dev In a previous post, I explained that there is a known issue with projecting a coefficient, which may be why you have problems with ParLinearform when handling VectorDeltaCoefficient. The true DOFs may not be set correctly. I am working on some large changes to this PR, to fix this issue, but it may take a while to get it working in all cases. …