Extend mhlo::slice for Tensor3D

As previously filled in #41, we need to extend mhlo::slice for Tensor3D:
model_generated.h:251:114: error: no matching function for call to ‘slice<Tensor3D<float, 1, 882, 1> >(Tensor3D<float, 1, 882, 6>&, <brace-enclosed initializer list>, <brace-enclosed initializer list>, <brace-enclosed initializer list>)’
  251 | Tensor3D<float, 1, 882, 1> v234 = mhlo::slice<Tensor3D<float, 1, 882, 1>>(v233, {0, 0, 5}, {1, 882, 6}, {1, 1, 1});
      |                                                                                ^
In file included from model_generated.h:3,
                 from test.cpp:2:
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:498:6: note: candidate: ‘template<class Dest, class Src, typename std::enable_if<is_tensor_of_dim<1, Src, void>::value, bool>::type <anonymous> > Dest mhlo::slice(Src, Tensor1D<long int, 1>, Tensor1D<long int, 1>, Tensor1D<long int, 1>)’
  498 | Dest slice(Src x, Tensor1D<int64_t, 1> start_indices,
      |      ^~~~~
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:498:6: note:   template argument deduction/substitution failed:
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:497:64: error: no type named ‘type’ in ‘struct std::enable_if<false, bool>’
  497 | template <typename Dest, typename Src, IsTensorOfDim<1, Src> = true>
      |                                                                ^~~~
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:512:6: note: candidate: ‘template<class Dest, class Src, typename std::enable_if<is_tensor_of_dim<2, Src, void>::value, bool>::type <anonymous> > Dest mhlo::slice(Src, Tensor1D<long int, 2>, Tensor1D<long int, 2>, Tensor1D<long int, 2>)’
  512 | Dest slice(Src x, Tensor1D<int64_t, 2> start_indices,
      |      ^~~~~
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:512:6: note:   template argument deduction/substitution failed:
/home/oliver/git/mlir-emitc/include/emitc/emitc_mhlo.h:511:64: error: no type named ‘type’ in ‘struct std::enable_if<false, bool>’
  511 | template <typename Dest, typename Src, IsTensorOfDim<2, Src> = true>
      |                                                                ^~~~

Fixed with #44 merged.


Any plans on updating this repo in the future?

Eliminating the dependency on JustMock ?
Updating the RevitTestRunner?

No plans at the moment, sorry.  I have not worked with the Revit API in some time now, but if that changes, I will likely revisit this.


array insertAt BUG.

The code below is correct in the browser, but after download and save into microbit, the result is wrong.
It seems inserting at the end of array, the old data will be replace with the new data instead of inserting, and a NULL char will be append in it.
The code
let a = ['A']
function dump() {
for (let i = 0; i < a.length; i++) {
serial.writeString(a[i])
}
serial.writeLine("")
serial.writeLine(a.length + "")
}
input.onButtonPressed(Button.A, () => {
a.insertAt(0, 'a')
dump();
})
The output in MU REPL window

@dailinyang please file microbit issues in github.com/microsoft/pxt-microbit


Clarify top-level Dockerfile usage

Description
There is a Dockerfile present at repository root, that apparently is used to build a Keycloak based image.
However, this file is:

Not using an aligned base image
Could potentially fail to build (I don't see the file that is copied into the image)
Does not have any Copyright and License header
Does not have an associated "Notice for Docker image"

Please clarify if that Dockerfile is needed.
If no -> please remove it
If yes -> please fix the above mentioned issues

I additionally removed the docker-compose.yml that was using the deleted Dockerfile
75e543b


Request - hide completed iterations

Is there a way to hide completed iterations?

We show only those iterations that have any Features in them. Additionally I show 1 iteration at each side so users can extend the Feature to them if needed. You can customize the view for N number of iterations as further customization.
Please reopen the issue if this does not work for you or if you see any bugs.


[Network] Service endpoint policies

Note: This is a merge into a dedicated branch for Ignite networking work. It won't pass CI right now and I'm only looking for code review so that specific features can be reviewed in small pieces instead of a huge PR later.
This adds support for service endpoint policies and policy definitions.
Closes #6932. cc/ @chandrasekarsrinivasan

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

Closes #7104.


Timestamp for this request was 1000ms ahead of the server's time.

Hi everyone,
I am trying make an android app that uses this library. I compiled .jar file and successfully added to my android studio project.
My problem is that on some devices, when I try to make an API call, an exception like above had been throwed. Same code working on some emulator, not working on physical devices.
Any ideas? Thanks.

I had the same problem, and I pulled in the PR which has fixed the issue. There are still many other issues that will affect the stability of a long running process but they seem to be in work too.

Duplicate of #8.

Library gets time from System.currentTimeMillis(); thats is your machin time, machine time can grift a bit, so you needjust synchronise your time.
for example for windows OS you need just:
a. Click on clock and select “Change date and time settings”.
b. Click on the “Internet Time” tab.
c. Check if it is set to “synchronize the time with time.windows.com”
d. If the option is selected, click on change settings to check the option “Synchronize with an Internet Time server”
e. Click on OK.

Your problem will be solved.
public long TimeZone() {
try {
String TIME_SERVER = "time.windows.com";
NTPUDPClient timeClient = new NTPUDPClient();
InetAddress inetAddress = InetAddress.getByName(TIME_SERVER);
TimeInfo timeInfo = timeClient.getTime(inetAddress);
long returnTime = timeInfo.getMessage().getTransmitTimeStamp().getTime();
Date time = new Date(returnTime);
return time.getTime();
} catch (UnknownHostException ex) {
Logger.getLogger(ActionTradeSpot.class.getName()).log(Level.SEVERE, null, ex);
} catch (IOException ex) {
Logger.getLogger(ActionTradeSpot.class.getName()).log(Level.SEVERE, null, ex);
}
return 0;
}


How Listener  All topic？

Hi,   How can I listen to all messages without distinguishing topic

@RocketMQMessageListener(topic = "all", consumerGroup = "all")

Like this

RocketMQ doesn't  have this feature.
Why you want to reveice all topic message?


Supermium 121 crashes on Windows XP - Crash dump

          Any issue in particular? Also, can you please send any dumps created in Local Settings\Temp\Supermium\User Data\Crashpad\reports?

Originally posted by @win32ss in https://github.com/win32ss/supermium/issues/183#issuecomment-1913300430
On my Latitude E6410, Supermium 121 was slow and unstable on XP, so that's why it crashed.
Here's the dump report.
2d11ec5d-6d7c-4d13-9535-a5d6dfa4ea9f.dmp
396a49fd-1a9c-4267-8ade-264dd4e73e6f.dmp
45dd1ca7-f6d3-4d39-99f8-be5488de704a.dmp
5632670d-d8ef-461a-aa8b-e7b0e20d879f.dmp
59cd09e4-f0d1-48c2-acae-4539cc8bad1d.dmp
5c8f6be6-f31c-4158-8991-ab824a7ff065.dmp
5e0191d3-4564-4e7a-9b00-16a0a119c172.dmp
61d6e360-776b-41ec-836f-675066bd143d.dmp
6575d104-12a7-4f30-956d-e4bb6042688e.dmp
67ece48d-d64e-45bc-af10-02a46a874f48.dmp
6860cb21-3a77-487d-a103-06c2f9e07daa.dmp
68ff09fa-f2c1-49c5-ad4e-131011c2b1bb.dmp
6acd9d10-5ae6-4db1-81d2-b9901a2535d9.dmp
6ba96351-bc4a-4b93-bea0-c7f7b6192fa6.dmp
73e1c1f9-690a-4402-8dd4-e7d8edb75515.dmp
2da23ad1-ae26-44ca-81e7-dc7d770e250b.dmp
34044009-7d90-4b1d-8dc8-3c8da697d47c.dmp
34360bc6-9b7c-46a3-9d6d-0d0f5e5e0ed9.dmp
39689913-47e5-4266-b300-3aa3c5cfd472.dmp
0bd6f5fd-42c9-4a08-a313-03c719def330.dmp
10e22af7-f8c2-4942-afa6-a468bbb5e4a7.dmp
147f31b4-5624-429c-a2e7-ba3f65ecee6e.dmp
1c55df6b-8fd8-4e25-8a8f-26e29bcc6f7a.dmp
1e1509d9-8dd9-47ee-b909-8b346042bc46.dmp
21907383-b786-4900-94ad-3bf9f8fd6030.dmp
25b08c3c-93f7-44ee-8ede-210734ac0cbd.dmp
74204c70-159d-4370-bd3f-f811e83cbec6.dmp
771ec867-53d1-48a7-b0f7-f0ab869934ed.dmp
8196a87a-a917-4f73-a0bf-41dea01b29b8.dmp
852aa823-ed3a-47eb-9bf6-fbaafadecaf5.dmp
86761055-8282-47e9-9b39-fa5009e2fa41.dmp
8ec84dec-7cb8-4a62-801b-0f2748921973.dmp
8f7059f9-7373-46da-b643-dbdcfe67b4f7.dmp
913df965-659d-4fb2-96ad-6360f6d2fbe7.dmp
94a41732-27cc-470b-8ad4-2f497d3856a0.dmp
96e467a5-a857-467c-96c3-7bdfed2ff9da.dmp
978eeed2-a2cd-4408-9053-49a235a3e0b7.dmp
97a831ac-c57c-4845-ba9c-0d1da4900415.dmp
99cdfe42-f91f-4d28-8ece-402d46cde9a1.dmp
a1d57bca-a369-46fc-9280-b0aaa137d8bc.dmp
a4d6f0d9-e8bd-4332-b3e0-788b5ac2ad00.dmp
a6dc0356-144a-477d-8d76-c2942be70109.dmp
abad0a0b-4fd0-419a-9964-0fcf1c97655d.dmp
aeeb1a55-f1ce-4ada-b2db-0145f0951630.dmp
af4d67d2-5334-405e-a280-ce366a1cd175.dmp
b30eec33-ee71-4576-8367-01636b9107a7.dmp
bc123bbf-6dd5-4af0-9c99-7b106e5d958c.dmp
c2271662-81e2-4f74-9787-cfbe2c54806d.dmp
c399a967-a773-4c83-b064-49480d86e9a2.dmp
c6658018-7945-4889-849d-4397daefa663.dmp
ce7f166b-1c19-45ab-a441-390d58cdb85e.dmp
d0a7e292-f186-4737-a866-78029424665c.dmp
d3d3c384-2e3f-4091-96c9-f97654f7c603.dmp
d3f38256-7d99-4f4c-a895-bda8b52a34cc.dmp
d795cb21-7afe-4cf0-b8e2-f221e8200557.dmp
da2f0f7b-9ff2-4328-acaf-570e20238d97.dmp
dace6e37-6a9f-4ebc-9daf-236f3ad21acf.dmp
e01b10a3-0019-4529-86a7-c05d23f7b9e1.dmp
02f21491-a303-4679-88ce-857a427fb10f.dmp
04e2903e-b4af-4725-b985-59c51920f724.dmp
051ef4cc-fe04-440f-aed2-948ecb816d72.dmp
076ea6db-ee7c-457b-ab8d-965304a7fec5.dmp
09d2a727-04c4-4476-8cf5-b390eb4b6e97.dmp
e7e8db16-efd5-46bd-b9e9-e037168f5e42.dmp
f0397720-756e-413b-a479-c5a30508929c.dmp
f38a53df-be72-4c35-bd09-93775b6377a4.dmp
f8dd4a52-3a0b-4570-9027-f237519f68fa.dmp
fba6848e-6282-472e-9557-d1bbc56967a9.dmp
fd8229d6-2a0a-4a2b-ae78-72a37e952cc4.dmp
ff75d998-fe30-4d21-97bf-819057ccff6f.dmp

No reply, huh??

Unfortunately, I am somewhat busy. I was intending on producing a special debug version of progwrp.dll to confirm the presence of a particular issue, but other events occurred preventing me from doing so.
Sometime tomorrow I should be able to produce it.

Sure!!

It appears to be a memory allocation issue in the field trial list (which is pointless to Supermium and will be curtailed). How much memory do you have?

3.42 GB of RAM, and a hard drive of 232 GB. Right now, I have 171 GB of free space.
I tried that switch, it worked for a little, but it was still slow and laggy.

Unfortunately, I am somewhat busy. I was intending on producing a special debug version of progwrp.dll to confirm the presence of a particular issue on XP, but other events occurred preventing me from doing so.
Sometime tomorrow I should be able to produce it.

Hi. I see that "progwrp.dll" is your API adapter for kernel32.dll of "legacy systems" like XP. Is this your private closed project or is it aslo open-source and we can see/debug the code ?

Ok. Can we get PDB for it, if no code available ?
I want to debug/profile some issues, targeted thread synchronization routines, like SRW-emulation API etc.

Unfortunately, I am somewhat busy. I was intending on producing a special debug version of progwrp.dll to confirm the presence of a particular issue on XP, but other events occurred preventing me from doing so.
Sometime tomorrow I should be able to produce it.

Hi. I see that "progwrp.dll" is your API adapter for kernel32.dll of "legacy systems" like XP. Is this your private closed project or is it aslo open-source and we can see/debug the code ?

It is closed at this time. It's actually far from being mature and Supermium for Vista and below is the "pilot" for it. The scope of progwrp will increase in the future.

Is there a possibility of extending it to XP (x86) "SP2" also? That would be super!


fix: macchiato flavor

The macchiato flavor had the mocha colors.

... i did not do a very good job. LGTM anyway


Wrong geometric intrepretation of computational basis

Describe the issue
In the Qiskit online textbook -- Introduction to Quantum Computing. In the index "Amplitudes" of "Describing quantum computers", the textbook gives a geometric relation between these standard basis in a Hilbert space. In the picture, basis |-> directed to the wrong direction.

Describe what you expect to see

I can work on this

@themathqueen sorry I missed this comment. I've just made a PR using the solution proposed in #1790. There are two issues for the same problem so I'm going to close this in favour of #1790.


[Event Request] page 401 "Purch. Credit Memo Statistics" - OnCalculateTotalsOnAfterAddLineTotals

We would like to add a parameter to the existing event OnCalculateTotalsOnAfterAddLineTotals in the following procedure:
local procedure CalculateTotals()
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeCalculateTotals(
        Rec, VendAmount, AmountInclVAT, InvDiscAmount,
        LineQty, TotalNetWeight, TotalGrossWeight, TotalVolume, TotalParcels, IsHandled);
    if IsHandled then
        exit;

    PurchCrMemoLine.SetRange("Document No.", "No.");
    if PurchCrMemoLine.Find('-') then
        repeat
            VendAmount += PurchCrMemoLine.Amount;
            AmountInclVAT += PurchCrMemoLine."Amount Including VAT";
            if "Prices Including VAT" then
                InvDiscAmount += PurchCrMemoLine."Inv. Discount Amount" / (1 + PurchCrMemoLine."VAT %" / 100)
            else
                InvDiscAmount += PurchCrMemoLine."Inv. Discount Amount";
            LineQty += PurchCrMemoLine.Quantity;
            TotalNetWeight += PurchCrMemoLine.Quantity * PurchCrMemoLine."Net Weight";
            TotalGrossWeight += PurchCrMemoLine.Quantity * PurchCrMemoLine."Gross Weight";
            TotalVolume += PurchCrMemoLine.Quantity * PurchCrMemoLine."Unit Volume";
            if PurchCrMemoLine."Units per Parcel" > 0 then
                TotalParcels += Round(PurchCrMemoLine.Quantity / PurchCrMemoLine."Units per Parcel", 1, '>');
            if PurchCrMemoLine."VAT %" <> VATPercentage then
                if VATPercentage = 0 then
                    VATPercentage := PurchCrMemoLine."VAT %"
                else
                    VATPercentage := -1;

            OnCalculateTotalsOnAfterAddLineTotals(
                Rec, //A108F0039.Event
                PurchCrMemoLine, VendAmount, AmountInclVAT, InvDiscAmount,
                LineQty, TotalNetWeight, TotalGrossWeight, TotalVolume, TotalParcels, VATPercentage)
        until PurchCrMemoLine.Next() = 0;
end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnCalculateTotalsOnAfterAddLineTotals(PurchCrMemoHeader: Record "Purch. Cr. Memo Hdr."; var PurchCrMemoLine: Record "Purch. Cr. Memo Line"; var VendAmount: Decimal; var AmountInclVAT: Decimal; var InvDiscAmount: Decimal; var LineQty: Decimal; var TotalNetWeight: Decimal; var TotalGrossWeight: Decimal; var TotalVolume: Decimal; var TotalParcels: Decimal; var VATPercentage: Decimal)
begin
    //A108F0039.Event Parameter PurchCrMemoHeader
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[move] Added u modules

Description

Added various primitive defining modules for integers.
Particularly useful with method syntax

Test plan

TODO

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

are you missing running the script for the snapshots?


Lab 5,.3 - exercise 3 -- there is no vendor called first up consultants

Module: 5.3
Lab/Demo: 3
Task: 1
Step: 1
Description of issue

Please update the lab with a vendor that is actually on the list: V9005 Fabrikam maybe?

also on this lab - there is an error when trying to bank > export... but it is not clear how to fix this.

Hi,
On the Business Central Home page, Click on search icon and Enter Bank Account and click on related link. Select the EUROP account and enter a value for Credit transfer msg field.

that's great. thank you.


Yihan Wang Worked with Xiayu Zeng

Writeup
Which parts of the lab specification have you found most confusing or difficult to understand? Be specific, and quote any parts of the lab description relevant to your answer (e.g., "modify the PIO/DMA logic analyzer example to record a timestamped trace of an RP2040-ADPS9960 exchange while the BOOT button is pressed.") I will make particular notice of anything that seems underspecified (e.g., words whose meaning is not clear from context, or conflicting interpretations of a deliverable).
The instruction is readable.
Which lab topics have you found most confusing or difficult to understand? E.g., "serial communication with Python," "aliased bitwise operations," "programming the PIO," etc. Be specific, and describe any lingering areas of confusion and/or anything that has helped you navigate them.
It's hard for me to understand the I/O operation with just reading the official document and without much explanation. Also the PIO made me confused as I have some misunderstanding of it when I do part 8.
Which parts of the lab have you found most difficult to implement? Again, be specific, citing any parts of the lab materials & your own code that are relevant to your answer.
I didn't used C language for many years so it's hard for me to implement my thinking in VS. I have to search a lot for the usage of C  language in google for both understanding the example and try my thinking.
What steps have you taken to resolve these difficulties? Any other barriers you have faced to completing this assignment? Mention any students, repos, or other resources you have found helpful in completing the lab so far.
I really have to say thanks to my lab partner. He helped me a lot with understanding the example and writing down the program. And the resource on github really helped a lot.


Refactor part of Wallet_Activate into VerifyIdentity

Hold on: https://github.com/Expensify/Web-Expensify/pull/34343
Details

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/218501
Tests

Pre-requisites:

Have an account with a pending wallet (all new accounts have a pending wallet created automatically)
Get access to LexisNexis as this service is required on the AdditionalDetails Step which you'll need to complete before moving forward to the OnfidoStep:

Run the following commands:

nslookup webapi.libertydata.net
nslookup wsonline.seisint.com

Each one will return one or more address fields. For each of those, run:
sudo route add -net ADDRESS/32 <IP_ADDRESS>

For the three addresses I got back, this means the actual commands I run are:
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>

Now, you'll need to mockup a successful response from Idology.
Note: To retest with the same wallet bank account, run the following to reset the KYC flow:

UPDATE bankAccounts SET additionalData = JSON_SET(COALESCE(NULLIF(additionalData, ''), '{}'), '$.verifications', JSON('{}')) WHERE bankAccountID = 40;

UPDATE bankAccounts SET additionalData = JSON_SET(COALESCE(NULLIF(additionalData, ''), '{}'), '$.currentStep', 'AdditionalDetailsStep') WHERE bankAccountID = 40;

Tests:

Navigate to /enable-payments, fill out the additional details form
You'll now see the Onfido Privacy page and will be taken through the Onfido flow as shown below. (Note: proceed through the flow without using any of your own ID if you can avoid it - Onfido recommends to not use real info)

Verify you can complete the process successfully:

If the check passes or wasn't able to complete synchronously, you'll be directed to the Terms page.

If the check didn't pass and you had some fixable errors, those errors will be displayed and you can retry the flow again. You can mock some fixable errors by setting $didPass to false and $onfidoFixableErrors to:

$onfidoFixableErrors = [Onfido_API::ERROR_IMAGE_QUALITY, Onfido_API::ERROR_SELFIE];

To mock fixable errors in Onyx:
let data = {
    errors: {<PHONE_NUMBER>299659: "Sorry we're unable to verify your identity. Please fix the errors before continuing." },
    fixableErrors: [
        "There's an issue with your selfie/video. Please upload a new selfie/video in real time."
    ]
};
Onyx.merge('walletOnfido', data);

If the check failed, you'll be directed to the failed KYC page. You can mock this failure by throwing:

throw new IdentityVerificationError('3D9B8AC1-C2C0-49CC-B817-7660A0E72EE7', IdentityVerificationError::VERIFICATION_ERROR]);

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] Any functional components have the displayName property
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps

None. We'll test the entire KYC process internally once all the steps have been refactored.

Screenshots

Web
Added in test steps
Mobile Web

Conflicts fixed. On hold till the Web PR goes out to prod but this is ready for reviews @marcochavezf @marcaaron @francoisl!

Off hold!

The code LGTM, I can't test it locally because for some reason I'm not getting the Verify Identity page yet.

For the three addresses I got back, this means the actual commands I run are:
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>
sudo route add -net <IP_ADDRESS>/32 <IP_ADDRESS>

Not sure it's related to the problem, but I got only two addresses in the pre-requisite steps.

Also I found two issues that I think are not related to the PR, but when I try to fill out the Additional details (which is the first page that appears when I navigate to /enable-payments) the server error is not displayed in the front-end:

Not sure it's related to the problem, but I got only two addresses in the pre-requisite steps.

I only had two addresses as well, and it worked. Is it possible you didn't mock the response from IDology here, and it failed at that step? Alternatively, you might be able to skip directly to the Onfido step by running thise query (replace the bankAccountID with your wallet's ID):
UPDATE bankAccounts SET additionalData = JSON_SET(COALESCE(NULLIF(additionalData, ''), '{}'), '$.currentStep', 'OnfidoStep') WHERE bankAccountID = XXX;

Alternatively, you might be able to skip directly to the Onfido step by running this query (replace the bankAccountID with your wallet's ID):
UPDATE bankAccounts SET additionalData = JSON_SET(COALESCE(NULLIF(additionalData, ''), '{}'), '$.currentStep', 'OnfidoStep') WHERE bankAccountID = XXX;

Thanks this helped! Also, I found in this SO that I had to enter 3333 in the SSN field to continue 😄

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @francoisl in version: 1.1.89-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Private endpoints

The proposed syntax is that a private endpoint still requires a [[trigger.http]], but has route = false instead of an actual route pattern.  (This creates the slight idiosyncrasy that route = false is allowed but route = true is not.  I can live with this!)
In addition to unit tests, the front-to-back service chaining test is modified to make the middle component a private endpoint.  Let me know if you think further tests are needed or if they should be reorganised to be more specific.

This is great!
Although it is already merged, I would also prefer using a property that reflects visibility as suggested by @fibonacci1729
Some additional ideas:

private = true
internal = true
public = false


Include a section on updating packages and R

See comments here.

@gcperk I think this is you?

Yes this is me - I just checked and there is already some text in lesson 01.


build 1.1.0 package for Python 3.7

This PR is trying to package current stable version for Python 3.7
CC @krischer @ThomasLecocq

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Whenever possible python packages should use pip. See https://conda-forge.org/docs/meta.html#use-pip

@conda-forge-admin, please rerender

@ocefpaf Ok thanks, I was hoping to just use the setup that worked for the old build and add py37 on top, but it looks like there were massive changes. So yeah, I guess we might as well adapt to them right now..

Hmm.. ok, I'll remove that restriction to Py37, I guess, since CI is not building anything right now seems like.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The requirements/ sections should be defined in the following order: build, host, run; instead saw: host, build, run.

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

@conda-forge-admin, please rerender

Somehow the recommended build command line errors out with a jinja error on Travis. I'm tempted to switch back to the seemingly discouraged python setup.py ..  command that used to work?

Awesome!
I'm here for support

Somehow the recommended build command line errors out with a jinja error on Travis. I'm tempted to switch back to the seemingly discouraged python setup.py .. command that used to work?

You can use the pip without the {{ PYTHON }} and just use python
I'm not sure why PYTHON isn't getting set. but you can still use pip

script: "{{ PYTHON }} -m pip install . --no-deps --ignore-installed --no-cache-dir -vvv"
to
script: "python -m pip install . --no-deps --ignore-installed --no-cache-dir -vvv"

I think the current packaging problems have to do with the newly added host subsection (inside requirements) in the recipe.. It's pretty confusing to me, since how I read it pretty much everything that used to be in the build section now should go in the host section, even though there still is a build section that now should contain other stuff exclusively..
Maybe that also explains why it couldn't resolve {{ PYTHON }} earlier.. dunno

https://conda.io/docs/user-guide/tasks/build-packages/define-metadata.html#requirements-section
https://travis-ci.org/conda-forge/obspy-feedstock/jobs/437134534#L3467

Let's see if these commits help..
@dsentinel you're very welcome to join the packaging fun.. you should have the rights to push to this branch on obspy/obspy-feedstock now (sent an invite)?

Ooof.. these were really big changes in conda-smithy it seems, keep running into more problems. Now the rerender created some numpy version combinations automatically that can't be resolved even during building. Dunno if we really have to fix sane numpy versions through this new conda_build_config.yaml mechanism: https://conda.io/docs/user-guide/tasks/build-packages/variants.html#creating-conda-build-variant-config-files
I'm stopping here for now @dsentinel feel free to pick up fixing the build.

Test reports are a real mess, mostly due to some changes in how numpy prints (doctests).. but it looks OK enough to be bold and merge this one..
http://tests.obspy.org/?version=1.1.0&node=conda-forge
CC @lijunzh


Unexpected error fetching Luftentfeuchter data: The length of the provided data is not a multiple of the block length.

Version of the custom_component

v0.8.0
Configuration

Describe the bug
After restarting HA I see in the log file the following error. Nevertheless it seems that the dehumidifier is working even with this error happening.
Debug log

Logger: custom_components.midea_dehumidifier_lan.appliance_coordinator
Source: custom_components/midea_dehumidifier_lan/appliance_coordinator.py:132
Integration: Midea Air Appliances (LAN) ([documentation](https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/blob/main/README.md), [issues](https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/issues))
First occurred: 13:24:32 (1 occurrences)
Last logged: 13:24:32

Unexpected error fetching Luftentfeuchter data: The length of the provided data is not a multiple of the block length.
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 190, in _async_refresh
    self.data = await self._async_update_data()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 150, in _async_update_data
    return await self.update_method()
  File "/config/custom_components/midea_dehumidifier_lan/appliance_coordinator.py", line 107, in _async_appliance_refresh
    await self._async_do_update()
  File "/config/custom_components/midea_dehumidifier_lan/appliance_coordinator.py", line 132, in _async_do_update
    await self.hass.async_add_executor_job(self.appliance.apply, self._cloud())
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 696, in apply
    responses = self.appliance_send(data)
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 679, in appliance_send
    return self._appliance_send_8370(data)
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 590, in _appliance_send_8370
    if packets := self._retry_send(original_data, response_buf):
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 664, in _retry_send
    packets = self.appliance_send(data)
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 679, in appliance_send
    return self._appliance_send_8370(data)
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/lan.py", line 605, in _appliance_send_8370
    response = self._security.aes_decrypt(response[40:-16])
  File "/usr/local/lib/python3.9/site-packages/midea_beautiful/crypto.py", line 351, in aes_decrypt
    decrypted = decryptor.update(raw) + decryptor.finalize()
  File "/usr/local/lib/python3.9/site-packages/cryptography/hazmat/primitives/ciphers/base.py", line 140, in finalize
    data = self._ctx.finalize()
  File "/usr/local/lib/python3.9/site-packages/cryptography/hazmat/backends/openssl/ciphers.py", line 215, in finalize
    raise ValueError(
ValueError: The length of the provided data is not a multiple of the block length.

Thanks for report @u20p17, is this happening often or only once?

Happens from time to time:

Hi @u20p17 , latest version has some additional logging when using debug mode. If you still have the issue can you see if you get more information in the logs.

Hi, since the latest update I do not get the error anymore - I will let you know if it appears again :)


Add narrative documentation for NIO's concurrency bridges

Motivation
We started to add bridges from NIO to Swift Concurrency and we should provide some narrative documentation for our users how to adopt them.
Modification
Adds a new article that explains the NIO concurrency bridges.

@gjcairo @Lukasa @glbrntt Thanks for the great feedback. I fixed all the nits and rearranged some sections to have a better story line. I also added a general guidance section in the end. Right now that only contains guidance around business logic vs protocol logic but we can extend it in the future with guidance around perf as well

Merging over the nightly failure which is an allocation regression


Update cert/key file names

Update cert/key file names to match the correct names generated by the script

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

#label:"aq-pr-triaged"

Thank you @wickste
#sign-off


Document and refactoring of loadOnInit

After the amazing PR of @flaviogranero it's time to push this feature on production.


:memo: added ideas for metrics in Compass. All work in progress.

@imlucas, I wrote down some ideas about improving metrics and a list of possible events I could think of for Compass and what data they should send. Have a look at ./docs/spec.md too, perhaps you can continue the design and/or start implementing some of this if you think this could work.

@rueckstiess Found an amazing comparison/what to aspire to for this module.... http://www.metabase.com/docs/v0.13.1/information-collection.html#events


add REPL to the startup

Closed by 06e99f8.

When we update the local docker build and the syzygy, don't forget to include these in there as well.


Potential bug in _compute_sampled_logits when num_true > 1

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Mojave (10.14.6)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.14.0
Python version: 3.6.4
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
When logits are computed in _compute_sampled_logits for the evaluation of tf.nn.nce_loss, the output related to the true labels from out_labels might be wrong.
Let me provide an example. Assume batch_size = 2, num_true = 1000, num_sampled = 20, then out_labels will be a Tensor of size 2 x 1020 (1020 is the addition of num_true and num_sampled).
According to the following line:
array_ops.ones_like(true_logits) / num_true (one of the last lines of _compute_sampled_logits function),
the first 1000 elements in each row of out_labels are: 1./1000 = 0.001, while the last 20 are 0.
The potential problem is that the output of _compute_sampled_logits is then fed to a sigmoid cross entropy (not softmax cross entropy which requires the labels to sum to 1 in each row). As a result, labels of an example are independent to each other. This means we can have as many labels as possible for each example, and labels do not necessarily have to sum to 1 (for each example).
By dividing array_ops.ones_like(true_logits) by num_true, the weights for the first 1000 (positive) labels of each example would try to produce a y_hat that would be close to 0.001 instead of 1. Predictions, which are generated via a sigmoid function, represent the probability that a given label is positive (for this example). Let's call this prediction, y_hat for simplicity. Due to the above formulation, y_hat would try to be as close to value 1/num_true (instead of 1) for labels that appeared with the corresponding x feature.
Describe the expected behavior
The expected behavior would be that all positive labels in each example be assigned value 1 instead of 1/num_true. This way, the sigmoid which applies independently to each label will try to produce a y_hat for each (positive) label close to 1 instead of 1/num_true.
The solution would be:
   out_labels = array_ops.concat([array_ops.ones_like(true_logits), array_ops.zeros_like(sampled_logits)], 1)
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import tensorflow as tf
import pandas as pd
import numpy as np
import math

batch_size = 4
num_classes = 2000
dim = 16
num_sampled = 20
num_true = 1000
sampled_values = None
remove_accidental_hits = False
partition_strategy = 'div'

weights = tf.get_variable('weights', initializer=tf.truncated_normal([num_classes, dim], stddev=1.0 / math.sqrt(dim)), trainable=True)
biases = tf.get_variable('biases', initializer=tf.zeros([num_classes]), trainable=True)
inputs = tf.random_normal([batch_size, dim])
df = pd.DataFrame({'labels': [np.array([0, 2], np.int32), np.array([1], np.int32),
                              np.array([6, 4, 1, 0, 3, 2], np.int32), np.array([5, 0, 2, 1, 6, 3, 9, 4], np.int32)]})
labels = tf.keras.preprocessing.sequence.pad_sequences(df['labels'].values, num_true, dtype=np.int64, value=0)
labels = tf.constant(labels)

logits, labels = _compute_sampled_logits(
    weights=weights,
    biases=biases,
    labels=labels,
    inputs=inputs,
    num_sampled=num_sampled,
    num_classes=num_classes,
    num_true=num_true,
    sampled_values=sampled_values,
    subtract_log_q=True,
    remove_accidental_hits=remove_accidental_hits,
    partition_strategy=partition_strategy)

Here, _compute_sampled_logits is from tensorflow.python.ops.nn_impl.

Looks like code is incomplete. In order to expedite the trouble-shooting process, please provide a minimal standalone code to reproduce the issue reported here. Thanks!

Hi @ravikyram thank you very much for your response! Thinking about it a bit more carefully, the code might probably be correct if you output predictions via a softmax function.
To extend a little bit more on what I meant above, let's assume inputs is a batch_size x 1024 matrix, nce_weights is num_classes x 1024 and nce_biases is num_classes x 1.
Then, all_logits = tf.nn.bias_add(tf.matmul(inputs, nce_weights, transpose_b=True), nce_biases)
is a batch_size x num_classes matrix. An element all_logits[i,j] represents the logit for label j (of example i).
For each row of all_logits we select num_true columns that correspond to the true labels of this example (row). If, for example, the first row (example) is comprised of num_true = 3 labels, which are 1, 4, 6, we will select the 1st, 4th, 6th column from the first row of all_logits (we assume here that columns start indexed at zero). These are the true_logits. In addition, we select num_sampled columns from each row of all_logits representing the negative label (0). These are the sampled_logits.
out_logits, which is concatenation of true_logits and sampled_logits, is a batch_size x (num_true + num_sampled) matrix. These out_logits are aligned with out_labels as follows:
out_labels = [[1./num_true  1./num_true  1./num_true ... 1./num_true 0 0 ... 0],
              [1./num_true  1./num_true  1./num_true ... 1./num_true 0 0 ... 0],
               ...
              [1./num_true  1./num_true  1./num_true ... 1./num_true 0 0 ... 0]]

An element of out_logits corresponding to a true label would try to match 1./num_true via the sigmoid: 1./(1 + exp(-out_logits)) ≈ 1./num_true.
Now, if we used a sigmoid function to retrieve predictions later on (and assuming the NCE loss did a good job bringing out_logits close to out_labels), the result of the prediction might have been deceptive. The predicted value for a label would be 1./(1 + exp(-out_logits)) and its maximum value (in places of a true label) would have been close to 1./num_true (that is, far from 1). However, if we retrieve predictions via a softmax function instead, then the max value (in places of a true label) would be something close to 1./num_true, which is ok since softmax imposes a valid distribution over the num_classes labels.


Update docs with the newer MDOP version (May 2019)

Please note: The URL you are suggesting to add to this page (https://support.microsoft.com/help/4505175/may-2019-servicing-release-for-microsoft-desktop-optimization-pack) seems to contain a broken link to the download page for the MDOP package for 2019. The URL https://www.microsoft.com/downloads/details.aspx?FamilyId=2afccd89-6f66-4396-a5a4-7819d37bbe01 gives me "404 - File or directory not found." all the time, at least during the last hour. Not sure if the link will be revived anytime soon.
Functional link to the download page: https://www.microsoft.com/download/details.aspx?id=58345

@illfated thank you for noticing that, will put in a separate request to update the KB itself. This PR is good as is.

Sounds good to me. Thank you for your informative reply. 👍

The link in 4505175 is updated. Thanks.

@simonxjx thank you very much.  :D

ty


Fix typo in mbedtls_ssl_set_bio description.

Description
Description referred to mbedtls_ssl_sent_t callback,
but the callback is named mbedtls_ssl_send_t.
Signed-off-by: Christopher Moynihan<EMAIL_ADDRESS>Requires Backporting
NO
Migrations
NO
Todos

[x] Changelog updated

Good point! It's the the Law of Conservation of Typos.

Considering the PR is only correcting a typo in the documentation, the CI failures look unrelated.

The continuous-integration/jenkins/pr-merge job also completed, even though it seems stuck on "in progress" on Github.

Actually, this typo is present in both the mbedlts-2.7 and mbedtls-2.16 LTS branches. If you could backport it, @cm6n , that whould be great (A cherry-pick should apply cleanly).

Since they were trivial, I did the backports myself: #3660, #3661 .

Thank you Bence!

All Jenkins tests except the Mbed OS ones passed. The Travis ones only failed because it timed out, but it is a subset of Jenkins, so this should be ready to merge.

pr-merge had glitched. A rerun of pr-merge passed, but didn't report to GitHub properly.
Travis had glitched, so I reran it yesterday. I triggered a rebuild from the Travis interface. Apparently, this reused the same .travis.yml as the previous run on that PR, but it checked out a refreshed merge (the target branch had changed in the meantime). One of the changes on development was the Travis job “check compilation guards” started taking much longer (due to testing more configurations), too long for a Travis job. So we removed that job from Travis (it's still running on Jenkins). But because this was a rebuild and not a new GitHub trigger event, Travis didn't take parse the new .travis.yml, and therefore it ran the “check compilation guards”. This is an edge case due to using the “rebuild” button on Travis, not a problem we can expect to run into on a regular basis.


AutoDMP replication clarification

Hi team! I was wondering: when you ran the experiments on AutoDMP (ariane_ng45) did you convert it to Bookshelf format or were you able to get it working with just LEF/DEF/Verilog formats provided here?

Addressed by AutoDMP author at: https://github.com/NVlabs/AutoDMP/issues/5


Clicking on DraftJS Kills Meteor's <Accounts.ui.LoginForm />

I'm really not sure what is happening here.  When I click on the "DraftJS" contentEditable - the <Accounts.ui.LoginForm /> instantly disappears.  I haven't the foggiest idea why.
Demo:

To see the bug in its current state:
Go to: http://draftjsmeteor.autoschematic.com/
To duplicate the problem in your own development environment:

Clone this repository: https://github.com/JeremyIglehart/DraftJSMeteor
Create a settings-development.json file (you can leave it blank)
Run with npm start

The main question:
Does anyone know why DraftJS kills the <Accounts.ui.LoginForm />?
After looking into this problem for two days now, I suspect the problem is hiding somewhere in how  <Accounts.ui.LoginForm /> is doing something - perhaps in their STATES API?  I'm really confused here.  Any help would really be appreciated.
Problems I've ruled out:

It's not a Session Variable problem.  DraftJS doesn't use them - in fact, there are no Session Variables being used at all right now as far as I can detect (using Meteor Toys)
It is not DraftJS killing the page somehow.  The reason why is because after the <Accounts.ui.LoginForm /> component I have placed a regular vanilla <p> tag and it is left alone.

It doesn't seem to me this problem has anything to do with Meteor, or React specifically.
Where I'm looking now to solve this:

something to do with std:accounts-ui
I have no idea where else to look.  Based on the gif demo above you can see that the div with the className accounts-ui is rendering just fine - after clicking in the DraftJS contentEditable area, however, something inside this component breaks.

Issues Tracking this Bug:
I've created three issues to try and track this down:

accounts-material-ui:Issue #26
std:accounts-ui:Issue #96
DraftJS's Github Issue #962
(I've also mentioned this on the Meteor Forums
StackOverflow Question

Once I find a solution I'll update all of the issues, forums and questions I've posted everywhere.

The issue is not with draft JS...
If you remove DraftJS from the equation completely the bug still exists.
import React, { Component } from 'react'
import { Accounts, STATES } from 'meteor/std:accounts-ui';

class MyEditor extends Component {
  constructor(props) {
    super(props)
  }

  render() {
    return (
      <div>
        <button onClick={() => this.forceUpdate()}>Rerender</button>
        <Accounts.ui.LoginForm />
      </div>
    )
  }
}

export default MyEditor;

Closing this issue.  If you have a clue how to solve this bug, please feel free to let me know.
It's not DraftJS though.
Thanks everyone


Project creation support for Slides and other

This can be use as a group of enhancement for basic support for

Create/open project for Asciidoc based Slide supports like dzslides,revealjs etc

We are working on that.

We implemented this feature in v1.3.7-RC!. You can try it  @rajmahendra :)


Allow specifying Semaphore limits with parameters/numbers rather than just config map references

Summary
Allow specifying semaphore value by explicit number/parameter
Use Cases

When using workflow generators, be them obscure scripts or couler, it's easier to encode the semaphore limits directly into the workflow definitions.
The semaphore value for a parallel operation might be decided in a prior step in the workflow, so outputting it as a parameter and using it as the semaphore value would make perfect sense. E.g. I want to perform a map-reduce and the amount of parallel maps should be no more than 10% of number of mappable items.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

This is superseded by #11890 which describes this with a concrete example and also has some commentary from me on it on why this changes the model -- it can cause issues with semaphores shared between Workflows.
For a semaphore that is only used in a single Workflow, you can already achieve this with the parallelism field


Using standard deviation or variance when sampling torch.distributions

Hi,

I noticed in your implementation you are passing gamma * (1 - gamma) to torch.distributions https://github.com/MaximeRobeyns/bayesian_flow_networks/blob/9b20e89b301b55105393a444da658b0fc55c870d/torch_bfn/bfn.py#L126 and similarly in sampling
https://github.com/MaximeRobeyns/bayesian_flow_networks/blob/9b20e89b301b55105393a444da658b0fc55c870d/torch_bfn/bfn.py#L202
But when I looked at the docs for torch.distributions.Normal it seems to be that the standard deviation is the second arg, not the variance.
I feel like this should be (gamma * (1 - gamma)).sqrt(). I've been struggling to get nice results with my continuous implementation, and yours are very good, so I definitely could be missing something
Thanks

Hi,
Thank you for the catch! Indeed, PyTorch expects a standard deviation, and passing this instead of the variance seems to lead to improved samples / training.
As a quick check, I re-ran the two moons example using the same hyperparameters, which yielded the following for the erroneous variance parametrisation:

And the following when using the standard deviation:

I feel like if you're results look substantially less nice then there could be something other than just this variance -> standard deviation difference...
Thanks!

No problem.
Possibly sigma values. I've been working more on the discrete side for an LM task. I was actually very much hoping I was wrong about the sqrt, it would've impacted my discrete loss too.


bugfix: windows cache path not found error

This bugfix is linked with issue #15

Thanks! (I'm not running windows so more windows specific bugs might be present)

Thanks! (I'm not running windows so more windows specific bugs might be present)


Horizontal Rule styling.

My issue is that there is no style. As in the horizontal rule (- - - ) does not display at all.

Perhaps my understanding is incorrect. Do you mean your horizontal rule (- - - ) has disappeared? In my environment, the horizontal rule (- - - ) is normal. I guess it might be conflicting with some plugins. Could you try again in a sandbox environment?

 I changed the color of some element to purely transparent to get rid of the annoying lines in the style settings, unaware that that also hid the horizontal rules. If possible, it might be wise to make those separate options in the style settings.


Fixes the crash in String::percent_decode() for empty strings

The String::percent_decode() and String::http_unescape() functions do not null-terminate the empty character buffers, so avoid the call to String::utf8() if the character buffer is null.
Fixes #22221.

This is actually a broader bug in String (regression from the new cowdata structure) that #21708 aims to address, so we shouldn't patch it in percent_decode and others. Thanks nevertheless for looking into it! A fix should happen before 3.1 beta as it's quite critical.


Refactor Rgb struct

Refactoring of the rmt_neopixel.rs example:

HSV conversion as an associated function, Rgb::from_hsv()
Add Rgb::new()
Implement From<Rgb> for u32
Use cyclic iterator to eliminate counter when looping over hue values
Replace if-else with pattern matching

Thanks for the PR!
Before the workload starts, make sure you run cargo clippy & cargo fmt beforehand, to pass CI.

Thanks for the PR! Before the workload starts, make sure you run cargo clippy & cargo fmt beforehand, to pass CI.

Thanks, fmt has been run and clippy passes with no warnings on my local setup.


Unable to access AKS Clusters

Which project are you reporting a bug for?
kr8s
What happened?
After installing on Python 3.11.5, and attempting to use the example in the documentation:
import kr8s

pods = kr8s.get("pods")

I get the following exception:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/_io.py", line 46, in wrapped
    return portal.call(wrapped)
           ^^^^^^^^^^^^^^^^^^^^
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/anyio/from_thread.py", line 261, in call
    return cast(T_Retval, self.start_task_soon(func, *args).result())
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/cpressland/.pyenv/versions/3.11.5/lib/python3.11/concurrent/futures/_base.py", line 456, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/Users/cpressland/.pyenv/versions/3.11.5/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/anyio/from_thread.py", line 198, in _call_func
    retval = await retval
             ^^^^^^^^^^^^
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/asyncio/_helpers.py", line 22, in get
    api = await _api(_asyncio=_asyncio)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/asyncio/_api.py", line 44, in api
    return await _f(
           ^^^^^^^^^
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/asyncio/_api.py", line 42, in _f
    return await _cls(**kwargs, bypass_factory=True)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/_api.py", line 59, in f
    await self.auth
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/_auth.py", line 48, in f
    await self.reauthenticate()
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/_auth.py", line 59, in reauthenticate
    await self._load_kubeconfig()
  File "/Users/cpressland/dev/starbug/.venv/lib/python3.11/site-packages/kr8s/_auth.py", line 122, in _load_kubeconfig
    **{e["name"]: e["value"] for e in self._user["exec"].get("env", [])}
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: 'NoneType' object is not iterable

The issue might be coming from the fact that kubectl auth is provided by kubelogin.
Anything else?
Using the kubectl proxy solution documented here is a semi functional solution as follow up commands return a number of other exceptions:
>>> import kr8s
>>> client = kr8s.api(url="http://<IP_ADDRESS>:8000")
>>> pods = kr8s.get("pods")
>>> pods[0].get()
TypeError: 'NoneType' object is not iterable
>>> pods[0].get()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1006)

I think this should be fixed by #156

Thanks @jacobtomlinson, tested via $ poetry add git+ssh://git@github.com:kr8s-org/kr8s.git and all working as expected. Looking forward to the next release.


Fixes #3

I didn't know you were using it from the renderer context, hence why you added electron.remote.BrowserWindow.
Thanks for the PR, will merge & publish tomorrow

oh yea.. forgot to mention that :D haha my bad

any update on this? lol just wondering :)

I just needed to test some more uses cases, nearly good to go :)

awesome :) thanks!

published in 0.3.0


The app is only a lib, remove erroneous mod field

Hello Benoit,
Small error in the ssl_verify_hostname app which breaks release creation with static embedded erlang start script containing hackney.
The ssl_verify_hostname_app module does not exist and no supervision tree has to be launched for this app.
Have a good day.
Arnaud

Ooops sorry actually the fix is already on https://github.com/deadtrickster/ssl_verify_hostname.erl/pull/3
so you just need to update this Repo with the master branch of deadtrickster. Is there a specific reason why you use your own fork as hackney dependency ?


Update aws-java-sdk-bundle to 1.12.512

About this PR
📦 Updates com.amazonaws:aws-java-sdk-bundle from 1.12.510 to 1.12.512
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2801.


Migrate BigNumber Types

Intended to close #458, but its going to be a while.
Most of this involves using ethers.BigNumber instead of bignumberjs.BigNumber but the differences are causing difficulties.
Goal is to remove our hand written "toWEI" and "fromWEI" functions.

Closing in favour of #478


ES6 component with propType shows as error for prefer-stateless-function

This will trigger error for prefer-stateless-function. Is there a way to define propTypes in a stateless function or it was a bug?
export default class Foo extends React.Component {
	render() {
		// ...
	}
}

Foo.propTypes = {
	children: React.PropTypes.node
};

Giving a stateless functional component propTypes works just fine.
export default function Foo({ children }) {
  return <div>{children}</div>;
}
Foo.propTypes = {
    children: React.PropTypes.node
};


Problem when testing on Android Device

Everything was working fine for iOS but when switched to android, I got the following error as soon as I call at
BackgroundRemover.removeBackground
The error says:
There was a problem loading the project.
Could not invoke
BackgroundRemover.removeBackground

null

No content provider. http://the_image.url/url.jpg

Would it be possible for you to share the device model and the brand? Also a stack trace would be really useful.
Another question is, is this a build-time or a run-time issue?

It's a Xiaomi MI 8, running MIUI Global 12.0.3 with Android 10 QKQ1.190828.002
The issue is happening in runtime, at the moment I call the function.
I'll try to get the stacktrace as it wasn't appearing to me. I'll try running Android studio directly to check if I can get any info

I see this on the react native console when crashing:
 ERROR  Your app just crashed. See the error below.
java.lang.NullPointerException: java.lang.NullPointerException
  expo.modules.kotlin.jni.JavaCallback.invoke(Native Method)
  expo.modules.kotlin.jni.JavaCallback.invoke(JavaCallback.kt:24)
  expo.modules.kotlin.jni.PromiseImpl.reject(PromiseImpl.kt:87)
  expo.modules.kotlin.Promise$DefaultImpls.reject(Promise.kt:15)
  expo.modules.kotlin.jni.PromiseImpl.reject(PromiseImpl.kt:26)
  expo.modules.kotlin.functions.AsyncFunction$attachToJSObject$2$functionBody$1.invoke(AsyncFunction.kt:74)
  expo.modules.kotlin.functions.AsyncFunction$attachToJSObject$2$functionBody$1.invoke(AsyncFunction.kt:66)
  expo.modules.kotlin.functions.AsyncFunction$attachToJSObject$2$3.invokeSuspend(AsyncFunction.kt:104)
  kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
  kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
  android.os.Handler.handleCallback(Handler.java:883)
  android.os.Handler.dispatchMessage(Handler.java:100)
  android.os.Looper.loop(Looper.java:224)
  android.os.HandlerThread.run(HandlerThread.java:67)

@atlj did the log helped in something?
Do you need more info?

Hey yeah, thanks for the logs and the device info. Sorry I'm running pretty busy for a while noe so couldn't find enough free time to come back to this. Will take another look once I find some time.

Aww seems like class is not prepared to work with remote URIs, only local files, right?

there is an issue in this library when i take picture from camera like putting the signature on paper and take picture to remove background and result of the image is black
before i remove the background

after the result

For me it doesn't work on android. Doesn't work with remote images and for local images the background is turned black too 🥲
I tried to make the fix and I have it working with remote images but still having the black background result

Same here on android emulator api 33, yes it removes the background but not transparent. Its now color black


Can't import stax from jax

Description
I am bit new in jax and I was trying to import the stax library from Jax library. But, it keeps giving me this error:
ImportError: cannot import name 'stax' from 'jax'

Please, help me to solve this error :)
Systems requirements:
Jax latest version
What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?
CPU
Additional system info
Windows
NVIDIA GPU info
No response

That answer my question, thank!


Add missing inherits dependency

inherits is not declared as a dependency.
https://github.com/alexanderGugel/sublevel-pouchdb/blob/master/lib/read-stream.js#L9

Thank you; this was definitely an oversight.


Set date lang on the localized middleware

Fixes https://github.com/cachethq/Cachet/issues/1034
/cc @jbrooksuk

That's way easier than what I had in mind :laughing:

Hmm, this still isn't translating dates?


Consider a way to represent color spaces

We need a way to represent color spaces. It may be desirable to have color spaces defined independantly of color formats, ideally allowing us to have color types defined as a combination of channel type, color format and color space, such as:
// color channels
trait Channel { .. }

// color spaces
enum Linear {}
enum sRGB {}

trait ColorSpace { .. }

impl ColorSpace for Linear { .. }
impl ColorSpace for SRGB { .. }

// color formats
struct RGB<T: Channel, S: ColorSpace> { r: T, g: T, b: T }

// color types
trait Color<T: Channel, S: ColorSpace> { .. }

impl<T: Channel, S: ColorSpace> Color<T, S> for RGB<T, S> { .. }

// possible color types: RGB<u8, sRGB>, RGB<f32, Linear>, etc

However, this may not work well in practice since color spaces are often tied to their own color formats.

there is another color library that has them: retep998/colors-rs
also there’s rudimentary color structs in PistonDevelopers/image
I’d love to see this stuff unified!

good info is to be found here and here

I'll support any initiative to have a standard way of working with colors and colorspaces, as long as it is done well.

do you have any opinions on #24?

Awesome! I am just finishing up a massive cleanup of noise-rs with the help of others, and I think it is well time for color-rs to get some love. I'll see what is in colors-rs and am happy to work with you to improve it.

since this is an info dump as well as an discussion: this project makes advanced use of color models (not in rust).
i think if we could elegantly support that use case, we’re set.


[Bug]  the open telemetry example doesn't export metrics

What are you really trying to do?
I want to export metrics to datadog.
Describe the bug
I don't see any metrics in any terminals.
Minimal Reproduction
follow the steps in the readme https://github.com/temporalio/samples-python/tree/main/open_telemetry
Environment/Versions

OS and processor: M2 Mac
Temporal Version: 1.22.0 and/or SDK version: 1.6.0
Are you using Docker or Kubernetes or building Temporal from source? docker

Additional context

It looks like the Jaeger all-in-one container no longer dumps OTel metrics as it once did. We will find another solution to show the metrics here (probably something like https://opentelemetry.io/docs/demo/docker-deployment/). Thanks for the report!


feat: usable constructor for TaggedTable

Closes #293.
Summary of Changes
Change constructor of TaggedTable to be more usable. The usual way to get a TaggedTable is still to call the tag_column method of Table.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
3
0
0
0.72s

✅ PYTHON
mypy
3

0
1.69s

✅ PYTHON
ruff
3
1
0
0.05s

✅ REPOSITORY
git_diff
yes

no
0.03s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

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

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


Update renovatebot/github-action action to v34.108.5

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
minor
v34.106.0 -> v34.108.5

Release Notes

renovatebot/github-action
v34.108.5
Compare Source
See the the changelog for changes in all releases.
34.108.5 (2023-01-22)
Bug Fixes

deps: update renovate/renovate docker tag to v34.108.5 (08028e4)

v34.108.4
Compare Source
See the the changelog for changes in all releases.
34.108.4 (2023-01-22)
Bug Fixes

deps: update renovate/renovate docker tag to v34.108.4 (7172919)

v34.108.3
Compare Source
See the the changelog for changes in all releases.
34.108.3 (2023-01-21)
Bug Fixes

deps: update renovate/renovate docker tag to v34.108.3 (e7d2b33)

v34.108.2
Compare Source
See the the changelog for changes in all releases.
34.108.2 (2023-01-21)
Bug Fixes

deps: update renovate/renovate docker tag to v34.108.2 (0ae2aa4)

v34.108.1
Compare Source
See the the changelog for changes in all releases.
34.108.1 (2023-01-21)
Bug Fixes

deps: update renovate/renovate docker tag to v34.108.1 (c24a753)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Branch automerge failure
This PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.


support integrated version control in Rasa X local version

Description of Problem:
currently integrated version control is not supported in local version. However, the use of git repositories is essential during testing prior to deployment which is what the local mode is ideally suited for. Also, low volume bots may be set up using rasa x local mode and integrated version control would make maintenance and updating easier.
Overview of the Solution:
Rasa X local would replicate the feature on Rasa X server.
Definition of Done:

hi @randywreed  ! Local mode writes all your project data back to your filesystem, so you can just use git to push and pull changes as you would for any other software project.

@amn41  while this is true, having RasaX integrate changes to the model requires a restart of the rasax local server. This brings the bot down. This is an unnecessary delay that integration into RasaX would prevent.

those are separate things, though.

Integrated version control pushes changes to your trainign data / domain file etc back to your remote git server (e.g. github).
to incorporate those changes into your assistant (locally or elsewhere), you have to train a new model and activate it. In Rasa X local mode, you can hit the train button and then activate your model in the models page, no need to restart.

Please reopen if there are more questions


About accuracy calculation in test function

Thanks for your code sharing, it helps me a lot!
One question about the accuracy calculation :
in your code(main.py, line 295), accu = accu + tp / (tp + fn + fp)
I just wondering why tn is ignored and accu is not: accu = accu + (tp + tn) / (tp + fn + fp + tn)?
It would be appreciated if you could give me some hints.

I used the definition in previous works, e.g. Pedestrian Attribute Recognition: A Survey (Eq. 2).

Thanks for your reply :)


Publish linux/arm-64 images on docker hub

Is your feature request related to a problem? Please describe.
It's annoying that there is no other option but just linux/amd-64 on the docker hub, I think it would be really nice to have a image for linux/arm-64
Describe the solution you'd like
I see that this was already discussed but no one bothered to do it: https://issues.apache.org/jira/browse/DRILL-8260
This is not an optimal process: having tickets opened that no one resolves.

@ionutpopa
Perhaps a little politeness might go a bit further.   Please understand that Drill is developed and maintained by volunteers who are not paid for their time.   If this is really important to you than you could either volunteer to assist, pay someone to do it, or simply ask nicely.

Of course, @ionutpopa is correct. Running a project with all volunteers is not an optimal process in terms of getting everything done. Running it with an infinite budget close source development team would be much more "optimal".
But @cgivre is even more correct. Drill enables freedom. If you want it, feel free to make it so. It may be no more than a one line patch to the CICD scripts.

Ok, I'll give a look on the CI/CD scripts later and what's there but first I'll probably search for a developer notice before doing anything.
Sorry for earlier, I get angry when I see old tickets.
@tdunning you're talking about the workflows folder inside .github, am I right?

@ionutpopa No worries.  We welcome any help you can provide.

@ionutpopa That is what I would have expected, but I don't see docker image packaging there.
There is likely another workflow that is part of cutting a release that generates the docker image. I have no idea where that might be. @cgivre should know more.

@ionutpopa the Docker image builds themselves happen in Docker Hub but we have custom "hooks" in Drill source repo, see the file hooks/build. Let's try introducing something like --platform linux/amd64,linux/arm64.

Hello everyone, I am curious why I am getting an error saying I don't have permissions to push branch to apache/drill. Can someone help with that regard?

Only members of the committers group have that permission. If you create a fork and open a PR to main repo then I can help with branches or tags in the main repo that will trigger image builds on Docker Hub for testing. Once we've got an arm64 image we're happy with we can merge your PR into master. When you open the PR please use the provided template to allocate this work to DRILL-8260.

Tout va bien, opened a PR.


Remove duplication in flexbox.md

First commit removes a lot of duplication by using loops. In fact, I can reduce the duplication even more by using {% cycle %} but then it starts looking like code-golf, so I'm not going to do that.
 A lot of this is pointless because it'll get replaced with ... later.
The third commit merges the 2 functions we had before (remove_holderjs and remove_example_classes) and puts it into a new function called cleanup_highlighted_code. The TODO comment was when I realized that regex does not parse HTML and should be replaced by something better.
The 2 new features that I've also added in this commit are:

The data-bd-hide attribute, which hides the attributes that follow it.

 =>

The data-bd-remove attribute, which replaces the entire element with ...

 => ...

Please note that these are not going to work 100% of the time, because of the way HTML and regex work.
The fourth commit uses the stuff that I added in the previous commit. I plan to do this to every single file, but I first need to fix the TODO.
For people that want to see what this changes, I've made a build of the docs here: https://bootstrap-tests.starsam80.net/more-bd/utilities/flexbox/

Hmm, I'm kinda torn on this. While I understand the looping here to reduce the repeated code bits, I kinda prefer the static examples. It's easier to copy-paste from the source code, ensures readability while viewing files here on GitHub, and is more approachable to beginners.

@mdo I don't personally care about if you want to merge this or not, it's just that when I look at the markup files, I tend to scroll a lot just to get to the bottom of the document and this fixes that. It also makes the markup files more 1:1 with the actual docs that get produced (if you ignore the loops). I noticed files like components/modal.md are kinda split with the actual content that they are showing and the highlighted code. Using the modal as the example, the 2 buttons show up as Close and Save changes in the shown example but Save changes and Close in the highlighted code. After I fix the TODO, we could use a single {% example %} tag to fix these kind of problems.

Going to punt on this. Sorry for keeping it open so long with no movement.

No worries. I totally forgot about this PR myself.


vStatsBot - A monitoring, governance and informative tool for Validators.

Description + Context
vStatsBot’s (Telegram Bot) initial and primary feature is to alert validators within 5 minutes of any missed blocks. This is designed so that the validator can jump onto a problem straight away before it becomes a larger issue that may risk election. The stability of the network is the primary goal here and I truly believe vStatsBot is the first line response.
Other key features of vStatsBot:

Validator Summary - List of stats for the validator such as stake, keys, bidding power, information on the next election (Time left, slot position, number of keys, bidding power and bounds information), votes yet to be actioned.
Governance Participation and Reminders - Shows pending votes (Name, Time left, URL) where the validator is yet to participate. This is shown in both the Validator Summary and Votes command. A targeted notification for those who missed the first two alerts will be added.
Volume Increase Alerts - With the increasing size of db_0 vStatsBot can alert you to the space left at regular intervals and alert you of the automatic size increase. This has helped slow down the number of validators from not singing due to running out of space.
Blockchain Height Sync Alerts - Quite often I am seeing validators falling behind on their node block heights due to under performing servers. This will alert you to when the drop off is occurring so action can be taken before signing would stop.
Wallet Summary ( Delegator based ) - The ability to add wallet addresses for scheduled and on command detailed breakdown of staked, unstaked, unclaimed rewards per address. You can also view your most recent claims for tax purposes.
Election Keys - Shows every possible position on the 900 slots for any number of keys. Especially useful to know where a validator will sit without having to bog the network down in key changes.
Global Alerts - A message sent out to all participants for any critical alerts.

Future plans:

Endpoint for Harmony - This will enable Harmony to send one message and it will notify all validators instantly for node updates, news, critical information etc.
Further expand on the delegators section - Bring informative information to the delegators on their wallets, validator etc.

https://talk.harmony.one/t/vstatsbot-a-monitoring-governance-and-informative-tool-for-validators/8123
Acceptance Criteria

[ ] Uptake by the validator community. Bot is to be used by a large portion of validators to aid in stability of the network and participation.

Reward
$24,000 in ONE

I would like to add this to Harmony's Validator Tool Set. the Application is just about finished. Would you like to see anything before granting?
I see there's over 80 Active Validators utilizing this tool already. What is your thoughts? Can we push it to gitcoin and David can submit ?
@lijiang2087 @sophoah @givp

This has already been approved: https://talk.harmony.one/t/vstatsbot-a-monitoring-governance-and-informative-tool-for-validators/8123/21


Inconsistent behaviour with lang.value:substring when the effective end index is less than the start index

Description:
lang.value:substring creates a substring from the start index (inclusive) to the end index (exclusive).
https://github.com/ballerina-platform/ballerina-lang/blob/master/langlib/lang.string/src/main/ballerina/string.bal#L63
But the behaviour seems to be inconsistent for different scenarios where the effective end index is less than the start index.
When "effective end index - startIndex"

is -1 - returns empty string
is < -1 - panics

Steps to reproduce:
import ballerina/io;

public function main() {
    string str = "ABCDEF";

    io:println(str.substring(1, 3)); // "BC"
    io:println(str.substring(1, 2)); // "B"
    io:println(str.substring(1, 1)); // empty string - start index 1, effective end index - 0
    io:println(str.substring(2, 1)); // panics - "message":"invalid substring range. Length:'6' requested: '2' to '1'"
                                     //            start index 2, effective end index - 0
}

Affected Versions:
slp7

Current behaviour is correct according to this spec issue answer. Hence closing this issue.


Spark 3

With the availability of Spark 3.0, there is a porting effort required to produce a version that runs with Spark 3, e.g. 2.12-3.0.0-5.0.0.
Of note:

it can use Hadoop version 3.2.1
it uses Scala 2.12
there are some changes to the handling of files in Hadoop due to the upgraded Hadoop version

Spark context Web UI available at http://<IP_ADDRESS>:4041
Spark context available as 'sc' (master = local[*], app id = local-1573335188812).
Spark session available as 'spark'.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /___/ .__/\_,_/_/ /_/\_\   version 3.0.0-preview
      /_/

Using Scala version 2.12.10 (OpenJDK 64-Bit Server VM, Java 1.8.0_222)
Type in expressions to have them evaluated.
Type :help for more information.

Opened branch spark-3.0.0.
- CIMReader seems to function just fine
- CIMExport relies on spark-cassandra-connector which has no available version for Scala 2.12 or Spark 3.0.0
Pending inquiry over spark-cassandra-connector support going forward.

The spark-cassandra-connector version 2.4.2 supports Spark 3.0.0.
This seems to work just fine.

With the release of 2.12-3.0.0-5.0.2 support for Spark 3.0.0 is complete.


Contribution to the Chapter "Onboarding of new team members"

Thank you for the recommendations. I have included them in the last commit.

Excellent. Should be ready to merge.


Some D. aurantiacus obs retain RG status even with a CID at section level

EG:
https://www.inaturalist.org/observations/14406011
https://www.inaturalist.org/observations/14111138

I wasn't able to replicate this in tests, so my best guess is that something went haywire when these changes were processing, e.g. a prematurely finished delayed job or something. I went through and reset the community taxon on all obs of Diplacus, which updated 137 observations, and also reset the identification categorization, since a bunch were incorrectly marked as "maverick."


pretrained models

Thank you for this awsome repo!. Are the pre-trained models from the previous repository compatible with the models defined here?

Hi @aswanthkrishna,
glad you like our new code base! Unfortunately, the pre-trained models from the previous repos are not compatible since we had significant code changes.
Which models (i.e. properties for conditioning) are you interested in? I might add pre-trained models at a later point if there is demand for it. I cannot promise that this will happen soon, though, as training the models and testing/validating them + adding some instructions how to load and use them will take some extra time and I'm a little busy at the moment. So if you have the computational resources, training a model on your own would most likely be faster.
Best regards,
Niklas

I will close this for now, feel free to re-open the issue if specific pre-trained models are needed.


{Doc} Add instructions to use Azure CLI in Airgapped Clouds

Description

Add current instructions to use azure cli in airgapped clouds.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Doc

Doc


Where is modelnet40_half1_rm_rotate.txt ?

Hi, currently the files are still missing.
Please upload them.
(Sorry, this is a duplicate of the issue. https://github.com/megvii-research/OMNet/issues/7
I create a new issue because I can not re-open the old one.)

Oops, sorry, I open the issue in the wrong repository.
These two files are missing in the Pytorch version.
Nevertheless, I find it here.


Introduce ExplicitArgumentEnumeration check

Suggested commit message:
Introduce `ExplicitArgumentEnumeration` check (#985)

I found this on an old branch from early last year. Extended it a bit and now putting it up for review. Perhaps we can find a better name. Likely this code should also be tested against the internal code base before landing. But it's certainly ready for review.

Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.13.0:compile (error-prone-compile) on project checkstyle: Fatal error compiling: The default --should-stop=ifError policy (INIT) is not supported by Error Prone, pass --should-stop=ifError=FLOW instead -> [Help 1]

From the IT ☝🏻.
Not checking the error in depth right now, but curious 🤔.

And this....
Caused by: org.eclipse.aether.transfer.ArtifactTransferException: com.github.PicnicSupermarket.error-prone:error_prone_core:jar:v2.36.0-picnic-1 failed to transfer from https://jitpack.io/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of jitpack.io has elapsed or updates are forced. Original error: Could not transfer artifact com.github.PicnicSupermarket.error-prone:error_prone_core:jar:v2.36.0-picnic-1 from/to jitpack.io (https://jitpack.io/): Checksum validation failed, no checksums available

Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.13.0:compile (error-prone-compile) on project checkstyle: Fatal error compiling: The default --should-stop=ifError policy (INIT) is not supported by Error Prone, pass --should-stop=ifError=FLOW instead -> [Help 1]

From the IT ☝🏻.
Not checking the error in depth right now, but curious 🤔.

I filed #1455 for this.

And this....

Caused by: org.eclipse.aether.transfer.ArtifactTransferException: com.github.PicnicSupermarket.error-prone:error_prone_core:jar:v2.36.0-picnic-1 failed to transfer from jitpack.io during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of jitpack.io has elapsed or updates are forced. Original error: Could not transfer artifact com.github.PicnicSupermarket.error-prone:error_prone_core:jar:v2.36.0-picnic-1 from/to jitpack.io (jitpack.io): Checksum validation failed, no checksums available

Same also happened for a master build, but a restart fixed it. The new PR mentioned above also has a passing build.

Oops, sorry, don't know what happened there 😓.

/integration-test

Changed the scope of the new dependency. @rickie: also here, feel free to merge if you agree with the final changes.

/integration-test


Mark large benchmark data files as binary to filter greps

When doing a git grep (of rust-analyzer or of rust-lang/rust with
--recurse-submodules), if the grep happens to match within the large
benchmark data files, the resulting long single lines can cause a text
pager or editor to slow down and distract from more useful matches.
These test data files aren't formatted for human consumption, so mark
them as binary, which causes git grep to instead just state that they
match without printing the matching "line".

Thanks!
bors r+

changelog internal (first contribution) tell Git that the benchmark files are binary


Option to include redirected page in sitemap

All redirected pages don't occur in sitemap since https://github.com/jekyll/jekyll-redirect-from/pull/69 was merged long time ago. Is it possible to make this configurable? I have a special case where I need to include such pages in sitemap, but currently I'm not able to achieve it.

I have a special case where I need to include such pages in sitemap

Can you expand a bit more on what that is?


Group consumer silently fails if message size exceeds max bytes

I came across this issue when initially setting up the group consumer. If the message size is larger than the consumers max bytes then the consumer will silently fail without error and stuck on the same offset.
The kaka-console client throws an error MessageSizeTooLargeException and appears to adjust the offset allowing for the next message to be processed.
While obviously having both the consumer and producer running the same max bytes is the logical and correct way to set up, I believe that it could be beneficial for such errors to be passed on from the client.

Confirmed


Dual wielding thudwoosh sound has too much range

Description

Reproduction

Screenshots

Additional context

Pretty sure I fixed the wielding position


missing neuro_learning NIPS text data

It seems that the file neuro_learning_np832_nq293_d2000.p is missing from the data directory. The other entries present in ex4_text.label2fname are all there.

Thank you. Added.


Disable back navigation on specific Location

When having something like:
    Navigator
    /   |   \
Form Loading Success

How con I avoid back button navigating to LoadingPage again when pressing the Back Button in FinalPage?

On a secondary note, how can I make a Location to not be saved in History

This will soon be solved by https://github.com/slovnicki/beamer/issues/127

I see, any workarounds or estimated time for delivery at this point? This is a dealbreaker for my project, but we can delay navigation implementation a little.

@FeleConvo the PR is already ready, I just need to merge it and publish. Was thinking to release it with nested-routers as a part of v0.10.0 later today. If nested-routers don't go as planned, I will release just that as v0.9.3 later today

After a closer examination, I believe this will not solve your problem. You have Form, Loading and Success pages in one BeamLocation? Are those pages at different URLs or at the same one?

Pages are in different BeamLocations.
I guess the issue in general is that wherever I BeamTo I'm always able to walk back the full history.
For example my application's root is a loading page, so whenever I start my application I'm at ('/') which is a BeamLocation with a single Page with a loading screen, then I BeamTo ('/login') another BeamLocation with single Page and finally after login I BeamTo ('/main') another BeamLocation with single Page.
In every case I can press the back button on my Android device and would back from '/main' to '/login' to '/ '.

@FeleConvo Thank you for this elaborate explanation.
Yes, then this will be solved in v0.9.3 that will get published today (didn't have time yesterday). You will just need to pass replaceCurrent = true when beaming with beamTo or beamToNamed. For example:
context.beamToNamed('/login', replaceCurrent: true)

@slovnicki No problem.
I see in the code of #128 you are internally calling
if ((preferUpdate && location.runtimeType == _currentLocation.runtimeType) || replaceCurrent) {
      _beamHistory.removeLast();
}

Will this handle the case where your initial navigator has more than one Page?
             Navigator           -- BeamTo(Main('/main')) -->             Navigator
            /.       \                                                     /    \
           /          \                                                   /      \
 Login('/login')  SignUp('/login/signup)  <-- BackButton() --    Main('/main') Detail('/main/detail')

@FeleConvo yes, it will.
Items in beamHistory are BeamLocations which contain those stacks of pages that Navigator builds. So replaceCurrent will remove this entire stack [Login,SignUp] that would normally be behind in history.

v0.9.3 has been published with replaceCurrent feature.