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

Request for adding exportmaps plugin to DFHack #903

Closed
wants to merge 14 commits into from
Closed

Request for adding exportmaps plugin to DFHack #903

wants to merge 14 commits into from

Conversation

ragundo
Copy link

@ragundo ragundo commented Apr 20, 2016

For approval and, hopefully, addition to DFHack.

The plugin has a github repo in https://github.com/ragundo/exportmaps with the corresponding doc.

For the moment, Windows and Linux only.
Tested in Ubuntu 15.10 64 bits
Windows XP 32 bits
Windows 7

Added support for OSX but untested

@PeridexisErrant
Copy link
Contributor

Nice, I really like this idea - you're absolutely right that even exportlegends takes ages!

I'll leave code-related comments to someone qualified, but regarding documentation and compatibility with utilities which use exported maps:

  • It would be really really nice if maps identical to the vanilla maps had the vanilla names. This ensures that Isoworld, all the various legends viewers, Dorven Realms, and any others I've forgotten will still work - it would be a real shame to thrown that away.
  • Adding more exports (eg the true-heightmaps) is great; making sure the complete set of vanilla maps is available would also be good (see above)
  • Your documentation is great! It would be really nice if it could use .rst, ReStructuredText markup - it could then be easily incorporated into the online (or offline) DFHack docs. Look at how Stonesense does it for an example, and if you get stuck I'm happy to fix any problems - it's just more elegant to fix before merging.

@ragundo
Copy link
Author

ragundo commented Apr 22, 2016

OK. I changed the names to mimic the DF original ones (but the extension will be .PNG)

About the heightmaps, the DF ones are already true heightmaps, only elevation and elevation respecting water aren't correct, so, I generated them.

About the doc, I'll keep in my mind for the future.

Thanks for your constructive comments PeridexisErrant!!

@lethosor
Copy link
Member

Following up on what I mentioned on IRC:
I made a fork of your plugin at https://github.com/DFHack/exportmaps. It looks like the commits you've made here aren't in your repo, though. Could you add them there?

@lethosor lethosor self-assigned this Apr 25, 2016
@ragundo
Copy link
Author

ragundo commented Apr 26, 2016

The code is the same at both repositories.
As I'm very bad at git, I keep one copy of the code in /dfhack/plugins/exportmaps (my fork of dfhack)
and another in a differente folder, which is the one that have it's own repository.
I commited in the dfhack fork (for updating the pull request) and, after that I copy the files to the external folder and commit again to my github repository for the plugin
Not very elegant and for sure that it can be done more efficiently, but if the plugin is linked as a submodule, I'll only need to commit to my repository.

@lethosor
Copy link
Member

lethosor commented Jun 9, 2018

From IRC, this only supports 32-bit DF right now. If anyone is motivated to work on this besides Ragundo, there's a DFHack fork of the plugin (DFHack/exportmaps, linked above); otherwise, this is probably on hold for now.

@lethosor lethosor added this to Wishlist in 0.44.12-r2 Jun 19, 2018
@lethosor lethosor added this to Wishlist in 0.44.12-r3 Dec 28, 2018
@lethosor lethosor removed their assignment Jul 22, 2019
@ragundo
Copy link
Author

ragundo commented Dec 10, 2022

I close this PR as it would not work in 64 bits DF

@ragundo ragundo closed this Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
0.44.12-r2
  
Wishlist
0.44.12-r3
  
Wishlist
Development

Successfully merging this pull request may close these issues.

None yet

3 participants