Open Source Heartbeat: Open Source Heartbeat

Issue Comment Events

v-dobrev commented on issue spack/spack#44684.

> > Also, in my tests, Spack was using zlib-ng@2.1.6, so the issue that I saw cannot be the one fixed v2.1.2 (unless the issue was re-introduced in v2.1.6 after it was fixed in 2.1.2). …

View Comment

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

We don’t have an example of this, however, it’s quite simple to implement based on Example 1: one just needs to add the appropriate weights from the spherical change of variables to the coefficients of the bilinear and linear forms. You can find the weights e.g. here: https://en.wikipedia.org/wiki/Spherical_coordinate_system….

View Comment

gardner48 commented on issue LLNL/sundials#520.

> I have a slight preference to the formatting in this PR over #521. Also, cmake-format seems to be much more popular so I think its probably the better choice for that reason too. …

View Comment

balos1 commented on issue LLNL/sundials#520.

I have a slight preference to the formatting in this PR over #521. Also, cmake-format seems to be much more popular so I think its probably the better choice for that reason too. …

View Comment

dylan-copeland commented on issue LLNL/libROM#287.

Could another reviewer please test on mac and approve?…

View Comment

grondo commented on issue spack/spack#44775.

Nice! I’d just squash to two commits for each package, since the second commit undoes most of the first. Thanks for doing this!…

View Comment

jandrej commented on issue mfem/mfem#4367.

Highlighting this as a bug internally….

View Comment

garlick commented on issue flux-framework/flux-sched#1222.

> Any idea what makes a good baseline delay or is it too case-specific? …

View Comment

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

It isn

grondo commented on issue flux-framework/flux-sched#1222.

I’ll open an issue in flux-core to investigate the job-manager sending alloc requests after the jobs have a fatal exception raised, but I wonder if the alloc requests have all already been sent (I think Fluxion uses the unlimited alloc limit, so no alloc requests are “held back”) and are sitting in the message queue of the qmanager? We could perhaps check if the behavior is duplicated with sched-simple. …

View Comment

vsoch commented on issue flux-framework/flux-python#11.

Give this a try: https://pypi.org/project/flux-python/0.62.0/…

View Comment

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

It may be useful to add some more stats to flux module stats job-manager such as the number of pending sched.alloc and sched.cancel requests, where the latter is defined as a sched.alloc request that is still pending even after a cancel request has been sent. …

View Comment

chu11 commented on issue chaos/powerman#193.

doubling the CP_LINEMAX to 16K seemed to solve the above problems. …

View Comment

garlick commented on issue chaos/powerman#193.

I don’t know if it will fix the hostlist error but we might want to consider pulling in the hostlist implementation from flux since it’s actually had some attention paid to it in the last decade. …

View Comment

jeremylt commented on issue CEED/libCEED#1611.

Yea …

View Comment

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

At the very least we should put a note here: https://visit-sphinx-github-user-manual.readthedocs.io/en/stable/python_scripting/quickrecipes.html?highlight=Handling%20Command%20line%20arguments%C2%B6#handling-command-line-arguments…

View Comment

tgamblin commented on issue spack/spack#44713.

> Except I forgot we wanted to get rid of -c, so new output: …

View Comment

vsoch commented on issue spack/spack#44775.

This change is saying that if flux is provided as an external, it’s up to the external provider to set the LUA paths, which I think is logical. The issue is that adding it as an external is going to look for spec with lua, and that likely is not present if lua was only a dependent for flux. It would still not work to add it, because it might be an actual mismatch in version. My only critique might be to check for external and just return early, and then not indent the rest of the function. It would also be good to add a comment above that, explaining the logic for a future package.py reader….

View Comment

eliasboegel commented on issue pop-os/cosmic-applets#497.

I just found #225 and #232 which would at least cover the quite common WPA2/PEAP use-case including networks like Eduroam. I must have missed that issue. Is it preferred to close this issue as a duplicate, or should it remain open as it is more general than #225? …

View Comment

vsoch commented on issue oras-project/oras-py#142.

In case you missed the message above, we already have the docker client included and “login” is called from it: …

View Comment

tzanio commented on issue mfem/mfem#3138.

There is a documentation failure in the CI…

View Comment

wihobbs commented on issue flux-framework/flux-test-collective#22.

I don’t know if we’re going to be able to do this because the service user doesn’t have root access, so it can’t run systemctl start user@60943.service. We have rootful gitlab runners in LC, but they’re pretty niche and usually for deployment workflows. …

View Comment

vsoch commented on issue flux-framework/flux-k8s#76.

Thanks for the review! I’ll wait to push (tiny changes so far) until we come to consensus on the above. …

View Comment

v-dobrev commented on issue GLVis/glvis#283.

You can try building GLVis and MFEM separately from your normal MFEM build – for this you can use Apple clang. GLVis built this way can work with data sent from any other MFEM build, e.g. a build with a different compiler and with different optional libraries. GLVis does not use anything parallel from MFEM, so you don’t loose anything by building serial MFEM for use by GLVis. Also, most optional packages that can be used by MFEM do not add any functionality to GLVis so building without them is perfectly fine. The one exception here, that I can think of, is NetCDF which adds support for reading NetCDF meshes which then allows GLVis to read these meshes too….

View Comment

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

I believe hcoll is being replaced by UCC so i don’t think there’s going to be an investement of time in embiggening hcoll unless it already is. …

View Comment

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

hmmm should we be handling this in a lot more modules than we do? Just skimmed kvs-watch and job-info and I don’t think we handle it there either….

View Comment

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

> Would you like me to add new release info for chai and care to this merge request? …

View Comment

adrienbernede commented on issue LLNL/Umpire#882.

@davidbeckingsale no it is not….

View Comment

markcmiller86 commented on issue markcmiller86/hello-numerical-world#18.

@Josee1031 I recall you mentioning a _config.yml file in an email to me. But, I don’t see one here in the PR…which is fine. I was just checking if you intended one to be there?…

View Comment

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

Thanks! I’ve set MWP….

View Comment

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

> I’m pretty sure the cause of the actual hang behavior is that the process_provisional_reconsider method needs a set_schedulability(true). It solves it for me …

View Comment

jedbrown commented on issue FreeCAD/FreeCAD#14667.

Thanks for your comments. Agreed about this being post-1.0. We wanted to get some feedback, but understand it can’t merge during the feature freeze. …

View Comment

jedbrown commented on issue CEED/libCEED#1110.

It’s possible the derivative of the Riemann solver is not exact. If HLL gives quadratic, that would be a smoking gun. I would hope Newton convergence is similar with the different variable sets, even if not quadratic for the Riemann-type conditions. I wish it were easier to unit test this so we would at least have a record of what is quadratic and what is inexact….

View Comment

SteVwonder commented on issue spudlyo/clipetty#30.

I had the same issue. Walking through the code, my issue turned out to be due to TMUX, TERM, and SSH_TTY env vars returning as nil from the call to (getenv "<ENVVAR>" (selected-frame)). Dropping the (selected-frame) bit makes things work again: …

View Comment

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

@GuillaumeMercier #176 should help with this. Let’s discuss how we can handle this now inside the threaded code….

View Comment

aperloff commented on issue LLNL/SSAPy#6.

We’ve been using the workaround for a while and I also suspect that it’s needed because of how we use setup.py. We don’t have a way to install a development version of SSAPy that references the local code rather than the installed version. I totally agree that we should migrate to pip. We just haven’t had the person power to do that as of yet….

View Comment

adrienbernede commented on issue LLNL/RAJA#1605.

@long58 please see my comment here: https://github.com/LLNL/RAJA/pull/1644#discussion_r1642359362 …

View Comment

trws commented on issue LLNL/RAJA#1666.

This looks good, it should also work on the newer intel compilers IIRC, but I’m not sure we have those tied into CI for anything but sycl right now….

View Comment

tzanio commented on issue mfem/web#253.

@IdoAkkerman, can you resolve conflicts and take a final look now that https://github.com/mfem/mfem/pull/3999 is merged?…

View Comment

tzanio commented on issue mfem/mfem#4354.

Merged in next for testing……

View Comment

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

> Thanks! This LGTM!…

View Comment

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

MPI version is cray-mpich/8.1.23

View Comment

adayton1 commented on issue LLNL/RAJA#1672.

@MrBurmark, would you be willing to look at the changes to hip atomics? The previous implementation of atomicCAS relied on volatile, so I changed the implementation to use an atomic load. To avoid a circular dependency, I’ve changed in the implementation of atomicLoad to use the intrinsic if available, otherwise I fall back to atomicOr(address, 0). Does that make sense, or would it be better to fall back to atomicCAS(address, 0, 0) or something else? I think atomicOr will be better than atomicAdd, but I’m not sure if atomicCAS can avoid the write in some cases. I also modified atomicExchange to use reinterpret casting instead of atomicCAS in a loop, then made atomicStore use atomicExchange if the intrinsic is not available….

View Comment

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

One thing is to make sure all packages are built with the same compiler and that mpicc and mpicxx use the correct compiler, e.g. by running mpicc --version – this should show the real compiler used by the MPI C wrapper, mpicc. …

View Comment

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

I can confirm that the test that simply creates and cancels 10,000 jobs doesn’t get killed by oomkiller. It does eventually complete after 47 minutes, so it may not reproduce the title issue….

View Comment

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

just pushed some tweaks to the admin description to fix my poor word choices and also add the --with-imp option to the perilog-run example….

View Comment

tgamblin commented on issue flux-framework/flux-core#6041.

Ping @jefflong6 @xorjane…

View Comment

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

I think that should be OK! A quick test might be a good idea though. We’ve been surprised on el cap before :-)…

View Comment

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

> Thoughts @grondo ? Would there be performance issues from having every node read the eventlog? (I hope not, because I think we do some coral2 plugins do this elsewhere.) …

View Comment

jedbrown commented on issue aakritisi/FreeCAD#1.

This looks good in terms of content. I think the commits should be squashed and rebased (rather than merging from upstream) into atomic parts intended for review. And then we need to write out in the PR what changes are being made and why. …

View Comment

alecbcs commented on issue spack/spack#44420.

@tldahlgren does this PR pass your checks for homepages and versions now?…

View Comment

adrienbernede commented on issue spack/spack#44571.

@white238 I think this https://github.com/spack/spack/pull/44699 is what you were looking for….

View Comment

markcmiller86 commented on issue markcmiller86/hello-numerical-world#14.

Hmmm…that looks like a new warning with your changes from snprintf to sprintf. It was using snprintf before. Can you revert your changes and produce the error message the code was originally emitting?…

View Comment

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

Oh wow. They died on a cancel from resource.acquire. I’m actually not sure how we could get to a FATAL: exception not rethrown like that, it actually looks like what would happen if you hit an uncaught throw inside a destructor and terminated the process. Did it OOM or something?…

View Comment

balos1 commented on issue LLNL/sundials#501.

@gardner48 The “if” version sounds like a nice addition as well. …

View Comment

adrienbernede commented on issue CEED/libCEED#747.

@jeremylt I

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

There are a few ways to go here. …

View Comment

v-dobrev commented on issue spack/spack#44620.

I see that in develop almost all deprecated versions of openssl were removed in #44629. What do we want to do with the versions that are being deprecated by this PR: 3.0.13, 3.1.5, 3.2.1, and 3.3.0? Should we keep them as deprecated or remove them? @haampie, what do you suggest? …

View Comment

cyrush commented on issue spack/spack#44571.

Thanks @tldahlgren. …

View Comment

jedbrown commented on issue rsmpi/rsmpi#183.

If this is something we can document or otherwise make more robust, feel free to open a PR to update the docs. …

View Comment

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

Thanks @victorapm! …

View Comment

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

Oh, oops, I had forgotten this was open as a PR. Ignore that push, that was just to sync across machines. …

View Comment

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

This job-list w/ a database has been re-worked: …

View Comment

bgunnar5 commented on issue LLNL/merlin#486.

Need to fix issue where latest changes weren’t pulled for some reason. This is a reminder to not merge this branch until that’s fixed…

View Comment

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

Thanks for trying this. It looks like some of these may well be attributed to Open MPI. Most of the functions caught will only be run once in the call to MPI\_Init but i’m not sure about the others. Not sure about your second question.

View Comment

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

OK, I’ve force-pushed up some changes (notably to the plugin: add "age" factor in priority calculation commit) based on the feedback above that I hope begins to capture the number of cases we talked about. I can break this up into multiple commits if needed, but I thought it might be simpler to have it all in just one commit. …

View Comment

jedbrown commented on issue SRI-CSL/PVS#92.

Hi :wave:, thanks for looking into this. Have you gotten a chance to make progress or would scripting around Cl

CamStan commented on issue LLNL/scr#452.

This feature was added to bootstrap.sh in #543 …

View Comment

artv3 commented on issue LLNL/RAJA#1665.

Adding an example could be really nice too! You can probably just take one from the unit test. …

View Comment

adrienbernede commented on issue LLNL/CHAI#263.

@adayton1 Thank you for spotting that. …

View Comment

v-dobrev commented on issue GLVis/glvis#283.

We primarily use and test with Apple clang on Mac, so I’d recommend using that. …

View Comment

corbett5 commented on issue GEOS-DEV/LvArray#321.

Yeah after a tad more thought I doubt it’s an OOM error since in those cases we usually see the error from Umpire directly. But certainly if it’s not terribly slow running in full debug mode will hopefully tell us more….

View Comment

adrienbernede commented on issue LLNL/RAJA#1644.

@long58 apparently Spack does not like your new compiler. I

jedbrown commented on issue RDycore/RDycore#185.

I think the question is which site will be maintained when requirements change. I didn’t think of the static site when a student was running into issues building. Shall we delete the build instructions in the repo and just link, or shall we move build-from-source docs from the static site to the repo?…

View Comment

correaa commented on issue include-what-you-use/include-what-you-use#1553.

I see….

View Comment

CRobeck commented on issue triton-lang/triton#4026.

Depended on: …

View Comment

vsoch commented on issue spack/spack#44602.

Thanks @alecbcs!…

View Comment

trws commented on issue spack/spack#44581.

None…

View Comment

cyrush commented on issue spack/spack#44571.

These are the prefered major deps: …

View Comment

adayton1 commented on issue serizba/cppflow#254.

Just remove the call to std::move. You already have an rvalue reference, so the move constructor will get called without an additional call to std::move….

View Comment

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

Could you set this env variable in the shell where the parent process is started?…

View Comment

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

I think @jandrej is right – all your source files that use mfem::forall need to be marked as CUDA sources when MFEM_USE_CUDA is true. This can be done with something like this: https://github.com/mfem/mfem/blob/c3eb769a2a316a68fd9a00448e20ca7cff4c8b7c/config/cmake/modules/MfemCmakeUtilities.cmake#L87-L90 …

View Comment

jeremylt commented on issue freeCodeCamp/freeCodeCamp#55118.

Yes, we should use the phrase “single space” in all three spots…

View Comment

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

Now that the release is together, I rebased the reformat here. Short of adding CI to verify it, which might accidentally work because it’s in the format script now that I think about it, this should be good to go….

View Comment

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

Yeah, I guess if a job’s age is reset any time the job is modified then that is fair and wouldn’t require recalculating the priority for all other jobs at the time of that job’s priority update, since it would by definition have the smallest age. …

View Comment

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

I think I see what you’re saying - sorry for it taking me until now to understand.

correaa commented on issue boostorg/serialization#309.

> The source code for the isolated test is here: https://github.com/johnwason/boost_serialization_1_84_bug . It includes an actions workflow that does a parametric test of different boost versions….

View Comment

benzwick commented on issue SlicerCBM/SlicerCBM#73.

This seems to have been caused by some recent changes to the location of the documentation (Slicer is looking for a module in the docs folder). …

View Comment

adrienbernede commented on issue LLNL/Umpire#795.

@davidbeckingsale +tools currently conflicts with +rocm. Depending on what you want to achieve, you

adrienbernede commented on issue LLNL/RAJAPerf#452.

@rhornung67 Sounds good. …

View Comment

jeremylt commented on issue CEED/libCEED#1597.

Most of the core pieces are there, need to debug the parallelism, esp in 2/3d…

View Comment

jandrej commented on issue zed-industries/zed#12644.

Yea I think it’s fixed in the preview….

View Comment

gardner48 commented on issue spack/spack#44601.

This should be fixed by LLNL/sundials#434 and LLNL/sundials#437 in the sundials develop branch which fixed build problems with ROCM 6.x…

View Comment

v-dobrev commented on issue spack/spack#44010.

@cnpetra, @cameronrutherford, …

View Comment

jandrej commented on issue mfem/mfem#4346.

MFEM requires all files to be compiled with nvcc. Are you sure that your project files that use mfem_forall are compiled with nvcc? What’s the output of make VERBOSE=1 of your project….

View Comment

tzanio commented on issue mfem/mfem#4337.

Merged in next for testing……

View Comment

hppritcha commented on issue grimme-lab/xtb-python#114.

some context. we are trying to use xtb and the xtb python bindings with examol and were building it using spack and hit this issue….

View Comment

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

> Anything else you want to make sure happens @milroy? …

View Comment

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

Oh,wait. I guess probably the version of systemd I have isn’t yet using cgroup.kill so systemd running as flux cant signal the shell running as garlick. The SIGTERMs are forwarded by the IMP, and the SIGKILLs take the IMP out. So back to the original idea of having the IMP iterate over the cgroup. Sorry for the noise!…

View Comment

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

I thought https://github.com/flux-framework/flux-core/pull/6013 was going to handle this by “buffering” the stdin RPCs?…

View Comment

balos1 commented on issue LLNL/sundials#500.

@bangerth Can you try setting the matvec operation on the empty matrix object created here? It can just be a no-op. …

View Comment

markcmiller86 commented on issue LLNL/Silo#378.

Ok, I figured out how to do it same with h5py… …

View Comment

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

@tharman-git can you try building with VTK-9 support? …

View Comment

jedbrown commented on issue fish-shell/fish-shell#6931.

Thanks for making this issue. The solution isn’t quite equivalent. For example, typing pacman -S xf86- and triggering insert-completions yields pacman -S xf86-xf86-input-elographics xf86-input-evdev [...]. I’m newly tinkering with Fish so don’t know how to remove the stem before inserting the completions….

View Comment

bgunnar5 commented on issue LLNL/merlin#481.

So it looks like the local run will pass with or without the patch. However, the distributed run does not so I think we have to keep this sleep business embedded in the test for now. I have another branch that’s working on updating the test suite so maybe once I work on that more we can use some of those changes to resolve this….

View Comment

brendankeith commented on issue mfem/web#266.

> is it ready-for-review? …

View Comment

artv3 commented on issue LLNL/RAJAPerf#442.

Can you add the SYCL version too? It should be straightforward. Perhaps also call this kernel “UNROLLED_MATVEC_3D” kernel? …

View Comment

jandrej commented on issue EnzymeAD/Enzyme#1904.

@GregTheMadMonk did you check the output values? I only get zeros from the fwddiff in your branch….

View Comment

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

Summarizing… …

View Comment

correaa commented on issue seanbaxter/circle#118.

Okay, I think the problem is that Circle was the only compiler reaching the end of the priority list for adl_uninitialized_copy. This, in turn, is invalid code, and since it is an auto function, Circle was probably very confused. …

View Comment

tzanio commented on issue mfem/mfem#4330.

I also like option 2 with a mention in CHANGELOG

View Comment

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

I also prefer option 2, which has a minor disadvantage only for some users with classes derived from GridFunction. The necessary changes for users should be easy….

View Comment

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

I also switched over flux-exec(1) to LOCAL_UNBUF, which should cause any line buffering issues introduced by this PR to cause stdout tests to fail in t2409-sdexec.t. …

View Comment

hppritcha commented on issue spack/spack#44255.

unfortunately i do not have access to a system with LSF installed. @naughtont3 could you check this out on summit if its available?…

View Comment

tgamblin commented on issue spack/spack#44425.

ok @alalazo @becker33 this is passing and ready for review…

View Comment

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

I don’t think we need np 5 for relocated binary. I agree with @jsquyres aobut that one. …

View Comment

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

Ok, figured this one out. It was a loop ordering issue, but I think this is now fixed, and should be a significant improvement. The one question I think we need to figure out is what a reasonable trip count is for this….

View Comment

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

i played around with the branch and equally befuddled, especially from the non-sdexec side of the code. The changes seem so simple and obvious.

View Comment

homijan commented on issue elvy-chrudim/hst-transformer#2.

@loepa01 repo s popisem k FWT…

View Comment

cyrush commented on issue LLNL/conduit#1284.

note, this is debugging code and isn’t active, so I don’t expect a hang in the wild, but it would still be good to fix…

View Comment

CRobeck commented on issue triton-lang/triton#3953.

@antiagainst I think this should be good to go. …

View Comment

v-dobrev commented on issue spack/spack#44010.

@spackbot fix style…

View Comment

cjekel commented on issue huggingface/transformers#30056.

@fxmarty Sorry for the spam. I have not tested https://github.com/ROCm/flash-attention and was curious if your team did and ran into the same issue? Otherwise, I’ll test at some point and will report back….

View Comment

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

Ah, I believe you are right. The age is in fact updated with the most recent transition to SCHED. If I simulate the scenario you proposed above, the eventlog for the job looks like this: …

View Comment

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

If this is the case, I would add a comment to the age calculation with the details above because it probably isn’t clear at first look….

View Comment

v-dobrev commented on issue GLVis/glvis#282.

> Perhaps, there could be a test with visualization of a vector field to prevent this in the future. Currently, I do not see any

balos1 commented on issue AMReX-Combustion/PelePhysics#500.

> @balos1 Do these changes look right to remove the deprecation warnings? …

View Comment

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

@markcmiller86 I tried with rect3d.silo, defined an expr named field that aliased d. …

View Comment

trws commented on issue spack/spack#44205.

It looks like it just failed because of black not liking something? Can’t immediately see what, probably whitespace?…

View Comment

adrienbernede commented on issue spack/spack#24292.

@alalazo I acknowledge reading this. It

brendankeith commented on issue mfem/mfem#4248.

Thanks for reviewing my PR, @vladotomov, @dohyun-cse, @psocratis, and @tzanio !!…

View Comment

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

@GuillaumeMercier your issues should be fix by #164. I’ve also pushed a new branch named thread-bug-work that has the fixes to build your code. When you are ready, please issue a pull request so I can merge your work into master….

View Comment

bgunnar5 commented on issue LLNL/merlin#481.

I just tried a run in local mode and it went all the way through the workflow…

View Comment

cjvogl commented on issue mfem/mfem#4322.

>

adrienbernede commented on issue mfem/mfem#4318.

LGTM…

View Comment

adrienbernede commented on issue LLNL/RAJA#1644.

@rhornung67 The intel 19.1.2 + gcc 10.3.1 job is timing out on poodle. …

View Comment

alizma commented on issue openjournals/joss-reviews#6531.

> @alizma can https://github.com/LLNL/echemfem/issues/10 be closed?…

View Comment

vsoch commented on issue spack/spack#44205.

@alalazo looks like we have the same issue - let me know what you’d like to do….

View Comment

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

after messing around today I think I got it to work. The only gotcha is that we still need the buffer when the caller writes to the subprocess before the subprocess is running. But we can completely remove the remote input prep/check. …

View Comment

markcmiller86 commented on issue betterscientificsoftware/bssw.io#2060.

My reason for mentioning is…I think there is a story here…worth an article on bssw.io… …

View Comment

gardner48 commented on issue LLNL/sundials#487.

The gfortran version Jenkins uses still segfaults with the additional Fortran warning flags so these flags are only added when using the docker GitHub actions tests….

View Comment

trws commented on issue spack/spack#44205.

Ah, I must have miscommunicated, sorry about that. I was hoping you would do that here yes….

View Comment

tgamblin commented on issue spack/spack#43796.

Per @haampie we should wait until Piz Daint is no longer online. @bcumming any opinions?…

View Comment

v-dobrev commented on issue spack/spack#41673.

@victor-decaria-nnl, can you take a look at #44010 and in particular commit ea8cd7d6 to see if that fully addresses this request?…

View Comment

vsoch commented on issue pydicom/deid#261.

You could also just check to see that the recipe is not None on the class. …

View Comment

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

undo effect of third commit and see what happens……

View Comment

tzanio commented on issue mfem/mfem#2854.

There are some CI failures with the latest merge

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

> > @v-dobrev unfortunately I cannot test it on LC yet (working on it). Would you mind running the test for me ? I think this particular PR shouldn

adrienbernede commented on issue mfem/mfem#4316.

@v-dobrev unfortunately I cannot test it on LC yet (working on it). Would you mind running the test for me ? …

View Comment

grondo commented on issue hpc/Spindle#53.

Eh, like I said the naming isn’t critical, by default the Flux job shell will load *.so from the plugin search path, so either fluxspindle.so or libfluxspindle.so would work….

View Comment

mplegendre commented on issue hpc/Spindle#53.

I’m cautious about going to spindle.so for the flux plugin. Spindle already has a libspindle.so for its user interface library, and mixing those names could be confusing. Could we symlink a spindle.so in the flux plugin space to a libspindleflux.so in the spindle space? …

View Comment

bhatele commented on issue hatchet/hatchet#541.

Superseded by #546….

View Comment

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

Great! I’ll set MWP….

View Comment

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

Yeah, we should open an issue there. I’m not sure a systemd user instance is running for the CI user in that environment and the CI user probably doesn’t have sudo access, but perhaps we can figure something out….

View Comment

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

OK, after a lot of tinkering I think I was finally able to get the man pages infrastructure set up.

balos1 commented on issue LLNL/sundials#447.

@gardner48 I think this is ready to merge, if you approve. …

View Comment

eleon commented on issue LLNL/mpibind#44.

The options of mpibind for Flux have been documented via 27d7fe5fe4d1efb610a85494d3030cc349a0b4df…

View Comment

bgunnar5 commented on issue LLNL/merlin#481.

image

View Comment

tzanio commented on issue GLVis/glvis#279.

Just to clarify – we can already adjust the size at runtime with something like -fn "Helvetica:style=Bold-16", this is for the default font size….

View Comment

jedbrown commented on issue CEED/libCEED#1592.

Looks good in principle, though this job failed. https://gitlab.com/libceed/libCEED/-/jobs/6936904108…

View Comment

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

Awesome, thanks for confirming! …

View Comment

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

XWin 32 Ver 21 seems to work well for multiple users. …

View Comment

brendankeith commented on issue mfem/mfem#4248.

> This is a really cool example, thanks @brendankeith! …

View Comment

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

@ndusek we ask that you please post jobs directly to our form - https://hpc.social/jobs/about/ …

View Comment

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

Thanks @grondo, on my list for today to look into this. I think it should be solved by the change I pushed in the other day to deal with flux-framework/flux-sched#1208, but it’s so easy for this to go wrong in unexpected ways I want to be completely sure….

View Comment

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

Great! Unfortunately I think some infrastructure is still needed to build and install the man page. I think you could check out flux-sched for the requirements, but I can try to help later …

View Comment

JustinPrivitera commented on issue LLNL/conduit#1111.

Polyhedral topology have a node ordering requirement in Silo that is not required for Blueprint. See the Silo docs for more information. What this means is that Blueprint data that does not conform to that requirement will not render properly in VisIt. So, meshes like basic don’t look right. Perhaps we could look at some kind of inside-out test and then reorder nodes. We can investigate for the future….

View Comment

jeremylt commented on issue CEED/libCEED#1454.

Note to me, I got too clever with my QF assembly setup instead of respecting memory access conditions. Easy fix but delicate …

View Comment

CRobeck commented on issue triton-lang/triton#3953.

Yeah, that is fairly straight forward. I guess we probably want a lit test to make sure the passes are actually still getting inserted into the pass pipeline after a refactor and then probably a runtime test for the ENV….

View Comment

cyrush commented on issue spack/spack#44308.

@alalazo thanks!…

View Comment

cjvogl commented on issue mfem/mfem#4321.

@v-dobrev, I’ll try out the suggestion (thanks for the suggestion!)…

View Comment

brendenpetersen commented on issue huggingface/chat-ui#1153.

@snps-ravinu I use this with my own custom API. Which may or may not use a model underneath. It just has to follow the interface this frontend repo uses, which is the OpenAI API. …

View Comment

jedbrown commented on issue SRI-CSL/PVS#92.

I’m also hitting this with emacs-29 and emacs-30. The patch linked above is now 404 because the package has been removed from Fedora. I salvaged the patch file from this commit. …

View Comment

IanLee1521 commented on issue LLNL/scraper#79.

Thanks!…

View Comment

markcmiller86 commented on issue LLNL/Silo#374.

> Tests fail, with or without my modifications. There is something basically wrong with my Ubuntu 20.04. …

View Comment

hppritcha commented on issue spack/spack#44275.

hmm looking at your log note what the EOF compiler is saying: …

View Comment

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

Hi, @joglekara, @gnorman7

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

Ok, test added….

View Comment

IanLee1521 commented on issue LLNL/HPC-Tutorials#20.

Thank YOU for contributing back :)…

View Comment

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

> Any reason to not run the test yourself locally, rather than waiting to see if the nightly fails after this commit? …

View Comment

alizma commented on issue openjournals/joss-reviews#6531.

@tlroy Thank you for the improvements and fixes. Looks good to me. …

View Comment

alecbcs commented on issue spack/spack#44201.

