-
Notifications
You must be signed in to change notification settings - Fork 1
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
Documentation read through #219
Open
raphaelshirley
wants to merge
50
commits into
main
Choose a base branch
from
issue/216/documentation-read-through
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #219 +/- ##
==========================================
- Coverage 71.93% 71.93% -0.01%
==========================================
Files 50 50
Lines 6432 6435 +3
Branches 937 937
==========================================
+ Hits 4627 4629 +2
- Misses 1805 1806 +1 ☔ View full report in Codecov by Sentry. |
There was a repeated line with a typo which did not do anything so I removed it.
Following testing by Olivier we realised it was easy to have a notebook kernel that was not inside the correct environment. We decided that this was a common enough mistake that we should add explicit instructions tot eh documentation
Since including the pybind11 code using the MANIFEST.in file I could install on my mac from source using pip. I am concerned that this wouldn't work if I did not already have compilers and libraries present so I recommned people follow the developer install instructions which install the required libraries if there is no binary available
A number of changes suggested in the PR. Some section renames and addition of extra lines to code snippets for clarity
…f version into the rtf
…/216/documentation-read-through
Some of the change requests were ignored as they are required for the notebook to run. I have so far left in the onesource examples at the end as they were previously requested. We can discuss if to remove them this week. The issue with curl to download the para file was that it needed the url updating to the raw file not the html. The example para file does not run through the downloader because it needs the sed list links to be relative. We can either update the para file or the downloader to append these links
He requested a couple of changes which will increase run time. We need to check that all notebooks still complete in time for the documentation. I also added a simple z-z plot
There are various small changes and cleaning things up. We have to set up the environment variables but given the new structure a user may also need a further environment variable to reference the executables but I will not mention that for now for simplicity.
Mainly adding git clone code for clarity following Olivier sugestion
I also added a custom css to make the tables wrap so be easier to read Many changes to keyword descriptions including removing the defunct keywords
Mainly installation instructions
We have moved the developer install to a new section and reordered gettign started
We wanted to be more informative regarding the contruction of the input tables which have a strict format
simplified and retured the standard filters and removed low level functionality section
We have run through the first main notebook examples and called other developer examples
calzetti.dat not present so changed to SB_calzetti.dat
These are from changes to process. I need to check the notebooks are still working
Also deleted the seperate notebooks page as it is the core of the Python interface page which now has all teh notebooks
… issue/216/documentation-read-through
When this documentation is merged we will define v0.2.0 and deprecate the GitLab version
hub is requireing a custom user now to prevent getting spammed. see readthedocs/readthedocs.org#11763
It seems for docs to build we need to add a user but this broke the test so I have removed the test to see if it will tehn build
I had to send a specific downloader to check the call
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We are working through installing according to the documentation and making changes as we find them.
Closes #216
Code Quality
Project-Specific Pull Request Checklists
Bug Fix Checklist
New Feature Checklist
Documentation Change Checklist
Build/CI Change Checklist
Other Change Checklist