Skip to content

Fixing merge conflicts for PR#550 #551

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 18 commits into from
Nov 26, 2024
Merged

Conversation

validbeck
Copy link
Collaborator

Internal Notes for Reviewers

#550

validbeck and others added 18 commits November 22, 2024 08:28
* view-reports.qmd > working-with-analytics.qmd

* WIP view-report-data.qmd

* **Reports** > **Analytics**

* Tweaks to Dashboard fa-icons

* Tweaks to Model Inventory fa-icons

* Tweaks to Model Findings fa-icons

* Tweaks to Documentation fa-icons

* Tweaks to Validation Report fa-icons

* Tweaks to Ongoing Monitoring fa-icons

* Tweaks to Model Activity fa-icons

* Tweaks to Getting Started fa-icons

* Tweaks to Settings, Profile & User Directory fa-icons

* Tweaks to Model Inventory Fields & Model Lifecycle Statuses

* Tweaks to Templates fa-icons

* manage-custom-reports.qmd WIP

* Single-source files for visualization config

* Visualization options

* Set up for example visualization

* Updated outdated analytics screenshots

* Updated view-report-data.qmd

* Moved analytics into its own directory

* Updated example

* Minor tweaks

* Temporarily removing adding reports to dashboard

* Final cleanup
…540)

* Drafting new placeholder support landing

* New platform feedback section

* Tweaking style
* New intro_for_model_developers_EXECUTED.ipynb -secrets

New excuted copy without edits

WIP quarto render to execute the notebook so it displays widgets

Renaming executed file

Switching to notebook front matter

Hmm

Tweaked workflow?

Removed _freeze test files & commented out checkout in Makefile

Ugh trying again

Called make get-source for latest vm-library files

Reverting makefile

Changing API keys to secret variables

New intro_for_model_developers_EXECUTED.ipynb -secrets

* Test adding validmind[llm]

* Added variables locally

* Adding dependencies

* Testing full page layout

* Latest from vm-library & missing libraries?

* That didn't work, ugh

* I give up for today

* NVM reverting to a version that worked in theory

* Forgot comment

* Adding rendered notebook output to .gitignore

* SHAPGlobalImportance error workaround

* updating shap version pls work

* ???

* ?

* .

* ..

* Final touches to the workflows

* Updated .env.example

* Tweaking freeze settings

* Spellingggg

* Local execution instructions in front matter

* Wow I forgot to add the changes

* Changing to ValidMind Academy model

* Edited comments in notebook front matter

* Updated variables for mobile quirk

* Added _quarto-notebooks.yml

* Updated model, removed global execution & reverted deploy-prod & staging to main

* WIP moved notebook execution to its own workflow & removed front-matter

* Commenting out execution for now & added exe-demo profile

* Added exe-staging & exe-prod

* New single-source action for handling Quarto for rendering

* Testing single-sourcing Fetch Quarto on demo

* Single-sourced AWS credentials

* Whoops moving the checkout back into the main workflow

* Adding shell to the action file?

* Cleaned up aws action files

* Applied single-source to staging & prod workflows, testing AWS 1

* Hmmmmm

* Trying one last time

* I give up, reverting to the old version

* Test — if condition for site/notebook/ changes

* Test — if condition for site/notebook/ changes in an action

* Hide navbar 4 executed notebook & clear quarto cache b4 execution

* Testing a change to the executed notebook

* Trying again

* Swapping where the secret lives

* Applying same changes to staging & prod workflows/actions

* Removing clean command orz

* No changes to site/notebooks

* Latest from vm-library
* Update archive-delete-models.qmd

adding a note that only admin users can see archived models

* Making changes for Mehdi :)

---------

Co-authored-by: mehdi0501 <98982652+mehdi0501@users.noreply.github.com>
* Add login button & switch search

* Tweak login options

* Padding?

* Ughhh padding

* Add VPV to login options

* Wording tweak

---------

Co-authored-by: Nik Richers <nik@validmind.ai>
Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@validbeck validbeck added the internal Not to be externalized in the release notes label Nov 26, 2024
@validbeck validbeck self-assigned this Nov 26, 2024
Copy link
Contributor

A PR preview is available: Preview URL

@validbeck validbeck merged commit 1256cf5 into prod Nov 26, 2024
4 checks passed
@validbeck validbeck deleted the beck/fixing-2024feb14-release branch November 26, 2024 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Not to be externalized in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants