GROMACS: included topology files (force-field folders such as min.ff/) are not collected
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
Piste de recherche
Start with src/wxgui/mdtools/GromacsInputsPanel.C, where include files are attached one at a time and stored by file name only, and trace how the launch check reports grompp's error. The change needs recursive #include discovery relative to each including file, kept paths in the run directory, and a missing-files state that blocks launch. Done means the tests/gromacs_md case with a nested local force field passes both the launch check and the job script.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
A GROMACS topology that includes local files, here a force field in a folder next to it, cannot be run:
grompp rejected the inputs, so the job was not submitted.
command: /usr/bin/gmx grompp -f gromacs.mdp -c conf.gro -r conf.gro -p topol.top ...
Fatal error:
Topology include file "min.ff/forcefield.itp" not found
The topology (a MINFF clay system):
#include "min.ff/forcefield.itp"
#include "Cu2_MMT1.itp"
#include "Cu2_MMT2.itp"
#include "Cu2_MMT3.itp"
#include "min.ff/ions.itp"
#include "min.ff/opc3.itp"
min.ff/ holds about 15 more files, which forcefield.itp includes in turn.
Cause: the GROMACS Inputs panel (src/wxgui/mdtools/GromacsInputsPanel.C) takes only what is attached by hand.
- Include files are attached one at a time ("Attach an include file").
- They are stored as auxiliary inputs by file name only, so a relative path such as
min.ff/is lost. - Nothing reads the topology's
#includelines, and the launch check only reports grompp's error.
Proposed fix
- On attaching a topology, follow its
#includedirectives recursively, relative to each including file. Honour#ifdefonly as far as is needed to find the files: collect every include in either branch. - Attach every file that exists, keeping its path relative to the topology (
min.ff/forcefield.itp), and recreate those subfolders in the run directory. - Skip includes that GROMACS resolves itself: its own force fields under
share/gromacs/toporGMXLIB, e.g.amber99sb-ildn.ff/forcefield.itp. - Show what was found: list the attached includes in the panel (count and tree), and the ones not found as "missing".
- Not ready to run until nothing is missing, naming the missing files, so the launch check is not the first place this shows up.
- Attach folder… as an alternative to attaching files one by one.
Test: a small topology with a local x.ff/forcefield.itp folder and a nested include, run through the launch check and the job script, like the restrained case in tests/gromacs_md (#253).
Workaround until then: use a preprocessed topology (gmx grompp ... -pp processed.top), which has every include expanded; it works since #253.
- Langage dominant
- C
- Étoiles
- 10
- Forks
- 8
- Merge moyen
- 21 min
- PR mergées (30 j)
- 1
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de FriendsofECCE/ECCE
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
FriendsofECCE/ECCE#336 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
FriendsofECCE/ECCE#339 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 28/100
FriendsofECCE/ECCE#338 · 2 commentaires ·
-
Basis Set Tool: periodic table cut off on the right; elements 110-118 still have placeholder namesOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 40/100
FriendsofECCE/ECCE#337 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 8/100
FriendsofECCE/ECCE#335 ·
Toutes les issues de FriendsofECCE/ECCE
Issues similaires
-
[Issue]: hipsparselt.h C API as shipped now requires C23 -- ONLY because of missing include or guardOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ROCm/rocm-libraries#13736 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 80/100
Les mainteneurs répondent en général sous 1 jour
-
`inverse()` of a controlled `CUGate` drops the phase parameter γPeut-être pris @rupasri118 l’a pris aujourd’hui. Ouvertebug mod: circuit
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Qiskit/qiskit#17223 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
OpenPrinting/libcupsfilters#284 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
libsdl-org/SDL#16530 ·
Les mainteneurs répondent en général sous 1 jour