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

removed 'BatchKey' from 'encoders.py' and replaced necessary parts with strings... #329

Closed
wants to merge 2 commits into from

Conversation

devojyotimisra
Copy link

Pull Request

Description

As the issue stated to get rid of "BatchKey" from "encoders.py" i have successfully done that and as the issue also stated to replace necessary parts with string, i have done that also.

Fixes #

How Has This Been Tested?

In a very humble tone, i am just getting started and i think that no tests were required since it was a very small change, please help me out if i am wrong.

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@devojyotimisra
Copy link
Author

I just wanted to know that do you guys have any social discussion forums like "Discord Server" or "Slack Server", as i am new and i couldn't find any such forums for discussion rather than the GitHub Discussion forum, if i was provided with some then it would be really helpful

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