Closed marcelstoer closed 8 years ago
I've never used Read The Docs, but as I've mentioned elsewhere I'm all in favour of switching to something like this. I had a look through the docs but I didn't see much in terms of what mark-up/down/sideways could/would/should be used. I'm familiar with JavaDoc / Doxygen syntax which works pretty well for documenting code. For user-facing docs it's probably not as appropriate (though the easy API cross-referencing is most handy).
Anyway, I guess what I'm trying to say is that I'd be happy to give anything a go as long as it seems reasonable, and especially if you have experience working with this particular flavour and can set a good example for the rest of us :)
I had a look through the docs but I didn't see much in terms of what mark-up/down/sideways could/would/should be used.
It supports reStructuredText and Markdown.
+1
El mié., 18 de nov. de 2015 a la(s) 8:39 a. m., Marcel Stör < notifications@github.com> escribió:
I had a look through the docs but I didn't see much in terms of what mark-up/down/sideways could/would/should be used.
It supports reStructuredText and Markdown.
— Reply to this email directly or view it on GitHub https://github.com/nodemcu/nodemcu-firmware/issues/774#issuecomment-157685647 .
+2
I've never come across Read The Docs before but I have used and am familiar with Doxygen both for end-user and programming documentation, so it would get my vote simply on the grounds that it is one less system to learn, but what the heck if someone sets up a usable system then I'll use it. eLua uses another system -- can't recall which off the top of my head.
I just found out that the SmartArduino/DOIT/MXCHIP.INC folks do a pretty good job with the introductory documentation for WiFiMCU at https://www.gitbook.com/book/fineshang/wifimcu-based-on-emw3165-user-manual/details. As you can see they're on GitBook which is veeery similar to RTD.
Others are in a similar situation as we are: https://civicrm.org/node/3268, http://j.mp/1YqisPD, http://wp.me/p2YeUz-1sG.
Gitbook looks as viable as RTD to my untrained eyes.
What I really meant to ask in my previous comment was about schemas and/or templates. With e.g. Doxygen you get a bunch of semantic keywords/commands (\param, \returns, \see, etc) which not only have associated styling, but can also do Useful Things(tm) such as automatic cross-linking to other pages. In terms of templates, I'm thinking along the pre-ordained each-function-shalt-have-intro-and-param-plus-return-descriptions-at-minimum, or what we currently have on the wiki for example. Whenever you start documenting a new function you have a clear template of what must be included at minimum.
Would we get anything along those lines with either of these approaches? Both seem to want to me sign up before they tell me any particulars grumble. Not a deal-killer if they don't offer these things, but anything that makes it easy to write good, consistent documentation is highly valued in my opinion.
Johny, +1 to your comment, but we are of similar generation in terms of our development experience. The young guys might disagree. My feeling is that there are many types of documentation, and its hard to do one cap fits all. Where things like Doxygen is particularly strong is documenting code. What we are talking about here is
The first of these needs to be locked to the source IMO, so that if you change the API then you change the documentation for it in the same file. But again that's just my POV.
I'm quite happy to let Marcel pick something he thinks is a good fit, and point me at instructions on how to use it. In the meantime I'm just throwing my $0.02 pieces into the thread.
We have a saying in Sweden which translates as "Much chatter, no hockey", meaning to just get on with the action and not worry about whether the call was the best or not. As long as we're doing things which are better than they used to be, we're moving in the right direction. Let's not overthink things too much.
Oh, and the L10n support of RTD seemed very applicable to this project, and I don't see that listed on GitBooks.
I mentioned to use RTD as the doc to the original nodemcu developer like a year ago, and I tried it out myself and created https://readthedocs.org/projects/nodemcu-firmware/ which connected to my own fork. But sense than I haven't updated it. @marcelstoer emailed me today and I deleted my RTD project so that it can be pointing to this repo.
I think RTD is a very good tool to maintain a readable and user friendly document and I'll help if the decision is made to move the doc to RTD :)
Since I promised I'd spend some of my leave working on the docs, I took the liberty of setting up a new (temporary) branch here called newdocs
. This branch contains @marcelstoer 's work on getting us up and going with RTD, and I've transferred/added documentation for ~10 modules so far. I've been focusing on the ones which I actually know something about, so that I've been able to add a little bit of useful information here and there while copying/transcribing.
RTD itself is still configured to look at Marcel's repo, so there'll be nothing visible there just yet.
Unless anyone objects to the approach of getting the docs prepped in a sidebranch, I'd invite people to join in the fun of getting the docs across ;P
setting up a new (temporary) branch here called
newdocs
You beat me to it, thanks. I had planned that for tonight. I rushed to remove it from the cloud builder.
RTD itself is still configured to look at Marcel's repo
I'll fix that in the next hours.
Unless anyone objects to the approach of getting the docs prepped in a sidebranch
Would surprise me as I had proposed this a month ago and no one has objected since :wink:
I spent some time in the past days on a consistent and cohesive presentation of the content. I just committed some tweaks to the theme and an overhaul of the node, adc, and bit module docs. Here's a list of issues I found and what I changed:
grep '## ' xxx.md | sort
):
after parameter name.
at end of parameter and return value description if complete sentence (hardly ever)nil
consistenlynil
, this makes a lot more sense to me but I wonder how that feels for English native speakersIf you want to compare before/after please compare node, adc, and bit with the other modules.
I'll continue updating the others for consistency unless there's objection from the collaborators.
Ack with all your points except
if there's an "Example" (see node.md) then I removed the "Syntax" because it's totally superfluous
I'd keep the "Syntax" for formal reasons. Grammar aspects like nesting and optional parameters should be shown here in an exhaustive way. IMO better remove "Example" in case it's a trivial one.
Thanks for checking.
Grammar aspects like nesting and optional parameters should be shown here in an exhaustive way. IMO better remove "Example" in case it's a trivial one.
Ok, understood. Makes sense.
Other than what @devsaurus already pointed out, sounds good to me!
I'm on the fence when it comes to trivial examples. One the one hand they're largely redundant, but on the other hand they might be nice for someone new to NodeMCU. Multiple-return (e.g. foo, _, bar = baz()
) examples should always stay imho - many languages don't support them so usage might not be obvious otherwise.
I'm not sure I'll have any more time this week to transfer docs, but I'll see how I go. More hands appreciated, wink-wink-nudge-nugde ;)
Is it OK if I hijack the Module origin table and add a column "Transfer status" to it? We'd see what's TBD and also if someone else is currently busy with a module. Just to avoid merge conflicts / double effort while the big chunks are being moved. The column should be deleted afterwards, it's probably not useful to capture ongoing maintenance.
This is looking really good, but just looking at your example link, I really need to add another issue which is to do a complete API review of return statuses. If a call can errors, then it should either throw an error or return an error status. In the former case the documentation should say that it throws an error on error. (This struck me when reading node.compile()
which just says "returns nil".)
Is it OK if I hijack the Module origin table and add a column "Transfer status" to it?
+1 if you call it "Doc transfer status" :wink:. Have been thinking about that last night as well.
if you call it "Doc transfer status"
Done.
Apparently, we're missing API docs for a couple of modules (while some come with a README.md example at least):
Are the contributors still planning to provide an API description? Either in the wiki nodemcu_api_en or - preferably - as a new file in newdocs branch.
HX711 docs written. Pull req https://github.com/nodemcu/nodemcu-firmware/pull/919
Thanks @devsaurus, I will do the coap docs ASAP when I get some time.
@christakahashi @nodemcu thanks a lot!
Only 4 more modules to transfer, way to go :wink:
Are the contributors still planning to provide an API description?
At some point (how about now?) we might wanna make this a prerequisite for "accepting" a module.
Looking at what's outstanding to be transferred into RTD, I would suggest merging in newdocs
once we have the wifi
docs in there. The others can straggle in later without too much issue in my opinion.
And yes! No modules without API docs in the future! But let's put the barrier after we've gotten newdocs into dev, and cleared the PR log. I'm happy* to do the docs for any modules I merge in (I'm eyeing off the http client module next).
*) Happy not being the actual feeling, but I'd do it for the good of the project ;)
I would suggest merging in
newdocs
once we have thewifi
docs in there.
What about the FAQ? @TerryE will you manage to clean this up say within the next week?
What about the FAQ? Will you manage to clean this up say within the next week?
The honest answer is no. I think it really needs a fundamental restructuring into topic areas and I need a couple of hard reviewers / contributors who would be willing to work with me on this. @pjsg Philip, @nickandrew Nick are you up for this?
I think that a 3-4 week timescale is a more realistic target. Another issue is that I have limited time available, and I would like to break the back of this interrupts restructuring first as this directly impacts the stability of the firmware.
Shall we then keep the FAQ in newdocs
and just iron out Markdown issues (without changing contents) or would you prefer to have it removed for the time being?
I'm unbiased & happy with either solution. Just let me know and I'll do the window dressing while Marcel's busy with wifi.
My vote is to keep it because I'd like to have single source we can link to if we send people to RTFM.
BMP085 docs done #936
Thanks @kbeckmann!
I can certainly help with the FAQ. I get short periods of time when I can work on stuff at home. My day job is keeping me rather busy at the moment....
I used to have that sort of problem. So I understand. Now I am retired, but an a bit occupied with building a house instead! It's mainly constructive feedback and comments that I will need -- your chance to get your own back now that our roles are reversed :wink:
I realise that it would add some work, but it would really enhance the readability of each documentation section if it were prefixed by a sort table of contents for example in the case of node, this would be as follows (note that the first column would be a MD anchor link to the relevant section of the documentation):
Method / Constant | Short Description |
---|---|
node.bootreason() | Returns the boot reason and extended reset info |
node.chipid() | Returns the ESP chip ID |
node.compile() | Compiles a Lua text file into Lua bytecode, and saves it as .lc file |
node.dsleep() | Enters deep sleep mode, wakes up when timed out |
node.flashid() | Returns the flash chip ID |
node.heap() | Returns the current available heap size in bytes |
node.info() | Returns NodeMCU version, chipid, flashid, flash size, flash mode, flash speed |
node.input() | Submits a string to the Lua interpreter. Similar to pcall(loadstring(str)) , but without the single-line limitation |
node.key() --deprecated | Defines action to take on button press (on the old devkit 0.9), button connected to GPIO 16 |
node.led() --deprecated | Sets the on/off time for the LED (on the old devkit 0.9), with the LED connected to GPIO16, multiplexed with [node.key() ] |
node.output() | Redirects the Lua interpreter output to a callback function. Optionally also prints it to the serial console |
node.readvdd33() --deprecated | Moved to adc.readvdd33() |
node.restart() | Restarts the chip |
node.restore() | Restores system configuration to defaults. Erases all stored WiFi settings, and resets the "esp init data" to the defaults. This function is intended as a last-resort without having to reflash the ESP altogether |
node.setcpufreq() | Change the working CPU Frequency |
node.stripdebug() | Controls the amount of debug information kept during node.compile() , and allows removal of debug information from already compiled Lua code |
We could also drop the rotable prefix, e.g. node.
in this case. I also think that tabling the inline entries will significantly compact the layout and make it more readable, e.g
node.compile() | Compiles a Lua text file into Lua bytecode, and saves it as .lc file. |
---|---|
Syntax | node.compile("file.lua") |
Parameters | filename name of Lua text file |
Returns | nil |
Example |
file.open("hello.lua","w+")
file.writeline([[print("hello nodemcu")]])
file.writeline([[print(node.heap())]])
file.close()
node.compile("hello.lua")
dofile("hello.lua")
dofile("hello.lc")
...enhance the readability of each documentation section if it were prefixed by a sort table of contents...
True, but that should be generated rather than manually maintained. I don't think MkDocs has support or extensions for that but I'll spend some time to find alternatives.
I also think that tabling the inline entries will significantly compact the layout and make it more readable.
Terry, where were you when we discussed the new layout a month ago :wink:. Except for 1-wire everything has since been transferred and corrected (see status on Module-origin). Hell of a task... Anyone is welcome to change it again but I'm done with it.
This is not to say that your proposal didn't have its charm. You might even be able to batch-convert all .md
files with a script since the structure should be pretty consistent now.
I don't think MkDocs has support or extensions for that but I'll spend some time to find alternatives.
Turned out to be a correct assumption. Since we can't directly tinker with the static doc generation process I resorted once again to JavaScript. The TOC is now generated on each API doc page on the fly in the browser (you might have to hit shift-reload to get the updated .js file).
Marcel, I was up to eyes in my bloody house.
We can only do stepwise improvement. Yes, your exercise is a major improvement, but I made the mistake of using the Github viewer rather than RTD. The RTD TOC is close enough, and and inline TOC embedding a short description is difficult to generate inline automatically. Also the RTD CSS is far better than the default MD one, so the RTD view is a major step forward compared to the Github MD renderer. Hwever at the moment, the only way that you can see this is online. What would be brilliant is a mechanism for dumping out the entire documentation to CHTML so that the developer always has this reference to hand.
Marcel, I've just looked at your latest version of the RTD documentation. I think that this is really good. Thanks for your work
@TerryE
Shall we then keep the FAQ in
newdocs
... or would you prefer to have it removed for the time being?
That's the last gating topic before merging newdocs
into dev
. IMO #937 is a kind of :-1:, isn't it?
@devsaurus Arnim, I don't understand your thumbs down. Documentation can always be improved. My current unofficial FAQ contains a lot of extremely useful information. I think that what you've all done is a major improvement in the documentation base, and I don't really want to hold up the merge, so we could include the current FAQ. It's only a few hours work to convert it from dokuwiki markup to github MD, and Marcel has already done this as at 15 Dec to create the current faq.md. So my vote is :+1:
What my #937 is really about is an admission that there are a couple of points where (after months of working and developing with the firmware and Lua) I now think that my advice is wrong, in particular my advice to avoid using upvals to store context, and to avoid coroutining. I now understand how the Lua core, the nodemcu additions and the SDK all interact and these are solid, so it is sensible to use these features in the appropriate use case.
There are also few area where I would restructure the advice to make it more readily understandable to programmers new to Lua and the esp8266, and there a quite a few extra topics that I'd like to add.
In hoisting this to an official FAQ, what I suggest is an approach a bit like a lot of release strategies work. We have a fairly dynamic "living" FAQ that we can work on collaboratively say in a wiki, but that we snapshot stable updates by my doing a PR to merge these into the dev into every 3-6 months, or after any major improvements. #937 is just an issue to collaborate on the next "catch-up".
Hey @marcelstoer I like the TOC at the top, but the net module's Constants header shows up a bit strange. Could you have a look please?
@TerryE where would said HTML bundle go/come from? It shouldn't be part of the repo. It could potentially be built during CI, but then where would it go? Should the cloud builder include a link to said bundle? Should it be pushed as a release back to the github repo? Should the bundle only be a download link via RTD, for whatever is shown on that RTD branch?
Also @TerryE some of the links in the faq.md don't seem to show up properly on RTD.
Also @TerryE some of the links in the faq.md don't seem to show up properly on RTD.
That's what I plan to fix right now. #937 was holding me back from spending the effort - thus my earlier request. Terry clarified this, so way to go... :grin:.
eLua generates its CHTML as part of the build, but I think that doing this is a pain since the build than requires a whole stack of Latex-style authoring components. I think that a pragmatic alternative is to have this HTML bundle as some form of downloadable resource, just like the sdk zip and the open-sdk tarball used in the Travis CI builds. Maybe include the link to this in the Docs header or useful links. That way developers who want to access the documentation on the laptop when out of network range can download the bundle and access it locally.
Arnim, if you do make any changes or corrections to the faq.md, then I'll diff these to track them and backload them into working version, so we don't repeat this issue on the next update. Thanks.
I see that RTD has some sort of Downloads area - I wonder if we could drop the per-branch documentation there. I'd say that's for later down the track though - in the meantime you can just grab the repo and fire up your favourite markdown editor to peruse the docs locally.
Terry, the links are now ported to Markdown syntax - no changes to the actual document contents. Someone will have to go through it again when importing updates as long as the FAQ master is maintained in a different ecosystem.
Marcel, no blocking points from my side anymore for merging into dev
.
Incidentally Arnim, what I feel that you need for collaborating and working on a markup syntax is (i) the ability to work on an individual section and (ii) to have an immediate preview before commit, (iii) preferably a wysiwyg editor. The github viewer/editor suck at all of these, and it also lacks some of the basic markup features that I'd really like to use such as indented paragraphs. But I guess that I'll just have to live with this. WP markup is a lot more flexible, IMO.
Whatever the markup syntax used for collaborating, I'll end up writing or porting some hack perl to do the conversion to md. That's what I've done in the past. I've used Perl for this type of work for 25 years so its gettting a bit late to change old habits :laughing: but don't work this quick & dirty stuff won't go near this repo.
My $0.02 to the "HTML bundle" discussion:
.md
files with the editor of your choice.A few minutes ago a put the finishing touches on the new docs. My plan for the immediate next steps is as follows:
newdocs
back to dev
Sunday night CET and remove itMy $0.02 to the "HTML bundle" discussion:
- IMO just a nice-to-have goodie, aren't developers always online? If you really really need it offline you can clone the repo (in advance...) and look at the
.md
files with the editor of your choice.
No. I don't think that this is a valid assumption, yet.
- Should definitely not be part of our repository
No more than any derived product like object libraries or binaries. But there is a stronger argument for saying that the scripts to create them should be.
No. I don't think that this is a valid assumption, yet.
I'm not sure if you're speaking more philosophically here or are actually arguing against "releasing" newdocs until we also provide an offline bundle. If the former I agree, but if the latter I strongly disagree. newdocs
is a huge improvement over the existing situation (which also doesn't provide offline access without cloning the separate wiki repo). The wiki API docs are incomplete and out-of-date, and we're doing everyone a disservice the longer we have them around.
Since it also seems likely that someone else will provide the necessary glue for also getting a PDF generated, I'm in favour of waiting for that and then getting that into the RTD Upload section via CI. Of course, if you want to steam ahead and provide patches I'm not going to stop you, but I'd rather you keep focus on the ISR/task cleanup for now.
@marcelstoer :+1: to your points above
First, this isn't a philosophical point. It's a pragmatic one. We shouldn't assume that every developer has 24×7 Internet access. Second this shouldn't delay anything. As I've already said a few times, as you have just done, the new documentation is a material step forward. This is just a tidy up flag for a future todo and a low priority one at that.
I want to drop the Wiki documentation and move the content to the repository so we can serve it with Read The Docs. See http://read-the-docs.readthedocs.org/en/latest/features.html for details. This would solve two major pain points:
Documentation not in line with the code
If the documentation is moved to the code repository we can verify with every commit/PR that the documentation is updated accordingly. Furthermore, the documentation is merged automatically with the code if it moves from branch X to Y. The human-readable docs are built automatically with every commit.
L10n
Read The Docs (and similar solutions) offer L10n to easily serve the documentation in multiple languages. This would replace the inconsistent Wiki pages for non-EN languages.