@dipietrantonio can you try installing using a temporary download of spack@develop and see if you get the same result? (Make sure to backup/move your existing ~/.spack directory prior to running the newer version of Spack.)…

View Comment

tzanio commented on issue mfem/mfem#4271.

@j-signorelli what do you think about the discussion and suggestions above

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

I noticed a potential issue when SymmetricMatrixCoefficient::ProjectSymmetric is called with QuadratureFunction &qf that is valid on device. Since we are accessing its entries using aliases, and all of that is done on host, it is best to call …

View Comment

vsoch commented on issue lima-vm/lima#2351.

Actually this looks to be the same one I started from, albeit many months ago. :cry: I’ll start fresh anyway in case something is different….

View Comment

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

Posting the results of some profiling on corona …

View Comment

alizma commented on issue LLNL/echemfem#14.

Looks good, thank you for the clarifications!…

View Comment

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

merge w/ develop in #19527…

View Comment

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

@JustinPrivitera would you consider using -42? Some magic numbers are more awesome than others, maybe one of them can fit the bill :-)…

View Comment

trws commented on issue spack/spack#44205.

Currently we test most heavily with gcc12+ and clang15+. In principle gcc10 and 11 might also work, but I haven’t done much work on ensuring that. I have another fix that needs to go into this package later as it happens, as soon as this one goes in I’ll post that up (it has to do with the version override code added recently)….

View Comment

tgamblin commented on issue spack/spack#44212.

A few thoughts: …

View Comment

v-dobrev commented on issue spack/spack#44010.

@spackbot fix style…

View Comment

trws commented on issue git/git#1719.

/submit…

View Comment

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

Honestly, I’m seriously considering saying we should just have a global (or class static maybe) instance that we use for stats. It’s global context. Much as I still feel the pangs of the classic “global state is evil” this is a case where even if we go multi-threaded, I think I would still want it to be global and just add some atomics….

View Comment

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

The current state of this PR puts the iteration count into jobspec to avoid altering the traverser run function signature or return type. This works but feels hackish. @trws do you have ideas for a cleaner implementation, or should I change run? …

View Comment

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

Thanks! I’ll set MWP….

View Comment

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

Thanks for filing this @grondo! To add a bit of context, here’s a slice of the perf output from the updated version of sched: …

View Comment

trws commented on issue flux-framework/flux-coral2#24.

I don’t see the context elsewhere, or another issue, so I’ll add it here. We need to implement VNI assignment at least local to each node. The switch reconfiguration, which is the part that has performance and interface concerns, we don’t need to deal with, but it’s also possible to exhaust resources on the NIC if we don’t. From what I understand, there are two parts to this. …

View Comment

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

I am not aware of that ever being a problem or I have forgotten. It’d be good to have a separate, new issue on that if it is a problem or is likely to become one….

View Comment

markcmiller86 commented on issue betterscientificsoftware/bssw.io#2002.

Following the check-list, I guess I need one reviewer at least to approve also….

View Comment

bgunnar5 commented on issue LLNL/merlin#480.

I’ll add tests for nested workflow functionality next week and then ping you again…

View Comment

dylan-copeland commented on issue LLNL/libROM#287.

On mac, I get this error for ./scripts/compile.sh -m: …

View Comment

jeremylt commented on issue CEED/libCEED#1586.

Cool, let me know if it helps. Its definitely a bugfix though, so this PR should get merged…

View Comment

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

Thanks for the updated info, I will see if I can reproduce and get to the bottom of this. …

View Comment

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

yes the failures are due to some group construct issue.the mpi create from group(s) methods use pmix group construct/destruct methods….

View Comment

aschaf commented on issue mfem/mfem#4140.

Sure no problem, lets aim for May 29th at 4pm CST?…

View Comment

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

#357 now adds a specific jobs table to the flux-accounting database with the data it requires for job usage calculations, so I think I can safely close this. Will re-open more specific issues if we find that we need other kinds of data not already captured….

View Comment

balos1 commented on issue LLNL/sundials#454.

Closed by #455. …

View Comment

artv3 commented on issue LLNL/RAJA#1648.

I think this is great, but I do wonder if maybe the development should happen within the new reducer interface to start motivating folks to transition over. …

View Comment

adrienbernede commented on issue LLNL/RAJA#1644.

@rhornung67 the new data point I find in your comment is that you don

jandrej commented on issue mfem/mfem#4307.

I’m not sure if QuadratureFunctions worked with VisIt before. They definitely do with ParaView, where they are just displayed as a “node” or “point” in space associated with values. They don’t require interpolation to visualize as they’re point-discontinuous and not element-discontinuous….

View Comment

vladotomov commented on issue mfem/mfem#4306.

@Wayne901 you can still use SetIntRule coming from NonlinearFormIntegrator::SetIntRule….

View Comment

wihobbs commented on issue flux-framework/rfc#414.

Tons of great feedback on this. Thanks to everybody for participating and sharing your expertise. …

View Comment

BradWhitlock commented on issue LLNL/axom#1338.

@rhornung67 - Adding a full-blown axom::numeric_limits<> sounds useful to me. I wasn’t really aware of that work in #1196 so I used a related axom::numerics::floating_point_limits<>::max() function that was in the code today. I changed the numeric_limits in 2 places that were marked AXOM_HOST_DEVICE….

View Comment

cyrush commented on issue E4S-Project/testsuite#61.

Thanks for the file, its a bit hard to move through. It does look like it should be built with the python command line. …

View Comment

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

:wave: @dannys4 @joglekara @MikeHeiber :wave: Hi, would any of you be available to review this submission for JOSS?…

View Comment

tomstitt commented on issue mfem/mfem#4228.

Here is another option: https://github.com/mfem/mfem/pull/4305…

View Comment

adayton1 commented on issue magma/magma#15428.

My apologies, this was the wrong magma repo….

View Comment

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

Thanks! Setting MWP….

View Comment

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

> @dylan-copeland, @tzanio, what do you think about the sgl vs flt discussion? Do you have a preference? …

View Comment

jedbrown commented on issue linalg-rs/rlst#82.

Is this that you’re not using math in documentation or is there a better way now?…

View Comment

IanLee1521 commented on issue elastic/kibana#21640.

@kevinsweet - I get not prioritizing this now, and having it as a lower priority, but it seems like closing as “complete” isn’t quite the right answer… Did it move to another tracking place? …

View Comment

jedbrown commented on issue Ondsel-Development/OndselSolver#48.

Note that it’s widely believed Apache-2.0 is incompatible with GPL-2.0/LGPLv2.1. There are two common theories for why this is: …

View Comment

adayton1 commented on issue LLNL/axom#1331.

I’ve made the following changes: …

View Comment

CRobeck commented on issue llvm/llvm-project#82787.

I’m happy to update the PR to something that we are OK merging but it wasn’t clear there was a consensus on a path forward….

View Comment

jedbrown commented on issue hpddm/hpddm#118.

My distro updated to gcc-14 so I’m bumping into this….

View Comment

vsoch commented on issue flux-framework/spack#169.

This was fixed by installing libsodium from GitHub, but now I’m seeing timeouts for xz (sourceforge): …

View Comment

alecbcs commented on issue spack/spack#44131.

@robertu94 It looks like dlib was misconfigured with an invalid maintainers list. I fixed it in this PR and you should now start to get messages when the package is modified….

View Comment

vsoch commented on issue singularityhub/singularity-compose-examples#8.

All set: https://pypi.org/project/singularity-compose/0.1.19/ thanks!…

View Comment

tzanio commented on issue mfem/web#253.

Waiting for https://github.com/mfem/mfem/pull/3999…

View Comment

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

Hello @kaipengliuHIT, the example nurbs_patch_ex1.cpp currently works only in 3D. The sample runs use the 3D mesh data/beam-hex-nurbs.mesh. There is an error message in DiffusionIntegrator::AssemblePatchMatrix_reducedQuadrature if the dimension is not 3. …

View Comment

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

> Would it be better to change the assertion into a subprocess failure, e.g. that ends up calling ops->on_state_change (FLUX_SUBPROCESS_FAILED)? …

View Comment

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

> Ahhh ok. I remember with my prototype I left failed units running for later analysis. Perhaps we should have an option for that in the future for situations such as this? I suppose it could be system wide or a per job option? …

View Comment

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

My worry was that type of solution might cause a subsequent crash (because we can’t reproduce the issue and thus can’t really test a fix, and we don’t know what would happen next) We’re trying put a bandaid here to prevent a crash and (hopefully) not just cause a different one. …

View Comment

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

there are still issues with the new raja implementation, but this is good forward progress. Old binning implementation is wired up with new tests. …

View Comment

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

That’s awesome!…

View Comment

adrienbernede commented on issue spack/spack#41049.

@scheibelp the current state of the branch does combine flags successfully in my use case….

View Comment

jedbrown commented on issue rust-lang/rust-clippy#6816.

A backward-stable algorithm evaluating a function $f$ with condition number $\kappa$ has a relative error bounded by $\kappa \, \epsilon_{\text{machine}}$. In many cases for which $f(x) = 0$, the condition number blows up. For example, the algorithm |x| (1.0 + x) - 1.0 is unstable because its second operation has unbounded condition number as $x\to 0$, and indeed produces relative error of size 1. We can’t give reliable advice for a (relative or absolute) tolerance without knowing the condition numbers involved. In the above example, the condition number of the entire function is 1 (great!) but the algorithm is unstable and thus violates any useful relative error bound. It would be nice if users always wrote backward-stable algorithms, but it’s going to be really confusing if the lint is telling people to test with a tolerance that can only be achieved with by doing so. When the functions are differentiable, we could estimate a condition number and provide diagnostics about numerical stability using Enzyme (cf. https://github.com/rust-lang/rust/issues/124509)….

View Comment

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

Sounds good, thanks!…

View Comment

vladotomov commented on issue mfem/mfem#4294.

See example 10….

View Comment

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

By comparison, keeping everything else constant but working off master instead of this PR: …

View Comment

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

At the moment this is mostly needed for flux-coral2 testing, because in actual usage jobspecs like mentioned above are constructed after submission and only the job-manager’s copy (which it sends to the scheduler) is modified. However, as @grondo mentioned on a call, the copy the shell receives will still be the user’s original, signed version. So I think this issue shouldn’t be seen in production, under the current design of flux-coral2….

View Comment

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

Heh, Tom’s way is way better :) and has the advantage of 1) not having to reinstall every time the manpages change, 2) not having to install dependencies just to get Makefiles for the manpages, and 3) taking less time. I just threw what you suggested in my .bashrc and it works great. Thanks @trws!…

