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.

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

Make create_sup_file() actually use the argument simname (fix issue #21) #22

Merged
merged 4 commits into from
Dec 4, 2024

Conversation

ohexel
Copy link

@ohexel ohexel commented Nov 26, 2024

Replace hard-coded sup.fn <- "socsim.sup" with logic that uses simname as the file name or at least its stub. Inform user of file name and full path via print statement before returning the file name.

Replace hard-coded `sup.fn <- "socsim.sup"` with logic that uses
simname as the file name or at least its stub. Inform user of file
name and full path via print statement before returning the file
name.
@ohexel ohexel changed the title Make create_sup_file() actually use the argument simname (fix issue #10) Make create_sup_file() actually use the argument simname (fix issue #21) Nov 26, 2024
@ohexel ohexel changed the title Make create_sup_file() actually use the argument simname (fix issue #21) Make create_sup_file() actually use the argument simname (fix issue #21) Nov 26, 2024
@ohexel
Copy link
Author

ohexel commented Nov 28, 2024

Added a default simname to avoid errors when no simname is supplied by user. Simplified ifelse(grepl()) into standard if statement.

@tomthe
Copy link
Member

tomthe commented Dec 4, 2024

very good. But now it has merge conflicts that have to be resolved.

@ohexel
Copy link
Author

ohexel commented Dec 4, 2024

Fixed?

@tomthe tomthe merged commit 0421ebf into MPIDR:main Dec 4, 2024
1 check passed
@tomthe
Copy link
Member

tomthe commented Dec 4, 2024

yes, fixed.

@ohexel ohexel deleted the fix-issue-21 branch December 4, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants