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

Proof of concept for loading cycle list definitions from separate files ... #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jdugge
Copy link

@jdugge jdugge commented Sep 11, 2014

...using the 'cycle list name' key. This is an attempt to address issue #2.

When 'cycle list name' is called, we first check whether that cycle list is already defined. If it isn't, we check whether a file with the cycle list name exists. If it does, we construct the cycle list from that file, using code provided by Christian Feuersänger at http://tex.stackexchange.com/a/32064/2552. If it doesn't, we display an error message.

This is only a proof of concept, only the "YlGn-3" and "Set1-4" cycle lists are defined.

@vtraag
Copy link
Owner

vtraag commented Jan 27, 2015

OK, took a bit longer than a week... :)

Seems to work fine when using cycle list name. For cycle mulit list this doesn't work. Would it be possible to ensure that both would be working correctly?

Also, I wonder if something similar can be done for all the colormaps? I've taken a brief look, and I'm not sure if you can hook similarly into /colormap name/.code. Simply trying \pgfplotsset{/pgfplots/cycle list name/.code={#1}} should presumably then work correctly, no? But alas, that doesn't seem to work.

If this would lead to a somewhat buggy implementation, working for some specifications and not for others, I'm hesitant to use it. At least I'm willing to sacrifice speed and wait a little longer if I'm sure it will work as usual (and I guess some other as well). Put differently, I don't want to put in too much time debugging things for this library. :) Perhaps another (simpler and more stable) option is to split the file in parts, so that you don't always load everything when you only need a single style.

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

Successfully merging this pull request may close these issues.

None yet

2 participants