feat: add custom errors

Add and use sdk core custom errors to facilitate sdk usage and debug.

errors should have a type (INVALID_START_AMOUNT, INVALID_END_DATE, ...)
errors should have a message with context ("Invalid start date. Start date (${startDate}) cannot be in the past.", ...)
errors should include a link to related docs (https://docs.arkproject.dev/)

Hi @gershon can I work on this?

I would love to work on this issue @gershon here is my profile : https://app.onlydust.com/u/Jemiiah

hello @gershon please can i work on this issue tomorrow once the OD hack starts :)

hello @gershon please i would love to work on this issue tomorrow once the OD hack starts :)

Hi @gershon would be happy to take up this issue by tomorrow

@gershon  kindly assign me this task as i would like to make it my first contribution to this project.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, I have been contributing in odhacks for quite some time now and have expertise in rust and blockchain specs , few of them are

Contributed to Floresta as Summer of Bitcoin 2024 fellow
contributed to RUst lang's codegen GCC and implemented new traits and tests.
Contributed to starknet rust devnet in odhack

How I plan on tackling this issue
I will integrate sdk core custom errorin sdk for better error handling and ease of debugging
I will add error messages with proper context and links to debug them.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a Web3/frontend developer
With a lot of experiences in open source contribution.
How I plan on tackling this issue
I will

Define Custom Errors: Create custom error types (e.g., INVALID_START_AMOUNT, INVALID_END_DATE) in the SDK core, using an enum to represent different error types. Include fields for error message, context, and a link to the docs.

Error Messages with Context: Implement detailed error messages that include relevant context (e.g., "Invalid start date. Start date (${startDate}) cannot be in the past.")

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
i'm a solidity and cairo smart contract  developer with over 2 years experience and belive i have the skill set for the task

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
i am a frontend developer with experience using JS and TS for type safety
How I plan on tackling this issue
I will start by defining the custom error types that I need to create, such as INVALID_START_AMOUNT, INVALID_END_DATE, etc. These error types will help categorize specific issues that may arise during SDK usage.
Create Error Messages:
For each error type, I will craft a meaningful error message that provides context. For example, I will create a message like "Invalid start date. Start date (${startDate}) cannot be in the past." This will give users a clear understanding of what went wrong.
Link to Documentation:
I will ensure that each error message includes a link to the relevant documentation (https://docs.arkproject.dev/) to assist users in troubleshooting and understanding the error better.
Implement Custom Errors:
I will implement these custom errors in the SDK core, making sure to use the appropriate syntax and structure as per the coding standards of the project. I will ensure that these errors are thrown at the correct instances where validation fails.
Test Error Handling:
After implementing the custom errors, I will write tests to validate that the errors are thrown correctly under the intended circumstances. This will help confirm that the error handling works as expected.
Update Documentation:
I will update any relevant documentation to include information about the new custom errors, their meanings, and how they can be resolved, ensuring that developers using the SDK have clear guidance.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have experience in building SDKs and handling error management in various systems. I’ve implemented custom error handling in JavaScript/TypeScript projects, providing developers with clear error messages, types, and links to relevant documentation. This ensures a smooth debugging experience and improves SDK usability. My familiarity with tools like Next.js and TypeScript enables me to create a robust error-handling framework for this project.
How I plan on tackling this issue
I would start by creating a custom error class that standardizes error types, messages, and links to documentation. This class would include parameters for the error type (e.g., INVALID_START_AMOUNT), a message that provides context (e.g., Invalid start date. Start date (${startDate}) cannot be in the past.), and a link to the related documentation for easy reference. I would then integrate these custom errors throughout the SDK, ensuring consistency and clarity for users. Finally, I would ensure that each error includes a link to the appropriate section in the documentation to facilitate troubleshooting.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
hello i am a frontend dev and blockchain developer
please can i work on this issue :) and would love to be a contributor
How I plan on tackling this issue
Have a type (e.g., INVALID_START_AMOUNT, INVALID_END_DATE).
Include a message with context (e.g., "Invalid start date. Start date (${startDate}) cannot be in the past.").
Provide a link to related documentation (e.g., https://docs.arkproject.dev/).

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
i am a blockchain developer with strong background in cario, typescript,javascript etc. i will love to work on this issue, kindly assign
How I plan on tackling this issue
Implement custom SDK errors by defining error types such as INVALID_START_AMOUNT and INVALID_END_DATE, along with contextual messages like "Invalid start date. Start date (${startDate}) cannot be in the past." Each error should also include a link to the relevant documentation at https://docs.arkproject.dev/ to aid in debugging and enhance user experience.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hey, I am Bhavya, a seasoned frontend engineer with two years of professional experience. I have been working with react, react native, nextjs, typescript and javascript and have become very proficient in it.
How I plan on tackling this issue
To approach the problem, we can create a custom error handling system that includes error types, context-specific messages, and documentation links.
Each error can include:
An errorType that describes the nature of the error.
A message that provides the specific context.
A documentationLink to guide developers to relevant sections in your docs.
This approach improves SDK usage by:
Providing meaningful error messages with context.
Guiding developers to the relevant part of the documentation.
Facilitating debugging with clear and categorized error types.

Hi @NueloSE any update on this?
ODHack end is quite near.

@ptisserand I'd need more context on

where am I defining the errors
At what point in the SDK usage am I to throw those errors...

This would help me implement, use, and link it to the related docs.
Kindly assist

Hi @NueloSE,

you can define errors in packages/core/src/errors
you can take inspiration from https://github.com/wevm/wagmi/tree/main/packages/core/src/errors
you can use them by replacing existing errors, like in https://github.com/ArkProjectNFTs/ark-project/blob/feat/contract-v2/packages/core/src/actions/order/createOffer.ts#L67.

You should create your contribution branch from feat/contract-v2 as it contains latest changes to be released soon.
Lmk if you need more guidances or informations.

Thank you this makes more sense now, I will be making a pr b4 end of 2mrw

@gershon I made a PR and no review yet...

Hi @NueloSE, I'll review it shortly.

Closed by #480


Does SAM2 and further version support multi-gpus?

As illustrate in the title.
Thank you.

@mellody11 did you figure this out? I'm also trying to use a Multi-GPU setup to segment throughout the video and the single gpu runs out of memory.

Sorry, I have no idea yet.


Webhook - Support "/connections"

Pairwise Connection Record Updated - https://github.com/hyperledger/aries-cloudagent-python/blob/master/AdminAPI.md#pairwise-connection-record-updated-connections

Figure out if we need to support this from https://github.com/hyperledger/aries-cloudagent-python/blob/master/AdminAPI.md#administration-api-webhooks. There is currently a struct defined in payloads.go with this structure that has a TODO above it. If it turns out that we don't need it, then delete it.

This will be done as part of #551 .


Skip XML element from mod sec rule scanning from xml payload

Hi Team,
We have the below xml payload getting blocked due to mod sec rule
{"messages":["Access denied with code 403 (phase 2). Pattern match "(/\\!?|\\/|[';]--|--[\\s\\r\\n\\v\\f]|(?:--[^-]?-)|([^\\-&])#.?[\\s\\r\\n\\v\\f]|;?\\x00)" at XML. [file "mod_perl"] [line "409"] [id "942440"] [rev "2"] [msg "SQL Comment Sequence Detected."] [data "Matched Data: --ZXbyVq6U8rOy6un57Q1Z7k97CQpDudZSDekE3ng1enjF3dKfiuHk4zpyfq_MMy_qj_0w_z5zmkHMgIQw- found within XML:     REG_CODE<PHONE_NUMBER>    600            FCM:                  "] [severity "CRITICAL"] [ver "OWASP_CRS/3.0.2"] [maturity "8"] [accuracy "8"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS/WEB_ATTACK/SQL_INJECTION"] [tag "WASCTC/WASC-19"] [tag "OWASP_TOP_10/A1"] [tag "OWASP_AppSensor/CIE1"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]"],"error_messages":["[file "apache2_util.c"] [line 271] [level 3] [client <IP_ADDRESS>] ModSecurity: Access denied with code 403 (phase 2). Pattern match \\"(/\\\\\\\\!?|\\\\\\\\/|[';]--|--[\\\\\\\\s\\\\\\\\r\\\\\\\\n\\\\\\\\v\\\\\\\\f]|(?:--[^-]?-)|([^\\\\\\\\-&])#.?[\\\\\\\\s\\\\\\\\r\\\\\\\\n\\\\\\\\v\\\\\\\\f]|;?\\\\\\\\x00)\\" at XML. [file \\"mod_perl\\"] [line \\"409\\"] [id \\"942440\\"] [rev \\"2\\"] [msg \\"SQL Comment Sequence Detected.\\"] [

xxxxx

FCM:xxxx/notificationProfile>

we would like to rewrite the rule to update it to skip the element  FCM:xxxx/notificationProfile> from scanning in case of xml payload.
I am able to rewrite it for json payload like below and its working for me.
SecRuleUpdateTargetById 942440 "!ARGS:notificationProfiles.notificationProfiles"
For XML payload i am trying to use it like below but its not working and request is still getting blocked. Kindly suggest
SecRuleUpdateTargetById 942440 "!XML:/RegisterRequest/notificationProfiles"
Overall :
SecRuleUpdateTargetById 942440 "!XML:/RegisterRequest/notificationProfiles"
SecRuleUpdateTargetById 942440 "!ARGS:notificationProfiles.notificationProfiles"

Hello @Vikasjava60 ,
Could you please post the payload? (Or at least a representative payload with the relevant tags included, and with any confidential information sanitized.)
Also: Are you using ModSecurity v2 or v3?

Hi @martinhsv
Please find the payload:

RXXXXCODE
0273771812
600

FCM:dVxH7htmS8aaebk4egDxAw:APA91bHmfz****************

and want to skip the  element from scanning. I am able to do it for json payload using
SecRuleUpdateTargetById 942440 "!ARGS:notificationProfiles.notificationProfiles"
when trying for xml payload request using : SecRuleUpdateTargetById 942440 "!XML:/RegisterRequest/notificationProfiles"
ModSecurity V3 we are using

Hi @Vikasjava60 ,
My wording was a bit imprecise; what I meant by payload is the actual xml request body (or equivalent, but including the tags). But I'll go ahead and give some explanation in general terms ...
The XML collection is quite a bit different from ARGS, as it uses XPath expressions for specifying elements within the collection. This allows for a lot of functionality and flexibility in specifying what you want. But it does have the consequence that constructs like SecRuleUpdateTargetById cannot be used in quite the same way as with those other collections.
The best way, in principle, to perform your desired exclusion is to do so with the initial XPath specification. I.e. instead of including something like XML:/* in the list of targets, just perform your exclusions directly in that XPath string.
(This has a downside, of course, if you're using externally-sourced rules, as it means some extra maintenance if/when the ruleset has a newer version published.)

request_payload.txt
Hi @martinhsv
Thanks for your suggestion.
Attaching the actual xml request body. Could you please help me here to rewrite this rule id 942440 in order to exclude the elements:

If you simply want to exclude anything that has RegisterRequest as the first tag in the path, you could do something like this:
XML:/*[local-name()!='RegisterRequest']

However, perhaps you wish to be more cautious and only exclude /RegisterRequest/notificationProfiles and not all of /RegisterRequest. You could combine the above with the following to specifically include those other items within /RegisterRequest:
XML:/*[local-name()='RegisterRequest']/*[local-name()!='notificationProfiles']

Putting those together, you could replace the existing XML:/* in the rule with:
XML:/*[local-name()!='RegisterRequest']|XML:/*[local-name()='RegisterRequest']/*[local-name()!='notificationProfiles']

(Note: I'm not a XPath expert so be cautious with the above suggestion and test it out yourself.)

Hi @martinhsv
Thanks a lot for sharing the xpath expression i tried it with sec rule but not luck as i am getting below errors
Tried to update the rule target but error out as below:
SecRuleUpdateTargetById 942440 "XML:/*[local-name()!='RegisterRequest']"
Error: add_config() has failed: Error to update target - [='RegisterRequest']] is not valid target
Tried to rewrite the rule but error out as below:
SecRule XML:/*[local-name()!='RegisterRequest'] "phase:2,id:942440"
add_config() has failed: ModSecurity: Rules must have at least id action

Hi @Vikasjava60 ,
It doesn't surprise me that your example using SecRuleUpdateTargetById does not work.
Rewriting the rule will work -- I tried it myself before posting my previous response. If your attempted rewrite is as you've shown above, note that you are missing the Operator. E.g. you need something like:
SecRule XML:/*[local-name()!='RegisterRequest'] "@rx ..." "id:98100,phase:2, ..."

Closing due to no further activity.

Hi @martinhsv ,
Thanks a lot for your suggestion. It worked for me partially. Looking for your experties suggestion as i saw some strange behviour.
As per your suggestion i updated the rule defination as shown below. The expection was to skip the notificationProfiles xml element from validation which is now working and data inside the element is not validating against the pattern.
but in case if i am sending the same set of data in other xml element of the same request my expection was the request should be blocked like if i sending the same data in registration code it should be blocked.
but i am observing the warning messeage in the logs and request is getting passed but not getting blocked. Anything i am missing here.
ModSecurity: Warning. Pattern match "(?:/\\!?|\\/|[';]--|--[\\s\\r\\n\\v\\f]|--[^-]?-|[^&-]#.?[\\s\\r\\n\\v\\f]|;?\\x00)" at XML. [file "mod_perl"] [line "142"] [id "MSM_DIS_942440"] [msg "SQL Comment Sequence Detected"] [data "Matched Data: --D7R_Nexck3kgv- found within XML: FCM:f3AK3nt3Zd4:APA91bHvq21zkMuZcCtJ-e4oYAF0VbJgpKaPLZRwov88dpxBFQFSU6m--D7R_Nexck3kgv-NNomPDIGxCQZcAvp5ePVDFkCWAeWjLFDJoJvJ_cw_pj6DqAViXf-01yO88_wFtcYKYgju"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]
"audit_data":{"messages":["Warning. Pattern match "(?:/\\!?|\\/|[';]--|--[\\s\\r\\n\\v\\f]|--[^-]?-|[^&-]#.?[\\s\\r\\n\\v\\f]|;?\\x00)" at XML. [file "mod_perl"] [line "142"] [id "MSM_DIS_942440"] [msg "SQL Comment Sequence Detected"] [data "Matched Data: --D7R_Nexck3kgv- found within XML: FCM:f3AK3nt3Zd4:APA91bHvq21zkMuZcCtJ-e4oYAF0VbJgpKaPLZRwov88dpxBFQFSU6m--D7R_Nexck3kgv-NNomPDIGxCQZcAvp5ePVDFkCWAeWjLFDJoJvJ_cw_pj6DqAViXf-01yO88_wFtcYKYgju"] [severity "CRITICAL"] [ver "OWASP_CRS/3.3.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"]"],"error_messages":["[file "apache2_util.c"] [line 280] [level 3] [client <IP_ADDRESS>] ModSecurity: Warning. Pattern match \\"(?:/\\\\\\\\!?|\\\\\\\\/|[';]--|--[\\\\\\\\s\\\\\\\\r\\\\\\\\n\\\\\\\\v\\\\\\\\f]|--[^-]?-|[^&-]#.?[\\\\\\\\s\\\\\\\\r\\\\\\\\n\\\\\\\\v\\\\\\\\f]|;?\\\\\\\\x00)\\" at XML. [file \\"mod_perl\\"] [line \\"142\\"] [id \\"MSM_DIS_942440\\"] [msg \\"SQL Comment Sequence Detected\\"] [data \\"Matched Data: --D7R_Nexck3kgv- found within XML: FCM:f3AK3nt3Zd4:APA91bHvq21zkMuZcCtJ-e4oYAF0VbJgpKaPLZRwov88dpxBFQFSU6m--D7R_Nexck3kgv-NNomPDIGxCQZcAvp5ePVDFkCWAeWjLFDJoJvJ_cw_pj6DqAViXf-01yO88_wFtcYKYgju\\"] [severity \\"CRITICAL\\"] [ver \\"OWASP_CRS/3.3.0\\"] [tag \\"application-multi\\"] [tag \\"language-multi\\"] [tag \\"platform-multi\\"] [tag \\"attack-sqli\\"] [tag \\"OWASP_CRS\\"] [tag \\"capec/1000/152/248/66\\"] [tag \\"PCI/6.5.2\\"] [tag \\"paranoia-level/2\\"] [hostname \\"dispatcher-ecm-demo.rnd.gemaltodigitalbankingidcloud.com\\"] [uri \\"/dispatcher/domains/default/users/defaultUser/applications/defaultApp/register\\"] [unique_id \\"ZVymQ_3bisqLPPTC9IuAtAABWAA\\"]"],"handler":"proxy-server","stopwatch":{"p1":777,"p2":3259,"p3":189,"p4":607,"p5":219,"sr":49,"sw":0,"l":0,"gc":0},"response_body_dechunked":true,"producer":["ModSecurity for Apache/2.9.4 (http://www.modsecurity.org/)","OWASP_CRS/3.3.4"],"server":"WAF","engine_mode":"ENABLED"}}
original request passing

123456789

FCM:XXXXXXXX-YYYYYYY

modified request expected to be blocked.

FCM:XXXXXXXX-YYYYYYY

SecRuleRemoveById 942440
SecRule REQUEST_COOKIES|!REQUEST_COOKIES:/__utm/|!REQUEST_COOKIES:/_pk_ref/|REQUEST_COOKIES_NAMES|ARGS_NAMES|ARGS|XML:/[local-name()!='RegisterRequest']|XML:/[local-name()='RegisterRequest']/*[local-name()!='notificationProfiles'] "@rx (......)"
"id:XXX_942440,
phase:2,
block,
capture,
t:none,t:urlDecodeUni,
msg:'SQL Comment Sequence Detected',
logdata:'Matched Data: %{TX.0} found within %{MATCHED_VAR_NAME}: %{MATCHED_VAR}',
tag:'application-multi',
tag:'language-multi',
tag:'platform-multi',
tag:'attack-sqli',
tag:'OWASP_CRS',
tag:'capec/1000/152/248/66',
tag:'PCI/6.5.2',
tag:'paranoia-level/2',
ver:'OWASP_CRS/3.3.0',
severity:'CRITICAL',
setvar:'tx.anomaly_score_pl2=+%{tx.critical_anomaly_score}',
setvar:'tx.sql_injection_score=+%{tx.critical_anomaly_score}'"


bubblesort.py added

Added python code for bubble sort. commented the code

@ADARSHULTIMATE Sure, I will make the changes right away


let-line-go-through-element-center

Hi @davidwang1988, thank you for the PR.
This seems incomplete.

You may have a try. I have used this in my project and it works.
On 08/16/2020 21:56, anseki wrote:
Closed #155.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Could you explain that?

The code could figure out the start and end points' coordinate instead of the 8 possible points which provided in current code.
Through the two points the leaderline will go through the centers of the two elements.
Especially, it looks very nice when the two elements are border radius 50%
On 08/17/2020 13:08, anseki wrote:
Could you explain that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

My code may need optimization, but the feature - go through element center, is useful and required in my opinion
On 08/17/2020 13:30, DavidWang wrote:
The code could figure out the start and end points' coordinate instead of the 8 possible points which provided in current code.
Through the two points the leaderline will go through the centers of the two elements.
Especially, it looks very nice when the two elements are border radius 50%
On 08/17/2020 13:08, anseki wrote:
Could you explain that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Do you mean that the PR makes the line connect to center of specific element instead of specifying startSocket/endSocket options?


Add eslint-plugin-import

Add eslint-plugin-import to help clean up imports
https://github.com/import-js/eslint-plugin-import
Config for Sen but can be adapted to be generic
"import/order": [
      "error",
      {
        "groups": ["external", "index"],
        "pathGroups": [
          {
            "pattern": "src/components/**/*",
            "group": "internal"
          },
          {
            "pattern": "src/core/**/*",
            "group": "external",
            "position": "after"
          }
        ],
        "newlines-between": "always"
      }
    ]

I've actually started to dislike this 🙃


Setup PWA

Create manifest.json
POC about screenshots on manifest
Prepare icons
Add service worker

[x] Manifest.json
[ ] Installable
[ ] Custom Install Launcher
[ ] Basic cache static file: JS, CSS, Images

Already use next-offline but seems not working at all.
Need to find another alternative


[Discussion] Add the ability to control concurrency of functions

Description
In the actual running of the function, we may need to control the concurrency of the function.
The current serving engines of OpenFunction - Knative Serving and Dapr - both have the ability to control runtime concurrency. We need to express this in the Function CRD in some concise and explicit way (annotations, properties, etc.)
Additional context
Reference:
https://knative.dev/docs/serving/autoscaling/concurrency/
https://docs.dapr.io/operations/configuration/control-concurrency/

for all these configs, are you planning to expose it as it is or have the OpenFunction own-defined parameters/annotations? like:
autoscaling.openfunction.io/target: "200" or openfunction.io/app-max-concurrency: "1"
then map to different annotations based on the serving engine used by the user.
openFunction has different serving engines. The users probably don't want to know and learn what is running underneath. Maybe it will be good if OpenFunction can abstract them and provide documentation for them.
And a global Config will be very useful as well.

for all these configs, are you planning to expose it as it is or have the OpenFunction own-defined parameters/annotations? like: autoscaling.openfunction.io/target: "200" or openfunction.io/app-max-concurrency: "1"
then map to different annotations based on the serving engine used by the user.
openFunction has different serving engines. The users probably don't want to know and learn what is running underneath. Maybe it will be good if OpenFunction can abstract them and provide documentation for them.
And a global Config will be very useful as well.

Good Point, we'll considering this @tpiperatgod @wanjunlei

If it is possible to pass these parameters through as much as possible I think it is the better solution, but maybe we have to use a high-level wrapper to handle them?
I will try to make a rough option first for reference and we can also talk about this in meeting.

Yeah.

As a temporal solution maybe, can we allow adding label and annotation to the serving.
for example:
apiVersion: core.openfunction.io/v1alpha2
kind: Function
metadata:
  name: hello-world-go
  labels:
    environment: dev
spec:
  version: "v1.0.0"
  image: "lizzzcai/sample-go-func:v0.4.0"
  imageCredentials:
    name: push-secret
  port: 8080 # default to 8080
  build:
    builder: openfunction/builder:v1
    env:
      FUNC_NAME: "HelloWorld"
      FUNC_TYPE: "http"
    srcRepo:
      url: "https://github.com/lizzzcai/openfunction-samples.git"
      sourceSubPath: "hello-world-go"
  serving:
    runtime: Knative # default to Knative
    labels:
      environment: dev
    annotations:
      # knative
      autoscaling.knative.dev/minScale: "3"
      # dapr
      dapr.io/enabled: "true"
      dapr.io/app-id: "hello-world-go"

This can help for Knative serving backend. We can use knative annotations, and even enable dapr as well. Which can probably help this issue as well: https://github.com/OpenFunction/OpenFunction/issues/173
Once OpenFunction has its own abstracted parameters, you can overwrite the annotation for knative serving.

@lizzzcai For now we can use pod template to add labels and annotations for now, and yes we'll add a unified param as serving top-level param to control this.

https://github.com/OpenFunction/OpenFunction/pull/181 address this issue by adding labels/annotations to pod metadata.
We also consider adding a unified param to control the concurrency of different runtimes later.

Soft limit has been supported by the target annotation https://openfunction.dev/docs/concepts/function_scaling_trigger/, while hard limit is not supported yet

Soft limit has been supported by the target annotation https://openfunction.dev/docs/concepts/function_scaling_trigger/, while hard limit is not supported yet

Would it be appropriate to put the "hard limit" in the FAQ?

Would it be appropriate to put the "hard limit" in the FAQ?

Sure, would you help to move FAQ in github to a FAQ section in https://openfunction.dev/docs/reference/ ?

@tpiperatgod @wanjunlei can openfunction support adding dapr annotations mannually now?
to support  https://docs.dapr.io/operations/configuration/control-concurrency/

@tpiperatgod @wanjunlei can openfunction support adding dapr annotations mannually now? to support https://docs.dapr.io/operations/configuration/control-concurrency/

openfunction can support this

OK, then this issue can be closed by adding FAQ for both knative and dapr in https://openfunction.dev/docs/reference/faq


Improve the color schemes page in the SUI

Summary of the Pull Request

[x] Clicking a color scheme in the list view immediately takes you to the page to edit the scheme
[x] Adding a new scheme immediately takes you to the page to edit the new scheme
[x] 'delete' and 'set as default' buttons have been moved to the edit scheme page

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed
The schemes page still works, keyboard navigation also works

Screenshots:

A lot of the 'currently selected scheme' stuff has still been left in because we want to continue to allow keyboard navigation.

Closes  #14289?

Closes https://github.com/microsoft/terminal/issues/14289?

Thanks! Updated the description


Review style implementation for each lib

As a developer I should make sure that all the libs are implementing Style correctly and that the example app builds for Android and iOS.
zendesk_flutter
upvoty
money
decision
user_account
info_carousel
Acceptance criteria
1 - Reference the commit that implements Style lib in each library.

zendesk flutter
https://github.com/tiki/zendesk_flutter/pull/27

money
https://github.com/tiki/money/pull/27

decision
https://github.com/tiki/decision/pull/33

user_account
https://github.com/tiki/user_account/pull/25

info_carousel
https://github.com/tiki/info_carousel/pull/22


How can I initially have the drawer open?

Hi,
I have tried having the drawer initially open, but have not seen the possibility.
You can tell me how to do it and if I can't, you can implement this functionality.
Thanks.
Regards,
Jose.

Duplicate of #12


[FEAT] API to fetch Single Testimonial  #255

Description
Fetch a single testimonial by Id.
Acceptance Criteria

The endpoint should be accessible at GET /api/v1/testimonials/:testimonial_id

User should be authenticated to access the endpoint.

The endpoint should return a 404 status code if the testimonial does not exist.

The endpoint should return a 401 status code if the user is not authenticated.

The endpoint should return a 200 status code if the testimonial is fetched.

The endpoint should return a single testimonial.

The endpoint must accept access token Authorization: Bearer:

Purpose
Be able to view a single testimonial and leave a comment.
Requirements

Implement API endpoint for single testimonial.

Testimonial should be fetched successfully if it exists.

Handle unexpected errors and return the appropriate status code.

User must be authenticated to access this endpoint.

The endpoint must accept access token Authorization: Bearer:

API Endpoint
 GET /api/v1/testimonials/: testimonial_id

Response
{
      "message": "Testimonial fetched successfully",
      "status_code": 200,
      "data": {
        "id": "String",
        "author": "String",
        "testimonial": "String",
        "comments": Array,
        "created_at": "DateTime",
  }
}

Error Responses
{
  "status_code": 404,
  "message": "Testimonial not found"
}

{
  "status_code": 401,
  "message": "Unauthorized "
}

Expected Outcome
User should be able to view Single Testimonial and all the comments associated with it.
Tasks

[ ] Develop API endpoint to view Testimonial.

[ ]  Return proper error cases.

[ ]   Write tests for the API endpoint.

Testing

Unit tests should be written for the API endpoint.

Integration tests should be written for the API endpoint.

slackid: iotstudent
This issue is linked to an already approved issue Link


docs: added DatePicker styles

GitHub Issue: #1063
PR Type
What kind of change does this PR introduce?

Documentation content changes

Description

Adds documentation for DatePicker styles and lightweight styling keys.

PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[ ] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[ ] Tested WASM
[ ] Tested MacOS
[ ] Contains No breaking changes

If the pull request contains breaking changes, commit message must contain a detailed description of the action to take for the consumer of this library. As explained by the Conventional Commits specification

Waiting on #1097 before merging this.

Closed since this will be addressed by https://github.com/unoplatform/Uno.Themes/pull/1118


Recursive call causes runtime error

The following code causes a runtime error.
    import Graphics.Element exposing (..)
    import Dict exposing (..)

    main =  let
        dict = update (ActOnSecondLevel "somekey" "secondkey" (\a -> a ++ "b")) Dict.empty
      in show dict

    type alias FirstLevelKey = String
    type alias SecondLevelKey = String
    type alias SomeData = String
    type alias FirstLevelDict = Dict FirstLevelKey SecondLevelDict
    type alias SecondLevelDict = Dict SecondLevelKey SomeData
    type Action =
        ActOnFirstLevel FirstLevelKey (SecondLevelDict -> SecondLevelDict)
        |  ActOnSecondLevel FirstLevelKey SecondLevelKey (SomeData -> SomeData)

    update : Action -> FirstLevelDict -> FirstLevelDict
    update act model =
        case act of
            ActOnFirstLevel  key1 updater ->
                Dict.insert key1 (updater (Maybe.withDefault Dict.empty (Dict.get key1 model))) model
            ActOnSecondLevel  key1 key2 updater ->
                update (ActOnFirstLevel key1 (\p ->  Dict.insert key2 (updater (Maybe.withDefault "DefaultData" (Dict.get key2 p))) p ) ) model

Are you sure that this should terminate normally? Elm does not prevent you from writing nonterminating recursions. For example, this program:
import Graphics.Element

main = Graphics.Element.show (loop "a")

loop = loop

also leads to a runtime error.
Is your program of this kind, or are your recursive calls legitimate and you would be entitled to get something different from runtime error?

Thanks for the quick reply.
I'm not at all familiar with the compiler but.
It doesn't do much recursion.
The expected result was :
 Dict.fromList [("somekey",Dict.fromList [("secondkey","DefaultDatab")])]

Which i actually do get when for instance i write it like this and 'update' then calls 'update2' ( the same function but in a different scope)
update : Action -> FirstLevelDict -> FirstLevelDict
update act model =
    case act of
        ActOnFirstLevel  key1 updater ->
            Dict.insert key1 (updater (Maybe.withDefault Dict.empty (Dict.get key1 model))) model
        ActOnSecondLevel  key1 key2 updater ->
            **update2** (ActOnFirstLevel key1 (\p ->  Dict.insert key2 (updater (Maybe.withDefault "DefaultData" (Dict.get key2 p))) p ) ) model

update2 : Action -> FirstLevelDict -> FirstLevelDict
update2 act model =
    case act of
        ActOnFirstLevel  key1 updater ->
            Dict.insert key1 (updater (Maybe.withDefault Dict.empty (Dict.get key1 model))) model
        ActOnSecondLevel  key1 key2 updater ->
            update2 (ActOnFirstLevel key1 (\p ->  Dict.insert key2 (updater (Maybe.withDefault "DefaultData" (Dict.get key2 p))) p ) ) model

This basicly 'avoids' the javascript pattern of
  update: while (true) { var _p0 = act;  ... continue; ...}
It seems to be something with tail recursion.

Yes, it's a compiler bug with tail recursion. Your original update function is currently translated to the following JavaScript:
   var update = F2(function (act,model) {
      update: while (true) {
         var _p0 = act;
         if (_p0.ctor === "ActOnFirstLevel") {
               var _p1 = _p0._0;
               return A3($Dict.insert,
               _p1,
               _p0._1(A2($Maybe.withDefault,
               $Dict.empty,
               A2($Dict.get,_p1,model))),
               model);
            } else {
               var _p2 = _p0._1;
               var _v1 = A2(ActOnFirstLevel,
               _p0._0,
               function (p) {
                  return A3($Dict.insert,
                  _p2,
                  _p0._2(A2($Maybe.withDefault,
                  "DefaultData",
                  A2($Dict.get,_p2,p))),
                  p);
               }),
               _v2 = model;
               act = _v1;
               model = _v2;
               continue update;
            }
      }
   });

The problem is the line
               act = _v1;

Correct there would be
               _p0 = _v1;

So somehow, a renaming is not carried through enough. I think @JoeyEremondi wrote the tail recursion optimization pass. Maybe he can quickly identify where the compiler is forgetting a rename.

I'll take a look, but Evan modified my TR code quite heavily, so I'll have
to take some time to figure out what's going on.
On Feb 9, 2016 10:22 AM, "Janis Voigtländer"<EMAIL_ADDRESS>wrote:

Yes, it's a compiler bug with tail recursion. Your original update
function is currently translated to the following JavaScript:
var update = F2(function (act,model) {
update: while (true) {
var _p0 = act;
if (_p0.ctor === "ActOnFirstLevel") {
var _p1 = _p0._0;
return A3($Dict.insert,
_p1,
_p0._1(A2($Maybe.withDefault,
$Dict.empty,
A2($Dict.get,_p1,model))),
model);
} else {
var _p2 = _p0._1;
var _v1 = A2(ActOnFirstLevel,
_p0._0,
function (p) {
return A3($Dict.insert,
_p2,
_p0._2(A2($Maybe.withDefault,
"DefaultData",
A2($Dict.get,_p2,p))),
p);
}),
_v2 = model;
act = _v1;
model = _v2;
continue update;
}
}
});
The problem is the line
           act = _v1;

Correct there would be
           _p0 = _v1;

So somehow, a renaming is not carried through enough. I think
@JoeyEremondi https://github.com/JoeyEremondi wrote the tail recursion
optimization pass. Maybe he can quickly identify where the compiler is
forgetting a rename.
—
Reply to this email directly or view it on GitHub
https://github.com/elm-lang/elm-compiler/issues/1282#issuecomment-181991103
.

