-
Notifications
You must be signed in to change notification settings - Fork 7
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
Nn tiles #381
Conversation
src_py/apiServer/stats.py
Outdated
|
||
return fig, ax | ||
|
||
def get_distributed_train_labels(self , normalize : bool = False ,plot : bool = False , saveToFile : bool = False): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add an explanation what this method does
src_py/apiServer/stats.py
Outdated
Parameters | ||
---------- | ||
results : dict | ||
A mapping from question labels to a list of answers per category. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a link to an explanation of what is question labels.
Please explain generally what this function does
confusion_matrix_source_dict, confusion_matrix_distributed_dict = self.get_confusion_matrices_tiles(normalize ,plot ,saveToFile ) | ||
return confusion_matrix_source_dict, confusion_matrix_distributed_dict | ||
|
||
def get_confusion_matrices_tiles(self , normalize : bool = False ,plot : bool = False , saveToFile : bool = False): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add explanation - what this function does
src_py/apiServer/experiment_flow.py
Outdated
@@ -57,12 +58,12 @@ def get_events_sync(self): | |||
def get_csv_dataset(self): | |||
return self.csv_dataset | |||
|
|||
def generate_stats(self, experiment_phase = None) -> Stats: | |||
def generate_stats(self, experiment_phase = None) -> StatsTiles: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change it back to Stats object.
If you want to generate StatsTiles object please define new function that generate it.
if experiment_phase is None: | ||
experiment_phase = self.get_current_experiment_phase() | ||
return Stats(experiment_phase) | ||
return StatsTiles(experiment_phase) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix it here to
add nn tiles - stats tiles