Implement Database Connection Pooling

After having a rough time in our backend and after noticing some connection latency to our Database due to many reasons some of them might be:

Poor DB performance
Poor connection latency
Poor Query Optimisation
No Timeout set to DB Connection sessions to be cleared
No replication on the DB connection backend.

Although I'm not qualified to assume how (Or if) our queries can be optimized more to fix the problem at the moment, but from my findings I noticed that we can do some measures to prevent latency from happening (Or at least decrease it by a lot)
NOTE:
If I am assuming right:

Latency in the Database --> 2. Causes Disconnections In the backend --> 3. Causes the frontend errors that we see

Connection pooling documentation for Digital Ocean: https://docs.digitalocean.com/glossary/connection-pool
Connection pooling for Typeorm: https://github.com/brianc/node-postgres/tree/master/packages/pg-pool
Note in TypeORM the connection pooling is supported out-of-the-box as I've read here:  https://stackoverflow.com/questions/67414360/connection-pooling-in-typeorm-with-postgressql
CC: @jainkrati @laurenluz @mohammadranjbarz @aminlatifi @geleeroyale @Rolazo
Note that this is different from supporting READ and READ/WRITE replicas, the issue that was opened here: https://github.com/Giveth/impact-graph/issues/1016

@mhmdksh Thanks for the report, you say we don't need to do anything to utilize the connection pool and it's already utilized? otherwise, what would be the next step in your view?

@mhmdksh Thanks for the report, you say we don't need to do anything to utilize the connection pool and it's already utilized? otherwise, what would be the next step in your view?

Yes @aminlatifi As I was researching and like I said this is supported natively when using TYPEORM (Thanks guys). We were able to support this on both Production and Staging with Digital Ocean and AWS.
However there was one small configuration field added later by in this commit https://github.com/Giveth/impact-graph/commit/9a70d1162e1d77cd4c1536944c3262b85ceb658f that will help us in modifying the pooling config. Thanks to @mohammadranjbarz
We are monitoring the performance, but I think this issue can be closed now

This issue is on UAT column
How can I test it ?

@mhmdksh pls work with @maryjaf to close this as its already in production and we can close after testing.

I've checked with @mhmdksh and we can close this issue.


Ion-Select is hidden behind Ion Modal

Bug Report
Ionic version:
"@ionic/core": "^5.0.7",
Current behavior:
If I put an ion-select inside an ion-modal, the ion-select disappear behind the modal and it's impossibile to use.
Expected behavior:
ion-select is located in front of the modal.
Steps to reproduce:
Create an ion-modal and add an ion-select inside the modal

@liamdebeasi so there is no solution for this problem?

There is likely a solution to this issue, though it has not been implemented. I am only closing this issue because it is a duplicate of another issue (https://github.com/ionic-team/ionic/issues/21181).


[Settings] Right-clicking on Settings in taskbar shows the correct title

Summary of the Pull Request
What is this about?
PR Checklist

[x] Applies to #5234
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
This configures the generated "AssemblyTitleAttribute" which is used in TaskManager / Right Click Task Menu
[assembly: System.Reflection.AssemblyTitleAttribute("PowerToys Settings")]
Validation Steps Performed

Run Microsoft.PowerToys.Settings.UI.Runner
Right click Taskbar icon and validated name
Open Task Manager and validated name

Task Bar:

Task Manager:

Oooo, good catch!


[bulk] validation: validation fixes JM 5/13a

This PR fixes 43 issues.

Template message:
Global effort to fix build validation errors
@Rick-Anderson
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you'd like to see. Thank you!


Missing conv.h5

Hi,
I read through the paper and tried executing the code.
Seems something missing.
it asks for conv.h5, but it's not present.
SPReID/data/dump/conv.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
thanks,
Kind Regards
Arun

@emrahbasaran due to size limit on GitHub, not all the h5 files are properly uploaded to data/dump. Please update the repo with an external link to Dropbox or GoogleDrive, linking to all the ImageNet pre-trained weights.

I have updated the repo and added the links to the README file.

Thanks you !!


DBAASCLI-66 Add 'out of memory' cluster pods check

License check is ok.

License check is ok.

License check is ok.

License check is ok.

License check is ok.

License check is ok.


Improve generated TypeScript Result types for Cedar WASM

Category
User level API features/changes
Describe the feature you'd like to request
The cedar-wasm and ffi crates currently use #[serde(untagged)] quite extensively -- I assume to minimize serialisation overhead.
Here's an example:
#[derive(Tsify, Debug, Serialize, Deserialize)]
#[serde(untagged)]
#[tsify(into_wasm_abi, from_wasm_abi)]
pub enum AuthorizationResult {
    Success { response: InterfaceResponse },
    Error { errors: Vec<String> },
}

This results in TypeScript types like the following:
export type AuthorizationResult = { response: InterfaceResponse } | { errors: string[] };

These types are quite difficult to work with in TypeScript -- I work with TypeScript fairly frequently and rely on language server autocomplete to help me work with new SDKs. When using the my prerelease build of the Cedar WASM SDK I was surprised to find no autocomplete suggestions in my IDE for the result type above, which makes it more difficult to discover the available attributes on the result type.

The approach I've seen in many libraries is to instead use a tagged representation of the struct. For example:
export type AuthorizationResult = { response: InterfaceResponse, type: "success" } | { errors: string[], type: "error" };

This helps immensely with the TypeScript developer experience:

This is a matter of changing #[serde(untagged)] to #[serde(tag = "type")], although we may want to limit this to the WASM crate only.
Describe alternatives you've considered
I hit this problem when authoring a GitHub Action for validating Cedar policies and ended up forking the package and implementing #[serde(tag = "type")] myself on the structs I was using for validation
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

We considered that. We opted for the current approach to bias towards an error-first convention:
if ('errors' in result) {
  //handle the error
  //early return
}

Once you're past that early return, typescript has no problem giving you syntax hints.
Notice in the screenshot of your suggested approach, the only thing it can show in intellisense is type, because that's the only property that's present in both members of the union.
With the current approach, have flatter types for the relevant happy case, with less nesting. These allows some of our result unions to be like TypeIActuallyWantWithoutNesting | { errors: string[]}
For instance:
export type JsonToPolicyResult = { policyText: string } | { errors: string[] };
export type PolicyToJsonResult = { policy: Policy } | { errors: string[] };
export type CheckParsePolicySetResult = { policies: number; templates: number } | { errors: string[] };
export type CheckParseTemplateResult = { slots: string[] } | { errors: string[] };
export type FormattingResult = { formattedPolicy: string } | { errors: string[] };

completed by #824


ReadItLater not creating file, creating error from YouTube page

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Expect ReadItLater to create a markdown file from a YouTube page without an error.
Current behaviour
When using ReadItLater plugin to create markdown file for YouTube page, I receive an error.  This has recently started and has changed from its previous exhibited behavior. Showing error screenshots attached from Obsidian console.

Steps to reproduce
I’ve deleted and reinstalled the plugin after closing out Obsidian and restarting computer.
I’ve started with the templates as they come with the plugin (and how I had them set up previously that had worked).
I’ve tried a Twitter page and other articles with no issue. Only seeing the trouble on YouTube pages. I’ve tried changing my browser defaults around to see if that may have any bearing on this, and haven’t noticed any differences.
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
1.2.7
Plugin Version
0.3.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

It’s working now.  I’m not sure what changes may have occurred, but closing this out.


Fix # It doesn't download images anymore due to updated HTML

The HTML was recently updated on unsplash.com. There's problem to use your downloader after that. But this fix will return it to work.
I desired to rewrite to to use new Unsplash API but still has no time. Perhaps I will do it a bit later on non working time.

Thanks! For future reference, please try to keep the coding style of projects you contribute to.

Ok, it's cool! Thank you for your great tool.
I'll be very appreciate for comment about my code-style. I've just tried to make it same to yours :) But I just learning. Yeah.
By the way lines 158-161 are also might be removed. I couldn't find any image without an author.

Cool, will check it out :)


Extend the component actor interface with online reconfiguration

All component actors now have to implement a handler that receives the configuration in the same way as their parent plugins during initialization.

Closing as superseded by #4344


Fix GetTasksResponse TaskFailure deserialization

Description
The GetTasksResponse failures is currently trying to desierialize a list of String errors, whereas in reality the response is a map which results in a deserialization error.
There was also a missing property "cancelled" on the Status object, that I have added in this PR.
Issues Resolved
https://github.com/opensearch-project/opensearch-java/issues/666
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock  @Bfindlay the Info changes look good, but Status is not , I have been looking what OpenSearch server returns and it is indeed just opaque bytes for error or response, we may follow the same fix as Elastic folks [1] (this repository is ASF v2.0 licensed, for a reference).
[1] https://github.com/elastic/elasticsearch-specification/pull/2250

@dblock @Bfindlay the Info changes look good, but Status is not , I have been looking what OpenSearch server returns and it is indeed just opaque bytes for error or response, we may follow the same fix as Elastic folks [1] (this repository is ASF v2.0 licensed, for a reference).
[1] elastic/elasticsearch-specification#2250

Good eyes. @Bfindlay

@reta Interesting 👀 let me take a look.

Ok I see whats happening here, I will update the PR shortly.
Regarding the changes to the missing property on Info I'm going to split it out into its own PR for visibility so its not mixed in with this larger change.

@reta I have updated the GetTasksStatus.response to now return JsonData. This however raises the question, on Info.status  and if it should be following the same pattern?

This however raises the question, on Info.status  and if it should be following the same pattern?

Thanks @Bfindlay , I think it may need to return JsonData as well

@reta PR updated to return JsonData for both properties 👍

@reta PR updated to return JsonData for both properties 👍

@Bfindlay it looks great, thank you, I think the test case we have is a bit isolated from the real flow, could we add the integration test for it? I could certainly help you here

@reta PR updated to return JsonData for both properties 👍

@Bfindlay it looks great, thank you, I think the test case we have is a bit isolated from the real flow, could we add the integration test for it? I could certainly help you here

I can take a look at some iTests. If you had any specific scenario ideas let me know.

I can take a look at some iTests. If you had any specific scenario ideas let me know.

We have some very limited coverage in org.opensearch.client.opensearch.integTest.AbstractNodesIT (only tasks list), would be great to add more test cases there to cover the get APIs

I can take a look at some iTests. If you had any specific scenario ideas let me know.

We have some very limited coverage in org.opensearch.client.opensearch.integTest.AbstractNodesIT (only tasks list), would be great to add more test cases there to cover the get APIs, thank you

@reta I've added an iTest for JsonData response however facing an iTest issue in the pipeline thats not happening locally. I'll probably have time to get to this sometime over the week.

@reta I have put this back into draft, sorry for the commit spam on this one.
After adding the iTest for tasks, it has caused side effects in many other iTests due to the 'strict' configuration of the underlying http transport in regards to the OpenSearch warnings.
As it creates a .tasks system index, a lot of existing iTests are requesting all indices, or manipulating all indices which now includes the .tasks index.
Example: [this request accesses system indices: [.tasks], but in a future major version, direct access to system indices will be prevented by default] and therefore throws a WarningFailureException
I'll probably put this one on hold for a bit and can get back to it next week as this has expanded the scope quite a bit.
Let me know if you have any thoughts.

@reta I might not be able to finish this one anytime soon just low on time at the moment. Will see how I go but the additional work added as side effect of the tasks iTest just added more time.

@reta I might not be able to finish this one anytime soon just low on time at the moment. Will see how I go but the additional work added as side effect of the tasks iTest just added more time.

@Bfindlay please take your time and thank you for working on this, there should be no pressure on you to finish the change as soon as possible, in any case, if you need a hand - happy to help there. Thanks again!

Hello @reta  and @Bfindlay , where are we with this ? can I help to solve the issue ?.
Thank you

Hello @reta and @Bfindlay , where are we with this ? can I help to solve the issue ?.

@uriofferup the issue seems to be stuck at the moment, if you have time to pick it up, would be highly appreciated, thank you


feat: Add Client Info to available Google Cloud Clients

This is used internally to gather aggregate usage metrics for the LangChain integrations

Note: This cannot be added to some of the Vertex AI integrations at this time because the SDK doesn't allow overriding the ClientInfo

Added to:

BigQuery
Google Cloud Storage
Document AI
Vertex AI Model Garden
Document AI Warehouse
Vertex AI Search
Vertex AI Matching Engine (Cloud Storage Client)

@baskaryan, @eyurtsev, @hwchase17

@holtskinner looks good -- could you resolve linting issue?

@eyurtsev I think I resolved the issue.
Is there a recommended procedure for accessing the LangChain package version?

@eyurtsev Anything else needed to merge?


Visual Studio 2015 RC support

This is my initial attempt at adding support for Visual Studio 2015 RC. I'm new to Gradle development and not an expert on Visual Studio so comments and suggestions are all welcome.
Through a bit of experimentation and some help from this blog post (http://blogs.msdn.com/b/vcblog/archive/2015/03/03/introducing-the-universal-crt.aspX) it seems the major difference that breaks Gradle support is the inclusion of the Universal CRT, which requires extra include and library paths. I haven't been able to find much other information on the UCRT, everything just seems to point back to that blog post.
The DefaultVisualStudioLocator and DefaultWindowsSdkLocator both work fine as-is. Windows Kit 8.1 must still be included as before, so no changes were required there. It seems to me to make sense to introduce the concept of the UCRT into Gradle, which is the approach I have taken. The DefaultUcrtLocator will look either in the registry or a user-specified location, there isn't really an option of locating it through the system path as there are no executables in there. I have made locating a UCRT optional, if it does not find one it will just not add any paths. This allows it to be backwards compatible with previous VS versions, but could potentially result in confusing missing headers or linker errors if it for some reason didn't locate the UCRT properly.
The UCRT is by default installed under C:\Program Files (x86)\Windows Kits\10, but the layout under that is slightly different from the other Windows Kits directories in that there is also a version directory under include. The default installation location from VS2015 RC will result in adding C:\Program Files (x86)\Windows Kits\10\Include\10.0.10150.0\ucrt to the include path and C:\Program Files (x86)\Windows Kits\10\Lib\10.0.10150.0\ucrt\x64 to the library path. The version number 10.0.10150.0 changed between CT6 and the RC, so I have assumed it can change and handled that accordingly in the locator. I don't know if there is a possibility of having multiple versions of the UCRT installed concurrently, but have catered for that in the code too, picking the greatest version found as the default. We could perhaps allow the user to override that, but as I'm not even sure if it's a valid use case I haven't included that ability for now.
With regards platform support, from my installation it appears that:

Windows Kit 8.1 has support for arm, x64 and x86 but not arm64
The UCRT (aka Windows Kit 10?) has support for arm, arm64, x64 and x86
As there is not yet support for arm64 in Gradle, I have disregarded it for now. I'm not sure how that would work anyway, with no support for it in Windows Kit 8.1 - perhaps someone else knows more?
ia64 is supported by Gradle, but as there is no support for it in VS2015 so I have disregarded it. I think the existing behaviour of the DefaultWindowsSdkLocator will just end up adding a non-existent library directory and so result in a linker error. I haven't handled the possibility explicitly in locating the UCRT, should this raise an exception or is it okay to just ignore the possibility and let the user see the resultant linker error?

Using this patch I've been able to compile all the native samples as well as some other projects. I have also added cunit and google-test libraries compiled using VS2015 RC so that those samples can be used with VS2015.
Comments, suggestions?

Fixes #GRADLE-3313, much appreciated!

Yeah, I've also used it to work with VS2015 and it works a treat.

FYI, we're working on our Windows test infrastructure to support testing VS2013/2015. Once we have that done, we'll be able to process this PR.

BTW the above-mentioned work on test infrastructure is in progress.

The work on the performance test infrastructure is completed. I'm marking it as waiting for Gradle team's response as the engineer that owns this PR is currently not available.

Has anybody tested Visual Studio solution and project file generation tasks with this fix?

hey guys, any forecasts on when this could get pulled in officially?

This has not fallen off the radar, and we're keen to add the support.  There is a little work that needs to be done before we can merge this and we expect to start that soon.  I can't provide a concrete date, but it shouldn't be much longer.  Thanks for the continued patience!

Looking forward to this MR. Visual C++ in 2015 support latest C++11 which is quite helpful in development.

hey guys, any forecasts on when this could get pulled in officially? This PR is one year old ;)

Superseded by https://github.com/gradle/gradle/pull/695


JBPM-4625 Generation of JPA enabled models

@wmedvede I'll have to defer to @psiroky's knowledge about EAP/Wildfly modules. Whilst drools-wb-webapp isn't a supported deployment; I assume we'd need something similar to kie-wb-distributions?

Since JAXB is also used in other xml parsings around the platform (e.g. jbpm runtime, rest services) I can guess the different different distributions has already the proper configuration


[Feat] Font size option

Having Font size option will be a great feature

This is not that easy, as the fonts within the page itself are all detached from one another, referenced from the main CSS stylesheet. I did try implementing it but seems like there would be a bigger change required to do something this dynamic. Maybe this can be expected in later stages of development, but not right now.

No problem. Thanks


Update Apache parent POM, modernizer

Update modernizer and fix new recommendations, primarily replacing
Guava Iterables and making more use of Stream
Add ConfigurationCopy constructor to accept a Stream
Add sequential stream() to ScannerBase to easily stream results and
replace new StreamSupport(scanner.spliterator(), false) with this
Fix incorrect imports of dependencies shaded into Hadoop
Remove no longer needed m2e ignores in the POM
Also bump sortpom and datasketches and add two new enforcer QA rules

I made some extra changes... I didn't expect the dependency analyzer changes, but I fixed the problems due to that.

@DomGarguilo Let me know when you're done with your review. I have verified all the ITs pass now. I had a slight problem with RowHashIT, but fixed it.


Deprecate DefaultArtifactPublicationSet

This is a stepping-stone to deprecating the archives configuration. We simplify the DefaultArtifactPublicationSet to simplly pass-through all candiates to the backing archives configuration.
Future changes to deprecate the archives configuration will update the configuration to emit deprecation warnings when adding artifacts to it. Instead, users should explicitly add task dependenices to the assemble task.
Note: We did not remove this class entirely since KMP relies on it.

The behavior changes here are too great. We just need to deprecate archives altogether and this class will go away when it is deleted.


Link with Time Zone Database on Iana

I'm trying to comprehend the connection between IANA's Time Zone Database and this project.
I've observed that IANA's latest version is 2023c, while on this project, it's 2023b.
Is information from IANA's TZ Database utilized to define boundaries within this project?
Is the release naming convention for this project aligned with IANA's versions?
Understanding how this project aligns with IANA's latest updates would be crucial for contributions, especially when discrepancies arise between the two projects.

The IANA's TZ Database sort of contains boundary information in terms of general geographic descriptions, but this project defines the precise location of those boundaries in a GeoJSON and shapefile format.
Regarding your other questions, the Readme should hopefully be able to answer your question as follows:

To maintain consistency with the timezone database, this project will only create a new release after the timezone database creates a new release. If there are no new timezones created or deleted in a timezone database release, then this project will only create a release if there have been changes performed to the boundary definitions of an existing zone within this project.

So, since the 2023c tz database release didn't include any new zones or zone deprecations, there was no need to create a new timezone-boundary-builder release. The reason the release is at 2023b is that I was still working on a 2023a timezone-boundary-builder release and when 2023b came out, so I just set the release to that version number.

Thanks a lot for the clarification.


Calc can now deprecate "lib_mort" (and maybe others) since the functionality is inside "library.calc"

Pending investigation to make sure nothing gets lost.

Closing as redundant, since this was done ages ago.

Closing as redundant, since this was done ages ago.


Regression In v3.4.3, Characters Go Missing, Worked Fine In v3.4.2 (only shows up in some fonts)

Tested with two versions
its broken in the latest version, and it is a weird bug does not show up in all fonts.
First here is the font
Mirza-Regular SDF.zip
The sentence i was testing with is:
السلام عليكم
when i used RTLTMPro.v3.3.0.unitypackage
it worked just fine

but using v3.4.3.unitypackage it is broken (latest at the moment), it looks like this:

Literally, the only thing i did was install package, and remove package.
Note: not sure if it effects anything, but in both tests i moved the package in a sub folder
This is the range i used for the font asset creation and its 2kx2k atlas resolution with padding of 5
48-57,1548,1563,1567,1570-1594,1600-1641,1648,1662,1670,1688,1705,1711,1740,1776-1785,64343-64345,64379-64381,64395-64401,64403-64405,64509-64511,64606-64611,65154-65164,65166-65170,65172-65176,65178-65180,65182-65184,65186-65188,65190-65192,65194-65204,65206-65208,65210-65212,65214-65216,65218-65220,65222-65224,65226-65228,65230-65232,65234-65236,65238-65240,65242-65244,65246-65248,65250-65252,65254-65256,65258-65260,65262-65264,65266-65276,32-126,160-255,8192-8207,8210-8226,8230,8234-8240,8242-8244,8249-8250,8252,8254,8260,8286,8298-8303,8364,8482,9633,0600-06FF,0750-077F,08A0-08FF,FB50-FDFF,FE70-FEFF
Also, every time i reimported, i recreated the font assets.
Note:
using Segiou (the one that came in the package) it works
and using Amiri font (found on google) also worked.
but i can not imagine it is an issue with the font, because in the older version mirza simply worked, i guess there are some requirements on which font to use ?
Note 2:
i did one more test using RTLTMPro.v3.4.2.unitypackage and it worked, without the issue, so the change appeared only in the latest version.
Note 3: upon further testing it happens in other fonts, it happened in Arslan font as well
(A) Arslan Wessam A (A) Arslan Wessam A.zip

Same issue with me was you able to solve it


Convert all fields to optional

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
class HeroBase(SQLModel):
    name: str
    secret_name: str
    age: Optional[int] = None

class HeroUpdate(HeroBase, all_optional=True):
    pass

Description
Is it possible to add the ability to modify the fields of a base class to convert them to all optional with a parameter like the table=True but in this case all_optional=True? This would help eliminate the duplication when creating a class for making updates.
Wanted Solution
Set all base fields to Optional based on keyword argument.
Wanted Code
class HeroBase(SQLModel):
    name: str
    secret_name: str
    age: Optional[int] = None

class HeroUpdate(HeroBase, all_optional=True):
    pass

Alternatives
No response
Operating System
Linux, Windows
Operating System Details
No response
SQLModel Version
0.0.4
Python Version
3.8.5
Additional Context
No response

This is pydantic-related. You can make optional fields required in child classes, but you cannot make required fields optional.
Maybe the following resources will help you:

https://github.com/samuelcolvin/pydantic/issues/1141#issuecomment-571247696
https://stackoverflow.com/a/65907609/3778854

You can use a pattern like this instead:
class HeroBase(SQLModel):
    # optional, aka nullable
    name: Optional[str]
    secret_name: Optional[str]
    age: Optional[int]

class HeroCreate(HeroBase):
    # all required
    name: str
    secret_name: str
    age: int

class HeroUpdate(HeroBase):
    pass

You can make optional fields required in child classes, but you cannot make required fields optional.

Perhaps this is not possible with class, however I think it is possible with type()?
In https://github.com/tiangolo/sqlmodel/issues/166 I link to a proposed method, make_updater_cls, that does this.
I am new to Pydantic + SQLModel so certainly welcome any feedback on this approach.

I really find interesting the idea of creating a schema that inherits all fields from a SQLModel  but make all of them Optional. Did you ever find a solution to this, @cisaacstern or @nickleman?

Also - it would be cool to have better control of which field should be optional and which not.
I was thinking about some kind of "decorator" mechanism to generate desired new class similar to this:
@Partial
class Hero(SQLModel):
    id: int
    @option
    name: str
    @option
    secret_name: str
    @remove
    age: Optional[int] = None

Above code should generate 2 classes, first one with name Hero and the second one, like this:
class HeroPartial(SQLModel):
    id: int
    name: Optional[str]
    secret_name: Optional[str]

@FilipeMarch apologies, I somehow missed your ping initially, and now a huge amount of time has passed. I have not thought about this issue in a while, but yes, the solution for a make_updater_cls method, proposed in my comment above, did work for me at the time.


Callback in form_validation not working

function verify () {
$this->form_validation->set_rules('user', lang('User'),'callback_check');
$this->form_validation->run()
}
public function check() {
return true;
}
Message: 'Unable to find callback validation rule: '

i me too with similar problem en ci 2.2 usinf HMVC
http://www.mahbubblog.com/php/form-validation-callbacks-in-hmvc-in-codeigniter/

Yes, I found the solution in that post. :dancers:


fix: Flash toggle should not be displayed when the quick view is visible

Fix for issue #4534
I have also added that if the quick view is visible, the back button should close it

merging and releasing @VaiTon @g123k
I created #4542 based on @VaiTon 's comment


Cross domain Video Play Issue

I have edit my /etc/hosts "<IP_ADDRESS> example.localhost" according to business logic. sub domain always needed to properly fetch all the required items from mongo, but I am stuck arround the images/video
because if i use .link() method it returns the url http://localhost:3000/cdn/storage/videos/uh2RbzwQ9PeAPm8yL/m3u8/uh2RbzwQ9PeAPm8yL.m3u8
where as i needed https://football.localhost:3000 to make the video to bypass the cross domain security issue

@dr-dimitru can you please share some information here

Hi @salmanhasni ,
Quickest solution for you is overwrite .link() method to:
const files = new FilesCollection({ /*...*/ });

const _origLink = files.link;
files.link = function (v) {
  return _origLink(v).replace(__meteor_runtime_config__.ROOT_URL, 'https://football.localhost:3000');
}

Where https://football.localhost:3000 can be stored as env.variable or in Meteor.settings
Let me know if it helps

Hi @dr-dimitru I will give it try today and will surely let you know. thanks for the quick response and heads up.

for the quick fix it worked, I extended your idea to make it more dynamic than to store all possible subDomains in settings.
_origLink.replace("://", "://" + subDomain + ".");

files.link = function (v) {
  return _origLink(v).replace(__meteor_runtime_config__.ROOT_URL, 'https://football.localhost:3000');
}

fantastic solution!
Big thanks!

I follow the stream files shared code, insert a video file and succeed and try play it on the page, but nothing play on the page, in the elements,
this is my storagePath: 'assets/app/uploads/files'.

I follow the stream files example code, insert a video and succeed, try play it on the page, but nothing displayed. In the Elements place src=http://localhost:3000/cdn/storage/files/PKgKqxPXLJaEKEQdt/original/PKgKqxPXLJaEKEQdt.mp4?play=true
my storagePath:'assets/app/uploads/files', this is the template code :src="{{file.link}}?play=true", what's wrong?

Hello @haibo32012 ,
Looks good to me.

Where do you store file FS or GridFS?
Does it throws any errors on Client or Server?
What "Network" tab displays, any network errors?

@dr-dimitru I test on local, what's your mean store file FS or GridFS, when I create collection have already set storagePath, is this not enough?

when I create collection have already set storagePath, is this not enough?

Enough.
What about?:

Does it throws any errors on Client or Server?
What "Network" tab displays, any network errors?

No Errors, Client or Server or Network.

I am still confuss when i upload a file, where the file is, what i say is on local FS, when i succeed upload a file, i  check the database and find it,i searched it on the my computer FS, can't find it.

i searched it on the my computer FS, can't find it.

Where have you checked?

No Errors, Client or Server or Network.

I'm still would like to see what network tab may tell us.
Could you post few screenshots?

First is client picture, Second is mongo database result.

Hi @salmanhasni ,
According to original issue:

try play it on the page, but nothing displayed

I need to see what server respond with when you're request file in a browser.
On screenshot - no request for uploaded file is sent from browser to server.
Please make sure:

Uploaded file is somehow requested from browser (with video/audio/img/etc. element)
"Headers" tab is open under "Network" tab

this is when i use simple stream demo code test, it can't find data.what I confuse is where  the data store when i  upload a file under local development enviroment.

Hello @haibo32012 ,
Well - 404 means file not found by library.
Could you locate uploaded file inside directory defined via storagePath?
And what your storagePath is?
If you will enable debug-mode via debug flag set to true and passed into FilesCollection constructor - it will help a lot to solve your issue. Please, enable it and post full logs from Server and Client Console here.
Thank you.

thanks, I have figured out what's going on, the storeagePath set the wrong place, thanks so much for your patience.

@haibo32012 I'm glad it helped. Do not hesitate to ask more.

instead of replacing original link I found a bit hacky general way to fix the cross domain issue
// Listen to incoming HTTP requests, can only be used on the server
http://www.something.com/path/to/intercept
WebApp.rawConnectHandlers.use("/path/to/intercept", function(req, res, next) {
  //define white list of urls, or can be done like adding subdomain prefixes before Meteor.absoluteUrl();
  let whiteList = [];
  const found = _.find(whiteList, function(url) {
    return url == req.headers.origin;
  });
  if(found){
    res.setHeader("Access-Control-Allow-Origin", req.headers.origin);
  }
  return next();
});

@salmanhasni ,
You're welcome to send PR with this snippet to docs :)


Superscript and subscript

There needs to be a way to superscript and subscript text. Perhaps wrap things in ^.

I think we'll go with ^superscript^ and ^vsubscript^


Replace anyhow with structured errors

The issue at https://github.com/ipld/libipld/issues/115 contains a discussion about error handling. This PR is an experiment in using structured, distinct errors instead of anyhow.
I mark it as draft, as I hope to get feedback. This is really just and idea, if someone has other thoughts, please post them here.
This PR might be easier to review with its individual commits.

Thanks @Stebalien for the review. I've updated the PR. Should the Conversion error type also be non-exhaustive?
It sounds like you're generally OK with having 3 different error types instead of a single catch-all enum, is that right?

It sounds like you're generally OK with having 3 different error types instead of a single catch-all enum, is that right?

Absolutely!

IMO, we should default to non-exhaustive everywhere.

Of course, unless we're sure that it's actually exhaustive...

@Stebalien Please have another look. I agree, the structured AccessError makes things better.
I'll have a look at the Serde error tomorrow, whether there's a good way to make it also more structured.

Ok, I'm just quibbling at this point so I'm going to approve this.

No. Your feedback is really welcome. I want to get this done properly. So please keep going.
I've addressed your points, I think it's good to go now (I know you already approved it, but another re-approval would be appreciated).

I've moved back to having the SerdeError as newtype struct wrapping a String. The only additional thing is that I fixed tests that were broken due to earlier changes.
I merge this PR as it was already approved. In case there's disagreement on changes later on, things can still be changed before a new release it cut.


Compile error with GS project

@MariannaPaszkowska identified the following compiler error today
fontc 484450e970dbdf36a2229ed7198976b95dd576be
rustc 1.72.1 (d5c2e9c34 2023-09-13)
macOS Monterey v12.6.7
GS main branch commit
cargo run --package fontc -- --source /Users/…/GitHub/googlesans/source/GoogleSans/GoogleSans.designspace
    Finished dev [unoptimized + debuginfo] target(s) in 1.70s
     Running `target/debug/fontc --source /Users/…./GitHub/googlesans/source/GoogleSans/GoogleSans.designspace`
thread '<unnamed>' panicked at 'assertion failed: `(left == right)`
  left: `Space(0)`,
 right: `Space(1)`', /Users/…/.cargo/registry/src/index.crates.io-6f17d22bba15001f/write-fonts-0.18.0/src/graph.rs:797:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[2023-11-09T13:21:38.379561Z main fontc::workload ERROR] Be(Features) failed Job panicked: 'assertion failed: `(left == right)`
  left: `Space(0)`,
 right: `Space(1)`'
At least one work item failed

Ty for the detailed report; I believe this is fixed by #560


Plugin automatically creates "library roots" which kill performance

Greetings,
After a long debugging session with JetBrains' support, we've deduced that this plugin is creating library roots out of my projects node_modules and bower_components directories. I'm not sure what features this powers within the plugin, but it makes performance in any non-trivial project completely unusable.
I can try to provide as much information as I can on request, though I'm not sure what might be relevant at this time. I have a very complex Ember 1.x app (we're in the process of updating!), and would like to use this plugin, but I simply can't in its current state.

@k3n this sounds like a duplicate of #24

Yup! Thanks, I'll close this one in favor of that one.

@k3n if you got a bit of time feel free to help with #24

I left a comment, but I'm not sure what else I can offer -- I'm still pretty green to Ember, and have never done any Idea plugin work.


In the desktop view it shows scroll bar

When we see a list of items in paper-items in desktop view. It shows a scroll  bar in the right side which is not expected. But it is okay with mobile view.

@codersfriend You mean in the demo?

I used your paper-swipe elements in my custom element. After that when I was viewing it in the desktop view it is showing scroll bar in the right side.
here is the code below.
`

          <paper-item content>
            <paper-item-body two-line>

[[item.fileName]]

[[item.modifiedDate]]

            </paper-item-body>
            <paper-icon-button  icon="icons:chevron-right" alt="View the config" class="icon-sensea-color" on-click="_onEditButtonClick"></paper-icon-button>
          </paper-item>
        </paper-swipe>
      </template>

`

Solved, it was the main div issue.

Closed as it's solved.


cant connect for some reason. was working before...

Options:
Command prefix: !
Default volume: 30%
Skip threshold: 4 votes or 50%
Now Playing @mentions: Disabled
Auto-Summon: Enabled
Auto-Playlist: Disabled
Auto-Pause: Enabled
Delete Messages: Disabled
Debug Mode: Disabled
Downloaded songs will be deleted
Attempting to autojoin Music bot in Zxanth's Mountain Hangout
Attempting connection...
Traceback (most recent call last):
File "/home/pi/MusicBot/musicbot/bot.py", line 256, in get_voice_client
await asyncio.wait_for(voice_client.connect(), timeout=10, loop=self.loop)
File "/usr/local/lib/python3.5/asyncio/tasks.py", line 382, in wait_for
raise futures.TimeoutError()
concurrent.futures._base.TimeoutError
Failed to connect, retrying (1/3)...
Attempting connection...
Traceback (most recent call last):
File "/home/pi/MusicBot/musicbot/bot.py", line 256, in get_voice_client
await asyncio.wait_for(voice_client.connect(), timeout=10, loop=self.loop)
File "/usr/local/lib/python3.5/asyncio/tasks.py", line 382, in wait_for
raise futures.TimeoutError()
concurrent.futures._base.TimeoutError
Failed to connect, retrying (2/3)...
Attempting connection...
Traceback (most recent call last):
File "/home/pi/MusicBot/musicbot/bot.py", line 256, in get_voice_client
await asyncio.wait_for(voice_client.connect(), timeout=10, loop=self.loop)
File "/usr/local/lib/python3.5/asyncio/tasks.py", line 382, in wait_for
raise futures.TimeoutError()
concurrent.futures._base.TimeoutError
Failed to connect, retrying (3/3)...
Failed to join Music bot
I have tried doing
git pull
python3.5 -m pip install -U -r requirements.txt
and still nothing

Sometimes i've this issue, and sometimes that's because i've more than one bot connected

its odd though, i've done a hard reset and remade them (via the wiki steps). i've only tried the one bot....

This should be less of an issue in recent updates to the bot.


Allow using a different config file name from package name

As we discussed at FlatCamp it seems the best option is to add another parameter and update the navigation to use a new handle parameter that is then used by Forma to find the file.

+1 for this feature. Would be good for when addons have their name changed but retain the original addon handle.
e.g. I recently renamed SendInBlue for Statamic to Bravo for Statamic but had to retain the configuration file name as sendinblue.php

@vmitchell85 @MeMattStone give this a try: https://github.com/edalzell/statamic-forma/pull/20

Need to re-open because version 3.1 broke things, will try again in a bit.

Anything specific I can help test?


[BUG]: Possible Resource leak

Link to faulty endpoint
N/A
The name of your client/application
Self-Hosted API
Request information
CPU and RAM usage seem to be increasing over time (up to 50% CPU and 500MB RAM after around 2 weeks)

^ on startup

^after 24 hours

^ on startup

^ after 24 hours
Steps to reproduce

RUN API
?????
Usage goes up

I'm not 100% on the process of setting it up
What is expected?
CPU and RAM levels to stay stable
What is actually happening?
CPU and RAM usage increases over time (CPU seemingly capped at around 50%)

System Info
RAM: 8GB
CPU: AMD EPYC-Milan Processor (2C\4T) (as much info as we can get)

Any additional comments?
Since this is an old version of the API (still V1) we'll update it before proceeding.

I kept the API running for about a day and it didn't increase too much.
So it's probably not related to the synchronization but the API responding to endpoint requests.
This narrows it down

I have the API running as a Docker container, with another container requesting almost every endpoint sequentially, every 10mins.
Resource usage of both containers is stable.
If required for further investigation, I could reduce the time between sequential API requests...?

I've profiled the application but memory usage didn't increase beyond reasonable growth (aka hitting the endpoint with 100 concurrent users creates more memory, but that's because it's servicing a higher load).
@Stonemercy did mention he's running an older build, perhaps the issue was already resolved with one of the later updates?

After running an updated version of the API for 2 days and having it idle at basically 1%, I think it's safe to assume it was an issue with the previous version we were running on which has since been fixed.
Happy to close this unless you want to investigate more 👍🏻

I'll close this, but should we run into resource leaks in the future feel free top open a new bug :)


Report per project failing tests

Description:
Adjust changes from #166 to allow inclusion of each test once per project.
Related issue:
Only the comment here
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

I just tested that without the above changes then any tests that only fails in one project may not get reported at all - if the same tests passes in a later project. If I add something like:
  // fail only in firefox:
  if (browserName === 'firefox') {
    await expect(page.locator('.does-not-exist')).toBeVisible();
  }

And the firefox project is run before other passing tests, then the failed test is not included in summaryResults.failures  at all 🔥


[FLINK-27451] Enable the validator plugin in webhook

@SteNicholas Thanks for your comments, I have addressed your comments, please take a look again.


Cleanups

Mostly cosmetic changes. There's some intersection with #390, so I tried to keep them non-overlapping.
A few questions to consider:

Maybe it's better to use ChannelInitializer from Netty instead of manually defined pipeline-initializer handler? In case of using own handler here, we probably need to cover exceptions here (as at some point in time it's a single handler in the whole pipeline).

Do we really need Throwable here? Maybe relax to Exception?

Should this be removed? I don't see it's used somewhere.

Using Exception rather than Throwable will ignore assert failures, among other things.  If we can't be more specific than Exception, I think using Throwable is almost always preferable.

1, 3. Will do!

Got it. AssertionError is a good point, missed that.

@ztellman Okay, #3 is done. I will create a separate PR with ChannelInitializer instead of pipeline-initializer as it's a more of a standalone feature than just a "cleaning up" work.


Unknown Configuration Setting: tab.lastPinnedBorder

VSCode Version: 1.50 - Stable
OS Version: OSX Catalina 10.15.7

It appears the new configuration option tab.lastPinnedBorder that is meant to be included with the latest 1.50 release is not actually available to set, with a tooltip warning appearing when hovering over the new configuration reading "Unknown Configuration Setting".
See annotated screenshot below:

Steps to Reproduce:

Open the VS Code Command Palette
Search for and open Preferences: Open Settings (JSON)
Include the new configuration option tab.lastPinnedBorder, and save. (eg, "tab.lastPinnedBorder": "#fffca6")

Does this issue occur when all extensions are disabled?: Yes

It's a color. It should be inside workbench.colorCustomizations setting.

It's a color. It should be inside workbench.colorCustomizations setting.

@usernamehw thanks, that did the trick!
Here's an example below for reference:
"workbench.colorCustomizations": {
    "tab.lastPinnedBorder": "#fffca6"
},

Will close the ticket.


Dragonbane: version 1.1

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Bug Fixes
General Fixes

Numerous fixes to responsive elements so that they are properly sized across screen sizes
Numerous color fixes so that dark mode is legible
Fixed some incorrect font usage

PC Sheet

The rest checkboxes now actually roll for recovered HP/WP.

Also prompted for whether you are being cared for on a stretch rest

HP/WP are now calculated correctly
Current and max encumbrance are now calculated
Succeeding on three death rolls now resets the tracker and rolls for hp
Secondary skills should now be addable
Fixed a display issue in Firefox where the dragonbane header did not wrap correctly
Weapon skills should now calculate correctly
Dropping an NPC should now default their skill chance to a 5

Monster Sheet

Fixed an error that caused monsters with subtables to drop with their subtable not expanded
Fixed an error when rolling for monster attacks that caused subtable to not be output correctly

New Features
PC Sheet

Adds the ability to mark spells as "prepared"
Spells can now be cast at a dynamic power level (prompted for power level at roll time)
You can now specify an amount of bonus hp/wp that your character has (located in the settings page)

Monster Sheet

Monsters now have skills!
Monster skills will default to 15 unless a monster has a specific spell value
Specific monster skills are hidden in view mode unless they are a value other than 15

Character Sheet Info Roll20 Internal Use only.


feat: improve headless typings

Closes #1643

Replaced incorrect usage of HtmlHTMLAttributes with HTMLAttributes: HTMLAttributes is the correct supertype, while HtmlHTMLAttributes actually refers to attributes of an html element.
implemented createElRef, which creates a special writeable computed that references a DOM Element and unwraps components.
createElRef returns a HeadlessElRef, which is a special subtype of WritableComputedRef. It uses a type differentiator, so no other Ref types can be assigned to it.
updated createBuilder HeadlessElement typings to only accept HeadlessElRef for ref keys. So createElRef must be used to create these.

Can you add a breaking change changeset for your changes?

Done


error

Command that caused the error
&meme
Image that caused the error
https://media.discordapp.net/attachments/834923061413150723/925831010879410226/illust_95113601_20211229_181102.jpg
Steps to reproduce
replied to an image I tried to meme when the bot was offline
copied the command I tried to run and pasted it in the reply
bot gave me the error with the file telling me I should report it
Self-hosted instance?
Yes
Error file
in error.txt I got:
Message: request to https://media.discordapp.net/attachments/834923061413150723/925831010879410226/illust_95113601_20211229_181102.jpg failed, reason: getaddrinfo EAI_AGAIN media.discordapp.net
Stack Trace: undefined
Additional context
No response

I only get this error if I type spicificly "&meme you should kill yourself,now" I tried the &meme command on other images and it seems to be working just fine

Seems like a network issue, considering you're using a self-hosted version of the bot. I'd suggest checking the network connection of the server you're running it off of.


Fix link time error reintroduced by previous commit.

Everything's in the title.

Thanks, sorry about that...!

Well, unless Travis can also automate builds for Windows (I doubt it can), it can't be helped anyway.


Use the local_files_only option available as of fastembed==0.2.7. It …

fastembed has a new feature "local_files_only" as of version 0.2.7 (see https://github.com/qdrant/fastembed/issues/229). It allows to use the model files that were already cached without looking online if there is a new version.
This change allows to optionnaly use this feature on the fastembed embedders (FastembedTextEmbedder, FastembedDocumentEmbedder,  FastembedSparseTextEmbedder, FastembedSparseDocumentEmbedder).

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

@paulmartrencharpro thanks for the contribution!
I will have a proper look later.
In the meantime:

please sign the CLA
fix linting errors: you can try fixing them automatically with hatch run lint:fmt

Hello,
CLA signed & lint fixed
Thanks

This is not working for me, it still attempts to download from huggingface, even with local_files_only=True, and cache_dir set to where the model files are. We have to approve models first, therefore no direct HF downloading, why does it still reach out?? Using 'prthivida/Splade_PP_en_v1' with FastembedSparseDocumentEmbedder.

Hi @satyaloka93,
Are you using fastembed >= 0.2.7? We did not update the minimum version of fastembed with this PR.

Hi, yes I confirmed as I also installed qdrant-client[fastembed]. I'm wondering if the issue is because it expects to see the huggingface cache structure instead of just the model files? I see the local_files_only is referenced in both haystack integrations and fastembed, I tried loading model with fastembed directly as well as the haystack implementation, it attempts to download from hugginface, which is blocked by my development container.

:wave:
@satyaloka93 it is better to open a new issue to discuss this bug.
Comments on merged PRs are likely to go unnoticed.

Thanks @anakin87 , I think this might be more pertinent for qdrant/fastembed, so I made this PR - https://github.com/qdrant/fastembed/issues/321. Let me know if I should raise one here as well.


Added BarChart et LineChart + Added colors, labels, colorMapping and labelMapping properties

Keen.io released two new charts : BarChart and LineChart (https://github.com/keen/keen-js/blob/master/docs/visualization.md#selective-chart-types)
Moreover, there is also the possibility to customize several elements :

labels (https://github.com/keen/keen-js/blob/master/docs/visualization.md#customized-labels)
colors (https://github.com/keen/keen-js/blob/master/docs/visualization.md#customized-colors)

This pull request adds all this to your plugin.
Best regards

Very nice - thank you! Merged.

Cool !
I think you will need to create a new release for bower to provide an update.
Best regards

Yes - I'll do that next week when back home from vacation.
Thank you!


[Flutter_Build_iOS] success on 2.10.3 and fail on 3.0.1

Hello
we are trying to update our flutter env , currently using 2.5.3.
we have successfully ran flutter 2.10.3 but we fail on 3.0.1 with this exception:
2022-05-22 14:49:59.472 xcodebuild[64808:11482349] Requested but did not find extension point with identifier
Xcode.IDEKit.ExtensionSentinelHostApplications for extension Xcode.DebuggerFoundation.AppExtensionHosts.watchOS of plug-in
com.apple.dt.IDEWatchSupportCore
2022-05-22 14:49:59.472 xcodebuild[64808:11482349] Requested but did not find extension point with identifier
Xcode.IDEKit.ExtensionPointIdentifierToBundleIdentifier for extension Xcode.DebuggerFoundation.AppExtensionToBundleIdentifierMap.watchOS of plug-in
com.apple.dt.IDEWatchSupportCore
there might be problems with compatability to watchos 8.5?
using :
xcode 13.4
flutter 3.0.1
pods 1.11.3
can you please provide some insight on that? Thank you!

Same issue for me.

Same here, the app works 2.10.5 but not in 3.0.1

To bring some precision: in my case, it has been working with Flutter 3 during a few days, I think it stopped working when Xcode updated to 13.4.

@grisvladko
The log you provided earlier doesn't seem to be complete. Please provide entire verbose log in properly formatted text so that we can take a look at what's going on.

Failed to build iOS app
Error output from Xcode build:
↳
2022-05-23 11:21:29.078 xcodebuild[96515:11740175] Requested but did not find extension point with identifier
Xcode.IDEKit.ExtensionSentinelHostApplications for extension Xcode.DebuggerFoundation.AppExtensionHosts.watchOS of plug-in
com.apple.dt.IDEWatchSupportCore
2022-05-23 11:21:29.079 xcodebuild[96515:11740175] Requested but did not find extension point with identifier
Xcode.IDEKit.ExtensionPointIdentifierToBundleIdentifier for extension Xcode.DebuggerFoundation.AppExtensionToBundleIdentifierMap.watchOS of plug-in
com.apple.dt.IDEWatchSupportCore
** BUILD FAILED **
Xcode's output:
↳
Writing result bundle at path:
/var/folders/rp/fcvyjnf158l6ncn927tfbr4m0000gn/T/flutter_tools.vjkhgV/flutter_ios_build_temp_dirq7ZYhb/temporary_xcresult_bundle
lib/widgets/ez_count/ez_count_product_page/product_page_content.dart:59:22: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which
excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../development/flutter/packages/flutter/lib/src/widgets/binding.dart').
      WidgetsBinding.instance!
                     ^
lib/widgets/currency/currency_transfers/transfer_summary.dart:249:23: Warning: Operand of null-aware operation '!' has type 'String' which excludes null.
                if (!(toolTipTxt!.contains("null")))
                      ^
lib/common_widgets/ToolTipWidget.dart:1025:20: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../development/flutter/packages/flutter/lib/src/widgets/binding.dart').
    WidgetsBinding.instance!.addPostFrameCallback((_) {
                   ^
lib/widgets/currency/currency_amount_selection/currency_amount_selection_content.dart:260:60: Warning: Operand of null-aware operation '??' has type
'String' which excludes null.
                              GeneralErrorPopUpWidget(msg: error ?? "", img: "assets/declined_icon.png"));
                                                           ^
libraries/searchable_dropdown/lib/dropdown_search.dart:546:26: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../development/flutter/packages/flutter/lib/src/widgets/binding.dart').
          WidgetsBinding.instance!.addPostFrameCallback((_) {
                         ^
libraries/searchable_dropdown/lib/dropdown_search.dart:578:26: Warning: Operand of null-aware operation '!' has type 'WidgetsBinding' which excludes null.
 - 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../development/flutter/packages/flutter/lib/src/widgets/binding.dart').
          WidgetsBinding.instance!.addPostFrameCallback((_) {
                         ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selectable.dart:459:24: Warning: Operand of
null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!.addPostFrameCallback((_) {
                       ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay_manager.dart:138:28: Warning:
Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
      if (SchedulerBinding.instance!.schedulerPhase ==
                           ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay_manager.dart:140:26: Warning:
Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
        SchedulerBinding.instance!
                         ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/web_selection_manager.dart:119:24: Warning:
Operand of null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!
                       ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay.dart:141:26: Warning: Operand of
null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
    if (SchedulerBinding.instance!.schedulerPhase ==
                         ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay.dart:143:24: Warning: Operand of
null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!.addPostFrameCallback(_markNeedsBuild);
                       ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay.dart:193:26: Warning: Operand of
null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
    if (SchedulerBinding.instance!.schedulerPhase ==
                         ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_math_fork-0.5.0/lib/src/widgets/selection/overlay.dart:195:24: Warning: Operand of
null-aware operation '!' has type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../development/flutter/packages/flutter/lib/src/scheduler/binding.dart').
      SchedulerBinding.instance!.addPostFrameCallback(_markNeedsBuild);
                       ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:554:31: Error: Member not found: 'UnicodeChar'.
  int get UnicodeChar => Char.UnicodeChar;
                              ^^^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:555:38: Error: Setter not found: 'UnicodeChar'.
  set UnicodeChar(int value) => Char.UnicodeChar = value;
                                     ^^^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:557:29: Error: Member not found: 'AsciiChar'.
  int get AsciiChar => Char.AsciiChar;
                            ^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:558:36: Error: Setter not found: 'AsciiChar'.
  set AsciiChar(int value) => Char.AsciiChar = value;
                                   ^^^^^^^^^
Failed to package /MYPACKAGEPATH.
Command PhaseScriptExecution failed with a nonzero exit code
note: Using new build system
note: Planning
note: Build preparation complete
note: Building targets in dependency order

Result bundle written to path:
    /var/folders/rp/fcvyjnf158l6ncn927tfbr4m0000gn/T/flutter_tools.vjkhgV/flutter_ios_build_temp_dirq7ZYhb/temporary_xcresult_bundle

Encountered error while building for device.

Same issue

@grisvladko
Thanks for the log. From it, it seems you and others in this thread are hitting https://github.com/flutter/flutter/issues/103561. Some of the warnings are coming from community packages for which take a look at https://docs.flutter.dev/development/tools/sdk/release-notes/release-notes-3.0.0#if-you-see-warnings-about-bindings and https://docs.flutter.dev/development/tools/sdk/release-notes/release-notes-3.0.0#dependencies for various solutions.
Closing this per issue linked above.

@darshankawar  I did dart fix --apply  and I don't have any "SchedulerBinding" functions in my app.
I tested both 3.0.0 and 3.0.1, the app doesn't works in them.  The app still can't build in iOS or Android.
My app works well in 2.10.5

@darshankawar Thank for the references, though it is not necesserely a duplicate, because i report a build failure while the issues address warnings. wernings don't prevent you from flutter build ios to succeed, but i can't build my app for ios.
you suggest that this is a problem in our bindings which are coused by the change of :
[COPIED FROM FLUTTER WEBSITE : https://docs.flutter.dev/development/tools/sdk/release-notes/release-notes-3.0.0#if-you-see-warnings-about-bindings]
These are caused by a simplification of the API (the instance property on bindings is now non-nullable), combined with an eager compiler that wants to report any case where redundant null-aware operators (such as ! and ?.) that are used when they’re not necessary

Thanks for the feedback. Reopening.

../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:554:31: Error: Member not found: 'UnicodeChar'.
int get UnicodeChar => Char.UnicodeChar;
^^^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:555:38: Error: Setter not found: 'UnicodeChar'.
set UnicodeChar(int value) => Char.UnicodeChar = value;
^^^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:557:29: Error: Member not found: 'AsciiChar'.
int get AsciiChar => Char.AsciiChar;
^^^^^^^^^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/win32-2.3.1/lib/src/structs.g.dart:558:36: Error: Setter not found: 'AsciiChar'.
set AsciiChar(int value) => Char.AsciiChar = value;

These seem to be the errors from the log which are affecting your build. Again, these seem to be coming from third party package win32-2.3.1 for which there was a similar issue and flutter team member closed it per this comment.
You may try this solution and this explaination as why it need to be reported in dedicated repo.
Check if that helps.

we will check out the dependencies and see how upgrading them slowly will effect the project, for now we are staying on 2.10.5 so thank you for the help. if this keeps happening without any solution i will file another issue or ask to reopen this one. Thank you for the provided help.

Thanks for update. Based on your comment and the one I shared earlier for reasoning, I'll close it for now.


Make it a kubectl plugin

Hi, this is not a real issue but it would be nice to be able to install this tool via krew as a kubectl plugin, I think that would also help users discovering this tool.

Good idea. Can't commit to a timeline on this at the moment, but if anyone wants to contribute it would be welcome!


Invalid JSON response: "Unexpected token U in JSON at position 0"

From what I understand, Ghost is using the oembed-parser to allow users to embed videos in their blog posts. I tried to embed a Youtube video in one of my blog posts today and encountered the issue described in Ghost issue 10624.
Youtube seems to deliver a response the parser is not expecting. The whole error message I receive is:
{
  "err": {
    "code": null,
    "domain": "https://mydomain.tld",
    "id": "95093690-a7bc-11e9-8a98-bdc933ed02ad",
    "level": "critical",
    "message": "invalid json response body at https://www.youtube.com/oembed?format=json&url=http%3A%2F%2Fyoutube.com%2Fwatch%3Fv%3Dcc-elFcs96Y reason: Unexpected token U in JSON at position 0",
    "name": "InternalServerError",
    "stack": "InternalServerError: invalid json response body at https://www.youtube.com/oembed?format=json&url=http%3A%2F%2Fyoutube.com%2Fwatch%3Fv%3Dcc-elFcs96Y reason: Unexpected token U in JSON at position 0\n    at new InternalServerError (/var/www/ghost/versions/2.25.7/node_modules/ghost-ignition/lib/errors/index.js:77:23)\n    at extract.catch (/var/www/ghost/versions/2.25.7/core/server/api/v2/oembed.js:62:43)\n    at tryCatcher (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/util.js:16:23)\n    at Promise._settlePromiseFromHandler (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/promise.js:517:31)\n    at Promise._settlePromise (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/promise.js:574:18)\n    at Promise._settlePromise0 (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/promise.js:619:10)\n    at Promise._settlePromises (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/promise.js:695:18)\n    at _drainQueueStep (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/async.js:138:12)\n    at _drainQueue (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/async.js:131:9)\n    at Async._drainQueues (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/async.js:147:5)\n    at Immediate.Async.drainQueues (/var/www/ghost/versions/2.25.7/node_modules/bluebird/js/release/async.js:17:14)\n    at runCallback (timers.js:810:20)\n    at tryOnImmediate (timers.js:768:5)\n    at processImmediate [as _immediateCallback] (timers.js:745:5)",
    "statusCode": 500
  },
  "hostname": "<redacted>",
  "level": 50,
  "msg": "invalid json response body at https://www.youtube.com/oembed?format=json&url=http%3A%2F%2Fyoutube.com%2Fwatch%3Fv%3Dcc-elFcs96Y reason: Unexpected token U in JSON at position 0",
  "name": "Log",
  "pid": 23343,
  "req": {
    "headers": {
      "accept": "application/json, text/javascript, */*; q=0.01",
      "accept-encoding": "gzip, deflate, br",
      "accept-language": "en,en-US;q=0.9,de;q=0.8",
      "app-pragma": "no-cache",
      "connection": "close",
      "content-type": "application/json; charset=UTF-8",
      "cookie": "**REDACTED**",
      "dnt": "1",
      "host": "mydomain.tld",
      "referer": "https://mydomain.tld/ghost/",
      "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.100 Safari/537.36",
      "x-forwarded-for": "<IP_ADDRESS>",
      "x-forwarded-proto": "https",
      "x-ghost-version": "2.25",
      "x-real-ip": "<IP_ADDRESS>",
      "x-requested-with": "XMLHttpRequest"
    },
    "meta": {
      "requestId": "894d5321-e801-4f2c-930f-d0f257460b77",
      "userId": "1"
    },
    "method": "GET",
    "originalUrl": "/ghost/api/v2/admin/oembed/?url=https%3A%2F%2Fwww.youtube.com%2Fwatch%3Fv%3Dcc-elFcs96Y",
    "params": {},
    "query": {
      "url": "https://www.youtube.com/watch?v=cc-elFcs96Y"
    },
    "url": "/oembed/?url=https%3A%2F%2Fwww.youtube.com%2Fwatch%3Fv%3Dcc-elFcs96Y"
  },
  "res": {
    "_headers": {
      "cache-control": "no-cache, private, no-store, must-revalidate, max-stale=0, post-check=0, pre-check=0",
      "content-length": "386",
      "content-type": "application/json; charset=utf-8",
      "etag": "W/\"182-+So45N9SFD0nfP2kp+OfniY4gac\"",
      "vary": "Accept-Encoding",
      "x-powered-by": "Express"
    },
    "responseTime": "182ms",
    "statusCode": 500
  },
  "time": "2019-07-16T11:26:40.769Z",
  "v": 0
}

As far as I could reproduce, the issue is not with all videos, quite the contrary. So far I encountered the issue just with this one video. This video, for example, works fine.

@jbfriedrich thank you

@jbfriedrich hello, after several test I always get no problem with the video you provided (cc-elFcs96Y).

Yes, a day after I filed the report, Youtube did not show the behavior anymore. It seems that this happens to videos only sometimes, and after a while the issue is gone – on its own.

@jbfriedrich yes, it looks like something went wrong at your test, caused invalid JSON response. Let's me know if you get it again. Thanks.


404 while running gk login on GitHub Enterprise instance

Hi,
we're using Greenkeeper Enterprise with GitHub Enterprise. Whenever I try to run greenkeeper login and I login as a user I want to configure webhook as, I'm redirected to 404 page on GH.
Mentioned user is a part of a team which has admin rights to the repository.

I will close this and write to<EMAIL_ADDRESS>instead.

For posterity, this was an oauth setup issue.


fix: azure tests are replaced to the test/azure folder

test files under modules/azure folder are re-placed to test/azure folder

Looks good! You also need to update the imports for test/azure/compute_test.go and test/azure/region_test.go

yep, I just pushed a commit to add azure package to the imports


[Event Hub] Buffered send API review

Setup arch board review for buffered send APIs in Event Hubs

Review done: https://github.com/Azure/azure-sdk/issues/1259


https://github.com/restic/restic/blob/master/VERSION needs updating

After installing brew install --HEAD restic, I get:
$ restic version
restic 0.17.2-dev (v0.17.2-257-g408ec41a1) compiled with go1.23.3 on darwin/arm64

It should use 0.17.3, not 0.17.2. I think you forgot to update https://github.com/restic/restic/blob/master/VERSION.

It's all in the patch-release branch and will be merged to master shortly. Thanks for keeping track on things :)


How to document a lane? Is there a better mechanism than desc "some description here"

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Is there a mechanism in place so that we can use --help at the end of the lane call in order to see what options can be passed?
For example:
lane :something do |options|
  flag1 = options[:flag1]
  string2 = options[:string2]
end

The lane above would be called as such:
$ bundle exec fastlane something --help

And I'd want to see something like:
Options for build:
    flag1, BOOL This flag is used to determine something or other
    string2, STRING This is the name of the project or something

The only way I've been able to do this is by using some desc commands above the lane. but this is just messy, has to be maintained manually and can only be viewed when you don't pass a lane to the fastlane command:
desc "    flag1, BOOL This flag is used to determine something or other"
desc "    string2, STRING This is the name of the project or something"
lane :something do |options|
  flag1 = options[:flag1]
  string2 = options[:string2]
end

Great idea @bsarrazin, thanks for sharing this. For now that's not something we plan on building. If you feel like it, you can contribute to fastlane add add this functionality, however it is important to not add any complexity for the user 👍


Pluggable transports

TCP and UTP are great, but it should be easy to add more transport types. This will be really useful for seamlessly adding the ability to route your traffic through mixnets like Tor or I2P.

Support an API for adding transports
Tor?
I2P?
libp2p?

What should a transport be able to do?

connect(port, host)
listen(port)
get data to announce on the DHT?

I think before committing work to pluggability, we should think through how many transports could really be added this way. If the answer is less than, like, five, then we may be better off just building in those transports and making them option flags. Pluggability adds interface complexity by being more abstract, and is harder to implement overall.

I haven't used this yet, but node-i2p has a very similar API surface to net. Maybe just shoving that in and seeing if it works would be a good start?

1 for webRTC

Some clients are also likely to be behind public wifi that has a firewall that only allows local DNS traffic, http, https. It would be great if some of the discovery servers / DHT could work over http/https, so these clients can still discover other http/https endpoints.
Supporting HTTPS connections could also allow hyperswarm traffic to blend in and work across even the most restrictive corporate firewalls, because corporates can’t ban public cloud traffic eg AWS/Azure without breaking their own systems.

@karissa, @pvh and I were thinking it'd be cool if we could eventually integrate discovery-swarm-stream servers into the discovery layer for cases when you need a proxy to the p2p network. But that's probably a ways away and likely won't be within hyperswarm directly.


Recent dataset page 404

Not sure if this is the correct place to inquire, but the link for the Licenses and Inspections Building Permits page at the URL: https://opendataphilly.org/datasets/licenses-and-inspections-building-permits is resulting in a 404 today (07-29-2024).  I check this page weekly and pull a table of permits and noticed this week (not prior weeks) it lands on a 404.  Is this dataset not available anymore?  If not is there any indication as to why it is not longer available and is there a comparable replacement set?  Any insight would be appreciated.

is it this page? https://opendataphilly.org/datasets/licenses-and-inspections-building-and-zoning-permits/

That is the page.  It was definitely giving a 404 yesterday, I checked from several devices (windows and linux), two local and one in a data center and got the 404.  Thanks for following up Bryan.

@BryanQuigley should we look into plugins for aliases? i think jekyll doesn't have aliases, but it would be nice to be able to slot these in as they come up
https://github.com/jekyll/jekyll-redirect-from

Maybe I spoke to soon.  The data being returned on the query links (such as [L&I Building & Zoning Permits - 2016 to present (CSV)]) only shows data up to mid March 2020.  It was definitely showing data from 2024 on 07-21-2024.

@ctchadwick I tried as well and I'm confirming that there is no recent data anymore.  Please send an email to<EMAIL_ADDRESS>and I'll forward to a couple of people at the City with cc to you.

Opened a new data issue at #263


We should use algorithm from public key rather than alg from token in verification

In the code below, we use the alg set in the ID Token to verify the ID Token. This does not present a security risk because we also check it is the expected algorithm and for GQ256 we need to do this. However we should check that public key in pubKeyRecord.PublicKey is RS256.
	algStr := idt.GetSignature().GetProtectedClaims().Alg
	if algStr == "" {
		return fmt.Errorf("provider algorithm type missing")
	}
	alg := jwa.SignatureAlgorithm(algStr)
	if alg != gq.GQ256 && v.options.GQOnly {
		return fmt.Errorf("non-GQ signatures are not supported")
	}

	switch alg {
	case gq.GQ256:
		if err := v.verifyGQSig(ctx, idt); err != nil {
			return fmt.Errorf("error verifying OP GQ signature on PK Token: %w", err)
		}
	case jwa.RS256:
		pubKeyRecord, err := v.providerPublicKey(ctx, idToken)
		if err != nil {
			return fmt.Errorf("failed to get OP public key: %w", err)
		}
		alg := jwa.SignatureAlgorithm(algStr)
		if _, err := jws.Verify(idToken, jws.WithKey(alg, pubKeyRecord.PublicKey)); err != nil {
			return err
		}
	}

I would like to work on this. Are you talking about this file and line of it?

@thebigbone That is the line and anywhere else you find it. We should always be checking that alg we are verifying against matches algorithm of the public key downloaded from the JWKS.
Glad to have your help =)

@thebigbone That is the line and anywhere else you find it. We should always be checking that alg we are verifying against matches algorithm of the public key downloaded from the JWKS.

Got it.

Glad to have your help =)

Happy to help!

@thebigbone If you want to talk through any of this feel free to drop by the OpenPubkey slack channel on the OpenSSF slack. I'm usually around http://slack.openssf.org/


[SPARK-24365][SQL] Add Data Source write benchmark

What changes were proposed in this pull request?
Add Data Source write benchmark. So that it would be easier to measure the writer performance.

Test build #91030 has started for PR 21409 at commit 11d612a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3502/
Test PASSed.

Test build #91033 has started for PR 21409 at commit 0f0908e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3504/
Test PASSed.

Test build #91030 has finished for PR 21409 at commit 11d612a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91030/
Test FAILed.

Test build #91033 has finished for PR 21409 at commit 0f0908e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91033/
Test FAILed.

Test build #91043 has started for PR 21409 at commit abf3837.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3512/
Test PASSed.

Test build #91043 has finished for PR 21409 at commit abf3837.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91043/
Test FAILed.

Merged build finished. Test FAILed.

Test build #91237 has started for PR 21409 at commit 8ffba61.

Test build #91237 has finished for PR 21409 at commit 8ffba61.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91237/
Test PASSed.

Test build #91269 has finished for PR 21409 at commit e90fa00.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91269/
Test PASSed.

thanks, merging to master!


[rebranch][cxx-interop] fix MSVC testcase check line for indirect return

@swift-ci please test

yeah I think so, I can double check if this also works on main too locally


View statistical information of pipeline requests in a graph view

As a Pipeline Builder/ HKube manager I want to be able to view statistical information of  pipeline requests in a few graph views (one graph per information type), so that I can analyze the resources allocation mechanism and sizing of system resources over time.
User Story implementation details:

The information to be displayed in the graph view will be detailed in other user stories.
The user will be able to filter the information displayed in all the graphs by time frame.
The user will be able to filter the information displayed in all the graphs by pipeline types (multiple choice/ all).
The statistical information will be displayed in separate line per algorithm type.

TID-206


Creating flutter projects take too long time and doesn't complete

Steps to reproduce

when i want to create flutter project using IDEs or flutter tools "flutter create porject_name" i stuck in reslolve dependencies
it takes long time more than 30 minutes and doesn't complete

Actual results

Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

Hi @el-ghnnam, please provide the full output of flutter create <foo> -v. The output may be huge so kindly put it into a .txt file and attach it here.
Please also upgrade to stable 3.24.1 to see if you still experience this issue.
Thank you

@el-ghnnam try this cmd then re-run.
killall -9 dart

Hi @el-ghnnam, please provide the full output of flutter create <foo> -v. The output may be huge so kindly put it into a .txt file and attach it here where <foo> is the name of the project
Please also upgrade to stable 3.24.1 to see if you still experience this issue.
Thank you

New Text Document.txt

Project successfully created. but in some cases i don't know why, when i create new one it stuck in resolving dependencies take more than 30 mintues.

@el-ghnnam try this cmd then re-run.
killall -9 dart

my main OS is Windows

Hi @el-ghnnam, according to the logs provided in https://github.com/flutter/flutter/issues/154478#issuecomment-2325628882, the command completed as expected. Did you change do anything before re-running the command?

Hi @el-ghnnam, according to the logs provided in #154478 (comment), the command completed as expected. Did you change do anything before re-running the command?

convert to stable mode.
but the actual problem is that any resolving dependencies take long time even if i run dart pub upgrade --major-versions.

but the actual problem is that any resolving dependencies take long time even if i run dart pub upgrade --major-versions.

This could be a network issue. Can you check with --offline mode to see any difference? (flutter create --offline helloworld)

but the actual problem is that any resolving dependencies take long time even if i run dart pub upgrade --major-versions.

This could be a network issue. Can you check with --offline mode to see any difference? (flutter create --offline helloworld)
It is indeed an internet problem, so thanks for everything.


Bug: Trying to sync a team requires logout

Describe the bug
Frequently when I'm trying to sync a team to a list, I get this error:

The solution is to log out and log back in again.
Steps to reproduce

Go to List
Create New List
Choose and organization
See error

This often happens when I'm already logged in for a while, but today it happened right after I freshly logged in.
Browsers
Chrome
Additional context (Is this in dev or production?)
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I think this is kind of related to the github token timeout with supabase... Just thinking out loud: Looking at https://github.com/supabase/gotrue-js/issues/131 from supabase, if a provider_refresh_token is obtainable it would be a potential fix for this issue across the app. thoughts @open-sauced/engineering?

Yes, we would need to get the provider_refresh_token and use it to refresh the token in place. I saw a post in the issue that may work. I'll take another look

Note: we have the same problem when generating AI text in highlights: https://github.com/open-sauced/app/issues/1440
My hypothesis is we need to probably fix this app wide since we'll keep hitting sharp edges with OpenSauced <--> Supabase <--> GitHub integration

Did some investigation here, and provider_refresh_token is available but only when using a GitHub App, but not a GitHub OAuth App, which is what we're using for authentication. I created a repo here
https://github.com/brandonroberts/supabase-github-app-auth
Steps to reproduce:

Clone the repo
Install the dependencies
Run the dev server npm run dev
Navigate to http://localhost:3000/login
Open Browser Devtools
Click on the "Login" button
Login with the GitHub prompt
After redirected back to http://localhost:3000, see the browser console and look at the session data
Note the presence of the provider_refresh_token value instead of being null

We removed organization permissions as a requirement


Potenial inefficiencies related to virtualization of distance function computation.

It seems that being able to use the pointer to the distance function directly should be a sufficiently good fix.

This is subsumed by #34


Error when cloning git repository into container

When I'm trying to use "git clone" to clone the repository into a mounted folder, the git command fails at certain positions.
At first I thought it had something to do with our driver, but I was able to reproduce this on the master branch on dokan-dotnet, running with stock settings (I mounted the drive to V:)
Here's the output of the command:
PS V:\> git clone https://github.com/phobeus/dokan-dotnet.git
Cloning into 'dokan-dotnet'...
remote: Enumerating objects: 75, done.
remote: Counting objects: 100% (75/75), done.
remote: Compressing objects: 100% (56/56), done.
remote: Total 2369 (delta 39), reused 40 (delta 19), pack-reused 2294
Receiving objects: 100% (2369/2369), 2.07 MiB | 431.00 KiB/s, done.
Resolving deltas: 100% (1580/1580), done.
error: unable to append to 'V:/dokan-dotnet/.git/logs/refs/remotes/origin/HEAD': Invalid argument
fatal: update_ref failed for ref 'HEAD': cannot update the ref 'HEAD': unable to append to 'V:/dokan-dotnet/.git/logs/HEAD': Invalid argument
Unlink of file 'dokan-dotnet/.git/objects/pack/pack-726956d72141b339e8fa1feda01e32f0fd7c7330.idx' failed. Should I try again? (y/n) n
Unlink of file 'dokan-dotnet/.git/objects/pack/pack-726956d72141b339e8fa1feda01e32f0fd7c7330.idx' failed. Should I try again? (y/n) n
PS V:\>

If I attach a trace logger, it looks like I'm running into a timing issue:
PS V:\> git clone https://github.com/phobeus/dokan-dotnet.git
Cloning into 'dokan-dotnet'...
error: failed to write new configuration file V:/dokan-dotnet/.git/config.lock
fatal: could not set 'core.filemode' to 'false'

In the trace log, I see the following errors:
DokanNetMirror.exe Information: 0 : WriteFileProxy : \dokan-dotnet\.git\config.lock
DokanNetMirror.exe Information: 0 :   NumberOfBytesToWrite  18
DokanNetMirror.exe Information: 0 :   Offset  7
DokanNetMirror.exe Information: 0 :   Context {System.IO.FileStream, False, False, False, False, #36056, True, False}
DokanNetMirror.exe Information: 0 : Buffer size 18 not power of 2 or too large, returning unpooled buffer.
DokanNetMirror.exe Information: 0 : -1 (size 18) outside pool range, discarding buffer.
DokanNetMirror.exe Information: 0 : WriteFileProxy : \dokan-dotnet\.git\config.lock Return : Success NumberOfBytesWritten : 18
DokanNetMirror.exe Information: 0 : ReadFileProxy : \dokan-dotnet\.git\config
DokanNetMirror.exe Information: 0 :   BufferLength  4096
DokanNetMirror.exe Information: 0 :   Offset  0
DokanNetMirror.exe Information: 0 : Exception thrown: 'System.FormatException' in mscorlib.dll
DokanNetMirror.exe Error: 0 : ReadFileProxy : \dokan-dotnet\.git\config Throw : Input string was not in a correct format.

In my own application, I'm getting the following error, but I was never able to reproduce this on mirror unfortunately:
2020-10-22 13:00:58.479 +02:00 [DBG] GetFileInformationProxy : \test\repo\.git\logs\refs\remotes\origin\HEAD
2020-10-22 13:00:58.479 +02:00 [DBG] 	Context	{<null>, False, False, False, False, #29660, False, False}
2020-10-22 13:00:58.479 +02:00 [DBG] 	FileName	HEAD
2020-10-22 13:00:58.479 +02:00 [DBG] 	Attributes	"Normal"
2020-10-22 13:00:58.479 +02:00 [DBG] 	CreationTime	"2020-10-22T13:00:58.4775505+02:00"
2020-10-22 13:00:58.479 +02:00 [DBG] 	LastAccessTime	"2020-10-22T13:00:58.4775505+02:00"
2020-10-22 13:00:58.479 +02:00 [DBG] 	LastWriteTime	"2020-10-22T13:00:58.4775505+02:00"
2020-10-22 13:00:58.479 +02:00 [DBG] 	Length	0
2020-10-22 13:00:58.479 +02:00 [DBG] GetFileInformationProxy : \test\repo\.git\logs\refs\remotes\origin\HEAD Return : "Success"
2020-10-22 13:00:58.480 +02:00 [DBG] WriteFileProxy : \test\repo\.git\logs\refs\remotes\origin\HEAD
2020-10-22 13:00:58.480 +02:00 [DBG] 	NumberOfBytesToWrite	191
2020-10-22 13:00:58.480 +02:00 [DBG] 	Offset	-1
2020-10-22 13:00:58.480 +02:00 [DBG] 	Context	{<null>, False, False, False, False, #29660, True, True}
2020-10-22 13:00:58.480 +02:00 [DBG] Buffer size 191 not power of 2 or too large, returning unpooled buffer.
2020-10-22 13:00:58.504 +02:00 [DBG] -1 (size 191) outside pool range, discarding buffer.
2020-10-22 13:00:58.504 +02:00 [ERR] WriteFileProxy : \test\repo\.git\logs\refs\remotes\origin\HEAD Throw : Specified argument was out of the range of valid values.
Parameter name: from

Which is weird, from what I've gathered on the dokan side, the offset of -1 should be handled as an "ammend" by fuse? I'm not 100% certain though and I'm also not certain that this is the same issue, we just had similar errors when cloning repositories.
Please let me know if I should attach better logging for this or if this is reproduceable enough.
The clone runs through if I use mirror.exe though, so I thought posting this here seems like the right spot.

Hi @phobeus ,
You are right, -1 should be taken as an "APPEND_DATA"! Good catch.I haven't even looked if the C# Mirror is supporting this case 😢
https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-writefile
To write to the end of file, specify both the Offset and OffsetHigh members of the OVERLAPPED structure as 0xFFFFFFFF. This is functionally equivalent to previously calling the CreateFile function to open hFile using FILE_APPEND_DATA access.

FileStream.Position used will be set to -1 and the doc says nothing of what will happen. I might be better to add a check for -1 offset and seek the end of the file ourself. Would you be able to do the test ?

Hey @Liryna ,
We were able to reproduce this inside our software in a unit test.
Here's the code for the curious:
    public void TestWriteOverlapped() {
      var fileName = GetTemporaryFileName();

      var handle = Win32Api.CreateFile(fileName,
        Win32Api.GENERIC_READ | Win32Api.GENERIC_WRITE,
        Win32Api.FILE_SHARE_NONE,
        IntPtr.Zero,
        Win32Api.CREATE_ALWAYS,
        Win32Api.FILE_FLAG_OVERLAPPED,
        IntPtr.Zero
      );

      var writeBuffer = new byte[] { 65, 66, 68 };
      var result = WriteFileWithOverlappedAppend(handle, writeBuffer, out var bytesWritten);
      Assert.That(bytesWritten, Is.EqualTo(writeBuffer.Length));
      Assert.That(result, Is.True);
    }

    private bool WriteFileWithOverlappedAppend(SafeFileHandle handle, byte[] writeBuffer, out uint bytesWritten)
    {
      var stateChangeEvent = new ManualResetEvent(false);
      var nativeOverlapped = new NativeOverlapped
      {
        OffsetHigh = -1,
        OffsetLow = -1,
        EventHandle = stateChangeEvent.SafeWaitHandle.DangerousGetHandle()
      };

      // See http://www.pinvoke.net/default.aspx/kernel32/GetOverlappedResult.html for more information
      var result = Win32Api.WriteFile(handle.DangerousGetHandle(), writeBuffer, (uint)writeBuffer.Length, out bytesWritten, ref nativeOverlapped);
      var lastError = Marshal.GetLastWin32Error();
      if (!result && (lastError == Win32ErrorCodes.ERROR_IO_PENDING || lastError == Win32ErrorCodes.ERROR_SUCCESS))
      {
        return Win32Api.GetOverlappedResult(handle.DangerousGetHandle(), ref nativeOverlapped, out bytesWritten, true);
      }

      bytesWritten = 0;
      return false;
    }

When running this test on windows, it's fine, but in dokan-dotnet it crashes. I have not had the time to try to fix it yet, just wanted to update for now.

I created a pull request: https://github.com/dokan-dev/dokan-dotnet/pull/262 with the fix. After the fix, I was able to pull git repositories.

@phobeus Thank you very much for reporting, investigating and contributing the fix ! That's very much appreciated it really deserves a 🏆 !


:ambulance: Fix the problem where pressing Enter after deleting a value in the custom input results in the first value being incorrectly formatted as '2001'.

I have an issue where, after typing the year and pressing Enter in the customInput field of React Datepicker, the value is automatically formatted when I delete it.
I'm handling the issue by blurring the field when Enter or Escape is pressed, and ensuring the selected value is preserved.

It may be worth switching from casting to a type guard (if (event.target instanceof HTMLInputElement) { ... }) and handling the case where that isn't true, in order to help avoid future bugs. Other than that, this seems pretty straightforward and looks good to me.
 Ryan Lester <@buu700>
Reviewed with ❤ by PullRequest

so i must handle this event using instanceof HTMLInputElement to get approval? or is there another way?


How to omit an endpoint from generated swagger doc?

I'm sorry... I'm sure this has been covered but I can't find it in issues or documentation.
I have some endpoints that I don't want to include in the generated documentation.  How do I mark for exclusion/omission from the generated API documentation, but still have them served by the wrapped actix App?

hmm not that I know of other than adding it before you call wrap_api or after the build. Is there a reason why it has to be added within the "wrap"?

No.  Just didn't realize there was an option to add unmountable actix endpoint methods outside of the wrapper.  I'll give that a try.  Thanks so much!

Like a charm.  Exactly what I was looking for.  Thanks again @tiagolobocastro !

Happy I could help @mharkins-cosm :+1:


Response get find by id became array

First expand response "200" https://rebilly.github.io/ReDoc/#operation/findPetsByStatus where it must be array.
After expand list https://rebilly.github.io/ReDoc/#operation/getPetById. Response became array to.

@DaftTrick, thanks for finding! Will look into this ASAP

Should be fixed with the latest  v1.5.2 release.


⚠ Fortnite has degraded performance

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

HTTP code: 200
Response time: 1516 ms

Resolved: Fortnite performance has improved in d9e983e.


Clobbering CSS files for external pages

If you have a page that you don't want to have the default design for the site, but it still has CSS files, and you put those CSS files in static, those files will possibly clobber the CSS files of the site.
To fix, maybe...
have a flag that tells Docusaurus which are the canonical CSS files that should be used when rendering. While the others will be ignored.

separateCss https://docusaurus.io/docs/en/site-config.html#optional-fields