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

Maya Plugin - Add C++ Logging Library #118

Open
david-cattermole opened this issue Jul 14, 2019 · 2 comments
Open

Maya Plugin - Add C++ Logging Library #118

david-cattermole opened this issue Jul 14, 2019 · 2 comments
Assignees
Labels
core solver Related to the low-level solver functions. maya plugin Maya API Plug-In (C++)

Comments

@david-cattermole
Copy link
Owner

Feature

Logging inside the mmSolver plug-in is handled very simply, with std::out and std::err.

The solver is getting larger, and we need a logging infrastructure to help us save log files to disk for all the different parameters. This will also help debugging, since we get ask users for log files.

After looking at many different C++ loggers, loguru seems to be a good choice for mmSolver.
loguru's license is Public Domain, which is compatible with LGPL (mmSolver).
loguru doesn't seem to be C++11 dependent, which is important because we still officially support Maya 2016 and 2017.
loguru seems simple to use. loguru does not need be compiled into a library to be used (header only or static library).

https://github.com/emilk/loguru

While implementing logging using loguru, if a problem is encountered, we will search for another logging library.

Another alternative library is Google's glog. https://github.com/google/glog

@david-cattermole david-cattermole added core solver Related to the low-level solver functions. maya plugin Maya API Plug-In (C++) labels Jul 14, 2019
@david-cattermole david-cattermole added this to the v0.3.0 milestone Jul 14, 2019
@david-cattermole david-cattermole self-assigned this Jul 14, 2019
@david-cattermole david-cattermole modified the milestones: v0.3.0, v0.4.0 Aug 29, 2019
@david-cattermole
Copy link
Owner Author

I'm pushing this feature to v0.4.0 (or perhaps v0.3.x), as it's not critical for the features in v0.3.0.

@david-cattermole david-cattermole removed this from the v0.4.0 milestone Nov 6, 2022
@david-cattermole
Copy link
Owner Author

I'm not sure this is really needed. It's fairly low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core solver Related to the low-level solver functions. maya plugin Maya API Plug-In (C++)
Projects
None yet
Development

No branches or pull requests

1 participant