Oh, actually, I was just going to write I am not so sure of my diagnosis anymore.

At least the act = _v1; is not obviously wrong as I claimed. After all, _p0 will be set to act right at the beginning of the next loop iteration.

How about this: Shouldn't there be a var before _v2 = model;? Else, where is _v2 declared?

There is a var before _v2 = model .
it is declared in multi assignment mode.
   var _v1 = A2(.....),
   _v2 = model

For the time being , the following works for me
  ActOnSecondLevel  key1 key2 updater -> let
                up = identity  updater
            in
  update (ActOnFirstLevel key1 (\p ->  Dict.insert key2 (up (Maybe.withDefault "DefaultData" (Dict.get key2 p))) p ) ) model

@AZon8, about the var, yes, I was blind to that.
But now I think I know what the issue is. And it seems rather serious/deep. Here goes:
The function
               function (p) {
                  return A3($Dict.insert,
                  _p2,
                  _p0._2(A2($Maybe.withDefault,
                  "DefaultData",
                  A2($Dict.get,_p2,p))),
                  p);

refers _p0. But it is not closed over this, so as to capture the value of _p0 current at the point when that function is created. Instead, it refers to the variable _p0 in the overall while block. So when the time comes that this function (p) { ... } is applied, the content of variable _p0 will have changed. And in fact it will not anymore be the case that its _p0._2 contains a/the function one wants to apply. Bang!

BTW, in a slightly different constellation, the same problem could occur with _p2.

@evancz, this code generation bug is somewhat similar to https://github.com/elm-lang/elm-compiler/issues/1223.

Here's a simpler example showing, I think, the same problem:
import Html exposing (..)

f : List Int -> (Int -> Int) -> Int
f xs g = case xs of
   [] -> g 0
   y::ys -> f ys (\_ -> y)

main = text (toString (f [1] identity))

That program should output "1". In fact it does if one destroys tail recursion by replacing the recursive call f ys (\_ -> y) with 0 + f ys (\_ -> y). But for the tail recursive form given, the compiled program blows up (try at http://elm-lang.org/try).
And I know how to fix it! It's really simple, actually. The JS code currently compiled from the above is:
   var f = F2(function (xs,g) {
      f: while (true) {
         var _p0 = xs;
         if (_p0.ctor === "[]") {
               return g(0);
            } else {
               var _v1 = _p0._1,_v2 = function (_p1) {    return _p0._0;};
               xs = _v1;
               g = _v2;
               continue f;
            }
      }
   });

Replacing the line
         var _p0 = xs;

by
         let _p0 = xs;

fixes the program.
In general, further occurrences of var would have to be replaced by let. See, for example, my comment further above about _p2 in the other example.

I think the issue title should be changed. This is not only about causing runtime errors. Actually, this bug can even make an Elm program (keep terminating but) produce a wrong value.
Consider this program:
import Html exposing (..)

f : List Int -> (Int -> Int) -> Int
f xs g = case xs of
   [] -> 0
   [_] -> g 0
   y::ys -> f ys (\_ -> y)

main = text (toString (f [1,2] identity))

On http://elm-lang.org/try, it produces output 2. But that is mathematically wrong. (Hint: add 0 + to the recursive call to see what the correct value would be.)

Following a comment by @gyzerok on the mailing list, I tried whether catch-try could serve as a substitute for let here (the latter not existing in Safari, for example). The answer appears to be yes. For the latest example above, where Elm currently computes the wrong result, rewriting
   var f = F2(function (xs,g) {
      f: while (true) {
         var _p0 = xs;
         if (_p0.ctor === "[]") {
               return 0;
            } else {
               if (_p0._1.ctor === "[]") {
                     return g(0);
                  } else {
                     var _v1 = _p0._1,_v2 = function (_p1) {    return _p0._0;};
                     xs = _v1;
                     g = _v2;
                     continue f;
                  }
            }
      }
   });

to
   var f = F2(function (xs,g) {
      f: while (true) {
         try { throw xs } catch (_p0) {
            if (_p0.ctor === "[]") {
                  return 0;
               } else {
                  if (_p0._1.ctor === "[]") {
                        return g(0);
                     } else {
                        var _v1 = _p0._1,_v2 = function (_p1) {    return _p0._0;};
                        xs = _v1;
                        g = _v2;
                        continue f;
                     }
               }
         }
      }
   });

fixes the issue.

try/catch prevents the function that contains it from being optimized in v8 (and maybe other engines) ref. I would avoid using it just to simulate block scoping.
I'm not sure what the best solution is there, but seems like more aggressive variable name mangling (to avoid collisions in cases like this) might be an easy fix.

Yeah, Evan already indicated on the mailing list that he is going to fix this by other means than let (or, I guess, an equivalent of it).
About easy fix, no, I don't see how name mangling might help here. Note that such loops could be iterated through an arbitrary number of times. And the accumulation of values could be such that values from each iteration are needed at the end. So in general, for a fix there must be allocation of new variables going on (as let would do; IIFEs might also be an option). It's not enough to just rename existing variables. At least I am confident that I can massage my example above to twart any such attempt.

I'm going to close this issue because it is hard to follow. @jvoigtlaender, can you turn your minimal repro into a new issue that explains exactly what's going on?


Any plans to do some research on masking based removing methods?

It's a great work!
I wonder if this method can be transferred to mask based removing? As instruction sometimes cannot convey the user's accurate intention! I have tried SD-Inpaint, but its removing ability is not that good, it usually generate some unwanted objects.
Do you have some ideas about diffusion based masking removing strategy?

It's a great work! I wonder if this method can be transferred to mask based removing? As instruction sometimes cannot convey the user's accurate intention! I have tried SD-Inpaint, but its removing ability is not that good, it usually generate some unwanted objects. Do you have some ideas about diffusion based masking removing strategy?

Actually, in this version, how can you specify which object is desired to be removed? In which parameter should we explain (prompt) the script about the object that needs to be removed?

It's a great work! I wonder if this method can be transferred to mask based removing? As instruction sometimes cannot convey the user's accurate intention! I have tried SD-Inpaint, but its removing ability is not that good, it usually generate some unwanted objects. Do you have some ideas about diffusion based masking removing strategy?

@lawsonxwl  I have the same concentration as you did , and even the latest model seems to fail on the simple object removal ,using mask as guided. I have tried sd-inpaint and powerpaint_v2 . Always worse than even lama. Did you find any solutions？

We have been working on this issue recently. Today we published the preprint of our paper, you can check it at: https://yigitekin.github.io/CLIPAway/. Our method adapts Stable Diffusion to the object removal task. The code will be released soon, stay tuned!


Missing id for Client interface

It`s exposed in:
https://github.com/Automattic/socket.io/blob/master/lib/client.js#L28

@FalloutPL thanks mate!


Remember and restore previous responses by LLM

User should be able to choose different content strings on the left side and any corresponding responses from LLM should be displayed on the right side again.

Implemented.


Broken build: dropdown not working

Hello
My dropdown isn't working, here's a jsfiddle testcase

closing this and taking it to Gitter.

Reopening this issue, since there seems to be a broken build. The js file was required through semantic-ui.com/dist/semantic.js and the bower build doesn't seem to be working for me, either.

Maybe consider using travis to run a linter check?

Don't mean to sound like an ass, but I really do think the current build in master is broken, and thus the bower builds.
This is a high priority issue
cc: @jlukic

@amingilani Good spot!
Merging this into #4523.


location should be Assets

If you look at the release zip files for the cores, you'll notice that the assets directory is actually Assets, not assets. This mostly doesn't matter that much since most people are running this on their SD cards with case insensitive FAT filesystems. But I'm also running this on my NAS as a back up on a case sensitive filesystem, and it results in some files being in assets and others in Assets.

great catch thank you


Plaso no longer installed after update-sift

Starting from the SIFT appliance, ran REMnux install, rebooted, ran update-sift, ran update-remnux, rebooted.  After reboot, plaso is no longer found (log2timeline.py: command not found).
Attempted to install manually:
sansforensics@siftworkstation:~$ sudo apt-get install python-plaso
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
python-plaso : Depends: python-dfvfs but it is not going to be installed
Depends: pytsk3 but it is not going to be installed
Recommends: pyelasticsearch but it is not installable
E: Unable to correct problems, you have held broken packages.
Showing the relavant(?) lines in sift-install.log (can provide if needed):
The following extra packages will be installed:
libtsk libtsk-dev
The following packages will be REMOVED:
python-dfvfs python-plaso pytsk3
The following packages will be upgraded:
libtsk libtsk-dev sleuthkit
3 upgraded, 0 newly installed, 3 to remove and 3 not upgraded.
Need to get 7,493 kB of archives.
After this operation, 36.7 MB of additional disk space will be used.
Get:1 http://ppa.launchpad.net/sift/stable/ubuntu/ trusty/main sleuthkit amd64 <IP_ADDRESS>-1sift1~trusty [4,808 kB]
Get:2 http://ppa.launchpad.net/sift/stable/ubuntu/ trusty/main libtsk-dev amd64 <IP_ADDRESS>-1sift1~trusty [2,166 kB]
Get:3 http://ppa.launchpad.net/sift/stable/ubuntu/ trusty/main libtsk amd64 <IP_ADDRESS>-1sift1~trusty [519 kB]
Fetched 7,493 kB in 14s (503 kB/s)
(Reading database ... 239039 files and directories currently installed.)
Removing python-plaso (1.3.0-2ppa1~trusty) ...

Please re-run the bootstrap script from the github repository. It should correct these problems.

Same here as of a second ago:

ERROR: Install Failure: python-plaso (Error Code: 100)
ERROR: Install Failure: pytsk3 (Error Code: 100)
ERROR: Install Failure: mantaray (Error Code: 100)

This was done using update-sift

This has temporarily been fixed. Reverted to using Sleuthkit from GIFT repository (it does not support libewf)
To fix this, you may have to force remove sleuthkit and then re-run the update and try and re-install plaso.

You'll need to run apt-get update, you may also need to run apt-get clean and apt-get purge to clear out the caches.

Giving this a go now...thanks so much Erik.

Thanks Erik, log2timeline is back!  No ewf support in tsk just as you mentioned.
Followed the steps you outlined above:
sudo apt-get remove sleuthkit
sudo apt-get update
sudo apt-get clean
sudo apt-get purge
(and for good measure, I also ran sudo apt-get autoremove)
wget --quiet -O - https://raw.github.com/sans-dfir/sift-bootstrap/master/bootstrap.sh | sudo bash -s -- -i -s -y

Good to hear. I'm working on the libewf problem.

On Aug 20, 2015, at 09:55, esecrpm<EMAIL_ADDRESS>wrote:
Thanks Erik, log2timeline is back! No ewf support in tsk just as you mentioned.
Followed the steps you outlined above:
sudo apt-get remove sleuthkit
sudo apt-get update
sudo apt-get clean
sudo apt-get purge
(and for good measure, I also ran sudo apt-get autoremove)
wget --quiet -O - https://raw.github.com/sans-dfir/sift-bootstrap/master/bootstrap.sh | sudo bash -s -- -i -s -y
—
Reply to this email directly or view it on GitHub.

Confirming that all things are looking good...even Mantaray got installed...woo hoo!  Thanks Erik...always appreciate your hard work.


DNS Zone

Is there any way that the DNS Zone nameservers can be exposed when we make a request to Get a zone?
The Azure SDK for .net supports this https://github.com/Azure/azure-sdk-for-net/blob/master/src/ResourceManagement/Dns/DnsManagement/Generated/Models/ZoneProperties.cs#L53
Thanks
Paul

@garimakhulbe it looks like this property is introduced in DNS swagger's new 2016-04-01 version. https://github.com/Azure/azure-rest-api-specs/blob/master/arm-dns/2016-04-01/swagger/dns.json#L1073-L1079
It appears like we'll need to make a release with newly generated code.

How soon can a new release happen?

today EOD.

great! thank you.

any news?

@isamuelson it is under code review at https://github.com/Azure/azure-sdk-for-go/pull/350 , we'll probably tag it later today, feel free to grab the code from there and start using in the meanwhile.

@isamuelson Please check the v3.0.0-beta release.


Dropping a procedure in beforeMigrate

Which version and edition of Flyway are you using?
6.2.4
If this is not the latest version, can you reproduce the issue with the latest one as well?
N/A using latest version
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Java API
Which database are you using (type & version)?
MySQL 5.7 tried with both driver version 8.0.19  and 5.1.47
Which operating system are you using?
Mac OS Catalina 10.15.3
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
#-------------- FIX SCHEMA CHARSET AND COLLATION ------------------------
DROP procedure if exists update_schema_char_set;
DELIMITER $$

CREATE PROCEDURE update_schema_char_set()
  BEGIN
      IF (@@character_set_database != 'utf8mb4'OR @@collation_database != 'utf8mb4_unicode_ci') THEN
        ALTER DATABASE ${SM_CAL_DB_NAME} CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci;
      END IF;
  END $$
DELIMITER ;

CALL update_schema_char_set();
DROP procedure if exists update_schema_char_set;

What did you expect to see?
executes successfully
What did you see instead?
org.flywaydb.core.api.FlywayException: Unable to parse statement in db/migration/beforeMigrate.sql at line 11 col 12: Incomplete statement at line 11 col 12:

CALL update_schema_char_set();
DROP procedure if exists update_schema_char_set;

Also, commenting first DROP procedure if exists update_schema_char_set; seems to fix the problem

I believe this has been fixed in Flyway 6.3.0. Could you update and try again please?

(I think it's a duplicate of #2687)

Can confirm fixed in 6.3.0


Do you have more example for ControlSend method

Thank you for the sequencesender!
ss.Load("[ControlSend, , {Space}, ahk_class Notepad]") doesn't seem to work for me?  Is there a special setup?

I found the issue.  It will only work with ahk_class method  I was using ahk_exe.  Is it possible to add hwd or exe?


Add a minesweeper game extension

A Minesweeper browser extension is a lightweight add-on for web browsers that brings the classic puzzle game Minesweeper to your web browsing experience. Here are the key features and characteristics typically found in such an extension:
User Interface:
Minimalistic Design: The interface is simple and intuitive, closely resembling the classic Minesweeper game. It usually consists of a grid of squares, each of which can be clicked to reveal either a number or a mine.
Grid Customization: Users can often customize the grid size and the number of mines. Common grid sizes include beginner (9x9 with 10 mines), intermediate (16x16 with 40 mines), and expert (30x16 with 99 mines).

Already added to my repository #187


why the function have 'self' param in  view/core.py?

def check_datasource_perms(
self, datasource_type: str = None, datasource_id: int = None
) -> None:
def check_slice_perms(self, slice_id):
"""
Check if user can access a cached response from slice_json.
This function takes `self` since it must have the same signature as the
the decorated method.

"""
form_data, slc = get_form_data(slice_id, use_slice_data=True)
datasource_type = slc.datasource.type
datasource_id = slc.datasource.id
viz_obj = get_viz(
    datasource_type=datasource_type,
    datasource_id=datasource_id,
    form_data=form_data,
    force=False,
)
security_manager.assert_datasource_permission(viz_obj.datasource)

This is to show that it is a class method and this param can be used to access other class members.


Closed via #210

Closed via #210


Move the CRD controllers to a single controller

Having multiple controllers contributing to the same configmap is racy.
We had an occurrence in CI where both bfd profile and addresspool were received, and the first write was slower than the second, resulting in a malformed configmap.
Here, we reduce everything to a single controller watching for all the resources, making the logic being mono-goroutine and removing the risk for races.

LGTM


Merged regulation of liquid surface tension into surfactant homeostas…

…is. #24222

@balhoff CI failed twice, not sure why. Does it have something to do with the removal of dcterms:conformsTo? Thanks.

@raymond91125 how did that happen that go-pattern-conformance.ttl was edited? It's autogenerated and shouldn't be changed manually.

I guess merging terms results in some tricky Protege behavior with the external file. I would recommend just not committing any changes that appear in files outside of go-edit.obo. Could you redo the PR and not include the changes to that file?

I think protege did that. I was merging a regulation term into a 'non-regulation' term.


unchecked_use_of_nullable_value for a checked nullable callback

When analyzing the following file with NNBD enabled:
int Function()? foo;

int callFoo() => foo != null ? foo() : 0;

I get the following error:
  error • An expression whose value can be 'null' must be null-checked before it can be dereferenced. • test.dart:3:32 • unchecked_use_of_nullable_value

This error seems bogus, since we are specifically are checking foo != null. Note that doing the check in an if statement also exhibits this error.
Tested with 2.12.0-110.0.dev.

Above, foo is a field; unfortunately, those don't type promote. There's a discussion of that here: https://github.com/dart-lang/sdk/issues/35525.
cc @leafpetersen @stereotype441 wrt improving the diagnostics we provide in these cases.


Update file-send/receive commands

the current implementation expects flags for name and path

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 35.00%. Comparing base (689c89a) to head (36e4b6d).
Report is 202 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #367      +/-   ##
==========================================
+ Coverage   34.32%   35.00%   +0.68%
==========================================
  Files          26       26
  Lines        2057     1637     -420
==========================================
- Hits          706      573     -133
+ Misses       1249      962     -287
  Partials      102      102

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ping @mudler


Use default asset path for frontend apps that use it

Trello: https://trello.com/c/oNEtjIVp/99-serve-static-assets-from-www-hostname
These apps have all been modified to now use the default assets
directory for storing their assets so can have the custom paths removed.

Can we remove that option and where it's used?

Chatted off github, don't think there's anything further that can be removed.


Login using BCeID

Access to environments requested.

Tomorrow we'll need to paste the JSON here from this process. https://bcgov.github.io/sso-requests/my-requests?id=2014&mode=edit

Tomorrow we'll need to paste the JSON here from this process. https://bcgov.github.io/sso-requests/my-requests?id=2014&mode=edit

@rgreensp Have you gotten the JSON to paste here?

Dev: {
"confidential-port": 0,
"auth-server-url": "https://dev.oidc.gov.bc.ca/auth",
"realm": "onestopauth-basic",
"ssl-required": "external",
"public-client": true,
"resource": "connectivity-intake-2014"
}

Test:
{
"confidential-port": 0,
"auth-server-url": "https://test.oidc.gov.bc.ca/auth",
"realm": "onestopauth-basic",
"ssl-required": "external",
"public-client": true,
"resource": "connectivity-intake-2014"
}

Rachel needs to email Tim about instruction language and "are we good?" based on last email asking for information.

Testing is part of SSO express package. https://github.com/bcgov/cas-template-app/tree/develop/packages/sso-express

Should test if BCeID Business works in the BCeID basic box.


Install: Fix directory path for prometheus install.sh

Why are these changes needed?

Right now the /install/prometheus/install.sh script is assuming the directory path is /install/prometheus/. However, the main README instruction is asking users to run this script from the top directory which will result the following errors:
+ monitor_dir=../../config/prometheus
+ pushd ../../config/prometheus
./install/prometheus/install.sh: line 14: pushd: ../../config/prometheus: No such file or directory

Therefore, we need to make sure monitor_dir is set to the right directory without depending on the user's current directory path.
Related issue number

None
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

This is a nice catch and thanks for the fix. Prometheus support is recently added and community may not have enough testing on user guide. Really appreciate your findings!

This is a nice catch and thanks for the fix. Prometheus support is recently added and community may not have enough testing on user guide. Really appreciate your findings!


dev -> master

Web app was deployed to IPFS:

Private Gateway: ipfs.kira.network/ipfs/bafybeicwpce7vde55awsuz2jc4nfgkryjrvjjgeyrkhrar726iiwpwpq2i
Public Gateway: ipfs.io/ipfs/bafybeicwpce7vde55awsuz2jc4nfgkryjrvjjgeyrkhrar726iiwpwpq2i

This PR is obsolete, dev branch is ahead of your branch.


Updating readme from branch example

Updates
I updated the readme file. Please review and accept

Thoughts
I probably, shouldn't, but okay. 👍


Support relative datetime in bulkdata.QueryData and bulkdata.LoadData

NOTE: This branch has the feature/bulkdata branch merged into it since some of that work was needed for testing.  Please merge the feature/bulkdata PR first which should simplify the diff for reviewing this PR.
Critical Changes
Changes

The new relative_date key can be used in a mapping to specify that a date or datetime field should be handled as a relative date which stores the value as an integer of seconds from a timedelta.
The relative_day_offset option adjusts the datetime.now() date used to determine relative dates.  This allows capturing a data set now as if it were captured N days ago.

Issues Closed

Replaced by #2015


Update SHELLCHECK_PATH's default to the new default from bash-lsp 3.2.0

Fixes #39
The alternative is to just comment out SHELLCHECK_PATH line since then it will default to "shellcheck" as well but I figured the default behavior will be more obvious this way.

Thanks. I think it makes sense to follow server's default.


Appstudio update test-component-pac-laor

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-laor-on-pull-request has failed.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
16 seconds
clone-repository

❌ Failed
26 seconds
build-container

✅ Succeeded
8 seconds
show-summary

Failure reasontask build-container has the status "Failed":
Storing signatures
STEP 2/4: COPY asf asdfsa
Error: building at STEP "COPY asf asdfsa": checking on sources under "/workspace/source": copier: stat: "/asf": no such file or directory

test

test

/retest


Add a pause after Vault starts

I came across an issue where Vault would break it's state if it was
started, then immediatly restarted.
It seems as thought Vault didn't auto-unseal and join the raft cluster
before the restart happened.
To fix this, I added a pause to give Vault enough time to get going.

LFTM


UI Not showing good

Is your feature request related to a problem? Please describe.
the ui is not showing coorectly please asign the task to me under GGsoc"24

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

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

Hello I am interested to work on the issue.


Sync

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ
Complexity Δ

.../seedu/address/logic/commands/AddBuyerCommand.java
85.18% <ø> (ø)
8.00 <0.00> (ø)

...seedu/address/logic/commands/AddSellerCommand.java
85.71% <ø> (ø)
8.00 <0.00> (ø)

...edu/address/logic/commands/DeleteBuyerCommand.java
81.48% <100.00%> (ø)
7.00 <0.00> (ø)

...du/address/logic/commands/DeleteSellerCommand.java
80.76% <100.00%> (ø)
7.00 <0.00> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
31.25% <ø> (ø)
3.00 <0.00> (ø)


[Contoso] Update widgets.json

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Contoso.WidgetManager


Optimized view in iFrame is broken

This Issue tracker is only for reporting bugs and tracking code related issues.
Before posting, please make sure you check community.jitsi.org to see if the same or similar bugs have already been discussed. General questions, installation help, and feature requests can also be posted to community.jitsi.org.
Description
Using an old version I got an optimized view with mute and camera toggle only, see here: https://prnt.sc/11gl46z
Current behavior
Now I got such one: https://prnt.sc/11gl66x
Expected Behavior
https://prnt.sc/11gl46z
Possible Solution

Steps to reproduce
Use newest build DEB package
Environment details
Any idea how to fix that?

Wrong repo and use the community forum for questions as stated in the remplate.

Okay, wrote it into the forum. Really hoping for help.


Added Maximum Product of Two Elements in an Array (Java Solution)

Hey Avinash,
Please review the code and merge the PR on 1st Oct :)
Thanks!

@aditya-517  you can contribute to this repo by adding resume -https://github.com/avinash201199/Hacktoberfest-Guide


SUBMARINE-1144. Replace tfjob and pytorchjob sub-charts with training operator

What is this PR for?

Replace tfjob and pytorchjob sub-charts with training operator.
It is tested that training operator can run on Kubernetes version v1.21.10.
However, it fails on Kubernetes version v1.16.1
What type of PR is it?
Improvement
Todos

[x] - add training operator helm charts yaml
[x] - remove tfjob and pytorch job  from helm charts

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-1144
How should this be tested?

Follow the README and start the submarine
Create experiments

Screenshots (if appropriate)

Questions:

Do the license files need updating? No
Are there breaking changes for older versions? Yes
Does this need new documentation? Yes

@Kenchu123 Why the PR title contains a hashtag?

hi @Kenchu123
Please change PR title to SUBMARINE-1144. Replace tfjob and pytorchjob sub-charts with training operator

@MortalHappiness  @xunliu I have modified the PR title. Thanks!

@MortalHappiness I have removed tfjob. Thanks.

@Kenchu123 I tried to run your PR but got this error. Did you see the same error in your local machine before?

@MortalHappiness No, I hadn't seen the error before. It seems to be successful on my local machine.
I'm using k8s version v1.21.10.

@MortalHappiness
It seems like there are two resources (deployments) use the same volume. Can you check which volume is? So that we can know where is the problem.

@Kenchu123 I finally successfully run it on my local machine after set timeoutSeconds to 10 in livenessProbe and readinessProbe. Can you set these two values? Thank you.

@MortalHappiness I have added timeoutSeconds.


Bad loggingService import path

Try on version 2.20, but the path for import loggingService is invalid, we need to use server/core/logs/loggingService

Nice catch, thanks @bastienlm for your contribution! 🥳


Navigation Usage sample does not work?

Describe the bug
The navigation sample usage does not actually work, as rememberSlideDistance() cannot be called from the lambda, it needs to be in a composable function
To Reproduce
Steps to reproduce the behavior:
Implement the usage sample
Expected behavior
It works

Fixed in version 0.7.1.
Now you don't need rememberSlideDistance().
Remove rememberSlideDistance() or declare Dp value.


Some user snippets don't want to appear in the snippets list

VSCode Version: 1.14.2
OS Version: win7x64

Steps to Reproduce:
Here is my snippet for getElementsByClassName in javascript.json file:
"getElementsByClassName": {
  "prefix": "cla",
  "body": [
    "getElementsByClassName(\"$1\");$0"
  ],
  "description": "getElementsByClassName"
}

So, as you can see in GIF, VS Code doesn't want to include my snippet right after typing, for example, document.body., and it appears only after removing 1 or more letters from "cla" and starting again.
Should be fixed.
GIF:

duplicate of https://github.com/Microsoft/vscode/issues/22205

@jrieken omg. This problem for me is unbearable. I've read https://github.com/Microsoft/vscode/issues/22205, but I didn't really get what you wrote in the last message:

We won't trigger snippets after trigger character because they are kind of private to the extensions that have registered for it.
Sorry, my English is not perfect, maybe that's why.

I really don't understand how everyone can deal with it. 70-80% of all my custom snippets goes after the . (dot). I write my own snippets because I don't like basic snippets.
Consider this:
If I need to get elements by class name from any element:

the basic snippet system offers you to type some (2, 3 or more) letters, then you will see your snippet in the list (it can be on the 3rd, 5th or whatever place in the list) and you need to press some down arrows, and after that finally press Enter... and it will expand into "dry" method getElementsByClassName, without parenthesis, without quote marks (as you would expect)
the custom snippet system offers you everything! You know EXACTLY what letters you need to type, how many letters (1, 2 or 3). You don't need to press Arrow buttons. Just 2-3 letters and Enter... and it becomes into method you wrote it yourself - getElementsByClassName("$1");$0

It's like to compare basic Windows notepad with VS Code.
And, as you can see in the GIF, snippets list will include custom snippet after removing one letter: cl --> c --> again cl --> and it will appear in the list. I really don't get this trigger algorithm.
How can we stay with basics, if it really makes coding harder for everyone?
If you don't want to implement this "showing custom snippets after dot" feature by default, can you at least make an option in the settings, so we can turn it to true?
Thanks in advance!

and it will expand into "dry" method getElementsByClassName, without parenthesis, without quote marks (as you would expect)

There are setting for JavaScript and TypeScript to change this (    "typescript.useCodeSnippetsOnMethodSuggest": true and "javascript.useCodeSnippetsOnMethodSuggest": true). When set method/function don't complete "dry" but with the invocation operator and has arguments pre-filled (as snippets). That setting might help you.

How can we stay with basics, if it really makes coding harder for everyone?

See when typing a dot in TypeScript or JavaScript not "basic snippets" are computed but the truth, which is member completion. Many people have complained that we add to that finite set "random" snippets, consider that not everyone has the majority of snippets designed to come after the dot.
I hope you understand that there are folks that want the opposite and that we cannot change default behaviour with ease. I believe that the above settings can make you life a lot easier. Also not that there are different ways to insert a snippet. One is IntelliSense, but there is also tab-completion, the insert-snippet-action, and the ability to bind a keybinding to a snippet.

@jrieken, sorry for bothering you, but for me it's really huge problem.

"javascript.useCodeSnippetsOnMethodSuggest": true

Well, there is no such setting:

"typescript.useCodeSnippetsOnMethodSuggest"

Adding just parenthesis...
Can you say where I can find all basic snippets, so I could edit them?
That will be the only thing that would help me.

I hope you understand that there are folks that want the opposite and that we cannot change default behaviour with ease.

That's why I asked you with all my respect, if you can, to do an OPTION (in settings), that by default would be false, but if someone wants to see custom snippets right after typing the dots - turn it to true and will get them.

the insert-snippet-action

do you mean that?

and the ability to bind a keybinding to a snippet.

Too bad, but there are not so many keys on keyboard, so that's not gonna work.


Safety for 32b and 64b on Windows

I am not sure on licensing issues from stackoverflow, but can we add some code like this to the startJVM for windows?
Ported from https://stackoverflow.com/questions/1345632/determine-if-an-executable-or-library-is-32-or-64-bits-on-windows

import jpype
import struct
import sys

def _checkJVMArch(jvmPath):
    IMAGE_FILE_MACHINE_I386=332
    IMAGE_FILE_MACHINE_IA64=512
    IMAGE_FILE_MACHINE_AMD64=34404

    is64 = sys.maxsize > 2**32
    with open(jvmPath, "rb") as f:
        s=f.read(2)
        if s!=b"MZ":
            raise RuntimeException("JVM not valid")
        else:
            f.seek(60)
            s=f.read(4)
            header_offset=struct.unpack("<L", s)[0]
            f.seek(header_offset+4)
            s=f.read(2)
            machine=struct.unpack("<H", s)[0]

            if machine==IMAGE_FILE_MACHINE_I386:
                if is64:
                    raise RuntimeError("JVM mismatch, python is 64 bit and JVM is 32 bit.")
            elif machine==IMAGE_FILE_MACHINE_IA64 or machine==IMAGE_FILE_MACHINE_AMD64:
                if not is64:
                    raise RuntimeError("JVM mismatch, python is 32 bit and JVM is 64 bit.")
            else:
                raise RuntimeError("Unable to deterime JVM Type")

# Success if correct
_checkJVMArch(jpype.getDefaultJVMPath())

# Fail if wrong
_checkJVMArch(r"c:\Program Files (x86)\Java\jre7\bin\client\jvm.dll")

thats a cool hack we should introduce also for linux (bsd?) and osx.


Error while trying to launch the game

I cannot launch the game with the mod installed
using version 0.0.3
This is the error being spammed:
[Error  : Unity Log] InvalidOperationException: Burst failed to compile the function pointer `Void Test(ReadableViewConstants* ByRef, Int32 ByRef, ReadableShaderVariablesXR*)`
Stack trace:
Unity.Burst.BurstCompiler.Compile (System.Object delegateObj, System.Reflection.MethodInfo methodInfo, System.Boolean isFunctionPointer, System.Boolean isILPostProcessing) (at <367QbHx4fe5DvLAiMy3esiKtTvxDudxuBe>:0)
Unity.Burst.BurstCompiler.Compile (System.Object delegateObj, System.Boolean isFunctionPointer) (at <367QbHx4fe5DvLAiMy3esiKtTvxDudxuBe>:0)
Unity.Burst.BurstCompiler.CompileFunctionPointer[T] (T delegateMethod) (at <367QbHx4fe5DvLAiMy3esiKtTvxDudxuBe>:0)
LethalPerformance.Patches.Patch_HDCamera..cctor () (at _/Patches/Patch_HDCamera.cs:9)
Rethrow as TypeInitializationException: The type initializer for 'LethalPerformance.Patches.Patch_HDCamera' threw an exception.
(wrapper dynamic-method) UnityEngine.Rendering.HighDefinition.HDCamera.DMD<UnityEngine.Rendering.HighDefinition.HDCamera::UpdateShaderVariablesXRCB>(UnityEngine.Rendering.HighDefinition.HDCamera,UnityEngine.Rendering.HighDefinition.ShaderVariablesXR&)
UnityEngine.Rendering.HighDefinition.HDRenderPipeline.UpdateShaderVariablesXRCB (UnityEngine.Rendering.HighDefinition.HDCamera hdCamera, UnityEngine.Rendering.CommandBuffer cmd) (at <b8abf62aeac646cea5c5cde032ff9314>:0)
UnityEngine.Rendering.HighDefinition.HDRenderPipeline.UpdateGlobalConstantBuffers (UnityEngine.Rendering.HighDefinition.HDCamera hdCamera, UnityEngine.Rendering.CommandBuffer cmd) (at <b8abf62aeac646cea5c5cde032ff9314>:0)
UnityEngine.Rendering.HighDefinition.HDRenderPipeline.ExecuteRenderRequest (UnityEngine.Rendering.HighDefinition.HDRenderPipeline+RenderRequest renderRequest, UnityEngine.Rendering.ScriptableRenderContext renderContext, UnityEngine.Rendering.CommandBuffer cmd, UnityEngine.Rendering.HighDefinition.AOVRequestData aovRequest) (at <b8abf62aeac646cea5c5cde032ff9314>:0)
UnityEngine.Rendering.HighDefinition.HDRenderPipeline.Render (UnityEngine.Rendering.ScriptableRenderContext renderContext, System.Collections.Generic.List`1[T] cameras) (at <b8abf62aeac646cea5c5cde032ff9314>:0)
UnityEngine.Rendering.RenderPipeline.InternalRender (UnityEngine.Rendering.ScriptableRenderContext context, System.Collections.Generic.List`1[T] cameras) (at <e27997765c1848b09d8073e5d642717a>:0)
UnityEngine.Rendering.RenderPipelineManager.DoRenderLoop_Internal (UnityEngine.Rendering.RenderPipelineAsset pipe, System.IntPtr loopPtr, UnityEngine.Object renderRequest) (at <e27997765c1848b09d8073e5d642717a>:0)

Please send the full log

Should be fixed in 0.0.4, reopen if not :)


interactive checkmarks

Please select the area your bug applies to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Describe the bug
Checkmarks are not interactive in the Dendron Preview in vscode
To Reproduce
Steps to reproduce the behavior:

Open an MD file with - [ ] checkmark(s)
Open the Dendron Preview
Click the checkmark(s) in the preview..

Expected behavior
Checkmarks state should be toggled
Desktop (please complete the following information)

OS: Win10 21H1
VSCodium: 1.62.3

Thank you for the issue. There is a duplicate issue that goes over this.
Interactive checkboxes are something that were supported with our legacy preview but haven't been ported over to our current preview.
We are currently working on the stability of the preview command, and once we get to a stable point we can consider porting this feature into the current preview.
I'll mark this as a duplicate. Please upvote #1571 and continue any further discussions there if you have any more outstanding concerns.


Doc link from Servo -> Sevos is a 404

Looks like the link to the Servos class here goes to a 404:
http://johnny-five.io/api/servo/
It goes to http://johnny-five.io/api/servo/servos when it should go to http://johnny-five.io/api/servos
It works correctly within the wiki. I remember you telling me that the website is generated from the wiki, but I'm not sure how that's done, if you give me a pointer I can look into a fix!

Ugh. The content in question is:
For working with multiple servos check out the [Servos](servos) class.

...in the wiki. There must be a simple way to fix this :|

Indeed there is! Looks like Led just uses a fully qualified path:
- [Led.Digits](https://github.com/rwaldron/johnny-five/wiki/led.digits)
- [Led.Matrix](https://github.com/rwaldron/johnny-five/wiki/led.matrix)
- [Led.RGB](https://github.com/rwaldron/johnny-five/wiki/led.rgb)
- [Leds](https://github.com/rwaldron/johnny-five/wiki/leds)

Fixed! Thanks for filing <3
http://johnny-five.io/api/servo/


File Specific Tolerances yielded fewer PSMs

Analyzing dmorgen's data with default precursor/product tolerances of 5/20 yield 23662 PSMs, but using the calibration suggested tolerances of 6.8/42.07 yield only 21242 PSMs.
Spectra: QEP1_SpikeIn_230914_1_3ng_270914-calib.mzml
Databases: crap_2.0GPTMD.xml AND Ecoli_UP_Jan2018-Human_UP_Jan2018GPTMD.xml
Toml and filespecific tomls below.

SearchTaskconfig.txt

QEP1_SpikeIn_230914_1_3ng_270914-calib.txt

i guess that it might be variable b/c the score cutoff for global search and individual file search will be different.

There was only one file searched, so there should be no disparity.

might point to unexpected parameter override (.e.g changed tolerance). could be a bug.

Is the problem just that the suggested ppm tolerances give worse results than the default tolerances? or some other bug?

Correct. The suggested ppm tolerances give worse results than the default tolerances.

@zrolfs  please check to see if this issue persists in 0.0.262?

Confirmed this is still an issue in 0.0.262.
Before calib: 8214 PSMs within 1% FDR
After calib: 6842 PSMs within 1% FDR

Merged with #518

I think these are separate issues; may need to keep an eye out that consistency and increased psms are both resolved.


Doucmentation error w.r.t combined security information registration and AAD Audit logs

[Enter feedback here]
This page states the following:
The events logged for combined registration are in the Authentication Methods category in the Azure AD Audit logs. However I believe this to be incorrect. Authentication Methods is under the Service filter and not under the Category filter. This need to be corrected.
Also the UI is completely different in my AAD tenant.

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

ID: 209406f7-3ef9-7424-a11d-46146b232906
Version Independent ID: 53f76c21-5d6d-54d2-8166-d6c3e0d2d508
Content: Troubleshoot combined registration - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-registration-mfa-sspr-combined-troubleshoot.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

Any updates?
Peter Johnson | Senior Cloud Infrastructure Consultant |<EMAIL_ADDRESS>| +27 83 306 0019 | http://www.inobits.com
1.This email disclaimer applies to the original email, all attachments and any subsequent emails sent by Netsurit (Pty) Ltd (“the Company”). 2. This e-mail is intended only for the named person or entity to which it is addressed and contains valuable business information that is privileged, confidential and/or otherwise protected from disclosure. If you received this e-mail in error, any review, use, dissemination, distribution, printing or copying of this e-mail is strictly prohibited and may be unlawful and / or an infringement of copyright. Please notify us immediately of the error via e-mail to<EMAIL_ADDRESS>and delete the e-mail from your system, retaining no copies in any media. 3. No employee or agent is authorized to conclude any binding agreement on behalf of the Company with another party or give any warranty by email without express written confirmation by an authorized representative or a director of the Company and nothing in this email shall be construed as a legally binding agreement or warranty or an offer to contract. The Company will not be responsible for any damages suffered by the recipient as a result of the recipient not taking cognisance of this principle. 4. Please note that the recipient must scan this e-mail and any attached files for viruses and the like. The Company accepts no liability of whatever nature for any loss, liability, damage or expense resulting directly or indirectly from the access of any files which are attached to this message. 5. Views and opinions are those of the sender unless clearly stated as being that of the Company's. 6. Any email addressed to the Company shall only be deemed to have been received once receipt is confirmed by the Company orally or in writing. An automated acknowledgment of receipt will not suffice as proof of receipt by the Company. 7. This email disclaimer shall be governed by the laws of South Africa.

From: Marilee Turscak - MSFT<EMAIL_ADDRESS>Sent: Wednesday, November 18, 2020 8:05:01 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Peter Johnson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Doucmentation error w.r.t combined security information registration and AAD Audit logs (#66265)
[X]
@peterjoinobitshttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpeterjoinobits&data=04|01|peterjo%40inobits.com|ebd9b8f9bb464ab766be08d88bec7907|5da736f8138e4b36adab2236f975b6f1|0|0|637413195057838811|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Yf032wiH4DFCq64c5Ud9GtXxunfvE3VYgrBEjdfhMgc%3D&reserved=0
Thanks for your feedback! We will investigate and update as appropriate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F66265%23issuecomment-729857873&data=04|01|peterjo%40inobits.com|ebd9b8f9bb464ab766be08d88bec7907|5da736f8138e4b36adab2236f975b6f1|0|0|637413195057838811|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PLjSPGoa2E4DpT0Lsy6xWC7fQVA3HZNZ%2FZiLpZEniSM%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMYWMCMDL46ZUIJVDQZVWD3SQQEE3ANCNFSM4TZZ5NMQ&data=04|01|peterjo%40inobits.com|ebd9b8f9bb464ab766be08d88bec7907|5da736f8138e4b36adab2236f975b6f1|0|0|637413195057848805|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=kXI65OL25dppnNB94Yabvfn3bRIKhdaNV9N3T77pYBE%3D&reserved=0.


keyof losing type information when used with generics extend

🔎 Search Terms
extends generics keyof
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about types

⏯ Playground Link
https://www.typescriptlang.org/play?#code/C4TwDgpgBASg9gdygXigbwFBW1BBLAE2AAsAuKAOwFcBbAIwgCcBuDAXww1EigGE4ANrQop0WHPiJlKtBi3HZiEPAHNiwctXpNWHDAGM4FAM7AoxkDVEBlS3UEAKAJRdw0AKoU8ARyrRUmDhQANoWNAC6mrI6ClAAjOQAhhQgsUYCIAAKjHBg5KaMeBQqupz6AonGxlAAQlR4AgRMADwAKgB8YrFgVHQCePpQjFQUDnjkAKIAHsCMifrAzQDWECBwAGZQrQA05rNFKp0APlAOziidWnJOTmJBQXibDtwQG1B4KMioAOQFB9+3QL3YGGEyCCAAOgEcBUYyhEGKJBcwOwbCgEAExmgQJROFBxnBUJhY2cyJRHCCHD06xGCzwRlwczAkEYbXRMwRBGq8CQJ34QhoFHaDjo9UaTHIdQaTVZHUBsVF0qYEOGo2+khIAOYUAA9DqoAA5OBmOBLKAAWk6AEFGCpaAiTZsXlBfvtit93tUKMaoJVjKoKIk+tBgHAoGBEnMaBBgEwoG9nd8zrdkJdooxbidprN5osVmtNjs9oViu1vhCHAAmADMABYAKwuPQUCBIKXi1k84VOFUjBzqwialwtttimXNLvOXtqjXESsA3X6gDyAGkLejGDlGBgR7Uxy1Jz3VcmLlBG5wgA
💻 Code
type Row = {
    width: number;
}

type Column = {
    width: number;
    height: number;
}

const sym = Symbol()
type Unique = {
    [sym]: number;
    1: any
    onlyProp: string;
}

class Builder<T> {

    public run(i: Extract<keyof T, string> | (() => number)) {
        if (typeof i === 'string') {
            console.log(i.length)
        } else {
            console.log(i())
        }
    }
}

function wrapper<T extends Row | Column>(builder: Builder<T>) {
    builder.run('width'); // Not ok -> Argument of type 'string' is not assignable to parameter of type '(() => number) | Extract<keyof T, string>'.(2345)
}

new Builder<Row>().run('width')
new Builder<Row>().run('width2') // OK - error
new Builder<Row>().run(() => 5)

🙁 Actual behavior
Argument of type 'string' is not assignable to parameter of type '(() => number) | Extract<keyof T, string>'.(2345)
🙂 Expected behavior
No errors for existing string properties
Additional information about the issue
No response

No type information is “lost”, the Extract is just deferred because T is generic.

I suggest writing it like this to avoid our limited ability to reason about generic conditional types

@andrewbranch thanks, we are using this way too, but I was expecting to see Extract to behave the same.
@fatcerberus thanks, is there anywhere more detailed explanation why it's not an issue?

Extract is implemented as a distributive conditional type, specifically T extends U ? T : never; such types have to be evaluated for each member of a union individually, so in general they can't be resolved until the compiler knows exactly what type T is.  Until then, nothing is considered assignable to it.


opensfm.io.json_dump minification

Minifying the reconstruction.meshed.json file can dramatically reduce the filesize: In one example, my 267MB file shrunk to 63MB. The original file failed to load in the web-viewer: My minified version loaded immediately.
Is there a reason the whitespace is left in (perhaps for human-readability)? If not, I'd be happy to pull-request this change.

For those who just want to try minifying their files too, you can run:
import json
with open('reconstruction.meshed.min.json', 'w') as fh:
    fh.write(
        json.dumps(
            json.loads(open('reconstruction.meshed.json', 'r').read())
        )
    )

This problem and solution both affected me too.  Ideally the reconstruction viewer would also warn you in some way that the file was too big rather than hanging.  Certainly I couldn't get a file to load at 486MB but the minified 188MB worked fine.

@j6k4m8 the white space is indeed only for human-readability. Happy to accept that PR and remove it, thanks. A human can use a json formatter if he needs to read it.


Bump version 1.1.1

Bump new version of the module 1.1.1

includes a fix regarding checking autoload file

Wait for https://github.com/PrestaShop/ps_themecusto/pull/22 validation and merge before merging this one!

Thanks @jolelievre


Make items in filter popup of a categorical column clickable

Closes Caleydo/lineupjs#508

change to 'Select All'


[HOLD for payment 2024-10-17] [$125] [Search v2.3] IOS - Emoji is cut off in the dropdown button

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: 9.0.41-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

There is at least one saved search

Launch New Expensify app
Go to Search
Tap on the dropdown button
Tap 3-dot menu next to the saved search
Tap Rename
Enter an emoji and save it
Dismiss the dropdown modal

Expected Result:
The emoji will not be cut off in the dropdown button
Actual Result:
The emoji is cut off in the dropdown button
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/44146f16-1207-4e7a-99eb-1b23098f749f
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021840184545557049275
Upwork Job ID:<PHONE_NUMBER>557049275
Last Price Increase: 2024-09-29
Automatic offers:

ahmedGaber93 | Reviewer | 104219452
Nodebrute | Contributor | 104219453

Issue OwnerCurrent Issue Owner: @dylanexpensify

Production:
https://github.com/user-attachments/assets/1d2fac2a-988a-4315-b9c8-0991ec2e546e

Issue stemming from https://github.com/Expensify/App/pull/49528

I think that @Nodebrute's proposal looks reasonable, but want @dannymcclain to double-check first

The solution presented by @Nodebrute seems legit but I would suggest one small improvement.
I can see that we have a dedicated style called label here: https://github.com/Expensify/App/blob/d47e1562fa6adb9968623512ee59af0fa6a4acba/src/styles/index.ts#L371
and it is defined like this:
label: {
      fontSize: variables.fontSizeLabel,
      lineHeight: variables.lineHeightLarge,
}

That tells me we do want this specific lineHeightLarge combined with fontSizeLabel.
So I think instead of directly adding lineHeightLarge in the component we should just use styles.label. This style is used in multiple other places as well.

The solution also seems good for me.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01fdb5331a4bc5b415

Awaiting 👍🏼 from @ahmedGaber93 since he'll have to review the PR. I set the price lower because this seems like a very simple issue and it affects only one platform.

@Nodebrute's proposal LGTM! And not affected on others platforms
🎀 👀 🎀 C+ reviewed

Do we solve this https://github.com/Expensify/App/issues/49890#issuecomment-2383200176 here too? If yes, we can handle it in the PR.

Do we solve this https://github.com/Expensify/App/issues/49890#issuecomment-2383200176 here too? If yes, we can handle it in the PR.

I'll let @roryabraham weigh in on that one/decide—it's pretty unrelated but if it's a super simple update it'd be nice to get it knocked out.

Do we solve this https://github.com/Expensify/App/issues/49890#issuecomment-2383200176 here too? If yes, we can handle it in the PR.

Nah, let's do a separate issue for that. @dannymcclain can you create it, and label it Bug, External plz?

PR is approved by me. Last step is for @dannymcclain to review and merge, then it's just awaiting deploy and payment

I think this is a minor fix and no regression test require.

Tomorrow!

Done!


Fix IPC http server error logging

What does this PR do?
Add a header on all the logs from the http server
Motivation
Make these logs easier to understand

With a stack depth of 4 we can have this message:
2018-10-25 12:18:14 EDT | ERROR | (server.go:1763 in serve) | Error from the agent http API server: http: TLS handshake error from <IP_ADDRESS>:54547: remote error: tls: bad certificate

It's a bit better but we don't know that server.go is part of the stdlib. If we use a separate logger, I guess we can change the format to specify it

other suggestion: extend seelog's API in our logger

Codecov Report

Merging #2494 into master will decrease coverage by 0.04%.
The diff coverage is 4.16%.

@@            Coverage Diff             @@
##           master    #2494      +/-   ##
==========================================
- Coverage   56.61%   56.57%   -0.05%
==========================================
  Files         379      379
  Lines       24049    24071      +22
==========================================
+ Hits        13616    13618       +2
- Misses       9514     9534      +20
  Partials      919      919

Impacted Files
Coverage Δ

pkg/config/log.go
1.49% <0%> (ø)
:arrow_up:

pkg/util/log/log.go
62.74% <4.34%> (-4.13%)
:arrow_down:

pkg/forwarder/domain_forwarder.go
92.91% <0%> (+1.57%)
:arrow_up:


Hi

My Public Address to Receive LTC ltc1qyktw4dp6qnsfg9dhs4ey6qphemz34s4rr9n2de
Pay me via Trust Wallet: https://link.trustwallet.com/send?coin=2&address=ltc1qyktw4dp6qnsfg9dhs4ey6qphemz34s4rr9n2de&amount=2.01

Hi


Subspec for App Extension to define AF_APP_EXTENSIONS

I added empty subspec for App Extension with   #define AF_APP_EXTENSIONS 1
So in you podfile you can just add some lines to fix extension compilation:
target 'YourExtensionTargetName' do
    pod 'AFNetworking/AppExtension'
end

pod 'AFNetworking'

This should be addressed once I get #2737 in. Hopefully this week.


Getting errors when trying to connect multiple entities in many-to-many relation

Bug description
Getting errors when trying to connect multiple entities in many-to-many relation.
I'm doing it the same as it is described in official documentation.
Here is my code:
async addProduct(@Body() { attributes, ...addProductDto }: any) {
   return this.prisma.product.create({
     data: {
       ...addProductDto,
       attributes: {
         connect: attributes.map((id: number) => ({ id })),
       },
     },
     include: {
       attributes: true,
     },
   });
 }

The error:
Error:
Invalid `this.prisma.product.create()` invocation in
/Users/helios/Documents/other projects/docker+nginx+jenkins+nest/api/src/app.controller.ts:20:32

  17
  18 @Post('/products')
  19 async addProduct(@Body() { attributes, ...addProductDto }: any) {
→ 20   return this.prisma.product.create({
         data: {
           name: 'T-Shirt',
           price: 20,
           attributes: {
             connect: [
               {
                 id: 2
                 ~~
               }
             ]
           }
         },
         include: {
           attributes: true
         }
       })

Unknown arg `id` in data.attributes.connect.0.id for type AttributesOnProductsWhereUniqueInput. Available args:

type AttributesOnProductsWhereUniqueInput {
  productId_attributeId?: AttributesOnProductsProductIdAttributeIdCompoundUniqueInput
}

How to reproduce
call method with this payload:
{
    "name": "T-Shirt",
    "price": 20,
    "attributes": [2]
}

Expected behavior
No response
Prisma information
model Product {
  id         Int                    @id @default(autoincrement())
  name       String
  price      Int
  attributes AttributesOnProducts[]
}

model Attribute {
  id       Int                    @id @default(autoincrement())
  name     String
  products AttributesOnProducts[]
  variants Variants[]
}

model AttributesOnProducts {
  productId   Int
  product     Product   @relation(fields: [productId], references: [id])
  attributeId Int
  attribute   Attribute @relation(fields: [attributeId], references: [id])

  @@id([productId, attributeId])
}

Environment & setup

OS: macOS
Database: Postgres
Node.js version: 14.17.3

Prisma Version
4.9.0

I figured out. I looks weird imo.
async addProduct({ attributes, ...addProductDto }: any) {
    return this.prisma.product.create({
      data: {
        ...addProductDto,
        attributes: {
          create: attributes.map((id: number) => ({
            attribute: {
              connect: {
                id,
              },
            },
          })),
        },
      },
      include: {
        attributes: true,
      },
    });
  }

And the output of selecting is also weird, with huge nesting:
{
        "id": 3,
        "name": "T-Shirt",
        "price": 20,
        "attributes": [
            {
                "id": 2,
                "name": "Size",
                "variants": [
                    {
                        "id": 1,
                        "value": "S",
                        "attributeId": 2
                    },
                    {
                        "id": 2,
                        "value": "M",
                        "attributeId": 2
                    },
                    {
                        "id": 3,
                        "value": "L",
                        "attributeId": 2
                    },
                    {
                        "id": 4,
                        "value": "XL",
                        "attributeId": 2
                    },
                    {
                        "id": 5,
                        "value": "XXL",
                        "attributeId": 2
                    }
                ]
            }
        ]
    },

So original problem is resolved?

And the output of selecting is also weird, with huge nesting:

The attributes being included is expected as you asked for that. If each individual attribute and then all the variants should be included by default I am not sure. Can you make any sense of that?


(BUG) [RN] The "We were unable to complete the signup" error message isn't displayed after canceling sign up process

Precondition:
The "Welcome to good dollar" page is opened
Steps to reproduce:

Click the "Sign up (Create new wallet)" button
Click the "Agree & Sign up with Google" button
Pay attention to the lack of error message

Actual result: The "We were unable to complete the signup" error message isn't displayed after canceling sign up process
Expected result: The "We were unable to complete the signup" error message should be displayed after canceling sign up process
Note: the issue is reproduced only for native web/ also reproduced for login

Checked on  native 1.3.1-1
Desktop: Windows 10 // Google Chrome  86.0.4240.183
iPhone 8 //13.3.1
Video: https://www.screencast.com/t/l5sKsn1s2u7

@sirpy @serdiukov-o-nordwhale faced that error on QA env

@sirpy fixed https://www.screencast.com/t/FW3AtEXZ5XrO
Checked on  QA 1.10.1-1 & native
Desktop: Windows 10 // Google Chrome  86.0.4240.183
Samsung S9 // Android 10


[CSV] Support Export CSV on native NewDot apps

This is a follow up to https://github.com/Expensify/App/issues/44576
Problem: Because of the lib we use to download files on mobile native we can't export the CSV file the same way we do it for web, since the lib doesn't allow a request body.
https://github.com/RonRadtke/react-native-blob-util/tree/master#:~:text=Download Manager can only support GET method%2C which means the request body will be ignored.
Problem: Find a way to download the CSV on the native apps.

@filip-solecki can you comment on this issue so we can assign it to you as well?

HI! I am Filip from SWM and I'd like to work on this!

@rushatgabhane assigning you for C+ if we need it here.

Confirming payment as follows:

$250 to @rushatgabhane for the C+ review

Regression tests have been added centrally for this project. Feel free to go ahead and request, Rushat. Closing!

$250 approved for @rushatgabhane