Skip to content

Model score example #650

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

Merged
merged 9 commits into from
Aug 3, 2023
Merged

Model score example #650

merged 9 commits into from
Aug 3, 2023

Conversation

jcheng5
Copy link
Collaborator

@jcheng5 jcheng5 commented Jul 29, 2023

Depends on #649

Fixes #638

  • Performance is oddly slow since 6c8f665, run profiler Previously, we were syncing each new Plotly figure's .data tuple with the existing widget. This was pretty fast. Then I added updating of each figure's layout, which is pretty slow. I made that optional and disabled it in our app--now it's fast again.

@jcheng5 jcheng5 marked this pull request as ready for review July 31, 2023 23:17
jcheng5 added 3 commits July 31, 2023 16:46
Caused by plotly layout update, which isn't even needed
head_content(), not tags.head(), is how you hoist arbitrary HTML
into the <head> of the enclosing page
Copy link
Contributor

@gshotwell gshotwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks great, I think it would be good to do a bit of work on the plot to make the model lines stand out a bit more.

jcheng5 added 2 commits August 2, 2023 13:26
Also change db initialization routine to ensure
the sqlite database is created before the app starts
@jcheng5 jcheng5 merged commit ca122cf into posit-dev:main Aug 3, 2023
@jcheng5 jcheng5 deleted the model-score-example branch August 3, 2023 22:27
schloerke added a commit that referenced this pull request Aug 9, 2023
* main:
  docs: Add Tooltip section in experimental (#673)
  Bump version to 0.5.1
  Add missing sidebar stylesheet dep (#667)
  GitHub action cleanup (#662)
  Add a triage label to issues created by non-maintainers (#647)
  Bump version to 0.5.0.9000
  Model score example (#650)
schloerke added a commit that referenced this pull request Aug 10, 2023
* main: (26 commits)
  Use a renderer generator to create sync/async render methods with full typing (#621)
  docs: Add Tooltip section in experimental (#673)
  Bump version to 0.5.1
  Add missing sidebar stylesheet dep (#667)
  GitHub action cleanup (#662)
  Add a triage label to issues created by non-maintainers (#647)
  Bump version to 0.5.0.9000
  Model score example (#650)
  Fix malformed changelog entry
  Update CHANGELOG.md for data frame filters (#654)
  Bump version to 0.5.0
  Allow ui.update_slider to handle non-numeric values (#649)
  Add updated path to examples (#645)
  Bump griffe version
  Update README
  Fix jumpy cursor in numeric filter (#640)
  Build API docs (#633)
  Pyright: ignore docs dir
  Filtering feature for data grid/table (#592)
  Add E2E tests for accordion and autoresize (#601)
  ...
schloerke added a commit that referenced this pull request Aug 10, 2023
* main: (40 commits)
  Use a renderer generator to create sync/async render methods with full typing (#621)
  docs: Add Tooltip section in experimental (#673)
  Bump version to 0.5.1
  Add missing sidebar stylesheet dep (#667)
  GitHub action cleanup (#662)
  Add a triage label to issues created by non-maintainers (#647)
  Bump version to 0.5.0.9000
  Model score example (#650)
  Fix malformed changelog entry
  Update CHANGELOG.md for data frame filters (#654)
  Bump version to 0.5.0
  Allow ui.update_slider to handle non-numeric values (#649)
  Add updated path to examples (#645)
  Bump griffe version
  Update README
  Fix jumpy cursor in numeric filter (#640)
  Build API docs (#633)
  Pyright: ignore docs dir
  Filtering feature for data grid/table (#592)
  Add E2E tests for accordion and autoresize (#601)
  ...
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.

Create dashboard example
2 participants