Skip to content

Option: Change simulate to do the integration over model numbers in the bql function rather than the metamodel #286

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

Closed
axch opened this issue Oct 27, 2015 · 3 comments

Comments

@axch
Copy link
Contributor

axch commented Oct 27, 2015

Benefits:

  • Easy to generalize to accepting a range of model numbers
  • Simplifies metamodels a little

The usual reason to push things into the metamodel doesn't seem to apply, because I can't imagine anything smarter the metamodel could do than what the bql function would do.

@gregory-marton
Copy link
Contributor

Or a set of model numbers, perhaps identified by some sort of tag. See #313 .

@axch
Copy link
Contributor Author

axch commented Nov 19, 2015

Yes. Tag-based identification is orthogonal.

@fsaad
Copy link
Collaborator

fsaad commented Nov 13, 2017

Should be the case for most functions in bqlfn.py except:

  • dependence probability.
  • row similarity

fsaad pushed a commit that referenced this issue Nov 13, 2017
@fsaad fsaad closed this as completed in 499d140 Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants