Mailman 3 Re: [yt-dev] Issue #1047: Radial Column Density (yt_analysis/yt) - yt-dev

Re: [yt-dev] Issue #1047: Radial Column Density (yt_analysis/yt)

Bili Dong

23 Jul 2015 23 Jul '15

10:07 a.m.

Hi all, I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option? Regards Bili

Show replies by date

Bili Dong

23 Jul 23 Jul

10:19 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Here’s the link to the issue https://bitbucket.org/yt_analysis/yt/issues/1047/radial-column-density. Sorry for not including it. Regards, Bili

...

On Jul 23, 2015, at 10:07 AM, Bili Dong wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

Nathan Goldbaum

11:17 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Healpy is GPL, so that won't work. One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package. On Thursday, July 23, 2015, Bili Dong wrote:

...

Here’s the link to the issue https://bitbucket.org/yt_analysis/yt/issues/1047/radial-column-density. Sorry for not including it.

Regards, Bili

On Jul 23, 2015, at 10:07 AM, Bili Dong javascript:_e(%7B%7D,'cvml','qobilidop@gmail.com');> wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

Bili Dong

11:38 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right? To keep the project in BSD, I’m thinking about adding a parameter to the class RadialColumnDensity. By using that parameter, the user can feed the healpix functionality to the class. Is that possible? -Bili

...

On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package.

On Thursday, July 23, 2015, Bili Dong mailto:qobilidop@gmail.com> wrote: Here’s the link to the issue https://bitbucket.org/yt_analysis/yt/issues/1047/radial-column-density. Sorry for not including it.

Regards, Bili

...
On Jul 23, 2015, at 10:07 AM, Bili Dong javascript:_e(%7B%7D,'cvml','qobilidop@gmail.com');> wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

Bili Dong

11:55 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Another option I can think about is to reimplement the needed healpix functionality from scratch. Since only the coordinates transformation part are involved here, it would be some reasonable amount of work. But I’m not sure if only using the algorithm resolves the license problem, does it? -Bili

...

On Jul 23, 2015, at 11:38 AM, Bili Dong wrote:

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right?

To keep the project in BSD, I’m thinking about adding a parameter to the class RadialColumnDensity. By using that parameter, the user can feed the healpix functionality to the class. Is that possible?

-Bili

...
On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum mailto:nathan12343@gmail.com> wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package.

On Thursday, July 23, 2015, Bili Dong mailto:qobilidop@gmail.com> wrote: Here’s the link to the issue https://bitbucket.org/yt_analysis/yt/issues/1047/radial-column-density. Sorry for not including it.

Regards, Bili

...
On Jul 23, 2015, at 10:07 AM, Bili Dong javascript:_e(%7B%7D,'cvml','qobilidop@gmail.com');> wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org mailto:yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

Matthew Turk

24 Jul 24 Jul

12:54 p.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Hi Bili, I think it depends. But, there are likely other ways to get this functionality implemented; for now, even just using something like HTM would work (even though it's not equal area) and then replace it later once we have licensing figured out. -Matt On Thu, Jul 23, 2015 at 1:55 PM, Bili Dong wrote:

...

Another option I can think about is to reimplement the needed healpix functionality from scratch. Since only the coordinates transformation part are involved here, it would be some reasonable amount of work.

But I’m not sure if only using the algorithm resolves the license problem, does it?

-Bili

On Jul 23, 2015, at 11:38 AM, Bili Dong wrote:

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right?

To keep the project in BSD, I’m thinking about adding a parameter to the class RadialColumnDensity. By using that parameter, the user can feed the healpix functionality to the class. Is that possible?

-Bili

On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package.

On Thursday, July 23, 2015, Bili Dong wrote:

...
Here’s the link to the issue. Sorry for not including it.

Regards, Bili

On Jul 23, 2015, at 10:07 AM, Bili Dong wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

Bili Dong

25 Jul 25 Jul

11:41 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Hi Matt, Thanks for your suggestion. It’s a good idea to start with HTM and see how it works. I have a further question. From my observation, the healpix related functionality are mainly implemented in yt.utilities.lib.grid_traversal as several functions and in yt.visualization.volume_rendering.camera as several classes. When I’m implementing the HTM functionality, should I imitate that? Or should I start with implementing those methods locally within the radial_column_density submodule? -Bili

...

On Jul 24, 2015, at 12:54 PM, Matthew Turk wrote:

Hi Bili,

I think it depends. But, there are likely other ways to get this functionality implemented; for now, even just using something like HTM would work (even though it's not equal area) and then replace it later once we have licensing figured out.

-Matt

On Thu, Jul 23, 2015 at 1:55 PM, Bili Dong wrote:

...
Another option I can think about is to reimplement the needed healpix functionality from scratch. Since only the coordinates transformation part are involved here, it would be some reasonable amount of work.

But I’m not sure if only using the algorithm resolves the license problem, does it?

-Bili

On Jul 23, 2015, at 11:38 AM, Bili Dong wrote:

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right?

To keep the project in BSD, I’m thinking about adding a parameter to the class RadialColumnDensity. By using that parameter, the user can feed the healpix functionality to the class. Is that possible?

-Bili

On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package.

On Thursday, July 23, 2015, Bili Dong wrote:

...
Here’s the link to the issue. Sorry for not including it.

Regards, Bili

On Jul 23, 2015, at 10:07 AM, Bili Dong wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

Matthew Turk

11:42 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Hi Bili, I think it would be best to keep similar API structures -- that way we can use HTM functionality elsewhere, too! -Matt On Sat, Jul 25, 2015 at 1:41 PM, Bili Dong wrote:

...

Hi Matt,

Thanks for your suggestion. It’s a good idea to start with HTM and see how it works.

I have a further question. From my observation, the healpix related functionality are mainly implemented in yt.utilities.lib.grid_traversal as several functions and in yt.visualization.volume_rendering.camera as several classes. When I’m implementing the HTM functionality, should I imitate that? Or should I start with implementing those methods locally within the radial_column_density submodule?

-Bili

...
On Jul 24, 2015, at 12:54 PM, Matthew Turk wrote:

Hi Bili,

I think it depends. But, there are likely other ways to get this functionality implemented; for now, even just using something like HTM would work (even though it's not equal area) and then replace it later once we have licensing figured out.

-Matt

On Thu, Jul 23, 2015 at 1:55 PM, Bili Dong wrote:

...
Another option I can think about is to reimplement the needed healpix functionality from scratch. Since only the coordinates transformation part are involved here, it would be some reasonable amount of work.

But I’m not sure if only using the algorithm resolves the license problem, does it?

-Bili

On Jul 23, 2015, at 11:38 AM, Bili Dong wrote:

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right?

To keep the project in BSD, I’m thinking about adding a parameter to the class RadialColumnDensity. By using that parameter, the user can feed the healpix functionality to the class. Is that possible?

-Bili

On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt package.

On Thursday, July 23, 2015, Bili Dong wrote:

...
Here’s the link to the issue. Sorry for not including it.

Regards, Bili

On Jul 23, 2015, at 10:07 AM, Bili Dong wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem is to find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

Bili Dong

11:51 a.m.

New subject: Issue #1047: Radial Column Density (yt_analysis/yt)

Okay, I'll do it that way. -Bili On Jul 25, 2015 11:42 AM, "Matthew Turk" wrote:

...

Hi Bili,

I think it would be best to keep similar API structures -- that way we can use HTM functionality elsewhere, too!

-Matt

...
Hi Matt,

Thanks for your suggestion. It’s a good idea to start with HTM and see how it works.

I have a further question. From my observation, the healpix related functionality are mainly implemented in yt.utilities.lib.grid_traversal as several functions and in yt.visualization.volume_rendering.camera as several classes. When I’m implementing the HTM functionality, should I imitate that? Or should I start with implementing those methods locally within the radial_column_density submodule?

-Bili

...
On Jul 24, 2015, at 12:54 PM, Matthew Turk wrote:

Hi Bili,

I think it depends. But, there are likely other ways to get this functionality implemented; for now, even just using something like HTM would work (even though it's not equal area) and then replace it later once we have licensing figured out.

-Matt

On Thu, Jul 23, 2015 at 1:55 PM, Bili Dong wrote:

...
Another option I can think about is to reimplement the needed healpix functionality from scratch. Since only the coordinates transformation

...
...
...
are involved here, it would be some reasonable amount of work.

But I’m not sure if only using the algorithm resolves the license

...
...
...
does it?

-Bili

On Jul 23, 2015, at 11:38 AM, Bili Dong wrote:

I see where the problem is. And since the whole HEALPix stuff is GPL. It’s even not possible to reimplement something like healpy, right?

To keep the project in BSD, I’m thinking about adding a parameter to

...
...
...
class RadialColumnDensity. By using that parameter, the user can feed

...
...
...
healpix functionality to the class. Is that possible?

-Bili

On Jul 23, 2015, at 11:17 AM, Nathan Goldbaum wrote:

Healpy is GPL, so that won't work.

One way around this would be to implement the functionality you are concerned about in a new GPL package, split off from the main yt

On Sat, Jul 25, 2015 at 1:41 PM, Bili Dong wrote: part problem, the the package.

...
...
...
On Thursday, July 23, 2015, Bili Dong wrote:

...
Here’s the link to the issue. Sorry for not including it.

Regards, Bili

On Jul 23, 2015, at 10:07 AM, Bili Dong wrote:

Hi all,

I’m bringing the update of the discussion here. The current problem

is to

...
...
find a workaround for the code removed for licensing concerns. Could healpy could be an option?

Regards Bili

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

3195

Age (days ago)

3197

Last active (days ago)

List overview

Download

8 comments

3 participants

participants (3)

Bili Dong
Matthew Turk
Nathan Goldbaum

Re: [yt-dev] Issue #1047: Radial Column Density (yt_analysis/yt)

tags

participants (3)