Brazil - Text Change

Hi @kazlauskis
Brazil requested a text change to flower descriptions so that it was more appropriate for their users. Initially brazil requested a change to the English as well, but we've pushed back on this so the change is only to the Portuguese (actually I think we discussed this in email)
"spikes" to "long inflorescences" ("inflorescências compridas" in Portuguese)
"flower heads" to "circular inflorescences" ("inflorescências circulares" in Portuguese).
Brazil have changed where they can in Transifex, but searching manually do not think they could find every instance. I have checked and from what I can see spike/spikes and flower head/heads have been updated.
Please can you verify if there is anywhere else these terms need to change, that may not be in Transifex and then we'll want to update the Beta.
(As discussed, I think we can just update the online demo for review to make sure it's all sorted and then move to beta in the next round of updates - which will probably be adding Germany)
Cheers Jim

The app shouldn't have any references to the flower type that aren't in the Transifex.
As for the demo app update. I am not sure what you mean as we have moved past that and there already exists a Beta that people can install to their devices.

Thanks Karolis. Regarding the demo app, I assumed everything was updated in there by default, so that it reflects everything that will go on to either beta or live, or do you have to maintain the demo app seperately?
I just mentioned that so the brazil team could check the terms now displayed as they needed, before the effort of uploading the Beta. In theory, as long as each instance of spikes and flower heads is updated then we're good to go.
I've checked Transifex again and don't see any issue, so this change should be final. I'll check with Brazil and hopefully we can go straight live, rather than via another beta.

No problem. Transifex changes don't automatically come to the demo app as we have to manually pull these in. I think, the flower type translations were done after we have pushed the app for Beta testing so it didn't make it to the public yet. We could update the translations and push these to the demo if you'd like? Then if these are OK we could release a new app version with the translations change alone to the public. Since there aren't any other features waiting in the pipeline for Beta testing this translations update would be straightforward.

If we could update translations and push to demo that would be useful thanks.
I didn't realise that Brazil was actually live, I thought it was only Beta? Out of interest, do you know if it's possible to have both the live and beta installed on the same device? Doesn't seem possible to download the public app from google play, it defaults to the beta.

Out of interest, do you know if it's possible to have both the live and beta installed on the same device? Doesn't seem possible to download the public app from google play, it defaults to the beta.

In general, no, you can't install different versions of the same app. Users can leave the Beta testing program to install live versions but by default, Beta users should get whatever is the latest version.

I didn't realise that Brazil was actually live, I thought it was only Beta?

Sorry for the confusion. Brazil was in Beta testing for a long time now and after we've added the Ireland to Beta and released it to live both countries went to the live app stores. In case we had multiple countries in Beta but you wanted to release only certain countries we could in theory push a special update just with those countries. Having said this, dealing with multiple pipelines could get rather messy. It is the best to keep the Beta testing of new features short-lived and frozen for adding new features until all the bugs are found and the Beta is released to the public.

Got it! Makes perfect sense, we'll keep everythink on the demo online app, until country is completely satisfied with it and push to beta at an agreed time and for an agreed period, just long enough to test and have just one country in beta at a time. The S.American countries are a bit more of a challenge as they don't have an existing paper based FITCount already, so proving a bit harder for them to work out what they need. Thanks


ps_emailalerts -> new order mails not being send, since last mollie update

Expected behavior
When a new order is placed, a notification email is being send to the merchant with the official Mail Alerts module by Prestashop.
Actual behavior
Since the last Mollie update this email is not being send when the order is paid with Mollie. I gues this has to do with the new change of only creating a new order after payment is success and not triggering the right script?
Steps to reproduce the behavior
On prestashop 1.7.7 of 1.7.8 with the official Mail Alerts module installed and the latest mollie 4.4.2.
Mail alerts module can be installed in the module catalog in BO. Or downloaded on github: https://github.com/PrestaShop/ps_emailalerts/
Place a new order and no "new order mail" is being send.
Spend a lot of time figuring out it was due to the mollie module as this is the only payment method we used. But when testing with a different payment method everything works.
Hope you guys have a quick fix for this as this is
Environment

PrestaShop version: 1.7.8
Mollie module version: 4.4.3
Mail Alerts module version 2.3.0 https://github.com/PrestaShop/ps_emailalerts/
PHP version: 7.1.26

UPDATE: I downgraded mollie
4.4.0 -> email works!
4.4.1 -> email works!
4.4.2 -> email NOT being send

Hi @wzzly .
Thanks for the ticket!
This will be fixed next release but for faster fix you do this:
remove Mollie – Effortless payments : +-810line
remove or comment:
if ('new_order' === $template) { return $newOrderMailValidator->validate((int) $order->current_state); }
That should fix the issue for now, until the next release.
--
Best regards,
Invertus Support team

Thanks for all the efforts @Saimis777 :)

Hello,
As there are duplicates of this ticket, we will keep the information regarding email case in the ticket - #444
Closing this one.
--
Best regards,
Invertus Support team

@Saimis777 thanks the temporary fix works!

Same error here - We'll try the temporary fix and inform you if it works within a day.

Thanks @wzzly , @waschier-design ,
If you'll have any other questions, let me know!

Hi @wzzly @waschier-design!
The issue has been resolved in the new release, which is out now, and can be downloaded here - https://github.com/mollie/PrestaShop/releases/tag/4.4.3
Let us know if you have any other questions, and any feedback regarding the release!
--
Best regards,
Invertus Support team


added author and comment in notification

Fix #14

Hey,
Sorry I had a lot of work to do and just forgot to review my PR :(
But thanks for your commit :)
Cheers,
Marcus


Ensure html_for_tweet() works if symbols is missing

Some Tweets, such as those in a downloaded archive, don't have a symbols element in their entities dict.
This fix ensures html_for_tweet() still works if that is missing.

Coverage increased (+0.06%) to 55.43% when pulling 4a99ec0157e142f73afcac8690a8bc1f3b83b8c9 on philgyford:fix-no-symbols into 5a87fc7d842994c48d32da10dc817fb1a9af16d7 on ryanmcgrath:master.

I'm closing this as I have a conflicting, but better, PR on the way.


bugfix: checking of input types in plotting

This fixes an error that made its way onto main, via change of input checks:

plot_series and plot_correlations now allow arbitrary input series via conversion
the second invalid input example in test_plot_series_with_unequal_index_type_raises_error has been changed from one-column pd.DataFrame to a two-colum pd.DataFrame

@mloning ?


Egg is not displaying

When i clicked on egg, there is no pop up. Using 1.0.5

Which bot are you using ? necro or mobbot ?
Which version ? Compiled or from the source ?

Using necro latest and from source.
On Aug 1, 2016 8:36 PM, "Nicolas Schmitt"<EMAIL_ADDRESS>wrote:

Which bot are you using ? necro or mobbot ?
Which version ? Compiled or from the source ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nicoschmitt/necrobotvisualizer/issues/36#issuecomment-236546812,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACV2QIkpJF23hJ_6tYsk0WmaYTMfamvnks5qbcwqgaJpZM4JZbMc
.

Ops I mean it's necro latest and is already compiled.

More information: using latest necrobot from their released, which is already compiled v1.0.1 and using the visualizer from the latest released that is already compiled in release.

1.0.1 so you mean PokeMobBot and not NecroBot right?
NecroBot latest source works so you may have to wait a bit.
For MobBot, I'll have to check.

My bad it's NecroBot v0.5.0. Was looking at the wrong git. Yeah saw v0.5.0 on my log.

There is no notification and the map does not follow players, even thou i saved the setting in settings. When i open settings again, it became untick.

indeed, there seem to be an issue with the option

I am using the same stuff, but the eggs are working fine.

I'm on window 10. If that's help for more information.
On Aug 1, 2016 11:04 PM, "li81"<EMAIL_ADDRESS>wrote:

I am using the same stuff, but the eggs are working fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nicoschmitt/necrobotvisualizer/issues/36#issuecomment-236574645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACV2QMObxcKMlS00mIR1cPVvXU6o8lQxks5qbe75gaJpZM4JZbMc
.

Same, windows 10, 64bit.

Weird, have no idea what's wrong with it then.
On Aug 1, 2016 11:14 PM, "Cr@zymdfkr" <EMAIL_ADDRESS>wrote:

Same, windows 10, 64bit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nicoschmitt/necrobotvisualizer/issues/36#issuecomment-236576816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACV2QFs1LwghfoX1QR6K7d3Cy3IhtiR9ks5qbfEtgaJpZM4JZbMc
.

It may be a version issue on the bot side. If you compile the bot yourself it will probably work.

Have the same from time to time, seems to be an error on necrobot side while its in an loop stage or something...

Yes when the bot is starting for some time communication is buggy. Once launched it should be ok.


Inconsistency in range checks

Background
The circuit checks for all levels in the tree if the sibling node's balance (and two leaf balances) is less than m = 2 ** (NBYTES * 8). In the first look, this suggests that the max balance for the immediate parent of the leaf nodes at level 1 would be 2 * m and the parent at level 2 would be 3 * m, ..., and the max balance at the root will be (NLEVEL - 1) * m.
The inconsistency
Let's consider two scenarios for a tree with just four leaves. Scenario 1: The user corresponding to Leaf 1 generates a proof. Scenario 2: The user corresponding to Leaf 3 generates a proof. The following diagram represents node balance range checks. A green node represents a direct check in the circuit, and a red node represents forced ranges.

Notice that depending on which leaf is generating the proof, the inner nodes have different ranges of values allowed. For all the cases to be consistent together, the root node can't be more than 2 * m (because you can find cases in which either of the root's children are checked to have max value m).
Expected behavior
Root's max balance is (NLEVEL - 1) * m as can be inferred from circuits/contracts.

But it only checks for overflow relative to the prime field.

I believe so

The focus of the inconsistency is on the fact that calculate_max_root_balance assumes the maximum value that root can take is m * (NLEVEL - 1) whereas the subtle constraints enforce that if root is more than m * 2, then someone's proof is bound to fail.
Steps involved when we are setting the parameters:

The maximum value that the root can have is determined by the modulus. (As max_root_balance < modulus)
The maximum value that a leaf can take is calculated as max_root_balance / (NLEVEL - 1). (As max_root_balance = max_leaf_balance * (NLEVEL - 1))
Using the max_leaf_value, NBYTES is calculated. (As max_leaf_value = 2 ** (NBYTES * 8))

Given the inconsistency between the intention while setting the parameters (NBYTES , max_leaf_value , etc.) and the actual constraints the circuit is enforcing, we can run into issues demonstrated by @teddav above.

But it only checks for overflow relative to the prime field.

Overflow is not an issue here, unexpected constraints leading to unexpected failure of proofs is a possibility.

I see your point.

A green node represents a direct check in the circuit, and a red node represents forced ranges.

Do you think that modifying the direct check inside the circuit such that the range is different from any level would fix this issue? For example in your first illustration, the Node 2 would be constrained in the 2*m range.

then someone's proof is bound to fail.

Overflow is not an issue here, unexpected constraints leading to unexpected failure of proofs is a possibility

Also, can you please better define what is the unintended consequence of this issue? I think that is not a soundness bug, namely a prover can generate a valid proof of existence of a leaf that is not in the tree.

Also, can you please better define what is the unintended consequence of this issue?

I'll give an example. Let's say that the number of levels (NLEVEL) is 5. Let modulus = 2^10 + d, where d is a small number. With these initial values and the specifications of the protocol (as mentioned in the bullet points in my previous comments):

We set max_root_balance = 2^10.
We set max_sibling_balance and max_leaf_value = max_root_balance / (NLEVEL - 1) = 2^10 / 4 = 2^8
We set NBYTES = log(max_leaf_value)/8 = 1

Thus, a leaf can have values upto 2^8 = 256. Let's construct a tree in which all the leafs (i.e. users) have balances equal to this upper limit.
L5:                                                 2^12
L4:                     2^11                                                     2^11
L3:        2^10                        2^10                       2^10                       2^10
L2:   2^9          2^9            2^9          2^9        .......
L1: 2^8  2^8    2^8  2^8    ........

(Notice how the root balance exceeds max_root_balance)
Now, for the first user (L1[1]), proof generation/verification happens as follows (✅=pass, ❌=fail):

L1[1] <= 2^8 and L1[2] <= 2^8 (leaf node and sibling are checked) ✅
L2[2] <= 2^8 (as mentioned here) ❌
L3[2] <= 2^8 ❌
L4[2] <= 2^8 ❌
L5[1] <= 2^10 ❌

The reason for these failures is that the constraint max_sibling_balance = max_leaf_value = max_root_balance / (NLEVEL - 1) is incorrect.

Do you think that modifying the direct check inside the circuit such that the range is different from any level would fix this issue? For example in your first illustration, the Node 2 would be constrained in the 2*m range.

I believe (under the assumptions I mention below) the correct constraints should be:

max_leaf_balance = max_root_balance/(2^(NLEVEL - 1))
max_current_level_sibling_balance = 2^(current_level-1) * max_leaf_balance (this introduces a dependency on current level while checking sibling's range)

Under these rules, our parameters in the above example would be:

max_root_balance = L5_max = 2^10
max_leaf_balance = L1_max = 2^6
L2_max = 2^7
L3_max = 2^8
L4_max = 2^9

It can be seen that with these parameters, a valid tree will be generated.

I assume that we want to constrain each user to an upper limit such that each user can independently reach that upper limit simultaneously while the root doesn't exceed the modulus. An alternative to this would be not limit any user but just ensure that the root (i.e. the sum total of all users) doesn't exceed the modulus. Even if that's the case, there remain inconsistencies that I can expand upon (with an example similar to the first comment in the thread and new rules).

Expanding upon the alternate assumption where only root is to be limited:
Let max_root_balance = 2^10 and NLEVEL=5. As calculated above, max_leaf_balance = max_sibling_balance = 2^8 Let the first user have the maximum balance allowed, second user have balance = 1, and the rest of the users have 0 balance. Then the tree would look like:
L5:                              2^8+1
L4:               2^8+1                             0
L3:      2^8+1             0               0               0
L2:  2^8+1      0       0       0       0       0       0       0
L1: 2^8   1    0  0    0  0    0  0    0  0    0  0    0  0    0  0

For the first user (L1[1]), the checks (as per the current protocol) will be as follows:

L1[1] <= 2^8 and L1[2] <= 2^8 (leaf node and sibling are checked) ✅
L2[2] <= 2^8 ✅
L3[2] <= 2^8 ✅
L4[2] <= 2^8 ✅
L5[1] <= 2^10 ✅

For the third user (L1[3]), the checks will be:

L1[3] <= 2^8 and L1[4] <= 2^8 (leaf node and sibling are checked) ✅
L2[1] <= 2^8 ❌
L3[2] <= 2^8 ✅
L4[2] <= 2^8 ✅
L5[1] <= 2^10 ✅

The proof will fail for all users >= 3. It can be shown that if second user with balance =1 was permuted to become the last user, all checks would have passed. Despite the sum total of all users not exceeding the max_root_balance, depending on the permutation of user balances, the proof fails.
Other cases which fail can be similarly constructed.
Under these assumption, our rules should be updated to check at each level if the sum of balances of the current and sibling node exceeds max_root_balance rather than the present heuristic of just checking the sibling node.

I follow your point of view, but let me challenge the assumption that underlies the issue that you pointed out. You interpret N_BYTES as the max value that a balance of the user can be.
Instead, when we set N_BYTES to 8 (https://github.com/summa-dev/summa-solvency/blob/master/zk_prover/src/circuits/tests.rs#L22), we are setting it to avoid any overflow in the summation of the balances. What we are implicitly saying is that the order of magnitude of the summation (=root balance) should be "around" 8 bytes.
In particular, this is how you can calculate the maximum value that the root balance should be equal to. And that script is useful to check whether that value is less than the modulus of the circuit and therefore there's no risk of overflow.
L5:                                                 2^12
L4:                     2^11                                                     2^11
L3:        2^10                        2^10                       2^10                       2^10
L2:   2^9          2^9            2^9          2^9        .......
L1: 2^8  2^8    2^8  2^8    ........

In the example you provided here, the proof generation for the first user will fail and this is the expected behaviour, because the root balance will be greater than the limit we expect (see the script).
Your suggestion is valid and it is probably a more logic way of handling the range checks, but it's an alternative approach. I don't see any issue with the current implementation.

Can you also critique my comments on the alternate assumption? If you feel that's not a problem either, we should be good to close this issue.

I don't follow your example,
L1[2] <= 2^8 isn't it 1 in the representation?
L1[3] <= 2^8 isn't it 0 in the representation?

Your suggestion is valid and it probably offers a more logical way to handle the range checks, but it's an alternative approach. I don't see any issue with the current implementation.
I agree with this. However, I would like to add more.

In the early stages, we assumed that the custodian, who generates the commitment proof, would also generate inclusion proofs for all users. However, we've realized that this requires substantial resources, which means that the prover will likely start generating inclusion proofs only upon users' requests.
Considering this, I think this issue is a valid concern in the current implementation. During the commitment phase, the prover completes the root hash without necessarily detecting potential failures that could occur in the subsequent phase of generating inclusion proofs.
This potential oversight can become apparent if the total balance exceeds 2^{N_BYTES * 8}, the range check limit, which is precisely the scenario Y5Yash described (i.e., 2^8 + 1). Y5Yash and Teddav have raised a good point here.
Thus, we can conclude that Summa relies on the provers' diligence when generating the commitment, which is the root hash of the MST, to avoid these failures if not generating all inclusion proofs.

@Y5Yash I understand your concern and I think that both this issue and this other issue denote that our implementation of the range checks creates some confusion. I propose here a way to fix both issues trying to limit the sacrifice in terms of performance of the circuit.
The simple rule is to let the exchange set N_BYTES as the range in which the sum of the balances should exist. The range checks will be performed as it is performed right now. In addition, also the root is checked to be inside N_BYTES.
The rule must be clear to the prover, which, when generating the commitment, should be warned (as suggested by @sifnoc) that if the sum of the balances is greater than N_BYTES, the generation of any proof of inclusion will fail.
Furthermore, similar to what happens right now, the user should still perform some checks on the verifier logic as there might be combinations of N_BYTES and LEVELS that make the root balance (or, potentially, even intermediary sums) overflow.
One example of that is the following one, by considering a prime p=13 and enforcing a range check of 3 bits on the balance of the nodes in the merkle proof and the root.

I have a similar suggestion with a well defined rule:
"No subtree should exceed max_root_balance/2"
How to implement:

At each level check if the sibling node's balance is <= max_root_balance/2
Also check if the current node's balance is <= max_root_balance/2

This is similar to the current constraint but let's the nodes gain a value higher than max_root_balance/(NLEVEL - 1). It also addresses the issue you mentioned (I understand that not checking sum overflow is deliberate) and gives a well-defined rule with less potential for confusion. The downside, however, is that at each level you need to do two range checks (with a bigger number). More importantly, the permutation concern (as demonstrated in the 2^8+1 example) is not resolved either which might make me uncomfortable (it's worth investigating if the invalidity of certain permutations gives away information about users) but can still be reasoned by asking the custodian to be careful.
For a quick demonstration:
m = max_root_balance
NLEVEL = 3

A valid permutation:
L3:         m
L2:   m/2       m/2
L1: m/2   0    0   m/2

An invalid permutation:
L3:           m
L2:     m          0
L1: m/2   m/2    0   0

I believe both implementations can be benchmarked and if the time differences aren't much, this can be considered.

@alxkzmn @sifnoc what's your take on that?


#194 Fix LinkedIn parsing id.

@eg04lt3r Thanks, I will find someone to review this PR soon

@pinaf please review,thanks

@davvd @yegor256 is this valid? I reviewed #123 for this same issue, which is still open. Now issue was reassigned and there's another PR + review going on.. @yegor256 please advise.

@pinaf we should review each pull request individually. as a reviewer you shouldn't really care what issue this pull request is solving. everything should be clear from the branch. look at the branch and make a decision - does it improve the code base? does it follow the design idea of the product? If yes, approve it.

@eg04lt3r I have a feeling some commits don't belong here. this code looks like it is mixed up. maybe all you wanted to push was the linkedin patch?

@eg04lt3r 3 comments above. something is wrong with this PR.

@pinaf Sorry, i deleted wrong code from this pull request. Please, review last commit.

@eg04lt3r no problem

@rultor merge

@rultor merge

@pinaf Thanks for your request. @yegor256 Please confirm this.

@eg04lt3r how about a test first?

@yegor256 Review last commit.  Having this unit test in place you basically prohibit us to change the class in the future. If we want to refactor the class and start using Apache Http Client instead of jcabi-http - we won't be able to do this. We'll have to throw away the unit test first.

@yegor256  according to puzlle of @dmzaytsev I implemented unit test for PsLinkedIn. It's good solution for similar tests in the future. And I do not see any difficulties with using it. Also, if PsLinkedin will use Apache instead of Jcabi no reason to rewrite test.

@eg04lt3r ah, I got it, sorry about the last comment, I was wrong. But I think it would be better to use jcabi-http mocking, since we're using the library already, see: http://www.yegor256.com/2014/04/18/jcabi-http-server-mocking.html

@yegor256 Ok. No problem. Does it track additional time of this implementation?

@eg04lt3r see http://www.teamed.io/policy.html#5 and http://www.teamed.io/policy.html#6

@eg04lt3r are you still working on this one? If not, please close the PR.

@pinaf I'll commit today.

@yegor256  After investigation of jcabi-http, I should say that more flexible way to test similar cases is to use WireMock. If use jcabi I should refactor PsLinkedin constructor to use JdkRequest, what obliges   us to rework class if we decide use Apache Http Client.

@eg04lt3r you can use Takes itself, like it's done in https://github.com/yegor256/takes/pull/228/files

@yegor256 I reworked it, and remove WireMock dependency. Please, review.

@rultor try to merge

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor try to merge

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

@elenavolokhova review this ticket please, for compliance with our quality rules

@rultor deploy now pls

@rultor deploy now pls

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy now pls

@davvd Done! FYI, the full log is here (took me 11min)

@davvd Quality is good here.

@davvd Quality is good here.

@elenavolokhova thank you

@pinaf I added 10 mins to @elenavolokhova (for QA review) in transaction 57125656... done, I added 21 mins in payment AP-34575035VX696643H, 383 hours and 47 mins spent to complete this... review comments (c=6) added as a bonus... +21 added to your rating, at the moment it is: +7809


Depth stream start failure

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
(a)Win10Enterprise b17763 / (b)Ubuntu 18.04.04LTS

Kernel Version (Linux Only)
(b)4.9.140

Platform
(a)PC / (b)NvidiaJetsonNano

SDK Version
(a)2.38.1 / (b)2.35.2

Language
N/A

Segment
N/A

Issue Description
Hello,
I've recently bought a Realsense D435 camera for integration on a robot in scope of obstacle avoidance, based on a Jetson Nano board.
The first step I took was to test it using the Realsense Viewer from the latest SDK release for Windows.
Once the firmware was updated to the latest production version (<IP_ADDRESS>), I activated the two streams, stereo and RGB.
The RGB stream works flawlessly even at 1080p/30fps, but I don't receive any stereo frame.
The errors messages I get are three, continuously every few seconds:
types.h:1099 - USB SCP overflow
types.h:1099 - Depth stream start failure
types.h:1099 - REC error
The camera SN is 022422070xxx, ID 022523020xxx. The same HW/SW setup (USBcable/port, PC/devboard, SDK, configuration) works perfectly with another D435 camera with SN 014122072xxx, ID 015223022xxx.
Only one camera is connected at any moment to the PC/devboard.
I've attempted the following changes, none of which yielded any success:
-hardware: USB cable, USB type-c plug flip, USB port, USB2.0 port, PC to JetsonNano, type-C socket cleanup, temperature check (ASIC-32dgrC, emitter-30dgrC),
-software: gold factory reset, update to latest development firmware (<IP_ADDRESS>) and previous production (<IP_ADDRESS>), reflash and also recovery flash to latest production firmware (<IP_ADDRESS>), reduce/disable laser emitter, enable only stereo stream, reduce stereo stream features (resolution, processing, etc.).
I've also tested the camera by simply using Windows's built-in Camera UWP app - the older D435 is recognized and the stream is displayed without any issues.
The non-working D435 cannot be accessed as a webcam until I've uninstalled the "Intel(R) RealSense(TM) Depth Camera 435 with RGB Module Depth" device, even though "Intel(R) RealSense(TM) Depth Camera 435 with RGB Module RGB" is also recognized.
Please advise on how to fix this issue. Thank you!

Hi @pcmihnea   Because you mentioned UWP, as a first step in the investigation, can you confirm please whether or not you have installed the Universal Windows Platform (UWP) driver?

If you have installed this driver, could you uninstall it please and let us know if it solves your problem.  This driver is not required to use RealSense with Windows, as it is intended to allow a RealSense camera to interface with products that support UWP, such as the Microsoft Hololens headset.  Most RealSense users therefore do not require it.
There have also been issues recently regarding having the latest UWP driver version <IP_ADDRESS> installed that are resolved if that driver is uninstalled.

Hi @MartyG-RealSense, thanks for the quick response,
The UWP issue I've observed was just a sidenote, as using the D435 camera just as a webcam is not the targeted application.
As I've mentioned in the description, another D435 camera works perfectly on the same hardware/software setup - I've even done a clean reinstall today on the Jetson Nano to rule out any OS-related issues.
If I simply replace the non-working D435 with the older one (even using the same cable), I can receive both streams - this leads me to believe the issue is with the camera.
The difference between the two camera seems to boil down to SN and FW ID's, as both were bought brand-new from a reputable supplier, with a few months between them.

Thanks for the further details.
If you uninstall the Depth driver from the Device Manager then the RGB driver should also be uninstalled.   If you then unplug the camera, wait a couple of seconds and re-insert the camera, Windows should reinstall the two drivers automatically over an internet connection.  That is the appropriate procedure for an uninstall and clean reinstall of the RGB and Depth drivers as though it is a first-time setup of the camera after purchase.
My experience is that if RealSense functions such as depth are not working but the RGB is working then the computer is treating the camera like a basic RGB webcam instead of a RealSense camera.  You should definitely not have to uninstall the RealSense RGB driver in order for RGB to function.
For the next step to test, could you unplug the camera please and then check the Camera section of the Device Manager to confirm that there are no hidden RealSense drivers present.  When there is no camera attached the two drivers (RGB and Depth) should not be listed in the Device Manager.  There was a case though with a RealSense user who had eight drivers in the Device Manager instead of two though because of hidden driver entries that should not have been there.
https://github.com/IntelRealSense/librealsense/issues/7185#issuecomment-688837669

I've done as you've instructed, including a SDK and driver reinstall, but the end result is the same.
To eliminate any OS-related problems, I've installed the SDK for the first time on another PC with Windows 10 b19041, and still get the same log. So far two Windows PCs and one Ubuntu board all return the same errors.
After a quick overview on the cod , "Depth stream start failure" may be a notification from the camera, and not from librealsense, so it's signalling a hardware issue?
Are there any operations to be made on the d435 besides reflash, fw recovery, and gold reset, all of which I've tried multiple times?

Thanks very much for your patience.  I analysed all of the information that you provided and came to the same conclusion that it is likely to be an issue related to the hardware of that specific camera.
I gather from the supplied information that you detached the camera with the problem from the USB cable and attached the other camera to the same cable and that camera worked fine, which would seem to eliminate the USB cable as a cause of the problem.
'
I wonder if the 50-pin 'interposer' cable that joins together the Vision Processor D4 board and the D430 Depth Module board inside the camera has become partially unseated somehow.   There would be no way to check this though without opening the camera up to check it.  This would invalidate the sales warranty though.  So if it is less than 30 days since you bought the camera with the problem, it may be worth considering returning it to the place of purchase for a replacement, since you have clearly tried virtually everything possible to rectify the problem.

If there is no other non-hardware-alterating solution, then I will submit a RMA to the supplier.
Thank you for your help!


Password prompt is output to --log-file

Hi there,
this has been discussed in forum.
When the config password is enabled, the log file displays the password prompt line, and It should not be there.
I use rclone 1.40 on a linux i386 architecture.
Thanks,
Marco

Currently when --log-file is engaged it redirects stderr to the log file.
However we write the password prompts to stderr so these end up in the log file too.
What we should probably do is not do the redirection until we've asked for the password, or somehow keep the old stderr handle and get the password prompt routines to use that.

Just to remember that when using environment variables (which is very comfortable), the log only shows:
2018/04/04 21:14:30 DEBUG: Using RCLONE_CONFIG_PASS password.

This is now fixed in the latest beta which will be released in 1.42.


Resolved Issue #1

Updated the README.md with the repo link instead of the gist.

Resolved Issue #1

Thanks! @mrkhdly


Allow $ to be escaped, thanks to @sga001, fixes #1197

This includes the previous pull request, having a hard time separating the two, sorry...

you should mention, that you are the syntax file maintainer, otherwise we might tell you to discuss this with the syntax file maintainer :) BTW: you can always mail comlete file to bram for inclusion with Vim. As far as I know, that is easier for Bram.

Codecov Report

Merging #2169 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2169      +/-   ##
==========================================
- Coverage   73.97%   73.94%   -0.03%
==========================================
  Files          90       90
  Lines      131932   131933       +1
  Branches    28977    28977
==========================================
- Hits        97592    97556      -36
- Misses      34317    34358      +41
+ Partials       23       19       -4

Impacted Files
Coverage Δ

src/gui_beval.c
39.87% <0%> (-4.61%)
:arrow_down:

src/gui_gtk_x11.c
47.26% <0%> (-0.61%)
:arrow_down:

src/window.c
81.12% <0%> (-0.47%)
:arrow_down:

src/channel.c
82.54% <0%> (-0.13%)
:arrow_down:

src/libvterm/src/screen.c
71.35% <0%> (-0.03%)
:arrow_down:

src/terminal.c
60.69% <0%> (+0.06%)
:arrow_up:

src/ex_cmds.c
77.8% <0%> (+0.08%)
:arrow_up:

src/if_py_both.h
76.23% <0%> (+0.14%)
:arrow_up:

Continue to review full report at Codecov.

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

I'll include it, thanks.


DTO naming convention

Dear Edreyer,
looking at your swagger URL:
http://localhost:8080/webjars/swagger-ui/index.html
I could see that your DTOs have different naming conventions. Sometimes you use Input, Output, other moments you use In, Out. And in some occasions, you just don't even use anything to indicate if the DTO is for in or out purpose.
I would suggest, that you take to one convention and stick to that. I understand that sometimes a DTO can be used for in and out purposes but it might be interesting to have separate classes just for the sake of clarity.
This is just my 2p.
Thanks once again for your repostory.

Nice catch!
This project was a fun experiment with Software Architecture. I was really thinking hard about the structure and how it all fit together. It seems I missed cleaning up some of the finer details like this :)


Fixes Issue 9 and 10

Fixes Issue #9 by checking if the name is present, otherwise username from params is loaded.
Fixes Issue #10 by checking if the location is present, otherwise, the child is not rendered.

Manually merged to dev branch after checking it on the local repository.

Added a little optimization in #9


Adding GitHub Action to produce Python Wheels for Linux and Mac

This short PR adds a GitHub action that will automatically trigger the build of python wheels for all versions of python above 3.6 for both Mac (intel) and linux architectures. It uses the excellent https://cibuildwheel.readthedocs.io/ tool that makes all of this very simple.
For maintainers, here are the steps to follow upon a new code release:

Use the github release tool to tag a new version
Go get a coffee for like 30 mins
Go the Actions tab and find the latest Build wheels run, download the attached artifact
Unzip the artifact
Upload to pypi with: twine upload  *.whl

This is in addition to uploading the normal source distribution. Eventually the entire process can be automatized if you create a pypi key so that the action can directly push to pypi.
After that, a user can trivially install halotools by:
$ pip install halotools

and they wont have to recompile anything, this will automatically grab the appropriate python wheel.
If you want to test it out before merging, I've run the action on my fork:

The output of the github action is here (notice the attached artifact): https://github.com/EiffL/halotools/actions/runs/1094382382
I pushed to test.pypi.org for testing: https://test.pypi.org/project/halotools/
To try installing halotools with binary wheel from the test pypi, do the following:

$ pip install -i https://test.pypi.org/simple/ halotools

Annnnd to demonstrate how cool this is: https://colab.research.google.com/drive/1_KjhW8zV7EGDyIzeIHC9tZiZpffs69G7?usp=sharing

Really nice work getting this going @EiffL! I can confirm that running tests locally on my mac works as expected.


Canjs compatibility update

As it sits right now, when using the latest CanJS (2.1.4), findAll and findOne return plain js objects.  This PR fixes that issue by changing findAll to makeFindAll.
findOne still has an issue.  I'm not quite sure how to fix it.  It wasn't as simple as changing findOne to makeFindOne.
I've also updated the tests and included instructions in a new readme.md file on how to run them.  If you run the tests, you'll get the following result:

The error is occurring when trying to run .attr() on the returned findOne data.  @daffl hopefully you will have some insight on what needs to be done.  It seems like you've been busy, but if you can give me an idea of what might be happening, I'll see what I can do.

Ok.  I was mistaken.  It's not fixing the findAll problem.  It's falling back to an Ajax request.

Tests are passing, now, but it's using Model.models and Model.model manually on the data.  I know those are deprecated, but I can't wrap my head around how to use makeFindAll and "pipe" into parseModels the way it says here: http://canjs.com/docs/can.Model.parseModels.html
So this is a fix, but it's a temporary one.

As a result of this current "fix" is that you can no longer use this syntax:
model.findAll({}, function(dbs){ // Just note this line.
	self.attr('databases').replace(dbs);
	dbDef.resolve(dbs);
});

But instead have to use then():
model.findAll({}).then(function(dbs){  // Just note this line.
	self.attr('databases').replace(dbs);
	dbDef.resolve(dbs);
});

I've updated this to use makeFindAll() and makeFindOne().  All tests are passing.  It's still using model() and models() manually.

Justin let me know in the CanJS Gitter chat that makeFindAll() is supposed to still use models() manually.  Same for makeFindOne and model(), so this is done.

Would this still work with older versions? Oh, I'd also like to use https://github.com/feathersjs/feathers-websocket-client here for the next version to consolidate things a little bit.

I tested with CanJS 2.1.4 and 2.0.  All tests pass.
I tested with CanJS 1.1.7.  All the CRUD tests pass, but the tests for live binding choke because can.view.mustache() doesn't exist.

Can you publish feathers-websocket-client to Bower?

Yep, will do. You can merge your PRs already though and tag a new version. We'll migrate it after that (and make a new release).

Cool.  I did a little prep work for publishing feathers-websocket-client to bower here: https://github.com/feathersjs/feathers-websocket-client/pull/2


Allow implicit float8 and int4 promotion under numpy_dtype_promotion='full'

This adds a new type promotion mode, jax.numpy_dtype_promotion('full') which enables implicit type promotion of float8 values. For example:
In [1]: import jax

In [2]: import jax.numpy as jnp

In [3]: x = jnp.array(1, dtype='float8_e5m2')

In [4]: y = jnp.array(1, dtype='float32')

In [5]: with jax.numpy_dtype_promotion('full'):
   ...:     print(x + y)
   ...:
2.0

I'm still unsure of whether this should be included in the default standard promotion table for the reasons discussed in
#16705, but at least this gives users a way to opt-in for the time being.

The last push updates the float8 and int4 implicit promotion error messages to mention the new promotion mode cc @mattjj

I think we're not going to do this, because the desirable semantics are unclear.


Add go:embedded files to skaffold dependencies

How to categorize this PR?
/area dev-productivity
/kind enhancement
What this PR does / why we need it:
Adds dependencies introduced by embedded files to skaffold.yaml. Unfortunately, go list does not recursively resolve embeddings, so we have to approximate, by checking every (indirect) local dependency. This includes some false positives, since this does not check if the embedding is actually used, but I think its good enough.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Release note:

`skaffold` now rebuilds components if embedded files have changed.

/ok-to-test


[Bug] Animated Gif Lag

Device information

Tachiyomi version: Tachiyomi SY Preview r131 (f1a6218a4)
Android version: 8.1
Device: Mi Pad

Steps to reproduce

Open Series
Open chapter with gif

Expected behavior
Gif loads then plays smoothly
Actual behavior
Gif loads and plays around 1 frame per second
Other details
I have the same issue on Tachiyomi AZ.
It works perfectly without any lag on J2K.
Haven't tried the normal version or neko so can't say anything about those.
Maybe you can check what he does to fix it and port it over 👀

This is a Tachi stable issue, I probably wont fix this myself, if a fix is added to Tachi stable, I will pull it


Ferdi doesn't log in automatically anymore

Usually when I close Ferdi and open it again it automatically logs in. Now it doesn't giving the following error:
There were errors while trying to perform an authentication request. Please try logging out and back in if this error persists.
I can confirm I have tried logging out and back in again, but it doesn't help the issue. I also uninstalled and reinstalled Ferdi, but that also doesn't help the issue.
Also (not sure if it's related), but when I manually log back in again I can't log back into Whatsapp. Just a loading screen saying "organization messages" forever. While it works fine when I try in my browser.

@NLatuny - please provide more details: what version of Ferdi, what specific service, what version of said recipe, etc would be welcome for starters.
If its specific to Whatsapp, and the new multi-device beta, there's already a few bugs - please close this one if so

@vraravam I'm using Ferdi v5.6.3. Not sure what service matters here as I mean the Ferdi login.
The thing about whatsapp was a separate thing, but you can scratch that for now.

ok - so which server are you trying to connect to (for Ferdi login) - is it the Franz server, (hosted) Ferdi server or some other?

I use the Ferdi server.

ok - unfortunately, as a user, I moved away from the Ferdi server due to such flakiness.
I do not have visibility into the current state of the deployed version, and so cannot comment on this.
@vantezzen @kytwb - maybe you can help debug?

I tried Ferdi's latest flatpak. Logged into discord. Every time I restart Ferdi I'm logged out again.
I am using Ferdi without an account (at the startup screen). Surely the app doesn't force us to store login cookies online just to use it? I'm uninstalling Ferdi now and going back to Discord's own app...

@vraravam I decided to try Ferdi again. Latest flatpak version 5.6.5. The login is kept between Ferdi restarts now, with local-server credentials. Good! :)
Ferdi uses more RAM than Discord (500mb vs 300mb) but it's more flexible so I think I'll switch to Ferdi.

@NLatuny @vraravam Wellllll damn, I'm logged out again after restarting Ferdi flatpak. Hrm...


Run FindCompactionTmpFiles utility via accumulo Admin

Added command option FindCompactionTmpFIles to admin commands.
closes issue #3986

Should there be a check to see if the system is up or if there running compactions?  If this was run on a system with active compactions, would it find and delete files that were currently being written to be the compaction process?
As a stand-alone utility it is a little harder to run the command - if elevated to an admin utility, would that increase the chances that it would be run on an active system?

Should there be a check to see if the system is up or if there running compactions? If this was run on a system with active compactions, would it find and delete files that were currently being written to be the compaction process?
As a stand-alone utility it is a little harder to run the command - if elevated to an admin utility, would that increase the chances that it would be run on an active system?

FindCompactionTmpFiles first finds the tmp files, then removes from that set any file that is referenced from currently executing compactions.

May want to wait on merging this until after #4807 is merged, because I suspect there will be conflicts, or at least some related changes.

I remembered this was supposed to wait until https://github.com/apache/accumulo/pull/4807 was merged. https://github.com/apache/accumulo/pull/4807 has since been merged. I don't think tests have been added, maybe that needs to be addressed before this is merged


Sort yearn vaults on the Earn Page by balance, total value locked, and APY.

IMPORTANT: NO BOUNTIES WILL BE PAID FROM 2/5 - 2/14 - IF YOU COMPLETE THIS BOUNTY DURING THIS TIMEFRAME WE WILL REMIT PAYMENT ON 2/14
Background
On the Earn Page, there are a list of yearn vaults that a user can interact with. We want to sort this list first by user's fiat balance if they have assets locked in vaults. So a user will see all of the vaults for which they have a balance at the top. Next we want to sort by a combination of the total value locked in the vault and the vault's APY, with the highest TVL/APY at the top (desc).
Our sorting strategy here will be all vaults with a user balance will be at the top and sorted desc by fiat value of the user's balance. All vaults greater than $10M USD in TVL are directly below vaults with a user balance and sorted by APY. For vaults under $10M USD but greater than $1M USD in TVL they are listed below vaults over $10M TVL and sorted by APY. All vaults with a TVL of less than $1M USD are listed below the vaults with greater than $1M USD and are sorted by APY.
Acceptance Criteria:
1: Vault's with a user balance are at the top and sorted desc by the user's total fiat balance in the vaults.
2: Vaults with a TVL of over $10M USD, but no user balance, are listed below vaults with a user balance and sorted desc by APY.
3: Vaults with a $1M USD < TVL < $10M USD are listed below vaults with greater than $10M USD in TVL and are sorted desc by APY.
4: All other vaults with a TVL of less than $1M USD are listed below vaults with a TVL of greater than $1M USD and they are sorted desc by APY.
Screenshot

Bounty Hunters

Join our discord here. Once you join you can see more about this issue here
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info

@chxash - can you please submit work on Github and I can get this paid out ASAP!
Thanks!

@chxash - can you please submit work on Github and I can get this paid out ASAP!
Thanks!

oops .. done ! thank you


Using table alias to qualify column name in ORDER BY results in error on SELECT DISTINCT

Describe the bug
When using SELECT DISTINCT with a table alias, we get a syntax error telling us that ORDER BY expressions must appear in select list, even though the column is in the select list (but is aliased).
Error message/log
psql -h localhost -p 4566 -U root -d dev <<SQL
with integers as (
  select 1 as numeral, 'first' as ordinal
)

select distinct numeral, ordinal
from integers i
order by i.numeral
SQL
ERROR:  Failed to run the query

Caused by:
  Invalid input syntax: for SELECT DISTINCT, ORDER BY expressions must appear in select list

To Reproduce
This works:
psql -h localhost -p 4566 -U root -d dev <<SQL
with integers as (
  select 1 as numeral, 'first' as ordinal
)

select distinct numeral, ordinal
from integers i
order by numeral
SQL
 numeral | ordinal
---------+---------
       1 | first
(1 row)

Whereas this doesn't (only difference is that we use order by *i*.numeral:
psql -h localhost -p 4566 -U root -d dev <<SQL
with integers as (
  select 1 as numeral, 'first' as ordinal
)

select distinct numeral, ordinal
from integers i
order by i.numeral
SQL
ERROR:  Failed to run the query

Caused by:
  Invalid input syntax: for SELECT DISTINCT, ORDER BY expressions must appear in select list

I also tried ensuring that it's the literal same expression in the select list, but that fails as well:
psql -h localhost -p 4566 -U root -d dev <<SQL
with integers as (
  select 1 as numeral, 'first' as ordinal
)

select distinct i.numeral, ordinal
from integers i
order by i.numeral
SQL
ERROR:  Failed to run the query

Caused by:
  Invalid input syntax: for SELECT DISTINCT, ORDER BY expressions must appear in select list

Expected behavior
I expected to be able to use the table alias to qualify the column name in the ORDER BY clause of my query.
How did you deploy RisingWave?
Both via docker-compose.yml locally, and RisingWave Cloud.
The version of RisingWave
 psql -h localhost -p 4566 -U root -d dev <<SQL
select version()
SQL
                                    version
--------------------------------------------------------------------------------
 PostgreSQL 13.14.0-RisingWave-1.8.1 (5b6bef82dc55638c024190654c7ef9896e21687d)
(1 row)

Additional context
No response

Thanks for reporting the issue. We will try to get it fixed.
However, the preferred syntax of ORDER BY is to use the unqualified output name, rather than a (qualified or unqualified) input name.
select distinct numeral as column_a, ordinal
from integers i
order by column_a; -- This is referring to the output after distinct.

select distinct numeral as column_a, ordinal
from integers i
order by numeral; -- This now fails similar to `i.numeral` because RisingWave fails to recognize it is same column as `column_a`

Thanks -  good to know!


Subscriptions - example.

Hi Mr Ali Rizwan,
Could you please create some example of how to use Subscription?
{mySubs: { output: '[myType]', resolver: customResolver, subscriber: customSubscriber }}

Especially how to build a subscriber from the documentation example above. Many other examples are in the tests in the repo, unfortunately I could not find subscription examples.
Thank you!

Hi, Subscriptions are being re-done in the newer version. If you want to keep using the current version I can prepare an example of that and share with you.

Hi, thank you for answer. Will it be a complete change of approach in the new version? We still use v.0.1.71 in our project.

It most probably would be different because I’m trying to build a tool agnostic subscription model so you can use redis or any other tool in front of it to achieve this. Current implementation in 0.x is a bit tightly coupled.
Sent from my iPhone
On Mar 17, 2021, at 9:12 AM, mrfy @.***> wrote:

Hi, thank you for answer. Will it be a complete change of approach in the new version? We still use v.0.1.71 in our project.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/almost-full-stack/graphcraft/issues/45#issuecomment-800886262, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADAUKBSDNNOEJ6SANK6HW4DTEBP6NANCNFSM4ZAR44EA.

Then could you please make an example lin version 0.x?  We will update the package.

unfortunately 0.x also don't support subscriptions out of the box however it's in progress for the next version. However if you want some help in this regard it would be better if we can schedule a quick zoom/gmeet/discord call?

Hi, i am interested in subscriptions. do you have an example with the actual version or when do you plan the next?


Switch from failure to anyhow and thiserror.

Fixes #861.

@daxpedda great! How about merging this into master instead of web-sys?

@daxpedda great! How about merging this into master instead of web-sys?

Will rebase when I'm back home.

Done.


Not support for rails-assets

I used to use rails-assets gem to build my projects, just like rails-assets-bootstrap,so can you tag your repository so I can add it to the rails-assets components lists?You can find rails-assets here:"https://rails-assets.org/components"

closing as a duplicate of https://github.com/zurb/twentytwenty/issues/20
rails-assets runs on bower. Once we support bower, rails-assets will start working too.


Improve shell scripts

This fixes some slight inaccuracies within the shell scripts including

Use -r so backslashes aren't mangled
Use $PWD for "speed"
Use $() instead of deprecated backticks for subshells
Quote arguments to prevent word splitting
Don't use == in single equals sign
Use command -v instead of which (which is deprecated in the last version of debian)

Some of these aren't strictly required in these exact cases, but in my opinion its better to be safe than sorry, and people are going to be using these as baseline examples, so it might be a good idea to make 'em robust

Hi! Thanks
LGMT :)


Use NumPy logic for lessthan in sort to move NaNs to the back.

Fix #8382.
With this patch Numba is faster than NumPy using the reproducer from the issue:
from numba import njit
import numpy as np

@njit
def test(a):
    return np.sort(a)

arr = np.full(100_000, np.nan)
# compile it
test(arr[:10])

%time test(arr)
# CPU times: user 1.27 ms, sys: 153 µs, total: 1.42 ms
# Wall time: 1.42 ms

%time test.py_func(arr)
# CPU times: user 2.65 ms, sys: 39 µs, total: 2.69 ms
# Wall time: 2.72 ms

mainline is broken due to a problem scipy package. Needs to rerun CI when that is fixed

Thank you for the patch!


linuxPackages.virtualBoxGuestAdditions: fix build

Motivation for this change
Fixes #70470
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @svanderburg

I tried to file a bug report upstream, but after persistent requests after things like my phone number I got fed up and gave up.


Can not install Humanizer with nuget on VS2015 Beta

How to reproduce:

Install VS2014 CTP

Create a new "XXX vNext project" (Console vNext appliation  is just fine)

Add Humanizer to the project.json

{
    "dependencies": {
        "Humanizer": "1.26.1"
    },

    "configurations" : {
        "net45" : { },
        "k10" : {
            "dependencies": {
                "System.Console": "<IP_ADDRESS>"
            }
        }
    }
}

Build

It says that '.' is wrong assembly
    Error	1	The command ""C:\Users\hazzik\.kre\packages\kre-svr50-x86.0.1-alpha-build-0446\bin\klr.exe" "C:\Users\hazzik\.kre\packages\kre-svr50-x86.0.1-alpha-build-0446\bin\lib\Microsoft.Framework.Project\Microsoft.Framework.Project.dll" build --check "C:\Users\hazzik\Documents\Visual Studio 14\Projects\ConsoleApp1\ConsoleApp1"" exited with code -1.	C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\AspNet\Microsoft.Web.AspNet.targets	124	5	ConsoleApp1

------ Build started: Project: ConsoleApp1, Configuration: Debug Any CPU ------
  System.BadImageFormatException: Could not load file or assembly '_._' or one of its dependencies. An attempt was made to load a program with an incorrect format.
  File name: '_._' ---> System.BadImageFormatException: Could not load file or assembly '_._' or one of its dependencies. An attempt was made to load a program with an incorrect format.
  File name: '_._'

     at System.Reflection.AssemblyName.nGetFileInformation(String s)
     at System.Reflection.AssemblyName.GetAssemblyName(String assemblyFile)
     at Microsoft.Framework.Runtime.NuGetDependencyResolver.GetAssemblyName(String fileName)
     at Microsoft.Framework.Runtime.NuGetDependencyResolver.Initialize(IEnumerable`1 packages, FrameworkName targetFramework)
     at Microsoft.Framework.Runtime.WalkContext.Populate(FrameworkName frameworkName, IList`1 libraries)
     at Microsoft.Framework.Runtime.DependencyWalker.Walk(String name, SemanticVersion version, FrameworkName targetFramework)
     at Microsoft.Framework.Project.BuildManager.PrepareCompiler(Project project, FrameworkName targetFramework)
     at Microsoft.Framework.Project.BuildManager.CheckDiagnostics(Project project, FrameworkName targetFramework, List`1 diagnostics)
     at Microsoft.Framework.Project.BuildManager.Build()

  Build succeeded.
      0 Warnings(s)
      0 Error(s)

  Time elapsed 00:00:05.3851980
C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\AspNet\Microsoft.Web.AspNet.targets(124,5): error MSB3073: The command ""C:\Users\hazzik\.kre\packages\kre-svr50-x86.0.1-alpha-build-0446\bin\klr.exe" "C:\Users\hazzik\.kre\packages\kre-svr50-x86.0.1-alpha-build-0446\bin\lib\Microsoft.Framework.Project\Microsoft.Framework.Project.dll" build --check "C:\Users\hazzik\Documents\Visual Studio 14\Projects\ConsoleApp1\ConsoleApp1"" exited with code -1.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

I'm going to close this out as it's been a long time. Please reopen if it's still an issue with the latest RTM VS bits and DNX beta 8.


fix: bump minimum ios version due to localization

As localization was recently added here, this requires a bump in the minimum OS version.
Refer to https://developer.apple.com/documentation/swift/string/init(localized:table:bundle:locale:comment:) for more details.

Thanks for catching that.


Error with annotate.nf

I'm trying to run annotate.nf on my vcf files (called using strelka). With both snpeff and vep I am repeatedly facing issues. Any help or suggestion will be very grateful.
Error for vep:
.command.sh: line 2: /opt/vep/src/ensembl-vep/vep: No such file or directory
Command: nextflow run /workspace/pipeline/sarek/Sarek/annotate.nf --annotateTools Strelka --genome_base /workspace/pipeline/sarek/Sarek/References/GRCh38/ --genome GRCh38 --tools vep --tag latest -profile singularity
Error for snpeff:
.command.sh: line 2: SNPEFF_HOME: unbound variable
Command: nextflow run /workspace/pipeline/sarek/Sarek/annotate.nf --annotateTools Strelka --genome_base /workspace/pipeline/sarek/Sarek/References/GRCh38/ --genome GRCh38 --tools snpEff --tag latest -profile singularity
Full error message for snpeff
Sarek - Workflow For Somatic And Germline Variations ~ 2.2.1 - 587f366d31
Command Line: nextflow run /workspace/pipeline/sarek/Sarek/annotate.nf --annotateTools Strelka --genome_base /workspace/pipeline/sarek/Sarek/References/GRCh38/ --genome GRCh38 --tools snpEff --tag latest -profile singularity
Profile     : singularity
Project Dir : /workspace/pipeline/sarek/Sarek
Launch Dir  : /home/anurag/sarek_processing
Work Dir    : /home/anurag/sarek_processing/work
Out Dir     : /home/anurag/sarek_processing
Genome      : GRCh38
Genome_base : /workspace/pipeline/sarek/Sarek/References/GRCh38/
Tools       : snpeff
Annotate on : strelka
Containers
Repository   : docker://maxulysse
Tag          : latest
Reference files used:
snpeffDb    :
GRCh38.86
[warm up] executor > local
WARN: Singularity cache directory has not been defined -- Remote image will be stored in the path: /home/anurag/sarek_processing/work/singularity
Pulling Singularity image docker://maxulysse/sarek:latest [cache /home/anurag/sarek_processing/work/singularity/maxulysse-sarek-latest.img]
[6a/85b18d] Submitted process > RunSnpeff (strelka - Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels.vcf.gz)
[6d/6f7fc9] Submitted process > RunSnpeff (strelka - Strelka_MDB_02_91_vs_MDB_05_118_somatic_snvs.vcf.gz)
[7a/55e80e] Submitted process > GetVersionSnpeff
ERROR ~ Error executing process > 'RunSnpeff (strelka - Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels.vcf.gz)'
Caused by:
Process RunSnpeff (strelka - Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels.vcf.gz) terminated with an error exit status (1)
Command executed:
java -Xmx7g   -jar $SNPEFF_HOME/snpEff.jar   GRCh38.86   -csvStats Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels_snpEff.csv   -nodownload   -canon   -v   Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels.vcf.gz   > Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels_snpEff.ann.vcf
mv snpEff_summary.html Strelka_MDB_02_91_vs_MDB_05_118_somatic_indels_snpEff.summary.html
Command exit status:
1
Command output:
(empty)
Command error:
.command.sh: line 2: SNPEFF_HOME: unbound variable
Work dir:
/home/anurag/sarek_processing/work/6a/85b18d86eff1348fd19fd2ff181130
N E X T F L O W  ~  version 18.10.1 5003
Thanks

Hi,
I'm guessing you're using the containers tagged lastest, forcing the use of a previous version like 2.2.1 or even 2.2.2 should definitively fix that.
Can you try the same command with --tag 2.2.1, or --tag 2.2.2?
What is your use case, and where are you running Sarek?

Hello,
Thanks for the prompt help. It's working perfectly fine with the --tag 2.2.1. We have some exome sequencing data and we are trying to call variants from them. We are using it on our computing cluster.
Thanks.

Don't hesitate if you have any more issue ;-)


Add ReportBuildStatus Parameter to CodeBuild Source

It looks like there's an additional parameter for the codebuild Source object that is missing
AttributeError: Source object does not support attribute ReportBuildStatus
AWS Reference:  https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-codebuild-project-source.html#cfn-codebuild-project-source-reportbuildstatus

https://github.com/cloudtools/troposphere/commit/6c53a9b5cc99fe43a1d190ea09ff7c689fafffa7


Necessary documentation improvements

extensive entry for search_dates()
extend list of supported languages and locales

TBC

I believe that removing references to Scrapy Cloud is interesting too (https://dateparser.readthedocs.io/en/latest/installation.html#deploying-dateparser-in-a-scrapy-cloud-project), as the library is general enough and not directly related to Scrapy.

* extensive entry for `search_dates()`

Could you elaborate? What do we need to cover beyond what is already covered in https://dateparser.readthedocs.io/en/latest/#search-for-dates-in-longer-chunks-of-text?

I believe that removing references to Scrapy Cloud is interesting too (https://dateparser.readthedocs.io/en/latest/installation.html#deploying-dateparser-in-a-scrapy-cloud-project), as the library is general enough and not directly related to Scrapy.

Is this change still needed, @rennerocha ? I could work on it

I have just created a ticket to improve the search_dates section in the docs: https://github.com/scrapinghub/dateparser/issues/737
As we currently have a list of locales (it is in the README, I think we should move it to the docs, but that's another point) and we have an open ticket regarding the section about Scrapy Cloud in the docs: https://github.com/scrapinghub/dateparser/issues/724, I think we can close this ticket, as it's too broad and incomplete.
Thank you, guys!


Windows agent stop/start service commands inconsistency

It there any particular reason the Windows agent stop/start commands are different
Stop command
agent.exe stopservice     (stops the agent within the service control manager)
Start command
agent.exe start-service   (starts the agent within the service control manager)
can we have either
stopservice
startservice
or
stop-service
start-service

fixed in https://github.com/DataDog/datadog-agent/pull/13870


debug-bridge: passing parameters

I am misusing the debug-bridge a bit for some prototyping since it is the most workable way for me to edit JS code to be executed in Zotero (that I have found so far anyway). One thing I noticed that could be improved for the regular use case too is that it does not seem to be possible to pass parameters from the outside. I have tried the one obvious way that curl allows (i.e., adding another --data-binary argument), e.g., curl -s -H "Content-Type: application/javascript" -X POST --data-binary @$js_file --data-binary "testvar=123" http://<IP_ADDRESS>:23119/debug-bridge/execute.
That almost makes it to the final script but with the & from the encoding prepended, e.g., for an empty $js_file the call above logs the following:
(3)(+0000002): debug-bridge: executing &testvar=123

(3)(+0000001): debug-bridge failed (0ms): SyntaxError: expected expression, got '&'

(5)(+0000001): HTTP/1.0 500 Internal Server Error X-Zotero-Version: 5.0.87 X-Zotero-Connector-API-Version: 2 Content-Type: application/text debug-bridge failed: SyntaxError: expected expression, got '&' init@chrome://zotero-debug-bridge/content/debug-bridge.js:33:24 Zotero.Server.DataListener.prototype._processEndpoint<@chrome://zotero/content/xpcom/server.js:524:26 tryCatcher@resource://zotero/loader.jsm -> resource://zotero/bluebird/util.js:16:16 module.exports/PromiseSpawn.prototype._promiseFulfilled@resource://zotero/loader.jsm -> resource://zotero/bluebird/generators.js:97:18 module.exports/Promise.coroutine/<@resource://zotero/loader.jsm -> resource://zotero/bluebird/generators.js:201:9 Zotero.Server.DataListener.prototype._bodyData@chrome://zotero/content/xpcom/server.js:344:5 Zotero.Server.DataListener.prototype._headerFinished@chrome://zotero/content/xpcom/server.js:316:5 Zotero.Server.DataListener.prototype.onDataAvailable@chrome://zotero/content/xpcom/server.js:210:7

One might be able to hack around that by ending the JS file with something like testvar=" and using --data-binary "123\";"or so but... BUT. ;)
This could facilitate testing by supplying external information to the script that it cannot receive otherwise.

In the code that executing, there's an object called query that holds all url query parameters, so if you call http://<IP_ADDRESS>:23119/debug-bridge/execute?par1=v1&par2=v2 you'd have the equivalent of having
let query = { par1: 'v1', par2: 'v2' }

at the top of the source.

or to be precise: if you submit return 'hello', the debug bridge wraps that as
async function(query) {
  return 'hello'
}

and then calls that function with the object holding the query parameters as the argument.

Ah indeed. My JS skills are to poor to recognize the construction of the parameter of the callback (https://github.com/retorquere/zotero-better-bibtex/blob/4167e61948684930e58ab4c99cfb207ce7e6a846/test/fixtures/debug-bridge/content/debug-bridge.js#L33). Thanks a lot for the quick enlightenment (and the debug bridge)!

No worries. Questions are free.


Darker Theme: Low contrast on excluded folders in tree

In this situation, the contrast between the gray folder name text and the blue highlight is pretty low. It's fairly hard to read.
I know the folders are excluded (and that's why the name is grayed back and the folder icon is red), but I'm not sure why there is a permanent row highlight.

FYI, for the Darker theme there is a predefined "File Colors" in Appearance > File Colors. Just put Darker above all else and you should have a better option.
As for the gray color, this is because this is the color of ignored files. I will look at it.

try the latest release (hotfix) and tell me if it happens again (grayed file). I didn't test thoroughly bcz im pretty tired but I will in a short while.

Ah, cool. That does look a lot better; thanks.

Good eyes noticing where that blue came from :wink:.

Looks the same after the hotfix:

But, TBH, the contrast is way better once those file colors get ordered with darker first. That's kind of a fiddly thing to have to know to do, though, isn't it? Are there any alternatives?
Get some sleep! I'll be here tomorrow :+1:.

I agree, and I wrote a whole section in the README, because I really don't have a lot of options here, because I can't change the user's settings programmatically.
Think that you could have your own file colors and don't need to use the ones I'm providing.
TL;DR: read the README ;)
--
Elior
On Jul 13, 2017, 00:49 +0300, Chris Harwood<EMAIL_ADDRESS>wrote:

Looks the same after the hotfix:
But, TBH, the contrast is way better once those file colors get ordered with darker first. That's kind of a fiddly thing to have to know to do, though, isn't it? Are there any alternatives?
Get some sleep! I'll be here tomorrow 👍.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Oh boy. Now I'm that guy.


make example work when pasted into the console

arguably less beautiful but copy-n-paste friendly

Thanks for the input @bryanhuntesl!
Personally, I think the majority of the value in our documentation is as something to read, not something to copy and paste into the console, so I think it's smart to err on the side of what folks would expect to read. Since the Elixir formatter would format that code (almost) as we have it now, I would prefer to keep it as it, but I'll wait for @PragTob to weigh in as well before we make the final call here.

@devonestes is spot on here imo. Formatting all samples in samples and then also in the README is actually something I'd like to do :)
I'd just like to add that benchmarking in the console is highly discouraged - it's not your actual environment, performance characteristics might be different. Not sure we're clear on that in the README but yeah one shouldn't. So in that sense, it'd be the wrong affordance imo.
Thanks a lot for your contribution though @bryanhuntesl !
test is/was a small flaky:
  1) test .run_scenarios very fast function times are almost 0 with function call overhead elimination (Benchee.Benchmark.RunnerTest)
     test/benchee/benchmark/runner_test.exs:185
     Assertion with <= failed
     code:  assert median <= 60
     left:  127.0
     right: 60
     stacktrace:
       test/benchee/benchmark/runner_test.exs:197: (test)


Make prune true by default

Currently prune is false by default which makes users, including myself, too easily "forget" putting it on.
This PR makes prune on by default and adding --no-prune option to make it false. --prune is still supported, so nothing should break.

Just thinking should we have a sanity check

Check that there are resources to be shot and warn&prompt if user is about to make an empty shot?

Actually, getting empty shot from the src folder would hit this before shooting the stack:
if resources.empty?
        warn 'nothing to do!'
        exit
      end


Question: Difference between ripgrep and fastmod?

Amber looks cool but what is the difference between ambs and rg as well as ambr and fastmod?
It would be cool as well to include benchmark for rg. And hyperfine could be used for benchmark.

Thank you for your suggestion.
I updated benchmark results with hyperfine.
rg/fastmod is fast in the case of many files,
and amber has advantage in the case of large file search.

Nice, cool to see the benchmark. I am surprised that amber seems faster here. Maybe @BurntSushi might be interested in this.

@BurntSushi
Thank you for your comment.
Currently amber show line by match. So the line matched repeatedly is duplicated.
I'll fix it, and try regex::bytes::Regex.


build: add some workarounds for Windows, explicitly link dependencies

CMake prior to 3.16 did not correctly emit the import library into the requested
location.  Adjust the library emission path instead to workaround missing import
libraries.  Explicitly link against Foundation in TSCBasic which is needed on
non-Darwin targets.  Add the ability to build against an explicit build of
Foundation or the toolchain provided one.

@swift-ci please smoke test

@swift-ci please test

@swift-ci please test Linux platform

@swift-ci please test

Windows CI: https://dev.azure.com/compnerd/windows-swift/_build/results?buildId=17110&view=logs&j=b9e62f99-1a98-5ed7-01d2-f4794231ed79

@swift-ci please test

I think that these have been redone already.


Run Tests on Device

Is your feature request related to a problem? Please describe.
I think the only feature that I see GDUnit3 missing compared to GUT is the ability to run the tests in game. For instance sometimes weed bugs out on other peoples devices by creating tests around my assumptions on their device then I can give them the game and have them run the tests to see if I am correct. This very handy for mobile devices.
Describe the solution you'd like
I would like to be able to call some code from a scene and have it execute the tests and display the results. I also need a signal or variable that I can check to know when the tests are finished running and I would like to be able to retrieve results.
Describe alternatives you've considered
I am still very new to GDUnit3 and perhaps this is already doable. But with few examples I do not see how I could achieve the same thing and call all of my tests.
Additional context
N/A.

hi @FeralBytes,
to test Godot scenes you have to use the scene_runner where is well documented.
The Tutorials section is work in progress and needs to be updated.
If you wan't you can also download the advanced examples direct over the GdUnit3 tool.

If you have future question feel free to join the discord server linked on the main page.

@FeralBytes is this feature request still valid?

closed for now, will be reopen if it need


Rx Nullable

Example:
Rx< String? > rx = RxString(null);
The most necessary times is when we need to store an object like "loggedUser", it is very necessary a nullable option.

Simply use this:
Rxn<String> rx = Rxn<String>();

Simply use this:
Rxn<String> rx = Rxn<String>();

Is this somewhere in the documentation? How did you find this?

Simply use this:
Rxn<String> rx = Rxn<String>();

Thank you man, this is what I needed, is this somewhere in the documentation? How did you find this?

I just came across this feature when I was looking for that exact problem on Stackoverflow. I believe that this is not well documented


Issue with the rendering of the program

It sometimes goes black and then gets redrawn. My OS is Windows 11 and my GPU is an AMD Radeon 680M.

bump


BUG: coo_matrix does not validate the shape argument.

In a stackoverflow question, a user created a coo_matrix as follows:
a = coo_matrix([0, 11, 22, 33], ([0, 1, 2, 3], [0, 0, 0, 0]))

Apparently they neglected to put the values and indices into a tuple, and instead of creating an array with shape (4, 1), they got an array with shape (1, 4):
In [4]: a = coo_matrix([0, 11, 22, 33], ([0, 1, 2, 3], [0, 0, 0, 0]))

In [5]: a.shape
Out[5]: (1, 4)

In [6]: a.A
Out[6]: array([[ 0, 11, 22, 33]])

The problem is that the call shown above is equivalent to
a = coo_matrix(arg1=[0, 11, 22, 33], shape=([0, 1, 2, 3], [0, 0, 0, 0]))

and clearly that shape value is not valid.  The call should generate a ValueError instead of silently ignoring the shape argument.  Note that the shape is completely ignored--the following does not generate an error:
b = coo_matrix([[0, 1], [2, 3]], shape='foo')

SciPy version is the master branch:
In [9]: import scipy

In [10]: scipy.__version__
Out[10]: '1.3.0.dev0+5539300'

Thanks for reporting this, Warren. Thankfully it's an easy fix, see gh-9920.


Bug When Updating Payload

Since the panel updates the payload by replacing it with a payload sent by the client using a GET request.
The GET request is used because of Chrome blocks HTML code in POST requests for some XSS reasons.
Once replaced the

$_SERVER['REQUEST_URI']

contains the payload parameter.

The issue has been fixed and committed.


Research: WebEx Meetings Bombing

[ ] Check for the possible endpoint to get the list of attendees for the meetings
[ ] List the payload required if any
[ ] Implement the changes in TA
[ ] Implement the Dashboard Panel / SPL comparing the list of attendees when creating meeting vs those who joined during the live meeting.

Here is the endpoint that may help get enrollment based on sessionKey
https://developer.cisco.com/docs/webex-xml-api-reference-guide/#!getenrollmentinfo/getenrollmentinfo
Part of Payload:
<bodyContent
            xsi:type="java:com.webex.service.binding.attendee.GetEnrollmentInfo">
            <sessionKey>987566789</sessionKey>
</bodyContent>```


OIDC: backchannel logout default configuration only accepts a single client id

Describe the bug
Our code verifying the backchannel logout token, tries to verify the aud claim of the token by default. It is possible to configure a single value that is allowed to appear in that claim (defaulting to web).
I think we're doing something wrong here. Keycloak for example by default sets the aud claim to the value of the clientid for which the session was created. So with the current approach only sessions initiated  one specific oidc client can be logged out (web by default). For sessions created by any other clients (i.e. the desktop of mobil apps) the validation of the logout token will fail because the aud claim does not match the expected value.
As a workaround we can disable the validation of the aud claim for now with PROXY_OIDC_SKIP_CLIENT_ID_CHECK=true
I am still not sure what the correct validation of the aud claim would look like in ocis. We simply don't know all the valid client ids that could appear in logout tokens. (At least when DCR is enabled in the idp).

cc @micbar @wkloucek

Thanks. I see the PR was merged. IMO that makes sense for the time being.

Fixed.


Enhancement/29 support keyboard binds for pull requests

Added support for key bindings to specific events in bit bucket pull requests.
Default Key Mapping
Tab Selection

Key
Action

1
Selects the Overview tab

2
Selects the Commits tab

3
Selects the Activity tab

Page Navigation and Scrolling
Upper case letters mean that you need to press shift and the letter

Key
Action

N
Scrolls to the Next comment on the page.

P
Scrolls to the Pprevious comment on the page.

g g
Scrolls to the top of the page

G
Scrolls to the bottom of the page

Oh geez I didn't realize the push would kick off CI again... I'm working on that part now :)

Code climate and travis have conflicting opinions on what to do about IFFEs, so they won't both pass for my code, it will always be one or the other.
code climate wants the definitions like
(function ($) { }(jQuery));

Travis (npm test) wants it like this:
(function ($) { })(jQuery);

... I personally prefer the latter and since npm test is what's going to run on most people's machines (is it possible to run the same lint as on code climate?) I think it makes sense to stick with that and update the rules on code climate. But it's ultimately your decision.

No problem, I'll ignore Code Climate for now, and then I'll disable this check there.

this is  a very useful addition and will absolutely be using it.  Thanks!
I noticed a small typo in the readme.   "scrolls to the Ppreviou"  Sorry, I would have liked to give this a proper review this weekend but I wasn't around my lappy.
No biggie but maybe something that can get corrected on the next PR that goes in.
I did pull the code down and verified the keymaps work in Windows.  Looks great.
👍

Good catch on the typo, looks like it got vimmed by mistake somehow


[boost-atomic] build failure on alpine linux due to missing headers

Host Environment

OS: Alpine Linux 3.15.3
Compiler: g++ (Alpine 10.3.1_git20211027) 10.3.1 20211027

To Reproduce
Steps to reproduce the behavior:

Install minimum set of system packages needed to run vcpkg on an alpine system.
git clone vckpg
set env variable VCPKG_FORCE_SYSTEM_BINARIES to 1
run bootstrap-vcpkg.sh (it pulls down the pre-build muslc binary of vcpkg)
Attempt to install boost-atomic

./vcpkg install boost-atomic

vcpkg reports:
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:158 (message):
    Command failed: /usr/bin/cmake --build . --config Debug --target install -- -v -j9
    Working Directory: /src/vcpkg/buildtrees/boost-atomic/x64-linux-dbg
    See logs for more information:
      /src/vcpkg/buildtrees/boost-atomic/install-x64-linux-dbg-out.log

Failure logs
Output log shows root cause...
    "/usr/bin/c++"   -fPIC -g -m64 -pthread -O0 -fno-inline -Wall -g  -DBOOST_ALL_NO_LIB=1 -DBOOST_ATOMIC_SOURCE -DBOOST_ATOMIC_STATIC_LINK=1 -DBOOST_ATOMIC_USE_SSE2 -DBOOST_ATOMIC_USE_SSE41  -I"../include" -I"/src/vcpkg/installed/x64-linux/include" -I"../src"  -c -o "/src/vcpkg/buildtrees/boost-atomic/x64-linux-dbg/boost/build/599c3892010ad001b78ddcb47e1afb19/lock_pool.o" "../src/lock_pool.cpp"

In file included from ../include/boost/atomic/detail/platform.hpp:151,
                 from ../include/boost/atomic/detail/capabilities.hpp:18,
                 from ../include/boost/atomic/capabilities.hpp:18,
                 from ../src/lock_pool.cpp:33:
../include/boost/atomic/detail/futex.hpp:52:10: fatal error: linux/futex.h: No such file or directory
   52 | #include <linux/futex.h>
      |          ^~~~~~~~~~~~~~~
compilation terminated.

Additional context
This problem can be resolved by adding a system package.

apk add linux-headers

Expectation
The port should tell the user what system packages are missing, if any, in order to build the port.  For example, if you attempt to install openssl and the perl package is not installed, the user is presented with a cmake message indicating the package could not be found.

@mprather, thanks for reporting this issue!
I'm not sure if it's neccessary to add a message to boost-atmoic, I don't have this problem on ubuntu, the linux-headers looks installed on my machine by default, our CI test machines are same, so can I say the problem on specific alpine linux?
cc @yurybura   cc @BillyONeal

@PhoebeHui yes, this is an alpine only config issue. I imagine that since alpine tends to be a minimalist config, it tends steer away from large dependency trees like what you find on other distros. for example, on debian, those headers are provided by linux-libc-dev, which is needed by libstdc++-10-dev, needed by g++-10). on arch, those headers are managed by linux-api-header and dependency tree is different (linux-api-header->glibc->gcc-libs->gcc).

I think adding a message to boost-atomic is reasonable.

I have submitted a PR #23921 to add the message to boost-atomic.


Fix the link to the Laravel logo in the README

#34178 Made this change on the 6.x branch now

@iruoy Want to submit the same (if not done already) to laravel/laravel?
There was a PR there but was closed because working on a single machine is more than enough lol


added CONTRIBUTING.md in Indonesian (CONTRIBUTING-id.md)

What does this PR do?
Add resource(s)
For resources
Description
I added CONTRIBUTING.md into the Indonesian translation.
Why is this valuable (or not)?
this will be useful if there are Indonesians who want to contribute to this repository, then they will have no difficulty in reading the contributing guide
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[✓] Read our contributing guidelines
[✓] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Sorry about conflicts, we're merging oldest PRs first

I would separate the translation and the links to it into two separate PRs. The links generate the conflicts, and we can do a cleanup PR after things have settled a bit.

remove your changes to the other language files, that will fix the conflicts.

No deletions. Only a hard reset to previous state 😉 for this files

Stale.
@reski-mulud-muchamad Could you resolve conflicts? If context is needed, tell us!

added in #5816

added in #5816

Better... this complete #5816 translation, I mention in the backlog #5762. To don't deal with conflicts I'll pick up it in the sync PR that I'm working


Send regardless of IDS availability (#BE-7149)

This sends messages without checking for the recipient's current IDS availability. This will hopefully stop spurious IDS failures from causing messages to fail to send, and also prevent downgrading to different chats when one currently isn't available (which we also don't want, so this is good)
There is an issue (not a bug, just a design issue) here where the CBChat object contains an IMChats array which references all the correlated chats of the same service (iMsg vs SMS) for the contact this chat refers to. And CBChats are the ones who handle sending, so we need to use some data (currently we're using the guid of the IMChat) to find the IMChat to do the actual sending, and this just makes things more difficult than they need to be.
Ideally we wouldn't be using CBChats now since they are a relic of correlating chats, and we would just use Chats instead (since each Chat is 1:1 with an IMChat, which makes a lot of things easier for us), but that'll take a lot of work to untangle it, which time I assume we don't want to put in right now.
This also removes some dead code and adds more logs

Thanks for making this work with the existing code, agreed that CBChat should die longer term.


replaceState not updating state on first load

render(
  <Provider store={store}>
    <div>
      <Router history={history} children={appRoutes} />
      <DevTools />
    </div>
  </Provider>,
  documentSelector
);

my routes
export const routes = (requireAccess, accessLevels, accessEquals) => {
    return {
      component: 'div',
      childRoutes: [{
        path: '/',
        component: require('../containers/App'),
        indexRoute: require('./home')(requireAccess, accessLevels),
        childRoutes: [
          require('./dashboard')(requireAccess, accessLevels),
          require('./login')(requireAccess, accessLevels, accessEquals),
        ],
      }],
  }
};

home route
module.exports = (requireAccess, accessLevels) => {
  return {
      getComponent(location, cb) {
      require.ensure([], (require) => {
          cb(null, require('./components/Home'));
        });
      },
      onEnter: requireAccess(accessLevels.user),
  }
};

requireAccess will eventually return this
export const NotAuthorizedException = (nextState, replaceState) => {
    // console.log(nextState) - nextState.location.pathname is '/'
    replaceState({next: nextState.location.pathname }, '/login');
}

i go to http://localhost/ - i get pointed to /login but my state is null. this only happens on accessing / directly.

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Would you like to post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router.
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.

That's a bit much for us to digest since there's a ton of not-react-router stuff going on and we don't have the time hunting around for bugs that might not exist. Please boil it down to just react router API.


[AutoPR datafactory/resource-manager] Update swagger for DataFactory-SapOpenHub

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5146

(message created by the CI based on PR content)

Codecov Report

Merging #4299 into restapi_auto_datafactory/resource-manager will decrease coverage by <.01%.
The diff coverage is 53.62%.

@@                              Coverage Diff                              @@
##           restapi_auto_datafactory/resource-manager    #4299      +/-   ##
=============================================================================
- Coverage                                      53.45%   53.45%   -0.01%
=============================================================================
  Files                                          10222    10224       +2
  Lines                                         215048   215112      +64
=============================================================================
+ Hits                                          114960   114992      +32
- Misses                                        100088   100120      +32

Impacted Files
Coverage Δ

...ctory/azure/mgmt/datafactory/models/dataset_py3.py
37.5% <ø> (ø)
:arrow_up:

...y/azure/mgmt/datafactory/models/copy_source_py3.py
54.54% <ø> (ø)
:arrow_up:

...ctory/azure/mgmt/datafactory/models/copy_source.py
54.54% <ø> (ø)
:arrow_up:

...tafactory/azure/mgmt/datafactory/models/dataset.py
37.5% <ø> (ø)
:arrow_up:

...zure/mgmt/datafactory/models/linked_service_py3.py
46.15% <ø> (ø)
:arrow_up:

...ry/azure/mgmt/datafactory/models/linked_service.py
46.15% <ø> (ø)
:arrow_up:

...afactory/azure/mgmt/datafactory/models/__init__.py
100% <100%> (ø)
:arrow_up:

...mgmt-datafactory/azure/mgmt/datafactory/version.py
100% <100%> (ø)
:arrow_up:

.../datafactory/models/sap_open_hub_linked_service.py
35.71% <35.71%> (ø)

...afactory/models/sap_open_hub_linked_service_py3.py
35.71% <35.71%> (ø)

... and 14 more

Continue to review full report at Codecov.

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

PR was closed


UnicodeDecodeError: 'utf-8' codec can't decode byte 0xaf in position 10: invalid start byte

Crash Report
This crash report was reported through the automatic crash reporting system 🤖
Traceback
  File "site-packages\electron_cash-3.1.6-py3.5.egg\electroncash_gui\qt\qrtextedit.py", line 50, in file_input
  File "c:\python3.5.4\lib\codecs.py", line 321, in decode

UnicodeDecodeError: 'utf-8' codec can't decode byte 0xaf in position 10: invalid start byte

Reporter
This issue was reported by 8 user(s):

Electrum Version
Operating System
Wallet Type
Locale

3.1.6
Windows-10-10.0.16299
standard
en_US

3.1.5
Linux-4.14.0-kali3-amd64-x86_64-with-Kali-kali-rolling-kali-rolling
standard
en_US

3.1.6
Windows-10-10.0.16299
standard
en_US

3.2
Windows-10-10.0.14393
imported_addr
ru_RU

3.2
Windows-10-10.0.14393
imported_addr
ru_RU

3.2
Windows-10-10.0.14393
imported_addr
ru_RU

3.2
Windows-10-10.0.14393
imported_addr
ru_RU

3.2
Windows-10-10.0.16299
imported_addr
en_US

Additional Information
The reporting user(s) did not provide additional information.

This is the user selecting a file which cannot be decoded.  It can be reproduced by opening a binary file.  This fix shows a similar dialog shown in the same situation in other places through the code-base.


Implemented composite value merging using deep-extend module

Hi Nicholas,
Here is the PR as per #2.
Please make any changes you feel appropriate to the PR.
Cheers,
Damo.

Thanks for this enhancement. I am going to do some testing, then make a release.


test: migrate build_script_env to snapbox

What does this PR try to resolve?
part of https://github.com/rust-lang/cargo/issues/14039.

Thanks!
@bors r+

:pushpin: Commit 2d10bd5a9fb30f30de91a33a5f287f2c32a9e77c has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 2d10bd5a9fb30f30de91a33a5f287f2c32a9e77c with merge c13a39472ad18693bfe2cb356a8d00c19cada39f...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing c13a39472ad18693bfe2cb356a8d00c19cada39f to master...


ML 2.17 : Random Forest Algorithms (V)

Description
📌  Issues for Week 2
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
Random Forest Algorithms.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Supervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

Dataset to be used : IRIS DATASET. (No other datasets will be entertained.)

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do video on this issue.

I would like to contribute for this issue. I request you to assign me the same.
Thanks
Name: Samridhi Kapoor
Batch No.: 3
Github Profile link: https://github.com/samridhikapoor
Type of contribution I want to make: Video (as specified)

@samridhikapoor issue assigned to you. Go ahead and complete this by this wednesday. 🚀

@abhisheks008 , Can i work on it , As the deadline is reached.

@samridhikapoor Update?

@Isha307 , please assign the issue to me .

@samridhikapoor Update?
I am done making the ppt. I will record the video and upload it by today evening for sure.

If this issue is still open I would like to contribute to this issue. @Isha307
Name: Riya Srisailam Sailesh
Batch: 4
Profile link: https://github.com/riyasai22
Contribution type: Audio

@samridhikapoor This is your last call, complete this issue and make the PR by EOD otherwise this will be unassigned from you.


Cannot install library

Trying to run go get github.com/easonlin404/ksm/ fails with an error:
go: finding github.com/easonlin404/ksm latest
go: github.com/easonlin404/ksm imports
	github.com/easonlin404/ksm/crypto/rsa imports
	github.com/89hmdys/toast/rsa imports
	github.com/Sirupsen/logrus<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/sirupsen/logrus
	        but was required as: github.com/Sirupsen/logrus

It seems the issue actually comes from https://github.com/89hmdys/toast. But even updating it there wouldn't fix this repo without an update here too.
Any idea how to work around this issue?
Thanks

oh, there is already a PR https://github.com/89hmdys/toast/pull/3/files


METRON-1770 Add Docs for Running the Profiler with Spark on YARN [Feature Branch]

I updated the README with advice for running the Batch Profiler with Spark on YARN.  I also added a link with instructions for running on a secure cluster.  This is all transparent to the Batch Profiler, which is why this is only a doc change.
Pull Request Checklist

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?
[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:cd site-book
mvn site

+1 by inspection.

Thanks.  This has been merged into feature/METRON-1699-create-batch-profiler


alphabetized and updated FEATURES.md

Coverage remained the same at 95.752% when pulling 2bc3639e1d3e241f323728e6a546931e8b70fd5d on aportner:alphabetized-features into 631830cd06bf959f4a9c1e8694bc09975d288f54 on LPGhatguy:master.


feat: do not exec diagnostics if gwmfr running

Why
Some ECC chips are being corrupted during manufacturing. There may be a race condition between gwmfr and diag. We want to wait for gwmfr to complete before running diagnostics.
How
Write to an empty file (AKA touch) when gwmfr completes successfully. Wait for this file to be present before running diagnostics.
References

Slack convo: https://pisupply.slack.com/archives/C024BNQ1Y6T/p1632912403459500
gwmfr: https://github.com/NebraLtd/hm-gwmfr/pull/18
helium-miner-software: https://github.com/NebraLtd/helium-miner-software/pull/155

@marvinmarnold the things we need to hold back is the /initFile.txt generation. That is what Hotspot Production Tool is hammering.

@vpetersson does the check need to happen in both places? I think we want to stop Hotspot-Production-Tool from continuing and make sure diagnostics doesn't access ECC.