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

Remove unused observers from the database #2142

Closed
Yuri05 opened this issue Mar 28, 2022 · 1 comment
Closed

Remove unused observers from the database #2142

Yuri05 opened this issue Mar 28, 2022 · 1 comment
Milestone

Comments

@Yuri05
Copy link
Member

Yuri05 commented Mar 28, 2022

The observers in the table below, which are part of the PK-Sim database, seems to be unused.

unused observers
observer
Fraction dissolved particles
Fraction dissolved particles1
Fraction dissolved particles10
Fraction dissolved particles11
Fraction dissolved particles12
Fraction dissolved particles13
Fraction dissolved particles14
Fraction dissolved particles15
Fraction dissolved particles16
Fraction dissolved particles17
Fraction dissolved particles18
Fraction dissolved particles19
Fraction dissolved particles2
Fraction dissolved particles20
Fraction dissolved particles3
Fraction dissolved particles4
Fraction dissolved particles5
Fraction dissolved particles6
Fraction dissolved particles7
Fraction dissolved particles8
Fraction dissolved particles9
Fraction insoluble particles
Fraction insoluble particles1
Fraction insoluble particles10
Fraction insoluble particles11
Fraction insoluble particles12
Fraction insoluble particles13
Fraction insoluble particles14
Fraction insoluble particles15
Fraction insoluble particles16
Fraction insoluble particles17
Fraction insoluble particles18
Fraction insoluble particles19
Fraction insoluble particles2
Fraction insoluble particles20
Fraction insoluble particles3
Fraction insoluble particles4
Fraction insoluble particles5
Fraction insoluble particles6
Fraction insoluble particles7
Fraction insoluble particles8
Fraction insoluble particles9
Fraction solid particles
Fraction solid particles1
Fraction solid particles10
Fraction solid particles11
Fraction solid particles12
Fraction solid particles13
Fraction solid particles14
Fraction solid particles15
Fraction solid particles16
Fraction solid particles17
Fraction solid particles18
Fraction solid particles19
Fraction solid particles2
Fraction solid particles20
Fraction solid particles3
Fraction solid particles4
Fraction solid particles5
Fraction solid particles6
Fraction solid particles7
Fraction solid particles8
Fraction solid particles9

There are no entries in tab_model_observes for them; in PK-Sim code we have only one place where the names of those observers are used - but the breakpoints are never hit when creating a simulation with particles dissolution formulation(s).

/// <summary>
/// Return true if an application observer should not be hidden from user
/// </summary>
/// <param name="observer"></param>
private bool applicationObserverShouldBeShown(IObserver observer)
{
string name = observer.Name;
if (name.StartsWith(CoreConstants.Observer.FRACTION_SOLID_PREFIX))
return true;
if (name.StartsWith(CoreConstants.Observer.FRACTION_DISSOLVED_PREFIX))
return true;
if (name.StartsWith(CoreConstants.Observer.FRACTION_INSOLUBLE_PREFIX))
return true;
return false;
}

grafik

I assume those observers were created in the past somewhere in the code (in ParticleApplicationCreator or so), but were dropped when the implementaion of particles dissolution was improved some years ago.

So I think it is safe to delete them.

@msevestre FYI

@msevestre
Copy link
Member

lol if yo usay so...happy to delete

msevestre added a commit that referenced this issue Mar 28, 2022
@msevestre msevestre added this to the Version 11 milestone Mar 28, 2022
@Yuri05 Yuri05 closed this as completed in d97339c Mar 28, 2022
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

2 participants