Color clear

Hello!
My code
Kirki::add_field( 'pt_theme', array(
    'type'        => 'color',
    'settings'    => 'pt_header_color',
    'label'       => __( 'Background', '' ),
    'section'     => 'header_general',
    'default' => '',
     'output' =>  array(
          array(
              'element'  => '.header-inner',
              'property' => 'color',
         )
    ),
) );

When I select a color, then all is well. When I click the "Clear" button, nothing happens. How do I happened to update previews and property "color" is not excreted in the css?
Thank you

This is something I've been working on and I just pushed a commit to the develop branch.
I added an "exclude" argument.
Your code was like this:
Kirki::add_field( 'pt_theme', array(
	'type'        => 'color',
	'settings'    => 'pt_header_color',
	'label'       => __( 'Background', '' ),
	'section'     => 'header_general',
	'default'     => '',
	'output'      =>  array(
		array(
			'element'  => '.header-inner',
			'property' => 'color',
		)
	),
) );

you will have to modify your code and write it like this:
Kirki::add_field( 'pt_theme', array(
	'type'        => 'color',
	'settings'    => 'pt_header_color',
	'label'       => __( 'Background', '' ),
	'section'     => 'header_general',
	'default'     => '#FFFFFF',
	'output'      =>  array(
		array(
			'element'  => '.header-inner',
			'property' => 'color',
			'exclude'  => array(
				'#FFFFFF',
				'#DEDEDE',
			),
		),
	),
) );

Note the differences:

The default value is NOT empty, it has a color in it... say for example the default color for your header background. If you don't want to use a background by default on that, then just set the default value to rgba(255,255,255,0) which is transparent.
an exclude argument was added and it's formatted as an array of elements. In this example, the field will not output any CSS if the value is equal to #FFFFFF OR #DEDEDE.

This will be included in the next release, but if you want to test it out you can get the latest develop branch from here: https://github.com/aristath/kirki/archive/develop.zip

I'm trying to use the exclude for the spacing element, but it seems to be not working:
 Kirki::add_field( 'caramelomedia', array( 	'type'      => 'spacing', 		'settings'  => 'footer_padding_top_bottom', 		'section'   => 'footer_margin_padding', 		'label'     => esc_html__( 'Padding top & bottom', 'caramelomedia' ), 		'default'   => array( 			'top'    => '0', 			'bottom' => '0', 			), 		'transport' => 'auto', 		'output'    => array( 			array( 				'element'  => 'footer', 				'property' => 'padding', 				'exclude' => array( 					'0', 					), 				) 			), 		) );
However I get this working on a number element:
Kirki::add_field( 'caramelomedia', array( 		'type'      => 'number', 		'settings'  => 'footer_padding_top_top', 		'section'   => 'footer_margin_padding', 		'label'     => esc_html__( 'Padding top', 'caramelomedia' ), 		'default'   => '0', 		'transport' => 'auto', 		'output'    => array( 			array( 				'element'  => 'footer', 				'property' => 'padding-bottom', 				'exclude' => array( 					'0', 					), 				) 			), 		) );
Could you please help me out to exclude values from a spacing element.

The value in the exclude argument should have the same format as the control itself. So similar to the default value...
If you want to exclude top:0 & bottom: 0, then you should use it like this:
'exclude' => array(
    array(
        'top' => '0',
        'bottom' => '0',
    ),
)

I almost missed this comment... Next time please create a new ticket instead of commenting on an old unrelated & already closed ticket to make sure someone sees it. 👍

Thank you very much!!
I was thinking about that, creating a new topic but I was like I will
post it here because it had something to do with it.
In the future I will create a new ticket.
Met vriendelijke groet,
Ronald de Rooij
E-mail<EMAIL_ADDRESS><EMAIL_ADDRESS>Telefoon:<PHONE_NUMBER>
Website: www.caramelo-media.nl http://www.caramelo-media.nl
Op 23/08/2016 om 02:53 schreef Aristeides Stathopoulos:

The value in the exclude argument should have the same format as the
control itself. So similar to the default value...
If you want to exclude top:0 & bottom: 0, then you should use it like
this:
'exclude' => array(
array(
'top' => '0',
'bottom' => '0',
),
)
I almost missed this comment... Next time please create a new ticket
instead of commenting on an old unrelated & already closed ticket to
make sure someone sees it. 👍
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aristath/kirki/issues/697#issuecomment-241595823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APiIFY__wxZ_c6WiY-STxJbWIxWAhsy4ks5qikSIgaJpZM4HlPFd.


Said in the docs to add the Softmax Activation function but never did in code.

URL(s) with the issue:
https://www.tensorflow.org/tutorials/images/cnn#add_dense_layers_on_top
Description of issue (what needs changing):
In the last sentence "CIFAR has 10 output classes, so you use a final Dense layer with 10 outputs and a softmax activation."
"and a softmax activation" but I guess some one forgot to add the softmax activation at the end.
OLD CODE:
model.add(layers.Flatten())
model.add(layers.Dense(64, activation='relu'))
model.add(layers.Dense(10))

NEW CODE:
model.add(layers.Flatten())
model.add(layers.Dense(64, activation='relu'))
model.add(layers.Dense(10))
model.add(layers.Activation('softmax'))

Closing this issue since PR has been merged. Thank you.


Bugs fixes And Improvements of MBE

This PR fixed / improve followings things:

Add token $repeat separator support: Previously $repeat only support single punct separator.
Fixed a bug which expand infinite pattern, see test_match_group_in_group
Correctly handle +,*,? case of $repeat patterns
Increase the limit of $repeat patterns (128 => 65536), personally i think we could remove this limit as we seem to fix all major loop bugs
Re-enable tt matcher
Better meta item parsing.
Add related tests and add some real world test cases.
Add more debug information.

bors r+

Finally i feel confident to refactor the lexer and mbe with enough test case :)

@edwin0cheng yeah, I also feel that that's just the right approach: build the basic test-suite using simplistic solution, then refactor to the proper infra, and then the remaining 90% of fixing all of the bugs :D


Revised recruitment emails

Two email scripts to develop:

Follow up email to Summer 2020 participants
Initial recruitment to Focus Group 2020 participants

Hi Nic-- please see the drafted scripts here

I slightly tweaked the subject line - but this looks great - please go ahead and send


macOS build script fixes

@maekclena Could you check if this PR as well as #32 should be closed ?

These changes just need to be rebased, if you want I can open a PR for it. For issue #32, PR #34 implements all items on the list so it can be closed.

@maekclena thanks so much for your help on this. This will help us add itk-vtkglue packages for macOS, too. A rebase and new PR would be appreciated. :1st_place_medal:
CC: @fbudin69500

Closing. This was superseded by #35


Add an option to have predictable filename

We are running HIPS protection on our devices.
This piece of software protect us from executable (trying to) starts other application. The main issue is that the filter option isn't quite advanced and it doesn't allow us to whitelist unpredictable filename such as CodeSetup-stable-6a995c4f4cc2ced6e3237749973982e751cb0bf9.exe
So when Code.exe Start (the) new application C:\Users\User\AppData\Local\Temp\vscode-update-user-x64\CodeSetup-stable-6a995c4f4cc2ced6e3237749973982e751cb0bf9.exe we get an annoying security popup.
same when vscode-update-user-x64\CodeSetup-stable-6a995c4f4cc2ced6e3237749973982e751cb0bf9.exe Start (the) new application C:\Users\User\AppData\Local\Temp\is-BVE93.tmp\CodeSetup-stable-6a995c4f4cc2ced6e3237749973982e751cb0bf9.tmp
Thus, the request for feature: Would it be possible to have an option to use ONLY predictable filename for those cases ?

This isn't possible on the current setup infrastructure we're sitting atop of.
Additionally, filename based protection seems very brittle. What if a malicious actor calls its virus CodeSetup.exe? We sign our binaries for that exact same reason. Can you instead write your rules based on codesigning?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add freetype 2.8.1 patch

Fixes #384

@alexeykuzmin When this is merged it can be deleted in the chromium 61 branch (this patch is in chromium 61+)

@MarshallOfSound 👍

@MarshallOfSound Should it be merged? All the checks seem to be fine.

@alexeykuzmin Yeah, if someone approves all 3 PR's we can merge them in 👍

@ckerr If a patch just backports something from a newer version as is,
it will simply fail when we try to apply it to that version. It actually happens almost every Chromium and Node.js upgrade.
But if a patch is a temporary workaround, I would prefer to have an open issue to remove it when it's no longer needed.


Canary

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: heldic:white_check_mark: etoster:x: dependabot[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Functionality: Import-Dependency

It may be worthwhile to provide functionality to read a depend.psd1 file, install dependencies if they are missing (iffy on this, maybe require a -Force), and import supported dependencies (e.g. a PSGalleryModule would be easy to handle)
Used in conjunction with options that allow saving to a non-PSModulePath, this could help enable a virtual-environment-lite scenario.

+1 on this. I'd like to be able to create "portable" packages with no external dependencies. For ex., if I have a script that uses modules and functions from *.ps1 files, I'd like to build it as one package and have dependecies automatically injected into it via some kind of loader function.

Initial code is in place, but untested.  Current state:
# List dependencies, with a DependencyExists column tacked on with the test output
Get-Dependency ... | Test-Dependency

# Check if any dependencies do not exist - quiet switch returns only test result per dependency
( Get-Dependency ... | Test-Dependency -Quiet ) -Contains $False

To do:

Invoke-PSDepend should have a mechanism for gating behavior - e.g. Invoke-PSDepend -Test or Invoke-PSDepend -Import.  I'll likely leave the default behavior to include an install, allowing folks to override, i.e. [bool]$Install = $True
Testing.  Uncovered a number of bugs just adding rudimentary tests for the 'Test' and Install PSDependActions

On a related note, have a 'global options' node working locally, where you might specify a global target (that could be overridden on a dependency-by-dependency basis) - e.g.:
PSDependOptions = @{
    Target = 'C:\myproject'
}

This might help with tediousness of specifying the same target for every dependency...
Cheers!

Alrighty,  the code is in place, and an example of this scenario is documented here.
Still need more thorough testing, but closing this out...


Fix Early Return Bug in load_docs Method

Bug Details

Early return when docs is not None: The original code returned self.docs, None directly when docs was not None, which led to self.docs not being updated with the new docs parameter.
Save functionality not reached: Due to the early return, the save flag's condition was never evaluated, and the _save method was not called when expected.

Resolution

The docs parameter is now correctly assigned to self.docs, and self.timestamps is set to None when custom documents are provided. This ensures that self.docs is always up to date.
The early return has been removed to guarantee that the save condition is evaluated and the _save method is executed when save is True.

Hello Maarten,
I just found the previous pull request by TanKaiYi0220 and your comment on it. I have closed this pull request.


Fix typo in readme

You already fixed it 5 mins before me sorry.

All good, thanks for the catch! :)


Upgrade third_party/protobuf to v3.14.0 (for v1.35.x)

Same as https://github.com/grpc/grpc/pull/24855, but for the already-cut v1.35.x branch (and including third_party/protobuf commit that hopefully fixes the build problem encountered in https://github.com/grpc/grpc/pull/24855)

CC @stanley-cheung

CC @stanley-cheung

Lets wait for the tests to see if all problems encountered in https://github.com/grpc/grpc/pull/24855 have been fixed.

Lets wait for the tests to see if all problems encountered in https://github.com/grpc/grpc/pull/24855 have been fixed.

Still seeing problems:
Python artifact build on Windows
https://source.cloud.google.com/results/invocations/82b67290-2e1a-4882-b3b0-bd026e6c8814/targets/github%2Fgrpc/tests;group=tasks;test=build_artifact.python_windows_x86_Python27_32bit;row=2
third_party\protobuf\src/google/protobuf/stubs/mutex.h:119:13: error: explicitly defaulted function 'constexpr google::protobuf::internal::WrappedMutex::WrappedMutex()' cannot be declared as constexpr because the implicit declaration is not constexpr:
   constexpr WrappedMutex() = default;
             ^~~~~~~~~~~~

Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

Still seeing problems:
Python artifact build on Windows
https://source.cloud.google.com/results/invocations/82b67290-2e1a-4882-b3b0-bd026e6c8814/targets/github%2Fgrpc/tests;group=tasks;test=build_artifact.python_windows_x86_Python27_32bit;row=2
third_party\protobuf\src/google/protobuf/stubs/mutex.h:119:13: error: explicitly defaulted function 'constexpr google::protobuf::internal::WrappedMutex::WrappedMutex()' cannot be declared as constexpr because the implicit declaration is not constexpr:
   constexpr WrappedMutex() = default;
             ^~~~~~~~~~~~

Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

@gnossen  @acozzette  looks like the backport https://github.com/protocolbuffers/protobuf/pull/8115 doesn't work 100%:
windows python artifact build:
third_party\protobuf\src/google/protobuf/stubs/mutex.h:119:13: error: explicitly defaulted function 'constexpr google::protobuf::internal::WrappedMutex::WrappedMutex()' cannot be declared as constexpr because the implicit declaration is not constexpr:
   constexpr WrappedMutex() = default;
             ^~~~~~~~~~~~
In file included from C:/tools/msys64_win7/mingw32/include/c++/6.3.0/mutex:44:0,
                 from third_party\protobuf\src/google/protobuf/stubs/mutex.h:33,
                 from third_party\protobuf\src/google/protobuf/descriptor.h:65,
                 from third_party\protobuf\src/google/protobuf/compiler/scc.h:38,
                 from third_party\protobuf\src/google/protobuf/compiler/js/js_generator.h:41,
                 from third_party\protobuf\src\google\protobuf\compiler\js\js_generator.cc:31:
C:/tools/msys64_win7/mingw32/include/c++/6.3.0/bits/std_mutex.h:94:5: note: defaulted constructor calls non-constexpr 'std::mutex::mutex()'
     mutex() noexcept = default;
     ^~~~~

@gnossen  @acozzette  looks like the backport https://github.com/protocolbuffers/protobuf/pull/8115 doesn't work 100%:
windows python artifact build:
third_party\protobuf\src/google/protobuf/stubs/mutex.h:119:13: error: explicitly defaulted function 'constexpr google::protobuf::internal::WrappedMutex::WrappedMutex()' cannot be declared as constexpr because the implicit declaration is not constexpr:
   constexpr WrappedMutex() = default;
             ^~~~~~~~~~~~
In file included from C:/tools/msys64_win7/mingw32/include/c++/6.3.0/mutex:44:0,
                 from third_party\protobuf\src/google/protobuf/stubs/mutex.h:33,
                 from third_party\protobuf\src/google/protobuf/descriptor.h:65,
                 from third_party\protobuf\src/google/protobuf/compiler/scc.h:38,
                 from third_party\protobuf\src/google/protobuf/compiler/js/js_generator.h:41,
                 from third_party\protobuf\src\google\protobuf\compiler\js\js_generator.cc:31:
C:/tools/msys64_win7/mingw32/include/c++/6.3.0/bits/std_mutex.h:94:5: note: defaulted constructor calls non-constexpr 'std::mutex::mutex()'
     mutex() noexcept = default;
     ^~~~~

@gnossen @lidizheng   how can we fix the other issue?
Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

@gnossen @lidizheng   how can we fix the other issue?
Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

I think this pull request might fix the WrappedMutex error.

I think this pull request might fix the WrappedMutex error.

@gnossen @lidizheng how can we fix the other issue?
Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

@gnossen  @lidizheng  it looks like this is the last issue that's preventing us from upgrading third_party/protobuf.
I tried to fix it by upgrading rules_python to 0.1.0 but that seems to lead to other test failures (see https://github.com/grpc/grpc/pull/25143). Can you take a look and suggest how to fix this issue (a hotfix is fine)?
Since we're trying to backport this into 1.35.x branch, we don't want to any major changes, so I'd prefer to fix this in some simple way and then perform the rules_python upgrade properly on the master branch.

@gnossen @lidizheng how can we fix the other issue?
Python basic bazel tests
https://source.cloud.google.com/results/invocations/fa5b00dd-ab16-47bd-ae0b-328bbaaef574/log
FAIL: //src/python/grpcio_tests/tests_aio/interop:local_interop_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/interop/local_interop_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/interop:local_interop_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/local_interop_test.py", line 24, in <module>
    from tests_aio.interop import methods
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2698/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/interop/local_interop_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/interop/methods.py", line 29, in <module>
    from google import auth as google_auth
ImportError: cannot import name 'auth'
================================================================================
[3,626 / 3,642] 71 / 138 tests, 1 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python2; 13s processwrapper-sandbox ... (16 actions running)
FAIL: //src/python/grpcio_tests/tests_aio/status:grpc_status_test (see /root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/testlogs/src/python/grpcio_tests/tests_aio/status/grpc_status_test/test.log)
INFO: From Testing //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
==================== Test output for //src/python/grpcio_tests/tests_aio/status:grpc_status_test:
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/d2dc70c3d9da3fab488ba0dcbbd35051/sandbox/processwrapper-sandbox/2873/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/src/python/grpcio_tests/tests_aio/status/grpc_status_test.runfiles/com_github_grpc_grpc/src/python/grpcio_tests/tests_aio/status/grpc_status_test.py", line 22, in <module>
    from google.rpc import code_pb2, error_details_pb2, status_pb2
ModuleNotFoundError: No module named 'google.rpc'
================================================================================
[3,913 / 3,915] 136 / 138 tests, 2 failed; Testing //src/python/grpcio_tests/tests/unit/_cython:_channel_test.python3; 30s processwrapper-sandbox ... (2 actions running)

@gnossen  @lidizheng  it looks like this is the last issue that's preventing us from upgrading third_party/protobuf.
I tried to fix it by upgrading rules_python to 0.1.0 but that seems to lead to other test failures (see https://github.com/grpc/grpc/pull/25143). Can you take a look and suggest how to fix this issue (a hotfix is fine)?
Since we're trying to backport this into 1.35.x branch, we don't want to any major changes, so I'd prefer to fix this in some simple way and then perform the rules_python upgrade properly on the master branch.

@jtattermusch Sorry, I missed the last notification. Richard and I discussed this problem, and it can be resolved by an existing workaround in our repo: https://github.com/grpc/grpc/blob/master/src/python/grpcio_tests/tests/bazel_namespace_package_hack.py.

@jtattermusch Sorry, I missed the last notification. Richard and I discussed this problem, and it can be resolved by an existing workaround in our repo: https://github.com/grpc/grpc/blob/master/src/python/grpcio_tests/tests/bazel_namespace_package_hack.py.

And it looks like it was caused by this PR. I'm still investigating to determine the best way to resolve the issue.

And it looks like it was caused by this PR. I'm still investigating to determine the best way to resolve the issue.

So it turns out that the Envoy project dealt with this problem before us. The Protobuf team has intentionally removed their namespace package declarations from the __init__.py file, instead declaring them in their setup.py file, which we do not consume at all. It seems that this solution is only necessary until all other packages using the google namespace package (google-auth) do the same. I've implemented that solution in our repo here.

So it turns out that the Envoy project dealt with this problem before us. The Protobuf team has intentionally removed their namespace package declarations from the __init__.py file, instead declaring them in their setup.py file, which we do not consume at all. It seems that this solution is only necessary until all other packages using the google namespace package (google-auth) do the same. I've implemented that solution in our repo here.

I cherry-picked the patch from https://github.com/grpc/grpc/pull/25171/. After that, all tests should be green.

I cherry-picked the patch from https://github.com/grpc/grpc/pull/25171/. After that, all tests should be green.

The interop toprod tests are also failing on master: https://source.cloud.google.com/results/invocations/09ff4cb1-e4af-4841-a86e-3e48b2c99952/targets/github%2Fgrpc%2Finterop_test/tests

The interop toprod tests are also failing on master: https://source.cloud.google.com/results/invocations/09ff4cb1-e4af-4841-a86e-3e48b2c99952/targets/github%2Fgrpc%2Finterop_test/tests

@gnossen  @veblush @stanley-cheung I checked all the test results and they are looking good, and this PR is now ready to merge. Please review so I can merge.
After this is merged, I'll create a PR to upport this change to the master branch.

@gnossen  @veblush @stanley-cheung I checked all the test results and they are looking good, and this PR is now ready to merge. Please review so I can merge.
After this is merged, I'll create a PR to upport this change to the master branch.

LGTM. I was also following this and came to the same conclusion. Tests are looking good. Thanks for taking care of this!

That was quick! Happy presidents day!

LGTM. I was also following this and came to the same conclusion. Tests are looking good. Thanks for taking care of this!

That was quick! Happy presidents day!


result_languages.txt file is not being included in setuptools package_data

the file result_languages.txt is not being installed with the package, resulting in this print statement everytime the library is loaded:
There was an issue loading the result languages file.  Exception: [Errno 2] No such file or directory: '/Users/nippur/src/linkedin-scraper/venv/lib/python3.11/site-packages/yagooglesearch/result_languages.txt'

Thanks for submitting an issue @pguridi - give me a few days to take a look at it.


Kaitaan add overrides

@pacejackson @mattknox @spladug
I think this should be the last major experiments addition coming into baseplate for the near future.

💇♂

💇♂

💇♂


Update autosummary cross-reference syntax

This is meant to fix #39.
New versions of autosummary make it much more difficult to specify cross references.  Before, it was always correct to use fully-qualified names. Now, if the autosummary directive is contained within a class, its cross references must not start with the name of the module that that class is defined in.  In other words, the cross references must be relative, not absolute, in this case.  If the autosummary directive is not contained within a class, then fully-qualified names are still required.  This adds a bunch of annoying complexity to the code.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.67%. Comparing base (881da02) to head (654b1e4).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   98.51%   98.67%   +0.16%
==========================================
  Files           6        6
  Lines         270      303      +33
  Branches       37       28       -9
==========================================
+ Hits          266      299      +33
  Misses          1        1
  Partials        3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


error with npm install

Getting this error after running npm install.
I'm on macOS Mojave with node v12.4.0
Unhandled rejection Error: EACCES: permission denied, open '/Users/jarllyng/.npm/_cacache/index-v5/2b/cb/51f2ceaa13b8e23157f8a54d87d3ebc561e954022b8a220bffc0d8b2fed2'

npm ERR! cb() never called!

npm ERR! This is an error with npm itself. Please report this error at:
npm ERR!     <https://npm.community>

npm ERR!      /Users/jarllyng/.npm/_logs/2019-06-11T22_34_18_468Z-debug.log

This is a popular npm issue with your local npm install, not a Vue Design System issue - raised here: https://github.com/npm/npm/issues/17839

Thanks @barclayd I'll close this issue then :)


Adds a custom resolver option

Summary
This diff adds a new configuration (customResolver) that allows one to hook into jest-resolve to bypass the Node resolution while still using the Haste resolution.
Test plan
Added a tests, previous ones should still pass.

Erh, I missed the resolve option. Nevermind.


Tile layout make-over

Next funding rate information

Timestamp when it will occur
Costs

Historical:

When was the CFD opened?
When was the last funding round and at what cost?

relevant discussion #942

Closing this in favor of #244
UI changes will be done together with finishing the funding rate implementation.
The only change listed in this ticket that is not related to the funding rate is

When was the CFD opened?

which might not be necessary as discussed in #942
We might still incorporate it into the changes if deemed necessary.


[e2e] Add debugging code

In commit 6f12074, circle was configured to run the integration test repeatedly. It passed 9 times and failed on the 10th: https://app.circleci.com/jobs/github/statechannels/monorepo/10974 . One of the new errors was thrown:
frame.click(#join) failed on frame Rock Paper Scissors
This error is thrown when #join is found by page.waitOnSelector but then not found by page.click.
By clicking on the artifacts tab, we can see the screenshot saved just afterwards:

Player A is in the lobby and there is no open game to click "join" on.
It seems that either
i) the game appeared and then disappeared before it could be clicked (currently slowMo=100ms)
ii) the game never appeared but puppeteer thought that it did.


Do you plan to add request stiching support?

Hi,
Thank you for creating this node.
Do you plan to support request stiching?
It provides way better results for anything longer than a couples sentences.
Thanks!
https://elevenlabs.io/docs/api-reference/how-to-use-request-stitching

Hi @AlexfromSonar,
That is one cool suggestion.
I will dig it next week.
🥷

@AlexfromSonar
I deployed version 0.04 that supports request stitching. By default, it is not active; you can activate it in the additional fields.
I decided to implement only the first method. Let me know if it works well for your use case, and if needed, I will work on implementing the second possibility.

Thanks @n8n-ninja ,
I've tried but I can't see the additional fields related to the request stitching, what am I missing?
Here are all additional fields I have.


HTMLElementObject.click()

Script stop execute after call .click()
https://gist.github.com/3447051

https://github.com/ariya/phantomjs/issues/10795.
Seems to be closed with 2.0.0.

@istr This issue is closed on the PhantomJS side.  What docs need to be updated?

This issue is closed on the PhantomJS side. What docs need to be updated?

A hint that the lower versions supported by 1.1 have this problem -- to be honest I am currently a bit unsure where to put it best. Feel free to find a good place and put it there. Documentation updates are non-critical so will be merged if basic grammar and spelling looks ok. ;-)

This issue has been added to the docs under a new section.  Closing issue.


[Bug] breadcrumb.bottom does not have any effect (Treemap)

Version
5.4.2
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=treemap-simple&code=PYBwLglsB2AEC8sDeAoWsDOBTAThLGAXLANprrLkWxgCeIWxA5GDllgLYCGITANFQoATLmC7Ey1aqilToXDo1hNowIVgCC_QbIBuXADYBXJQEYADANnUAxgAsIBoW2gSd1mddnzFzVeo0ubS89QxNiABZ3WQBfKxD0TwT0HyUVNU0AI2Dk9H1jJQA2aKkYkvQAXRK4kqTrVL8MgCEcr3zw2AAmS3LYe0dnLFdSXsTRigblfywmoPjc9qVu-eT-pxc3XOlx7wU06dnTVq28sKXzHYoyk6rc65Db63vqCpX0TLYuIRscIw5M4h1agYOzAADuxFYJjeFEywDAYGAHGYAFZzABSJjuGrUe5VGIAbiAA
Steps to Reproduce
Put breadcrumb: {         show: true,         bottom: '50%'       }
Current Behavior
breadcrumb.bottom property does't have any effect.
Expected Behavior
The breadcrumb.bottom property should position the breadcrumb.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

@erzenhaxhi It seems the default value of breadcrumb.top is bottom, so it can't work as expected. You can try to set top to undefined. The default value for breadcrumb.top seems wrong in the documentation.


typo update default value of compaction_promotion_min_size_mbytes

Proposed changes
in 2.0 version, default value of compaction_promotion_min_size_mbytes is 128, update doc
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

docs will be updated in another repo


"File not found" when opening script files from file explorers

App Version:
2021.1.3
Affected Device(s):
Android 11
Describe the bug
No files can be opened when selecting "Open with Squircle IDE" from several external file explorers.
To Reproduce
Steps to reproduce the behavior:

Navigate to a file, tap it.
Select "open with Squircle IDE"
Error is always returned: "File not found"

Expected behavior
File will open in in the app to be edited.
I am trying to edit bash shell scripts in the termux .shortcuts folder. I can open the files with several managers (Amaze, Material Files, Ghost Commander, Files etc.) however when I tap on the file and select open with Squircle IDE. I get an error "File not found" I have tried several files, they all result in the same error.
All of these files can be opened and edited in several other editors. Only Squircle returns this error.

Hi @ioogithub, it depends on the file explorer.
Most of apps using the correct path /root/storage/emulated/0/Download,
but some apps using weird paths like this /external_files/Download.
I cannot guarantee that my app received the correct path from external file manager, but I can manually replace the /external_files/ with /storage/emulated/0/.

Same problem here as well. And it IS from the internal storage

@Whitespace-code could you give me the names of these file explorers you tried?

any idea shen fixed , because realy anoying issue thank

Yup, this issue makes Squircle mostly unusable for me, using Material Files.

Also unusable on Android 8, Galaxy A3, "File not found" for everything.

I also have this problem on my smartphone (under android 12), I use the file manager of the OS (Grapheneos) only Squircle IDE reacts like this, it is indeed quite annoying as behavior, but by passing through the file browser internal to Squircle, it happens normally, for me it does not make it unusable, but it is annoying, thank you

Can anyone check the latest update v2022.1.1??
This issue is probably fixed, I tried like 10-12 different explorers and the code editor for almost all of them worked fine


DisplayLink installation issue - Missing component: Linux headers for kernel, 3.14.0.

I have tried for trusty 14.04 and xenial 16.04 - and can even print out linux header kernel version to be 3.14.0 but no matter what I do I get:
Unsatisfied dependencies. Missing component: Linux headers for running kernel, 3.14.0. This is a fatal error, cannot install DisplayLink Linux Software.
Any ideas?
From what I can tell it doesn't seem like anyone else who isn't using chroots has this problem.

I think this issue: https://issuetracker.google.com/issues/187791868 with the chromeOS has to with the "missing" kernel headers. It looks like google is not going to fix it at all.
Thanks for using google chromeOS! :)


NamespaceInfo.Data should be a map to bytes

What changed?
The namespace data map now has bytes values rather than string.
Why?
We embed binary data (cast to strings) here in a few places, and that can cause google/protobuf to yell at us for having string data that's invalid UTF-8. Protobuf strings and bytes are wire-compatible
How did you test it?
Potential risks
This will change all code generated by protoc plugins to use the language's type for raw bytes. I have paired Go API and server PRs that will go along with this change

We've decided to axe this and migrate the relevant fields internally


Fix config.json and remove healthechk metrics

update requirements, small fixs readme, initial.sh, docker-compose.yml

Codecov Report

Merging #82 (44b7f0b) into master (70cf70d) will decrease coverage by 0.00%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
- Coverage   84.29%   84.29%   -0.01%
==========================================
  Files          69       69
  Lines        5024     5023       -1
==========================================
- Hits         4235     4234       -1
  Misses        789      789

Flag
Coverage Δ

unittests
84.29% <75.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

utils/basic/village.py
65.16% <0.00%> (ø)

tribal_wars_planer/middlewares.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


feat: add optimistic update

Pull Request type
Issue Number: #39

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Why the kernel of BCHT implemented as "tiled_find_kernel" ?

Hello,  I have found that there are two kinds find kernels:
__global__ void tiled_find_kernel(InputIt first,
                                  InputIt last,
                                  OutputIt output_begin,
                                  HashMap map)

__global__ void find_kernel(InputIt first,
                            InputIt last,
                            OutputIt output_begin,
                            HashMap map)

this Kernel find_kernel   is quit easy to understand,  but tiled_find_kernel   seams little hard to understand.   The bucketed cuckoo hash use tiled_find_kernel,    I am fused why BCHT needs tile, what's the meaning of these  code of  tiled_find_kernel ?  Could you please help me explain the meaning of it?  Thank you in advance

  auto work_queue = tile.ballot(do_op);
  while (work_queue) {
    auto cur_rank = __ffs(work_queue) - 1;
    auto cur_key = tile.shfl(find_key, cur_rank);

    typename HashMap::mapped_type find_result = map.find(cur_key, tile);

    if (tile.thread_rank() == cur_rank) {
      result = find_result;
      do_op = false;
    }
    work_queue = tile.ballot(do_op);
  }

Hi @alphaRGB,  Bucketed cuckoo hash table (BCHT) insertion and find operations are performed in a tile-wide fashion. The two main goals are 1) achieving a coalesced memory access when reading the bucket, 2) avoiding branch divergence between threads.
The code snippet that you showed builds a queue (within a tile) of find operations then performs each operation one at a time.
  auto work_queue = tile.ballot(do_op);  // build a queue bitmap of all threads that hash `do_op = true`
  while (work_queue) { // while there are items in the queue
    auto cur_rank = __ffs(work_queue) - 1; // find the first thread rank in the queue
    auto cur_key = tile.shfl(find_key, cur_rank);  // all tiled bucketed hash table operations expect the same key for all threads in  a tile
    typename HashMap::mapped_type find_result = map.find(cur_key, tile); // every thread in the tile calls find

    if (tile.thread_rank() == cur_rank) {
      result = find_result; //  thread (in the tile) result
      do_op = false; // pop operation
    }
    work_queue = tile.ballot(do_op); // rebuild the bitmap queue
  }

Here is where we read a bucket from memory and where the coalesced memory access is ( the tile size is the same as bucket_size).
The same idea is used for all bucketed hash tables. find_kernel and insert_kernel are only used by 1CHT (cuckoo hash table with a bucket size of one).  Coalescing memory accesses is one of the main reasons why BCHT is much faster than 1CHT.
Below is a picture that also explains the code snippet for insertion (same for find) (see slides 36-42 for explanation with pictures):

Let me know if you have more questions!


Newtypes for strings

I think this is a good idea but don't have a good intuition for where to do this and where not too.
I imagine there could be things like:

ModuleName
WebIDL
PureScript

and also possibly things like:

MemberName
NodeName
NodeTarget
TypeElement
TypeName
TypeFamily

but generally unsure.
I'm also not clear in which file(s) each of these should be defined.
@paf31 – what did you have in mind?

Those sound good. I would just go through the source and extract them one by one. Some is better than none, you don't have to get them all 😄

That definitely wasn't mean to be an exhaustive list.
Any advice on where to define them? It seems like at least some will need to live in a separate file to avoid circular references.


adding test for undici issue 971

See https://github.com/nodejs/undici/pull/2971

@anonrig I am adding back, under a different name, a formatting script using docker. The modified version does not work for me.

@anonrig I am adding back, under a different name, a formatting script using docker. The modified version does not work for me.

Yup, my PR seems to break it, can you replace the existing file instead of creating a new one?


Get result as ActiveRecord::Relation

Is there a way to get a query result as an AR relation rather than as an array of AR objects? All I seem to be able to do is ThingIndex.query(...).load.to_a. My strategy to get a relation has been to do something like:
ids = ThingIndex.query(...).only(:id).map(&:id)
Thing.where(id: ids)

Forgive me if I'm missing something in the documentation/code. I'm new to this gem and I've been struggling to find an answer.

I was thinking about this, bu unfortunately since it is possible o query multiple types, it will not be working in all the cases. New DSL has pluck BTW, get the latest master, it should be stable enough.


register and lost password button on login page

How can we show register and lost password buttons on the login page, is there a configuration for this?

No, there is no config for that and the default login page do not provide those buttons. However, you can add link to the registration page (/identity/account/register) and forgot password page (/identity/account/forgotpassword) on your app.
Or customize the login  view  (Views/Account/Login.cshtml)if you use the code.
I can also add it in a next release or you can propose a PR if you want to contribute.

I added these under the form tag on the page, thanks:
`
@Localizer["Forgot Password"]
@Localizer["Register"]
`


Inout can't be read with constant value

Version
Yosys 0.40+45 (git sha1 dd2195543, g++ 13.2.1 -march=x86-64 -mtune=generic -O2 -fstack-protector-strong -fPIC -Os)
On which OS did this happen?
Linux
Reproduction Steps
Here's a minimal example.
module MI_alchitryTop (
    output reg [7:0] P_led,
    inout [4:0] P_button
  );

  reg [4:0] IO_P_button;
  assign P_button = IO_P_button;

  always @* begin
    IO_P_button = 5'bzzzzz;
    P_led = P_button;
  end

endmodule

I'm targeting the iCE40HX8K-CB132
Expected Behavior
The P_button signal to be read and output on P_led.
Actual Behavior
The P_button signal is treated as a constant and the LEDs never turn on.

I don't have any experience with using inout ports or tristate logic, but having a quick play with the show command and stepping through the synth command (cough cough) it looks like any semblance of tristate logic is being dropped as soon as it reaches proc.  Looking at a few examples online for tristate logic it seems like you need to assign it with conditional logic, i.e. assign P_button = P_button ? P_button : 'bz; for it to correctly identify the tristate logic, rather than simply writing z's because then it will optimise to always read back z's (which, again you can see with show).  This is probably also why your other issue, #4371, works with IO_P_button = D_flip_q ? 5'bzzzzz : 5'h0;.

For what it's worth, this works correctly in both iCEcube2 and Vivado.
I understand this case is kind of pointless since the inout is always z but it should act just like an input at that point.

I agree, but given that read_verilog does report back Warning: Yosys has only limited support for tri-state logic at the moment. for the relevant line, I think this is more of a feature request than a bug.


feat: Add publish workflow

Internal PR for internal code review. Do not merge.

Superceded by https://github.com/cloudquery/cq-source-scaleway/pull/3


Temperature on 5700XT

I am calling rsmi_dev_temp_metric_get() with RSMI_TEMP_CURRENT
That then calls get_dev_mon_value with the following params:
type: amd::smi::kMonTemp
dv_ind: 1
sensor_ind: 1
The issue is that val_str = "" and the later stoi (line 397) causes the application to crash.
Do you know how I can modify this to support the 5700XT ?

Hi,
It sounds like the contents of the sysfs file from which the temperature is
being read may be not what is expected.
The device index may vary from run to run, but on my machine, the value
being read looks like this:
$ cat /sys/class/drm/card0/device/hwmon/hwmon0/temp1_input
30000
This is for "card0" on my machine.It may be card1, card2,... on your
machine. What do you see if you cat that (those) files?
On Wed, Nov 27, 2019 at 10:04 AM Sean<EMAIL_ADDRESS>wrote:

I am calling rsmi_dev_temp_metric_get() with RSMI_TEMP_CURRENT
That then calls get_dev_mon_value with the following params:
type: amd::smi::kMonTemp
dv_ind: 1
sensor_ind: 1
The issue is that val_str = "" and the later stoi (line 397) causes the
application to crash.
Do you know how I can modify this to support the 5700XT ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RadeonOpenCompute/rocm_smi_lib/issues/52?email_source=notifications&email_token=ADDTF2NUTJWHVGGFFX6DEEDQV2LAFA5CNFSM4JSI7ZOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H4OTRJA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDTF2LX7VT3TQ6G4XK7OLDQV2LAFANCNFSM4JSI7ZOA
.

Okay I didn't realize it used sys.
I think maybe the Linux kernel I have doesn't support this card.
temp1_input exists but when I try to cat it I get:
"cat: temp1_input: Invalid argument"
I will update my setup and try again next week. Thanks

Also, I think the main issue for me is that stoi("") leads to rocm_smi to exit. But it would be helpful if it checked for a blank string and returned an error/fail rather than exiting.

Can you cat that file if you do it as root? It's weird that it says invalid
argument. What are the permissions on that file?
On Thu, Nov 28, 2019 at 8:55 AM Sean<EMAIL_ADDRESS>wrote:

Also, I think the main issue for me is that stoi("") leads to rocm_smi to
exit. But it would be helpful if it checked for a blank string and returned
an error/fail rather than exiting.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RadeonOpenCompute/rocm_smi_lib/issues/52?email_source=notifications&email_token=ADDTF2NLZZSOAFSWCMIWZW3QV7LVTA5CNFSM4JSI7ZOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFM2OFI#issuecomment-559523605,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDTF2K4HLWLTK5A4IKWBNLQV7LVTANCNFSM4JSI7ZOA
.

--

Added a fix for empty files in here: https://github.com/ROCm/rocm_smi_lib/commit/a4b470fe71f723fe2c3b90480922820ae8102558


Picker does not respond when in Interactable.View

So I have a Picker component inside an Interactable.View (it is for a pull-down screen with customer billing information, works something like notifications pull-down area on iOS). The Picker does not respond to touches, the Interactable.View just trembles and that's all.
I have checked that if I just replace the Interactible.View with regular View, all works well.
Am I doing something wrong or is it a bug in the library?
Thanks

Most likely a bug in the library. Handling of touches is pretty complex. We can have a scroll view holding an interactable that can hold a scroll view. The mitigation is not trivial.
The naive example we currently support is this:
https://github.com/wix/react-native-interactable/blob/master/example/src/HandleTouches.js
I will add more complex touch examples, including your picker example so we can figure it out.

I added a test example in the commit above, here it is:
https://github.com/wix/react-native-interactable/blob/master/example/src/TouchesInside.js
you can run it inside the example app
just tested it on iOS and it actually works very nicely with the touches inside the interactable.view
can you please try it out?

On iOS it works well. I had some problems when resizing the container, but overall it was ok.
On Android however.. The only thing that works is the button. Everything else triggers the view movement.

The touches issues on Android will be fixed by @LoyalBlocks-tzachi next week

Hi guys! I fixed most of the Android touch issues in this branch: https://github.com/wix/react-native-interactable/tree/androidFixes
I'll appreciate if you can give feedback before I merge to master
The issue that still remain unresolved (I'm still working on it) is horizontal scroll inside horizontal moving interactable (or anything with horizontal drag like a slider)

Hi guys, this was pushed into master and published (v0.0.7). Please confirm the fix so we can close this issue

Let's do this the other way around.. I'll close and if the issue is still there please reopen


viewer can not see cursor and latest edits in onlyoffice

Describe the bug
latest edits and the mouse cursor are not shown in onlyoffice. if the file is locked onlyoffice won't show the latest version of a flie.
Steps to reproduce

admin share new.docx with marie (viewer role)
marie opens new.docx with onlyoffice
admin edits new.docx

Expected behavior
marie should be able to see the cursor and added text.

Actual behavior
marie cannot see the cursor and added text.

Setup
Please describe how you started the server and provide a list of relevant environment variables or configuration files.

OCIS=6.6.1
onlyoffice=8.2.0

Additional context
Add any other context about the problem here.
@ScharfViktor @micbar @jvillafanez @kobergj

This is a duplicate of #4570 which has been closed due to lack of progress.
@hodyroff @dj4oC Do you want to push this?

Yes. Checking the documentation this must be enabled with an additional sharing function.
https://helpcenter.onlyoffice.com/ONLYOFFICE-Editors/ONLYOFFICE-Document-Editor/HelpfulHints/CollaborativeEditing.aspx#:~:text=The Live Viewer mode is used to see,checkbox is active in the editor's Advanced Settings.
@jvillafanez @kobergj

In any case a missing feature in the integration, not a bug.

@hodyroff this is a setting in the php connector app.
Is there any resource how to achieve that via WOPI?

We have to ask Onlyoffice ... the link just explained to me that its a normal behaviour or better that there are two different viewer behaviours.


feat: fromDefinedNullable

Motivation
Add fromDefinedNullable util.
Changes

++fromDefinedNullable

Tests

/did.utils/fromDefinedNullable

Merging it so that we have it next week, hope you do not mind @mstrasinskis


Envoy-Mobile seems to systematically use HTTPS

Looks like that Envoy-Mobile, by default, always ends up sending an HTTPS request, regardless of what was requested. In other words, if the RequestHeader is set line this: new RequestHeadersBuilder("GET", "HTTP", "api.lyft.com", "ping"), Envoy-Mobile still sends an HTTPS request. Is this expected?
The configuration used is the default one provided by BaseConfiguration.Standard.

Solved by https://github.com/envoyproxy/envoy-mobile/pull/1419
Thanks, Mike!


[client-keystone-auth] Snap doesn't support local files for e.g. OS_CACERT

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug
What happened:
Running kubectl (any subcommand) with keystone integration via client-keystone-auth snap results in :
An error occurred: failed: Cannot read CA file: open /home/someuser/tls/root_ca.crt: permission denied

And corresponding apparmor messages:
audit: type=1400 audit(1595426719.726:311816): apparmor="DENIED" operation="open" profile="snap.client-keystone-auth.client-keystone-auth" name="/home/someuser/tls/root_ca.crt" pid=5660 comm="client-keystone" requested_mask="r" denied_mask="r" fsuid=3620 ouid=3620

What you expected to happen:
Kubectl should return successfully
How to reproduce it:

Install kubectl snap
Install client-auth-keystone snap
Export OS_CACERT=/home/someuser/tls/root_ca.crt
Run kubectl

Anything else we need to know?:
The client-auth-keystone snap runs in strict confinement. To be able to access OS_CACERTs in user homes it'd need to support the snap :home plug (currently only supports the :network  plug)
Environment:
client-keystone-auth  v1.14.0+git36.e163280
kubectl               1.18.6

@sabaini I don't think this is the client-keystone-auth issue, I'm not sure who is maintaining client-keystone-auth snap, but maybe you should ask help from them.

@lingxiankong The snapcraft.yaml file that defines the snap is part of this repository, this led me to believe that the snap is maintained here.

@lingxiankong The snapcraft.yaml file that defines the snap is part of this repository, this led me to believe that the snap is maintained here.

@sabaini Sorry for the confusion, as far as I know, that file has never been tested and I am not sure if there is someone actually maintain that.
I would suggest you build the binary by yourself, should be as easy as one command: GOOS=linux BUILD_CMDS="client-keystone-auth" make build. You are also welcome to fix if you want.

BTW, client-keystone-auth is also included in the release, please check https://github.com/kubernetes/cloud-provider-openstack/releases

The snapcraft.yaml file has been removed from the repo and is not going to be supported.


Multiple canvas state support + implementation

Make use of the new system to not rely on render being the last statement in the program, and allow for multiple canvases.

Currently getting more information about module contexts.

Partially implemented. Data is passed around by module context rather than relying on program evaluation result. Groundwork has been laid to support multiple render groups of multiple shapes, although renderer controls state etc is still shared. Currently, only the last render group actually gets used for rendering on the single canvas.

Refer to #35.


using i18n.js with teaspoon ?

I'm using teaspoon with qunit to test my ember.js application, which uses i18n.js for internationalization.
When I run tests I get :
   Uncaught ReferenceError: require is not defined

It is called in this file, line 6.
Any clue on this?
Thanks!

Have you loaded require.js, and read the documentation for the libraries you're using?

Jeremy Jackson

On Jan 26, 2015, at 7:24 AM, Jeremy Venezia<EMAIL_ADDRESS>wrote:
I'm using teaspoon with qunit to test my ember.js application, which uses i18n.js for internationalization.
When I run tests I get :
Uncaught ReferenceError: require is not defined
It is called in this file, line 5.
Any clue on this?
Thanks!
—
Reply to this email directly or view it on GitHub.

Yes, I know what I'm doing, and I searched for similar problems before submitting an issue.
I don't need require.js since i'm using rails witch uses sprockets to require files.
I've just realized that Qunit's module may be the problem. So this condition is true, but it should not.
So It may be a I18n.js issue, and not teaspoon...
I'm closing this issue and checking this out...

But I see a require call in the library, so I'm unsure how it works without the dependency. Sorry it's not what you want to hear, but it seems to be the problem.

Jeremy Jackson

On Jan 26, 2015, at 10:30 AM, Jeremy Venezia<EMAIL_ADDRESS>wrote:
Yes, I know what I'm doing, and I searched for similar problems before submitting an issue.
I don't need require.js since i'm using rails witch uses sprockets to require files.
I've just realized that Qunit's module may be the problem. So this condition is true, but it should not.
So It may be a I18n.js issue, and not teaspoon...
I'm closing this issue and checking this out...
—
Reply to this email directly or view it on GitHub.

Fixed here https://github.com/fnando/i18n-js/issues/294 !
The issue was from I18n.js
Thanks!


Delete CONTRIBUTING.md

old template; removing in favor of new contributing info for /swiftlang

@swift-ci test


Update pl.json

Update Polish translate

I adjusted the error messages to better handle substitutions. I hoped I'd finish before you did the translation, but it didn't work out that way.

I understand. I hope this can be resolved.

You need to pull the latest and resolve the conflict on your end. I'll then pull it into main.

Ok, I'll close this, I'll add it again with a new translation


does not work with DS.ManyArray

When you pass in a DS.ManyArray the function will not loop over it at all cause it is not aware of it's content property.

addressed by #2


How to start omnicore-qt??

in documentation you mentioned that: "./qt/omnicore-qt if built with UI". But i got below error:
./qt/omnicore-qt: No such file or directory
I am not seeing any .qt directory in omnicore src directory. Am i missing anything when installing omnicore?
I am using ubuntu 14.04.

Hello, did you follow the build instructions here?
https://github.com/OmniLayer/omnicore/blob/master/doc/build-unix.md


fix: email content for bcsc approved clients

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.64%. Comparing base (42525cc) to head (98cedac).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1239   +/-   ##
=======================================
  Coverage   71.64%   71.64%
=======================================
  Files         315      315
  Lines       10960    10960
  Branches     2550     2550
=======================================
  Hits         7852     7852
  Misses       3108     3108

Flag
Coverage Δ

unitests
71.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Feature/wjiang/exit after connected

When connected is sure, exit without continue to improve performance
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

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

Checklist:

[ ] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

@atrayano , Please verify that my understanding is correct.


added keycloak preprocessing with required keycloak info, and also keycloak configs with argocd

@yangliu138 to open-source config code to images used after de-sensitising the contents.


Clear GO Data

Everything works for me aside from the clear data apk; it clicks it but aside from that it doesnt clear the actual data and if it does then FATE GO won't launch until a full data clear. Any advice on a workaround/fix?

Are you using the modified ova that I link in the instructions?
Does it work if you click it manually. You should see a toast notification.

Currently reinstalling memu to see if it fixes things

Used modified ova from the link in the instructions; launched and then updated via play store then launched again with python and it works normally.
It will give a notification sometimes, but when it doesnt wipe it will give none and continue on to the old account. If it does wipe it gives a notification that there is no user data found and launches into the support window of selecting a team (which cannot be cleared unless you have a minimum team of 3, or doesnt let it be launched).

That's really weird. Can you clear the data manually with ES file explorer like the post says? The apk is as simple as it can be. The 'no user data' found is especially weird, since I've never seen that happen.
Does the script successfully close GO through the app switcher before attempting to clear the data?

It does clear it through the app switcher and then attempts to clear the data via the clear app. I'll try manually doing a clear if it persists with the python script after a clean install.

It did a complete cycle fine after a complete reinstall of memu. Memu may have issues or encounter them over time it appears if it is not a fresh install of it (it worked for a few cycles initially before things derped out). This can be closed as it is solved by reinstalling memu.. Script works great, thanks! Time to get a waifu filled account ;D ^^

I'm currently having the same problem ; I've tried reinstalling Memu and I'm still not able to complete a second cycle due to the 'Clear Go Data' not working. I've also noticed that once I try to complete a second cycle my Memu has left off on this screen and the PythonShell leaves this.

Same as Muttans, but to clarify things, the Clear Data button (at least for me) DOES work when I manually click it. But the script, I'm guessing due to the way Memu often slows down, does not manage to clear the data, then tries launching FGO without clearing the last roll, leading to the screen shown by Muttans above. The script also sometimes falls off on name or password entry. When this occurs, it looks like it tries to select the field but, again possibly due to slowdown, the app fails to properly register the click and then the script doesn't know to try again and fails out.

For me when I try to manually click on the Clear Data button it doesn't work. Usually what I've noticed is that the Clear Data button worked before the first cycle (It showed the clear data notification). Though once I ran my second cycle the Clear Data button breaks (It doesn't show the clear data notification).

@Djinnistorm do you have any other apps open in memu when you run the script? the current version of the script assumes that only grand order will be open.
@Muttans the failsafe try changing this line to false instead on your version and see if it helps. The memu screen is the normal relaunch screen. I do have a bit of code that should detect that and try to clear the data again but it never comes up in my runs so it was probably never tested. I'll look into that further I guess.

@chaosking121 no other apps, got memu running only for this script.

@chaosking121 I'm having the same problem it goes to the clear data and doesn't seem to click on it and goes back into fate and gets stuck at the news screen with it fails to click or it's not registering it

I too seem to be having an issue with the Clear Go Data apk.  If there isn't any save data, it seems to work on clicking (I get the toast popup at the bottom saying data has been cleared), but if there is data that needs to be cleared, clicking the clear go data app icon doesn't seem to do anything.  I tried chaning the failsafe to false like you suggested to Muttans, but i think the issue lies with the apk itself, not the script.

@tdub415 the failsafe thing is totally unrelated. The issue also can't really be with the apk because there's nothing that can go wrong really there. Well, that is, assuming you're using the modified ova. Are you?

I installed the first fgo apk in your GitHub, and updated it using the 2nd
fgo apk.  I've tried the lollipop and KitKat shells (I guess shell is the
right term) from the memu setup post you linked to in the instructions, and
having the same issue with both.
On Jul 14, 2017 6:52 PM, "Arshad Hosein"<EMAIL_ADDRESS>wrote:

@tdub415 https://github.com/tdub415 the failsafe thing is totally
unrelated. The issue also can't really be with the apk because there's
nothing that can go wrong really there. Well, that is, assuming you're
using the modified ova. Are you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chaosking121/Grand-Order-Reroller/issues/4#issuecomment-315492953,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALf-xa07LCb4kSiNJWPGj6Wuifgf3pR6ks5sN_8ugaJpZM4OWeu7
.

@tdub415 This is the image you should be using.


fix: sync dns records after sync vpcs

这个 PR 实现什么功能/修复什么问题:
fix: sync dns records after sync vpcs

[x] 功能、bugfix描述
[x] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.4

/area region
/cc @zexi

Codecov Report

Merging #8321 into master will decrease coverage by 2.56%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #8321      +/-   ##
=========================================
- Coverage    6.10%   3.54%   -2.57%
=========================================
  Files         762     304     -458
  Lines      118204   62507   -55697
=========================================
- Hits         7222    2213    -5009
+ Misses     110328   60141   -50187
+ Partials      654     153     -501

Flag
Coverage Δ

#aFlag
3.54% <ø> (-2.57%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/keystone/models/roleresource.go

pkg/cloudcommon/db/rbac.go

pkg/compute/guestdrivers/utils.go

pkg/cloudcommon/db/standalone.go

pkg/cloudcommon/db/keystonecache.go

pkg/compute/regiondrivers/qcloud.go

pkg/compute/guestdrivers/managedvirtual.go

pkg/multicloud/aws/s3object.go

pkg/multicloud/ucloud/vpc.go

pkg/multicloud/aws/loadbalancercert.go

... and 448 more

Continue to review full report at Codecov.

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

/lgtm
/approve


Add glaber.io support into plugin

Is your feature request related to a problem? Please describe.
There is a fork of zabbix named Glaber . It has and use many improovments like async pollers, clickhouse database, glaber worker scripts ( do not confuse these scripts with custom scripts, they are not the same thing and perform a slightly different task many times), it has victoriametrics support oit of box
and many others. But as I know your plugin do not support it yet as it can't work with clickhouse to query  history and trends from it.
Describe the solution you'd like
We will be glad to help you to add Glaber support. We have a telegram community - https://t.me/glaber_group and will be glad to see you there.

The API is the same. I don't think any changes required unless the plugin uses the direct database queries

unless the plugin uses the direct database queries

Plugin can be configured to do so.

unless the plugin uses the direct database queries

Plugin can be configured to do so.

Yep, but Glaber work with Mysql/Postgres/etc default zabbix databases and history+trends are stored in clickhouse.

@denisgolius I didn't use  Glaber, but as mentioned above, if it's compatible in terms of API, there's no difference for plugin. The only thing required to be implemented is direct db connection support for Clickhouse. It's not complicated, just a description of queries for history and trend, here's example for postgres:
https://github.com/alexanderzobnin/grafana-zabbix/blob/3410593f9eaf5f10bc1b111b6f6bea4489a82279/src/datasource-zabbix/zabbix/connectors/sql/postgres.js#L7-L18

@denisgolius I didn't use Glaber, but as mentioned above, if it's compatible in terms of API, there's no difference for plugin. The only thing required to be implemented is direct db connection support for Clickhouse. It's not complicated, just a description of queries for history and trend, here's example for postgres:
https://github.com/alexanderzobnin/grafana-zabbix/blob/3410593f9eaf5f10bc1b111b6f6bea4489a82279/src/datasource-zabbix/zabbix/connectors/sql/postgres.js#L7-L18

Thank you for your work.
we will make some research to understand what's going wrong.
Created issue in Glaber project https://gitlab.com/mikler/glaber/-/issues/65 .

What bothers me is that Glaber doesn't stick with ClickHouse only.
A few weeks ago VictoriaMetrics integration came out and Influx supposedly
will be in a month, as well as dedicated support of some log storage
engines, like Elastics and perhaps, Loki.
So I am not sure if there should be a direct Glaber to Clickhouse
integration or should we invest in additional direct integration of each
storage engine Glaber will support.
Regarding ClickHouse support, database structure is almost identical to
classic SQL tables there:
https://gitlab.com/mikler/glaber/-/blob/master/database/clickhouse/history.sql
On Thu, 20 May 2021, 11:53 Alexander Zobnin, @.***>
wrote:

@denisgolius https://github.com/denisgolius I didn't use Glaber, but as
mentioned above, if it's compatible in terms of API, there's no difference
for plugin. The only thing required to be implemented is direct db
connection support for Clickhouse. It's not complicated, just a description
of queries for history and trend, here's example for postgres:
https://github.com/alexanderzobnin/grafana-zabbix/blob/3410593f9eaf5f10bc1b111b6f6bea4489a82279/src/datasource-zabbix/zabbix/connectors/sql/postgres.js#L7-L18
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alexanderzobnin/grafana-zabbix/issues/1202#issuecomment-844764011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQFRBPRHCFQHDXVSD2Y5V3TOSWXDANCNFSM436QQZZA
.

What bothers me is that Glaber doesn't stick with ClickHouse only. A few weeks ago VictoriaMetrics integration came out and Influx supposedly will be in a month, as well as dedicated support of some log storage engines, like Elastics and perhaps, Loki. So I am not sure if there should be a direct Glaber to Clickhouse integration or should we invest in additional direct integration of each storage engine Glaber will support. Regarding ClickHouse support, database structure is almost identical to classic SQL tables there: https://gitlab.com/mikler/glaber/-/blob/master/database/clickhouse/history.sql
…
On Thu, 20 May 2021, 11:53 Alexander Zobnin, @.***> wrote: @denisgolius https://github.com/denisgolius I didn't use Glaber, but as mentioned above, if it's compatible in terms of API, there's no difference for plugin. The only thing required to be implemented is direct db connection support for Clickhouse. It's not complicated, just a description of queries for history and trend, here's example for postgres: https://github.com/alexanderzobnin/grafana-zabbix/blob/3410593f9eaf5f10bc1b111b6f6bea4489a82279/src/datasource-zabbix/zabbix/connectors/sql/postgres.js#L7-L18 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1202 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGQFRBPRHCFQHDXVSD2Y5V3TOSWXDANCNFSM436QQZZA .

ok


C++: Remove redundant MemberVariable character

Hmm, I think I was testing the wrong branch: This does actually affect what MemberVariables we get. I suspect that it's related to uninstantiated templates or something, and that we want to make this change anyway, but I'll get a small testcase before we decide.

I now understand this better, and it's going to be easier to deal with it post-discover, so I'll close this for now.


Version should include minor version number

I noticed that this package is listed as version 19, without minor or bugfix releases.
The official AMBER conda distribution includes a minor version number because typically multiple patch versions are released during the lifecycle of AmberTools to either add new features or fix bugs. Since Amber ff19SB was just released, it seems very likely that there will be more releases during the lifecycle of AmberTools 19, so it may be good to include this minor version number---if at least for consistency!

Just to show how important this is, here are the patch release update versions that are already available for AmberTools 19:
(base) lski2838:amber18 choderaj$ ./update_amber  --check-updates
Checking for available patches online. This may take a few seconds...

Available AmberTools 19 patches:

update.1 (modifies test, suite)
update.2 (modifies packmol-memgen)
update.3 (modifies rism3d.snglpnt)
update.4 (modifies sander, rism3d.snglpnt, rism1d)
update.5 (modifies configure_python)
update.6 (modifies packmol-memgen, and, AmberTools, testing)
update.7 (modifies MMPBSA.py, rism3d.snglpnt)
update.8 (modifies sander.MPI)
update.9 (modifies Parameter, files, for, tleap)

It would be optimal if we could help ensure that a new version is built and released as 19.1, 19.2, etc. for each patch release update version that is released by the Amber developers.
cc: @dacase

I agree that this would be optimal but I'm a bit sceptical about whether builds for versions which are released in the form of patches will be easily reproducible.

I'm not sure I follow. This is (perhaps unfortunately), the standard way AmberTools is distributed, so ensuring that we have a one-to-one correspondence between the integer-versioned patch number within the release is the more reproducible way to go.
It should be simple to adapt the update_amber script within build.sh, instead including a small python script that updates the source tree to the desired ambertools major.minor release version---something like this:
import os
# Get desired AmberTools major.minor version
ambertools_version = os.environ['PKG_VERSION']
# Update source tree to this version
from updateutils import main
from updateutils.exceptions import AmberUpdaterError, PatchingUpdater
main.ALLOW_UPDATES = True
app = main.MainUpdater()
app.update_to(ambertools_version)

This way, we can simply set the desired release number from meta.yaml and the appropriate version would build.
I don't see why this wouldn't be reproducible, given that this is the official mechanism the Amber developers use to deploy official versioned releases.

This may actually be even simpler---it looks like this will work:
./update_amber --update-to=AmberTools.$(echo $PKG_VERSION | cut -d. -f2)

I'll create a PR to see if this proof-of-concept can work.

I was just concerned if there was an easy way to access older versions in the source tree. But this seems okay.
Thanks for the PR!

The update_amber script also has a "revert-to" option to go backwards to older (minor) versions.

With the current logic, each build starts from AmberTools 19.0 and then adds patches to reach the desired patch level.

Handled by #12


Implement certificate installing/uninstalling in Windows adapter

See CertificateInstaller.cs for API.

/core/pull-requests/600


Unit tests

this was a short lived branch because of the quick solution such that the unit tests works properly again
Closes #38

i made an error and removed inMemoryTests by mistake, be aware of this when merging the branches


Flaky tests are actual failed tests

Example
https://play.clickhouse.com/play?user=play#U0VMRUNUIGNoZWNrX25hbWUsIHRlc3RfbmFtZSwgcmVwb3J0X3VybApGUk9NIGNoZWNrcwpXSEVSRSBjaGVja19uYW1lIExJS0UgJ0ludGVncmF0aW9uJScKICAgIEFORCBjaGVja19zdGFydF90aW1lID49IG5vdygpIC0gSU5URVJWQUwgMjQgSE9VUgogICAgQU5EIHB1bGxfcmVxdWVzdF9udW1iZXIgPSAwCiAgICBBTkQgdGVzdF9zdGF0dXMgIT0gJ1NLSVBQRUQnCiAgICBBTkQgdGVzdF9zdGF0dXMgTElLRSAnRiUnCk9SREVSIEJZIGNoZWNrX25hbWUsIHRlc3RfbmFtZSwgY2hlY2tfc3RhcnRfdGltZQ==
We may want to drop all FLAKY logic from master

Sometimes we have a check that is forced green, so the condition is still needed.
But I've removed the notion of FLAKY tests as you've proposed: https://github.com/ClickHouse/ClickHouse/pull/52142


2 Questions using aria2c in python

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.03.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2019.03.09

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2019.03.09
[debug] Python version 3.6.4 (CPython) - Windows-10-10.0.17763-SP0
[debug] exe versions: ffmpeg 3.4.2
[debug] Proxy map: {}
[youtube] DLzxrzFCyOs: Downloading webpage
[youtube] DLzxrzFCyOs: Downloading video info webpage
[youtube] {18} signature length 42.41, html5 player vflkyt12p
[youtube] {271} signature length 42.41, html5 player vflkyt12p
[youtube] {137} signature length 42.41, html5 player vflkyt12p
[youtube] {248} signature length 42.41, html5 player vflkyt12p
[youtube] {136} signature length 42.41, html5 player vflkyt12p
[youtube] {247} signature length 42.41, html5 player vflkyt12p
[youtube] {135} signature length 42.41, html5 player vflkyt12p
[youtube] {244} signature length 42.41, html5 player vflkyt12p
[youtube] {134} signature length 42.41, html5 player vflkyt12p
[youtube] {243} signature length 42.41, html5 player vflkyt12p
[youtube] {133} signature length 42.41, html5 player vflkyt12p
[youtube] {242} signature length 42.41, html5 player vflkyt12p
[youtube] {160} signature length 42.41, html5 player vflkyt12p
[youtube] {278} signature length 42.41, html5 player vflkyt12p
[youtube] {140} signature length 42.41, html5 player vflkyt12p
[youtube] {171} signature length 42.41, html5 player vflkyt12p
[youtube] {249} signature length 42.41, html5 player vflkyt12p
[youtube] {250} signature length 42.41, html5 player vflkyt12p
[youtube] {251} signature length 42.41, html5 player vflkyt12p
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://r2---sn-4g5e6nsk.googlevideo.com/videoplayback?lmt=1538219388413002&dur=212.579&txp=5532432&itag=271&clen=88019015&gir=yes&fvip=2&sparams=aitags%2Cclen%2Cdur%2Cei%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Crequiressl%2Csource%2Cexpire&requiressl=yes&source=youtube&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C271%2C278&mv=m&mt=1552535616&ms=au%2Conr&pl=16&keepalive=yes&id=o-AD37HiS6Wta-Q-vceNFTu7Jg5HmlTkKI876F0HtWEAj9&ipbits=0&mn=sn-4g5e6nsk%2Csn-h0jeener&mm=31%2C26&ip=<IP_ADDRESS>&initcwndbps=1548750&c=WEB&expire=1552557338&ei=udCJXO3vO8a_gQeA8bbQCA&key=yt6&mime=video%2Fwebm&signature=3B28054F94099C4551264ADCA2A5CF12EF0D5A56.05D567FFE576E7FDABD4A9213728EA89E50CB6DA&ratebypass=yes'
[download] Destination: Rick Astley - Never Gonna Give You Up [HQ]-DLzxrzFCyOs.f271.webm
Traceback (most recent call last):
  File "testen.py", line 12, in <module>
    down.download(["https://www.youtube.com/watch?v=DLzxrzFCyOs"])
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 2006, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 805, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 859, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 1640, in process_video_result
    self.process_info(new_info)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 1907, in process_info
    partial_success = dl(fname, new_info)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\YoutubeDL.py", line 1852, in dl
    return fd.download(name, info)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\downloader\common.py", line 364, in download
    return self.real_download(filename, info_dict)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\downloader\external.py", line 35, in real_download
    retval = self._call_downloader(tmpfilename, info_dict)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\downloader\external.py", line 101, in _call_downloader
    cmd = [encodeArgument(a) for a in self._make_cmd(tmpfilename, info_dict)]
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\downloader\external.py", line 187, in _make_cmd
    '--min-split-size', '1M', '--max-connection-per-server', '4'])
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\downloader\external.py", line 97, in _configuration_args
    return cli_configuration_args(self.params, 'external_downloader_args', default)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python36\lib\site-packages\youtube_dl\utils.py", line 2923, in cli_configuration_args
    assert isinstance(ex_args, list)
AssertionError

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

URL I used: https://www.youtube.com/watch?v=DLzxrzFCyOs

Description of your issue, suggested solution and other information
First of all, this is the simple python code:
# -*- coding: utf-8 -*-
import os
import youtube_dl
cwd = os.getcwd()

download_opts = {
'external_downloader':"aria2c",
'external_downloader_args':"-x 16 -s 16 -k 1M",
'verbose':True,
}
with youtube_dl.YoutubeDL(download_opts) as down:
	down.download(["https://www.youtube.com/watch?v=DLzxrzFCyOs"])

So, my questions are:

How do I set the path for aria2c? I don't want the aria2c.exe inside the folder where the script is, it should be in an subfolder called "bin".
I tried 'external_downloader':"bin\\aria2c", or 'external_downloader':cwd+"\\bin\\aria2c", but either it ignored it or it gave me an error.

How do I specify the external_downloader_args? It works when aria2c is in the same folder as the .py script and I delete the line 'external_downloader_args':"-x 16 -s 16 -k 1M", but with it, it just outputs the error above.

This question was asked, but you said we should open a new Issue with reproducable environment, so I attached a .zip if needed. It just contains the script, a .bat file, aria2c and an empty "bin" folder ^^ test.zip
https://github.com/ytdl-org/youtube-dl/issues/15337#issuecomment-358764727
Thanks!

You should put the aria2c binary in a folder in your PATH.

As for 2, external_downloader_args must be a list not string as clearly stated in option's description.

cwd = os.getcwd() YDL_opts = {         "verbose": False,         "external_downloader": cwd + "\\binaries\\aria2c",     }


Question about flashdecoding with appendKV

Hi,
I am curious about why n_blocks_per_split is calculated using params.seqlen_k instead of actual_seqlen_k in the following code:
https://github.com/Dao-AILab/flash-attention/blob/b443207c1fc4c98e4532aad4e88cfee1d590d996/csrc/flash_attn/src/flash_fwd_kernel.h#L525
It seems to be wrong in some cases.
Considering:
seqlen_k = 1024;
seqlen_k_new = 1;
BlockN = 128;
num_split = 4;
the n_blocks_per_split would be equal to 2. And then n_block_max can only reach a maximum of 8 ((3 + 1) * 2) according to:
https://github.com/Dao-AILab/flash-attention/blob/b443207c1fc4c98e4532aad4e88cfee1d590d996/csrc/flash_attn/src/flash_fwd_kernel.h#L529
If we attempt to append KV, n_block_copy_min is also equal to 8, which means there is no condition that allows gKNew to append to gK:
https://github.com/Dao-AILab/flash-attention/blob/b443207c1fc4c98e4532aad4e88cfee1d590d996/csrc/flash_attn/src/flash_fwd_kernel.h#L727
https://github.com/Dao-AILab/flash-attention/blob/b443207c1fc4c98e4532aad4e88cfee1d590d996/csrc/flash_attn/src/flash_fwd_kernel.h#L730
Am I missing something here?

seqlen_k is the total kv cache length. i think in your case, the cache_seqlen_k should be strictly less than seqlen_k if there are new tokens to be appended.


PIMOB-1141: eager card checks in checkout and update parts missed in merge

Proposed changes
2 sets of changes

update checkout sdk to later version (older version was committed previously by accident)
add eager card checks pr and into checkout sdk

note that eager card checks are not setup with frames currently. this will be added in a follow up PR

Types of changes
What types of changes does your code introduce to Frames Ios?
Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

I see no changes to git ignore, so I guess you just deleted the files?

Yes. I think they are already ignored and they were added whilst trying to fix the Carthage stuff