Menu

#20 8580 filter bug

None
closed
None
5
2019-07-30
2017-09-10
No

The new 8580 filter model in the 2.0.0 beta release is bugged, as reported in [vice-emu:bugs:#909]

Some tunes that expose the problem:
/MUSICIANS/B/Blues_Muz/Roestoeen_Kristian/Klogge.sid (at 0:15)
/MUSICIANS/T/Toggle/Defier_of_Deadlines.sid (at 0:15)
/MUSICIANS/B/Blues_Muz/Gallefoss_Glenn/Title_Needed.sid (at 1:10)
/MUSICIANS/H/Hannula_Antti/Fatamorgana.sid (at 0:00)
/MUSICIANS/C/Crowley_Owen/Croaker.sid (at 0:18)
/MUSICIANS/J/Jammer/Drum_Fool.sid (at 0:00)

Some of these fail with the following assertion:
src/builders/residfp-builder/residfp/Integrator8580.h:128: int reSIDfp::Integrator8580::solve(int) const: Assertion `vi < kVgt' failed.

Attached a sample from this csdb discussion

1 Attachments

Related

Bugs: #19
VICE: Bugs: #909

Discussion

  • Leandro Nini

    Leandro Nini - 2017-09-12
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -4,5 +4,11 @@
     /MUSICIANS/B/Blues_Muz/Roestoeen_Kristian/Klogge.sid (at 0:15)
     /MUSICIANS/T/Toggle/Defier_of_Deadlines.sid (at 0:15)
     /MUSICIANS/B/Blues_Muz/Gallefoss_Glenn/Title_Needed.sid (at 1:10)
    +/MUSICIANS/H/Hannula_Antti/Fatamorgana.sid (at 0:00)
    +/MUSICIANS/C/Crowley_Owen/Croaker.sid (at 0:18)
    +/MUSICIANS/J/Jammer/Drum_Fool.sid (at 0:00)
    +
    +
    +Integrator8580.h:128: int reSIDfp::Integrator8580::solve(int) const: Assertion `vi < kVgt' failed.
    
     Attached a sample from this [csdb discussion](http://csdb.dk/forums/?roomid=14&topicid=124672)
    
    • Group: -->
     
  • Leandro Nini

    Leandro Nini - 2017-09-12
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -8,7 +8,7 @@
     /MUSICIANS/C/Crowley_Owen/Croaker.sid (at 0:18)
     /MUSICIANS/J/Jammer/Drum_Fool.sid (at 0:00)
    
    -
    -Integrator8580.h:128: int reSIDfp::Integrator8580::solve(int) const: Assertion `vi < kVgt' failed.
    +Some of these fail with the following assertion:
    +src/builders/residfp-builder/residfp/Integrator8580.h:128: int reSIDfp::Integrator8580::solve(int) const: Assertion `vi < kVgt' failed.
    
     Attached a sample from this [csdb discussion](http://csdb.dk/forums/?roomid=14&topicid=124672)
    
     
  • Leandro Nini

    Leandro Nini - 2018-06-26

    Fixed with r2551

     
  • Leandro Nini

    Leandro Nini - 2018-06-26
    • status: accepted --> fixed
     
  • Leandro Nini

    Leandro Nini - 2019-07-30
    • status: fixed --> closed
     

Log in to post a comment.

MongoDB Logo MongoDB