-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improved Grid_points and rand_p #50
Open
ryboselm
wants to merge
35
commits into
JoeyT1994:main
Choose a base branch
from
ryboselm:grid_points
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 31 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
6e19de0
Gridpoint improvement (#1)
ryboselm e87ea6d
Add RNG passing to rand_p
ryanlevy f4032a2
add default rng tests
ryboselm 5d3c2a2
Merge pull request #2 from ryboselm/rng
ryboselm f85ab83
Methods to convert Data and Functions into QTT format (#3)
ryboselm ec1333a
get rid of warning about redefining default_dimension()
ryboselm 87c7477
Merge branch 'main' into dev
ryboselm 3a966d4
update interpolation functions to take complex coefficients
ryboselm e4504b9
remove coeffs as an output of function_itn
ryboselm 78e4eea
remove coefficients output from examples
ryboselm 58a789a
added clenshaw interpolation (#5)
ryboselm 0cd8f57
clean up old examples
ryboselm e86726c
Merge remote-tracking branch 'upstream/main' into dev
ryboselm 620d7cc
stop tracking config files
ryboselm 64c73ed
oops add back .gitignore
ryboselm 22c3707
formatting
ryboselm 62d23f7
test case
ryboselm f03c518
formatting
ryboselm bd4e0d4
added back grid_points tests, made naming of args more consistent, cl…
ryboselm 050cb3e
added functional grid_points and rand_p functions with passing tests
ryboselm bac29f1
formatting
ryboselm 3e37e32
add test cases for interpolation_functions
ryboselm 554100b
added dimension specification inside interpolation_functions and made…
ryboselm 28ad64d
removed interpolation changes
ryboselm 7b1ea30
Merge branch 'main' into grid_points
ryboselm fd31bb2
add GraphRecipes dep
ryboselm 108d3ab
formatting
ryboselm 8e319c2
re-organize test sets for indexmaps
ryboselm f81e4bc
multi-dimensional grid_points
ryboselm 3923d75
simplify grid_points exact_grid condition significantly
ryboselm ddbdc33
formatting
ryboselm 5743314
clean up grid_points and rand_p and their tests
ryboselm f942557
formatting
ryboselm e157242
generalize round_to_nearest_exact_point and added complexindexmap ran…
ryboselm 1b64e12
formatting
ryboselm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should also add support for multidimensional grid points.
Like we have a function where we pass a vector of
dimensions
and then zip up the result ofgrid_points
called on each of the dimensionsThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you elaborate more on what your idea for this would look like?
for example, if you passed in a vector
dimensions = [1,3]
, and suppose thatgrid_points(s,1) = [0, 1]
andgrid_points(s,3) = [0.3, 0.4, 0.5]
, wouldgrid_points(s, [1, 3])
then be something like[[0, 0.3], [0, 0.4], [0, 0.5], [1, 0.3], [1, 0.4], [1, 0.5]]
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes so if for each dimension
I
you input you generate some vector ofNi
points then it would return the iterative product which consists of every unique combination of points:e.g.
[0.25, 0.5], [0.625, 0.975, 0.9875] -> [(0.25, 0.625), (0.25, 0.975),(0.25, 0.9875), (0.5, 0.625), (0.5, 0.975), (0.5, 0.9875)]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it be better to return an array of tuples or an array of arrays here?
I initially leaned towards having an array of arrays because
evaluate
takes in an array to specify the point it should evaluate the function at.