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/pile up app #4

Closed
wants to merge 13 commits into from
Closed

Conversation

kmtsui
Copy link

@kmtsui kmtsui commented Nov 9, 2023

Make the pile-up generation application be compatible under the new structure.
Also save the raw Cherenkov hits in output which are missing before.

@pdeperio
Copy link
Collaborator

@guiguem any idea about this CI fail?

@kmtsui
Copy link
Author

kmtsui commented Nov 21, 2023

@guiguem any idea about this CI fail?

The error seems to be related to secret setup.

Error: The ssh-private-key argument is empty. Maybe the secret has not been configured, or you are using a wrong secret name in your workflow file.

@guiguem
Copy link

guiguem commented Nov 22, 2023

I believe the issue is that you are working from your own branch, therefore you cannot use the hyperk/mdt secrets were the ssh key was setup.
you would need to set your own ssh key in your repository or make the branch appear in the hyperk repo?

@kmtsui
Copy link
Author

kmtsui commented Nov 22, 2023

I don't have write access in the hyperk/MDT repo. Can you grant me the right?

@guiguem
Copy link

guiguem commented Nov 27, 2023

Sorry, I didn't get notified... Can you check now?

@kmtsui
Copy link
Author

kmtsui commented Nov 28, 2023

Move to PR #5

@kmtsui kmtsui closed this Nov 28, 2023
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.

3 participants