chore: add an archive of meeting notes

Signed-off-by: Feynman Zhou<EMAIL_ADDRESS>

@shizhMSFT Resolved your comment. Ready to merge it?


Update common-tips.md

汉化不完整，导致用例会产生歧义。

ping @Jinjiang

@kazupon I think the change is not necessary.
Thanks.

@Jinjiang
Thanks reviewing!

I think the change is not necessary.

@nachao
close for this reason.
Thank you for your feedback.


Wireless Crafting Monitor Dupe Bug

Issue description: When you shift click items into the refined storages system it puts the item into the System but doesnt take it from your inventory
What happens: The item dupes
What you expected to happen: No dupe preferably
Steps to reproduce: (Not completely certain but it seems to be something to do with the wireless transmittor)
What happens:
What you expected to happen:
Steps to reproduce:

Connect a wireless crafting monitor to the System
Open main system
When you attempt to shift click an item into the system the item will appear in the system but will not leave your inventory
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12
Forge: <IP_ADDRESS>54
Refined Storage: refinedstorage-1.6.16

Does this issue occur on a server? yes
If a (crash)log is relevant for this issue, link it here:
https://pastebin.com/9aDfM1mP

The wireless crafting monitor is for seeing currently auto crafting tasks. Did you mean something else?

Look at the Null Pointer Error in the Pastebin

Fixed for 1.15


Increase macOS build to at least 10.9 SDK

When trying to notarize our app via Apple's notarization service, we get the following warning for libjffi-1.2.jnilib:
The binary uses an SDK older than the 10.9 SDK.

See official documentation on this warning: https://developer.apple.com/documentation/security/notarizing_your_app_before_distribution/resolving_common_notarization_issues#3087723
That means it's not an issue to keep the deployment target at 10.6 but it would be great if the linked SDK could be increased from 10.7 to 10.9. Would that be possible?
Actually, it wouldn't just be great, it will be necessary soon. If I'm not mistaken, beginning with January 2020, see: https://developer.apple.com/news/?id=09032019a
And also:

Notarisation requires the hardened runtime, the hardened runtime requires library validation, and library validation requires that you link with (at least) the 10.9 SDK.

(https://forums.developer.apple.com/thread/115679#357047)

Happy to help with this one.
Just as a heads up, rebuilding with the latest MacOS (10.15) would mean losing i386 and ppc arch support.
We should be able to keep i386 alive by using MacOS 10.14 for the rebuild. (But I'm unable to check that atm)

@MSNexploder you might try to build a binary and report what notarization thought about it ...

@kares This is literally it:

The binary uses an SDK older than the 10.9 SDK.

@albertvaka so if you re-generate using >= 10.9 no other warnings pop up?
(sorry can not test as I do not have a Mac around)

I think it is even reasonable to target El Capitan for new releases. According to this stats website the global market share of all versions >= 10.11 is 94% of all Macs.
I doubt that any new releases of desktop software is targeting older macOS versions, while mere maintenance releases can still use old versions of jffi.
Just as a hint for the repo owners if you like to create some margin to the approaching front of deprecated tools and APIs.

not 100% sure about that ... let's try doing the minimal upgrade necessary (if doable).

I doubt that any new releases of desktop software is targeting older macOS versions

you'll also get the warning (on newer versions) if you just try running jruby -e 1 right?
at least that is my understanding of the issue (wout being able to try it out myself).
if someone is on 10.9 upgrading a patch release JRuby 9.2.9 (-> 9.2.10) ideally should not break.

Can someone rebuild the binary on MacOS 10.9? I'm not sure how to even do that since I'm on 10.14.

Oh nevermind, I see now @MSNexploder started a PR. I'll comment there.


RPCServer makes the COM+ unavailable

I download a sample code from this repos.
When I build it and deploy the service to my computer(RS4), It will make the COM+ unavailable.
Here are my steps:

Open solution CustomCapability.sln (Windows-universal-samples\Samples\CustomCapability\cs\ CustomCapability.sln)
Build project RpcServer. It will create a .exe (Windows-universal-samples\Samples\CustomCapability\cs\Win32\Debug\RpcServer.exe)
Install windows service.(RpcServer.exe -install)
Open Windows Services manager and find out “Hsa Sample Service(HsaService)”, Change the startup type of it to Automatic, save it.
Restart the OS.
Open Component Services and select Computers node. We will find that COM+ is disabled.

This bug already effect lots of models in the market. All PCs can't use DCOM.

Here is solution.
pull request https://github.com/Microsoft/Windows-universal-samples/pull/1003


rename arcmap TBX, and launch tools with separate BAT files

Updates for better file distribution and easier test runs on different clients.
Description
Basically renamed the ArcMap toolbox from "Military_Tools_dsk.tbx" to "Military_Tools_arcmap.tbx", and separated test runner into two BAT files, and updated instructions.
Related Issue
N/A
Motivation and Context
Clarity of toolbox names and easier to run tests on client platforms.
How Has This Been Tested?
Ran unit tests against new files.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

@ACueva
Please change Jenkins to run the BAT files accordingly:

For ArcGIS Pro run .\utils\test\ArcGISProTestKickStart.bat.
For ArcMap run .\utils\test\ArcMapTestKickStart.bat

Merged, kicking off Jenkins Build.

@mfunk I added the calls for the unit tests.  ArcGIS Pro one is working. Kicked off another build to see if the ArcMap one is functioning after I made these changes.  The changes in the files are now showing up.  I think we're good for this pull request/issue.


Added Priority field to ZoneLockdown type

Description
This adds the Priority field for ZoneLockdown. It's currently undocumented but it seems to be already functional for both reading and writing.
Has your change been tested?
Yes, I ran the code in lockdown_example_test.go against a real Cloudflare account and it worked.
Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Note: I am under the impression that the documentation is automatically generated in this case so that's why I marked yes for "I have updated the documentation accordingly"

:wave: @mbrowne - thanks for the effort in getting this change together!
While I too can see this in the UI, I don't think we want to make a habit of introducing undocumented functionality in case it hasn't yet been intended for public API usage. Instead, do you mind raising a support ticket with Cloudflare (via http://support.cloudflare.com/) and they can pass through the information to the appropriate team? If not, please let me know and I'll raise one.
Once we have an answer on that, we can proceed with adding this one.

@jacobbednarz Given that this repo is owned by Cloudflare, I figured one of the developers here would know the status of this field. But I did also submit a support ticket yesterday. This is the relevant part of the response I received so far:

Yes, we have escalated this issue to our documentation team to check further.
Meanwhile, can you please try to add this field  "priority":1 in the API call and see if sets the priority while creating zone lockdown rules and let us know is it giving any error?
example:

curl -X POST "https://api.cloudflare.com/client/v4/zones/023e105f4ecef8ad9ca31a8372d0c353/firewall/lockdowns" \
     -H "X-Auth-Email<EMAIL_ADDRESS>\
     -H "X-Auth-Key: c2547eb745079dac9320b638f5e225cf483cc5cfdda41" \
     -H "Content-Type: application/json" \
     --data '{"paused":false,"priority":1,"description":"Restrict access to these endpoints to requests from a known IP address","urls":["api.mysite.com/some/endpoint*"],"configurations":[{"target":"ip","value":"<IP_ADDRESS>"}]}'

So apparently they are telling customers it's OK to use this, but we still need an official answer. So feel free to keep this PR on hold until I hear back from the "documentation team". But it seems likely this was just an oversight and not deliberately left out of the documentation.

I received a new response from Cloudflare:

As far as I'm concerned the "priority" parameter is meant to be documented. While you create a zone lockdown rule with priority in the dashboard, if you open the developer tools of browser, you will see the corresponding POST request with payload, which includes "priority" field as well. Hope this helps verify the API call you want to make.
We have an internal ticket to follow up the documentation update and we will keep you posted once we have updates.
Thanks for your patience.

So I think it would be fine to go ahead and merge this, but it's your call of course. Thanks :)

Priority is considered an 'advanced' feature and doesn't show up in the UI unless configured by API. Once set by the API it actually will populate in the UI. It is an officially supported field that should be included in here.
Here are some official docs about priority: https://developers.cloudflare.com/firewall/cf-firewall-rules/order-priority/

@garrettgalow Have you mixed up issues here? 😛 This issue is about Zone Lockdown whereas your link is for Filter based firewall. Or have I missed something here?

Looks like it's now documented:
https://api.cloudflare.com/#zone-lockdown-properties
Can this be merged now (when you have a moment)?

@mbrowne have you tested this with a priority of 0? i.e. the zero value of this new field. I checked some code internally and it looks like the "priority" should either be null, missing, or in the range [1,999999]. If correct, then this change would break clients that do not provide a value for "priority".
Either setting omitempty in the json struct tag, or using a *int, would likely work. I don't have an account to test this against at the moment. Let me know if you can confirm. Cheers!

Sorry for the delay, and good catch—I've now added omitempty to the Priority field and tested adding a rule both with and without a Priority, and it's working well.

@mbrowne Looks like CI is failing due to your change (see 1.11.x https://travis-ci.org/cloudflare/cloudflare-go/jobs/533886264#L528). Could you take a look?

cc @patryk @zackproser

@patryk or @zackproser, can you approve? Just a friendly reminder that this should be ready to go now. Thanks @jacobbednarz.

Oh yes, absolutely! Sorry for the delay, @mbrowne. I appreciate your patience and thanks for help.


Can't reproduce the result of ERM.

train PACS and Office Home one time, and the result is
Office Home: Last: 62.4, SWAD: 70.8 , Oracle: 66.3
PACS: Last: 82.2, SWAD: 88.3, Oracle: 88.1
The Last result if ERM? It is not consistent with the result of your paper. So the ERM result is Oracle?

The Last is not the SWAD results, but the performance of the last model. As you wrote, SWAD results are 70.8 and 88.3. They are consistent with the results of the paper.

So, where I can see the result of ERM w/o SWAD

Our ERM results were from HP search. Searched ERM HPs are different from the SWAD reproduced commands we provided. This repository does not aim to reproduce ERM results, but here are some tips for this: 1) Set HP by referring to https://github.com/khanrc/swad/issues/3#issuecomment-998474029 2) Turn off SWAD 3) Check the iid row results instead of last or oracle row.

OK, thank you.


Feature request: ability to stop nyc process propagation

It would be nice if there was some environment variable that could be set that would disable the nyc injection in a child process (and any of its descendants).
The use case is for writing multi-process tests for things like process supervisors where instrumenting both the supervisor and the example app it is running is needless overhead when what is being tested is how the supervisor responds in integration tests.
I'm thinking something like NYC_DISABLE or something, or maybe a max depth on the number of process boundaries to cross.

I like this idea, feel free to submit a patch if you have a chance to look at this before I do.

I spent a few minutes on it when I reported it, but couldn't see a straightforward way to do it. I'll definitely submit a PR if I do figure it out, but I wouldn't put any wagers on when that'll happen.


Grant-CsOnlineVoiceRoutingPolicy -Identity<EMAIL_ADDRESS>-PolicyName "Policy" generating error

With 2.5.1 Invoke-CsCommandInfo : User does not exist
At C:\Program Files\WindowsPowerShell\Modules\MicrosoftTeams\2.5.1\net472\SfBORemotePowershellModule.psm1:173 char:13

$steppablePipeline.Begin($isExpectingInput, $ExecutionCon ...

CategoryInfo : InvalidOperation: ({ Identity = Ce...ntUserRequest }:<>f__AnonymousType39`3) [Grant-CsUs
erPolicy_GrantExpanded], Exception
FullyQualifiedErrorId : ClientError,Microsoft.Teams.ConfigAPI.Cmdlets.Generated.Cmdlets.GrantCsUserPolicy_GrantE
xpanded

When downgraded to 2.3.1 it works
Unsure how to attach both psm1's but hopefully this is enough to help

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

ID: a58aa424-e87c-e4cb-d6a3-49a0d842f831
Version Independent ID: 8d1e09fa-7896-56e8-c294-8cac6c35d8d3
Content: Grant-CsOnlineVoiceRoutingPolicy (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Grant-CsOnlineVoiceRoutingPolicy.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

Sorry, should add this was with a Connect-MicrosoftTeams, but I can't find this command among the teams commands though it does say at the top it will work in teams

Hello @cegough
How are you? hope you are doing good.
I am sorry, I did not fully understand you, to analyze if we need to correct any documentation, can you please detail what you tried to run? thanks

Sorry for the delayed response
We were working with a customer, this was their first time using direct routing, we'd already setup the domains / calling plans / policies etc... and were waiting for the number to port, had a resource account created with a license
Connect-MicrosoftTeams
They entered their credentials, we ran a Get-CSOnlineUser to check and it worked
When we tied to apply the Policy to the resource account via: Grant-CsOnlineVoiceRoutingPolicy -Identity<EMAIL_ADDRESS>-PolicyName "Policynamewashere"
It returned the error I pasted in the ticket, referencing the lines also in the call
We had tried it on a standard user previously and it worked, we troubleshot it a while and then downgraded to the other version of teams I mentioned in the note I submitted (sorry, it was a while ago) it worked on the resource account
It seems like this version may not like the Virtual License possibly or recognize a resource account as a user?
Claire Gough, Tier III Technical Specialist
<EMAIL_ADDRESS>Office:<PHONE_NUMBER>
@.***
British Columbia - Alberta - Manitoba - Ontario - Quebec - Arizona
www.unityconnected.comhttps://www.unityconnected.com/
This message contains confidential information and is intended only for the individual named. If you are not the named addressee, you should not disseminate, distribute or copy this email. Please notify the sender immediately by email if you have received this email by mistake and delete this email from your system.
@.***
From: Andres Mariano Gorzelany @.>
Sent: Thursday, September 30, 2021 7:04 PM
To: MicrosoftDocs/office-docs-powershell @.>
Cc: Claire Gough @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Grant-CsOnlineVoiceRoutingPolicy -Identity @.*** -PolicyName "Policy" generating error (#8336)
Hello @cegoughhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcegough&data=04|01|cgough%40unityconnected.com|b14fced6617b4a951ef608d984774940|8ddfdc7886484e968aceb311c211c8b2|0|0|637686470133934087|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gZViG%2FHFvnYgANP5ADu9ovLxw9AhJ6lxJzm0gHIjdA0%3D&reserved=0
How are you? hope you are doing good.
I am sorry, I did not fully understand you, to analyze if we need to correct any documentation, can you please detail what you tried to run? thanks

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Foffice-docs-powershell%2Fissues%2F8336%23issuecomment-931812423&data=04|01|cgough%40unityconnected.com|b14fced6617b4a951ef608d984774940|8ddfdc7886484e968aceb311c211c8b2|0|0|637686470133944042|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=o6lsukTnUAkjfbZAyQCqH%2F4swvI5I6FXjcn96jZdHYU%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQWKKGBLTWGUUW5GQ4MLHCTUEUCGFANCNFSM5EASKGTQ&data=04|01|cgough%40unityconnected.com|b14fced6617b4a951ef608d984774940|8ddfdc7886484e968aceb311c211c8b2|0|0|637686470133944042|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gYD4WTX2RomTK6HZ2w5GxE5%2Bs220TvMi4kqHgMYi%2BBM%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|cgough%40unityconnected.com|b14fced6617b4a951ef608d984774940|8ddfdc7886484e968aceb311c211c8b2|0|0|637686470133953998|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Zx4b%2Fr8416Q2zroz9svMHxQ%2Bomqou%2Fwx56Q4nY4Pv4o%3D&reserved=0 or Androidhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|cgough%40unityconnected.com|b14fced6617b4a951ef608d984774940|8ddfdc7886484e968aceb311c211c8b2|0|0|637686470133953998|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=jvKiBaNNcyHlbY%2FG2bNTQ%2FXEjkhscTcOTfnVy1z2kW0%3D&reserved=0.

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


Hold Alt to Drag

Description of the new feature / enhancement
hold alt and click on a window to move it around. or right click to resize
Scenario when this would be used?
Any time you need to move a window you need to go grab the 20 pixels that are the top of the window, what if you could just click anywhere on the window to move it.
Moving a window is a usecase for every user of a muse/windowed user interface
Supporting information
There is a tool that does this for windows: https://stefansundin.github.io/altdrag/ which does a phenomenal job, the only thing it doesn't do is integrate with Fancy Zones.
I'd like either altdrag to work with fancyzones or powertoys to integrate it themselves so it isn't a second app

/dup #269


PowerToys Run bug report

Microsoft PowerToys version
0.82.0
Installation method
PowerToys auto-update, Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
everytime when I rund powertoys run

2024-09-09.txt
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

This was fixed in a newer build.  Please upgrade.  /dup https://github.com/microsoft/PowerToys/issues/33504


Allow rpm versioning to be customized via env flags

Set RPM_VERSION to change rpm version string
Set RPM_RELEASE to change rpm release string

Looks fine, I'll just tweak a couple of variable names before merging.

I added a commit on top of this here: https://github.com/logcabin/logcabin/compare/pull-138
@cseawood: I'm ready to merge if you're ok with it


68000: Multiple registers returning

For example, I have a function that reads (A0) to D0 which is return result, and also changes A0, because of operation (A0)+ or -(A0), or adda.l A0, #4.
Is it possible to specify that some function changes multiple registers?

You can check "Use Custom Storage" when editing a function, then double click on the Storage cell  related to the return (it will be D0:4 for example). You can then add another register there, or change the return value in any way.

But it does require special returning type for that. For example I want to
return one word register and one dword. 6 bytes. What type should I use?
On September 9, 2019 7:18:06 PM Edu Garcia<EMAIL_ADDRESS>wrote:

You can check "Use Custom Storage" when editing a function, then double
click on the Storage cell related to the return (it will be D0:4 for
example). You can then add another register there, or change the return
value in any way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Well, you can create a struct with two fields and try returning that, not sure if it will work but it might.
Otherwise, I'm not really sure, maybe somebody has a better answer 🙂

Seems that you're right. It is possible to create a structure, which will store all registers, and then set it as the return type, with specifying which registers it will store.

Glad it worked!


tools/test-help-documentation: Verify api/ docs.

Fixes: #7327
@timabbott: FYI. :) Btw, I have been working on implementing this for the integrations docs, but it is turning out to be trickier than I thought. If you use the same code on /integrations as a start_url, every sub-page seems to get visited but the links don't seem to be linted for some reason. So I instead tried passing URLs like /integrations/doc-html/<webhook_name> (similar to how we test docs in test_docs.py) but the script gets stuck at /integrations/doc-html/github_webhook for some reason. I'm investigating this further. If you have any thoughts, I'd love to hear them! Thanks! :)

This looks pretty good; I posted one comment.  Did you also try adding a broken link to make sure the test actually catches bugs?

@timabbott: Cool, I'll make a common subclass yeah. And yes, I did add a broken link and unused images and test it. :)

@timabbott: Updated and tested again for both /api and /help docs for both unused images and broken links! :)

Cool, merged this, using it to fix a few broken links from content being moved over in @derAnfaenger's recent PR :).  Thanks @eeshangarg!!


read_summary.jl may fail when no summary.csv file exists

I believe the update "Update read_summary.jl" on Aug 4 may have introduced an error, though it is only likely to be encountered when the "summary".csv file does not exists.
Specifically, line 23 or read_summary.jl calls stan_summary(m) where m is a Stanmodel.  However, as I understand it, stan_summary requires two arguments. I.e.: stan_summary(model::Stanmodel, filecmd::Cmd; CmdStanDir=CMDSTAN_HOME). Consequently, read_summary throws the following error:  "> MethodError: no method matching stan_summary(::Stanmodel)

Thank you, think I checked this, but apparently not! As in your other issue (https://github.com/StanJulia/CmdStan.jl/issues/108#issue-806567628), which version of cmdstan are you using?

I am presently using 2.23.0. I know your notes suggest cmdstan.jl may only work up thru 2.19, but it had been working until recently.
The only reason I saw this, however, is because of the absence of the summary.csv file (as identified in #108)

On Feb 11, 2021, at 12:38 PM, Rob J Goedman<EMAIL_ADDRESS>wrote:
Thank you, think I checked this, but apparently not! As in your other issue (#108 (comment) https://github.com/StanJulia/CmdStan.jl/issues/108#issue-806567628), which version of cmdstan are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/StanJulia/CmdStan.jl/issues/109#issuecomment-777666934, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2MOIBFO4Q3GJISR2PDKO3S6QIZHANCNFSM4XPHIPHA.

Aah, I need to update that note. I believe this issue became a real issue in cmdstan v2.26. That's when I had to change the command switch.
As a work around you should be able to a previous version of CmdStan.jl. I'll dig a bit bit deeper this afternoon.


bump premailer version

This gem relies on a very old version of premailer, namely one that includes hpricot whose hpricot_slate_method_added magic causes issues with sorbet. Please update this dependency to use a more recent version of premailer.

This is the dependency definition:
https://github.com/fphilipe/premailer-rails/blob/v1.11.1/premailer-rails.gemspec#L24
premailer is currently at 1.11.1. If you do a fresh install of premailer-rails, it will install the latest premailer version. Thus, you should be able to simply run bundle update premailer to get a newer version of premailer in your project.

Thank you for the quick and educational reply, new ecosystem I'm working in.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in 83368a7.


Duplicate id in svg

I render multiple avatars in a custom avatar editor and when some of them are removed from the DOM, then the remaining svg does not render well anymore on Safari only.

I have tried the following code (https://github.com/shivam1410/fangpenlin-avataaars-generator-angular/pull/5) but it did not work, on the contrary, it broke the svg:
var svgSel = '#root > div > div > main > main > div > div > div.avatar-main-preview > svg';
document.querySelectorAll(svgSel + ' [mask^="url("]').forEach(node => {
    var attributMask = node.getAttribute("mask");
    node.setAttribute("mask",  + attributMask.replace('url(', 'url(' + location.href));
});
document.querySelectorAll(svgSel + ' [filter^="url("]').forEach(node => {
    var attributFilter = node.getAttribute("filter");
    node.setAttribute("filter",  + attributFilter.replace('url(', 'url(' + location.href));
});
document.querySelectorAll(svgSel + ' [fill^="url("]').forEach(node => {
    var attributFill = node.getAttribute("fill");
    node.setAttribute("fill",  + attributFill.replace('url(', 'url(' + location.href));
});

Could this be because:

there is a duplicate id of "Avataaar" inside src/avatar/index.tsx?
there is id="Mask" in 2 separate files: src/avatar/index.tsx and src/avatar/top/WinterHat1.tsx?

My confusion here is:

should we use a class instead of an id
or have completely 2 different names for the 2 ids that are conflicting here?

Thank you!

This issue is resolved when instead of removing my avatars from the DOM, I put them away from the viewport and set their opacity to 0.


Specific LLVM versions fail

When I specified version: "17.0.6", the action failed:
 Checking known assets (os=linux, arch=x64, version=17.0.6)...
Error: Unsupported version for platform (os=linux, arch=x64, version=17.0.6)!
    at $73b874f14e970995$export$da446a5ee2572b7 (/home/runner/work/_actions/KyleMayes/install-llvm-action/bb56a8457a77b95a5fa576c23731cbb162fd2056/dist/main.js:7615:41)
    at $73b874f14e970995$var$install (/home/runner/work/_actions/KyleMayes/install-llvm-action/bb56a8457a77b95a5fa576c23731cbb162fd2056/dist/main.js:7633:60)
    at $73b874f14e970995$export$889ea624f2cb2c57 (/home/runner/work/_actions/KyleMayes/install-llvm-action/bb56a8457a77b95a5fa576c23731cbb162fd2056/dist/main.js:7663:16)
    at $933db609b41224a7$var$main (/home/runner/work/_actions/KyleMayes/install-llvm-action/bb56a8457a77b95a5fa576c23731cbb162fd2056/dist/main.js:7681:61)
Error: Unsupported version for platform (os=linux, arch=x64, version=17.0.6)!
    at Object.<anonymous> (/home/runner/work/_actions/KyleMayes/install-llvm-action/bb56a8457a77b95a5fa576c23731cbb162fd2056/dist/main.js:7688:1)
    at Module._compile (node:internal/modules/cjs/loader:1241:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1295:10)
    at Module.load (node:internal/modules/cjs/loader:1091:32)
    at Module._load (node:internal/modules/cjs/loader:938:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12)

...although the corresponding entry exists in assets.json. The README says it should be supported, however, from the code it looks like it always appends a dot to the version number, so no matches can be found. Looks like it was broken with this recent commit.

Thanks for the contribution!
I've released v2.0.1 and pointed v2 at that version.


Fix https://github.com/bayasdev/envycontrol/issues/121

@Boria138 are you sure nvidia-fallback isn't a Fedora only service?

@Boria138 are you sure nvidia-fallback isn't a Fedora only service?

This is only a Fedora service, but Fedora has two Nvidia driver packages from rpmfusion and negativo17, negativo17 does not have this service, so I added a check for the existence of this service


Plugin fetches jwkEndpoints a lot of times over a short period of time

Hello,
We're running into an issue where the plugin will start fetching jwkEndpoints a lot, as opposed to every 15 minutes as I'd expect based on the source code in jwt.go.
I initially discovered this through tracing, and confirmed it by enabling DEBUG level logs in Traefik.
This is how we are configuring the Middleware (headers and namespace cleaned for simplicity):
spec:
  plugin:
    jwt:
      JwtHeaders:
        x-head-aud: aud
        x-head-exp: exp
        x-head-sub: sub
      Keys:
        - http://http.<namespace>/oidc/v1/jwks
      PayloadFields:
        - sub
        - exp
        - aud
      Required: true

Checking the Debug log, I noticed "time" is zeroed, and ip, port, url and sub are empty. I wonder if the plugin doesn't really support internal services as key provicers?
Note that everything else seems to work fine with the Middleware, but the constant fetching of the jwks seems to get worse the longer we let the traefik pod run. Once we restart the pod, it seems to go back down to every 15 minutes for a little while before it starts querying jwks non-stop.
traefik.log

Hello
As a maintainer I am not using this functionality.
Are you willing to provide a PR or an investigation ?

Willing to assist in whichever way I can.
This is the part of the code that I'd expect will keep the keys "cached" for 15 minutes: https://github.com/traefik-plugins/traefik-jwt-plugin/blob/main/jwt.go#L198-L203
But I'm not sure how Traefik handle plugins. Does it load a copy of the plugin every new request? Is the plugin reused across requests? I wonder if it is simply a matter of old running plugins not being "cleaned up" after a request is completed.

Willing to assist in whichever way I can.
This is the part of the code that I'd expect will keep the keys "cached" for 15 minutes: https://github.com/traefik-plugins/traefik-jwt-plugin/blob/main/jwt.go#L198-L203
But I'm not sure how Traefik handle plugins. Does it load a copy of the plugin every new request? Is the plugin reused across requests? I wonder if it is simply a matter of old running plugins not being "cleaned up" after a request is completed.

it may be the case. I've been fixing one cashing issue in the other plugin.
Since traefik docs doesn't supply proper explanation - it's only matter of experiments.
are you able to research this?

Hi
We ran into a similar situation. We use a lot of dynamic configurations (file provider) for each of our customers. We deploy them all at the same time.
This is from our daily access.log:
[root@proxy logs]# grep "/certs" access.log | jq .'RequestPath' | cut -d'/' -f3 | sort | uniq -c
   8021 customer1-prod
  20358 customer2-prod
  20358 customer3-prod
  20360 customer4-prod
  20358 customer5-prod
  ...
  20359 customerX-prod
  123 other-service

These are requests to the URL configured in the key section of the plugin (see below), grouped by customer name. Interestingly, customer1-prod is one of our newest customers.
We suspect that this high number of requests could be caused by the dynamic files, spawning a new BackgroundRefresh() routine every time the config changes/reloads. The other-service does not get deployed that often, therefore it's count is smaller. We do not restart Traefik after our deployments.
Current middleware config:
http:
  middlewares:
    # Validate access token using Traefik Plugin.
    web-token:
      plugin:
        token:
          PayloadFields: ["exp", "sub", "sid", "iss"]
          Required: true
          JwtCookieKey: token
          JwtQueryKey: token
          Keys:
            - https://example.com/realms/customer1-prod/protocol/openid-connect/certs

Mabye this helps.

Nice findings @spezifanta.
From my end, what I noticed is that Traefik 3.0.0 (beta and rc versions) spam this a lot more than Traefik 2.x. We are only using Traefik 3 in one of our clusters vs other clusters, so could also be that something changed between v2 and v3 in terms of how plugins / middleware is used.

We did a bit of more diggging and forked the plugin.
At the moment we are pretty sure, that go routinges are not garbe collected when reloading a dynamic config.
Basically, we added an unique identifer every time BackgroundRefresh() is called to see whats going on.
func (jwtPlugin *JwtPlugin) BackgroundRefresh(context context.Context, identifier string) {
	for {
		logInfo(fmt.Sprintf("Started BackgroundRefresh %s",identifier)).print()
		select {
		case <-context.Done():
                        # This gets never called?!
			logInfo(fmt.Sprintf("Ended BackgroundRefresh %s",identifier)).print()
			return
		default:
			logInfo(fmt.Sprintf("Do BackgroundRefresh %s",identifier)).print()
			jwtPlugin.FetchKeys()
			time.Sleep(15 * time.Minute) // 15 min
		}
	}
}

This is the output after starting Traefik and changing a dynamic config two times.
 This should not happen.
We have not figgured out how to cleanup this routine.

[Looks like we are not the only one](https://community.traefik.io/t/go-routines-in-middlewares-are-orphaned-after-config-reload/2513)

So dows it make sense to introduce module level variable for keeping goroutine instance ?
So om start it can be cancelled if exists.

Since i couldn't get anything to work with runtime.SetFinalizer(). I drafted a PR (https://github.com/traefik-plugins/traefik-jwt-plugin/pull/68) that will (at least in our scenario) mitigate this issue greatly. Hope it helps. Maybe someone can come up with a more complete solution for this issue. But the Traefik project seems unwilling to help with this issue from their side so far.


[Bug]:  setCamera animaition issue

Mapbox Implementation
Mapbox
Mapbox Version
10.0.0-beta.60
Platform
iOS, Android
@rnmapbox/maps version
10.0.0-beta.11
Standalone component to reproduce
const focusUser = useCallback(() => {
if (setLocationValue) {
camera.current?.setCamera({
pitch: 30,
heading: 0,
zoomLevel: 12,
animationMode: 'none',
animationDuration: 0,
centerCoordinate: [setLocationValue.lon, setLocationValue.lat],
});
}
}, [setLocationValue]);
useEffect(() => {
setTimeout(() => {
focusUser();
}, 2000);
}, [focusUser]);
<MapboxGL.Camera
followUserLocation={false}
defaultSettings={{ zoomLevel: isIos ? 0 : 5, centerCoordinate: [127, 34] }}
ref={camera}
maxZoomLevel={20}
/>
Observed behavior and steps to reproduce
After connecting the map, other buttons cannot be pressed until the animation of the setCamera function is finished in camera.
Expected behavior
No response
Notes / preliminary analysis
No response
Additional links and references
No response

@junleejaehyuk thanks for the report and the sample code, but I don't understand what's the issue. What's are other buttons?


Cannot get it to start/work

Need Auto click to click a button that says "Next" after a countdown. Right clicked the button and clicked Auto Clicker. What are my next steps to get it to start?

For a quick start refer below
Video: https://www.youtube.com/watch?v=M3yNuBkJW3g&t=2s
Wiki: https://getautoclicker.com/docs/getting-started


[Parquet] Intermittent test failure thread 'encodings::encoding::tests::test_bool' panicked at 'Invalid byte when reading bool'

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-11027
The following parquet tests intermittently fail
{code}
test encodings::encoding::tests::test_bool ... FAILED
thread 'encodings::encoding::tests::test_bool' panicked at 'Invalid byte when reading bool', parquet/src/util/bit_util.rs:73:18
{code}
ARROW-10943 tracks  an intermittent error in the parquet tests. We think we fixed one error but there is still something wrong.
I can reproduce this locally  at ca685a0c08bb41f43a80e5605e4cc8f9efb77cca by running the following shell script:
{code}
#!/bin/bash
run a command in a loop
cd /Users/alamb/Software/arrow2/rust
export PARQUET_TEST_DATA=pwd/../cpp/submodules/parquet-testing/data
export ARROW_TEST_DATA=pwd/../testing/data
export RUST_BACKTRACE=1
i=0
while true; do
echo "iteration $i"
cargo test -p parquet  -- test_bool
i=$((i+1))
done
{code}
Here is an example of the full stack trace / errors:
{code}
test encodings::encoding::tests::test_bool ... FAILED
failures:
---- encodings::encoding::tests::test_bool stdout ----
thread 'encodings::encoding::tests::test_bool' panicked at 'Invalid byte when reading bool', parquet/src/util/bit_util.rs:73:18
stack backtrace:
0: std::panicking::begin_panic
at /Users/alamb/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panicking.rs:505
1: ::from_ne_bytes
at ./src/util/bit_util.rs:73
2: parquet::util::bit_util::from_ne_slice
at ./src/util/bit_util.rs:32
3: parquet::encodings::rle::RleDecoder::get_batch
at ./src/encodings/rle.rs:402
4: <parquet::encodings::decoding::RleValueDecoder as parquet::encodings::decoding::Decoder>::get
at ./src/encodings/decoding.rs:322
5: parquet::encodings::encoding::tests::put_and_get
at ./src/encodings/encoding.rs:1271
6: <T as parquet::encodings::encoding::tests::EncodingTester>::test_internal
at ./src/encodings/encoding.rs:1214
7: parquet::encodings::encoding::tests::EncodingTester::test
at ./src/encodings/encoding.rs:1159
8: parquet::encodings::encoding::tests::test_bool
at ./src/encodings/encoding.rs:969
9: parquet::encodings::encoding::tests::test_bool::{{closure}}
at ./src/encodings/encoding.rs:966
10: core::ops::function::FnOnce::call_once
at /Users/alamb/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/ops/function.rs:227
11: core::ops::function::FnOnce::call_once
at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/core/src/ops/function.rs:227
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
{code}

Comment from Andrew Lamb(alamb) @ 2020-12-24T13:24:32.210+0000:
FYI<EMAIL_ADDRESS>
When  tried running with some of the code from https://github.com/apache/arrow/pull/8948 (a previous attempt to figure out what is going on here) the problem still happens on my machine:

{code}
(arrow_dev) alamb@MacBook-Pro:~/Software/arrow/rust$ git cherry-pick 16051b53cd3a1eebdba7a50356ee49585b69e152

(arrow_dev) alamb@MacBook-Pro:~/Software/arrow/rust$ bash ~/Documents/loop.sh 2>& 1 | tee /tmp/loop.log | grep panic
thread 'encodings::encoding::tests::test_bool' panicked at 'Invalid byte when reading bool', parquet/src/util/bit_util.rs:73:18
   0: std::panicking::begin_panic
             at /Users/alamb/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/panicking.rs:505
{code}

When I picked the second commit from that PR the problem still happens

{code}
(arrow_dev) alamb@MacBook-Pro:~/Software/arrow/rust$ git cherry-pick 04f680e121eed9fe524f60b91faa5e7b64fa9921

(arrow_dev) alamb@MacBook-Pro:~/Software/arrow/rust$ git cherry-pick 04f680e121eed9fe524f60b91faa5e7b64fa9921
[master 870ff05f9] [Rust][Hack] Replace murmur2-64a with wyhash
 Author: Greg Bowyer
 Date: Wed Dec 16 17:13:24 2020 -0800
 2 files changed, 3 insertions(+), 2 deletions(-)
(arrow_dev) alamb@MacBook-Pro:~/Software/arrow/rust$ bash ~/Documents/loop.sh 2>& 1 | tee /tmp/loop.log | grep panic
thread 'encodings::encoding::tests::test_bool' panicked at 'assertion failed: `(left == right)`
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:483
   1: std::panicking::begin_panic_fmt
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:437
thread 'encodings::encoding::tests::test_bool' panicked at 'assertion failed: `(left == right)`
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:483
   1: std::panicking::begin_panic_fmt
             at /rustc/7eac88abb2e57e752f3302f02be5f3ce3d7adfb4/library/std/src/panicking.rs:437
^C
{code}


Implement getting RPM package version

Work on #1187

This is ready for review after minor changes. Just requires re-approving.

@johnsonw @utopiabound please take a look at the PR, all the comments have been handled

@jgrund PR is ready for merge


Fix to display sessions upon login

@kotfic Put this one in a while back, it got stuck in a limboed PR.
I've split it out and am LGTMing it.

Fixes #https://github.com/Kitware/minerva/issues/147.  I'm assuming.
:stuck_out_tongue_closed_eyes:

Thanks @jbeezley and @zachmullen for tracking this down.


placement of round window

MP has round window terms under middle ear, foramen and temporal bone petrous part
HPO has this under middle and inner ear
MP uses http://purl.obolibrary.org/obo/UBERON_0002502 in the EQ so it seems odd to not have this under inner ear.
Looking in Uberon this is under foramen of skull and petrous part of temporal bone but not under ear at all
Need to review the placement of this, at the least I think moving it out from under middle ear or adding the inner ear parent may be needed

Looking at diagrams in Wikipedia it seems correct to have this as middle https://en.wikipedia.org/wiki/Round_window

MP should add this to inner ear as well

abnormal round window morphology MP:0004480 (and child terms) has been additionally placed under abnormal inner ear morphology MP:0000026.


Endless Listview to Json very long

Hello, its a great this tutorial and code example, but. Can you helpme to implement Endless on bottom scroll list, to load more items in the Listview of movies if my Json file contan more of 50 movies?
Thanks!

private int preLast;
--> listQuestions.setOnScrollListener(this); // this goes inside oncreate or something
@Override
public void onScrollStateChanged(AbsListView absListView, int i) {
// not used
}
@Override
public void onScroll(AbsListView lw, final int firstVisibleItem,
final int visibleItemCount, final int totalItemCount) {
switch(lw.getId()) {
    case R.id.listQuestions:

        // Make your calculation stuff here. You have all your
        // needed info from the parameters of this function.

        // Sample calculation to determine if the last
        // item is fully visible.
        final int lastItem = firstVisibleItem + visibleItemCount;
        if(lastItem == totalItemCount) {
            if(preLast!=lastItem){ //to avoid multiple calls for last item
                Log.d("Last", "Last");
                preLast = lastItem;

                // Call your code here, Congrats you have reached the bottom of the list.
            }
        }
}

}

Not work, i implement the code, but when the application run, the list of more 50 movies appear, i want to  see the first 10 movies, and when the scroll top to bottom, load the others 10 movies. :/

@HackJoues  Have you got it?  (... the scroll to bottom, load the others 10 movies ...)
@hishamMuneer  Congratulation for your tutorial! It's realy good. Did you have extended other functionalities in this example? (Like slider in the photo of DetailActivity.java or  Animation in the transition)
It would really be useful any extension!!


Saw transient GcMetricsIT failure

Stacktrace below. Given the code where this failed and the fact that it failed on Jenkins, I suspect this was because it picked up a metrics test output file from a previous run. This test should be reworked, so it uses a unique temporary file name for the metrics output, so it doesn't have issues when run concurrently with other tests, or accidentally pick up output files from previous runs.
java.lang.AssertionError
	at org.junit.Assert.fail(Assert.java:87)
	at org.junit.Assert.assertTrue(Assert.java:42)
	at org.junit.Assert.assertTrue(Assert.java:53)
	at org.apache.accumulo.test.functional.GcMetricsIT.sanity(GcMetricsIT.java:155)
	at org.apache.accumulo.test.functional.GcMetricsIT.gcMetricsPublished(GcMetricsIT.java:109)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:288)
	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:282)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.lang.Thread.run(Thread.java:834)

The filename is determined by the metrics config file - so just changing the filename may not be straight forward. Also, concurrent tests will be another issue - normally there would be only one gc running.

It's also possible my assumption about how the test failed is wrong. I was going off of the embedded comment in the gcMetricsPublished() method, which explains that a previous run or another test could affect what is parsed from the file, since the file is appended to.
In the stack trace above, it appears the value parsed was of an event time that occurred before the current test started, which I thought could only make sense if it was from a different test... but I did not evaluate deeply.

I didn't look closely either - but I will.  Either this test or possibly one of fate metrics tests was written so that it looked for an update - I'll check that this code does the same thing and that it's working as I intended.


Upgrade Go version to 1.23

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Before we release the next minor version (v0.9), upgrading Go version would be better.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/hold
for test-infra

Also we should update https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/Dockerfile#L1 and https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/cmd/experimental/kjobctl/README.md?plain=1#L12.

Should we update minimal version to 1.23? Maybe it's enough to update to 1.22.7? WDYT?

As I described, before new release, it should be great to upgrade to 1.23.
If we do not update it now, we lose the chance about 4 months since after releasing 0.9, we want to keep using the same Go version for the Kueue patch version.

As I described, before new release, it should be great to upgrade to 1.23.
If we do not update it now, we lose the chance about 4 months since after releasing 0.9, we want to keep using the same Go version for the Kueue patch version.

That's a good point. Thanks for the explanation.

Also we should update
https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/Dockerfile#L1
and
https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/cmd/experimental/kjobctl/README.md?plain=1#L12
.

Good point. Let me see it.

Also we should update
https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/Dockerfile#L1
and
https://github.com/kubernetes-sigs/kueue/blob/0d73e6a10a0b71110dd4f8706189d66085f69a58/cmd/experimental/kjobctl/README.md?plain=1#L12
.

Good point. Let me see it.

Done.

/retest

/lgtm
Thanks!

It seems that it takes a little time to synchronize the test-infra repository to the infra cluster.

/retest

Get ready 🚀
/hold cancel


Ignore reserved bit in WINDOW_UPDATE frame

Fixes https://github.com/grpc/grpc/issues/17840.
When Kestrel server sends the per-connection window update, the top bit of "Windows Size Increment" is set (which is against the spec!), but at the same time gRPC doesn't ignore the set bit (also against the spec!).
This PR makes grpc ignore the top bit.
See
https://http2.github.io/http2-spec/#WINDOW_UPDATE
https://http2.github.io/http2-spec/#FramingLayer

R: A reserved 1-bit field. The semantics of this bit are undefined, and the bit MUST remain unset (0x0) when sending and MUST be ignored when receiving.

These are the raw bytes I got in the client side when debugging:
(gdb) x/13b t->read_buffer->slices[0].data.refcounted.bytes
0x7f8edc00c448:	0x00	0x00	0x04	0x08	0x00	0x00	0x00	0x00
0x7f8edc00c450:	0x00	*0x80	0x01	0x00	0x08
(*) reserved bit set here

Awesome job!

Btw, there's a few more places in the spec where "R" bit is used and I'm not sure we are honoring them (e.g. Stream ID seems to have a top R bit). It might be worth do make a pass over the parser code.

Add a test?

@yashykt I was trying to see if there are tests for frame parsers and wasn't able to find any (and I don't feel like adding a whole new category of C core tests because I don't know the codebase that well).

Could we merge this now and worry about a test (if desired) in a follow up PR? I'd like to get a build with this change and re-test against Kestrel to make sure there are no other issues. If there are more issues we can try to solve them before next weeks release.

@JamesNK Can you verify that this change fixes you before we merge it, i.e., can you build with this PR in it? That would be one way to verify that this helps you

I don't have C build tools setup on my dev machine.

We believe we have located the issue in Kestrel's HTTP/2 stack. I think the same issue will show up in the GOAWAY frame
https://tools.ietf.org/html/rfc7540#section-6.8
    +-+-------------------------------------------------------------+
    |R|                  Last-Stream-ID (31)                        |
    +-+-------------------------------------------------------------+
    |                      Error Code (32)                          |
    +---------------------------------------------------------------+
    |                  Additional Debug Data (*)                    |
    +---------------------------------------------------------------+

It also has a reserved bit. Double check in gRPC that you ignore the that bit as well.

I went through the spec and there are a few places that is affected by the same behaviour:

Stream ID of every frame:

    +-----------------------------------------------+
    |                 Length (24)                   |
    +---------------+---------------+---------------+
    |   Type (8)    |   Flags (8)   |
    +-+-------------+---------------+-------------------------------+
    |R|                 Stream Identifier (31)                      |
    +=+=============================================================+
    |                   Frame Payload (0...)                      ...
    +---------------------------------------------------------------+

Promised Stream ID of Push Promise:

    +---------------+
    |Pad Length? (8)|
    +-+-------------+-----------------------------------------------+
    |R|                  Promised Stream ID (31)                    |
    +-+-----------------------------+-------------------------------+
    |                   Header Block Fragment (*)                 ...
    +---------------------------------------------------------------+
    |                           Padding (*)                       ...
    +---------------------------------------------------------------+

Note that Push Promise isn't implemented in Kestrel so we shouldn't run into this.
3. Go away and Window update as mentioned.

@JamesNK Can you verify that this change fixes you before we merge it, i.e., can you build with this PR in it? That would be one way to verify that this helps you

I tried running the benchmark with patched version of C-core and I got no more errors (I was able to reproduce before the fix quite easily). So I think it's safe to say the fix is effective.

Also try with the channel count increased beyond 1. That generated errors as well.

Fixed simple_request_bad_client_test that was assuming that 0xffffffff (all ones) windows size increment is illegal.

Also try with the channel count increased beyond 1. That generated errors as well.

I tried different channel counts and didn't see a single error.

Known failures:
https://github.com/grpc/grpc/issues/17795
https://github.com/grpc/grpc/issues/17856


shapes: cross, xcross, utriangle, dtriangle, star1, star2, hexagon

Thanks!

No problem... glad I can contribute.
On Thu, Sep 17, 2015 at 11:56 PM, Daniel C. Jones<EMAIL_ADDRESS>wrote:

Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/dcjones/Gadfly.jl/pull/693#issuecomment-141336444.


Set runAsNonRoot and use RuntimeDefault seccomp profile

Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.
[ ] (Giant Swarm) If creating a release, bump the version and appVersion in Chart.yaml.

Tested on ghost


[Release] Bump to 1.4.0

Summary
Upgrades the Recorder to v1.4.0.
Implementation details
Increases the Playwright and Synthetics deps.
How to validate this change
Standard testing. Run the recorder, run the test browser, test code gen.

https://github.com/elastic/synthetics-recorder/pull/510/files supersedes this one


Fix for session clashes also for Single-Page-Apps

According to https://github.com/ohmybrew/laravel-shopify/pull/298 and the PR that i closed before for the issue https://github.com/ohmybrew/laravel-shopify/issues/295

Coverage decreased (-0.1%) to 99.497% when pulling fd9fb8a59bc61d64489e46af95277bbfdd9ff83b on StefanNeuser:master into 01c49909546c2583225147dadaa920277df9083e on ohmybrew:master.

@StefanNeuser @darrynten Can't tell on mobile which one of you did what :D so just tagging here to discuss.
Looks good at a glance, can you explain why this is needed... also the verifyRequest, should not be needed on every request? If the session is bad, going to a full or partial authentication flow will already verify the request.. unless I'm missing something (https://github.com/ohmybrew/laravel-shopify/blob/master/src/ShopifyApp/Requests/AuthShop.php#L46).

@ohmybrew take a look at my explanation on #298 for the reason behind my change.

@StefanNeuser I've opened some other PRs too with some refactoring - what I will do is after those are in I will be able to make an implementation that will allow an SPA to announce itself more securely.
I need to do this anyway, and this would achieve your objective in #299.

@darrynten , i agree and removed the code from your PR (patch)


Refactor the metrics collector

Create an object for the metrics collector to abstract this layer. Now it is easy to change the implementation of the backend service like prometheus and you can create some mocked test cases. For the mocked tests, you only need to re-implemented the interfaces of client and collector.
Please change the import paths from github.com/am3o/... to github.com/ben-st/...
Try it out and let me know your feedback about this kind of changes.

thanks, LGTM


opening your own dat/rotonde url in two tabs crashes beaker

i presume its due to both instances tying to have write access to the same dat-archive which causes it

It's more likely to be the common crash-bug that occurs when accessing dat:// pages. (Im actually debugging it right now.)

That shouldn't be happening anymore.


Created Base Search Page

created handlebars pages for search results and the partials for the cards. Edited some structure on the main page

Forgot to put the route back to homepage. Shouldn't be too much of an issue.


Update: KDE.CrowTranslate version 3.0.0

Microsoft Reviewers: Open in CodeFlow

@stephengillie https://github.com/microsoft/winget-pkgs/issues/152555


added support for the http patch method

Not quite sure what went wrong with the greeting step, but hope the rest is ok

Haha how odd! Thanks for raising this PR! I'll do a release with these changes today!

Once again, thank you so much for your contribution. Out of curiosity, may I ask how you heard about data-mocks @Dinosys

Once again, thank you so much for your contribution. Out of curiosity, may I ask how you heard about data-mocks @Dinosys

Well, we saw your presentation @ Angular Connect in London and soon after we started using your tool and told people like @Dinosys about it :) Now we're using it a lot in several apps. Nice work!

Quite right, loving this tool for local development.
We use it in angular

That's awesome 👍
I've just published version 4.0.2!

This is the API extension. You should bump minor version and reset the patch, instead of bumping patch version. BTW This is great tool, which I use in my daily routine (also saw it for the first time on YT #AngularConnect).


[BUG]: Crash when try embed a document using milvus

How are you running AnythingLLM?
Docker (local)
What happened?
I've tried search  in the docs but nothing is here https://docs.useanything.com/setup/vector-database-configuration/local/milvus
I've used a docker image of milvus, and it allow me to access web panel from my pc, but anythingllm have a error when try embed a document.
My milvus is https://github.com/milvus-io/milvus/releases/tag/v2.4.9

milvus-standalone-docker-compose-gpu.yml

And I've use it with podman-compose.
Milvus console output:
[minio]      | Status:         1 Online, 0 Offline.
[minio]      | API: http://<IP_ADDRESS>:9000  http://<IP_ADDRESS>:9000
[minio]      | Console: http://<IP_ADDRESS>:9001 http://<IP_ADDRESS>:9001
[minio]      |

Error from podman logs:
[collector] info: Collector hot directory and tmp storage wiped!
[collector] info: Document processor app listening on port 8888
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma

✔ Generated Prisma Client (v5.3.1) to ./node_modules/@prisma/client in 128ms

Start using Prisma Client in Node.js (See: https://pris.ly/d/client)

import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()

or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)

import { PrismaClient } from '@prisma/client/edge'
const prisma = new PrismaClient()

See other ways of importing Prisma Client: http://pris.ly/d/importing-client

Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "anythingllm.db" at "file:../storage/anythingllm.db"

21 migrations found in prisma/migrations

No pending migrations to apply.
[backend] info: [EncryptionManager] Loaded existing key & salt for encrypting arbitrary data.
[backend] info: [TELEMETRY ENABLED] Anonymous Telemetry enabled. Telemetry helps Mintplex Labs Inc improve AnythingLLM.
[backend] info: prisma:info Starting a sqlite pool with 29 connections.
[backend] info: [TELEMETRY SENT] {"event":"server_boot","distinctId":"7eecf98b-ccfd-40c5-a83b-5885cdac43b6","properties":{"commit":"--","runtime":"docker"}}
[backend] info: [CommunicationKey] RSA key pair generated for signed payloads within AnythingLLM services.
[backend] info: [EncryptionManager] Loaded existing key & salt for encrypting arbitrary data.
[backend] info: Primary server in HTTP mode listening on port 3001
[backend] info: [BackgroundWorkerService] Feature is not enabled and will not be started.
[backend] info: [MetaGenerator] fetching custome meta tag settings...
[backend] error: LMStudio:listModels Connection error.
[backend] info: [Event Logged] - update_vector_db
[backend] error: LMStudio:listModels Connection error.
[backend] info: Adding new vectorized document into namespace normal-es
[backend] info: [RecursiveSplitter] Will split with {"chunkSize":128,"chunkOverlap":20}
[backend] info: Chunks created from document: 317
[backend] info: [LMStudioEmbedder] fetch failed
[backend] error: addDocumentToNamespace LMStudio service could not be reached. Is LMStudio running?
[backend] error: Failed to vectorize TopfoodGuntry.txt
[backend] info: Adding new vectorized document into namespace normal-es
[backend] info: [RecursiveSplitter] Will split with {"chunkSize":128,"chunkOverlap":20}
[backend] info: Chunks created from document: 725
[backend] info: [LMStudioEmbedder] fetch failed
[backend] error: addDocumentToNamespace LMStudio service could not be reached. Is LMStudio running?
[backend] error: Failed to vectorize interpretacion_suenos_V2.md
[backend] info: [TELEMETRY SENT] {"event":"documents_embedded_in_workspace","distinctId":"7eecf98b-ccfd-40c5-a83b-5885cdac43b6","properties":{"LLMSelection":"lmstudio","Embedder":"lmstudio","VectorDbSelection":"milvus","TTSSelection":"native","runtime":"docker"}}
[backend] info: [Event Logged] - workspace_documents_added
[backend] info: Adding new vectorized document into namespace normal-es
[backend] info: [RecursiveSplitter] Will split with {"chunkSize":128,"chunkOverlap":20}
[backend] info: Chunks created from document: 386
[backend] info: [LMStudioEmbedder] Embedding 386 chunks of text with jina/jina-embeddings-v2-base-es/jina-embeddings-v2-base-es.gguf.
[backend] error: addDocumentToNamespace 13 INTERNAL: Received RST_STREAM with code 2 triggered by internal client error: Protocol error
[backend] error: Failed to vectorize Libretranslate.md
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error: 13 INTERNAL: Received RST_STREAM with code 2 triggered by internal client error: Protocol error
    at callErrorFromStatus (/app/server/node_modules/@grpc/grpc-js/build/src/call.js:31:19)
    at Object.onReceiveStatus (/app/server/node_modules/@grpc/grpc-js/build/src/client.js:193:76)
    at /app/server/node_modules/@grpc/grpc-js/build/src/call-interface.js:78:35
    at Object.onReceiveStatus (/app/server/node_modules/@zilliz/milvus2-sdk-node/dist/milvus/utils/Grpc.js:146:25)
    at Object.onReceiveStatus (/app/server/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:360:141)
    at Object.onReceiveStatus (/app/server/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:323:181)
    at /app/server/node_modules/@grpc/grpc-js/build/src/resolving-call.js:129:78
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
for call at
    at ServiceClientImpl.makeUnaryRequest (/app/server/node_modules/@grpc/grpc-js/build/src/client.js:161:32)
    at ServiceClientImpl.<anonymous> (/app/server/node_modules/@grpc/grpc-js/build/src/make-client.js:105:19)
    at /app/server/node_modules/@zilliz/milvus2-sdk-node/dist/milvus/utils/Function.js:32:31
    at new Promise (<anonymous>)
    at /app/server/node_modules/@zilliz/milvus2-sdk-node/dist/milvus/utils/Function.js:29:16
    at Generator.next (<anonymous>)
    at fulfilled (/app/server/node_modules/@zilliz/milvus2-sdk-node/dist/milvus/utils/Function.js:5:58)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 13,
  details: 'Received RST_STREAM with code 2 triggered by internal client error: Protocol error',
  metadata: Metadata { internalRepr: Map(0) {}, options: {} }
}

Node.js v18.20.4

Thank u very much guys!
Are there known steps to reproduce?
Upload a file, nice
Then search the file and move to the space, OK
but when i try "Save and Embed" the docker image crash

New updated (Not solve) but I've tried change from Ollama to Docker Ollama, why? Because I also have other problems realted with ollama
And Now have:

docker.io/ollama/ollama:latest
docker.io/milvusdb/milvus:v2.4.9-gpu
And the docker.io/mintplexlabs/anythingllm:latest

But it not work while try use milvuys, when use a docker chroma instead of milvus, then it works very well.

While the error for the milvus connection is not very helpful, we can at least handled the exception to prevent a full crash of AnythingLLM, since that is an annoyance

While the error for the milvus connection is not very helpful, we can at least handled the exception to prevent a full crash of AnythingLLM, since that is an annoyance

How can i help you to depurate it? what logs or what do you need?

Oh there are no more logs to get, i was just saying how the logs from their library does not help us out much since it has to do with GRPC connection termination. The stack you provided should be enough to help us get started to replicate this - assuming your connection information is accurate and that is not what is causing the issue

Oh there are no more logs to get, i was just saying how the logs from their library does not help us out much since it has to do with GRPC connection termination. The stack you provided should be enough to help us get started to replicate this - assuming your connection information is accurate and that is not what is causing the issue

Ahh I understand, thank u very much, hey bro I have a technical question not directly related, but the reason I was trying another vector database, is because I thought it would help me solve my problem.
you see, I was trying to embed a document with words as if they were dictionaries.
example: Cat. It has four legs and says meow
Dog Its a crazy zombie... etc etc
Jagar Its not humanoid
I want the LLM, when the user asks for cat, to tell him what it is like.
The problem is that I do not know if it is the embedding model or the vector database, but when I type the word, the search that is done does not find anything that has the keywords, when I search for dog, nothing appears. In many occasions it returns me a context or several useless contexts where the word is not there.
Is it that I misunderstand how all this works? or should I change the database or the embedding?
So far I have changed the size of the contexts, the size in which I part the vectors, tried some different embedding models, and changed to chroma, but when I ask the LLM using only one word as it should be, then I get results that are NOT related, they are useless, so the LLM will never know the answer.

Email me this question at<EMAIL_ADDRESS>- I dont want to pollute this GH issue with something off topic!

Thank u very much bro !

Email me this question at<EMAIL_ADDRESS>- I dont want to pollute this GH issue with something off topic!


Syntax Errors

Make exceptions for all the syntax errors for the current available keys.

Still to manage a combination such as 'x' '+' that it is not a syntax error.

The manage the aforementioned combination, we should change a little bit the philosopy. Add the option to have signs when the input is being splitted.

Solved by adding a specific case to the "cleanParts" method.


ISY based Fan entities undiscovered as of 0.37.0.  Lights on the same module are still there.

Home Assistant release (hass --version): 37.1
Python release (python3 --version): 3.4.2 (pi), 3.5.2 (OS X) seems unrelated
Component/platform: ISY Fan control  / Climate
Description of problem:  Just noticed that all entities of fan.whatever are now undiscovered on startup of home assistant.  The lights off the same module (2475F FanLinc V.45) are all present and working.  I set up a new VM and installed v0.37.0 - same behavior, no fans.  And then v0.36.1 - the fans are back.
It took me awhile to even notice since most of my day-to-day control is through Alexa, which ultimately controls ISY scenes and so was unaffected by the change, or by wall switches.  The HA gui didn't work though and that's because there was no entity for the input-selects to feed into.
I noticed some other issues posted around MQTT and Nest fans, but I couldn't see them as related.
Expected:
A connection ISY944 with FanLincs should discover fan.something entities along with the corresponding light.something entities for any given FanLinc.
I checked through the 37.0 and 37.1 release notes and didn't see anything directly related. Though there are these mentions of Insteon:

Light - Insteon local: Improve Insteon configuration

and of climate related items:

Fan - MQTT: Dont set a speed when fan turns on (@robbiet480)
Climate - eq3btsmart: Expose away attribute
Climate - Ecobee: Made target temperature sensitive to auto mode
Climate - Generic thermostat: Update
Climate: Hold mode
Climate - EQ3 BT smart: Add reporting for availability

I haven't had a chance to run them all down.  It'd certainly be handy if the Release Notes line items were links back to the Pull Requests (or I don't know what I'm doing... quite likely) because I can't easily go find the code changes for each one.
I'll try to search for a breaking change in 0.37.0 when I get a chance.

I forgot to mention, there was one ISY update in 0.37.0:

Light - ISY994: Not overwrite state_attributes (@rmkraus)emul-hue

None of the updates I listed seem very promising, but that's where I'll start looking.

The problem seems to be in here #5457 somewhere.  That particular one wasn't listed in the release notes, though it was in the changes.
I started with 36.1 and started adding on days worth of commits until it broke.  That's the one that did it.  Then beyond that, I skipped it and keep adding in others beyond it and the ISY fan's stayed working/discovered.  I did have to leave out a followup #5514 since it seemed to build on #5457.
I haven't actually looked through the code yet to see what the issue might be.

...  I just updated my first two posts since I just realized I was tagging people unrelated to this thread by referencing their contributions in the Release Notes.  I took out their tags.
I left in @robbiet480 because I believe his contributions for MQTT and cleaning up the fan code are what broke the ISY fan discovery.
I might need help identifying it.  I don't really know how the discovery process through PyISY works yet.

There doesn't seem to be anything in my ISY changes that would have caused any problems with discovery... The person to ask would be @Teagan42 as she has done the most work recently with ISY. Maybe it was a firmware update on the ISY side that broke discovery? The PyISY repo has issues mentioning that...

Thanks for jumping in.  I can say right now that I can run 36.1 and it works.  And I can run 37, minus those two commits referenced above and it works.  (I haven't actually made it ALL the way to 37... I've made it to 1/24's worth of commits though).  So, the firmware of the ISY is a constant.

I had a chance to follow up on the lead you passed about PyISY issues.  Those are all on the experimental 5.x release.  I'm on the stable, production release 4.5.4.
Firmware: Insteon_UD994_OADR v.4.5.4 ( 2016-09-30-16:09:53 )

Narrowing it down a bit but not figuring it out any better...
Looking solely at the file:
home-assistant/homeassistant/components/fan/isy994.py
Swapping that one file old for new makes the difference between working and not.
  This installation of HA I'm using is completely stripped down to no configuration other than pointing to my ISY.  I've tried with both 36.1 only adding in your new components/fan/isy994.py and with my ~37.0 cherrypicked around your pull request, also adding in that file.  It really makes no sense to me in that I can see you're not doing anything other than substituting SPEED_MEDIUM for SPEED_MED, both of which exist in both versions I have.
That the SETTING UP FAN start never happens makes it seem as though the fans suddenly aren't recognized as fans.
Just curious... do you have an ISY with FanLincs and do they work with the change?  Does anyone?  Because if it's just me...

I've also confirmed from the other side, I can take release 0.37.1 make a new branch and revert commit 074f9315d7f0d20837ed19d1846a295120ed63c9 (#5457) and it all works again.  There was a conflict with homeassistant/components/fan/mqtt.py from add-on changes.  I resolved that a little cavalierly for my testing, but I don't have any mqtt fans.
So, again, I'd ask:  do you have an ISY with FanLincs and do they work with the change (37, 37.1)? Does anyone? Because if it's just me, I'll take it on as just my problem.
Also, I didn't see anything in the Breaking Changes section of the release notes about removal of fan speed "med" for a new preference for "medium".  Anyone with automations that used "med" would be out of luck.  Mine are all templated off of state changes and use the new state, so they'd survive.  But it's definitely a breaking change.  Sure, there may have been both SPEED_MED and SPEED_MEDIUM, but you did just kill a global constant without making notice of a breaking change.
I don't think the change to medium is the problem here.  Is it possible the change out in the global init.py, minor as it might be and something that all platforms,  is affecting the ISYs?  Because I'd agree the changes made directly to the isy944.py is so minor it's unlikely to be the problem.  It's just an unannounced breaking changed based on a personal style choice.
Regardless, what do you think about rolling back the fan improvements commit until you can figure out how it's breaking the other platforms?

@dethpickle It's impossible for me to test this, as I don't have (and don't plan to get) any ISY devices. I agree that removing SPEED_MED was a breaking change and it should have been noted, however I had never heard of anyone using ISY fans other than @Teagan42 and had (falsely, obviously) assumed that removing it would only affect a very small number of users, if only one. She is also the only one I know who has an ISY fan (other than you) and actually wrote the ISY fan platform to begin with.
Which change to __init__.py are you talking about? The removal of SPEED_MED? If you re-add SPEED_MED everywhere that I had removed it does it fix the problem?
I am opposed to backing out the entire commit just to fix one platform, at least without first having you report back results of re-adding SPEED_MED.

There are no changes that I can see from 074f9315d7f0d20837ed19d1846a295120ed63c9 that would cause ISY fans not to be setup. It's for ISY only importing different constants.
Are you sure there are no errors logged at all ? Sometimes the errors get logged later.

Hey guys, sorry didn't see this chain - can you catch me up? I can spin up
a new HASS instance and see what's going on. I have 4 fans and my 36.0 HASS
instance is doing just fine.
On Wed, Feb 8, 2017 at 5:15 PM, Robbie Trencheny<EMAIL_ADDRESS>wrote:

@dethpickle https://github.com/dethpickle It's impossible for me to
test this, as I don't have (and don't plan to get) any ISY devices. I agree
that removing SPEED_MED was a breaking change and it should have been
noted, however I had never heard of anyone using ISY fans other than
@Teagan42 https://github.com/Teagan42 and had (falsely, obviously)
assumed that removing it would only affect a very small number of users, if
only one. She is also the only one I know who has an ISY fan (other than
you) and actually wrote the ISY fan platform to begin with.
Which change to init.py are you talking about? The removal of
SPEED_MED? If you re-add SPEED_MED everywhere that I had removed it does
it fix the problem?
I am opposed to backing out the entire commit just to fix one platform, at
least without first having you report back results of re-adding SPEED_MED.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/5779#issuecomment-278505919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZTPvmWUCUAMblq3bmZdUmcL_RfnBks5ralqxgaJpZM4L4gEC
.

Spinning up a VM and installing latest HASS give me a few minutes.
On Wed, Feb 8, 2017 at 6:31 PM, Paulus Schoutsen<EMAIL_ADDRESS>wrote:

There are no changes that I can see from 074f931
https://github.com/home-assistant/home-assistant/commit/074f9315d7f0d20837ed19d1846a295120ed63c9
that would cause ISY fans not to be setup. It's for ISY only importing
different constants.
Are you sure there are no errors logged at all ? Sometimes the errors get
logged later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/5779#issuecomment-278519319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZSJOhrKfhUkWW-myrAN7lez1R7_cks5ramyIgaJpZM4L4gEC
.

You are correct, 37.1 does not discover the fan devices. I'll see what I can dig through

I know why!

The issue is that the way it filters for devices is via the states - the ISY994 sends over a list of states or it's unit of measure for each device. For fans on the ISY platform it's off|low|med|high - which is WHY I had the constant med and not medium

The ISY platform tries to be smart and determine the device type based on the UOM - you guys changed the medium to medium instead of med which means it no longer matches all the states. I will make a PR real quick to fix

@robbiet480 FYI I am no longer a "contributor" because I lost access to my 2-FA and had to get Github to remove 2-FA to restore my account.

Restoring the constant SPEED_MED to __init__.py and restoring isy944.py to pre- #5457 works.
https://github.com/dethpickle/home-assistant/tree/fanfix-try3

...or I suppose I could have refreshed my screen.  So it wasn't the constant at all, but the value of the constant.
Thanks both of you!


Added the remainder of the assets.

linked to issue #2

@SPOCTONE, we don't want all the assets in Readme directory. It'll be better to structure it to their own directories.
Please remove files other than the ones used in main Readme.md

all right. on it.

done with the changes. please check.

Awesome. Thank you for making these changes


Adds Caching for Windows Feature Powershell provider

Description
Adds Caching for Windows Feature Powershell provider. Thanks @tas50 for doing all the heavy lifting

I'm sure there's more efficient ways to doing a few things since we're dealing with json. It was more a straight port of what was done on the dism side
This cut our subsequent runs from ~41 seconds to ~1.7 seconds for the ~30 features we install

Checks source definition key in registry when determining feature availability
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

So that failure confuses me. Is it not coercing the value correctly into an array or does it just write it out oddly?
windows_feature_powershell("Web-Asp-Net45, Web-Net-Ext45") do
  action [:install]
  default_guard_interpreter :default
  declared_type :windows_feature_powershell
  cookbook_name "test"
  recipe_name "feature"
  feature_name ["Web-Asp-Net45, Web-Net-Ext45"]
end

It would appear that it's not properly getting the difference between the two arrays at
https://github.com/chef-cookbooks/windows/pull/538/commits/21f32831b1244face392b426ceab837e922f54f5#diff-d10018fdcbb8eea540d70ca7f60e2031R132

I'm getting this error when I try to run the feature test suite locally:
         * windows_feature[Remove FTP for Appveyor] action remove
           * windows_feature_powershell[Remove FTP for Appveyor] action remove

             ================================================================================
             Error executing action `remove` on resource 'windows_feature_powershell[Remove FTP for Appveyor]'
             ================================================================================

             JSON::ParserError
             -----------------
             765: unexpected token at ''

Anything particular about your environment in which it's failing? That's working fine for me locally and in Appveyor.

Been trying to get the array input to work as well, not getting very far. Looks like it affects dism too
windows_feature_dism ['Web-Asp-Net45', 'Web-Net-Ext45']

@tas50 Any thoughts on the failure to install via an array?

Let me pull it local and see what I can figure out

Added a test to install an array of features with dism.
From what I can tell, it's failing on this line when an array is supplied
https://github.com/chef-cookbooks/windows/blob/master/resources/feature_dism.rb#L124

I have this all checked out locally and I'm working on a few things. I'll have something for you tomorrow

Here's the branch I'm working on https://github.com/chef-cookbooks/windows/commits/jakauppila-powershell_caching

Thanks for noticing the array functionality was broken and for putting this PR together. I'll get a PR open to add it to chef-client so we can ship this in Chef 14. It should seriously speed up windows runs.

Thanks for digging into it!

I put the tests into master as well. Thanks for those


Initiate imperative mood

I'm getting a D401 should be imperative for:
Initiate an a/b test.
Unless I'm confusing my grammar, that is already in the imperative mood. 'Initiates' would be incorrect.

What happens if you remove the /

Nothing, error still happens with this docstring:
"""Initiate test."""

The exact output is:
web/utils/ab_test.py:55:1: D401 First line should be in imperative mood

Just making sure. I remember a / causing other problems in the past and wanted to verify that it hadn't cropped back up.

I've added "initiate" to the imperative word whitelist. Should work fine now.
Thanks for reporting this!

This is causing an issue (and failing tests) with:
D401: First line should be in imperative mood ('Initiate', not 'Initialize')

I looked into this and it's because the stemmer stems them both to initi.
The stemmer actually succeeds in stemming the British English initialise:
>>> snowballstemmer.stemmer('english').stemWord('initialise')
'initialis'

So it looks like the stemmer is unaware of Americanisms. I am surprised. Perhaps we can teach it '-ize' endings?

Ah, you have a feature branch for fixing this.


Keep track of branches at pixel level

We can use a sparse list-of-lists (jagged array) to do this. This will enable colormapping of "true" skeleton branches (instead of the Euclidean map) in overlays.

This was fixed by #59


WIP: Improve performance and PathTooLong handling for native directory/path methods

So far builds long path projects on latest Windows after opting in via group policy and inserting
<AppContextSwitchOverrides value="Switch.System.IO.UseLegacyPathHandling=false;Switch.System.IO.BlockLongPaths=false" />

in MSBuild.exe.config after building. Need to test other platforms.

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

Be sure to mention #53 in your commit message to get the association.

@rainersigwald tried closing and reopening... Should we have seen the WIP bot?

@AndyGerlicher I expected that too, but it's appearing on new PRs so 🤷♂

Closing since this PR was split between #3442 , #3443, #3454, and #3507 ages ago


Update page to also include how to do this in Intue

This article reference how to set this via Intune.
https://msendpointmgr.com/2023/08/11/the-curious-case-of-the-missing-onedrive-sync-app-health-reports/
[Enter feedback here]

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

ID: 9f65565e-e318-a6d3-2216-b81243b753f3
Version Independent ID: 905fabb5-f3cb-0dd4-af58-f785a45002a5
Content: OneDrive sync reports in the Apps Admin Center - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/sync-health.md
Service: one-drive
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thanks for your feedback


feature request: json path support

JSON path support like gjson will greatly help. What do you think?

You can add gjson as an external helper.

Can you give me an example of the expression? I still think it is possible to do it without modifying Expr and implement your feature using the API provided by Expr.

You are right, we can write gjson expression using filters and maps but there are capabilities that are offered by jsonata that can't be done on Expr.
But the following things are easy to do with path context variables.
{
  "Account": {
    "Account Name": "Firefly",
    "Order": [
      {
        "OrderID": "order103",
        "Product": [
          {
            "Product Name": "Bowler Hat",
            "ProductID": 858383,
            "Price": 34.45,
            "Quantity": 2
          },
          {
            "Product Name": "Trilby hat",
            "ProductID": 858236,
            "Price": 21.67,
            "Quantity": 1
          }
        ]
      },
      {
        "OrderID": "order104",
        "Product": [
          {
            "Product Name": "Bowler Hat",
            "ProductID": 858383,
            "Price": 34.45,
            "Quantity": 4
          },
          {
            "ProductID": 345664,
            "Product Name": "Cloak",
            "Price": 107.99,
            "Quantity": 1
          }
        ]
      }
    ]
  }
}

Let's say we want to generate a summary of products using order, ProductID, quantity and price then n json template engine, we can do this.

.Account.Order@order.Product@product.({
    orderId: order.OrderID,
    productId: product.ProductID,
    quantity: product.Quantity,
    price: product.Price
})

Explanation: @order refers to an order in that context almost like an iteration variable which gets assigned new values as we iterate.
Output:
[
  {
    "orderId": "order103",
    "productId": 858383,
    "quantity": 2,
    "price": 34.45
  },
  {
    "orderId": "order103",
    "productId": 858236,
    "quantity": 1,
    "price": 21.67
  },
  {
    "orderId": "order104",
    "productId": 345664,
    "quantity": 1,
    "price": 107.99
  }
]

You can try it here

I see.  In what form data is presented?

Do you think it will be useful? Yes, it will help us to adapt expr more at our company. I have already added a plugin type based on expr. Expr already has a pipe feature in expr I think it works in stages, for example, if one expression in pipe gets executed fully then it will be sent to the next expression, if we can make it like a stream, one data element goes all the way and then next element then we can do more interesting with it. Input data can be struct, map, or array of structs or maps.

What about defining a custom helper function?
data | jsonpath('some.json.path')

we can do that but it is no different from using gjson right? if the path can be compiled and stored then it will be faster.

It can be compiled. Check out expr.ConstExpr.

That works with static data but if the data is dynamic then it won't work right?

I don't understand what do you mean by dynamic data? If your env is map[string]any you can still use jsonpath.
Also, what do you mean about "path to be compile"?
Please explain more. With some examples of golang and expr code of what do you would like to do.

To extract the data using the path, first we need to parse the path and, then according to the path, we need to extract data, if we parse the path every time then it will not be efficient. For example: in my JsonTemplate library, I generate a javascript code for the path which can generically handle different types of data. In Expr, you are generating ByteCode and then using it you are running on given data so my question is, how can we generate bytecode corresponding to the path.

I don’t understand what you’re trying to do. Please, provide example code.

Fill free to reopen the issue if this is an issue.


[FlatList] numColumns glitch when rerendering

Description
When you set the property numColumns in the new FlatList and then rerender a column when eg. pressing that column, the entire row rerenders and you can see it rerendering, it blinks.
Reproduction
Set numColumns on FlatList and then cause a rerender.
Solution
Looking at the implementation it seems that each row is only one instance of CellRenderer in VirtualizedList. And the column-splitting is taken care of by the parent component FlatList.
My best guess is that this is somehow what's causing the problem. It seems that it might be fixed if each column in the row was a instance of CellRenderer.
Additional Information

React Native version: master
Platform: iOS
Operating System: macOS

+1 Also noticing some flickering/stuttering with FlatList numColumns={2} in various situations (like when I just add a SeparatorComponent.

Hmm, what kind of re-rendering are you doing? I don't know why it would blink unless you're doing something like rendering a placeholder briefly or something like that? You can see in the <FlatList> - MultiColumn example in UIExplorer that pressing a row will re-render it without causing any sort of blinking as far as I can tell. Are you seeing something else?

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

I'm also experiencing this issue :/ @olofd did you manage to fix it?


Enhanced Editor - Modify existing geometries

implement the same logic as the find edit filter app to modify features location and vertices.

Use a toggle button for now to allow editing geometry of the current feature being edited.  Window popup will be disable when this feature is turned on.

Verified


Port to Java

Hello, I would love to have a try and replicate the bluetooth/wifi communication to android/java/kotlin to make the AAGateWay work as if it was native, but I don't find how it is done, no architecture, no point to start... how is the protocol? Is it wifi or wifi direct? Bluetooth socket in, response with the ip, the phone connects to the ip?
If you could provide some info or doc or point to start digging, it would be awesome.
Thank you very much!

There are already some android/java/kotlin implementation. You can have a look at nisargjhaveri/AAWirelessGateway for a Kotlin implementation. You can also have a look at borconi/AAGateWay for a more basic Java implementation.
Unfortunately, I don't have any documentation, you can try to go through AAWirelessProfile and AAWirelessLauncher in bluetoothProfiles.cpp to have an idea of the bluetooth communication to initiate the connection. Kotlin implementation for the same can be found at AABluetoothProfileHandler.kt in nisargjhaveri/AAWirelessGateway.
Hope this helps! :)

Hahaha, no need, you are awesome! My point was to do exactly that app... but you already did it.
OK, for now Im testing this project with a zero2w and a pi3a (which also supports OTG, I'm figuring out the dtb situation)... my pi0w are all broken....
Then I will play with a FireStick that I have with LineageOS18 to convert it into a dongle... maybe auto enabling hotspot on boot, pairing via adb shell... etc... would it be easy to add command line support on that app to configure hotspot details via adb shell?

Then I will play with a FireStick that I have with LineageOS18 to convert it into a dongle... maybe auto enabling hotspot on boot, pairing via adb shell... etc... would it be easy to add command line support on that app to configure hotspot details via adb shell?

If you can get the app running, it does take care of enabling hostspot and connecting via bluetooth. Are you talking about the one-time configuration since it might not be easy to get the app on screen with the FireStick?

Yes, I mean, I got root access via adb shell so I can do anything from there at any time, but everything has to be headless...

It works as follows: (could be not perfectly accurate, as I am writing this on phone)
When you connect to Bluetooth, RPi send via RFCOMM info about it's Android Auto (protobuf serialization), with wifi name, password, IP address of server and port. Phone follow this and connects to specified wifi, and connects to TCP server specified before. Afterwards, dongle acts as Android phone in MTP mode, Headunit checks if it supports AOA (android accessory), if yes, then it will tell you to switch to it, you do as follows (AOA have different usb descriptor), Headunit detect it and sends initial handshake packet. You then forward this packet to phone through TCP server, and the respond from phone back to Headunit, and afterwards it just acts as gateway of packets.
If you have any questions, let me know.

Thank you for the explanation! I was going crazy, trying to understand how the phone can be connected to a hotspot AND not be sending traffic through it... It seems Android Auto can handle a wifi connection without specifying a route.
I have been playing with AAGateWay project, using hotspot in slave, in master... and I was not fully understanding how the process works... I guess that the hotspot created here does not provide any DHCP, therefore no GW, therefore no route... and this "dhcp" is handle by the RFCOMM connection...
Thank you a lot!

Let me put it in other words... Why if I connect to the hotspot manually, the phone (Pixel 7) wants to route all traffic through hotspot (with no internet)... but if Android Auto connects, it doesn't?

This is just my speculation, but my previous experience with using WiFi IoT API -> Request Network API, when you get information about the WiFi through RFCOMM, Android treats the WiFi as it is without connectivity, so not only that Android keeps connection until app which uses this WiFi runs, but also tells the network manager to use 4G or any other connectivity source for internet.

Thank you for the explanation, that makes much more sense! I guess we can close this issue :)


[WIP] Feature/source from markdown file

resolves #10
This PR aims to add an extra command to handle posting directly from a local  markdown file.
This is achieved by :

Extracting the core functionalities from the run.js file. Especially

Removing some of the globals
Validating the platforms option
Publishing to the platforms
Showing progress

Using the fs module to be able to read from a local file
Creating a new run_local command exploiting the above changes to allow posting a local markdown file.

This is the work in progress as need to run more tests and do some more work to ensure everything works as expected. The loading.message seems  to have stopped working, as I do no know this library I need to research it a bit more !
EDIT : Also I'm using prettier so the first two commits are some formatting fixes. I tried to applied some settings that seems to match the style of the current CodeBase, but I can change that as needed of course !

Thank you @jbrocher for your work on this. Unfortunately, the reason I haven't gotten around to merging this is that it does a lot of changes in general to the CLI tool, and I haven't had the time to completely test it and see if everything works properly.
I like the idea of reading from a source file instead of a URL but we can do it in a way that allows the user to use the same run command except it can take a file instead of a URL. It's less changes to the code in my opinion.
I'll close this PR for now and open an issue with this new idea. If you're up to work on it then please comment on the new issue and I can assign you.

Thank you @jbrocher for your work on this. Unfortunately, the reason I haven't gotten around to merging this is that it does a lot of changes in general to the CLI tool, and I haven't had the time to completely test it and see if everything works properly.
I like the idea of reading from a source file instead of a URL but we can do it in a way that allows the user to use the same run command except it can take a file instead of a URL. It's less changes to the code in my opinion.
I'll close this PR for now and open an issue with this new idea. If you're up to work on it then please comment on the new issue and I can assign you.