About how to divide a .xmodel

I found that the total execution time of each layer in vaitrace "fine_grained" mode is greater than that of the network that does not use "fine_grained".
May I ask why? Or is there a way to read the .xmodel file and divide a network into several networks according to the order of DPU execution and execute them separately? (because I know that many layers are not parallel, so it doesn't affect the total execution time)?

hi @jx991220 , while fine grained profiling model is enabled. dnn model will be spitted into several sub-graphs(one layer -> one sub-graph) and executed one-by-one.

It will introduce significant scheduling and memory access overhead, especially for small models.
Some xcompiler's optimizations (like layer fusion) will be turned-off.

and I don't think we can dictatorially edit the xmodel . you can modify the original pytorch/tf model then re do the vitis-ai quantization and compile flow to get a new .xmodel

Hi @jx991220
Is this issue solved?

Hi @jx991220
Since we haven't received your reply for a long time, we assume you have solved this issue and I'm going to close it. If you still have any questions, please feel free to reopen it. Thank you very much.


Support custom block types inlined in Portable Text

Sanity now supports inline custom objects/_types!

These objects are embedded on the block level, but you may also need objects that appear inline with text useful for stuff like footnotes, ticker-symbols or sparklines. Add these to an array under the of key in the block type object:
{   title: 'Rich text',   type: 'array',   of: [     {       type: 'block',       of: [         {type: 'footnote'}       ]     }   ] }

This may not be the best way to achieve this
I've added another layer of testing the _type for custom serializers before finally testing the Marks.

This looks great!
Would you add a test + snapshot to cover possible options in the line you edited? For example:
https://github.com/nuxt-community/sanity-module/blob/e7b1a9bf323e1393b342f0fcacdbafa2f54bd68f/test/unit/sanity-content.test.ts#L15-L28

This looks great!
Would you add a test + snapshot to cover possible options in the line you edited? For example:
https://github.com/nuxt-community/sanity-module/blob/e7b1a9bf323e1393b342f0fcacdbafa2f54bd68f/test/unit/sanity-content.test.ts#L15-L28

Done!
I wrote a customLink component as an example but decided to keep it simple and didn't include. Code below if helpful.
`
`


Add required tiller version for chart.

As istio chart requires helm 2.7.2 or later from https://istio.io/docs/setup/kubernetes/helm-install.html#prerequisites
It's necessary to add tillerVersion to Chart.yaml

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: morvencao
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test istio-unit-tests

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.

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test istio-unit-tests

prow/e2e-bookInfoTests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test e2e-bookInfo

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test istio-pilot-e2e

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test e2e-bookInfo

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test e2e-bookInfo

prow/e2e-bookInfoTests-v1alpha3.sh
97ebeb0ebad900d03e5d8b8f90cce169a885edd2
link
/test e2e-bookInfo-envoyv2-v1alpha3

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.


How to invoke hidden cmdlets from custom C# cmdlets?

Hi, I'm trying to invoke hidden cmdlets from a custom cmdlet developed in C#.
Since the auto generated cmdlets inherit from PSCmdlet one cannot just create a new instance and call Invoke (and seems like there is no way to change the base class type?). I've been trying multiple ways to execute the generated cmdlets, like<EMAIL_ADDRESS>without much success, always getting the error:

The module 'MyModule.internal' could not be loaded.

From a custom powershell cmdlet simply doing MyModule.internal\Hidden-Command works fine, the command executes, but from custom C# cmdlets I haven't found a working way. Does anyone know a way to do this?

You can not do that in your custom C# cmdlet because the module {module}.internal is not loaded as the error tells. You can do that in custom PowerShell script is because we use 'import-module Az.{module}.internal.psm1' in az.{module}.custom.psm1. Instead, you could call the backend Client API directly in your C# custom cmdlet.

Thanks, that's what I ended up doing 🙂


More fallbacks for get_or_create firewall

Fixes first error in https://github.com/CloudVE/cloudbridge/issues/167 along with https://github.com/CloudVE/cloudbridge/commit/5328b65db9162b29b5179aa4a61bce6122f88e9d

I'll merge this to move on but we still have the router creation issue remaining.


Bugfix - Session Delegate Precondition

This PR removes the URL session delegate precondition check.
Goals :soccer:
Users should be able to create Session instances alongside performance monitoring frameworks such as NewRelic that are swizzling our URLSessionDelegate APIs by proxying.
Implementation Details :construction:
While the session delegate precondition check is a good idea, enforcing the check causes Alamofire to be unusable with other frameworks that are swizzling out URL session delegate APIs. In cases where say Alamofire and NewRelic are being used within the same application, the precondition check will always fail.

The above example demonstrates what you get when you po the session.delegate and delegate inside the precondition removed in this PR. The session.delegate is an NRMAURLSessionTaskDelegate (courtesy of NewRelic proxy swizzling) and the delegate is an Alamofire.SessionDelegate.

I'm open to other ideas here as well. We could add a flag to disable the precondition that's false by default, but I thought it might be overkill to expose this precondition controls through the public API.
Testing Details :mag:
N/A.

My 2 year old code started crashing on Alamofire 5.
I am using AlamofireObjectMapper "6.0.0" which install Alamofire at "5.0.0-beta.2"

My 2 year old code started crashing on Alamofire 5.
I am using AlamofireObjectMapper "6.0.0" which install Alamofire at "5.0.0-beta.2"

Did you fix it?

Hi @cnoon
i'm creating the framework which is like monitoring framework such as Newrelic and dynatrace. Could you please tell me how to monitor using URLSessionDelegate APIs
Thanks in advance


uploading a video on android doesnt work

the condition to check if the item is a video is failing thus passing an undefined to the path on the uploader

after it finishes uploading the video, it crashes with setState on unmounted component

The video uploading works on iPhone 7 OS 12.1.1 (16C50). Maybe the problem is related when uploading an image and a video on a single post.

Working on Nexus 5X Android 9 API 28 (Video Uploading, Video FullScreen and Video Thumbnail are all working)

@alexcroitoru95 ill leave this to you, if u can please check it once more on Nexus 6 Api 28 and if its okay, feel free to close this issue 😝

@Shadyzpop cool

Works on Nexus 5 Android 8.1 API 26, Nexus 6 Android 9 API 28


When opening at the top and while filtering the element gets dettached

Hi.
First of all, I have to congratulate you on the amazing work you're doing with this lib.
I found a use case that could be improved from an experience point of view that is related to the fact that when the dropdown opens at the top and we start filtering the options, the ballon with the options gets detached from the selected element as you can see on the GIF:

Also, how can we manage the event responsible to reattach the element when we scroll on the page?
Thank you in advance!

@gnbm thanks for letting me know this issue.
I will fix this issue in the future release.
For positioning drop-down I have used my another plugin https://github.com/sa-si-dev/popper

@sa-si-dev Do you have an estimate about when would you do a release with this fix?
Once again, great work and thank you for the fast reply 🚀

This issue has been fixed in the release v1.0.27


Mysql is also supported in gcloud IAM auth

Mysql is also supported now in gcloud IAM database authentication
Adding Mysql in readme Note to let others know that Mysql is also supported with flag -enable_iam_login.
https://cloud.google.com/sql/docs/mysql/authentication
It works exactly similar to Postgres so no other documentation is required.
Checklist

[x] Make sure to open an issue as a
bug/issue
before writing your code!  That way we can discuss the change, evaluate
designs, and agree on the general idea.
[x] Ensure the tests and linter pass
[x] Appropriate documentation is updated (if necessary)

Relevant issues:

Fixes #903

Duplicate of https://github.com/GoogleCloudPlatform/cloudsql-proxy/pull/865. I've added some additional context in #903.


Get-ComputerInfo : Unable to find an entry point named 'GetFirmwareType' in DLL 'kernel32.dll'."

Error Message:
"Get-ComputerInfo : Unable to find an entry point named 'GetFirmwareType' in DLL 'kernel32.dll'."
Environment:
Windows Management Framework 5.1
Operating System: Windows 7 x64
Examples
PS C:\Users\user> Get-ComputerInfo -Property OsInstallDate
Get-ComputerInfo : Unable to find an entry point named 'GetFirmwareType' in DLL 'kernel32.dll'.
At line:1 char:1
+ Get-ComputerInfo -Property OsInstallDate
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Get-ComputerInfo], EntryPointNotFoundException
    + FullyQualifiedErrorId : System.EntryPointNotFoundException,Microsoft.PowerShell.Commands.GetComputerInfoCommand

PS C:\Users\user> Get-ComputerInfo -Property TimeZone
Get-ComputerInfo : Unable to find an entry point named 'GetFirmwareType' in DLL 'kernel32.dll'.
At line:1 char:1
+ Get-ComputerInfo -Property TimeZone
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Get-ComputerInfo], EntryPointNotFoundException
    + FullyQualifiedErrorId : System.EntryPointNotFoundException,Microsoft.PowerShell.Commands.GetComputerInfoCommand

Name                           Value
----                           -----
PSVersion                      5.1.14409.1005
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1005
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@n3l5 Thanks for bug report!
GetFirmwareType is introduced in Windows 8 and Windows Server 2012

Given the behavior isn't there on Win7, I'm not sure what we can do other than do a better of job of erroring out. We also wouldn't ship a change for that unless we had a hypothetical WMF 5.2, though I'm not speaking here to whether that's happening or not.

Unless there is an option to feature request a kernel fix for Win 7 to support this, can be closed out since this appears to be a kernel feature and Win 7 is on the way out.

Thanks @n3l5!

Isn't solution is just one if check for windows version in GetComputerInfoCommand.cs?

We should wrap the api calls so this fails silently.

Just as a point of interest, at the time this cmdlet was written the target OS was Win10.
Fortunately there aren't too many native API calls, so wrapping them shouldn't be too onerous.
@SteveL-MSFT  I can take this one if you like.

Perhaps it is time to start porting this cmdlet instead of doing partial fixes. It would be good to do it in Beta time.

Making it cross-platform is a pretty big task and probably something we'd do for 6.1.0.  @jeffbi it should be pretty straightforward to wraps so they silentlycontinue for current release.

Alpha 18 no longer crash on GetFirmwareType in Windows 7, but other strange crash - #3541.


Should be offering unenrolled runs as enrollable on the product page

I enrolled in https://rc.mitxonline.mit.edu/courses/course-v1:MITxT+14.740x/ for the "no certificate" course run.
Then I unenrolled.
When I went to the product page and clicked the enroll button again it no longer offered this run for enrollment.
Expected Behavior
If I have enrolled in a run, and unenrolled from it, I should be able to enroll again through the product page.
Possible Solution
When finding enrollable runs we should make sure we are not filtering out "unenrolled" runs.

I am not able to reproduce this issue issue any longer. Closing it


typo: dependancy -> dependency

Cleaned up code fence blocks as well

@nschonni : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@nschonni Thanks! #sign-off


OP Low Gear/High RPM issue on 2020 Lexus RX 350

Describe the bug
Most notably at freeway speeds, engaging OP results in two downshifts shortly thereafter and an unnecessarily high RPM until disengaged or upon encountering a vehicle within following distance intervention.  Manual or stock cruise control driving (OP disabled) over the same stretch of road at the same speed does not result in this low gear/high RPM behavior.  As indicated by several Discord posts, this same behavior is also evident on other RX’s of earlier year models as well.
How to reproduce or log data
Engage OP at freeway speeds and observe downshifts/high RPMs (2500-4000) shortly thereafter.  Alternatively, accelerate to freeway speeds with OP engaged and observe high RPMs.  Disengage OP and observe the car promptly upshift and RPMs return to a normal freeway cruising range 1500-1800.
Expected behavior
Engaging OP or using it at freeway speeds should not result in downshifts and maintaining a highly inefficient DOUBLING of engine RPM just because OP is in use.  OP long should result in mimicking driving the car manually at speed or as the stock cruise control does on the freeway—overdrive gear and 1500-1800 RPM.
Device/Version information

Dongle ID:  3a5c5101bd71ad5d
Route:  3a5c5101bd71ad5d|2020-08-30--13-12-43
Timestamp:  Reference multiple engagements/disengagements starting 2 minutes into the clip.
Version:  release2 v0.7.8
Car make/model:  2020 Lexus RX 350

Additional context
There appears to be a discrepancy between UI_SET_SPEED and SPEED (actual) that could be causing the problem.  @nelsonjchen developed a “set speed scaling” solution that does resolve the problem.  Reference his comment to @cecnic1989’s PR about the same issue:  https://github.com/commaai/opendbc/pull/279#issuecomment-679826539
When implementing @nelsonjchen’s solution and adjusting the speed scale factor to 0.974 (which represents the 2.6% difference between UI_SET_SPEED and SPEED on my vehicle at approximately 65 mph), the low gear/high RPM issue is completely resolved.  Reference route 3a5c5101bd71ad5d|2020-08-30--14-30-28 where over the same stretch of freeway the correct gear is maintained and the RPM stays within 1500-1800 except for a brief period of road surface incline and then it promptly upshifts and returns to the normal RPM cruising range.
OP code needs to be modified so that the speed discrepancy between what is set and what the actual speed is does not trigger inappropriate OP long commands that cause undesired downshifts and high RPMs that result in considerable inefficiency and make OP use on the freeway impractical.  This modification could be in the form of a “Set Speed Scale” menu interface as proposed by @nelsonjchen (which I support) or fundamental changes to OP code on a per-vehicle basis to account for the speed discrepancies.  In any case, a change is needed to correct this undesired behavior.  Thanks.

Just to be clear, the proof-of-concept I produced only modifies the cruise state's speed and not the speed reading of the vehicle. I actually like the GPS/close to GPS reading of the speed of the Comma 2.

It might have to do with the PERMIT_BRAKING bit in the ACC message (https://github.com/commaai/openpilot/blob/master/selfdrive/car/toyota/toyotacan.py#L39). Stock Toyota only sets this when there is a lead car, since otherwise it won't need to brake. So far we haven't seen any problems by always settings this to 1, but your car might react differently.

@pd0wm I noticed this change in the code from the solution provided by @nelsonjchen.  However, with his code left intact except for the speed scaling (i.e. PERMIT_BRAKING set to lead and the speed scale factor set to 1.0) the low gear/high RPM issue returns.  The only thing I have found so far that corrects the issue is adjusting the speed scale factor (in my case to 0.974) as he proposed.

The change in the code was a noop. We have always been sending that bit. But reading your complaint better I don't think it's related.
My current guess is you're triggering a safety in the Toyota PCM. I have seen this happen when you accelerate too much beyond the set speed. The car then aggressively slows you down. This could be caused by a scaling issue somewhere.
We should probably try to figure where this discrepancy is coming from. When you're using stock ACC, does the car's speed stay below the set speed (according to EON set speed and car speed)?

Stock ACC is a bit sloppy compared to OP.  Whether in town at low speed or on the freeway, it allows the car to drift a few mph both below and over the set speed before it takes action to either accelerate or slow the car to the set speed.  Perhaps this is by design for fuel economy?  In equilibrium (level road surface, no curves) it does a good job of maintaining the set speed.  Overall, OP seems more aggressive in maintaining a set speed over a variety of changing road conditions--not abrupt but more precise.  The car's CAN speed and the speed displayed on the Comma Two is ALWAYS a few mph less than the set cruise speed as displayed on the vehicle's HUD.  In reviewing the cabana info from my drives, that's how I determined that the car's speed was 2.6% (on average at 65 mph) less than the set speed and thus the 0.974 speed factor.

@pd0wm here is a Stock ACC route I just performed over the same stretch of freeway as the other two routes in this issue report (albeit at a slightly slower speed to eliminate a lead vehicle over the majority of the route.)
3a5c5101bd71ad5d|2020-08-31--10-03-02
Interestingly, although I set the car's cruise to maintain 62 mph, the SET_SPEED parameter of PCM_CRUISE shows 97 kph which equates to 60.27 mph.  The SPEED parameter seemed to stay within 96 to 99 kph (59.65 to 61.52 mph).  On my vehicle's HUD, with the cruise set at 62 mph, the speedometer indicating steady at 62 mph, the Comma Two displayed 59 mph.  The car's speedometer stayed within 2 mph of the set speed.  When it reached 64 mph, it didn't downshift/rev but simply eased back to 62 mph.  There were a couple brief single gear downshifts with RPMs rising to approximately 2,100 RPM to handle a decline in the road surface but nowhere near the constant maintaining of 2 gears too low and 3,000 RPM with OP engaged and without speed scaling.
@cecnic1989 has an earlier model RX with the same high rev issue and approached the solution differently by modifying the relevant opendbc files so that the set speed and speed matched.  I think @BobMeeks01 has a 2020 RX with the same high rev issue and he also modified the relevant opendbc files to solve the problem as well.  His fork is at https://github.com/bobmeeks01/openpilot-071. His custom .dbc file is  lexus_rx350_2020_cabana.dbc.
Being a newbie, it was easier for me to implement @nelsonjchen's solution since his code changes didn't require the files to be compiled on release2.
I hope this additional context and the additional route demonstrating stock ACC behavior helps you identify the problem and the best way to correct the issue.  Thanks.

Additionally, with the cruise set to 62 mph as above, the PCM_CRUISE_SM UI_SET_SPEED matches the 62 mph as set whereas, simultaneously, the PCM_CRUISE_2 SET_SPEED shows 97 kph which equates to 60.27 mph.

FWIW My 2021 RX350 also suffers from this issue and I too decided to use one of @nelsonjchen's solutions.

Glad that is working for you.  I settled on a speed scale factor of 0.98
and that seems to work fine for me.  Also, changing the toyotacan.py file
doesn't seem to make any difference.  So, I have been just leaving that one
as is as I apply the changes to each new release.  Hopefully, Comma
develops a permanent and more elegant solution to fix this problem for
good.
On Sun, Dec 6, 2020 at 12:56 PM brown1428<EMAIL_ADDRESS>wrote:

FWIW My 2021 RX350 also suffers from this issue and I too decided to use
one of @nelsonjchen https://github.com/nelsonjchen's solutions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/2106#issuecomment-739546565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2ETWP2U7G7OQXOMFKG7D3STPHVRANCNFSM4QP2DUCQ
.

I have also (just today) experimented with @bobmeeks01 opendbc revisions, which also work well. The opendbc approach is attractive because now all speeds (speedo, cruise, OP) appear to very closely match.

What OP versions are you guys working with?  I've been stuck on 0.7.1 with my custom .DBC file but just tried to upgrade to 0.8.1.  Stock version still has this issue.  I tried re-applying my custom .DBC values but it's like the changes aren't being recognized.  No matter what I do it still drives the same and the mph and cruise set read-outs vary substantially between OP and car.  If you are able to update .DBC values in recent OP versions then friend me on discord.  I'd like to find out more.

What OP versions are you guys working with?  I've been stuck on 0.7.1 with my custom .DBC file but just tried to upgrade to 0.8.1.  Stock version still has this issue.  I tried re-applying my custom .DBC values but it's like the changes aren't being recognized.  No matter what I do it still drives the same and the mph and cruise set read-outs vary substantially between OP and car.  If you are able to update .DBC values in recent OP versions then friend me on discord.  I'd like to find out more.

I am on 0.8.1 and, yes, the issue has not yet been addressed.  With each new version, I simply add two lines of code that @nelsonjchen came up with to scale the speed to 98% and completely eliminate the issue.  This, of course, will not make the speedometer and Comma 2 speeds match, but I would much rather have the proper shifting behavior.
/selfdrive/controls/controlsd.py
if self.v_cruise_kph != 255:
controlsState.vCruise = controlsState.vCruise * 0.98

I am on 0.8.1 and, yes, the issue has not yet been addressed.  With each new version, I simply add two lines of code that @nelsonjchen came up with to scale the speed to 98% and completely eliminate the issue.  This, of course, will not make the speedometer and Comma 2 speeds match, but I would much rather have the proper shifting behavior.
/selfdrive/controls/controlsd.py
if self.v_cruise_kph != 255:
controlsState.vCruise = controlsState.vCruise * 0.98

Adding my voice to the chorus to get this addressed. '17 RX 450h with high rpm issue has taken OP off the table for freeway driving.

Adding my voice to the chorus to get this addressed. '17 RX 450h with high rpm issue has taken OP off the table for freeway driving.

Confirming that issue is still present with 0.8.3. 2016 Lexus Rx with SDSU.

I built this continuous Lexus RX fork(s) generator as a stopgap:
https://github.com/LexusRXopenpilotUG/openpilot
It'll continually apply my hardcode hack atop branches daily and push them to the LexusRXopenpilotUG/openpilot repository. Also, some light instructions for no-SSH/no-Git installation via Shane's Fork installer are included at the bottom.

Well, for some reason the WHEEL_SPEED_xx values I was using to fix this issue no longer seem to have the desired effect beginning with 0.8.6. Not clear why.

@nelsonjchen Thank you for creating the continuous Lexus RX fork! I returned my comma two because of the weird RPM issue (on top of a few other issues) but am excited to use your fork once my comma three arrives.

I ordered the C3 about a month ago and installed on my Lexus Rx350L.  Very excited about the whole autonomous driving experience but something happened when upgraded to 0.8.10, there was an issue of shifting to lower gears when driving on the highways.   revving the engine to as high as 4500 when I was doing 75mph. This was a very serious issue.
Went to discord to see if other users have had this problem and thankfully @nelsonjchen suggested the fork he created.  I uninstalled comma software and reinstalled using the url.  Now I dont have the issue anymore.  Had a good driving day today using C3, city and highways, and the rpms stayed to under 2000.  I drove for 2 hours in city driving didn't see/feel any bad behavior.  I cruised  a good half an hour in highways and to about 75mph and saw no issues.  Thanks for the fork @nelsonjchen

@carl98nes Do you have a segment for where this happens?

Hi Harald:
Thanks for looking into this.  Here is the camera log I have for Nov 7
which ran for 30 min.  Notice that on a fairly flat highway road, I had to
disengage many times due to the very high rev and rpms.  I have a 2021
Lexus RX 350L, if that makes any difference on the code. I have other days
of logs but this captured my highway driving.
Also I need to point out that this happened only after the upgrade to
0.8.10. on Nov 1.  Before the upgrade, when I first received the C3 version
0.8.9,  it did not have this problem.
Let me know if you need anything else.
Thanks,
Carl98Nes
On Thu, Nov 11, 2021 at 12:10 PM HaraldSchafer @.***>
wrote:

@carl98nes https://github.com/carl98nes Do you have a segment for where
this happens?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/2106#issuecomment-966472387,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJLILDYMZFEN3AXCKLNHJDULP2HVANCNFSM4QP2DUCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

+1 where my custom dbc file updates to address this rpm issue no longer seem to work after going to 0.8.10.  I just upgraded from 0.8.2.   I’ll give @nelsonjchen fork a try.

@HaraldSchafer If you need any additional data please reach out.  I'm on Discord @bobmeeks01.  I'd be happy to grab driving segments to help troubleshoot.  We've been hacking our way past this issue for 2 years now and it's not safe for anyone who isn't aware and thinks the RX350 is "supported" and tries to drive this way.

Some driving segments with rlogs would be great (turn on the "upload raw logs" or upload trough useradmin). We would also need a stock drive (turn off the "enable openpilot" toggle in the settings) and drive the same stretch of road.

@bobmeeks01  Do you mind testing this branch just to confirm getting too close to the set speed is the issue?
https://github.com/commaai/openpilot/tree/fudge_cruise_speed

@HaraldSchafer Certainly!  I'll throw it on when I get home from work today and give it a try.

@HaraldSchafer I'm getting a black screen on startup.  Here's what I did to install via SSH:
cd /data; cp -rf ./openpilot ./openpilot.bob;
rm -rf ./openpilot;
git clone https://github.com/commaai/openpilot.git openpilot; cd openpilot; git checkout fudge_cruise_speed;
reboot;
Everything seemed to download and install, but no luck getting past the black screen after two reboot attempts.  Is there a better way to install??
Bob

You need to init the submodules, but there's an easier way. Uninstall openpilot and use the following URL: installer.comma.ai/commaai/fudge_cruise_speed.

Ok, seems pretty good!  No RPM issues up to 80 mph (speedometer).  RPM's sat around 2k, which is normal, and did not spike up to 3k then 4k after about 5 seconds ... which is the behavior I've seen since 0.7.1 without some code intervention.
With cruise set at 80 mph on the speedometer, the Comma had cruise set at 75 MPH.  At 35 mph (speedometer), comma had 33 mph.
I would encourage other RX owners to give it a try to verify.  I have rlog upload on, but not sure how to point you to it.  I can give you a URL from Cabana and the time when I had OP engaged if that helps.
Thanks for looking into this.  It will be great to have this resolved in released version!

You need to init the submodules, but there's an easier way. Uninstall openpilot and use the following URL: installer.comma.ai/commaai/fudge_cruise_speed.

Thanks, @adeebshihadeh.  That did the trick.

@pd0wm Here are a couple of routes.
This one is with Openpilot disabled.  There is good highway driving starting around 370 seconds with cruise enabled.  I did notice that even with just the dashcam software (no OP) that when speedometer was at 80 mph, dashcam on comma showed 75 mph.
41e96f2034770443%7C2021-11-24--15-19-20
This one is from yesterday (referenced above) using the "Fudge_cruise_speed" branch and OP enabled.  There is a good stretch of highway driving starting at 930 with OP engaged.
41e96f2034770443%7C2021-11-23--17-23-31
Let me know what else you need.  Thanks again to everyone helping with this.
Bob

vs a random rav4 segment:

Seems like there is a massive difference between the speeds reported by the wheels and the actual speeds (compared to GPS). Instead of fixing this in the wheel speed sensors they added some stuff to the PCM and the dashboard to fix this.
@cydia2020 UI_SET_SPEED is even higher in this case (after converting from mph to km/h) to match the units. So that's also not the solution.
Seems like the intention of the initial DBC change was correct, but changed both the wheel speeds and the set speed. It would work if you only scale the wheel speeds but leave the set speed as is.
@HaraldSchafer What do you think about adding a wheel speed multiplier in carParams to fix cars like this with egregious errors?

I have installed the fudge_cruise_speed and at first it seemed to be working fine on slow speeds and at high speeds on the freeway.  the RX speed is actually closer to c3 speed.  Then it happened on just one occasion and only for a brief moment (I was not sure why it happened).  I was on a freeway on a little uphill so I expected the gears to change to a lower gear and thinking the rpm would increase to about 3000rpm but the rpm shot up to 5000 rpm. I knew then something is wrong  I had to disengaged immediately by pressing the brake.  it happened this afternoon Thu, 25 Nov @ 15:07:20.  It didn't happen any other time but only on that instance.
Maybe it's an anomaly but I can't trust C3 on highway speeds.
dongle: c5c8fc431bbe68e5_2021-11-25--14-34-47--32--qlog.bz2

Given Willem's plots, the 3% fudge is likely just not enough.

Made it 3.5%.
@carl98nes @bobmeeks01 Can you try this branch? https://github.com/commaai/openpilot/pull/23079 (wheel-speed-factor). Make sure to update your submodules.
While at a steady speed with this branch please write down the following speeds and the date/time:

Speed on car HUD
Speed on openpilot UI
Set speed on car HUD
Set speed on openpilot UI

Here is a route from this morning:  41e96f2034770443%7C2021-12-01--08-22-41
Starting at around 8:30 am (480 seconds) I engaged for about 2 minutes.  The car set speed was 73 mph.  OP set speed was 71 mph.  When not following (full speed) the OP speed was 71 mph, but the car speed was 76 mph!!  3 mph higher than the set speed?  I've never seen that before.  I'll upload pictures I took with my phone.
Unfortunately, though, after about 2 minutes, the car starting downshifting like crazy and ramped up over 4k RPM's again.  As with @carl98nes, this might have started on a slight uphill grade but once the downshifting occurred the car got stuck and wouldn't resolve itself. I'll upload pictures but the set speeds and actual speeds were the same as before, but RPM's are double.  On this route, this was happening at around 570 seconds for about 10 seconds before I had to disengage.
These are at normal RPM's (480 seconds)

These are during rev'd RPM's (570 seconds)

Hi All,
I have the similar experience, not exactly the same but something is wrong
with the rpm that it just keeps on downshifting on high speeds.  On city
driving it's not apparent but when speeds go up to around 60mph then the
car would just suddenly downshift.
Cruising along at around 8:08 when I turned C3 but I should have known
something was wrong because the rpm did not go lower than 2200 and at
around 8:10, it started downshifting to 3.5K then go down to 2500 and back
up.  I had to disengage, I dont want to ruin my engine.
On Wed, Dec 1, 2021 at 5:02 PM bobmeeks01 @.***> wrote:

Here is a route from this morning: 41e96f2034770443%7C2021-12-01--08-22-41
Starting at around 8:30 am (480 seconds) I engaged for about 2 minutes.
The car set speed was 73 mph. OP set speed was 71 mph. When not following
(full speed) the OP speed was 71 mph, but the car speed was 76 mph!! 3 mph
higher than the set speed? I've never seen that before. I'll upload
pictures I took with my phone.
Unfortunately, though, after about 2 minutes, the car starting
downshifting like crazy and ramped up over 4k RPM's again. As with
@carl98nes https://github.com/carl98nes, this might have started on a
slight uphill grade but once the downshifting occurred the car got stuck
and wouldn't resolve itself. I'll upload pictures but the set speeds and
actual speeds were the same as before, but RPM's are double. On this route,
this was happening at around 570 seconds for about 10 seconds before I had
to disengage.
These are at normal RPM's (480 seconds)
[image: IMG_1447]
https://user-images.githubusercontent.com/58351215/144321039-5c7fc654-4a40-4363-ade8-1380c6a8e803.jpg
[image: IMG_1448]
https://user-images.githubusercontent.com/58351215/144321048-68e6302a-1f7d-4970-ae75-498d280c0a23.jpg
These are during rev'd RPM's (570 seconds)
[image: IMG_1455]
https://user-images.githubusercontent.com/58351215/144321070-9a633aca-2e06-4a46-8655-dbf8038fad11.jpg
[image: IMG_1456]
https://user-images.githubusercontent.com/58351215/144321083-7915bbb4-d371-4118-bb4c-b0ac1a2ecb59.jpg
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/2106#issuecomment-984092933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJLILCVMDPZS55H5RTSGL3UO2LOFANCNFSM4QP2DUCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@bobmeeks01 Thanks for the detailed report. I accidentally applied the changes to the TSS1 version of the RX instead of the TSS2 (LSS2) version. Let me know if it's better now.

Ha!  No problem.  Here's a new route:  41e96f2034770443%7C2021-12-02--11-42-10
Much better.   Drove engaged at 70+ mph for about 8 or 9 minutes straight with no revving issue, including some uphill driving.
Starting at 11:51:
Speed on car HUD = 73 mph
Speed on openpilot UI =  70 mph
Set speed on car HUD = 73 mph
Set speed on openpilot UI = 70 mph
Then starting at around 11:54 I bumped the speed:
Speed on car HUD = 80 mph
Speed on openpilot UI =  77 mph
Set speed on car HUD = 80 mph
Set speed on openpilot UI = 77 mph
I might be paranoid by now, but I did seem to think that there were 4-5 odd little bumps or hits in the RPM's ... like on uphills it would bump up but immediately come down, or on downhills the RPM's would suddenly drop and then come back up.  I can't really point to an exact time because it just seemed a little random.  But again, this might have happened even without OP ... I can't tell.

I'll merge this to master then! Feel free to update this issue if you have more feedback after merging it.

Thank you all for resolving this issue for RX owners! This is perfect.
I've been using my fork with the custom opendbc modifications for the past year. I tried the latest release from OP, and I'm still facing this issue on my 2017 Lexus RX. It looks like the above PR didn't add the wheelSpeedFactor to the 2016-2017 Lexus RX section. Therefore, I raised a quick PR to add wheelSpeedFactor fix for 2016-2017 Lexus RXs. See #23367.


DEP Use caret for silverstripe/recipe-plugin

Issue https://github.com/silverstripe/silverstripe-installer/issues/341
Note CMS 4 does define recipe-plugin even though silverstripe/installer does - https://github.com/silverstripe/recipe-kitchen-sink/blob/4/composer.json#L9 -
This is possibly because it needs to load "before" other things, I'm not sure. It's fairly harmless leaving it here regardless
Failing unit tests are unrelated

I think this MFA failure is new? Not mentioned in https://github.com/silverstripeltd/product-issues/issues/672

It does seem new.  But it's unrelated to this PR.


Encoding problem

Hi Mark,
I have an Encoding issue and maybe I have missed someting.
I changed the Encoding to default or Encoding.UTF8, but the value is always the same
if (stream.CanSeek) { stream.Seek(0, SeekOrigin.Begin); }

TextReader tx = new StreamReader(stream, **true**);
var opt = new CsvDataReaderOptions();

var pool = new StringPool(20);
var separator = HandleSeparator(separatorEnum);
if (separator != null)
{
    opt.Delimiter = separator;
}
opt.OwnsReader = false;
opt.StringFactory = pool.GetString;
var dataReader= await CsvDataReader.CreateAsync(tx, opt);
....
var val = dataReader.GetValue(columName)

Can you check this?
Thanks,
nk-cmd

You would have to provide me the file, otherwise I'd only be guessing.
This definitely looks like an encoding issue though.

That file appears to be using ISO-8859-1 encoding, or possibly Windows-1252. The detectOrderFromByteOrderMarks flag, will use the Unicode BOM to identify which Unicode encoding is being used, and if there is no BOM (as in this file) it will default to Utf-8. So, the StreamReader ends up using the wrong encoding for this file.
You can fix this, by specifying the Latin1 encoding explicitly:
using var r = new StreamReader(s, Encoding.Latin1);
            // make sure the console is writing an encoding
            // that can display all characters.
            Console.OutputEncoding = Encoding.UTF8;

            using var r = new StreamReader("error_ft.csv", Encoding.Latin1);
            using var csv = CsvDataReader.Create(r);
            while (csv.Read())
            {
                for (int i = 0; i < csv.RowFieldCount; i++) {
                    Console.WriteLine(csv.GetString(i));
                }
                Console.WriteLine("------");
            }

This will write the correct values from the file.


Add Fibonacci backoff reconnection

When the bot disconnects, it should try immediately
If a reconnect doesn't work, back off using fibonacci backoff
Used for server outages and daily server reboots.

We can detect if a server is down using the status ping protocol, and delay reconnection attempts until it is up.


runtimeCaching networkFirst doesn't work when there is any navigateFallback

Hello, I have a sw-precache-config.js like this:
module.exports = {
  stripPrefix: "build/",
  staticFileGlobs: ["build/static/**/*.{js,css,svg}", "build/manifest.json"],
  dynamicUrlToDependencies: pagesMappings,
  navigateFallback: "index",
  runtimeCaching: [
    {
      urlPattern: "/",
      handler: "networkFirst"
    },
    {
      urlPattern: /login/,
      handler: "networkFirst"
    }
  ],
  dontCacheBustUrlsMatching: /\.(js|css)/,
  swFilePath: "build/service-worker.js"
};

I need both my root and login pages to always go to network first to see if there is a redirect from the server, and I noticed it stopped working after adding a navigateFallback, I've even added a navigateFallbackWhitelist: [/notmatchableurl/], with no luck.
The only way I got networkFirst working again was by removing navigateFallback entirely.

actually, I believe this is not the problem


C++ compile tensorflow r2.5

tensorflow_cc.dll.if.exp : error LNK2001: 无法解析的外部符号 "public: __cdecl tensorflow::TensorShapeBase::TensorShapeBase(class absl::lts_2020_02_25::Span<__int64 const >)" (??0?$TensorShapeBase@VTensorShape@tensorflow@@@tensorflow@@QEAA@V?$Span@$$CB_J@lts_2020_02_25@absl@@@Z)
bazel-out\x64_windows-opt\bin\tensorflow\tensorflow_cc.dll : fatal error LNK1120: 1 个无法解析的外部命令
Target //tensorflow:tensorflow_cc.dll failed to build
INFO: Elapsed time: 28.003s, Critical Path: 25.68s
INFO: 4 processes: 2 internal, 2 local.
FAILED: Build did NOT complete successfully

Hi @MisRight! We see that the issue template has not been filled, could you please do so as it helps us analyse the issue [tf version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced].Thanks


Generate ARNs in order to be able to get the right ones from cloudformation lists.

With cloudformation I can recover a list of resources created by each stack but the data about each resource does not include the ARN.
In order to be able to match resources to cloudformation stacks I would like to be able to be able to use arnparse to generate ARNs from the constituent data recovered from a cloudformation stack.  See also the output of aws cloudformation list-stack-resources
I'd propose implementing a __str__ method which returns the ARN and a __repr__ method which returns an ARN instantiation call.
In fact I'm probably going to go ahead and do this, but I'd like to put up an issue to explain it first.

Hey @mikedlr, that sounds cool. However, for it to work it would require the code to gather your account id, current region and somehow find the service the resource belongs to. It would have to know all the corner cases regarding ARNs, like the fact that S3 buckets don't specify the region, etc. Would you input all those properties? or would you expect arnparse to make the appropriate boto3 calls to gather this information?

I think I'd normally provide the missing parameters since it's impossible to be sure what parameters were actually used during a call that created the JSON but I would provide a function like guess_aws_params which would use normal AWS credential resolution (arguments / environment / default credentials in .aws from environemnt / default in .aws) to fill in the rest on demand.  Maybe a guess=true parmeter when creating the ARN object?
In my use case I would get an ARN for a cloudformation stack and then parse it and then call the cloudformation API call, so I would already know (from the original ARN) all the parameters I need.  Also I will be scanning over all regions so I couldn't rely on the guess function.

@laurrentt further on the comment on the need to understand the structure of ARNs - that's precisely the reason I went looking for a library.  My most important need is to be able to match against other ARNs so I'll need a consistent way of understanding and generating them correctly.

I was skeptical at first, mainly because I didn't get that the actual gathering of information would be left outside arnparse. But I like the idea and I think it's compatible with the goal of this library.


Grey images - some posts

Sometimes when I scroll some images appear grey and don’t seem to load. Tapping image doesn’t load them either however tapping Open in Safari the posts will load fine.
I am not sure if there is any similarities with posts as it’s different users plus Force quitting sunlit seems to load some previous grey images and some more images appear in the total timeline but some different ones are grey normally near the bottom but again sometimes a photo will load after a few grey images so it’s not just the bottom of the list.
Sometimes the timeline is longer and loads more
Posts and sometimes it stops earlier too.
iPhone X
iOS 13.6

Thanks. Not sure if this is the same thing you're seeing, but I've noticed on some screens (like Discover) that the photos won't load until scrolling.

@adamprocter Let us know if this is still happening in the latest beta. Should be much better.

Seems much improved with 337. Feel free to close

Great, thanks.


Issue/3872 fix doc link check (#3873)

Description
Update dock link check ignore list to ignore all docker ips.
closes #3872
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requaested feature)
[x] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso4 in 53801ad4073ce8056fec76fbc3620ce51f312985


DataFrame has too many partitions after performing concat operation

System information

Modin version (modin.__version__): d81b118104b39be3a14eff4658e96fcc64036e26
Code we can use to reproduce:

import modin.pandas as pd

def test3():
    dfs = [None]*1000
    for i in range(1000):
        dfs[i] = pd.DataFrame({"A": [1,2,3,4,5]*3, "B": [6,7,8,9,10]*3})
    result = pd.concat(dfs)
    result.to_csv("result.csv")

test3()

Describe the problem

This leads to the creation of many more workers than the available cores and can be a reason for a freeze or significant slowdown in code execution (the problem was found on Ray, but most likely it is present for Dask too).
We need to develop a process for repartitioning if necessary or maybe it already exists but how it works?
Source code / logs

An example of one of the many warnings:
...
2021-11-12 12:23:07,131 WARNING worker.py:1215 -- WARNING: 136 PYTHON worker processes have been started on node: 4008fd6949b9629e54217435b571113a31e1fbd35e93cfbf575fb461 with address: <IP_ADDRESS>. This could be a result of using a large number of actors, or due to tasks blocked in ray.get() calls (see https://github.com/ray-project/ray/issues/3644 for some discussion of workarounds).
...

Also related #3657

#3886 will need more time to address all of the comments, it will likely be in the release after this

@devin-petersohn what about temp solution?

The important function is to_csv. I can call the rebalance right before it (not after concat function). This should be acceptable for a major release.

I don't even have to create a public API. Just do rebalancing directly in to_csv function. This will be enough for the customer to start. @devin-petersohn what do you think? This approach can be quickly implemented from what I have already written.

@anmyachev Current version works quite well for to_csv. Is it possible for this user to use an experimental branch?

@devin-petersohn I'm not sure, but https://github.com/modin-project/modin/pull/4028 looks easily replaceable with your implementation. I just need to change the test and clear the commit history.

@anmyachev Current version (in #3886) works quite well for to_csv. Is it possible for this user to use an experimental branch?

@devin-petersohn unfortunately, the user only uses conda packages, without modifying the source codes. So this option is not suitable. What about #4028? This approach shouldn't slow down the execution of to_csv function in any case.


TypeError: Cannot call method 'send' of null

When launching a bot, (code below), this happens. Any ideas on how to fix this?

var PlugAPI = require("plugapi");

var bot = new PlugAPI({
    email:"***",
    password:"***"
});

bot.connect("***");
bot.sendChat("/me ZudoBot enabled!")

I see the issue, next version will fix it

Issue should be resolved now, please let me know if it isn't :+1:


Wrong redirection to the GUI

In GitLab by @pabretonniere on Apr 20, 2022, 18:18
Hi @dbeltrankyl (as I can't tag Wilmer, I imagine you will be taking care of this?).
While looking at https://earth.bsc.es/shiny/cp_shiny-figure/ with @angel5331238, we noticed something that seems to be new.
We have a link to http://bscesweb04.bsc.es/autosubmitapp/experiment/a32p in this page, but when clicking on it, it redirects us to https://earth.bsc.es/autosubmitapp/?user=pbretonn. But it seems that this redirection is not always done. I've tried to open several times https://earth.bsc.es/autosubmitapp/experiment/a32p/ pasting it in my browser and sometimes it opens, sometimes I'm redirected to https://earth.bsc.es/autosubmitapp/?user=pbretonn.
When you have time (not very urgent), can you have a look please?
Thanks!

In GitLab by @ jberlin on May 4, 2022, 17:11
hi @pabretonniere, I see that you are using the name of the webserver when you should use the alias:
url in shiny app:
http://bscesweb04.bsc.es/autosubmitapp/experiment/a32p

correct url
https://earth.bsc.es/autosubmitapp/experiment/a32p

anyways, if you are not logged in, it shuld route to the url requested short after, but for instance if you are logged in and click on the second url it will load the experiment

In GitLab by @dbeltrankyl on Apr 21, 2022, 10:18
Hello @pabretonniere ,
I'm not currently in charge of the GUI. I think the appropriate person to tag is @ jberlin. FYI @mcastril

In GitLab by @pabretonniere on May 4, 2022, 14:41
Thanks @ jberlin! Note that I tried the link both with and without having
logged in previously in the GUI and it seems to have a similar behavior.
...
On 04/05/2022 12:45, jberlin (@ jberlin) wrote:

GitLab
jberlin https://earth.bsc.es/gitlab/jberlin commented
https://earth.bsc.es/gitlab/es/autosubmitreact/-/issues/13#note_167903:
I think the problem is when you need to authenticate (login.js) is
missing the original url, I will try to get it fixed this week
—
Reply to this email directly or view it on GitLab
https://earth.bsc.es/gitlab/es/autosubmitreact/-/issues/13#note_167903.
You're receiving this email because of your account on earth.bsc.es.
If you'd like to receive fewer emails, you can unsubscribe
https://earth.bsc.es/gitlab/-/sent_notifications/REDACTED/unsubscribe
from this thread or adjust your notification settings.

--
Pierre-Antoine Bretonnière
Computational Earth Sciences Group (CES) | Earth Sciences Department
Barcelona Supercomputing Center - Centro Nacional de Supercomputación
Address: Plaça Eusebi Güell, 1-3, 08034 Barcelona
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>>
Phone: +34 934 137 725 <tel:+34 934 137 725>

In GitLab by @ jberlin on May 4, 2022, 12:45
I think the problem is when you need to authenticate (login.js) is missing the original url, I will try to get it fixed this week

In GitLab by @ jberlin on Apr 21, 2022, 10:48
I will have a look

In GitLab by @pabretonniere on May 5, 2022, 09:45
Hi @ jberlin
Indeed, now it works! Thank you!


AWSSDK.S3 ListObjectsV2 API returns Size incorrectly every other request for some subset of files +/- 2 bytes

Describe the bug
There appears to be some prior bugs around this (see: https://github.com/aws/aws-cli/issues/5216), with Amazon stating the issue is caused by third parties using the old ListObjects API.  In this case, we are using ListObjectsV2 with GetObjectAsync and us-east-1 region for all requests.
Every other request, the Size is logged as either 84676 or 84674.
This is coming through a third party bucket we are licensed to data on, but I thought to log a formal public issue here for people confused by this as much as I am, and link the third party to this ticket so they can reference it in a AWS support request.  This problem has been happening since 2023-03-29 04:30:11.6484437 UTC, but we only caught it today.
I'm also guessing this has caused extra data charges, given this ~84674-84676 byte file has been re-downloaded 57,265 times and counting.
Expected Behavior
S3 Object Size should not be spurious across ListObjectsV2 requests.
Current Behavior
S3 Object Size is not consistent across multiple requests.
Reproduction Steps
Example C# code - you will need to populate AwsClientConfiguration with the appropriate Region, AccessKey and SecretKey, otherwise this is a http://sscce.org/ example
using Amazon.S3;
    public class AwsClientConfiguration
    {
        public string AccessKey { get; set; }
        public string SecretKey { get; set; }
        public string Region { get; set; }
    }
    public interface IS3ClientFactory
    {
        IAmazonS3 Make();
    }
// helper code to make mocking an AmazonS3Client easier.
public class S3ClientFactory : IS3ClientFactory
{
    private readonly AwsClientConfiguraiton _awsClientConfig;
    private readonly ILog _log;

    public S3ClientFactory(AwsClientConfiguration awsClientConfiguration, ILog log)
    {
        _awsClientConfig = awsClientConfiguration ?? throw new ArgumentNullException(nameof(awsClientConfiguration));
        _log = log ?? throw new ArgumentNullException(nameof(log));
    }
    public IAmazonS3 Make()
    {
        var awsClientConfig = _awsClientConfig;
        if (awsClientConfig?.Region == null)
            throw new ArgumentNullException(nameof(AwsClientConfiguration.Region));

        // Unfortunately, the AmazonS3Client takes a concrete RegionEndpoint class instance, and does not accept an IRegionEndpoint interface.
        // Note, some Amazon code samples online still use Amazon.Internal.RegionEndpointProviderV2.GetBySystemName
        // but the API was updated to Amazon.RegionEndpoint.GetBySystemName
        var region = RegionEndpoint.GetBySystemName("us-east-1");
        AmazonS3Client client;
        if (!string.IsNullOrWhiteSpace(awsClientConfig.AccessKey))
        {
            var awsCredentials = new BasicAWSCredentials(awsClientConfig.AccessKey, awsClientConfig.SecretKey);
            client = new AmazonS3Client(awsCredentials, region);
        }
        else
        {
            throw new NotImplementedException("Only AccessKey/SecretKey is supported");
        }

        return client;
    }

// helper code to make mocking an AmazonS3Client easier.
var s3ClientFactory = new S3ClientFactory();

using (var client = s3ClientFactory.Make())
{
    var listRequest = new ListObjectsV2Request
    {
        BucketName = "bucketName",
        MaxKeys = 1000,
        Prefix = "prefix"
    };

    ListObjectsV2Response response;
    do
    {
        try
        {
            response = await client.ListObjectsV2Async(listRequest);
        }
        catch (Exception ex)
        {
            _log.Error(ex.Message, ex);
            throw;
        }

        var message = $"Found {response.S3Objects.Count} items in bucket {listRequest.BucketName} with prefix {listRequest.Prefix}.";

        var s3Files = response.S3Objects;

        var filesToDownload = s3Files.ToList();

        foreach (var fileToDownload in filesToDownload)
        {
            var s3File = fileToDownload;
           _log.Info($"{nameof(s3File.Size)}=[{s3File.Size}]");
// Implementation not provided for ProcessEntry, as it is irrelevant to repro, so commenting out
            //await ProcessEntry(client, s3File, System.IO.Path.GetTempPath());
        }

        listRequest.ContinuationToken = response.NextContinuationToken;
    } while (response.IsTruncated);
}

Possible Solution
No idea - requires S3 Amazon Engineers to investigate.
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.S3 <IP_ADDRESS>
Targeted .NET Platform
.NET 6
Operating System and version
Microsoft Windows Server 2016 Datacenter

Upgrading to the latest AWSSDK.S3 seems to fix the issue in local testing.  I do not know why, as I don't see any release notes or Pull Request history indicating issues with these two methods.  I will deploy the fix to PROD tonight, and if it seems to go away over the next 3 days, will close this request.

The problem is still happening on the latest AWSSDK.S3 nuget package. https://www.nuget.org/packages/AWSSDK.S3/3.7.308.5
Strange that I could not reproduce the problem locally.
Locally, i have:
dotnet --list-runtimes

which returns:
Microsoft.NETCore.App 6.0.29 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

On different servers, I have:
Microsoft.AspNetCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]

Microsoft.AspNetCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]

I've asked our IT department to patch the .NET runtime on these servers to rule out the runtime as the root cause of this bug.

We upgraded the .NET Runtime to 6.0.31 and the bug still exists.  Please do the needful and revert back if you need any confidential information with guidance on how to confidentially provide the exact bucket and prefix values.

Closing. Sorry - this is a bug on our end.  This file had more than one version, and the previous version had more bytes than the previous version. When writing the new version, we were using the old file handle, which left 2 trailing bytes from the end of the previous file.  Moving to use GetObjectResponse.WriteResponseStreamToFileAsync(string filePath, bool append, CancellationToken cancellationToken) helper method solves the issue.


v0.1.2

Deploy to Kovan
Update package details
Fix all npm audit vulnerabilities

Pull Request Test Coverage Report for Build 4926

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 4901:
0.0%

Covered Lines:
926

Relevant Lines:
926

💛  - Coveralls


Add quote search functionality

Add quote search functionality

@joswin18 Hi, I hope you are doing well. Would you like to assign it to me? I would love to try implementing it.

@mohsenkhosroanjam, I'll assign it to you. Let me know if you have any questions.

thank you for participating, This issue has been resolved. In the meantime, feel free to check out other open issues


Clarification on Egress IP

Cloudflare Gateway is not limited to WARP users

👋Hi there, we’re going to close this PR. All PRs prior to August 12 need to be reworked due to the platform migration to Astro.


Publishing typings

Hi! I've been using wait-queue but currently it is not possible to use class generics (e.g. new WaitQueue<string>()). I saw that version 1.1.4 is not yet published in npm.
Is it possible to include a reference to the typing definitions file in package.json so the typings will be accessible, by adding:
 "typings": "dist/index.d.ts",

but also update the npm package with the latest changes?
Thanks!

@dinikolop Thanks for the reminder, I have published v1.1.4

Thank you!


Simulator does not support coreir.wire

This is essentially a nop but can be useful in designs.

Changed in PR 432

Merged!


Owncloud support

It would be great if owncloud would be natively supported.

When we get round to doing WebDav in #580 - this should make Owncloud work

An OwnCloud App that makes calls directly to rclone might be a simpler way to approach this - I'm surprised one doesn't exist already!
Useful things:

OwnCloud Developer Manual - External Storage Backends
Adapter could parse JSON Output #1063 or use nasty regex like mmozeiko/RcloneBrowser does

Here is the first version of the webdav remote
https://beta.rclone.org/v1.38-017-g7f3dc9b5/ (uploaded in 15-30 mins)
I've tested it with owncloud and nextcloud but I haven't tested it with any other webdav servers though.
Here are the docs
Please report issues with this in #580.


Fix issue with relay activation

Previous updates broke the ability for Kegbot to activate relays connected to the Arduino/Kegboard. This fixes that issue by updating the listener on the Kegboard Daemon and publishing a different event from the Manager App.

Meant to Pull this into my Fork, but feel free to review and merge in if it looks good.


Remove the Add Survey and Add Poll button from Survey and Poll admin pages

Surveys and Polls will be added directly from the Pages section. The "Add Survey" and "Add Poll" buttons on the Survey and Poll pages which has radio buttons for the parent section is not very user friendly and not needed so these buttons should be removed.

@ChrisMarsh82 this isn't done yet.

Removing the "Add" button from Quizzes as well


[IA-4659] Fix UI models representing Leo data

Originally when these models were written, there was a mismatch between Typescript and Leonardo's definition of Optional. Leo serializes fields it labels as optional into null. Typescript expects a field to be omitted if it's optional, and provides no protection against something you state as myVar?: string ending up as null.
As such, the fields here were not correctly typed.

You might need to do some surgery on how the test data is being generated though 😬


Add necessary parameters to errors in smart contracts

Currently some errors in BoltValidatorsV1 like ValidatorDoesNotExist or ValidatorAlreadyExists do not take any argument which makes hard to debug which validator public key is actually not correct, especially in batch calls like getProposerStatuses
Ideally all similar errors should be parametric.

Closed by #336


EPIC: Remove Nav A-Z and List panel components from frontend

What
We need to remove nav A-Z and list panel components from NHS.UK frontend and instead use existing components and (possibly new) styles and utility classes.
Why
So we will reduce the duplication of code – making the frontend easier to maintain and increase its performance.
Related to
https://github.com/nhsuk/nhsuk-service-manual/issues/1157
Done when
We have:

[x] created a proof of concept
[ ] tested the code in a live service
[ ] made notes on upgrade path
[x] removed the code from the frontend (with changelog notes)
[x] communicated the changes to our users

I remember Adam saying he was waiting on #880 so they both could be released at the same time. I know this EPIC #1186 & last FE ticket is in review #740 is a breaking change.
Is there much work to do on this one @DomBaker as #880 is due to be released Monday can this be released together? @sarawilcox @henocookie

@DomBaker or @truecroft2 do you know if the changes have been tested on another live service? I know we had an issue with an old Nav A-Z implementation on the A to Z of NHS health writing page in the Content style guide. Interested to know if other services that have upgraded to use the latest nhsuk-frotnend have similar issues.

Hi Henry, as long as other services are using the new Nav A-Z code and have updated to the latest version of the frontend library they shouldn't have any issues, now that it is no longer a component and just straight HTML it shouldn't display differently on other services unless they have conflicting custom CSS in their project


[Feature] Ensuring the safety of stopSpan

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
In the agent, createSpan and stopSpan should appear in pairs. In fact, in many plugins, such as HttpClient, dubbo, etc., createSpan is in beforeMethod, while stopSpan is in afterMethod. There is a problem here. If createSpan fails, or an exception occurs in some code before createSpan, causing createSpan not to be executed, then stopping Span in afterMethod will be a dangerous action. This will cause the parentSpan to be stopped prematurely. This will lead to TraceContext being destroyed prematurely, causing chain tracking interruption, and will also cause NPE when the upper plugin calls stopSpan.
Therefore, a mechanism is needed to ensure that in this scenario, stopSpan can only stop the span created in the current Interceptor. Since beforeMethod and afterMethod must execute in the same Interceptor instance, a instance variable can be used to solve this problem. My idea is to create a base class and provide createSpan and stopSpan methods in the base class. Interceptors that need to ensure stopping the corresponding span can inherit this base class, and then call the base class methods to replace ContextManager's createSpan and stopSpan methods.
Use case

Skywalking Agent 8.14.0, HttpClient-4.x-plugin
SpringBoot Application, Controller requestMapping is "/test_http_client", server port=8080
The logic in the Controller uses HttpClient to request another URL : "http://localhost:8080:8080/test" (This URL is invalid and contains 2 port)
start application, request URL: "http://localhost:8080/test_http_client"

Based on the current situation:
in beforeMethod, new URL() (this operation is before createSpan) will throw an error because the URL is invalid. This causes the span to not be created, but afterMethod's stopSpan will still be executed. At this time, the span stopped is the one created by Tomcat. And because there is only one span left, after this span is stopped, the TracingContext object in ThreadLocal will also be deleted. Therefore, calling stopSpan in Tomcat plugin's afterMethod will produce an NPE. If there are other RPC calls after the HttpClient call, those RPC call chains can no longer be linked with this chain either.
Based on my base class design:
When calling stopSpan, it will check if the span variable in the current Interceptor instance has a value. If not, it will be ignored. If there is a value, the stop operation will be performed.
Related issues
No response
Are you willing to submit a pull request to fix this on your own?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

If you think more carefully, you will see you are wrong.
You can never check span available when stop, because the active span could be another plugin created.
V2 interceptor has provided you context to propagate span between before and after. But still stop will fail if the active span is not this span.
Think again. SkyWalking had been there for 8+ years, this had been proposed and rejected many times, because this proposal is wrong.

If you think more carefully, you will see you are wrong. You can never check span available when stop, because the active span could be another plugin created.
V2 interceptor has provided you context to propagate span between before and after. But still stop will fail if the active span is not this span.
Think again. SkyWalking had been there for 8+ years, this had been proposed and rejected many times, because this proposal is wrong.

I'm very curious about the "active span could be another plugin created" scenario you mentioned - could you give an example? I'm thinking this scenario should be special, fixed, and determined by how a plugin is designed, right?

For a plugin like HttpClient, it must create and stop span in the same interceptor, it's impossible for the stopped span to be created by another plugin. So in this case, if we want to avoid errors when stopping spans, don't we need to do something to ensure it? The question is just whether the scope of this refactoring is only for this plugin or making it more general.

I really am tired to explain this scenario. Really not interesting.
You are only describing a very simple case of one plugin being up and running, how about

Two plugins having a bug at the same time?
One plugin closes span twice but only creates one?
A plugin creates a span, but B plugin fails to create a span, but tries to close A's span?
A needs to create a span in two interceptors?

I can see, many many common cases you didn't think through. Anyway, I most likely would not follow this discussion anymore. I have at least been asked dozens of times about this in 8 years. Forgive me don't want to repeat it again.


Build Flannel on Debian Issue

Hello,
I am currently building a Kubernetes cluster on dedicated Linux machines for our company to run our Dev/Uat/Prod environments.
The architecture consists of the following for each host in the cluster of servers:

Debian Jessie amd64 OS

ETCD running on host OS

Flannel running on host OS

Docker running on host OS

Currently there seem to be 2 methods to get flannel running on Debian based on the github documentation:

Flannel service running on host OS

Flannel running on Docker container with ETCD

I am able to run Flannel in a docker container however this is not idea for the setup I am going for - I do not want to run container management services within a container ;) When I try to build the Flannel service on the Debian host (using make, ./build or make dist/flanneld) I get the following:
root@cch114:/home/tyrone/flannel# make
go build -o dist/flanneld \
-ldflags "-X github.com/coreos/flannel/version.Version=v0.6.1-47-gb5ea6d3"
main.go:37:2: cannot find package "github.com/coreos/flannel/backend/alloc" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/alloc(from $GOROOT)
($GOPATH not set)
main.go:38:2: cannot find package "github.com/coreos/flannel/backend/awsvpc" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/awsvpc(from $GOROOT)
($GOPATH not set)
main.go:39:2: cannot find package "github.com/coreos/flannel/backend/gce" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/gce(from $GOROOT)
($GOPATH not set)
main.go:40:2: cannot find package "github.com/coreos/flannel/backend/hostgw" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/hostgw(from $GOROOT)
($GOPATH not set)
main.go:41:2: cannot find package "github.com/coreos/flannel/backend/udp" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/udp(from $GOROOT)
($GOPATH not set)
main.go:42:2: cannot find package "github.com/coreos/flannel/backend/vxlan" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/backend/vxlan(from $GOROOT)
($GOPATH not set)
main.go:30:2: cannot find package "github.com/coreos/flannel/network" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/network(from $GOROOT)
($GOPATH not set)
main.go:31:2: cannot find package "github.com/coreos/flannel/remote" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/remote(from $GOROOT)
($GOPATH not set)
main.go:32:2: cannot find package "github.com/coreos/flannel/subnet" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/subnet(from $GOROOT)
($GOPATH not set)
main.go:33:2: cannot find package "github.com/coreos/flannel/subnet/kube" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/subnet/kube(from $GOROOT)
($GOPATH not set)
main.go:34:2: cannot find package "github.com/coreos/flannel/version" in any of:
/usr/lib/go/src/pkg/github.com/coreos/flannel/version(from $GOROOT)
($GOPATH not set)
main.go:26:2: cannot find package "github.com/coreos/pkg/flagutil" in any of:
/usr/lib/go/src/pkg/github.com/coreos/pkg/flagutil(from $GOROOT)
($GOPATH not set)
main.go:27:2: cannot find package "github.com/golang/glog" in any of:
/usr/lib/go/src/pkg/github.com/golang/glog(from $GOROOT)
($GOPATH not set)
main.go:28:2: cannot find package "golang.org/x/net/context" in any of:
/usr/lib/go/src/pkg/golang.org/x/net/context(from $GOROOT)
($GOPATH not set)
Makefile:38: recipe for target 'dist/flanneld' failed

Any feedback on the following?
Thanks

You should specify the GOPATH. If you already have one then you can git clone so that
flannel is at $GOAPTH/src/github.com/coreos/flannel
Run go  get ./... after cd flannel to install all the required packages

Well you can skip go get ./... as the project already has dependencies in the vendor folder which will be made available to go compiler >= 1.6 :)

https://blog.gopheracademy.com/advent-2015/vendor-folder/

Hi, I replied with a solution that worked for me in #552
Basically:
cd ~
export GOPATH=$(pwd)
mkdir -p src/github.com/coreos
cd src/github.com/coreos
git clone https://github.com/coreos/flannel.git
cd flannel
make dist/flanneld

Hi, please re-open if you're still having problems


Added message deleted moderation

A little help for when people post bad things and delete them before moderators can see it.

This isn't working as intended.


impl for GitHub action

resolve #73
Signed-off-by: Sam Yuan<EMAIL_ADDRESS>

@davidkhala , I try to add github action and replace azp for java gm. could you please take a look at this pr?


HTTP 401 ERROR

my command:
pip install torch==1.7.1+cu102 -f https://nelsonliu.me/files/pytorch/whl/torch_stable.html --default-timeout=100

log:
Looking in indexes: http://mirrors.jd.com/pypi/simple
Looking in links: https://nelsonliu.me/files/pytorch/whl/torch_stable.html
Collecting torch==1.7.1+cu102
  WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ReadTimeoutError("HTTPSConnectionPool(host='objects.githubusercontent.com', port=443): Read timed out. (read timeout=100.0)",)': /github-production-release-asset-2e65be/282146602/c734bf80-7da9-11eb-98a9-f91364aa0d2f?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211109%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211109T073838Z&X-Amz-Expires=300&X-Amz-Signature=aeaf6a97a9fb783dc990d0bb8cadc90fa9ae6e574dc3f966f73bd22732bc79d1&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=282146602&response-content-disposition=attachment%3B%20filename%3Dtorch-1.7.1%2Bcu102-cp36-cp36m-linux_x86_64.whl&response-content-type=application%2Foctet-stream
  WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ReadTimeoutError("HTTPSConnectionPool(host='objects.githubusercontent.com', port=443): Read timed out. (read timeout=100.0)",)': /github-production-release-asset-2e65be/282146602/c734bf80-7da9-11eb-98a9-f91364aa0d2f?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211109%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211109T073838Z&X-Amz-Expires=300&X-Amz-Signature=aeaf6a97a9fb783dc990d0bb8cadc90fa9ae6e574dc3f966f73bd22732bc79d1&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=282146602&response-content-disposition=attachment%3B%20filename%3Dtorch-1.7.1%2Bcu102-cp36-cp36m-linux_x86_64.whl&response-content-type=application%2Foctet-stream

my command:
pip install torch==1.7.1+cu102 -f https://nelsonliu.me/files/pytorch/whl/torch_stable.html --default-timeout=100

log:
Looking in indexes: http://mirrors.jd.com/pypi/simple
Looking in links: https://nelsonliu.me/files/pytorch/whl/torch_stable.html
Collecting torch==1.7.1+cu102
  WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ReadTimeoutError("HTTPSConnectionPool(host='objects.githubusercontent.com', port=443): Read timed out. (read timeout=100.0)",)': /github-production-release-asset-2e65be/282146602/c734bf80-7da9-11eb-98a9-f91364aa0d2f?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211109%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211109T073838Z&X-Amz-Expires=300&X-Amz-Signature=aeaf6a97a9fb783dc990d0bb8cadc90fa9ae6e574dc3f966f73bd22732bc79d1&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=282146602&response-content-disposition=attachment%3B%20filename%3Dtorch-1.7.1%2Bcu102-cp36-cp36m-linux_x86_64.whl&response-content-type=application%2Foctet-stream
  WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ReadTimeoutError("HTTPSConnectionPool(host='objects.githubusercontent.com', port=443): Read timed out. (read timeout=100.0)",)': /github-production-release-asset-2e65be/282146602/c734bf80-7da9-11eb-98a9-f91364aa0d2f?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211109%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211109T073838Z&X-Amz-Expires=300&X-Amz-Signature=aeaf6a97a9fb783dc990d0bb8cadc90fa9ae6e574dc3f966f73bd22732bc79d1&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=282146602&response-content-disposition=attachment%3B%20filename%3Dtorch-1.7.1%2Bcu102-cp36-cp36m-linux_x86_64.whl&response-content-type=application%2Foctet-stream

after searching at internet,  it seems the website server error。

Hmm bizarre, https://nelsonliu.me/files/pytorch/whl/torch_stable.html is up and github status isn't reporting any issues https://www.githubstatus.com/ . Can you let me know if this eventually resolves itself?

thanks. it is my network access permission。 finally，i get the whl file successfully~    👍


Prevent PythonAction from modifying task attributes by passing copies

Currently, if a PythonAction calls pop() or otherwise modifies dependencies, targets or changed, these changes are reflected in the task itself. This leads to the dependency or target in question not being added to the doit database properly after execution has finished.

Actually PythonAction being able to modify task attributes is a feature :grinning:
It is even documented: https://github.com/pydoit/doit/blame/master/doc/tasks.rst#L205
We should add unit tests for that...

It is a quite powerful feature.. if you search the mailing list you can find some use cases for that.

Ah sorry I missed that in the docs! It would only take me 10 minutes to invert the unit tests I wrote and test the correct behaviour, if you think that would be helpful?

It would only take me 10 minutes to invert the unit tests I wrote and test the correct behaviour, if you think that would be helpful?

that would be cool. thanks
I was also thinking about a simple example. Like creating a target where the filename includes the timestamp. It is a simple example but show how this feature can be powerful...

Coverage increased (+0.0003%) to 99.788% when pulling 778e49816cc0ef67ab068b10db7ef340786fd684 on rbeagrie:pyactions_cannot_modify_tasks into 773a4a48475e671a215055c32f2a2b47bb72f950 on pydoit:master.

Changes Unknown when pulling 6656d04f2233bbf543345d6a4a3358dc5f4d5b80 on rbeagrie:pyactions_cannot_modify_tasks into ** on pydoit:master**.


Update the contacts section of the User's Guide

Reported by dietmarw on 24 Nov 2009 10:09 UTC
In Modelica.UsersGuide.Contacts the list of library officers needs to be updated (affiliation, etc.) before the release.

Migrated-From: https://trac.modelica.org/Modelica/ticket/244

Comment by dietmarw on 25 Nov 2009 11:53 UTC
Milestone MSL3.1.1 deleted

Comment by otter on 16 Jul 2010 11:27 UTC
Fixed in 0106398a3a1956fb08bef1af27b0018fafc318f1.

Comment by dietmarw on 16 Jul 2010 11:33 UTC
Not quite sure here but shouldn't "Dynasim AB" be changed to "DS Lund"?

Comment by otter on 16 Jul 2010 11:43 UTC
Yes you are right. Replaced "Dynasim AB, Lund" by "Dassault Systemes, Lund"


K.Dᴀᴠɪs: Watch peanutbuttermachinery.com

K.Dᴀᴠɪs requests the watch of the watch_keyword peanutbuttermachinery\.com. See the Metasmoke search here and the Stack Exchange search here.

peanutbuttermachinery.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

!!/approve

Coverage remained the same at 64.657% when pulling eaa46a93284efd138925af7329c186f5d17b7e0a on auto-blacklist-1521078425.2369506 into f37a505c9e75cc56cacb8ce3f58a9f06e41e4ffd on master.


Cant install big desk plugin as plugin-descriptor.properties in plugin zip is missing

Hi Team,
I am trying to install big desk plugin to evaluate for my elastic search, and below is the error I am getting
user@localhost:/usr/share/elasticsearch/bin$ sudo ./plugin install lukas-vlcek/bigdesk/2.5.0
-> Installing lukas-vlcek/bigdesk/2.5.0...
Trying https://download.elastic.co/lukas-vlcek/bigdesk/bigdesk-2.5.0.zip ...
Trying https://search.maven.org/remotecontent?filepath=lukas-vlcek/bigdesk/2.5.0/bigdesk-2.5.0.zip ...
Trying https://oss.sonatype.org/service/local/repositories/releases/content/lukas-vlcek/bigdesk/2.5.0/bigdesk-2.5.0.zip ...
Trying https://github.com/lukas-vlcek/bigdesk/archive/2.5.0.zip ...
Trying https://github.com/lukas-vlcek/bigdesk/archive/master.zip ...
Downloading ....................................................................................................................................................................................................................................DONE
Verifying https://github.com/lukas-vlcek/bigdesk/archive/master.zip checksums if available ...
NOTE: Unable to verify checksum for downloaded plugin (unable to find .sha1 or .md5 file to verify)
ERROR: Could not find plugin descriptor 'plugin-descriptor.properties' in plugin zip
Please guide me on how to fix this issue.

same here
#./bin/plugin install lukas-vlcek/bigdesk/2.4.0
-> Installing lukas-vlcek/bigdesk/2.4.0...
Trying https://download.elastic.co/lukas-vlcek/bigdesk/bigdesk-2.4.0.zip ...
Trying https://search.maven.org/remotecontent?filepath=lukas-vlcek/bigdesk/2.4.0/bigdesk-2.4.0.zip ...
Trying https://oss.sonatype.org/service/local/repositories/releases/content/lukas-vlcek/bigdesk/2.4.0/bigdesk-2.4.0.zip ...
Trying https://github.com/lukas-vlcek/bigdesk/archive/2.4.0.zip ...
Trying https://github.com/lukas-vlcek/bigdesk/archive/master.zip ...
Downloading .........................................................................................................................................................................................................................................................DONE
Verifying https://github.com/lukas-vlcek/bigdesk/archive/master.zip checksums if available ...
NOTE: Unable to verify checksum for downloaded plugin (unable to find .sha1 or .md5 file to verify)
ERROR: Could not find plugin descriptor 'plugin-descriptor.properties' in plugin zip

elasticsearch version:
version: {
number: "2.3.1",
build_hash: "bd980929010aef404e7cb0843e61d0665269fc39",
build_timestamp: "2016-04-04T12:25:05Z",
build_snapshot: false,
lucene_version: "5.5.0"
},```

+1

Same Problem:
{
"name" : "Dust",
"cluster_name" : "elasticsearch",
"version" : {
"number" : "2.3.3",
"build_hash" : "218bdf10790eef486ff2c41a3df5cfa32dadcfde",
"build_timestamp" : "2016-05-17T15:40:04Z",
"build_snapshot" : false,
"lucene_version" : "5.5.0"
},
"tagline" : "You Know, for Search"
}

+1

+1.
ES 2.3.0 here. AFAIK, the properties file is mandatory in 2.3.0 onwards. :S

+1 I am using ES 2.3.3. Same error as @narendraalla

+1, on the new elasticsearch 2.4.0

+1

any solution?

+1 on 2.4.1
any solution?

Hi Team any update on this issue?

+1

I have modified bigdesk code to be compatible with elasticsearch 2.x
https://github.com/nishantsaini/bigdesk
Hope this helps

hard to install and make me confuse

I am archiving this repository now. Thanks for contributions.


WASM Supports?

hi,team:
Did anyone tested it on wasm platform?

It works, although it needs a patch because the system expects to read from a real filesystem using standard commands. I guess the "better" way would be to have some virtual filesystem approach available.

It works, although it needs a patch because the system expects to read from a real filesystem using standard commands. I guess the "better" way would be to have some virtual filesystem approach available.

yes. I think we can point fs operator outside, somethings like sfizz_set_read_func() sfizz_set_write_func(). then we can use custom fs operation function impl them.

Yes. It's a more involved piece of work though, and I don't have so much expertise in how such interface should work with Web technologies. I do have some experience in how it could interface with Android walled garden and base myself on that.
I most probably won't have the time to accomplish this soon, but if you feel up to the task I can offer some guidance.

we can still support old style blocking calls in wasm, we just need to load entire sfz as resource.
like the stuff in https://minicardinal.kx.studio/ which loads many different files from the filesystem, but all works here seamlessly because the resources are loaded before the application runs so by the time the wasm is active any blocking filesystem call works the same as in a regular machine.

and regarding threads and sync, things seem to have evolve quite well in that regard.
both firefox and chrome support the needed bits for pthreads in emscripten, but still requires a secure context (https page) with specific server-side headers set.
the main issue for wasm at the moment is that MIDI can only be done on the main thread, even if we use audio worklets to handle audio outside the main thread.
this has a very high chance of jitter and makes it unsuitable for a pro-audio usage.
if we ignore live midi handling, then perhaps it is a good enough of a platform to target.

@falkTX thanks for these tips, I have very little knowledge about web technologies and just hacked around the (now non-working) wasm demo in between jobs so I'm just a big newbie. The source for the minicardinal site is available somewhere?
The midi information is interesting. At least for one of my desired use cases (drum backing tracks) latency is less of an issue. To demo/try SFZ files, it should also be okay-ish.

and regarding threads and sync, things seem to have evolve quite well in that regard. both firefox and chrome support the needed bits for pthreads in emscripten, but still requires a secure context (https page) with specific server-side headers set.
the main issue for wasm at the moment is that MIDI can only be done on the main thread, even if we use audio worklets to handle audio outside the main thread. this has a very high chance of jitter and makes it unsuitable for a pro-audio usage. if we ignore live midi handling, then perhaps it is a good enough of a platform to target.

In fact, I am working on the latest JUCE migrate to WASM, the audio device callback is works with openal. So, I will trying make the sfz player as JUCE Plugin for play first.

you dont need a library to abstract those, web audio via javascript is not too complex.
though for now I only used script processors and not the new worklets, mostly for compatibility reasons.
my WIP implementation is on https://github.com/DISTRHO/DPF/blob/develop/distrho/src/jackbridge/WebBridge.hpp, this is what makes Cardinal on the web do its thing and also https://aida.kx.studio/ as a more recent example (built directly from AIDA-X)


[SMALLFIX] Bumping hadoop client dependency versions

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15388/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15392/
Test PASSed.


The example link doesn't work anymore. Remove sentence

Works for me. Takes to https://cluster.ipfs.io/0.11.0_service.json which opens just fine.

@hsanjuan Ah! I see the issue, I was reading this document on Github, not on the web page. Since the link is relative, it point's to a not existing file in the git.
Sorry for the fuss. :)


Added proper bounds checking for object handle

Windows was requesting information on an invalid object handle that wasn't being checked causing invalid memory access. This fixes it for that instance. Fix should be easily applied to other instances but I would like a better understanding of how MTP works before putting it everywhere.

#4

ty


feat: add wasm light client module

Description
Wasm light client module
This PR adds a new light client to support wasm light client contracts. A chain that would like to use a wasm light client contract would add this module to their app.go. An existing chain will need to add 08-wasm to the allowed clients list. The module will create a wasm vm to run the wasm contracts. New contracts will be submitted through a governance proposal. Once approved, the contract can be used to create any number of clients. Gas consumption for the wasm vm is based off of wasmd's implementation. The bulk of the light client logic is executed in the contract with the module providing the interface and management of the contracts.
IBC spec: https://github.com/cosmos/ibc/tree/main/spec/client/ics-008-wasm-client
Commit Message / Changelog Entry
type: feat: adding 08-wasm light client module

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards and Go style guide.
[x] Wrote unit and integration tests. Unit tests have been written providing code coverage of the interface to a contract and the management of contracts. Currently, a version of the grandpa light client contract is being used with a couple methods stubbed out as they are still in development, but the interface is still fully exercised.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[x] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Moving to https://github.com/cosmos/ibc-go/pull/3486.


refactor: use simple boolean for parity in signature

Motivation
Right now we use Parity enum for parity value in signature: https://github.com/alloy-rs/core/blob/9c53778e4d40a8ac5b7d3e36221efc1d79fec050/crates/primitives/src/signature/parity.rs#L10-L17
Because of it containing EIP-155 related variants it basically captures transaction-related logic which does not belong to primitives and results in inconsistencies and need for dynamic checks. e.g https://github.com/alloy-rs/alloy/pull/1305 https://github.com/alloy-rs/core/issues/705 https://github.com/alloy-rs/alloy/issues/1510
Primitive signature representation in context of Ethereum is (U256,U256,bool), this PR changes Signature to have this structure and leaving specialized parity encoding to types holding signatures and having enough context to perform correct encoding.
At this point in any context we know how parity should be encoded and storing a simple boolean is enough to figure this out. i.e legacy transaction knows its chain_id and can adjust parity accordingly.
API changes in this PR:

serde now defaults to serializing/deserializing parity as "yParity": "0x{0,1}"
rlp now defaults to encoding/decoding parity as boolean
write_rlp_vrs now accepts encodable v value as a parameter, allowing callers to provide arbitrary encoding for parity
decode_rlp_vrs now accepts a decode_parity closure allowing callers to provide arbitrary decoding for parity, possibly capturing additional context (e.g legacy chain_id)

This PR is accompanied with PR in alloy to demonstrate/discuss needed API changes https://github.com/alloy-rs/alloy/pull/1540 https://github.com/alloy-rs/eips/pull/12
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

Extracted new signature type to primitive_sig::PrimitiveSignature


Improve stamps end screen

This PR aims in general to fix the stamps end screen and add the Card-Jitsu ones
Stamps end screen fix
This is how the end screen would always look like:

However, if you didnt earn any new stamps, it should display like this

which is fixed now. In AS3, the stamps from the current session show highlighted, and the stamps from previous session don't, which was not correct before and is now fixed:

Card-Jitsu stamps end screen
At the end of the card-jitsu games, if a player gained a stamp, a special card-jitsu end screen shows up. Here are some screenshots from AS2 and AS3 of how they look like in this PR:

Card-Jitsu Water and Fire fixes
Two minor fixes because they got in the way of testing this PR:

Card-Jitsu water properly handles when a player exits now
Card-JItsu fire stamps for the Coat and Suit were wrong

I have changed it to use redis so it no longer needs database migrating

Instead of using redis or new db column what about using the recent column included with stamps?
Ref: https://github.com/solero/houdini/blob/f6f9f39c22122c91246fb0a9043f36ed829348b7/houdini/data/stamp.py#L56

It could be used, but as far as I know that is not how it worked in the original


[bug] No Auth Provider found for name "oidc"

Describe the bug
When I try to use pxctl kubeclt plugin with OIDC authenticated clusters, I'm facing the below errors --
❯ kubectl-pxc config cluster set --portworx-service-namespace=portworx
Cluster information set

❯ pxctl status
unable to connect to Kubernetes: no Auth Provider found for name "oidc"
Unable to find node that is online

Kubernetes suggests to import k8s.io/client-go/plugin/pkg/client/auth/oidc for OIDC based clusters for authentication.
Version
Additional Information

Thank you for PR #199. I used that as the basis of #200 and took the opportunity to update a lot of the go-mods.
Please try one of the builds below and let me know how it goes.
pxc-v0.35.0-5-g885b872-issue198.windows.amd64.zip
pxc-v0.35.0-5-g885b872-issue198.linux.amd64.tar.gz
pxc-v0.35.0-5-g885b872-issue198.darwin.arm64.tar.gz
pxc-v0.35.0-5-g885b872-issue198.darwin.amd64.tar.gz


Add C++ modules to CODEOWNERS

Description
(Brief description on what this PR is about)
Checklist
Essentials

[ ] Passed code style checking (make lint)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Added myself to everything but would be great if other ppl can add themselves so I can take my name off of some and not get spammed so much :)


What is powders take one the HSTS issue?

The pow repository was recently updated on the matter
https://github.com/basecamp/pow/pull/547
How can we get powder to work again?

Pull request waiting but you can also change POW_DOMAINS variable from ~/.powconfig file. (If you don't have just create it)
export POW_DOMAINS=test

I think it's absolutely correct to follow the pow project with this, thanks for bringing this up @NielsKSchjoedt and thanks for the PR @hamitturkukaya, it's merged and released as version 0.4.0.


Add check if snakemake is installed

The error one get's if snakemake is not installed doesn't indicate that it is related to the missing installation. A upfront check for the installation might be useful

Maybe we can just include a conda environment file; something along the lines of:
name: step

channels:
 - bioconda
 - conda-forge

dependencies:
 - python>=3.8
 - pip
 - pandas
 - snakemake-minimal
 - pip:
      - otoole>=1.0.0

Sounds good, I guess that will also help once the scripts are packaged?

I think this one can be closed, @HauHe!