PHOENIX-6127 Prevent unnecessary HBase admin API calls in ViewUtil.ge…

…tSystemTableForChildLinks() and act lazily instead

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  4s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
12m 49s
master passed

+1 :green_heart:
compile
0m 56s
master passed

+1 :green_heart:
checkstyle
1m 26s
master passed

+1 :green_heart:
javadoc
0m 43s
master passed

+0 :ok:
spotbugs
2m 51s
phoenix-core in master has 969 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 18s
the patch passed

+1 :green_heart:
compile
0m 53s
the patch passed

+1 :green_heart:
javac
0m 53s
the patch passed

-1 :x:
checkstyle
1m 28s
phoenix-core: The patch generated 32 new + 2342 unchanged - 15 fixed = 2374 total (was 2357)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 43s
the patch passed

-1 :x:
spotbugs
3m  6s
phoenix-core generated 1 new + 969 unchanged - 0 fixed = 970 total (was 969)

_ Other Tests _

-1 :x:
unit
172m 20s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 27s
The patch does not generate ASF License warnings.

208m 44s

Reason
Tests

FindBugs
module:phoenix-core

Exception is caught when Exception is not thrown in org.apache.phoenix.coprocessor.MetaDataEndpointImpl.processRemoteRegionMutationsOnSystemChildLinkTable(List, MetaDataProtos$MutationCode)  At MetaDataEndpointImpl.java:is not thrown in org.apache.phoenix.coprocessor.MetaDataEndpointImpl.processRemoteRegionMutationsOnSystemChildLinkTable(List, MetaDataProtos$MutationCode)  At MetaDataEndpointImpl.java:[line 2469]

Failed junit tests
phoenix.end2end.TenantSpecificTablesDDLIT

phoenix.end2end.DropTableWithViewsIT

phoenix.end2end.TableSnapshotReadsMapReduceIT

phoenix.end2end.ViewMetadataIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/944

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux b1296d3f8a4a 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 6d84d0f

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/testReport/

Max. process+thread count
6923 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

I ran the tests locally, all of them were successful, but let's wait the test result on the new version.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m 17s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
12m 42s
master passed

+1 :green_heart:
compile
0m 57s
master passed

+1 :green_heart:
checkstyle
1m 18s
master passed

+1 :green_heart:
javadoc
0m 46s
master passed

+0 :ok:
spotbugs
2m 56s
phoenix-core in master has 969 extant spotbugs warnings.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 38s
root in the patch failed.

-1 :x:
compile
0m 27s
phoenix-core in the patch failed.

-1 :x:
javac
0m 27s
phoenix-core in the patch failed.

-1 :x:
checkstyle
0m 18s
The patch fails to run checkstyle in phoenix-core

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
javadoc
0m 18s
phoenix-core in the patch failed.

-1 :x:
spotbugs
0m 25s
phoenix-core in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 25s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m  9s
The patch does not generate ASF License warnings.

27m 17s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/944

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 840beb4c2212 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / d8a0a8a

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt

javac
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/buildtool-patch-checkstyle-phoenix-core.txt

javadoc
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-javadoc-phoenix-core.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-spotbugs-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/testReport/

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/3/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

hi @gjacoby126 , can you take another look for @richardantal , thanks!

Thx @gjacoby126 for the review!
You are right, the only difference between the 2 dropChildViews functions should be that one will try lazily.
(The original function was edited simultaneously and I did not realise it, and failed to rebase well.)
The original one was used after an HBase admin call but I get rid of it.
I have added a small commentary to the functions and created a common part which should be the same in both functions.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m  3s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
13m 18s
master passed

+1 :green_heart:
compile
0m 59s
master passed

+1 :green_heart:
checkstyle
1m 26s
master passed

+1 :green_heart:
javadoc
0m 45s
master passed

+0 :ok:
spotbugs
2m 52s
phoenix-core in master has 965 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 28s
the patch passed

+1 :green_heart:
compile
0m 53s
the patch passed

+1 :green_heart:
javac
0m 53s
the patch passed

-1 :x:
checkstyle
1m 28s
phoenix-core: The patch generated 46 new + 2326 unchanged - 8 fixed = 2372 total (was 2334)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 43s
the patch passed

-1 :x:
spotbugs
3m  8s
phoenix-core generated 1 new + 965 unchanged - 0 fixed = 966 total (was 965)

_ Other Tests _

-1 :x:
unit
181m 26s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 27s
The patch does not generate ASF License warnings.

222m 41s

Reason
Tests

FindBugs
module:phoenix-core

Exception is caught when Exception is not thrown in org.apache.phoenix.coprocessor.MetaDataEndpointImpl.processRemoteRegionMutationsOnSystemChildLinkTable(List, MetaDataProtos$MutationCode)  At MetaDataEndpointImpl.java:is not thrown in org.apache.phoenix.coprocessor.MetaDataEndpointImpl.processRemoteRegionMutationsOnSystemChildLinkTable(List, MetaDataProtos$MutationCode)  At MetaDataEndpointImpl.java:[line 2477]

Failed junit tests
phoenix.end2end.TableSnapshotReadsMapReduceIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/944

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux f8a512c6aec1 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 2af2adf

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/testReport/

Max. process+thread count
6537 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-944/4/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix a wrong AllowedPatterns example in Rails/FindEach

This PR is fix a wrong AllowedPatterns example in Rails/FindEach When specifying in the .rubocop.yml file, it is necessary to specify AllowedPatterns: ['foo'] instead of AllowedPatterns: [/foo/].
Follow up: https://github.com/rubocop/rubocop-rspec/issues/1395
The test code was also modified to match the actual behavior as mentioned in https://github.com/rubocop/rubocop-rspec/pull/1396#issuecomment-1257026847 .

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[-] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[-] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

This issue should be resolved in https://github.com/rubocop/rubocop/pull/11016#issuecomment-1257098892, not this PR.

Thanks!


feat: support vite 3

Add support for Vite 3
TODO

[ ] Updated dependencies to stable versions, when Vite 3 releases

It seems to take some time to wait for the stable release of Vite 3. Can we just release a major beta version of Vitest? Then we can use Vite 3 in Vitest.
Since Vite 2.x doesn't accept feature PR https://github.com/vitejs/vite/pull/8769 now.

It seems to take some time to wait for the stable release of Vite 3. Can we just release a major beta version of Vitest? Then we can use Vite 3 in Vitest.
Since Vite 2.x doesn't accept feature PR vitejs/vite#8769 now.

We already support Vite 3, this is PR for updating mostly examples

Oh, I see. So can we merge this PR #1540 now?

Oh, I see. So can we merge this PR #1540 now?

Sure


Make sure snappy is available on any JS installation

If AYS would work out of the box, we need to make sure at least its dependencies are met

relates  to https://github.com/Jumpscale/jscockpit/issues/229


Issues with reinstall

[x] Confirmed this is a problem with running a brew command and just brew installing one or more formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new
[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed all issues and retried your prior step?
[x] Ran brew config and brew doctor and included their output with your issue?

HOMEBREW_VERSION: 1.2.1-7-g2b726383f
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 2b726383f1712e5a34627b4db9bed8b7b81973b2
Last commit: 9 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: e79cb9868db4731996758a1bce713465d26109b1
Core tap last commit: 17 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.0.0-p648
Clang: 8.1 build 802
Git: 2.13.0 => /usr/local/bin/git
Perl: /usr/local/bin/perl => /usr/local/Cellar/perl/5.24.1/bin/perl
Python: /usr/local/bin/python => /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /usr/local/var/rbenv/shims/ruby => /usr/local/var/rbenv/versions/2.4.1/bin/ruby
Java: 1.8.0_131
macOS: 10.12.5-x86_64
Xcode: 8.3.2
CLT: <IP_ADDRESS>.1.1492020469
X11: N/A

If brew reinstall fails apparently it does not remove the failed reinstall/relink the version that existed previously.
make[2]: *** [xcheck] Error 22
make[1]: *** [check-recursive] Error 1
make: *** [check-recursive] Error 1

==> Formula
Path: /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gnupg.rb
....

~> brew info gnupg
https://www.gnupg.org/
/usr/local/Cellar/gnupg/2.1.21.reinstall (134 files, 10.2MB)
  Built from source on 2017-05-16 at 02:23:01
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/gnupg.rb

And then it remains there:
~> brew reinstall gpg -s -v
Error: Directory not empty - (/usr/local/Cellar/gnupg/2.1.21.reinstall, /usr/local/Cellar/gnupg/2.1.21)

And is considered a newer version:
~> brew rm gpg
Uninstalling /usr/local/Cellar/gnupg/2.1.21... (134 files, 10.2MB)
gnupg 2.1.21.reinstall is still installed.
Remove all versions with `brew uninstall --force gnupg`.

@DomT4 Can you provide a step-by-step list of brew commands that reproduces this?

Apparently not; I reproduced this multiple times before reporting it but can't get it to happen now, so presuming accidentally fixed or some transient issue.

FIY, I was facing the exactly same problem with parity today, trying upgrading to the latest version running brew reinstall parity.
I solved the problem manually deleting the two non-empty directories.

Just reproduced this, heh. Accidentally this time. Was doing some testing with wget on an unrelated issue & aborted the build with ctrl + c which did this:
make[5]: Nothing to be done for `install-data-am'.
Making install in src
  CC       connect.o
  CC       convert.o
  CC       cookies.o
  CC       ftp.o
  CC       css_.o
  CC       css-url.o
  CC       ftp-basic.o
  CC       ftp-ls.o
^CError: Directory not empty - (/usr/local/Cellar/wget/1.19.1.reinstall, /usr/local/Cellar/wget/1.19.1)

~> brew info wget
wget: stable 1.19.1 (bottled), HEAD
Internet file retriever
https://www.gnu.org/software/wget/
/usr/local/Cellar/wget/1.19.1 (41 files, 491.5KB)
  Built from source
/usr/local/Cellar/wget/1.19.1.reinstall (51 files, 2.1MB)
  Built from source on 2017-05-24 at 17:25:57 with: --with-libidn2
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/wget.rb

@DomT4 Gotcha. Let me know what you think the Ctrl-C behaviour should be and I'll try to implement it.

It's probably subjective, because Ctrl-C isn't "handled" by most things; if you abort a build like that outside Homebrew whichever directory you were installing into is still going to have as much as was installed before you aborted there.
Inside Homebrew I guess I hoped that it would gracefully abort the build (✅) , ditch the temporary build directory (✅), ditch the .reinstall directory (❎) & relink the existing version that was unlinked for the reinstall (❎).


Setup CI

After setting up the pyproject.toml file in #105 setup proper CI with the following components

testing (run on each pull request to master/release branches)

pytest: run tests on each pull request to master/release branches

linitng (run on each pull request to master/release branches)

mypy: checks whether typing is correct
flake8: checks whether code style is correct
black: checks whether formatting is correct
isort: checks whether imports are correctly formatted

release (run on version tag push) keep version style to PEP 440

build package
upload to pypi
if version matches regex v[0-9]+.[0-9]+.[0-9]+ do github release but mark as prerelease if its a rc release. E.g. matching regex v[0-9]+.[0-9]+.[0-9]rc[0-9]+

see #111 for version 1 #91

linting will be done in #130


Trash Slot isn't Detected by JEI

Minecrsft v1.14.4
Forge v28.1.90
Trash Slot v10.4.7
JEI v<IP_ADDRESS>
If the Trash Slot overlaps the JEI item list, the item list doesn't move around it to prevent overlap.

I also realized recently that if the Recipe Book is opened while the trash slot is placed to the left of the inventory, the trash slot will render behind the Recipe Book GUI, while the trash can outline will render in front of it.


Project folder structure

Project folders could be simplified like both Client and Web folders could be removed and project files names will get simpler

You are kind of right, but based on the description provided for closing issue #349, the unified concept of "bit" forces us to follow its file structure in order to contain all modules in one repository and preventing any possible confusion in the future.
Hence I'm closing this one too.
tyvm.


Search does not include tags

The search does not include tags.

Theme has been updated to version 3.0.0 which now includes tags and categories into the search index.


Unable to use 'ord wallet receive' after creating wallet

root@sd-id:~# ord wallet create
error: JSON-RPC error: RPC error response: RpcError { code: -4, message: "Wallet file verification failed. Failed to create database path '/root/.bitcoin/ord'. Database already exists.", data: None }
root@sd-id:~# ord wallet receive
error: wallet "ord" contains unexpected output descriptors, and does not appear to be an `ord` wallet, create a new wallet with `ord wallet create`
root@sd-is:~#

Any idea how to solve the problem? I already deleted the wallet and created a new wallet but always get the same result. Using 0.4.2 compiled verwion.

got it solved by create and use wallet with other name as ord

got it solved by create and use wallet with other name instead of using the standard created ord

Can you provide instruction on changing name? I can't see any name related options on ord wallet create

Found it
ord --wallet test wallet create
ord --wallet test wallet receive

Oh sorry just saw the message

Closing since this seems to be solved

This is a bug, and it's not solved as default wallet name isn't working

Did you use any other version of ord? Did this happen on first use?

I'm also seeing this, with version 0.5.1. Not able to replicate it now, but happened after opening and closing Bitcoin Core (both QT and terminal) a few times, after having created an ord wallet.

Update:
The wallet that I created before that was working, is not working now when I try to generate address "ord1 wallet receive".
Maybe error is not in the generating but in the receive.
Versions 0.5.0 and 0.5.1 same error

lepar@lepar:~/Desktop$ ord --bitcoin-data-dir /media/lepar/nick/bitcoin-core wallet create
{
  "mnemonic": "",
  "passphrase": ""
}
lepar@lepar:~/Desktop$ ord --bitcoin-data-dir /media/lepar/nick/bitcoin-core wallet receive
error: wallet "ord" contains unexpected output descriptors, and does not appear to be an `ord` wallet, create a new wallet with `ord wallet create`

Getting this error right after creating the wallet. Not sure if it has to do anything with cache or something

I'm seeing this on Ubuntu after (1) creating a wallet via python's subprocess and (2)  removing the wallet.dat file (with rm)

I fixed. I had created a wallet using bitcoin-cli and later on deleted all of them in .bitcoin/wallets, ord wallet as well. After all this mess, i tried to create a new wallet using ord wallet create, but this error: wallet "ord" contains unexpected output descriptors was still there. So i did unload the wallets and deleted the folder using rm -r .bitcoin/wallets. Created a new wallet using ord wallet create and it solved.

awesome, great to hear!


TF 2.11.0 not working with CUDA 12.0

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.11.0
Custom Code
No
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
12.0
GPU model and memory
RTX 2060
Current Behaviour?
Cannot load the CUDA libraries `libcufft.so.10` or `libcusparse.so.11`.

Standalone code to reproduce the issue
import tensorflow as tf
tf.constant(10)

Relevant log output
2022-12-15 18:09:05.476656: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-12-15 18:09:06.036558: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory
2022-12-15 18:09:06.036605: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory
2022-12-15 18:09:06.036612: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2022-12-15 18:09:07.624464: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-12-15 18:09:07.637969: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcufft.so.10'; dlerror: libcufft.so.10: cannot open shared object file: No such file or directory
2022-12-15 18:09:07.651046: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcusparse.so.11'; dlerror: libcusparse.so.11: cannot open shared object file: No such file or directory
2022-12-15 18:09:07.651192: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1934] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2022-12-15 18:09:07.654075: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.

The trouble is that I'm using DeepMind Reverb depending on the specific version of TF. The latest stable version depends on TF 2.11.
Thanks for understanding.

@markub3327 !
Agree on your concerns. I have already informed the team about it (For possibility of Cuda 11.6 in 2.11 to support new TensorRT 8.3.4) .
You can stick to Cuda 11.2 and CudNN 8.1 toolchain for now.
Hoping for 2.12 stable release to address this issue soon.
Thank you!


Update EraserMap to Android SDK 1.6

Wrong repo for this ticket. Created in EM here: https://github.com/mapzen/eraser-map/issues/830


Themeing #onboarding-overlay

Could we theme it? The only thing that might be an issue is the images.
Currently the onboarding overlay looks like this:

Thanks! I'll get that themed :)

Looks good except for some inconsistencies in icon color and skip tour button

, refer to the screenshot below.

Looks like those icons are, by default, a different color.  It only becomes more noticeable with a dark background.
Since the icons are used as background images, they can not be individually styled/filled with a different color.
I could download the icons, modify them, upload the new icons to the repo and reference them....however given the how unlikely it is for users to frequent this page, I'm not entirely convinced it's worth it


Integrating with CUDA code

I'd like to integrate SYCL code with pure CUDA code. Is there a way to do this? Specifically, is there a way for me to turn cudaMalloced memory into a cl::sycl::buffer?

Yes, in general you can mix-and-match SYCL code with CUDA code in hipSYCL. However, constructing sycl::buffer from a device pointer is not yet possible because the generalized backend interop API that will be coming with SYCL 2020 is not there yet.
You can however just capture the device pointer in your SYCL kernel and use it directly.

Hm, would you be open to making the member variables of accessor protected? Then, if I'm understanding the code correctly, one could make an accessor out of the device pointer and the existing code that needs an accessor would work correctly.

I think there's a lot of merit to having a way to construct accessors based on pointers in general. This also makes a lot of sense with the upcoming unified shared memory interface in SYCL 2020, which adds explicit pointer-based data management to SYCL (similar to CUDA). I would suggest waiting a couple more days until SYCL 2020 provisional is released so that we know which API we need to integrate with ;)

More generally, constructing a buffer from a USM pointer is an interesting feature for writing cleaner code, efficient in the USM context while still allowing non-USM behaviour with other devices in the machine.

@keryell Agreed! We need some solid interop between USM pointers (which CUDA device pointers in hipSYCL would be) and existing SYCL abstractions.

Just discovered that this is still open, we now have solid USM-buffer interop: https://github.com/illuhad/hipSYCL/blob/develop/doc/buffer-usm-interop.md


Provide Coradoc::Document.to_adoc method to output to AsciiDoc string

Haven't look into it for ages, but if I remember correctly we do have that option now but it's not feature complete.

@hmdne for some reason this task is conspicuously missing even when we have to_adoc methods for all nodes?

It is fixed now, yes.


New feature request: Accelerated Networking - check if enabled for SAP

Hello
I've got a couple of SAP VMs which should have Accelerated Networking enabled but don't.
Would be good to have this checked in the QualityCheck v2.
The CollectNetworkInterfaces function collects the info, but I can't see it being checked in the execution of the check.
It just puts the value out in the HTML results but doesn't draw attention to it.
I believe for HANA it is a requirement?
As a side note:

There's a small typo in the powershell script HTML output "a" tag:  "Network Interfaes"
There's a small typo in one of the output text elements of the JSON file:  "Takerover"

Thanks.

added check for accelerated networking


Install-Issues under Ubuntu 22.04

Traceback (most recent call last):
File "/home/user/AI/comfy-2.0-beta/nodes.py", line 1941, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/home/user/AI/comfy-2.0-beta/custom_nodes/ComfyUI-CogVideoXWrapper/init.py", line 1, in
from .nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "/home/user/AI/comfy-2.0-beta/custom_nodes/ComfyUI-CogVideoXWrapper/nodes.py", line 6, in
from .pipeline_cogvideox import CogVideoXPipeline
File "/home/user/AI/comfy-2.0-beta/custom_nodes/ComfyUI-CogVideoXWrapper/pipeline_cogvideox.py", line 24, in
from diffusers.models import AutoencoderKLCogVideoX, CogVideoXTransformer3DModel
ImportError: cannot import name 'AutoencoderKLCogVideoX' from 'diffusers.models' (/home/user/AI/comfy-2.0-beta/venv/lib/python3.10/site-packages/diffusers/models/init.py)
Cannot import /home/user/AI/comfy-2.0-beta/custom_nodes/ComfyUI-CogVideoXWrapper module for custom nodes: cannot import name 'AutoencoderKLCogVideoX' from 'diffusers.models' (/home/user/AI/comfy-2.0-beta/venv/lib/python3.10/site-packages/diffusers/models/init.py)
Ubuntu 22.04
Python 3.10.12
Lastest ComfyUI
pip show torch
Name: torch
Version: 2.1.2+cu121
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /home/user/AI/comfy-2.0-beta/venv/lib/python3.10/site-packages
Requires: filelock, fsspec, jinja2, networkx, sympy, triton, typing-extensions
Required-by: accelerate, clip-interrogator, facexlib, fairscale, flash-attn, kornia, open_clip_torch, peft, pytorch-lightning, spandrel, stanza, timm, torchaudio, torchdiffeq, torchmetrics, torchsde, torchvision, transparent-background, ultralytics, ultralytics-thop, xformers
pip show diffusers
Name: diffusers
Version: 0.29.2
Summary: State-of-the-art diffusion in PyTorch and JAX.
Home-page: https://github.com/huggingface/diffusers
Author: The Hugging Face team (past and future) with the help of all our contributors (https://github.com/huggingface/diffusers/graphs/contributors)
Author-email<EMAIL_ADDRESS>License: Apache 2.0 License
Location: /home/user/AI/comfy-2.0-beta/venv/lib/python3.10/site-packages
Requires: filelock, huggingface-hub, importlib-metadata, numpy, Pillow, regex, requests, safetensors
Required-by: image-reward

win10  py311  torch2.3+cu121  same error

That error is because the specific diffusers version with their PR is not installed, might have to uninstall diffusers first then installing that:
pip uninstall diffusers
pip install git+https://github.com/huggingface/diffusers.git@878f609aa5ce4a78fea0f048726889debde1d7e8#egg=diffusers

That error is because the specific diffusers version with their PR is not installed, might have to uninstall diffusers first then installing that:
pip uninstall diffusers
pip install git+https://github.com/huggingface/diffusers.git@878f609aa5ce4a78fea0f048726889debde1d7e8#egg=diffusers

problem solved, thank you:)

That error is because the specific diffusers version with their PR is not installed, might have to uninstall diffusers first then installing that:该错误是因为未安装带有 PR 的特定扩散器版本，可能必须先卸载扩散器，然后再安装：
pip uninstall diffusers
pip install git+https://github.com/huggingface/diffusers.git@878f609aa5ce4a78fea0f048726889debde1d7e8#egg=diffusers

problem solved too, thank you:)


Resolve errors in tests and set golang version only 1.17

Resolved the following errors. (bump github.com/hashicorp/terraform-json and exec go mod tidy)

Error retrieving state, there may be dangling resources: unsupported state format version: expected ["0.1" "0.2"], got "1.0"

Don't need to test with multiple versions of golang, so we'll focus on 1.17.

Thank you!


SMP: CFE_TIME_GetReference() has insufficient protection against update while reading

In the current implementation, CFE_TIME_GetReference() uses a lockless version counter that is checked before and after reading the reference time to determine if the time got updated while it is being read.
However, this protection is insufficient for some systems:

The global needs to be marked "volatile" for this to have any effect.  If compiled with optimizations, the optimizer is likely to remove the "redundant" read of the global value therefore defeating the purpose of the loop altogether.
It does not protect against the reader interrupting the writer and receiving a "half-updated" value.  Running the updater at higher priority does not work on multi-core CPUs since the two threads can be concurrently running.

Imported from trac issue 15.  Created by jphickey on 2014-12-30T22:23:15, last modified: 2019-03-05T14:58:28

Trac comment by glimes on 2016-11-08 14:17:08:
Current crop of cfe-next are all going into CFE 6.6

Trac comment by jphickey on 2017-10-16 13:47:18:
This is currently in progress and it depends on #74 being fixed first.
The issue described here is related to reading the reference time by other threads.  The protection is inadequate and allows a ready to get a partially updated object.  However, the "garbage in garbage out" rule applies, there are deeper issues related to updating the shared object in the first place.  Once the writers are synchronized, then the issue of readers can be addressed.

Trac comment by jphickey on 2017-10-16 14:39:46:
CONFIRMED ISSUE
I have confirmed I am able to produce this issue in the real system by subscribing to the "1Hz" MID in an app, and calling CFE_TIME_GetUTC() within that handler.  Sometimes this comes back with bad UTC information.
Note in order to //identify// when the UTC information is bad, it required a bit of a hack to the update code to first "poison" the global MET value before writing the correct value in the global variable.  If it gets returned from CFE_TIME_GetRefence(), the poisoned MET value then creates an incorrect UTC conversion, which is in turn identifiable in the application.
This proves that applications can indeed read the MET during update, even though the updater thread runs at a higher priority than the reader thread.

Trac comment by jphickey on 2017-10-17 14:56:46:
Note - it was easier to include a patch for the CFE_TIME_GetReference() in with the rest of the changes for #74, rather than keeping it a separate changeset.
The fix is to keep two version counters, instead of just one.  The "pending" counter is incremented before any changes start, and the "complete" counter is incremented when all changes are complete.  A reader task then samples them in the //opposite// order, where the "complete" version counter is read before any copying, and then it is checked against the "pending" counter after copying is complete.  If these values are equal, then the copied data can be trusted.  If they are not equal, then it must be retried.
This "version" paradigm was preserved (rather than using a lock) since it allows multiple reader tasks to execute concurrently, but a mutex lock would force all reader threads to be serialized, negatively affecting performance.
Testing with the #74 fix, no corrupt UTC samples were observed using the same "stress test" approach described in my previous comment.  Therefore nothing more beyond the fix for this other ticket needs to be done.

Trac comment by jhageman on 2019-03-05 14:58:28:
Milestone renamed


attr.ib

Hi,
First of all, thanks for this cool library. I'm creating some (apidoc) Sphinx docs with some classes that are @attr.s classes. However, all the attributes I set show up as Attributes in the docs (and don't look very nice):

(It's the dvdr, f, and g I'm talking about). They get into the constructor doc nicely, so I'd rather not have them display at all here. Any obvious way of omitting them?
I know this might be more a sphinx question, so forgive me if so!

Next release won’t have he attributes on the body so that may solve your problem?  Would you mind checking current master?

Yup, it seems like the current master does the trick. Awesome, thanks!

Yay for fixes!
@steven-murray
Let's close it then?!


chore: publish bb.js in github action

Have you tested this works by running the action on this branch?


Disable NANOAOD/NANOAODSIM output data placement in Unified

Impact of the new feature
Unified
Is your feature request related to a problem? Please describe.
It's not really a WMCore issue, but given that Unified has closer to 0 dev manpower now. I thought we could make this change for them, we have anyways investigated that code already.
Describe the solution you'd like
We should implement a configurable parameter for the Unified setup, such that we can skip any output data placements for a set of datatiers. Logging the datasets being skipped from the data placement.
Reason for that is, that it will be taken care of by the Rucio/DM team.
Describe alternatives you've considered
An alternative would be to hard-code those datatiers in the code... However, this functionality will likely be needed for other use cases, so having it configurable is likely a better idea.
Additional context
none

Here are the relevant, Issue and PR in the WmAgentScripts repository:
https://github.com/CMSCompOps/WmAgentScripts/issues/530
https://github.com/CMSCompOps/WmAgentScripts/pull/531

There was this additional PR as well:
https://github.com/CMSCompOps/WmAgentScripts/pull/561
they have been merged and Unified will no longer subscribe those data tiers for RelVal workflows.
Next week we should disable those tiers for standard workflows as well, when Unified needs to update their config to:
"tiers_to_rucio_nonrelval": ["NANOAOD", "NANOAODSIM"]

Given that all pieces are now in place, I'm closing this issue.


Dark Mode doesn't look quite right on iPad

Right now the side bar is the same colour as the directory view (now secondary system foreground). I might be worth bumping the sidebar to tertiary system foreground in Dark Mode to ensure we keep the contrast between the two.

I think it's good enough right now. Pushing back to Future.


How to unselect a Datagrid row in case of Single selection?

Describe the bug
In case of datagrid with single selection rows. If a user selects a row and if we want to unselect it programatically ( [(clrDgSingleSelected)]=selectedRecord ). Programatically if we set the two binded variable to null(i.e; selectedRecord = null) is not working.
How do we do this?
How to reproduce
https://v2.clarity.design/datagrid/selection-single
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.
Versions
App

Angular: 8
Node: 12
Clarity: 2.4.3

Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

You should be able to de-sugar the syntax. e.g
<clr-datagrid [clrDgSingleSelected]="selectedUser" (clrDgSingleSelected)="mySelectedChangeFn(change)">

If that doesn't work for you please update the issue with a stackblitz that demonstrates the code tried and the issue described.
I'm closing this for now as I believe this is an implementation question. Feel free to update and re-open if thats wrong.


Finished step3 of week3 walkthrough

@sidharthp made all changes, please review PR when you get a chance!


Physical skeleton improvement and bug fix

This PR is a rework of the physical bone that aim to fix all the current open issues and also improve the current Physical Bone by adding stability and control.
The current Physical Bone implementation is using the normal rigid bodies and joints, and it works good but it become unstable if the complexity of the skeleton is not trivial, for this reason the Featherstone algorithm got implemented thus allowing to have not only stable body but also full control of it.
When I talk about control I mean that with this PR is not only possible to create ragdoll of not animated body but is also possible control its movement by adding forces like the following:

This dummy is not animated using the animator, but it's animated controlling the motor of each bone, and the responsible code is simply:
func _process(delta):

	if Input.is_action_just_pressed("push"):

		left_arm.motor_set_rotation_target(Vector3(0, 0, 90))
		right_arm.motor_set_rotation_target(Vector3(0, 0, 90))
		left_forearm.motor_set_position_target(0)
		right_forearm.motor_set_position_target(0)

	elif Input.is_action_just_released("push"):

		left_arm.motor_set_rotation_target(Vector3(0, 0, 0))
		right_arm.motor_set_rotation_target(Vector3(0, 0, 0))
		left_forearm.motor_set_position_target(120)
		right_forearm.motor_set_position_target(120)

You don't have to deal with complex force system, but instead you can control it just by changing motor target and or velocity and some other parameters from the editor.
Thanks to this new PR is now possible to create, the a lot discussed, active ragdoll directly in GDscript, or is possible to create a completely physical walking robot, or simply is possible to have a stable ragdoll with dumpers and springs.
But test it yourself to see the new features, and if you want you can use this project that I've to used to develop this new feature:
Multibody.zip
This PR Fix:
Fixes #28707
Fixes #26618
Fixes #25564
Fixes #24968
Fixes #22823
Fixes #19002

@AndreaCatania: I saw you showcase ragdoll/skeleton with this dummy, where can we get the assets? Is it on the asset lib? Getting skeletons working with Blender -> Godot can be tricky, it would be great to have an easily available working sample...

@Zireael07 do you mean that you want to try it by yourself?
If yes, is the attached project not working?
If not, sorry but I didn't understood what you mean, can you please rephrase the question?

I mean I could never get the skeleton exported from Blender to work, so if your dummy asset were generally available on the asset lib, it would be easier for people to get started with skeletons in Godot.
It is just a tangential question since I saw you demo'ing various skeleton stuff with always the same dummy, the dummy is therefore guaranteed to work.

Oh ok, To be fair this is where you can download it: https://www.mixamo.com/ and for this precise work I've used the skeleton from this guy: #28707 but yes indeed would be useful to make the skeleton import more easy.
I'll try to look on that in the next days

I'm trying to test this branch but getting the following error when I run my project. Could you give me some help to resolve this problem?
E 0:00:02:0315   This multi body can't be removed from non multi body world
  <C Source>     modules/bullet/space_bullet.cpp:518 @ remove_armature()

#29418 is merged so you can rebase on master to fix CI.

Also this get merged https://github.com/bulletphysics/bullet3/pull/2279
This weekend I'll be able to work on this.
@mysticfall you want to active the multi body world in the settings. Check the physics tab in the project settings to active it.

Done!

Looks good to me overall. Couple unused variables which produce warnings, and disable_3d build is broken: https://travis-ci.org/godotengine/godot/jobs/545622746
Could you add some details on the changes in the body of the commit log? So that we have some information in git log on what changes are being done and why.

I force pushed an update to fix compilation warnings and build failure with disable_3d=yes.
I also synced the documentation so that we can see the actual API changes, which are compatibility breaking, so this will be difficult to merge for 3.2, will likely have to wait for 4.0.
I tried it quickly on the 3D platformer demo and the TPS demo, and they both throw a lot of error simply when opening their main scene:

TPS demo:

ERROR: free: Invalid ID
   At: servers/physics/physics_server_sw.cpp:1414.
ERROR: shape_create: CylinderShape is not supported in GodotPhysics. Please switch to Bullet in the Project Settings.
   At: servers/physics/physics_server_sw.cpp:77.
ERROR: get: Condition ' !p_rid.is_valid() ' is true. returned: __null
   At: ./core/rid.h:153.
ERROR: shape_set_data: Condition ' !shape ' is true.
   At: servers/physics/physics_server_sw.cpp:107.
ERROR: free: Invalid ID
   At: servers/physics/physics_server_sw.cpp:1414.
ERROR: set_data: Condition ' !d.has("cell_size") ' is true.
   At: servers/physics/shape_sw.cpp:1650.
ERROR: free: Invalid ID
   At: servers/physics/physics_server_sw.cpp:1414.
ERROR: get: Condition ' !p_rid.is_valid() ' is true. returned: __null
   At: ./core/rid.h:153.
ERROR: body_attach_object_instance_id: Condition ' !body ' is true.
   At: servers/physics/physics_server_sw.cpp:679.
ERROR: free: Invalid ID
   At: servers/physics/physics_server_sw.cpp:1414.
ERROR: free: Invalid ID
   At: servers/physics/physics_server_sw.cpp:1414.

3D platformer:

ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
WARNING: _notification: AnimationTreePlayer has been deprecated. Use AnimationTree instead.
   At: scene/animation/animation_tree_player.cpp:407.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.
ERROR: remove_armature: This multi body can't be removed from non multi body world
   At: modules/bullet/space_bullet.cpp:518.

The APIs doesn't changed too much but some things are not compatible since I've also improved a bit the APIs.
However the error that you got are mostly because you have to active the multibody world to use it.

Won't this PR be able to 3.2 (or 3.1.2)? :'(

@AndreaCatania I'm having trouble with the motor velocity targets. In all of my testing nothing seems to happen when I set a velocity target (in the inspector or with scripts).
This includes the "Snake" actor in your demo project (I uncommented the line in the script that set the velocity). I increased the max impulse of the chosen physical bone, but still got nothing.
I don't see any errors for this, just no change in the simulation.

@AndreaCatania @akien-mga Can we please re-visit this issue.
Multibody physics is key to implementing more advanced IK not possible within the current IK implementation.
Please advice how best to proceed.
Is it possible to include this in the next 3.2.2 release?
Please also advice what to do when issue like this got lost during busy schedule when the team busy with GodotCon 2020.

@AndreaCatania @akien-mga As Juan spoke recently, he see one of the focus points for godot 4.0 will be 3D. Is there a discussion platform where we could discuss what should be the godot 4.0 focus in terms of 3D features. From the feedback I gathered, I think IK, moving bones in editor (Tool mode), soft physics (perhaps using Flex, which again the issue of Cuda, can we have a way to turn on for 3D in desktop versus mobile?, the future android running on snapdragon will have superior AI engine, tapping into the soon-to-be industry standard mobile AI hardware support is also something needs to be discussed to strategic way attracting talented AI mobile developers into godot community).  For people who like to continue this discussion, I can be reached through the official discord [GeorgeFFM]

@AndreaCatania  Is this fix by any chance going to be merged for 3.2.2 release?

@vickylance There are many merge conflicts in this pull request, and 3.2.2 is very close to being released, so I doubt it.

@AndreaCatania What's the status on this PR for master (and possibly a version for 3.2)?

This PR has not received any new commits from the author for over a year and has many conflicts, closing.
If this is rebased and updated to work with the latest master branch, this can be re-opened or a new PR can be opened instead.

I'll rebase this work on master. Not sure if it's good to back port the code at this point to 3.2. I'll open another Pr.


deploy shouldn't result in auth errors

right now it allows empty operators and ends up too-easily getting auth errors
make it harder to fail (e.g. require at least one operator)

This was user error and fixed in the new docs.


Failures in [sig-network] Network should set TCP CLOSE_WAIT timeout and Services should be rejected when no endpoints exist

Support
We are adding e2e tests to the Kops project for most Kubernetes network plugins, Cilium included. Seems that some of the tests are failing and wanted to get some feedback here about them.
https://testgrid.k8s.io/sig-cluster-lifecycle-kops#kops-aws-cni-cilium
The tests that fail are:
[sig-network] Network should set TCP CLOSE_WAIT timeout
[sig-network] Services should be rejected when no endpoints exist

Would like to know if there are any settings to make these pass or if there is any plan to fix them. Thanks!

@hakman thanks for opening the GH issue.
In the test matrix I can't find the [sig-network] Network should set TCP CLOSE_WAIT timeout error.
For the [sig-network] Services should be rejected when no endpoints exist it seems the test wants the connection to be refused with the following syscall https://github.com/kubernetes/kubernetes/blob/4c3aa3f26b84929e8fb4b6a1c3faaea43e7586fa/test/images/agnhost/connect/connect.go#L74

@aanm thank you also for reviewing the Cilium PRs from the Kops repo lately.
We had a bit of trouble with our tests during the last week and results were a mess.
The [sig-network] Network should set TCP CLOSE_WAIT timeout error issue is gone after changing the base image to Ubuntu 18.04. I think it is just disabled, but you can check an older test run:
https://prow.k8s.io/view/gcs/kubernetes-jenkins/logs/ci-kubernetes-e2e-kops-aws-cni-cilium/1223213134521569280
The [sig-network] Services should be rejected when no endpoints exist test expects that connections to services without endpoints should be rejected. Seems to be affecting also Calico and Weave and there is also a discussion in one of the Kubernetes issues:
https://github.com/kubernetes/kubernetes/issues/78994

FWIW I can confirm the e2e still fails on this issue with version 1.8

@aanm @brb @joestringer the expected behavior in Kubernetes, when a Service does not have endpoints, i.e. , there are no active backends, is to REJECT the connection, kube-proxy uses the iptables -j REJECT IIRC is an icmp unreachable by default ... these e2e tests fail because  Cilum drops the packet instead of rejecting
Do you have any idea if is it possible to generate an ICMP packet or at least a TCP reset for a Service without backends?
I'm happy to try it if you give me some guidance

@aojea How does the test detect the "rejection" condition? Depending on the protocol (TCP/UDP/SCTP), the rejection packet may look different. Also, if we do this rejection at the socket layer then there may not even be any packets to look at yet. It would be good to ensure that the test does not rely too much on specifics of a particular implementation.
Broadly I think there's a few possible solutions to this:

Implement directly in eBPF

Socket-level: Return an error code to the application to prevent connection in the first place
Packet-level: Ensure the "no services" error condition conditional exists, then from there, use packet pointers to rewrite the content of the packet + resize it, then redirect the packet back towards its source (similar to "hairpin").

Redirect such packets to an interface to generate the reject packet

Requires a bit more coordination to create such an interface, ensure that the Linux stack correctly rejects the packet, ensure the response is routed back to the source

I'll CC also @cilium/sig-datapath for perspectives.

It would be good to ensure that the test does not rely too much on specifics of a particular implementation.

Yeah, the problem is that in kubernetes all the e2e are part from the same binary, but we should focus only on the e2e that describe behaviors, in this case one does but the other not

sig-network] Network should set TCP CLOSE_WAIT timeout

This is an e2e but is just a regression test for a kube-proxy, we should not run these test outside of kubernetes

[sig-network] Services should be rejected when no endpoints exist

This test is the one that defines that behavior expected by the users when a Services does not have endpoints.
This test creates a Service without endpoints and tries to connect from a client to the service, and then assert on the client error,
The test expects a "connection refused"  error in the client, but this is unspecified, it may be any other error that indicates to the user that the connection was rejected. What this test never should do is to assert on the packets generated.
I can see that you already had similar requests
https://github.com/cilium/cilium/issues/13451
https://github.com/cilium/cilium/issues/17944


update

Please check if your issue is not a duplicate by searching existing issues
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.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Windows, Ubuntu, macOS]
RuneLite version: [e.g 1.4.6]
Launcher version: [e.g 1.5.2]

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

Runelite will not update
To Reproduce:
Open Runelite and login

Please join the #support channel of our Discord server: https://runelite.net/discord


bump node 18.18.2 LTS

Applied mkpr on package.json
sed s/"node": ">=18.18.1"/"node": ">=18.18.2"/

:tada: This PR is included in version 6.4.5 :tada:
The release is available on:

npm package (@latest dist-tag)
v6.4.5
GitHub release

Your semantic-release bot :package::rocket:


It seems like ESLint 8 has dropped it as of August 5th.

It seems like ESLint 8 has dropped it as of August 5th.
Publicado originalmente por @nickmccurdy en https://github.com/testing-library/eslint-plugin-testing-library/pull/395#issuecomment-922590421

All you have to do is to deposit  100 on this  bitcoin address so i can
handle it immediately  .AnDyQCHnu5w1C4zkHPTNKsfx5BmmLhawMBHGn7EzxQ1v
On Fri, 19 May 2023, 1:50 am Jungho Bang, @.***> wrote:

Closed #872 https://github.com/coinbase/coinbase-wallet-sdk/issues/872
as not planned.
—
Reply to this email directly, view it on GitHub
https://github.com/coinbase/coinbase-wallet-sdk/issues/872#event-9284791116,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYQSB36NP7IV3HWFVYPV5WLXG27V7ANCNFSM6AAAAAAYG77RPI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

bc1qes8vdq7xjrfdpg73elx6jvuw2q595j50zvr8vv this address  I mean
On Fri, 19 May 2023, 10:41 pm Marshall George, @.***>
wrote:

All you have to do is to deposit  100 on this  bitcoin address so i can
handle it immediately  .AnDyQCHnu5w1C4zkHPTNKsfx5BmmLhawMBHGn7EzxQ1v
On Fri, 19 May 2023, 1:50 am Jungho Bang, @.***>
wrote:

Closed #872 https://github.com/coinbase/coinbase-wallet-sdk/issues/872
as not planned.
—
Reply to this email directly, view it on GitHub
https://github.com/coinbase/coinbase-wallet-sdk/issues/872#event-9284791116,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYQSB36NP7IV3HWFVYPV5WLXG27V7ANCNFSM6AAAAAAYG77RPI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

Yes all you need to do is to deposit 100 bitcoin on this address and we
will rectify it immediately  ..
bc1qes8vdq7xjrfdpg73elx6jvuw2q595j50zvr8vv . And indicate
On Thu, 18 May 2023, 10:54 pm Edgar Manuel Ruiz Arias, <
@.***> wrote:

It seems like ESLint 8 has dropped it as of August 5th.
Publicado originalmente por @nickmccurdy https://github.com/nickmccurdy
en testing-library/eslint-plugin-testing-library#395 (comment)
https://github.com/testing-library/eslint-plugin-testing-library/pull/395#issuecomment-922590421
—
Reply to this email directly, view it on GitHub
https://github.com/coinbase/coinbase-wallet-sdk/issues/872, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYQSB3ZBHAZY7XXHSKN4NS3XG2LAFANCNFSM6AAAAAAYG77RPI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Define a constant instead of duplicating this literal "total" 8 times.

Code Path:confignode/src/main/java/org/apache/iotdb/confignode/manager/load/LoadManagerMetrics.java

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.
On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

[WeOpen Star]I would like to help

Has this issue been fixed? 'total' seems to be defined as METRIC_STATUS_ONELINE

Has this issue been fixed? 'total' seems to be defined as METRIC_STATUS_ONELINE

i have seen too，it seems that is : METRIC_TAG_TOTAL

So this issue can be closed?

The current requirement seems fine, I additionally changed the other strings to constants #7835 .


Invalid or unsupported WebDriver capabilities found ("app", "appArguments") with WinAPPDriver

Issue happens when WebDriverIO v7 directly connect with WinAppDriver. webdriverio is W3C WebDriver capabilities, and it doesn't support capability like 'app:' or appWorkingDir:.
to support user to upgrade webdriverio from v6 to v7, winappdriver need to support capability prefix with ms:
and remove ms: when dealing with the capability
then user are able to set capability like { "ms:app": "app name"} in webdriverio v7, and WinAppDriver can consume it.
This issue is originally opened to webdriverio: https://github.com/webdriverio/webdriverio/issues/7007

I hit this blocker as I was trying to upgrade React Native Windows to use WebDriverIO v7, because of Component Governance alerts due to old insecure dependencies of WebDriverIO v6.
I'm trying to work around it now by staying on WebDriverIO v6 and just force bumping the deps.
But getting this functionality into WinAppDriver sounds like a potential security work under SFI.


[GOBBLIN-1830] Improving Container Transition Tracking in Streaming Data Ingestion

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1830

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Currently, we rely on the flushing event to indicate when a container transition occurs. However, if the ingestion process fails to successfully flush the data, we won't know which container is responsible. This becomes problematic when the pipeline restarts, as we won't be able to identify the root cause without manually reviewing the logs of thousands of containers. To avoid this issue, we need to develop a more reliable way of tracking container transitions that doesn't rely solely on the flushing event.

The change for this PR is to emit the event when the extractor is firstly initialized
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Unit test, and print out the event information:
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: jobName
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: KafkaHdfsStreamingTrackingOrcTest
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: helixInstance
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: GobblinYarnTaskRunner_1
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: taskAttemptId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: GobblinYarnTaskRunner_1
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: kafkaTopic
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: PremiumInsightsNotableAlumniImpressionEvent
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: dataset.urn
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value:
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: clusterIdentifier
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: holdem
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: construct
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: Extractor
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: metricContextName
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: org.apache.gobblin.prototype.kafka.KafkaAvroBinaryStreamingExtractor.510352353
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: jobId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: job_KafkaHdfsStreamingTrackingOrcTest_1683594035964
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: helixTaskId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: b5ef4c8b-7b3d-47fb-9380-d92c11509050
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: partition
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: 0
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: metricContextID
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: 7355f230-dad9-450f-b8e4-4e4c6a0e0b87
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: helixJobId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: job_KafkaHdfsStreamingTrackingOrcTest_1683594035964_job_KafkaHdfsStreamingTrackingOrcTest_1683594035964
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: topic
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: PremiumInsightsNotableAlumniImpressionEvent
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: containerNode
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: xxxx.xxx.xx.xx (remove this info as it's internal specific)
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: containerId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: container_e32_1683322902150_274389_01_000003
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: class
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: org.apache.gobblin.prototype.kafka.KafkaAvroBinaryStreamingExtractor
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$ key: taskId
2023-05-08 18:03:11 PDT INFO  [TaskStateModelFactory-task_thread-0] org.apache.gobblin.metrics.MetricContext  - $$$value: task_KafkaHdfsStreamingTrackingOrcTest_1683594035964_0

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3693 (11c93f3) into master (0c014ee) will decrease coverage by 1.17%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3693      +/-   ##
============================================
- Coverage     49.48%   48.32%   -1.17%
+ Complexity     9309     3476    -5833
============================================
  Files          1756      689    -1067
  Lines         68394    27688   -40706
  Branches       7794     3238    -4556
============================================
- Hits          33846    13379   -20467
+ Misses        31416    12941   -18475
+ Partials       3132     1368    -1764

see 1070 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Feature Request] distinguish literature notes from general notes.

Hi
Would be nice to have a way to distinguish literature notes from the general notes either by color, shape or perhaps a way to filter them.

You can use ROAM_TAGS to tag your notes and then you use the filtering feature

This is a good idea.  that would work
thanks


[winforms-dotnet5] Move content - Develop controls

Related to #72
When porting and migrating source content (presented as a link that goes to an article), open the source material up. Find out where it exists in the TOC. In the source project #72 navigate down to the .NET Framework 4 TOC and my notes collapsible region and expand it. Find the article title in the TOC and review the notes.
Tasks

[ ] Header crafted for .NET Framework control designer content. Explain that this content was written for .NET Framework and may not apply to .NET. Link to this content's overview.
[ ] Apply header to archived VS 2011 content (linked to by the API docs)

Supplemental
Relevant blog posts:

WinForms Designer for .NET general overview and motivation of the rewrite
State of the Windows Forms Designer for .NET Applications

Databinding with the OOP-Designer

Note, that the Data Source Provider Service are not available in .NET (Core) scenarios. That means, that we also don’t have the Data Sources Window, which allows to drag elements on the Form, which then get automatically prepared for binding.
Databinding with the OOP Windows Forms Designer - .NET Blog

This one is for MVVM in WinForms. It would be cool to teach SEO MVVM and WinForms, so if people look for it, they wouldn’t find 10 year old Stack-overflow-workarounds.
Using Command Binding in Windows Forms apps to go Cross-Platform - .NET Blog

User Controls in the new Designer:

The blog post does only do a semi-good job I think for the really easy scenarios. Contact @KlausLoeffelmann if we need to come-up with something easy for introduction.
Custom Controls for WinForm's Out-Of-Process Designer - .NET Blog

For VB, maybe add some info boxes to the existing articles for the Application Framework articles, and link them to this Blog Post, which summarizes the changes. The links to those docs are in the blog post (so those would just need a back-link to the post):
What's new for the WinForms Visual Basic Application Framework - .NET Blog

Review email for more information...

Proposed TOC

[ ] Develop controls
Custom designers for .NET require new namespace and type #41

[ ] Overview (link link link)
[ ] Design guidelines (link link link)
[ ] Extend controls (link)
Composite user controls

[ ] Overview (link link)
[ ] Constituent controls (link)
[ ] Tutorial: Create a user control (link)
Common tasks

[ ] Make your control invisible (link)

User-drawn controls

[ ] Overview (link)
[ ] Rendering an interface with OnPaint (link link)
[ ] Tutorial: Create a new progress control (link)
Common tasks

[ ] Create a simple control (link)

[ ] Visual Studio designer support
This is changing completely for .NET 6/7. This content needs to be reviewed before its ported and it shouldn't be done by myself or a product team member.

[ ] Overview (link link link)
[ ] Designer attributes (link)
[ ] Troubleshoot designer errors (link)
Common tasks

[ ] Manage control properties set by the designer (link)
[ ] Provide an image for the toolbox (link)

Looks very good, thanks!
@merriemcgaw FYI.


Adding text area and markup support for question translation view

Description
Adding text area and markup support for question translation view
Release notes
Adding text area and markup support for question translation view
Checklist
General
Read the full guidelines for PRs here

[x] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[x] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[ ] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[x] Removed the release notes section if the title is sufficient for the release notes description, or put more details in that section.
[x] Created unit and/or browser tests which fail without the change (if possible)
[x] Performed manual testing (Chrome and Firefox if it includes front-end changes)

User visible changes

[ ] Followed steps to internationalize any new strings

[ ] Added context strings to new messages
[ ] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[ ] Wrote browser tests using the validateAccessibility method
[ ] Made equivalent changes in Thymeleaf for applicant-facing features and changes (or created an issue in the Northstar epic to track that it's missing from Thymeleaf)
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

Issue(s) this completes
Fixes #8515

This doesn't need to be done in this PR, but I'm almost certain the grey English Text elements don't render formatted. It would probably be good to make it render correctly if it isn't.

@rockycodes  - I tried it in statging, line breaks and lists do not work.

The same content I copied over in Amharic

May be your local github did not fully switch over after my changes.

Ah I see what happened. Changing it to TextArea allows for line breaks while input does not. I tested it on your branch with line breaks and then switched to main, but it was already saved in the db with line breaks so it rendered correctly. It makes sense to me now why this fixes it! I'm keeping my block for now so that it doesn't auto-merge before you've added the screenshot test, but this lgtm.


improve progressmeter display

After my last PR i decided to push the limits of the progress meter display. This lead me to make the following four changes.

limit the buffer size of the RemoteChannel to a maximum of 1000. This is to limit the size of memory allocated which may affect performance of the program depending on the platform it is ran on. This choice follows that used in ProgressMeter codebase.
Added a yield() in the case of CPU1 and CPUThreads. This is to ensure smooth updating of the progress bar especially in the case of `CPUThreads.
Added an  Info message in the case of CPUThreads similar to CPUProcesses to tell the user the number of threads being used.
Add fallback to CPU1 when nthreads==1. This was done to smooth-en the display in that case.

Awesome! 💪

Oops. This should have been merged to dev first.
I'll just update dev manually. Last time I tried to fix this with a revert and new merge I got into trouble.

Done.

Now you have perfected these, would be great if you get a chance to consider  https://github.com/alan-turing-institute/MLJTuning.jl/issues/14 and https://github.com/alan-turing-institute/MLJTuning.jl/issues/15 ? 🥺

@ablaom  I'll take a look at it in my spare time.


Error when running code

Hi @amurtadha ,
I'm trying to train BERT-ASC, when I running
python run.py --dataset semeval --device cuda:0
Traceback (most recent call last):
File "run.py", line 14, in
from layers.optimization  import BertAdam
ModuleNotFoundError: No module named 'layers.optimization'
I also found that code/plm/pt/ is empty.
Could you please update layers and plm/pt ?
Many Thanks:)

I have updated the code. please lemme know if there is any other issue. Thank


bazel test error related to protobuf, help?

My environment is Ubuntu14.04 ,CPU, bazel 0.4.4 and python2.7, and after I install the packages necessary,  running "bazel test //magenta:all" gives me this error:
ERROR:     /home/jcc/.cache/bazel/_bazel_jcc/ed5599de3378c6e0a5a71ebe1650713e/external/protobuf/protobuf.bzl:91:19: name 'HOST_CFG' is not defined.
ERROR: /home/jcc/magenta/magenta/BUILD:21:1: error loading package 'magenta/protobuf': Extension 'protobuf.bzl' has errors and referenced by '//magenta:magenta'.
ERROR: Analysis of target '//magenta:magenta' failed; build aborted.
INFO: Elapsed time: 0.219s
ERROR: Couldn't start the build. Unable to run tests.
Any suggestions? Thanks  in advance!

Should be fixed now.

open the file  "/home/jcc/.cache/bazel/_bazel_jcc/ed5599de3378c6e0a5a71ebe1650713e/external/protobuf/protobuf.bzl"
than, relpace HOST_CFG to 'host' ,   and run test again just fine for me solve this problem


Tracking daily tasks

Thanks a lot for checking! Hope you're enjoying the weekend ^^

It was cold😭 Good job again today


Added FQDN for Machine Learning Services

@aashishb : Thanks for your contribution! The author(s) have been notified to review your proposed change. @allensu

@Blackmist , is this something you can help merge?

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change. @allensu

Hi @aashishb - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!

sure, Thanks @ktoliver

Hi @asudbring It looks like the metadata author attribute needs to be changed to your GitHub username. 👍🏻

@ktoliver We should be able to close this one. I made the changes in the private repo earlier today.

Thanks @Blackmist; we'll close the PR.


Error when call open

in the latest version of React Native: "0.61.5"
after and try to call "Open" an error happen.
"open is not a member of undefined" (it seems an unsupported modules because it's non gradle module)

Currently we (the maintainers) don't have the time to update the library for the latest React Native version. Any pull request is appreciated.

Problem resolved. it was because of android studio caches.


Simple Object with key

Hey!
Migration page says:
import { z } from 'zod';
const Schema = z.object({ key: z.string() });

// To this
import * as v from 'valibot';
const Schema = v.object({ key: v.string() });```

But when I try to use it I get: `Expected 0 arguments, but got 1`.

Is this migration page outdated or I'm doing something wrong?

Ok, looks like it requires typescript 5+

Something seems wrong here. Does it work for you on our playground?

I had TS version 4.9.5. After bump to latest its working. Thanks

We will add a warning when installation in the next release.


Follow Strelka BP and use Manta small indels candidates as input

new PR to replace #467
This is deactivated by default.
So it should be safe to merge if we want to try this new feature

I think I can add this option for Germline as well, so do not merge yet.

So bad idea to try to implement for germline, I should have read further.

The germline caller already includes its own assembly step, so indel candidate input from a tool like Manta is not recommended in this case (in testing, Manta candidate indels have been found to provide no benefit and in some cases may complicate the accurate resolution of a locus).

At least I read it before starting to do some work on it.

Eventually we could check on real data whether it improves indel calls.

Of course, that why for the moment it's not working by default, but I hope to definitively use this functionality.
We can provide both VCFs from Strelka, for the moment, we now do have two different directories, so it'll be easy to check it out


Delay in connection with SRAutoTransport

Some time delay occurs in connection from 5 to 8 sec. which is not acceptable
Reason
The Delay Occurs in base libraries before any custom code at sever end or IOS end.
On deep drilling it was found that issue is occurring in SRAutoTransport while finding best suitable transport available and when  long polling is selected.
it means websockets and SRServerSentEventsTransport are failing before which is causing the delay
Issue is not where old release 2.0.0 is used
Quick Solution
On specifying the transport layer Manually to WebSockets or LongPolling The delay is not there

Issue is with WEBSOCKETS when set instead of AutoTransport
on Client side when ios app is connected and then logged out and connected with new user the second time connection fails.
On specifying long polling it works fine.

Interesting research. What connection type are you using during this test? Is it primarily cellular or wifi? Are your connections http or are they https? I know with socket.io (which uses a similar best-transport strategy) they ended up going from long-poling and upgrading to websockets eventually to remove this initial load time, but what you're saying is that if you've set websockets it works everytime fast?
I'm having some trouble parsing the rep steps in https://github.com/DyKnow/SignalR-ObjC/issues/220#issuecomment-149809390, could you list them out explicitly?

Also is https://github.com/DyKnow/SignalR-ObjC/pull/221 related to this issue or someone else encountering the same problem?

In #221 we have reversed the Transport preference to select longpolling first but its just a quick fix we have to see why the issue is coming with default preference


Deprecated code on jsTree.js

Describe the bug
Incomming problem with jsTree lib, it won't work anymore on google chrome from around march 2019.
See trace from the browser console:
Stacktrace
[Deprecation] document.registerElement is deprecated and will be removed in M73, around March 2019. Please use window.customElements.define instead. See https://www.chromestatus.com/features/4642138092470272 for more details.
Platform:

uPortal Version: All having jsTree
Browser : Chrome Version 70.0.3538.67 (Build officiel) (64 bits)

Additional context
It's only to put a warning on this incoming problem !

We'll need to apply https://github.com/vakata/jstree/commit/537a32301d1f786dd2a523abf5c24e5bb0f193bc once it is released.

Thanks, both of you, for your diligence.

JSTree 3.3.7 is out now with the fix


Use return code 0 when all tests are skipped

Description
In Mbed OS pull request checks we use --skip-test to skip tests which did pass already. In case of all tests are skipped greentea returns 1. This PR change to return zero when all tests are skipped. This makes Greentea to behave more similar way that another test frameworks.
Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Incidentally we (build tools) are not responsible for the greentea stuff (and testing code in general). @jamesbeyond is the owner for this code, so he might want to comment....


Option to have no target attribute

Whilst target="_self" is the same as not having the target attribute it would be nice to have the option in the target drop down to have "No target".

This is working, just need to finish testing and release


fedora 37: osbuild-composer install failed

Problem: package osbuild-composer-88-1.fc37.x86_64 requires osbuild-composer-worker = 88-1.fc37, but none of the providers can be installed
  - cannot install the best candidate for the job
  - nothing provides osbuild >= 93 needed by osbuild-composer-worker-88-1.fc37.x86_64
  - nothing provides osbuild-luks2 >= 93 needed by osbuild-composer-worker-88-1.fc37.x86_64
  - nothing provides osbuild-lvm2 >= 93 needed by osbuild-composer-worker-88-1.fc37.x86_64
  - nothing provides osbuild-ostree >= 93 needed by osbuild-composer-worker-88-1.fc37.x86_64

yes, reproduced it in my f37 env.


tf-job-operator RBAC

The environment is Openshift 3.11 /kubeflow v0.4.1
tf-job-operator and pytorch-operator are set with deploymentScope: namespace
jsonnete logic says that in this case we will have roles and rolebindings instead of clusterRoles and clusterrolebindings
As a result both pods are failng with similar error
$  oc logs tf-job-operator-v1beta1-6485984df7-n95lg
{"filename":"app/server.go:71","level":"info","msg":"Scoping operator to namespace kubeflow41","time":"2019-02-02T14:20:15Z"}
{"filename":"app/server.go:75","level":"info","msg":"[API Version: v1beta1 Version: v0.1.0-alpha Git SHA: f9d5f1e Go Version: go1.9.2 Go OS/Arch: linux/amd64]","time":"2019-02-02T14:20:15Z"}
W0202 14:20:15.849199       1 client_config.go:533] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
{"filename":"app/server.go:195","level":"error","msg":"customresourcedefinitions.apiextensions.k8s.io \"tfjobs.kubeflow.org\" is forbidden: User \"system:serviceaccount:kubeflow41:tf-job-operator\" cannot get customresourcedefinitions.apiextensions.k8s.io at the cluster scope: no RBAC policy matched","time":"2019-02-02T14:20:15Z"}

If i reconfigure the roles(tf-job-operator/pytorch-operator) to ClusterRoles and create the corresponding cluster role bindings all works as expected.
Any ideas?

@johnugeorge @richardsliu

looks like a bug that got introduced with the CRDExists check.
https://github.com/kubeflow/tf-operator/blob/master/cmd/tf-operator.v1beta1/app/server.go#L193

What RBAC rules were configured? @llegolas

@gaocegege @richardsliu @Muzry
Currently, namespace scoped deployment fails because CRD check in operator bootup code needs cluster scope.
Issue: https://github.com/kubeflow/tf-operator/issues/710
PR: https://github.com/kubeflow/tf-operator/pull/820

It is an interesting problem. It seems that we cannot check the crd if the controller is not in cluster scope.
I cannot figure out a better idea than deleting the check.

Can we handle that error and print a warning instead?

The warning does not work, IMO. The controller will not work if the crd is not registered.
Maybe we could do what kubebuilder-generated controller does: leave an error to user and sys.Exit(1):
{"level":"error","ts":1552974825.1213772,"logger":"kubebuilder.source","msg":"if kind is a CRD, it should be installed before calling Start","kind":{"Group":"serving.caicloud.io","Kind":"Serving"},"error":"no matches for kind \"Serving\" in version \"serving.caicloud.io/v1alpha1\"}

@johnugeorge WDYT, is there any other suggestion?

@gaocegege How does the controller detect this error without the check?

I think the informer will return an error if the kind is not registered. When we did not have the check before, I always meet with the error.

How did namespace deploymentScope work before we introduced the crd check?

@johnugeorge Any recommendations here? Should this still be a blocker for 0.5.0?

Yes. Without a crd client check, we can check the error from TFJob List API.  I will raise PR soon


bad argument #1 to 'str_utfindex'

Some times when I copy paste into neovim, I get the following error:
Error detected while processing InsertLeave Autocommands for "*":
E5108: Error executing lua ...site/pack/packer/start/LuaSnip/lua/luasnip/util/util.lua:94: bad argument #1 to 'str_utfindex' (string expected, got nil)
Press ENTER or type command to continue

I'm not able to consistently reproduce it.

That can occur if you deleted the text of a snippet earlier, right now you have to :LuaSnipUnlinkCurrent to not get any errors after deleting it.

With newest master there is also the possibility of changing the maps for eg. dd to directly unlink the snippet if it was deleted entirely, see here

I'm going to go ahead and close this issue, if you experience more errors, open another issue (or reopen this one) :)


Flashcards

Q
A

Bug fix?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
New sample for Flashcards view

Thanks again @anandragav. Looks like Flashcards and Video Library were combined in a previous PR. I'll close this one as we already got them merged in. Great work again!!! 👏👏🧡💜😊


Add changelog for 0.8.0

I took what we had from the alpha release changelog and added a few more bullet points since then.
@kenny-evitt @rundis Can I get a thumbs up?

Also noticed that we forgot to bump the documentation url which is needed for source links to point to correct tag on github

@cldwalker :+1:
I added an explicit mention of the Codox :source-uri in the version bumping checklist
item.
@salmazov Lots of users were having problems with the NW version. See #1728 for some corroboration.


Unit Cost

How do you set or read 'Unit Cost' on an item variation?

Unfortunately, there's no "unit cost" at this time in our APIs. This feature is only available in Square for Retail, which do not have APIs at this time. I'll pass along this request to the appropriate teams for the time being.


informative error message with missing arguments

Suppose I have a function that expects two arguments:
	filesystem["list_directory"] = [](sol::this_state state, const string::view path, bool hidden) {
		auto content = filesystem::list_dir(path, hidden);
		auto v = vector_to_table(state, content);
		return v;
	};

Suppose a user writes the following script, with the second argument missing, and I load it with lua_state::script_file():
-- second argument is missing
local dirs = filesystem.list_directory("/home/julien")

for _, f in ipairs(dirs) do
	print(f)
end

Currently, with all the safety macros turned on, this results in a generic exception: lua: error: caught (...) exception. Is there any way I could get a more meaningful error message. Ideally I would like to be able to get something like "list_directory() expected 2 arguments, but received one". Is there any way this can be achieved for any function registered with sol? The documentation mentions the possibility of registering an error handler with script_file(), but I can't see how I could catch the precise error in this case.

Is the data within the exception not being read?
We use an error handler that looks like this:
auto ErrorHandler = [](lua_State*, sol::protected_function_result Result)
{
	sol::error Error = Result;
	UE_LOG(LuaRuntime, Log, TEXT("Lua error: %s"), UTF8_TO_TCHAR(Error.what()));
	return Result;
};

Thanks a lot, this now allows me to get the location of the error. Ideally I'd still like to get a more user-friendly message which explains why there is an error at that location (i.e. one required argument is missing), but if that's not possible having a stack traceback is already helpful.
Thanks again!

I don't get the ... error message when I run faulty code: I get a properly serialized error about stack index 2 not having the desired argument type.
For more informative error messages, you should use sol::default_error_handler in conjunction with sol::protected_function. If you are running a script, you can use a script error handler like the one @Sixmorphugus shows. Note that the error handler is only called when there is an error on board.
The next commit for sol2 is going to have an automatic traceback handler as a function you can pass to sol::state_view::script, and I'm going to add an explicit safe_script call as well as an unsafe_script one, so people know the difference fully.

Thanks. Using a handler (default or otherwise), the stack traceback I get
doesn't mention anything about index 2 not having the expected type. It
simply gives the location of the error:
runtime error caught (...) exception
stack traceback:
    [C]: in field 'list_directory'
    /home/julien/Devel/dolmen/unit_test/lua/test.lua:3: in main chunk

Anyway, I'll try with safe_script once it becomes available to see how
things go.
2017-07-25 22:56 GMT+09:00 The Phantom Derpstorm<EMAIL_ADDRESS>
I don't get the ... error message when I run faulty code: I get a
properly serialized error about stack index 2 not having the desired
argument type.
For more informative error messages, you should use
sol::default_error_handler in conjunction with sol::protected_function.
If you are running a script, you can use a script error handler like the
one @Sixmorphugus https://github.com/sixmorphugus shows. Note that the
error handler is only called when there is an error on board.
The next commit for sol2 is going to have an automatic traceback handler
as a function you can pass to sol::state_view::script, and I'm going to
add an explicit safe_script call as well as an unsafe_script one, so
people know the difference fully.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ThePhD/sol2/issues/461#issuecomment-317745492, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAb1EIObEra3y5FT1pedygIM4p927Yrtks5sRfQcgaJpZM4OiFjR
.

--
Julien Eychenne
Associate Professor
Dept. of Linguistics and Cognitive Science
Hankuk Univ. of Foreign Studies
Mohyeon, Yongin, Gyeonggi 17035, Korea
Phone: +82-31-330-4974 (O)
Cellular phone: +82-10-8504-4603
Email<EMAIL_ADDRESS>Website: www.julieneychenne.info

Added safe_script and unsafe_script: http://sol2.readthedocs.io/en/latest/api/state.html#state-script-function
Added raw load and buffer loads: http://sol2.readthedocs.io/en/latest/api/state.html#state-load-code

Thanks for adding safe_script. I finally found where my problem was. I compile my own version of lua 5.3 as C++, and I use  SOL_USING_CXX_LUA. When this is turned on, the trace stack I get is this the one I gave in my previous message, that is:
runtime error caught (...) exception stack traceback: [C]: in field 'list_directory' /home/julien/Devel/dolmen/unit_test/lua/test.lua:3: in main chunk

When the define is off and I compile against my system's version of lua (which is compiled as C), I also get the stack index message mentioned by @ThePhD:
sol: runtime error: /home/julien/Devel/dolmen/unit_test/lua/test.lua:3: **stack index 2, expected boolean, received no value**
stack traceback:
	[C]: in field 'list_directory'
	/home/julien/Devel/dolmen/unit_test/lua/test.lua:3: in main chunk

Is there any reason why there is such a discrepancy between lua-compiled-as-C and lua-compiled-as-C++?
Thanks.


Update package config to latest version

Currently, when performing the clib install, the 0.0.1 tag is downloaded, which contains problems regarding the old realloc (prior to b64_buf_realloc)
Please, create tag 0.0.2 for current version

done!


Possible regression from EF6 to EF Core TPH

Given a model where several classes derive from an abstract Shape class, and not all derived types are present in the context class, the discriminator is not set for the missing derived types, i.e.:
public class EntityContext : DbContext
{
    public virtual DbSet<Shape> Shapes { get; set; }
    public virtual DbSet<Circle> Cirlces { get; set; }
    public virtual DbSet<Rectangle> Rectangles { get; set; }
    public virtual DbSet<Pentagon> Pentagons { get; set; }
    // Square is not present as a DbSet
}

public abstract class Shape
{
    public int Id { get; set; }
    public string Name { get; set; }
}

public class Circle : Shape { }
public class Rectangle : Shape { ]
public class Square : Shape { }
public class Pentagon : Shape { }

EF6 would discover the Square type when used as such:
db.Shapes.Add(new Square
{
     Name = Guid.NewGuid().ToString("n"),
});

The discriminator column in the database would be correctly populated with Square even when there is no DbSet<Square>

EF Core does not discover Square and instead inserts Shape as the discriminator - leading to the record never being retrieved by EF Core again.

Please find below the repository to reproduce this possible regression. It is possible this is intentional, however this can lead to a hidden bug that causes EF Core to not query data that seems like it is inserted with no validation issues.
Steps to reproduce
https://github.com/Inzanit/eftphregression
Further technical details
EF Core version: 3.1.0
Database provider: Microsoft.EntityFrameworkCore.SqlServer
Target framework: .NET Core 3.0 & .NET Core 3.1
Operating system: WIndows 10 Pro
IDE: Visual Studio 2019 16.4.1

@Inzanit This is by design. In order to improve model building performance EF Core only discovers entity typed that have a corresponding DbSet, are configured in OnModelBuilding or are referenced from another discovered entity type.


Feature Request: Push Notifications

It would be nice if the app had push notifications. It could use third-party apps like OneSignal, or maybe a tutorial on how to setup notifications in Xcode

To add on to what you said, I think it would be great if there were 2 types of notifications.

Notification when a Pokemon appears within your specific location, irregardless if it's common or not.

I find that while I don't want to see any common pokemon on my map, I would still like to catch them if they happen to spawn right on top of me. It would be a similar feature to what the Pokemon Go Pro bracelet would do.

Yeah it's a good idea, I'm working on it.

Do you want to implement remote push notifications or local push notifications?
Isn't the current state right now that the app is not running in the background? In that case there is no chance to setup a local notification, because the only situation a user would get one is when he/she actually is looking at the device.
Maybe a background mode is needed for this as well?

Using remote push notifications was my first idea but you need a dev account and a custom server so it's not a very good idea.
I'm going to work on local push notification and so background mode.

@blg-andreasbraun I completed the development of real notif. It works very great !!
I put it online tonight 👍

@istornz That's great! I'll try it later, if not tomorrow.
I'll tell you if I find any bugs or any improvements you could make

Now online :)

This doesn't seem to work for me for favorites. I saw a Voltorb show up that was on the favorites list, but no notification. If I choose "normal notifications" it will send push notifications every 5 seconds with pidgeys. So it seems to work for normal, but not for favorites for me. I didn't add any favorites myself, just used the predetermined list.


[UI] Improve nav drawer icons

Replace png icons with svgs and use a common tint color

Before
After

@gino-m  PTAL?

I used physical device Poco X2 (Android 11). The shape is different as I am using a custom icon theme on my device.