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

Fix QR method #16

Closed
jirihnidek opened this issue Feb 6, 2017 · 1 comment
Closed

Fix QR method #16

jirihnidek opened this issue Feb 6, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@jirihnidek
Copy link
Contributor

Make it possible to use QR method again.

@jirihnidek jirihnidek added the bug label Feb 6, 2017
@jirihnidek jirihnidek added this to the Version 0.0.5 milestone Feb 6, 2017
@jirihnidek jirihnidek self-assigned this Feb 6, 2017
@jirihnidek
Copy link
Contributor Author

The program is terminated with this error:

Creating B matrix ...
Traceback (most recent call last):
  File "./src/main.py", line 51, in <module>
    main(parse_arguments())
  File "./src/main.py", line 29, in main
    terrain.approximate_terrain()
  File "/home/jiri/GitHub/bapprox/src/terrain_data.py", line 307, in approximate_terrain
    raw, diffs = approx.terrain.approx(solver_method, terrain, u_knots, v_knots, sparse=sparse)
  File "/home/jiri/GitHub/bapprox/src/approx/terrain.py", line 882, in approx
    return approx_qr(terrain_data, u_knots, v_knots, sparse)
  File "/home/jiri/GitHub/bapprox/src/approx/terrain.py", line 607, in approx_qr
    b_mat, interval = build_ls_matrix(u_knots, v_knots, terrain_data, True)
  File "/home/jiri/GitHub/bapprox/src/approx/terrain.py", line 239, in build_ls_matrix
    u_base_vec, i_idx = spline_base_vec(u_knots, terrain[idx, 0], 0, sparse)
  File "/home/jiri/GitHub/bapprox/src/approx/terrain.py", line 132, in spline_base_vec
    idx = find_index(knot_vec, t_param)
  File "/home/jiri/GitHub/bapprox/src/approx/terrain.py", line 105, in find_index
    if _t_param >= _knot_vec[est]:
IndexError: index -8207998 is out of bounds for axis 0 with size 15

jirihnidek pushed a commit that referenced this issue Feb 8, 2017
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

1 participant