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

Hidden Attributes #1060

Closed
dimitri-yatsenko opened this issue Oct 7, 2022 · 4 comments
Closed

Hidden Attributes #1060

dimitri-yatsenko opened this issue Oct 7, 2022 · 4 comments
Assignees
Labels
Milestone

Comments

@dimitri-yatsenko
Copy link
Member

dimitri-yatsenko commented Oct 7, 2022

Feature Request

Problem

Implement support for hidden attributes. These attributes will be prefixed with an underscore and not displayed. DataJoint may store metadata like creation timestamps, usernames, etc in the hidden attributes. U

Requirements

  • DataJoint will support hidden attributes in tables. Any attribute whose name begins with a _ will not be displayed and will not appear as part of the reverse-engineered table definition.
  • DataJoint declaration will allow declaring such attributes for internal purposes but the user API will be prohibited (by default) from using them.
  • Hidden attributes will not participate in joins.
  • Hidden attributes will not be included in results of user queries

Justification

@CBroz1
Copy link
Contributor

CBroz1 commented Feb 7, 2023

Hi @dimitri-yatsenko - What protections did you have in mind when you envisioned that the user API would prohibit using hidden attributes? How would a super user circumvent this protection?

@CBroz1
Copy link
Contributor

CBroz1 commented Oct 27, 2023

@A-Baji - Do you have a sense of your timeline for #1091? I think it could be really useful for something I'm working on

@A-Baji
Copy link
Collaborator

A-Baji commented Oct 27, 2023

@A-Baji - Do you have a sense of your timeline for #1091? I think it could be really useful for something I'm working on

Hey Chris! I think that PR has just been waiting for a review from @dimitri-yatsenko. If any changes are requested feel free to PR to me with the fixes as I'm not currently working on any of the open source projects.

@dimitri-yatsenko
Copy link
Member Author

Fixed in #1091

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants