Open Source Heartbeat: Open Source Heartbeat

Issue Comment Events

tzanio commented on issue mfem/mfem#5190.

Re-merged in next for testing……

View Comment

vsoch commented on issue flux-framework/flux-core#7341.

The beauty of try/except!…

View Comment

grondo commented on issue flux-framework/flux-core#7341.

You can also use modprobe to schedule these kinds of tasks before and after the execution of a batch or alloc instance….

View Comment

chu11 commented on issue flux-framework/flux-core#7303.

re-pushed going with a slightly alternate implementation. …

View Comment

alecbcs commented on issue spack/spack-packages#3144.

@adamjstewart I’ve rebased this PR now that #3119 is merged. I’ll need you to approve before we can proceed since you requested changes….

View Comment

cyrush commented on issue spack/spack-packages#3200.

I appreciate the proposed solution, but I don’t think we should take such measures in the spack package. …

View Comment

tgamblin commented on issue spack/spack-packages#3200.

One other thought. We could: …

View Comment

alecbcs commented on issue spack/spack#51521.

Gotcha. I’ll drop the tag on the PR. Thanks for checking….

View Comment

becker33 commented on issue spack/spack#51632.

This PR does not apply cleanly to releases/v1.0 and will be removed from v1.0.3…

View Comment

tgamblin commented on issue spack/spack#51881.

I’m ok with this (enthusiastic, even) if we can add an audit like the one @haampie mentions. I think we need that soon though – otherwise I think we’re going to start getting a lot of packages with invalid constraints….

View Comment

camierjs commented on issue mfem/mfem#5190.

For lower orders, the performances are fine - this issue appears specifically at higher ones….

View Comment

adrienbernede commented on issue llnl/radiuss-spack-configs#157.

> Ah I see, that makes sense. And yes, I do have access to Corona, but just not the ability to relaunch radiuss-spack-config jobs from gitlab (I do have access for Umpire and maybe RAJA) …

View Comment

cyrush commented on issue llnl/conduit#1495.

with new builds of metis and parmetis, we get a different result for the polyhedra test on macos than on linux. …

View Comment

amagela commented on issue llnl/benchmarks#9.

@pearce8 , I haven’t finished this quite yet. …

View Comment

trws commented on issue flux-framework/flux-sched#1425.

That sounds right. I think as it stands, the R returned by the scheduler is what the shell uses for this, since that provides the actual resources to use, but for correct output wherever the jobspec_parse is used instead we would need either a separate annotation or to consult the R to determine what the result is….

View Comment

trws commented on issue flux-framework/flux-core#7340.

@grondo mentioned offering a flux python<num> for each of the installed bound pythons, these could have a “get path” option or similar, or maybe we could have arfuments on flux python itself to list or select the available version to use?…

View Comment

garlick commented on issue flux-framework/flux-core#7333.

Thanks - I added that and will set MWP….

View Comment

garlick commented on issue flux-framework/flux-coral2#446.

Thanks!…

View Comment

trws commented on issue desul/desul#164.

That would be pretty painful yeah. Do any of the implementations actually trap on an atomic add to a non-writable location? I haven

camierjs commented on issue GLVis/glvis#353.

Closing for in-place solution in #336…

View Comment

trws commented on issue spack/spack-packages#3144.

Darn, thanks for trying it!…

View Comment

jedbrown commented on issue openjournals/joss-reviews#9244.

Thank you @valsson!…

View Comment

v-dobrev commented on issue mfem/mfem#5210.

I thought about this issue and I think a better solution is to = delete the move-assignment operator of DenseMatrix. This way assignments to DenseMatrix will always be copy-assignments. The reason I think this is a better solution is that re-defining the move-assignment (especially at the level of Array) to do a deep copy, in the case when the LHS is an alias, goes against the idea of move-assignment. We will lose the ability to move-assign DenseMatrix but that is not a big deal, I think – this was already the case for a long time before we added it in #4028. …

View Comment

gardner48 commented on issue llnl/sundials#836.

#837 deprecates the <TPL>_WORKS options and replaces them with SUNDIALS_ENABLE_<TPL>_CHECKS to make the purpose of the option clearer (and namespace the option). …

View Comment

wihobbs commented on issue flux-framework/flux-security#213.

@chu11 Can you add the MWP label? I don’t have that ability; this repo must not inherit maintainer privileges from the core team (IIRC pmix doesn’t either)….

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#810.

Thanks @jameshcorbett! Setting MWP…

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#165.

@copilot Add manufactured tests for P1_H1 mixed problems similarly as you did with the P0_P1 tests. Put them in the folder called P1_H1 and go up to K = 2….

View Comment

adrienbernede commented on issue llnl/RAJAPerf#577.

@rhornung67 This is ready for review….

View Comment

wihobbs commented on issue flux-framework/flux-core#7325.

> Did you spot a way to easily “exempt” paths from a rule in dorny/paths-filter? This action seems to easily enable conditional execution based on files changed, not based on ‘only files changed’,…

View Comment

cmoussa1 commented on issue flux-framework/flux-core#7303.

I’ll second @jameshcorbett’s vote to just add a raw=False flag…

View Comment

cjvogl commented on issue E3SM-Project/E3SM#8012.

> @cjvogl can you confirm that with this change E3SM will build on LC machines? …

View Comment

trws commented on issue spack/spack-packages#3088.

I have not seen that no. I’m not sure what’s causing it either, it’s a bufreadpost from your dotfiles trying to load a lua parser, but I don’t know why that would be failing there. Maybe something is out of order in the load order specifically the way we set it up with spack?…

View Comment

tzanio commented on issue mfem/mfem#5110.

Merged in next for testing……

View Comment

dylan-copeland commented on issue mfem/mfem#5203.

What is the policy for output in unit tests? There is std::cout << std::flush; in fem/test_var_order.cpp and other calls to cout if (verbose_tests) in test_quadinterpolator.cpp. When should cout be used with verbose_tests, vs. the more common mfem::out? There is usage of mfem::out if (verbose_tests). Should we add if (verbose_tests) in addition to the change to mfem::out? I suppose unit tests should rely on REQUIRE statements to check conditions, and output should be unnecessary but possible in the case if (verbose_tests). If so, cout could also be changed in test_quadinterpolator.cpp and test_var_order.cpp, along with use of if (verbose_tests)….

View Comment

jameshcorbett commented on issue flux-framework/flux-sched#1419.

I think I’ve addressed your comments @milroy ! Thanks for the review….

View Comment

garlick commented on issue flux-framework/flux-core#7321.

Restarted el8 builder after hitting …

View Comment

jameshcorbett commented on issue flux-framework/flux-core#7303.

I vote B) and I would recommend not type-checking the input with isinstance but instead just converting it to bytes() (if you even need to do that) and letting any errors propagate…

View Comment

alecbcs commented on issue spack/spack-packages#3123.

Looks like https://github.com/davidhalter/jedi/issues/2064 is an open issue for Python 3.14 compatibility in Jedi with a few known bugs. …

View Comment

jeremylt commented on issue openjournals/joss-reviews#8388.

Ratel 1.0 has been released. I appreciate your patience and apologize again for the delay….

View Comment

balos1 commented on issue llnl/sundials#796.

chu11 commented on issue flux-framework/flux-core#7191.

> It makes me lean to the idea of having a “mixed” db as I call it above. The eventlog is stored as it is above, with possibly some special handling of exceptions (maybe other events too). We have DB columns for important fields. Longer out, we can add sqlite plugin extensions for faster / simpler hostlist / rank searches in sqlite for those columns. …

View Comment

alecbcs commented on issue davidhalter/jedi#2064.

Hi all, just wanted to reach out to see if any additional progress had been made on this front. Thanks!…

View Comment

tzanio commented on issue GLVis/glvis#335.

@najlkin can you please resolve the conflicts?…

View Comment

jeremylt commented on issue CEED/libCEED#1694.

MacOS + weak symbols fixed here: https://github.com/CEED/libCEED/pull/1919…

View Comment

jedbrown commented on issue CEED/libCEED#1919.

Thank you. This looks clean and decoupled given the constraints. I agree it looks like the best option….

View Comment

cyrush commented on issue visit-dav/visit#20748.

I think this was resovled in #20768…

View Comment

vsoch commented on issue rootless-containers/usernetes#152.

We tried to use helm first, which is going to deploy the tigera operator, and could not get that setup working….

View Comment

chu11 commented on issue networkupstools/nut#3193.

Adding -u root worked for me….

View Comment

cyrush commented on issue llnl/conduit#1495.

windows path style issue on some of the tests on windows: …

View Comment

wihobbs commented on issue flux-framework/flux-core#7316.

In the last push: …

View Comment

grondo commented on issue flux-framework/flux-core#7318.

Thanks! …

View Comment

homijan commented on issue conservation-laws/ryujin#285.

Dead end here….

View Comment

chu11 commented on issue chu11/freeipmi-mirror#78.

a few of the issues I posted above don’t seem to be resolved. Did you push the wrong update here?…

View Comment

becker33 commented on issue spack/spack#51835.

@FlyingSamson I was finally able to reproduce your bug (accidentally, while trying to do something else). I was also able to backport the fix and include it in the upcoming release. Thanks for bisecting that and making it easy!…

View Comment

hppritcha commented on issue open-mpi/ompi#12979.

The application uses persistent send/recv requests with the comm pattern being an all-to-one pattern. The individual messages for the problem suggested above are about 40000 bytes in size when using 60 ranks. I’m seeing the hang for fewer than 64 procs. The behavior appears to be that the target rank (0) does receive all its messages but some of the senders do not “think” that all of their sends are complete. Again this appears to be btl-sm specific. turning off the fast boxes does not appear to help. investigation continues……

View Comment

JustinPrivitera commented on issue llnl/conduit#1495.

> ``` …

View Comment

jakubcerveny commented on issue jakubcerveny/gilbert-paper#17.

No need to apologize. Thank you…

View Comment

trws commented on issue hpsfoundation/tac#61.

Great news, thank you all!…

View Comment

vsoch commented on issue hpsfoundation/tac#61.

Hooray!!

JustinPrivitera commented on issue visit-dav/visit#20794.

To test I need to check what ends up in the analogous place for 3.5.0: /usr/gapps/visit/3.4.2/linux-x86_64/lib/python/lib/python3.9/site-packages/

View Comment

markcmiller86 commented on issue spack/spack-packages#2346.

@tldahlgren, @patrickb314 and @cyrush thanks so much!…

View Comment

wihobbs commented on issue flux-framework/flux-sched#1372.

Hi @ax3l, in looking back at this issue the EINVAL was actually raised on the job in your subinstance of Flux, not the top-level allocation itself. Here’s a theory on what could have happened:…

View Comment

vsoch commented on issue flux-framework/.github#7.

Updated. …

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#163.

@copilot I have fixed the tests. Please add more 3D tests inspired from the P1 2D tests using other equations….

View Comment

v-dobrev commented on issue spack/spack-packages#3040.

> @v-dobrev please see #1557 and revert your llvm_amdgpu change. It’s wrong. Even if this was the case, it shouldn’t be mixed with a mfem change. …

View Comment

jandrej commented on issue mfem/mfem#5192.

> Discussed in the PR meeting: instead of changing all files every year, does it make sense to have a top-level COPYRIGHT file that is updated yearly, and other source files reference that one. This is what Spack does, see https://github.com/spack/spack …

View Comment

artv3 commented on issue llnl/RAJAPerf#599.

> > @rchen20 , you can also squash merge …

View Comment

benzwick commented on issue SlicerCBM/SlicerCBM#77.

## Current ruff check results (as of 2026-01-20)…

View Comment

brendankeith commented on issue mfem/web#305.

This is now ready for review!…

View Comment

tzanio commented on issue mfem/mfem#5198.

is this ready-for-review?…

View Comment

garlick commented on issue flux-framework/flux-core#7292.

Oh, hey, didn’t we have a problem a while back where the flux systemd instance was getting killed when someone logged in and out of a node? I thought we had a workaround in place for that, but don’t recall the details. Maybe the 90s default stop timeout is in effect for the system unit that contains the user instance?…

View Comment

grondo commented on issue flux-framework/flux-core#7292.

A quick test on Tuolumne shows this is working in the basic case. I simulated an unkillable process by attaching gdb to the job shell then canceled the job….

View Comment

markcmiller86 commented on issue visit-dav/visit#20752.

Ok, so both the HDF5 and Silo build failures are fixed by specifing -DMPI_HOME:STRING=/path/to/mpi/install/home. Now, Silo of course DOES NOT REQUIRE mpi. But, it does a find_package() on HDF5 and that will fail if the HDF5 it is finding depends on MPI but there is nothing telling Silo where to find MPI. …

View Comment

becker33 commented on issue spack/spack#51835.

@alalazo thanks! I’ve included it now…

View Comment

jeremylt commented on issue openjournals/joss-reviews#8388.

My apologies, I was trying to wait for the libCEED release to finish, but unfortunately that became a much longer wait than I expected. I will proceed one way or another with a new Ratel release by next Friday no matter what libCEED does….

View Comment

wihobbs commented on issue flux-framework/flux-core#7289.

> Actually, would it address this issue if we got rid of the names and affiliations in the list and just left the github IDs? …

View Comment

cyrush commented on issue visit-dav/visit#20782.

We reviewed the error. The logic detects the case where int could cause badness and handles this with an error message. So the generic integer to int logic is ok, we can remove the #error. …

View Comment

alecbcs commented on issue spack/spack#51854.

Yep! Thanks for the suggestion @haampie. This works with ghcr.io/spack/centos7: …

View Comment

hppritcha commented on issue open-mpi/ompi#12979.

some notes on this. this doesn’t seem to be a problem with partial finalization in some ranks verses others. when the hang occurs I observe some process stuck in waiting for completion of the PMIx_fence_nb near the beginning of finalization while others are stuck in some waitall (on several hundred requests per call) within the application….

View Comment

markcmiller86 commented on issue llnl/H5Z-ZFP#159.

> I see. So let me sure I get this right. The use case we’re discussing is when an application like Matlab knows nothing about zfp or H5Z-ZFP (e.g., it has not been linked against those libraries), but instead relies on the HDF5 library to invoke zfp. This is done by having the application/user set the filter number (32013) and enough cd_values to specify zfp compression mode and parameters. The HDF5 library will then, when encountering the filter number, dynamically load the H5Z-ZFP and zfp libraries, which will encode additional cd_values (the zfp header) and then compress the data. Did I get that right?…

View Comment

adayton1 commented on issue llnl/CARE#364.

Can you update the title? Nvm, I got it….

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#162.

@copilot Try this solution:…

View Comment

markcmiller86 commented on issue HDFGroup/hdf5#6153.

It may be useful to adopt a string format standard for stringified filter params instead of allowing each filter to define syntax for the param strings. So, instead of "mode=rate; rate=3.5", a json string "{"mode":"rate","rate":3.5}" or maybe YAML, {mode: rate, rate: 3.5}….

View Comment

jeremylt commented on issue CEED/libCEED#1917.

Thoughts - Do we also want CeedObjectDestroy ? (Easy to add) And do we want the view tabs to be added to the CeedObject? I’m thinking yes for both …

View Comment

vsoch commented on issue vsoch/zenodo-validator#5.

Thanks @ax3l !…

View Comment

bgunnar5 commented on issue llnl/merlin#597.

@doutriaux1 gave the nod of approval to go ahead and merge this…

View Comment

adayton1 commented on issue llnl/axom#1772.

It might be helpful to add some kind of readme containing the compiler errors encountered and which patches fix them with an explanation….

View Comment

adayton1 commented on issue python-gitlab/python-gitlab#3322.

It looks like there is an abandoned attempt: https://github.com/python-gitlab/python-gitlab/pull/2552/files…

View Comment

jbschroder commented on issue pyamg/pyamg#459.

Update: Consider…

View Comment

brendankeith commented on issue mfem/web#305.

Hopefully by tomorrow

adrienbernede commented on issue llnl/radiuss-shared-ci#66.

@tdrwenski could you please add a changelog entry?…

View Comment

astrophpeter commented on issue llnl/popclass#81.

Thanks, can you add a short description in the docs about these new datasets? …

View Comment

grondo commented on issue flux-framework/flux-security#207.

Done. …

View Comment

wihobbs commented on issue codecov/feedback#665.

@asmacdo A while back I tried to chase down a rabbit hole of looking in our organization’s audit log for what event might trigger codecov to run again 90 days later, but we never quite figured it out….

View Comment

acfisher commented on issue mfem/mfem#5116.

@dylan-copeland Thank you for the through review. I believe I have addressed all of your comments and have updated the code accordingly….

View Comment

JustinPrivitera commented on issue llnl/conduit#1496.

we will want this to be as fast as possible, so we will want to write three converters, one from each specset representation to silo. Ignore the comment in the code….

View Comment

tgamblin commented on issue andrew/nesbitt.io#34.

I could add more – lemme know if any of these seem interesting. …

View Comment

hppritcha commented on issue open-mpi/ompi#12278.

is this PR still being worked on or should it be closed?…

View Comment

balos1 commented on issue llnl/sundials#809.

> Or should we switch to the clang-format installed in the github runner image? ubuntu-latest should have 16.0.6, 17.0.6, and 18.1.3 installed. …

View Comment

vsoch commented on issue hpsfoundation/tac#61.

I forgot that we also have the Flux Locator map!

vsoch commented on issue hpc-social/jobs#29.

I still don’t see it in the Google Sheet, and there must be an issue with submitting this form (that is shown in your browser console of that page). To bypass that (and see it more directly) please use the Google Form directly:…

View Comment

garlick commented on issue flux-framework/flux-core#7260.

This test is repeatedly failing in alpine. I thought this was a known test issue but maybe this PR has somehow made it worse. Anyway, further investigation is probably warranted. …

View Comment

chu11 commented on issue flux-framework/flux-core#7272.

list_all() appears to return a tuple of files and directories. So the second list is simply the list of directories in that directory, which in this case is no directories, thus an empty list….

View Comment

cyrush commented on issue visit-dav/visit#20784.

bv test failed b/c the container ran out of space. Sounds like we arrived at a resource wall….

View Comment

markcmiller86 commented on issue visit-dav/visit#20752.

Ok, so that is the build of HDF5-2.0.0 then via bv_hdf5.sh. I thought I copied that logic verbatim from bv_conduit.sh. What was your build_visit command-line and env?…

View Comment

v-dobrev commented on issue mfem/mfem#5134.

One additional issue: the sample run …

View Comment

tzanio commented on issue mfem/mfem#5134.

@IdoAkkerman – can you take a look at this

jandrej commented on issue mfem/mfem#4551.

Closing because this was merged through separate smaller PR’s….

View Comment

cyrush commented on issue llnl/uberenv#124.

closing #155 should be used instead. …

View Comment

cyrush commented on issue llnl/conduit#1495.

``` …

View Comment

tgamblin commented on issue llnl/Umpire#1046.

@adayton1: …

View Comment

cyrush commented on issue llnl/H5Z-ZFP#159.

@markcmiller86 that sounds great to me, much simpler way to provide options…

View Comment

markcmiller86 commented on issue llnl/H5Z-ZFP#159.

@brtnfld I like the idea of HDF5 library offering a string-based way of controlling filters….

View Comment

garlick commented on issue flux-framework/flux-security#207.

It looks like cgroup_info_create() will fail (and this is immediately fatal to the IMP) if we set the cgroup->unified flag wrong, so just running a job and for extra credit, sending it a signal ought be a sufficient test. Correct me if I’m wrong! …

View Comment

chu11 commented on issue flux-framework/flux-security#207.

just re-pushed. I split your change into two commits to mirror my original two fixes (checking the wrong FS, and the not erroring out if the unified dir doesn’t exist). I listed you as co-author in the commit message. So we should get someone else to review and approve….

View Comment

wihobbs commented on issue flux-framework/flux-core#7270.

If you’re a member of the core team on GitHub, you automatically have write/maintainer access to a bunch of the repositories in the flux-framework org (see flux-pam MAINTAINERS.md). …

View Comment

cmoussa1 commented on issue flux-framework/flux-core#7270.

Do we have more than one MAINTAINERS.md document other than flux-core? I’m not sure.

garlick commented on issue flux-framework/flux-accounting#803.

CONTRIBUTING.md was updated in flux-core with the governance and CoC RFCS if you want to pull that in too….

View Comment

vsoch commented on issue flux-framework/.github#5.

No worries! I think repository specific files might be in order given that we have different maintainers in different repositories. We can definitely change our strategy after this initial first shot….

View Comment

garlick commented on issue flux-framework/.github#5.

Oh sorry I missed this and started mindlessly making copies. I guess at least the short term requirement is met and we have space to sort out a longer term plan….

View Comment

cyrush commented on issue Alpine-DAV/ascent#1650.

Does image_prefix help instead of image_name ?…

View Comment

adayton1 commented on issue llnl/Umpire#1046.

Is there some git configuration variable that can be set to avoid all the commits coming from “different” emails?…

View Comment

artv3 commented on issue llnl/RAJAPerf#609.

I think this PR should now be good to go, @rhornung67 I fixed the compiler warnings as well. …

View Comment

tzanio commented on issue GLVis/glvis#335.

ping: @justinlaughlin and @v-dobrev – do you want to take a look? …

View Comment

jeremylt commented on issue CEED/libCEED#1818.

I tried to bully MacOS into respecting the weak symbols here: https://github.com/CEED/libCEED/pull/1913 …

View Comment

adrienbernede commented on issue spack/spack-packages#2858.

@chapman39 this is moving in the opposite direction than https://github.com/spack/spack-packages/pull/1402, as discussed in https://github.com/llnl/axom/discussions/1726#discussion-9196039. …

View Comment

jbschroder commented on issue pyamg/pyamg#459.

I would support …

View Comment

adrienbernede commented on issue llnl/radiuss-spack-configs#158.

@kab163 is this specific to Umpire or does it apply to every project ? (You can update CMake directly in Umpire CI)….

View Comment

adrienbernede commented on issue llnl/quandary#105.

@tdrwenski the GITHUB_TOKEN variable has been replaced by GITHUB_WORKFLOW_TOKEN. the latter should be defined in GitLab UI….

View Comment

amagela commented on issue llnl/benchmarks#23.

I think this was failing because the containers were not being set up with Gnuplot and Matplotlib. I just pushed a change that should, hopefully, alleviate that. Will check on it in the morning. …

View Comment

cjvogl commented on issue mfem/mfem#4496.

@tzanio, this should be ready for re-merging into next

View Comment

chvillanuevap commented on issue humanmade/S3-Uploads#612.

Adding a note here for posterity….

View Comment

samuelkgutierrez commented on issue openpmix/openpmix#3758.

I would advocate for leaving the comments alone unless there is a compelling reason to remove them. Since the code related to the comments was left untouched, I would personally leave those comments alone. …

View Comment

vladotomov commented on issue mfem/mfem#5176.

> Can you just remove static from the convection and DG trace integrators and see if that works in Remhos? If so, that is a better solution than adding the specializations here. …

View Comment

hppritcha commented on issue open-mpi/ompi#13605.

ignoring borked up NVIDIA CI….

View Comment

chu11 commented on issue flux-framework/flux-core#7194.

@garlick thanks! will set MWP…

View Comment

becker33 commented on issue spack/spack#51746.

We discussed passing these as tuples instead of dictionaries. The methods called here are not called anywhere else, so it’s fine to change their signatures. …

View Comment

tzanio commented on issue mfem/mfem#5134.

@mlstowell, can you take a quick look when you get a chance?…

View Comment

hppritcha commented on issue spack/spack-packages#2900.

This PR is incorrect. The UCX developers botched up the mail about 1.20.0. it was actually for an RC not a release. That’s the the URL is different. Please do not merge this PR….

View Comment

artv3 commented on issue llnl/RAJAPerf#610.

Super nice! I’m super interested in trying out this feature, if I can get a day or two I can give some user experience feedback. …

View Comment

artv3 commented on issue llnl/RAJA#1969.

Super nice! Can you add something in the RAJA docs about this new feature? And what do folks think about adding a note about this in our RELEASE_NOTES in this PR. …

View Comment

garlick commented on issue flux-framework/flux-core#7254.

OK now that I have the next phase just about working, I’ll drop the WIP here. …

View Comment

JustinPrivitera commented on issue llnl/conduit#1322.

Documentation and implementation done in #1487 so that we are not waiting for this from the device support PR….

View Comment

vsoch commented on issue kubeflow/trainer#2909.

Thank you @andreyvelich @tenzen-y - the review was excellent, and I’m glad to see this moving through. It might be a bit early, but Happy New Year! I appreciate everything you do for the Kubeflow (and larger Kubernetes) communities….

View Comment

jeremylt commented on issue freeCodeCamp/freeCodeCamp#64929.

Is there an issue associated with this change? If so please add it. If not, please create an issue before making a pull request….

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#798.

Thanks @garlick! Setting MWP here…

View Comment

vsoch commented on issue compspec/fractale-mcp#4.

cc @knglugo @milroy autogen is added as an engine, to join the default “native” state machine. I’ve created sample plans for building a docker container and translating a jobspec. See design in the README for the abstractions. The tools are now separate libraries too: …

View Comment

hppritcha commented on issue open-mpi/ompi#12658.

do we still want to pull in this PR? …

View Comment

imguoguo commented on issue sipeed/NanoKVM-USB#6.

fixed by https://github.com/sipeed/NanoKVM-USB/pull/83, you can try the latest version…

View Comment

vsoch commented on issue kubeflow/trainer#2841.

This is a WIP and should not be closed. Thanks!…

View Comment

garlick commented on issue flux-framework/flux-core#7256.

Thanks! Setting MWP…

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#796.

Thanks @jameshcorbett! Setting MWP here…

View Comment

chvillanuevap commented on issue humanmade/S3-Uploads#612.

I don’t mean to reopen this issue, but I ran into the same problem recently while searching for a solution, and I wanted to share the patch I ended up using in case it helps someone else….

View Comment

tgamblin commented on issue archspec/archspec#225.

In memory. This isn

tgamblin commented on issue spack/spack#51781.

@haampie: with the change I added I get similar performance to your removal, so I went ahead and updated the commit message and merged. Feel free to revert if you think there are still issues….

View Comment

adayton1 commented on issue llnl/CARE#350.

I want to merge this in before I release….

View Comment

gauthier12 commented on issue lapce/lapce#3592.

I confirm the bug….

View Comment

alecbcs commented on issue spack/spack-packages#2836.

@spackbot run pipeline…

View Comment

jandrej commented on issue mfem/mfem#4908.

Superseded by merged PR…

View Comment

camierjs commented on issue GLVis/glvis#339.

Thanks for testing this out, @hughcars

garlick commented on issue flux-framework/flux-core#7251.

Fixed those things, and made another pass, expanding the slurm migration section to include more information. …

View Comment

cyrush commented on issue visit-dav/visit#20768.

@biagas can you try out the new mp4 logic to see if that helps with #20748 ?…

View Comment

tgamblin commented on issue spack/spack#51781.

So, I was hoping to use protocols to make typing of spec data structures much easier. This doesn’t bode well for that, so I guess I’ll ask some questions: …

View Comment

hppritcha commented on issue open-mpi/ompi#13583.

@jsquyres check now. fixed typo inherited from slurm page too….

View Comment

dylan-copeland commented on issue mfem/mfem#5123.

Thank you @IdoAkkerman for your careful review. There were many important issues raised in this review, which are hard to find in the resolved comments. Here is a list to summarize some issues to revisit later. …

View Comment

balos1 commented on issue llnl/sundials#806.

Thanks for the report. This header is unused, so I have removed it entirely in #807….

View Comment

cyrush commented on issue llnl/conduit#1438.

@mlohry these jobs failed b/c the windows image in ci was removed around the time of testing. If we merge develop into your branch, we can fully test…

View Comment

adayton1 commented on issue llnl/axom#1599.

This is currently a blocker for us having any CUDA builds….

View Comment

adayton1 commented on issue llnl/Umpire#1033.

> It seems there are some C++20 compatibility issues in the current version of fmt, so will see how updating to 11.1.0 works …

View Comment

artv3 commented on issue llnl/RAJAPerf#599.

How big can meshes get? I’m wondering how the meshes will impact the repo size….

View Comment

artv3 commented on issue llnl/RAJA#1964.

Hey this looks pretty cool and simpler than we were originally trying, let me try it within the RAJA PerfSuite and I can get back to you …

View Comment

jeremylt commented on issue freeCodeCamp/freeCodeCamp#64787.

How about:…

View Comment

trws commented on issue flux-framework/flux-sched#1423.

Doing it in that literal order is impossible. We may be able to get you to something like what you want, but let me lay out the situation a bit so you know where I’m coming from. In order to give a predicted start time we need to run a match against the current resource state, allocations and reservations to determine when the job would be reserved. Without doing that match, any result we give out would be even more wrong than without doing it (even doing the match is still an estimate at best due to new job arrival and constant state changes)….

View Comment

chu11 commented on issue flux-framework/flux-core#7191.

Continued prototyping with the eventlog in this format as I describe above: …

View Comment

vsoch commented on issue rseng/jobs-updater#12.

Thanks @crd477 ! …

View Comment

adrienbernede commented on issue llnl/uberenv#154.

I have the same error on my end with quandary….

View Comment

adrienbernede commented on issue llnl/radiuss-spack-configs#155.

@davidbeckingsale building went well. (Be mindful that the tests are not run). …

View Comment

uvilla commented on issue hippylib/hippylib#84.

@finsberg: Thank you for your contribution. …

View Comment

vsoch commented on issue flux-framework/flux-core#7217.

A contributor just needs to be able to find “how to contribute” quickly. The CONTRIBUTING.md is (to me) preferred over what I had here, which was a workaround to not having one. …

View Comment

grondo commented on issue flux-framework/flux-core#7241.

Thanks! I’ll set MWP and perhaps @ryanday36 could give feedback and open a new issue if there needs to be adjustments or improvements in the future….

View Comment

wihobbs commented on issue flux-framework/flux-core#7245.

James is definitely also someone who’s approved and merged my PRs as a maintainer. …

View Comment

vsoch commented on issue converged-computing/usernetes#2.

@AkihiroSuda if you are interested, we were testing calico for usernetes https://github.com/converged-computing/usernetes/tree/test-calico (hoping to not need to set net.ipv4.conf.default.rp_filter to 2) and got it deployed (calico pods running), but the connectivity between pods isn’t there yet. I think packets are being dropped. Maybe by the kernel (which could be related to that setting still being 2). We were trying to use vxlan although I’m not sure if that is the right path, and if we can test with the rp_filter setting still at 2….

View Comment

tgamblin commented on issue spack/spack-packages#2709.

@olesenm: welcome back! See: …

View Comment

becker33 commented on issue spack/spack#51422.

Can spack -E commands update-completion do what you need for this?…

View Comment

adrienbernede commented on issue llnl/Umpire#1012.

