Allow strings as well as char
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia dalla riga 867 di dj.internal.Relvar, dove attr.isString viene verificato con ischar(), e analizza come vengono gestiti i valori string nelle altre parti del codice DataJoint MATLAB. Verifica il comportamento in MATLAB R2022b sia per i valori string sia per i vettori char; il lavoro è completato quando entrambi possono essere usati in modo intercambiabile senza causare problemi altrove.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Feature Request
Problem
On line 867 of dj.internal.Relvar attr.isString is equated with ischar(),, which disallows the use of Matlab strings .
A simple fix would be to write
assert(ischar(value) || isstring(value), ...
instead.
This has worked fine for me, but I did not check whether using string instead of char leads to problems elsewhere in the DJ code.
Requirements
Ideally, it should be possible to use string and vector of char interchangeably.
Justification
Alternative Considerations
Do you currently have a work-around for this? Provide any alternative solutions or features you've considered.
Related Errors
Add any errors as a direct result of not exposing this feature.
Please include steps to reproduce provided errors as follows:
- OS (WIN | MACOS | Linux)
- MATLAB Version ; R2022b
- MySQL Version
- MySQL Deployment Strategy (local-native | local-docker | remote)
- DataJoint Version 3.5.0
- Minimum number of steps to reliably reproduce the issue
- Complete error stack as a result of evaluating the above steps
Screenshots
If applicable, add screenshots to help explain your feature.
Additional Research and Context
Add any additional research or context that was conducted in creating this feature request.
For example:
- Related GitHub issues and PR's either within this repository or in other relevant repositories.
- Specific links to specific lines or a focus within source code.
- Relevant summary of Maintainers development meetings, milestones, projects, etc.
- Any additional supplemental web references or links that would further justify this feature request.
- Lingua principale
- MATLAB
- Stelle
- 41
- Fork
- 42
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di datajoint/datajoint-matlab
-
awaiting-triage enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
datajoint/datajoint-matlab#437 ·
-
awaiting-triage bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
datajoint/datajoint-matlab#436 ·
-
awaiting-triage bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
datajoint/datajoint-matlab#430 · 2 commenti ·
-
SSL Connection errorApertaawaiting-triage bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
datajoint/datajoint-matlab#425 · 3 commenti ·
-
Officially use the `+` operator for unionsForse di nuovo libera @A-Baji l’ha presa 1365 giorni fa e non c’è nessuna pull request aperta. Apertaawaiting-triage enhancement
datajoint/datajoint-matlab#422 · 1 assegnatario ·
Tutte le issue di datajoint/datajoint-matlab
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
type:bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
VH-Lab/DID-matlab#220 ·
I maintainer di solito rispondono entro 1 giorno
-
ATSU: received telex messages lose their line breaks on the MCDUForse già presa @wimatte l’ha presa 5 giorni fa. ApertaRequest
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
flybywiresim/aircraft#10986 ·
I maintainer di solito rispondono entro 2 giorni
-
Distrib: Compile standalone version with >= R2023b as macOS (28) will not support Runtime R2023aAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
brainstorm-tools/brainstorm3#952 · 3 commenti ·