Look at improving 'expandable things'

What
Look at expandable components to solve common problems between them and find a more unified approach.
Why

The details component has seen several issues with users thinking they link to other pages: https://github.com/alphagov/govuk-design-system-backlog/issues/44
The placement of the +/− icons and 'open all' controls on accordions can be missed by users with screen magnifiers: https://github.com/alphagov/govuk-design-system-backlog/issues/1#issuecomment-440652162
We like the look of how NHS have approached their 'expander' component: https://service-manual.nhs.uk/design-system/components/expander
There seem to be big overlaps between details and accordions which are worth exploring

Who needs to know about this
Designers, developers
Further detail
Components with expandable things include:

Accordion
Details

Should also consider:

Conditionally revealing checkboxes and radios
Step-by-step pages

Done when
TBC

Related: Feedback from testing highlighting that the <details> element is styled to look like a link, but is (natively) treated as a button by assistive technology.
https://github.com/alphagov/govuk-design-system-backlog/issues/44#issuecomment-628082040
https://github.com/alphagov/govuk-design-system-backlog/issues/44#issuecomment-633949857

Just noting some additional instances of this "open-and-close" pattern on GOV.UK that is all somewhat linked,
including:

Mobile Menus (GOV.UK)
Search filtering options (GOV.UK/search)


fix: default service name

:tada: This PR is included in version 0.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated lists

Updated hbbtv->smarttv list, since its name changed by maintainer.
Removed RansomWare Tracker, since it's no longer maintained nor accesible.
Hopefully this is correct this time.

Ping @mimugmail

No need to ping, there is still one thing pending for @marcushoff  ;)

No need to ping, there is still one thing pending for @marcushoff ;)

Sorry, I must have missed that, what's pending on me?


Conversation search support for Message APIs

Limitation on Conversational search support for Messages API with Claude 3.x family
I am building a public workshop for a public event and the purpose of the workshop is to build OpenSearch conversational search with Bedrock Claude 3.5 Sonnet, but this is not possible with the current Conversational search LLM connector.
The current Conversational search in OpenSearch supports Text Completions API for Anthropic Claude, however, Claude 3.x family only works with Messages API.
Anthropic Claude models in Amazon Bedrock from Claude v1 up to Claude v3.5, all support Messages API so we should update the plugin to support Messages API instead of Text Completions API (supported only by Claude v1 and v2).
Customers are interested in building RAG within OpenSearch to create conversational applications and we should make sure that the conversational search supports the state of the art APIs with foundational models.
Tutorial to be updated:

https://github.com/opensearch-project/ml-commons/blob/main/docs/tutorials/conversational_search/conversational_search_with_Cohere_Command.md

https://github.com/opensearch-project/ml-commons/blob/main/docs/tutorials/agent_framework/RAG_with_conversational_flow_agent.md

Anthropic Claude Messages API: https://docs.aws.amazon.com/bedrock/latest/userguide/model-parameters-anthropic-claude-messages.html
Thanks

This is a workshop required in the 2024 re:invent too.

https://docs.anthropic.com/en/api/migrating-from-text-completions-to-messages
So, basically, they are moving to an array of messages like OpenAI did when they updated their chat completion API.

@b4sjoo if this is going to be connected through bedrock, we need to think about adding the model interface as well.

I'm going to add support for Bedrock's "converse" endpoint.  This will allow connector to talk to Claude 3.5 sonnet via Bedrock.
Do we need to support direct connections to Anthropic?  I guess there will be users who use Anthropic, but not AWS?

Do we need to support direct connections to Anthropic? I guess there will be users who use Anthropic, but not AWS?

I think so, we should also support customers who use Anthropic directly

I'm just documenting some of the issues and limitations of Bedrock Converse API.
When I use the document with Claude 3.5 sonnet, I get this error:

This model doesn't support documents.

This is not clear from Bedrock's documentation.  But for both OpenAI 4o and Anthropic 3.5-sonnet, I only see examples for text and image.
I am, however, able to use document if I use Claude 3 sonnet.


k0s: run Argo CD server with --insecure flag to support ingress

As described in the helm chart, the server needs an --insecure flag to allow TLS termination at the ingress.  Without that setting, Ingress into port 80 or port 443 of the server results in an infinite HTTP 307 redirect loop.
Similar to https://github.com/argoproj/argo-cd/issues/3169 , but configuring the URL as https://argocd.k8s was insufficient.
It's possible the insecure flag would not be needed if the server.certificate.domain value is set.
Also updated to the latest release of the Helm chart.  That was just opportunistic maintenance, not a requirement for the fix.

bors r+


Please release a new version for Rails 7

7.beta is still not released as stable, please release a new version, thanks

done


Some recipe are overriding vanilla recipes

Some of the recipes to create tool part overrides vanilla recipes. Especially the pickaxe head recipe. I need to look into ways of creating tool parts that are guaranteed to not conflict with mods and vanilla Minecraft. I also need the recipes to be very easy to craft.

Idea
To craft a tool part, you need to first create a schematic for it. This can be done by combining a tool with a piece of paper. The paper will be consumed to craft a schematic. All tool parts and tools can have schematics created by combining them with a piece of paper. This will let you create schematic from tool parts if you find them in the world.
This will make it a lot harder to initially get into tool crafting but I don't really know any other way to make tool part recipes not override vanilla.

And yes, this is very similiar to how Tinker's does it, as well as all other tool creation mods.

TODO:

[ ] Create new Item: Tool patterns
[ ] Create recipe for creating tool patterns
[ ] Add patterns to tool part recipes
[ ] Create textures for different patterns


Transcription Search

Hey,
Thank you so much for fixing this extension! I use it all the time and it's been immensly frustrating with it not working, this is awesome :D
Was wondering if there was any way to get transcript search working as I also like using the extension for this too?
Thanks!

you may find this useful, I wanted to generate subtitles for YouTube videos that didn't have them and found 'transcribe-anything'.  It quickly auto-generates subtitles/transcriptions in multiple formats:
pip install transcribe-anything
# insanely fast using the insanely-fast-whisper backend.
transcribe-anything "https://www.youtube.com/watch?v=yourVideo" --device insane

you may find this useful, I wanted to generate subtitles for YouTube videos that didn't have them and found 'transcribe-anything'. It quickly auto-generates subtitles/transcriptions in multiple formats:
pip install transcribe-anything
# insanely fast using the insanely-fast-whisper backend.
transcribe-anything "https://www.youtube.com/watch?v=yourVideo" --device insane

Thanks for this! But even on videos where YT has auto-generated captions, it still doesn't work for searching them. I appreciate it nonetheless :)

if you had a generated subtitle .srt file, couldn't you search it using a text application?

if you had a generated subtitle .srt file, couldn't you search it using a text application?

No I know. It's more just for a convenience thing. Having the extension right there and if I'm looking for a mention of something in particular takes like 5 seconds because it's built in

Thank you for fixing @pc035860 ! <3


Update AppArmor docs for GA

Update AppArmor Docs for GA release to include usage of fields in favor of annotations.
https://github.com/kubernetes/enhancements/issues/24

/milestone v1.26
/sig node
/assign @krol3

/milestone 1.26

/sig security

Since AppArmor does not go GA in 1.26 I think we can remove the milestone.
/milestone clear

cc @krol3

/milestone clear

Since this is not targeting GA for 1.26, and there is conflict to be resolved anyway, shall we close it for now?


Missing Details for AdditionalFields

The details of the AdditionalFields values is missing. Without this information you cannot parse it using KQL:

AdditionalFields
string
Additional information about the event in JSON array format

[Enter feedback here]

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

ID: 3896293b-ab88-1f86-6829-082d6d2e5f73
Version Independent ID: ca04da4a-4d14-5dd4-20ee-b7bb22b64354
Content: DeviceInfo table in the advanced hunting schema
Content Source: microsoft-365/security/defender/advanced-hunting-deviceinfo-table.md
Service: microsoft-365-security
Sub-service: m365d
GitHub Login: @schmurky
Microsoft Alias: maccruz

@schmurky Please help us in resolving this issue. Thanks

