-
Notifications
You must be signed in to change notification settings - Fork 140
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
regression: no model for tags type=default #564
Comments
But if I switch the branching of the conditional: --- a/kraken/rpred.py
+++ b/kraken/rpred.py
@@ -336,10 +336,10 @@ def _resolve_tags_to_model(tags: Optional[Sequence[Dict[str, str]]],
"""
if not tags and default:
return ('type', 'default'), default
+ elif tags and default:
+ return next(tags.values()), default
elif tags:
for tag in tags.items():
if tag in model_map:
return tag, model_map[tag]
- elif tags and default:
- return next(tags.values()), default
raise KrakenInputException(f'No model for tags {tags}') then I get
so the moved line also needs an extra But then, still, I get lots of segmentation errors:
and the text result is empty. |
thx! |
On 24/01/17 04:15AM, Robert Sachunsky wrote:
But if I switch the branching of the conditional:
```diff
--- a/kraken/rpred.py
+++ b/kraken/rpred.py
@@ -336,10 +336,10 @@ def _resolve_tags_to_model(tags: Optional[Sequence[Dict[str, str]]],
"""
if not tags and default:
return ('type', 'default'), default
+ elif tags and default:
+ return next(tags.values()), default
elif tags:
for tag in tags.items():
if tag in model_map:
return tag, model_map[tag]
- elif tags and default:
- return next(tags.values()), default
raise KrakenInputException(f'No model for tags {tags}')
```
I've fixed it in main (your change isn't correct as the last check
should run if the `elif tags` branch falls through).
But then, still, I get lots of segmentation errors:
```
Tensor conversion failed with 'default'. Emitting empty record.
```
and the text result is empty.
Also fixed in main. I've switched from using only the value of tags to
(key, value) pairs so the `next(iter(...` part was also incorrect.
PS: Regarding the scikit-image thing. I was mistaken, newer scikit-image
up to 0.21.x works with scipy 0.10.x as the simplices attribute existed
concurrently with the removed vertices one. Current main has the
requisite patches for py38 compatibility.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In trying to reproduce #525, I encountered this:
Looks like the cause is somewhere in 8ff27d1.
The text was updated successfully, but these errors were encountered: