Wrong comment char for rust code

I'm not sure if I'm on the right place for this:
https://michael-f-bryan.github.io/rust-ffi-guide/fun/problems.html
All your examples have the following code inside:
# #![allow(unused_variables)]
#fn main() {

This won't ever compile as # is not a correct comment char in rust.
Please correct it to // or /* or leave this part out.
error: expected `[`, found `#`
 --> adder.rs:2:3
  |
2 | # #![allow(unused_variables)]
  |   ^

error: aborting due to previous error(s)

I'm sorry, seems like without both JS CDN's I'm seeing some code to parse.
And the first example uses "uit" not "uint", unknown type.

I'm not sure if I'm on the right place for this:

Yep, this is the right spot :smile:

This won't ever compile as # is not a correct comment char in rust.
Please correct it to // or /* or leave this part out.

You are right in that the # character isn't a valid comment character in Rust, but it's used by rustdoc to mark code which should be part of an example but not displayed by default. You usually use it to hide a bunch of unnecessary lines like imports so they don't clutter the example code. The Book mentions this towards the end of the "Documentation as Tests" section.
In this case it looks like mdbook automatically wraps all Rust code snippets in that automatically so you have a valid program which can be run in the playpen. I'm not sure there's much I can do from my side of things to remove the lines, short of using a patched version of mdbook...
If you look at the actual source for that page you'll see that the lines in question aren't part of the code snippet:

Updated, looks like it's done by the two JS-CDN's embedded. So my default external block stopped parsing this. If I enable these in uMatrix everything works fine again ^^

@0xpr03, you shouldn't be seeing those lines by default... Did you hit the "expand" button by any chance?
This is what I see without hitting the expand button:

And after:

If you are seeing the latter screenshot when you first visit the page then that's a bug in whatever JS mdbook uses.

No, the problem is that without code.jquery & cdn.jsdelivery the site looks as attached.

I didn't even consider the # to be part of the code formating JS.
So this was kinda my fault.

That's interesting. I never would have thought to see what happens when you block javascript and other background requests. I guess mdbook should degrade gracefully, but I'm not exactly sure how they'd accomplish that.
I got curious about what uMatrix does so I had a quick skim through the README on their GitHub repo. Apparently it's not uncommon for it to break things.

Regarding broken sites
uMatrix does not guarantee that sites will work fine: it is for advanced users who can figure how to un-break sites, because essentially uMatrix is a firewall which works in relaxed block-all/allow-exceptionally mode out of the box: it is not unexpected that sites will break.
So this means do not file issues to report broken sites when the sites are broken because uMatrix does its job as expected. I will close any such issue without further comment.

Apparently it's not uncommon for it to break things.

That's true, as I'm using it since some time ago together with it's better known browther uBlock Origin I've gotten around to see some really horrifying site constructions, displaying a blank page without 4+ external JS provider ;)
Anyway, I just didn't expect the raw site to contain stuff that's obviously not even required ( like the main() ) which will in turn be removed by the JS code. I looked like you've had some old code around there which was commented out.


feature/TECH XXX bring intellij project settings under vcs

Bring general intellij settings under version control
Add Intellij files
Add .idea gitignore

Note: #128 may be closed

⚠ Could not find ticket corresponding to `feature/TECH-XXX-bring-intellij-project-settings-under-vcs. Does your branch name follow the correct pattern?


Updating link

Quick PR with a small docs change. The link seems to have updated, I got a 404 when clicking, but was able to search and update to the current URL. If you need CLAs and other stuff to merge, please just copy my change into your own commit and close this PR.

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

thanks for the PR. @chmac just curious, is there a reason you can't sign the CLA (eg. work restrictions)?

@kevinslin No really good reason no, primarily laziness. I haven't even looked at it. My assumption is that it allows you to relicense my freely contributed code under a non free license. As a general principle, I'm opposed to such things, although I understand the commercial advantages. If I was going to write a real contribution in terms of code, then I'd check the CLA first and see if I'm happy with it. But for a one line URL change it seems like overkill, or rather, it would probably quadruple the amount of effort required for this 1 change. :-)

updated the readme: https://github.com/dendronhq/dendron/commit/fceb49373b3f862399db3bcbaacc2a54701b6ba0
thanks for the correction


[typescript] Improve definitions with strictNullChecks disabled

While disabling strictNullChecks is still potentially dangerous this removes one false positive that marked the theme as required. Disabling strictNullChecks will still allow the access of theme which will throw at runtime.
The workaround was discovered by @geirsagberg in #12891. I combined this with some necessary documentation so that we can refer to the documentation in future issues because we should not support strictNullChecks in the future (DefinitelyTyped isn't either).
Pinging @pelotom in case he has some objections to our tsconfig requirements.

Do you think it's enough to close #12891?

@oliviertassinari Since op already confirmed that this patch resolved the issue in his codebase I think we can close it.


Update stats for prod81

Adjusts the stats for a few recent releases using https://github.com/IDR/idr-utils/pull/8. Also includes the comments from https://github.com/IDR/idr.openmicroscopy.org/pull/81#discussion_r419266655

Looks good. Guess I don't really get the terminology in these files because I got quite a few numbers wrong earlier it seems.

Pushed another commit fixing the calculated average sizes.
Happy to have a very stance at clarifying the terminology. What would be the most suitable format? A datapackage JSON accompanying the TSV files  would be one option.

👍 Good to merge from my point of view. Could give this datapackage json a try... a bit worried though that it's overcomplicating things.

Merging in order to open a follow-up prod82 stats PR. The discussion around the vocabulary used for IDR metrics can happen separately including the question of how and where to store and exchange it.


check_random_url fails, I don't know why

I would need help to understand what is expected for check_random_url, I cannot figure it out myself.
My SCIM endpoint returns this payload on when querying non-existent endpoint. As you can see, it answers with a 404 status code and there is an error payload. Is there anything that doesn't conform to the spec?
curl -H 'Authorization: Bearer api_key_admin' -H 'Content-Type: application/json' http://localhost:3000/scim/whatever -v | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>:3000...
* Connected to localhost (<IP_ADDRESS>) port 3000 (#0)
> GET /scim/whatever HTTP/1.1
> Host: localhost:3000
> User-Agent: curl/7.88.1
> Accept: */*
> Authorization: Bearer api_key_admin
> Content-Type: application/json
>
< HTTP/1.1 404 Not Found
< X-Powered-By: Express
< Content-Type: application/scim+json; charset=utf-8
< Content-Length: 104
< ETag: W/"68-zh+4yIoz1govWdaghCJSScM0qgQ"
< Date: Sun, 01 Sep 2024 11:25:08 GMT
< Connection: keep-alive
< Keep-Alive: timeout=5
<
{ [104 bytes data]
100   104  100   104    0     0   7679      0 --:--:-- --:--:-- --:--:--  8000
* Connection #0 to host localhost left intact
{
  "schemas": [
    "urn:ietf:params:scim:api:messages:2.0:Error"
  ],
  "status": "404",
  "detail": "Endpoint Not Found"
}

Fixed and release in 0.1.5. This was due to the recent update to scim2-models. I wish I had written the unit test suite earlier...
check_random_url simply checks that an invalid URL returns a 404 HTTP code with a SCIM Error object payload in json format.

Thank you so much!


[Bug][Figma]: goa-Input-Label component does not wrap properly when length of label exceeds width of field.

Describe the bug
In Figma, the goa-Input-Label component should have auto-layout set to Wrap, so that if there is not enough room for the label, the label itself and/or the (optional)/(required) elements will wrap to a second line underneath the first line of the label.
Wrong Behaviour

Correct Behaviour

Provide Code
No response
To Reproduce

Place a component that uses the Goa-Input-Label sub-component into a Figma page and set the width of the main component to a fixed width.
Set the field/component to "(optional)".
Type in a long field label until it exceeds the width of the component.

Relevant console log output
No response
What browsers are you seeing the problem on?
No response
Additional context
No response

Sorry about the extra images - can't seem to delete them.

copying from Slack support channel:
@jim-muirhead  Unfortunately this is a limitation of Figma. We are not yet able to wrap text WITH other items. In this case, two different text blocks, same as an icon or badge inline with text.
This forum thread explains the issue a bit:
https://forum.figma.com/t/how-to-wrap-texts-with-auto-layout/59844/3
For now we just need to adjust that manually in design. But this works in development, so that's good!
There are often cases where we don't want the text label to wrap with the input - e.g. a narrow checkbox list, a known width of 6 characters on an input, and other shorter inputs, we wouldn't want the text label to wrap with the input.

Given Figma's limitation here, I think manually setting these to wrap or fill when needed is a decent work around until we can wrap text inline with other items like we can in code.


Cannot unzip HtmlAgilityPack

paket.dependencies:
source https://www.nuget.org/api/v2
nuget HtmlAgilityPack

.paket\paket restore
Paket version <IP_ADDRESS>
found: paket.dependencies
Downloading HtmlAgilityPack 1.4.9 to C:\Users\...\AppData\Local\NuGet\Cache\HtmlAgilityPack.1.4.9.nupkg
Something went wrong with the download of HtmlAgilityPack 1.4.9 - automatic retry with --force.
Downloading HtmlAgilityPack 1.4.9 to C:\Users\...\AppData\Local\NuGet\Cache\HtmlAgilityPack.1.4.9.nupkg
Paket failed with:
   Error during unzipping lib/sl3-wp/_._ in HtmlAgilityPack 1.4.9: CRC error: the file being extracted appears to be cor
rupted. Expected 0x00000000, Actual 0x6AF4413C

Thanks, now it works ok :)


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #27.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin please rerender

I don't understand why there were no tests. This needs recompiled for a newer gsl.

@conda-forge/root5 Can someone try to bump the build number here to elicit tests? The previous build was never uploaded, so anything requiring root5 can't be rebuilt do to old gsl pinnings, which I presume should get resolved with a new build. That would also solve #26


Add looping instructions

I was exploring some examples, and I saw the looping statements loop and while, but I didn't find it in instructions tab, please add the description for it.

Hi, we will check this one; thanks for the feedback.


chore: add default assignee for issues

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

@zoercai it seems this yml file is templated, meaning it will be overwritten on synth auto generated PRs. Could you check with the yoshi-java team how to make this change?


{Storage} az storage file metadata copy: Track2 SDK Migration

Related command

Description

Testing Guide

History Notes

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

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

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

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


[Bug][Rails] Null image tag displaying on popup when story card is activated

Describe the bug
The popups on the map are set up to show an image, if there is one set for that place (via the photo_url field); if the value is null then it will not (see this code in Popup.jsx here).
This works when you press on a marker point on the map:

However, when you press on a story card in the sidebar -- which causes the map to navigate to the related Place and open up the popup -- if there is no photo, it still inserts an empty <img> tag.

To Reproduce
Steps to reproduce the behavior:

Open the Terrastories map view
Press on a marker for a Place that does not have a photo
You will not see an <img> tag
Now, press on a Story that is linked to that Place
You will see the empty <img> tag in the Popup

Expected behavior
If there is no image in the Place photo_url field, it should not insert an <img> tag in the popup when you press a story card, much like it doesn't when you press a marker and open a popup that way.

Hi, I can take this issue!


Possible issue in gradings by Bashar Awaad Marouf in Jami At Tirmidhi

Assalamualekum.
Grading source for Tirmidhi:2 - https://fawazahmed0.github.io/maktaba-grades-backup/33861/6#p1
Marked as hasan sahih by Bashar Awaad Marouf. (https://www.hadithhub.com/tirmidhi:2)
But in the text I find "هَذَا حَدِيثٌ حَسَنٌ صَحِيحٌ , وَهُوَ حَدِيثُ مَالِكٍ، عَنْ سُهَيْلٍ، عَنْ أَبِيهِ، عَنْ أَبِي هُرَيْرَةَ." which is the wordings of Imam Tirmidhi (check the Arabic text in hadithhub which makes it very clear.) and not Bashar Awaad Marouf. I could not find the gradings of Dr Bashar for that hadith in alMaktaba backup.

And https://fawazahmed0.github.io/maktaba-grades-backup/810/1 starts from hadith 859. Where did you get the gradings for the rest?

hadith 1 starts from here: https://fawazahmed0.github.io/maktaba-grades-backup/810/73


Lookups need support for SLDS 2.0.1

The slds-lookup element now needs slds-is-open to show child menus.

Completed on the development branch.


Handling variable occurrences at run time

This library is excellent for fixed_width conversion.
There is a use case which I am trying to resolve:
A field might occur a number of times depending on a count(in some other field, say the previous field's value) at run time.
In such a case, will start_pos have to be changed dynamically or can there be a field_index property with only the field_length?

Hi @pi-bansal. The situation you describe would no longer be a fixed-width file. So, that functionality would be out-of-scope for this.
If we add it, it would increase the complexity of the library, which I wouldn't consider to be worth the trade-off since it's fairly widely-used and this hasn't come up before.

Fair enough...!


Display buy MP

Also need to add buy MP population to pipeline and tables

Apparently I added this a while back and didn't notice.


Simple Iron Chisel should break bits smaller than 4x4

It'd be really annoying to place a single bit, then only be able to get it back as a Bit Collection Block without making a Simple Diamond Chisel. I presume more complex chisels (that can break in different shapes?) are also planned in the future?

You can still break the 4x4x4 the single bit is in with the iron chisel

You can still break the 4x4x4 the single bit is in with the iron chisel

Doesn't look like it to me, am I doing something wrong?
https://user-images.githubusercontent.com/1138417/105804994-f846f680-5f98-11eb-8f9a-0602a4bfec64.mp4

Doesn't look like it to me, am I doing something wrong?
https://user-images.githubusercontent.com/1138417/105804994-f846f680-5f98-11eb-8f9a-0602a4bfec64.mp4


Package name

Description of feature
I've used genomic-annotations for now, since it's descriptive and seems available on pypi. But no need to keep it, and would be nice to have something shorter.

why not genomic-features?


Can't get godep to work with local packages

I have a problem with an existing project that I try to migrate to godep and I can't really solve it.
If I run godep save ./...  I get the following outputs:
godep: directory "/Users/daseinhorn/Documents/go/src/bitbucket.org/myorg/myproject/utils" is outside source root "src"
godep: directory "/Users/daseinhorn/Documents/go/src/bitbucket.org/myorg/myproject/utils/cors" is outside source root "src"
godep: directory "/Users/daseinhorn/Documents/go/src/bitbucket.org/myorg/myproject/utils/db" is outside source root "src"

How can I get godep to work even if this repos are used in my code?
Thanks,
daseinhorn

I had this problem recently and my GOPATH was pointing to the wrong project. Check your GOPATH, echo $GOPATH to eliminate that as the issue.


Add glances serivce

Description of changes
This creates a module for running glances as a systemd service. I've had this in my local config for a few months.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build glance.tests.service

@drupol  Finally got time to address your remarks, I think they should all be resolved now :)

Can you please add nixosTests to glances derivation?

To make sure I do the right thing. Add passthru.tests... to https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/system/glances/default.nix ?

yeah, something like this: https://github.com/NixOS/nixpkgs/blob/71ccc3b0b040a68100275f497e264c3abf373f58/pkgs/servers/honk/default.nix#L53

yeah, something like this:
https://github.com/NixOS/nixpkgs/blob/71ccc3b0b040a68100275f497e264c3abf373f58/pkgs/servers/honk/default.nix#L53

Added!

@drupol @fsnkty Your issues have been addressed and I just rebased to fix som merge conflicts agains upstream


Compile problem

Hi,
I'm trying to compile this script, but i don't know how.
I tried with umake but i dont know if i prepare it right because im getting error:
----- Core (Release)
----- Engine (Release)
'Engine' is not a DLL package; can't find export 'intAPlayerControllerexecSetViewTarget'
History: UPackage::GetDllExport <- UFunction::Bind <- UField::PostLoad <- UStruct::PostLoad <- UFunction::PostLoad <- UObject::ConditionalPostLoad [(Function Engine.PlayerController.SetViewTarget)] <- PostLoadObjects <- UObject::EndLoad <- UObject::LoadPackage <- OpenPackage <- CompilePackage <- LoadEditPackages <- UMakeCommandlet::Main
Exiting due to error
Can you explain me what i am doing wrong and how to compile it properly?
Greetings from Poland.

Ok, i learn a bit and go forward ;-)
Now I'm getting that error:
D:\Gry\SWAT 4_2\ContentExpansion\System>ucc make
--------------------Utils - Release--------------------
Analyzing...
Superclass Object of class ArrayUtils not found (if superclass is in a different package, did you fully specify it with "extends .Object" ?)
History: CompilePackage <- LoadEditPackages <- UMakeCommandlet::Main
Exiting due to error

Hello.
Make sure to include the following lines under [Editor.EditorEngine] in UCC.ini
EditPackages=Core
EditPackages=Engine
EditPackages=IGEffectsSystem
EditPackages=IGVisualEffectsSubsystem
EditPackages=IGSoundEffectsSubsystem
EditPackages=Editor
EditPackages=UWindow
EditPackages=GUI
EditPackages=SwatEd
EditPackages=IpDrv
EditPackages=Scripting
EditPackages=AICommon
EditPackages=Tyrion
EditPackages=SwatAIAwareness
EditPackages=SwatAICommon
EditPackages=Gameplay
EditPackages=RWOSupport
EditPackages=SwatGame
EditPackages=SwatEquipment
EditPackages=SwatObjectives
EditPackages=SwatProcedures
EditPackages=SwatGui
ServerActors=Utils.Package

Thx for fast reply.
On SWAT4 (Content) everything is ok now.

Log: Waiting for file streaming thread to finish...
Uninitialized: Name subsystem shut down
Uninitialized: Log file closed, 05/29/15 17:29:44

It looks you are using the wrong Source directory.

Can you give me proper one?

Log: Waiting for file streaming thread to finish...
Uninitialized: Name subsystem shut down
Uninitialized: Log file closed, 06/02/15 10:46:39


access denied when i try to connect

how do i fix it

Please, add your RDP users to «Remote desktop users» group.


fix: multiple named tuple result from contract

Codecov Report

Merging #1333 (2123544) into main (e2f00f0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #1333   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          314       314
  Lines        21447     21451    +4
=========================================
+ Hits         21447     21451    +4

Impacted Files
Coverage Δ

internal/blockchain/ethereum/ethereum.go
100.00% <100.00%> (ø)

This PR changes the contract between Firefly core and the ethereum blockchain connector (EVMConnect and ETHConnect) by relaxing the schema validation when reading the result of a contract query operation.
Before this PR, FF core relied on the result being in the format "output": {} now it can accept any arbitrary map of string to any (interface{}). Yet as the firefly interface nests the result in an "output" we still do it but in ff instead

Resolves https://github.com/hyperledger/firefly/issues/1304


Add subscriber for pub/sub.

Implements #6.
Might need a better way to initialise the setup code.

@ddikman Can we just approve this for now? I'm having to run the script manually every day because this PR hasn't gone through and there's no subscriber function.


Generate column modification changesets

This includes:

New columns
Column modification
Dropped columns

A lot of column generation code currently exists on table. We need to move this off again and write tests to confirm this works (as well as fix existing tests). The tests we need to cover are:

Add new column when table doesn't exist
Add new column when table exists already and column doesn't
Modify an existing column when it already exists
Drop a column when it does not exist in the source but does exist in the target (and drop columns enabled in publish profile).


pd: reduce frequent redundant log output

What is changed and how it works?

Issue Number: Close #15041
What's Changed:

Related changes
Release note

/merge

@tonyxuqqi  plz help me to merge it

/merge

/merge


remove deprecated Buffer constructor use in README

Coverage remained the same at 100.0% when pulling 1ebf02c30c5f8eaf2cfaac3fb1056e239a7719b1 on graingert:patch-1 into 597a9ac7004fc0278e8080c2c2f512553c60ac0d on stream-utils:master.

Coverage remained the same at 100.0% when pulling 1ebf02c30c5f8eaf2cfaac3fb1056e239a7719b1 on graingert:patch-1 into 597a9ac7004fc0278e8080c2c2f512553c60ac0d on stream-utils:master.


Docker Compose - The bot crashes after receiving a command or a new event in the game.

Describe the bug
The bot crashes after receiving a command or a new event in the game.
To Reproduce
Steps to reproduce the behavior:

Clone project from github
Build and run project using Docker Compose
Send command to bot using discord or wait for new event
See error
Bot crash

Screenshots

Additional context
After launching the containers using Docker Compose and entering any command into the Discord bot on the server, I receive an error in the container logs:
docker-bot-2       | PrismaClientKnownRequestError:
docker-bot-2       | Invalid `this.guilds.findUnique()` invocation in
docker-bot-2       | /bot/lib/db/postgresql/repository/Guild.js:21:43
docker-bot-2       |
docker-bot-2       |   18 async findOrCreate(guildId, skipCache) {
docker-bot-2       |   19     const cache = await this.client.cache.get(`guilds:${guildId}`);
docker-bot-2       |   20     if (!cache || !!skipCache) {
docker-bot-2       |   21         let guild = await this.guilds.findUnique(
docker-bot-2       | The table `public.Guild` does not exist in the current database.
docker-bot-2       |     at An.handleRequestError (/bot/node_modules/@prisma/client/runtime/library.js:174:6929)
docker-bot-2       |     at An.handleAndLogRequestError (/bot/node_modules/@prisma/client/runtime/library.js:174:6358)
docker-bot-2       |     at An.request (/bot/node_modules/@prisma/client/runtime/library.js:174:6237)
docker-bot-2       |     at async GuildRepository.findOrCreate (/bot/lib/db/postgresql/repository/Guild.js:21:25)
docker-bot-2       |     at async InteractionHandler.run (/bot/events/InteractionHandler.js:111:21)
docker-bot-2       |     at async EventHandler.run (/bot/events/EventHandler.js:69:9) {
docker-bot-2       |   code: 'P2021',
docker-bot-2       |   clientVersion: '4.15.0',
docker-bot-2       |   meta: { table: 'public.Guild' }
docker-bot-2       | }

Additionally, when a new event occurs in the game, the bot throws an error:
docker-postgres-1  | 2023-06-20 01:04:34.825 UTC [126] ERROR:  relation "public.Guild" does not exist at character 63
docker-postgres-1  | 2023-06-20 01:04:34.825 UTC [126] STATEMENT:  SELECT "public"."Guild"."id", "public"."Guild"."guildId" FROM "public"."Guild" WHERE ("public"."Guild"."id") IN (SELECT "t0"."id" FROM "public"."Guild" AS "t0" INNER JOIN "public"."Event" AS "j0" ON ("j0"."guildId") = ("t0"."guildId") WHERE ("j0"."type" = $1 AND "t0"."id" IS NOT NULL)) OFFSET $2
docker-bot-2       | /bot/node_modules/@prisma/client/runtime/library.js:174

Hello there! So, there is a hotfix for this since we need to actually fix this in our docker-compose, it's mainly cause the tables doesn't exist yet in the DB.
The things I would recommend is, you launch the bot, even if the bot is down, you do npx prisma migrate dev --name Init in the actual folder when the database is running. Then when it's done you should be able to make it run normally.

Can you join our support ? We'll help you there.

I managed to start the bot. I'm currently working on the Docker Compose for your project ;)

Sorry to reopen this. But I have solved this problem by using this Dockerfile:
FROM node:bullseye

RUN apt-get update && \
    DEBIAN_FRONTEND=noninteractive apt-get install -y \
         git

RUN git clone https://github.com/lilith-discord-bot/lilith /build

RUN mkdir /tmp/build

RUN cp /build/package.json /tmp/build/package.json

WORKDIR /tmp/build

RUN npm install
RUN mv /tmp/build/node_modules /build/node_modules

WORKDIR /build

RUN npx prisma generate
RUN npm run build
RUN npm prune --production

RUN cp -R /build/dist /bot
RUN cp -R /build/node_modules /bot/node_modules
RUN cp /build/package.json /bot/package.json

WORKDIR /bot

COPY entrypoint.sh .
RUN chmod +x entrypoint.sh

ENTRYPOINT [ "./entrypoint.sh" ]

with this entrypoint.sh:
#!/usr/bin/env bash
cd /build
npx prisma migrate dev --name Init
cd /bot
npm start

and this docker-compose file:
version: "3"
services:
  lilith:
    container_name: lilith
    depends_on:
      - lilith-postgres
      - lilith-redis
    build:
      context: .
      dockerfile: Dockerfile
    pull_policy: build
    environment:
      - PUID=1000
      - PGID=1000
      - TZ=Europe/Brussels
      - TOKEN=${TOKEN}
      - POSTGRES_HOST=postgres
      - POSTGRES_USER=${POSTGRES_USER}
      - POSTGRES_PASSWORD=${POSTGRES_PASSWORD}
      - POSTGRES_DATABASE=${POSTGRES_DATABASE}
      - REDIS_HOST=${REDIS_HOST}
      - REDIS_PORT=63798
      - REDIS_PASSWORD=${REDIS_PASSWORD}
      - DATABASE_URL=${DATABASE_URL}
    image: lilith-discord-bot/lilith:1.0.0
    network_mode: host
  lilith-redis:
    container_name: lilith-redis
    image: redis:7.0.11
    # TODO: configure redis properly, either by using arguments or using a separate volume for configs.
    command: redis-server --requirepass ${REDIS_PASSWORD}
    ports:
      - "63798:6379"
    volumes:
      - redisdata:/data

  lilith-postgres:
    container_name: lilith-postgres
    image: postgres:13.11
    environment:
      - POSTGRES_USER=${POSTGRES_USER}
      - POSTGRES_PASSWORD=${POSTGRES_PASSWORD}
      - POSTGRES_DATABASE=${POSTGRES_DATABASE}
    ports:
      - "54322:5432"
    volumes:
      - pgdata:/var/lib/postgresql/data

volumes:
  pgdata:
  redisdata:

hope this helps

Hey, thanks to you both, we'll integrate of your solution in the actual project.
Appreciate your work!

@Piekario @glazk0 I have managed to solve this using a dockerfile that actually does git clone to download entire project inside a docker container
I think that this should be helpful for people who just want to run it like a "ready-to-go" bot
You could add an installer that just downloads the compose file, a ready-to-go .env, the dockerfile and the entrypoint and then build the bot locally


How to open bottom-view as default state without animation?

I try this :
swipeLayout.open(                             smooth =  false,                             notify =  false,                             edge =  dragEdge                         )
but nothings changed!

想创业，开网店，低投资，高收入，叫你月月赚翻，美滋滋。本店诚招代理商，包教包会。QQ:2020633271旺旺:梦凌love13

谢谢，已收到


Mouse scroll bug between accessories and mouse tweaks

When you scroll, you unequip all trinket on your mouse location

There is a setting within Accessories Config to tackle such problem by disabling slot scrolling as that is all I can do for such.

got it!


Add at least one test

It would be important for users to validate that the script has the expected behaviour. You could simply add a testing script which performs the segmentation on a dataset supplied in the repository and compare the result against a saved previous result (regression test). This would catch unexpected changes in behaviours due to different MATLAB versions or operating systems.

Hey @olivecha  👋
Thanks for bringing this up!
We've created a validation function to help users verify unexpected changes in behavior due to different MATLAB/Octave versions or operating systems.
This validation function compares results with the Test1.mat data results and should be run with the user's data of interest.
Best,
Thiago

I looked at and ran the validation function, and it does not behave like a software test...
The tests should not be run against arbitrary data (well they could to be more robust, but simpler tests would need to be run first).
For your project, a simple test would be to run the main function and compare with known output:
load("Data/Test1.mat");
fff_segmenter(Acoustic_signal,Dir_X,Dir_Y,200e3);
% Load the result
load("Segmentation results/points segmentation results Test1.mat")
% Create a copy of the Segmentation results in another folder
% like "test_data/"
load("test_data/Points segmentation results test.mat")
% Do this for each result array
test1 = all(abs(result_contour(:) - test_contour(:)) < 1e-10);
if all(test1)
    disp('Contour match test reference')
else
    disp('Test failed when comparing contour')
end

This would allow you to validate that changes in the code don't break the existing behaviour (like when accepting pull requests) and that the environments of other users provides the expected functions (various Matlab/Octave versions).

@olivecha
Understood.
I have updated the validation function to follow the type of test that you proposed.
Cheers,
Thiago

Closing as this is solved


Unable to run cypress open

I'm getting this error
Couldn't find a package.json file in "/Users/abyrd1/Code/store_project"

When calling yarn cypress open --project ./spec
I followed the instructions in the README for installing in spec/cypress

cypress-on-rails (1.13.1)
rails (<IP_ADDRESS>)

where is the package.json file located and try changing into that directory and running yarn cypress open
EG
cd spec/cypress
yarn cypress open

The package.json is at spec/package.json. When I go into the directory that works. So need to update the Procfile, which I guess is fine.
Procfile
server: CYPRESS=1 bundle exec rails s -p 5017
cypress: cd spec && yarn cypress open


Wrappers forscipy.linalgquadratic control solvers (lyapunov, ARE)

Please:

[x ] Check for duplicate requests.
[x ] Describe your goal, and if possible provide a code snippet with a motivating example.

I'm interested in implementing solve_discrete_lyapunov, solve_continuous_lyapunov, and solve_discrete_are from scipy.linalg as JAX primitives. My particular use-case is Kalman filtering -- these functions are handy for computing initial and steady-state covariance matrices, but they also have wide application in linear-quadratic control applications. There are gradients computed in this paper, https://arxiv.org/pdf/2011.11430.pdf and I've also done implementations in PyTensor here and here. I'm relying heavily on compiling pytensor graphs to JAX for high-performance scans, and not having these functions is a bit of a pain-point for me at the moment.
I didn't see these functions from a quick search of the codebase, but I just wanted to check that 1) a contribute would be welcome, and 2) they didn't exist elsewhere in the JAX ecosystem before starting a PR.

I think they're on the margin. They fail axis 1 spectacularly, but arguably pass the other axes, with varying levels of difficultly in making those arguments. I think they pass on 2, 3, 5, and arguably on 6 -- github code search finds 500-1000 snippets using each of solve_discrete_lyapunov and solve_discrete_are, so it's clearly not as popular as linalg.solve, but more widely used than  bessel_jn. I guess the weakest case is on 4, ideally I'd hope to just wrap up some calls to LAPACK for forward computation together with some gradients, but this is likely to be more complicated than I realize (hardware targeting issues? introduction of additional package requirements? I have no idea if either of these would be issues, but I can imagine that they could be).
On the other hand, quadratic control generally and Kalman filtering specifically aren't exactly niche topics in scientific computing, so I'm sure these functions would see some use if they were available. I could see them belonging in something more like jax-opt, though.


Create public Slack channel

Requested from twitter: https://twitter.com/elimallon/status/870086003809173504
@sgoings do we have the ability for something like creating a public azurecontainers.slack.com to host these channels?

I'd also recommend just using the kubernetes slack. Anybody can create new channels there.

Apparently not any more. I will contact someone and see if we can create those channels.

#draft-users and #draft-dev are now up at kubernetes.slack.com.

re-opening so we can document this in the README.


[Feature]: Change the Logo of Extension

Is your feature request related to an issue? Please describe.
Please change the logo.png. Currently, it is very detailed and not compatible with dark mode.
Describe the solution you'd like
A logo can be less detailed and something which explains the Extension.
Additional Information
No response

@savi-1311 I have changed the logo and pushed it to the master branch. You can check it out and tell me if it needs any modification.

It looks good! Closing the issue :)


Documentation spelling in README.md

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


Ennourmous border around comboxes

FMX
RX 11.0
Windows 11
GlobalUseSkia := True ;

There is very wide border around combo boxes

Hi,
I will test it more on Monday o different machines. But it seems to be related to Nvidia Quatro P620.

Hi,
Attached collected info form several computers. It seems that GeForce is a factor
SystemInfo.zip
.

@cosina Your machine have 2 monitors?

Hi @cosina,
We need to do some tests to see if the problem is solved, but none of our machines can simulate it. It would help a lot if you did these tests for us... Do you have telegram or skype?
Paulo's Telegram: https://t.me/paulocesarbot
Telegram of lib: https://t.me/skia4delphi
Paulo's Skype: paulocesarbot

Fixed in new version v3.3.0


Fixed build in the newest daily build of the .NET SDK.

Checklist

[x] I have read the Code of Conduct and the Contributing files before opening this issue.
[x] I have verified the issue this pull request fixes or feature this pull request implements is to the current release.
[x] I am using the latest stable release with the above code fix or the current main branch if code changes are present (prerelease code changes).
[x] I have debugged and tested my changes before submitting this issue and that everything should just work.

Describe the issue this fixes / feature this adds

@mergifyio rebase


Invalid YAML in trigger documentation

The documentation lists (for tags and branches)
trigger:
  tags:
    include:
    - *

Is invalid yaml. You're probably looking for:
trigger:
  tags:
    include: ['*']

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

ID: f5673ec1-c7d1-e0ac-043a-8fc3d5b226a9
Version Independent ID: b4872831-51c0-6cf8-1c56-de20beb38a80
Content: Build pipeline triggers - Azure Pipelines
Content Source: docs/pipelines/build/triggers.md
Product: devops
GitHub Login: @steved0x
Microsoft Alias: sdanie

@steved0x let me know if that's OK, ran into the same thing and since it was already reported and pretty easy, figured I'd help out.

Thanks man, I just spent like an hour on a basic task of creating a pipeline for all non master branches, thanks to misleading docs I was hitting the wall until I read your post and tried:
  branches:
    exclude: ["master"]
    include: ["*"]```

