Different IPs for docker registry should be tried if available

See the second problem from https://github.com/docker/docker/issues/10614#issuecomment-74160895.
Currently only the first one is used. If node that owns the first IP dies, then registry looks unavailable for docker server, even if there are more alive nodes.
See https://github.com/bobrik/buzzer for an example.

cc @stevvooe, @dmp42

According to previous discussion, trying other ips should happen only if the host actively refused the connection.

Are you sure?
~ λ cat /etc/hosts | fgrep whatever.org
<IP_ADDRESS>	whatever.org
<IP_ADDRESS>	whatever.org
<IP_ADDRESS>	whatever.org

~ λ telnet whatever.org 9999
Trying <IP_ADDRESS>...
telnet: connect to address <IP_ADDRESS>: Operation timed out
Trying <IP_ADDRESS>...
telnet: connect to address <IP_ADDRESS>: Connection refused
Trying <IP_ADDRESS>...
telnet: connect to address <IP_ADDRESS>: Connection refused
telnet: Unable to connect to remote host

~ λ curl -v http://whatever.org:9999/
* Hostname was NOT found in DNS cache
*   Trying <IP_ADDRESS>...
* connect to <IP_ADDRESS> port 9999 failed: Operation timed out
*   Trying <IP_ADDRESS>...
* connect to <IP_ADDRESS> port 9999 failed: Connection refused
*   Trying <IP_ADDRESS>...
* connect to <IP_ADDRESS> port 9999 failed: Connection refused
* Failed to connect to whatever.org port 9999: Connection refused
* Closing connection 0
curl: (7) Failed to connect to whatever.org port 9999: Connection refused

~ λ nc -v whatever.org 9999
nc: connectx to whatever.org port 9999 (tcp) failed: Operation timed out
nc: connectx to whatever.org port 9999 (tcp) failed: Connection refused
nc: connectx to whatever.org port 9999 (tcp) failed: Connection refused

~ λ wget http://whatever.org:9999/
--2015-02-16 10:01:58--  http://whatever.org:9999/
Resolving whatever.org... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>
Connecting to whatever.org|<IP_ADDRESS>|:9999... failed: Operation timed out.
Connecting to whatever.org|<IP_ADDRESS>|:9999... failed: Connection refused.
Connecting to whatever.org|<IP_ADDRESS>|:9999... failed: Connection refused.

Thanks, @bobrik.


Add regex tokenizer

This adds a regex tokenizer which tokenizes the text by using a regex pattern to split. This is my first attempt and works for my usecase, but it's not ideal from a code and a configuration perspective.
Closes https://github.com/quickwit-oss/tantivy/issues/1670

Wow, snap! I just authored #1759 to do much the same, albeit with capture group support.
Didn't find a better solution for the cloned Regex either though ;)

Wow, snap! I just authored #1759 to do much the same, albeit with capture group support. Didn't find a better solution for the cloned Regex either though ;)

Haha, great. If you have capture group support then you have probably the better version.

Codecov Report

Merging #1759 (d6b8cb1) into main (b78dc5e) will decrease coverage by 0.00%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##             main    #1759      +/-   ##
==========================================
- Coverage   94.13%   94.13%   -0.01%
==========================================
  Files         267      268       +1
  Lines       50900    51036     +136
==========================================
+ Hits        47915    48041     +126
- Misses       2985     2995      +10

Impacted Files
Coverage Δ

src/tokenizer/mod.rs
95.61% <ø> (ø)

src/tokenizer/regex_tokenizer.rs
92.85% <92.85%> (ø)

src/schema/document.rs
87.64% <0.00%> (-4.50%)
:arrow_down:

src/schema/schema.rs
98.91% <0.00%> (+0.13%)
:arrow_up:

src/query/query_parser/query_parser.rs
95.09% <0.00%> (+0.19%)
:arrow_up:

src/query/boolean_query/boolean_query.rs
93.33% <0.00%> (+0.66%)
:arrow_up:

src/query/fuzzy_query.rs
91.86% <0.00%> (+0.81%)
:arrow_up:

bitpacker/src/bitpacker.rs
100.00% <0.00%> (+1.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@adamreichold Thank you for your feedback and the review. I like your idea of first merging this simpler version and then going ahead and adding the group capture support afterwards. I can wrap this up pretty quickly. @Gearme would that be OK for you as well? We could make the follow-up based on your implementation. I would also like to try out a benchmark to address https://github.com/quickwit-oss/tantivy/pull/1759#pullrequestreview-1238542538, but also as a follow-up.

Ups, @PSeitz removing you from review was not intentionally.

Sorry for requesting multiple reviews. I did not realize I can only request one review at the time.

I just discovered there is a off-by one on the offsets of the second token. I will address that.

I just discovered there is a off-by-one on the offsets of the second token. I will address that.

I think this is fine, wrong alarm.


feat(PasswordStrength): add security against common passwords

add onChangeStrength to PasswordStrengthcomponent
add list of insecurePasswords from here

add onChangeStrength to PasswordStrengthcomponent
add list of insecurePasswords from here

As we have rules for the password, we can get a more extensive list and filter the passwords that match the rule. This will create a shorter list of non-passwords yet secure.


Please provide advice on which version to use....

As we have now both a deprecated/archived version of this sdk and a v2, I would like some recommendations or a roadmap, as to when I can expect things to work, and with what versions. This info would benefit a lot from being on the readme files here and on pypi.
Thanks

We recommend you use the V2 SDK. Many customers experienced significant trouble installing and using the V1, and we recommend you avoid it for this reason. Although V2 is not totally feature complete, we are working rapidly to bring things up to parity.
We do not have an explicit roadmap or timeline at this time, as we are also addressing customer feedback and issues as they come in.
Please also note that the V2 SDK is still in preview, and APIs are subject to change until the preview is completed.

As there is no associated work item, and no further response, this issue is being closed

@njbuch, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


Requests takes too long

I have an Omnicore node up and running. It's synced.
But every request after connecting to network takes too long:
$ time docker exec -ti nodes_omni_online_node_1 omnicore-cli omni_getinfo
{
  "omnicoreversion_int": 30000000,
  "omnicoreversion": "0.3.0",
  "mastercoreversion": "0.3.0",
  "bitcoincoreversion": "0.13.2",
  "block": 491555,
  "blocktime":<PHONE_NUMBER>,
  "blocktransactions": 23,
  "totaltrades": 1495,
  "totaltransactions": 520985,
  "alerts": [
  ]
}

real    0m40.896s
user    0m0.017s
sys     0m0.017s

$ time docker exec -ti nodes_omni_online_node_1 omnicore-cli omni_gettransaction 4b67962786fc863f7b53145ce6bcaa49ec3bb72b90d3cc7115b72ad5aadf4941
{
  "txid": "4b67962786fc863f7b53145ce6bcaa49ec3bb72b90d3cc7115b72ad5aadf4941",
  "fee": "0.00054015",
  "sendingaddress": "17fMTPgSAJTfcnZ2gCcBcxxwi7wzNdLtrF",
  "referenceaddress": "115E3baxJZsJHeTay1jvUh3nSTHBJhkskc",
  "ismine": false,
  "version": 0,
  "type_int": 0,
  "type": "Simple Send",
  "propertyid": 31,
  "divisible": true,
  "amount": "14712.00000000",
  "valid": true,
  "blockhash": "000000000000000000978ffc57d4b2b902a0f3ec69804833d59f51008c8021a3",
  "blocktime":<PHONE_NUMBER>,
  "positioninblock": 1686,
  "block": 490613,
  "confirmations": 1199
}

real    2m3.093s
user    0m0.031s
sys     0m0.014s

What can be the reason? How it's supposed to work with that slow requests? What information can I provide to resolve the issue?

Hi! I have the same problem and suspect issue with threads synchronization and subsequent mutex deadlock because wallet will finally freeze after several RPC calls.

I am having the same issue , every request takes minutes, even when I am trying to do getinfo or omni_getinfo

Hey guys,
when the client isn't fully synchronized (current chain length: 532281), things are very slow, because the processing of new blocks is blocking.
It looks to me as this is the case here.

why getinfo so quick, bug omni_getinfo  need 14s ?
root@usdtwallet:~# time usdt-cli getinfo
{
"version": 130200,
"protocolversion": 70015,
"walletversion": 130000,
"balance": 0.00000000,
"blocks": 569136,
"timeoffset": 0,
"connections": 8,
"proxy": "",
"difficulty":<PHONE_NUMBER>411.053,
"testnet": false,
"keypoololdest":<PHONE_NUMBER>,
"keypoolsize": 100,
"paytxfee": 0.00000000,
"relayfee": 0.00001000,
"errors": ""
}
real	0m0.014s
user	0m0.000s
sys	0m0.000s
root@usdtwallet:~#
root@usdtwallet:~#
root@usdtwallet:~# time usdt-cli omni_getinfo
{
"omnicoreversion_int": 40000000,
"omnicoreversion": "0.4.0",
"mastercoreversion": "0.4.0",
"bitcoincoreversion": "0.13.2",
"block": 569136,
"blocktime":<PHONE_NUMBER>,
"blocktransactions": 409,
"totaltrades": 1815,
"totaltransactions": 6088456,
"alerts": [
]
}
real	0m14.373s
user	0m0.000s
sys	0m0.000s
root@usdtwallet:~# ll which usdt-cli
lrwxrwxrwx 1 root root 37 Mar 22 17:50 /usr/bin/usdt-cli -> /root/omnicore-0.4.0/bin/omnicore-cli*

omni_getinfo takes 20s to respond. I ran omnicored with txindex. It is fully synced thought.

any idea?


pdfsam-basic: 5.0.3 -> 5.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/pdfsam-basic/versions.
meta.description for pdfsam-basic is: Multi-platform software designed to extract pages, split, merge, mix and rotate PDF files
meta.homepage for pdfsam-basic is: https://github.com/torakiki/pdfsam
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.1.1 with grep in /nix/store/v1z0p8qq44660amzxxlpdxhppbm4k6hd-pdfsam-basic-5.1.1
found 5.1.1 in filename of file in /nix/store/v1z0p8qq44660amzxxlpdxhppbm4k6hd-pdfsam-basic-5.1.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pdfsam-basic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v1z0p8qq44660amzxxlpdxhppbm4k6hd-pdfsam-basic-5.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pdfsam-basic https://github.com/r-ryantm/nixpkgs/archive/2a05c984e05845bd02a08e81c24f9ab3e5142579.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v1z0p8qq44660amzxxlpdxhppbm4k6hd-pdfsam-basic-5.1.1
ls -la /nix/store/v1z0p8qq44660amzxxlpdxhppbm4k6hd-pdfsam-basic-5.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pdfsam-basic

Maintainer pings
cc @1000101 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/torakiki/pdfsam/releases.
meta.description for pdfsam-basic is: Multi-platform software designed to extract pages, split, merge, mix and rotate PDF files
meta.homepage for pdfsam-basic is: https://github.com/torakiki/pdfsam
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.1.2 with grep in /nix/store/zqq21i646a84g8zgdqdnfwj9yxn9x6jl-pdfsam-basic-5.1.2
found 5.1.2 in filename of file in /nix/store/zqq21i646a84g8zgdqdnfwj9yxn9x6jl-pdfsam-basic-5.1.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pdfsam-basic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zqq21i646a84g8zgdqdnfwj9yxn9x6jl-pdfsam-basic-5.1.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pdfsam-basic https://github.com/r-ryantm/nixpkgs/archive/a4fa099484ac6ec5bf34194822118f0b9fb3f753.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zqq21i646a84g8zgdqdnfwj9yxn9x6jl-pdfsam-basic-5.1.2
ls -la /nix/store/zqq21i646a84g8zgdqdnfwj9yxn9x6jl-pdfsam-basic-5.1.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pdfsam-basic

Maintainer pings
cc @1000101 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/torakiki/pdfsam/releases.
meta.description for pdfsam-basic is: Multi-platform software designed to extract pages, split, merge, mix and rotate PDF files
meta.homepage for pdfsam-basic is: https://github.com/torakiki/pdfsam
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.1.3 with grep in /nix/store/x7zz7a5i1lfjari89rmsxrdwwydhakf9-pdfsam-basic-5.1.3
found 5.1.3 in filename of file in /nix/store/x7zz7a5i1lfjari89rmsxrdwwydhakf9-pdfsam-basic-5.1.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pdfsam-basic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x7zz7a5i1lfjari89rmsxrdwwydhakf9-pdfsam-basic-5.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pdfsam-basic https://github.com/r-ryantm/nixpkgs/archive/e68694a425f9e9a6d9419838a278a4d95c9c8c24.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x7zz7a5i1lfjari89rmsxrdwwydhakf9-pdfsam-basic-5.1.3
ls -la /nix/store/x7zz7a5i1lfjari89rmsxrdwwydhakf9-pdfsam-basic-5.1.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pdfsam-basic

Maintainer pings
cc @1000101 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/pdfsam-basic/versions.
meta.description for pdfsam-basic is: Multi-platform software designed to extract pages, split, merge, mix and rotate PDF files
meta.homepage for pdfsam-basic is: https://github.com/torakiki/pdfsam
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.2.0 in filename of file in /nix/store/dc2ig5cj6y58bdbylbnmy660fifilnj3-pdfsam-basic-5.2.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pdfsam-basic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dc2ig5cj6y58bdbylbnmy660fifilnj3-pdfsam-basic-5.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pdfsam-basic https://github.com/r-ryantm/nixpkgs/archive/93e1d62920966c9e5b7b6dd95a7df3f8295185e8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/93e1d62920966c9e5b7b6dd95a7df3f8295185e8#pdfsam-basic

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dc2ig5cj6y58bdbylbnmy660fifilnj3-pdfsam-basic-5.2.0
ls -la /nix/store/dc2ig5cj6y58bdbylbnmy660fifilnj3-pdfsam-basic-5.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pdfsam-basic

Maintainer pings
cc @1000101 for testing.

Result of nixpkgs-review pr 222992 run on x86_64-linux 1

1 package built:

pdfsam-basic


Credential Passthrough Limitation on GPU clusters

Hello Team,
We see limitations on credential passthrough clusters to have only mlFlow. But there are customers who are trying to use GPU enabled High Concurrency clusters for Deep learning, and they are not able to use Credential passthrough on them.
We have got confirmation from Databricks engineering team that DBRs for GPU enabled High concurrency clusters, do not support Credential passthrough.
Ask: Can we please update the document to highlight this limitation (GPU Enabled High concurrency cluster do not support credential passthrough).

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

ID: 8d55b056-a9c1-5e92-4fdd-23d7269129f1
Version Independent ID: 82a77acc-2107-33f5-bfa2-28e6b11128b9
Content: Access Azure Data Lake Storage using Azure Active Directory credential passthrough - Azure Databricks
Content Source: databricks/security/credential-passthrough/adls-passthrough.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@poornimaputtaswamy  Thanks for the feedback! I have assigned the issue to the content author to evaluate and add the above details in the document.

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7104) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


Synthesis failed for recommender

Hello! Autosynth couldn't regenerate recommender. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to a new branch 'autosynth-recommender'
Cloning into '/tmpfs/tmp/tmp6tqxcbz_/googleapis'...
Note: checking out '5760f405f86d69277f78fe7fb5989016ace9c36f'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 5760f405 docs: update value documentation (#2977)
Note: checking out 'a3a0bf0f6291d69f2ff3df7fcd63d28ee20ac727'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at a3a0bf0f BREAKING_CHANGE: Removing TimeSeriesQueryLanguageCondition as an alert condition type. The condition type is unsupported and unused. It was originally added for the Monitoring Query Language Alpha feature. refactor!: Drop support for TimeSeriesQueryLanguageCondition as an alert condition type.
Switched to a new branch 'autosynth-recommender-377'
2020-05-06 04:06:42 [INFO] Running synthtool
2020-05-06 04:06:42 [INFO] ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-05-06 04:06:42,453 synthtool > Executing /tmpfs/src/github/synthtool/working_repo/Recommender/synth.py.
On branch autosynth-recommender-377
nothing to commit, working tree clean
2020-05-06 04:06:42,670 synthtool > Ensuring dependencies.
2020-05-06 04:06:43,338 synthtool > Cloning googleapis.
2020-05-06 04:06:43,338 synthtool > Using precloned repo /tmpfs/tmp/tmp6tqxcbz_/googleapis
2020-05-06 04:06:43,342 synthtool > Generating code for: //google/cloud/recommender/v1:google-cloud-recommender-v1-php.
2020-05-06 04:22:00,368 synthtool > Failed executing bazel build //google/cloud/recommender/v1:google-cloud-recommender-v1-php:

Starting local Bazel server and connecting to it...
Loading:
Loading: 0 packages loaded
Loading: 0 packages loaded
INFO: SHA256 (https://github.com/protocolbuffers/protobuf/archive/fe1790ca0df67173702f70d5646b82f48f412b99.zip) = 419f654d36998a51de2e0b79a67885ec48f8d803615ce66ec82cd15f54936730
Loading: 0 packages loaded
DEBUG: Rule 'com_google_protobuf' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "419f654d36998a51de2e0b79a67885ec48f8d803615ce66ec82cd15f54936730"
DEBUG: Call stack for the definition of repository 'com_google_protobuf' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:42:1
Loading: 0 packages loaded
INFO: SHA256 (https://github.com/grpc/grpc/archive/8347f4753568b5b66e49111c60ae2841278d3f33.zip) = 8d87aa54941db316e8a3370fdee1173ac4b2ede5724877d36a57edbc6dd136f0
Loading: 0 packages loaded
Loading: 0 packages loaded
DEBUG: Rule 'com_github_grpc_grpc' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "8d87aa54941db316e8a3370fdee1173ac4b2ede5724877d36a57edbc6dd136f0"
DEBUG: Call stack for the definition of repository 'com_github_grpc_grpc' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:103:1
INFO: SHA256 (https://github.com/googleapis/gax-java/archive/730046855da122b26adbaa3fcf535becd8afa292.zip) = 97c2a418cf3acf3eba0bb25ac01a617b03023711ff031c081a78f98b55beee55
DEBUG: Rule 'com_google_api_gax_java' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "97c2a418cf3acf3eba0bb25ac01a617b03023711ff031c081a78f98b55beee55"
DEBUG: Call stack for the definition of repository 'com_google_api_gax_java' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:131:1
INFO: SHA256 (https://github.com/grpc/grpc-java/archive/v1.27.2.zip) = 92ffb4391f847e02e115933a761e243dd1423f3fcafdc9b7ae0327eca102d76b
Loading: 0 packages loaded
DEBUG: Rule 'io_grpc_grpc_java' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "92ffb4391f847e02e115933a761e243dd1423f3fcafdc9b7ae0327eca102d76b"
DEBUG: Call stack for the definition of repository 'io_grpc_grpc_java' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/com_google_api_gax_java/repositories.bzl:114:5
 - /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/com_google_api_gax_java/repositories.bzl:60:5
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:146:1
INFO: SHA256 (https://github.com/googleapis/gapic-generator/archive/a59457d24bd6f3c962c8e699c708121c928eedc8.zip) = 843bcf262d44f6a2c1c131844ff8ef40daf95dd3b7c95dd761656f9e010d06d7
Loading: 0 packages loaded
Loading: 0 packages loaded
DEBUG: Rule 'com_google_api_codegen' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "843bcf262d44f6a2c1c131844ff8ef40daf95dd3b7c95dd761656f9e010d06d7"
DEBUG: Call stack for the definition of repository 'com_google_api_codegen' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:70:1
INFO: SHA256 (https://github.com/googleapis/protoc-java-resource-names-plugin/archive/64dafb71ea9a385a8da89989f0b9dab925bf4610.zip) = f475903e83904dbbe3651abae8cbf57ed804960a0dadd9cbde41b746db645052
DEBUG: Rule 'com_google_protoc_java_resource_names_plugin' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "f475903e83904dbbe3651abae8cbf57ed804960a0dadd9cbde41b746db645052"
DEBUG: Call stack for the definition of repository 'com_google_protoc_java_resource_names_plugin' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:163:1
INFO: SHA256 (https://github.com/googleapis/protoc-docs-plugin/archive/b2502d56b5ec2d47e063976da773206af295362d.zip) = 765ec120bb165ae98c3bae78705d2a127e64016e59738552e909fc8b11d06338
DEBUG: Rule 'protoc_docs_plugin' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "765ec120bb165ae98c3bae78705d2a127e64016e59738552e909fc8b11d06338"
DEBUG: Call stack for the definition of repository 'protoc_docs_plugin' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:187:1
INFO: SHA256 (https://github.com/bazelbuild/rules_python/archive/748aa53d7701e71101dfd15d800e100f6ff8e5d1.zip) = d3e40ca3b7e00b72d2b1585e0b3396bcce50f0fc692e2b7c91d8b0dc471e3eaf
DEBUG: Rule 'rules_python' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "d3e40ca3b7e00b72d2b1585e0b3396bcce50f0fc692e2b7c91d8b0dc471e3eaf"
DEBUG: Call stack for the definition of repository 'rules_python' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:36:1
INFO: SHA256 (https://github.com/googleapis/gapic-generator-python/archive/d18ed416240a064fffac0fd7915b61f6415fe140.zip) = 52301db31e5a1757017123f1ccc46a4d80db81310eba95d0c19553b047f365e4
DEBUG: Rule 'gapic_generator_python' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "52301db31e5a1757017123f1ccc46a4d80db81310eba95d0c19553b047f365e4"
DEBUG: Call stack for the definition of repository 'gapic_generator_python' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/bazel_tools/tools/build_defs/repo/http.bzl:292:16):
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:210:1
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
INFO: Call stack for the definition of repository 'gapic_generator_python_pip_deps' which is a pip_import (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/rules_python/python/pip.bzl:51:14):
 - /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/gapic_generator_python/repositories.bzl:77:5
 - /home/kbuilder/.cache/bazel/_bazel_kbuilder/8f273e333eb6aecd46d26c131d55729a/external/gapic_generator_python/repositories.bzl:12:5
 - /tmpfs/tmp/tmp6tqxcbz_/googleapis/WORKSPACE:221:1
ERROR: An error occurred during the fetch of repository 'gapic_generator_python_pip_deps':
   pip_import failed:  (Timed out)
ERROR: no such package '@gapic_generator_python_pip_deps//': pip_import failed:  (Timed out)
ERROR: no such package '@gapic_generator_python_pip_deps//': pip_import failed:  (Timed out)
INFO: Elapsed time: 909.430s
INFO: 0 processes.

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmpfs/src/github/synthtool/working_repo/Recommender/synth.py", line 31, in <module>
    bazel_target='//google/cloud/recommender/v1:google-cloud-recommender-v1-php',
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 60, in php_library
    return self._generate_code(service, version, "php", **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 177, in _generate_code
    shell.run(bazel_run_args)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bazel', 'build', '//google/cloud/recommender/v1:google-cloud-recommender-v1-php']' returned non-zero exit status 1.
2020-05-06 04:22:02 [ERROR] Synthesis failed
HEAD is now at 5760f405 docs: update value documentation (#2977)
Switched to branch 'autosynth-recommender'
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 584, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 465, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 574, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 360, in synthesize_loop
    synthesize_inner_loop(toolbox, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 370, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 115, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth is still having trouble generating recommender. :sob:
Here's the output from running synth.py:
2020-05-08 16:11:40 [INFO] logs will be written to: /tmpfs/src/github/synthtool/logs/googleapis/google-cloud-php
2020-05-08 16:11:40,039 autosynth > logs will be written to: /tmpfs/src/github/synthtool/logs/googleapis/google-cloud-php
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 596, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 470, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 515, in _inner_main
    working_repo_path = synthtool_git.clone(f"https://github.com/{args.repository}.git")
  File "/tmpfs/src/github/synthtool/synthtool/sources/git.py", line 65, in clone
    preclone = get_preclone(url)
  File "/tmpfs/src/github/synthtool/synthtool/sources/git.py", line 168, in get_preclone
    preconfig = synthtool.preconfig.load()
AttributeError: module 'synthtool' has no attribute 'preconfig'

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


testing: fuzz corpus collection regression

What version of Go are you using (go version)?

go version devel go1.18-b2dc66c64d Thu Jan 27 16:07:55 2022 +0000 darwin/arm64

What did you do?
Run fuzzing before and after commit b2dc66c64db933120c34d2223e670e8594543bd9.
What did you expect to see?
More or less the same results.
What did you see instead?
Before that commit
$ go clean -cache -testcache -fuzzcache

$ go test -fuzz=FuzzArray -fuzztime=10s
fuzz: elapsed: 0s, gathering baseline coverage: 0/3 completed
fuzz: elapsed: 0s, gathering baseline coverage: 3/3 completed, now fuzzing with 10 workers
fuzz: elapsed: 3s, execs: 115569 (38294/sec), new interesting: 16 (total: 16)
fuzz: elapsed: 6s, execs: 115569 (0/sec), new interesting: 16 (total: 16)
fuzz: elapsed: 9s, execs: 115569 (0/sec), new interesting: 16 (total: 16)
fuzz: elapsed: 12s, execs: 122775 (2403/sec), new interesting: 18 (total: 18)
fuzz: elapsed: 12s, execs: 122775 (0/sec), new interesting: 18 (total: 18)
PASS
ok  	github.com/FerretDB/FerretDB/internal/bson	12.374s

go test -fuzz=FuzzArray -fuzztime=10s
fuzz: elapsed: 0s, gathering baseline coverage: 0/21 completed
fuzz: elapsed: 0s, gathering baseline coverage: 21/21 completed, now fuzzing with 10 workers
fuzz: elapsed: 3s, execs: 42415 (14138/sec), new interesting: 10 (total: 28)
fuzz: elapsed: 6s, execs: 42471 (19/sec), new interesting: 10 (total: 28)
fuzz: elapsed: 9s, execs: 42471 (0/sec), new interesting: 10 (total: 28)
fuzz: elapsed: 12s, execs: 42471 (0/sec), new interesting: 10 (total: 28)
fuzz: elapsed: 12s, execs: 42471 (0/sec), new interesting: 10 (total: 28)
PASS
ok  	github.com/FerretDB/FerretDB/internal/bson	12.294s

The first run has a baseline coverage of 3 (provided by f.Add). The second run starts with a coverage of 21: 3 + 18 from the previous run.
After that commit
$ go clean -cache -testcache -fuzzcache

$ go test -fuzz=FuzzArray -fuzztime=10s
fuzz: elapsed: 0s, gathering baseline coverage: 0/3 completed
fuzz: elapsed: 0s, gathering baseline coverage: 3/3 completed, now fuzzing with 10 workers
fuzz: elapsed: 3s, execs: 123346 (41046/sec), new interesting: 11 (total: 11)
fuzz: elapsed: 6s, execs: 183855 (20197/sec), new interesting: 15 (total: 15)
fuzz: elapsed: 9s, execs: 189491 (1876/sec), new interesting: 16 (total: 16)
fuzz: elapsed: 12s, execs: 189491 (0/sec), new interesting: 16 (total: 16)
fuzz: elapsed: 12s, execs: 189491 (0/sec), new interesting: 16 (total: 16)
PASS
ok  	github.com/FerretDB/FerretDB/internal/bson	12.561s

$ go test -fuzz=FuzzArray -fuzztime=10s
fuzz: elapsed: 0s, gathering baseline coverage: 0/4 completed
fuzz: elapsed: 0s, gathering baseline coverage: 4/4 completed, now fuzzing with 10 workers
fuzz: elapsed: 3s, execs: 49002 (16328/sec), new interesting: 6 (total: 7)
fuzz: elapsed: 6s, execs: 61815 (4271/sec), new interesting: 10 (total: 11)
fuzz: elapsed: 9s, execs: 61815 (0/sec), new interesting: 10 (total: 11)
fuzz: elapsed: 12s, execs: 61815 (0/sec), new interesting: 10 (total: 11)
fuzz: elapsed: 12s, execs: 61815 (0/sec), new interesting: 10 (total: 11)
PASS
ok  	github.com/FerretDB/FerretDB/internal/bson	12.443s

$ ls /Users/aleksi/Library/Caches/go-build/fuzz/github.com/FerretDB/FerretDB/internal/bson/FuzzArray | wc
      23      23    1495

$ go test -fuzz=FuzzArray -fuzztime=10s
fuzz: elapsed: 0s, gathering baseline coverage: 0/4 completed
fuzz: elapsed: 0s, gathering baseline coverage: 4/4 completed, now fuzzing with 10 workers
fuzz: elapsed: 3s, execs: 92355 (30774/sec), new interesting: 15 (total: 16)
fuzz: elapsed: 6s, execs: 92355 (0/sec), new interesting: 15 (total: 16)
fuzz: elapsed: 9s, execs: 102979 (3550/sec), new interesting: 17 (total: 18)
fuzz: elapsed: 12s, execs: 102979 (0/sec), new interesting: 17 (total: 18)
fuzz: elapsed: 12s, execs: 102979 (0/sec), new interesting: 17 (total: 18)
PASS
ok  	github.com/FerretDB/FerretDB/internal/bson	12.445s

$ ls /Users/aleksi/Library/Caches/go-build/fuzz/github.com/FerretDB/FerretDB/internal/bson/FuzzArray | wc
      32      32    2080

Notice how the second and the third runs start with 4.
I also noticed that my generated corpus, which was already quite big and did not grow much before that commit, started growing much faster after it. So it seems to be a corpus loading problem, not just a logging problem.

@gopherbot fuzz

/cc @katiehockman @rolandshoemaker

@katiehockman @rolandshoemaker After updating Go, I noticed the following in my fuzzing output::
fuzz: elapsed: 0s, gathering baseline coverage: 0/23 completed
fuzz: elapsed: 0s, gathering baseline coverage: 23/23 completed, now fuzzing with 6 workers
fuzz: elapsed: 3s, execs: 5420 (1806/sec), new interesting: 2 (total: 21)

Does it mean that four entries (23-(21-2)) in my corpus were duplicates? And those four entries are duplicates because they are present in both generated corpus and code? That makes sense, but I think that would be a source of question and should be mentioned in docs.

This is currently documented at https://go.dev/doc/fuzz under "Command Line Output":

new interesting: the total number of “interesting” inputs that have been added to the generated corpus during this fuzzing execution (with the total size of the generated corpus)

I can see how that output might be a bit confusing. Maybe the total should be the size of the entire corpus, not just the generated corpus. @rolandshoemaker what do you think?

Yeah making the total the combined size makes sense to me (without looking at the docs that was what I initially assumed that total was.)


How to invoke "Align CSV"

From the documentation it is not clear how to invoke the "Align CSV".
From the animated screenshot it is not clear where to click for the magic to happen.
I tried many left and right clicks on every imaginable and non-imaginable places but none duplicated the screenshot.
Please help....
Meir

@mguttman Could you post a small example of CSV that does not work for you?


Fixed #24858 -- Added custom _get_FIELD_display to ArrayField

To support choices on the contrib.postgres ArrayField, added a custom get_FIELD_display through contribute_to_class.
https://code.djangoproject.com/ticket/24858

From a quick glance, I don't think that the one new test covers all the new code.
I would like if we could solve the issue without repeating so much code from the base method, but I haven't looked into the details.

Closing due to lack of follow up. Please send a new PR if you return to it.


Update README.md

curl install is insecure, and it's a bad habit to suggest it as the preferred method, by including it first.

curl install is insecure

Hey @evelynmitchell,
Thanks for the PR, but I don't think it is a valid excuse for not allowing the users to install it via cURL, as it is quite visible what that is going to run and nothing "insecure" with it.


FunctionApp:az functionapp wont work

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp
Errors:
cannot import name 'DefaultErrorResponseException' from 'azure.mgmt.web.models' (/usr/lib/python3/dist-packages/azure/mgmt/web/models.py)
Traceback (most recent call last):
python3/dist-packages/knack/cli.py, ln 233, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 584, in execute
    parsed_args = self.parser.parse_args(args)
python3/dist-packages/knack/parser.py, ln 262, in parse_args
    return super(CLICommandParser, self).parse_args(args)
...
cli/command_modules/appservice/custom.py, ln 39, in <module>
    from azure.mgmt.web.models import KeyInfo, DefaultErrorResponseException
ImportError: cannot import name 'DefaultErrorResponseException' from 'azure.mgmt.web.models' (/usr/lib/python3/dist-packages/azure/mgmt/web/models.py)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az functionapp

Expected Behavior
Environment Summary
Linux-5.11.0-37-generic-x86_64-with-glibc2.33
Python 3.9.5
Installer: DEB

azure-cli 2.18.0 *

Extensions:
azure-devops 0.19.0

Additional Context

route to service team

Please provide the full az function command which triggers you the error.

closing due to lack of response. Please file a new issue if this is still a problem.


train own pair data x2

Thanks for the great GAN.
I have one question.
How can I do x2 training using my own paired data?
Is it enough to change scale: 4 to 2 in finetune_realesrgan_x4plus_pairdata.yml?

You also need to modify the scale in the network arch and the pre-trained model paths.
You can  refer to this option file for X2 model: https://github.com/xinntao/Real-ESRGAN/blob/master/options/train_realesrgan_x2plus.yml


[Search pipelines] Pass "adhocness" flag to processor factories (#8164)

Description
A named search pipeline may be created with a PUT request, while an "anonymous" or "ad hoc" search pipeline can be defined in the search request source. In the latter case, we don't want to create any "resource-heavy" processors, since they're potentially increasing the cost of every search request, whereas named pipeline processors get reused.
This change passes a configuration flag to a processor factory if it's being called as part of an ad hoc pipeline. The factory can use that information to avoid allocating expensive resources (maybe by throwing an exception instead).
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta @andrross @dblock @owaiskazi19 @saratvemulapalli requesting reviews, we are targeting at GA for search pipeline for 2.9.0

Thanks @tlfeng !!


Readme clarification request: difference between using this and directly using sequelize v5

I am new to TS, so this might be trivial, so sorry about that in advance. But, asking directly: now that Sequelize v5 ships TS typings, what advantages does this library still add? I think it would be nice to have this clarified in the readme, so that one can immediately know it without further digging. (Unless there is a super obvious answer, in which case I am respectfully sorry, I am very new to TS).
Thanks for looking :)

This lib provides a lot of magnificent features like clear model definition with methods, quick hints for data fields, decorators and etc

Hey @papb thanks for your question :) the main goal of sequelize-typescript is not to provide typings - this was already solved with @types/sequelize. As BadCoder1337 stated, sequelize-typescript provides decorators and some other features in order to simplify model definition, create relations between models and more. For example, see model definition with pure sequelize: http://docs.sequelizejs.com/manual/models-definition.html and with sequelize-typescript on the other hand https://github.com/RobinBuschmann/sequelize-typescript/blob/master/README.md#model-definition. With the latter one you‘ll have your type and your table setup at one place.

I see!! Thanks @BadCoder1337 and @RobinBuschmann  :grin:


AVT 1 - Carbon X Radio Button has DAP violation

macOS Mojave version 10.14.2, Chrome version 72.0.3626.96 (Official Build) (64-bit), DAP w/ latest rule set Aug. 2018
Run DAP on Radio Button (See screenshot for example and violations).

Note: The 3rd button is disabled so it should include aria-disabled="true" in the label to fix the contrast issue.

Although the recommended fix will remove the violation it turns out that there is a DAP GitHub issue 1184 that is causing the violation in error and it should be fixed in the next DAP release expected later this month.


Task List

[x] Additional markup is supported, including @mentions, #references, links, emphasis, and HTML tags.
[x] List syntax is required.

You can nest lists below, too!

[x] This item is completed.
[ ] This item is not complete.

yes

[ ] Turn on GitHub Pages
[ ] Outline my portfolio
[ ] Introduce myself to the world

sorry

Completed


docs website: right arrow when table of content sidebar is shown cannot be clicked

When you make your browser window broad enough so that you can see a chapter table of contents on the right, it becomes impossible to click the right arrow in the navigation interface at the bottom. This is likely become something in the sidebar interferes with the nav buttons. When I make the user interface more narrow I can successfully click the right arrow. I tried this in both Firefox and Chrome.
Example, make the window full width and try to click the right arrow here:
https://xsltng.docbook.org/guide/2.2.0/ch-using#epub-in-ation

This is a serious bug. When the TOC is open there is no way to click Forward/Right to the next topic. I noticed setting the z-index: 1 on the "bottom" class fixes the layers so the right arrow is clickable.
<nav class="bottom" style="z-index: 1;">

The TOC height should be recalculated to not overlap with the bottom footer row.


Release 8.0.0 to rubygems

It seems that since https://github.com/rails/rails/pull/27503, a Gemfile with:
source 'https://rubygems.org'

gem 'rails', github: 'rails/rails'

fails bundle install with:
Fetching git://github.com/rails/rails.git
Fetching gem metadata from https://rubygems.org/..............
Fetching version metadata from https://rubygems.org/..
Resolving dependencies...
Bundler could not find compatible versions for gem "arel":
  In Gemfile:
    rails was resolved to 5.1.0.alpha, which depends on
      activerecord (= 5.1.0.alpha) was resolved to 5.1.0.alpha, which depends on
        arel (~> 8.0)

Could not find gem 'arel (~> 8.0)', which is required by gem 'activerecord (= 5.1.0.alpha)', in any of the sources.

Checking https://rubygems.org/gems/arel reveals that 8.0.0 has not been released to rubygems.

Need to add gem "arel", github: "rails/arel" line also.
https://github.com/rails/rails/blob/master/guides/bug_report_templates/active_record_master.rb#L11

Ah, so is 8.0.0 not ready?

Yeah, I think so.

Ok, thanks. I'll close this then.


Query no longer works

Query no longer runs because AADSpnSignInEventsBeta no longer exists (ErrorCode and Timestamp are also invalid)
https://github.com/reprise99/Sentinel-Queries/blob/211f6e959e5aecf1230c1810e2ee01a1e8ca706f/Azure Active Directory/Identity-ServicePrincipalswithSingleIP.kql#L27

I just tested it in a lab tenant, and it is still ok, are you running that query in Advanced Hunting and not Sentinel? Unless those tables are slowly moving out of beta and your environment has migrated over already

I was running the query in Sentinel. I just tried it through Advanced Hunting  in the Security Portal and it works.
Looking at the results in Advanced Hunting, I think I saw the same results in Sentinel when I changed the query so that:
AADSpnSignInEventsBeta -> SigninLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
IsManagedIdentity (can't remember what I used here)
If I go to connectors and search for "Service Principal sign-ins" I don't get any results (using an E5 license). Maybe we have migrated out of preview?

From: reprise99 @.>
Sent: Friday, August 11, 2023 11:42 PM
To: reprise99/Sentinel-Queries @.>
Cc: Craig Chambers @.>; Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I just tested it in a lab tenant, and it is still ok, are you running that query in Advanced Hunting and not Sentinel? Unless those tables are slowly moving out of beta and your environment has migrated over already
—
Reply to this email directly, view it on GitHubhttps://github.com/reprise99/Sentinel-Queries/issues/26#issuecomment-1675660032, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIYDMNZGWP6VEGHDSUYE2ZLXU33SXANCNFSM6AAAAAA3NFA2MQ.
You are receiving this because you authored the thread.Message ID: @.***>

Correction:
AADSpnSignInEventsBeta -> SigninLogs AADServicePrincipalSignInLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
and remove the line:   | where IsManagedIdentity == 0
The query runs as expected and gives me the same results as I get in Advanced Hunting using the published query.

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:33 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I was running the query in Sentinel. I just tried it through Advanced Hunting  in the Security Portal and it works.
Looking at the results in Advanced Hunting, I think I saw the same results in Sentinel when I changed the query so that:
AADSpnSignInEventsBeta -> SigninLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
IsManagedIdentity (can't remember what I used here)
If I go to connectors and search for "Service Principal sign-ins" I don't get any results (using an E5 license). Maybe we have migrated out of preview?

From: reprise99 @.>
Sent: Friday, August 11, 2023 11:42 PM
To: reprise99/Sentinel-Queries @.>
Cc: Craig Chambers @.>; Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I just tested it in a lab tenant, and it is still ok, are you running that query in Advanced Hunting and not Sentinel? Unless those tables are slowly moving out of beta and your environment has migrated over already
—
Reply to this email directly, view it on GitHubhttps://github.com/reprise99/Sentinel-Queries/issues/26#issuecomment-1675660032, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIYDMNZGWP6VEGHDSUYE2ZLXU33SXANCNFSM6AAAAAA3NFA2MQ.
You are receiving this because you authored the thread.Message ID: @.***>

Below is the full query I used in Sentinel to receive the same results as this one that worked in Security Portal -> Advanced Hunting
//Find Azure AD service principals that have only signed in from a single IP address in the last month. You should apply Conditional Access for workloads on them - https://docs.microsoft.com/en-us/azure/active-directory/conditional-access/workload-identity
//Data connector required for this query - Azure Active Directory - Service Principal Signin Logs
//Microsoft Sentinel query
//First create a distinct count of IP addresses for each AppId and return any AppId with only 1 distinct IP
let appid=
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| summarize dcount(IPAddress) by AppId
| where dcount_IPAddress == 1
| distinct AppId;
//Query the same data for only those AppIds and summarize each IP by the AppId and friendly service principal names
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| where AppId in (appid)
| summarize ['Application Ids']=make_set(AppId) by IPAddress, ServicePrincipalName
//Advanced Hunting query
//Data connector required for this query - Advanced Hunting with Azure AD P2 License
//First create a distinct count of IP addresses for each AppId and return any AppId with only 1 distinct IP
let appid=
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| summarize dcount(IPAddress) by AppId
| where dcount_IPAddress == 1
| distinct AppId;
//Query the same data for only those AppIds and summarize each IP by the AppId and friendly service principal names
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| where AppId  in (appid)
| summarize ['Application Ids']=make_set(AppId) by IPAddress, ServicePrincipalName

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:49 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
Correction:
AADSpnSignInEventsBeta -> SigninLogs AADServicePrincipalSignInLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
and remove the line:   | where IsManagedIdentity == 0
The query runs as expected and gives me the same results as I get in Advanced Hunting using the published query.

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:33 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I was running the query in Sentinel. I just tried it through Advanced Hunting  in the Security Portal and it works.
Looking at the results in Advanced Hunting, I think I saw the same results in Sentinel when I changed the query so that:
AADSpnSignInEventsBeta -> SigninLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
IsManagedIdentity (can't remember what I used here)
If I go to connectors and search for "Service Principal sign-ins" I don't get any results (using an E5 license). Maybe we have migrated out of preview?

From: reprise99 @.>
Sent: Friday, August 11, 2023 11:42 PM
To: reprise99/Sentinel-Queries @.>
Cc: Craig Chambers @.>; Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I just tested it in a lab tenant, and it is still ok, are you running that query in Advanced Hunting and not Sentinel? Unless those tables are slowly moving out of beta and your environment has migrated over already
—
Reply to this email directly, view it on GitHubhttps://github.com/reprise99/Sentinel-Queries/issues/26#issuecomment-1675660032, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIYDMNZGWP6VEGHDSUYE2ZLXU33SXANCNFSM6AAAAAA3NFA2MQ.
 Your git contains two queries, one for Sentinel and one for Advanced Hunting. I was putting both into Sentinel which caused it to use the Advanced Hunting one (blush)
Sorry!

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:51:25 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
Below is the full query I used in Sentinel to receive the same results as this one that worked in Security Portal -> Advanced Hunting
//Find Azure AD service principals that have only signed in from a single IP address in the last month. You should apply Conditional Access for workloads on them - https://docs.microsoft.com/en-us/azure/active-directory/conditional-access/workload-identity
//Data connector required for this query - Azure Active Directory - Service Principal Signin Logs
//Microsoft Sentinel query
//First create a distinct count of IP addresses for each AppId and return any AppId with only 1 distinct IP
let appid=
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| summarize dcount(IPAddress) by AppId
| where dcount_IPAddress == 1
| distinct AppId;
//Query the same data for only those AppIds and summarize each IP by the AppId and friendly service principal names
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| where AppId in (appid)
| summarize ['Application Ids']=make_set(AppId) by IPAddress, ServicePrincipalName
//Advanced Hunting query
//Data connector required for this query - Advanced Hunting with Azure AD P2 License
//First create a distinct count of IP addresses for each AppId and return any AppId with only 1 distinct IP
let appid=
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| summarize dcount(IPAddress) by AppId
| where dcount_IPAddress == 1
| distinct AppId;
//Query the same data for only those AppIds and summarize each IP by the AppId and friendly service principal names
AADServicePrincipalSignInLogs
| where TimeGenerated > ago (30d)
| where ResultType == 0
| where AppId  in (appid)
| summarize ['Application Ids']=make_set(AppId) by IPAddress, ServicePrincipalName

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:49 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
Correction:
AADSpnSignInEventsBeta -> SigninLogs AADServicePrincipalSignInLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
and remove the line:   | where IsManagedIdentity == 0
The query runs as expected and gives me the same results as I get in Advanced Hunting using the published query.

From: Craig Chambers @.>
Sent: Saturday, August 12, 2023 4:33 PM
To: reprise99/Sentinel-Queries @.>; reprise99/Sentinel-Queries @.>
Cc: Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I was running the query in Sentinel. I just tried it through Advanced Hunting  in the Security Portal and it works.
Looking at the results in Advanced Hunting, I think I saw the same results in Sentinel when I changed the query so that:
AADSpnSignInEventsBeta -> SigninLogs
Timestamp -> TimeGenerated
ApplicationId -> AppId
ErrorCode -> ResultType
IsManagedIdentity (can't remember what I used here)
If I go to connectors and search for "Service Principal sign-ins" I don't get any results (using an E5 license). Maybe we have migrated out of preview?

From: reprise99 @.>
Sent: Friday, August 11, 2023 11:42 PM
To: reprise99/Sentinel-Queries @.>
Cc: Craig Chambers @.>; Author @.>
Subject: Re: [reprise99/Sentinel-Queries] Query no longer works (Issue #26)
I just tested it in a lab tenant, and it is still ok, are you running that query in Advanced Hunting and not Sentinel? Unless those tables are slowly moving out of beta and your environment has migrated over already
—
Reply to this email directly, view it on GitHubhttps://github.com/reprise99/Sentinel-Queries/issues/26#issuecomment-1675660032, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIYDMNZGWP6VEGHDSUYE2ZLXU33SXANCNFSM6AAAAAA3NFA2MQ.
You are receiving this because you authored the thread.Message ID: @.***>

All good! And yeah some I write both because the schema isn't a match (as you found out!)


AttributeError: module 'tensorflow.python.ops.gen_logging_ops' has no attribute '_scalar_summary'v

Hello
I getting this error while train my data i  already install latest version of tensor flow
errors
AttributeError: module 'tensorflow.python.ops.gen_logging_ops' has no attribute '_scalar_summary'

Can you provide a sample code? Please use proper makrdown syntax to highlight the code

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

@rahimullah11 ,
Any update on the issue ?Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Update vm-specialized-image-version.md

Following should be clearly stated in the documentation for clarity:
"Specialized VM image contains operating system disk and all the attached disks."

@sandeepraichura
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Make the footer

Decided footer isn't needed as our site works well with only the navbar and honestly, the space it would take is a pain. I'd rather be able to style it the way I want then worry about fitting in a footer that I feel is unnecessary.


Add helper to sync maintainers file

Ran black and replaced format with f-strings (forgot those exist 😄)


Droping a Graph

I have found an error when trying to drop a graph. The code is the following:
  db.dropGraph(graphName)

The stack of the error is the following:
Unhandled rejection ArangoError: TypeError: Cannot read property '_properties' of undefined
  at new ArangoError (/path/to/project/node_modules/arangojs/lib/error.js:10:13)
  at Request._callback (/path/to/project/node_modules/arangojs/lib/connection.js:78:81)
  at Request.self.callback (/path/to/project/node_modules/arangojs/node_modules/request/request.js:368:22)
  at Request.emit (events.js:98:17)
  at Request.<anonymous> (/path/to/project/node_modules/arangojs/node_modules/request/request.js:1219:14)
  at Request.emit (events.js:117:20)
  at IncomingMessage.<anonymous> (/path/to/project/node_modules/arangojs/node_modules/request/request.js:1167:12)
  at IncomingMessage.emit (events.js:117:20)
  at _stream_readable.js:943:16
  at process._tickCallback (node.js:419:13)

The error behaves as following:

The error is raised even if the collection exists or not
Even though the error the Graph is droped

Thanks for spotting this one.

Forget this one, is related to the #31, i've just tested it right now

Could you have a look at it? I still have this error, the behavior is the same as commented on the issue. I thought it works because i had in a catch the code because it deletes the Graph anyway.
More trace:
{
        error: true,
        code: 400,
        errorNum: 17,
        errorMessage: 'TypeError: Cannot read property \'_properties\' of undefined'
}

That's a server-side error message. Can you confirm that this doesn't happen if you use the HTTP API directly?
i.e. curl -X DELETE http://localhost:8529/_api/gharial/graphName

Hi @pluma,
Sorry for making you wait, i've tried the CURL and this is the result I get:

First i've checked that the Graph was created (via web)
Then I execute the CURL: curl -X DELETE http://localhost:8529/_api/gharial/recipesGraph
Answer: {"error":true,"code":404,"errorNum":1924,"errorMessage":"graph not found"} i've checked the name and it's the correct one.
I go to check the Graph to the web with a wierd web error because when i open it, it turns the screen in black with 11 alerts saying: "Sorry your graph seems to be empty"
And some times the graph opens normally.
it's a 'big' (more or less i don't know jeje) graph, 117k edges and 11k + 4k nodes

You can reach me via Gitter if you want :smile:

That appears to be a bug in the REST API of ArangoDB itself then. Could you please open an issue on the arangodb issue tracker and cross-reference this issue? That will help make sure the right people see it.
Thank you for the patience.
On 24 Aug 2015 12:51 am, Francesc Gil<EMAIL_ADDRESS>wrote:Hi @pluma,
Sorry for making you wait, i've tried the CURL and this is the result I get:
First i've checked that the Graph was created (via web)Then I execute the CURL: curl -X DELETE http://localhost:8529/_api/gharial/recipesGraph
Answer: {"error":true,"code":404,"errorNum":1924,"errorMessage":"graph not found"} i've checked the name and it's the correct one.I go to check the Graph to the web with a wierd web error because when i open it, it turns the screen in black with 11 alerts saying: "Sorry your graph seems to be empty"And some times the graph opens normally.it's a 'big' (more or less i don't know jeje) graph, 117k edges and 11k + 4k nodes
You can reach me via Gitter if you want
—Reply to this email directly or view it on GitHub.


TypeScript Support

Are there plans to provide TypeScript support?
I'm not suggesting a rewrite of the lib in TypeScript (although that would be nice from a maintainer's perspective), but adding a type definition export.
For example defining all the component options, e.g. for Block:
import type { Object3D } from "three";

export interface BlockOptions extends Object3D {
  width: number;
  height: number;
  borderWidth?: number;
  // ...
}

It wouldn't involve editing any of the existing source files, only adding a new types.d.ts file, and updating the build process to include the types definition, e.g. updating the package.json to specify the types:
{
  "name": "three-mesh-ui",
  "main": "dist/three-mesh-ui.js",
  "module": "src/three-mesh-ui.js",
  "types": "dist/types.d.ts"
}

Well if it does not involve adding anything to the existing files, then yes it's something that could be added to the lib.

Understood, I will work on adding the types.d.ts at some future point, when I'm more familiar with the project. I plan on working on some of the features in the Roadmap first.

thanks a lot for your help 😊

Would love types for this. I like the library but can't use it without types as I'm a TypeScript developer.

@Enijar any update on this ?

I'm still in the process of working on this (when I have the free time). I have the types for Block almost done, but there's still lots more types to add for the other components

If you submit the PR as draft @kevinjardine could maybe help you ?

I'd be happy to help where I can.
Kevin

I've created a WIP pull request here #146

Hey @Enijar, happy to see your WIP TS support PR made it into 6.3.0. This update is a great first step towards supporting TS further, but is currently missing a lot of the existing interface, breaking any TS user that updates to 6.3.0 and may also have TS users believe the library is much more limited than it really is. Any thoughts on how to disable Types for existing users such as us for now? Or would it be better off having Types updated to expose everything?

Hey @Enijar!
We've been using Three Mesh UI for about a year now and have built a majority of our UI with the library. In that time we have used a lot of different APIs and properties set on the components, which is why my list below will be a little long. That being said, it is probably also incomplete, as there are also still areas of the library's code base that we have yet to utilize. We have the // @ts-ignore workaround for now of course, so we will be able to integrate 6.3.0 without breaking our application. But I do worry about future TS users attempting to integrate the library for the first time and find it potentially limiting.
I have several examples so bare with me here:
High Priority:

Base class MeshUIComponent is missing from Types, this core module exposes several principal methods of components, such as set, setupState and setState

Without set, no Block can be updated after creation

Dimension related functions on Blocks, such as getWidth and getHeight, derived from BoxComponent, are not exposed
Core components BoxComponent, InlineComponent, InlineManager, MaterialManager, MeshUIComponent, TextManager, and UpdateManager are not exposed. Block, InlineBlock and Text all "inherit" from these components. The functions they provide are used to manage complex UIs, an example being again set or onAfterUpdate

FontLibrary seems to have made its way in though, so that is a plus

Medium Priority:

Width and height on BlockOptions should be set to optional given BlockOptions can be used on the set API
Direct access to a Blocks properties do not exist. Examples include backgroundColor, frame, size, padding, margin, borderRadius and a lot more

Low Priority:

import ThreeMeshUI from "three-mesh-ui"; no longer works, needs to be updated to import * ThreeMeshUI from "three-mesh-ui";

I've been able to use the types file - the only change I needed was to replace
fontFamily?: string;
with
fontFamily?: any;
Thanks!
Will tell you if I find other issues.

Hey @enijar!
We've been using Three Mesh UI for about a year now and have built a majority of our UI with the library. In that time we have used a lot of different APIs and properties set on the components, which is why my list below will be a little long. That being said, it is probably also incomplete, as there are also still areas of the library's code base that we have yet to utilize. We have the // @ts-ignore workaround for now of course, so we will be able to integrate 6.3.0 without breaking our application. But I do worry about future TS users attempting to integrate the library for the first time and find it potentially limiting.
I have several examples so bare with me here:
High Priority:

Base class MeshUIComponent is missing from Types, this core module exposes several principal methods of components, such as set, setupState and setState

Without set, no Block can be updated after creation

Dimension related functions on Blocks, such as getWidth and getHeight, derived from BoxComponent, are not exposed

Core components BoxComponent, InlineComponent, InlineManager, MaterialManager, MeshUIComponent, TextManager, and UpdateManager are not exposed. Block, InlineBlock and Text all "inherit" from these components. The functions they provide are used to manage complex UIs, an example being again set or onAfterUpdate

FontLibrary seems to have made its way in though, so that is a plus

Medium Priority:

Width and height on BlockOptions should be set to optional given BlockOptions can be used on the set API
Direct access to a Blocks properties do not exist. Examples include backgroundColor, frame, size, padding, margin, borderRadius and a lot more

Low Priority:

import ThreeMeshUI from "three-mesh-ui"; no longer works, needs to be updated to import * as ThreeMeshUI from "three-mesh-ui";

I'm trying to build a simple UI with this library in my angular project but as you say I dont have access to MeshUIComponents or de set method. Do you have any idea about how could I solve this?

Hey! I am having troubles building an app with this UI, but due to type errors in types.d.ts file
The exact line that does not let my app to be builded remotely is this:
     borderRadius?: number | [topLeft: number, topRight: number, bottomRight: number, bottomLeft: number];
That is a part of `BlockOptions ` type description

If you need help defining types in the lib - reach out to me. I am glad to help.


March 22nd has stale data for New York, Colorado, others

Guys, for several states all measures (cases, etc.) for March 21 and 22 are the same.

What file @ekaterinakuzmina?

It sounds like you're saying that the numbers did not change between the 21st and 22nd?
Can you provide an exhaustive list of states you saw this in?

Just verified this is definitely happening for Colorado on 3/21 :
"2020-3-19": {
        "cases": 264,
        "growthFactor": 1.2394366197183098
      },
      "2020-3-20": {
        "cases": 343,
        "growthFactor": 1.2992424242424243
      },
      "2020-3-21": {
        "cases": 474,
        "growthFactor": 1.3819241982507289
      },
      "2020-3-22": {
        "cases": 474,
        "growthFactor": 1
      },
      "2020-3-23": {
        "cases": 715,
        "growthFactor": 1.5084388185654007
      },
      "2020-3-24": {
        "cases": 894,
        "growthFactor": 1.2503496503496503
      },
      "2020-3-25": {
        "cases": 1068,
        "growthFactor": 1.1946308724832215
      }

https://covidatlas.com/colorado-us shows different case numbers for those dates.
Same for https://covidatlas.com/new-york-us.
I believe this has been resolved, closing! z


Add pull request template

Adds a pull request template with questions to help repo maintainers know how to process.
With datasets and tutorials there is a need for more information on PRs.
We could add ones specifically for datasets and tutorials (etc) in the future, just want to initialize a generic one as a starter.

@gino-m
@schwehr
@simonff
All comments addressed or commented on, PTAL.


when using html2canvas , why image with on-line url can't appear in result ,is there anyone know it? please tell me,thanks!

is any one to answer this???

nobody？

Your question is not very clear. Perhaps if you provided a jsfiddle
illustrating it, or eve just some code with images of what you're expecting
versus what you're getting, you may get more lucky with an answer.
On Wed, 17 May 2017 at 09:37, cheng<EMAIL_ADDRESS>wrote:

nobody？
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/niklasvh/html2canvas/issues/1077#issuecomment-302024362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQbyPOYTYckXrlw3Txj5bjjEVXUmtRhZks5r6rHagaJpZM4MdcVu
.

--
Chris Fox
Chris C Fox Consulting Limited
email<EMAIL_ADDRESS>phone: +44 77 860 21712
web: http://www.chriscfox.com
linkedin: http://www.linkedin.com/in/chriscfox
twitter: http://www.twitter.com/chriscfox
Google+: http://gplus.to/chriscfox
Facebook: http://www.facebook.com/pages/StrategicCoffee/102920468071
skype: chriscfox
Have you tried https://www.StratNavApp.com https://www.stratnavapp.com/,
the online collaborative tool for strategy development and execution?
Chris C Fox Consulting Limited is registered in England and Wales as a
Private Limited Company: Company Number 6939359. Registered Office: Unit 4
Vista Place, Coy Pond Business Park, Ingworth Road, Poole BH12 1JY

I believe you are talking about cross-domain images. For Internet security reasons, trying to use an image from a different domain in a canvas causes problems. There are two html2canvas options, useCORS and proxy, which are designed to try to get around that problem. They're not perfect solutions though - you can read more over at #951 and #592.

    html2canvas($('.page-list .page-thumb').eq(0), {
      logging: true, // 日志
      useCORS: true, // 允许跨域
      width: 110, // 缩略图宽
      height: 173 // 缩略图高
    }).then((canvas) => {
      var url = canvas.toDataURL("image/png");

      this.$store.dispatch('saveScene', url);
    });

log:
163ms html2canvas: Document cloned
html2canvas.min.js?ca29:7 166ms html2canvas: Initialized CanvasRenderer with size 1920 x 900
html2canvas.min.js?ca29:7 167ms html2canvas: Starting NodeParser
html2canvas.min.js?ca29:7 174ms html2canvas: Fetched nodes, total: 8
html2canvas.min.js?ca29:7 174ms html2canvas: Calculate overflow clips
html2canvas.min.js?ca29:7 178ms html2canvas: Start fetching images
html2canvas.min.js?ca29:7 179ms html2canvas: Added image #1 http://<IP_ADDRESS>:8088/custom/images/daq_a15e85aab2e2f6ada76e86867f3bd179.jpg
html2canvas.min.js?ca29:7 181ms html2canvas: Added image #2 http://<IP_ADDRESS>:8088/custom/images/daq_5e10973fc6763f93bb97976f624ffc43.png
html2canvas.min.js?ca29:7 182ms html2canvas: Finished searching images
localhost/:1 Access to Image at 'http://<IP_ADDRESS>:8088/custom/images/daq_5e10973fc6763f93bb97976f624ffc43.png' from origin 'http://localhost:8080' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access.
html2canvas.min.js?ca29:7 187ms html2canvas: Succesfully loaded image #2 d {src: "http://<IP_ADDRESS>:8088/custom/images/daq_a15e85aab2e2f6ada76e86867f3bd179.jpg", image: img, tainted: null, promise: Promise}
html2canvas.min.js?ca29:7 188ms html2canvas: Failed loading image #1 d {src: "http://<IP_ADDRESS>:8088/custom/images/daq_5e10973fc6763f93bb97976f624ffc43.png", image: img, tainted: null, promise: Promise} Event {isTrusted: true, type: "error", target: img, currentTarget: img, eventPhase: 2…}
html2canvas.min.js?ca29:7 189ms html2canvas: DummyImageContainer for http://<IP_ADDRESS>:8088/custom/images/daq_5e10973fc6763f93bb97976f624ffc43.png
html2canvas.min.js?ca29:7 189ms html2canvas: Images loaded, starting parsing
html2canvas.min.js?ca29:7 190ms html2canvas: Creating stacking contexts
html2canvas.min.js?ca29:7 190ms html2canvas: Sorting stacking contexts
html2canvas.min.js?ca29:7 191ms html2canvas: Render queue created with 16 items
html2canvas.min.js?ca29:7 201ms html2canvas: Finished rendering
html2canvas.min.js?ca29:7 201ms html2canvas: Cropping canvas at: left:<PHONE_NUMBER>171875 top: 79.89250183105469 width: 110 height: 173
html2canvas.min.js?ca29:7 202ms html2canvas: Resulting crop with width 110 and height 173 with x<PHONE_NUMBER>171875 and y 79.89250183105469
html2canvas.min.js?ca29:7 232ms html2canvas: Cleaned up container
one image success,but other one failed,why?


Update installing_clowder.rst

I believe there was a formatting issue in this yaml code on lines 216 and 239
Description

Review Time Estimate

[ ] Immediately
[x] Within one week
[ ] When possible

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the CHANGELOG.md.
[x] I have signed the CLA
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@lmarini Can we merge this doc fix? Looks good.


Missing icons when watching a stream

The Start an Acitivity and Stop Watching button icons when watching a stream are missing.
Screenshots
Start an Activity

Stop Watching

Note on the referenced commit: The fix for the activity button is speculative as I don't have access to run activities anywhere.
I'll leave this issue open for a little after the next release in case it wasn't caught.

So I made the same changes as you did on the #58 and I got this icon instead:

Not consistent with the Start Activity icon here:

Hey I don't mind if these are intended, just checking.


Safer terminal state management and better panic output

manage panics in a safe run func
rename unsafe run to _run func
ensure terminal state is restored

Fixes #118
Fixes #119

I strongly agree that the application should not panic. But at the same time, I am not convinced that another crate, which looks very cool, is needed to handle the panic. Since this is a small program of about 4000 LOC, scopeguard seems a bit overkill.
Anyway, I'll consider your suggestion.

I removed scopeguard and added a manual restore after the unwind of the panic.

Great! I learned a lot by this PR, thank you.
Added clear_all() and move_to(1, 1) (move cursor to top-left) to clear terminal state. How do you think?

Great! I learned a lot by this PR, thank you. Added clear_all() and move_to(1, 1) (move cursor to top-left) to clear terminal state. How do you think?

Yes, why not. It makes a much cleaner output after the panic.


[REQ] add a (GH-compliant) license file

Hi there, 1st of all an heartfelt thanks for what you do !
Since we've collected this repository in the Resources section \ FPGA subsection in our RecRack project, can you please add a GH-compliant license file ?
As you know, expliciting licensing terms is extremely important to let anyone better/faster understand how to reuse/adapt/modify your "sources" (and not only) in other open projects and vice-versa.
Although it may sounds like a minor aspect, license file omission obviously causes an inconsistent generation of the relative badge too:

(badge-generator URL: https://flat.badgen.net/github/license/nanomixer/nanomixer/?label=LICENSE)
Anyway you can easily set a standard one through the GH's license wizard tool.
Please let us also know how - in your opinion - we could improve our categorizations and links to resources in order to favor collaboration between developers (and therefore evolution) of listed projects.
Last but not least, check out this Hackaday.io list for possible ideas/collaborations.
Thanks in advance.

Cool to see others picking up on this idea. I added a License file.
If you're trying to actually use this code, it's probably easiest to jump on a quick call and I (or perhaps Martin) can explain what we remember.


Handle orbital overlap in spin operators

Summary
Fixes #1273. For more details, please refer to that issue.
Details and comments

[x] implements an actual fix for the underlying problem
[x] unittests the above
[x] use the above fix properly, directly within the drivers

~This is potentially problematic right now, because the QCSchema does not have a field to specify the overlap. I will need to investigate a good solution for this.~
I settled on adding an additional attribute to the QCSchema which stores the AO overlap. This seems like the best solution at this time.

[x] ensure the more complex AngularMomentum is handled properly by any Transformer class
[x] add a release note

The change is backward compatible though with any existing code is it not - the new parameter is optional and it will still do what it did before no? But technically it is an API change, though needed to sort the bug.


Air Quality Prediction By ML

Tech stacks used :-

numpy
pandas
matplotlib
seaborn
scikit-learn
shap
joblib
chardet
Dataset :-
global air pollution dataset.csv

@sanjay-kv Can you please assign me this issue under gssoc.
90% work is already done.

@sanjay-kv Hey please assign me this issue.


Integration tests: send and receive text and file

This PR adds integration tests (see: flutter and integration_test docs) which cover using the dart_wormhole_william library for both text and file transfer in the these scenarios:

Sender
Receiver

Dart API
Dart API

Dart API
Go CLI

Go CLI
Dart API

Where the Go CLI refers to wormhole-william.
Depends on https://github.com/LeastAuthority/dart_wormhole_william/pull/10
Depends on https://github.com/LeastAuthority/wormhole-william/pull/26

Code Review Checklist (to be filled out by reviewer)

[ ] Description accurately reflects what changes are being made.
[ ] Description explains why the changes are being made (or references an issue containing one).
[ ] The PR appropriately sized.
[ ] New code has enough tests.
[ ] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[ ] Existing documentation is up-to-date if impacted.

This PR is included in #2


Federation Heroes and Pirates

These units and buildings are used for the Federation campaigns.

Outdated by #725


Missing alleles in PANDORA database can be retrieved from IMGT database

The PANDORA database can be improved by adding alleles present in the IMGT database.
Both for MHCI and MHCII can be found here:
http://ftp.ebi.ac.uk/pub/databases/ipd/imgt/hla/fasta/

Thanks for the link, I have emailed the author and asked them what source we should use.
In the meantime, it seems like the missing alleles issue was actually caused by a sequence length interval too strict I set. Fixed in 542ede1 .


Cartoon Network Clone

Description
html css
Cartoon network
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


iconfoundation.io

Fake airdrop site asking for private keys
https://urlscan.io/result/dca9a0ed-15d3-4a7a-9ada-fbde4806b203/#summary
https://urlscan.io/result/8a8ee6ec-6ec2-4d97-bb02-f6e8d584ba43#summary
https://urlscan.io/result/1a49cf36-850f-40bf-8756-09e0db1614b7#summary

I put it in https://github.com/MrLuit/EtherScamDB/pull/344


Update akenza connector

Summary

Updates the akenza webhook to work with the new akenza platform.
Changes

Updates the akenza webhook to work with the new akenza platform.

Notes for Reviewers

Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[x] Compatibility: The changes are backwards compatible, they don't break existing deployments.
[x] Testing: The changes are tested.
[ ] Documentation: Relevant documentation is added or updated.

The 64 character limitation applies to message paths (uplink_message), but not to base_url. So it is entirely possible to put the full URL there for now, and to use just "" as a message path.

If you use https://data-gateway.akenza.io/v3/capture?secret=<secret-value> the data doesn't show up on akenza. On the other hand, if you use https://data-gateway.akenza.io/v3/capture?secret=<secret-value>&deviceId=<device-id> or https://data-gateway.akenza.io/v3/capture?secret=<secret-value>&deviceId=<device-id>&topic=<topic-name>, in both cases data shows up on akenza.
The base-url in this template is defined as https://data-gateway.akenza.io/v3/capture?secret={uplink_secret}, i.e. it's missing the deviceId part to make it work. However, if you add that part and define https://data-gateway.akenza.io/v3/capture?secret={uplink_secret}&deviceId={device_id}, the predefined maximum base URL length is exceeded.

The deviceId and topic are taken from the uplink body. I verified that the URL is correct using our current implementation which uses the default webhook instead of the akenza template.

Sent test uplink

uplink received on akenza

Here some documentation links to recreate this:
https://docs.akenza.io/get-started/your-integration/the-things-network
https://docs.akenza.io/get-started/your-data-flow
https://docs.akenza.io/tutorials/add-devices/how-to-integrate-a-decentlab-sensor-with-ttn-on-akenza select passthrought as a device type and send the uplink manually from TTN side


Trying to train model to understand enums in DB

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Error logs/Screenshots
If applicable, add logs/screenshots to give more information about the issue.
Desktop (please complete the following information where):

OS: [e.g. Ubuntu]
Version: [e.g. 20.04]
Python: [3.9]
Vanna: [2.8.0]

Additional context
Add any other context about the problem here.

Hmm, it seems like you just submitted the bug template instead of adding context under each header.
Without any context, I am unable to assist you any further, but this sounds like a discussion topic instead of a bug request. Am I correct?
If so, could you close this issue and open a Q/A Discussion in the Discussions tab instead?
https://github.com/vanna-ai/vanna/discussions/new?category=q-a
Then I or someone else could assist you further there :]


fix(streaming): less exchange permits (and chunk_size)

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
The motivation is to fix the OOM failure of TPC-H Q2 (#10330), and also improve the streaming latency for any streaming queries.
Check https://github.com/risingwavelabs/risingwave/issues/10330#issuecomment-1594151474 for more context. In short, the root cause is that the buffered message (StreamChunks) amid the graph used too much memory and caused OOM at some point.
As expected, this will decrease the streaming performance a little bit, but I think it worths.
Performance Comparison

TPC-H Q2 Passed
By the way, I have also tried keeping the chunk_size but only decreasing concurrent_barrier_limit to 1, but the test failed.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Have we tested only decreasing the "total permits"? I guess making the chunk size to be 256 hurts the most due to negative effects on vectorization and more overhead from dynamic dispatches.
BTW, it's interesting to find that this configuration influences stateless queries most, while there's no other memory usage for these queries so the risk of OOM is also lower. Perhaps we really need to introduce some dynamic strategies for the permit values. 🥵

perhaps we really need to introduce some dynamic strategies for the permit values.

Agree, that can be a long-term solution in the future, e.g. decided by the front end according to the largest number of exchanges among all the paths from source to sink of a job

Have we tested only decreasing the "total permits"?

Good idea. Just ran a benchmark: https://buildkite.com/risingwave-test/tpch-benchmark/builds/862
Previously I thought that chunk_size also determines the memory usage, more or less. But you are right, if that can solve the problem, I prefer not to touch the chunk_size

Have we tested only decreasing the "total permits"?

Updated comparisons:

Have we tested only decreasing the "total permits"?

Updated comparisons:

TPC-H Q2 succeeded under this configuration as well. I'm unsure if we need to adjust the numbers further for other more complex queries...

Unfortunately, OOM still happened... (but the test didn't fail, is it expected? cc. @cyliu0)
https://g-2927a1b4d9.grafana-workspace.us-east-1.amazonaws.com/d/EpkBw5W4k/risingwave-test-dashboard?orgId=1&from=1687246527000&to=1687247309000&var-namespace=eric-tpch-q2-concurrent-barrier-1

but the test didn't fail, is it expected?

Actually, it's failed all the time. It appeared in every day's brief like this. And this is quite an old known issue because of the OOM. I suppose everyone knows this. :(
Failed Execution List:
- tpch-q2-bs-medium-1cn: 4787

Unfortunately, OOM still happened...

IMO, 1024 rows in a chunk and 2 chunks in a buffer are really reasonable for a pipelined and vectorized system. 🥵 If this is still problematic, I would suggest decreasing the parallelism, upgrading the machine, or introducing dynamically allocated buffers. 😕

Unfortunately, OOM still happened...

IMO, 1024 rows in a chunk and 2 chunks in a buffer are really reasonable for a pipelined and vectorized system. 🥵 If this is still problematic, I would suggest decreasing the parallelism, upgrading the machine, or introducing dynamically allocated buffers. 😕

Yeah, 1024 looks like a typical setting for batch vectorized execution engine. While streaming is somehow special because it also has to trade off between latency and throughput. Larger chunk also increases the latency, because total latency = streaming processing time + time of waiting for chunk_size rows. Does this make sense to you?
"Decreasing the parallelism" also looks good. Q2 runs stably under parallelism=4 with same machine specs. But we don't have any better idea yet, right...

Larger chunk also increases the latency, because basically total latency = streaming processing time + time of waiting for chunk_size rows. Does this make sense to you?

I'm also considering this... Using a large chunk size causes operators to be more likely to buffer data in the memory and yield them all at once on the barrier, which also increases the total latency since this procedure is not pipelined.
FYI, we use ready_chunks when doing data buffering in the connector, which seems to be a good balance between batching and latency. Perhaps we should apply a similar idea to other stateful executors as well.

@BugenZhao So do you think it's okay to merge this PR? If so, please just click "Merge when ready" to avoid ping-pong

I'll help to fix the unit tests.


CLI Refresh: API key client option

Describe the solution you'd like
See https://github.com/temporalio/features/issues/426

Note, this will include an env var for this --api-key setting.


Type Safety for enumerable items

*Added constructor properties to prototypes for inheritance
*Added freezeEnum property to Enum to ensure type safety for instances of enum types.
Please see the comments for more details.

Hey bro, been away for holidays but finally got to finish the tests.
Defined Getter/Setter on freezeEnums to throwError on new set values,
Updated Readme and Added tests to enumTest.js file.
Hope all is well, let me know, Thx!


Adding support for Tensor.conj() and related functions.

This is a straight-forward edit: just adding a few Tensor methods related to conjugates.

@NiklasGustafsson I'd love to see the cause for the test failure. I see nothing wrong with the code. 😲

@NiklasGustafsson I'd love to see the cause for the test failure. I see nothing wrong with the code. 😲

So would I, and I will. :-)

Huh!
The fix is to avoid 'bool' as the return type of the native function:
int64_t THSTensor_is_conj(const Tensor tensor)
{
    CATCH_RETURN_RES(int64_t, 0, res = tensor->is_conj();)
}


Negotiating or Viewing MTU Settings

Hi everyone,
I wanted to ask if there’s a way to negotiate the MTU (Maximum Transmission Unit) on a connection, or if it’s at least possible to see what the MTU is currently set to. Any advice or insights on how to manage or view this would be greatly appreciated.
Thanks in advance for your help!
Best regards,
Bob

You can get the MTU by calling max_write_len on a characteristic. Most Bluetooth APIs don't provide any facility for userspace to re-negotiate the MTU selected by the OS.

Thank you, @alexmoon, for the explanation. I didn’t consider that max_write_len represents the actual MTU. I was under the impression that the desired MTU could be negotiated, as is possible in Android with FlutterBlue, which led to my incorrect assumption.


removed unused construction and added CI

Hello! Thank you for the pull request!
Could you please explain the purpose of this exact CI?
Also please note that this repo is just a mirror, so I'll repeat your changes on GitLab which will sync with GitHub itself.

Hello! Thank you for the pull request!
Could you please explain the purpose of this exact CI?
Also please note that this repo is just a mirror, so I'll repeat your changes on GitLab which will sync with GitHub itself.

This CI checks for vulnerabilities in the code.  It was added to prevent admission in the code of vulnerabilities

This CI checks for vulnerabilities in the code.  It was added to prevent admission in the code of vulnerabilities

I guess it only works with GitHub, right?

I guess it only works with GitHub, right?

perhaps.  I don't use gitlab so I can't say for sure

perhaps.  I don't use gitlab so I can't say for sure


Not sure if I was supposed to add it to enabled tags or just delete it

@kailynw You need only to remove it. But also you need to fix the linter.  Checks job on CI must pass. You can check it locally by executing make checks.

Hi @kailynw
Are you still working on this PR?

Sorry I am not anymore.
On Thu, Nov 5, 2020 at 6:47 AM George Aristy<EMAIL_ADDRESS>wrote:

Hi @kailynw https://github.com/kailynw
Are you still working on this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/trustbloc/agent-sdk/pull/18#issuecomment-722328271,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ22N5WDAIROUW5SXWD5U6LSOKGGNANCNFSM4TBE6TKA
.

@kailynw no worries - thanks