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

Collapsing a and b variables to linkID #43

Open
rickdonnelly opened this issue Aug 21, 2019 · 4 comments
Open

Collapsing a and b variables to linkID #43

rickdonnelly opened this issue Aug 21, 2019 · 4 comments

Comments

@rickdonnelly
Copy link
Contributor

I cannot tell that the variables a and b are used for anything in this package. It makes sense to carry them along so that data problems can be easily traced in the original network databases, but otherwise they appear superfluous. Can we please replace a and b with a single linkID field? Seems like it'd be trivial to concatenate a and b for platforms that still use them (e.g., Emme, Cube), and would require no translation for those that already use IDs (e.g., TransCAD, MATSim).

@dkyleward
Copy link
Contributor

Where do you see a and b? You're right that I've never used them.

@rickdonnelly
Copy link
Contributor Author

If you look at the data schema in R/data.R you will see them listed there. Since the functions seem to ignore all extraneous link data I assumed that those variables listed are used in one or more functions. So, maybe the focus should be on revisiting the list of variables in that file rather than assuming that they're all keywords expected by one or more functions.

@gregmacfarlane
Copy link
Contributor

The original dataset we used to build the package came from Cube. I agree that we can concatenate and move forward with a single linkid.

@dkyleward
Copy link
Contributor

@rickdonnelly it will be easier for me if you just add me as a collaborator to the repo. That way I don't have to fork for each change.

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

No branches or pull requests

3 participants