Macaulay2 / Macaulay2/M2

Reloading a package silently discards the Configuration it was loaded with

Open
#4,521 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bugs directory Core
Dominant language
Macaulay2
Stars
435
Forks
297
Avg merge
4d 20h
Merged PRs (30d)
11

Description

This issue was triaged from bugs/dan/0-reloading-packages, one of the 857 files removed from the pre-GitHub bugs/ tree by d2c8d27826 and catalogued in #36. The commentary below was written by Claude (Claude Opus 5, via Claude Code), not by @d-torrance, whose account posted it -- please weigh it accordingly.

The original file, verbatim
check and installPackage will reload a package on occasion
perhaps they should reload with the same configuration options as the previous time
figure that out
Where it stands today

Still drops the configuration, and the path is short enough to quote.

loadPackage Package forces Reload => true (Core/packages.m2:193), and neither of the two callers
that reload passes Configuration along:

  • installPackage.m2:670
  • testing.m2:87

So a package loaded with Configuration => {...} comes back with the defaults when check or
installPackage reloads it.

Why it bites quietly

Nothing errors. The package simply behaves as though the user had never configured it — which for a
package whose configuration names an external program or a directory means the failure appears later,
somewhere unrelated, as a missing executable or a wrong path.

Notes for whoever picks this up

The configuration in force is recoverable at the point of reload: it is on the loaded package object
itself, so the fix is plausibly to read it back and pass it rather than to thread it through from the
original call. Compare #4536, another consequence of reloading that this catalogue filed, and
#3852, which added the warning about recreating instances after a reload.

open · disposition issue · source of truth: bug-triage/catalog.tsv

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at Core/packages.m2:193, then trace the reload callers in installPackage.m2:670 and testing.m2:87. Inspect how the loaded package stores its Configuration and compare behavior when check or installPackage triggers a reload. Done means a configured package retains that configuration after either reload path, with no silent fallback to defaults.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
74/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.