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

update greedy examples removing incompatible params #59

Merged
merged 2 commits into from
Jun 23, 2023

Conversation

mirianfsilva
Copy link
Member


Status

READY

Description

Updated examples that were using incompatible parameters with the greedy method.

Impacted Areas in Library

User Examples.

Which issue(s) does this pull-request fix?

#58

Any special notes for your reviewer?


Checklist

  • Automated tests exist
  • Updated Package Requirements (if required, and with maintainers' approval)
  • Local unit tests performed
  • Documentation exists link
  • Local pre-commit hooks performed
  • Desired commit message set as PR title and description set above
  • Link to relevant GitHub issue provided

Signed-off-by: Mírian Silva <mirianfrsilva@ibm.com>
@mirianfsilva mirianfsilva changed the title update greedy examples removing some params update greedy examples removing incompatible params Jun 23, 2023
@coveralls
Copy link

coveralls commented Jun 23, 2023

Pull Request Test Coverage Report for Build 5357359733

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.172%

Totals Coverage Status
Change from base Build 5336735944: 0.0%
Covered Lines: 1018
Relevant Lines: 1081

💛 - Coveralls

Signed-off-by: Mírian Silva <mirianfrsilva@ibm.com>
@mirianfsilva mirianfsilva changed the base branch from main to develop June 23, 2023 14:51
@onkarbhardwaj onkarbhardwaj merged commit 5f9a2e0 into IBM:develop Jun 23, 2023
4 checks passed
onkarbhardwaj added a commit that referenced this pull request Jun 23, 2023
* Informative error messages in async generator and type change in GenAI exception (#57)

* update greedy examples removing incompatible params (#59)

* Fix/setup cfg (#49)

* fix: including README in pypi deployment

* fix: adding build dependency and changing workflow to use it


Signed-off-by: James Sutton <james.sutton@uk.ibm.com>
Co-authored-by: Mírian Silva <mirianfrsilva@ibm.com>
Co-authored-by: James Sutton <1068763+jpwsutton@users.noreply.github.com>
moneill0 pushed a commit to moneill0/ibm-generative-ai that referenced this pull request Jun 29, 2023
* Informative error messages in async generator and type change in GenAI exception (IBM#57)

* update greedy examples removing incompatible params (IBM#59)

* Fix/setup cfg (IBM#49)

* fix: including README in pypi deployment

* fix: adding build dependency and changing workflow to use it


Signed-off-by: James Sutton <james.sutton@uk.ibm.com>
Co-authored-by: Mírian Silva <mirianfrsilva@ibm.com>
Co-authored-by: James Sutton <1068763+jpwsutton@users.noreply.github.com>
@mirianfsilva mirianfsilva deleted the hotfix/greedy-examples branch July 7, 2023 15:10
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.

None yet

4 participants