Skip to content
New issue

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

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

Already on GitHub? # to your account

[ENH]: Specify which shell to use when using junifer queue #273

Merged
merged 10 commits into from
Apr 9, 2024

Conversation

synchon
Copy link
Member

@synchon synchon commented Mar 20, 2024

Are you requiring a new dataset or marker?

  • I understand this is not a marker or dataset request

Which feature do you want to include?

I would like to be able to specify the shell that junifer uses to submit jobs. For some reason when using bash I could not get jobs to work. When using zsh by manually changing it, it seemed to work (maybe something from with my bash environment, maybe magic or maybe coincidence). Either way I think this could be a desirable feature. What do you think?

How do you imagine this integrated in junifer?

as a param that prints different text

Do you have a sample code that implements this outside of junifer?

No response

Anything else to say?

No response

@LeSasse LeSasse added enhancement New feature or request triage New issues waiting to be reviewed labels Nov 2, 2023
@synchon
Copy link
Member

synchon commented Nov 2, 2023

This could be quite useful in my opinion.

@synchon synchon requested review from fraimondo and LeSasse March 20, 2024 13:49
@synchon synchon removed the triage New issues waiting to be reviewed label Mar 20, 2024
@synchon
Copy link
Member

synchon commented Mar 20, 2024

@LeSasse Can you check if the branch solves your issue?

@synchon synchon changed the title [ENH]: Specify which shell to use when preparing condor submit files [ENH]: Specify which shell to use when preparing HTCondor submit files Mar 20, 2024
Copy link

github-actions bot commented Mar 20, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-04-09 12:13 UTC

Copy link

codecov bot commented Mar 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (b0c0a61) to head (de69448).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #273   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            1         1           
=========================================
  Hits             1         1           
Flag Coverage Δ
docs 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@LeSasse
Copy link
Collaborator Author

LeSasse commented Mar 21, 2024

@LeSasse Can you check if the branch solves your issue?

Will check it

@synchon synchon requested a review from LeSasse March 21, 2024 08:13
@synchon synchon added this to the 0.0.4 (alpha 3) milestone Mar 21, 2024
@synchon synchon force-pushed the feat/queue-shell branch 2 times, most recently from ff54f47 to a6d0bb8 Compare March 26, 2024 11:20
@synchon synchon force-pushed the feat/queue-shell branch 2 times, most recently from ec24b7a to 8600679 Compare April 4, 2024 10:20
@LeSasse
Copy link
Collaborator Author

LeSasse commented Apr 5, 2024

@LeSasse Can you check if the branch solves your issue?

I think this branch needs a rebase: the dag files still use single quotes and submitting therefore fails.

@synchon synchon force-pushed the feat/queue-shell branch from 8600679 to 902aa4e Compare April 5, 2024 10:05
@synchon
Copy link
Member

synchon commented Apr 5, 2024

@LeSasse Can you check if the branch solves your issue?

I think this branch needs a rebase: the dag files still use single quotes and submitting therefore fails.

I see that the quotes are proper in dag, can you try rebasing your local branch?

@LeSasse
Copy link
Collaborator Author

LeSasse commented Apr 5, 2024

@LeSasse Can you check if the branch solves your issue?

I think this branch needs a rebase: the dag files still use single quotes and submitting therefore fails.

I see that the quotes are proper in dag, can you try rebasing your local branch?

thats quite odd because i actually cloned and checked out this branch just minutes before that comment, and after i rebased it locally it worked, but before, no.

Ah no, this was supposed to go on the other PR #161 so thats the one that would need rebasing

@synchon synchon force-pushed the feat/queue-shell branch from 902aa4e to bedb8f7 Compare April 8, 2024 10:23
@synchon synchon changed the title [ENH]: Specify which shell to use when preparing HTCondor submit files [ENH]: Specify which shell to use when using junifer queue Apr 8, 2024
@LeSasse
Copy link
Collaborator Author

LeSasse commented Apr 9, 2024

Tested this now and it works.

@synchon synchon merged commit c36165b into main Apr 9, 2024
20 of 23 checks passed
@synchon synchon deleted the feat/queue-shell branch April 9, 2024 11:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants