buffer empty on sprite hitTest when using compressed texture

Describe the bug
I have the following error when my mouse hits a sprite filled with beginBitmapFill using compressed texture (ATF).
caught TypeError: Cannot read properties of null (reading 'buffer')
    at lime__$internal_graphics_ImageCanvasUtil.convertToCanvas (ImageCanvasUtil.hx:32:16)
    at openfl_display__$internal_CanvasGraphics.createBitmapFill (CanvasGraphics.hx:95:3)
    at openfl_display__$internal_CanvasGraphics.playCommands (CanvasGraphics.hx:765:27)
    at openfl_display__$internal_CanvasGraphics.endFill (CanvasGraphics.hx:226:3)
    at openfl_display__$internal_CanvasGraphics.hitTest (CanvasGraphics.hx:325:7)
    at openfl_display_Graphics.__hitTest (Graphics.hx:1678:12)
    at openfl_display_Sprite.__hitTest (Sprite.hx:316:53)
    at openfl_display_Sprite.__hitTest (DisplayObjectContainer.hx:839:22)
    at openfl_display_Sprite.__hitTest (Sprite.hx:312:7)
    at DocumentClass.__hitTest (DisplayObjectContainer.hx:839:22)

To Reproduce
Steps to reproduce the behavior:

Use a ATF texture
create a sprite using drawRect or drawTriangles and beginBitmapFill
move your mouse onto the sprite
See error

Expected behavior
The buffer shouldn't be empty.
OpenFL Targets
all
Code Sample
var context3D = stage.context3D;
var atfData = Assets.getBytes ("assets/kenney-poly.atf");
var texture = context3D.createTexture (4096, 4096, Context3DTextureFormat.COMPRESSED_ALPHA, false);
texture.uploadCompressedTextureFromByteArray (atfData, 0);
var img1:BitmapData = BitmapData.fromTexture (texture);
graphics.beginBitmapFill(img1);
graphics.drawRect(0, 0, 100, 100);
graphics.endFill();

Based on the current implementation, this appears to be the expected behavior.
As far as I can tell, BitmapData.fromTexture() is intended to be a way to make it possible to render the texture in an openfl.display.Bitmap only (and only when rendering OpenFL on the GPU). I would expect that calling most of the common methods on the BitmapData class will probably fail with a similar exception because the texture is never converted from a hardware format to a software format. (It's worth mentioning that BitmapData.fromTexture() does not exist in Flash, and that's probably for good reason because converting hardware textures to software bitmaps is very expensive, performance-wise).
BitmapData.fromTexture() has the following code:
var bitmapData = new BitmapData(texture.__width, texture.__height, true, 0);
bitmapData.readable = false;
bitmapData.__texture = texture;
bitmapData.__textureContext = texture.__textureContext;
bitmapData.image = null;

Notice, in particular, that the image private variable is set to null.  That's where the raw software-renderable pixel data is supposed to be stored normally. Only the hardware texture is saved, and it isn't converted for software. However, a number of important operations happen in software. For instance, when you use beginBitmapFill(), it ends up calling ImageCanvasUtil.convertToCanvas(bitmap.image);. But image is null. With normal BitmapData it wouldn't be null.
Basically, in Flash, ATF textures weren't intended for use on the classic display list. They were meant for Stage 3D. OpenFL happens to render the classic display list using Stage 3D by default, which is pretty clever, but OpenFL has not provided a way to make ATF textures work on the classic display list as completely as PNG/JPG/etc. image formats can be.

I added some details to the documentation for BitmapData.fromTexture() (including specifying that Graphics.beginBitmapFill() is one of the operations that does not work).
I also stopped the exception from being thrown when calling Graphics.beginBitmapFill() and bitmapData.readable == false. Instead, a hardware-only texture will render as plain black when passed to Graphics.beginBitmapFill(). So it won't render the texture, for the reasons specified, but it will prevent the app from crashing.
It also makes native targets more consistent with html5 because those previously failed silently without rendering anything.

Thanks for your answer.

It also makes native targets more consistent with html5 because those previously failed silently without rendering anything.

No. It renders actually. But crashes on mouse over. I don't know what the convertToCanvas is doing. But in case it's for delimit the bounds of the sprite, we could get/build that information when calling drawRect or drawTriangles. I don't get why it calls that from a hitTest... At least, it would be preferable to stop the exception from being thrown from there (in convertToCanvas instead of displaying a full back sprite. In that case, as an user, I could use the differ library to mimic hitTest.

As far as I can tell, BitmapData.fromTexture() is intended to be a way to make it possible to render the texture in an openfl.display.Bitmap only

No we can use it in Tilemap actually.
I came across this bug during some tests I made. Drawing 464 sprites with Tilemap on html5 takes 275ms. With my polygon library that uses drawTriangles, it takes 178ms and with ATF it takes 45ms. So I'm afraid to lose something, especially because Stage3D looks less convienent and uses Agal instead of GLSL if I'm correct.


Balancer v2 pools

Support all required operations

[ ] swaping
[ ] creating pools
[ ] add/remove liquidity

I understood we won't work with balancer v2 after all. Is this still a valid ticket?


Error in CLI with Vue support

Describe the bug
A clear and concise description of what the bug is.
Quick start with Storybook with Vue supported error.
To Reproduce
Steps to reproduce the behavior:

run npx -p @storybook/cli sb init
Choose Vue
See error

     TypeError: Cannot create property 'dependencies' on boolean 'false'
    at _callee$ (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/generators/VUE/index.js:33:3)
    at call (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/node_modules/regenerator-runtime/runtime.js:62:40)
    at Generator.tryCatch [as _invoke] (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/node_modules/regenerator-runtime/runtime.js:288:22)
    at Generator._invoke [as next] (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/node_modules/regenerator-runtime/runtime.js:114:21)
    at asyncGeneratorStep (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/generators/VUE/index.js:34:103)
    at _next (/Users/kamontat/.npm/_npx/94048/lib/node_modules/@storybook/cli/generators/VUE/index.js:36:194)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Expected behavior
Create project
Screenshots
If applicable, add screenshots to help explain your problem.

Code snippets
If applicable, add code samples to help explain your problem.
System:

OS: [e.g. iOS, Windows10, MacOS] MacOS
Device: [e.g. iPhoneX, Macbook Pro 2018] Macbook Pro 2015
Browser: [e.g. chrome, safari] chrome
Framework: [e.g. react, vue, angular] vue
Addons: [if relevant]
Version: [e.g. 4.0.0] cli version: 6.4.1

Additional context
Add any other context about the problem here.

you need to prepare package.json first

I see.

You might need to add this into documentation.

Can someone tell me the exact steps to setup storybooks

@Lahirutech creating a package.json file and filling it with the following worked for me:
{
  "scripts": {
    "storybook": "start-storybook"
  }
}

1 - npm init
2 - npx -p @storybook/cli sb init --type vue

cc @ndelangen for #6002

+1 just had to search to resolve this.  It would be really nice if the error said, "Do you have a package.json?"

Can I delete the package.json after installing Storybook? I only made one to satisfy the install requirements, but I already have a main package.json in the parent folder, separate from the React-specific stuff.

@anevaude Can you just install it in the parent directory?

@anevaude Can you just install it in the parent directory?

@shilman - Oh yep! That's no problem at all. I just didn't know if it was "proper" to put it in the app directory itself. Some guides make it sound like that's the right way to go about things. :)


Do not add duplicate LB services to avoid sending multiple events to same target

Sending event to same target for same resourceID causes the event framework to not acknowledge the replies coming for those multiple events.
https://github.com/rancher/rancher/issues/10005
@cjellick pls. review

TF's are not related to the commit.

@cjellick updated the PR, pls review.

LGTM


feat(36): added svg to icon library + added function to create icons …

…as fonts
By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Creating svg font additionally to existing build output

📦 Published PR as canary version: 3.0.1--canary.37.92aa521ad79b6a7aa90038b1f4149113e9346496.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

No Icon-Font in dist folder of canary release, can you check?

 No Icon-Font in dist folder of canary release, can you check?

The build process creates a separate folder, called generated_fonts. Should everything be built into /dist?


[Feat] If possible, you can proxy celebrities imgs

Not necessary, but thanks.

I haven't used celebrities, maybe you can tell me how to get the picture and I can help implement it

A rust API of douban:

https://github.com/eallion/douban-api-rs/blob/master/src/api.rs#L238

Done~

https://dou.img.lithub.cc/celebrity/1041028.jpg
https://dou.img.lithub.cc/celebrity/1041028.jpg

Great!


api.connect to start sync with retry: true

On api.connect we currently do api.sync with live: true,
but without retry: true, which means it doesn't try to reconnect
when for example a connection gets lost, see http://pouchdb.com/api.html#replication
That PR fixes that

Looks good. :+1:
So now it will keep trying to reconnect if the connection gets lost?

So now it will keep trying to reconnect if the connection gets lost?

yes, exactly, that's what hoodie does today as well, POuchDB is just smarter about it :)

Note that this could potentially be an issue for battery performance so might be worth thinking about a 'quiet' mode or documenting how to disable automatic retries.

@nickcolley yeah, I know, definitely room for improvement. But I love the PouchDB gives us all the APIs for very fine grained control. For now, you can do store.disconnect(), e.g. if you have battery APIs, you could do that when battery is low, but I think this is app-specific. We'll see :) Exciting times I think


cannot create new project with sdk

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I am attempting to create a new blank project with the cvat_sdk. The api client raises an ApiType error
Steps to Reproduce (for bugs)

from cvat_sdk.api_client import Configuration, ApiClient, exceptions
from cvat_sdk.api_client.model.project_read import ProjectRead
from cvat_sdk.api_client.model.project_write_request import ProjectWriteRequest
from pprint import pprint
Set up an API client
Read Configuration class docs for more info about parameters and authentication methods
configuration = Configuration(
host = "http://localhost:8080",
username = 'django',
password = 'password',
)
api_client = ApiClient(configuration)
project_write_request = ProjectWriteRequest(
name="name_example",
labels=[],
)
(data, response) = api_client.projects_api.create(
project_write_request,
)
Expected Behaviour
This should post a new project with name.
Current Behaviour
cvat_sdk.api_client.exceptions.ApiTypeError: Invalid type for variable 'tasks'. Required value type is list and passed type was str at ['received_data']['tasks']
Possible Solution
Not sure
Context
I am trying to create a project using the sdk api and then assign tasks to this project.

@SpecLad , could you please look at the issue?

Most likely, it's a mismatch between the server and SDK versions. Which versions are you using?

I will close the issue. Don't hesitate to reopen it if you can provide more information.


VSCE: Replace "New Chat" icon

context: https://sourcegraph.slack.com/archives/C05AGQYD528/p1699402968832279
Add this svg to cody-icons.woff and replace the current "new chat" icon with the new one.
Tim:

The instructions are in the readme there, let me know if that works!

Also: it’d be nice to hide the Cody logo button in the chat window editor title. So it would be Restart Chat, New Chat, Chat History if we keep this new restart button.

Here is the current new chat icon:

Thanks for looking into this @iswarmondal !
Basically, we want to replace the current "New Chat" icon ("icon": "$(comment)") for the cody.chat.panel.new command with the icon mentioned in https://github.com/sourcegraph/cody/pull/1560#issuecomment-1788464054
But the new icon is not built-in, so we will need to convert the SVG image into text but adding it into the vscode/resources/cody-icons.woff file first before we can use it in the package.json. You can find the instructions on how to convert it into font in this README.md
. Let me know if you have other questions 😄

On it

This is my progress so far. I was able to indirectly edit the .woff file.
Let me know if this is what we need.

@iswarmondal yes, that's what the task is about, thank you so much for the quick turnaround!
The image looks out of place to me compare to the nearby icons. Cc @toolmantim who requested this change - does this look alright to you?

Thanks @abeatrix, after hearing from @toolmantim and fixing the alignment of the icon I'll create a PR for this

Thanks @iswarmondal! How did you generate the woff there? It appears the baseline is incorrect for both the new icon, but also the Cody icon seems to have changed too.

You'll need to follow https://github.com/sourcegraph/cody/tree/main/lib/icons, not the README.md linked above. I'd recommend we skip some letters in case we need more variants of the Cody logo… so go with "H" for this new one.

Hey @toolmantim, I converted the WOFF file into a TTF file and then edited that TTF file, then converting back the edited TTF file.
So as you said, I have to add the SVG icon with the name of H.svg inside svg-origional table and regenerate the icons using pnpm run font command.
This will generate me a new WOFF file which we will be using (Also need to update the MD file).
Is that right?

@iswarmondal yep, that's it! I believe it's the svg-outlined folder that's used, the originals are just for reference, but I'd add it to both directories just for consistency.

On it...

Here's the updated icon

I have raised a PR for this #1727
@abeatrix  @toolmantim please take a look

@iswarmondal thank you! Will review the PR first thing tomorrow!


Update for -v option.

I noticed the difference between README and the man page.
Additionally, I think that -v has no option.
Thank you.

Merged, thanks!


⚠ Bofrost.lu has degraded performance

In 84f7e26, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2704 ms

Resolved: Bofrost.lu performance has improved in a472949 after 42 minutes.


Fancy overly long lines command.

Detect overlong lines in technicolor, noting files and line numbers, not just grepping the lines.


Ensure writing works with partition_freq

Test cases with partition_freq.
Closes https://github.com/dask-contrib/dask-deltatable/issues/32.

Codecov Report

Merging #44 (e55af4f) into main (9adc684) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   76.17%   76.17%
=======================================
  Files           6        6
  Lines         340      340
=======================================
  Hits          259      259
  Misses         81       81

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Explicitly use UTF-8 codepage when using MSVC

One of C++ tests contain non-ASCII letters:
https://github.com/dmlc/xgboost/blob/8cbcc53ccbd3d2c4c74ef35fcec018be64d084cd/tests/cpp/common/test_config.cc#L154-L156
Microsoft Visual C++ (MSVC) uses the system codepage by default when compiling C++ sources. This resulted in error Error C2001: Newline in constant when I compiled XGBoost on my Windows laptop, which had Korean (CP949) locale.
This PR explicitly sets the compilation flag /utf-8 so that MSVC always uses UTF-8 encoding. See related discussion at https://github.com/libusb/libusb/issues/207.

Consider using u8  string prefix.

@trivialfis Using u8 prefix alone is insufficient, because MSVC will try to parse the source file using the system codepage unless /utf-8 is specified. This will result in error on some locales, such as Korean (CP949).

Makefile is also related if we rely on build system.

No, since the problem is limited to Visual Studio on Windows, for which we exclusively use CMake already. GCC and other compilers do not suffer from this issue.

https://docs.microsoft.com/en-us/cpp/build/reference/utf-8-set-source-and-executable-character-sets-to-utf-8?view=vs-2019


user-docs: Update use of save-changes macro.

The user documentation utilizes a save-changes macro that adds a step to click on a 'Save changes' button after some setting has been changed.
Update articles that describe cases where the application now automatically saves changes instead of presenting a button.

@zulipbot label "area: documentation (user)"

@rishig @Lyla-Fischer can you folks review this?

There are a bunch of places where we have save-changes.md in the docs, but for which we have autosave in the actual product. This pull request is progress on getting some of those docs updated, and therefore I think it would be worthwhile to just merge it, but a more complete pull request would go through all of the save-changes.md places and check to see if we are using autosave there now.

Great, will merge now then. Thanks @jackrzhang and @Lyla-Fischer!

@Lyla-Fischer I searched through and manually checked all instances of save-changes.md when making this pull request, so I think this should be comprehensive in that sense.
That being said, another set of eyes to double-check can't hurt :)

We have autosave for basically everything that isn't "dangerous" enough that we want to have a confirmation prompt.


chore: add platforms to hakari

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Info here https://github.com/risingwavelabs/risingwave/issues/12315

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

headache

Looks like a Linux-specific problem because the build succeeded on my Mac.


Correct stress/v2 tests to not depend on $GOPATH

The test assumed that you were working within a single directory
$GOPATH. go test actually sets the current directory to the directory
of the package being tested, so the relative path to the test file could
be used instead.
/cc @jackzampolin

LGTM 👍


⚠ Bofrost.fr has degraded performance

In 734e252, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1607 ms

Resolved: Bofrost.fr performance has improved in d2f4fb3 after 1 day, 5 minutes.


Migration to flutter 3.

This PR did not pass but it is a valid migration to flutter 3:
https://github.com/qiuxiang/cross_file_image/pull/1/files

resolve in v0.0.2

@qiuxiang You forgot to add flutter 3.0.0 in the pubspec, it's going to break existing apps if you do it without doing that.
A breaking change is also a good place to increment the version (v1.0.0)

i dont think it needs to add flutter 3.0.0 in the pubspec. i have upgraded flutter to v3 and this package works fine.

No it does not work fine

i dont think it needs to add flutter 3.0.0 in the pubspec

Yes you have to , else it's a breaking change if you correct the above warning


Fundamentals Week 1: Importance of Professional Development section is confusing about "boards"

On https://cyf-curriculum.netlify.app/fundamentals/sprints/1/day-plan/ (content in https://github.com/CodeYourFuture/CYF-PD/blob/main/content/blocks/fundamentals-test/readme.md), the references to "board"s is a bit confusing.
In general we've been referring to "boards" to mean people's planning boards. I'm not sure from reading this whether the expectation is that trainees will end up with issues on their planning boards, or whether "boards" here is referring to the interactive whiteboard used in the online variant of this class (if so, the idea of using multiple boards wasn't really discussed).
Could we reword to make the following sentences more clear:

"Make sure you are adding at least one ticket for each board." (what tickets / boards is this referring to)
"Brainstorm the characteristics of the “Worst Boss” and add tickets to the board." (I think this is referring to the paper/whiteboard?)
"Now think about the characteristics of the best boss you ever had or you would want to have and add them to the board." (as above)
"Look at the Best Boss circle and group the ‘hard skills’ (technical skills) and the ‘soft skills’ (professional skills) on the board." (as above, and particularly, if we're doing this on paper, how? Maybe we should reframe the wording to talk about using post-its (which may be digital or physical)?

/cc @kfklein15 @esma-g

I don't know how to edit the new curriculum yet, so I'm rewriting the above sentences in case it helps the editor:

Make sure you have at least one contribution during each part of the exercise
Think about the characteristics of the "Worst Boss", write your ideas on post-its and share them on the collaborative sheet
Now think about the characteristics of the "Best Boss" you have ever had or you would want to have, write your ideas on post-its and share them on the collaborative sheet
Work with your cohort to group the ‘hard skills’ (technical skills) and the ‘soft skills’ (professional skills) shared for the "Best Boss". You can move the post-its under these two headings on a separate sheet

Thank! I'll quickly make those edits, and hopefully everyone will know how to edit things soon!

Let me know if I need to do anything on this or if it has been done, please :)


Update SQLite to version 3.46.1

This is an automated pull request, updating SQLite to version 3.46.1.

This needs to be regenerated after #1239 is merged.


How to add like of current card on right and dislike on left?

Hello!
I am developing an app using swipe card library and I want to store like and dislikes in data base so tell me a method how to add cards as a like on right and dislike on left?

When I swipe right I want to add that card as like how to do that kindly help me.
Thanks

This is personal implementation and not part of this library.


add data channel again, with edits

LGTM, modulo question of whether it would be easier to verify in scada without node name format changes. But we'll have to verify those at some point regardless.


Update application.conf

Changing tie breaker default from 0 to 0.5. This helps with addresses with number and suffix ranges. The SAO test dataset went from 97 to 13 false positives.

Does it work good on the edge cases?

Good question - yes, it does work on EdgeCases. It reduces the number of false positives by 7, which isn't as much as one would hope, but at least it improves things. I will be running other datasets soon.


Issue with the loss of Vendor Specific Attributes when OTPs are used