@nickbabcock @ryanmaclean Sorry that you ran into trouble, we have got this updated in our docs with a slightly different syntax:
trigger:
- master
- releases/*

and:
trigger:
  branches:
    include:
    - '*'  # must quote since "*" is a YAML reserved character; we want a string

I think I remember seeing this PR come in to the public repo, and it conflicted with a nearly simultaneous PR to the private repo (possibly after seeing this issue?) and the merge conflict was resolved in favor of the private repo.
Sorry for the trouble and for the great delay in getting back to this GH issue, and thank you for letting us know.


Add very basic support for properties (getproperty, setproperty!)

Hopefully dsl.jl is the "right place" for this code.
See https://github.com/JuliaLang/julia/discussions/43912

It depends on the exact thing you want to assert. I think the most natural thing is probably to say that reading/writing the field does not error. In the language of this package, you might write this as:
@spec Readable(T::Type, s::Symbol) begin
    @∀ x::T->x.:($s)
end
@spec Writable(T::Type, s::Symbol, fT=fieldtype(T, s)) begin
    @∀ (x::T, f::fT)->x.:($s) = f
end

In any case, I've been putting these kinds of things into the specs/ directory as a place for staging.

OK, that makes sense, feel free to close the PR (I can't).
I'm super happy you're working on this!

d'oh! That was from my GitHub test account :)


Update Rust version to 1.77.2

Bump rules_rust to make the new Rust version available in Bazel
Bump the flake to make the new Rust version available in Nix
Sync toolchains with the updated flake lockfile

Addresses CVE-2024-24576.

This change is

cc @chrisstaite-menlo


Edges missing in Graph

There are missing edges in the Apply to each container

Fixed in https://github.com/modery/PowerDocu/commit/3b0cf2dfc7fe787d523e786b1ba5f3ca00cb3518
For example, "Compose_Flight_or_Hotel" now points to the container around "Check_if_its_flight", which now also points to the container around "Check_if_location_is_empty"


DataValidationError is not reset when OneWayToSource binding change from invalid value to valid value

Describe the bug
Bind NumericUpDown.Value (nullalbe decimal) to a non-nullable decimal in view model. Initially it shows data validation error, which is correct behavior. But when I input a valid number, the error still stays.
This happens when mode is OneWayToSource
To Reproduce
https://github.com/rabbitism/AvaloniaIssueRepro/tree/main/OneWayToSourceBindingError
Expected behavior
Error is cleared when value is valid.
Avalonia version
latest nightly
OS
Windows
Additional context
No response

OneWayToSource means, that the UI only updates your ViewModel, but the UI will not recieve the new Value or validation from your ViewModel as you opted out for that. So I feel like this is by-design.


Signifier emojis should be allowed in the description

Expected Behavior
It should act as any emoji.
Current Behavior
The text after the emoji disappears in preview mode unless it's a date.
Steps to Reproduce
Copy-paste this in obsidian with the tasks plugin activated, then turn on preview mode.
- [ ] 📅 test

Context (Environment)

Obsidian version: 0.12.4
Tasks version: 1.1.1
[x] I have tried it with all other plugins disabled and the error still occurs

Possible Solution

Thank you for the report @charle36 . This is something I have been meaning to address for a while. I didn't have time, yet, but this will get fixed eventually.

Fixed in the latest 1.1.2 release!

Thanks! Amazing job!


Chrome. Color picker show on wrong position when user scroll (long menu)

If dat.gui has scrollbar because of long menu, Color Picker is shown wrong position.
User cannot pick color because it disappear when mouse just moves

finally fixed! thanks for flagging the problem


[BUG]: VSTest Failure: No tests were discovered while attempting to rerun failed tests

Task name
VSTest_ef087383-ee5e-42c7-9a53-ab56c98420f9
Task version
2.219.0
Environment type (Please select at least one enviroment where you face this issue)

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

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Microsoft Windows NT 6.2.9200.0
Task log
2023-04-04T22:54:01.1805798Z ##[section]Starting: Run Framework Integration Tests
2023-04-04T22:54:01.2060851Z ==============================================================================
2023-04-04T22:54:01.2061199Z Task         : Visual Studio Test
2023-04-04T22:54:01.2061292Z Description  : Run unit and functional tests (Selenium, Appium, Coded UI test, etc.) using the Visual Studio Test (VsTest) runner. Test frameworks that have a Visual Studio test adapter such as MsTest, xUnit, NUnit, Chutzpah (for JavaScript tests using QUnit, Mocha and Jasmine), etc. can be run. Tests can be distributed on multiple agents using this task (version 2).
2023-04-04T22:54:01.2061764Z Version      : 2.219.0
2023-04-04T22:54:01.2061812Z Author       : Microsoft Corporation
2023-04-04T22:54:01.2061866Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/test/vstest
2023-04-04T22:54:01.2061988Z ==============================================================================
2023-04-04T22:54:04.2951845Z Running tests using vstest.console.exe runner.
2023-04-04T22:54:04.2952218Z ======================================================

...

2023-04-04T22:55:51.5353894Z **************** Rerunning failed tests for Test run<PHONE_NUMBER> *********************
2023-04-04T22:55:51.5354457Z Max attempts: 3; Current attempt: 1; Failed test cases threshold: 20; Failed test cases max limit: 0
2023-04-04T22:55:51.5355351Z [command]C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\IDE\Extensions\TestPlatform\vstest.console.exe "@D:\a\_temp\fz5rargpawg.tmp"
2023-04-04T22:55:51.6336102Z Microsoft (R) Test Execution Command Line Tool Version 16.11.0
2023-04-04T22:55:51.6336780Z Copyright (c) Microsoft Corporation.  All rights reserved.
2023-04-04T22:55:51.6406081Z vstest.console.exe /TestCaseFilter:"FullyQualifiedName=Microsoft.Azure.Maps.DataPipeline.StatusManagementIntegrationTests.StatusManagerTests.StatusManagement_InitialJobTimesOutButStillCompletes (Succeeded)"
2023-04-04T22:55:51.6406794Z "D:{redacted}.dll"
2023-04-04T22:55:51.6407904Z /Settings:"D:\a\_temp\4k2fcp52xtu.tmp.runsettings"
2023-04-04T22:55:51.6408353Z /Logger:"trx"
2023-04-04T22:55:51.6408738Z /TestAdapterPath:"D:{redacted}\Tests"
2023-04-04T22:55:52.2944742Z Starting test execution, please wait...
2023-04-04T22:55:53.8144268Z 2.707
2023-04-04T22:55:53.9523289Z A total of 1 test files matched the specified pattern.
2023-04-04T22:55:59.4927275Z ##[error]Incorrect format for TestCaseFilter Missing Operator '|' or '&'. Specify the correct format and try again. Note that the incorrect format can lead to no test getting executed.
2023-04-04T22:55:59.7278102Z No test matches the given testcase filter `FullyQualifiedName=Microsoft.Azure.Maps.DataPipeline.StatusManagementIntegrationTests.StatusManagerTests.StatusManagement_InitialJobTimesOutButStillCompletes (Succeeded)` in D:{redacted}.dll
2023-04-04T22:56:00.1607663Z Results File: D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx
2023-04-04T22:56:00.1708287Z ##[error]Test Run Failed.
2023-04-04T22:56:00.1907680Z Vstest.console.exe exited with code 1.
2023-04-04T22:56:00.1908252Z **************** Completed test execution *********************
2023-04-04T22:56:00.1956047Z Test results files: D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx
2023-04-04T22:56:00.2045861Z No Result Found to Publish 'D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx'.
2023-04-04T22:56:00.2046510Z Updating test results: 0
2023-04-04T22:56:00.2464521Z Publishing test results to test run '1420630129'.
2023-04-04T22:56:00.4364334Z TestResults To Publish 0, Test run id:1420630129
2023-04-04T22:56:00.4365149Z Updated test results: 0
2023-04-04T22:56:00.4370959Z Publishing Attachments: 1
2023-04-04T22:56:00.6023453Z ##[error]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.
2023-04-04T22:56:00.7415454Z Execution Result Code 1 is non zero, checking for failed results
2023-04-04T22:56:07.2025868Z Retry count is exhausted while performing System.Threading.Tasks.Task`1[Microsoft.TeamFoundation.TestManagement.WebApi.TestRunStatistic] <GetTestRunSummaryByOutcome>b__0()
2023-04-04T22:56:07.2196457Z Failure attempting to call the test run summary and retry counter is exhausted Microsoft.TeamFoundation.DistributedTask.Task.TestResults.RetryCountExhaustedException: Max retries count: 3 is exhausted ---> Microsoft.TeamFoundation.TestManagement.WebApi.TestObjectNotFoundException: Run summary is not available, It is recommended to try again after 1 sec.
2023-04-04T22:56:07.2197308Z    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.<HandleResponseAsync>d__53.MoveNext()
2023-04-04T22:56:07.2197656Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2198167Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2198508Z    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
2023-04-04T22:56:07.2198882Z    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.<SendAsync>d__51.MoveNext()
2023-04-04T22:56:07.2199359Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2199678Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2199985Z    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
2023-04-04T22:56:07.2200790Z    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.<SendAsync>d__47`1.MoveNext()
2023-04-04T22:56:07.2201114Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2201393Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2201720Z    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
2023-04-04T22:56:07.2202933Z    at Microsoft.VisualStudio.Services.WebApi.VssHttpClientBase.<SendAsync>d__28`1.MoveNext()
2023-04-04T22:56:07.2203263Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2203544Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2203868Z    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
2023-04-04T22:56:07.2204199Z    at Microsoft.TeamFoundation.TestClient.PublishTestResults.TestRunPublisher.<GetTestRunSummaryAsync>d__9.MoveNext()
2023-04-04T22:56:07.2204724Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2205196Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2205544Z    at Microsoft.TeamFoundation.DistributedTask.Task.TestResults.RetryHelper.<Retry>d__1`1.MoveNext()
2023-04-04T22:56:07.2205900Z    --- End of inner exception stack trace ---
2023-04-04T22:56:07.2206251Z    at Microsoft.TeamFoundation.DistributedTask.Task.TestResults.RetryHelper.<Retry>d__1`1.MoveNext()
2023-04-04T22:56:07.2206942Z --- End of stack trace from previous location where exception was thrown ---
2023-04-04T22:56:07.2207235Z    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
2023-04-04T22:56:07.2207580Z    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
2023-04-04T22:56:07.2207947Z    at Microsoft.TeamFoundation.DistributedTask.Task.TestResults.TestRunSummaryManager.<GetTestRunSummaryByOutcome>d__3.MoveNext()
2023-04-04T22:56:07.2325552Z ##[warning]Unable to fetch test run summary from the Test Management Server due to exception: One or more errors occurred.
2023-04-04T22:56:07.2330147Z PERF WARNING: TestRunCompletionPhase:GetTestRunOutcomeFromPublishTestResultLibrary: took 6398.4196 ms
2023-04-04T22:56:07.3239053Z Completed TestExecution Model...
2023-04-04T22:56:07.3496525Z ##[warning]Vstest failed with error. Check logs for failures. There might be failed tests.
2023-04-04T22:56:07.3498140Z ##[error]Error: The process 'D:\a\_tasks\VSTest_ef087383-ee5e-42c7-9a53-ab56c98420f9\2.219.0\Modules\DTAExecutionHost.exe' failed with exit code 1
2023-04-04T22:56:07.3508637Z ##[error]Vstest failed with error. Check logs for failures. There might be failed tests.
2023-04-04T22:56:07.3624343Z ##[section]Finishing: Run Framework Integration Tests

Relevant log output
2023-04-04T22:55:51.5353894Z **************** Rerunning failed tests for Test run<PHONE_NUMBER> *********************
2023-04-04T22:55:51.5354457Z Max attempts: 3; Current attempt: 1; Failed test cases threshold: 20; Failed test cases max limit: 0
2023-04-04T22:55:51.5355351Z [command]C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\IDE\Extensions\TestPlatform\vstest.console.exe "@D:\a\_temp\fz5rargpawg.tmp"
2023-04-04T22:55:51.6336102Z Microsoft (R) Test Execution Command Line Tool Version 16.11.0
2023-04-04T22:55:51.6336780Z Copyright (c) Microsoft Corporation.  All rights reserved.
2023-04-04T22:55:51.6406081Z vstest.console.exe /TestCaseFilter:"FullyQualifiedName=Microsoft.Azure.Maps.DataPipeline.StatusManagementIntegrationTests.StatusManagerTests.StatusManagement_InitialJobTimesOutButStillCompletes (Succeeded)"
2023-04-04T22:55:51.6406794Z "D:{redacted}.dll"
2023-04-04T22:55:51.6407904Z /Settings:"D:\a\_temp\4k2fcp52xtu.tmp.runsettings"
2023-04-04T22:55:51.6408353Z /Logger:"trx"
2023-04-04T22:55:51.6408738Z /TestAdapterPath:"D:{redacted}\Tests"
2023-04-04T22:55:52.2944742Z Starting test execution, please wait...
2023-04-04T22:55:53.8144268Z 2.707
2023-04-04T22:55:53.9523289Z A total of 1 test files matched the specified pattern.
2023-04-04T22:55:59.4927275Z ##[error]Incorrect format for TestCaseFilter Missing Operator '|' or '&'. Specify the correct format and try again. Note that the incorrect format can lead to no test getting executed.
2023-04-04T22:55:59.7278102Z No test matches the given testcase filter `FullyQualifiedName=Microsoft.Azure.Maps.DataPipeline.StatusManagementIntegrationTests.StatusManagerTests.StatusManagement_InitialJobTimesOutButStillCompletes (Succeeded)` in D:{redacted}.dll
2023-04-04T22:56:00.1607663Z Results File: D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx
2023-04-04T22:56:00.1708287Z ##[error]Test Run Failed.
2023-04-04T22:56:00.1907680Z Vstest.console.exe exited with code 1.
2023-04-04T22:56:00.1908252Z **************** Completed test execution *********************
2023-04-04T22:56:00.1956047Z Test results files: D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx
2023-04-04T22:56:00.2045861Z No Result Found to Publish 'D:\a\_temp\TestResults\VssAdministrator_fv-az364-167_2023-04-04_22_56_00.trx'.
2023-04-04T22:56:00.2046510Z Updating test results: 0
2023-04-04T22:56:00.2464521Z Publishing test results to test run '1420630129'.
2023-04-04T22:56:00.4364334Z TestResults To Publish 0, Test run id:1420630129
2023-04-04T22:56:00.4365149Z Updated test results: 0
2023-04-04T22:56:00.4370959Z Publishing Attachments: 1
2023-04-04T22:56:00.6023453Z ##[error]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.

Aditional info
We have experienced our test pipeline fail intermittently due to this issue.

Pls use **$(BuildConfiguration)**tests.dll under VsTest-testAssemblies -->paste under Test files.It will start taking all the tests

Still fails for us on Classic Pipeline . VsTest3.
Looks like the same issue https://github.com/microsoft/azure-pipelines-tasks/issues/16990 we're still waiting to be resolved.


⚠ Mobitopup has degraded performance

In 00768cb, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 12136 ms

Resolved: Mobitopup performance has improved in a97fded.


replace [Discrete]Distribution trait for traits for moments and entropy separately

feat: define new traits Covariance, Entropy, and CentralMoment
feat!: replace Distribution with new traits for binomial
feat!: replace Distribution with new traits for Gamma
feat!: replace Distribution with new traits for uniform
feat!: replace DiscreteDistribution for new traits for negative binomial
feat!: replace Distribution for new traits with F-distribution
feat!: replace Distribution for new traits for laplace
feat!: replace Distribution for new traits for normal
feat!: replace Distribution for new traits for multivariate vector distributions

@FreezyLemon this removes the statistics::DiscreteDistribution and statistics::Distribution traits.
I'm relying a bit on unimplemented here - was wondering if I should have meaningful associated types for those trait methods as some are unit.

Would be nice to have the Never ! type, but that's not stable. Will do this for now. I'll write out the tests next.

Okay, I'll write the tests for Covariance now, but I realized that I didn't have to find a way to express in associated type if something is unimplemented, I could just separate the traits.
Aside: I learned about ast-grep and it was useful for the rewrite.

I also dropped the sparse trait method from the Covariance trait and renamed the trait to CovarianceMatrix perhaps there's another name more suitable, but the intent was to support various types as if they expressed covariance as an operator and how it acts on states. It felt a little ambiguous across types unlike the dense method.

Not sure what I'm misunderstanding about colorize/whiten and its testing regarding this.
Since square roots of matrices aren't unique, I'm relying on the quadratic forms like seen in distribution functions that have covariance matrices, like in the gaussian $-\frac{1}{2}x^T \Sigma^{-1}x$
This is also how scipy does their tests


Little code clean up.

Little code clean up.

Coverage increased (+0.01%) to 69.693% when pulling 16f865987dd9baf1a4790bbd9b869670048c2d9f on AndrienkoAleksandr:codeCleanUp into 618768c7ed727136db17d78ed07ce4a84fbbe247 on sourcelair:master.

Coverage increased (+0.01%) to 69.693% when pulling d1712424c87434c9342adbe60bfaafc8d42a4358 on AndrienkoAleksandr:codeCleanUp into 618768c7ed727136db17d78ed07ce4a84fbbe247 on sourcelair:master.

Perfect 👌

Coverage increased (+0.01%) to 69.679% when pulling 401886c7c336d82a3d98091d21e9000a9dcf90d0 on AndrienkoAleksandr:codeCleanUp into 4eac7661c2f06283c79ac9a8e026fb92331cf0f1 on sourcelair:master.


migrate the tutorial to LLVM 6.0 (and -fsanitize=fuzzer)

update the docs
update the get_clang script
start using -fsanitize=fuzzer by default
don't build libFuzzer from source (by default, and in the tutorial docs)
rebuild the tutorial docker files (once everything else is done, I'll do that)

@kcc: Last step is to rebuild the tutorial docker image and upload.

docker images updates.
thanks!


update all roles, replace module

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @yalait this PR seems removing all the necessary md files.


Import specific polyfills

Currently, this script is importing the whole babel-polyfill https://github.com/Grsmto/simplebar/blob/0a7e1877b4f36862f45cb0471fd19811d243e1ae/src/simplebar.js#L5 which results in a 81.2 kb minified dist file.
IMO there are lots of unneeded polyfills in babel-polyfill package, it would be much better to import necessary polyfills only:
import 'core-js/fn/array/from';
...

What do you think about?

We are using (babel-preset-env)[https://github.com/babel/babel-preset-env#usebuiltins-entry] so unless I'm using it wrongly, it should not import every polyfills but only the ones required.
The script size seems big tho, I'll investigate.
Ref #112

babel-preset-env imports all polyfills that target environment requires. You might need babel-transform-runtime plugin to import polyfills that your code requires, or wait for https://github.com/babel/babel-preset-env/issues/246 😃.

Ahhhh damn :p
Thanks very much for this, seems like I rushed this pretty badly in the latest release.
I'll take a serious look at it! :)


Use editPost with isCached: true

See https://developer.wordpress.org/block-editor/explanations/architecture/entities/#cached-changes
Use isCached: true while streaming and then isCached: false (the default) when finalized.
https://github.com/swissspidy/ai-experiments/blob/d1977e1bec3bbdea3867a487709c109b99beb3bc/packages/edit-post/src/menu.tsx#L167

For blocks there is unstableMarkNextChangeAsNotPersistent


[WIP] docs: protocol table

Description
Add supported protocols and neural network layers to readme.
Closes #248.
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests
[x] Support Protocol table
[ ] Support Operations table
[ ] Support Torch neural network table

@danielorihuela Make the operations rowise like in the issue: https://github.com/OpenMined/SyMPC/issues/248

NN lay
Okay, let me put them here, while I am still working out which operations have each protocol. I will move them at the end.

@kamathhrishi I think I have the operations more or less. I extracted the relations from the protocols and the type of tensor they used. Replicated or Share. However, I was not able to know which protocols supports sigmoid, tanh, relu, exp, softmax, reciprocal,maxpool and argmax.

@danielorihuela Removing WIP so that others can feel free to review it

I tried to apply all the specified improvements and divide the methods by visibility: public and private. I took as premise that the operation works for both, malicious and semi-honest, for the protocols that have both security guarantees. If that is not the case I think we could improve the table by adding it next to the protocol. Example: - FSS (Semi-honest)

@kamathhrishi I added your comments and rebased the branch with the changes in main.

Good work @danielorihuela


Update to<EMAIL_ADDRESS>

Is dependency on<EMAIL_ADDRESS>necessary?

Yes. It fails with newer versions, see also #144
This has to do with the version of Clang.jl used to generate the C wrapper.
One could regenerate the C wrapper from the SCIP header files with a newer version of Clang and then also use a more recent version of CEnum. See #77.


Avoid using default config while requesting available GPUs

This is a port of https://github.com/openai/baselines/pull/863. A copy of my comment for the original PR is below.
Previously, if one called get_available_gpus before the first call to get_session, subsequent attempts to create a session would result in the custom config being ignored. In particular, this would cause Tensorflow to allocate all GPU memory, even though sess._config would happily report that allow_growth is set to True.
A call to get_available_gpus before all other code could be used to assert that the code is running on a GPU at all:
if __name__ == '__main__':
    assert get_available_gpus()

Also, I took the liberty to make the link to the StackOverflow answer a little bit shorter.

Hello,
it seems that this feature is not used in the codebase, maybe we should delete it. What do you think @hill-a , @erniejunior , @AdamGleave ?

I'd be in favour of deleting this method. Does not seem hard for the user to write their own if they need one, and it's not obviously connected to Stable Baselines more than any other ML library.

@Pastafarianist could you change your PR, so that it removes that function ? (instead of fixing it)

Agreed and removed the function. How should I record this in the changelog? It's currently unclear where to put changes from an unreleased version.

How should I record this in the changelog? It's currently unclear where to put changes from an unreleased version.

Good question, just create a new section "Pre-release" (cf HER pull request), I will merge the two when HER PR is finalized ;)


[Tachyon 470] Modify getClass of TachyonConf to take no default value

https://tachyon.atlassian.net/browse/TACHYON-470
getClass is still taking a default value. This PR aims to remove the useage of this method and only set class properties in configuration file not in code.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4267/
Test PASSed.

+1 LGTM


WebGLRenderer: review / refactor updating renderer props.

[ ] Close #88
[ ] Close #89

for both #88 and #89 should be closed by using SvelthreeProps (sProps) for updating WebGLRenderers props object:

Do we even have any test using props attribute?
Shouldn't  props attribute be the same thing as config attribute?

Anyway, handling WebGLRenderer props is currently definitely out of sync / outdated compared to other components and should be refactored.


Add ROCm PyTorch Official RunPod Image

Added official PyTorch images for ROCm

ROCm
PyTorch
Ubuntu
Python

5.6
2.0.1
20.04
3.8

5.7
2.0.1
22.04
3.10

6.0
2.1.1
20.04
3.9

6.0.2
2.1.2
22.04
3.10

6.1
2.0.1
22.04
3.10

6.1
2.1.2
22.04
3.10

is this already tested? do we know how much it increases the container size?

Oh lol we did the exact same thing with removing two dockerfiles @justinmerrell runpod/containers#b53c1a8d046105e509b548527070e0b86b18a371


Update README file

Ipsum Text

Filler text
Got from <lipsum.com>

Updates
I updated Readme file. Please check and accept.

Comments
That's okay 👍


Willow framework crash

Environment

iOS version: 13
Xcode version: 11.3
Azure.iOS SDK:
Azure.iOS SDK version: 0.5.0
CocoaPods or Carthage: Carthage

Bug Summary
When add a dependency using Carthage, my app crashes with an exception "Willow: no matching architecture in universal wrapper"
Willow framework has only armv7 arm64, that couldn't be used for simulator.
Steps to Reproduce
Code snippet
// TODO: add code here (if applicable)

Expected Behavior
Actual Behavior
Screenshots

Closing this issue - it seems it's most likely to be an issue with the Willow framework itself, so I would try updating that dependency. If it is in fact an issue with our library, unfortunately we aren't planning any further changes to the Track 1 SDK.


Check whole protein in check_validty

Should trigger error if most hydrogens are bad predictions, like from solvent or units.

Fixed by 693dfec18d962edcc33739aec5f62ca2af1b3c1f


[5.6] Cast model attribute to any class

Currently, you can cast a model attribute to a primitive type or JSON. This PR will let model attributes to cast to any class. Here's an example:
Class MyModel extends EloquentModel {
	protected $casts = [
		‘age’ => ‘int’,
		‘settings’ => Settings::class
	];
}

Class Settings implements Castable{
	protected $settings;

	public function __construct($value)
	{
		$value = json_decode($value);
		if ($value === false) {
			throw new InvalidArgumentException(“Invalid value”);
		}

		$this->settings = $value;
	}

	public static function fromModelValue($value) {
		return new static($value);
	}
}

$myModel = MyModel::first();

dump($myModel->settings);

// will return an instance of Settings class.

Casting model attributes to classes PRs raised couple of times already, but all the time they were rejected. This will be useful to create Value Objects.

How is this attempt better than anything previously attempted?

This implementation assumes that I am the creator of the value objects and can change them. I cannot use this if I want someone else's value objects.
How would this implement value objects that requires two columns? For example; a Money class that has both an amount and a currency.
How would this handle the reverse; User::where('email', '=', $emailObjects)?

@sisve
I wasn't following all the previous propositions. This is just something I did a couple times in a Laravel app and I thought it could be useful to have out of the box instead of falling back to mutators.

If you're not the creator of the value object, create a factory class for the value. Which can build the value from different inputs.

Normally, a model attribute is just one column not too. So in this case you use your factory method to decide how to build the value object.

This can be in another PR maybe. We can just call toString on the object?

@Whyounes My implementation of something similar can be found at https://github.com/hnhdigital-os/laravel-model-schema
It's more than just attribute casting though, I've worked in a different approach to casts/hidden etc into a single schema array.

If you're interesting in catching up on previous discussions, start at https://github.com/laravel/framework/pull/18229


[Snyk] Security upgrade mongoose from 7.3.1 to 7.3.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

798/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 8.1
Prototype Pollution SNYK-JS-MONGOOSE-5777721
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: mongoose
    The new version differs by 85 commits.

0cb0757 chore: release 7.3.4
aef309e Merge branch '6.x'
e9eb8ab chore: release 6.11.3
688da8f test: fix flakey tests, remove test for #9597 because it affects global state and fails intermittently on deno
4f264a8 test: fix tests re: #13317
9616af7 fix(schema): correctly handle uuids with populate()
305ce4f fix: avoid prototype pollution on init
90d84fd chore: release 7.3.3
82b9c3c style: fix lint
02699fa Merge branch 'vkarpov15/avoid-prototype-pollution'
2188458 Merge pull request #13577 from Automattic/vkarpov15/gh-13529
e94ca23 Merge pull request #13589 from Automattic/vkarpov15/gh-13582
1a998e2 Merge pull request #13588 from Automattic/vkarpov15/gh-13575
cc722a1 test: add coverage for constructor properties
e29578d fix: avoid prototype pollution on init
422dff4 perf: avoid adding all doc array subpaths when 1 path is modified
eb9a4f7 fix(document): clean up all array subdocument modified paths on save()
8b8f37b types: apply suggested alternative handling for TOverrides = any
79a4bda Merge branch 'master' into vkarpov15/gh-13529
22b1e25 fix(populate): correctly set `populatedModelSymbol` on documents populated using `Model.populate()`
b336ed8 chore: release 7.3.2
7ad5eef Merge pull request #13579 from Automattic/vkarpov15/gh-13531
c01cff6 Merge pull request #13581 from hasezoey/updateDev
7b3b027 chore(dev-deps): bump mkdirp from 2.1.3 to 3.0.1

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

Issue solved in #111


Implement API keys delete functionality.

r+wc

Thanks. I removed a forgotten inline style and tweak a bit the spaces.


Remove: 7S2P.Effie version 3.6.3

403 Access Denied
Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://download.effie.pro/effie/effieapp_setup_3.6.3.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 872)


Feat/add streaming to frontend component

Adds streaming capability to frontend.

This PR adds support for parsing streamed response (in real time) 🔥
https://github.com/Azure-Samples/azure-search-openai-javascript/assets/1699357/0d6ebe9a-3a93-4d93-9e6a-c13d28e93a47


Heroic vs SuperHero

Endurance Use differences 5/STR vs 10/STR.  Weight already implemented.  Possibly other differences.


Possible bug with type inference

I asked a question at the ReScript forum. The answers suggested that there could be a bug in the extension, hence this issue.
I have created a repo containing the example here. Please have a look in FormTest.res to see the type inference I'm wondering about.
Can you confirm that it's a bug, or is it because I'm missing to open the other module or something else?
Recreating the example in the ReScript playground shows that the type is inferred.
Thanks for considering this.

Closing this as a "duplicate" (well, not really) of https://github.com/rescript-lang/rescript-vscode/issues/310. Or rather, if I understand correctly, https://github.com/rescript-lang/rescript-vscode/issues/310 covers this as well. Feel free to reopen if I'm wrong.

Closing this as a "duplicate" (well, not really) of https://github.com/rescript-lang/rescript-vscode/issues/310. Or rather, if I understand correctly, https://github.com/rescript-lang/rescript-vscode/issues/310 covers this as well. Feel free to reopen if I'm wrong.

Happy to hear this is resolved in another issue. But it looks like the reference is back to this issue, so not sure what the reference is.


Allow disabling runtime-specific profilers

Description
Allow disabling the runtime-specific profilers via --no-java and --no-python.
Since there are only 2 runtime-specific profilers now, I didn't invest too much in the "NoopProfiler" infrastructure. Later on we can replace those sets of funny ifs with a proper class registry for profilers (profilers can be @register("java") and then we can decide if we use the registered class or NoopProfiler at 1 spot)
Motivation and Context
Give more control to the user. It might be useful to disable runtime-specific profilers in case either of them cause issues / compatibility problems. If need the results only from part of them, disabling the others (especially as we grow to add more profilers) might be helpful in such cases.
How Has This Been Tested?
Tested running locally with the 4 combinations of --no-java, --no-python and verified that Java/Python profilers are not initialized when not requested.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the relevant documentation.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Build failure is fixed in https://github.com/Granulate/gprofiler/pull/56


[pigeon] upgrade analyzer package

Use case

When I use the freezed package and pigeon, I can't install the latest version of freezed because the version of analyzer that pigeon depends on is lower than 3.0.0.

Thanks, looking forward to the update

@gaaclarke FYI


Add sample template of systemd configuration

@nak3  why using /etc/sysconfig?

Hi @f0
It depends on the policy of distribution.
I am a RPM(Fedora/CentOS/RHEL) person, so I wrote the example to use under /etc/sysconfig/. (it is the same as flannel or kubernetes RPM).
If majority of the people prefer /etc/torus/ or some other directory, I will update the example.

Eh, works for me.


new logo for fuzzy search

hey dude, i made a logo for you

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


docs: initial release.md

What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
Although the releases process is not fully implemented, I believe it's time to add release instructions since we are planning to tag releases very soon.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
The document reflects the current development state. Most of the manual checks described in the document may be automated in future.
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

Thanks @leogr ! This is really good!
What i am looking forward is an automation job that, as soon as the +driver release gets tagged, opens up a PR on test-infra driverkit to add the new driverversion to the driver build matrix :)
I hope we will be able to have something like this for the 0.33 release!

Looking good! Left a few comments, mostly on wording, but otherwise very excited to see versioning coming to the libs repo!

I loved your nitpicking :star_struck:  (especially because I wrote the document on Friday afternoon :smile_cat: )
Thank you, @Molter73
It should be all right now.

Btw, this PR :point_right: https://github.com/falcosecurity/libs/pull/196 should help too.

/unhold


CanvasRenderingContext2D drawImage ImageBitmap source inaccurate for Safari

For this feature: https://caniuse.com/mdn-api_canvasrenderingcontext2d_drawimage_imagebitmap_source_image
The data regarding Safari seems to be inaccurate, as I am able to use an ImageBitmap as the source of a drawImage call for a 2d canvas context. I've made an example here: https://jsfiddle.net/chocolatkey/jf9b5hoy

I'm not sure how far back in Safari this goes, as I don't have an older version available. But at the very least it's the case in 16.1 where it's currently shown as "Not supported".

The data for this particular feature comes via MDN (here), so I'm afraid you have to open an issue over at https://github.com/mdn/browser-compat-data.
Feel free to leave this one open and cross-reference, though!


MurmurHash3 hash128 crashing

MurmurHash3 hash128 method crashes for strings of length greater that 15.

Hi there, I understand that this is 2 year old issue but I recently needed a MurmurHash3 implementation in Swift and this one seemed the best despite this bug. I've made a fork cleaning it up a bit, porting to Swift 4 and extracting into separate repository: https://gitlab.com/shadone/MurmurHash3


gmock.h:10090:60: runtime error: member call on null pointer of type 'const struct ResultHolder'

Compiling with gcc-6 leads to segfault in the testsuite:
markus@x4 build % make test
Running tests...
Test project /var/tmp/cppformat/build
Start 1: assert-test
1/8 Test #1: assert-test ......................   Passed    0.01 sec
Start 2: gtest-extra-test
2/8 Test #2: gtest-extra-test .................   Passed    0.01 sec
Start 3: format-test
3/8 Test #3: format-test ......................***Exception: SegFault  0.00 sec
Start 4: format-impl-test
4/8 Test #4: format-impl-test .................   Passed    0.00 sec
Start 5: printf-test
5/8 Test #5: printf-test ......................   Passed    0.01 sec
Start 6: util-test
6/8 Test #6: util-test ........................***Exception: SegFault  0.01 sec
Start 7: posix-mock-test
7/8 Test #7: posix-mock-test ..................***Exception: SegFault  0.01 sec
Start 8: posix-test
8/8 Test #8: posix-test .......................   Passed    0.01 sec
63% tests passed, 3 tests failed out of 8
This happens because undefined behavior is invoked:
markus@x4 bin % ./util-test
[==========] Running 35 tests from 6 test cases.
[----------] Global test environment set-up.
[----------] 1 test from AllocatorTest
[ RUN      ] AllocatorTest.AllocatorRef
/var/tmp/cppformat/test/../gmock/gmock/gmock.h:10090:60: runtime error: member call on null pointer of type 'const struct ResultHolder'
[1]    15511 segmentation fault  ./util-test

I've opened an upstream issue:
https://github.com/google/googletest/issues/705

Thanks for reporting. I've added -fno-delete-null-pointer-checks to tests as a workaround, but I don't have gcc-6 at hand. Could you by any chance check if it solves the issue?

Could you by any chance check if it solves the issue?

Yes, indeed -fno-delete-null-pointer-checks fixes the issue.
Thanks.


Tenants who have no collegist status cannot apply for collegist

Again discovered by @AdamSulan. After following the link, the site redirects them to the "tell me when you are going to move"-screen, and to the home page after saving that. If they try again, the same happens.

They should see an apply as a collagist button on the bottom of that page. Can not you see that @viktorcsimma @AdamSulan ?

It looks like this:

I checked. It should only happen with Adam because he is already an alumni... I'll fix it


Fixed GIL release issue with Python System and Python TestFixture.

…gned-off-by: Amal Dev Haridevan<EMAIL_ADDRESS>
🦟 Bug fix
Fixes #
Add explicit scoped acquire and release of GIL, so that Python Systems can be executed from a Python Server (run using TestFixture for example).
Summary

Python system plugin, which is attached to my model cannot be run using TestFixture in Python. This is due to GIL not being explicitly released in the PythonSystemLoader.
To overcome this issue, we need to do a scoped_acquire of GIL explicitly and then perform scoped_release after each of the system methods, namely, PreUpdate, Update, PostUpdate of the PythonSystem, so the GIL can be accessed by the TestFixture after.
As a safety mechanism, for future, I also added the scoped_acquire and release of GIL within the pybind code for TestFixture.
Checklist

[X] Signed all commits for DCO
[X] Added tests
[X] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[X] Consider updating Python bindings (if the library has them)
[X] codecheck passed (See contributing)
[X] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.
🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@AmalDevHaridevan just wanted to give you a heads-up that I won't be able to review this for the next couple of weeks due to ROSCon.

@AmalDevHaridevan just wanted to give you a heads-up that I won't be able to review this for the next couple of weeks due to ROSCon.

Hi Azeey, any updates on the pull request?


Multiple Brid.gy URL:s pinged for same tweet

When debugging a duplicate post issue in my endpoint (https://github.com/voxpelli/webpage-webmentions/issues/35) I found that multiple different Brid.gy URL:s had been pinged to my endpoint for the same tweet and thus that I had imported the tweet multiple times as I only do deduplication on a normalized source URL, not on the actual content of the Microformats 2 data itself.
I searched the issues and PR here and found no mentioning of any issues around it so opening a new one.
These pings was received for http://www.kevinmarks.com/decentralweb2016-06-08.html at 2016-06-12 03.26 (unsure what timezone, but I guess UTC).
The first three URL:s pinged was:

https://brid-gy.appspot.com/post/twitter/kevinmarks/741827388313456640
https://brid-gy.appspot.com/post/twitter/kevinmarks/741826520285470720
https://brid-gy.appspot.com/post/twitter/kevinmarks/741826676758216705

interesting. thanks for filing!
looks like<PHONE_NUMBER>85470720 is the original tweet that quotes this @kevinmarks tweet, and<PHONE_NUMBER>13456640 and<PHONE_NUMBER>58216705 are RTs of it. example RT tweet object from the API:
"id_str": "741827388313456640",
"retweeted_status": {
  "created_at": "Sun Jun 12 02:56:39 +0000 2016",
  "id":<PHONE_NUMBER>85470700,
  "id_str": "741826520285470720",
  "text": "A great guide to the full videos https://t.co/DhQoGDv3MT #DWebSummit. We want u to share, reuse open knowledge. https://t.co/cddL6GC8Nr",
  "..."
}

we try to avoid backfeeding RTs of quote and link tweets, but this might be a bug in that. i'll investigate.
...having said that, i'd encourage you to de-dupe on u-url instead of normalized source URL. i saw your discussion in https://github.com/voxpelli/webpage-webmentions/issues/35#issuecomment-232399605 , and know it'd probably be a big switch, but it's arguably the right thing to do. it looks like https://indieweb.org/Webmention-brainstorming#de-duplication agrees.
re:

that would in a way be a Brid.gy specific fix as such a dedupe would enable a third party to block a true Webmention by being the first to ping with a specific u-url, no matter of it owns that url or not.

you'd still want to accept updates, so the third party wm wouldn't necessarily "block" future wms with the same u-url, but it definitely could spoof or overwrite them afterward. from https://indieweb.org/Webmention-brainstorming#de-duplication :

security / identification / preventing hijacking. An attacker could overwrite or delete an existing webmention by sending a new one from their own site with the same u-url. To prevent this, receivers can compare source domain as well as uid, u-url, etc., and only interpret two webmentions as duplicates if both match.

cc @kevinmarks


Update badges

For issue #31
Changes include:

Increasing font-size of small badge
Making padding-top equal to padding-bottom for all badges
Increasing padding and font-size in equal increments for all badges
Adding border-radius to primary badge as well. In provided examples and other design systems, variants have been distinguished purely by color.
Making font-weight for all badges equal. Reason being the same as above.

PS: Designer by profession, first time contributing. Feedback is appreciated!

Looks good. Thanks for contributing!


[CSGen] Removing code that was only necessary for failure diagnostics

Failure diagnostics is long gone, and by the comment over this code that was the only reason for it.
So we just removing it =]

@swift-ci Please smoke test

@swift-ci Please test Windows Platform

@xedin Windows build is failing in all PRs. Could we land this anyways or better wait for it?

@rintaro @nathawes Do you have any concerns about removing this? Is there anything in code completion which would still try to re-typecheck a closure sub-expression?

If there are no concerns I think we can merge even with Windows failing.

If there are no concerns I think we can merge even with Windows failing.

Awesome =]

@swift-ci Please test Windows Platform

@swift-ci Please test Windows Platform


Support for Quarkus deployment with odo

Workflow:

deploy the test application using odo push

Stuff to figure out:

How to push and use custom Maven settings and repository with the odo image?

What needs to be done?

implement new simple Java wrapper for odo
use the wrapper to manage new implementation of ManagedResource for odo
update @*scenario annotations

I'm gonna close this, as this is not a target anymore


R0 and R1 waveform merging

This PR closes #2368. I don't know if there are any R0 and/or R1 files for doing the unit tests. Even with a handful of files, the merging takes quite some time. Please let me know.

@TjarkMiener Thanks, this looks good.
Could you please also add a changelog entry? The file should be called docs/changes/2386.feature.rst and have one sentence about the change like "Add support for including r1 and r0 waveforms in ctapipe-merge`


Use the TIMESTAMP_NTZ SQLConf to see if the user wants to use the local time without timezone

I am not sure but we may want to read this setting in conjunction with the timezone setting to see if we can use UTC time
Read SPARK-35975 for details

Not immediately required since this is a new type.

spark.sql.timestampType is used to control timestamp behavior in many places, such as  SQL DDL, Cast clause, type literal, and the schema inference of data sources.
If the user specifies TIMESTAMP_NTZ, which we do not currently support, then we could have incorrect behavior in some cases. We should review and see if we need to fall back in any of these areas.

It looks like the current behavior is correct because we fall back to CPU if a TimestampNTZType is involved.


Setting LSP document version in Language Server

Each time that a document is change in an editor, the Language Server Protocol includes a document version number as part of its textNotification/didChange updates. This document version can be specified as part of WorkspaceEdit objects when setting up code actions.
Up to Visual Studio 16.9, it was optional by the LSP protocol to include a matching version, and the extension defaulted to a version of 1 in all cases. However, starting from 16.10, the protocol expects the version of the document to match the latest one provided by the LSP, otherwise code actions will be ignored.
This change keeps track of the last received version from the LSP and uses it when setting up code actions.

Testing completed for VS and VS Code with those changes.

/azp run

/azp run

/azp run


Java API and Node.js: java.lang.IllegalArgumentException: graphDef and prefix cannot be null

I'm using node-java to run the JAVA API within a Node.js application. Most of the work is done in Java, but I have some issues when loading the graph through node using the Java api graph.importGraphDef I get a java.lang.IllegalArgumentException: graphDef and prefix cannot be null.
I'm loading the inception graph as a binary file tensorflow_inception_graph.pb and passing it to the api like:
var graph = java.newInstanceSync("org.tensorflow.Graph");
var builder = java.newInstanceSync("org.example.GraphBuilder", graph);

const getBinary = (graphPath, asBuffer = false, cb) => {
    let readStream = fs.createReadStream(graphPath)
    let data = ''
    readStream.setEncoding('binary')
    readStream.once('error', err => {
        return cb(err)
    })
    readStream.on('data', chunk => (data += chunk))
    readStream.on('end', () => {
        return cb(null, asBuffer ? Buffer.from(data, 'binary') : data)
    })
}

getBinary(graphPath, true, (error, graphDef) => {
    console.log("type is", typeof(graphDef));
    console.log("read graph %d", graphDef.length);
    graph.importGraphDefSync(graphDef, "");
});

The graph object is loaded by Node.js since I can see its bytes length in the logs:
The output is:
Tensorflow.getVersion: 1.4.0
2017-11-08 09:59:51.249872: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.2 AVX AVX2 FMA
type is object
read graph 53884595
/MyProject/tensorflow/index.js:106
                        graph.importGraphDefSync(graphDef,"");
                              ^

Error: Error running instance method
java.lang.IllegalArgumentException: graphDef and prefix cannot be null
	at org.tensorflow.Graph.importGraphDef(Graph.java:127)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)

    at Error (native)
    at getBinary (/MyProject/tensorflow/index.js:106:31)
    at ReadStream.readStream.on (/MyProject/tensorflow/index.js:99:34)
    at emitNone (events.js:91:20)
    at ReadStream.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Node and node-java bridging isn't officially supported by the TF team. Try asking a question on Stack Overflow, which is better suited for problems that are not bugs or feature requests. If you can track this back to a problem isolated within the Java API, please file another issue. Thanks!

@angersson Yes sir of course I did before posting here, but the response did not help me https://stackoverflow.com/questions/47160365/read-binary-data-with-node-js-stream


Vulnerability roundup 87: unicode-2.7: 1 advisory [8.1]

search, files

[ ] CVE-2016-10578 CVSSv3=8.1 (nixos-unstable)

Scanned versions: nixos-unstable: b3251e04ee4. May contain false positives.
Cc @woffs

The CVE affects the node library, not https://github.com/garabik/unicode.


Figure out how to share runners between tt-mlir and tt-forge

https://docs.github.com/en/enterprise-cloud@latest/actions/hosting-your-own-runners/[…]d-runners/managing-access-to-self-hosted-runners-using-groups for some pointers

This is a task for the organization's admin, please assign this ticket accordingly.
Here is the step-by-step guide on how to create a group, assign projects to it, and move existing runners to the new group. Guide is generated by LLM, it haven't tested it.
Step-by-Step Guide
1.	Create a Runner Group in the Organization
•	Go to your organization’s main page on GitHub.
•	Click on Settings under your organization’s name.
•	In the left sidebar, click on Actions, then select Runner groups.
•	Click on New runner group.
•	Enter a name for your runner group (e.g., “shared-runners”).
•	Configure repository access by specifying that both “tt-mlir” and “tt-forge” can use this runner group. You can do this by selecting these repositories from the list.
•	Optionally, set workflow access policies if you need more granular control over which workflows can access the runner group.
•	Click Create group to finalize.

2.	Add Runners to the Runner Group
•	Once the runner group is created, you need to add your self-hosted runners to this group.
•	Go to the Actions settings again.
•	Click on Runners and then select the runner you want to move to the new group.
•	Click Edit next to the runner, and change the group to the newly created runner group (e.g., “shared-runners”).
•	Repeat this step for each runner you want to add to the group.

3.	Configure Repositories to Use the Runner Group
•	Ensure both “tt-mlir” and “tt-forge” repositories are configured to use runners from the “shared-runners” group.
•	In each repository, go to Settings > Actions > Runners.
•	Verify that the runners listed are part of the “shared-runners” group.

Mentioned on the JIRA ticket
The common GitHub CI project should be helping with this.
By the way, we'll need to come up with a plan of ownership, maintenance, and most importantly, accountability for the project. MLIR/Forge definitely has vested interest, and in my view, should have some degree of ownership and responsibility over the product once we roll it out. Metal is in the same boat.
@vmilosevic is actively involved in those meetings.


Devspaces combine folders

Resolves #282
Combines the two existing devspaces folders into a single example
Updated the base checluster to match the default checluster when creating a new instance.
Added several "components" to capture some of the existing customizations that were present in the two folders.

@codekow Tagging you since I know you have done a decent amount of work with dev spaces.
Anything in the old configs that I am missing that could be component-ized?