-
Notifications
You must be signed in to change notification settings - Fork 0
#117 Change how trees are assigned into PFTs #123
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
base: main
Are you sure you want to change the base?
Conversation
Hi @davidorme, it's the first time pushing a commit from a new laptop, I think I set up everything except for the pre-commit tool. Can you please remind me how this is done or where I can read how to do this? Note to self: |
for more information, see https://pre-commit.ci
Hi @davidorme, I'm stuck here, do you have any suggestions on how to fix this? |
@arne-exe We also have the https://imperialcollegelondon.github.io/ve_data_science/getting_started/ Although that page is a little out of date and doesn't say anything much different from the VE site. We do need to improve this though - some people are relying on the GitHub CI to catch On this specific issue... I'm not sure but I think the first thing to do is to update the |
@arne-exe that PR is now in. Try switching to |
Hi @davidorme, thanks for looking into this. I still can't get it to work, even after the following:
I've attached a copy of the error below:
|
Sorry for budging in this PR, but Arne and I found out that the Getting Started from the Same for the link to GitHub overview I have created a PR to change the links. See PR #126 |
This PR will provide a set of scripts to implement a new way to assign trees into PFTs: