Skip to content

Commit aa8b443

Browse files
committed
🐛 Fix(streamlit_reportview.py): add f string to print df_index and create this variable once
1 parent 44cf8a6 commit aa8b443

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/vuegen/streamlit_reportview.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -468,7 +468,6 @@ def _generate_dataframe_content(self, dataframe) -> List[str]:
468468

469469
# Displays a DataFrame using AgGrid with configurable options.
470470
dataframe_content.append("""
471-
df_index = 1
472471
# Displays a DataFrame using AgGrid with configurable options.
473472
grid_builder = GridOptionsBuilder.from_dataframe(df)
474473
grid_builder.configure_default_column(editable=True, groupable=True)
@@ -482,11 +481,11 @@ def _generate_dataframe_content(self, dataframe) -> List[str]:
482481
# Button to download the df
483482
df_csv = df.to_csv(sep=',', header=True, index=False).encode('utf-8')
484483
st.download_button(
485-
label="Download dataframe {df_index} as CSV",
484+
label="Download dataframe as CSV",
486485
data=df_csv,
487-
file_name="dataframe_{df_index}.csv",
486+
file_name=f"dataframe_{df_index}.csv",
488487
mime='text/csv',
489-
key="download_button_{df_index}")
488+
key=f"download_button_{df_index}")
490489
df_index += 1""")
491490
except Exception as e:
492491
self.report.logger.error(f"Error generating content for DataFrame: {dataframe.title}. Error: {str(e)}")
@@ -739,12 +738,13 @@ def _generate_component_imports(self, component: r.Component) -> List[str]:
739738
plot_type = getattr(component, 'plot_type', None)
740739
if plot_type in components_imports['plot']:
741740
component_imports.extend(components_imports['plot'][plot_type])
742-
elif component_type == r.ComponentType.DATAFRAME:
743-
component_imports.extend(components_imports['dataframe'])
744741
elif component_type == r.ComponentType.MARKDOWN:
745742
component_imports.extend(components_imports['markdown'])
746743
elif component_type == r.ComponentType.CHATBOT:
747744
component_imports.extend(components_imports['chatbot'])
745+
elif component_type == r.ComponentType.DATAFRAME:
746+
component_imports.extend(components_imports['dataframe'])
747+
component_imports.extend("df_index = 1")
748748

749749
# Return the list of import statements
750-
return component_imports
750+
return component_imports

0 commit comments

Comments
 (0)