[Feature Request?] Dimension Whitelist #237

Closed
opened 2020-03-23 00:40:24 +00:00 by import_user_oitsjustjose_95mQ · 7 comments
import_user_oitsjustjose_95mQ commented 2020-03-23 00:40:24 +00:00 (Migrated from git.oitsjustjose.com)

Correct me if I'm wrong but I can't find anything in the documentation saying that there can be a dimension whitelist. Best I can think of to do right now for this behavior is give it an array of every possible dim id with the wanted dims removed. Is there any better way of doing this? I plan on using multiple mods that can dynamically create new dimensions so even just giving it a blacklist of every dimension the pack adds wouldn't work.

Correct me if I'm wrong but I can't find anything in the documentation saying that there can be a dimension whitelist. Best I can think of to do right now for this behavior is give it an array of every possible dim id with the wanted dims removed. Is there any better way of doing this? I plan on using multiple mods that can dynamically create new dimensions so even just giving it a blacklist of every dimension the pack adds wouldn't work.
import_user_oitsjustjose_95mQ commented 2020-03-23 00:40:24 +00:00 (Migrated from git.oitsjustjose.com)

assigned to @oitsjustjose

assigned to `@oitsjustjose`
import_user_oitsjustjose_95mQ commented 2020-03-23 01:14:17 +00:00 (Migrated from git.oitsjustjose.com)

changed title from Dimension Whitelist{-?-} to {+[Feature Request?] +}Dimension Whitelist

changed title from **Dimension Whitelist{-?-}** to **{+[Feature Request?] +}Dimension Whitelist**
import_user_oitsjustjose_95mQ commented 2020-05-05 18:11:47 +00:00 (Migrated from git.oitsjustjose.com)

This sounds like a good feature for the last update I plan for 1.12! Thanks for the input, I'll get on this as soon as I can!

This sounds like a good feature for the last update I plan for 1.12! Thanks for the input, I'll get on this as soon as I can!
import_user_oitsjustjose_95mQ commented 2020-07-08 13:08:02 +00:00 (Migrated from git.oitsjustjose.com)

So I'm considering refactoring the config style to look something like this:

{
  "ores": [
    {
      ...,
      "biomes": {
        "list": [ "SANDY", "minecraft:plains", "biomesoplenty:meadow" ],
        "whitelist": true
      },
      "dimensions": {
        "list": [ "the_nether", "the_end" ],
        "whitelist": false
      }
    }
  ],
  "stones": [
    {
      ...
    }
  ]
}

This would apply for all versions - 1.12, 1.14 and 1.15. It would be a little bit of work but it'd help a lot with general usability.

So I'm *considering* refactoring the config style to look something like this: ```json { "ores": [ { ..., "biomes": { "list": [ "SANDY", "minecraft:plains", "biomesoplenty:meadow" ], "whitelist": true }, "dimensions": { "list": [ "the_nether", "the_end" ], "whitelist": false } } ], "stones": [ { ... } ] } ``` This would apply for all versions - 1.12, 1.14 and 1.15. It would be a little bit of work but it'd help a lot with general usability.
import_user_oitsjustjose_95mQ commented 2020-07-08 15:04:51 +00:00 (Migrated from git.oitsjustjose.com)

That looks a lot better, one thing I would ask for is also being able to give just a mod in the dimensions list for when you'd want generation in one mod's dynamically added dimensions and not another's.

That looks a lot better, one thing I would ask for is also being able to give just a mod in the dimensions list for when you'd want generation in one mod's dynamically added dimensions and not another's.
import_user_oitsjustjose_95mQ commented 2021-02-14 17:05:07 +00:00 (Migrated from git.oitsjustjose.com)

I haven't forgotten about this by the way -- I've been really busy lately and this would be a pretty big overhaul to the pluton generation code (the configs are now datapacks so it's actually much easier), but I just don't have the time nor the motivation to really work on it currently. But it hasn't been forgotten.

Thanks for your patience ❤️

I haven't forgotten about this by the way -- I've been really busy lately and this would be a pretty big overhaul to the pluton generation code (the configs are now datapacks so it's actually much easier), but I just don't have the time nor the motivation to really work on it currently. But it hasn't been forgotten. Thanks for your patience :heart:
import_user_oitsjustjose_95mQ commented 2021-03-13 00:51:38 +00:00 (Migrated from git.oitsjustjose.com)

That was a huge pain, but it's done :)

I'm happy with the result. So basically for a breakdown:

{
  ...
  "dimensions": {
    "isBlacklist": false,
    "filter": [ "overworld" ]
  }
}

If you're using the old geolosys.json method, you won't get this feature at all. That's it.

If you're using the new datapack mechanic and you DON'T use this new syntax, it'll gladly use the old one (for as long as that code is maintainable at least) -- NO old datapacks should be broken 😄

That was a huge pain, but it's done :) I'm happy with the result. So basically for a breakdown: ```json { ... "dimensions": { "isBlacklist": false, "filter": [ "overworld" ] } } ``` **If you're using the old `geolosys.json` method, you won't get this feature at all. That's it.** If you're using the new datapack mechanic and you **DON'T** use this new syntax, it'll gladly use the old one (for as long as that code is maintainable at least) -- NO old datapacks should be broken :smile:
import_user_oitsjustjose_95mQ (Migrated from git.oitsjustjose.com) closed this issue 2021-03-13 00:51:38 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
oitsjustjose/Geolosys#237
No description provided.