I know that this is a documented issue in the article without a known work around.  However I've written a script that will allow for this functionality.  We have seen this be a major road block for customers who leverage these attributes to assign certain groups to specific VLANs, for example.  There are other use cases for the scripted solution, but they are even more niche.  However the use of VSAs for VLAN assignment is very mainstream and are used by organizations that have a big investment in their NPS infrastructure.
My script will allow for the use of OTPs while still maintaining the use of the VSAs, but it will require moving the functionality from the network policies to Connection Request Policies.  In short the script will enumerate all the users in a group, or groups, and populate the Username condition in the CRP with those names, so you get as psuedo Group Membership condition in a CRP.
Please find it here:  https://github.com/OneMoreNate/CrpUsernameStuffing
As so many customers have experienced this issue I would appreciate any feedback on the script and perhaps a reference to it in the document, as we've helped many customers who otherwise would not have been able to deploy the MFA Extension.

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

ID: a61a7e9a-44bf-7bdc-f810-87e2fd6792e1
Version Independent ID: 71d11105-b83e-952e-d107-1ff9a7c8d107
Content: Use Azure AD Multi-Factor Authentication with NPS - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@OneMoreNate The Github feedbacks are associated with particular documentation. Please provide the link to correct document so we can route it correctly.
For any other question, please ask.

It deals with this document at the location shared below:
https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-nps-extension#determine-which-authentication-methods-your-users-can-use

@OneMoreNate sorry for delay, I added a link to the script.

#please-close


Add new class 'b vitamins' to ChEBI

I would like to request that a new Class ‘B vitamins’ be added to the ChEBI database.  This class would include as subclasses:
Cobalamin       CHEBI:30411
Biotin          CHEBI:15956
Pyridoxine      CHEBI:16709
Riboflavin      CHEBI:17015
Pantothenic acid    CHEBI:7916
Folic acid      CHEBI:27470
Nicotinic acid      CHEBI:15940
Thiamine        CHEBI:26948
This is a common mixture of vitamins used to grow microorganisms.  Reference: Wikipedia:B_vitamins.  Proposed Outgoing: ‘B vitamins’ is_a ‘water-soluble vitamin’
-Carrine
Reported by: carrineblank

But what about adding the subclasses listed in the definition?  Any of the group of eight water-soluble vitamins ... vitamin B1, B2, B3, B5, B6, B7, B9, and B12. ...

Hi Damion,
As far as I can see, these entities are already present in ChEBI. If you go to CHEBI:75769 (B vitamin; synonyms B vitamins; B group vitamin; vitamin B) and scroll down below the Wikipedia section to the ChEBI Ontology section, you will find them listed (and linked to) in the "Incoming" section. Admittedly

Admittedly the linked entries all have "chemical" names as their display names, but the "vitamin" names (B1, B2, etc.) are present as synonyms.
Hope this helps!
Gareth

I'm looking at:
https://www.ebi.ac.uk/ols/ontologies/chebi/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FCHEBI_75769
In it I see various ROLE axioms - but they are pointing to material entities.  Can't have it both ways.  Either B1, B2, B3, B5, B6, B7, B9, and B12 must be moved to role (and renamed with "role" suffix). Or we consider them primarily molecular entities, and create a "B Vitamin" molecular entity group with those as subclasses.

I prefer the latter, and others can get around to defining roles and for example what it takes to be a B1 vitamin role.

It is always a problem with vitamins - the names can be used to mean both structure and role. The problem with treating them as entities comes in defining the structural features that they (and only they) can have...

Well, I think the suffix "role" should be added if entity has role semantics.  Can't have the entity be both role and material entity. It will break reasoning into the future.


Introduce data query / mutation hook factories

Uncertain if this is the way to go or if we should just continue with useFunction

Done but not sure if I'll keep it (exclusively).

Done but not sure if I'll keep it (exclusively).


[Bug]: RepoDB should read-lock imageStore before using GetIndexContent() and GetBlobContent()

zot version
main
Describe the bug
example: https://github.com/project-zot/zot/blob/main/pkg/meta/repodb/storage_parsing.go#L236
should look like this: https://github.com/project-zot/zot/blob/main/pkg/extensions/sync/utils.go#L315
To reproduce

Configuration
Client tool used
Seen error

Expected behavior
No response
Screenshots
No response
Additional context
No response

@peusebiu  Can i work for it? It's seems like just need create a time.Time to use for lock and use is around imageStore.GetBlobContent.

Helo Lian, yes that's right, just a time.Time and locks around GetBlobContent and GetIndexContent
There is nothing to reproduce:D, without this change it will race sometime.
The imagestore can be write locked while we read it.


Add ctpn?

I assume you are referring to this paper. I'm not able to commit to working on it. But, if you would like to add an alternative detector class that uses CTPN, I think it could be a great addition to the package. I will close this issue now but if you want to work on it, please post back and I'll re-open. Thanks!


Moved the ready state information from the beta section to the job section.

fixes #44632
As suggested in issue #44632, ready pod status is still listed as a beta feature and it should moved under the job section.
I moved the content to that section.

As noted. updating the doc is a 2 step process. In https://github.com/kubernetes/website/issues/44632, alculquicondor indicated the code changes were made in https://github.com/kubernetes/kubernetes/pull/121765. I believe this first step is associated with this doc, Contributing to the Upstream Kubernetes Code. The second step is Generating Reference Documentation for the Kubernetes API. I believe it is this second step that folks have indicated needs to be performed.

As @sftim mentioned, the edited file in this PR is generated during the reference doc generation. The change proposed in this PR will be overwritten the next time the reference docs are generated.


refactor: Drop unneeded Q_DECLARE_METATYPE

Ported from https://github.com/bitcoin/bitcoin/pull/19108.
For typedefs, which is CAmount, qRegisterMetaType(const char*) only is required.
No behavior change.

bitcoin/bitcoin#19108 has ACK from @promag.

Updated 9657a8661be3e79a579b38893d58fc0a06f56bd9 -> 7c944086b3c62f9833fa9fbdbda2d5d4b34b37e9 (pr61.01 -> pr61.02, diff):

addressed @promag's comment:

nit, could mention env QT_FATAL_WARNINGS=1 as a way to identify missing registrations.

There was unanswered comment from @luke-jr:

Is not needing it for typedefs considered defined behaviour?

I think this excerpt from Qt docs is an answer:

This [qRegisterMetaType] function is useful only for registering an alias (typedef)...

If there's no behaviour change, I'd figure just leave it alone so things don't subtly break if the type of CAmount were to ever be changed. Kinda the point of the typedef anyway is that you don't need to care so much what the underlying type is...

If there's no behaviour change, I'd figure just leave it alone so things don't subtly break if the type of CAmount were to ever be changed. Kinda the point of the typedef anyway is that you don't need to care so much what the underlying type is...

Closing.

if the type of CAmount were to ever be changed

Ok 😅


Fix SSR entering infinite loop when multiple query hooks update

Fix #457
This bug can be replicated by:

Having a useQuery hook in a component
Either: having a second with the same operation or another hook in the
component
Triggering an update causing the component to rerender

This will cause the component to rerender in SSR.
The useCallback is a noop so some dependencies will trigger,
and this will subsequently trigger another update, which then
enters an infinite loop.
This is because useCallback is a noop in SSR: https://github.com/facebook/react/blob/0f64703edf5970b12a878ea3b5e1e30ef1d71c74/packages/react-dom/src/server/ReactPartialRendererHooks.js#L448-L454

Wow that's wild. I was actually hitting this yesterday working on a small NextJS app and found that changing the dependency array arbitrarily on a useCallback didn't appear to be doing anything. Great find!

@parkerziegler it really is and really was hard to pin down. I'm totally aware that useMemo and useCallback make no strong guarantees, but I was totally expecting useCallback to still work in most simple cases.

:thonking-hard: 😅


create answer edit view for students

View for students to answer questions generated by graders

should contain:

single question and editable answer view
submit button

requirements

action to go to the next question
questions are generated from the grader view (see #3)
submit store in the database

repo migrated to https://github.com/DigiKlausur/ccm_components


Keep all the memory metrics in the seed prometheus

How to categorize this PR?

/area monitoring
/kind enhancement
What this PR does / why we need it:
Keep all the memory metrics in the seed and shoot prometheus
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
The meminfo collector of the node exporter exposes
the fields of the /proc/meminfo as metrics.
See https://github.com/prometheus/node_exporter/blob/c6e1a5b74277557045f0620e6d37259a291cb03b/collector/meminfo_linux.go#L33-L75
Previously, we kept a single metric:

 Active: Memory that has been used more recently and usually not
         reclaimed unless absolutely necessary.

See https://www.kernel.org/doc/Documentation/filesystems/proc.txt
and we noticed that the other fields could be also useful to
understand the memory usage of a node.
We shall list all the metrics in the allow list individually for consistency,
but for now I think using a regular expression is more concise
and reflects the intent to capture all of them.

List of the 50 metrics with documentation
$ curl -s localhost:16909/metrics | grep '# HELP node_memory' | nl
     1	# HELP node_memory_Active_anon_bytes Memory information field Active_anon_bytes.
     2	# HELP node_memory_Active_bytes Memory information field Active_bytes.
     3	# HELP node_memory_Active_file_bytes Memory information field Active_file_bytes.
     4	# HELP node_memory_AnonHugePages_bytes Memory information field AnonHugePages_bytes.
     5	# HELP node_memory_AnonPages_bytes Memory information field AnonPages_bytes.
     6	# HELP node_memory_Bounce_bytes Memory information field Bounce_bytes.
     7	# HELP node_memory_Buffers_bytes Memory information field Buffers_bytes.
     8	# HELP node_memory_Cached_bytes Memory information field Cached_bytes.
     9	# HELP node_memory_CommitLimit_bytes Memory information field CommitLimit_bytes.
    10	# HELP node_memory_Committed_AS_bytes Memory information field Committed_AS_bytes.
    11	# HELP node_memory_DirectMap1G_bytes Memory information field DirectMap1G_bytes.
    12	# HELP node_memory_DirectMap2M_bytes Memory information field DirectMap2M_bytes.
    13	# HELP node_memory_DirectMap4k_bytes Memory information field DirectMap4k_bytes.
    14	# HELP node_memory_Dirty_bytes Memory information field Dirty_bytes.
    15	# HELP node_memory_FileHugePages_bytes Memory information field FileHugePages_bytes.
    16	# HELP node_memory_FilePmdMapped_bytes Memory information field FilePmdMapped_bytes.
    17	# HELP node_memory_HugePages_Free Memory information field HugePages_Free.
    18	# HELP node_memory_HugePages_Rsvd Memory information field HugePages_Rsvd.
    19	# HELP node_memory_HugePages_Surp Memory information field HugePages_Surp.
    20	# HELP node_memory_HugePages_Total Memory information field HugePages_Total.
    21	# HELP node_memory_Hugepagesize_bytes Memory information field Hugepagesize_bytes.
    22	# HELP node_memory_Hugetlb_bytes Memory information field Hugetlb_bytes.
    23	# HELP node_memory_Inactive_anon_bytes Memory information field Inactive_anon_bytes.
    24	# HELP node_memory_Inactive_bytes Memory information field Inactive_bytes.
    25	# HELP node_memory_Inactive_file_bytes Memory information field Inactive_file_bytes.
    26	# HELP node_memory_KReclaimable_bytes Memory information field KReclaimable_bytes.
    27	# HELP node_memory_KernelStack_bytes Memory information field KernelStack_bytes.
    28	# HELP node_memory_Mapped_bytes Memory information field Mapped_bytes.
    29	# HELP node_memory_MemAvailable_bytes Memory information field MemAvailable_bytes.
    30	# HELP node_memory_MemFree_bytes Memory information field MemFree_bytes.
    31	# HELP node_memory_MemTotal_bytes Memory information field MemTotal_bytes.
    32	# HELP node_memory_Mlocked_bytes Memory information field Mlocked_bytes.
    33	# HELP node_memory_NFS_Unstable_bytes Memory information field NFS_Unstable_bytes.
    34	# HELP node_memory_PageTables_bytes Memory information field PageTables_bytes.
    35	# HELP node_memory_Percpu_bytes Memory information field Percpu_bytes.
    36	# HELP node_memory_SReclaimable_bytes Memory information field SReclaimable_bytes.
    37	# HELP node_memory_SUnreclaim_bytes Memory information field SUnreclaim_bytes.
    38	# HELP node_memory_ShmemHugePages_bytes Memory information field ShmemHugePages_bytes.
    39	# HELP node_memory_ShmemPmdMapped_bytes Memory information field ShmemPmdMapped_bytes.
    40	# HELP node_memory_Shmem_bytes Memory information field Shmem_bytes.
    41	# HELP node_memory_Slab_bytes Memory information field Slab_bytes.
    42	# HELP node_memory_SwapCached_bytes Memory information field SwapCached_bytes.
    43	# HELP node_memory_SwapFree_bytes Memory information field SwapFree_bytes.
    44	# HELP node_memory_SwapTotal_bytes Memory information field SwapTotal_bytes.
    45	# HELP node_memory_Unevictable_bytes Memory information field Unevictable_bytes.
    46	# HELP node_memory_VmallocChunk_bytes Memory information field VmallocChunk_bytes.
    47	# HELP node_memory_VmallocTotal_bytes Memory information field VmallocTotal_bytes.
    48	# HELP node_memory_VmallocUsed_bytes Memory information field VmallocUsed_bytes.
    49	# HELP node_memory_WritebackTmp_bytes Memory information field WritebackTmp_bytes.
    50	# HELP node_memory_Writeback_bytes Memory information field Writeback_bytes.

    MemTotal: Total usable ram (i.e. physical ram minus a few reserved
              bits and the kernel binary code)
     MemFree: The sum of LowFree+HighFree
MemAvailable: An estimate of how much memory is available for starting new
              applications, without swapping. Calculated from MemFree,
              SReclaimable, the size of the file LRU lists, and the low
              watermarks in each zone.
              The estimate takes into account that the system needs some
              page cache to function well, and that not all reclaimable
              slab will be reclaimable, due to items being in use. The
              impact of those factors will vary from system to system.
     Buffers: Relatively temporary storage for raw disk blocks
              shouldn't get tremendously large (20MB or so)
      Cached: in-memory cache for files read from the disk (the
              pagecache).  Doesn't include SwapCached
  SwapCached: Memory that once was swapped out, is swapped back in but
              still also is in the swapfile (if memory is needed it
              doesn't need to be swapped out AGAIN because it is already
              in the swapfile. This saves I/O)
      Active: Memory that has been used more recently and usually not
              reclaimed unless absolutely necessary.
    Inactive: Memory which has been less recently used.  It is more
              eligible to be reclaimed for other purposes
   HighTotal:
    HighFree: Highmem is all memory above ~860MB of physical memory
              Highmem areas are for use by userspace programs, or
              for the pagecache.  The kernel must use tricks to access
              this memory, making it slower to access than lowmem.
    LowTotal:
     LowFree: Lowmem is memory which can be used for everything that
              highmem can be used for, but it is also available for the
              kernel's use for its own data structures.  Among many
              other things, it is where everything from the Slab is
              allocated.  Bad things happen when you're out of lowmem.
   SwapTotal: total amount of swap space available
    SwapFree: Memory which has been evicted from RAM, and is temporarily
              on the disk
       Dirty: Memory which is waiting to get written back to the disk
   Writeback: Memory which is actively being written back to the disk
   AnonPages: Non-file backed pages mapped into userspace page tables
HardwareCorrupted: The amount of RAM/memory in KB, the kernel identifies as
	      corrupted.
AnonHugePages: Non-file backed huge pages mapped into userspace page tables
      Mapped: files which have been mmaped, such as libraries
       Shmem: Total memory used by shared memory (shmem) and tmpfs
ShmemHugePages: Memory used by shared memory (shmem) and tmpfs allocated
              with huge pages
ShmemPmdMapped: Shared memory mapped into userspace with huge pages
KReclaimable: Kernel allocations that the kernel will attempt to reclaim
              under memory pressure. Includes SReclaimable (below), and other
              direct allocations with a shrinker.
        Slab: in-kernel data structures cache
SReclaimable: Part of Slab, that might be reclaimed, such as caches
  SUnreclaim: Part of Slab, that cannot be reclaimed on memory pressure
  PageTables: amount of memory dedicated to the lowest level of page
              tables.
NFS_Unstable: NFS pages sent to the server, but not yet committed to stable
	      storage
      Bounce: Memory used for block device "bounce buffers"
WritebackTmp: Memory used by FUSE for temporary writeback buffers
 CommitLimit: Based on the overcommit ratio ('vm.overcommit_ratio'),
              this is the total amount of  memory currently available to
              be allocated on the system. This limit is only adhered to
              if strict overcommit accounting is enabled (mode 2 in
              'vm.overcommit_memory').
              The CommitLimit is calculated with the following formula:
              CommitLimit = ([total RAM pages] - [total huge TLB pages]) *
                             overcommit_ratio / 100 + [total swap pages]
              For example, on a system with 1G of physical RAM and 7G
              of swap with a `vm.overcommit_ratio` of 30 it would
              yield a CommitLimit of 7.3G.
              For more details, see the memory overcommit documentation
              in vm/overcommit-accounting.
Committed_AS: The amount of memory presently allocated on the system.
              The committed memory is a sum of all of the memory which
              has been allocated by processes, even if it has not been
              "used" by them as of yet. A process which malloc()'s 1G
              of memory, but only touches 300M of it will show up as
	      using 1G. This 1G is memory which has been "committed" to
              by the VM and can be used at any time by the allocating
              application. With strict overcommit enabled on the system
              (mode 2 in 'vm.overcommit_memory'),allocations which would
              exceed the CommitLimit (detailed above) will not be permitted.
              This is useful if one needs to guarantee that processes will
              not fail due to lack of memory once that memory has been
              successfully allocated.
VmallocTotal: total size of vmalloc memory area
 VmallocUsed: amount of vmalloc area which is used
VmallocChunk: largest contiguous block of vmalloc area which is free
      Percpu: Memory allocated to the percpu allocator used to back percpu
              allocations. This stat excludes the cost of metadata.

If we gain more experience which of these metrics are useful,
we shall replace the regular expression and list them explicitly.
The seed prometheus is the one in the garden ns.
The shoot prometheus is in the control plane.
https://gardener.cloud/docs/gardener/monitoring/
Release note:

Keep all the memory metrics in the seed prometheus

/approve


fix(datagrid): adds a11y attributes to datagrid filter button

In the conversion to the new popover utility a11y attributes were missed. This adds: aria-label, aria-expanded and aria-controls
to the datagrid filter button. This change closes #3985.
Signed-off-by: Matt Hippely<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
Datagrid filter buttons were missing aria-label, aria-controls and aria-expanded attributes on the toggle button.

Issue Number: #3985
What is the new behavior?
Datagrid filter buttons have aria-label, aria-controls and aria-expanded attributes on the toggle button.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Fix. verified on https://clarity.design/documentation/datagrid/custom-filtering


Support an option to disable-direct-write

VSCode Version: 1.1.1
OS Version: Windows 10

I'm passing the --disable-direct-write command-line option to VSCode to work around the blurry font issue on Windows. This works fine, but every time VSCode updates, the installer overwrites my changes to the VSCode start menu shortcut and registry setting. It would be great if I could put the --disable-direct-write option somewhere in my VSCode settings that doesn't get overwritten when a new update is installed.

One workaround is to copy the shortcut and use that one instead.

@bpasero BrowserWindows seem to support passing this option directWrite on creation. Can we make a setting out of this?

Instead of another setting, how about a generic command line args setting? --disable-gpu is another common one people want.

That's what the user already has.

settings.json:
{
  "window.commandLineArgs": [
    "--disable-direct-write",
    "--disable-gpu"
  ]
}

@Tyriar is that setting already available in the software, or is that a proposed solution?

@RandScullard just a suggestion

:thumbsup:

Oh I see, I misread your suggestion, sorry! Might be nice, yeha.

Closing, Chrome 52 decided to remove this option. direct write is always on from then on.

--disable-directwrite-for-ui source

@eladkarako This is not work for me

@foisonocean
// found in the LICENSE file.

#include "build/build_config.h"
#include "ui/gfx/switches.h"

namespace switches {

#if defined(OS_WIN)
// Disables DirectWrite font rendering for general UI elements.
const char kDisableDirectWriteForUI[] = "disable-directwrite-for-ui";
#endif

#if defined(OS_MACOSX)
// Enables the HarfBuzz port of RenderText on Mac (it's already used only for
// text editing; this enables it for everything else).
const char kEnableHarfBuzzRenderText[] = "enable-harfbuzz-rendertext";
#endif

only works on Windows, try --enable-harfbuzz-rendertext or --disable-harfbuzz-rendertext if you're on a different OS.


[FEATURE] A new window for GPT-Prompter

Hi. I have a couple of thoughts on improving this extension.

Customizing the appearance of the window. Changing the font size and adjusting the light/dark theme.
To have a handle to move instead of clicking the whole window. I suggest a bar on the left that you can hold on and move the window, or on the top.

Also option to turn off the typewriter animation would be good.

Also option to turn off the typewriter animation would be good.

The typewriter animation is not actually intended as animation. is the option "stream" from the gpt API to get the answer as fast as possible, getting each token as soon as it is ready, similar to what is done in the Playground of OpenAI. The alternative would be to wait for the entire answer to be generated and then you would receive the whole message back. This could take several seconds for  long answer. The very first version was like this, and often times while waiting I was wondering if it actually was working

Hi. I have a couple of thoughts on improving this extension.

Customizing the appearance of the window. Changing the font size and adjusting the light/dark theme.
To have a handle to move instead of clicking the whole window. I suggest a bar on the left that you can hold on and move the window, or on the top.

I ll add in the near future a page with "advanced option" to add features like you suggested. For now I changed the font size to be inherited from the page, rather than fixed. In the next days the new version <IP_ADDRESS> should be available, with other new features


Incorrectly reporting its version number <IP_ADDRESS> ... says it's v<IP_ADDRESS>

This is a:
Bug Report
Version:
<IP_ADDRESS>
Steps to reproduce:
download and install v<IP_ADDRESS> from releases page
What is expected:
that the app knows it is version <IP_ADDRESS>
What is actually happening:
it thinks it is version <IP_ADDRESS> and prompts for updates ... after updating I'm still on version <IP_ADDRESS> :)
I think the version tag within the software is incorrectly set in this build. :)
Any additional comments:
I love this product guys keep up the good work, and thanks so much !!!

Fixed incorrect version in release zip.
You can check for updates manually in 'Help' menu to get the updated version correctly.
Thanks for reporting.


Update Light Theme header colors

Signed-off-by: Nicolas Farruggia<EMAIL_ADDRESS>What this PR does / why we need it:
Update Light Theme header colors
Before

After

Which issue(s) this PR fixes

Fixes #639

Special notes for your reviewer:
Release note:
release-note

@nfarruggiagl Both the "Filter by labels" and the cluster switcher text, in this case "kubernetes-admin@kind" should be white. Also the cluster switcher caret should be white as well


name of forecastIcon to public

Please add one more method in WundegroundClient by analogy with getTodayIconText
in .h  (section public: )
 String getIconText(int period);
in .cpp
String WundergroundClient::getIconText(int period) {
return forecastIcon[period];
}
TNX

There's actually a pending PR for this https://github.com/ThingPulse/esp8266-weather-station/pull/59 -> we'll track it there

Some of those Icon descriptions can be (relatively) long, at least 12 and up to 16 chars and with screen space limited there seems little point in including this within the core programme and IMO it starts to degrade the advantage of the less is more display as is now.

This is not for draw on display, it's for the icon selection

There's already a variable called  'iconText' of type String and it's used for icon selection. e.g.
if (iconText == "chanceflurries") return "X";

From the ESP8266 weather station have you tried compiling it with that variable e.g. serial.print(iconText)

dual conversion - first: from icon name to symbol; after: from symbol to "iconText" variable.

You have to trace it through, the includes are relatively complex/convoluted but it's all there.


Apple Silicon and basic cross compiling support

Followup from https://github.com/aws/aws-sam-cli/issues/3132, refer to that issue and @jfuss for context.
This PR adds the possibility to build the docker images on Apple Silicon, by activating the environment variable:
$ export SAM_CLI_CROSS_BUILD=1

Otherwise none of the containers in build-image-src/build_all_images.sh will successfully build on an Apple M1 machine (Darwin/arm64). Some of the errors of not including buildx in the docker image building process include the following, predictably:
(...)
#8 43.57 ---> Package binutils.aarch64 0:2.29.1-30.amzn2 will be installed
#8 43.57 --> Processing Dependency: libdl.so.2(GLIBC_2.17)(64bit) for package: binutils-2.29.1-30.amzn2.aarch64
#8 43.57 Package: glibc-2.26-48.amzn2.aarch64 - can't co-install with glibc-2.26-48.amzn2.x86_64
#8 43.58 --> Processing Dependency: ld-linux-aarch64.so.1(GLIBC_2.17)(64bit) for package: binutils-2.29.1-30.amzn2.aarch64
#8 43.58 Package: glibc-2.26-48.amzn2.aarch64 - can't co-install with glibc-2.26-48.amzn2.x86_64
#8 43.58 --> Processing Dependency: ld-linux-aarch64.so.1()(64bit) for package: binutils-2.29.1-30.amzn2.aarch64
#8 43.58 Package: glibc-2.26-48.amzn2.aarch64 - can't co-install with glibc-2.26-48.amzn2.x86_64
(...)

The exec format error is specially confusing because I'm building everything as arm64/aarch64 now, no x86_64 or similar:
(base) rvalls@m1 s3-rust-noodles-bam % find . -iname bootstrap | xargs file
./target/debug/bootstrap:                                              Mach-O 64-bit executable arm64
./target/debug/deps/bootstrap:                                         Mach-O 64-bit executable arm64
./target/debug/deps/bootstrap.dSYM/Contents/Resources/DWARF/bootstrap: Mach-O 64-bit dSYM companion file arm64
./.aws-sam/build/s3Bam/bootstrap:                                      ELF 64-bit LSB pie executable, ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-aarch64.so.1, for GNU/Linux 3.7.0, BuildID[sha1]=cf3b3e130123119762ee6b373d6e93c6326f3667, with debug_info, not stripped

What am I doing wrong? :-S

Also, as a side note, the local building performance is quite disappointing since I guess it doesn't leverage the local machine's .cargo cache (6 minutes of build on a powerful M1, takes seconds locally :-!!):
% time sam build -u -bi provided.al2-rust
Starting Build inside a container
Building codeuri: /Users/rvalls/dev/umccr/s3-rust-noodles-bam runtime: provided.al2 metadata: {'BuildMethod': 'makefile'} architecture: ARM64 functions: ['s3Bam']
Failed to download a new provided.al2-rust image. Invoking with the already downloaded image.
Mounting /Users/rvalls/dev/umccr/s3-rust-noodles-bam as /tmp/samcli/source:ro,delegated inside runtime container

Build Succeeded

Built Artifacts  : .aws-sam/build
Built Template   : .aws-sam/build/template.yaml

Commands you can use next
=========================
[*] Invoke Function: sam local invoke
[*] Deploy: sam deploy --guided

Running CustomMakeBuilder:CopySource
Running CustomMakeBuilder:MakeBuild
Current Artifacts Directory : /tmp/samcli/artifacts
sam build -u -bi provided.al2-rust  0.45s user 0.19s system 0% cpu 6:02.99 total

This is a perfect example of one more paper cut when it comes to DX, see: https://github.com/aws-samples/serverless-rust-demo/issues/4

@hoffa, nevermind the last three messages, I managed to have it running on https://github.com/umccr/s3-rust-noodles-bam/commit/5d2ce7a659bcaee35561a488dee4dc0ca529e4aa ;)
Even the sam local start-api works now:  https://github.com/umccr/s3-rust-noodles-bam/commit/49534879a0170e1d97982401e7a72078a0e34ae4
Although I now I have figure out if I should define a dummy credential provider chain for S3, otherwise I'm getting:
Invalid lambda response received: Invalid API Gateway Response Keys: {'errorType', 'errorMessage'} in
{'errorType': '&alloc::boxed::Box<dyn std::error::Error+core::marker::Send+core::marker::Sync>',
'errorMessage': 'failed to construct request: No credentials in the property bag'}

I guess that there are two problems here:

sam local start-api is not mounting/seeing my host's ~/.aws/credentials, so it's not able to access the (non-public) S3 bucket I'm pointing to.
AWS SSO, which is the authentication my org uses, is not yet supported by aws-sdk-rust (currently using 0.0.19)? /cc @jdisanti?

From aws-sdk-rust README:

I guess that for now the only workaround possible to address local debugging with auth'd resources from my org is using the following with sam local start-api?:
  -n, --env-vars PATH             JSON file containing values for Lambda
                                  function's environment variables.


File 'RoleRepository.cs' has a small error

MySqlHelper.ExecuteNonQuery(conn, @"INSERT INTO aspnetroles (Id, Name) VALUES (@name, @id)", parameters);
Notice the sql statement, those two values are reversed.

Hi, thanks for the advice.
Yes I Know.. and actually I did the fork in your library to solve this problem and try to continue its development.
Thank you very much. Great Job

I have pushed fix for this issue.Thanks for reporting.


Add SPM support

Add Swift Package Manager Support

@nakajijapan How does this look?

@nakajijapan When do you think you can take a look?

Thanks for great PR!!!!!!!!


Add custom app support

Describe the solution you'd like
Allow users to add custom apps in addition to the k8s ingress apps. These will be defined in the customApps configuration variable.

added in 7773bc3


send cells or multiple lines without filling the repl history

Hi, in my Julia programming workflow I actively use both the .ipynb file and the REPL. I have two issues with how cells and multiple lines are handled.

after sending a "big cell" to the REPL, all lines are saved in the history. Then if I want to work with the REPL and use the command before the send, I have to go back as many times as the number of lines in the cell, which is annoying for big cells/selections.
If there was an error in the middle on the cell, I'd get a cascade of errors and it is very difficult to understand where the error was coming from.

Because of these problems, when I was using vimcmdline, I had a custom map that saved the cell into a temporary file, and then send the command include(tmpfile) to the REPL. In this way I solved both problems, since only a single line was added to the history, and if an error was there the julia debugger was able to tell me the exact location in the temp file. It is still not ideal though, as I would like to jump from the j-th line in the tempfile where the error occurred to the corresponding line in the cell...
How am I suppose to deal with such problems in jukit? I could implement my workaround, but I can't fine anything line VimCmdLineSendCmd to send a custom command to the REPL. Better ideas?

actually I realized that everything works out of the box in ipython, where several commands are sent to the repl to initialize the %jukit_run stuff. I guess that Julia support is very preliminary then. Any plans for a full Julia support?


[#150] Configuration location and filename and linenumber

LCOV of commit e777ff4 during CI #312
Summary coverage rate:
  lines......: 82.3% (2559 of 3110 lines)
  functions..: 78.1% (523 of 670 functions)
  branches...: 69.1% (992 of 1435 branches)

Files changed coverage rate:
                                                                               |Lines       |Functions  |Branches
  Filename                                                                     |Rate     Num|Rate    Num|Rate     Num
  ===================================================================================================================
  lib/config.js                                                                |91.7%    109| 100%    28|90.0%     60
  lib/instrumentation/instrument-method.js                                     |92.4%     66| 100%     6|79.6%     49
  lib/instrumentation/module/express/patch-layer.js                            |91.7%     48| 100%     7|75.0%     24
  lib/instrumentation/module/koa/koa-register-interceptor.js                   |72.2%     36| 100%     3|52.2%     23
  lib/instrumentation/module/mysql/mysql-create-connection-interceptor.js      | 100%     16| 100%     3|50.0%      2


[backport] Include UnownedMemory in the API docs

Backport of #3086

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit<PHONE_NUMBER>a4500b83a95245b95e6afe4cdd3e26, target branch v7) succeeded!


Install cumm on Orin

Similar to the previous issue (https://github.com/FindDefinition/cumm/issues/13) , I also failed to install cumm on Orin.
Can you tell me how to install it on Orin? Any comments would be highly appreciated.
The options I have set are as follows:
export CUMM_DISABLE_JIT="1"
export CUMM_ARCH_LIST="8.7+PTX"
sudo pip install -e.

The error message is:
[39/39] [GCC][Link]/home/siyeong/workspace/cumm/build/temp.linux-aarch64-cpython-38/cumm/build/core_cc/core_cc.cpython-38-aarch64-linux-gnu.so
FAILED: /home/siyeong/workspace/cumm/build/temp.linux-aarch64-cpython-38/cumm/build/core_cc/core_cc.cpython-38-aarch64-linux-gnu.so
g++ @/home/siyeong/workspace/cumm/build/temp.linux-aarch64-cpython-38/cumm/build/core_cc/core_cc.cpython-38-aarch64-linux-gnu.so.rsp -lcudart -lnvrtc -lcufilt -ldl -L "/usr/local/cuda/lib64" -L "/usr/local/cuda/lib64" -Wl,--no-as-needed -lnvrtc-builtins -shared -o /home/siyeong/workspace/cumm/build/temp.linux-aarch64-cpython-38/cumm/build/core_cc/core_cc.cpython-38-aarch64-linux-gnu.so
/usr/bin/ld: /usr/local/cuda/lib64/libcufilt.a(decode.o): relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol display_func_params' which may bind externally can not be used when making a shared object; recompile with -fPIC   /usr/local/cuda/lib64/libcufilt.a(decode.o): in function demangle_bare_function_type(char const*, int, int, a_decode_control_block*)':
/dvs/p4/build/sw/rel/gpgpu/toolkit/r11.4/compiler/drivers/compiler/edg/EDG_6.2/src/../util/decode.c:4996:(.text._ZL27demangle_bare_function_typePKciiP22a_decode_control_block+0x53c): dangerous relocation: unsupported relocation
/usr/bin/ld: /usr/local/cuda/lib64/libcufilt.a(decode.o): relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol display_func_params' which may bind externally can not be used when making a shared object; recompile with -fPIC   /usr/local/cuda/lib64/libcufilt.a(decode.o): in function decode_identifier(char const*, char*, unsigned long, int*, int*, unsigned long*)':
/dvs/p4/build/sw/rel/gpgpu/toolkit/r11.4/compiler/drivers/compiler/edg/EDG_6.2/src/../util/decode.c:8766:(.text._Z17decode_identifierPKcPcmPiS2_Pm+0x24): dangerous relocation: unsupported relocation
/tmp/pip-build-env-mjzpnmvl/overlay/lib/python3.8/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
warnings.warn(
/tmp/pip-build-env-mjzpnmvl/overlay/lib/python3.8/site-packages/pkg_resources/init.py:123: PkgResourcesDeprecationWarning: 0.1.36ubuntu1 is an invalid version and will not be supported in a future release

CUMM_ARCH_LIST should be 8.7+PTX for orin. this environment only valid for spconv, build cumm don't need this
libcufilt problem should be fixed in cumm newest commit (master branch, hasn't released to tags yet), which commit do you use for orin?

Sorry. I wrote it wrong. I set CUMM_ARCH_LIST to "8.7+PTX".
Thank you!

The issue is closed, but I'm leaving a reply for others.
After applying the commit, the build was done successfully.
I also succeeded in installing spconv. Thank you again.


Basic Website Layout

Description
The basic layout of the website states the static parts of the website, which remain the same on all of the screens of the application.
Like the header, footer, etc.
The aim of this issue is to create a static template, which will be the same across the application.
Definition of Done
The website should have at least more than one route, where the template code should be static and should no update/reload with the change in route. Although some of the content of the template screen will be dynamic, which is beyond the scope of this issue.

Wireframe for the template


fix: Include barcode format on iOS when using mlkit

Pull Request
Summary
Change Description

Summary of the change. Please also include relevant motivation and context.

Type of change: (select all that apply)

[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Change requiring an ADR
[ ] Other (please specify)

Dependencies

Dependencies that are required for this change.

N/A
Tests
Testing Summary

Description of tests run to verify changes, including instructions to reproduce .

Build on iOS
Scan a barcode using MLkit and confirm the format field on the Barcode object is defined and has the correct value

Testing Configuration

Relevant details of your testing configuration

Tested on iPhone 8 with iOS 15.2.1
Test Outcomes

Tests listed below should correspond to the ones listed in your summary

[ ] Scan a barcode using MLkit and confirm the format field on the Barcode object is defined and has the correct value

Checklist

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code if needed, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation if needed
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or verified that my feature works
[ ] New and existing unit tests pass locally with my changes

Unchecked items above are because I am unsure what the current state of style guidelines and unit tests are in this project, since this is the first PR in our fork.
Pull Request Review Template

Information for the Pull Request Reviewer

[x] I have read through the code and provided comments that follow - please use https://conventionalcomments.org/
[ ] I have built the code locally and tested the linked tickets acceptence criteria (Ignore for small changes)
[ ] I have build the code on App Center and Deployed (If necessary)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Show UPD port forwarding status in the control panel

Would be great if the user could see if the port forwarding is successfully set up.

This will probably be very difficult to do.

Replaced by #19


[Errno 101] Network is unreachable -- Some packages may not be found!

Hello,
When i try to build meta-ros with some ROS tools, i keep hitting the below error, any way to fix this or get around this ?
ERROR: python-rosdep-0.11.5-r0 do_compile: python setup.py build execution failed.
ERROR: python-rosdep-0.11.5-r0 do_compile: Function failed: do_compile
Log data follows:
| DEBUG: Executing shell function do_compile
| Download error on https://pypi.python.org/simple/nose/: [Errno 101] Network is unreachable -- Some packages may not be found!
| Couldn't find index page for 'nose' (maybe misspelled?)
| Download error on https://pypi.python.org/simple/: [Errno 101] Network is unreachable -- Some packages may not be found!
| No local packages or download links found for nose>=1.0
Thanks

Dear @vjayara1, I have never encountered that issue. In the python-rosdep recipe [1], it states that it depends on python-nose, so I assume python-nose is installed in the sysroot... While I was writing this, I wondered if python-nose is actually a native dependency, but this is not clear to me at the moment.
If I would have to guess the error, I would assume that your internet connection is has been disconnected or you are not allowed to make https connections from your build environment.
[1] https://github.com/bmwcarit/meta-ros/blob/master/recipes-devtools/python/python-rosdep_0.11.5.bb

Hello Lukas,
What I am wondering is why the recipe file is attempting to “fetch” during do_compile. During do_compile, the proxy settings are not taking effect and those of us who are behind a corporate firewall will not be able to build. is there a way that the fetch can be avoided during the compile task and done instead during do_fetch ?
Thanks
Vjayara1
From: Lukas Bulwahn<EMAIL_ADDRESS>Sent: Friday, September 30, 2016 6:24 AM
To: bmwcarit/meta-ros<EMAIL_ADDRESS>Cc: Jayaraman, Venkat<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [bmwcarit/meta-ros] [Errno 101] Network is unreachable -- Some packages may not be found! (#416)
Dear @vjayara1https://github.com/vjayara1, I have never encountered that issue. In the python-rosdep recipe [1], it states that it depends on python-nose, so I assume python-nose is installed in the sysroot... While I was writing this, I wondered if python-nose is actually a native dependency, but this is not clear to me at the moment.
If I would have to guess the error, I would assume that your internet connection is has been disconnected or you are not allowed to make https connections from your build environment.
[1] https://github.com/bmwcarit/meta-ros/blob/master/recipes-devtools/python/python-rosdep_0.11.5.bb
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bmwcarit/meta-ros/issues/416#issuecomment-250743208, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASoyBxCQiBolZyBs4cLb6VEYrcjruB2oks5qvQ1tgaJpZM4KJfjb.

That is a good question, and I was also expecting that the setup is made in such a way that it should not fetch further sources during the do_compile step. I do not know in detail why the recipe makes do_compile fetch that; it is defined in OpenEmbedded-Core's python-setuptools bbclass and python's setuptools package. This might be a good question to ask on the OpenEmbedded-Core mailing list.

@vjayara1, could people involved in OpenEmbedded-Core help you further with this question?

I had a similar problem and solved it by providing http_proxy settings during the distutils3_do_compile step.
I added the following function to the .bb file of the recipe that does not build:
distutils3_do_compile_prepend() {
    export http_proxy=${http_proxy}
    export https_proxy=${https_proxy}
}

@vjayara1 < Can this issue be closed?

"Silence implies consent" (see http://wiki.c2.com/?SilenceImpliesConsent).


Show Menu when holding to a day cell

Our project currently have a requirement that show menu ("Copy, Delete, Cut...") when user hold a day cell.
I've tried to modify your code base(implement UICollectionViewDelegate for showing menu) but it didn't work.
public func collectionView(_ collectionView: UICollectionView, shouldShowMenuForItemAt indexPath: IndexPath) -> Bool {
    return true
}

public func collectionView(_ collectionView: UICollectionView, canPerformAction action: Selector, forItemAt indexPath: IndexPath, withSender sender: Any?) -> Bool {
    return true
}
public func collectionView(_ collectionView: UICollectionView, performAction action: Selector, forItemAt indexPath: IndexPath, withSender sender: Any?) {
}

Hope you will support this feature.
Thank you

You do not have to modify the code to do this.
All you have to do is add a long press gesture to the calendar -> UILongPressGestureRecognizer
An example of adding gestures can be found here
You can also find implementation of long press gesture in the continuous range selection code located at the bottom of this page here.
Basically, once long press is activated, simply present your view.

If you need more help on this, then let me know.

It would be much easier if we implement default Apple menu. We don't have to handle long press gesture, finding target rect, target cell....

Code for handling long press is very short
override func viewDidLoad() {
   super.viewDidLoad()

   let longPressGesture = UILongPressGestureRecognizer(target: self, action: #selector(didStartLongPressing(gesture:)))
   longPressGesture.minimumPressDuration = 0.2    // User has the flexibility to set the time
   calendarView.addGestureRecognizer(longPressGesture)
}

func didStartLongPressing(gesture: UILongPressGestureRecognizer) {
   if let cellState = calendarView.cellStatus(at: gesture.location(in: gesture.view!)) {
         // Code for shouldShowMenuForDate
         // Code for canPerformActionForDate
         // Code for performActionForDate
   }
}

Now what you said is true. It would be easier for developers if we implemented menus into the library as you have suggested. But remember, this library is being used by very many people. We would not want to create specific cases for some developers.
Currently the library is so flexible that any developer can create the showing of menus however they want it to be. I we implemented a default behavior, then I think we will be making the library less flexible, right?

I've tried your code to get cellState, but i need to get current cell, and it always return nil
if let cellState = calendarView.cellStatus(at: sender.location(in: sender.view!)) {
      print(cellState.cell()) // nil
}

Its always nill. That's true. It's a bug.
I just fixed it on master branch.
Can you test it now?
Put this in your Podfile.
pod 'JTAppleCalendar', :git => 'https://github.com/patchthecode/JTAppleCalendar.git'

and then do a pod install.

Thank you, it's worked.

I am still getting nil in the following function print(cellState.cell())

@rajhcss provide a sample project?

Hi JT, thanks for your reply.  I probably found the issue.  I modified your code slightly and it seems to work.  Sorry I cannot provide the source code that I am working on.  Please find below the modified function in the UserInteractionFunctions.swift
    /// - Parameter: point of the cell you want to find
    /// - returns:
    ///     - CellState: The state of the found cell
    public func cellStatus(at point: CGPoint) -> CellState? {
        if let indexPath = indexPathForItem(at: point) {
            let cell = self.collectionView(self, cellForItemAt: indexPath) as? JTAppleCell
            //let cell = cellForItem(at: indexPath) as? JTAppleCell
            return cellStateFromIndexPath(indexPath, cell: cell)
        }
        return nil
    }

awesome.

@Dyanu create a new issue. I am lost on the context you are experiencing.
I'll respond there.


[Advice] To Place the URL button of This Repository In Main Page

I guessed the url of this repo through the domain name of the deployed site, and I think you should place a repo button in index.html so that others can join into the contribution.

Star

The HTML code you may need to add has been generated and supplied above. I am not good at front-end development, so you can customize it according to your preferences.
Finally, in totally, this project is good and I really support your work. Come on, for a new China with no deceptive words said by gov and its f***ing media.

Thank you for your suggestions and support. ❤
I will now begin creating a sign (button) to guide the source code and contributions for it.

Hi, I have created a button link to the code repository for the webpage, please refer to #4. It doesn't directly use your code, and the style of the button is also different from what you suggested, for two reasons:

The project requires rendering the button using JavaScript instead of HTML, so I couldn't directly adopt your solution;
The style of the github-button doesn't suit the webpage, so I designed a new style myself;

Finally, thank you for your contribution. Now people can more easily access its source code and even contribute new data.

Please click here to view the new button!


Setup dependency reusable

https://github.com/nimblehq/elixir-templates/issues/??
What happened
Describe the big picture of your changes here to communicate to the team why we should accept this pull request.
Insight
Describe in details how to test the changes, which solution you tried but did not go with, referenced documentation is welcome as well.
Proof Of Work
Show us the implementation: screenshots, gif, etc.

Close as Github doesn't allow to combine both uses and steps at this moment.


chore: add doc links

Add two doc links: one for the standard Go docs and one to a relevant page relating to Zitadel.

Thank you for this PR.
Does look good to me.


Implement missing logic for CollectFrameDataFromAssetEntity

Changelog Description
resolve #50

CollectFrameDataFromAssetEntity

Testing notes:

everything should work as before.
Please report if the missing logic should be implemented instead of removing the redundant settings.
Please report if the PR is not backward compatible (copy settings should work fine without implementing conversions)

Resolved with https://github.com/ynput/ayon-traypublisher/pull/51 (please don't create doubled PRs if only one should be merged next time).


Cartesian3

Hey,
Currently I am using the EventClickMouse Handler example provided, but I am receiving Cartesian2(x,y).
Anyway I can receive Cartesian3 instead?

Duplicated: https://github.com/reearth/resium/issues/219


Sad sprite rotation inaccuracy

I think I discovered a bug concerning sprite rotation.
I inserted a sprite with 270° rotation
but it shows weird artifacts on its borders, as if it was not rotated all the way.
I added my simplified code base for recreating this bug:
sprite_bug.zip

Looks like linear interpolation to me, which is expected behavior. (Especially when rotating in non-90 degree intervals.)
If you don't want linear you can change it to nearest by doing the following before loading any textures:
pyglet.image.Texture.default_min_filter = pyglet.gl.GL_NEAREST
pyglet.image.Texture.default_mag_filter = pyglet.gl.GL_NEAREST

The rotation math can be found here:
https://github.com/pyglet/pyglet/blob/pyglet-1.5-maintenance/pyglet/sprite.py#L402
It looks like it's probably just lack of float precision...
If you set the image anchor to the center, it still maintains enough precision to appear as expected.
Perhaps we could replace this with some Matrix math and see if it makes a difference.

Closing for now since workarounds exist. We can't really do anything to fix the float precision on the 1.5 branch anyway, but once 2.0 is out for release it will no longer be an issue.
Thanks!


Import aia project via url

Describe the desired feature
I would like to be able to import an aia via an url.
Give an example of how this feature would be used
With extensions it looks like this.

I would like the same for aia files. When teachers or tutorial developers make aia files available on their website, the user would just need to copy and paste an url into the textbox.
Why doesn't the current App Inventor system address this use case?
With the current version you can use the import option and drag and drop an aia. There is no option to specify an url. The only options for using an url is for extensions and librabry templates.
Why is this feature beneficial to App Inventor's educational mission?
Users who use a shared computer, like in schools, could overwrite each others aia by downloading the aia or could import the wrong aia when looking for it on that shared computer. Being able to use an url would make certain that the right aia is loaded into App Inventor.

Note that if the URL is hosted on a server that will send the appropriate CORS headers, you can drag and drop a link into the project list to import it. For example, AIAs posted to the community site or via GitHub pages. I feel like the extensions import mechanism works under the same constraint. Now in theory we could move the retrieval of the project to a server-side operation to bypass CORS, but we'd have to think about whether that is the best approach. Certainly it should be possible to add the proposed affordance.


Launch X Layer

Background
Any information that gives more context as to why this task exists, and why it matters. This might include: links to discussions, documentation, or other Jira issues / tasks
Linked Issues & Documentation

List of linked issues, external links, etc

Definition Of Done
Describes in 1-2 sentences what the output of the Task is, so that it can be verified by another team member easily. “Feature X is implemented as described and can be verified by using it”
Tasks

[ ] Task 1
[ ] Task 2
[ ] Task 3

Chain has been deprioritised, Moving this in backlog


chore: Fixing Anvil DnD in zones

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description
In this PR, we are making changes to how widgets are moved inside zones.
previous changes missed to cater to basic small widget movement within a zone.
Now when widgets move inside a zone.

for small widgets, we look for layout based on highlight info and add widgets to it.
for large widgets, we create a new layout next to the layout mentioned in the highlight info and add widgets to it like adding new widgets

Same logic propagets to copy paste based widget addition as well.
Additional fixes:
Widgets were being moved in the order of selection, now we use position data to sort them and place based on position data.
PR fixes following issue(s)
Fixes # (issue number)

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test<EMAIL_ADDRESS>


add ability to sign commits with git-xargs

Description
Closes #93.

This PR allows users to sign commits with git-xargs using a global signingkey in their git config.
see: https://docs.github.com/en/authentication/managing-commit-signature-verification/telling-git-about-your-signing-key for more info on configuring git to use a signing key.
TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added ability to sign commits with git-xargs. This depends on the user's git config to be configured with a signing key. If not configured, it will default to not signing commits.
Migration Guide
N/A

@zack-is-cool I saw this PR was waiting on an approver and since I also need this feature I just fetched your repo/branch zack-is-cool/git-xargs and checked out the feat/add-commit-signing branch. I have commit signing setup correctly and I tried to run git-xargs directly from the branch via go run main.go --branch-name 'git-xargs-test' --commit-message 'Testing my sig' --repo gekitsuu/git-xargs-text-repo /home/gekitsuu/repos/git-xargs/test.sh. The command succeeded and it cut a PR but the commit wasn't signed.

@gekitsuu try bumping up the logging level for more info. You can look at the debug logging I added.  I've been using my fork since I made this PR to be able to sign commits for my org's security settings.

@zack-is-cool I did that and realized that I hadn't checked out the branch correctly. I am able to run the command now, and it prompted me for my GPG key password. Even though I'm entering the passphrase correctly, I get [git-xargs] DEBU[2024-08-14T14:53:23-04:00] Error encountered while processing repo       Error="openpgp: invalid argument: signing key is encrypted" Repo name=git-xargs-test-repo Any ideas what's causing that? I'm on Debian 12 running  gpg (GnuPG) 2.2.40  and git 2.39.2 if that helps in any way.

@zack-is-cool Not sure I tagged you correctly in the last comment

Not totally sure tbh,
if you look at this:
https://github.com/gruntwork-io/git-xargs/pull/140/files#diff-daf8735f223c2102f6c67b11ae0f6c605ae36798314376c9b9d976d30ff3d930R311-R364
you can kinda see what's going on - specifically here it's calling gpg program with arguments --export-secret-keys", "--armor" and your signing key id. You could try to call those manually in your terminal to see if you get a similar error. I'm running this on mac gpg (GnuPG) 2.4.5
https://github.com/gruntwork-io/git-xargs/pull/140/files#diff-daf8735f223c2102f6c67b11ae0f6c605ae36798314376c9b9d976d30ff3d930R340

@zack-is-cool I'm going to guess that you don't have a password on your GPG key. When I removed the passphrase from mine, then your branch signed the commit as expected, but it should handle taking a user through supplying the passphrase if they use one.


SaveChanges on entity not persisted in Cache

I've tried out Cacheable, and found that it doesn't seem to register new changes to entities. I kinds of expected Cacheable to purge any entity that was modified using the DbContext, so as to make sure it's fetched again.
Am I using this wrong?
I've noticed none of the tests test a SaveChanges() after the initial setup.
Steps to reproduce
Blog aa;
using (MyContext db = services.GetRequiredService<MyContext>())
{
	// Cache the first entity in the DB
	Blog c = db.Blogs.Cacheable(TimeSpan.FromMinutes(60)).First();
	aa = c;

	Console.WriteLine(c.Id);
	Console.WriteLine(c.Name);
}

Blog bb;
using (MyContext db = services.GetRequiredService<MyContext>())
{
	// Pick the first entity in the DB, without caching, so we can save changes
	// Else #10 will kick in
	Blog c = db.Blogs.First();
	bb = c;

	c.Name = DateTime.UtcNow.ToString();

	Console.WriteLine(c.Name);

	db.SaveChanges();
}

Blog cc;
using (MyContext db = services.GetRequiredService<MyContext>())
{
	// Fetch the Blog again elsewhere, observe that Name has not changed
	Blog c = db.Blogs.Cacheable(TimeSpan.FromMinutes(60)).First();
	cc = c;

	Console.WriteLine(c.Name);
}

// Here we see that the two Cached entries (aa, cc) are the same object
Console.WriteLine(ReferenceEquals(aa, bb));
Console.WriteLine(ReferenceEquals(aa, cc));
Console.WriteLine(ReferenceEquals(bb, cc));

Further technical details
EntityFrameworkCore.Cacheable version: (found in project.csproj or packages.config)
EF Core version: 2.2.4
IDE: VS 2019 (16.1.2)

Same here, doesn't seem to invalidate the cache after new inserts/updates in the dbcontext


[Question]: Information request for resource utilization warnings feature re-enablement

Describe your question
The resource utilization warnings feature is currently disabled by removing $(AZP_ENABLE_RESOURCE_UTILIZATION_WARNINGS) agent knob as temporary workaround for performance degradation issue https://github.com/microsoft/azure-pipelines-agent/issues/4781.
Please update here for users' awareness once the issue is fixed and the feature is re-enabled.
Versions
Agent version '3.239.1'
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[X] Microsoft Hosted
[X] VMSS Pool
[X] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Operation system
No response
Version controll system
No response
Azure DevOps Server Version (if applicable)
No response

Hi @kemats,
We completed and merged the PR with the fixes for the resource utilization monitor - https://github.com/microsoft/azure-pipelines-agent/pull/4803.
In scope of this PR we:

Fixed the issue with the incorrect CPU metrics for Linux/Windows/MacOS
Added metrics catching between debug and warning monitors, which decrease the amount of metric requests and improve performance
Switched the Windows metrics methods to use .Net ManagementObjectSearcher queries instead of direct calls of the deprecated WMIC.exe which will resolve the issue with thread pool exhaustion.
Replaced synchronous metric methods with asynchronous requests that support cancellation to avoid the issue with the blocking requests
And other minor improvements that will improve overall performance and will provide more accurate resource usage data

The changes will be available on the next agent rollout. We'll inform you after deployment will be completed.

Hi @kemats!
We deployed the new agent version with implemented fixes, and started rollout for the FF which enables the resource monitor warnings.

Hi, resource utilization warnings have disappeared since agent version 3.243.1. They worked fine in 3.242.1. Is this somehow related, and how can we bring them back?

Hi - do you have an update here?

Hi @jappenzesr, the resource utilization monitor and related features including warnings were reworked and it is available for the agent versions start from https://github.com/microsoft/azure-pipelines-agent/releases/tag/v3.243.1.


useMeasure documentation has example for useLongPress instead

The documentation for useMeasure gives an example for useLongPress instead
https://usehooks.com/usemeasure

Duplicate of Issue #181


Prettified task description, added syntax highlighting

We can’t automatically merge this pull request.
Please create another one.


How to ensure POW-Results are not re-used

I am working on integrating Altcha into our site and I have come across a point that seems to disqualify Altcha entirely for me. Since there is zero state associated with the challenge (i.e. there is no nonce or something like that), how do we prevent a malicious client from simply skipping the challenge issued and just... re-using a previous challenge they did? Since the system is stateless, the parameters of the challenge are returned back to the server with the solution, so all the server can do is verify that the client responded with a valid solution, not the solution to the challenge that was actually issued for this form.
Setting an expiry value is only a workaround imho.

Hi, for that purpose your server should store used challenges in to some sort of registry such as Redis and check the registry before validation. To make sure, that the challenge is issued to the specific form or page, add custom parameters to the salt as <salt>?form_id=123 and check the parameters before validation (the parameters in the salt are part of the signature).

Thank you for your input. I think expiry value plus a Redis set could work for my needs.


Using image source instead of webcam causes security error

Describe the bug
Webcam works great but when I try to use the image source get a security error - operation insecure?
To Reproduce
Steps to reproduce the behavior:

Expected behavior
Image to be run through the detection phase
Screenshots

Desktop (please complete the following information):
Win 10, Firefox (latest)
Smartphone (please complete the following information):
Not used
Additional context
Add any other context about the problem here.

Hi @hillj8 which image are you using? Does it is hosted in the same origin or in another place?

Hi there, please reopen issue in the new AR.js repository: https://github.com/AR-js-org/AR.js
Thank you.


Bug: using column named Group results in exception on inserts

Table has a column named Group. When inserting new records exception occurs. Could it have something to do with the GROUP BY sentence in SQL?
System.Data.SqlClient.SqlException
  HResult=0x80131904
  Message=Incorrect syntax near the keyword 'Group'.
  Source=Core .Net SqlClient Data Provider
  StackTrace:
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString)
   at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async, Int32 timeout, Task& task, Boolean asyncWrite, SqlDataReader ds)
   at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean asyncWrite, String method)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Mighty.MightyOrm`1.Execute(DbCommand command, DbConnection connection)
   at Mighty.MightyOrm`1.ActionOnItem(Object& modified, OrmAction originalAction, Object item, DbConnection connection)
   at Mighty.MightyOrm`1.ActionOnItemsWithOutput(OrmAction action, DbConnection connection, IEnumerable`1 items)
   at Mighty.MightyOrm`1.Insert(Object item)

I'm pretty sure Massive would have done this too, it doesn't by default quote column names (e.g. [Group] instead of Group is what would be required for SQL Server). In Mighty you could write a SqlNamingMapper which overrides ColumnNameMapping only, to do the required quoting.

@brgrz Does this provide you with a workaround? (Re-open if not.)
As I say, Massive would have had this problem too.
I could update the code to always quote all DB names (although IIRC there are cases where exactly which quote characters are needed depends on context, so this might be non-trivial).
It's pretty much a corner case - at least seemingly the same behaviour wasn't a real problem in Massive. And Mighty does provide a work-around (other than changing the code yourself!), as described above.

No problem, I just changed the name of the column and it's fine.


Restyle example formatting for Style/Alias cop

Follow up of https://github.com/bbatsov/rubocop/pull/4880#issuecomment-338499947.
This commit is a change of document format for Style/Alias cop.
Since this example was the opposite of bad and good, it was rearranged
for unification. Also this commit adds one sample which was not mentioned.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

Btw, we should also include the example names in the generated manual, otherwise the output becomes a bit confusing.


u-boot-ti-staging: Fix SPL boot loop

The BSP layer at the rocko branch has an issue with the boot
stuck in a loop. See details here:
https://lists.yoctoproject.org/pipermail/meta-ti/2018-October
/011363.html
Changelog-entry: Fix boot loop hang when switched to meta-ti rocko
Signed-off-by: Florin Sarbu<EMAIL_ADDRESS>

@resin-ci retest