@davidbeckingsale this PR is now ready….

View Comment

adrienbernede commented on issue llnl/RAJA#1935.

@rhornung67 This PR is now ready for review….

View Comment

adayton1 commented on issue llnl/CHAI#341.

> @adayton1 #343 should be merged first. …

View Comment

adrienbernede commented on issue llnl/CHAI#341.

@adayton1 https://github.com/LLNL/CHAI/pull/343 should be merged first….

View Comment

tgamblin commented on issue hpsfoundation/tac#61.

I mean if you have something like, say, Flux Operator that has a name with “Flux” in it, it needs to either be renamed to something like “Kubernetes Operator for Flux” (in compliance with the doc I linked above), or it needs to be in the Flux org as part of the official upstream Flux project….

View Comment

alecbcs commented on issue spack/spack-packages#2365.

@tgymnich @vchuravy @wsmoses @amd-toolchain-support any objections here? …

View Comment

tzanio commented on issue mfem/mfem#5107.

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….

View Comment

v-dobrev commented on issue mfem/mfem#5122.

Both mfem+umpire and mfem~umpire should work well – try it out to see which one works best. Here’s how this is done in Laghos: https://github.com/CEED/Laghos/pull/198….

View Comment

vladotomov commented on issue CEED/Laghos#201.

@rfhaque Let me know when it’s done….

View Comment

markcmiller86 commented on issue visit-dav/visit#20752.

I added logic to silo plugin to provide a little more info about the file (and plugin) software versions… …

View Comment

tgamblin commented on issue llnl/axom#1740.

The axom package looks way better (160 added lines, nearly all declarative vs. 242 in the first revision with custom consistency checking). We can prioritize improving the error messages for package requirements in Spack….

View Comment

garlick commented on issue flux-framework/flux-foundry#20.

@grondo any thoughts on this one? This should be sufficient to get tomlc99 amd tomltk.c out of flux-core. More work would be needed for flux-security….

View Comment

markcmiller86 commented on issue spack/spack-packages#2346.

@tldahlgren I noticed you had some recent activity on an MFEM PR here and I wanted to reach out and ask if your (or someone you might wanna recommend) could merge this PR for Silo?…

View Comment

v-dobrev commented on issue spack/spack-packages#143.

Please do not merge yet. I’ll review this soon – I’ll do it right after I’m done with the PR to add and test the new v4.9. (For now I just have the initial commit for that pushed: https://github.com/spack/spack-packages/tree/mfem-4.9.)…

View Comment

tzanio commented on issue mfem/mfem#5153.

Merged in next for testing……

View Comment

garlick commented on issue flux-framework/flux-core#7238.

Thanks - setting MWP…

View Comment

garlick commented on issue chaos/diod#164.

My test setup:…

View Comment

cyrush commented on issue visit-dav/visit#20752.

@markcmiller86 I think we should try the new version, would be good to remove ancient patches and have same code used on windows as linux if possible. …

View Comment

becker33 commented on issue spack/spack#51731.

I’m not convinced “every node is a separate object” is a good assumption in abstract specs, even without this bug. But that can be a separate conversation….

View Comment

jandrej commented on issue mfem/mfem#5151.

```…

View Comment

cmelone commented on issue LLNL/hubcast#190.

> I’d vote for the 1st option, but let’s make it optional via a config. Some users might want to disable draft PRs and not get a message from Hubcast while others probably do want a place holder status check with a message. …

View Comment

adrienbernede commented on issue LLNL/RAJAPerf#587.

@rhornung67 This PR is ready. I reduces the number of specs running in CI (shared specs in RSC have been updated) and clearly deactivates some of the shared specs to focus on specs for interest for RAJAPerf….

View Comment

cyrush commented on issue Alpine-DAV/ascent#1636.

Ok, I understand. If someone takes over they can change the version to a malicious commit. …

View Comment

trws commented on issue vsoch/hpsf-tac#1.

Depending on what you need, it also often works well to include just user-mode flux (core and sched is often enough) inside the container, then in the allocation use srun to launch a flux instance with all brokers running in the container with your optimizer running as the initial program. It requires changing the container, where the sidecar does not, but avoids the need for kubernetes to use the operator. It

hppritcha commented on issue open-mpi/ompi#13460.

bingo! now after a little bit of rebasing fixes I can get #13280 to pass mpi4py declaring OMPI is MPI 5.0 compliant and not needing to exclude the big count tests! …

View Comment

v-dobrev commented on issue mfem/mfem#5057.

Merged in next for testing….

View Comment

acfisher commented on issue mfem/mfem#4689.

@tzanio it looks like this ended up getting 2 approvals and then never pulled in. I am resurrecting it and want to shepard it through….

View Comment

uvilla commented on issue hippylib/hippylib#83.

Thank you @dc-luo. I agree with the proposed class renaming. We could …

View Comment

trws commented on issue flux-framework/rfc#487.

Got it, thanks!…

View Comment

jameshcorbett commented on issue flux-framework/flux-sched#1398.

Yes I think so!…

View Comment

wihobbs commented on issue flux-framework/flux-sched#1398.

@jameshcorbett I think we got packages into TOSS 5 but correct me if I’m wrong, ok to close this one?…

View Comment

grondo commented on issue flux-framework/flux-core#7220.

After implementing the noverify = ["core"] support, I realized this approach does not cleanly handle the case for falling verification when more resources than configured are discovered….

View Comment

cyrush commented on issue LLNL/conduit#1482.

``` …

View Comment

adrienbernede commented on issue LLNL/RAJA#1955.

> This also needs to be turned on in the GItLab UI, correct? …

View Comment

acfisher commented on issue mfem/web#204.

@tzanio ok I put my changes in, so I think this is ready for another reviewer….

View Comment

markcmiller86 commented on issue llnl/H5Z-ZFP#155.

@brtnfld and @jhendersonHDF thanks for submission and review. @brtnfld I think you have authority to merge at this point…or, you should. …

View Comment

milroy commented on issue flux-framework/kubeflux#1.

This came up in the Flux meeting today. We can delete this repo….

View Comment

milroy commented on issue flux-framework/flux-sched#1419.

I took a first pass on this PR, and it generally looks good. I did post a question about the function of a parameter. …

View Comment

markcmiller86 commented on issue LLNL/H5Z-ZFP#155.

@brtnfld and @jhendersonHDF thanks for submission and review. @brtnfld I think you have authority to merge at this point…or, you should. …

View Comment

v-dobrev commented on issue xsdk-project/xsdk-issues#280.

I just merged the Spack PR, so hopefully this issue is now resolved….

View Comment

jameshcorbett commented on issue vsoch/hpsf-tac#1.

> @jameshcorbett

imguoguo commented on issue sipeed/NanoKVM-USB#89.

Bug(fixed):

wihobbs commented on issue sam-maloney/flux-core#1.

Another thought I just had is perhaps the plugin should be able to slurp up json/yaml files with a resource specification in them based on RFC 14 in addition to shape. But maybe that should be a separate plugin/option….

View Comment

cjvogl commented on issue mfem/web#291.

> Along this line: for my version of hipcc I manually specified the MPI include directory for METIS and added -Wno-implicit-int and -fPIC to get it to work. …

View Comment

chu11 commented on issue flux-framework/rfc#482.

I don’t think there are any gotchas, will set MWP…

View Comment

adayton1 commented on issue LLNL/axom#1599.

> I noticed that Axom+CUDA is built in a funny mode where nvcc appears to use a host compiler. If I change the host-config to CUDA 12.2..2 and the host compiler to newer clang then it gets farther….

View Comment

samuelkgutierrez commented on issue hpc/quo-vadis#424.

Perhaps the environment variable can be a CPU mask that encodes the allowed resources. hwloc-calc can help sites generate this….

View Comment

BradWhitlock commented on issue LLNL/axom#1732.

@bmhan12 - I ran into “yaml_parser->problem is NULL (missing)” recently in some other work when my data submodule was not up to date. When I went in and made sure it had “main” checked out rather than a detached HEAD, it worked. I don’t know how it got into the detached HEAD state though….

View Comment

v-dobrev commented on issue mfem/mfem#4977.

Adding the tolerance for changing dt did not actually change the result since the actual difference in dt on the last step is slightly above 2.7569e-15 (this is for the sample run I posted above). However, this is probably fine. …

View Comment

vsoch commented on issue flux-framework/rfc#481.

Actually I don’t see your feedback yet @grondo - let me know if you have thoughts on the above!…

View Comment

grondo commented on issue flux-framework/flux-docs#309.

Thanks!…

View Comment

trws commented on issue flux-framework/flux-core#7191.

Honestly I just didn’t know that, that makes me think that we might want to expand on that rather than going from that to a big object though. We’ve already seen some scaling issues on large json objects in other places, and JGF is likely to get that treatment at some point, so may as well keep it that way here. Still a good idea to be able to get the events efficiently by type, but keeping it incremental would be good….

View Comment

grondo commented on issue flux-framework/flux-core#7191.

> Kinda orthogonal, and I

garlick commented on issue flux-framework/flux-core#7227.

Having a way to run a local health check in rc1 sounds like it would be useful. I think if rc1 fails the broker exits with the special exit code that inhibits restart….

View Comment

chu11 commented on issue flux-framework/flux-core#7191.

A random thought I had tonight, regarding my idea 1 and idea 2 that I listed above. …

View Comment

vsoch commented on issue flux-framework/flux-core#7229.

Will need guidance on the mac-os test - not sure if it’s flaky (and I can’t figure out from the output what it isn’t happy about). …

View Comment

grondo commented on issue flux-framework/flux-accounting#787.

It would have to be available per job, since there could be a different limit for different banks in the flux-accounting case. …

View Comment

garlick commented on issue flux-framework/flux-accounting#787.

I suppose another option is to allow the jobs into sched state but withhold them from the scheduler. That is already available except not per user….

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#159.

@copilot Please remove these sections or add appropiate documentation under the section Documentation from the README …

View Comment

cyrush commented on issue visit-dav/visit#20729.

@Sam-Briney I tried with develop version of visit, and I see the same values. …

View Comment

alecbcs commented on issue spack/spack-packages#2679.

@spackbot run pipeline…

View Comment

gardner48 commented on issue spack/spack-packages#2557.

@jrood-nrel does https://github.com/spack/spack-packages/pull/2679 fix the problem?…

View Comment

adrienbernede commented on issue spack/spack-packages#2558.

@alecbcs @davidbeckingsale …

View Comment

becker33 commented on issue spack/spack#7890.

This appears to have been fixed at some point in the intervening years. Tested with spack install zlib tcl which does not print output for zlib twice….

View Comment

vsoch commented on issue singularityhub/singularity-hpc#694.

That looks like an issue with parsing the module. Take a look at the module file that shpc generates and see how it compares to modules on your system (and make sure you are using the right one)….

View Comment

vsoch commented on issue pydicom/deid#295.

https://pypi.org/project/deid/0.4.10/…

View Comment

jedbrown commented on issue openjournals/joss-reviews#6099.

I think this is really an author preference matter. In general, I’d say benchmarking the official version is best. I’m not sure how much performance is expected to have changed from earlier, nor whether it’s the same machine. For dense operations, the relative performance of MKL to BLIS and OpenBLAS is routinely benchmarked….

View Comment

tzanio commented on issue mfem/web#310.

Thanks @nmnobre !…

View Comment

acfisher commented on issue mfem/web#204.

@tzanio There is some good work in here and I think we should rescue it. I can make the couple of changes that I suggested and dust this off if you want to provide another review/reviewer. What do you think?…

View Comment

tzanio commented on issue mfem/mfem#4986.

Re-merged in next for testing……

View Comment

vladotomov commented on issue mfem/mfem#4986.

Update CHANGELOG…

View Comment

chu11 commented on issue flux-framework/rfc#482.

re-pushed, adding backwards compatibility. I wasn’t sure how to organize things. So I just added a blurb at the end of the section saying backwards compatibility MAY be supported via the old response format….

View Comment

trws commented on issue flux-framework/rfc#481.

Dang, it sounded like such a good idea, and it looks like it works for HPCToolkit on gitlab. It looks like roles aren’t easy to make visible on github either. I suppose a list is fine, as long as it’s a single place, that’s where lists burn is if they have to be maintained in multiple places without a way to ensure they stay in sync….

View Comment

adrienbernede commented on issue LLNL/radiuss-spack-configs#152.

@tdrwenski thank you for the review. Since @kab163 really needs it and I have no problems with my projects, I say we merge it and further investigate your issues. If we need a patch, no problem with me creating a minor release….

View Comment

artv3 commented on issue LLNL/RAJAPerf#589.

TODO: remove Bt as it is not used. …

View Comment

adayton1 commented on issue LLNL/RAJAPerf#592.

> > What does google benchmark do to prevent things being optimized away? …

View Comment

jeremylt commented on issue CEED/libCEED#1905.

In every instruction vpd though? We’ve run into situations I think where most but not all of the QF is vpd…

View Comment

v-dobrev commented on issue xsdk-project/xsdk-issues#280.

What compiler is this using? I can’t figure out what’s behind mpic++….

View Comment

cmelone commented on issue spack/spack-gantry#208.

@dependabot rebase…

View Comment

hppritcha commented on issue open-mpi/prrte#110.

Could you show the error message mpirun is reporting? Also what configure options were you using? …

View Comment

cjvogl commented on issue mfem/mfem#5126.

Pinging some people who may be interested in this PR: @jandrej and @nmnobre …

View Comment

grondo commented on issue flux-framework/rfc#482.

I’m talking about one version of Flux talking to another, e.g. as would happen with flux top or flux pstree if a subinstance is launched using a different version. We’ve been pretty good about keeping interfaces compatible up to this point….

View Comment

garlick commented on issue flux-framework/rfc#482.

Wouldn’t most users of this RPC (apart from job-list) be using the python API which would also be updated? As I type that I am doubting myself. …

View Comment

wihobbs commented on issue flux-framework/flux-core#7211.

That’s great! We tend to be a heavily GitHub driven project, so this + other issues and PRs for sure. We also lean on Slack, and I sent you an invite to our workspace for collaboration and quick questions….

View Comment

trws commented on issue desul/desul#160.

Yup, that transition is officially upstream for us, and I think 20 will follow soon….

View Comment

garlick commented on issue chaos/diod#164.

FYI I’m going offline for a few days for travel but next week I’ll try to set up a 32 bit test environment to run this down….

View Comment

alecbcs commented on issue LLNL/hubcast#190.

No worries haha :)…

View Comment

adrienbernede commented on issue LLNL/Umpire#1009.

@kab163 OK, adding the camp release and updating the umpire dependency constraint now….

View Comment

adayton1 commented on issue LLNL/CARE#357.

Can you add a release note about these fixes?…

View Comment

jedbrown commented on issue CEED/libCEED#1905.

In particular, the v*pd (“packed double”) instructions, which I expect GCC is indeed emitting. I like to use Linux perf and then look at the hot instructions. If those are v*pd, then vectorization has been effective. What does upstream clang do on your kernels? Is there a reason you are using legacy icc?…

View Comment

milroy commented on issue compspec/fractale#21.

Alright, the latest updates are pushed. Let me know if this needs anything else….

View Comment

benzwick commented on issue Slicer/Slicer#8883.

What if when multiple segments are selected the tools that only work on single segments are disabled and “grayed out”. I think this would be quite intuitive and easy for users to understand….

View Comment

adayton1 commented on issue LLNL/axom#1723.

@kennyweiss, @rhornung67, I don’t have permission to merge, so one of you will have to push the button….

View Comment

jeremylt commented on issue freeCodeCamp/freeCodeCamp#64241.

Please use the forum to request help on the curriculum …

View Comment

bgunnar5 commented on issue LLNL/merlin#596.

@doutriaux1 I can. The only thing that’s really changing here is the backend functionality and the --steps option being removed. From the frontend, the user can still use merlin monitor ${YAML} with the exact same call….

View Comment

artv3 commented on issue LLNL/RAJA#1949.

> Any thoughts on blockIdx and gridDim? …

View Comment

adayton1 commented on issue LLNL/MPIDiff#11.

Could you add a release note and tick the version number in CMakeLists.txt?…

View Comment

jedbrown commented on issue EnzymeAD/Enzyme#2504.

I’m assuming the audience that cares about repo size will always be using a shallow clone? I just want to make sure everyone here knows it’s not possible to actually shrink the (non-shallow) repo size without invalidating old hashes. It is possible to split the history, but it changes hashes, which is confusing for people who have clones lying around and disrupts workflows that depend on existence of old hashes. I have experience doing such things, though it isn’t a decision to make lightly and I’m not recommending it….

View Comment

emily-howell commented on issue Alpine-DAV/ascent#1615.

Sure thing!…

View Comment

trws commented on issue vsoch/hpsf-tac#1.

> > This is spot on - afaik there is no central governance model or document (beyond a brief contributing guide). This is something we need to put together before we submit this to the upstream repository as an issue for review. I am suggesting that we make a central .github repository for all projects to share a single source of truth. …

View Comment

jedbrown commented on issue CEED/libCEED#1905.

It would depend on the loop body. For example, if it calls functions that don’t vectorize (we did the series work with log1p because it doesn’t vectorize in common implementations) or has branches that are unsafe/hard for the compiler to transform into masked vector arithmetic….

View Comment

vsoch commented on issue vsoch/hpsf-tac#1.

> I will ask to reserve that date for you during next TAC meeting. …

View Comment

garlick commented on issue vsoch/hpsf-tac#1.

> This is spot on - afaik there is no central governance model or document (beyond a brief contributing guide). This is something we need to put together before we submit this to the upstream repository as an issue for review. I am suggesting that we make a central .github repository for all projects to share a single source of truth. …

View Comment

grondo commented on issue flux-framework/flux-core#7215.

Thanks! …

View Comment

trws commented on issue vsoch/hpsf-tac#1.

Given this state, maybe it would be useful to do just a couple of tweaks in this PR then merge so we can collaborate via further PRs? Guess it depends on what all we want/need to do before trying to move this forward….

View Comment

imguoguo commented on issue sipeed/NanoKVM-USB#82.

Does it manifest as a dialog box prompting you to select a serial device, but then displaying “Failed to connect serial. Please try again” after selection? If so, it’s likely because the current user doesn’t have permission to connect to the serial port. You need to add the current user to the uucp group (this worked successfully on my Arch Linux) or the dialout group (this might be necessary on Fedora or Ubuntu)….

View Comment

vsoch commented on issue pydicom/deid#294.

I’m not sure I’m following - so let’s say you have a dicom with some set of Rows/Columns:…

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#156.

@copilot Add interpolation tests on 8x8 grids. Also add tests to verify the case of one cell meshes. Across all geometries for K = 3….

View Comment

markcmiller86 commented on issue LLNL/Silo#503.

Resolved in #505 on 4.12RC and #508 on main….

View Comment

alecbcs commented on issue FrameworkComputer/EmbeddedController#73.

I’m also experiencing this issue on an AMD Framework 13 when using gnome-adaptive-brightness (issue linked above)….

View Comment

v-dobrev commented on issue xsdk-project/xsdk-issues#279.

> BTW: I’m not yet sure if I can use hypre-3 in xsdk - due to mfem dependencies [as there is no mfem release with hypre-3.0 fixes]. cc: @v-dobrev

View Comment

JustinPrivitera commented on issue visit-dav/visit#20666.

@IdoAkkerman I have added your changes here: #20737 …

View Comment

v-dobrev commented on issue mfem/mfem#4028.

Re-merged in next for testing……

View Comment

becker33 commented on issue spack/spack#51566.

Closing this – now that we’re not trying to get it in for v1.1.0, we have time for a better solution….

View Comment

hppritcha commented on issue openpmix/openpmix#3722.

this is the commit that tag 5.0.10rc1 points to …

View Comment

hppritcha commented on issue open-mpi/ompi#13484.

okay i got this to run but i don’t observe this heap-use-after-free warning in the voluminous output….

View Comment

tzanio commented on issue mfem/mfem#4996.

Merged in next for testing……

View Comment

benzwick commented on issue benzwick/StreamVorti#10.

Thanks @will-wz-li this fixed the problem. I updated the build badges on the README.md as well. …

View Comment

adrienbernede commented on issue spack/spack-packages#2558.

@spackbot re-run pipeline…

View Comment

tgamblin commented on issue spack/spack#51633.

@haampie: I think -p should do exactly this for the new installer, with default of 0 (for which p == j), and then we should just allow j to exceed p (I think we currently cap it, right?)…

View Comment

wihobbs commented on issue flux-framework/flux-core#7211.

Great summary, thanks @kush-gupt! And it was nice to meet you at SC25….

View Comment

grondo commented on issue chaos/pdsh#171.

Looks good, thanks!…

