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.

Sign up for GitHub

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

Jump to bottom

Celeste 64: v1.3.0 Content Update #4581

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Celeste 64: v1.3.0 Content Update #4581

wants to merge 22 commits into from

Conversation

PoryGone
Copy link
Collaborator

Features:

  • New optional Location Checks
    • Checkpointsanity
  • Hair Color
    • Allows for setting of Maddy's hair color in each of No Dash, One Dash, Two Dash, and Feather states
  • Other Player Ghosts
    • A game config option allows you to see ghosts of other Celeste 64 players in the multiworld

Quality of Life:

  • Checkpoint Warping
    • Received Checkpoint items allow for warping to their respective checkpoint
      • These items are on their respective checkpoint location if Checkpointsanity is disabled
    • Logic accounts for being able to warp to otherwise inaccessible areas
    • Checkpoints are a possible option for a starting item on Standard Logic + Move Shuffle + Checkpointsanity
  • New Options toggle to enable/disable background input

Bug Fixes:

  • Traffic Blocks now correctly appear disabled within Cassettes

PoryGone added 19 commits January 10, 2025 02:54
@PoryGone
Functionality and first logic set for checkpointsanity
3bec408
@PoryGone
Checkpointsanity tweaks
1092abc
@PoryGone
Hair Color Shuffle
83e9bf9
@PoryGone
Put locations in proper regions, and lock Badeline Checkpoint item/lo…
505724d 
…cation
@PoryGone
Finished Region Logic
50b14f9
@PoryGone
Refactor goal handling
efcae67
@PoryGone
Rerefactor goal condition
deff882
@PoryGone
Checkpoint connections and last cleanup
79063d6
@PoryGone
move_shuffle on and move_shuffle off + start with all moves can just …
6a84b9e 
…share logic
@PoryGone
Logic tweaks
fe9b356
@PoryGone
Changelog update
e4af112
@PoryGone
Option doc fix
b48bbac
@PoryGone
Double Dash and Air Dash for Feather Maze Hard
ebbdc0d
@PoryGone
Merge branch 'main' into celeste64-v1.3
d43fbd7
@PoryGone
Fix item name in rule
b25509b
@PoryGone
Update Changelog
7bcb20e
@PoryGone
Clarify Color option tooltip
59fab1b
@PoryGone
Logic tweak
f9a124e
@PoryGone
Use Checkpoints as possible starting items when relevant
448b1d1
@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Jan 30, 2025
@PoryGone PoryGone changed the title Celeste 64: v1.3 Content Update Celeste 64: v1.3.0 Content Update Jan 30, 2025
qwint
qwint reviewed Jan 30, 2025
View reviewed changes
worlds/celeste64/Options.py Outdated Show resolved Hide resolved
qwint
qwint suggested changes Jan 30, 2025
View reviewed changes
Copy link
Collaborator

@qwint qwint left a comment

Choose a reason for hiding this comment

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

most nitpicks, but I think the connection Tuple construction is on a hot path and should be changed (or proven otherwise ig)

pure code review; ran no tests/gens/games

worlds/celeste64/Regions.py Show resolved Hide resolved
worlds/celeste64/__init__.py Show resolved Hide resolved
worlds/celeste64/Rules.py Outdated Show resolved Hide resolved
@PoryGone PoryGone added the is: enhancement Issues requesting new features or pull requests implementing new features. label Jan 30, 2025
qwint
qwint approved these changes Jan 30, 2025
View reviewed changes
Copy link
Collaborator

@qwint qwint left a comment

Choose a reason for hiding this comment

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

my comments were all addressed

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

@Exempt-Medic Exempt-Medic Exempt-Medic left review comments

@qwint qwint qwint approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
is: enhancement Issues requesting new features or pull requests implementing new features. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PoryGone @qwint @Exempt-Medic