quoted sets names

Hello, thanx for the driver!
Seems there is a problem working with quoted sets. And if set name is numeric, this is the only way the query could be made
For instance, I've made 2 queries, first using driver, second using aql
insert into test."123" (__key,value) values (1,"value1")

aql> insert into test."321" (pk,value) values (1,"value1")

First query had made set "123" whilst second 321, and I've not founded the way to select from "123"
aql> show sets
+------------------+----------------+------------------+---------+-------------------+---------+--------------+----------+-------------------+-------------------+--------------+------------+
| disable-eviction | ns             | index_populating | objects | stop-writes-count | set     | enable-index | sindexes | memory_data_bytes | device_data_bytes | truncate_lut | tombstones |
+------------------+----------------+------------------+---------+-------------------+---------+--------------+----------+-------------------+-------------------+--------------+------------+
| "false"          | "test"         | "false"          | "1"     | "0"               | ""123"" | "false"      | "0"      | "45"              | "80"              | "0"          | "0"        |
| "false"          | "test"         | "false"          | "1"     | "0"               | "321"   | "false"      | "0"      | "24"              | "64"              | "0"          | "0"        |
+------------------+----------------+------------------+---------+-------------------+---------+--------------+----------+-------------------+-------------------+--------------+------------+

After that I've tried to select data from correctly created 321 set.
aql> select * from test."321" where pk=1
+----------+
| value    |
+----------+
| "value1" |
+----------+

Using driver, I get empty result quering with __key and selecting all
select * from test."321" where __key=1
select * from test."321"

hello, can I do anything to clarify the issue?

Hi @Gaudeamus, the issue is clear. The documentation says:

Names can include only Latin lowercase and uppercase letters with no diacritical marks (a-z, A-Z), digits 0-9, underscores (_), hyphens (-), and dollar signs ($). This naming guideline is not enforced; however, if you do not follow it, some Aerospike features and tools might not function properly.

Yet this is not the priority, but it's good to have quoted schema/table names supported in the JDBC driver. We'll update here once this is implemented and released.

@Gaudeamus, version 1.6.0 has been released and it supports quoted table names.


Wrong language file used

When i use trans('someword') it works fine for locale 'nl', but when my locale is set on 'en' it uses nl/default.php instead of en/default.php. I am using L4.2. What is it that i am doing wrong?

I don't think 4.x is still supported.

You should probably ask on the forums. I'm sure someone can help you there.


Send temp passwords to admin

SHUSH Sean!!!!

Thank you!!!!!11!
Sent from my iPhone

On Jul 5, 2018, at 16:09, Thomas Keeley<EMAIL_ADDRESS>wrote:
Merged #183.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add entrypoint option

Fixes #258
The idea is to add an entrypoint option so we can override the one specified in the docker-compose file.
I don't have a lot of experience with docker or buildkite, so I'm not sure if this is the right way to do it. Please let me know if i missed anything, or if there is a better way to do this.

LGTM

Thanks @emilio-notable! In general we hesitate to add options that can be set in the docker-compose.yml file itself — was there a use case that prevented you from doing that?

Hi @toolmantim,
our use case is that we have one docker-compose.yml that we would like to use for two buildkite steps, and the only difference should be the entrypoint.
right now we solved it using another docker-compose file, and specifying both in the buildkite step, but we were wondering if there was a cleaner way to do it, overriding just the entrypoint without having to create another docker-compose file
Thanks!

Thanks for the details! That totally makes sense, and given the small footprint of the addition I think it’s totally reasonable, even if it’s something that can be achieved with separate docker compose config files.


Change cardsort responses keys [LEI-239]

Refs: https://openscience.atlassian.net/browse/LEI-239

Use 'sort1' & 'sort2' instead of  'cardSort1' and 'cardSort2'
Make the card id "rsq1", "rsq2", etc. instead of using the card name.

Based on meeting discussions, would you like me to hold on reviewing this one until we decide on appropriate column names?


CRD Helm chart is not installable

When trying to install the crd helm chart the following error occurs.

Error: library charts are not installable

@ericlarssen-wf


Unhandled Rejection (TypeError): Cannot use 'in' operator to search for 'clientId' in undefined

Your team may be aware of it, but I am posting it just in case.
Everything worked just fine a few hours ago. I suddenly received the error message.
Is there something I need to do in order to fix the error? Or will you fix it?
Unhandled Rejection (TypeError): Cannot use 'in' operator to search for 'clientId' in undefined
resolveConfig
../src/implicit-grant.js:55:6
  52 | }
  53 |
  54 | function resolveConfig(rules, location) {
> 55 |   if ('clientId' in rules) {
     |      ^  56 |     return rules;
  57 |   }
  58 |

Module.configure
../src/implicit-grant.js:36:14
  33 |  * @param location
  34 |  */
  35 | export async function configure(cfgOrRules, location = window.location) {
> 36 |   const cfg = resolveConfig(cfgOrRules, location);
     |              ^  37 |   const globalConfig = window[GLOBAL_CONFIG_KEY];
  38 |
  39 |   const config = Object.assign({

Thank you.

Just pushed the official 1.0.0 release, that's what broke for you, sorry. I'll fix the bug, but for immediate fix on your end you can have your code explicitly point to the previous release, 0.2.3
Change
https://cdn.byu.edu/browser-oauth-implicit/latest/implicit-grant.js

to
https://cdn.byu.edu/browser-oauth-implicit/0.2.3/implicit-grant.js

@sot232 I might need a little more context for how you're setting it up in your code.
What are you passing to implicit.configure()?

@snelg The website is working again as soon as I correctly use implicit.configure parameters.
To be honest with you, I didn't pass any parameters. However, it worked. I knew I was supposed to pass some values. I was ignoring it.
Thank you for your help!

You did help me find a bug anyhow :)
You can set the config in the global window object and then call implicit.configure() :
window['byu-oauth-implicit-config'] = {
  clientId: "blahblahblahblah",
  callbackUrl: "https://mysite.byu.edu"
}

implicit.configure()

But our unit tests were checking if you pass an empty object into the configure call, e.g.
implicit.configure( { } )

I fixed the code to properly handle that completely empty implicit.configure() call, as long as you set the configuration in window


correct windows package paths

The paths for the windows packages in compute-image-windows are incorrect. Added arch and option .0 suffix. Real files in bucket right now for comparison:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

/lgtm


Add example for custom JSON schema's

As custom JSON is a popular request for ACE, please provide an example how to add a JSON Schema Service to ace-linters.
See:

https://github.com/ajaxorg/ace/issues/2883
https://github.com/ajaxorg/ace/issues/3885

@whazor hey! This wiki page should make it


Converting DEXTR model to Inference Engine format

Hello!
I'm going to fine-tune (hope this will help) a DEXTR model on my data and integrate it into CVAT.
Could you please guide me how to convert trained in PyTorch (DEXTR training code is only available in PyTorch) and replace current DEXTR model file with my own?
Thank you!

@AlexFridman
Hi, long time ago I've written a script to convert the model. It worked, but I am not sure it works now. Nevertheless you can investigate how conversion can be done.
You must install at least git, openvino, python, tensorflow, and keras and probably some minor packages.
I attached scripts as an archive. Just execute convert.sh.
arch.zip

@AlexFridman
To replace a default model in CVAT, please consider modification of Dockerfile.
ENV WITH_DEXTR=${WITH_DEXTR}
ENV DEXTR_MODEL_DIR=${HOME}/dextr
RUN if [ "$WITH_DEXTR" = "yes" ]; then \
        mkdir ${DEXTR_MODEL_DIR} -p && \
        curl https://download.01.org/openvinotoolkit/models_contrib/cvat/dextr_model_v1.zip -o ${DEXTR_MODEL_DIR}/dextr.zip && \
        7z e ${DEXTR_MODEL_DIR}/dextr.zip -o${DEXTR_MODEL_DIR} && rm ${DEXTR_MODEL_DIR}/dextr.zip; \
    fi

Finally, DEXTR_MODEL_DIRis expected to specify a directory with dextr.bin and dextr.xml

Thank you, guys! If I'll manage to run it -- I'll write you back!

@bsekachev, could you please specify keras, tensorflow and pytorch version you used?
And what is networks package?

@AlexFridman
networks is https://github.com/scaelles/DEXTR-KerasTensorflow/tree/master/networks
Unfortunately I cannot help you with exact versions.


[Windows] Add cbindgen and bindgen pester test

Description
Add additional Pester tests for Rust:

cbindgen
bindgen

Replace bindgen to bindgen-cli - https://github.com/rust-lang/cargo/issues/11249
Related issue:
https://github.com/actions/runner-images-internal/issues/4478
Check list

[x] Related issue / work item is attached
[x] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp windows2019,windows2022

/azp run windows2019,windows2022

/azp run windows2019,windows2022


Adds dependency injection instructions and tests

Before, READMEs were all over the place with regards to configuration.
This standardizes them and corrects some dependency and test
parameters. Notably, this adds tests for Jersey 1.x (they never existed
previously).
Once this is in, I can help rebase the JAXRS-2 feature on top (#272)

cc @cschneider @jplock


Add Shijoshugi Adtruck by hololive IDOL PROJECT

Song Name
Shijoshugi Adtruck
Artist Name
hololive IDOL PROJECT
Source
Youtube
Youtube Link
OFibr_nsnQ8
Direct File Link
No response
Song ID
590577
Start Offset [ms]
600
Extra
Created from Auto Nong.
Created for level: 10092077
Downloaded: Yes

accept


Arming issue

I am new to drone simulators and doing what was laid out, i am getting this issue
in the mav console
AP: PreArm: Main loop slow (271Hz < 400Hz)
AP: PreArm: Main loop slow (272Hz < 400Hz)
Flight battery 100 percent
AP: PreArm: Main loop slow (272Hz < 400Hz)
AP: PreArm: Main loop slow (272Hz < 400Hz)
Flight battery 100 percent
AP: PreArm: Main loop slow (272Hz < 400Hz)
what should i do?

@AradhyaSpace11 Hi. This error usually indicated that the simulator is not getting enough resources such as CPU. You can start with only 1 drone to see if it is running ok or not. As a workaround you can disable the arming check by typing >> param set ARMING_CHECK 0 in the first terminal like below:
param set ARMING_CHECK 0
mode guided
arm throttle
takeoff 5

Thank You so much for the quick response. I tried your solution and it worked as intended. I have been searching for a proper simulator tool for beginners and almost all of the documentations are too technical, which I believe helps if the users are experts but it does confuse newcomers. Thank you so much for providing concise documentation. My team appreciates the effort a lot

@AradhyaSpace11 Your welcome <3

Thank You for the quick reply to the other issue as well


hub clone dose not work well

Command attempted:
hub clone somerepo
What happened:
did not work, and displayed the permission error
More info:
Under macOS
I like the clt, since it help me shorten the commands, but it dose not work well recently. I don't know the reason.
git clone https://github.com/Me/somerepo  OK
hub clone https://github.com/Me/somerepo  OK
but
hub clone Me/somerepo  dose not work
hub clone somerepo  dose not work
and shows the following error
cloning to 'life-log'...
Enter passphrase for key '/Users/william/.ssh/id_rsa':
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: can not read the remote repo.

Check the permission of accession and the existence of repo.

hub's clone command use git/SSH protocol by default, so you have to check your public SSH keys are added to Settings > SSH and GPG keys, or SSH passphrase is correct.
If you want to change default protocol to HTTPS, use this command.
$ git config --global hub.protocol https


Docs for basic usage

Is it possible to have some simple basic examples which show a cargo project and a dune project working together

I think it is possible

May some way possible

it is possible to get builds going with ocamlrep, dune & cargo. see errpy for example which utilizes ocamlrep. errpy is available in opam and is integrated into pyre-check
i would advocate for the use of buck2 though. one can use reindeer for the rust dependencies and ocaml-scripts for the opam dependencies. there are some ocaml examples (including rust embedding) in the buck2 repo (sadly not involving ocamlrep though). ocamlrep itself builds with buck2 and reindeer in addition to cargo.

Hopefully https://github.com/facebook/ocamlrep/issues/45#issuecomment-1971799618 helps.
There hasn't been action here for a while, so closing.
Sorry there aren't more docs–I don't think this project is under active dev.


[Bug]: Adding webpack plugin skips the actual build

Version
System:
    OS: macOS 14.6.1
    CPU: (12) arm64 Apple M3 Pro
    Memory: 80.20 MB / 36.00 GB
    Shell: 5.9 - /bin/zsh
  Browsers:
    Chrome: 128.0.6613.113
    Safari: 17.6
  npmPackages:
    @rsdoctor/webpack-plugin: ^0.3.11 => 0.3.11
<EMAIL_ADDRESS>
Details
I'm trying to debug this, when adding plugin at the end of the webpack plugin list, it makes the actual build skips and the report is not generated as well. Tried enabling debug but doesn't seem to be helpful.
RSDOCTOR=true RSDOCTOR_DEBUG=true NODE_OPTIONS=--max-old-space-size=8192 WEB_ENVIRONMENT=production yarn build
yarn run v1.22.22
$ NODE_ENV=production webpack --mode=production
debug   [RsdoctorWebpackSDK][bootstrap start]<PHONE_NUMBER>228
debug   [RsdoctorWebpackSDK][bootstrap end]<PHONE_NUMBER>912 http://<IP_ADDRESS>:8330
info    Rsdoctor analyze server running on: http://<IP_ADDRESS>:8330/index.html
info    Rsdoctor analyze server running on: http://localhost:8330/index.html

Webpack is @5.94.0
Reproduce link
no repro, trying to debug
Reproduce Steps
RSDOCTOR=true RSDOCTOR_DEBUG=true NODE_OPTIONS=--max-old-space-size=8192 WEB_ENVIRONMENT=production yarn build
yarn run v1.22.22
$ NODE_ENV=production webpack --mode=production
debug   [RsdoctorWebpackSDK][bootstrap start]<PHONE_NUMBER>228
debug   [RsdoctorWebpackSDK][bootstrap end]<PHONE_NUMBER>912 http://<IP_ADDRESS>:8330
info    Rsdoctor analyze server running on: http://<IP_ADDRESS>:8330/index.html
info    Rsdoctor analyze server running on: http://localhost:8330/index.html

Webpack is @5.94.0
Would like to share webpack config but it is behind corp can't be shared publicly.

Update: there are some unhandled exceptions thrown
0.9243589743589743 sealing { percentage: 0.9243589743589743, message: 'sealing' }
TypeError: Cannot read properties of undefined (reading 'EVENT')

in internalprogressplugin when call sendAPIDataToClient.
const api = import_types.SDK.ServerAPI.APIExtends.GetCompileProgess;
            sdk.server.sendAPIDataToClient(api, {
              req: {
                api,
                body: void 0
              },
              res: currentProgress
            });

Not sure if it's related, the report page also shows
611.152126b1.js:460 WebSocket connection to 'ws://localhost:8368/socket.io/?EIO=4&transport=websocket&sid=E49DoQWZM-YUvM_mAABu' failed: WebSocket is closed before the connection is established


Content: Observability as Code guide for developer website

Objective
Create a guide for implementing observability as code based off this blog post.
Acceptance Criteria:

A new guide for practicing observability as code has been added to developer.newrelic.com
The guide uses the latest version of the Terraform provider
The golden signals can be provisioned via the Terraform provider by a new and existing user

Resolved by https://github.com/newrelic/developer-website/pull/837


Socket hosting model not working

I faced issue with newly created Socket hosting model. It not respond for any requests. I not really sure what exact goes wrong. It could be enviroment issue or so.
My machine is win10 x64 with latest updates.
Steps to reproduce

Open ReactGrid sample
Specify Socket hosting model by adding services.AddNodeServices(new NodeServicesOptions { HostingModel = NodeHostingModel.Socket }); to your ConfigureServices method.
Run dotnet run command
Try to open localhost:5000

Expected behaivior
Page rendered
Actual behaivior
Page stuck in infinite load
Note:
If run under old fashion Http hosting model everything work perfect.
UPD:
after long long wait for response i recieve following error:
System.IO.IOException: Pipe is broken.
   at System.IO.Pipes.PipeStream.CheckWriteOperations()
   at System.IO.Pipes.PipeStream.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.NodeServices.HostingModels.VirtualConnections.VirtualConnectionClient.<WriteAsync>d__19.MoveNext() in C:\dev\Source\Repos\NodeServices\src\Microsoft.AspNetCore.NodeServices\HostingModels\VirtualConnections\VirtualConnectionClient.cs:line 188

Thanks for pointing this out.
In my final commit I'd missed out the PipeOptions.Asynchronous option that's needed on Windows for the named sockets transport not to just hang. Fixed now.

Is the fix for this one out? I keep getting One or more errors occurred. (Pipe is broken.) intermittently  (no error log or stack trace)

Is the fix for this one out?

Yes, the original issue reported by @laskoviymishka - that it never works - is fixed and was released as beta-000004.
If you have a scenario where it fails intermittently, that must be something different.
Based on your description, I'm pretty sure your intermittent issue is the same as #128, hence not reopening this issue. Let's keep any discussion about the intermittent thing in #128.


fix(speedtest-cli): avoid speedtest DNS error

Description
Avoid intermitten 403 Forbidden error caused by speedtest DNS by using --secure switch.
Related Issue(s)
None on this repo.

Hello there, thanks for the PR. Were there any updates, or why is this flag needed?

Hi, i have been testing on multiple hosts and VPSs( EC2, Hosthatch, DO, local). There seems to be intermitten issues with resolving on http. Intermittenly getting 403 Forbidden errors when not using https.
Seems like many others also having this issue, most resolve by itself given enough time. But forcing https does solve the issue.
https://ubuntu-mate.community/t/speedtest-cli-error/25722
https://www.reddit.com/r/Ubiquiti/comments/xnhze3/speedtestcli_on_erx_stopped_working/
https://www.reddit.com/r/linux4noobs/comments/w6d5hg/speedtestcli_only_works_with_secure/
https://community.ui.com/questions/Speedtest-403-error/8a4153fe-1259-48c5-abd7-bd3140b7f277

Looks good to me! Thank you for your contribution. I will let it run on the dev branch and eventually merge it into main, if it works.

:tada: This issue has been resolved in version 4.5.4
Please check the changelog for more details.


[BUG] CSV Parser Requires Undocumented Column

Describe the bug
I am using the CSV parser to generate statements from my IB and E*TRADE accounts. I created manually a CSV file which I want to use as an input to the CSV parser. Upon executing the latter requires column symbol_description which is not mentioned in the README.md.
To Reproduce

Use the following csv:

trade_date,activity_type,company,symbol,quantity,price,amount
30.10.2020,BUY,Hecla Mining Inc.,HL,215,4.64,997.83
17.12.2020,SELL,Hecla Mining Inc.,HL,215,6.36,1344.8

Run python3 stocks.py -i ~/Desktop/input-dir -o ~/Desktop/output-dir -p csv

You will get:

Traceback (most recent call last):
  File "stocks.py", line 51, in <module>
    main()
  File "stocks.py", line 46, in main
    parsed_args.in_currency,
  File "/<some-path>/Projects/revolut-stocks/libs/process.py", line 62, in process
    statements[parser_name] = supported_parsers[parser_name](parser_input_dir).parse()
  File "/<some-path>/Projects/revolut-stocks/libs/parsers/csv.py", line 98, in parse
    activities = self.extract_activities(viewer)
  File "/<some-path>/Projects/revolut-stocks/libs/parsers/csv.py", line 73, in extract_activities
    "symbol_description": row[headers["symbol_description"]],
KeyError: 'symbol_description'

I don't see anything about symbol_description and it is not mandatory in CSV_REQUIRED_COLUMNS.
NOTE: I "fixed" it by deleting line csv.py#73.
Expected behavior
I expect the parser to work without specifying this column.
Desktop (please complete the following information):

OS: MacOS
Version: 10bc324ed7a62d9e34ce88c801033af7e4d0f73b

The symbol description column became mandatory since the introduction of SC activities. I've documented the column and added it to CSV_REQUIRED_COLUMNS.


feat: dependent resource in the condition instead of resource

Will add migration guide.

We should add an example where the dependent resource is actually used because, as it stands, this change only seems to complicate things while not actually addressing any particular issue…

We should add an example where the dependent resource is actually used because, as it stands, this change only seems to complicate things while not actually addressing any particular issue…

when this will be merged: https://github.com/java-operator-sdk/java-operator-sdk/pull/1688
I will adjust that sample.


Improve Account.send return format

Account.send (when called from a script) currently returns a string literally like the following:
Invoke transaction was sent.
Contract address: 0x0002baa99045493ae32f220712fcbebd7319b45013b2f0863d8f02c3af5d29dc
Transaction hash: 0x608e0d8880123eacbb62aeab8ed765d42d19186b5dfc220f54827c6f8f40056

If consuming this result in a Nile script, the user needs to parse this string to determine the transaction hash.
Instead, return a result that is more programmatically consumable, for example return only the transaction hash itself, or an actual object with fields.

The format has been improved as part of #320.


Gopro MAX .360 format

Hi,
There seems to be an issue with the video format coming from a GoPro MAX.
This article explains the problem.
File format: .360
Projection: EAC
However, the projection is in two different streams, so the file requires some processing before it can be converted into an equirectangular projection.
There is a fork of ffmpeg available to handle this format.
Here's what the current ffmpeg command does:
We can see that it only takes a single stream (missing the top and bottom of the original video, which is the second stream), and there are two lines where the pixels are duplicated.

And here's the result of exporting an image from the video.

I have tried playing around with ffmpeg, but I'm not familiar with all the filters. For example, one can combine the two streams to create the equirectangular view.
ffmpeg -i *.360 -filter_complex "[0:0]scale=iw:ih/2[top]; [0:1]scale=iw:ih/2[bottom]; [top][bottom]vstack[ou]; [ou]v360=eac:e" -r 0.1  frame_%05d.png

Did you try https://github.com/trek-view/max2sphere ?
Also note that equirectangular images coming from 360 cameras will have issues, because it's physically impossible to build a 360 snapshot camera with a single optical center. The stitched 360 images from your GoPro MAX or insta 360 or Ricoh Theta may look nice, but they will have parallax issues, especially if your scene contains objects that are close. So don't expect much quality from the resulting nerf.
This should probably be mentioned in the doc https://docs.nerf.studio/en/latest/quickstart/custom_dataset.html#data
Probably a better way to use these images is to use the raw fisheye images rather than the stitched equirectangular images.
True panoramic devices use either a catadioptric system (no classical lens), or a rotating optical system (eg the classical DSLR-on-a-gimbal), so that all optical rays pass through the same optical center.
More info: https://cave.cs.columbia.edu/projects/categories/project?cid=Computational+Imaging&pid=Catadioptric+Cameras+for+360+Degree+Imaging


[Forge 1.15.2] Boats uncontrollable

Aloha again,
the airships are now working 1a and are really fun, but still the boats are crazy.
When you use a boat, it always goes forward, even if you leave it, it always goes straight on xD
Unfortunately, you can then also no longer sit on it and it is also not possible to take it apart
Forge - 31.2.36
Mod - 1.2.4

fixed in new versions


Question

"When you create a virtual machine from an image, Azure creates a disk for the virtual machine that is a copy of the source .vhd file." => Does it mean that now we have an image, which is a VHD file with a lease and whose content doesn't change, and a disk, which is another VHD file whose content changes as the VM is being used?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3799662b-8f0f-c4e6-26b5-b2ff24a9242a
Version Independent ID: 0c93223c-f28b-7ca7-0d10-c19e5fea9197
Content: About unmanaged (page blobs) and managed disks storage for Microsoft Azure Windows VMs
Content Source: articles/virtual-machines/windows/about-disks-and-vhds.md
Service: virtual-machines-windows
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the question! We are investigating and will update you shortly.

@mcurko When a new machine is created from an image, a copy of the image VHD is created and you will be able to read and write on that. Generally that will be your C drive in windows. (https://docs.microsoft.com/en-us/azure/virtual-machines/windows/about-disks-and-vhds#operating-system-disk)
Another VHD (https://docs.microsoft.com/en-us/azure/virtual-machines/windows/about-disks-and-vhds#temporary-disk) is the temporary disk which is used for swap, pagefile etc.
We can set the data disks to be read-only or read-write.
Lease will be placed on all disks which are attached to the a virtual machines to prevent accidental deletion of underlying VHD files.
More information about VHD is here(https://docs.microsoft.com/en-us/azure/virtual-machines/windows/about-disks-and-vhds#about-vhds)

@mcurko Did the above answer help? Let us know.

@mcurko We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


The future of Simple.Odata.Client

Currently we have no new versions for a long time and pull requests are piling up forever. It is on a level where I would assume a lot of people do not even submit issues anymore, or put in any work to actually fix issues. Which means this library is falling behind - slowly, but faster.
I am working on a new odata client that is supposed to sit on top of simple odata client (removing the "totally untped" approach and generate client classes and libraries so I am able to combine ease of use together with the ability to get compile time errors in case the api surface changes) and I am i.e. considering not using simple.odata.client because of lack of updates.
It is quite clear that @object has moved on to greener pastures - most likely because his work has taken him away from odata so this is not an element he is actually using.
Could we please open a discussion about the future of this? Anyone has a contact of Vagif? Does it make sense to officially fork this off, rename it and put it under new maangement? Or would Vagif be ok to appoint a number of maintainers so that we could start updating references and merging in pull requests? I understand his terrific focus on quality - but at the same time, I start seeing this falling behind with no releases and this is not exactly helping with quality either.  There is a lot of errors that are showing up with more advanced usage (thanks also to the odata team finally moving forward with some core functionality). I am not sure at this point which part of this would be worse. I would be willing, i.e., to donate some time towards some bugs - either from me personally, or from some people in my team. But with PR piling up for months - not going to happen. Had this dicussion with the EfCore team too. What sense is a PR if it does not go into a release within weeks? Wasted time. So, some maintainers that put that into more active PR's would be highly appreciated.
@object Can you please comment?

@NetTecture
Thomas, thank you for bringing this up. Yes, you are right, I have been busy (actually very busy) with other things that no longer use OData. Moreover, I even no longer use C#! I am using F# for my daily assignments. My last project that used OData was completed about 4 years ago, so it's been quite a challenge to keep up with news in OData world without ability to eat your own dog's food. So I will be happy to handle future maintenance of the library to other developers assuming they are willing to keep up with quality and principles of the library. I have been in contact with @phatcher and @watfordgnf, and of course @NetTecture has been using and contributing to the library for a long time.
So I am open to suggestions. The current state is not satisfactory and turns people down. There should be a continuous focus on both bug fixes and new feature development. I can also be contacted at vagif.abilov at gmail to discuss possible future of the library.

@object Well, I am willing at least temporarily just to step in and validate pull requests and stuff. For me Odata is a core of client projects AND our own infrastructure (which is more long term important to me). While it has some quite serious problems (odata - the standard) it is imho the best thing that is around. What we do need fast is being able to push up new releases and get maintenance going, then put in some sort of more community oriented way.
As direction, we definitely need to get the library up to standards. Not in the way of "make serious architecture changes" for now but "fix the bugs that we found and implement new features that odata has - like 'in' which finally was added". There is hardly any damage you can make to the codebase (tracked in git anyway) that is not offset by people actually SEEING something happening. The next step, imho, should be to get this somehow working in webassembly - blazor client side is a major step forward but it also is in dire need of odata ;)
This should IMHO get a council of multiple people that are maintaining it - it needs not to be revived, it needs to prosper. Damn to MS for not having a decent client - without decent client odata has no chance of larger scale use. I am maintaining a software for a major customer at the moment and they actively avoided odata in the past for other projects because of the complexity of dealing with it on both ends (client and server). This is the typical landscape these days. Now, the webapi odata team is getting their act together - 3.1 nightly are quite good and seem brutally faster than old libraries - but the client is a sour page and I find myself, i.e. more and more writing pure html unit tests instead of simple odata to test backend functionality because simple odata does not work in certain scenarios tue to bugs or limitations.

@NetTecture That's very good. I hope we will have feedback from others, then we can decide how we proceed.

This library is a very important part of our products, and I'm on the hook for keeping this client library viable for our work. I'm available to review PR's and help shepherd the product going forward. However, because of the industry I work in we do not take updates frequently (our product is on an air-gapped network).
[aside: we're still on 5.6.1--and 4.3.0 before that--and are evaluating the jump to 5.10.X in our next release]

Thank you @watfordgnf, I have sent invitations to Simple.OData.Client organization to both you and @NetTecture, first as members, but we can promote you to owners.

Already seen it. I think the "most" important thing is getting the flow going, which will include whatever exists as build infrastructure (build servers, publishing). Is there anything? If not, it may be an idea to set up a (separate) microsoft azure devops organization for this - they integrate with github and can be used to host the build and deployment. Not saying we leave github - contrary. Azure would have all tracking disabled, basically just use the pipelines to make builds and upload upload them. I propose SHOULD come with a MINOR change on how branch naming conventions to allow gitversion to determine automatic version numbering ;) The goal basically being able to publish autoamtically to nuget. And make a new release based on merging in as many of the pull requests as we can. And then we should somehow make a rn through the open issues and prioritize them.
@object Can you make a wiki page on how to run the unit tests? Or is that absolutely trivial not requiring any setup?

@NetTecture It is trivial to run existing tests, they should just run. Extending tests may require taking taking care of HTTP response from the server and storing it in a repo, then using the stored file as for response validation. Library's unit tests are implemented in such away - they mock HTTP traffic.
And integration tests test against Microsoft's publicly available OData services, so they are also no brainers.

@object Interesting. Any automatic build setup so far or do you make deployments manually?

I should probably make a wiki page explaining how to do this. Currently it includes following steps:

Update version number
Build a solution
Run nugetpack.cmd  to create NuGet packages
Run nugetpush.cmd   to push packages to NuGet.

Oh that's surprising somehow but it's also good to know that you are ready to appoint some successors. @watfordgnf is a very strict reviewer which means this is good and helps a lot. :-)
It is important for me whether this repository will be active in the future or not, because I have so many ideas for the future. For example, I am working on a temporary Manager-Library that wraps this library and extends it with features like API versioning, authentication or performance topics. It would be nice to see these features in this repo someday.

@SiberaIndustries No doubt that the project will be active in the future. With almost a million of downloads of its packages there is no way we can stop it :-) This is why we are trying to organize it better so it won't suffer from a bottleneck of a single individual.

We are integrating with microsoft Dynamics365/CDS extensively and a lack of a propert microsoft client made using Simple a no brainer. We have taken the same path as @NetTecture  and built a client on top, with a class generator etc.
I've commited a few PRs recently and I'd be happy to help out with a few more on some features that we'd love to have like deep inserts, nested selects etc.

@NetTecture I don't mind setting up a Azure DevOps pipeline, we just need to come up with a sensible branching/versioning strategy so that we can publish betas etc,
For example, I didn't push my subtype changes for a long while as although they worked for me, I wasn't sure if they broke something else out in the wild.

What about GitHub Sponsor? I guess a lot of people use this library in a commercial way. I also use it in a commercial environment.
I want to ask every customer of me to pay 1€ per hour more, so that I can spent this money on open-source projects.


refetchInterval is missing in types for QueryOptions

Describe the bug
In my QueryCache.onSuccess / QueryCache.onError functions I use the property query.options.refetchInterval. Typescript complains and claims that property is not there, but it clearly is.

Your minimal, reproducible example
https://codesandbox.io/s/react-typescript-playground-forked-j6mf7d?file=/src/index.tsx
Steps to reproduce
If you look at the onSuccess and onError functions in index.tsx, you will see that Typescript complains and claims refetchInterval does not exist. In the console, however, you will see that the refetchInterval property does in fact exist.
Expected behavior
Typescript should accept query.options.refetchInterval without complaining.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
I guess this happens everywhere.
Tanstack Query adapter
None
TanStack Query version
4.28.0
TypeScript version
5.0.4
Additional context
No response

it's not missing. you opened a PR today with a "fix" for it, and I explained why that's not right:

https://github.com/TanStack/query/pull/5459#issuecomment-1562499459

nope, because it's an observer level property. Again, you could have two instances of useQuery, where each one has a different refetchInterval specified.
Why do you need access to it? What problem are you trying to solve?

@TkDodo I was just about to raise exact same issue, but luckily found this one.
My use case is that I check for throwOnError property. If it throws I do nothing, if it doesn't I show error toast.
An argument that we could have different useQuery with different observer options just doesn't click for me. QueryObserverOptions type properties are all optional and we could check them inside QueryCache handlers.
It is really frustrating to pass QueryObserverOptions to an error argument of an onError handler and at the same time declare in Types that we don't.

@nik-webdevelop when you look at a query in the cache, it can't know what the observers are doing. You can use the same query in two places, once with throwOnError: true and once with false:
const useTodos = (options: { throwOnError: boolean }) => {
  return useQuery({
    queryKey: ['todos'],
    queryFn: fetchTodods,
    ...options
}

then use this with useTodos({ throwOnError: true }) in one component and useTodos({ throwOnError: false }) in another. That's a feature. Maybe in one place, you're only interested in the count of todos, and you can fall back to 0 if there is an error, so you don't want to throw to an error boundary here. In other cases, you might. The same is true for refetchInterval. You can have a todo list that you want to refetch every 5 seconds, but you might want to have different observers that don't refetch the whole list. Those can be mounted in different parts of the app, at different times.
So when you look at the query how it's stored in the cache, it doesn't know anything about those options because there can be multiple observers with different values. We can't give you the one throwOnError value because that doesn't exist.
observers on a query are public but if it makes sense, we could expose those so that you could look at query.observers, map over them, and look at all options and then do what you want with them. I'm not fully convinced that this is necessary though. You can set throwOnError globally and set onError, and have one be the inverse of the other. It's what I do personally for example:
new QueryClient({
    defaultOptions: {
        queries: {
            throwOnError: (error, query) => query.state.data === undefined,
        },
    },
    queryCache: new QueryCache({
        onError: (error, query) => {
            if (query.state.data !== undefined) {
                showToast.error(error)
            }
        },
    }),
})

it works quite well.

Thanks for a quick response and great explanation!

look at a query in the cache, it can't know what the observers are doing

I've just tested that onError handler gets query.options with specific to observer options:
const { data } = useTodo();
// if error occurs here `query.options` will NOT have `staleTime` and `throwOnError`

const { data } = useTodo({ staleTime: 5000, throwOnError: true });
// if error occurs here `query.options` will have `staleTime` and `throwOnError`

So, although the cache doesn't keep track of observers, the onError handler itself gets query.options from a specific observer (where the error occurred) and thus gets QueryOptions + QueryObserverOptions
Regarding example that you provided: I've ended up with almost a simillar approach, but there is one issue with presented solution. If I set throwOnError: true and there is a refetch error, I end up with both ErrorBoundary and toast.
My attempt:
onError: (error, query) => {
  const { throwOnError } = query.options as QueryObserverOptions;

  const isThrowingErrorToBoundary =
    typeof throwOnError === 'function'
      ? throwOnError(error, query)
      : throwOnError;

  if (!isThrowingErrorToBoundary) {
    showToast({ ...toastParams });
  }
}

the onError handler itself gets query.options from a specific observer

I think this is just because internally, on useQuery, we set the options to the observer, which then spreads it onto the query. so we would need to omit the options that are observer specific when doing that, but it would add additional runtime code for no real gain. the typescript types forbid access to those fields and thats good enough for me. you probably see the options from the last observer. don't rely on it.

Ok, i see. So, this means that the logic that i am trying to implement is impossible?

If I set throwOnError: true and there is a refetch error, I end up with both ErrorBoundary and toast.

well do you want the boundary displayed or the toast?
but yeah, if you set something on useQuery directly that is an observer property, the global callback can't know about it

It would be nice to have access to the observer options, but I get it, thank you very much.


Add allowed characters after mai yamook

Refs https://github.com/bountonw/translate/pull/359#issuecomment-2254327051.

@bountonw The opening bracket is needed for mai yamook immediately preceding a footnote [^1].


issues with dask arrays returned by nd2:  depend on ND2File being open, can't be pickled

Hi Talley,
a  couple of observations regarding the dask arrays returned by nd2. None of this is terribly surprising (so you are probbably aware), and I am not expecting this can really be fixed. But maybe some lines in the readme file could save users some headaches.
keep ND2File object in open() state when working with dask array
For my use case I created a little helper function (pseudo-code):
def provide_tiles_as_daskarray_from_nd2(filepath):
      with nd2.ND2File(filepath) as f:
            arr = f.to_dask()
            # do a few more things on array: cropping, sorting dimensions
            return arr

I have similiar functions for other file types, e.g. something like provide_tiles_as_daskarray_from_folder_of_imgs(filepath).
On first sight, this works. I used this function from a Jupyter Notebook and it seemed as if it returns a working dask array.
Until you do anything that forces a compute(), in which case the python kernel crashes without a traceback.
The issue is that f is closed when returning out of the context manager. The same thing happens if you don't use a context manager and call f.close() before returning. Again, not terribly surprising. When running in the VS Code debugger I actually saw a trace back and saw that there is a segmentation fault when the nd2 library is trying to read data.
My workaround is to not close f which is a bit dirty as there will be open file handles hanging around possibly causing memory leaks.
cloudpickle
The dask arrays returned by nd2 can't be serialized using cloudpickle due to the ND2File object not being pickleable:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-20-8a76360da592> in <module>
----> 1 cloudpickle.dump(array,f)

~/miniconda3/envs/napari_latest/lib/python3.9/site-packages/cloudpickle/cloudpickle_fast.py in dump(obj, file, protocol, buffer_callback)
     53         compatibility with older versions of Python.
     54         """
---> 55         CloudPickler(
     56             file, protocol=protocol, buffer_callback=buffer_callback
     57         ).dump(obj)

~/miniconda3/envs/napari_latest/lib/python3.9/site-packages/cloudpickle/cloudpickle_fast.py in dump(self, obj)
    561     def dump(self, obj):
    562         try:
--> 563             return Pickler.dump(self, obj)
    564         except RuntimeError as e:
    565             if "recursion" in e.args[0]:

~/miniconda3/envs/napari_latest/lib/python3.9/site-packages/nd2/_sdk/latest.cpython-39-x86_64-linux-gnu.so in nd2._sdk.latest.ND2Reader.__reduce_cython__()

TypeError: no default __reduce__ due to non-trivial __cinit__

Again, not terribly surprising. However, the implication is (I haven't tried it, but am fairly certain this is the case) that the dask arrays returned by nd2 cannot be used in a dask.distributed context where the pickled array needs to be sent to the various worker processes.
A workaround for both would be to create a temporary .zarr file (pseudo-code):
def provide_tiles_as_daskarray_from_nd2(filepath):
      with nd2.ND2File(filepath) as f:
            arr = f.to_dask()
            # do a few more things on array: cropping, sorting dimensions
            arr.to_zarr("_tmpfile.zarr")
            f.close()
            return dask.array.from_zarr("_tmpfile.zarr")

with obvious drawback of duplicating the required storarge space.
I don't know whether there is much that can be done about this, but it may be useful to add a note of caution in the Readme.

Actually, I just looked a bit into un/pickling and it seems that adding these to methods to the ND2File class solve the pickling problem:
    def __getstate__(self):
        state = self.__dict__.copy()
        del state['_rdr']
        return state

    def __setstate__(self, d):
        print("I'm being unpickled with these values: " + repr(d))
        self.__dict__ = d
        self._rdr = get_reader(self._path)
        if not self._closed:
            self.open()

yeah, this is a tricky one... I actually wrote a wrapper over at aicsimageio for just this purpose: https://github.com/AllenCellModeling/aicsimageio/blob/main/aicsimageio/utils/dask_proxy.py
could use it here as well?

Ugh. I would have to do a bit more reading to fully understand how that proxy works, so not sure about whether this would work here. However, as aicsimageio also incorporates nd2, maybe I should just aicsimageio instead of nd2 (I'd prefer not to have that dependency though).

the gist of that proxy is essentially these two lines:
    def compute(self, **kwargs: Any) -> np.ndarray:
        with self.__wrapped__._ctx_:
            return self.__wrapped__.compute(**kwargs)

    def __array__(self, dtype: str = None, **kwargs: Any) -> np.ndarray:
        with self.__wrapped__._ctx_:
            return self.__wrapped__.__array__(dtype, **kwargs)

that is, it's a dask array that, whenever you try to call compute or np.asarray, will re-open the underlying file (with self.__wrapped__._ctx_ is essentially just with ND2File()....)
It looks a tad bit risky at first, but I haven't run into any issues with it yet.  In any case, I suspect the issue of trying to use a dask array after closing the file is far more common than whatever hidden issues there are with this proxy.  I'm inclined to try it

It looks a tad bit risky at first, but I haven't run into any issues with it yet. In any case, I suspect the issue of trying to use a dask array after closing the file is far more common than whatever hidden issues there are with this proxy.

I agree.
Also, if you do try, you will currently just see a crash, most likely without any usable error message.
So if I understand you correctly, the idea would be that ND2File.to_dask() returns such an ObjectProxy instead of the dask.array?

So if I understand you correctly, the idea would be that ND2File.to_dask() returns such an ObjectProxy instead of the dask.array?

yeah, exactly.

I think #27 and #26 have improved the situation here. closing until we get another reproducible bug.
I tried on my ubuntu box to get the segfault you showed, but am still failing (even running it a few hundred times in case of race condition)... but let's keep an eye out for it

Hi @tlambert03,
didn't get around to it yesterday but installed the latest version including #27 from the main branch just now.
Unfortunately, on my machine this has not been an improvement.
The segfault issue when returning the dask array from context manager still persists.
In addition, I now also get dying kernels (probably segfault, but this is something I run in a notebook) when I run some more complex computation on the dask array that worked previously when avoiding returning from a context manager.
I will try and provide a reproducible example, but probably won't get around to it today.
For the time being I'm holding on to an older state of the repo where everything except the context manager works.

ugh 😔  i'm sorry. this is a really frustrating one.
I can't get it to segfault on any of my computers (tried mac, win, ubuntu20)... maybe it's a race condition that depends on dataset size?
It's certainly an issue though, since it looks like aicsimageio is also having CI issues with the new release.  Will keep digging... but let me know if you hit on anything (and send files if necessary)

Will do. The newly observed segfault happens when I perform map_blocks on
the dask array, presumably things are happening concurrently for different
blocks.
I hope to create a minimal example by the end of the week.
On Tue, 16 Nov 2021 at 16:15, Talley Lambert @.***>
wrote:

ugh 😔 i'm sorry. this is a really frustrating one.
I can't get it to segfault on any of my computers (tried mac, win,
ubuntu20)... maybe it's a race condition that depends on dataset size?
It's certainly an issue though, since it looks like aicsimageio is also
having CI issues with the new release. Will keep digging... but let me know
if you hit on anything (and send files if necessary)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tlambert03/nd2/issues/19#issuecomment-970372564, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ3R7D4PXFVJIZS2AO67TLUMJYSNANCNFSM5HXM6MZA
.
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.

BTW, no need to be sorry and thanks for all the great support so far !
On Tue, 16 Nov 2021 at 16:22, Volker Hilsenstein <
@.***> wrote:

Will do. The newly observed segfault happens when I perform map_blocks
on the dask array, presumably things are happening concurrently for
different blocks.
I hope to create a minimal example by the end of the week.
On Tue, 16 Nov 2021 at 16:15, Talley Lambert @.***>
wrote:

ugh 😔 i'm sorry. this is a really frustrating one.
I can't get it to segfault on any of my computers (tried mac, win,
ubuntu20)... maybe it's a race condition that depends on dataset size?
It's certainly an issue though, since it looks like aicsimageio is also
having CI issues with the new release. Will keep digging... but let me know
if you hit on anything (and send files if necessary)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tlambert03/nd2/issues/19#issuecomment-970372564, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ3R7D4PXFVJIZS2AO67TLUMJYSNANCNFSM5HXM6MZA
.
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.

Ok, I started debugging this. The problem appears to be that the pickling doesn't work correctly if there are additional operations in the compute graph. I don't have a stand-alone example yet but maybe the following will already be giving you enough information:
I have the following function that loads an nd2 and performs some re-sorting (using np.einsum) of dimensions before returning it
def load_tiles_nd2(file_path: PathLike) -> da.array:
    file_path = Path(file_path)
    if file_path.suffix.lower() != ".nd2":
        raise (ValueError, "Can only read nd2 files.")
    f = ND2File(file_path)
    tile_array = f.to_dask()
    dim_order_tuple = f.sizes.keys()
    dim_order = "".join(dim_order_tuple).lower()
    # the followig is basically np.einsum (f"{dim_order}->pcyx")
    ordered_tile_array = to_tczyx(tile_array, in_order=dim_order, out_order="pcyx")
    return ordered_tile_array

So when I use this as in
arr = load_tiles_nd2("samplefile.nd2")
arr[..., :10].compute()

there is no problem.
However, when I do
tmp = load_tiles_nd2("samplefile.nd2")
arr = pickle.loads(pickle.dumps(tmp))
arr[..., :10].compute()

I get a segmentation fault.
When I look at the dask graph of the ResourceBackedArray that hasn't been pickled looks like that:

In contrast, the dask task graph for the ResourceBackedArray that has been pickled and regenerated by unpickling looks like this:

So the layers 0 to 2 lose important attributes such as .shape and .chunksize when pickling/unpickling.

Arghh. Now I can't reproduce the SegFault with a simple [...,:10].compute() reliably anymore, I can only reproduce it with a more complicated map_blocks. Will try to narrow it down further, but maybe the differences in the compute graph layers are relevant.

So the segfault seems to depend on the slicing.
tile_arr[...,:10].compute() works
tiles_arr[0,0,...] causes segfault.

Thanks for digging!
By the way I got a good response on SO about the general problem of adding tasks to a dask array graph if you're curious. , might try it here:
https://stackoverflow.com/a/70012999

Thanks, interesting, I didn't even think about graph optmiziation steps.
I sent you a screen recording of a debug session in action through the napari zulip chat. Unfortunately can't attach .webm to Github.

I think this has been solved over the iterations.  currently:
In [1]: import nd2

In [2]: f = nd2.ND2File('tests/data/dims_p1z5t3c2y32x32.nd2')

In [3]: d = f.to_dask()

In [4]: f.close()

In [5]: import cloudpickle

In [6]: p = cloudpickle.dumps(d)


Roll engine 21ae92651a38..4003fbc3ed48 (6 commits)

git@github.com:flutter/engine.git/compare/21ae92651a38...4003fbc3ed48
git log 21ae92651a38..4003fbc3ed48 --no-merges --oneline
2019-08-19<EMAIL_ADDRESS>Notify framework to clear input connection when app is backgrounded (#35054) (flutter/engine#9498)
2019-08-19<EMAIL_ADDRESS>Re-enable firebase test and don't use google login (flutter/engine#11228)
2019-08-19<EMAIL_ADDRESS>Update tflite_native and language_model revisions to match the Dart SDK (flutter/engine#11230)
2019-08-19<EMAIL_ADDRESS>Roll src/third_party/skia f8221786d088..8566dda51b42 (4 commits) (flutter/engine#11220)
2019-08-19<EMAIL_ADDRESS>Update metal layer drawable size on relayout. (flutter/engine#11224)
2019-08-19<EMAIL_ADDRESS>Make firebase testlab always pass (flutter/engine#11226)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (stuartmorgan@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5731769213517824

Commit queue failed; closing this roll.


Nav bar collapse/dropdown doesnt seem to work.

Describe the bug
The navigation toggle button for small screen sizes to display nav-links doesn't work/ doesn't serve its purpose of showing nav links, in short nothing happens when the toggle button is clicked.
To Reproduce
Steps to reproduce the behavior:
1.
2.
3.
4.
Expected behavior
A clear and concise description of what you expected to happen.
Logs
If applicable, add logs to help explain your problem (e.g. the error message and/or exception traceback).
Environment (please complete the following information):

OS: Windows 10
Browser: Chrome

Additional context
Add any other context about the problem here.

I am Vedant Rahane, GSSOC'21 participant.
I would like to work on this issue.
Please @urvashi-code1255 and @Vishal-raj-1, can you assign it to me.

Hello There! I am Prince Kumar, GSSOC 2021  participant  I would like to work on this issue.
Please @urvashi-code1255 and @Vishal-raj-1, can you assign it to me.

Already resolved!


Add YugabyteDB in the extension list

@FranckPachot I don't see hacktoberfest label in your repo.
Something like this:

Please add and let us know so that we can merge

Ok, done. Added the hacktoberfest topic

Add it


Improve deterministic dataset behaviour

[X] Address #33
[X] fixes glob(*) behaviour which is known to not be deterministic and can cause non-reproducible behaviour. See this paper: https://pubs.acs.org/doi/10.1021/acs.orglett.9b03216 !
[X] remove default augmentations for all datasets
[X] use centered segements for sourcefolder dataset

@TE-StefanUhlich can you review this? I think this is an even simpler solution

@TE-StefanUhlich I've tested deterministic behaviour now for all datasets. Seems work fine.
Also now switched to center segments instead of start=0 when random_chunks=False for the sourcefolder dataset.

I increased the number of unit tests for the dataloaders
@TE-StefanUhlich  can we merge this?

I am fine with the changes - you can merge.


Strange behaviour for tags with dashes

If I try to add a tag with dash (-) inside it, the tag will be placed as continuation of task title instead of placing in tags list. The same thing happens for /. For \ app is crashed completely.

It's not strange, it's an expected behaviour. From Taskwarrior docs:

A tag may be any single word that does not start with a digit, punctuation, or mathematical operator.

I'll investigate about crash caused by \ symbol.

I'll create an issue in TW repo. But for now we could prevent app from crashing by filtering out such symbols, or forbid to save such data.


how do we setup this?

i don't know how to setup this, can you help?

well i have repo called gamesense.dll; it gives the sentry DSN and you can replicate the source. You can build the other gamesense.crackcrackcrack easily with a azure dev ops starter and github workflow; build artifcats are in the<EMAIL_ADDRESS>branch; the build itself will be under demo.


Got key code 255 (0xff), did not match any known key (masked vals tried: 255 (0xff), 255 (0xff))

cv2::waitKey() is continually returning a value of 255.   The following message repeats forever:
"Got key code 255 (0xff), did not match any known key (masked vals tried: 255 (0xff), 255 (0xff))"
The endlessly repeated key causes the application to behave badly, preventing it from computing and displaying the visualizations.
Ubuntu 16.04, 64 bit
OpenCV 3.2.0

Successfully working around the problem by not appending keys if they have a value of 255
at line 160 live_vis.py
             if (key != 255):
                    keys.append(key)

Thanks Jim, spent a good few hours trying to debug this issue.

Merged PR #99; thanks for the fix!


Remove version: Hesan.He3 version 1.3.23

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

This appears to be an infrastructure issue, and we should wait to see if the publisher resolves it before merging this.

162573

URL: https://he3-1309519128.cos.accelerate.myqcloud.com/1.3.23/He3_windows_installer_ia32_1.3.23.exe
Response status code does not indicate success: 400 (Bad Request).
(Automated message - build 886)

Close with reason: PR submitted to change host;


Fix missing supplier_pack_filename variable from template

supplier_pack_filename had been forgotten from our variables passed through to the template, causing the 'Download guidance and legal documentation (.zip)' link to break.

👍  once it's green.


Reference applications and devices by name instead of id

@Page- Can you re check that the filters are applied correctly on the places you commented please?


Install-Module does not work 'Could not get response from query' when Register-PSRepository is ran with v3 endpoint

For anyone like me who was struggling to complete the last Install-Module step, beware it does not seem to work when Register-PSRepository is ran with a v3 nuget endpoint (which Azure DevOps suggests you use).
WARNING: Could not get response from query
'https://pkgs.dev.azure.com/executivecentre/_apis/packaging/4195951b-9f46-4fbb-8dae-75019870fcdc/nuget/commitLog/9a9
fb5d43cc3496e94a670054a121492'.
PackageManagement\Install-Package : No match was found for the specified search criteria and module name
'Install-TecDeploymentPrerequisites'. Try Get-PSRepository to see all available registered module repositories.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1:1772 char:21
+ ...          $null = PackageManagement\Install-Package @PSBoundParameters
+                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package],
    Exception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

I am using PowerShellGet v<IP_ADDRESS>, the default for most Windows machines I believe.
If the v2 endpoint is registered instead (which this article suggests to use), then things worked fine for me.
Maybe a footnote should be added about v3 not working?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 35b3cd51-b365-433b-9149-697582d64e88
Version Independent ID: 1cb1f4a3-5928-223d-592f-f3462fef4e4c
Content: Use Azure Artifacts as a private PowerShell repository
Content Source: docs/artifacts/tutorials/private-powershell-library.md
Product: devops
Technology: devops-artifacts
GitHub Login: @elbatk
Microsoft Alias: elbatk

Hi @stephentung-tec, thanks so much for reaching out. I'm glad it worked out for you in the end. I've created a user story for my backlog to address this. Thanks again!

Hi @stephentung-tec, Thank you again for the feedback. I ran through the process and you are correct that you have to define a v2 endpoint as PowerShell doesn't support NuGet v3. Here is a Note specifying this under the copy block:

https://docs.microsoft.com/en-us/azure/devops/artifacts/tutorials/private-powershell-library?view=azure-devops#connecting-to-the-feed-as-a-powershell-repo
I believe that clarifies correctly. However, if you have more feedback or feel like it's not clear enough, the input is greatly appreciated!

I've been trying to push a package to a v2 url but it either doesn't show up in the feed, or it shows up in the v3 list.
I'm not sure that AzureDevOps feeds suppord a v2 feed at all.


set qps and burst for agent rest config

https://issues.redhat.com/browse/ACM-5096

Will set the log level in another pr

/hold

/retest

/retest

/retest

/retest

/unhold


Bulk group update throwing error

The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/gam/wiki

Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email” show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv from our directory
Actual outcome (what errors or bad behavior do you see instead?): I am receiving Group<EMAIL_ADDRESS>Update Failed: Missing/Invalid attribute value: id

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam update group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam update group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email” show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv from our directory
Actual outcome (what errors or bad behavior do you see instead?): I am receiving Group: @.***, Update Failed: Missing/Invalid attribute value: id
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1623, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI.
You are receiving this because you are subscribed to this thread.

Ross, I wonder if we could detect smart quotes anywhere on the gam command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in doubt,
start at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv
from our directory
Actual outcome (what errors or bad behavior do you see instead?): I am
receiving Group: @.***, Update Failed: Missing/Invalid attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

Jay,
GAM could step across the arguments and toss an error if the arg starts with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in doubt,
start at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv
from our directory
Actual outcome (what errors or bad behavior do you see instead?): I am
receiving Group: @.***, Update Failed: Missing/Invalid attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI.
You are receiving this because you commented.

I'd go so far as to say smart quotes don't belong in any part of a gam
command. I suppose they could be used in a description string but it's rare
they are on purpose and seems unlikely they'd be expected to be preserved.
Performance -wise I think it's be pretty quick. I'm split between throwing
an error (clean up your data) or just replacing with non smart quotes on
the fly. But I do think it'd be a better, more informed user experience
than GAM's current naive handling.
Jay
On Sat, Apr 22, 2023, 9:08 AM Ross Scroggs @.***> wrote:

Jay,
GAM could step across the arguments and toss an error if the arg starts
with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam
command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam
update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam
update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in doubt,
start at the discussion forum and return here only when instructed to
do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki <
https://github.com/jay0lee/gam/wiki>
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv
from our directory
Actual outcome (what errors or bad behavior do you see instead?): I
am
receiving Group: @.***, Update Failed: Missing/Invalid attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

Jay,
I've thought about trying to clean up the command but because the parsing is all messed up, I'd just throw an error.
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:16 AM, Jay Lee @.***> wrote:
I'd go so far as to say smart quotes don't belong in any part of a gam
command. I suppose they could be used in a description string but it's rare
they are on purpose and seems unlikely they'd be expected to be preserved.
Performance -wise I think it's be pretty quick. I'm split between throwing
an error (clean up your data) or just replacing with non smart quotes on
the fly. But I do think it'd be a better, more informed user experience
than GAM's current naive handling.
Jay
On Sat, Apr 22, 2023, 9:08 AM Ross Scroggs @.***> wrote:

Jay,
GAM could step across the arguments and toss an error if the arg starts
with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam
command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam
update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam
update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in doubt,
start at the discussion forum and return here only when instructed to
do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki <
https://github.com/jay0lee/gam/wiki>
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv
from our directory
Actual outcome (what errors or bad behavior do you see instead?): I
am
receiving Group: @.***, Update Failed: Missing/Invalid attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518655391, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL6TGL3QVRTGYC55FT3XCPK4DANCNFSM6AAAAAAXHQQXKI.
You are receiving this because you commented.

Ok, then let's just throw an error if smart quotes are anywhere in the
command with a helpful explanation of what to look for to fix the command.
Jay
On Sat, Apr 22, 2023, 9:58 AM Ross Scroggs @.***> wrote:

Jay,
I've thought about trying to clean up the command but because the parsing
is all messed up, I'd just throw an error.
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:16 AM, Jay Lee @.***> wrote:
I'd go so far as to say smart quotes don't belong in any part of a gam
command. I suppose they could be used in a description string but it's
rare
they are on purpose and seems unlikely they'd be expected to be
preserved.
Performance -wise I think it's be pretty quick. I'm split between
throwing
an error (clean up your data) or just replacing with non smart quotes on
the fly. But I do think it'd be a better, more informed user experience
than GAM's current naive handling.
Jay
On Sat, Apr 22, 2023, 9:08 AM Ross Scroggs @.***> wrote:

Jay,
GAM could step across the arguments and toss an error if the arg starts
with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam
command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam
update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam
update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do
I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in
doubt,
start at the discussion forum and return here only when instructed
to
do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this
issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki <
https://github.com/jay0lee/gam/wiki>
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my
csv
from our directory
Actual outcome (what errors or bad behavior do you see
instead?): I
am
receiving Group: @.***, Update Failed: Missing/Invalid
attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
<
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855>,
or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885>,
or
unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518655391>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL6TGL3QVRTGYC55FT3XCPK4DANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518666499, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMA6NMQ22LRE7D2JD5LXCPPZHANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

Left     single curly quote: Latin-1 \x91 MacOS \xd2 UTF-8 \x20\x18
Right double curly quote: Latin-1 \x93 MacOS \xd4 UTF-8 \x20\x1c
Ross Scroggs
@.***

On Apr 22, 2023, at 7:03 AM, Jay Lee @.***> wrote:
Ok, then let's just throw an error if smart quotes are anywhere in the
command with a helpful explanation of what to look for to fix the command.
Jay
On Sat, Apr 22, 2023, 9:58 AM Ross Scroggs @.***> wrote:

Jay,
I've thought about trying to clean up the command but because the parsing
is all messed up, I'd just throw an error.
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:16 AM, Jay Lee @.***> wrote:
I'd go so far as to say smart quotes don't belong in any part of a gam
command. I suppose they could be used in a description string but it's
rare
they are on purpose and seems unlikely they'd be expected to be
preserved.
Performance -wise I think it's be pretty quick. I'm split between
throwing
an error (clean up your data) or just replacing with non smart quotes on
the fly. But I do think it'd be a better, more informed user experience
than GAM's current naive handling.
Jay
On Sat, Apr 22, 2023, 9:08 AM Ross Scroggs @.***> wrote:

Jay,
GAM could step across the arguments and toss an error if the arg starts
with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam
command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam
update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam
update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do
I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in
doubt,
start at the discussion forum and return here only when instructed
to
do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this
issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki <
https://github.com/jay0lee/gam/wiki>
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my
csv
from our directory
Actual outcome (what errors or bad behavior do you see
instead?): I
am
receiving Group: @.***, Update Failed: Missing/Invalid
attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
<
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855>,
or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885>,
or
unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518655391>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL6TGL3QVRTGYC55FT3XCPK4DANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518666499, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMA6NMQ22LRE7D2JD5LXCPPZHANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518667456, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL3P3UKRUDS24G6V5CLXCPQLDANCNFSM6AAAAAAXHQQXKI.
You are receiving this because you commented.

Correction:
Left     single curly quote: Latin-1 \x91 MacOS \xd2 Unicode \x2018 UTF-8 \xE2\x80\x98
Right double curly quote: Latin-1 \x93 MacOS \xd4 Unicode \x201c UTF-8 \xE2\x80\x9C
Ross Scroggs
@.***

On Apr 22, 2023, at 7:34 AM, Ross Scroggs @.***> wrote:
Left     single curly quote: Latin-1 \x91 MacOS \xd2 UTF-8 \x20\x18
Right double curly quote: Latin-1 \x93 MacOS \xd4 UTF-8 \x20\x1c
Ross Scroggs
@.*** @.***>

On Apr 22, 2023, at 7:03 AM, Jay Lee @.*** @.***>> wrote:
Ok, then let's just throw an error if smart quotes are anywhere in the
command with a helpful explanation of what to look for to fix the command.
Jay
On Sat, Apr 22, 2023, 9:58 AM Ross Scroggs @.***> wrote:

Jay,
I've thought about trying to clean up the command but because the parsing
is all messed up, I'd just throw an error.
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:16 AM, Jay Lee @.***> wrote:
I'd go so far as to say smart quotes don't belong in any part of a gam
command. I suppose they could be used in a description string but it's
rare
they are on purpose and seems unlikely they'd be expected to be
preserved.
Performance -wise I think it's be pretty quick. I'm split between
throwing
an error (clean up your data) or just replacing with non smart quotes on
the fly. But I do think it'd be a better, more informed user experience
than GAM's current naive handling.
Jay
On Sat, Apr 22, 2023, 9:08 AM Ross Scroggs @.***> wrote:

Jay,
GAM could step across the arguments and toss an error if the arg starts
with a curly single or double quote.
Is it worth the cost?
Ross
Ross Scroggs
@.***

On Apr 22, 2023, at 6:03 AM, Jay Lee @.***> wrote:
Ross, I wonder if we could detect smart quotes anywhere on the gam
command
and just throw an error with details to fix?
On Sat, Apr 22, 2023, 8:57 AM Ross Scroggs @.***> wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam
update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam
update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do
I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager https://groups.google.com/group/google-apps-manager>. When in
doubt,
start at the discussion forum and return here only when instructed
to
do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases https://github.com/GAM-team/GAM/releases> and I still have this
issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki <
https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki>
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my
csv
from our directory
Actual outcome (what errors or bad behavior do you see
instead?): I
am
receiving Group: @.***, Update Failed: Missing/Invalid
attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623 https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
<
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855 https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855>,
or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ABDIZMFPF6GDPGZG7QDGZALXCPIVBANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885 https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518649885>,
or
unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ACCTYL5KLHQICHOBOGYZ2Y3XCPJJLANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
<https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906 https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518653906>,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ABDIZMAZ35TDZASTQDUFGPLXCPJ6LANCNFSM6AAAAAAXHQQXKI

.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518655391 https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518655391>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL6TGL3QVRTGYC55FT3XCPK4DANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ACCTYL6TGL3QVRTGYC55FT3XCPK4DANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you commented.

—
Reply to this email directly, view it on GitHub
<https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518666499 https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518666499>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABDIZMA6NMQ22LRE7D2JD5LXCPPZHANCNFSM6AAAAAAXHQQXKI https://github.com/notifications/unsubscribe-auth/ABDIZMA6NMQ22LRE7D2JD5LXCPPZHANCNFSM6AAAAAAXHQQXKI>
.
You are receiving this because you were assigned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518667456, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL3P3UKRUDS24G6V5CLXCPQLDANCNFSM6AAAAAAXHQQXKI.
You are receiving this because you commented.

I should have posted back that I solved it, that was the exact problem UGH,
gets me every dang time.
Thank you!
Jon Tattersall  https://slack.com/app_redirect?channel=WKS4BS5B6[find me
on Slack]
Productivity Engineer
C :<PHONE_NUMBER>
https://smart.salesforce.com/sig/jtattersall//us_mb_kb/default/link.html
On Sat, Apr 22, 2023 at 8:57 AM Ross Scroggs @.***>
wrote:

Your command has smart/curly quotes: gam csv adminsecgroups.csv gam update
group “~email” show_in_group_directory false
Replace them with straight quotes: gam csv adminsecgroups.csv gam update
group "~email" show_in_group_directory false
Smart/curly quotes are not recognized.
Ross
Ross Scroggs
@.***

On Apr 21, 2023, at 7:23 PM, jontatt @.***> wrote:
The issue tracker is for reporting product deficiencies. "How do I?"
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager <
https://groups.google.com/group/google-apps-manager>. When in doubt,
start at the discussion forum and return here only when instructed to do so.
Please confirm the following:
I have upgraded to the latest GAM release from
https://github.com/GAM-team/GAM/releases <
https://github.com/GAM-team/GAM/releases> and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/gam/wiki https://github.com/jay0lee/gam/wiki
Full steps to reproduce the issue:
gam csv adminsecgroups.csv gam update group “~email”
show_in_group_directory false
Expected outcome (what are you trying to do?): hide groups in my csv
from our directory
Actual outcome (what errors or bad behavior do you see instead?): I am
receiving Group: @.***, Update Failed: Missing/Invalid attribute
value: id
—
Reply to this email directly, view it on GitHub <
https://github.com/GAM-team/GAM/issues/1623>, or unsubscribe <
https://github.com/notifications/unsubscribe-auth/ACCTYL2FGOIQAD7ASN6OAJDXCM6LFANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/GAM/issues/1623#issuecomment-1518645855, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A7LJN7NLFWDFABIEXFILVU3XCPIVBANCNFSM6AAAAAAXHQQXKI
.
You are receiving this because you authored the thread.Message ID:
@.***>

@taers232c did we add support for detecting smart quotes and throwing an error at the admin?