ignore-mode "display" makes possible to delete invisible articles

newsboat 2.21.0 - https://newsboat.org/
Copyright (C) 2006-2015 Andreas Krennmair
Copyright (C) 2015-2020 Alexander Batischev
Copyright (C) 2006-2017 Newsbeuter contributors
Copyright (C) 2017-2020 Newsboat contributors

Newsboat is free software licensed under the MIT License. (Type `newsboat -vv' to see the full text.)
It bundles:
- JSON for Modern C++ library, licensed under the MIT License: https://github.com/nlohmann/json
- optional-lite library, licensed under the Boost Software License: https://github.com/martinmoene/optional-lite
- expected-lite library, licensed under the Boost Software License: https://github.com/martinmoene/expected-lite

newsboat 2.21.0
System: Linux 5.9.12-1-default (i686)
Compiler: g++ 10.2.1 20200825 [revision c0746a1beb1ba073c7981eb09f55b3d993b32e5c]
ncurses: ncurses 6.2.20201031 (compiled with 6.2)
libcurl: libcurl/7.73.0 OpenSSL/1.1.1h-fips zlib/1.2.11 libidn2/2.3.0 libpsl/0.21.1 (+libidn2/2.3.0) libssh/0.9.5/openssl/zlib nghttp2/1.41.0 (compiled with 7.73.0)
SQLite: 3.34.0 (compiled with 3.33.0)
libxml2: compiled with 2.9.10

Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
*read below

Steps to reproduce the issue:

Have some articles in a feed with certain url containing some 'keyword' in content body

In config set

ignore-mode "display"
ignore-article "the url of the feed" "content !~ keyword"

Run newsboat and open the feed (the killfile works fine)
Press CTRL+D to delete all articles visible in the feed
Exit newsboat
Remove the 'ignore-article' line
Run newsboat again and go to the same feed

Expected:
The articles containing 'keyword' should remain. IOW only the ones displayed in step 2 should have been deleted.
Actual:
All articles in the feed are deleted.

Thank you for the report! I don't have time to reproduce this right now, but I'll get back to you once I did.

Reproduced on 6dc08295efd6bcd31695bb66ec197305695be304.


"Exiting" message is displayed immediately after opening win app driver on the Virtual Machine.

I am having the following issue: I have a cake script that runs the automated tests on a virtual machine . The windows application driver is opened but it gets closed at the same time stamp that opens, something like this:
**Windows Application Driver listening for requests at: http://<IP_ADDRESS>:4723/
Press ENTER to exit.
Exiting...**
Do you have any idea how to keep the driver running?
P.S. The developer mode is enabled on the virtual machine!

Having the exact same problem. I'm using Powershell Start-Process to start winappdriver in background. When running the startup scipt manually  winappdriver starts nicely - but when I run it through Jenkins, winappdriver exits immediately.
Not sure, but it might be related to how winappdriver handles stdin. The manual run has an interactive session of course, but the Jenkins initiated run probably does not.
By setting RedirectStandardInput option for Start-Process and pointing it to empty file, the problem happens also for manual runs. Wish I knew what does winappdriver expect from stdin...?

I now switched from Power-Shell:Start-Process to plain old START, and the problem went a way. Looking back, I switched to Start-Process so I could easily 1) prevent the extra console window and 2) to redirect stderr and stdout to logs.

Workaround that works for me, in a powershell-script (.ps1):
<clip>
Start-Process 'cmd' -ArgumentList '/c "C:\ProgramFiles (x86)\Windows Application Driver\WinAppDriver.exe" 1>winappdriver_out.log 2>winappdriver_err.log` -WindowStyle Hidden
<clip>

What I assume happens with that piece of script:  the hidden cmd window satisfies the odd stdin-behaviour of winappdriver.
To get the logs in file ( which is a very basic requirement in CI ), the redirection of stdout and stderr had to be done in the cmd-script part instead of using the RedirectStandardOutput and RedirectStandardError flags of Start-Process: if I used the flags, it probably also affected RedirectStandardInput...and again did not work.

Hi @karniemi . Sorry for the late reply.
Thank you for your answer. The same kind of workaround worked for us in the end.

Hi @karniemi . Sorry for the late reply.
Thank you for your answer. The same kind of workaround worked for us in the end.

We are seeing this also,  but we would really like to keep the WinAppDriver output in the same console output as VSTest output  so we can see exactly what tests cause what WinAppDriver failures.
The solution propsed above by @karniemi is nice but then the WinAppDriver output is kept separate and it isn't clear what tests cause what WinAppDriver output.
For more context our build system is Teamcity running WinAppDriver and VSTest based UITests.
Here's what we tried in powershell
start-process -NoNewWindow -FilePath "C:\Program Files (x86)\Windows Application driver\winappdriver.exe" -ArgumentList "1234"
start-process -NoNewWindow -FilePath "C:\Program Files (x86)\Microsoft Visual Studio\2017\TestAgent\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" -ArgumentList "UITests\UITests.dll /Logger:trx /Platform:x86" -Wait

And with batch script
start /b cmd /c "C:\Program Files (x86)\Windows Application driver\winappdriver.exe" 1234
start /wait /b cmd /c "C:\Program Files (x86)\Microsoft Visual Studio\2017\TestAgent\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" UITests\UITests.dll /Logger:trx /Platform:x86

Both work locally, but under teamcity because it is noninteractive there isn't a stdin, it doesn't work.

@LogicuL: could you reopen this ticket? Users will hit this odd issue, and our workarounds are not the correct solution...

The workarounds described above are not working for everyone. Reopening the issue.

I'm also having this issue using TeamCity. What I've done as a workaround is to create a Windows scheduled task and keep it always up and running.
I wonder if there is any progress on this fix, though.

I created a repository that can reproduce the issue here: https://github.com/icnocop/WinAppDriverBug
WinAppDriver.exe unexpectedly exists when the test is run in a TFS 2015 Update 5 on-prem build vNext agent (VsoAgent.exe) and an interactive console running in Windows Server 2019.
WinAppDriver.exe works without issues if I run vstest.console.exe on the command line interactively on the test agent.
It works fine in Azure Pipelines and AppVeyor.
I've tested with both WinAppDriver v1.1.1 and v1.2 RC.
In my case, the workaround is to just set the RedirectStandardInput property of ProcessStartInfo to true.
🤷♂

@icnocop Thanks for sharing this workaround! This works great!

@LogicuL did you ever found a reason or solution? Started facing similar issue when we updated our surefire version to 3 from 2.20

orks fine in Azure Pipelines and AppVeyor.

Could you share the Azure pipeline configuration, I am facing this issue in Azure pipeline and none of the above solution works for me.
I even tried using the WinAppDriver plugin but, still the same.

I created a repository that can reproduce the issue here: https://github.com/icnocop/WinAppDriverBug
WinAppDriver.exe unexpectedly exits when the test is run in a TFS 2015 Update 5 on-prem build vNext agent (VsoAgent.exe) and an interactive console running in Windows Server 2019.
WinAppDriver.exe works without issues if I run vstest.console.exe on the command line interactively on the test agent.
It works fine in Azure Pipelines and AppVeyor.
I've tested with both WinAppDriver v1.1.1 and v1.2 RC.
In my case, the workaround is to just set the RedirectStandardInput property of ProcessStartInfo to true.
🤷♂

Could you share the Azure pipeline configuration, I am facing this issue in Azure pipeline and none of the above solution works for me.
I even tried using the WinAppDriver plugin but, still the same.

I'm also facing the same issue, could any one let me know
Do I need to set the RedirectStandardInput property to true in Azure Pipeline? or my code?
Appreciate your response.
Thanks

In the sample repo I shared, I launch WinAppDriver.exe using C# .NET's Process.Start method.
To fix the issue in my sample repo, it requires changes to the C# code:
ProcessStartInfo processStartInfo = new ProcessStartInfo(
  exeFilePath,
  commandLineArguments)
  {
    CreateNoWindow = true,
    UseShellExecute = false,
    RedirectStandardInput = true, // <-- this line will fix the issue where WinAppDriver.exe unexpectedly exits
    RedirectStandardOutput = true,
    RedirectStandardError = true,
    StandardOutputEncoding = Encoding.UTF8,
    StandardErrorEncoding = Encoding.UTF8
  };

@icnocop Thanks for the response
I tried your code. I do not have start WinAppDriver step in the Azure pipeline .However, I am receiving below error .
Below is the code ..I am using.

string exeFilePath = @"C:\Program Files (x86)\Windows Application Driver\WinAppDriver.exe";
string commandLineArguments = $"{winAppDriverUri.Host} {winAppDriverUri.Port}{winAppDriverUri.PathAndQuery} /forcequit";
ProcessStartInfo processStartInfo = new ProcessStartInfo(
exeFilePath,
commandLineArguments)
{
CreateNoWindow = true,
UseShellExecute = false,
RedirectStandardInput = true, // <-- this line will fix the issue where WinAppDriver.exe unexpectedly exits
RedirectStandardOutput = true,
RedirectStandardError = true,
StandardOutputEncoding = Encoding.UTF8,
StandardErrorEncoding = Encoding.UTF8
};
        Console.WriteLine($"Launching \"{processStartInfo.FileName}\" { processStartInfo.Arguments}");
        Process.Start(processStartInfo);
        Console.WriteLine("Started WinAppDriver");

@Karjun330 that seems like a different issue.
Check the std out and std error for relevant messages.

ProcessStartInfo processStartInfo = new ProcessStartInfo(
exeFilePath,
commandLineArguments)
{
CreateNoWindow = true,
UseShellExecute = false,
RedirectStandardInput = true, // <-- this line will fix the issue where WinAppDriver.exe unexpectedly exits
RedirectStandardOutput = true,
RedirectStandardError = true,
StandardOutputEncoding = Encoding.UTF8,
StandardErrorEncoding = Encoding.UTF8
};

@icnocop  - Would you be aware of the JAVA equivalent for the same?

ProcessStartInfo processStartInfo = new ProcessStartInfo(
exeFilePath,
commandLineArguments)
{
CreateNoWindow = true,
UseShellExecute = false,
RedirectStandardInput = true, // <-- this line will fix the issue where WinAppDriver.exe unexpectedly exits
RedirectStandardOutput = true,
RedirectStandardError = true,
StandardOutputEncoding = Encoding.UTF8,
StandardErrorEncoding = Encoding.UTF8
};

@icnocop - Would you be aware of the JAVA equivalent for the same?

We were able to configure WAD on JENKINS using instructions here :
#1368


[Bug]: Onboarding fails if SSH server uses other port than 22

Description
Onboarding gets stuck and gives error when host's SSH server port is not 22. Onboarding apparently defaults to that, won't allow changing it or notify about it.
Only way was to pass error was to skip onboarding and setup localhost server manually in panel. Slightly confusing on a first try.
After changing SSH port back to default 22 and reinstalling, the onboarding goes through nicely.
Partially similar error as in https://github.com/coollabsio/coolify/issues/3252.
https://github.com/user-attachments/assets/92ba022a-4fc2-475c-96c8-9ddb6248abc7
Minimal Reproduction (if possible, example repository)
Debian 12 server with openssh-server previously configured to listen to non-default port in /etc/ssh/sshd_config/.
Using the cdn.coollabs.io/coolify/install.sh installation script.
Exception or Error
Localhost server is not found. Something went wrong during installation. Please try to reinstall or contact support.
ssh: connect to host host.docker internal port 22: Connection timed out
Version
v4.0.0-beta.329
Cloud?

[ ] Yes
[X] No

Same problem here!

This will be fixed in the next version

This has been released. If you use a different port (and are going through onboarding) after the first check if the server is available (it will fail because of a different port), you have the option to change the port in the UI and then revalidate.


does not overwrite user.token when it is already present in original config

When using kubectl config-import -f $new_config, if user.token in original config already exists, it will not be overwritten by $new_config.
In our scenario, the kubeconfig has expiration and we need to download new kubeconfig to replace the old one. However, since user.token is not overwritten, we need to first remove the old configure using editor, and then kubectl config-import ... to import new configuration. It would be better if kubectl config-import -f $new_config can just overwriite everything.

Yes, it makes sense to overwrite with provided file. Just released v0.6.7 with this change. Let me know if that works for you.

It is fixed, thanks for the help.


Use sensu-install

https://sensuapp.org/docs/0.21/plugins
It might make sense to provide a lwrp for sensu-install which would collect all needed plugins and calls the sensu-install script only once with all plugins. I hope that such approach would reduce download times.

link is now a 404 here is an updated link: https://sensuapp.org/docs/0.25/reference/plugins.html#use-sensu-install-to-install-sensu-plugins

There is one called sensu_plugin, but it is not documented. I wonder why it cannot do a simple sensu-install -p, instead it is trying to find a local cookbook file.

You can also use sensu_gem like this:
default['cc']['sensu']['plugins'] = {
  ## pretty much all checks rely on this
  'sensu-plugin' => '2.1.0',
  ## check consul
  'sensu-plugins-consul' => '1.4.1',
  ## check cpu
  'sensu-plugins-cpu-checks' => '1.1.2',
  ## check disks
  'sensu-plugins-disk-checks' => '2.4.0',
  ## check disks
  'sensu-plugins-http' => '2.6.0',
  ## check elasticsearch
  'sensu-plugins-elasticsearch' => '1.5.1',
  ## check load
  'sensu-plugins-load-checks' => '3.0.0',
  ## check memory
  'sensu-plugins-memory-checks' => '3.0.2',
  ## check network
  'sensu-plugins-network-checks' => '2.0.1',
  ## check processes
  'sensu-plugins-process-checks' => '2.4.0',
  ## check rabbitmq
  'sensu-plugins-rabbitmq' => '3.2.0',
  ## check redis
  'sensu-plugins-redis' => '2.0.0',
  ## check chef
  'sensu-plugins-chef' => '3.0.2',
  'hashie' => '3.5.6',
  ## check nginx
  'sensu-plugins-nginx' => '2.2.0',
  ## check executables
  'mimemagic' => '0.3.2'
}

node['cc']['sensu']['plugins'].each do |plugin, version|
  sensu_gem plugin do
    version version
  end
end

I can probably add some documentation on how to use this. As in 2.x it is unclear how the asset pipeline will work for ruby I am hesitant to invest in using sensu-install (which will likely be replaced with some sub command of sensuctl) when it essentially does the same thing that sensu_gem already does (installs plugin into embedded ruby). Also I would argue that it's better than sensu-install since it can be used to install any arbitrary gem.

@sekrett I have created a PR to show an example of managing sensu-plugins using sensu_gem the way I do (and recommend) it.

Thanks, sensu_gem is exactly what I was looking for.


the requirement to include the package libmodsecurity3 is insufficient.  Missing files

Error message from Cargo
  --- stderr
  libmodsecurity cannot be found on the system:
  pkg-config exited with status code 1
  > PKG_CONFIG_PATH= PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 PKG_CONFIG_ALLOW_SYSTEM_CFLAGS=1 pkg-config --libs --cflags modsecurity modsecurity >= 3.0.6

  The system library `modsecurity` required by crate `modsecurity-sys` was not found.
  The file `modsecurity.pc` needs to be installed and the PKG_CONFIG_PATH environment variable must contain its parent directory.
  PKG_CONFIG_PATH contains the following:
      -

  HINT: you may need to install a package such as modsecurity, modsecurity-dev or modsecurity-devel.

  Vendoring is not supported at this time.

Dockerfile:
FROM rust:latest

RUN apt-get update && apt-get install -y \
    cmake zlib1g-dev lld

RUN apt-get update && apt-get install -y \
    libmodsecurity3 modsecurity-crs clang

Cargo.toml:
[package]
name = "test-modsec"
version = "1.0.0"
authors = ["redacted<EMAIL_ADDRESS>edition = "2021"

[profile.release]
incremental = true

[profile.dev]
incremental = true

[dependencies]
async-trait="0.1"
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
bytes = "1"
modsecurity = { git = "https://github.com/rkrishn7/rust-modsecurity", version = "0.1.4"  }

output of dkpkg -L from the docker container:
root@052754b216a5:/app# dpkg -L libmodsecurity3
/.
/usr
/usr/lib
/usr/lib/x86_64-linux-gnu
/usr/lib/x86_64-linux-gnu/libexec
/usr/lib/x86_64-linux-gnu/libexec/modsec-rules-check
/usr/lib/x86_64-linux-gnu/libmodsecurity.so.3.0.9
/usr/share
/usr/share/doc
/usr/share/doc/libmodsecurity3
/usr/share/doc/libmodsecurity3/changelog.Debian.gz
/usr/share/doc/libmodsecurity3/changelog.gz
/usr/share/doc/libmodsecurity3/copyright
/usr/lib/x86_64-linux-gnu/libmodsecurity.so.3

the missing package appears to be libmodsecurity-dev
this line in Dockerfile  appears to fix the problem
RUN apt-get update && apt-get install -y \
    libmodsecurity3 modsecurity-crs clang libmodsecurity-dev

You want a merge request for your README.md?

Hey @pszabop, thanks for raising this!
For context, modsecurity-sys relies on pkg-config to probe the system for libmodsecurity and emit necessary linking flags. pkg-config retrieves information about packages from special metadata files (.pc files) which it is configured to search for on some default set of paths. If the associated .pc file is located somewhere else, this needs to be conveyed to pkg-config by setting the PKG_CONFIG_PATH environment variable.
We probably don't want to include any comment about needing a specific package like libmodsecurity-dev, since that differs across systems and some users may even be building from source. However, it could be useful to point out a condensed version of the above information regarding pkg-config in the README.

You could say:
libmodsecurity-dev satisfies the need the .pc file on top of that.  That will help those not building from source.


cdml-prometheus-pumper to 0.0.1

Promote cdml-prometheus-pumper to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Do not merge/review - Initial commit #354

Initial commit for upgrading gradle to version 8.0.2
Context
Explain context and other details for this pull request.
Checklist

[ ] ./gradlew build compiles code correctly
[ ] Added new tests where applicable
[ ] ./gradlew test passes all tests
[ ] Extended README or added javadocs where applicable

Hello @sundargates,
I have made a few changes but this is not resolving the build. I believe this is due to java version 1.8.
The root project gradle was resolved but now issues are coming in mantis-client gradle file.
I'm still working on it.
Thanks.


RN 0.54 - Stylesheet Flow types incompatible with spread operator

Hi everyone, since the last react-native update, all my Stylesheet files are generating flow errors because of the spread Operator. Can someone help me resolve it ?
Environment
OS: macOS High Sierra 10.13.3
Node: 8.6.0
Yarn: 1.3.2
npm: 5.3.0
Watchman: 4.7.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 2.3 AI-162.4069837
Packages: (wanted => installed)
react: ^16.3.0-alpha.1 => 16.3.0-alpha.1
react-native: 0.54.0 => 0.54.0
Expected Behavior
No flow error, recognizing spread.
Actual Behavior

Steps to Reproduce
Just use a spread operator in a Stylesheet definition.

Unfortunately, due to a bug in Flow, stylesheets don’t support being spread into. My recommendation is to create a second style that is just fonts.BnpSansCondensed and pass your component an array containing both styles.
The good news is that flow will start enforcing that only view styles can be passed to view, and only text styles can be passed to text.


Issue on HA: 2021.6.2

Hey,
getting this issue, ive only just tried to set this up, never had it working before.
HA: 2021.6.2
sensor:

## https://github.com/jlamendo/ha-sensor.owlet
  - platform: owlet
    username: !secret owletuname
    password: !secret owletpassword
    region: "world"

Logger: homeassistant.components.sensor
Source: custom_components/owlet/sensor.py:145
Integration: Sensor (documentation, issues)
First occurred: 20:48:01 (1 occurrences)
Last logged: 20:48:01

Error while setting up owlet platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 250, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/owlet/sensor.py", line 286, in setup_platform
    for SmartSockDSN in Owlet.devices:
  File "/config/custom_components/owlet/sensor.py", line 193, in devices
    self.get_devices()
  File "/config/custom_components/owlet/sensor.py", line 168, in get_devices
    r = self.api_get("/devices.json")
  File "/config/custom_components/owlet/sensor.py", line 145, in api_get
    headers={"Authorization": "auth_token " + self.__OWLET_TOKEN},
TypeError: can only concatenate str (not "tuple") to str

This is probably a result of an authentication error. I'll admit, the error checking on this isn't up to what I'd normally do - I wrote it in the early months of my kid's life when I was sleep deprived and short on time :D I'd recommend double checking to make sure that the "username" is the email address you use to log into your account and not any actual username.

I double checked, it is and was the email.

I double checked, it is and was the email.

If you're based in Europe, as I am, then you need to change the region value on the sensor to "europe". I had the same issue and resolved it by doing that, as mentioned in the mbevand/owlet_monitor repo readme :)


hq2_14: crossbow no sound?

in hq2_14, the crossbow does not seem to have any firing sounds
Okay, it makes a firing sound if I kill some of the enemies...

seems like the channel is just full with weapon sounds, spirit bug I guess


TST/DOC: bump Python baseline 3.9 -> 3.11

And add 3.12 to the test matrix.

Number of failures is the same across both Python versions. For now, we consider that "good enough". ;-)
Merging.


orders container erroring on a frequent basis

The behaviour I expect from the app is that I can add items to my cart and see orders appear, however the orders pod is erroring and when I look at what the frontend pod is doing:
kubectl logs front-end-6585d48b5c-qxb5k -n sock-shop
I get:
Received response: "{\"timestamp\":1667937022064,\"status\":500,\"error\":\"Internal Server Error\",\"exception\":\"org.springframework.data.mongodb.UncategorizedMongoDbException\",\"message\":\"Query failed with error code 352 and error message 'Unsupported OP_QUERY command: find. The client driver may require an upgrade. For more details see https://dochub.mongodb.org/core/legacy-opcode-removal' on server orders-db:27017; nested exception is com.mongodb.MongoQueryException: Query failed with error code 352 and error message 'Unsupported OP_QUERY command: find. The client driver may require an upgrade. For more details see https://dochub.mongodb.org/core/legacy-opcode-removal' on server orders-db:27017\",\"path\":\"/orders/search/customerId\"}"
/usr/src/app/api/orders/index.js:31
            callback(null, JSON.parse(body)._embedded.customerOrders);

Use an older mongo image. mongo:5.0.11 works fine


fix build in RN 0.19+

build will fail in 0.19+ without this

sorry for the delay, and thanks for the pull request


Design Maybe: Get started buttons don't feel like buttons

This is based on some feedback we've received. However, let's let this lay for a bit for more feedback before committing to any changes.
Other feedback: Font too large?

I suggest that we wait and see if user feedback corroborates this.
These buttons were based on the design for https://alpha.ca.gov/ which was thoroughly tested. We even added the "arrow" icon to make them feel more actionable.

This is outdated & will be replaced by new work in Design System, closing.


Re-enable block-scoped-var + padded-blocks

Does this finally work? We've had this rule break so much stuff...

Can't sure for sure, all I know is the CI has passed. These two issues in regards each issue were fixed a couple months ago, dunno if they were the ones that were causing a problem before.
padded-blocks: babel/babel-eslint#33
block-scoped-var: eslint/eslint#2242

I'm merging with great fear...


The app crashes when using new Hermes engine

Issue type

[x] bug report
[ ] feature request

Issue description
Current behavior:

The app crashes when using new Hermes engine.
This only happened once I added the Icons Module, before that the app was working correctly with the Hermes engine.

Expected behavior:

The app works with or without the Hermes engine.
Steps to reproduce:

Other information:
OS, device, package version
"@eva-design/eva": "^1.2.0"
"@ui-kitten/eva-icons": "^4.2.0"
"react-native": "0.61.1"
"react-native-ui-kitten": "^4.2.0"

Hi @johanquiroga 👋
I guess this is mostly not a framework issue, just because Hermes engine does not currently support Proxies. See the related thread
And yes, this is because Icons module relies on Proxies, but this was the simplest solution for us. Btw, MobX users have the same issues with Hermes.

I think we should use module proxy-polyfill for this line:
https://github.com/akveo/react-native-ui-kitten/blob/05a3471c49916e8bdf8f7caaf3070b409582025e/src/eva-icons/createIconsMap.ts#L7
ie:
import PolyfillProxy from 'proxy-polyfill/src/proxy'

...
return new PolyfillProxy({}, {...})

@anhtuank7c does it resolves an issue?

@anhtuank7c I did some review on this now, and what I get is that we don't get application crashes, but for any reason it's unable to find requested property, e.g:
const HeartIcon = () => <Icon name='heart' />

will lead to runtime error of @ui-kitten/eva-icons package saying 'heart' does not exist

Hi all
In my case, this issue is present only on android and my currently workaround is :

use the solution showed by @anhtuank7c
removed all the import { Icon } from '@ui-kitten/components'; and use:

import React from 'react';
import { Image } from 'react-native';
import { Button } from '@ui-kitten/components';

const RemoteBulbIcon = (style) => (
  <Image
    style={style}
    source={{ uri: 'https://akveo.github.io/eva-icons/outline/png/128/bulb-outline.png' }}
  />
);

export const IconExternalSourceShowcase = (props) => (
  <Button icon={RemoteBulbIcon}>
    Login with Facebook
  </Button>
);

icons from: (https://akveo.github.io/eva-icons/#/)
Other information:
OS, device, package version:
    "@eva-design/eva": "^1.3.1",
    "@ui-kitten/eva-icons": "^4.3.2",
    "react-native-ui-kitten": "4.3.2",

@AndreaTurco the reason is that Hermes engine doesn't support proxies. This engine is used to increase Android performance, and it doesn't affect iOS in total. Currently we have no workaround to get rid of proxies, so if you really need Hermes - your solution would work, but will affect UX because of loading icons remotely instead of using SVGs.

Good news ever!!!!
https://twitter.com/HermesEngine/status/1245136667414913024

@anhtuank7c thanks for sharing 👍

@anhtuank7c  Please any workaround? am using react-native 0.62  but I still have this error...

@lewatt23 Did you try npm install<EMAIL_ADDRESS>

@anhtuank7c My bad thought I was installed by default since I was using rn 0.62.2, just installed and is working fine, thanks :)

for those using 0.63 use npm install<EMAIL_ADDRESS>

for those using 0.63 use npm install<EMAIL_ADDRESS>
It worked after clean gradle builds
Thanks!!

0.64 will have Proxy enabled by default on Hermes. Woohoo!

for those using 0.63 use npm install<EMAIL_ADDRESS>
Workssss


WebAssembly with Reference Types cannot be parsed

Reopened #13559
Bug report
WebAssembly modules with enabled Reference Types cannot be loaded. The webpack version I tested this with is 5.38.1
For a minimal reproduction I created this repository: https://github.com/Tarnadas/webpack-5-wasm-reftypes
The error message is the following:
ERROR in ./pkg/webpack_5_wasm_reftypes_bg.wasm
Module parse failed: Internal failure: parseVec could not cast the value
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
Error: Internal failure: parseVec could not cast the value
    at new CompileError (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/helper-api-error/lib/index.js:40:109)
    at parseVec (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/wasm-parser/lib/decoder.js:328:15)
    at parseTypeSection (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/wasm-parser/lib/decoder.js:360:22)
    at parseSection (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/wasm-parser/lib/decoder.js:1363:24)
    at Object.decode (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/wasm-parser/lib/decoder.js:1712:25)
    at decode (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/@webassemblyjs/wasm-parser/lib/index.js:248:21)
    at WebAssemblyParser.parse (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/webpack/lib/wasm-async/AsyncWebAssemblyParser.js:48:19)
    at /home/marior/projects/webpack-5-wasm-reftypes/node_modules/webpack/lib/NormalModule.js:1018:26
    at processResult (/home/marior/projects/webpack-5-wasm-reftypes/node_modules/webpack/lib/NormalModule.js:745:11)
    at /home/marior/projects/webpack-5-wasm-reftypes/node_modules/webpack/lib/NormalModule.js:809:5
 @ ./pkg/webpack_5_wasm_reftypes.js 1:0-58 4:0-21
 @ ./src/index.js 1:0-16

webpack 5.38.1 compiled with 1 error in 572 ms
error Command failed with exit code 1.

I used Rust with wasm-bindgen, which provides a cli option to enable reference types. Instead of directly using wasm-bindgen, I’ve been using wasm-pack, but since it’s unmaintained, you will need my fork to pass the cli option down to wasm-bindgen. You can see my PR for wasm-pack here: rustwasm/wasm-pack#888
By looking at the error message, it seems to be a problem with the wasm parser library. I checked its repository and couldn’t find any mentions about reference types unfortunately, so I’m not sure how to proceed with this.

I think parser does not support them.. (@webassemblyjs/wasm-parser) . Right now there is no workaround for this.

Why webpack try to parse it? can we just use file-loader?

Why webpack try to parse it?

There are some exports optimization.. so webpack parsing wasm.

can we just use file-loader?

yes, sure. also in webpack@5 there is asset type.

@xtuc

Last time you at least gave a warning, and it's marked as completed?? This has to be the worst bot ever.

Ok dude

We can't fix it on our side, but you can use:
module: {
      rules: [
        {
         // You can provide better regexp
          test: /\.wasm$/,
          type: "asset/resource"
        },
        {
          test: /\.js$/,
          exclude: /node_modules/,
          use: [
            {
              loader: "babel-loader",
            }
          ]
        }
      ]
    }

But you lose some optimizations. Feel free to feedback

In two days (2024-10-17) Rust 1.82 will be released, and reference types will be on by default. The only way to build without reference types will be to use nightly and rebuild the standard library without reference types support.

Rust 1.82.0 is now released with the following note

The WebAssembly target features multivalue and reference-types are now both enabled by default. These two features both have subtle changes implied for generated WebAssembly binaries. For the multivalue feature, WebAssembly target support has changed when upgrading to LLVM 19. Support for generating functions with multiple returns no longer works and -Ctarget-feature=+multivalue has a different meaning than it did in LLVM 18 and prior. There is no longer any supported means to generate a module that has a function with multiple returns in WebAssembly from Rust source code. For the reference-types feature the encoding of immediates in the call_indirect, a commonly used instruction by the WebAssembly backend, has changed. Validators and parsers which don't understand the reference-types proposal will no longer accept modules produced by LLVM due to this change in encoding of immediates. Additionally these features being enabled are encoded in the target_features custom section and may affect downstream tooling such as wasm-opt consuming the module. Generating a WebAssembly module that disables default features requires -Zbuild-std support from Cargo and more information can be found at rust-lang/rust#128511.

https://github.com/rust-lang/rust/releases/tag/1.82.0

I just updated to 1.82 and now this problem started happening, what is the recommended solution?

This is btw not just a Rust issue, as it should affect C, C++, Zig and co. the same way as it got changed in LLVM to be turned on by default.

@Havunen can you try with the beta compiler? I'm weirdly not able to reproduce it with 1.83 or 1.84.

Stripping solves the problem:
[profile.release]
strip = true

This isn't great for debug builds however, as you usually want the debug symbols. I believe the reason why stripping works isn't because it does any stripping, but because it likely optimizes the overlong immediates encoding back to how it was before LLVM 19. Running wasm-opt probably behaves the same way.

@CryZe What problem did you solve by stripping? I tried stripping but I still get the same error:
Can't import the named export '__wbindgen_start' (imported as 'wasm') from default-exporting module (only default export is available)

Without the babel-loader workaround that you tried.

Sorry, yeah, I just tried that and it worked (edited my reply)

Just as a FYI, it's still not possible to actually use reference types (as far as I can tell). Stripping, gets me as far as "my old binary without reference types works".
To get reference types to even pass compile/wasm-pack/wasm-opt I needed to:
//.cargo/config.toml
[target.wasm32-unknown-unknown]
rustflags = [
    "-C", "target-feature=+bulk-memory,+mutable-globals"
]

// Cargo.toml
[package.metadata.wasm-pack.profile.release]
wasm-opt = ['-Oz', '--enable-bulk-memory']

// Shell / Building
  wasm-pack build \
    --release \
    --weak-refs \
    --reference-types \
    --target bundler \
    --scope "myscope" .

But then webpack says
Module parse failed: Internal failure: parseVec could not cast the value
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
Error: Internal failure: parseVec could not cast the value

I don't need reference types, I just wanted to see how far I would get.

I believe the reason why stripping works isn't because it does any stripping, but because it likely optimizes the overlong immediates encoding back to how it was before LLVM 19.

I can confirm this, as switching back to WebAssembly MVP features with:
[package.metadata.wasm-pack.profile.release]
wasm-opt = ['-Os', '-mvp', '--strip-target-features']

does not solve the problem. I did not find a way to use wasm-opt to change the immediate encoding back to something supported by webpack.

Note that in big projects with a workspace having multiple crates, the current webpack limitation is really annoying, because due to a cargo limitation, the strip option must be specified at the workspace level, so that would affect all crates including the ones not compiled for WASM targets. As @CryZe mentioned, that is really a problem for debugging.

On the weekend I tracked down the issue back into wasm-bindgen which apparently uses the target_features section to detect if it should automatically enable its full reference types support. I opened a PR that ensures that you still have to opt into that manually, so that should fix all issues that were caused by Rust 1.82: https://github.com/rustwasm/wasm-bindgen/pull/4213

@bes can you make a reproducible example so that I can examine it?
@stephanemagnenat

Note that in big projects with a workspace having multiple crates, the current webpack limitation is really annoying, because due to a cargo limitation

This is not our limitation because we simply cannot run the parser, it crashes with an error, this should be fixed in @webassemblyjs/wasm-parser, that's why I ping @xtuc

@alexander-akait here you go https://github.com/bes/webpack-wasm-reference-types-1.82-poc

Version 1.13.1 has been published with reference types. We need to update in webpack.

@xtuc I tried 1.13.1 update and still get - Error: Internal failure: parseVec could not cast the value, repro - https://github.com/bes/webpack-wasm-reference-types-1.82-poc

The same - https://github.com/webpack/webpack/pull/18940

Thanks for checking @alexander-akait . I'm having a look.

@xtuc Works for me!
    "@webassemblyjs/ast": "^1.14.1",
    "@webassemblyjs/wasm-edit": "^1.14.1",
    "@webassemblyjs/wasm-parser": "^1.14.1",

https://github.com/mateuszJS/rust-webpack-v5-template/tree/webpack-v5-migration
Thank you for great work 👍

@xtuc works for me as well! Thank you very much!

I'm still getting the error "Module parse failed: Internal failure: parseVec could not cast the value"
https://github.com/ivnsch/molsimnext
the only way to fix it is still the workaround
[profile.release]
strip = true

@ivnsch what version do you use?


Fix missing code during in_podman build

Signed-off-by: Chris Evich<EMAIL_ADDRESS>

LGTM

This will fix the in_podman builds that have been failing for the last few days.

/approve
@mheon @baude @giuseppe @vrothberg @TomSweeneyRedHat @jwhonce @QiWang19 @ashley-cui PTAL


Expanding the allowable descriptions for vision system tests.

Fixes #3681.
Also updating the nox configuration to allow posargs pass through and be a little more Windows path friendly.

If the build goes green I'm gonna throw a party. Last green build May 17th!!!!!!!!!

OK oops. I've gotta get all the google-cloud-core upper bounds increased to get green. But it shall be done!

Fingers crossed: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/2718

ZOMG GREEEEEEEN! https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/2736
:tada: :balloon: :zap: :rocket: :panda_face:


ERR_UNHANDLED_REJECTION with killOthers

Hi!
I have a basic setup like:
import concurrently from 'concurrently'
const processes = concurrently([{
  command: 'free --human --seconds 1',
  name: 'Log memory',
}, {
  command: 'sleep 5',
  name: 'Sleep'
}], {
  killOthers: [
    'failure',
    'success'
  ]
})

This works like:
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       824Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Log memory]
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       824Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Log memory]
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       827Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Log memory]
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       825Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Log memory]
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       825Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Log memory]
[Log memory]                total        used        free      shared  buff/cache   available
[Log memory] Mem:           7.6Gi       825Mi       5.9Gi       3.0Mi       887Mi       6.5Gi
[Log memory] Swap:          2.0Gi          0B       2.0Gi
[Sleep] sleep 5 exited with code 0
--> Sending SIGTERM to other processes..
[Log memory] free --human --seconds 1 exited with code SIGTERM
node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "[object Array]".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

As you can see, it's working fine for the most part except towards the end. Not sure why it fails with ERR_UNHANDLED_REJECTION.
If I remove killOthers, and manually exit by pressing Ctrl + C, it works without an issue. I also tried the following:
import {spawn} from 'node:child_process'
const freeProcess = spawn('free', [
  '--human',
  '--seconds',
  '1'
])
freeProcess.stdout.on('data', (data) => {
  console.log(data.toString())
})
setTimeout(() => {
  freeProcess.kill('SIGTERM')
}, 5000)

And it exits correctly without any issues. Am I missing something?
I'm on Ubuntu 22.04 running Node.js 20.12.2.

Hey. You need to handle the result, like this:
concurrently([ /* commands */ ], { /* options */ }).result.then(
  () => console.log('exited successfully'),
  () => console.error('exited with error'),
)

Thank you for the quick reply! That works! I read that part in the readme, but didn't think that was necessary to do. My assumption was this won't result in an error, but I should have just tried that path out first, sorry for the noise.


Add alternate gradient version for light mode

Current purple gradient only blends well with dark themes. I hope you can consider alternative shade of the purple color (or the bottom  side of the gradient?) to make it compatible visually with light mode themes (Light and Zen Dream)

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

And the icon is hidden behind the URLbar, if you have stretched URLbar.

A more pleasant purple has been added for light mode and icon overlapping has been prevented. A PR has been raised with the Zen theme store - https://github.com/zen-browser/theme-store/pull/223

This has just been merged in, and updating your themes from the Look and Feel settings should add the light mode variations as well as a togglable border around the web page frame.
If updating doesn't work now, then it should do in a few hours when the updated themes are added to the Zen themes page.


Added license to the pyproject.toml file

Hi, this PR was sent as part of OSDC taught by: @szabgab.
By adding the license to this file, it will be shown on the website pypi.org

Looks nice, @ranms25 ! Thank you!


Address Search

Sorry to ask this question here but i suspect it will take someone all of 2 seconds to give me an answer. I see a lot of single sentence examples but none where there are large sets of data....
I have a list of 70,000 addresses that i need to be able to search through that are stored in a MySql database. I want to use natural to perform searches probably using string distance to determine if the address is a match to show user. Some of the use cases are where they spell it wrong or where they have abbreviated (i.e. Ave vs Avenue, St. vs Saint, Altanta vs Atlanta). Additionally, after it is found i need to return the identification code for the address, so that it may be used on UI... If the user selects the address the code will be used for downstream processing.
How do i load all 70,000 into memory like that so that natural can search all 70,000? Is this dangerous loading so much data into memory....
I guess I am lost as to how you search large datasets like this... particularly when they are stored in a database.?

I think you are looking for an autocomplete solution for addresses. Which is not functionality of natural...

@Hugo-ter-Doest no that is not my question... the question is using naturaljs what is the best approach to searching through large datasets?


StackedBarCharts needs to explain "stringsAsFactors = F"

Addressed in https://github.com/ox-it/LiveData_htmlwidgets/commit/8106fd986156add43f9859c0b72d8c52665f1339


Add datacenter to the glossary

Proposed changes
Added datacenter to the glossary

Related issues (optional)
Fixes #2734

Great work, thanks, @shubheksha :)


Increase timeout and vite version

Pull Request Description

The electron test hit timeouts from time to time, probably because of slow response from the engine: increased timeout.
Unify vite version across packages.

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.
[ ] If meaningful changes were made to logic or tests affecting Enso Cloud integration in the libraries,
or the Snowflake database integration, a run of the Extra Tests has been scheduled.

If applicable, it is suggested to paste a link to a successful run of the Extra Tests.

:white_check_mark: electron watch script still works, i am assuming GUI dev mode has been tested already as dashboard and project view use the same dev server.


Implement Artifact Garbage Collection for OSS Artifacts

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

This is already supported, right?

This is already supported, right?

https://github.com/argoproj/argo-workflows/blob/3b4551b6f33bd22519cc98e2d8c157a92720b5d7/workflow/artifacts/oss/oss.go#L225
Not yet, I will submit a PR to implement it.:) @shuangkun


Hangs on "list_buckets"

This is an awesome looking tool, and I can't wait to use it. Thank you for creating it.
When attempting to mount B2, the software hangs at "list_buckets"
$ sudo python b2fuse.py /mnt/b2
2016-04-07 08:48:30,571:INFO:list_buckets

The bucket contains no files.

This confused me at first, as well. I believe that is all there is to it. You can run it in the background or something, and then start using your mount-point. However, what I've seen is that the mount-point has no permissions at all-- I had to use the root account to even use it! I have not figured out how to get around that. For now I'm only "playing" with this to see what it is capable of, though. I hope someone can offer suggestion on this issue. I might even be using it wrong, too.

Hi @hopechannel  and @CafeLungo!
There is really not much too it, the program is not designed to run as a daemon yet so it will block the terminal launching the program. @hopechannel you should have gotten some more output from the terminal, what happens if you try to create a file in the mounted folder?
Inherently, B2 does not support permissions. Some applications mimic permissions in B2 using extra file attributes (meta information stored with the file). B2Fuse does not care about permissions, so you should be able to access the files as any user. Have you checked that the mount point itself is accessible using a non-root user?
As for performance, the non-local version of this driver attempts to be as transparent as possible in exposing the B2 API. As such, what you are seeing in terms of delay for opening, deleting and listing files are the result of the the latency in B2 API calls. Typically a single call takes 1-2.5 second to execute. There has been significant work on speeding up this driver, in particular the file listings are cached at multiple levels (which should make exploring folders faster). There is only so much one can do about the performance when deleting files and opening (downloading). If I were to use the same approach as with B2Local (which use threads for upload, deletions and downloads), you would get inconsistent results. Especially, if you terminate the driver before it has completed all operations or try to access the bucket through another application (accessing the B2 API).
Future work includes making use of the recently released B2 large file upload, which may allow for large files to be handled better by downloading and uploading parts of a file at a time.
Hope that answers some of you questions.
Sondre

Hi @sondree! Hey thanks a lot for the robust reply!
When I first tried running your program, it gave me errors when trying to access the mount.
I tried again after reading your your message, and it worked great.
The only thing I can surmise is that my initial attempt was only a few minutes after creating my B2 account and the new bucket.
I'm very excited about this software you're writing, and am looking forward to the new large file support. We're a television network with thousands of broadcast master video files that are over 20GB each.


Small bindata + net-att-def cleanup

Removes bindata from Dockerfile and temporarily moves net-attach-def perms to cluster-scope for the meantime

/retest

/lgtm


How do you implement controlled select?

I'm using this in a controlled way. My problem is that it seems it's not possible to do a controlled select.
I mean the whole component is controlled, but select is going through a spaghetti (in my case) of onSelect + treeAPI.deselectAll() + treeApi.selectMulti().
But then it can get in an infinite loop, with a selectMulti triggering onSelect, etc. triggering selectMulti, etc.
Normally in React these are solved by controlled components. In Arborist's case, it'd be something like a selectedMap={} property, which takes a Map of selected id and selects them.
How do you solve this problem?

Yes, that's exactly what I did as well. Basically I manage the whole
selected state myself, instead of using the built-in one.
On Wed, 29 May 2024 at 14:59, Pancham Khaitan @.***>
wrote:

Okay, so I got around this with a little hack if you will.
I overrode the props of the Node and passed a locally maintained state to
the checkbox.

{(props) => <Node isSelected={isSelected} setIsSelected={setIsSelected} {...props} />}
—
Reply to this email directly, view it on GitHub
https://github.com/brimdata/react-arborist/issues/243#issuecomment-2137356301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYVD3MPTSHVSRRJ3G2M4TZEXGKHAVCNFSM6AAAAABGE4G2I6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZXGM2TMMZQGE
.
 Been struggling with implementing a solution for a bit now.
I'm not sure why you're passing in setIsSelected as props. Where is that coming from? Is it a function that exists in your outer react component (outside of the tree?)
Thanks

@Pancham97 Can you give a little more detail on this? Been struggling with implementing a solution for a bit now.
I'm not sure why you're passing in setIsSelected as props. Where is that coming from? Is it a function that exists in your outer react component (outside of the tree?)
Thanks

@Joshlha yeah pretty much. Since React Arborist wouldn't maintain the state of selection, I made my own state using useState and pass the state and the setter as props. Whenever the checkbox is selected, it calls the setState method. I could whip up a small code gist if you want.

@Pancham97 I think I get what you're saying. I would imagine that isSelected holds the state of all nodes? And the set function sets the state of a particular node.
A code sample would be extremely helpful! I've been fighting with the built-in selection management provided by the tree and it's just not very flexible.

@Joshlha I agree.
Here's something that I might have done. I do not have the exact piece of code because I didn't think this was an elegant use of a tree. For my use-case, I went with the standard accordions since that fit my need better.
function Node({ isSelected, setIsSelected, ...props }) {
  return (
    <input
      type="checkbox"
      checked={isSelected}
      onChange={() => setIsSelected(!isSelected)}
      {...props}
    />
  );
}

function Parent() {
  const [isSelected, setIsSelected] = React.useState(false);
  return (
    <Tree
      ref={treeRef}
      data={data}
      disableDrag
      disableMultiSelection={false}
      className="selection-tree"
      onSelect={handleSelect}
    >
      {(props) => (
        <Node
          isSelected={isSelected}
          setIsSelected={setIsSelected}
          {...props}
        />
      )}
    </Tree>
  );
}


fix #3613

添加了对key中存在冒号':'、逗号','时的支持。
fix #3613

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

Codecov Report

Merging #3739 (dfd97ef) into master (14a95cc) will decrease coverage by 0.03%.
The diff coverage is 73.91%.

@@             Coverage Diff              @@
##             master    #3739      +/-   ##
============================================
- Coverage     80.30%   80.27%   -0.04%
+ Complexity     7519     7518       -1
============================================
  Files           157      157
  Lines         28096    28103       +7
  Branches       6790     6791       +1
============================================
- Hits          22563    22559       -4
- Misses         3499     3504       +5
- Partials       2034     2040       +6

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/alibaba/fastjson/JSONPath.java
74.49% <73.91%> (-0.10%)
279.00 <0.00> (ø)

...com/alibaba/fastjson/serializer/SerialContext.java
82.69% <0.00%> (-3.85%)
21.00% <0.00%> (-1.00%)

...main/java/com/alibaba/fastjson/util/FieldInfo.java
85.76% <0.00%> (-1.00%)
97.00% <0.00%> (-2.00%)

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
86.89% <0.00%> (-0.17%)
216.00% <0.00%> (ø%)

...java/com/alibaba/fastjson/parser/ParserConfig.java
80.46% <0.00%> (-0.15%)
226.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


⚠ Turnos 24x7 has degraded performance

In eefb27a, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1063 ms

Resolved: Turnos 24x7 performance has improved in 5b04cae.


Merge bitcoin#14636: Avoid using numeric_limits for sequence numbers and lock times

Merges bitcoin/bitcoin#14636: Avoid using numeric_limits for sequence
numbers and lock times.
535203075e Avoid using numeric_limits for sequence numbers and lock
times (Russell Yanofsky)
bafb921507 Remove duplicated code (Hennadii Stepanov)
e4dc39b3bc Replace platform dependent type with proper const
(Hennadii Stepanov)
Pull request description:
Switches to named constants, because numeric_limits calls can be
harder to read and less portable.
Change was suggested by jamesob in
https://github.com/bitcoin/bitcoin/pull/10973#discussion_r213473620
There are no changes in behavior except on some platforms we don't
support (ILP64, IP16L32, I16LP32), where SignalsOptInRBF and
MutateTxAddInput functions would now work correctly.

The diff looks good. However, please base all work off of the develop branch. I have changed the 'base branch' in github, however you PR now has conflicts that need to be resolved by rebasing on develop. Let me know if you need help on doing that.
Also I changed the title to be more inline with expected.
Overall looks good!

Ahh, I see. Thanks! Rebase done and conflicts resolved.


jQuery 1.7 support

Hi!
We pretty much love what you have created but using jQuery 1.7 and your component drives us nuts as the component behaves extremly different/buggy - is there a 1.7 compliant version?
Cheers, Florian

What version are you using? The latest release 0.2.8 or the dev version from the master branch? Could be a little more specific what does exactly not work?


FancyZones "Launch editor on the display" feature does not appear to honor setting

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce

Open PowerToys Settings
Click on FancyZones
Go to "Launch editor on the display" and set to "Where the mouse pointer is"
Place PowerToys Settings window to monitor 2 and use mouse to click on Launch layout editor

Please note that repeating the same steps but with the setting to "With active focus" results in the same behavior.
✔ Expected Behavior
Layout editor opens on Monitor 2
❌ Actual Behavior
Layout editor opens on Monitor 1
Other Software
No response

This appears to be fixed! Going to close :)


Downgrade kafka-python to 1.3.4 to prevent DNS spikes when broker doesn't exist

What does this PR do?
Temporary downgrade kafka-python to version 1.3.4
Motivation
We noticed a huge spike in the number of DNS calls coming from the check when a broker listed on the config file doesn't exist. While this is definitely a misconfiguration problem, we want to investigate the spike, reverting to a known state in the meantime.
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

This was fixed in https://github.com/dpkp/kafka-python/pull/1507, which is part of the recently released 1.4.3 version.


Getting Error "Starting to build a new database" in Local BLAST

Hi Till,
I am getting an Ix_xI unexpected error Ix_xI when I try to use "Build a new database" option in Local BLAST section.
1. I'm selecting my "...._done.fasta" and "read (...table_filtered.xlsx)" files from here as requested files:

2. Then selecting Midori2 and relevant "...gb...fasta" file :

3. ...and getting error like this:

and my "log.txt" says

Your database: DB_MIDORI2_LONGEST_NUC_GB255_CO1_BLAST
Your task: blastn

Is there something I am missing?
Thanks for your time and efforts in developing compatible programs.
Onur

Can retry to build the DB and press the EXIT button and tell me what is printed to the console?
For now everything looks correct from the screenshots you posted.

Hey again, here they are:

12:13:06: Starting to build a new database.
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/bin/apscale_gui", line 33, in
sys.exit(load_entry_point('apscale-gui==1.1.6', 'console_scripts', 'apscale_gui')())
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/apscale_gui/main.py", line 768, in main
create_database_midori2(values2['database_source_file'], project_folder, databases_path)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/apscale_gui/blast_utilities.py", line 479, in create_database_midori2
subprocess.call([makeblastdb_exe, '-in', str(midori2_fasta), '-dbtype', 'nucl', '-out', str(db_name)])
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 349, in call
with Popen(*popenargs, **kwargs) as p:
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 1821, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'makeblastdb

It seems like you do not have the blast+ software installed. APSCALE is looking for the 'makeblastdb' command but cannot find it.  Could you please install blast+ and then try it again?
cheers
Till

Heya,
Thanks for pointed out this. Actually, it had already been installed before, but it was holding my download directory instead of intended location "/Usr/local/bin". This is likely why it didn't work . After moving it to the "bin" folder, I encountered a security alert window from the apscale_gui because it thought that "makeblastdb" had been installed from an untrustworthy source. To resolve this, I provided some permissions, and it worked properly afterwards.
In summary, I did something, and it worked well thanks.
Onur

Hey again,
I encountered an error while trying to create a new database in the same location where I had previously created a successful one -this older one is still functioning efficiently.
Steps:
Load Requires Files

I've provided both "...fasta" and "...OTU_filtered.xlsx (as Read file)" files as required input files, which are located in the "9_lulu_filtering/otu_clustering" folder.

Build a new database

I've selected "Midori2" as a format.
I've selected a new database source file named [MIDORI2_UNIQ_NUC_GB257_CO1_BLAST.fasta] which is locating in the APSCALE_Databases folder.
Build a new database [Go]

then...
onurdogan@Onurs-iMac ~ % apscale_gui
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/bin/apscale_gui", line 8, in <module>
  sys.exit(main())
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/apscale_gui/__main__.py", line 772, in main
  create_database_from_fasta(values2['database_source_file'], project_folder, databases_path)
TypeError: create_database_from_fasta() missing 1 required positional argument: 'databases_path'

Enjoy,


Add an identifier to work done progress messages

Description
Work done progress messages ($/progress) contain a message and a title, but there is no id to match begin/end events up together. It is possible to use the title, but if developers want to check for specific types of work progress they'll have to key off the title and if that title ever changes in future then their code will break.
It would be nice to add an id field to these progress messages that is considered immutable. One catch is there is no id field in the protocol definition so this would be out of spec.

Synced to Apple’s issue tracker as rdar://129678779

Adding a consistent prefix to the token of the work done progress here: https://github.com/apple/sourcekit-lsp/pull/1493
For crashed sourcekitd, this will be sourcekitd-crashed.


gunzip FILE, but output file is not complete

I have a file compressed to xxx.gz, through ZLIB I think, not gzip from toybox.
size of xxx.gz is 277KB.
When decompressed in windows, the output file's size is 2351KB, which is as expected.
However, when I decompress xxx.gz througn gunzip of toybox, the output file  turns out to be incomplete, size is only 49KB, the decompression seems to stop at 49KB.
Is there something wrong about gunzip?

what does 'final' mean? Is it part of GZIP format?

Yes, it's in rfc 1951: https://www.ietf.org/rfc/rfc1951.txt#:~:text=BFINAL
"BFINAL is set if and only if this is the last block of the data set."
That said, zcat should be able to accept multiple concatenated zip files and treat them as one file, which isn't a thing I knew when I wrote it. You didn't attach a test case reproducing the issue, but let me see if I can reproduce it here...

Yes, it's in rfc 1951: https://www.ietf.org/rfc/rfc1951.txt#:~:text=BFINAL
"BFINAL is set if and only if this is the last block of the data set."
That said, zcat should be able to accept multiple concatenated zip files and treat them as one file, which isn't a thing I knew when I wrote it. You didn't attach a test case reproducing the issue, but let me see if I can reproduce it here...

sorry, I am writing from inside my company, can't send out any files. Only pictures is okay~
BUT, I added some log,
I think that in decompress, or infalte process, It is trying to convert from huffman code to literbal symols. And it stops at dd->pos[50160], at the 2439th symbol. It corresponds to the output file's size.
 50165 Dec 13 18:40  hilog_kmsg.001.19700122-114349

What could possilbly caused this? Inflating stops early

256 is the end-of-block symbol. Again from the RFC:
https://www.ietf.org/rfc/rfc1951.txt#:~:text=end-of-block

mbol. Again from the RFC:

yes. So the issue left, gunzip(toybox) stops early, decoded an EOF symbol. Whereas by gunzip(Ubuntu) and windows, it runs till the end of the file.

I've reproduced this, by the way:
$ cat <(echo hello | gzip) <(echo there | gzip) | zcat
with host zcat, shows both. With target zcat, just shows hello. I'll try to come up with a fix later today.

Hi @landley , any progress with the fix :)
Apparently I tried continue instead of break the loop when EOF is decoded, and the decoding after that goes wrong :)

Ah, sorry. Got distracted by family and the holidays.
The deflate plumbing buffers input data. Discarding the buffer it hasn't processed yet discards the next part of the file, so I need to change the internal API a bit to tell it how to resume with existing input.
Not a big deal, but not the 5 minute fix I originally thought.

Try commit 8f8b49ffedde

Try commit 8f8b49f

Seems fixed, Thanks a lot :)


Update suggested clangd flags in lang/cc docs

This adds a missing clangd flag for LSP, needed to make it work correctly with company-mode.
It's from the default value of lsp-clients-clangd-args.
This is with clangd version 11.1.0.
If not set, I find C++ completion will produce a single space instead of showing up the list, in a case like myVariable., triggering completion after the dot character.

LGTM. Thanks!


Required node engine

It really require Node >= 20.10.0 ?
Stylelint require Node >= 18.12.0, stylelint-formatter-gitlab-code-quality-report may require the same version.

@ben12 could be that it works on >= 18.12.0. I did not test it. I was using 20.10.0 on my local machine.

Thx! merged it and created https://github.com/tijsverkoyen/stylelint-formatter-gitlab-code-quality-report/releases/tag/v1.1.0
I had mistagged the first release, so that is why it is v1.1.0


Renamed tech name in klayout .lyt file to sky130A

The underlying problem is that the file, like most files processed by the Makefile, is supposed to not have the PDK name in it at all.  The PDK name should be replaced by the bareword TECHNAME:
${CPP} ${SKY130A_DEFS} klayout/${TECH}.lyp > ${KLAYOUT_STAGING_A}/${SKY130A}.lyp
where SKY130A_DEFS = -DTECHNAME=sky130A -DREVISION=${REVISION} (plus other stuff).

Ah, got it. Updated the commit with TECHNAME instead.

Looks good;  merged and closed.


[Relay] Add a unit test for structural equality

This is CORE-135 from the forums, which suggested structural equality
was deeply broken. But unable to repro. No harm including unit test.

@comaniac might as well put this in.


Refcator: rename utilities to helpers

Preview is available here:http://laboratory-pr572.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr572.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr572.previews.kube001.services.stellar-ops.com/


Some code will be checked by google play store and report a issue that your app(s) are using an unsafe implementation of the HostnameVerifier interface.

The code below will be checked by google play store and report a issue:

Your app(s) are using an unsafe implementation of the HostnameVerifier interface.

android/src/main/java/com/pichillilorenzo/flutter_inappwebview/Util.java
      builder.hostnameVerifier(new HostnameVerifier() {
        @Override
        public boolean verify(String hostname, SSLSession session) {
          return true;
        }
      });

有进展吗？

I'm now able to publish an update of my app due to this issue. Google is rejecting any binary that has this security vulnerability.

@pichillilorenzo do you have any thoughts on this? Seems like quite a major issue for a popular library. What was the initial intention of accepting all ssl certs?

Have you experienced again this issue? This is used by the ContentBlocker Android implementation to convert any http request to https.
In simple terms, to implement the make-https Content blocker action as on iOS https://developer.apple.com/documentation/safariservices/creating_a_content_blocker


Drop support for Python 3.2 and 3.3

Python 3.3.7's end-of-life was September 2017.
https://www.python.org/downloads/release/python-337/

Hey, thanks for the PR and apologies for the delay in handling it. Will be reflected in the next release


Swift version.

Is there any supported version for swift?

You can use it via bridging header.
Check out this link

Noted.


windows builds gone?

Comment:
I couldn't see who/why the windows was removed from the build matrix upon the v3.5.0 update. Could they be reactivated, or is there technical trouble? Thanks!

I deactivated the Windows build after hitting more problems with GCC 5.3, patching was also not trivial like before.

ok, thanks. Intel compilers seem increasingly available, so I'll let you know if I spot a Win+Fortran+Intel recipe.

With the update of MinGW I could reenable the build for dftd3 on Windows already. For dftd4 I am stuck with linking Lapack, see https://github.com/conda-forge/dftd4-feedstock/pull/22. Any suggestions are welcome.


ModuleNotFoundError: No module named 'deepspeed.ops.op_builder' on import deepspeed

Getting this error on import of deepspeed. I am currently using torch 1.8.0 and installed the requirements.txt as directed. I am also not able to install the apex link provided.

It would be of great assistance if you could provide more detail about the nature of the issue. What is your environment? What steps can we take to reproduce? We ask that you follow the Issue template for bug reports, as it specifies the information we are interested in.

I must note that pip install torch==1.8.0 torchvision==0.9.0 torchaudio==0.8.0 installs PyTorch wheels retrieved from PyPi and not retrieved from PyTorch directly. I also see that the APEX install says that you have installed the CPU version of PyTorch, which seems to conflict with other information I have (both installing from PyPi and your own words). By following that trail I was able to replicate the issue with APEX install with torch==1.8.0+cpu.
I therefore recommend installing PyTorch as outlined in the PyTorch Getting Started guide. In your case that would be pip install torch==1.9.1+cu111 torchvision==0.10.1+cu111 torchaudio==0.9.1 -f https://download.pytorch.org/whl/torch_stable.html (alternatively any of the CUDA 11.1 builds of PyTorch since 1.8 will do). If this gives you trouble, install CUDA toolkit 11.1 for the installation of DeepSpeed, APEX and the Megatron kernels, as if my memory serves me right APEX will complain and installation will fail if the CUDA toolkit version does not match the one used to build PyTorch.

Thanks I followed your instructions for installing torch directly from PyTorch without changing my nvidia toolkit and I'm not getting the previous error anymore. But now I'm getting the error that the triton version is not available.:
ERROR: Could not find a version that satisfies the requirement triton==0.4.2 (from deepspeed) (from versions: 0.1, 0.1.1, 0.1.2, 0.1.3, 0.2.0, 0.2.1, 0.2.2, 0.2.3, 0.3.0)
ERROR: No matching distribution found for triton==0.4.2 (from deepspeed)
I am using python 3.8.0 and updated my pip version as well.

To answer your questions, yes I am currently trying to run gpt-neox on windows. But that's just for testing if the model is working for my use case, I plan to later on run on a cloud compute server called computeCanada, which I'm sure I'll run into problems with as well later.
How do I install deepspeed excluding triton? I am getting this error when trying to install deepspeed. git+git://github.com/EleutherAI/DeeperSpeed.git@c45ec1c0ac05803c02763d7e43be67919e475a2c#egg=deepspeed

Windows is a nightmare and you shouldn’t be trying to install things on it to test.


Pseudo effect will miss after restarting AE

An issue with pesudo effect created by Flex Effector

can't reappear this bug. Closed the issue


Minor JS fixes

After catching a leaked global var in marked.js, I ran Deepscan on the codebase and found these similar instances (and other minor things) to fix.
Full list of other errors/warnings is here

Merged, thanks @aero31aero!  Several of the other things listed there in our own code seem worth fixing as well (e.g. the extra arguments to parseFloat).