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

Documentation update #160

Merged
merged 13 commits into from
Oct 30, 2023
Merged

Documentation update #160

merged 13 commits into from
Oct 30, 2023

Conversation

piterand
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #160 (8effea7) into main (434e018) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #160   +/-   ##
=======================================
  Coverage   70.81%   70.81%           
=======================================
  Files          18       18           
  Lines        1737     1737           
=======================================
  Hits         1230     1230           
  Misses        507      507           
Files Coverage Δ
ammico/summary.py 66.28% <ø> (ø)

@iulusoy
Copy link
Member

iulusoy commented Oct 27, 2023

Update the demo notebook some more:

  • no more copying it in the CI, as some variables need to be set differently for the DemoNotebook in the docs/source folder.
  • there is an issue with the way summaryDetector is called in the DemoNotebook, can you please take a look?
    image
  • The Demo notebook for multimodal search - is that up do date?

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@piterand piterand merged commit fe1e937 into main Oct 30, 2023
7 checks passed
@piterand piterand deleted the documentation_update branch October 30, 2023 15:18
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