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

Fix: Visualization Bug #92

Open
danfke opened this issue Jun 22, 2022 · 3 comments
Open

Fix: Visualization Bug #92

danfke opened this issue Jun 22, 2022 · 3 comments

Comments

@danfke
Copy link
Collaborator

danfke commented Jun 22, 2022

Currently if only one MU is accepted the dashboard doesn't work. Also it won't work with both the Hug et al. data and our data with the way we've currently implemented it. We need to make sure that the data structures match within the function to allow it to work for both sets of data.

@danfke
Copy link
Collaborator Author

danfke commented Jun 28, 2022

@Radascript the RMSE is not working, I'm not sure when it broke.

@danfke
Copy link
Collaborator Author

danfke commented Jun 28, 2022

The only bug left is that the dashboard won't work if only one MU is accepted.

@danfke
Copy link
Collaborator Author

danfke commented Jul 1, 2022

From @Rowansiv:

Data Compatability bug with Viz Tool

  • Issue is that the muap_dict and pulse_plot functions are written assuming the input data 'looks a certain way'. That is, it works on the basis that the data has multiple motor units and therefore is an array of arrays.

  • However, if a dataset containing only a single motor unit is passed, it will incorrectly loop through the spike timings as opposed to each motor unit.

  • The output of the decomposition function removes all unnecessary dimensionality so any if/else statements would need to find a differentiator between a dataset with multiple arrays (motor units) and a dataset that is an array for a single motor unit.

  • Possible solutions could include using an if/else statement or adding a 'dummy' motor unit so that each dataset contains multiple motor units which would allow the functions to appropriately loop through each motor unit.

@danfke danfke changed the title bug: dashboard Visualization Bug Jul 1, 2022
@danfke danfke changed the title Visualization Bug Fix: Visualization Bug Jul 1, 2022
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

No branches or pull requests

1 participant