NOVA backend is disabled - unstable package
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Review src/pleiades/sammy/factory.py and the skipped tests in tests/unit/pleiades/sammy/ to understand the current NOVA-disabled behavior. Check the stability of the nova-galaxy dependency before proposing re-enablement; done would mean restoring availability and removing the relevant test skips.
Written by the indexing model from the issue text.
Description
Summary
The NOVA backend for SAMMY execution has been disabled in the codebase. The nova-galaxy package required for this backend is unstable and not a development priority.
Background
PLEIADES supports three SAMMY execution backends:
- Local - Direct execution using local SAMMY binary (primary)
- Docker - Containerized execution (secondary)
- NOVA - ORNL web service execution (disabled)
Why NOVA is Disabled
- Package instability: The
nova-galaxypackage (required dependency) is not stable - No immediate plan: There is no immediate plan to make PLEIADES fully support NOVA
- Development focus: Current development priorities are:
- MCP (Model Context Protocol) integration for AI-assisted workflows
- 2D resonance fitting capabilities
- Core functionality improvements
Code Changes
The following changes were made to disable NOVA:
src/pleiades/sammy/factory.py
- Commented out
NovaSammyRunnerimport - Added
_NOVA_AVAILABLE = Falseflag list_available_backends()now returnsFalsefor NOVAcreate_runner()raisesBackendNotAvailableErrorif NOVA is requested
Tests
tests/unit/pleiades/sammy/backends/test_nova.py- Entire module skippedtests/unit/pleiades/sammy/test_factory.py- NOVA tests skippedtests/unit/pleiades/sammy/test_config.py-TestNovaSammyConfigclass skipped
Code Preserved
The NOVA backend implementation code remains in the codebase for future use:
src/pleiades/sammy/backends/nova_ornl.py- Full implementationsrc/pleiades/sammy/config.py-NovaSammyConfigclass
Re-enabling NOVA
When the nova-galaxy package stabilizes, NOVA support can be re-enabled by:
-
Uncommenting the import in
factory.py:from pleiades.sammy.backends.nova_ornl import NovaSammyRunner -
Setting the flag:
_NOVA_AVAILABLE = True -
Removing skip markers from tests
Related
- Development focus: MCP integration (#163), 2D resonance fitting
- Supported backends: Local (recommended), Docker (containerized)
- Dominant language
- Python
- Stars
- 7
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from lanl/PLEIADES
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
-
Difficulty 5/5 Over a week Newbie friendliness 28/100
-
Add validation constraints to FitConfig Card 2 integer fieldsMay be free again @along4 claimed this 258 days ago, and no pull request is open. Open
-
Clarify precedence of nuclear data cache dir across config layersMay be free again @along4 claimed this 258 days ago, and no pull request is open. Open
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
FinanceFlash/unvibecode#509 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 82/100
FAIRmat-NFDI/nomad-distro-dev#161 ·
-
bug v2
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
modelcontextprotocol/python-sdk#3821 · 1 comment ·
Maintainers usually reply within 1 day
-
bug severity:low
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 2 days
-
Log-odds calibration uses upper middle score instead of median for even candidate setsPossibly taken @flysheep-ai claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day