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

Padding and unpadding of the data. #45

Closed
wants to merge 7 commits into from

Conversation

Priyanka-43
Copy link

For this assignment all pull requests going forward should contain the following (please delete the placeholder text and add your own description)

Title

  • A ledgible title that briefly describes the contents of the pull request.

Linting

  • Will update soon.

Contents

  • A ledgible description that indepthly describes the contents of the pull request.
  • A list of files that are new / changed
  • The contents that you have been assigned to submit labeled with titles that acuratly describe the assignment they are for

Labels and Milestones

  • ALL pull requests must have an attached label and Milestone before a reviewer can be assigned.

Reviewer

  • add a sugggested reviewer here

Any further questions on formatting pull requests can be asked in the issues tab or on discord.
This document can be changed at any time.

Added padding/unpadding of the data.
Added unit test for padding/unpadding.
fixed pylint error.
Fixed pylint error.
FIXED pylint error for paddin/unpadding part.
fixed error for pytest.
pytest error fixed.
@laserlab
Copy link
Contributor

laserlab commented Nov 6, 2024

Closing this since this should be fixed in the original PR #37

@laserlab laserlab closed this Nov 6, 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.

2 participants