Recaptcha doesn't fully render to DOM

When using recaptcha inside JSX, Recaptcha is not fully rendered. Div is blank.
<div class="g-recaptcha" data-sitekey="my-own-tested-and-valid-key" data-theme="light" data-type="image" data-size="normal" data-tabindex="0" data-reactid=".<IP_ADDRESS>">

react-recaptcha is imported and rendered as <Recaptcha sitekey="my-own-tested-and-valid-key" />

Are you using the recaptcha script on your index-page?

Possibly same problem as #181

I am seeing this problem using React 15.4.1 on mac os in Chrome. Trying to solve for #181 does not work in my case

@hantc You can't embed script tag inside JSX. This just won't work. You should dynamically add the script tag to the HTML using Javascript. I've already opened an issue regarding this to be supported in the library out of the box!
Please refer to https://github.com/appleboy/react-recaptcha/issues/212 for the workaround.

I ran into the empty div issue as well. I used scriptjs in componentDidMount and the script was added to the head and the reCAPTCHA rendered as expected.
$script('https://www.google.com/recaptcha/api.js', 'recaptcha');

any example how to do so?

Any updates on this issue?

Any updates on this issue?

there is a nope from the community..


Tree View

Add Variables Editor and Tree View.

Codecov Report
Patch coverage: 50.00% and project coverage change: +2.98 :tada:

Comparison is base (f50bb73) 54.10% compared to head (0408647) 57.09%.

:exclamation: Current head 0408647 differs from pull request most recent head 578c441. Consider uploading reports for the commit 578c441 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   54.10%   57.09%   +2.98%
==========================================
  Files          14       18       +4
  Lines         146      282     +136
  Branches       19       40      +21
==========================================
+ Hits           79      161      +82
- Misses         67      117      +50
- Partials        0        4       +4

Impacted Files
Coverage Δ

src/styles.ts
100.00% <ø> (ø)

src/components/VariablePanel/useTable.tsx
30.76% <10.00%> (+1.60%)
:arrow_up:

src/components/FieldsEditor/FieldsEditor.tsx
16.66% <16.66%> (ø)

src/components/Table/Table.tsx
37.50% <22.22%> (ø)

src/components/FieldsEditor/styles.ts
66.66% <66.66%> (ø)

src/components/VariablePanel/utils.ts
77.90% <77.90%> (ø)

...rc/components/VariablePanel/useRuntimeVariables.ts
93.75% <93.75%> (ø)

src/components/FieldsEditor/index.ts
100.00% <100.00%> (ø)

src/components/VariablePanel/VariablePanel.tsx
100.00% <100.00%> (ø)

src/components/index.ts
100.00% <100.00%> (ø)

... and 1 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


--virtualbox-share-folder on windows

Hi,
I have installed docker-machine Version: 0.9.0-rc2, build 7b19591 on Windows 10 x64. I have VirtualBox 5.1.10 r112026. I tried to use the new --virtualbox-share-folder switch.
I try the following call:
docker-machine create -d virtualbox --virtualbox-no-share --virtualbox-share-folder c:\\workspace:workspace default

I expect it to mount the specified directory on /workspace. However, nothing is mounted and nothing is available in shared folders in VirtualBox GUI.
I have also tried the Windows directory path in different formats, e.g.:

c:\workspace
c:/workspace
/c/workspace

How to use the switch in Windows?

See if you can get it to work without --virtualbox-no-share.  That flag will skip the parts of Create used to set up the share.

It does not work either. Tried with all the c:/workspace path formats as above.
If the flag -virtualbox-no-share skip setting up the share then I would like to propose to fix the docs - https://github.com/docker/docker.github.io/blob/master/machine/drivers/virtualbox.md - because I did earlier exactly as in the docs: I used -virtualbox-no-share to "disable the mount of your home directory" (as I wanted to mount another directory).

It does not work either. Tried with all the c:/workspace path formats as above

Can you please gist.github.com the output of at least one such attempt?

https://gist.github.com/michzimny/5a42a1fa1a4139b7be6b67bb3a81fdc1

Can you show for:
$ docker-machine create -d virtualbox --virtualbox-no-share --virtualbox-share-folder "c:\\workspace:workspace" default

Note the quotes.

For the call given above:
https://gist.github.com/michzimny/ebef21474f3c061dd59358866d2ee743
And for the same but without --virtualbox-no-share:
https://gist.github.com/michzimny/2b7c6226b59d69e3cc4e60caa8f055ad

What shell are you using? If you try it with Powershell or cmd.exe do you get different result?
btw --virtualbox-no-share should not ever be needed for this case.

I use Cygwin. I tried now with Powershell, but is does not work either.

@nathanleclaire, another issue I noticed is that the main partition is in tmpfs. So, the /dev/sda1 is not properly mounted to /. Is it expected? Are the problems somehow related?
# df -h
Filesystem                Size      Used Available Use% Mounted on
tmpfs                   896.2M    179.0M    717.1M  20% /
tmpfs                   497.9M      8.0K    497.9M   0% /dev/shm
/dev/sda1                17.9G      1.1G     15.8G   7% /mnt/sda1
cgroup                  497.9M         0    497.9M   0% /sys/fs/cgroup
workspace               465.3G    117.1G    348.2G  25% /workspace
/dev/sda1                17.9G      1.1G     15.8G   7% /mnt/sda1/var/lib/docker/aufs
none                     17.9G      1.1G     15.8G   7% /mnt/sda1/var/lib/docker/aufs/mnt/3c4f13d8c296191d5c705c39c3fb85447538e522c5b3a7d731ce84324df22b10

@michzimny That seems normal to me.  The root FS in boot2docker is deliberately reset on reboot (mounted only as tmpfs like you notice) and only the relevant Docker directories are mounted as persistent partitions (based on /dev/sda device):
docker@default:~$ sudo ls -lah  /var/lib/docker
lrwxrwxrwx    1 root     root          24 Dec 14 21:46 /var/lib/docker -> /mnt/sda1/var/lib/docker
docker@default:~$ mount | grep docker
/dev/sda1 on /mnt/sda1/var/lib/docker/aufs type ext4 (rw,relatime,data=ordered)

As documented here https://docs.docker.com/machine/drivers/virtualbox/
The option --virtualbox-share-folder is delimited by ':'.
So a value like this 'should' work --virtualbox-share-folder /c:c
But I guess when ran from MINGW64, the '/c' is not expanded out correctly, and the code
here skips processing it as 'os.IsNotExist' returns true.
In the meantime, you can get it working using VBoxManage from the Docker Quickstart Terminal:
Here I'm sharing the root c:\ drive as '/c' from MINGW64  into the docker-machine VM using:
VBoxManage sharedfolder add default --name c --hostpath /c --automount
For reference,
The parsing code is here
split := strings.Split(d.ShareFolder, ":") shareDir, shareName = split[0], split[1]
Also curiously , the default value that is used for 'c:\Users' is "\\?\c:\Users"
See here

@rooneyp1976 Maybe that block of code you link should run https://golang.org/pkg/path/filepath/#FromSlash on it first.  To be honest it's tough to say what the right expected behavior should be since the various Windows shells treat this differently.

Perhaps the author of the new option could comment? @u-foka would you have any ideas how come when using VBoxManage on MINGW64, the passing of '/c' gets expanded out to 'c:/' in VirtualBox SharedFolders but doesn't happen with docker-machine/GoLang and --virtualbox-share-folder?
The below command ran on MINGW64 gives a Shared Folder Path of 'c:/'
VBoxManage sharedfolder add default --name c --hostpath /c --automount
Note: my MINGW64 came from git version 2.10.2.windows.1

Hello, I'm running Docker Toolbox for Windows (Windows 7) and running into similar mount volume issues.
docker-machine.exe version 0.10.0
Docker version 17.04.0-ce

docker-machine --debug create -d virtualbox --virtualbox-no-share --virtualbox-share-folder "c:\\users\\installer\\iad:iad" test2

I'm not able to see the "iad" folder mounted when I ssh into the test2 docker machine
Filesystem                Size      Used Available Use% Mounted on
tmpfs                   896.2M    183.6M    712.6M  20% /
tmpfs                   497.9M         0    497.9M   0% /dev/shm
/dev/sda1                17.9G     44.6M     16.9G   0% /mnt/sda1
cgroup                  497.9M         0    497.9M   0% /sys/fs/cgroup
/dev/sda1                17.9G     44.6M     16.9G   0% /mnt/sda1/var/lib/docker/aufs

I was able to get this working:
docker-machine --debug create -d virtualbox --virtualbox-share-folder "\\?\c:\Program Files\installer:installdir" test2

I have no problems mounting local directories after I followed tips 3 and 4 here, where I mapped my whole drive D to /d inside the default docker VM.
After that, I can issue docker commands from my git-bash shell on windows adding an extra leading "/" to the local paths I want to mount in order to avoid auto-conversion of paths:
docker run -it --rm -v /$(pwd):/source debian:jessie bash

or
docker run -it --rm -v //d/my/local/source:/source debian:jessie bash

try a setting as below.
docker-machine create --driver virtualbox --virtualbox-share-folder "c:\project:project" default

the result is...

This worked for me:
$ dm --version
docker-machine.exe version 0.15.0, build b48dc28d
$ docker-machine create -d virtualbox --virtualbox-share-folder "e:\\:/e" default
Running pre-create checks...
Creating machine...
(default) Copying E:\VMs\Docker\cache\boot2docker.iso to E:\VMs\Docker\machines\default\boot2docker.iso...
(default) Creating VirtualBox VM...
(default) Creating SSH key...
(default) Starting the VM...
(default) Check network to re-create if needed...
(default) Windows might ask for the permission to configure a dhcp server. Sometimes, such confirmation window is minimized in the taskbar.
(default) Waiting for an IP...
Waiting for machine to be running, this may take a few minutes...
Detecting operating system of created instance...
Waiting for SSH to be available...
Detecting the provisioner...
Provisioning with boot2docker...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...
Checking connection to Docker...
Docker is up and running!
To see how to connect your Docker Client to the Docker Engine running on this virtual machine, run: C:\Program Files\Docker\Docker\Resources\bin\docker-machine.exe env default
$ _


rmf_traffic_editor: 1.6.1-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository rmf_traffic_editor to 1.6.1-1:

upstream repository: https://github.com/open-rmf/rmf_traffic_editor.git
release repository: https://github.com/ros2-gbp/rmf_traffic_editor-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.6.0-1

rmf_building_map_tools
* Switch changelogs to rst format.
* Add texture for a white wall (#463 <https://github.com/open-rmf/rmf_traffic_editor/pull/463>)
* Fix navgraph generation for connected docking waypoints (#452 <https://github.com/open-rmf/rmf_traffic_editor/pull/452>)
* Added 5 retries for model downloading failure (#455 <https://github.com/open-rmf/rmf_traffic_editor/pull/455>)
* Migrate to using gzsim server url for fuel (#454 <https://github.com/open-rmf/rmf_traffic_editor/pull/454>)
* Exiting model downloader with non-zero exit code (#453 <https://github.com/open-rmf/rmf_traffic_editor/pull/453>)
* Contributors: Aaron Chong, Esteban Martinena Guerrero, Luca Della Vedova, Yadu, Yadunund

rmf_traffic_editor
* Switch changelogs to rst format.
* Contributors: Yadunund

rmf_traffic_editor_assets
* Switch changelogs to rst format.
* Migrate to using gzsim server url for fuel (#454 <https://github.com/open-rmf/rmf_traffic_editor/pull/454>)
* Contributors: Aaron Chong, Yadunund

rmf_traffic_editor_test_maps
* Switch changelogs to rst format.
* Fix test map generation with new commands (#456 <https://github.com/open-rmf/rmf_traffic_editor/pull/456>)
* Contributors: Aaron Chong, Yadunund

Manually updated doc and source versions to humble.


Index stat total documents count mismatch?

We have created an index in our meilisearch server and pushed bulk data from our database into the index. We are using the meilisearch version - 0.20.0 The current number of documents we have retried from the below code is 76,252.
var stats = await client.Index(index).GetStats();
Console.WriteLine(stats.NumberOfDocuments);
After that we pushed two batches of data each containing 96 & 4 documents each. So totally 100 documents pushed to the index. But when we checked the documents using the above code. Count has increased to 76,259. Remaining 93 counts are not retrieved from the stats of the index.
We have also checked the status of the update ID's of the above two batches(96 & 4) using the below code and it is marked as "Processed".
var updateStatus = await client.Index(index).GetUpdateStatus(updateId);
Console.WriteLine(updateStatus.Status);
Is there any delay in moving the processed updated Id documents into the "GetStats()" method "NumberOfDocuments" property as we are facing a discrepancy of 93 records.? Please provide any insight on this.

Hello @Vijay4657
Some users reported the stats route of the v0.20.0 was slow -> https://github.com/meilisearch/MeiliSearch/issues/1257
We did not succeed to reproduce, but we might have fixed it in the coming v0.21.0 with a new implementation of the stats route.
Also, if after a long time your route is still not up to date with the expected number of documents, it means you add documents that was already in your MeiliSearch server. MeiliSearch looks at the id of your document, if a document with the same id exists, it will replace it.
If you did not push the same document twice (again, I mean using the same id), please, share with us your dataset and the exacts steps you did so that we can reproduce the issue.

Closing this issue due to a lack of answer, feel free to re-open


Missing mini-signal typing

Hi the published version to npm is missing the mini-signal declaration:

<EMAIL_ADDRESS>

There hasn't been a release since they were added:
https://github.com/englercj/resource-loader/commits/master

Just published 2.2.4 which should have the goods.


fixing  WORKDIR in the Dockerfile of Auth-bypass-1

the WORKDIR is not correct

Ty for spotting this and the pull request


nixos/matrix-synapse: protect created files

Motivation for this change
Enforce UMask on the systemd unit to restrict the permissions of files
created. Especially the homeserver signing key should not be world
readable, and media is served through synapse itself, so no other user
needs access to these files.
Use a prestart chmod to fixup the permissions on the signing key.
Discovered by @andir
# ls -lah /var/lib/matrix-synapse/
total 4.0K
drwx------ 1 matrix-synapse matrix-synapse  54 May  8  2020 .
drwxr-xr-x 1 root           root           276 May 13 06:15 ..
-rw-r--r-- 1 matrix-synapse matrix-synapse  59 May  8  2020 homeserver.signing.key
drwxr-xr-x 1 matrix-synapse matrix-synapse 176 May 17  2020 media

Not critical, since the StateDirectory is 0700.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Deploying this to an existing matrix-synapse causes the following error:
May 22 17:16:31 matrix matrix-synapse-pre-start[5446]: chmod: changing permissions of '/var/lib/matrix-synapse/homeserver.signing.key': Operation not permitted

@Ma27 Who owns your signing key, if not the matrix-synapse user that runs the unit?

@mweinelt root:
[root@matrix:/var/lib/matrix-synapse]# ls -lah homeserver.signing.key
-rw-r--r-- 1 root root 59 Mar  3  2019 homeserver.signing.key

I guess this is because the matrix-synapse module had PermissionsStartOnly = true until Dec 2019 and the key is generated in ExecStartPre.

I've added a separate, privileged prestart command that should cover this legacy situation.


Add manual installation instructions

As witnessed in https://github.com/denoland/deno/issues/5727 and https://github.com/denoland/deno_install/issues/129, people may not expect Deno to just be an .exe in a .zip.
I think adding manual installation instructions might help.
To reduce installation complexity, it might also help to provide both a .zip and an .exe for download. (cc @ry - Is this something you'd consider?)
 Fixing this, I think, is something that would help make running Deno for the first time more beginner friendly.

@MarkTiedemann I don't know why would anyone be so lazy to download the .zip and extract it?

@divy-work How is downloading the .zip lazy? That's manual work, whereas if you use the installer, it does the work for you automatically. So I think it's the other way around; the installer script is the "lazy" way to install Deno.
I think a lot of people are not familiar with PowerShell... I think "download a zip, extract the zip, double-click the exe" maybe more familiar.

@MarkTiedemann Agreed. I'm of no authority. It's better to have something that helps but I still feel this is not needed.

I mean, I don't see any reason why we shouldn't have an exe installer. Just cos there's a command line method doesn't mean it's the best on any platform, heck, there's a reason that i'm going to the trouble of maintaining this: https://aur.archlinux.org/packages/deno-bin, because AUR packages are more typical in an arch / manjaro system than using shell script installers. Also see my comment here: https://github.com/denoland/deno/issues/5727#issuecomment-634394910

I still feel this is not needed.

I think the two linked issues above show that this may be helpful for beginners. Most Windows users have extracted a zip-file and double-clicked an exe. Running Powershell commands, I feel like, is not a very common thing.

AUR packages are more typical in an arch / manjaro system than using shell script installers

There are also multiple package managers for Windows, for example, Scoop, Chocolatey and NuGet, and we already support both Scoop and Choco.

I don't see any reason why we shouldn't have an exe installer.

Me neither. In fact, I have started working on a private C++ project I call "deno-up". Manual install instructions and a setup.exe are not mutually exclusive.


[JN-535] edit notification configs

This enables some properties of notification configs to be editable via UI.  Not the email templates, but the param values like max number of reminders.  (The backend supports creating a new email template, but updating the frontend was outside the scope of this ticket).  This also upgrades the notifcation config UX to match the recently restyled populate UX.

TO TEST:

go to http://localhost:3000/ourhealth/studies/ourheart/env/sandbox/notificationContent/configs
choose a config
update the Max reminders value
hit save
confirm the updated value is saved, and persists on refresh

oops, sorry, I must have fat-fingered github and added Nick as a reviewer by mistake on this.  didn't mean to do that, but thanks for the great review, Nick!


Transfer NFT Proposal on NFT Treasury Page not Generating Transaction Confirmation / Proposal

I am trying to create a proposal to transfer an NFT from the NFT treasury page. It will not load the transfer.
Here is the Error Log.
and what i see on the screen.
proposal form for NFT Transfer error log - brave browser - windows 10.log

ProposalForm.tsx:134 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'transfers')     at ProposalForm.tsx:134     at createFormControl.ts:1008     at l (runtime.js:63)     at Generator._invoke (runtime.js:294)     at Generator.next (runtime.js:119)     at n (asyncToGenerator.js:3)     at s (asyncToGenerator.js:25)

Ahh it seems I am on "voting on proposals" phase. -- Maybe a better error tooltip would help in this type of situation.

Yes, and agreed, added proper tooltips and disabled buttons if not on proposal period in #243


Tectonic Plates

We should be able generate tectonic plates for more detailed mountain generation.

Because we are using only one noise its adjusting to this level is not possible


Double icons sometimes with inherit icons, can't unset icons afterwards

Describe the bug
Setting an inherit icon on a folder with notes already configured with icons causes double icons to display, and persist after removing the icon and restarting obsidian
To Reproduce
Steps to reproduce the behavior:
To reproduce this I had to create parent folders, set an icon on various notes with in it, set an icon on the parent folder, and then set inherit icon on the parent folder with icon already set, which triggered the irreconcilable double display.
Some icons can't have icons unset after this, I haven't perfectly tracked it down but there's something a bit wonky going on!
Expected behavior
Able to set and unset icons/inherit icons affecting display of one icon only
Screenshots

repro:

Additional context
Add any other context about the problem here.

I've tried to reproduce this, and I cannot reproduce this issue with the following steps:

Set an icon in various notes within the folder
Set icon of folder
Inherit icon of folder

For me, everything works as expected. Is it possible that other things interfere with your data.json file?

I'm not at my computer right now, but indeed this could be an interference from another plugin, I had issues before with losing icon assignments. I will investigate wsoon8 and report back.

I've not managed to narrow it down to another plugin yet, and am avoiding it, but I do think it is probably caused by another plugin if not reproducible. I've had icons disappear in the past as well, so likely a side effect of another plugin affecting settings?
I will close this for now and re-open if becomes a problem again and figure out exactly what is causing this.
For the record, my installed plugins are:
['obsidian-advanced-uri', 'obsidian-attachment-name-formatting', 'auto-class', 'calendar', 'chatgpt-md', 'obsidian-checklist-plugin', 'cmdr', 'obsidian-contextual-typography', 'obsidian-custom-attachment-location', 'custom-sort', 'obsidian-custom-frames', 'dataview', 'obsidian-day-planner', 'dbfolder', 'draw-harada-method', 'folder-note-plugin', 'obsidian-hide-sidebars-when-narrow', 'hotkeysplus-obsidian', 'obsidian-hover-editor', 'obsidian-icon-folder', 'obsidian-excalidraw-plugin', 'cm-editor-syntax-highlight-obsidian', 'obsidian-minimal-settings', 'obsidian-meta-bind-plugin', 'open-in-new-tab', 'open-vscode', 'periodic-notes', 'quickadd', 'obsidian-quiet-outline', 'obsidian-smart-typography', 'obsidian-style-settings', 'obsidian-task-progress-bar', 'obsidian-tasks-plugin', 'templater-obsidian', 'obsidian-toggle-list', 'make-md', 'mysnippets-plugin', 'nldates-obsidian', 'no-dupe-leaves', 'file-explorer-note-count', 'obsidian-columns', 'ob-table-enhancer']


Clear error before re-sending request

Looking at the code it is evident that somebody has to clear the value in r.Error before calling Send().
This is important in the pageQuery() case, where we re-use the same request each time round the loop.  However, inside pageQuery() we don't have direct access to Error, so it seemed simplest and safer to clear on every Send().
It does not seem particularly clear from the docs, or from the aws sdk itself.  I believe when they do retries it gets cleared here
I really need to try forcing an error and see what happens, to prove out this theory.

I tested as follows:
Wrote a small command-line program which exercises the chunk API talking to DynamoDB.
Installed a netem discipline to slow things down:
  sudo tc qdisc add dev eth0 root handle 1: prio
  sudo tc qdisc add dev eth0 parent 1:3 handle 30: tbf rate 20kbit buffer 1600 limit  3000
  sudo tc qdisc add dev eth0 parent 30:1 handle 31: netem delay 1000ms 10ms distribution normal
  sudo tc filter add dev eth0 protocol ip parent 1:0 prio 3 u32 match ip dst <IP_ADDRESS>/16 flowid 1:3

Pattern for the above taken from netem wiki page; the CIDR for DynamoDB is not exactly correct but useful most of the time.
When I run without this change, I get the pattern of first failure being slow then subsequent failures being instantaneous; removing the tc delay while it is failing makes no difference.
When I make the change in this PR each failure takes time, and removing the tc delay makes it succeed.


Closes #354 | Implement dataloader for ETOS

Closes #354
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<etos/etos.py.
[x] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Removed unnecessary commits to this PR.

Hi @khelli07 and @akhdanfadh, I've resolved all of your previous requests. Please let me know if you have other concerns! Thanks :)

LGTM! Thank you!


First example (ArrayClass)

This line does not compile:
arr = (arr.Reverse()).ToArray();
I could write "Array.Reverse(arr);" instead but then the array is modified even outside the method.

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

ID: a887e08e-64e4-9169-42b3-1cdbf829316f
Version Independent ID: 0d58869d-c6a9-8a7b-d345-df56b0873ca6
Content: Passing Arrays as Arguments (C# Programming Guide)
Content Source: docs/csharp/programming-guide/arrays/passing-arrays-as-arguments.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out, @johnnyagerard.
Aside from the fact that the code doesn't compile because Array.Reverse is a static method that takes an array as an argument, it also confuses the difference between passing reference types and value types by value. Array.Reverse does indeed reverse the elements of the array when the method returns because an array is a reference type.
We'll update this topic later today so that it both compiles and is accurate.
Again, thanks for helping us to address the limitations of this topic!

This code actually works.
I just needed to add the following line of code:
using System.Linq;

@rpetrusha the PR that fixes this issue dramatically changes the example. Should we present both "reversing" methods and explain why only one of them "works"? (By the way, then the same example might be used to show how ref works.)
The code below shows both methods:
using System;
using System.Linq;

class Program
{
    static void Main(string[] args)
    {
        var arr = new int[] { 1, 2, 3 };
        Display(arr);

        Reverse(arr);
        Display(arr);  // outputs 1 2 3

        ReverseReally(arr);
        Display(arr);  // outputs 3 2 1

        Console.ReadLine();
    }

    static void Reverse(int[] arr)
    {
        arr = (arr.Reverse()).ToArray();
    }

    static void ReverseReally(int[] arr)
    {
        Array.Reverse(arr);
    }

    static void Display(int[] arr) => Console.WriteLine(string.Join(" ", arr));
}


Fix crash in acquiring credentials

When a creedntial store is provide but no name is provided we end up
crashing by dereferencing the name.
Allow the request to proceed without a name, as it may be selected out
of a credential file, but if none was found, return an error.
Fixes #40
Fixes #42

@frozencemetery the changes have been QAd by @kvv81 (see comment in #41), so they are redy for final review.


New PR to test SignCLA at 2018-03-26 10:06:22.905934129 -0700 PDT m=+6.172737169

Body of PR

indiv-pass-fork-staging: test succeeded


gui-wm/river: Removed = subslot operator

The = operator makes no sense for the live version of river, since it says subslot changes should trigger rebuild, and the subslot is explicit in the live version.
Additionnally, Zig version is specified now by subslot rather than >= dependency, which locks the Zig version to 0.10 (the only version you can build river with)

Thanks for the PR!


wip: fix scripted tests

Work in progress to fix scripted tests. Do you prefer to discuss here or in https://github.com/sbt/sbt-native-packager/pull/1647?
The biggest decision to make is what should target be? Keep backwards compatibility with ./target/universal? Use the new default target ( ./target/out/jvm/scala-3.3.3/universal )?  Or something else like ./target/out/universal?

Work in progress to fix scripted tests. Do you prefer to discuss here or in https://github.com/sbt/sbt-native-packager/pull/1647?

Either is fine. Thanks for this! I haven't done it yet, but I was thinking sbt should add some special scripted test command in the next 2.x version like exists-in-target or something, but hardcoding the target works for me too.

the commits setting scalaVersion can be revert when https://github.com/sbt/sbt/issues/7725 is fixed

The biggest decision to make is what should target be? Keep backwards compatibility with ./target/universal? Use the new default target ( ./target/out/jvm/scala-3.3.3/universal )? Or something else like ./target/out/universal?

After thinking for this a while, I think we should use the new default target ( ./target/out/jvm/scala-3.3.3/universal ). It makes sense that sbt native packager plugin uses whatever the target key is defined to. It also makes sense that if one runs +stage, the results are in different directories (./target/out/jvm/scala-3.3.3/universal , ./target/out/jvm/scala-2.13/universal, ./target/out/js/scala-3.3.3/universal, etc). The only downside is that existing scripts that might depend on ./target/universal might have to be rewritten (like what happened in the scripted tests)

I haven't done it yet, but I was thinking sbt should add some special scripted test command in the next 2.x version like exists-in-target or something, but hardcoding the target works for me too.

I think that is a good idea

@eed3si9n as you can see here, the only failing test is related to this issue. The ConcurrentModificationException
issue is still a problem, but it usually goes away after a retry of the job...

Yea. It turns out Retry(...) only does IOExceptions, and what's failing is concurrent something.

I'm going to merge this as-is.


Slow rendering is not reported correctly on tvOS

Description
We received around 150x higher values for Slow rendering in our app on tvOS (~60 %) than iOS (<1 %). This led us to an investigation why this could be happening.

Most iOS devices have 60 Hz screen, so this works almost correctly on iOS. Except for devices with ProMotion screens with refresh rates from 10-120 Hz. If a device is appropriately rendering in slower refresh rates it will be reported as a slow rendering, which is undesirable.
Apple TVs are connected to different TVs and the default setting for refresh rate can be different in some parts of the world. (In Central Europe it is usually 50 Hz). If the default TV setting or user setting is 50 Hz or less then slow rendering will be reported. This seems undesirable too.

I am not sure what would be the best solution to improve this metric. I think the least what can be done is to mention this in the documentation.
Reproducing the issue
Set your TV to slow refresh rate and check slow rendering values.
Firebase SDK Version
9.4.1
Xcode Version
13.4.1
Installation Method
CocoaPods
Firebase Product(s)
Performance
Targeted Platforms
iOS, tvOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

For tvOS, we can programmatically check the attached TV's refresh rate and update our definition of slow frames accordingly. For variable refresh rate displays like ProMotion this becomes much harder, Googlers see this doc for more details.

Thanks for raising this @mkj-is. The Perf SDK has the assumption about the device having a refresh rate of 60fps when calculating slow and frozen frames, and I will definitely be adding this to our documentation. I've also raised this with the eng team and we will see when we can prioritize looking into this and making the screen traces metrics more accurate for devices with different refresh rate. This might not be something we can easily change so we will have to investigate more, but it's definitely on our radar now!

Thanks for acknowledging this issue. I get why this might be hard to implement from your side, it would interfere with historic data when updating Firebase. But with more and more ProMotion devices the importance of this might be larger in the future.

Since this issue hasn't had any updates recently and I am not able to reproduce it, I am closing this issue. If this issue occurs again, please file a new issue with information about your current environment. Thank you

Oh I apologize. I was testing on my physical device but I had ProMotion turned off. I will look at this issue once again


How to test pro version?

Hello,
We are developing a sotware that must edit and publish styled excell files. Probably pro version can handle this. But I can't find a download link to test the library.
How can I test the library?
Thanks.

Send an email to<EMAIL_ADDRESS>

I am interest to perform a few tests...


Roll Flutter Engine from f79030440948 to 0d374a919a20 (7 revisions)

https://github.com/flutter/engine/compare/f79030440948...0d374a919a20
2023-01-16<EMAIL_ADDRESS>Roll Skia from 34a14247bd4b to c727016cc09e (1 revision) (flutter/engine#38903)
2023-01-16<EMAIL_ADDRESS>Roll Skia from 5a51583d09dd to 34a14247bd4b (1 revision) (flutter/engine#38901)
2023-01-15<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from nPbEqgIfE8TLOokQH... to SmEVvAFBC8oriPBou... (flutter/engine#38897)
2023-01-15<EMAIL_ADDRESS>Roll Skia from aa99fa0bdd31 to 5a51583d09dd (2 revisions) (flutter/engine#38892)
2023-01-15<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from w0hr1ZMvYGJnWInwK... to nPbEqgIfE8TLOokQH... (flutter/engine#38889)
2023-01-15<EMAIL_ADDRESS>Roll Skia from c64a10d525d1 to aa99fa0bdd31 (10 revisions) (flutter/engine#38886)
2023-01-15<EMAIL_ADDRESS>Roll Dart SDK from 645fd748e79e to 0097dcbada06 (3 revisions) (flutter/engine#38883)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from w0hr1ZMvYGJn to SmEVvAFBC8or
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Not an issue, rather question

Hi Chris,
new here - so not sure how i may contact you in other way.
So i'm a bit confused with this: "This is a port of the Katana WsFederation middleware for ASP.NET Core. This project has a hard dependency on the full .NET Framework..." sounds a bit contradictory to me.
Since .NET Core run-time is a light-weight rival of .NET Framework run-time, if the project has hard dependency on the full .NET Framework, how it can be "for" .NET Core?  :) Just can't figure it out, sorry if my question sounds silly :)

You are confusing ASP.NET Core and .NET Core as the same thing--they are
not. ASP.NET Core (a web framework) can run on top of .NET Core (a runtime,
which is cross platform) or the full .NET Framework (a runtime, which is
Windows only). This library was designed to work with the new ASP.NET Core
web framework, but depends on BCLs that only exist in the full .NET
Framework at this precise moment. Does that make more sense?
On Thu, May 4, 2017 at 6:24 AM alexandis<EMAIL_ADDRESS>wrote:

Hi Chris,
new here - so not sure how i may contact you in other way.
So i'm a bit confused with this: "This is a port of the Katana
WsFederation middleware for ASP.NET http://ASP.NET Core. This project
has a hard dependency on the full .NET Framework..." sounds a bit
contradictory to me.
Since .NET Core run-time is a light-weight rival of .NET Framework
run-time, if the project has hard dependency on the full .NET Framework,
how it can be "for" .NET Core?  :) Just can't figure it out, sorry if my question sounds silly :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chrisdrobison/aspnetcore-wsfed/issues/7, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABu24VYepd9s-choBVaJiifIP_o4ZdwEks5r2cN6gaJpZM4NQmLd
.

Thanks Chris, this makes a perfect sense. I've really confused ASP.NET Core and .NET Core ;-)


New example to demonstrate newly added runner functionality.

A new example to demonstrate the all the recent changes made to runner and these may be incorporated with the IO package to save the data.
This example will translate over well in to the cilib-tutorial.
I will work on this.

@KyleErwin How's this coming along?

Morning Gary. Sorry, progress has been slow. Been working on the multi
objective stuff for a projects. Sorry. Will try get some done today. If you
have any ideas please let me know.
On Wed, 27 Jun 2018, 05:52 Gary Pamparà<EMAIL_ADDRESS>wrote:

@KyleErwin https://github.com/KyleErwin How's this coming along?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cirg-up/cilib/issues/278#issuecomment-400613094, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AOMAQUqtbHit-JkXMLq73vgEHNHqGhtzks5uA1XwgaJpZM4TXIOP
.

--
Kyle Erwin
Assistant Researcher
Cell:<PHONE_NUMBER>
Computational Intelligence Research Group:
http://cirg.cs.up.ac.za

Ok, no issue. I was just curious. Maybe have a look at this example for inspiration.

Addressed in #294 and #295


Sorting button 69

#69 addressed, ready for integration of sorting functionality that Bikram previously created...
@bikramkawan

Is this icon only for ascending and descending? Will there be another icon to sort boxplot values?

How would you like it to look like and work? Ideas?

There are two different features that are related to sorting:

change sorting direction (ascending and descending) - could be indicated via a tiny arrow pointing up or down. The columns that is sorted, will also be the top one in the sorting hierarchy (which will also change the order of filters in the filter view on the right)
change sorting criterion (min, max, median, etc.) -> should only be added for aggregated columns (can be a separate issue)

This should be changed into radio button selection.For now, I have just merged it.


Azure Eventhub to DBx Table Mappings

Below is a Matrix for Eventhubs to Table mappings.

Event   Hubs
Delta Tables Per Eventhub
MMG-In-SQL Model Tables     Bronze

hl7-file-dropped
tbl_hl7_file_dropped

hl7-mmg-validation-err
tbl_hl7_mmg_validation_err
tbl_hl7_mmg_err_bronze

hl7-mmg-validation-ok
tbl_hl7_mmg_validation_ok
tbl_hl7_mmg_ok_brone

hl7-recdeb-err
tbl_hl7_recdeb_err

hl7-recdeb-ok
tbl_hl7_recdeb_ok

hl7-structure-err
tbl_hl7_structure_err
tbl_hl7_structure_err_bronze

hl7-structure-ok
tbl_hl7_structure_ok
tbl_hl7_structure_err_bronze

hl7-mmg-based-err
tbl_hl7_mmg_based_err
eh_hl7_mmg_based_err_bronze

hl7-mmg-based-ok
tbl_hl7_mmg_based_ok
eh_hl7_mmg_based_ok_bronze

hl7-mmg-sql-err
tbl_hl7_mmg_sql_err

hl7-mmg-sql-ok
tbl_hl7_mmg_sql_ok

hl7-lake-segments-err
tbl_hl7_lake_segments_err

hl7-lake-segments-ok
tbl_hl7_lake_segments_ok

hl7-quarantine
TBD

hl7-message-processor-err
tbl_hl7_message_processor_err
To be Deleted

hl7-message-processor-ok
tbl_hl7_message_processor_ok
To be Deleted

@raman-cdc Can you put an estimate on this one?

It. is complete for now and will keep updating it as there are changes in Event hubs or Raw streaming tables

Updated Eventhub to Delta Table Mapping


Service Edit Failed Konga 0.11

Editing an existing Service (it has a route on it too, not sure if that matters) seems to fail. Maybe not using PATCH or something?

That's strange,

it's working for me

Upgraded to latest master and still facing issues. let me see if I can export my routes/services data in non-prod and give you a sample to import THEN try to edit that in konga 👍  . Not sure what else to do. We are running same Kong version.

Just to update this issue to speed its due to different underlying Kong DB's.

+1

A temporary fix is up on the next branch. You can use that branch with no probs,  while waiting an update from Kong's team.

Closing this. For now, Konga works with the monkey patch.
Kong fixed it on this commit https://github.com/Kong/kong/pull/3559.


[http/s] config.ignoreUrls not used

http/s plugins are using config.ignoreIncomingPaths / config.ignoreOutgoingUrls, but generic plugin typings call for ignoreUrls. ignoreUrls is unused by http/s plugins.
I think Plugin.ts should be updated to contain special types for http, https, grpc, etc or remove all plugin specific configs and add a generic indexer type instead.

True, came for that as well, seems like ignoreUrls is enforced when defining rules per plugin basis, but it's completely ignored.
Is there a way to define ignore rules at least on a global level?

We should update the global plugin options to only keep options which apply to every plugin. Behavior-changing options should be moved to plugin-specific configs.

The only global option is enabled now


Code ability to define the constants with other types besides Float64 ??

I'm not sure this is useful but it should be discussed.

Given some recent experience with pytorch, openmpi, and multithreading on cpus, I think we should have a Float32 option available as well.

Parametric types could help with this, e.g.
struct Species{T}
  name::String
  charge::Int
  mass::T
  spin::T
  mu::T
end


On the events page the question mark beside heading is opening docs

Description
The question mark besides the heading Community Events is opening a documentation page. Are they linked by mistake or is this intentional. If by mistake then what should that icon link to?
Below are the screenshots-

And on clicking, the below page is getting opened.

Screenshots
https://user-images.githubusercontent.com/92542124/218960917-be5ebaeb-29e7-4d9f-974b-664fd6469d78.mov
Additional information
No response

This feature was requested by the community so they can understand how to get their events on this page. I will be closing this issue as is intentional.

Okay..sure!


error with group written in Cyrillic

Describe the bug
The group name contains characters in Cyrillic. An error occurs when going to the detailed data view(/statuses).
What do you see?
No response
What do you expect to see?
No response
List the steps that must be taken to reproduce this issue

set in the configuration file for the site being checked a group(Endpoint groups) whose name contains characters in Cyrillic.
launch Gatus
go to the detailed information page (statuses)

Version
5.6.0
Additional information
The error originally occurred when migrating to 5.2

I understand your point, but in the foreseeable future, I plan on supporting English.
Supporting non-English languages, therefore potentially problematic characters for URLs, is not a problem I currently think is not important enough in the grand scheme of things (i.e. feature-wise) to prioritize.
I apologize for the inconvenience.

@TwiN Good afternoon.
It may be possible to consider additional titles for
endpoints[].name and endpoints[].group (endpoints[].name_url and endpoints[].group_url)?
It turns out that the user will have the opportunity to set one name for the visual part, and another in English for the URL. in my opinion, this will solve my problem and add more flexibility to the overall solution. When endpoints[].name_url and endpoints[].group_url are filled in, the address for the URL is formed using them, if not specified, then according to the current logic.


Not sure about ACLHound workflow

From the guide, it appears that I can use ACLHound to actually deploy ACLs, but I've seen notes around that imply that the changes are getting committed to a git repo or similar for code review before getting pushed out to the actual devices.  That would be a very nice feature, allowing for a code review prior to configurations getting pushed out to devices.
Is there a doc, that I've missed, explaining such a workflow? The TODO has "Subcommands to be created - aclhound diff" etc, so maybe this is work-in-progress somewhere.
This may be quicker to discuss via skype or similar, if you are available and interested.  I'm doing a comparison of a few ACL automation tools, and have assessed this and Google's Capirca.
Thanks, jz

Hey,
There is some documentation on jenkins integration here: https://github.com/job/aclhound/blob/master/INSTALL-CONFIG.md
And basically, aclhound has a directory, if you just push that to a repository and use something like gerrit to do approvals (like we at eCG do) it shouldn't be a problem.
the extra commands in aclhound (like reset and diff) are something that i guess we're gonna abandon as they are just aliases for git commands

Hello @mgmoerman, thanks for the response.  I read that config, but still wasn't clear on usage.
The aclhound build command returns the following to stdout:
(venv) $ aclhound build ~/aclhound/devices/fw001
INFO: working with data in /Users/jzohrab/aclhound
INFO: building configuration for /Users/jzohrab/aclhound/devices/fw001

Seed policy name: x
   IPv4:
   access-list x-v4 extended permit tcp <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> eq [22]
   access-list x-v4 extended permit tcp <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> eq [80]
   access-list x-v4 extended permit tcp <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> eq [443]
   access-list x-v4 extended deny ip any any
     ---------
   IPv6:
   ipv6 access-list x-v6 deny ip any any

This is output to stdout, I couldn't see a built in command to save to a directory.  Do you recommend that I run this command and redirect to output, and then commit that?
The issue I thought of for redirecting and saving is that things may change.  The redirected data would be a representation of the actual changes I will run on the device, and not the actual changes themselves.  When aclhound actual deploys the changes, it will again read from the source files, and update the devices - but if the source files have changed, then it wouldn't match the data that I had saved earlier.
Can ACLHound this redirected output and use that for its config changes?  As far as I can see from the code and examples, it doesn't.

It's not intended for saving the configuration. You use 'aclhound build all' or 'aclhound build devices/blabla' for deploying ACLs to the device.

Got it, thanks @mgmoerman.  Ah, and when you were saying "aclhound has a directory", you were referring to the ~/aclhound directory which has the definitions/devices/policyfiles, and then that's used to push.  For some reason, I was thinking that you'd generate, files, commit, push, and review those before applying, which is wrong - don't know why I missed that.  I'll submit a small PR for a docs update.

Closing, PR issued.


Update sbt-pgp to 1.1.2-1

Updates com.jsuereth:sbt-pgp from 1.1.2 to 1.1.2-1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.jsuereth", artifactId = "sbt-pgp" } ]

labels: sbt-plugin-update, semver-pre-release

Codecov Report

Merging #1248 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1248   +/-   ##
=======================================
  Coverage   81.07%   81.07%
=======================================
  Files          55       55
  Lines        1009     1009
  Branches       42       42
=======================================
  Hits          818      818
  Misses        191      191

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be0e764...7f9ceb1. Read the comment docs.


feat!: Create Gamut-kit

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages<EMAIL_ADDRESS>
📬Published Alpha Packages<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60414b135e61cd08af667fda--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60414c7a951db20f8382dff7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60414e7224314d11673dc32f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6041561e1dd2e418ba08e73d--gamut-preview.netlify.appDeploy Logs


Breaking migration issue - ref no longer available on RemoteAdapter?

After upgrading hooks_riverpod to 1.0.1 I need to also update flutter_data.
When updating from 0.8.2 to 1.1.1 (tried later ones as well) I get the following issues after generating with build_runner:
In any mixin's on RemoteAdapter

Seems like the RemoteAdapter no longer has access to provider ref?

Good news. This seems to have been a ghost issue.
When I came back to my code today, ref was recognised as a ProviderReference.
No idea what changed but I'll blame VSCode.

After digging in previous commits I found the answer in ec41c901e9f5aa2c4dbbf32aa6526523c61308f5
ProviderReference ref is replaced by Reader read
Before:
... ref.read(configurationProvider).instance.apiUrl
After:
... read(configurationProvider).instance.apiUrl


⚠ Bofrost.at has degraded performance

In 6c642ca, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 879 ms

Resolved: Bofrost.at performance has improved in 0700e74 after 1 hour, 47 minutes.


[Release template] Check mpi.persons triggers

Coverage remained the same at 91.117% when pulling 27ac14376b5ca6b91f27fe6d379160bdac47ea2d on AlekzandrV-patch-2 into 8c60441c38e991e87fb017faa69553fe8047679d on develop.


NULLs are not being estimated when sampling

Offending function / line:
https://github.com/capitalone/DataProfiler/blob/1a47b8e92ee39992508eb63e94e73544ee152769/dataprofiler/profilers/profile_builder.py#L294-L302
The gist is that these should either be estimates OR if they are real numbers, it should be made clear to the end-user. Probably with an explanation of the command they can use to either estimate it, give raw counts and / or how to do the full sample.
Incidentally, this is also one of the slowest functions. Taking the most cumulative time (see **) below:
         4688319 function calls (4610366 primitive calls) in 2.778 seconds

   Ordered by: internal time
   List reduced from 1162 to 20 due to restriction <20>

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
    23976    0.403    0.000    0.559    0.000 numerical_column_stats.py:386(_get_percentile)
   755201    0.146    0.000    0.146    0.000 {method 'search' of 're.Pattern' objects}
       80    0.140    0.002    0.378    0.005 {pandas._libs.lib.map_infer_mask}
       **10    0.138    0.014    0.881    0.088 profile_builder.py:217(get_base_props_and_clean_null_params)**
   755160    0.092    0.000    0.237    0.000 object_array.py:120(<lambda>)
       20    0.088    0.004    0.216    0.011 utils.py:50(shuffle_in_chunks)
   107900    0.071    0.000    0.170    0.000 base_column_profilers.py:47(_combine_unique_sets)
      350    0.069    0.000    0.137    0.000 {pandas._libs.lib.map_infer}
    32993    0.068    0.000    0.068    0.000 {method 'reduce' of 'numpy.ufunc' objects}
      222    0.056    0.000    0.056    0.000 {pandas._libs.lib.infer_dtype}
       10    0.054    0.005    0.220    0.022 text_column_profile.py:95(_update_vocab)
123750/123726    0.054    0.000    0.054    0.000 numerical_column_stats.py:85(__getattribute__)
   107880    0.052    0.000    0.119    0.000 random.py:174(randrange)
      168    0.051    0.000    0.114    0.001 numerical_column_stats.py:239(_total_histogram_bin_variance)
   107880    0.047    0.000    0.047    0.000 numerical_column_stats.py:545(is_int)
12798/10902    0.047    0.000    0.056    0.000 {built-in method numpy.array}
   107930    0.046    0.000    0.068    0.000 random.py:224(_randbelow)
       92    0.037    0.000    0.037    0.000 {built-in method pandas._libs.missing.isnaobj}
227832/227831    0.036    0.000    0.049    0.000 {built-in method builtins.isinstance}
   107880    0.034    0.000    0.034    0.000 numerical_column_stats.py:526(is_float)

The main time is the regex:
https://github.com/capitalone/DataProfiler/blob/1a47b8e92ee39992508eb63e94e73544ee152769/dataprofiler/profilers/profile_builder.py#L266-L278
Possible solution: Merge all the null searches into one query (as opposed to multiple queries) then split once indexes are matched. I believe it's possible to get a 8x speedup here.

Thoughts:

Throw warning if switching to sampling mode
Determine if there should be estimations or raw calculations
Provide example (in readme) and / or in warning on how to change sample size


Video Player Not Woring

hello,
i was trying to use your tool to show up some videos when  detect the main logo. But, the video player gets totally black when try to run the index.html.

I´m using the pc webcam to test it.


Enable Blend Splits in DirectionalLight3D by default

People occasionally wonder why sharp transitions between splits can be seen in DirectionalLight3D.
It's generally expected that modern renderers hide those sharp transitions by default, so let's do it.

Lets wait a bit with this one, I have to do some rework of the light code. Also this is expensive and should be moved to a general quality setting.

Closing in favor of https://github.com/godotengine/godot/pull/48776, which I think is a better solution to the problem.


[Improvement][Task Plugin] Refactor ResourceInfo class

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

Fields res and resourceName in class ResourceInfo are quite confusing. We should use a clearer name and remove those comments. BTW, the position of the comment for res is wrong.

https://github.com/apache/dolphinscheduler/blob/cf3b4424d7125a5513bb744f4dd964ac879dd607/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/model/ResourceInfo.java#L20-L34
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

WIP, no stale


Create 1609-even-odd-tree.js

Solved even-odd-tree in JS.
File(s) Added: 1609-even-odd-tree.js
Language(s) Used: JavaScript
Submission URL: https://leetcode.com/problems/even-odd-tree/submissions/1337061242/

@aadil42 Solution looks good, look over my suggestions about spacing. I'll merge this solution once you follow the suggestions.

@Ykhan799, I added spaces after control flow statements. Please review.

@aadil42 Looks good, thanks 👍


Override for team-like prefixes and suffixes

I didn't leave a description for myself to know what I meant here, so I'll assume this is now implemented with the name_decoration override.


test: add test cases for astb-padding

Goal

LPAD: Left-pads a string with a specified string up to a certain length.
RPAD: Right-pads a string with a specified string up to a certain length.

Todo

test: add test cases for astb-padding

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

28 of 29 (96.55%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 98.586%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test-suite/src/ast_builder/function/text/padding.rs
28
29
96.55%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
50346

Relevant Lines:
51068

💛  - Coveralls


x/tools/gopls: add .if and .not snippets for bool types

Is your feature request related to a problem? Please describe. The
introduction of postfix snippets has brought very convenient functionality to
gopls, especially when working outside of Goland. I wanted to contribute more
ideas for postfix completions inspired from what's already available from their
editors. In particular, I wanted to propose having a if! and not! prompts,
which do this (respectively):
(complex bool expression).if|
if (complex bool expression) {
	|
}

(expr).not|
!(expr)

Describe the solution you'd like
Implementing these in completion/postfix_snippets.go.
Describe alternatives you've considered
As a vim developper personally, I use
snippetsEmu. Other
potential alternatives are available in VSCode and Goland.

CC @muirdm.

Thanks for the suggestion. Muir might have some thoughts on this.

Wanted to add, that we could totally over-engineer the not by handling stuff like ==, >= ^, etc specially but I personally think it's over kill. Most of the time (at least personally) that this is useful is just negating a boolean var or a complex boolean expression surrounded by parenthesis.
In my mind, for the more edge cases,:
if x == y.not

should expand to
if x == !y
```.

This helps keep the completion snippets more light weight, and if the user so _desires_, they can get the functionality by surrounding the expression with parentheses,  or alternatively manually changing it.

The if! snippet sounds reasonable, but you will need to do some work to make a == b.if! be handled as (a == b).if! (i.e. to apply to the entire statement instead of the selector expression). You also may run into issues with the keyword if as a selector.
I'm less convinced about the not! snippet. As described it would only apply to bool expressions, so for example it wouldn't work in cases like if a == "foo".not. If it did work in cases like that, then it would be hard to predict what it would do in more complex cases like if (a == b) == (c == d).not. Does Goland offer a "not" snippet?

I don't think this needs to go through the proposal process, so taking it out.  Please comment if you disagree.

You also may run into issues with the keyword if as a selector.

Sorry, I am not entirely sure I understand what you mean here.
Goland's implementation for these snippets is actually a bit complex. When there is an abiguous case, like "a == b.not!", it shows a context menu to let the user pick whether to only do the first part, or the entire thing:

From my understanding (correct me if I am wrong) is not possible with gopls atm, so I think it would be reasonable to ditch the .not snippet unless you have a suggestion on how we can circumvent this?

You also may run into issues with the keyword if as a selector.

Sorry, I am not entirely sure I understand what you mean here.

The Go parser has trouble with keywords showing up in unexpected/invalid places (i.e. the "if" in foo || bar.if). Gopls has a workaround that tries to make "foo.if" parse the "if" as a plain identifier rather than an if statement, but I'm not sure if it will work in the case of the new snippet.

From my understanding (correct me if I am wrong) is not possible with gopls

LSP snippets have choices which might allow for a similar experience. But simpler and better, probably, would be just to have gopls offer multiple completion candidates, one for each possibility.
Regardless, I would skip not and start with if first because it seems more useful and straightforward.

Sounds good. I will take a stab at it. Will report back if I come across any difficulty 😄

@hhhapz Are you still working on this issue?


Try to remove all sonar problems

https://sonarcloud.io/organizations/oxygenxml-github/projects

cd2d8a58d96b32dce71a00a992c60acb69b67656
18882241bd15bfeb5e721a724d9692e3289566bf
89be291280e001148088b598741b0574819ea6bc
9705f9f1d00bf6bb31614a0d06d9f65d4153bdaf

Code review la code review:
The field "com.oxygenxml.ditareferences.tree.references.incoming.IncomingReference.translator" should be static and private.
Cand avem if-uri in "com.oxygenxml.ditareferences.tree.references.incoming.IncomingReference.equals(Object)" trebuie sa punem si brackets, chiar daca nu este decat o linie de cod in if.
com.oxygenxml.ditareferences.tree.references.incoming.IncomingReferencesPanel.translator => can be static. In fact all places where the translator is instantiated, it can be used there as a static final field I think.
"com.oxygenxml.ditareferences.workspace.rellinks.RelLinksAccessor.getRelationshipTableTargetURLs(URL)" why do you use the "forTests" both on the If and the else clauses? Why not use only the code which is on the else clause?
Why is the "com.oxygenxml.ditareferences.i18n.DITAReferencesTranslator" declared as serializable?
Here "com.oxygenxml.ditareferences.tree.references.outgoing.OpenReferenceAction.actionPerformed(ActionEvent)" all the if clauses related to keyref, conkeyref, datakeyref
com.oxygenxml.ditareferences.workspace.StringUtilities.trimNodeText(FontMetrics, String, int) => the negation of "<=" is not ">=".

I don't understand this:
Here "com.oxygenxml.ditareferences.tree.references.outgoing.OpenReferenceAction.actionPerformed(ActionEvent)" all the if clauses related to keyref, conkeyref, datakeyref

@mmmmmcr I think I did not finish my idea, I wanted to say that most of the if clauses invoked similar code and probably they could be compressed further somehow, but it does not matter, you can leave the code as it is.


Fix undefined variable

Last refactor (using a shell loop instead of every role separately)
introduced a regression.
Connected  https://github.com/rcbops/u-suk-dev/issues/1535

Approved by @cloudnull , merging.


Shows import costs text one line lower than it should be in .vue files

It happens in my main VSCode with lots of extensions but I've just freshly installed an Insiders version and it occurs there also.
It works fine in .js files.
VSCode version: 1.77.3
VSCode Insiders version: 1.78.0
System:  Apple M1 Pro / macOS 13.1

Sorry, that's a mistake in calculating the line offset and I never tested. Will fix that in next release.

I can confirm that it's been fixed in the new version. Thanks!


README.md     :slug won't be changed according to the Rails course instructions

Please refer to https://thinkster.io/tutorials/rails-json-api/adding-articles-crud

@xemexpress sorry, I'm not quite following you here — is there an error in our README or something?


chore: running bash scripts on windows

Исправил команду postinstall, так как не работала на OS Windows.
Добавил пункт в Readme с описанием того, как запускать bash скрипты на OS windows.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.004%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5725

Relevant Lines:
6441

💛  - Coveralls


Forked VM failed with exit code 134

Hi,
I've run into VM crash when profilin JMH benchmark.
Iteration  11: #
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007fc98b469494, pid=18575, tid=18576
#
# JRE version: OpenJDK Runtime Environment (11.0.1+13) (build 11.0.1+13-Ubuntu-3ubuntu116.04ppa1)
# Java VM: OpenJDK 64-Bit Server VM (11.0.1+13-Ubuntu-3ubuntu116.04ppa1, mixed mode, sharing, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# C  [libasyncProfiler.so+0x1c494]  StackFrame::pop(bool)+0x84
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /home/sergei/dev/benchmarks/hs_err_pid18575.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
#
<forked VM failed with exit code 134>
<stdout last='20 lines'>
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007fc98b469494, pid=18575, tid=18576
#
# JRE version: OpenJDK Runtime Environment (11.0.1+13) (build 11.0.1+13-Ubuntu-3ubuntu116.04ppa1)
# Java VM: OpenJDK 64-Bit Server VM (11.0.1+13-Ubuntu-3ubuntu116.04ppa1, mixed mode, sharing, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# C  [libasyncProfiler.so+0x1c494]  StackFrame::pop(bool)+0x84
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /home/sergei/dev/benchmarks/hs_err_pid18575.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
#
</stdout>
<stderr last='20 lines'>
</stderr>

Benchmark had encountered error, and fail on error was requested
Exception in thread "main" org.openjdk.jmh.runner.RunnerException: Benchmark caught the exception
	at org.openjdk.jmh.runner.Runner.runBenchmarks(Runner.java:578)
	at org.openjdk.jmh.runner.Runner.internalRun(Runner.java:320)
	at org.openjdk.jmh.runner.Runner.run(Runner.java:209)
	at com.luxoft.logeek.benchmark.BenchmarkRunner.main(BenchmarkRunner.java:46)
Caused by: org.openjdk.jmh.runner.BenchmarkException: Benchmark error
	at org.openjdk.jmh.runner.Runner.doFork(Runner.java:771)
	at org.openjdk.jmh.runner.Runner.runSeparate(Runner.java:663)
	at org.openjdk.jmh.runner.Runner.runBenchmarks(Runner.java:561)
	... 3 more
	Suppressed: java.lang.IllegalStateException: Forked VM failed with exit code 134
		... 6 more

How to reproduce

Run this benchmark on openjdk-11 under Linux

@State(Scope.Thread)
@BenchmarkMode(Mode.AverageTime)
@OutputTimeUnit(TimeUnit.NANOSECONDS)
@Fork(jvmArgsAppend = {"-XX:+UseParallelGC", "-Xms2g", "-Xmx2g"})
public class ConcurrentReferenceHashMapInstantiation {

  @Benchmark
  public Object constructor() {
    return new org.springframework.util.ConcurrentReferenceHashMap<>();
  }
}

Try to profile it using ./profiler.sh -d 180 -i 10 -o flat=50 -f ~/dev/async-profiler/log.txt pid
VM craches.

Here's the hs_err_pid18575.log: https://yadi.sk/d/gv-C7ywMiMj2rQ

Thank you for the report!
It was an interesting bug. The profiler crashed when the sampled PC occured at the very first byte of the mapped classes.jsa archive - it really happened to be an executable instruction, but the attempt to read a byte at PC-1 caused segmentation fault.
I committed the fixed. Please check if it helps.

It's very cool to have this issue fixed in a day. Thanks, it works!


Update data_requests PUT logic and tests

Context

Like #503 , some parts of data_requests PUT logic are not accounted for:
Selective edit permissions do not exist for github_issue_number or github_issue_url
API Documentation is out of date
Integration tests do not currently test all parameters

Requirements

Address the above issues in a manner most pleasing

Tests

Some tests will be updated as per above

Docs

API documentation will be updated as per above

Open questions

No open questions at this time.

@josh-chamberlain This does raise a question: Since we already have logic which separately creates a github_issue_number and github_issue_url, do we also want that to be editable within the PUT request? I could foresee that as being useful for maintenance, but maybe we only want it to be creatable via the automated endpoint, and not accidentally modified.

@maxachis yes, it's probably good to be editable for easy maintenance—but it's tricky because issue number and issue url would need to be edited together, right? I'm actually not clear on why we need both of those. Maybe there's a fancy way to account for editing just one of them.

@maxachis yes, it's probably good to be editable for easy maintenance—but it's tricky because issue number and issue url would need to be edited together, right? I'm actually not clear on why we need both of those. Maybe there's a fancy way to account for editing just one of them.

That was a design decision on my part (which as we all know are unimpeachable and perfect). The reason I did so was because, otherwise, I needed to either extract the issue number or construct the issue url on every query related to them, which struck me as questionable.
That may have been the wrong decision, and we can always adjust it, if needed.

@maxachis makes sense to me. Extracting the issue number seems most workable—especially because we'll always link to the URL. I'm not even sure we'll need to show the number in-app. It's fine to leave it for now, probably!


Fix schema migration for non-member nodes

This will likely fix the schema agreement issues we've been
experiencing. Schema needs to be migrated regardless of the node type.
The previous logic was only pulling the schema if the other node was a
stargate (non-member) node. That was incorrect schema should be pulled
regardless of the node type.

No! java.lang.IllegalStateException: Failed to reach schema agreement after 10 seconds.


Remove BuildConfigurations

Artifacts from local engine builds will be found based on the --local-engine flag

@devoncarew

It looks like the only two remaining flags are --local-engine-src-path and --local-engine.
I'm confused by an if statement in build_aot.dart. Assuming that's the way it's supposed to be, lgtm.

lgtm


Implement ModArithType for mod_arith dialect

See #1084
This PR is in draft state, opened here for reviewing the ModArithType type itself.
It is worth noting is that the type explicitly does not allow modulus value to be too large to fill the entire underlying type as there would be signedness issues. As we are working in FHE situations where modulus width typically goes up to only 60 for 64bit native int, this requirement does not harm the functionality and edge cases for lowering to native int could be avoided.
Later pushes will replace all existing ModArithOps to use the ModArithType and corresponding ModArithToArith lowering. Currently mreduce and madd serves only as a demo for the type verification and would be renamed to reduce/add once finished migration.
Syntax now:
!Zp1 = !mod_arith.mod_arith<modulus = 65537 : i32>

module {
  func.func @dot_product() -> !Zp1 {
    %c = arith.constant 1 : i32
    %0 = mod_arith.mreduce %c : i32 -> !Zp1
    %1 = mod_arith.madd %0, %0 : !Zp1
    return %1 : !Zp1
  }
}

IMO this looks great. I think the use of the : i64 type specifier for the "storage type" is a nice way to avoid extra verbosity.

Encountered issues of integer range when trying to migrate mod_arith.barrett_reduce and subifge because barett reduction will give a result in [0, 2p), and [p, 2p) are not represented in mod_arith.mod_arith type.
HEaaN.mlir resolves such issue by using a modf

Both ModArith and Poly operations may optionally carry modulus factor (modf ). If an instruction is tagged with a modulus factor 𝑓𝑖, its output is allowed to fit in [0, 𝑓𝑖· 𝑝) where 𝑝 is the exact modulus. Assigning a larger 𝑓𝑖 generates a faster loop because SubIfGE in Barrett reduction can be removed. We describe our algorithm for tagging modulus factors of instructions at Sec. 6.1.

Should we annotate the modf in the op or in the type? I think the former is easier to implement and can be done on the fly by the analysis pass, and lowering can use such information to do a cheaper lowering. We can also verify how many modf are availble given the storage type and the modulus in the analysis pass. These are all for later PRs.
For this PR. I think migrating basic ops like add/sub/mul with basic lowering is enough. And as barett_reduce is not used for now (though there was a PR #712 doing such transform), I'll just use the ModArithType for barrett_reduce without ensuring its range.

Should we annotate the modf in the op or in the type? I think the former is easier to implement and can be done on the fly by the analysis pass, and lowering can use such information to do a cheaper lowering. We can also verify how many modf are availble given the storage type and the modulus in the analysis pass. These are all for later PRs.
For this PR. I think migrating basic ops like add/sub/mul with basic lowering is enough. And as barett_reduce is not used for now (though there was a PR #712 doing such transform), ~I'll just use the ModArithType for barrett_reduce without ensuring its range~, I'll leave it as is so that we can have a useable ModArithType earlier.

Your reasoning is sound for the decision here. From my perspective, the main trade-off here is the inefficiency loss by incurring an extra field on every mod_arith type vs the extra work of lowering that type. The former doesn't seem that bad alone, but the worry is always that 5 or 10 more such optimizations will lead to 5 or 10 more fields on the type, so having a systematic strategy to avoid that is good.
If we put it on the ops, I just want to quickly sketch out the extra work that would require. Say we want to lower modarith<5 : i3> and there's a factor of 3 on some ops. The pass ultimately needs to be able to type-convert types without knowing which op it's on. So the pass setup would need to first run an analysis over the IR (or, say, at the func level) to determine the appropriate storage type to lower to. This might be as simple as just a max over modulus factors found across all ops, in which case perhaps we could reuse int range analysis from upstream. In this example, the analysis would determine we lower to i5 (because fi*p = 3*5 = 15 fits in an i4 but is too close to the max), and then that width would be given to the TypeConverter's constructor, and it would use that instead of the : i3 annotation when converting the type (the analysis would default to a width of 3 if no modulus-factors are found among the relevant ops).


Add https://github.com/ml-tooling/ml-workspace

docker containers for various use case scenarios

can we also consider niflows . there will be additional work being done it as part of several neuroimaging labs, so i would expect that ecosystem to grow, and it's based on all of our other tooling.

Sure thing - just add a dedicated issue for it or could even add a PR against create-singularities script
ATM the whole collection here is singularity containers, at some point I should look back into storing both (datalad-container supports it)


Remove iOS platform check for running devtools

Currently, DevTools only work under ios (although this is undocumented!), because the JavaScriptEngine initialization process skips setupDevTools() on android.
DevTools work fine with Android, as tested on 0.26, 0.27, and 0.28 using Nuclide's inspector.
For reference, the relevant issue on react-devtools.

By analyzing the blame information on this pull request, we identified @davidaurelio and @lexs to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification.

I'm working on resolving some internal dependency issues (devtools requires WebSocket to be present) so this can land.

Thanks @andreicoman11!


Dracula for Typora

I created dracula theme to Typora.
my repository is https://github.com/oguhpereira/Dracula-Typora

Hey @oguhpereira 👋 Would you like to move the theme to the organization?

Yes @hacknug  :smile:

Just invited you. Let me know once you've moved the repo and I'll set the repo's permissions.

created https://github.com/dracula/dracula-typora

All set. Thank you! :tada:


Reef Types

https://www.npmjs.com/package/@types/reefjs
@cferdinandi
I don't mean to be rude, so It's up to you whenever you want to have a repository for it or not. I can help! Routes types will be available soon.
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/50443
Made sure to include you as owner as well.

I'm sorry, I don't really understand what this is

I'm sorry, I don't really understand what this is

I'm sorry, I don't really understand what this is

Typescript type definitions.
https://www.typescriptlang.org/docs/handbook/2/type-declarations.html

I'm sorry, I don't really understand what this is

Typescript type definitions.
https://www.typescriptlang.org/docs/handbook/2/type-declarations.html

Oh, yea, I have no interest in maintaining or supporting that. I’m not a Typescript fan. Sorry!

Oh, yea, I have no interest in maintaining or supporting that. I’m not a Typescript fan. Sorry!


Cannot run on Azure web app or apache

Notice: Trying to get property of non-object in D:\home\site\wwwroot\phpdocumentdb.php on line 186 Warning: Invalid argument supplied for foreach() in D:\home\site\wwwroot\phpdocumentdb.php on line 187 Notice: Trying to get property of non-object in D:\home\site\wwwroot\phpdocumentdb.php on line 194 Fatal error: Call to a member function selectCollection() on boolean in D:\home\site\wwwroot\index2.php on line 14

Check CURL SSL and update API to 2015-08-06

I have SSL enabled and latest API, still getting the error on a fresh Azure app or freshly installed LAMP with CURL and OPENSSL enabled.

Just disable curl ssl. Then you will success.

i also get the errors listed (in azure). what did you mean by disabling curl ssl?

change request function.
private function request($path, $options)
{
$curl = curl_init($this->host . $path);
curl_setopt($curl, CURLOPT_HEADER, $this->debug);
curl_setopt($curl, CURLOPT_SSLVERSION, false);
curl_setopt($curl, CURLOPT_SSL_VERIFYHOST, false);
curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
curl_setopt($curl, CURLOPT_VERBOSE, $this->debug);
($this->debug) ?
	print "[[Debug: \nReq: ".curl_getinfo($curl, CURLINFO_HEADER_OUT) : $t=1;

curl_setopt_array($curl, $options);

$result = curl_exec($curl);
$code = curl_getinfo($curl, CURLINFO_HTTP_CODE);
curl_close($curl);
($this->debug) ? print "\nRes: $result]]\n" : $t=1;
return $result;

}

I'm experiencing same issue using an Azure web app. Disabling cURL SSL did not fix the problem for me.

I gave up to use cURL.
I rewrite to use HTTP_Request2.


Avoid renaming udim indexes

Brief description
Texture batch publishing renamed texture udim indexes like they were image sequence indexes. This fixes that, leaving udim indexes untouched.

Thanks for the fix.


Run tests from command line

Move tests to src/test/java from src/test. Copypaste test dependencies from pom.xml to build.gradle
Can now run tests with mvn test, gradle test

I'l take a look at travis to see why it failed.
Btw do you know travis does nothing right now? Taken from https://travis-ci.org/RuedigerMoeller/fast-serialization/jobs/163650294
:test UP-TO-DATE
:check UP-TO-DATE
BUILD SUCCESSFUL

Total time: 6.771 secs

Runing all tests required ~5 minute. And travis runs no tests at all in 6 seconds.

There are two problems:

Some of the tests are very resource expensive and I belive travis kills test process.
TestObjectServer is red on travis, but green on my machine.

If I junit.Ignore this tests, the build will be green https://github.com/korniltsev/fast-serialization/pull/2

So the problem with StringOffHeapTest#hardcore and other was memory related and fixed by increasing heap for test.
The only one red test is TestObjectServer with "Connection refused" exception in one of three setups(jdks).

Thanks for your help !

thanks for helping. Hm .. what does the "All checks have failed" message above means (note i am not familar with travis, was provided by a contributor)

TestObjectServer is still red when run on travis. It is the only red test I failed to fix.

java.net.ConnectException: Connection refused
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:579)
at java.net.Socket.connect(Socket.java:528)
at java.net.Socket.(Socket.java:425)
at java.net.Socket.(Socket.java:208)
at org.nustaq.net.TCPObjectSocket.

You can click details and see the build log on travis.


Note signatures sometimes are not valid

Sometimes our notes are discarded as invalid by relays due to the signature not matching. I believe this is caused by encoding/escaping differences when serializing the content as json.

Should be fixed by 01d423c4589dcfc137cd2c762de050c2a833d8a2


Add HOPE dataset

I've integrated the HOPE dataset (https://bop.felk.cvut.cz/datasets/#HOPE) into the necessary configs and BOP scripts.
I have verified that the following scripts result in a correctly formatted BOP submission file that achieves the expected scores in the BOP evaluation server:
python -m cosypose.scripts.download --bop_dataset=hope
python -m cosypose.scripts.convert_models_to_urdf --models=hope
# manually copied trained model weights
python -m cosypose.scripts.run_bop_inference --config bop-pbr --dataset hope  # added --dataset flag
python -m cosypose.scripts.run_bop20_eval_multi --result_id=bop-pbr--383758 \
  --method=maskrcnn_detections/refiner/iteration=4 --convert_only

The final script (cosypose.scripts.run_bop20_eval_multi) does currently produce an error that I would like help to resolve:
Traceback (most recent call last):
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/big_white/code/cosypose_SR/cosypose/scripts/run_bop20_eval_multi.py", line 61, in <module>
    main()
  File "/home/big_white/code/cosypose_SR/cosypose/scripts/run_bop20_eval_multi.py", line 57, in main
    print(scores_path.read_text())
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1216, in read_text
    with self.open(mode='r', encoding=encoding, errors=errors) as f:
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1203, in open
    opener=self._opener)
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1058, in _opener
    return self._accessor.open(self, flags, mode)
FileNotFoundError: [Errno 2] No such file or directory: '/home/big_white/code/cosypose_SR/local_data/bop_eval_outputs/challenge2020-383758_hope-test/scores_bop19.json'

The following files need to be uploaded to the appropriate cloud drive for cosypose.scripts.download to work correctly:

Trained model weights (https://drive.google.com/file/d/1ohQP6jFUwlidWwko4pNz5L-ubRz6FRl5/view?usp=sharing), to allow:

python -m cosypose.scripts.download detector-bop-hope-pbr--15246
python -m cosypose.scripts.download coarse-bop-hope-pbr--225203
python -m cosypose.scripts.download refiner-bop-hope-pbr--955392

URDF meshes (https://drive.google.com/file/d/17A7AFh0Wc5YXQqkNnQxFCuBX4gHCZc1D/view?usp=sharing), to allow:

python -m cosypose.scripts.download --urdf_models=hope

PBR training images are not publicly available, so I have commented out the relevant settings in the HOPE config, e.g. in cosypose/bop_config.py:
    # train_pbr_ds_name=[hope.pbr]

I uploaded the network weights and urdfs in the common google drive with the other ones.

I just resolved the merge conflict that resulted from #14. I also confirmed that the following download scripts are working:
python -m cosypose.scripts.download --urdf_models=hope
python -m cosypose.scripts.download --model=detector-bop-hope-pbr--15246
python -m cosypose.scripts.download --model=coarse-bop-hope-pbr--225203
python -m cosypose.scripts.download --model=refiner-bop-hope-pbr--955392

The error mentioned in my opening comment is still unresolved. I will take another look, but @ylabbe let me know if you have any ideas for fixing it.

FileNotFoundError: [Errno 2] No such file or directory: '/home/big_white/code/cosypose_SR/local_data/bop_eval_outputs/challenge2020-383758_hope-test/scores_bop19.json'

As noted above, there is one unresolved error:

The final script (cosypose.scripts.run_bop20_eval_multi) does currently produce an error that I would like help to resolve:
Traceback (most recent call last):
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/big_white/code/cosypose_SR/cosypose/scripts/run_bop20_eval_multi.py", line 61, in <module>
    main()
  File "/home/big_white/code/cosypose_SR/cosypose/scripts/run_bop20_eval_multi.py", line 57, in main
    print(scores_path.read_text())
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1216, in read_text
    with self.open(mode='r', encoding=encoding, errors=errors) as f:
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1203, in open
    opener=self._opener)
  File "/home/big_white/miniconda3/envs/cosypose_SR/lib/python3.7/pathlib.py", line 1058, in _opener
    return self._accessor.open(self, flags, mode)
FileNotFoundError: [Errno 2] No such file or directory: '/home/big_white/code/cosypose_SR/local_data/bop_eval_outputs/challenge2020-383758_hope-test/scores_bop19.json'

Is it possible that the following loop needs to be skipped when using the --convert_only flag?
https://github.com/Simple-Robotics/cosypose/blob/f526e290baa615bea561b35da1e70dd64a3a147e/cosypose/scripts/run_bop20_eval_multi.py#L52-L57


What about a feature for "following" uninstalled packages?

Please confirm these before moving forward

[X] I have searched for my feature proposal and not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This proposal is a completely new feature. If you want to suggest an improvement or an enhancement, please use this template

Describe the new feature
Imagine a top level tab that is peered with "Discover Packages, Software Updates, Installed Packages" called "Follow Packages" or perhaps "Watch Packages".   When you search for package in "Discover Package"  there would be an addition to the right-click menu that says something like "Follow/Watch Package".  Selecting the tab would show a similar "toolbar" to "Discover Packages" - but have the disclaimer stated clearly that the packages listed are NOT installed but only being followed.  All other features would work the same as "Installed packages" except the "ignore packages" would become "unwatch/unfollow packages" and the right-click menu would have the unfollow addition.
Some users might not like this feature so perhaps an option settings to enable the tab/right-click additions. (perhaps an experimental feature to start?)
Describe how this new feature could help users.
There are times when you might want to wait to install a package until a specific milestone has been met.  Or perhaps you are watching packages that you are responsible for/use on other machines.  Or you just want to follow and not install.

Some kind of whishlist/cart for package, right?


[Suggestion] chisel and bits support

Are there any plans to add chisel and bits support for rockets and space stations?

Chisel and bits already seem to work (Advanced Rocketry 1.10.2-0.8.6 and chiselsandbits-12.6)

Oh thanks for letting me know that's awesome. Do you want me to close this issue?

Minecraft: 1.10.2
C&B: 12.12
Advanced Rocketry: 1.1.7

@Jummit preliminary support was added in build 95, it turned out to be really difficult compared to 1.7 due to changes in the rendering engine and it's a bit hacky.  The lighting on C&B may or may not be darker that it's supposed to some of the time and I've not yet been able to pinpoint why.

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


Confusing error while set the keyPath for the ProtocolProxy

Here is my code:

It is totally as same as the example in documentation, but i got this error

Incorrect argument label in call (have 'for:keyPath:', expected 'for:selector:')

Can any body help?

Hi @luziqin - did you manage to sort this issue out?


Improve support for orjson; Add explicit serializers for json, ujson, and orjson

Closes #994
Changes:

Changed behavior for serializer='json', depending on installed packages:

Priority: orjson -> ujson -> stdlib json

Added the following serializers to explicitly use a specific JSON library: json_serializer, ujson_serializer, and orjson_serializer

@JWCook

Can you confirm that me minimal reproducible example below is correct?
If the example is correct, do you know what goes wrong with orjson memory usage and how it can be fixed?

Minimal reproducible example
import requests_cache
import time
import gc

from requests_cache.backends import SQLiteCache

backend = SQLiteCache(serializer='json', use_memory=True, fast_save=True)
with requests_cache.CachedSession(backend=backend, expire_after=60, serializer="json") as session:
    for i in range(1, 5):
        r = session.get(f'http://<IP_ADDRESS>:8080/json', headers={"accept": "application/json"})
        assert r.status_code == 200, r.status_code

time.sleep(1)
backend.clear()
del backend
time.sleep(1)
gc.collect()
time.sleep(1)

orjson
orjson eats +10 MiB (~30 MiB instead of ~20 MiB) and never releases the memory until the program exit.

json and ujson
json and ujson have the same memory usage (no memory overuse or leaks).

Interesting. Thank you for testing this! There's nothing wrong with your example, and that definitely looks like a memory leak to me. I went looking for other reports and found your issue here: https://github.com/ijl/orjson/issues/483, which was closed as 'invalid' without explanation?
This is making me lean towards making the user explicitly select a JSON library instead of doing any automatic selection with serializer='json'.
Any thoughts on this @pickfire?

serializes to bytes rather than str, i.e., is not a drop-in replacement

orjson is not a drop-in replacement for std, I think would be better to default to std json and let users choose the json serializer of their choice.


SignalR additional xml docs

Reviewed our xml docs as both ramping back up after leave and docathon. Things look pretty good, added a few things to some public methods.

https://github.com/aspnet/AspNetCore/issues/7265 - which should now be skipped. Rerunning

/azp run AspNetCore-helix-test


Fixed withListener to make listenerSupplier lazy again

.withListener accepted a Supplier<String> that immediately got its value - defeating the purpose of using a supplier.
This change (and accompanying test) fixes it, and parses the entire listener lazily.

Resolves https://github.com/testcontainers/testcontainers-java/issues/9215

H @YanivKunda, can you share what are you trying to achieve? I'm just curious if your real life code will work with the fix submitted. Your fix is technically correct but just want to see if it will fix what you want to do (I have an idea).

Actually I just encountered this code and considered it a bug without a real-life use case -
so I could only write a test to check that.
Basically I think that getting a listener that can only be computed after the container starts (similarly to the docker mapped ports) is a valid use case.
Also, if it turns out there's no real-life use case for that, meaning most users are going to pass a constant supplier, then there's no real point in keeping the method accepting a Supplier<String> - you could just have a withListener(String) method instead (or in addition, for backwards-compatibility).

@eddumelendez Do you think this is ready for review, or does it require any further work or discussion?

Thanks @YanivKunda.
Your reasoning with evaluating the supplier makes sense, but the example does not:

Basically I think that getting a listener that can only be computed after the container starts (similarly to the docker mapped ports) is a valid use case.

The configured listeners are evaluated in the configure() method, which happens before the container is created and started.
I also don't understand, why we need to make changes to a deprecated method. This issue is not present in the other withListener() methods?

@kiview you're right about my example, and I'm not sure I would be able to come up with a better one.
But it probably doesn't matter since that is deprecated - which is probably the reason @eddumelendez deprecated it in https://github.com/testcontainers/testcontainers-java/pull/9247.
And obviously, unless it's a critical fix, there's no need to change a deprecated method.
It's just that it wasn't deprecated when I first created this PR :-)


No overload matches session.initMiddleware()

Hi,
First i think that there is an error on the import of the usage section of the README
"https://deno.land/x/sessions/mod.ts" sould be "https://deno.land/x/oak_sessions/mod.ts" ?
I can't make the exemple in the readme work, i got an error
TS2769 [ERROR]: No overload matches this call.
Type 'RouterContext<RouteParams, State>' is not assignable to type 'Context<State, Record<string, any>>'.

I use last version of both oak and oak_sessions

Looking into this a little, it appears something in Oak changed from version 9.0.0 to 9.0.1.
In the meantime, it should work to just explicitly use Oak 9.0.0 to make this work again.
import { Application, Router } from<EMAIL_ADDRESS>

The other option is to run the code with the --no-check CLI flag. This will skip TypeScript checking if you just want the app to run. A better long-term solution will be looked at.

Thanks for the help :) !

Fixed with https://github.com/jcs224/oak_sessions/commit/7645b590d39608d4172a1ed87839864f8c6baa39


Migrate scripts tests to integration tests

Migrate these https://github.com/radumarias/syncoxiders/tree/main/file-tree-merge/tests/scripts
https://youtu.be/MiLp0InGOkI?si=xrt3V57DuL1uG2Vr

@thaodt how are you with this?

temporarily postponed until I finish wasm ticket.

@thaodt what wasm ticket?

this one https://github.com/radumarias/syncoxiders/issues/25

great, let me know if you need any help ith any of these
On Tue, Dec 3, 2024 at 10:31 AM Amidamaru @.***> wrote:

this one #25 https://github.com/radumarias/syncoxiders/issues/25
—
Reply to this email directly, view it on GitHub
https://github.com/radumarias/syncoxiders/issues/6#issuecomment-2513857871,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5TC4FRY6BDIKMLFHVNTY32DVT7DAVCNFSM6AAAAABL7FA6BWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTHA2TOOBXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
And in the end, it's not the years in your life that count. It's the life
in your years.