So, there was a question on the cdk-devel mailing list about bounding boxes, reactions, and text. An unfortunate consequence of the new design is that the renderer will not calculate bounding boxes that can fully contain the text. Concretely, this is what it would look like (not made in JCP!)
The blue box is the bounds that would be created, which is minimal with respect to the atom centers. The black box is the bounds that should be created, if we respected the text size. The problem is, the size of text is not known until the point it is drawn. Or, more precisely, until we have some sort of GraphicsContext to ask about the width in a particular font.
The blue box is the bounds that would be created, which is minimal with respect to the atom centers. The black box is the bounds that should be created, if we respected the text size. The problem is, the size of text is not known until the point it is drawn. Or, more precisely, until we have some sort of GraphicsContext to ask about the width in a particular font.
So, a two-pass system was suggested. When this was mentioned before, I was dismissive - perhaps even rude. Sorry about that Egon, Sam. I still think it is better avoided; in the case of transparency, I don't know why alpha values can't be used for fill colours. I understand there was some SWT problems..?
Anyway, here is a sketch of a possible two-pass system, that would allow some of these adjustments to be made:
That's unreadably small in thumbnail - click for bigger, as usual. The basic idea would be to have one element tree with model-space values, and one with screen space values. I've made the distinction between double and integer, but Java2D will draw with doubles, so that is not important.
Comments
I fully agree with that statement!
Can you please repeat the rudeness, so that I understand what to be upset about? ;)
Also, I dismissed a two-pass sytem without discussion, so that's what you can get annoyed about, if you like :)
In general with the rendering, I focus too much on speed at the expense of flexibility. I guess that small molecules need multiple precise representations more than they need ultra-fast drawing.
At this moment, I am not sure what the bottleneck of drawing speed is... maybe Arvid and I can find some time next week to run YourKit (tm) against jchempaint-primary...
1) given a chem-model, generate the intermediate representation.
2) run through the element tree, either making a new scaled tree (as in the diagram) or calculating some properties, like a bounding box.
3) draw the tree, possibly after changing the scale.
in concrete terms, if a correct bounding box is to be made, the width on screen needs to be calculated, then the scale adjusted to compensate.
The double-tree approach is clean, but memory inefficient. Just calculating some properties (eg width, height) would be better, but less extensible.