Use RxJS6 as main RxJS lib instead of needing to download rxjs-compat library

Dear @Peshou ,
is it possible to use rxjs 6 instead of rxjs 5 for this package? Currently, if you want to use rxjs 6 you will need to add the compat lib.. Wouldn't it be better to use rxjs 6 as default and use a compat in case someone needs to use it with rxjs 5 instead? because sooner or later one will need to switch to rxjs 6.
what do you think on this?
All the best and cheers,

Hey @johannesschobel, yes, it is better to use rxjs 6, but for the need of backwards compatibility for the release with premade shapes, I needed to ignore the rxjs change and allow all people to have access to the package with lower angular / rxjs versions, since changing the rxjs version would make me create a breaking changes version.
For the next release, this should be reversed, and the core should use rxjs 6, but everyone else should use rxjs-compat for the sake of old versions and compatibility.

yeah, but the 2.0 version was already a major release - therefore, i thought it would be good to introduce this "breaking change" (switch rxjs) in this course..
the next release (3.0) will be most likely some time away :disappointed:

Yeah, the problem is that if the canvas is using rxjs^5 and you are using rxjs^6, then you can use the compat lib and the canvas will work, but if it's the other way around, you will not be able to use the canvas, since your Angular rxjs^5 app will not be able to compile since it will not know where the imports to the rxjs 6 operators are and such.
That's why if the rxjs version is to be changed, then all the older apps and platforms will not be supported from that version onward.
For now I wanted to allow everyone to use the library. At some point soon I should completely rewrite the imports so that rxjs^6 will be used.
I will also change the name of this issue so that it describes the problem better.
Best Regards,
Stefan

Using rxjs-compat fixes the problem, sort of.
No Child Left Behind often also results in Every Children Left Behind.
Having to use rxjs-compat sounds like a penalty against developers who could always keep the dependencies up-to-date, in this case with rxjs 6.
In software releases, we can do better.
I myself also write components/tools for developers. So in the repository, I tagged the component with something like ForAngular4.2 and ForAngular5, especially for breaking changes in my component and major dependencies. And in Nuget, keep the older releases, and in the Readme.MD file, remind application developers to use older releases if they are still using older versions of Angular.
It is generally advised to keep dependencies up-to-date for many good reasons. For those who can't enjoy the benefits of Angular 5 or 6 and upcoming 7, they should be able to use the older releases of ng-white-board in npm, along with older versions of Angular.
Don't you think this may be a more optimal balance?

Yesterday I had spent hours trying to make it work in my app on NG6, with rxjs-compat. Today, I try this component on NG6 Heroes Demo, not working either, then try it on NG5 Heroes Demo, then this component is working right away. Your @Peshou component is exactly what I need.
I told my boss I may spend 1 day to fix this. And if not progressing well, I would have to wait for your working of moving to NG6.
So today, I will do something on the fork. And if progressing well, I may create a pull request. Or, if you @Peshou is doing similar thing right now, I would wait.

OK. I had created a fork and make this component wok well with Angular6/RXJS6, so far so good working in the Heroes NG6 demo. When this fork works well with my app, I will push it, likely next Monday.

It would be awesome if you could create a PR here

I had created a fork at https://github.com/zijianhuang/ng2-canvas-whiteboard
I don't think the fork is in production quality since I don't know how to bundle a module into a npm package, and I don't have time to learn such skills in near future. However, you @johannesschobel can still use the TS codes directly I am doing right now, and in the future you may still switch to the new release of ng2-canvas-whiteboard which then will support NG6.
And the fork works well now with my NG 6 app with Angular 6.1.7. I got stuck a little bit. While I had made the fork work well with the Heroes demo program on last Friday, it does not compile well with my app and the compiler kept complaining about Map<K, U, T> which I am not using such generic type anywhere in the app or in 3rd lib. After spending hours, I found that the Azure Storage js lib has Map<T..> defined but very different from Map<K,U> used in ng2-canvas-whiteboard. So I had hacked the type definition file of Azure Storage js lib a little bit, ng2-canvas-whiteboard works happily with my app, no rxjs-compat lib needed.

Hey @zijianhuang , thank you very much for your time and effort.. However, wouldn't it be better to specifically target the issue in this repository (e.g., create a new ng6 branch)? What do you think @Peshou ?

@johannesschobel , only core member of the repository could create a branch, while other contributors like me could create forks and create PR. I made the fork so you can use the improvement with NG 6 before @Peshou could have time.

yeah, but you can provide a PR and send it to master branch.. @Peshou can then move the PR to another branch ;)

just done a PR, though I was expecting people just copy/paste from my fork. :) if they have urgent need for their Angular 6 app and dislike rxjs-compat.

@Peshou, from my dim memory, npm tool chains allow detection of dependencies. For example, if ng2-canvas-whiteboard 3.0 package with npm will check if other dependencies still include Rxjs 5 and Angular 4.x. If so, just raise a warning or stop installation of v3 if the programmers don't read the release note.

Okay the new version is v3.0.0, the new branch is origin/ng6. Please import it try it out and tell me if everything is okay since i have changed the way that the package is built, but using it should be the same as before for the end users.

Okay let me do that first and then I can publish the library version change

I will just add rxjs^6.0.0 as a peer dependency, so that will be the warning that's going to be raised.

Okay the version is 3.0.1, check it out please, the way the module is built was changed, but for end users everything should be the same, please check it out and give me some feedback if all is okay


enable css customization of graph based on node distance

Hey @treymo , is  there anything I can add or clarify?

One alternative: The case with maxDegree > 0 currently is called selection-based graph in code.
After having further look, I am wondering if it is better named "distance-based graph". What makes more sense for you?

I prefer distance-based graph personally.   Sorry for the delay, been very busy but I will review ASAP


"govc: please specify a datacenter"

Hello,
while running the playbook we encounter the following problem during the task "TASK [static_ips : Upload all the custom generated ISOs to the datastore]"
Problem:
/ocp4-vsphere-upi-automation/downloads/ISOs/okdmaster03.iso", "okdmaster03.iso"], "delta": "0:00:00.025456", "end": "2020-09-21 16:31:38.623916", "item": {"ipaddr": "OUR IP", "macaddr": "OUR MAC", "name": "okdmaster03"}, "msg": "non-zero return code", "rc": 1, "start": "2020-09-21 16:31:38.598460", "stderr": "govc: please specify a datacenter", "stderr_lines": ["govc: please specify a datacenter"], "stdout": "", "stdout_lines": []}
We encountered the problem on a centos 7.7 machine aswell as a centos 8.2 machine. Both patched to the latest version of ansible.
We defined the datacenter in the /group_vars/all.yml file. Which doesn't help. The datastore which is also defined in the file works flawlessly.

Can you share your group_vars/all.yaml?

Hey mallmen,
sure. I think the vcenter part will be enough?
vcenter:
ip: OUR_VCENTER
datastore: OUR_DS
network: OUR_LAN
service_account_username: OUR_USER
service_account_password: OUR_PW
admin_username: OUR_USER
admin_password: OUR_PW
datacenter: OUR_DATACENTER
folder_absolute_path:
vm_power_state: poweredon
template_name: rhcos-vmware
I fixed the playbook bei adding the govc_env to the static_ips.yaml and referenced the datacenter var from the all.yaml
environment:
PATH: "{{ playbook_dir }}/bin:{{ ansible_env.PATH }}"
GOVC_USERNAME: "{{ vcenter.admin_username }}"
GOVC_PASSWORD: "{{ vcenter.admin_password }}"
GOVC_URL: "https://{{ vcenter.ip }}"
GOVC_DATACENTER: "{{ datacenter }}"
GOVC_INSECURE: 1

Added with PR 54


Float calculation error

Please explain someone, why this calculation is not matching
select  case when (convert(float,2.8) + convert(float, 56.04)) <>  convert(float,58.84) then 1 else 0 end
this case retrun 1, that means not equal !! but if calculate result is 58.84
select (convert(float,2.8) + convert(float, 56.04))  --- 58.84
select  case when (convert(float,2.81) + convert(float, 56.03)) <>  convert(float,58.84) then 1 else 0 end
this one retrun 0

@felixkjoseph -- Felix, that is strange!  Are you following a particular docs.microsoft.com article?  Meanwhile, please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

Hi @felixkjoseph. This is happening because floating point values are binary fractions that are a close approximation to the decimal value in question. If you want to do arithmetic with non-integer values, you should use the DECIMAL / NUMERIC data type instead of FLOAT.
#please-close

got it, thank you


Can't save Go's time variable to cell

xl.SetCellValue("Sheet1","A1", time.Now())
The excel file will show empty

@dlintw Thanks for you issue.  Function SetCellValue() support time.Time data type parameter now.

@xuri  the same issue exists.


Value of isSubmitting not defined.

Hello guys,
Really nice content. On the Blazor trainning, I cannot finish the @layout part because of this. In the Checkout.razor file, I have a property named isSubmitting for a button :
<button class="checkout-button btn btn-warning" @onclick="PlaceOrder" disabled=@isSubmitting>

and this is obviously resulting in an error there too.
@code {
    Order Order => OrderState.Order;
    async Task PlaceOrder()
    {
        isSubmitting = true;
        var response = await HttpClient.PostAsJsonAsync($"{NavigationManager.BaseUri}orders", OrderState.Order);
        var newOrderId = await response.Content.ReadFromJsonAsync<int>();
        OrderState.ResetOrder();
        NavigationManager.NavigateTo("/myorders");
    }

}

Resulting in an error and I cannot do my dotnet watch.
error CS0103: The name 'isSubmitting' does not exist in the current context [/home/computer/Documents/sidePrj/CsharpProjects/BlazingPizza/BlazingPizza.csproj]

Please shed some lights.

The code section should have the variable declaration:
@code {
    Order Order => OrderState.Order;
    **bool isSubmitting;**

    async Task PlaceOrder()
    {
        isSubmitting = true;
        var response = await HttpClient.PostAsJsonAsync($"{NavigationManager.BaseUri}orders", OrderState.Order);
        var newOrderId = await response.Content.ReadFromJsonAsync<int>();
        OrderState.ResetOrder();
        NavigationManager.NavigateTo("/myorders");
    }
}


toolbar pops out of container

Jodit Version:  3.2.34 (I think)
In my app, I'm using Jodit in containers. When Jodit has enough text that I can scroll up and down, when I expand the window for more horizontal typing space, the Jodit toolbar pops outside of the container.
Expected behavior:
I expect Jodit to stay inside the container.
Actual behavior:
Here's a video showing it popping out.
https://www.loom.com/share/7ae20b95ade14f9eb0baca0ba87d46f0

Hi, you can switch off sticky option
https://xdsoft.net/jodit/play.html?toolbarSticky=false
I will try fix this issue in another version

I would love to have it work properly and stay at the top of the small pane. That would be wonderful.
Oak
From: Valeriy Chupurnov<EMAIL_ADDRESS>Reply-To: xdan/jodit<EMAIL_ADDRESS>Date: Saturday, July 6, 2019 at 3:44 PM
To: xdan/jodit<EMAIL_ADDRESS>Cc: Oak Norton<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [xdan/jodit] toolbar pops out of container (#226)
Hi, you can switch off sticky option
https://xdsoft.net/jodit/play.html?toolbarSticky=false
I will try fix this issue in another version
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/xdan/jodit/issues/226?email_source=notifications&email_token=ABJC6DPBN4WXBDNTKQDZY4LP6EG4PA5CNFSM4H6LDGGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLAXRQ#issuecomment-508955590, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABJC6DIIDQ55XIW6Q5H6NHDP6EG4PANCNFSM4H6LDGGA.

@oaknorton Can you create small html example with problem?

I can’t set that up easily. My developers would have to build a whole test setup for this. They are going to try and force it to stay in the container. I’m not sure if the code would exactly be transferable over to you but if I can share it back to you and it helps, I’ll do that.

I have the same problem. I can not share the source, but I tried to reproduce the error in this schematic example: https://embed.plnkr.co/1dRdysMb1GQXsBeo51Gk/
In this example, the error is reproduced only in the iFrame mode, but in my case the sticky toolbar does not work in all modes. It is version 3.2.65 (but it is still reproduced in latest version)
@xdan, can you check this problem? Hope this helps in finding a bug.


WIP: raise async exceptions

@jamesdbrock Currently the program hangs when the exceptions are raised so this shouldn't be merged in its current state.

@jamesdbrock Would it make sense to pick this up again or is this irrelevant now?

I don't think it's irrelevant, if this is the semantics that we want then it's worth doing.
Right now what we have is “if an exception is thrown in the coroutine then that exception pops the coroutine stack and then stops.”
This PR introduces “if a non-CancelledError exception is thrown in the coroutine then it pops the coroutine stack and then propagates to the main event loop.” Which means the whole app will halt, I think, right? Because there is no other opportunity to catch that exception.
Can you think of an example case where this PR behavior is clearly better?
If we just want to halt the App then currently the best explicit way to do that is use_stop.

So the reason to do this PR would be so that async Tasks would fail obviously instead of failing silently, and that would be a better developer experience?

@jamesdbrock If I remember correctly, the reason for not just letting the async task swallow the errors is that errors weren't properly reported but it caused crashes and the application to hang.
I don't know if there's a case now where such behavior occurs so trying to be more explicit about raising the errors in the engine if a task dies might no longer be necessary.

I just tested raising an uncaught ValueError in a use_async Task and it seems to behave the correct way: The Task ends and the rest of the App is fine. https://github.com/pyedifice/pyedifice/commit/bca91f64aca46a35f7ba21be9b49a4287896c8a5


How to unpack/extract a bundle.js to it's component and js files.

Hi,
Could you please help on unpacking the bundle.js files to it's component and js files?
Regards,
Johib

Please submit support requests and questions to StackOverflow using the tag [webpack]. StackOverflow is better suited for this kind of support. The issue tracker is for bug reports and feature discussions. See also our CONTRIBUTING guidelines.

here we go
https://stackoverflow.com/questions/44276097/webpack-how-to-extract-bundle-to-respective-components


logo url no longer overrideable

The logo URL was previously controlled by default_url, but #168 has hardcoded the value to the nbclassic tree view root. As far as I can tell, it can no longer be overridden, as nbclassic_tree is always defined when running with jupyter_server.
I don't quite follow the discussion in #167 - the logo link is meant to go to the default URL for the server, not the tree root which is only the default default URL, but is meant to be able to be overridden. The tree root in the current view already has a link at the root:

This has been reported as a bug in jupyterhub at https://github.com/jupyterhub/jupyterhub/issues/4220 where the expected behavior of setting default_url no longer has the desired effect.

cc/ @mcrutch


How can I prevent SQL injection with epgsql?

Is there any build in functions?

Sure, just use equery or parse/bind/execute and pass any user-input data as parameters.

Great! Thanks!


[WAC] WAC 2110 unable to add "allowed groups"

Hello,
I've just installed WAC 2110 (on a Windows 2019 Server) in order to test it.
But I'm unable to add "allowed groups" under "Access" menu to manage WAC Gateway Access
There's simply no "add" button
I would try to install the previous version to test it, but i'm unable to find it to download
Regards

Hey @inferno66,
Can you please help me understand? Where is this within Windows Admin Center?

Hello
This is under "Access" Menu ==> "Gateway Access"
This is where you are supposed to add permissions (admin / read only) to the WAC console with AD Users

I see the same problem in multiple environments.
Maybe this patch has something to do with it?
https://support.microsoft.com/en-us/topic/november-14-2021-kb5008602-os-build-17763-2305-out-of-band-8583a8a3-ebed-4829-b285-356fb5aaacd7

Anyone try the patch or confirm that there is a fix for this?  Seeing this in a new install and it's driving me insane.

We are aware of this issue and our dev team is currently working on a fix for it.

We are aware of this issue and our dev team is currently working on a fix for it.

Any ETA?

Any update to this? Unable to modify access for WAC installation.

Do you know when it will be fixed? Or Is there an official older version of WAC available?

Any updates when a fix will be available?

And 2 months later there still is no update or patch... 😢

I'm a little upset that not only has this been a known issue for two months, but I had no warning it was an issue until after I updated my instance and locked out my coworkers. Why is this version still the download you're offering? I'm extremely unhappy.

@trungtran-msft - Can you please provide us with an install link for WAC 2103.2 here? I haven't had much luck finding it, and I need to downgrade until this issue is resolved. I have two other instances that auto-updated.

@MWRobertson try this
https://download.microsoft.com/download/1/0/5/1059800B-F375-451C-B37E-758FFC7C8C8B/WindowsAdminCenter2103.2.msi

Hi WAC team.
Would love to know if there's an ETA on a bug-fix for this.
Thanks.

3 month and problem is still here.

Trying to set up Windows Admin Center for the first time on a clean Windows 2022 Server. Encountering the same issue.

Hello everyone, thank you for your patience, this fix is taking longer than expected. If anyone here is interested, once we go to public preview, I can send you the build with the fix for you to try out before the official GA date.

Hello @trungtran-msft ,
Put me on the list! 😉

@trungtran-msft
I'd also be happy to test out the preview build please. Thanks!

Hello,
Add me also to the list please :)

Add me to the list also. Same problem!

Yes please
From: trungtran-msft @.>
Sent: 16 February 2022 01:12
To: MicrosoftDocs/Windows-Admin-Center-Ideas-and-Feedback @.>
Cc: Peter Roots @.>; Manual @.>
Subject: Re: [MicrosoftDocs/Windows-Admin-Center-Ideas-and-Feedback] [WAC] WAC 2110 unable to add "allowed groups" (Issue #70)
NOTE: This message originated outside Barnett Waddingham.

Hello everyone, thank you for your patience, this fix is taking longer than expected. If anyone here is interested, once we go to public preview, I can send you the build with the fix for you to try out before the official GA date.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/Windows-Admin-Center-Ideas-and-Feedback/issues/70#issuecomment-1040980626, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJ7TDRO2KT7FZ6J27TKLDDU3L2XTANCNFSM5HSGFFXQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>
Protect yourself from cyber fraud.
If you receive an email that appears to come from Barnett Waddingham providing
different bank details or informing of a change of account, it is unlikely to
be genuine. Do not reply to the email or act on any information contained in
it and contact us immediately by phone to check the account details. We will
not accept responsibility if you transfer money into an incorrect account.
Barnett Waddingham LLP is a body corporate with members to whom we refer
as “partners”. A list of members can be inspected at the registered office.
Barnett Waddingham LLP (OC307678), BW SIPP LLP (OC322417), and Barnett
Waddingham Actuaries and Consultants Limited (06498431) are registered in
England and Wales with their registered office at 2 London Wall Place, London,
EC2Y 5AU. Barnett Waddingham LLP is authorised and regulated by the Financial
Conduct Authority. BW SIPP LLP is authorised and regulated by the Financial
Conduct Authority. Barnett Waddingham Actuaries and Consultants Limited is
licensed by the Institute and Faculty of Actuaries in respect of a range of
investment business activities.
This message and any attachments are confidential and are solely for the
individual to whom they are addressed. Any use of, or action taken in reliance
upon this information by any other person is prohibited. If you received this
information in error, please contact the sender immediately and delete the
material from any computer that may contain it and destroy any printed copies
you may have made. We have taken all reasonable precautions to ensure that no
viruses are transmitted with this message. However, we can accept no
responsibility for any loss or damage resulting directly or indirectly from
the use of this e-mail or its contents. Incoming and outgoing e-mails are
subject to scanning and may be intercepted by our e-mail administrators.

@trungtran-msft Can you also share the public preview build with me? Thanks!

On Wed, Feb 16, 2022, 8:07 AM Paul Boyer @.***> wrote:

@trungtran-msft https://github.com/trungtran-msft Can you also share
the public preview build with me? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/Windows-Admin-Center-Ideas-and-Feedback/issues/70#issuecomment-1041527237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE34CSPPE7O3MCPPTFVXOG3U3OVRRANCNFSM5HSGFFXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
<MicrosoftDocs/Windows-Admin-Center-Ideas-and-Feedback/issues/70/1041527237
@github.com>

If approved, I can post a link here. No need to spam the comments. I'll send an update once we have a build for public preview.

Hello everyone, thank you for your patience. Please understand that this build is not supported. We are still working on fixes and validation before the official GA.
Here is the build with the RBAC fixes: https://microsoft-my.sharepoint-df.com/:u:/p/praror/EV_E1yve_b1KvHxkpTE3_YkBGfda_SMqMVCkNibjEi6h_g?e=4IEzpg&isSPOFile=1

Any news on this?

The GA is not out yet, but the Public Preview can be found here:
You can read about it and download the build here: https://techcommunity.microsoft.com/t5/windows-admin-center-blog/windows-admin-center-version-2110-2-is-now-in-public-preview/ba-p/3263839
Please try it out! :)

@trungtran-msft I'm having trouble trying to download from the Public Preview site. It's stuck at the "Validating your request" modal. Confirmed this is the case in two different browsers and two different PCs. Can you post a direct download link? Thanks!

@prboyer That's odd, are you part of the Windows Insider Program? Also, did you make sure you were signed in as well?
I tried the link again but it worked for me.


Update 1.13 version in UI to 1.13-dev

High-level description

Reset 1.13 version in UI for 1.13 branch back to 1.13-dev

1.13 branch will be long living 1.13-dev branch (like 1.12-dev: https://github.com/dcos/dcos/blob/1.12/gen/calc.py#L40) and when 1.13.1 is released, the minor branch will see the update of the version number.

@mesosphere-mergebot label ready for review

I would like to refer to my comment in https://github.com/dcos/dcos/pull/5496.


Configure continuous deployment from CircleCI to Heroku

...as soon as we are confident enough in our tests :)

Done.


feat(*): replace /bin/bash with /usr/bin/env bash

/usr/bin/env bash is more flexible and standardizes the header across the project.

Code LGTM.

Code LGTM


feat: move agents to new tile

Description

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

See comments. Can you also:

add a .catalog-onboard-pipeline.yaml (Ill share the offering Id after next commit to this PR)
update .releaserc to trigger the catalog pipeline

updated and added missing scripts

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@jor2 you need to resolve conflicts before pipeline can be triggered

/run pipeline

/run pipeline

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

GitHub release
v1.4.0

Your semantic-release bot :package::rocket:


have no filter isoform gtf files

-M APDF the result files only have a part of them.

install abPOA failed.

Hi,
When did you pull your Mandalorion version? I just did some updates over the weekend and it would be helpful to know.
There might be multiple issues. It seems like the P module isn't running properly because you don't get your files converted to psl. Could you send the terminal messages from running just the -M P so I can figure out what is going on here?
The second issue is the abpoa installation. I'm not an expert on abpoa. We just pull and use it. But from quickly googling the error, it seems to be an issue with the compiler. You could either update gcc and try again or add
CFLAGS=-std=c99
to the make file and try again. If we can't figure this out, I'm sure the abpoa people can help.
Best,
Chris

and the software abPOA is installed fail contribute to the results,I don't know how to make successfully.

have been soluted.Thank you.

have no gif file

seem  P mole is error

Thanks for soluting the questions!

Hi,
the error in P is just a warning. I does break anything so it should be fine.
Do you still have no gtf? It's not clear right now where things are breaking or if they are still breaking.
Best,
Chris

have succeeded, thank you very much.


[PRCI] Handle (re)fetching request status

(actually on line 116, so to solve later on)
Making a request every 3 seconds is a very bad approach here IMHO. It won't scale. Remember you should always try to make as few requests as possible. This request should be triggered by the user, directly (i.e. reload button) or indirectly (i.e. loading the view) to determine with @Clementinev. Alternatively, you could use a web socket if really necessary, but I don't think it is here (to discuss with @m4rk055).

Originally posted by @noelmace in https://github.com/blindnet-io/privacy-components-web/pull/87#discussion_r961398410
We must choose a better way to fetch the requests list and statuses for demands in a request. Options are:

Only fetch when the request list or demand status view is loaded
Include a reload button
Web socket

I think (2) is a better choice than (1), because:

If we don't include a reload button, a user that wants to reload the request or status page will just navigate within the component (e.g. back to the menu and then return to the requests page) to achieve a reload, which is a bad user experience
We can make the reload button clickable once every X seconds to  prevent the user from flooding the server with requests by spamming the button

As for (3), I've only done limited work with web sockets but I think the reload button is a good enough option, so web sockets aren't worth it, unless we really want the pages to update content without any user interaction. @m4rk055 thoughts?

Agree, web sockets are not worth it in this case. Users won't spend much time on the page (and in the future, they will get an email message about the request being finished).
Option (2) sounds good to me.


⚠ Fortnite has degraded performance

In a61ae75, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1615 ms

Resolved: Fortnite performance has improved in 86bc5ae.


Create nsutel

e95abe94b32a36d852149f61d484ecf1-1556013ee5f1e1530499214852a0feb6d750e4c6.zip

Nsutel is a name just working to brand this name

Nsutel is a name just working to brand this name


layout 2

grid based
sub-components: Layout, LayoutHeader, LayoutContent... etc

https://github.com/abernier/poc-pmndrs-docs-layout

https://github.com/pmndrs/docs/pull/326


Creating a new process is slow and resource heavy

this is regarding the old flyt 1.0
should do a new load test on flyt 2.0
this is to be closed

Close this issue according to Andreas comment above.


Polymur32?

I am wondering if we can construct a 32-bit hash function with 2³¹ - 1 instead of 2⁶¹ - 1. On 64-bit processors it would not be worth doing it as one will do hash ^ (hash >> 32) or just use a half of the result instead, but it might be nice on 32-bit processors? Any thoughts?

Unfortunately, the collision probability for n bytes is not 2^61 -1, but on the order of n * 2^-60.2. The collision probability linearly degrades with the input length, this is a property of all variable-length universal hashes.
This is (in my opinion) not a problem for reasonable string lengths for 2^-60.2 as a starting point, because we have so much headroom. The same can not be said for 2^31-1 as a starting point.
I am working on another hash function that will be much faster for large inputs by using a two-stage algorithm that feeds a fixed-length universal hash into the polynomial hash. This variant could potentially support a 2^31-1 collision level more reasonably, but it is still dicey.
What 32-bit processors are you thinking of using this on?

Thank you for your kind and prompt response!

Unfortunately, the collision probability for n bytes is not 2^61 - 1, but on the order of n * 2^-60.2. Note the factor of n: the collision probability linearly degrades with the input length, this is a property of all variable-length universal hashes with fixed entropy.
This is (in my opinion) not a problem for reasonable string lengths for 2^-60.2 as a starting point, because we have so much headroom. The same can not be said for 2^31 - 1 as a starting point.

I see. So, even when the hash value is truncated to 32-bit anyway, P611 would be better than P311.
What if the length of final value is even shorter than 32-bit, i.e. use it with hash tables? While distributed key-value stores use huge indices, hash tables in programming languages' standard library has much less number of entries.

I am working on another hash function that will be much faster for large inputs by using a two-stage algorithm that feeds a fixed-length universal hash into the polynomial hash. This variant could potentially support a 2^31 - 1 collision level more reasonably, but it is still dicey.

Cool!

What 32-bit processors are you thinking of using this on?

My starting point was not actually 32-bit processors but Janet, a scripting language for embedding into applications (like Lua), that use 32-bit integers and thus 32-bit hash functions. However, I can imagine that Arm Cortex-M (RP2040 for instance) or 32-bit RISC-V microprocessors will benefit from such a hash function.

What if the length of final value is even shorter than 32-bit, i.e. use it with hash tables? While distributed key-value stores use huge indices, hash tables in programming languages' standard library typically have much less number of entries.

To be entirely formally correct,  in general to get a universal k-bit hash out of a universal n-bit hash where k < n... you need to hash the hash!
A common such hash is Dietzfelbinger's multiply-shift: generate a random odd number a and compute a*h >> (n - k).
In practice it's super common to just take the hash mod 2^k, which is technically not valid (consider the identity function, it technically is universal hash function without any collisions, yet mod 2^k it is not universal at all). However it being so common contributed to the reason I added the murmur-style permutation at the end, which should make it fine to just take the last k bits in practice.

What if the length of final value is even shorter than 32-bit, i.e. use it with hash tables? While distributed key-value stores use huge indices, hash tables in programming languages' standard library typically have much less number of entries.

To be entirely formally correct,  in general to get a universal k-bit hash out of a universal n-bit hash where k < n... you need to hash the hash!
A common such hash is Dietzfelbinger's multiply-shift: generate a random odd number a and compute a*h >> (n - k).

I did not aware of this, thanks a lot!


Fix a few typos

This fixes a few minor typos. (Using a spell checker on the raw .ipynb still works best in my experience).
Very nice material, thank you for making it available!

Thanks!


"ValueError: too many values to unpack" with SQLAlchemy 1.2.2

When using SQLAlchemy 1.2.2 with Flask-WTF 0.14.2, using a QuerySelectField, Flask-WTF raises "ValueError: too many values to unpack" in wtforms/ext/sqlalchemy/fields.py line 189, in get_pk_from_identity. It appears that SQLAlchemy 1.2 and later returns a third member from that call, and the library only expects two. Here's separate project's PR about the issue, and whose fix may be worth considering:
https://github.com/wtforms/wtforms-sqlalchemy/pull/10
I've put together a small Flask app demonstrating the bug here:
https://github.com/commandtab/flask-wtf-example

That's part of WTForms, not Flask-WTF.

@lepture what do you want to do here? WTForms isn't being updated, but I don't really want to pull this into Flask-WTF.

@davidism Oh, silly me! I should have seen that it was an issue directly in WTForms.
What do you mean when you said WTForms isn’t being updated? Is the project no longer maintained?
Thanks!

I don't have access to make releases, only to merge stuff.

Hmm. Should I open a similar issue or PR over in WTForms for someone to review and merge, then a project owner to release?

You can, I can review and merge it at some point when I have time.


hack: add dev scripts

Adding some frequently used commands to Makefile under hack

can we remove the stuff put inside e2e since they are not in our CI yet?

lgtm


Create LICENSE

This pull request is part of an initiative to make the licensing information about OBO Foundry ontologies more actionable.
While this ontology contains licensing information in its metadata, GitHub is not aware of the license unless you create a LICENSE file with standardized text. This PR creates a LICENSE file with the standard text for the CC BY 4.0 license.
More information about this initiative and further discussion is taking place here: https://github.com/OBOFoundry/OBOFoundry.github.io/issues/1840.

This was what was written in the OBO foundry

Fine for me


[EPIC] Custom base project

This epic will update the base structure created from the Android Studio

Add theme day/night
Add splashActivity
Add package structure
Add icon app images

This epic will update the base structure created from the Android Studio
Add theme day/night
Add splashActivity
Add package structure
Add icon app images

This epic will update the base structure created from the Android Studio

[ ]  Add theme day/night
[ ]  Add splashActivity
[ ]  Add package structure
[ ]  Add icon app


[Feature][seatunnel-transforms-v2][Sql] use the built-in visitor mode of jsqlparse

Purpose of this pull request
#5524

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Can someone help me take a look
All tests for transformer-v2 have been passed and are basically usable
If there are any issues, I will modify them as soon as possible
It has certain advantages in extensions and performance

Good job.

I also plan to use reflection to call SQL functions
Avoid lengthy case when statements
When it cannot be called, call udf again

It's best not to tightly couple with the jsqlparser parsing engine in ST, as there's a possibility that we may switch to Calcite in the future to generate physical execution plans for implementing complex calculations such as join/union

I think it's unlikely that two parsing engines will appear simultaneously in a project
should decide what to use from the beginning
Union/join jsqlparse can also capture token streams
Just temporarily not supported, so an exception was thrown
The specific data operations still need to be written by oneself

will still need this feature? I have time to modify it

Rewrite Sql Function:

Using enumeration types instead of 'case when'
Check the number of input parameters for SQL functions
Ensure a certain degree of type safety
Provide some implicit conversion capabilities
Rewrite calculator logic and use BigDeciall for calculations


Unable to make the app running ok

Hello
The app deploys and I get the screen where I can give the details about the patient  I want to score.
However when I click score I get an error saying that the app cannot find the model.
here it is :
Failed to score drug1N, msg=Can not find model:eu-gb$90c9c2c0-848d-4580-905d-9e2cfa466680/drug1N, details:java.lang.RuntimeException: Can not find model:eu-gb$90c9c2c0-848d-4580-905d-9e2cfa466680/drug1N at com.ibm.spss.internal.scorer.pool.ScorerPoolImpl.get(ScorerPoolImpl.java:96) at com.ibm.spss.blackbox.score.resource.ScoreUtil.score(ScoreUtil.java:191) at com.ibm.spss.blackbox.score.resource.ScoreResource.scoreWithInputdata(ScoreResource.java:69) at
I have added the model Drug1n.str thru the service dashboard
I have modified the app.js file with my credentials
var defaultBaseURL = 'https://palblyp.pmservice.ibmcloud.com/pm/v1';
var defaultAccessKey = '====EbjWYrA472TtheuyABbgmIv4Gf3OM3oS964AoaiK9b6iNjiik+2n997pvelDBj2EWArRQzCnErs5G6xF7OPG2R5H0oB=====';
Can you help ?
Thank you very much

Following the read.me notice about the context.id name, everything runs ok
Why is the context id so specific ?  is it referenced in the app somewhere  ?
Prerequisites
Model available in \model subdirectory (Drug1n.str) has to be uploaded to the Watson Machine Learning instance using Dashboard (drop box). Make sure that uploaded model is named correctly (contextID=drug1N).
Drug1n.str --> drug1N

@PhilippeComte the Context Id is hardcoded in public\js\app.js


Fix wrong panel link

Link to /jobs results in 404, the working link is http://okfn.org/about/jobs.
I don't know if the error is the location of the page or the link to the location, I'm assuming it's supposed to follow the same pattern as nearby links in the header panel, so I'm submitting a fix for the link.

Thanks! I see that you found another copy and fixed there too. I pulled into this repo with https://github.com/okfn/opendefinition/pull/144 because I was tired of and embarrassed that PrivacyBadger blocks styling and buttons the site uses/recommends and couldn't figure out how to find right person to get assets.okfn.org (now a.?) to stop sending unnecessary cookies. If anyone ever figures this out probably right thing to pull back out of this repo.


Error pyvista update and imod.visualize.grid_3d()

In GitLab by @betsyromero on Dec 1, 2020, 10:46
After updating pyvista to the latest version 0.27.3 and trying to use imod.visualize.grid_3d(), the following error comes up:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-2-6a415f7c0e94> in <module>
     12
     13 # Convert to pyvista 3D grid.
---> 14 grid_3d = imod.visualize.grid_3d(conc)

c:\src\imod-python\imod\visualize\pyvista.py in grid_3d(da, vertical_exaggeration, exterior_only, exterior_depth, return_index)
    483         )
    484
--> 485     grid = pv.UnstructuredGrid(offset, cells, cell_type, points)
    486     grid.points[:, -1] *= vertical_exaggeration
    487     grid.cell_arrays["values"] = values

~\AppData\Local\Continuum\miniconda3\envs\coastar\lib\site-packages\pyvista\core\pointset.py in __init__(self, *args, **kwargs)
    543             if all([arg0_is_arr, arg1_is_arr, arg2_is_arr, arg3_is_arr]):
    544                 self._from_arrays(args[0], args[1], args[2], args[3], deep)
--> 545                 self._check_for_consistency()
    546             else:
    547                 raise TypeError('All input types must be np.ndarray')

~\AppData\Local\Continuum\miniconda3\envs\coastar\lib\site-packages\pyvista\core\pointset.py in _check_for_consistency(self)
    670         else:
    671             if self.n_cells != self.offset.size:
--> 672                 raise ValueError(f'Size of the offset ({self.offset.size}) '
    673                                  f'must match the number of cells ({self.n_cells})')
    674

ValueError: Size of the offset (151186) must match the number of cells (151185)

In GitLab by @JoerivanEngelen on Jan 4, 2021, 15:21
I could not reproduce this issue at my local machine, but after some discussing with Betsy, I think the following happened:
First pip install pyvista vtk was called, this installed two compatible packages (v0.25 and v8.2).
Then a while later pip update pyvista was called. This updated pyvista to v0.27, but still lead to an old version of VTK being present (v8.2). This lead to a dependency conflict that PyVista thinks it can handle, but apparently it cannot (?).
Unfortunately VTK did not want to update to 9.0 in this environment, without throwing conflicts, so Betsy's environment appeared to be broken. The only thing I could advise is creating a clean environment, for example using the environment.yml file in the repository.


New language server not working without a .sln

Without having a .sln file in my project folder, intellisense is not working.
Why am I forced now to have a .sln file in my project ????
Dotnet path: c:\Users\xxxxx\AppData\Roaming\Code\User\globalStorage\ms-dotnettools.vscode-dotnet-runtime.dotnet\7.0.9\dotnet.exe
Activating C# standalone...
info: LanguageServerHost[0]
Starting server...
[Info  - 11:16:33 AM] [LanguageServerProjectSystem]Loaded MSBuild at C:\Program Files\dotnet\sdk\7.0.306
[Info  - 11:16:33 AM] [LanguageServerProjectSystem]Loading d:\Repos\Shop\Shop.Marketplace\Shop.Marketplace.sln...
[Error - 11:16:33 AM] [LanguageServerHost]Microsoft.Build.Exceptions.InvalidProjectFileException: The project file could not be loaded. Could not find file 'd:\Repos\Shop\Shop.Marketplace\Shop.Marketplace.sln'.  d:\Repos\Shop\Shop.Marketplace\Shop.Marketplace.sln
at Microsoft.Build.Shared.ProjectFileErrorUtilities.VerifyThrowInvalidProjectFile(Boolean condition, String errorSubCategoryResourceName, BuildEventFileInfo projectFile, Exception innerException, String resourceName, Object[] args)
at Microsoft.Build.Shared.ProjectFileErrorUtilities.ThrowInvalidProjectFile(BuildEventFileInfo projectFile, String resourceName, Object[] args)
at Microsoft.Build.Construction.SolutionFile.ParseSolutionFile()
at Microsoft.Build.Construction.SolutionFile.Parse(String solutionFile)
at Microsoft.CodeAnalysis.LanguageServer.HostWorkspace.LanguageServerProjectSystem.OpenSolutionCoreAsync(String solutionFilePath) in //src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/HostWorkspace/LanguageServerProjectSystem.cs:line 89
at Microsoft.CodeAnalysis.LanguageServer.HostWorkspace.LanguageServerProjectSystem.OpenSolutionAsync(String solutionFilePath) in //src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/HostWorkspace/LanguageServerProjectSystem.cs:line 80
at Microsoft.CommonLanguageServerProtocol.Framework.QueueItem`3.StartRequestAsync(TRequestContext context, CancellationToken cancellationToken) in /_/src/Features/LanguageServer/Microsoft.CommonLanguageServerProtocol.Framework/QueueItem.cs:line 136

not sure if its SLN issue.
but I also have not sln and the new version is not working
my env:
vs code workspace on wsl (ubuntu) on win10

Same issue here.

Really piss me off.

solved setting "dotnet.server.useOmnisharp": true and restarting vscode
View > Command Palette > Preferences: Open User Settings (JSON)

it works. Thank you

solved setting "dotnet.server.useOmnisharp": true and restarting vscode
View > Command Palette > Preferences: Open User Settings (JSON)

This is a workaround, not a solution. I rolled back to the previous version until there's a fix.
As per the extension documentation:

If you don’t want to take advantage of the great Language Server features, you can revert back to using OmniSharp by going to the Extension settings and setting dotnet.server.useOmnisharp to true. Next, uninstall or disable C# Dev Kit. Finally, restart VS Code for this to take effect.

That's it, just a workaround... but it solves the problem at this moment... :)

This works: https://github.com/dotnet/vscode-csharp/issues/6016#issuecomment-1664305893

Duplicate of https://github.com/dotnet/vscode-csharp/issues/5722
I would recommend enabling the dotnet.server.useOmnisharp flag as a workaround until that issue is resolved (downgrading the extension is also valid).

too add insult to injury, the sln file must be named exactly like the folder.
This makes it unusable for me.


TypeError when trying to use requests.get

Any time I try to use the requests library on my Pyportal since upgrading to CircuitPython 7.1.1 I get the following error:
code.py output:
Connecting to AP...
Connected to WBurn-2G 	RSSI: -50
Fetching text from http://wifitest.adafruit.com/testwifi/index.html
Traceback (most recent call last):
  File "code.py", line 58, in <module>
  File "adafruit_requests.py", line 847, in get
  File "adafruit_requests.py", line 719, in request
  File "adafruit_requests.py", line 210, in __init__
  File "adafruit_requests.py", line 297, in _readto
  File "adafruit_requests.py", line 241, in _recv_into
TypeError: function takes 2 positional arguments but 3 were given

Code done running.

I've tried multiple example files that use the library with the same result. The currently loaded version is  the code below.
# SPDX-FileCopyrightText: 2021 ladyada for Adafruit Industries
# SPDX-License-Identifier: MIT

# adafruit_requests usage with an esp32spi_socket
import board
import busio
from digitalio import DigitalInOut
import adafruit_esp32spi.adafruit_esp32spi_socket as socket
from adafruit_esp32spi import adafruit_esp32spi
import adafruit_requests as requests

# Add a secrets.py to your filesystem that has a dictionary called secrets with "ssid" and
# "password" keys with your WiFi credentials. DO NOT share that file or commit it into Git or other
# source control.
# pylint: disable=no-name-in-module,wrong-import-order
try:
    from secrets import secrets
except ImportError:
    print("WiFi secrets are kept in secrets.py, please add them there!")
    raise

# If you are using a board with pre-defined ESP32 Pins:
esp32_cs = DigitalInOut(board.ESP_CS)
esp32_ready = DigitalInOut(board.ESP_BUSY)
esp32_reset = DigitalInOut(board.ESP_RESET)

# If you have an externally connected ESP32:
# esp32_cs = DigitalInOut(board.D9)
# esp32_ready = DigitalInOut(board.D10)
# esp32_reset = DigitalInOut(board.D5)

# If you have an AirLift Featherwing or ItsyBitsy Airlift:
# esp32_cs = DigitalInOut(board.D13)
# esp32_ready = DigitalInOut(board.D11)
# esp32_reset = DigitalInOut(board.D12)

spi = busio.SPI(board.SCK, board.MOSI, board.MISO)
esp = adafruit_esp32spi.ESP_SPIcontrol(spi, esp32_cs, esp32_ready, esp32_reset)

print("Connecting to AP...")
while not esp.is_connected:
    try:
        esp.connect_AP(secrets["ssid"], secrets["password"])
    except RuntimeError as e:
        print("could not connect to AP, retrying: ", e)
        continue
print("Connected to", str(esp.ssid, "utf-8"), "\tRSSI:", esp.rssi)

# Initialize a requests object with a socket and esp32spi interface
socket.set_interface(esp)
requests.set_socket(socket, esp)

TEXT_URL = "http://wifitest.adafruit.com/testwifi/index.html"
JSON_GET_URL = "https://httpbin.org/get"
JSON_POST_URL = "https://httpbin.org/post"

print("Fetching text from %s" % TEXT_URL)
response = requests.get(TEXT_URL)
print("-" * 40)

print("Text Response: ", response.text)
print("-" * 40)
response.close()

print("Fetching JSON data from %s" % JSON_GET_URL)
response = requests.get(JSON_GET_URL)
print("-" * 40)

print("JSON Response: ", response.json())
print("-" * 40)
response.close()

data = "31F"
print("POSTing data to {0}: {1}".format(JSON_POST_URL, data))
response = requests.post(JSON_POST_URL, data=data)
print("-" * 40)

json_resp = response.json()
# Parse out the 'data' key from json_resp dict.
print("Data received from server:", json_resp["data"])
print("-" * 40)
response.close()

json_data = {"Date": "July 25, 2019"}
print("POSTing data to {0}: {1}".format(JSON_POST_URL, json_data))
response = requests.post(JSON_POST_URL, json=json_data)
print("-" * 40)

json_resp = response.json()
# Parse out the 'json' key from json_resp dict.
print("JSON Data received from server:", json_resp["json"])
print("-" * 40)
response.close()

I'm currently running CircuitPython 7.1.1 Stable with the adafruit_requests library from adafruit-circuitpython-requests-7.x-mpy-1.10.5.zip

I am seeing the same results on PyPortal with 7.1.1 and 7.2.0-alpha.1

Also confirmed same error with version 1.10.4 of this library.

Very strange... Same error on PyPortal with 7.1.0 and 7.0.0

@FoamyGuy could https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI/pull/151 explain this issue? They added a recv_into() method in that PR but it only accepts one argument (buffer), while in adafruit_requests we pass two arguments (buf, and size).

Issue is not present on FunHouse 7.2.0-alpha.1-163-g1cd3faa06. So it does appear to be somewhat limited. That PR in ESP32SPI does look like it could be related.

Confirmed issue is not present with version 3.5.14 of ESP32SPI library so I think you've narrowed it down correctly to that change.


Sharp resizing quits with - exit code -1073741819 (0xC0000005)

Trying to track down an error while using the Sharp lib V0.21.0
Environment:
Win10
Node 9.10.1
Sharp V0.21.0
IISNode v0.2.18
chokidar V2.0.4
express V4.16.3
At initialization of express app, of the all the files in a watched folder are fed to sharp for resizing.
Local file folders are set as the source and destination. There are 15,000 files or so, and the startup thread dies with the exit code above.
Had it die with this exit code also:
Process finished with exit code -1073740940 (0xC0000374)
I am guessing some kind of interaction between it and iisnode
https://stackoverflow.com/questions/12282249/iis-app-pool-crashing-with-exception-0xc0000005
Any help would be appreciated.
Thanks

Has chokidar been configured via awaitWriteFinish to wait for the file to be complete before the event is fired?

"By default, the add event will fire when a file first appears on disk, before the entire file has been written" https://github.com/paulmillr/chokidar#performance

The use of Node 9 suggests sharp was compiled from source. Are you able to use Node 8 or 10 and try with the pre-built binaries?

Hello Lovell,
Thank you for the suggestions!It is not configured to wait - !!! THANK YOU, this makes the most sense to be the cause.I will investigate this first!
*******During NPM install, I did need to make sure the python and compiler were available - so yes I believe it was compiled during install.I did not explicitly compile the binaries.
Will try and change the Node version - but this may not be so easy for me. Quite a few dependencies, and there is a happy medium where all the libs function.
Thank you
Et
  From: Lovell Fuller<EMAIL_ADDRESS>
To: lovell/sharp<EMAIL_ADDRESS>Cc: redevill<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Tuesday, November 20, 2018 1:46 AM
Subject: Re: [lovell/sharp] Sharp resizing quits with - exit code -1073741819 (0xC0000005) (#1468)
Has chokidar been configured via awaitWriteFinish to wait for the file to be complete before the event is fired?
"By default, the add event will fire when a file first appears on disk, before the entire file has been written" https://github.com/paulmillr/chokidar#performance
The use of Node 9 suggests sharp was compiled from source. Are you able to use Node 8 or 10 and try with the pre-built binaries?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Documenting Trials:
Short answer - File watcher was isolated away from sharp so, determined it is not the cause of the Error.
Documented Detail:
Lovell suggested, (nicely) that my file watcher implementation was bad.
Raising file addition events before the file was completed writing. This was indeed the case.
I changed this:
private init_FileWatcher(srcPath: string) {     chokidar.watch(Config.ImageRepoPath,       {ignored: [/(^|[\/\\])\../, /Thumbs\.db$/],       // Does NOT fire add events when watch on dir starts       ignoreInitial: true,       // Make sure the file has finished writing before sending to Sharp for compression       awaitWriteFinish: { stabilityThreshold: 2000, pollInterval: 200 }       }).on('all',       (event: string, path: string) => this.imageChangeHandler(event, path));   }
I also changed the action so that the file watcher would not be involved at startup. Simply feed the entire directory into sharp, to ensure the resized folder is synchronized with the source.
As follows:
 private startup_ProcessAllImages(srcPath: string) {     const files: string[] = fs.readdirSync(srcPath);     files.forEach((file: string) => {       //Sending as a change event should ensure an overwrite for all compressed files.       //Bringing the folder under synchronization when starting up.       this.imageChangeHandler('change', srcPath + path.sep + file);     });   }
The imageChangeHandler calls this relevant piece for the resize:
    sharp(srcPath)         .resize({width: myWidth})         .toFile(outPath)         .then((data: any) => {           console.log('Done shrink of ' + srcPath + ' to ' + outPath);         })         .catch((err: Error) => {           console.log('Resize for: ' + srcPath + ' had Error: ' + err.message);         });

Great, thanks for confirming this was the problem.


Count/truncate number of tokens before processing

Create a function that takes in text as input, converts it into tokens, counts the tokens, and then returns the text with a maximum length that is limited by the token count. The returned text will be truncated if it exceeds the specified token count, ensuring that it does not exceed the maximum context size.

@jakvb what's the use case for this?
Something like:
def truncate(llama: Llama, input: str, maxlen: int) -> str:
    return llama.detokenize(llama.tokenize(input)[:maxlen])

@jakvb If you're using  langchain as a wrapper around your LlamaCpp model, you can count the number of tokens before calling the llm with the following method  get_num_tokens
    def get_num_tokens(self, text: str) -> int:
        tokenized_text = self.client.tokenize(text.encode("utf-8"))
        return len(tokenized_text)

Example usage
num_tokens: int = llama_model.get_num_tokens(question)

And if we don't use LangChain?

How about ollama ?

And if we don't use LangChain?

# Where llm is an instance of the Llama class with a model loaded
tokens = llm.tokenize(b"Q: Name the planets in the solar system? A: ")
print(len(tokens))

How would you truncate the final dictionary that includes the system etc?

How would you truncate the final dictionary that includes the system etc?

I actually haven't tested if this includes the system prompt. One quick and easy way to test this and resolve it if it does would be to tokenize an empty string. If you get tokens back it includes the system prompt, if it doesn't then it doesn't.
Then you can just subtract the length of the system prompt dictionary from the start of the final dictionary.


mcfly: 0.9.0 -> 0.9.1

Description of changes
Replacement of #326515, also moving the package to by-name to be able to automerge
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

AFAIU this also fixes some CVEs from #141368 as cargo.lock is updated


Reload data on tab change

Hi
I have 4 tabs and on the 4. tab I want to reload data by en allready given endpoint.
So I know how to reach this endpoint in my ts file in my dialog.ts but I dont know

how to set the result value into a textfield
how to do this whenever the tab is selcted (which event, where to register to that event

You can handle the tab click using following code.
this.tabs.click((e, h) => {
	let key = e.target.innerText;
	/* your code/logic goes here */
});

Tried that. I allways get
Message: TypeError: this.tabs is undefined
I tried it within the constructor and afterLoadEntity, but both same error.

@Wasseratem
Try in  markupReady or the updateInterface.
Also are your tabs visible at load or do you toggle that ?
Are you sure you are in the dialog and not an editor ?
Also make sure you do not have any errors in your code - the typescript T4 translation does not transform if there are errors.
You can also get Tabs byId   , this.byId('TabNotes')
Example setting a field value - if it is not a field but a text there is another way to do that
                OrdersDB.InventoryItemService.Retrieve({
                    EntityId: itemId
                }, response => {
                    this.setFieldValues(response.Entity);
                });

        // Set field values
        private setFieldValues(details: InventoryItemRow) {

            this.form.UnitPrice.value = Number((details.UnitPrice).toFixed(2));
            this.form.QuantityPerUnit.value = details.QuantityPerUnit;

        }

if you need to set the value of an element and not a data point
                this.orderTotal = formatter.format(total).toString();

// Find the element that I want to set the text value of.
                var objOrderTotal = this.element.parent().parent().find(".grid-container").parent().find(".OrderTotal");

                objOrderTotal.html("Order Total: " + this.orderTotal + "");


Issue message overlaps code

When you hover over the issue warning icon, a message appears that overlaps the code.  Such an interface makes it very difficult to fix the issue because a user wants to see the message and the line with code simultaneously.

Also, @vyahhi what do you think about this message? I think it is very difficult for students...Maybe should we make it more clear?
E.g.:
Found usable augmented assign pattern. You can use shorthand notation if the left and right parts of the expression have the same variable, e.g. x = x + 2 is the same with x += 2

@nbirillo agree.

@dubchenko please take a look on this issue.

Possible partial solution: make this message box 30-70% transparent.


[GOBBLIN-1074] Sort job status array when returning flow status

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1074

Description

[x] Here are some details about my PR, including screenshots (if applicable):

Sort job status array by execution start time when returning flowstatus/flowexecution
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
trivial

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@sv2000 please review


Replace larapack/dd with symfony/var-dumper

larapack/dd no longer provides any value, because now symfony/var-dumper ships dd and dump helpers on its own.

Thanks :)


[BACKEND] Adding batched reduce algorithm to support #warps > #threads-per-warp

Currently, Triton's reduce op, especially accumulatePartialReductions function doesn't support the case when number of warps is greater than number of threads per warp.
Although this scenario doesn't happen in current NVIDIA GPUs (as the maximum CTA size is 1024 and the warp size is 32), it can become relevant in other GPU architectures when:

When the warp size is smaller than 32 (e.g., numWarps=64, warpSize=16—supported by Intel GPU).
The CTA (or thread block) size is bigger than 1024.

The modified function runs the warp reduction multiple times when the number of warps is bigger than the size of each warp.
The lit test in tests/TritonGPU/reduce_op_lowering.mlir checks if the lowering generates multiple partial reduction passes when the CTA size is 2048 (numWarps=64, warpSize=32).
The modified algorithm does work for numWarps <= warpSize cases.
However, to be safe, it falls back to the previous implementation in standard scenarios.
Co-authored-by: Chengjun Lu<EMAIL_ADDRESS>PR checklists:

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[x] I have used an LLM to copyedit my PR description and and code comments.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[x] I have added tests.

/test for lit tests

Select one of the following.

[ ] I have not added any lit tests.
[x] The lit tests I have added are "minimal" -- they contain only the
instructions necessary to exercise the bug.  (Usually running Python code
and using the instructions it generates is not minimal.)

Since it doesn't affect the upstream targets and this is not a trivial patch could you maintain this downstream?

Or could you justify a case that will help nvidia GPUs?

Since it doesn't affect the upstream targets and this is not a trivial patch could you maintain this downstream?

It does not affect NVIDIA GPUs for now. But I think it is better if TritonGPU IR supports different GPU architectures in general.
Also, NVIDIA can change the maximum CTA size in the future. In that case, the current implementation can be a potential bug.
What are your thoughts?


WIP: Add median filtering function

Description
This PR adds the function median_filter in utils module.
This is still a work in progress, tests and tutorial are missing. Do not merge.
Release note
Summarize the changes in the code block below to be included in the
release notes:
Add `median_filter` function.

Checklist

[ ] The pull request title, summary, and description are concise.
[ ] Related issues are linked in the description.
[ ] New dependencies are explained.
[ ] The source code and documentation can be distributed under the MIT license.
[ ] The source code adheres to code standards.
[ ] New classes, functions, and features are thoroughly tested.
[ ] New, user-facing classes, functions, and features are documented.
[ ] New features are covered in tutorials.
[ ] No files other than source code, documentation, and project settings are added to the repository.

Hi @cgohlke! I opened this PR to start the discussion of a couple of thing related to the filtering methods, before I continue working on this:

I tried implementing the filtering in Cython so that it works faster, and it does marginally better (15-20%) than SciPy's, which is not that much because it's quite fast. For an image of 100 million pixels SciPy's takes 10.31 seconds and this one 8.80 seconds. Do you think this is worth it? The advantage is that this is more under our control and we can modify if necessary certain aspects.

As I am not too familiar with Cython, I may have some ways to optimize the code, so maybe you can have a look and this can help also to improve performance and/or memory handling.

Regarding Cython code formatting, I tried black formatting but is not working. Do you have an alternative or you check manually the format?

I left the median calculation separated from the filtering function for two reasons. One is that we now can have a separate median calculation in case it is needed for some other purpose, and the other is that the same filtering function can be easily modified to apply another method (mean, gaussian, etc) for future implementations.

We think the way SimFCS handles the borders is by keeping the original from the image, so by default the filter does that, but it also allows for the reflection method, which is the default of SciPy's and can be useful.

I added also the num_iter parameter to the median_filter function since it is very common to apply the filter 3 or more times. But if you think this adds complexity I can remove it very easily.

I removed the num_iter parameter from the median_filter function. I realized it was not needed and it actually made the performance of the function worse.

I tried implementing the filtering in Cython so that it works faster, and it does marginally better (15-20%) than SciPy's, which is not that much because it's quite fast. For an image of 100 million pixels SciPy's takes 10.31 seconds and this one 8.80 seconds. Do you think this is worth it? The advantage is that this is more under our control and we can modify if necessary certain aspects.

Good to see you getting familiar with Cython. I think that 20% is not worth the implementation/maintenance overhead, but that can probably be improved. The huge advantage of ndimage's implementation is that it works with any-dimensional arrays, for example of 3D phasors. The median_filter functions should fall back to the scipy or skimage implementation for cases that _apply_2D_filter can't handle. I think there is ongoing work in the scikit-image library to improve the median filter. Also check literature for optimized algorithms, such as https://ieeexplore.ieee.org/document/1163188

As I am not too familiar with Cython, I may have some ways to optimize the code, so maybe you can have a look and this can help also to improve performance and/or memory handling.

In general, use ssize_t instead of int for sizes and indices such that the code can work with > 2GB arrays. You also want to release the GIL in _apply_2D_filter such that the function can be efficiently used in multi-threading cases.

Regarding Cython code formatting, I tried black formatting but is not working. Do you have an alternative or you check manually the format?

Try cython-lint. It's in the project's dev requirements but not used in CI yet.

I added also the num_iter parameter to the median_filter function since it is very common to apply the filter 3 or more times. But if you think this adds complexity I can remove it very easily.

Exactly, and that is probably a huge opportunity for optimizing the Cython function: Move the num_iter (how about renaming it repeat) loop inside the _apply_2D_filter and repeat the filter on the kernel while it is in memory.
In general, how about adding (or replacing with) a function taking phasor coordinates as input for convenience, something like
real, imag = phasor_median(real, imag, repeat=3, **kwargs)
or
real, imag = phasor_filter(real, imag, method='median', repeat=3, **kwargs)
instead of
real = median_filter(real, repeat=3, **kwargs)
imag = median_filter(imag, repeat=3, **kwargs)

Great suggestions @cgohlke, I will look a little bit more into the algorithms but these are all good ideas. I will fall on SciPy's for cases we can't handle yet.
I will look into the iterations, I didn't think of including it in the Cython function but it makes a lot of sense.
I will let you know when it's ready again for review, but thanks already for all the suggestions!

Move the num_iter loop inside the _apply_2D_filter and repeat the filter on the kernel while it is in memory.

I realize that this will not be that trivial with the default algorithm.

I will look a little bit more into the algorithms

Don't spend too much time on it though. I think we agree that the purpose of the median filter functionality in the PhasorPy library is not to provide yet another median_filter function as an alternative to scipy, scikit_image, or opencv, but to provide the user with an convenience function that fits well into the library's (hopefully) consistent workflow, such as phasorplot.hist2d(*phasor_median(*phasor_calibrate(*phasor_from_signal(signal, ...), ...), ...), ...).

tutorial are missing

It's probably enough for to update the introduction tutorial.
I noticed that the Cython code works on signal_t instead of float_t. Do you plan/expect to apply median filter on the signal or phasor coordinates?

I noticed that the Cython code works on signal_t instead of float_t. Do you plan/expect to apply median filter on the signal or phasor coordinates?

You are right, I was testing that. If it uses float_t it doesn't  work with arrays of dtype int, that's why I used signal_t which contains also int types. I know the median filter is intended to be used with phasor coordinates which are float but I thought it may be useful to allow for the use of int, but maybe we leave it only for float. If not we can convert int to float, but I think memory-wise this is not optimal right?
Maybe for now it's better to work only with float since I can't think of now of cases where int is going to be used. Let me know if you believe this is the best option and I will make the changes.

Maybe for now it's better to work only with float since I can't think of now of cases where int is going to be used. Let me know if you believe this is the best option and I will make the changes.

The issue with using signal_t is that Cython will create separate C code for each function for each of the types in signal_t, which quickly results in large compiled extension sizes. In some cases, such as the _phasor_from_signal function, it is worth. In the case of the median function probably not unless your implementation offers significant advantages over what's in scipy, skimage, or opencv.

Hi @cgohlke, I have been researching as we talked with the cython implementation of the median filter and tested moving the iteration inside the cython function for memory and execution efficency, but It didn't have much of an impact in performance, mainly compared to scipy's and skimage filters, so I rolled back and made a new implementation using scipy's median filter instead. I think it's the best approach for now as it is already very efficent and maybe it's not worth spending time now trying to improve this to only have a very minor impact.
I used scipy's instead of skimage's filtering function because skimage actually uses scipy's and altough it seems to be more efficent with large images, I tested and actually were quite similar. I think scipy is a little more personalizable with the parameters and also can handle more than 3D, which may be of use in the case of multidimensional phasors.
As you may see, I am having some error when building the cibuildwheel because it doesn't seem to have scipy installed, although it is already listed as a dependence. Do you know what can be the problem?

I am having some error when building the cibuildwheel because it doesn't seem to have scipy installed, although it is already listed as a dependence. Do you know what can be the problem?

Scipy is in the development dependencies. You need to add it to the install dependencies in pyproject.toml too:
https://github.com/phasorpy/phasorpy/blob/fee878ee160902ba9978f0cc0d61886e6fa57c52/pyproject.toml#L22

This looks good. I will review later today when I'm back.

A few general thoughts:

How about moving phasor_filter to the phasor module. We should also discuss again moving all functions in that module to the main package namespace.
Since scipy is a heavy dependency and is currently not used elsewhere, how about moving the import statement from scipy.ndimage import median_filter inside the _median_filter function?
How does ndimage.median_filter handle NaN values, which are, for example, common in phasor coordinates returned from phasor_from_signal_fft? We should probably review the whole library for NaN handling.

Thanks @cgohlke for all your comments, I will go through them and let you know when it is ready for review.

How about moving phasor_filter to the phasor module. We should also discuss again moving all functions in that module to the main package namespace.

I agree, this is something we can discuss in the next meeting as I believe is a big change that should be done (if we agree to do it) before our first release.

How does ndimage.median_filter handle NaN values, which are, for example, common in phasor coordinates returned from phasor_from_signal_fft? We should probably review the whole library for NaN handling.

This is a very good issue you raised. It is also something I was thinking about in the thresholding function implementation also. Maybe this is something we can discuss in it's corresponding PR, but I was thinking of masking below the threshold as NaN, if it makes sense and all downstream function work well with `NaN, but is something we have to check first.
Regarding the filtering, how do you think it should handle NaN values? I am thinking of two alternatives:

NaN values are always kept out of the filtering and remain as NaN.
NaN values  are filtered with the surrounding values.

Maybe we can discuss this or @schutyb and @lmalacrida can give some insight into this. I am inclined to the first alternative, but I don't know what is usually done in phasor processing in this case.

Regarding the filtering, how do you think it should handle NaN values?

Good question. What does ndimage.median_filter do?
Using NaN is tempting, for example, to indicate masked out values in a potential phasor_threshold function. NaNs are already used internally in the pseudo_color function to indicate masked out values. Matplotlib can handle NaN I think. There are some functions in the plot module that currently choke on NaN. I will include a fix in my next PR. Probably phasor_from_signal should return NaN instead of zero to be compatible with the FFT implementation...

Good question. What does ndimage.median_filter do?

It seems that the topic of NaN handling when filtering is still debated in the imaging forums. I tested with a small array with NaN and ndimage.median_filter seems to fill the NaN with the median of the kernel. But looking the discussions from the community regarding NaN handling, it seems that both scipy's and skimage's filters have undefined behaviour with NaNs and this has been going for a while without a fix. The discussion seems to center in the compromise between being NaN safe at the expense of performance, which is something they don't seem to want to sacrifice.
Considering this I think we have to discuss if implementing our filter makes sense to be able to control what happens to NaN. If we consider NaN are not going to be too frequent, maybe we can stick with ndimage of skimage ones, but I think it will not be consistent.


Instructions for downloading CodeQL are incorrect.

The instructions state the following

Download version 2.4.6 of the zip file if you are certifying a driver for the Windows Hardware Compatibility Program in 2021. For example for 64 bit Windows "codeql-win64.zip".

However the instructions on the github page for Windows-Driver-Developer-Supplemental-Tools (https://github.com/microsoft/Windows-Driver-Developer-Supplemental-Tools) state that the vresion should be 2.6.3.

The main branch of this repo currently uses [version 2.6.3 of the CodeQL CLI](https://github.com/github/codeql-cli-binaries/releases/tag/v2.6.3) and version lgtm/1.29.0 of the CodeQL queries. When using this repo to certify a driver, please use the appropriate sub-branch for the Windows release in question:

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

ID: 5660df59-18ee-441d-afc7-023da97a7da5
Version Independent ID: 1e1c4d4d-7c4b-6af2-64f2-d51162a8992f
Content: CodeQL and the Static Tools Logo Test - Windows drivers
Content Source: windows-driver-docs-pr/devtest/static-tools-and-codeql.md
Product: windows-hardware
Technology: devtest
GitHub Login: @DOMARS
Microsoft Alias: domars

@ergins23 - Thanks for filing this issue, and this page had fallen behind the latest version information, which is available on the on the Windows-Driver-Developer-Supplemental-Tools (https://github.com/microsoft/Windows-Driver-Developer-Supplemental-Tools) as you mentioned. I checked with the developers, and we updated the directions here, to defer to that location, for the specifics on versioning, for the different use case scenarios. Thanks again for filing this issue and your interest in these important code security tools.


Fix example uami issue

Overview/Summary
The UAMI for policy int he example code fails due to hard coded resource id. This fixes it by creating the UAMI as part of the example.
This PR fixes/adds/changes/removes
N/A
Breaking Changes
None, example only
Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.

/azp run unit


Remove Kiln and Ropsten as supported networks

PR Description
Kiln and Ropsten have been shut down for a long time, so there is no point in supporting them as a --network option.
Fixed Issue(s)
N/A
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

Might be worth adding a changelog entry?

What's the logic behind removing ropsten?

What's the logic behind removing ropsten?
ropsten.beaconcha.in shows blocks being produced..

Ropsten has been shut down since December last year though. ropsten.beaconcha.in points to mainnet for some reason and when you try connecting to it, there are no peers, etc.

https://blog.ethereum.org/2022/11/30/ropsten-shutdown-announcement

Prysm also removed option few releases back: https://github.com/prysmaticlabs/prysm/pull/12058/

I got confused by ropsten.beaconcha.in

ok all good, just no context as to why we were removing support for a network still progressing, but happy now.

I got confused by ropsten.beaconcha.in

Yeah same. I will bring it up in their discord.


v7 Migration, Data Retention Policies not migrated

My migrated v6->v7 has reverted to a default Data Retention policy

duplicate of #1615 ?


Dashboard status doesn't reflect reality

Do you want to request a feature or report a bug?

Bug
Looking at the dashboard the Errors doesn't reflect the state of the Service.

Dashboard says all is okay, but one Service doesn't have any servers working.

This should mean the Service is having an error?
If it would have 1 failing out of 1+ servers, it would mean a Warning?
Output of traefik version: (What version of Traefik are you using?)
v2.2

What is your environment & configuration (arguments, toml, provider, platform, ...)?
running in docker.
docker swarm
consul

Duplicate of #6117


New Emails System

Summary
The current emails system is in the most part broken and email layout is not matching the whole CircuitVerse theme and not look good

The New System
I uploaded all assets and description on my idea to Google Drive as a part of GCI. My approach to doing it is: Smart Implementation - Best way to implement this would be creating only one file and creating a function to fill it with arguments. Read about it on the drive.

Note: Have in mind that you can't use SVG inside emails: #964
Are you working on this issue? (Yes/No)
No

What has been the update here ? Have the email problems been resolved ?

@ShubhankarKG that is slighty blocked by #849 but still, you can work on it like that:

I also said in that document something about that user should be able to customize emails preferences, I made an issue for that #849 so for that moment you can just use if true do and fix that when 849 will be merged


songbird::driver::tasks::udp_rx: Failed to decode received packet: Opus(BufferTooSmall)

Hello. I'm attempting to use this library to receive voice data. However, all the VoicePackets I receive are either empty (when talking) or silent (as expected after I stop talking).
I enabled tracing and also added some logging to audiopus in order to debug this issue. It looks like the failure is caused by a BufferTooSmall error from Opus. Indeed when I replace the buffer size of STEREO_FRAME_SIZE with 9600 in udp_rx.rs the packets are successfully decoded.
I have included a sample Opus packet that fails to decode. Hopefully this can help get to the root cause of the issue.
Speaking state update: user Some(UserId(XXX)) has SSRC XXX, using MICROPHONE
asked to decode Opus packet Packet([123, 131, 47, 46, 11, 32, 223, 10, 132, 144, 147, 61, 250, 196, 24, 255, 86, 235, 228, 71, 243, 149, 78, 24, 128, 150, 72, 7, 206, 93, 149, 61, 231, 176, 14, 4, 251, 236, 76, 161, 243, 240, 254, 63, 150, 186, 156, 239, 217, 149, 78, 10, 136, 24, 76, 222, 60, 244, 212, 213, 126, 89, 217, 60, 149, 38, 195, 197, 112, 44, 116, 98, 82, 143, 103, 252, 247, 122, 206, 191, 213, 191, 197, 172, 61, 140, 89, 87, 177, 77, 11, 180, 232, 51, 126, 101, 251, 10, 136, 17, 103, 209, 212, 138, 70, 9, 157, 122, 0, 3, 213, 115, 44, 134, 54, 104, 224, 211, 152, 139, 227, 72, 166, 93, 244, 174, 159, 241, 41, 150, 96, 44, 85, 230, 71, 166, 249, 186, 168, 77, 83, 206])
Nov 29 00:02:56.230 ERROR songbird::driver::tasks::udp_rx: Failed to decode received packet: Opus(BufferTooSmall).

Thanks for your work on this library.

That's surprising, and suggests that the packet is larger than 20ms. I'll have a look later; was the sender a user, or another bot?

On 29 Nov 2020, at 07:19, Bradley Walters<EMAIL_ADDRESS>wrote:

Hello. I'm attempting to use this library to receive voice data. However, all the VoicePackets I receive are either empty (when talking) or silent (as expected after I stop talking).
I enabled tracing and also added some logging to audiopus in order to debug this issue. It looks like the failure is caused by a BufferTooSmall error from Opus. Indeed when I replace the buffer size of STEREO_FRAME_SIZE with 9600 in udp_rx.rs the packets are successfully decoded.
I have included a sample Opus packet that fails to decode. Hopefully this can help get to the root cause of the issue.
Speaking state update: user Some(UserId(XXX)) has SSRC XXX, using MICROPHONE
asked to decode Opus packet Packet([123, 131, 47, 46, 11, 32, 223, 10, 132, 144, 147, 61, 250, 196, 24, 255, 86, 235, 228, 71, 243, 149, 78, 24, 128, 150, 72, 7, 206, 93, 149, 61, 231, 176, 14, 4, 251, 236, 76, 161, 243, 240, 254, 63, 150, 186, 156, 239, 217, 149, 78, 10, 136, 24, 76, 222, 60, 244, 212, 213, 126, 89, 217, 60, 149, 38, 195, 197, 112, 44, 116, 98, 82, 143, 103, 252, 247, 122, 206, 191, 213, 191, 197, 172, 61, 140, 89, 87, 177, 77, 11, 180, 232, 51, 126, 101, 251, 10, 136, 17, 103, 209, 212, 138, 70, 9, 157, 122, 0, 3, 213, 115, 44, 134, 54, 104, 224, 211, 152, 139, 227, 72, 166, 93, 244, 174, 159, 241, 41, 150, 96, 44, 85, 230, 71, 166, 249, 186, 168, 77, 83, 206])
Nov 29 00:02:56.230 ERROR songbird::driver::tasks::udp_rx: Failed to decode received packet: Opus(BufferTooSmall).
Thanks for your work on this library.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It's a 30ms packet (2880 samples when decoding to stereo), which I don't think should ever be sent if documentation is to be believed. I can modify the maximum decoded packet size to include 60ms frames/120ms packets (the largest allowed by the opus spec), but I think I'd really like to know more of the specifics you were testing with.
Can you tell me:

Whether any users in the voice call were bots?
Any users were on custom clients?
Whether you're using this as part of a true bot, selfbot, or a custom client?
Were any users on the mobile client?
What voice server location you were using?

I know that Discord does some weird custom re-encoding and mixing for mobile clients, but I'm curious as to whether they've started some experiment somewhere. Voice receive is undocumented, after all.

Two users in the call (human speaker and my bot).
Human speaker is connected through the web client.
Songbird is used via a true bot.
Voice server location: I can reproduce this 100% of the time with the original buffer size. Today I checked and this occurred using us-west359.discord.media:443

Took me a while to repro, but the root cause is in fact Discord Web Client on Firefox. I tested with nightly.
Source is naturally minified and unreadable, but this seems to be some incredibly platform-specific hack. I'm not sure why this exists or is needed, since SDP/ICE logs suggest that both Chrome and Firefox are using WebRTC for audio and signalling.
I think that the solution might be to store a known-good size for each SSRC so that most clients pre-allocate 20ms, and only heavy-packet clients need to scan for the correct size once.


bug: kubearmor-operator helm chart doesn't remove kubearmor-controller

Bug Report
General Information

Environment description: k3s or any k8s
Kernel version: N/A
Orchestration system version in use: v1.23.9+k3s1
Link to relevant artifacts: kuebarmor-operator helm charts
Target containers/pods: kubearmor-controller

To Reproduce

Install KubeArmor using kubearmor-operator helm chart.
Uninstall the helm release.
You'll see that kubearmor-controller pod stays installed and goes into error.

Reason
The secret containing the cert for mutating admission webhook, mounted as a volume on the controller, is deleted before kubearmor-controller deployment is.
Expected behavior
Cleanup should be ordered.
Screenshots

code to add operator as owner for controller deployment is missing with control loop here:
https://github.com/kubearmor/KubeArmor/blob/1e31e1ae46aa99629410239afac16e98ada1fa7d/pkg/KubeArmorOperator/internal/controller/resources.go#L390


outputs: iam role id

Description
Slightly increases the versatility of the module. By including the iam role id in outputs, one can add other policy attachments to the iam role outside of the module itself.

Is anything blocking this change from being merged? It would be nice to support SSM out of the box.

Heads-up @antonbabenko ⏫

Thanks @frimik for this PR.
v2.4.0 has been just released.


Allow clicking on the Top Predictions box to open market

Many times I try to click this box to open market and place wager. May as well make it open the market.

@Newportkiwi's call

No longer relevant.


Duplicated CWWKM2144I messages

run
git clone https://github.com/openliberty/guide-getting-started.git
cd guide-getting-started\finish
mvn liberty:run

There is a duplication having two CWWKM2144I messages:
[INFO] CWWKM2102I: Using artifact based assembly archive : io.openliberty:openliberty-kernel:null:<IP_ADDRESS>:zip.
[INFO] CWWKM2102I: Using installDirectory : /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/target/liberty/wlp.
[INFO] CWWKM2102I: Using serverName : defaultServer.
[INFO] CWWKM2102I: Using serverDirectory : /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/target/liberty/wlp/usr/servers/defaultServer.
[INFO] Copying 1 file to /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/target/liberty/wlp/usr/servers/defaultServer
[INFO] CWWKM2144I: Update server configuration file server.xml from /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/src/main/liberty/config/server.xml.
[INFO] CWWKM2185I: The liberty-maven-plugin configuration parameter "appsDirectory" value defaults to "apps".
[INFO] CWWKM2160I: Installing application guide-getting-started.war.xml.
[INFO] Copying 1 file to /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/target/liberty/wlp/usr/servers/defaultServer
[INFO] CWWKM2144I: Update server configuration file server.xml from /Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/src/main/liberty/config/server.xml.
[INFO] CWWKM2001I: Invoke command is [/Users/gkwan/tasks/CNAI/guides/testsite/getting-started/prod/guide-getting-started/finish/target/liberty/wlp/bin/server, run, defaultServer].

I do believe that is because multiple maven goals get executed when you use the liberty:run goal, and the file gets copied over as part of more than one goal. This is because someone could be running goals individually and the file could change between the various goals. For example, a user could create the server and then later start the server.

@gkwan-ibm This is expected behavior. Can I close this issue?

hi @hutchig, Do you accept this behavior having 2 messages?


Support video in the picker

Provide options to display images only, videos only, or both images and videos in the ImagePickerController

@hungnguyenvn, can you review this pull request?

          1 Error

:no_entry_sign:
Please update CHANGELOG.md.

  Generated by :no_entry_sign: Danger

@daveluong @bcylin i made some changes based on your comments. Please help review again 🙏
Do I need to update CHANGELOG? And when should we update version?

It should be all right to add a ## Next release section in CHANGELOG.md.
Since the master branch is protected, I usually have another PR to run make bump version=<number> and update README if needed.

Thank you @bcylin for helping us to review the PRs! Truly appreciate it :)

Thank you @bcylin and @daveluong ! 😄


feat(button): focus the submit button #1233

Adding auto-focus to prompt
TODO: typecheck is failing. Will solve this

Will be added in a separate PR to master.


ECONNRESET

Sometimes I hit a bad server and I get
events.js:141
throw er; // Unhandled 'error' event
^
Error: read ECONNRESET
at exports._errnoException (util.js:837:11)
at TCP.onread (net.js:544:26)

Is it because I try to do more than 1 connection at a time? Can I only do only one smtp at a time?

One of the mail servers was bad - need to find a way to catch that error.

Another one I got was
events.js:85
throw er; // Unhandled 'error' event
^
Error: connect ECONNREFUSED
at exports._errnoException (util.js:746:11)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1000:19)
Need a way of catching that error and associating it with the server - so a general error handler would not work.

Aha, figured it out - I needed to get the stream and use
.on('error', function (error) {
console.dir(error);
})


Improve explaination for prefixes and find command in User Guide

There is an overlapping usage for the term "parameter" which currently refers to both the prefixes "e/, n/, etc." and the user supplied input of "EMAIL, NAME, etc." which can cause confusion for users. The instructions for the find command are particularly confusing as it requires dynamic usage of prefixes.
Let's update the user guide to be clearer in explaining the usage of prefixes and parameters.
Fixes #128

Yep this is better with thee consistent naming


Constant parent id for new components in a list

Sorry it's me again.
Found an issue with this functionality
When i add a new department, and then a new sub-department for that new department, the text field created has the same parent id of colleague:department:0:sub_departments when it should be colleague:department:1:sub_departments. The parent id seems to always be colleague:department:0:sub_departments for any number of sub departments i want to add. Hence, the input changes for the sub departments of the first department in the list.

I tried digging into the code which is how I found out about the id being constant.

Thanks for raising it, I've added some fixes for list fields and dict fields to ensure the ids work properly. Also added some safeguards in from_form_data that should help with your other issue. It's available in 0.3.1, let me know if that works for you!

wow you're super quick! Thanks for the assistance @RenaudLN


World 'isRemote' vs. 'isClient' vs. ???

I think World.isRemote is awkward, It might make more sense to use the client/server binary instead of local/remote.

fromServer

I believe @asiekierka spoke out in favor of isRemote before and explained his reasoning for it.

isRemote is more correct (as to the intention rather than the practice?), IMHO. You could theoretically have a remote world which is not just an instance of WorldClient.

The field is a bit awkward on its own. Theoretically it's redundant for client<->server checking since one could just test (World instanceof WorldServer) instead.
Considering the common usage in vanilla and mods, having a mismatch between the field and the execution environment is likely to cause major breakage. A name like isClient[World] reinforces this implicit contract, which I'd consider a good thing.

I'm with asie. isRemote is more along the lines of "does someone else own/control this world", which I believe is the intend..?

Yeah, there are cases where isRemote is true that would be confusing for isClient at first glance, like singleplayer worlds. Yes, it's speaking of the logical client not the physical client, but most people don't see that distinction. +1 for isRemote.
There should also be better terminology for physical/logical client/server. It's quite confusing in general.
Client/Server for physical sides, Master/Replica for logical sides?

It's anecdotal but isRemote was a major source of confusion for me when I started out modding. Remote to what?

 Let's please change that.

Can we get some more thoughts so we can resolve this?

isClient is good to be changed


[Feature Request]  add api to get differents properties(or fields) when map to exists dst?

Could be add some api to compare differents properties(or fields) with two instance of same class ?
i know we can use reflection,but performance slow.
like:
//same class  (ClassAObj1 vs ClassAObj2)
var fieldNames= ClassAObj1.Diff(ClassAObj2);  //Api-1 get different fieldnames,but not map
bool hasDifferents= ClassAObj1.MapDiff(ClassAObj2,out string[] fieldsName);  //Api-2 get get different fieldnames and map them
//not same class (ClassAObj1 vs ClassBObj2)
var fieldNames= ClassAObj1.Diff(ClassBObj2);  //Api-1 get different fieldnames,but not map
bool hasDifferents= ClassAObj1.MapDiff(ClassBObj2,out string[] fieldsName);  //Api-2 get get different fieldnames and map them

//same like mapper's config it will use the global config(custom ignore some properties(or fields) ),that's will be better.
why need:
if i got not differents,then i wont need to update to database.

mark a link https://github.com/agileobjects/AgileMapper/issues/229