Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
ggprior
force-pushed
the
docs/update-auth-flow-ungating
branch
from
b08c6c5 to
9a4f14e
Compare
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the documentation to include TabPFN-2.6 and replaces the previous Hugging Face-based authentication flow with a new browser-based login system and the use of the TABPFN_TOKEN environment variable. Feedback was provided regarding a potentially broken anchor link to the license section caused by header changes and a minor styling inconsistency in the contact email link.
how-to-access-gated-models.mdx
Outdated
| --- | ||
|
|
||
| TabPFN-2.6 is released under a [non-commercial license](models#tabpfn-model-license), which you need to accept before the model files can be downloaded. | ||
| TabPFN-2.5 and TabPFN-2.6 are released under a [non-commercial license](models#tabpfn-model-license), which you need to accept before the model files can be downloaded. |
There was a problem hiding this comment.
The link to the license section uses a relative path and an anchor that is likely broken due to the header update in models.mdx. It is recommended to use an absolute path (starting with /) and update the anchor to match the new header slug for better reliability and consistency with other links in the repository.
TabPFN-2.5 and TabPFN-2.6 are released under a [non-commercial license](/models#tabpfn-2-5-tabpfn-2-6-model-license), which you need to accept before the model files can be downloaded.
There was a problem hiding this comment.
Fixed in e4f529d — changed to absolute path /models#tabpfn-model-license. Note: the anchor Gemini suggested (#tabpfn-2-5-tabpfn-2-6-model-license) doesn't match the actual heading on main, which is "TabPFN Model License" → #tabpfn-model-license.
how-to-access-gated-models.mdx
Outdated
| - Ensure that "Notebook access" is enabled | ||
|
|
||
| For more information, refer to the Hugging Face [documentation](https://huggingface.co/docs/huggingface_hub/en/quick-start#authentication). No newline at end of file | ||
| If access via the browser-based flow is not an option for you, please contact us at [`sales@priorlabs.ai`](mailto:sales@priorlabs.ai). |
There was a problem hiding this comment.
The email address is wrapped in backticks inside the link text, which is inconsistent with other contact links in the documentation (e.g., in models.mdx). Removing the backticks will ensure a uniform visual style for contact links.
If access via the browser-based flow is not an option for you, please contact us at [sales@priorlabs.ai](mailto:sales@priorlabs.ai).
There was a problem hiding this comment.
Fixed in e4f529d — removed backticks from email link.
Use absolute path for license link, remove backticks from email link for style consistency. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
oscarkey
left a comment
oscarkey
left a comment
There was a problem hiding this comment.
thank you for updating!
Should we also include a section for notebooks other than colab, so less technically minded users know what to do in case the browser workflow doesn't work?
| You can also set `TABPFN_NO_BROWSER` to disable the automatic browser login if needed (e.g. in environments where opening a browser is undesirable). | ||
|
|
||
| ### **If using the terminal** | ||
| ## Google Colab |
There was a problem hiding this comment.
Maybe we should move this up one section so it's under ## Automatic Setup (recommended)? Just so non-technical users see it quickly if the automatic login doesn't work.
| ### In scripts or CI pipelines | ||
|
|
||
| Configure your environment to specify the `HF_TOKEN` environment variable. | ||
| - Select "Secrets" from the left-hand menu |
There was a problem hiding this comment.
These instructions should also include how to get the token, as in Headless / CI Environments?
| - Ensure that "Notebook access" is enabled | ||
|
|
||
| For more information, refer to the Hugging Face [documentation](https://huggingface.co/docs/huggingface_hub/en/quick-start#authentication). No newline at end of file | ||
| If access via the browser-based flow is not an option for you, please contact us at [sales@priorlabs.ai](mailto:sales@priorlabs.ai). |
There was a problem hiding this comment.
Maybe this should say "if access via one of these options is not possible for you"?
|
|
||
| <Warning> | ||
| The first time you use the model, you will be asked to accept the model license and configure the access token, see the [detailed instructions](/how-to-access-gated-models). | ||
| The first time you use the model, a browser window will open for you to log in and accept the model license. For headless environments, see the [detailed instructions](/how-to-access-gated-models). |
There was a problem hiding this comment.
I would say something more like "For more information, see the detailed instructions", because quite a lot of people won't know what a headless environment is.
2 participants
No description provided.