Comments (11)
related to the ongoing discussion over here: #40 (comment)
It would be good if the docs included a table with an overview of available atlases
from brainglobe-atlasapi.
Hey, do you want to open an issue in pydoc2md with suggestions for improving things?
from brainglobe-atlasapi.
Youp! I was waiting for it (writing docs) a bit as I guess that in these first times of development and testing on our side things might change fast and in my experience it's quite easy to forget outdated docs around.
I personally don't like much the sphinx/reST format and would prefer either Numpy or Google format + napoleon sphinx tool. It should be possible to convert the docs format from one version to the other using something like this although I have not tried!
But I have no strong opinion there and would comply to the sphinx/reST format if you like it more
from brainglobe-atlasapi.
I was thinking that writing docs could be more useful for development (so that other developers can better understand what things do).
I also don't have a strong opinion about the docstring format. My main reason is that amap/cellfinder etc use reST, but as long as it's one of the ones that PyCharm can do automatically, I don't mind.
from brainglobe-atlasapi.
I don't mind either, as long as I don't have to manually change brainrender's format. I did that a while ago to start using reST and it's not fun :P
from brainglobe-atlasapi.
I don't have much experience with this kind of documentation, but building it with sphinx from the docstrings seems like the easiest way
@adamltyson @vigji is this what we're going for in the end?
from brainglobe-atlasapi.
I think so. I think I would favour API docs automatically generated from the docstrings, in addition to a small amount of manually edited markdown illustrating general concepts, and any user-facing command-line tools.
from brainglobe-atlasapi.
any news on that front? I am reviewing for JOSS and that is the only problem I see. At least the docstrings of the BrainGlobeAtlas
class should be exposed in the documentation (although help(BrainGlobeAtlas)
does the job).
On a more minor note, one item for the review is:
Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support
It is quite implicit as the code is on github, but perhaps you can add a small paragraph to the README so I can check off the item.
from brainglobe-atlasapi.
Hi,
I've added a short paragraph to the README (on the main branch) and to the docs with very simple guidelines for contributing.
Regarding the docstrings, I have generate docs like these for brainrender and added them to the docs online: https://docs.brainglobe.info/autogenerate-docs/bg_atlas/core
Hope this is what you had in mind, but please do let use know if there is a better way to document this.
from brainglobe-atlasapi.
Great, thanks!
from brainglobe-atlasapi.
I guess we can close this. Long term there's some aesthetical improvements that can happen on the markdown generated docs but that is minor
from brainglobe-atlasapi.
Related Issues (20)
- [Feature] Add Ambystoma mexicanum (axolotl) atlas
- Decide what to do with `atlas_generation/main_script` HOT 1
- deal with local and remote atlas versions in validate_atlases.py HOT 9
- validate_atlases.py additional_references error HOT 5
- Add updated version of the kim atlas HOT 1
- [Feature] flexibly generate atlases with different resolutions
- perens_atlas alignment trouble with new version [BUG] HOT 4
- perens_lsfm_mouse_20um_v1.1 is oriented incorrectly HOT 1
- Use pooch to download data
- User is informed (twice) that the atlas is not the latest when they're updating it
- The warning that an atlas is not the latest is missing a space
- Standardise the axial resolution of the allen spinal cord atlas HOT 1
- Save atlases to `.brainglobe/atlases`
- Atlases should be returned by `brainglobe list` in alphabetical order
- Enforce standardised orientation
- Atlas resolution as an attribute, not a name.
- Move atlas generation code out of the `brainglobe-atlasapi` package
- Ensure all tests are run
- Ensure tests do not affect local system when run locally.
- [BUG] Kim developmental atlases fail validation
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from brainglobe-atlasapi.