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?…
hppritcha commented on issue open-mpi/prrte#52.
closing this for now. we can reopen if need be….
hppritcha commented on issue mpi-forum/pympistandard#30.
@jsquyres @wesbland please review…
homeomorfismo commented on issue mfem/mfem#4926.
@tzanio yes!…
jedbrown commented on issue RDycore/RDycore#295.
Does one need gradients or integrals, or only point samples of the solution field from the mesh without interpolation?…
becker33 commented on issue LLNL/uberenv#142.
I think @cyrush ‘s suggestion is probably better – you could still technically create a situation that required cleaning the bootstrap, but by far the most likely reason is two different versions of Spack trying to use the same bootstrap store….
becker33 commented on issue spack/spack-packages#53.
@scottstraughan is there any mechanism other than a file in the intel-oneapi-compilers prefix for oneapi to know where to look for plugins? Anything equivalent to the PYTHONPATH variable for python searching for python modules?…
v-dobrev commented on issue spack/spack#49871.
Migrated to https://github.com/spack/spack-packages/pull/496….
uvilla commented on issue pecos/tps#306.
Sounds good. …
garlick commented on issue garlick/brewcop#5.
Yeah we don’t have those modes anymore, so closing….
vsoch commented on issue rootless-containers/usernetes#373.
We got this working. I wish I could report why - it started working on a restart to the cluster. My suspicion is that the admin didn’t have all the kernel modules loaded that we need, but I don’t know….
imguoguo commented on issue osbuild/bootc-image-builder#963.
> Fwiw, I submited an RFC patch to the qemu-devel mailinglist that return “0” unconditionally for the set_robust_list() syscall. When I do this locally the podman locking works so I’m 90% confident that it solves the problem. I have no sense in how likely it is that the patch gets accepted though (my guess is not very likely), its arguably wrong but also in practice glibc does not care - it is a risk for non-glibc threading implementations though which would be a good reason to reject it (otoh most languages like rust/java use glibc it seems, go does not use set_robust_list and musl https://git.musl-libc.org/cgit/musl/tree/src/thread/pthread_mutex_trylock.c#n30 also does not check the return code ] …
vsoch commented on issue google/dranet#150.
Lost my allocation
uvilla commented on issue pecos/tps#306.
Another options could have been using MPI gatherv to handle all the …
hppritcha commented on issue open-mpi/ompi#13321.
@cpshereda could someone from Cornelis provide a list of sanity checks to run to verify an OPA network is correctly set up?…
tomstitt commented on issue mfem/mfem#4918.
@camierjs already did this in https://github.com/mfem/mfem/pull/4899 =]…
tzanio commented on issue mfem/mfem#4898.
Merged in next for testing……
v-dobrev commented on issue mfem/mfem#4899.
Merged in next for testing……
artv3 commented on issue mfem/mfem#4913.
Can we update the branch with main? I would like to try it, but I need the latest and greatest. …
grondo commented on issue flux-framework/flux-core#6898.
@wihobbs: if you approve I’ll go ahead and set MWP here….
BradWhitlock commented on issue LLNL/axom#1440.
This code does not compile anymore due to a static_assert so I’m closing it….
jeremylt commented on issue CEED/libCEED#1850.
If a user is bypassing our access model, there’s nothing we can do about that. It will always be a bug….
cyrush commented on issue Alpine-DAV/ascent#1552.
win2025 it is….
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thanks both. @athulpg007 we just added some more examples. @andrea-sesta we’re working on updating the JOSS paper text again. Could you please let me know of any more outstanding items we need to take care of for the review? …
jameshcorbett commented on issue flux-framework/flux-sched#1372.
Another user reported this on Tuo:…
garlick commented on issue flux-framework/flux-coral2#376.
Fixed that - thanks! I’ll set MWP…
adayton1 commented on issue LLNL/Umpire#956.
Sorry for all the nitpicks. A lot of these changes have user-facing error messages, which is why I’m being a little pedantic….
adrienbernede commented on issue LLNL/RAJA#1859.
- [x] Merge change in RSC: https://github.com/LLNL/radiuss-spack-configs/pull/133 …
cyrush commented on issue visit-dav/visit#20449.
We should see if this happens with vtk 9.5 since the rendering paths will be different….
tpatki commented on issue kokkos/kokkos-tools#295.
Hi @ethan-puyaubreau …
chu11 commented on issue flux-framework/flux-core#6798.
removed WIP, now that #6772 has been merged…
grondo commented on issue flux-framework/flux-accounting#682.
I wonder if this error is the problem: …
cmoussa1 commented on issue flux-framework/flux-accounting#682.
No, not a problem. I opened this PR mostly as an experiment to see if removing the duplicate coverage.rc that I had in flux-accounting’s checks_run.sh (I think due to my my negligence in #644) would actually trigger the Python bindings from actually getting picked up by Codecov, but I’m still not seeing coverage there.
camierjs commented on issue mfem/mfem#4899.
> This looks good to me. It builds a few heavy libraries (including libcxx), but they are cached and preserved between individual runs right? …
wihobbs commented on issue flux-framework/flux-core#6898.
As I looked deeper in to it, I realized it’s not a problem as I thought it would be. I was expecting we’d have several places where the phrase “job environments for Jobspecs created in Python have an empty environment by default” would show up, but that proved not to be true….
trws commented on issue flux-framework/flux-core#6704.
That sounds right, we could probably get away with a replace op for this one. Maybe allow updates to job.update.<json-patch-path-root> so you could do (I forget the exact path, but something like) job.update./attributes/dependencies to allow updates to it. Depending on how granular we want the control to be that might not be quite right but it would make it easy to allow updates to specific roots of the jobspec, or even restrict it to specific array members if we ever needed to….
balos1 commented on issue LLNL/sundials#736.
FS permissions should be fixed too…
JustinPrivitera commented on issue visit-dav/visit#20154.
Can we reschedule this ticket for a later release now that we have addressed mili?…
v-dobrev commented on issue mfem/mfem#4757.
Re-merged in next for testing……
tzanio commented on issue mfem/mfem#4909.
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-core#6704.
The first thing that went through my mind on this is we could use json patch formatted content. Jansson doesn’t support it natively, but there are examples around the internet for supporting it with anything that has a json_pointer_set or similar. There are a number of tools that can work with or produce it, like a diff output for json in general, so it would likely end up useful other places too. …
balos1 commented on issue LLNL/sundials#732.
I found a workaround with the generator, so this is not needed anymore….
BradWhitlock commented on issue LLNL/axom#970.
I added ArrayView::fill() in an earlier PR so I’ll finish this one….
tpatki commented on issue spack/spack-packages#148.
@slabasan …
vsoch commented on issue singularityhub/singularity-hpc#670.
If you have interest! I didn’t comment because it’s marked as a Draft. Let me know when it’s ready for review….
jedbrown commented on issue rsmpi/rsmpi#209.
Closing in favor of #214 …
jeremylt commented on issue openjournals/joss-reviews#8467.
I would be interested in reviewing, as long as me being out for travel for a couple of weeks in Jul and Aug isn’t a problem for the review timeline….
grondo commented on issue flux-framework/flux-core#6774.
Also forgot to mention that this now has support for a module priority value. Module alternatives are sorted first by priority then by load order, so the last module configured gets preference if priorities are the same. The default priority is 100, and sched-simple has a configured priority of 50.
…
jameshcorbett commented on issue flux-framework/flux-coral2#373.
Looks like a transient error, it’s passing now! Setting MWP….
jedbrown commented on issue Tehforsch/diman#91.
I just wanted to share this update from the const generics group lead:…
adayton1 commented on issue LLNL/spheral#258.
> @adayton1 is this something you’re still working on / planning to work on in the near future? …
cyrush commented on issue LLNL/conduit#1447.
:-) …
markcmiller86 commented on issue LLNL/Silo#448.
FYI, if you have a chance, see #450….
jeremylt commented on issue CEED/libCEED#1847.
Note to me - bump Ratel CI on Monday…
cyrush commented on issue Alpine-DAV/ascent#1571.
This work builds off @nicolemarsaglia’s uniform grid sampling work to add sampling at a list of points, or via a line (start, end, and number of samples). …
vsoch commented on issue zenodo/zenodo#1606.
You know, it’s been enough time this would be a fairly straight forward task for an LLM. As long as you provide it the structure and data you need to populate it….
becker33 commented on issue spack/spack#50936.
@haampie added…
jedbrown commented on issue rust-lang/rust#138689.
@wesleywiser :wave: What are the next steps for this PR?…
garlick commented on issue flux-framework/flux-core#6887.
I was mistaken. pthread_cancel() is only called on each broker module during broker shutdown, from modhash_destroy()…
cmoussa1 commented on issue flux-framework/flux-accounting#677.
Thanks for the review and the helpful feedback @jameshcorbett! I ended up changing this command’s name to just show-usage - I agree with you that it might be a better name for it than list-usage. I also force-pushed up some changes based on your suggestions above. Will set MWP here shortly…
grondo commented on issue chaos/pdsh#170.
Well one thing you could do now is to put a wrapper script ssh into someplace like ~/bin/ssh with the following:…
cmelone commented on issue spack/spack-packages#183.
@G-Ragghianti please rebase your PR on the latest version of spack-packages in order to get CI to pass…
wihobbs commented on issue flux-framework/flux-core#6348.
@kulnaman also suggested this today, specifically for seeing the fluxion versions, which are currently only available through flux dmesg (which is restricted to the instance owner)….
alecbcs commented on issue spack/spack#50931.
Todd and I considered this in #50868 and ultimately decided against it since setting a value would have no effect for users without rerunning spack repo update and thus would be confusing to debug.
…
garlick commented on issue flux-framework/flux-coral2#370.
> We should double check that we aren’t configuring the plugin under the old name anywhere so I’ll mark this WIP pending that. …
adrienbernede commented on issue LLNL/radiuss-spack-configs#133.
Another nice side effect of anticipating the release is that RADIUSS Spack Configs will automatically pick that release in its CI. It wouldn
adayton1 commented on issue LLNL/Umpire#795.
> > What’s the current status of this merge request? I’m testing out asan and finding it works for simple tests. I’m working on enabling it in my application, but until this goes in, I’ll have to disable memory pools. …
adrienbernede commented on issue LLNL/RAJA#1859.
@rhornung67 I
astrophpeter commented on issue openjournals/joss-reviews#8147.
Hi @fruzsinaagocs,…
adayton1 commented on issue LLNL/radiuss-spack-configs#133.
> > > > Where do the 2025.04.0 and 2025.06.0 version numbers come from? …
astrophpeter commented on issue LLNL/SSAPy#88.
This PR Aims to address the JOSS review comment https://github.com/LLNL/SSAPy/issues/80…
jedbrown commented on issue rust-lang/rust#138689.
I rebased. The warnings for implied features were fixed by https://github.com/rust-lang/rust/pull/140920. I updated a stale comment left behind in that refactor, and my handling of features transitively implied by base features moved to rustc_codegen_ssa. …
homeomorfismo commented on issue mfem/mfem#4769.
> I noticed that now the following two sample runs are actually identical: …
v-dobrev commented on issue mfem/mfem#4903.
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/flux-core#6882.
Thanks! I’ll set MWP…
v-dobrev commented on issue xsdk-project/xsdk-issues#277.
Hi @balay,…
adrienbernede commented on issue spack/spack#50916.
For the records, here is a pre-v1 compiler compiler config that I am trying to port over to v1:…
tgamblin commented on issue spack/spack#50920.
This one’s more complete – with updated docs…
garlick commented on issue flux-framework/flux-core#6885.
I’ll set MWP - thanks!…
chu11 commented on issue flux-framework/flux-core#6880.
just pushed a commit on top that implements --tail=N in flux job attach. Everything appears to work as expected.
…
cjekel commented on issue cjekel/similarity_measures#39.
Thank you!…
JustinPrivitera commented on issue LLNL/conduit#1424.
With my two comments above I believe I have addressed all of your feedback @cyrush. I think this is ready to be merged. Please let me know if there is anything else we should discuss….
trws commented on issue LLNL/camp#179.
They could if someone wants to set up an image with one of the CPU-backed SYCL implementations like AdaptiveCPP or the clang upstream offload version (which I think now has “run in separate process” support to give you memory errors on CPUs if you mess up transfers). We don’t have that set up though. …
markcmiller86 commented on issue LLNL/Silo#432.
Already handled in f10090abcc63f5e93d52130db0b8bb0773c765d9…
tgamblin commented on issue spack/spack-packages#44.
@kwryankrattiger …
hppritcha commented on issue open-mpi/ompi#13279.
@edgargabriel i pushed the remainder of the fortran related files to the PR - hopefully….
vsoch commented on issue kubernetes-sigs/node-feature-discovery#2170.
The nfd worker generates a lot of logs (and other unexpected output) so I wasn’t sure how to cleanly do that. Whereas with writing the file explicitly, I can control exactly the contents….
trws commented on issue flux-framework/flux-sched#986.
Not substantively, we operate on the IDs as logical IDs (effectively assuming, in absence of actual structure, that lower IDs are close to other lower IDs). We still have a direct hwloc reader, which I believe is still tested, though we’re not currently using it. We would still need a way to connect the two together. 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. Having the actual locality in fluxion would definitely be good, but with what @ryanday36 showed in the issue description just doing that logical->physical translation even with hwloc-calc right before doing the env var assignment would fix it. Admittedly not fix it the long-term “right” way, but we’ll need that logical to physical translation anyway because we don’t want to be tied to whatever crazy variable numbering cuda or hip use from 1pm to 2pm on alternate Thursdays….
grondo commented on issue flux-framework/flux-sched#986.
This issue came up again in the flux dev meeting yesterday, so just commenting here to revive the issue….
mplegendre commented on issue flux-framework/flux-core#6883.
We’d previously discussed starting a spindle session at allocation time that lasts the whole allocation. That would also solve this problem. But the drawback is it’s hard for users to pass configuration arguments to spindle, since spindle’s session starts before the user gets control. Though I’ll go this route if we can’t do alternatives….
cmoussa1 commented on issue flux-framework/flux-accounting#667.
Rebased this PR to catch up after #672. Setting MWP here…
vsoch commented on issue expfactory-experiments/attention-network-test#2.
What sticks out to me is defining “blocks” that are then looped through for trials. So possible reduce the sections there, and within each entity added to blocks, you can also delete….
balos1 commented on issue LLNL/sundials#721.
Thank you for opening this pull request. Due its size, it will take us awhile to be able to consider and review it. …
alecbcs commented on issue LLNL/hubcast#19.
Oh good point! I think we can close this one out :)…
cmelone commented on issue LLNL/hubcast#19.
@alecbcs is this still relevant given the addition of the config handler?…
cmelone commented on issue spack/spack-packages#107.
If you’d like a review, try posting in the #pull-requests channel on Slack. The CI errors will need to be fixed though…
alecbcs commented on issue spack/spack#50874.
Yep. Working on a few other things today / tomorrow. I can do this Friday….
markcmiller86 commented on issue openjournals/joss-reviews#8106.
I recommend this paper for publication….
adayton1 commented on issue kokkos/kokkos#8195.
Thank you! I’ll give Kokkos_IMPL_AMDGPU_FLAGS a try!…
grondo commented on issue flux-framework/rfc#454.
Setting MWP….
artv3 commented on issue LLNL/RAJA#1853.
Consider moving the plugins into src/plugins – keep as cpp files. …
emily-howell commented on issue Alpine-DAV/ascent#1502.
## TLDR: …
markcmiller86 commented on issue visit-dav/visit#20442.
> Here’s a question: the query output object for Spatial Extents queries is a single entry with key = ‘extents’, and value an array/vector of the values. With the changes to the output string, should the output object (XmlResult) be modified to use the XMin/XMax, etc keys instead of just ‘extents’? If so, this would possibly be more of a problem, even if it is the better way to report. …
imguoguo commented on issue osbuild/bootc-image-builder#963.
> Thank you! This looks nice! Ideally we would have a proper test for this but given that its not an official fedora arch yet it fine. …
jandrej commented on issue mfem/mfem#4898.
> @jandrej, could I suggest a couple small changes that I noticed in the DFEM code? …
tzanio commented on issue mfem/mfem#4898.
This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch….
vsoch commented on issue kubernetes/website#51164.
@graz-dev do you need any more clarification from us? …
samuelkgutierrez commented on issue hpc/quo-vadis#342.
Is something up with Github?…
adayton1 commented on issue LLNL/Umpire#967.
I think you are going to need to update the MemoryResourceType enum in MemoryResourceTypes.hpp. Otherwise, “SHARED” could map to the wrong implementation….
JustinPrivitera commented on issue LLNL/Silo#448.
This would be a nice feature!…
balos1 commented on issue spack/spack-packages#219.
@everythingfunctional Can you try adding depends_on("blas", when="+lapack") to the sundials package.py to see if that fixes this. …
trws commented on issue flux-framework/flux-core#6492.
I don’t recall seeing this when it was being done, but it was mentioned in one of our recent meetings so I came looking. There are at least some updates here, in that uv_ref/unref now exist at least, and the child handling could be handled one of a couple of ways depending on the requirements (either uv_process_t if we can, or uv_signal_t if not). The others I think are still at least different, and the close item is vaguely interesting, I would think we’d be able to tolerate that even though it is a bit annoying. It’s probably safer for io_uring style setups as well as windows, but I’m not 100% sure about that….
gardner48 commented on issue LLNL/sundials#713.
Fixing this typo in another PR….
adrienbernede commented on issue LLNL/radiuss-spack-configs#129.
@tdrwenski Could you open a PR in Quandary that checks whether your CI works well with the present PR in RSC?…
camierjs commented on issue mfem/mfem#4326.
There were some changes to bring from doftrans-three-arg, done in 2bef31b and 4a46614.
…
grondo commented on issue flux-framework/flux-core#6873.
> I wonder if we could add an exec_bypass flag and let this plugin use both flags and post alloc (adding a fake or empty R), free, start, and finish (with job result) and have the proxy job track the state of the actual job? …
garlick commented on issue flux-framework/flux-core#6873.
This is pretty neat. However it seems like its usefulness will be limited as long as the result of the proxy job is always reported as failure. I see why a fatal exception was used to get the job from DEPEND to CLEANUP state in the prototype, since there’s no other way to do that currently, but I wonder if we can think up a better way that allows the proxy job result to reflect the result of the actual job. That way workflow tools, not to mention Flux native dependency schemes like afterok ,would not need to treat delegated jobs specially when they need to fetch the job result.
…
vsoch commented on issue vsoch/oci-python#24.
Thanks! …
markcmiller86 commented on issue visit-dav/visit#4914.
Here is a potential confusion. When listing the dimensions of an multi-dimensional mesh (e.g. 2DCoRectMesh as above), the dimensions are specified as ZYX, not XYZ. So, the mesh is 9 nodes in Y and 33 nodes in X….
JustinPrivitera commented on issue visit-dav/visit#20440.
> > One question - does it make sense to rename bv_qt6.sh to bv_qt.sh, and to change the name in associated build scripts and in modules.xml?
…
jedbrown commented on issue rsmpi/rsmpi#215.
This would be resolved by #214 . There has also been an issue with the runtime feature (cf. #203 ). Do you have an opinion about this question? https://github.com/rsmpi/rsmpi/pull/214#discussion_r2136959267…
jedbrown commented on issue openjournals/joss-reviews#7881.
@editorialbot recommend-accept…
tzanio commented on issue mfem/mfem#4841.
@dylan-copeland , can you please resolve conflicts?…
trws commented on issue flux-framework/flux-core#6315.
> > add the resource.rediscover to the sub-instance config by default…
garlick commented on issue flux-framework/flux-coral2#369.
Cool, thanks. I’ll set MWP….
vsoch commented on issue flux-framework/Tutorials#54.
It also looks like there is a “Slides coming soon!” message that can now be updated….
artv3 commented on issue LLNL/RAJA#1859.
> > > This PR changes cmake to support a minimum cxx standard of 17. …
JustinPrivitera commented on issue Alpine-DAV/ascent#1556.
Let’s have renders for optical_depth fields as well. We should attempt to have tests that capture every single thing included in the output and the inputs we provide in tests should cover as many of the edge cases as possible….
emily-howell commented on issue Alpine-DAV/ascent#1526.
All of these action items were addressed in #1524 …
cyrush commented on issue visit-dav/visit#5770.
> > For AMR, we shouldn’t use VTK. For blanking out cells, we should use nesting….
hppritcha commented on issue spack/spack-packages#209.
@RonRahaman please open a PR for this, thanks….
v-dobrev commented on issue mfem/mfem#4880.
The normal on a boundary edge is not uniquely defined – typically, the edge has two adjacent faces that have, generally, different normals. How do you want to treat these cases?…
grondo commented on issue flux-framework/flux-accounting#659.
It does seem like a good idea to store the priority factor weights in the database, especially if there’s going to be an easy method to view the current weights….
bgunnar5 commented on issue LLNL/merlin#512.
@lucpeterson after spending hours trying to research and implement this, turns out Redis only accepts plaintext in the redis.conf file. Therefore, even if we encrypt the password with Fernet, Redis will just use the encrypted string as the password. I don’t think there’s any way around this so those last three CodeQL issues are unable to be resolved….
cyrush commented on issue Alpine-DAV/ascent#1486.
resolved in #1527 …
chu11 commented on issue flux-framework/flux-core#6871.
thanks! and I just noticed I typoed “updated” in the PR title and the commit message. Fixed that too. Will set MWP…
jameshcorbett commented on issue flux-framework/flux-core#6870.
Force-pushed to rebase with master and expand the test a little bit, setting MWP….
wihobbs commented on issue flux-framework/flux-core#6315.
@eleon has a few thoughts here shared today with @trws and me. Tagging him so he can share more….
jameshcorbett commented on issue flux-framework/flux-coral2#367.
Force-pushed to fix release date, setting MWP….
jedbrown commented on issue RDycore/RDycore#313.
These would be useful for dependencies:…
eleon commented on issue LLNL/mpibind#45.
For mpibind to work as intended, you need to tell Flux you want the full node (-x). I also suggest using as few parameters as possible in your command line since you can, inadvertently, add conflicts between Flux affinity and mpibind. …
tgamblin commented on issue spack/spack#50867.
closing as the RHEL8 UBI issue seems to have sorted itself out….
chu11 commented on issue flux-framework/flux-accounting#656.
I wasn’t able to find where in the code you configure journal_mode and synchronous modes. In content-sqlite it is set to “WAL” and “NORMAL”, for a smidge better performance….
tgamblin commented on issue spack/spack-packages#127.
I think the answers would be: …
tgamblin commented on issue spack/spack#50595.
I think we should do this but we are attempting to get #47615 in for 1.0, which also adds the hash of the spack instance to the misc_cache. …
jedbrown commented on issue rsmpi/rsmpi#203.
@mscroggs Following up on this, we get build failures on MacOS without the runtime feature. It works everywhere else. Do you know of a way to fix this?
…
tzanio commented on issue mfem/mfem#4804.
Re-merged in next for testing……
tpatki commented on issue flux-framework/flux-core#6873.
Hi @trws @garlick @grondo: …
alecbcs commented on issue spack/spack#48820.
Closing since this PR was sucessfully migrated to spack/spack-packages….
v-dobrev commented on issue spack/spack#49790.
bwpriest commented on issue grid-parity-exchange/Egret#318.
> @bwpriest I do not understand the failures … maybe I messed something up in 160a4a4? …
artv3 commented on issue LLNL/RAJAPerf#515.
> > yes, 2 is equivalent to the x index in cuda/hip …
cyrush commented on issue Alpine-DAV/ascent#1550.
…
v-dobrev commented on issue mfem/mfem#4851.
I’m not sure why this was merged without approvals and with requested changes pending. Is the idea to review these changes as part of #4800?…
trws commented on issue flux-framework/flux-core#6858.
I really like this, had one tiny docstring tweak and a question but it looks really good @grondo….
grondo commented on issue flux-framework/flux-core#6868.
Thanks! Setting MWP….
cmoussa1 commented on issue flux-framework/flux-accounting#631.
Looks like #632 ended up fixing this issue on tuolumne at least, so I’ll go ahead and close this now….
markcmiller86 commented on issue visit-dav/visit#20396.
@JustinPrivitera the best workaround I have found so far is to just create an alias expression and then use that alias in any further expressions requiring the original variable. On my mac with current develop, I don’t get the garbage data condition then. So, in other words, define foo as an alias for ElementBlock and then define bar as foo*1 and that works fine….
cmelone commented on issue spack/spack-tutorial#443.
The changes LGTM (after rebase), my other comments are not blockers…
hppritcha commented on issue open-mpi/ompi#13288.
With big count disp_unit argument is of type MPI_Aint for the “_c” routines. So that’s the reason why we’re using ptrdiff_t now….
brendankeith commented on issue mfem/mfem#4873.
@mdmeeker I think this breaks something in navier_kovasznay.
…
tomstitt commented on issue mfem/mfem#4821.
> @tomstitt, is this ready for review? If it is, can you please add the ready-for-review label?
…
garlick commented on issue flux-framework/flux-core#4201.
…
jameshcorbett commented on issue flux-framework/flux-accounting#652.
Oh my bad!…
BradWhitlock commented on issue LLNL/conduit#1439.
This is working with Conduit (and the host code). I plan to do some more testing to see whether other test suites are affected….
markcmiller86 commented on issue visit-dav/visit#20396.
If I look at an integer variable from a Silo file such as ivar from specmix_quad.silo, I DO NOT experience the same behavior. So, it may be somehow specific to the Exodus reader….
alecbcs commented on issue spack/spack#50565.
@alalazo after this PR/commit I can no longer concretize emacs +native. I’ve checked my packages configuration and the gcc external correctly shows as having languages=jit so I’m a bit at a loss for why the package is suddenly failing to use the external.
…
vsoch commented on issue oras-project/oras-py#204.
Why are you calling this in one off functions instead of at the class init?…
tzanio commented on issue mfem/mfem#4804.
Merged in next for testing……
grondo commented on issue flux-framework/flux-core#6843.
Closing as duplicate of #5035….
BradWhitlock commented on issue LLNL/conduit#1434.
I implemented adjset point sorting based on spatial bins. This works for the Blueprint tests but for some reason, it totally fails on the test problem I’ve been debugging. …
jedbrown commented on issue CEED/libCEED#1830.
I feel it. FWIW, it was once Chop and changed to Filter in https://gitlab.com/petsc/petsc/-/merge_requests/6699….
trws commented on issue spack/spack#50702.
Yup, go for it….
v-dobrev commented on issue mfem/mfem#4821.
@tomstitt, is this ready for review? If it is, can you please add the ready-for-review label?…
chu11 commented on issue flux-framework/flux-core#6840.
thanks, I’ll set MWP and adjust the github settings for the CI adjustment….
BradWhitlock commented on issue LLNL/axom#1580.
The idea was to add workarounds for double into the new wrappers for axom::atomicMin and axom::atomicMax (https://github.com/LLNL/axom/blob/3666e7888c7542c50ab4177044089da539b1c933/src/axom/core/execution/atomics.hpp#L51) …
v-dobrev commented on issue GLVis/glvis#336.
> @justinlaughlin, what happens if we load a steam with “screenshot” command in the web version? …
jandrej commented on issue mfem/mfem#4804.
Reminding all reviewers to approve or suggest changes asap….
jameshcorbett commented on issue flux-framework/flux-core#6842.
> After typing that up, I realized one issue with a special exit code is that exiting early from the prolog due to a potentially nonfatal error is probably not a good idea. This could skip other parts of the prolog that are necessary for preparing a node for use by the user. The node may be drained, but the user could still log in, or choose to undrain the node….
grondo commented on issue flux-framework/flux-accounting#648.
> LGTM and sure, a PR in flux-core sounds good! …
garlick commented on issue flux-framework/flux-accounting#648.
LGTM and sure, a PR in flux-core sounds good!…
cmelone commented on issue cmelone/hubcast-test#1.
@lc-hubcast help…
balos1 commented on issue LLNL/sundials#706.
> > @gardner48 I believe this is ready to be merged. …
gardner48 commented on issue LLNL/sundials#706.
> @gardner48 I believe this is ready to be merged. …
alecbcs commented on issue LLNL/hubcast#137.
Looking like a great start! Instead of referring to Hubcast as /hubcast <cmd> let’s instead add a configuration variable (maybe HC_GITHUB_BOT_USER or similar) so that you instead refer to the bot like @lc-hubcast <cmd>. …
jeremylt commented on issue openjournals/joss-reviews#7124.
Overall looking good….
homeomorfismo commented on issue mfem/mfem#4769.
From https://github.com/mfem/mfem/pull/4769#issuecomment-2822389336 @v-dobrev’s comments …
cmoussa1 commented on issue flux-framework/flux-accounting#638.
This is now completed by #643, #645, and #647, so I’ll go ahead and close this. I can re-open more specific issues if they come up….
wihobbs commented on issue flux-framework/Tutorials#48.
Re-triggered, thanks. …
jeremylt commented on issue bempp/kifmm#147.
As far as I can tell…
cmelone commented on issue LLNL/hubcast#138.
ok that worked, not sure why that wasn’t necessary to get it to work for repligit. thanks!…
tzanio commented on issue GLVis/glvis#339.
To clarify: “DOF” here means the numbering of the Mesh nodes (as a high-order grid function), right?…
cjvogl commented on issue E3SM-Project/E3SM#7261.
> I fixed a few minor things and I added docs. Pls take a look before we merge …
astrophpeter commented on issue openjournals/joss-reviews#8147.
Thank you. We are actively working on the two issues raised. …
hppritcha commented on issue open-mpi/ompi#13279.
note that mpi4py is probably not testing this since it keys off our MPI_Get_version return values for major/minor. …
garlick commented on issue flux-framework/flux-coral2#344.
Update: two internal documents were shared by HPE. Although not HPE-confidential, it was requested that they not be published on github. They are:…
astrophpeter commented on issue LLNL/SSAPy#78.
@SuperdoerTrav please take a look. …
artv3 commented on issue LLNL/RAJA#1751.
Done, we added a helper function to simplify construction. …
artv3 commented on issue CEED/Remhos#54.
Can I merge? …
cyrush commented on issue Alpine-DAV/ascent#1540.
We do not have general support for reading vtk files. …
brendankeith commented on issue mfem/mfem#4645.
@thatguynoe, I put the WIP label back because I noticed one of the items supposed to be in this PR is using the bisection method instead of Newton for root-finding…
homeomorfismo commented on issue mfem/mfem#4769.
> * abs-l1-jacobi:
…
tgamblin commented on issue spack/spack#47590.
@spackbot run pipeline…
grondo commented on issue flux-framework/flux-core#6835.
> I’m building an EasyBuild recipe for Flux,…
JustinPrivitera commented on issue visit-dav/visit#20404.
> So, ChatGPT is telling me that Hybrid in cmocean colormaps (which applies to oxy map only I think) is that it combines features of both Sequential and Diverging. So, I think simply including both tags on that map achieves the notion of Hybrid. Thoughts?
…
alecbcs commented on issue spack/spack-distroless#47.
@dependabot rebase…
vsoch commented on issue singularityhub/shpc-registry#327.
What do you suggest the shpc library do about a registry rate limit? I’m not sure there is anything in our control to change or help….
jedbrown commented on issue rust-lang/rust#138689.
Thanks. I made the tracking issue #141468 and have updated the documentation to warn against use of #[target_feature(enable = "ptx80")], as well as clarifying how to build. Any further discussion of intrinsics in the documentation can happen after https://github.com/rust-lang/stdarch/pull/1805….
astrophpeter commented on issue openjournals/joss-reviews#7769.
Thanks! All changes look good to me and I just merged them. …
hppritcha commented on issue mpi-forum/mpi-issues#733.
Okay we’ll put the attributes thing back on the table. I’ve no objections myself and its not like they’re difficult to implement….
jedbrown commented on issue mpi-forum/mpi-issues#733.
> they may be targeted for deprecation/removal at some point…
trws commented on issue flux-framework/flux-sched#1372.
Ok, that sounds like a failure misidentified as ENOMEM then… will have to look and see if we have some that might go rogue….
grondo commented on issue flux-framework/flux-sched#1372.
The sar data is sill there, here’s a report on memory around that time:…
garlick commented on issue flux-framework/flux-core#6827.
That does seem more flexible. Up to you - I’m fine either way….
BradWhitlock commented on issue LLNL/axom#1231.
I came across this issue and it was sort of related to issues I ran into when doing #1581 so I made additional changes there to fix this….
garlick commented on issue spack/spack#49033.
> I think logic to use flux directly would likely be more robust in the long run, but if the slurm wrapper is maintained as part of flux then I don’t see a hard requirement not to use it. …
gillette7 commented on issue mfem/mfem#4868.
Thanks for your prompt response! This is extremely helpful….
v-dobrev commented on issue mfem/mfem#4804.
Note: I updated the GNU make build system for Enzyme a bit: it is no longer needed to set ENZYME_LLVM_VERSION – it should be auto-detected just from ENZYME_DIR similar to CMake.
…
chrundle commented on issue meta-llama/PurpleLlama#109.
Thank you for the quick and detailed response. I will apply with Azure to modify my filter and hopefully that will resolve the issue….
vsoch commented on issue kubernetes/enhancements#4671.
> Would anyone be interested in pursuing the possibility of having filter extension point to take a group of pods and a group of nodes and perform the searching for the combination of nodes for a list of pods at once? The current filter extension point is not built for this. Yet, some plugins might be extended this way and if a proper implementation is provided this might be a path forward. Has anyone already concluded this is no-go? Or, this path has not been explored in more detail due to its complexity?…
vsoch commented on issue hpc-maths/samurai#322.
I would basically do:…
garlick commented on issue flux-framework/rfc#454.
Rerunning the link check as the link it was complaining about works for me….
chu11 commented on issue flux-framework/flux-core#6828.
I began to work on what would be a async-blobref check hidden behind a future as discussed above, but
…
cmoussa1 commented on issue flux-framework/flux-accounting#644.
Thanks @grondo! Rebased to catch up after #643. Setting MWP here…
tpatki commented on issue flux-framework/PerfFlowAspect#122.
Closing this issue as it is Lassen-specific, and Lassen has been removed from our preferred systems list due to older clang version….
bgunnar5 commented on issue bgunnar5/merlin#21.
@doutriaux1 coverage isn’t implemented yet but I think I have a branch that might have added it…
cyrush commented on issue Alpine-DAV/ascent#1532.
No worries thanks for trying out this possible approach and for the feedback. …
emily-howell commented on issue Alpine-DAV/ascent#1524.
@cyrush I think this has all the requests (minus robust documentation which you said should go in a future PR). I added the family value as a fallback, default formats (06d for cycle and family g for time) and an _ in front of the default fmt value iff the path does not already end with a _ character.
…
balos1 commented on issue LLNL/sundials#703.
Fixed by #704….
cmelone commented on issue LLNL/hubcast#121.
is there any situation where a user not in the account map would need to interact with the bot?…
astrophpeter commented on issue LLNL/SSAPy#77.
@SuperdoerTrav can you take a look please? …
tpatki commented on issue spack/spack#50550.
Hi @vlkale The following tells me that an older version of Variorum is getting picked up in this build. …
jandrej commented on issue mfem/mfem#4804.
> I would suggest slight reordering of the example and adding some more explanations. It will be helpful to have two additional examples. One with inputs gradients and primal field and one with several different filed inputs. Adding a recipe will help users to build their own operators. …
wihobbs commented on issue flux-framework/flux-accounting#636.
> Just a thought but would command completions like flux-core’s be another way to solve this problem? …
cmelone commented on issue cmelone/hubcast-test#2.
/hubcast approve…
gardner48 commented on issue LLNL/sundials#660.
Closing per our discussion today (20 May 2025). If this becomes an issue again in the future, we’ll revisit requiring only one of the vector inputs to be the same as the output. …
adayton1 commented on issue LLNL/CHAI#314.
Just testing this out in my application….
tpatki commented on issue ovis-hpc/ldms#1664.
@nichamon Rebased to b4.4. Tests are passing now….
tzanio commented on issue mfem/mfem#3837.
Merged in next for testing……
btalamini commented on issue LLNL/serac#1384.
On our clang version (14.0.0), compilation should fail for this if -Wuninitialized is set (or any flag that includes this) (example) Please change the CI builds to include this….
jeremylt commented on issue CEED/libCEED#1826.
Yeah, at some point it would be nice to tidy that up more. I have mixed feelings sometimes about how much code to collapse together as these are sorta intended as tutorial examples?…
jedbrown commented on issue CEED/libCEED#1824.
For GPUs, where would we store this product $CB$? With non-tensor elements, we have the same matrix for all elements and thus have efficient reuse. I don’t know if this $CB$ technique is good or bad, but storing multiple such matrices per thread block is a lot….
v-dobrev commented on issue mfem/mfem#4768.
Hi @adam-sim-dev,…
jakubcerveny commented on issue jakubcerveny/gilbert-paper#10.
Thank you @drupol, Typst indeed looks great. I’m not sure yet if we want to switch to Typst at this point for this particular paper, but I’ll definitely be using it for my other work, for example I recently needed to add math to a Markdown document and convert it to PDF, and Typst would have been perfect, so next time I’ll go straight to it….
jakubcerveny commented on issue jakubcerveny/gilbert#19.
Amazing, thank you!…
v-dobrev commented on issue spack/spack#50461.
Just FYI: https://github.com/spack/spack/pull/50401 is now merged in develop, so merging develop here should fix the Gitlab CI issue….
vsoch commented on issue rootless-containers/usernetes#372.
It’s called TOSS, and it’s a derivative of RHEL 8.10…
tzanio commented on issue mfem/mfem#4804.
> I don’t intend to have serial only versions for now. …
dylan-copeland commented on issue mfem/mfem#3862.
This PR removed the input flag -r and added serial and parallel flags -rs and -rp, but the first sample run still has -r. The sample run should be updated….
garlick commented on issue flux-framework/flux-core#6811.
Thanks - fixed that and rebased on master. I’ll set MWP….
cyrush commented on issue visit-dav/visit#20388.
I think it would be good to preserve -no-thirdparty.
…
JustinPrivitera commented on issue visit-dav/visit#20398.
No worries; glad you got it figured out! Let us know if there is anything else we can help with!…
jandrej commented on issue mfem/mfem#4804.
> > > MFEM must be compiled with the same Clang as Enzyme …
wihobbs commented on issue flux-framework/flux-sched#1372.
Ah, thanks for the tip! The “around the same time” comment made me think you were grep’ping for timestamps or something. Now I see you were just commenting that fluxion reported the memory error around the same time as core reported an exception….
grondo commented on issue flux-framework/flux-docs#302.
> Edit: I often want to connect to my most recent running job, (which is usually flux job last but maybe I submitted a bad job last) so I use flux proxy $(flux jobs -no {id.f58}). I also often add -u flux to the flux jobs call to connect to a multi-user subinstance….
chu11 commented on issue flux-framework/flux-docs#302.
As an aside, I did write this tutorial a long time ago….
wihobbs commented on issue flux-framework/flux-docs#302.
> And we could start with the most basic examples and build on this whenever we think of another convenient example to have?…
cmoussa1 commented on issue flux-framework/flux-docs#302.
IMHO a new sheet (perhaps on the docs site) with the most frequently and most useful options could be nice for an “ever-expanding” list of examples. You could create one batch of 10 jobs (5 running, 3 scheduled, 2 completed, etc.) and use a varying list of flux jobs commands to list different lists of jobs based on the formats you discussed above? And we could start with the most basic examples and build on this whenever we think of another convenient example to have?…
grondo commented on issue flux-framework/flux-core#6822.
BTW, I did consider adding a --resilient option that selected a “good” value for tbon.topo and add_brokers but I feel like that’s premature at this point.
…
chu11 commented on issue flux-framework/flux-core#6802.
thanks! Tweaked that minor documentation and will set MWP….
jameshcorbett commented on issue flux-framework/flux-coral2#361.
Will set MWP once https://izlc.llnl.gov/gitlab/LC/ansible/-/merge_requests/19077 is merged….
trws commented on issue LLNL/camp#174.
I was thinking so too, but it isn’t entirely straightforward because there’s no good way to have a fold skip elements or filter into an argument list, especially using emulated folds in 14. Definitely something to think about when we can though….
jeremylt commented on issue CEED/libCEED#1808.
Perhaps we just slap a max version on the options above since we’re about to release v0.13…
jedbrown commented on issue CEED/libCEED#1808.
I’d think it isn’t even necessary to explicitly OPT=. In practice, what sort of options get passed? We liked -fopenmp-simd because it vectorized better, but I think we worked out detection of vendor-specific attributes so that we can get good performance without it. I like simpler….
markcmiller86 commented on issue visit-dav/visit#19549.
> With the test file, the problem has been fixed. However, I am still hitting the same problem with the simulation output….
astrophpeter commented on issue openjournals/joss-reviews#7769.
Thanks! The duplicated bib item has now been removed….
homeomorfismo commented on issue homeomorfismo/STAT409Project#14.
Should have been rebased…
grondo commented on issue flux-framework/flux-sched#1372.
FYI - flux job id JOBID is an easier way to convert Flux jobids:…
wihobbs commented on issue flux-framework/flux-core#6503.
Ok, this is ready for review. I did fix two issues I saw where a parameter had been accidentally appended to a Returns: instead of with the Args:.
…
cmoussa1 commented on issue flux-framework/flux-accounting#633.
thanks! setting MWP here…
jameshcorbett commented on issue flux-framework/flux-accounting#628.
> I see now that the message is prefixed with the actual module name, which is nice …
tpatki commented on issue flux-framework/PerfFlowAspect#130.
New PR created #190. Closing….
aidan-epperly commented on issue aidan-epperly/structured-sdp-solvers#4.
Great catch thanks!…
cyrush commented on issue LLNL/uberenv#141.
This will be very helpful, thanks for working through a solution that allows you to reuse the common chunks of the configs. …
balos1 commented on issue LLNL/sundials#705.
Value change was made in https://github.com/LLNL/sundials/pull/565. …
alecbcs commented on issue LLNL/hubcast#40.
Completed in #122….
bwpriest commented on issue Pyomo/mpi-sppy#516.
Addressed by PR #519 …
adayton1 commented on issue LLNL/Caliper#636.
What symptoms did this fix?…
JustinPrivitera commented on issue Alpine-DAV/ascent#1526.
filename{cycle:d} appears to partially work:…
emily-howell commented on issue Alpine-DAV/ascent#1526.
@cyrush @JustinPrivitera As requested, these are all of the current issues that I am aware of. Please feel free to add comments with any I have missed….
hppritcha commented on issue spack/spack#50366.
Thanks for pointing this out. I’ll rebase this PR and see if it fixes this issue….
hppritcha commented on issue open-mpi/prrte#53.
@jsquyres could you delete the main branch?…
tzanio commented on issue mfem/mfem#4847.
Merged in next for testing……
grondo commented on issue flux-framework/flux-core#6806.
Thanks! I gave #6808 a very quick test this evening on 64 real nodes with 54 brokers per node (size=3456) using 3level:2 and 3level:8. This does seem to have a measurable impact. 3level:2 got up to ~9 job/s total throughput and 3level:8 got 28 job/s….
vsoch commented on issue mpi4jax/mpi4jax#280.
Will be testing on multiple nodes later today - in the meantime I built the container and made the animation, it’s gorgeous!
vsoch commented on issue converged-computing/flux-apps-helm#24.
And a quick note - I’d also like to be able to list container tags for a specific app, and then see the associated compatibility metadata tags. I’d also like to preview commands….
vsoch commented on issue DLR-AMR/t8code#1615.
Hi @Davknapp ! I had a go at testing this today - first on a small, single container (it seemed to work OK there) but then when I moved to 4 nodes in Kubernetes, it hung here for 8 minutes:…
JustinPrivitera commented on issue visit-dav/visit#20385.
Hi @Sam-Briney, thanks for these changes. One of us will review. Do you have any tests/test files that we can add to our test suite?…
tpatki commented on issue ovis-hpc/ldms#1664.
Thank you @morrone! That’s indeed great news. …
v-dobrev commented on issue mfem/mfem#4800.
@mlstowell, as long as the new enum values are documented, I’m fine with the names you proposed. …
homeomorfismo commented on issue homeomorfismo/STAT409Project#10.
Can you do an interactive rebase? Some commits can be squashed, and the branch as a whole can be rebased (so you don’t have to deal with a merge commit. …
jameshcorbett commented on issue flux-framework/flux-core#2875.
Yeah, for --rabbits the plugin would be pretty much the same as the --bar plugin in this comment above….
wihobbs commented on issue flux-framework/flux-core#2875.
@jameshcorbett, if you were going to do that, does the CLIPlugin interface as we’ve currently designed it make it clear how to do that? It’s pretty similar in all approaches….
v-dobrev commented on issue LLNL/RAJA#1830.
Regarding:…
cyrush commented on issue visit-dav/visit#20346.
We need to figure out if the current version can work, however this will be deferred to future releases. …
markcmiller86 commented on issue visit-dav/visit#20341.
> Steps to reproduce the behavior. For example:…
hppritcha commented on issue spack/spack#50371.
would you mind uploading the config.log from the pmix build?…
jeremylt commented on issue openjournals/joss-reviews#7124.
Ok, I’ve got a good build now, thanks….
jameshcorbett commented on issue flux-framework/flux-sched#1371.
Rebased with master and fixed the release date, setting MWP….
chu11 commented on issue flux-framework/flux-core#6794.
> Correct me if I’m wrong - if the “real” output cap is 1000 right now, we keep going until we get 1000, right? …
trws commented on issue LLNL/camp#174.
Interesting, not a bad idea. Would you be willing to add a link to where it’s used in that PR? I’m curious about the usage pattern, the implementation looks easy to follow, which is great, but we might be able to make it a bit faster on template expansion depending on the usage….
artv3 commented on issue LLNL/RAJA#1838.
> > I’m a bit confused on why this PR is out of date with develop, additionally it won’t let me update the branch using the web interface …
tzanio commented on issue CEED/Remhos#52.
Aside from git tags and version numbers, the Spack packages for Laghos and Remhos should be very similar, if not identical…
vladotomov commented on issue CEED/Remhos#52.
Once MFEM is built for GPU, the build of Remhos follows. Same as Laghos….
cyrush commented on issue Alpine-DAV/ascent#1509.
spack case for macOS, having an issue with gfortran rpath. …
cmelone commented on issue spack/spack#49243.
@wdconinc what did your DAG look like to build v2.6.0? I can’t get it to build :( …
camierjs commented on issue mfem/mfem#4848.
For the scaling study, you can directly use some command line like:…
cmoussa1 commented on issue flux-framework/flux-accounting#589.
Just tested this on corona and it looks like toggling WAL-mode did not seem to help with the 30-second hang while the job-usage and fairshare update scripts were running. I’ll have to look into other potential techniques. …
cjekel commented on issue cjekel/piecewise_linear_fit_py#130.
With that many data points and two segments, I would want to do one of the optimizations that you’ve done….
cyrush commented on issue Viskores/viskores#72.
I see – I had the clip inverted in my head (that we were keeping what was inside the box vs removing it)…
bgunnar5 commented on issue LLNL/merlin#498.
@lucpeterson @doutriaux1 This PR needs a new review as I’ve changed it a lot. See the new description above to understand what all is in this….
tomstitt commented on issue mfem/mfem#4305.
Closing because of https://github.com/mfem/mfem/pull/4660, I will profile and maybe make a new PR on top of that…
trws commented on issue flux-framework/flux-sched#1362.
Ping, it would be good to get this merged to take care of #1180….
balos1 commented on issue LLNL/sundials#699.
To match Spack’s formatting, I think we should add a pyproject.toml in the root of the repot with the two relevant settings: …
imguoguo commented on issue milkv-pioneer/issues#37.
If you’re still using Fedora 38 on Milk-V Pioneer, maybe try changing the repo in /etc/yum.repos.d to this URL:…
markcmiller86 commented on issue betterscientificsoftware/bssw.io#2310.
Also, https://ladybird.org…
mplegendre commented on issue LLNL/Adiak#34.
We’ve got clustername and launchdate as already collected metadata. Launchdate will be similar to submit time, but not precisely….
vsoch commented on issue LLNL/benchpark#748.
How can we use something from a private repository?…
vsoch commented on issue tudasc/cfdSCOPE#1.
Will do! I ran a study this morning - I am using a helm chart that will install and run on on one node (Kubernetes), and then I “scale” via the OMP_NUM_THREADS. I’m not sure more threads made it faster - I’ll share with you the plots when I make them (I have still about 20 other apps to test)!…
v-dobrev commented on issue spack/spack#47642.
The reason the Gitlab CI failed for the last commit (cf172861) is the following: …
hppritcha commented on issue open-mpi/ompi#13179.
I refreshed my checkout and now do not see the man page build problem….
tzanio commented on issue mfem/mfem#4228.
@tomstitt, can we close this (in favor of https://github.com/mfem/mfem/pull/4305)?…
cjekel commented on issue cjekel/piecewise_linear_fit_py#128.
@matt8s I’m not opposed to adding this as a method if you find it useful. …
JustinPrivitera commented on issue LLNL/conduit#1424.
Ok, let’s look side-by-side. Debugging it the first time took several hours and I never could find out what was going wrong….
cyrush commented on issue LLNL/conduit#1424.
I still want to discuss why we have to use the unique pointers. Something is off if you were seeing crashes with and I want to understand….
markcmiller86 commented on issue LLNL/Silo#22.
> how usable do you consider your CMake build system at this point, and do you plan for a new release? Distributions are starting to [or are right in the middle of] cleaning up Qt5 revdeps….
cyrush commented on issue Alpine-DAV/ascent#1509.
…
JustinPrivitera commented on issue visit-dav/visit#5770.
> Is there a way to write it to minimize the pain of transisitioning to AMR in the future, e.g., by returning a vtkOverlappingAMR dataset?…
vsoch commented on issue oras-project/oras-py#197.
@Sojamann it’s a developer library by default - I would argue that your use case is niche and should not refactor the entire structure of the library to support when it’s as easy as tweaking the session on the class….
jeremylt commented on issue openjournals/joss-reviews#7124.
This week was eaten by an interview process for me. The review work is slated for early next week on my end….
v-dobrev commented on issue mfem/mfem#4806.
Merged in next for testing……
grondo commented on issue flux-framework/flux-core#6790.
Thanks @wihobbs! Fixed that typo and will set MWP….
cmoussa1 commented on issue flux-framework/flux-accounting#621.
thanks! setting MWP here…
jedbrown commented on issue coreylowman/cudarc#401.
I think the weakest answer to these depends on the implementation language doing the copy or handling results on the device. But suppose the kernel that’ll operate on the device data was written in Rust? In that case, the uninitialized memory cannot exist in a slice or similar object (it must be behind MaybeUnint<T> or a raw pointer). I think it’s preferable for cudarc to enforce that slice-like objects on host or device contain only initialized memory. (One can use MaybeUninit<T> if they really need to handle uninitialized memory.)…
aperloff commented on issue MakieOrg/GeoMakie.jl#322.
Related to #308 …
artv3 commented on issue LLNL/RAJA#1837.
> We have advertised that old-style reductions are not supported for SYCL. Not all of the old reduction types work with SYCL. However, we have not yet removed the code for them. …
jeremylt commented on issue CEED/libCEED#1805.
Removal - we don’t need ex3_volume_qfunctions.py or setup.py
…
tpatki commented on issue ovis-hpc/ldms#1664.
Thanks @morrone for clarifying on our behalf! …
wihobbs commented on issue flux-framework/flux-core#6789.
> Oh, I guess this is perhaps by design, though I can’t think of why at the moment:…
balos1 commented on issue LLNL/sundials#464.
> Is it possible to update the Sundials spack recipe to reflect this issue? Either as a patch or a conflict or something?…
eliasboegel commented on issue CEED/libCEED#1813.
I also faced the ar D issue, and worked around it by using a third party bintools distribution rather than that shipped with Apple Clang. Made no difference on the weak symbol issue though….
jedbrown commented on issue CEED/libCEED#1813.
Revisiting this while trying to understand the root issue of weak symbols not working with static linking on MacOS. I don’t have a solution there, but I found that ar crD doesn’t work because Apple ar does not support D (deterministic), but one can use the environment variable ZERO_AR_DATE=1.
…
alecbcs commented on issue spack/spack#50241.
You can run spack gc after your build to clean up the build dependencies. (Which in this case will just leave you with gh since it’s statically compiled.) I believe that src/ is required for language server support for users who are developing go applications since src is the go standard library….
jedbrown commented on issue rust-lang/cargo#15210.
Any ideas to mitigate the issue of tests being redundant, so that when the add function is edited, they don’t have to manually update/delete two duplicate tests? There could be two functions, like add and square, one with a doctest and one with #[test], but that’s a few more lines….
bgunnar5 commented on issue LLNL/merlin#503.
@doutriaux1 I’m so confused on what’s going on with these tests. It looks like they’re failing due to an issue with pip. However, yesterday I re-ran the unit test jobs and those just ended up passing on the second time around. When I re-run the local tests and integration tests they just continue to fail. Nothing I’ve changed in this PR should be breaking any tests… I’ll keep looking…
adayton1 commented on issue LLNL/RAJA#1836.
Thanks, @artv3 !…
jedbrown commented on issue tov/libffi-rs#120.
MSRV is considered a minor change, but the breakage happened before the last release (cf. https://github.com/tov/libffi-rs/commit/d8705c3836e24bd024d6881d0cfd73d0ea9749af#r155795763) so this is just formalizing so cargo can take it into account in dependency resolution….
jedbrown commented on issue rsmpi/rsmpi#211.
Good. I think one can still “cheat” with something like…
markcmiller86 commented on issue openjournals/joss-reviews#8106.
@junghans I believe I have completed my review in the above comment. Let me know if you need anything else….
camierjs commented on issue mfem/mfem#4757.
The cmake & make tests look good, thank you @helloworld922.
…
dylan-copeland commented on issue mfem/mfem#4125.
Thanks @JacobLotz for the explanation. I agree that we can just close this PR for now. Of course, we can always make this change later, if we find a test that needs it. Although it is a very small PR, my concern is that the changes are difficult to understand without an example to demonstrate the problem. We can keep this PR in mind if we run into issues with the edge to knot mapping….
jakubcerveny commented on issue jakubcerveny/gilbert-paper#4.
Thanks for the pictures and explanation, this is now much more clear. I like the adaptive idea, but I have two concerns:…
chu11 commented on issue flux-framework/flux-core#6785.
oh wait, the path is used within the libarchive code … which probably something more filesystem like….
jbschroder commented on issue Parallel-in-Time/pySDC#547.
I also strongly support this idea. This project would make PySDC more accessible, including to students at other institutions (like mine) who would be interested, and it would help open up collaborations with other Python packages (like PyAMG, which implements algebraic multigrid, AMG, for spatial solves, where I’m a co-developer)….
astrophpeter commented on issue LLNL/SSAPy#60.
This is amazing! @aperloff can you take a quick look?…