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

Roadmap (or todo) #14

Open
3 of 13 tasks
ly29 opened this issue Nov 24, 2015 · 28 comments
Open
3 of 13 tasks

Roadmap (or todo) #14

ly29 opened this issue Nov 24, 2015 · 28 comments

Comments

@ly29
Copy link
Contributor

ly29 commented Nov 24, 2015

The system is basically working now. As #7 is closed it is time to look forward a bit. All of these deserves their own issues.

@ly29
Copy link
Contributor Author

ly29 commented Nov 24, 2015

To test.
Testing, easiest is to use git and then create a symlink in the addons folder pointing to the SverchokRedux folder.
There isn't much to do yet with the system.

@nortikin
Copy link
Member

but it not installing error in importing core/execute and compiler items from the same folder

PS

same problem was in initial Sverchok with dot import. dont remember how we solved

@nortikin
Copy link
Member

  File "/home/ololo/.config/blender/2.76/scripts/addons/SverchokRedux/core/compiler.py", line 1, in <module>
    from SverchokRedux.core.execute import GraphNode
  File "/home/ololo/.config/blender/2.76/scripts/addons/SverchokRedux/core/execute.py", line 3, in <module>
    from SverchokRedux.core.compiler import create_graph
ImportError: cannot import name 'create_graph'

@nortikin
Copy link
Member

maybe

sys.modules["SverchokRedux"] = sys.modules[__name__]

please
register in paths

@ly29
Copy link
Contributor Author

ly29 commented Nov 24, 2015

Ahh, will take of that, it seems does errors don't appear on reloading after doing the changes.

@ly29
Copy link
Contributor Author

ly29 commented Nov 24, 2015

Should work now

@ly29
Copy link
Contributor Author

ly29 commented Nov 24, 2015

I won't mess with the sys.modules again, I think a better distribution system is in order

@ly29
Copy link
Contributor Author

ly29 commented Nov 28, 2015

Tried to play a bit with opengl, but all the new functions where ´´´None´´´ in the bgl module. Didn't have time to look into that...
@zeffii Any idea?

@zeffii
Copy link
Contributor

zeffii commented Nov 28, 2015

I'll make time to check the bgl stuff :) haven' looked at new functions yet

@ly29
Copy link
Contributor Author

ly29 commented Nov 28, 2015

It is nifty, you give open gl vertices either as ordered triangles or as verts and topology. From a silly amount of function calls it becomes just a few.
And then a shader programs paints it.
We will need a good way to triangulate though.

@ly29
Copy link
Contributor Author

ly29 commented Nov 28, 2015

And giving means passing a memory buffer in one call.

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

Oh, I just did something stupid, now the functions show up.

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

got any code to experiment with?

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

@ly29 i know it's probably only cosmetic.. and your code isn't in a final state but I do read the code. And am wondering: https://gist.github.com/zeffii/7979b80c1dd61d3028c3 if that might have any effect on speed

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

@zeffii
Working on OpenGL code.

Yeah, thumbs from me about that change. Even thinking about turning it into a named tuple/class for easer use, for the links and the node. all the lookups are getting my nerves.

Link = namedtuple("Link", ["from_node", "from_socket", "to_node","to_socket"])

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

or even the shorter forms.

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

ideasman42 expressed an unfavourable opinion about the speed and implementation of namedtuple, quite some time ago -- i'm not sure if it is relevant here: http://blenderscripting.blogspot.nl/search/label/namedtuple

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

well if turns out to be a problem we can shift it along those lines. easy enough to do, the main thing I want is the interface of not having to look up things.

node=nodes[name]
node["bl_idname"]
# vs
node.bl_idname
# etc

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

yeah, massive headache those " " all the time. I agree.

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

I tried to, but I think I get the theory of the more modern open gl but the specifics are bit new. A working piece of code would nice.
Will hold of a bit with it.

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

Red book

@ly29
Copy link
Contributor Author

ly29 commented Nov 29, 2015

@zeffii

import bgl
bgl.glGenVertexArrays is None

is also true for you?

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

>>> bgl.glGenVertexArrays()
Traceback (most recent call last):
  File "<blender_console>", line 1, in <module>
TypeError: function takes exactly 2 arguments (0 given)

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

I don't think it's supposed to have any explicit return value, except None

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

I am interested to see what the bgl equivalent of https://gist.github.com/zeffii/131fb3a0f6fc3ca117fb is

@zeffii
Copy link
Contributor

zeffii commented Nov 29, 2015

see #16 for bgl talk

@ly29
Copy link
Contributor Author

ly29 commented Nov 30, 2015

A bit of a break, will be quite busy the next couple of days. Will try to advance a bit here and there anyway.

@ly29
Copy link
Contributor Author

ly29 commented Dec 13, 2015

###Update

So right now everything is kind broken, will focus on getting it to run again with consequences of the new type system. But will be day or two before I can do that.

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