Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merge executable fields in config #123

Open
JavierCladellas opened this issue Nov 15, 2024 · 0 comments · May be fixed by #133
Open

Merge executable fields in config #123

JavierCladellas opened this issue Nov 15, 2024 · 0 comments · May be fixed by #133
Assignees
@JavierCladellas
Labels
enhancement A new feature or request i/o Issues related to input/output handling

Comments

@JavierCladellas
Copy link
Collaborator

Currently, users must specify "executable_dir" and "executable" if they want to provide an executable in a specific directory.
Merge these two fields by using os.path.basename to extract the executable name.
WARNING: Might need to parse the filename if it contains an extension (like for python scripts)
@JavierCladellas JavierCladellas added enhancement A new feature or request i/o Issues related to input/output handling labels Nov 15, 2024
@JavierCladellas JavierCladellas self-assigned this Nov 15, 2024
JavierCladellas added a commit that referenced this issue Nov 15, 2024
@JavierCladellas
rm executable_dir #123
a304df1
JavierCladellas added a commit that referenced this issue Nov 15, 2024
@JavierCladellas
use only base of "executable" for naming. Also add the use_case level…
a964dd9 
… to the output hierrarchy #123
@JavierCladellas JavierCladellas linked a pull request Nov 15, 2024 that will close this issue
Open
@JavierCladellas JavierCladellas linked a pull request Nov 15, 2024 that will close this issue
123 merge executable fields in config #133
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JavierCladellas JavierCladellas

Labels
enhancement A new feature or request i/o Issues related to input/output handling
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

123 merge executable fields in config feelpp/benchmarking
1 participant
@JavierCladellas