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

Reverted old ore generation algorithm and applied the latest changes from the main branch #657

Open
wants to merge 33 commits into
base: 0.8.1-latest-unstable
Choose a base branch
from

Conversation

venglov
Copy link

@venglov venglov commented Dec 29, 2024

What is the new behavior?

This PR reverts the new ore generation algorithm due to its current issues, which cannot be resolved on our side. Additionally, the latest changes and fixes from the main branch have been merged into this branch, with all conflicts resolved.

Implementation Details

  • Reverted the new ore generation algorithm because it lacks the necessary configurability to fix its issues, such as setting an offset for layers in the GTCE algorithm. Implementing a fix would require forking and maintaining a custom version of GTCE, which is not ideal.

  • Merged the latest changes from the main branch including enhancements for the old generation algorithm introduced by @Pyritie

Potential Compatibility Issues

No significant compatibility issues are expected

I am shitcommitter, please squash & merge :)

dependabot bot and others added 30 commits December 18, 2024 03:36
Bumps the dependencies group with 2 updates: [actions/checkout](https://github.com/actions/checkout) and [actions/upload-artifact](https://github.com/actions/upload-artifact).


Updates `actions/checkout` from 4.2.0 to 4.2.2
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.2.0...v4.2.2)

Updates `actions/upload-artifact` from 4.4.0 to 4.5.0
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4.4.0...v4.5.0)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
…ependabot/github_actions/dependencies-fbfbf083d1

Bump the dependencies group with 2 updates
Signed-off-by: Pyritie <[email protected]>
- Added ore indicators for everything
- Buffed naquadah and tetrahedrite veins because those sound like the biggest bottlenecks of late game players because there's no other planets

Signed-off-by: Pyritie <[email protected]>
Signed-off-by: Pyritie <[email protected]>
Bumps the dependencies group with 1 update: [actions/upload-artifact](https://github.com/actions/upload-artifact).


Updates `actions/upload-artifact` from 4.4.3 to 4.5.0
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4.4.3...v4.5.0)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
…ependabot/github_actions/dependencies-16239e05d6

Bump actions/upload-artifact from 4.4.3 to 4.5.0 in the dependencies group
Fixed oilsands, added missing english strings, added indicators for all ores
# Conflicts:
#	.github/ISSUE_TEMPLATE/BUG_REPORT.yml
#	.github/PULL_REQUEST_TEMPLATE.md
#	.github/README.md
#	.github/workflows/build.yml
#	.pakku/server-overrides/user_jvm_args.txt
#	kubejs/assets/tfg/lang/en_us.json
#	kubejs/server_scripts/tfc/tags.js
@Xikaro Xikaro added this to the 0.8.1 milestone Jan 7, 2025
@Xikaro Xikaro added 1. Type: Feature New feature or request. and removed 4. Version: 0.8.1 labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Type: Feature New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants