Bulk insert with has many association

Your Question
I'm trying to perform a bulk insert with has many association, for example: author has many book. So what I do is:
for (...) {
  author := User{Name: "xxx"}
  author.Books = append(author.Book, &book1)
  author.Books = append(author.Book, &book2)
  authors = append(authors, author)
}
db.Create(authors)

The problem here is that Gorm tries to create the books before the authors so I get the error Error 1452: Cannot add or update a child row: a foreign key constraint fails because the author_id on the books table is 0:
INSERT INTO `books` (`author_id`,`title`) VALUES (0,"Title 1"),(0,"Title 2");

Is there a way to achieve this? If yes how?
Thanks for the support.

No.

@pioz Quick question, are you using gorm.Model in your user struct?

No.

how do you create the author id?

how do you create the author id?

I have a beforecreate hook which I create a uuid.
Also, if you look in your db at your tables, is the Id of the Author table the primary key?

from my understanding, you have to create the id somewhere, if you are not using gorm.Model.
I have a beforecreate hook which I create a uuid.
Also, if you look in your db at your tables, is the Id of the Author table the primary key?

from my understanding, you have to create the id somewhere, if you are not using gorm.Model.
I have a beforecreate hook which I create a uuid.
Also, if you look in your db at your tables, is the Id of the Author table the primary key?

Yes, the Id field is the primary key. As the doc says (https://gorm.io/docs/models.html#Conventions):
GORM prefer convention over configuration, by default, GORM uses ID as primary key

from my understanding, you have to create the id somewhere, if you are not using gorm.Model.
I have a beforecreate hook which I create a uuid.
Also, if you look in your db at your tables, is the Id of the Author table the primary key?

Yes, the Id field is the primary key. As the doc says (https://gorm.io/docs/models.html#Conventions):
GORM prefer convention over configuration, by default, GORM uses ID as primary key

You will still need to create the value of the id yourself.
However, still a bit strange that you don't see a query trying to insert the Author.

You will still need to create the value of the id yourself.
However, still a bit strange that you don't see a query trying to insert the Author.

I see the query.
I do not have the IDs value of the authors cause I'm trying to insert them in bulk. So the IDs do not exist yet. I can create the author before and then create the books with the author_id values, but I want to know if Gorm is able to manage the bulk insert with has many associations itself.

I see the query.
I do not have the IDs value of the authors cause I'm trying to insert them in bulk. So the IDs do not exist yet. I can create the author before and then create the books with the author_id values, but I want to know if Gorm is able to manage the bulk insert with has many associations itself.

Here is a tidbit of code I have in my application. I didnt use the gorm.Model cause at first I wasn't going to use UUIDs.
func (base *BaseStruct) BeforeCreate(tx *gorm.DB) error {
base.ID = base.createUuid()
t := base.createUTCTime()
base.CreatedAt = t
base.UpdatedAt = t

return nil

}
if you create a base struct and have all your struct use that struct. anytime one of those structs gets inserted, this code will run.
Its a really nice hook.
There is a similiar one for update.
I am also going to add createdby and updatedby to this struct.

Here is a tidbit of code I have in my application. I didnt use the gorm.Model cause at first I wasn't going to use UUIDs.
func (base *BaseStruct) BeforeCreate(tx *gorm.DB) error {
base.ID = base.createUuid()
t := base.createUTCTime()
base.CreatedAt = t
base.UpdatedAt = t

return nil

}
if you create a base struct and have all your struct use that struct. anytime one of those structs gets inserted, this code will run.
Its a really nice hook.
There is a similiar one for update.
I am also going to add createdby and updatedby to this struct.

I don't know. Following this example in the doc https://gorm.io/docs/create.html#Create-With-Associations I shouldn't need hooks.

I don't know. Following this example in the doc https://gorm.io/docs/create.html#Create-With-Associations I shouldn't need hooks.

Try db.Create(&authors), if it doesn't work, please create a reproducible GORM playground PR

Try db.Create(&authors), if it doesn't work, please create a reproducible GORM playground PR

@pioz GORM will not create data for you, unless you use gorm.Model. If you are using your own primary key, they you need to create the data for the primary key for GORM. In those examples, you will notice those structs using gorm.Model

@pioz GORM will not create data for you, unless you use gorm.Model. If you are using your own primary key, they you need to create the data for the primary key for GORM. In those examples, you will notice those structs using gorm.Model


fix #5 import of FSharp.Targets for VS 2015

Ignore the whitespace changes with w=1:
https://github.com/ctaggart/froto/compare/vs2015?expand=1&w=1

Everything switched to lf from crlf which is fine.


Why the field name must lower case?

Below will work:
q := db.NewQuery("SELECT id, username as name FROM tbl_users order by id LIMIT 10 ")
However, below not work:
q := db.NewQuery("SELECT id, username as Name FROM tbl_users order by id LIMIT 10 ")
It seem ozzo-dbx only prefer the field name as lowercase.
Could you advise why we have to use lower case filed name?
How to lift this limitation? Thanks.

This is explained in README: https://github.com/go-ozzo/ozzo-dbx#executing-queries
You can use struct field tags to explicitly specify which column names to use. You can also specify Query.FieldMapper to use your own field mapper. The default one will do what you described.

Thanks for the confirmation.


bug: Invalid app version when building from source

Description:
When building binaries from source app has invalid version.
How to reproduce:
git clone https://github.com/palomachain/paloma.git
cd paloma
git checkout v0.11.6
make build
./build/palomad version

Output:
name: paloma
server_name: palomad
version: HEAD-6a6c9280f76e0c5a510d80aebcfef4e94b7043c0
commit: 6a6c9280f76e0c5a510d80aebcfef4e94b7043c0

After starting palomad service we see an error in the logs:
12:06AM INF running app args=["start"] module=cosmovisor path=/home/paloma-testnet/.paloma/cosmovisor/genesis/bin/palomad
panic: invalid app version 'HEAD-6a6c9280f76e0c5a510d80aebcfef4e94b7043c0'

What is expected:
After building binaries we expect to get correct app version and no errors when running node service

i have changed the
VERSION := $(shell git describe --exact-match 2>/dev/null) to
VERSION := v0.11.6 at the https://github.com/palomachain/paloma/blob/master/Makefile
and after that built from the source.
palomad binary worked without any problem on a raspberry pi.
It is a temporary solution but will be easier to debug with this.

my commit
version: v0.11.6
commit: 6a6c9280f76e0c5a510d80aebcfef4e94b7043c0


ARTCipherParams and ARTCrypto doc comments

Added docstrings according to https://github.com/ably/canonical-api-reference-prototyping/pull/8
Was checked for compatibility with jazzy.

https://sdk.ably.com/builds/ably/ably-cocoa/pull/1439/jazzydoc/Classes/ARTCrypto.html

I've added @m-hulbert as a reviewer here, too, since he might be picking up any necessary changes once @tbedford rolls off this project.

Closed in favor of https://github.com/ably/ably-cocoa/pull/1491


Unable to start container

Container refuses to start and always exits

What is your docker run command ?  Can you provide the container's log?

  Looking in config/log/init_db.log, I see the following:
Installing MariaDB/MySQL system tables in '/config/mysql' ...
2019-11-22  8:09:48 0 [ERROR] InnoDB: preallocating 12582912 bytes for file ./ibdata1 failed with error 95
2019-11-22  8:09:48 0 [ERROR] InnoDB: Could not set the file size of './ibdata1'. Probably out of disk space
2019-11-22  8:09:48 0 [ERROR] InnoDB: Database creation was aborted with error Generic error. You may need to delete the ibdata1 file before trying to start up again.
2019-11-22  8:09:48 0 [ERROR] Plugin 'InnoDB' init function returned error.
2019-11-22  8:09:48 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
2019-11-22  8:09:48 0 [ERROR] Unknown/unsupported storage engine: InnoDB
2019-11-22  8:09:48 0 [ERROR] Aborting

There's definitely plenty of space, but I've noticed that config/mysql is owned by root, while all the other files and directories are owned by the user I've designated in the USER_ID and GROUP_ID environment vars.

On which filesystem /config is mapped to?  Looks like the filesystem you are using doesn't support the fallocate system call used to initialize the database.

This was on zfs.  Seems that fallocate isn't supported per https://github.com/zfsonlinux/zfs/issues/326.
I've moved on to another solution, but will be glad to re-install to debug if needed.

Same issue here. i just made this docker-compose.yml
version: '3'
services:
  nginx-proxy-manager:
    image: jlesage/nginx-proxy-manager
    build: .
    ports:
      - "8181:8181"
      - "8080:8080"
      - "4443:4443"
    volumes:
      - "./config:/config:rw"

and ran docker-compose up
I'm running docker in an LXC container. When I try do run fallocate in the LXC (so outside of docker) I get fallocate: fallocate failed: Operation not supported
Is it maybe possible to use truncate instead of fallocate? Or does someone know how to make fallocate work in an LXC?

fallocate is an operation used by the mariadb database.  Also, it is usually a limitation of the filesystem.  So are you sure that the issue is with LXC itself ?

To be fair, I did't know fallocate exists two days ago. So the only thing I can say with certainty is that if I run fallocate -l 8G disk.img on my Ubuntu desktop, I get an 8G file. If I run it on my proxmox server (using ZFS) or in an LXC container on it I get
user@pve:~$ fallocate -l 8G disk.img
fallocate: fallocate failed: Operation not supported

I hope that helps

fallocate is an operation not supported on ZFS :(


Way to tap before other taps?  Are stage and before options supported?

We're thinking about using tapable as a replacement for typedoc's internal eventing stuff.  Is there a way to tap a hook before other already-registered taps?  I see Hook options before and stage.  Are they supported, or will they go away when v2 is released?  If we start using tapable, should we use the v2 beta or stick to v1?
Thanks for any insight you can offer.  Let me know if I should be looking elsewhere for answers.

before and stage are safe to use. We do not intent to deprecate or remove them.
However, before using tapable, please make sure you understand the library limitations. tapable is designed to produce static event dispatchers (hooks). Which means that you must subscribe (tap) and then fire the event (call).

You cannot unregister a listener
You should add all the listeners before using call

Thank you for clarifying, that's good to know.  I think it should work for us; we want to support a composable, pluggable architecture, very similar to webpack.
The dispatchers are static because it's faster, right?

The dispatchers are static because it's faster, right?

That's debatable but yes. A compiled hook should be faster than a polymorphic function.  But if you compile your hooks upfront, then the handlers should be faster.
Under the hook, tapable compiles this:
listeners.forEach(listener => {
  listener(...args)
})

into this:
listeners[0](_0, _1, _2)
…
listeners[13](_0, _1, _2)

We always preferred stage over before. I found it easier to work with.

Is there documentation or an example of using stage?  I'm having trouble understanding how to use it, and neither type declarations have JSDoc comments to explain.
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/tapable/index.d.ts#L244
https://github.com/webpack/tapable/blob/master/tapable.d.ts#L30

It's a number, Taps with lower number execute before Taps with higher number. Taps with equal stage execute in order of attaching.

Very cool, I can declare multiple stages of a hook using a numeric enumeration, and then plugins can tap into any stage by name.
Thanks for all the information; this is very helpful.


floating label stopped working

For some reason, when I visit https://codyhouse.co/ds/components/app/floating-label, the component doesn't work.  It was working earlier on.  All browsers are having the issue. (Mac, Mojave)

I just tested on Chrome in an incognito window and it seems to be working fine. Prob a cache issue?
ps: just to be 100% sure, the label floats only after you start typing.

Right - the "starts typing" point is a good one to note.
I found one issue, and it has to do with adding the js class to the html tag.  I think instead of
<script>document.getElementsByTagName("html")[0].className += "js";</script>
it needs to be
<script>document.getElementsByTagName("html")[0].className += " js";</script>
note the space before the js - otherwise, if you have any other classes in the html tag, it will append that class instead of adding a new one.
So, now that I've done that, the label hides on load, but the js isn't picking up input text yet so the label is always remaining hidden.  I'm still debugging that bit.

Hi, thanks for providing this additional info.
I double-checked and it looks like the space before the class name was correctly inserted both in the framework (index.html file) and the documentation page.
Is there anywhere else where you found it without the space? So that I can fix the typo (I wasn't able to find it). Thanks.

I can't see it for now either.... maybe it was from an original doc file or something.  It was copy/pasted so it must have come from somewhere.  If I find it again I'll . let you know.  Sorry to bother.


Spawner doesn't break if it spawns non-living entity like fireworks or arrows

General Info

Client Info

Minecraft Version: 1.16.5

Forge Version: 1.16.5-forge-36.1.0

Mod Version: 1.1.0

Java Version: 1.8.0

Issue Description
Spawner doesn't break if it spawns non-living entity like fireworks

Reproduce Steps

Place a spawner and set its SpawnData/SpawnPotentials to firework entity.

Additional Information

I think it could be because of these entities are not living entity.

This issue affects when dungeons arise mod.
https://www.curseforge.com/minecraft/mc-mods/when-dungeons-arise
The mob spawners generated in this mod actually spawns a firework, and the monster rides on the firework so that the spawner won't be destroyed.

Could you tell me this problem will be fixed soon or not?
My modpack is about to be released, but because of this issue, players can farm monsters, which breaks the balance...
I'm waiting for it to be fixed so long...

I'm still looking into it. The event I rely on only fires for entities that extend MobEntity. Which the firework obviously doesn't. So I have to figure out another way


Add a property ShowInBrowser

Add a property "ShowInBrowser".
This should, when is set to true, start the internet explorer / edge and show the generated pivot table.

published with released 1.07.05


Double-scroll areas after expanding all lines

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.
[X] The bug happens after clearing extension cache. The "Clear cache" button below can also be found at the bottom of the options page.

Example URLs
https://github.com/refined-github/refined-github/pull/5422/commits/1bb5e36ce3734a045c5e4aaeeba6e0d29d381789#diff-84b7041bfeae43fea26aac38d2eb71d1c2ce5d7bdb2cd6a9a7b3502662a26da8
Extension version
22.3.3
Browser(s) used
Brave – Version 1.35.100 Chromium: 98.0.4758.87 (Official Build) (arm64)
Description

View 1bb5e36 (#5422)

Expand file source/features/collapsible-content-button.tsx

Scroll up and down

Expected: Only one scroll area for the webpage's document's body or window is present
Actual: There is another scroll area present for the file container element with a little amount of scroll leeway.
Screen Recording:
https://user-images.githubusercontent.com/517469/157112498-a2ec92ff-bc3c-4f2f-a6fc-bec678a3683c.mp4
Include in this issue:

[X] Screenshots/video/gif demonstrating the bug, if it’s visual
[ ] Console errors, if any

This is not a Refined GitHub bug, I can replicate it without RG

This is not a Refined GitHub bug, I can replicate it without RG

I didn't know GH has the expand all lines feature now. I've been using RG mainly for this feature.

I didn't know GH has the expand all lines feature now.

It was dropped in https://togithub.com/refined-github/refined-github/pull/4150.
Maybe the fix for this is as simple as:
.js-blob-wrapper {
	overflow-y: hidden !important;
}

overflow-y: hidden

vade retro satana
As the name suggests, that just hides the issue.
I found the cause by deleting one element at a time in the table:

This is the line number and that top: 1px pushes it 1px outside the table, which lengthens the scrollable content. A fix could probably just add margin-bottom: -1px to it but I don't remember if that actually reduces the scrollable height.

How about a simple top: 0 !important? Tried it in my custom CSS and it doesn't seem to break anything.

I haven't checked but I assume that that would change (break) the number text alignment

I haven't checked but I assume that that would change (break) the number text alignment

Actually this is not the line numbers, but the +/- diff signs, which are hidden by RGH anyway.

Sounds like our GitHub bugs CSS file will receive a +6 diff then 😌

This bug is really annoying


Cross-Origin Read Blocking (CORB) blocked cross-origin response = Tableau Web Data Connector

We are trying to pull a JSON structure by invoking an API from the localhost with port number :9097  from Tableau Web Data Connector (Simulator),
The server (node.js) on which Simulator is hosted is on port 8888.  The request is passed from http://localhost:8888/Simulator/index.html to https://localhost:9097/dcj-ors-pc-datawarehouse-sys-api/v1/caseDetails
The chrome browser throws back the below error
Cross-Origin Read Blocking (CORB) blocked cross-origin response https://localhost:9097/dcj-ors-pc-datawarehouse-sys-api/v1/caseDetails with MIME type application/json. See https://www.chromestatus.com/feature/5629709824032768 for more details.

Hi @SriniRMTUG,
Can you check that:

Your "Content-Type" response header is set as text/json
The Answer from your API (https://localhost:9097/dcj-ors-pc-datawarehouse-sys-api/v1/caseDetails) is not empty
Please?
Cheers,
Gigi

No answer, closing the issue


Added Favoriting feature

By favoriting URLs you can ensure that these URLs show up at the top of
the beacon browser list and will be shown in the notifications.

Imperative casing on commit message


[FEATURE] v2 supports volume expansion

Is your feature request related to a problem? Please describe (👍 if you like this request)

Describe the solution you'd like

Describe alternatives you've considered

Additional context

@AaronDewes
Thanks for your contribution. I created the feature request ticket for tracking the status of the v2 volume expansion and resize. Feel free to discussion and update the status here. Thank you.
cc @innobead @shuo-wu @DamiaSan

Thanks!
As I've mentioned before, I have not yet tested this properly yet. I'm planning to run this locally today and make improvements or let you know if I have questions. Because I've only used Longhorn before, but not contributed yet, I'm not familiar with the internal architecture.

@AaronDewes as this is an upcoming feature in 1.7.0, so I would expect you can actively work on the feature. Please work with @derekbit who is our maintainer as well to try to make the feature happen eventually.
Assigned to you first, as you are working on that. However, if you can't make it, just let us know and we will arrange the assignee. Thanks for your contribution!

Thanks!
As I've mentioned before, I have not yet tested this properly yet. I'm planning to run this locally today and make improvements or let you know if I have questions. Because I've only used Longhorn before, but not contributed yet, I'm not familiar with the internal architecture.

@AaronDewes
Feel free to ping me on the Slack channel or leave messages here if you have any questions.
BTW, please let us know what docs or materials we can improve for external contributors.
Thank you.

Just found the related PR in spdk side
https://review.spdk.io/gerrit/c/spdk/spdk/+/22619
cc @AaronDewes @DamiaSan

Just found the related PR in spdk side https://review.spdk.io/gerrit/c/spdk/spdk/+/22619

I think this patch will be merged soon on SPDK master branch.

@DamiaSan Could you collaborate with @AaronDewes to land the implementation in Longhorn? Thanks.
cc @innobead @shuo-wu

@DamiaSan Could you collaborate with @AaronDewes to land the implementation in Longhorn? Thanks.
cc @innobead @shuo-wu

Yes, sure. @AaronDewes do you already have some questions or something I can do for you?

Hi @AaronDewes , I have just had a look at the work to do:

SPDK side, we have simply to start using the upcoming release 24.09, which include the raid1 resize functionality
go-spdk-helper side, the bdev_lvol_resize has already been wrapped into the API BdevLvolResize
go-spdk-engine side, the work has to be done. We must implement the expand functionality at replica and engine level
longhorn-instance-manager, we must implement the function VolumeExpand for the V2DataEngineProxyOps.

I will create a new SPDK branch as soon as the new 24.09 release will be available, meanwhile you could start working on longhorn-instance-manager and longhorn-spdk-engine.
A good way could be to take as example how the function VolumeSnapshot is implemented for both V1DataEngineProxyOps and V2DataEngineProxyOps of longhorn-instance-manager/pkg/proxy/snapshot.go and make something similar for the VolumeExpand in longhorn-instance-manager/pkg/proxy/volume.go. Also in longhorn-spdk-engine, you can "copy" the way a snapshot is done to implement the expansion functionality, the chain of calls is the following:

EngineSnapshotCreate of SPDKClient, this is called by longhorn-instance-manager
EngineSnapshotCreate of spdkrpc.SPDKServiceClient
EngineSnapshotCreate of Server, which is a spdkrpc.UnimplementedSPDKServiceServer
SnapshotCreate of Engine
snapshotOperation and snapshotOperationWithoutLock of Engine
replicaSnapshotOperation of Engine for all the replicas of the volume
ReplicaSnapshotCreate of SPDKClient
ReplicaSnapshotCreate of spdkrpc.SPDKServiceClient
ReplicaSnapshotCreate of  Server, which is a spdkrpc.UnimplementedSPDKServiceServer
SnapshotCreate of Replica
BdevLvolSnapshot, which is the API of go-spdk-helper

WDYT? Tell me if you need other informations or any kind of help.
cc @innobead @derekbit

2 questions:

For a lvol, should the size keep the same as that of its parent or its children?
After resizing a lvol, should we somehow reload/refresh its nvmf used by a RAID?

2 questions:

For a lvol, should the size keep the same as that of its parent or its children?

When you resize a lvol, its snapshot parent isn't affected by the resize operation, only lvol change. The size of a snapshot can't be changed.

After resizing a lvol, should we somehow reload/refresh its nvmf used by a RAID?

No, the nvmf layer is automatically refreshed with the new size.

Then the v2 volume expansion implementation looks less complicated.

Hi @derekbit , what about the suspension of the IO over the volume before to proceed with its expansion, it is not needed. Because  SPDK, before to resize a lvol, freeze IO over it. Then, raid resize happens only when all its base bdevs have resized, but in this case the resize is simply a notification, it doesn't need any new space allocation.
I have just made a test resizing a raid1 made of 2 lvols meanwhile IOs were running over it and all worked fine.

Found an issue in SPDK: https://github.com/spdk/spdk/issues/3540

The development in longhorn-spdk-engine is here: https://github.com/DamiaSan/longhorn-spdk-engine/tree/volume_expansion
The work to solve the issue in SPDK is here: https://github.com/DamiaSan/spdk/commits/nvmf_lvol_resize/
Working only with SPDK all seems ok, but executing the new test in longhorn-spdk-engine sometimes SPDK get stuck in lvstore creation, so something is still to be fixed.

Working only with SPDK all seems ok, but executing the new test in longhorn-spdk-engine sometimes SPDK get stuck in lvstore creation, so something is still to be fixed.

This stuck is not related to this development and has been resolved in ther new longhorn-v24.09  branch with this commit: https://github.com/longhorn/spdk/commit/10463b5c13b454e173707628b9eac608081f5082
So I can proceed.


HTTP Connection Getting Killed

Past around 700 stars, hosting services (Heroku) kill the HTTP connection for running too long.

Fixed in commit  1fba654

Release 1.2.2 has a much better better fix for this.

Fixed in release 1.2.2.


[Enhancement] Serverless(.com) Provider

Hey Folks,
Has anyone considered submitting a provider to the Serverless.com project so it can manage the lifecycle of serverless code  in Pulsar? It would be a great addition.
-J

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Scroller should only be applicable for filter option.This issue exists for ever grid where filter is used.

Framework Version:0.1.4
Scroller is working for page also.

Framework Version: 0.1.4
Fixed.Now Scroller is removed from filter

Framework Version: o.1.5
Fixed


Add banners and character

Can you add Hu Tao in here too .-.

And the banners of the new characters too, like Xiao and Hu Tao

Yay thank you :D


Errors in Input Mouse Events examples

So I'm playing around with Phaser 3 and noticed that the input/mouse circle and ellipse hit area examples don't work. They are probably not yet updated, but the huge "Open Github Issue" button tempted me ;)
It seems like this.input.events.on() is undefined.
Through a little bit of guesswork and reading newer syntax somewhere, I managed to get it working via
this.input.on('pointerover', function (pointer, gameObject)
{
        gameObject[0].setTint(0x7878ff);
});

2 questions though:

Are the events documented somewhere? The examples still have (I guess old?) other events in them, like POINTER_OVER_EVENT
Why exactly does the second parameter deliver an array? I'm probably missing some important point, but it doesn't make much sense to me in this case.

Hope I didn't miss an overlapping issue.
Cheers! :)

These examples have been fixed now.
pointerover delivers the pointer and an array of game objects, because it can be over more than one at once. gameobjectover delivers the pointer + one game object.


binder: support natural join

Is this issue open to be taken?

Is this issue open to be taken?
https://github.com/singularity-data/risingwave/blob/main/src/frontend/src/binder/relation/join.rs#L114-L116

It requires some prior refactoring to be fully correct. The current BindContext implementation is buggy (#2771 #3352) and seems not easily extensible to keep only one copy of the join columns (#2683)


Why not use multi-scale?

I am curious why multi-scale is not applicable, is it because of the amount of calculation?

Hi, thank you for your interest. :)
We have the multi-scale version SAM-DETR in this repo, but multi-scale is not used in our CVPR paper. Our paper about the multi-scale version of SAM-DETR (with even better results reported in this repo) is currently under review in a premier journal. We will release them in the future. Please stay tuned!


Logging

Fixes packit/packit-service#1971
RELEASE NOTES BEGIN
Packit should now correctly catch the logs, if any of the user actions fail during the propose-downstream.
RELEASE NOTES END

This won't include the logs anyways… It will just make sure that they are correctly returned to the worker.
I »could« print the logs out even for the failed command, but that would result in the logs being present twice in the worker, cause first time they are outputted by the sandcastle and second time would be me…


Publish rate frequency depends on the device manufacturer

:red_circle:  Issue
Android Publisher app updates localization with different frequencies for different device manufacturers.
Battery level, proximity to destination, and channel subscription state were the same for both devices.
🏞 Environment
Tested using  Android Customer app v1.0(29)
P120 Samsung Galaxy M51 A. 10
Google Pixel 5 Android 12
Test conditions:

Spotted during LONG RUN test(~8h)
Network: 5G
Batteries were fully charged and devices were connected to the power supply
No destination point was selected
For both devices channel was subscribed

🦶 Steps to reproduce

Start publisher app
Add trackable ID
Observe the Enhanced Location Update Messages(Dev Console)

:x: Messages from Pixel are updated with twice the frequency as from Samsung.
┆Issue is synchronized with this Jira Uncategorised by Unito

@Jaceken could you attach an export of logs from Dev Console?

We can no longer reproduce the issue - closing, discussion with Mapbox to be concluded on this issue.


Improvement for k8s.io/docs/reference/kubectl/generated/kubectl_cordon/

As of 11:22 am 8/7/24
Synopsis reads: Mark node as unscheduable.
Synopsis should read: Marks node as unscheduable which will no longer watch for newly created pods with no assigned node and select a node for them to run on.
Thanks, Richard (Parkofadown)

Please bear https://kubernetes.io/docs/reference/kubectl/generated/kubectl_cordon/#auto-generated-edit-notice in mind @Parkofadown
We can transfer the issue for you to the right place (Prow command /transfer kubectl) if you like.

Please bear https://kubernetes.io/docs/reference/kubectl/generated/kubectl_cordon/#auto-generated-edit-notice in mind @Parkofadown
We can transfer the issue for you to the right place (Prow command /transfer kubectl) if you like.

Thank you , what is a Prow command /transfer kubectl ?

We have a bot to help us manage issues. It's sufficiently advanced that I can:
/transfer kubectl


AUTO: Adds project board syncer for image issues

Sync all opened issues to our internal project board.

(kicking DCO)

DCO wake up we need you


VGG-16 fine-tuning is hard to converge

I have tried to fine-tune VGG-16 on Stanford Cars dataset, but it's hard to converge. The loss is around 5.27 and doesn't decrease any more. What's the matter?
'''
Train on a Titan X GPU.
'''

from __future__ import print_function
import numpy as np
np.random.seed(1337)  # for reproducibility

from keras.preprocessing.image import ImageDataGenerator
from keras.layers import Input, Dense, RepeatVector, Permute, merge
from keras.layers import Dense, Dropout, Activation, Flatten, Reshape
from keras.layers import Convolution2D, MaxPooling2D, Convolution1D
from keras.models import Model
from keras.optimizers import SGD
from keras.utils import np_utils
from keras import backend as K
from keras.utils.visualize_util import plot
from deep_learning_models.vgg16 import VGG16
import stanford_cars

batch_size = 64
nb_epoch = 12

# number of classes
nb_classes = 196
# input image dimensions
img_rows, img_cols = 224, 224

# the data, shuffled and split between train and test sets
(X_train, y_train), (X_test, y_test) = stanford_cars.load_data(target_size=(img_rows, img_cols))
if K.image_dim_ordering() == 'th':
    X_train = X_train.reshape(X_train.shape[0], 3, img_rows, img_cols)
    X_test = X_test.reshape(X_test.shape[0], 3, img_rows, img_cols)
    input_shape = (3, img_rows, img_cols)
else:
    X_train = X_train.reshape(X_train.shape[0], img_rows, img_cols, 3)
    X_test = X_test.reshape(X_test.shape[0], img_rows, img_cols, 3)
    input_shape = (img_rows, img_cols, 3)
X_train = X_train.astype('float32')
X_test = X_test.astype('float32')
X_train /= 255
X_test /= 255
print(X_train[0][1])
print('X_train shape:', X_train.shape)
print(X_train.shape[0], 'train samples')
print(X_test.shape[0], 'test samples')

# convert class vectors to binary class matrices
Y_train = np_utils.to_categorical(y_train, nb_classes)
Y_test = np_utils.to_categorical(y_test, nb_classes)
print(y_train[-10:])
# model define
inputs = Input(shape=input_shape)
model = VGG16(input_tensor=inputs, include_top=True,
              weights='imagenet', nb_classes=nb_classes)
plot(model, show_shapes=True, to_file='final_gru_model.png')

# data augment this will do preprocessing and realtime data augmentation
datagen = ImageDataGenerator(
    featurewise_center=False,  # set input mean to 0 over the dataset
    samplewise_center=False,  # set each sample mean to 0
    featurewise_std_normalization=False,  # divide inputs by std of the dataset
    samplewise_std_normalization=False,  # divide each input by its std
    zca_whitening=False,  # apply ZCA whitening
    rotation_range=0,  # randomly rotate images in the range (degrees, 0 to 180)
    width_shift_range=0.1,  # randomly shift images horizontally (fraction of total width)
    height_shift_range=0.1,  # randomly shift images vertically (fraction of total height)
    horizontal_flip=True,  # randomly flip images
    vertical_flip=False)  # randomly flip images

datagen.fit(X_train)

# train
model.compile(loss='categorical_crossentropy',
              optimizer='adam', metrics=['accuracy'])
model.fit_generator(datagen.flow(X_train, Y_train,
                    batch_size=batch_size),
                    samples_per_epoch=X_train.shape[0],
                    nb_epoch=nb_epoch,
                    validation_data=(X_test, Y_test),
                    verbose=1)

sgd = SGD(lr=0.01, decay=1e-6, momentum=0.9, nesterov=True)
model.compile(loss='categorical_crossentropy',
              optimizer=sgd, metrics=['accuracy'])
model.fit_generator(datagen.flow(X_train, Y_train,
                    batch_size=batch_size),
                    samples_per_epoch=X_train.shape[0],
                    nb_epoch=nb_epoch,
                    validation_data=(X_test, Y_test),
                    verbose=1)

# test
score = model.evaluate(X_test, Y_test, verbose=0)
print('Test score:', score[0])
print('Test accuracy:', score[1])

Maybe it's because the learning rate is too large. I set lr=0.001, then it converges but converges slowly.


[5.4] Removes redundant else structure

Removes a redundant else after a return statement.

Unless it fixes a bug I don't typically merge these kinds of changes.

Also see: #19026 #19027 #19028 #19029


Cross linking Memory issue, urgent

Trying to use MetaMorpheus from XL data analysis using RAW files, Search is taking so long for 1 file (2 days and counting).
Metamorpheus latest version W7 64 GB RAM intel 10 cores.
is this normal that It takes so long ?

Hi, Thanks for reporting the issue.
It is not normal. What database are you using? If you are using a large database, you may want to try to set Total Partition: 50 or 100. If that doesn't work, please let us know and we could run your data and find the problem.

Thanks a lot, Yes I was using a full database, will try to do it as you suggested and report again.
thanks again

Thanks for your advice, I followed your recommendation about using the partition, and it worked, but still using multiple RAW files resulted in the software bugging and not responding. using only one RAW file worked nicely. Maybe the database I'm using is big, will try again with a reduced database and see how it goes, for now analyzing the results from the single RAW file.
Do you have a newly updated user guide for the new version as you have quite new features that are not included in the first user guide version. for example keeping the option Keep top N candidate empty (no number specified) resulted in the software crashing when trying to save the parameters without warning. will be helpful to have a user guide for the updated features in the XLsearch.
Thanks

Thanks for the information. May I know what type of data you are searching? MS2 only or MS2-MS3? Are you using the Uniprot Human database?
Thanks for the suggestions. I will update the user guide soon and let you know.

Hi, I'm using MS2 and MS3 data and uniprot human database (but I figured out I was using the Uniprot with all the canonical and isoforms) will try to use next the canonical entries only.
thanks for the help.

Hi DaliJarboui, Thanks for letting me know the information. It will help to speed up if you remove the isoforms. Otherwise you can try to change 'Top Search Candidate' to a small number such as 20.
I've updated the user manual here: https://github.com/smith-chem-wisc/MetaMorpheus/wiki/1_NewTask:-Crosslink-Search, and updated a pdf manual. Please let me know if there is anything confusing.
Just let you know, MS3 data is generally in low-resolution. So far, MM is not well supported for low-resolution data. The best fragmentation type for DSSO would be HCD only, EThcD or HCD-ms2ETD etc.

Hi Lei,
So the analysis of 1 file run smoothly, however when using 4 files Metamorpheus seems to take longer (4days) with high load on the RAM, when stopping the software all the tmp files are deleted and it doesn't it even generate results for the other files, experimental setup was designed in a way that each sample was treated as a different experiment. the data were acquired using  HCD for DSSO fragmentation. thanks for your help and advice.
Best

Hi Dali, The information you reported suggested a potential RAM usage related bug. This is very very useful to us! I will try some search similar to yours to see if I can repeat the issue. It may take us a long time to solve the issue.
Just let you know, MM currently combine the results into one from several files. Based on your description, you may want to search them separately.

In the interest of providing more details: I face the same issue when using XL search over multiple large files, but not when searching separately.
The first two files or so are searched normally, but by the third file, RAM is used up completely (all 128 GBs), effectively freezing the program/search progress. This has been ultimately true regardless of number of partitions (1-250).
Sequence of tasks: 1. Calibration, 2. GPTMD search, 3. Search, 4. XL search
Starting XL search separately using files from steps 1 and 2 has the same result.
Thank you for all your work on this software.

In the interest of providing more details: I face the same issue when using XL search over multiple large files, but not when searching separately.
The first two files or so are searched normally, but by the third file, RAM is used up completely (all 128 GBs), effectively freezing the program/search progress. This has been ultimately true regardless of number of partitions (1-250).
Sequence of tasks: 1. Calibration, 2. GPTMD search, 3. Search, 4. XL search
Starting XL search separately using files from steps 1 and 2 has the same result.
Thank you for all your work on this software.

Hi, thanks for following up on this issue. Please let us know if you have other suggestions about the program.
According to you and several others, the XL search has a memory issue. I will put this as my priority.
If you don't mind, please share a few (seems like 3 ) of your file with me<EMAIL_ADDRESS>to test. I understand the privacy of your project.
I will let you know when this is fixed.
Lei

Hi, thanks for following up on this issue. Please let us know if you have other suggestions about the program.
According to you and several others, the XL search has a memory issue. I will put this as my priority.
If you don't mind, please share a few (seems like 3 ) of your file with me<EMAIL_ADDRESS>to test. I understand the privacy of your project.
I will let you know when this is fixed.
Lei

I'll prepare a task set to reproduce the issue and send it with the files. Thank you!

I'll prepare a task set to reproduce the issue and send it with the files. Thank you!

Problem targeted and will be solved in near future.

In the interest of providing more details: I face the same issue when using XL search over multiple large files, but not when searching separately.
The first two files or so are searched normally, but by the third file, RAM is used up completely (all 128 GBs), effectively freezing the program/search progress. This has been ultimately true regardless of number of partitions (1-250).
Sequence of tasks: 1. Calibration, 2. GPTMD search, 3. Search, 4. XL search
Starting XL search separately using files from steps 1 and 2 has the same result.
Thank you for all your work on this software.

I have been having a similar issue with RAM in XL searches (tasks are just Calibration and then XL search). It will gradually creep up to use all 64 GB and then freeze, though changing partitions to 50-100 will just barely get it to complete on four .raw files for me in <90 minutes. I tried increasing virtual memory, but it doesn't seem to be using the virtual memory when it is about to max out the physical memory. I am using a large database (yeast + human). It's nice that MetaMorpheus can handle it under certain settings, but it seems like there ought to be a way to limit physical memory use so that more files can be analyzed together without freezing.

let me check with the main contributor

Hi @jsalbin. Thanks for reporting the issue. We targeted the RAM usage problem and are very close to solving it together with some other optimizations. Please wait for a few days. I or @trishorts will inform you here when we got a new release.

Thank you!
From: Lei Lu @.>
Sent: Friday, July 9, 2021 1:09 PM
To: smith-chem-wisc/MetaMorpheus @.>
Cc: Albin, John S., M.D.,Ph.D<EMAIL_ADDRESS>Mention @.***>
Subject: Re: [smith-chem-wisc/MetaMorpheus] Cross linking Memory issue, urgent (#1962)
    External Email - Use Caution

Hi @jsalbinhttps://secure-web.cisco.com/1ttam-eB7U24asWrX7dV5GgdJ6W-O1zb8KEpFwKst4TK6b7lDUh2iQHYPxskQ_mnh8269EFVXAc31vnFo0_kAYVlERcR_qew4nQLwPlkTHeSu1Cc2nCQk3tl9exKvo0aXpCnLkiaI0xMEj1VxcfxpEJU3JQDzNxfs25Vs0P-A00cMMegbi_l3rLdDLRFQq1pqhzvmAPQ2fPbNdagMKFg3WPb7c64KgzPSB6RKZ2ALVZvy--xV6OheCfRuYAeT84u4/https%3A%2F%2Fgithub.com%2Fjsalbin. Thanks for reporting the issue. We targeted the RAM usage problem and are very close to solving it together with some other optimizations. Please wait for a few days. I or @trishortshttps://secure-web.cisco.com/13wuF_Bx9d2A2gPMHyl-QFdF_eQ8ZU2Xpnaw_s58qHl0HllqrY6YhJtarlW3osak6Z6BxTxDGQtV1-kR8VkZwTAQ-dk9odStITv54UK7HaLkEXNv_hmuRXBzZb8VId6FjSijUZ0vTP3ERNXn9ll-vpBJ-5o-y7PigQfjGUKt7_K4ZMnf1n1xVvjahC4AQFcfm9EYUwlffNKGsG7UlKE7tqDfIbPbZYDqYxqWVXmM9rI1iWC1nBQU7cZgrkbqBugNu/https%3A%2F%2Fgithub.com%2Ftrishorts will inform you here when we got a new release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://secure-web.cisco.com/1krXOW0eBdY6ifZ_EHw5qvGORfiwtbvFmtLLXtmIybDzHKmANtyPx2VpxRinrQzcDF3d_SMwZhjIWlADZWrbGAe2iH5NvDBddti7vBXiDMS9kbkMFOdpz7Pjgy_Fc6EMj85jmXF_iK_8njZrZJOZQzbQdoGqVwMzTGfaP8Kn9ivLDSnxC5mCbhrokeeC3c-Qg0ssMAJir_klkfKKd2pGPPop0mKHqAFc_vxQg6dAPKof3zlvbhM3idchoB1kLl5nc/https%3A%2F%2Fgithub.com%2Fsmith-chem-wisc%2FMetaMorpheus%2Fissues%2F1962%23issuecomment-877331656, or unsubscribehttps://secure-web.cisco.com/1Drr6ajEGFx8TpLtKcdH7YZkLMCHpVKG9rKIC0cFQYKPL8kj3UTqagRsVw_OMctCWIwQMp2eAhCLpNLSf-GgLKF1RMaoBnf0SDu2NXuWU3p_HI9GSIkB9jbBKnOTpRYS_RrOaIWvYUWVGwt9sSu_JEx_gW0d4l38SBsP2T4nKN4Vr6r50kHqIS69oNwVg_SGQ9O4sb4raLvDOA-4jmWOy30PcE4t_4hBDicRfMzY3qV-rQXtpvH49HmM5RPfj_F9u/https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUKLJPUQCN5MH3LEIRHHGM3TW4ULTANCNFSM4SAERD6A.
The information in this e-mail is intended only for the person to whom it is addressed. If you believe this e-mail was sent to you in error and the e-mail contains patient information, please contact the Mass General Brigham Compliance HelpLine at http://www.massgeneralbrigham.org/complianceline . If the e-mail was sent to you in error but does not contain patient information, please contact the sender and properly dispose of the e-mail.
Please note that this e-mail is not secure (encrypted).  If you do not wish to continue communication over unencrypted e-mail, please notify the sender of this message immediately.  Continuing to send or respond to e-mail after receiving this message means you understand and accept this risk and wish to continue to communicate over unencrypted e-mail.


Fix arviz.to_dataframe for prior predictive

If we have m chains of n draws in the posterior and sample m * n draws from the prior, those prior draws are all part of chain 1. When we then call arviz.to_dataFrame there are many rows with blank entries because there are no posterior draws for chain 1, draw n+1, …, m*n.
We can fix this by post-processing the DataFrame, but it would be better to make the modification before calling arviz.to_dataFrame.

This should not be necessary. We don't plan to save the arviz.to_dataFrame output directly. Instead, we will create tabular data via https://github.com/CDCgov/forecasttools-py/issues/18.
cc @AFg6K7h4fhy2


add closing tx field to ClosedChannelPaymentDetails

It would be nice to be able to fetch the txid for a closing channel (pending) and also a closed transaction. At them moment we can only fetch the channel opening transaction.

@roeierez Yes of course. The only question is, should it fit in this release? Or better do it in the next.

Right. After the release


Creating new session files while keeping the old ones!

Package version
3.0.7
Node.js and npm version
v10.24.0
I'm working on a project in which there are almost 50k users which most of them are logged in.
The proplem is that the sessions folders size keeps increasing day by day and I think the reason is that adonisjs create a new session file while keeping the old ones. after 1 week the size of sessions folder is 3GB.
If it helps, I set session age to 90d.

You have to manually write a script to delete the old files by comparing their mtime. Also, I think, redis will be wiser choice in this scanerio


Check if OptMeowt meets accessibility standards

As @SebastianZimmeck mentioned in Teams, we need to make sure that OptMeowt meets accessibility standards. For websites, you use this, but it may be different for browser extensions. We may be able to just load the UI's html in the browser.

Most extensions and chrome's built in lighthouse only checks https or http. I'm not sure if it worths converting our extension into a web application. So, instead, I found an online checker that could check if html meets accessibility standards and uploaded the UI's html into it. At least the result can serve as a reference.
Attached are results and I'll look more into it.

OK, we should fix at least the "Known Problems."


Reddit bridgy fix

Primarily want to include a function to convert from praw.Redditor to intermediate dictionary in order for granary's user_to_actor to be compatible with this line in bridgy.  See comment in https://github.com/snarfed/bridgy/pull/935
Also, it appears that I need to include a "feature" param in my call to the reddit api due to feature being required here.  Correct me if I'm wrong.

Also, it appears that I need to include a "feature" param in my call to the reddit api due to feature being required here. Correct me if I'm wrong.

oh feature is a bridgy-specific thing, doesn't belong in o-d. maybe see other example sources in bridgy? should be able to keep it entirely there.


Optimize default resolver chain construction

We can figure out the right SegmentContextResolver at init time since it should remain static throughout the lifetime of an application (we're either running within Lambda or we're not - it should never change). Trying to resolve through the resolver chain at runtime seems unnecessary.
While typically it should not be very expensive to resolve both resolvers at runtime, I've noticed some performance overhead in environments where JSM (Java Security Manager) is enabled since the System.getenv() call that the LambdaSegmentContextResolver makes internally requires a permissions check (https://github.com/openjdk/jdk/blob/jdk-11%2B28/src/java.base/share/classes/java/lang/System.java#L991-L994)
Issue #, if available:
N/A
Description of changes:
See above
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I fixed the tests.
The Lambda environment variables were not set at the time the X-Ray Recorder was created so it defaulted to thread context.


我在多卡训练speech_paraformer-large-vad-punc_asr_nat-zh-cn-16k-common-vocab8404-pytorch的时候报错了

我在多卡训练的时候报了这个错误，单卡训练是没问题的
Task related config: error: unrecognized arguments: --local-rank=0
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 2) local_rank: 0 (pid: 185479) of binary: /opt/conda/envs/modelscope/bin/python
Traceback (most recent call last):
File "/opt/conda/envs/modelscope/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/conda/envs/modelscope/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/launch.py", line 196, in
main()
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/launch.py", line 192, in main
launch(args)
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/launch.py", line 177, in launch
run(args)
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/run.py", line 785, in run
elastic_launch(
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 134, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/opt/conda/envs/modelscope/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 250, in launch_agent
raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
这是我的启动命令：CUDA_VISIBLE_DEVICES=2,3 python -m torch.distributed.launch --nproc_per_node 2 finetune.py
请问有大佬知道这是什么问题吗

Bug has been fixed. Please update funasr and try it again.
https://github.com/alibaba-damo-academy/FunASR/blob/main/docs/tutorial/README_zh.md#微调


Playing mp3 from kbfs

Playing an mp3 file from inside a KBFS directory result in constantly repeated desktop notifications that the file is being decrypted.
OSX Client
File opened in VLC player.

This is intentional.  Sorry if you find it annoying, but we are trying to showcase the fact that encryption/decryption is happening, since the user can't otherwise tell.  We might dial this back or remove it in the future, but for now it's intentional.

Okay, thanks for the information.  :)


Support different formats

Reading the code https://github.com/github/view_component/blob/0fd7cbf087248f3ce9c0becb4a00f598e361e3ea/lib/view_component/base.rb#L416 and documentation, I'm unsure we are able to use different formats like :xml or :text, right?
Problem:
I have a component that I want to render as :html and :text. For that, I created two views for my component, (i.e foo_component.html.haml and foo_component.text.haml. Somehow I expected to be able to pass the format to render, however I'm not able to, right?
Possible solution:
To render text "inline", I should have a method call to my class where I bypass any assigned template and render a string inline
Update
looks like I cannot have a template (i.e foo_component.html.haml) and implement the call method to deal with the inline rendering in the same class
Is that correct or did I miss anything? Any idea how to deal with that?

@vpereira I'd be open to moving to support other formats. Could you open a PR that adds a test component with an API you think makes sense?

@vpereira I'd be open to moving to support other formats. Could you open a PR that adds a test component with an API you think makes sense?

Hi  @joelhawksley, thank you for your answer :)
So I will have to think about it. We have so many scenarios,  basically we have:
a) regular components
b) XML components (we have an XML API, where we use a lot of partials)
c) helpers with some raw() calls embedded, which we wanted to extract to partials (with different formats), and consequently it would become a component itself.
the first case, is straight forward. For XML endpoints, would just be possible if ViewComponent would support XML format. The last case, I'm kind of solving it like https://github.com/openSUSE/open-build-service/pull/11583, where for html format, i render a template and for text format, i call a simple method (and bypass any template).  BTW the "OBS" is probably one of the oldest rails code base being actively developed. We are on rails 6.1 and it exists since 2006.
For which scenario you would like to see a test component? for the XML API, or a component supporting many formats?

@vpereira thanks for sharing the example PR! How cool to see ViewComponent in use in OBS ❤
I think it would be best to prototype something that supports both raw and html like that PR would benefit from.

@vpereira I have started a discussion around how view_component handles formats because their are other issues with similar interests, and hopefully that will help shed some light on this feature

Would be helpfull to have an html and turbo_stream format


with('relation') . cause utf8 encode error

the problem is that laravel toArray add pivot data and if it include uuid it will make utf8 error
i fixed this by removing pivot from the array
if (isset($array['pivot'])){
unset($array['pivot']);
}
this is the new trait toArray

    public function toArray()
    {
        $uuidAttributes = $this->getUuidAttributes();
        $array = parent::toArray();
        if (! $this->exists || ! is_array($uuidAttributes)) {
            return $array;
        }
        foreach ($uuidAttributes as $attributeKey) {
            if (! array_key_exists($attributeKey, $array)) {
                continue;
            }
            $uuidKey = $this->getRelatedBinaryKeyName($attributeKey);
            $array[$attributeKey] = $this->{$uuidKey};
        }

  if (isset($array['pivot'])){
            unset($array['pivot']);
        }

        return $array;
    }

     Schema::create('roles', function (Blueprint $table) {
            $table->uuid('id');
            $table->string('name')->unique();
        });

        Schema::create('permissions', function (Blueprint $table) {
            $table->uuid('id')->primary();
            $table->string('name');
        });
    Schema::create('permission_role', function (Blueprint $table) {
            $table->uuid('permission_id')->index()->foreign('permission_id')->references('slug')->on('permissions')->onDelete('cascade');;
            $table->uuid('role_id')->index()->foreign('role_id')->references('id')->on('roles')->onDelete('cascade');
        });

class Role extends Model
{
    use HasBinaryUuid;

    protected $uuids = [
        'id'
    ];
  public function permissions()
    {
        return $this->belongsToMany(Permission::class,'permission_role','role_id','permission_id');

    }

}
class Permission extends Model
{
    use HasBinaryUuid;

    protected $uuids = [
        'id'
    ];

 public function roles()
    {
        return $this->belongsToMany(Role::class,'permission_role','permission_id','role_id');
    }

}

    Route::any('roles', function () {
        return Role::with('permissions')->get() ;
    });

Malformed UTF-8 characters, possibly incorrectly encoded
i think the model to Array not working on collections and join
any help

I think... toArray() works just fine in the trait. However, i need to dig deeper into the cause of this.
In your case, what does $array['pivot'] contain exactly ?

pivot: {
role_id: Binarydata,
permission_id: Binarydata
}

No need to modify the code in the manner you're suggesting. Simply add the '[pivot]' to your $hidden property array on your model like so:

<?php

class Role extends Model
{

    protected $hidden = array('pivot');

}

See: https://github.com/laravel/framework/issues/745 for more info

I think this issue can be closed out (for now). It would be reopened if there's any further concerns

This package has been abandoned. Alternatives: https://github.com/michaeldyrynda/laravel-efficient-uuid & https://github.com/michaeldyrynda/laravel-model-uuid


ESP32S2 Support second CDC channel

Hi,
we'd like to have a second CDC channel over USB, to have separate communication channels between the host and the ESP32.
Is there a plan to support this?

Thanks for raising this feature request.

would like to have this feature too. The USB protocol is capable of supporting up to 5 independent cdc ports

I added multi-cdc feature to esp-idf and tinyusb, in the form of patches. Since I'm not yet versed in git, please visit https://github.com/moefear85/esp-idf. If possible, please merge into the mainstream esp-idf.

Hello @moefear85 : Thanks for the information. We don't need this feature anymore. We took a different approach and use Ethernet-over USB. We implemented a driver for this, which you can find here: https://github.com/ci4rail/esp-idf/tree/update_mdns_comp/components/tinyusb/additions/src. Perhaps this might be also valuable for you.


Fix version info when built without git repo

When built from archive, downloaded from https://github.com/rust-lang/cargo/releases
version info looks like this: cargo 0.3.0 ( ) (built 2015-05-16).
Patch will make it look like this: cargo 0.3.0 (built 2015-05-16)

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ c6989822a2e22f8a706b1cdd113620702d11bc81
Thanks!

:hourglass: Testing commit c698982 with merge 9c5ffcb...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-64


CLI generate error, possible documentation missing?

I'm trying to execute this:
wildfly::util::exec_cli{'spring_properties':
command => '/system-property=spring.config.location:add(value=file:/opt/wildfly/)',
require => File['/opt/wildfly/application.properties'],
}
But I get this error:
Error: Failed with: WFLYCTL0383: No operation is defined {
"address" => [{"system-property" => "spring.config.location"}],
"operation" => undefined,
"operation-headers" => {"access-mechanism" => "HTTP"}
} for {"address":[{"system-property":"spring.config.location"}],"operation":null}
Error: /Stage[main]/Main/Node[default]/Wildfly::Util::Exec_cli[spring_properties]/Wildfly_cli[spring_properties]/executed: change from false to true failed: Failed with: WFLYCTL0383: No operation is defined {
"address" => [{"system-property" => "spring.config.location"}],
"operation" => undefined,
"operation-headers" => {"access-mechanism" => "HTTP"}
} for {"address":[{"system-property":"spring.config.location"}],"operation":null}
Is it me that I'm writing a bad command or there is some problem in the docs or a bug ?

Tryed with:
wildfly::util::resource { '/system-property=spring.config.location':
content => {
'value' => 'file:/opt/wildfly/'
},
require => File['/opt/wildfly/application.properties'],
}
And it works.
Feel free to close unless otherwise useful.

Sorry, wildfly_cli doesn't have a proper parser in the current version of the module, it's a simple/naive implementation to handle just a few cases, therefore, I always recommend using wildfly_resource as you did.
By the way, I've managed to implement a proper parser ([node-type=node-name (/node-type=node-name)*] : operation-name ['('[name=value [, name=value]*]')'] [{header (;header)*}]) for the next version (1.0.0), but I won't be able to backport to 0.5.x.


Renaming class with Primary+r in coder fails

To reproduce:

Place the cursor on a class in a coder
Primary+r to start a rename
Make any changes will result in a talkback

Pharo 10 changed the argument of the rename class refactoring from a class to a symbol. I updated the call to use a symbol instead of the class.


gil is required to iterate over unicode

    cdef char letter
    cdef unicode key
    for letter in key:
        pass

Calling gil-requiring function not allowed without gil
it should be possible to iterate over borrowed references of a unicode string in C chars, right?
In my case a c++ string would be sufficient, as the data is guarenteed to be alphanumerical

it should be possible to iterate over borrowed references of a unicode string in C chars, right?

Getting at the Unicode character buffer is not necessarily possible/valid without holding the GIL.

Any particular reason why? isn't it already mapped continous memory that could be read from?

The memory is not the problem. Asking the CPython API for the pointer to the memory might be. In the worst (although unlikely) case that the Unicode string isn't fully initialised yet (e.g. because it came from the Win32 or wstr API), it can even trigger a data conversion, which definitely requires the GIL.

Ah. unfortunate. Well, since ascii is sufficient for my case I guess I'll just map the unicode with gil into a char* array and use that later without gil. Thanks for clarifying.


Range is supposed to be half-open

Range is defined in Rust as half-open: so 12..127 means “at least 12, and less than 127”. In your validators you’ve implemented it with incorrect semantics, treating it as an inclusive range, which is instead 12...127 (and experimental).

I wasn't aware of that when I Implemented it, but you're correct.
What our situation is:

validators range and length uses wrong semantics for std::ops::Range
neither validator actually break since they are not using the Ranges as opposed to just using the range.start and range.end property on the Range.

Since inclusive ranges are experimental our options, as I see them, are:

let things be as they are since only semantics are broken, code still works as it should (my least favorite solution)
add the inclusive_range_syntax feature to the crate and fix the semantics in the examples and tests, which might not be a big problem since accord is requiring nightly anyways
dont use ranges, use a,b-variables instead

I'm not sure what to do right now. Will have to think about it. What are your thoughts, @chris-morgan?

I added some tests to show that range and length works as I intended them (although with wrong semantics): https://github.com/ChrisBuchholz/accord/commit/131e31c2cbd3bf1c278b5d789fcc65fb288a8f7e

Inclusive range syntax seems to be fairly unstable, and it’s quite possible that it will change syntax notably. What I observe is that other Rust APIs that could logically take inclusive ranges tend instead to take two arguments.
Then again, allowing both length(5) and length(5...8) is nice, but you can’t make functions variadic so length(5, 8) can’t work.

@chris-morgan: I brought Accord back to stable yesterday and are gonna add unstable features to Accord via feature gates. I think the solution simply is to use inclusive range on unstable and leave stable as it is right now, although its quite ugly with the half-open range on stable that isn't actually half-open.

length(5, 8) is better than length(5..8) if that means [5, 8] rather than [5, 8). Bad semantics are damaging and confusing.

@chris-morgan: you are right, the better solution is that the stable version of length and range takes two variables, and the nightly version take inclusive range.

@chris-morgan: I finally got around to fix it. Now stable uses a, b-variables and on nightly, one can use the feature gate inclusive_range to enable use of RangeInclusive<T>, which in fact the two unstable examples bundled with Accord does.


Relax requirements

Solve issue with requirements

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.18%. Comparing base (3e66ce5) to head (a112d85).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #139   +/-   ##
========================================
  Coverage    76.18%   76.18%
========================================
  Files           73       73
  Lines         3388     3388
========================================
  Hits          2581     2581
  Misses         807      807

Flag
Coverage Δ

unit-tests
76.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


chore(main): release 0.0.6

:robot: I have created a release beep boop
0.0.6 (2024-01-11)
Other

deps: update golang.org/x/exp digest to 02704c9 (#83) (ce252b1)
deps: update golang.org/x/exp digest to 0dcbfd6 (#90) (d467b99)
deps: update golang.org/x/exp digest to be819d1 (#87) (8f9a749)
deps: update golang.org/x/exp digest to dc181d7 (#79) (863e769)
deps: update module github.com/go-logr/logr to v1.4.0 (#81) (3656433)
deps: update module github.com/go-logr/logr to v1.4.1 (#82) (d552896)
deps: update module github.com/spectrocloud-labs/validator to v0.0.30 (#80) (1fbe4c6)
deps: update module github.com/spectrocloud-labs/validator to v0.0.31 (#84) (4588d3e)
deps: update module github.com/spectrocloud-labs/validator to v0.0.32 (#85) (11832ca)
deps: update actions/upload-artifact action to v4 (#68) (6d63031)
deps: update anchore/sbom-action action to v0.15.2 (#86) (29a19ac)
deps: update anchore/sbom-action action to v0.15.3 (#88) (95965e7)
deps: update gcr.io/spectro-images-public/golang docker tag to v1.22 (#89) (9397d20)
deps: update google-github-actions/release-please-action digest to cc61a07 (#77) (18e976f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/spectrocloud-labs/validator-plugin-azure/releases/tag/v0.0.6 :sunflower:


A log error isnt completely formated by template engine.

When entering wrong email configuration (or if the mail server has an error), the following error log is generated :
[2017-08-30 15:38:18,341] [ERROR] Cannot send email SupExt alert for {{ service }}

i think that {{ service }} should have been replaced by the name of the test probe.

Fixed by PR #2


auth for "https:" not found

Creating an Image repository with this YAML
apiVersion: image.toolkit.fluxcd.io/v1alpha2
kind: ImageRepository
metadata:
  name: inspector
  namespace: flux-system
spec:
  image: https://quay.io/repository/adoreme/inspector
  interval: 1m0s
  secretRef:
    name: quay-secret

and this docker-registry secret
{"auths":{"https://quay.io":{"username":"****","password":"****","auth":"****"}}}

results in this error
auth for "https:" not found in secret flux-system/quay-secret

This issue is gotten from this message on slack

The string given in the image: field is not a valid image ref -- there should not be a scheme (https://). It is an easy mistake to make, because image refs do include a hostname and a path. If we can detect this mistake, we can either correct for it or flag it specifically in an error.

@relu could you help out with this? I think we should parse the image URI and error out in a meaningful way.

Indeed, removing the https:// fixed my issue. I think a heads up in the docs or a better error message as @squaremo suggested should do it.
From my part you guys can close this issue.
Great job and thanks for the help 👍

Thanks for following up on this, I'm going to go ahead and close it.

Sorry, I did not see that there was an assignment already here.


Child container resources are never released

Hi,
while playing with the library I found an unexpected behavior: resources of child containers are never released and they are being keep in the service's memory even after being disposed.
All problematic resources are being referenced in disposables field of the ResolutionScope class.
I prepared sample code that reproduces the issue.
using Stashbox;
using Stashbox.Configuration;

var container = new StashboxContainer(c =>
{
    c.WithRegistrationBehavior(Rules.RegistrationBehavior.PreserveDuplications);
});

while (true)
{
    var scope = container.CreateChildContainer();

    RegisterToContainer(container);

    var x = scope.Resolve<IAnimalRepository>();

    GC.KeepAlive(x);

    scope.Dispose();

    GC.Collect();
    GC.WaitForPendingFinalizers();
    GC.Collect();
}

Console.ReadKey();

static void RegisterToContainer(IStashboxContainer container)
{
    container.Register<AnimalRepository>(c => c.WithScopedLifetime().AsImplementedTypes());

    //another trial
    container.RegisterDecorator<CachedAnimalRepository>(c => c.AsImplementedTypes().When(t => t.Type.IsAssignableTo(typeof(IAnimalRepository))));
}

internal interface IDataRepository : IDisposable
{
    void CommonMethod()
    {
    }

    void IDisposable.Dispose()
    {
    }
}

internal interface IAnimalRepository : IDataRepository
{
}

internal sealed class AnimalRepository : IAnimalRepository
{
}

internal sealed class CachedAnimalRepository : IAnimalRepository
{
    public CachedAnimalRepository(IAnimalRepository service)
    {
    }
}

After a while we have hundreds of the AnimaRepository instances in the memory:

For testing reasons I have cleaned manually disposables field of a main container using a VS debugger and after that all disposed resources were collected by GC.
I cannot reproduce this issue while using scopes instead of child containers.
It would be nice to have it fixed because this may lead to memory leak :)

I wonder why you do the registration on the main container and not on the child container (scope).

Hi @eValker, thank you for reaching out!
As @schuettecarsten pointed out, the issue with your test is that in your loop you register the services into the main container, which never gets disposed. You dispose only the child container that doesn't contain any services, so it won't dispose anything.
The test would be correct if you pass the child container to the registration method like: RegisterToContainer(scope);.
Let me know what you think!

Hello,
Sure, you are both right :) I meant to do registrations on the child container, but I was testing different things and I accidentally pasted wrong sample code.
However it does not change anything - issue remains the same - child containers are not being properly released. Main root scope (on the main container) keeps references to the child containers and to all services instances inside those containers. Because of that GC cannot collect them, so they remains in the memory.
Changed sample code:
using Stashbox;
using Stashbox.Configuration;

var container = new StashboxContainer(c =>
{
    c.WithRegistrationBehavior(Rules.RegistrationBehavior.PreserveDuplications);
});

while (true)
{
    var childContainer = container.CreateChildContainer();

    RegisterToContainer(childContainer);

    var x = childContainer.Resolve<IAnimalRepository>();

    GC.KeepAlive(x);

    childContainer.Dispose();

    GC.Collect();
    GC.WaitForPendingFinalizers();
    GC.Collect();
}

Console.ReadKey();

static void RegisterToContainer(IStashboxContainer container)
{
    container.Register<AnimalRepository>(c => c.WithSingletonLifetime().AsImplementedTypes());

    //another trial
    container.RegisterDecorator<CachedAnimalRepository>(c => c.AsImplementedTypes().When(t => t.Type.IsAssignableTo(typeof(IAnimalRepository))));
}

internal interface IDataRepository : IDisposable
{
    void CommonMethod()
    {
    }

    void IDisposable.Dispose()
    {
    }
}

internal interface IAnimalRepository : IDataRepository
{
}

internal sealed class AnimalRepository : IAnimalRepository
{
}

internal sealed class CachedAnimalRepository : IAnimalRepository
{
    public CachedAnimalRepository(IAnimalRepository service)
    {
    }
}

And memory state after a while of running:

As you can see, even after closing and disposing child container, we have over 2k instances of the AnimalRepository in the memory. Proper behavior IMO is that we should have single instance of that service.

Ah, I see the issue now, somehow I totally skipped your explanation about the disposables, and yes, that field is the bad guy here. Thank you for pointing it out! I'm going to work on the fix.

@eValker I've released a new version v5.11.1 with the fix, could you please check that it works now at your end as expected? Thanks!

Thank you @z4kn4fein for the fast fix :)
I can confirm that now child containers are properly released.


Major version bump to prevent backwards-incompatible changes

 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 posting this Ricky and sorry to cause frustration for you. That said, I don't think we're stable enough yet to publish a major API version.

Apologies, I called you Ricky, Allan. updated the original comment.

I believe that we should bump to 1.0.0 because other projects depend on the stability of this code and the files that it generates. It's a 2 year old project that has relatively few changes, so it's fairly stable in this regard. Bumping to 1.0.0 doesn't mean that the code can't be updated regularly, it only defines that breaking changes must bump the major version, while new features or backwards compatible changes should bump the minor and patch versions (patch is mostly used for bug fixes).
It adds clarity to changes and stability for end users, it doesn't demand that the code stop iterating.


Security static analysis

Hi, I was looking around but there is nothing in this project regarding security static code analysis. Is there any plan to implement that in the future?
Thanks

Can you give an example of "security static code analysis"?


Add Scylla Sphinx Theme to Rust Driver Docs

Just like the Python and Java Driver, the Scylla Sphinx Theme should be applied to the Rust Driver.
The Process includes:

Following the steps - https://github.com/scylladb/sphinx-scylladb-theme/blob/master/README.rst
Testing the output
Making sure everything we display is branded Scylla not Datastax
Making sure the links are all directed to Scylla
Adding multi-version when the next version goes out.

Fixed in https://github.com/scylladb/scylla-rust-driver/pull/215


Deadline: missing OPENPYPE_VERSION job submitter

Running version
3.15.2-nightly.3
Describe the bug
Broken workflow in case job is submited from source rather then build version.
=======================================================
Error
=======================================================
Error: FailRenderException : OpenPype executable was not found in the semicolon separated list "". The path to the render executable can be configured from the Plugin Configuration in the Deadline Monitor.
   at Deadline.Plugins.DeadlinePlugin.FailRender(String message) (Python.Runtime.PythonException)
  File "C:\ProgramData\Thinkbox\Deadline10\workers\JEZA-ROG-rog-1\plugins\640089c1e2dbd50cd85667fa\OpenPype.py", line 185, in RenderExecutable
    "from the Plugin Configuration in the Deadline Monitor.")
   at Python.Runtime.Dispatcher.Dispatch(ArrayList args)
   at __FranticX_GenericDelegate0`1\[\[System_String\, System_Private_CoreLib\, Version=4_0_0_0\, Culture=neutral\, PublicKeyToken=7cec85d7bea7798e\]\]Dispatcher.Invoke()
   at FranticX.Processes.ManagedProcess.RenderExecutable()
   at Deadline.Plugins.DeadlinePlugin.RenderExecutable()
   at FranticX.Processes.ManagedProcess.Execute(Boolean waitForExit)
   at Deadline.Plugins.PluginWrapper.RenderTasks(Task task, String& outMessage, AbortLevel& abortLevel)

Deadline OpenPype settings

To Reproduce
Steps to reproduce the behavior:

Submit job from your dcc run from OpenPype at source

[cuID:OP-5158]

Got introduced here; https://github.com/ynput/OpenPype/pull/4489

It should be possible to change the directory path to {your openpype repo path}/tools. There should be openpype_console.bat file that can replace openpype_console.exe.

Closed by https://github.com/ynput/OpenPype/pull/4552


Change CouchDB image

Currently for CouchDB we are using images provided by Hyperledger. It's not good option anymore because:

It's deprecated: https://github.com/hyperledger/fabric/releases

Fabric CouchDB image is deprecated
v2.2.0 added support for CouchDB 3.1.0 as the recommended and tested version of CouchDB.
If prior versions are utilized, a Warning will appear in peer log.
Note that CouchDB 3.1.0 requires that an admin username and password be set, while this was optional in CouchDB v2.x. See the Fabric CouchDB documentation for configuration details.
Also note that CouchDB 3.1.0 default max_document_size is reduced to 8MB. Set a higher value if needed in your environment.
Finally, the fabric-couchdb docker image will not be updated to v3.1.0 and will no longer be updated, maintained, or published.
Users can utilize the official CouchDB docker image maintained by the Apache CouchDB project instead.

Seems that it doesn't work in some setups: https://github.com/hyperledger-labs/fablo/issues/263

Resolved in https://github.com/hyperledger-labs/fablo/pull/269


Project Manager validation of duplicated children

List of bugs in Project Manager

https://user-images.githubusercontent.com/2439881/120720487-729a0100-c4cc-11eb-8626-4b001cf6e1a5.mp4
3.0.0 tag
Hitting this bug where I can't create a sibling asset.

Visually it looks like you can change the Task type on existing saved tasks - but in practice you can't. It correctly does NOT propagate to the database - still it's confusing.
https://user-images.githubusercontent.com/2439881/120720784-f8b64780-c4cc-11eb-82a4-f1bc306aa238.mp4

Not sure how I'm having this happen - but it tends to get to a state where pressing Save does nothing. Where should I look to see what might be causing this?
https://user-images.githubusercontent.com/2439881/120721251-b93c2b00-c4cd-11eb-889f-e614153cdf6b.mp4

Selecting a Task and then clicking on Add Asset crashes the tool. :)
Traceback (most recent call last):
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\window.py", line 189, in _on_add_asset
    self.hierarchy_view.add_asset()
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\view.py", line 404, in add_asset
    return self._source_model.add_new_asset(parent_index)
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\model.py", line 396, in add_new_asset
    result = self.add_item(new_child, item, new_row)
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\model.py", line 459, in add_item
    result = self.add_items([item], parent, row)
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\model.py", line 440, in add_items
    parent.add_child(item, row)
  File "S:\openpype\OpenPype\openpype\tools\project_manager\project_manager\model.py", line 1949, in add_child
    raise AssertionError("BUG: Can't add children to Task")
AssertionError: BUG: Can't add children to Task

The crash could be avoided by instead maybe popping up a pop-up using sys.excepthook that shows the error instead.

Double clicking (or pressing ENTER) on the name of existing assets (that are in database) allows you to rename the asset and save it to the database. Should renaming an asset be allowed? Is this correct behavior?

Hitting this bug where I can't create a sibling asset.
and
Not sure how I'm having this happen - but it tends to get to a state where pressing Save does nothing. Where should I look to see what might be causing this?

Fixed with: https://github.com/pypeclub/OpenPype/pull/1645
Description

save did not trigger refreshing of attributes holding data and asset duplications

Double clicking (or pressing ENTER) on the name of existing assets (that are in database) allows you to rename the asset and save it to the database. Should renaming an asset be allowed? Is this correct behavior?

It won't allow to change asset name when has published content (at least one subset).

Selecting a Task and then clicking on Add Asset crashes the tool. It takes down Pype Tray along with it.

Fixed with https://github.com/pypeclub/OpenPype/pull/1646

Visually it looks like you can change the Task type on existing saved tasks - but in practice you can't. It INCORRECTLY does propagate the change to the database.

I think this was issue connected with #1645 Let me know if issue persist because it works on my side.

By the way, I started posting multiple bugs here due to the top message saying "List of bugs" but I'm not sure whether it's alright I'm actually dropping all of these in one issue.

As it's in beta stage it is better to have it here. As you can see 2-3 issue were solved with 1 PR because were cause due to same bug cause by missing 2 lines.

Closing as it seems that all initial bugs are fixed and all other should be reported as separated issues.


Use parent dir from tar file

Also allow the user to specify where to put the lib/ directory.
This simplifies setting the environment for automated builds.

Pending testing


"This app can't run on your PC"

Downloaded the two files in the bin, as the instructions state. Placed them into my existing SmartHunter folder.
And Windows told me to go take a hike.

Are you running a 32-bit version of Windows? I wouldn't at all be surprised if it's a 64-bit binary.

Are you running a 32-bit version of Windows? I wouldn't at all be surprised if it's a 64-bit binary.

@gabrielefilipp - is it possible there's a deviation between the application and the 1909 build of W10? I can't imagine why windows would tell me the application doesn't work. It's just an executable...
I'm not an expert. But I am confused.

@SpeedJaeger I don’t know neither. You can try to build the app yourself: just clone the repo, install Visual Studio with C# and build it. I tried to google for that message “This app can’t run on this PC” but I didn’t find anything useful

@SpeedJaeger I don’t know neither. You can try to build the app yourself: just clone the repo, install Visual Studio with C# and build it. I tried to google for that message “This app can’t run on this PC” but I didn’t find anything useful

So I went ahead and deleted my original folder and just tried again with a fresh download of the two and it still does the same thing. I did notice the executable is zero kb's though, and I'm pretty sure that's not a thing.

I'm not versed enough in coding to confidently do what you mentioned for me to be honest. I'd have to offload that to a friend.  You can guess how successful that was. Running as admin doesn't help either. It seems the executable isn't set up correctly once it hits my system. I can't imagine my AV would do anything to it either. ESET and MBAM.
I had a friend go and download it from here (I bugged him cause he's a decent coder) and it worked for him. So I'm at a loss.

@SpeedJaeger I just saw this flicking through on the off chance, I'd recently been having the same issue when downloading the files from the Debug folder. Hopefully what caused my issues (and the same Windows prompt) are the same as yours.
You mentioned that the .exe file is 0 KB, it may be an issue with your browser. If you're using Chrome, double check that there isn't a prompt on the download bar along the lines of "SmartHunter.exe is not commonly downloaded and may be dangerous."

If so, make sure you hit the context arrow and select Keep, then see if the download size matches that of the Github directory. Then repeat for the .dll file. I'd assume that there's a similar set of hoops to jump through with other browsers, too, though I can't comment.
Hopefully this helps, happy hunting!

@stolenmateria
Worked like a charm. I never actually stopped to read the download in Firefox so I didn't see the text about it not being common. It didn't look unusual or alert me to it. So I was never actually completing the download.
The bottom is my original attempt at download last night. The top is the file, actually downloaded (and then moved to its new folder).

Thanks for the assist.
@gabrielefilipp - Something like that may be worth noting somewhere in the notes if it isn't already.


add linux_ppc64le, linux_aarch64, osx_arm64

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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.

This is ready for review @conda-forge/slang

A gentle ping for a review  @conda-forge/slang @conda-forge/core


Build a shared library and install headers

I am curious why luau does not build a common shared library so one can package/update it separately from whatever project uses it? That's how it's done in the reference Lua and LuaJIT. Luau seems to only build static archives and hence needs to be linked statically. Any idea why -- is that by design or did the use case just not come up?
I'm looking to package Luau for Debian and only shipping static libraries is kind of frowned upon, since that would mean that an update of the library would need rebuilds of everything that depends on it if there are only static libraries.
I (and maybe others as well) would prefer to have a clear, installed set of library and headers (maybe also pkgconfig info etc) that would make Luau as a library immediately usable as most developers would expect it, with the least amount of surprise.

This mostly just didn't come up. All production applications that use Luau so far ship with statically linked library I believe.
A couple things to know off the top of my head:

Luau currently doesn't have a completely set ABI for VM. Our API should be source-level stable by now, but we occasionally make changes that break ABI while preserving API via macros for example. I am not sure what that does to SONAMEs et al for packaging; this doesn't happen often but it does happen, and we don't really want to use major version of the actual releases to indicate ABI level compatibility.
I briefly tried to do shared library builds out of curiosity a few months ago and we had issues around CodeGen component integration, in that it wasn't possible to compile Luau.VM and Luau.CodeGen as separate shared libraries because CodeGen relied on a host of internal symbols that VM defined but didn't officially export. fyi @vegorov-rbx - I was thinking about either moving NativeState to VM and exposing that through a separate function (eg lnative.cpp/h) that we export, or about exporting all functions CodeGen relies on but we'd need a way to actually test that the exports are complete in that case. We could also technically export all internal functions although that doesn't feel great either.


SwitchCaseSemicolonToColonFixer - Fix bug with nested constructs

closes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3103

Thank you @SpacePossum.


Copyright Songs

How can i listen to Copyrighted songs... The most songs are copyrighted wich means i cant even use this library

How can i listen to Copyrighted songs... The most songs are copyrighted wich means i cant even use this library

Maybe update the xsound?

Well anyway, update xsound and it will work fine.
if you can't update because you have "crewPhone" a phone that can play music from youtube, look in the config for addon list and switch the value from false to true & copy, paste the music files from "his" version.
There won't be any further support because if you cannot drag and drop files + switch from false to true


[fud2] Remove flags argument from calyx stage

The calyx stage has both flags and args which seem to be doing the same thing. Also, flags defaults to -p none which is a bad default to use with the compiler.

Fixed by #2334.


content update

Summary by Bito
Removal of two UI components: personal information page and user dashboard. The removed pages contained user profile editing functionality and dashboard overview features including personal details, order history, and VIP status information. Both pages shared common UI elements.

Unit tests added: False

Estimated effort to review (1-5, lower is better): 2

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

Changelist by Bito
This pull request implements the following key changes.

Key Change
Files Impacted

Feature Improvement - User Interface Pages Removal

- personal_info.php - Removed personal information page with user profile editing functionality

- user dashboard.php - Removed user dashboard page containing user information and navigation

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

Code Review Agent Run Status

Limitations and other issues: ❌ Failure - The AI Code Review Agent skipped reviewing this change because it is configured to exclude certain pull requests based on the source/target branch or the pull request status. You can change the settings here, or contact the agent instance creator at<EMAIL_ADDRESS>


Why not depend on firebase-token-generator-python instead of copying it?

The comment at the top of firebase/firebase_token_generator.py :
##############################################################################
# THE ENTIRE CODE HAS BEEN TAKEN FROM THE OFFICIAL FIREBASE GITHUB           #
# REPOSITORY NAMED `firebase-token-generator-python` WITH SLIGHT             #
# MODIFICATIONS.                                                             #
#                                                                            #
# FOR MORE INFORMATION, PLEASE TAKE A LOOK AT THE ACTUAL REPOSITORY:         #
#  - https://github.com/firebase/firebase-token-generator-python             #
##############################################################################

Why not just depend on it in your setup.py / requirements.txt? Also, for the more paranoid among us, what exactly were the "slight modifications" made to the original code? Unless you have an excellent reason (which are rare), modifying someone else's crypto code doesn't seem like a good idea (easy to mess up, you miss out on future bug fixes, etc.). I would recommend just depending on the original code rather than maintaining a copy of it here.

+1. You can still easily create an OO wrapper around it, since that seems to be the local modifications. The resulting code in your local repo would be tiny.
They seem to have changed their code to support python 3 and python < 2.7 in a different way than you have, and you're missing those fixes in your library by copy pasting and letting the code get out of sync.


Adding GraphHopper Directions API

Currently only our Route Optimization API has a swagger spec, but the other APIs will follow at some point :)
We also create the java client out of the spec which is really nice :)

@karussell Thank you for submission.
I already added your API on Nov 16, 2015 :)
But it's really great to collaborate with API owners, please check if your API correctly represented:
https://github.com/APIs-guru/api-models/blob/master/APIs/graphhopper.com/1.0/swagger.yaml
P.S. You can use your Swagger spec not only to generate SDKs but also to power API docs.
For example. you can use documentation engine we developed for one of our clients:
https://rebilly.github.io/ReDoc/?url=https://apis-guru.github.io/api-models/graphhopper.com/1.0/swagger.yaml
Here is demo with all of the features:
https://rebilly.github.io/ReDoc/

Oh, wow & ups. Never thought that we are already in the list and so I didn't care reading :)
Thanks!

But it's really great to collaborate with API owners, please check if your API correctly represented:

Looks good. Do you update this somehow? As we often add new features.

P.S. You can use your Swagger spec not only to generate SDKs but also to power API docs.

Yeah, we know and did, but at the end it was more descriptive and readable for our customers via self-written docs.
Thanks for the link to rebilly&redoc we'll reconsider this and think about it :) !
BTW: something is wrong with my font (firefox&linux) - see the 'pagination':

Looks good. Do you update this somehow? As we often add new features.

@karussell Yes, see this:
https://github.com/APIs-guru/api-models#update-procedure

BTW: something is wrong with my font (firefox&linux) - see the 'pagination':
Thank you for reporting it :+1:
Which browser&OS are you using?

Thank you for reporting it :+1:
Which browser&OS are you using?

firefox 46.0.1 under ubuntu 14.04.1 with 3.16.0-71-generic


Add new required(ish) "parser" parameter to markdownlint custom rule examples in Content Linter README.md.

Why:
parser is a new required (tho it's okay if it's not present for now) property on markdownlint custom rules. This PR adds it to the two relevant examples.
Closes:
N/A

What's being changed (if available, include any code snippets, screenshots, or gifs):
README

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@DavidAnson Thanks so much for opening a PR! I'll get this triaged for review ✨

Hi @DavidAnson thanks for reaching out and for the contribution! I did notice it recently when I was looking at updating one of our custom rules. I'll make sure to update them all and get this added. Thanks again!

FYI @gracepark if you are a rule maintainer, this new property does nothing at the moment, but I intend to use it soon to avoid loading the markdown-it parser once I have migrated all the core rules to micromark. Of course, if there are rules that need markdown-it tokens, it will be used and the tokens will still be available. From what I saw, GitHub rules take advantage of it, so there may not be a performance benefit for you folks. micromark is not yet available for public use, but that is also planned. Finally, note that if a rule doesn't use tokens at all, it can declare that.

@DavidAnson - yes I'm apart of the team that maintains the rules. I'll let the others know as well. Thanks for the heads up.


Unable to add QR image ios

Unable to link QR image shared to someone in IoS.
User gets just gift text not the QR image linked to it.

https://github.com/bithyve/hexa/issues/6328 duplicate

duplicate issue


Add @mozilla/readability as document transformer

Adds @mozilla/readability as Document transformer. I didnt actually do any tests comparing html-to-text with @mozilla/readability, but I am planning to.
It does basically the same as html-to-text, except different.
@bjoern_rave on twitter ✌

This is excellent! Didn't know about this one - would love to see a comparison.
Will merge tonight!


Added a max length to the log with default

In very large projects with 10's of thousands of commits, Atom was crashing so I implemented a max length for the list that can be changed in the settings view.

Yeah! This would probably really help with #47 and #21 as well.
This package is pretty killer, but the performance is important, especially in production codebases (My main one has 23,000 commits).


create wrapper check() function

Codecov Report

Merging #24 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   88.47%   88.47%
=======================================
  Files           9        9
  Lines         269      269
=======================================
  Hits          238      238
  Misses         31       31

Continue to review full report at Codecov.

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

👍 updated @joyeecheung

Thanks!


Firmware DUMPS

Hello
here i can provide some firmware dumps from several GPON SFP modules. Can be useful
https://github.com/BittorB/SFP_GPON_FIRMWARES

Hi,
due to copiright issues, as a policy we don't archive any firmware.


docs: add section about host

Documents the :host tag used to style the prefixed element :sparkles:

looks good to me, thanks for this @yoshuawuyts. :smile:

landed as https://github.com/sheetify/sheetify/commit/da894b0c10e90b7817cd11a35715201e114a0b7f, published v4.0.5.

:tada: wooh!