InlinedVector doesn't protect against method arguments aliasing the vector

It seems that the append, insert, emplace and assign methods of InlinedVector don't protect against arguments being invalidated when the capacity of the vector has to be increased or existing values have to be moved. For example, vector.insert(vector.begin(), vector[0]) is currently never safe, and vector.push_back(vector[0]) is not safe if vector.size() == vector.capacity().  If this is the intended behaviour, it should probably be documented, as it deviates from what the C++ standard requires for the standard library (<IP_ADDRESS>.1.3).
Similarly,  InlinedVector::operator=(const InlinedVector&) will call std::copy(vector.begin(), vector.end(), vector.begin()) for self-assignments, which technically violates the precondition for the output iterator argument passed to copy (<IP_ADDRESS>).
Even ignoring issue #32, InlinedVector currently can't provide the same strong exception guarantee that the C++ standard requires (<IP_ADDRESS>) for some basic container operations like push_back, because if the construction of a new element throws an exception after the capacity of the vector has been increased, the vector isn't left in the same state as before the call (since the capacity remains changed and all pointers into the vector remain invalidated). This only really matters if you intend to support arguments that could be invalidated by capacity changes, but it may be worth documenting in any case.

I'll take a look at the aliasing issues.
For exception safety, the short story is that we know InlinedVector isn't exception safe.  I'm currently writing tooling for us to thoroughly test our classes for exception safety.  Once we have that, we will start updating all of our types which need to worry about exceptions accordingly.
I would guess that InlinedVector will have slightly weaker guarantees than std::vector, partly for the reasons you outlined.  Our current thought is that we will be going for basic exception safety everywhere, and strong exception safety where reasonable and where it won't affect performance.

I took a look at this today.  Insert() indeed has a bug, and it is fixed in an internal change which will be upstreamed once accepted.  Similarly with operator=.
I don't see the issue with push_back.  push_back calls emplace_back, which if we need to grow, calls GrowAndEmplaceBack.  GrowAndEmplaceBack first allocates a new buffer, then constructs the new object into it, then moves everything else into the new buffer, so it should be safe.

You're right, I was mistaken about push_back, sorry.
The insert(const_iterator position, size_type n, const value_type& v) and insert(const_iterator position, InputIterator first, InputIterator last) (for forward iterators) overloads also suffer from this issue, but you've probably seen that too. Making the latter completely safe would hurt performance, but maybe an assert that checks e.g. the address of the first element in the range would be worth the overhead.

The two-iterator insert, we won't support aliasing, just like in std::vector (#4 in http://en.cppreference.com/w/cpp/container/vector/insert).
The iterator-count overload is also being fixed, yes :)

This is fixed in https://github.com/abseil/abseil-cpp/commit/5fcbe86e7ba65b6457d98764aa511c4f05c9435b


anyone have trouble with smithy-client?

Builds started breaking looking for 3.3.4 and its not finding it.  appears it was pushed about an hour ago of this posting.
16:42:04  39.24  ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>16:42:04  39.24
16:42:04  39.24 This error happened while installing the dependencies of<EMAIL_ADDRESS>16:42:04  39.24  at<EMAIL_ADDRESS>16:42:04  39.24
16:42:04  39.24 The latest release of @smithy/smithy-client is "3.3.3".

resolved now. pnpm able to pull what it needs


fix: move comment parsing to the scanner

This disallows -- in a comment but - is fine

I was just in the process of writing that. I'll use your version if mine doesn't work.

I will merge it locally to deal with some conflicts. Thanks again!


[0.9.1] [0.9.2] [nightly] Unable to query fields starting with keywords ("user", "in", ...)

In 0.9.1, 0.9.2 and the current nightly build, values with names starting with sql keywords cannot be queried. Try inserting this :
network,server=someserver,interface=eth0 in-bytes=1765,in-packets=16,in-errors=0,out-bytes=9719,out-packets=9,out-errors=0<PHONE_NUMBER>696

And then query it with :
select * from network;

It throws a "Server returned error: found IN, expected identifier, string, number, bool at line 1, char 8"
It was not happening in 0.9.1 with "user", but it does with 0.9.2 and later. I only tested it with "user" and "in".
Edit: I read the guidelines too late...
OS: Debian 7.8
Installed from the pre-built .deb.
Test case :
HOST=localhost
curl -G "http://$HOST:8086/query" --data-urlencode 'q=create database mydb'
curl -X POST "http://$HOST:8086/write?db=mydb" --data-binary 'network,server=someserver,interface=eth0 in-bytes=1765,in-packets=16,in-errors=0,out-bytes=9719,out-packets=9,out-errors=0<PHONE_NUMBER>697'
curl -G "http://$HOST:8086/query" --data-urlencode 'q=select * from network' --data-urlencode 'db=mydb'

Edit 2: It does not need to start with the keyword. An other example :
HOST=localhost
DB=test001
curl -G "http://$HOST:8086/query" --data-urlencode 'q=create database '"$DB"
curl -X POST "http://$HOST:8086/write?db=$DB" --data-binary 'network,server=someserver,interface=eth0 net-in-bw=42<PHONE_NUMBER>697'
curl -G "http://$HOST:8086/query" --data-urlencode 'q=select * from network' --data-urlencode "db=$DB"

Edit 3: It seems that it is releated to the dash character -. With spaces (escaped at insertion step), it works. Also having a backslash in a value name may create an error too : "found \, expected FROM", which may be releated to this issue.

It looks like the in-bytes field gets expanded to select in-bytes when select * is used and because it contains a keyword and is a valid expression, the select clause doesn't know what to do. A workaround for now would be to insert the measurement as network,server=someserver,interface=eth0 "net-in-bw"=42<PHONE_NUMBER>697, with ""s around the value name. That would make you able to query the measurement again.

Thank you, the insertion and query works! :-) . However, the field name now contains the double quotes, so to query it, I have to write select "\"net-in-bw\"", and the response also contains the double quotes.
I'll use camelcase for now :stuck_out_tongue: .


Stream end encountered error

Hey, thanks for a great library.
There is the problem (or not): after sometime after connect to server client disconnects with "Stream end encountered" error. Clients reconnects and everything works good, but I am wondering: is it OK?
Private Pub service used as server https://github.com/ryanb/private_pub

By the way. The problem was because of short timeout period on Thin server. So your client works fine :)

Good to know, thanks :)


openCL crash in models converted by the newer versions of TF (e.g. TF 2.9)

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
TensorFlow installation (pip package or built from source): source
TensorFlow library (version, if pip package or github SHA, if built from source): TF 2.8 - TF 2.9 - TF Nightly

2. Code
We have implemented a small tool (with a comprehensive ReadMe) to reproduce the mentioned issue. Here is the link to the repository:
https://github.com/Bahar-BM/dense_3D_inputs
3. Failure after conversion
Converting models in which there is at least one Dense layer fed by a 3D input should be done using older versions of TensorFlow (we have tested v2.3 and v2.4). Otherwise, the obtained TFLite model will crash with the OpenCL delegate with this error message:
ERROR: TfLiteGpuDelegate Init: FULLY_CONNECTED: Amount of input data should match weights width

Hi @Bahar-BM !
Could you check this comment from similar issue which suggests using 1x1 convolution.
Thank you!

Hi, try after this commit https://github.com/tensorflow/tensorflow/commit/61b98be3c21f0ae40657f62eaaced4bb8e904382

@roserg
Hi, your new commit resolved my issue. Thanks!


Xamarin App isn't scanning code on real phone (Oneplus 8 Pro)

Hello.
I use ZXing.Net.Mobile library in Xamarin Forms app.
On Android emulator app is working fine but on real phone it only shows the preview but doesn't scan code.
I see "E/libc    (26182): Access denied finding property "vendor.camera.aux.packagelist" error in debug log.

Not sure if issue is actually exists. Sorry

Not sure if issue is actually exists. Sorry


WELLETC in rft results

Occasionally the underlying code of ecl2df produces a column called WELLETC in extraction of rft data
This is only seen on specific Snorre case.
Possible solutions:

Include a special treatment of rft data, delete WELLETC column if encountered
Fix underlying ecl2df code

#13 closes


Rewrite add_gap with dataframe merging

Significant performance boost during testing in Jupyter enviornment

Reconciliation
Checked against old method for seasons 2022 - 2024
Absolute difference is negligible for all seasons.
The new method will cause some "data loss" where the column won't be populated if the driver has DNFed. This behavior is verified by testing with VER.


Re-work our REST API doc generation

Task Description
This is the master task for the re-working of our REST API-s doc generation.
Tasks

[x] Restore the generation of our Swagger documentation ( #1124 )
[x] ~Upgrade the swagger-maven-plugin to 3.1.8, or higher ( #1128 )~
[x] Set up our documentation to work with Spring Boot + Springfox + Swagger ( #1125 )
[ ] Stylize our REST API's docs ( #1126 )

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

Can i work on this?

Yes, you pick #1126.


[RFC]: add findLastIndex method to array/fixed-endian-factory

Description
This RFC proposes adding support for a findLastIndex method to @stdlib/array/fixed-endian-factory.
This method is the equivalent of the TypedArray.prototype.findLastIndex method, but for fixed-endian typed arrays.
When adding support, the following tasks should be completed:

The ./lib/main.js file should be updated with the added method.
A new test file should be created which includes unit tests testing against expected behavior.
Two benchmark files should be added: one measuring baseline performance and another measuring performance as the array length increases.
The README should be updated to include documentation for the added method.

To provide a concrete example of what a PR adding the desired method should contain, see https://github.com/stdlib-js/stdlib/commit/956a4624c788689b1bca285856b987ea3aa32eb6, which is a commit adding forEach support. This should provide an idea of what is expected.
For an example of specialized typed arrays having customized methods, see

@stdlib/array/bool
@stdlib/array/complex128
@stdlib/array/complex64

While the method proposed in this RFC will likely not match the implementations found in the above packages, those packages should provide a conceptual idea of what is desired. Do not simply copy-paste the code found in those packages.
Notes

If you are interested in contributing a PR which addresses this RFC and are a first-time contributor or not yet familiar with our project conventions, please do not submit LLM-generated code. Please consult our contributing guidelines and the associated development guide. Failure to respect project conventions will result in your PR being rejected without review. Thank you for understanding!

Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with [RFC]:.

working on this!


fix(clippy): MSRV should be 1.70.0

Why
Since Rust 1.78.0, make clippy does not pass anymore for the following reasons:

How

cargo clippy --fix
update MSRV from 1.64.0 to 1.70.0
make fmt

ah my bad 😓
done ✅


hylafaxplus & nixos/hylafax: small improvements

Motivation for this change
I prepared this branch due to hylafax failing with libtiff 4.2.  However, I missed to open an issue early and got outpaced by #122650 -- sorry.  Yet, in the course of fixing the build, I also used the opportunity to prepare a couple of minor improvements in the hylafaxplus package and the hylafax module, which I don't want to waive.  The commits' title lines should give a proper summary of the changes.
Besides this, the pull request replaces the patch file for libtiff 4.2 compatibility from #122650 with a patch provided by Gentoo (following the suggestions in https://nixos.org/manual/nixpkgs/stable/#sec-patches).
Notifying libtiff patch contributor @ajs124 (#122650) and potential user @RamKromberg (#19392): maybe you can test and leave a review, if you can spare the time.
Things done

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

[x] NixOS: only x86-64
[ ] 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"
[x] Tested execution of all binary files (usually in ./result/bin/): only those needed to send/receive faxes
[ ] 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.

More:

[x] sent and received faxes with analog fax modem
[x] sent and received faxes with asterisk/iaxmodem
[ ] other hylafax communication like SSL Fax or paging

Result of nixpkgs-review pr 123084 at c2af1ff2 run on x86_64-linux 1

1 package built successfully:

hylafaxplus

1 suggestion:

warning: build-tools-in-build-inputs
util-linux is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/servers/hylafaxplus/default.nix:82:3:
   |
82 |   buildInputs = [
   |   ^

Thanks @r-rmcgibbo for this nice tool!

  util-linux is a build tool so it likely goes to `nativeBuildInputs`, not `buildInputs`.
  Near pkgs/servers/hylafaxplus/default.nix:82:3:

Not here, as the build process collects the agetty path from util-linux and stores in in the package, to be used later at runtime.


Simplify installation process on MacOS w/ homebrew

Not quite right  :-(  This does not install CH341SER_MAC.ZIP but installs CP210x USB to UART Bridge VCP Drivers instead.


[BUG] Limit CPU usage for 3Di prediction

phold version: 0.2.0
Python version: 3.11.9
Operating System: CentOS

Description
During CPU prediction of 3Di tokens phold uses all available CPUs, even though the threads parameter is passed.
Command to reproduce:
phold run -i largest_virus/pharokka.gbk \
    -d work/phold --threads 32 --cpu \
    -o phold --ultra_sensitive --force

I had this same issue

Thanks @valentynbez @jamesck2 ,
This bug is fixed now thanks for spotting it - the fix is available in the main branch (I'll make a release soon).
Let me know if it doesn't work, my tests suggest it works.
George


HotExit query editors don't work correctly from stable to insiders

Azure Data Studio Version: master

Steps to Reproduce:

Open some SQL scripts and type of content in them using 1.13.1 stable build
Close ADS stable with documents unsaved\open for HotExit
Open ADS insiders

Expected: SQL editor windows behavior normally
Actual: Closing the SQL editors creates "untitled plain text" tab documents which can't be closed.  Clicking close button on tab doesn't do anything.

@anthonydresser I think this is likely related to the custom editors change that went in recently.  Could you please take a look?

This is interesting since aren't insiders and table supposed to use different data store directories? so they don't share hot exit?

@anthonydresser insiders and stable share cached state for ADS (unlike with vscode).  However, the real issue here isn't going back-and-forth with stable\insiders.  I'm expecting this issue should impact all users when they upgrade from 1.13 to 1.14, right?

I see, yes, I'm looking into the issue, I know where the issue is, I'm thinking on the best way to solve it.

Does this affect notebooks as well, or just query editors?

Yes it will affect every custom edtior we have that uses inputs, I have a patch for this upgrade scenario.


Documentation for configuring the angular plugin

Spent some time figuring out how to configure Raven using the angular plugin. Couldn't find anything in any documentation, so this might be of use to someone, sometime.
Add this in the angular configuration phase:
myApp.value("RavenConfig", {
dsn<EMAIL_ADDRESS>config: {
whitelistUrls: ['myurl.com/scripts']
//Additional config options here
}
});
Note that this may not show errors that happen before angular is bootstrapped. To get that, use normal normal configuration, and use dummy RavenConfig object (which will yield an error saying that Raven already has been configured).

hi there. I'm the one who use Raven-js with angular. https://github.com/gdi2290/angular-raven this Readme page helped me and had enough info to configure it correctly.

I am willing to help however as this error is insanely annoying in headerless testing (only place i get it) and i really dont want to mock/stub the entire ravenjs lib...
If I figure out a workaround to check for Raven already being config'd in angular.js plugin in the context of window shall I pr?

@cerdman https://github.com/getsentry/raven-js/blob/master/src/raven.js#L724 there is Raven.isSetup() to check if Raven is configured or not. Although I am not sure how it could be helpful to configure angular-raven plugin.
I wonder if adding a link https://github.com/gdi2290/angular-raven to docs could be enough or not. Thoughts?

So I've installed angular-raven using bower, but I'm curious whether I need the angular plugin included with the raven-js source files.  If so, how do I actually include the plugin code into my angular project?  it's not clear what I'm supposed to do with the plugin code snippet.
Thanks,
Henry

Docs finally added in #405
You can read them here: https://github.com/getsentry/raven-js/blob/master/docs/integrations/angular.rst
And they'll soon be published at https://docs.getsentry.com


Pre-trained final model

Hi
Thanks for the brilliant implementation.
Do you or anyone from the community have trained the model and can give access for downloading for everyone usage?
I don't want to train an already trained model, it's not healthy for the environment!
Thanks.

Yes if anyone could share a pre-trained model on a large corpus that would be great. I could only pre-train on a small CNN corpus as a proof-of-concept.
Will leave this open for anyone to share their pre-trained models


[Welcome to PowerToys]: Text 'Toys' of the 'Welcome to PowerToys' is getting cropped at 225% resize mode.

Microsoft PowerToys version
"0.35.0"
Running as admin
Area(s) with issue?
Welcome / PowerToys Tour window
Steps to reproduce
Test Environment:
OS: Windows 10 Build 21354.1
App version: 0.35.0
App name: PowerToys
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Pre-Requisite:

Go to settings > Display settings > Set the resolution to 1280*768.
Set the slider value to maximum i.e. 225% (Settings -> Ease of access -> text scaling) and Apply.

Repro Steps

Launch 'PowerToys' application.
Application will get open with default list item 'General' selected.
Navigate to 'Welcome to PowerToys' link present at the right side of the pane and activate it.
'Welcome to PowerToys' window will get open.
Observe the issue.

User Impact:
If any application does not support resizing of text without loss of content or functionality, the users with mild visual disabilities who scales the text instead of using the assistive technology would be impacted as zooming will not work as desired.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/sites/accessibility/_layouts/15/Doc.aspx?sourcedoc={5BE8E205-7508-414C-A273-657770321DA0}&file=MAS 1.4.4 – Resize Text.docx&wdLOR=cD434D0F2-F515-4FD8-B4DA-BA99282968ED&action=default&mobileredirect=true&cid=e8a7e0cb-a049-4c6a-b020-862f20088e4c

✔ Expected Behavior
Text 'Welcome to PowerToys' should not get cropped and should be properly visible at 225% resize mode setting .
❌ Actual Behavior
Text 'Toys' of the 'Welcome to PowerToys' is getting cropped at 225% resize mode setting.
Other Software
No response

This looks to be a limitation of NavView.

@crutkas Looking at this, I'm wondering if "Welcome" is maybe sufficient in the NavView item instead of "Welcome to PowerToys"?

Yeah, that would work i just think how fragile anything is when it comes to loc.

i do think that is a fine solution

This is now fixed in 0.37 https://github.com/microsoft/PowerToys/releases/download/v0.37.0/PowerToysSetup-0.37.0-x64.exe


Have a story about / support for experiment tracking: graphs of observables e.g. loss, device health

This needn't be the core library though...

I thought of supporting numerical logging in ppx_minidebug, it's one direction of resolving this issue.

W&B is the gold standard. Some ideas outside the Weights and Biases space:

Ketrew: Keep Track of Experimental Workflows, not machine-learning specific, but an OCaml library already, might turn out a great match
labml: Monitor deep learning model training and hardware usage from mobile, Python API, MongoDB for the monitoring
Aim 💫 — An easy-to-use & supercharged open-source experiment tracker. -- would need to integrate via Python


Gh 1444 1443 add participate/follow button channel rhs

Summary
Add participate and follow buttons to Channels RHS.
Participate flow has been simplified, messages were different when you had no access to the channel (now you can participate anyways)
Some code has been moved to share logic properly between RDP and channels RHS:

new run.tsx file in the hooks folder with custom hooks for folowers/follow/unfollow/participate
RDP code migrated to use the hooks
new Code in rhs_about or rhs_title to use the same hooks

Follow/Unfollow Run
It's possible to follow/unfollow run from 4 places: run_list, channels_rhs, rdp-rhs and playbooks-lhs (some of them still don't use the new hook).
They all have telemetry (and additionally include where the action was triggered).
E2E tests included for follow/unfollow (tested telemetry too)
Participate
It's possible to participate in a run:

from RDP if you were not a participant but had access to RDP
from Channel RHS, if you were a channel member but not a participant
E2E tests included for participate/leave with telemetry

Telemetry
Tiny refactor on telemetry to accommodate new events with the new approach. Enum types for telemetry have been improved.
demo videos
Some flows are still not reproducible (without circuit breakers in the code). I added some recordings to see them in action and they will be available after the run-action is done.
follow/unfollow run from channels rhs
https://user-images.githubusercontent.com/4096774/193800082-725be264-8d15-426e-92b7-de5cbd33b52b.mp4
participate from channel when there are no participants
https://user-images.githubusercontent.com/4096774/193797716-4112d5e2-f558-4f2e-aa3f-63e1ef1ee643.mp4
participate from channel when there are participants
https://user-images.githubusercontent.com/4096774/193797699-bb76a659-b380-46ba-88ce-71ef9bf40fc4.mp4
I'm the owner and there's no any other participant
Still needs new code at RDP RHS participants by @gbochora .
https://user-images.githubusercontent.com/4096774/193812377-1948230c-3faf-4fb9-ae86-88429db5122c.mp4
Ticket Link
Fixes https://github.com/mattermost/mattermost-plugin-playbooks/issues/1444
Fixes https://github.com/mattermost/mattermost-plugin-playbooks/issues/1443
Checklist

[X] Telemetry updated
[ ] Gated by experimental feature flag
[X] Unit tests updated

Looks good @trilopin! Thank you for the recordings, they were super helpful because it was difficult to reproduce some of the cases! A couple of non-blocking requests:

For the follow button in the RHS header, the font should beOpen sans and not Metrolpolis. Basically we should make the styling identical to the follow button on the threads RHS.

Could you confirm that follow/unfollow button is 'Open Sans' in all his versions (channels_rhs, run_detail_rhs, run_list, playbooks_lhs)?


kernel: start: relocate vector table

Without this vector table initiated, an system exception
will be raised under arm arch M0.
Signed-off-by: likai liu<EMAIL_ADDRESS>

@galak An abnormal crash was encountered in our chips with zephyr OS, and the reason for this crash is that, on the arm architecture ARMV6_M, the ARMV6_M system requires an exception vector table at the 0 address. Because the relocate_vector_table function is not implemented, the exception vector table does not move to the 0 address, so the system generates an exception.
Now the function is implemented and the exception vector table moves the 0 address with this fix.

Just checked with @cvinayak and we don't think this makes sense. You cannot write to address 0, which is typically flash, with a memcpy. There must be something else here that we're missing. Can you give us additional details about the config you are running? are you executing in-place?

@Likailiu Agree that vector table should  be at address 0, but thats always done by the linker in Zephyr OS for CM0. Besides, your patch with memcpy to address 0 (in flash) will not work.

@carlescufi @cvinayak @andrewboie Hi all,  if the address plan about flash is just as you said, there will be some problems. There is another situation, the 0 address is mapped by bootlink (SRAM) and can be written. Our implementation is the solution to this situation, and the address plan we are using can be found in the linker.cmd file. Thanks.

@carlescufi @cvinayak @andrewboie Hi, all, if the address plan about flash is just as you said, there will be some problems. There is another situation, that is,  the 0 address is mapped by bootlink (SRAM) and can be written. Our implementation is the solution to this situation, and th address plan we are using can be found in the attached linker.cmd.txt. Thanks.
linker.cmd.txt


Add optional vault exporter for Prometheus.

Signed-off-by: Dimitri Graf<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR adds the possibility to deploy a Vault exporter sidecar container by setting some parameters.
This is needed if you want to monitor your Vault with Prometheus.
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.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

Hi Guys
I'm still waiting for feedback on this one. Is there anything missing? Can I do anything to speed up the process?
Best regards
Dimitri

/lgtm

/ok-to-test

/lgtm


Defects found in Accessibility Check

Detailed description

Describe in detail the issue you're having.
We need do AVT(Accessibility Verification Test) for our project, I found 10 issues(7 of them are below to 1 type problem, 2 of them are belong 1 type problem) cannot be solved from my side, could you help fixing them. Thanks.

You can download the IBM Equal Access Accessibility Checker from website
"https://www.ibm.com/able/toolkit/tools#verification-tools"
to install the Accessibility Checker. If you are using Chrome, for example, after the installation, open the developer tool, and select 'Accessibility Checker'.

Element location:
<ibm-table"/>
<ibm-checkbox>
<input></input>
</ibm-checkbox>
<ibm-table"/>
problem:
Form control element  has no associated label
Each form control must have an associated label
IBM 3.3.2 Labels or Instructions | WCAG 2.1 technique H44 | WCAG 2.1 technique ARIA16
What to do:
Add a for attribute to a visible label with the id of this control (e.g. );
OR put the UI element inside a visible  element;
OR add an aria-labelledby attribute to the control. It must point to visible text on the page that is meaningful as a label;
OR only if the design cannot have a visible label, use the title attribute to provide a label.

Element location:
<ibm-table>
<ibm-checkbox>
<label></label>
</ibm-checkbox>
</ibm-table>"
problem:
The  element does not have descriptive text that identifies the expected input
A  element must have non-empty descriptive text that identifies the purpose of the interactive component

IBM 2.4.6 Headings and Labels | WCAG 2.1 technique G131
What to do:
Add text to the empty  element that clearly identifies the purpose of the interactive component being labelled.

Element location:
<ibm-checkbox>
<input></input>
</ibm-checkbox>
problem:
Checkbox input is not in the group with another Checkbox with the name "undefined"
Related sets of radio buttons or checkboxes should be programmatically grouped
IBM 1.3.1 Info and relationships
What to do:
Group related controls of the same type by enclosing them within a  element;
OR, group related radio buttons by using role="radiogroup";
OR, group related checkboxes by using role="group";
OR, disassociate unrelated controls or controls of different types by using different "name" attribute values.

Is this a feature request (new component, new icon), a bug, or a general issue?
a bug

Is this issue related to a specific component?
No

What did you expect to happen? What happened instead? What would you like to see changed?
Fixed them so that our project could pass AVT.

What browser are you working in?
Firefox.

What version of the Carbon Design System are you using?
^3.15.4

What offering/product do you work on? Any pressing ship or release dates we should be aware of?
4Q of 2020

Steps to reproduce the issue

Step one: Download the IBM Equal Access Accessibility Checker for AVT. (https://www.ibm.com/able/toolkit/tools#verification-tools)
Step two: Scan page with tabs mentioned before.
After scan(very quickly), it's obvious to see the problems.

Additional information

Screenshots or code
Notes

Hi, this seems to have been fixed by https://github.com/IBM/carbon-components-angular/pull/1280, try upgrading to version 3.51.0. If the issue still exists please feel free to reopen!


Deploying Micro-Bosh & CF on Hyper-V Manager Baremetal

Hi everyone, is bosh-azure-cpi-release useful to deploy Micro-Bosh VM , Upload Setmcell, CF Bosh VMs creation etc, on windows 2016 datacenter Hyper-V Manager (Baremetal) (not azurestack VM).
If not, can please let me know if there's any reference for this feature. Why because many of my clients having  windows 2016 datacenter Hyper-V Manager, they want to deploy Micro-Bosh & CF and give service on baremetal same like VMware.
Thanks in Advance
Abhilash S

Azure CPI doesn't support Hyper-V manager (Baremetal). AFAIK, there's no other CPI to support it.


palindrome-products: update tests to v1.2.0

Resolves #626

@cmccandless would you be so kind to resolve this so we can get it merged?

@SleeplessByte resolved!

Accidental close. Please ignore that.

Merge/rebase issue has been cleaned up. Once Travis finished, this should be good for squash-merge.

@SleeplessByte OK, now I'm lost. What of my changes would've affected the Travis build like that? I'm just ran make test-travis locally, and it passed just fine, and I didn't commit any changes to package-lock files or anything like that.

@tejasbubane have you seen this weird Symbol issue before?

This is an issue with Jest and latest node on travis. Fixed by using LTS in https://github.com/exercism/javascript/pull/634. Please merge or rebase this branch on master.

@tejasbubane rebased, thanks for the info!


Unable to disable smoothing

As far as I understood, it was possible in the previous versions to turn off the native canvas interpolation/smoothing by changing the "pixelReplication" value on the viewport, but it seems to be no longer supported (at least, I could not find the way in the documentation).
Is it possible somehow to disable smoothing? I would like to use cornerstone to develop an application for viewing MR images of mice and rats. The resolution of the images is usually relatively low, and displaying with native resolution is frequently not desirable.

这是来自QQ邮箱的假期自动回复邮件。 您好，邮件我已收到，我将在看到后第一时间给您回复。


Add ability to record and validate output of demo files

Over time the RxPHP project has added demos for various observables and
operators. From time to time the demos tend to break because of
refactoring or other changes.
This commit introduces a "test" command inside the demo folder that will
find demos that have an accompanying .expect file and ensure that
running the demo yields the same output as is expected. This will ensure
that demos keep working, but it also turns each demo into an end to end
tests for the RxPHP project.
Example usage:
$ php demo/test.php
Found 2 demos with an expect file.
[v] demo/combineLatest/combineLatest.php
[v] demo/concat/concat.php

WIP as @davidwdan is shuffling around the demo code. Once that's done I'll rebase and record the output of the demos.

"Proof" that if the contents of the expected file doesn't match the output of the demo it will break CI:
$ echo "" > demo/create/create.php.expect
$ php demo/test.php
Found 47 demos with an expect file.
...
[v] demo/count/count.php
[x] demo/create/create.php
[v] demo/defaultIfEmpty/defaultIfEmpty.php
...
$ echo $?
1


Native backup

Replace pg_basebackup with a native solution based on the replication protocol

@Jubilee101 Please, fill this out

@Jubilee101 Please, fill this out

Uhh just to make sure, you mean with commits right? Or do I need to put doc here?

It should be the brief summary of your doc

Currently pgmoneta generates backups by sending pg_basebackup command to Postgres database server. We want to rid of this dependency on Postgres binaries by implementing the process this command triggers. This roughly involves:

Sending the BASE_BACKUP command to the server.
Implementing the Streaming Replication Protocol to receive the backup and the necessary write-ahead log.
Saving the backup locally, verifying checksum and performing compression if necessary.

Hey should we close this issue? Or do we want to use this for more advanced backups as well?


Apk analysis problem on Fire TV

I can't install the better xcloud apk on my Fire TV 4K. It informs that occurs a problem in package analysis. I tried three last releases of the apk. Can anyone help me?

I download APKs on my phone and transfer them to the TV stick using an app: send fires to tv

I download APKs on my phone and transfer them to the TV stick using an app: send fires to tv

I transfer the apk from different ways. It always fail to install showing the message: "There was a problem parsing the package"

It requires Android 8+


allow autocorrect on last word on submit

The old method was when you click 'Send' we'd grab the current text value and send it and clear the input. This makes it such that if you see an autocorrect suggestion it will not apply it.
Instead we blur the input, which will apply the autocorrect, then that emits an onEndEditing, which we grab and submit instead
@keybase/react-hackers
cc: @mmaxim

@keybase/react-hackers bump


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 6810 ms

Resolved: Turnos 24x7 performance has improved in 59c4c09.


Add Async methods

Asynchronous methods are the recommended pattern for network operations. I noticed that the ExecuteRequest method was internally creating an awaiter to execute operations synchronously. My latest pull request #12 adds async methods that let the calling code control this. My client code can then queue multiple operations up to run in parallel instead of forcing serial operations.

This has been fixed in https://github.com/Invoiced/invoiced-dotnet/pull/24


Add CODEOWNERS files

This adds a GitHub CODEOWNERS file, which is supposed to automatically request reviews on every pull requested submitted to the repo.
Assuming you guys are interested in being involved in every change, this will make it easier to manage the process. I propose that by default we continue as we have up to now, i.e. wait for all three of us to review before merging (aside from trivial changes). If anyone sees that they have no time (or interest) in reviewing a particular PR, we can just leave a comment saying that, and merge without that person's approval. Does that sound like a good process?
PS I can also add the same on the http://github.com/npgsql/npgsql/ repo, let me know if that's OK for you.

PS I can also add the same on the http://github.com/npgsql/npgsql/ repo, let me know if that's OK for you.

Sure, that works for me.

This has been really nice to work with the last few days. Could we backport this to hotfix/2.1.4 and hotfix/2.2.1?

Sure thing, go ahead and commit the file directly.


added release name to README.md

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind new feature
/kind bug fix
/kind cleanup
/kind revert change
/kind design
/kind documentation
/kind enhancement

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Test Report Added?:

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind TESTED
/kind NOT-TESTED

Test Report:

Special notes for your reviewer:

Please link the issue corresponding to this fix. If no issues you can raise one and give the issue number in this PR.
If the issue os raised as a part of sodacode2022, please add #SODACODE2022 in the issue description

Alread merged another PR for same fix. Sorry. Unable to accept this. Please check other issue and contribute


[DS-4208] Fix hidden field creation issue when browsing by date

https://jira.duraspace.org/browse/DS-4208
Fixes #7548

Fixed the code conflict in this PR (since I had rights to do so).

Has approval & passes tests. Merging.  Belated thanks @terrywbrady !


Invalid Ethereum network name [Arbitrum Testnet (on Rinkeby)]

Just want to use Graph in Arbitrum Testnet , How to set the network in the subgraph.yaml
subgraph.yaml
network: Arbitrum Testnet (on Rinkeby)
✖ Failed to deploy to Graph node https://api.thegraph.com/deploy/: Invalid manifest: Invalid Ethereum network name

Try setting:
network: arbitrum-rinkeby
I hope that helps. If not, please feel free to reopen this issue.

Try setting:
network: arbitrum-rinkeby
I hope that helps. If not, please feel free to reopen this issue.

Tred network: arbitrum-rinkeby


Dashboard crashes

Hey, im trying to create a iot garden controller and found your node. The first time i installed it worked like a charm but then i stopped the node-red server and the ui crashed.If i uninstall nodejs and node-red and install them again it fixes the problem until i stop the server again.Im using the latest versions of node-red nodejs and ui time scheduler, im also running the server locally on windows 10

.

hi @postolis,
the UI can't work without the server. If you want to have access to the UI and all the UI nodes you have to have the server running.
Why do you want to stop the server?

Added another switch node but the ui didnt came back , when i restarted without the ui scheduler enabled everything came back back plus the new switch i added.Here is a screenshot with the flow and theme settings.

Please post the output of the node-red-log from the last restart with the UI scheduler enabled here.

The first screenshot is with the scheduler disabled (ui working normally)
second screenshot is with the scheduler enabled (only scheduler is visible and theme changed to default)+
third screenshot is enabling scheduler when ui worked normally(ui page didnt refreshed it self when deployed and showed everything exept the scheduler.When i refreshed the page only the scheduler was visible)

The flow :
flows.txt

I tried your flow on different systems - ubuntu local install (nodejs 14) as well locally on Windows 10 with Nodejs 16.13 and node-red 2.1.3 and rebooting works just fine. Did you try to run it in Docker yet?

Hey, no it didnt work in Docker but a week ago i upgraded my windows to 11 also upgraded node-dashboard and the ui-scheduler and since then everything runs smoothly.Been testing with restoring the schedules, stopping the server then starting it again not a single problem!


SMART FAN TOKEN

I Add the FAN with the IP Address (nothing else)
Log says ERROR

miio.0
2019-08-28 18:03:30.688
warn
XXXXXXXXXXX can not be connected.Error: Could not connect to device, token might be wrong

miio.0
2019-08-28 18:03:15.007
info
Start auto discover

SOLVED:
https://www.home-assistant.io/components/vacuum.xiaomi_miio/#retrieving-the-access-token

Close this as you can retrieving token by your self


Nerd Emoji not available

saw this passing by in the #bot_testing text channel in appu1232's selfbot server
sauce:

Added


Question about interplay between sriov-cni, sriov-network-device, multus

I was wondering if you could shed light on the exact interactions between sriov-cni plugin, the sriov-network-device plugin and multus plugin.
Based on the README example, here is my understanding of how the three plugins work together.
sriov-cni: Needs to deployed on each node. It defines deviceIDs (VFs). The config values that need to be defined are closely tied with the capabilities of the underlying node and OS. As such, whoever installs this plugin on each node needs to have this understanding.
sriov-network-device: Needs to be deployed on each node. It defines logical resourceNames. A list of selectors are provided through a ConfigMap which act as a filter to associate the defined resourceName with specific deviceIDs (VFs) available on a node. Note the deviceIDs specified in the selector need to be known to the person configuring this plugin.
multus: Defines the NetworkAttachmentCRD. When creating a Custom Resource, a resourceName needs to be provided as an annotation. The resourceName has to be one of the names defined in the ConfigMap created when configuring the sriov-network-device plugin.
Pod: Name of the multus's NetworkAttachment Custom Resource is added as an annotation on it.
Overall flow is as follows:

Setup:
(a) Figure out the VF device ids on each node.
(b) Run sriov-cni on each node by using the device ids from step (a).
(c) Run sriov-network-device plugin on each host by defining resourceNames and using
the device ids from step (a).
(d) Run multus plugin on the cluster

NetworkDefinition:
(a) Create NetworkAttachment Custom Resource. Use the resourceName defined in 1.c above as an annotation on NetworkAttachment Custom Resource.

Pod creation:
(a) Create Pod with an annotation with NetworkAttachment Custom Resource name in the annotation.

When 3.a is performed, Multus deferences the Custom Resource name to find out the resourceName. It then figures out the deviceID for that resourceName by querying/working with the sriov-network-device plugin. It associates that deviceID to the Pod.
At a high-level is this flow correct?
If so, I have one follow up question. Since the deviceID needs to be specified in the ConfigMap when installing the network device plugin, it means it has to be known to the cluster administrator. If so, what is the exact role of sriov-cni plugin in the overall flow?
Thanks!

If so, I have one follow up question. Since the deviceID needs to be specified in the ConfigMap when installing the network device plugin, it means it has to be known to the cluster administrator. If so, what is the exact role of sriov-cni plugin in the overall flow?

Pod requesting SR-IOV resource in its resource request/limit field will be allocated with a device (identified by unique device id) by kubelet, this device id information will be published by kubelet in its endpoint, then be retrieved by Multus which passes down to SR-IOV CNI.
User doesn't need to specifiy the device ID.

Thanks @zshi-redhat.
I have couple of more follow up questions, which I have outlined below.
Consider the following configuration snippet:
"resourceName": "intel_sriov_netdevice",
"selectors": {
"vendors": ["8086"],
"devices": ["154c", "10ed"],
"drivers": ["i40evf", "ixgbevf"]
}

Since a resourceName is associated with a list of devices, in what order are the devices assigned to Pods? Is it in first come first serve basis?
If for some reason, assignment of first device fails, will sriov-network-device-plugin select the next device in the list and associate that with the Pod?
What happens when none of the devices in the list are 'available' (by this I mean, say, there are two devices associated with a resourceName and those are already assigned to two Pods. What happens when a third Pod comes along that defines that resourceName in its NetworkAttachment CR?

Related to: https://github.com/intel/multus-cni/issues/425
As a reference to above question, here is the ConfigMap that I am using

Since a resourceName is associated with a list of devices, in what order are the devices assigned to Pods? Is it in first come first serve basis?

Kubelet keeps devices in a resource map internally, so devices are selected at random from available devices in resource endpoint. Selection of device is not deterministic.

If for some reason, assignment of first device fails, will sriov-network-device-plugin select the next device in the list and associate that with the Pod?

If it fails during device allocation of Pod creation phase Pod will enter into error state in my understanding.

What happens when none of the devices in the list are 'available' (by this I mean, say, there are two devices associated with a resourceName and those are already assigned to two Pods. What happens when a third Pod comes along that defines that resourceName in its NetworkAttachment CR?

Like any other allocable resources in Kubernetes, Kube-scheduler will try to find next available node that has sufficient VFs under same resource name to run this Pod. If none found Pod will enter into pending schedule state.

Related to: intel/multus-cni#425
As a reference to above question, here is the ConfigMap that I am using

Thanks @ahalim-intel for the answers. They are helpful.
I have one more question. When a Pod is deleted, I assume that the associated device is released back to the pool of available devices on that Node. So when another Pod is created afterwards that device can be bound to the new Pod. Is this correct?

Thanks @ahalim-intel for the answers. They are helpful.
I have one more question. When a Pod is deleted, I assume that the associated device is released back to the pool of available devices on that Node. So when another Pod is created afterwards that device can be bound to the new Pod. Is this correct?

correct.

closing as stale


Implement new core brand ingredients

Note: This issue is part of the greater Implement Our New Branding in NewDot project.
Design Doc: https://docs.google.com/document/d/197fXHLlHM_RJA4-3MITwdu0N5e0HX5Q74sA4W80YMD8/edit#
What is it:
Updating our old colors, logos, icons, and fonts with new brand ingredients. Updating our old button styles to look like pill shapes, and updating our text input styles to use a single bottom-border style.
We've also decided to make the dark colors the new default colorway.

Does it require a mini design doc:
Yes
Design finished:
Yes
cc @GabiHExpensify

We want this design doc to be reviewed ASAP because it's a fast change that dramatically improves our app. Also it's been lumped into WAQ (WhatsApp Quality) so the level of urgency is high (n7 to be specific).

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

I have read and reviewed this design doc

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

I have read and reviewed this design doc

I have read and reviewed this Design Doc!

Whoops, I reviewed this 2 weeks ago and didn't comment.

I have read and reviewed this Design Doc!

I have read and reviewed this Design Doc!

Still in progress! Right now we're fixing the last few dark-mode/font bugs and TestRail QA for Applause

Out of curiosity, why are we keeping this main issue open if the initial implementation is now complete? I think it's great that we already have a bunch of follow-on issues to track bugs, I just thought that everything from the initial push was already done. Is that right, and if so, how about closing this issue?

I think because we're still waiting on text inputs to get merged

Gotcha, thanks for pulling out that PR. Hopefully we can close the loop on that one when everyone returns next week, and at that point, close this individual issue out.

I'm iterating on reviews for Text Inputs daily. Hoping to see it merged soon!

I think we can close this one out since the text inputs have been merged.


AutoYaST adapted for CaaSP 5 based on SLE-15-SP2

Why is this PR needed?
AutoYaST needs to be adapted to use current versions of modules for CaaSP 5 based on SLE-15-SP2
What does this PR do?
By providing this autoyast profile a machine will be able to install SLE15-SP2 and the installation will meet all the requirements for CaaSP 5.
Anything else a reviewer needs to know?

Module for caasp/5 has not been provided yet over SCC and its name and version may change in future

To workaround the caasp/5 module unavailability apply following diff to use Devel v5 project instead of SCC channel in meantime:
@@ -205,12 +197,14 @@
         <version>15.2</version>
         <arch>x86_64</arch>
       </addon>
-      <addon>
-        <name>caasp</name>
-        <version>5</version>
-        <arch>x86_64</arch>
-        <reg_code><!-- replace this comment with a CaaSP registration code --></reg_code>
-      </addon>
     </addons>
   </suse_register>
+  <add-on>
+    <add_on_products config:type="list">
+      <listentry>
+        <media_url>http://download.suse.de/ibs/Devel:/CaaSP:/5/SLE_15_SP2/</media_url>
+        <alias>INCIDENT1</alias>
+      </listentry>
+    </add_on_products>
+  </add-on>
 </profile>

Info for QA
Confirm that the AutoYaST profile is suitable for unattended installation on bare-metal or libvirt machines.
I've successfully tested the profile with http://download.suse.de/install/SLE-15-SP2-Online-RC3/SLE-15-SP2-Online-x86_64-RC3-Media1.iso image by providing ifcfg=eth0=dhcp autoyast=http://<mymachine>/autoyast.xml installer/linuxrc bootargs.
Related PR
docs https://github.com/SUSE/doc-caasp/pull/867
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

In preparation for the transition to V5 this PR this PR has changed the target branch from master to experimental-v5. There are some opened PRs required for transitioning from v4 to v5:

#997
#1107
#1132
#1138
#1140
#1141

We want to merge them all into an experimental branch so we can actually test it all together before merging it to master.
Note this PR commits are expected to get into master along with #1137


Resolve pdf error generation again.

Reviewer
@hmunfru
Description
Detected some errors again in the pdf generation
Testing
in github

LGTM


Catch v0.15.x branch up with master ahead of release

Codecov Report

Merging #1794 into v0.15.x will increase coverage by 0.07%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           v0.15.x   #1794      +/-   ##
==========================================
+ Coverage    24.52%   24.6%   +0.07%
==========================================
  Files           98      98
  Lines         5085    5069      -16
==========================================
  Hits          1247    1247
+ Misses        3699    3682      -17
- Partials       139     140       +1

Impacted Files
Coverage Δ

pkg/api/worker_node_pool.go
12.17% <0%> (-0.22%)
:arrow_down:

pkg/api/cluster.go
0% <0%> (ø)
:arrow_up:

pkg/model/compiler.go
55.69% <50%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


zip docker tar after updating the image

https://github.com/istio/istio/issues/14581
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-shellcheck-master
2b31906a1d8fbc46d9726756c2de31f910a76a4d
link
/test istio-shellcheck-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test integ-mixer-k8s-presubmit-tests-master

istio-unit-tests-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test integ-mixer-k8s-presubmit-tests-master

istio-unit-tests-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests-master

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
2f36b8a118c6fe02e2bcba0da83d3293736fbe36
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-racetest-master


Sort by date feature doesn't work

https://search.opengazettes.org.za/search?q=eskom&offset=0&sort=oldest
https://search.opengazettes.org.za/search?q=eskom&offset=0&sort=newest
both give the same, unsorted result

I've noted this in our pivotal task around dates, it's definitely not doing what it advertises on the tin.


issue #3: lambda needs publish rights to SNS

allow lambda function to publish to SNS topics

Looks good to me.


Fix build failure of Examples.sln for branch-1.6

Examples.sln build failed on branch-1.6 due to the wrong reference of Mobius preview in csproj files

LGTM. Thanks for fixing it.

Current coverage is 82.30%

Merging #403 into branch-1.6 will not affect coverage as of 6d8254f

@@            branch-1.6   #403   diff @@
=========================================
  Files               45     45
  Stmts             4645   4645
  Branches           416    416
  Methods                     0
=========================================
  Hit               3823   3823
  Partial            191    191
  Missed             631    631

Review entire Coverage Diff as of 6d8254f

Powered by Codecov. Updated on successful CI builds.


None of your spec sources contain a spec satisfying the dependency: DrawerKit (~> 1.0)

Installing DrawerKit (0.7.0) version is available only. What happened to 1.0?

Hi, @jpbonjour thanks for flagging this. We will fix it on Monday. Thanks!

The latest release is indeed 0.7.0. We would update the docs. :)


Delay Not Working Before Stage / Action

What is your Quests version / build number (do not say "latest")?
3.8.8
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
When a delay for a stage is set the actions run before the delay is over. The actions run when the stage is initiated. (This may be similar to issue #1108.)

Unable to replicate. Please provide an example quest from quests.yml using Pastebin/Hastebin re-open.


@damassi => [Documentation] Add note about incognito browser

I've found this helpful over the last few weeks.

Oo ty 👍


Update cmake-presets-vs.md

Add instructions for configuring clang-cl with the ClangCL toolset configure preset instead of by setting CMAKE_C_COMPILER and CMAKE_CXX_COMPILER in cacheVariables.
Fixes #3446.

@sweemer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@bobbrow Do you have a delegate who can review this change and the related issue #3446? I've let this go too long without a ping.

I commented on the original issue too, but I don't believe ClangCL is a valid toolset for the Visual Studio generators.

@sweemer did using ClangCL as the toolset work for you? If so can you share your VS and CMake versions?

@sweemer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Just copying this over here from the issue:
I dug into why it wasn't working for me and it's because of microsoft/vcpkg#15623 :( After I removed vcpkg using ClangCl for the toolset works. I believe it only works for the VS generators though (based on this bit in CMake source)

@sweemer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sweemer Thanks for the contribution, and for your patience and collaboration. Thanks to @sfreed141 too, for verifying and suggestions.


feat(components.file): add an onremove callback on file input

Add an onremove callback on file input
Ref: UK-196
Description of the Change
Currently, there isn't a trigger to know if a user remove an attachment on the file input component. To handle that. I add a onRemove callback to call when an attachment is removed.
Benefits
Be able to check the attachment removal without using a watch or watchCollection
Possible Drawbacks

Applicable Issues

CDS Report yarn-build#2906.0 ✘

build & test

build ✔
test ✘


Results depend on feature scaling

Is there a reason why output of xgboost would be dependent on scaling features?
I don't have any NANs. All features are numeric. If I scale all features by some number, say 100.0, I get slightly different output from xgboost. I tried both exact and hist methods.
I tried xgboost 0.81 and 0.72.
I see that split conditions are slightly different. They start similar, but at the end trees diverge.
Foe example, here is first tree output:
{ "nodeid": 0, "depth": 0, "split": "f7", "split_condition": 0.0167161003, "yes": 1, "no": 2, "missing": 1, "children": [
{ "nodeid": 1, "depth": 1, "split": "f7", "split_condition": 0.0104448413, "yes": 3, "no": 4, "missing": 3, "children": [
{ "nodeid": 3, "depth": 2, "split": "f196", "split_condition": 0.258707613, "yes": 7, "no": 8, "missing": 7, "children": [
{ "nodeid": 7, "leaf": -0.00222015986 },
{ "nodeid": 8, "leaf": 0.011970697 }
]},
{ "nodeid": 4, "depth": 2, "split": "f195", "split_condition": 0.481782585, "yes": 9, "no": 10, "missing": 9, "children": [
{ "nodeid": 9, "leaf": -0.0198136196 },
{ "nodeid": 10, "leaf": -0.00636780029 }
]}
]},
{ "nodeid": 2, "depth": 1, "split": "f17", "split_condition": 0.00645940099, "yes": 5, "no": 6, "missing": 5, "children": [
{ "nodeid": 5, "depth": 2, "split": "f199", "split_condition": 0.48925361, "yes": 11, "no": 12, "missing": 11, "children": [
{ "nodeid": 11, "leaf": -0.035415493 },
{ "nodeid": 12, "leaf": 0.0152317053 }
]},
{ "nodeid": 6, "depth": 2, "split": "f195", "split_condition": 1.06237507, "yes": 13, "no": 14, "missing": 13, "children": [
{ "nodeid": 13, "leaf": -0.0514464751 },
{ "nodeid": 14, "leaf": 0.00441155536 }
]}
]}
]}
After scaling all features by 100.0, I get this tree:
{ "nodeid": 0, "depth": 0, "split": "f7", "split_condition": 1.67161012, "yes": 1, "no": 2, "missing": 1, "children": [
{ "nodeid": 1, "depth": 1, "split": "f7", "split_condition": 1.04448414, "yes": 3, "no": 4, "missing": 3, "children": [
{ "nodeid": 3, "depth": 2, "split": "f196", "split_condition": 25.8707619, "yes": 7, "no": 8, "missing": 7, "children": [
{ "nodeid": 7, "leaf": -0.00222015986 },
{ "nodeid": 8, "leaf": 0.011970697 }
]},
{ "nodeid": 4, "depth": 2, "split": "f195", "split_condition": 48.1782608, "yes": 9, "no": 10, "missing": 9, "children": [
{ "nodeid": 9, "leaf": -0.0198136196 },
{ "nodeid": 10, "leaf": -0.00636780029 }
]}
]},
{ "nodeid": 2, "depth": 1, "split": "f17", "split_condition": 0.645940125, "yes": 5, "no": 6, "missing": 5, "children": [
{ "nodeid": 5, "depth": 2, "split": "f199", "split_condition": 48.9253616, "yes": 11, "no": 12, "missing": 11, "children": [
{ "nodeid": 11, "leaf": -0.035415493 },
{ "nodeid": 12, "leaf": 0.0152317053 }
]},
{ "nodeid": 6, "depth": 2, "split": "f195", "split_condition": 106.237503, "yes": 13, "no": 14, "missing": 13, "children": [
{ "nodeid": 13, "leaf": -0.0514464751 },
{ "nodeid": 14, "leaf": 0.00441155536 }
]}
]}
]}

@Denisevi4 We had seen this a few times before, see #4017 for example,  one another recent discovery is #4069.  Our drawn conclusion is transformation can cause precision loss.


support for tomcat websocket session

Will existing Redisson tomcat session support tomcat websocket session too?
Tried persisting manually and encountered serialization exception.
Looking into the code it seems redisson session extends standardsession and might not support websocket session. Hence the question. Tried writing my own impl of RedissonWebocketsession, but guess my newbie status with redisson was a stumbling block. Need to learn/understand more before i attempt it.
Expected behavior
Application is supposed to store websocket session in redis. Can it be done?
Actual behavior
Steps to reproduce or test case
Redis version
Redis server v=4.0.11 sha=00000000:0 malloc=libc bits=64 build=13f90e3a88f770eb
Redisson version
3.3.0
Redisson configuration

Could you share exception stacktrace


Clean up getMirror + Fixing tests

:tophat: What? Why?
Cleaning up #57 :

Returning Self instead of object to avoid force castings
Fixing broken Tests

Also did a carthage update
:ghost: GIF

:+1:


Basic authentication

Please add it

I'm not quite sure what you mean. There is no authorization layer on sixpack server, so there is no client component. I'd assume you'd probably just want to lock down traffic from your web server to the sixpack server with Security Groups, or whatever firewall solution you use.

For example I'm using sixpack + gunicorn behind a nginx and have a basic auth on nginx. How should I connect to it in this case?
I'm curious because of the scenario with dynamic IPs (like auto provisioned servers)

With auto-provisioned servers I would assume that you have a DNS entry for something like sixpack.yourdomain.com. I'd probably avoid using basic auth with sixpack because it's going to increase connection time. You could, however, put your basic auth in the URL<EMAIL_ADDRESS>

Thanks

Just to add more details to this conversation, here is the relevant part from the Heroku documentation:

Because the Heroku dyno grid is dynamic in nature, the IP address that a given dyno will be assigned over time will be both dynamic and unpredictable. This dynamic sourcing of outbound traffic can make it difficult to integrate with APIs or make connections through firewalls that require IP-based whitelisting.

So this will require either basic auth on a proxy server or a token-based authentication inside sixpack.


1tv.ru subtitles

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Czech
Example URLs
https://www.1tv.ru/shows/kukly-naslednika-tutti/vypuski/dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024
Provide a description that is worded well enough to be understood
When open video in site player I can choose "subtitles on", and subtitles appear (russian - probably automatic subtitles). Yt-dlp says there are no subtitles.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--list-subs', 'https://www.1tv.ru/shows/kukly-naslednika-tutti/vypuski/dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [615a84447] (zip)
[debug] Python 3.11.2 (CPython x86_64 64bit) - Linux-6.1.0-17-amd64-x86_64-with-glibc2.36 (OpenSSL 3.0.11 19 Sep 2023, glibc 2.36)
[debug] exe versions: ffmpeg 6.0.1 (fdk,setts), ffprobe 6.0.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, pyxattr-0.8.1, requests-2.28.1, sqlite3-3.40.1, urllib3-1.26.12, websockets-10.4
[debug] Proxy map: {}
[debug] Request Handlers: urllib
[debug] Loaded 1803 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[1tv] Extracting URL: https://www.1tv.ru/shows/kukly-naslednika-tutti/vypuski/dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024
[1tv] dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024: Downloading webpage
[1tv] dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024: Downloading JSON metadata
[1tv] dieta-mayka-ili-posledniy-shans-pompeo-kukly-naslednika-tutti-vypusk-ot-07-02-2024: Downloading m3u8 information
[download] Downloading playlist: Диета Майка, или Последний шанс Помпео. Куклы наследника Тутти. Выпуск от 07.02.2024
[1tv] Playlist Диета Майка, или Последний шанс Помпео. Куклы наследника Тутти. Выпуск от 07.02.2024: Downloading 1 items of 1
[download] Downloading item 1 of 1
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
745012 has no subtitles
[download] Finished downloading playlist: Диета Майка, или Последний шанс Помпео. Куклы наследника Тутти. Выпуск от 07.02.2024

The extractor for the site doesn't currently look for subtitles.
Perhaps things have changed since the extractor was written, but the JSON downloaded may now include

a sources member that may list m3u8 with subtitles, m3u8, mp4, as well as the mbr member from which the video formats are currently extracted
a sources_subtitles member with a stand-alone subtitle URL (the site doesn't allow for multiple languages).

However, the Русский subtitles in the m3u8 are tagged English, which must be a defect in 1TV's playout system:
#EXT-X-MEDIA:TYPE=SUBTITLES,GROUP-ID="subs0",LANGUAGE="en",NAME="English",AUTOSELECT=YES,DEFAULT=YES,URI="index-f4.m3u8"
The stand-alone subtitles are more accurately described:
      'src': 'https://static.1tv.ru/uploads/video/material/caption/6/_original/1236_94e40d0b0b.vtt',
      'label': 'Русский',
      'track': 'ru',
      'type': 'text/vtt',
      'shift': 0

The existing code guesses a manifest URL. The HLS formats extracted from that and the two m3u8 URLs in sources appear to be duplicates, and the mp4 format in sources appears to duplicate one of the mbr formats. However the actual hostnames may differ; it's not clear whether there's any need to collect more than the MP4 formats and one set of HLS formats.

The extractor for the site doesn't currently look for subtitles.
Perhaps things have changed since the extractor was written, but the JSON downloaded may now include

Thank you for  studying problem and reply so fast.


First release cleanup

Description
First part of the cleanup task from https://github.com/terraform-ibm-modules/terraform-ibm-websphere-liberty-operator/issues/8
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Scaled down complete test cluster
reviewed WebSphere Liberty naming
Removed commented code
Fixed kubeconfig path for datasource
Removed default region for extension
Prefixed sample app URL with https:// to make it clickable when deployed through DA
Made resource groups names prefixed with random prefix with SLZ test override json

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@SirSpidey I think I addressed all your comments and questions and pushed the changes

/run pipeline

I'm fine with leaving the kubeconfig readme, I just don't understand what it does and why we would leave a link to an internal issue.

right, the internal link is to be removed, done

/run pipeline

Here scripts/confirm-wsloperator-operational.sh:

-o wide flag doesn't work in the following command:

kubectl describe Subscription ibm-websphere-liberty -n "${namespace}" -o wide

Should update examples/complete/main.tf

If we are not consuming 3 subnets, probably we can create only 1 subnet. https://github.com/terraform-ibm-modules/terraform-ibm-websphere-liberty-operator/blob/main/examples/complete/main.tf#L35
If you agree to create 1 subnet, we should create only 1 public gateway as well. https://github.com/terraform-ibm-modules/terraform-ibm-websphere-liberty-operator/blob/main/examples/complete/main.tf#L26

The title is wrong in this readme extensions/landing-zone/README.md

Need to update readme for the examples/complete/README.md.
* Not creating 3 subnets anymore.

* Not creating 3 worker pools anymore.

2 workers are the min for OCP. Reduced to 2.
All the other comments are addressed and pushed, please review

/run pipeline

a few more small suggestions

thank you for the suggestions, committed all of them, could you please review?

/run pipeline

/run pipeline

@vburckhardt @vbontempi @rajatagarwal-ibm The PR was not merged with a commit message that will generate a release. If you need a release like you indicated in the PR description, a new PR needs to be created and merged with correct prefix

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

GitHub release
v1.0.1

Your semantic-release bot :package::rocket:


Please publish this - 0.1.0

Isn't it published already? https://pub.dartlang.org/packages/dart_orm_adapter_mysql
Publishing dart_orm_adapter_mysql 0.1.0 to https://pub.dartlang.org:
|-- .gitignore
|-- CHANGELOG.md
|-- LICENSE
|-- README.md
|-- lib
|   '-- dart_orm_adapter_mysql.dart
'-- pubspec.yaml

Looks great! Are you ready to upload your package (y/n)? y
Uploading... (4.2s)
Version 0.1.0 of package dart_orm_adapter_mysql already exists..


BuildParser  doesn't use buildExtensions parameter for EBNF grammars

The following code fails to execute the buildExtensions callback.
            var parserInstance = new Parser();
            var builder = new ParserBuilder<Token, Model.Root>();
            var parser = builder.BuildParser(parserInstance, ParserType.EBNF_LL_RECURSIVE_DESCENT, "sequence", UserCreatedExtension).Result;

I've got a fix built already, I'll open a pull request and link it back to this ticket.

Ah, it looks like this is fixed in dev and the current release but not in master.
I'll close this now. @b3b00 Are you doing releases from the dev branch, rather than master?

Yes, CI (appveyor and now github actions) are releasing on version changes on dev branch. I will merge dev on master soon


Update smith_light_limitation to allow for 0 valued inputs

Specifically, if both alpha and maximum_growth_rate are 0 then the function should return 0 (instead of NaN as is currently the case).
Currently, the function raises a warning if this is the case and the user is expected to pass a small number instead. Preferably, the function just returns 0 in this case (as expected).

Change  current warn message to return 0

On this implementation:
https://github.com/agate-model/Agate.jl/blob/8a8fd74a6dabe67e1138b760f36a9e565ba4e287/examples/N2P2ZD/functions.jl#L7


ambiguous argument '0000000' when running project:diff command

> rm -r deploy_folder && sfdx sfpowerkit:project:diff --output deploy_folder --revisionfrom 6b563c8 --revisionto HEAD
fatal: ambiguous argument '0000000': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

I am on latest sfdx and plugin version.

@sfdcale  n the revision from can you  pass the full sha id and observe whether the issue persists?

@azlam-abdulsalam  Still same result.
Here is the command I tried.
> rm -rf deploy_folder && sfdx sfpowerkit:project:diff --output deploy_folder --revisionfrom 6b563c8e9034227c723706a3b322a90f0c131336 --revisionto HEAD
fatal: ambiguous argument '0000000': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

@azlam-abdulsalam  In spite of this error, diff is produced and written to the output folder but I haven't checked if the diff files are accurate.
Thanks again for creating this awesome plugin.

@sfdcale  Looks like its a message thrown by the git diff command which sfpowerkit uses. Can you confirm your directory is all good

@azlam-abdulsalam  I haven't seen this message lately and I haven't been able to reproduce. We can close this issue.

@sfdcale  thanks for the update

Just wanted to callout I am still seeing this error when running the diff command. Diff still appears to be generated correctly, not sure exactly why it's showing up.

Hello @blaskowski94 . Which version of sfpowerkit are you using? I recently fix an issue on the diff command that was occurring when files was renamed. https://github.com/Accenture/sfpowerkit/pull/210.
Could this be the cause of this issue? Can you check if the git diff command is showing renamed files?


fix(iOS): #698 Re-generate env file on each build

This is a fix for #698.
Updated the podspec's before_compile script to touch the BuildDotenvConfig.rb file so that Xcode/xcodebuild can pick up on the changes.
I am not sure if this is the ideal solution, but it works for me.
I've tested it locally, by installing the local version on a RN project.

I came to a slightly different solution based on the same idea. I simply removed the input_files and output_files entries from both of the script_phases in the podspec.
These options are optional and help XCode to determine if the output files need to be rebuilt. In this case, the input file (BuildDotenvConfig.rb) never changes, only the argument passed to it changes. And output file is already present, so it doesn't appear to need to be rebuilt. The solution in this PR works around that by making a no-op modification to the input file which causes the script phase to trigger.
I'm no expert at Ruby or iOS things, so I'm not sure which approach is better. But I would love to see a solution merged in and released soon.
For reference, here is another issue that would be properly resolved with this change.


Add web editor to video page

fix #738 Adds the option to add a web_editor

This looks really good! Thanks a lot! I'll do a little bit of testing just to make sure, but it looks like there's nothing that could go wrong.

Amazing! @nilsweb this looks great to me, feel free to merge after you've taken a look!

Works like a charm! Thank's a lot! I'll also add a short entry to the docs.


How to save the spoke light after burn wrong file as firmware?

I accidently burn a raw image file (.png) to spoke light, now the LED is off, and I can't burn another file, it seems the spoke light is bricked.
Is there anyway like "restore to factory setting" to save the spoke light? or, I must disassemble the spoke light, find something like serial port and do something via that serial port to save the spoke light?

It’s been a while since I touched this. I don't think I have ever broken it that badly, but sometimes I did upload bad firmware. I think I always managed to recover by using the reset button on the spoke light (or maybe even plugging it in) and then very quickly start the upload. It usually took a few tries, but with the right timing I could get the upload to start.
That’s the best I can offer, though… if it doesn't work for you and you investigate other ways, please do let us know here!

That’s the best I can offer, though…

Thanks for your quick response. I had revived my spoke light! I like to share it in case someone else did same thing.
How to revive YQ-8003 spoke light if wrong firmware was burned

Connect the spoke light with USB cable
Connect USB cable (another side) to computer, the spoke light does not light up any LED
There will be a /dev/ttyUSB0 device file occurs in linux OS, which is a good sign means that the spoke light is not bricked completely
Use the stcgal flash tool to try burn a correct firmware as normal
The flash procedure will stopped at Waiting for MCU, please cycle power, because the spoke light is bricked. Don't panic
Press the RESET button, a single click is enough, no need to keep pressing for several seconds.
The flash procedure should continue now, wait for a moment, and, Voilà, the light is back

What I had tried, which are not working

1st try
After I connect the spoke light to computer (or battery then power on), I tired press the reset button for at least 8 seconds (even 30+ seconds), but this does not trigger factory reset

2nd try
Press the RESET button before power on, then connect the spoke light to computer (or battery then power on), then wait for at least 8 seconds, still, it does not trigger factory reset

3rd try
Then, since there's /dev/ttyUSB0 device file occurs, I tried to use picocom to communicate to the spoke light, but I got nothing, no output, no echo when press some random keys

4th try
Then, I read the manual of the stcgal flash tool, and try it without specifying a firmware file:

root @ hpc /tmp 2022-07-16 20:41:46
# stcgal -a
Cycling power: done
Waiting for MCU: ^Cinterrupted
root @ hpc /tmp 2022-07-16 20:48:03
stcgal -r RESET
Waiting for MCU, please cycle power: ^Cinterrupted

5th try
Last try, what will happen if I press RESET button when stcgal is waiting for MCU?

root @ hpc /tmp 2022-07-16 20:48:31
# stcgal -P stc12
Waiting for MCU, please cycle power: done
Target model:
  Name: STC12LE5A60S2
  Magic: D1FE
  Code flash: 60.0 KB
  EEPROM flash: 2.0 KB
Target frequency: 22.071 MHz
Target BSL version: 6.2I
Target options:
  reset_pin_enabled=True
  low_voltage_reset=False
  oscillator_stable_delay=32768
  por_reset_delay=long
  clock_gain=high
  clock_source=external
  watchdog_por_enabled=False
  watchdog_stop_idle=True
  watchdog_prescale=256
  eeprom_erase_enabled=False
  bsl_pindetect_enabled=False
Disconnected!

Actually, I tried this for several times, each time with different key press duration.
At last, I found that it doesn't matter how long I hold the key press on RESET button, it only matter when the key press is up.

Thanks for writing that up, hopefully it will be found by and useful for whoever needs it


ViroReact v.2.5.1 that includes support for x86_64 architecture doesn’t seems to build to successfully

Tried to test the new ViroReact v.2.5.1 that includes support for x86_64 architecture. Enables the iOS simulator  But it doesn’t seems to build to successfully. We’re getting:
❌  fatal error: lipo: -remove's specified would result in an empty fat file
** BUILD FAILED **
The following commands produced analyzer issues:
Analyze Base/RCTModuleMethod.mm
(1 command with analyzer issues)
The following build commands failed:
PhaseScriptExecution [CP]\ Embed\ Pods\ Frameworks build/Build/Intermediates.noindex/projectName.build/Debug-iphonesimulator/projectName.build/Script-56B600998EAC79E7AF04CD4F.sh
(1 failure)

This issue has been fixed in our latest release, ViroReact 2.6.0. Release notes and upgrade instructions here -> https://virocore.viromedia.com/docs/3d-objects#section-loading-3d-objects
Please re-open is you are still having issues

I'm having the same issue with ViroReact 2.6.1. (Deleted node_modules before reinstalling and double checked package.json that I got the latest version)
I get these warnings during build:

⚠  ld: ignoring file /Users/kimmartinsen/Projects/Entur/entur-clients/node_modules/react-viro/ios/dist/lib/libViroReact.a, missing required architecture x86_64 in file /Users/kimmartinsen/Projects/Entur/entur-clients/node_modules/react-viro/ios/dist/lib/libViroReact.a (2 slices)
⚠  ld: ignoring file /Users/kimmartinsen/Projects/Entur/entur-clients/node_modules/react-viro/ios/dist/ViroRenderer/ViroKit.framework/ViroKit, missing required architecture x86_64 in file /Users/kimmartinsen/Projects/Entur/entur-clients/node_modules/react-viro/ios/dist/ViroRenderer/ViroKit.framework/ViroKit (2 slices)

Then finally:

❌  fatal error: lipo: -remove's specified would result in an empty fat file
** BUILD FAILED **

When checking libViroReact.a with lipo I get the following output
$ lipo -info ../../node_modules/react-viro/ios/dist/lib/libViroReact.a

Architectures in the fat file: ../../node_modules/react-viro/ios/dist/lib/libViroReact.a are: armv7 arm64

Thanks for reporting this; we've resolved the issue and the next release of ViroReact will include a Universal framework (arm64 and x86_64) for use on both device and simulator.

Sounds great, thanks! 😊

Hi @KimAM, this should be fixed in  our latest major release v2.7 -> See release notes and upgrade instructions here -> https://docs.viromedia.com/docs/releases


[Backdrop] Doesn't appear before JS is loaded

I wanted to render Backdrop server side and use it as sort of a loading indicator before the javascript is downloaded. This does not work; I guess because of the Fade wrapping it. Setting transitionDuration to 0 doesn't help either.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Thoughts on adding Fade as an inherited component for Backdrop?
  describeConformance(<Backdrop open />, () => ({
    classes,
    inheritComponent: Fade,
    mount,
    refInstanceof: window.HTMLDivElement,
    skip: [
      'componentProp',
      // react-transition-group issue
      'reactTestRenderer',
    ],
  }));

This would allow someone to traverse from: Backdrop to Fade to Transition within the API docs.
The following would be added to Backdrop's API docs:

@Josh-Weston Smart, do you want. to go for it? :)


⚠ Bofrost.lu has degraded performance

In 773b1c8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1323 ms

Resolved: Bofrost.lu performance has improved in 473d6dc after 17 minutes.


🚀 - Add Visual Basics Support

Which @codeimage/* package(s) are relevant/releated to the feature request?
Don't known / other
Description
Add support for visual basics!

Released in https://github.com/riccardoperra/codeimage/releases/tag/v1.8.2 😄


[controller][common][admin-tool] Fix two store migration issues

Problem:

During store migration, the source cluster controller sends create, update, and push store requests to the destination cluster controller. However, there is no code to check if these requests have errors. As a result, when a store update fails, we continue to push the store in the destination cluster, resulting in incorrect store and version configurations after migration.
The migrated store does not carry over system store flags (isStoreMetaSystemStoreEnabled and isDaVinciPushStatusStoreEnabled) in the parent region. During store deletion, the delete code will not clean up system store RT topics in the parent region. The root cause is that the parent controller's addVersionAndStartIngestion - addVersionOnly does not have code to set system store flags as addVersion does.

Fix:

Check for errors and throw an exception when creating, updating, and pushing stores to the destination cluster.
Set system store flags in addVersionOnly. Also skip meta system store RT topic creation in the parent region as there is no write to the parent region meta system store RT.

Other changes:

Deflake TestStoreMigration, and remove codes and configs that explicitly enable system stores since they are enabled by default.
Remove the "Optional" parameter from testStoreMetaDataUpdateFromParentToChildController. The parameter causes the entire Jenkins CI to fail when the test fails once. Even if the test is retried later, the new object's string representation for the parameter is different from the original test, causing the original test to be regarded as a failure without a retry success.

Summary, imperative, start upper case, don't end with a period

Resolves VENG-10192 and VENG-10322
How was this PR tested?

Internal CI:
JDK8 1209, 1210, 1211, 1212, 1213
JDK11 7897, 7898, 7899, 7900, 7901
No TestStoreMigration failure.
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Lets not add ticket IDs to venice OSS PRs.


Expand on Installation Instructions

This is reasonably likely to be someones first jenkins plugin install, in which case they may not be so familiar with the installation workflow.  If they're already very familiar they probably don't need to reference the readme anyways.

Build failure looks unrelated, thanks merged


IOS 13 doesn't play a notification sound when the notification is received.

Bug Information
Version Number of Plugin: 3.3.10
Device Tested On: iphone X, OS: 13.4.1
Simulator Tested On: N/A (simulators don't receive notifications)
Version of VS: VS for Mac Community, 8.6.6 (build 11)
Version of Xamarin: Xamarin.IOS <IP_ADDRESS>, Xamarin.Forms v<IP_ADDRESS>7
Versions of other things you are using:
Steps to reproduce the Behavior

Register tags/topic on the phone.
Send out notification via firebase for specific tag.
iPhone receives notification

Expected Behavior
A default sound should occur.
Actual Behavior
The notification appears on the list of notifications but no sound is played.
Code snippet
AppDelegate.cs:
        public override bool FinishedLaunching(UIApplication app, NSDictionary options)
        {
 bool fbaseStarted = false;
            try
            {
                // This method does all the UNUserNotificationCenter.Current.RequestAuthorization() code so we don't have to.
                FirebasePushNotificationManager.Initialize(options, true);
                fbaseStarted = true;
            }
            catch
            { }
            LoadApplication(new App());
            if (!fbaseStarted)
            {
                try

                {
                    FirebasePushNotificationManager.Initialize(options, true);
                }
                catch { }
            }

            // Set the default notification type and sound - not sure if this works at the moment
            FirebasePushNotificationManager.CurrentNotificationPresentationOption = UNNotificationPresentationOptions.Badge | UNNotificationPresentationOptions.Alert | UNNotificationPresentationOptions.Sound;
     }

I subscribe a list of tags (please note that I unsubscribe because the first time the code runs it fails silently if the notifications aren't approved - resulting in the model thinking the notifications was subscribed when it wasn't):
               CrossFirebasePushNotification.Current.UnsubscribeAll();
               CrossFirebasePushNotification.Current.Subscribe(Constants.NotificationTagsArray);

Screenshots

The issue actually lies with the sending of the notifications nothing to do with the Xamarin App. The issue resided in the services that sends the notifications to firebase (to then be sent out to the phones).
In the service we were sending a FirebaseNet.Messaging.Message() to the phones:
  Message FireBasemessage = new Message()
                        {
                            To = "/topics/" + PushNote.Tag,
                            TimeToLive = 86400,
                            Priority = MessagePriority.high,
                            ContentAvailable = true,

                            Notification = new AndroidNotification()
                            {

                                Tag = "/topics/" + PushNote.Tag,
                                Body = enhancedMessage,
                                Title = xtitle,
                            }
                            ,
                            Data = new Dictionary<string, string>
                                {
                                    { "param", PushNote.Tag },
                                    { "text",  enhancedMessage}
                                }
                        };

In the AndroidNotification() object required Sound = "default" to be added for it to work. Please note that this works for both Android and IOS notifications despite the fact it is an AndroidNotification object.


Transpile merged butane configs into ignition

I have separated my butane config into logical units and they are bundled back using config merging. I assumed that referenced butane configs would be stored as ignition files, but it looks like they are just the same butane files in gzip'd base64. I think that they should be converted upon read to remove this footgun.

Thanks for the report.  This is a duplicate of #118.


Proposal: allow unused_imports

The unused_imports deny is deeply annoying. There are both overlapping and non-overlapping sets of platforms that need different imports and it sucks to try and satisfy those requirements, especially since you can only really tell when an import's gone bad if CI yells at you.
I'd like to propose allowing all unused_imports. Thoughts?

cc @asomers @rtzoeller

Mmmm, now I'm actually not sure if this is a good idea. Or at least, it seems like if we allow unused_imports we should just allow any unused. Maybe that's not a bad idea? I wish there was a way to say "tell me if this is unused across all platforms and features" since that's what we actually care about.

Yeah, I wrestled with this problem too, when I first added features.  That's why I added #[allow(unused_imports)] if the default feature is enabled.  But I couldn't find a good way to say "allow imports that are used on at least one platform".  Unless we can do that, I think we need to keep denying unused imports.

Yeah, it's sad but I think I agree. Bummer.


Fix an issue when multiple files are changed at once

so that if multiple files are saved, we don't try to do too much at once.
if multiple files are saved, we should just remove all parts of the tree they affect and then just reload the root module

This is particularly painful where you may be serving files (which may be transpiled, ngAnnotated, etc) from a temp folder, where there may be a slight delay, multiple events, or a .js and .html file both loaded via SystemJS that undergo separate compilation steps with indeterminate completion times.
Is there any workaround in the meantime, or a suggested plan for a PR that may be accepted?

Tracking here -> https://github.com/alexisvincent/systemjs-hmr/issues/17


express.urlencoded() undefined extended

Hi there! I'm sorry if this isn't the place to ask, but I cannot think of how else to find this answer other than asking right on the repo. I've been googling and rereading the docs like crazy around urlencoded() and I see on the docs its "default" is set to true.
I understand I'm supposed to pass in either true or false on extended, but why do I have to if there's already a default to true? I also understand that this may change anyways in Express 5, but for Express 4, it's really bothering me without an explanation.
app.use(express.urlencoded()); produces this error: body-parser deprecated undefined extended: provide extended option
Providing the line and error for a visual example.
TL;DR Why do I have to pass in  {extended: bool} instead of leaving it empty for the default of true?
If this doesn't belong in issues, feel free to delete/close it. Just hoping someone can answer my burning question on this. I teach Express to students and this has always been hard to dance around when I'm not sure why in the first place!
TIA!

Hi @NikkiHmltn it's no problem to ask here; we get many questions in the issue tracker and just where ever.
So for your question, it comes down to a bit of history, really. Back a long time ago, Express.js started using the Qs module to parse the URL query string, and then also the urlencoded request body. But how exactly Qs worked was non-standard, and also it turned out just surprising to users. Users would end up writing applications that didn't expect urlencoded bodies to have rich values like objects and such, and it was causing applications to contain accidental security issues. This was reported to the Express.js project over the years, and though it's not a vulnerability in Express.js itself, the decision was made in conjunction with application security experts to change the default to a more "normal" version and then add this "warning" to the current Express.js such that users would explicitly state how they wanted it parsed, so they should more consciously make the decision to use the more rich "extended" parser and know they'll be expecting these rich values from user input and write their code to handle them appropriately.
I hope that helps.

That helps so much!! I deeply appreciate this answer and will pass it along! :pray:


netrw 171 (regression?) LONGLIST file size column missing left-padding with spaces

Yesterday I did an update from Ubuntu 21.10 to ubuntu 22.04. My vim got updated, and my netrw went from v170 to v171. With that change, I noticed that the LONGLIST format no longer left-pads the size information column with spaces to make each entry of the column the same width. It causes the entries of the time column to the right become unaligned. I made a feeble attempt to browse the netrw code. I am not familiar with vim scripting, so I might say or guess stuff that is wrong.
I did notice that the following definition of fsz (a left-padded-with-spaces) doesn't seem to be used? (I'm assuming the definition is scoped to the "if-block"). If it's intended to be used instead of sz, that could be why, in which case I wonder why fsz is defined before the following modification of sz in the case of human-readable units.
https://github.com/vim/vim/blob/202b4bd3a452898cfe3ed72facfbf7cb8199fa4b/runtime/autoload/netrw.vim#L11088
I'm not confident that this is actually the problem because the git blame of those lines says that they haven't been changed for several years.
Sorry if this is a noob issue. Let me know if there's any more information I can provide to help find out why this seems to have changed. I don't know how to go back at test the older version but I do have a screenshot.

before and after screenshots
before:

after (note: the size is now in human readable format ("H") because I changed it to be that way after learning that that setting exists):

@cecamp

I also noticed that if a file name is long (longer than around 32 characters), it overflows into the columns for file size and timestamp info, and then the file size and timestamp info get treated as part of the file name when opening the file, which creates a new file with that concatenated name instead of the intended file.
I suppose I should file a separate issue for this?


Standardize CI setup

https://artsyproduct.atlassian.net/browse/TRUST-216
Standardizes CI with

Readme updates
Shared orb config
auto publishing to NPM

:rocket: PR was released in v0.3.1 :rocket: