useStorage inconsistency between live and test stores in @shopify/ui-extensions-react/checkout

Please list the package(s) involved in the issue, and include the version you are using

@shopify/ui-extensions-react/checkout version ^2024.7.0

Describe the bug
I am experiencing an inconsistency when using the useStorage hook in the Checkout Extension between a live store and a Partner development (test) store. The issue manifests when saving and retrieving state during the checkout process.
Example Code:
import { useStorage } from '@shopify/ui-extensions-react/checkout';

export default function App() {
  const storage = useStorage();

  useEffect(() => {
      storage.read('state').then((res) => {
          console.log('----state---', res);
       });

      storage.write('state', 'true')
  }, []);
}

The Issue:

On a live store: When navigating to the checkout page and refreshing it, the stored value logs as 'true' in the console as expected. However, if the user navigates back to the cart page and then returns to the checkout page, the stored value becomes null.
On a test store: The issue does not occur. The stored value logs as 'true' consistently even after navigating away from and returning to the checkout page.

Steps to reproduce the behavior:

Use the provided example code in a Checkout Extension.
Deploy the extension to both a live store and a test store.
On the live store:

Go to the checkout page.
Refresh the checkout page and observe the console log ('true').
Navigate back to the cart page.
Return to the checkout page and observe the console log (null).

On the test store:

Follow the same steps.
Observe that the value remains 'true' consistently.

Expected behavior
The useStorage hook should consistently retain and return the stored value across different navigation events, regardless of whether the store is live or in a test environment. The behavior should be consistent between live and test stores.
Additional context

This issue occurs on a live store but not on a Partner development store.
My current dependencies are:

@shopify/ui-extensions-react version ^2024.7.0
react version ^18.0.0

I've tried reproducing this issue on a Partner development store but was unable to do so. However, it consistently occurs on a live store.

@mehdizarepour the storage is cleared if the cart token is changed. This can happen on a storefront if a new cart is created rather than re-using the existing one before progressing to checkout. This would be expected behaviour, and the likely cause of this.
I would inspect what is happening on the storefront in order to further debug this issue. I will close this issue, but feel free to re-open it if you confirm the same cart is being used and you are still getting this issue.


npm install errors

Having issues installing this package with the example you gave.
Running Centos7
Error: http://pastebin.com/1sWk0W38

Was this ever working?

for me, never. not sure about others.

How familiar are you with C++? I personally am sort of clueless. However, I'm noticing that this may be related to the fact that we never 'include' the v8 header files or whatever.
Thoughts?


Tl321x Bringup

For compliance check only.

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_telink
https://github.com/telink-semi/hal_telink/commit/36283106d9b0b1d5fb3025a98be5a58007ac3d77 (develop,telink_tl321x)
https://github.com/telink-semi/hal_telink/commit/c52db89f759a4257b43f812e807096ce4902a32b (tl321x_bringup)
<EMAIL_ADDRESS>

openthread_telink_lib
https://github.com/telink-semi/openthread_telink_lib/commit/80645ba0b4b589afd8c0712cb1a01091f4b35ab8 (main)
https://github.com/telink-semi/openthread_telink_lib/commit/84e12fd24613b2871c47378c39c2f44447f528ba (tl321x_skeleton)
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.


common-utils:2 ending rc files without a trailing newline

With the common-utils:2 feature being released, the various rc files (~/.bashrc, /etc/bash.bashrc, /etc/zsh/zshrc) no longer end with a trailing newline.
This silently breaks every successive feature that doesn't version the dependency on common-utils (ie, pretty much all of them) and uses/copies the updaterc function for appending to the files.

It should be fixed by #342


[BUG] Missing spotless-lib-2.30.0.jar causing build failures

Describe the bug
Gradle checks are failing due to a missing dependency JAR issue (404 error).
A problem occurred configuring root project 'OpenSearch'.
> Could not resolve all files for configuration ':classpath'.
   > Could not find spotless-lib-extra-2.30.0.jar (com.diffplug.spotless:spotless-lib-extra:2.30.0).
     Searched in the following locations:
         https://plugins.gradle.org/m2/com/diffplug/spotless/spotless-lib-extra/2.30.0/spotless-lib-extra-2.30.0.jar
   > Could not find spotless-lib-2.30.0.jar (com.diffplug.spotless:spotless-lib:2.30.0).
     Searched in the following locations:
         https://plugins.gradle.org/m2/com/diffplug/spotless/spotless-lib/2.30.0/spotless-lib-2.30.0.jar

It appears that the JAR has vanished from this repo for version 2.30.0, but is available for other versions.
Failed builds:

https://build.ci.opensearch.org/job/gradle-check/5935/console
https://build.ci.opensearch.org/job/gradle-check/5937/console

To Reproduce
Trigger Gradle checks via Jenkins.
Expected behavior
Dependencies should be found.

@ketanv3 I don't see this issue happening for gradle checks now. Are you still facing this issue?

This seems to be a transient issue that impacted all builds for a brief period yesterday. I'll go ahead and close this for now.


Fix Opening the channel switch modal triggers a JS error when user is undefined

Summary
Sometimes when using CMD/CTRL+K the users to build the unread dm's haven't yet loaded making the user object undefined, if that is the case the PR just skips it.
Ticket Link
https://mattermost.atlassian.net/browse/MM-10878
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

Should we fire a request to load the missing user, or is the client already loading that user when this happens?

@hmhealey If I understand this correctly it should be triggered by then, also it should fetch the users based on the search term


Make WireHopperTest runnable alongside Docker containers

Issue #, if available: N/A
Description of changes:
WireHopperTest used to spin up a testing server on port 9650, which is
used by the Docker container to host our GRPC server. This meant that if
you had your Docker containers running, WireHopperTest would always
fail.
This commit modifies WireHopperTest to use a different port for
testing.
Tests: WireHopperTest
Code coverage percentage for this patch: N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The logic in this PR has already been merged through another PR


Fix the StdMultiWidgets QSplitters

Fix the visual glitches on Linux, Windows and OS X. This is tested only on Windows and Linux. I don't have an OS X computer near. My last PR which was almost identical was tested on all OS and was working.
I don't know if it's just me but on my Linux VM, when I click on fullscreen on one of the views, it also moves the level-window and the image becomes white (you may need to move the crosshair or return the standard view). This PR fixes that.
There's a lot of changes but it's not my fault if there are 13 possible layouts! Anyway, most of my changes are deletions; the 13 methods are shorter and cleaner than they were, and there's less code copy.

The whole layout was changed: Now, there is a QmitkMultiWidgetLayoutManager which allows to generalize the layout setting. This was desired, since we now provide a MxNMultiWidget, where the number, layout and view direction of the render windows is not fixed anymore.
Could you check if your visual glitches still appear?
Also, feel free to include your code-changes in the new QmitkMultiWidgetLayoutManager anyway.

Just for housekeeping purposes: I close this PR. Feel free to re-open it, if the issue still persists. :-)


deepDiffer code triggers "Your code is broken" warnings about for...in Array iteration

This code in Libraries/Utilities/differ/deepDiffer.js triggers the warning mentioned in #869
        for (var key in one) {
            if (deepDiffer(one[key], two[key])) {
                return true;
            }
        }
        for (var twoKey in two) {
            // The only case we haven't checked yet is keys that are in two but aren't
            // in one, which means they are different.
            if (one[twoKey] === undefined && two[twoKey] !== undefined) {
                return true;
            }
        }

Dupe of #868, it looks like.


Fix SELECT after session.sql() with params

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
No issue, quick fix

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Query constructor was being called with params passed to the wrong argument. This uses the keyword argument correctly. I also changed the Query constructor to require any parameters after the SQL string to be keyword arguments, to prevent this in the future.

I have read the CLA Document and I hereby sign the CLA

@sfc-gh-sfan What are the next steps? Is this PR good to merge? I don't have merge access for this repo, so if it's good to merge, can someone merge it for me?

Merged! Sorry I'm not aware that you don't have merge access :)

No worries. Thanks a ton!


Make Browser.click/2 return the parent element

This addresses issue #134

Looks great! Thanks :tada:


"only" modifier

Is there something like mocha's only modifier (i.e. it.only) to only run a certain test?

Not right now, but that would be nice. How about a flag or environment variable on the command line, e.g. SPECS='Foo.*' pulp test?
Thanks!

That would be cool as well, but I can see more use if I could do it from code. The reason being that if I have pulp -w test running, I change this setting mid-way through.

I just had a look through the code, I think adding this would be hard because each it block immediately executes if I am not mistaken. I think this whole project would be a perfect match for the Free monad? What do you think? I would be keen to do some work for the learning experience.

Yes, I agree, getting that lazy execution of tests is something that I've wanted to do. The current design makes it quirky to implement the Mocha reporter and support async, I think that might be simpler with a Free monad design as well.
If you'd like to have a stab at it that would be great!

@felixSchl Did you try anything out with the Free monad?

Nothing worth mentioning. I still think it's something very desirable to have, I just didn't have the time to invest into it. Not sure if the Free monad would work, how would you capture the arbitrary user code that runs as part of it in a data type?

I guess the ADT would need to keep that Aff value and run it on interpretation. Would be cool to investigate this track. I have so little time right now, though. 😞

Alternatively we could put it behind a function: Unit -> Aff _ _

Sure, that may be the best way.
If we could sync a time then maybe we could hack on this over Skype, hangouts or whatever?

At least to have some starting point.

yeah hangouts would be best, I'm not big on skype. It's just this email address.

I'm closing this as we're not currently progressing. If we want to try the Free monad approach later on I think that's a bigger redesign. Thanks!


Skip copy of r10k modules directory.

If, during development, a user installs modules locally using r10k, the created module directory will conflict with the step where onceover installs modules via r10k. There could also be hidden problems in the modules directory that users don't have as much control over. This Code skips copying of the modules directory and warns users that they have an existing modules directory, as if they have actual code in the modules directory, it will likely not work with onceover.

Nice, looks good to me


ENH: update vendored SuperLU version

Is your feature request related to a problem? Please describe.
SuperLU is at the core of sparse.linalg. Recently, version 6.0 was released with support for 64bit index support. Changing from 32bit to 64bit indexing in scipy 's sparse arrays caused a lot of issues such as #18644 downstream because scipy's SuperLU cannot handle them at the moment. Updating the SuperLU version would resolve this issue.
The biggest issue is likely the complexity of the wrapper as scipy not only needs to wrap the C code in a safe way but additionally applied some patches to work around other issues in SuperLU.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

I'm in the middle of untangling this. Not sure I can finish soon, but if not I'll submit a PR for what I have so far.


Novacom installation

I downloaded the latest release of universal installer. I have java installed on my win 7 32 bit, but double clicking the installer nothing happens. please help
im trying to install TPToolbox to do a clean install.
Thanks

Hi,
try to execute it from the command line with:
java -jar UniversalNovacomInstaller-1.4.1.jar

That should give you at least an errormessage so we can narrwo down where it fails.

@paulnwale I haven't heard from you in 2 month so I assume that you found a solution.
If you stil need help feel free to reopen this issue.


Wrong image ratio / Image cut off

Describe the bug
The image is displayed in the ratio 1:1 if opened the first time. If I open the photo viewer again, the image is displayed correctly. See the video for more details.

Screenshots

Smartphone (please complete the following information):

Device: iPhone 11 mini
OS: 15

@mariusbolik Well strange behavior i will have a look

@mariusbolik Can you share the code or a small repro app on github

@mariusbolik this is fixed in 1.0.3. Feel free to close the issue when tested on your side

@jepiqueau works perfectly! Thank you for the fast help!


Outdated dependencies

Hello, the dependencies for this package are outdated, which is causing an install warning, because globlue 0.1.0 uses lodash 1.0.2. This results in:
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>
Could you update to the latest globule version 0.2.0?

I'm seeing this too.
Looks like the package.json has been updated but it hasn't been pushed to npm.

What version of gaze? Looks fine with @0.6.4:
$ npm view<EMAIL_ADDRESS>dependencies.globule
~0.2.0

0.5.2 it seems, which is what node-sass 3.4.2 is using, which is what gulp-sass is using...
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ └─┬<EMAIL_ADDRESS>│   │   └──<EMAIL_ADDRESS>
Gah, it's a nightmare to find out who is responsible for deprecation warnings :P

Sorry all. I'm in the process of publishing a 1.0.0. master is the latest code but not the latest published release.

Just a :+1: here. This has a couple deprecated versions in its' dependence tree, and since quite a few packages depend on this, many installs are affected:

