AEModernMCPort / Applied-Energistics-3-Fork

A Minecraft Mod about Matter, Energy and using them to conquer the world..
http://ae-mod.info/
Other
37 stars 12 forks source link

CD:Andromeda Generic TODO #113

Open Elix-x opened 7 years ago

Elix-x commented 7 years ago

This is a TODO list of thing in order to make AE compileable without adding new stuff yet

(some stuff may be temporarily removed for the purpose of simpler porting, like P2P).

I may have forgotten some things, if so, please comment them down below.

If you want to contribute to AE, this is where you begin. Do something that is listed here (even if not completely), then submit a pull request. Eventually, we will invite you to join the team.

p455w0rd commented 7 years ago

The caps system is not at a stage where it should be used to replace interfaces. I would say add support where there can/needs to be dual support, but definitely don't depend on them quite yet. The system needs a decent overhaul before it's a viable replacement in a bigger mod like AE. @covers1624 here's your starting point :)

Elix-x commented 7 years ago

@p455w0rd Actually, no. Caps system is working on working well. I'm using it (or will be if you consider that i did not release yet) caps for my own caps & objects. Once you get hang of it, it is very versatile. It does have some things that might be changed to decrease learning curve, but in any case, it won't break anything. So AE is moving to caps with forge's caps for now. Because later it will be a good idea to move AE's ifaces to caps (not all, don't get me wrong, caps cannot replace interfaces. But it can replace some of them and in a better way).

Also, i already informed @covers1624 .

p455w0rd commented 7 years ago

@Elix-x Actually, no. They're broken in many cases (see several issues on the Forge repo mainly regarding syncing, but other things as well). Where they're not broken, they aren't optimized. This is pretty common knowledge to anyone that frequents the Forge repo. I'm by no means new to caps. I fully understand them which is why I made this claim. I use them in Teslafied and p455w0rd's Things..but these are small mods where the impact is minimal and I'm able to work around the bugs. #3658, #3603, #3483, #3239, #3179, #2523

Elix-x commented 7 years ago

I saw these. And there are a bunch more. If these are going to be fixed, this won't create many issues for AE. And if we stay on interfaces, we loose mod compatibility - caps are not interface backwards compatible and other mods moved to caps (vanilla inventories are the only ones wrapped with caps by forge).

In any case, at the rate we're currently going at, we're not touching TEs any time soon. Though this topic has to be revived once we reach it.