Skip to content

Use ttim line elements#167

Merged
dbrakenhoff merged 10 commits intodevfrom
use_ttim_line_elements
Jan 27, 2026
Merged

Use ttim line elements#167
dbrakenhoff merged 10 commits intodevfrom
use_ttim_line_elements

Conversation

@mbakker7
Copy link
Owner

No description provided.

@mbakker7
Copy link
Owner Author

mbakker7 commented Jan 26, 2026

Not sure what to do with the besselnumba.py file. We now started to share that between timml and ttim. So should we give that its own repository? @dbrakenhoff, what do you think?

@dbrakenhoff
Copy link
Collaborator

Yea, I've been wondering about this question too. We also have plotting that is very similar between the two packages which could be shared, and there's probably more logic that could be used by both packages. Two options I think: we put all shared logic in one of the two and make that the dependency, or like you say, we create a new repository with stuff that's used in both.

I'm a little more tempted to stuff all shared logic into one package, as that saves us the hassle of maintaining a third repository... but I'm not entirely sure yet either.

@mbakker7
Copy link
Owner Author

I am afraid we have some version issues here. It seems dev is one commit behind master and this branch is based on dev. I am encountering notebooks that still use HeadLineSink type elements without calling them River (like the BuildingPit notebook). @dbrakenhoff can you see what is going on? At least dev should be fixed and then, if possible, this branch.

@mbakker7
Copy link
Owner Author

Yea, I've been wondering about this question too. We also have plotting that is very similar between the two packages which could be shared, and there's probably more logic that could be used by both packages. Two options I think: we put all shared logic in one of the two and make that the dependency, or like you say, we create a new repository with stuff that's used in both.

I'm a little more tempted to stuff all shared logic into one package, as that saves us the hassle of maintaining a third repository... but I'm not entirely sure yet either.

If we put besselnumba in one of the two, it also a hassle. I don't expect there will be many changes once we make a separate besselnumba package. Other than updates to dependencies like numpy and numba.

@dbrakenhoff
Copy link
Collaborator

Fair enough, then we make a special bessel module, and other logic (e.g. plotting) is either repeated and/or borrowed from its neighbor.

And I'll check what's going on with this PR and dev.

@dbrakenhoff
Copy link
Collaborator

I update dev from master, if we fix all the remaining naming issues here we should be good?

@mbakker7 mbakker7 marked this pull request as ready for review January 27, 2026 14:46
@dbrakenhoff dbrakenhoff merged commit b2b8932 into dev Jan 27, 2026
4 checks passed
@dbrakenhoff dbrakenhoff deleted the use_ttim_line_elements branch January 27, 2026 15:53
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

Successfully merging this pull request may close these issues.

2 participants