Response with HTTP 206 code is not returned

api.request.Request#request returns actual response only if the API returned HTTP 200 code, in other cases exception is raised. /whois/parsed/ endpoint sometimes returns HTTP 206 for not parsed records. In such case client raises ServiceException instead of returning actual response.

I've noticed this as well, most recently with parsed_whois on https://whois.domaintools.com/nrgpetrol.hu
domain:         nrgpetrol.hu

Tovabbi adatokert ld.:
http://www.domain.hu/domain/domainsearch/
For further data see:
http://www.domain.hu/domain/English/domainsearch/

The way I see it, there are two issues with this:

Is a 206 partial content the best response to handle this case ?

https://www.domaintools.com/resources/api-documentation/parsed-whois/ describes it as Parsing was unsuccessful due to an unknown formatting of the Whois record., but that sounds to me like a 400 response would be more appropriate if that were the actual case.
Or why not just return a whois object with just the domain, leave the unparsed fields blank ?

The second point is, should the python API still return the content instead of raising an exception ?
or as stated previously, handle the invalid fields but in the python wrapper.

I'm no expert on API architecture, but something seems strange about the way this is currently handled.

In the newer https://github.com/DomainTools/python_api, parsed whois will raise an IncompleteResponseException, but the response is still there in the exception message.
["response"]["whois"] gives the unparsed whois


benchmark GFS

Thanks! Are these JSON files supposed to be committed? Looks like they are not really used except for TAXI_properties.json?


Add cover to MP4, picture filename should not be limited to "cover" only.

Hi Stax,
noticed that for adding a mp4 cover we need to have the name of the picture named "cover" or it won't be accepted and we will get a warning about it, on the GUI,

Only the file extension should be checked here for MP4's, not the name of the input picture itself
For MKV's i know you need to have a specific names for cover files and this should stay for mkv's,  but for MP4's as far as i know it does not matter, the cover can have whatever names for input  it does not matter as the cover will somehow be turned automatically by mp4 into "cover" that i know, at least never had any problem using a lot of different names for input cover,
the output itself when using dump-cover command will always be "cover" tho, automatically set by mp4box.
So it will be nice to have this input file name limitation removed for MP4's when adding a cover please.
Thank you very much

Hi Yanak, I Changed it for the next build.

Tested and seems to work with any input name now, perfect,
Thanks a million for all the nice work you put into this Stax


Update aws-core, http-client-spi, ... to 2.17.32

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.31 to 2.17.32.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, semver-patch

Superseded by #507.


mim download mmdet --config xxxxx --dest . 报错

用mim download 下载配置文件报了一个错，下载没成功。

@FengRongYue This model weight is not uploaded, so it will fail to download


rePatch crashes when entering sleep mode with Trinity

Greetings. Just a small disclaimer, I know that rePatch doesn't currently support 3.70 and I understand if it never will. I would just like to share my current experience with rePatch while using the Trinity exploit. Please close if the issue is irrelevant.
Describe the bug
Currently with the Trinity exploit, rePatch actually works with games. For my testing I was using the PSN store version of Super meat boy and I was able to put the original soundtrack in the game using rePatch. However, every time my vita would enter sleep mode, it would instead crash and reboot the system. This would happen in-game and in the main menu.
To Reproduce
Steps to reproduce the behavior:

Install rePatch latest version by following the instruction. (I used ur0:tai),
Reboot the system
Start the trinity exploit
Put the system in sleep mode (manually or waiting for the screen to go dark)

Additional context
For my testing I was running Trinity with nonpdrm and rePatch at first. After I unistalled nonpdrm but I was having the same behaviour. Only when I removed rePatch from the config file that my vita stopped crashing.
Once again this is purely informative. If rePatch doesn't have plan to support patch 3.70 in the future please close this issue. Thank you for your good work!

taiGetModuleInfoForKernel(KERNEL_PID, "SceIofilemgr", &tai_info);

switch(tai_info.module_nid) {
	case 0xA96ACE9D://3.65
	case 0x90DA33DE://3.68

you have to add 3.70's modid ...

Me too! I encountered the same problem. I thought Nonpdrm was the sole cause, but until a few trial and error that I found out it was RePatch which made mess with standby or sleep mode where in that state, my Vita wouldn't wake up. The solution was remove it and never install games that require RePatch. HAHA

I hope dots-tb will add support for version 3.70

Me too! I encountered the same problem. I thought Nonpdrm was the sole cause, but until a few trial and error that I found out it was RePatch which made mess with standby or sleep mode where in that state, my Vita wouldn't wake up. The solution was remove it and never install games that require RePatch. HAHA

Greetings. Just a small disclaimer, I know that rePatch doesn't currently support 3.70 and I understand if it never will. I would just like to share my current experience with rePatch while using the Trinity exploit. Please close if the issue is irrelevant.
Describe the bug
Currently with the Trinity exploit, rePatch actually works with games. For my testing I was using the PSN store version of Super meat boy and I was able to put the original soundtrack in the game using rePatch. However, every time my vita would enter sleep mode, it would instead crash and reboot the system. This would happen in-game and in the main menu.
To Reproduce
Steps to reproduce the behavior:

Install rePatch latest version by following the instruction. (I used ur0:tai),
Reboot the system
Start the trinity exploit
Put the system in sleep mode (manually or waiting for the screen to go dark)

Additional context
For my testing I was running Trinity with nonpdrm and rePatch at first. After I unistalled nonpdrm but I was having the same behaviour. Only when I removed rePatch from the config file that my vita stopped crashing.
Once again this is purely informative. If rePatch doesn't have plan to support patch 3.70 in the future please close this issue. Thank you for your good work!

Hi, if you are not lazy to translate from Russian, here is the solution to your problem.
https://4pda.ru/forum/index.php?s=&showtopic=315777&view=findpost&p=86454513

taiGetModuleInfoForKernel(KERNEL_PID, "SceIofilemgr", &tai_info);

switch(tai_info.module_nid) {
	case 0xA96ACE9D://3.65
	case 0x90DA33DE://3.68

you have to add 3.70's modid ...

the default block contains the same code as the 3.65/3.68 block, I don't think it would work

FUCKING DOWNGRADE ALREADY


Adds vm.json file for snake example

IncludeOS now uses external vm config.
The snake example requires graphics output
to function correctly.
This change defines a new vm.json file
which enables qxl graphics and 32M of ram.

Can one of the admins verify this patch with one of the following commands:

"jenkins test please" for a one time test run
"ok to test" to accept latest and future commits on this pull request for testing
"add to whitelist" to add the author to the whitelist

ok to test


HyperGCN normalising constant

Hi,
I was working on the HyperGCN and noticed that for some datasets (mainly the heterophilic ones) the normalisation constant (c) in the Laplacian method could become negative if the length of the hyperedge was 1. This will result in negative weights, which seems to corrupt a lot of the results you have gotten from the HyperGCN (from the performance row in the paper). I believe you could fix this by simply removing the "-3" on row 112 in utils.py.
As an example of performance boost: HyperGCN + Mushroom now gives a final test accuracy of 96.69 ± 0.82.

Hi @Fladmark ,
Thanks for sharing your findings. This is indeed really interesting. We use the same implementation as the official HyperGCN implementation, where they do have the "-3" in their implementation. See here.
We decide not to modify the code so that it is indeed a "HyperGCN", as it is an intrinsic drawback of their design. In fact, your finding shows that the design of HyperGCN is not suitable for general hypergraph learning. How to design the "normalization constant c" in HyperGCN in a principled way is an interesting question. Yet the real question is whether we should really use the HyperGCN methodology for hypergraph learning. Nevertheless, maybe your finding can lead to a new type of method for hypergraph learning, which I would love to learn if you develop it in the future.
Let me know if you have further questions!
Best,
Eli

Hi @Fladmark ,
Since there's no follow-up question for about 3 weeks, I will close the issue for now. Nevertheless, please feel free to reopen the issue if you have any further questions.
Best,
Eli


copy config method

Also add ?s just in case :wink:

Did it myself 🤷♂ https://github.com/amazingalek/owml/pull/262

@TAImatem wanna merge this?


Ensure everyone involved have access to Google Analytics and GTM

What
Ensure everyone in the squad has access to Google Analytics and GTM, including the possibility to set up new properties for GA4.
Why
So we can do the work
Who needs to work on this
Anyone in the team
Who needs to review this
Not applicable
Done when

[ ] Folks in the squad have access to Google Analytics
[x] Folks in the squad have access to GTM

We've been given the right level of permissions, so closing this one 🥳


Switch voting from plurality to approval

About the PR
This switches the voting system from plurality voting, the worst one, to approval voting, one of the better ones, actually.
Approval voting is exactly as easy to implement as plurality voting and leads to significantly better results for the kinds of polls done in this kind of game. It's significantly better that players be able to say "yeah, I'm fine with these 4 maps" than to have to pick one in particular, for example, and the same is true of game rules and similar things.
Eventually I could allow multiple voting systems, but I'm still new to this whole SS14 thing (and C# too, to a lesser extent), so, like, not right now. This would entail adding new UIs for ranked systems and score-based systems. Plurality itself could probably be implemented separately, but, frankly, I think that's kinda pointless, since it's essentially strictly worse approval (if everyone bullet votes their favorite, it has identical results).
This is a draft because I don't have a proper testing setup yet and thus haven't actually tested it (I can run unit tests, and did for my last PR, but no proper testing). Once I get that all squared away I'll open it properly.
Screenshots

Changelog

:cl:

add: Added fun!
remove: Removed fun!

what's the point of even having abstain in this
you should be able to pick what kind of vote it is, such as for yes or no votes

abstain was part of a previous PR, not this one
for yes/no votes I also raised the same issue but if you think about it approval is basically identical to plurality for 2 choices, except for the case where people select both choices, but that doesn't really matter as an actual case, unless people misread and don't deselect their other choice when switching votes which may be an issue

You coming back to this @Putnam3145 ?


Added Web Selector-- let user drill down to select web

Q
A

Bug fix?
no - yes?

New feature?
no - yes?

New sample?
no - yes?

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
Guidance
You can delete this section when you are submitting the pull request.

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'dev' branch.

Thanks for your contribution! Sharing is caring.

Hi @russgove, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


V2 - Appropriate error message is not displayed when ICLA template is not generated

Prerequisite

Create a CLA group with only ICLA enabled

Add github repository for the CLA group
Do not generate template

Steps to Follow

Login to Github as individual contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Individual Contributor
Click on SIGN CLA when the button is enabled
Observed error is returned in API call while preparing document as template is not generated

However SIGN CLA button is enabled and CLA Ready For Signature dialog is displayed

Click on SIGN CLA button
Observed below error message is displayed. Appropriate error message should be displayed

@pranab-bajpai
Could you please provide the error message that needs to be displayed in this scenario.
Since the issue is identified while preparing the document, instead for CLA Ready For Signature, should another dialog be displayed? If so can you please provide the wireframe design

Error message should be "CLA system is not able to support your request please open a support ticket"

Verified that the issue is fixed. Also observed that support link is opened in new tab when clicked on create a ticket link


Fix protobuf dependency, as 1.4.1 makes builds fail.

Fixes #172.

Thanks! As a hot fix I'm accepting this PR but I prefer to fixing the build failure instead of rollbacking the dependency version.


refactor: implement encapsulation for theme logic into structured functions 🪄✨

Related Issue
Close #693
Proposed Changes

Encapsulate the dark mode logic into functions: The existing code will be refactored to encapsulate the theme setting logic into functions. This will improve the readability and make the code easier to understand.

Use constants for theme values: Instead of using hard-coded theme strings, we will introduce constants for the theme values to avoid repetition and enhance code clarity.

Optimize code for setting and toggling themes: The functions will be designed to handle setting the appropriate class and updating UI elements based on the selected theme. The toggleTheme() function will be responsible for toggling between light and dark mode.

Improve theme initialization: The theme initialization process will be streamlined to handle user preferences or default to the dark theme if no preference is found.

Checklist

[x] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[ ] 🌟 ed the repo

[x] Refactor (Enhancing the existing codebase)

Your review is appreciated @agamjotsingh18 👍🏻


Export Stack output to amplifyconfiguration.json

Environment information
System:
    OS: macOS 13.2
    CPU: (10) arm64 Apple M1 Max
    Memory: 16.72 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 21.6.0 - /opt/homebrew/bin/node
    Yarn: Not Found
    npm: 10.2.4 - /opt/homebrew/bin/npm
    pnpm: 8.14.1 - /opt/homebrew/bin/pnpm
    bun: Not Found
    Watchman: Not Found
  npmPackages:
    @aws-amplify/backend: ^0.10.3 => 0.10.3
    @aws-amplify/backend-cli: ^0.10.0 => 0.10.0
    aws-amplify: Not Found
    aws-cdk: Not Found
    aws-cdk-lib: ^2.126.0 => 2.126.0
    typescript: ^5.3.3 => 5.3.3

Description
Hello,
I believe the idea of amplifyconfiguration.json is to gather all environment related configuration generated by the backend in one place, and that's great. There are cases however that we must extend the resources using cdk stacks, and it would be very useful we can add information to the generated file based on CfnOutputs.
Example:
For long running queries, we must create alternative solutions using load balancer and lambdas, for example. As those resources will be created in sandbox environments with different aws domains by default, it would be very useful to have these domains exported in the generated amplifyconfiguration.json for reusing in the frontends.

Hey @malaquf, the latest version of the backend-cli: 0.11.0 should now support adding custom outputs to the amplifyconfiguration.json. PR that adds this functionality.
We are currently working on adding this information to the documentation.

Glad to see it @ykethan! Thank you.

Closing this issue as we are tracking this on https://github.com/aws-amplify/amplify-backend/issues/841


Running storybooks on now.sh fails

Running storybooks on now.sh fails because Now does not allow to modify the filesystem in runtime
Module build failed: Error: EACCES: permission denied, mkdir '/home/nowuser/src/node_modules/.cache/react-storybook'

Is there a way to change Webpack in such way that filesystem modifications aren't required ?

in our scenario we deploy storybooks on every pull-request, building static version takes too long and makes it useless.

Thanks!

This can't be quite right because Zeit allows you to run build scripts that write to disk. Something else must be going on here...

I got around this by doing:
npm i serve
{
  "scripts": {
    "build": "rm -rf .out && build-storybook -c .storybook -o .out",
    "start": "cd .out && serve -p 9001"
  }
}

If the workaround isn't sufficient, please reopen this. PRs are always welcomed!

Solved the problem by adding sudo to the story command
"storybook": "sudo start-storybook -p 6006"


Update for Oceananigans 0.79 and expand README

viscoelastic_ice.jl doesn't compile, or at least it takes longer than 1 hour.
is due to this step
ocean_model = NonhydrostaticModel(; grid, closure,                                   timestepper = :RungeKutta3,                                   advection = WENO(),                                   buoyancy = nothing,                                   tracers = nothing)
by default, it runs on 8 threads, changing that doesn't solve the problem

it works. once I made sure to have the right project environment. thanks


org.openrewrite.java.cleanup.Cleanup breaks class @SpringBootApplication

Hi,
The cleanup recipe includes org.openrewrite.java.cleanup.HideUtilityClassConstructor and this one adds a private constructor to the class annotated with @SpringBootApplication.  This will prevent the Spring Boot Application to run.
Thanks

Hi @rbosse -- good catch, thanks for logging this. That's being looked into.
In the mean time, I don't want you to be stuck, so if you're interested, you can add this to your rewrite.yml config file in the project root to tell HideUtilityClassConstructorStyle to ignore any classes which are annotated with @SpringBootApplication:
type: specs.openrewrite.org/v1beta/style
name: com.example.cleanup.HideUtilityClassConstructorSuppression
styleConfigs:
  - org.openrewrite.java.cleanup.HideUtilityClassConstructorStyle:
      ignoreIfAnnotatedBy:
        -<EMAIL_ADDRESS>        -<EMAIL_ADDRESS># if you're using lombok, etc.
        -<EMAIL_ADDRESS>
and then activate it along with your other config in your pom.xml with this, for example:
...
<activeRecipes>
    <recipe>org.openrewrite.java.cleanup.Cleanup</recipe>
</activeRecipes>
...
<activeStyles>
    <style>com.example.cleanup.HideUtilityClassConstructorSuppression</style>
</activeStyles>

(or similarly in your gradle build file, full examples are in the maven-plugin and gradle-plugin reference docs)
If you try that out and run into any issues, let me know and I'll help out as soon as possible. No worries if not, just trying to make sure you aren't blocked 👍

After looking into this, the reason this was happening is because the recipe was not properly identifying any public static void main(String[] args) method signature within a class. The recipe is supposed to ignore any classes which have the main entrypoint in it to prevent exactly this problem.
In other words, the root of this isn't specifically because of the @SpringBootApplication annotation-- it just so happens that most classes annotated with @SpringBootApplication also have the main method in that same class 🤷  (see also: https://stackoverflow.com/questions/37219469/why-does-the-main-spring-boot-application-always-trigger-pmds-hideutilityclassc). It's because the recipe was failing to see the main method signature.
With that said, https://github.com/openrewrite/rewrite/pull/539 fixes this so this shouldn't be an issue anymore in the next release

Thanks!  It was not urgent but I liked the tip you mentionned.

No prob 👍  thanks for logging it. I'll close this issue since it'll be included in the next release. But if there are other issues in the future, feel free to log them. The feedback helps us out.


can't mix ...$ with anything else in parameter lists

scala> val paramss = List(List(param"x: Int"))
paramss: List[List[meta.Term.Param]] = List(List(x: Int))

scala> q"def foo(...$paramss)(x: Int) = ???"
<console>:31: error: Implementation restriction: can't mix ...$ with anything else in parameter lists.
       q"def foo(...$paramss)(x: Int) = ???"
                  = ...). We just need to bite the bullet and implement liftTreess correctly.

Looks like this was incorrectly marked as fixed by Github PR integration.

@olafurpg and I had a discussion, and we decided to pronounce quasiquotes feature-complete.
There are still some open issues (#46, #223, #251, #280 and others), but they all have two things in common: 1) they require non-trivial design work to be fixed, 2) they aren't popular at all. Given our recent decision to start avoiding unwarranted breaking changes in 2.0, we'll consider quasiquotes good enough at this point and will close all outstanding feature requests.
We will still be fixing bugs (in fact, I've fixed about a dozen of such bugs for 2.0), but without strong evidence our design efforts will be directed elsewhere.


Create your first containerized functions on Azure Container Apps

This command does not work,
docker build --tag <DOCKER_ID>/azurefunctionsimage:v1.0.0 .
With this response
"docker build" requires exactly 1 argument.

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

ID: 6c0173e7-c268-c1d6-2ca8-9f00f18cd867
Version Independent ID: 77c6f256-52a1-319b-36d4-35386d86b147
Content: Create your first containerized Azure Functions on Azure Container Apps
Content Source: articles/azure-functions/functions-deploy-container-apps.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Ok found problem. The . at the end is very important, this is the path to build

resolved


Feature : Starting a Component Cluster

Discussed in https://github.com/OpenSmock/Molecule/discussions/33

Originally posted by labordep May 14, 2021
Starting automatically a Component Cluster composed by a set a Components Classes linked by a componentName.
For example :
MolUtils startComponentsCluster: aComponentList named: aComponentName.
This method start each components classes using "aComponentName" instead of #default key and map automatticaly each of them inside the cluster.
Components are linked with the default component name without specified declaration, the component name is "#default".
The cluster connexion principles are to re-use this default settings but with another component name, for example "#cluster1". When the cluster is started all components are automaticaly linked with #cluster1 key.

@ELePors don't hesitate to expose your ideas on this function to brainstorm on it.

An example of use:
I want to start MyComponentA implementation in two clusters.
Starting same component in two clusters:

component := MyComponentA startInNewCluster.
component componentName. "#default_cluster12542 (generated unique cluster name)"
component clusterName. "#cluster12542"

"another examples"
MyComponentA startInNewCluster. "default->cluster59634 (generated unique cluster name)"
MyComponentA startInClusterNamed: #clusterA. "default->clusterA"
MyComponentA startInClusterNamed: #clusterB. "default->clusterB"

Used this component in a ponctual cluster external component:
MyComponentAClient>>componentInitialize
self forServices: MyComponentAServices useProvider: (ComponentName default; clusterName: #cluster12542; yourself value). "need an object to build the key properly: ComponentName default; clusterName: #cluster12542; yourself value"

Used cluster deployment capacity inside a "clusterized" component.
=> Actual behavior, no changes. but in background check if the component belongs to a cluster:
MyComponentA>>componentInitialize

"starting some sub-component (used when a Component tree is deployed)"
MyComponentBServices start: self componentName. "component used as a cluster => #myKey_clusterX"
MyComponentCServices start. "common component used by all components: warning it is necessary to test before is this component is not already started (dirty example)"

"using sub components"
self forServices: MyComponentBServices useProvider: self componentName. "component used as a cluster => #myKey_clusterX"
self forServices: MyComponentCServices useProvider: #default "common component used by all components, not necessary to write this line, this is for example illustration (explicit)"

MolComponentImpl>>componentName
"return the cluster name or the component name"
^ self cluster ifNil:[ componentName "#componentName" ] ifNotNil:[ :c |
        ComponentName name: componentName; clusterName: c name; yourself value
]

MolComponentImpl>>clusterName
"return the cluster name"
^ self cluster ifNotNil:[ :c | c name "#clusterXXX"]

=> I don't find a use case when 'componentName' is not suffisant to deploy a component stack :/
My be this feature is a reflect of an structuration problematic and not to be managed by Molecule, but by the app designer.

I think that this feature should be implemented by a sub-framework on top of Molecule (or by an application architecture). Si I close it.


Print global parameters

The Parameter Definitions Object should is currently ignored and should be included into the templates.

Contained in @0.6.9 once released


nordic-rse logo is missing

hey @bast! seems like the nordic-rse logo is not available anymore at https://nordic-rse.org/img/logo.png which is why the CI for this repository is failing
Could you please provide a new URL that we can add to
https://github.com/SORSE/sorse.github.io/blob/879fb83116cb82271d2aa2324709d0ceaf6c305a/_data/committee/national_chapters.yml#L38-L46
?

Sorry that I broke that. I will fix that with a pull request in a moment.


Feature/edit er before approval or rejection

Date:  27 Nov 2024

Developer Name:  @tanu-chahal

Issue Ticket Number

Closes #877

Description
This PR implements functionality that allows the ER owner to edit an ER before it is approved or rejected.
Context
Previously, the ER update UI existed for super users, this PR fixes the bugs makes improvements in the current UI, and makes the API request-related changes so that the owners can edit their ERs.
Changes

Input Validation: Ensures the following constraints for the ER update form:

ER Title cannot be empty.
ER Reason cannot be empty.
ER New Deadline cannot be a past date.

Conditional Rendering:

Displays the "Edit" button only for ER cards that the current user is authorized to edit.

Feedback via Toast Notifications:

Displays success or error messages based on the API response when an ER is updated.

API Endpoint Update:

Replaced the previous API endpoint with the under dev flag endpoint that supports these changes.

Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[x] Yes
[ ] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots

Screenshot 1
https://github.com/user-attachments/assets/137a1bb7-c3ac-4162-941c-0454c223c905

Test Coverage

Screenshot 1

Additional Notes

Add screenshots for working solution

Added

Can you pass me design doc?

Here it is @pankajjs:  https://docs.google.com/document/d/19MsDrdyHonUvjf6vKsphe0TjM-ED5xPh_SOyRq8ZrnI/edit?tab=t.0

Hello @prakashchoudhary07 sir are we good here


Refactor mutate mapper

Description
Refactor mutate mapper to be more extensible
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

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.

Codecov Report

Merging #1199 (e78648f) into main (4229dcc) will decrease coverage by 0.33%.
The diff coverage is 90.62%.

@@             Coverage Diff              @@
##               main    #1199      +/-   ##
============================================
- Coverage     94.19%   93.86%   -0.34%
- Complexity     1075     1100      +25
============================================
  Files           142      153      +11
  Lines          3051     3066      +15
  Branches        275      255      -20
============================================
+ Hits           2874     2878       +4
- Misses          128      138      +10
- Partials         49       50       +1

Impacted Files
Coverage Δ

...repper/logstash/mapping/NestedSyntaxConverter.java
100.00% <ø> (ø)

.../dataprepper/logstash/mapping/SubMutateAction.java
0.00% <0.00%> (ø)

...logstash/mapping/mutate/DeleteEntryConversion.java
85.71% <85.71%> (ø)

...tash/mapping/mutate/LowercaseStringConversion.java
85.71% <85.71%> (ø)

.../logstash/mapping/mutate/TrimStringConversion.java
85.71% <85.71%> (ø)

...tash/mapping/mutate/UppercaseStringConversion.java
85.71% <85.71%> (ø)

...er/logstash/mapping/mutate/AddEntryConversion.java
90.90% <90.90%> (ø)

...r/logstash/mapping/mutate/CopyValueConversion.java
90.90% <90.90%> (ø)

...r/logstash/mapping/mutate/RenameKeyConversion.java
90.90% <90.90%> (ø)

...logstash/mapping/mutate/SplitStringConversion.java
90.90% <90.90%> (ø)

... and 6 more

Continue to review full report at Codecov.

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


Cluster Names can be capitalized during provisioning, but not when retrieving Zookeeper and Broker Information

Document Location: Get the Apache Zookeeper and Broker host information -> 2. To set an environment variable to the cluster name, use the following command:
The HDInsight Provisioning process allows Capital Letters in cluster names.  Regardless of the capitalization of the Cluster Name, the user must use all lowercase letters in this case.  If they do not, the subsequent commands will fail.  This should be called out explicitly.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c4963b2d-c40b-b6b7-fbcd-d5d25b720fa6
Version Independent ID: 2c938fe3-0240-6fc6-94ec-1c2a8ef02970
Content: Start with Apache Kafka - Azure HDInsight Quickstart
Content Source: articles/hdinsight/kafka/apache-kafka-get-started.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

@BreakingBI  Thanks for your feedback! We will investigate and update as appropriate.


add clomonitor exception file

Signed-off-by: Bill Mulligan<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

added a CLOMonitor exception file for Slack

would @cilium/tophat be the best code owner for this file?

This week's tophat is here.

would @cilium/tophat be the best code owner for this file?

I don't have a good answer for this right now. It is assigned to the tophat because it is a new file and doesn't have an owner. I would suggest creating a new owner entry like cilium/cncf but let me ping @aanm @lizrice for their opinion.

Could you elaborate more about why we need this on the commit message?

Yes, I've just update the commit message

No need to wait for Maciej's review since it was requested for cilium/contributing and that's already covered by André. Merging.


An ISO timestamp anywhere in a String column causes a PrismaClientKnownRequestError

Bug description
I've spent the last few months working on a Discord bot called Puddleshine. It's a gacha card collector bot that includes all of the characters from the popular novel series Warriors by Erin Hunter. The bot is written in TypeScript and runs as a serverless function on Cloudflare Workers following Discord's official tutorial. D1 is the primary database and I query it using Prisma by following a tutorial in Cloudflare Docs.
The collector cards require art and I've thus had members of the community design and submit Warriors fanart for me to add to the bot with credit. Every time you roll a card, it also shows the name of the person who designed it. Then, you can use the command /catcha artist (artistname) to see that artist's profile and a list of all of the cards they've designed. The profile is supposed to let the artist include links to their social medias, for instance, for some additional credit. When viewing your own artist profile, there's an additional button to edit the profile description. The artist names, discord user IDs, and profile descriptions are stored in a Cloudflare D1 database with Prisma. It looks something like this:

However, I recently noticed something odd. If you put an ISO date string (e.g. 2024-08-24T15:50:04.382Z) anywhere in your profile description (which anyone who has submitted art to the bot could do) it will completely break the /catcha artist command for everyone due to a bug in Prisma.

In this screenshot I've set my description (used the identical test bot Puddlepaw to do it because I don't want to mess with the production bot's database) to This is just a test description, here's the date string: 2024-08-24T15:09:40.854Z and I'm trying to run /catcha artist Bristlefrost but it fails with a Prisma error telling me it has failed to convert the entire column to DateTime even though I've never specified its data type as DateTime in my schema. It's specified as String? in my schema and always has been. Prisma is up-to-date and I've run all of the migrations and prisma generate. There's no way the data type is set as DateTime anywhere. It must therefore be a bug in Prisma.
I'd ideally not want to have any game-breaking exploits in my bot and would like to get it sorted as quickly as possible. I'm not very familiar with the Prisma codebase but I could probably just fork the repo and locate where it's trying to do the conversion and simply comment it out to disable it as a temporary workaround. But I have no clue how to navigate this codebase.
How to reproduce
Simply attempt to query a String column that contains an ISO timestamp somewhere, it doesn't matter where, from a Cloudflare D1 database through Prisma. I've tried it in multiple places and it fails every time. I haven't tried any other databased as D1 is what I use for my project.
Expected behavior
I'd expect Prisma to return the description as string like it's defined in my schema. I didn't specify DateTime as the type in the ArtistProfile model so I don't want Prisma to attempt to convert it into a date automatically just because someone happens to have put an ISO timestamp somewhere in their description. It should be treated as plain text like I've specified in the schema. I would expect it to automatically convert it if I specified DateTime as the type in my schema, but I haven't in this case.
Prisma information

model ArtistProfile {
	name         String   @id @unique
	displayName  String?  @map("display_name")

	discordId  String?  @map("discord_id")

	description  String?

	@@index([discordId])
	@@map("artist_profiles")
}

// artist-db.ts
async function findArtistProfile(name: string, prisma: D1PrismaClient) {
	return await prisma.artistProfile.findUnique({
		where: {
			name: name.toLowerCase(),
		},
	});
}

// In another file for the artist subcommand
async execute(options) {
	const { artist } = parseCommandOptions(options.commandOptions);

	if (artist === undefined || artist.type !== DAPI.ApplicationCommandOptionType.String)
		return simpleEphemeralResponse('No artist option provided');

	const artList = listArtistArt(artist.value);
	if (artList.length === 0) return embedMessageResponse(errorEmbed('No art found with this input.'));

	const artistProfile = await artistDB.findArtistProfile(artist.value, options.env.PRISMA); // This is where the error is thrown
}

Environment & setup

OS: I develop my project in WSL (Ubuntu 24.04 LTS) on Windows 11 but the code runs on Cloudflare Workers
Database: Cloudflare D1 (SQLite)
Node.js version: v22.5.1

Prisma Version
prisma                  : 5.18.0
@prisma/client          : 5.18.0
Computed binaryTarget   : debian-openssl-3.0.x
Operating System        : linux
Architecture            : x64
Node.js                 : v22.5.1
Query Engine (Node-API) : libquery-engine 4c784e32044a8a016d99474bd02a3b6123742169 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 4c784e32044a8a016d99474bd02a3b6123742169 (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.18.0-25.4c784e32044a8a016d99474bd02a3b6123742169
Default Engines Hash    : 4c784e32044a8a016d99474bd02a3b6123742169
Studio                  : 0.502.0
Preview Features        : driverAdapters

Also meeting this problem when insert data to sqlite.
prisma                  : 5.19.1
@prisma/client          : 5.19.1
Computed binaryTarget   : darwin
Operating System        : darwin
Architecture            : x64
Node.js                 : v18.20.4
Query Engine (Node-API) : libquery-engine 69d742ee20b815d88e17e54db4a2a7a3b30324e3 (at ../../node_modules/.pnpm/@prisma+engines<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 69d742ee20b815d88e17e54db4a2a7a3b30324e3 (at ../../node_modules/.pnpm/@prisma+engines@5.19.1/node_modules/@prisma/engines/schema-engine-darwin)
Schema Wasm             : @prisma/prisma-schema-wasm 5.19.1-2.69d742ee20b815d88e17e54db4a2a7a3b30324e3
Default Engines Hash    : 69d742ee20b815d88e17e54db4a2a7a3b30324e3
Studio                  : 0.502.0
Preview Features        : driverAdapters

Change to use $queryRaw to work temporarily.

It 's weird that prisma client works fine after I changed my column field name: params -> param. (@map(params) is not working)
I guess that description also the keyword which result in this problem. @Bristlefrost1


[HOLD for payment 2024-03-20] App loads chats on the left with tag list on the right when refreshing tag list

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.48-0
Reproducible in staging?: y
Reproducible in production?: new feature
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:
Precondition:

User is an admin of Collect workspace with tags.
The tags include tag in Parent: Child format and some regular

Go to staging.new.expensify.com
Go to Workspace settings > Collect workspace.
Go to Tags.
Refresh the page.

Expected Result:
Workspace settings on the left are preserved.
Actual Result:
App loads chats on the left with tag list on the right.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/0a946eee-059b-4028-8e51-a6a240f70d8d

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
When refreshing on the workspace tags page, the left page shows LHN while the central page shows the workspace tags page.
What is the root cause of that problem?
When we refresh the page, it will call getAdaptedState and see what navigator is inside the nav stack. In our case, we have a central pane (workspace tags page).
https://github.com/Expensify/App/blob/ffa731a28a2d1402148eef54e51d86fbb64321fe/src/libs/Navigation/linkingConfig/getAdaptedStateFromPath.ts#L253-L254
It will try to find the matching bottom tab navigator with the current central pane.
https://github.com/Expensify/App/blob/ffa731a28a2d1402148eef54e51d86fbb64321fe/src/libs/Navigation/linkingConfig/getMatchingBottomTabRouteForState.ts#L16-L20
However, we don't have a mapping for the workspace tags page and the bottom tab navigator page.
https://github.com/Expensify/App/blob/ffa731a28a2d1402148eef54e51d86fbb64321fe/src/libs/Navigation/linkingConfig/TAB_TO_CENTRAL_PANE_MAPPING.ts#L4-L18
So, it will default to HOME, that is the LHN.
https://github.com/Expensify/App/blob/ffa731a28a2d1402148eef54e51d86fbb64321fe/src/libs/Navigation/linkingConfig/getAdaptedStateFromPath.ts#L52-L63
What changes do you think we should make in order to solve the problem?

Add the mapping for workspace tags page.
[SCREENS.WORKSPACE.INITIAL]: [
    ...
    SCREENS.WORKSPACE.TAGS,
],

@waterim since this came from the Tags PR. Let's fix this as part of the next settings PR.

@luacmartins this one is already fixed in settings PR!

Nice one!

IMO there's no need for the BZ checklist since this is a new feature in development

I think we can close this one!


Better multi-tenant support

Is your feature request related to a problem? Please describe.
In Compass when inserting a new Asset, the payload looks as follows
{
  "urn": "main-postgres:my-database.orders",
  "type": "table",
  "service": "postgres",
  "name": "orders",
  "data": {
    "database": "my-database",
    "namespace": "tenant-1"
  }
}

Separating assets based on tenants can only be done via data key/value attributes. While insertion, a key of namespace can be utilized, and querying can use a filter of namespace as the key. This design looks suitable for small-scale implementations but will start to break as the number of tenants increases. Compass uses two data repositories(Postgres & ES) and both stores are suffering from this flaw that will eventually limit the scale.
Postgres
One quick fix is to create a btree index on (data->namespace) of the assets table. It can improve the querying speed but still, the design doesn't mandate whether each asset really belongs to a namespace or not. The next step is, adding an application-level handler that will simply add a default namespace id if not supplied in the insert request. That means we are mandating a key in a blob of JSON data that was supposed to be generic. Okay, maybe we can add this in the documentation but I still feel we should have a dedicated column of the namespace which in the future can be utilized for sharding a database if ever Postgres grew to a substantial size which I feel it will looking at the use-case of what Compass is doing(ingesting whole company's metadata).
ElasticSearch
Compass for each service type(kafka/table/etc) creates its own index in ElasticSearch. All the tenants share all the shards and indexes across the ES cluster. To segregate tenant-specific queries, Compass uses filters just like Postgres over data keys. One problem with this design is all the queries span across the complete ES cluster for searching which document belongs to which tenant. For example, if tenant A has 1 GB of data but tenant B has 100 GB of data, both tenants whenever Compass makes a request will search 101 GB of documents. Ideally, tenant A search should limit it to 1 GB(or in the worst case couple of more GBs) but not the whole cluster size.

Describe the solution you'd like
To support multi-tenant from the ground up, I propose
Postgres
Adding a new column for the namespace in Assets table. Our APIs can choose to accept the namespace as one more field, if it's passed in the request it will be populated in the database but if not it can be set as null uuid(0000...). So in the case where users don't care about multi-tenancy, everything will be pushed to a single null namespace but individual tenants can also query assets by namespace id.
ElasticSearch
One index for all services and tenants but all requests should be routed to a unique shard for a single tenant. Compass can categorize a tenant into two tires, shared and dedicated. For shared tenants, all the requests will be routed by namespace id over a single shard in an index. For dedicated tenants, each tenant can have its own index. Note, a single index will have N number of types same as the number of Services supported in Compass. This design will ensure, all the document insert/query requests are only confined to a single shard(in case of shared) or a single index(in case of dedicated).

This design looks suitable for small-scale implementations but will start to break as the number of tenants increases.

I don't think the issues raised for individual datastores justofy this satisfactorily.

One quick fix is to create a btree index on (data->namespace) of the assets table. It can improve the querying speed but still, the design doesn't mandate whether each asset really belongs to a namespace or not.

In my opinion, we are jumping to solutions without clearly demarkating the problem. One problem I see today with our queries being run on the database is that some of them could be offloaded to Elasticsearch. Once you limit the queries against the database to create/update and fetch by ID/version, where is the issue with performance?

One problem with this design is all the queries span across the complete ES cluster for searching which document belongs to which tenant.

This is not correct. When executing a term query in Elasticsearch, the query is executed against all shards that contain the relevant data. However, Elasticsearch is designed to distribute data across shards based on the hash of the document's primary key, so the term query will typically only need to be executed against a subset of the total number of shards.
The exact number of shards that need to be searched will depend on the specific query and the distribution of data across shards. For example, if the term being searched for is very common, it may be present in a large number of shards and require a significant amount of searching. Conversely, if the term is very rare, it may only be present in a small number of shards. In our case, applying a term query for something like a project_id field already ensures we query the minimum number of shards required.
Overall, Elasticsearch is optimized to minimize the number of shards that need to be searched for a given query, while still ensuring that all relevant data is included in the search results.
The proposed solution brings a significant breaking change for a problem that does not exist. In it's current form, I would not be in favor of merging it in.
cc: @StewartJingga

When executing a term query in Elasticsearch, the query is executed against all shards that contain the relevant data.

@sudo-suhas Can you explain how elasticsearch knows to search only shards containing the data query is looking for? Help me understand the current design, if the data is hashed using its _id primary key(as you said) which is distributed in 5 shards(default number) and the input that we got from a user is a text and project_id. Now the input query contains a search text as "orders table" and a project uuid, in this case, how will ES select which shard to query?

For example, if the term being searched for is very common, it may be present in a large number of shards and require a significant amount of searching. Conversely, if the term is very rare, it may only be present in a small number of shards.

How can ES know a term is rare or common before actually searching all shards? Does ES have some pre-search optimization phase? Can you explain this in detail?

I would request you invest some time and effort in reading through documentation and blog posts concerning Elasticsearch. Suggesting architectural changes without understanding key characteristics, and showing benchmarks for performance issues, is not going to be productive.
At the very basic, shards are used for partitioning data and that data in this case is inverted indices. Inverted indices can be used to look up all the documents that contain a specific phrase. Elasticsearch uses hashing to identify the shard that contains the relevant data for the given search term. While I am not very familiar with the internals, since it is a mature and performant product, I would imagine that it would be capable of identifying the intersection of shards that contain text and project_id to limit the search.

When executing a term query in Elasticsearch, the query is executed against all shards that contain the relevant data.

I think I was wrong about this. Each shard is a lucene index into which documents are indexed. And documents are distributed based on hashing. Therefore, each query would need to look up each shard to check if there are any documents in that shard that match the search query.
That said, I am still not convinced with the proposed approach. Do we have any benchmarks or evidence to show that there is a problem with queries being run on all shards? Have we looked at the optimisation strategies recommended by Elasticsearch?

Also see https://www.elastic.co/guide/en/elasticsearch/reference/current/size-your-shards.html

After discussion we are approaching this RLS and shared+ dedicated indexes for multi tenancy.


cxxbridge error: Failed to create symlink

Seeing the following intermittent build failure in our pipeline builds.  Ubuntu 20 based container.  It occasionally fails.
cxxbridge error: Failed to create symlink `/builds/-F39GCyv/2/.../target/cxxbridge/rust/cxx.h` pointing to `/builds/-F39GCyv/2/.../.cargo/registry/src/github.com-1ecc6299db9ec823/cxx-1.0.49/include/cxx.h`

 Caused by:

File exists (os error 17)

Could be file is not synchronized to disk, and the create symlink call is reading through to disk?

Sounds like a race condition! Sorry about the flakiness. I've published a fix in cxx-build 1.0.55.

Thanks!  We'll see how it goes and re-open if it pops up again.


No way to interrupt a client request (app shutdown during long polling)

Hello,
I'm doing long polling with client_https.hpp but I'v a problem when the app is exiting. The long polling can be very long and I must interrupt. I think that it should be possible to close the socket in order to force to stop the request.
Or maybe there is already an other way?

Sadly, it has proven difficult to implement timeouts with the synchronous asio operations previously. I think the solution is to move away from these operations and use asynchronous reads and writes instead. I'll try find some time to do this soon.

OK, now I must detach the thread and then it's just killed by the OS when the main process is exiting. It's very ugly but if it's just a temporary workaround; I can live with it...

You can now set timeouts on client requests using Client::config::timeout

That's not really the fix for this issue. I need to stop the request at any time.
It's not tested but it should be possible with something like that, no?
diff --git a/client_http.hpp b/client_http.hpp
index a9b6344..2594f39 100644
--- a/client_http.hpp
+++ b/client_http.hpp
@@ -147,6 +147,12 @@ namespace SimpleWeb {

             return request_read();
         }
+
+        void abort() {
+            boost::system::error_code ec;
+            socket->lowest_layer().shutdown(boost::asio::ip::tcp::socket::shutdown_both, ec);
+            socket->lowest_layer().close();
+        }

     protected:
         boost::asio::io_service io_service;
@@ -181,11 +187,8 @@ namespace SimpleWeb {
             auto timer=std::make_shared<boost::asio::deadline_timer>(io_service);
             timer->expires_from_now(boost::posix_time::seconds(config.timeout));
             timer->async_wait([this](const boost::system::error_code& ec) {
-                if(!ec) {
-                    boost::system::error_code ec;
-                    socket->lowest_layer().shutdown(boost::asio::ip::tcp::socket::shutdown_both, ec);
-                    socket->lowest_layer().close();
-                }
+                if(!ec)
+                    abort();
             });
             return timer;
         }

Yes, this is an option. Would maybe call it close() instead though, since that is what it basically does; closing the socket.

Added the close (abort) method you suggested.

Thank you very much

I made the Client::close method in commit https://github.com/eidheim/Simple-Web-Server/commit/76bf8942fde3f2042091c8e69ea6388c7bad0c6d just now.

OK; I was thinking that was thread-safe because no mutex was used in timer->async_wait in your previous code. Then it seems that even without the close() method, it was not thread-safe.

The asio handlers are running in the same thread, so they are thread safe in the Client. The close method, however, introduced thread unsafe code.


Include arbitrary Git submodules when assembling a distributable tarball

Hello,
cargo package does not always take into account all Git submodules of the current project. For example, a top-level submodule needed for a custom build will not be included, and, thus, cargo package will fail unless ran with --no-verify.
It has been suggested that this problem appears when a submodule is not within the path of some local dependency; although, child packages are not supported at all.
Thank you.
Regards,
Ivan

Fixed in #1131

It has become so much easier to publish. Thanks a lot, Alex!


make -f make/Makefile binaries fails

(kind-kind:cert-manager)james@Jamess-Mac-Pro-890 cert-manager % make -f make/Makefile binaries
jq --arg name "cert-manager-kubectl-cert_manager-linux-arm.tar.gz" \
		--arg sha256 "8b9d7e26905b2074676cd3e35a807a956e123c23985c417c9eaec48ace8d98a6" \
		--arg os "linux" \
		--arg architecture "arm" \
		'.name = $name | .sha256 = $sha256 | .os = $os | .architecture = $architecture' \
		hack/artifact-metadata.template.json > bin/metadata/cert-manager-kubectl-cert_manager-linux-arm.tar.gz.metadata.json
jq --arg name "cert-manager-kubectl-cert_manager-darwin-amd64.tar.gz" \
		--arg sha256 "3064b64252f9997f4ca43ec1ebe68c5e45965f2d586604f4d54146ecf0f300f1" \
		--arg os "darwin" \
		--arg architecture "amd64" \
		'.name = $name | .sha256 = $sha256 | .os = $os | .architecture = $architecture' \
		hack/artifact-metadata.template.json > bin/metadata/cert-manager-kubectl-cert_manager-darwin-amd64.tar.gz.metadata.json
jq --arg name "cert-manager-kubectl-cert_manager-darwin-arm64.tar.gz" \
		--arg sha256 "85b6f9878637f1027971d0f21539422f65a473edb78379ef5bcc470e1a019e65" \
		--arg os "darwin" \
		--arg architecture "arm64" \
		'.name = $name | .sha256 = $sha256 | .os = $os | .architecture = $architecture' \
		hack/artifact-metadata.template.json > bin/metadata/cert-manager-kubectl-cert_manager-darwin-arm64.tar.gz.metadata.json
mkdir -p bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip
cp bin/kubectl-cert_manager/kubectl-cert_manager-windows-amd64.exe bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/kubectl-cert_manager.exe
cp bin/scratch/cert-manager.license bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/LICENSE
cp: bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/LICENSE: No such file or directory
make: *** [bin/release/cert-manager-kubectl-cert_manager-windows-amd64.zip] Error 1
(kind-kind:cert-manager)james@Jamess-Mac-Pro-890 cert-manager % make -f make/Makefile binaries
make: Nothing to be done for `binaries'.

Looks like this is failing due to something to do with licenses - I'm not too sure exactly where this error comes from or what's causing it. I'm also uncertain why I can't rerun the target after it failed the first time (I assumed that given the previous invocation failed, invoking it again would at the very least fail again in the same way?)

These are the important commands:
mkdir -p bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip

cp bin/kubectl-cert_manager/kubectl-cert_manager-windows-amd64.exe bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/kubectl-cert_manager.exe

cp bin/scratch/cert-manager.license bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/LICENSE

cp: bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/LICENSE: No such file or directory

I don't understand how this flow of events can happen sequentially without outside interference or without say a broken filesystem
First we create a directory called bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip which succeeds
Then we successfully copy kubectl-cert_manager-windows-amd64.exe into that directory which we just made
Then we try to copy bin/scratch/cert-manager.license into that same directory and it looks like cp fails because the directory doesn't exist? If bin/scratch/cert-manager.license didn't exist, cp would mention that filename in its error:
$ cp --version
cp (GNU coreutils) 9.0
$ mkdir -p bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip
$ cp notexist bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/
cp: cannot stat 'notexist': No such file or directory
$ rm -r bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip
$ touch notexist
$ cp notexist bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/notexist
cp: cannot create regular file 'bin/scratch/cert-manager-kubectl-cert_manager-windows-amd64.zip/notexist': No such file or directory

At my best guess, maybe two separate terminals were running make simultaneously and interfered with each other? The actual commands being run are simple, and it looks to me like the exact correct commands are being run based on the output you've posted. I'm not sure I can debug further without more information; the command does exactly what I'd expect it to when running locally, from a clean slate and when re-running afterwards.


Auto comment generation

Hi,
Is there a way the plugin can generate comments? For example I would like to be able to type:
/**[hit enter ENTER] on top a function/class/interface and get some comment generated based on the object that the comment is placed up on.
Any advice?

This seems to be implementable via snippets. We may add it at some point (as we have tons of other enhancement to prioritize right now), but pull requests are always very welcome.

Check out the DocBlockr package https://packagecontrol.io/packages/DocBlockr, it works for Typescript as well for the most part. It does not work for classes/interfaces, but i have found it quite useful for creating comments for functions. Give it a try until this issue is resolved.


Doc: Missing "of" in randomSwitch title in the Cheat sheet

Current user will continue in one [of] the specified ...
https://github.com/gatling/gatling/blob/b7111bd131993656a1ac8b17511414ece98f743e/src/sphinx/cheat-sheet.html#L119

Thanks for reporting!


Feat/idp architecture refactor

Refactor the architecture to be in line with the code standards

@rubensworks @RubenVerborgh @joachimvh , sorry to keep bothering you, but I've found something else I can't figure out about components JS. The project now builds, but when I run it I get this error:
Error: Could not run config urn:solid-server:default:OidcProviderFactory because exactly one valid component type was expected, while 0 were found in the defined types []. Alternatively, the requireName and requireElement must be provided.
Found: Resource {
  value: 'urn:solid-server:default:OidcProviderFactory',
  termType: 'NamedNode'
}
All available usable types: [ ...

I double-checked to be sure everything's wired up properly.
Here (https://github.com/solid/community-server/blob/feat/idp-architecture-refactor/config/presets/identity/oidc-provider-factory.json#L9-L10) the configuration maps urn:solid-server:default:OidcProviderFactory to SolidOidcProviderFactory.
SolidOidcProviderFactory is properly exported in index.ts (https://github.com/solid/community-server/blob/feat/idp-architecture-refactor/src/index.ts#L87)
SolidOidcProviderFactory is exported with the right name and includes arguments in the constructor (https://github.com/solid/community-server/blob/feat/idp-architecture-refactor/src/identity/provider/SolidOidcProviderFactory.ts)
As for its arguments, DevEmailPasswordOidcProviderConfigurationFactory is exported with the correct name (https://github.com/solid/community-server/blob/feat/idp-architecture-refactor/src/identity/email-password/DevEmailPasswordOidcProviderConfigurationFactory.ts) and required in index.ts (https://github.com/solid/community-server/blob/feat/idp-architecture-refactor/src/index.ts#L84)
issuer comes from urn:solid-server:default:variable:baseUrl.
There's the possibility that I'm not interpreting the error message correctly. If that's the case, could you help me interpret it?

@jaxoncreed AFAICS, the file config/presets/identity/oidc-provider-factory.json is not being imported anywhere from your main config.
urn:solid-server:default:OidcProviderFactory only seems to be used in config/presets/identity-provider.json, in which case the error makes sense, as Components.js is not able to determine its @type.
So just making sure that config/presets/identity/oidc-provider-factory.json is being imported should fix the problem.

Ah thanks @rubensworks


[WIP] Inlay blend mode

The technique to render inlay blend mode is here different than in tangram-js (https://github.com/tangrams/tangram/pull/161), since in tangram-es labels are rendered with no depth, in an orthographic camera (a different virtual camera than the one used to render usual world placed polygons), and have no information about the world position or actual depth at which they are attached to (a projection to the viewport, client-side, happens at an early stage before drawing in order to manage collisions in screen space).
An alternative way to render the inlay blend mode uses the stencil buffer, which makes it a potentially more intensive blending mode to use.
The stencil render pass sets 1s in the stencil buffer whenever an occluding geometry for this blend mode appears (geometry off the ground, should consider the order property), and 0s whenever there is no occluding geometry, which would output this in the stencil buffer (debug colors):

The result of drawing inlay labels with this stencil technique is then the following:

Related design issue: https://github.com/tangrams/eraser-map/issues/74
Todo:

[ ] Add stencil polyline style
[ ] Benchmark on devices

Known issue, unfortunately (for now), this blend mode is incompatible with other transparent styles:

Closing for the time being, the technique used here solves the issue but is not a feature priority even if diverging from -js, feedback would be appreciated though.
Improvements to be considered:

as the technique was more of a proof of concept, inlay geometry was a direct copy from the initial geometry, it can be either reduced or point to the same buffer.
inlay shader should get injection points, otherwise shader injection from stylesheet won't have effect on the inlay blend mode


suppressing .NET false positives

Hi,
I was wondering if the ability to suppress .NET issues is possible via  as opposed to ; when I generate the report; i don't have the option to suppress via gav, only sha1; I'm using 3.3.1 version btw.
Thanks,
AD

GAV does not make sense for .NET.  Better suppression capabilities are being planned in the 4.x release. At the moment the only suppression capabilities for .NET is the SHA1.

okay; the problems is per build; for example; I try to suppress CVE-2007-6357 as below:

      file name: XXXXX.YYYY.ZZZZZ.DataAccess.APF.dll

0faf3edfd18185518e72ca446a36eea0aea4c026
CVE-2007-6357

this is not picked up by the next scan; when I look at the same CVE and try to suppress it, the SHA-1 has changed:

      file name: XXXXX.YYYY.ZZZZZ.DataAccess.APF.dll

8c50ef20e7ded3d5f82bc776949cf95d693b0002
CVE-2007-6357

Or am I missing something?

That is odd - the sha should not change unless the dependency changed...  Not sure why I thought the best current solution was the hash - the best is likely to use a suppression based on the file path. See the examples on Suppressing False Positives.
    <suppress>
        <notes><![CDATA[
        This suppresses a specific cve for any test.jar in any directory.
        ]]></notes>
        <filePath regex="true">.*\btest\.jar</filePath>
        <cve>CVE-2013-1337</cve>
    </suppress>

I have been having a similar problem in a C# project. A particular vulnerability is being reported in one of our own non-public DLLs, which I suppose I could omit from the scan entirely. The SHA1 and SH256 values change every time I run dependency-check -- even though the binaries did not change at all.
Certainly can be worked around, but it's very unexpected, and makes the very nice copy-and-paste suppression help offered by the HTML report less useful.

This was enhanced in 5.0.0.


explain_prediction with OneVsRestClassifier and SVC

Hi
Thx for this great package
It looks like there is an error when you use OneVsRestClassifier with SVCClassifier
I have this error
    if clf.kernel != 'linear':
AttributeError: 'OneVsRestClassifier' object has no attribute 'kernel'

Here is the little code snippet to reproduce the issue:
from sklearn.datasets import make_multilabel_classification, fetch_20newsgroups
from sklearn.feature_extraction.text import TfidfVectorizer
from sklearn.multiclass import OneVsRestClassifier
from sklearn.svm import SVC

data_train = fetch_20newsgroups(subset='train', shuffle=True, random_state=42)
vectorizer = TfidfVectorizer(stop_words='english')
X_train = vectorizer.fit_transform(data_train.data[0:100])
y_train = data_train.target[0:100]
classif = OneVsRestClassifier(SVC(kernel='linear'))
classif.fit(X_train, y_train)
expl = eli5.explain_prediction(classif, data_train.data[0], vec=vectorizer)

Thx a lot again
Olivier

Hi
If you change SVC by LinearSVC in the previous code snippet, it works ... Any reason SVC is not supported the same way?
Best regards
Olivier

Hi
If you change SVC by LinearSVC in the previous code snippet, it works ... Any reason SVC is not supported the same way?
Best regards
Olivier


Fix documentation

It's now signInProviders instead of providers
Note: CLA already signed.

I believe this is the same issue as #70, though @chaogoog can confirm.

Yes, this is the same issue. I guess it doesn't hurt to fix the README now and then update it to providers in the next release.


Unpin Rust on Linux ARMv6 and ARMv7

Use the workaround in the mentioned issue instead.

Bedankt!

Bedankt!


Cancel request by TAG

I think it would be a good feature for manage queries workflow. Just like in picasso project.

OkHttp and Volley all has this feature but not here.

Provided, can you please review ? @andrewchen5678 @eGorets
Will be released with version 1.4.8
Implementation: https://github.com/loopj/android-async-http/commit/8b1243cfaab1822cb27bd475715c9af1a9d14454
Sample usage: CancelRequestByTagSample.java

Also included in latest snapshot uploaded to snapshot repository

@smarek cool, thanks, but in some case I think it would be better to set 'Object' as a TAG, for example, in activity we can use 'setTag(this)' and 'cancelQueriesByTag(this)' in lifecycle methods without String tags.

@eGorets that's what the purpose of Context as first param of every AsyncHttpClient call is. You pass the context, and then you can cancel requests for context using cancelRequests(final Context context, final boolean mayInterruptIfRunning) (here: https://github.com/loopj/android-async-http/blob/master/library/src/main/java/com/loopj/android/http/AsyncHttpClient.java#L760 )

@smarek oh, sorry, you are right. This is a really good job.


Possible width bug

Initially, thank you for your excellent work.
However, I have encountered an issue that requires me to modify the following line in order for v2ce.py to work with arbitrary sizes of video:
https://github.com/ucsd-hdsi-dvs/V2CE-Toolbox/blob/f54da43f9643adbdc2a522a1c7e8cb5ee18be9d2/v2ce.py#L204
The variable resized_width needs to be changed to width.
Do you think this is a solid solution, or have I overlooked something?
Thank you!

I face a slightly different case, where my input video is 448 * 448, which means width=height, the code can get an error, and I have to change the function (image_pre_processing)
#images = np.stack([cv2.resize(img, (int(img.shape[1]/img.shape[0]*height), height)) for img in images], axis=0)
images = np.stack([cv2.resize(img, (width, height)) for img in images], axis=0)

Thanks for your feedback! I'll check the code and fix it asap.

Bug fixed. Close this thread. Thanks again for your support!


Add XADC INT and ICAP INT fuzzers to solve 2 of 4 missing INT columns.

Fixes #618, and solves 2 of the 4 missing INT columns from #566 .

Merging this now to get Kintex7 green, please review when you get a chance.


Update autostop loop

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1262  👈
#1261
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @carolineechen and the rest of your teammates on  Graphite


Crash when running tensorflow.python.ops.gen_nn_ops.fractional_max_pool_grad

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
binary
Tensorflow Version
2.10.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Very large tensor or arguments.

Standalone code to reproduce the issue
import tensorflow as tf
import os
import numpy as np
from tensorflow.python.ops import gen_nn_ops
try:
  arg_0_tensor = tf.random.uniform([1, 4, 4, 1], dtype=tf.float32)
  arg_0 = tf.identity(arg_0_tensor)
  arg_1_tensor = tf.constant(-16268772587557, shape=[2, 1, 3, 1], dtype=tf.float32,)
  arg_1 = tf.identity(arg_1_tensor)
  arg_2_tensor = tf.random.uniform([1, 3, 2, 1], dtype=tf.float32)
  arg_2 = tf.identity(arg_2_tensor)
  arg_3_0 = 0
  arg_3_1 = 1
  arg_3_2 = 3
  arg_3_3 = 4
  arg_3 = [arg_3_0,arg_3_1,arg_3_2,arg_3_3,]
  arg_4_0 = True
  arg_4_1 =<PHONE_NUMBER>51
  arg_4_2 =<PHONE_NUMBER>51
  arg_4 = [arg_4_0,arg_4_1,arg_4_2,]
  overlapping = False
  out = gen_nn_ops.fractional_max_pool_grad(arg_0,arg_1,arg_2,arg_3,arg_4,overlapping=overlapping,)
except Exception as e:
  print("Error:"+str(e))

### Relevant log output

```shell
2023-01-25 21:33:53.822517: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-25 21:33:53.917687: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-01-25 21:33:54.342722: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/nimashiri/anaconda3/envs/cuda11.2/lib/
2023-01-25 21:33:54.342886: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/nimashiri/anaconda3/envs/cuda11.2/lib/
2023-01-25 21:33:54.342893: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2023-01-25 21:33:54.828965: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:54.833685: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:54.833791: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:54.834090: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-25 21:33:54.834689: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:54.834824: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:54.834918: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:55.188747: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:55.188899: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:55.188995: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-25 21:33:55.189084: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 4304 MB memory:  -> device: 0, name: NVIDIA GeForce GTX 1660 Ti, pci bus id: 0000:01:00.0, compute capability: 7.5
free(): invalid size
Aborted

</details>

Hi, @SuryanarayanaY
Could you please look into this issue? Thank you!

I tried the code on a VM with Ubuntu 22.04 OS with 2 GPUs(NVIDIA A100-SXM4-40GB) and the ran the code multiple times and found different behaviour like sometimes execution success and some times errors like Segmentation fault (core dumped) and double free or corruption (out) Aborted (core dumped) . May needs to fix this.
Please refer attached log below.
(tf2.10) suryanarayanay@surya-ubuntu-22-04:~$ python 59373.py
2023-01-27 12:22:10.414070: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:10.540608: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2023-01-27 12:22:10.578259: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-01-27 12:22:11.273289: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:11.273394: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:11.273410: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2.10.0
2023-01-27 12:22:16.089567: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:17.399070: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 38239 MB memory:  -> device: 0, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:04.0, compute capability: 8.0
2023-01-27 12:22:17.400698: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:1 with 38239 MB memory:  -> device: 1, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:05.0, compute capability: 8.0
Error:{{function_node __wrapped__FractionalMaxPoolGrad_device_/job:localhost/replica:0/task:0/device:CPU:0}} tensor_out_dup is not the same as tensor_out [Op:FractionalMaxPoolGrad]
Code executed successfully
(tf2.10) suryanarayanay@surya-ubuntu-22-04:~$ python 59373.py
2023-01-27 12:22:28.499709: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:28.628360: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2023-01-27 12:22:28.666042: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-01-27 12:22:29.363626: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:29.363735: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:29.363749: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2.10.0
2023-01-27 12:22:34.179568: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:35.492700: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 38239 MB memory:  -> device: 0, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:04.0, compute capability: 8.0
2023-01-27 12:22:35.494364: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:1 with 38239 MB memory:  -> device: 1, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:05.0, compute capability: 8.0
Segmentation fault (core dumped)
(tf2.10) suryanarayanay@surya-ubuntu-22-04:~$ python 59373.py
2023-01-27 12:22:45.373669: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:45.501051: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2023-01-27 12:22:45.538254: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-01-27 12:22:46.235981: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:46.236107: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/home/suryanarayanay/miniconda3/envs/tf/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/:/home/suryanarayanay/miniconda3/envs/tf2.11/lib/:/home/suryanarayanay/miniconda3/envs/tf2.10/lib/
2023-01-27 12:22:46.236123: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2.10.0
2023-01-27 12:22:51.061559: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-27 12:22:52.372397: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 38239 MB memory:  -> device: 0, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:04.0, compute capability: 8.0
2023-01-27 12:22:52.374072: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:1 with 38239 MB memory:  -> device: 1, name: NVIDIA A100-SXM4-40GB, pci bus id: 0000:00:05.0, compute capability: 8.0
double free or corruption (out)
Aborted (core dumped)
(tf2.10) suryanarayanay@surya-ubuntu-22-04:~$

@learning-to-play
@SuryanarayanaY "awaiting tensorflower" should also mean that there is some TF engineer looking at this

@mihaimaruseac , We are tracking these issues using Hotlist ID, we have notified TF eng team and they are working on it.

Hi @dmc1778 ,
I have tested the code in tf-nightly(2.15.0-dev20231004) and got exception raised. The issue seems resolved in nightly.
Please refer to attached gist and feel free to close it.
Thank you!


Fixing streaming connection handler duplicating the connection status…

I'm trying to dispose of a StreamingConnectionHandler, but it keeps adding another _onConnectionStatusChanged listener.
Now, we can use the dispose method in the StreamingConnectionHandler. This method correctly removes the streamingConnectionStatusListener, preventing the listener from duplicating.
I searched for the test file for the StreamingConnectionHandler but couldn't find any. Do I need to create one? If so, where should I place it?
Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[ ] I added new tests to check the change I am making.
[x] All existing and new tests are passing.
[x] Any breaking changes are documented below.

I have read the CLA Document and I hereby sign the CLA


Godot crashes when trying to generate lightmap for the specific model

Godot version:
3.1 Release
OS/device including version:
Windows 10 / Acer Aspire E 15
Issue description:
Godot is crashed on project open during lightmap generation
Steps to reproduce:

Open attached project
Godot starts importing the model and crashes
There is a message in the log right before crash:

If I manually change meshes/light_baking=2 to meshes/light_baking=0 in the kitchen_sink_table.dae.import file, everything goes fine. But when I try to change it back to 'Generate lightmap' in editor and reimport, Godot crashes again.

Minimal reproduction project:

lightmap.zip

Tested it. Yep switching to a Gen Lightmap and reimport, then crash. Its a bug!

Another cases for this bug:
https://github.com/godotengine/godot/issues/30789


Remove std.prql_version in favor of std.prql.version

See #4006

What's the motivation for this change? Do we have other things to put in std.prql?

What's the motivation for this change? Do we have other things to put in std.prql?

It's more readable and maybe in future we will need it for something

I was +1 on this — we could have other things such as dialect?

Closed by #4006

It wasn't actually deleted it in #4006.
My intention was to keep this open so I wouldn't forget to delete it


229 events on expand

Trello card:
https://trello.com/c/K6GY0FNG/229-vvv-eee-add-google-analytics-with-cookie-banner
Description

Event on expand / close (definition & help self singles)

@neontribe/sea-map

I'm going to see if a sensible refactor hits me - but yes, if we want to log analytics only when consent has been granted then we need to pass that hasConsented flag to any function that wants to do analytics.


Change logout icon

The icon on the logout button on the profile screen is not as common as other icons (power, help, etc) so users wouldn't know what that button does.
Suggest adding text to button to make it more obvious or a modal/alert to let user know they are about to be logged out.

A pop-up was added in the most recent build for this icon - build 56 (2.2.1)


Add TS130 device

Codecov Report

Merging #1256 (01e2895) into dev (d65fd8d) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev    #1256      +/-   ##
==========================================
+ Coverage   80.27%   80.28%   +0.01%
==========================================
  Files         207      207
  Lines        5693     5696       +3
==========================================
+ Hits         4570     4573       +3
  Misses       1123     1123

Impacted Files
Coverage Δ

zhaquirks/tuya/ts130f.py
81.63% <100.00%> (+1.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 80.284%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
4573

Relevant Lines:
5696

💛  - Coveralls


Should we support push (and splice)?

FileMaker's JSONSetElement function will only insert an object into an array at a specific index. What I really need is a way to push an object at the end of an array and I don't see that being possible with the native functions.
With JSON.ArrayLength it's pretty simple, but I think a CF makes sense (see below). I haven't written any tests yet. I wanted to make sure that we think this (and possibly splice as well) would be valuable before spending time on it.
/**

=====================================
JSON.ArrayPush ( theArray ; theObject )

RETURNS:
The array with the object inserted at the end

PARAMETERS:
theArray
theObject

DEPENDENCIES:
JSON.ArrayLength custom function
FileMaker 16 or Later

NOTES:

FAMILY:
JSON - Extensions to the Native JSON Functions

HISTORY:
Created, 2017-07-20<EMAIL_ADDRESS>
=====================================
*/

Let ( [
// Since JSON arrays are zero-based, the size of the array
// will be the index that we are going to insert the object
insertAtIndex = JSON.ArrayLength ( theArray ; "" )
] ;
// Insert the object at the end of the array
JSONSetElement ( theArray ; insertAtIndex ; theObject ; JSONObject )
)

I added JSON.Push and also fixed some errors in the doc block for JSON.IsValid
See attached with tests.
JSONAdditions with Push.fmp12.zip

Why does this throw an error?
JSON.Push ( "[]" ; "foo bar" )
why not this?
JSON.Push ( Test::JSON ; "foo bar" )
//=> ["foo bar"]

["foo bar"] is a perfectly valid array

Good point. I've updated the function to validate the target instead of the source. If there's an error with the source, the JSONSetElement function will return a standard error. I also included a lot more tests, including one that fails.
JSONAdditions with Push.fmp12.zip

4 tests with this file fail.  Maybe you attached the wrong file?

Crap. I accidentally changed a couple things after I had tested, and forgot to re-test. Here you go...
JSONAdditions with Push.fmp12.zip

:-) . This is EXACTLY why you do things this way.

I’ll learn eventually (maybe). ;-)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Todd Geist<EMAIL_ADDRESS>Sent: Friday, December 1, 2017 7:38:22 AM
To: geistinteractive/fm-json-additions
Cc: Dave Graham; Author
Subject: Re: [geistinteractive/fm-json-additions] Should we support push (and splice)? (#13)
:-) . This is EXACTLY why you do things this way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/geistinteractive/fm-json-additions/issues/13#issuecomment-348526597, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATtiGaDgb_YUkxGP6cJD2P6_RFJhXjSmks5s8B1ugaJpZM4OgNnq.

Yeah that wasn't an accusation by Todd, it was praise for test driven
devlopment. :-)
Best regards,
Michael A. Wallace, Sr.
Senior Engineer
Richard Carlton Consulting Inc.
Office<PHONE_NUMBER> (Primary)
Corp<PHONE_NUMBER>
Fax<PHONE_NUMBER>
<EMAIL_ADDRESS>www.rcconsulting.com
On Fri, Dec 1, 2017 at 9:48 AM, Dave Graham<EMAIL_ADDRESS>wrote:

I’ll learn eventually (maybe). ;-)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Todd Geist<EMAIL_ADDRESS>Sent: Friday, December 1, 2017 7:38:22 AM
To: geistinteractive/fm-json-additions
Cc: Dave Graham; Author
Subject: Re: [geistinteractive/fm-json-additions] Should we support push
(and splice)? (#13)
:-) . This is EXACTLY why you do things this way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
geistinteractive/fm-json-additions/issues/13#issuecomment-348526597>, or
mute the thread<https://github.com/notifications/unsubscribe-
auth/ATtiGaDgb_YUkxGP6cJD2P6_RFJhXjSmks5s8B1ugaJpZM4OgNnq>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/geistinteractive/fm-json-additions/issues/13#issuecomment-348529448,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHkmjqWZagEZQzOwl9J_1HXCBCSy6l_Bks5s8B_ggaJpZM4OgNnq
.

I get it...just being difficult. ;-)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Michael A Wallace Sr<EMAIL_ADDRESS>Sent: Friday, December 1, 2017 8:06:36 AM
To: geistinteractive/fm-json-additions
Cc: Dave Graham; Author
Subject: Re: [geistinteractive/fm-json-additions] Should we support push (and splice)? (#13)
Yeah that wasn't an accusation by Todd, it was praise for test driven
devlopment. :-)
Best regards,
Michael A. Wallace, Sr.
Senior Engineer
Richard Carlton Consulting Inc.
Office<PHONE_NUMBER> (Primary)
Corp<PHONE_NUMBER>
Fax<PHONE_NUMBER>
<EMAIL_ADDRESS>www.rcconsulting.com
On Fri, Dec 1, 2017 at 9:48 AM, Dave Graham<EMAIL_ADDRESS>wrote:

I’ll learn eventually (maybe). ;-)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Todd Geist<EMAIL_ADDRESS>Sent: Friday, December 1, 2017 7:38:22 AM
To: geistinteractive/fm-json-additions
Cc: Dave Graham; Author
Subject: Re: [geistinteractive/fm-json-additions] Should we support push
(and splice)? (#13)
:-) . This is EXACTLY why you do things this way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
geistinteractive/fm-json-additions/issues/13#issuecomment-348526597>, or
mute the thread<https://github.com/notifications/unsubscribe-
auth/ATtiGaDgb_YUkxGP6cJD2P6_RFJhXjSmks5s8B1ugaJpZM4OgNnq>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/geistinteractive/fm-json-additions/issues/13#issuecomment-348529448,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHkmjqWZagEZQzOwl9J_1HXCBCSy6l_Bks5s8B_ggaJpZM4OgNnq
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/geistinteractive/fm-json-additions/issues/13#issuecomment-348534505, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATtiGc60CW-Okq2-8spKDmaXT5Gvjtumks5s8CQMgaJpZM4OgNnq.