@schmurky Thank you very much for the contribution and sharing this explanation. @PhilippEisenmann Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Error in console when inspecting a model (doesn't impact the inspection)

This error occurs in multiple places but doesn't impact the utilisation of giskard. It's caused by the migration to composition API by Vue

Strange issue, Andrey committed this: https://github.com/Giskard-AI/giskard/commit/3e2a8de6170b0f0de7c8df6881924d13b8b63d6d which should be a fix for it...


Unable to use excalidraw in browser extension [CSP lvl. 3]

I am trying to use excalidraw inside chrome extension that injects excalidraw into a webpage but I am not able to due to strict CSPs. Excalidraw depends on some remote resources from https://unpkg.com/ but it is mentioned in the documentation that it is possible to host those files by myself. And that is what I am doing in my extension. However in order to make excalidraw actually use the local version of those files instead of the remote one I need to set window.EXCALIDRAW_ASSET_PATH to chrome.runtime.getURL("") (that's where these files reside) but it is not possible as fetching of those resources happens from the top level window (the webpage) and not content script. I cannot set global variables of the webpage as extensions content sripts live in an isolated world, where only DOM is shared between content script and the webpage. I tried to set this variable by injecting inline script (since DOM is the only shared thing) but this is not allowed due to strict CSPs of chrome extensions. I really do need a way to force excalidraw to use my local resources but I don't want to modify code of the library.

I am hitting this issue too:

excalidraw.development.js:4347 Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

at ./entry.js (excalidraw.development.js:4347:1)
at __webpack_require__ (excalidraw.development.js:5625:42)
at excalidraw.development.js:5889:37
at excalidraw.development.js:5892:12
at webpackUniversalModuleDefinition (excalidraw.development.js:17:27)
at excalidraw.development.js:18:3


consider adapting Apple's implementations of Foundation and Dispatch publishers

Apple's implementations of the Foundation and Dispatch publishers and schedulers are part of the Swift open-source project. Have you considered adapting those implementations for OpenCombine?
Here are the files of interest:

Publishers+KeyValueObserving.swift
Publishers+Locking.swift
Publishers+NotificationCenter.swift
Publishers+Timer.swift
Publishers+URLSession.swift
Schedulers+Date.swift
Schedulers+OperationQueue.swift
Schedulers+RunLoop.swift
Schedulers+DispatchQueue.swift

Nice! I'll do it.

The implementations of all those entities are in the main branch. They're based on Apple's, but tweaked to work with the platforms and language versions that we support.
The only exception is Publishers+KeyValueObserving.swift. There are certain issues with it. If anyone feels they should be prioritized, please create a new issue.


Add onramp.money

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

[ ]


Failed to connect with the camera

i have added this via HACS, restarted Pi, go to add the Integration and when i'm putting in any of the camera IP, Port 8000, admin and password, i get this -- Failed to connect with the camera
i've tried all of them, even the NVR. i am using this: https://www.amazon.com/Security-8-Channel-Surveillance-Recording-RLK8-800B4/dp/B07K734ZTW/ref=sr_1_10?dchild=1&keywords=reolink+4k&qid=1623701302&sr=8-10
i am wanting to basically get this set up to get image preview notifications (iOS) so if there's another way to get this going other than this option let me know. i am very new to HA so sorry if this is a stupid issue that's easily resolved.

Great, thanks for the help

Hi,
I get exactly the same error.
For my part, I tried to use the port 80 as described into the doc.
My Reolink nvr is RLN8-410
Regards

Same problem Reolink E1 PRO

Same problem Reolink E1 PRO

It mentions this camera isn't supported on the main page.


Build operator into the child chain

Use web3.py in child_chain.py to communicate directly with the ganache-cli Ethereum chain for deposits and submitting blocks.

By this, do you mean to change this:
https://github.com/omisego/plasma-mvp/blob/c263f78fe4836c53212c19bdf69332c809727848/plasma/child_chain/child_chain.py#L25-L34
so that child_chain.py is making the web3 interactions instead of the client?


Update README instructions Dispose logic

In the example code, the Dispose() method needs += corrected to -= thereby effectively removing the assigned delegate.

Duplicate Issue


[WS] Priority Tags for Counters (Special/Normal)

On each branch, there must be at least one special counter for pwd's and senior citizens.

counters now classified either special or normal type


Alerting via Node.js API?

Hi,
Do you have plans on adding the MarkLogic Alerting capabilities to the Node.js API anytime soon?

Yes, can you tell me more about your specific use case?

@jmakeig If I remember correctly I have emailed you the requirements - would you like me to paste them here as well?

Do we have any plans to support Alerting from Node.js?

Expanding the description of the use case would be helpful.

The information that I sent to Justin was pretty much saying that I'd like to execute matchmaking and reverse queries from a pure Node.js environment.


Update BaseYii.php

Q
A

Is bugfix?
✔/❌

New feature?
✔/❌

Breaks BC?
✔/❌

it's deprecated method


HAWQ-1476. Augment enable-ranger-plugin.sh to support kerberos.

Now ranger can lookup hawq resource in kerberized environment. So we also need to change enable-ranger-plugin.sh to support automatically fill the authentication type and hawq kerberos service name fields

LGTM. +1


correlate CVEs to other CVEs based on "ref" element in CVE XML

There are many CVEs that are identical or closely related, but which have been reported separately. It might be useful to extract this information. When looking at the CVE XML download: apart from references in the CVE <description> there is also possibly useful information in the <refs> element. Many references, like URLs, are identical, meaning that the bugs are related.
If there are related CVEs it is usually a handful, although there is one extreme outlier with a reference that occurs 1390 times in today's CVE data.
A small proof of concept script of how this data could be extracted (but only processes identical URLs, which not all references have): https://github.com/armijnhemel/compliance-scripts/blob/cve_extract_paths/cve/cve_match_cves.py

You're going to get a lot of false positives. Many vendors have large security updates and thus references that include dozens or a hundred CVEs, or reference a single large page like Oracle.
Examples:
https://www.oracle.com/security-alerts/cpujan2022.html
Also as to the "CVEs that are identical or closely related" yes there are many because some vendors break it up more, some less, but many don't, so for example:
https://nvd.nist.gov/vuln/search/results?form_type=Basic&results_type=overview&query=This+CVE+ID+is+unique+from&search_type=all&isCpeNameSearch=false
The real (more common case) is one-off duplicates, which means you're looking at a huge pile of data and manually validating it all.

@kurtseifried
Thank you ++ and I excited to see you drop by!

The real (more common case) is one-off duplicates, which means you're looking at a huge pile of data and manually validating it all.

We can eventually tackle the manual validation, as this is part of the project charter... but do you know if there would be alternative to this (even a small, partial and inefficient one)?

You're going to get a lot of false positives. Many vendors have large security updates and thus references that include dozens or a hundred CVEs, or reference a single large page like Oracle.
Examples: https://www.oracle.com/security-alerts/cpujan2022.html

Those are fairly easy to filter, even automatically.
I did a short test to see what the most common situation is when looking at just the references without looking at what kind of reference it is:

Amount
Linked CVEs

16960
1

9812
2

6468
3

4686
4

3571
5

2977
6

2480
7

2171
8

1872
9

1623
10

1558
12

1397
11

1389
1389

1291
13

1192
14

1077
16

1052
19

1039
20

960
23

926
18

My guess is that the results when cleaning up the ref references and only looking at the ones that are (likely) to be unique or useful these numbers will be different. Some thought would need to go into which advisories to look at and which ones not. The proof of concept script I wrote is really dumb, with very little intelligence.

Also as to the "CVEs that are identical or closely related" yes there are many because some vendors break it up more, some less, but many don't, so for example:
https://nvd.nist.gov/vuln/search/results?form_type=Basic&results_type=overview&query=This+CVE+ID+is+unique+from&search_type=all&isCpeNameSearch=false
The real (more common case) is one-off duplicates, which means you're looking at a huge pile of data and manually validating it all.

@kurtseifried
Thank you ++ and I excited to see you drop by!

The real (more common case) is one-off duplicates, which means you're looking at a huge pile of data and manually validating it all.

We can eventually tackle the manual validation, as this is part of the project charter... but do you know if there would be alternative to this (even a small, partial and inefficient one)?

Several strategies come to mind:

Comparing URLs as mentioned (lots of false positives, filtering by vendor will generally eliminate a lot of these)
Shingling search approach (https://en.wikipedia.org/wiki/W-shingling) e.g. break it up into chunks and look for similar bits, project/product names, in theory should occur across duplicates
Deeper downloads, e.g. ID-A and ID-B might have different source references, but chances are both of those share text or links or point at each other or something (e.g. one is a commit message, the other a link to the issue, but the issue then references the commit message) - the trick to this is enriching the data and then doing search mapping
Having easy feedback loops for people to contact, I used to email the CVE people monthly or more back in the olden days to inform them of duplicates (I had a series of contracts that involved reading pretty much ALL the CVEs, which is totally doable when it's only a few thousand a year) but let people know about these loops.


Add django 1.7 compatibility

moved migrations to south_migrations
created django 1.7 migrations
bump version

LGTM! Great job.


add logic

I think in this PR we also want to handle and update the JSKeyManager logic to better handle the localStorage and sessionStorage types


RecursionError: maximum recursion depth exceeded

run python treeqn/nstep_run.py, got the following RecursionError,
File "/home/ubuntu/projects/treeqn/treeqn/utils/pytorch_utils.py", line 67, in ortho_init
   ortho_init(tensor.data, scale=scale)
 File "/home/ubuntu/projects/treeqn/treeqn/utils/pytorch_utils.py", line 67, in ortho_init
   ortho_init(tensor.data, scale=scale)
 File "/home/ubuntu/projects/treeqn/treeqn/utils/pytorch_utils.py", line 67, in ortho_init
   ortho_init(tensor.data, scale=scale)
 File "/home/ubuntu/projects/treeqn/treeqn/utils/pytorch_utils.py", line 66, in ortho_init
   if isinstance(tensor, Variable):
RecursionError: maximum recursion depth exceeded

Then I dig more, found:
ERROR - treeqn_log.treeqn - Failed after 0:00:03!
Traceback (most recent calls WITHOUT Sacred internals):
  File "/home/ubuntu/anaconda3/lib/python3.5/contextlib.py", line 77, in __exit__
    self.gen.throw(type, value, traceback)
  File "treeqn/nstep_run.py", line 323, in main
    train()
  File "treeqn/nstep_run.py", line 198, in train
    model = create_model(env)
  File "treeqn/nstep_run.py", line 163, in create_model
    model = policy(env.observation_space, env.action_space, env.num_envs, nsteps, nstack)
  File "/home/ubuntu/projects/treeqn/treeqn/models/models.py", line 63, in __init__
    encoder = atari_encoder(ob_shape[1])
  File "/home/ubuntu/projects/treeqn/treeqn/models/encoding.py", line 8, in atari_encoder
    nn_init(nn.Conv2d(in_channels, 16, kernel_size=8, stride=4), w_scale=np.sqrt(2)),
  File "/home/ubuntu/projects/treeqn/treeqn/utils/pytorch_utils.py", line 87, in nn_init
    w_init(module.weight, w_scale)

Any suggestion?

Hi,
I'm pretty sure this will have been caused by using the latest pytorch version 0.4 whcih was just released.
Our code runs with pytorch 0.3
I'll update the dockerfile soon so it has the proper versions locked in for all packages.

cool~ close it now.


Remove token as start screen

Description
With the new transfer screen, the usage flow has changed and is not correctly working anymore (eg. back arrow on the connect to hub screen)
Therefore I would propose

Remove the token screen as the start screen
Make the transfer screen the start screen
Check the whole flow (eg. back arrow on the connect to hub screen)

Open questions:

What are we showing, if no token is connected yet?

This should be just a quick fix and focus should be on ease of implementation.
Acceptance criteria

Tasks

[ ]

@christianbrb The proposed solutions in the problem description sounds like the way to go. And the implementation can be broken down in the steps outlined above.
For the empty screen, if the user has no token connected, we could display the plus-button that prompts the user to "Connect new token".

@taleldayekh Looks good :)
We need to decide which token to preselect within the token transfer screen

The preselected token could be the last one used. If I made a transfer with TalelToken before closing the dApp then that would be the one appearing next time I open the dApp, if I then switch and make a transfer with FranziToken then that is the one preselected next time I open the dApp.
I belive this is the most logical way of deciding what token should be displayed on startup. Thoughts?

From a user perspective showing the last used token makes absolute sense, but we have to persist it somewhere (e.g localStorage)

@taleldayekh @kelsos Thanks for your input :)
Maybe we better iterate on that and define the preselected token as a separate issue. Scope for this issue should be as small as possible as it is supposed to be a fix for the current situation.

@christianbrb the simplest thing is just to change the landing component, which should be fairly easy. I like the idea of scoping the preselected token out as part of a new issue since it would require more effort and though.

The design for the updated start screen is finished. Pretty straight forward design. :)
Click here to access the design file. I'm currently making the mockups in Adobe XD but it shouldn't be any differance for you guys.
I'm happy to try and implement this myself if you have other tasks and if I have time over.


Increase AR service timeout to 15 minutes

High-level description
10 minutes may not be enough for large files. Increase the timeout to 15 minutes.
Corresponding DC/OS tickets (required)

D2IQ-ID JIRA title / short description.

Related tickets (optional)

D2IQ-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

Not the problem


Update ATAT CSP API spec for new requirements

Changes to support requirements from October TEM

Renames title to ATAT CSP API
Removes DRAFT notice
Adds narrative context to description
Adds tag access to endpoints ending in /operator-roles
Adds operation addApplications
Adds operation addEnvironments
Adds operation addFundingSource
Adds operation getCostsByClin which uses CostRequest and CostResponse and TimePeriod schema objects
Removes operation getCostsByApplication
Removes operation getCostsByEnvironment

Ticket: AT-6774

@jeffsegal-ccpo I believe this satisfies the ACs of the ticket and incorporates what we talked about recently.  This is ready for your review.


Inactive channels, reconnect fails with "general SOCKS server failure"

Hi!
I am running my raspiblitz for some weeks now (behind tor) and have 8 channels open. Overnight, two of these channels became inactive. The nodes don't appear in the list of peers anymore. When I try to reconnect to these peers, I get the error message "socks connect tcp <IP_ADDRESS>:9070->peer-IP:port: unknown error general SOCKS server failure"
I can connect to other peers without problem, tor and clearnet. I am sure the two nodes aren't offline, they are big nodes, zigzag and rompert.
What can I do to solve this problem?
Thank you in advance!

Try restarting Tor
sudo systemctl restart tor
then lnd
sudo systemctl restart tor
Alternatively restart the whole node.
Can also test removing  tor.streamisolation from the lnd.conf.


Fix get method to retrieve requested relationships

Resolve issue #6  : get method not retrieving requested relationships.
Updated the get method to include the requested relationships in the query by calling the existing resolveRelationship method.
Tested the changes and verified that the relations property is no longer empty.
Thank you!

Nice found @AcostaLm
thanks


bug: Chain not found in WAGMI

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.9.2
Current Behavior
Error: after install and try and run
import { useProvider, chain as wagmiChains } from "wagmi";
                      ^^^^^
SyntaxError: The requested module 'wagmi' does not provide an export named 'chain'```

### Expected Behavior

Should run

### Steps To Reproduce

"wagmi": "^0.9.2"
"ethers": "^5.7.1",

### Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)

_No response_

### Anything else?

_No response_

0.9.x no longer exports chain. Read here for a migration guide. 😊


Add documentation about all the upstream objects

This is a very boring task, so if anyone wants to help... :-)

[x] Add documentation for botogram.User
[ ] Add documentation for botogram.Chat
[x] Add documentation for botogram.Photo
[x] Add documentation for botogram.PhotoSize
[ ] Add documentation for botogram.Audio
[ ] Add documentation for botogram.Voice
[ ] Add documentation for botogram.Document
[ ] Add documentation for botogram.Sticker
[ ] Add documentation for botogram.Video
[ ] Add documentation for botogram.Contact
[ ] Add documentation for botogram.Location
[ ] Add documentation for botogram.UserProfilePhotos
[ ] Add documentation for botogram.Message
[ ] Add documentation for botogram.ReplyKeyboardMarkup
[ ] Add documentation for botogram.ReplyKeyboardHide
[ ] Add documentation for botogram.ForceReply
[ ] Add documentation for botogram.Update

Hi Pietro, I should be able to help with this...

Thank you very much @tsculpt :-)
The documentation about those objects should be at docs/api/telegram.rst, and they don't require digging into the code to document them. Each one (except Photo, which I already documented) is the representation of the object in the Telegram documentation. You should maybe rephrase the description of them, add which type they are and if they are optional (as I did in the other objects).
You don't have to do every object, when you do some feel free to send a pull request.

Documentation just landed on master. Thanks @tsculpt!


Upgrade php to 8.1.28 to fix CVE-2024-2756, CVE-2024-3096

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] If you are adding/removing a .spec file that has multiple-versions supported, please add @microsoft/cbl-mariner-multi-package-reviewers team as reviewer (Eg. golang has 2 versions 1.18, 1.21+)
[x] Ready to merge

Summary

Upgrade php from 8.1.22 to 8.1.28 to fix CVE-2024-2756 and CVE-2024-3096
Change Log

Upgrade php from 8.1.22 to 8.1.28 to fix CVE-2024-2756 and CVE-2024-3096
Update BuildRequires, libpq is now provided by postgresql

Does this affect the toolchain?

NO
Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2024-2756
https://nvd.nist.gov/vuln/detail/CVE-2024-3096

Test Methodology

Pipeline build id: 561390

Auto cherry-pick results:

main :white_check_mark: -> https://github.com/microsoft/azurelinux/pull/9009

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=562543&view=results


Uncaught TypeError: Action.Add is not a function

How to fix this issue?

snabbdom has direct JSX support out of the box these days. You probably don't need this fork of an old library unless you are looking at very old legacy code.


@electron-forge/webpack-plugin Add ability to disable liveReload in webpackDevServer

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem Description
I am working on a React + Redux application using the Webpack + Typescript template and want to get Hot Module Replacement working for my React components. By default, whenever I update a module, the log will say Content base changed. Reloading... and the entire app reloads which resets my Redux state.
Proposed Solution
https://github.com/electron-userland/electron-forge/blob/master/packages/plugin/webpack/src/WebpackPlugin.ts#L297
I'd like the ability to pass in additional options here. Specifically liveReload: false. After I hardcoded this option in the WebpackPlugin.ts file, I was able to have the desired functionality of an individual module updating without breaking my Redux store. However I am not sure if this breaks other important functionality as I am currently just testing in a sandbox project.
Alternatives Considered
In the past I've used react-hot-loader but that was on Webpack 4 and I'm not sure if there will be issues with Webpack 5. Another possibility that I have not yet looked into would be React Fast Refresh which has a Webpack plugin, though I have yet to try that with electron-forge.
Additional Information
In my testing, this did prevent the Redux store from resetting. However it does not prevent state within a React component from resetting. Ideally preservation of React component state would also work though this is less important for my purposes.
Even if there are better solutions for electron-forge + webpack HMR + react + redux, it could still be beneficial to allow custom options (and overrides) to be passed to the webpackDevServer.

I would suggest seeing if react-hot-loader works for you before we start adding more options to the Webpack plugin.

@malept Alright, I'll give it a try soon and report back. I was hoping to avoid adding more packages if a single config option can solve it. 😄

I am generally reluctant to add more code to the Webpack plugin if I can help it, because it's currently pretty complex (even after a lot of it got refactored for Webpack 5, which got rid of a lot of complexity), which makes it pretty difficult to maintain.

@malept Unfortunately I wasn't able to get either of the alternatives working on my end.
react-hot-loader didn't work and I found some conversations over there stating that the package was deprecated and instead react-refresh-webpack-plugin should be used. So I attempted that but it seems there is an incompatibility with webpack-dev-server somewhere as it throws the error below. I also had to do a --force install due to electron-forge using some beta webpack stuff (IIRC I saw webpack-dev-server)

try with:
new ReactRefreshWebpackPlugin({ overlay: false })

@malept we are having the same issue in our React project.
We are using react-refresh through webpack plugin.
On each update (even to css files) live reload kicks in and does not give a chance to hot module replacement or react refresh. It just reloads the whole application every time, resulting in reduced DX.
And our setup worked just fine before upgrading to the recently released beta versions.

I see this issue is marked as Needs Info, but I'm not sure what other info is needed? Please let me know if there's anything else I can provide. Thanks!

Thanks @malept!


added common issue chapter

I found out that this may be an issue if you haven't worked with python on the system yet and this package is not installed

You might should add a note that the pip command has to be executed in a Kicad command prompt (on windows at least)

Will close this due to inactivity


feat(auth): Add local fake authentication for testing.

This closes #9.
This adds an AddFakeZitadel method extension to
the authentication builder. With this local fake,
one can bypass authentication for development.
To fail a specific request, add the "x-zitadel-fake-auth"
header with the value "false" to the request.

/cc @dwirz

/cc @dwirz


Introduce Knip to clean up unused files, dependencies and exports

WIP - DO NOT MERGE
The idea is that each package in the monorepo can be looked after separately, or all together. The nice thing about Knip is that, even though I created it and it's far from perfect, it gives starting points to keep the monorepo clean & tidy. Otherwise it's "mopping with the tap open" now that we're growing and keep building new features. Automation for the win!
TODOs

[ ] The issue I'm facing is that it's not yet possible to run Knip from root with Rush, which is required. When running for a single package, it doesn't know about the other packages in the workspace.
[ ] I hope it's possible to have Rush simply pass through any CLI arguments to the underlying tool (otherwise we'd have to add each and every argument to rush.json?)
[ ] Add configuration and fine-tune Knip

Example
Here's an example of how I would intend to use it:
❯ knip --workspace packages/libs/kadena.js
Unused dependencies (4)
bignumber.js  packages/libs/kadena.js/package.json
blakejs       packages/libs/kadena.js/package.json
node-fetch    packages/libs/kadena.js/package.json
tweetnacl     packages/libs/kadena.js/package.json
Unused devDependencies (8)
@kadena-dev/eslint-plugin  packages/libs/kadena.js/package.json
@kadena-dev/heft-rig       packages/libs/kadena.js/package.json
@rushstack/eslint-config   packages/libs/kadena.js/package.json
@types/heft-jest           packages/libs/kadena.js/package.json
@types/node-fetch          packages/libs/kadena.js/package.json
serve                      packages/libs/kadena.js/package.json
ts-loader                  packages/libs/kadena.js/package.json
webpack-cli                packages/libs/kadena.js/package.json
Unlisted binaries (1)
python  packages/libs/kadena.js/package.json
Unused exports (3)
keyPair             packages/libs/kadena.js/src/api/tests/mockdata/contCommand.ts
payload             packages/libs/kadena.js/src/api/tests/mockdata/contCommand.ts
stringifiedPayload  packages/libs/kadena.js/src/api/tests/mockdata/contCommand.ts

Some dependencies may actually be used, but that's usually either a bug in Knip or implicit/magic from whatever tool we're using (we can configure stuff to be ignored).

Autoinstallers or rush plugins are the likely way to integrate this.
https://rushjs.io/pages/maintainer/autoinstallers/
https://rushjs.io/pages/maintainer/using_rush_plugins/

Giving up on this one, mostly because Rush and Knip aren't really "compatible":

Rush has no package.json in the root of the project, while Knip requires one (or a pnpm-workspaces.yaml) in the root of the project (at least to know about all workspaces).
Rush has more levels of indirection around rig stuff (https://rushstack.io/pages/heft/rig_packages/), forcing more config on Knip to ignore certain devDependencies in most packages.

Knip can still be used by running it in a package directory (either globally installed or as a package devDependency), but the idea of this PR to run it integrated within Rush standards seems not really feasible/worthwhile.
I'll look into cleaning up packages separately and open PRs along the road.


Remove apt.txt, environment.yml, and refactor Dockerfile

I'm trying to learn to use and contribute with maintenance of this project. Doing so, I'm looking at all files and trying to understand their purpose etc.

I noted that the apt.txt seemed outdated and no longer relevant as having a Dockerfile makes a binderhub ignore the apt.txt file, the same is true for the environment.yml file.
I noted that websockify was required but it wasn't listed in install_requires of setup.py, but installing it with pip didn't help - but instead one was required to install it with conda/mamba.
I hoped to put files that I believed were binderhub related into .binder/ but that wouldn't work, so I felt it was nice to embed the environment.yml file into the Dockerfile to minimize files in the root folder that increase the complexity.

@manics hmmm okay we can add the environment.yml back, but without any instructions, I couldn't say it was part of what was required or not for the actual local dev etc or if it was something needed by the dockerfile only.
In practice, the environment.yml only installs websockify besides the local python package, but besides having installed that, one need to install other things as well as i understand it, such as dbus-x11 and xfce4-session it seems. I'm not sure about this, but to me, I ended up very confused no matter what.

I added back dev-environment.yml to clarify it was not associated with the Dockerfile, but as something for development.

I still think we should keep it as environment.yml since it's not just for dev, it's a bit in between. If you want we could put this PR on hold until I've dealt with https://github.com/jupyterhub/jupyter-remote-desktop-proxy/issues/12 ?

If you want we could put this PR on hold until I've dealt with #12 ?

Sure!

@manics I just re-arrive to this repo and ended up wishing to update things that could come in conflict with this PR, can we go for a merge here without doing #12 first?

I think the Dockerfile here should eventually lead towards something we can use to test changes to this repo. New users should probably steal from https://github.com/binder-templates/binder-desktop-app-template instead, although this repo itself needs more docs.

I'm just going to merge this for now, as removing the apt.txt is definitely an improvement.


sap_netweaver_preconfigure: verify compatibility with the applicable SAP notes for Adobe Document Services

https://github.com/sap-linuxlab/community.sap_install/blob/d36cc155dc82fdb1848b1f36d6dbbedb9d06a3ab/roles/sap_netweaver_preconfigure/vars/RedHat_9.yml contains libssh2, which is not available on RHEL 9. So let's use this opportunity to verify all Adobe Document Services packages with the applicable SAP notes now.

Solved in #888.


[KYUUBI #3648][SUBTASK][FEATURE][UI][REST] Add  session detail page

Why are the changes needed?

Close #3648
Sql statistics => Session Detail
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Close, due to modify the corresponding API structure


doc(#230): Add docs for RuleOnlyTestMethods

Docs for RuleOnlyTestMethods
Closes: #230

PR-Codex overview
Focus of this PR:
Add documentation for the "Only test methods" rule.
Detailed summary:

Added a new file docs/rules/only-test-methods.md for the rule documentation.
Added a description of the rule and its purpose.
Provided examples of correct and incorrect usage of the rule.
Added instructions for enabling the rule as an experimental feature.
Included information on how to suppress the rule if needed.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


build: re-enable VM tests

The following issues have now been resolved:

https://github.com/actions/runner-images/issues/8730
https://github.com/actions/runner-images/issues/8642

Unfortunately the issue with VirtualBox still persists


Investigate integration with SFOS Forum Viewer

Inspired by: https://github.com/szopin/harbour-sfos-forum-viewer/pull/63#issuecomment-1396920756
Things like:

User API key
report submission
file upload
...

could be lifted from/done through SFV instead of a NIH implementation.

@szopin, would you be open to implementing something like a small D-Bus api, or SFOS Share Action receiver in SFV, so that Bugger could call native SFV bits? That would be a bit cleaner (well...) than duplicating code.

Whatever works for you is fine by me, sadly way out of my league I'm afraid. I'm happy to accept any PRs, just won't be able to help as have 0 experience with dbus outside of playing with it in terminal (action receiver first time I hear of)

action receiver first time I hear of

Yes I dont think that's an official term, I mean the thing that sits on the other end of a Sailfish.Share ShareAction and handles the data. I don't know whether there is documentation about it anywhere, but one can look at how the email app does it:

/usr/share/jolla-email/email.qml (DBus interface to receive shared data)
/usr/share/jolla-email/pages/ShareComposerPage.qml (handling the data)

Interesting, if it's doable from QML only then a big plus as would increase my available troubleshooting time immensely. Text data should be no problem, not sure about file uploads, for picture upload had to workaround with imgbb as they accept base64 encoded data and qml has no support for binary strings, for logs maybe some kind of online pastebin could be used and links automatically generated (don't think discourse has attachments support, looks like only images)

Yeah I have explored a couple of avenues, in that branch and a couple of local ones.
There's issue #29 but it's stalled by the fact that I can launch email either pre-filling subject (mailto:), or adding files (ShareAction) but not both.
Tried to work around the first but doing things like compression or MIME-encoding in JS is painful (for the user/CPU -- there's plenty of ready-to-use JS libraries that can do it) for non-trivial string lengths/file sizes.

something like a small D-Bus api, or SFOS Share Action receiver in SFV, so that Bugger could call native SFV bits?

A third option could be a MIME scheme handler. If SCV registered itself as a handler for either 'https://forum.sailfishos.org' or maybe a custom scheme like sfos-forum://, Bugger could call it per URL as it does now, and SFOS would present a choice of browser and SFV.
I have done something like that in one of the first SFOS hacks I did, youtube-dl-helper:

https://gitlab.com/nephros/youtube-dl/-/blob/sfos-fixes/files/youtube-dl-open-url.desktop#L11

https://gitlab.com/nephros/youtube-dl/-/blob/sfos-fixes/files/youtube-dl-open-helper.sh#L4

https://forum.sailfishos.org/t/sailfish-community-news-11th-march-sdk-mimetypes/5409

If we go with this custom scheme is probably better to avoid hijacking normal links to the forum

If we go with this custom scheme is probably better to avoid hijacking normal links to the forum

Well, it could also be a viewed as a feature...

With no UI accessible way of overriding those 'features' it kinda gets messy in sfos when every app registers itself (on my old jolla C I would get like 5-6 options and then due to some bug even Browser started showing twice... Not a good experience)
On Thursday, 19 January 2023, Peter G. wrote:

If we go with this custom scheme is probably better to avoid hijacking normal links to the forum

Well, it could also be a viewed as a feature...
--
Reply to this email directly or view it on GitHub:
https://github.com/sailfishos-chum/bugger/issues/36#issuecomment-1397457062
You are receiving this because you were mentioned.
Message ID: @.***

--
Sent from my Jolla


[Documentation] How to "set the environment properly"?

This is the error I got when pushing changes over SSH:
remote: Gitea: Rejecting changes as Gitea environment not set.
remote: If you are pushing over SSH you must push with a key managed by
remote: Gitea or set your environment appropriately.

From the docs: Config Cheat Sheet:

ONLY_ALLOW_PUSH_IF_GITEA_ENVIRONMENT_SET: true: Set to false to allow local users to push to gitea-repositories without setting up the Gitea environment. This is not recommended and if you want local users to push to gitea repositories you should set the environment appropriately.

Is it just me or are the error and documentation a little vague?  I don't understand what environment needs to be set.  Googling just brought up the doc on specific variables.

This message should only appear if you are bypassing Gitea and pushing directly to a repository.
In general this leads to irregular behavior and is not recommended.

@keith24 it means you are trying to bypass Gitea by not using the proper remote for your copy of the repository:

You normally use:
git clone {....address-from-gitea.....}

And then you can push your changes there. If you've already have your changes in a local copy and you want to use those and push to Gitea, you can:
# Remove previous "origin"
git remote remove origin
# Add Gitea as "origin"
git remote add origin {....address-from-gitea.....}

{....address-from-gitea.....} is the HTTP or SSH address for your repo, as exposed in the box shown above.

@keith24 Lately I was encountering the same problem. In my case the problem was that gitea did not find the correct .ssh/authorized_keys file. In my installation I had to move the home directory of the git user. When adding a SSH key via UI the key was added to /home/git/.ssh/authorized_keys and not /.ssh/authorized_keys.
I solved it for me using a symlink.

I had been trying to push an already-existing repo to gitea.  But I did replace the remote with the SSH URL that gitea provided.  IIRC it was actually the same as the one I'd already had.
@castrov was probably having the same issue as me: I was using the ssh keys from the server, not managing them in git.  Maybe the URL was the same, but git assumed the original ssh identity because that's what it had a key for.
I just started using HTTPS and forgot I ever had this issue (sorry about not replying).
I'm gonna close this for now because I'm not having any issue anymore.  Whether some of the information in this thread should go in the docs is up to the maintainers' discretion.

For all who are still facing this issue. Please check, if your user you are using to run gitea (e.g git or gitea) is set up without a password, try to disable empty passwords in your sshd.config file by setting PermitEmptyPasswords to No. You can do this in the Match User section to set this only for the gitea user. You should also check the configuration for your authorized_keys file and if the file contains the expected configuration.


AnimationPlayer - Nodepath in a track in a animation stored in a PackedScene get their instance name automatically when other instance of that packed scene is reparented in the container scene.

Godot version:

5e837b3f13ab1e3b31bb8d705e87820fa4eff21e
OS/device including version:

Win7
Issue description:

An Animation in PackedScene with an AnimationPlayer with a track called:
EnemyPath2D/PathFollow2D:unit_offset
is automatically renamed to:
"../**InstancedPackedSceneName**/EnemyPath2D/PathFollow2D:unit_offset"
on reparenting other copy of the PackedScene in the Scene that contains that PackedScene....
This change produces that only the first packed scene loaded in the currentscene can play the animation (because the "InstancedPackedSceneName" is stored in the animation), and only this instance can follow that track.  The packed scene remains inservible by itself (That packed scene doesn´t have that track name outside the scene). This behaviour happend with the unsaved resource or the animation stored in an independent tres file.
To note that InstancedPackedSceneName is unique per Node (godot doesn´t allow instances with the same name) and relative to every scene (you can change manually), so , this name never should be added to the tracks from the original PackedScene or the original resource (Godot should not change animation tracks names ever, this is not good behaviour, it can break to many things). If you change that track name manually, the animation is reproduced in all nodes, but on next saving, godot by itself destroy the reference other time.
Minimal reproduction project:

Will try...

Sorry about corrections, but this bug is annoying me and it was not clear the origin of that track name change...

Original packed scene with the animation:

Working:

A Scene with 4 instances working:

I add a new scene..... all works:

I add other scene as a child of other.... all stills work:

I reparent and only one can follow the animation... the original track was renamed:

This is the auto-modified animation:

For the sake that you need to experiment by yourself:
Godot3_1_bug_Catch.zip

I don´t know hot to do that in c++ and godot codebase, but in:
https://github.com/godotengine/godot/blob/master/editor/scene_tree_dock.cpp
In this lines:
1183	bool autorename_animation_tracks = bool(EDITOR_DEF("editors/animation/autorename_animation_tracks", true));
1184
1185	if (autorename_animation_tracks && Object::cast_to<AnimationPlayer>(p_base)) {

i think is needed to test if ... p_base is a PackedScene or something like that,

I think this issue it´s important to 3.1 stable, since can make a big headache to people without version control system....

The provided project seems to work fine in 3.2.2 RC 2 after reparenting the instance, can you confirm if it's solved for you too?

No, it´s not fixed. The problem is the "autorename" of nodes that godot performs. The operation that should be done to catch the bug is reparent a node, rename to some name that godot needs to change to not have duplicity(Ej. "Creature") , and reparent other time, and at this moment, godot changes the  name to "Creature6", the animation changes to ../ parent and the others nodes doesn´t perform the animation... If you follow the gifs, i think it´s clear the problem.
I tested in Godot 3.2.2 RC2 with the attached demo.

More specifically, the problem is that the "PackedScenes" shares the track names in the AnimationPlayer, and the Parent Scene of that instanced scene, changes all track names in the PackedScene instead in the animation track of the "Instanced Scene", breaking all other animations. But main problem described here is still valid.
I think in 2 solutions:

Doesn´t allow a Scene to change animation track names of the child instanced scenes by packed scenes in reparent. (Respecting encapsulation)
Store animation track changes in Parent scene instead change child instanced scenes (BTW deep change in dessign and complicated too much, breaking encapsulation and other "programmer things", but it´s posible, the same that you can change "modulate" and "transform" or "exported variables").

I want to note my comment of 23/02/2019 and the "autorename animation tracks" project setting. It´s still valid. The bug is related with this setting.

Still valid in 3.3.2 and f030b0c
This is weird, because I remember similar issue being fixed. Maybe only some cases were fixed.


[4.5] CLOUDSTACK-8848: ensure power state is up to date for missing PowerState handling

1:1 cherry-picks, see #909

LGTM, merging on 4.5 as we've already merged this on master. Thanks for the PR @resmo


TODO 2

Routing

[x] Deprecate chained route? #105
[x] Support nested(like) routing? #105
[x] One handler
[x] noMatch
[ ] RegExpRouter as default?

[ ] Web sites?
[x] Stacktrace #158
[x] Fix API
[ ] Promotion
[ ] How to test app / example
[x] Add project template to Cloudflare QuickStart
[x] Web Sockets
[x] Determine Cloudflare or Compute@Edge or not
[x] Dynamic import / lazy load third party modules if really want to use some Middleware
[x] Image logo #153
[ ] Documentation #71
[ ] README
[x] Document in Japanese #131
[x] Badges on README 68baca09492ba5b4fef057b247c3285887a99626
[x] View
[x] slurp content from files in KV #72
[x] source map
[x] uglify
[x] custom error object #111
[x] app.onError #111
[x] Customize error message #111
[x] DEBUG flag?
[x] Error handling: => app.get('/', (c) => 'Hello Hono!')
[x] Debugging`
[x] Redirect trailing slash?
[x] Return method not allowed
[x] c.stash ?
[x] 405 method not allowed
[x] c.render with other middleware
[x] Obsolete chained route and c.route ??
[x] TypeError: c.req.header is not a function
[x] Cache-Control for serve-static
[x] set HTTP statusText automatically
[x] Fix examples #101
[x] Durable Objects example
[x] Benchmark for Compute@Edge
[x] Rewrite test for blog. Using jest-miniflare
[x] Do not include tests into NPM package?
[x] pretty option on c.json #150
[x] About TypeScript in Japanese README
[x] types.ts
[x] Directory structure

