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

Mod sorting #177

Closed
wants to merge 8 commits into from
Closed

Mod sorting #177

wants to merge 8 commits into from

Conversation

thepaperpilot
Copy link
Contributor

Added a button to the fuzzy upgrade to toggle sorting by mod
This won't compile until the changes to the API are made as well

AlgorithmX2 and others added 8 commits September 27, 2014 18:11
Now anyone can open ME chests, but if there is a security terminal on the
network, you must have withdraw or deposit permissions in order to extract or inject
items, respectively.
This won't compile until the changes to the API are made as well
Conflicts:
	build.properties
	src/main/java/appeng/client/gui/implementations/GuiCellWorkbench.java
	src/main/java/appeng/container/implementations/ContainerFormationPlane.java
	src/main/java/appeng/container/implementations/ContainerLevelEmitter.java
	src/main/java/appeng/container/implementations/ContainerStorageBus.java
	src/main/java/appeng/items/tools/ToolMemoryCard.java
	src/main/java/appeng/me/GridStorage.java
	src/main/java/appeng/tile/storage/TileChest.java
	src/main/java/appeng/util/inv/AdaptorIInventory.java
@Cisien
Copy link
Contributor

Cisien commented Sep 29, 2014

Does anyone else feel that this change violates #3 in http://ae-mod.info/Suggestion-Guidelines/?

I feel that it is

@FireBall1725 @thatsIch, thoughts?

@thatsIch
Copy link
Member

@Cisien How is that achieved atm? I am not aware.

@Cisien
Copy link
Contributor

Cisien commented Sep 29, 2014

By partitioning normally cells and and storage busses.

cells and storage busses have a lot of available slots to configure them, these can be used to achieve the same thing

Also, i'm concerned that this feature is made redundant as soon as someone adds mods such as refined relocation and logicstics pipes into the mix.

@FireBall1725
Copy link
Contributor

As much as I like this idea, and the fact that I have a use for this, I feel that for right now I think it would be best to hold off on this feature request.

@FireBall1725 FireBall1725 added the feature A suggestion for a completely new feature label Sep 29, 2014
@PlasmaPower
Copy link

While I agree with @FireBall1725 that it is best to hold off, I think that this is a good idea. While the current solution is larger, it is also a lot more time consuming. To configure something for all of the items of a mod takes a lot of work, you can't AFK, and it's very boring. The main reason that this would be added is the current solution is too boring and time consuming, not too large. Lowering the time it takes to do mod sorting currently would not change the balance of AE2, but would make it less boring and more fun.

TL;DR: I think this would make AE2 less boring without changing the balance.

@Cisien
Copy link
Contributor

Cisien commented Sep 30, 2014

Keep the code around, it's a great idea. I'm closing this for now though

@Cisien Cisien closed this Sep 30, 2014
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature A suggestion for a completely new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants