v0.4.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Obsoletes #4

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

For linux, failed at fluidsim-test -v during test_curl and core dumped. Not sure why.
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=266000&view=logs&j=4f922444-fdfe-5dcf-b824-02f86439ef14&t=b2a8456a-fb11-5506-ca32-5ccd32538dc0&l=2132
For osx. something to do with mpi during test_bench:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=266000&view=logs&j=4b3757f9-99c5-5891-6c0c-93d7030cdf7d&t=f047df50-a233-5ae9-bf8f-c7dd8e5684ad&l=2778

For linux, failed at fluidsim-test -v during test_curl and core dumped. Not sure why.
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=266000&view=logs&j=4f922444-fdfe-5dcf-b824-02f86439ef14&t=b2a8456a-fb11-5506-ca32-5ccd32538dc0&l=2132
For osx. something to do with mpi during test_bench:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=266000&view=logs&j=4b3757f9-99c5-5891-6c0c-93d7030cdf7d&t=f047df50-a233-5ae9-bf8f-c7dd8e5684ad&l=2778

For linux, failed at fluidsim-test -v during test_curl and core dumped. Not sure why.

"Illegal instruction"
It means that something has been built in conda with arch=native (or something like that).

For linux, failed at fluidsim-test -v during test_curl and core dumped. Not sure why.

"Illegal instruction"
It means that something has been built in conda with arch=native (or something like that).

For osx. something to do with mpi during test_bench:

The second one is something really simple (just a fluidsim bug when cpu_count is 1).

For osx. something to do with mpi during test_bench:

The second one is something really simple (just a fluidsim bug when cpu_count is 1).

Next release it should work on OSX: https://foss.heptapod.net/fluiddyn/fluidsim/-/merge_requests/193

Next release it should work on OSX: https://foss.heptapod.net/fluiddyn/fluidsim/-/merge_requests/193

