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

Attribute injection needs to be smarter #24

Open
jbusecke opened this issue Apr 13, 2024 · 0 comments
Open

Attribute injection needs to be smarter #24

jbusecke opened this issue Apr 13, 2024 · 0 comments

Comments

@jbusecke
Copy link
Contributor

I am actually pretty happy I got this to work: https://github.com/leap-stc/proto_feedstock/blob/d3d198b496e42a7ca3702e1b34a9c3426220575d/feedstock/recipe.py#L68-L101

But it causes all kinds of trouble in the local execution context (which I am now testing in #23 and linked there). Unless we can make that work in both contexts I think we should not package it in this template. Maybe something that we should publish as a self contained stage in the data-management-utils repo (also pinging @norlandrhagen here whos work might have overlapped with this at some point?)

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

No branches or pull requests

1 participant