Donated repo to zalando-incubator

Fixes #7

Changes Unknown when pulling 61d676d6e521dce95267944488e7119322743515 on feature/zalando-incubator into ** on master**.


[e2e] add map of all existing e2e environment variables

What does this PR do?

Add an explicit list of E2E environment variables
Motivation

This does not change the behaviour and the mechanism of the parameter store based on environment variables, but it improves its documentation.
Additional Notes

Will add a link to this map in the e2e documentation
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


[graalvm] Set up automation

See https://github.com/endoflife-date/release-data/pull/138.

Relevant: the link breaks for JDK releases.

Relevant: the link breaks for JDK releases.

Good catch, fixed.

LGTM, merging both


[Documentation] [Features] Various cosmetic enhancements

These are various cosmetic enhancements to the Markdown source files of the Cucumber features. The commit messages are hopefully self-explanatory. Let me know if squashing is required, or if the commit messages are not clear enough.

Thanks for doing this, I have a style quibble that I'd like Myron to weigh in on before we merge this but otherwise it looks good :)

Thanks, @fabschurt!

Glad I could help :wink:


Style changes to the view and model remake

DoD: Change the Event Response layout to now show the title named Requested Event instead of pending. Then change the column to now show : Event Name, club name, event date,  event location, event description, img. Then in the dropdown we should have 3 options listed in the following order: approved, request change, denied. Then add in a textbox with submit if it hasn't been added already. If a textbox has been added then something MUST be added in the textbox.

This is finished


Possible to register crashlyticsDidDetectCrashDuringPreviousExecution callback in Unity?

Please fill in the following fields:
Unity editor version: 2018.4.12f1
Firebase Unity SDK version: firebase_unity_sdk_6.6.0
Firebase plugins in use (Auth, Database, etc.): Analytics, Messaging
Additional SDKs you are using (Facebook, AdMob, etc.): Facebook
Platform you are using the Unity editor on (Mac, Windows, or Linux):  Windows
Platform you are targeting (iOS, Android, and/or desktop): Android, iOS
Scripting Runtime (Mono, and/or IL2CPP): IL2CPP
Please describe the issue here:
Is there a way to register the crashlyticsDidDetectCrashDuringPreviousExecution with the Crashlytics instance generated during the normal Crashlytics startup in Unity?  It's clear how to do this if we were to instantiate the Crashlytics object ourself, but not sure if this is possible with the Unity SDK.  (We're doing this manually on iOS by patching the .mm file in a post build step, which works great for us).

It seems the underlying function has since been removed from the Android and iOS SDKs.  Also, the Unity SDK is now open source (located where this bug is now located).  I'm going to close this issue.


Can't make inferences with my own image. (not for video)

I am a colab user.
I uploaded my own images to the folder "demo_000" and tried to infer, but it didn't work.
The inference worked for the pre-prepared images.
Here is traceback.
"""
2022-01-13 13:16:36,348 - mmedit - INFO - load checkpoint from torchvision path: torchvision://vgg19
load checkpoint from local path: checkpoints/RealBasicVSR_x4.pth
Traceback (most recent call last):
File "inference_realbasicvsr.py", line 144, in
main()
File "inference_realbasicvsr.py", line 97, in main
inputs = torch.stack(inputs, dim=1)
RuntimeError: stack expects each tensor to be equal size, but got [1, 3, 352, 352] at entry 0 and [1, 3, 604, 900] at entry 2
\n# show the first image as an example\nimport mmcv\nimport matplotlib.pyplot as plt\n\nimg_input = mmcv.imread('data/demo_000/00000000.png', channel_order='rgb') \nimg_output = mmcv.imread('results/demo_000/00000000.png', channel_order='rgb') \n\nfig = plt.figure(figsize=(25, 20))\nax1 = fig.add_subplot(2, 1, 1) \nplt.title('Input image', fontsize=16)\nax1.axis('off')\nax2 = fig.add_subplot(2, 1, 2)\nplt.title('RealBasicVSR output', fontsize=16)\nax2.axis('off')\nax1.imshow(img_input)\nax2.imshow(img_output)\n
"""
Note that the current ipynb file does not have a script to generate "results/demo_000".
Please add mkdir codes if you can.

From the error log RuntimeError: stack expects each tensor to be equal size, but got [1, 3, 352, 352] at entry 0 and [1, 3, 604, 900] at entry 2, it seems that the images you put in the folder do not have the same size. You may need to check for the size of the images.
Regarding the path results/demo_000, the inference code should have created the folder in line 133.

Thank you for your kind answer, and this issue was cleared.
It was a problem caused by my own inability to decipher that error message.
Please forgive my lack of programming skills.


Use defer

I'm not sure about this one since maybe it's about personal taste but I think it's good practice to use defer to ensure Shutdown() will be called when the current function returns.
If you don't know about defer, you can know more:

https://golang.org/doc/effective_go.html#defer
https://blog.golang.org/defer-panic-and-recover

The defer should be after the error check if runtime is null when there's an error.


ModuleNotFoundError: No module named 'graph_transformer'

Hi,
I tried to replicate the results in your paper, but I have the following issues:
(1) this package does not have the correct version, and thus failed when I ran bash setup.sh
Since I cannot install dgl==0.8.1, I used dgl==0.9.0 instead, which is the closest version I could find.
(2) With the above, I cannot import the following:
from graph_transformer import GraphTransformerNet
Could you kindly point out why? Thanks.
Regards,
David

Hello David,

Could you provide more details regarding the error you faced?

You can remove the line "from graph_transformer import GraphTransformerNet" and run the code. GraphTransformerNet is not necessary. I will soon update the file.

Best regards,
Bhanu Prakash

Hi Bhanu,
I got your code running now. Thanks.
However, after reading your paper again, I have some comments.
(1) In your table 1, you reported the results of relation extraction (RE)
for various transformer models, and I think the results are problematic.
[1] reported the RE results for LayoutXLM (BASE), and it is 0.5483. We have
implemented the LayoutLMv3 (base) RE head, and got results around F1 0.56+,
using unilm repo (https://github.com/microsoft/unilm).
Your paper reported LayoutLMv3 RE can reach F1 0.805, which is
significantly higher.
I have come across a similar issue concerning the BROS [2], where they
reported a RE score as high as 71.46 for their base model. We found that
they assumed that each token has at most one in-coming relation in their
implementation. This assumption is not valid for the Funsd dataset. Below
is a snippet from image “0001476912.png” in the Funsd dataset:
[image: image.png]
*Please double check whether your implementation has the same issue. *
(2) You do not use any textual information for entity extraction, but only
layout. This makes me feel how general your solution will be for general
information extraction like on a CORD dataset.
https://github.com/clovaai/cord
Maybe a better solution will design a GNN head on top of a transformer
model like LayoutLMv3.
Regards,
David
[1] Xu, Y., Xu, Y., Lv, T., Cui, L., Wei, F., Wang, G., . . . Zhou, L.
(2021). LayoutXLM: Multimodal Pre-training for Multilingual Visually-rich
Document Understanding. Proceedings of the 59th Annual Meeting of the
Association for Computational Linguistics and the 11th International Joint
Conference on Natural Language Processing (pp. 2579–2591). Association for
Computational Linguistics.
[2] Hong, T., Kim, D., Ji, M., Hwang, W., Nam, D., & Park, S. (2021). BROS:
A Pre-trained Language Model for Understanding Texts in Document.
International
Conference on Learning Representations.
On Wed, 28 Jun 2023 at 11:34, Bhanu Prakash Voutharoja <
@.***> wrote:

Hello David,

Could you provide more details regarding the error you faced?
2.
You can remove the line "from graph_transformer import
GraphTransformerNet" and run the code. GraphTransformerNet is not
necessary. I will soon update the file.
Best regards,
Bhanu Prakash
—
Reply to this email directly, view it on GitHub
https://github.com/Bhanu068/LAGNN/issues/1#issuecomment-1610361489, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFH7FN2LX3QMB3OAFCZHCGTXNNUZLANCNFSM6AAAAAAZWG74KA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@xli458
In Table 1, we conducted a comparative analysis between our model LAGNN and other state-of-the-art (SOTA) models, employing our unique approach to Relation Extraction for a fair assessment. Our approach involved utilizing specific relation labels, namely "question-answer," "proximate_V," "proximate_H," "no-link," "header-question," and "same-entity." Conversely, existing methods employed different techniques for relation extraction. To gain a better understanding of their methodologies, I recommend referring to the respective papers. These differences in approach account for the differences in performance.
Furthermore, our primary objective was to develop a technique that extracts word or entity relations solely based on layout information, without relying on textual data. By doing so, we aimed to achieve generalizability across diverse forms and documents, regardless of the language used. Our method leverages relative distances between words as edge features, enabling its applicability to different forms and documents. Notably, the spacing between word pairs and the distance between sections or headers tend to remain relatively consistent across various forms and documents, particularly those generated by computers.


Document sourcing of token from env var

You will need to run go generate to update the docs. https://github.com/1Password/terraform-provider-onepassword#generating-documentation

Good point! I had actually already created a GitHub Action to detect this. But forgot to push it. Opened a PR for that.


Problem copying objects with a / in the path with latest client

This was not a problem with the old client (0.22.1) we went up to the latest version and then back down.
When we try to copy an object with a name like: assets/abcde.png from one bucket to another, the from side in the request becomes URL escaped so the / becomes %2F and seems to break it.
https://storage.googleapis.com/storage/v1/b/asset-inbox/o/assets%2Fabcde.png/copyTo/b/asset-storage/o/assets/abcde.png

Reverting back to 0.22.1 fixed the issue for us but wanted to raise the issue.

I'm glad I found this issue. I've spent some time wondering where did I break my code that this stopped working. Well then, I'm just leaving a note that it affected me too, and I would appreciate it fixed.


Run test on local

Since we removed gomock in #15 , we don't need Docker environment for CI anymore.

CI became 40 seconds faster than before 🚀

before: 1 min 47 sec https://travis-ci.com/github/dtan4/nature-remo-to-cloud-watch-function/builds/153234511
after: 1 min 7 sec https://travis-ci.com/github/dtan4/nature-remo-to-cloud-watch-function/builds/153235371


error in example for trigger.startTime

Time when the trigger actually fired to invoke the pipeline run. For example, for a trigger that fires every 5 min, this variable might return something like this 2017-06-01T22:20:00.4061448Z, 2017-06-01T22:25:00.7958577Z, 2017-06-01T22:29:00.9935483Z respectively. (Note: The timestamp is by default in ISO 8601 format) --> SHOULD BE: Time when the trigger actually fired to invoke the pipeline run. For example, for a trigger that fires every 5 min, this variable might return something like this 2017-06-01T22:20:00.4061448Z, 2017-06-01T22:25:00.7958577Z, 2017-06-01T22:30:00.9935483Z respectively. (Note: The timestamp is by default in ISO 8601 format)

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

ID: befd9a68-45a2-9105-1b1b-16bf792a2d73
Version Independent ID: 2c2e1fa5-7824-5fe9-7382-bf5573a25be3
Content: System variables in Azure Data Factory
Content Source: articles/data-factory/control-flow-system-variables.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@aldelar Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@aldelar The PR (#87496) for the fix has been created and it will be merged soon. Changes will reflect in 24hrs. Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.
Thank you!


api: Support file uploads to the API through a Client.

Now, the Client.do_api_query() method supports sending files to the API.
This allows uploading all kinds of files from the backend, through a instance of the Client class.

Automated message from Dropbox CLA bot
@YagoGG, it looks like you've already signed the Dropbox CLA. Thanks!

Thanks for the PR!  I think a better way to implement this might be to add an optional "files" parameter to call_endpoint and do_api_query and maybe send_message.  This would make it possible for API users to upload files without us having to do isinstance checks in the way more common case of not passing in files.

yeah, I like @showell's suggested approach.

Okay, working on it!
mypy doesn't seem to have a type for a file object. Should we use Any instead?

You probably want to use typing.IO for a file-like object.

Okay, after talking with @showell, we decided that it was better having a separate method inside the client to upload files, instead of using send_message.
This has been added in the latest commit, as well as support for multiple files in do_api_query.

Awesome!

By the way, forgot to mention I created a small script to test this. You may want to use it, so you don't have to re-write your own testing tool. It can be found in this Gist.

You should clean up that script and add it as api/examples/upload-file

Sorry for the commit mess, Travis is reporting some strange problems.
I'll let you know when everything looks okay, so you don't have to be constantly fetching new updates.

Most of the CI issues are just lint/mypy things that should be easy enough to fix, I posted a guess for what the production suite issues are (basically I think it's just that it's using the API)

I really don't see where the problem is here. I've been struggling with it for a few days now, and couldn't find the bug.
Any advice or ideas on how can I get the full output of /zulip/zulip/tools/../puppet/zulip/files/nagios_plugins/zulip_app_frontend/check_send_receive_time --nagios --site=http://<IP_ADDRESS>:9991?

Thank you for your feedback, @showell!
I have made the changes you suggested, but the tests are still failing. Any ideas on why could this be happening? Surprisingly, no errors are reported in my remote dev instance.

The place where it's failing is the main place where we use the API bindings with existing endpoints in CI, so I think it's possible you're still breaking non-file-upload API usage.

The part of CI that is failing is tools/test-run-dev

Hmm, weird, I cannot reproduce this failing locally either.  I think it may be a caching issue -- CI may not be using the latest API bindings when you change them?

I suspect your code is good now; I'm going to try pushing a new branch to run CI fresh.  @YagoGG in the meantime, can you rebase to squash your changes together and write a nice commit message?

My retest passed, so I squashed the commits, line-wrapped the commit message, and merged this as 25dbe3574757357f5727bf7f7f88c9d07b3d7cfc
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/version-control.html#commit-messages
Thanks for all your work on this @YagoGG!!

I was waiting for everything to be okay to tidy it up. I'll try to keep everything squashed in future PRs, thanks @timabbott!


the UI for "Automate threat responses" is different

The current UI to assign playbooks to rules is completely different than the one described in "Automate threat responses" section.

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

ID: 9962c711-a464-85d7-07dc-29140ff9a715
Version Independent ID: bc9eaa33-6bb4-6c91-724e-589f022c2159
Content: Tutorial: Run a playbook in Azure Sentinel
Content Source: articles/sentinel/tutorial-respond-threats-playbook.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

@mariuszo-at-microsoft-com
Thanks for your feedback! We will investigate and update as appropriate.

Hi @mariuszo-at-microsoft-com thanks again for your continued feedback on docs!
I'm adding this into our backlog to check into and validate with the updated UI.
In the meantime, I'm going to close this issue, but feel free to continue commenting if you have more input.
We value your contributions on Microsoft docs!
#please-close


Error while sourcing with "Allow listing non-live and otherwise inactive elements." disabled in Schema

Description
Guess who's back.. na na na.. back again ;)
Gatsby build fails when this is unchecked,
 ERROR #11321  PLUGIN

"gatsby-source-craft" threw an error while running the sourceNodes lifecycle:

Failed to execute query LIST_forhandlere_forhandler_Entry.
Errors:
Unknown argument "status" on field "entry" of type "Query".

  510 |     if (remoteConfigVersion !== localConfigVersion || !localContentUpdateTime) {
  511 |         reporter.info("Cached content is unavailable or outdated, sourcing _all_ nodes.");
> 512 |         await sourceAllNodes(config);
      |         ^
  513 |     }
  514 |     else {
  515 |         reporter.info(`Craft config version has not changed since last sourcing. Checking for content changes since "${localContentUpdateTime}".`);

File: node_modules/gatsby-source-craft/gatsby-node.js:512:9

From Craft's web.log:
2021-12-02 11:40:33 [-][-][-][error][GraphQL\Error\Error] Unknown argument "status" on field "entry" of type "Query".

GraphQL (2:35)
1: query NODE_produkter_glasdore_Entry($id: [QueryArgument], $siteId: [QueryArgument]) {
2:   entry(id: $id, siteId: $siteId, status: null) {
                                     ^
3:     remoteTypeName: __typename

Steps to reproduce

Uncheck "Allow listing non-live and otherwise inactive elements." in your Craft GQL schema settings
Source data
Pull your hair out ;)

Additional info

Craft version: 3.7.23
PHP version: 7.3.31
Database driver & version: MySQL 5.6.51
Plugins & versions:
Cloner	1.2.2
Freeform	3.12.7
Gatsby Helper	1.1.0
Redactor	2.8.8
Smith	1.1.14
Spoon	3.6.1
Sprout Notes	2.2.3
Tags	1.0.9

That is the expected behavior, as Live Preview on a Gatsby site will fail otherwise. Also, there's a search option for the repository :)


Movie is not saved correctly in movie.APi service when receive data from rabbitMQ

Id is null when save to database

done


Revert "No gems in this project use Gemfury anymore, so remove its sourc...

Reverts alphagov/business-support-finder#73
Following a conversation on Slack in #developers:
[16:27] Alex Tomlins: Note: you can only drop the gemfury line if it's using a semi=recent version of govuk_frontend_toolkit.
[16:27] Alex Tomlins: where semi-recent means >= 0.41.1
[16:27] Alex Tomlins: Versions older than that are only on gemfury for $reasons.
Sorry, @benilovj. Didn't realise this!

@issyl0 would you mind adding a comment which explains the reasoning in the Gemfile and in the git commit?

@benilovj Comment added.


Versioning + NuGet

Enable automatic versioning and Nuget packaging of shippable Vipr binaries

Other than the folders renaming, everything LGTM!


chore: fix to allow ranged vscode versions

Codecov Report

Merging #44 (77fea51) into main (e509d4c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   98.73%   98.73%
=======================================
  Files           4        4
  Lines          79       79
  Branches       14       14
=======================================
  Hits           78       78
  Partials        1        1

Continue to review full report at Codecov.

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


Deleting objects that are still trying to hold on to old reactor after reset

What is the change?
This change is a follow on to #1729 after it was observed that the garbage collector was not clearing Reactor data from a state distribution on worker processors after a reset. This is due to objects retaining references to the distributed Reactor. Upon investigation, these references came from a cyclic reference between Core, Reactor, and spatialGrid objects, shown by this diagram created by objgraph:

The solution was quite simple: setting spatialGrid.core attribute to None, as well as the parent attribute in the Reactor children.
Why is the change being made?
Retaining the distributed state of the Reactor on workers is unnecessary memory overhead for interactions that don't require a distributed state.

Checklist

[x] This PR has only one purpose or idea.
[ ] Tests have been added/updated to verify any new/changed code.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes have been updated if necessary.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

Approved from my end! Not sure this can be tested, so I'll let John S comment on that.

Coming up with the test for this is not trivial to me. I'm not even sure what would be tested. Plus, the effectiveness of the change is largely dependent on the plugins and what interfaces are active.

Could you add this and the previous PR to the release notes? I think they are pretty important-to-mention updates!

Added an item to the release notes. I wasn't sure whether to put it at the top or bottom of the list lol.


Partner eCDN Support for Teams Live Events not present

I have a RAMP eCDN installed and it does not work with Teams Live Events, purportedly because that endpoint is still using older Skype Meeting Broadcast backend. Can someone confirm if that is old information, because it isn't working for me with Ramp, and Ramp has told me it's a known issue with Microsoft and work that's in progress but not completed yet.
We are working out logistics right now for a large event hosted by our CEO and trying to figure out ways around this, but the Teams Live Event method (with eCDN support) is exactly what we need, but it's not functioning right now.
Support call would be great if this is truly supported, but if it's not quite ready yet then might need to update this page.  Thanks!!

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

ID: 2b679199-caa5-4cc5-27e5-60a2cf999f69
Version Independent ID: 6878c954-e248-94c2-7602-47437ec7a2dd
Content: Plan for live events in Microsoft Teams - Microsoft Teams
Content Source: Teams/teams-live-events/plan-for-teams-live-events.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@c016smith thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Finish killing signalToFire

Apparently I forgot that with a change a while ago, now corrected.

This change is

:+1:


访问的时候产生了报错：{"data":null,"cause":{},"code":422,"name":"AssertionFailureError","status":42206,"message":"Failed to goto https://ddjjjc.gov.cn/pages/news.asp?id=4173: Error: net::ERR_CERT_COMMON_NAME_INVALID at https://ddjjjc.gov.cn/pages/news.asp?id=4173","readableMessage":"AssertionFailureError: Failed to goto https://ddjjjc.gov.cn/pages/news.asp?id=4173: Error: net::ERR_CERT_COMMON_NAME_INVALID at https://ddjjjc.gov.cn/pages/news.asp?id=4173"}

访问的时候产生了报错：{"data":null,"cause":{},"code":422,"name":"AssertionFailureError","status":42206,"message":"Failed to goto https://ddjjjc.gov.cn/pages/news.asp?id=4173: Error: net::ERR_CERT_COMMON_NAME_INVALID at https://ddjjjc.gov.cn/pages/news.asp?id=4173","readableMessage":"AssertionFailureError: Failed to goto https://ddjjjc.gov.cn/pages/news.asp?id=4173: Error: net::ERR_CERT_COMMON_NAME_INVALID at https://ddjjjc.gov.cn/pages/news.asp?id=4173"}
我是用requests请求的，这是i请求代码：
import requests
headers = {
'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36',
'Accept':'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3;q=0.7'
}
urllib = "https://r.jina.ai/https://www.ddjjjc.gov.cn/pages/news.asp?id=4173"
resource = requests.get(urllib,headers=headers)
print(resource.text)

hmm i cant access this website even from browser


Pestle modules are having security key/form issues in Magento 2.3

Seems like this might be the problem: https://github.com/magento/magento2/blob/5e07374555d09fca3522a19bb8f5e6e9d389a1a7/app/code/Magento/Backend/App/Request/BackendValidator.php#L173

OK, the above is a farwded controller that's missing a list -- the real problem seems to be the controller is missing from vendor/magento/framework/App/Router/ActionList.php

Yeah -- so it seems like Magento isn't picking up the controller files in the composer based module paths.  LONG SIGH HOBBIES ARE SUPPOSED TO BE FUN.

Magento thinks my controllers are in this folder.
/Users/alanstorm/Sites/magento-2-3-x/project-community-edition/extensions/pulsestorm-helloworld/
So -- I still need to get etc registration.php and composer.json sorted out.


why is m4a higher then mp3 in format sorting when using --prefer-free-formats

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.08.19 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
the reason i bring this up is because mp3's patents expired in 2017
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Why shouldn't it be?

Why shouldn't it be?

because m4a is not a free format (since it is a subset of mp4), mp3 is a free format since 2017.
shouldn't mp3 be higher in format sorting when using --prefer-free-formats then

Ok, makes sense. This order was set long before the mp3 patents expired. But why is this a question and not a bug report? Give me an example URL where both formats are available

Actually, since this is a change that breaks compatibility, I suggest you request this in youtube-dl too

actually the description of the --prefer-free-formats option is inaccurate, since it says "Prefer video formats with free containers
over non-free ones of same quality." despite it working with audio

On a side note, why, according to the readme, is Ogg preferred over Opus, unless --prefer-free-formats is passed?
Also, while it's no longer an issue, why is aac preferred over mp3 unless --prefer-free-formats is passed?

The ordering b/w formats that are generally never encountered together is probably not implemented correctly. Instead of asking me why, tell me how you think it should be instead - with justifications

Actually, since this is a change that breaks compatibility, I suggest you request this in youtube-dl too

ping @dirkf

Ping received and my thumbs support the two comments here regarding clarification of the desired functionality.

The historic options in yt-dl and yt-dlp aren't necessarily well chosen. Eg --recode-video should have been --remux-video. Similarly, despite the fact that format is generally used throughout to describe a combination of video and/or audio codec parameters in a specific container type, we have --prefer-free-formats instead of --prefer-free-container-formats or similar

No, ytdl's --recode-video does re-encode the video, not just remux it.

Yes, sorry for that misinformation.
Inquiring minds may like to know that ffmpeg doesn't default to just copying the streams: rather, it associates a default video and audio encoder with each container type and transcodes each stream with that encoder, unless overridden, and even if the source stream actually has a compatible encoding. Contrary to my assumption, yt-dl doesn't use -c copy in the --recode-video option.
This behaviour is handy, improving the likelihood of a valid playable output file, unless (a) transcoding is not required but you lose a generation anyway (b) the platform is poorly matched to the transcoding task.


docs(Quay): update user doc

GitHub issue: https://github.com/janus-idp/backstage-plugins/issues/303
cc @rkratky

Approving with a few suggestions. It's encouraging to see you all using advanced Markdown features (I learned some new tricks myself while reviewing this PR). One suggestion that I would have overall for these README files is to put the user-related sections on top of administrator-related sections if you expect more frequent readership among the users (compared to an admin who just installs it).

@max-cx Thank you for the review and suggestions, I've incorporated the changes as suggested.

@tumido Could you please review the changes in this PR?


Expose RSS feed

From what I can see, the website doesn't say that an RSS feed is available anywhere. I had to go to this repository and search for "RSS" in order to find the feed URL. It would be nice to have it available somewhere, perhaps through a <link> in the <head>.

It is exposed on the about page now, but a <link rel="alternate" type="application/rss+xml" href="/feed.xml" /> would still help.


Move common env to runs.env

Moves GITHUB_TOKEN, GH_REPO, TRUE to runs.env

This is invalid syntax with composite


canvas blob for WORD

Unhandled Rejection (TypeError): Cannot read property 'toBlob' of null
canBlob
src/components/DrawWordTool.jsx:147
  144 | const castWord = async () => {
  145 |   let canvas = document.getElementById('drawword-canvas');
  146 |   const canBlob = await new Promise((resolve, reject) => {
> 147 |     canvas.toBlob((blob) => resolve(blob));
      | ^  148 |   });
  149 |   let ipfsCanvasResult;
  150 |   try {
View compiled
castWord

fixed

castWord ipfsResult:
cid:
QmRp2t5VZJEZkG4bMEvzcN3wuuy8dZqQ6BkxZGUhZVkACq
{"description":"A Grapheme NFT Word","external_url":"https://austingriffith.com/portfolio/paintings/","image":"ipfs://QmP5LQvuVvBpSNJpmniaohCMk8rHW7gKQxHwLS9cU6yG6E","name":"LLM","attributes":[{"trait_type":"tokenId_0","value":16},{"trait_type":"row_0","value":1},{"trait_type":"col_0","value":1},{"trait_type":"tokenId_1","value":17},{"trait_type":"row_1","value":1},{"trait_type":"col_1","value":21},{"trait_type":"tokenId_2","value":15},{"trait_type":"row_2","value":1},{"trait_type":"col_2","value":41},{"trait_type":"tokenId_3","value":13},{"trait_type":"row_3","value":1},{"trait_type":"col_3","value":61}]}
https://cloudflare-ipfs.com/ipfs/QmP5LQvuVvBpSNJpmniaohCMk8rHW7gKQxHwLS9cU6yG6E

This works.


productid is no longer required field for revenue

This relaxes the constraint that productId is required when logging revenue via logRevenueV2. This update is necessary before we can update the Segment JS integration to use logRevenueV2 since Segment assumes productId can be null, which is valid for web.

lgtm


Replace link with code tag for Android framework classes

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


[Testing Confirmation] Confirm current testing requirements

As part of the discussion around implementing an organization-wide testing policy, I am visiting each repo to see what tests they currently perform. I am conducting this work on GitHub so that it is easy to reference.
Looking at the Notifications repository, it appears there is

Repository
Unit Tests
Integration Tests
Backwards Compatibility Tests
Additional Tests
Link

Notifications
- [ ]
- [x]
- [ ]
Link checker
https://github.com/opensearch-project/notifications/issues/622

I don't see any requirements for testing. If there are any specific requirements could you respond to this issue to let me know?
From looking at the workflow file, it seems like the security-notifications-test-workflow.yml test is an integration test? Is this correct?
If there are any tests I missed or anything you think all repositories in OpenSearch should have for testing please respond to this issue with details.

I believe we do have unit tests as well for this plugin?
https://github.com/opensearch-project/notifications/tree/main/notifications/notifications/src/test


Failed to find post key

it show that:
GET: https://accounts.pixiv.net/login? with params: None >>> Reason: HTTPSConnectionPool(host='accounts.pixiv.net', port=443): Max retries exceeded with url: /login (Caused by ConnectTimeoutError(<urllib3.connection.VerifiedHTTPSConnection object at 0x0000000003999898>, 'Connection to accounts.pixiv.net timed out. (connect timeout=10)'))
!!! GET failed: https://accounts.pixiv.net/login? params: None

params: None in dicates that there isn't any paramter passed along with the login.
I guess that you attempted login without username and password perhaps?
Could you please show me how did you login?
Thanks

I added a proxies variable in settings.py which is used when sending requests

.... emmm I assume it solved your problem


withComponent question

I'm working on a project where I have external component repos all built with styled components.
When trying to use withComponent internally like below it works as intended:
const FunnelTitle = ({ funnelName, style }) => {
  const Container = styled.div`
    display: flex;
    justify-content: flex-start;
  `;

  const Title = styled.h1`
    font-family: Raleway, sans-serif;
    font-size: 28px;
    fontWeight: 600;
    line-height: 1.21em;
    margin: 0;
    text-transform: capitalize;
  `;

  const SubTitle = Title.withComponent('h3');

  return (
    <Container style={style}>
      <Title>
        {funnelName} funnel
      </Title>
      <SubTitle>
        Subtitle me
      </SubTitle>
    </Container>
  );
};

Yet when I try to use an external repo, all built with styled-components I get a _Button2.default.withComponent is not a function error. With the below code:
import Button from '@external/atoms/dist/atoms/Button';

const FunnelTitle = ({ funnelName, style }) => {
  const Container = styled.div`
    display: flex;
    justify-content: flex-start;
  `;

  const Title = styled.h1`
    font-family: Raleway, sans-serif;
    font-size: 28px;
    fontWeight: 600;
    line-height: 1.21em;
    margin: 0;
    text-transform: capitalize;
  `;

  const SubTitle = Button.withComponent('h3');

  return (
    <Container style={style}>
      <Title>
        {funnelName} funnel
      </Title>
      <SubTitle>
        Subtitle me
      </SubTitle>
    </Container>
  );
};

Is this the intention or is this a bug? I am using version 2.1.2.
Thanks

Version
2.1.2

First of all, you should move your styled-components definitions out of your render method-otherwise you're creating a new component on every render unnecessarily which could have a large performance impact:
const Container = styled.div`
    display: flex;
    justify-content: flex-start;
  `;

  const Title = styled.h1`
    font-family: Raleway, sans-serif;
    font-size: 28px;
    fontWeight: 600;
    line-height: 1.21em;
    margin: 0;
    text-transform: capitalize;
  `;

  const SubTitle = Title.withComponent('h3');

const FunnelTitle = ({ funnelName, style }) => {
  return (
    <Container style={style}>
      <Title>
        {funnelName} funnel
      </Title>
      <SubTitle>
        Subtitle me
      </SubTitle>
    </Container>
  );
};

Second, it sounds like your external <Button /> is not a styled component directly. Only styled-components have a .withComponent method, not normal React components!
If it is a styled component, not a normal React component, then the issue might be that your external component ships with styled-components v1.

@mxstbr thanks very much

I was just wondering how the name of withComponent was choosen...
Wouldn't a name like copyStyleTo make more sense?


One-Hot-Encoding

Hi,
I just added a question and answer inc. a ressource and explanation of myself to the lesson2.md about one-hot-encoding.

Hello NiWaRe, Thanks a lot for your pull request. I would have love to add this pull request. Although, as of now there are some issues with this pull request. I would recommend you to to do following: do a spell check, simplify your explanation and do simple formatting like in adding links. Thanks again for making this effort. :)

Hey, alright just rewrote / reformatted everything.
Is it fine now?

Hi, it definitely is better now but I am unable to merge your pull request.  Getting this error: "This branch has conflicts that must be resolved". Can you try doing this: https://github.com/githubteacher/github-for-developers-sept-2015/issues/648

Updated Pull-Request, should be mergable now!
(Added One-Hot-Encoding at the end of lesson2.md)


KAFKA-12648: minor followup from Pt. 2 and some new tests

Addresses the handful of remaining feedback from Pt. 2, plus adds two new tests: one verifying a multi-topology application with a FKJ and its internal topics, another to make sure IQ works with named topologies (though note that there is a bit more work left for IQ to be fully supported, will be tackled after Pt. 3

also cc @wcarlson5

Merged to trunk


Changing ContainerType property to string

The possible values returned by the service can't be modeled as values of an enum.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[ ] 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 information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

This change is

Merged

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-ruby/pull/525

No modification for Python


Publish packages on pull request

Not sure what the best CI solution is, but it would be nice to automatically publish to NPM from a Github action or something like that.

I'm a big fan of GitHub actions. This looks like a prebuilt one that handles it: https://github.com/marketplace/actions/publish-to-npm

Sweet

any idea how to run yarn prepare beforehand?

Can't the commands just run beforehand?
For example, here's some potentially how the YAML would look:
# Github workflow for running the build process for the vitruly website
# this workflow will only run when a push is made to either the dev branch
# or the master branch

name: npm-publish
on:
  push:
    branches:
      - master

jobs:
  publish:
    runs-on: ubuntu-latest

    steps:
      - name: prepare
         run: yarn prepare
Other steps from:  https://github.com/marketplace/actions/publish-to-npm goes here

Got it thanks, new to Github Actions so appreciate it.

https://intuit.github.io/auto/

https://intuit.github.io/auto/

I think we should integrate auto now that we have the monorepo merged.

I think we should integrate auto now that we have the monorepo merged.


saveCells with unchanged cell throws 400 bad request error

This happens when calling saveCells with an unchanged/already-saved cell in the array:
Error: Google API error - [400] Invalid requests[0].updateCells: At least one field must be listed in 'fields'. (Use '*' to indicate all fields.)
    at createError (/mnt/c/Users/david/proj/google-spreadsheet-v2v3-wrapper/node_modules/axios/lib/core/createError.js:16:15)
    at settle (/mnt/c/Users/david/proj/google-spreadsheet-v2v3-wrapper/node_modules/axios/lib/core/settle.js:17:12)
    at IncomingMessage.handleStreamEnd (/mnt/c/Users/david/proj/google-spreadsheet-v2v3-wrapper/node_modules/axios/lib/adapters/http.js:236:11)
    at IncomingMessage.emit (events.js:214:15)
    at IncomingMessage.EventEmitter.emit (domain.js:476:20)
    at endReadableNT (_stream_readable.js:1178:12)
    at processTicksAndRejections (internal/process/task_queues.js:80:21) {
  config: {
    url: ':batchUpdate',
    method: 'post',
    data: `{"requests":[{"updateCells":{"rows":[{"values":[{}]}],"fields":"","start":{"sheetId":603529019,"rowIndex":2,"columnIndex":0}}},{"updateCells":{"rows":[{"values":[{}]}],"fields":"","start":{"sheetId":603529019,"rowIndex":2,"columnIndex":1}}}],"includeSpreadsheetInResponse":true,"responseIncludeGridData":true,"responseRanges":["'Test 1'!A3","'Test 1'!B3"]}`,
    headers: {
      Accept: 'application/json, text/plain, */*',
      'Content-Type': 'application/json;charset=utf-8',
      Authorization: 'Bearer XXXXXXXXX',
      'User-Agent': 'axios/0.19.2',
      'Content-Length': 356
    },
    baseURL: 'https://sheets.googleapis.com/v4/spreadsheets/1H7xxxxxxxxxx',

...

Do you have a larger code sample of what you're trying to do?

It should be as simple as this to reproduce:
await sheet.loadCells('A1')
const cell = sheet.getCell(0, 0)
await sheet.saveCells([cell]) // fails

If cell.value = 123 is before the save, it doesn't fail.
I'd expect it to just do nothing if the cell wasn't changed.

Ah got it. All my code and the tests are using on saveUpdatedCells which filters out unchanged cells and then calls saveCells. Generally this is the easier way to do things.
But regardless this shouldn't break. I'll take a look and try to get a fix in.

fixed in the latest version. Thanks for the report!


how to import ai.0.21.5-build00175.js file

I have download the package from nuget. It creates a file called ai.0.21.5-build00175.js.  I couldnt find any sample html code about how to include this on my html code. All of them show examples for using the cdn. How do I use my local version?

we expose 'URL' config parameter, so it should look like this in your js snippet:
`

Thank you.
One more question. I would like to set the user id. However, since the javascript is loaded async, the browser executes the user setting function early, and causing a not defined error. How can I set the user id, after the javascript it loaded

You can set user id by calling setAuthenticatedUserContext. This is available in snippet before AI is fully loaded.
Note that it's different from our internal user id which is not supposed to be set - one disadvantage of using it is that it's not updated in cookies and therefore not sent to server SDK (if you use it on server side) - unless you update cookies manually. With setAuthenticatedUserContext there's no such problem.

Hello,
I'm getting the following error.
setAuthenticatedUserContext is not defined
Code is like below:
<script type="text/javascript">
    var appInsights = window.appInsights || function (config) {
        function s(config) { t[config] = function () { var i = arguments; t.queue.push(function () { t[config].apply(t, i) }) } } var t = { config: config }, r = document, f = window, e = "script", o = r.createElement(e), i, u; for (o.src = config.url || "//az416426.vo.msecnd.net/scripts/a/ai.0.js", r.getElementsByTagName(e)[0].parentNode.appendChild(o), t.cookie = r.cookie, t.queue = [], i = ["Event", "Exception", "Metric", "PageView", "Trace"]; i.length;) s("track" + i.pop()); return config.disableExceptionTracking || (i = "onerror", s("_" + i), u = f[i], f[i] = function (config, r, f, e, o) { var s = u && u(config, r, f, e, o); return s !== !0 && t["_" + i](config, r, f, e, o), s }), t
    }({
        instrumentationKey: "XXXXX",
        url: '/scripts/ai.0.21.5-build00175.min.js'
    });

    window.appInsights = appInsights;
    appInsights.trackPageView();
</script>

    <script>
        setAuthenticatedUserContext('45c072bd-eef3-46bb-8c14-5b362f04098a');
    </script>

You have an old snippet, you have to take a new one from AI portal.

thank you


Second USB Type C port not working.

Hi, I've tried to use the second USB C port (not the charging port) and It din't work.
The charging port works perfectly.
Do I need to activate it in some way?
Best regards.
Btw, thank you @stevezhengshiqi for your awesome support!

Hi, my second USB C port works great. Could you send me a trouble shooting? I need a ioreg file generated by IORegExplorer and your Clover folder.(Plug the U disk into the second USB C port when you generate the ioreg file)
IORegistryExplorer.app.zip

Sorry for the long wait answer, I don't have a  U disk. I'm connecting a USB C to HDMI adapter and is not working. Should I do the same troubleshooting anyway?
Regards!

Second USB-C is only for usb devices.
Only first usb-c can charge and act like HDMI
p.s. it's not macOS specific, it's Xiaomi Pro's port limitation.

Ok, so this Issue is not really a Issue :P I need to buy a dongle to use the same port for hdmi and charging.
Thank you anyway for all your help!


Improved the abstraction of dynamics - [merged]

In GitLab by @cmastalli on Nov 21, 2018, 09:31
Merges topic/dynamics -> devel
This PR improves the abstraction of dynamics (task #19). The main contribution is that the dynamics systems as the form of:
d/dt[q; v] = [v; a(q,v,u)].
and, as a consequence, its linearise model is:
[dv; da] = [0, I; fx, fv]*[dx; dv] + [0; fu]*du
where dx is the linearised configuration point. Note that the linearised point belongs to the tangential space.
These tasks can be listed as:

Updated the state and control in the DDP backward calculations, not in the dynamic one.
Removed some inefficiencies in the floating-base dynamics
Included operators in the discretizer class (there aren't used yet)
Created a data class for the discretizer.
Added aq, av, au and discretizer data in the base class for dynamic data.
Used the dynamic model for the data creation, not the ddp model.
Added some documentation to state clear our definition of dynamic model.
fx, fu and dt in the base class for the discretization data.

In GitLab by @cmastalli on Nov 21, 2018, 09:32
mentioned in commit 5c68b798f638f100929d3f049c7d62283a7f85d0

In GitLab by @cmastalli on Nov 21, 2018, 09:32
merged


Start proxy in worker

When pod is scheduled to worker, it cannot be started correctly - missing proxy in worker. Starting proxy in worker is aligned with production minion deployment.

Any reason kube-proxy is still needed when Mizar is in place ? is it a workaround for mizar issues ?


Log improvement

Hi GioviQ
I'll put online a new release later today with your improvements.
Thanks for your contributions!


Current state of support on ARM platforms

I am wondering what the current status of support for ARM architecture is (specifically armv7 raspberry pi running ubuntu 16.04) From my research on the web and this github project community, I've learned that Gperftools memory profiling on ARM is not as robust as on x86.
I've had some success on ARM using libunwind with very simple applications to verify that memory profiling is possible, but as soon as I attempt to instrument a complex application, it crashes somewhere in libunwind. With all the different ways of capturing the stacktrace (libunwind, libgcc unwind etc.) validation of the unit tests via make check results with, at best, 8 failures. Most of these failures happen in the heap-checker tests.
The discussion in https://github.com/gperftools/gperftools/issues/751 (closed Feb 21, 2016) seems to indicate that memory profiler seems to work with liggcc-based unwinder. I have been unable to get it to work. Like the author in issue #751, the heap info is generated but it is missing the information. Is there anything else I am missing? Are there any other flags or parameters I need to pass other than
--enable-stacktrace-via-backtrace --enable-emergency-malloc
What is the recommended way to use the heap-profiler feature on ARM, if any?

Yes, unfortunately libunwind doesn't seem to be 100% supported on less major platforms. That's why not linking with libunwind and using, say, libgcc should work better.
Keep in mind however, that a) this won't work with signal handlers and thus with cpu profiler b) all dwarf-based unwinders (and whatever ARM uses are similar) are super slow. Which makes them nearly useless for stuff that is doing backtrace capturing often (e.g. heap profiler).
As for your problem specifically, keep in mind that libunwind does interpose it's own backtrace() implementation over glibc's. So if you want to use glibc's backtrace that uses libgcc, you need to stop linking with libunwind.
Ideal solution for quick backtrace support for arm would be frame-pointer-based asm unwinder. But that requires someone contributing one.

Thank you for your response.
Sounds like at this point, libgcc's backtrace is the only option left. I have not been able to profile on ARM with libgcc's stacktrace. Specifically, I am using the gperftools API and calling HeapProfilerDump(...).. but the snapshot that gets created does not have the information so that pprof can visualize the call graphs.
I am wondering if you have successfully profiled the heap on ARM using libgcc's backtrace. From #751, it sounds like you have. Perhaps I am mis-configuring it. I set these flags:

./configure --enable-stacktrace-via-backtrace --enable-emergency-malloc

(although my assumption is that by default it will configure to use libgcc's s backtrace if libunwind is installed) Or I am just currently out of luck.

Try running ./stacktrace_unittest with TCMALLOC_STACKTRACE_METHOD_VERBOSE set to 1.
E.g.
$ TCMALLOC_STACKTRACE_METHOD_VERBOSE=1 ./stacktrace_unittest
It will tell you what it is using.
Also in order to get libgcc-enabled stacktrace capturing you don't need to pass --enable-stacktrace-via-backtrace. Just emergency malloc should be enough.

Results below.
TCMALLOC_STACKTRACE_METHOD_VERBOSE=1 ./stacktrace_unittest

Chosen stacktrace method is libgcc
Supported methods:
* libgcc
* generic
* arm

Obtained 8 stack frames.
Obtained 8 stack frames.
./stacktrace_unittest() [0x10df6] 0x10df6
./stacktrace_unittest() [0x11042] 0x11042
./stacktrace_unittest() [0x110fc] 0x110fc
./stacktrace_unittest() [0x111b8] 0x111b8
./stacktrace_unittest() [0x11274] 0x11274
./stacktrace_unittest() [0x11322] 0x11322
./stacktrace_unittest() [0x10bb8] 0x10bb8
/lib/arm-linux-gnueabihf/libc.so.6(__libc_start_main+0x9d) [0x76d7e8aa] 0x76d7e8aa
CheckStackTrace() addr: 0x112fd
Backtrace 0: expected: 0x10dec..0x10eda  actual: 0x10df6 ... OK
Backtrace 1: expected: 0x11038..0x11046  actual: 0x11042 ... OK
Backtrace 2: expected: 0x110f0..0x11100  actual: 0x110fc ... OK
Backtrace 3: expected: 0x111ac..0x111bc  actual: 0x111b8 ... OK
Backtrace 4: expected: 0x11268..0x11278  actual: 0x11274 ... OK
Backtrace 5: expected: 0x11314..0x11326  actual: 0x11322 ... OK
PASS

$ TCMALLOC_STACKTRACE_METHOD_VERBOSE=1 ./heap-checker_unittest.sh
Testing ./heap-checker_unittest with HEAPCHECK= ... OK
Testing ./heap-checker_unittest with HEAPCHECK=local ... FAILED
Output from the failed run:
----
WARNING: Perftools heap leak checker is active -- Performance may suffer
Chosen stacktrace method is libgcc
Supported methods:
* libgcc
* generic
* arm

0x0 does not seem to point into code of function Mmapper at 0x13c1d! Stack frame collection must be off in MemoryRegionMap!Aborted (core dumped)
----

ARM does have a number of failing unittests, but basic things still work.
So in your case, we do have working backtrace capturing. Which means your heap profiles should be working.

Unfortunately, the heap profile does not have information needed by pprof. Below is a quick test I am trying to run to validate that heap profiling works. Do you have any other suggestions? I appreciate your help thus far.
#include "gperftools/malloc_extension_c.h"
#include "gperftools/heap-profiler.h"
#include <stdlib.h>

void foo()
{
    malloc(12345);
    HeapProfilerDump("Reason");
}

int main()
{
    HeapProfilerStart("profile");
    if (IsHeapProfilerRunning() == 0) {
        return -1;
    }
    foo();
    foo();
    foo();
    foo();

    return 0;
}

 g++ Hello.cxx -I$(GPERF_HOME)/src -L$(GPERF_HOME)/.libs -fno-builtin-malloc  -fno-builtin-calloc -fno-builtin-realloc -fno-builtin-free -ltcmalloc
cat profile.0003.heap
heap profile:      3:    37035 [     3:    37035] @ heapprofile
     3:    37035 [     3:    37035] @

MAPPED_LIBRARIES:
....

am facing the similar issue where heap profiler for ARM doesnt report the exact trace where memory leak is observed rather it shows only leak is observed

heap profile:      6:      221 [     6:      221] @ heapprofile
1:       33 [     1:       33] @
1:       10 [     1:       10] @
1:       10 [     1:       10] @
1:       10 [     1:       10] @
1:       10 [     1:       10] @
1:      148 [     1:      148] @

MAPPED_LIBRARIES:

any solution is available in gperftools 2.6.1 version

Most recent everything is nearly perfect now. legacy 32 bit arms don't have working frame pointer stacktrace capturing but libgcc/libunwind work. And aarch64 is just pefect. All tests pass.


[SERV-243] Use GitHub Actions instead of Travis CI

You need to remove the Travis-CI run as a required check from the branch protections.

Naming the workflow file run-tests.yml seems to be a convention, at least for a few large Python projects e.g. requests, aws-cli.


Disambiguate files with same name in the tab bar

When multiple files are opened, the title of each tab is set equal to the name of the file. However, if two files have the same name, this makes thing confusing. One disambiguating scheme that could be used is to copy Emacs that scans each path back one folder at a time until it finds the folder whose name doesn't clash, and then adds that folder's name to the tab name. For example, two files with the path C:/a/b/c/d.txt and P:/a/e/c/d.txt will be shown as d.txt <b> and d.txt <e> respectively.

Done in 9c1f8ef37b7ab450284f8183a163ca01bde5fdd0.


Add typst-chords

Hi, I wrote this lib https://github.com/ljgago/typst-chords to generate chords diagrams and lyrics with chords, you can generate chords of stringed instruments such as guitar, ukulele, etc.
I added a Music section with this lib.
Thanks!

Thanks for the awesome contribution @ljgago !


FreePROBATE MEGA epic

Done (this sprint)

Create tools/pathway for  Digital Photograph ⇒ Machine learning ⇒ CSV code
Document how to do this
Test document to ensure we understand what we need to do

Further steps
A. Create structure for the digital photographs of the books
B. Place existing books in the structure
C. Photograph remaining books
4. Create a branch of Myopic Vicar
5. Write ingestion script
User can search on FreePROBATE by name, county and date
6. Automate OCR, processing and ingestion
[after 5, but before 8]. Add link to image
Options
(a) cropped portion
(b) full page
(c) hybrid, full page with cropped portion clear, but remaining part of page greyed
7. Use geocoded linked open data and personal names from our other sites to produce master lists against which to check quality of data, marking records that are suspect.
8. Create correction of OCR app (cf FreeREG #1362 - may be adaptation of this, or vice versa)
Options
(a) corrected data is on FreePROBATE as additional, rather than amended version, or,
(b) corrected data is the only version (as is the case with FreeREG now.

Examples of Norfolk Probate Images

Hi @Captainkirkdawson adding such images would be quite different from the modern half FreePROBATE project (the subject of this issue).  .  Earlier wills (handwritten, rather than already transcribed, and printed) are being considered, but needing an entirely funded project to do so (preliminary bid sent to the Heritage Lottery Fund). Documentation is in the FreePROBATE folder - shared now. we do need people like yourself with familiarity to advise on the wider context.


Use Windows 2019 for build-test instead of 2016

Codecov Report

Merging #4611 (8031d79) into dev (f905fd0) will not change coverage.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from f905fd0

msal-browser
87.80% <ø> (ø)

Carriedforward from f905fd0

msal-common
85.08% <ø> (ø)

Carriedforward from f905fd0

msal-core
82.67% <ø> (ø)

Carriedforward from f905fd0

msal-node
82.20% <ø> (ø)

Carriedforward from f905fd0

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from f905fd0

msal-react
92.46% <ø> (ø)

Carriedforward from f905fd0

node-token-validation
83.41% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.


New post ideas

Technically, this is a test issue for the Magic Eight Ball action, but hey, feel free to comment with post ideas too!

🎱 Should I write another article this week?

🎱 Should I write another article this week?

I thought it'd be a good idea to check if normal comments would not trigger the action :-)

🎱 Will I have a good day today?


endStep stepUnits in subhourly wrf grib2 output

Enable reading grib2 files with subhourly data from the WRF model such as gs://high-resolution-rapid-refresh/hrrr.20230928/conus/hrrr.t00z.wrfsubhf01.grib2
The error: WrongStepUnitError(Wrong units for step (step must be integer))
Is now caught and logged. The bad coordinate attribute is dropped.
2023-10-04T17:40:23.741Z MainProcess MainThread WARNING:grib2-to-zarr:Ignoring coordinate 'step' for varname 'dswrf', raises: eccodes.WrongStepUnitError(Wrong units for step (step must be integer))

(ref #363 )

(ref #363 )

@dcherian Do you have any suggestions on how to resolve this based on your work last week (very excited by the way!)?
My read is that the HRRR grib2 attributes/coords are actually an invalid type according to eccodes... but I am not sure.
My solution was to just catch and log the exception and move on, but would love to solve the actual problem if possible?

Have you asked over at ecCodes? I think this needs to be solved there.
Can you link to a file that exhibits this problem?

Can you link to a file that exhibits this problem?

gs://high-resolution-rapid-refresh/hrrr.20230928/conus/hrrr.t00z.wrfsubhf01.grib2

Have you asked over at ecCodes? I think this needs to be solved there.

@martindurant will you merge a PR that catches and logs exception till we can get this sorted?

I will try upgrading to the latest eccodes and then file a bug if this is still an issue...

will you merge a PR that catches and logs exception till we can get this sorted?

Certainly. The HDF module already does this.

xref https://github.com/ecmwf/cfgrib/issues/335

xref ecmwf/cfgrib#335

Thank you for finding those - I don't see anything here that is likely to get fixed quickly here in eccodes - do you?

Absolutely, this is what kerchunk.grib._split_file does, e.g., as used in scan_grib:
    with fsspec.open(url, "rb", **storage_options) as f:
        for offset, size, data in _split_file(f, skip=skip):

Should be ready to go now?

Something changed in numcodecs entrypoints?

I am so confused - that can't be related to my changes right?

Perhaps pin to numcodecs<=0.11, and we'll figure it out in a separate PR

OK, done!
We'll come back to the numcodecs thing...


Add function to ingest OpenSearch documentation

Signed-off-by: Abbas Hussain<EMAIL_ADDRESS>Description
Code for AWS Lambda function to ingest documentation. The documents to index will generated by Jekyll plugin _plugins/search-indexer.rb
Issues Resolved
#17
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[x] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Need to close this as this PR is not accepting new commits after repository was changed from private to public.


dont send CANCEL from RequestResponseHandler

Addresses #271
A RequestResponseHandler will never have to send CANCEL for response (according to spec)

Should handleStream be updated as well?


main index page Active Connections CPU load is very heavy

http://xxxx/cgi-bin/luci/;stok=xxxx/admin/status/

Active Connections

wc -l /proc/net/nf_conntrack # CPU load is very heavy
cat /proc/sys/net/netfilter/nf_conntrack_count #Please use this cmd.

Source code

modules/luci-base/luasrc/sys.lua|321 col 46| local ok, nfct = pcall(io.lines, "/proc/net/nf_conntrack")
modules/luci-mod-admin-full/luasrc/view/admin_status/index.htm|41 col 41| fs.readfile("/proc/sys/net/netfilter/nf_conntrack_count")) or 0
modules/luci-mod-admin-full/luasrc/view/admin_status/index.htm|44 col 30| luci.sys.exec("sysctl net.nf_conntrack_max") or
modules/luci-mod-admin-full/src/luci-bwc.c|428 col 36| const char *ipc = stat("/proc/net/nf_conntrack", &s)
modules/luci-mod-admin-full/src/luci-bwc.c|429 col 43| ? "/proc/net/ip_conntrack" : "/proc/net/nf_conntrack";

Do you have an example where that code wc -l /proc/net/nf_conntrack is currently used in the LuCI master?
That line was recently replaced in master and 17.01 with /proc/sys/net/netfilter/nf_conntrack_count by https://github.com/openwrt/luci/commit/7ae244a26ef516511a5e476dd14ec033501a1d8d , so I am not sure where you run into that code in the current LuCI. (naturally it can still be found in some ancient branches)

Great.
15.05.1  LuCI (git-16.018.33482-3201903) have this issue


Desktop right sidebar

Description
Add Desktop right sidebar for Trending quests etc

[ ] Only available in Desktop
[ ] Create a RightSidebar screen, some implementation are in the layout already
[ ] Add TABS for this Right bar:  Trending, Quests.
[ ] Add some data as examples using Nostr hooks, Launchpad hooks etc
[ ] Add Right sidebar to the Router.ts and navigator for Main, Degen etc
[ ] Add this in the Router.ts component maybe (some impl have been try to add two DrawerNavigator)

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hey! I'm Lau Chaves, and I would like to contribute to this issue!
I have over 5 years of experience with JavaScript, ReactJS,React Native TypeScript, and Ruby. I’m now enthusiastic about web3 and eager to learn more! I’m also a member of Dojo Coding Costa Rica.
How I plan on tackling this issue
Id follow these steps:

Create a new screen: Make a new screen called "RightSidebar".
Use existing layout: Some parts of the layout might already be helpful.
Add tabs: Create tabs for "Trending" and "Quests".
Use sample data: Ill get example data from Nostr and Launchpad.
Add sidebar to router


[BEAM-3187] Ensure that teardown is called in case of Exception on the Spark runner

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @jbonofre

This is just to ask for feedback and validate the idea(s).
R: @amitsela @aviemzur

SUCCESS
--none--

SUCCESS
--none--

SUCCESS
--none--

Run Spark ValidatesRunner

FAILURE
--none--

FAILURE
--none--

Run Spark ValidatesRunner

Ok, this should be ok now, I just rebased and made the exception testing methods of ParDoLifecycleTest part of the ValidatesRunner test suite, notice that if you executed this before it broke for the SparkRunner, so it seems we should be ok now. Waiting for feedback/merge in case you find it is ok.

Thanks for the update. I gonna take a look.

Waiting for #4170 to be merged with the enabled ValidatesRunner before I can update this one.

Run Spark ValidatesRunner

Tests failed for timeout in Spark ValidatesRunner, just created #4194 to address this.

Retest this please

Run Spark ValidatesRunner

@amitsela  Please take a look it should be ok now, and the tests pass but jenkins is a bit crazy.

@amitsela just rebased after the move to spark 2. Should be now ready to merge. PTAL.

Run Spark ValidatesRunner

retest this please

Run Spark ValidatesRunner

Run Spark ValidatesRunner


No permission handler detected - via command line

Bug summary
If I run the project from Xcode, all works fine.
If I run npx react-native run-ios --simulator="iPhone 13" I have the no permission handler detected.
I've read the README, and probably I need to add the pre_install part but I don't know how to modify it from my actual situation.
This is  my Podfile. Is it correct? how should I add the pre_install?

I've deleted DerivedData, uninstalled the app,

require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '11.0'

target 'eehe' do
  config = use_native_modules!

  use_react_native!(
    :path => config[:reactNativePath],
    # to enable hermes on iOS, change `false` to `true` and then install pods
    :hermes_enabled => false
  )

  target 'eeheTests' do
    inherit! :complete
    # Pods for testing
  end

  use_frameworks! :linkage => :static
  pod 'AmazonFreeRTOS'
  pod 'AWSAuthUI'
  pod 'AWSMobileClient','~> 2.17.0'
  pod 'AWSUserPoolsSignIn'
  pod 'react-native-aws-freertos', :path => '../node_modules/react-native-aws-freertos'

  # react-native-permissions
  permissions_path = '../node_modules/react-native-permissions/ios'
  pod 'Permission-BluetoothPeripheral', :path => "#{permissions_path}/BluetoothPeripheral"
  pod 'Permission-Notifications', :path => "#{permissions_path}/Notifications"
  pod 'Permission-LocationWhenInUse', :path => "#{permissions_path}/LocationWhenInUse"
  pod 'Permission-LocationAccuracy', :path => "#{permissions_path}/LocationAccuracy"
  pod 'Permission-LocationAlways', :path => "#{permissions_path}/LocationAlways"

  # Enables Flipper.
  #
  # Note that if you have use_frameworks! enabled, Flipper will not work and
  # you should disable the next line.
  # use_flipper!()

  post_install do |installer|
    react_native_post_install(installer)
    __apply_Xcode_12_5_M1_post_install_workaround(installer)
    installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['BUILD_LIBRARY_FOR_DISTRIBUTION'] = 'YES'
        config.build_settings["ONLY_ACTIVE_ARCH"] = "YES"
        config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64"
        config.build_settings['OTHER_CPLUSPLUSFLAGS'] = ''
      end

      if (target.name&.eql?('FBReactNativeSpec'))
        target.build_phases.each do |build_phase|
          if (build_phase.respond_to?(:name) && build_phase.name.eql?('[CP-User] Generate Specs'))
            target.build_phases.move(build_phase, 0)
          end
        end
      end
    end
  end
end

Here the pods from Xcode

Thank you!
Library version
3.6.1
Environment info
"react-native": "0.66.4"

Steps to reproduce
.
Reproducible sample code
`npx react-native run-ios --simulator="iPhone 13"`

Yes... unfortunately
I'm also using "react-native-clean-project": "^4.0.1"

I see you are using use_frameworks! :linkage => :static
Don't forget to follow the workaround, as mentioned in the red screen error you're probably seeing.
https://github.com/zoontek/react-native-permissions#workaround-for-use_frameworks-issues

Yes, I've tried to added but probably in the wrong way and I don't understand exactly where.
Could you give me please an hint at the line of my Podfile where you would insert it?

This should do the trick:
require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '11.0'

use_frameworks! :linkage => :static

# Convert all permission pods into static libraries
pre_install do |installer|
  Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_static_framework_transitive_dependencies) {}

  installer.pod_targets.each do |pod|
    if pod.name.eql?('RNPermissions') || pod.name.start_with?('Permission-')
      def pod.build_type;
        Pod::BuildType.static_library # >= CocoaPods 1.9
      end
    end
  end
end

target 'eehe' do
  config = use_native_modules!

  use_react_native!(
    :path => config[:reactNativePath],
    # to enable hermes on iOS, change `false` to `true` and then install pods
    :hermes_enabled => false
  )

  target 'eeheTests' do
    inherit! :complete
    # Pods for testing
  end

  pod 'AmazonFreeRTOS'
  pod 'AWSAuthUI'
  pod 'AWSMobileClient','~> 2.17.0'
  pod 'AWSUserPoolsSignIn'
  pod 'react-native-aws-freertos', :path => '../node_modules/react-native-aws-freertos'

  # react-native-permissions
  permissions_path = '../node_modules/react-native-permissions/ios'
  pod 'Permission-BluetoothPeripheral', :path => "#{permissions_path}/BluetoothPeripheral"
  pod 'Permission-Notifications', :path => "#{permissions_path}/Notifications"
  pod 'Permission-LocationWhenInUse', :path => "#{permissions_path}/LocationWhenInUse"
  pod 'Permission-LocationAccuracy', :path => "#{permissions_path}/LocationAccuracy"
  pod 'Permission-LocationAlways', :path => "#{permissions_path}/LocationAlways"

  # Enables Flipper.
  #
  # Note that if you have use_frameworks! enabled, Flipper will not work and
  # you should disable the next line.
  # use_flipper!()

  post_install do |installer|
    react_native_post_install(installer)
    __apply_Xcode_12_5_M1_post_install_workaround(installer)
    installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['BUILD_LIBRARY_FOR_DISTRIBUTION'] = 'YES'
        config.build_settings["ONLY_ACTIVE_ARCH"] = "YES"
        config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64"
        config.build_settings['OTHER_CPLUSPLUSFLAGS'] = ''
      end

      if (target.name&.eql?('FBReactNativeSpec'))
        target.build_phases.each do |build_phase|
          if (build_phase.respond_to?(:name) && build_phase.name.eql?('[CP-User] Generate Specs'))
            target.build_phases.move(build_phase, 0)
          end
        end
      end
    end
  end
end

Clean your project using react-native-clean-project, then perform a pod install and rebuild your app.

As for me this help:
1. ⚠ If you see a No permission handler detected error: Make sure that you have at least one permission handler set up. In some cases the Xcode cache needs to be cleared (Xcode -> Product -> Clean Build Folder)


Bugfix miDebuggerPath problem with fallback behavior

This change addresses item #375 and #242
Same as  #420.
Additional it has a test debugger command mechanism and a fallback to gdb in search path.
Fallback behavior
[proc] Executing command: /usr/bin/lldb-4.0 --version
[proc] Executing command: /usr/bin/gdb-4.0 --version
[proc] Executing command: gdb --version

Removed debugger integration into kit file.

Can #420 be closed?

The testing fails on windows, it needs the extension cpptools for the cppvdbg debug launch.
Do we want this kind of extension test?

Hmm... what about separating the generation of the DebugConfiguration and testing the generation of that config without launching a debugger?


Feature/dse integration

Changed

Updated DSE to be compatible with Objective-C
Replaced old database with DSE
Updated initialization process to include DSE initialization

Note:
This is a version to test DSE in the current sense library. Refactoring is not included.
The following changes are planned in the refactoring phase:

Refactor the initialization process. (Get rid of ugly callback structures in initialization using PromiseKit or so)
Remove CSSensePlatform class, since it is just a wrapper around CSSensorStore which is a signleton.
Remove addDataPoints related methods from CSSensorStore.
Add EventBus
Replace Spatial provider with Scheduler and let sensors be responsible for broadcast and store data using Event Bus

@kanekin why there's commit from @yulrizka  here?


Augmenter.draw_grid works with a list of grayscale images

Fixed case when Augmenters.meta.Augmenter.draw_grid was called with a list of grascale images and didn't add dimension.

Thanks! Merged it.


refactor code to separate caching data vs return data

#84

Current coverage is 61.53%

Merging #109 into master will increase coverage by +21.26% as of b4de42b

@@            master    #109   diff @@
======================================
  Files           24      25     +1
  Stmts          653     650     -3
  Branches         0       0
  Methods          0       0
======================================
+ Hit            263     400   +137
  Partial          0       0
+ Missed         390     250   -140

Review entire Coverage Diff as of b4de42b

Powered by Codecov. Updated on successful CI builds.

change caching to saving json blob data and append the header contents after fetching the json blob

you didn't update error_metadata.py to use data_map instead.


Story 182553: [HCL] As a new front-end focused squad, I want to learn the OneCommerce front-end development standards/processes so I can deliver production quality UIs

As-is: The new squad from HCL does not know what the frontend development standards and processes are
To-be: The new squad from HCL does know what the frontend development standards and processes are in order to create production quality UIs and automation for pipeline integration

Epic is completed.


fix(swap): prevent extra rerenders after fetching swap quote

Description
I noticed a weird flicker on the swap screen.
https://github.com/valora-inc/wallet/assets/20150449/c1334b07-8733-46b1-a56b-5164ccaed7df
It is because the refreshQuote is invoked and the loading state is activated after the swap amount changes and after the quote is fetched, but it is only part way through the refreshQuote function that we decide that we don't need to fetch the quote for the latter scenario.
The solution in this PR is to prevent calling refreshQuote when we're not supposed to, rather than try to catch this from inside the refreshQuote function.
Test plan
No more flicker.
https://github.com/valora-inc/wallet/assets/20150449/c573a7a7-a841-43ed-b961-b107e9c73e55
Related issues
n/a
Backwards compatibility
Y

Thanks! I wonder if there is also another problem where the onSuccess of refreshQuote might get called when not relevant anymore 🤔 Since they aren't being cancelled.

I guess react-async-hook likely "cancels" (as in stop subscribing to) the current promise in flight whenever we call .execute()
I'll double check to be sure.

Thanks! I wonder if there is also another problem where the onSuccess of refreshQuote might get called when not relevant anymore 🤔 Since they aren't being cancelled.

I guess react-async-hook likely "cancels" (as in stops subscribing to) the current promise in flight whenever we call .execute() I'll double check to be sure.

TY for raising this! i did think about this too during the implementation, i modified the swap amount where the quote fetch was kicked off, and during the refresh i changed the swap amount again. the result was the "to" amount remains in the dark grey loading state, but it does change twice. so it seems like the first request is not cancelled, but that also seems like an acceptable experience for now (we could also improve the loading state of the "to" amount rather than cancelling the first request)


Upgrading to<EMAIL_ADDRESS>

Warnings

:warning:

Please include a Pivotal story at the beginning of the PR title (see below).

:warning:

Please include a description of your PR changes.

Example of PR titles that include pivotal stories:

single story: [#123456] my PR title
multiple stories: [#123456,#123457,#123458] my PR title

New dependencies added: @azure/functions.
@azure/functions
Author: Microsoft
Description: Azure Functions types for Typescript
Homepage: http://npmjs.com/package/@azure/functions

Created5 months agoLast Updatedabout 2 months agoLicenseMITMaintainers5Releases5Keywordsazure-functions and typescript

README
Type definitions for Azure Functions
This package contains type definitions for using TypeScript with Azure Functions.
These typings are for common objects that will be passed to your Azure Functions function code. Azure Functions supports TypeScript development, but does not support directly running TypeScript code without transpilation.
Read more on configuring entry points in your Azure Functions function app.
Install
Because this package only contains TypeScript type definitions, it should be saved under devDependencies.
npm install @azure/functions --save-dev
Usage
import { AzureFunction, Context, HttpRequest } from "@azure/functions";

const index: AzureFunction = async function (context: Context, req: HttpRequest) {
    context.log('JavaScript HTTP trigger function processed a request.');
    if (req.query.name || (req.body && req.body.name)) {
        context.res = {
            status: "200",
            body: "Hello " + (req.query.name || req.body.name)
        };
    } else {
        context.res = {
            status: 400,
            body: "Please pass a name on the query string or in the request body"
        };
    }
}

export { index };

Versions
Versioning of @azure/functions is tied to the version of the Azure Functions Node.js Worker the types were generated from. You can find the Azure Functions Node.js Worker version of a given Function Runtime Version here. It is recommended that you take the latest version available.
Getting Started with Azure Functions
If you are getting started with Azure Functions, you can follow this tutorial to create and deploy your first JavaScript function. We recommend that you use Visual Studio Code and the Azure Functions extension.
The Azure Functions developer guide and the JavaScript-specific developer guide are good resources to gain an understanding of more Azure Functions concepts.

  Generated by :no_entry_sign: dangerJS


Other bug is that in the Toplevel module, the port of instantiated hardware is different:

          Other bug is that in the `Toplevel` module, the port of instantiated hardware is different:

The ports' name of the my hardware module is:
  input logic [31:0] ext_mem0_read_data,
  input logic ext_mem0_done,
  input logic [31:0] ext_mem1_read_data,
  input logic ext_mem1_done,
  input logic [31:0] ext_mem2_read_data,
  input logic ext_mem2_done,
  input logic clk,
  input logic reset,
  input logic go,
  output logic [31:0] ext_mem0_write_data,
  output logic [3:0] ext_mem0_addr0,
  output logic ext_mem0_write_en,
  output logic [31:0] ext_mem1_write_data,
  output logic [3:0] ext_mem1_addr0,
  output logic ext_mem1_write_en,
  output logic [31:0] ext_mem2_write_data,
  output logic [3:0] ext_mem2_addr0,
  output logic ext_mem2_write_en,
  output logic done

In the top level, the instantiated ports' name is:
        .clk(ap_clk),
        .done(kernel_done),
        .ext_mem0__addr0(ext_mem0__addr0),
        .ext_mem0__clk(),
        .ext_mem0__done(ext_mem0__done),
        .ext_mem0__read_data(ext_mem0__read_data),
        .ext_mem0__write_data(ext_mem0__write_data),
        .ext_mem0__write_en(ext_mem0__write_en),
        .ext_mem1__addr0(ext_mem1__addr0),
        .ext_mem1__clk(),
        .ext_mem1__done(ext_mem1__done),
        .ext_mem1__read_data(ext_mem1__read_data),
        .ext_mem1__write_data(ext_mem1__write_data),
        .ext_mem1__write_en(ext_mem1__write_en),
        .ext_mem2__addr0(ext_mem2__addr0),
        .ext_mem2__clk(),
        .ext_mem2__done(ext_mem2__done),
        .ext_mem2__read_data(ext_mem2__read_data),
        .ext_mem2__write_data(ext_mem2__write_data),
        .ext_mem2__write_en(ext_mem2__write_en),
        .go(kernel_start),
        .reset(reset || memories_sent)

There are two underscores __. So the Toplevel  cannot be synthesized as well.
Originally posted by @zzy666666zzy in https://github.com/cucapra/calyx/issues/1470#issuecomment-1567362877

Not clear if this is necessarily fixable since the discover-external pass finds the prefix by matching with memories. The current fix is mentioned here: https://github.com/cucapra/calyx/pull/1534

@zzy666666zzy can you confirm whether the solution works for you or not? Not sure if there is anything actionable for this issue currently

@zzy666666zzy did the above fix for the ap_clk work for you?

Hi @rachitnigam ,
Sorry for replying you late.
gemm4x4_external.txt
I manually change the @clk clk: 1 with @clk ap_clk: 1,at line 74 in the attached file. And use the command
$PATH_calyx/calyx -l $Calyx_PATH -b xilinx --synthesis --disable-verify --nested $CURRENT_PATH/gemm4x4_external.mlir -o $CURRENT_PATH/gemm4x4_wrapper.v

Unfortunately ext_mem*_clk signals are still not connected with ap_clk at the output gemm4x4_wrapper.v file.

Okay! That seems like a bug. I'll investigate!

@zzy666666zzy okay, looks like this is a problem coming from #1056. If you look at the output of running the -p externalize pass which is responsible for externalizing the memories in Calyx, we see that the _clk and _reset ports of the memory are added to the signature:
calyx -p externalize <file> -m file

Generates the output:
import "primitives/binary_operators.futil";
import "primitives/core.futil";
component main<"toplevel"=1>(...) -> (
  ..., ext_mem0_clk: 1, ext_mem0_reset: 1,
  ..., ext_mem1_clk: 1, ext_mem1_reset: 1,
  ... ext_mem2_clk: 1, ext_mem2_reset: 1) { ... }

This is problematic because we probably don't want the component to provide the clk and reset signals to the memory. Instead, it should be the test harness that connects the memories with the component that should be in charge of handling these signals.
I'm going to change the compiler to not add these ports to the component when a cell is externalized and change the AXI generator to not add it either. Once done, I'll ping the thread so you can try out the solution.
(Also paging @sampsyo to get a sanity check on this)

(Sorry, the correct issue in question is #1034)

@zzy666666zz the changes have been merged. Can you try updating the repository and see if that works for you?

Hi @rachitnigam ,
Thanks for your time investigate this issue.
I have rebuilt the latest repo and successfully remove the unused clock ports.
    main kernel_inst (
        .clk(ap_clk),
        .done(kernel_done),
        .ext_mem0_addr0(ext_mem0_addr0),
        .ext_mem0_done(ext_mem0_done),
        .ext_mem0_read_data(ext_mem0_read_data),
        .ext_mem0_write_data(ext_mem0_write_data),
        .ext_mem0_write_en(ext_mem0_write_en),
        .ext_mem1_addr0(ext_mem1_addr0),
        .ext_mem1_done(ext_mem1_done),
        .ext_mem1_read_data(ext_mem1_read_data),
        .ext_mem1_write_data(ext_mem1_write_data),
        .ext_mem1_write_en(ext_mem1_write_en),
        .ext_mem2_addr0(ext_mem2_addr0),
        .ext_mem2_done(ext_mem2_done),
        .ext_mem2_read_data(ext_mem2_read_data),
        .ext_mem2_write_data(ext_mem2_write_data),
        .ext_mem2_write_en(ext_mem2_write_en),
        .go(kernel_start),
        .reset(reset || memories_sent)
    );

Thanks.

Perfect! I'm going to close this and work on #1576 next!


[Dubbo] refactoring to remove feature envy

I found a refactoring opportunity on Dubbo project.
I believe the online method of the Online class can be improved. This method calls 8 other methods from other classes.
Besides that, the online method calls many times ProviderModel.RegisterStatedURL inner class. In my view, a part of the online method should be extracted to a new method, and this new method should be moved to RegisterStatedURL. What do you think?
The same logic is applied to the offline method.
The advantages of theses changes are:
Remove a feature envy (this method has 4 calls to the ProviderModel.RegisterStatedURL).
Regards

Sorry, but the change cannot be applied.
ProviderModel is a basic class in the common module, it cannot depend on Registry and RegistryFactory.

We may need to consider move RegisterStatedURL out of the common module to somewhere else.


Double Click to Edit: Add settings button to toggle setting

Hello 👋 , I love the app. Thanks for making it awesome and open source.
===
Update Details
When I go to highlight text in the chat window I double click on words a lot and it brings me to the edit window instead of selecting the word.
This update adds a toggle setting to turn that feature on an off. (Default setting is doubleClickToEdit: true)

Thanks @DeFiFoFum! Merged.


Composer overriders base/default/template

"app/design/frontend/base/default/template/",
are being overwritten by the composer install.
FIX:
"magento-map-overwrite": {       "sveaekonomi/magento1-checkout": [         ["app/code/community/Svea/Checkout", "app/code/community/Svea/Checkout"],         ["app/design/frontend/base/default/layout/sveacheckout.xml","app/design/frontend/base/default/layout/sveacheckout.xml"],         ["app/design/adminhtml/default/default/layout/sveacheckout.xml","app/design/adminhtml/default/default/layout/sveacheckout.xml"],         ["app/design/frontend/base/default/template/agreements.phtml","app/design/frontend/base/default/template/agreements.phtml"],         ["app/design/frontend/base/default/template/sveacheckout.phtml","app/design/frontend/base/default/template/sveacheckout.phtml"],         ["app/design/frontend/base/default/template/sveacheckout_snippet_renderer.phtml","app/design/frontend/base/default/template/sveacheckout_snippet_renderer.phtml"],         ["app/design/frontend/base/default/template/sveacheckout","app/design/frontend/base/default/template/sveacheckout"],         ["app/design/frontend/base/default/template/payment/form/sveacheckout.phtml","app/design/frontend/base/default/template/payment/form/sveacheckout.phtml"],         ["app/design/adminhtml/default/default/template/sveacheckout/","app/design/adminhtml/default/default/template/sveacheckout/"],         ["app/etc/modules/Svea_Checkout.xml","app/etc/modules/Svea_Checkout.xml"],         ["js/Svea/Checkout/","js/Svea/Checkout/"],         ["skin/frontend/base/default/css/sveacheckout/","skin/frontend/base/default/css/sveacheckout/"],         ["skin/adminhtml/base/default/css/sveacheckout/","skin/adminhtml/base/default/css/sveacheckout/"]       ]     },

Thanks for your input, can you please make a pull-request from this?
It makes it easier to discuss the actual changes (or even see what changes are requested)
Another tip would be to make the json blob encapsulated in three backtics.
like this:
"magento-map-overwrite":
        {
     "sveaekonomi/magento1-checkout": [
            [
                "app/code/community/Svea/Checkout",
                "app/code/community/Svea/Checkout"
            ],
            [
                "app/design/frontend/base/default/layout/sveacheckout.xml",
                "app/design/frontend/base/default/layout/sveacheckout.xml"
            ],
            [
                "app/design/adminhtml/default/default/layout/sveacheckout.xml",
                "app/design/adminhtml/default/default/layout/sveacheckout.xml"
            ],
            [
                "app/design/frontend/base/default/template/",
                "app/design/frontend/base/default/template/"
            ],
            [
                "app/design/adminhtml/default/default/template/sveacheckout/",
                "app/design/adminhtml/default/default/template/sveacheckout/"
            ],
            [
                "app/etc/modules/Svea_Checkout.xml",
                "app/etc/modules/Svea_Checkout.xml"
            ],
            [
                "js/Svea/Checkout/",
                "js/Svea/Checkout/"
            ],
            [
                "skin/frontend/base/default/css/sveacheckout/",
                "skin/frontend/base/default/css/sveacheckout/"
            ],
            [
                "skin/adminhtml/base/default/css/sveacheckout/",
                "skin/adminhtml/base/default/css/sveacheckout/"
            ]
        ]
    }
}

The problem is that when magento-deploy-strategy: symlink is used, the current greedy mapping replaces app/design/frontend/base/default/template/ with a symlink to vendor/..
Thus, everything in app/design/frontend/base/default/template/ of the magento installation is gone.
When one could argue that magento-deploy-strategy: symlink is not "supported by the module", another one could argue that it's not magento praxis to put template files directly into app/design/frontend/base/default/template/ (just incase that discussion comes to show)


Fix alert count/interval, changes to eventrule types

Fix alertNotifyCount/alertNotifyInterval using Int64type.
Only allow creating sql alert rules. Remove ScriptConfig.
Make alert_tags a readonly since theyre managed on eventrules

lgtm


Provide truly immutable types for immutable-js?

method syntax strikes again:
import { Map } from "immutable";
const foo = Map([["key", "value"]]);
foo.delete = () => foo; // This compiles

With mutable types like the built in Array, you must not implement structural sharing (else you get bugs) and with immutable types on the other hand you must implement structural sharing (else you get terrible performance because you have to make whole copies of structures all the time).
The big (mostly unspoken) downside of ReadonlyArray and similar type-level only approaches is you get all the pain of the underlying mutable type's poor performance, but no access to the mutative methods that make up for it.
The best answer is to use a purpose-built immutable type that implementation that structural sharing, which requires more than just readonly type "views" that constrain what you can do to the underlying mutable runtime types.
That's the benefit of, for example, immutable-js.
The problem here is that the immutable-js types fall prey to the annoyance that TypeScript's method syntax implies mutability, which is detected by https://github.com/eslint-functional/eslint-plugin-functional via https://github.com/RebeccaStevens/is-immutable-type.
That's why the immutable-js types appear in the ReadonlyDeep column and not in the Immutable (or even the "Even Better 🚀") column in our readme.
If we were fix this, we could move it into the "Even Better 🚀" column and recommend without reservation.

Another small issue that might help make immutable-js more recommendable: https://github.com/RebeccaStevens/is-immutable-type/issues/47

https://youtu.be/uiJycy6dFSQ?t=1578 🙃

Close enough: https://github.com/agiledigital/readonly-types/blob/master/README.md#purpose-built-immutable-data-structures


Well-Known Text

To make adding geometries to a database easier, adding toWKT method to Position and Polygon classes.
If you're not already familiar with WKT, Wikipedia has a good article on it.
WKT is not database brand specific, so geokit would not be playing favorites to include these methods.
Merged latest from your master branch the right way this time, so you see only the relevant changes here.

Thanks for your effort, but unfortunately have to reject this patch since i feel this is out of the scope of this library.
You might be interested in my packages from https://github.com/geo-io which provide generators and parsers for WKT and other formats.


FAQ - update 'battery_optimization_background' entry

Changelog:

Rephrase to "is not turned off [...] immediately"
"switch on" -> "enable"
refer to #no_risk_update

@MikeMcC399 Ah yes, thank you for reminding me. This issue again... I don't think there is a quick solution for this PR right now because I do not want to link to https://www.coronawarn.app/en/faq/#no_risk_update before it has been reworked. Maybe we'll split the instructions from the https://www.coronawarn.app/en/faq/#no_risk_update entry into a new FAQ entry and then link it.

@larswmh
My comment (https://github.com/corona-warn-app/cwa-website/pull/1085#pullrequestreview-634734358) from one year ago is still relevant.
The Android app no longer changes the status itself. It puts the user into device settings where they make the change themselves.
Maybe it would be better just to remove this article and redirect it to #no_risk_update?

@MikeMcC399

Maybe it would be better just to remove this article and redirect it to #no_risk_update?

Changes have been applied in https://github.com/corona-warn-app/cwa-website/pull/1085/commits/1ed9d007b8d0bab33971f09ba267c9bad236f7f9, thanks for the suggestion. Waiting for current build problems to be resolved and then merging this into master.


[Question] Custom CSS, calling local files

How do I call local files in a custom made css?
url ("file:///Users/username/Desktop/background.png") is not working. Not sure what that means.

What OS are you using? Windows? if so, you might need to do
url ("file://C:/Users/username/Desktop/background.png")

That doesn't work either, nor does file:///~Users/username/Desktop/background.png or file://

Well for Starters. 1 Do not load local images. 2 Upload them to some server like puu.sh r something.
@TheLastMelody

@AraHaan Yeah, that's what I'm doing. It's just a pain to have to upload pictures just because I want to change background. I know I'm not the only one with this issue. But, meh, if that's how it is I'll just have to live with it.
Just a simple question thou, why doesn't it work?

Idek

I made a plugin for it, because only base64 and https data is allowed.

What plugin is it?

https://github.com/iVEnoVaLue/BetterPlugins/tree/master/plugins
It's a little bit tricky to use, cause i rely on betterdiscord (this could easily be implemented in BD btw)
The plugin seeks for links starting with local:// reads image's contents and translates it into base64, then replaces the link with base64 data. You have to have in %appdata%/BetterDiscord/ a folder 'images' (it gets created after the plugin runs once)

Damn, it was the whole moving images thing I really wanted to avoid, but oh well. Thanks pal ^_^
I too rely on BD nowadays.

You specify local://imagename.ext, it points directly to that folder

I don't follow. If I use local://imagename.ext will it look in the images folder, or the root folder?

In the images folder.
local://   ->   %appdata%/BetterDiscord/images

That was what I was asking. But thanks pal. How do I close this?

Nevermind, found the button. derpface


Upgrade k8s.io/api dependency for compatibility with Tekton Chains/sigstore

We vendor in knative in Tekton Chains, and have pinned the k8.io/api dependency to v0.20.7 for compatibility with knative.
I'm running into issues integrating Chains with cosign because cosign requires the same dependency @  v0.22.x
I was wondering if it would be possible to upgrade this dependency in knative so that everything works -- the main issue seems to be that v1alpha1 and v2alpha1 aren't supported in the new version.
knative.dev/pkg/client/injection/kube/client imports
	k8s.io/api/batch/v2alpha1: module k8s.io/api@latest found (v0.22.1), but does not contain package k8s.io/api/batch/v2alpha1
knative.dev/pkg/client/injection/kube/client imports
	k8s.io/api/discovery/v1alpha1: module k8s.io/api@latest found (v0.22.1), but does not contain package k8s.io/api/discovery/v1alpha1

I'm happy to open a PR if someone could lmk what the best way to do this upgrade might be, cc @mattmoor

@dprotaso When are we due to bump to v0.21?
As for why we lag, it generally has to do with the window of supported versions for a given client-go, so we generally lag 1-2 versions.  I'm sort of surprised cosign would need to track HEAD so closely (or why it vendors K8s at all 🤔 ).  Happy to help game out options here.

I'm sort of surprised cosign would need to track HEAD so closely (or why it vendors K8s at all 🤔 ).

I think cosign has a feature where it'll read your k8s manifests and try to verify the images in them have been signed! It probably doesn't require tracking HEAD though, we could consider downgrading the version in cosign instead. @dlorenc wdyt?

It might make sense to make some of that stuff use Knative's pod-speccable stuff anyways, so it might be worth making it consistent regardless 🤔

We're due for a bump this release - I'll probably get to it next week

LMK if you need any help.