Skip to content

Fix df merge issue.#106

Open
vattay wants to merge 1 commit into
fastai:masterfrom
vattay:model-merge-fix
Open

Fix df merge issue.#106
vattay wants to merge 1 commit into
fastai:masterfrom
vattay:model-merge-fix

Conversation

@vattay

@vattay vattay commented Nov 30, 2023

Copy link
Copy Markdown

Fix to get around the df merge resulting in an empty df since the results-imagenet.csv file has longer form names like "model.extrainfo".

I'm not sure why jupyter notebook is changing to much of the notebook formatting and code... if you want just the fix it is quite short:

Ensure plotly and statsmodels are installed.

! git clone --depth 1 https://github.com/rwightman/pytorch-image-models.git
%cd pytorch-image-models/results
%pip install plotly statsmodels

Merge on the prefix of the long dot split names in the results-imagenet.csv.

import pandas as pd
df_results = pd.read_csv('results-imagenet.csv')
df.rename(columns={'model': 'model_long'}, inplace=True)
df_results['model'] = df_results['model'].apply(lambda x: x.split('.')[0])

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@whatadewitt

Copy link
Copy Markdown

Thanks for this!

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.

2 participants