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

Cleanup #62

Open
7 of 10 tasks
wawando opened this issue Apr 13, 2022 · 0 comments
Open
7 of 10 tasks

Cleanup #62

wawando opened this issue Apr 13, 2022 · 0 comments
Labels
#low low-priority task

Comments

@wawando
Copy link
Contributor

wawando commented Apr 13, 2022

  • Remove unused Hierarchical constructors from textfile
  • Remove unused unique_id parameter from Dense class (done in template branch)
  • Add const modified to non modified variables
  • Use constexpr modifier for compile time constants
  • Compatibility with newer C++ compiler (gcc-11) (done in Thomas' branch)
  • Move OMM declarations from extension_headers into individual header file (looks clean but need to modify separate file that may not be intuitive when adding a new function)
  • Remove GPU example files
  • Remove TODO.md?
  • Move CONTRIBUTING.md to other place?
  • Rename and refactor test files to <function_name>_test.cpp?
@wawando wawando self-assigned this Apr 13, 2022
@wawando wawando removed their assignment Apr 18, 2022
@wawando wawando added the #high high-priority task label Apr 20, 2022
@wawando wawando self-assigned this Apr 20, 2022
@wawando wawando removed their assignment Apr 21, 2022
@wawando wawando added #low low-priority task and removed #high high-priority task labels Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#low low-priority task
Projects
None yet
Development

No branches or pull requests

1 participant