Greymerk / minecraft-roguelike

This is a mod for minecraft that adds randomly generated dungeon complexes.
GNU General Public License v3.0
130 stars 53 forks source link

Using /roguelike settings reload #103

Closed Alan19 closed 7 years ago

Alan19 commented 7 years ago

When I use /roguelike settings reload, it says that there's no errors but I'm not getting any loot. I'm using 1.10.2-1.5.7.

Greymerk commented 7 years ago

what sort of settings are you using?

Alan19 commented 7 years ago

Dungeon Rooms and Mobs and Loot for B1-B2: https://paste.ee/p/bIDw7] B3-B5:https://paste.ee/p/TDGZ0

Greymerk commented 7 years ago

Hard to say without the required mods. Can you tell me what mods i'd need to test these?

Alan19 commented 7 years ago

I think you would need

I might have missed some and it would probably be easier if I did a binary search by removing the files but I thought /roguelike settings reload should say if there's an error

Greymerk commented 7 years ago

Thanks, i'll give it a try.

Alan19 commented 7 years ago

I forgot to mention Railcraft and Embers

Alan19 commented 7 years ago

While editing the json files, I made a mistake and it was caught by /roguelike settings reload and printed out what was wrong. However, after spawn a new dungeon, there was still no loot.

Alan19 commented 7 years ago

I'll try updating to 1.5.8 and see if that fixes anything

Alan19 commented 7 years ago

It's still occurring on 1.5.8 and everything but the loot is being inherited.

Alan19 commented 7 years ago

The default loot still works though

Greymerk commented 7 years ago

I did some small tests with loot overrides, and I'm convinced now there's a bug with how this behaves.

I'm not sure what's causing it. will need to do more testing. It may be there's nothing actually wrong with your settings.

Greymerk commented 7 years ago

I think i figured this out. Will post a build in a bit and you can try it.

Greymerk commented 7 years ago

Try this one out. You'll need to explicitly add "builtin:base" to your inherit list now. It's necessary to make that change in order for inheritance/overrides to function in a sane way.

https://github.com/Greymerk/minecraft-roguelike/releases/tag/v1.6.5.1

Greymerk commented 7 years ago

It's still not right.

I'm thinking I'll need to change how overrides work to be something like this:

{
    "name" : "main",
    "criteria" : {},
    "exclusive" : true,
    "inherit" : [
        {
            "id" : "builtin:base",
            "override" : ["LOOTRULES"]
        },
        "loot",
        "rooms"
    ]   
}

You'd specify filters for things you don't want to inherit from the settings you include. It'd be much easier to understand.

I'll work on implementing it and see how it works.

Greymerk commented 7 years ago

I've given this some more thought and realized that it's probably not worth having the override system. It's too complicated to implement and to use. Rather it'd be much simpler to provide a clearer way to build up settings from smaller parts using inheritance. You could just leave out the loot settings.

That's what i'm trying to do now.

Greymerk commented 7 years ago

I worked it out, and did some playing with it, it's much easier to work with, and easier to understand. Will post a build of this sometime this afternoon hopefully.

Greymerk commented 7 years ago

Your loot system appears to work as expected now. I just have a couple other things that need to be addressed

I'd like the system to report when a block is unavailable, if possible.

Greymerk commented 7 years ago

this one should work: https://github.com/Greymerk/minecraft-roguelike/releases/tag/v1.6.5.2

But you'll need to add a new file to your inheritance chain:

default:base

{
    "name" : "base",
    "inherit" : [
        "builtin:secrets",
        "builtin:theme",
        "builtin:generator",
        "builtin:size",
        "builtin:segments"
    ]
}

and remove the overrides.

Alan19 commented 7 years ago

It's working now. Thanks!

Greymerk commented 7 years ago

Cool. I'll post the update to curse sometime this weekend probably.

resinate commented 7 years ago

alan are u able to make ur stuff naturally generate cuz my stuff isnt. all i did was custom block changes