View Comment

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

Extra tip for those who want this, if you don’t want to install them globally you can add them to MANPATH in place, I actually have direnv do this in my VM. For example MANPATH=$flux_core_build/doc:$MANPATH….

View Comment

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

Thanks for reviewing @jameshcorbett! Setting MWP here…

View Comment

jedbrown commented on issue flame/blis#790.

I hit this error in the default Arch Linux build because it now uses -fno-omit-frame-pointer as do many distributions now. The rationale, as desribed in the second link, is to facilitate full-system profiling. To reproduce on BLIS master, …

View Comment

IanLee1521 commented on issue elastic/eui#7702.

> We’ve received a request from another team for 8.14 backports, so I’ll go ahead and add this one to the list as well since we’re doing one anyway and since you’ve raised it! …

View Comment

artv3 commented on issue LLNL/RAJA#1640.

I’m going to close this PR in favor of a new one. Hopefully the new PR will have better luck with the CI. …

View Comment

jeremylt commented on issue Eudicods/outworlds-wastes#21.

Mention common number of BSPs…

View Comment

trws commented on issue spack/spack#44068.

Looks like this all passed, but the approve got cleared by merging the comment @alecbcs….

View Comment

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

rather odd, if i build with –enable-debug and run mpi4py by hand singleton doesn’t fail….

View Comment

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

With this modification, runtime is down to ~26min – no need to change the time limit….

View Comment

jameshcorbett commented on issue NearNodeFlash/NearNodeFlash.github.io#159.

There’s already a fix for this (I’m fairly sure) but the rest of the flux team doesn’t want me to put it in place yet on elcap, because they’re working on sorting out some other issues. One thing you can do to avoid it (inconvenient I know sorry) is to force flux to choose specific compute nodes with flux run –requires=hosts:elcap[12-15] or similar….

View Comment

bgunnar5 commented on issue LLNL/merlin#476.

@lucpeterson I think the python 3.7 test is throwing warnings since it has an older version of kombu/celery. I think it’s time we drop support for 3.7 considering it’s already reached EOL. I know 3.8 reaches EOL this year as well but we should probably maintain support for it so long as LC keeps it on systems….

View Comment

markcmiller86 commented on issue LLNL/Silo#374.

> Another step shall be for mixed materials ; I think mixed materials is a typical Finite Volume issue, so high order Finite Elements are orthogonal to that question ; so, as for external faces, one could degenerate quadratic to linear for mixed materials. …

View Comment

jedbrown commented on issue rust-lang/rust#124874.

> @jedbrown So, T-libs-api gets to decide whether the name on f32/f64 is good, basically. It might be best to land the intrinsic and then PR the public safe API. …

View Comment

alizma commented on issue openjournals/joss-reviews#6531.

Yes, I added the errors I ran into with examples to an existing issue about errors in the examples: https://github.com/LLNL/echemfem/issues/10, which has some discussion about fixes. …

View Comment

brendankeith commented on issue mfem/mfem#4289.

@tarikdzanic, @dohyun-cse I am very excited to see this! …

View Comment

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

I almost merged this, but to my utter shock I think I just ran into a bug in git of all things… revising….

View Comment

milroy commented on issue flux-framework/flux-k8s#69.

I’m in the process of reviewing this PR, but it is taking quite a while. There’s a lot of churn in the commits, e.g., fluence.go is updated 17 times across commits. That makes it difficult to track the desired final state and also causes me to wonder if my comments will be obviated by a later commit….

View Comment

IanLee1521 commented on issue elastic/kibana#17888.

Another one I thought of that would be good to see is being able to user configure / override my personal default time window. …

View Comment

tgamblin commented on issue spack/spack#44061.

Let’s move the deprecation rate to 2 releases based on feedback from @haampie that users generally don’t want to change this quickly. After 1.0 we should probably go even slower than that (if we remove options at all)…

View Comment

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

i’ll open a separate pr to add some of the upstream ci sanity checks then rebase this one. the description is a little light because its to commits. I’ll fix that up….

View Comment

cjekel commented on issue huggingface/transformers#30056.

Thanks @fxmarty …

View Comment

jedbrown commented on issue RDycore/RDycore#174.

The $L_\infty$ norm will generally not converge unless the solution is smooth. What does this solution look like?…

View Comment

dylan-copeland commented on issue LLNL/libROM#284.

We need 1 more approval to fix the broken libROM build. @chldkdtn @siuwuncheung @dreamer2368 …

View Comment

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

@trws: yes, looking now….

View Comment

balos1 commented on issue LLNL/sundials#464.

What CMake version are you using? As of SUNDIALS 7, when you build SUNDIALS with MPI enabled all of SUNDIALS requires MPI because of the new SUNComm (so that is why the symbol is required). However, CMake should be adding the MPI::MPI_CXX target in and thus the MPI libraries should be getting linked to (even though youre compiling with g++). In older CMake versions it did a bad job of propagating things related to MPI correctly. …

View Comment

artv3 commented on issue LLNL/RAJAPerf#441.

Hi @rhornung67 , what are the issues that still need to be worked out?…

View Comment

benzwick commented on issue vmtk/vmtk#444.

I tried to use vmtkBoundaryLayer2 like this (import from vmtkcontribscripts because it cannot be found in vmtkscripts): …

View Comment

vsoch commented on issue spack/spackbot#80.

Can you make it longer than wider so it’s easier to read / zoom around?…

View Comment

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

There was a small period of time last night were our docs on ReadTheDocs were down. I was debugging issues with doc builds and created a problem there that I corrected within 10-15 mins. So, if you were unlucky and tried to follow the link then, that may have been the cause….

View Comment

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

> I don’t know how to find that branch, and that link is dead. …

View Comment

alecbcs commented on issue spack/spack#43691.

Sorry @wdconinc, @haampie beat you on this with #43921….

View Comment

tzanio commented on issue mfem/mfem#3844.

Re-merged in next for testing……

View Comment

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

I don’t like the idea of changing the output format to accommodate a change in the reader, especially if the change is not backward compatible. …

View Comment

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

Turns out autogen being missing just pops a warning in docker-run-checks.sh AFAICT, so assuming this passes that should be cleaned up….

View Comment

grondo commented on issue flux-framework/flux-sched#1196.

flux-core dropped the czmq dependency awhile ago….

View Comment

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

Thanks! I fixed the conflict and will set WMP….

View Comment

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

Thanks! I’ll set MWP….

View Comment

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

> One other thought - if we’re changing the prototype anyway, would it be a slight improvement if the pointer to the data were returned as an argument and the length as the function return value, more like read(2)? …

View Comment

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

Something strange happened with the flux-pmix test, it looked like the build succeeded then the next step failed with no output. Bounced it assuming it’s something spurious in actions, but worth noting….

View Comment

markcmiller86 commented on issue LLNL/Silo#374.

We’ll also have to adjust the README.md and some of the docs/*.md files too but we do this in a sep. PR on the ghpages branch….

View Comment

jeremylt commented on issue Eudicods/outworlds-wastes#6.

Note - swap to double heat sinks level 3?…

View Comment

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

Also, GetExportOptions seems to hang if OpenGUI() was called prior….

View Comment

uvilla commented on issue pecos/tps#275.

Containers for Tioga and LS6 at TACC…

View Comment

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

could use sinfo to get version info. not sure that’s really any different though….

View Comment

cyrush commented on issue mfem/mfem#4280.

I think that is correct. …

View Comment

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

Oh, heh. @grondo beat me!…

View Comment

adrienbernede commented on issue LLNL/uberenv#132.

--setup-only stops before the environment is create. I need to stop after the environment is created, so that I can customize it before installing….

View Comment

hppritcha commented on issue spack/spack#43961.

yes. including it in this correction for romio in this PR would be fine with me. nice catch!…

View Comment

hppritcha commented on issue openpmix/prrte#1973.

we still need PR https://github.com/openpmix/prrte/pull/1972…

View Comment

hppritcha commented on issue openpmix/openpmix#3342.

clearly something’s changed for macos….

View Comment

alizma commented on issue openjournals/joss-reviews#6506.

Hi, we’d also like to suggest jaimeliew1 as a potential reviewer who has done similar work. …

View Comment

tomstitt commented on issue mfem/mfem#4279.

looks great! thanks @v-dobrev !…

View Comment

aschaf commented on issue mfem/mfem#4140.

Hi @psaber09, sure. When do you have time? I am in CEST, so I guess we have a 6h difference….

View Comment

cjvogl commented on issue donavanbecker/homebridge-resideo#817.

+1 I also have the same problems as discussed above: cannot relink account after upgrading v2.0.0 or v.2.0.1. Second time rolling back to v.1.4.6 and having to reset all related automations….

View Comment

bwpriest commented on issue boya66/MuyGPyS#1.

sorry this was a mistake…

View Comment

tpatki commented on issue LLNL/msr-safe#160.

Thanks for testing with the PR #158 and glad that fixed the issue for you!…

View Comment

cyrush commented on issue LLNL/blt#687.

thanks @white238 !…

View Comment

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

@nicolemarsaglia this looks good to me, we can discuss today!…

View Comment

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

@GuillaumeMercier does the new threading methodology cover this case now?…

View Comment

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

> In fact I think my existing test would work, it just needs some extra jobs added and a stats check. …

View Comment

adayton1 commented on issue LLNL/camp#146.

> > The Docker rocm tests are failing, I think probably because the machine doesn’t have any AMD GPUs. Should I turn off the testing and just do the build? Or maybe I could try to build in a filter so only the host tests run. …

View Comment

BradWhitlock commented on issue LLNL/UMT#25.

You are correct. Try the following in the meantime or always define TETON_CONDUIT_HAS_TILED_FUNCTION. …

View Comment

artv3 commented on issue LLNL/RAJAPerf#439.

> template parameter name change request for consistency. …

View Comment

CamStan commented on issue LLNL/UnifyFS#790.

Thanks for testing this @wangvsa …

View Comment

artv3 commented on issue LLNL/RAJA#1628.

> Please add some notes about this in the user guide. Also, did you notice any performance differences with this change? …

View Comment

jedbrown commented on issue CEED/libCEED#1570.

It passed, but somehow the result wasn’t reported….

View Comment

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

> And is the idea of “jobtap” like you are tapping a job on the shoulder? tap tap, are you my dependency? …

View Comment

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

I can try writing one! And I understand this point: …

View Comment

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

Given @grondo ‘s comments, these changes are not necessary. Closing this PR….

View Comment

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

@GuillaumeMercier, I think qvi_scope_ksplit() is ready for consideration in your threading work. Please see tests/test-mpi-scope-ksplit.cc as an example on how to use it….

View Comment

garlick commented on issue chaos/powerman#184.

Thanks! Excellent work running that down by the way….

View Comment

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

This happens only for the non-fixed-width font choices. So, there is some kind of feedback mechanism that is causing the setting to oscillate slightly between two different width states….

View Comment

trws commented on issue spack/spack#43811.

It does, but it has a direct dependency on libmpfr.so.6, when RHEL and company only provide .so.4….

View Comment

vsoch commented on issue snakemake/snakemake-storage-plugin-gcs#41.

Thansk @jjjermiah ! You probably want to remove the __pycache__ stuff….

View Comment

hppritcha commented on issue ofiwg/libfabric#9835.

oh I’m on main at 717ebc5dcba5a41221374bdfd64b75efe20b5b05…

View Comment

vsoch commented on issue memgraph/memgraph#1975.

Thanks! So if I understand correctly, I’d need to have everything represented in key values pairs associated with nodes. For example, if a physical node resource is scheduled, it might have scheduled=true, or (because that is too simple for a need to find a time into the future) more likely, a timestamp when the node will next be free that increases as it is scheduled. …

View Comment

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

For future reference, Fluxion vertices can be manually marked as down by sending an RPC like so (Python): …

View Comment

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

Coverage got stuck here - restarting. …

View Comment

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

Capturing notes from discussion with @slabasan . …

View Comment

balos1 commented on issue LLNL/sundials#461.

Closed by #462…

View Comment

BradWhitlock commented on issue LLNL/axom#1218.

I’m just making a note here that it looks like the target host code builds Axom without MFEM, and with a serial RAJA. This impacts our ability to take MIR topological output and compute volume fractions using the intersection shaper. The intersection shaper currently uses MFEM….

View Comment

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

Thanks for your report! That is concerning. If you are not doing so already, I would recommend using visit -v 3.3.3 for your needs. If you could provide us a reproducer that would be very helpful. If you need to provide via LC let me know and I can give you my username….

View Comment

alecbcs commented on issue spack/spack#43854.

Try building all the way up locally and see if that fixes it. We may have a bug in the public MacOS cache. I know there was some discussion about a bug in binary relocation on MacOS a while back. This may be a good reproducer if true….

View Comment

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

@Heinrich-BR, can you post the exact command line you use (and any modifications you made to examples/ex1p.cpp) when you get the segfault? I can try to reproduce it. Also which version/git-hash of MFEM are you using?…

View Comment

jandrej commented on issue mfem/mfem#4268.

Are you using the “performance” examples or is this the regular examples/ex1p?…

View Comment

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

> Oh, there was some specific issue with musl (or whatever libc is used by alpine) here. It doesn’t really reload DSOs on dlclose()/dlopen() so you’ll need a memset(). …

View Comment

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

Hmm, is it of concern this didn’t get caught in our GitLab CI? Our logs from last night with intel-classic and the mvapich2 compiler show: …

View Comment

markcmiller86 commented on issue LLNL/Silo#372.

So, it looks like this is possible… …

View Comment

adayton1 commented on issue LLNL/CHAI#255.

> @adayton1 It is ready for review. I increased the lassen allocation in the last commit. …

View Comment

tgamblin commented on issue spack/spack#43597.

> @tgamblin said this is by design, but that sounds incorrect to me as the following two things aren’t equivalent, but I would expect them to be …

View Comment

tomstitt commented on issue mfem/mfem#3844.

Thank you @sebastiangrimberg for fixing the unit tests! I completely neglected those =| …

View Comment

v-dobrev commented on issue mfem/github-actions#10.

Note that the master branch is actually behind the branch we actually use in the mfem actions, v2.4. Maybe we should just merge v2.4 in master….

View Comment

bhatele commented on issue hatchet/hatchet#541.

@amovsesyan can you fix the formatting changes?…

View Comment

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

Thanks for the detailed and helpful feedback @trws and @garlick! Setting MWP….

View Comment

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

Bounced the go test runner, looks like it failed to fetch the image for some reason. Feel free to set MWP @milroy, great stuff!…

View Comment

cmoussa1 commented on issue flux-framework/flux-docs#216.

Thanks for the review @garlick! I just pushed up some fixes based on your suggestions and opened an issue in flux-accounting to go ahead and move this document in the near future. Since you approved already, I can set MWP here shortly :-) …

View Comment

IanLee1521 commented on issue elastic/kibana#152957.

I just bumped into this myself in my 8.13.2 instance. The following was the Kibana UI alert message: …

View Comment

markcmiller86 commented on issue spack/spack#28930.

On macOS 12.7.4 anyways, I was able to rid myself of libffi dependence using -DCMAKE_DISABLE_FIND_PACKAGE_LibFFI=TRUE. It at least built and installed without error. I don’t know if the build is somehow invalid as a result….

View Comment

alizma commented on issue openjournals/joss-reviews#6506.

Hi @HaoZeke, thank you for being our editor! …

View Comment

brendankeith commented on issue mfem/mfem#4248.

@vladotomov, I updated the example description in an attempt to address your comments above. Please take a look when you get a chance…

View Comment

uvilla commented on issue hippylib/hippylib#70.

The derivation in Ghattas Willcox’s monograph follows an optimize-then-discretize approach. …

View Comment

markcmiller86 commented on issue betterscientificsoftware/bssw.io#2077.

> So anyone who didn’t give us a GitHub link for their profile would be represented by the BSSw.io B logo rather than a picture. …

View Comment

adrienbernede commented on issue LLNL/Umpire#875.

Ready for review….

View Comment

jeremylt commented on issue CEED/libCEED#1502.

Ok, need a multi-element apply method and to tweak the tests for restrictions to handle different packing strategies of the e-vec…

View Comment

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

Just capturing here how I did this with expressions. The problem with the expression approach is that applying the scaling to different variables being plotted requires editing the expressions. …

View Comment

samuelkgutierrez commented on issue openpmix/openpmix#3339.

I’m glad you found the issue, @rhc54. Thanks for the explanation of the problem and possible solutions….

View Comment

tzanio commented on issue mfem/mfem#3844.

thanks @sebastiangrimberg !…

View Comment

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

I fixed a case in find_vertex where the uninitialized vtx was getting returned when not found in the graph. Since an uninitialized vtx_t != boost::graph_traits<resource_graph_t>::null_vertex () that caused the check to identify duplicate vertices when initializing the graph. …

View Comment

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

Awesome, thanks again @vsoch!…

View Comment

grondo commented on issue flux-framework/flux-sched#1183.

Thanks, I’ll set MWP …

View Comment

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

I am sorry codecov bot, please forgive my sins :pray: …

View Comment

vsoch commented on issue flux-framework/flux-k8s#74.

Looks like the upstream is still a moving target! They bumped kubernetes now up to 1.29x. https://github.com/kubernetes-sigs/scheduler-plugins/commit/2d20310880323ae307312d4d4fdfa78c2267073c. It changed a few function signature, trying to figure that out now….

View Comment

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

@tzanio, I merged master here but some of the edits in CHANGELOG made it difficult to figure out what’s new and what’s been editted here, so I kept both versions of the conflict; to remind us about this, I added this note to the TODO bullet about the CHANGELOG: …

View Comment

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

@GuillaumeMercier qvi_scope_ksplit() will likely be a convenient call for your threading work. I still need to iron out some limitations, but for splits that have no more colors than pieces (a limitation that I need to lift, in general), this seems to be working okay….

View Comment

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

I’ll check the match policy, but I’ll also see if I can reproduce locally….

View Comment

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

I pushed that change. OK to set MWP?…

View Comment

balos1 commented on issue LLNL/sundials#458.

@drreynolds Are you OK if we merge this PR before #459 since it only adds a new unit test? …

View Comment

cyrush commented on issue LLNL/conduit#1275.

@stgeke …

View Comment

JustinPrivitera commented on issue LLNL/conduit#1111.

I think it might actually be possible to handle strided structured case when reading materials. I just need to be smart about how I index into the matlist. The tools to help me do that should be in the DBmaterial. I don’t think it will require transforming the matset to remove entries….

View Comment

cyrush commented on issue LLNL/camp#149.

@rhornung67 awesome, thanks for quick add!…

View Comment

adrienbernede commented on issue LLNL/CARE#255.

@adayton1 this is ready for review !…

View Comment

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

Azure running ROCm 6.0.0 container is good, still working spack updates to related to umpire vs fmt vs blt for the other azure cases. …

View Comment

tgamblin commented on issue spack/spack#43724.

LGTM but can we get a test?…

View Comment

adrienbernede commented on issue spack/spack#43644.

I found that in the commit at the root of my PR: …

View Comment

IanLee1521 commented on issue herrBez/elastic-blog-openshift-logging#5.

Also opened https://github.com/elastic/integrations/issues/9648 to make this suggestion there….

View Comment

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

Comment addressed, rebased with master, setting MWP….

View Comment

jedbrown commented on issue EnzymeAD/rustbook#11.

I think this can be merged now, and will be used by https://github.com/EnzymeAD/rust/pull/117…

View Comment

jedbrown commented on issue rsmpi/rsmpi#183.

Something must be different between your configuration and the Windows CI where rsmpi is tested. The error is coming from bindgen. I’d recommend seeing if you can run cargo test in the bindgen repo, and if not, reporting in their issues or discussions. https://github.com/rust-lang/rust-bindgen/discussions…

View Comment

jedbrown commented on issue openjournals/joss#1297.

I wrote this earlier on slack: …

View Comment

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

I also checked current main without the commits in this PR and test_ulfm.py does still fail. Passes for me with these commits added in….

View Comment

brendankeith commented on issue mfem/mfem#4248.

Thank you for the comments, @dohyun-cse! With help from @psocratis, I was able to implement all of them. …

View Comment

bhatele commented on issue hpcgroup/pipit#114.

New reader in #123….

View Comment

artv3 commented on issue LLNL/RAJAPerf#439.

Hi @rhornung67, I believe I have this working now. We will need to update RAJA though. …

View Comment

alizma commented on issue openjournals/joss-reviews#6506.

@editorialbot generate pdf…

View Comment

tomstitt commented on issue mfem/mfem#3844.

View Comment

jandrej commented on issue mfem/mfem#4251.

You have to export using Exodus .e format with the option Use NetCDF4/HDF5 Format

View Comment

adrienbernede commented on issue LLNL/CHAI#255.

@adayton1 It is ready for review. I increased the lassen allocation in the last commit. …

View Comment

adayton1 commented on issue LLNL/CHAI#255.

@adrienbernede, is this still a draft, or is this ready for review?…

View Comment

bgunnar5 commented on issue LLNL/merlin#472.

Luc approved this via teams…

View Comment

artv3 commented on issue LLNL/RAJA#1626.

> Please change casts to idomatic C++ …

View Comment

jedbrown commented on issue EnzymeAD/rust#84.

I don’t think the alias can work because that’s only in the shell, not even an exported environment variable. Perhaps we could place a ld.lld symlink in PATH and export it. https://lld.llvm.org/#using-lld …

View Comment

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

I will look into a fix for this next week….

View Comment

tzanio commented on issue mfem/mfem#3904.

Merged in next for testing……

View Comment

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

By the way, I tested this PR with both GNU make and CMake and both single and double precision – everything seems to work as expected. …

View Comment

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

I’ve been banging on this every way I can think of, and so far can’t reproduce the scenario without a resource.cancel RPC coming from somewhere. …

View Comment

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

There is also something called a key value flyweight, I wonder if we need to use that for some of the subsystem maps? https://www.boost.org/doc/libs/1_79_0/libs/flyweight/example/key_value.cpp. I also don’t know the difference between when they show: …

View Comment

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

Thanks for the feedback @garlick! Setting MWP….

View Comment

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

Hm, it seems like we should have gotten a slightly more descriptive error message here: …

View Comment

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

I didn’t find anything wrong with this in testing but I did discover one omission that could be corrected at a later date. Only the rank 0 broker prohibits peers from reconnecting during cleanup. The other ranks are still in run state and are perfectly happy to let peers return to service, only to ask them to shutdown once cleanup completes. It’s not wrong, but it seems like wasted effort that we could fix. Unfortunately it’s a little more than I think I can get done today. …

View Comment

vsoch commented on issue flux-framework/Tutorials#32.

Thanks for the fix @ilumsden ! Next year we can get some basic tests in here to sanity check before merge….

View Comment

agcapps commented on issue LLNL/axom#1321.

Does anyone know of any other IndexType occurrences in slam or elsewhere that might need de-wrinkling?…

View Comment

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

Can we close this ticket then?…

View Comment

samuelkgutierrez commented on issue open-mpi/ompi#12464.

View Comment

jeremylt commented on issue freeCodeCamp/freeCodeCamp#52718.

This test is failing for me and a user on the Discord. I’m using Firefox and they are using Edge….

View Comment

garlick commented on issue flux-framework/flux-sched#1168.

> I made your suggested change, but I didn’t need sudo to run stats before. …

View Comment

grondo commented on issue flux-framework/flux-sched#1170.

I don’t think this was a ctrl-C because those exceptions are logged like this: …

View Comment

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

Ok, this is the behavior now when an equals sign w/ an option is passed to the script on macOS: …

View Comment

chu11 commented on issue chaos/powerman#94.

> Edit: oh, it would be visible on the command line so not equivalent. Hmm, not sure if that’s a big deal though. I mean not for us anyway on a management node with restricted access. …

View Comment

garlick commented on issue chaos/powerman#175.

Eh, that still feels pretty hacky for me….

View Comment

balos1 commented on issue LLNL/sundials#454.

Hi @adriendelsalle, yes this is a known issue. A fix is currently in development. We would appreciate it if you could test the fix out when it is ready. …

View Comment

BradWhitlock commented on issue LLNL/conduit#1269.

Wait a minute. I forgot to re-enable a test that started misbehaving….

View Comment

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

Looks like in the past you may have merged from this branch into the cinema output path branch: …

View Comment

trws commented on issue spack/spack#43341.

None that are all that useful, it should definitely be there unless it’s cleared out for some reason….

View Comment

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

To point 1: version compatibility is challenging enough that we’ll stick to our internal version until there is a compelling reason to also support system-installed hwloc libraries. …

View Comment

cyrush commented on issue Phlow/feeling-responsive#277.

Sorry, opened in upstream on accident. …

View Comment

daboehme commented on issue LLNL/Caliper#403.

The time for event.begin#region events is the time since the previous event (or, for the first region, the time since Caliper was initialized). Note that these are exclusive times, if you need inclusive times (i.e. the total time in a each region including its children) add CALI_TIMER_INCLUSIVE_DURATION=true. For the MPI rank add mpi to the CALI_SERVICES_ENABLE list….

View Comment

uvilla commented on issue FEniCS/dolfinx#3092.

@garth-wells @jhale @jorgensd : Requested changes have been addressed. …

View Comment

jedbrown commented on issue EnzymeAD/rust#110.

Thanks! Is this deliberately using x: &[&f64] instead of x: &[f64]? As written, I think the reverse will be unsound due to writing through an immutable reference….

View Comment

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

If you want to poke at these failing tests, you can xsu fluxci and see the logs, along with the binary they’re running under: …

View Comment

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

I wonder if somehow an older version of ion-R.py is being picked up?…

View Comment

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

Thanks! Set MWP….

View Comment

daboehme commented on issue LLNL/axom#1315.

I’d generally recommend using cali_begin_region(const char* name) and cali_end_region(const char* name) for marking the regions. The cali::Caliper API is considered internal so there’s a higher risk of things changing in the future, although it’s not very likely here. The cali_end_region call checks if the given value matches the top of the Caliper region stack and gives an error if it doesn’t. You can do the same with cali::Caliper::end_with_value_check(const Attribute&, const Variant&), which is what cali_end_region does under the hood. That said your current version is going to be slightly faster since it skips this check….

View Comment

jeremylt commented on issue CEED/libCEED#1557.

There’s a failure on re-run, so we need to sort that out before merging…

View Comment

tgamblin commented on issue spack/spack#43548.

> I was talking about forward compatibility specifically, which is relevant for sharing build caches across systems …

View Comment

brendankeith commented on issue mfem/mfem#4236.

Thanks for the fix, @psocratis!…

View Comment

CRobeck commented on issue llvm/llvm-project#88076.

Probably want to add a test in: …

View Comment

bhatele commented on issue hatchet/hatchet#542.

Do you know why some tests failed?…

View Comment

bgunnar5 commented on issue LLNL/merlin#458.

@lucpeterson yeah I think ultimately baking this into Merlin rather than having it as a standalone example is probably the way to go. This also means that I could expand it to work for more than just slurm too. …

View Comment

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

@Himscipy @IgorBaratta

tomstitt commented on issue mfem/mfem#4229.

> @tomstitt, do we need to expedite this for the mfem-4.7 release? …

View Comment

adayton1 commented on issue LLNL/axom#1307.

I don’t have any specific examples yet, but I will be attempting to discover the cases where I am seeing inconsistencies over the next couple weeks….

View Comment

tzanio commented on issue mfem/web#248.

@v-dobrev, should we review/merge this now that https://github.com/mfem/mfem/pull/3598 is merged?…

View Comment

jedbrown commented on issue EnzymeAD/enzyme-explorer#3.

Thanks. It’s probably possible to tar up the whole build directory, but I wanted to get x dist working because that’s clearly the right way to distribute, and necessary for upstreaming….

View Comment

vsoch commented on issue snakemake/snakemake-executor-plugin-googlebatch#29.

That

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

fixed via multiple commits both in openmpi and prrte…

View Comment

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

Thanks for the reviews! Setting MWP….

View Comment

grondo commented on issue flux-framework/flux-sched#1162.

> ``` …

View Comment

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

Thanks!…

View Comment

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

We need this bug fix for single builds: https://github.com/mfem/mfem/pull/4235…

View Comment

tzanio commented on issue mfem/mfem#4135.

@ebchin, can you add that

brendankeith commented on issue mfem/mfem#4231.

After talking to @psocratis, I’ve decided to add one more new function to this PR. I removed the “ready for review” label for now…

View Comment

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

Thanks! Setting MWP….

View Comment

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

would it be worthwhile to do the ls -l test2 after the chmod, to make sure that is actually working?…

View Comment

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

» I think if you have SQL access you don’t need an injection attack :-) …

View Comment

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

You might also want to think through what happens if the service goes from up to down, e.g. the connection terminates, if that’s even possible. The service should be able to shut down cleanly without affecting jobs - same with restarting :thinking: …

View Comment

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

was just typing up a response! Sorry that I didn’t make that clearer @chu11 - I should update the PR description…

View Comment

chu11 commented on issue flux-framework/flux-accounting#357.

slowly starting to review this, and just wondering what is the plan to transition from the old job-archive? perhaps there needs to be a script to do a “one time copy” of what you need from job-archive? (apologies if this is discussed above and I missed it)….

View Comment

chu11 commented on issue chaos/powerman#150.

closing this as almost everything has been divided up…

View Comment

daboehme commented on issue LLNL/Caliper#545.

Hm, that’s interesting. Looks like it’s simply not capturing anything at runtime. Are there any Caliper annotations in the code that you’d expect to see? If there are MPI calls you can try runtime-report,profile.mpi again so at least they should show up….

View Comment

jeremylt commented on issue CEED/libCEED#1554.

@ me when its ready to go on the PETSc side and I’ll rebuild CI on Noether…

View Comment

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

Hmm….if glxgears won’t work in that scenario…neither will VisIt. …

View Comment

adrienbernede commented on issue spack/spack#41145.

@rbberger @alecbcs Umpire versions 2023.06.0 and 2024.02.0 have been added in https://github.com/spack/spack/pull/41375….

View Comment

alizma commented on issue openjournals/joss-reviews#6506.

@editorialbot generate pdf…

View Comment

cjvogl commented on issue mfem/mfem#4115.

@acfisher, I believe all suggestions have been applied & concerns addressed. If am I missing any, I am happy to quickly tackle them….

View Comment

acfisher commented on issue mfem/mfem#4115.

@cjvogl I believe there is one suggestion by Veselin and then this is ready to get added to next, is that correct?…

View Comment

tomstitt commented on issue mfem/mfem#4229.

> Can’t you just set the desired Umpire allocators with MemoryManager::SetUmpireHostAllocatorName, MemoryManager::SetUmpireDeviceAllocatorName, and MemoryManager::SetUmpireDevice2AllocatorName? What is not going to work? …

View Comment

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

Merged in next for testing……

View Comment

garlick commented on issue flux-framework/flux-sched#1129.

(Sorry I thought this had recurred but I was a little too far back in the logs. This has not been seen since the update to flux-sched-0.33.0) Reclosing….

View Comment

chu11 commented on issue flux-framework/flux-docs#267.

looks good…

View Comment

garlick commented on issue flux-framework/flux-docs#267.

> Why is it wrong? It’s mostly meant to demonstrate why the instances are useful. There isn’t really anything I can find that shows it beyond that. …

View Comment

milroy commented on issue flux-framework/flux-core#4312.

> It was prototyped in https://github.com/flux-framework/flux-core/pull/5818 but I only tested with sched-simple since I knew Fluxion would not handle it. …

View Comment

garlick commented on issue chaos/powerman#168.

This turned out really nice! I added a couple of comments….

View Comment

bgunnar5 commented on issue LLNL/merlin#470.

Link to the page with most of the updates: https://merlin–470.org.readthedocs.build/en/470/user_guide/configuration/containerized_server/

View Comment

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

We’ll want to add that to visit-build-closed as well then, and both visit-install-open and visit-install-closed. Alternatively we could just make sure the hardcoded result directories have permissions set correctly….

View Comment

jedbrown commented on issue EnzymeAD/rust#106.

Or should I try working around this by using an output argument instead of return value?…

View Comment

jedbrown commented on issue CEED/libCEED#1456.

I suppose we’d like for the noether-sycl job to run. You can push the branch to this repo (I don’t think it’ll need to create a PR) or to the GitLab repo, where you should be able to see the pipeline. https://gitlab.com/libceed/libCEED/-/pipelines…

View Comment

trws commented on issue spack/spack#41375.

Excellent!…

View Comment

jandrej commented on issue mfem/mfem#4217.

In general this is possible, yes. Creating the derivative on an element-vector level is more resource intensive than necessary. We’re in the process of creating streamlined interface to allow Enzyme enabled derivatives on the quadrature point level which will be the most efficient and GPU compatible way….

View Comment

IanLee1521 commented on issue elastic/endpoint#69.

@nfritts - Makes sense why you’d do it that way. …

View Comment

vladotomov commented on issue CEED/Laghos#185.

See the paper listed at the start of the README, the code mostly follows those equations. …

View Comment

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

@editorialbot assign @jedbrown as editor…

View Comment

vsoch commented on issue singularityhub/singularity-cli#220.

I think it

tzanio commented on issue mfem/mfem#4203.

Hi @stefanozampini, …

View Comment

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

To review the current situation, in rc1 we have: …

View Comment

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

Sorry I missed that in review!…

View Comment

jedbrown commented on issue EnzymeAD/rust#31.

Closing since further activity will be in https://github.com/EnzymeAD/rustbook…

View Comment

trws commented on issue spack/spack#41375.

@spackbot rebuild everything…

View Comment

tgamblin commented on issue spack/spack#43094.

It looks like this is fixed in xz upstream, or at least @skosukhin has some commits there. Please don’t add newer versions of xz until this unfolding compromise of xz 5.6 gets resolved: …

View Comment

adrienbernede commented on issue spack/spack#41375.

Also, I am tempted to ask for a “rebuild everything” since changes in CachedCMakePackage are not captured well by the rebuild detection mechanism. What do you think?…

View Comment

vsoch commented on issue spack/spack#43331.

Again tonight. …

View Comment

grondo commented on issue flux-framework/flux-sched#1159.

There are also some minor improvements in flux-sched v0.33.0. I think the system in question is still at v0.32.0….

View Comment

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

Thanks both of you, that’s very helpful. They’re all registered in qmanager.cpp, it’s a bit tangled but now that I know where the root is it’s much easier to follow….

View Comment

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

alternate idea i thought of this morning, under the assumption that job-list is still processing things, we could simply put some waits into the tests to wait for those particular tests to become inactive. The fact job-list is still loading things may not necessarily be something we care about? …

View Comment

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

thanks for approving here @jameshcorbett! Just rebased to catch up after #435 and added it as part of the release notes…

View Comment

brendenpetersen commented on issue dso-org/deep-symbolic-optimization#91.

@iglesias Any chance you want to share your pytorch implementation? :)…

View Comment

trws commented on issue desul/desul#120.

This is a significantly different case than when using relaxed to protect a reduction. That said, we do have an issue here. Currently using this with acq_rel or similar atomics should work just fine, because they’ll add the appropriate fences around it, but relaxed as it sits is wrong in that test. Not only because this is relaxed though, it’s because both this is relaxed and the memory operations inside the SuperScalar class are non-atomic, so when using a relaxed memory order on the outer atomic there’s no flush to ensure those writes leave the inner caches of the writer. …

View Comment

balos1 commented on issue LLNL/sundials#447.

Related to #423….

View Comment

adayton1 commented on issue LLNL/Umpire#879.

> Is there a good way to test the various supported fmt external vs. submodule combinations? @adayton1 will you check your use case? …

View Comment

daboehme commented on issue LLNL/Caliper#500.

Fixed in #549 …

View Comment

vladotomov commented on issue CEED/Laghos#183.

@sungho91, sure, more orders can be added. …

View Comment

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

#19003…

View Comment

acfisher commented on issue mfem/mfem#3443.

@jandrej, I believe this is all being addressed in PR https://github.com/mfem/mfem/pull/4115 …

View Comment

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

One remark: …

View Comment

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

With the assumption that a fragment contains a subset of the job’s broker ranks but the entire R (i.e., the full R for each broker rank) for each fragment broker rank, adding this support should be straightforward. …

View Comment

vsoch commented on issue flux-framework/flux-docs#267.

Oh neat, readthedocs has a new dashboard! …

View Comment

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

> we can unpack R like the following: …

View Comment

IanLee1521 commented on issue elastic/kibana#174497.

@teresaalvarezsoler and I were talking about this a bit today, and I wanted to through a particular use case for this where it could help a lot, namely if this was a viable way to do some “advanced editing” of an existing dashboard. …

View Comment

bgunnar5 commented on issue conda-forge/staged-recipes#25898.

I’m ok to be a maintainer…

View Comment

chu11 commented on issue chaos/powerman#167.

> Oh I was figuring you would cap the delay at 10s or so (or whatever makes sense) …

View Comment

garlick commented on issue chaos/powerman#167.

Oh I was figuring you would cap the delay at 10s or so (or whatever makes sense). If we need to make it tunable, then I would add a command that lets you sets the start, multiplicative factor, and cap instead of only allowing only a non-adaptive delay to be configured. But I’m not sure its needed….

View Comment

IanLee1521 commented on issue LLNL/llnl.github.io#674.

I admit I’m not the best front end expert, but this seems reasonable to me….

View Comment

cyrush commented on issue LLNL/axom#1301.

Do you need to use fmt as a library – could header only be sufficient? …

View Comment

acfisher commented on issue LLNL/acrotensor#3.

I obviously haven’t looked at this in quite a while and had no idea folks were looking at it. I believe that should work as long as you don’t call anything that needs the data on the host. Did you end up giving it a try?…

View Comment

jeremylt commented on issue CEED/libCEED#1542.

Confirmed fix works for Ratel too …

View Comment

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

@vijaysm and @iulian787 …

View Comment

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

This came up in the quarterly today. It seems that admins may want to exclude some cores from scheduling even on compute nodes, so those cores can be dedicated to OS tasks, kind of like is done on Sierra….

View Comment

adayton1 commented on issue LLNL/CHAI#254.

> Noooooo ! (lol) I forgot to mark it as [WIP]: typically, when updating RADIUSS Spack Configs (RSC), as long as I am referencing a commit in a RADIUSS Spack Configs PR (not main), it for testing. Then I merge in RSC, and update the related PRs in the projects. …

View Comment