Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Dec 02 2020 08:54
    samuelsinayoko opened #40
  • Aug 24 2020 07:48
    cool-RR opened #39
  • Aug 21 2018 21:43
    kdorr opened #38
  • Aug 21 2018 21:40
    kdorr opened #37
  • Aug 21 2018 21:40
    kdorr opened #36
  • Aug 21 2018 21:39
    kdorr opened #35
  • Aug 21 2018 18:50
    palnabarun synchronize #34
  • Aug 21 2018 18:28
    palnabarun synchronize #34
  • Aug 21 2018 04:36
    palnabarun synchronize #18
  • Aug 21 2018 01:27
    palnabarun opened #34
  • Aug 21 2018 01:24
    palnabarun assigned #33
  • Aug 21 2018 01:24
    palnabarun opened #33
  • Aug 20 2018 23:55
    kdorr synchronize #24
  • Aug 20 2018 23:52
    kdorr synchronize #26
  • Aug 20 2018 23:51
    kdorr synchronize #23
  • Aug 20 2018 23:47
    kdorr synchronize #25
  • Aug 20 2018 22:22
    kdorr synchronize #22
  • Aug 20 2018 21:49
    kdorr synchronize #24
  • Aug 20 2018 21:36
    kdorr synchronize #26
  • Aug 20 2018 00:03
    story645 closed #21
hannah
@story645
also a reminder that mpl-altair is Python 3 only
Nabarun Pal
@palnabarun
Hi, on a close look at the master branch, I saw that most of the code implemented in palnabarun/encodings is not there in mpl-altair/master . Is it supposed to be like that? Also, the same is true about some of the tests.
Some files are missing too, like _utils.py and tests/test_utils.py
kdorr
@kdorr

There were a lot of merge conflicts because of the two different approaches. A few meetings ago, I think we said we wanted to use the encoding conversion part from palnabarun/encodings and the data extraction/chart reading part from kdorr/convert-temporal and kdorr/convert-numeric, which is what I tried to do in the merge.

As for _utils.py and test_utils.py, since the way that kdorr/convert-temporal and ‘kdorr/convert-numeric` extract the data from the chart didn’t require reading any urls, the functions in _utils.py weren’t being used after the merge.

Nabarun Pal
@palnabarun
Ah. Okay.
But _utils.py had code to handle cases where the user gave an online dataset as a source.
kdorr
@kdorr
Oops! You're right. I overlooked that way to specify data in an Altair chart.
Sorry about that! We should put _utils.py back.
Nabarun Pal
@palnabarun
I will add that code in a PR. Don't worry.
And integrate with the way currently data is being handled.
hannah
@story645
meet.google.com/tje-udbm-cpu
sorry forgot to add it on the invite
hannah
@story645
@kdorr don't want to sidetrack you, but the powernorm code might be in your wheelhouse ( or throw a not implemented error for now) https://github.com/matplotlib/matplotlib/blob/cf38546d06e53e640215fdba1105ba8e5fd5d3b8/lib/matplotlib/colors.py#L1169
kdorr
@kdorr
Thanks! I'll look into it
hannah
@story645
it very possibly would have been quicker/easier to set up the docs using sphinx, but you've got a page now: https://matplotlib.org/mpl-altair/
the sites on https://github.com/matplotlib/mpl-altair/tree/gh-pages
and you update it by making a new file in https://github.com/matplotlib/mpl-altair/tree/gh-pages/_posts (use the existing one as a template)
technically speaking, this theme supports a project logo...
will also send this info in an email
kdorr
@kdorr
I’ve been looking into why the travis build is breaking for PR #12 with little success. All of the failing tests use pytest-mpl and are passing on my local machine (same version of Matplotlib, pytest, pytest-mpl). Could it be a travis configuration thing? Also, is it possible to see the images produced by the failing pytest-mpl tests in the travis build?
I meant PR #16
hannah
@story645
@kdorr do you know what version of freetype you're using on your machine?
that's usually what breaks these sorts of things
also can you ask on the matplotlib channel? I think there is a way to see the images but I don't quite know how
kdorr
@kdorr
I'm using 2.8.0 of freetype
hannah
@story645
yeah, it's using freetype 2.6 - I think @tacaswell plans to sort this out on the travis side
Thomas A Caswell
@tacaswell
there is an ENV you can set when installing matplotlib which will locally install the correct version, but I think you have to install mpl from source for that to work
There may be a conda-forge package with the correct dependancies
hannah
@story645
https://github.com/matplotlib/matplotlib/blob/master/setup.cfg.template
Nabarun Pal
@palnabarun
Hi
What is the link to the call?
hannah
@story645
@kdorr try export MPLLOCALFREETYPE=1 and python setup.py build_ext --inplace
create a new conda env, run git clean to remove build artifacts
and try then the export and the above setup command
Nabarun Pal
@palnabarun

  • [ ] axis

    • [x] Disable DefaultConfig Overrides
    • [ ] axis.values
    • [ ] axis.tickColor Not Supported by altair

    • [ ] axis.tickOpacity Not Supported by altair

    • [x] axis.labelAngle

    • [ ] axis.labelColor Not Supported by altair
    • [ ] axis.labelOpacity Not Supported by altair
    • [ ] axis.tickCount Not applicable to non-quant scales
    • [x] axis.labels
    • [x] axis.orient
Thomas A Caswell
@tacaswell
I am having trouble finding a link for the call today...
Nabarun Pal
@palnabarun
Me too.
kdorr
@kdorr
Yep
Nabarun Pal
@palnabarun
Shall I create one?
Thomas A Caswell
@tacaswell
yes please
Nabarun Pal
@palnabarun
This message was deleted
meet.google.com/gwj-rpxd-ixx
Nabarun Pal
@palnabarun
https://gist.github.com/palnabarun/dbe3e5525e5a84263fc0b62a0be255cd
https://gist.github.com/palnabarun/4a7a12563ad1c07eb107db8ee1a89bc5
kdorr
@kdorr
https://gist.github.com/kdorr/1c13edb8d35d45738e83b3e7fb09cd03
Nabarun Pal
@palnabarun
https://317470be.rorocloud.io/lab/tree/working_draft.ipynb?token=rorocloud
hannah
@story645
https://matplotlib.org/mpl-altair/
kdorr
@kdorr

I’ve had an idea to create a class that would allow the data/metadata from the altair chart to be parsed and stored in an object. This seems a little cleaner to me than our current implementation, which is to re-parse a portion of the chart for each channel in each step of the conversion.

I’m thinking it would look something like this: https://gist.github.com/kdorr/bff03aee0593102c46fa4568568b0076

It would simplify convert_axis() (https://github.com/matplotlib/mpl-altair/blob/master/mplaltair/_axis.py#L222), _convert() (https://github.com/matplotlib/mpl-altair/blob/master/mplaltair/_convert.py#L121), and current/future implementations of different chart types.

However, it’s currently possible to add other plot types without this and lot of the current codebase would need to be tweaked to work with the change (namely _axis.py and _convert.py). It would probably take me .5-1.5 days to implement it.

Could I get feedback on this? Is it something that would be useful to work on? If so, with the limited time left this summer, is it worth pursuing right now?

hannah
@story645
Hi, sorry just saw this now (my notifications have been super wonky)
I think it's a really good idea and worth doing now before even more code is built around the other implementation, but I also want to know if @tacaswell has concerns about this approach