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.

Sign up for GitHub

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

Jump to bottom

Add documentation to tem_tutorial.ipynb for simSPI/path_config.yaml #131

Open
geoffwoollard opened this issue Jun 24, 2022 · 3 comments · May be fixed by #132
Open

Add documentation to tem_tutorial.ipynb for simSPI/path_config.yaml #131

geoffwoollard opened this issue Jun 24, 2022 · 3 comments · May be fixed by #132
Assignees
@geoffwoollard
Labels
documentation Improvements or additions to documentation

Comments

@geoffwoollard
Copy link
Contributor

geoffwoollard commented Jun 24, 2022 •
edited
Loading

Giving heads up about paths in simSPI/path_config.yaml that must exist to avoid errors for non-existing paths.

output_dir: './simSPI/temp_workspace/output/'

@ninamiolane I tried to push a new feature branch but got

remote: Permission to compSPI/simSPI.git denied to geoffwoollard.
fatal: unable to access 'https://github.com/compSPI/simSPI.git/': The requested URL returned error: 403
@geoffwoollard geoffwoollard added the documentation Improvements or additions to documentation label Jun 24, 2022
@geoffwoollard geoffwoollard self-assigned this Jun 24, 2022
@geoffwoollard
Copy link
Contributor Author

Did the permissions change in the last months? Should I push from my own fork rather than the comSPI organization's fork?

@ninamiolane
Copy link
Contributor

@geoffwoollard yes we have restricted access to the branches of compSPI/simSPI to enforce GitHub forks - as is common in public github repositories.

You have to:

  • fork the repo
  • push your branch there
  • open the PR

@geoffwoollard geoffwoollard linked a pull request Jun 24, 2022 that will close this issue
Open
8 tasks
@geoffwoollard
Copy link
Contributor Author

geoffwoollard commented Jun 24, 2022 •
edited
Loading

While simSPI/notebooks/path_config.yaml has

output_dir: './notebooks/'

simSPI/path_config.yaml does not

@geoffwoollard geoffwoollard reopened this Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoffwoollard geoffwoollard

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add documentation to tem_tutorial.ipynb for simSPI/path_config.yaml geoffwoollard/simSPI
2 participants
@geoffwoollard @ninamiolane