Skip to content

Conversation

@louis-e
Copy link
Owner

@louis-e louis-e commented Dec 4, 2025

image image

@louis-e louis-e marked this pull request as ready for review December 4, 2025 17:15
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@louis-e
Copy link
Owner Author

louis-e commented Dec 4, 2025

retrigger-benchmark

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

⏱️ Benchmark run finished in 0m 28s
🧠 Peak memory usage: 898 MB

📈 Compared against baseline: 69s
🧮 Delta: -41s
🔢 Commit: 94388e4

✅ This PR improves generation time.

📅 Last benchmark: 2025-12-04 17:26:00 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

@louis-e
Copy link
Owner Author

louis-e commented Dec 4, 2025

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@louis-e
Copy link
Owner Author

louis-e commented Dec 5, 2025

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

arnis/src/gui.rs

Lines 836 to 840 in a8e3170

update_player_position(
&selected_world,
spawn_point,
bbox_text.clone(),
world_scale,

P1 Badge Skip Java spawn update for Bedrock worlds

When the GUI requests a Bedrock export with a custom spawn marker, gui_start_generation still calls update_player_position(&selected_world, …) before the format switch. For Bedrock flows selected_world is empty (no world was selected) and there is no level.dat yet, so update_player_position returns Level.dat not found and the generation aborts as soon as the user sets a spawn point. Bedrock generation with a user-defined spawn point is therefore broken; the spawn update should be skipped or pointed at the Bedrock output instead of the non-existent Java world.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@louis-e louis-e merged commit 438b2be into main Dec 5, 2025
2 checks passed
@louis-e louis-e deleted the bedrock-support branch December 5, 2025 16:34
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.

[FEATURE] Make world generation also for the Bedrock version of minecraft

2 participants