Skip to content

Conversation

@sayalaruano
Copy link
Collaborator

No description provided.

@sayalaruano sayalaruano requested a review from enryH June 13, 2025 08:30
@enryH
Copy link
Collaborator

enryH commented Jun 13, 2025

Should I test the notebooks in colab?

https://vuegen--123.org.readthedocs.build/en/123/vuegen_apicall_case_study.html

  • the numbering starts with zero (i don't mind).
  • the notebook above is a dummy example?
  • in the last cell you could also use the get_report function directly from the library (showcasing the usage as a Python module)

@sayalaruano
Copy link
Collaborator Author

  • Yes, it would be a good idea to test them on colab
  • Yes, starting in 0 was on purpose because section 0 is just a work env setup
  • Yes, maybe in the future we can create a better example for the APICall component using a resource like the Uniprot API.
  • True

@sayalaruano
Copy link
Collaborator Author

Also, I noticed that if we pass a config with just one section, the Streamlit report is empty. I created an issue for this - #124, so we can fix it in a new PR.

@sayalaruano sayalaruano merged commit 954110a into main Jun 17, 2025
16 checks passed
@sayalaruano sayalaruano deleted the apicall-chatbot-notebooks branch June 17, 2025 08:19
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

Successfully merging this pull request may close these issues.

3 participants