rodluger / Limbdark.jl

Analytical transit light curves for limb darkened stars
MIT License
14 stars 4 forks source link

Scaling of timing on different processors and implementations #40

Closed ericagol closed 6 years ago

ericagol commented 6 years ago
julia> @time timing=profile_transit_poly(trans,nb);
306.725521 seconds (460 allocations: 22.321 MiB, 0.00% gc time)
ericagol commented 6 years ago

Perhaps the problem with v0.7.0 is due to the factorial issue?

rodluger commented 6 years ago

@ericagol Did you fix this? The profile_transit_poly is running in the correct amount of time with 0.7.0 on the cleanup branch.

ericagol commented 6 years ago

@rodluger Yes, I replaced lfact with lfactorial, and fixed the other deprecations such as 1.+/-.

ericagol commented 6 years ago

@rodluger But, there are some other warnings that occurring that I need to fix before we close this issue.

rodluger commented 6 years ago

OK. I'm working on the cleanup branch as well, so there may be some merge conflicts we'll have to resolve when we try to push.

ericagol commented 6 years ago

@rodluger It also looks like there are some conflicts between the master and cleanup branches.

ericagol commented 6 years ago

@rodluger I've now updated the cleanup branch code and tests, and all of the tests seem to work now, both in Julia v0.7 and v1.0!

rodluger commented 6 years ago

Cool. Can you push? I've been editing those files as well.

rodluger commented 6 years ago

I'm going to set them up to run on Travis automatically.

ericagol commented 6 years ago

Yes, for some reason my first push didn't work; now I think it has (github asked me to log in again, for some reason).