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

Feature initial staging #138

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Feature initial staging #138

wants to merge 5 commits into from

Conversation

g-walley
Copy link
Owner

@g-walley g-walley commented Aug 1, 2023

No description provided.

@JackStorrorCarter
Copy link
Collaborator

Noticed a small bug but I think I've lost my access to fix it! Could you change the line
def _validate_hyperstage(initial_staging, hyperstage):
to
def _validate_hyperstage(self,initial_staging, hyperstage):

@JackStorrorCarter
Copy link
Collaborator

Also I have found that by using the initial stage along with the hyperstage arguments, you are able to make any ST you like (not restricted to the AHC output). A nice little trick it might be worth pointing out in the documentation so that people can make completely custom stagings

@g-walley
Copy link
Owner Author

Noticed a small bug but I think I've lost my access to fix it! Could you change the line def _validate_hyperstage(initial_staging, hyperstage): to def _validate_hyperstage(self,initial_staging, hyperstage):

You should have no problem editing it, you are listed as a collaborator on the project.

@g-walley
Copy link
Owner Author

Also I have found that by using the initial stage along with the hyperstage arguments, you are able to make any ST you like (not restricted to the AHC output). A nice little trick it might be worth pointing out in the documentation so that people can make completely custom stagings

Ok, I can have a look at doing that :)

Bug fix adding self to definition of validate_hyperstage
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.

None yet

2 participants