View Comment

markcmiller86 commented on issue spack/spack-packages#2346.

Just wanted to give praise for all of the CI checks here :clap: :tada:. Although it was sometimes a bit difficult to paw through the CI logs to diagnose failures, I was able ultimately to find and fix many issues without much hand-holding from other busy developers. …

View Comment

dylan-copeland commented on issue mfem/mfem#4936.

Should this PR have the “bug” label? If so, I’m not sure what the bug is….

View Comment

jandrej commented on issue mfem/mfem#5120.

I’m loading cuda/12.9.1 and use clang version 21.1.5 to build mfem with debug flags. The generated output file is ~15MB. I can share that with you internally if needed….

View Comment

adrienbernede commented on issue LLNL/Umpire#1009.

@kab163 Let me know if you need me to merge https://github.com/LLNL/radiuss-spack-configs/pull/150. I

vladotomov commented on issue CEED/Remhos#47.

@justinlaughlin is there a more recent version to use or do we merge this?…

View Comment

vladotomov commented on issue CEED/Laghos#195.

Hi @lpottier, sorry we missed this….

View Comment

markcmiller86 commented on issue spack/spack-packages#2346.

@patrickb314 sorry for false alarms…I discovered I was missing the compression lib options in the CMake builds….

View Comment

camierjs commented on issue mfem/mfem#5120.

In serial, CUDA mode, compiling fem/integ/bilininteg_vecmass_pa.cpp does not show up the issue so far:…

View Comment

vsoch commented on issue flux-framework/flux-sched#1420.

#$#%# I just ran pre-commit on the entire flux-sched codebase… …

View Comment

garlick commented on issue flux-framework/flux-core#7209.

Thanks, fixed those and I’ll set MWP….

View Comment

grondo commented on issue chaos/pdsh#171.

Meh, the commit validator is complaining that the commit message subject is too long. Not a huge deal, but you could reduce it to something like just …

View Comment

emily-howell commented on issue Alpine-DAV/ascent#1621.

Looks like the only thing that failed was the dray-cuda issue! So that’s a win at least. …

View Comment

JustinPrivitera commented on issue visit-dav/visit#20665.

> I am wondering if this PR conflicts with PR #20666. I will try to suggest changes to PR #20666, so this PR might be dropped?? …

View Comment

alecbcs commented on issue spack/spack#51617.

Looks good to me. …

View Comment

hppritcha commented on issue open-mpi/ompi#13520.

would you mind reconfiguring with the --enable-debug option and report the traceback? should help narrow down which opal_argv_join is causing the behavior. Also you may want to allocate a 2-element array of char ptrs and set the argv1[1] to NULL and see if that changes the behavior….

View Comment

tzanio commented on issue mfem/mfem#5115.

This PR has been updates, so the diffs are much smaller now and should be easier to review. …

View Comment

dylan-copeland commented on issue mfem/mfem#5054.

Commit 49cadecc82765de3d928ac5cdafebae921e4acb1 changed invS to CGSolver and then commit b145001e800a7359f7737844019af5a62063bc05 changed it back to GMRESSolver. It seems CGSolver is necessary to make the sample runs fast. Is there a good reason to have GMRESSolver? I changed it to CGSolver and found that the manufactured solution test nurbs_stokes -m ../../data/square-nurbs.mesh -o 2 -c 3 -sli 100 results in error on the order of 1e-6, which seems accurate enough and is much faster….

View Comment

jameshcorbett commented on issue flux-framework/flux-sched#1414.

Based on recent testing on El Cap, it looks like the first three checkboxes will not be enough to enable JGF there. At some point last week, the El Cap system instance was loaded temporarily with JGF, and the whole instance became unresponsive (see e.g. https://github.com/flux-framework/flux-core/issues/7207). I believe this is still being investigated on the core side….

View Comment

grondo commented on issue flux-framework/flux-core#7210.

Perhaps this should be obvious, but I’d like to understand the rationale for moving conf_bootstrap.c into libflux-core.so as a non-public function. What are the tradeoffs compared to keeping it in src/broker? …

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#150.

@copilot I have implemented various fixes for the code you provided. The tests seem to pass but I fear that they don’t test the correct behaviour. Please try to verify correctness of the code. …

View Comment

brendenpetersen commented on issue NVIDIA-NeMo/Guardrails#1409.

+1 to this, major blocker for me!…

View Comment

adayton1 commented on issue LLNL/camp#191.

I would prefer that formatting changes be in a different pull request….

View Comment

adrienbernede commented on issue LLNL/Umpire#1003.

Replaced by #1017 : matrix job required a change in RADIUSS Spack Configs which is in the release candidate….

View Comment

adrienbernede commented on issue LLNL/RAJAPerf#529.

@tdrwenski I think we now need to turn on pages on LLNL/RAJAPerf. I don’t know which setting you used for your fork: branch or actions ?…

View Comment

adayton1 commented on issue LLNL/RAJA#1947.

Do we know why omp::mutex was used instead of std::mutex to begin with? Was the omp version more performant?…

View Comment

cyrush commented on issue Alpine-DAV/ascent#1621.

Looks like dray has some cuda issues, those would be unrelated to viskores, but may side effect of container update, I’ll take a look…

View Comment

adrienbernede commented on issue spack/spack-packages#1402.

@alecbcs I think this is ready. Just need to wait for you to be back from SC ;)…

View Comment

alecbcs commented on issue spack/spack-packages#2512.

@fsimonis has this patch been upstreamed to precice yet?…

View Comment

v-dobrev commented on issue mfem/mfem#5072.

Re-merged in next for testing……

View Comment

cyrush commented on issue LLNL/conduit#1479.

@mathstuf yes, that sounds like a much nicer way to formalize this …

View Comment

adrienbernede commented on issue LLNL/RAJA#1946.

Thanks for the details:…

View Comment

jedbrown commented on issue CEED/libCEED#1905.

> But, for the QFunction for loop, that would be 8 quadrature points per AVX register, not 8 elements. Is the idea just to keep the number of quadrature points divisible by 8?…

View Comment

chu11 commented on issue flux-framework/flux-core#7190.

hmmm, for backwards compatibility, should we stream only if the request is a streaming rpc?…

View Comment

BradWhitlock commented on issue LLNL/axom#1724.

@rhornung67 - I agree this is probably rare. Calling the Umpire function to make the resource manager is a more explicit workaround. Still, I ran into it because Umpire is not front and center in Axom so I did not realize I might run into problems calling axom::copy from an OpenMP loop. I thought I’d note the issue. I could also switch to memcpy() in this instance….

View Comment

becker33 commented on issue spack/spack-tutorial#505.

Closing in favor of #502, which includes the same commit…

View Comment

samuelkgutierrez commented on issue hpc/quo-vadis#411.

Perfect, @GuillaumeMercier. Thank you!…

View Comment

adayton1 commented on issue LLNL/spheral#430.

Mike’s already working on similar and more effective changes….

View Comment

alecbcs commented on issue LLNL/repligit#106.

Can you include a full reproducer performing an ls_remote() on the GitHub side as well?…

View Comment

becker33 commented on issue spack/spack#51587.

We will need to cherry-pick this to the release branch and add it to the release docs…

View Comment

tzanio commented on issue mfem/mfem#5115.

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….

View Comment

vsoch commented on issue jlowellwofford/sc25-flux-eks#6.

Going to merge so we have the base there, and we can always tweak. Bon voyage everyone! :airplane: …

View Comment

grondo commented on issue flux-framework/flux-security#209.

Ok, the CI failures are fixed here now. The necessary changes were: …

View Comment

markcmiller86 commented on issue visit-dav/visit#20651.

> I’m not sure how that resolves this situation. The code is not using DBAll to turn off the read mask flags. It is just using DBAll to turn everything back on after the read, and that part never stopped working. …

View Comment

eliasboegel commented on issue tracel-ai/cubecl#1040.

I would also be extremely interested in this….

View Comment

alecbcs commented on issue spack/spack-bootstrap-mirrors#46.

Completed in #51 …

View Comment

garlick commented on issue flux-framework/flux-core#6457.

Apparently an experiment was just done with ~250MB JGF on El Cap that went poorly…

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#782.

Thanks @jameshcorbett!…

View Comment

cbritopacheco commented on issue cbritopacheco/rodin#145.

@copilot I want to support building the dof arrays for space K>= 2 for the lower dimensional entities….

View Comment

balos1 commented on issue LLNL/spheral#428.

> Thanks for the help. I can confirm that adding the UCX library directory as an additional rpath did the trick….

View Comment

hppritcha commented on issue spack/spack-packages#2396.

never mind my bad. use the numerical sequence that you get when the package is installed. …

View Comment

cyrush commented on issue spack/spack-packages#143.

@tldahlgren dray is no longer developed as a stand alone package, and wont be using newer mfem versions. Given this – I support whatever the best path is to not block this pr. …

View Comment

tgamblin commented on issue spack/spack-packages#2391.

@jwillenbring @balay @victorapm do you have recommendations for how to fix this?…

View Comment

hppritcha commented on issue open-mpi/ompi#13514.

@AntonDaumen would you mind opening a PR for the v6.0.x branch based on a cherry-pick of this commit?…

View Comment

camierjs commented on issue mfem/mfem#5059.

The isf solver has been moved into a new miniapps/fluids subdirectory as suggested….

View Comment

markcmiller86 commented on issue LLNL/Silo#445.

@kmorel thanks for the note here :clap:….

View Comment

JustinPrivitera commented on issue visit-dav/visit#20073.

No worries, can we close this then?…

View Comment

tgamblin commented on issue spack/spack#51542.

@alalazo should this go here or in #51533 ? …

View Comment

v-dobrev commented on issue mfem/mfem#3658.

There are some strange commits in this PR, e.g. b46bb0d4 – we should consider squash-merging it….

View Comment

jameshcorbett commented on issue flux-framework/flux-sched#1417.

Merging manually because Mergify doesn’t want to….

View Comment

adayton1 commented on issue LLNL/axom#1721.

I do not have permissions to merge, so feel free to push the button @kennyweiss or @rhornung67….

View Comment

balos1 commented on issue LLNL/sundials#796.

Review Requests

View Comment

jeremylt commented on issue spack/spack-packages#2361.

I cannot speak to the MFEM changes, but that’s the right way to enable it for libCEED…

View Comment

markcmiller86 commented on issue spack/spack-packages#2346.

I am getting this error in one of the CI checks… …

View Comment

markcmiller86 commented on issue LLNL/Silo#312.

This has been started here, https://github.com/spack/spack-packages/pull/2346…

View Comment

JustinPrivitera commented on issue visit-dav/visit#20152.

> Is this correct? Shouldn’t the <td></td> tags be within the <tr></tr> tags?…

View Comment

tgamblin commented on issue spack/spack#51445.

Looks mostly good modulo the failing test :)…

View Comment

garlick commented on issue chaos/diod#160.

LMK = Let Me Know…

View Comment

chrundle commented on issue FLASK-LLNL/ChARGe#74.

> @chrundle Let’s put the LLNL specific files in our local project space for now and we can then add it to a gitlab internal project to keep it version controlled. …

View Comment

cyrush commented on issue visit-dav/visit#20707.

@JustinPrivitera thanks for this fix!…

View Comment

becker33 commented on issue spack/spack-packages#2176.

@dguibert did you get a chance to look at this again? Let me know if you need to hand it off….

View Comment

cmelone commented on issue spack/spack-packages#2290.

I can certainly add a +serial option to the package, but this is what I get when running spack spec cabana-pd

View Comment

becker33 commented on issue spack/spack#50560.

I have not gotten a chance to debug it yet….

View Comment

hppritcha commented on issue open-mpi/ompi#13280.

the arrays were’t being cleaned up anyway in the existing code. some checks for predfined datatypes resulted in not running the callback that was freeing these arrays. …

View Comment

tzanio commented on issue mfem/mfem#5097.

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….

View Comment

cyrush commented on issue mfem/mfem#4307.

@JustinPrivitera this can be closed in lieu of #5097 - we can continue discussion in the PR….

View Comment

garlick commented on issue flux-framework/flux-core#7112.

Thanks!…

View Comment

grondo commented on issue flux-framework/flux-core#7112.

Strange. I had to re-approve and then mergify finally merged this one. :shrug: …

View Comment

cyrush commented on issue LLNL/axom#1709.

thanks @kennyweiss …

View Comment

adrienbernede commented on issue LLNL/Umpire#1009.

Now that I read your full question, here is the full answer: …

View Comment

jeremylt commented on issue CEED/libCEED#1904.

```py …

View Comment

alecbcs commented on issue spack/spack#51522.

I’m nervous about picking the “preferred” version of a package without interpreting any constraints. This works now because cmake’s preferred version is v3.31.9, but the second that flips to v4 it’ll break since clingo-bootstrap@spack <= clingo@5.7.1 which conflicts with cmake@4….

View Comment

chu11 commented on issue flux-framework/flux-core#7186.

re-pushed, throwing an extra flux-fsck cleanup in there…

View Comment

grondo commented on issue flux-framework/flux-coral2#439.

Ok, I kind of doubt that is where the time in the sysadmin prolog (/etc/flux/system/prolog) is being spent anyway, since we know the dws_environment event is already available by the time it starts running….

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#772.

It’s also worth mentioning that #752 added a continuous re-check of held jobs after every flux account-priority-update, so at least there is the potential for the plugin to sort itself out on a periodic basis….

View Comment

grondo commented on issue flux-framework/flux-accounting#772.

Looking at the current job-manager code, a newly loaded plugin is sent the list of added jobs using zhashx_values() on the active job list:…

View Comment

JustinPrivitera commented on issue LLNL/conduit#1475.

You are welcome. Please let us know if you have further questions….

View Comment

camierjs commented on issue mfem/mfem#4333.

Tested on macos with latest Caliper - works fine!…

View Comment

wihobbs commented on issue flux-framework/flux-core#7043.

Yes, let’s check this off!…

View Comment

jameshcorbett commented on issue flux-framework/flux-coral2#439.

> Oh, I see now, there’s zstop_nnf_services.sh in /etc/flux/system/prolog-job-manager.d and zstop_nnf_clientmount.sh in /usr/libexec/flux/prolog.d. Do we need both of them?…

View Comment

artv3 commented on issue LLNL/RAJA#1832.

@bechols97 , can we bring this up to date?…

View Comment

wihobbs commented on issue flux-framework/flux-pmix#113.

Hmm, it looks like I don’t have write access here to add labels….

View Comment

trws commented on issue flux-framework/flux-core#7181.

That sounds right to me, but we should probably check and document. I think hwloc logical cores is exactly what we want it to be, should mean we can mock the topology however we want, but best to be sure….

View Comment

eleon commented on issue flux-framework/flux-core#7182.

> > Another approach to this PR would be to move the adjustment of the cores and gpus strings into the shell itself so that the values returned to plugins are the adjusted values. …

View Comment

tgamblin commented on issue spack/spack#51165.

I think this is subsumed by #51162, but let me know if there is something here that is not covered there….

View Comment

jedbrown commented on issue openjournals/joss-reviews#9333.

Hi @lindsayad @mprobson

tzanio commented on issue mfem/mfem#5023.

> I also saw that an ex41 is already in-review (#4289); please let me know if I should update this to be ex42. …

View Comment

grondo commented on issue flux-framework/flux-core#7182.

I’ve force pushed a new version of this PR that fixes up the core/gpu assignments in the shell proper, so that flux_shell_rank_info_unpack(3) reflects the update core and gpu sets. This allows mpibind to just work: …

View Comment

tzanio commented on issue visit-dav/visit#19046.

@cyrush – I am happy to elaborate further on this if it will be useful…

View Comment

cyrush commented on issue visit-dav/visit#19046.

Yes this does help - thanks for this explanation. …

View Comment

cyrush commented on issue spack/spack-packages#2222.

The concretizer can be unpredictable, frustrating aspect of all of this …

View Comment

imguoguo commented on issue radxa-pkg/radxa-firmware#29.

>

imguoguo commented on issue radxa-pkg/qualcomm-profiles#15.

>

hppritcha commented on issue open-mpi/ompi#13280.

all good ideas. i’ll work on these next week….

View Comment

tomstitt commented on issue mfem/mfem#4333.

> @tomstitt, is this backward compatible, and if not, should we check for the Caliper version? …

View Comment

v-dobrev commented on issue mfem/mfem#4993.

Re-merged in next for testing……

View Comment

cjvogl commented on issue mfem/mfem#4496.

> I’m happy to see this through @cjvogl. Do I have your permission to commit and push to your branch? :) …

View Comment

dylan-copeland commented on issue mfem/mfem#5087.

Thank you @chunyu79 for the update, and congratulations on finding a solution! I was able to run your test and reproduce the issue, but did not have time yet to investigate deeply. This was labeled as an AMR issue, but it seems more closely related to contact problems with constraints to couple boundaries of different domains. There has been recent work on this topic in PR 4996, and @psocratis may have some advice….

View Comment

jeremylt commented on issue freeCodeCamp/contribute#1008.

Those linked PRs are from staff. Staff are not external contributors….

View Comment

garlick commented on issue chaos/diod#158.

Oh this changed the github workflow so might need a manual merge. I’ll press the button….

View Comment

adrienbernede commented on issue LLNL/RAJA#1934.

(cancelled the CI pipeline to restart and force Matrix to run with variable)…

View Comment

adayton1 commented on issue LLNL/CARE#351.

@adrienbernede thank you!…

View Comment

adrienbernede commented on issue LLNL/CARE#351.

@adayton1 This is ready ;)…

View Comment

markcmiller86 commented on issue visit-dav/visit#20682.

Wow…great catch @biagas! No idea that logic existed….

View Comment

becker33 commented on issue spack/spack#51490.

Looking at the optimization criteria for variant_default_not_used and variant_not_default, it looks like we might have to swap the ordering some in addition to removing the one for variant_not_default for already built packages….

View Comment

alecbcs commented on issue spack/spack#51433.

Trying this PR I ran into a couple of errors on edge cases. Happy to help work on fixing these if needed. …

View Comment

gardner48 commented on issue mfem/mfem#5073.

> Should the calls be added to the examples? …

View Comment

jandrej commented on issue mfem/mfem#5073.

Should the calls be added to the examples?…

View Comment

bgunnar5 commented on issue LLNL/merlin#587.

@lucpeterson I think they are but then again I’m biased cause I wrote them…

View Comment

JustinPrivitera commented on issue visit-dav/visit#20666.

@IdoAkkerman please see continued discussion in #19046…

View Comment

imguoguo commented on issue radxa-pkg/radxa-system-config#54.

> etc

wihobbs commented on issue flux-framework/flux-core#7153.

For reference, here’s how mpibind prints affinity in verbose mode:…

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#778.

Thanks @jameshcorbett!…

View Comment

tpatki commented on issue flux-framework/PerfFlowAspect#191.

Closing with PR #199 and #200 …

View Comment

tpatki commented on issue LLNL/variorum#590.

@slabasan and I are in the process of testing ESMI+HSMP to obtain CCD-only (CPU) power on the APU. …

View Comment

trws commented on issue ACCBDD/reclamation-dev#276.

Not to say it’s “good” but a create deployer with buckets of water will fill the cauldron. Still trying to work out if I can automate this thing the rest of the way, but it’s something….

View Comment

JustinPrivitera commented on issue LLNL/conduit#1471.

put this in the blueprint spec docs…

View Comment

mplegendre commented on issue LLNL/Spindle#43.

Closing since Nick is putting together updated containers….

View Comment

vladotomov commented on issue CEED/Laghos#199.

Should I merge it or do you plan more additions?…

View Comment

balos1 commented on issue LLNL/sundials#789.

What compiler and version are you using? Can you attach your CMakeCache.txt file?…

View Comment

gardner48 commented on issue LLNL/sundials#787.

@drreynolds I’ve updated the correction modification to address the roundoff issues and resolved the open comments. However, this PR also includes updates from #788 (makes adding logging to the constraint checks easier), so hold off on taking another pass over this PR until that one is merged….

View Comment

jameshcorbett commented on issue LLNL/scr#604.

I fixed this up so all the actions pass, I think this is ready to go in. Will check in next team meeting. Actions are now passing: https://github.com/jameshcorbett/scr/actions/runs/18811341893…

View Comment

hppritcha commented on issue open-mpi/ompi#9837.

this PR can be closed. we moved well beyond this as part of the big count work. and there’s more to come sometime later after the ‘c’ ABI stuff is complete and we want to venture into a fortran ABI world….

View Comment

camierjs commented on issue mfem/mfem#3658.

> mpirun -np 4 pmesh-optimizer -m kershaw_6x6x6.mesh -o p -rs rl -mid 321 -tid 5 -ls 3 -nor -rtol 1e-8 -vl 1 -lc 0.5 -pa

View Comment

v-dobrev commented on issue mfem/mfem#5085.

Some thoughts: some of the methods in ARKStepODE will be common with other similar classes. How do you envision avoiding repeated definitions? Also, when you inherit from both TimeDependentOperator and ARKStepODE, you get Mult from both and you have Width/Height (from the former) and Size from the latter which is not ideal. …

View Comment

garlick commented on issue chaos/diod#147.

I’ve got it, thanks!…

View Comment

grondo commented on issue chaos/diod#147.

Yeah, want me to press the button?…

View Comment

gardner48 commented on issue LLNL/sundials#788.

The main logging updates are in tools/suntools/logs.py. The changes to the source files and outputs are to update logging region labels to the new convention of <begin|end>-<region-name>[-list] (mostly adding -list where needed)….

View Comment

markcmiller86 commented on issue LLNL/Silo#477.

Am recording here a smoking gun reproducer of the issue in error stack error messages. Attached is an intentionally corrupted HDF5 file. 1.14.4 produces an error message indicating a checksum failure. 1.14.6 does not. …

View Comment

tgamblin commented on issue spack/spack#51461.

Per discussion in Slack, @alalazo’s going to make a PR that fixes the docs and does this: …

View Comment

tzanio commented on issue mfem/mfem#5086.

Merged in next for testing……

View Comment

artv3 commented on issue mfem/mfem#5082.

This is great! I’ll give it a spin these next few days. The one suggestion I have is lets add a constructor that enables us to specify the device memory type — I just pushed something up for quadrature function here: https://github.com/mfem/mfem/pull/5086 …

View Comment

vladotomov commented on issue mfem/mfem#3658.

@camierjs …

View Comment

jandrej commented on issue mfem/mfem#5022.

> Are there some GPU Tests enabled ? …

View Comment

cjvogl commented on issue mfem/mfem#4977.

This is now out of scope for this PR (given what was discussed at the technical meeting today), but if anyone is interested in future work into how refactoring TimeDependentOperator into modular interfaces might look (as @bslazarov had suggested at one point), I’ve started a WIP PR that showcases how an ARKStepSolver-specific interface could work: https://github.com/mfem/mfem/pull/5085…

View Comment

chakshinglee commented on issue liruipeng/ml2#37.

The PR looks good to me too. I only have a design question around the Chebyshev frequencies (see above). Another thing, which is beyond the scope of this PR, is that how do you guys think about using config files instead of relying on command line options to pass in values of various parameters?…

View Comment

grondo commented on issue flux-framework/flux-core#7168.

Well, I don’t actually see anything in RFC 44 that says an undrain event can only be posted for ranks that were previously drained, so one idea would be to add a new “mode” to the undrain RPC like “update” which simply posts an extra undrain event to the eventlog with the new reason. A tool would then have to process the entire resource eventlog to be sure the most updated undrain reason event for any given rank has been processed (instead of assuming the first undrain event after the corresponding drain event has the correct reason)….

View Comment

chu11 commented on issue flux-framework/flux-core#7158.

re-pushed, removing the “common function” discussed above. Fixed up a few commit messages I noticed still said --async

View Comment

imguoguo commented on issue RadxaOS-SDK/rsdk#112.

>

adayton1 commented on issue LLNL/blt#737.

Could you do CUDA builds on another toss 4 machine? There would just be no testing the executables. But testing the build is better than nothing….

View Comment

jeremylt commented on issue CEED/libCEED#1901.

Yup, that should be the fix - use the blocking as if we actually cared about the basis for the operation…

View Comment

JustinPrivitera commented on issue visit-dav/visit#20576.

TODO…

View Comment

imguoguo commented on issue radxa-pkg/qualcomm-profiles#10.

https://applink.feishu.cn/client/message/link/open?token=AmfHwOsK4gAEaPnma7hoQAQ%3D…

View Comment

tomstitt commented on issue mfem/mfem#5027.

I see that ipiv_base is used in some places but not all, does it make sense switch everywhere to use it? or remove it?…

View Comment

garlick commented on issue flux-framework/flux-core#7158.

> So, we need to store all the lookup error indexes and errnos on a list somewhere for later retrieval. But that just seemed like a lot of effort for the amount of gain. (thus my eventual “err_cb” mechanism instead) …

View Comment

cmoussa1 commented on issue flux-framework/flux-accounting#775.

Actually, I think I might have just discovered it

JustinPrivitera commented on issue Alpine-DAV/ascent#1620.

``` …