Middleware

[x] Serve Static for Cloudflare cf-static serve-static #76
[x] Basic Auth support Compute@Edge? #86
[x] Basic auth support multiple users #112
[x] throw error if basic auth user/pass is not set #112
[x] Favicon ?
[x] compress
[x] Etag #119
[x] GraphQL
[x] View with mustache #72
[x] View with React
[x] Stack trace
[x] logger use waituntil?
[x] parsedBody as any?
[x] stack trace in graphql

Not sure

[x] Development / Production mode?
[x] Try esbuild

Almost done!! To be continue ... => https://github.com/yusukebe/hono/issues/166


pytest-profiling  does not profile fixtures

Consider the following example:
import pytest

@pytest.fixture()
def work_func_fix():
    s = 0
    for i in range(1000):
        s += i * i
    return s

def work_func_test():
    s = 0
    for i in range(1000):
        s += i * i
    return s

def test_example(work_func_fix):
    assert work_func_fix == work_func_test()

Apparently, only test_example is profiled. work_func_test appears in the profiling results, but work_func_fix does not.
Is it a design choice? I've been searching but found no reference to that. Or is there a flag I should pass to enable fixture profiling?
We use pytest to test a Flask application. Fixtures create the database, and instantiate and initialize the application. We'd like to profile the initialization process but it does not appear in the results.
In fact, the profiling result in my example above shows a lot of things I don't care about (everything that is not under test_example), but not the fixture do I care about.

I do get the fixtures profiled with this change:
     @pytest.hookimpl(hookwrapper=True)
-    def pytest_runtest_call(self, item):
+    def pytest_runtest_protocol(self, item, nextitem):
         prof_filename = os.path.abspath(os.path.join("prof", clean_filename(item.name) + ".prof"))
         prof = cProfile.Profile()
         prof.enable()

This is probably obvious to the devs. I suppose the choice of using pytest_runtest_call was deliberate.
Do you consider my use case invalid? Could pytest_runtest_protocol be used, at least with an opt-in argument?

I couldn't say. @eeaston might know

@eeaston what do you think?

looking over the documentation, I think the change would actually be fine. pytest_runtest_call simply executes the test item whereas pytest_runtest_protocol calls the setup, test, then tear down and the associated hooks.
I think we can safely change this @eeaston but it will mean the behavior would change. I think its behavior we want, but I wanted to double check and see what you thought.

Any update on this issue?

I am getting different results when I profile with cProfile and pytest --profile. Since I use fixtures, I think that is due to this issue. Can somebody explain why pytest-profling skips fixtures? In my case I use caching in my fixture to run subsequent tests faster.

Hi @lafrech  - thanks for raising this and it's definitely a valid use case.  There's a number of issues with the profiling that I'm working through at the moment and I'll cover this as well.
Your example implementation is good - we need to get the profiler turned on early enough to cover fixture execution.


Coverage decreased (-0.002%) to 96.173% when pulling 53622ab2bbf7dcc70c313e1998799b960bceae4b on dw--cleanup into d04d6c03170ac6a51d91bd0b0845e1cce7441acc on release-3-0-0.


Check built output

Reason (Why?)
Google Lighthouse claims that our text (js / css) is not served with compression (like gzip) and that we serve unused javascipt.
Solution (What?)
Check and fix text compression. This probably needs some work with the vue build command and webpack.

Google Lighthouse claims that our text (js / css) is not served with compression (like gzip)
This can be fixed in nginx by enabling zip (and brotli if possible) compression. Precompressing ressources is ususally not necessary

This will be fixed in #483


FTUE | Educational | Drag and Drop

Context
First Time User Experience, or FTUE, helps a new user familiarize themselves with certain parts of the product. We strive to keep the product generally intuitive and easy to understand by following common app patterns. As such, we focus FTUE on a narrow set of features that may be more novel to users and we believe will be useful to the majority of users.
Explaining new concepts
A few concepts in web story creation may be new to users, thus the editor introduces them to the user at the appropriate time.
Feature Description

Drag and drop/masking:
From product brief:

Drop zones are an expedient way to set a background or apply a mask to a media element. The interaction is intuitive once the user first sees it, but it is not a standard pattern users may have seen. As such, the editor lets users know about drop zones:
Alternatives Considered

The first time the user clicks to add a media element to the page → drag to page borders to set as background.
When a user adds a shape to the page → drag any media to shape borders to apply as mask.

Additional Context
See Ux research
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
Implementation Brief

@aaecheverria checking in on this and moving to UX needed. This is from November so I'm not sure if it's still relevant?

Hi @BrittanyIRL Thanks for checking. This is not relevant anymore since we already released this with FTUE helper menu.
I'm closing it out!


feat(autoware_auto_msgs_adapter): add map adapter

Description
Add msg adapter for autoware_map_msgs and autoware_auto_map_msgs
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Thanks for working on this!
I've rebased it to the latest commit. Now will review it.

need help with ci pipe line @mitsudome-r  mits-san, there seems no change since last week

Hi, @isamu-takagi isamu-san. Please help us with this pr's review then we can merge it ASAP

@liuXinGangChina
It is enough if one of the code owners approves it. If you can't perform the merge, it's because of GitHub permissions. Is it okay to merge this PR now?

@liuXinGangChina It is enough if one of the code owners approves it. If you can't perform the merge, it's because of repository permissions settings. Is it okay to merge this PR now?

sorry @isamu-takagi , i don't have the merge right, please help us to merge it, thanks.


Own PS1 in .bashr override

I use my own PS1. I have it in my .bashrc file like export PS1=... which overrides the one passed to container. And then it is really confusing if I am in toolbox or on my machine.
Is there some simple way how to easily recognize from toolbox that I am inside (so I can edit my PS1 to recognize this)?

$HOSTNAME is toolbox in toolbox

Thank you for the hint.
Prepending $(if test $HOSTNAME = "toolbox"; then echo -n "🔹[\u@\h \W]\\$ "; fi;) to my PS1 solves this issue :)


factory.componentForType(aProtocol) does not work in Swift

This may be because the protocol wasn't marked with @objc

Doesn't work even for @objc protocols.

We strongly recommend to use the pure Swift version of Typhoon.

Perhaps we can make it more clear Typhoon Swift is ready for heavy use in the Readme.


update the Azure multi tenants for Exchange

Original line 158 was creating confusion by mentioning
"Only one of the Azure AD tenants can enable an Exchange hybrid with the on-premises Active Directory instance"
Original line 158 needs to be updated as we now support up to 5 Hybrid tenants for Exchange as per https://techcommunity.microsoft.com/t5/exchange-team-blog/september-2020-hybrid-configuration-wizard-update/ba-p/1687698

@zachza : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@msdmaguire: I'm sorry - only the author of this article, @billmath, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@msdmaguire Do you want to make additional edits to the update that's submitted in the PR? You could indicate them here or use the comment functionality in diff view, and I'll implement before merging.
The image file names currently use casing that doesn't match what's in master, so in GitHub Preview, the images appear broken (although they display on the live site). (GitHub is case-specific.) I've changed them so that they match, and images display in GitHub Preview: https://github.com/MicrosoftDocs/azure-docs/blob/92e2d1939e2b5376917efdaec626cf017e2ad8a1/articles/active-directory/hybrid/plan-connect-topologies.md

@ktoliver I saw all your edits and they look good to me. The image file names we don't own, though that's a good catch. This PR is just myself and a partner in support wanting to update a few lines related to Exchange (which you copy-edited nicely). Please feel free to accept all changes and merge. Thank you.


mp4 location

Hey,
thanks for this nice tutorial and sorry for opening an issue (I cannot find any contact details).
I have successfully deployed the application
and I am able to call the link for my project (Transcode -> Done).
However the webm file is not generated. Where should the mp4 file be located on the Cloud? In the bucket of xxx.appspot.com?
Thanks in advance and I am looking forward to your help!!
Hannes
<EMAIL_ADDRESS>

did you get any feedback on this?

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @kisasexypantera94, @desynched0, @jfr4nc0, @Arooj28, @ryc8889, @jbaerga, @D3Ext, @tianrking, @javierjrueda, @ZairRueda, @TheWaWaR, @timonteutelink, @jh-333, @yujuiting, @HalmaiErik, @aeryskyB, @Princeprince559, @gregorynicholas, @jyopp, @hee10k, @galvin7, @lurodrigo, @airtonGit, @Jamonek, @likewu, @Mdhesari, @wheng3, @buchekalex, @oscarrock, @Syuq


Reference latest CRI-O release

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Use CRI-O v1.31.0 as latest minor release reference.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

@cri-o/cri-o-maintainers PTAL

/approve
/lgtm


Bug Report

Describe the bug

I added my email but it says User not pre-authorized to register is required.
Steps to reproduce bug

I am using the master branch


Change test file names to match Scala convention.

For example, in robot-name, the filename for the spec is robot_name_test.scala.
Underscores are unusual in Scala filenames. I'd expect something like RobotNameSpec.scala.

You are right, these names could be changed.
The newer exercises have the naming convention to end with "Test", for example "BankAccountTest.scala" or "AllYourBaseTest.scala". So we should stick to that.

Agreed, both with the PascalCase naming and the "Test" suffix.


Activation Transactions - Activation Stake - Signaling/Endorsement mechanism

So far, we have said that:

When the Update Proposal (UP) is revealed the update software is downloaded on the node
When the UP is voted by the majority of stake (we still have to agree on the threshold), then the update software is automatically installed on the node.

What if, we marked the event of this installation with yet another new type of transaction, the "Activation Transaction" that will be broadcasted to the network and eventually committed into the blockchain?
This way, we can calculate at any time the Activation Stake and this will correspond to all stakeholders and not just the miners (as is in the case of counting the block proportion). Also, we will keep a historical record of the activation events per se.
We have to decide, if we will allow for a node to opt-out from the installation of an update regardless of his/her vote for an UP. If we allow this (which is reasonable I think in a democratic setting), then we really need to keep track of the Activation stake, in order to monitor real adoption of the update and avoid chain splits. In that case, maybe we might need to reconsider the existence of a Signaling/Endorsement mechanism that is based on a threshold of the Activation Stake. This is similar to the Bitcoin BIP9 logic and also to the current update mechanism for Cardano, with the difference that this is not a Miner-based Activation but a Stake-based Activation, since we count activation transactions issued by stakeholders and not blocks mined by miners.
The Activation Stake threshold could be high for hard forks, medium for soft forks and minimal for velvet forks and UPs that dont impact the consensus.
What do you think?

The activation phase has been described in detail in the FC20 submitted paper.


New scaler: Google Cloud Tasks

Proposal
I would like to be able to set up autoscaling for Google Cloud Tasks.

 Using Cloud Tasks, you can perform work asynchronously outside of a user or service-to-service request.

Scaler Source
api/request_count, queue/depth
Scaling Mechanics
I see this as being similar to the existing RabbitMQ triggers where you could define a request rate trigger to scale up with the number of incoming items, as well as a queue depth trigger in case consumption slows down and you begin to accumulate a backlog.
Authentication Source
Google Cloud IAM
Anything else?
No response

That's a good idea! Are you willing to contribute it?

@tomkerkhove I'm working on a proposal for using KEDA at Sentry - if that goes through and we implement it it may be something I can look at.. don't think I can put in the time if we're not using KEDA yet though. :D

Sounds great though!

Hi @tomkerkhove I would like to take this up

Awesome, thank you! Let us know if you need help, you can learn more in our contribution guide.

Hey @Shulammite-Aso are you already working on this? I also need this and have time to try to do this. Tell me if you are on it not to waste time. Thank you!

Hi @jmalvarezf-lmes I started working on this, but I have gotten a bit too busy and can't seem to pay close attention to this for now. If you have the time you can take it up.


Effort edit modal is not reporting errors when update fails

When a user is in the Effort Edit modal and clicks "Update Entrant," if validations fail no errors are reported back on the form.

Resolved by #1084


TTLState node after TTLIn output not working properly

Hi Aarón! Is there any difference in the implementation of TTLIn in the Acquisition Board node for the OE FPGA Acq Board with respect to the Opal Kelly FPGA Acq Board RHDEvaluation Board node?
On using TTLState after TTLIn, a user got the following error, are were not able to run the workflow with the demultiplexer:

This previously did not require any special configuration so I'm not sure what could be happening. Happy to investigate more when I have the hardware back in my hands.

That part should be the same. What is the full workflow that caused that?


Fix magic comment: frozen_string_literal: true

This should be started with #, not enumerated with comma.
Current (Not expected)
# coding: utf-8, frozen_string_literal: true

a = "a"
a.frozen? #=> false
a.succ! #=> "b"

Expected
# coding: utf-8
# frozen_string_literal: true

a = "a"
a.frozen? #=> true
a.succ! #=> can't modify frozen String: "a" (FrozenError)

Thanks!


ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None

Please suggest how to fix this?
mona@pascal:~/computer_vision/VPilot$ python train.py
Using TensorFlow backend.
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcudnn.so.5.0 locally
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcurand.so.8.0 locally
/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py:1938: UserWarning: Expected no kwargs, you passed 1
kwargs passed to function are ignored with Tensorflow backend
  warnings.warn('\n'.join(msg))
Epoch 1/1000
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 429, in data_generator_task
    generator_output = next(self._generator)
  File "/home/mona/computer_vision/VPilot/model.py", line 61, in dataGenerator
    x[batchCount, frameIndex, :, :, :] = cv2.imread(frameFile, mode).astype('float32')
AttributeError: 'NoneType' object has no attribute 'astype'

Traceback (most recent call last):
  File "train.py", line 56, in <module>
    callbacks=[ckp_callback]
  File "/usr/local/lib/python2.7/dist-packages/keras/models.py", line 935, in fit_generator
    initial_epoch=initial_epoch)
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 1528, in fit_generator
    str(generator_output))
ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None

code is: https://github.com/ai-tor/VPilot/blob/master/train.py

Your generator threw an exception:
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 429, in data_generator_task
    generator_output = next(self._generator)
  File "/home/mona/computer_vision/VPilot/model.py", line 61, in dataGenerator
    x[batchCount, frameIndex, :, :, :] = cv2.imread(frameFile, mode).astype('float32')
AttributeError: 'NoneType' object has no attribute 'astype'

cv2.imread() returned None, meaning, I guess, that it couldn't open the file that was specified.

If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.

Traceback (most recent call last):
File "model.py", line 163, in
main()
File "model.py", line 159, in main
train_model(model, args, *data)
File "model.py", line 120, in train_model
verbose=1)
File "/home/shyam/anaconda3/envs/car-behavioral-cloning/lib/python3.5/site-packages/keras/models.py", line 924, in fit_generator
pickle_safe=pickle_safe)
File "/home/shyam/anaconda3/envs/car-behavioral-cloning/lib/python3.5/site-packages/keras/engine/training.py", line 1481, in fit_generator
str(generator_output))
ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None

can someone help me how to solve this error

Your generator yielded None, or nothing. It could be caused either by the generator not being in a loop and running out of data to yield, or the generator itself might cause an Exception which would give this error.
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------From: shyam1992atri<EMAIL_ADDRESS>Date: 6/26/17  10:54 AM  (GMT-08:00) To: fchollet/keras<EMAIL_ADDRESS>Cc: Pat York<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [fchollet/keras] ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None (#5196)
can someone help me how to solve this error
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/fchollet/keras","title":"fchollet/keras","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/fchollet/keras"}},"updates":{"snippets":[{"icon":"PERSON","message":"@shyam1992atri in #5196: can someone help me how to solve this error\r\n"}],"action":{"name":"View Issue","url":"https://github.com/fchollet/keras/issues/5196#issuecomment-311133455"}}}

Thank you for your reply.
I got it fixed from here
https://github.com/llSourcell/How_to_simulate_a_self_driving_car/issues/7

My fix was making sure batch size was int, not float when passing to flow from directory.

I got the same error and traceback as follow:
Traceback (most recent call last):
File "c:\Users\asojasingarayar.vscode\extensions\ms-python.python-2019.1.0\pythonFiles\ptvsd_launcher.py", line 45, in
main(ptvsdArgs)
File "c:\Users\asojasingarayar.vscode\extensions\ms-python.python-2019.1.0\pythonFiles\lib\python\ptvsd_main_.py", line 348, in main
run()
File "c:\Users\asojasingarayar.vscode\extensions\ms-python.python-2019.1.0\pythonFiles\lib\python\ptvsd_main_.py", line 253, in run_file
runpy.run_path(target, run_name='main')
File "C:\Users\asojasingarayar\AppData\Local\Continuum\anaconda3\envs\car-behavioral-cloning\lib\runpy.py", line 254, in run_path
pkg_name=pkg_name, script_name=fname)
File "C:\Users\asojasingarayar\AppData\Local\Continuum\anaconda3\envs\car-behavioral-cloning\lib\runpy.py", line 96, in _run_module_code
mod_name, mod_spec, pkg_name, script_name)
File "C:\Users\asojasingarayar\AppData\Local\Continuum\anaconda3\envs\car-behavioral-cloning\lib\runpy.py", line 85, in _run_code
exec(code, run_globals)
File "d:\MyProjects\finding-lanes\Simulate-Self-driving-Car\model.py", line 174, in
main()
File "d:\MyProjects\finding-lanes\Simulate-Self-driving-Car\model.py", line 170, in main
train_model(model, args, *data)
File "d:\MyProjects\finding-lanes\Simulate-Self-driving-Car\model.py", line 131, in train_model
verbose=1)
File "C:\Users\asojasingarayar\AppData\Local\Continuum\anaconda3\envs\car-behavioral-cloning\lib\site-packages\keras\models.py", line 924, in fit_generator
pickle_safe=pickle_safe)
File "C:\Users\asojasingarayar\AppData\Local\Continuum\anaconda3\envs\car-behavioral-cloning\lib\site-packages\keras\engine\training.py", line 1481, in fit_generator
str(generator_output))
ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None
Model train script is available here

I solve this problem.
I think that 'segmentation_models' is implemented using 'keras' not 'tf.keras'.
Error version(for using segmentation models)
import tensorflow    Class Datagenerator('tf.keras.utils.Sequence')
Revised Version
import keras class Datagenerator('keras.utils.Sequence')
and it worked. I hope you guys get help. thanks

validation_data should be a tuple (val_x, val_y, val_sample_weight) or `(val_x, val_y)

@sunshower76 where would one make the edit you specified about tf.kera vs. keras? Specifically what .py file? Thank you.


Create a StrmanType

Each function should return a StrmanType that contains all functions of project, thereby we could make function chains, the result of all functions on the chain should be a string.

won't be implemented for now.


Add Rule to Format Yaml Arrays

Fixes #172
Fixes #291
Fixes #182
Changes Made:

Added a rule to account for updating yaml array formats and forcing a regular string value to be a yaml array
Added a bunch of tests to handle some possible test cases
Added a couple of examples

Still needs work around unindented array items.

I need to fix that it it is removing the quotation marks around single and double quotes in the yaml arrays when it adds them back.


Refactor to add missing switch cases for enum

🔍 Search Terms
"switch", "enum", "refactor", "code action"
✅ Viability Checklist

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[x] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
Provide a refactor to autocomplete the missing switch cases for a enum value.
📃 Motivating Example
enum E  {
    a = 1,
    b = 2,
    c = 3
}

function f(e: E) {
    switch (e) {
        // provide refactor here to add missing case branches
    }
}

💻 Use Cases

What do you want to use this for? Better DX
What shortcomings exist with current approaches? manual input
What workarounds are you using in the meantime? manual input

It's already there in the case completion:

Thanks, I didn't noticed this completion exists. Maybe it is too hard for users to discover? I think it would help to add this completion at the "enum dot" position:


It isn't possible to deploy to a virtual application

Azure Pipelines App Service Deploy tasks have the "VirtualApplication" input to allow deployment to a virtual application within the main App Service. There is no equivalent in this Action. Is this by design?

@snorydr  Yes for now we do not have virtual application deployment in Actions. We will be adding it on a later stage.

Any news on this? Am in need for it (and I figure most SPA <-> API devs not liking pre-flights are...) as actions can automate every workflow very nicely, ecxept for this one. Thanks!

Any news on this? Am in need for it (and I figure most SPA <-> API devs not liking pre-flights are...) as actions can automate every workflow very nicely, ecxept for this one. Thanks!

This will be a very helpful feature to include.


Added new hyperv vsock patches for addressing connection close racing…

… condition issue
This new patch addressed the vmbus_on_event() kernel crash" that rneugeba reported last week.

@soccerGB,
Thanks for your contribution.  To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
The agreement will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


Extends export onnx to code to support inner API

Codecov Report
Attention: 33 lines in your changes are missing coverage. Please review.

Comparison is base (75d62a0) 81.32% compared to head (ec49a87) 81.78%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   81.32%   81.78%   +0.46%
==========================================
  Files          46       48       +2
  Lines        6228     6403     +175
==========================================
+ Hits         5065     5237     +172
- Misses       1163     1166       +3

Files
Coverage Δ

onnx_array_api/light_api/__init__.py
93.75% <88.88%> (+1.44%)
:arrow_up:

onnx_array_api/plotting/_helper.py
69.00% <50.00%> (-0.39%)
:arrow_down:

onnx_array_api/light_api/inner_emitter.py
96.05% <96.05%> (ø)

onnx_array_api/light_api/translate.py
76.84% <78.26%> (+8.21%)
:arrow_up:

onnx_array_api/light_api/emitter.py
84.86% <84.86%> (ø)

... and 1 file with indirect coverage changes

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


assert vec length (size)

Is there a way to assert vector length, e.g.
let result = call canister.getItems();
assert size(result) == (1 : nat);

Thanks!

@chenyan2002 Nice to hear! Can you please add this? Also I think length is a bit more descriptive name, to not confuse size with memory size in bytes. Looking forward for this.Thanks in advance!

Hey @chenyan2002 ! Thank you for looking into it. Can you please document somewhere how can I use it? I checked out both merged PR-s and unfortunately it's not clear to me how to do it

You can see a full example from: https://github.com/chenyan2002/ic-repl/blob/master/examples/func.sh.
For your specific use case,
let result = call canister.getItems();
function count(acc, x) { let _ = acc + 1 };
assert result.fold(0, count) == 1;


Use a platform-dependent uintptr_t process id

This is what BuildSystemFrontendDelegate::ProcessHandle expects, right now this uint64_t pid will break the build on all the 32bit platforms.

@swift-ci please smoke test

Yeah, this same change caused failures in #382 and we haven't had a chance to dig in to understand why.

Completely unexpected error, looks like something that would require an inordinate amount of digging...

Is anyone planning on picking this up? If not, I’d like to close it out until someone is actually going to give it attention.

@ddunbar - interestingly, this is the same functional change as #382 - just changing the process ID type.

@swift-ci please smoke test

@swift-ci please smoke test

I am unable to reproduce the macOS failure locally.
@dmbryson is this a known flaky test or something?

It is not a known flaky test, which is what is perplexing about this failure.

That is very bizarre.

This is what BuildSystemFrontendDelegate::ProcessHandle expects

Not anymore, 111d982.

@buttaface thanks A LOT for pointing this out, I'll check how this behaves on arm32 and report back.

No problem, it fixed breaking builds for 32-bit ARM/x86 with Android, so I backported it for that Android package.

Ok, that merged commit fixes the original build issue (and that finicky test seems to pass), I'm closing this. SPM seems to be working normally too.


[Suggestion] Make the bot as administrator

Please change the invite link from
self.INVITE_URL = os.getenv('INVITE_URL', 'https://discordapp.com/oauth2/authorize?client_id={}&scope=bot')
to
self.INVITE_URL = os.getenv('INVITE_URL', 'https://discordapp.com/oauth2/authorize?client_id={}&permissions=8&scope=bot')
This useful if you want to play the bot in hidden / private voice channel

Hello there.
Yeah, I think that this change can be very nice for the Bot.
Would you like to create a Pull Request with this change?
If not, I'll change this line when it's possible for me.

Done


Hello microverse project

Created a new GitHub repo.
Created a new branch
Add a .gitignore file.
Add node_modules to gitignore file.
Set up a linter for HTML and CSS
Add descriptive README file to your project
Add 3 files (each one in a separate commit):
A .html file that includes a header with the text "Hello Microverse!". Use h1 tag for that.
A .css file that includes one class with styles for your header (make it in your favorite color).
A test.md file with the text "This file should be ignored by git".

This a simple HTML and CSS project.

I have made the changes.

Merging the branch to the main after approval.


camera is mirrored

On a Nexus 5X, the camera image is mirrored in both directions. This makes it hard to line up.

Hmm... Thanks for reporting this. I'll try to look into it.

Found some info about it here https://stackoverflow.com/questions/37560871/camera-is-inverted-in-nexus-device

I forgot to respond to this. Sorry about that.
I think where I'm at on this is:

it seems like (mostly) hardware specific behavior

android devs may protest that in fact they're following the standard and it's everyone else that is wrong, and they may be technically correct. But it seems to be only Nexus devices that use this rotation feature and as a result the rest of the world (including the opencv tutorial code I'm using) doesn't account for it.

I don't have the hardware in my possession

If somebody wants to take a look at a change and make a PR to address this, I'll take it (and test it on the devices I can), but I'm unlikely to spend more time on it than that. If it's any consolation, if/when I get a web-based version of the decoder to work, I'd expect we won't be battling too much hardware specific stuff like this. (I hope... :grimacing: )
Marking as help needed to signify that...


When alter a column from the Table properties view it freezes if tabled is locked

Description
I just tried to change a column of a table that was locked by another query.
While the table was locked I couldn't go out of the Table properties tab.
DBeaver Version
23.3.0 community
Operating System
macOS 14
Database and driver
Redshift JDBC <IP_ADDRESS>
Steps to reproduce
lock a table.
try to change the length of a column
dbeaver freezes
Additional context
No response

Hello @alejandrofm
Please provide more info about the step one. How exactly did you lock your table?

Hi! didn't lock the table, the table was locked by a process that had an exclusive lock on it at that time.
The issue here is that DBeaver freezes until it has access to the table, when it should wait but let me use the other tabs/queries, etc.
Thanks!

the table was locked by a process that had an exclusive lock on it at that time.

Steel I need more info about it to reproduce your issue.
Could you tell me more about this process?

you can do anything that locks a table, vacuum, delete, another way to force a lock will be:
begin;

LOCK table;

long query so the lock last a while;

end;

https://docs.aws.amazon.com/redshift/latest/dg/r_BEGIN.html
https://docs.aws.amazon.com/redshift/latest/dg/r_LOCK.html

Hi @alejandrofm
I couldn't reproduce it in the latest v23.3.2. Could you please provide capture of the freezing you see?

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


[flutter_tools] flutter create -t plugin message can be improved.

flutter create -t plugin's message in below situations are too verbose.
1. When flutter create -t plugin runs without a --platforms flag.
The current message looks like
Must specify at least one platform using --platforms.

Run `flutter create -t plugin --platforms <platforms> .` under the plugin/ directory.
You can also find detailed instructions on how to add platforms in the `plugin/pubspec.yaml` at
https://flutter.dev/docs/development/packages-and-plugins/developing-packages#plugin-platforms.

To find out which platforms are supported, run `flutter create -h`.

2. When flutter create -t plugin --platforms=<> runs on an existing plugin folder.
The current message looks like
The `pubspec.yaml` under the project directory must be updated to support ios,
Add below lines to under the `platform:` key:

ios:
 pluginClass: PluginName15Plugin

If the `platforms` key does not exist in the `pubspec.yaml`, it might because that the plugin project does not
use the multi-platforms plugin format. We highly recommend a migration to the multi-platforms plugin format.
For detailed instructions on how to format the pubspec.yaml to support platforms using the multi-platforms format, see:
https://flutter.dev/docs/development/packages-and-plugins/developing-packages#plugin-platforms

We should try to reduce the action items in above situations to a single sentence, then link to a detailed explanation to a web documentation.

For no platforms: https://flutter.dev/docs/development/packages-and-plugins/developing-packages#step-1-create-the-package-1
For add platforms: https://flutter.dev/docs/development/packages-and-plugins/developing-packages#plugin-platforms

Waiting for https://github.com/flutter/flutter/pull/70215 to land to close this issue.


Update pagination labels to be 14px

:tada: This PR is included in version 4.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Animation plays itself when a state changes

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
First I'm rendering an animation with autoplay set to false.
this.options = {       loop: false,       autoplay: false,       animationData: require("./leo_anim.json"),       rendererSettings: {         preserveAspectRatio: "xMidYMid slice"       }     };
The animation does not start which is perfect, but then I call setState() function and the animation starts playing
See the behavior here: https://codesandbox.io/s/2wk6nv3pvr
What is the expected behavior?
If the animation has autoplay set to false, then it should only start when the play function is called.
Which versions of Lottie
Both "react-lottie": "1.0.0" & "react-lottie": "1.1.0"
Thank you for your help!

It is still not fixed

@gisellezeitoune I've encountered the same problem - it's still not fixed
However, a workaround is as followed;
Instead of using isStopped as the above post suggests use isPaused as things SEEM to work fine
<Lottie
  options={defaultOptions}
  isPaused={!isStopped}
  style={{
    width: 50,
    height: 50
  }}
/>

@chenqingspring

Yes, I think that is a bug to be fixed. I'll handle the logic in the wrapper.

Sorry to bother you about this open issue from 2018 but it's causing our team a lot of pain. Any state change in the component knocks out autoplay: false.
Is there any chance you can look into this or do you need help with it, it's a fairly critical bug. Thanks!


Request: Configurable folders to serve the client from

Right now in Blazor, unless you specify "_framework", the server is going to try to send you index.html - if the URL is "bad".
(See:
https://github.com/aspnet/Blazor/blob/master/src/Microsoft.AspNetCore.Blazor.Server/Builder/BlazorApplicationBuilderExtensions.cs#L162)
It would be useful to be able to configure the URLs from which the framework will be serviced.  ie - If I specify a bad URL, I would rather my server send a 404 than deliver a couple of megabytes of framework before deciding to render the default page.
Ideally, a "strict" mode would return only for the root URL (whatever it may be) and all other URLs would be at the server-side leisure for what to return.

Generally the server doesn't know which URLs are valid or not, because that's only determined by the client-side router, which can make any decision it wants dynamically.
If your app follows some URL pattern whereby you do know which URLs should serve the client app and which shouldn't, you can put in your own server-side ASP.NET Core middleware that halts the request with a 404 if it doesn't think the client app should be loaded.

It would be useful to be able to configure the URLs from which the framework will be serviced

Hopefully, ASP.NET Core middleware covers this use case. If not, please let us know what extra you need. Hope that's OK!


Artifact contains R.txt

AAR artifacts in versions 1.1.0+ and 1.0.14 contain file R.txt which causes gradle to fail during task process{buildType}Resources when I try to build library project.
Log:
:library:processReleaseResources FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':library:processReleaseResources'.
> java.io.FileNotFoundException: /users/user/project/library/build/intermediates/bundles/release/R.txt (No such file or directory)

This file is not included in the artifact if I build it locally.

I cannot reproduce the issue. The build works fine for me. Are you using the gradle wrapper? Did you try a clean?

Additional info, the library project is only affected if it doesn't contain any resources

You mean that you use the job library in your library project, which doesn't contain any resources?

Yes, so far I was using version 1.0.13 without any issues.

Thanks Android gradle plugin! The problem is, that gradle packaged an old R class, which shouldn't be there. I'll publish another version :(

This is fixed in version 1.0.15 and 1.1.2. Please let me know if that works and feel free to reopen the issue if necessary.


Docs: improve output documentation

Thanks for contributing to jsPDF! Please follow our
Contribution Guidelines
when creating a pull request.
#3081

Thanks :) Looks good, I'll merge it.


ERROR: keras-rl2 1.0.3 has requirement tensorflow==2.0.0-beta1, but you'll have tensorflow 2.1.0 which is incompatible.

Hello,
requirement for keras-rl2 is tensorflow ==2.1.0
When I installed keras-rl2 by pip, it removed my tensorflow 2.1.0 and instaled 2.0.0beta version.
But my code did nor ran and I tryed install tensorflow 2.1.0 as it requires, but got this error
"ERROR: keras-rl2 1.0.3 has requirement tensorflow==2.0.0-beta1, but you'll have tensorflow 2.1.0 which is incompatible.
Installing collected packages: tensorflow
Attempting uninstall: tensorflow
Found existing installation: tensorflow 2.0.0b1
Uninstalling tensorflow-2.0.0b1:
Successfully uninstalled tensorflow-2.0.0b1
Successfully installed tensorflow-2.1.0"

[ X] Check that you are up-to-date with the master branch of Keras-RL. You can update with:
pip install git+git://github.com/wau/keras-rl2.git --upgrade --no-deps

[ X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

The pypi wheels have been updated with the latest changes in the github branch.
Running
pip install keras-rl2

now installs keras-rl2 with the latest 2.1.0 tensorflow version.