matnwb icon indicating copy to clipboard operation
matnwb copied to clipboard

API generation tool is dangerously deprecrated

Open bendichter opened this issue 4 years ago • 8 comments
trafficstars

Hey @ln-vidrio, it looks like the API index is out of date. Would you be able to update it? How straightforward would it be to maintain old versions of the schema as well?

bendichter avatar Aug 18 '21 17:08 bendichter

The current tool I'm using no longer works with newer MATLAB versions so a better solution may be required. I can update the current API docs for 2.4.0. What do you mean by maintaining old versions of the schema or do you mean expanding the docs to all available versions?

lawrence-mbf avatar Aug 18 '21 17:08 lawrence-mbf

The current tool I'm using no longer works with newer MATLAB versions so a better solution may be required

Dang, that's a bummer.

expanding the docs to all available versions

This sounds like what I mean. Like with MATLAB you can select documentation from a previous version. Not a huge deal, but might be nice to have. I'm more interested getting the API docs updated.

bendichter avatar Aug 18 '21 17:08 bendichter

Do you think we could use something like this?

bendichter avatar Aug 18 '21 17:08 bendichter

Like with MATLAB you can select documentation from a previous version

So you can technically do this by linking to a specific git commit right now but I get what you mean. I can attempt something like this but that'll depend on what tools I'm using.

Do you think we could use something like this?

This is good but would require some automation work to get info.xml correct. It might be worth it though I'll have to think about it.

lawrence-mbf avatar Aug 18 '21 17:08 lawrence-mbf

Actually, looking at the link again, we do need static HTMLs for the pages on GitHub though I wonder how many users actually reference these API docs instead of locally.

lawrence-mbf avatar Aug 18 '21 18:08 lawrence-mbf

Well I came across this because I was trying to link to the API from the new icephys tutorial and realized that the new types are not in the docs. I'd be fine with changing it to a different form, and changing the URL would be OK, but it would be nice to have some static URL for the API that we can link to from the tutorials

bendichter avatar Aug 18 '21 18:08 bendichter

I've updated the API but I'll leave this issue open for the feature request.

lawrence-mbf avatar Aug 18 '21 18:08 lawrence-mbf

awesome, thanks! This is most of what I wanted. The rest was just exploring ways we could improve the docs.

bendichter avatar Aug 18 '21 18:08 bendichter