View Comment

cyrush commented on issue visit-dav/visit#20636.

First pop up:…

View Comment

garlick commented on issue flux-framework/flux-foundry#21.

Oh right, no mergify. Pressing the button….

View Comment

wihobbs commented on issue flux-framework/flux-core#7165.

> You didn’t provide any argument for using a long_help variable or against a docstring, though, so perhaps I’m missing something obvious. …

View Comment

jbschroder commented on issue XBraid/xbraid#96.

Hi,…

View Comment

adayton1 commented on issue LLNL/camp#190.

I’m looking at the definition for unordered_map and unordered_multimap, and as far as I can tell they only require the equality operator….

View Comment

adayton1 commented on issue LLNL/RAJA#1923.

@rhornung67, is this still a WIP or is it ready for review?…

View Comment

vsoch commented on issue flux-framework/flux-coral2#411.

This does not require review yet. Updates for this PR include: …

View Comment

tzanio commented on issue mfem/mfem#5072.

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….

View Comment

chu11 commented on issue flux-framework/flux-core#7158.

re-pushed, converting –async to –fast….

View Comment

cjekel commented on issue cjekel/similarity_measures#40.

LGTM! Thank you! Sorry this took so long to get in….

View Comment

trws commented on issue LLNL/camp#190.

As it sits I’m ok with this, but I expected based on the description that there would be a specialization of std::hash in here so that unordered_map and similar could hold them. Is that not part of what you wanted to accomplish?…

View Comment

JustinPrivitera commented on issue visit-dav/visit#17791.

3.5.0 deliverables have been completed so I am leaving the milestone at 4.0….

View Comment

tgamblin commented on issue spack/spack#51137.

> Finally, just a comment on colors (disregard this if you don’t agree). I wouldn’t use red for run dependencies, cause red is usually associated with failures. Maybe yellow? …

View Comment

imguoguo commented on issue radxa-pkg/qualcomm-profiles#7.

>

v-dobrev commented on issue mfem/mfem#4431.

@tzanio, I just pushed a small update to increase the timeout on Matrix – please re-merge it in next when you get a chance….

View Comment

artv3 commented on issue mfem/mfem#2176.

I think we can close this PR, its from a long time ago… …

View Comment

jameshcorbett commented on issue flux-framework/flux-sched#1403.

Hi @kostiscpp sorry I forgot to answer this. Are you still interested in a solution? As @grondo mentioned, Fluxion needs to be configured so that it knows about the topology of each node, and specifically about the placement of sockets. How are you starting your Flux instance at the moment?…

View Comment

grondo commented on issue flux-framework/flux-core#7158.

Just for my own curiosity, when would we not want the “fast” behavior? …

View Comment

garlick commented on issue flux-framework/flux-core#7157.

Dropcache drops “clean” blobs from the cache but that doesn’t mean other things (KVS, metadata) aren’t still referencing those blobs….

View Comment

hppritcha commented on issue open-mpi/ompi#13436.

closing this and opening a new one with different selection of shas…

View Comment

hppritcha commented on issue mpi-forum/mpi-forum.github.io#27.

@wesbland updating sessions wg leads…

View Comment

JustinPrivitera commented on issue mfem/mfem#4307.

@cyrush can we let this be automatically closed since you have solved this in VisIt? Or do we still need this?…

View Comment

jandrej commented on issue mfem/mfem#4485.

ping @nnpp-schneier @victor-decaria-nnl …

View Comment

adrienbernede commented on issue mfem/mfem#4431.

@v-dobrev thank you for updating this PR. Glad to see you leverage latest support for matrix added very recently. …

View Comment

homeomorfismo commented on issue mfem/mfem#4498.

@pazner Makes sense to me!…

View Comment

grondo commented on issue flux-framework/flux-coral2#430.

I opened an issue here to discuss possible solutions: …

View Comment

jameshcorbett commented on issue flux-framework/flux-coral2#430.

Putting documentation in the docstring makes sense to me. This plugin certainly does not require much documentation, what it does is reasonably straightforward….

View Comment

garlick commented on issue flux-framework/flux-coral2#430.

Hmm, that is a conundrum. I wonder if it would just make sense to have one flux-coral2 page in section 7 with a subsection titled RUN COMMAND EXTENSIONS or similar?…

View Comment

vladotomov commented on issue CEED/Laghos#196.

@wdhawkins please add 1-2 lines to the README to mention building with Caliper, and I’ll merge this….

View Comment

cyrush commented on issue visit-dav/visit#5770.

@BenWibking thanks for the update – I am sure adding AMR support was a good amount of work and others could benefit from these improvements even if there are some outstanding issues. Would you like to get your new plugin into visit proper?…

View Comment

jedbrown commented on issue openjournals/joss-reviews#9244.

Hi @PabloIbannez, @valsson

camierjs commented on issue mfem/mfem#5057.

I updated the [MTOP] solver with GPU support #5072 TODO item to point to the new PR (#5072)….

View Comment

btalamini commented on issue jax-ml/jax#24295.

Thanks, makes sense. In my case, if I knew that a particular use of the solver wasn’t going to be differentiated, I wouldn’t bother wrapping it with custom_root at all. It’s not really buying you anything if you’re not differentiating over the closed-over variables….

View Comment

jedbrown commented on issue RDycore/RDycore#348.

An alternative would be to write partition statistics on stdout, like we do in Honee. I guess we could put something like this into -dm_view_partition….

View Comment

tpatki commented on issue Olympus-HPC/Mneme#51.

Hi @koparasy Yes, please merge. Sorry, just saw this, had missed the notification. …

View Comment

bgunnar5 commented on issue LLNL/merlin#585.

Thanks for the review @jwhite242. I’ll go back through all of the docstrings/type hints to make sure they’re accurate with the latest changes once this is done….

View Comment

chrundle commented on issue FLASK-LLNL/ChARGe#28.

Cancelling pull request (separated into two separate requests for backend and experiments/)…

View Comment

cmelone commented on issue cmelone/hubcast-test#4.

@lc-hubcast help…

View Comment