Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev fix generate_metadata #106

Merged
merged 2 commits into from
May 25, 2018
Merged

Conversation

taraspiotr
Copy link
Contributor

No description provided.

utils.py Outdated
@@ -171,11 +171,10 @@ def _generate_metadata(dataset):
if masks_dir_name.startswith(masks_dir_prefix):
masks_overlayed_sufix_to_write.append(masks_dir_name[len(masks_dir_prefix):])

df_columns = ['ImageId', 'file_path_image', 'is_train', 'is_valid', 'is_test', 'n_buildings']
df_dict = {'ImageId': [], 'file_path_image': [], 'is_train': [], 'is_valid': [], 'is_test': [], 'n_buildings': []}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@taraspiotr it is recommended to do something with either setdefault method or
use defaultdict object

df_dict={}
for ...
   df_dict.setdefault('bla',[]).append(cool_value)

@jakubczakon jakubczakon merged commit 79cef73 into neptune-ai:dev May 25, 2018
jakubczakon added a commit that referenced this pull request May 29, 2018
* Dev fix generate_metadata (#106)

* fix generate_metadata

* temp fix in callbacks

* Dev loader refactor (#111)

* refactored loaders, fixed weighed loss calculation

* updated config

* reverted resnet 152 to 32 filters with no dropout

* removed print

* back to 1000 valid size

* back to old batch size

* back other params

* config

* fix bug in loaders (#112)

* fix validation bug (#115)

* Dev tta (#116)

* scheleton added

* transformation and inverse done

* tta working

* Dev validation using mAP precision (#117)

* mege TTA with mAP validation

* mAP validation

* fix mAP valid in eval (#120)
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