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

Unify processor architecture and avoid exposing internal implementation classes #172

Merged
merged 4 commits into from
Jul 28, 2024

Conversation

glopesdev
Copy link
Collaborator

During reorganization of the project for packaging we noticed that there were a few bits still raising warnings on the compiler, specifically related to mismatched processor architectures. The current version of the package targets 64-bit platforms.

Additionally, we also removed internal undocumented implementation classes from the public API surface. These are all types which, while being important for the internal library architecture, are not currently required outside of the library. Keeping them internal will allow us to change their API surface until such time when we need to make them public (if ever).

Finally, the ConfigureDS90UB9x operator was removed since it is not really possible to work with the raw serializer stream directly from the workflow without extensive configuration in code and dedicated data nodes.

@glopesdev glopesdev added the fix Pull request that fixes an issue label Jul 27, 2024
@glopesdev glopesdev added this to the 0.1.0 milestone Jul 27, 2024
@glopesdev glopesdev requested a review from jonnew July 27, 2024 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull request that fixes an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant