Skip to content

Resolve issue #624 (Tutorial related issues)#625

Draft
OFR-IIASA wants to merge 2 commits intomainfrom
issue/624
Draft

Resolve issue #624 (Tutorial related issues)#625
OFR-IIASA wants to merge 2 commits intomainfrom
issue/624

Conversation

@OFR-IIASA
Copy link
Contributor

This PR resolves issues identified in #624.

How to review

Please review all the tutorials and changes made. Check that the tutorials work and that the corresponding descriptions match.

PR checklist

  • Continuous integration checks all ✅
  • Add or expand tests; coverage checks both ✅
  • Add, expand, or update documentation.
  • Update release notes.

@OFR-IIASA OFR-IIASA marked this pull request as draft June 8, 2022 11:27
@OFR-IIASA OFR-IIASA self-assigned this Jun 8, 2022
@OFR-IIASA OFR-IIASA added bug Doesn't work as advertised/unintended effects enh New features & functionality labels Jun 8, 2022
@OFR-IIASA OFR-IIASA added this to the 3.6 milestone Jun 8, 2022
@codecov
Copy link

codecov bot commented Jun 8, 2022

Codecov Report

Merging #625 (f994b59) into main (c497a5f) will not change coverage.
Report is 355 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #625   +/-   ##
=====================================
  Coverage   93.0%   93.0%           
=====================================
  Files         43      43           
  Lines       3374    3374           
=====================================
  Hits        3138    3138           
  Misses       236     236           

@glatterf42
Copy link
Member

Shifted due to prioritization of ENGAGE baseline unit issue.

@glatterf42 glatterf42 modified the milestones: 3.8, 3.9 Dec 6, 2023
@glatterf42
Copy link
Member

Postponed due to lack of capacity/urgency.

@glatterf42 glatterf42 modified the milestones: 3.9, 3.10 May 23, 2024
@khaeru khaeru modified the milestones: 3.10, 3.11 Feb 3, 2025
@khaeru
Copy link
Member

khaeru commented Feb 3, 2025

I've changed the milestone to 3.11. If this is priority, then someone should be identified who will have bandwidth to:

  • Rebase, addressing conflicts.
  • Complete the PR checklist.

If not (i.e. if every capable person is occupied with tasks that are designated as higher priority), then we can remove the milestone until that changes.

@khaeru khaeru removed this from the 3.11 milestone May 27, 2025
@glatterf42
Copy link
Member

Some more notes on tutorial-related issues can be found in this Slack thread (not public).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Doesn't work as advertised/unintended effects enh New features & functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Westeros tutorials inconsistent with changes made to baseline technical_lifetime not defined for historical vintages in Westeros tutorial

3 participants