Skip to content

Feature/261 example curriculum#262

Merged
LorenzzoQM merged 2 commits intodevelopfrom
feature/261-example-curriculum
May 9, 2025
Merged

Feature/261 example curriculum#262
LorenzzoQM merged 2 commits intodevelopfrom
feature/261-example-curriculum

Conversation

@LorenzzoQM
Copy link
Copy Markdown
Contributor

Description

Closes #261

Includes an example script on how to use curriculum learning with the single spacecraft tasking environment and RLlib.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New example script.

How should this pull request be reviewed?

  • By commit
  • All changes at once

How Has This Been Tested?

N/A.

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation and release notes
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@LorenzzoQM LorenzzoQM force-pushed the feature/261-example-curriculum branch from affd203 to 998bab8 Compare May 8, 2025 22:51
@Mark2000 Mark2000 self-requested a review May 8, 2025 22:55
@Mark2000
Copy link
Copy Markdown
Contributor

Mark2000 commented May 8, 2025

Let me know when you're done with any pushing and I can review

@LorenzzoQM LorenzzoQM force-pushed the feature/261-example-curriculum branch from 998bab8 to f0bd123 Compare May 8, 2025 23:17
@LorenzzoQM
Copy link
Copy Markdown
Contributor Author

@Mark2000 It is ready for your review

Copy link
Copy Markdown
Contributor

@Mark2000 Mark2000 left a comment

Choose a reason for hiding this comment

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

Nice example script!

Comment thread examples/curriculum_learning.ipynb Outdated
"metadata": {},
"outputs": [],
"source": [
"def capacity_fn(TIME_SEED, INIT_VAL, FINAL_VAL, difficulty):\n",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think these should be snake case since upper case usually implies a package-wide constant

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point! I will change those.

@LorenzzoQM LorenzzoQM force-pushed the feature/261-example-curriculum branch 2 times, most recently from bcb1b00 to 8783cfd Compare May 8, 2025 23:53
@LorenzzoQM LorenzzoQM force-pushed the feature/261-example-curriculum branch from 8783cfd to 422633f Compare May 9, 2025 00:24
@LorenzzoQM LorenzzoQM merged commit 392da3d into develop May 9, 2025
5 checks passed
@LorenzzoQM LorenzzoQM deleted the feature/261-example-curriculum branch May 9, 2025 00:32
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.

Example script on how to use curriculum learning

2 participants