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

Some issues when compiling under Visual Studio Community Edition 2022 (latest version) #28

Open
ukaprch opened this issue Apr 1, 2023 · 0 comments

Comments

@ukaprch
Copy link

ukaprch commented Apr 1, 2023

First of all, many thanks for publishing this on GitHub.
I am running Windows 10 64 bit OS and your Stable.Model.Simd.Float 64 bit version of the product version 2.1
I notice a bit of a difference from version 1.0 with regard to more realistic color differentiation.
With the SIMD it is crazy fast compared to without.

  1. For users of Visual Studio you will need to go into properties for each resource and mark them as imbedded resources for manifest to work. Having done that I needed (for all resources) to provide the following for each call to
    GetManifestResourceStream("Info.png")
    to this:
    GetManifestResourceStream("DeOldify.Resources.Info.png")

  2. If you run the app, then close the app without doing anything I got a null exception error on the following in Ui.cs:
    try
    {
    if (this.__ColorizationThread != null) <<<====needed to add this check for null exception.
    {
    this.__ColorizationThread.Abort();
    }
    }
    catch
    {
    }

  3. In Ui.cs private void StartHandler(object sender, EventArgs e)
    I needed to do the following because of a cross thread exception error:
    this.__BlurryOutput = __Blurify(this.__NormalOutput);
    this.__OutputImage.Image = this.__NormalOutput;
    this.Invoke((Action)delegate <<<===added this check to avoid cross thread exception
    {
    this.__OutputImage.Enabled = true;
    this.__InputImage.Enabled = true;
    this.__StartButton.Text = "Done!";
    this.__StartButton.Enabled = false;
    this.__StartButton.ShowProgress = false;
    this.__StartButton.Click -= this.StopHandler;
    this.__StartButton.Click += this.StartHandler;
    });

  4. Nice to have: you can add more extensions i.e. *.tif (I see you only ref *.tiff) and I'm sure other variants of well know extensions when you select the open image dialog. This is just a minor thing.

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

1 participant