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

When creating an engine, the window title is VTK #1018

Closed
Meakk opened this issue Sep 30, 2023 · 12 comments
Closed

When creating an engine, the window title is VTK #1018

Meakk opened this issue Sep 30, 2023 · 12 comments

Comments

@Meakk
Copy link
Member

Meakk commented Sep 30, 2023

Describe the bug
When creating an engine in Python, the window title is VTK

To Reproduce
Steps to reproduce the behavior:

import f3d
e=f3d.Engine()

image

Expected behavior
The Window title should be F3D

System Information:

  • OS: Windows

F3D Information
2.2 RC2

@mwestphal
Copy link
Contributor

more libf3d than python I believe.

@mwestphal mwestphal added the good first issue Good for newcomers label Oct 1, 2023
@mwestphal mwestphal added this to the 2.4.0 milestone Oct 1, 2023
@Meakk
Copy link
Member Author

Meakk commented Oct 2, 2023

Probably yes, but I haven't checked in C++ so I cannot tell for sure.

@shaikan
Copy link
Contributor

shaikan commented Oct 17, 2023

Hi
I would like to work on this issue.
thanks,

@Meakk
Copy link
Member Author

Meakk commented Oct 17, 2023

Great! Thank you @shaikan
Please submit a PR when you are ready and don't hesitate to reach out on discord if you need help.

@mwestphal
Copy link
Contributor

mwestphal commented Oct 17, 2023 via email

@mwestphal
Copy link
Contributor

Need any help @shaikan ?

@shaikan
Copy link
Contributor

shaikan commented Oct 23, 2023

Hi
sorry I was busy with office work so couldn't work on this issue.
I am setting up the build environment right now.
Will seek help if I get stuck.

@shaikan
Copy link
Contributor

shaikan commented Oct 26, 2023

So I checked and here is my analysis.
The renaming of the window to F3D - A fast and minimalist 3D viewer is done by F3DStarter.
The actual vtkRenderWindow gets created inside window_impl at which time it gets the default name from VTK i.e. "Visualization Toolkit..".
There are two things here.

  1. In C++ F3D application, there is a slight hiccup while creating the window which causes it to show with the default name for a fraction of a second before it gets resized and renamed.
  2. In case of Python, the script directly creates the engine object and hence the window never gets renamed and shows with the original VTK name.

I propose that setting the window name and icon should be shifted from f3d application to inside libf3d
either inside engine or the root of window creation window_impl.
This would make the window display behaviour same across c++ and python.

@Meakk
Copy link
Member Author

Meakk commented Oct 26, 2023

Ok that makes sense.
We have a public API to set the window title f3d::window::setWindowName and as you said, it's called by the application which is fine to me.
I think the issue is the lack of default value in libf3d.
It does not have to be F3D - A fast and minimalist 3D viewer, maybe something simple like just libf3d or f3d or f3d window...

@mwestphal
Copy link
Contributor

maybe something simple like just libf3d or f3d or f3d window...

Indeed, sounds resonnable

@shaikan
Copy link
Contributor

shaikan commented Oct 26, 2023

So which one shall I set "libf3d", "f3d" or "f3d window" ?

@mwestphal
Copy link
Contributor

f3d I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants