Skip to content

Fix bug with context size in new nns #951

Merged
merged 3 commits into from
Sep 22, 2022
Merged

Fix bug with context size in new nns #951

merged 3 commits into from
Sep 22, 2022

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Sep 21, 2022

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Fix bug like in #936.

Closing issues

@Mr-Geekman Mr-Geekman added the bug Something isn't working label Sep 21, 2022
@Mr-Geekman Mr-Geekman added this to the Inference milestone Sep 21, 2022
@Mr-Geekman Mr-Geekman self-assigned this Sep 21, 2022
@github-actions
Copy link

github-actions bot commented Sep 21, 2022

🚀 Deployed on https://deploy-preview-951--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request September 21, 2022 16:15 Inactive
@martins0n martins0n self-requested a review September 22, 2022 07:39
@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2022

Codecov Report

❗ No coverage uploaded for pull request base (inference@d5eb926). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             inference     #951   +/-   ##
============================================
  Coverage             ?   85.05%           
============================================
  Files                ?      132           
  Lines                ?     7580           
  Branches             ?        0           
============================================
  Hits                 ?     6447           
  Misses               ?     1133           
  Partials             ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions github-actions bot temporarily deployed to pull request September 22, 2022 09:18 Inactive
@Mr-Geekman Mr-Geekman merged commit b622657 into inference Sep 22, 2022
@Mr-Geekman Mr-Geekman deleted the fix-new-nns-bug branch September 22, 2022 12:10
@Mr-Geekman Mr-Geekman mentioned this pull request Oct 26, 2022
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants