Applying patch does not work.

I tried to commit my code locally and it detected violations in different files. It suggested me to run git apply /tmp/pre-commit-uncrustify-1420355533.patch this command to apply patch. this command works fine and few of files are changed but when I try to commit again it gives me same violations which I observed on first run.
Please suggest.

Hmmm, interesting. I presume you can not share your actual code to reproduce the issue?
Let's see whether it does what we would expect it to do:

You try to commit changes and it is the right files that you changed that are being checked? It shows up similar to

Running hook: pre-commit-uncrustify
Parsing: test.c as language C
...

Then, when you apply the patch with git apply, what does actually change? Does git print anything helpful? Try calling it with git apply --verbose to get some more detailed output and check whether it applies it to the correct files. Regardless, after applying the patch the changes should be applied but not yet staged for the next commit. Which means, a

git diff --cached

shows your original commit which violates the uncrustify rules and
git diff

shows the patched changes to make it comply with the rules.

You then have to stage the patched changes with git add -p for example and commit again and it should be fine.

Please let me know where it seems to break and we can have a closer look.

Ah, and could you let us know which OS you are using and versions of git and uncrustify you are running? Basically what git --version and uncrustify --version tell you. Thanks!

David,
Thank you so much for prompt response and writing detailed instructions.
I have following setup
git version 1.9.3 (Apple Git-50)
uncrustify 0.60
OS X 10.9.4
When I tried to commit it showed parsing different source files
Parsing: Project/Application/App Delegate/AppDelegate.h as language OC
--------------List of files goes on
Then one by one it displayed file names and code that was not according to rules specified
The following differences were found between the code to commit and the uncrustify rules:
--- "a/Vente/Common/ASValueTrackingSlider/ASValuePopUpView.m"	2015-01-05 19:57:07.000000000 +0500
+++ "b/Project/Common/ASValueTrackingSlider/ASValuePopUpView.m"	2015-01-05 20:01:17.000000000 +0500
Then at the end
You can apply these changes with:
git apply /tmp/pre-commit-uncrustify-1420470077.patch
(may need to be called from the root directory of your repository)
Aborting commit. Apply changes and commit again or skip checking with --no-verify (not recommended).
I ran git apply /tmp/pre-commit-uncrustify-1420470077.patch on my machine and it changed some files
Then i ran git add . command and tried to commit locally. It again displayed same violations.
Thanks in Advance.
Asif

You are most welcome, Asif. At this point I'm just guessing though as I have no idea yet what the problem is. Maybe you are hitting some corner case that no one else had yet. Some things coming to my mind:

How many files are you committing? How big is the patch file or how many lines does it have?
Try committing just a single file and see whether it works then. At least then we know a bit more about whether it is a problem in general or more specific.

David, I think you are right about big patch and impressed with your finding because I have too many files around 60  that I wanted to commit. Some of my own code and some of the third party libraries files, that is why patch became so big.
For the time being, i removed pre-commit and canonicalized-file.sh from hooks folder and committed my code to git. Now I have put back above two files back in hooks after doing commit.
I will proceed with new code and will try to commit with fewer files. If it reports violations, I will try to apply patch and will share my results on same thread.
Thanks,

David,
I tried with fewer files and it worked. You were right about big patch.
Thanks man.

Hi Asif,
I'm surprised that it already failed at about 60 files. I would have thought the script would be robust enough against that. I was thinking more along the lines of 10,000 files or so.
Would you be able to tell me how big the patch file was that failed? As in how many lines did it have? I may try and reproduce the issue locally and see what we can do about it. Because it would be good if the scripts just worked regardless or at least exited gracefully with a helpful error message.
Some general comments:

I would not uncrustify third party sources. For example if you would want to merge in a patch from upstream it could become a nightmare as the formatting would be different. Or trying to get a meaningful diff of your third party sources against a different version.
If you want to start using uncrustify for an existing project, it might make sense to uncrustify everything once, do one big cosmetic commit and then just make sure that all new changes are according to the rules. Without having tried it, find might be a good start. Something like find . -type f -name "*.m" -exec uncrustify -c uncrustify.conf --no-backup

Thanks for your feedback!

David ,
Following are the stats in that commit
140 files changed, 3409 insertions(+), 11040 deletions(-)
Let me know if you want any more details.
Thanks,

I will see when I get some time to have a look into it. If I need any more details I will be in touch. Thanks very much for the help so far, Asif!

You are most welcome. I am thankful to you for your prompt responses and
looking into issue that I was facing.
On Wednesday, January 14, 2015, David Martin<EMAIL_ADDRESS>wrote:

I will see when I get some time to have a look into it. If I need any more
details I will be in touch. Thanks very much for the help so far, Asif!
—
Reply to this email directly or view it on GitHub
https://github.com/githubbrowser/Pre-commit-hooks/issues/5#issuecomment-69875538
.

--
Sent from iPhone. Please excuse any typos


Add the option to format drives to the right click context menu on the sidebar and on the drives widget

In v.0.20, it supports formatting the drives. However, I can't format the drives. I mean "formatting" like deleting all files in certain drives. Do you mean that added support formatting the drives like that?
I want to format the drives in Files.

Why I can't format the drives (like deleting all files in certain drives) in v.0.20

You can find the format option by:

Navigating to a drive by clicking on the Files sidebar
Right-clicking on the empty space in the file list, this will open the context menu
Under "More" submenu you should find the "Format" option

What version is it?

Oh yes! But, then... I think the format button should go to the drive page, like the original file explorer.

What happens to this issue after you added it to the backlog?

@seohyun1108 contributors are free to pick any open Backlog issue and fix it. Being in the backlog means the issue is accepted and will be fixed but does not define a deadline/specific release.

I think this issue can be solved easily. Why it's not fixing till now?

I think this issue can be solved easily. Why it's not fixing till now?

@seohyun1108 there's so many higher priority things to do, but we'll get to this!

@seohyun1108 there's so many higher priority things to do, but we'll get to this!

First part of this issue is already implemented as now there are shell context menu items loaded when right clicking on a drive on the sidebar. The second part could be merged with #3114.


decouple take_until from observable

Decoupled take_until from observable,
please review.
Thank you,
Grigoriy

Thank you!


Add episode count on covers and Option to reduce grid and cover size and .....

CPod version: 1.27.1
Description:

Please add option to reduce cover size. Covers are too large for my taste.
Episode count should be displayed on Cover.
As asked earlier please add ability to open podcast details by just clicking the cover. There should be no need of Show Details button.
Add ability to resize panels. I mean podcast cover grid panel is small. One should be able to resize it.

@shad90 I agree with all of these suggestions. Particularly point 4, which I was going to post here myself until I found you posted this. Many of us have several podcast subscriptions and the panel showing all podcasts listed is too small. The ability to resize that or just allow expanding it would be great.


[WFLY-13647] Annotation processing error sun.reflect.annotation.TypeNotPresentExceptionProxy does not indicate issue

Issue: https://issues.redhat.com/browse/WFLY-13647
There is a bug with Class.getAnnotation() call in JDK < 11 [1]  which throws ArrayStoreException when the annotation refers to some classes not found in current class path. The ArrayStoreException confuses user why it failed because the root cause is missing.
This PR tries to catch the exception and give more readable feedback in this case.
[1] https://bugs.openjdk.java.net/browse/JDK-7183985

@chengfang or @tadamski or @fl4via please review.

@chengfang This needs a conflict resolved.

@gaol can you update your local repo and increment the log id in EjbLogger @Message(id = 520 to 521, since 520 is already taken?
The failure in Windows JDK11 seem unrelated intermittent failure. I just created WFLY-13682 (JBossWSStringReplaceTestCase.incorrectValueOfModifyWsdlAddressOpeningElement failed intermittently on Windows) to track it separately.

@chengfang Thanks for the instructions, I just updated the PR to fix the conflicts. :)

Linux JDK8 and Linux JDK11 jobs failed on July 20, 2020, with 12 failures in  MixedDomainDeployment700TestCase. I think they are unrelated failures and can be ignored for this PR. I've just reopened the following JIRA to track it separately:
WFLY-12649
Deal with mixed-domain failures in the OpenJ9 jobs


TODOs: v0.2

AWS

[ ] Allowing Prod to be read, but not written on dev/staging environments

Server

[ ] ORM/Model Mapping Research for Validation and Maintainability Purposes
[ ] Implement Authentication using Google's Outh id_token
[ ] Email Confirmation Send through AWS SES

Misc

How to implement authentication:
Frontend sets id_token as a cookie for the backend to take either as a parameter or some kind of input
https://developers.google.com/identity/sign-in/web/backend-auth#calling-the-tokeninfo-endpoint
https://developers.google.com/identity/sign-in/web/backend-auth#using-a-google-api-client-library
Either use Google's API or use or use https://oauth2.googleapis.com/tokeninfo?id_token=XYZ123 and validate if token is valid or not to allow user's to make admin API endpoints -> potentially could be set as a decorate for abstracting authentication code on the backend.

https://github.com/whyphi/zap/issues/3#issuecomment-1732186465


Windows/MSVC: Cannot open file 'contrib\yaml-cpp-pm\RelWithDebInfo\libyaml-cpp-pm.lib'

Steps to reproduce:
Building libpointmatcher with MSVC following the current instructions, excluding the instructions to use an external yaml-cpp and external gtest
What should happen:
Build succeeds (C++ compatibility issues anticipated)
What happens instead:
I get the following error:
   "C:\Users\username\Documents\registration_builds\libpointmatcher-msbuil
   d\libpointmatcher.sln" (default target) (1) ->
   "C:\Users\username\Documents\registration_builds\libpointmatcher-msbuil
   d\pointmatcher.vcxproj.metaproj" (default target) (10) ->
   "C:\Users\username\Documents\registration_builds\libpointmatcher-msbuil
   d\pointmatcher.vcxproj" (default target) (16) ->
   (Link target) ->
     LINK : fatal error LNK1104: cannot open file 'contrib\yaml-cpp-pm\RelW
   ithDebInfo\libyaml-cpp-pm.lib' [C:\Users\username\Documents\registratio
   n_builds\libpointmatcher-msbuild\pointmatcher.vcxproj]

133065 Warning(s)
1 Error(s)

I have a file <build>\contrib\yaml-cpp-pm\RelWithDebInfo\libyaml-cpp-pmmd.lib not libyaml-cpp-pm.lib
Workaround:
Make a copy of libyaml-cpp-pmmd.lib with the correct name, and build again.

To contributed a yaml copy is only a temporary hack. This is why we haven't cleaned up (or even tested ?) the yaml-cpp-pm/CMakeLists.txt file for windows.
The line https://github.com/ethz-asl/libpointmatcher/blob/master/contrib/yaml-cpp-pm/CMakeLists.txt#L116 is apparently introducing this suffix. The best way is probably to get rid of this fine-grained support of different co-existing output files and rather always use the same (without any extra suffix). What happens if you just remove the two if(MSVC) blocks? Does it work for you?

That worked! I've included it in my clean branch.

Great! And thanks for testing!

Should I remove those two blocks on the master?

I can confirm that it worked for me. Apologies for not submitting my branch as a pull request yet, but feel free to take anything useful from it :)


Support SecretStore in container resource (Native Radius Way) for k8s Secrets.

Overview of feature request
Right now, Radius do not support secretstore for referring existing k8s secret as a volume in container.  So we are have to use podSpec.
Maybe container resource could support this or a documentation could be present in secretstore as a starting point ?
Here is an example:
resource container 'Applications.Core/containers@2023-10-01-preview' = {
  name: 'ctnr-podpatch'
  location: location
  properties: {
    application: app.id
    container: {
      image: magpieimage
      ports: {
        web: {
          containerPort: port
        }
      }
    }
    connections: {}
    runtimes: {
      kubernetes: {
        pod: { // This is free-form object, you can use any kubernetes podSpec properties.
          containers: [
            {
              name: 'ctnr-podpatch' // ensure that it matches the container name above
              volumeMounts: [
                {
                  name: 'se-charlie-server'
                  mountPath: '/credentials'
                  readOnly: true
                }
              ]
            }
          ]
          volumes: [
            {
              name: 'se-charlie-server'
              secret: {
                secretName: 'se-charlie-server'
              }
            }
          ]
        }
      }
    }
  }
}

FYI @youngbupark
Acceptance criteria
Options:
Additional context
See the topic on discord forum
Would you like to support us?

[ ] Yes, I would like to support you

AB#10953

@willtsai  This is related to https://github.com/radius-project/radius/issues/5520. We need to support more use-cases of secretstore for containers. e.g. volume mount, secret reference in fields in container resource or any radius resource.
cc/ @rynowak

Yup. This is badly needed.

this is completed as a part of https://github.com/radius-project/radius/pull/7744/


Super strange terminal behavior

I've been using the Code Runner extension to run small snippets of code for sometime. In addition I'm using the esm package to patch node, so I can work with ES6 modules. This has all worked smoothly for sometime.
Today however is another story. Running snippets/files was working fine in the AM and then it started acting weird. Now running a code snippet that ran fine before results in a cursor that just sits there or else jumps down a number of lines, without showing any console.log results in the terminal.
I'm running the latest VSCode under Windows 10 Pro version 1809.
I've tried dis/enabling Code Runner. This resulted in it working again but then it dies again. At this point I do not know what to do. Hopefully there is some solution to this complete frustration!

Still have this issue. I've cleaned out the project so only a few files that I know run are left. I have even removed prefix numbers on the files I use for sorting files. Have also dis/enabled Code Runner over and over. Sometimes Code Runner works but usually it does not. The terminal cursor just freezes or else jumps down a number of lines. The only recent new thing I'm using is pnpm.

Please update this issue after reading https://github.com/Microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions#writing-good-bug-reports-and-feature-requests. Thanks.

This is probably an issue with code runner or this bug in Windows: https://github.com/Microsoft/vscode/issues/57803


Report when a task has a missing dynamically-requested input

Currently, we don't have the necessary information to report which input is missing, but higher-level build systems can use other delegate callbacks to reconstruct that information. If no missing input is reported at all, we can easily end up in a situation where tasks are unexpectedly and silently cancelled, making debugging these sorts of issues difficult.
rdar://58130864

@swift-ci please smoke test


Invalid App Packages?

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Enhancement / Suggestion
[ ] Additional article idea

Expected or Desired Behavior
Upload a packaged .spapp file to the SharePoint App Catalog and get a valid app package.
Observed Behavior
As of recently, uploading previously valid .spapp packages built with Drop 5 to the app catalog results in an invalid app package.
Steps to Reproduce
Using drop 5, use Yoeman to generate the Hello World app. Build with gulp package-solution and upload the .spapp to the app catalog. The catalog indicates that the package is invalid.

Hi Jared - can you go to the app catalog list and add the column that shows what the error is?  The column is "App Package Error Message".  What does the error say?

The error is "No errors."

Here's a screenshot. Two apps uploaded today are not valid. The one at the base was a replaced app, the hello world at the top was new. Any apps installed a while back, using the same build process, are valid.

Can you let us know what your site collection URL is so we can verify that the feature should be enabled?

Tenant from Dev Kitchen.
a830edad9050849spdk3024.sharepoint.com/sites/sj/

Running into the same problem on our production O365 that is set for First Release and supposedly has SPFx since Nov. 11 according to our notifications.

The change is still rolling out to production tenants for First Release so that is expected that it is not working at this time... The reason your other tenant is no longer working is some changes were made to the flights included as part of the rollout to First Release.  I am re-enabling it for that farm and will respond once has been reverted.

Hi Jared, Please let me know if you are still experiencing the issue on your dev kitchen tenant.  The changes that were made to start the rollout of first release that affected your tenant have been fixed now.

I'm running into the same issue as Jared.  I'm working through the Hello World tutorial for SPFx, and when I upload the app to my App Catalog, it shows that it is not a valid app package, but provides no app package error message.  I'm on O365 set up for First Release.

Hi Gary,
It is still in the process of rolling out to First Release so it is not expected yet to be enabled for all tenants on First Release.

Thanks - any idea on when it will enabled?

My best guess is about 2-4 days.

I'm running into the same problem...We are signed up for "First release for selected users"...

Same here.  I could have sworn it worked in production yesterday... but today, no dice.  Guess there's still some wrinkles to iron out.

Trying to track down the issue here.  @clechner77 - did you have a developer tenancy that stopped working?  A dev-kitchen tenancy?  Just wondering what you had previously, and when it was working and when it stopped.  Some first release tenants have it, and other farms are still pending the update.
Thanks.

@mcmynn83 I can confirm that the fix you did for the dev kitchen tenant worked.
Thank you!

@mcmynn83 I'm still having some trouble. Now the app package installs fine in the catalog but when I add the app to a site it tells me the version of the app is already installed and to remove it from the site and the site recycle bin. I do that but still get the error.
Screenshots from the Dev Kitchen site

Also, the same app package now installs fine into the app store of our First Release tenant, but when trying to add the app to any of the First Release Sites I get a different error but still an error.
Screenshots from the First Release site

Hi Jared,
I'm sorry you are having problems.  Could you please click the technical details and send a screen shot back (or just give me the correlation ID you get when you click the technical details button)?
As for the 1st error you saw, when you said you deleted the app from the recycle bin, there are 2 stages to the recycle bin. First you would have to go and delete from the recycle bin, then you would have to go to site settings -> Site Collection Administration -> Recycle bin -> "Check the second-stage recycle bin" and remove your app from there (or shortcut the process by going to this URL: _layouts/5/AdminRecycleBin.aspx?View=2).  If you actually removed from both places I will need the correlation ID (ie click the technical details and include that correlation ID).
Thanks,
Graham

@patmill -- Sorry, got caught up.  My dev tenant is still functional.  I was referring to prod/first release that mysteriously went back in time.

@clechner77 - were you previously able to activate it on the first release tenant, or were you just accessing the hosted workbench pointing at your local gulp serve (ie, no package deployment)?

@patmill -- I was speaking with another guy on our team and he didn't see it working on our prod tenant and thinks I might have been been on the dev tenant instead.  Like I said, I could have sworn it was the prod tenant but now I'm second guessing myself.  I know.... not very helpful.

@mcmynn83 I went through and cleared all apps from the app store site including from both recycle bins. I then removed all apps from a test site on the same tenant including removal from both recycle bins. I then created a fresh "yo @microsoft/sharepoint" and built an spapp with "gulp bundle --ship" and "gulp package-solution --ship". A CDN container was created and assets were deployed to Azure. The installation on the app site went smoothly. I was also able to add the app to the test site. Everything seemed to work without a problem.... until
Modern pages
"There was an error fetching the web parts. Not all web parts may be available."
The hello world web part is not available in the list.

Classic Pages
All of the web parts that were completely removed from the site are still listed in the ribbon web part selector.

After adding the new Hello World web part to a classic page I get an error.
"The value for 'webpart context configureStart' must not be undefined"

So from what I gather, "There were errors when validating the App Package" when adding it to a site is due to not everything being pushed to my tenant yet.

@mcmynn83 Is "there were errors when validating the app package" still a waiting game for the completion of the rollout or are there potentially other issues?  Same package works fine in dev tenant but not on First Release when adding to a site.

@jaredbuttars - the apps showing up in classic even after deleting has been fixed but not rolled out.  It probably will not make it to customers environments until early December.  The issue with webparts not showing up in modern pages has been fixed and should already not be reproing for you.  If you still see that issue please let me know.  The issue with configureStart must not be undefined is new to me.  By any chance could you send me your app package?  My alias at Microsoft is GrahaMc.
@clechner77 - If you send me your tenant info I can tell you whether it is a waiting game or another issue.  Without any further info I can only guess.  GrahaMc is my alias at Microsoft if you don't want to post your tenant info on this forum.

@mcmynn83 -- no issues, our tenant is mlmins

@mcmynn83 -- The changes that were made to allow web parts to show up on modern pages seems to have also fixed the context configureStart error I was getting on classic pages. All seems well in our dev tenant. Thank you!
I continue to have an issue on our first release tenant. The app package uploads to the app store and deploys fine. The trouble comes during install to a site. I click "Trust It" then I get an error page with the following message:
Sorry, something went wrong
There were errors when validating the App Package.
TECHNICAL DETAILS

Troubleshoot issues with Microsoft SharePoint Foundation.
Correlation ID: 72ecb99d-2036-3000-d4a9-97dc149b19e4

Date and Time: 11/22/2016 7:14:32 AM

This error is on the site at:
https://office365lds.sharepoint.com/sites/jbuttars/

Hi Clint,
So the good news is it is not a waiting game.  The patches have rolled out to the farm you are on.  However, that leaves the question of why is your package not validating.  If you go into the view on the app catalog you can modify the fields in the view to include App Package Error Message.  That should hopefully  give you more detail into what is going wrong.  Let me know if that helps at all.
Thanks,
Graham

@jaredbuttars - Thank you for reporting this and giving me the information I needed to investigate the issue.  It looks like we may be having further issues with the First Release tenants roll out (a different problem this time).  I am looking into it now and will keep this thread updated.

I need to make  a change to how First Release tenants are checked so that again will take a few days to roll out to all of the production farms.  Until then I have to disable the feature, so everybody will start seeing invalid app package on First Release tenants when uploading a client side solution (dev tenants will continue to work).  The current ETA for it being enabled (and actually working this time) for FirstRelease tenants is 4-5 days (basically as farms get patched it will start working).
Sorry about the delays and thank you for reporting the problem.

Hello, I am experiencing the same (?) issue at a customer right now.
What exactly can I do to check everything is setup correctly at tenant-side?
Uploading a package to my DEV tenant works, but not on the customer's.

Hi mcmynn83,
I am facing the same issue while uploading the app package for our SPFx client side webpart to our customer tenant. The app package shows as valid when we upload to dev tenants but not when we upload to our customer tenant which has first release set to selected users only. I got my account added to the list of these selected users as well and tried uploading the app , but the same issue still occurs. The 'App Package Errors' column is also empty. Please let us know if there us any ETA for this fix as we are running against timelines for delivering this functionality.
Thanks,
Bhavya

Select users in first release is not sufficient. It needs to be tenant wide first release.

From: Bhavya-Chhabra<EMAIL_ADDRESS>Sent: Thursday, December 8, 2016 12:26:18 AM
To: SharePoint/sp-dev-docs
Cc: Pat Miller (SHAREPOINT); Mention
Subject: Re: [SharePoint/sp-dev-docs] Invalid App Packages? (#276)
Hi mcmynn83,
I am facing the same issue while uploading the app package for our SPFx client side webpart to our customer tenant. The app package shows as valid when we upload to dev tenants but not when we upload to our customer tenant which has first release set to selected users only. I got my account added to the list of these selected users as well and tried uploading the app , but the same issue still occurs. The 'App Package Errors' column is also empty. Please let us know if there us any ETA for this fix as we are running against timelines for delivering this functionality.
Thanks,
Bhavya

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-dev-docs%2Fissues%2F276%23issuecomment-265681869&data=02|01|Pat.Miller%40microsoft.com|7fa5627b43b54d85ff4508d41f43e28a|72f988bf86f141af91ab2d7cd011db47|1|0|636167823805953338&sdata=0Visc8FuSArRZqLgt4keroisDPO6n9QV%2BN4KRd37bNk%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADl-SdOZ4EOGH_t9eCKFOxvvdED7jMGSks5rF78qgaJpZM4Kxp10&data=02|01|Pat.Miller%40microsoft.com|7fa5627b43b54d85ff4508d41f43e28a|72f988bf86f141af91ab2d7cd011db47|1|0|636167823805953338&sdata=ynzGDohbJGqivgY2ass3E2HSM7j%2FVON4flWVGtS%2FEr0%3D&reserved=0.

I am facing the same issue. What needs to be done in existing tenant for the SharePoint Framework .spapp file to get deployed in AppCatalog. it is showing Invalid in existing Tenant and working properly in new tenant? My existing tenant has First Release for selected users as enabled. Any help would be really appreciated.

@MaheshakaMudli - as per stated in the last response to this thread:
Select users in first release is not sufficient. It needs to be tenant wide first release.
That means either you will have to wait for the feature to be release everywhere or enable tenant wide first release.  Please let me know if you determine that your new tenant is in the user only First Release but it actually works for them.  That would be unexpected.
Thanks,
Graham

Hi @mcmynn83,
Thanks for your response earlier. As per the latest update from Microsoft , we got to know that the General availability release of SharePoint Framework is available now which has fixed this problem. Microsoft has started rolling it out to all tenants now. I just wanted to ask if there is any date/timeline by when this GA release will be deployed to all tenants worldwide? We checked our tenant and the SharePoint framework app packages are still showing as invalid which means that the GA release is not applied on our tenant till now.
Thanks,
Bhavya

I am still experiencing the same issue on my tenant. Tried to upload it on another tenant and it works well. I contacted MS support and the support guy said they don't have any solution guide for SPFX yet.

We are facing the same issue in our development tenant. But everything works fine in test tenant.

We should be at 50% of tenants this week, and 100% next week.  Appreciate your patience.

Sure, Thank you

@patmill
Is it done on all the tenants?

Hello,
When I deploy SPFx package to SharePoint 2016 on premise, I get an error of faulty package with a message: There were errors when validating the App manifest.: Xml Validation Exception: 'The 'SkipFeatureDeployment' attribute is not declared.' on line '1', position '276'.
In the debug folder ( files generated in the package put separately ) I find AppliationManifest.XML with following content:

helloworld-client-side-solution

As you can see, and I checked in config/package-solution.json and the property is initialized as shown in ApplicationManifest.xml.
What is the solution to this problem? Thank you in advance,
Antonio.

@Antonio-Bacar - Could you please open a new thread with this request.  That being said, did you choose an on-prem package from Yeoman?  Skip Feature Deployment is not supported on the on prem release at this time.  The on prem yeoman generator should not be including that attribute.


License for Project Jupyter software

In the Jupyter governance, it indicates that when a project is adopted by Jupyter, it should use the license specified in the governance: https://raw.githubusercontent.com/jupyter/governance/master/projectlicense.md, which is BSD.
According to this, the license in, for example, https://github.com/jupyter-widgets/ipyleaflet/blob/master/LICENSE probably should have been changed from MIT to BSD when it was adopted as a Jupyter project.
However, according to the main governance document, "The Software developed by The Project is released under the BSD (or similar) open source license". I think MIT would qualify as "similar" to BSD.
So which is it? If a project is incorporated into Jupyter, should it be converted to BSD for consistency with other Project Jupyter software, or is it fine to stay, say, MIT licensed? ("converting" I think more properly means that the LICENSE file changes to BSD, but it retains the MIT copyright notice for code before it was adopted into the project)
It would good for us to have clarification on the intent and our interpretation. I don't know of any other Project Jupyter projects that are released under licenses other than the standardized BSD project license.
CC @ellisonbg (original author of ipyleaflet), @SylvainCorlay and @martinrenou (main current maintainers) of ipyleaflet.
For the record, I'm okay with an official subproject being MIT licensed if it was licensed MIT before being adopted. I think new projects that start in Project Jupyter should carry the project license.

My thinking is that we should accept 1 or a couple explicit licenses that we'll allow, rather than using more vague language like 'or a similar license', in order to avoid the extra labor and ambiguity associated with deciding whether a license is "similar" or not.


[DrawablePainter] Fix rememberDrawablePainter for BitmapDrawable

This PR removes special case handling for BitmapDrawable. The previous handling was problematic in several ways:

bitmap in BitmapDrawable can be null. This was not handled.
BitmapDrawable handles density scaling internally. The density information was dropped in previous code, and the returned BitmapPainter would have wrong intrinsic size if specified density for drawable resource was different from device density, which is a common case in resource density fallback.
Mutation of BitmapDrawable after creating the Painter is ignored. But mutation of drawables with non-BitmapDrawable types are reflected in the returned DrawablePainter. This behavior difference seems not appropriate to me.

The ColorDrawable special case suffers from 3rd point but not 1st or 2nd point. If removal for ColorDrawable special case is wanted, I'd be glad to do that job too.

And this fixes https://github.com/google/accompanist/issues/1115 .

Hi @bentrengrove , sorry for the disturb if any. But it has been a month since this PR was created. Is there anything I can do to get it reviewed?

Yes, I have been chasing the person who originally added this code for you. Still trying!


better documentation for DNS config

Thank you @ike , it looks really better! :) Just that tiny change to keep dig and I'll merge it right away :)


How about adding Hot Module Replacement?

Its a preference thing really - I like refreshing the page to see changes.


Crit Chance lowers Total DPS with elemental overload

I'm using a character with Elemental Overload. Account: bane2571 Character: poloniusthesilent
With that passive tree I'm looking at adding king of the hill but it is reporting that I would lose DPS by doing so. I've also tried using the accuracy and crit chance nodes which, given the Lioneye's bow, should only affect crit chance and the DPS also decreases.
Given that it might be calculating a 100% up time at 15% crit (which I doubt is true), worst case adding more crit% should do nothing. Why is it decreasing the estimated DPS?
PS: damn this program is awesome, the fact that I can immediately tell something weird is going on in the calcs shows how much normally invisible info you're providing to players - kudos.

The program always assumes 100% uptime on Elemental Overload, as that's usually quite easy to achieve.
You have no crit multiplier, so the difference between your crit and non-crit damage is entirely determined by the enemy's resistances. And since you have Inevitable Judgement, your crit damage is calculated as if the enemy's resistance is 0, but your non-crits use the normal calculation (enemy resistance minus penetration).
Therefore, if your penetration would reduce the enemy's resistance to below 0, then your non-crits will deal more damage than your crits, and so gaining more crit chance really will lower your DPS. If on the other hand the enemy's resistance ends up positive (which in your case would be true against any boss), then more crit chance increases your DPS slightly.

Wow, the interaction on inevitable judgement - I had no idea the neg resistance=neg damage potential even existed. Good news is you have a enemy resistances option - after filling that in for merciless (45%), king of the hill now gives +dps instead of -dps.
Thanks!


PyJWT==2.2.0

new release today:

causing issues as referenced by:
https://github.com/jazzband/djangorestframework-simplejwt/issues/464#issuecomment-937422628
pinning PyJWT<2.2 or alternatively PyJWT==2.1.0 in personal projects is a workaround for now, but ideally this would be either pinned appropriately here:
https://github.com/jazzband/djangorestframework-simplejwt/blob/master/setup.py#L59
or handled in source code

configmed!

@Andrew-Chen-Wang  re:
https://github.com/jazzband/djangorestframework-simplejwt/issues/464#issuecomment-937445562
I can't do this in the short term but if someone doesn't pick it up by the weekend I will handle this

Confim, same with both 4.7.2 (we used so far) and 4.8.0. In requirements we've hardcoded PyJWT==2.1.0 and "miracle" happened.

Confirmed.
return jwt.decode(token, self.verifying_key, algorithms=[self.algorithm], verify=verify,
TypeError: decode() got an unexpected keyword argument 'verify'

Quick fix is to set PyJWT==2.1.0 in your requirements.

Fixed the issue with PyJWT==2.1.0 and djangorestframework-simplejwt==4.8.0 in requirements.txt
PyJWT==2.1.0
djangorestframework-simplejwt==4.8.0


Issue with k8s.io/docs/tasks/tools/install-kubectl/

This is a...

[X] Feature Request
[] Bug Report

Problem:
I'm using macOS 10.13.4 but I'm not clear as to why the documentation is requiring the installation of Homebrew to install the bash completion for kubectl.
Proposed Solution:
Please provide a solution which doesn't require the installation of Homebrew for kubectl bash completion.
Page to Update:
k8s.io/docs/tasks/tools/install-kubectl/
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.6", GitCommit:"6260bb08c46c31eea6cb538b34a9ceb3e406689c", GitTreeState:"clean", BuildDate:"2017-12-21T06:34:11Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"darwin/amd64"}
Unable to connect to the server: dial tcp <IP_ADDRESS>:8443: i/o timeout

/assign

@conradwt bash-completion is a separate utility that needs to be installed for kubectl command auto-completion to work in Bash. The alternative to Homebrew installation is given in that's README:
autoreconf -i
./configure
make
make check
make install

I feel that this is too cumbersome an installation method to be included in the kubectl setup doc and that it's reasonable to expect users to install via Homebrew given its popularity. I'll close this now but feel free to reach out if you feel this is unreasonable.

@lucperkins I agree with you 100%.  Anyway, I was able to get things working by doing the following:

installed bash-completion via MacPorts

installed kubectl using gcloud

added the following to the bottom of ${HOME}/.profile:
[ -x $(command -v kubectl) ] && source <(kubectl completion bash)

change Terminal.app -> Shells open with
Command (complete path):
/opt/local/bin/bash

Note:  This allows the kubectl bash-completion to work as expected.

Finally, I just noticed that there's also a kubectl port available for MacPorts.  Thus, one can do the following:
 ```
 sudo port install kubectl
 ```


Split format field into document_type and schema_name

The format field is being replaced by two fields, see RFC 41. document_type identifies the kind of the document for filtering and display purposes, and schema_name identifies which schema is used to validate the content. Often these will be the same, but some - in particular specialist documents - will have differing values.
The document_type/schema_name pair will eventually replace format everywhere. During the deprecation period we accept these as alternatives; on PUT, the one not supplied will be populated from the other. The index endpoint will filter by whichever is supplied.

Linkables spec is now failing.


Improve tick generation for linear scales

There are a few problems in tick generation for linear scales:

When maxTicksLimit is not set, the number of ticks can exceed 11 which is the default value of maxTicksLimit (see a and b in the example below).
#5922 fixed an issue in _autoSkip so that the number of ticks stay within maxTicksLimit, however tick values are not aligned to a nice number and the tick at the min value is missing (see c and d).
When both stepSize and maxTicksLimit are set, maxTicksLimit doesn’t limit the number of ticks as reported in #2539. In this case, stepSize should work as the minimum unit of stepSize for ticks, and it should be multiplied by a nice number (see d).
Auto-generated ticks differ depending on maxTicksLimit despite it doesn’t actually limit the number of ticks (see e and f).
Ticks in a radar chart overlaps each other as mentioned in #5914. #5922 didn’t fix this as the maximum number of ticks based on the scale size is not used in _autoSkip but in generateTicks (see g).

Version 2.7.3: https://jsfiddle.net/nagix/eygnmo5b/

Master: https://jsfiddle.net/nagix/jds1c98z/

This PR improves tick generation for linear scales by adding the following logic:

In the getTickLimit method, maxTicks is calculated based not only on maxTicksLimit, the default limit (11), the default spacing (40 pixels) or the tick font size but also on stepSize.
In the generateTicks function,

Find a nice number of the dataRange divided by maxNumSpaces for spacing. stepSize is used as a minimum unit if it is specified.
Find the number of spaces (numSpaces) such that both dataRange.min and  dataRange.max are contained in the range.
If numSpaces exceeds maxNumSpaces,  spacing is set to a nice number of numSpaces * spacing / maxNumSpaces, which guarantees that dataRange.min  and  dataRange.max  are contained in the range. stepSize is used as a minimum unit if it is specified.
niceMin and niceMax are adjusted separately based on min, max and stepSize.

Master + This PR: https://jsfiddle.net/nagix/7jxefc94/

Fixes #2539

I have simplified the code and updated the description above.

Rounding could be smarter (works the same in all 3 versions):

Rounding could be smarter (works the same in all 3 versions):

I have been aware of this, which is the separate issue in the tick formatter. Actually, generateTicks generates accurate values (['2.5', '2.25', '2.0', ...]) but Chart.Ticks.formatters.linear rounds values.

@kurkle @simonbrunel Thanks for your comments. I refactored getTickLimit and added minor modification.


Add Spell Schools to Traits

Also removes Schools as a checkbox in the Compendium Browser, making it show in Traits only.

Pull back to the single update to the spell traits record. We can get everything else in line after.

Done


Issues running GEOSgcm with GCC 12

Testing on discover as well as Intel MacBooks has shown that GCC 12 + GEOSgcm has an...issue. Namely, if you run at C12 or C24, with GCC 12 the model will occasionally die in FV3 during mapz.
For example, at C12, when you do many consecutive runs with GCC 11.3 and GCC 12.1, the C12 case will 100% succeed with GCC 11, but only succeed about 20-30% with GCC 12.
However, preliminary testing has shown that if we move the target arch for GNU Intel chips to haswell from westmere all is happy.
I will continue to test this to make sure it works on my systems. But, I invoke the name of @climbfuji to ask if he would be okay with this change for Rosetta2 M1 Macs. He provided the code for M1 Rosetta in https://github.com/GEOS-ESM/ESMA_cmake/pull/274 (which is soon to be brought in).
I think it should be fine (not many Westmere era systems exist anymore), but for now I'm only testing the GNU Intel Linux side and I'm not setup to test Rosetta2.

Thanks for asking! I don't see any problem with updating to "haswell" instead of "westmere".

@climbfuji Thanks. But I might not change it for Rosetta2. If I look here:
https://developer.apple.com/documentation/apple-silicon/about-the-rosetta-translation-environment
I see:

Rosetta translates all x86_64 instructions, but it doesn’t support the execution of some newer instruction sets and processor features, such as AVX, AVX2, and AVX512 vector instructions. If you include these newer instructions in your code, execute them only after verifying that they are available. For example, to determine if AVX512 vector instructions are available, use the sysctlbyname function to check the hw.optional.avx512f attribute.

So, unless sysctl -a | grep hw.optional.avx turns up a 1 for avx and avx2, we might want to keep using Westmere for Rosetta2. My reading of the GCC options seems to say Westmere is basic enough.
Then again, maybe GCC is smart enough to handle that? Eh. We know Westmere works for you, we keep it working! 😄


Rework Branching and CI/CD Pipeline

Things implemented for the CI/CD (as listed in #23):

[x] generate releases and a release branch on tag creation
[x] build binary's for different platforms on release and add them to the release
[x] publish new versions to crates.io
[x] build, test, and format check pull requests and releases

Things implemented for the branching (as listed in #23):

[ ] only have one master branch
[x] change contributing file to make aware of this change
[x] add a note in the readme (right at the top)
[ ] remove the old stable and experimental branches (only after everything is finished)

Will merge this with the stable branch and then do the following things:

merge experimental with stable
rename stable into 'main'
remove the experimental branch


Certificates are shared between entrypoints

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
When two entrypoints with different certificate resolvers are created they should respect them and not use certificate from other entrypoint (both domains point to same server).
If I enter app from entrypoint  "entry-two" (port 2443):

from domain "myapp.duckdns.org:2443/myapp" it works as expected by returning duckdns domain certificate.
from domain "mydomain.com:2443/myapp" it should return either default self signed treafik cert or duckdns one and result in browser warning.

What did you see instead?
Traefik returns certificate from cloudflare provider for domain "mydomain.com", even though it's not configured for this entry point.
What version of Traefik are you using?
Version:      2.9.6
Codename:     banon
Go version:   go1.19.4
Built:        2022-12-07T14:17:58Z
OS/Arch:      linux/amd64
What is your environment & configuration?
traefik.yml:
entryPoints:
  entry-one:
    address: :443
    http:
      tls:
        certResolver: letsencrypt
        domains:
          - main: "mydomain.com"
            sans:
              - "*.mydomain.com"
  entry-two:
    address: :2443
    http:
      tls:
        certResolver: duckdns
        domains:
          - main: "myapp.duckdns.org"

certificatesResolvers:
  letsencrypt:
    acme:
      email<EMAIL_ADDRESS>      storage: /etc/traefik/acme-cloudflare.json
      dnsChallenge:
        provider: cloudflare
  duckdns:
    acme:
      storage: /etc/traefik/acme-duckdns-lan.json
      dnsChallenge:
        provider: duckdns

providers:
  file:
    directory: /etc/traefik/cfgs

/etc/traefik/cfgs/myapp.yml
http:
  routers:
    myapp:
      entryPoints:
        - entry-two
      service: myapp
      rule: PathPrefix(`/myapp`)

  services:
    myapp:
      loadBalancer:
        servers:
          - url: https://myapp-ip

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hello @waterler ,
At the end of the day, internally there is only one TLS store for certificates in traefik. Which means all your certificates are stored in the same map.
So when traefik retrieves a certificate to find the one that matches the best (with the given host name, etc), it looks into that store that contains all of the certificates.
It is a similar situation to the one with routers and TLS options, as mentioned here:
https://doc.traefik.io/traefik/routing/routers/#options
So yes, even though it might be a bit unexpected, but it is working as intended.

I have looked into stores, but seems they are configurable only for static certs, and also there is no other store, just "default" if I understood correctly.
Also documentation for resolvers states "Defining a certificate resolver does not result in all routers automatically using it. Each router that is supposed to use the resolver must reference it." so it contradicts you statement that it works as expected. And then whats the point having it configured on router/endpoint if it does not matter in the end?

The http configuration on entrypoints is just the default configuration of routers.
It's not related to the behavior of entrypoints.
Currently, only the default TLS store works, the fact to have the option to configure TLS store was a design error when we created Traefik v2.
I think we will remove that in v3.
If you want to manage several environments, our recommendation is to use several Traefik.

will it work differently if I will define cert provider directly on router?
My initial setup was two instances of traefik, but I wanted to remove duplication so joined into single instance... guess I could share that by docker mount points.

will it work differently if I will define cert provider directly on router?

no, all the certificates are put in the same set.


[DOC] start_node script doesn't work out of the box without docker

We should add to the documentation (or to the start_node script) comment by @ailisp from
https://github.com/nearprotocol/nearcore/pull/1551.

these are all the start_* scripts we have in nearcore
./scripts/start_localnet.py
./scripts/start_staging_testnet.py
./scripts/start_stakewars.py
./scripts/start_testnet.py
./scripts/start_unittest.py

./pytest/tests/adversarial/start_from_genesis.py
./pytest/tests/sanity/restart.py

closing since this is replaced by nearup


Check NREL external libraries for std prefix before abs() and isnan()

Related to #1223

Will be fixed when ssc pull request 1192 is merged https://github.com/NREL/ssc/pull/1192


python311Packages.lightning-utilities: 0.11.3.post0 -> 0.11.4 + fix on python 3.12

Description of changes
Changelog: https://github.com/Lightning-AI/utilities/releases/tag/v0.11.5

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 327202 run on x86_64-darwin 1

2 packages failed to build:

python312Packages.rising
python312Packages.rising.dist

14 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on aarch64-darwin 1

4 packages marked as broken and skipped:

python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

6 packages failed to build:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
python312Packages.rising
python312Packages.rising.dist
whisper-ctranslate2
whisper-ctranslate2.dist

16 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on x86_64-linux 1

6 packages marked as broken and skipped:

piper-train
piper-train.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

6 packages failed to build:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
python312Packages.rising
python312Packages.rising.dist
whisper-ctranslate2
whisper-ctranslate2.dist

16 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on aarch64-linux 1

2 packages marked as broken and skipped:

piper-train
piper-train.dist

6 packages failed to build:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
python312Packages.rising
python312Packages.rising.dist
whisper-ctranslate2
whisper-ctranslate2.dist

16 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on x86_64-darwin 1

2 packages failed to build:

python312Packages.rising
python312Packages.rising.dist

14 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on aarch64-darwin 1

4 packages marked as broken and skipped:

python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

6 packages failed to build:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
python312Packages.rising
python312Packages.rising.dist
whisper-ctranslate2
whisper-ctranslate2.dist

16 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

Result of nixpkgs-review pr 327202 run on x86_64-linux 1

6 packages marked as broken and skipped:

piper-train
piper-train.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

6 packages failed to build:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
python312Packages.rising
python312Packages.rising.dist
whisper-ctranslate2
whisper-ctranslate2.dist

16 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist


[WINDOWS 10] - issue when trying to run react-native init AwesomeProject

Description
I want to start React Native for the first time.
Additional Information

React Native version: 0.37
Platform: both
Operating System: Windows 10

Error Log
C:>react-native init AwesomeProject
This will walk you through creating a new React Native project in C:\AwesomeProject
Installing react-native from npm...
Consider installing yarn to make this faster: https://yarnpkg.com
Setting up new React Native app in C:\AwesomeProject
events.js:160
throw er; // Unhandled 'error' event
^
Error: This socket is closed
at WriteStream.Socket._writeGeneric (net.js:683:19)
at WriteStream.Socket._write (net.js:734:8)
at doWrite (_stream_writable.js:334:12)
at writeOrBuffer (_stream_writable.js:320:5)
at WriteStream.Writable.write (_stream_writable.js:247:11)
at WriteStream.Socket.write (net.js:661:40)
at WriteStream.stream.write (C:\AwesomeProject\node_modules\ansi\lib\newlines.js:36:21)
at Console.warn (console.js:51:16)
at process.on (internal/process/warning.js:21:17)
at emitOne (events.js:96:13)

the problam was related to nodejs
https://github.com/nodejs/node/issues/9542
I just reinstall nodejs with msi (from the offical site)
and it seems to fix the problam.


Commit & tag missing for 0.9.5 release

This release exists on crates.io and was created on commit f7134ee43d155d4034b8e95836016a592e6325a1 (not clickable because the commit wasn't pushed):
https://docs.rs/crate/tame-oauth/0.9.5/source/.cargo_vcs_info.json

No need to yank this, unless you could no longer find and push the relevant commit in local history?


Classes for use in form validation

I think it would be useful to have some clean CSS classes that can be used in form validation. Style changes like text color, background color...
It is easy to put this in custom CSS, but I think it should be in the basic framework itself. Like bootstraps has-error, has-warning and so on.

It might be worth reasoning about what the best way to do this would be.
You've already mentioned one. But there might be a better way to inform a user that they need to change their input to conform to guidelines.

I'm no designer myself, so here are a couple of examples from other frameworks:

Semantic-UI - Changes background and label color; Adds either a list of messages below the form or some pop up input field captions
Boostrap - Label and input field outline color changes with optional icons
Material Design - Has a very slim approach; Changing the color of the input field (which is basically a line) and adds a text under the input


Become a sponsorship page. A page where people can sponsor or I would like to sponsor.

John will provide the content to add, but the goal is the following:

Communicate current sponsors.
Place information that helps convince someone that a sponsorship adds value.
Button for an actionable next step by user (e.g. [Become a Sponsor] -> form to submit information)

Keep in mind: ideally, the admin page would be used to add a new sponsor, which would dynamically reflect in the "current sponsors".

Keep in mind: ideally, the admin page would be used to add a new sponsor, which would dynamically reflect in the "current sponsors".

Add a button to Email to<EMAIL_ADDRESS>blank e-mail.
2nd step will be a form

@peterbogdan will work on this, this week.

@antoniumolnar need to update the css of the Button and the page for Sponsors.

Link to page: https://www.asrg.io/impact/sponsor/?edit&language=en-us
Frontend needs to be fixed.

Need to have the same width as text above
Need to add spacing from card border to pictures and text.

Planned Hours: 1 Hour
End Date: July 12th, 2021

1 Hour


train_prefix = args[1] IndexError: list index out of range

It is alright to use the package for torch version, but when I run it with tf version, it pops the error:
train_prefix = args[1]
IndexError: list index out of range

The code I use is :
python run_scholar_tf.py data/imdb/processed/ -k 10 --test test --labels sentiment

How can I fix it ?

My apologies. It looks like you're trying to using the tensorflow version, and I just realized that it had a slightly out-of-date set of options. However, I have just pushed an update that brings it closer to the pytorch one. If you pull the latest update, your command should now run. For more examples with the tensorflow model, please have a look at the tutorial.txt file (though I would still recommend using the pytorch one).

Thanks a lot!


husttrans.cls Missing \begin{document}

After I installed the required fonts there is another problem and I think it's hard for me to solve.

Please update the template. I think you are using an outdated version.


Minor statistics tab UI/UX issues

There are a few very minor purely cosmetic issues with the Statistics tab, some of these are my suggestions, and subject to personal preference. Neither of these issues in any way impacts the core functionality of the app, or it's usability in meaningful ways.

The dates are truncated when 7+ days are graphed. I'm using a Pixel 8 Pro with GrapheneOS, not sure if this happens on other screen sizes.
When many days a large number of Wi-Fis/Cells/Beacons are graphed, it becomes hard to extrapolate data from it. Maybe a bar graph would be more suited to this purpose? The graph could also show an exact number when the user taps on a specific bar.
On any tab, the app does not reach all the way down to the bottom. Not sure if this needs to be fixed, as it seems that apps targeting Android 15  be edge-to-edge by default.
Would you consider using Material You system accent colors for the app's accent bars?

I'll see what can be done. Unfortunately the chart library wasn't quite easy to use when configuring how to display the x-axis
Good idea. Maybe there could be a toggle or something for switching between the current line chart and the bar chart
I'm planning on doing some overhaul to the UI in general, will take a look at this then
I don't see problem with that, maybe create a separate issue for it?

I'll let you know if the chart gets any more broken when I add more days.
I was going to suggest this, but didn't want to over complicate things.
The documentation of Android 15 changes has this note to it:

"Apps using Material 3 Compose components might not be negatively impacted because most components apply insets. Apps using Material 2 Compose components might be negatively impacted because Material 2 components don't automatically apply insets."
Now, I'm too stupid to understand what that actually means, but I wanted to see if perhaps just changing the targetSdk would automatically make NeoStumbler display as shown in the documentation. I went into Android Studio, opened build.gradle changed targetSdk to 35, ran Gradle Sync, and ran the app inside an API 35 emulator. This did not make NeoStumbler display edge-to-edge, but it also didn't break the UI in any immediately noticeable way.

I made #155

This is what the tab looks like after another week:


Cannot load a dataset with only compressed shards

** Environment **

OS: [Ubuntu 20.04]
Hardware (GPU, or instance type): N/A

** To reproduce
Steps to reproduce the behavior:

Write an MDS dataset with zstd compression.
Delete the *.mds files.
Attempt to load the dataset via StreamingDataset.

Expected behavior
The dataset is loaded correctly.
Additional context
Instead of loading, the dataset hangs, suggesting that both the uncompressed and compressed files are required. This would mean I need to upload the entire dataset, both compressed and uncompressed, to S3.

Hi @nharada1 , thanks for filing an issue. Once you generated an zstd compression shard files locally using MDSWriter(), you can use the same shard files directory in StreamingDataset() object, the StreamingDataset() under the hood uncompress it for you, you don't need to do it manually. If you are streaming the dataset from a cloud, you just need to upload the compressed shard files to your S3 bucket (you don't need to upload uncompressed shard files to your S3 bucket), and as I mentioned above, the StreamingDataset() under the hood uncompress it for you in your local parameter directory.

Hmmm okay I think the issue was that the local path should have been a separate temporary folder. I'm still seeing strange issues though, for example I'm seeing IndexError: Relative sample index 16 is not present in the shard.00780.mds file. which seems to be a batch size issue during data loading -- is the shard smaller than the batch? When I reduce the batch size it seems to work, although I do get a ton of warnings about not closing the shard files.

Further debugging suggests that this is an issue with shuffling. Here is the code I'm using: https://github.com/nharada1/repro-mosaic-shuffle-issue

Can you include the code used to generate the streaming dataset that you used? That should help us figure out exactly what is happening, thanks!
Some things I noticed:

Write an MDS dataset with zstd compression. 2. Delete the *.mds files.

Unless I am not getting something, you should only be seeing index.json and sequentially numbered #####.mds.zstd files in the created dataset directory?

IndexError: Relative sample index 16 is not present in the shard.00780.mds file.

Offhand, this may indicate that a shard got truncated or overwritten, for example by overwriting different dataset splits to the same directory when creating the streaming dataset.

seems to be a batch size issue during data loading

(Elaborating in case this is helpful) A streaming dataset is a collection of shards strapped together. A shard is a collection of samples, which are available by random access. So an entire streaming dataset has random access, taking an absolute sample index. Within a shard, we call that offset a relative sample index. Batches can be composed out of samples from any shard(s) irrespective of their boundaries. So batch size and shard size don't interact. To debug what exact sample positions go into what batches (before shuffling), you can use scripts/partition/web.py or scripts/partition/txt.py.

although I do get a ton of warnings about not closing the shard files

What warnings are you seeing exactly?

Hey James,
Thanks for your help so far. I believe the issues I was seeing was from re-using my local directory on subsequent runs. I changed things to create a temporary folder instead each time the dataset is loaded and that seems to prevent the issues. Let me do a little more experimenting but if I can't find anything else I may close this issue.

Alright, well every ease of use feedback is appreciated!
Also note StreamingDataset has an argument keep_zip that may be relevant.

I will go ahead and close this. I am having a lot of trouble getting mosaic streaming to work still with my workflow. I see that there's a community slack, but it says I need an @mosaicml.com email to sign up or to contact the admin. Is there a way I can access this slack workspace?

How about this link? https://join.slack.com/t/mosaicml-community/shared_invite/zt-1btms90mc-GipE2ufuPkKY0QBrmF3LSA

That worked! I'll follow up there thank you!


Improve arity error messages

Add a specific errors for incorrect number of arguments or of type arguments in resolveCall. I'm not sure how we lived without this before.

@RyanCavanaugh @DanielRosenwasser mind taking a look? This PR is pain to keep up to date because it changes so many error numbers ...

@DanielRosenwasser I replied to your now-hidden replies, but my replies are now also hidden. (!)


Form Support

Add form support according to https://www.hlx.live/developer/forms
Used on the german homepage as the contact form.
Ask customer what kind of processing is desired, propose ingestion the helix way.

customer confirms excel sheet form definition and data ingestion is ok


feat(bindCallback): rename fromCallback to bindCallback

since fromCallback does not behave in a manner consistent with other fromX methods, it is being renamed to bindCallback as it returns a bound function that returns an observable
closes #876

:+1:

Cool :+1:  :)

It looks like it doesn't quite work properly when scheduled, too, for some reason, so I'm going to try to fix that as well.

So I ended up rewriting this to use @mattpodwysocki's AsyncSubject (which might need a better name :p). This is because the observable returned has some shared state and is effectively multicast. That means that it either needs to maintain it's own list of internal subscribers, or it should use something like AsyncSubject. The former is almost definitely more performant, and we should probably move to that at some point. But the latter was convenient at this point in time.

It's probably something silly with Jasmine, we can leave this here for now, and since we have something that is reproducible (hopefully) we can PR Jasmine with a resolution.

So I'm going to merge this one in, but leave the weird jasmine test in that "jasmine is weird" file for now. Then if we can consistently reproduce that problem, I'll put an issue on Jasmine and try to help them resolve it. Also, those folks might be able to help us improve our test helpers or something. Perhaps that's the source of some of the weirdness.

I'll also try to triage jasmine issue once this is merged.


Remove makeMathExpressionSelector from ColumnSelectorFactory.

Half of the second part of https://groups.google.com/forum/#!msg/druid-development/_Sd78s7yU5U/RJ8qLOJ5DwAJ, skipping the aggregator api changes.

@leventov, pushed some updates you suggested, please let me know what you think about the others. Thanks for the review.

@leventov, I broke up that big method and added tests.

:+1:

👍


Replace AtomicLong with LongAdder in HitsThresholdChecker

The value for the global count is incremented a lot more than it is read, the space overhead of LongAdder seems irrelevant => lets use LongAdder. The performance gain from using it is the higher the more threads you use, but at 4 threads already very visible in benchmarks.
For wikimediumall I see:
                            TaskQPS baseline      StdDevQPS my_modified_version      StdDev                Pct diff p-value
           BrowseMonthSSDVFacets        4.75      (6.3%)        4.65      (8.2%)   -2.2% ( -15% -   13%) 0.348
                       LowPhrase       40.41      (9.1%)       39.59      (7.0%)   -2.0% ( -16% -   15%) 0.427
          OrHighMedDayTaxoFacets        2.49      (7.0%)        2.48      (4.6%)   -0.1% ( -10% -   12%) 0.957
     BrowseRandomLabelSSDVFacets        3.53      (6.2%)        3.53      (6.4%)   -0.0% ( -11% -   13%) 0.989
                       MedPhrase      586.66      (4.8%)      588.40      (5.0%)    0.3% (  -9% -   10%) 0.849
                      AndHighMed      251.69      (9.1%)      253.84      (3.6%)    0.9% ( -10% -   15%) 0.698
                      AndHighLow     1321.95     (10.0%)     1335.73      (5.9%)    1.0% ( -13% -   18%) 0.688
        AndHighHighDayTaxoFacets        4.15      (3.8%)        4.20      (2.9%)    1.0% (  -5% -    7%) 0.323
            MedTermDayTaxoFacets       20.33      (4.5%)       20.58      (4.1%)    1.2% (  -7% -   10%) 0.367
                        PKLookup      245.17      (3.6%)      248.31      (3.6%)    1.3% (  -5% -    8%) 0.266
           BrowseMonthTaxoFacets        8.21     (55.1%)        8.32     (55.6%)    1.3% ( -70% -  249%) 0.940
             LowIntervalsOrdered       50.77      (9.8%)       51.48     (10.5%)    1.4% ( -17% -   24%) 0.662
         AndHighMedDayTaxoFacets       24.18      (1.5%)       24.56      (1.8%)    1.6% (  -1% -    4%) 0.003
     BrowseRandomLabelTaxoFacets        4.21      (5.9%)        4.28      (5.9%)    1.8% (  -9% -   14%) 0.341
       BrowseDayOfYearSSDVFacets        4.56      (5.9%)        4.64     (10.7%)    1.8% ( -13% -   19%) 0.512
                      HighPhrase       51.95      (6.3%)       52.90      (6.8%)    1.8% ( -10% -   15%) 0.381
                          Fuzzy2       77.27      (3.6%)       79.02      (5.7%)    2.3% (  -6% -   11%) 0.130
                         Respell       45.66      (1.8%)       46.73      (1.7%)    2.3% (  -1% -    5%) 0.000
                          Fuzzy1      103.35      (4.0%)      106.17      (3.6%)    2.7% (  -4% -   10%) 0.022
                          IntNRQ       27.92      (8.6%)       28.70      (6.7%)    2.8% ( -11% -   19%) 0.254
            BrowseDateSSDVFacets        1.24     (13.2%)        1.28     (11.8%)    3.7% ( -18% -   33%) 0.351
                      OrHighHigh       61.55     (12.0%)       64.18     (12.0%)    4.3% ( -17% -   32%) 0.260
            BrowseDateTaxoFacets        4.92      (5.8%)        5.14     (12.5%)    4.6% ( -12% -   24%) 0.139
                    OrNotHighLow      985.77      (6.9%)     1031.87      (3.7%)    4.7% (  -5% -   16%) 0.008
                 MedSloppyPhrase       98.83      (8.9%)      104.44      (9.1%)    5.7% ( -11% -   26%) 0.046
               HighTermMonthSort     1340.66     (10.5%)     1419.43      (7.1%)    5.9% ( -10% -   26%) 0.038
       BrowseDayOfYearTaxoFacets        4.93      (5.7%)        5.23     (12.4%)    6.0% ( -11% -   25%) 0.049
                        Wildcard       51.85      (6.9%)       55.03      (5.4%)    6.1% (  -5% -   19%) 0.002
                     AndHighHigh      145.76      (6.2%)      155.93     (10.0%)    7.0% (  -8% -   24%) 0.008
                     MedSpanNear       27.61      (8.1%)       29.85      (7.1%)    8.1% (  -6% -   25%) 0.001
                 LowSloppyPhrase      125.48     (12.5%)      135.78     (11.1%)    8.2% ( -13% -   36%) 0.028
                       OrHighMed      146.22      (8.3%)      159.53      (9.3%)    9.1% (  -7% -   29%) 0.001
                         Prefix3      923.91     (15.1%)     1013.30      (6.7%)    9.7% ( -10% -   37%) 0.009
                       OrHighLow      747.82      (4.7%)      820.26      (4.7%)    9.7% (   0% -   19%) 0.000
                HighSloppyPhrase       24.47      (4.6%)       26.93      (4.1%)   10.0% (   1% -   19%) 0.000
             MedIntervalsOrdered       68.46     (11.9%)       75.68     (12.4%)   10.5% ( -12% -   39%) 0.006
                    HighSpanNear       13.57      (3.2%)       15.32      (5.0%)   12.9% (   4% -   21%) 0.000
                     LowSpanNear       84.84      (6.0%)       95.80      (4.7%)   12.9% (   2% -   25%) 0.000
            HighIntervalsOrdered       19.93      (8.4%)       23.28     (11.2%)   16.8% (  -2% -   39%) 0.000
                         MedTerm      670.60      (4.6%)      897.88      (9.5%)   33.9% (  18% -   50%) 0.000
                    OrNotHighMed      201.35      (4.0%)      271.43      (6.6%)   34.8% (  23% -   47%) 0.000
                         LowTerm      583.19      (3.6%)      872.31      (5.1%)   49.6% (  39% -   60%) 0.000
                   OrNotHighHigh      227.42      (4.3%)      353.58      (8.3%)   55.5% (  41% -   71%) 0.000
            HighTermTitleBDVSort       22.16      (4.1%)       35.54     (12.1%)   60.4% (  42% -   79%) 0.000
                      TermDTSort       79.33      (3.7%)      140.30     (17.6%)   76.9% (  53% -  101%) 0.000
                   OrHighNotHigh      184.76      (3.6%)      339.75     (12.3%)   83.9% (  65% -  103%) 0.000
               HighTermTitleSort       33.86      (3.2%)       64.43     (14.3%)   90.3% (  70% -  111%) 0.000
                    OrHighNotMed      187.30      (4.1%)      357.81     (14.8%)   91.0% (  69% -  114%) 0.000
                    OrHighNotLow      245.45      (6.5%)      492.07     (11.3%)  100.5% (  77% -  126%) 0.000
                        HighTerm      152.40      (4.3%)      313.65     (17.1%)  105.8% (  80% -  132%) 0.000
           HighTermDayOfYearSort      135.35      (2.3%)      311.95     (25.0%)  130.5% ( 100% -  161%) 0.000

The results are easily explained by the code change, the top methods in profiling go from:
10.79%        129797        org.apache.lucene.facet.sortedset.SortedSetDocValuesFacetCounts#countOneSegmentNHLD()
5.81%         69901         org.apache.lucene.facet.taxonomy.FastTaxonomyFacetCounts#countAll()
4.60%         55316         org.apache.lucene.codecs.lucene90.Lucene90DocValuesProducer$25#advance()
4.35%         52334         org.apache.lucene.util.packed.DirectReader$DirectPackedReader20#get()
4.28%         51449         org.apache.lucene.search.TopScoreDocCollector$SimpleTopScoreDocCollector$1#collect()
3.74%         45035         org.apache.lucene.util.packed.DirectReader$DirectPackedReader12#get()
3.44%         41324         org.apache.lucene.search.TopFieldCollector$TopFieldLeafCollector#countHit()

to
11.43%        131366        org.apache.lucene.facet.sortedset.SortedSetDocValuesFacetCounts#countOneSegmentNHLD()
6.19%         71133         org.apache.lucene.facet.taxonomy.FastTaxonomyFacetCounts#countAll()
4.88%         56035         org.apache.lucene.codecs.lucene90.Lucene90DocValuesProducer$25#advance()
4.48%         51495         org.apache.lucene.util.packed.DirectReader$DirectPackedReader20#get()
4.01%         46096         org.apache.lucene.util.packed.DirectReader$DirectPackedReader12#get()
3.30%         37975         jdk.internal.foreign.MemorySessionImpl#checkValidStateRaw()
3.15%         36257         org.apache.lucene.util.packed.DirectMonotonicReader#get()

with the top score collector moving way down to:
0.93%         10727         org.apache.lucene.search.TopScoreDocCollector$SimpleTopScoreDocCollector$1#collect()

I pushed an annotation, the speedup on some queries is huge: https://people.apache.org/~mikemccand/lucenebench/CombinedHighHigh.html.

Nice! Does it make sense to change other AtomicLong  occurences to LongAdder? Is see couple of other usages where there might be some opportunity to squeeze some more gains probably?

Does it make sense to change other AtomicLong occurences

@shubhamvishu I would say only if those usages are a "many write & few read" scenarios.

@shubhamvishu what Ben says, plus keep in mind that the adder is less consistent and probably slower for <= 2 or 3 threads competing scenarios on x86 hardware (that seems to be what it comes down empirically today at least). This needs handling with care :) Also, the better solution (if possible) is generally to just not have the contention rather than implementing a better solution. I think this one in particular has an even stronger possible solution (it still shows up in profiling and it should no), looking into that once I find some time :)

@benwtrent @original-brownbear Yes, I agree with both points. I haven't looked into the profiler but I was mainly referring to the usages I see in DocumentsWriter or BufferedUpdatesStream where we could have some high contention between the indexing threads?. This seems like the scenario of multiple threads(I assume usually greater than 2 or 3 indexing threads) updating it simultaneously ("many write & few read" as Ben mentioned or maybe "many write & many read")?

@shubhamvishu stuff like this gets tricky when you see patterns like
if(pendingNumDocs.incrementAndGet() > IndexWriter.getActualMaxDocs()) {
There, you want to be ABSOLUTELY sure that you incremented and trip on a get. This to me seems to make using LongAdder for something like pendingNumDocs a bad idea.

@shubhamvishu looking at BufferedUpdatesStream & bytesUsed, that may be a candidate. However, benchmarking would have to prove it out.

Makes sense @benwtrent! For BufferedUpdatesStream as its on the index side we should check on the indexing time and not the regular luceneutil benchmarks to check the QPS?

@shubhamvishu I do not know off hand which benchmarks should be done


Fix application bundle not being served by webpacker in dev-setup

The application bundle was not correctly served in the local docker setup and instead a debug version was used and an error thrown in the browser console.
We fix this by specifying where webpacker should save the static files (default location where rails will pick it up in the "old" sprockets assets pipeline we use). Production and test environments are not affected by this as they precompile all the assets in advance and do not use the webpacker dev server to hot reload files during development.
See also this medium post and this comment for the issue.

Please test locally if you encounter any issues. If the error still occurs there might be caching issues, so do this once:

Remove the public/packs folder entirely.
After that: Restart the webpacker docker container.

Then the errors should be gone in the browser console. Only warnings should be shown, not one single error.


Can we create an API for the hotspot areas are declared in India and point them out in Map

We should create an API for the declared hotspot places over India and point them out in map, for as an information.

The data is only available for districts. eg: We can know the reported cases in Jaipur as a whole but not more refined areas such as 'Malviya Nagar'. This would make sense too, seeing as making the information of the home of the infected public, could cause panic in that area.
If you can find a more refined data set, would love to make this enhancement

Doubt if we will ever have this data. Closing.


Improvements to pac4j-oidc

This pull request contains 3 minor improvements to the OIDC implementation, vital for our implementation:

Upgrade to a the latest, stable SDK (5.1).
Allow setting custom timeouts for all socket operations. The defaults are quite high and can cause a DoS in case of a failing or slow OIDC provider.
Give the possibility to use a different ResourceRetriever. We need this to add caching and to work around a problem in the discovery metadata returned my Azure AD. https://login.microsoftonline.com/common/.well-known/openid-configuration contains invalid characters in the issuer URL. This is an error on the side of Azure AD, but at the moment cannot be easily worked around. In a custom ResourceRetriever, the invalid characters can be escaped.

:+1:

Excellent! Thanks. See my only question before I merge.
BTW, would you mind adding a MicrosoftOidcClient with the apprpriate discovery url and ResourceRetriever (https://github.com/pac4j/pac4j/issues/415)? It might be another PR.
It will be available in pac4j v1.8.3.

I've moved the default timeouts to HttpConstants and set them to a more sensible 15s. 0, which was used before this PR means infinite, which is probably not what you want. Also, I've opened up IDTokenValidator to subclasses. The same issuer url is giving trouble with the validator as well.
I'll submit another PR next week with the AzureADOidcClient.

Good. I will merge "as is" and update the default timeouts later on. I think 15s for a connect time out is too much as it lets you reach a computer on the Sun from the Earth ;-) Paris - San Francisco ~ 100ms to connect

See: https://github.com/pac4j/pac4j/pull/419


Management > Configuration (ossec.conf) in Manager Configuration missing from Wazuh v4.8.0 Dashboard (running in docker)

Hi all
I've just deployed Wazuh v4.8.0 as a single node in docker and I don't have the Configuration option (ossec.conf).
I know the ux was changed in v4.8.0 but this is the first time I am using Wazuh.
Any help is appreciated!
Thanks.

Ah, I found it under Settings.
The issue can be closed. sorry for wasting people's time to read my stupid issue.
Regards


Support caller-specified time source via callback

This change adds a new callback that allows the caller to specify
a time-providing function for s2n. This allows us to use faster
time sources in some environments, and will also allow for time
to be mocked in certain unit tests.

I'm confused why f36f5bc caused the s2n_cbc_verify_test to fail for the .5 sigma case. Does the initial random seed of data give this unit test a chance of failing at any given run?

A single-tailed uniform random distribution should fail a .5 sigma test about one in every 200 iterations. We could make it less sensitive, but then the test wouldn't be as valuable. I should really convert those tests to something more discerning. I have some replacements from Martin Albrecht that do a different kind of test but haven't integrated them yet.


Parent child relationship

I have quick question, does parent child relationship work while saving the data?
For example, I have following classes:
public partial class Application
{
    public Application()
    {
        Stations = new HashSet<Station>();
    }

    public int ApplicationId { get; set; }
    public string ApplicationName { get; set; }

    public ICollection<Station> Stations { get; set; }
}

public partial class Station
{
    public Station()
    {
        TestResults = new HashSet<TestResult>();
    }

    public int StationId { get; set; }
    public string StationName { get; set; }
    public int ApplicationId { get; set; }

    public Application Application { get; set; }
    //other properties removed for brevity
}

Below code is inside OnModelCreating
modelBuilder.Entity<Application>(entity =>
{
    entity.Property(e => e.ApplicationName)
        .IsRequired()
        .HasMaxLength(50);
});

modelBuilder.Entity<Station>(entity =>
{
    entity.HasIndex(e => e.StationName)
        .HasName("IX_Stations")
        .IsUnique();

    entity.Property(e => e.StationName)
        .IsRequired()
        .HasMaxLength(50);

    entity.HasOne(d => d.Application)
        .WithMany(p => p.Stations)
        .HasForeignKey(d => d.ApplicationId)
        .OnDelete(DeleteBehavior.ClientSetNull)
        .HasConstraintName("FK_Stations_Applications");
});

Following is my input for seeding the Applications and Stations table.
void SeedApplications()
{
	using (var repository = RepositoryFactory.GetRepository())
	{
		var applications = repository.SeedApplications(new Application[] {
			new Application
			{   ApplicationName = "Front EOL", ApplicationId = 0, Stations = new Station[]
				{
					new Station{ StationName = "1600N" ,StationId = 0, ApplicationId = 0 },
					new Station{ StationName = "1600S" ,StationId = 0, ApplicationId = 0 },
					new Station{ StationName = "1700S" ,StationId = 0, ApplicationId = 0 },
				}
			},
			new Application
			{   ApplicationName = "FAM", ApplicationId = 0, Stations = (new Station[]
				{
					new Station{ StationName = "1650N",StationId = 0, ApplicationId = 0  },
					new Station{ StationName = "1700N",StationId = 0, ApplicationId = 0  },
					new Station{ StationName = "1750N",StationId = 0, ApplicationId = 0  }
				}
				)
			}
		});
	}
}

Following is a class to create a Repository instance:
static class RepositoryFactory
{
	public static Repository GetRepository()
	{
		DbContextOptionsBuilder b = new DbContextOptionsBuilder();
		b.UseSqlServer(System.Configuration.ConfigurationManager.ConnectionStrings["TestResultServer"].ConnectionString);
		return new Repository(b.Options);
	}
}

Actual repository class:
public partial class Repository : DbContext
{
        public virtual DbSet<Application> Applications { get; set; }
        public virtual DbSet<Station> Stations { get; set; }

	public IEnumerable<Application> SeedApplications(IEnumerable<Application> data)
	{
		///This code was auto generated, if you make any change in this method then next time when the code is regenerated your changes will be overwritten
		if(data.Any() == false)
			return data;
		var items = data.ToList();
		using (var transaction = Database.BeginTransaction())
		{
			Applications.AddRange(items);
			this.BulkInsert(items, new BulkConfig {  SetOutputIdentity = true });
			transaction.Commit();
			return items;
		}
	}
}

I was hoping that after inserting Applications records, Stations records will be inserted too. But that never happens. Following is my debug window output which shows after insert Stations  property is cleared.
Before bulk insert:

After bulk insert:

If use regular Insert from EF Core it just works fine (code which is commented in the above screen shot). The problem only occurs if I use BulkInsert method.
I am wondering if you can please clear up my doubt and let me know if this is possible or not :)
Thanks.

Saving related(child) data in not done by automatically.
However you can save them explicitly.
In README.md there is following info:

SetOutputIdentity is useful when BulkInsert is done to multiple related tables, that have Identity column.
So after Insert is done to first table, we need Id-s that were generated in Db becasue they are FK in second table.
It is implemented with OUTPUT] as part of MERGE Query, so in this case even the Insert is not done directly to TargetTable but to TempTable and then Merged with TargetTable.
When used if PreserveInsertOrder is also set to true Id-s will be updated in entitiesList, and if PreserveInsertOrder is false then entitiesList will be cleared and reloaded.

So you should first use PreserveInsertOrder = true in order for Items to be updated and that way SubItems won't be cleared, and then just call another BulkInsert for SubItems:
using (var transaction = Database.BeginTransaction())
{
    //Applications.AddRange(items); // this is not required
    this.BulkInsert(items, new BulkConfig {  SetOutputIdentity = true, PreserveInsertOrder = true });
    var subItems = new List<Station>();
    foreach(var item in items)
    {
        foreach(var station in item.Stations)
            station.ApplicationId = item.ApplicationId; // setting FK to match its PK that was Db generated
        subItems.AddRange(item.Stations)
    }
    this.BulkInsert(subItems);
    transaction.Commit();
    return items;
}

PS
Also in SeedApplications method Id (int) is 0 by default so you don't need = 0, setting just StationName is enough:
new Application
{   ApplicationName = "Front EOL", ApplicationId = 0, Stations = new Station[]
    {
        new Station{ StationName = "1600N" },
        new Station{ StationName = "1600S" },
        new Station{ StationName = "1700S" },
    }
}

You can make a test with these scripts.
First create EFCoreBulk Test DB:
EFCoreBulkTestDB.sql
Then Run this Sql script.
EFCoreBulkTestSQL.sql
It should give you correct error and then refresh it and check it Temp table remaind in DB.


Provide a more flexible transport

The client / server architecture, as described by https://modelcontextprotocol.io/docs/concepts/architecture#overview, doesn't fit the serverless use-case (talking for Cloudflare's Workers AI), in which: either the client is running on a server or both client/server are running on the server.
Would it be possible to add a new transport layer that's essentially a JS API, not involving any IO?
Moreover, MCP server are hardcoding a transport: https://github.com/modelcontextprotocol/servers/blob/df0b733c7d086432de14504863c9c0288fc36686/src/gitlab/index.ts#L526-L527, which makes them not easily portable.

I think this issue is describing three different things:

Agreed that the nature of the protocol is currently a poor fit for serverless deployments. I started https://github.com/modelcontextprotocol/specification/discussions/102 to discuss this in more detail.
Transports are very easily written, so yes, it's possible to create a transport that runs whatever code you want instead of doing I/O.
It's a good point that perhaps servers don't need to hardcode transports! Would welcome any thoughts on how to generalize this without dictating too much of how the server itself is implemented.