Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0032550Open CASCADE[OCCT] OCCT:Documentationpublic2021-09-01 14:492021-09-12 08:48
Assigned Tobtokarev 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.6.0*Fixed in Version 
Summary0032550: Proofreading for "Tutorials and Samples -> Tutorial"
DescriptionProofreading and new images for solid model for MakeBottle sample description.
Steps To ReproduceImages are reworked
TagsNo tags attached.
Test case number
Attached Filespng file icon big_figure.png (51,964 bytes) 2021-09-09 08:41

- Relationships
child of 0031896newbtokarev Documentation - Proofreading User manual 

-  Notes
git (administrator)
2021-09-01 16:29

Branch CR32550 has been created by btokarev.

SHA-1: ef53136996944b2ed1486c7ecb165983c9bd8590

Detailed log of new commits:

Author: btokarev
Date: Wed Sep 1 16:29:33 2021 +0300

    0032550: Proofreading for Tutorial
    New images added for better representation of solid model;
    Minor fixes for misprints and old classes;
    Proofreading done.
kgv (developer)
2021-09-01 16:43

I'm not familiar with this tutorial.

New images have been obviously dumped from 3D viewer, but why this has been done without beautifier options (rendScale for antialiasing, mapping isoLines on triangulation to avoid artifacts)?
Does purple really a good color for a bottle in this document?
btokarev (developer)
2021-09-01 16:56

The goal of new images were to explicitly show that the result of the build is not a blue carcass thingy but an actual solid model. These images can be redone with better visual parameters if needed.
As for the purple color, this is kind of shade that is used for default 3DS MAX models and I thought it fits for representing the first modelling steps using this bottle. Additionally, I think it will both fit the overall color schema of the HTML Overview webpage and stand out enough if the user is scrolling the page to find the bottle building stage he is willing to check out. This also can be changed if it seems unfit here.
aml (developer)
2021-09-09 08:41
edited on: 2021-09-09 08:43

Boris, thank you very much for the patch.

I have one but important remark: resulting images are much bigger than older ones. So, the page looks a bit weird with new images.

You will have to rework images, so it is fruitful to utilize beautifiers mentioned by Kirill. Please, ask him how to embed them into a code and keep them in the patch.

git (administrator)
2021-09-10 17:35

Branch CR32550 has been updated by btokarev.

SHA-1: 8657a55192f82a0af6520f9d476daba0a6ff0755

Detailed log of new commits:

Author: btokarev
Date: Fri Sep 10 17:35:07 2021 +0300

    0032550: Proofreading for "Tutorials and Samples -> Tutorial"
    Remarks implemented:
    New graphic settings applied;
    New color for visual output set;
    New images added that are grabbed from the default window size.

aml (developer)
2021-09-12 08:48


1. Please include beautifiers in the patch.
2. Images are too big; it is necessary to downscale them. The maximal image size should be 400x400.

- Issue History
Date Modified Username Field Change
2021-09-01 14:49 btokarev New Issue
2021-09-01 14:49 btokarev Assigned To => abv
2021-09-01 14:49 btokarev Assigned To abv => btokarev
2021-09-01 14:50 btokarev Relationship added child of 0031896
2021-09-01 16:29 git Note Added: 0103636
2021-09-01 16:31 btokarev Assigned To btokarev => aml
2021-09-01 16:31 btokarev Status new => resolved
2021-09-01 16:31 btokarev Steps to Reproduce Updated View Revisions
2021-09-01 16:43 kgv Note Added: 0103637
2021-09-01 16:56 btokarev Note Added: 0103638
2021-09-09 08:41 aml Note Added: 0103932
2021-09-09 08:41 aml File Added: big_figure.png
2021-09-09 08:43 aml Note Edited: 0103932 View Revisions
2021-09-09 08:44 aml Assigned To aml => btokarev
2021-09-09 08:44 aml Status resolved => assigned
2021-09-10 17:35 git Note Added: 0103990
2021-09-10 17:37 btokarev Assigned To btokarev => aml
2021-09-10 17:37 btokarev Status assigned => resolved
2021-09-10 17:37 btokarev Steps to Reproduce Updated View Revisions
2021-09-12 08:48 aml Note Added: 0104049
2021-09-12 08:48 aml Assigned To aml => btokarev
2021-09-12 08:48 aml Status resolved => assigned

Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker