Add TTL option to cloudflare_api_token

Current Terraform and Cloudflare provider version
v1.0.10, 3.8.0
Description
API tokens have TTL option available via Dash and API. Please make this option available via Terraform as well.
Use cases
Security generally requires TTL on API tokens to be defined as part of rotation policy.
Potential Terraform configuration
resource "cloudflare_api_token" "api_token_create" {
  name = "api_token_create"

  policy {
    permission_groups = [
      data.cloudflare_api_token_permission_groups.all.permissions["API Tokens Write"],
    ]
    resources = {
      "com.cloudflare.api.user.${var.user_id}" = "*"
    }
  }

  condition {
    request_ip {
      in     = ["<IP_ADDRESS>/32"]
      not_in = ["<IP_ADDRESS>/32"]
    }
    ttl {
      not_before = "2018-07-01T05:20:00Z"
      expires_on = "2019-07-01T05:20:00Z"
    }
  }
}

References
No response

Would be great to have short lived narrow scoped tokens auto-generated to hide away complexity of generating these via UI/API from users/system.


Added pre-receive-app and pre-destroy hooks

Unauthorized users should not be allowed to destroy apps or push to them. Adding pre-receive-app probably makes pre-build-buildstep redundant.

10/10 with rice. Thank you for your suggestion.


Disable the calendar when attribute is readonly

When Date and Time attribute marked as read only on the page and developer used customization as
show icon true and
show icon inside false
then calendar icon should not still remain clickable. it should also disabled.

Hi Amit, this should be fixed in marketplace release 1.4.0


uninstall_cmd fix in Windows

make uninstall_cmd available in Windows

Can you expand a bit on why this is necessary, and how to reproduce the current issue?

Verified.


Repo should be renamed (and templates updated) to reflect consulting role of the TAG

The world "review" incorrectly describes what we do when we do it well: instead of reviewing finished (or near-finished) designs, our work here is most effective when we consult with teams designing features as soon as their design "hangs together".
Therefore, I propose we change the name of this repo to design-consulting-and-review.

I support that change.

It's a little unwieldy.  Remember the wise teachings of The Guide:

Ford flipped the switch which he saw was now marked ‘Mode Execute Ready’ instead of the now old-fashioned ‘Access Standby’ which had so long ago replaced the appallingly stone-aged ‘Off’.

If we have a problem with 'review', maybe we should remove that word.  It is specs that we examine, so removing that seems counter-intuitive to me.  How about 'spec-feedback'?

Fwiw, ”Feedback"  reads mostly as also more passive and late to me. Consult/consultation seems to better indicate the point/collaboration/advice I think @slightoff was making.  Sorry I hate to add potentially worthless bikeshedding, but I've always thought this had been kind of weird so I'm glad to see it come up.

OK, "spec-consults"?

spec-pwc

I took an action today to change it to 'design-review'.


Added email notification feature

I have made a pr to add email notification that sends a notification to user email.
https://github.com/Kritika30032002/To-Do-List-Application/assets/122620769/d9f66aeb-ee83-49d3-b061-509a8831f62d

donot use your personal email as default in website code

That can be changed with the emailJS template. I did it in order to verify and create a account in emailJS

search for other method


Build merging function

Maybe very useful to have the functionality to merge two variants.

Hi,
On which level? I guess first step is being able to compare 2 variants?

Hi Lars,
i often have the task to create a Z….-Variant by merging 2 or more other Variants, for example:
ZEUH_DB = FUNCTIONAL_DB + FUNCTIONAL_DB_ADDITION + PERFORMANCE_DB
I have tested this by merging the 3 xml’s inxml-level and it might be work.
So the only tasl is to merge XML-Content.
Tomorrow I test method/IWFND/CL_SUTIL_ODATA_MAPPER.GET_BUSINESS_DATA
Idea is to put in the xml’s to get structured data.
Best regards, Stefan
Best regards
Stefan Riedel-Seifert

Expert ABAP Development Technology | ATP Appl. Techn. Prod. and Logistics
Endress+Hauser InfoServe GmbH+Co. KG | Heinrich-von-Stephan-Str. 8 | 79100 Freiburg | Germany
Phone: +49 761 7059 115 | Mobile: +49<PHONE_NUMBER> | Fax: +49 761 7059 5 115
<EMAIL_ADDRESS>| www.infoserve.endress.com

Endress+Hauser InfoServe GmbH+Co. KG
Location: Weil am Rhein
Registered: Amtsgericht Freiburg i. Br. HRA 411272
Individual liable partner: Endress+Hauser InfoServe Verwaltungs-GmbH, Weil am Rhein
Registered: Amtsgericht Freiburg i.Br. HRB 412621
Managing director: Pieter de Koning

According to the General Data Protection Regulation,
we are obliged to inform you when collecting your personal data.
Wie comply with this information duty with the following Data Protection Statementhttps://www.de.endress.com/de/cookies-endress+hauser-website.

Disclaimer:
The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential, proprietary, and/or privileged
material. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon, this information by persons or entities
other than the intended recipient is prohibited. If you receive this in error, please contact the sender and delete the material from any computer.
This e-mail does not constitute a contract offer, a contract amendment, or an acceptance of a contract offer unless explicitly and conspicuously designated or stated as such.
Von: Lars Hvam<EMAIL_ADDRESS>Gesendet: Sonntag,3.Juni2018 19:54
An: larshp/upDOWNci<EMAIL_ADDRESS>Cc: Stefan Riedel-Seifert<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [larshp/upDOWNci] Build merging function (#11)
Hi,
On which level? I guess first step is being able to compare 2 variants?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/larshp/upDOWNci/issues/11#issuecomment-394179286, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMbiuFGzu80vhr1rbr_lNqHb37Se7U_aks5t5CKqgaJpZM4UYK3p.

For the above example I think the variants contains different checks, in this case it is quite easy.
But if 2 checks to be combined contains different configurations for the same check, then the user has to decide which one to take, or to somehow merge it

Yes, you’re right. As a mathematican i prefer the union set of all, as a first approach 😊
Best regards
Stefan Riedel-Seifert

Expert ABAP Development Technology | ATP Appl. Techn. Prod. and Logistics
Endress+Hauser InfoServe GmbH+Co. KG | Heinrich-von-Stephan-Str. 8 | 79100 Freiburg | Germany
Phone: +49 761 7059 115 | Mobile: +49<PHONE_NUMBER> | Fax: +49 761 7059 5 115
<EMAIL_ADDRESS>| www.infoserve.endress.com

Endress+Hauser InfoServe GmbH+Co. KG
Location: Weil am Rhein
Registered: Amtsgericht Freiburg i. Br. HRA 411272
Individual liable partner: Endress+Hauser InfoServe Verwaltungs-GmbH, Weil am Rhein
Registered: Amtsgericht Freiburg i.Br. HRB 412621
Managing director: Pieter de Koning

According to the General Data Protection Regulation,
we are obliged to inform you when collecting your personal data.
Wie comply with this information duty with the following Data Protection Statementhttps://www.de.endress.com/de/cookies-endress+hauser-website.

Disclaimer:
The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential, proprietary, and/or privileged
material. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon, this information by persons or entities
other than the intended recipient is prohibited. If you receive this in error, please contact the sender and delete the material from any computer.
This e-mail does not constitute a contract offer, a contract amendment, or an acceptance of a contract offer unless explicitly and conspicuously designated or stated as such.
Von: Lars Hvam<EMAIL_ADDRESS>Gesendet: Sonntag, 3. Juni 2018 21:03
An: larshp/upDOWNci<EMAIL_ADDRESS>Cc: Stefan Riedel-Seifert<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [larshp/upDOWNci] Build merging function (#11)
For the above example I think the variants contains different checks, in this case it is quite easy.
But if 2 checks to be combined contains different configurations for the same check, then the user has to decide which one to take, or to somehow merge it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/larshp/upDOWNci/issues/11#issuecomment-394183656, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMbiuKdlAkdS9DPpRWJ3tJNRKDTXsHTsks5t5DLfgaJpZM4UYK3p.


Add floating-alt style variant in Button element

Self Checklist

[x] I wrote a PR title in English.
[x] I added an appropriate label to the PR.
[x] I wrote a commit message in English.
[x] I wrote a commit message according to the Conventional Commits specification.
[ ] I added the appropriate changeset for the changes.
[x] I wrote a unit test about the implementation.
[x] I wrote a storybook document about the implementation.
[ ] I tested the implementation in various browsers.

Windows: Chrome, Edge, (Optional) Firefox
macOS: Chrome, Edge, Safari, (Optional) Firefox

Related Issue
Fixes #0000
Summary

This PR adds a floating-alt style variant in Button element.
Details

Breaking change or not (Yes/No)

No

PR Checklist의 add changeset도 확인 부탁드립니다!


Only refresh when the document is saved, not when it is edited

Parsing operates on the saved file, not on the in-memory contents of the editor,
so triggering a refresh whenever the text changed is pointless.
Alternatively, one might wish that parsing should operate on the in-memory text,
which would be easy enough to change, I guess. However, parsing can be expensive
for very long files, so I actually like it that it isn't done too often; VS Code is
already slow enough when tons of extensions are installed.

Agree.
Merged.
Thank you for your contribution.


Perform multiple JsonRPCActions within single query

Hello!
Is there a way to perform multiple actions for a single query? I am looking to copy to clipboard, change the query, and possibly more upon the selection of an option.
Thanks!

Hi @Marlstar,
Sorry due to the way Flow Launcher communicates with JsonRPC plugins the plugin is exited after Flow Launcher detects output in stdout.
This makes sending multiple actions impossible.
You may have luck using a library like clipboard instead of relying on JsonRPCAction to copy to the clipboard.

Helll @Garulf,
I am currently working on a plugin to get passwords from Bitwarden in FlowLauncher.
I am able to use pyperclip to copy to the clipboard, but I was also aiming to clear the query to hopefully free memory (removing the passwords from memory). I am concerned about this due to the fact that when flowlauncher is reopened, the query is still there, and options (with the passwords in their action) still persist.
Do you happen to know how to clear the options so that memory is hopefully freed?
Thanks!

As stated above, use a 3rd party library to copy to clipboard and send the JsonRPCAction to clear the results.


Stack Overflow when attempting to create a circular dependency

When working on Conc Liquidity, I stupidly decided to implement a from trait for a type which depended on the type I was hoping to impl from on, and so I imported this type Q64x64 in the definition of Q128x128, but Q64x64 already depended on this type, this caused the compiler to break with
$ forc build
  Compiled library "core".
  Compiled library "std".

thread 'main' has overflowed its stack

This can be solve by deleted this dep and this use on this branch of amm

My theory would be that
multiply method of Q64x64 calls from on the class of Q128x128.
However, there is a from implementation on Q128x128 which uses Q64x64, so the circular dependency occurs even earlier with the compilation of the associated functions

This is about recursive dependencies, not recursive types. I'm sure we already have issues covering that somewhere...

This is about recursive dependencies, not recursive types. I'm sure we already have issues covering that somewhere...

Ah makes sense. I guess that's something that the collection context will fix.

This should no longer be a problem since we no longer allow circular dep definitions.


Fuzzer can get stuck on small input ranges

The fuzzer can get stuck on small input ranges like [2,100] and not return to the user. The user must close and re-open the NaNofuzz tab.

Agree, and I've also had trouble reproducing it. In the most-recent user study, which we ran in Codespaces, maybe two users ran into this problem when fuzzing small input spaces using the lcm program in nanofuzz-examples.
But I don't think any user encountered it more than once. At the time, I figured it was likely a bug in the fuzzer that we should investigate and resolve. But I haven't seen or encountered the issue in several months.

Love it! In the lab users accidentally swapped min and max way more often than we expected. Users almost always fixed it by swapping the two values when we stopped their workflow in prior versions, so we decided to do the swap automatically for them in that situation rather than interrupting them. It was a similar observation that lead to clamping, e.g., minlength. Stopping the workflow is the more traditional choice ... just at the cost of some user time and attention.


Create a template for exporting data to a google sheet

Write a template to export data to Google Sheets.

It's added in this PR: https://github.com/mage-ai/mage-ai/pull/3828


undefined variables

All variables that are used in this role are undocumented and not defined in defaults/
Added them to defaults/main.yml (still undefined):
https://github.com/CSC-IT-Center-for-Science/ansible-role-provision-vm/tree/default_variables

Thanks for reporting!
This has been addressed by these pull requests:
https://github.com/CSC-IT-Center-for-Science/ansible-role-provision-vm/pull/20
https://github.com/CSC-IT-Center-for-Science/ansible-role-provision-vm/pull/22


Cannot resolve acme-v02.api.letsencrypt.org

I have this running some time. But yesterday wanted to request a certificate for a new subdomain of mine, but received and "internal error" looking at the logs:
Saving debug log to /config/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator webroot, Installer None
An unexpected error occurred:
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/urllib3/connection.py", line 141, in _new_conn
(self.host, self.port), self.timeout, **extra_kw)

File "/usr/lib/python3.6/site-packages/urllib3/util/connection.py", line 60, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):

File "/usr/lib/python3.6/socket.py", line 745, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):

socket.gaierror: [Errno -3] Try again
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
chunked=chunked)

File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 346, in _make_request
self._validate_conn(conn)

File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 850, in _validate_conn
conn.connect()

File "/usr/lib/python3.6/site-packages/urllib3/connection.py", line 284, in connect
conn = self._new_conn()

File "/usr/lib/python3.6/site-packages/urllib3/connection.py", line 150, in _new_conn
self, "Failed to establish a new connection: %s" % e)

urllib3.exceptions.NewConnectionError: <urllib3.connection.VerifiedHTTPSConnection object at 0x7f4e5b54bf60>: Failed to establish a new connection: [Errno -3] Try again
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/requests/adapters.py", line 445, in send
timeout=timeout

File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
_stacktrace=sys.exc_info()[2])

File "/usr/lib/python3.6/site-packages/urllib3/util/retry.py", line 388, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))

urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='acme-v02.api.letsencrypt.org', port=443): Max retries exceeded with url: /directory (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4e5b54bf60>: Failed to establish a new connection: [Errno -3] Try again',))
During handling of the above exception, another exception occurred:
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='acme-v02.api.letsencrypt.org', port=443): Max retries exceeded with url: /directory (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4e5b54bf60>: Failed to establish a new connection: [Errno -3] Try again',))
Please see the logfiles in /config/log/letsencrypt for more details.
So running from the container I CAN ping the IP of the API but not the FQDN of the API..

Found the issue. On my docker machine two stale DNS servers where still configured... Removed them and now it works


Typescript package support

When I using electron-packager to package electron-quick-start-typescript,
it packaged successfully, but the out put app doesn't work well.
I try to package js version of quick-start that works well.
What should I suppose to deal with it?

You can use Electron Forge (v6), which has first-class support for Webpack. Adding TypeScript support to Electron Packager is outside of its scope, and can be added through the use of hooks (e.g., running tsc in an afterCopy hook).
If you need more assistance, please use one of the community forums.


Remove cache package from pkg/cli dependencies list

What this PR does / why we need it:
Should handle cache.ErrNotFound error in pipedStatBuilder instead of handler layer.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

/approve


Split flang.exe into flang-driver

From https://github.com/conda-forge/numpy-feedstock/issues/79#issuecomment-366313872

One idea is to create a new flang-driver package that is just the flang executable and have it depend on clangdev (similar to how we split libflang out).

cc @isuruf

Does flang.exe depend on other libraries in clangdev package?

Yes. So the run requirements for flang-driver would be clangdev and flang. The run requirements for flang would be flang-driver.

Not sure where to put this best, but since I see this issue: we should probably create a new feedstock for flang starting with LLVM 14 (code generation will not be complete, but should help shake out issues)?


Standardise user guide format

Refer here for how the user guide looks like.

In Command Syntax section point 6, you can update the sentence as below for better clarity.
Arguments must be specified in the format /{argument_name} {value} if the argument takes a value. If the argument expects no value, the format is in /{argument_name}, but if specified, the value will be ignored. Note that there must be a whitespace before /{argument_name}.


Fix deliveryMode.ExpectReplies for skills.

Description
When callling /api/messages the return type is always a InvokeResponse<T> data structure.
When a skill receives an activity with deliveryMode=expectReplies, the botframeworkAdapter caches all outbound activities and returns them as a InvokeReponse<ExpectedReplies> :
{
   "status": status code
   "body": { // type expectedReplies
      "activities": [... ]
    }
}

This is the correct speced behavior.
The bug is that the BotFrameworkHttpClient code is interpreting the body of the response as ExpectedReplies instead of InvokeResponse<ExpectedReplies>
The properties don't align and hence the activities are dropped on the floor.
The unit tests test both sides independently but with mismatched assumptions, so only when e2e is used it doesn't work. (classic metric vs english measurement situation).
This changes client to do correc thing and aligns unit tests to same assumptions as the bot framework adapter class.
Specific Changes

The response from the skill is already InvokeResponse<ExpectedReplies>, there is no need for special code to other then deserializing it as such.

Testing

Updated the unit tests to correctly mock returning InvokeResponse<ExpectedReplies>

NOTE
We should make sure this bug was not ported to other platforms

Thanks @carlosscastro and @tomlm, good catch, I need to wrap up something else today but let me double check this before we merge it, I need to check my notes and we had some issues with InvokeReplies and want to make sure we don't break other things.

Thanks @gabog, that sounds great!!!

After some testing, I think this assumption is incorrect, the Body is effectively ExpectReplies not InvokeResponse, reverting this PR, the activity returned is an InvokeResponse


Handroll Solidity assembly for optimized masked bytecode replacement

The guardedArrayReplace function (https://github.com/ProjectWyvern/wyvern-ethereum/blob/master/contracts/common/ArrayUtils.sol#L25), used by the Exchange contract to test whether the possible transaction bytecodes specified by two orders can be matched, is presently written in plain Solidity. I have not inspected the output EVM code yet, but I'm guessing it's not very efficient. At minimum, there are probably unnecessary array bounds checks, and given the particularities of the EVM, this may not be the optimal way to implement this algorithm in general (some kind of batched masking might be more efficient).
Breaking changes to the bytecode mask format will be considered if they would lead to substantial gas savings, but they must preserve the property that individual bytes can be masked/unmasked independently.

An additional bounty in WYV will be added as soon as this PR is merged.

Closed in 26b6c9f2552646b78a47b93f8d3a22c853752a6d - although if anyone has even more optimized assembly, HMU!

Issue Status: 1. Open 2. Expired

This issue now has a funding of 0.03 ETH (20.76 USD @ $692.1/ETH) attached to it.

If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
Questions? Checkout Gitcoin Help or the Gitcoin Slack
$1,512.27 more funded OSS Work available on the Gitcoin Issue Explorer


Inconsistent content

The content provided here is inconsistent with the product by regions (with Azure Government filtered) here https://azure.microsoft.com/en-us/global-infrastructure/services/ Which one is valid?

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

ID: cbc3731f-da9b-4c27-9f02-03b7ee7e13ba
Version Independent ID: f2c7122f-9af7-ff07-28c9-e182a874703c
Content: Azure Government AI and Cognitive Services
Content Source: articles/azure-government/documentation-government-services-aiandcognitiveservices.md
Service: azure-government
GitHub Login: @jglixon
Microsoft Alias: jglixon

@thuansoldier Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

<EMAIL_ADDRESS>can you pls help?

From: femsulu-MSFT<EMAIL_ADDRESS>Sent: Monday, August 6, 2018 5:25 PM
To: MicrosoftDocs/azure-docs
Cc: Julie Glixon; Assign
Subject: Re: [MicrosoftDocs/azure-docs] Inconsistent content (#12972)
@thuansoldierhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthuansoldier&data=02|01|jglixon%40microsoft.com|00f8b9fdde8844d96f6308d5fbfc4a1d|72f988bf86f141af91ab2d7cd011db47|1|0|636691983348520571&sdata=Bv0JT43OUAKtYZqQ48WgUfkde%2B9hLOpTuSNiFABLo6w%3D&reserved=0 Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F12972%23issuecomment-410893894&data=02|01|jglixon%40microsoft.com|00f8b9fdde8844d96f6308d5fbfc4a1d|72f988bf86f141af91ab2d7cd011db47|1|0|636691983348520571&sdata=op%2FWwcqiSfqBBLWtEwPraFMBtE7oVUzhj432mBrLq8g%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVLTMg8TsojFq_YmolRuDWxsM_9dKVNBks5uON58gaJpZM4VxOmC&data=02|01|jglixon%40microsoft.com|00f8b9fdde8844d96f6308d5fbfc4a1d|72f988bf86f141af91ab2d7cd011db47|1|0|636691983348530580&sdata=WMK8Y8Zee%2B39L75sC6MN0cw8JY77qWc5%2Bb4Sg6WEZrA%3D&reserved=0.

@thuansoldier - the products by region page is the ultimate source of truth.
C you elaborate on the inconsistency so that we can resolve it in the doc?


[CARBONDATA-4099] Fixed select query on main table with a SI table in case of concurrent load, compact and clean files operation

Why is this PR needed?
There were 2 issues in the clean files post event listener:

In concurrent cases, while writing entry back to the table status file, wrong path was given, due to which table status file was not updated in the case of SI table.
While writing the loadmetadetails to the table status file during concurrent scenarios, we were only writing the unwanted segments and not all the segments, which could make segments stale in the SI table
Due to these 2 issues, when selet query is executed on SI table, the tablestatus would have entry for a segment but it's carbondata file would be deleted, thus throwing an IO Exception.

What changes were proposed in this PR?
Added correct table status path as well sending the correct loadmetadatadetails to be updated in the table status file. Now when select query is fired on the SI table, it will not throw carbondata file not found exception
Does this PR introduce any user interface change?

No

Is any new testcase added?

No

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5250/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3489/

@vikramahuja1001 please change the title and update description

@akashrn5 done

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5255/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3494/

LGTM


fix(front): 进程属性中搜索进程异常 (fixed #206)

Codecov Report

Merging #224 (5a318f2) into v1.0.20-rc (281d4a0) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           v1.0.20-rc     #224   +/-   ##
===========================================
  Coverage       53.55%   53.55%
===========================================
  Files             182      182
  Lines            8975     8975
===========================================
  Hits             4807     4807
  Misses           4168     4168

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix: set diagnostic setting destination type

Set explicit destination type for diagnostic setting. Value is hard coded, as this is the only acceptable value for key vault diagnostic settings as of today.

Test passes 🎉


Add readme

cc @diasdavid @jbenet @whyrusleeping @kyledrake @mappum @richardlitt
First request is #1

@RichardLitt yep updated! I made ipfs.io and webchat.freenode.net use https

and omited the contribute gif, it's just too much blinking for reading the text below :)

Haha aw.


Feature/jen cal fs

Critical Changes
Changes
Fixed all instances of field sets to properly display all field types.
Issues Closed

Looks awesome, great work on this @jjbennett!


Global throttler per Domain for API calls

Currently we only throttle API calls per domain based per host.  Although this is useful to throttle domains hitting the system hard, but is significant operational overhead to manage these limits per domain per host.  As each time we increase the size of cluster this limit needs to be adjusted.
This issue is created to throttle domains based on aggregated request rate for the entire cluster.  Some high level requirements are:

Enforce request rates aggregated across the entire cluster
Ability to dynamically change quotas for the entire domain or for specific API call
Ability to assign priority to certain API calls like StartWorkflowExecution, Signal, etc to give precedence when system is overloaded.

https://github.com/uber/cadence/pull/3169
https://github.com/uber/cadence/pull/3161


[#951] build(operator): Add the HADOOP_VERSION for Docker image.

What changes were proposed in this pull request?
(Please outline the changes and how this PR fixes the issue.)
Why are the changes needed?
(Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.)

Fix: #951
Does this PR introduce any user-facing change?
(Please list the user-facing changes introduced by your change, including

Change in user-facing APIs.
Addition or removal of property keys.)

No.
How was this patch tested?
(Please test your changes, and provide instructions on how to test it:

If you add a feature or fix a bug, add a test to cover your changes.
If you fix a flaky test, repeat it for many times to prove it works.)

@qijiale76 Could you correct the description of pr?

@connorlwilkes Could you help review this pr if you have time?

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

Shuffle Server must use the HDFS jars. Because some configuration use HDFS jars.

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

Shuffle Server must use the HDFS jars. Because some configuration use HDFS jars.

Oh, I forgot this. This should be optimized in the later PR.

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

@zuston
Hadoop version is introduced by me in #896
I think we should bind hadoop version, even if we don't use HDFS. We know mapreduce, spark and tez depend on hadoop. Because the API is changed, some version of computing framework may not be compatibility with some verison of hadoop.
In production cluster, use the computing framework which is not  compatibility with hadoop may produce many unexpected error.

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

@zuston
Hadoop version is introduced by me in #896 I think we should bind hadoop version, even if we don't use HDFS. We know mapreduce, spark and tez depend on hadoop. Because the API is changed, some version of computing framework may not be compatibility with some verison of hadoop. In production cluster, use the computing framework which is not compatibility with hadoop may produce many unexpected error.

You are right. I have checked that all the spark distribution are all bind to hadoop.

I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird

@zuston
Hadoop version is introduced by me in #896 I think we should bind hadoop version, even if we don't use HDFS. We know mapreduce, spark and tez depend on hadoop. Because the API is changed, some version of computing framework may not be compatibility with some verison of hadoop. In production cluster, use the computing framework which is not compatibility with hadoop may produce many unexpected error.

You are right. I have checked that all the spark distribution are all bind to hadoop.

I just copy spark! ^_^


make stable against rendering errors by using multiple tools

Mediawiki (e.g. commons) use imagemagick convert in order to process image media. In order to be prone against rendering errors of such tools use at least 2 (complementary) tools to render images and compare against each other.

Related pages on commons:

https://commons.wikimedia.org/wiki/Category:PDF_files_affected_by_MediaWiki_restrictions
https://commons.wikimedia.org/wiki/Category:Images_without_thumbnails
https://commons.wikimedia.org/wiki/Category:Images_with_render_problem

Was thinking about this a bit more, and it seems like this is probably out of scope for file-metadata.
file-metadata aims to provide meta information about the file by using exif data or computer vision, and rendering faults (although a useful feature) would not be a part of meta information.
It would be better to focus on a single core competency, rather than delve into multiple things.
Possibly, we could close this and open a issue on phabricator "pywikibot script idea: Check render faults in mediawiki" ?

Perhaps the in-scope part is that file-metadata should distinguish between failures

specific to one of its dependencies
caused by the file bits being incorrect

Rather than wrapping existing exceptions from client libraries, create two tuples:
FileEncodingError = (dlib.xxxx.FooError, opencv.xxxx.Error, etc)
ConverterFailure = (dlib.xxxx.RefuseProcessing, opencv.xxxx.Blah, etc)

Then calling code can do
if isinstance(e, file_metadata.exceptions.FileEncodingError):
    print('file ... is broken: %s' % e)
elif isinstance(e, file_metadata.exceptions.ConverterFailure)::
    print('file ... can not be processed by file_metadata: %s' % e)
else:
    print('file failed processing by file_metadata for unknown reason: %s' %e)

May be the scope shifted - catimages should use any mean to get info about a file for categorization. This is one of it. So as I see it:

this rendering check is part of catimages
may be not of file-metadata but then it needs to be put into another package
(I don't think it is very high priority right now)

So it belongs to catimages todo or fudo (future todo) but IHMO it's not a separate bot - e.g. you don't want another bot script to download all files again just for render check.
Am 13. Juni 2016 06:15:59 MESZ, schrieb AbdealiJK<EMAIL_ADDRESS>
Was thinking about this a bit more, and it seems like this is probably
out of scope for file-metadata.
file-metadata aims to provide meta information about the file by using
exif data or computer vision, and rendering faults (although a useful
feature) would not be a part of meta information.
It would be better to focus on a single core competency, rather than
delve into multiple things.
Possibly, we could close this and open a issue on phabricator
"pywikibot script idea: Check render faults in mediawiki" ?

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/AbdealiJK/file-metadata/issues/37#issuecomment-225486169

Dr. Trigon

Yep, that's what I meant - it may be potentially in catimages, but not in file-metadata.

Am 13. Juni 2016 06:27:47 MESZ, schrieb John Vandenberg<EMAIL_ADDRESS>
Perhaps the in-scope part is that file-metadata should distinguish
between failures

specific to one of its dependencies (1)
caused by the file bits being incorrect (2)

Totally agree. We need to catch errors that give use info about the file (1) and raise only the ones we cannot interprete (2 - e.g. cause by true issue).
Although (1) could get involved and might need to involve machine learning (categorization per se) and training in order to separate (1) from (2).

Rather than wrapping existing exceptions from client libraries, create
two tuples:
FileEncodingError = (dlib.xxxx.FooError, opencv.xxxx.Error, etc)
ConverterFailure = (dlib.xxxx.RefuseProcessing, opencv.xxxx.Blah, etc)

Then calling code can do
if isinstance(e, file_metadata.exceptions.FileEncodingError):
   print('file ... is broken: %s' % e)
elif isinstance(e, file_metadata.exceptions.ConverterFailure)::
   print('file ... can not be processed by file_metadata: %s' % e)
else:
print('file failed processing by file_metadata for unknown reason: %s'
%e)

Did you mean:
isinst = lambda inst: isinstance(e, inst)

if True not in map(isinst(file_metadata.exceptions.FileEncodingError)):
print('file ... is broken: %s' % e)
elif ...

May be you can avoid the lambda.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/AbdealiJK/file-metadata/issues/37#issuecomment-225487054

Dr. Trigon

No. isinstance can be given a tuple.


Update Release Compilation Check

Description
This PR updates the compilation check step to improve the following:

If compilation wasn't possible the build would previously use all retries available then pass. Now, it will exit with a failure code explicitly.
If the job was cancelled it wouldn't terminate the pipeline until all retries were used. Now, it checks the job status before attempting to check compilation and will exit if cancelled.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Closing for #26609


mwc713, Android, read inputs

Seems like Android version has issues with reading. I can't unlock wallet with a password. Seems like trailing enter is there.
Issue:  qt wallet on exit can't close mwc713. Istead it shows a message that Operation takes too much time.

Fix now in the master. Nightly build has it.

Logout from QT wallet happened very quickly and unable to find a message that Operation takes too much time.


Adds debug logs to aro operator controllers

What this PR does / why we need it:
Helpful to understand whether controllers are doing anything at all.
Test plan for issue:
Run the operator locally with -loglevel debug.
Note: for this to actually work we need #2579 to be merged first.

Forgot to update few unit tests to include a logger. Should be good now.

/azp run e2e


Sample project with appcompat for android-22

So I think the issue with android-22 comes down to the fact that, if you have an sdk beyond 22 installed, the Android Studio project wizard is a bit dense when in creates the build.gradle file for the module.   Specifically it uses the later api  com.android.support libraries.
If you uninstall later sdk's then the wizard generates the correct build.gradle file for the module that works with the hidden-api.  I would like to suggest that you document the build.gradle requirements for the module for each API.  But maybe it should be obvious.  In any case for android-22 I have attached a sample project that builds with the main issue being the build.gradle for the module that will work is this:
apply plugin: 'com.android.application'

android {
    compileSdkVersion 22
    buildToolsVersion "22.0.1"

    defaultConfig {
        applicationId "com.burningthumb.simpletest"
        minSdkVersion 22
        targetSdkVersion 22
        versionCode 1
        versionName "1.0"
    }
    buildTypes {
        release {
            minifyEnabled false
            proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.pro'
        }
    }
}

dependencies {
    compile fileTree(dir: 'libs', include: ['*.jar'])
    testCompile 'junit:junit:4.12'
    compile 'com.android.support:appcompat-v7:22.2.1'
    compile 'com.android.support:design:22.2.1'
}

SimpleTest.zip

I have updated the sample project. Also, I have added Internal Accessor to make us able to retrieve internal Android resources.


404 response from ollama on prompt

Hi there - I installed llm and this project using pipx, and there seems to be an error accessing the ollama server. Here are some details - please let me know what additional information would be helpful.
% ollama list
NAME            ID              SIZE    MODIFIED
llama2:latest   7da22eda89ac    3.8 GB  3 months ago

% curl -s http://localhost:11434/api/generate -d '{"model": "llama2", "prompt":"what is 2 + 2", "stream":false}' | jq -r -c '.response'
 The answer to 2 + 2 is 4.

% pipx runpip llm freeze | grep -E 'llm|ollama'
llm==0.13.1
llm-ollama==0.2.0
ollama==0.1.4

% llm ollama list-models
llama2:latest

% llm -m llama2:latest 'How much is 2+2?'
Error: 404 page not found

Thanks for the help troubleshooting. The /api/chat endpoint gives me a 404 (and works on another machine), so that's my problem right there. No idea what's wrong since the /api/generate endpoint seems fine, but I'm closing this since the problem is clearly on my end. Thanks for the plugin!


refactor: tons of refactoring

Refactor a browser class and consolidate utils
Add a user agent generator
Move prints to each activity

Very smart modifications, I'll review that more in details soon, thanks again for your contribution 👍

Ya so far looks good

Just reverted changes on the "profiles-directory" part because it doesn't work as is. You first need to create profiles to be able to use it like that (it was "working" because you didn't quote the session name so it was ignored and always the same profile)


WIP: Add validation schema to primitives generated by 'federate enable'

This PR adds validation schema to the primitive crds created by the kubefed2 federate enable command implemented in #340.  The schema added to override and placement types is equivalent to that included in CRDs generated by kubebuilder, and the schema added to templates includes a full schema for embedded target types (vs kubebuilder's output of type: "object").

Lookup of openapi schema isn't quite right, non-core (e.g. outside of Namespaces, Secrets, etc) kube resources aren't quite as easy to map as I was expecting.

I'm reworking this to leverage vendored kubectl openapi lookup.

I have some cleanup of the generated validation that I'll be posting tomorrow.


semantic-release changelog outputs nothing

There's no getting started guide that I've found. It doesn't tell you if you did anything wrong, just fails.
Due to some docs and trial and error I have this flow:
cat << EOF > setup.cfg
[semantic_release]
upload_to_pypi=False
version_variable=version.py:__version__
EOF

cat << EOF > version.py
__version__ = '0.0.1'
EOF

git init
git add .
git commit -m "feat(*): Does all the things"
curl https://www.gitignore.io/api/python > .gitignore
pip install python-semantic-release
semantic-release changelog

Publish fails, probably due to a lack of tag?
Current version: 0.0.1
Traceback (most recent call last):
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/bin/semantic-release", line 11, in <module>
    sys.exit(main())
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/click/core.py", line 610, in __call__
    return self.main(*args, **kwargs)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/click/core.py", line 590, in main
    rv = self.invoke(ctx)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/click/core.py", line 936, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/click/core.py", line 782, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/click/core.py", line 416, in invoke
    return callback(*args, **kwargs)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/semantic_release/cli.py", line 171, in cmd_publish
    return publish(**kwargs)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/semantic_release/cli.py", line 114, in publish
    level_bump = evaluate_version_bump(current_version, kwargs['force_level'])
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/semantic_release/history/logs.py", line 35, in evaluate_version_bump
    for _hash, commit_message in get_commit_log('v{0}'.format(current_version)):
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/semantic_release/vcs_helpers.py", line 18, in get_commit_log
    for commit in repo.iter_commits(rev):
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/git/objects/commit.py", line 265, in _iter_from_process_or_stream
    finalize_process(proc_or_stream)
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/git/util.py", line 154, in finalize_process
    proc.wait()
  File "/home/halkeye/.virtualenvs/it-jira-bamboohr/local/lib/python2.7/site-packages/git/cmd.py", line 309, in wait
    raise GitCommandError(self.args, status, self.proc.stderr.read())
git.exc.GitCommandError: 'git rev-list ...v0.0.1 --' returned with exit code 128
stderr: 'fatal: bad revision '...v0.0.1'```

same with version

git tag v0.0.1
git remote add origin git@github.com:halkeye/test_test_test_test.git
git push -u origin master
halkeye@blackpanther:~/git/it-jira-bamboohr/a/i_like_pie$ semantic-release publish
Current version: 0.0.1
Creating new version..
Current version: 0.0.1
No release will be made.
Version failed, no release will be published.

date > date
git add date
git commit -m "feat(date): I love dates"
semantic-release publish

^^^ would have worked, but i entered the wrong GH_TOKEN so it failed, now I can't publish though I can changelog.
I'm thinking part of the problem might be my "feat(*): feature" message?

I tried adding a few more commits:
...
git commit -m "feat(*): dates are chewie"
...
git commit -m "feat: dates are chewie"

but changelog just shows the initial release..
I've found now that I can publish again, and changelog shows the last release.
Does that sorta describe the flow? commit, publish, changelog?

I will continue to poke at this to see if i can't make it more user friendly
#68 helps a bit

@halkeye Is this fixed by #68?

I think its safe to close. I'm having a hard time setting up the reproduction steps quickly enough to test it.


include req_body size in sizes list

Coverage increased (+0.07%) to 75.646% when pulling 8d02cd5918b621779992d1a9ce15eca3c32e91f6 on tsloughter:req_body_size into e58a099308ac14a1efad93e8583e0f8b4c3641a2 on elli-lib:develop.

maybe not relevant but how ContentLength et al work with chuncked requests?

If there is one it should be ignored. Right now this won't get a size for a chunked request. You have to use a handover elli handler for it as well anyway, right? So there would need to be a way for the handler to record sizes.

Ah yea, good point, I'll add a test after Infinity War :)

Sounds good to me. Enjoy! It was one of the least worst (loose) screen adaptations of a comic (universe) I've seen in a while. :smile:


Fix attribute error

From new disableScrollZoom prop in #16

@emilyzhong I have been to quick to merge on this one. Will check it and merge, but it should all be good now!

This could have been caught by a simple CI flow type check. Are you interested in adding a CI build? Could help this library being used by more people out there.

I don't think in that case Flow would have helped. We don't know what is expected inside restPassThroughProps as you are just passing down the props to the inner div. This error is a runtime error and would only have been spotted using a unit test for it.
For now I want to keep the CI as simple as it is, Flow type checking, unit tests and that's all. I might consider a more advanced CI pipeline in the future.

Alright, I expected flow to actually catch setting unexpected types on the div element.

We could create a test that renders the component and checks whether any warning has been logged to the console.

@n1ru4l It would not be that easy because someone might use specific attributes that are not regular HTML attributes. You would not be able to specify all possible attributes for a div.
But we should definitely think of a better way to removed unsupported props that we use in PanZoom but don't pass down. Using hooks would be a solution as you only need to deconstruct your props once, but it would force our users to have a React version compatible with hooks (>= 16.8.0).
I will investigate the value of having CI build and CD in the near future.

@n1ru4l

We could create a test that renders the component and checks whether any warning has been logged to the console.

Yes that's what I was thinking, you can help on that one if you feel like it, otherwise I will take care of it. But for now I need to go to work 😄


Document Page - Migrate to new Theme

Problem -

Plugin which we are using is outdated, or not possible to migrate in new theme
We need Customized Paging in File list, Current Plugin do not support this
We need new Inline popover to View Related File, that is not possible with Plugins
Other Sorting , Uploading File can be improved

Suggested Solution

New Design with Paging, Sorting Filtering
Support for Popover design
Support for Customized View, Filtered View

Technical details-

Extensive Time required to migrate / develop / test new design
new API structure Required for design
Paging Filtering is tricky part as Parent GCP do support in limited way

Please all add their requirement here
@vinpahilwani @sureshsirvi

@mitulpanch what is my role in this?

I remember you suggested that, Popover of this page should come under Project Detail for Company documents without redirect
anything like this should be covered here

There are 4 total Views

Company View - which requires Company Id to open
Common View - Common Files anybody in company can access
Vault View - shows some Private files with Roles
Public View - Any file saved here is Public to Internet, mostly used in Website section , File Uploads, Blog Images etc, bascially all files which are required with public url without login/ authentication

I suggest there should 3 Menu for documents

Company and Common, as they both required at same time
Common view has some common files that needs to share between all Companies
Vault View - for user who can access vault
Public View - show all files used as public role

Design -

Design will not have old plugin
Following theme

Not suggesting any other plugins, just plane design will give as advantage to use it on any page
Multiple select and Copy, Cut Paste action will be good
Search , filter ,sorting, paging all present to search file easily

Company selection will be at top and required, for Company view
On Project Section , Company view will be auto opened if no heavy plugins used

Kindly show who added this file i.e. added by

not possible - that is going to Log entries

Kindly show who added this file i.e. added by

Ok

Requirement:

Documents section is required on project page.
To maintain project wise documents, we need project wise folder / TAG to find out project related documents. Mean Whenever we open the project we can get documents relating to such project.

I had discussion with mitul and he says if I allow project files then project files will not be displayed in files section which will create confusion among users.
I asked him then we should remove file section and only allow files on project, he does not agree with this idea also.
Considering these things will confuse users i do not approve this

I had discussion with mitul and he says if I allow project files then project files will not be displayed in files section which will create confusion among users.
I asked him then we should remove file section and only allow files on project, he does not agree with this idea also.
Considering these things will confuse users i do not approve this

https://github.com/Fammeo/EasyCloudBooks/issues/1560#issuecomment-1243229464
past comments are not valid anymore,
we have now developed new File Section like Custom Field called Custom File. with discussion with @vinpahilwani and @sureshsirvi

still no discussion about this documents section,
I strongly believe that documents section is still required for Common Files, Vault, Public , Website related sections


fix media order

Since firefox 59 the media order in the sdps has changed. this hacky fix (until a complete refactor of SdpInfo is done) will resolve the issue.
[x] It maybe needs and it does NOT includes Unit Tests
[] It includes documentation for these changes in /doc.

No problem, I would also prefer to have a medialist ordered. But required a bit of a refactor and I'm not too practical in c++ as you know, so if you would manage this on your own way it's better for everyone!!  this should be done before Firefox 59 anyway. Keep this in mind


[2987] Link to readthedocs

Closes: #2987

I've re-enabled the RTD GitHub integration so hopefully the docs will be automatically rebuilt when I merge this.


Clone macro

So we currently have a few issues with the clone macro: since it only allows idents, it cannot take input like x.y or even *x. Updating to expr would be enough I guess but not sure if that will be this simple. :3
cc @EPashkin @sdroege @antoyo

Updating to expr would be enough I guess but not sure if that will be this simple. :3

Well it's not. let x.y = foo; is not valid syntax :) It's the same thing as with self: we require that the caller uses a valid variable name.
We could probably add a compile_error! with a better error message for that.

@GuillaumeGomez Thanks.

And finally working! \o/


Added a pink theme

Screenshot of my A11y card:

Hello! Your theme is off to a good start, but before merging it the A11y card needs to display each check as AA or AAA compliant.
Thanks!

@12vanblart I have added the theme and checked for AA or AAA compliant. Its passed. Thanks!

Perfect! 🖤
Thanks for your contribution!


[e2e]: delegator-lowperms release main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v43.0.481
Run: https://github.com/slsa-framework/example-package/actions/runs/11605023982
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Trigger: release
Branch: v43.0.481
Date: Thu Oct 31 02:42:09 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v43.0.482
Run: https://github.com/slsa-framework/example-package/actions/runs/11623198665
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.release.main.default.slsa3.yml
Trigger: release
Branch: v43.0.482
Date: Fri Nov  1 02:43:08 UTC 2024
Tests are passing now. Closing this issue.


Is it possible to set default headers?

I am trying to use jsonwebtoken, so I want to set headers like Authorization': Bearer ${localStorage.jwt}`. How can I do it? If I use axios, then I can do this with interceptor, but your library use fetch, so I don't know how to implement to this.
reference: https://stackoverflow.com/questions/44820568/set-default-header-for-every-fetch-request
Do you guys have any idea?

Not exactly default but one way is to look it up when creating your RSAA and passing a custom headers property: https://github.com/agraboso/redux-api-middleware#rsaaheaders
Here is a basic example:
{
  [RSAA]: {
    ...
    headers: {
      Authorization: `Bearer ${localStorage.getItem('jwt')}`
    }
    ...
  }
}

Does that help?
Once #171 is done, you'll be able to create a middleware with defaults. I think you could set defaults for requests invoked via this middleware by providing a custom fetch. Something like https://github.com/moll/js-fetch-defaults

You could also achieve the same type of "interceptor" functionality with normal fetch behavior by adding your own middleware, before apiMiddleware in the chain.  Here, you could test the action with isRSAA, handle it as necessary, or ignore, then pass action to next middleware.  Your "auth middleware" can then be setup to dynamically apply headers to RSAAs as they get dispatched :)

If you don't want to write it by yourself check-out this simple package
https://github.com/TheSoftwareHouse/react-packages/tree/master/packages/redux-api-auth-middleware

If anyone looking for an interceptor that intercepts all API calls made with redux-api-middleware then this might be of some help:
https://github.com/hamxabaig/redux-api-middleware-interceptor

With v3.x you can pass your own fetch function. And can customize anything about request.
// ...
import { createMiddleware } from 'redux-api-middleware'

const apiMiddleware = createMiddleware({ fetch: customFetch })
const store = createStore(reduce, applyMiddleware(apiMiddleware))

function customFetch(uri, options = {}) {
  return fetch(uri, {
    ...options,
    headers: {
      ...options.headers,
      Authorization: `Bearer ${localStorage.getItem('jwt')}`
    }
  })
}


the new pytorch.bin  is bigger than original model issue

When I choose save model, I found some strange things。The new pytorch.bin  is bigger than original model。I choose Baichuan-7B ,--pruning_ratio 0.5 for test, and add  --save_model  for save the model after pruning. but the new pytorch.bin is 17GB, and the  original model is only 13GB?
Could you please tell me why? Thank you!

Hi @lb553024300, don't worry about the bin size. We store the whole model object into the disk with torch.save(model), so the saved file will be larger than torch.save(model.state_dict())

Hi @lb553024300, don't worry about the bin size. We store the whole model object into the disk with torch.save(model), so the saved file will be larger than torch.save(model.state_dict())

Get it,thanks

Hi. Could you please check if you deleted the gradient for calculating Taylor before saving the model?

Hi @VainF, can you tell me how to convert it back to the same format as the original model? The lighter capacity is what I need for storage. Thank you very much

Hi @lb553024300, don't worry about the bin size. We store the whole model object into the disk with torch.save(model), so the saved file will be larger than torch.save(model.state_dict())


action-menu: animate remaining transitions

Enhancement Request
Desired behavior
Animate as menus EXIT (reverse or ENTER), and animate item background (fade)
Today's behavior
Slick animation of menus appearing, but no transition when disappearing. Menu item background doesn't currently animate.
The Value Add
✨ Sparkle ✨
The Tradeoffs
None

Additional context:
Animating exits in react are a lot more complicated than animating entrances.  The entrance animation can be done with pure css transitions -- it's one time, it happens when the element is mounted, it's managed by the browser in css.  On exit, however, React removes the element from the dom immediately, leaving no opportunity to animate. There are libraries that work around this and provide extra hooks to handle this, such as react-motion.
But as a tradeoff, they will add complexity and library weight.

@jaketrent, makes sense, and not a tradeoff we want to immediately invest in then (menu animation). How about the item background color transition?

@dborodyansky a background color transition on entry seems fine to me.  I'm still a bit unclear on the desired behavior on that though.  Please elaborate.

@jaketrent, this line on the menu item button achieves the desired behavior: transition: background 150ms;

Our decision: no animate on exit for now; do the bg transition


Correct link to full source for example

This id is used to form the URL to the full source for the example at https://github.com/terezka/elm-plot/blob/b592b9cc098064bd3f9f40103726e2b2a7f0e2a4/docs/src/Docs.elm#L141, so it must be the same as the module name for the correct URL to be formed.

Oh, thanks!

I'll see what's going on with the tests!


Miner does not update after linking Twitch account to game drop campaign

I am encountering an issue with the TwitchDropsMiner where the miner does not update and show that a game is linked to my Twitch account, even after successfully linking my account to a game drop campaign. As a result, the miner does not start mining.
Steps to Reproduce:
Link Twitch account to a game drop campaign.
Check the miner to see if it updates and shows the linked campaign.
Notice that the miner does not update.
What I've Tried:
Logged out and back into my Twitch account multiple times.
Restarted the TwitchDropsMiner app several times.
Waited a couple of hours to see if it would eventually update.
Despite these efforts, the miner still does not show that it is linked to the campaign and does not start mining.
Expected Behavior:
The miner should update and show that it is linked to the Twitch account and the game drop campaign, allowing it to start mining.
Screenshots:

System Information:
Operating System: Windows 11 Pro
App Version: v16.dev27

On the https://www.twitch.tv/drops/campaigns page, find the affected campaign and ensure that it's "Connection" section says "Linked". If it doesn't, use the link button youłl find there to perform the linking process again. Alternatively, perform a clean reconnect: https://github.com/DevilXD/TwitchDropsMiner/issues/309#issuecomment-1749628379
Even with that solved, you'll still run into #462, just so you know. Let me know if your linking issue is solved.

It is working now. Before, I had checked on the https://www.twitch.tv/drops/campaigns page and it showed it connected. Only thing I did different this time is run/build the Dev build from the source code. For whatever reason it showed only rust in the priority list but it had it linked. When I jumped back to the other build it was fixed. No Clue.

Glad you could get it resolved :)


Debug ValueSet retrieve

If you try to retrieve a ValueSet in CQL it causes infinite recursion.

Fixed in develop.


Adds a new "medchem hud" for CMOs

About the PR

This PR replaces the medical hud in chief medical officer's lockers with a unique "medchem hud", which combines the functionality of the medical hud and chemical analysis goggles into just one fancy gadget.
Why / Balance

This is a QoL addition for chief medical officers who don't want to have to swap between two different items just to be able to see what their chemists are making. It will also be especially handy for CMOs on lowpop servers who have to juggle making medicine and treating patients more often.
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

add: Added a new unique "medchem hud" for chief medical officers, which acts as a medhud and chemical analysis goggles in one item!

its feature freeze from may 10 to june 14


Add language support for HTML files

Allow for a language tag to be passed to the program with a -l, --lang tag that will update any converted HTML's lang tag to the passed language. For example:
./textml.py french.txt --lang fr would set <HTML lang="fr"> in the root <html> element.
A way to implement this could be to add a new option to argparse to accept the -l, --lang tag and pass the value to each convert function to be generated in the final HTML file function write_html_content()
If no language is provided default to en-CA using argparse's default value

Added support in commit https://github.com/ijacobs-cpa/textml/commit/c1e4bff3aa7c9be4f370bd1ef5c2ea5991bb8a86


picovoice-ros: 1.0.0-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository picovoice-ros to 1.0.0-1:

upstream repository: https://github.com/reinzor/picovoice_ros.git
release repository: https://github.com/reinzor/picovoice_ros-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.7
previous version for package: 0.0.4-1

picovoice_driver
* feat: git submodules, pv_recorder, intent filter (#4 <https://github.com/reinzor/picovoice_ros/issues/4>)
* feat: Use v2.1 api
* Contributors: Rein Appeldoorn

picovoice_msgs
* feat: git submodules, pv_recorder, intent filter (#4 <https://github.com/reinzor/picovoice_ros/issues/4>)
* feat: Use v2.1 api
* Contributors: Rein Appeldoorn

Abandoned in favor of https://github.com/ros/rosdistro/pull/32586


Class to enable and disable the 2FA

Hi dev!
I would like to request a class for view where we can disable and enable it without using the jetstream. so we can integrate it to the custom profile setting.

Hey @itsmejyv Thanks for the suggestion. Can you please clarify what you mean by this. Additionally we are going to be closing issues here and are requesting that users submit issues and suggestions here: https://devdojo.com/questions.
Thanks so much!


s/group/groupid/

lgtm.
The PWS change has landed but hasn't been deployed.  I'll push a version to the dev server, so we can play with this.


Turn the axis of Face

It's not comfortable to turn Face and its shadow because axis goes through the nose of Face model.
Need to move it to the top of head both for rotation and light.
This will allow to get more effect from rotation and use fitting time more rationally.

Problems with shadows occurred. Need to complete #53 first.
Problems are following:

shadow map isn't calculated properly, because rotation and light matrices product gives not a rotation matrix, but rotation and perspective;
flat shadows seem to be calculated improperly: with fixed light vector and light vector rotated with the model shadows are changing during rotation, when expected to stay.


{ARM} Fix live test

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

Will you open a new one?

Due to some unnecessary code submitted in PR, there were other team members in the code reviewer


NTR: inviable binucleate aseptate cell with mitotic cell cycle arrest before cell separation

FYPO:0003825 - inviable binucleate aseptate cell with mitotic cell cycle arrest in G2 before cell separation)
A cell phenotype observed in the vegetative growth phase of the life cycle in which a cell is inviable and has two nuclei but no septum, and the mitotic cell cycle is arrested with nuclei in G2 phase."
the cells have done mitosis,  but the 2 nuclei arrest in a  G2 state in the following cell cycle (I.e they complete S-phase before they arrest).
We would not always know this though so we probably need
NTR inviable binucleate aseptate cell with mitotic cell cycle arrest before cell separation
as a parent

inviable binucleate aseptate cell with mitotic cell cycle arrest before cell separation FYPO:0005689
edit file: 774d5dd9b89cea0676a37c022a8b71b72d5d1585
release: e0bbc398974d9c8b721de72cddd6079c2c82ce6a


can't run wasm module in browser: LinkError: WebAssembly Instantiation

What version of Go are you using (go version)?

> go version
go version go1.11.2 linux/amd64

Does this issue reproduce with the latest release?
I don't know yet. All the example code I've seen runs on go 1.11
What operating system and processor architecture are you using (go env)?
go env Output
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/bob/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/bob/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/golang"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build169530480=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I tried following various example of how to get a wasm module compiled via golang to work in the browser, primarily: https://github.com/golang/go/wiki/WebAssembly
hello.go

package main

import (
    "fmt"
)

func main() {
    fmt.Println("hello world")
}

Compiled into wasm module via:
GOOS=js GOARCH=wasm go build -o hello.wasm hello.go
I then copied the js / html fiels for running wasm in browser via:
https://github.com/golang/go/tree/master/misc/wasm
Lastly, I created simple http server to serve up the assets:

~/go/bin/goexec 'http.ListenAndServe(":8080", http.FileServer(http.Dir(".")))'

What did you expect to see?
I expected to see a active button, so that when clicked it was display "hello world" in the browser console
What did you see instead?
Uncaught (in promise) LinkError: WebAssembly Instantiation: Import #5 module="go" function="runtime.scheduleCallback" error: function import requires a callable

Using Ubuntu 18.04, I also have no /usr/lib/go-1.11/misc/wasm/wasm_exec.js file


[content] including Teams as option to notification in k8s-quick-start

Just adding an option for send sidekick notification to Teams Message via webhook as option.
I'm including since I hope there's a lot of companies that have this solution as main communication channel.
What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
Including on Quick Start option for message to Microsoft Teams channel via standard incoming webhook.
Which issue(s) this PR fixes:
Not an issue.

Fixes #
Special notes for your reviewer:
Please feel free to reject if you think this not helping anyway. 😄

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

a4600a7 including Teams as option to notification

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dneto82
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Configure XLink chunk size

Adding new API:
/**
 * Set chunk size for splitting device-sent XLink packets, in bytes. A larger value could
 * increase performance, with 0 disabling chunking. A negative value won't modify the
 * device defaults - configured per protocol, currently 64*1024 for both USB and Ethernet.
 */
void Pipeline::setXlinkChunkSize(int sizeBytes);

The defaults on device side were not changed, as it might be possible (mainly for USB case) that increasing the chunk size won't work with certain hosts / OSes.
For USB, if supported by the host, best performance would be achieved with:  pipeline.setXlinkChunkSize(0);
This should increase the max available USB bandwidth and reduce the device CPU load (less overhead from splitting).
Related PR: https://github.com/luxonis/depthai-shared/pull/53

Also, would it make sense adding an RPC call for this?


Add overflow:hidden to columns

Firefox doesn't listen to flex-basis values, for instance if flex-basis: 50% and contains a large image, etc.. it expands. Adding overflow:hidden fixes the issue.

Fixed with 0.3.2

Oh I literally just ran into this, @buschschwick.
Firefox (and soon to be Chrome) need min-width : 0 on columns. Without it Firefox sets min-width: -moz-min-content and Chrome sets min-width : auto on any column with an image in it.

Okay, have you tested on 0.3.2 yet? Either way, I can add that as a property for columns.

Added min-width: 0 to all columns in version 0.3.3


OPRUN-3063: Microshift manifests remove collection-profiles cronjob from kustomization

Removes the collect profiles cronjob from the microshift kustomization.yaml file.

/jira refresh

Could you retitle your commit messages to include the UPSTREAM: <carry>:  prefix so we know to keep them when we downstream add'l feature content?

/retest-required

/retest-required

/retest-required

/retest-required

/retest

/lgtm


fastapi type stubs

Would be nice if fastapi shipped with .pyi type stubs so I can type check my application using pyright.

yes, this would be great!!

harder to develop without autocomplete.

Hey there, FastAPI already has the source code fully annotated and the metadata to mark it like that. .pyi files are only when the code is not annotated. You should be able to check your code with Pyright or mypy.
For reference, directly from Pyright: https://github.com/microsoft/pyright/blob/main/docs/typed-libraries.md#inlined-type-annotations-and-type-stubs


Unable to Like,Tweet or Retweet after Logging In

Hello,
First, here is my code:
python
Copier le code
import twAuto
tw = twAuto.twAuto(
username="",
email="",
password="",
chromeDriverMode="manual",
driverPath="chromedriver.exe",
pathType="xPath",
headless=False,
debugMode=True,
createCookies=False
)
tw.start()
tw.login()
print("Logged in")
tw.like(url="https://x.com/elonmusk/status/1809251925306777944")
print("Retweeted")
tw.close()
I can log in successfully using the tw.manualCookieCreation() function. However, when I launch the code with headless=False, I see that it logs in correctly, navigates to the tweet page, and then nothing happens.
Here is the output when I close the window:
less
Copier le code
twAuto started.
Starting twAuto...
Using Chrome Driver from the path: chromedriver.exe
Successfully Logged In
Logged in
twAuto Error: Message: no such window: target window already closed
from unknown error: web view not found
(Session info: chrome=126.0.6478.127)
Stacktrace:
GetHandleVerifier [0x00007FF61B24EEA2+31554]
(No symbol) [0x00007FF61B1C7ED9]
(No symbol) [0x00007FF61B08872A]
(No symbol) [0x00007FF61B05D995]
(No symbol) [0x00007FF61B1044D7]
(No symbol) [0x00007FF61B11C051]
(No symbol) [0x00007FF61B0FCDD3]
(No symbol) [0x00007FF61B0CA33B]
(No symbol) [0x00007FF61B0CAED1]
GetHandleVerifier [0x00007FF61B558B1D+3217341]
GetHandleVerifier [0x00007FF61B5A5AE3+3532675]
GetHandleVerifier [0x00007FF61B59B0E0+3489152]
GetHandleVerifier [0x00007FF61B2FE776+750614]
(No symbol) [0x00007FF61B1D375F]
(No symbol) [0x00007FF61B1CEB14]
(No symbol) [0x00007FF61B1CECA2]
(No symbol) [0x00007FF61B1BE16F]
BaseThreadInitThunk [0x00007FFE0C88257D+29]
RtlUserThreadStart [0x00007FFE0E10AF28+40]
Retweeted
Do you know what could be causing this issue?
Thank you for your help!

Hi just saw your issue, do you still need help about the project?


chore(ci): report memory and cpu usage in e2e bench

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (Token:transfer_public): 45.6 (+50%)
avm_simulation_time_ms (Token:_increase_public_balance): 58.7 (+385%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,696 (+1%)
1,540 (+1%)
702 (-1%)
755 (+1%)
767 (+1%)

proof_construction_time_sha256_30_ms
11,671
3,123
1,405
1,426
1,468

proof_construction_time_sha256_100_ms
43,596 (+2%)
12,390
5,661
5,535 (-1%)
5,496

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0 (-2%)
87.0 (-2%)

proof_construction_time_poseidon_hash_30_ms
1,514 (+2%)
415 (+1%)
199
221 (-3%)
271 (+1%)

proof_construction_time_poseidon_hash_100_ms
5,591
1,512 (+1%)
673
743 (+3%)
746 (-1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
708
708
708

l1_rollup_calldata_gas
6,600
6,590
6,600

l1_rollup_execution_gas
611,442
611,583
611,744

l2_block_processing_time_in_ms
761 (+1%)
1,452 (+2%)
2,708

l2_block_building_time_in_ms
10,126 (-1%)
19,963 (+1%)
38,972

l2_block_rollup_simulation_time_in_ms
10,126 (-1%)
19,962 (+1%)
38,971

l2_block_public_tx_process_time_in_ms
8,454
18,050 (+1%)
36,998

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,135 (+1%)
10,250 (+2%)

node_database_size_in_bytes
12,709,968
16,912,464

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Circuit
simulation_time_in_ms
witness_generation_time_in_ms
input_size_in_bytes
output_size_in_bytes
proving_time_in_ms
proof_size_in_bytes
num_public_inputs
size_in_gates

private-kernel-init
106
424 (-2%)
21,718
44,795
N/A
N/A
N/A
N/A

private-kernel-inner
173
741 (-1%)
72,440
44,942
N/A
N/A
N/A
N/A

private-kernel-tail
728 (+1%)
994 (-2%)
52,998
46,729
N/A
N/A
N/A
N/A

base-parity
6.59
674 (+2%)
160
96.0
1,927 (+2%)
13,188
19.0
131,072

root-parity
113 (-1%)
112 (-2%)
69,084
96.0
28,281 (+2%)
13,188
19.0
4,194,304

base-rollup
3,239
5,090
182,192
632
41,199
13,988
44.0
4,194,304

root-rollup
159
115 (+3%)
54,461
684
25,440
13,956
43.0
4,194,304

public-kernel-setup
93.6 (+6%)
2,643 (-1%)
98,393
65,654
15,962 (+1%)
123,012
3,451
2,097,152

public-kernel-app-logic
101
4,109 (+2%)
98,393
65,654
9,588 (+1%)
123,012
3,451
1,048,576

public-kernel-tail
631 (+1%)
26,243 (+3%)
396,134
10,814
62,080
26,372
431
8,388,608

private-kernel-reset-tiny
229
950 (-1%)
68,504
44,687
N/A
N/A
N/A
N/A

private-kernel-tail-to-public
5,179 (-1%)
1,714 (-7%)
918,792
1,697
N/A
N/A
N/A
N/A

public-kernel-teardown
87.3 (+5%)
4,099
98,393
65,654
16,977 (+1%)
123,012
3,451
2,097,152

merge-rollup
60.8
N/A
35,678
632
N/A
N/A
N/A
N/A

undefined
N/A
N/A
N/A
N/A
151,884 (+1%)
N/A
N/A
N/A

Stats on running time collected for app circuits

Function
input_size_in_bytes
output_size_in_bytes
witness_generation_time_in_ms
proof_size_in_bytes
proving_time_in_ms

ContractClassRegisterer:register
1,312
11,699
350
N/A
N/A

ContractInstanceDeployer:deploy
1,376
11,699
28.1
N/A
N/A

MultiCallEntrypoint:entrypoint
1,888
11,699
617
N/A
N/A

GasToken:deploy
1,344
11,699
449
N/A
N/A

SchnorrAccount:constructor
1,280
11,699
440
N/A
N/A

SchnorrAccount:entrypoint
2,272
11,699
828
N/A
N/A

Token:privately_mint_private_note
1,248
11,699
528
N/A
N/A

FPC:fee_entrypoint_public
1,312
11,699
93.3 (+1%)
N/A
N/A

Token:transfer
1,280
11,699
1,302
N/A
N/A

AuthRegistry:set_authorized (avm)
18,427
N/A
N/A
102,048
2,042 (-2%)

FPC:prepare_fee (avm)
22,894
N/A
N/A
102,112
2,322

Token:transfer_public (avm)
46,817
N/A
N/A
102,112
5,993 (-2%)

AuthRegistry:consume (avm)
26,670
N/A
N/A
102,080
2,624

FPC:pay_refund (avm)
26,163
N/A
N/A
102,080
3,593 (+3%)

Benchmarking:create_note
1,312
11,699
424
N/A
N/A

SchnorrAccount:verify_private_authwit
1,248
11,699
46.3 (+1%)
N/A
N/A

Token:unshield
1,344
11,699
1,098 (+1%)
N/A
N/A

FPC:fee_entrypoint_private
1,344
11,699
1,418 (+1%)
N/A
N/A

AVM Simulation
Time to simulate various public functions in the AVM.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
95.2 (-1%)
8,139

GasToken:set_portal
15.0
2,362

Token:constructor
123 (+1%)
16,400

FPC:constructor
94.1 (-4%)
9,304

GasToken:mint_public
82.9 (+1%)
6,150

Token:mint_public
95.5
11,720

Token:assert_minter_and_mint
85.2 (-13%)
8,028

AuthRegistry:set_authorized
11.5 (+25%)
4,537

FPC:prepare_fee
294 (+9%)
8,812

Token:transfer_public
:warning: 45.6 (+50%)
32,641

FPC:pay_refund
83.9 (+18%)
12,114

Benchmarking:increment_balance
995 (+1%)
7,450

Token:_increase_public_balance
:warning: 58.7 (+385%)
8,960

FPC:pay_refund_with_shielded_rebate
148 (+1%)
12,663

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.154 (+5%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.5
16.8
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.604
0.517
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.6 (+1%)
76.2
131
253 (+2%)
469

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.496
0.468
0.450
0.457 (+2%)
0.439

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.6 (+1%)
112
183 (+1%)
357
691

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.510 (+1%)
0.500 (-1%)
0.484
0.483
0.476

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
73.6
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.523
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
59,027
663,408

Transaction size based on fee payment method
| Metric |  |
| - |  |


Assumed role

In our setup developer are authenticated through our company but use a role on client company to deploy on their behalf and on their account.
Therefore we ask our client to create a role for us which we can assume for the intervention we do on their account.
I've added this patch which check for a aws_assumed_role_arn key in capistrano config and if present start a special session.
I do not know how frequent our setup is but I'm sharing the code back with you since your gem basically saved me days of work :)
require 'aws/sts'
require 'elbas/aws/credentials'

Elbas::AWS::Credentials.module_eval do

  def overidden_credentials
    @_credentials ||= begin
      _credentials = {
        access_key_id: fetch(:aws_access_key_id, ENV['AWS_ACCESS_KEY_ID']),
        secret_access_key: fetch(:aws_secret_access_key, ENV['AWS_SECRET_ACCESS_KEY'])
      }

      _credentials.merge! region: fetch(:aws_region) if fetch(:aws_region)
      _credentials
    end
  end

  def credentials
    return @patched_credentials if defined?(@patched_credentials)

    credential_hash = overidden_credentials
    assumed_role_arn = fetch(:aws_assumed_role_arn)

    if assumed_role_arn
      sts = ::AWS::STS.new(credential_hash)
      role_credentials = sts.assume_role(
        role_arn: assumed_role_arn,
        role_session_name: "capistrano-session",
      )
      @patched_credentials = role_credentials[:credentials].merge(region: credential_hash[:region])

    else
      @patched_credentials = credential_hash
    end

    @patched_credentials
  end

end

New version of ELBAS will grab creds from the Capistrano aws_credentials if specified, that should allow you to do any custom credentials logic without monkey patching :)

For what it's worth, here is how I do it with the new (v3) version :
set :aws_access_key, ENV['AWS_ACCESS_KEY_ID']
set :aws_secret_key, ENV['AWS_SECRET_ACCESS_KEY']
set :aws_region, ENV['AWS_REGION']
set :assumed_role_arn, ENV['AWS_ASSUMED_ROLE_ARN']

set :aws_credentials, -> {
  aws_access_key = fetch :aws_access_key
  aws_secret_key = fetch :aws_secret_key
  assumed_role_arn = fetch :assumed_role_arn

  sts = if aws_access_key.present?
          dev_credentials = ::Aws::Credentials.new(aws_access_key, aws_secret_key)
          ::Aws::STS::Client.new(credentials: dev_credentials)
        else
          ::Aws::STS::Client.new
        end

  role_credentials = sts.assume_role(
    role_arn: assumed_role_arn,
    role_session_name: "capistrano-session",
  ).to_h

  temp_access_key_id = role_credentials[:credentials][:access_key_id]
  temp_secret_access_key = role_credentials[:credentials][:secret_access_key]
  temp_session_token = role_credentials[:credentials][:session_token]

  ::Aws::Credentials.new(temp_access_key_id, temp_secret_access_key, temp_session_token)
}

it was definitely easier than the previous version !


ioredis Cluster support?

Per the README:

While this covers many use cases, it doesn’t work for the following:

Using ioredis in cluster mode.

But ioredis Clusters implements the CompatibleRedisClient interface, so why is cluster mode not supported?

It does implement that, but:

at https://github.com/lukechilds/keyv-redis/blob/9d650f21ce4609c07d5e6848efb1572eccf95d46/src/index.js#L10 it checks for instanceof Redis which is not the same as instanceof Redis.Cluster
at https://github.com/lukechilds/keyv-redis/blob/9d650f21ce4609c07d5e6848efb1572eccf95d46/src/index.js#L61 it bulk-deletes keys, which doesn’t work in cluster (you can only bulk-delete keys that are in the same cluster slot)

These are relatively easy to fix in the official client, but there’s no need to take the approach that everyone will be using ioredis since it’s relatively easy to support a wider range of clients, so I made this.

That's why @keyv/redis doesn't support cluster mode. keyv-anyredis doesn't seem to use @keyv/redis, so why doesn't keyv-anyredis support cluster mode?

keyv-anyredis definitely does support cluster mode. I guess I miscommunicated! I will update the README to be clearer.

Updated the README to hopefully be more clear. Thank you!


Incoming images or files

In RichText/Media_XXXX type messages, I am trying to download the associated file without success. From the content, I extract the object's URI in the following format:
preg_match_all(
    '/<URIObject\s+uri="([^"]+)"\s+url_thumbnail="([^"]+)"[^>]*>.*?<OriginalName\s+v="([^"]+)"/s',
    $message->content,
    $matches,
    PREG_SET_ORDER
);

($message->content contains the result of $resource->getContent())
If I download that URI, I get a JSON, but I have tried different ways to obtain the file without success.
Is there any way to achieve this? I couldn't find a method for it in the library.

After reading a lot of code from the skpy and easy_skype projects, I tried to reproduce the same behavior in PHP, but the file download fails. I suspect it's an issue with the libcurl library.
The solution I found is to use a Python application. I pass it the download URL, the target file, and the session token, and it handles the download in Python. The code is as follows:
#!/usr/bin/env python3
import sys
import requests
import logging

# Logger configuration
logging.basicConfig(
    filename="download_log.txt",
    level=logging.INFO,
    format="%(asctime)s - %(levelname)s - %(message)s"
)

def download_file(url, destination, token):
    headers = {
        'Authorization': f"skype_token {token}"
    }

    try:
        # Make the GET request with stream=True
        response = requests.get(url, headers=headers, stream=True, timeout=10)

        # Check if the request was successful
        if response.status_code == 200:
            with open(destination, 'wb') as file:
                for chunk in response.iter_content(chunk_size=8192):
                    if chunk:  # Filter out empty chunks
                        file.write(chunk)
            logging.info(f"File successfully downloaded to: {destination}")
            sys.exit(0)  # Successful exit
        else:
            error_msg = f"Error downloading file. Status code: {response.status_code}"
            logging.error(error_msg)
            logging.error(f"Error details: {response.text}")
            sys.exit(1)  # Exit with error

    except requests.RequestException as e:
        error_msg = f"Error making the request: {e}"
        logging.error(error_msg)
        sys.exit(2)  # Exit with error

if __name__ == "__main__":
    if len(sys.argv) != 4:
        print("Usage: python download.py <URL> <destination_file> <token>")
        sys.exit(1)

    url = sys.argv[1]
    destination = sys.argv[2]
    token = sys.argv[3]

    download_file(url, destination, token)


Is end-to-end training able?

I tried to implement end-to-end training but the network parameters soon became NaN. Do I have to use the same 5 step training as R-FCN?

We will release the code. Actually, end-to-end training works well for it.

Hi,
Is there any ETA on the code release? I am working on modifying DeepMask now for the simple task of auto-cropping, and am pretty unhappy with how slow it is to train and the quality of the results. Would be nice to have the code of this to work with too since I only have a few more weeks to experiment...
-Thanks

Hi, the arxiv paper would be released quite soon within one week. We are working hard on coding releasing as well, but it would take some time. I cannot promise, but hopefully we can get it done in one month. You can also modify based on R-FCN (https://github.com/daijifeng001/R-FCN).

I see, thanks.

Hi, the tech report is available at https://arxiv.org/abs/1611.07709

Thanks a lot.

Hi,
Can you tell me the threshold for predicting final results (vis_thresh) ? Is it 0.8 as in R-FCN or 0.5 as in MNC?

＠pqviet Hi, we use mask_size=21 in FCIS and vis_thresh=0.7 is used to visualize the result.


Remove .gitkeep from app/models folder [ci skip]

Plenty of files under app/models now, no need to use .gitkeep anymore. :relaxed:.

Failure is not related with the change in this pull request.

I think this is structure created by rails or suspenders
Do we really need to delete this file?

Do we really need to delete this file?

No need.


Removed usage of 'new' when using Ember.A

closes https://github.com/softlayer/sl-ember-components/issues/720

@notmessenger eslint is complaining about not using new https://travis-ci.org/softlayer/sl-ember-components/jobs/78290883#L545

@azizpunjani https://github.com/softlayer/sl-ember-components/pull/742 resolved this issue.  Merge in the latest changes in this branch to get the updates.

@notmessenger I have merged in the latest changes, this PR is ready for review.


Linux Docker environment updates

Linux has a number of different distributions, such as Ubuntu, CentOS, and Arch Linux. Each one has it's own package manager for installing dependencies, and it's own set of package names for needed build tools. Over time, those distributions have come out with newer versions, leaving old sets of packages and one-time setup steps to become out of date. These need to be updated, or perhaps removed.
One way to keep things more relevant, and documented, would be to add Docker images to build and test on these other distributions. In particular, the Dockerfile used to setup the environment would provide documentation on how to get setup and started on those distributions. Currently we only have Dockerfiles covering Ubuntu. If we are to maintain the current level of support for environment setup, we should add Dockerfiles for the following distributions:

CentOS
Arch Linux

In particular, the CentOS rules currently available work for CentOS 7, but not for CentOS 8. With CentOS 7, the needed SDL2 packages could be found in the EPEL (Extra Packaged for Enterprise Linux) repository. In CentOS 8, some of these have been moved to PowerTools, while others don't seem to be obviously available.
To enable the PowerTools repository on CentOS 8:
dnf --assumeyes install 'dnf-command(config-manager)'
yum config-manager --set-enabled PowerTools

For packages not found in repositories, a source install can be performed.
Additionally, the version of PhysFS available on CentOS 7 is too old to be usable, so a source install is needed there.

In addition to library dependencies, the Dockerfiles would also serve to document the needed build tools, such as make and g++ or clang++. As part of that, it would include steps to enable additional repositories for newer compiler versions, or any setup needed.
In particular, to make use of C++17 features, which this project does, the stock compiler for CentOS 7 is not sufficient. Instead, a newer GCC compiler can be installed with:
yum install centos-release-scl
yum install devtoolset-8-gcc-c++
scl enable devtoolset-8 -- bash

It seems the existing Linux Dockerfiles don't work with GitHub Actions. The use of USER in the Dockerfile creates permission problems as GitHub Actions expects commands within containers to run as root.
They also discourage the use of WORKDIR.
Documentation:
https://docs.github.com/en/actions/creating-actions/dockerfile-support-for-github-actions

It seems the build is able to run on GitHub Actions using the updated Docker images, though with the caveat of there being git warning messages in the output.
Adding reference to GitHub Actions issue relating to git generating warning messages:
https://github.com/actions/runner/issues/2033

I was looking back at the original post in this issue. It seems CentOS hit end of life at the end of last year.
Source:
https://www.centos.org/cl-vs-cs/
They now have CentOS Stream replacing it. I poked around a bit, and found a Docker base image for it. The main repos only have the SDL2-devel dependency. You can get glew-devel as well if you first install the epel-release repo. The remaining SDL2 library dependencies don't seem to be readily available.
FROM quay.io/centos/centos:stream9

# Packages not readily available:  SDL2_mixer-devel SDL2_image-devel SDL2_ttf-devel
RUN yum --assumeyes install epel-release SDL2-devel glew-devel

As no one (else) has indicated they're trying to run this project on CentOS, I'm thinking maybe I'll just remove the current antiquated support.


Simplify checking log (event) type

Description
Currently, inside airnode we fetch all logs and then we filter them by the topic (we check if the event contains a specific topic) and sometimes we just check the event name (mainly in tests).
It seems solidity supports events with same name, but I don't think we will (nor should) use it in airnode. Implementation could create utility functions for this and we should just use a single consistent style.
See also comment: https://github.com/api3dao/airnode/pull/304#discussion_r624689383

Should this be closed @Siegrift ?

I don't think it's worth doing so I'd just close, wdyt?

Agreed, closing


Reject subclassing subclasses of T::Enum

Motivation

Reported at Stripe by @aadi-stripe
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LFSLTEfeaOfMiq
→ https://go/builds/bui_LFSMmhM2Bq1b6L


What happens when an S3 GetObject fails?

Since the return value of a get_object is a string or populated IO is there a way to detect when a get fails?

Ooh, good call. There is not. It looks like I never added response checks at all. I've seen this shard fail (usually when trying it on different object-storage providers because they all have quirks) and it raises exceptions because those endpoints return unexpected XML, but get_object doesn't fail on that, though and I just never noticed.

It's not entirely uncommon for S3 shards to raise an exception, though I don't really know why that's the pattern the AWS APIs seem to have gone with.
I'm using it quite successfully with Tigris -- though they do have their quirks, I would call it a success.

The fix I'm working on for this issue raises an exception for responses like 400/5xx. For a 404:

get_object? (a new method) returns nil
get_object will be assertive and raise an AWS::S3::ObjectNotFound

What's Tigris?

That sounds like a great implementation, thank you.
Tigris is the s3-ish system which is pretty integrated in at fly.io. https://www.tigrisdata.com/


Pull Request Template Creation

I am writing to express my interest in contributing to your repository by creating a pull request template.
I believe that having a standardized template will help streamline the contribution process, ensuring that all necessary information is included and making it easier for maintainers to review and merge pull requests.
Could you please assign me the issue under gssoc24? I am excited about the opportunity to contribute to your project and help improve the workflow for all contributors.

I would like top work on this issue . Please assign.