search

CUNY-CL / yoyodyne

Small-vocabulary sequence-to-sequence generation with optional feature conditioning
Apache License 2.0
25 stars 15 forks source link

Bumps version #144

Closed Adamits closed 8 months ago

Adamits commented 8 months ago

New release fixing transformer bug.

kylebgorman commented 8 months ago

LGTM. Do you have the permissions needed to create a release and upload to PyPI? (Docs at the bottom of the README.)

Adamits commented 8 months ago

I don't see an option on the github interface, but I can try to tag and push from my terminal.

Adamits commented 8 months ago

Just realized I am being silly -- I tagged my fork lol. I don't think I can 'merge' a tag?

kylebgorman commented 8 months ago

Tags · CUNY-CL/yoyodynegithub.comTry here…On Oct 20, 2023, at 9:56 AM, Adam @.***> wrote: Just realized I am being silly -- I tagged my fork lol. I don't think I can 'merge' a tag?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Adamits commented 8 months ago

Hey Kyle,

Whatever you linked seems to not have formatted correctly. But I may have confused you. Typically there is an option to make a new release, but I do not have that with this yoyodyne in your CUNY-CL user. I thought maybe I can do this from the terminal with a tag, but forgot that locally I only have my own fork, so I don't think that does anything for us.

I assume only you have access to release.

kylebgorman commented 8 months ago

I’ll work on it, np. Traveling, so give me a few days.On Oct 20, 2023, at 11:51 AM, Adam @.***> wrote: Hey Kyle, Whatever you linked seems to not have formatted correctly. But I may have confused you. Typically there is an option to make a new release, but I do not have that with this yoyodyne in your CUNY-CL user. I thought maybe I can do this from the terminal with a tag, but forgot that locally I only have my own fork, so I don't think that does anything for us. I assume only you have access to release.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Adamits commented 8 months ago

Sounds good, enjoy your travel.

kylebgorman commented 8 months ago

Done BTW.

On Fri, Oct 20, 2023 at 1:19 PM Adam @.***> wrote:

Sounds good, enjoy your travel.

— Reply to this email directly, view it on GitHub https://github.com/CUNY-CL/yoyodyne/pull/144#issuecomment-1773113298, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABG4OI26CQIYAFWGWFK6IDYAKXD5AVCNFSM6AAAAAA6JDU4JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTGEYTGMRZHA . You are receiving this because you modified the open/close state.Message ID: @.***>