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.

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

docs oddities #113

Closed
CarloLucibello opened this issue Apr 28, 2022 · 2 comments
Closed

docs oddities #113

CarloLucibello opened this issue Apr 28, 2022 · 2 comments

Comments

@CarloLucibello
Copy link
Member

CarloLucibello commented Apr 28, 2022

At the very top of this doc page https://fluxml.ai/FluxTraining.jl/dev/i/?id=documents%2Fdocs%2Fcallbacks%2Fusage.md

using FluxTraining
using FluxTraining: Callback, Read, Write, stateaccess
model, data, lossfn = nothing, (), nothing, nothing

Is that intended?

Also, if from that page I follow a couple of links and land to
https://fluxml.ai/FluxTraining.jl/dev/i/?id=documents%2FREADME.md&id=documents%2Fdocs%2Fcallbacks%2Fcustom.md&id=documents%2Fdocs%2Ftutorials%2Ftraining.md
but then i don't see any buttons for closing all those panes or for going back. Even the browser back button doesn't have any effect.

One last thing is that I don't see a link when visualizing the docstring of types/methods to jump to the source code.

@lorenzoh
Copy link
Member

Thanks for reporting! These are known issues in Pollen that I have yet to fix

  • For the first code block (I assume this is from https://fluxml.ai/FluxTraining.jl/dev/i/?id=documents%2Fdocs%2Fcallbacks%2Fusage.md), this is supposed to be hidden.
  • You can close panes by hovering over the tab list in the sidebar, but I plan to add a close button on each individual tab
  • History is also only one-way right now, in that navigating by clicking updates the history and url, but not the other way around
  • Links to source files are broken only on GitHub Actions where the packages aren't devved

I'll leave this open until I've fixed these things 👍

@lorenzoh
Copy link
Member

Fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants