-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
winston backend #18
Comments
This is an excellent start... thanks! I mainly didn't start with dictionary mappings because
For this case, it seems like a dict should work fine. You should create a pull request for this on the |
Feel free to copy and paste. That will likely be easier for you too. As for checking, I just meant to check that a proper value was passed in so that in the backend the value passed as a key is known to be correct. But you are right, there are likely values that depend on the underlying package being loaded. |
One of the things I'm working on now is a more complete/robust approach for On Wed, Sep 16, 2015 at 10:44 AM, john verzani notifications@github.com
|
Lets leave it open... I'll close it when I'm done with Winston support. |
Pretty good progress! Thanks again for the head start. Check out the Winston examples and if you're curious where I'm going with the backend code, look here |
Nice. You are like a coding machine-- one new backend per day before lunch! On Wed, Sep 16, 2015 at 5:21 PM, Tom Breloff notifications@github.com
John Verzani |
+1 to coding machine |
Heh... Just trying to make sure the skeptics get a good first impression. I don't want people to brush it off because "you don't support XYZ so I can't use it". It's pretty easy to add features and backends now that the framework is somewhat stable.
|
@jverzani I'm curious... how much do you care about Winston as a backend? It's fallen a bit behind in the dev cycle, and I'm not sure I see a clear reason to use it over, say, Immerse. Would anyone be disappointed if I deprecate it at the same time I stop supporting 0.3? |
I don't actually use it personally, but for a little side project On Sun, Oct 18, 2015 at 1:10 AM, Tom Breloff notifications@github.com
John Verzani |
Okay, adding Immerse support to my project was easy. Deprecating Winston support won't cause any concerns on my end. |
Ok thanks. If anyone else has an opinion on Winston, please chime in. The code in Immerse is very similar (it might have even been copied directly?) and I think it's a better backend. I want to start supporting Vega and GLPlot, and eventually Qt5, which all offer some different strengths, so the fewer distractions the better. Also I may deprecate Qwt as well once Qt5 becomes usable.
|
use version number instead of release in .travis.yml
* add debug and error stuff * fix docstring * Update src/pipeline.jl
I didn't put this in a pull request, as it isn't complete, but here is a start for a Winston backend. Basic things work. A couple thoughts:
plot!
command. One benefit is that it would be easier to check if a backend supports a certain type of plot. The various types will only increase.Anyways, for what it is worth, feel free to use this as you see fit.
The text was updated successfully, but these errors were encountered: