vsoch commented on issue flux-framework/flux-coral2#411.
This does not require review yet. Updates for this PR include: …
tzanio commented on issue mfem/mfem#5072.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
chu11 commented on issue flux-framework/flux-core#7158.
re-pushed, converting –async to –fast….
cjekel commented on issue cjekel/similarity_measures#40.
LGTM! Thank you! Sorry this took so long to get in….
trws commented on issue LLNL/camp#190.
As it sits I’m ok with this, but I expected based on the description that there would be a specialization of std::hash in here so that unordered_map and similar could hold them. Is that not part of what you wanted to accomplish?…
JustinPrivitera commented on issue visit-dav/visit#17791.
3.5.0 deliverables have been completed so I am leaving the milestone at 4.0….
tgamblin commented on issue spack/spack#51137.
> Finally, just a comment on colors (disregard this if you don’t agree). I wouldn’t use red for run dependencies, cause red is usually associated with failures. Maybe yellow? …
imguoguo commented on issue radxa-pkg/qualcomm-profiles#7.
>
v-dobrev commented on issue mfem/mfem#4431.
@tzanio, I just pushed a small update to increase the timeout on Matrix – please re-merge it in next when you get a chance….
artv3 commented on issue mfem/mfem#2176.
I think we can close this PR, its from a long time ago… …
jameshcorbett commented on issue flux-framework/flux-sched#1403.
Hi @kostiscpp sorry I forgot to answer this. Are you still interested in a solution? As @grondo mentioned, Fluxion needs to be configured so that it knows about the topology of each node, and specifically about the placement of sockets. How are you starting your Flux instance at the moment?…
grondo commented on issue flux-framework/flux-core#7158.
Just for my own curiosity, when would we not want the “fast” behavior? …
garlick commented on issue flux-framework/flux-core#7157.
Dropcache drops “clean” blobs from the cache but that doesn’t mean other things (KVS, metadata) aren’t still referencing those blobs….
hppritcha commented on issue open-mpi/ompi#13436.
closing this and opening a new one with different selection of shas…
hppritcha commented on issue mpi-forum/mpi-forum.github.io#27.
@wesbland updating sessions wg leads…
JustinPrivitera commented on issue mfem/mfem#4307.
@cyrush can we let this be automatically closed since you have solved this in VisIt? Or do we still need this?…
jandrej commented on issue mfem/mfem#4485.
ping @nnpp-schneier @victor-decaria-nnl …
adrienbernede commented on issue mfem/mfem#4431.
@v-dobrev thank you for updating this PR. Glad to see you leverage latest support for matrix added very recently. …
homeomorfismo commented on issue mfem/mfem#4498.
@pazner Makes sense to me!…
grondo commented on issue flux-framework/flux-coral2#430.
I opened an issue here to discuss possible solutions: …
jameshcorbett commented on issue flux-framework/flux-coral2#430.
Putting documentation in the docstring makes sense to me. This plugin certainly does not require much documentation, what it does is reasonably straightforward….
garlick commented on issue flux-framework/flux-coral2#430.
Hmm, that is a conundrum. I wonder if it would just make sense to have one flux-coral2 page in section 7 with a subsection titled RUN COMMAND EXTENSIONS or similar?…
vladotomov commented on issue CEED/Laghos#196.
@wdhawkins please add 1-2 lines to the README to mention building with Caliper, and I’ll merge this….
cyrush commented on issue visit-dav/visit#5770.
@BenWibking thanks for the update – I am sure adding AMR support was a good amount of work and others could benefit from these improvements even if there are some outstanding issues. Would you like to get your new plugin into visit proper?…
jedbrown commented on issue openjournals/joss-reviews#9244.
Hi @PabloIbannez, @valsson
camierjs commented on issue mfem/mfem#5057.
I updated the [MTOP] solver with GPU support #5072 TODO item to point to the new PR (#5072)….
btalamini commented on issue jax-ml/jax#24295.
Thanks, makes sense. In my case, if I knew that a particular use of the solver wasn’t going to be differentiated, I wouldn’t bother wrapping it with custom_root at all. It’s not really buying you anything if you’re not differentiating over the closed-over variables….
jedbrown commented on issue RDycore/RDycore#348.
An alternative would be to write partition statistics on stdout, like we do in Honee. I guess we could put something like this into -dm_view_partition….
tpatki commented on issue Olympus-HPC/Mneme#51.
Hi @koparasy Yes, please merge. Sorry, just saw this, had missed the notification. …
bgunnar5 commented on issue LLNL/merlin#585.
Thanks for the review @jwhite242. I’ll go back through all of the docstrings/type hints to make sure they’re accurate with the latest changes once this is done….
chrundle commented on issue FLASK-LLNL/ChARGe#28.
Cancelling pull request (separated into two separate requests for backend and experiments/)…
cmelone commented on issue cmelone/hubcast-test#4.
@lc-hubcast help…
cmelone commented on issue brandonlieu2001/cse-data-status#2.
love it…
adayton1 commented on issue LLNL/spheral#415.
Superseded by #417 …
jedbrown commented on issue openjournals/joss#1465.
Looks like Typst recently implemented PDF/A support, which was a blocker earlier. I like Typst, but I have mixed feelings about the cost/benefit of JOSS adding support. JOSS articles are relatively short and Markdown is mostly sufficient. Typst would have the benefit of…
jedbrown commented on issue openjournals/joss#1465.
> Point 1. would be a benefit only if typst was to replace the current markdown+pandoc toolchain, right?…
chrundle commented on issue FLASK-LLNL/ChARGe#28.
Made updates we discussed during Webex meeting in most recent commit…
cyrush commented on issue visit-dav/visit#20586.
In this case, I think he was exporting to cache some results. …
markcmiller86 commented on issue visit-dav/visit#20586.
Ok, it looks like a VTK_CONVEX_POINT_SET is a special case of an arbitrary polyhedron. Apparently, with just the points alone, and nothing else, a VTK_CONVEX_POINT_SET specifies the faces (and edges) of the resulting points using an algorithm to extract the faces of the convex hull….
JustinPrivitera commented on issue visit-dav/visit#20624.
Glad you got this working!…
hppritcha commented on issue open-mpi/prrte#86.
This PR is needed before opening on against OMPI main to move the openpmix and prrte 3rd-party shas. @edgargabriel …
hppritcha commented on issue open-mpi/ompi#13280.
oh for inquiring minds, it looks like several opal params need to be adjusted if building libmpi_abi.so. It looks like these can all be controlled with configure options, e.g.: …
jandrej commented on issue mfem/mfem#5064.
What do you expect calling FormLinearSystem on a NonlinearForm? I don’t see how this should be defined….
v-dobrev commented on issue mfem/mfem#4524.
Re-merged in next for testing……
grondo commented on issue flux-framework/flux-foundry#17.
Feel free to push here directly. I’ll be out of town the next few days…
chu11 commented on issue flux-framework/flux-core#7082.
re-pushed, with possibly a disliked change. --quiet is now advertised that it only suppresses “non-core” diagnostic messages. So Checking integrity of ... diag message is always output to stdout.
…
garlick commented on issue flux-framework/flux-core#7082.
Also, since we’re at the beginning of a release cycle, I wonder if we ought to just add the --repair option and then give ourselves some time to poke at that on test systems?
…
grondo commented on issue flux-framework/flux-core#7137.
Minor suggestion: add a test for handle.attr_remove(not_a_string)…
garlick commented on issue flux-framework/flux-coral2#424.
See also…
garlick commented on issue chaos/powerman#205.
We already support an include directive….
antoniuk1 commented on issue FLASK-LLNL/ChARGe#30.
> @antoniuk1 Does this supersede PR #20 ? …
antoniuk1 commented on issue FLASK-LLNL/ChARGe#30.
> @antoniuk1 Does this supersede PR #20 ? …
jeremylt commented on issue CEED/libCEED#1806.
Yeah, since there are QFunctions that are not available via the gallery, you’d have to use the technique shown in this folder for ex3_volume.py (and associated files) to use a QFunction written in C….
jedbrown commented on issue openjournals/joss-reviews#7716.
Thanks @dostuffthatmatters and @abaer-kit. I’m personally sympathetic to the security/complexity issues (and would note that HPC facilities prefer to have people analyze/postprocess data from that facility to avoid egress/off-site transfer). From a usability standpoint, I see not much testing and a list of error codes, but I think it’s rarely going to provide enough information to diagnose and fix the problem from the client machine, which means someone has to manually log in and replicate an environment on the login node. As soon as you’re doing that, you’d be better off submitting jobs from the login node. At least that has been my experience (with many HPC systems and applications/software stacks over the years). But I’d like to defer to people who work with ICON….
trws commented on issue flux-framework/flux-core#7130.
That sounds right. The fluxion code that reads from these always uses the max currently though, and somewhat relies on there being an upper bound. We can certainly make the max optional, but the effect will be that the min will become the max until we have time to work through how to handle unbounded requests….
trws commented on issue flux-framework/flux-core#7130.
That sounds right. The fluxion code that reads from these always uses the max currently though, and somewhat relies on there being an upper bound. We can certainly make the max optional, but the effect will be that the min will become the max until we have time to work through how to handle unbounded requests….
chu11 commented on issue chaos/powerman#205.
recent offline conversation with @kkier and @watson6282…
markcmiller86 commented on issue betterscientificsoftware/bssw.io#1513.
@bernhold mind if I transfer this issue to my own, personal org, repo? To do this, I would have to create a temp. repo here, transfer the issue to that repo, then transfer the repo to my personal org….
jedbrown commented on issue assert-rs/libtest2#13.
This suggestion reminds me of quickcheck and proptest, which supports syntax like:…
jedbrown commented on issue assert-rs/libtest2#13.
This suggestion reminds me of quickcheck and proptest, which supports syntax like:…
tomstitt commented on issue Olympus-HPC/proteus#302.
thank you @ggeorgakoudis !!…
tomstitt commented on issue Olympus-HPC/proteus#302.
thank you @ggeorgakoudis !!…
balos1 commented on issue LLNL/sundials#784.
> It would be ideal if this error could have been caught by an example or unit test. …
jandrej commented on issue EnzymeAD/Enzyme#2309.
@wsmoses this came up in an internal discussion and we’d appreciate a slim repo as well….
jandrej commented on issue EnzymeAD/Enzyme#2309.
@wsmoses this came up in an internal discussion and we’d appreciate a slim repo as well….
tzanio commented on issue CEED/Laghos#196.
ping: @bslazarov…
tzanio commented on issue mfem/mfem#5016.
Merged in next for testing……
milroy commented on issue flux-framework/flux-sched#1412.
Thanks for the quick review @trws! Setting MWP….
milroy commented on issue flux-framework/flux-sched#1412.
Thanks for the quick review @trws! Setting MWP….
garlick commented on issue flux-framework/flux-foundry#13.
Per offline discussion, we’ll not move libtaskmap to this repo. It can stay in core….
vsoch commented on issue singularityhub/singularity-docker#17.
They are not automatically generated - they are branches in the repository….
vsoch commented on issue pydicom/deid#291.
@bghill we can also update the project version of jekyll. …
balos1 commented on issue pthom/litgen#35.
That would be great, or at least having an option to bypass it. …
trws commented on issue flux-framework/flux-sched#1406.
This review makes me wish really, really hard we had a good way to do a stacked PRs workflow. There are several tiny really nice obvious fix commits in here we could have landed almost instantly. Anyway, I’m working on a full review over the next hour or so….
trws commented on issue flux-framework/flux-sched#1406.
This review makes me wish really, really hard we had a good way to do a stacked PRs workflow. There are several tiny really nice obvious fix commits in here we could have landed almost instantly. Anyway, I’m working on a full review over the next hour or so….
cmoussa1 commented on issue flux-framework/flux-accounting#292.
This can probably be closed with the introduction of the jobs command from #674….
jameshcorbett commented on issue flux-framework/flux-accounting#768.
tzanio commented on issue GLVis/glvis#338.
Thanks @najlkin! …
tomstitt commented on issue mfem/mfem#5047.
thank you @nbeams and @tzanio ! …
tomstitt commented on issue mfem/mfem#5047.
thank you @nbeams and @tzanio ! …
gardner48 commented on issue LLNL/sundials#780.
> Have we confirmed that SUNDIALS can build without issue in C23? …
adayton1 commented on issue LLNL/spheral#417.
Should combining instantiations into one file or keeping them in separate 1D, 2D and 3D files be the default?…
artv3 commented on issue LLNL/RAJA#1925.
@johnbowen42 , not sure if this might be out of the scope of this PR. But there may also be a lot of duplicate capability here: …
eleon commented on issue spack/spack-packages#1736.
Thanks very much, @tgamblin …
eleon commented on issue spack/spack-packages#1736.
Thanks very much, @tgamblin …
grondo commented on issue flux-framework/rfc#470.
Just pushed a fixup commit that makes the exec protocol single response when the background flag is set based on discussion in flux-framework/flux-core#7107. If the new version is acceptable, I’ll squash the new commit….
garlick commented on issue flux-framework/flux-core#7111.
This was just an idle thought this morning. If we don’t want largish XML (144K here) blobs accumulating in our source tree, I can withdraw. …
markcmiller86 commented on issue visit-dav/visit#17308.
I have looked at this a number of different ways. The Mili dataset is a bit unique in that it is a mesh consisting of hex elements (3D), quad elements (2D) and line elements (1D). The stress variable is defined using 6 scalars, sx, sy, sz and sxy, syz and sxz. Its a symmetric tensor. And it is defined on both the hex (3D) and quad (2D) elements. As an aside, I never get a plot yielded no data warning. I printed and queried values using %16.16g and everything shows all zeros. When I run with -debug 5 on the command-line, I see the vtkMath.cxx:778 spew to stderr and in the .vlogs. I looked at the vtkMath.cxx source code and comments. …
tgamblin commented on issue spack/spack-packages#1736.
@eleon looks like style is falling – so you will need tor resolve the style issues….
alecbcs commented on issue spack/spack#51371.
My personal preference would be to educate users through improved documentation instead of writing a custom setup wrapper, git hooks, and our own bash rebase script. If other’s are strongly in favor of the wrappers, I’m okay if this is added….
vsoch commented on issue pydicom/deid#289.
Great news! We just need a bump to the version (maybe a larger one this time) and the corresponding entry to the changelog….
tzanio commented on issue mfem/mfem#5022.
> > Mention in CHANGELOG?
…
jandrej commented on issue mfem/mfem#5022.
> Mention in CHANGELOG?
…
vladotomov commented on issue mfem/mfem#3658.
@camierjs I’m almost done looking at this, can we rename all *coef.cpp files to *limit.cpp?…
vladotomov commented on issue mfem/mfem#3658.
@camierjs I’m almost done looking at this, can we rename all *coef.cpp files to *limit.cpp?…
trws commented on issue flux-framework/flux-sched#1296.
Awesome, great work!…
grondo commented on issue flux-framework/flux-core#7109.
This one seems pretty safe. The main use case is probably running newer or older Flux as jobs under a system instance. Since the single user jobs probably don’t have permission to send events, I doubt this is currently in use anywhere (even using the API or Python). …
balos1 commented on issue LLNL/sundials#768.
> General question on the C++ wrappers, how is error handling done? For examples, what happens if one of the C constructors returns NULL?
…
JustinPrivitera commented on issue visit-dav/visit#20114.
For my own future reference, data is here: /usr/workspace/justin/issue_visit_20114…
hppritcha commented on issue open-mpi/ompi#13280.
thanks. hopefully we can get most (all?) of the failures cleaned up within the next week or so. …
v-dobrev commented on issue mfem/mfem#4684.
@adrienbernede, I’ll merge this in #4431 and start updating that to replace ruby with dane, lassen with matrix, merge master etc….
milroy commented on issue flux-framework/flux-sched#1296.
Noting that @zekemorton discussed this PR with @trws in the group meeting yesterday, and Tom said he was good with the changes provided that the PR doesn’t alter the default behavior (which it doesn’t)….
chu11 commented on issue flux-framework/flux-core#7082.
> Could always use isatty() to change behaviors when run interactively. …
cmoussa1 commented on issue flux-framework/flux-accounting#768.
OK, well this turned into a larger cleanup than I was expecting
cmelone commented on issue LLNL/repligit#82.
@dependabot recreate…
adrienbernede commented on issue spack/spack-packages#1402.
@alecbcs Gitlab CI now passing. Style fixed….
alecbcs commented on issue spack/spack-packages#1747.
Yes to test installing v1.2 you can type:
…
adrienbernede commented on issue spack/spack-packages#1402.
@alecbcs Gitlab CI was passing, waiting for new run. Style fixed….
tgamblin commented on issue spack/spack#51047.
I will just echo what @haampie said – env vars in the environment are meant to affect the run environment, not the build environment. …
mplegendre commented on issue LLNL/Spindle#102.
I think this is looking great. Anything else before I merge?…
mplegendre commented on issue LLNL/Spindle#102.
I think this is looking great. Anything else before I merge?…
adrienbernede commented on issue LLNL/CARE#349.
@adayton1 Starting with #348 (which is ready), you can deactivate the lassen CI jobs by setting the variable ON_LASSEN to OFF in the GitLab UI / CARE project / Settings / CI/CD / variables….
adrienbernede commented on issue LLNL/CARE#349.
@adayton1 Starting with #348 (which is ready), you can deactivate the lassen CI jobs by setting the variable ON_LASSEN to OFF in the GitLab UI / CARE project / Settings / CI/CD / variables.
…
tzanio commented on issue mfem/web#291.
@helloworld922 and @cjvogl, what do you think about this PR?…
artv3 commented on issue LLNL/RAJA#1922.
@LLNL/raja-core , does anyone else want to take a look? otherwise I will release tomorrow …
jedbrown commented on issue rust-lang/cargo#4895.
This situation occurs a lot with MPI (the communication library) and more generally with C libraries in which build configuration affects the ABI (common in scientific computing libraries, unfortunately). In those cases, the libraries one wants to use are not in a default path and it would have disruptive side-effects to add them (even when using “modules”, which are popular on HPC systems)….
imguoguo commented on issue radxa-pkg/edk2-qualcomm#4.
https://applink.feishu.cn/client/message/link/open?token=AmfHwOsK4gAEaNpU1qt6AAM%3D…
tomstitt commented on issue mfem/mfem#5047.
> @tomstitt, is this ready-for-review?
…
trws commented on issue LLNL/RAJA#1919.
cyrush commented on issue Alpine-DAV/ascent#1616.
@emily-howell FYI: we should merge this PR before the vtk-m -> viskores update….
tzanio commented on issue mfem/mfem#5045.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
garlick commented on issue flux-framework/flux-core#7085.
It feels a bit like adding root to every group in /etc/group. While I guess you could argue that doing so reminds people that root can access their files, it doesn’t exactly telegraph the reality of the situation, which is that root cannot be prevented from accessing your files (at least in old school unix)…
grondo commented on issue flux-framework/flux-core#7090.
Maybe a new race condition introduced in another PR? Maybe I can try reproducing locally….
alecbcs commented on issue spack/spack-packages#1719.
Looks like harfbuzz is unable to find the freetype dependency on Windows….
vsoch commented on issue kubeflow/trainer#2841.
What Flux is going to help with:…
chu11 commented on issue flux-framework/flux-core#7085.
As an aside:…
jeremylt commented on issue CEED/libCEED#1892.
Yeah, that shouldn’t be valid. Only size >= 0…
cyrush commented on issue visit-dav/visit#20618.
@biagas can you test the windows logic?…
becker33 commented on issue spack/spack-packages#1531.
@adamjstewart this looks good to me, just need to resolve conflicts from the other deprecation removal PRs….
cyrush commented on issue LLNL/blt#732.
Since this isn’t needed for common use cases, we should remove it and projects that hit the limit can add windows specific flags for their build…
adayton1 commented on issue LLNL/RAJA#1918.
> It looks like we also set something for hip to C++17. Does that RAJA_HIPCC_STD_FLAG variable get used?
…
jandrej commented on issue mfem/mfem#5022.
If someone is interested in the complexity calculations, here you go. …
tomstitt commented on issue hypre-space/hypre#1383.
Hey @victorapm , fine by me to wait! Thanks…
tomstitt commented on issue hypre-space/hypre#1383.
Hey @victorapm , fine by me to wait! Thanks…
jameshcorbett commented on issue flux-framework/flux-coral2#413.
Sorry for the delay on this. I’m not sure what the best approach is. I’d definitely want the RPM build to fail and also some tests to be skipped if the module isn’t found, but we could probably add an optional flag to the configure line to make kubernetes a hard requirement, and then add that flag to the RPM spec. …
jeremylt commented on issue openjournals/joss-reviews#8388.
I have applied the minor edits. I will get back to you on a release timeline as soon as able…
tgamblin commented on issue hpsfoundation/tac#52.
I’m fine with this change, but I left out the COC b/c it’s a requirement for all Linux Foundation projects, which I thought @jlapolla-cray wanted (per #33). It’s something to check before voting, but if they make it into LF they’ll have a COC so it kind of doesn’t matter to me whether it’s in the vote. …
trws commented on issue flux-framework/flux-sched#1406.
I’m going to work through the code shortly, starting with the discussion points though: …
trws commented on issue desul/desul#150.
Yeah, if anything newer is better IMO….
hppritcha commented on issue open-mpi/ompi#13280.
okay thanks for the pointers @dalcinl . …
markcmiller86 commented on issue LLNL/Silo#378.
I believe it is fixed in PR #380. That is not yet merged to 4.11RC (or main) yet but will be shortly after I get more CMake testing working….
cjvogl commented on issue E3SM-Project/E3SM#7615.
> I am approving so that CI can run. I still think @bartgol or @mahf708 should have the final approval say. …
cjvogl commented on issue E3SM-Project/E3SM#7481.
> @cjvogl what is the status of this PR? …
milroy commented on issue flux-framework/flux-sched#1406.
Note that there’s a lot of code churn because I updated the power.json JGF to include a correctly enumerated rank in each vertex below the rack level. That was necessary to enable cross-rack allocation testing….
artv3 commented on issue LLNL/RAJA#1917.
Thanks for getting this started @MrBurmark. I took a follow up pass; but I think I could use a second set of eyes on the Kernel implementation. Would anyone in @LLNL/raja-core be able to take a look? …
vsoch commented on issue kubernetes-sigs/wg-ai-conformance#13.
@andreyvelich done! https://github.com/kubeflow/trainer/issues/2841. Really looking forward to working on this!
mplegendre commented on issue hpc/Spindle#99.
Can you rebase this into either: …
chu11 commented on issue flux-framework/flux-core#7070.
@mergifyio requeue…
vsoch commented on issue flux-framework/flux-coral2#411.
For our notes, here is the command that worked (for an interactive run) on hetchy. The reason we needed to ask for all 12 nodes was to get around fluxion scheduling and compute node to rabbit assignment. …
benzwick commented on issue Talk2View/SlicerBuildEnvironment#1.
@copilot see my comments…
cjvogl commented on issue E3SM-Project/E3SM#7615.
> Still has testing fails. …
cyrush commented on issue visit-dav/visit#20607.
Sounds like he is using XQuartz….
cmelone commented on issue spack/spack-packages#1360.
I believe she’s on vacation, you may need to get someone else to merge. Try posting in pull_requests on Slack?…
becker33 commented on issue spack/spack-packages#1546.
I would not deprecate a version simply because it is no longer receiving bugfixes. So much of the HPC and general open source ecosystem relies on things built once-upon-a-time against some old version of {perl or insert other package here} that still work because the interfaces are slow-moving. …
becker33 commented on issue spack/spack#51314.
I don’t think this is what we want. For a solve with a non-default compiler specified on the root, this will make it even more difficult to get the rest of the stack built with the compiler specified on the root. …
wihobbs commented on issue flux-framework/flux-sched#1383.
I do think we’ll want to look at a similar issue with easy, conservative, and hybrid backfill policies too. If I’m reading the docs right, the EBUSY that they return on an unsatisfiable job should be ENODEV. Of course, with those policies, the system never gets stopped up because backfill is on. …
wihobbs commented on issue flux-framework/flux-sched#1383.
I do think we’ll want to look at a similar issue with easy, conservative, and hybrid backfill policies too. If I’m reading the docs right, the EBUSY that they return on an unsatisfiable job should be ENODEV. Of course, with those policies, the system never gets stopped up because backfill is on. …
garlick commented on issue flux-framework/flux-core#7067.
I found it. The msg_deque class uses an eventfd(2) internally which raises POLLOUT when a pending POLLIN event is cleared. Fix coming….
grondo commented on issue flux-framework/flux-core#7069.
Yeah, it is subtle. “requires” defines a dependency relationship, while “before” and “after” are strictly for ordering or precedence of active tasks. If task A requires task B, and task A is active, then task B is forced to be active as well. However, at this point A can run in parallel with B. If A needs to be loaded before or after B, then “before” or “after” should be used to determine the precedence of tasks. …
cmoussa1 commented on issue flux-framework/flux-accounting#749.
I was able to reproduce this pretty easily in a Docker container where an association has max_nodes and max_cores limits of 1 each and submits two 1-node jobs. The second one gets held due to max-resources-user-limit, but even after upping the association’s limits and updating the plugin with flux account-priority-update, the second job remains in DEPEND state….
tzanio commented on issue GLVis/pyglvis#55.
Thanks so much for fixing this, @justinlaughlin !…
cyrush commented on issue Alpine-DAV/ascent#1614.
Note, this is with:…
v-dobrev commented on issue mfem/mfem#4800.
This needs fixes after the merge from master….
samuelkgutierrez commented on issue hpc/quo-vadis#424.
Perhaps we can provide an environment variable that somehow lets us determine the resources. This could be set on a per-site basis….
trws commented on issue flux-framework/flux-sched#1383.
Yeah, I think what’s happening is that the FCFS policy is treating the unsatisfiable job as the first job that couldn’t match. When that happens it stops trying to schedule more work, because more work can’t fit, but because that job is actually being rejected completely that assumption is wrong. The set_schedulability (true) is what we need to have happen….
grondo commented on issue flux-framework/flux-sched#1383.
Ah, ok. That is starting to make more sense. In the tests @wihobbs was doing it was clear the remainder were not getting resubmitted though……
tzanio commented on issue mfem/mfem#5002.
Merged in next for testing……
grondo commented on issue flux-framework/pr-validator#10.
Thanks! This repo doesn’t have mergify, so I’ll just press the button. …
grondo commented on issue flux-framework/flux-pam#11.
Oh oops. I forgot this now modifies .github so it will need a manual merge. Removing mwp….
wihobbs commented on issue flux-framework/flux-core#7063.
Thanks for doing this @grondo!…
adrienbernede commented on issue LLNL/uberenv#149.
> It is fine if we support both options but I personally don’t want the spack package information in multiple places. It makes the most sense in my opinion to be in the uberenv config file …
markcmiller86 commented on issue visit-dav/visit#20586.
Great diagnosis, @brugger1. I wonder where that cell type is coming from. Its either something that gets generated in VTK filter we use or there is code in VisIt that creates it directly. So, I searched our sources……
hppritcha commented on issue open-mpi/ompi#12077.
closed via #13401 …
tpatki commented on issue flux-framework/PerfFlowAspect#198.
@asantagata I think this PR now includes the fix for #156, correct? Could you update your description? I’m running the CI on it now and will test after that….
cmoussa1 commented on issue LLNL/cmvl#14.
That’s a really good question @morrone - I think I tend to agree with you. And the definition on the events in the ECS Documentation does not make it all that much clearer to me:…
adayton1 commented on issue LLNL/RAJA#1900.
Can you provide some context?…
artv3 commented on issue LLNL/RAJA#1912.
Is there anything to do for the OpenMP backend? …
jeremylt commented on issue CEED/libCEED#1461.
Yea, I can work on this some now, thanks for the update!…
hppritcha commented on issue pmodels/mpich#7571.
@dalcinl thanks for the heads up!…
vsoch commented on issue flux-framework/flux-python#18.
@trws @grondo @garlick looking for your advice! In testing building for Mac, I found that flux core and flux security need something called pipe2. …
trws commented on issue flux-framework/flux-python#17.
It is, we actually use one on flux-core now if you want to snag some configs from over there….
JustinPrivitera commented on issue visit-dav/visit#20582.
https://teams.microsoft.com/l/message/19:f2ed7be3682d40d1b8e038744e500a09@thread.skype/1756487055813?tenantId=a722dec9-ae4e-4ae3-9d75-fd66e2680a63&groupId=70162982-9587-4bcc-ad53-20178c76fe11&parentMessageId=1756487055813&teamName=VisIt&channelName=General&createdTime=1756487055813…
cyrush commented on issue visit-dav/visit#20582.
Main issue was confusion about the the scalar range of the tensor plot….
cjvogl commented on issue mfem/mfem#5001.
Noting a good idea from @kmittal2 in a Slack conversation: …
grondo commented on issue flux-framework/flux-python#17.
And I guess CI will check if the build still works :laughing: …
tpatki commented on issue flux-framework/flux-core#6873.
> Also, I wonder if we can find another way to pass in the instance URI, since embedding it in the dependency URI might cause parsing problem later on (like ambiguity about which scheme an appended ?option=value belongs to)? …
imguoguo commented on issue RadxaOS-SDK/rsdk#87.
adrienbernede commented on issue LLNL/radiuss-spack-configs#142.
@rhornung67 I guess you need the branch to live a little longer….
CamStan commented on issue LLNL/UnifyFS#821.
> Have you had a chance to figure out why the CI builds are not running? …
adayton1 commented on issue LLNL/CHAI#328.
CHAI is used successfully on Windows, though in a CPU-only configuration. It uses CMake as its build system, which is supported on Windows, so the build process should be very similar. You might have to select a specific generator (e.g. cmake -G “Visual Studio 17 2022” -A x64) to generate a visual studio solution file, for example. Or supposedly, in visual studio there is built in support for CMake projects (see https://learn.microsoft.com/en-us/cpp/build/cmake-projects-in-visual-studio?view=msvc-170). Other generators are listed here if you don’t use visual studio: https://cmake.org/cmake/help/latest/manual/cmake-generators.7.html. …
jedbrown commented on issue pyOpenSci/software-peer-review#331.
> We’re not a journal and people aren’t submitting papers but code….
hppritcha commented on issue open-mpi/ompi#13338.
hmmm not sure what’s going on with NVIDIA openshmem test….
cmoussa1 commented on issue flux-framework/flux-core#7047.
Thanks @chu11! Going to set MWP here…
garlick commented on issue flux-framework/flux-core#7046.
There was a little “override branch protections” selection that I used without going to settings. Which I needed in the powerman PR because a couple of our CI runners were using obsolete github runners and would never run. …
cmoussa1 commented on issue flux-framework/flux-accounting#740.
Thanks @wihobbs! Setting MWP here…
adayton1 commented on issue LLNL/Umpire#997.
I haven’t seen this before….
tgamblin commented on issue spack/spack#51266.
Should we try to get rid of the local imports after this PR, since they’re basically a sign of where circular imports used to be? I feel like we should use this to think about what the proper layering should be….
hppritcha commented on issue mpi-forum/mpi-issues#1004.
Fine with me. In this case I’d like to find an approach which limits API growth. For the ‘c’ interfaces could we use a more generic approach for treating the handle - like with MPI_T_PVAR_HANDLE_ALLOC? Also if there were some way to just say no to f90, then for f08 we could use polymorphism. I haven’t thought much about the copy/delete callbacks though….
jandrej commented on issue mfem/mfem#4994.
I skimmed through the changes and felt a little confused. Are you aware of the dFEM interface? See here for an example. dFEM covers a broad spectrum of functionality of automatic differentiation capabilities and I’m not sure why there is effort in recreating this functionality in different parts of MFEM. …
wihobbs commented on issue flux-framework/flux-accounting#714.
Thanks @cmoussa1!…
tpatki commented on issue Olympus-HPC/Mneme#43.
Closing to create a branch to trigger Gitlab CI….
cmelone commented on issue LLNL/hubcast#137.
unlinking this from #121 since we still need to add bot functionality for the gitlab client…
alecbcs commented on issue spack/spack-tutorial#491.
Looks good to me :)…
alecbcs commented on issue spack/spack-packages#1386.
> Can you add more defensive checks, and at least print the exception? In particular GitHub API requests may fail. …
jameshcorbett commented on issue flux-framework/flux-sched#1398.
For some reason when I tried again yesterday the checks passed in the TOSS 4 builder, so the package is out there. TOSS 5 still failed though….
wihobbs commented on issue flux-framework/flux-core#7042.
With that, setting MWP….
bgunnar5 commented on issue LLNL/merlin#549.
Bypassing the review process here as this is a combination of PRs that have already been reviewed….
trws commented on issue flux-framework/flux-core#1991.
@grondo, is this still something you want to consider? happened to pop up in a search today…
cbritopacheco commented on issue cbritopacheco/rodin#121.
@copilot I have fixed the syntax error and the test. Please finish it. Also most of these are just basic functionality tests which should be categorized as unit tests and moved to tests/unit/ under their respective path and file. So please separate the unit tests from the manufactured tests (which actually are solutions to test against and not class functionality). Also add a few more more manufactured tests….
tzanio commented on issue mfem/web#335.
deployed as of 1084806…
tzanio commented on issue mfem/mfem#4907.
Re-merged in next for testing……
markcmiller86 commented on issue visit-dav/visit#20110.
Just a thought on documentation work that could be relevant to this……
v-dobrev commented on issue mfem/mfem#4931.
Merged in next for testing……
tzanio commented on issue mfem/mfem#4906.
Re-merged in next for testing……
garlick commented on issue flux-framework/flux-core#7034.
:+1: for removal…
markcmiller86 commented on issue LLNL/Silo#453.
In 33e4935ff83aec904541d3d56281bfa7cde13721, I added tests that emulate the issue reported by Lila. …
alecbcs commented on issue spack/spack-packages#1304.
@haampie if you can follow up in another PR with those pkg name modifications I’m happy to review :)…
balos1 commented on issue spack/spack#51281.
CC @steven-roberts…
jedbrown commented on issue rust-lang/rfcs#3847.
Similar proposal: https://github.com/rust-lang/compiler-team/issues/907 …
vsoch commented on issue pydicom/deid#286.
I think this would be a fantastic development for the project. We can introduce changes (that are breaking) as long as it is clear / versioned with an appropriate bump. I can give feedback and review at each step - let me know on issues / PRs when the time comes….
jeremylt commented on issue openjournals/joss-reviews#8388.
@editorialbot generate pdf…
grondo commented on issue flux-framework/flux-sched#1399.
flux-cancel(1) is a reference to the manual page in section 1, a common practice in documentation and error messages to refer the user to the correct documentation. (i.e. man 1 flux-cancel). Perhaps in this case it would be clearer if the message was?…
wihobbs commented on issue flux-framework/flux-core#7033.
@chu11 or @grondo can you check that I did the completions right? I’ve never messed with them before, and are unfamiliar with how they work….
grondo commented on issue flux-framework/flux-core#7033.
Want to check if it is in the bash completions as well?…
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2251.
Ok, sorry for bad link in previous comment. Go here, https://github.com/markcmiller86/markdown-collab/blob/main/ExperiencesIntegratingMOABAndVisIt.md….
balos1 commented on issue LLNL/sundials#752.
Thanks for sharing your workaround @mottelet. …
cyrush commented on issue Alpine-DAV/ascent#1609.
braid case might be a subtle color difference (not that that helps too much with the underlying mystery) …
tgamblin commented on issue hpsfoundation/tac#35.
> If the “Align with the mission of the HPSF” requirement is enough to prompt others to scrutinize the dependency licenses, then I’m happy to close this pull request without merging. …
jameshcorbett commented on issue flux-framework/flux-sched#1398.
The same test fails in the TOSS4 build farm, although the failure was in test case #3 rather than #4. I didn’t see a segfault message, but it’s possible I just missed it….
jameshcorbett commented on issue flux-framework/flux-coral2#403.
@mergifyio requeue…
garlick commented on issue flux-framework/flux-coral2#397.
Thanks! MPW now….
wihobbs commented on issue flux-framework/flux-coral2#402.
It’s helpful to remember, if the node can be returned to service immediately, to raise a fatal exception on the job but exit 0 in the prolog (this is useful, for instance, if a user requests more storage than we cap them at):…
chu11 commented on issue chu11/freeipmi-mirror#76.
ipmitool’s output can be different as it intereprets data differently. some ideas…
gardner48 commented on issue LLNL/sundials#728.
> @gardner48 Looks like the extended precision test for cvsRoberts_FSA_dns_constraints_-sensi_stg1_t.c is failing because a line number is in the output file (due to an error message, see sundials-codes/answers#60) , and the line number changed. We should probably just disable that test with extended precision since it fails.
…
adayton1 commented on issue LLNL/camp#184.
> D’oh! Yes, good catch. Thank you. …
artv3 commented on issue LLNL/RAJA#1866.
@LLNL/raja-core , what do you guys think of updating the CHANGELOG within each PR? I think it may help us keep track of new features as they come in – instead of when we are making the release candidate. …
jeremylt commented on issue CEED/libCEED#1891.
oh, our vector access protected us accidentally here…
becker33 commented on issue spack/spack#51140.
It’s not done as a post-concretization step, the specs from the dev specs are imposed in the solver itself. Then there’s also a post-concretization step that ensures the dev_path variant is present, but it doesn’t worry about the rest of the spec….
hppritcha commented on issue open-mpi/ompi#13280.
Thanks for checking this stuff out in its early state @dalcinl. some of the above functions have been implemented but are sitting in various states in PRs. some of these should be defined - like MPI_Wtick and the aint related functions so i’ll see what’s going on. As for the fromint/toint that seems like an excellent project for a AI LLM. I’ll see what can be done there.
…
grondo commented on issue flux-framework/flux-coral2#383.
No objection here. …
jameshcorbett commented on issue flux-framework/flux-accounting#735.
go off king …
cmoussa1 commented on issue flux-framework/flux-accounting#733.
I was able to reproduce this in my docker container following your set of instructions above. After the first job, subsequently submitted jobs get rejected with a Queue not valid for user error message. For flux-accounting, this seems to be a particular issue when default queues are configured, e.g….
dylan-copeland commented on issue LLNL/libROM#316.
@dreamer2368 An example using this PR is in file src/rom_handler.cpp of https://github.com/LLNL/scaleupROM/pull/60
…
cyrush commented on issue spack/spack-packages#1218.
@adamjstewart It looks like it it is trying to build with a newer umpire than ascent 0.8.0 supports. …
cmoussa1 commented on issue flux-framework/flux-coral2#394.
Yes I can. Here goes nothing!…
cyrush commented on issue LLNL/conduit#1459.
@abhishek1297 glad you were able to work it out, we can add that method to the C-API. Thanks for the feedback!…
emily-howell commented on issue Alpine-DAV/ascent#431.
@cyrush Also, you mentioned wanting to compile with 64bit IDs and then use caliper to observe any performance changes. Is there a specific analysis you were hoping to see? I was just doing some runs of different areas of the test suite with caliper reports to see any notable changes….
becker33 commented on issue spack/spack-packages#1254.
@hfp this might be of interest to you….
dylan-copeland commented on issue mfem/mfem#3807.
> @dylan-copeland did you make the changes you planned here (after mfem/data#12 was merged)? …
cbritopacheco commented on issue cbritopacheco/rodin#119.
Merging even though not finished….
tzanio commented on issue mfem/mfem#4980.
Merged in next for testing……
vsoch commented on issue google/dranet#151.
Thank you @keithmattix much appreciated. Enjoy the holiday!…
v-dobrev commented on issue mfem/mfem#4801.
@tepperly, the PR will be merged by the editor (@pazner in this case, as seen in the table added to the first post) when there are at least 2 approvals from people in the Reviewers team and after some additional testing (run nightly) in the next branch. The procedure is documented in CONTRIBUTING.md: https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#pull-requests – we try to follow it closely but it’s not set in stone.
…
garlick commented on issue flux-framework/flux-core#6958.
I added testing and removed the WIP. I am not hard over on merging this as is, but wanted the option to be available….
chu11 commented on issue flux-framework/flux-core#6999.
> I wonder though if we could just create bad treeobjs directly instead of laboriously modifying good ones? Could even have some prepared JSON in test data files. Just a thought. This is fine for now. …
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2251.
> Agreed! Are you back from your break, @markcmiller86 ? …
tgamblin commented on issue spack/spack#51246.
This is very nice for #51137, where we start showing conditional deps in spack info.
…
trws commented on issue flux-framework/flux-sched#1339.
This one is mostly good too but currently has no testing….
grondo commented on issue flux-framework/flux-sched#1394.
Yes, sorry. I dropped the ball on this one. Rebased and will hit the button….
grondo commented on issue flux-framework/flux-python#16.
Oh, I see you answered in the email thread. Thanks!…
grondo commented on issue flux-framework/flux-core#6990.
This does seem more user-friendly!…
cmoussa1 commented on issue flux-framework/flux-accounting#731.
Thanks @jameshcorbett! Setting MWP here…
cmelone commented on issue cmelone/hubcast-test#3.
@lc-hubcast restart failed jobs…
becker33 commented on issue spack/spack#51190.
Looks like the tests need to be updated for the changed method name, but otherwise LGTM…
jeremylt commented on issue openjournals/joss-reviews#8388.
Ratel just will error if misconfigured. That’s to hard baked into the PETSc error and options handling….
jedbrown commented on issue openjournals/joss-reviews#8388.
Yeah, the options are extensible, akin to a programming language, and -help is used to show everything that can be specified in the current expansion. You can’t get to the entire expansion without running. It would be possible (at substantial cross-cutting effort and maintainability impact) to make a system that shows the useful options in a simulation, but even that would be unable to deal with solution-dependent logic (e.g., solve an auxiliary problem, enter an optimization loop, and/or write output only if the solution satisfies a numeric criteria). The decision to make -help comprehensive and extensible was made (by the PETSc team) long ago and responsive to community needs. We could bikeshed over a different name, but this is widely used and accepted in the PETSc community and we consider it a feature that Ratel be readily usable by people familiar with configuring PETSc solvers via run-time options (which you can liken to a domain-specific programming language)….
hppritcha commented on issue open-mpi/ompi#13280.
jenkins ci runs make distcheck…
cbritopacheco commented on issue cbritopacheco/rodin#118.
@copilot I have fixed your tests since they contained some mistakes and inaccuracies and renamed setInterface to setFront. Please write more tests a few unit tests and more manufactured tests. The manufactured tests should test with more than just point sources. You can specify that the initial front will be the boundary of the mesh. From another side, an interior interface is also useful to test (for example, a rectangular interface of edges inside a rectangular triangular mesh). Please test analytical solution vs the numerical….
adayton1 commented on issue LLNL/Umpire#986.
@rhornung67, @kab163, @davidbeckingsale, any other comments? Since I pushed a change, it dismissed Rich’s and Kristi’s approvals….
emily-howell commented on issue Alpine-DAV/ascent#431.
https://github.com/Alpine-DAV/ascent/blob/258e06ed902d251c1f8eb4d987043d049b77cade/scripts/build_ascent/build_ascent.sh#L697…
becker33 commented on issue spack/spack-packages#1123.
@hfp I changed the spec slightly to make it more legible for Spack 1.0, but I had success with: …
milroy commented on issue flux-framework/rfc#466.
These changes look good to me with respect to the interim need, and I’m fine with this PR as is. I agree with @trws that fluxion can be adapted to support nslots without too much difficulty.
…
chu11 commented on issue flux-framework/flux-sched#1394.
@grondo i assume has to be merged manually, wanna hit the button?…
chu11 commented on issue flux-framework/flux-accounting#727.
@cmoussa1 i think has to merged manually, wanna hit the button?…
cyrush commented on issue LLNL/conduit#1459.
We can add more c methods if needed. …
adayton1 commented on issue LLNL/camp#152.
@MrBurmark any other comments?…
adayton1 commented on issue LLNL/RAJA#1888.
I just noticed this doesn’t prevent trying to use the internal cub or rocprim submodules. So at the least the messages like “Using RAJA cub submodule” need to be converted to a fatal error and the message should be updated….
cyrush commented on issue visit-dav/visit#18884.
Resolved in #20494 …
cjvogl commented on issue mfem/mfem#4905.
@homeomorfismo , I’ve made a first pass through the code. Overall, I think this is in really good shape. Let’s focus on the corresponding write-up first, and only return to the code here if that write-up is completed before the end of the week….
tgamblin commented on issue hpsfoundation/tac#35.
> Today, we don’t formally vote on this. …
trws commented on issue flux-framework/rfc#466.
That’s a fair point. As I said I don’t see any problem with it, it’s easy enough to supply. @milroy do you have any thoughts here?…
wihobbs commented on issue flux-framework/flux-core#6993.
Thanks @jameshcorbett! Setting MWP. …
jameshcorbett commented on issue flux-framework/flux-accounting#726.
Yeah, it wouldn’t save you any indentation. It might be a bit easier to read though?…
chu11 commented on issue crate-ci/typos#1345.
```…
balos1 commented on issue LLNL/sundials#758.
I think this can be merged as-is, but it might be helpful to add a super quick demo in the developer docs in a “tools” section. I think just an example plot + the command that was used….
cmelone commented on issue LLNL/hubcast#167.
ok @alecbcs that worked, though for some reason I needed to pull out the permissions into ci.yml from security.yml……
alecbcs commented on issue LLNL/hubcast#167.
Can we merge the CodeQL CI YAML file into the existing security CI YAML so that we run both out of a single file and they’re triggered by the same conditions?…
imguoguo commented on issue radxa-pkg/linux-qcom#4.
mej commented on issue mej/nhc#162.
@adriansev - I don’t see anywhere that mentions which config file you have those settings in, but that can make a difference. You might try adding them to the command line just as a test. You can also enable tracing. Here’s an example of doing both at once:…
trws commented on issue flux-framework/flux-core#6986.
I mentioned it mainly because it showed up in two actions sets in a row on two different PRs for me today. Given the timeouts, it may just be that things are running extra slow, but something must have tickled it….
cbritopacheco commented on issue cbritopacheco/rodin#117.
I need an example of utilization….
cmelone commented on issue LLNL/hubcast#144.
@alecbcs my latest changes should address your comments. There’s no logging inside the library modules. …
tgamblin commented on issue spack/spack#51210.
Looks good to me… I think we could add more stuff to this page (environments, developer workflows, CI, OCI container builds, etc.) but this is a good clarification. …
jedbrown commented on issue pyOpenSci/software-peer-review#331.
Suppose a human types // sparse matrix transpose, presses tab, and gets a page of code that still has namespaces from an existing library with copyright/attribution stripped. What does it mean that “accountability remains with humans”? What if the algorithmic system obfuscates just enough that it isn’t obvious to the human that it’s plagiarizing an existing library (violating that project’s license)? Just because many people use a product doesn’t mean it isn’t manufacturing plausible deniability. If the claim is merely “I don’t think copyright holders will successfully sue me for infringing”, that is a statement about power rather than justice or consent….
garlick commented on issue flux-framework/flux-core#6589.
I think A is the sane thing to do, unlink the bad dirref. Isn’t it the same problem as removing a valref?…
chu11 commented on issue flux-framework/flux-core#6953.
re-pushed, --lost-and-found is now --repair, tweaked some documentation per comment above
…
grondo commented on issue flux-framework/flux-core#6983.
Thanks mentioning @kkier. The key thing to note is that the only change in behavior while /etc/flux/system/{prolog,epilog,housekeeping} exist is that any scripts dropped into libexecdir/flux/{name}.d (meant to be used by other packages) will be run before the system scripts.
…
trws commented on issue flux-framework/flux-core#6929.
I imagine you already found it, but here’s the link for git-absorb. I heard about it a long time ago but for whatever reason never actually tried it until today, wishing I had done it a long time ago….
imguoguo commented on issue RadxaOS-SDK/rsdk#81.
>
balos1 commented on issue LLNL/sundials#759.
> …
adayton1 commented on issue LLNL/blt#725.
I don’t have permission to merge, and I will be unavailable Monday and Tuesday. So feel free to merge whenever you are ready, @white238….
cjvogl commented on issue E3SM-Project/E3SM#7615.
> @cjvogl how urgent is this PR? After the last sprint with diags capabilities, I was thinking to pause new diags, and implement a more flexible yaml specification paradigm. Eg, I was thinking of a syntax along the line of …
becker33 commented on issue spack/spack-packages#1128.
@spackbot rerun pipeline…
becker33 commented on issue spack/spack#51069.
@XXVIMilia do you think this could be the same problem as #51180?…
jedbrown commented on issue rust-lang/rust#138689.
I believe the compute_* and sm_* semantics are consistently represented in this PR, and that single-invocation multi-target behavior is a major feature that would not entirely supplant ability to build kernels and device code using --target=nvptx64-nvidia-cuda. I think a similar amdgcn_target_feature would make sense….
v-dobrev commented on issue mfem/mfem#4948.
> First, I need to parallelize the operation of assembling N*N sparse matrices into a block matrix. Currently, I’m using HypreParMatrix and HypreParMatrixFromBlocks. I’d like to know if there are any problems with my approach….
becker33 commented on issue hpsfoundation/tac#34.
I think this is all subsumed under the requirement to join the LF – I don’t think we want to double-approve that…
tgamblin commented on issue hpsfoundation/tac#45.
@becker33 @jlapolla-cray @kmorel @ax3l @dalg24: after reviewing the (transcript of the 7/24 TAC meeting I tried to write what we discussed. …
wihobbs commented on issue flux-framework/flux-core#6972.
For reference, the same commands in a bookworm container work fine:…
cmoussa1 commented on issue flux-framework/flux-accounting#656.
Well, after a few months of searching, I think I found the culprit….
hppritcha commented on issue spack/spack-packages#1128.
we don’t support singularity in Open MPI so i’m fine with this change….
hppritcha commented on issue open-mpi/ompi#12085.
@naughtont3 any updates on adding this functionlity?…
vladotomov commented on issue mfem/mfem#4941.
Is the behavior of hypre_ParVectorToVectorAll invalid when the local size is zero, or is the MFEM usage incorrect?
…
jeremylt commented on issue mfem/mfem#4609.
sure, reverted…
cjvogl commented on issue mfem/mfem#4907.
> @cjvogl , @dohyun-cse , any chance looking at this? …
tzanio commented on issue mfem/mfem#4974.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
grondo commented on issue flux-framework/rfc#465.
One idea for now – now that we have CLI plugins – is to implement this as a CLI plugin that users requiring the more advanced specification can install and use. I think @wihobbs was talking about working on something similar that acts as a filter, but embedded in the cmdline via a plugin with a unique option namespace is an enticing way to offer the functionality. …
trws commented on issue flux-framework/flux-sched#986.
Actually, I think it comes down to this: We have the index into the list of RSMI devices that we want to pull, it looks like we have to iterate over them to find that (can probably also use a filter to get there, or use cousin walking or something smarter that I can’t think of right now). Once we have it, we can use the suffix of the name (which is utterly insane but it seems there isn’t a field for it for some crazy reason) or we can use the AMDUUID=a30d0d26ba377c92 field from the osdev attributes and put that directly into the ROCM_VISIBLE_DEVICES. That value is a lot less pretty, but it’s stable so no matter how many times that variable gets applied into child processes and restricted environments it stays correct. Where ROCM_VISIBLE_DEVICES=3 becomes an error if it’s applied twice….
grondo commented on issue flux-framework/flux-sched#1391.
Hm, this was tuolumne. I was also wondering if it is something in my environment. Sorry I haven’t had a chance to run this down yet. …
trws commented on issue desul/desul#144.
Also stacked on #146 …
cyrush commented on issue Alpine-DAV/ascent#1273.
related to : https://github.com/Alpine-DAV/ascent/issues/356…
gardner48 commented on issue xiaoyeli/superlu_mt#25.
Thanks Sherry, looks like the CI is happy again….
tgamblin commented on issue spack/spack#50933.
The script (slightly modified) succeeded on my MacBook just now on 95a332eb93, when building from scratch. When I try it again with the build cache, it fails very quickly. Should be able to debug this fairly quickly….
v-dobrev commented on issue mfem/mfem#4937.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
tzanio commented on issue mfem/mfem#4987.
Merged in next for testing……
garlick commented on issue flux-framework/flux-sched#1390.
Thanks! I’ll go ahead and set MWP…
grondo commented on issue flux-framework/flux-sched#986.
> Really what would likely be most useful in the short term would be to ensure that those logical IDs are translated back to physical IDs in exec or the shell…
JustinPrivitera commented on issue visit-dav/visit#20189.
sounds like a me problem…
imguoguo commented on issue radxa-pkg/linux-qcom#4.
jeremylt commented on issue mfem/mfem#4932.
I made some updates in the linked branch (https://github.com/mfem/mfem/pull/4609) that might help….
jandrej commented on issue mfem/mfem#4984.
Have a look at https://github.com/mfem/mfem/blob/master/miniapps/dfem/dfem-minimal-surface.cpp#L175….
trws commented on issue flux-framework/flux-sched#1015.
I actually did some digging on this, so some more context. We only show an ETA for those jobs that enter the reserved state. Since we normally run easy backfill, that means only the highest priority job that can’t run immediately gets an estimated time. If we run with conservative, then we should get estimates up to reservation depth (which is usually shorter than full backfill depth). …
grondo commented on issue flux-framework/flux-core#6970.
(in case someone works on this) …
imguoguo commented on issue RadxaOS-SDK/rsdk#80.
All but two models passed the test on GitHub Action. The two models failed due to Incident with Actions but passed the manual test….
adayton1 commented on issue LLNL/camp#180.
I’m not sure what’s going on with the docker jobs, but I don’t think it’s related to my changes….
adayton1 commented on issue LLNL/RAJA#1875.
Ruby and corona being down are the reasons for the failed CI jobs….
artv3 commented on issue LLNL/RAJA#1875.
> > I’m actually wondering if we should just use caliper if the user wants annotations rather than trying to fall back on NVTX directly. …
cyrush commented on issue Alpine-DAV/ascent#1603.
modest replay output example w/o detailed annotations: …
becker33 commented on issue spack/spack#51140.
@psakievich I think it would be possible to refactor this as a generic Environment mutator that calls a generic Spec mutator, with a callback for the particular change to execute on the spec. I decided that as long as this is the only feature that works this way it’s not worth doing, but let’s talk about extending it to commit variants once this is in….
alecbcs commented on issue spack/spack#51144.
> @alecbcs I tried yours, but it has too many issues: …
jedbrown commented on issue rust-lang/stdarch#1805.
Yeah, I can rebase. I’m having trouble finding docs for a development workflow for stdarch. The rustc josh docs describe making PRs to rust-lang/rust. I would like to build stdarch and use it with either +nightly or +stage1 to test external crates that actually use the GPU (which CI does not yet do)….
jedbrown commented on issue mfem/mfem#4609.
Pushed. It builds cleanly for me with both 0.12 and main…
chu11 commented on issue flux-framework/flux-core#6963.
> a private KVS namespace would have been created. Do we simply destroy it? Are there any races if the job immediatley restarts?…
jedbrown commented on issue flux-framework/flux-core#3556.
Would you consider Rust for the submission tool? That would give good library support for CLI handling and JSON construction, and you get a statically-linked executable that’s every bit as fast as C….
trws commented on issue flux-framework/flux-core#4534.
Yeah, we do. Asking hwloc would be ok, and actually if we have a topology or a way to refer to the gpu device there that would make the ID translation issue trivial to fix too because it’s the same object we’d want to query. The ultimate solution would be to pull the unique ID for each GPU from it, and then use that in whichever env var is for the GPU (to handle changing GPU orders and multiple runtimes on a machine) but for systems that actually exist outside my office, all we really need is:…
cmoussa1 commented on issue flux-framework/flux-accounting#720.
Thanks! Setting MWP here…
cjvogl commented on issue E3SM-Project/E3SM#7481.
> Needs a rebase and test fails should be looked at. …
adrienbernede commented on issue spack/spack-packages#869.
@v-dobrev for the records….
hppritcha commented on issue open-mpi/ompi#13279.
did we add the tests for this to ompi-tests? i’m triaging compile failures for the tests in my mtt runs: …
wihobbs commented on issue flux-framework/flux-sched#1285.
Since the approvals were dismissed due to a rebase I’ll approve and add MWP….
jameshcorbett commented on issue DataWorkflowServices/dws#254.
I think related to this change I am now seeing the following error when I try to run nnf-deploy in my kind env:
…
wihobbs commented on issue flux-framework/flux-core#6952.
I’m very uneducated on this so apologies for the stupid question but @trws or @grondo, can you explain why its important that:…
tpatki commented on issue LLNL/msr-safe#185.
Correction of a typo in bullet 4 below, the cp command should be:…
jameshcorbett commented on issue flux-framework/flux-coral2#385.
Thanks @cmoussa1, setting MWP!…
alecbcs commented on issue spack/spack-packages#1028.
Unfortunately the CI failures are related. I’m still figuring out a fix. …
jeremylt commented on issue freeCodeCamp/freeCodeCamp#61453.
I recommend reading the contributing guide. Issues are not assigned and the first valid PR is accepted….
grondo commented on issue flux-framework/pr-validator#9.
Tested and seems to be working, thanks!…
grondo commented on issue flux-framework/flux-core#6959.
Ok, that makes sense. It only came up because modprobe wants a list of all modules, but has to treat connector-local differently since it is auto-loaded and should not be unloaded, etc. Easy enough to work around though….
grondo commented on issue flux-framework/rfc#464.
Yes, please merge….
JustinPrivitera commented on issue visit-dav/visit#20520.
Thank you! Let’s leave this ticket open then until we get the dmg file….
tzanio commented on issue mfem/mfem#4970.
ping: @jandrej, @sohailreddy, @alexanderianblair …
wihobbs commented on issue flux-framework/rfc#414.
This is long out of date, and a lot of the proposed components are unnecessary now due to improvements in the python-based job-manager JournalConsumer class and the ability to extend the command line via plugins. Maybe we’ll still need an RFC for notifications, but it will look very different from this one. Closing….
chu11 commented on issue flux-framework/flux-core#6953.
> Seems like repair should be the default …
trws commented on issue flux-framework/flux-core#6954.
Yup, sounds right. Feel free to hit it, or I can if you prefer….
trws commented on issue flux-framework/flux-accounting#714.
Yes, definitely. It’s useful to have the requested time because it has a significant impact on queue policy outcomes, especially with backfill policies, and can help us determine what queue policies and settings to use in the future….
cmoussa1 commented on issue flux-framework/flux-accounting#714.
OK, wrote up a small test and it looks like flux-accounting is fetching the updated duration, so this is something that flux-accounting could store in a separate column in its job-archive if that was to be desired?…
benzwick commented on issue Slicer/Slicer#6388.
Is there any documentation that explains how to compile Slicer with Qt6 so we can test this on Linux (and later on Mac and Windows)? And which branch should be used to test the latest progress on this issue?…
jeremylt commented on issue CEED/libCEED#842.
Given that XSMM targets this hardware and already gives us better performance on AVX hardware, I propose we close this. Any objections?…
alecbcs commented on issue spack/spack-tutorial#477.
Merging without waiting for a review since this exact file is already in spack/spack….
vladotomov commented on issue mfem/mfem#4289.
I might be missing something, but I’m not seeing the value of this example. Remhos already handles advection, offers more sophisticated limiting options, computes local bounds correctly based on the stencil, and supports general meshes. I realize this showcases the new bounds calculation, but there are already dedicated examples for that….
garlick commented on issue flux-framework/flux-sched#1380.
With FLUX_RC_USE_MODPROBE=t set for tests, flux-core 0.77 is required, but looking at one of the bookworm test failures, it appears we’re picking up something less than: …
jameshcorbett commented on issue flux-framework/flux-coral2#383.
I’d like to read through the core and sched modprobe PRs first so I can understand what’s going on. I’m doing that now. So far looks totally fine though….
garlick commented on issue flux-framework/flux-coral2#383.
NP if we want to hold off. I guess I am enthusiastic about moving fowrard. :-)…
adayton1 commented on issue LLNL/CHAI#298.
Could you add a new CMake configuration variable? Something like CHAI_ENABLE_EXPERIMENTAL? Then in CMakeLists.txt only add the new files if CHAI_ENABLE_EXPERIMENTAL is on. …
v-dobrev commented on issue mfem/mfem#4923.
@lindsayad, if it is not too much trouble, can you try this branch to see if it solves the issues you encountered in #4959?…
wihobbs commented on issue flux-framework/flux-sched#1285.
> How did you test this? I added the change in this commit to t1020 in current master and the test still passes. …
milroy commented on issue flux-framework/flux-sched#1285.
> Are we still OK with leaving in the uncovered code in resource_match.cpp:select_subsystems? …
bgunnar5 commented on issue LLNL/merlin#524.
TODO: add documentation…
tgamblin commented on issue spack/spack#51061.
@haampie We discussed a number of solutions for this, and I don’t see an easy one. The proposal is not to ignore the provenance – it’s to keep it but build one version as another. As you say, when you tell Spack to treat sources as a particular version, you are doing the same thing – assigning a version. In fact, you could already do what is in this PR if you had the git repo for every version. We’re simply extending it to work for releases/tarballs instead of just git versions….
bwpriest commented on issue LLNL/ygm#368.
Ok, good to know. Should we cancel this PR?…
tgamblin commented on issue spack/spack-packages#924.
fixed comments…
tgamblin commented on issue spack/spack-packages#739.
Finally
tzanio commented on issue mfem/mfem#4955.
Merged in next for testing……
cjvogl commented on issue E3SM-Project/E3SM#7481.
@tcclevenger , can you please add the CI: Approved flag to run the tests?…
JustinPrivitera commented on issue Alpine-DAV/ascent#1587.
…
JustinPrivitera commented on issue visit-dav/visit#19488.
I think this is a good idea and a good feature to add but it would be nice to also fix the underlying issue causing things to be so slow….
hppritcha commented on issue open-mpi/ompi#13351.
@edgargabriel could you check what’s going on with rocm ci?…
v-dobrev commented on issue mfem/mfem#4944.
Unfortunately, I don’t think there’s a way to automate the synchronization of the validity flags, at least not with the current interface….
mplegendre commented on issue hpc/Spindle#4.
Closing old PRs…
trws commented on issue flux-framework/flux-sched#1385.
@mhyatt000, what version of spack are you using? The dependency is listed, and has the correct version. Is it not pulled in as one of the dependencies of your sched build? Here’s the relevant part of the package….
milroy commented on issue flux-framework/flux-sched#1382.
Thanks for the review @trws! Adding MWP….
chu11 commented on issue flux-framework/flux-core#6589.
Wanted to start a design discussion on what we would like flux-fsck to look like. First, summarizing several PRs I submitted….
jameshcorbett commented on issue flux-framework/flux-coral2#379.
I think I understand now. As can be seen in the eventlog, Flux was restarted while the job was in the queue, so the plugin was reloaded. The exception callback in the jobtap plugin is only triggered if the plugin has subscribed to events for the job, but the call to flux_jobtap_job_subscribe() is only done in the callback for the SUBMIT state. (See here.) So the exception callback was never invoked and the prolog was never removed….
vladotomov commented on issue CEED/Remhos#53.
@pearce8 it’s merged, thank you, sorry for the delay….
cmelone commented on issue spack/spack-tutorial#467.
@scheibelp …
becker33 commented on issue spack/spack-packages#885.
I think we should we backport this to v1.0.1. It may come up in release CI for future 1.0.x releases…
garlick commented on issue flux-framework/flux-core#6260.
Oh, I thought that one was already in! Sorry, will look at that now….
tgamblin commented on issue curl/curl#18104.
Thanks for this!…
tpatki commented on issue LLNL/variorum#587.
Thanks for testing this on Tuo, @kulnaman!…
jeremylt commented on issue CEED/libCEED#1881.
Ok, the blocking issue here seems to be around permissions to create files/folders in the runner…
jedbrown commented on issue rust-lang/rustup#3397.
It would be very useful if we could rustup which llvm-link or perhaps rustup which --component llvm-tools llvm-link. If you agree this would be useful, do you have a preference on the two syntaxes, or pointers to where you think it should be implemented?…
tomstitt commented on issue mfem/mfem#4028.
> Thanks for checking @tomstitt! Can you give it another try with the latest commit? …
chu11 commented on issue flux-framework/rfc#459.
i will simply merge this manually until we can figure out what happened (or perhaps mergify updated some check and we need to update our mergify config)…
wihobbs commented on issue flux-framework/Tutorials#55.
Smart idea, my only worry is that we have sent and posted links that reference the existing directory structure. Is there a way to invisibly symlink everything so that the current links continue to work? (probably not :/)…
cyrush commented on issue LLNL/conduit#1453.
@Arlie-Capps since there were changes to the amr bent grid example as part of the PR – this has diverged. …
cyrush commented on issue visit-dav/visit#20509.
This is currently running tests using test data from an http mirror, w/o baselines (which of course will fail). …
alecbcs commented on issue spack/spack-packages#571.
@adamjstewart this is a bit our of my wheelhouse. Can you speak to this situation a bit more? …
wihobbs commented on issue flux-framework/flux-core#5531.
I wanted to update this issue with the outstanding to-dos as I see them….
gardner48 commented on issue LLNL/sundials#753.
Hi @mottelet, between versions 6.6 and 7.4 there have not been any changes to the projection capability in CVODE. The error handling across SUNDIALS was updated in v7 however, this should not have changed any of the return codes from CVODE. There were some fixes related to the stop time in versions v6.6.1 and v6.7.0 that could be related to what you are seeing. Before these fixes, it was possible that CVODE would reach a stop time but return CV_SUCCESS instead of CV_TSTOP_RETURN when the output time and the stop time are the same. …
adayton1 commented on issue LLNL/RAJA#1754.
I’m encountering this issue when I configure RAJA with ENABLE_CUDA=ON and ENABLE_OPENMP=ON and a downstream library (LEOS) with ENABLE_CUDA=ON and ENABLE_OPENMP=OFF (since LEOS only supports one backend at a time)….
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thanks @warrickball we have merged your changes. …
jedbrown commented on issue openjournals/joss-reviews#7716.
Hi, @jfe-dkrz.
trws commented on issue flux-framework/flux-core#6925.
Thanks for the tmate work, going to try that now that I have the source cleaned up. Also going to see if I can get the annotate failures script (or part of it) to go so we get the debug info more reliably….
grondo commented on issue flux-framework/flux-core#6925.
> It appears in my workflow output but doesn’t look like it executes. (gray checkbox, 0s elapsed, workflow continues on) …
samuelkgutierrez commented on issue hpc/quo-vadis#354.
> Question: I issued a git rebase --whitespace=fix HEAD before pushing into my repo and the whitespace test still failed. …
JustinPrivitera commented on issue visit-dav/visit#20504.
nope, nothing. I’ll pick this up again on monday….
tzanio commented on issue mfem/mfem#4957.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
trws commented on issue flux-framework/flux-sched#1384.
They are not meant to be, but some of them operate on timeouts and can have failures like that especially if the resource is overloaded. The first of those tests is on a ten second timeout, so I wouldn’t expect a spurious failure there unless either launching the subinstance is very delayed (say a very, very slow filesystem) or running too many tests in parallel. …
cjekel commented on issue cjekel/piecewise_linear_fit_py#131.
Thank you for this!…
v-dobrev commented on issue xsdk-project/xsdk-issues#277.
@balay, By the way, now that the Spack packages are in a separate repo, the xSDK pipelines may need to be updated to pull the latest spack/spack-packages repository, e.g. by calling spack repo update along with some additional info like spack repo list and some command(s) to show what commit of spack/spack-packages is being used….
tgamblin commented on issue spack/spack-packages#739.
Ok bailing on this as a back port – tried something less disruptive in #792, which should go into 2025.07.1 instead of this. …
cyrush commented on issue spack/spack-packages#744.
this looks great thank you @vicentebolea !…
grondo commented on issue flux-framework/pr-validator#7.
I just noticed that crate-ci/typos publishes pre-built binaries in their releases. …
grondo commented on issue flux-framework/flux-core#6908.
> I don’t know if it is relevant but I did find a single remaining case of using a hardcoded path to lua:…
wihobbs commented on issue flux-framework/flux-core#6906.
oops y’all saw nothing
trws commented on issue flux-framework/flux-core#6926.
Oh and it can definitely wait, there’s a heisenbug in fileref of all things on GHA right now that’s going to take a bit to work out. Thanks for looking it over!…
garlick commented on issue flux-framework/flux-core#6925.
Apologies, I made a comment on this stuff in the other PR that is based on top of it. Just that I’m traveling today and wanted to have a closer look at this before it goes in. Great work @trws….
cyrush commented on issue conda-forge/conduit-feedstock#52.
@ h-vetinari thank you for trying to solve this mystery. …
artv3 commented on issue LLNL/raja-suite-tutorial#67.
I think is good to now, @tdrwenski are you able to hit merge? or should I merge?…
jeremylt commented on issue Eudicods/outworlds-wastes#134.
In #135…
cyrush commented on issue AMReX-Codes/amrex#4572.
@ax3l our current ascent containers (0.9.4) are using CMake 3.28. The internal layout changed a bit, but I can address that. …
hppritcha commented on issue open-mpi/ompi#13343.
this looks good to me…
wihobbs commented on issue flux-framework/flux-sched#1375.
Hi @BagchiS6, sorry we missed this issue, but I looked back and it seems like we made some progress on Slack a few days after you opened this….
grondo commented on issue flux-framework/flux-sched#1383.
Or the scheduling loop isn’t being triggered to consider the next pending job….
milroy commented on issue flux-framework/flux-sched#963.
We discussed this issue in the Fluxion group meeting today. I mentioned one more disadvantage of modeling a user (as opposed to an admin DAT) advanced reservation as a reservation rather than an allocation: re-reserving the reservation during each scheduling loop can cause the start time to change, necessitating feedback to the user submitting the request….
jedbrown commented on issue CEED/libCEED#1875.
This seems like only a naming convention thing, and like it would be possible to preserve deprecated stubs for the old names. That would ease migration at what looks to be minimal cost….
gardner48 commented on issue spack/spack-packages#496.
> I would also like to hear from at least one maintainer per package. …
vsoch commented on issue kubernetes-sigs/node-feature-discovery#2183.
Ping @adrianchiris @zvonkok @ArangoGutierrez @marquiz we’ve gone through several rounds of review and have LGTM from the compatibility working group. Can any of you review this?…
samuelkgutierrez commented on issue hpc/quo-vadis#349.
Done in #351. Thank you again for the reminder, @GuillaumeMercier….
cmoussa1 commented on issue flux-framework/flux-accounting#703.
thanks @jameshcorbett! Setting MWP here…
cyrush commented on issue Alpine-DAV/ascent#1585.
More build tweaks for mi300a:…
v-dobrev commented on issue spack/spack-packages#496.
@balay, can you please take a look at the petsc addition?
…
jandrej commented on issue mfem/mfem#4951.
Probably because the algoim integration in mfem is outdated and algoim doesn’t use versioning. ping @bslazarov …
v-dobrev commented on issue mfem/mfem#4827.
@mlstowell, I merge this PR in next since we have two approvals, however, you can still review it before merging into master if you want – it is a very small change….
chu11 commented on issue flux-framework/flux-core#6928.
thanks, tweaked the commit message a bit based on your suggestion and removed the setting of $jq in sharness….
gardner48 commented on issue LLNL/sundials#721.
@wuzhiqiao-ninecube thank you for the contribution. The development branched was recently rebased and, rather than trying to resolve the conflicts here, it might be easier to migrate your changes to new branch and open a new PR. As such, I’m going to close this PR….
adrienbernede commented on issue LLNL/CARE#336.
@adayton1 This is ready ;)…
adayton1 commented on issue LLNL/CARE#336.
@adrienbernede is this ready or is it still a work in progress?…
tgamblin commented on issue spack/spack-packages#649.
@alecbcs can you submit another pr for that?…
tgamblin commented on issue spack/spack#51018.
Can you revert the change to move the links to the bottom? I think if you are looking through the README in a text editor you likely want to see the URLs next to the links… or at least I would. The old way is much easier to read….
garlick commented on issue flux-framework/flux-core#6798.
There is a fix for that test failure over in the pr for syslog support….
v-dobrev commented on issue xsdk-project/xsdk-issues#277.
I created a PR that fixes the issue with mfem@develop: https://github.com/spack/spack-packages/pull/496, however, it is still a WIP….
cyrush commented on issue visit-dav/visit#20487.
Ran into some merge issues, closing this branch in favor of #20494…
JustinPrivitera commented on issue visit-dav/visit#20493.
Can you provide a reproducer so we can take a look?…
becker33 commented on issue spack/spack-tutorial#455.
I’ve reverted this PR on main and moved the current state of main to the spack-1.0 branch to facilitate running the PEARC tutorial on Spack v0.23 on Monday…
becker33 commented on issue spack/spack-packages#638.
Closing as superseded by #645 because #645 is the one I backported to the release branch…
vsoch commented on issue oras-project/oras-py#216.
Is this something you’ve discussed with the primary oras maintainers in slack, or is this feature already implemented in oras go?…
dylan-copeland commented on issue mfem/mfem#4326.
Thanks @IdoAkkerman for your updates. I verified that the results for the nurbs_surface are unchanged in the eyeball norm, with differences on the order of round-off, due to the different choice of points used by GetInterpolant. My understanding is that GetInterpolant generalizes FindInterpolant, with Demko points usually being preferable. The changes in this PR seem good to me, to deprecate FindInterpolant in favor of GetInterpolant….
grondo commented on issue flux-framework/flux-core#6908.
@ocaisa, can you try again now that #6915 is merged. There’s some new debug in those tests that may help determine why those tests are failing for you….
jeremylt commented on issue CEED/libCEED#1869.
Why the drop in codecov? I don’t see an obvious reason…
jedbrown commented on issue CEED/libCEED#1865.
@jrwrigh Do you do this (or want to) for the diffusive flux projection? The collocated quadrature produces a lumped mass matrix so it could be considered more consistent to evaluate the flux term in collocated quadrature prior to the lumped mass projection….
trws commented on issue spack/spack#41801.
Yes, I think you’re right about that….
cmelone commented on issue spack/spack#46447.
I’ll try to work on this soon–need to find some time to work on gantry…
vsoch commented on issue pydicom/deid#280.
I would interact through python and put an IPython.embed() here and walk through the logic to understand what is going on (that is what I would try)….
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thanks @fruzsinaagocs…
garlick commented on issue flux-framework/rfc#458.
Eh, I vaguely recollect us discussing this a while back and concluding that it’s probably not worth it….
chu11 commented on issue flux-framework/flux-core#6798.
hit this build failure, it’s a very new test from #6911 so just documenting in case this is a racy pattern (could hypotheticlaly just be a timeout) …
adrienbernede commented on issue LLNL/Umpire#970.
@davidbeckingsale ready for merge. (Test should pass)…
adrienbernede commented on issue LLNL/CHAI#317.
@adayton1 Ready for merge. Test should pass….
v-dobrev commented on issue mfem/mfem#4917.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
chakshinglee commented on issue mfem/mfem#4909.
@tzanio other than “Sanitizer”, I think the failing checks are probably due to the upstream branch, which is still under development….
vsoch commented on issue google/dranet#151.
@michaelasp I have an equivalent example started - but I have a catch22. The environment I’m working in is user space kubernetes. We can’t see any devices without a privileged pod. When we add that, we see everything. This means the selector is irrelevant. I need an environment with production Kubernetes that I can run without rootless. I had Google Cloud credits but they expired, and I’d need to pay out of pocket. @aojea can you help? It shouldn’t be too many to test this out. Once I have that, I can add the full analogous example to the docs….
hppritcha commented on issue open-mpi/ompi#13299.
@jsquyres please review when you have time. we need this in main before branching 6.0.0…
jedbrown commented on issue rsmpi/rsmpi#152.
BorrowedBuf as in read_buf is an alternative interface, though we would need to make our own generic version (preferably in a separate crate)….
vsoch commented on issue pydicom/deid#280.
I can give a suggestion for a strategy to take - if you have very specific, scoped fields you want to keep, you can use the KEEP directive. If it’s more a global thing a strategy akin to the reverse remove_private_tags might make sense….
vsoch commented on issue google/dranet#151.
> I’d like to see an example of this at work, i.e spin up a pod with a resource request and see that the RDMA device gets properly mounted to the pod. Since a lot of that logic depends on NIC devices there may need to be some work done there for proper allocation/release of the device from the host to the pod. …
grondo commented on issue flux-framework/flux-core#6909.
To summarize offline debugging, it was discovered via running rpm --verify that /etc/flux/rc1 and /etc/flux/rc3 had been modified by configuration management and thus the system was running with v0.74.0 with the rc scripts from v0.71.0. This caused the sdmon modules to not be loaded, so the instance was stuck waiting for sdmon to check in….
gardner48 commented on issue LLNL/sundials#702.
@emprice Thanks for reporting this problem. I’ve modified one of our examples using inequality constraints and was able to reproduce similar behavior where the step size is increased rather than cut when a constraint violation is found. This occurs when the predictor and corrector both produce states that do not satisfy the inequality constraint. In this case the computation used to estimate the step size needed to satisfy the constraint is invalid. There are a couple ways to address this. I’ll discuss this we the rest of a team and update this issue when I’ve added a fix….
artv3 commented on issue LLNL/raja-suite-tutorial#40.
I think we can close this PR, we have decided to leave this lesson out of the intro tutorial. …
cmelone commented on issue LLNL/hubcast#145.
the handlers and clients have names like GitLabSrcClientFactory and GitLabSrcHandler. I haven’t changed the destination equivalents yet, or GitHub classes if there’s a more attractive option available…
gardner48 commented on issue AMReX-Microelectronics/FerroX#72.
Requires updates to the AMReX SUNDIALS interface in AMReX-codes/amrex#4562…
cyrush commented on issue visit-dav/visit#20472.
I think I know the issue with the vtkm unstructured grid, its not the coordinates. …
markcmiller86 commented on issue visit-dav/visit#20461.
@Sam-Briney I just wanted to reach out and ask if you might know if you have any automated scripting or some other such activity that is routinely pulling down LFS content in the visit-dav/visit repo? We’ve run into some odd issues regarding our lfs bandwidth quota and I am just trying to investigate….
tgamblin commented on issue spack/spackbot#109.
Spackbot can only request reviews from people in the spack organization. @arezaii and @bonachea are in the Spack organization – specifically they are part of the maintainers team that Spackbot invites org members to….
tgamblin commented on issue spack/spack-tutorial#456.
thanks!…
adrienbernede commented on issue spack/spack#49771.
@becker33 This is the error I am running into….
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thanks @fruzsinaagocs I have added the missing DOIs and replaced the Tudat reference. https://github.com/LLNL/SSAPy/pull/93 . Please let me know if you would like me make any further changes. …
tzanio commented on issue mfem/mfem#4928.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
v-dobrev commented on issue mfem/mfem#4811.
Re-merged in next for testing……
cmoussa1 commented on issue flux-framework/flux-accounting#686.
Thanks for reviewing this @grondo! I force-pushed up a change to check the return code of flux_jobtap_event_post_pack () as well as added a comment explaining that an error there is not fatal. I’ll go ahead and set MWP here…
adrienbernede commented on issue LLNL/radiuss-shared-ci#24.
Pascal was retired on october 2024….
cyrush commented on issue Alpine-DAV/ascent#1559.
I think we can resolve by modifying the bounds data flow filter tracking logic to provide the bounds per topology instead of one overall set of bounds. The results can be in a conduit node instead of a vtkm::Bounds` object….
gardner48 commented on issue AMReX-Codes/amrex#4562.
FYI @ajnonaka, various updates I made when testing with FerroX. I’ll open a PR with some corresponding updates on the FerroX side as well….
becker33 commented on issue spack/spack-tutorial#455.
@tgamblin @kshea21…
v-dobrev commented on issue spack/spack-packages#143.
I’d like to get the PR that adds MFEM v4.8, #496, merged before this PR. That’s why I’ve been postponing the review of this PR for some time now….
hppritcha commented on issue open-mpi/ompi#13299.
related to https://github.com/open-mpi/prrte/pull/61 . We need updates to ompi_main branch on prrte fork to get help messages working for prrte help….
wihobbs commented on issue flux-framework/flux-core#6907.
As for the actual change, looks good to me but we should probably get an approval from @garlick….
benzwick commented on issue Slicer/SlicerBuildEnvironment#25.
It looks good to me too. Thank you so much for helping with this….
alecbcs commented on issue spack/spack-packages#507.
I use labeler in other projects, but it doesn’t support tags that come from the patch of the PR. For example, we should have update-package but we couldn’t have new-version because it only looks at the file modified….
tgamblin commented on issue spack/spack#50972.
> Is saving the compiler wrapper output (Spack debug logs) in the same directory as spack-build-out.txt easy to implement?…