Beta 1 released

Dirk Hohndel dirk at hohndel.org
Fri Sep 4 06:58:58 PDT 2015


> On Sep 4, 2015, at 2:05 AM, Guillaume Gardet <guillaume.gardet at free.fr> wrote:
> 
> Hi,
> 
> Le 03/09/2015 23:10, Dirk Hohndel a écrit :
>> Let the testing begin.
> 
> I tried beta 1 just now. Here are a few comments/problems:
> * Translation : UI not fully translated whereas French is 100% translated on transifex. I guess you need to run "lupdate" tool and upload to transifex new TS files.

Correct - since I hadn't even mentioned translations (way too many strings aren't marked, yet, and I expect quite a bit of text changes) I didn't bother pulling translations. That was of course a mistake.

> * Export window => HTML Tab : checkbox number 2 : text (Yearly stats text translated in french) is misaligned (see attachment). English version is not affected!

Is there possibly a newline in the French translation?

> * Cloud : first sync took several minutes and seems to be blocked (circular item turned a bit and stopped as it was frozen) but wireshark showed network activity with lots of normal SSL and TCP packets but also lots of "TCP Dup ACK" and "TCP Retransmission". Is there a problem with the cloud server? My XML file is only 319K, upload was about at 100K and it took several minutes! (Just think that I might have a few photos, does it sync photos too?)

The cloud server sits on AWS. I have begged for people to test this over the past month and more or less no one did. It works nice and fast for me from my system, even when doing it through a reverse proxy. I have seen the frozen UI issue and can't figure out what causes it... I even moved the whole save_file() call into its own thread and still got the same frozen spinner, so I didn't bother adding this code to Subsurface. But of course for me it only freezes for maybe a second or two. Several minutes would be very frustrating.
It would be nice if someone could help debug this one.

> * Print : Dive list print window has "Custom" template by default which is empty and thus display pages with only "This template is empty". We should choose a real default template.

Definitely. I ran into this in one of my tests on a different Mac yesterday. One of the challenges for me is to remember to blow away my settings every once in a while to go back to a "new user" experience.

> * File browser : folder with special characters (e.g. é) are displayed with "??" in place of "é" and cannot be accessed (clic on it does not bring into it).

Which file browser exactly? We use the Qt dialogs in most cases and they should deal with all of this just fine. Can you give steps to reproduce?

> * Facebook: once connected I should have a button to disconnect but I do not.

No idea. Need to investigate. It obviously disconnects you when you exit Subsurface, but that's not the answer to your concern.

> * "seg fault" : I got a "seg fault" while doing nothing with Subsurface! Subsurface was just opened and I was writing this e-mail. Here is the end of the log:
> **********************************************************************
> QObject::connect: Cannot connect (null)::configurationAdded(QNetworkConfiguration) to QNetworkConfigurationManager::configurationAdded(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationAdded(QNetworkConfiguration) to QNetworkConfigurationManager::configurationAdded(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationRemoved(QNetworkConfiguration) to QNetworkConfigurationManager::configurationRemoved(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationAdded(QNetworkConfiguration) to QNetworkConfigurationManager::configurationAdded(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationRemoved(QNetworkConfiguration) to QNetworkConfigurationManager::configurationRemoved(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationChanged(QNetworkConfiguration) to QNetworkConfigurationManager::configurationChanged(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationChanged(QNetworkConfiguration) to QNetworkConfigurationManager::configurationChanged(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::onlineStateChanged(bool) to QNetworkConfigurationManager::onlineStateChanged(bool)
> QObject::connect: Cannot connect (null)::configurationRemoved(QNetworkConfiguration) to QNetworkConfigurationManager::configurationRemoved(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationUpdateComplete() to QNetworkConfigurationManager::updateCompleted()
> QObject::connect: Cannot connect (null)::configurationAdded(QNetworkConfiguration) to QNetworkConfigurationManager::configurationAdded(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::onlineStateChanged(bool) to QNetworkConfigurationManager::onlineStateChanged(bool)
> QObject::connect: Cannot connect (null)::configurationChanged(QNetworkConfiguration) to QNetworkConfigurationManager::configurationChanged(QNetworkConfiguration)
> QObject::connect: Cannot connect (null)::configurationUpdateComplete() to QNetworkConfigurationManager::updateCompleted()
> Erreur de segmentation
> **********************************************************************
> Probably a problem with cloud sync in background? I could not reproduce it. :(

Weird. All those connect calls against null are making me nervous. But I'm not sure they are related to the crash.
Again, another reason why I am so disappointed how little testing the cloud storage has received.

> A few ideas:
> * Does the cloud storage would work with any GIT repo? If so, could be nice to be able to define the GIT repo address to use our own repo.

You can set your own default file to any backend server you want. But the authenticated cloud storage needs a service running on the server to create the account and setup the git repo. I believe that it's not reasonable to assume that the average user would be able and/or interested to set this up. For those with excessive paranoia or a personal need for pain, the URLs are stored in the settings and can be changed with a settings editor of your OS's choice. But I don't want to expose this to a regular user.

> * We should have a script in GIT repo to update translation files with current sources. But the next step should be transifex and I guess it requires some login/password.

I toyed with automated updates but the simple answer is for me to pull translations. I didn't do it this time because I hadn't realized that anyone already had done translations there. I'll fix that when I'm on my other machine later this morning.

> Otherwise looks good. :)

Thanks for the testing. Now we need to figure out how to address all this.

/D



More information about the subsurface mailing list