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

Seed searching #60

Closed
tmewett opened this issue Feb 22, 2020 · 9 comments
Closed

Seed searching #60

tmewett opened this issue Feb 22, 2020 · 9 comments
Labels
enhancement New feature or request

Comments

@tmewett
Copy link
Owner

tmewett commented Feb 22, 2020

As a supplement to, or instead of, the seed catalog, it would be good to be able to search through seeds.

I picture a command ./brogue --search blah, which will start at the current default seed and go backwards, looking for items or allies (or other lines generated in the seed catalog) which have "blah" as a sub-string and printing them and their found depth to standard output. It would only search as deep as the shallowest depth a match was found so far, so each match will be a seed with a match earlier in the dungeon, until one is found on D1 where the search may terminate.

@tmewett tmewett added the enhancement New feature or request label Feb 22, 2020
@Patashu
Copy link

Patashu commented Apr 6, 2020

Why not just merge in the changes from https://github.com/Patashu/Brogue-Seed-Scummer so you don't have to do a bunch of work again?

Although your 'find as good of a seed as possible' idea isn't implemented in it, so there's definitely room for improvement.

@AndrewSav
Copy link
Contributor

@Patashu last time I looked at it, it was not in shape to be merged. That is in my opinion it potentially could result in the same amount or more work then otherwise.

@Patashu
Copy link

Patashu commented Apr 6, 2020

It wasn't very hard for me to merge it into the latest non-CE version - I just slapped https://www.dropbox.com/s/v8gur4ur4aq0m0m/174scum051.patch?dl=0 on it. are there CE changes that made it incompatible to merge?

@AndrewSav
Copy link
Contributor

@Patashu I'm sorry, it's been awhile and my memory is not that good. I think it was reliant on TCOD and CE does not have TCOD any more.

@Patashu
Copy link

Patashu commented Apr 7, 2020

Aaah, okay. If CE's changed substantially then yes, it'd have to be looked at again. Thanks for the info!

@tmewett
Copy link
Owner Author

tmewett commented Apr 7, 2020

Hi @Patashu, thanks for the idea! Having a brief glance at the patch, I think it could be possible to merge it at a technical level. I need to read it in detail to form an opinion on it.

@tmewett
Copy link
Owner Author

tmewett commented Mar 11, 2021

I'm going to close this - --print-seed-catalog partially addresses this, and the CSV output could be easily used to build a searchable database. (This may be the best route as generating seed catalogs is not that fast, so neither would be searching for rare items.)

@tmewett tmewett closed this as completed Mar 11, 2021
@Medwar
Copy link

Medwar commented Nov 18, 2022

Is --print-seed-catalog still supported?
Can't find any doc or help on how to use it.
running ./brogue --print-seed-catalog 13 15 or ./brogue --print-seed-catalog runs the process (it loads the processor and generates broguesave files, but I can't see the output CSV anywhere. Windows version.

@tmewett
Copy link
Owner Author

tmewett commented Nov 18, 2022

It is supported - you need to pipe to an output file by adding > your-filename-here to the command - but there is also an issue #399 on Windows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants