- Inkscape Devel - Inkscape Mailman

Coding workflow and reading practices
by Jon Cruz 02 Aug '13

02 Aug '13

Hi Johan, In the thread on GObject instantiation you happened to raise an interesting issue that probably warrants some good discussion. You had mentioned that adding more to the headers "Saves some time wading through lengthy .cpp files only to find an almost empty function". Do you think you can expand a little on your use case where this type of practice is an issue? That is, can you give a quick description of some of the steps you would be going through when this problem pops up? I know that different people have different learning and processing styles. (As an extreme example I've even worked with someone whose workflow was impacted by a change in Emacs that limited the number of open windows to only a couple hundred.) It would be good to be sure that we try to get things structured well for easier maintenance, etc. Thanks.

1 0

GObject object instantiation problem
by Sebastian Götte 02 Aug '13

02 Aug '13

Hi, I'm currently having some problems allocating GObjects. I just added a class called SPPoint [0]. When I try to instantiate an object of that type (whether through SPObject/sp_object_child_added or manually from GDB using g_object_new does not matter), g_object_new returns an GObject* that contains a zero class: {g_type_instance = {g_class = 0x0}, ref_count = 0, qdata = 0x0} (this is not good as it breaks everything else about G_OBJECT). I was able to track down the problem to g_type_create_instance returning that when passed the new SPPoint type: (gdb) print sp_object_get_type() $13 = 20593552 (gdb) print sp_point_get_type() $14 = 20619024 (gdb) print *((GObject*)g_type_create_instance(sp_point_get_type())) $20 = {g_type_instance = {g_class = 0x0}, ref_count = 0, qdata = 0x0} (gdb) print *((GObject*)g_type_create_instance(sp_object_get_type())) $21 = {g_type_instance = {g_class = 0x16ec560}, ref_count = 1, qdata = 0x0} I'm currently building a glib with debugging symbols to further track down this problem. If anybody here has an idea about what might be the cause of this problem, your suggestions will likely be tremendously useful since I'm already pondering over this for a few hours. The code can be found at [1]. Thanks, Sebastian [0] https://bazaar.launchpad.net/~h-e-6/inkscape/connector-wip/view/head:/src/s… [1] https://code.launchpad.net/~h-e-6/inkscape/connector-wip

5 16

[Units] Completion of Phase One
by Matthew Petroff 01 Aug '13

01 Aug '13

I have now eliminated "unit-constants.h", removing the last of the duplicate general unit code. The one remaining section of somewhat duplicate unit code is the "SVGLength" class, but that is specific to the SVG specification, not general units. I plan on leaving it alone for now but may make it a child class of "Util::Unit" during phase two. I have submitted a merge proposal for my branch [1]. I plan on beginning phase two by hand-crafting SVG files with various types of unit configurations and work from there. http://www.mpetroff.net/archives/2013/08/01/completion-of-phase-one/ -Matthew [1] https://code.launchpad.net/~matthewpetroff/inkscape/gsoc-2013-unit-refactor…

2 1

Re: [Inkscape-devel] RDF nodes
by Markus Engel 01 Aug '13

01 Aug '13

I don't seem to get what you mean by "values" and "building blocks"? Is it necessary to create a class just holding a key and a value? Wouldn't a std::map suffice? 1) The SPObject class exposes a method "connectModified" which you can use to register a sigc::slot with the modification signal. This should work I hope. Or look at "src/ui/dialog/document-metadata.cpp". Inside this file there's a method called "on_repr_attr_changed": /** * Called when XML node attribute changed; updates dialog widgets. */ I don't know why this does not work currently. 2) Sorry? When the xml tree is modified, the corresponding SPObject is automatically informed, I don't think you have to do anything there yourself. When you want to update the xml node after you made a change to an SPObject, just call the modified SPObject's updateRepr method. 3) Yes, I think so. The SPObject will stay, not as a struct but as a real class ;) . For the moment you'll have to get into creating a derived GObject "class". There is an example in sp-skeleton.cpp. I'm just waiting for the 0.49 release to propose for merging, as I made some big changes to the code base with probably some nasty bugs. -----Ursprüngliche Nachricht----- Von: inkscape-devel-request(a)lists.sourceforge.net [mailto:inkscape-devel-request@lists.sourceforge.net] Gesendet: Mittwoch, 31. Juli 2013 14:59 An: inkscape-devel(a)lists.sourceforge.net Betreff: Inkscape-devel Digest, Vol 86, Issue 47 Send Inkscape-devel mailing list submissions to inkscape-devel(a)lists.sourceforge.net To subscribe or unsubscribe via the World Wide Web, visit https://lists.sourceforge.net/lists/listinfo/inkscape-devel or, via email, send a message with subject or body 'help' to inkscape-devel-request(a)lists.sourceforge.net You can reach the person managing the list at inkscape-devel-owner(a)lists.sourceforge.net When replying, please edit your Subject line so it is more specific than "Re: Contents of Inkscape-devel digest..." Today's Topics: 1. Re: [CAD] Next steps, a trivia question and discussion of the semantics for what objects a connector should avoid (Sebastian G?tte) 2. Re: [Tracing] Report #6 (Vin?cius dos Santos Oliveira) 3. GObject object instantiation problem (Sebastian G?tte) 4. Re: GObject object instantiation problem (Jon Cruz) 5. Re: GObject object instantiation problem (Sebastian G?tte) 6. Re: GObject object instantiation problem (Tavmjong Bah) 7. Re: RDF nodes (Christoffer Holmstedt) ---------------------------------------------------------------------- Message: 1 Date: Tue, 30 Jul 2013 21:20:42 +0200 From: Sebastian G?tte <jaseg@...2974...> Subject: Re: [Inkscape-devel] [CAD] Next steps, a trivia question and discussion of the semantics for what objects a connector should avoid To: inkscape-devel(a)lists.sourceforge.net Message-ID: <51F8120A.6060506@...2974...> Content-Type: text/plain; charset=ISO-8859-1 On 07/30/2013 01:57 AM, Michael Wybrow wrote: > Hi Sebastian, > > A couple of quick thoughts: > - You won't get the behaviour of avoiding specific obstacles with different connectors using libavoid, since it routes connectors using a single visibility graph built from the obstacles for the whole diagram. Ok, I think that's fine. The only use case I can imagine is something like PCB routing where you could want one visibility graph per layer. > - I think it would be fine to mark objects as obstacles using something other than a property. > - The reason for not avoiding all obstacles by default is that libavoid can be slow -- O(n^2 log n) -- especially if there are a lot of obstacles. You don't want this slowness if you open a file with hundreds or thousands of objects that libavoid tracks as objects by default. That was the reason for the original decision to have the user explicit mark objects as obstacles for routing. Shouldn't this be in O(n log n)? >> I'm now moving on to implement (somewhat minimalistic) "points" as described in [0] so I can then head towards renovating connector-context.cpp. My latest commit to launchpad breaks in an assertion when drawing a connector since it needs these changes. The commit before that should work, though. >> >> These "points" will be placed whenever one places a connector's end, >> except if one places a connector's end on an existing "point". When >> drawing a connector, all visible points will be rendered as knots. >> This enables template-based diagram elements which are just are >> groups that contain a shape and one or more points for the connector >> to attach > > FWIW, newer versions of libavoid understand and track "connection pins". These are points specified based on a position on shapes. They move if the parent shape is moved and can be specified as the endpoints for connectors. You might want to make of these. Thanks. This behavior can easily be implemented by putting a few SPPoints into a group with some shape. The new connector code just creates these SPPoints as needed, and handling of object centers/corners/nodes etc. should be done by just snapping their locations to these points. > It could be worth updating the version of libavoid in Inkscape to the latest version found here: > https://github.com/mjwybrow/adaptagrams > The only real work there would be how you find and track attached objects once using connection pins. By moving to SPPoints this is actually pretty non-problematic since they will just move with the groups containing them. > I think there was a previous dev who started doing the connection pin work in Inkscape but I'm not sure what happened to that. It doesn't seem to be on the trunk. > >> I will remove any logic from connector-context responsible for handling anything but two endpoints (if you want to draw a connector with multiple "stops", just chain together multiple connectors) and remove the "snap connector end point to connector's intersection with destination shape outline"-logic since I do not think this is too useful. > > Do you mean the functionality that adjusts the endpoint of a connector to stop at the intersection with the destination shape? If so, the reason for this is that the arrowhead is drawn at the end of the connector and people frequently want this to coincide with the edge of the shape. I removed that because it would have been even more work porting and because with the addition of SPPoints it is not really necessary anymore (I am not totally against re-adding it, though). Best regards, Sebastian [1] http://dev.w3.org/SVG/modules/connector/SVGConnector.html ------------------------------ Message: 2 Date: Tue, 30 Jul 2013 18:02:00 -0300 From: Vin?cius dos Santos Oliveira <vini.ipsmaker@...400...> Subject: Re: [Inkscape-devel] [Tracing] Report #6 To: Inkscape-dev <inkscape-devel(a)lists.sourceforge.net> Message-ID: <1375218120.1439.6.camel@...16...> Content-Type: text/plain; charset="utf-8" Em Ter, 2013-07-30 ?s 04:23 -0300, Vin?cius dos Santos Oliveira escreveu: > I'll upload a short video later today showing the project summary. Video uploaded: http://vimeo.com/71364024 -- Vin?cius dos Santos Oliveira https://about.me/vinipsmaker

2 1

Re: [Inkscape-devel] GObject removal branch
by Markus Engel 31 Jul '13

31 Jul '13

> Yeah, refactoring in a branch for too long is nuts. I agree with Josh that it's fine to merge into trunk if you think it's ready. For you, I think the sooner the better. Well, I think it is. Sometimes the factory issues a warning about rdf:RDF nodes, but as far as I know, Christoffer Holmstedt is currently working on license management and metadata handling. This has no effect on the functionality of the metadata dialogs, though. Regards, Markus Btw: Sorry if you received this message twice, I think I'll have to alter my mailing list subscription settings...

1 0

Re: [Inkscape-devel] Inkscape-devel Digest, Vol 86, Issue 49
by Markus Engel 31 Jul '13

31 Jul '13

> Yeah, refactoring in a branch for too long is nuts. I agree with Josh that it's fine to merge into trunk if you think it's ready. For you, I think the sooner the better. Well, I think it is. Sometimes the factory issues a warning about rdf:RDF nodes, but as far as I know, Christoffer Holmstedt is currently working on license management and metadata handling. This has no effect on the functionality of the metadata dialogs, though. Regards, Markus

1 0

Re: [Inkscape-devel] GObject object instantiation problem
by Markus Engel 31 Jul '13

31 Jul '13

> However, using it would require you to rewrite the init functions of all SPObjects. ... > Replacing GObject-style initialization with constructor / destructor shims in the style of NRObject would considerably simplify removing the GObject dependency later and reduce the burden of maintaining the init / dispose functions. Well, please don't do that, I'm already done removing the GObject dependency, take a look at my branch at https://code.launchpad.net/~engelmarkus/inkscape/cppify . Regards, Markus

1 0

Re: [Inkscape-devel] RDF nodes
by Markus Engel 31 Jul '13

31 Jul '13

Hi, I generated the documentation for the object tree and picked out the relevant parts for you: https://dl.dropboxusercontent.com/u/37427492/sp-object.pdf https://dl.dropboxusercontent.com/u/37427492/sp-metadata1.pdf https://dl.dropboxusercontent.com/u/37427492/sp-metadata2.pdf Note that this is not how it is currently implemented in trunk, but in my own branch instead. This shouldn't matter for understanding, though, I just converted the tree to "real" c++ classes. SPObject is the base class for all tree nodes. As you can see in the first pdf, every svg element has got one counterpart of the right class in the object tree. Whenever a change is made to an xml node, it informs the tree node belonging to him by calling for example onUpdate(). The tree node then updates its internal state. Whenever possible, one should work on the tree nodes and leave the xml nodes alone. You'll find lots of information on this in the inkscape wiki, e. g. "Changes in the XML tree are automatically propagated to the object tree via observers, but not the other way around-a function called updateRepr() must be explicitly called." As the rdf information is not a drawable object (-> SPItem subclasses), it should probably derive directly from SPObject. Maybe you could then add it as a member to SPMetadata. I'm not quite sure how to handle that best either. We should wait for some more opinions. or just try it out :) . Regards, Markus > Hi Markus > I started work on Inkscape earlier this year (April/May). During the summer > I work with Commons Machinery [1] to improve metadata handling and the GUI > for metadata and license information. In this prototype I'm working with > the idea that multiple metadata tags are allowed and therefore multiple rdf > tags (though only one per metadata tag). With my limited experience with > Inkscape codebase and the object tree I'm not sure how to best refactor RDF > to fit in. > > In an earlier discussion with Peter Liljenberg [2] he said that a refactor > of RDF should be done to be more general and easy to work with. I'm not > sure about the specifics about this but I will look into it later on in > August hopefully. > > You seem to have a lot more experience with the Inkscape codebase than I > have so it would been fun to discuss any changes that is done here in this > area with you. The goal now for me is to complete the prototype with as > simple code as possible and then during the summer/fall (when time permits) > refactor so it can be merged into Inkscape trunk. > > Concerning GSOC merge before 0.49, I don't know. > > [1] http://commonsmachinery.se/ > [2] https://launchpad.net/~peter-liljenberg <https://launchpad.net/%7Epeter-liljenberg> > > -- > Christoffer Holmstedt >

2 2

[Tracing] Report #6
by Vinícius dos Santos Oliveira 30 Jul '13

30 Jul '13

All algorithms proposed on the previous blog posts are implemented now: * http://vinipsmaker.wordpress.com/2013/07/22/splines-extraction-on-kopf-lisc… * http://vinipsmaker.wordpress.com/2013/07/21/splines-extraction-on-kopf-lisc… I'm attaching some samples. The samples are visually the same of the report #3, but if you edit the files with Inkscape or examine the XML, you notice that all polygons are being grouped. I'll upload a short video later today showing the project summary. Now I can play with B-Splines generation and optimization. Better yet, the smoothness computing is already done (and it worked in my tests also). -- Vinícius dos Santos Oliveira https://about.me/vinipsmaker

1 1

[CAD] Status report
by Sebastian Götte 30 Jul '13

30 Jul '13

Hi, the last few days I dug myself into connector-context.cpp, making that thing compatible with the routing changes I did last week. So far, this has proven quite a task and I am not done yet. A snapshot of the current changes (not yet compiling since I'm midways in it) can be found on launchpad[0]. The main missing parts (apart from actually connecting everything back together after ripping it apart) are fitting g_signal and sigc++ together and getting all the rainbow-colored curves right. I'll follow up if I have any questions. Best regards, Sebastian [0] https://bazaar.launchpad.net/~h-e-6/inkscape/connector-wip/files

2 3