zackxconti / bnmetamodel_gh

Repo for bnmetamodel lib version for Lab Mouse Grasshopper plug-in.
1 stars 2 forks source link

Remove unused functions #23

Closed kallewesterling closed 8 months ago

kallewesterling commented 11 months ago

alphanum_key, len_csvdata, kfoldToList are unused functions in the Helper_functions.py file. (ranges, draw_barchartpd, draw_histograms are other functions not currently used, but still in the commented-out code)

Suggested solution

Remove unused functions.

Related issues

ghubrakesh commented 11 months ago

Are you sure all the functions you mentioned above are not used in the code? If they are used, removing them might cause errors in the code. Please confirm, so I can proceed to work on it.

kallewesterling commented 11 months ago

Thank you so much for engaging @ghubrakesh. We'd love to have your help soon. We're only just now starting to work on this package, and opening it up for open-source contribution. Unfortunately, the codebase is still too fragile and we need to first establish a workflow where we know which issues to tackle first. If you'd like, I'll make sure to stay in touch with you here when we'd love to see your contribution to the codebase.

kallewesterling commented 11 months ago

Adding more marking of functions, ready for review in this issue:

kallewesterling commented 8 months ago

These have been moved to the removed directory, so closing this issue.