Runtime Panic Within Node Plugin

We observed a go runtime panic within device local node plugin.

HI Akhil.
This is the latest version. The issue is because of the pvc name is not
state. Please check the parameter value for the pvc name in the call stack.
It is not pointing to an address.
On 28-Jun-2021 at 6:22:11 PM, Akhil Mohan @.***> wrote:

Also, which is the version of device-driver that you are running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openebs/device-localpv/issues/20#issuecomment-869657476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOK42MBLZYSKOOUEWVP343TVBV7XANCNFSM47NPHE4Q
.

--
-----------------------------------------------------------------------------------------
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed. If you have received this email in error, please notify the
system manager. This message contains confidential information and is
intended only for the individual named. If you are not the named addressee,
you should not disseminate, distribute or copy this email. Please notify
the sender immediately by email if you have received this email by mistake
and delete this email from your system. If you are not the intended
recipient, you are notified that disclosing, copying, distributing or
taking any action in reliance on the contents of this information is
strictly prohibited.****
****
Any views or opinions presented in this
email are solely those of the author and do not necessarily represent those
of the organization. Any information on shares, debentures or similar
instruments, recommended product pricing, valuations and the like are for
information purposes only. It is not meant to be an instruction or
recommendation, as the case may be, to buy or to sell securities, products,
services nor an offer to buy or sell securities, products or services
unless specifically stated to be so on behalf of the Flipkart group.
Employees of the Flipkart group of companies are expressly required not to
make defamatory statements and not to infringe or authorise any
infringement of copyright or any other legal right by email communications.
Any such communication is contrary to organizational policy and outside the
scope of the employment of the individual concerned. The organization will
not accept any liability in respect of such communication, and the employee
responsible will be personally liable for any damages or other liability
arising.****
****
Our organization accepts no liability for the
content of this email, or for the consequences of any actions taken on the
basis of the information provided, unless that information is
subsequently confirmed in writing. If you are not the intended recipient,
you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly
prohibited.
-----------------------------------------------------------------------------------------

@akhilerm it happened once as of now and since the pod have been restarted we don't have the state of partition table during that error time interval. As per initial logs of device localpv, we are using plugin at 1cae175 commit id (but i think there is no major functional code changes happened after this commit id). Let me know if you still want get latest output of parted command (doesn't seems to be relevant here). Here is the output of lsblk to understand the disk & partitions created on the node bit better.
NAME                            MAJ:MIN RM           SIZE RO TYPE MOUNTPOINT
sda                               8:0    0<PHONE_NUMBER>0  0 disk
|-sda1                            8:1    0<PHONE_NUMBER>6  0 part /
|-sda14                           8:14   0        3145728  0 part
`-sda15                           8:15   0      130023424  0 part /boot
sdb                               8:16   0<PHONE_NUMBER>7872  0 disk
|-sdb1                            8:17   0        1048576  0 part
`-sdb2                            8:18   0<PHONE_NUMBER>752  0 part /var/lib/kubelet/pods/9c5a96b7-ca07-4d92-9cc8-ad3ca9e2c7ab/volumes/kubernetes.io~csi/pvc-40e33246-3bee-461d-824b-e0ccec803911/mount
sdc                               8:32   0<PHONE_NUMBER>7872  0 disk
|-sdc1                            8:33   0        1048576  0 part
`-sdc2                            8:34   0<PHONE_NUMBER>752  0 part /var/lib/kubelet/pods/9c5a96b7-ca07-4d92-9cc8-ad3ca9e2c7ab/volumes/kubernetes.io~csi/pvc-5d8d56cb-e291-4dfd-81ac-fb664dd5ec75/mount
sdd                               8:48   0<PHONE_NUMBER>280  0 disk
`-sdd1                            8:49   0<PHONE_NUMBER>128  0 part
sde                               8:64   0<PHONE_NUMBER>7872  0 disk
|-sde1                            8:65   0<PHONE_NUMBER>296  0 part
| |-abc                  254:0    0<PHONE_NUMBER>00  0 lvm  /var/lib/abc
| |-xyz                  254:1    0<PHONE_NUMBER>00  0 lvm  /var/lib/xyz
| `-pqr                 254:2    0<PHONE_NUMBER>00  0 lvm  /var/lib/pqr
`-sde2                            8:66   0<PHONE_NUMBER>9424  0 part

This is the latest version. The issue is because of the pvc name is not
state. Please check the parameter value for the pvc name in the call stack.
It is not pointing to an address.

@praveengt the backtrace is suggesting that node daemonset is crashing in wipefsAndCreatePart. It seems like getAllPartsUsed returned an emplty list and we are trying to access the first entry which is not present.  We are getting the vol name and I can see in the backtrace that the length is 36 (see this => 0xc0004ac2a4, 0x24).
@akhilerm, We should be checking if pList  is not empty then only should proceed here https://github.com/openebs/device-localpv/blob/develop/pkg/device/device-util.go#L108.

@akhilerm one thing we need to see that why we got this error
2021-06-07T16:57:10.817343241+05:30 stderr F I0607 16:57:10.817191       1 device-util.go:170] GetPart Error, {DiskName:sda Size:10737418240}
2021-06-07T16:57:10.91741737+05:30 stderr F I0607 16:57:10.917210       1 device-util.go:288] Disk: DiskName not correct

This is leading to emplty partition list and causing the crash.

This is expected, These are debug logs which needs to be removed. This is
what was planned as part of my refactoring, unfortunately not able to get
to it.
On 28-Jun-2021 at 11:10:35 PM, Pawan Prakash Sharma <
@.***> wrote:

@akhilerm https://github.com/akhilerm one thing we need to see that why
we got this error
2021-06-07T16:57:10.817343241+05:30 stderr F I0607 16:57:10.817191       1 device-util.go:170] GetPart Error, {DiskName:sda Size:10737418240}
2021-06-07T16:57:10.91741737+05:30 stderr F I0607 16:57:10.917210       1 device-util.go:288] Disk: DiskName not correct
This is leading to empty partition list and causing the crash.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openebs/device-localpv/issues/20#issuecomment-869881948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOK42L2H3XBGLOD6TV7NDLTVCXZHANCNFSM47NPHE4Q
.

--
-----------------------------------------------------------------------------------------
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed. If you have received this email in error, please notify the
system manager. This message contains confidential information and is
intended only for the individual named. If you are not the named addressee,
you should not disseminate, distribute or copy this email. Please notify
the sender immediately by email if you have received this email by mistake
and delete this email from your system. If you are not the intended
recipient, you are notified that disclosing, copying, distributing or
taking any action in reliance on the contents of this information is
strictly prohibited.****
****
Any views or opinions presented in this
email are solely those of the author and do not necessarily represent those
of the organization. Any information on shares, debentures or similar
instruments, recommended product pricing, valuations and the like are for
information purposes only. It is not meant to be an instruction or
recommendation, as the case may be, to buy or to sell securities, products,
services nor an offer to buy or sell securities, products or services
unless specifically stated to be so on behalf of the Flipkart group.
Employees of the Flipkart group of companies are expressly required not to
make defamatory statements and not to infringe or authorise any
infringement of copyright or any other legal right by email communications.
Any such communication is contrary to organizational policy and outside the
scope of the employment of the individual concerned. The organization will
not accept any liability in respect of such communication, and the employee
responsible will be personally liable for any damages or other liability
arising.****
****
Our organization accepts no liability for the
content of this email, or for the consequences of any actions taken on the
basis of the information provided, unless that information is
subsequently confirmed in writing. If you are not the intended recipient,
you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly
prohibited.
-----------------------------------------------------------------------------------------

wipefsAndCreatePart(0xc000cc8cd5, 0x3, 0x2, 0xc0004ac2a4, 0x24,
0x8a4000, 0xc000d085e0, 0xa, 0x0, 0x1)
func wipefsAndCreatePart(disk string, start uint64, partitionName
string, size uint64, diskMetaName string) error {
Isn't Third parameter should be a string, I dont know how backtrace in
Solang is built, so not sure. Only problem with the theory is that the
same string is accesses multiple times in createVolume (caller od
wipfefs...) so if that is the issue it should panic in the caller
On 28-Jun-2021 at 11:34:07 PM, Praveen Kumar G T @.***>
wrote:

This is expected, These are debug logs which needs to be removed. This is
what was planned as part of my refactoring, unfortunately not able to get
to it.
On 28-Jun-2021 at 11:10:35 PM, Pawan Prakash Sharma <
@.***> wrote:

@akhilerm https://github.com/akhilerm one thing we need to see that
why we got this error
2021-06-07T16:57:10.817343241+05:30 stderr F I0607 16:57:10.817191       1 device-util.go:170] GetPart Error, {DiskName:sda Size:10737418240}
2021-06-07T16:57:10.91741737+05:30 stderr F I0607 16:57:10.917210       1 device-util.go:288] Disk: DiskName not correct
This is leading to empty partition list and causing the crash.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openebs/device-localpv/issues/20#issuecomment-869881948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOK42L2H3XBGLOD6TV7NDLTVCXZHANCNFSM47NPHE4Q
.

--
-----------------------------------------------------------------------------------------
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed. If you have received this email in error, please notify the
system manager. This message contains confidential information and is
intended only for the individual named. If you are not the named addressee,
you should not disseminate, distribute or copy this email. Please notify
the sender immediately by email if you have received this email by mistake
and delete this email from your system. If you are not the intended
recipient, you are notified that disclosing, copying, distributing or
taking any action in reliance on the contents of this information is
strictly prohibited.****
****
Any views or opinions presented in this
email are solely those of the author and do not necessarily represent those
of the organization. Any information on shares, debentures or similar
instruments, recommended product pricing, valuations and the like are for
information purposes only. It is not meant to be an instruction or
recommendation, as the case may be, to buy or to sell securities, products,
services nor an offer to buy or sell securities, products or services
unless specifically stated to be so on behalf of the Flipkart group.
Employees of the Flipkart group of companies are expressly required not to
make defamatory statements and not to infringe or authorise any
infringement of copyright or any other legal right by email communications.
Any such communication is contrary to organizational policy and outside the
scope of the employment of the individual concerned. The organization will
not accept any liability in respect of such communication, and the employee
responsible will be personally liable for any damages or other liability
arising.****
****
Our organization accepts no liability for the
content of this email, or for the consequences of any actions taken on the
basis of the information provided, unless that information is
subsequently confirmed in writing. If you are not the intended recipient,
you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly
prohibited.
-----------------------------------------------------------------------------------------

@iyashu , I asked about the version, because there is a difference in the line numbers mentioned in the logs and commit https://github.com/openebs/device-localpv/commit/1cae175066e562893aeb50f54c4378b6106f1aea
I will add a check to make sure that the pList[] is not empty before proceeding.

Isn't Third parameter should be a string, I dont know how backtrace in
Golang is built,

string is passed as an address + length , so corresponding to string there will be 2 parameters in the stack trace.
Also, the last 0x0, 0x1 in the parameters specify the return value. Since the function returns an error, which is an interface, we have 2 values here.


Fixing README and adding suggestions

Suggestion: So non-CFPB people can use the Rate Checker app, we provide mock data.

+1 or at the very least we could prevent build errors when no api url is provided


analytics: Change Number of Users chart to use realm_active_humans::day.

Previously we showed the total number of users with an active account. This
changes it to show only the number of users that have logged in in the past
two weeks.

Travis fails due to lint.

Oh, I bet I know what's going on here.  The error is in master, not in the branch.
Lint error fixed in faf47fe147de31b1a591b6358ae4a208e7efe987.  @tommyip FYI, this is why we were getting random weird reports.

Merged, thanks @rishig!


Library does not support VBO indexing

I think this is a major feature of the API and should not be zero'ed out as it is currently.

What exactly is missing? GL.Objects.Buffers provides Array_Buffer and Element_Array_Buffer and has Draw_Elements.

Indeed, my mistake. I was looking at the "element array buffer offset" parameter being set to Low_Level.Zero as it was labeled "Indicies" when you imported it.
procedure Draw_Elements (Mode : Connection_Mode; Count : Size; Index_Type : Unsigned_Numeric_Type; Indices : Low_Level.Zero);

Thanks for the quick response BTW : )

Ah yes, that is one of those very strange parts of OpenGL where the function takes a void pointer which actually is an integer offset for whatever reason. So you actually found a missing feature: OpenGLAda does not allow you to specify that offset. I'll rename the issue, this should be fixed.

On a related note, I think you should have a similar package GL.Objects.Vertex_Array for Indices like "GL.Objects.Index_Array" which uses Draw_Elements etc


The session under FAILED status. Please try to restart it

How to solve this?
{
"statusCode": 422,
"message": "The session under FAILED status. Please try to restart it.",
"error": "Unprocessable Entity"
}

Have tried closing the docker, restarting server, everything tried.

downloaded new docker, worked fine

Restarted and downloded previous versions and not working:
docker pull devlikeapro/whatsapp-http-api:latest
docker pull devlikeapro/whatsapp-http-api:2023.7.16

Result:
{
  "statusCode": 422,
  "message": "The session under FAILED status. Please try to restart it.",
  "error": "Unprocessable Entity"
}

And console:
[Nest] 34  - 07/29/2023, 2:33:26 PM     LOG [SessionManager] 'default' - starting session...
[Nest] 34  - 07/29/2023, 2:33:27 PM   ERROR [WhatsappSession - default] Error: Navigation failed because browser has disconnected!

Btw: I am using macos Monterrey + docker desktop "4.19.0 (106363)"


Define a constant instead of duplicating this literal '[class.%s]

[x] Add constant

Define a constant instead of duplicating this literal '[class.%s] the token has expired: %s' 3 times. [+2 locations]sonarlint(python:S1192)

Example
            log.warning(
                '[class.%s] the token has expired: %s',
                __class__.__name__,
                forbidden
            )

LOG_CLASS_HEADER = f"[class.{__class__.__name__}]"
            log.warning(
                '%s the token has expired: %s',
                LOG_CLASS_HEADER,
                forbidden
            )

[x] Or consider tools for automatic class name detection in log format

was fixed through the logging.Filter


just to let you know

Krapht is also working on a new version https://github.com/Krapht/LogisticsPipesRedux
maybe you can work w/ him

@loordgek Id love to wok with him together! I have a lot of ideas for this and allready put quite a lot of work into Models and textures. Maybe Krapht and I can come to a point of working together. We shall see.
If he denies, ill definitly go on with this one.

cool

Krapht is much as before, way to buisy to work on his mod. Both projects have been linked on the original Logistics Pipes Github page.
This issue is hereby closed.


how to upload file by input(type = 'file')?

i want to upload file by panel(use sketch-module-web-view) ,bui when i create uploadBtn with the input(type= 'file'),Nothing happens when I click the button?thx!!!

see #58

Is there any code examples for this workaround?


auth vs config.auth

i don't like that auth is added to the route instead of config.auth as i think it's supposed  to be. breaking change, i know, but i'd still like it changed. could support both for some time

What do you mean?

turns out i meant settings(.auth)
http://aurelia.io/docs.html#/aurelia/router/1.0.0-beta.1.1.3/doc/api/interface/RouteConfig

closed as both authFilter and authPipelinestep are already optional. further tweaks can be implemented in permissions


Updated function name and some new steps.

Issue #, if available:
Description of changes:
Updated some steps according to the new console UI and corrected the function name in the testing steps.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for pointing out these changes in the console UI and the error in the function name.

Thank you for pointing out these changes in the console UI and the error in the function name.


Add tests

Suggestion for an initial test files
current results:
 tests/test_language_extract_go.py ✓✓✓✓✓ss                                                                                100% ██████████

Results (2.92s):
       5 passed
       2 skipped

@Arker123 consider using this as a start for the tests


Add GITHUB_RUN_ID to the traceId components

If a workflow is re-run, it will use the same GITHUB_RUN_NUMBER as the previous run. This would cause gha-buildevents to produce two traces with the exact same trace ID. By adding the run ID to the trace ID, each run should be uniquely identified.
Fixes #16

This doesn't work, the GITHUB_RUN_ID stays the same across re-runs of workflows.


Issues from other team members and users are not synched

I am an admin of an open source Python library (https://github.com/qilimanjaro-tech/qililab).
I recently linked Linear with this repo, but I found out that only the Issues I create are synched with Linear. Issues created by my team members and/or external users are not synched...

Hey @AlbertMitjans, each Linear team member who would like their issues to sync will need to sign up! The app acts on a user's behalf so consent is necessary. Please let me know if there are any issues with that.


named map tutorial example

Files for named maps tutorial:

template used
live example

Both need to be accessible through a link like:  http://cartodb.github.io/cartodb.js/examples/tutorials/named-maps-example.html

LGTM


Ciphersuites are not in order

What feature do you want to see added?
In java there is an option (jdk.tls.client.cipherSuites) to provide list of ciphersuites that should be used during client hello message. But the order of ciphers are decided by JVM. This can be overridden using the method javax.net.ssl.SSLParameters.setUseCipherSuitesOrder(boolean honorOrder). This configuration we can set only during socket connection. Since Socket creation happens at the "syslog-java-client" library, there is no option for end-user to set this up.
It would be helpful to provide option or callback function to resolve this.
Upstream changes
No response

Just wanted to add notes.
I was trying to fork and provide the fix, but I couldnt able to setup the environment. Below is the solution which might help.
TcpSyslogMessageSender.java

Add the below line to allow user to update the SSLParameter. Add Getter and Setter to allow user the callback function
private Function<SSLParameters, SSLParameters> userDefinedSslParams = null;

Add the below lines in ensureSyslogServerConnection() method, at line#164 which is before socket connection.
         if (socket instanceof SSLSocket && userDefinedSslParams != null) {
             SSLSocket sslSocket = (SSLSocket)socket;
             sslSocket.setSSLParameters(userDefinedSslParams.apply(sslSocket.getSSLParameters()));
         }

@sownaRajkumar
Can you please help on below issue
https://github.com/jenkinsci/syslog-java-client/issues/103


puppet-bolt: init at 3.17.0

Motivation for this change
Bolt is an open source orchestration tool that automates the manual work it takes to maintain your infrastructure.
https://puppet.com/docs/bolt/latest/bolt.html
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@SuperSandro2000 Do you mind merging this one? Sorry for tagging you personally, but you seem to be very active in PR reviews (thanks a lot for that btw).
I just replicated the way puppet-lint was packaged, so should be quite straightforward.


Add rule to allow traffic from singleuser pods to proxy pod

This PR fixes the issue raised here https://discourse.pangeo.io/t/trying-to-open-gateway-cluster/1912 that was temporarily mitigated in PR https://github.com/2i2c-org/infrastructure/pull/819
It adds an egress rule to the singleuser pods allowing traffic to the proxy pod, this is particularly important for daskhubs that use the proxy to create clusters (schedulers and workers)

I deployed this to Pangeo staging to test and it works (future warning is expected)


Include Spanish in the languages shiped by default in Flask-User

Hello,
Actually I already have the po file for Spanish because I did it for one personal project. I am more than happy to contribute with the translation to Spanish.
If it is ok for you, just tell what shall I do next.
Thanks

See #152 . This will be available in the next 0.6.10 release.

Great, thanks for the very fast answer.


Fix deprecated code

Reg(init -> RegInit(
log2Up -> Log2Ceil

Is this obsoleted by https://github.com/ucb-bar/chisel-testers/pull/125 ?

@edwardcwang No, this is not obsoleted.  Nothing too critical here, just making things consistent with standard API, do you have time to review


link to playground should show preview

If someone posts a link to golang playground, we should comment with the body of the message for easy access for everyone, even though everyone can run it in browser, maybe this can be done if it's less than N lines or something, and it can be collapsed by default if it's longer, or maybe don't do at all if it's long?

The purpose of gopher sending the code to playground is to make it easier to share / collaborate. This would be the reverse of it. Sorry but this won't be fixed


First Rapid Move from z=0

Inventables: NeilFerreri15h to StewartAllen
@StewartAllen I just came back to check out your tool. I am still seeing an issue when the stock size is set. The first rapid move should be at a safe height, but it appears that with any stock that is greater than the height of the part it will do a rapid move to the safe height as if the part was at the bottom of the stock. For example, let's say I had a part that was 10mm in Z height, a safe height of 1mm, and my stock set at 50mm height. I'd want my first move to go to whatever XY at a height of 1mm. I am seeing that the first move would be G0 Xwhatever Ywhatever Z-39 Then it goes on to plunge UP to Z-2 or whatever.
Does that make sense?

that's a rendering artifact.  it assumes a start point of z=0, which is not always true depending on your gcode pre.  if you look at the gcode, there is not move to that start point.

sorry, this is a real bug.  working on a fix.


Blog post: Share permalink

Social media: we had envisioned 4th social tile being permalink. Not sure that was made clear. Are you suggesting we provide a permalink vial email? Or is email placeholder?

Permalink would just be a link to the page you're on, right? We can change this.

Permalink should now be set up.


feat: add hawker centers cleaning

add hawkers centers and cleaning dates

Default api call is 10 hawker centers if no limits are set when using data.gov.sg api.
We currently have around 114 hawkers in Singapore and would probably not hit the 150 limit at least till a decade time.


Tetragon: Repeatable benchmarks

We run a set of benchmarks against Tetragon, usually around release time. It would be nice to have some way via tetragon.io report what we do and results out. Then we can make these automated and report them every week or so.

@jrfastab
I'm sure benchmarking data for tetragon is important to many people, and before we get into tetragon.io I wonder if you'd prefer to share this in some other way first?


Update de.json

had to rewrite the sentence a bit, since a more direct translation wouldn't work that well in german.
also changed the word "password" to "password field" in the second sentence, because it works better in a german context

just realized how i can make it work better

"password field" is now "password" again

Thanks!
I also tried two variations using password or password field when I wrote the original text but ended up going for the final one mostly to make it shorter :) I think both ways convey the meaning correctly but still have some doubts.


[9.18] Allow to search and find issues by "assignee_id"

assignee_id is a valid parameter in GitLab's Issue API https://docs.gitlab.com/ce/api/issues.html#list-issues
It would be great, if you would allow your library to use it as filter parameter.

Please add tests

Thanks for this. It looks like this was already partially implemented. I have rebased and merged your addition to the phpdoc.


Error: Unable to start service guichaguri.trackplayer.logic.services.PlayerTask

I've been having trouble with this error for a few days now, any idea why I get this?
This happens only on Android.
Thanks.
I/ReactNativeTrackPlayer: Creating an ExoPlayer instance...
D/Exponent: Using cached or embedded response.
            Got cached OkHttp response for https://d1wp6m56sqw74a.cloudfront.net/%40expo-home-dev%2Fexponent-home-dev-bf2c6c97bc64f2ddfcee51d2d0cdea42ed33055a%2F25.0.0%2F0dafbec286ce3094cdfd69c3cddde606-25.0.0-android.js
            Have cached source file for https://d1wp6m56sqw74a.cloudfront.net/%40expo-home-dev%2Fexponent-home-dev-bf2c6c97bc64f2ddfcee51d2d0cdea42ed33055a%2F25.0.0%2F0dafbec286ce3094cdfd69c3cddde606-25.0.0-android.js
I/ExoPlayerImpl: Init ExoPlayerLib/2.4.3 [generic_x86, Android SDK built for x86, Google, 25]
D/ReactNative: ReactInstanceManager.ctor()
D/ReactNative: ReactInstanceManager.createReactContextInBackground()
               ReactInstanceManager.recreateReactContextInBackgroundInner()
               ReactInstanceManager.recreateReactContextInBackgroundFromBundleLoader()
               ReactInstanceManager.recreateReactContextInBackground()
               ReactInstanceManager.runCreateReactContextOnNewThread()
W/AsyncCondition: Could not find listener for key: experienceActivityKernelDidLoad
D/Kernel: Kernel started running.
D/ReactNative: ReactInstanceManager.createReactContext()
D/ReactNativeTrackPlayer: Updating current track...
D/ReactNativeTrackPlayer: onTrackUpdate
D/ReactNative: Initializing React Xplat Bridge.
D/ReactNativeTrackPlayer: onPlaybackUpdate
D/ReactNativeTrackPlayer: onPlay: The service is now on foreground, audio focus, wake and wifi locks have been acquired
D/ReactNative: Initializing React Xplat Bridge before initializeBridge
D/ReactNativeTrackPlayer: Requesting audio focus...
D/ReactNativeTrackPlayer: Audio focus granted!
D/ReactNativeTrackPlayer: Marking the service as started, as there is now playback
D/ReactNativeTrackPlayer: onStateChange
D/ReactNativeTrackPlayer: Sending event playback-track-changed...
D/ReactNative: Initializing React Xplat Bridge after initializeBridge
               CatalystInstanceImpl.runJSBundle()
D/ReactNative: ReactInstanceManager.setupReactContext()
               CatalystInstanceImpl.initialize()
D/AndroidRuntime: Shutting down VM
E/AndroidRuntime: FATAL EXCEPTION: main
                  Process: com, PID: 6741
                  java.lang.RuntimeException: Unable to start service guichaguri.trackplayer.logic.services.PlayerTask@5173cd5 with Intent { cmp=com/guichaguri.trackplayer.logic.services.PlayerTask (has extras) }: java.lang.ClassCastException: host.exp.exponent.MainApplication cannot be cast to com.facebook.react.ReactApplication
                      at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:3343)
                      at android.app.ActivityThread.-wrap21(ActivityThread.java)
                      at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1582)
                      at android.os.Handler.dispatchMessage(Handler.java:102)
                      at android.os.Looper.loop(Looper.java:154)
                      at android.app.ActivityThread.main(ActivityThread.java:6119)
                      at java.lang.reflect.Method.invoke(Native Method)
                      at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
                      at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)
                   Caused by: java.lang.ClassCastException: host.exp.exponent.MainApplication cannot be cast to com.facebook.react.ReactApplication
                      at com.facebook.react.HeadlessJsTaskService.getReactNativeHost(HeadlessJsTaskService.java:171)
                      at com.facebook.react.HeadlessJsTaskService.startTask(HeadlessJsTaskService.java:100)
                      at com.facebook.react.HeadlessJsTaskService.onStartCommand(HeadlessJsTaskService.java:53)
                      at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:3326)
                      at android.app.ActivityThread.-wrap21(ActivityThread.java)
                      at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1582)
                      at android.os.Handler.dispatchMessage(Handler.java:102)
                      at android.os.Looper.loop(Looper.java:154)
                      at android.app.ActivityThread.main(ActivityThread.java:6119)
                      at java.lang.reflect.Method.invoke(Native Method)
                      at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
                      at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

You're probably getting this because your Application class does not implement the ReactApplication interface

Thank you for your answer. I guess this is because I'm using Expo. Is there a way to get around it?

@adidoes Ejecting from Create React Native App

I should have mentioned that I already ejected my app.

@adidoes  did you find a solution because I am stuck now with guichaguri.trackplayer.logic.services.PlayerService error
it might be related?

I made some changes and it seems to work
I downgraded from
"com.android.support:appcompat-v7:23.0.1" (1)
to
"com.android.support:appcompat-v7:27.0.2" (2)
I would have to do further testing to find the exact problem but I think some libraries needed are only in (2) and not in (1) for me. You might have the same problem.

I made some changes and it seems to work
I downgraded
from
"com.android.support:appcompat-v7:27.0.2" (1)
to
"com.android.support:appcompat-v7:23.0.1" (2)
I would have to do further testing to find the exact problem but I think some libraries needed are only in (2) and not in (1).
You might have the same problem.

@efleurine Thanks for the info. I gave up trying to use ExpoKit because I didn't really need it. The library works without issues with ejected crna, I also think the performance of the app has improved.

Looks like this issue was resolved by not using expo. I'll close this now :)


Microsoft.Azure.Cosmos.Table.StorageException: 'Not Implemented' | CreateIfNotExistsAsync

Which service(blob, file, queue, table) does this issue concern?
table
Which version of the Azurite was used?
2.7.1
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
npm
What's the Node.js version?
8.11.2
What problem was encountered?

Microsoft.Azure.Cosmos.Table.StorageException: 'Not Implemented'

Steps to reproduce the issue?
using Microsoft.Azure.Cosmos.Table;
var table = tableClient.GetTableReference("blah");
await table.CreateIfNotExistsAsync();

Have you found a mitigation/solution?

Hi @edwin-huber Any comments here?

Using Package Microsoft.Azure.Cosmos.Table from Nuget, v1.0.8
and following snippet, this is no longer an issue with the new release and table api support.
            var tableClient = new CloudTableClient(new Uri("http://<IP_ADDRESS>:10002/devstoreaccount1/"), new StorageCredentials(storageAccount, accountKey));
            var table = tableClient.GetTableReference("blah");
            try
            {
                table.CreateIfNotExistsAsync().Wait();
            }
            catch (Exception e)
            {
                throw e;
            }

🎉, thanks @edwin-huber

Thanks, @edwin-huber. It is working

I've got one more question: The above apporach works, but requires code changes, when moving to prod. Is there any possibility to test without changing the code later?

Hi @TarekSalha
I'm not sure I can answer you without being sure of the issue you face.
Could you perhaps elaborate?

Ok, I see, my question was not clearly formulated:
What needs to be set as URI, if I want to use a real table storage and not azurite emulator?


Issues with Reconftw on OS X

Describe the bug
Perform the installation according to documentation for OS X Ventura 13.4.1, install all the apps, try to run any command,
Expected behavior
Help to fix this loop behavior on my Mac
Screenshots

Desktop (please complete the following information):

OS: OSX Ventura 13.4.1,

Additional context
It goes to the same start menu when running any command eg  ./reconftw.sh -d owasp.org  -r

I reinstall and update to the latest version, still same behavior :/

MacOS is not supported anymore

Thanks, Understood I will give it a try on Kali Linux


Coverage for nyc executed outside current working directory

Hi,
I have a lerna repo using npm's relatively new "file:" dependency feature to load resources from parent directories. For a given package when I run "npm run test" within a package it should use nyc/mocha from ../../node_modules/.bin.
        "test:unit": "NODE_ENV=test ../../node_modules/.bin/mocha ./src/*.spec.js ./src/**/*.spec.js",     "coverage:unit": "../../node_modules/.bin/nyc -- npm run test:unit",
If I install nyc globally (wish to avoid) and run directly (without ../../node_modules/.bin/nyc) files are be covered correctly. However when running as above only one file will show coverage.
I've tried the --cmd param (which --help shows correctly as the package dir) and also various combinations of "include".
Here's my nyc (13.1.0) config:
  "nyc": {     "sourceMap": false,     "all": true,     "instrument": true,     "include": ["src"],     "exclude": [       "**/*.spec.js"     ],     "check-coverage": true,     "per-file": true,     "lines": 100,     "statements": 100,     "functions": 100,     "branches": 100,     "reporter": [       "text"     ]   },

Hi @andrewcharnley sorry for the delayed response. We use lerna and nyc for the istanbuljs monorepo which contains all the instrumentation libraries nyc uses. You can take a look at the config we use there, but without a repo or minimal reproduction I can't offer much more help than that.


ReadOperation handle FIXME

What context is your feature request related to?
Handle this fixme in ReadOperation. // FIXME: Rewrite this to CodableTransaction
What solution would you like?
Remove the FIXME and provide a long term solution
Any additional context?

Ask @yaroslavyaroslav as he left the FIXME, we need some context about what is his original idea
Then we need some time to provide a solution

@yaroslavyaroslav I would like to understand what is your idea about this FIXME.
I will be reading the code and figure out what can I purpose as a solution from my point of view

I have checking and WriteOperation inherits ReadOperation and has a FIXME FIXME: Rewrite this to CodableTransaction.
We should apply same/similar strategy

To be honest I don't get the point of moving ReadOperation to CodableTransaction.
This Read/Write is highly coupled with the contract itself not a transaction(If I am wrong let me know)
What I can see right now is that it could be nice to:

protocolize ReadOperation and make the implementation internal
protocolize WriteOperation, make the implementation internal and remove inheritance from ReadOperation

@albertopeam it was a self notes that I've made in sake of convenience while developing 3.0.0 so it's all deprecated for now and should be deleted from code.

ok, then it should be enough to do points 1 and 2? @yaroslavyaroslav

@albertopeam not sure if I've got your last message, but anyway, my point was that the desired solution for those fixme is just to delete them from the code 💁🏻♂. Like it's all done priori 3.0.0 release and was addressed to merging different types of transaction types that library had has back there.


Update stable version in repo from 0.10.3 to 0.10.5

Think about update stable version to 0.10.5
Version: stable=0.10.3 (91 days old) => latest=0.10.5 (15 days old)
Installs: stable=607 (63.49%), latest=80 (8.37%), total=956
Click to edit

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.10.5 but the current stable version is already 0.10.5.
So this issue should be closed.


[FORMATTING] 11.0.2 commaPositon after doesn't work well when come up with comment end with comma

Input data
Which SQL and options did you provide as input?
SELECT
  name, -- user name,
  COUNT(distinct address) -- address
FROM
  user;

Expected Output
SELECT
  name -- user name,
, COUNT(distinct address) -- address
FROM
  user;

Actual Output
SELECT
  name, -- user name
, COUNT(distinct address) -- address
FROM
  user;

Usage

How are you calling / using the library?
demo website & ESM
What SQL language(s) does this apply to?
{
language: 'spark',
commaPosition: 'before',
}
Which SQL Formatter version are you using?
11.0.2

Yeah, this is essentially the same bug as #236
The implementation of these features is just very problematic. I've kept them around, though maybe it would be altogether better to just drop these problematic features. For now there's no simple way to properly fix them.
Probably should add a warning to docs to label these as buggy/problematic.

I am a little busy right now, I will submit a Merge Request to fix this problem when I have some free time, maybe in December.


Logger Timer Interface

Description
Create an interface that:

Configures
Starts
Stops
a hardware STM32L4 timer. The requirements:
The Timer must interrupt every 100 ms
The Interrupt must call the Experiment Logger function

100 ms = 10 Hz.
With the Timer base clock at 1MHz, this can be achieved using:

ARR = 199
PSC = 499


About SLAYER deConv feature

Objective of issue: Introducing SLAYER deConv block

Lava DL version:

[ ] 0.3.0 (feature release)
[ ] 0.2.1 (bug fixes)
[x] 0.2.0 (current version)
[ ] 0.1.2

Lava version:

[ ] 0.4.0 (feature release)
[ ] 0.3.1 (bug fixes)
[ ] 0.3.0 (current version)
[x] 0.2.0
[ ] 0.1.2

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] documentation request

Current behavior:

Is there any way of introducing convTranspose on SLAYER for deConv layers?

Expected behavior:

Something similar to slayer.block.cuba.deConv()

Thanks in advance,
Alex.

@bamsumit Thanks for pointing us to the right direction. Implementing it on the block api seems doable so we'll try to contribute to it. Any guidance during the process would be appreciated.


Editing HTML content

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hello,
I don't figure out how to edit the HTML content.
I would like to personnalise it for my customers.
Which files should I edit?
Thank you
🐻 Uptime-Kuma Version
1.16.1
💻 Operating System and Arch
Ubuntu 20.04 x64
🌐 Browser
Brave 1.39.111
🐋 Docker Version
No response
🟩 NodeJS Version
14.19.3

Also read https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md for how to build the project.


Create Blog "Thread pools: the simplest concurrency platform"

See https://web.archive.org/web/20081227233324/http://www.cilk.com/multicore-blog/bid/6270/Thread-Pools-The-Simplest-Concurrency-Platform

This blog is a follow-up to "The folly of DIY multithreading" (which @ailop is now updating) and it starts a series about various flavors of concurrency platforms

Thread pools ("the simplest")
OpenMP
Intel's Threading Building Blocks
Message passing
Data-parallel languages
Cilk++

It discusses some practical pros and cons of thread pools.


Updated to danish standards

I signed it!

Please check https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#-found-an-issue under "Localization Issue"

@lgalfaso thanks for the link. An edit have been requested at the right place

@Squazz once the change goes into Google closure, feel free to open a new issue to generate the locale files once again.

@lgalfaso I will try to, but I'm sure I know how to :P

if you just open an issue with "Please regenerate the locale to see the changes from XX reflected", is good enough. Someone will recreate the files

@lgalfaso ok, thanks man! :+1:


DatePicker header overflow fix

Description
In this PR I removed constant, hardcoded DatePicker header height. Leaving Container with no specified height enables it to wrap its content and prevent clipping/overflowing.
How it looked:

How it looks now:

Related Issues
#29736
#28778
#29681

This needs tests. I'm also pretty surprised this didn't break any existing tests.
/cc @darrenaustin who is working on layout stuff for Date Picker.
/cc @willlarche who was getting us some guidance on this control as it's currently broken when text scale is off or when large fonts are used.

Does this still look right with a smaller font (the case where it's not currently overflowing)?

Does this still look right with a smaller font (the case where it's not currently overflowing)?

I checked it on a couple of devices with different ratios/resolutions/densities & languages and didn't encounter any noticeable difference. I guess it shouldn't break anything, as it will wrap around the actual content, so there might be a couple of pixels difference at most.

This needs tests.

Well, I'm not very experienced with tests yet and I was convinced that changing widget height isn't something actually testable. But if it is, then I'll have a look at this.

You should be able to run some tests that change the font scale and verify that it works.
Something like this: https://github.com/flutter/flutter/pull/29734/files#diff-421a3407c743099e7c05ef094015f095
But try it with text scale factors larger than 1.5 as well.

Thanks for your work on this. There were several other layout problems we needed to fix in addition to this one, so I put them all in a single PR #31514 which removes the hard coded constraints on the layouts.


Monolog error

I found the error. It is a conflict with an older library used by a plugin (Gravity PDF).
I'll alert the plugin author.
Is there a way for me to prevent this from happening in the future? Maybe namespace monolog with the version number?
I'm still interested in disabling the logger altogether as the odds of another conflict is greater than any other type of error.
Thank you
Fatal error: Declaration of Monolog\Handler\ProcessableHandlerTrait::pushProcessor(callable $callback): Monolog\Handler\HandlerInterface must be compatible with Monolog\Handler\AbstractHandler::pushProcessor($callback) in /path/lib/google-api-php-client-2.4.0/vendor/monolog/monolog/src/Monolog/Handler/AbstractProcessingHandler.php on line 22
Environment
Linux m51.siteground.biz 3.12.18-clouder0 #1 SMP Tue Oct 9 18:14:47 EEST 2018 x86_64
PHP version 7.3 (also tried 7.2 and 7.4)
Package name and version: google-api-php-client-2.4.0
Steps to reproduce:
$token = $client->fetchAccessTokenWithRefreshToken($packet['refresh_token']);
I had no problems with the library when obtaining the original access token response (which included the refresh token).  $client was the same object and builds fine without issue until this point.
For the record, I also have no issue testing this code on two other servers.
I've tried uploading Monolog again.
How do I disable logging?
UPDATE:
I can confirm that the issue rears it's head anytime a log is attempted.
Thank you

Update: Gravity PDF is namespacing Monolog since they want to support PHP 5.6. A lot of plugins want to support 5.6.
I am trying to do something similar. I cannot namespace the entire lib because some dependencies are calling classes inline with \Classname instead of use statements.

You could provide a different Psr\Log\LoggerInterface implementation (perhaps one that doesn't do anything) to Google_Client::setLogger.
For instance, if you're using PHP 7:
$client = new Google_Client;

$client->setLogger(new class implements Psr\Log\LoggerInterface {
    public function emergency($message, array $context = array()) {}
    public function alert($message, array $context = array()) {}
    public function critical($message, array $context = array()) {}
    public function error($message, array $context = array()) {}
    public function warning($message, array $context = array()) {}
    public function notice($message, array $context = array()) {}
    public function info($message, array $context = array()) {}
    public function debug($message, array $context = array()) {}
    public function log($level, $message, array $context = array()) {}
});

match p :Private person
вт, 23 июн. 2020 г. в 21:43, Amy Hill<EMAIL_ADDRESS>
Update: Gravity PDF is namespacing Monolog since they want to support PHP
5.6. A lot of plugins want to support 5.6.
I am trying to do something similar. I cannot namespace the entire lib
because some dependencies are calling classes inline with \Classname
instead of use statements.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-api-php-client/issues/1855#issuecomment-648247170,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNECNL3PZBLP6JXIJ6PPT3RYDETNANCNFSM4ODB3DGQ
.

I will give it a try. Thank you for the example.
Amy
From: John Pedrie<EMAIL_ADDRESS>Sent: Tuesday, June 23, 2020 4:20 PM
To: googleapis/google-api-php-client
Cc: Amy Hill; Author
Subject: Re: [googleapis/google-api-php-client] Monolog error (#1855)
You could provide a different Psr\Log\LoggerInterface implementation (perhaps one that doesn't do anything) to Google_Client::setLogger.
For instance, if you're using PHP 7:
$client = new Google_Client;
$client->setLogger(new class implements Psr\Log\LoggerInterface {
public function emergency($message, array $context = array()) {}
public function alert($message, array $context = array()) {}
public function critical($message, array $context = array()) {}
public function error($message, array $context = array()) {}
public function warning($message, array $context = array()) {}
public function notice($message, array $context = array()) {}
public function info($message, array $context = array()) {}
public function debug($message, array $context = array()) {}
public function log($level, $message, array $context = array()) {}
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view https://github.com/googleapis/google-api-php-client/issues/1855#issuecomment-648395874  it on GitHub, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADKWXIB5R65BEOIDA6NJ27LRYEE6FANCNFSM4ODB3DGQ .  https://github.com/notifications/beacon/ADKWXID6GYBQQSA44YUWXE3RYEE6FA5CNFSM4ODB3DG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE2S3YYQ.gif

I'm going to close this issue out. If you're still having trouble, please feel free to re-open!


Add perimeter to morphometrics

Checklist

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[ ] I've added relevant tests for my contribution
[ ] I've updated the documentation and/or added correct docstrings
[ ] I've assigned a reviewer
[x] I've consulted ADS's internal developer documentation to ensure my contribution is in line with any relevant design decisions

Description
This PR aims to add axon and axonmyelin perimeter in the morphometrics file.
Linked issues
Fixes #497

I added tests for validating this feature on the simulated axon-myelin instances. Here is the simulated image where axon shape is circular on which I tested this PR.

The inner perimeter ( axon perimeter) and the outer perimeter (axon myelin perimeter) of the simulated axon are equivalent to the perimeters computed by the skimage regionprops inbuilt perimeter attribute.
However, note the functionality was only tested for axon shape being circular in nature.

TEST 2
________
For ellipsoidal shape, since there is no way to compute the morphometrics, I manually computed the perimeter.
Here is the simulated image.

The ground truth perimeter computed manually using ImageJ on this simulated image

Instance   no
Axon Perimeter
Axonmyelin Perimeter

1
317.301
351.858

2
289.035
361.31

3
263.978
377.05

4
245.296
402.621

5
225.616
441.851

6
190.714
473.511

7
180.503
586.685

8
157.717
788.764

Perimeter computed using this PR (skimage regionprops inbuilt perimeter attribute)

Instance no
Axon Perimeter
Axonmyelin Perimeter

1
326.391919
365.019336

2
297.4213562
374.67619

3
270.1076477
388.333044

4
246.7939392
415.646753

5
227.4802307
456.617316

6
192.8528137
490.27417

7
179.882251
608.901587

8
162.9116882
821.185858

Looks close enough for me for the ellipse case. You could do a % difference comparison to give us an order of magnitude of the possible error in terms of manual vs. our tool, my guess is it will be around 2-3% which I think is sufficiently low.

You are right @mathieuboudreau

Instance   no
Axon Perimeter
Axonmyelin Perimeter

0
2.86507732
3.74052487

1
2.90150198
3.69936903

2
2.32202977
2.99245312

3
0.61066599
3.23523934

4
0.862685
3.34192199

5
1.12147705
3.54018597

6
0.3450863
3.7867999

7
3.2936768
4.11046374

However, the axon_perimeter and axonmyelin_perimeter are missing from the stats array in CLI here and here.

I actually forgot that we had merged the morphometrics CLI into ADS, so skipped adding the functionality there. Thanks @mariehbourget for catching it. I've fixed this and you can now test axondeepseg_morphometrics CLI.

I tested again, both files from CLI and fsleyes are identical.
Thanks @vs74!

Thanks a ton, @mariehbourget for thoroughly testing this PR. I'm merging this PR.


itest flake: test_multi-hop_htlc

Very flaky itest, making issue to avoid wasted work:
 test_harness.go:94: Error outside of test: *errors.errorString invoice update err: rpc error: code = DeadlineExceeded desc = context deadline exceeded
        /Users/nsa/lnd/lntest/itest/lnd_multi-hop_test.go:178 (0x1b56daa)
        	waitForInvoiceAccepted: t.Fatalf("invoice update err: %v", err)
        /Users/nsa/lnd/lntest/itest/lnd_multi-hop_htlc_aggregation_test.go:160 (0x1b4e127)
        	testMultiHopHtlcAggregation: waitForInvoiceAccepted(t, carol, h)
        /Users/nsa/lnd/lntest/itest/lnd_multi-hop_test.go:142 (0x1b56aeb)
        	testMultiHopHtlcClaims.func1.1: subTest.test(
        /usr/local/go/src/testing/testing.go:1259 (0x11054e2)
        	tRunner: fn(t)
        /usr/local/go/src/runtime/asm_amd64.s:1581 (0x1068641)
        	goexit: BYTE	$0x90	// NOP
=== CONT  TestLightningNetworkDaemon

I think this one might have been fixed.

i still see this one


Fixing flaky tests in PortTelnetHandlerTest

What is the purpose of the change
Tests in PortTelnetHandlerTest fail when ran after the ones in DubboProtocolTest even when there’s no bug in the code. Tests in DubboProtocolTest open new ports and servers in DubboProtocol, and tests in PortTelnetHandlerTest implicitly assume no such new ports and servers. The changes ensures DubboProtocolTest closes all the newly opened ports and servers after all tests finish running, so later tests in PortTelnetHandlerTest do not fail.
Brief changelog
dubbo-rpc/dubbo-rpc-dubbo/src/test/java/org/apache/dubbo/rpc/protocol/dubbo/DubboProtocolTest.java
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #2741 into master will increase coverage by 1.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2741      +/-   ##
=========================================
+ Coverage   62.38%   63.4%   +1.02%
=========================================
  Files         598     576      -22
  Lines       27911   25922    -1989
  Branches     4828    4540     -288
=========================================
- Hits        17412   16437     -975
+ Misses       8233    7316     -917
+ Partials     2266    2169      -97

Impacted Files
Coverage Δ

.../remoting/transport/netty4/NettyClientHandler.java
75% <0%> (-11.12%)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
54.11% <0%> (-8.24%)
:arrow_down:

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
73.58% <0%> (-7.55%)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
58.69% <0%> (-6.53%)
:arrow_down:

...org/apache/dubbo/rpc/filter/ActiveLimitFilter.java
77.77% <0%> (-5.56%)
:arrow_down:

...onfig/spring/extension/SpringExtensionFactory.java
78.94% <0%> (-5.27%)
:arrow_down:

...ing/exchange/support/header/AbstractTimerTask.java
66.66% <0%> (-4.17%)
:arrow_down:

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
41.66% <0%> (-4.17%)
:arrow_down:

...rg/apache/dubbo/rpc/protocol/dubbo/DubboCodec.java
60.2% <0%> (-4.09%)
:arrow_down:

...bo/remoting/transport/netty/NettyCodecAdapter.java
53.12% <0%> (-1.57%)
:arrow_down:

... and 30 more

Continue to review full report at Codecov.

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


Use Msg.(type) instead of Msg.Type()?

Context
The message interface currently has a function Type, which returns the type of the message in form of an integer. The integer type is useful for encoding and decoding. However, it seems like we are misusing for distinguishing two instantiated messages, because then we can already check for the inherent type.
Problems

Exposing the Type on the message interface requires us to implement that function for every concrete message type. This tends to bloat our code base, especially noticeable in files with a lot of message type definitions.
Moreover, it is error prone. It is easy to copy and paste the Type function from another message type and forget to return the correct integer type. This will not be caught by the compiler, but will most often require time-consuming debugging to be detected.
Overall, I don't see any significant benefits from exposing the integer type.

Suggestion
I believe we can get rid of exposing the Type on the interface completely. Whenever we encode and decode, we can write and read the type manually, and whenever we check the type of an instantiated message, we use the inherent type.

There is currently no intention of implementing this. Closing for now.


Fix code scanning alert - Exception text reinterpreted as HTML

Tracking issue for:

[ ] https://github.com/shelcia/Socialgram/security/code-scanning/2

@shelcia I would like to contribute to this repo
kindly assign me this issue under iwoc
Thankyou!!


Missing Option

This section says there is an option to manage users.  This does not exist in the UI
https://docs.microsoft.com/en-US/microsoftteams/create-and-manage-dial-plans#add-users-to-a-dial-plan

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

ID: 451b44cf-961a-5bf5-e4b3-16b96e008dd7
Version Independent ID: 54e706e1-aaa2-3a03-b8c6-5e96244b85ac
Content: Create and manage dial plans - Microsoft Teams
Content Source: Teams/create-and-manage-dial-plans.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@danmyhre  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

I did find that the option to assign a user to a dial plan is available under:
Users -> User -> Policies -> Edit -> Dial plan

@danmyhre Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Overriding default keybinds

Can't override the default keybind. Similar to the previous issue here #6
# Pomodoro
set -g @pomodoro_start 'h'                  # Start a Pomodoro with tmux-prefix + p
set -g @pomodoro_cancel 'H'                 # Cancel a Pomodoro with tmux-prefix key + P

When I press tmux-prefix + p It starts the menu. It also works with the new keybind I created.

@DMunkei - Any luck?


Add progressive archive extraction pipe

To support handling of nested archives inside a scanned codebase we have possible ways:

extract everything upfront recursively at full depth
extract on demand when there is a need

The first solution is easy but likely noisy. The second solution would be better

Some workflow ideas for #2:

Run your usual pipeline
Flag nested archives with a new "extract" status (or a new db field) using the Web UI or API. A list of archives can be filtered using on the codebase resources the is_archive field
Run a new pipeline that will extract the flagged archives and scan their content


Ideas for adding checks

setup.cfg:
[flake8]
max-complexity = 12
ignore = E203, E231, E501, E722, W503, B950
select = C,E,F,W,B,B9

.pre-commit-config.yaml:
repos:
- repo: https://github.com/psf/black
  rev: 20.8b1
  hooks:
  - id: black

- repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v3.2.0
  hooks:
  - id: check-added-large-files
  - id: check-case-conflict
  - id: check-merge-conflict
  - id: check-symlinks
  - id: check-yaml
  - id: debug-statements
  - id: end-of-file-fixer
  - id: mixed-line-ending
  - id: requirements-txt-fixer
  - id: trailing-whitespace

- repo: https://github.com/mgedmin/check-manifest
  rev: "0.43"
  hooks:
  - id: check-manifest
    stages: [manual]

- repo: https://gitlab.com/pycqa/flake8
  rev: 3.8.3
  hooks:
  - id: flake8
    additional_dependencies:
    - flake8-bugbear
    - flake8-docstrings
    - flake8-import-order
    - flake8-print

pre-commit run -a to run locally (pip install pre-commit).

Haha thanks, I was right on the aay on adding that! I'll put it there


🔨 Fix the bug for data not reflected when perform CRUD operation in t…

Solve the issue of data not reflect in screen after performing CRUD operation in task, note and project screen.
Change the toast in all screen in application.

Remove unnecessary package from the pubspec.yaml


fixes #1488 fakes with casts

Fixes #1488 in a different way than #1362

There are a bunch of ways to getting an attribute:
$this->getAttribute($column));
$this->attributes[$column]);
$this->{$column});

If the attribute (say... extras) is NOT cast as anything, they all output the same thing:
string(112) "{"number_with_prefix":null,"number_with_suffix":null,"text_with_both_prefix_and_suffix":"122","fake_table":null}"

string(112) "{"number_with_prefix":null,"number_with_suffix":null,"text_with_both_prefix_and_suffix":"122","fake_table":null}"

string(112) "{"number_with_prefix":null,"number_with_suffix":null,"text_with_both_prefix_and_suffix":"122","fake_table":null}"

If the attribute (say... extras) IS cast as array/object, the output is totally different:
array(4) {
  ["number_with_prefix"]=>
  NULL
  ["number_with_suffix"]=>
  NULL
  ["text_with_both_prefix_and_suffix"]=>
  string(3) "122"
  ["fake_table"]=>
  NULL
}

string(112) "{"number_with_prefix":null,"number_with_suffix":null,"text_with_both_prefix_and_suffix":"122","fake_table":null}"

array(4) {
  ["number_with_prefix"]=>
  NULL
  ["number_with_suffix"]=>
  NULL
  ["text_with_both_prefix_and_suffix"]=>
  string(3) "122"
  ["fake_table"]=>
  NULL
}

In short:

$this->attributes[$column] gets the value BEFORE the cast has been applied;
$this->getAttribute($column)); and $this->{column} gets the value AFTER the cast has been applied;

In this case, we definitely prefer BEFORE.

Yeah, no. Remember when I said we don't need to check if the attribute is set? We do. Desperately. Otherwise it will fail when no fake column is present. Good think we have unit tests.
Added on master. Issue is really closed now.

@tabacitu Unfortunately this introduced an issue with translations. see the reference

Thank you @OliverZiegler ! On it!

@tabacitu I'm currently preparing a PR
Just implementing unit tests for this at the moment ;)
@tswonke and me already have a fix 😊

@tabacitu see #1548 for the fix.
Working on the unit tests to verify every edge case will work 😉


Add template for start of namespace for TOSCA components

Add template functionality to the wineryNamespaceSelector. This part is optional and can be set via the useStartNamespace parameter (default = true). In addtion the toscaType parameter is added to specify the component type

[x] Ensure that the commit message is a good commit message
[ ] Ensure to use auto format in all files
[x] Change in CHANGELOG.md described
[ ] Tests created for changes
[x] Screenshots added (for UI changes)

Can one of the admins verify this patch?

The conflicts in section.component.html could come from CSS improvements at https://github.com/eclipse/winery/pull/177 and https://github.com/eclipse/winery/pull/178

I think that we should introduce a start namespace because if there is no start namespace stored in the local storage, the user might introduce an invalid namespace as shown here:

In my opinion a valid namespace must be set before the Add button is clickable.

Didn't @tstadelmeier add a validation to the form? Did that get lost?
Am 17.10.2017 11:52 schrieb "Lukas Harzenetter"<EMAIL_ADDRESS>I think that we should introduce a start namespace because if there is no
start namespace stored in the local storage, the user might introduce an
invalid namespace as shown here:
[image: grafik]
https://user-images.githubusercontent.com/23058331/31658482-661cb3a6-b331-11e7-8953-8b2ba3763f33.png
In my opinion a valid namespace must be set before the Add button is
clickable.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/winery/pull/179#issuecomment-337179856, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABTafp6mmwVu_4ALeQDugZ_L-7fLl8Mhks5stHjYgaJpZM4P7xPV
.

Maybe this is because @pmeyer should merge the current Master and resolve the conflicts
On Tue, Oct 17, 2017 at 12:07 PM +0200, "Oliver Kopp"<EMAIL_ADDRESS>wrote:
Didn't @tstadelmeier add a validation to the form? Did that get lost?
Am 17.10.2017 11:52 schrieb "Lukas Harzenetter"<EMAIL_ADDRESS>I think that we should introduce a start namespace because if there is no
start namespace stored in the local storage, the user might introduce an
invalid namespace as shown here:
[image: grafik]
https://user-images.githubusercontent.com/23058331/31658482-661cb3a6-b331-11e7-8953-8b2ba3763f33.png
In my opinion a valid namespace must be set before the Add button is
clickable.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/winery/pull/179#issuecomment-337179856, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABTafp6mmwVu_4ALeQDugZ_L-7fLl8Mhks5stHjYgaJpZM4P7xPV
.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

i fixed this issue through a empty string check to ensure that this behavior cannot happen

@meyerpp please use http://www.example.org/tosca as default namespace

Further it would be nice to have this feature in the WineryNamespaceSelectorComponent

Moved functionality to the WineryNamespaceSelectorComponent

@meyerpp We urgently need this update. Could you please fix the minor issues?
If possible, can you also check whether the button "Add" works:

@koppor i fixed this issue within this PR. It know works as excepted


Unable to Step into Selection Sets and Entity Objects through Visual Code

Describe the bug
I have been trying to use Visual Code to debug Autolisp and one of the issues that I am noticing is that I can't seem to step into the selection set or entity objects like I could do with the VLIDE. Below are some screenshots showing both the Visual Code and VLIDE screen.
To Reproduce
Steps to reproduce the behavior:
To reproduce the behavior create a simple Autolisp function like the one I have in the screenshot or below. Next, draw some lines, circles, rectangles, on the screen and then use the function to select the objects.
(defun SelectionSetTest ()
(setq test (sseget))
)
Expected behavior
Ideally, I would like to be able to double click or right click on the selection set and get the entities that are in that selection set. From there I would like to be able to double click or right click on the entity to see information about the entities. In the image showing the VLIDE you can see what I am talking about.
I double clicked on the selection set and it opened up the "PICKSET" window that showed the 4 entities (3 Circles and 1 Polyline) from there I was able to double click on the first circle in the list and get the "ACAD CIRCLE" window to pop up. From there I could double click on the layer and get the "AUTOCAD TABLE ENTRY" window to pop up.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Version AutoCAD 2022

Additional context
N/A

Hi @rubinoj21 ,
Thanks a lot for reporting this! And yes, I totally agree with you.

Marked as 'enhancement'.

Hello,
Have there been any updates on this possibly being added to Visual Code? We utilize the VLIDE often and are unable to transition to Visual Code without this.
Also, do you know if there any plans to remove the VLIDE in future releases of AutoCAD and force the use of Visual Code?
Thank you

Hello @rubinoj21 ,
There's no plan to retire the VLIDE. We just don't add new features into it.
As to this issue/limitation, there will be a slight enhancement with the upcoming ACAD 2025. And meanwhile, we'll continue to make a thorough solution to the problem.


Add Task.combine

Add Task.combine to facilitate using record builders with Tasks while they're still part of Roc.
Also, put a message by the top of the Task.batch function that Task.combine is what should now be used for record builders.

@ayazhafiz thanks for the approval!


Update gapic-generator hash

Includes some changes to resource name parsing.
Goal: Remove explicitly-configured fixed resource names from all GAPIC configs, so we can simplify gapic-generator.

PTAL


Show Reinstall VSCode Error Message when our Bundled tsserver.js is Missing

Fixes #21689

This doesn't handle all possible situations that would break the bundled version of TypeScript, but it seems like the tsserver.js file is what is being deleted most often


⚠ ClamAV has degraded performance

In de936b2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in 5ef1a94.


Multiple columns in returning()

I was trying to do something like the following:
squel.insert().into('my_table')
  // ...
  .returning('id')
  .returning('created');

But it would appear that Squel only stores a single value for returning(), so whichever value it was most recently called with is the only one that ends up in the query. This is in contrast to other methods, like field(), where successive calls add to an internal list.
It is valid in PostgreSQL to return multiple values as in the above. What is the best way to accomplish this in Squel?

You could implement your own returning Block and then replace the insert query construction call with one which uses your custom block.


move kfctl downloads page from reference section

Is the reference section the right place for information about downloading kfctl
Is there a reference guide for kfctl
Could the kfctl download page be moved to the Getting Started/requirements section; update information about how to get a kubeflow release with how to get kfctl

+1 to the suggestion of moving the kfctl download info into the getting-started section.
@kbhawkey is this something you'd like to tackle during the doc sprint?

Sure!

/assign @kbhawkey

This is done.
/close


HDDS-5042.[FSO] Improve KeyDeletingService to cleanup FSO files

What changes were proposed in this pull request?
Added unit test to verify whether KeyDeletingService deletes FSO files. The DirectoryDeletingService is responsible for moving subfiles of deleted directories into the DeletedTable. The DeletedTable however expects the older key format (volume/bucket/key) . The fix in this change ensures that the right key format is written into the DeletedTable
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5042
How was this patch tested?
UT

Thanks @rakeshadr @linyiqun . Addressed  review comments.

Thanks @sadanand48 for the good work.
Thanks @linyiqun for the useful comments.
+1 LGTM, I will merge it shortly.


Add api/version for proxy client

Coverage increased (+0.2%) to 90.909% when pulling 038d0e88344156d14218762d118571ddf62b61f0 on proxy_version into 20fc0bfac68814d548e65e6c71c8a6b6d88558ae on dev_proxy_version.


[Feature Proposal] Pull Request to add support for Erlang-only validate_doc_read functions

Overview
Context
This is not really a feature request because we do have this functionality implemented in our CouchDB 2.x fork. However I consider this feature to be a good fit for upstream repository and I decided to open a discussional issue first instead. Let me know if it is better to directly submit a PR with this functionality.
Background and strategic fit
CouchDB already has support for validate_doc_update functions which if defined in the ddoc can be used to prevent invalid or unauthorized document update requests from being stored. In other words document-level write security. I'm proposing a patch which adds the same type of security but for all document reads. This functionality have been used in our production cluster and proved to be useful. There are a lot of valid use case scenarios akin to "Some particular user may only read documents he created" and so on. This also may alleviate overhead caused by alternative solutions such as per-user DBs.
Implementation details
First things first. This feature can lay waste to read performance if validate_doc_read functions will be implemented in JS due to notorious external query server and serialisation/deserialisation and standard I/O overhead. Thereby I propose to impose a limitation on these functions namely they have to be implemented only in Erlang and marked as advanced functionality. Perhaps they have to be available solely by means of some additional config such as validate_doc_read = true. Therefore this will not in any case affect default CouchDB setup but might be useful for advanced users. When implemented as Erlang ones potential overhead of these functions is negligible but benefits are perceptible. In a nutshell implementation extends load_validation_funs to also find, load and parse validate_doc_read functions from design documents and then make appropriate calls in the end of make_doc(). Unauthorised exception will be thrown and status code 403 returned from the chttpd handler when some invariants in validate_doc_read logic are violated.
Conclusions
I've decided to create this issue instead of directly sending a PR because at first I would like to gather feedback from core Apache and Cloudant maintainers regarding the usefulness and expediency of this feature. In case of a more or less positive one I would be glad to submit a PR which will shed more light on the technical part (which is pretty straightforward) as well as open a discussion about potential improvements.

I'm -0.5 on the proposal.
Concerns:

Overall throughput decrease if a database doesn't have any of these new functions, just from having to traverse your new code path on every read,
Lack of security and sandboxing in Erlang-based functions, which is why we ship with the Erlang query server disabled by default

I know that point 2 above is why this feature will never land at Cloudant and would likely never land at anyone running a public/SaaS CouchDB offering. A potential alternative would be #1554 's approach.
The only way to handling point 1 above is to actually have proper load testing & metrics showing the impact. This is something Cloudant devs have done with their major functionality changes in PRs over the past couple of years. Do you have any data points to share?
The alternative to this was already proposed by the committer list and is in the roadmap is the per-document permissions: the _access proposal. See<EMAIL_ADDRESS>This approach already has a bunch of code behind it as well, and would vastly improve on the current db-per-user access model. With _access would we even need validate_doc_read functions? I don't think so.

There exist approaches to filter outgoing data streams very fast without intervening CouchDB codebase – see ie https://github.com/ermouth/covercouch – however I doubt it’s useful for 2.x.


Adjusting sliders using arrow key results in odd behaviour (v0.2.24)

steps to reproduce:

bring up any image layer
select the layer
select the config tab
click the right hand side vertical slider
use the keyboard arrow keys to change the value
sit back and enjoy hours of fun with the elastic band behavior.

@PhillipsOwen this has been fixed and ready for testing (upgrade/form-filter) branch

This appears to work fine now. closing this issue...


:apple: add support for Linux arm64 and macOS M1

add ability to get linux-arm64 releases
also support darwin-arm64 for Apple Silicon
finally use curl only as wget may be missing (e.g. on macOS)

Thank you for your contribution, will add the compatible OS type on ReadMe before merging.

Thank you very much!  I found your software quite useful for working around https://github.com/zplug/zplug/issues/418 :tada:


Create es5 class helper methods

Methods that allow es5 users to create classes
cc @Cmdv

ideas here: https://gist.github.com/BerkeleyTrue/5a1f64eae5a3502622b4/

I have update the gist with examples of how to do the things using es5. I have also removed some of the invariants that would be triggered during construction. I don't think adding a createClass method would be helpful and it would force users to create actions/stores in a certain way.
Instead I will do my best to keep creation of stores/actions/cats possible in many different ways. I would also like to add examples to the gist above using stampit.
If there are certain things that are a pain to do without es6 classes please do let me know and I will try to accommodate within reason.


Code for "Table 2: Performance Comparison of SSAMBA and SSAST Models" is needed!

Will you mind providing the code related to this "Table 2: Performance Comparison of SSAMBA and SSAST Models" in your paper?
Thank you!

Hi,
Thank you for your interest in SSAMBA, the code for fine-tuning on downstream datasets is available under src/finetune. You can use the .sh recipes to reproduce our results.


Payment support: Stripe

From David Nimorwicz in Hyvä slack:
Is there any planned implementation for the Stripe payment integration?

Eltrino team developed Stripe Payment method for Hyvä Checkout


npm run install-libs error

I had no issues running your previous version of pogo-optimizer, but this one is causing me issues when I run "npm run install-libs"
Here's the log:
bryces-Mac:pogo-optimizer bryce$ npm run install-libs

><EMAIL_ADDRESS>install-libs /Users/bryce/Documents/pogo-optimizer
> bower install

/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/node_modules/configstore/index.js:54
				throw err;
				^

Error: EACCES: permission denied, open '/Users/bryce/.config/configstore/bower-github.json'
You don't have access to this file.

    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at Object.create.all.get (/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/node_modules/configstore/index.js:35:26)
    at Object.Configstore (/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/node_modules/configstore/index.js:28:44)
    at readCachedConfig (/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/config.js:19:23)
    at defaultConfig (/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/config.js:11:12)
    at Object.<anonymous> (/Users/bryce/Documents/pogo-optimizer/node_modules/bower/lib/index.js:16:32)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)

npm ERR! Darwin 15.0.0
npm ERR! argv "/usr/local/Cellar/node/6.3.1/bin/node" "/usr/local/bin/npm" "run" "install-libs"
npm ERR! node v6.3.1
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install-libs: `bower install`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install-libs script 'bower install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the pogo-optimizer package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     bower install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs pogo-optimizer
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls pogo-optimizer
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/bryce/Documents/pogo-optimizer/npm-debug.log

Any ideas? Thanks! Love the app!

Try sudo npm run install-libs

I had actually tried that as well, sorry I should have mentioned that:
bryces-Mac:pogo-optimizer bryce$ sudo npm run install-libs --allow-root

><EMAIL_ADDRESS>install-libs /Users/bryce/Documents/pogo-optimizer
> bower install

bower ESUDO         Cannot be run with sudo

Additional error details:
Since bower is a user command, there is no need to execute it with superuser permissions.
If you're having permission errors when using bower without sudo, please spend a few minutes learning more about how your system should work and make any necessary repairs.

http://www.joyent.com/blog/installing-node-and-npm
https://gist.github.com/isaacs/579814

You can however run a command with sudo using --allow-root option

npm ERR! Darwin 15.0.0
npm ERR! argv "/usr/local/Cellar/node/6.3.1/bin/node" "/usr/local/bin/npm" "run" "install-libs" "--allow-root"
npm ERR! node v6.3.1
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install-libs: `bower install`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install-libs script 'bower install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the pogo-optimizer package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     bower install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs pogo-optimizer
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls pogo-optimizer
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/bryce/Documents/pogo-optimizer/npm-debug.log

Okay, try bower install and sudo bower --allow-root install. One of those should work.

Still seems to give me the same error. Weird... thanks for the assistance

try sudo chown -R $USER:$GROUP ~/.npm sudo chown -R $USER:$GROUP ~/.config
It works for me.

I cleared out everything and tried again with the sudo bower --allow-root install and it works :) thanks!!

I got the same problem?
But was the command you used and where did you use it?

The chown commands listed above are likely the ones that fixed it - you just need to run them in a terminal


Nested types support inside of TypeArguments

Is your feature request related to a problem? Please describe.
Currently nested types aren't supported by x:TypeArguments. The IDE support for Rider does mark it as valid syntax using the +, like x:TypeArguments="ns:Class+NestedType" yet the compiler gives the following error:

Error AVLN:0004 Avalonia: Unable to resolve type Class+NestedType from namespace...

Describe the solution you'd like
Support for nested types in x:TypeArguments just like Xamarin has, see:
https://github.com/xamarin/Xamarin.Forms/issues/12455
Describe alternatives you've considered
Un-nesting the type
Additional context
None needed

Duplicate of #2725


Support stdin as yaml source

Problem
crd2pulumi only accepts CRD yaml files as file path args. It is common to support stdin as an input source indicated by a - (dash). This opens the doors for command pipelining and integrations.
Eg.
$ cat my-crd.yaml | crd2pulumi -n -

Solution
Add a special case for "-" that will read the yaml contents from stdin instead of a file.

Thanks for the PR, @gregnr!


state should be immutable

One of the best features of Redux is the state is never actually modified, instead it is cloned and the clone is modified and returned as current state. This allows for the most helpful feature of time travel debugging where actions can be recorded and replayed to create a situation and tests. I may PR if I get time.

Good point. Is your concern just that the state mutations don't happen atomically with the current implementation? With the current implementation, if an ISR were to fire during the cedux_run_x function and that ISR had a handle to the store, it's possible that it would access the store in an intermediary state. Is that what the issue you had in mind?


[Feature Request] A way to instruct how to infer this

TypeScript Version:  2.3.2
Code
I have this definition:
extend<T, K>(cls: (...args: {}[]) => T, xtn: K): new (...args: {}[]) => T & K

Usage:
var NewClass = extend(MyClass, {
  init(a) {
    this.doSomething();
  },

  // ....
})

The NewClass will still have all expected method init ..., but the this cannot be inferred inside the extension object.
It would be great if there is a way to instruct Typescript how to infer the this inside extension object based on the first passed param meter. (MyClass in the example). The reason is to have better intellisense.

We actually shipped this in TypeScript 2.3 to support APIs like those in Ember and Vue.
See #14141.

For the record, you probably want
extend<T, K>(cls: (...args: {}[]) => T, xtn: K & ThisType<T & K>): new (...args: {}[]) => T & K

It looks awesome. But I couldn't make it work. Tried adding noImplicitThis to the jsconfig but i got warning that it is not a valid config.
And makes me wonder if the following is doable
interface Observable {
  /**
   * Very long long long comment to explain how to use `options`
   */
  on(event: string, handler: Function, options?: object): Observable
}

interface MyClass extends Observable {
  // ...
}

We know that using new MyClass().on(...) will return MyClass instance, but how to tell Typescript to return MyClass instead of Observable. I don't wanna rewrite all the comment on the overload method on MyClass interface.

interface Observable {
  /**
   * Very long long long comment to explain how to use `options`
   */
  on(event: string, handler: Function, options?: object): this
}

interface MyClass extends Observable {
  // ...
}


[WIP] PUT/GET/DELETE lifecycle configurations

What this PR does / why we need it:
This PR is to support the APIs of PUT/GET/DELETE lifecycle configurations, lifecycle schedule and data migrations are left behind.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
This PR will fix a part of #694
Special notes for your reviewer:
When DELETE lifecycle is called, all lifecycle rules of the bucket will be deleted, this is different from the last version, we changed it to be like this because this is compatible with AWS S3. And if the user wants to delete a specific rule, the PUT API can be used.
Release note:

it is fine, LGTM

With respect to changes to lifeCycle's delete function, dashboard also needs to synchronize the changes to ensure that the delete function is used properly.


BatchInsert with more than 1000 rows

I use MSSQL and get the error:
The number of row value expressions in the INSERT statement exceeds the maximum allowed
number of 1000 row values.

To solve the problem we have two solutions:

Don't delete all rows, just delete the not anymore used ones and insert just the new ones.
Do a array_chunk($junctionRows, 1000) and then the batchInsert on the arrays.

I would recommend to implement both solutions.

Yes, I know about this problem. In developing the second version which doesn't pass the tests and cannot be released. There is a desire, see branch "linker". I will be glad for your help.

As far as I can see it in the new linker version the problem is just that the default value is not given to the OneToManyUpdater.

Hi! Released the new version of behavior. It's already available for use on the link Yii2 Linker Behavior.
In the new version solve your problem easier.


Railway Platform tagged as way displays as footpath

When a way is tagged as "railway=platform", it shows like it is a footpath.

If tagged with area=yes, it fixes

Yes, I did this by design in e55f680 - you don't like it?
My reasons were because, 1. the old rendering was very hard to see, 2. people should connect footpaths/sidewalks/steps to the platforms.  Platforms are really part of the pedestrian network.
I'm definitely open to changing it though.

I agree that it was quite hard to see before, but I also think that that design should be kept for footpaths.
Maybe the we change the colour slightly, to something that is not already used.
It's just that it confused me at first, as I thought that somebody had tagged this incorrectly as a path.

Maybe the we change the colour slightly, to something that is not already used.
It's just that it confused me at first, as I thought that somebody had tagged this incorrectly as a path.

Ok that's fair.. Along the lines of #4157, people like special rendering to distinguish between "the default thing" and "a special thing", so this would be similar.  We can't support every kind of feature in OSM having it's own unique rendering, but we can use color to indicate that a feature is interesting.

Sounds good, thanks!

The platforms are slightly different from the sidewalks now..


Regression: NavigationAgent doesn't calculate path since 3.5-rc2

Godot version
3.5-rc2
System information
Arch Linux AMD Ryzen 2500U GLES3
Issue description
I've noticed when developing a game that with version 3.5-rc2 navigation agents no longer move to target location at all while in 3.5-rc1 build they are capable to do so. I'll use this github repo as an example since it's one of the projects affected by the regression. I have other project as well that is affected by the issue but I can't share the code for it.
https://github.com/warriormaster12/FinalStand
Steps to reproduce

Open the project
Launch game
character should move but it doesn't with rc2 build, rc1 build still works.

Minimal reproduction project
No response

Found the PR that changed things: #61235


Flake8 rule 'W291' reported an invalid point

Flake8 reported an invalid point for the rule W291, with the messge trailing whitespace.

Debug information:
Atom version: 1.14.1
Flake8 version: 3.3.0 (mccabe: 0.6.1, pycodestyle: 2.3.1, pyflakes: 1.5.0) CPython 2.7.12 on Darwin

Are you able to share the file that triggers this? Or can you come up with a minimal reproduction case?

Closing due to a lack of response. If you can provide a file to reproduce this I'd be happy to investigate the root cause!


Imbalanced source actors' throughput may cause trouble when there is watermark

This is a simple Nexmark q10 query with 12 parallelism on a single computing node with 16 CPU.
The number of records in each Kafka partition:

The throughput of each source connector:

If the speed of source connectors does not match with each other,
and if the job has event time and watermark,
then the job will probably throw away more events than usual
because one partition may make the watermark progress quickly
and thus other partitions' events get discarded more aggressively.
Some random ideas:
throttle some source actors by considering both the current throughput and the value of the event timestamp.

Some more context
The plan is:
StreamMaterialize { columns: [auction, bidder, price, date_time, date, time, _row_id(hidden)], pk_columns: [_row_id] }
   StreamExchange { dist: HashShard(_row_id) }
     StreamProject { exprs: [Field(bid, 0:Int32), Field(bid, 1:Int32), Field(bid, 2:Int32), Field(bid, 5:Int32), ToChar(Field(bid, 5:Int32), 'YYYY-MM-DD':Varchar), ToChar(Field(bid, 5:Int32), 'HH:MI':Varchar), _row_id] }
       StreamSource { source: "nexmark_source", columns: [event_type, person, auction, bid, _row_id] }

But both StreamMaterialize and StreamExchange do nothing in this case as their code of processing data chunk is commented out.

throttle some source actors by considering both the current throughput and the value of the event timestamp.

Should it be applied to NexMark source only or all kinds of sources?

Currently it does not block doing benchmark on NexMark sources,
I feel it should be applied to all kinds of sources as long as the job involves watermark?

Another example:

No matter high throughput or low throughput, both are quite stable.

Will the situation be better after #6170? I'd like to try it out.

The throughputs are at least really balanced if 3 nodes running on the same host. 🤔

The setting I tested is a single-node setting, so there should be no remote exchange but only local exchange.
I am uncertain whether this should be expected to be solved in #6170?

The setting I tested is a single-node setting

That's weird. 😕

The setting I tested is a single-node setting

That's weird. 😕

No worries, let me track this issue manually when we have the performance dashboard

Related: https://github.com/risingwavelabs/risingwave/issues/8451#issuecomment-1475819678

Grafana: https://g-2927a1b4d9.grafana-workspace.us-east-1.amazonaws.com/d/EpkBw5W4k/risingwave-bench-dashboard?orgId=1&from=1679413185000&to=1679414780000&to=&var-namespace=nexmark-bs-0-1-2-3-4-daily-20230321

seems pretty easy to reproduce, almost every day recently

Per some discussions with @liurenjie1024 and @hzxa21, I tend to think that this is an issue of Kafka (server or client, both possible) when the workload is heavy. 😟

When testing with Nexmark datagen sources, I cannot observe any imbalance. This is feasible as it's not natural to write some codes to make the performance of actors not balanced, if there's no interference with each other.
There's also no Tokio's I/O in this case, as there's no remote exchange, and rdkafka is also driven by its own async poll interface.

I'll keep investigating if I get a new idea, but I guess we need some metrics of the Kafka cluster in order to make things clearer. 🤔

The EBS bandwidth of the Kafka machine is likely limited.

The maximum bandwidth of GP2
and
the default bandwidth of GP3 are both smaller than the 298MB/s shown in the figure.
Regarding why the throughput during the second half does not go up, probably because there are some extra IO credits at the beginning, and then we need to pay them back. This is hust a guess.

But the weird thing is that if looking at the history of q2, it is less severe than q0
q2 does more filtering than q0.
The good news is that q2 can achieve 1M rows/s if this phenomenon does not happen.
And also the client of Kafka since Flink seems not to suffer from the same problem. It uses Java native client while rdkafka is an independent implementation.

But the weird thing is that if looking at the history of q2, it is less severe than q0
q2 does more filtering than q0.

I took a quick look and found that this happens randomly for all queries that reach the throughput of around 800k rows/s, like q0 q1 q3 q10 q22. All imbalances occur when the bytes per sec is under 300 MB/s.

Can we get the split distribution of each actor? may be we can see the information from the logs.

Can we get the split distribution of each actor? may be we can see the information from the logs.

The splits are assigned equally. See the legends of "Source Throughput Per Partition". 🤔

Oh, actually different from #15142, as the splits are assigned equally here while assigned unequally in #15142


Linked table reverts back to original fields

This is a great extension - one I really felt would add tremendous value for our users.  Things seemed to be working OK in the development work area, but after publishing to a QA stream, I'm noticing a big issue: when I start using the control panel to remove fields and/or add new fields to the table, after 15-20 seconds, the table will flip back to the original set of fields.  This basically kills the usefulness of the extension - I'm guessing users would get pretty frustrated if the table keeps reverting back to its original state.  Since no one else has reported this issue, I'm wondering if there's something I'm doing wrong - so any help or feedback would be appreciated.
Thanks,
Steve

Hello Steve, it was referred by unsould a while back (https://github.com/rvspt/QuickAdhoc-QS/issues/4). My first guess is that this is related to the app being published and, once that happens, the sheet cannot be changed.
Try this one: duplicate the sheet on the published app and see if the issue persists with the copied sheet. Should work well when it is brought to the "personal area".


I've created a discord server for Sokol

Hello,
I've created a discord server for Sokol, as it doesn't seem it's already exist.
I'm rather IRC, but the new kids on the (code)block seem to like better discord, so let's get modern.
@floooh
If you don't agree with that, tell me and i will remove the server.
If you want i can transfer you the admin role.
The invite link is:
https://discord.gg/7f8Ak6zm

Go ahead but please don't expect me to get involved ;)
I will probably open up the Github Wiki and Discussions feature in the future once using Issues for questions becomes to messy, but I'm really not a fan of using a chat platform for this type of stuff.

Yes i understand.
I prefer the classic forums that have a greater retention, but it was more work to create one.
Let's see what it turns out !

@jeangit your invite link has expired, could you share it again please?
I think an unofficial discord would be cool as well. At very least, some place where people are free to talk about Sokol.
The only chat I see for this is on Twitter, and it's like 1 post every 2 weeks by 1 person.

Sorry @brytonsf , the chan has been removed due to lack of activity. May be IRC will be better, next time :)