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

Correct develop flow #57

Merged
merged 3 commits into from
Jun 22, 2022
Merged

Correct develop flow #57

merged 3 commits into from
Jun 22, 2022

Conversation

gaborod16
Copy link
Contributor

Align releases into develop.

gaborod16 and others added 2 commits June 21, 2022 09:47
## Release ONDEWO NLU Python Client 2.6.0

### New features
* [[OND211-1959]](https://ondewo.atlassian.net/browse/OND211-1959) - Intents. GetAllTags and GetAllIntentTags endpoints.

### Improvements
* [[OND211-1927]](https://ondewo.atlassian.net/browse/OND211-1927) - Intents. Remove old Intent classification algorithms
* [[OND211-1928]](https://ondewo.atlassian.net/browse/OND211-1928) - Entities. Remove old Entity Recognition algorithms
@gaborod16
Copy link
Contributor Author

It doesn't look like much, but this is actually aligning the Git History so new additions to the master branch avoid conflicts. :)

Copy link
Contributor

@teddius teddius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the desired development workflow since specifying the requirements in the setup.py is required for the installation dependency checks, hence this PR is rejected.

@gaborod16
Copy link
Contributor Author

@teddius please read the code again.

The variable requires gathers all the information of the requirements from the requirement.txt dynamically, so you do not need to maintain duplicated information.

@@ -26,7 +26,7 @@ def get_all_entities(parent: str, language_code: str, client: Client) -> Dict[st

for et in tqdm.tqdm(iterable=response.entity_types, desc='Loading entity map...'):
for ev in et.entities:
_map[et.name][ev.name] = _map[et.name][ev.name].union({s for s in ev.synonyms})
_map[et.name][ev.name] |= {s for s in ev.synonyms} # Merge sets together in place
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fancy operators.

Copy link

@kNalj kNalj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@gaborod16 gaborod16 merged commit 4357049 into develop Jun 22, 2022
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.

4 participants