Sendkeys is sending the incomplete string

Hello folks,
I'm having trouble sending sendkeys in my application. Sendkeys itself works, but sometimes it does not send the full string to the field of my application. I tried to solve this in two different ways (as you can see the examples below), but even then this error occurs sometimes.
My application is running in Codeship (CI) and also runs locally, in both environments the error occurs

Node Version: Node Version read from package.json: 6.2.2; Node Version used: 6.2.2
Protractor Version: 5.1.1
Angular Version: I'm not using angular for that
Browser(s): ChromeDriver; Driver info: chromedriver=2.29.461571
Operating System and Version platform=Linux 4.2.0-42-generic x86_64
Your protractor configuration file: See here
A relevant example test: I've added two examples just below
Output from running the test

Steps to reproduce the bug: I have already tried using two ways to solve this, but with both ways the same error occurs

        this.name.click();
        this.name.sendKeys(text);

        var nameValue = 'odit';
        nameValue.split('').forEach((c) => this.name.sendKeys(c));

The URL you are running your tests against (if relevant): Private project, I can not send the url

I've been seeing this as well. It's been failing for me on Chrome.

Very interesting. Just curious, do you see this when running the example tests or just in your application? What happens if you downgrade your chromedriver version?

@NickTomlin I'll do these tests soon and let you know

@rafaelcs great, thanks!

@NickTomlin Using chromedriver version 2.28, the problem is the same. See:

The error did not occur with the example script

Just from the information provided, it looks like it is not an Angular page. If you turn ignoreSynchronization to true, then you are responsible for your own waiting. Waiting could mean that:

you are waiting arbitrarily for your page to load / stabilize
you chain your promises with then methods to wait for previous promises to resolve
a combination of arbitrarily waiting and chaining promises

Something like this should work with ignoreSynchronization set to true.
import {promise as wdpromise} from 'selenium-webdriver';

someFunction = (): wdpromise.Promise<any> => {
  return this.name.click().then((): wdpromise.Promise<any> => {
    return this.name.sendKeys(text);
  });
}

@rafaelcs
Did you manage to solve this or is it still open? Did you also tried the solution @cnishina provided?
There is also no activity on this issue since the 17th of May. If there is no further activity at the end of the week I assume the issue is solved and I'll close this issue.

@wswebcreation I'm seeing this issue regularly but indeterministically on my project too… I don't think it is solved.

@victornoel
Do you see this when running the example tests or just in your application?

@wswebcreation

it does not happen with those tests
it only happens on one specific place in my application, but not on any other
it happens sometimes, usually it is dependent on performances

I'm using chormium 61, chromedriver 2.32, protractor 5.1.2
I can try to replicate it on a smaller example, but I suspect it won't happen because it is tied to the fact my application is complex. I will try to replicate it deterministically on my application and provide you with instructions to witness it maybe first :)

@victornoel
Tnx. I happened to see something similar in a previous project I was working on. The validations on the input made Protractor go completely crazy, I fixed it with a browser.executeScript('//some code injection');  to get it working. Meaning I was not really testing the input, but the input was more testing my patience ;-)

@wswebcreation ah yes, me too there is some validation (from angular reactive forms) on this input I'm testing!


update README.md to remove reference to /edge

the "getIronSession" seems to work for me when I remove the /edge from the import. Using the /edge resulted in an error that says. " Error: no native implementation of WebCrypto is available in current context at getCrypto". Just thought this might be a good upgrade to the documentation to help new users. Thank you so much for your work on this library!

It'll be removed in v7.


Deposited vs Receipted Comparison in check_output

Hi there,
I was reviewing the check_output function and noticed that it compares deposited and receipted as follows:
https://github.com/ickb/v1-core/blob/92be3c3ea31226dece0f7235b12cf5b96766a8a3/scripts/contracts/ickb_logic/src/entry.rs#L132-L137
From what I understand:
deposited is incremented by 1 each time a deposit occurs, which seems to track the number of deposit cells
https://github.com/ickb/v1-core/blob/92be3c3ea31226dece0f7235b12cf5b96766a8a3/scripts/contracts/ickb_logic/src/entry.rs#L109
receipted is incremented by u128::from(deposit_quantity), which appears to track the total deposit quantity.
https://github.com/ickb/v1-core/blob/master/scripts/contracts/ickb_logic/src/entry.rs#L119
Given that deposited counts the occurrences while receipted tracks the total quantity, it seems these two values represent different concepts, I was wondering if this comparison might lead to an unexpected mismatch error?

Hey @jlguochn, thank you for publicly expressing your interest in iCKB by auditing the proposal and L1 scripts source code as part of the Scalebit external audit, I personally appreciate a lot!! 🙏
In particular I really appreciate this question (that was never asked before) and the use of links (if you want, feel free also to include the relevant code as code blocks).

Given that deposited counts the occurrences while receipted tracks the total quantity, it seems these two values represent different concepts, I was wondering if this comparison might lead to an unexpected mismatch error?

So we agree that the check is already sensitive, but maybe too sensitive, right?
If you look at the proposal:

A single receipt tracks a group of deposits with same unoccupied capacity in the current tx output. Multiple receipts for a specific unoccupied capacity may be created where each one keep track of a different group of deposits. [...] A tx may create many deposits with the same unoccupied capacity. This counter [deposit_quantity] keeps track of how many deposits with the same unoccupied capacity are being tracking in the current tx by this specific receipt.

The starting point is that in a single tx many receipts may be created and many more deposits may be created.
Let's make an example, let's assume the following iCKB deposits are created in the current tx:

100082 CKB, so 100000 CKB of unoccupied capacity (100082 CKB - 82 CKB).
100083 CKB, so 100001 CKB of unoccupied capacity.
100084 CKB, so 100002 CKB of unoccupied capacity.
100082 CKB, so 100000 CKB of unoccupied capacity.
100084 CKB, so 100002 CKB of unoccupied capacity.
100083 CKB, so 100001 CKB of unoccupied capacity.
100082 CKB, so 100000 CKB of unoccupied capacity.
100082 CKB, so 100000 CKB of unoccupied capacity.

They could be tracked by the following receipts:

deposit_amount (DA) : 100000, deposit_quantity (DQ) : 4.
DA : 100001, DQ : 2.
DA : 100002, DQ : 2.

But also a valid receipt configuration could be the following:

DA : 100002, DQ : 2.
DA : 100000, DQ : 1.
DA : 100001, DQ : 2.
DA : 100000, DQ : 2.
DA : 100000, DQ : 1.

What matters is that that the receipt accounting for each deposit size checks out.
@jlguochn does it make more sense now?

@phroi  OK, I see. Thank you so much for your detailed explanation and the examples.

No worries, my pleasure!!
@jlguochn do you have any more doubts on this? If so, feel free to ask more questions, otherwise feel free to close the issue 🤗

Thanks!  I'll close the issue now.

Thank you again @jlguochn for taking your time to understand and review the iCKB proposal and Scripts 🙏


[java] Pin Ubuntu Snapshot

Motivation

Changes

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Not needed anymore, at least for Java weblogs, since I removed any use of apt-get.


Basic compute

Closes: #120
The goal of this PR is not to add a high level API for GPU compute to bevy, rather to add a corner stone for compute, which will allow different people to think of high level API's that can be used to expose compute functionality to bevy users. This adds and exposes pipelines and a compute pass and all of the required stuff from wgpu to make "compute" functional from a very low level.
Note: This adds zero easy of use compute functionality. This does not auto handle bind groups or any fancy stuff yet. This does not auto work with render resources.
A list of major changes:

PipelineDescriptor > RenderPipelineDescriptor
Pipeline > RenderPipeline
PipelineSpecialization > RenderPipelineSpecialization
RenderContext

begin_pass > begin_render_pass
new begin_compute_pass

PipelineCompiler

new get_specialized_compute_pipeline
compile_pipeline > compile_render_pipeline
new compile_compute_pipeline
new iter_compiled_compute_pipelines
new iter_all_compiled_compute_pipelines

new ComputePass

There might be more I missed.. 😅
Edit: Fixed my commit author stuff.

This is a bit of a nit and a style thing of mine but when I have #[derive(..)] for a struct, I like when they are in alphabetical order.

The goal of this PR is not to add a high level API for GPU compute to bevy, rather to add a corner stone for compute, which will allow different people to think of high level API's that can be used to expose compute functionality to bevy users.

Do you think it makes sense to add a simple compute example at this stage, or is this still too low level? I'm thinking it could be super simple, like doing a few multiplications on the gpu, and passing the results back to the cpu. Similar to wgpu's hello-compute.

This was a first pass at the current implementation.
Note: I'm not a rendering expert so I'm not able to review any of the rendering specific code, I only reviewed the Rust/API code.

This is more than enough thank you very much! <3

The goal of this PR is not to add a high level API for GPU compute to bevy, rather to add a corner stone for compute, which will allow different people to think of high level API's that can be used to expose compute functionality to bevy users.

Do you think it makes sense to add a simple compute example at this stage, or is this still too low level? I'm thinking it could be super simple, like doing a few multiplications on the gpu, and passing the results back to the cpu. Similar to wgpu's hello-compute.

A lot of stuff has to get added to bevy in order for you to bring back uniform data to the CPU(RenderResource). Along with the need for some sort of global render resource. You also need a way to be able to block/wait for the GPU in bevy for some cases. Currently none of that is possible yet and I think a lot of those things can be a separate PR.

Closed because of the new pipelined renderer, this code change and PR have been moved to:
https://github.com/cart/bevy/pull/1


Return value from on_body ignored.

All callbacks check return value and set an error on non-zero except for on_body:
if (settings->on_body) settings->on_body(parser, p, to_read);
Oversight?  Seems like it should work similarly to the other callbacks and was perhaps missed because it bypasses the CALLBACK macros.

This will be fixed when #55 gets merged.


Unable to permanently delete file(s)/folder(s) when right-clicking them in the Explorer, holding Shift and clicking 'Delete'

Issue Type
Bug
Description
When right-clicking file(s)/folder(s) in the Explorer it used to be possible to hold Shift before clicking 'Delete' in the context menu to permanently delete them. This no longer works and it always prompts to move them to the Recycle Bin instead.
It now seems it is only possible to permanently delete file(s)/folder(s) by tapping them in the Explorer to highlight them and pressing Shift+Delete.
Would it be possible to restore the old behaviour?
VS Code Info
VS Code version: Code - Insiders 1.21.0-insider (91120eb786357aa1e16217ad6a67cf0740f26e8e, 2018-02-08T05:16:13.920Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

Memory (System)
15.93GB (7.96GB free)

Process Argv
D:\Programs\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

gitignore
cod
0.5.0

vue-peek
dar
1.0.2

vscode-eslint
dba
1.4.5

gitlens
eam
8.0.0

vscode-auto-open-markdown-preview
hnw
0.0.4

vue-snippets
hol
0.1.5

discord-vscode
icr
2.2.1

edge
luo
0.1.4

vetur
oct
0.11.7

vscode-icons
rob
7.19.0

sass-indented
rob
1.4.6

vue-vscode-snippets
sdr
1.3.0

code-settings-sync
Sha
2.8.7

pdf
tom
0.2.0

vscode-import-cost
wix
2.6.2

html-css-class-completion
Zig
1.16.1

(2 theme extensions excluded)

Reproduces without extensions

This is now possible by hodling the alt key before clicking on the context menu.
We should have put this in the release notes, my apologies for missing on that.
We will not bring back the old behavior with shift.

I can learn to live with this, but it took me a good couple of minutes to realise I had to let go of the Alt key after the context menu has appeared!
Out of curiosity, is there a particular reason why the old behaviour with Shift will not be brought back?
Holding Shift whilst clicking Delete in a context menu in order to permanently delete something is common practice in Windows - if not most OSes.
It seems strange to drop this in favour of the current solution (which most people will not even realise exists) but keep the standard "Press Shift+Delete to permanently delete selected files" functionality.

@isidorn - Note it says Shift+Del next to Delete Permanently in the context menu that appears when alt+right-click even though shift has nothing to do with it anymore.
Also, I am curious like @liam-potter about the reasoning for this alteration. Can you at least link to the issue where this was discussed so I can see the points made?

@liam-potter @cretz thanks for your interest. The reason why we started using the alt key for the click is because we have long ago defined that the alt key presents an alternative command in the context menu. More about that can be found here
The alt key is a shorthand for alternative, so that is why we choose that long ago for alternative contributable commands.
This milestone we have changed the explorer context menu to also use contributable commands and not treat any command specailly, so we simply changed the delete key to conform to our spec for commands.
As for the shortcut it still says Shift+Del, because that is still the keyboard shortcut for delete. The alt is only for the mouse click.

@isidorn - Thank you for your reply.
Personally I feel that that the VSCode team has it wrong on this occasion and that Delete Permanently should not be considered as an alternative command at all.
The main operation of the Delete command, with or without Shift, is to remove the selected item(s) from the folder within which they exist. That's it.
Yes, the default behaviour after this operation is to move the selected item(s) to Bin/Trash but this is why Windows, and probably most other OSes with contextual menus, use the Shift key as a modifier for the same Delete command - you're still just removing item(s) from a folder but modifying the way in which they are removed.
You are more than welcome to disagree with me, but I do think this is worth further discussion.

I understand the alternative actions reason from a point of view, however I don't think It justifies this change at all. If its about getting alternative options, why does the menu have the same entries except the delete one? Maybe replacing shift by alt on the old behavior would have more sense.
I'm Currently on Debian and using alt+right-click doesn't work but ctrl+alt+right-click does.
I miss the good old behavior

We brought back this behavior. SO try out the latest vscode stable and holding down shift and triggering the menu should offer the delete peremenantly action.

Thanks for your answer @isidorn. I tried in latest stable but new behavior still (Linux, .tar.gz):
Version 1.21.0
Commit 9a199d77c82fcb82f39c68bb33c614af01c111ba
Date 2018-03-07T11:01:43.521Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
Last stable version with previous behavior where delete permanently was triggered by having shift pressed before clicking the delete menu item, not before triggering menu:
Version 1.19.3
Commit 7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba
Date 2018-01-25T10:32:23.601Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

@arturock my mistake we brought it back only for windows.
@bpasero maybe we should also do this for linux since it also has some builtin alt behavior?
https://github.com/Microsoft/vscode/blob/master/src/vs/platform/actions/browser/menuItemActionItem.ts#L34

@isidorn sure why not

The behaviour is still a little buggy and not quite what I was expecting. If you're using Alt you have to let go of the key pretty quickly or the context menu disappears. If you already have a file right clicked and then try to do the "hold modifier key whilst right clicking" action on another file it often will not work (and accidentally right clicking the wrong file is a pretty common occurrence).
The problem I have with all of this is that it's taking me longer than it really should to do a very simple operation. VSCode used to do this perfectly and replicate default OS behaviour - and it has not yet been confirmed where it was discussed that the Delete command should be changed to this new behaviour.
The default behaviour in almost every OS/software I am aware of is to treat Delete as a single command but allow a modifier key (usually Shift) to modify the way in which that operation is performed.

You do not need to hold any modifier keys whilst right-clicking the files/folders (only whilst clicking the Delete command itself in the context menu)
The Delete command is treated as a single command since both soft/hard deletes have the same end result - the items are removed from whatever folder they are in
Clicking Delete without holding any modifier keys should prompt the user to send the items to trash/recycle bin.
Clicking Delete whilst holding the Shift key should prompt the user to remove the items permanently.

It feels very strange that VSCode is breaking this trend.

No problem @isidorn. May I suggest to return to the old behavior just like it was?

We can not return to the old behavior due to the reason specified in my comment above. Thanks for your comments and suggestions though.

Also we will probably improve this based on your suggestion once we have async context menus -> which require adoption to electron 2.0. For that you can track our work on the tyriar/electron-2.0.x branch. Thanks

Just to note It will be kinda weird the inconsistency between shift+del on keyboard and alt+delete with menu

Now the menu supports both shift and alt.


Update README page

Add a UI mockup of your intended final product.
Update all contents to match your own project.
Update the link of the GitHub Actions build status badge (Build Status) so that it reflects the build status of your team repo.
Acknowledge the original source of the code

Updated the README


feat(qrm): support shared_cores with numa_binding

feat(qrm): support shared_cores with numa_binding

dedicated_cores with numa binding的pod,在计算hint的时候，允许调度到shared_cores  numabinding 的numa node上么


"Cannot Create Raw Socket" How to fix?

When using some Layer4 methods, "Cannot Create Raw Socket" appears, what is the problem?

run using sudo (this method only with spoofable servers)

run using sudo (this method only with spoofable servers)

I have the same problem,but run sudo command still doesn't work

vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS> pip3 install pyroxy
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: pyroxy in /home/vadiki/.local/lib/python3.8/site-packages (1.0b4)
Requirement already satisfied: maxminddb>=2.2.0 in /home/vadiki/.local/lib/python3.8/site-packages (from pyroxy) (2.2.0)
Requirement already satisfied: pysocks>=1.7.1 in /usr/lib/python3.8/site-packages (from pyroxy) (1.7.1)
Requirement already satisfied: requests>=2.27.1 in /usr/lib/python3.8/site-packages (from pyroxy) (2.27.1)
Requirement already satisfied: yarl>=1.7.2 in /home/vadiki/.local/lib/python3.8/site-packages (from pyroxy) (1.7.2)
Requirement already satisfied: certifi>=2017.4.17 in /usr/lib/python3.8/site-packages (from requests>=2.27.1->pyroxy) (2021.10.8)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/lib/python3.8/site-packages (from requests>=2.27.1->pyroxy) (1.26.7)
Requirement already satisfied: charset_normalizer~=2.0.0 in /usr/lib/python3.8/site-packages (from requests>=2.27.1->pyroxy) (2.0.12)
Requirement already satisfied: idna<4,>=2.5 in /usr/lib/python3.8/site-packages (from requests>=2.27.1->pyroxy) (3.3)
Requirement already satisfied: multidict>=4.0 in /home/vadiki/.local/lib/python3.8/site-packages (from yarl>=1.7.2->pyroxy) (6.0.2)
vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS> sudo pip3 install pyroxy
[sudo] пароль для root:
Requirement already satisfied: pyroxy in /usr/local/lib/python3.8/site-packages (0.1)
Requirement already satisfied: bottle in /usr/local/lib/python3.8/site-packages (from pyroxy) (0.12.19)
Requirement already satisfied: lxml in /usr/local/lib64/python3.8/site-packages (from pyroxy) (4.8.0)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS> sudo python3 start.py dns <IP_ADDRESS>:53 256 600 resolvers.txt
Traceback (most recent call last):
File "start.py", line 27, in
from PyRoxy import Proxy, ProxyChecker, ProxyType, ProxyUtiles
ModuleNotFoundError: No module named 'PyRoxy'
vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS> python3 --version
Python 3.8.12
vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS> python3 start.py dns <IP_ADDRESS>:53 256 600 resolvers.txt
[16:45:57 - ERROR] Cannot Create Raw Socket
vadiki@localhost:/run/media/vadiki/D_238_GB_WD/Кибервойска/DDoS/MHDDoS_proxy/MHDDoS>


Update service-fabric-reliable-actors-timers-reminders.md

Simple fix to first sentence.

Hi @bcraun, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@bcraun Thanks for the contribution!


Add option to enable/disable animation of Collapse on a per-show/hide basis

See #15747 for the original feature request.
What we want: An additional JavaScript option of $(e).collapse() that let us disable the animation.
My proposal is that such option would be named animate, and has a default value of true.
Example usage:
// Collapses the element without any animation.
$(element).collapse({
  toggle: true,
  animate: false
});

// Collapses the element with animation, as animate is true per default.
$(element).collapse({
  toggle: true
});

There are probably other Bootstrap APIs that could also benefit by having such option.
If this sounds good, I could look into creating a PR.

For clarity, you mean animate should be a parameter of the .collapse('show')/.collapse('hide')/.collapse('toggle') methods, right?

Yes
On Mon, 2 Nov 2015 at 10:50, Chris Rebert<EMAIL_ADDRESS>wrote:

For clarity, you mean animate should be a parameter of the
.collapse('show')/.collapse('hide')/.collapse('toggle') methods, right?
—
Reply to this email directly or view it on GitHub
https://github.com/twbs/bootstrap/issues/18127#issuecomment-152972631.

So do you think this is a good approach? Shall I create a PR?

Yeah, sure. Note that this will require adding visual tests (which are run interactively in real browsers and require manual verification by humans) since it's transition-related.
Also, I'd recommend only tackling the Collapse plugin for now, so that any code style or test-related kinks can get worked out first.

Sure, sounds good.

So I just realised that I have misunderstood the API a bit. I thought that invoking $(e).collapse(config) updates the current config of the element. This seems to not be the case.
I've created a branch that is able to disable/enable animations, but it is currently not possible to change the behaviour of an already inited element. I think that a logical next step would be to read the config of an element per invocation so that changed attributes or collapse(config) calls are considered.
The other approach would be having an optional config object parameter along with the hide/show method calls, such as $(e).collapse(config, 'hide') as we previously discussed. But I feel like the first approach might be desired.
Shall I submit the PR with the animation option and open a separate issue for updating the config of an element, or shall it also be done in this PR?

Separate PRs please.

Another optimizaiton is if the element in start position and in end position are both not in viewport, then show/hide right away, not use animation, as user won't see it.

This PR #21589 might be a solution for this requirement. It allows to adapt the animation duration based on the css value rather than a static value as it is currently.
Then in order to show/hide without an animation, we can define
.no-transition {
    transition: none !important;
}

Then either set the no-transition class directly in the markup for the collapse or
$('.collapse').addClass('no-transition').collapse('toggle')

Closing per #25662
demo: https://codepen.io/anon/pen/KLPMrQ

.no-transition {
transition: none !important;
animation: none !important; /* This is required for my solution, too */
}


incorrect result

result are inaccurate...even after using 2 image of same person it gives me an error that they are not same. and sometime 2 different person face is shown as a match.

Hi @bajjajjrajjesh
I'm having a similar issue. I've provided a fix where the developer is provided not only with a "match found" or "match not found" value, but also with a matching coefficient that can then be used to further limit accepted values, making sure a match is indeed found.

Also pls take a look on dlib ml model specification and pros/cons


DBZ-6684 Rename back fields with avro_unicode

https://issues.redhat.com/browse/DBZ-6684

closed with some unexpected git issue


Zappi/Eddi/Hub firmware versions in sensors

Is your feature request related to a problem? Please describe.
There is no push notification when new Zappi/Eddi/Hub firmware versions are available.
Describe the solution you'd like
I would like a sensor (or attribute on another sensor) for each myenergi device that provides the installed firmware version, the serial number, and if possible the latest firmware version so I can setup HA alerts.
Alternatively, this could be integrated with the Home Assistant updates for devices - https://www.home-assistant.io/integrations/update/
Describe alternatives you've considered
I periodically look at either the device display or https://myaccount.myenergi.com/location#products and then compare this to the table in https://support.myenergi.com/hc/en-gb/articles/360020109017-Current-firmware-versions-updating-your-firmware
Additional context
The myenergi RESTI API detailed at https://github.com/twonk/MyEnergi-App-Api shows:
"fwv" = firmware version
"sno" = serial number
We could raise an issue on the myenergi forums to add an API to return the latest firmware versions.

+1 This would be a great addition

Could we use the newAppAvailable and/or new BootloaderAvailable boolean properties?
https://s18.myenergi.net/cgi-jstatus-*
[{"eddi":[{"sno":SERIAL_NO,"dat":"01-02-2023","tim":"23:16:45","ectt1":"Internal Load","ectt2":"Grid","ectt3":"None","bsm":0,"bst":0,"cmt":254,"dst":1,"div":0,"frq":50.04,**"fwv":"3202S4.097"**,"gen":-12,"grd":-6,"pha":1,"pri":1,"sta":1,"tz":0,"vol":2411,"hpri":1,"hno":1,"ht1":"Tank 1","ht2":"None","r1a":0,"r2a":0,"rbc":0,"tp1":127,"tp2":127,"newAppAvailable":false,"newBootloaderAvailable":false,"batteryDischargeEnabled":false}]},{"zappi":[{"sno":SERIAL_NO,"dat":"01-02-2023","tim":"23:16:45","ectp2":1,"ectp3":-12,"ectt1":"Internal Load","ectt2":"Grid","ectt3":"Generation","bsm":0,"bst":0,"cmt":254,"dst":1,"div":0,"frq":50.04,**"fwv":"3562S4.525"**,"gen":-12,"grd":-6,"pha":1,"pri":2,"sta":1,"tz":0,"vol":2417,"bss":0,"lck":16,"pst":"B1","zmo":3,"pwm":1200,"zs":1026,"rdc":1,"rac":4,"zsh":4,"zsl":2,"ectp4":318,"ectt4":"AC Battery","ectt5":"None","ectt6":"None","newAppAvailable":false,"newBootloaderAvailable":false,"beingTamperedWith":false,"batteryDischargeEnabled":false,"mgl":100,"sbh":17,"sbk":5}]},{"harvi":[]},{"asn":"s18.myenergi.net","fwv":"3401S4.525","vhub":1}]


Add events timeline for containers

@miq-bot add_label providers/containers

@miq-bot add_label enhancement

@miq-bot add_label wip

@miq-bot add_label ui

Current status: Conteiner timeline events are successfully presented! (with the help of @zeari).
2 Issues that needs to be solved:

when clicking on another container in the explorer tree the page (right cell) isn't changed except for the title (the container's name).
back button doesn't work, relates to the above.

@h-kataria can you help?
@simon3z

@dkorn you will need to make appropriate change in replace_right_cell method as well to render correct partial for timelines, see https://github.com/ManageIQ/manageiq/blob/master/app/controllers/vm_common.rb#L1435 for example

@dkorn i tried to run your PR and getting an error when i try to go to Timeline for a container
[----] I, [2015-11-12T22:08:49.649964 #11906:73b98c]  INFO -- : Completed 200 OK in 160ms (Views: 0.1ms | ActiveRecord: 16.6ms)
[----] I, [2015-11-12T22:08:54.215140 #11906:73b98c]  INFO -- : Started GET "/container/explorer/10000000000006?display=timeline" for <IP_ADDRESS> at 2015-11-12 22:08:54 -0500
[----] I, [2015-11-12T22:08:54.255541 #11906:73b98c]  INFO -- : Processing by ContainerController#explorer as HTML
[----] I, [2015-11-12T22:08:54.255616 #11906:73b98c]  INFO -- :   Parameters: {"display"=>"timeline", "id"=>"10000000000006"}
[----] F, [2015-11-12T22:08:54.276311 #11906:73b98c] FATAL -- : Error caught: [NoMethodError] undefined method name' for nil:NilClass /home/hkataria/dev/manageiq/app/models/container.rb:34:in event_where_clause'
/home/hkataria/dev/manageiq/app/models/mixins/event_mixin.rb:36:in find_one_event' /home/hkataria/dev/manageiq/app/models/mixins/event_mixin.rb:4:in first_event'
/home/hkataria/dev/manageiq/app/models/mixins/event_mixin.rb:14:in first_and_last_event' /home/hkataria/dev/manageiq/app/controllers/application_controller/timelines.rb:287:in tl_build_init_options'
/home/hkataria/dev/manageiq/app/controllers/application_controller/timelines.rb:458:in tl_build_timeline' /home/hkataria/dev/manageiq/app/controllers/container_controller.rb:68:in show_timeline'
/home/hkataria/dev/manageiq/app/controllers/container_controller.rb:82:in `explorer'
do you see the same errors in the log?

@dkorn i tried to run your PR and getting an error when i try to go to Timeline for a container

@h-kataria where did you get the container entities from? I guess your error comes from a nil container_project in:
["container_namespace = ? AND #{events_table_name(assoc)}.ems_id = ? AND container_name = ?",
 container_project.name, ext_management_system.id, name]

but every  container belongs to a container_group and a container_project.

@dkorn I tested this (on master with hash aa18d85) and it works for me. Please remove all the wip names and labels. Let's merge.
LGTM :+1: cc @chessbyte @blomquisg ready for merge (when wip will be removed).

@miq-bot remove_label wip

@simon3z @h-kataria please review

LGTM :+1: cc @h-kataria @chessbyte

looks good.

@miq-bot add_label backport/yes


Node 6.x.x fs.extname(undefined) call throws exception

The new Node 6.x.x throws an exception when you call fs.extname with an undefined argument.
(See https://nodejs.org/en/blog/release/v6.0.0/)

Hi @tomymolina
Thank you for PR
I found that the same problem exist in other places, so I fixed all of them in #71

Issue is fixed in version 1.0.3


fix cargo check config with custom command

fixes #3871
Previously if get::<Vec<String>>(value, "/checkOnSave/overrideCommand") returned Some we'd never execute set(value, "/checkOnSave/command", command), even if the overrideCommand was empty.
I am not sure of the best way to prove this, but I believe the LSP clients send this config with a default value if it is not set by the user, which means get::<Vec<String>>(value, "/checkOnSave/overrideCommand") would return Some(vec![]) and thus we'd never set the command to the user specified value (in the case of #3871, "clippy").
I have tested this fix manually by installing this modified version of rust-analyzer and verifying I can see clippy lints in my editor (coc.nvim) with rust-analyzer.checkOnSave.command": "clippy".
As best I can tell this would have affected rustfmt extra args too, so this PR also applies the same fix there.

Thanks!
bors r+


not for merge: test if ofborg tests passthru.tests

please ignore :)

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Bear in mind that this commit doesn't follow the CONTRIBUTING.md commit naming scheme, so ofborg won't actually start a build for this package.
IIRC the logic works something like if the eval-check-meta stage passes, then any packages which were changed in that and have a corresponding commit will have ${pkg} and ${pkg}.passthru.tests built for them.
(See e.g. https://github.com/NixOS/nixpkgs/pull/119781, where you can see that veracrypt, veracrypt.passthru.tests got triggered)

Bear in mind that this commit doesn't follow the CONTRIBUTING.md commit naming scheme, so ofborg won't actually start a build for this package.

TIL, thanks!

IIRC the logic works something like if the eval-check-meta stage passes, then any packages which were changed in that and have a corresponding commit will have ${pkg} and ${pkg}.passthru.tests built for them.
(See e.g. #119781, where you can see that veracrypt, veracrypt.passthru.tests got triggered)

Gotcha, that is enough validation for me I think :laughing:


How to change the borderColor ?

Hi , I want to change the bordercolor to whiteColor ,but it dosent work , and still show with tintcolor ! How can I solve it?

@JavenLeo
The following example code, it's work!
https://github.com/UrbanApps/UAProgressView/blob/2455af67bb3d8776ace7c6a782fd9646c9ae5e3e/UAProgressView-Example/UAProgressView-Example/UABasicExampleViewController.m#L46
Maybe, now borderColor and tintColor is the same, but you need to set the difference color the each?

refs #18 #19


Restructure how data is passed around

Currently have a mess of methods chained together passing values around and it's only going to get worse.
Consider fixing it with additional structs to deal with time stamps and any other thing needed to carry out its function.

Instead of using structs, simplified the code by refactoring.


build of h5reader.c fails when compiling against hdf5-openmpi 1.12.0

I have been trying to set up a package of pynio on archlinux with the following package versions:
pynio 1.5.5
python 3.9.1-2
ncl 6.6.2-1
hdf-eos2 2.20-1
hdf-eos5 1.16-2
hdf4 4.2.15-1
hdf5-openmpi 1.12.0-2
Trying to build  pynio I encountered the following errors:
libsrc/h5reader.c:42:14: error: too few arguments to function ‘H5Oget_info_by_name3’
   42 |     status = H5Oget_info_by_name(fid, root_name, &oi, H5P_DEFAULT);

libsrc/h5reader.c:2491:83: error: ‘H5O_info2_t’ has no member named ‘addr’
 2491 |             if(NULL == (already_visited = _NclHDF5_addr_visited(udata->seen, oinfo.addr)))

libsrc/h5reader.c:2693:12: error: too few arguments to function ‘H5Oget_info3’
 2693 |         if(H5Oget_info(type, &oi) >= 0)

libsrc/h5reader.c:4486:8: error: too few arguments to function ‘H5Oget_info_by_name3’
 4486 |     if(H5Oget_info_by_name(fid, grp_name, &oinfo, H5P_DEFAULT) < 0)

and similar repetitions.  This looks to me like a version incompatibility or a bug in pynio's h5reader.c. All of the errors point to a discrepancy to the header files associated to my hdf5 installation (with openmpi support). Has anyone else encountered this problem?
Any information on a fix would be very much appreciated.

Take a look here https://www.hdfgroup.org/downloads/hdf5/
I've managed to build the package pynio on conda-forge by adding -DH5_USE_110_API

Great find @fulminemizzega ! Thanks for pointing this out. That actually means that this is an out-of-date issue for a dependency of pynio as one is required to use the legacy extension of HDF 5, version 1.10 (as opposed to the now up-to-date version 1.12).


可以配置代理吗

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

终端的HTTP_PROXY和HTTPS_PROXY有效果吗？


Huge issue: Masses of guardians spawning in ocean biomes for no reason

I have the issue that thousands of guardians keep spawning in my ocean biome.
I don't know why they spawn there, but it happens.
Here is my ocean biome config file:
http://hastebin.com/inuxevafun.coffee
I can't see anything wrong with it.
I am on Spigot 1.9 and TerrainControl v2.8.1-SNAPSHOT.
Any ideas?

Are you on Vanilla or modded? Cause Quark makes guardians spawn randomly in the ocean.

What is Quark?
I am on Vanilla.


Quickstart.UI signout-redirect.js does not need load event

Recent change in signout-redirect.js file in Quickstart.UI replaces window.addEventListener() with $() [1]. I think none is necessary actually.
As far as I see, in all scenarios the script is included after the relevant anchor element in the DOM. Therefore it's safe to just query for the element. In fact this is pretty much exactly what is done in the sibling file signin-redirect.js [2].
[1] https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/commit/bedfed33462f4223bee4a0519c361c59aa8d3753
[2] https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/blob/bedfed33462f4223bee4a0519c361c59aa8d3753/wwwroot/js/signin-redirect.js

It needs to run after all the iframes are done running. That's wh originally it was running in the load event, but IE that runs before iframes are done, so jQuery's document ready event normalizes it across browsers and runs after the iframes are done.

I see. I did not realize this was about iframes.
However, if that's the case then jQuery document ready handler is not appropriate, because it runs when "DOM is ready" which is earlier than "window is loaded". jQuery documentation hints at the difference (https://api.jquery.com/ready/):

Browsers also provide the load event on the window object. When this event fires it indicates that all assets on the page have loaded, including images.

and also (https://learn.jquery.com/using-jquery-core/document-ready/)

Code included inside $( document ).ready() will only run once the page Document Object Model (DOM) is ready for JavaScript code to execute. Code included inside $( window ).on( "load", function() { ... }) will run once the entire page (images or iframes), not just the DOM, is ready.

If the change was due to IE7 not supporting addEventListener then maybe the old attachEvent will work instead? https://stackoverflow.com/a/6927800

Yep, absolutely right. This is a bug.

This change seems to correct the problem:
$("iframe.signout").load(function () {
    var a = document.querySelector("a.PostLogoutRedirectUri");
    if (a) {
        window.location = a.href;
    }
});

thoughts?

I was kind of hoping that you'd drop jQuery from here, because that is the only place it is used in the project and that would open the possibility of dropping it from the project altogether (I know that bootstrap.js depends on jQuery, but as far as I see it isn't used either).
When it comes to the code, I'd change load(handler) to on('load', handler). The former doesn't exist in jQuery 3+.
Also , I'd say that if you use jQuery then let's use it, so:

the whole thing should probably be in a ready handler (I realize I said it wasn't needed, but if jQuery remains, then it's the appropriate solution)
replace document.querySelector with $(), for consistency

Feel free to close the issue.

Well, as you saw it used to be the window.load event, but I changed it since I had customers saying load doesn't work properly in IE 11 (We don't care about older IE). I (incorrectly) assumed jQuery's ready event was better. So what native even works, do you suggest?

I don't know why I thought the change was triggered by having to work in IE7. My earlier mention of attachEvent is irrelevant then, I think. Sorry for the confusion.
I just realized that the code above might not be correct. Looking at the code of LoggedOut view, the iframe and the link are both optional and independent of each other. So the redirect can't depend only on the iframe load, because that iframe might not be present on the page at all.

Ok, so seems the load event is the right one (https://developer.mozilla.org/en-US/docs/Web/API/Window/DOMContentLoaded_event), and was what I originally had. Le sigh. So whatever -- screw IE11 then.

PR merged, thanks for the report on this.


[20.x] Upgrade jboss-ejb-client to allow running on Java 17

Wildfly 20.x currently ships with a version of jboss-ejb-client which breaks the stricter encapsulation rules introduced in Java 17.
jboss-ejb-client has fixed this issue in later versions, so it would be good to update Wildfly 20.x to use version which works on Java 17 and later.

Hi @eirbjo -- Thanks for the report. If @chengfang and @tadamski are ok with this change to the 20.x branch I'm fine with merging it, but please note that we are not going to produce any further releases in the WildFly 20 series. We don't even have CI for that branch.

Apart of lack of the release, my understanding is that there may be a number of other components requiring modifications to work with jdk17.

My work here was mainly to assess the impact of running a Java EE 8 implementation on a JRE where the deprecated java.security.Identity class and some related classes are removed. This removal is an ongoing discussion in the OpenJDK project: https://bugs.openjdk.org/browse/JDK-8191136
With the change in this PR, I was able to build Wildfly 20 and run it on my custom Java 21 build where the APIs are removed.
So the modified Wildfly has been tested to run on Java 21. I used the ejb-security-context-propagation quickstart project to verify this:
https://github.com/wildfly/quickstart/tree/20.x/ejb-security-context-propagation
So while I did not run a full test of all features, at least this quickstart had no problems running on 21.

diff: https://github.com/wildfly/jboss-ejb-client/compare/4.0.33.Final...4.0.52.Final
There are many changes in the diff and they haven't been tested with WildFly 20.x. But if the new version works for your purpose, I have no objection.

Thanks @eirbjo, @chengfang and @tadamski


Add 'emulators' subcommand to flutter-tizen

This pr adds flutter-tizen emulators subcommand feature.

flutter-tizen emulators : lists created emulators.
flutter-tizen emulators --launch <name>: launches one of the created emulators with the name <name>. <name> can also be specified as a prefix of one of the emulators.
flutter-tizen emulators --create --name <name>: creates a new TV or wearable emulator. If both images are installed, TV profiles have priority over wearable ones. If multiple platform versions of the same profile are installed, the one with the highest version will be chosen.

Limitation

flutter-tizen emulators -h help message currently describes about creating Android emulators.
Android Studio(AVD Manager) must be installed for the command to work.

Both limitations can be resolved by creating the TizenEmulatorCommand class that inherits the FlutterCommand. This should be revisited when its necessary.

@swift-kim
Thanks for the thoughtful review. I've updated the code based on the suggested improvements. A summary of the changes is listed below:

Add ','s for better formatting.
Change to sync APIs for simple file operations; existsSync, listSync, readAsLinesSync.
Add sdkDataDirectory getter in TizenSdk class.
Move parseIniLines function to tizen_sdk.dart file and reuse the function to parse sdk.info file.
The parseIniLines function now splits at the first occurrence of the equal sign(=).
The canLaunchAnything and canListAnything getters now evaluate the bool value by checking the existence of the em-cli executable.
Use canLaunchAnything and canListAnything getters in null-checking.
Handle exitcode from the process that launched the emulator.

Please let me know if I've misunderstood any part of the review, or if further updates are needed.
Commenting on the side note:
I agree that flutter-tizen emulators command should eventually work on handling Android emulators. Currently the command can list and launch Android emulators, but can't create them. This is mostly due to the lack of option/flag settings in the emulator command. I'll also share any idea if it pops up.


Add onMoveFloatingView(x,y)

It fires an event when the floatingView is moved, so we can save the last position of the view

Thanks @CoderProject.
The Pull Requests merged into Ver 2.1 (with improvement)


iTunes crash on DRM protected films

Testing macOS High Sierra with Shiki 2.1.0, Lilu 1.2.0, NvidiaGraphicsFixup 1.2.0
boot argument shikigva=12 to enable hardware encoding
I experience crash while trying to read a film on iTunes. Attached is crashlog.
iTunes_2017-10-12-043311_iMac.crash.zip
Other configuration details can be seen here.

Hi. Not much we could do here, Apple's bug unfortunately. You could try standard ways mentioned in the FAQ, but I don't think it will help. See https://github.com/vit9696/Lilu/issues/20.

FYI, I was also seeing this, but repairing permissions as stated here fixed it, although not entirely sure why: https://support.apple.com/en-us/HT203538


[docs] Automated release notes do not respect release.yml changes

Automated release notes do not respect release.yml changes
When attempting to generate release notes for Flutter 3.17 beta, it was discovered that there was a missing labels parameter in release.yml.  Though, with changes the automated release notes do not account for the the structure determined by release.yml.

Will pushing new tags kick off any CI changes? @godofredoc

Yes. It'll run the entire devicelab suite, so ideally we'd want to land these CPs outside 10 AM - 4 PM PST.

Are there concerns with this approach? @CaseyHillers

I don't have large concerns. This might be the easiest path forward, and I suspect in a few months the release.yml will have stabilized.
I do wonder if there's a way we can explicitly tell it to use the release.yml from HEAD instead of the tag.

Update:  Release configuration is generated at the end tag, not the beginning tag.
new tag name for the 3.17 beta notes is 3.17.0-0.1.pre-rev2


teach In for node-red nodes

we can not write to node properties from running nodes. we might be able to use node.context for automatic teach in.
https://nodered.org/docs/user-guide/context
but some user settings are required for that to work as intended. (set contextStorage to localfilesystem) i will have to experiment

fixed by pr #89


is_cpu_supported() added

call this once upon entering main() , if it returns false
there is no point of proceeding

Looks good: we shall need to fix the amalgamation.

@lemire I have gone much further than "amalgamation".  I have added simdjson_nanolib.h and I have done some changes in simdjson.h ... CI is failing saying that  simdjson_nanolib.h is not found ...

removed (temporarily) the separate simdjson_nanolib.h
going to close this one and open a new PR


Float division is no longer IEEE 754 after njit

After applying numba.njit to a function, float division behavior is no longer that of NumPy, but the one of Python:
# coding: utf-8
import numpy as np
from numba import njit

def func_ok(a, b):
    c = np.sin(a * b)
    return np.divide(a - b, c)

numba_func_ok = njit(func_ok)

def func_fail(a, b):
    c = np.sin(a * b)
    return (a - b) / c

numba_func_fail = njit(func_fail)

if __name__ == '__main__':
    print(func_ok(0., 0.))  # nan
    print(numba_func_ok(0., 0.))  # nan
    print(func_fail(0., 0.))  # nan
    print(numba_func_fail(0., 0.))  # ZeroDivisionError

My workaround is to use numpy.divide, which luckily is supported in nopython mode.

Unfortunately we can't implement both semantics at the same time. We chose to implement the Python semantics over scalars and the Numpy semantics over Numpy arrays.

We could provide a mechanism to disable raising zero division errors.  This would also have a performance benefit for those who are willing to deal with the NaN.

this is now available with the error_model="python" or error_model="numpy" argument to @jit


Unable to access exported variables from imported http file

I'm trying to access variables exported from the imported file, without success.
Sample
singup.http
POST /signup HTTP/1.1
...headers

{
  "var1": "var_value"
}

{{
exports.varFromSignup = "testValue"
}}

init.http
# @import ./signup.http

###
{{
  console.info(varFromSignup) // This line throws error
}}
GET /users/me HTTP/1.1
...headers

console statement throws error.
Is this expected behaviour? If so, is there a way I can access exported variables in another file?
Thank you

Only the global variables are imported. In this case varFromSignup is not global, because this variable is assigned to the request /signup.

You need to use @Ref or change variable to a global variable

Would be useful if you can provide an example with httpbin. Will try it on my own.

In connection with @import/ @ref I have an error, that if the @ref is not present, it comes to an endless loop. I wanted to fix this with<EMAIL_ADDRESS>but it was insufficient. This leads to a crash because of stackoverflow. Maybe this was your problem.


Update messenger extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

There seems to be a similar extension Messages already available in the Store

There seems to be a similar extension Messages already available in the Store

My extension is based on that extension. It has added support for arguments (contact name and message text) to be specified in the raycast search box and I have also added another command to silently deliver messages as well.


[releases] Info about third-party-components were added to licence file and repo

Description
Cherry pick from #302

make test


Stripe webhooks express middleware implementation in hono

I'm working on a stripe payment backend in hono. Stripe hooks needs a middleware to work properly. This code is for express.

  app.use((req, res, next) => {
    if (req.originalUrl === "/api/v1/webhook") {
      next();
    } else {
      express.json()(req, res, next);
    }
  });

How I implement similar middleware in hono?

cc: @hideokamoto

Hi @shahreazebd .
Do you mean that you want to get to know about how to get the original url or path data on the Hono like the express one?
Or, do you want to know about how to verify the Stripe Webhook API request on the Hono?
Thanks.

I just want  to know about how to verify the Stripe Webhook API request on the Hono?

@shahreazebd
We have a template to use Hono and Stripe API / Webhook:
https://github.com/stripe-samples/stripe-node-cloudflare-worker-template
You can learn how this example verify the Stripe Webhook API request on the Hono from this file:
https://github.com/stripe-samples/stripe-node-cloudflare-worker-template/blob/main/src/index.js#L44-L73
Thanks.

@hideokamoto-stripe
Not sure if the example you provided works anymore...
Here's my code

And when I send a test webhook via the Stripe CLI, it is unable to find the signature from the body.

I'm using Bun + Hono for reference. LMK if I'm missing something, but as far as I know, all the logic is matching the example yet it doesn't work.

Thanks for reaching out @dickeyy .
Just for confirm, is your Hono app running on Cloudflare Workers or Wrangler?
Or using this for the another runtime like Node.js or AWS Lambda like that?
Thanks.

@hideokamoto-stripe same problem with Wrangler on my end

i just tried to deploy this without any change:
https://github.com/stripe-samples/stripe-node-cloudflare-worker-template/blob/main/src/index.js
on wrangler and signature fail.

Ok forgot what i said i was using the prod key with a test event

@dickeyy
Thank you for the details. The previous sample I mentioned was intended for Cloudflare Workers. Since you're utilizing Bun, it may be necessary to switch to the following sample code:
https://docs.stripe.com/webhooks/quickstart?lang=node
I haven't had experience with Bun, so I'm currently investigating the phenomenon you've reported. If the Node sample code provided in the documentation doesn't work for you, I would appreciate it if you could share any error messages or additional details.


Errors with Generics - missing trait bounds.

During cleaning up some of the error handling in paritytech/substrate I encountered that using #[error("yada {0:?}")] for something along the lines of struct Erroris<T>(pub Wrapper<T>) does not enforce a T: Debug bound on the underlying generic type, but requires adding a bound on the struct declaration/definition rather than the generated fmt::Debug impl. That goes against what is stated as a best practice in the future proofing guide https://rust-lang.github.io/api-guidelines/future-proofing.html?#data-structures-do-not-duplicate-derived-trait-bounds-c-struct-bounds .
https://github.com/paritytech/substrate/pull/7631#discussion_r532700450
It would be awesome if this could be addressed :heart:

Probably a duplicate of: https://github.com/dtolnay/thiserror/issues/79

Yeah this is tracked in #79, and is just waiting on a PR.


rowma_ros: 0.0.2-2 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository rowma_ros to 0.0.2-2:

upstream repository: https://github.com/rowma/rowma_ros
release repository: https://github.com/rowma/rowma_ros.git
distro file: kinetic/distribution.yaml
bloom version: 0.9.3
previous version for package: null

rowma_ros

No changes

Closing in favor of #24453


Add ability to append to lists

It'd be good if at the falco rules level you could define a list in a secondary rules file which is automatically appended to a list with a given name in an earlier file, for example:
- list: shell_binaries
  items: [bash, csh, ksh, sh, tcsh, zsh, dash]

- list: user_shell_binaries
  append_to: shell_binaries
  items: [xcsh]

It also might be desirable to have the new list replace the existing list rather than define a list with a different name, for example:
- list: shell_binaries
  append_to: shell_binaries
  items: [xcsh]

I'd prefer the second option

For the second option, is the semantics then if "append_to" not present, its a replace? in which case why not make it a boolean with a default false?
- list: shell_binaries   append: yes   items: [xcsh]
Otherwise the repeating of the name on the append is awkward; and makes the first option a bit more comprehendible as to whats going on.

@argh128 agreed that it'd be nice to have a compact syntax for "append to this list and then replace it". And now that I think about it, there's already an easy way to append to an existing list and give it a new name:
- list: user_shell_binaries
  items: [shell_binaries, xcsh]

So sounds like the best approach will be that list elements will have an optional append boolean, defaulting to false. If true, it appends to a previously found list with the same name (and probably errors if no previously found list exists).

Btw, I added some wiki docs for this feature: https://github.com/draios/falco/wiki/Falco-Rules#appending-to-lists.

I am trying to do this now but I keep getting a Undefined macro kk_acme used in filter.. Exiting.
I am trying to append to the write_etc_common macro in my local.yaml file, like so:
 - macro: kk_acme
   condition: >
     user.name=root and
     (proc.name="acme.sh" or
     proc.pname="acme.sh")

 # Append to /etc write macro
 - macro: write_etc_common
   append: true
   condition: >
     and not kk_acme

Any ideas?

However, it seems to work if I put everything from my kk_acme macro's condition and use as a condition in the appended write_etc_common macro:
- macro: write_etc_common
  append: true
  condition: >
    and not (user.name=root and (proc.name="acme.sh" or proc.pname="acme.sh"))

But that kind of defeats the purpose of the macros and reusable config code though.

Ok, I'll reopen this and double-check that you can append to macros properly.


Adding support for Column changes on Tasks

About
This PR adds the capability to move tasks across Columns in Project during the different stages of a PR.
How
I'm introducing three new inputs: BOARD_COLUMN_OPENED, BOARD_COLUMN_MERGED and BOARD_COLUMN_CLOSED. The tasks would be moved to the specified columns as the PR is opened, merged or closed (respectively).
On our workflow this would be extremely helpful, since we use the Board view in Teamwork to visually see what's the status on each task, we would start the task by moving it to "In Development", then once a PR is open we move it to "Has PR" and finally once the PR is merged the task is ready to be tested and moved to "Nightly". Optionally we may move the task back to "Selected For Development" in case the PR was rejected and closed.

It'd be nice to have this released soon.

Closing in favour of https://github.com/Teamwork/github-sync/pull/45


CD-352687 Add a logging thread and pipe for logging after signal hand…

…ling
https://coupadev.atlassian.net/browse/CD-352687
Resque has this issue that is still not fixed https://github.com/resque/resque/issues/1493 where after trapping a signal (TERM, INT, etc.), it writes logs and results in log writing failed. can't be called from trap context.
Used suggestion from https://github.com/resque/resque/issues/1493#issuecomment-358711299 to make a thread and a pipe to write logs after signal trapping.

[ ] @abdulchaudhrycoupa
[ ] @revathi-murali
CC: @johnny-lai

Generated Reviewers

:warning: Do not modify anything in this section! :warning:
The content in this section is managed automatically, and any manual changes to it will be ignored and overwritten the next time the code review status is refreshed! View this PR on Cody to see the current code review status.

[lockit v5.0.2] merged via Lockit CLI with --skip-pending-status on 2023-07-03 12:07:24.122912 +0800 CST m=+7.059329112


dimensions-of-terrain

@rultor release, tag is 0.0.39

@rultor release, tag is 0.0.39

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.39

@yegor256 Done! FYI, the full log is here (took me 10min)


Support PostgreSQL as a session store

Hi! Are there any plans for supporting postgres as a session store database? Thank you.

@srstsavage Thanks for the suggestion, I'm going to try and add that support.
Do you mean only for the session state, or for the main DB too?

@srstsavage support for postgres and microsoft sql server has been added in latest v1.1 release.

Awesome, thank you!


createOpenFetchClient ignores $fetch

createUseOpenFetchClient ignores $fetch.
Reproduction:
const { data } = await useClientFetch("/user/profile/", {
  method: "post",
  $fetch: globalThis.$fetch.create({
    onRequest({ options }) {
      console.log("Making request")
      options.headers = {
        ...options.headers,
        authorization: `Bearer: ${token.value}`,
      },
    },
  }),
})

Expected: onRequest called.
Actual result: onRequest never called.
This is because even thought UseFetchOptions advertises that it accepts $fetch as an option:
https://github.com/enkot/nuxt-open-fetch/blob/abdfdf447a3a7c2fb24447518c1b9908ca29a4ab/src/runtime/clients.ts#L35
it is then probably lost when merging with default options.

For comparison, using useFetch("https://api-host/user/profile/", { $fetch }) works.
Using useClientFetch("/user/profile/", { onRequest }) also works.

@IlyaSemenov Thank you for investigating all these bugs, the module might become stable even sooner than I thought 💚


To buy list

Add stuff we need but don't have here

[ ] Green eggs
[ ] Ham

[] RT5370 WiFi Dongle

[ ] Spool of elastic cordage for deck, 2-4mm. #13


Unable to get wifi list on android Phone.

I am able to connect to ESP32 AP. it open up a HTML page but there is no Wi-Fi list in that page.
I am trying this on android phone as well as on windows phone. Not working on android but the same is working on windows phone.
Can anyone help me out. Thanks in advance.

The library used to Ajax with jQuery v3.2.1 and get WiFi list data from HTTP API. It might be not worked on Android WebView. That is why you don't see anything on Android but it's working fine on other regular browsers (Chrome/Firefox).

I tried with Chrome and Firefox browser on android phone. Still not working.

Did you compile it from scratch? What Android are you using?
I had this error once when I manipulated the event loop.
be sure that your init
ESP_ERROR_CHECK(esp_event_loop_init(wifi_manager_event_handler, NULL));
as well as your event handler
esp_err_t wifi_manager_event_handler(void *ctx, system_event_t *event) { .... }
work properly

I am using Mi Phone (android 8). But i tried with different phones. I am not able to get the list on any android as well as IOS phone.
On windows laptop it only show list on internet explorer rest no list on other browsers.
Yes,
ESP_ERROR_CHECK(esp_event_loop_init(wifi_manager_event_handler, NULL)); and wifi_manager_event_handler both are working fine.
log.txt
As I am from embedded background, I have no clue about this issue. Please help me out.
Thanks in advance.

Please suggest me how to resolve jquery issue.

I got the issue. we got an extra character on server in code.js file due to which browser is unable to render from code.js file.
Error: In http_server.c
Line 186 : netconn_write(conn, code1_js_start, code1_js_end - code1_js_start -1 , NETCONN_NOCOPY);

@ankur2854 what is your IDE/build system? This issue appears in platform.io but not on a classic make environment as far as I know


Issue with start-component

I get following error while using npm run start start-component
$ npm run start start-component

><EMAIL_ADDRESS>start C:\Users\HOME\Documents\projects\redux-micro-frontend\sample\todoApp
> webpack serve "start-component"

`resolve/lib/core` is deprecated; please use `is-core-module` directly
[webpack-cli] Unknown argument: start-component
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>start: `webpack serve "start-component"`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\HOME\AppData\Roaming\npm-cache\_logs\2020-12-03T13_24_20_148Z-debug.log

Can you provide the complete debug log

The issue is likely in your command. start and start-component are 2 different commands. Use only npm run start-component.

Yes, it worked with npm run start-component. Seems like the README.md was misleading. Should have read package.json first.

Thanks about pointing to the mistake in Readme. I will get it fixed.


Run checks on pull requests

Codecov Report

Merging #203 (efab1f5) into main (fb4eea4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #203   +/-   ##
=======================================
  Coverage   39.87%   39.87%
=======================================
  Files          35       35
  Lines        3162     3162
=======================================
  Hits         1261     1261
  Misses       1741     1741
  Partials      160      160

Continue to review full report at Codecov.

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


Slowness on downloads fonts

Hi, my company has a proxy that causes slow loading of fonts, and sometimes it crashes redmine when the theme is enabled, is there any way to store the fonts so they are not downloaded every time?
the display is great, it just does not work well inside my company network and I would like to get around this if possible.
Thank you and congratulations for the theme.

Hi.
Fonts for this theme are stored on your server, probably a problem in the outbound channel of your network.
I also worried about the font size and I will soon make it lightweight. when this happens, I will report on this topic.


Can't drag in space between (some) elements

Check that this is really a bug

[X] I confirm

Reproduction link
https://swiperjs.com/demos#slides-per-view
Bug description
With the default setting for touchEventsTarget, you cannot drag the carousel if you start from the space between slides (with a couple of exceptions).``
Expected Behavior
See the Slides Per View demo on your site. Using a mouse (haven't tested touch events, but would guess it's similar), you should be able to drag the slider no matter where your drag starts from.
Actual Behavior
If you click and drag in the space between slides 1 + 2, or slides 2 + 3, everything works fine. If you click and drag in the space between any other slides, the carousel does not swipe.
Setting touchEventsTarget to wrapper does somewhat mitigate this problem. However, it's not ideal; for example, I like to reposition the pagination to be underneath the swiper, rather than overlaid on top, and I don't want all of the area around the pagination swipable (especially when grabCursor is on).
Swiper version
8.0.7
Platform/Target and Browser Versions
Windows Firefox 98.0
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

It can be wrapper or container and this is how it suppose to work


Feature: Add option to rotate more image formats

What feature or improvement do you think would benefit Files?
Phones nowadays shoot mostly in .HEIF/.HEIC (Apple/Samsung) and RAWs like .DNG (altough .NEF and .CR2 plus Sony formats are also really widespread)
Files would be really extra useful if it allowed rotating photos with the wrong orientation out of the camera, for the file formats above beyond the currently supported JPG. I know exiftool handles all of them, with a simple exif metadata edit:
exiftool -orientation#=1 -m -P -overwrite_original photo001.heic
to set it as landscape
exiftool supports most image formats already.
Requirements

Enable rotation options for heif, heic, dng, nef and cr2 files

Files Version
Versão: <IP_ADDRESS> OS Version: 10.0.22621.819
Windows Version
Microsoft Windows 11 22H2 [Version 10.0.22621.819]
Comments
open-source exiftool usually does the trick.

@puppetsw fyi

@puppetsw fyi

After a bit of looking, HEIF, DNG, RAW etc formats aren't supported by .NET. I think if we want to have this function we would need to use an external library to handle this, as there is a fair amount of work to implement a decoder for these formats.

I don't think we should use a third party library for this.

Although BitmapEncoder API seems to have no HEIF/HEIC support, File Explorer can rotate .heic images. I wonder if there is any public API for HEIF/HEIC.
By the way, Files can't rotate gif and tiff images, but BitmapEncoder supports them. So we can add support for these formats right now.
https://learn.microsoft.com/en-us/uwp/api/windows.graphics.imaging.bitmapencoder?view=winrt-22621

@hishitetsu do you want to open a PR? I think it's just a matter of adding them to the list of supported file extensions.

Either is fine. I think anyone can do it, so we may open another issue and make it a good first issue.

I opened another issue. https://github.com/files-community/Files/issues/12133


Make configuration source of truth in AlluxioFuse

What changes are proposed in this pull request?
Make configuration the source of truth.
CLI inputs will overwrite configuration.
Why are the changes needed?
Ease the configuration and fuse options logics in AlluxioFuse.
Does this PR introduce any user facing changes?
NA

@ssz1997 i wanted to add three fuse configuration and find it's relatively complicated for moving options from AlluxioConfiguration to fuseOpts.

@yyongycy can you help take a look, thanks!


Selenium browser Headless Mode with Datacenter proxy

Hello,
I am facing one issue which stated below:
When I am trying to add extension without headless mode then it works perfectly with datacenter proxies as explained in https://github.com/Smartproxy/Selenium-proxy-authentication, whereas it is not working in headless mode. Can you suggest me what should I have to do here?
selenium.common.exceptions.WebDriverException: Message: unknown error: failed to wait for extension background page to load: chrome-extension://aaofpcleknmphbboehgnfonkinbjapel/_generated_background_page.html
from unknown error: page could not be found: chrome-extension://aaofpcleknmphbboehgnfonkinbjapel/_generated_background_page.html
(Driver info: chromedriver=97.0.4692.71 (adefa7837d02a07a604c1e6eff0b3a09422ab88d-refs/branch-heads/4692@{#1247}),platform=Windows NT 10.0.22000 x86_64)
This is the error.
Thank you in advance.
Regards,
Denish Vaghasiya

Same here.

Solution: you need to use pyvirtualdisplay and disable headless mode.

It might actually work using options.add_argument("--headless=chrome") ?
@NoahDrucker Maybe include in README.md in case it works?
Reference: https://stackoverflow.com/a/73079789/20443541

@NoahDrucker Maybe include in README.md in case it works?

Nevermind, allready included in main.py ==> close that issue?


--use_fp8_context_fmha is broken with Llama models

System Info

1x H100 SXM
tensorrt-llm 0.12.0.dev2024080600

Who can help?
@Tracin
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
1a. Build FP8 quantized checkpoint using LLaMAForCausalLM.quantize API, by passing QuantConfig(quant_algo=QuantAlgo.FP8) as an argument
1b. Same but with QuantConfig(quant_algo=QuantAlgo.FP8, kv_cache_quant_algo=QuantAlgo.FP8) (are there any difference expected in practice?)
2. Try building engine with --use_fp8_context_fmha
Expected behavior
Engine is successfully built and performs better
actual behavior
for 1a engine build fails with:
terminate called after throwing an instance of 'tensorrt_llm::common::TllmException'
  what():  [TensorRT-LLM][ERROR] Assertion failed: getIdx() should not be used with entry 17
 (/home/jenkins/agent/workspace/LLM/main/L0_MergeRequest/tensorrt_llm/cpp/tensorrt_llm/plugins/gptAttentionPlugin/gptAttentionPlugin.cpp:131)
1       0x7b2f125ba395 /usr/local/lib/python3.10/dist-packages/tensorrt_llm/libs/libnvinfer_plugin_tensorrt_llm.so(+0x51395) [0x7b2f125ba395]
2       0x7b2f1260481b tensorrt_llm::plugins::GPTAttentionPlugin::getIdx(tensorrt_llm::plugins::GPTAttentionPlugin::IdxEntry const&) const + 107
3       0x7b2f12604ad4 tensorrt_llm::plugins::GPTAttentionPlugin::supportsFormatCombination(int, nvinfer1::PluginTensorDesc const*, int, int) + 660
4       0x7b338e7ddae4 /usr/local/lib/python3.10/dist-packages/tensorrt_libs/libnvinfer.so.10(+0xb8aae4) [0x7b338e7ddae4]

for 1b it fails fails with:
RuntimeError: Paged Context FMHA doesn't work with int8 kv cache currently.

which seems to be set in config.json:
...
"producer": {
        "name": "modelopt",
        "version": "0.15.1"
    },
...
    "quantization": {
        "quant_algo": "FP8",
        "kv_cache_quant_algo": "INT8"
    },

After manually fixing kv_cache_quant_algo to FP8 it started working, but not sure if it's correct fix.
additional notes
Is LlamaModel.quantize recommended way to build checkpoints? Should I use something else instead?

please use https://github.com/NVIDIA/TensorRT-LLM/blob/main/tensorrt_llm/quantization/quantize.py for fp8 quant

Can you also show us some code or the specific TRT-LLM example you are using to help clarify the related APIs?

@Superjomn I'm using LlamaModel.quantize API, to be precise this:
LLaMAForCausalLM.quantize(model_dir, dtype="float16", QuantConfig(quant_algo=QuantAlgo.FP8, kv_cache_quant_algo=QuantAlgo.FP8), mapping, output_dir)

And then use trtllm-build command with output_dir.
It's not clear to me when/if I should be using https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/quantization/quantize.py vs this method. Should I use  quantize_and_export method directly then?
Thanks!

Hi @ttim LLaMAForCausalLM.quantize should work in your case.
I tried the below code snippet on the latest main branch, and everything went well.
from tensorrt_llm.models import LLaMAForCausalLM
from tensorrt_llm.models.modeling_utils import QuantConfig, QuantAlgo

hf_model_dir = "TinyLlama/TinyLlama-1.1B-Chat-v1.0"
ckpt_dir = "/tmp/ckpt-tinyllama"
LLaMAForCausalLM.quantize(hf_model_dir, ckpt_dir, quant_config=QuantConfig(quant_algo=QuantAlgo.FP8, kv_cache_quant_algo=QuantAlgo.FP8))

The quantization field of exported checkpoint config is
"quantization": {
        "quant_algo": "FP8",
        "kv_cache_quant_algo": "FP8"
    }

Could you please try with the latest main branch? Also, I noticed that the parameter order of your use of LLaMAForCausalLM.quantize is not exactly correct, would you also check that?

@syuoni seems like it got fixed in the last release! - it works with main branch, thank you!


[QUESTION] How to predict bounding boxed on big images with model trained on small images?

I have efficientdet-d1 trained on images 640x640. And trying to predict bboxes on image 2048x2048.
I loaded model from snapshot and tried to change image_size for model.config to recreate anchors. But got error "omegaconf.errors.ReadonlyConfigError: Cannot change read-only config container" .
I look to the reset_head() , but it do not change anchors.
Think that I am going wrong way.
How to predict bounding boxed on big images with model trained on small images?

I found solution:
        config = get_efficientdet_config(model_name)
        config.image_size=(2048, 2048)

        self.model = create_model_from_config(config,
            bench_task='predict',
            num_classes=num_classes,
            pretrained=False,
            pretrained_backbone=False,
            checkpoint_path = '/snapshots/efficientdet-d1.pth',
        )

Correct me if I am wrong!
So the Efficientdet-D0 model requires (512, 512) as the input image size.
My dataset consists images of size (600, 800)(H*W) input image size.
So 1) I resize the original image(600*800 ) to (512, 512) and scale the groundtruth bounding box accordingly.
While observing the bbox prediction output I see the bbox coordinates are higher than 512.
For eg, consider the bbox as [x1,y1,x2,y2] format I observe the output bbox as [0,0,700,600]
This means the output bbox is according to the original image and not the resized (512, 512) image.
I see out of 100 boxes 80 are in the wat discussed above. The shocking part is these boxes have higher classification scores like above 0.77 and all.
Am I understanding the concept correctly?
Any help appreciated!

I found solution:
        config = get_efficientdet_config(model_name)
        config.image_size=(2048, 2048)

        self.model = create_model_from_config(config,
            bench_task='predict',
            num_classes=num_classes,
            pretrained=False,
            pretrained_backbone=False,
            checkpoint_path = '/snapshots/efficientdet-d1.pth',
        )

Is it correct?

I found solution:
        config = get_efficientdet_config(model_name)
        config.image_size=(2048, 2048)

        self.model = create_model_from_config(config,
            bench_task='predict',
            num_classes=num_classes,
            pretrained=False,
            pretrained_backbone=False,
            checkpoint_path = '/snapshots/efficientdet-d1.pth',
        )

Is it correct?

I have one more question, when you input the (2048, 2048) image in the loader while predicting the create_loader takes you to transforms ResizePad function where the image resizes to (640, 640) because the efficientdet-d1 requires 640*640 image to operate, and then calculate the predictions correspondingly right?
So I don't think the larger image would arise problem.

@Ekta246 , I am using https://albumentations.ai/ for data augmentation. It can crop and resize image and bboxes together.

While observing the bbox prediction output I see the bbox coordinates are higher than 512.

I am clipping bbox.


Roll Flutter Engine from 4190543cb093 to c9a6ca05d8f8 (1 revision)

https://github.com/flutter/engine/compare/4190543cb093...c9a6ca05d8f8
2024-07-03<EMAIL_ADDRESS>Roll Skia from c14cce59222b to 86ee8cc61508 (1 revision) (flutter/engine#53713)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

/gcbrun

Commit queue failed; closing this roll.


[New game]: Dragon Run

🎮 Game Request
Game logic and basic description

This is a retro styled 2D game in which you have to tackle the enemy by jumping in air, and the muskic makes it even more funnier ; )

Point down the features
Game points:

Easy to play
Exciting sound effects to immerse you and keep you engaged.
Scoring system so that you can compare points with your fellow friends

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @codewithganeshhh !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @codewithganeshhh ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Make lab more useable for non-onmetal boxes

Hi - I am raising this issue after a conversation with mrhillsman this morning. I wonder if it would be possible to increase the script's POWER(!!!) by allowing user to more easily specify when the installation host does not have the expected amount of, or naming convention for resources such as network interfaces and disk (especially SSDs). "An Openstack in every home, a lab beneath every desk!"

Thanks for raising this @cemason


pull request for e2e test at 01/15/2017 23:26:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[incubator/rundeck] deprecating chart

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

https://github.com/helm/charts/issues/24310

will there be a "successor" chart? i didnt find any rundeck charts on the hub right now ...

@flypenguin I have moved it to my repository github.com/dwardu89/helm-charts

#21103


Feature Wanted? Recurrent Additive Networks. I'd be happy to implement/PR but...

A new paper from Lee et al. details a new, simplified recurrent neural network dubbed the "Recurrent Additive Network." It seems to do well on language modeling tasks when compared to the LSTM, despite using simpler computation with less parameters. This model would be easy to implement in Keras as a subclass of Recurrent, and I'd be happy to do it. However, my question is: is this something that would be pulled into the main repo? What is the standard policy for adding new sorts of RNNs?

You can submit your PR to keras-contrib and then if people like it, it will get merged into Keras.

Cool! Just wanted to see if that was the main path still.

I think its now more interesting to focus on SRU (https://github.com/fchollet/keras/issues/7870#issuecomment-328642850) since it has the same philosophy, its more up to date, and it already takes RAN as a reference.


Question: Will there be a pure BoTorch Implementation for SEBO?

Hey,
first thank you very much for the interesting Paper and SEBO.
Will there be a pure BoTorch Implementation in the next time? Maybe together with a tutorial.
Best regards,
Philipp

@pehzet Hey Philipp, applogize for missing your message. (not sure why but there is no notif). You can check out our tutorial in Ax https://ax.dev/tutorials/sebo.html This is not a pure BoTorch implementation but it is supposed to be more usage friendly. Let me know if you have any question about it.


first draft of finding-versions content

First draft of how-to-find-component-version content. Addresses Issue 2903.

Thanks @1000TurquoisePogs  -- I really appreciate the input

I added a user warning note, please take a look if you get a chance.
Re: the two other places: Since these are server-side components, I'll leave this documentation to @samanthasusu and @nellietan. The plan w/ this article was to start w/ client-side content, then hand this off to the other writers to add their documentation. The only reason I included the server side content that I did was b/c it already existed in another article (Understanding the Zowe release) and I wanted to re-organize the content for the sake of cohesion.


Updated auto-adjusting Y axis.

This pull request
This pull request contains an improvement to the Y-axis automatic setup.

While making steps more granular, multipication by 5 is also allowed. This allows for example 2 unit steps for data between 0 and 20.
After the granulation is completed, the unneeded steps (those below or above actual data) are removed.

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


CB-12098: explicit gcp backup location validation

Currently user may use gcs://cloudbreak-dev/rdoktorics/telemetry insteand of gs:// url. We should be explicit to enforce a gs:// protocol.
See detailed description in the commit message.

Before this change, if user mistakenly input gcs:// URL. Cloudbreak will still provision environment and freeipa clusters and fail later when access the gs bucket. Now we want to fail it earlier as gcs:// is a common protocol mistake.

the error message will be "Status: Creation Failed
The Google storage location [gcs://gcp-eng-sdx-daily-logs/yuwang-gcp-525-t7/] should be a gs:// URL. Refer to Cloudera documentation at https://docs.cloudera.com/cdp/latest/requirements-gcp/topics/mc-gcp_minimum_setup_for_cloud_storage.html for the required rights."
Now the creation will fail within one minutes at the beginning instead of 10 minutes in provision.

@doktoric
I tested with CB-CLI. Now it will fail immediately if filled with "gcs//" url within one second now instead of failing after 10 minutes during provision. I think this is the intention of the JIRA. Would you kindly take a look again? Thanks!


MacOS is not found at Opencore

Hi, I managed to boot from Opencore, it also finds my Windows partition, only my USB stick where MacOS is on it. It is not listed. I also made the config.plist after my system. no success. What is the reason for this?

OMG I just have to press the spacebar in Opencore then I see the DMG BOOT USB STICK LMAO LOL


feat!: delete getTreeNameFromConstructorFn

Closes https://github.com/celestiaorg/rsmt2d/issues/286 by deleting getTreeNameFromConstructorFn.
Closes https://github.com/celestiaorg/rsmt2d/issues/288, https://github.com/celestiaorg/rsmt2d/issues/289 by fixing those tests.
Inspired by https://github.com/celestiaorg/rsmt2d/pull/278
Includes two breaking changes:

ComputeExtendedDataSquare now accepts a parameter treeName: string
ImportExtendedDataSquare now accepts a parameter treeName: string

Ready for re-review. I would like to hold off on merging until after I verify that celestia-app is able to use this.


Implement scan package to scan the system remarks

Implement scan package to scan the paid-qa related system remarks and store the remarks in DB

[ ] Save system remark starts with "osn:q:" witch means it is a OpenSquare collaboration qa remarks.
[ ] Handle batch call that contains a system remark and a transfer in one extrinsic, like the FUND interaction.
[ ] Should support scanning different chains: Polkadot/Kusama/Statemint/Statemine/Karura...
[ ] ~Handle un-finalized blocks~

We don't have to handle unFinalized blocks.


Initial implementation of a builtin SSH connection support

Main motivation of implementing this is to bypass local PTY (ConPTY) on Windows. But there may be more use to it than that on other platforms maybe as well, so we make it available on all platforms, if possible.
Closes #1319, #1064.
Implement builtin SSH support. While being not THAT useful on Linux desktop or MacOS, it turns out to be a game changer on Windows platform, as we can fully bypass ConPTY with a builtin SSH client in case would simply SSH away only anyways.
Maybe this would also bypass ConPTY on Windows when using to get into WSL / WSL2 (I do not know yet).
Checklist

[x] build: make sure it builds on MacOS/X (CI) (currently has linker errors, can't find lib)
[x] build: make sure it builds on Windows (CI)
[x] build: make sure it builds on Linux (CI)
[x] SSH backend (using libssh2, bridging to C++ API: SshClient)
[x] vtpty::Pty implementation: SshSession
[x] frontend: configuration for login shell over SSH (how could the YAML config look like?)
[x] frontend: support SSH session configuration for SSH-agent based authentication
[x] frontend: support SSH session configuration for SSH key pair based authentication with password (to be prompted!) (needs code to interactively ask for password within the TE grid)
[x] frontend: support username/password based auth
[x] configuration additions documented
[x] DNS resolution of hostname to IP via getaddr (API matches on UNIX + Win32, yay)
[x] support TCP/IPv4
[x] support TCP/IPv6
[x] properly propagate login shell's exit code
[x] server host key verification
[x] configurable SSH agent forwarding (current problem: upon ssh-add -l the session terminates - ref)

Postponed post-merge and maybe post-0.4.0-release

@whisperity added the suggestion to auto-restart on closed sessions. See #1326.
when PTY got closed, print close-reason (exit code / signal) => hitting CR (or ESC?) should then destroy the session (close the window). See #1327.

Notes
this feature is only built if libssh2 is installed (its development package).

Right off the bat, here is something weird. When using the build from https://github.com/contour-terminal/contour/actions/runs/6763840917, I immediately get a complete failure when just starting Contour if I let contour create its default configuration:

nice catch. the instant failure is because it tries to connect to SSH (localhost here) via IPv6. apparently that's not working (either not possible or i need to fix IPv6 support, will check).
On the other hand, SSH should not be used when using default config. i'll address that asap. (it's because ssh.hostname defaults to "localhost" and makes the frontend think that it should connect to SSH rather than spawn a local process.

@whisperity I cannot repro what you described. my earlier comment was based on an old state of this PR. :(
I've found some other minor issue though. the default user name on windows was not properly inferred. it was set to "unknown" as fallback. I've just fixed that.

I cannot repro what you described. my earlier comment was based on an old state of this PR. :(

I linked exactly the build I was using... 😦 Although, yeah, it does not point to a commit.
I will try the latest build of the branch tomorrow!

Using this build, I can confirm that Contour auto-starts PowerShell on Windows using the default configuration.

Now it asks password for three times, but if i insert correct password second time it does not work here is output log for two different runs

we weren't clearing the input buffer on retry. I did not notice, because I only tried success and login failure excess. thx for pointing it out. fixed.


feat(serverless-contracts): control sendMessages throughput

Tests are broken because I added p-queue which is an ESM only package. I think it means it breaks the Swarmion contract package for all CJS users.
What is your recommendation ? Downgrade p-queue to the lastest cjs version ? Or is there a trick to make it compatible ?


A concise title for the bug

Bug description
not even showing option and also when i press f to fullscreen the video reloads
Steps to reproduce the behavior
not even showing option and also when i press f to fullscreen the video reloads
Browser
Chrome, Microsoft Edge
Operating system
Windows
Extension version
latest
The bug still happens in these conditions

[X] The bug still persist when I disable all other extensions (ex. in incognito mode with only this extension enabled)
[X] When I logged out of my YouTube account (ex. in incognito mode)
[X] In incognito mode

Additional context and/or screenshots
yeahdsdvsdvsfafa affgggGGQ REGGGEQGE GQEQGEQGE

Several things are unclear in this issue:

This issue is missing a title
"latest" is not a version number
"yeahdsdvsdvsfafa affgggGGQ REGGGEQGE GQEQGEQGE" What do you expect me to do with this additional context?
"not even showing option" What option do you mean?

"when i press f to fullscreen the video reloads"
I'm currently not aware of any functionality that could cause this.

Could you post a screenrecording of this bug (with all other extensions disabled)?
And do you see any javascript errors in the DevTools console? (Press F12 and select the "Console" tab)


Write CafeHTTPRequest body to file

Background uploads are only supported for file-based data. This makes sense because the app may be terminated during an upload, and therefore all the app’s memory goes away. We need to write the request payloads to disk and CafeHTTPRequest should just have a property that is the path to the file.

Moved to this sprint (11).
I guess we'll have to make an assumption about who is doing the queue handling since on mobile we'll have to write them to disk.
An option to get around that is to do the writing when the handler calls SetCafeRequestPending, or if that timing does't work, just another method, WriteCafeRequest or something.

Oh, nvm, I guess we just write the data to disk when the handler calls CafeHTTPRequest.

Do you want the file deleted when SetCafeRequestComplete is called?

Seems like a good time to delete it, yea.

It's not clear to me yet what "cleanup" is all about. It's something you mentioned needs to be called.

Cleanup check to see if a group can be deleted. It's just more efficient to allow the handler to call this rather than internally every time a req is marked as complete.


[T3A2][W11-A2] Ho Yi Hang

Ready for review

@yihangho Please acknowledge and close PR. We are now advised to follow this protocol for PR closing to ensure student read our comments.


Support for 'maintain' and 'triage' as team permissions

Simple PR to add support for triage and maintain as team roles.
Successfully tested using a terraform apply in our environment.  It works despite Github API documentation not officially showing this is available (as of 11/15/2019).

Looks like this is public stable: https://github.blog/2019-10-03-delegate-responsibilities-with-expanded-repository-permissions/

Duplicate of #279.

Duplicate of #303.


GPU program failed to execute at ___

Not sure if it is a driver problem on my end or a code bug. my other torch models train fine. so hard to say . . . .
Attaching the output.
1%|█▎        | 6/1000 [00:00<?, ?it/s]
| 0/100 [00:00<?, ?it/s
File "main.py", line 138, in
beliefs, prior_states, prior_means, prior_std_devs, posterior_states, posterior_means, posterior_std_devs = transition_model(init_state, actions[:-1], init_belief, bottle(encoder, (observations[1:], )), nonterminals[:-1])
File "/MinicondaInstallation/envs/planet_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
RuntimeError: cublas runtime error : the GPU program failed to execute at /opt/conda/conda-bld/pytorch_1556653183467/work/aten/src/THC/THCBlas.cu:259

Given that other people are running this code fine, I suspect that there is an issue with your PyTorch setup. Note that you do need the latest version of PyTorch (1.1). Doesn't look like a memory issue, but easy way to test is use a small batch size and chunk size. You can also find out if it is a CUDA issue by running with --disable-cuda. Otherwise probably best to post to the PyTorch discussion forums first.

must be some bug, I could not make it run in that server but ran just fine in another without any environment change. closing the issue.


Changes required for 2023

See attached file(s)
Adjustments_ for_ 2023.txt
2023-phase.txt
2023-age.txt
2023-moon-phase.zip

thanks


WIP: Single hotel page

Closes #40
I'm sharing this WIP so you can tell me if I'm doing something fundamentally wrong.
A few notes:

I intend to squash it before merging, but during the development, I find it practical to have a more granular history.
The general idea about the folder structure:

SingleHotel/index.js will contain a root QueryRenderer.
Different parts of the screen will be fragment containers. For example, the header will have a fragment container in SingleHotel/Header/HeaderContainer.js and this container will render the pure component located in SingleHotel/Header/Header.js.

I didn't connect it to Relay yet, so I'm faking containers and their flow types. I'm using data copy-pasted from the API or from the Booking.com website.

Please test these changes in Expo after build has finished:

I intend to squash it before merging, but during the development, I find it practical to have a more granular history.

It never worked for me but it's your history. I am interested only in the result... 🙂

Different parts of the screen will be fragment containers. For example, the header will have a fragment container in SingleHotel/Header/HeaderContainer.js and this container will render the pure component located in SingleHotel/Header/Header.js.

I usually keep it in one file because you can see props and GraphQL fields next to each other (it's strictly related). But again - I don't care. It's not that important. Do what you like.

Integrated, thanks.


Deployment for Data Mesh MVP components

Airflow
postgress DB
single node Airflow cluster - Web Ui, Scheduler and Worker all runs on single pod
Trino
Trino cluster
with 1  co-ordinator
2 Worker Node
basic authentication
Hive MetaStore service
Postgres DB for MetaData
OpemMetaData
OpenMetaData Web UI
Elastic Search
Postgres Database
Airflow integration for Data ingestion
Minio
Deployment

Build helm chart / Kustomize for deployment

Initial MVP deployment

initial MVP deployment completed


[Bug] Favourites section does not scroll

A user has reported that the favourites section in the latest version does not scroll. I haven't confirmed this yet.

Fixed in 24c6e6abea4e14b2bfc89ae05e6954bf392e3a21.