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

Integrating D-ETM #49

Open
cayaluke opened this issue Dec 16, 2021 · 3 comments
Open

Integrating D-ETM #49

cayaluke opened this issue Dec 16, 2021 · 3 comments

Comments

@cayaluke
Copy link

Dear @silviatti ,

Related to issue #1

I would like to resubmit the feature request to integrate the DETM into your OCTIS suite. It appears the person who originally proposed issue #1 back in April 2021 has lost interest, or no longer wants to pursue this.

Is it possible if you could complete the integration?

With kindest regards
Luke

@silviatti
Copy link
Collaborator

Hi Luke,
currently I don't have much time to dedicate to this project. Since this is a super busy period for me, I'm focusing more on the maintenance (as much as time allows me) than on the integration of additional features.

However, I'd be happy to give you my support if you want to work on a PR to integrate D-ETM :)

Best,

Silvia

@cayaluke
Copy link
Author

cayaluke commented Jan 4, 2022

Dear @silviatti

Thank you for your response.

I guess we can start with the time indexes.

I've included an extra column in the corpus.tsv file containing a time index.

What would you do to make the octis/detm model reads/loads the corpus with the indexes?

Cheers
Luke

@espoirMur
Copy link
Contributor

I started something, but I got lost in multiple projects now. Feel free to implement it. I would be more than happy to review and contribute if I got time.
@cayaluke cayaluke

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

No branches or pull requests

3 participants