Calculate how many shares a user can buy/sell

Coverage increased (+0.2%) to 67.146% when pulling b0427d432132d62385060959a96a8185be6a3eb6 on tomasprie/ch349/how-many-shares-a-user-can-buy-of-each-outcome into 16f4a47bac3e51efaee8014a04eba7d0c58c993a on master.

Coverage increased (+0.2%) to 67.146% when pulling b0427d432132d62385060959a96a8185be6a3eb6 on tomasprie/ch349/how-many-shares-a-user-can-buy-of-each-outcome into 16f4a47bac3e51efaee8014a04eba7d0c58c993a on master.

Coverage increased (+0.3%) to 67.216% when pulling d562738802cefc1b3b46960a90d957230f51b36f on tomasprie/ch349/how-many-shares-a-user-can-buy-of-each-outcome into 16f4a47bac3e51efaee8014a04eba7d0c58c993a on master.

@priecint Your calculation is not right.  Here is a binary market where it says 0 shares max (which is clearly wrong):

And, here is a scalar market where it appears to just be pulling the amount in the best offer (I should be able to buy more than 3.17 given the eth I have available):


Fix performance regression related to #528

As explained here the previous fix for #528 was reverted, possible because of a bad merge.
This PR resolves the performance warning in same way as before: https://github.com/dakrone/clj-http/pull/532

Thank you!
That was my bad for re-introducing the performance regression. 🤦
I'll look into adding lein check as part of the CI process to detect these sooner.


Add support for generating JSON fields

Easiest implementation to support generating JSON fields I could think of.
Developers still need to change schema but at least generation does not fail
and generated file compiles.

@anivanovic I made a similar contribution, I addressed more cases with JSON field, would you like to talk in Discord in order to push it together?

Sure! Can you give me Discord link? I am also in Slack #ent channel. So we can open a thread there also.

Thanks for the support @zeevmoney, and thanks @anivanovic for the contribution.
Here's a link to the Discord server. Feel free to create a thread there - https://discord.gg/qZmPgTE6RX
I also suggest you use go/parser.ParseExpr to construct and validate the expression for configuring the JSON type. Please, feel free to mention me in Discord 😃

Thanks, @anivanovic for your contribution! ❤


Cross-tenant mailbox migration

i have read through this document and tried to follow it and boy do you guys make it difficult.
so many missing pieces. has anyone that has actually written an article tried to follow these directions on a fresh Tenant?
will the future of this be to run it all via PowerShell?
will there be a way to monitor the migration via the EAC?

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

ID: c61c960c-f5c3-76f7-ac18-dfeaf30230e4
Version Independent ID: 3dc607b2-869e-976f-80da-d73cfb1210e5
Content: Cross-tenant mailbox migration - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/cross-tenant-mailbox-migration.md
Product: microsoft-365-enterprise
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@RichAcevedo1 Thank you for your feedback. This feature is still under preview and we are not exactly sure how the future of this would be (PowerShell or eac). I'd recommend you post questions directly on our feedback site( https://feedbackportal.microsoft.com) to get the latest updates and roadmaps about this feature.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @RichAcevedo1 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Notifications Obscures Input

When using a shortcut to activate Ask CodeGPT, it displays a notification that obscures the input, and you have to close the notification with your mouse (or wait a really long time for it to disappear on its own). This means I can't see what I'm typing unless I use my mouse, which defeats the purpose of defining a keyboard shortcut for it.
The ask:
Could we maybe have an option to disable info-notifications?

Hi everyone!
I hope you're all doing well. Please take a moment to update to the latest version here: Visual Studio Marketplace.
Thank you!


fix: filtering for ctrader and dxtrade

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 52.68%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
1194

Relevant Lines:
1912

💛  - Coveralls


No shell commands exist to assign item types

The API exists, but no shell commands have been added yet.

Done.


fix(dataplane): ServiceVersion should be pascal case in manual codes

fix case problem in ServiceVersion in manual codes
remove suppression of AZC0016

part of https://github.com/Azure/autorest.csharp/issues/1524
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

This PR is to fix the ServiceVersion case issue in manually written codes. Those are all beta SDKs.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.MixedReality.ObjectAnchors.Conversion

approved on behalf of purview


Bump pydeck to 0.1.dev3

For #

Background

Change List

Coverage remained the same at 83.022% when pulling ae9da83c59fd2c465d0014a660394522c820e3e5 on ajd/pydeck-release-0.1-dev3 into 46ab2abc219269bd95e97c4bbb1ff8200977aa11 on master.


Advance slide from outside

I need to be able to advance the slide via code from outside the CarouselProvider. It looks like currentSlide is not for this and is also deprecated.
Describe the feature you'd like:
I have to advance the slide after audio finishes for the slide. I thought it would be a matter of just firing an event and advancing the slide but changing the index of the slide, but I don't see where to change it.
It seems to work ok if I do set the currentSlide, but I'm afraid this option is going away. Could it be retained for a purpose such as this?

:+1:
pure-react-carousel now provides only a uncontrolled carousel. I want a controlled carousel too.


Different height per option

I would like to be able to set the height of the options individually.
Reason:  I want to support the concept of option groups and the group header would be a different height to the other options in the list, e.g.

I have the option group working using a custom renderer with different styles depending on whether it is a group or not, but currently the height must be specified at the  control level so cannot be overridden per option.
The underlying controls you use support per-row height - e.g. passing in a function to calculate the height) so it would be great if this was supported by your control too.

Hi @freeranger. Sorry for the slow response. I'm traveling in Japan this week and my schedule has been a bit disrupted.
I agree that supporting a callback function for optionHeight would be nice.

Just remembered why I only supported a numeric property for this. It's because filtered selects may shrink smaller than the default/max height, in which case the select menu should resize to fit its content. It's harder to calculate this if the option heights are dynamic.

Still, I think it's worth adding a tiny bit of complexity to support this use case. Give me a few minutes and I'll put up a new version. :)

Hit an unexpected snag with regard to filtering. VirtualizedSelect only has access to an iterable set of options but not the visible (filtered) set. So its indices don't necessarily match up to what's being rendered.

Hi, if the function received the option data itself rather than the index of the option then it wouldn't matter if the indices are out because you are calculating the height based on the data rather than its position.
An alternative to supporting dynamic option heights would be to support the concept of option groups and allow the height of a group to be different to the height of actual options - thus you know in advance the height of options and of groups so the overall height is easier to calculate.

Yeah, there's a tricky case here though that's a combination of the things I mentioned above. If the list has been filtered, VirtualizedSelect doesn't know what the filtered options are and so it can't match an index up to an option. This means it can't determine if the select menu should be shrunk b'c the number of options are smaller/shorter than the size of the select.

D'oh. Sorry. Re-reading through Select code and I believe I do actually have those options as a parameter to menuRenderer. :)

This feature is now available in version 1.1.0. Thanks for the suggestion!
A demo can be seen here with source code here.

@bvaughn awesome - that is exactly the functionality we need - many thanks for implementing it!

No problem! Very happy to hear it will help you out. :)

don't say that - I might be back with a load more ;^)

Haha, well contributions are also welcome ;)


Warnings in log that custom integration will stop working with HA 2025.1 / 2025.6

Version of the custom_component
Latest version 0.0.11
Describe the bug
Integration using deprecated functions that will be removed with HA 2025.1 and 2025.6.
Debug log

Will be removed with HA 2025.6:
2024-07-06 09:59:05.349 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'thermiagenesis' calls async_forward_entry_setup for integration, thermiagenesis with title: thermia-atlas.lan:502 and entry_id: 0e7e79a6deee69f0a85ab6081b2ce185, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/thermiagenesis/__init__.py, line 53: hass.async_create_task(, please create a bug report at https://github.com/CJNE/thermiagenesis/issues

Will be removed with HA 2025.1:
2024-07-06 09:59:05.386 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration thermiagenesis with title: thermia-atlas.lan:502 and entry_id: 0e7e79a6deee69f0a85ab6081b2ce185, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.
Stack (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
    sys.exit(main())
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2122, in async_forward_entry_setup
    _report_non_awaited_platform_forwards(entry, "async_forward_entry_setup")
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1175, in _report_non_awaited_platform_forwards
    report(
  File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 151, in report
    _LOGGER.warning(msg, stack_info=True)

+1

+1

+1


Publish from Visual Studio 2022 v4 Net7 to linux fail, window fine

Hi, reporting a new project, V4 Isolated Net 7.
From Visual Studio 2022 17.5.4, publishing to a linux azure function  fails with
6>The attempt to publish the ZIP file through https://xx.scm.azurewebsites.net/api/zipdeploy failed with HTTP status code Failed.
Publishing on windows azure function works fine. I know it was like this on V2, but is it still not supported?

@imtrobin are you still experiencing this issue?
In order to better assist, it would be helpful to have an application name so we can take a closer look at deployment logs. Is that something you'd be able to share?
Do you have additional information logged about this issue in VS?
Can you also share whether this was on Linux consumption or dedicated/Elastic Premium?

Linux comsumption. I updated the app with latest v4 sdk, and I'm able to publish now, so I think we can consider this closed.

I spoke too soon. I deleted the one that published fine , recreated  a new function linux app, consumption. And the same publishing error. It is same code as the one that published fine.
App name: casstageadmin
Visual studio log
5/4/2023 2:51:09 PM
System.AggregateException: One or more errors occurred. ---> Microsoft.WebTools.Shared.Exceptions.WebToolsException: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.
--- End of inner exception stack trace ---
---> (Inner Exception #0) Microsoft.WebTools.Shared.Exceptions.WebToolsException: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details. <---
Microsoft.WebTools.Shared.Exceptions.WebToolsException: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.
===================

@imtrobin What version of the worker packages were you using? Would you mind upgrading to the latest packages and see you still experience the issue?

https://www.nuget.org/packages/Microsoft.Azure.Functions.Worker/
https://www.nuget.org/packages/Microsoft.Azure.Functions.Worker.Sdk

I have same problem here on Zip Deploy for Azure Function
System.AggregateException: One or more errors occurred. ---> Microsoft.WebTools.Shared.Exceptions.WebToolsException: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.
--- End of inner exception stack trace ---
---> (Inner Exception #0) Microsoft.WebTools.Shared.Exceptions.WebToolsException: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.
I am trying to Publish Azure Function v4 .NET 6 on Linux with Microsoft.NET.Sdk.Functions 4.2.0 but the error ocurred
Publishing \obj\Release\net6.0\PubTmp\fxName -<PHONE_NUMBER>1127488.zip to https://subdomain.scm.azurewebsites.net/api/zipdeploy...
4>Zip Deployment failed.

I resolved my problem with 2 actions:

I delete some mount network of storage account (file share) > it was producing slow access to my SSD and it was getting timeout.
I dont know why, but it not the same to use their publishing profile from Deployment Center (Linux) or Overview (Windows), use the Deployment Center for Linux


Finishing collapsed nodes support (e.g. adding and sorting args)

This task is automatically imported from the old Task Issue Board and it was originally created by Wojciech Daniło.
Original issue is here.

No description provided.

Closed as duplicate now.


how to run a demo for a single picture？

hello, i want to use the model to infer my own picture,which file in the project should i use ?

Since the code is heavily biased toward the KITTI dataset, especially for the codes of camera calibration, I am now working on something for the inference of single images or videos.
Also, the camera calibration w.r.t. the image is needed for inference.

@zhangyp15  Does it support some other dataset such as dataset from Waymo?

@jinfagang  Currently we have only done experiments with KITTI.

Hi, Is there an update regarding the inference script ?

Since the code is heavily biased toward the KITTI dataset, especially for the codes of camera calibration, I am now working on something for the inference of single images or videos.
Also, the camera calibration w.r.t. the image is needed for inference.

Is there an update regarding the inference script ?

Hi, Is there an update regarding the inference script ?

Do you have the inference demo?


Added 3 new FAQ questions

Added three new frequently asked questions (FAQs) to the ambulance monitoring system's FAQ section. These questions focus on ambulance cancellation, safety instructions, and handling technical issues related to the tracking system.

@4F24L

issue no??

@mansiruhil13 the issue no. is #661

there are conflicts, resolve

and as landing page is added. can you change into your code. so that this pr doesn't make any blunder

@mansiruhil13 I will make the necessary changes.


Remove deleted submodule folders (#369)

In commit 5a8493a3a4e23e4315308fe75c97269df1d9a256 several submodules were not fully removed form the cache. This commit deletes the references completely, allowing git submodule update --init to succeed.

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Sections of README are out-dated

The following sections of the README are out-dated (apply to v1 and not v2) and should either be removed or corrected:

How does Money$afe work? https://github.com/ericelliott/moneysafe/blob/97726a87a7020402b488452c3d443b13dfda263b/README.md#how-does-moneyafe-work
It mentions "storing and acting on the amounts in cents", which is no longer accurate. I suggest we either remove this section entirely or update it to describe the use of bignumber.js instead.

$(dollars) => Money https://github.com/ericelliott/moneysafe/blob/97726a87a7020402b488452c3d443b13dfda263b/README.md#dollars--money
The Example uses .cents, which no longer works.
The next section says "The resulting value will be in cents", which is no longer correct.

in$ Utility https://github.com/ericelliott/moneysafe/blob/97726a87a7020402b488452c3d443b13dfda263b/README.md#in-utility
This utility no longer exists. This section should be removed.
Perhaps, in its place, we should add some advice for how to migrate from v1 to v2.

Thank you! I've tried to update this several times but apparently, I missed some things. Would you like to take a crack at a PR?

I'll take a crack at it, but how do you recommend rounding a Money type to its nearest supported significant digit (after arithmetic operations are complete) now that the in$ utility is gone?
Would you recommend one of the following or something else?

Using built-in .toFixed() with Number coercion:
+$(-45).div(99).toFixed() // -0.45

Using your own version of round_to_precision from MDN:
function round_to_precision(x, precision) {
  var y = +x + (precision === undefined ? 0.5 : precision/2);
  return y - (y % (precision === undefined ? 1 : +precision));
}

round_to_precision($(-45).div(99).valueOf(), 0.01) // -0.44

Using your own version of this StackOverflow answer:
function roundTo (num, precision) {
  return +(Math.round(num + ('e+' + precision))  + ('e-' + precision));
}

roundTo($(-45).div(99).valueOf(), 2) // -0.45

What do you think about building in some convenience for this into the Money type itself, so that we can leverage the desired number of decimals for a particular currency?

I think option 1 makes the most sense, because we can't assume that it's safe to simply try to convert any monesafe object to a string. Option 1 forces the user to consider the impact of converting to a number.

Ok, that's what I suggested in docs in #35


Change rekey stale thread notifications

This changes the stale thread notifications from a finished rekey to be based off of kbfs rekey finished notifications as opposed to changes in the problem sets.
The problem set solution had an issue that if your current device (including a paper key you typed in) solved the problem set, it would take 24 hours to figure out that the rekey finished.
This version uses existing rekey notifications from kbfs instead.

cc @chrisnojima


Copy DropDown does not displaying a data

When i try to display select2 dropdown from one place to another place but after copy select2 into other place its not working. Can someone tell me how i can fix it.
Copy dropdown from source to destination.
var h = $("#source").html();
$("#target").html(h);
I am also updating a DOM after copy but its not working.
$('#').find('.sel-inline').select2();
![copy_dropdown](https://cloud.githubusercontent.com/assets/20266392/22198055/9dbfebbc-e176-11e6-8dc0-e7926bacc20f.png
Thanks

Hi another that may left is the rigger option I've found this on SO maybe can help
http://stackoverflow.com/a/40822502 <-- with current version
http://stackoverflow.com/a/18893057 <-- whith version 3.4.2
Hope it helps, Good luck!

It is unclear what you are trying to do here.  Are you trying to use the currently selected value in one Select2 to initialize another Select2 control?  If so, see https://select2.org/programmatic-control/add-select-clear-items
Or, are you trying to copy the entire control over?
Unfortunately, we cannot provide general support for your JS code - this is better asked in Stack Overflow.


Add example for filtering key-value pairs

Add example for filtering JSONB values through key-value pairs
Solves #7294
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f8df51f217763b30cb6ee8731cee0c95faf3a2d6/
Edited pages:

v19.2/jsonb.md

I think this looks good, but someone with more JSON expertise than me should verify that this is correct.

Got an internal LGTM 🎉
@vipulgupta2048 Can you update the 20.1 and 20.2 docs too?

Awesome @SpicyThrow @Amruta-Ranade Just a side thought, I like looking into the documentation pipelines and am always curious about them (part of the job) and I wanted to suggest if you folks have thought of going more into docs as code style for your documentation. I haven't looked at Jekyll for quite some time, but I feel the documentation's versioning and generation could be automated. Removing a lot of overhead on writers and contributors because it wasn't clear to me which version to edit and but issue had a link mentionining 19.2. Nonetheless, love to have a discussion on issue or something.
Will update the PR for 20.1 and 20.2

We have had discussions about it but never enough bandwidth to dive into the issue. We now have a product operations engineer onboard, so we might revisit this issue some time in the near future :)

That's great to hear, thank you for the prompt response @Amruta-Ranade More power to you folks! I will be especially interested to see that happen. I have updated the PR.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/55a33b9cb3849e5ad2f09bfdf08f23e130400d83/
Edited pages:

v19.2/jsonb.md
v20.1/jsonb.md
v20.2/jsonb.md


[RadioButtonGroup] Does not respond to defaultSelected changes

I don't think this is specific to the RadioButtonGroup, but when using dynamic routes with react-router, I noticed that if I try to set the value of a RadioButtonGroup in the render method via the "defaultSelected" property, it doesn't update the component underneath.
I am able to change the component by calling setSelectedValue() on the RadioButtonGroup ref in my componentWillReceiveProps() method.
I'm guessing this is expected behavior, but I don't feel I fully understand why. So the property changes in the render method, but it was already drawn, and so it doesn't result in a dom update.
Are there more properties like this? Should it be documented or is there a clear logic behind it? I'm just trying to understand...

+1. The workaround also worked for me, although I can delay calling setSelectedValue() until the render() method of the calling component.

@0x80 : Were you able to get a solution to this?
I am closing this issue for now as its really old and there has been no response again from your side so I an guessing that you were able to fix it.
If you still feel it issue exists or if its still not clear to you feel free to leave a comment below and we will try and help you.

Seems that the defaultSelected option is not meant to be dynamically changed, event during component re-rendering.
I think you were looking for the valueSelected option


[ISSUE #4323] Batch example add the default NamesrvAddr

fix : #4323
Make sure set the target branch to develop
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4328 (6b9deb9) into develop (a28dc65) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #4328      +/-   ##
=============================================
- Coverage      48.15%   48.11%   -0.05%
  Complexity      5076     5076
=============================================
  Files            642      642
  Lines          42767    42767
  Branches        5598     5598
=============================================
- Hits           20595    20576      -19
- Misses         19659    19673      +14
- Partials        2513     2518       +5

Impacted Files
Coverage Δ

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
66.07% <0.00%> (-7.15%)
:arrow_down:

...apache/rocketmq/remoting/netty/ResponseFuture.java
85.00% <0.00%> (-5.00%)
:arrow_down:

...rocketmq/remoting/netty/NettyRemotingAbstract.java
47.08% <0.00%> (-4.02%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (-2.57%)
:arrow_down:

...va/org/apache/rocketmq/logging/inner/Appender.java
34.83% <0.00%> (-2.25%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

.../rocketmq/broker/filter/ConsumerFilterManager.java
72.19% <0.00%> (-1.80%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
36.41% <0.00%> (-1.13%)
:arrow_down:

...ketmq/common/protocol/body/RegisterBrokerBody.java
83.69% <0.00%> (-1.09%)
:arrow_down:

...che/rocketmq/namesrv/kvconfig/KVConfigManager.java
59.18% <0.00%> (-1.03%)
:arrow_down:

... and 13 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 a28dc65...6b9deb9. Read the comment docs.

Coverage decreased (-0.05%) to 52.183% when pulling 6b9deb9d4137f68ea4acba51f87e6e1882290d76 on li-xiao-shuang:feature_example_3 into 7982c9c09a9da754f42d0cf46dd5ddaa9d2c7be1 on apache:develop.

Coverage decreased (-0.05%) to 52.183% when pulling 6b9deb9d4137f68ea4acba51f87e6e1882290d76 on li-xiao-shuang:feature_example_3 into 7982c9c09a9da754f42d0cf46dd5ddaa9d2c7be1 on apache:develop.


A lot of errors after update to 22-1

Please provide us with the following information:

OS?
Windows 10.
Versions.
angular-cli: 1.0.0-beta.22-1
node: 7.2.1
os: win32 x64
Mention any other details that might be useful.
I have a very big application, all worked fine for 20-4 version. Today, I tried to update  to 22-1 version.
I know how to update. I did it before, all worked fine. But, now, on build I have a huge amount of errors(218 errors).
But, when I create app with help of ng new command all works fine.
Some errors are repeatable, so I post unique only:
Some errors contains Subscription, some errors contain Observable, 'Subject', 'Subscription'.
Uncaught Error: Module build failed: Error: C:/Projects/qwe/src/app/shared/layout/layout.service.ts (50,3): Return type of public method from exported class has or is using name 'Subscription' from external module "C:/Projects/qwe/node_modules/rxjs/Subscription" but cannot be named.)
    at _checkDiagnostics (C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:115:15)
    at C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:140:17
    at _checkDiagnostics (C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:115:15)
    at C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:140:17
    at Object.<anonymous> (http://localhost:4200/main.bundle.js:543:7)
    at __webpack_require__ (http://localhost:4200/inline.bundle.js:53:30)
    at Object.<anonymous> (http://localhost:4200/main.bundle.js:29:81)
    at __webpack_require__ (http://localhost:4200/inline.bundle.js:53:30)
    at Object.<anonymous> (http://localhost:4200/main.bundle.js:12908:84)
    at __webpack_require__ (http://localhost:4200/inline.bundle.js:53:30)
    at Object.<anonymous> (http://localhost:4200/main.bundle.js:12977:70)
    at __webpack_require__ (http://localhost:4200/inline.bundle.js:53:30)
    at Object.<anonymous> (http://localhost:4200/main.bundle.js:8002:64)
    at __webpack_require__ (http://localhost:4200/inline.bundle.js:53:30)

I have errors for jsdom, location etc.
vendor.bundle.js:79373 ./~/jquery/lib/node-jquery.js
Module not found: Error: Can't resolve 'jsdom' in 'C:\Projects\qwe\node_modules\jquery\lib'
resolve 'jsdom' in 'C:\Projects\qwe\node_modules\jquery\lib'
  Parsed request is a module
  using description file: C:\Projects\qwe\node_modules\jquery\package.json (relative path: ./lib)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: C:\Projects\qwe\node_modules\jquery\package.json (relative path: ./lib)
    resolve as module
      looking for modules in C:\Projects\qwe\src
        using description file: C:\Projects\qwe\package.json (relative path: ./src)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: C:\Projects\qwe\package.json (relative path: ./src)
          using description file: C:\Projects\qwe\package.json (relative path: ./src/jsdom)
            as directory
              C:\Projects\qwe\src\jsdom doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\jsdom doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\jsdom.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\jsdom.js doesn't exist
      looking for modules in C:\Projects\qwe\node_modules
        using description file: C:\Projects\qwe\package.json (relative path: ./node_modules)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: C:\Projects\qwe\package.json (relative path: ./node_modules)
          using description file: C:\Projects\qwe\package.json (relative path: ./node_modules/jsdom)
            as directory
              C:\Projects\qwe\node_modules\jsdom doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\jsdom doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\jsdom.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\jsdom.js doesn't exist
[C:\Projects\qwe\src\jsdom]
[C:\Projects\qwe\src\jsdom]
[C:\Projects\qwe\src\jsdom.ts]
[C:\Projects\qwe\src\jsdom.js]
[C:\Projects\qwe\node_modules\jsdom]
[C:\Projects\qwe\node_modules\jsdom]
[C:\Projects\qwe\node_modules\jsdom.ts]
[C:\Projects\qwe\node_modules\jsdom.js]
 @ ./~/jquery/lib/node-jquery.js 5:13-29
 @ ./src/polyfills.ts
 @ ./src/main.ts
 @ multi main

This error for all files where I have routing:
./src/app/app.routing.ts
Module build failed: Error: C:/Projects/qwe/src/app/app.routing.ts (71,14): Exported variable 'routing' has or is using name 'ModuleWithProviders' from external module "C:/Projects/qwe/node_modules/@angular/core/src/metadata/ng_module" but cannot be named.)
    at _checkDiagnostics (C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:115:15)
    at C:\Projects\qwe\node_modules\@ngtools\webpack\src\loader.js:140:17
 @ ./src/app/app.module.ts 13:0-40
 @ ./src/app/index.ts
 @ ./src/main.ts
 @ multi main

And, of course, a lot of errors about module not found:
./src/app/shared/user-shared/login-info/login-info.component.ts
Module not found: Error: Can't resolve 'login-info.component.html' in 'C:\Projects\qwe\src\app\shared\user-shared\login-info'
resolve 'login-info.component.html' in 'C:\Projects\qwe\src\app\shared\user-shared\login-info'
  Parsed request is a module
  using description file: C:\Projects\qwe\package.json (relative path: ./src/app/shared/user-shared/login-info)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: C:\Projects\qwe\package.json (relative path: ./src/app/shared/user-shared/login-info)
    resolve as module
      looking for modules in C:\Projects\qwe\src
        using description file: C:\Projects\qwe\package.json (relative path: ./src)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: C:\Projects\qwe\package.json (relative path: ./src)
          using description file: C:\Projects\qwe\package.json (relative path: ./src/login-info.component.html)
            as directory
              C:\Projects\qwe\src\login-info.component.html doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\login-info.component.html doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\login-info.component.html.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\src\login-info.component.html.js doesn't exist
      looking for modules in C:\Projects\qwe\node_modules
        using description file: C:\Projects\qwe\package.json (relative path: ./node_modules)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: C:\Projects\qwe\package.json (relative path: ./node_modules)
          using description file: C:\Projects\qwe\package.json (relative path: ./node_modules/login-info.component.html)
            as directory
              C:\Projects\qwe\node_modules\login-info.component.html doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\login-info.component.html doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\login-info.component.html.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              C:\Projects\qwe\node_modules\login-info.component.html.js doesn't exist
[C:\Projects\qwe\src\login-info.component.html]
[C:\Projects\qwe\src\login-info.component.html]
[C:\Projects\qwe\src\login-info.component.html.ts]
[C:\Projects\qwe\src\login-info.component.html.js]
[C:\Projects\qwe\node_modules\login-info.component.html]
[C:\Projects\qwe\node_modules\login-info.component.html]
[C:\Projects\qwe\node_modules\login-info.component.html.ts]
[C:\Projects\qwe\node_modules\login-info.component.html.js]
 @ ./src/app/shared/user-shared/login-info/login-info.component.ts 30:22-58
 @ ./src/app/shared/user-shared/user-shared.module.ts
 @ ./src/app/app.module.ts
 @ ./src/app/index.ts
 @ ./src/main.ts
 @ multi main

Does this help? http://stackoverflow.com/questions/40035976/how-to-configure-and-use-jquery-with-webpack

Yes, the same errors as in the question. But, I don't think it's possible to use  ProvidePlugin in cli.
Anyway, I solved it, just deleted node_modules and installed again.

Revert your angular-cli to<EMAIL_ADDRESS>following the instructions at https://github.com/angular/angular-cli#updating-angular-cli


Add missing getter for sharp window corners.

Noticed while issuing #99856, seems to just been forgotten from #97005.

Thanks!


How to display the picture after the attack？

Hello, after running attack.py, the pkl file is saved. I would like to ask how to display the image after the attack? Attack is to perturb a single pixel. Has the value of other pixels in the image changed? Thank you and look forward to your reply.

Have you solved this problem? I also want to know how to display the attacked pictures. Thank you!

Sorry, I've been busy. The notebook/Colab should show you how to do this. More specifically, helper.py has a function called plot_image that you can use. The attack function in the Colab calls this helper function to display the attacked image.
Does this answer your question, or are you looking for something more specific?

Hello, I want to ask a question about the dataset.
Here, "Attack on 1,3,5 pixel perturbations (100 samples)" refers to the result of an attack on 100 images randomly selected from the original cifar10 dataset?
Or is it the result of generating 100 adversarial images for each of the 10,000 images in the cifar10 test dataset? Thank you！

can I download the pictures too?


Content-Type header missing with codable routing

When invoking a route that uses codable routing, JSON data is returned in the message body, but the Content-Type header is missing.
Example project:  https://github.com/djones6/CodableTest
Here's the route and the response:
    router.get("/codable") {
        (completion: ([Hello]?, RequestError?) -> Void) in
        let hello = Hello(message: "Hello World")
        completion([hello], nil)
    }

$ curl -i http://localhost:8080/codable
HTTP/1.1 200 OK
Date: Thu, 23 Nov 2017 15:53:40 GMT
Content-Length: 27
Connection: Keep-Alive
Keep-Alive: timeout=60

[{"message":"Hello World"}]

The equivalent using the existing API does include the header:
    router.get("/json") {
        request, response, _ in
        let hello = Hello(message: "Hello World")
        try response.send(json: [hello]).end()
    }

$ curl -i http://localhost:8080/json
HTTP/1.1 200 OK
Date: Thu, 23 Nov 2017 15:53:48 GMT
Content-Type: application/json
Content-Length: 27
Connection: Keep-Alive
Keep-Alive: timeout=60

[{"message":"Hello World"}]

This has just been reported by @svanimpe as well.
In each of the "safely" functions we do the encoding directly, e.g.  for postSafely():
https://github.com/IBM-Swift/Kitura/blob/master/Sources/Kitura/CodableRouter.swift#L235
    let encoded = try JSONEncoder().encode(result)
    response.status(.created)
    response.send(data: encoded)

we have two choices:

Add a call to response.headers.setType("json")
Call response.send(result) instead and delegate the encoding to the send call

Approach #2 makes the most sense as there's no duplication of code/function. Currently there's no way to tell the response which encoder to use - that's not a problem at the moment, but we'll need to provide a setter in the future once we support other encoding types.

@ddunn2 I think this issue is solved by https://github.com/IBM-Swift/Kitura/pull/1190

@tunniclm I'll have a read through of the item you've linked, thanks! :)

@ddunn2 @tunniclm Yes, @sandmman noticed this while implementing the two new proposals and has added code to address this issue.

Reassigning to @sandmman, to be closed once this aspect of #1190 is complete.

Fixed in #1190


link broken

The link at the top that says Non-host console apps are discussed later in this document. is broken, and there doesn't appear to be documentation on how console apps should use Microsoft's logging library (just ASP.Net Core apps?). This text is found in the Logging providers section.

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

ID: 7184a308-a254-9350-a5cd-5bced1f369ae
Version Independent ID: 726e3bf1-f367-d733-8933-bccc04da0e16
Content: Logging in .NET Core and ASP.NET Core
Content Source: aspnetcore/fundamentals/logging/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @cdwiegand ... Thanks for reporting. It looks like this is a 5.0 section link and that section isn't supposed to appear in the 6.0 doc. Therefore, the 6.0 version link should be dropped. If you want to read the 5.0 version section, it's here ...
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-5.0#non-host-console-app


eqnarray isn't handled properly

There is an issue with tag 01LR (also on the current website actually):

it isn't numbered (also not on the section page 01LQ)
it is described as eqnarray on tag 01LE, which is not really pretty

This is probably plasTeX-related too.

It was a plasTeX issue, and it has now been fixed (but I haven't done a new plasTeX run, so maybe I borked it).


404 on download link (missing/moved asset?)

Pressing the download link on the sinonjs.org results in a 404. After looking into the issue it is simply a case of a relative link that should have been absolute. Fix: href="releases/sinon-1.14.1.js" => href="/releases/sinon-1.14.1.js"

Just saw that the docs have their own site. Pushing a PR there.


Theme Picker didn't follow System settings.

My smartphone is set to dark theme.
I started the app for the first time, and all is white.
Manually switch to dark, works.
:iphone: Tech info

Device: Pixel 7 pro
OS: Android 14
App Version: 11.0

Thanks for the report, @Dtrieb. 👍
Do you have an idea about this, @Rashxz?

Ill take a look into it.


WASM Support

Hi.
Qt now supports compiling to WASM. Is ritual support this?
Related:

https://github.com/rust-qt/examples/issues/19

There are no plans to support Qt on WASM, as of now.

Thank you!


Create a document to describe performance tests

@menghanl

Assign to @apolcyn as he's already had a PR for this.

closed via https://github.com/grpc/grpc/pull/10311


even more informative progress reporting for long-running testing

Is your feature request related to a problem? Please describe.
Some more information could help with long-running mutation testing such as on Prettier (see prettier/prettier#6681, which seems to take multiple days to finish on a 40-CPU virtual server).
I may contribute a PR with this kind of an improvement.
Describe the solution you'd like
On both progress reporters:

show elapsed time
show percentage down to multiple decimal places, for example: 0.12% or 12.34%

additional item to show on append-only progress reporter:

show time stamp for each mutation testing progress line, in log entry format

Describe alternatives you've considered

Manual calculation of elapsed time, etc.

Additional context
Here is some sample output of testing in prettier/prettier#6681:
23:10:21 (24247) INFO ConfigReader Using stryker.conf.js in the current working directory.
23:10:21 (24247) INFO BroadcastReporter Detected that current console does not support the "progress" reporter, downgrading to "progress-append-only" reporter
23:10:21 (24247) INFO InputFileResolver Found 40 of 5929 file(s) to be mutated.
23:10:21 (24247) INFO InitialTestExecutor Starting initial test run. This may take a while.
23:10:34 (24247) INFO InitialTestExecutor Initial test run succeeded.
 Could you add an example of how you would like the output to look? In my opinion, once we start talking in hours, we can ommit the seconds in the ETC.

Could you add an example of how you would like the output to look?

I am working on a proposal right now, which will show an example.

once we start talking in hours, we can ommit the seconds in the ETC

should be easy:
--- a/packages/core/src/reporters/ProgressKeeper.ts
+++ b/packages/core/src/reporters/ProgressKeeper.ts
@@ -60,7 +60,7 @@ abstract class ProgressKeeper implements Reporter {
     let output;

     if (hours > 0) {
-      output = `${hours}h, ${minutes}m, ${seconds}s`;
+      output = `${hours}h, ${minutes}m`;
     } else if (minutes > 0) {
       output = `${minutes}m, ${seconds}s`;
     } else {

I think this was all done in #1856, closing now. Thanks!


OpenZWave: Roll-back alpine, adjust open-zwave commit

A number of users have reported issues with ozw-admin crashing after the 0.5.0 addon update, which sometimes crashes the entire container and requires it to be restarted.
I talked with Fishwaldo and we believe the issue is related to the alpine upgrade since it upgraded QT 5.12 to 5.14, and Fishwaldo is targeting QT 5.12 for the current ozw-admin and qt-openzwave builds.
I rolled back the alpine update and qt versions and have not experienced any crashes or issues on my machine over a few hours of testing today.
This also rolls open-zwave back a few commits, as there were some issues with the last couple commits that were included in the original update related to transitions and value refreshes. The following commits are reversed: https://github.com/OpenZWave/open-zwave/compare/6c2ca6131cbba4200460256293d719053107a9e8...5d3978d5cac8d1bf556bd20c56ec2309b7cd7091

Hold on merging... collecting some additional information regarding qt versions...

OK, was able to confirm problem related to qt 5.12 vs 5.14

Please merge. I can't start ozw since 0.5.0.

Is there an open-zwave issue for the transitions and refreshing problem?

@cgarwood what are these duration changes that you mention? Since running this update, it takes like 5 minutes (which used to take 1-2 seconds) for my Fibaro Dimmers 2’s to turn on from 0% to 100%, it goes at like 1% every few seconds. Needless to say, turning them off takes just as long...

Please don't ask questions in merged PRs. Open an issue if there's a problem. Thanks!


[Error Report] Crunchy [403: Forbidden] - JWT Error on Non-DRM Request Stream URLs

Program version
5.0.0
Operating System
Windows
Type
GUI
Service
Crunchyroll
Command used
NIL
Show ID
ALL
Episode
ALL
Console Output
[ERROR] 403: Forbidden
[ERROR] {"error":"JWT error"}
[ERROR] Non-DRM Request Stream URLs FAILED!

Additional Information
The error appears for every stream, including when using DRM (DL not affected from DRM side), as AniDL looks for all streams/servers.
It was reported elsewhere that Crunchy's side has made changes.

This is already fixed, update to the latest version (5.0.1)

Thank you. Didnt see that.My aplologies.


Don't use react and react-router as dependencies

It causes redundancies. Also, I am receiving this because of this:
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/es6/useBeforeUnload.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/es6/useQueries.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/Actions.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/AsyncUtils.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/DOMStateStorage.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/DOMUtils.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/ExecutionEnvironment.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/PathUtils.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createBrowserHistory.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createDOMHistory.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createHashHistory.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createHistory.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createLocation.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/createMemoryHistory.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/deprecate.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/enableBeforeUnload.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/enableQueries.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/index.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/runTransitionHook.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/useBasename.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/useBeforeUnload.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/lib/useQueries.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/umd/history.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/umd/history.min.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/CHANGELOG.md shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/LICENSE.md shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/README.md shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/browser.js shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/package.json shouldn't be copied twice in one bulk copy
warning The case-insensitive file /Users/nick/dev/allvest/node_modules/react-router-sitemap/node_modules/history/node_modules/warning/warning.js shouldn't be copied twice in one bulk copy

@iDuuck React not included in result bundle, but needed for run example script. React Router used directly in bundle for get paths. But it looks like your warnings is common problem in yarn.

Though, React should be a devDependency and React-Router should be a peerDependency... am I wrong?

@iDuuck yep, you're right. I'm will changing config in the near future. Thank you!

@kuflash we're using this on a project and trying to upgrade to React 16 but the react dependency is causing us issues. Any chance that a patch could be worked on just to remove the react dependency?

@ricardo-quinones yep. I use React only for example script.

Any updates on this?


chore: LT supported browerlists and test matchers

Please add a one-paragraph summary here, suitable for a release notes description. This will help with documentation.

Overview

Adding LT equivalent of supported browser lists (json)
Supported browsers have been trimmed down to latest and oldest supported versions for desktop & mobile
Matcher rule comparison updated
CLI -b argument for desired targets enhanced/updated
Script to fetch latest browser version specs on LT post release updated
Addition of LT capabilities for requesting env enhanced

Related Issue(s)
https://new-relic.atlassian.net/browse/NR-254279
https://new-relic.atlassian.net/browse/NR-274401
Testing
Not yet ran on existing test suite. iOS facing issues and a couple matchers may need updating. That has been split out into separate issues.

LT test with retry and 25 concurrency passes on 4 desktop browsers latest & -10 versions, on separate runs. IE11 & 2 mobile browsers to be changed.


Is tsconfig.json required?

Search terms
tsconfig required
Question
I recently had some .d.ts files that I had a bunch of documentation in. This was all generated from native Rust bindings using https://napi.rs/
Basically:
use napi_derive::napi;

#[napi]
/// Compute it!
fn fibonacci(n: u32) -> u32 {
  match n {
    1 | 2 => 1,
    _ => fibonacci(n - 1) + fibonacci(n - 2),
  }
}

// GENERATED FILE
/** Compute it! */
export function fibonacci(n: number): number;

And I wanted to run typedoc --entryPoints index.d.ts and have it just work. No need to have a tsconfig.json file when I'm not even using JavaScript! 😂 It's all native codegen magic.
And... I couldn't.

Is there a way to magically say "no i don't need a tsconfig, just give me docs" ?? sort of like how i can do tsc file.ts and it will just work even with no tsconfig.json.

A hack for anyone else to get around this is this:
echo '{}' >tsconfig.json && typedoc --entryPoints index.d.ts; rm tsconfig.json

Hi @Gerrit0. Thanks for all your work on TypeDoc along with other present and past maintainers!.
Cool, glad this was asked as I'm coming from the ESM native side of things and generating d.ts files from ESM source and building docs, so kind of similar angle to the OP question. However, I don't mind building a CLI around TypeDoc for this use case using the programmatic API. A potential PR on my side would be to get the programmatic API to also accept an object for the tsconfig option that defines the tsconfig.json data.
Something like this will then be possible:
// Create a new TypeDoc application instance.
const app = new Application();

await app.bootstrapWithPlugins({
   // A few other options...

   tsconfig: { /* TSConfig data */ }
});

Is there any general known timing on when 0.25 might ship? I suppose a change like this doesn't require a major version. I'll get right on this with a PR this week though as this would be rather handy on my end.
And general heads up.. I do have really other neat TypeDoc related things I'll get in touch about / release / drop a note in a couple of weeks. So cool stuff incoming. :D

You can already set tsconfig options (app.options.setCompilerOptions)
0.25 has no planned release date yet, I've been very busy recently, so have barely even started on what I want to include with it.

It looks like this can probably be done via ts.parseCommandLine([]), if someone wants to give a PR a shot.

Yeah.. You can use ts.parseJsonConfigFileContent().  In another tool of mine I parse just compilerOptions as JSON w/ ts.convertCompilerOptionsFromJson(). I'll definitely take a look soonish. This is more for the next tool I'd like to build, so this potential PR is in the queue.


Version 2.0.1 fails to start with NameError on PeriodicStats

This morning we updated to prometheus_exporter 2.0.1 but after doing so, the service no longer starts properly.
To Reproduce

use gem prometheus_exporter in version 2.0.1  (2.0.0 works fine)
bundle exec prometheus_exporter
=> uninitialized constant PrometheusExporter::Instrumentation::PeriodicStats

Stacktrace:
bundler: failed to load command: prometheus_exporter (/home/deploy/releases/20220224115715Z/vendor/bundle/ruby/2.6.0/bin/prometheus_exporter)
/home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/instrumentation/unicorn.rb:11:in `<module:Instrumentation>': uninitialized constant PrometheusExporter::Instrumentation::PeriodicStats (NameError)
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/instrumentation/unicorn.rb:9:in `<top (required)>'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/server/runner.rb:4:in `require_relative'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/server/runner.rb:4:in `<top (required)>'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/server.rb:15:in `require_relative'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/lib/prometheus_exporter/server.rb:15:in `<top (required)>'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/bin/prometheus_exporter:9:in `require_relative'
        from /home/deploy/shared/vendor/bundle/ruby/2.6.0/gems/prometheus_exporter-2.0.1/bin/prometheus_exporter:9:in `<top (required)>'
        from /home/deploy/releases/20220224115715Z/vendor/bundle/ruby/2.6.0/bin/prometheus_exporter:23:in `load'
        from /home/deploy/releases/20220224115715Z/vendor/bundle/ruby/2.6.0/bin/prometheus_exporter:23:in `<top (required)>'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli/exec.rb:63:in `load'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli/exec.rb:63:in `kernel_load'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli/exec.rb:28:in `run'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli.rb:474:in `exec'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli.rb:30:in `dispatch'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/cli.rb:24:in `start'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/exe/bundle:49:in `block in <top (required)>'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/lib/bundler/friendly_errors.rb:128:in `with_friendly_errors'
        from /home/deploy/.gem/ruby/2.6.0/gems/bundler-2.2.21/exe/bundle:37:in `<top (required)>'
        from /usr/local/bin/bundle:23:in `load'
        from /usr/local/bin/bundle:23:in `<main>'

I am on the fence about this @spiderpug  / @CvX
All the code samples say:
require 'prometheus_exporter/instrumentation'
Nothing is recommending
require 'prometheus_exporter/instrumentation/unicorn'
https://github.com/discourse/prometheus_exporter/blob/e0d664b80c4b11cc9035e46bed293d3236d3044b/lib/prometheus_exporter/instrumentation.rb#L4-L4

Similarly this is not relative requiring Base
https://github.com/discourse/prometheus_exporter/blob/e0d664b80c4b11cc9035e46bed293d3236d3044b/lib/prometheus_exporter/metric/gauge.rb#L1-L4

@SamSaffron it comes from here :
https://github.com/discourse/prometheus_exporter/blob/main/lib/prometheus_exporter/server/runner.rb#L4
I removed this line and it works

I seen @n-rodriguez fixed this per #225  , new release is coming out shortly.


What is difference between this library and Code Contracts?

If you are just starting out on a new project is recommend using https://github.com/adamralph/liteguard instead of conditions. Especially if using in mobile application, less allocations. Code Contracts is compile time validation/static analysis; Conditions is runtime.


Permissions Error

New to command line stuff and I have a mighty need for rainbow output, but when I do
gem intall lolcat
I get this error:

"You don't have write permissions for the /Library/Ruby/Gems/2.3.0 directory."

How do I fix this, I am the root user of this comp (I think) so I should have all permissions. I have conda/anaconda installed, is this messing it up? I'd also like to set rainbow output as the default, is that possible?
Thanks for helping in 2019...

Hi! :)
You may have to do sudo gem install lolcat.
However, a better way to install on OSX is via homebrew.
(just follow the install instructions on that page, then afterwards you can brew install lolcat)
Unfortunately there is no easy way to make rainbow output the default everywhere.
The world is not ready for that, yet. But you can append | lolcat to any command
that outputs text and include that in scripts as well.

Ahhh ok, that would too powerful..... well thank you so much for the help, my scripts are gonna be so obnoxious now buahahahahahaha🦄🌈!


please add china region

up

should not be a problem, you can do it in 5 mins. Just create an folder named "zh-Hans.lproj" and create a new file named "GMImagePicker.strings" under it, then translate the items you want to show to end user. ^.^


As noted  on this page https://docs.microsoft.com/en-us/sharepoint/manage-sites-in-new-admin-center

[Enter feedback here]

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

ID: 3106844b-caae-38c5-3c88-fb17a9d67536
Version Independent ID: 3e88e60e-24a3-cd83-9bfc-8bc3037a86ce
Content: Change the external sharing setting for a site - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/change-external-sharing-site.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Add codecov as github action.

Signed-off-by: Kenny Leung<EMAIL_ADDRESS>
Summary

This adds codecov as the code coverage tool for PRs.
Similar to what is being done in rekor in https://github.com/sigstore/rekor/pull/676
Resurrecting #430 @nsmith5
Ticket Link

Fixes
Release Note

None

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@6a12159). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage        ?   45.66%
=======================================
  Files           ?       14
  Lines           ?      957
  Branches        ?        0
=======================================
  Hits            ?      437
  Misses          ?      460
  Partials        ?       60

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 6a12159...12d65a0. Read the comment docs.

@k4leung4 When I click on the links in the codecov comment, I only see errors about missing pages, such as https://app.codecov.io/gh/sigstore/fulcio/compare/449?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr comments&utm_term=None


Updates to installation doc

Pillow 3.0.0 pre-requisites: zlib and jpeg libs required now, else pip install -r requirements.txt will fail
https://pillow.readthedocs.org/en/3.0.0/installation.html#os-x-installation
https://pillow.readthedocs.org/en/3.0.0/installation.html#linux-installation

Also, on OS X, requires (latest?) Xcode command line tools, installed via xcode-select --install

List required environment vars (e.g., AASHE_DRUPAL_URI)

Note requirement that elasticsearch be installed

I agree that documenting the required environment variables is a good idea. I don't think we need to indicate the requirements for Pillow, OSX and elasticsearch in the main readme. I could see a server-specific readme that indicates what packages should be installed on the server and elasticsearch should be available.

@jamstooks and @rerb - I'm making this a 1c milestone but feel free to change as needed - it's all greek to me!


fix: include snappy and lz4 jars for deployment

The Cassandra interpreter supports both SNAPPY and LZ4 protocol
compression options but the required libraries are not included in the
distribution. By including both jars in the build, deployment is
simplified allow for simple configuration changes to utilize the
either the SNAPPY and LZ4 protocol compression choices.
Fixes https://issues.apache.org/jira/browse/ZEPPELIN-208

Looks good to me

LGTM


Log error detail in SDS request

Please provide a description for what this PR is for.
Log error detail in a SDS request if that field is not nil, and update metric accordingly.
The error detail field shows secret update failure at Envoy side, which would be helpful for debugging if citadel agent sends a bad secret that causes the failure.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
https://github.com/istio/istio/issues/13439
https://github.com/istio/istio/issues/15850

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ
To complete the pull request process, please assign myidpt
You can assign the PR to them by writing /assign @myidpt in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Test name
Commit
Details
Rerun command

istio-lint-master
9824240ab764b340d46183dbc36d48f1336271d4
link
/test istio-lint-master

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

Do we want to get #15545 in first?

Do we want to get #15545 in first?

Sure, I will hold the PR first.

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

Test name
Commit
Details
Rerun command

istio-lint-master
9824240ab764b340d46183dbc36d48f1336271d4
link
/test istio-lint-master

integ-new-install-k8s-presubmit-tests-master
9824240ab764b340d46183dbc36d48f1336271d4
link
/test integ-new-install-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

istio-lint-master
9824240ab764b340d46183dbc36d48f1336271d4
link
/test istio-lint-master

integ-new-install-k8s-presubmit-tests-master
9824240ab764b340d46183dbc36d48f1336271d4
link
/test integ-new-install-k8s-presubmit-tests-master

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-codecov-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-codecov-master

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-codecov-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-codecov-master

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-codecov-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-codecov-master

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

integ-framework-k8s-presubmit-tests-master
478a8a5dc30e2466c06e46a81512ab948f7e75d0
link
/test integ-framework-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-pilot-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-codecov-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-codecov-master

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

integ-framework-k8s-presubmit-tests-master
478a8a5dc30e2466c06e46a81512ab948f7e75d0
link
/test integ-framework-k8s-presubmit-tests-master

istio-lint-master
478a8a5dc30e2466c06e46a81512ab948f7e75d0
link
/test istio-lint-master

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

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

integ-framework-k8s-presubmit-tests-master
478a8a5dc30e2466c06e46a81512ab948f7e75d0
link
/test integ-framework-k8s-presubmit-tests-master

istio-lint-master
478a8a5dc30e2466c06e46a81512ab948f7e75d0
link
/test istio-lint-master

istio-racetest-master
900bc337d38b6c7d0aaac54b3eaf63dc7d254f1a
link
/test istio-racetest-master

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

/test istio-racetest-master

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test integ-new-install-k8s-presubmit-tests-master

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

istio-racetest-master
900bc337d38b6c7d0aaac54b3eaf63dc7d254f1a
link
/test istio-racetest-master

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

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

Test name
Commit
Details
Rerun command

istio-pilot-multicluster-e2e-master
0215f98ab74df11deebde92d31a4cebc08491240
link
/test istio-pilot-multicluster-e2e-master

istio-racetest-master
900bc337d38b6c7d0aaac54b3eaf63dc7d254f1a
link
/test istio-racetest-master

integ-new-install-k8s-presubmit-tests-master
900bc337d38b6c7d0aaac54b3eaf63dc7d254f1a
link
/test integ-new-install-k8s-presubmit-tests-master

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

/test istio-racetest-master

/lgtm


Convert Markdown tables into HTML tables for proper rendering in javadocs

Hello people,
This pull request replaces two tables (written in Markdown) in the README file of the module, azure-identity with their alternative representations in HTML in order to have them render properly in the module's javadocs.
Thanks,
Jafer

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: jaferkhan sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Can one of the admins verify this patch?

@jianghaolu can you please review this for the September milestone?

This has been fixed in the latest README. Thanks!


Don’t print stack traces from babel.

Summary
Trying to make @gaearon happy. Fixes #1929
Test plan
Add a syntax error, run jest and don't see a stack trace.


matrix_constructor.py stops at 55 %

Hello team,
A bit of context first, we are trying to produce a preliminary matrix and tree, where we had to speed up our SGTs parsing by deleting the sequences for the one we did not have time to parse yet. But eventually, each ortholog has its parsed tree.
Running apply_to_db.py and prep_final_dataset.py went well (I just had the warning 'There are empty files in the output. Please check for errors'). But when running matrix_constructor.py -i prep_final_dataset_out_Mar.28.2024 -o matrix_constructor_Edouard -t 18, it seems the process always stop when encountering the file nsfNP597649.fas, which looks fine.
[Fri Mar 29 10:44:03 2024]
rule remove_gaps:
    input: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_Edouard_concatenate_only/trimal/nsfNP597649.trimal
    output: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_Edouard_concatenate_only/trimal/nsfNP597649.final
    log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_Edouard_concatenate_only/logs/trimal/nsfNP597649.log
    jobid: 643
    wildcards: gene=nsfNP597649

Job counts:
        count   jobs
        1       remove_gaps
        1
Failed to solve scheduling problem with ILP solver. Falling back to greedy solver. Run Snakemake with --verbose to see the full solver output for debugging the problem.
[Fri Mar 29 10:44:05 2024]
Finished job 643.
536 of 967 steps (55%) done
Exiting because a job execution failed. Look above for error message
Complete log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/.snakemake/log/2024-03-29T102020.202337.snakemake.log

By the way all the trimal and divvier logs are emtpy in the obtained matrix folder.
I attach here the fas file in question nsfNP597649.txt  and the complete matrix log 2024-03-29T102020.202337.snakemake.log and hope you will be able to help me on this issue.
Thank you and my best wishes,
Edouard

PS: I run PhyloFisher 1.2.12

EDIT: I try to run matrix_constructor.py without this particular file but it again stopped at step 536 (55 %)

Hi Edouard,
Thank you for your email. Robert and I will look into this as soon as we can. Just know we may not be as quick as usual. Robert is defending his dissertation in the next few weeks, and I have a full teaching load along with all that is involved in getting my new lab up and running. However, we will try our best to get back to you in a timely manner. As always, thank you for using PhyloFisher in your research.
Alex
Alexander K. Tice, Ph.D.
Assistant Professor
Department of Biological Sciences
Texas Tech University
Office: 313
Lab: 308 & 313A
Phone:<PHONE_NUMBER>

From: Edouard Bessette @.>
Sent: Friday, March 29, 2024 8:33 AM
To: TheBrownLab/PhyloFisher @.>
Cc: Subscribed @.***>
Subject: Re: [TheBrownLab/PhyloFisher] matrix_constructor.py stops at one particular fas file (Issue #119)
This email originated outside TTU. Please exercise cautionhttps://askit.ttu.edu/phishing!
EDIT: I try to run matrix_constructor.py without this particular file but it again stopped at step 536 (55 %)
—
Reply to this email directly, view it on GitHubhttps://github.com/TheBrownLab/PhyloFisher/issues/119#issuecomment-2027251336, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADA4OK5U3WVEP3YUNRJ5ZEDY2VNQRAVCNFSM6AAAAABFOI6ZUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRXGI2TCMZTGY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello Alex,
Thank you for your message. No worries. Good luck to Robert and thank you to look at the issue when you have time.
Best wishes,
Edouard

Hi @Edouard94,
I am unable to find the issue based on the files you shared.
Do you mind sharing the one of the prequal logs for a gene that failed, such as "/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_Edouard_concatenate_only/logs/prequal/psmc6NP585947.log"?
I apologize for just now getting back to you on this issue.
Best,
Robert

Hi @robert-ervin-jones,
No worries, thank you for coming back to me and I hope your defense went well!
The matrix script seems to always fail at 55 %, I reran the script and it stopped here:
/usr/bin/bash: line 1: 304244 Aborted                 (core dumped) prequal /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024/prequal/rpl13aNP584823.aa > /home/Shared/PhyloFisher>[Tue Apr 30 18:02:44 2024]
Error in rule prequal:
    jobid: 299
    output: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024/prequal/rpl13aNP584823.aa.filtered
    log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024/logs/prequal/rpl13aNP584823.log (check log file(s) for error message)
    conda-env: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/.snakemake/conda/ef33e40ff013e38483a591b14ff8ec4b
    shell:
        prequal /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024/prequal/rpl13aNP584823.aa >/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_const>        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Job failed, going on with independent jobs.
Exiting because a job execution failed. Look above for error message
Complete log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/.snakemake/log/2024-04-30T152000.550247.snakemake.log
snakemake -s /home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.smk --config out_dir=/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024 in_dir=/home/Shared/PhyloFisher_Mi>Traceback (most recent call last):
  File "/home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.py", line 140, in <module>
    run_snakemake()
  File "/home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.py", line 89, in run_snakemake
    subprocess.run(smk_cmd, shell=True, executable='/bin/bash', check=True)
  File "/home/edouard/miniconda3/envs/fisher/lib/python3.7/subprocess.py", line 512, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command 'snakemake -s /home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.smk --config out_dir=/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30>

I then attach the corresponding prequal log as you requested Robert, here it was: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_Apr.30.2024/logs/prequal/rpl13aNP584823.log --> rpl13aNP584823.log
Let me know if I can share other files and thank you for your help.
Best wishes,
Edouard

Hi @Edouard94,
I just noticed you said you got the 'There are empty files in the output. Please check for errors' warning. Did you cleanup the empty files?
Best,
Robert

Also, it appears that nsfNP597649 does not error out.

Hi @robert-ervin-jones,
Thank you for your message.
Yes true I had the error 'There are empty files in the output. Please check for errors'; well noticed! I continued the workflow without paying attention to that...
I will remove the empty files from prep_final_dataset_out/ (I indeed do have some), and rerun the matrix_constructor script. Hopefully it will fix my issue.
Thank you Robert.

Hello again @robert-ervin-jones,
I ran again the matrix_constructor without the empty files in prep_final_dataset_out/ and the script almost went through but stopped at 97 % with the following error (snake log):
[Tue May  7 14:47:17 2024]
Finished job 182.
452 of 469 steps (96%) done
Select jobs to execute...
Failed to solve scheduling problem with ILP solver. Falling back to greedy solver. Run Snakemake with --verbose to see the full solver output for debugging the problem.

[Tue May  7 14:47:17 2024]
rule remove_gaps:
    input: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_May.07.2024/trimal/mcm3NP597158.trimal
    output: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_May.07.2024/trimal/mcm3NP597158.final
    log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_May.07.2024/logs/trimal/mcm3NP597158.log
    jobid: 181
    wildcards: gene=mcm3NP597158

^[[33mJob counts:
        count   jobs
        1       remove_gaps
        1^[[0m
^[[33mFailed to solve scheduling problem with ILP solver. Falling back to greedy solver. Run Snakemake with --verbose to see the full solver output for debugging the problem.^[[0m
[Tue May  7 14:47:18 2024]
Finished job 181.
453 of 469 steps (97%) done
Exiting because a job execution failed. Look above for error message
Complete log: /home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/.snakemake/log/2024-05-07T121913.653590.snakemake.log
snakemake -s /home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.smk --config out_dir=/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_May.07.2024 in_dir=/home/Shared/PhyloFisher_Mi>Traceback (most recent call last):
  File "/home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.py", line 140, in <module>
    run_snakemake()
  File "/home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.py", line 89, in run_snakemake
    subprocess.run(smk_cmd, shell=True, executable='/bin/bash', check=True)
  File "/home/edouard/miniconda3/envs/fisher/lib/python3.7/subprocess.py", line 512, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command 'snakemake -s /home/edouard/miniconda3/envs/fisher/bin/matrix_constructor.smk --config out_dir=/home/Shared/PhyloFisher_MicrosporidiainInsectGenomes/Microsporidiatoadd/matrix_constructor_out_May.07>

The last trimal job returned an empty log (trimal/mcm3NP597158.log). The fasta file mcm3NP597158 looks normal: mcm3NP597158.txt.
Any idea on why the script stopped? Let me know if I can attach some files.
Thank you for your help,
Edouard

Yes sure, here it is:
psma2NP597286.log

Hi @Edouard94,
Sorry for just now getting back to you. In this case, you only have one sequence. I would look at prep_final_dataset_out and consider removing genes with that few sequences.
Best,
Robert

Hello Robert,
Sorry for the radio silence, I was finishing my thesis and had a summer break.
Based on your last comment and after filtering out the files with only one sequence, the script is now working fine!
Thanks a lot,
Edouard


sqlite3 crash on require with nw.js node.js app

Issue Summary
Appears to be installing wrong module. I am running windows 10. I used npm i sqlite3 -D
the first time. The program crashed application crashed after including the "const sqlite3 = require('sqlite3').verbose();"
It gave no visible error.
Steps to Reproduce
Running on windows 10
Create basic nwjs project with nodejs make sure it works then try adding sqlite3 to project.
Once you put the line shown above it will crash. when you try and run it.
Version
5.1.7
Node.js Version
20.11.0
How did you install the library?
npm i sqlite3

I have the same problem. I have also tried using import sqlite3 from 'sqlite3'; by adding '"type":"module"` in the package.json file. If I run it with node inspect like:
$ node inspect test2
< Debugger listening on ws://<IP_ADDRESS>:9229/6ac6169c-e11b-4540-834f-4424cb5ad943
< For help, see: https://nodejs.org/en/docs/inspector
<
connecting to <IP_ADDRESS>:9229 ... ok
Break on start in test2:1
> 1 #!/usr/bin/env node
  2
  3 import sqlite3 from 'sqlite3';
debug> n
debug> There was an internal error in Node.js. Please report this bug.
read ECONNRESET
Error: read ECONNRESET
    at TCP.onStreamRead (node:internal/stream_base_commons:217:20)
    at TCP.callbackTrampoline (node:internal/async_hooks:130:17)
node:events:497
      throw er; // Unhandled 'error' event
      ^

Error: kill EPERM
    at ChildProcess.kill (node:internal/child_process:512:26)
    at process.handleUnexpectedError (node:internal/debugger/inspect:360:42)
    at process.emit (node:events:531:35)
    at process.emit (node:domain:488:12)
    at process._fatalException (node:internal/process/execution:178:25)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess.kill (node:internal/child_process:512:12)
    at process.handleUnexpectedError (node:internal/debugger/inspect:360:42)
    [... lines matching original stack trace ...]
    at process._fatalException (node:internal/process/execution:178:25) {
  errno: -4048,
  code: 'EPERM',
  syscall: 'kill'
}

Node.js v21.7.2

woodbri@DESKTOP-MD7Q6H6 MINGW64 ~/work/test-sqlite3
$ uname -a
MINGW64_NT-10.0-19045 DESKTOP-MD7Q6H6 3.5.3.x86_64 2024-05-06 06:45 UTC x86_64 Msys

woodbri@DESKTOP-MD7Q6H6 MINGW64 ~/work/test-sqlite3
$ node -v
v21.7.2

woodbri@DESKTOP-MD7Q6H6 MINGW64 ~/work/test-sqlite3
$ npm -v
10.5.0

I also tried to load<EMAIL_ADDRESS>and it also fails with node -e 'require("sqlite3") with the following:
$ node -e 'require("sqlite3")'
node:internal/modules/cjs/loader:1464
  return process.dlopen(module, path.toNamespacedPath(filename));
                 ^

Error: A dynamic link library (DLL) initialization routine failed.
\\?\C:\msys64\home\woodbri\work\test-sqlite3\node_modules\sqlite3\lib\binding\napi-v6-win32-unknown-
x64\node_sqlite3.node
    at Module._extensions..node (node:internal/modules/cjs/loader:1464:18)
    at Module.load (node:internal/modules/cjs/loader:1205:32)
    at Module._load (node:internal/modules/cjs/loader:1021:12)
    at Module.require (node:internal/modules/cjs/loader:1230:19)
    at require (node:internal/modules/helpers:179:18)
    at Object.<anonymous> (C:\msys64\home\woodbri\work\test-sqlite3\node_modules\sqlite3\lib\sqlite3
-binding.js:4:17)
    at Module._compile (node:internal/modules/cjs/loader:1368:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1426:10)
    at Module.load (node:internal/modules/cjs/loader:1205:32)
    at Module._load (node:internal/modules/cjs/loader:1021:12) {
  code: 'ERR_DLOPEN_FAILED'
}

Node.js v21.7.2


LSP can crash if many text change messages are sent at once

I have seen this on Neovim v0.5 when cycling through the omnifunc completion menu really fast will cause the lsp to die.
I believe the tree-sitter web binding is crashing. I have not found a way to prevent this so far. Hopefully with tree-sitter updates fixes this.
This issue will track any info I know or if others can provide more info

have not seen this in a while closing


I just want to compile t265 driver, how to modify makefile

It takes too long to compile the complete driver on raspberry pi. I just want the t265 driver. How to modify makefile.

@liu-jiaming hello,
T265 module support is an integral part of the SDK and must be built along with the library using -DBUILD_WITH_TM2=true flag.  Several Cmake/Make improvements that should improve build time:

Skip building examples (-DBUILD_EXAMPLES=false) it may take about 30% of the compile time
Make sure you have swap memory file defined (see RPI installation guide for details).
Use make -j$(($nproc)-1)) for parallel build.  Don't use 'make -j' as it will most certainly get the board stuck.

Hi @liu-jiaming
Were you able to try the information that ev-mp outlined?
*If we don’t hear from you in 7 days, this issue will be closed.

Hi @liu-jiaming
Were you able to try the information that ev-mp outlined?
*If we don’t hear from you in 7 days, this issue will be closed.


Remove expected file checks

Deleted expected template and parameter files
Removed generation and comparing of template and param files with expected ones.

This change is

@acs-bot test this please

@anhowe could you review this ?

@acs-bot test this please

@acs-bot test this please


API Documentation: Remove duplicate token string in authorization header

Having two apiToken strings is confusing. authHeader already has apiToken in it

@SohamJ Can you please sign the CLA for this change? https://docs.google.com/forms/d/19LpBBjykHPox18vrZvBbZUcK6gQTj7qv1O5hCduAZFU/viewform


New build v2.0.9

Comment:
Hi there!
I'm not really sure where to post this or if this is the correct format, but could someone (@swainn ?) please make a new conda build of the geoserver-restconfig package. v2.0.8 is incompatible with urllib3 > 2.0 due to the renaming of the 'method_whitelist' parameter to 'allowed_methods'. The workaround at the moment is just to specify 'urllib3<2.0' when making a new environment, but it would be nice not to have to do this going forward.
Thank you.

Hi @j-kapp this is the right place. I'll add the restriction to the requirements and prepare a new build.

I've added pull request #4 to address this. This will build a new version of 2.0.8 with the fix.

Thank you @swainn. It's working for me now without the workaround


v1.13 Backports 2024-06-13

[ ] #33093 (@sayboras) :warning: resolved conflicts

ℹ Hit conflicts due to the usage of MetalLB rather than Cilium LB. Preserved the main hunk, and adapted the debug output.

Once this PR is merged, a GitHub action will update the labels of these PRs:
 33093

/test-backport-1.13

The changes are only for ci-ingress and ci-gateway-api jobs, so jenkins jobs are not a blocker to merge. Marking this ready to merge.


Allow user defined dockerfile

If I compiled docker before, I want to use the built image directly in case that some library becomes unavailable in the future.
$ echo "FROM docker-dev:open-1.11.2
COPY . /go/src/github.com/docker/docker" > Dockerfile.mine
$ DOCKERFILE="Dockerfile.mine" BIND_DIR=. make binary

Signed-off-by: Chun Chen<EMAIL_ADDRESS>

LGTM

/ping @jfrazelle @tianon

Have you tested your one-liner without this change?  Make should by default already function in this way (ie, := should only set DOCKERFILE if it isn't already set).  If DOCKERFILE='something' make binary doesn't work, try make DOCKERFILE='something' binary or make binary DOCKERFILE='something' instead.

make DOCKERFILE='something' binary works. I was mislead by BIND_DIR := $(if $(BINDDIR),$(BINDDIR),$(if $(DOCKER_HOST),,bundles))
Thanks, closing this one.


Add Photon - LL Comparison Table In Hardware

Creates an unnecessary tension in my eyes. People can see the performance for themselves via the performance matrix.


Implement autodiscovery of Arduino WiFi shield.

Broadcast UDP packets with Cordova chrome.socket. An Arduino running the example's Arduino code should reply. Code from ETC's autodiscovery feature can be re-used for the client.

Broadcast from Arduino should include some unique ID to distinguish between multiple devices. The feature should also be optional, in case it's not wanted.


  Please download one of these urls and extract it into 'build/' at the top-level of the source repository.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for matplotlib
Failed to build matplotlib
ERROR: Could not build wheels for matplotlib, which is required to install pyproject.toml-based projects

Hi mkazanov, I also meet the same issue. Did you solve this problem? How do you solve it?

Hi mkazanov, I also meet the same issue. Did you solve this problem? How do you solve it?

Hi MeteorLiu96,
No, still waiting for answer from developers of this package. I am on Mac and my OS is Sonoma 14.1
I am going try to install on Windows. I let you know if it successful or not.

Hi @mkazanov,
Thank you for reaching out and apologies for the late response.
Could you please let me know what version of Python you are using and how are you installing the SigProfilerClusters tool?
My suggestion will be to create a new conda environment with Python version 3.8 to 3.10(conda create -n test python=3.10) and then try to re-install the SigProfilerClusters pipeline. Please let me know if that does not work.
Best,
Mousumy

Hi all, I solved this issue. I thought the reason is the access to this website (http://www.qhull.org/) was banned by my internet. I also tried to open this website by browser, it also failed. When I switched to another internet supplier which could access this website, it successfully installed.

Hi all,
If you encounter any further problems with the installation of the tool, please feel free to reopen the issue.
Thanks!
Mousumy


New UI - Last infoton in full page list is cut off

See attached printscreen
Last infoton is cut off

Verified and closed


Can you register this api on "maven"?

Can you register this api on "maven"?
There are many projects that need to use this API through Maven.
If the registration is completed, can you tell me groupId, artifactId, and version?
It might sound weird because i use a translator.

bump

Also me need this!

Use JitPack: https://jitpack.io/#BeYkeRYkt/LightAPI


[HOPS-96] Added checkAccess rpc to DFS based on HDFS-6570

https://hopshadoop.atlassian.net/browse/HOPS-96

Can one of the admins verify this patch?

jenkins test this please.