Skip to content

Conversation

@sayalaruano
Copy link
Collaborator

@sayalaruano sayalaruano commented Feb 24, 2025

The solution to the df bug - dexplo/dataframe_image#127
Also, publish a new streamlit report upon a new release and check quarto installation

@sayalaruano sayalaruano requested a review from enryH February 24, 2025 17:30
Copy link
Collaborator

@enryH enryH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the main change is to call the method from the dataframe accessors (df.dfi)? Great to capture if quarto is installed or not.

@enryH
Copy link
Collaborator

enryH commented Feb 25, 2025

The documentation did not build do to a timeout. I can increase the limit of the per cell maximum execution time.

@enryH
Copy link
Collaborator

enryH commented Feb 25, 2025

Just fyi: In case the latest version of a library, e.g. dfi introduces a bug, you can also just exclude this specific subversion from the dependencies using something like dfi!=1.1.1.

@sayalaruano
Copy link
Collaborator Author

So the main change is to call the method from the dataframe accessors (df.dfi)? Great to capture if quarto is installed or not.

Yes, with the new version of dfi, this is the only way it worked. I tried other solutions suggested in dexplo/dataframe_image#127 and dexplo/dataframe_image#124, but they did not work.

@sayalaruano sayalaruano merged commit 5f24bcc into main Feb 25, 2025
13 checks passed
@sayalaruano sayalaruano deleted the dfi-export-bug-fix branch February 25, 2025 09:31
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