Super Slow UI on 5.2.*

Describe the bug
After auto-updating to 5.2, my Lens installation became laggy and almost impossible to use, opening absolutely any tab leads to 10-20 second lags and brakes on the UI. I even reinstalled lens.dmg for intel from the official website.
To Reproduce
Steps to reproduce the behavior:

Click on pods (5k+)
Click on deployments (8k+)

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Environment (please complete the following information):

Lens Version: 5.2
OS: OSX
Installation method: .dmg from official site

Can you provide logs from when it is laggy? From the Lens main menu: View -> Toggle Developer Tools, and open the Console tab.
Presumably this only happens for a cluster with a large number of pods or other resources?

Hello, I have also notice slow down of UI, sometimes it even freeze. Lens consume a lot of CPU when switching between Pods / Deployments view. I have two cluster one with 15 nodes and cca 400 pods, seconds with 15 nodes and cca 900 pods.

I have seen this behavior as well.  It will just plan hang.  Depending on load we are talking 19 Nodes and around 350 Pods.  The issue seems most pronounced when searching/filtering.

It started happening with the latest update.

Are these local clusters (i.e. clusters that lens discovers by sync'ing your local kubeconfig), or clusters reached through Lens Spaces?

Local.  The cluster is AWS EKS.  Other teammates (in other locations) have the same issue so we have eliminated a internet connectivity issue.

Local as well, but mine are in GCP.  Definitely started with the latest upgrade.
On Thu, Sep 16, 2021, at 10:38 AM, Andy S wrote:

Local. The cluster is AWS EKS. Other teammates (in other locations) have the same issue so we have eliminated a internet connectivity issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/lensapp/lens/issues/3773#issuecomment-920961667, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD4PPY5XLO44TO4XYTJDQTUCH6N5ANCNFSM5DZUOHNQ.

Hello, On W10 I have like crazy RAM usage (not even connected to a cluster)

Is it possible to have a download link for previous version (5.1.3 for example) ?

Same issue in 5.2.2...

This is while searching for a deployment...

When I compile Lens by my own (running make dev), it works like charm, without any freeze or CPU spike.

Is anyone experiencing this on macOS Catalina?

Can you provide logs from when it is laggy? From the Lens main menu: View -> Toggle Developer Tools, and open the Console tab.
Presumably this only happens for a cluster with a large number of pods or other resources?

I'm sorry, but I can't provide the logs in their pure form.
In my clusters, there are from 5000 to 18000+ pods and 100+ nodes in each. But the most interesting thing is that in version 4.* everything worked perfectly.

For anyone looking for old binaries (at least for mac), see this historical https://github.com/Homebrew/homebrew-cask/commits/0ab6f251d96b12c40b281f69f4deb02e728977e0/Casks/lens.rb
I am affected as well on Big Sur, generally speaking app seems to be very sluggish and not registering clicks
I have rolled back for now to 5.1.3, hopefully it will solve it for now.

I had the same issue and it turned out my Pi-hole was blocking a lot. After disabling all Telemetry settings the speed is like before.
Hope this helps someone.

@ahousseini Thx, disabling telemetry did speed up ui by a lot (at least on 5.0.0)

@maxUo Can you try Lens 5.2.4? We fixed an issue which impacted performance noticed on windows. The problem we fixed could have impacted macOS users too, though I was not able to reproduce your issue to confirm.

@maxUo Can you try Lens 5.2.4? We fixed an issue which impacted performance noticed on windows. The problem we fixed could have impacted macOS users too, though I was not able to reproduce your issue to confirm.

After the upgrade to this version (5.2.4), it got a little better, but there are still strong drawdowns compared to the last 4.x major version.

https://github.com/lensapp/lens/releases/tag/v4.2.0 Works for me))

Upgraded back to the 5.2.4
Slowness (and I mean for example the lack of highlighting on mouseover) by disabling completely all telemetry in settings.
I am behind the pihole as well so that might be influencing the issue (by blocking telemetry dns), so potentially that would be one direction to investigate (in case there is an issue in solving dns just disable it?)

@maxUo what do you mean by drawdowns? Do you mean drawbacks? Are they performance related or do you not like some of the other changes that have been made?

@alekc I am confused with your comment, are you saying that even on 5.2.4 you are experiencing UI slowness unless you turn off telemetry? And by the second comment, should I take you to mean that you are behind a proxy?

I can also confirm that on a Mac 5.2.4 is not better than previous versions.

@Nokel81 yes, exactly that.
With 5.2.4, telemetry on, I've been experiencing the slowness on UI (note, not the slowness on getting the data, but generally registering clicks, mouse movements, etc).
With telemetry off, ui was snappy and working fine (maybe some slowness in getting the actual data, but nothing major).  Pretty much the same as reported by @ahousseini
Just for the sake of test: I've now reenabled the telemetry, and I am getting the slow UI again.
I am behind pihole dns server as well, and as consequence, some dns records are blocked from resolving (probably it might be a scenario in some corporate environments as well if they are behind some proxy).
Once I've disabled pihole, it looks like it's working even with telemetry on (at least for the past 5 min), so it suggests that it might be the case of telemetry trying to perform some actions and blocking the main ui thread.
@andyast could you try to disable telemetry as well (Catalog->preferences->telemetry) and see if that fix the issue (after the app restart)?

fwiw, I have telemetry disabled, but it's still slow.
On Thu, Sep 30, 2021, at 11:22 AM, Alexander Chernov wrote:

@Nokel81 https://github.com/Nokel81 yes, exactly that.
With 5.2.4, telemetry on, I've been experiencing the slowness on UI (note, not the slowness on getting the data, but generally registering clicks, mouse movements, etc).
With telemetry off, ui was snappy and working fine (maybe some slowness in getting the actual data, but nothing major). Pretty much the same as reported by @ahousseini https://github.com/ahousseini
Just for the sake of test: I've now reenabled the telemetry, and I am getting the slow UI again.
I am behind pihole dns server as well, and as consequence, some dns records are blocked from resolving (probably it might be a scenario in some corporate environments as well if they are behind some proxy).
Once I've disabled pihole, it looks like it's working even with telemetry on (at least for the past 5 min), so it suggests that it might be the case of telemetry trying to perform some actions and blocking the main ui thread.
@andyast https://github.com/andyast could you try to disable telemetry as well (Catalog->preferences->telemetry) and see if that fix the issue (after the app restart)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/lensapp/lens/issues/3773#issuecomment-931421728, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD4PP5W47D6REDWE464KXDUER6CRANCNFSM5DZUOHNQ.

After this latest release being completely unusable. I downgraded to 4.2.0 & performance is back to where I expect it.

@maxUo can you try Lens v5.2.6? It has some performance fixes.


[BUG] No module name click._bashcomplete

Describe the bug
The latest version of click rolled out today 12/05/2021 and it doesn't have the _bashcomplete component. Can this be fixed .
I know I can use click==7.1.2, but that's a temporary workaround.

Not sure if click.shell_completion.BashComplete is the replacement — not familiar with the use in Typer today for this, but figured I'd mention it

Looks like it'll be more involved than 1:1 replacement, but certainly achievable
PR is useful to understand how the change works
https://github.com/pallets/click/pull/1622
Official Docs do well to expand upon that with concrete information:
https://click.palletsprojects.com/en/master/shell-completion/

This also breaks https://github.com/openapi-generators/openapi-python-client
pip install click==7.1.2 did not fix it.

Thanks for the discussion and interest! :cake:
I just released Typer 0.4.0, which is compatible with both Click 7 and Click 8. :tada:


Set Serializer is_human_readable to false.

Closes #51
BREAKING CHANGE: This results in a different serialization
for common types e.g. Ipv4Addr.

Do you think it's worth making this configurable?

I am not sure. Maybe there are people who need/want to retain the old behavior, but I don't know why.

I could see it potentially causing issues when trying to interoperate with other CBOR processors. We can always add it later though if it ends up being a problem.


unknown token in expression

cmake can't compile a test program
host: ubuntu 2204 x86_64
app.gradle:
android {
    ndkVersion "25.0.8775105"
    compileSdk 34

    defaultConfig {
        minSdk 27
        targetSdk 34

        ndk {
            abiFilters 'arm64-v8a'
        }
        externalNativeBuild {
            cmake {
                cppFlags '-fpass-plugin=/home/ubuntu/Android/Sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/linux-x86_64/lib64/omvll_ndk_r25c.so'
                cFlags   '-fpass-plugin=/home/ubuntu/Android/Sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/linux-x86_64/lib64/omvll_ndk_r25c.so'
            }
        }
    }
}

C++ compiler (assembler) output:
The output was:
1
No available targets are compatible with triple "aarch64-none-linux-android27"

Compiling the CXX compiler identification source file "CMakeCXXCompilerId.cpp" failed.
Compiler: /home/ubuntu/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++
Build flags: -g;-DANDROID;-fdata-sections;-ffunction-sections;-funwind-tables;-fstack-protector-strong;-no-canonical-prefixes;-D_FORTIFY_SOURCE=2;-Wformat;-Werror=format-security;;-fpass-plugin=/home/ubuntu/Android/Sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/linux-x86_64/lib64/omvll_ndk_r25c.so
Id flags:

The output was:
1
error: <inline asm>:3:11: unknown token in expression
  adr x1, #0x10;
          ^

Compiling the CXX compiler identification source file "CMakeCXXCompilerId.cpp" failed.

config file:
import omvll
from functools import lru_cache

class MyConfig(omvll.ObfuscationConfig):
    def __init__(self):
        super().__init__()

    def obfuscate_string(self, module: omvll.Module, func: omvll.Function,
                               string: bytes):

        return True

#     def anti_hooking(self, mod: omvll.Module, func: omvll.Function) -> omvll.AntiHookOpt:
#         if "nativeobfuscatetest" in func.name:
#             return True
#         return False

#     def anti_hooking(self, mod: omvll.Module, func: omvll.Function) -> omvll.AntiHookOpt:
#         return True

#     def anti_hooking(self, mod: omvll.Module, func: omvll.Function) -> omvll.AntiHookOpt:
#         return True

    def obfuscate_arithmetic(self, mod: omvll.Module,
                                   fun: omvll.Function) -> omvll.ArithmeticOpt:
        return True

    def break_control_flow(self, mod: omvll.Module, func: omvll.Function):
        return True

    def flatten_cfg(self, mod: omvll.Module, func: omvll.Function):
        return True

    def obfuscate_constants(_, __, func: omvll.Function):
        return True

    def obfuscate_struct_access(self, _: omvll.Module, __: omvll.Function,
                                      struct: omvll.Struct):
        return True

    def obfuscate_string(self, _, __, string: bytes):
        return omvll.StringEncOptStack()

@lru_cache(maxsize=1)
def omvll_get_config() -> omvll.ObfuscationConfig:
    """
    Return an instance of `ObfuscationConfig` which
    aims at describing the obfuscation scheme
    """
    return MyConfig()

It seems the toolchain you're using (NDK clang on linux-x86_64) does not support the specified target architecture (aarch64-none-linux-android27). Could you please provide the output of the following command?
$ /home/ubuntu/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ -print-targets

Is there aarch64-linux-android27-clang++ under /home/ubuntu/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin?

here is my gradle project with all sensitive information removed.
project.tar.gz
my environmental varibles are:
ubuntu@ubuntu:~/AndroidStudioProjects$ echo $LD_LIBRARY_PATH
/home/ubuntu/Android/Sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/lib64
ubuntu@ubuntu:~/AndroidStudioProjects$ echo $OMVLL_CONFIG
/home/ubuntu/AndroidStudioProjects/NativeObfuscateTest3/ollvm-cfg.py
ubuntu@ubuntu:~/AndroidStudioProjects$ echo $OMVLL_PYTHONPATH
/home/ubuntu/AndroidStudioProjects/NativeObfuscateTest3/Python-3.10.7/Lib

I followed the documentation found here and the version of o-mvll so file was release 1.0.3.
thanks for your kind help :)

Thanks for the output. We are evaluating a few solutions, meanwhile would you disable break_control_flow in the config file for now? We tested it internally and it works correctly. We experienced other orthogonal issues in the execution though, so you may need to disable obfuscate_string too.

Thanks for your advice. Code now compiles and executes correctly. I also found that obfuscate_string will still work provided that obfuscate_string returns omvll.StringEncOptGlobal() rather than omvll.StringEncOptStack(). Besides, anti_hooking isn't working either.

obfuscate_string works if the config file returns omvll.StringEncOptGlobal() and there's only one file contains string literals. Otherwise, errors are emitted by the linker because of duplicate symbol. For example, if two source files, say foo.cpp and bar.cpp, are compiled into one shared library and each contains one string literal, the linker will report the following error.
  ld: error: duplicate symbol: __omvll_decode
  >>> defined at foo.cpp
  >>>            CMakeFiles/nativeobfuscatetest3.dir/foo.cpp.o:(__omvll_decode)
  >>> defined at bar.cpp
  >>>            CMakeFiles/nativeobfuscatetest3.dir/bar.cpp.o:(.text.__omvll_decode+0x0)

Dear lry127,
Many thanks for your input. We managed to reproduce the issue but couldn't find a quick solution to it. Our focus right now is on iOS, so it might take a while until we can fix this, but eventually we will.
Best regards,
Marc

Awesome! Please let us know if you need any assistance, we'd be more than happy to help!

Re-opening the issue as there's a bug that we have to figure out at some point, even if it doesn't have top prio

Closing this, as implicitly solved by bd1440f5ce8660781113d9ebb31d5852873b6e61, thanks @lry127 for the additional feedback.

Hi @antoniofrighetto ,
I kept this open because of the issue related to break_control_flow. So I am reopening it and changing the title to avoid future confusion.
BR,
Marc


Don't erase custom properties from Bar Chart data set

Expected Behavior
Custom properties in dataset for barcharts ARE NOT ERASED and AVAILABLE on 'customMouseOver' as a callback argument.
Current Behavior
Custom properties in dataset for barcharts ARE ERASED and NOT AVAILABLE on 'customMouseOver' as a callback argument.
Possible Solution
Just handle the dataset the same way as you do for Donut chart without erasing additional properties from the dataset with cleanData method.
Use reduce instead of map to format the data you need in the way you like, but don't erase the custom properties.
function cleanData(data) {
            let cleanData = data.reduce((acc, d) => {
                // Skip data without quantity
                if (!d[quantityLabel]) {
                    return acc;
                }

                d.quantity = +d[quantityLabel];
                d.name = String(d[nameLabel]);
                d.percentage = d[percentageLabel] || null;
                acc.push(d);

                return acc;
            }, []);
            let totalQuantity = sumValues(cleanData);
            let dataWithPercentages = cleanData.map((d) => {
                d.percentage = String(d.percentage || calculatePercent(d[quantityLabel], totalQuantity, percentageFormat));

                return d;
            });

            return dataWithPercentages;
        }

Steps to Reproduce

Build a Bar Chart with a dataset containing some custom properties eg.

var dataSet = [{
  value: 23,
  name: 'some label name',
  myCustomProp: 'my custom value'
 },
 ...
];

Add 'customMouseOver' event listener on Bar chart instance:

this.barInstance.on('customMouseOver', (data) => {
      tooltip.show();
});

Check with debugging tools what data argument contains. It'll be:

[{
 value: 23,
 name: 'some label name'
}]

myCustomProp was erased by cleanData method from bar.js:

function cleanData(originalData) {
            let data = originalData.map((d) => ({
                    value: +d[valueLabel],
                    name: String(d[nameLabel])
                }));
            let dataZeroed = data.map((d) => ({
                    value: 0,
                    name: String(d[nameLabel])
                }));

            return {data, dataZeroed}
}

Context
I need to add custom properties to dataset to access them on mouseover, but because of _cleanData method in bar.js all custom properties I've added are deleted.
I'm performing the same operation on donut chart and there I'm completely free to use whatever properties I like as far as they fulfill the donut chart dataset requirements.
Your Environment

Version used: 2.0.0
Browser Name and version: Chrome 60
Operating System and version (desktop or mobile): Windows 7 desktop
Link to your project:

The solution:
Replace this:
function cleanData(originalData) {
            let data = originalData.map((d) => ({
                    value: +d[valueLabel],
                    name: String(d[nameLabel])
                }));
            let dataZeroed = data.map((d) => ({
                    value: 0,
                    name: String(d[nameLabel])
                }));
            debugger;
            return {data, dataZeroed}
        }

with this:
function cleanData(originalData) {
            let data = originalData.reduce((acc, d) => {

                d.value = +d[valueLabel];
                d.name = String(d[nameLabel]);
                acc.push(d);

                return acc;
            }, []);
            let dataZeroed = data.map((d) => ({
                value: 0,
                name: String(d[nameLabel])
            }));

            return { data, dataZeroed };
        }

Hi @piotrsucharski , thanks for the issue and the fix!
I will try to create a PR as soon as possible with it.
Thanks a lot!

Fixed and merged on master!
Thanks again @piotrsucharski !

This has been included on the latest release!


Multiline imports don't respect indentation settings

System information

OS: Windows 10 x64
TypeScript Hero version: 2.1.1
VSCode version: 1.20.0-insider

Description
The organize-imports-command still uses spaces to indent multi-line imports, as described in #361
Steps to reproduce

Set indentation to tabs in a TypeScript-File
Create a long import-statement.
Use organize-imports-command

The multiline import now uses spaces instead of tabs, the indentation size seems to be respected.
I am opening a new issue, since the old one has been closed with the release of the new version.
However, I am still facing this issue and did not find any setting for this.

@buehler Is there a way we can find detect in current vscode setting of tabs or spaces in namedImports.ts ?

My current workaround is to do import * as config from './config' rather than importing a bunch of things explicitly.

@buehler Any news on this?
The referenced PR in node-typescript-parser has been merged on 23 Feb, but I am still facing this issue.
Is there any setting I am missing?


Pre built node

In my last test, parsoid isn't loading:

When I list the contents of /usr/local/bin, I see the symlinks to node and npm are red. So it seems they are breaking. I might try this again using full paths (/usr/local/bin/node-v0.12.7-linux-x64.tar.gz/bin/node node).
Also note the npm command was not recognized during script execution:
******* Installing parsoid *******
VE.sh: line 60: npm: command not found
******* Testing parsoid *******
VE.sh: line 66: npm: command not found

I just realized I had the full tar file name in the symlinks instead of the extracted directory. Going to try this again with the corrected symlinks.

That's the same error as we were getting on kmservices. See that issue for
some possible help.
On Aug 24, 2015 6:44 PM, "Daren Welsh"<EMAIL_ADDRESS>wrote:

I just realized I had the full tar file name in the symlinks instead of
the extracted directory. Going to try this again with the corrected
symlinks.
—
Reply to this email directly or view it on GitHub
https://github.com/enterprisemediawiki/Meza1/pull/162#issuecomment-134415873
.

Test successful. This seems to cut VE.sh by about 20 minutes. Addresses #133.

I ran this, and everything is working. However, this was my first run of installing on a 64-bit VM on my Windows machine and it was way faster. Times below:
yums: 12 min
apache: 3 min
php: 8 min
mysql: 1 min
mediawiki: 5 min
extensions: 2 min
VE: 8 min
ES: 3 min

For comparison, previous install times for steps including building from source:
apache.sh: 12 minutes
php.sh: 32 minutes
VE.sh: 35 minutes

This tells me that percentage of time taken on each step has stayed about the same. However, for a slower machine (with the same internet connection) the VE.sh step would probably still take about 8 minutes, so this change is a big improvement for slower machines. My Mac thanks you.
@darenwelsh unless you think we need further testing I'm go to merge this.


Break dependency of tools/lib/src from lib/src/commands/

This is considered an anti-pattern since it creates circular dependency within this large package. We compile each command separately internally for performance.
Instead of consuming the verbose flag, we can emit stack trace as informational message which only gets printed in verbose mode.
I also added a test for printing the stack trace.

It sounds like we may also need a test to avoid introducing these imports. See https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/general.shard/forbidden_imports_test.dart

Neat! I did not know we have one of these. I will add a test there.

Done. PTAL.


[M1] Write function spec for exercise_plan

Implemented in https://github.com/UBC-MDS/bmi-calculator-python/pull/29


Add Symfony 7 to test matrix

Follows

#130

Blockers:

https://github.com/symfony/panther/pull/604

Closed via #134


MSAL.js login reuses previous user token

We have a hybrid application that is first asking users to enter their email address, then based on whether their email domain supports the Federated Authentication feature we direct them to Azure AD B2C login. If it doesn't we ask them for their internal application username and password stored in the database. We are not using B2C "local accounts" feature, only external IDPs. Currently we only support Azure AD as external IDP, but we are planning to introduce others. Since we only support one IDP per customer we are using direct-sign-in and passing user's email as login_hint and their email domain as domain_hint in MSAL.js call. Domain_hint is then mapped in custom policies to their Azure AD tenant sign-in user flow. So users are not presented by the B2C screen, but are taken directly to the external IDP login.
We want to support SSO and single sign out is out of question, so if user signs out of our app they are not signed out of their respective Azure AD completely and can still use it with other applications.
The issue is that if a user browser is shared by multiple employees, which is a very likely scenario with our customers, if one user logs out of their application and then 2nd user tries to login to the same Azure AD domain, but with a different email, although I can see their tokens deleted from local storage, they are still able to sign-in with previous user token, without even being presented with a login prompt, even though we pass a different login_hint.
Adding prompt: 'login' doesn't help and makes no difference at all, possibly because we are using external IDP.
In the Chrome browser tools Console after "aquiring token interactive" I see the message "user session already exists, login not required", although login_hint in the MSAL request has changed.
I understand that session is managed cloud-side in the AAD, and if there is an exiting AAD session it reuses it, but shouldn't it recognize that login_hint has changed?

const loginRequest = {
scopes: ["openid profile"],
loginHint<EMAIL_ADDRESS>- changes dynamically based on user email
extraQueryParameters: { domain_hint: 'heliosb2c_com' } - changes dynamically based on email domain
};
Any ideas?

@MarinaGurevich Is there an amount of time that affects this behavior? Could you also provide the version of MSAL you are using? Using our issue templates can help us diagnose where the problem is.
@nickgmicrosoft Can you take a look at this? Is it related to the B2C service?

@MarinaGurevich Is there an amount of time that affects this behavior? Could you also provide the version of MSAL you are using? Using our issue templates can help us diagnose where the problem is.
@nickgmicrosoft Can you take a look at this? Is it related to the B2C service?

I am reproducing the issue with the sample pack https://github.com/Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp I modified for two subsequent users signing in. This is MSAL.js 1.0 since 2.0 was not supported by B2C till now.
I did not measure the time. It is definitely related to the Azure AD session still going, but for our corporate Azure AD where multiple applications are connected to it I think there always be a session going.
Personally I just found a workaround to pass prompt: 'login' in MSAL.js request if user email changes, but for that to work with B2C external IDP such as Azure AD I had to add  to the AAD Claims Provider.
But other people might still have this issue.

I am reproducing the issue with the sample pack https://github.com/Azure-Samples/active-directory-b2c-javascript-msal-singlepageapp I modified for two subsequent users signing in. This is MSAL.js 1.0 since 2.0 was not supported by B2C till now.
I did not measure the time. It is definitely related to the Azure AD session still going, but for our corporate Azure AD where multiple applications are connected to it I think there always be a session going.
Personally I just found a workaround to pass prompt: 'login' in MSAL.js request if user email changes, but for that to work with B2C external IDP such as Azure AD I had to add  to the AAD Claims Provider.
But other people might still have this issue.


Fix dead link in DatePicker documentation page

What package(s) are you using?

[ ] carbon-components
[x] carbon-components-react

Summary
In this section of the Carbon docs, there is a dead link in the text that reads:

See the date picker code documentation for more info.

The href for this link 404's: https://github.com/carbon-design-system/carbon/blob/master/packages/components/src/components/date-picker/README.md
Relevant information
After checking the Storybook and other places for a general overview on the DatePicker component, I'm not sure how this should be updated. With some direction I'd be happy to create a PR.

@aflashyrhetoric I cannot find the dead link anymore. We may have updated Date Picker usage docs since this issue was created. Also, there's a quick way to open a PR for things like this at the bottom of the page if you come across something like this in the future.

Awesome! Thanks so much for your awesome work in maintaining this - apologies, I'll try to submit a direct PR in the future!


Impossible to push viewController

New Issue Checklist

I have read the guidelines for contributing and I understand:

[x] My issue is happening in the latest version of SideMenu.
[x] My issue was not solved in the README.
[x] My issue can not be answered on stackoverflow.com.
[x] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[ ] My issue is reproducible in the demo project.

Issue Description
Hello and thanks for this great feature.
Here is my problem, hope it isn't that hard.
I am not using any storyboards and I have a navigationController having the rootViewController a tabbar. I'm setting this navigationController as the rootViewController of the window in appdelegate.
The sideMenu is integrated in the tabBar's viewDidLoad having the rootViewController a viewController.
The thing is that when swiping from left to right, nothing happens, which is normal because I didn't add any pan gestures, but when I'm trying to push a viewController from one of my tabbar's childs, the UI freezes and when i drag from left to right a menu appears from the right side. Too bad that you can't send videos on github, this bug is funny.

You’ll need to recreate the issue using the example project and share it
with me since there may be a bug in your other app code that’s causing this
problem. I’m only troubleshooting issues that are demonstratable in the
example project.
On Sat, Jul 7, 2018 at 9:14 AM Mihailstoica350<EMAIL_ADDRESS>wrote:

New Issue Checklist
I have read the guidelines for contributing
https://github.com/jonkykong/SideMenu/blob/master/.github/CONTRIBUTING.md
and I understand:

My issue is happening in the latest version of SideMenu.
[x ] My issue was not solved in the README
https://github.com/jonkykong/SideMenu/blob/master/README.md.
[x ] My issue can not be answered on stackoverflow.com.
My issue is not a request for new functionality that I am
unwilling to build and contribute with a pull request.
My issue is reproducible in the demo project
https://github.com/jonkykong/SideMenu/tree/master/Example.

Issue Description
Hello and thanks for this great feature.
Here is my problem, hope it isn't that hard.
I am not using any storyboards and I have a navigationController having
the rootViewController a tabbar. I'm setting this navigationController as
the rootViewController of the window in appdelegate.
The thing is that when swiping from left to right, nothing happens, which
is normal because I didn't add any pan gestures, but when I'm trying to
push a viewController from one of my tabbar's childs, the UI freezes and
when i drag from left to right a menu appear from the right side.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jonkykong/SideMenu/issues/372, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFdl_xjfC8ksYR0aajLv2SJaZdrckNITks5uEN51gaJpZM4VGaNV
.


type checking order

I'm getting errors caused by checking whether a term is present within data that is not a list or dict.
/api/buy_feed/api_client.py line 615
if (data is not None and
    klass.attribute_map[attr] in data and
    isinstance(data, (list, dict))
    ):

should (I think) be reordered to read:
if (data is not None and
    isinstance(data, (list, dict)) and
    klass.attribute_map[attr] in data
    ):

which suppresses the error, though the module then fails to do anything with the bytestream that the buy.feed api returns.

Great to hear from you. Which buy_feed call and with what parameters? Please supply a snippet of code that triggers the problem.

Will you be supplying additional information, or should I close the issue?

This issue has gone cold. Lacking more information to work with, I'm closing the case.


ARROW-1124: Increase numpy dependency to >=1.10.x

While we could still build with NumPy>=1.9 for Python 2, Python 3 builds
require >= 1.10 due to a bug in the C-headers.
Change-Id: I0f9e0ad72e4ce4b1c6b44883d5781347d33f7e5b

Appveyor build: https://ci.appveyor.com/project/xhochy/arrow/build/1.0.350


enable console plugin on operator

Signed-off-by: Vinayakswami Hariharmath<EMAIL_ADDRESS>

/cc @naveenpaul1


Enable lang testcases depend on table type & related bug fixes

Purpose

Fix part of https://github.com/ballerina-platform/ballerina-lang/issues/22334 ..

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #23115   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 e254148...1135ad5. Read the comment docs.


Remove for-of loops (fixes #31)

We still use the spread operator, but that seems to compile to using just arguments.

Want to publish a new v and update styled-components too?


Potential Security Vulnerability

When at a recent hackathon we discovered a potential security vulnerability in the scaffold-eth framework. We were able to reproduce on 2 completely separate environments. Not sure the most appropriate way to contact the right collaborators.

Feel free to create a group for it on Telegram (@carletex, @shiv_bhonde, @solidixen)


Fix wrong documentation on extraPorts hostPort property

PR Description
Correct extraPorts hostPort property.
Which issue(s) this PR fixes

Notes to the Reviewer
PR Checklist

[x] CHANGELOG.md updated

 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.

Thanks @tpaschalis great catching. I pushed the change.


Docker container conflict with nginx

I tried the docker image and mapped port 80 (443) of the container to 8090 (8091) in my existing nginx.
Now I can access the default page at http://[mydomain], but unable to access the admin page as it always returns https://<IP_ADDRESS>/admin. Please assist.

I think things should work if you change API Umbrella to listen on the same ports you're trying to publicly map the host ports to (while maybe not ideal, this prevents API Umbrella from getting confused on what its public URLs should be). So for your Docker setup, modify your config/api-umbrella.yml file to contain:
http_port: 8090
https_port: 8091

And then run:
docker run -d --name=api-umbrella -p 8090:8090 -p 8091:8091 -v "$(pwd)/config":/etc/api-umbrella nrel/api-umbrella

I think that should hopefully do the trick, but let us know if not.

@GUI , the port is mapped now, but when I typed https://[mydomain]/admin it stills returns to https://<IP_ADDRESS>:8091/admin then failed to access. I tried to set the hostname in config file as below:
"
hosts:

hostname: gate.ahamove.com
default: true
"

Please help

It turns out our case is quite different because we have a such kind of load balancer (nginx), hence I disabled the default ssl by
http_port: 8090
override_public_http_port: 443
override_public_http_proto: https

then run:
docker run -d --name=api-umbrella -p 8090:8090 -p 8091:443 -v "$(pwd)/config":/etc/api-umbrella nrel/api-umbrella
after that, I configured my nginx to redirect to port 8090, then it works nicely.
Thanks.

@thucnc can you give Nginx config that you used for?


Implement tooling for python formatting

There is currently no python formatting tooling in the repository, although it presumably follows the Google Python style guide.
I'd like to propose using yapf:

It seems to be the standard for Google open source projects.
Does require some reformatting of code (based on my testing, might be missing a flag or something)
Is fairly well maintained.

The only issue is that yapf currently doesn't support match-case statements due to some third-party dependencies being unmaintained. This is being tracked in this issue and some work has been going on recently in this PR, but nothing has been finalized yet. This means yapf fails when formatting gematria currently. Building from a fork seems like it should work in the mean time.
Noting here that other formatters like black or pyink don't have this issue.

The PR has been merged. Just waiting for a release to be cut (which was mentioned in the PR as supposedly happening soon) so that I can open a PR here (assuming no objections to using yapf for formatting).


Price Finalize Opcodes Based On Operand Types

Prelude
This PR presents initial suggestions for revised finalize pricing. This is in no way final or authoritative, but rather a basis for community discussion on how pricing of operations in finalize scopes should work.
Motivation
All opcodes currently have a fixed pricing regardless of inputs. For opcodes that encapsulate simple operations, this is
acceptable. However, for opcodes that encapsulate complex computations such as cryptographic hashes or mapping reads & writes, runtime is significant and affected non-trivially by the size of the opcode's operands.
This motivates pricing opcodes based on the size of their operands so that network performance is not impacted by inefficient or malicious use of complex opcodes.
Analysis
To quantify the runtime costs of Aleo opcodes, a benchmark analysis of each opcode was performed on hardware with
recommended validator specifications (Intel Xeon Platinum 8175 - 128CPU - 128GB RAM)
The analysis can be found here (Credit: @miazn - @iamalwaysuncomfortable)
Results
Key results from the benchmark analysis are summarized below.
Simple Opcodes
Most opcodes have runtimes under 10000 nanoseconds, with most in the range of 1000-3000 nanoseconds.
Outliers include:

mul(field,field) ~ 100000 ns
inv ~ 10000 ns
div(field,field) ~ 15000ns
squareroot ~ 25000ns
pow(field) ~ 15000ns

Hashes & Commits

Each hash family has a base runtime which increases in a (mostly) linearly fashion with input size.
BHP & Poseidon hash families have the highest runtime per byte. The remaining hash families have parity in runtime
per byte.

Set/Get/Contain

Sets are the slowest, having a similar runtime per byte as BHP hashes.
Get & Contain are faster, having a similar runtime per byte.

Initial Pricing Conclusions
1. Pricing should allow finalize scopes to be affordable to use
One of the core motivations of this PR is to discourage overuse of high runtime opcodes in finalize scopes to ensure
network performance and stability through pricing these opcodes per byte.
The other core motivation is to price opcodes fairly so that effective use of this scope for useful applications is
affordable for the foreseeable future. This motivates pricing computationally cheap opcodes at an affordable flat rate.
In pricing both "simple" and "complex" opcodes, this PR suggests that 1 second of runtime in a finalize scope should
cost roughly ~100 credits.
2. Flat Pricing for Simple Opcodes
Runtimes for simple opcodes are low enough to justify grouping these opcodes together and pricing them at a flat low
rate. The rate in this PR is proposed to be 500 microcredits per opcode as an average pricing.
Outliers
Suggested pricing per byte for outliers mentioned above:

mul(field,field) ~ 10000 microcredits
inv ~ 1000 microcredits
div(field,field) ~ 1500 microcredits
squareroot ~ 2500 microcredits
pow(field) ~ 1500 microcredits

3. Byte based pricing for complex opcodes (cast, commit, hash, set/get/contain)
For Casts, Commits, Hashes and GET/SET/CONTAIN operations, pricing should be based on the number of bytes in the operands. Since each opcode has a base cost each should follow an equation of base_cost + cost_per_byte * num_bytes.
6 cost tiers are proposed based on benchmark analysis:

BASE Hash Tier: 10,000 + 30*input_bytes (i.e. 0.8 microcredits for BASE hashes over 8192 bytes)
BPH Hash Tier: 50,000 + 300*input_bytes (i.e. 16 microcredits for BHP hashes over 8192 bytes)
PSD Hash Tier: 40,000 + 75*input_bytes (i.e. 4 microcredits for PSD hashes over 8192 bytes)
CAST Tier 500 + 30*input_bytes (i.e. ~0.25 microcredits for CASTs over 8192 bytes)
GET/CONTAIN tier: 10,000 + 10*input_bytes (i.e. 0.8 microcredits for GETs/CONTAINs over 8192 bytes)
SET Tier: 10,000 + 100*input_bytes microcredits/byte (i.e. 2.4 microcredits for SETs over 8192 bytes)

SET operations are limited to under 10 invocations per finalize block, so despite incurring a heavy cost per byte, they are priced lower due to their central importance for Aleo applications.

Converting the base to mainnet as fee changes will break verification of blocks on the current network.

^Note: The cost of inv and sign.verify was increased in this commit.

(Leaving a gist for myself)
Fee amount changes based on analysis and benchmarks (see Python notebook)
3 considerations:

Runtime: Target was 1 sec of runtime equates to ~100 credits in fees
Flat fee: Flat fee for fixed and common opcodes
Operand sizes: Based on size of inputs. Tiered system


证书一直生成失败  怎么解决？

[OK]  安装 SSL 证书生成脚本 完成
[Sun Oct  9 13:32:02 UTC 2022] Changed default CA to: https://acme-v02.api.letsencrypt.org/directory
[Sun Oct  9 13:32:03 UTC 2022] Using CA: https://acme-v02.api.letsencrypt.org/directory
[Sun Oct  9 13:32:03 UTC 2022] Create account key ok.
[Sun Oct  9 13:32:03 UTC 2022] Registering account: https://acme-v02.api.letsencrypt.org/directory
[Sun Oct  9 13:32:04 UTC 2022] Registered
[Sun Oct  9 13:32:04 UTC 2022] ACCOUNT_THUMBPRINT='mMOpiMi6T_WO6vJFLS-ivbId2a5tws3zI-BJc03lj3g'
[Sun Oct  9 13:32:04 UTC 2022] Creating domain key
[Sun Oct  9 13:32:04 UTC 2022] The domain key is here: /root/.acme.sh/www.aweikn.cn_ecc/www.aweikn.cn.key
[Sun Oct  9 13:32:04 UTC 2022] Single domain='www.aweikn.cn'
[Sun Oct  9 13:32:04 UTC 2022] Getting domain auth token for each domain
[Sun Oct  9 13:32:04 UTC 2022] Create new order error. Le_OrderFinalize not found. {
"type": "urn:ietf:params:acme:error:rateLimited",
"detail": "Error creating new order :: too many certificates (5) already issued for this exact set of domains in the last 168 hours: www.aweikn.cn, retry after 2022-10-10T11:54:14Z: see https://letsencrypt.org/docs/duplicate-certificate-limit/",
"status": 429
}
[Sun Oct  9 13:32:04 UTC 2022] Please add '--debug' or '--log' to check more details.
[Sun Oct  9 13:32:04 UTC 2022] See: https://github.com/acmesh-official/acme.sh/wiki/How-to-debug-acme.sh
[Sun Oct  9 13:32:04 UTC 2022] Using CA: https://acme-v02.api.letsencrypt.org/directory
[Sun Oct  9 13:32:04 UTC 2022] Single domain='www.aweikn.cn'
[Sun Oct  9 13:32:04 UTC 2022] Getting domain auth token for each domain
[Sun Oct  9 13:32:05 UTC 2022] Create new order error. Le_OrderFinalize not found. {
"type": "urn:ietf:params:acme:error:rateLimited",
"detail": "Error creating new order :: too many certificates (5) already issued for this exact set of domains in the last 168 hours: www.aweikn.cn, retry after 2022-10-10T11:54:14Z: see https://letsencrypt.org/docs/duplicate-certificate-limit/",
"status": 429
}
[Sun Oct  9 13:32:05 UTC 2022] Please add '--debug' or '--log' to check more details.
[Sun Oct  9 13:32:05 UTC 2022] See: https://github.com/acmesh-official/acme.sh/wiki/How-to-debug-acme.sh
[ERROR]  SSL 证书生成失败

同样的问题？证书过期了

修改 install.sh 脚本: githubusercontents.com  -> githubusercontent.com

我也遇到同样的问题了，按照@amsz 的示例替换后仍未解决

too many certificates (5) already issued for this exact set of domains in the last 168 hours。
一周后再试

是否修改过端口，然后失败的？如果是，临时的办法是改回去，更新证书，再改回去。

证书一直生成失败 是否可以添加手工导入选项？

申请的证书太多了，等。


runtime: Move main to a separate namespace

This ensures main has the same access to identifiers as other functions.
This serves to prevent cases where generated code is incorrect when something is used outside of main and prevent bugs like the one #427 fixed.

thanks


Is it possible to view the aggregate difference between specific layers?

Suppose I have a Dockerfile with many layers, then the following
RUN apk add --virtual .build-deps some package names
RUN pip install some python package names
RUN apk del .build-deps

Then perhaps some more layers.  Is there a way in dive to see the aggregate changes from the start of those three layers to the end, such that I can see what to COPY in a multi-stage build?  As far as I can tell, I can only see the diff from the start to the end of a single layer, or from the start of the most base docker image(?) to the end of the current layer, which has a great deal of noise.
Note that I could change the Dockerfile to read something like
RUN apk add --virtual .build-deps some package names \
  && pip install some python package names \
  && apk del .build-deps

But that involves installing the build requirements every time the python package requirements change, rather than simply reusing a previous layer.  (I'm also aware of buildkit's cache mounts and --no-cache for apk, but they're not relevant enough to the question to muddy the examples).

Just to clarify, you'd like to view aggregate changes from an arbitrary start layer, to an arbitrary end layer, instead of seeing the diff from the beginning to the current layer?
If so, that's currently not possible.

You've understood me perfectly, and also saddened me.  Thank you.
In that case I'll leave this issue open as a feature request.


Broadcast a message when an item is listed or bought.

Describe the feature
It would be nice to be able broadcast a message when an item is listed or bought. For example, if i buy/sell 64x oak planks, the plugin should broadcast "Plexi09 has purchased/listed 64x oak planks for 1 000$ in the auction house!"
Other
No response

A discord webhook that does the same would be cool to.

You'll just be able to run commands as well so you can use any plugin that hooks to Discord like DiscordSRV


Refresh authentication token fails when two request arrive at the same time

Protect against that in the API and allow only one request per user at the same time
Implement a similar check in the client(s) so that when a token expires and multiple requests are being sent, only one of them is triggering a token refresh and they all wait for the result

fail: OpenSky.API.Controllers.AuthenticationController[0]

      Error exchanging OpenSky token.

      Microsoft.EntityFrameworkCore.DbUpdateConcurrencyException: Database operation expected to affect 1 row(s) but actually affected 0 row(s). Data may have been modified or deleted since entities were loaded. See http://go.microsoft.com/fwlink/?LinkId=527962 for information on understanding and handling optimistic concurrency exceptions.

         at Microsoft.EntityFrameworkCore.Update.AffectedCountModificationCommandBatch.ThrowAggregateUpdateConcurrencyException(Int32 commandIndex, Int32 expectedRowsAffected, Int32 rowsAffected)

         at Microsoft.EntityFrameworkCore.Update.AffectedCountModificationCommandBatch.ConsumeResultSetWithoutPropagationAsync(Int32 commandIndex, RelationalDataReader reader, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.AffectedCountModificationCommandBatch.ConsumeAsync(RelationalDataReader reader, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.ExecuteAsync(IRelationalConnection connection, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.ExecuteAsync(IRelationalConnection connection, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.ExecuteAsync(IEnumerable`1 commandBatches, IRelationalConnection connection, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.ExecuteAsync(IEnumerable`1 commandBatches, IRelationalConnection connection, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.ExecuteAsync(IEnumerable`1 commandBatches, IRelationalConnection connection, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(IList`1 entriesToSave, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(DbContext _, Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken)

         at Pomelo.EntityFrameworkCore.MySql.Storage.Internal.MySqlExecutionStrategy.ExecuteAsync[TState,TResult](TState state, Func`4 operation, Func`4 verifySucceeded, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken)

         at Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken)

         at OpenSky.API.OpenSkyDbContext.SaveDatabaseChangesAsync(ILogger logger, String errorMessage) in /src/OpenSky.API/OpenSkyDbContext.cs:line 194

Will handle this on OpenSkyServiceBase class in both clients, api code will remain unchanged, mutex/user with mutex protecting that structure is total overkill


mount service account secret

Description
Mount reflected ServiceAccount secrets in the offloaded pods, in this way when an offloaded pod contact the home API server is able to authenticate itself with the ServiceAccount associated in the home cluster
How Has This Been Tested?

[x] Specific tests added to assert that secret volumes are correctly mounted in pods

Pull Request Test Coverage Report for Build 347296672

37 of 63 (58.73%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 46.365%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/virtualKubelet/apiReflection/controller/controller.go
0
3
0.0%

pkg/virtualKubelet/apiReflection/controller/outgoingReflectorsController.go
0
3
0.0%

pkg/virtualKubelet/apiReflection/controller/test/mockController.go
0
6
0.0%

pkg/virtualKubelet/apiReflection/reflectors/apiReflector.go
0
6
0.0%

pkg/virtualKubelet/translation/mutation.go
37
45
82.22%

Totals

Change from base Build 345586785:
0.06%

Covered Lines:
6461

Relevant Lines:
13935

💛  - Coveralls

Closing due to major refactoring.


[SW-1142] Smart contract address is displaying as aeternity when Tx is in pending status on multichain branch

LGTM


Fix/512 update GitHub actions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
5498

Relevant Lines:
8436

💛  - Coveralls


Web Push Notification Document required

I am trying to use Web Push Notification. The Django side I have followed all the way through. However, to test the system I have to run with frontend either.
Problem:
I don't understand how to follow the given Utils functions. They are many questions raise in my header for example. Where I can put server url to it?
I have seen it is opened issue but @goinnn said it is done #276
Question:
Where I can find the example of Web Push Notification according to django-push-notification? I am willing to contribute the document part in my blog in order to support the community

Article will come after this shortly. Thanks to mosetail for the answer

<EMAIL_ADDRESS>

How to change TITLE DEFAULT, and localhost:63342?

@elcolie late reply, but better late than never eh?
I've just been figuring this out myself, I've posted a solution here: https://github.com/jazzband/django-push-notifications/issues/557#issuecomment-584992502

Some one need to revise document!

I can update the docs with some tips once I've got everything a bit more figure out.

@elcolie I've submitted a PR with expanded documentation based on my experience over the last 24 hours of using Web Push. #558


Unit test unicode fixes

Seems like it all has something to do with non-UTF-8 characters in UTF-8 files...
For some reason, whenever I happened to open the unit test source files in question in Visual Studio, it automatically did some changes on the "dot", "degree" and "cubed" characters. Weird thing was they then showed up in Visual Studio as � and in my local git diff as � -> �. It also wouldn't let me commit anything before reverting the changes and closing without saving.
You can see in my first two commits that I first attempted to fix it by pasting in the correct characters again, the assumption being that they might now be formatted correctly. And that seemed to work too. Interestingly, the diff on this pull request showed the characters replaced by the same characters (but encoded differently..?), e.g. ° -> °.
Eventually, I figured these issues might have something to do with some local settings and that other users might have the same issues as I had before when opening the now changed files. Which is why I ten used the actual unicode, hoping that this is universally true.
Let me know what you think.
Btw, it seems like all these "comparison" tests are within the Angle class, which seems incorrect.

I have in the past also seen this UTF-8 warning in visual studio but never really taken the time to look into it! Great that you did!
Feel free to restructure the tests if something looks odd! I haven't looked at most of the tests for years!

I can see that some of the unittests are failing. Are they also failing when you run them?

Good catch. I must have forgotten to run the unit tests again after the very last commit, which introduced some errors


Name field missing in Customer details.

Looks like among the values passed in prefill, only contact and email are stored. name is missing when I check Customer details in dashboard or access using the API. What is the issue here or is it that name is not sent at all?

There's some confusion here. The prefill values you're passing to checkout only control what your payer sees on checkout, and allows them to skip entering these details themselves. They do not actually create customer entities on your dashboard. Those can only be created by you, using these APIs.
The name you're passing is actually getting used as cardholder name, so it won't get used at all if the payment is not done via card.
In summary: use prefill for easing checkout experience for the payer. Use customers API if you have other use cases.


Corrupt report when adding image in a shape in header/footer

Sure ! Here is a very simple template that is failing. I'm using the method injectSvg() that is in the documentation :
7078ebf0-16fe-68d6-0e45-e78a25feb224.docx
createReport({
    template: templateFile,
    data: {
    },
    noSandbox: true,
    cmdDelimiter: ['{{', '}}'],
    additionalJsContext: {
        injectSvg: () => {
            const svg_data = Buffer.from(...., 'utf-8');
            return { width: 6, height: 6, data: svg_data, extension: '.svg' };
        }
    }
})

(Just replace the Buffer.from(...) by the one that is in the documentation)
And the generated document :
fail.docx
Originally posted by @matcobat in https://github.com/guigrpa/docx-templates/issues/144#issuecomment-870393351

I'm having the same issue

Just to clarify, in my case I'm adding 2 images, a png in the header and a jpg in the body using IMAGE in the same manner as above. The png in the header renders but the jpg in the body doesn't.

Thanks for clarifying. I've reproduced it. I also noticed that if you put 4 <> commands in your repro template, only 3 images get rendered.
Experimenting with candidate fixes now.

Should be fixed in https://github.com/guigrpa/docx-templates/releases/tag/v4.6.4.
GH Actions should be publishing it to NPM right now. Let me know if you encounter anything weird. Thanks for reporting!

Many thanks ! It's working like a charm now !


Camera2BasicJava not working on pixel 2 api 28 emulator and api 28 real device

Camera2BasicJava not working on pixel 2 api 28 emulator and api 28 real device, it opens and suddenly crash app. I have not change any code just run first time

Thanks for reporting, I think this is a dupe of #82. Please reopen if you think they are different issues :)


Implement nk.logger_debug

Currently you can log message with severity levels of info, warning and error but there isn't one for debug.

Added in 7ce100697823114476e75f994e6a6b537e028a16 and will be part of the next release.


[NRCE] learn smartplug

Hi;
so just got an encoean plug (nodon ASP-2-1-01, EEP D2-01-0B), tested it with a physical 4 rocker, was able to control it from there, no problem.
Then reseted the plug and thought I would be able to get it to learn node-red using the second sample from the node-red-contrib-enocean page, but got nothing, the plug stays in learn mode, and does not learn my system.
Using node-red 0.19.4, node-red-contrib-enocean 0.5.5, it is working fine with the physical 4 rockers I have, got messages in node-red.
Any idea? (not much documentation...)
Thanks;
Lionel

yes, the same issue as #127 the issue here is twofold:

the 4 rocker uses f6-03-01 which unfortunately is not supported by nodeon devices. I will revert back to f6-02-01 which is supported. for the time being, you can use the sample flow from the aforementioned  issue.
d2-01-* eep description was flawed and is now fixed in my locaal branch. I will soon publish a new version with that patch. that will allow you to control even more aspects of your device

i will let you know when the new version is online

published v0.6.0

support for UTE teach in response for bidirectional teach in (experimatal)
fix eep d2-01-*
fix generating 4BS teach in  telegrams

should be in the palette manager in 15 min.

Hi;
thanks for the reactivity! so updated to 0.6.0, restarted node-red, imported afore mentionned work around, now works, thanks.
Lionel


Use offline with OpenAI compatible open source API and Pinecone alternative?

Cool project!
It would be neat if this could be run locally / offline without the need for cloud services or subscriptions.
Tools such as LocalAI, Text Generation Web UI, LM Studio provide OpenAI compatible API and functionality - from my experience it's as simple setting the API endpoint.
However with Pinecone - I'm not sure how standard it is, I've seen other projects using Chroma and Weaviate but I'm not sure if they're pluggable.

You could set the openAI base URL to a url proxied to your local machine using ngrok, if it's actually a drop-in replacement. Have you tried that? The env variable to set in Convex is OPENAI_API_BASE

This is now the default

Awesome work Ian!


Add new sample viewer

upgraded yarn and other packages as needed to keep pace with the Web ecosystem.
removed existing sample viewer and added a new one that uses the same code from @vertigis/web-sdk to build and run as the samples themselves
linted
added simple tests for the sample viewer

There's an issue still present with cypress that needs to be addressed. It can kind of be worked around here but right now it prevents loading samples other than the default, and interacting directly with the 'map' object.
Both of these require access to the parent window of the iframe, and this isn't resolving correctly within the cypress environment. This will be a much bigger problem in the vertigis-web-samples repo than here.

The issues with the cypress environment have been addressed. Apparently the modifyObstructiveCode flag (turned on by default) actually rewrites your code on the fly to break any references to the parent window.
Currently blocked because the deployment directory of the sample viewer build has changed to /libraries/sample-viewer/build and this needs to be reflected in the netlify configuration.

Do we know what is causing the double load?

Do we know what is causing the double load?

Yes, it loads the first time as an empty shell before it figures out what library should be active, adjusts the configuration and reloads. Maybe it would be possible to make this seem less glitchy by suppressing the visibility of everything until it's actually ready...

Added a loading spinner to make the initial load seem snappier and less glitchy. It only shows up on netlify or if you run yarn serve-library-viewer after building.


File nesting: allow for non-existant root

I love the file nesting feature, and I thought we could use it more generally. The idea I had was this:
files like jest.config.js, babel.config.js etc, I would group them in a configs root.
So if I have a configs file, they will all nest under there. but I would like to not have that file, and just vscode makes a fake dir in the explorer called configs, that then files can nest under.
This way the explorer would look much cleaner for me.

\duplicate #157848


parse json OOM

Description
I have a string, which is about 10MB in memory. However, when I use json::parse(), the memory cost is 1.6GB.
I wonder if it's a bug?
Reproduction steps
A long string with chinese charater encoded with utf-8.
call
json j = json::parse(str);
can trigger this bug.
Expected vs. actual results
lower memory cost.
Minimal code example
No response
Error messages
No response
Compiler and operating system
windows
Library version
branch: develop, commit hash: ee32bfc
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[X] I can successfully compile and run the unit tests.

Please share the input to check. Please also make sure you compile in release mode.

The input file is to large to upload(github limit 25MB), but the file is 64MB large.
The release mode costs less memory(600MB), while debug mode costs almost 1GB memory.
My code is like this:
`    std::ifstream file("C:\input.txt");
if (!file.is_open()) {
    return 1;
}

std::stringstream buffer;
buffer << file.rdbuf();

std::string content = buffer.str();
json j = json::parse(content);`

``
Is there a way to upload the input file here?

I would assume using xz would make the file small enough.

input.zip
Github doesn't support xz file, I attach a zip file instead.

I see 250 MB for that file. Not sure what your compiler is doing there.
My code for reference:
#include <fstream>
#include <iostream>
#include <nlohmann/json.hpp>

int main() {
	std::ifstream f("/Users/niels/Downloads/input.txt");
	auto x = nlohmann::json::parse(f);

	 do  {
	 	std::cout << '\n' << "Press a key to continue...";
	 } while (std::cin.get() != '\n');
}

g++ -I include -O3 -DNDEBUG=1 -std=c++11 parse.cpp


Fix spelling.

:heart: Thank you, good to see you over here @tgxworld !

@chancancode :smile: I'm trying to pick up EmberJS so I decided to fork and see how you implemented it :smile:

Let me know how it goes! Find me on hangouts/gchat if you have any questions!

@chancancode Would you recommend going with ember-rails or ember-cli? The last time I looked, ember-cli wasn't as stable. So I'm not sure where to start honestly XD

Today, definitely CLI. It's a lot better now! Awesome, even!
—
Sent from Mailbox
On Wed, Mar 4, 2015 at 5:55 AM, Guo Xiang Tan<EMAIL_ADDRESS>wrote:

@chancancode Would you recommend going with ember-rails or ember-cli? The last time I looked, ember-cli wasn't as stable. So I'm not sure where to start honestly XD
Reply to this email directly or view it on GitHub:
https://github.com/chancancode/hn-reader/pull/2#issuecomment-77161106


[CUMLOUDER] fix needed "POST"

search returns 'null' type error, have some issue -  POST Method.
if anyone can fix it, it'll be nice. @appdevelpo  @OshekharO

try  fetch()

try fetch()

i did change, but doesn't help
async search(kw) {
 const url = "https://www.cumlouder.com/search/";
 const res = await fetch(url, {
    method: "POST",
    headers: {
      "Content-Type": "application/x-www-form-urlencoded",
      "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
      "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/jxl,image/webp,*/*;q=0.8",
      "Accept-Encoding": "gzip",
      "Accept-Language": "en-US,en;q=0.5",
      "Connection": "keep-alive",
      "referer": "https://www.cumlouder.com/",
      "origin": "https://www.cumlouder.com",
    },
    body: `form[search]=${kw}`,
 });

got some different error,
Invalid argument (source): Must be a String or List<int>.: _Map len:5
#0      new HtmlInputStream (package:html/src/html_input_stream.dart:73)
#1      new HtmlTokenizer (package:html/src/tokenizer.dart:75)
#2      new HtmlParser (package:html/parser.dart:156)
#3      parse (package:html/parser.dart:42)
#4      ExtensionService.initRuntime.<anonymous closure> (package:miru_app/data/services/extension_service.dart:171)
#5      QuickJsRuntime2.initChannelFunctions.<anonymous closure> (package:flutter_js/quickjs/quickjs_runtime2.dart:251)
#6      Function._apply (dart:core-patch/function_patch.dart:11)
#7      Function.apply (dart:core-patch/function_patch.dart:35)
#8      _DartFunction.invoke (package:flutter_js/quickjs/object.dart:33)
#9      QuickJsRuntime2._ensureEngine.<anonymous closure> (package:flutter_js/quickjs/quickjs_runtime2.dart:82)
#10     channelDispacher (package:flutter_js/quickjs/ffi.dart:202)
#11     _FfiCallbackchannelDispacher (dart:ffi)
#12     FfiTrampoline_jsIsException (dart:ffi)
#13     QuickJsRuntime2._executePendingJob (package:flutter_js/quickjs/quickjs_runtime2.dart:157)
#14     QuickJsRuntime2.dispatch (package:flutter_js/quickjs/quickjs_runtime2.dart:168)
#15     QuickJsRuntime2.executePendingJob (package:flutter_js/quickjs/quickjs_runtime2.dart:230)
#16     HandlePromises._doHandlePromise.<anonymous closure> (package:flutter_js/extensions/handle_promises.dart:109)
#17     HandlePromises._doHandlePromise.<anonymous closure>.<anonymous closure> (package:flutter_js/extensions/handle_promises.dart:112)
#18     new Future.delayed.<anonymous closure> (dart:async/future.dart:427)
#19     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:18)
#20     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:398)
#21     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:429)
#22     _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184)

i don't know if this is related to the form token that comes with the keyword search in the request. as you see in the original code line 48.

const res = await this.request(/search/?q=${kw}, {       headers: {         "Content-Type": "application/x-www-form-urlencoded", 		"referer": "https://www.cumlouder.com/",       },       data: form[search]=${kw},       method: "post",     });     console.log(res);
does it mean success

const res = await this.request(`/search/?q=${kw}`, {
      headers: {
        "Content-Type": "application/x-www-form-urlencoded",
		"referer": "https://www.cumlouder.com/",
      },
      data: `form[search]=${kw}`,
      method: "post",
    });
    console.log(res);

does it mean success

it doesn't work.

@bachig26
async search(kw, page) {
const res = await this.request('', {
headers: {
"Miru-Url": https://www.cumlouder.com/search/${page}?q=${kw},
"Content-Type": "application/x-www-form-urlencoded",
"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
"Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/jxl,image/webp,/;q=0.8",
"referer": "https://www.cumlouder.com/",
"origin": "https://www.cumlouder.com",
},
data: {
'form[search]': kw,
},
method: "POST",
});
const bsxList = await this.querySelectorAll(res, "div.box-escena");
const novel = [];
for (const element of bsxList) {
  const html = await element.content;
  const url = await this.getAttributeText(html, "h3 > a", "href");
  const title = await this.querySelector(html, "h3 > a").text;
  const cover = await this.querySelector(html, "img").getAttributeText("data-src");
  novel.push({
    title: title.trim(),
    url: "https://www.cumlouder.com" + url,
    cover,
  });
}
return novel;

}
Can you try this and chr what response it's giving

Can you try this and check what response it's giving? Also don't create this many extension of 🙈.

It's because these are the ones that's easy to create and you know from my msg on TG on Non-NSFW extensions. I started creating lot of good non-nsfw extensions but they're either doesn't work or has encrypted. 🥹

Can you try this and check what response it's giving?
it just gives me an msg., "No data".

also i'm closing this anyway as i don't know how to fix it.


withCopyFilesToContainer should follow symlinks by default

Summary
I am trying to use withCopyFilesToContainer to copy files onto a container. The build system I am using is Bazel, which runs the javascript inside a sandbox. The sandbox contains only symlinks to the actual files that need to be copied and so I need a way for testcontainers to follow the symlink and copy the linked file instead of the link itself.
My workaround currently is to read the file and then use withCopyContentToContainer instead.
E.g.
  const schema = fs.readFileSync(
    `${RUNFILES}/_main/infra/local/router/schema/supergraph.graphql`,
  );

  const routerContainer = new GenericContainer(
    "ghcr.io/apollographql/router:v1.38.0",
  )
    .withCopyContentToContainer([
      {
        content: schema,
        target: "/dist/schema/local.graphql",
      },
    ]).start();

This would be simpler if the tar we copy to the container follows symlinks by default. With tar we can use -h, -H or -L for this behaviour:

-L, --dereference
(c and r modes only) All symbolic links will be followed.  Normally, symbolic links are archived as such.  With this option, the target of the link will be archived instead.

Expected Behaviour
withCopyFilesToContainer should follow symlinks referenced by source when copying files into the container so that the copied file can be read rather than remaining a link to the linked file.
Actual Behaviour
withCopyFilesToContainer puts the file at the right target, but if it is a symlink it remains a symlink rather than following the link and copying the linked file.
Testcontainer Logs
2024-02-03T00:12:12.808Z testcontainers [DEBUG] Creating container for image "ghcr.io/apollographql/router:v1.38.0"...
2024-02-03T00:12:12.886Z testcontainers [DEBUG] [d792c2d98366] Created container for image "ghcr.io/apollographql/router:v1.38.0"
2024-02-03T00:12:12.888Z testcontainers [DEBUG] [d792c2d98366] Putting archive to container...
2024-02-03T00:12:12.912Z testcontainers [DEBUG] [d792c2d98366] Put archive to container
2024-02-03T00:12:12.912Z testcontainers [INFO] [d792c2d98366] Starting container for image "ghcr.io/apollographql/router:v1.38.0"...
2024-02-03T00:12:12.912Z testcontainers [DEBUG] [d792c2d98366] Starting container...
2024-02-03T00:12:13.143Z testcontainers [DEBUG] [d792c2d98366] Started container
2024-02-03T00:12:13.143Z testcontainers [INFO] [d792c2d98366] Started container for image "ghcr.io/apollographql/router:v1.38.0"
2024-02-03T00:12:13.143Z testcontainers [DEBUG] [d792c2d98366] Inspecting container...
2024-02-03T00:12:13.322Z testcontainers [DEBUG] [d792c2d98366] Inspected container
2024-02-03T00:12:13.322Z testcontainers [DEBUG] [d792c2d98366] Fetching container logs...
2024-02-03T00:12:13.331Z testcontainers [DEBUG] [d792c2d98366] Demuxing stream...
2024-02-03T00:12:13.331Z testcontainers [DEBUG] [d792c2d98366] Demuxed stream
2024-02-03T00:12:13.331Z testcontainers [DEBUG] [d792c2d98366] Fetched container logs
2024-02-03T00:12:13.331Z testcontainers [DEBUG] [d792c2d98366] Waiting for container to be ready...
2024-02-03T00:12:13.331Z testcontainers [DEBUG] [d792c2d98366] Host port wait strategy complete
2024-02-03T00:12:13.331Z testcontainers [INFO] [d792c2d98366] Container is ready
2024-02-03T00:12:13.331Z testcontainers:containers [d792c2d98366] total 16
2024-02-03T00:12:13.331Z testcontainers:containers [d792c2d98366] drwxr-xr-x 1 root root 4096 Feb  3 00:12 .
2024-02-03T00:12:13.331Z testcontainers:containers [d792c2d98366] drwxr-xr-x 1 root root 4096 Jan 19 20:47 ..
2024-02-03T00:12:13.331Z testcontainers:containers [d792c2d98366] lrwxrwxrwx 1 root root   71 Feb  2 22:45 local.graphql -> ../../../../../../../../../infra/local/router/schema/supergraph.graphql

Steps to Reproduce

With a source that is a symlink.
Copy symlink to a target on the container.
See that the copied file is still a symlink, rather than a real file with the contents of the original.

E.g.
  const routerContainer = new GenericContainer("ghcr.io/apollographql/router:v1.38.0")
    .withCopyFilesToContainer([
      {
        source: `${RUNFILES}/_main/infra/local/router/schema/supergraph.graphql`,
        target: "/dist/schema/local.graphql",
      },
    ])
    .withEntrypoint(["ls"]).withCommand(["-al", "/dist/schema"])
    .start();

Environment Information

Operating System: Mac
Docker Version: Docker version 24.0.7, build afdd53b
Node version: 18
Testcontainers version: 10.6.0

Hi @jackvincentnz, thanks for raising. The archiving library we use is node-archiver, I've read through their docs and dereferencing is not supported. It only supports copying in symlinks, but without the source file the contents is lost. I'd happily accept a PR which swaps this library with another that supports this functionality


Make sure we can persist and load WFS layers

closes #393
SDK-313

Coverage decreased (-0.04%) to 70.245% when pulling 16f70d70c4a325a1f5f0590948b397c969782791 on wfs-save-mapconfig into 18a95a8b3910b5bfd6060a8dc4d684a8f33a7323 on master.


Feature Request: DataStreams/Datalogs streaming in EvalTool

May I suggest that it be possible / configurable to stream the data streams (or the data logs) to a remote machine via TCP / UDP. (Could also be accessible via a local port on the machine EvalTool is running on).
You  would need to include a destination IP address, and a port number. While not absolutely necessary and probably a significant amount of work, being able to choose a destination for each DID would be cool. Offering a "rotary" of say, 10 ports (# configurable), whereby the port could be selected
with a dropdown next to each DID. Also being able to specify TCP or UDP (initially TCP). If the ports
can be "labelled", the destination ports can be predefined and referred to by "label".
Being able to specify a single IP (or hostname) both a port number is a good start. I'm doing this with
the CLTool (and output redirection) with some "netcat-foo" but having this in the EvalTool would be a benefit. One has more access in the EvalTool to data sets than the CLTool (or so it seems).

Thank you. Will do.


Unexpected indexOf() on List of same type elements.

List#indexOf returns unexpected result if List consists of custom type elements.
Consider following example:
var one = new Immutable.Record();
var two = new Immutable.Record();
var list = Immutable.List([one, two]);

console.log(one === two); // false
console.log(list.indexOf(one)); // 0
console.log(list.indexOf(two)); // 1

This works well. Both one and two are similar empty Records, but they are not equal. one is first element of list, two is second element.
Something goes wrong if custom types are used:
var Type = Immutable.Record({});
var one = new Type();
var two = new Type();
var list = Immutable.List([one, two]);

console.log(one === two); // false
console.log(list.indexOf(one)); // 0
console.log(list.indexOf(two)); // 0, but should be 1

For some reason, one and two appears to be on the same place in the list. What is going on? Is that expected behaviour?

I believe the issue here is that your first example is mis-using the record API.
var one = new Immutable.Record();

This line of code will create a new type of record one. one is a function that when called will produce record values.
In your second example, you're using the record API correctly (though just a minor note, new is not necessary when calling these functions, but harmless). In those cases one and two are record values. We can use value equality to compare them.
Value is an important part of what is going on here. Lots of the objects Immutable.js creates are treated as values by the Immutable.js APIs. The most important difference is in how equality is determined. one === two will always return false here because === means object reference identity. However if you wrote one.equals(two) or Immutable.is(one, two), then you should get true as they are equivalent values.
Immutable.List's indexOf API uses value equality rather than object identity to find the matching index in the collection, and since one and two are value-equivalent, the first one encountered by indexOf is returned, which is why you're getting 0.
Please let me know if that's unclear, or if there's anything else I could do to make this sort of behavior more evident either via the API itself or via the documentation.

Thank you for explanation!
I was confused by example of === equality for Maps from documentation. It show following example:
var one = Immutable.Map({a: 1});
var two = one.set('a', 1)
console.log(one === two); // true

It looks like one and two compares by values resulting they're equal. Previously didn't notice text about Immutable always returns itself on trivial mutation. I thought if one.set('a', 1) is equal to one, so one.set('a', 2).set('a', 1) must be equal. Obviously, it must not.

Can you please explain me one more thing. Is it possible to use indexOf to search for reference?
For example, in my project I need to maintain list of editable objects. On certain event, like clicking on item on page, I copy selected object to another variable and mutate object's fields on that variable. By that, when edition completes and updates are not discarded, I still have fresh new object and reference to original. Now I want to replace original object to the new in the list. Is it possible to know index of object in the list without storing index separately? Is it possible to have something like:
var i = list.indexOfRef(object);
list = list.set(i, newObject):

indexOf uses Immutable.is to determine if it's found a match. That uses an equals method on each item if it finds one, which means if object in your example is an Immutable.js value, that it will use value equality not reference equality.
If you want to use reference equality instead, you could certainly write a bit of code to do that using findIndex which will let you define your own function to decide what is "found"
var i = list.findIndex(item => item === object);


Move to CRDs

This works when you launch a new PO in a 1.7 cluster.
TODO:

[x] Add compat to older k8s versions by creating TPRs when required.
[ ] Move from v1alpha1 to v1beta1
[ ] Move WaitForCRD over to the new API? But would it break compat?
[x] Add a migration helper and look at different strategies.

cc @brancz @fabxc @mxinden @nikhita

@coreosbot  test this please

@coreosbot test this please

@Gouthamve can you update me on the status of this?

@coreosbot test this please

@coreosbot test this please

@brancz This took way longer than I expected but now the tests pass. Can you take a look now?
I will first document what happens and how rollback happens and incase of a failed rollback how to restore state. I want to tackle v1alpha1 --> v1 in another PR if possible.

@Gouthamve no worries. Almost everything always takes longer than one expects. I'm fine with separating in multiple PRs. I'll give this a review as soon as I can.

@coreosbot test this please.
Once this is green it lgtm. Nice work @Gouthamve !

Awesome @Gouthamve ! Fantastic work!

I am not sure, that we can trust the test suite (see comment). @Gouthamve Do you have some spare time to fix this?


Confused by release tags

Win10 bash pnpm
$ npm uninstall -g @cloudflare/wrangler
removed 27 packages in 0.295s

$ pnpm install -g @cloudflare/wrangler
Packages: +34
++++++++++++++++++++++++++++++++++
Resolving: total 35, reused 17, downloaded 18, done
node_modules/.pnpm/registry.npmjs.org/@cloudflare/wrangler/1.7.0/node_modules/@cloudflare/wrangler: Running postinstall script, done in 2s
+ @cloudflare/wrangler 1.7.0

$ wrangler --version
 wrangler 1.7.0

$ pnpm uninstall -g @cloudflare/wrangler
Packages: -34
----------------------------------
Resolving: total 1, reused 1, downloaded 0, done

$ wrangler --version
bash: /c/Users/t/AppData/Roaming/npm/wrangler: No such file or directory

$ pnpm i @cloudflare/wrangler@beta -g
Packages: +34
++++++++++++++++++++++++++++++++++
Resolving: total 35, reused 34, downloaded 1, done
node_modules/.pnpm/registry.npmjs.org/@cloudflare/wrangler/1.8.0-rc.0/node_modules/@cloudflare/wrangler: Running postinstall script, done in 3s

C:\Users\t\AppData\Roaming\npm\pnpm-global\3:
+ @cloudflare/wrangler 1.8.0-rc.0

$ wrangler --version
 wrangler 1.8.0-rc.0

I was expecting v1.8.0-rc.1 that was released 2 days ago that was tagged 'latest'?
TIA :)

hey @tomByrer ! we tagged the new RC but we haven't published it to npm yet. i will be doing that shortly!

This is now released 👍


services.AddMvcGrid() not available

I have followed the installation instructions and successfully installed NonFactors.Grid.Mvc6 ver. 0.9.0-rc1-final via Nuget.  However, in the Startup.cs file of my ASP.NET 5 MVC6 project, the call to AddMvcGrid() is not recognized:
public void ConfigureServices(IServiceCollection services)
{
services.AddMvc();
services.AddMvcGrid();  // error here
......
}
What could I be doing wrong?

You need to add default namespace
using NonFactors.Mvc.Grid
I will add it to the docs as well :+1:

Thank you.  Great grid control you've made here.


Turn off warnings from torchmetrics while running inference.py

When I run inference I got the warning from torchmetrics as below. It makes inference get more time to executed.
python tools/inference.py --config anomalib/models/padim/config.yaml --weight_path results/padim/weights/model.ckpt --image_path datasets/1.jpg --save_path test_result
...\Anaconda3\envs\anomalib_env\lib\site-packages\torchmetrics\utilities\prints.py:36: UserWarning: Torchmetrics v0.9 introduced a new argument class property called `full_state_update` that has
                not been set for this class (AdaptiveThreshold). The property determines if `update` by
                default needs access to the full metric state. If this is not the case, significant speedups can be
                achieved and we recommend setting this to `False`.
                We provide an checking function
                `from torchmetrics.utilities import check_forward_no_full_state`
                that can be used to check if the `full_state_update=True` (old and potential slower behaviour,
                default for now) or if `full_state_update=False` can be used safely.

  warnings.warn(*args, **kwargs)
...\Anaconda3\envs\anomalib_env\lib\site-packages\torchmetrics\utilities\prints.py:36: UserWarning: Metric `PrecisionRecallCurve` will save all targets and predictions in buffer. For large datasets this may lead to large memory footprint.
  warnings.warn(*args, **kwargs)
...\Anaconda3\envs\anomalib_env\lib\site-packages\torchmetrics\utilities\prints.py:36: UserWarning: Torchmetrics v0.9 introduced a new argument class property called `full_state_update` that has
                not been set for this class (AnomalyScoreDistribution). The property determines if `update` by
                default needs access to the full metric state. If this is not the case, significant speedups can be
                achieved and we recommend setting this to `False`.
                We provide an checking function
                `from torchmetrics.utilities import check_forward_no_full_state`
                that can be used to check if the `full_state_update=True` (old and potential slower behaviour,
                default for now) or if `full_state_update=False` can be used safely.

  warnings.warn(*args, **kwargs)
...\Anaconda3\envs\anomalib_env\lib\site-packages\torchmetrics\utilities\prints.py:36: UserWarning: Torchmetrics v0.9 introduced a new argument class property called `full_state_update` that has
                not been set for this class (MinMax). The property determines if `update` by
                default needs access to the full metric state. If this is not the case, significant speedups can be
                achieved and we recommend setting this to `False`.
                We provide an checking function
                `from torchmetrics.utilities import check_forward_no_full_state`
                that can be used to check if the `full_state_update=True` (old and potential slower behaviour,
                default for now) or if `full_state_update=False` can be used safely.

  warnings.warn(*args, **kwargs)

How to turn off this warning?

Duplicate of #366. This started to appear after the recent torchmetrics release. We'll update the code base to suppress this.


#N/A: Improve support to Windows in no_command rule

Windows “not found” message is quite different from POSIX systems.

Looks good, thanks!


Add a View method that returns the actual contents of the view

Currently, if a view is wrapped, there seems to be no way to actually return the new view content, including the extra newlines. Buffer returns the content before the newlines have been calculated. This makes implementing scrolling with boundaries impossible.

First of all, sorry for the late response. Take a look at e5bf60e36b67610cf66ef779ad2b83c317b5ec49, I added the method View.ViewBuffer() that returns the contents of the buffer that is showed to the user.


Assuming I installed correctly, where does this input a notification?

I think I installed this correctly, it's been a couple weeks since I've had time to work on anything related. Where would this LLMVision notification be occurring? Do I make an automation to send to my phone? Is there a UI?

LLM Vision is an integration. You can build an automation to send the analysis to your phone or use the blueprint (although there have been some issues with that lately). If you want to test LLM Vision, you can do so in developer tools. Please see the examples and documentation (link in readme).


Nested tests fail in Java JUnit

I'm using nested tests in Java using JUnit's Nested annotation: https://junit.org/junit5/docs/5.4.1/api/org/junit/jupiter/api/Nested.html. When running those tests with :TestNearest, they fail. However, when I run them with :TestFile, they succeed. The same tests were working with vim-test before I introduced the Nested pattern.
Here's the output of :TestNearest:
> Task :test FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':test'.
> No tests found for given includes: [EntityTest.GetSpecificReservableBlock.getsReservableBlock](--tests filter)

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s
5 actionable tasks: 2 executed, 3 up-to-date

[Process exited 1]

The pattern EntityTest.GetSpecificReservableBlock.getsReservableBlock seems to be wrong because when I run gradle test --tests EntityTest.GetSpecificReservableBlock.getsReservableBlock, I get the same result.
My test file looks as follows:
class EntityTest {

	@Nested
	class GetSpecificReservableBlock {

		@Test
		void getsReservableBlock() { ... }
      }
}

The correct pattern would be EntityTest$GetSpecificReservableBlock.getsReservableBlock (with a $ instead of a . between the main class and the nested one).

Hello @cloudlena
I'm also facing this issue, I'm have never contributed to this repository before, so I started with some specs about @Nested tests, could you help me to validate the correct output?
See this spec:
https://github.com/vim-test/vim-test/compare/master...GiuseppeMP:master#diff-54917040ad12d6d52d470b7f7c18283ae3938e96febc27fce761c6cbfeffe2f0R19
https://github.com/vim-test/vim-test/compare/master...GiuseppeMP:master#diff-54917040ad12d6d52d470b7f7c18283ae3938e96febc27fce761c6cbfeffe2f0R19

@cloudlena It seems to be resolved in this merge request, but need to be merged yet. See #662

@cloudlena Hey, It was merged, could you test and update the issue status? ✌

@GiuseppeMP, I'm currently not working with Java so I cannot try the fix... Thanks so much for looking into it! For me, the issue can be closed.


Follow up to PR comments on #2642

What this PR does / why we need it:
Follow up to PR comments on #2642, specifically here.
Which issue(s) this PR fixes:
Related to work on OCPBUGS-11939
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/test e2e-aws

/approve

/test e2e-kubevirt-aws-ovn


Node search

Fixes #887
Returns list of nodes of all containers that are returned from a given search query
/dataexplorer/search/nodes
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Changes look good! I was originally confused about removing part of the by_container aggregation, but ordering the results is unnecessary for this type of endpoint.


[Subtask][Improvement][AQE] Introduce the new MergedShuffleDataSegment to reduce number of rpc

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the subtask
Currently, when using the LocalOrderSegmentSplitter, it will split multiple ShuffleDataSegment by three factors.

taskId in [startMapId, endMapId) taskIds bitmap
ShuffleDataSegment size should < readBufferSize
ShuffleDataSegment's blocks should be continuous

In some cases, these requirements will make the data size very small (<14M). So we could combine some ShuffleDataSegment to single one data-fetching request to reduce the RPC interaction number.
Parent issue
#137
Are you willing to submit PR?

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

Maybe global order will be a better solution for this case.

Yes, maybe we should introduce global order to compare with other distribution types in different data size
but this pr is still meaningful for local order


execute RedirectToCanonicalUrl before UseErrorPages

Hi,
I am using both RedirectToCanonicalUrl and UseStatusCodePagesWithReExecute in cms project.
My issue is that 404 Error Page is displayed before redirecting to Canonical Url.
For example, if "site/pagename" is valid url
"site/Pagename" should first redirect to "site/pagename" , and then if not found display 404 page.

Will investigate.

The templates behaviour is that if the page is not found there is no redirect. If you have changed the middleware pipeline order, this may have caused what you are seeing. Also the 404 page is only visible using the Production environment.


Improve JsonSchema.ToJsonSchema

https://github.com/awaescher/OllamaSharp/issues/153

This should correctly handle arrays. Does it make sense to add support for collection types like lists and hashsets?

I've tried it, and it should work on List and HashSet as well.


IOS Push Notification not opening the Zendesk Messaging View Screen

Describe the bug
In iOS devices, a message push notification does not open the Zendesk Messaging View screen, but this functionality works fine in Android after tapping on the push notification.
To Reproduce
Steps to reproduce the behavior:
Receive a push notification on an IOS device and tap on it, it will open the app but will not open the Zendesk messaging view screen in iOS only.
Expected behavior
On Push Notification tap, this should open the messaging view screen of Zendesk, like in Android.
Environment (please complete the following information):

Platform: IOS 17.5
React Native Version 0.74
Zendesk SDK Version ^0.2.1

@adilsaeed31 Hi, I had the same issue with this library and i fixed it. can you show your code in AppDelegate.m?


Move CI to buildkite

@xrimpy what is the status of this?

Waiting ng on your digital ocean kyc.
I am unable to access servers as the account is locked

Apart from that the builds I showed you on Wednesday are using build kite only

@xrimpy lmk if you need anything here. Vipul mentioned that we need to move away from Github CI ASAP.

@xrimpy As I understand this issue is done, can you please close it if its that correct.


tests: Mock init properly

Argilla Community Growers
Thanks for your contribution! As part of our Community Growers initiative 🌱, we're donating Justdiggit bunds in your name to reforest sub-Saharan Africa. To claim your Community Growers certificate, please contact David Berenstein in our Slack community or fill in this form https://tally.so/r/n9XrxK once your PR has been merged.
Pull Request Templates
Please go the the Preview tab and select the appropriate sub-template:

🐞-bug
📚-documentation
🆕-features
🔗-💥-integration

Generic Pull Request Template
Since changes introduced in https://github.com/argilla-io/argilla/commit/347dd9c216cd6f586f5566374bf79d6cb15f8326, some tests using a mocked version for the rg.init function are outdated. This PR updates them and fixes the pipeline execution.

LGTM thanks for fixing!
The issue was introduced at #3641, when replacing whoami + list_workspaces with list_workspaces_me.
Before merging we should change the base of the PR to releases/1.15.0

since the error will block also develop builds, we will merge them into develop and cherry-pick to the release branch. This will be easier.  Thanks!


Explorer menu does not show up in VS Code ver 1.63.2

I installed version 0.4.4 onto VSCODE version 1.63.2.
The extension is enabled and reloaded. Even re-launched VSCODE.

When I click the "View->Explorere", the explorer menus are not displayed on the screen.
Environment (please complete the following information):

OS: WIN-10
vscode version [1.63.2]

Not quite sure if I have the same issue here. I installed the extension but no "SOLUTION EXPLORER" is shown after ope the folder with the .sln file inside.  The documentation says nothing about the .Net frameworks supported nor what VS versions .sln files are supported.

In Settings you will find "Vssolution: Show Mode":

Depending on your selection, the solution explorer will be displayed:

In the activityBar with this icon:

in the Explorer pane

or if you select "none" it will be hidden.


ttsfrd 是不是可以提供py3.9等版本的wheel

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

no, there is no plan to update ttsfrd, dont wait for it


The "chrome.cookies" category of WebExtension APIs appears to be unsupported

Describe the bug
The chrome.cookies group of WebExtension APIs appears to be totally unsupported, but they fail silently, returning empty results instead of throwing an error. Neither Google searches nor searches in the issue tracker here turned up anything to suggest that this is a known or deliberate situation (e.g. the API being unimplemented/stubbed and on the to-do list).
To Reproduce

Download the "Cookie Editor" WebExtension.
Go to example.com.
Open the browser developer tools for this tab via the main menu.
Execute this line of JavaScript to set a new cookie: document.cookie = "test=123;"
Execute this line of JavaScript to verify that your cookie has been set: document.cookie.
Go back to the example.com tab.
In the browser menu, open Cookie Editor for this tab.

Expected behavior
Step 5 should show test=123, confirming that the cookie has been set. Step 7 should show the cookie.
Actual behavior
Step 5 behaves as expected, but in step 7, Cookie Editor claims that no cookies exist for the site.
Smartphone (please complete the following information):

Device: Samsung Galaxy A50
Android Version: A11
Kiwi Browser Version: 105.0.5195.33

Additional context
In WebExtensions I'm developing for private use, I've found that chrome.cookies.getAllCookieStores() always returns an empty array, and APIs like chrome.cookies.getAll() return empty results as well. A call to the chrome.permissions.contains API confirms that cookie permissions are granted for the domains I'm trying to target, as expected; the cookie APIs simply don't work in Kiwi. Chrome on desktop returns a list of cookie stores as expected when tested with my private-use extension under the same conditions.
The lack of chrome.cookies support is a nasty problem. The only way for WebExtension background pages/workers to send HTTP requests with credentials is by manually reading the user's cookies and jamming them into the outgoing request. This bug breaks that workaround, and the fact that the APIs are failing silently (coupled with Chrome's lackluster documentation for them) made this a hard problem to track down.
Possible duplicates from the old repo:

#163 (2020)
Vague bug report; user may be referring to this "IDM"? Report marked as "enhancement" without further comment; no action since.
kiwibrowser/android #193 (2021)
Bug could not be reproduced; closed; user redirected to this repo.

A more reliable approach is to intercept Cookie headers on outgoing requests, and Set-Cookie headers on incoming responses, and use these to maintain a cache of cookie states in the background page. This should allow you to get even HttpOnly cookies. Then, use declarativeNetRequests or other means to forcibly set the cookies you want to use for background page fetch requests, background page IFRAMEs, and similar.
The nice thing about this is that you never need to send the cookie across from a content script to the background page; the data stays in the background page/process.