globule currently depends on<EMAIL_ADDRESS>(which is now up to 7.0.0!), which depends on<EMAIL_ADDRESS>which is breaking on current nodes (bad!). This has been posted on glob (see cowboy/node-globule#29), but it's two years old, and probably won't be updating. I suspect you'll have to switch to another package, or fork it.
As @adambuczynski mentions, lodash is also unsupported pre-4.0.0, which is something you should update to for a multitude of reasons! (=

@shama - I don't mean to pile on, but I noticed that the package.json on the main page has<EMAIL_ADDRESS>as a dependency, but the npm bundle version mentions<EMAIL_ADDRESS>Not entirely sure what this means (I'm new to npm) but I've added some detail in https://github.com/kriasoft/react-starter-kit/issues/441#issuecomment-189986155
NB updating the npm version to<EMAIL_ADDRESS>would minimize the grumpy warnings.

Update: It appears that dependencies (including globule) were updated on 16 Jan 2016, but that the version number wasn't bumped. Perhaps npm is caching the older<EMAIL_ADDRESS>

Closing as it's now using latest dependencies.


Dummy login/register screen!

Every app needs a login screen, and ours is no exception. Create a super simple login/register workflow that we can use for actual login abilities. For the register screen, just make a dummy page, doesn't have to work. Make sure to include a screenshot! And make sure to use MUI
for now, maybe use a dummy username + password like
username: theonepiece
password: isreal

Create a new page that can be used to login and redirects to a main page

Just to clarify, this only needs to be frontend only, it doesn't have to be fully implemented on backend


fix: cloud-account sync ignore db failure

这个 PR 实现什么功能/修复什么问题:
修正：cloudaccount同步时忽略了数据库错误，导致import重复的子账号
是否需要 backport 到之前的 release 分支:

release/2.8.0

/cc @Zexi @ioito

/cherrypick release/2.8.0

/approve

/lgtm

Codecov Report

Merging #518 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #518      +/-   ##
=========================================
- Coverage    7.42%   7.42%   -0.01%
=========================================
  Files         345     345
  Lines       54271   54273       +2
=========================================
- Hits         4032    4028       -4
- Misses      49863   49868       +5
- Partials      376     377       +1

Flag
Coverage Δ

#aFlag
7.42% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/cloudaccounts.go
1.01% <0%> (-0.01%)
:arrow_down:

pkg/util/rbacutils/rabc.go
72.34% <0%> (-1.52%)
:arrow_down:

pkg/appsrv/response.go
66.15% <0%> (ø)
:arrow_up:

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 3d5bb8a...beee1fa. Read the comment docs.

/lgtm

/approve

/approve


Ability to hide Edit Cookies button; possible to add prop?

Greetings!
Thank you so much for this wonderful piece of code. Surprisingly few SvelteKit options that work well, but finally came across yours, which is excellent.
For my use case, we are using only necessary cookies, and tho I have toggled the other types of cookies to false by default, internal users are finding this confusing as it's not clear that those toggle are either on or off. And since they are unable to toggle off necessary cookies, many seem to feel that the toggle state of the others should match that, which they don't.
So for this simplest of all use cases, giving the user the ability to edit cookie settings is just confusing.
But we may eventually add other cookies, so keeping with a library that will support those should the time come is preferable to hacking up a vanilla solution...
I forked your repo, added a "canEditCookies" prop, and added an {#if} in Banner.svelte around the button, following your same code flow for the optional edit cookie settings overlay.
However, when trying to add the forked repo from github, ran into errors since the project actually has to be compiled to get a /distro folder. Experienced w/ SvelteKit, but not w/ building libraries for such...
So just did a build (unsure of the package command), but had to get into changing @beyonk in some places, but not all, to get it to build a working /dist that I've manually copied to the appropriate node_modules folder.
Wondering if you might consider adding this to the project proper? 🙏
I do believe there are others looking for the simplest use case, but with room to grow. Found several threads bemoaning lack of a good SvelteKit lib for same.
My diffs are so simple, don't know that it's worth a pull request. But if it's of help, I would clean it up, remove the name change, and update the documentation, and issue the pull request.
Thanks much,
Morgan

Hey Morgan,
This sounds great - if you open a PR I'll gladly review + merge.


Repository/package cleanup

Now, that we are merging repositories together, the question comes up whether we should remove the deprecated repositories or not.
Pros:

less repository, less confusion
less repository, less maintenance

Cons:

some of the repositories have already been tagged, which means they COULD be used somewhere. Fortunately packagist gives us information about dependent packages.
we lose issues, PRs, history (which is probably not a big deal in the current phase of the project)

i propose we mark all repositories we don't need anymore as abandoned on
packagist and explain that they are fading out and where the content
moved. one month after tagging 1.0, we can remove them. that gives
people some time to update.

i propose we mark all repositories we don't need anymore as abandoned on packagist

Already done.

explain that they are fading out and where the content moved

Also done. Marked as deprecated, moved to repository text is in the beginning of the README.

one month after tagging 1.0, we can remove them. that gives people some time to update.

Sounds good to me.

:+1:

Shouldn’t php-http/adapter be marked deprecated in favour of httplug?

It should be. The repo itself is already removed I think, the package should also be deleted at some point.

Can we begin to clear some deprecated packages ? Would be nice to have something clean for the first release

Yes.
@Seldaek regardless of the installation count, can you please remove php-http/adapter?
Please also remove the following packages:

php-http/encoding
php-http/authentication
php-http/cookie
php-http/client-tools

Also, package authors please update your packages and remove dependency on the following:
php-http/utils: @mekras, @ml, @swader
php-http/message-decorator: @dbu, @ddeboer

Done

Thank you very much.

thanks.
@sagikazarmark i rebased an older PR that was lying around. i think that was the reason for this dependency - in master we no longer depend on it.

Ok. @Seldaek can you please delete php-http/message-decorator as well? Thank you very much.
We only have utils then.

Alrighty

Thank you

WOW:
https://packagist.org/packages/php-http/promise/dependents
While client-tools does not exist anymore.

That's called caching :) It'll go away by tomorrow.

kk

Also, package authors please update your packages and remove dependency on the following:
php-http/utils

Sure @sagikazarmark. Is there a replacement for it? I think I used it for the Guzzle Message/Stream factories

yes there is https://github.com/php-http/message

Done. The dependency graph is getting bigger and bigger :-(

Done. The dependency graph is getting bigger and bigger :-(

^ this. :(
I, too, have now updated my client and all seems to be working well.

How is it getting bigger? You replaced on package with another.

For me in particular, through puli - I do love the package, but it's a rather enormous beast to depend on. The beta status of the composer plugin doesn't help things, either.

I think all packages without dependencies are now cleaned up. All the remaining repositories have dependencies.


[WIP] Test for 10 containers

Adding 10 containers test. Basically we look for reboots. Checking only 2 nginx via curl.
Signed-off-by: fleandr<EMAIL_ADDRESS>

It looks like docker_test.go needs to be tweaked to add the memory parameter. At the moment, by default, 1GB of memory is allocated for each container.

Hey @giggsoff -- can you please do that tweak so we can merge this?

@mydatascience @rvs done


fix: added the missing ids in parameterized tests

Closes #362
Summary of Changes
Some parameterized tests didn't have ids. The missing ids are added now.

There are a couple functions, where ids are missing in test_image.py

@PhilipGutberlet please add the "Time spent by team" for your Issue

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

v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


JSON File Size Too Large for an Int32

Thanks so much for this great library.  We have some very large files that we serve up in our Wistia account, and, on a few of them, I am receiving this error.
{"JSON integer<PHONE_NUMBER> is too large or small for an Int32. Path 'assets[0].fileSize', line 1, position 652."}
With the JSON integer being whatever the actual Wistia return value is.  This happens when calling the WistiaDataClient.Media.GetById.
Thanks so much!

I'll put a new release up, looks like it's as simple as using a long instead of int there

Released in v1.0.6110.19776

Thank you, sir!


CAMEL-10478 Support simple language expressions in header values in camel-salesforce

This is a PR against 2.17.x, as this functionality would be beneficial to 2.17
users as well.
Implements support for simple language expressions in header values for
the camel-salesforce component.
AbstractSalesforceProcessor was modified to post-process values
retrieved from message header and evaluate them with simple language.
This greatly simplifies usage in downstream salesforce components as
they can have access upstream values. For instance:
salesforce:queryMore?sObjectQuery=${header.nextRecordsUrl}...

There is no need for this in camel-salesforce, closing.


how to debug Extending the printer through Hardware Station in D365 any sample ??

@dsdharmen91, thanks for posting this feedback.
@mugunthanm, please consider updating this topic to reflect this feedback. Bug 280776 has been logged to track this issue.


 This appears to be because we're running a database query every time we need to fetch the description of a node. We should be fetching all these descriptions as part of the original query so we don't have to make new server requests every time the user drags their mouse over the graph.
Also, sometimes it takes quite a while for the pop-up box to go away. Since it can block other elements the user wants to look at, this is annoying. The behavior is not consistent, but it occurs fairly frequently. What is causing this?

Actually I think I may be wrong about the pop up disappearing, I am currently working on that issue right now and it seems like with simple nodes it works fine (faster than the old version of demo2) so I'm gonna see if changing the query will adjust this speed.


higher_is_better is not set on some classification metrics

🐛 Bug
There are some classification metrics, such as BinaryAUROC or BinaryAveragePrecision, where higher_is_better is not set, i.e. equal to None. There could be others, I have not checked all of them.
To Reproduce
import torchmetrics as tm
auroc = tm.classification.BinaryAUROC()
assert auroc.higher_is_better

Expected behavior
auroc.higher_is_better in the above snippet should be True
Environment

TorchMetrics version (and how you installed TM, e.g. conda, pip, build from source): 0.11.4, conda
Python & PyTorch Version (e.g., 1.0): python 3.9.16, pytorch 2.0.1
Any other relevant information such as OS (e.g., Linux): MacOS Ventura 13.4.1, Apple M2 Pro

Seems the same problem also exists for: BinaryRecallAtFixedPrecision
Config: Python3.11, torchmetrics 1.4.0.post0

@PierreOreistein mind send a PR fixing it?


[Agent] Custom plugin fails

基于v3.2.6新开发了一个插件，根据正则表达式匹配类名，并trace所有方法。
在enhance class 时报下面错误，目前看起来跟目标类无关，应该是我代码的问题，麻烦帮我看看是哪里的问题。
代码见附件
ERROR 2017-12-16 17:54:58 SkyWalkingAgent :  Enhance class com.operations.sessionmanager.AuthTokenRequestOperation error.
java.lang.IllegalArgumentException: Cannot resolve ambiguous delegation of public abstract java.lang.Object org.skywalking.apm.agent.core.plugin.interceptor.enhance.EnhancedInstance.getSkyWalkingDynamicField() to org.skywalking.apm.dependencies.net.bytebuddy.implementation.bind.MethodDelegationBinder$MethodBinding$Builder$Build@e577ee67 or org.skywalking.apm.dependencies.net.bytebuddy.implementation.bind.MethodDelegationBinder$MethodBinding$Builder$Build@74c4a417
        at org.skywalking.apm.dependencies.net.bytebuddy.implementation.bind.MethodDelegationBinder$BindingResolver$Default.doResolve(MethodDelegationBinder.java:604)
        at org.skywalking.apm.dependencies.net.bytebuddy.implementation.bind.MethodDelegationBinder$BindingResolver$Default.resolve(MethodDelegationBinder.java:553)
        at org.skywalking.apm.dependencies.net.bytebuddy.implementation.bind.MethodDelegationBinder$Processor.bind(MethodDelegationBinder.java:1028)
        at org.skywalking.apm.dependencies.net.bytebuddy.implementation.MethodDelegation$Appender.apply(MethodDelegation.java:1051)
        at org.skywalking.apm.dependencies.net.bytebuddy.dynamic.scaffold.TypeWriter$MethodPool$Record$ForDefinedMethod$WithBody.applyCode(TypeWriter.java:620)
        at org.skywalking.apm.dependencies.net.bytebuddy.dynamic.scaffold.TypeWriter$MethodPool$Record$ForDefinedMethod$WithBody.applyBody(TypeWriter.java:609)
        at org.skywalking.apm.dependencies.net.bytebuddy.dynamic.scaffold.TypeWriter$MethodPool$Record$ForDefinedMethod.apply(TypeWriter.java:526)
        at org.skywalking.apm.dependencies.net.bytebuddy.dynamic.scaffold.TypeWriter$Default$ForInlining$RedefinitionClassVisitor.visitEnd(TypeWriter.java:3854)
        at org.skywalking.apm.dependencies.net.bytebuddy.jar.asm.ClassReader.accept(ClassReader.java:702)
        at org.skywalking.apm.dependencies.net.bytebuddy.jar.asm.ClassReader.accept(ClassReader.java:500)

internal.zip

Suggestion first, next time, when you want to post some codes, please use markdown format to write in the issue directly. It is good for reading and documenting. Then all issues should be in English.

To resolve your issues, I want to know: Do you remove all existed plugins, we provided. Because I am not sure this caused your codes only, or the compatible issue between ours and your custom plugin.
And here is my guess:

You class is being enhanced more than once, for some reason I am not sure.
#getSkyWalkingDynamicField is a dynamic method generated by SkyWalking core. If you want to enhance class again(already enhanced once), please skip this method.

You class is being enhanced more than once, for some reason I am not sure.

@suxiaoyong This is the key, I think.

Closing this, no active comments in 4 months.


Lucene 9.9: Benchmark sparse improvements

Follow up to this: https://github.com/castorini/anserini/issues/2288#issuecomment-1840810694
It'd be great to re-benchmark the "Wacky Weights" paper: https://arxiv.org/abs/2110.11540
It's actually quite simple, we've gotten it down to a single command:
python src/main/python/run_regression.py --download --index --verify --search --regression msmarco-passage-splade-pp-ed

See: https://github.com/castorini/anserini/blob/master/docs/regressions/regressions-msmarco-passage-splade-pp-ed.md
@jpountz @benwtrent @ChrisHegarty Do either of you have cycles to try it out?

No further follow-up, closing.


Use a ticker, instead of time.After, for mempool idling

Fixes #1060

Codecov Report

Merging #1061 (dff5aa7) into master (828364b) will decrease coverage by 1.52%.
The diff coverage is 60.00%.

:exclamation: Current head dff5aa7 differs from pull request most recent head 118348e. Consider uploading reports for the commit 118348e to get more accurate results

@@            Coverage Diff             @@
##           master    #1061      +/-   ##
==========================================
- Coverage   50.51%   48.98%   -1.53%
==========================================
  Files         145      154       +9
  Lines        7544     8063     +519
==========================================
+ Hits         3811     3950     +139
- Misses       3066     3417     +351
- Partials      667      696      +29

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <ø> (ø)

pkg/config/genesis/config.go
0.00% <0.00%> (ø)

pkg/config/genesis/presets.go
50.00% <ø> (ø)

pkg/core/candidate/validator.go
57.14% <ø> (ø)

pkg/core/chain/legacy.go
26.08% <0.00%> (-2.49%)
:arrow_down:

pkg/core/chain/mock.go
8.33% <ø> (ø)

pkg/core/consensus/agreement/accumulator.go
83.01% <ø> (ø)

pkg/core/consensus/agreement/handler.go
56.14% <ø> (ø)

pkg/core/consensus/agreement/store.go
62.12% <ø> (ø)

pkg/core/consensus/agreement/testutil.go
100.00% <ø> (ø)

... and 141 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 a6c3f07...118348e. Read the comment docs.


improvement: change :wrap_list doc to x | list(x)

From comment in https://github.com/ash-project/ash/pull/700: changes order for :wrap_list doc from list(x) | x to x | list(x).
No idea if it is a "fix" or "improvement", put it in improvement to be on a safer side.

🚀 Thank you for your contribution! 🚀


feat: add git-lfs

Adds git-lfs.
Missing git-lfs leads to errors that are hard to debug.

Duplicate of https://github.com/docker-library/buildpack-deps/issues/112 :eyes:


How to create a backup YubiKey with existing PGP keys

Branch
master
Guide
How to generate and air gap PGP private keys using GnuPG, Tails and YubiKey
Operating system and version
Tails 4.22
Description
First of all, thank you Sun for the great work you are contributing here :)
I wanted to know how I could provision a second YubiKey as a backup (in case I lose my primary key traveling for example)?
I tried to repeat the steps in the guide to move the keys to a new card but I get the error: gpg: KEYTOCARD: Unusable secret key
I have the primary card also plugged in and imported the private key stubs from the YubiKey to make sure.

Great question!
When one moves subkeys to YubiKey, they are deleted from host computer.
The way I usually handle restoring from backup on Tails is importing master key and subkeys using gpg --import /path/to/master.asc and gpg --import /path/to/sub.asc.
Then, I run YubiKey related steps.

Thanks Sun for the quick response!
I may have done something wrong then in this case..
After restarting Tails and unlocking my veracrypt file I proceeded to import my key:
amnesia@amnesia:~$ gpg --import /media/amnesia/Tails/master.asc
gpg: key 0x529-------------: "<redacted>" not changed
gpg: To migrate 'secring.gpg', with each smartcard, run: gpg --card-status
gpg: key 0x529-------------: secret key imported
gpg: Total number processed: 1
gpg:              unchanged: 1
gpg:       secret keys read: 1
gpg:  secret keys unchanged: 1

amnesia@amnesia:~$ gpg --import /media/amnesia/Tails/sub.asc
gpg: key 0x529-------------: "<redacted>" not changed
gpg: To migrate 'secring.gpg', with each smartcard, run: gpg --card-status
gpg: key 0x529-------------: secret key imported
gpg: Total number processed: 1
gpg:              unchanged: 1
gpg:       secret keys read: 1

amnesia@amnesia:~$ gpg -K
/home/amnesia/.gnupg/pubring.kbx
--------------------------------
sec   ed25519/0x529------------- 2021-09-14 [C]
      Key fingerprint = 774E 05B8 C732 47D9 7C33  0762 529B DB1C 541F C1DD
uid                   [ unknown] <redacted>
ssb#  ed25519/0x6C8------------ 2021-09-14 [S] [expires: 2022-09-14]
ssb#  cv25519/0x0A6------------ 2021-09-14 [E] [expires: 2022-09-14]
ssb#  ed25519/0x71B------------ 2021-09-14 [A] [expires: 2022-09-14]

I can see there is # symbol next to the key types which means they are stubs and when attempting to copy these to the new card I get the following:
amnesia@amnesia:~$ gpg --edit-key 0x529--------------
gpg (GnuPG) 2.2.12; Copyright (C) 2018 Free Software Foundation, Inc.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Secret key is available.

sec  ed25519/0x529--------------
     created: 2021-09-14  expires: never       usage: C
     trust: unknown       validity: unknown
sub  ed25519/0x6C8--------------
     created: 2021-09-14  expires: 2022-09-14  usage: S
sub  cv25519/0x0A6--------------
     created: 2021-09-14  expires: 2022-09-14  usage: E
sub  ed25519/0x71B--------------
     created: 2021-09-14  expires: 2022-09-14  usage: A
[ unknown] (1). <redacted>

gpg> key 1

...

gpg> keytocard
Please select where to store the key:
   (1) Signature key
   (3) Authentication key
Your selection? 1
gpg: KEYTOCARD failed: No secret key

gpg>

I can see here that the keys are listed as sub which means they are public sub keys according to the legend:
sec => 'SECret key'
ssb => 'Secret SuBkey'
pub => 'PUBlic key'
sub => 'public SUBkey'

I think I have probably messed up the export of the keys in the first time around?

It was that.. I must have exported the keys AFTER I had already moved them to the YubiKey so the private keys were not in the backup. Went through the steps again and even looked at the linked guide, it was great fun to dive into this! I found one issue with the path for pinentry-mac that deviates from what is set in the guide, I will make a PR for that and maybe adding the step to trust the key after you import it on the mac which is listed here: https://github.com/drduh/YubiKey-Guide#using-keys
I will close this one out!


Multiple nginx-php deployments are not supported

Currently deploying multiple nginx-php or nginx-php-persistent deployments are not supported.
The way the templates values.yaml check works here means that it isn't possible to run more than one.
The line DEPLOYMENT_SERVICETYPE=$line is populated with nginx and php in the while loop based on what is defined in the values.yaml file in the templates here
This causes the next variable DEPLOYMENT_SERVICETYPE_IMAGE_NAME to be empty, as the values used to check the map ${SERVICE_NAME}:${DEPLOYMENT_SERVICETYPE} are set to the hardcoded values of nginx or php for DEPLOYMENT_SERVICETYPE.
Example, given a docker-compose.yaml that is like this
version: '2.3'
services:
  nginx:
    build:
      context: .
      dockerfile: lagoon/nginx.dockerfile
    labels:
      lagoon.type: nginx-php
  php:
    build:
      context: .
      dockerfile: lagoon/php.dockerfile
    labels:
      lagoon.type: nginx-php
      lagoon.name: nginx

  second-nginx:
    build:
      context: .
      dockerfile: lagoon/second-nginx.dockerfile
    labels:
      lagoon.type: nginx-php
  second-php:
    build:
      context: .
      dockerfile: lagoon/second-php.dockerfile
    labels:
      lagoon.type: nginx-php
      lagoon.name: second-nginx

The variable map of MAP_DEPLOYMENT_SERVICETYPE_TO_IMAGENAME would contain the following indexes
nginx:nginx
php:php
second-nginx:second-nginx
second-php:second-php

But since the exec-kubectl-resources-with-images.sh script is sourcing the value DEPLOYMENT_SERVICETYPE from the templates values.yaml, the map lookup for second-nginx and second-php fail because it tries to look up in the map for these indexes
second-nginx:nginx
second-php:php

but they don't actually exist.
Since all of our other templates are single-container templates, this restriction doesn't exist.
We should be able to support this configuration, even if it may not be ideal.
There are other issues with this too that need to be checked with how persistent volumes are used/consumed if a persistent volumes was to be shared between both of those deployments similar to how it is shared with the CLI deployment. Or if the second deployment is meant to have its own persistent volume created, separate to the other deployment.

This can only be addressed when we move to common templates for all services, treating the nginx-php  (and other multi-container pods) with post-template logic.


autocompleteDisabled porperty of MatAutocompleteTrigger doesnt work.

Bug, feature request, or proposal:
Bug
What is the expected behavior?
I should be able to use this property to turn off an autocomplete.
as in here: https://stackblitz.com/edit/angular-3ebz9m-tx7tf8?file=app%2Fautocomplete-overview-example.html
(but without needing to use ngIf and duplicating code)
As per the documentation I should be able to user autocompleteDisabled to turn off the autocomplete
What is the current behavior?
An Error: Can't bind to 'autocompleteDisabled' since it isn't a known property of 'input'
https://stackblitz.com/edit/angular-3ebz9m-6qsqbf?file=app/autocomplete-overview-example.html
What is the use-case or motivation for changing an existing behavior?
being able to have flexible components that can be both autocomplete input fields, or other types of input fields like password fields.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
6.0.2

The docs aren't displaying the property correctly. The input is called matAutocompleteDisabled, not autocompleteDisabled. It should work if you rename your input binding to matAutocompleteDisabled.

@crisbeto that still doesn't work https://stackblitz.com/edit/angular-3ebz9m-rbgurk?file=app/autocomplete-overview-example.html

It seems like the changes haven't been released yet, because https://github.com/angular/material2/pull/11142 is targeted for the next minor release @CurtisDS. I think the confusion comes from the fact that we build our docs from master, which doesn't necessarily reflect what is released.


Selector score ranges that include negative values are not marked as valid

Selector score ranges that include negative values are not detected as "valid range or integer".
Examples:
@p[scores={test=-1..}]
@p[scores={test=..-1.1}]

Fix has been made and added to update 1.4.1.

Fix has been made and added to update 1.4.1.


Removing STL headers from filameshio

Removing STL headers from the filamesh reader. Not sure that the jsbindings were amended correctly. Also still has <functional> from the CString header using it for std::hash specialisation, maybe that could be moved to another header to be included only by internal source files, as std::hash should only be used by STL headers anyway? (I think).

I signed it!

This seems reasonable and I think jsbindings look good too, but you can review our CODE_STYLE.md document and fix up the formatting?

@prideout Sorry I forgot about that, hopefully it's in line with the guide now.


[VS Code] Starting collaboration session: Could not connect to the server. HTTPS connection failed, verify there are no self-signed certificates or custom cert validation policies blocking Live Share.

Error:
Starting collaboration session: Could not connect to the server. HTTPS connection failed, verify there are no self-signed certificates or custom cert validation policies blocking Live Share.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.721

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.37.0

platformName
Linux

platformVersion
4.19.37-parrot1-amd64

Thanks for reaching out. Can you see if this workaround fixes your problem? https://github.com/MicrosoftDocs/live-share/issues/2216#issuecomment-501372336

Closing as dupe of #2216. Thanks for using Live Share!


Support exporting output values for cross-stack refrences

I have not been able to export the output values of a template, as described in http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/outputs-section-structure.html
I aim to get an ouptut section similar to
"Outputs" : {
  "Logical ID" : {
    "Description" : "Information about the value",
    "Value" : "Value to return",
    "Export" : {
      "Name" : "Value to export"
    }
  }
}

Looking at the code, it does not look possible.
Please add this option so we can make cross-stack references.

I propose a pull request to fix missing export.


Added C++ implementation of Median of Medians algorithm

Assigned Issue Number?
#239
Language Opted?
C++
Number of files added or updated?
1
Name of these files?
Algorithms => Searching => KthSmallestSearch => KthSmallestSearch.cpp
Checklist

[x] I was assigned for this otherwise my PR will be rejected.
[x] I've read the contribution guidelines.
[x] I've written code by myself. No Plagiarism allowed
[x] I've referred the correct issue number.
[x] I've filled up this entire template correctly.

:star2: this repository and support.

The code that you have written is straight off scrapped from this website:
https://www.geeksforgeeks.org/kth-smallestlargest-element-unsorted-array-set-3-worst-case-linear-time/
I request you to review the contribution guidelines that we have recently updated.
I thank you for your contribution, but please do make sure that you have worked on it on your own. If not, at least have it implemented it in your own unique way.
@krishrahul98  Waiting on your word to close this PR.

@rachitmanas You have directly copy-pasted this code from Geeksforgeeks that is not acceptable. Please write code by yourself and again raise new PR. That will help you as well.

The code that you have written is straight off scrapped from this website:
https://www.geeksforgeeks.org/kth-smallestlargest-element-unsorted-array-set-3-worst-case-linear-time/
I request you to review the contribution guidelines that we have recently updated.
I thank you for your contribution, but please do make sure that you have worked on it on your own. If not, at least have it implemented it in your own unique way.
@krishrahul98 Waiting on your word to close this PR.

 GeeksforGeeks**K'th Smallest/Largest Element in Unsorted Array | Set 3 (Worst Case Linear Time) - GeeksforGeeks**A Computer Science portal for geeks. It contains well written, well thought and well explained computer science and programming articles, quizzes and practice/competitive programming/company interview Questions.

@hemanth-kotagiri You are free to take any action and you are right. Plagiarism is not good for anyone.

Awesome @krishrahul98 . I just wanted to confirm my privileges for the first time. Sure, next time, I'll take charge.
Thanks!

Awesome @krishrahul98 . I just wanted to confirm my privileges for the first time. Sure, next time, I'll take charge.
Thanks!

@hemanth-kotagiri Yes, sure.


typo: install-adcscertificationauthority -> Install-AdcsCertificationAuthority

This lines might just be removed since they point to using the built-in help. You can see the other cmdlets using this pattern by searching "To see parameters, run the following command"

Thanks for this, I haven't seen this -? so maybe should be removed. Let me know if it doesn't work for you and I can dig to try to find a dev on windows team to look into if it should work.


Add saturday description in front page

- Summary

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

👷 Deploy Preview for festi-loire-roller processing.

Name
Link

🔨 Latest commit
a48637be2747eee530e29a0f4ac901b41d3c55a4

🔍 Latest deploy log
https://app.netlify.com/sites/festi-loire-roller/deploys/66ce021829c9a20008c2a407


Limit usage of SE-0409 (Access Level-on-Import) to only the Testing module

This is a follow-up fix to the changes recently landed in #87 to only adopt the SE-0409 feature in the Testing module/target, rather than in all targets.
The motivation is to fix a build failure in the TestingMacros target when using recent main branch toolchains, which appear to have stronger enforcement that access level of imports within the parameters of public declarations. But I think this is a reasonable fix even if not for that build failure, since our primary motivation for adopting SE-0409 is to carefully curate the modules which are publicly imported by the Testing module. We don't need to worry as much about the other targets in this package since their modules are not used by clients.

@swift-ci please test


static files. folder for image

❓Question
hello
hi i need to change the folder for pictures.
i did so
mediaPrefix: 'img',
but a folder 'static/media'  is still created in the build folder
so I tried to configure the server
server.use('/img', express.static(path.join(__dirname, 'img')));
but for some reason it does not work.
how to do it right

Should work, follow
https://github.com/jaredpalmer/razzle/blob/b8154c00f9bb19827176f1de4d9a05c47725df6d/packages/razzle/config/createConfigAsync.js#L158
to see how it should work

May be copyplugin that causes this


RSpec/ContextWording with nested contexts

How should one use the RSpec/ContextWording with nested contexts? Should it support 'and with/when' such that when I do:
describe 'The program' do
  context 'when configured' do
    context 'and when the user is valid' do
      it 'does the right thing'
    end
  end
end

and run $ rspec spec --format documentation
it shows a readable sentence?
"The program when configured and when the user is valid does the right thing"
Presently my example triggers:
Start context description with 'when', 'with' or 'without'.
If I follow that advice, the documentation is less readable:
"The program when configured when the user is valid does the right thing"

cc @pirj

when configured with valid user will pass the cop and IMO is better description.

@Darhazer I think you are implying that it is configured with the user, but I was saying something more along the lines of:  when the program has a correct configuration and a valid user

It makes sense to open this issue on https://github.com/reachlocal/rspec-style-guide or https://github.com/lelylan/betterspecs, we're just soldiers here, we don't create rules.

@pirj I can appreciate that:
https://github.com/reachlocal/rspec-style-guide/issues/17
There's already a question about the and coupler on https://github.com/lelylan/betterspecs/issues/3#issuecomment-31960370

@pirj I just talked with @bikeonastick (from ReachLocal) tonight and he said that they are no longer maintaining https://github.com/reachlocal/rspec-style-guide

@Darhazer it's ok for the example above but what if we would have more than one nesting?

feature "As a visitor, who has given a purchase link, I should not be allowed to see purchase page" do
  context "when the visitor has an email set up in session" do
    context "and visitor has corresponding user in db" do
      context "and visitor's domain of email is white-listed" do
      end
    end
  end
end

I don't yet have a strong opinion on this, but one thought is that while "and" makes the final docstring seem neat and readable in some cases, it makes reading the spec itself feel a bit more awkward to me. "Here's the context: and {x}" feels strange to me in a way that "Here's the context: when {x}" doesn't.
@IlkhamGaysin I think your example is reasonable, but I'd be more likely to write it as:
feature "As a visitor, who has given a purchase link, I should not be allowed to see purchase page" do
  context "when the visitor has an email set up in session" do
    context "with a corresponding user in the db" do
      context "with a white-listed email domain" do
      end
    end
  end
end

But I'm not convinced one is clearly better than the other. I generally try to avoid having deeply nested contexts when i can so I don't think about this problem that often. I do sometimes end up with something like "with a corresponding user in the db and the user has a white-listed email domain" and just skipping a context level. That may be cheating for the purpose of the discussion though.
One other very minor thought is that if you allow "and" it wouldn't be flagged if there was no parent context string for it to be connected onto, but I confess that I expect that case is exceedingly rare.
Individual users, of course, are free to add 'and' to their rubocop config file as it suits them. I don't have a strong opinion about adding it or not adding it to the defaults at the moment though.

I was actually thinking of adding a when/with and without helpers, e.g.:
describe '...' do
  when 'the visitor has an email' do
    with 'a corresponding user' do
      without 'a confirmation' do
        ...
      end
    end
  end
end

Problem with an and is that it's a Ruby keyword ;)

@pirj looks good! :+1:

Related to #757.
Context Descriptions guideline is now more permissive:

When an example block description is composed with context block descriptions, it should result in a sentence with proper grammar. This typically means the context should start with a term such as 'when', 'with', or 'without'.

Would you agree to configure prefixes for the cop on a per-project basis, or is it a major effort you wouldn't want to deal with, @jcoyne ?
There's a lengthy list of the frequently used prefixes I've gathered from real-world-rails.

I think the advice that "it should result in a sentence with proper grammar" leads one to the use of "and".
"when x is true when y is true" would be properly written "when x is true and when y is true"
Personally, I don't mind configuring my own prefixes, but by default you can't make proper sentences for the documentation formatter without the use of "and"

@jcoyne You can just write a different sentence that reads better.
The program when configured with a valid user does the right thing

Do you think there is something actionable here, or can this be closed?

I think "and when" and "and with" could be added to the default acceptable prefixes.


Add host info in ArrowFragmentGroup

Feature request
Problem
It may be useful to provide host info in a ArrowFragmentGroup, in case we want to know on which host every fragment lies. Currently we can only get instance ids.
Expected behavior.
For example.  A string which contains hosts and fragment ids can be obtained by calling fragGroup->GetFragmentHostIds().
i.e. "h1:1234;h2:6789"

As discussed off the record, we will add a method on arrowfragmentgroup to return a object id -> instance id mapping.

The API already exists, see: https://github.com/v6d-io/v6d/blob/main/modules/graph/fragment/arrow_fragment_group.h#L49-L53


As a curator I (may) want a subclass of "Not latin" that reflects plant style names in ICZN (e.g. Protozoa)

When I add Protozoan names, then they are governed by ICZN, yet they haven't got their act together, so they have Plant endings, so TaxonWorks requires the 'NotLatin' work around.  Technically they are Latin, so maybe we need a subclass of "NotLatin" (subclass so we can maintain the functionality, it works fine to override spelling), that is something like "NotLatin::UnconformingName"?

When protozoan names are used in zoological nomenclature, the names should be normalized according to ICZN rules. We can always use "FemilyGroupNameForm" to indicate the usage of the name.
There is no problem to create a Kingdom with the name "Protozoa"

Sorry for confusion. Protozoa is fine.  Families inside Protozoa are not.

I believe the families should be used with zoological endings, and we can record the original form as a family form under it.

@proceps Thanks, will do. We'll need to move this to the FAQ(esqe) section of docs.taxonworks.org).


Publish 04/14/2021, 10:30 AM

Docs Build status updates of commit 85b2ea1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-MyAnalyticsFeatureConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-UserBriefingConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[FEATURE] add a custom prompt agent

Describe the feature you'd like
I hope to add a custom prompt agent, and even customize the output_ parser, because in non openai llm scenarios, many models do not have good support for langchain's built-in prompt
Additional context
Alternatively, have existing agents support customizing prompts according to certain specifications

Hi @zhengxiang5965,
Could you explain more on how you wanna implement custom prompt agent using this reference?


Add support for Bamboo

https://marketplace.atlassian.com/plugins/com.sysbliss.bamboo.plugins.prepost-build-command/server/overview


Set seccompProfile on Envoy sidecar

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Set seccompProfile in the security context for the containers in the Envoy sidecar istio-validation and istio-proxy so that when istiod is installed with istio cni enabled, the sidecar containers obey the restricted pod security standards https://kubernetes.io/docs/concepts/security/pod-security-standards/
Describe alternatives you've considered
I don't know of any way to do this without setting this
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

As this is quite a simple code change, I've made a pull request here https://github.com/istio/istio/pull/41275


Safari / swiper not working properly

Hello,
I have developed a swiper for a PWA, that has to work also in users' browsers.
It works perfectly on Chrome, but not with Safari.
I have tried with the following devices :

iPhone 8Plus ios version 11.0
iPhone 7 ios version 12.2
iPhone 8 ios version 12.4.1

react-id-swiper version : 2.3.2
Here is what's happening :

The images are loaded, as well as the left and right arrows
The pagination bullets are not loaded (can't see them)
The swipe is possible but very slow, and the loop does not work (if I swipe to the left while I am on the first image, I see black instead of seeing the last image of the slider).

Does anyone know what is going on ?
Thanks a lot !

Hello @mlvrkhn,
I'm not working for the same company anymore.
I manage to find the code, but it seems reallyyy old (and needs to be cleaned)... The version of react-id-swiper was 2.4.0.
Here it is :
`import React, { memo, useEffect} from 'react';
import css from 'client:styles/components/step/content/storiesSlider.scss';
import ReactIdswiper from 'react-id-swiper';
const cx = classNames.bind(css);
export default memo(({children}) => {
const params = {
initialSlide: 0,
pagination: {
el: '.swiper-pagination',
type: 'bullets',
clickable: true
},
navigation: {
nextEl: '.swiper-button-next',
prevEl: '.swiper-button-prev'
},
spaceBetween: 0,
loop: true,
on: {
init() {
const mySwiper = document.querySelector('.swiper-container').swiper;
mySwiper.realIndex = 0;
mySwiper.slideToLoop(mySwiper.realIndex, 0, false);
},
slideChangeTransitionStart() {
const mySwiper = document.querySelector('.swiper-container').swiper;
let $wrapperEl = mySwiper.$wrapperEl;
let params = mySwiper.params;
$wrapperEl.children(('.' + (params.slideClass) + '.' + (params.slideDuplicateClass)))
.each(function () {
let idx = this.getAttribute('data-swiper-slide-index');
this.innerHTML = $wrapperEl.children('.' + params.slideClass + '[data-swiper-slide-index="' + idx + '"]:not(.' + params.slideDuplicateClass + ')').html();
});
},
slideChangeTransitionEnd() {
const mySwiper = document.querySelector('.swiper-container').swiper;
mySwiper.slideToLoop(mySwiper.realIndex, 0, false);
}}
};
useEffect(() => {
	const mySwiper = document.querySelector('.swiper-container').swiper;
	if (mySwiper) {
		mySwiper.$el[0].classList.add(cx('swiper-container-custom'));
		mySwiper.$el[0].children[1].classList.add(cx(['swiper-pagination-custom']));
		mySwiper.$el[0].children[2].classList.add(cx('swiper-button-next-custom'));
		mySwiper.$el[0].children[3].classList.add(cx('swiper-button-prev-custom'));
		mySwiper.update();
	}
});

return (
	<ReactIdswiper {...params}>
		{children}
	</ReactIdswiper>
);

});`
I don't have so much time to investigate more, but let me know if it helps you (try to implement the part with the pagination class maybe...), or if you didn't manage to do it. If you still need help, I'll take more time to elaborate a fresh and clean sandbox.
Best,
Camille

I have this issue wirh safari. What did you do for safari browser?


Implement a disconnected login flow

Users who are unable to run the CLI and a browser on the same machine need a means to login with jsctl auth login.
I think it makes sense to have them copy the URL into a browser, then copy their redirected URL containing the callback state and code into the terminal. The CLI can then use these to request the token as usual.

Closing in favour of https://github.com/jetstack/jsctl/issues/50


[help] want to diff the k8s、k0s、k3s

What happen?
want to help!
I can't diff the k8s、k0s、k3s which the user would like to install or manager.
If there is a best method for me to diff the k8s、k0s、k3s？
/var/lib/sealer/data/my-cluster/rootfs# cat Metadata
{
  "version": "v1.19.8",
  "arch": "amd64"
}

Relevant log output?
No response
What you expected to happen?
I seem to look a method to use a Metadata field or something else to diff k8s、k3s、k0s , but It seems there is no existing a func for me to use.
Another Bug is that I install a v1.24.1 Kubernetes, but the Metadata show the field Version:v1.19.8
How to reproduce it (as minimally and precisely as possible)?
No response
Anything else we need to know?
No response
What is the version of Sealer you using?
No response
What is your OS environment?
No response
What is the Kernel version?
No response
Other environment you want to tell us?

Cloud provider or hardware configuration:
Install tools:
Others:

@starComingup I think adding a filed clusterRuntime is fine for this feature.

@justadogistaken @allencloud thanks for reply, that is a good idea, i will try it!


Should Stream and WritableStream have an operational send?

Currently the Writable and WritableStream types have send be a synchronous method:
https://github.com/thefrontside/effection/blob/3805e41aef9f1b52c4981a5ed658a751f6771b06/packages/stream/src/writable-stream.ts#L3-L7
While this works for many streams which have a synchronous send method (such as Channel), it does not work for other types where send is async. Such as a hypothetical BoundedChannel with backpressure, or a socket where sending messages creates network requests.
An operational form of send is more general, and can apply to more streams. We would change the Writable interface to something like this:
export interface Writable<T> {
  send(message: T): Operation<void>;
}

And using send would require a yield or spawn.
This seems like a good idea, but there are some details that need to be decided on:

Is send a good name for the method? An interface named Writable implies a write method, IMO. We could change the interface to something like Sendable, but that seems odd.
Should we break compatibility for primitives such as Channel and Queue? In that case we need to do this before v2, probably? This is a very annoying and error prone breaking change if we decide on this.
Should we provide a synchronous alternative for those primitives that support a synchronous send? This could be useful for calling into effection from outside of effection-based code.

@cowboyd and I discussed several options during our meeting this morning. Unfortunately none of them are super great. The ideal solution would be to make everything go through operations, but this would incurr major breaking changes everywhere, and it's unclear if this is worth it. Unfortunately in the great tradition of red and blue functions, turning send into an operation seems to infect a lot of things. For example, because atom uses a channel under the hood, its set, update and remove methods need to become operations. Now we have a problem, because bigtest uses those methods a lot and so making this kind of change even in these two project becomes a huge, huge pain in the butt.


How to read from json file

I am trying to get my translation from and api endpoint and the code i have is:
i18n

  .use(Backend)
  .use(LanguageDetector)
  .use(initReactI18next)
  .init({
    fallbackLng: 'it-IT',
    debug: false,
    keySeparator:false,

    interpolation: {
      escapeValue: false,
    },

    react: {
      defaultTransParent: 'div',
      wait: true,
    },

    request: getTranslations('en')

  });

The function getTransaltions() returns json data. Now my question is what i need other in configuration to make it work and read my translations in code.
for example   {t('dashboard.title')}

This module is deprecated, please have a look at i18next-http-backend and this example: https://github.com/i18next/i18next-http-backend/blob/master/example/node/app.js
I think you don't need to define the request option at all...


Add SHOW JOBS WHEN COMPLETED

Add docs for SHOW JOBS WHEN COMPLETED
Closes #5291, #5249.
[ON HOLD until 19.2.0-beta20190909 goes out]

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d0d94aa82d72856f1c305863f1776476c3c309e5/
Edited pages:

v19.2/show-jobs.md

Hi @spaskob, I added you as the engineer to review this PR since you worked on this. To review, you can use Reviewable and check for technical correctness and completeness. If you want to see a rendered version of the changes, you can look at the generated AWS link. Lmk if you have any questions! Thanks :)

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/6609b8423f19a79a5a8f815804ba82c0b725b1ad/
Edited pages:

v19.2/show-jobs.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/490868851d59968caaa2d9e0157e58d055ef96be/
Edited pages:

v19.2/show-jobs.md


Split readme up into smaller files.

The readme has become huge and hard to manage; it needs to be broken up into multiple smaller files.

Yes, perhaps we could look into using a format that will work with a site generator like readthedocs or something similar and have that published to github pages. Should I look into this or have you already started work on it?

Sounds like a nice way to do it. I haven't done anything with regards to this issue. If you'd like to take this on go ahead :)

I investigated this and evalutated two options. One is to use jekyll, it is the most integrated way with github pages. However it involves ruby and I'm not using ruby for anything else so for that reason it was not a good option. I also found docusaurus which is based on nodejs and react and has github pages support. It seems like a good option.
However since we are now moving to eslint I'll focus my efforts there instead and at least initially try to follow how they do the docs over at typescript-eslint.

I haven't had much of a look at docusaurus yet but on surface it looks quite good.
I think typescript-eslint's docs are done the same way as eslint so that sounds like a good way start :)
We probably can leave this repo as is.

I just found docsify. Might also be worth looking into.


Fish-Redux入门教程

Fish-Redux入门教程
简单的fish-redux流程使用教程

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

Codecov Report

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

@@           Coverage Diff           @@
##           master     #242   +/-   ##
=======================================
  Coverage   50.04%   50.04%
=======================================
  Files          47       47
  Lines        1231     1231
=======================================
  Hits          616      616
  Misses        615      615

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 fc05eb8...f660da3. Read the comment docs.

后面等我空些，针对这个文档，做些更加完整的表述。
感谢

同求，Fish-Redux 入门到复杂页面的教程

fish-redux使用

教程：掘金：fish_redux使用详解---看完就会用！

备用：CSDN：fish_redux使用详解---看完就会用！

代码demo地址


Image endpoints

GET endpoint:

Added new handler for fetching previously uploaded images from Urchin.
Added structs to allow handling of responses with compiler support.
Added required functions for the database to fetch images.

DELETE endpoint:

Added new handler for deleting previously uploaded images from Urchin.
Added required functions for the database to delete images.

General update:

Added structs to allow handling of responses with compiler support.
Restructured API to used path variables for the delete endpoints.

I added the required functionality to the app. Also, refactored the GET image endpoint in the admin app.
Improved the handling of the NavBar.
Hope this is fine for you.

I need you to rebase the new main changes on this branch so I can merge!


Easier toggling of "hide cell output"

I find myself hiding the output of quite a few scala cells recently, and going to the cell settings menu is a little slow. Could there be a direct icon/button (for example, an "closed/open eye" icon under the play button to hide output?

Could do.
On Thu, Jun 18, 2015 at 12:41 PM, Sebastian Riedel<EMAIL_ADDRESS>
wrote:

I find myself hiding the output of quite a few scala cells recently, and
going to the cell settings menu is a little slow. Could there be a direct
icon/button (for example, an "closed/open eye" icon under the play button
to hide output?
—
Reply to this email directly or view it on GitHub
https://github.com/wolfe-pack/moro/issues/65.

The icon doesn't change, it's always "open", but should work fine otherwise.


Functions don't seem to work due to a canceled vehicle order on my account.

When I attempt to run any of the functions (e.g. --vehicles, --state, --last_seen, --ota, --poll, --charging_schedule, --live_charging_session,  --live_charging_history) I receive a traceback.
bin/rivian_cli --last_seen
Traceback (most recent call last):
File "/root/rivian-python-api/src/rivian_python_api/rivian_cli.py", line 1214, in
main()
File "/root/rivian-python-api/src/rivian_python_api/rivian_cli.py", line 742, in main
details = order_details(order['id'], verbose)
File "/root/rivian-python-api/src/rivian_python_api/rivian_cli.py", line 129, in order_details
'vehicleId': response_json['data']['order']['vehicle']['vehicleId'],
TypeError: 'NoneType' object is not subscriptable
The trace looks identical for each of these functions. I think this may be because I have a canceled order on my account and it isn't being handled properly. I'm happy to provide additional output if needed. Thanks!

Ok bit of an unusual case with the cancelled order but I added more protection around that area in 7cf05c6dc6ba816d4e93effa1f11a9ffd733ce23

I'm having a similar issue after the fix:
Order details missing key items, found: None Traceback (most recent call last):   File "src/rivian_python_api/rivian_cli.py", line 1232, in <module>     main()   File "src/rivian_python_api/rivian_cli.py", line 765, in main     vehicle_id = vehicle['vehicleId']                  ~~~~~~~^^^^^^^^^^^^^ KeyError: 'vehicleId'
I don't have a cancelled order but I do have an open R1S reservation (R1T was delivered)
I don't see the string vehicleId anywhere in the output from --vehicle_orders

I added protection for the exception but there's nothing I can really do about them returning odd data or not returning proper information. Does it show any of this stuff in their app or site?

No the app/website don't show that. Your change fixes it enough for me though, thank you


Add Whitelist config to Rails/SkipsModelValidations

Rails/SkipsModelValidations
Rails/SkipsModelValidations is a very useful rule as it can help catch some easy to miss gotchas regarding Validation in ActiveRecord.
Sometimes you need exceptions though, for instance in one of our projects we have this rule enabled, but would still like to allow the touch method as we feel it is relatively safe to call even without validation.
Currently to make this happen we have a rubocop config that looks like the following:
Rails/SkipsModelValidations:
  Blacklist:
    - decrement!
    - decrement_counter
    - increment!
    - increment_counter
    - toggle!
    - update_all
    - update_attribute
    - update_column
    - update_columns
    - update_counters

It is hard to tell just glancing at the config what this is doing, but if you look closely you can see we are missing the touch method in the Blacklist. While this works, it exposes us to two issues:

Config is less self-documenting, hard to notice touch is missing without looking up the full list
If new methods are added that skip validation, they will not be part of our Blacklist since we are overriding the blacklist from our upstream config.

I propose adding a new Whitelist config option that lets you explicitly declare that you are ok with certain methods from being ignored by the rule. The equivalent config from above would then be:
Rails/SkipsModelValidations:
  Whitelist:
    - touch

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default. It executes all tests and RuboCop for itself, and generates the documentation.

The cop documentation has to be updated as well.

@bbatsov Updated the docs, but not super familiar with the process so let me know if I need to change anything on it! 😃


CI for Node

Starts the game with a random player

Starts the game with a random player

Initializes with two players

Initializes with two players


docs (ldap) service directory mapping workaround

-- Created ldap_service_directory mapping workaround according to FTI-1758.
-- _include used on the following doc topics:

https://docs.konghq.com/hub/kong-inc/ldap-auth-advanced/
https://docs.konghq.com/hub/kong-inc/ldap-auth/
https://docs.konghq.com/enterprise/2.1.x/kong-manager/authentication/ldap/
Added as a known issue entry to release notes and changelog:
https://docs.konghq.com/enterprise/2.1.x/release-notes/
https://docs.konghq.com/enterprise/changelog/

@GayleNeumann I'm seeing this error in the build:
[11:39:37]   Liquid Exception: Invalid syntax for include tag. File contains invalid characters or sequences: /md//ldap/ldap-service-directory-mapping.md Valid syntax: {% include file.ext param='value' param2='value' %} in /srv/jekyll/app/_hub/kong-inc/ldap-auth-advanced/index.md
I have only tried using an include in the non-plugin hub area of the docs. I'll keep troubleshooting.


[v0.1.0] (Testing) Refactor the current lmcache-test repo to have multiple components

Finished in this commit: https://github.com/LMCache/lmcache-tests/commit/86db7bdb8795f99d09699c3112d3335442ed4f73


Persistence of index

Is there a way to persist an existing itemjs index and reload it again for search later? This would be a great enhancement.
If I see it correctly, the index is built via lunr, so the serialisation might also be possible

This is an excellent suggestion. Implementing serialization of the search index could potentially result even a 10x performance improvement during reindexing operations, such as when a user refreshes the page. The serialized index could be stored locally using localStorage or as a file on the server side for persistence.
There's a notable parallel here with how pandas can swiftly load Parquet files, which, despite containing the same data as CSV files, are loaded significantly faster due to their optimized format. I'm curious to see how much of a performance gain we could achieve in this context with JavaScript.
I'd be interested to hear if this aligns with your thoughts or if there's anything you might want to add.


Update to 1.2.1 does not appear in Homebridge UI nor through npm

Hello,
it seems there is a publish problem for version 1.2.1: the availability of the update is not displayed and proposed in the Homebridge UI interface. Moreover, trying to manually update using npm does not succeed:
MacMini2:~ mrabozee$ sudo npm install -g<EMAIL_ADDRESS>npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR!      /Users/mrabozee/.npm/_logs/2021-02-02T08_55_57_634Z-debug.log
MacMini2:~ mrabozee$

Using "sudo npm install -g homebridge-roomba2@latest" only reinstall version 1.2.0.
MacMini2:~ mrabozee$ sudo npm install -g homebridge-roomba2@latest
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>request-promise has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
+<EMAIL_ADDRESS>updated 1 package in 4.474s

+1 for this.

+1 for me as well.

Is there a way to get this 1.2.1 version to show up for us Homebridge users? Homebridge is fully updated, yet for this plugin, it thinks 1.2.0 is the latest version.

@brettstah @whreams sorry about that, it's now published.


Fallback to zero value for initiatedID in ExternalExecutionInfoFields struct

What changed?
Do not panic if initiatedID is missing when mapping ExternalExecutionInfoFields. Instead fallback to zero values.

Why?
To address https://github.com/uber/cadence/issues/4721

How did you test it?
unit test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build e62e02d2-7745-4d21-b0bb-25b0bc5bdb50

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
91 unchanged lines in 11 files lost coverage.
Overall coverage decreased (-0.01%) to 56.846%

Files with Coverage Reduction
New Missed Lines
%

service/history/execution/mutable_state_task_refresher.go
1
73.82%

common/cache/lru.go
2
92.2%

common/task/parallelTaskProcessor.go
2
92.75%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
55.04%

common/persistence/serialization/parser.go
4
65.41%

common/persistence/serialization/thrift_decoder.go
4
61.22%

service/history/task/fetcher.go
4
86.15%

common/persistence/sql/workflowStateMaps.go
12
85.34%

common/persistence/nosql/nosqlplugin/cassandra/workflowParsingUtils.go
14
83.65%

common/persistence/serialization/getters.go
16
60.6%

Totals

Change from base Build 8b618f7b-b302-4dd9-814d-21045c97973b:
-0.01%

Covered Lines:
82957

Relevant Lines:
145933

💛  - Coveralls


Loading a TensorFlow checkpoint, and turn it into a Keras model

Consider the process of loading a TF checkpoint in TF:
self.sess = tf.InteractiveSession()
self.sess.run(tf.initialize_all_variables())
saver = tf.train.Saver(tf.all_variables())
saver.restore(self.sess, tf_ckpt_path)

and then I am abel to do self.sess.run(stuff) etc.
The problem is, if I do the above in a Keras jupyter notebook, I am not really building a Keras model, but still using a TF model.
How can I change this model into a Keras model (without building the model from scratch again), such that I can follow the directions in https://github.com/transcranial/keras-js to run the model in Keras.js (which is ultimately what I'd hope to do)? Thanks.

+1
Hi @fchollet, I have the same question, it would be nice to convert trained TensorFlow models into Keras models for subsequently exporting model weights and architecture.
Please look into this.
Thanks.

+1

+1 :)

+1

And vice versa. From keras to tensorflow is also important.

bump...

+1

Recently, I also have this problem.

+1
I want a little more :-) after I have it in Keras I want to save inf HDF5 format so I can load it in deeplearning4j. (and execute in a "real world application" with java ... )

How can I change this model into a Keras model (without building the model from scratch again)

Fundamentally, you cannot "turn an arbitrary TensorFlow checkpoint into a Keras model".
What you can do, however, is build an equivalent Keras model then load into this Keras model the weights contained in a TensorFlow checkpoint that corresponds to the save model. In fact this is how the pre-trained InceptionV3 in Keras was obtained.
For instance, you can take a TensorFlow checkpoint that implements VGG16, then build the same VGG16 model in Keras and load the weights from the TensorFlow checkpoint.
It's not always easy: it involves iterating over the variables in the checkpoint and transferring them to the Keras model using layer.load_weights(weights).

From keras to tensorflow is also important.

Turning a Keras model into a TensorFlow checkpoint is easy: a Keras model built with the TF backend is already a TF graph, and you can just save the current TF graph to a TF checkpoint the way you normally would. Maybe see https://blog.keras.io/keras-as-a-simplified-interface-to-tensorflow-tutorial.html

@fchollet  Could you write something or examples on Keras document about this topic?  We all think this is very useful for research from TensorFlow. Thanks!

Say I built a model using keras, took its tensors (model.inputs and model.outputs) and trained the parameters in TF.  Now I want to get that keras model with the trained weights. What's the best way to do that?

What I'm unsuccessfully trying to do is (pseudo operational code)
with tf.Graph().as_default():
    input_tensors = build_the_input_tensors()
    model, _ = build_my_favorite_keras_model(input_tensors, **params)
    with tf.Session() as S:
        tf.train.Saver().restore(S, checkpoint)
        print(S.run(model.trainable_weights[0])) # p1
        model.save(modelfile)
        print(S.run(model.trainable_weights[0])) # p2
# the output is
# > p1 = 0.02982 as it should be
# > p2 = 0.05 as it shouldn't

Interestingly, model.save seems to (re)initialize values before it saves them.  How can this be avoided?

I figured it out.  model.save calls get_session in tensorflow_backend.py:2143.  Therein, if not _MANUAL_VAR_INIT: .. _initialize_variables().   We need to toggle that switch:
import keras.backend as K
K.manual_variable_initialization(True)

I have the same problem recently.

The correct API is layer.set_weights(weights)

How can i convert from tensorflow model to tf.keras model?

Is this possible in 2019?

For instance, you can take a TensorFlow checkpoint that implements VGG16, then build the same VGG16 model in Keras and load the weights from the TensorFlow checkpoint.
It's not always easy: it involves iterating over the variables in the checkpoint and transferring them to the Keras model using layer.load_weights(weights).

It isn't always easy, but I did it for the VGGish model and I want to share a few hints on the steps.  This is just for iterating over the variables in the checkpoint and transferring them to the equivalent model. I assume you've already built that equivalent model in Keras:

This will show you the names of all the layers in a checkpoint and their weight/bias shapes (from stackoverflow)

from tensorflow.python import pywrap_tensorflow
import os

checkpoint_path = os.path.join(model_dir, "model.ckpt")
reader = pywrap_tensorflow.NewCheckpointReader(checkpoint_path)
var_to_shape_map = reader.get_variable_to_shape_map()

for key in var_to_shape_map:
    print("tensor_name: ", key)
    print(reader.get_tensor(key).shape) # Remove this is you want to print only variable names

example output:

tensor_name:  vggish/conv1/biases
(64,)
tensor_name:  vggish/conv1/weights
(3, 3, 1, 64)
tensor_name:  vggish/conv2/biases
(128,)
tensor_name:  vggish/conv2/weights
(3, 3, 64, 128)
tensor_name:  vggish/conv3/conv3_1/biases
(256,)
tensor_name:  vggish/conv3/conv3_1/weights
(3, 3, 128, 256)
...

Look at the list of layers and shapes in your keras model:

model.summary()

example output:

Model: "MWE"

Layer (type)                 Output Shape              Param #
inputs (InputLayer)          (None, 96, 64, 1)         0

conv1 (Conv2D)               (None, 96, 64, 64)        640

pool1 (MaxPooling2D)         (None, 48, 32, 64)        0

conv2 (Conv2D)               (None, 48, 32, 128)       73856

pool2 (MaxPooling2D)         (None, 24, 16, 128)       0

conv3_1 (Conv2D)             (None, 24, 16, 256)       295168
...

Identify equivalent layer names and set weights layer by layer.  It helps if your Keras layers are named similarly to the TF checkpoint layers so that you can loop through.  Here I just demonstrate setting weights on one layer, vggish/conv3/conv3_1 in the TF checkpoint file, or conv3_1  as I've named it in Keras:

Note: it helps to check that the sizes are what you expect.  vggish/conv3/conv3_1/weights has 3x3x128x256=294912 parameters, and vggish/conv3/conv3_1/biases has 256.  Add them together and it equals 295168, the number of parameters in the equivalent Keras layer.  Good stuff.

weights_key = 'vggish/conv3/conv3_1/weights'
bias_key = 'vggish/conv3/conv3_1/biases'
weights = reader.get_tensor(weights_key)
biases = reader.get_tensor(bias_key)
model.get_layer('conv3_1').set_weights([weights, biases])

You can see here that Keras's model.get_layer(<name>).set_weights takes a list of numpy arrays with weights first and biases second.
Warning: it would make sense to check the shapes of model.get_layer('conv3_1').get_weights() to make sure that you are passing weights and biases in the correct order.  If you use the default initializer, however, TF has to calculate a lot of random values and gave me a memory error.  On the other hand, setting weights is fast.

Is it possible in 2021?

Is it possible in 2020?

Is it possible in 2023?

Is this possible in 2024 ?


fs_config_default unit test is flaky on CI

Failed here:
https://github.com/metalbear-co/mirrord/actions/runs/7466234067/job/20317188352?pr=2162#step:36:439
Successful here without any change in the code:
https://github.com/metalbear-co/mirrord/actions/runs/7466234067/job/20317958535

Also feature::fs::advanced::tests::advanced_fs_config_default

I think I remember doing a short investigation of this. The config unit tests may not be properly isolated - some tests are setting values in the current env and those values are later read by other tests when generating config.


add RFC requirement for mountLabel

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

ping @opencontainers/runtime-spec-maintainers

LGTM

@opencontainers/runtime-spec-maintainers  PTAL

LGTM

@opencontainers/runtime-spec-maintainers   PTAL

NACK
I'm :+1: with @wking https://github.com/opencontainers/runtime-spec/pull/934#pullrequestreview-73498295 here.
This would be a conditional MUST (the config generated should've accounted for whether selinux on the host is even supported). Which is needed for the trust model of handing down to the container runtime.
But to add a MUST here now, would be a revision in the spec and would be a compatibility issue for the community to need to update to using this newer version of the spec.

@vbatts: But to add a MUST here now, would be a revision in the spec and would be a compatibility issue for the community to need to update to using this newer version of the spec.

Wouldn't that compatibility issue arise anyway due to deprecating the prestart hooks?


Query Sampling

Hi @vzhong
Thanks for releasing this code.
I am looking for a method that can sample new queries from a DB (Section 2.1 of your paper)
Could you please suggest how to do that?
I guess db2sql.py is what I'm looking for. But it's not being used in generate.py
Thanks,

Hi,
I think you are actually looking for model/ans2sql.py, which includes the sampling logic.
On Jan 9, 2022, 6:57 PM -0800, Abhijeet Awasthi @.***>, wrote:

Hi @vzhong
Thanks for releasing this code.
I am looking for a method that can sample new queries from a DB (Section 2.1 of your paper)
Could you please suggest how to do that?
I guess this db2sql.py is what I'm looking for. But it's not being used in generate.py
Thanks,
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks much! :)


Make cpunks a python package

Ultimately, it should be possible to do this and get all the data necessary for working with cpunks
pip install cpunks-10k

Once this is possible, working with colorpunx also becomes easier because we can install this library into a colorpunx env and hack on both at the same time as necessary.
A series of steps like the following would be ideal.
cd some/work/dir
git clone https://github.com/tnn1t1s/colorpunks/
git clone https://github.com/tnn1t1s/cpunks-10k/

cd colorpunx
python -mvenv venv
source venv/bin/activate
pip install -e .

cd ../cpunks-10k
pip install -e .

From that, it should be very easy to work with notebooks and arrange all the paths to assets accordingly.

Closing as it will be tracked here #11