Skip to content

Refactor Task Internal Modules #538

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

Closed
wants to merge 13 commits into from

Conversation

charitylxy
Copy link
Collaborator

@charitylxy charitylxy commented Mar 14, 2024

What does this Pull Request accomplish?

This PR aims to address issue brought up in #461:

  • Currently, task subobject types are defined in internal modules contained in private nidaqmx._task_modules package.
  • We want to turn task.py into a package and merge it with the internal modules contained in nidaqmx._task_modules
  • Initial implementation completed in Refactor task.py and internal module (in_stream.py)  #534

Why should this Pull Request be merged?

In this PR, remaining task internal modules will be refactored and documentation will be updated:

  • _export_signal.py
  • _out_stream.py
  • _timing.py
  • collections: _channel_collection.py, _ai_channel_collection.py, _ao_channel_collection.py, _ci_channel_collection.py, _co_channel_collection.py, _di_channel_collection.py, _do_channel_collection.py
  • channels: _channel.py, _ai_channel.py, _ao_channel.py, _ci_channel.py, _co_channel.py, _di_channel.py, _do_channel.py
  • triggering: _triggers.py, _arm_start_trigger.py, _handshake_trigger.py, _pause_trigger.py, _reference_trigger.py, _start_trigger.py

Note: Documentation for some of the functions will be updated once the changes in https://github.com/ni/grpc-device-scrapigen/pull/227 are in.

What testing has been done?

In Progress

@charitylxy
Copy link
Collaborator Author

Will be splitting this PR into smaller piece.

@charitylxy charitylxy closed this Mar 15, 2024
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