Skip to content
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

When using image plane priors, I think it would be nice to give the source positions on source plane #250

Open
lirui10590 opened this issue Jan 7, 2017 · 2 comments
Labels

Comments

@lirui10590
Copy link

When I use image plane priors for the background source positions, it sees that the output parameter of the source position are on the image plane too. I think it would be nice to give the positions on source plane.

@ntessore
Copy link
Contributor

ntessore commented Jan 8, 2017

cc @fabiobg83 @rbmetcalf

I think that the output parameter in the posterior should always match the prior input parameter, so I would be against replacing the image plane output with a source plane output. However, I think it would be possible to introduce additional derived parameter that contain the source plane values.

@ntessore ntessore added the idea label Jan 8, 2017
@lirui10590
Copy link
Author

introducing additional derived parameter is a good way ,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants