pwin
commented
2 years ago Aha.... Perhaps. I'll dig tomorrow
On Mon, 29 Nov 2021, 22:01 Boris Pelakh, @.***> wrote:
@.**** commented on this pull request.
In onto_tool/onto_tool.py https://github.com/semanticarts/ontology-toolkit/pull/92#discussion_r758769627 :
@@ -1181,7 +1183,7 @@ def format_value(v):
Extend the width of each column to contain the longest value.
max_length = [max(a, b) for a, b in zip(max_length, [len(s) for s in as_text])] rows.append(as_text)
- row_format = " ".join(f"{{:{length}.{length}}}" for length in max_length) + "\n"
- row_format = "||" + "| ".join(f"{{:{length}.{length}}}" for length in max_length) + "||\n" rows.sort(key=lambda x: x[0])
@pwin https://github.com/pwin Could this sort be the reason your ORDER BY didn't work for the table?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/semanticarts/ontology-toolkit/pull/92#pullrequestreview-818356643, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIFYTELLAKVXNFLOYFFOS3UOPZ43ANCNFSM5IEMP66A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
@pwin This has not been tested at all. It's just an outline of an approach. It needs to be tested, unit tests should be written, and documentation needs to be updated. Feel free to ping me for reviews, etc.