Skip to content

Conversation

@newbee1905
Copy link

Pull Request Template

Thank you for contributing to our project! Please review the checklist and fill out the details below.

Description of Changes

I got some build error message:

/home/newbee/prjs/ai/qm9_ml/python-ace-master/lib/ace/ace-evaluator/lib/yaml-cpp/src/emitterutils.cpp:221:11: error: ‘uint16_t’ was not declared in this scope

221 | std::pair<uint16_t, uint16_t> EncodeUTF16SurrogatePair(int codePoint) {

| ^~~~~~~~

/home/newbee/prjs/ai/qm9_ml/python-ace-master/lib/ace/ace-evaluator/lib/yaml-cpp/src/emitterutils.cpp:13:1: note: ‘uint16_t’ is defined in header ‘<cstdint>’; this is probably fixable by adding ‘#include <cstdint>’

12 | #include "yaml-cpp/null.h"

+++ |+#include <cstdint>

So I just add the include for the type to the code in the file lib/ace/ace-evaluator/lib/yaml-cpp/src/emitterutils.cpp. Better solution would be updating yaml-cpp instead, but this is just a quick fix.

Checklist

  • Code is well-documented.
  • All tests have been run and passed.
  • Relevant documentation has been updated if necessary.

License Agreement

By submitting this pull request, I agree that:

  • The code submitted in this pull request will be distributed under the Academic Software License (free for academic non-commercial use, not free for commercial use), see LICENSE.md for more details.
  • The copyright for the code, including the submitted code, remains with Ruhr University Bochum. Ruhr University Bochum retains the right to transfer or modify the copyright.

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

1 participant