UltraStar-Deluxe / Play

Free and open source singing game with song editor for desktop, mobile, and smart TV
https://ultrastar-play.com
MIT License
377 stars 69 forks source link

Game stuck when opening song #261

Closed sfetzel closed 2 years ago

sfetzel commented 2 years ago

Issue type: Bug report

Actual behaviour

The game freezes when starting the song from the txt file "Toploader - Dancing In The Moonlight.txt" (from http://usdb.animux.de)

Expected behaviour

The game singing activity should start.

Steps to reproduce

  1. Add song with mp3 to songs folder (txt file attached)
  2. Open the ultrastarplay
  3. Click on "sing"
  4. Select Toploader - Dancing in the moonlight
  5. Game freezes

Details

Toploader - Dancing In The Moonlight - game crashes.txt

achimmihca commented 2 years ago

Confirmed, the error message is

VoicesBuilderException: Error at line 8: Linebreak encountered without preceding notes
VoicesBuilder.ThrowLineError (System.UInt32 lineNumber, System.String message) (at Assets/Common/Model/Song/Builder/VoicesBuilder.cs:235)

We should probably make the VoicesBuilder less strict and accept such minor issues. For example, the following should trigger the same error because there are two sentence endings without notes between:

#ARTIST:Toploader
#TITLE:Dancing In The Moonlight
#MP3:Toploader - Dancing In The Moonlight.mp3
: 228 4 72 We 
: 236 2 72 get 
: 240 4 75 it 
: 248 2 67 on
: 252 5 65 ~ 
: 260 2 63 most 
: 264 2 65 ev
: 268 2 63 'ry 
: 272 3 65 night 
- 280 296
- 297 298
: 300 2 63 when 

A superfluous sentence end can be ignored and logged with a warning.

achimmihca commented 2 years ago

@sfetzel Please re-test with the Linux build from here

BTW: can others download the build artifacts? Or only contributers?

bohning commented 2 years ago

I also sometimes see a line break before the final E (which unfortunately leads to a crash in UltraStar Manager). This should probably also simply be ignored and logged to Error.log.

dan-r95 commented 2 years ago

I dont think I can download a build, or at least I dont see a button

achimmihca commented 2 years ago

There is not a dedicated button. You have to click the "Build" text of "Artifacts".

For me, this is a download link (one archive with all target platforms). But when I log out then it is not a link, just text. As I said, I am unsure if others can download the artifact.

--

2021-12-06 20_57_10-github

-

achimmihca commented 2 years ago

This is the download link: https://github.com/UltraStar-Deluxe/Play/suites/4317071756/artifacts/112883183 It works for me when logged in. Is it working for you?

dan-r95 commented 2 years ago

Yes, thank you!