StellarCartography / pydis

DEPRECATED: Check out PyKOSMOS!
https://github.com/jradavenport/pykosmos
MIT License
43 stars 27 forks source link

Clarification in docuemntation #38

Closed solontoi closed 9 years ago

solontoi commented 9 years ago

I've gotten the autoreduce wrapper to work with my low resolution asteroid data, works really well. There needs to be a clarification on the wiki/instructions as written though, once you've added the pydis directory to the sys.path you should call autoreduce with

import wrappers

and then

wrappers.autoreduce(obj...

since autoreduce is defined in wrappers.py and not pydis.py

jradavenport commented 9 years ago

hm weird... here's what I do:

import pydis
pydis.autoreduce

because autoreduce is exposed in __init__.py

Maybe my sys.path hack is wrong in the instructions...

jradavenport commented 9 years ago

By the way: I'd love to see some of the results you had using it, or any comments on the instructions/results/etc!

solontoi commented 9 years ago

Might be it - here's what happens when I try running autoreduce from pydis and then from wrappers: out1

I'll post my dodgy asteroid spectra in a moment.

jradavenport commented 9 years ago

... hm, what happens if you try this?

import pydis-master
solontoi commented 9 years ago

Python doesn't like me trying to subtract master from pydis :) I solved the issue - copied everything into a directory named pydis inside my observing folder (same structure as before), and ran the following: temp2

Since I'm still getting my python feet under me I'm not sure if that wording of your sys.path line was confusing for anyone but me.

jradavenport commented 9 years ago

closing as "good enough"

:)