For the first problem (Illegal instruction), in fluidsim (https://foss.heptapod.net/fluiddyn/fluidsim/-/blob/branch/default/setup.py#L139) and fluidfft (https://foss.heptapod.net/fluiddyn/fluidfft/-/blob/branch/default/setup_make.py#L501).
I guess the simpler solution would be to set CARCH to something more reasonable for package builds (conda and wheels, but we don't upload wheels so only conda). I don't know what would be a reasonable value for CARCH on conda. We need to be compatible with simple CPU.

For the first problem (Illegal instruction), in fluidsim (https://foss.heptapod.net/fluiddyn/fluidsim/-/blob/branch/default/setup.py#L139) and fluidfft (https://foss.heptapod.net/fluiddyn/fluidfft/-/blob/branch/default/setup_make.py#L501).
I guess the simpler solution would be to set CARCH to something more reasonable for package builds (conda and wheels, but we don't upload wheels so only conda). I don't know what would be a reasonable value for CARCH on conda. We need to be compatible with simple CPU.

We should try with CARCH=x86-64.

We should try with CARCH=x86-64.

Since we have problem on all platforms and we won't be able to run fluidsim 0.4.0 tests on OSX, I think we can disable builds on Windows and OSX for this version.
Once fluidfft will be built without march=native on conda, we should be able to at least build the packages for Linux in this PR.

Since we have problem on all platforms and we won't be able to run fluidsim 0.4.0 tests on OSX, I think we can disable builds on Windows and OSX for this version.
Once fluidfft will be built without march=native on conda, we should be able to at least build the packages for Linux in this PR.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


New UI for Podcasts page

Shrink right bottom column size a bit, then expand right side and add various info to right column.
Left column will also include tabs for "Reviews", etc. Top section should allow for star ratings as well.

Is this better than current split view? Somehow Im not convinced. But I think with enough metadata, the design above represents more opportunity to surface that metadata (obviously more surface area)


ChromeDriver fails to load on v0.0.10

With the 0.0.10 update, chromeDriver no longer seems to load correctly, returning the following error:
 The driver executable does not exist: `/PATH/TO/PROJECT/node_modules/selenium-standalone/.selenium/chromedriver/2.33-x64-chromedriver

It seems to be defaulting to chromeDriver version 2.33 even though the default settings specify 2.36.
I'm running selenium 3.6.0, wdio 4.12.0.

I was facing the same issue. There might be a process which is running the 2.33 version of chromedriver. Kill that process and everything should work like normal.


New Website Logo: [AGUS WIDIYANTO]

To list your organization's logo on the Rust website, fill out the following information and click "submit new issue". Alternately, you may edit _data/users.yml as described therein and submit a pull request.

Organization name: (as you want it displayed)
Homepage url: (homepage/primary entry point for users)
Logo url: (svg if possible, pngs over 400x200px with transparent backgrounds are also acceptable)
How you are using Rust: (one sentence describing your use of Rust)
Url describing Rust usage: (optional link to e.g. blog post explaining how you use Rust)
Organization contact: (name and email. we may contact you when updating this page. alternately you may email this information to<EMAIL_ADDRESS>and it will be kept secret).

process operation

Belajar

best

<0/2>github to share[]


Merging collections and the /tpf endpoint

Collections and the /tpf endpoint have some similarities. Both allow for filtering all triples by property / value. However, collections are more powerful, as they also enable pagination and sorting.
The /tpf endpoint currently uses a lot of custom logic, which might not be needed anymore if we switch to using collections.
Also, the subject field in TPF is not really useful in Atomic data, except when you want to find an Atom by subject and value.

Perhaps use the standardized query params / Atomic Endpoints for this: https://github.com/ontola/atomic-data/issues/15

Another important difference between TPF endpoints and Collections is that Collections return Subjects, while TPF queries typically return Atoms (triples).

Done!


RPO threshold

"In RPO threshold, specify the RPO limit. Alerts are generated when continuous replication exceeds this limit."
Is the "RPO Threshold" setting only a threshold to trigger for alerts, but not the actual frequency of snapshots being taken?
It was mentioned that replication for VMware VMs is continuous. So, what is the lowest possible RPO?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bdeaac7a-e605-600d-cccf-b72e329f7845
Version Independent ID: 92b5c7a4-96df-9668-6dad-227cd0032d52
Content: Set up replication policies for VMware disaster recovery with Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/vmware-azure-set-up-replication.md
Service: site-recovery
GitHub Login: @sujayt
Microsoft Alias: sutalasi

@hugohhk Thanks for your comment! We will review and provide an update as appropriate.

@hugohhk Replication is continuous but snapshots are taken every 5 mins. The lowest possible RPO would be a few seconds but please note that this is not guaranteed and that we do not provide any SLA for RPO.
Regarding alerts, if the RPO threshold is set to 60 minutes in replication policy, this has nothing to do with the regular App or crash consistent snapshots to be taken by ASR. This basically says that if the replication of the instance breaks, and Azure does not receive any replicated data for this instance for 60 minutes, or no recovery points are created in Azure for those 60 minutes, Azure will mark that instance as critical and will trigger alert or Emails, if configured.

Hi,
Thanks for your answer.
So can I assume that for VMware-to-Azure ASR, crash-consistent snapshots are also taken every 5 minutes?
Because this wasn't mentioned in the VMware-to-Azure ASR FAQ. It was only mentioned for Azure-VM ASR FAQ.
And if crash-consistent snapshots are taken every 5 mins, then why the lowest RPO is not 5 mins, but could be as low as a few seconds?
In my understanding, those snapshots are used to create new managed disk and a new VM in Azure if failover occurs.
Please correct my misunderstanding. Thanks.

@SadiqhAhmed-MSFT
Any further feedback to my reply?

@hugohhk Yes, we generate crash-consistent recovery points every 5 mins in the V2A scenario as well. And you are right – the RPO could be a few seconds as well.

@SadiqhAhmed-MSFT
I don't understand why the RPO can be a few seconds given that the snapshot is taken at 5 min interval.  The new managed disk is to be created from those snapshots taken at a 5-min interval, so shouldn't the RPO be 5 mins?

@sideeksh For help

Let's consider 2 cases to make this clear:
Case 1: Snapshot was taken at 10:00:00am and failover was initiated at 10:04:21am. When the failover was initiated, the latest processed recovery point was chosen (the 10:00am one). Therefore, the total data loss would be 4min 21sec and this would be your RPO.
Case 2: Snapshot was taken at 10:00:00am and failover was initiated at 10:00:05am with the latest processed recovery point. In this case, the data loss/RPO would be 5 seconds.
In stray cases, the recovery point may not be generated every 5 mins, which explains why the RPO exceeds 5 mins in some cases.
Hope this helps.

@sideeksh
Thank you so much for your clear explanation.
Do you have any documentation mentioning for VMware-to-Azure scenario the crash-consistent snapshot is taken at 5 min interval?
I cannot find this piece of info. I can only find it for Azure-to-Azure scenario.
Thanks

I can confirm it is taken at 5 min intervals in the case of VMware to Azure as well. We will update the documentation with this information.

Thanks a lot! @sideeksh

@hugohhk Closing this issue for now. Feel free to reply if you have additional questions in this matter and we would be glad to continue the discussion.
@sideeksh Thanks for chiming in for help and providing detail information. Appreciate your time and help.


feat: add FlutterRename command

This command adds support for updateImportsOnRename setting (https://github.com/dart-lang/sdk/blob/master/pkg/analysis_server/tool/lsp_spec/README.md#client-workspace-configuration).
It is useful when the file is renamed after a class rename (when enabled renameFilesWithClasses setting).
It will fall back to vim.lsp.buf.rename() if no dartls client is found.
If the client is found, the file is the same as the class name, and the class is being renamed,  it will request for changes via workspace/willRenameFiles. Results of workspace/willRenameFiles will be applied after the class rename.
To test it try to rename the class that is in the file with the same file name.

with vim.lsp.buf.rename(): class renamed, file renamed, usage of class renamed, but imports are not updated.
with FlutterRename: class renamed, file renamed, usage and imports are updated.

will fix #233


Jesse: Watch fixityedx.com

Jesse requests the watch of the watch_keyword fixityedx\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fixityedx\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


[Issue 6043] Support force deleting subscription

Motivation
Fixes #6043
Modifications
Add method deleteForcefully to support force deleting subscription.

@codelipenghui yes, I hava already opened an issue (#6394) to track this, I am planning to fix it by a seperated pull request.

@codelipenghui please help to review again, thanks.

@codelipenghui @sijie @jiazhai Since #6394 is already fixed, I hava refined the unit test. Please help to review it again, thanks.

@murong00 There are some problems related to the async method in PersistentTopicsBase.java. I have sent a PR #6483 to fix them, please also help take a look.

@codelipenghui ok, I will refine this pr after #6483 is merged.

@codelipenghui I hava addressed your comments, please help to take a look.

@sijie Please help to review this again, thanks.


Update connection to external subscriber to support encrypted communication(TLS)

Currently, hypersdk only supports plaintext communication over grpc connection. We need to support TLS

This is now supported


Delay in Script Execution Due to Initial Schedule Interval

Issue Summary
There is a delay in the initial execution of the WhatsApp Watchdog script due to how the scheduling is currently set up.
Description
The current implementation uses schedule.scheduleJob(config.schedule, () => {...}) to set up recurring tasks. However, this approach leads to an initial delay equal to the specified schedule interval before the first execution of the script. For instance, if the schedule is set to run every 1 hour ( '1 * * * *'), the script won't start monitoring until an hour has passed since the script was launched.
Proposed Solution
A potential solution could be to modify the scheduling logic to trigger an immediate first run, followed by scheduled runs as per the specified interval.

Resolved in d4d741281f3fc7350800d059517718239a5cdf41.


Add devops recipe: github actions

add separate docker build and prefect deployment build workflows and associated dockerfile in a new dockerfiles folder
Description

minimal-prefect-deployment-build.yaml: run prefect deployment build if our flow file changes
docker-build-push-gcp-artifact-registry.yaml: build and push a docker image for our flow to Google Artifact Registry if our Dockerfile or dependencies change

Type of change

[x] This change requires a documentation update

New Recipe Checklist

[x] My PR is in the format of Add <project-name> recipe
[x] My recipe is reproducible and explains everything needed to run successfully. If my code has external dependencies, I make mention of them.
[x] My code is easily understandable and/or well-commented.
[x] If my recipe requires a new category (e.g. creating a monitoring/ folder in devops/), I have encapsulated my project within its own subfolder so that others can add their own recipes as well.

Looks great! I only commented on one of the workflows but I think the same comments could apply to both.


style: clean up Dockerfile

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


Fix datalake regression failure

Currently, datalake is use a cls method that's fetching the http_response attribute which did not exist in previous blob versions. This is now handled by a simple try, except.

/azp run python - storage - tests - ci

/azp run python - storage - tests - ci

/azp run python - storage - ci

/azp run python - storage - ci


Ability to highlight selected line(s) in source code

It would be useful to provide a way to highlight selected line(s) in source code to pay audience attention on specific aspect(s).

Has this feature been disabled now? The demos are not working. How do I bring it back?

Ok after digging through the code a bit I found that this has to be enabled manually using
      var slideshow = remark.create({
        highlightLines: true
      });

then it works. Leaving it here for anyone else who gets stuck. Thanks!


Input dropping characters. Performance issue?

I'm experiencing an issue quickly typing in a controlled input.
Here's a quick gif of what the problem is. (Sorry, if its not super clear -- I set the framerate to max).

Essentially, I was typing 12345 very quickly and some of the characters were getting clobbered.
If you look in the console, you can see the payloads for each keyPress:

Here's a fork of the demo application demonstrating the issue: https://github.com/seanders/react-chrome-redux-examples
Is this a real bug or simply a poor implementation on my part?

@seanders Thanks for creating an issue! This is interesting... first time I've seen something like this, myself. I will need to dive a bit deeper to figure out the root cause...

@seanders https://www.useloom.com/share/f7f9bd4e65f34aff800f1b4eb3ac6206

@this-miguel 🎉

@tshaddix should we keep open until @seanders gets back to us?

@vhmth @seanders, based in the video from @vhmth  what I did to solve my problem was:

rely on the local state of the component for rendering.
initialize the component state in the contructor with the related props.
use getDerivedStateFromProps to decide when to update the state of the component based  in comparison of the old state agains the new props.
do not dispatch actions in onChange function assigned to the input text, since this is what causes the race conditions, instead update the local state.
only dispatch actions related to the value of the input text sparingly (for me for example, it was acceptable to dispatch  the related actions only when the input text goes out of focus onBlur was ok, or when the input text was cleared, I was using a button to do that anyways)

This way race conditions can be avoided, and you can continue to use the value of the input text to render your components without dispatching too many actions.

Just to close the loop on this.
@vhmth Your explanation was exactly what I thought was happening (async message passing from content to background resulting in props clobbering the value in the input). I already went ahead with a local state solution (what you called approach #1).
@tshaddix All good with the close. There is no bug in the project--its just a trade-off of having this approach.
Thanks for you help & replies.
PS. I love this project. We had a home-rolled implementation with the same state-management-paradigm and were thrilled when we discovered this project!

@seanders Glad to hear it! Also glad to hear this project is working well for you! If you feel comfortable, would love to hear what you're working on

@seanders very cool!

@seanders ditto to @vhmth ! If you're down, we'd love to throw your logo on the projects section of the Readme 🥇


fix UI for flakes & nonblocking suites

Tired of the UI being totally wrong and confusing. Turns out to be easy.

That yellow/gray contrast is pretty poor, could you use orange instead?

done

This is an improvement and I have to restart anyway.


Custom session saving policy

Added save_session config and --save-session option to control whether sseions are saved without asking when exiting the session.
aichat -s session01 --save-session
aichat -s session02 --save-session 1 to 5, odd only
aichat -s session02 --save-session to 7

Deprecated solution

The current session saving policy:

On repl mode, ask the user whether to save the session when exiting the session.
On commmand mode, don't save the session.

This plicy was reasonable in the past, because tokens were expensive, and as the number of messages increased, the cost of messages would increase rapidly, and users tended to discard the change to save costs.
Recently, the token has become cheaper and aichat has session compress (#334), so the session cost is controllable.
It is necessary to add a new saving policy, making it more convenient to use the session.
Request to add two configuration items to control session saving policy.
save-session-on-repl: ask        # Set session saving policy on repl mode: ask, onexit, onchat
save-session-on-cmd: true        # Whether to save session automatically on command mode

For Repl Mode:

save-session-on-repl: ask: The default policy. Ask the user to confirm whether to save the session when exiting the session.
save-session-on-repl: onexit: Save the session automaticlly when exiting the session.
save-session-on-repl: onchat: Save the session automaticlly while chatting.

For Commmand Mode:

save-session-on-cmd: false: Don't save the session. If the session does not exist, throw an error.
save-session-on-cmd: true: Automatically save the session. If the session doesn't exist, create a new one.

It would be nice to disable auto-session saving as discussed. Thanks for projecting this.
However, I've found that most of the time we don't save the session because the data isn't valuable enough, except in certain cases. And when it happens you known you want to save the session. So in this case it would be nice to be able to explicitly save the session under the given name like .save my-session-name.
Does it make sense ?

@dtrckd  Why need a new repl command?
If you have not thought about the purpose of the session, just use aichat -s or .session to open a  new temp session. When you exit the session, you will be prompted to name it.

In the case where the default behavior is to not save the session on exit, so neither asking nor saving (which would be the option I'd use, if available).
The user should still have the possibility to save an interesting session, with a .save [session-name].
I believe it should behave like :w [file-name] in vim :

if no session-name is given, it saves it to the current session name (or ask for name if it is a new one).
else it saves it to the given session-name given.

#370 add save_session config item to control whether sessions are saved automaticcly without asking when exiting the session.
#377 add seperate save_session config item to session. You can enable save_session for some sessions only.
I don't feel the need to add an extra .save command.

all done.


Update deploy-a-dm-cluster-using-tiup.md

What is changed, added or deleted? (Required)

Which DM version(s) do your changes apply to? (Required)

[ ] master (the latest development version, including v3.0 changes)
[ ] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Have version specific changes
[ ] Might cause conflicts after cherry-picked

 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.

Please help manually cherry-pick this PR to master branch. @CharLotteiu


Issue with graphql-import

import FriendsRequestsSubscriptionPayload from “./prisma.graphql”
I am trying to import a type from the generated prisma file, but I get this error:
Error: Type “NotificationOrderByInput” not found in document.
And If I do try to import it I get the same error, though I shouldn’t have to as graphql import is supposed to recursively bring in what’s needed.

Hi.
Please provide this bug report filling out the bug report template.
Thanks.

Moving the conversation here: https://github.com/prismagraphql/prisma/issues/2639


npm run build: Cannot find module '/home/username/destreamer/node_modules/typescript/bin/tsc

Env
Ubuntu 16.04 on WSL Windows 10 1909
git version 2.7.4
node v14.0.0
npm 6.14.4
Steps
git clone https://github.com/snobu/destreamer
cd destreamer
npm install
npm run build

ER: success
AR: error
><EMAIL_ADDRESS>build /home/username/destreamer
> echo Transpiling TypeScript to JavaScript... & node node_modules/typescript/bin/tsc --listEmittedFiles

Transpiling TypeScript to JavaScript...
internal/modules/cjs/loader.js:1017
  throw err;
  ^

Error: Cannot find module '/home/username/destreamer/node_modules/typescript/bin/tsc'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:1014:15)
    at Function.Module._load (internal/modules/cjs/loader.js:884:27)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `echo Transpiling TypeScript to JavaScript... & node node_modules/typescript/bin/tsc --listEmittedFiles`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2020-04-24T07_10_52_463Z-debug.log

Log attached.
2020-04-24T07_10_52_463Z-debug.log

It won't work in WSL, puppeteer can't pop up a browser window. Run it with Windows Node.js in cmd.exe or PowerShell.

Did I miss this in readme? If not, please consider documenting it.

You didn't, we left it out by accident. I'll add a note for it right now. Thanks for reporting this one.

Quick update here, i've tested this on WSL2 and it does work, however you'll need an Xserver so puppeteer can launch the Linux-flavored Chromium that comes down with npm install.
# For WSL1
# Start Xming on Windows
export DISPLAY=:0
./destreamer.sh -i URL_TO_VIDEO

# For WSL2
# Start Xming on Windows
export DISPLAY=$(grep nameserver /etc/resolv.conf | cut -f2 -d ' '):0
./destreamer.sh -i URL_TO_VIDEO

Xming -
https://sourceforge.net/projects/xming/

Just confirming that @snobu 's comment about using your Windows version of Chrome (below) does work.
const browser = await puppeteer.launch({
        executablePath: "/mnt/c/path/to/your/chrome.exe"
        headless: false,
        args: ['--disable-dev-shm-usage']
    });

The XServer solution was not working for me, as the Chromium window was not rendering any content.
The only caveat is that you need to manually close the Chrome window after a successful login, as the script will fail to do it.

Hi, I'm trying to run destreamer on wsl2.
I changed the executablePath to my windows chrome.exe, but it is not rendering any content.
In console I get this error:
[WARNING] .token_cache not found.
Launching headless Chrome to perform the OpenID Connect dance...
[ERROR] [object Object]
Do you know what could be the problem?

Hi, I'm trying to run destreamer on wsl2.
I changed the executablePath to my windows chrome.exe, but it is not rendering any content.
In console I get this error:
[WARNING] .token_cache not found.
Launching headless Chrome to perform the OpenID Connect dance...
[ERROR] [object Object]
Do you know what could be the problem?

I have the same problem. I can't seem to make it work even with xming

Just confirming that @snobu 's comment about using your Windows version of Chrome (below) does work.
const browser = await puppeteer.launch({
        executablePath: "/mnt/c/path/to/your/chrome.exe"
        headless: false,
        args: ['--disable-dev-shm-usage']
    });

The XServer solution was not working for me, as the Chromium window was not rendering any content.
The only caveat is that you need to manually close the Chrome window after a successful login, as the script will fail to do it.

I think you need to make a double slash
Like this
        executablePath: "//mnt//c//path//to//your//chrome.exe"
        headless: false,
        args: ['--disable-dev-shm-usage']
    });

I ended up using this tutorial on yt, works as a charm https://www.youtube.com/watch?v=9Fx7UCsSG9g

I managed to get it working with WSL2 and GWSL (from the Microsoft Store). I had to install these dependencies beforehand:
sudo apt install -y libatk-bridge2.0-dev libatk1.0-dev libnss3-dev libcups2-dev libgtk-3-dev

I'm using an Ubuntu 20.04 distro from the store. After installing GWSL so that the X server is configured, as well as the dependencies above that are required to run GTK apps, it's all working smoothly.
I can make a PR that runs these steps / suggests GWSL if WSL2 environments are detected.
Thanks for the tool, guys! It's really useful!


IOS scan page without flashlight

IOS scan page without flashlight. Can you support it?thanks

Already support custom view, please use v1.2


Add a note into the docstring of sensor types that special characters should be avoided.

Special characters are causing problems. We don't have time to solve the exact problem, but a note in the docstring would be helpful.

I'm still unclear on whether this is an inherent issue with Unicode characters in the fields like sensor_unique_id, like maybe PostgreSQL mangles them or something, or if the problem is just that some Unicode characters look very much like each other, which does cause trouble for users, but I think isn't anything we can really solve.


Optimize instances when multiple Applications running on same machine

Description
I am using pupeeterr-sharp in my .net core application . Everything is running fine except I have to optimize the system resources . I have following two issues and I am looking for a way to resolve them .

Multiple chromium processes

Allow multiple applications to access same instance of browser

When I run my application and check my processes it display 4-5 processes of chromium for single application. Note that I am launching browser just once in my application also I am just opening a single page . I have tried to use "--single-process", "--no-zygote" flags in browser options but it gives  Failed to launch Base! [19096:1240:0930/140433.373:FATAL:content_main_runner_impl.cc(528)] --single-process is not supported in chrome multiple dll browser. Error .
Secondly I want to run different independent applications. So, I don not want every application downloads its own copy of browser and run . I want to download chromium once on my machine in some global location and I want all my applications to use that instance and open different tabs instead of launching browser multiple time
       try
               {
                LoggerUtility.LogMessage("Initializing Browser");
                WaitUntilNavigation[] waitUntil = { WaitUntilNavigation.Load, WaitUntilNavigation.Networkidle0 };
                await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
                // Checking If browser already running
                if (browser == null || !browser.IsConnected)
                {
                    LoggerUtility.LogMessage("Launching Browser");
                    browser = await Puppeteer.LaunchAsync(getLaunchOptions());
                    browser.Disconnected += OnBrowserDisconnect;
                }
                while (!isPageLoaded && config.Retry > 0)
                {
                    page = (await browser.PagesAsync())[0];
                    await page.ExposeFunctionAsync<string, string, bool>("onCustomEvent", OnCustomEvent);

                    LoggerUtility.LogMessage("Loading Url", config.BaseUrl);
                    try
                    {
                        var response = await page.GoToAsync(config.BaseUrl, new NavigationOptions
                        {
                            Timeout = 0,
                            WaitUntil = waitUntil
                        });
                        if (response.Status == System.Net.HttpStatusCode.OK)
                        {
                            isPageLoaded = true;
                            LoggerUtility.LogMessage("Page Loaded succesfully");
                            //await page.EvaluateExpressionAsync(" $('.event__match').bind('DOMSubtreeModified', function(e) { window.onCustomEvent(e.currentTarget)})");
                            await page.EvaluateExpressionAsync(" $('.event__match').bind('DOMSubtreeModified', function(e) { console.log('live event occured'); window.onCustomEvent($(e.currentTarget).prop('outerHTML'),$(e.currentTarget).attr('id'))})");
                            string html = await page.GetContentAsync();
                            extractLiveTable(html);
                        }
                        else
                        {
                            LoggerUtility.LogMessage("Error in loading page", response.Status.ToString(), response.StatusText);
                            config.Retry--;
                            Thread.Sleep(500);
                        }
                    }
                    catch (Exception ex)
                    {
                        LoggerUtility.Error(ex);
                        config.Retry--;
                        Thread.Sleep(500);
                    }
                }
            }
            catch (Exception ex)
            {
                LoggerUtility.Error(ex);
                throw ex;
            }

Expected behavior:
I am expecting to run single instance of browser for multiple application with minimum processes in background
Actual behavior:
Every application is launching it own instance of browser with multiple processes in background
Versions

puppeteer-sharp 2.0.4
.NET Core 3.1.

Additional Information

Have you tried PuppeteerSharp.Puppeteer.ConnectAsync()?
https://github.com/hardkoded/puppeteer-sharp#connect-to-a-remote-browser

Closed due to inactivity. Feel free to reopen it if needed.


Fix Money\Currency Json serialization

Currency JSON serialization should return an indexed array, as it should be treated as an object in JSON.
Therefore, when serializing a Money object, the result should be like
"moneyInstance": {
    "amount": "1000",
    "currency": {
        "code": "EUR"
    }
}

Instead of
"moneyInstance": {
    "amount": "1000",
    "currency":  "EUR"
}

Currency JSON serialization should return an indexed array, as it should be treated as an object in JSON.

I don't think that's necessarily true. The format is arbitrary, nothing says the currency has to be an object. Additionally, this is a breaking change.

@sagikazarmark It's true that it is a breaking change, and that shouldn't be an impediment to try to improve the code.
And following the current Entity structure, the default Object formatting should respect the JSON serialization. Money::$currency is a Currency object, and should be reflected as so in JSON, otherwise serialization/deserialization need special serializers/deserializers to manipulate the data. Another thing is that it's intended to apply such change

Unnecessary breaking change. Closing.


New quote for Inspiring by Jackie French Koller

I'm going to submit another PR with another quote I like. These are obviously very subjective.
https://twitter.com/taylorotwell/status/578392409853169664

This one is a hair too long :)

No worries, that's why I made them separate PR's. I'll keep my eyes open for more!


Pressing "Save personal bindings" does not work for a built project.

SteamVR beta 1.8.5
Unity 2019.1.6f1.
SteamVR Unity Plugin v2.3.2.
Inside Unity, when opening the binding UI I am able to edit and save my personal bindings. When I build the project as a standalone build and try to edit then save my personal bindings the UI stalls. I have to restart SteamVR to be able to edit any application further.

I have found a temporary solution to this so for anyone interested this is how to bypass this bug for now.
Tested on SteamVR 1.7.15.
Tested on various Unity and Unreal projects that are launched from the local harddrive (meaning they are not distributed on Steam).
When one tries to edit any legacy application, any changes made in the Devices>Controller Settings are immediately saved in a file located in my documents folder. The naming of the file is based on what controller type you choose. However, the naming has nothing to do with the application you are trying to edit. So, in my case, if I try to edit the controller settings for the vive controller for an app called "TestApp" the software will create the following file:
C:\Users\pt\Documents\steamvr\input\_vive_controller.json
If I edit the app for an index controller the software will create the following file:
C:\Users\pt\Documents\steamvr\input\_knuckles.json
If I edit the app for an omnideck controller the software will create the following file:
C:\Users\pt\Documents\steamvr\input\_omnideck.json
Looking in the same folder, one can see various locally saved bindings file with the naming scheme:
C:\Users\pt\Documents\steamvr\input\steam.app.ANUMBER_ACONTROLLER.json
This must clearly be a bug in SteamVR.
Now, for my case, starting with 1.7.15 - which introduced an ability to return the bindings  with lef/right hand in the extra settings - to make the app in question work with the Omnideck I copy the last options section from the file
C:\Users\pt\Documents\steamvr\input\_omnideck.json
   "options" : {
      "returnBindingsWithLeftHand" : true,
   }

and place it as the default in the legacy_bindings_omnideck.json file for my driver.
If this is already placed in the bindings file all is fine when you start a new application using the legacy input system. However, if you try to edit the legacy application in question you will end up with a stalling "Uploading..." notificiation in the UI and SteamVR will create the file in your local documents folder as stated above.

Yes, we've experienced the same issue on SteamVR 1.7
For testing purposes, we bypassed it in a brutal way: Clicking "Replace default bindings" worked for us. Of course, it affects the other default bindings of your controller as well.

Happy to be of help,
keep hacking.
/P

This has been fixed by valve as of SteamVR beta 1.8.13.


Minor issue: 'multiple-bibliographies' filter still throws one error per citation

Coming from the discussion over at pandoc discuss, I'd like to record that erroneous notifications such as
pandoc-citeproc: reference XY not found
have been reduced by the uploaded multiple-bibliographies filter, but not yet eradicated completely: There's still one error per citation.
While this is a minor issue - after all the functionality is not affected -, in a document with hundreds of citations, this may still lead to oneself missing warranted error notifications in case a cited work is really not included in the bibliography.
Don't want to be pedantic, but I thought noting the issue here couldn't hurt. Again, thanks for uploading the filter in the first place!

Thanks for the reminder. Warnings should disappear now if the pandoc-citeproc version is 0.14.5 or later.


support tail functionality (watch file-changes)

we need to optionally support for watching changes in a file similar to
tail -f /tmp/bla.txt

Work around - use "Shell" tab and call tail some_file_name.log


Can't get off my rocket when landed and everything is ¨frozen¨ around me.

the tittle explains it all

So... What did i do wrong? Do you need a log too?

is this in single player or multiplayer, are there any other mods installed?  And a log might be helpful.  It sounds just like the server or integrated server may be overloaded though.

This happens to me every time I go to a planet with a rocket. If the landing site is not chunk loaded, the above happens. If you save and quit to the main menu and reload again, you show up inside the rocket on the planet surface.
The logs don't show any error or other message when this happens.
This is on single player, MC 1.10.2 and the integrated server doesn't show any signs of being overloaded.

Its single player and i cant acces my log on my phone

I've only just started with this mod and my first rocket launched fine, but after transitioning to space and about to land on the moon I get the exact same issue. I then launch back to earth (same glitchy/jerky behavior) and when landed, it kills me.
I'm running 1.10.2 with 80 mods, my log DOES however mention the following on each rocket trip:

[Server thread/WARN]: Can't keep up! Did the system time change, or is the server overloaded? Running 3984ms behind, skipping 79 tick(s)

TBH not really encountered that before, what would be the solution?

@jrunestone 3984ms is nearly 4 seconds between ticks, so it definatly looks like a lag issue.  You may want to consider running a tool like visualVM to see what' taking up most of the tick time.  Unfortunatly there's not much I can really say other than that without more info as to what is actually eating all that time.
@azmicomico Is there something in particular you can do to make it occur?  I've not been able to replicate on my end.  Does this occur with only advanced rocketry and libvulpes installed?
@MajorLOLGaming You can still have lag in singleplayer, usually occurs when there's too much going on in the world, and If you can send the log when you get a chance that may help.  And like i said with azmicomico, does it continue to occur without mods?

@zmaster587 Thanks, yeah after researching a bit more I think this is just a result of too much crap lying around... I'm going to try without BoP and maybe bump up the memory. As far as I'm concerned this issue isn't really about advanced rocketry on its own.

It happens randomly with a high chance of failure. I've not found a sure way to always happen.
I cannot replicate with just Advanced Rocketry and libVulpes installed. I guess I'll continue saving and quitting when it happens.

First be patient when generating new dimension.
I for exampl have a SkyLands dimension, which is almost entirely generated on entry and it takes Advanced Rocketry good 2 minutes and ful power of all 8 cores before the dimension is generated and the rocket descents to the surface.
So, just don't haste, sit ight and wait. The rocket Should start descending eventually.

@azmicomico you might be able to make some headway with visualvm, if you run a CPU profile on the world as it's frozen for a minute or two then create a snapshot.  If you browse the server thread and use the hotspot tab, you'll likely find out what's going on I think.

@gdarai and @zmaster587 The world/game is not frozen when it happens and the dimension is already generated. See the original poster screenshot. You can see the terrain below, but there's no rocket. CPU usage doesn't increase at this time. The message to descend appears and it starts descending if you press space. The altitude changes in the altimeter, but the Y level doesn't and when it lands, the message changes to 'take off'. Pressing left shift to dismount at that time does nothing.

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.

I don't think this is an issue anymore
On Mon, 4 Mar 2019, 00.44 Dmodoom Sirius<EMAIL_ADDRESS>wrote:

Is this still a issue? Please comment if it is and i will reopen if if you
are unable? We are closing old issues to see what is still relevant and
needs to be fixed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/872#issuecomment-469078175,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ae9E_-QPocHWDk4FIaFhe_c0Jv3WwVijks5vTF5NgaJpZM4PylVG
.


RFE: placeholder for hashed ip

I'd like to have a new placeholder returning a hash of the IP address.
Rationale:

i do not want to see IP addresses in my logs, especially as i sometimes want to keep them for a long time.
masking the IP addresses, in way compatible with the privacy laws, makes them unusable for debugging in some cases (i do have a number of users who end up in the same /24 every few days).

a hash of the IP address, even a crc32, would satisfy privacy, and be unique enough for debugging.
It might be done this way in caddyhttp/httpserver/replacer.go:
`29a30

  "hash/crc32"

339a341,346

  case "{remotehash}":
          host, _, err := net.SplitHostPort(r.request.RemoteAddr)
          if err != nil {
                  host = r.request.RemoteAddr
          }
          return strconv.Itoa(int(crc32.Checksum([]byte(host), crc32.IEEETable)))

`
Is there an easy way to see whether the connection uses IPv4 or v6?
If yes, it might be useful to add another placeholder, as this information is lost with hashing.

I don't think hashing an IP address really offers any more privacy than just storing the IP address.
Especially IPv4 addresses: those can be enumerated in a matter of minutes. Why is hashing them more private, exactly? (Here's a recent, real example: https://twitter.com/einaros/status/951437795528265728)
(The same goes for IPv6, as the subset of allocated IPv6 addresses is extremely small compared to the size of the whole space.)


Elitebook 8470p Fans are not found and no wifi

I have an elitebook 8470p. The installation for Monterey 12 worked like a breeze. but when finished mac fans control says there are no fans on the system, (of course they are there) but they are spinning way out of control incredibly noisy. Anything I can do? I selected the Smooth option when installing your package but it made no difference. Included my Efi file to see if there is any solution? thanks beforehand.
I also bought the BCM943224HMS wifi card you recommend and put the brcmfx-delay=15000 bootflag but it still doesnt work. The card was tested on windows first and it does works.
p.s. I took out the resources folder withing the OC folder because its not relevant to this issue and github wouldnt let me upload the compressed efi folder because it was more than 25mb.
EFI.zip

Mac Fan Control not working in Hackintosh
You have HWMonitor.app in your Applications, lets the sytem quiet a bit then open HWMonitor.app
If the Wifi card not work is probably a bad card from China

well it does works, I tested it in Windows. So theres is no workaround to make it work? the wifi card?

ok thanks


Graphback cli  and mongo requires dependency cleanup

warning "graphback-cli ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.9.0 || ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "graphback-cli ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^15.0.0".
warning "graphback-cli ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "graphback-cli ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^15.0.0".
warning "graphback-cli > @graphql-toolkit/code-file-loader ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.9.0 || ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "graphback-cli > graphback ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^15.0.0".

Multiple warnings like this appear since update
warning "workspace-aggregator-12461c52-515e-461a-8376-abfb2c639e8c > datasync-server-starter ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

I believe many of these could be false negatives because we don't actually install graphql as a dependency to the graphback-cli package, but it is instead acquired as a peer dependency in the template.
Since graphback-cli does not have it as a dependency, these warnings are to be expected. We could actually add graphql as a devDependency to the root package.json (which will never be published anywhere anyway) - this should remove a lot of these warnings.

Perfect response from project lead


Behavior differs from default BLE library during BLE scan when passive mode is set

I had an issue earlier where a subset of BLE iBeacon devices that were showing up in the original ESP32 were not being picked up by my callback when using NimBLE.
From digging through the code, it appears that behavior differs between the two libraries when BLEScan->setActiveScan() is set true thereby disabling passive mode. In the default library the callback still gets triggered regardless of the state of passive mode, whereas in NimBLE the callback does not get triggered when passive mode is set.
This seems like this behavior changed in NimBLE-Arduino after some of the initial commits - if this behavior change was intentional and expected to remain part of the library, it might be helpful to document the change somewhere.
Passive check in the NimBLE Library: https://github.com/h2zero/NimBLE-Arduino/blob/15448515d4bda4a042929f7b6546917d8df6e75a/src/NimBLEScan.cpp#L144
Line /w behavior in the Non-NimBLE Library: https://github.com/espressif/arduino-esp32/blob/1efcd21ba91c08e702a0c6e47e00a2d0a8c06b84/libraries/BLE/src/BLEScan.cpp#L127

Yes I was never very happy with this code, but seemed necessary at the time because NimBLE will trigger the callback for the advertisement and the scan response, so the callback would be called twice when active scanning. This was a workaround to wait until all the data was available before calling the callback.

Thinking about this - of the three devices I'm trying to pick up, the two that aren't being picked up with this filter are the two that have advType of Non-Connectable - No Scan Response which I think aligns with what is happening.
That said - would it make sense to check the advType and return if it is Non-Connectable if the scan is active?
Advertised iBeacon Device: Name: , Address: ca:62:7a:c9:f9:da, manufacturer data: 4c000215a495bb80c5b14b44b5121370f02d74de004a045cfc, advType: Non-Connectable - No Scan Response
Advertised iBeacon Device: Name: , Address: f8:31:ac:ae:09:ed, manufacturer data: 4c000215a495bb20c5b14b44b5121370f02d74de004b04a9c5, advType: Non-Connectable - No Scan Response
Advertised iBeacon Device: Name: , Address: 88:c2:55:ac:26:81, manufacturer data: 4c000215a495bb40c5b14b44b5121370f02d74de004b044fc5, advType: Undirected - Connectable / Scannable

The first two devices in the log above were the two that weren't being picked up when the scan was set as active, the last (the connectable one) is the one that was being picked up.

Yes this should be changed for sure, I had thought about this before but didn't get around to working on it.
Should work if this line was changed to
if(pScan->m_scan_params.passive || event->disc.event_type == BLE_HCI_ADV_TYPE_ADV_NONCONN_IND) {

If you'd like to try that and let me know I'd appreciate it.

That worked! PR incoming. :)

Closed with the merge of #24


IT tests and CI / CD

/cc @ani-sha @rareddy

Looks like this has been addressed. Feel free to open if I am wrong.


class_radius_mean

There is class_numpoint_mean in config, but no class_radius_mean.
No need for class_radius_mean in softgroup?
Or is the class_radius_mean setting somewhere else?
Thank you.

SoftGroup doesnot requires class_radius_mean

Thank you.


Offer a Flux HelmRelease-based installer

Feature Request
Offer a Flux HelmRelease-based installer like https://flux-iac.github.io/tofu-controller/getting_started/#installation. The current Makefile-based installer is not GitOps friendly.

Agree currently this is not an option because I'd want to install it the gitops way here, an alternative would be to just install it using a Kustomization.

It's hard for an installer to be more flux native and flux friendly than https://raw.githubusercontent.com/flux-iac/tofu-controller/main/docs/rc.yaml.


Bring settings into functions

NOTE: Merge #5 first, this is built on top of it. There will probably be a conflict, so if that one is merged/closed, I'll modify this accordingly after.
This utilizes the settings file to automatically generate XPOS if not specified, and will also always use the settings file to render the image numbers on the stream. This avoids the possibility of configuring the emojis in a different order in settings, and having deploy screw up the order still, now order is consistent.
Also did some linting + validation checking on fblive.php as I noticed it wasn't very happy when there was no comments on the post yet, and we shouldn't be requesting profile images that don't request so often.

This PR addresses the issue #3 that I brought up. I left full customizability still via the settings file, but if the reactions are changed, all calls still function properly on the endpoint and now follow a universal order in both files.
Also included the auto-XPOS, which can be done by commenting it out or setting it to null. It also works at a per-emoji basis, so if one is slightly off-centered, you can adjust only that one.

Very nice work @ryanwhittaker. Top man! I will review these this evening after work.

Going to test this now and merge if all works fine.


Fixed Inferno minion having 12 tiers instead of 11

Can't find any info on T12 inferno neither on official wiki nor fandom wiki... and mage minions npc doesnt sell t12 inferno... ???

Can't find any info on T12 inferno neither on official wiki nor fandom wiki... and mage minions npc doesnt sell t12 inferno... ???

That's why I removed it from the code. I also had a guild member check for the recipe and it does not exist.

Can't find any info on T12 inferno neither on official wiki nor fandom wiki... and mage minions npc doesnt sell t12 inferno... ???

That's why I removed it from the code. I also had a guild member check for the recipe and it does not exist.

Oh I completely misread it my bad


manual publish to sonatype oss #964

Original PR by @natrem
Updates:

switch jackson-module-kotlin:2.9.+ to 2.9.8 (invalid for maven publication)
variable for version (if using -dev then replace by SNAPSHOT upon maven publication)
publish main jar, sources and javadoc to sonatype oss (snapshot or staging)
changed version from 1.0.0 to 1.0.0-dev to avoid publishing a release

publish task is not included in default build and should be triggered by ./gradlew build publish
allow #964 for manual publication
Notes on local execution:

commented out dokka (firewall issue when downloading oracle jdk 6)
commented out swagger downloads (firewall issue)
I executed ./gradlew build publish -x test (failing tests) and ./gradlew :zally-rule-api:publish :zally-rule-api:build.gradle
fails on the publication due to firewall (did not configure the proxy; my sample was built/published from travis)

⚠ please test before integrating because I was - obviously - not able to publish to zalando repository
I did not work on the travis updates for automatic publication

Substitutes #986

:+1:


cannot restore after double clicking to enlarge

I cannot restore after double clicking to enlarge.
Is there any solution to this terrible experience? There are 2 ideas

add an option to disable double clicking to zoom in
add an button to restore

For context this applies to 1.0.0b and 1.0.0d


Request: Flatter the tooltip health bar

I would like to have ability to make right bottom tooltip HP bar flat just like its on unit frames. You know, when I hover over other player or NPC, that box which pops up at right bottom. Can you please make it flat or consider adding ability to do so? Thank you.

This aint a tooltip mod, and will not modify tooltips


Models are not synced with MongoDB

Sails version: 1.0.0
Node version: 8.11.0
NPM version: 6.0.0
DB adapter name: sails-mongo
DB adapter version: 1.0.1
Operating system: MacOS Sierra - 10.12.6

I have created a fresh sails empty app and configured with sails-mongo adapter. On sails lift no error. Then, I proceed with api generate command:
sails generate api <MODEL>
I created 5 models through the command. After sails lift, only 1 model (user) synced with MongoDB. I have reproduce the error on another macOS with same configuration. What is wrong, could you help? Is it bug or my fault? Thanks.
Attached my work: https://bitbucket.org/aliemre/webet/src

Ok, fixed!

Hi @aliemre, I was wondering if you ever found out what was going on here? I'm interested in looking into this problem, but haven't been able to recreate it.

@aliemre I was looking at this again and noticed that your url in config/datastores was a little different than mine. Where you have localhost:27017, I have <IP_ADDRESS>:27017. This may be worth a try.

Hi @streleck
Thanks for your effort. I have rechecked my configuration. It's all same. However, It works currently. Collections are generated successfully.


ERROR: The file (./src/app/app/app.component.ts) does not exist.

INFO: Project property is set to 'fuse'.
ERROR: The file (./src/app/app/app.component.ts) does not exist.
ERROR: If you think that this error shouldn't occur, please fill up bug report here: https://github.com/maciejtreder/ng-toolkit/issues/new
INFO: stacktrace has been sent to tracking system.
Nothing to be done.
I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[ ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

Expected behavior

Minimal reproduction of the problem with instructions

Example repository

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

- Node version: XX
- Platform:

Others:

Closed for housekeeping purposes.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_85mhgrc7e9/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_85mhgrc7e9/clone/recipe/meta.yaml:26: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_85mhgrc7e9/clone/recipe/meta.yaml:34: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_85mhgrc7e9/clone/recipe/meta.yaml:34: missing_description: The recipe is missing a description
Errors were found


Add scope/collections information

Currently, information about scope/collections is not visualized yet. Add the required components to show the list of scopes & collections

I would like to see the scope/collections grouped as a accordion or collapse component of the table when they click on the bucket.
At the bucket level, a summary of the number of scopes and collections would be great to display.

now the library displays scopes and collections details


Datasplitter has issues with forward slash characters (/)

A simple data splitter handling key value pairs appears to insert spaces before the slashes.
Data Splitter:
<?xml version="1.0" encoding="UTF-8"?>
<dataSplitter xmlns="data-splitter:3"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="data-splitter:3 file://data-splitter-v3.0.xsd" version="3.0">
  <!-- Simple data spliter that breaks up key value pairs in the form of
        key=valuea
       into data records of the form
        <data name="key" value="valuea" />
  -->
  <!-- We split on new lines -->
  <split delimiter="\n">
    <group>
      <!-- Now get the key value pairs ensuring we check for escaped = chars -->
      <regex pattern=".*">
        <group reverse="true" ignoreErrors="true">
          <regex pattern="(([^=]|=\\)+)=([^ ]+)">
            <var id="temp" />
            <group value="$1">
              <split delimiter="=" escape="\">
                <data name="$temp$3" value="$2" />
              </split>
            </group>
          </regex>
        </group>
      </regex>
    </group>
  </split>
</dataSplitter>

with an input of
node=xyx date=26/03/2017 time=08:26:38 user=burn file=/home/burn/some_file.txt
generates
<?xml version="1.1" encoding="UTF-8"?>
<records xmlns="records:2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="records:2 file://records-v2.0.xsd" version="2.0">
  <record>
    <data name="file" value=" /home /burn /some_file.txt" />
    <data name="user" value="burn" />
    <data name="time" value="08:26:38" />
    <data name="date" value="26 /03 /2017" />
    <data name="node" value="xyx" />
  </record>
</records>

Will attach config and sample data file. Note that this is running on Stroom 5.0-beta.14 on either Chrome (57.0.2987.110) or Firefox (45.8)
StroomDataIssue173.zip
StroomConfigIssue173.zip

Wrote a simple Event pipeline to provide example of attempting to turn sample 173 data into an event. The pipeline is based on the Event Data pipeline found in the Stroom source.
StroomConfigIssue173e.zip


Feat: Allow localisation of emails

i18n is disabled by default, please make it possible to enable it.
Thanks.

@andrewshadura Can you test if #6 works for you?

Thanks, I cannot test it now, but it looks like it should be enough.


Refactor service api Record<string, never> to use exported Empty type

Description
Refactors duplicated Record<string, never> to use Empty type from common service API types file.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #3152

PR #3154   👈

This comment was auto-generated by Graphite.


Pluggable compact items

Makes the compactItem function overridable to allow custom packing strategies.

was supposed to be against my own fork - sorry for the noise.


[spring-framework] Automate releaseDate, eol and extendedSupport retrieval

Based on https://spring.io/projects/spring-framework#support.

Expected diff:
diff --git i/releases/spring-framework.json w/releases/spring-framework.json
index 81e7ec0..d8fa7fe 100644
--- i/releases/spring-framework.json
+++ w/releases/spring-framework.json
@@ -1,5 +1,30 @@
 {
-  "releases": {},
+  "releases": {
+    "6.1": {
+      "name": "6.1",
+      "releaseDate": "2023-11-16",
+      "eol": "2025-08-31",
+      "extendedSupport": "2026-12-31"
+    },
+    "6.0": {
+      "name": "6.0",
+      "releaseDate": "2022-11-16",
+      "eol": "2024-08-31",
+      "extendedSupport": "2025-12-31"
+    },
+    "5.3": {
+      "name": "5.3",
+      "releaseDate": "2020-10-27",
+      "eol": "2024-08-31",
+      "extendedSupport": "2026-12-31"
+    },
+    "5.2": {
+      "name": "5.2",
+      "releaseDate": "2019-09-30",
+      "eol": "2021-12-31",
+      "extendedSupport": "2023-12-31"
+    }
+  },
   "versions": {
     "6.1.4": {
       "name": "6.1.4",


bbc.co.uk/music/ support

Apparently, the BBC launched a new site. Please add support for bbc.co.uk/music/ support
youtube-dl  --verbose http://www.bbc.co.uk/music/clips/p02b9vkd
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.bbc.co.uk/music/clips/p02b9vkd']
[debug] Encodings: locale ISO8859-1, fs ISO8859-1, out ISO8859-1, pref ISO8859-1
[debug] youtube-dl version 2014.11.04
[debug] Python version 2.7.6 - IRIX64-6.5-IP35-mips-n32bit-ELF
[debug] exe versions: ffmpeg UNKNOWN, ffprobe UNKNOWN, rtmpdump 2.2b
[debug] Proxy map: {}
[generic] p02b9vkd: Requesting header
WARNING: Falling back on generic information extractor.
[generic] p02b9vkd: Downloading webpage
[generic] p02b9vkd: Extracting information
ERROR: Unsupported URL: http://www.bbc.co.uk/music/clips/p02b9vkd; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/nekoware/bin/youtube-dl/youtube_dl/extractor/generic.py", line 596, in _real_extract
doc = parse_xml(webpage)
File "/usr/nekoware/bin/youtube-dl/youtube_dl/utils.py", line 1311, in parse_xml
tree = xml.etree.ElementTree.XML(s.encode('utf-8'), **kwargs)
File "/usr/nekoware/lib/python2.7/xml/etree/ElementTree.py", line 1300, in XML
parser.feed(text)
File "/usr/nekoware/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
self._raiseerror(v)
File "/usr/nekoware/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
raise err
ParseError: mismatched tag: line 29, column 5579
Traceback (most recent call last):
File "/usr/nekoware/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 536, in extract_info
ie_result = ie.extract(url)
File "/usr/nekoware/bin/youtube-dl/youtube_dl/extractor/common.py", line 199, in extract
return self._real_extract(url)
File "/usr/nekoware/bin/youtube-dl/youtube_dl/extractor/generic.py", line 977, in _real_extract
raise ExtractorError('Unsupported URL: %s' % url)
ExtractorError: Unsupported URL: http://www.bbc.co.uk/music/clips/p02b9vkd; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

BBC Music Clips will be supported in the next version. Thanks for the report.

Verified working with 2015.01.03.
Thanks!


Additional Movable Commands

Greetings!
First of all, thank you for developing and sharing this awesome website with the world!
I briefly attempted to contribute some features on my own but seem to lacking an understanding of how to navigate the dart file ecosystem. Would you be so kind as to either reply with a brief explanation on how to alter movable token commands or help implement the following features?
User submitted feature request #1: Movable Token Rotations
Possible implementations:
(preferred)
When token selected, make Alt+Left Mouse Click  rotate the token character (originally the bottom side) towards the pointer position
or
When mouse pointer on movable token, make Ctrl or Shift + Left Mouse Drag  rotate the token
User submitted feature request #2: User Movement Sequencing Similar to the Measurement Tool
Possible implementation:
When moving a token, make Right Mouse Click leave a point that becomes the new next measurement origin, the cumulative distance is displayed
Thanks again for this beautiful tool!
Best regards,
Derin

Hi there, thanks for your feature request!
Right, so token rotation is something I hadn't thought of before but which is actually a great addition to the VTT. Kudos for your proposed implementations :)
Regarding the second request, I agree distance pins really should be possible when moving tokens across the board to further consistency and overall ease of use.
I'll admit the source code may be a mess to get into right now due to a lack of inline documentation (:trollface:) - sorry about that. I'll have a look at implementing your preferred proposals for the next update and maybe work out some code comments on the way.
Cheers!

Thank you very much for the quick reply, I appreciate your hard work!

Hi, your features have been implemented and can be tested at dungeonclub.net right now :)
Before publishing a new release for self-hosting, may I get your thoughts on the way token rotation is done? If you've got an idea on how to improve it, let me know.
Cheers!

Hello!
Thank you for implementing the changes promptly, right on time for the holiday season :)
The distance pin implementation feels great! As for token rotation, I think it is also a fine addition. I checked it on all the grid types and I like that you have chosen a snapping rotation for the gridded maps and to pointer postion for the ungridded map. I have a few questions:

Is there a reason why you chose the top of the token as the reference/indicator? (Given the token assets provided by 2 minute token editor are downwards facing)
Do you think it would be difficult to change token rotation on gridded maps so that the rotation can snap to any grid cell? (To more distant ones as well) I believe this would make the rotation more intuitive and useful while still preserving the option to snap to the directions you have implemented by choosing the nearest cells.

Thank you and happy holidays!

Come to think of it, how do you think tokens that aren't snapped to the grid should be rotated?
Should the rotation still snap towards an actual center of a grid cell, or should we fallback to the way rotations are currently done (e.g. simply snapping to any 45° angle on a square grid)?

This is a great question! I see value in both your propositions: I think being able to face a grid cell makes more sense while story telling but alligning the token with the grid would indeed be a bit circuitous (first allign, then rotate, then position back). A few ways you could approach this would be: (with my opinion in parenthesis)

Implement a way to snap to both to grid cells and the angles (most exciting, however this could turn into a disorderly creative coding problem if it's not straight forward to implement)
Implement a modifier key to snap to angles instead (fullfills most requests but we might be running out of modifier keys)
Leave it as is until we collect more feedback/opinions (the circuitous solution it is, which is not that bad)
Fallback to angle snapping (feels bad for facing other directions/objects)
Implement a modifier key to make rotation not snap, a la ungridded map (doesn't actually solve the implicit problem you posed :/ ; but could be a general answer to every future problem )

Feels great! Thank you!


”Lungani”, ”Crucea”, ”Goești”, ”Zmeu” are listed in the drop-down menu

Description:
When user writes ”Lungani”, ”Crucea”, ”Goești”, ”Zmeu” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Lungani"

Expected results
The location appears in the drop-down menu as "Sat Lungani, IAȘI (Lungani)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Crucea"

Expected results
The location appears in the drop-down menu as "Sat Crucea, IAȘI (Lungani)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Goești"

Expected results
The location appears in the drop-down menu as "Sat Goești, IAȘI (Lungani)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Zmeu"

Expected results
The location appears in the drop-down menu as "Sat Zmeu, IAȘI (Lungani)".


compute_face_descriptor(): incompatible function arguments

face_recognition version: 1.2.3
Python version: 3.10.12
Operating System: Ubuntu 22.04

Description
I'm running into a problem with a fresh installation of dlib and face_recognition according to the doc. The sample code I ran is https://github.com/ageitgey/face_recognition/blob/master/examples/facerec_from_webcam_faster.py
What I Did

Provided obama.jpg and biden.jpg in the root of my project
Created a test.py which is literally the sample code from_webcam_faster.py, but video capture replaced by RTSP input and video display replaced by image write. I can share the code, if required.

python3 test.py

I'm getting this (and I'm getting this with various other samples too, so I suppose some incompatibility between face_recognition and dlib):
~/dlibtest$ python3 test.py
Traceback (most recent call last):
  File "/home/ubuntu/dlibtest/test.py", line 50, in <module>
    face_encodings = face_recognition.face_encodings(rgb_small_frame, face_locations)
  File "/usr/local/lib/python3.10/dist-packages/face_recognition/api.py", line 214, in face_encodings
    return [np.array(face_encoder.compute_face_descriptor(face_image, raw_landmark_set, num_jitters)) for raw_landmark_set in raw_landmarks]
  File "/usr/local/lib/python3.10/dist-packages/face_recognition/api.py", line 214, in <listcomp>
    return [np.array(face_encoder.compute_face_descriptor(face_image, raw_landmark_set, num_jitters)) for raw_landmark_set in raw_landmarks]
TypeError: compute_face_descriptor(): incompatible function arguments. The following argument types are supported:
    1. (self: _dlib_pybind11.face_recognition_model_v1, img: numpy.ndarray[(rows,cols,3),numpy.uint8], face: _dlib_pybind11.full_object_detection, num_jitters: int = 0, padding: float = 0.25) -> _dlib_pybind11.vector
    2. (self: _dlib_pybind11.face_recognition_model_v1, img: numpy.ndarray[(rows,cols,3),numpy.uint8], num_jitters: int = 0) -> _dlib_pybind11.vector
    3. (self: _dlib_pybind11.face_recognition_model_v1, img: numpy.ndarray[(rows,cols,3),numpy.uint8], faces: _dlib_pybind11.full_object_detections, num_jitters: int = 0, padding: float = 0.25) -> _dlib_pybind11.vectors
    4. (self: _dlib_pybind11.face_recognition_model_v1, batch_img: list[numpy.ndarray[(rows,cols,3),numpy.uint8]], batch_faces: list[_dlib_pybind11.full_object_detections], num_jitters: int = 0, padding: float = 0.25) -> _dlib_pybind11.vectorss
    5. (self: _dlib_pybind11.face_recognition_model_v1, batch_img: list[numpy.ndarray[(rows,cols,3),numpy.uint8]], num_jitters: int = 0) -> _dlib_pybind11.vectors

Invoked with: <_dlib_pybind11.face_recognition_model_v1 object at 0x7814e5b164b0>, array([[[164, 138,  99],
        [164, 138,  99],
        [164, 138,  99],
        ...,
        [ 93,  80,  57],
        [ 92,  79,  56],
        [ 92,  79,  56]],

       [[164, 138,  99],
        [164, 138,  99],
        [164, 138,  99],
        ...,
        [ 93,  80,  57],
        [ 92,  79,  56],
        [ 92,  79,  56]],

       [[164, 138,  99],
        [164, 138,  99],
        [164, 138,  99],
        ...,
        [ 97,  82,  62],
        [ 95,  80,  60],
        [ 95,  80,  60]],

       ...,

       [[ 59,  28,   5],
        [ 62,  31,   8],
        [ 66,  35,  12],
        ...,
        [119, 121, 117],
        [117, 119, 118],
        [107, 109, 109]],

       [[ 56,  25,   2],
        [ 60,  29,   6],
        [ 64,  33,  10],
        ...,
        [119, 121, 114],
        [116, 120, 115],
        [106, 111, 107]],

       [[ 56,  25,   2],
        [ 59,  28,   5],
        [ 62,  31,   8],
        ...,
        [118, 119, 113],
        [116, 120, 115],
        [111, 116, 112]]], dtype=uint8), <_dlib_pybind11.full_object_detection object at 0x7814e5af3e30>,

Try this GRAY conversion before calling face_recognition.face_locations. It resolved this issue for me.
`
rgb_frame = frame[:, :, ::-1]
rgb_frame = cv2.cvtColor(rgb_frame, cv2.COLOR_BGR2RGB)
face_locations = face_recognition.face_locations(rgb_frame)  `

Yes, that solved the issue. Thanks

It is working.


Fix possible JavaDoc error in QueryStringEncoder. Type mismatch.

Old javadoc instantiated object of QueryStringDecoder. This seems like a type mismatch, copy-and-paste issue, but I might be missing something.

Comitted the change to 3.2 and modularization branch (as this will become new master soon)

Thanks!

No problem, let me know if you guys need/want anymore help. Features, docs, etc.


GLFW window crashes when "Mute speakers" or "Increasing/Decreasing a volume" keys pressed once.

Hi there!
I noticed one strange thing - when I press my volume keys on my keyboard ("Defender", made in China) my GLFW (3.2) window freeze and Win7 trying to close this program urgently. May be GLFW can't handle codes of this keys, I don't know. How can I solve this problem, without using 3.2.1 version? It's important for me to stay on 3.2.

Does the GLFW events test program have the same problem?

Events, cursor, gamma and other works fine. Events program gives:
Key 0xfffffffff Scancode 0x012e  - when "Increase volume"
Key 0xfffffffff Scancode 0x0130  - when "IDecrease volume"

Are you using the key passed to your key callback as an array index without checking if it's -1?

Yes, I use array of states and only two state of key callback -"press" and "release". How is the proper way to handle "unknown key"? Something like that is calls exception and I can't just skip this error:
static void keyCallback(GLFWwindow* window, int key, int scancode, int action, int mode)
{
if (action == GLFW_PRESS)
keys[key] = true;
if (action == GLFW_RELEASE)
keys[key] = false;
if (action == GLFW_KEY_UNKNOWN)
return;
}

@openGLnoob The simple fix is to add if (key == GLFW_KEY_UNKNOWN) { return; } to the top of your keyCallback function. In the snippet you posted you are 1) checking too late, after you have already used key to index into your array, and 2) checking action rather than key.

@tombsar Thank you! It works as expect!


Issue with PS4 entity

Description of the issue
I am not sure if you had the intention of supporting PS4s with this add-on, but when I setup the first version I already had the ps4 in my credentials.json I was using from a previous integration (PlayActor).  So my PS4 showed up and created an entity but it was unavailable.  I just ignored it assuming there was no support for the 4.  With your latest updates and the Web UI it appears to have the correct status for the PS4 and Standby/Awake States on the status page for each one but the entity remains unavailable.  I tried going through the Reauth steps on the 4 and everything appears to work correctly, and the WebUI still shows the 4 and the correct state, but the entity remains unavailable and I have this is my log:
2022-08-04T14:36:40.910Z @ha:ps5:error SyntaxError: Unexpected end of JSON input     at JSON.parse (<anonymous>)     at /app/server/dist/redux/sagas/check-devices-state.js:65:38     at step (/app/server/dist/redux/sagas/check-devices-state.js:24:23)     at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:5:53)     at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)     at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)     at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)     at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)     at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)     at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
Should this be working for the PS4 or is there I way I can exclude it from discovery if not?  Your add-on seems to be very quick and reliable as far as the status of the PS5s so I was hoping to use that for the 4 instead of the official integration.  Thank you for your time.
Addon version
0.7.0
Platform
Core: 2022.8.0
Supervisor: 2022.07.0
OS: Debian 11
Logs of the issue (if applicable)
No response

Originally, I didn't intend for PS4's to be supported, as mentioned in the FAQ. For reasons that I mentioned there.
I also took precautions to filter out PS4 devices by using filters in Playactor that apparently don't work. I can however add an additional filter that would be applied post Playactor discovery to still achieve this functionality.
However, I wasn't aware that my add-on delivers better performance than the PS4 integration that's currently available for HA.
So I'm now tempted to add a configuration option that allows you to filter out PS4 devices if you don't want this add-on picking them up. Meaning that you can opt-in for PS4 discovery with this add-on as well. That would however mean that, even though it's achievable for PS4's, PS4's would also only allow toggling Power state with this change.
Tell you what, I'll add the option mentioned above to the edge version and we'll see how it works.

I've just updated the edge version of the add-on to include the change mentioned above. Would you mind trying it out to see if it behaves as expected? 😊 You might need to remove the previously discovered PS4 device in Home Assistant first. Also don't forget to disable the regular add-on before starting the edge version.

However, I wasn't aware that my add-on delivers better performance than the PS4 integration that's currently available for HA.

Yes, my experience so far has been that PlayActor and the manually created switch entities don't reflect the state changes fast enough most of the time, and I have some automations that require the state to change for the Alexa response to be accurate.  So far it seems that your add-on is much more consistent in reflecting the state changes faster.  And that was part of the reason I was trying to get the PS4 to work through it, and the official PS4 integration seems to occasionally lose connection when we don't use it for awhile.
But thank you, I will try the edge version and see if it successfully filters out the PS4.  Thanks for your time, nice work with the Add-On.

Yes it appears to have ignored the PS4 after the initial discovery and no more errors in logs.  Thank you very much.


[Feature Request] Exclude by Name/Blacklist

I have two Radarr instances: one for HD/SD movies, and another for 4K.
Syncarr works great for getting the two mirrored, but some older movies don't have 4K versions, so they show up as "Missing", and there's no way to hide them.
It would be great if you could create a blacklist of sorts, so movie titles found within it are not synced between instances.

this is something doable. Thinking off the top of my head, we can filter by a number of things as long as they are absolutely unique. I think filtering by slugTitle would be the easier and most user friendly (unique url path for each movie). So it would be a commas separated list of slugs:
blacklist_slugs= movieA-123,movieB-242,movieC-343

What's the slugTitle?

the generated url path of the movie

added blacklist option in v1.10.0


⚠ SceneTime has degraded performance

In 8b7943e, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2154 ms

Resolved: SceneTime performance has improved in f571210 after 6 minutes.


Firefox, scroll does not work on an iframe

If I scroll with the mouse wheel on an iframe within the content, the scroll does not work.
Only in firefox

I am facing the same issue on Mac Safari 8, mousewheel doesn't work

Check if the iframe is on the same domain as the host page. If not, the browser prevents js events from running as a security feature.


fix(ds): update auth endpoints

Merge with rivet-gg/rivet-ee#141

#1046
#1045
#1044  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


When the number of rows in table is small and reload_data=True, the selection doesn't show up in UI

When the number of rows in the table is small and reload_data=True, the selection doesn't show up
Minimal example to reproduce:
import streamlit as st
from st_aggrid import GridOptionsBuilder, AgGrid
import pandas as pd

csv_url = (
    "http://archive.ics.uci.edu/ml/machine-learning-databases/wine-quality/winequality-red.csv"
)
df = pd.read_csv(csv_url, sep=";")
df = df.head(28)  # HERE!!! when set to 29, it's working

st.title("Not showing selection example")

gb1 = GridOptionsBuilder.from_dataframe(df)
gb1.configure_selection(selection_mode="multiple", use_checkbox=True)
gridOptions = gb1.build()
response = AgGrid(
    df,
    gridOptions=gridOptions,
    width="100%",
    update_mode="SELECTION_CHANGED",
    reload_data=True,
    key='example',
)

print(response['selected_rows'])

The row that is selected is printed but you can't see it in the UI

Looks like you're using reload_data to "prevent" grid to reload between streamlit refreshs. If this is the case, try setting a fixed key instead. By design streamlit's components will not redraw if key is fixed.

@smolendawid, a workaround for your issue is to set the "key" argument of the AgGrid widget depending on your filter settings.
For example you could just append your settings to a string. In this way it will not reuse the old one and create a new table when the settings change.

This workaround works, congrats you figured it out!
@PablocFonseca my issue is not a 'question' I think.


disable grpc window update

TiDB use 16 connections for each kv store (maybe it's not wise choosen) and http2 flow control seems to be useless, so int TiDB, dynamic window is disabled
https://github.com/pingcap/tidb/blob/461512b8273a7b64580cd96e34c8e2d7983b5e62/store/tikv/client.go#L249
https://github.com/grpc/grpc-go/blob/master/internal/transport/http2_client.go#L210
but unistore keep use window update frame, from TiDB scheduler pprof:

unionstore keep send updateWindow frame to TiDB, so this PR try to disable it in unistore too
and do some sysbench point-get test:
before:
sysbench --config-file=tidb-config oltp_point_select --tables=32 --threads=32 --table-size=100000 run
[ 5s ] thds: 32 tps: 54212.80 qps: 54212.80 (r/w/o: 54212.80/0.00/0.00) lat (ms,95%): 1.01 err/s: 0.00 reconn/s: 0.00
[ 10s ] thds: 32 tps: 37306.71 qps: 37306.71 (r/w/o: 37306.71/0.00/0.00) lat (ms,95%): 1.58 err/s: 0.00 reconn/s: 0.00
[ 15s ] thds: 32 tps: 31182.89 qps: 31182.89 (r/w/o: 31182.89/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00
[ 20s ] thds: 32 tps: 31335.87 qps: 31335.87 (r/w/o: 31335.87/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00
[ 25s ] thds: 32 tps: 31371.22 qps: 31371.22 (r/w/o: 31371.22/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00
[ 30s ] thds: 32 tps: 31386.28 qps: 31386.28 (r/w/o: 31386.28/0.00/0.00) lat (ms,95%): 1.76 err/s: 0.00 reconn/s: 0.00
[ 35s ] thds: 32 tps: 31403.74 qps: 31403.74 (r/w/o: 31403.74/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00
[ 40s ] thds: 32 tps: 31270.10 qps: 31270.10 (r/w/o: 31270.10/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00
[ 45s ] thds: 32 tps: 31571.52 qps: 31571.52 (r/w/o: 31571.52/0.00/0.00) lat (ms,95%): 1.73 err/s: 0.00 reconn/s: 0.00

after
sysbench --config-file=tidb-config oltp_point_select --tables=32 --threads=32 --table-size=100000 run
[ 5s ] thds: 32 tps: 50061.73 qps: 50061.73 (r/w/o: 50061.73/0.00/0.00) lat (ms,95%): 1.21 err/s: 0.00 reconn/s: 0.00
[ 10s ] thds: 32 tps: 32300.74 qps: 32300.74 (r/w/o: 32300.74/0.00/0.00) lat (ms,95%): 1.70 err/s: 0.00 reconn/s: 0.00
[ 15s ] thds: 32 tps: 32684.99 qps: 32684.99 (r/w/o: 32684.99/0.00/0.00) lat (ms,95%): 1.67 err/s: 0.00 reconn/s: 0.00
[ 20s ] thds: 32 tps: 32698.61 qps: 32698.61 (r/w/o: 32698.61/0.00/0.00) lat (ms,95%): 1.70 err/s: 0.00 reconn/s: 0.00
[ 25s ] thds: 32 tps: 32561.84 qps: 32561.84 (r/w/o: 32561.84/0.00/0.00) lat (ms,95%): 1.67 err/s: 0.00 reconn/s: 0.00
[ 30s ] thds: 32 tps: 32563.95 qps: 32563.95 (r/w/o: 32563.95/0.00/0.00) lat (ms,95%): 1.70 err/s: 0.00 reconn/s: 0.00
[ 35s ] thds: 32 tps: 32710.36 qps: 32710.36 (r/w/o: 32710.36/0.00/0.00) lat (ms,95%): 1.67 err/s: 0.00 reconn/s: 0.00
[ 40s ] thds: 32 tps: 32655.44 qps: 32655.44 (r/w/o: 32655.44/0.00/0.00) lat (ms,95%): 1.67 err/s: 0.00 reconn/s: 0.00
[ 45s ] thds: 32 tps: 33016.58 qps: 33016.58 (r/w/o: 33016.58/0.00/0.00) lat (ms,95%): 1.64 err/s: 0.00 reconn/s: 0.00

test result isn't very stablable, but it "seems" more better

This change is

LGTM


License?

cc @stvno @Holly-Transport
I propose MIT.

I agree :-)

I also agree, added a MIT License file for the Worldbank


Update german translation

It overcame me and so I finally did it.
I've spllit the work into different commits and described some of my changes so you can easily follow what I've done.
New strings are translated, old translations are fit to the current ones.
Some grammatical mistakes have been fixed as well as the wrong capitalisation of "du", "dich", "dein", etc.
You can open the commit messages to see the reasons behind the changes.

Sidenote: There is one change, I'm not very confident with and it's the change in line 39. "Liebe" to "Lieb's".
Am I correct to assume that this will appear on the button where you can "love" a picture/video/writing at some point? If so, than this is the more fitting translation. If not, can you tell me where it will be displayed?
I also saw some translations already been provided but not displayed on site, mainly in the settings area. Is this something you're still working on?
Last but not least: "Home" is right now translated as "Zuhause", which is not wrong. You can either translate is as that or as "Startseite" (home page).
"Zuhause" is kinda cute as it can be interpreted as your... safe harbour, your comfort zone... just, your home. (But it has to be an intended choice as otherwise it might seem badly translated)

@ProfessorBaum First of all, I really appreciate how you explained everything! Really really nice! It's very hard reviewing translations when it's not your mothertongue so this makes it a lot easier.

Am I correct to assume that this will appear on the button where you can "love" a picture/video/writing at some point? If so, than this is the more fitting translation. If not, can you tell me where it will be displayed?

It's used as the love button on the feel in the large stories:

So I assume Liebe would fit better?

I also saw some translations already been provided but not displayed on site, mainly in the settings area. Is this something you're still working on?

Yeah we need to add A LOT more strings. Which we will do at some point for sure.

"Zuhause" is kinda cute as it can be interpreted as your... safe harbour, your comfort zone... just, your home. (But it has to be an intended choice as otherwise it might seem badly translated)

Hmmm, I think it's indeed cute to use it like that for now. What do you think?

 If you have any questions about changes I made, please ask. I know I could've been more elaborate on some changes but I didn't want to do a line-by-line comment as that would be very much work. But if you request it, I can do so :)

Am I correct to assume that this will appear on the button where you can "love" a picture/video/writing at some point? If so, than this is the more fitting translation. If not, can you tell me where it will be displayed?

It's used as the love button on the feel in the large stories:
So I assume Liebe would fit better?

Okay, yeah. We can do Liebe, that's nice. What I did was a very short form for "I love it", so it's more like an expression.
Writing Liebe means more like you "give" some love to it. Which is also cool! Both are nice, and I'm fine with however you want to make it sound.

I also saw some translations already been provided but not displayed on site, mainly in the settings area. Is this something you're still working on?

Yeah we need to add A LOT more strings. Which we will do at some point for sure.

Yeah, no, I meant, that some strings aren't in use? There is the "following" section in settings which has translated strings in this file but they're not displayed on the site for example. (Just pointing it out in case it's a bug. I'm not sure if you look at the site in something else than English so these errors might be missed)

"Zuhause" is kinda cute as it can be interpreted as your... safe harbour, your comfort zone... just, your home. (But it has to be an intended choice as otherwise it might seem badly translated)

Hmmm, I think it's indeed cute to use it like that for now. What do you think?

It's nice. My first impression was "weird translation but okay" but I've grown to adore it because it really is cute to say it like that. And if the rest of the site is fully translated at one point, it won't stand out as a weird but very deliberate choice of words so it's cool. :)

If you have any questions about changes I made, please ask. I know I could've been more elaborate on some changes but I didn't want to do a line-by-line comment as that would be very much work. But if you request it, I can do so :)

If all of the PR's were like this, we would be very happy! Mind if I use yours as an example in the Readme?

Okay, yeah. We can do Liebe, that's nice. What I did was a very short form for "I love it", so it's more like an expression.
Writing Liebe means more like you "give" some love to it. Which is also cool! Both are nice, and I'm fine with however you want to make it sound.

Alright! Mind changing it to Liebe?

There is the "following" section in settings which has translated strings in this file but they're not displayed on the site for example. (Just pointing it out in case it's a bug. I'm not sure if you look at the site in something else than English so these errors might be missed)

Oh you might be right. @nenadjelovac mind having a look please?

It's nice. My first impression was "weird translation but okay" but I've grown to adore it because it really is cute to say it like that. And if the rest of the site is fully translated at one point, it won't stand out as a weird but very deliberate choice of words so it's cool. :)

Perfect! Let's keep it like that then :D @ProfessorBaum
Let me know when you made the changes so we can merge 👍

If you have any questions about changes I made, please ask. I know I could've been more elaborate on some changes but I didn't want to do a line-by-line comment as that would be very much work. But if you request it, I can do so :)

If all of the PR's were like this, we would be very happy! Mind if I use yours as an example in the Readme?

Sure. :)

Let me know when you made the changes so we can merge 👍

Will do, do you have an opinion on the other two comments I made? Then I will just do them in one commit.

@ProfessorBaum Commented on them!

Done 👍

@ProfessorBaum You rock! This will go live shortly!

Oh, somebody took over the German translation. Nice. Have fun!


Fix SunSpecClientBlockBase.setitem() parameter name mismatch

Also updated a few similar Comments.  If they don't get deleted they
ought to be maintained.
#31

LGTM +1 to merge


CI: introduce retry in sonobuoy status check

There is currently no retry set while checking the status of the
sonobuoy tests so any temporal issue will cause the whole test to
fail.
This patch introduces a new flag to allow a max number of retries
while checking the test status instead of failing directly
Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes #
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Maybe you could just use --wait instead of retrying, see https://gitlab.suse.de/mkravec/scripts/-/blob/master/tests/sonobuoy.sh#L5

thanks !@kravciak  I was actually trying to reproduce the issue from jenkins and wanted several retries in there to understand why it was failing (temporal issues, wrong command) althougth I dont think a retry will hurt for those cases it seems like the actual issue is that the containers are removed when the test ends? So the status check fails because there are no pods!
In that case, I think the wait option is the proper one, instead of just checking to see if its finished, lets just wait for it to be done and use the step timeout on jenkins to avoid it getting out of hand. Otherwise its gonna be a terrible mess of coded to check if it finished,


feat: custom dipole support in QCSchema

Summary
This adds custom support to store dipole moment integrals in the QCSchema format. While this is not part of the official specification, it is aligned well with the remainder of the QCSchema and is not the only custom extension which we support.
This will allow us to achieve a more consistent and simple implementation of our classical drivers.
Details and comments
The main motivator of this PR was seeing the state of #873 where the approach prior to this PR was to:

dump data into a private _QMolecule container
dump that into an HDF5 file
load the data back in Qiskit python's process and populate the _QCSchemaData utility
generate a QCSchema object from this data
load a problem from this data
possibly inject the dipole data loaded from _QMolecule

Using the changes from this PR will allow:

all drivers to consistenly handle the dipole data via the QCSchema directly (and via _QCSchemaData if needed)
simplify the Psi4 workflow as follows:

in Psi4 template generate a QCSchema object (optionally via the _QCSchemaData utility)
dump this to HDF5 using QCSchema.to_hfd5()
load the problem directly from the HDF5 file in Qiskit's python process

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

147 of 156 (94.23%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 85.83%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_nature/second_q/drivers/gaussiand/gaussiandriver.py
21
25
84.0%

qiskit_nature/second_q/formats/qcschema_translator.py
39
44
88.64%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
17209

Relevant Lines:
20050

💛  - Coveralls


fix(fmt): eof newline

address #2667

this regression was my bad! thanks for fixing it. im wondering though if we can just change this line https://github.com/foundry-rs/foundry/blob/master/fmt/src/formatter.rs#L3492 to actual compare the strings and use actual correct line endings in windows but that would be a slightly more intense story

i thought about it again and it seems even the standard library doesn't care about inserting carriage returns, so i think this is fine the way that it is! https://doc.rust-lang.org/std/macro.writeln.html


Examples lack Collection View example

The current set of example projects lack a collection view sample and the readme seems to have insufficient information on how to make this mode work.

I'm having trouble getting cells to load with the UICollectionView version and it seems there aren't examples anymore? My cellForItemAt is never being called although my numberOfItemsInSection is being called and returning a number above 0. I registered my cell using self.collectionView.register(CommentCell.self, forCellWithReuseIdentifier: CommentCellIdentifier) and init with:
    convenience init() {
        self.init(collectionViewLayout: UICollectionViewLayout())
    }

Any idea what step I could be missing?

FWIW - copying and pasting the UICollectionView version I made to a UITableView version almost directly (obviously converting the delegate methods) works so I really think a super basic example for UICollectionView would be very helpful