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

Enrich the CML Action using jupytext #176

Merged
merged 92 commits into from
Sep 7, 2022

Conversation

radistoubalidis
Copy link
Contributor

@radistoubalidis radistoubalidis commented Aug 30, 2022

Pull Request Template

Description

Use the jupytext tool to convert annual_summaries.py into a python notebook so its code can be used for display in the CML Action.

Additional Context (Please include any Screenshots/gifs if relevant)

The report would look something like this :

Headline

Some text about the report

Dependencies
import sqlite3
import pandas as pd
Connect to Database and Define SQL Query
conn = sqlite3.connect("tests/output/compiled_simulation_output.db")
query = <query_string>
Run Query and Generate plot
df = pd.read_sql_query(query, conn)
ax = df.plot.line("year")
ax.figure.savefig("tests/output/total_biomass_mt.png", dpi=300)

The plot image

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
@radistoubalidis radistoubalidis marked this pull request as draft August 30, 2022 11:59
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
radistoubalidis and others added 25 commits September 7, 2022 14:51
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
@HarshCasper HarshCasper merged commit 6535209 into moja-global:master Sep 7, 2022
Freeman-kuch pushed a commit to Freeman-kuch/FLINT.Cloud that referenced this pull request Oct 28, 2022
* reproduce FLINT.Cloud apis

* example sim report using cml

* Update cml-report.yml

* Update cml-report.yml

* Update cml-report.yml

* Update cml-report.yml

* test if cml is setup right

* add cml script

* changed file paths to not show local directories

* update cml report

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* update python script

* run model summary

* add sim start and end dates

* delete irrelevant files

* convert annual_summaries.py to py:percent format

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* converts and run annual_summaries.py as notebook usinng jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 4)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 5)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (fixed typo on 5, try 6)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (fixed typo on 7)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 8)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 9)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 10)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 11)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 12)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* check why cml comment is 404 not found

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* uncomment labeling filter

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* adds --uptate option to cml send-comment command

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action with --update

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action with --update

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test action with GITHUB_SHA var

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test action with GITHUB_SHA var (fix typo)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action without --pr option

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* run workflow only on labels

* reproduce FLINT.Cloud apis

* example sim report using cml

* Update cml-report.yml

* Update cml-report.yml

* Update cml-report.yml

* Update cml-report.yml

* test if cml is setup right

* add cml script

* changed file paths to not show local directories

* update cml report

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* setup GCBM summary

* update python script

* run model summary

* add sim start and end dates

* delete irrelevant files

* convert annual_summaries.py to py:percent format

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* converts and run annual_summaries.py as notebook usinng jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 4)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 5)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (fixed typo on 5, try 6)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (fixed typo on 7)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 8)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 9)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 10)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 11)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test if action works with jupytext (try 12)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* check why cml comment is 404 not found

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* uncomment labeling filter

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* adds --uptate option to cml send-comment command

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action with --update

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action with --update

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test action with GITHUB_SHA var

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test action with GITHUB_SHA var (fix typo)

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* test cml action without --pr option

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>

* run workflow only on labels

* fix lint

Signed-off-by: radis toubalidis <rtoumpalidis@gmail.com>
Co-authored-by: HarshCasper <erbeusgriffincasper@gmail.com>
Signed-off-by: Freeman <freemanoparaocha@gmail.com>
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

3 participants