Skip to content

Conversation

@ziweiq-29
Copy link

added qoi.cc and CMakeLists.txt.
I am not working with any external values(JSON format) extracted from Python scripts yet. I am working on getting the qoi integrating to Pressio so that I can run with the command "pressio....qoi...".

Copy link
Owner

@robertu94 robertu94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please send a copy of $BUILD_DIR / pressio_manual_register.cc to confirm that it is being registered correctly.

libpressio_plugin_file(io/posix.cc)
libpressio_plugin_file(domains/malloc.cc)
libpressio_plugin_file(domains/nonowning.cc)
libpressio_plugin_file(metrics/qoi.cc)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not needed.

libpressio_optional_component(copy_template "build the numpy io plugin" /io/copy_template.cc)
libpressio_optional_component(size "build the size metrics plugin" /metrics/size.cc)
libpressio_optional_component(time "build the time metrics plugin" /metrics/time.cc)
libpressio_optional_component(time "build the qoi metrics plugin" /metrics/qoi.cc)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be

libpressio_optional_component(qoi "build the qoi metrics plugin" /metrics/qoi.cc)

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.

2 participants