solresol / ultratree-results

This shows the results of the ultrametric tree-based, explainable, solar-powered language model
0 stars 0 forks source link

Sweep: Update README.md to include images #11

Closed solresol closed 1 day ago

solresol commented 1 day ago

Details

Create a section in README.md called "Key Metrics" that includes these two images:

Create a section called "Model Complexity" that includes these two images:

Checklist - [X] Modify `README.md` βœ“ https://github.com/solresol/ultratree-results/commit/bf561093721e4dc9ad292c2d9c160376a43d449d [Edit](https://github.com/solresol/ultratree-results/edit/sweep/update_readmemd_to_include_images/README.md) - [X] Running GitHub Actions for `README.md` βœ“ [Edit](https://github.com/solresol/ultratree-results/edit/sweep/update_readmemd_to_include_images/README.md)
ifost-autodev[bot] commented 1 day ago

πŸš€ Here's the PR! #12

See Sweep's progress at the progress dashboard!
πŸ’Ž Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: None)

[!TIP] I can email you next time I complete a pull request if you set up your email here!


Actions (click)

GitHub Actionsβœ“

Here are the GitHub Actions logs prior to making any changes:

Sandbox logs for f2084cc
Checking README.md for syntax errors... βœ… README.md has no syntax errors! 1/1 βœ“
Checking README.md for syntax errors...
βœ… README.md has no syntax errors!

Sandbox passed on the latest main, so sandbox checks will be enabled for this issue.


Step 1: πŸ”Ž Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I think are relevant in decreasing order of relevance (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/solresol/ultratree-results/blob/f2084cc963f7b9f7ef1ba954f04280106c748927/charts.py#L1-L38 https://github.com/solresol/ultratree-results/blob/f2084cc963f7b9f7ef1ba954f04280106c748927/validation_runs.sql#L1-L15 https://github.com/solresol/ultratree-results/blob/f2084cc963f7b9f7ef1ba954f04280106c748927/validation_runs.sql#L15-L26 https://github.com/solresol/ultratree-results/blob/f2084cc963f7b9f7ef1ba954f04280106c748927/README.md#L1-L1

Step 2: ⌨️ Coding

--- 
+++ 
@@ -1,2 +1,14 @@
 # ultratree-results
 This shows the results of the ultrametric tree-based, explainable, solar-powered language model
+
+## Key Metrics
+
+![Total Loss vs Time](total_loss_vs_time.png)
+
+![Model Node Count vs Time](model_node_count_vs_time.png)
+
+## Model Complexity
+
+![Average Depth vs Time](average_depth_vs_time.png)
+
+![Average In-Region Hits vs Time](average_in_region_hits_vs_time.png)

Ran GitHub Actions for bf561093721e4dc9ad292c2d9c160376a43d449d:


Step 3: πŸ” Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/update_readmemd_to_include_images.


πŸŽ‰ Latest improvements to Sweep:


πŸ’‘ To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. Join Our Discord