Comments (2)
tgregg
commented on August 20, 2024
All three variants are "correct", in that when the text is known, nothing else matters. Is this a concern about consistency across the reader options?
from ion-python.
rmarrowstone
commented on August 20, 2024
When the text is not known, the information about SId and import location is also missing.
I see. That currently raises an error as well, though we could be returning a SymbolToken with None text and Sid and import location.
Update: there is a correctness issue regarding imported Symbols with undefined text.
As it is now, imported symbols with undefined text are considered equivalent to each other (regardless of SId) and local symbols with undefined text.
from ion-python.
Related Issues (20)
- Missing symbol when loading ion: __ion_writer_add_annotation_sid_helper HOT 3
- ion-python-benchmark-cli's threshold value need to be more accurate. HOT 2
- ion-python-benchmark-cli's format conversion shouldn't always copy the input file
- Benchmark CLI `compare` can have inaccurate results for some metrics
- Ion Python Benchmark results display with "false precision"
- Decide which Python interpreters should be included in the performance testing workflow
- Pydantic support HOT 3
- Some IonPyDict Methods Behave Like Python 2 Dict
- 4 tests in `test_benchmark_cli.py` failed. HOT 1
- Ion Python's documentation needs to be improved HOT 1
- Improving IonPyObject Construction
- Soft-fail on ImportError when importing c libs HOT 1
- DEPRECATION: amazon.ion is being installed using the legacy 'setup.py install' method HOT 1
- Benchmark results report doesn't support `tabulate` on Windows.
- GHA windows build test doesn't work with python 3.8
- Improve Python Error Handling from ioncmodule
- Benchmark-cli `IO-type=buffer` should support benchmarking multiple top_level objects
- Benchmark-cli protobuf multiple top level objects use case HOT 1
- Test Pipeline need to support the latest python version 3.12
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ion-python.