Prisma adding NOT specified field to filter condition

Bug description
I'm executing a query to delete all the records that meet a condition:
for some reason when executing deleteMany({where: field1: var1, field2: { in: arr1 }}) the logs show
DELETE FROM mytable WHERE (field1, field2, field3) IN (?, ?, ?)

Why is prisma adding the third field, obviously the records are not deleted because I'm not providing that field3 value.
this table has an @@id([field1, field2, field3])
doing executeRaw works well.
client.$executeRaw`DELETE FROM mytable WHERE field1 = ${var1} AND field2 IN (${arr1});`;

How to reproduce
deleteMany({where: field1: var1, field2: { in: arr1}});

Expected behavior
The third NO specified field should not be included in the where clause.
Prisma information
model mytable {
  field1  String   @db.VarChar(36)
  field2  String   @db.VarChar(36)
  field3  String   @db.VarChar(36)

  @@id([field1, field2, field3])
}

Environment & setup

OS: Windows
Database: MariaDB
Node.js version:  v14.17.6

Prisma Version
3.1.1

Why is prisma adding the third field, obviously the records are not deleted because I'm not providing that field3 value.

Is it possible that record is actually getting deleted? Prisma in this case first does a SELECT query to get all the items the where matches, and then runs a DELETE query for these by using their @id values:
C:\Users\Jan\Documents\throwaway\9417<EMAIL_ADDRESS>λ npx ts-node script.ts
  prisma:tryLoadEnv Environment variables loaded from C:\Users\Jan\Documents\throwaway\9417\.env +0ms
  prisma:tryLoadEnv Environment variables loaded from C:\Users\Jan\Documents\throwaway\9417\.env +5ms
[dotenv][DEBUG] "DATABASE_URL" is already defined in `process.env` and will not be overwritten
  prisma:client clientVersion: 3.1.1 +0ms
  prisma:client:libraryEngine internalSetup +0ms
  prisma:client Prisma Client call: +24ms
  prisma:client prisma.mytable.create({
  prisma:client   data: {
  prisma:client     field1: 'foo',
  prisma:client     field2: 'bar',
  prisma:client     field3: 'telephone'
  prisma:client   }
  prisma:client }) +7ms
  prisma:client Generated request: +7ms
  prisma:client mutation {
  prisma:client   createOnemytable(data: {
  prisma:client     field1: "foo"
  prisma:client     field2: "bar"
  prisma:client     field3: "telephone"
  prisma:client   }) {
  prisma:client     field1
  prisma:client     field2
  prisma:client     field3
  prisma:client   }
  prisma:client }
  prisma:client  +1ms
  prisma:client:libraryEngine sending request, this.libraryStarted: false +47ms
  prisma:client:libraryEngine Search for Query Engine Library in C:\Users\Jan\Documents\throwaway\9417\node_modules\.prisma\client +2ms
  prisma:client:libraryEngine loadEngine using C:\Users\Jan\Documents\throwaway\9417\node_modules\.prisma\client\query_engine-windows.dll.node +1ms
  prisma:client:libraryEngine library starting +17ms
prisma:info Starting a postgresql pool with 9 connections.
  prisma:client:libraryEngine library started +269ms
prisma:query BEGIN
prisma:query INSERT INTO "public"."mytable" ("field1","field2","field3") VALUES ($1,$2,$3) RETURNING "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3"
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE ("public"."mytable"."field1" = $1 AND "public"."mytable"."field2" = $2 AND "public"."mytable"."field3" = $3) LIMIT $4 OFFSET $5
prisma:query COMMIT
  prisma:client Prisma Client call: +458ms
  prisma:client prisma.mytable.create({
  prisma:client   data: {
  prisma:client     field1: 'foo',
  prisma:client     field2: 'baz',
  prisma:client     field3: 'airplane'
  prisma:client   }
  prisma:client }) +2ms
  prisma:client Generated request: +5ms
  prisma:client mutation {
  prisma:client   createOnemytable(data: {
  prisma:client     field1: "foo"
  prisma:client     field2: "baz"
  prisma:client     field3: "airplane"
  prisma:client   }) {
  prisma:client     field1
  prisma:client     field2
  prisma:client     field3
  prisma:client   }
  prisma:client }
  prisma:client  +1ms
  prisma:client:libraryEngine sending request, this.libraryStarted: true +177ms
prisma:query BEGIN
prisma:query INSERT INTO "public"."mytable" ("field1","field2","field3") VALUES ($1,$2,$3) RETURNING "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3"
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE ("public"."mytable"."field1" = $1 AND "public"."mytable"."field2" = $2 AND "public"."mytable"."field3" = $3) LIMIT $4 OFFSET $5
prisma:query COMMIT
  prisma:client Prisma Client call: +103ms
  prisma:client prisma.mytable.findMany({}) +3ms
  prisma:client Generated request: +2ms
  prisma:client query {
  prisma:client   findManymytable {
  prisma:client     field1
  prisma:client     field2
  prisma:client     field3
  prisma:client   }
  prisma:client }
  prisma:client  +3ms
  prisma:client:libraryEngine sending request, this.libraryStarted: true +111ms
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE 1=1 OFFSET $1
items_before [
  { field1: 'foo', field2: 'bar', field3: 'telephone' },
  { field1: 'foo', field2: 'baz', field3: 'airplane' }
]
  prisma:client Prisma Client call: +69ms
  prisma:client prisma.mytable.deleteMany({
  prisma:client   where: {
  prisma:client     field1: 'foo',
  prisma:client     field2: {
  prisma:client       in: [
  prisma:client         'bar',
  prisma:client         'baz'
  prisma:client       ]
  prisma:client     }
  prisma:client   }
  prisma:client }) +3ms
  prisma:client Generated request: +12ms
  prisma:client mutation {
  prisma:client   deleteManymytable(where: {
  prisma:client     field1: "foo"
  prisma:client     field2: {
  prisma:client       in: ["bar", "baz"]
  prisma:client     }
  prisma:client   }) {
  prisma:client     count
  prisma:client   }
  prisma:client }
  prisma:client  +2ms
  prisma:client:libraryEngine sending request, this.libraryStarted: true +86ms
prisma:query BEGIN
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE ("public"."mytable"."field1" = $1 AND "public"."mytable"."field2" IN ($2,$3)) OFFSET $4
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE ("public"."mytable"."field1" = $1 AND "public"."mytable"."field2" IN ($2,$3))
prisma:query DELETE FROM "public"."mytable" WHERE ("public"."mytable"."field1","public"."mytable"."field2","public"."mytable"."field3") IN (($1,$2,$3),($4,$5,$6))
prisma:query COMMIT
deleted { count: 2 }
  prisma:client Prisma Client call: +182ms
  prisma:client prisma.mytable.findMany({}) +2ms
  prisma:client Generated request: +3ms
  prisma:client query {
  prisma:client   findManymytable {
  prisma:client     field1
  prisma:client     field2
  prisma:client     field3
  prisma:client   }
  prisma:client }
  prisma:client  +1ms
  prisma:client:libraryEngine sending request, this.libraryStarted: true +186ms
prisma:query SELECT "public"."mytable"."field1", "public"."mytable"."field2", "public"."mytable"."field3" FROM "public"."mytable" WHERE 1=1 OFFSET $1
items_after []
  prisma:client:libraryEngine library stopping +35ms
  prisma:client:libraryEngine library stopped +3ms
  prisma:client:libraryEngine hookProcess received: beforeExit +2ms
  prisma:client:libraryEngine runBeforeExit +1ms
  prisma:client:libraryEngine hookProcess received: exit +2ms

Code used for this reproduction additionally to the schema above:
import { PrismaClient } from '@prisma/client'

const prisma = new PrismaClient({
    log: ['query', 'info', 'warn', 'error'],
  })

// A `main` function so that we can use async/await
async function main() {
    await prisma.mytable.create({ data: {
        field1: "foo",
        field2: "bar",
        field3: "telephone"
    }})
    await prisma.mytable.create({ data: {
        field1: "foo",
        field2: "baz",
        field3: "airplane"
    }})

    const items_before = await prisma.mytable.findMany({})
    console.log('items_before', items_before)

    const arr1 = ["bar", "baz"]
    const var1 = "foo"
    const deleted = await prisma.mytable.deleteMany({ where: { field1: var1, field2: { in: arr1 }}})
    console.log('deleted', deleted)

    const items_after = await prisma.mytable.findMany({})
    console.log('items_after', items_after)
}

main()
    .catch((e) => {
        console.error(e)
        process.exit(1)
    })
    .finally(async () => {
        await prisma.$disconnect()
    })

@janpio thanks for taking the time to review. you're actually right the records are being deleted. I was focused on the slowness and didn't realized the records were deleted. By slowness I mean deleteMany takes about 25 seconds to delete 30K records in my dev env and the below statement takes less than a second.
client.$executeRawDELETE FROM mytable WHERE field1 = ${var1} AND field2 IN (${Prisma.joinarr1)});;
Probably something you should take into consideration.


Does not work with strict-di enabled.

If you enabled strict dependency injection it doesn't work cause apparently the provider does not have explicit annotation.
For more info, see here

Fixed by #11.


Cloudstack <IP_ADDRESS> - Static NAT not functioning in Advanced Zone with GRE Isolation

ISSUE TYPE

Bug Report

COMPONENT NAME
VPC / Router VM
CLOUDSTACK VERSION
<IP_ADDRESS>
CONFIGURATION
Advanced Network
GRE Isolation
XenServer 7.2
1 physical network hosting all network traffic types
SUMMARY
I am able to provision a Static NAT for a VPC defined in an Advanced Zone using GRE Isolation. The Static NAT does not behave as expected.
From the router VM I am able to SSH to the public IP allocated for the Static NAT; It redirects to the instance. Looking at the Iptables and routing table it looks like the router VM is not prepared to forward traffic as desired.
STEPS TO REPRODUCE
Create an Static NAT
Try to SSH / Ping the Public IP
EXPECTED RESULTS
Connection established / ping reply
ACTUAL RESULTS
Connection timeout / ping timeout

I tried troubleshooting this. I added an ACL with an ingress/egress rules to allow all traffic. I was not able to ping / ssh the instance behind the public ip. I had a look at the iptables and routing table and I do not see how the packet would be forwarded.

I have been digging into this further. I ran iptables-save on the router before and after adding a new ACL to the VPC. The ACL had two rules (ingress/egress) to allow all traffic form all protocols (<IP_ADDRESS>/0 CIDR). The iptables showed no difference at all.

I also tried adding an ACL to use the <IP_ADDRESS>/8 subnet (which is the VPC WAN) and saw no changes to the router.

As we discussed, this is not a problem. It was a misunderstanding when using ACLs.

Thanks @rafaelweingartner. Yes, user error on my part.


Updated Linux dependency list

Updated Linux package dependencies for Debian/Ubuntu systems now that
Lumina uses Qt5.

Thanks for sending that in!


Copyright and branding revamp

But it works OK with master branch in my localhost. So some modification from this PR incurs this error.

With master branch in my localhost, site works correctly

@rhdxmr That's correct, I failed to upload the CSS. Works on my box now.


Type-o in module

Great project! :)
Just noticed a type-o here while looking at the mappings and wonder why formatting wasn't working...
https://github.com/ray-x/navigator.lua/blob/master/lua/navigator/formatting.lua#L34
Also, is this a type-o too or is this what it's supposed to be? It's binding in normal mode but uses a _v function?
https://github.com/ray-x/navigator.lua/blob/master/lua/navigator/lspclient/mapping.lua#L64
I'd fork and create a PR, but I'm pressed for time currently and decided to post this instead since it's so small.

You are correct, there was a typo.
Also the range_format keymap only works if the lsp support it. AFAIK, most lsp does not support this feature.

Error executing vim.schedule lua callback: vim/_init_packages.lua:0: .../start/navigator.lua/lua/navigator/lspclient/mapping.lua:392: 'end' expected (to close 'function' at line 131) near '<eof>'
stack traceback:
        [C]: in function 'error'
        vim/_init_packages.lua: in function <vim/_init_packages.lua:0>
        [C]: in function 'require'
        ...r/start/navigator.lua/lua/navigator/lspclient/attach.lua:33: in function 'on_attach'
        .../start/navigator.lua/lua/navigator/lspclient/clients.lua:396: in function '_on_attach'
        ...ck/packer/start/nvim-lspconfig/lua/lspconfig/configs.lua:280: in function '_setup_buffer'
        ...ck/packer/start/nvim-lspconfig/lua/lspconfig/configs.lua:183: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

thanks, should been fixed #219


WIP: Add Pulumi Community Summit landing page

@zchase did this end up going out in a different pr? or did we decide against it?

@susanev Yeah we ended just including it with the PulumiUP stuff.


Update userRegex, fixes #1063

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #1063 )
[ ] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
Added the two unsupported symbols to the regex.
Testing
Required verification:

[x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x] I've verified that this change works as intended.

[x] Downloads all relevant content.
[x] Downloads content from multiple pages (as necessary or appropriate).
[x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.
I didn't, but I noticed the relevant test was still disabled and re-enabled it. I didn't add this change since you closed #243 just recently and I don't know if it's intentional.

Looks good to me


Localization of LogOn, Register, NotFound is impossible

If you have 2 languages in orchard (en-US and another) the url
Url.Action("LogOn", "Account", new {Area = "Orchard.Users"})
always render the LogOn.cshtml shape and it is always in en-US culture.
You can overide it in a theme but if you print
<EMAIL_ADDRESS>the culture is always 'en-US'.
That way THERE IS NO WAY to grab the localized resource strings from a .po file
Major issue. Need a workaround!!!!

@Piedone @BenedekFarkas any ideas?

I assume it's the default site culture, so you can change that from "en-US" to something else if you added other cultures. Have you tried the culture selectors in the Orchard.Localization.CultureSelector?

Yes, it should work with CultureSelector enabled.
If you take a look at Localization module source, you will find ways to select CurrentCulture (from browser, with a cookie or a route param).

@BenedekFarkas you win the race !

Sorry, @TFleury! :)
I'll close this as it's not an issue. Please re-open if localization doesn't work in this cases with a CultureSelector.

Do not close the issue!! Yet!!
You have 1 localized page lets say Home, one for en-US and one for el-GR.
You add the culture picker combo, you select a language and the page automatically changes from en-US and el-GR. Perfect!!!
When you are in the HOME el-GR page, the culture and cookies is already in el-GR!!!!
You have a header link on your site that points to
Url.Action("LogOn", "Account", new {Area = "Orchard.Users"})`
WHEN YOU CLICK on the link although you already in el-GR culture, the site switches culture to en-US and present to you the LogOn.cshtml.
if you print<EMAIL_ADDRESS>inside LogOn.cshtml it is always tthe en-US.
Have you tried it?

You just have to set a cookie :
document.cookie = "OrchardCurrentCulture-FrontEnd=el-GR"

Or customise the login link
Url.Action("LogOn", "Account", new {Area = "Orchard.Users", culture = "el-GR"})

It's not an issue, just a lack of documentation.

So, culture can be set as a route param. Tip of the day! Thanx guys.

Thanks!!!


TODO: add powershell

Leverage what is here: https://github.com/PietJankbal/powershell-wrapper-for-wine
This is necessary to run the AGS installer build script...

I can't figure it out which mingw is the right one to build that c file... Maybe I need an alternative approach...


ShortCodes Showing On page

I really want this schema plugin to work (all the others don't) but for some reason, all of the code is showing up on the page.
Any ideas where the problem lies?

Odd. Can you take a screenshot of what you're seeing? It sounds like the Plugin may be installed, but not active for some reason.

Sure, here's what I put in the back end (it's in the visible area, but no matter - when I put it in text only, same thing):

And here's what comes out on the other end:

Thanks for those. Can you make sure that it looks the same when you switch over to the "Text" tab (upper-right corner of your first screenshot)? I'm curious if there's some formatting that is being added that makes it not behave like a shortcode.
I'm not sure if you are doing any testing locally, but if you happen to be, you could see if the plugin is rendering properly in the default theme to eliminate this as the culprit.
I can see that you have the plugin enabled (it added its button), so that doesn't appear to be the issue. Do you have any other plugins enabled that may be in conflict with this one? That may prevent the short code from being processed.

Sure, here you go:

I'm not sure which of the billion plugins they have could be causing a conflict. I had installed another Schema one, but it didn't work so I got rid of it. The others are diverse and unfortunately, I'm not sure I'm in a position to remove them.

I'm still hoping to resolve this issue - any ideas? The client has many many MANY recipes and this could be very helpful for them, if we can ever get it to work right.

I can't think of anything else to suggest in this case short of debugging it by disabling the other plugins on the site to see if any are in conflict. As best I can tell, you have done everything correctly with the plugin. Something else seems to messing with short code parsing.
Another possibility: Many theme developers will do some strange stuff for various content areas, including disabling filters. The Schema Creator plugin relies on those filters to be in place. The fact that the short code is being left un-touched when published indicates to me that something is out of the ordinary there.
Here is the relevant section of the WordPress code that manages the filters. The add_filter bit is how this plugin "hooks" into the display layer of the theme.

Alright I'll pass this along to our developer and maybe he can help us sleuth out the issue


[PHPStanRules] Warning about (seemingly) correct autoload namespaces

I'm getting loads of errors from the CheckClassNamespaceFollowPsr4Rule-rule in my code.
I'm running a Symfony 5.3 project (developing on Windows/XAMPP in case that matters) with PHP 7.4.20, my autloading-configuration looks like this:
 "autoload": {
    "psr-4": {
      "App\\": "src/"
    }
  },
  "autoload-dev": {
    "psr-4": {
      "App\\Tests\\": "tests/"
    }
  },

So for all that I read this means that the class SomeEntity.php should be in src/Entity/SomeEntity.php and have the namespace "App\Entity" (which is the standard for Symfony anyway), but I keep getting "'#Class like namespace "App\Entity" does not follow PSR-4 configuration in composer.json#'".
I tried to add App\Entity in composer.json, mapping it to "src/Entity" but that doesn't change anything.
What "fixes" the error is changing the namespace in SomeEntity.php to just "App" - but that contradicts my configuration in composer.json - or what am I getting wrong?

Hi,
thanks for reporting. I think you're doing everything right.
My guess the rule does not work correctly with paths on Windows.
It will be somewhere in this class: https://github.com/symplify/symplify/blob/af6f11114e2aa787ff75f8dcc5d72614dd85466e/packages/phpstan-rules/src/Composer/Psr4PathValidator.php#L12-L38

Hello,
I have the same issue, with same configuration and same SF 5.3 namespaces but on Linux (Ubuntu) ...

Hi, how far did you got with debugging? Where does it happen in the rule?

it appears to be the trailing slash in the configured values (src/, tests/) that seems always return only one entry in $splitPaths and automatically fails the validation.
Without the trailing slash, it passes

Wow, thanks for narrowing it down to this one change :+1:
Could you send a PR with ommiting the slash?

(sorry for my late reply, I had a really busy w-e ...)
Yes of course, if no one is already on it

@TomasVotruba here's my proposal to fix this : #3406. I added few tests but I'm not sure if they're at the right place, and was quite confused about them (some cases that might not be valid but actually are ... )

Thank you. Could you downlograde latest dev-main version of the package and see if that helped?
If so, we can tag a new fixed version


Stream Deck Configuration disappear when change focus of application

I configured my Companion Satellite successfully, but I have a problem. I am using my desktop for other applications like audio cutting... every time I change my window focus to another application the buttons on the connected stream deck disappear... When I then click a button the clicked button appear again.
It looks like the view is in the background and with clicking the button again the view come to the front. But I don't know why.
Have some one similar problems and have a comfortable solution?

could some other application be trying to use the streamdeck as well as satellite?

It's a good point, will check it soon. Thank you for your input.

I don't think this is the fault of satellite, and there isnt enough info here to look into anything.
Closing until something more is provided which can be acted upon

Sorry for my late answer...
Your suggestion was the solution of my problem. I have an additional application that try to use the stream deck application.
Problem solved!


Need ability to backup DB

Firebase offers this but only on their $150/month plan.
Automated backups are cooler, but perhaps a button to backup to S3 would suffice for now.
Might be nice to store the last backup date, and nag users if it's been more than 5 days.

There's a REST API to download everything in an importable format, dunno that we would want to expose it via the UI though. Seems like something better suited to an automated process that is entirely separate from crucible.
And yes, their backup solution only being on the expensive plans is lame.

Won't be an option in the UI for this, but there's a couple of options:

Use the firebase-tools npm package that installs the firebase command.
firebase data:get / > firebase.json -f blazing-torch-1111

Use the REST API
https://blazing-torch-1111.firebaseio.com//.json?format=export

Set one of those up a repeating task and should be good to go.


Implemented isort

Breaking these into smaller chunks than the previous attempt -- this one only implements isort. Hopefully, it'll make it easier to review and merge.
Let's see what GitHub Actions has to say about this.

Good job, @smithdc1! :1st_place_medal:


Lag in front end

If you move too fast on board, AI's turns get stock piled up and get laggy

not fixing. <3


[Add] <Button that changes depending on cursor's location>

Details
Creating a location-dependent button that dynamically changes its appearance and behavior based on the position of the cursor.
This button will enhance the user interface and make it more fun to interact with.
Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Please check #1346

Sorry for the inconvinience, but we are closing all the issues! Please open a new issue if you want to contribute anything. Also now you can create a new button from the website itself! You will get points for those too! I will add the instructions on how to create a new button soon and create an empty pr for the same to get points!


Discriminator in request

Hello! I really like the idea of automatically generating tests for our api backend integration.
One thing I stumbled upon are discriminators in request. I found the related issue here https://github.com/apideck-libraries/portman/issues/125 and run in a problem if I do the exact same thing but instead of having the objects in the response I'd like to have it in the request.
The generation of the body seems to just ignore the discriminator and fills it with a random value (not even one that would match one of the discriminated sub-components).
So the api would look like this:
openapi: 3.0.1
info:
  title: DiscriminatorExample
  version: '1.0'
  summary: Help Portman debug contract test issue
servers:
  - url: 'http://test.com'
paths:
  '/children/{childId}':
    get:
      summary: Your GET endpoint
      parameters:
        - in: path
          schema:
            type: integer
            format: int64
          name: childId
          required: true
          description: ''
      tags: []
      requestBody:
        content:
          application/json:
            schema:
              oneOf:
                - $ref: '#/components/schemas/ChildA'
                - $ref: '#/components/schemas/ChildB'
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                oneOf:
                  - $ref: '#/components/schemas/ChildA'
                  - $ref: '#/components/schemas/ChildB'
      operationId: getChild
      description: ''
    parameters: []
components:
  schemas:
    Base:
      title: Base
      type: object
      discriminator:
        propertyName: childType
        mapping:
          ChildA: '#/components/schemas/ChildA'
          ChildB: '#/components/schemas/ChildB'
      properties:
        id:
          type: string
        name:
          type: string
        childType:
          oneOf:
            - enum:
                - ChildA
                - ChildB
          type: string
        createDate:
          type: string
          format: date-time
    ChildA:
      title: ChildA
      allOf:
        - $ref: '#/components/schemas/Base'
        - type: object
          description: ''
          properties:
            childAProperty1:
              type: string
            childAProperty2:
              type: string
      description: ''
    ChildB:
      title: ChildB
      allOf:
        - $ref: '#/components/schemas/Base'
        - type: object
          description: ''
          properties:
            childBProperty1:
              type: string
            childBProperty2:
              type: string
      description: ''

with these cli options:
{
  "local": "openapi.yml",
  "baseUrl": "http://localhost:7071/api",
  "includeTests": true,
  "bundleContractTests": true,
  "syncPostman": true,
  "runNewman": false
}

And the generated body like this:
{
  "id": "Duis magna",
  "name": "do Excepteur ut",
  "childType": "quis",
  "createDate": "1999-10-16T10:15:52.981Z",
  "childAProperty1": "anim est elit id",
  "childAProperty2": "laboris nisi tempor commodo"
}

As you can see the childtype is "quis" which does not make any sense.
Do you know what is wrong with my setup? I have the feeling that the discriminator support is quite limited in the tooling landscape of OpenAPI, swaggerUI seems to just ignore it, but we were able to render it correctly with redoc with this setup:

Thank you in advance!

In the upcoming version of Portman, we have improved the support of "anyOf", "allOf" and "anyOf" but I dont think that is the issue based on your screenshots.

So I ran a quick test and this seems to work:
      allOf:
        - $ref: '#/components/schemas/Base'
        - type: object
          description: ''
          properties:
            childAProperty1:
              type: string
            childAProperty2:
              type: string
            childType:
              type: string
              enum:
                - ChildA

and produces:
{
  "id": "ea Duis",
  "name": "est Excepteur magna",
  "childType": "ChildA",
  "createDate": "1969-08-18T08:25:15.392Z",
  "childAProperty1": "non in enim",
  "childAProperty2": "eiusmod pariatur exercitation deserunt"
}

But in my opinion this defeats the purpose of even using discriminator? Or am I understanding this not correctly?


Nested Navigation has bugs with the routings

I have an app where I have multiple nested Navigation. When used I can seem to get the Get.currentRoute and Get.arguments. My Get.currentRoute always gives the root navigator's route name.
**Reproduction code
I have created a whole new project if you want to try out. As you can see when you go to page 4 it dosnt bring the arguments and the Get.currentRoute still shows 'first_page'. Even though i have provided the settings when creating the Navigator
import 'package:get/get.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      initialRoute: FirstPage.routeName,
      getPages: [
        GetPage(
          name: FirstPage.routeName,
          page: () => FirstPage(),
        ),
        GetPage(
          name: SecondPage.routeName,
          page: () => SecondPage(),
        ),
        GetPage(
          name: ThirdPage.routeName,
          page: () => ThirdPage(),
        ),
        GetPage(
          name: FourthPage.routeName,
          page: () => FourthPage(),
        ),
      ],
    );
  }
}

class FirstPage extends StatefulWidget {
  static const routeName = 'first_page';

  @override
  _FirstPageState createState() => _FirstPageState();
}

class _FirstPageState extends State<FirstPage> {
  int currentIndex = 0;
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      bottomNavigationBar: BottomNavigationBar(
          onTap: (value) {
            setState(() {
              currentIndex = value;
            });
          },
          items: [
            BottomNavigationBarItem(icon: Icon(Icons.home), label: 'Home'),
            BottomNavigationBarItem(icon: Icon(Icons.search), label: 'Search')
          ]),
      appBar: AppBar(
        title: Text('First page'),
      ),
      body: SizedBox(
        width: double.infinity,
        child: IndexedStack(
          index: currentIndex,
          children: [
            Navigator(
                key: Get.nestedKey(0),
                initialRoute: ThirdPage.routeName,
                onGenerateRoute: (settings) {
                  if (settings.name == FourthPage.routeName)
                    return GetPageRoute(
                      settings: settings,
                      page: () => FourthPage(),
                    );
                  else
                    return GetPageRoute(
                      settings: settings,
                      page: () => ThirdPage(),
                    );
                }),
            Navigator(
              key: Get.nestedKey(1),
              initialRoute: SecondPage.routeName,
              onGenerateRoute: (settings) => GetPageRoute(
                settings: settings,
                page: () => SecondPage(),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

class SecondPage extends StatelessWidget {
  static const routeName = 'second_page';
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Second page'),
      ),
      body: Center(
        child: Text('Second Page'),
      ),
    );
  }
}

class ThirdPage extends StatelessWidget {
  static const routeName = 'third_page';
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: SizedBox(
        width: double.infinity,
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: [
            Center(
              child: Text('Third Page'),
            ),
            RaisedButton(
              onPressed: () => Get.toNamed(FourthPage.routeName,
                  id: 0, arguments: 'Hi from page 3'),
              child: Text('Go to fourth screen'),
            )
          ],
        ),
      ),
    );
  }
}

class FourthPage extends StatelessWidget {
  static const routeName = 'fourth_page';
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(Get.currentRoute),
      ),
      body: Center(
        child: Column(
          children: [
            Text('Fourth Page '),
            Text('arguments:' + Get.arguments.toString()),
          ],
        ),
      ),
    );
  }
}

Flutter Version:
1.22.5
Getx Version:
^3.24.0
Describe on which device you found the bug:
Samsung Galaxy A8 Star - Android

updated: I added observers: [GetObserver((_) {}, Get.routing)],  to the navigator and it seems to be working. However the screen name is now the class name runType and it gives error about a global key duplicating.

updated: I added observers: [GetObserver((_) {}, Get.routing)],  to the navigator and it seems to be working. However the screen name is now the class name runType and it gives error about a global key duplicating.

updated: I added observers: [GetObserver((_) {}, Get.routing)], to the navigator and it seems to be working. However the screen name is now the class name runType and it gives error about a global key duplicating.

Could you provide the full code? When i add this it just empties Get.arguments.

take a look at my solution to nested navigation using getx
https://github.com/tva77/getx_persistent_bottombar


sort field names in logger before output

{"level":"warn","@timestamp":"2023-02-25T12:51:25.834+0300","caller":"kgo/logger.go:123","msg":"unable to commit offset for topic partition","group":"TestAssign","topic":"mytopic","partition":35,"commit_from":120957}
{"level":"warn","@timestamp":"2023-02-25T12:51:25.834+0300","caller":"kgo/logger.go:123","msg":"unable to commit offset for topic partition","partition":42,"commit_from":119818,"group":"TestAssign","topic":"mytopic"}
not so serious , but if that possible best to sort field names

The logger implementation can do that -- far too complicated to manage at callsites.


Update generateDocument.ts

Return an empty array when the field is NOT REQUIRED. blockValues can be null (Literally not set) when it is not required so you should check if it is NOT required.
Otherwise... You will get this bug in production when the collection has a NON REQUIRED blocks field, but no blocks.

Could you have a look @dkirchhof?

Could you have a look @dkirchhof?

I hope when the bug gets fixed... Whether because of my commit or not, there is a hotfix released as a patch release

Hey @TomDo1234,
thanks for your PR. I think we have to handle both cases, right?
If it is required, we need an empty array as fallback, if it isn't, we can just return undefined.
if (field.required && !blockValues) {
    return [];
}

if (!field.required && !blockValues) {
    return undefined;
}

Would you like to test it and update your PR?

Hey @TomDo1234,
thanks for your PR. I think we have to handle both cases, right? If it is required, we need an empty array as fallback, if it isn't, we can just return undefined.
if (field.required && !blockValues) {
    return [];
}

if (!field.required && !blockValues) {
    return undefined;
}

Would you like to test it and update your PR?

I have made the changes, can YOU or @jpeiniger  test this and then merge?

Thanks again. Just released the patched version on npm :)


Include journal policy as motivation for sharing code

I think another motivation to publish code is that in response to research council pressure, journals are requiring that data and code is made available on request:
From Science editorial policy:

"We require that all computer code used for modeling and/or data analysis that is not commercially available be deposited in a publicly accessible repository upon publication. In rare exceptional cases where security concerns or competing commercial interests pose a conflict, code-sharing arrangements that still facilitate reproduction of the work should be discussed with your Editor no later than the revision stage."

From Nature editorial policy:

"An inherent principle of publication is that others should be able to replicate and build upon the authors' published claims. A condition of publication in a Nature Research journal is that authors are required to make materials, data, code, and associated protocols promptly available to readers without undue qualifications. Any restrictions on the availability of materials or information must be disclosed to the editors at the time of submission. Any restrictions must also be disclosed in the submitted manuscript."

However a study showed that despite these policies, many people still do not share their code 😞 : https://www.pnas.org/content/115/11/2584. This paper includes samples of charming author responses such as

"When you approach a PI for the source codes and raw data, you better explain who you are, whom you work for, why you need the data and what you are going to do with it."

Perhaps this could be included as another motivation for sharing code, although the presentation is already rather long as it stands.

Thank you for sharing


I/O metrics?

ctop is awesome!  Great tool.  It already tracks CPU, memory, and network usage.  But I'd also find it very useful to see metrics related to disk I/O, both inside and outside the container.  Would such a feature be possible?

I think something similar to my ports pull request could be done... I might get on it :)

Bytes read/sec and bytes written/sec.  Although metrics from within the union fs would be useful, I'd find metrics regarding reads and writes that traverse the container-host barrier to be the most helpful.

Aren't those already in the expanded view?

Believe it or not, I've never looked at the expanded view. I'm going to
have to check that out tomorrow to be sure.
On May 15, 2017 12:00 AM, "kenan-rhoton"<EMAIL_ADDRESS>wrote:

Aren't those already in the expanded view?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bcicen/ctop/issues/74#issuecomment-301376653, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKeZiV7KoJhCqsotgs8UTDPxInuBWKvYks5r59vogaJpZM4M_Q2F
.

Block I/O metrics were added in v0.5, both in the compact view and the expanded view.

Feel free to close this issue.
On May 24, 2017 2:23 AM, "bradley"<EMAIL_ADDRESS>wrote:

Block I/O metrics were added in v0.5
https://github.com/bcicen/ctop/releases/tag/v0.5, both in the compact
view and the expanded view.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bcicen/ctop/issues/74#issuecomment-303640257, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKeZid3tzV4Lh7FIP1CDEkhh1jacfRDRks5r89rYgaJpZM4M_Q2F
.


Progress bar for cell detection

As part of improving user feedback in the GUI #12 , we should show updates on progress.
An initial improvement (also allowing to familiarise ourselves with making a progress bar) would be to display a progress bar during cell detection. Estimation of progress should be easy, as this is ~sequential through all planes in the image, so we can estimate from total number of planes, and number of planes processed.
This is already displayed in the logs/stdout so we can see how it's done there.

The napari.utils.progress API docs look handy for this.
It's not immediately obvious to me whether it's possible to get the number of planes already progress from cellfinder-core into cellfinder-napari though 🤔

I think the first step here is to add options in the relevant cellfinder-core functions/methods to make tqdm progress bars optional (ie. add verbose arguments that can be set to False), and also allow custom callbacks to be passed so we can use these to trigger a napari progress bar.


numpy error when running CLI

Overview
I'm trying to just to run the CLI with either no commands, or with generate-all, and I'm getting this same error:
AttributeError: module 'numpy' has no attribute 'NAN'. Did you mean: 'nan'?
Stacktrace

Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/Users/joeflack4/projects/comploinc/src/comp_loinc/__init__.py", line 4, in <module>
    from comp_loinc.main import app
  File "/Users/joeflack4/projects/comploinc/src/comp_loinc/main.py", line 33, in <module>
    from comp_loinc.mapping.fhir_concept_map_ingest import ChebiFhirIngest
  File "/Users/joeflack4/projects/comploinc/src/comp_loinc/mapping/fhir_concept_map_ingest.py", line 6, in <module>
    from sssom.io import convert_file
  File "/Users/joeflack4/virtualenvs/comp-loinc/lib/python3.10/site-packages/sssom/__init__.py", line 12, in <module>
    from sssom.util import (  # noqa:401
  File "/Users/joeflack4/virtualenvs/comp-loinc/lib/python3.10/site-packages/sssom/util.py", line 510, in <module>
    def add_default_confidence(df: pd.DataFrame, confidence: float = np.NAN) -> pd.DataFrame:
  File "/Users/joeflack4/virtualenvs/comp-loinc/lib/python3.10/site-packages/numpy/__init__.py", line 410, in __getattr__
    raise AttributeError("module {!r} has no attribute "
AttributeError: module 'numpy' has no attribute 'NAN'. Did you mean: 'nan'?

What I tried
I've installed my setup using poetry install.

As expected / hoped, this error has gone away in Shahim's new refactor.


add 'Tickle' example under new 'Interaction' category

Hi all. I tried my hand at porting over the 'Tickle' demo from here. I hope it's satisfactory and that I did everything correctly. I tested it out with a local apache and all seemed well.

looks great, thanks!

Yay! Thanks @lmccart
I'll probably tackle a few more soon.


Chapter1/homework/6

Part 2 took me forever to realise that the password was actually being spit into the console 😂 I was digging through the debugger console trying to find the secret password. But I think I got it all now.
Part 1 -> "Debugger!", "51", "79", "77"
Part 2 -> "A minore ad maius a solis ortu usque ad occasum ab uno disce omnes"
-> "smaller, sunrise, sunset"

Well done! :)


Add HTTP header each query/mutation

such as @ Header annotation on Retrofit

You can achieve this with okhttp interceptor. Any reason you need the feature to also be in apollo?  https://futurestud.io/tutorials/retrofit-2-manage-request-headers-in-okhttp-interceptor

I wanna use with DI containner such as Dagger2. (´・ω・｀)

I'm not sure what you mean. Dagger has nothing to do with headers. Closing as this is not a bug report nor do I believe any work needs to be done


[Bug] Add typing_extensions requirement again.

This python dependency missing problem has been reported in https://github.com/apache/tvm/pull/14244, the same code was merged before, but strangely, the relevant changes were reverted in the latest commit. @areusch
But it can be confirmed that typing_extensions is still needed by tvmc at least: source code of tvm.meta_shedule.builder.builder
It is worth mentioning that this problem affects the tvm source code download page, at least version 0.12.0. In fact, I encountered the ModuleNotFoundError error when installing tvm from source code and trying to call tvmc. You can reproduce this error with the following command:
# download, compile and install.
wget https://dlcdn.apache.org/tvm/tvm-v0.12.0/apache-tvm-src-v0.12.0.tar.gz \
    && tar -zxvf apache-tvm-src-v0.12.0.tar.gz \
    && mkdir -p apache-tvm-src-v0.12.0/build \
    && cd apache-tvm-src-v0.12.0/build/ \
    && cp ../cmake/config.cmake ./ \
    && sed -i 's/set(USE_LLVM OFF)/set(USE_LLVM ON)/g' config.cmake \
    && wget https://apt.llvm.org/llvm.sh \
    && sed -i 's/add-apt-repository "\${REPO_NAME}"/add-apt-repository "\${REPO_NAME}" -y/g' llvm.sh \
    && cmake .. \
    && make -j$(nproc) \
    && python ../python/setup.py install
# try calling tvmc
tvmc

@leandron @driazati @junrushao

The test failed because of below check.
[2023-05-26T04:00:25.396Z] + python3 ci/scripts/jenkins/check_pr.py --pr 14960
[2023-05-26T04:00:25.396Z] [ci/scripts/jenkins/git_utils.py:192 INFO] Running ['git', 'config', '--get', 'remote.origin.url']
[2023-05-26T04:00:25.396Z] [ci/scripts/jenkins/git_utils.py:150 INFO] Requesting GET to https://api.github.com/repos/apache/tvm/pulls/14960
[2023-05-26T04:00:25.654Z] Running checks for PR title
[2023-05-26T04:00:25.654Z]     [Bug] Add typing_extensions requirement again.
[2023-05-26T04:00:25.654Z]     Checks:
[2023-05-26T04:00:25.654Z]         [1] non_empty: PASSED
[2023-05-26T04:00:25.654Z]         [2] trailing_period: FAILED: PR must not end in a tailing '.'

The test failed because of below check.
[2023-05-26T04:00:25.396Z] + python3 ci/scripts/jenkins/check_pr.py --pr 14960
[2023-05-26T04:00:25.396Z] [ci/scripts/jenkins/git_utils.py:192 INFO] Running ['git', 'config', '--get', 'remote.origin.url']
[2023-05-26T04:00:25.396Z] [ci/scripts/jenkins/git_utils.py:150 INFO] Requesting GET to https://api.github.com/repos/apache/tvm/pulls/14960
[2023-05-26T04:00:25.654Z] Running checks for PR title
[2023-05-26T04:00:25.654Z]     [Bug] Add typing_extensions requirement again.
[2023-05-26T04:00:25.654Z]     Checks:
[2023-05-26T04:00:25.654Z]         [1] non_empty: PASSED
[2023-05-26T04:00:25.654Z]         [2] trailing_period: FAILED: PR must not end in a tailing '.'

Sorry for missing that, now fixed.

@tvm-bot rerun

@junrushao It seems the failed tests have nothing to do with this PR, can we merge this in? maybe the failed tests will gone after rebase.

I don’t have permission to merge a PR without CI being green..retriggered the pipeline but it still fails

Hmm it still fails

The error indicate that the PR title(as in the commit message) should not end with a trailing period
So changing the PR message(by rebasing it) should work

@tqchen The specific failure at the moment is:
[2023-06-25T09:22:56.305Z] In file included from ../3rdparty/cutlass_fpA_intB_gemm/cutlass_kernels/fpA_intB_gemm/fpA_intB_gemm_fp16_int8.cu:17:
[2023-06-25T09:22:56.305Z] ../3rdparty/cutlass_fpA_intB_gemm/cutlass_kernels/fpA_intB_gemm/fpA_intB_gemm_template.h:20:10: fatal error: cutlass/gemm/device/gemm_universal_base.h: No such file or directory
[2023-06-25T09:22:56.305Z]    20 | #include "cutlass/gemm/device/gemm_universal_base.h"
[2023-06-25T09:22:56.305Z]       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[2023-06-25T09:22:56.305Z] compilation terminated.

@lpdink I think your PR haven't to do with the error of "cutlass_fpA_intB_gemm", can you please try to rebase your commit onto the latest changes of the main branch and try it again?

@Johnson9009 Done. Could you please re-run again?


[CI] Add CodeQL analysis

Part of #21. This is the stock GitHub template, generated by their onboarding wizard.
Mirror PR here to see it in use.

LGTM squash and merge when ready


1.20.2 ver?

🥺
for Fabric 🥺

Unfortunately, since this is an older version of Minecraft, this is not currently planned.


How to use this?

Sorry this isn't an issue but either the documentation isn't great or i'm completely confused.
Does this package allow multiple items to be stored into the database or do we have to create a model for each item?
Like:
T-shirt
Pants
Are they a model each or are they stored in a 'items' database?

The products table (which seems like more of what you need) is apart from this package. This only covers cart and checkout.


Feature/pipline infra setup

PR based on Penny's PR.
Not able to run because of postgres initialization problems.
Almost the same as the sample project.
Created a pipeline to test but fails on testing.

Finally successfully deployed!

Trigger url: https://cd-pipeline-wen-wcfy-dev.apps.sandbox-m4.g2pi.p1.openshiftapps.com
Deployed website url: https://promotions-wen-wcfy-dev.apps.sandbox-m4.g2pi.p1.openshiftapps.com

Changes: .tekton & k8s

Done behave test task.

Changed the browser to chrome.
Added compatible logics for converting - format date to / format date for chrome in web_steps.


⚠ Bofrost.nl has degraded performance

In c8806f1, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1189 ms

Resolved: Bofrost.nl performance has improved in d7646fb.


[CMAKE] Cmake changes, upgrade training test so it converge

Redo #8311
The reason failure happens after its mergeis due to the problem of test-case giving wrong parameters. I have fixed that so that now it actually converges to meanful value.

cc @piiswrong

failure in the tests appears due to the missing ci machine. need retrigger when they are up

Triggered a rebuild attempt due to CI problems

cc @cjolivier01 @piiswrong I just squash merged this


⚠ Spryr has degraded performance

In 4e1a764, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5659 ms

Resolved: Spryr performance has improved in e51bc6c after 12 minutes.


Could you help to remove the vulnerabilities?

Hi,@maugenst
Issue
9 vulnerabilities (5 high and 4 medium severity) are introduced in tabletojson:
1.Vulnerability CVE-2021-23337 (high severity) is detected in package lodash (versions: <4.17.21): https://snyk.io/vuln/SNYK-JS-LODASHTEMPLATE-1088054
2.Vulnerability CVE-2020-28500 (medium severity) is detected in package lodash (versions: <4.17.21): https://snyk.io/vuln/SNYK-JS-LODASH-1018905
3.Vulnerability CVE-2020-8203 (medium severity) is detected in package lodash (versions: <4.17.16): https://snyk.io/vuln/SNYK-JS-LODASHTEMPLATE-1088054
4.Vulnerability CVE-2019-10744 (high severity) is detected in package lodash (versions: <4.17.12): https://snyk.io/vuln/SNYK-JS-LODASH-450202
5.Vulnerability CVE-2018-16487 (high severity) is detected in package lodash (versions: <4.17.11): https://snyk.io/vuln/SNYK-JS-LODASHMERGEWITH-174136
6.Vulnerability CVE-2018-3721 (medium severity) is detected in package lodash (versions: <4.17.5): https://snyk.io/vuln/SNYK-JS-LODASHMERGEWITH-174137
7.Vulnerability CVE-2019-1010266 (medium severity) is detected in package lodash (versions: <4.17.11): https://snyk.io/vuln/SNYK-JS-LODASH-73639
8.Vulnerability SNYK-JS-LODASH-608086 (high severity) is detected in package lodash (versions: <4.17.17): https://snyk.io/vuln/SNYK-JS-LODASH-608086
9.Vulnerability SNYK-JS-LODASH-590103 (high severity) is detected in package lodash (versions: <4.17.20): https://snyk.io/vuln/SNYK-JS-LODASH-590103
The above vulnerable package is referenced by tabletojson via:
<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>Solution
Since<EMAIL_ADDRESS>is transitively referenced by 12 downstream projects (e.g., vitauth 2.0.0 (latest version), coinmarketcap-cli-api 1.1.0 (latest version), ncaa-stats 2.0.0  (latest version),  dsbiserv 1.3.2 (latest version), cekresi-cli 1.0.9  (latest version))
If<EMAIL_ADDRESS>removes the vulnerable package from the above version, then its fixed version can help downstream users decrease their pain.
Could you help update packages in this version?
Fixing suggestions
In<EMAIL_ADDRESS>you can kindly perform the following upgrades (not crossing their major versions):
cheerio 0.19.0 ➔ 0.20.0;
Note:
cheerio 0.20.0 directly depends on<EMAIL_ADDRESS>(a vulnerability patched version)
Thanks for your contributions to the npm ecosystem!
Best regards,
Paimon

Hi Paimon.
Thanks for this. I updated all libs to the latest version.
BR, Marius

@maugenst Thanks for your help!


Cannot request reviewers

Created a pull request https://github.com/prestodb/rfcs/pull/16 but cannot request reviewers. Seems like a permissions issue but not sure what to do.

I'll add you as a contributor to Presto, which will give you these permissions.


M44d/f missing rotation methods in PyIlmBase

I noticed that the M44d and M44f in PyIlmBase are missing the setEulerAngles and setAxisAngle methods that are in the IlmBase matrix classes in the master branch. Is this intentional, or should I try my hand at setting up a pull request?

Hey @scott-wilson i want to work on this issue kindly help

Honestly, I only have the vaguest idea of where to start. You'll probably want to ask someone with a few more commits under their belt.

This requires dealing with boost python, which can be challenging. The best way to approach it is to mimic what's already there. Look at the implementation of a method such as setScale, in PyImathMatrix33.cpp. Copy the various bits of code that implement setScale to implement setEulerAngles. Obviously, the guts of the function will be different, but the overall pattern of the wrapping will be similar. setAxisAngle will be a bit trickier since it takes 2 arguments, a scale and an tuple.

@scott-wilson @cary-ilm Hey, if this has not been resolved yet, can I take on this issue?

I haven't touched it. From my opinion, it's all yours

Fine with me if you'd like to take a stab at it, although as I mention, it requires dealing with boost python, which can be confusing. The best way to approach it is to copy and paste from similar routines.


Support finding version info in TOML files (#44)

This patch adds support for extracting version information from TOML files. For example, projects using Poetry can specify the following version literal in their pyproject.toml file:
[tool.scriv]
version = "literal: pyproject.toml: tool.poetry.version"

# PEP 621 example:
# version = "literal: pyproject.toml: project.version"

Then project metadata can be managed using Poetry and doesn't have to be extracted and passed to scriv separately:
poetry version minor
scriv collect

Changelog
Added

Support finding version information in TOML files (like pyproject.toml)
using the literal configuration directive.

@nedbat It looks like PyPy 3.6 is not available on Mac OS in the CI. All other tests pass.

Hi @nedbat! I hope you're doing well. Is there anything I need to do to improve this PR?

@kurtmckee sorry for the lag, yes, this looks good to go, though we clearly disagree about the way to import product code into test files :)

I see why you changed the import :)

Ned, thanks for merging this!

This is released as 0.13.0

Fantastic. Thank you!


[BUG] LSP Sees completely different input than what's being shown in the editor.

Can't programmatically recreate yet. Usually is triggered by an autocompletion, or editing something right after autocompleting it. Error sustains even after writing more and saving the file in the filesystem.

OS: Windows 11
LiteXL Version: Latest release when posted.
LSP Version: Master when posted.

Potentially fixed by dee1bf03880dddcb82b1a35645479688686ca282.


Plugin not working on large strings

I have this model
'use strict';

import mongoose from 'mongoose';
import version from 'mongoose-version';
import AutoIncrement from 'mongoose-sequence';
const sequence = AutoIncrement(mongoose);
import {
  registerEvents
} from './solicitud.events';
var SolicitudSchema = new mongoose.Schema({
  antecedentes: {
    type: String,
    default: ''
  },
  usos: {
    type: String,
    default: ''
  },
  alternativa: {
    type: String,
    default: ''
  },
  justificacion: {
    type: String,
    default: ''
  },
  __machina__: {
    type: {},
    default: {}
  },
  numero: Number
});

registerEvents(SolicitudSchema);
SolicitudSchema.plugin(version, {
  mongoose
});
SolicitudSchema.plugin(sequence, {
  inc_field: 'numero'
});

export default mongoose.model('Solicitud', SolicitudSchema);

When any of the String fields is more than 1500 characters on save() no new version is saved, also versioning stops working for that document no mater if you change other field
Sorry for the bad English

@Djalmar What is your MongoDB setting? Did you exceed the maximum JSON object size?

I run into similar problem.
I enabled debugging and I see that when saving, am getting an insert error:
MongoError: Btree::insert: key too large to index, failing

You can further see in the error (full error below) the index that causes the insert error (CM_DATA.datafills_versions.$site_1)
I checked the collection and it is created with an index on all the field in the versioned schema (I am not sure where it happens though as I don't see it in the code).
When I removed the specified index, consequent saves do insert into the versioning collection.
I am wondering what for there is an index on all the fields and whether it is really necessary... Currently, I don't see any change tracking or anything like that which requires to do queries (searches) on all fields.
If indeed indexing on all (most) of the fields is necessary, maybe it is possible to exclude some fields through the options...
mongoose:version MongoError: Btree::insert: key too large to index, failing CM_DATA.datafills_versions.$site_1 1148 { : LONG STRING VALUE HERE }
mongoose:version     at Function.MongoError.create (D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb-core\lib\error.js:31:11)
mongoose:version     at toError (D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\utils.js:114:22)
mongoose:version     at D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\collection.js:658:23
mongoose:version     at handleCallback (D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\utils.js:95:56)
mongoose:version     at D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\bulk\unordered.js:465:9
mongoose:version     at handleCallback (D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\utils.js:95:56)
mongoose:version     at resultHandler (D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb\lib\bulk\unordered.js:413:5)
mongoose:version     at D:\Documents\Work\Cellcom\CellXM\src\CellXM\node_modules\mongoose\node_modules\mongodb-core\lib\connection\pool.js:455:18
mongoose:version     at _combinedTickCallback (internal/process/next_tick.js:73:7)
mongoose:version     at process._tickCallback (internal/process/next_tick.js:104:9) +0ms


Screen is not darkened when Popup is displayed

I'm migrating from Xamarin Rg.Popups to Maui Mopups.
I would expect that when I call await MopupService.Instance.PushAsync(myPopupPage), the background outside the Popup darkens automatically with a semi-transparent overlay, like in the example gifs.  However this doesn't happen. The Popup displays, but the background remains at normal brightness.
My PopupPage isn't doing anything special. In fact the issue repros even on an empty page.
Android API 33
Maui 8.0.7
Mopups 1.3.0

aren't you supposed to set your own background color?

Yep setting BackgroundColor="#88000000" worked. I wasn't aware of this requirement, as it wasn't necessary in Rg.Popups.  I wish this were mentioned in the documentation somewhere...


support sql with sub query projection

New PR, ref #4696

Codecov Report

Merging #4845 into master will decrease coverage by 0.16%.
The diff coverage is 0.00%.

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

Pull Request Test Coverage Report for Build 10396

0 of 10 (0.0%)  changed or added relevant lines in 5 files are covered.
43 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.1%) to 59.636%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/impl/MySQLDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/impl/OracleDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/postgresql/visitor/impl/PostgreSQLDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/sql92/visitor/impl/SQL92DMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/sqlserver/visitor/impl/SQLServerDMLVisitor.java
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

shardingsphere-database-protocol/shardingsphere-database-protocol-mysql/src/main/java/org/apache/shardingsphere/database/protocol/mysql/constant/MySQLBinlogEventType.java
1
97.56%

shardingsphere-database-protocol/shardingsphere-database-protocol-mysql/src/main/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/time/MySQLTimeValueUtil.java
1
66.67%

shardingsphere-database-protocol/shardingsphere-database-protocol-mysql/src/main/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/management/MySQLBinlogFormatDescriptionEventPacket.java
2
90.0%

sharding-orchestration/sharding-orchestration-core/sharding-orchestration-core-registrycenter/src/main/java/org/apache/shardingsphere/orchestration/core/registrycenter/util/IpUtils.java
3
76.0%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/codec/MySQLBinlogEventPacketDecoder.java
5
92.41%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/packet/binlog/RowsEventPacket.java
12
81.05%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/MySQLConnector.java
19
67.9%

Totals

Change from base Build 1116:
-0.1%

Covered Lines:
12464

Relevant Lines:
20900

💛  - Coveralls


New version: MCMCChains v0.3.13

Registering package: MCMCChains
Repository: https://github.com/TuringLang/MCMCChains.jl
Created by: @cpfiffer
Version: v0.3.13
Commit: 300bf5b6a3a13ec99b4cfde41d6ac4c0482ee2a5
Reviewed by: @cpfiffer
Reference: https://github.com/TuringLang/MCMCChains.jl/issues/129

Please add (at least)
[compat]
julia = "1"

to your project file and retrigger the bot.

Updated, thanks.


.Net: SK Plan generation exception - invalid attribute character

Describe the bug
I don't get this reliably, but when running a plan multiple times (different inputs), I have occasionally run into an XML parse error: invalid attribute character
To Reproduce
Steps to reproduce the behavior:

Create a Sequential plan
Run the plan w/ some inputs
Continue running until error

Expected behavior
Based on what I've read of past issues, plans should be done in JSON now? Not sure why we're creating XML, then.
Screenshots

Platform

OS: Windows
IDE: Visual Studio
Language: C#
Source: NuGet package version 1.0.0-beta6

Additional context
My plan has Text memory and directory-loaded functions (ImportFunctionsFromDirectory)

Thank you for reporting this issue. Because of the number of issues we've run into with XML generation by the sequential planner, we've built a new planner that avoids these problems. You can see the new Handlebars planner here.
Because we've introduced this new planner, we plan to deprecate the current sequential planner (because of challenges like the one you faced). Can you try the new Handlebars planner and see if you get similar/different errors? Please create a new issue if you do.


Improved KAS key management tools

We currently don't support KAS key management very well

Introduce service keys subcommand to otdfctl:
a. init: similar to current init-temp-keys script, this creates new keys
b. create: Adds new key or keys of requested type
c. import: Adds existing keys of the requested type
These keys update the opentdf.yaml file with the new key information.

Things to think about:

the keys should be stored in jwk sets
all keys must be compatible with existing clients
should be an opentdf subcommand set, to simplify deployment
must work on windows

Subtasks:

[ ] #1085
[ ] #894
[ ] #778
[ ] #769
[ ] #616

this is currently blocked by the key management adr. moving to BLOCKED until that is complete.


Backup not working following the guides - part2

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
I believe the issue is related to this one https://github.com/cvat-ai/cvat/issues/7983 but not the same.
I need to update my local CVAT setup and wanted to back up everything. Checked the doc: https://docs.cvat.ai/docs/administration/advanced/backup_guide/. Trying to run:
cd /mnt/cvat
docker compose -f docker-compose.yml -f components/serverless/docker-compose.serverless.yml down
(... success...)
mkdir backup
docker run --rm --name temp_backup --volumes-from cvat_db -v $(pwd)/backup:/backup ubuntu tar -czvf /backup/cvat_db.tar.gz /var/lib/postgresql/datadocker run --rm --name temp_backup --volumes-from cvat_db -v $(pwd)/backup:/backup ubuntu tar -czvf /backup/cvat_db.tar.gz /var/lib/postgresql/data
docker: Error response from daemon: No such container: cvat_db.
See 'docker run --help'.

Could you help, please?
Expected Behavior
Backup starts
Context
Only nuclio containers running:
 docker ps -a
CONTAINER ID   IMAGE                                  COMMAND                  CREATED        STATUS                  PORTS                                         NAMES
2b0380daf548   gcr.io/iguazio/alpine:3.17             "/bin/sh -c '/bin/sl…"   6 hours ago    Up 6 hours                                                            nuclio-local-storage-reader
493da0b639ff   cvat.pth.dschoerk.transt:latest        "processor"              6 hours ago    Up 6 hours (healthy)    <IP_ADDRESS>:32772->8080/tcp, :::32772->8080/tcp   nuclio-nuclio-pth-dschoerk-transt
416fc2cb1820   cvat.dr_0822_1610_batch16:latest   "processor"              23 hours ago   Up 23 hours (healthy)   <IP_ADDRESS>:32770->8080/tcp, :::32770->8080/tcp   nuclio-nuclio-dr_0822_1610_batch16

Indeed, ther's no cvat_db container because it was stopped. It is present when CVAT is running:
docker ps -a | grep cvat_db
42fe7c2abdbc   postgres:15-alpine  "docker-entrypoint.s…"   20 hours ago   Up 20 hours  cvat_db

Environment
Ubuntu 18.04.6
Server version: 2.15.0
Core version: 15.0.6
Canvas version: 2.20.3
UI version: 1.63.11
Docker version 24.0.2, build cb74dfc

docker compose down stops and removes container.
This is why you get error cvat_db does not exist. The guide suggest using docker compose stop.

My bad, thank you very much.


fix: use pty aware charmbracelet/ssh

Based on https://github.com/charmbracelet/ssh/pull/8

merged manually in https://github.com/charmbracelet/wish/pull/197/commits/50a706ee42522a1f77fd30421a1f152f9b721213


proof of 46 implies 40

(had a proof_wanted)

Hey, I'll approve this PR, but do please follow the PR claim process mentioned in the Outstanding tasks v2 thread. We don't want people to end up working in parallel and find their work not being used.

thanks! yes, sorry, I didn't see any task for this ine and it was small, but for anything meaningful I will do in the future!


typescript support (for layout package)

@venegasalejandro Hi, thanks a lot for your contribution. I have to say I am not familiar with typescript. What is your suggestion?

@siriwatknp do you consider changing (part of) mui-treasury codebase to compiled with TypeScript? This way TypeScript users get always-updated typings for free, and JavaScript users can use the library normally.
It can be done gradually, starting from, I agree with you and @venegasalejandro , the layout package.

I think we should follow Material-UI's example: they keep all their source code in Javascript, but make sure to update Typescript definitions before releases, and of course all the Typescript definitions are packaged within their several npm packages.

Material-UI has a quite large codebase, it will takes lots of time do migration, they provide typescript definitions is a compromise solution,  since this project is just started, I strongly suggest  writing it with Typescript. the learning curve for Typescript is not  that steep.

@siriwatknp Hey thanks for doing the merge! I've been sort of busy at work and haven't been able to check in. Is there still interest in converting the package to be completely Typescript (rather than having the declarations separate)? If so, I would love to contribute once I have some free time.

@venegasalejandro That's awesome. However, I think the current version has some downside.

It does not work well with server side frame work. ex. NextJS, Gatsby
It use too much javascript instead of css for determining width &margin, therefore performance is bad (from what I measure with lighthouse)
I am thinking about rewriting the lib to move toward css approach but will remain the same api as much as possible, what do you think?

Following the tutorial with TypeScript doesn't work. The module can't be found, and I don't think the current layout package contains the .d.ts files. Anybody know what's going on?

Following the tutorial with TypeScript doesn't work. The module can't be found, and I don't think the current layout package contains the .d.ts files. Anybody know what's going on?

See #624


Problem checking if a point is inside a Shape

Hi, I'm a newbie in using spatial4j. I created a Shape that refers to Spagna square in Rome:
POLYGON ((12.481795102357864 41.90594219388104, 12.48123 41.906501, 12.481986 41.90666, 12.48234 41.90674, 12.482990026473999 41.906938296574225, 12.483067810535431 41.90682251790111, 12.48347282409668 41.906341435303666, 12.483869791030884 41.90585236421689, 12.482727 41.905173, 12.482585 41.905151, 12.482359 41.905383, 12.481795102357864 41.90594219388104))
I need to check if a given point is contained or not by this shape. So I use the code:
final static SpatialContext ctx = SpatialContext.GEO;
PointImpl p = new PointImpl(12.48402, 41.905759, ctx);
if (container.relate(inner) == SpatialRelation.CONTAINS) return true;
else return false;
I get "false", but the correct result should be "true".
Can somebody help me?
Thanks in advance.

Hi.
I believe the correct answer is false.  Notice that 12.484 is greater than all the 'X' values in the point list.  Why do you think it should be true?
P.S. consider this tool to experiment: http://arthur-e.github.io/Wicket/sandbox-gmaps3.html


Burger tabs & content

I know this is still under dev, but sharing here some suggestions:
Tabs:

About : This can contain the content of both What?, Why? and What's next? in GH
People: Same as in GH
Developers: Include the link to GH and also the 'Legal Disclaimer' section about the licence
Disclaimer: Here we need to write one about UCL/WCL responsiblity on the maps generated with Kapta. We need to speak with experts to write this. Let's leave it empty for now, but basically we need something to cover our backs on how people use Kapta.

have added the what's next to the rest of the content. Do you actually want a developers section? we could just have a linked GH icon and the legal disclaimer links as menu items rather than in a dropdown since they're very short

okay. Let's do that


Support preview Update Branch API

GitHub Developer API announcement:
https://developer.github.com/changes/2019-05-29-update-branch-api/
This would be a great PR for any new contributor to this repo or a new Go developer.
All contributions are greatly appreciated!
Feel free to volunteer for any issue and the issue can be assigned to you so that others don't attempt to duplicate the work.
Please check out our CONTRIBUTING.md guide to get started.
Thank you!

@gmlewis I'd like to work on this issue

Thank you, @waseem18!
It is yours.


[Bug]: Dark Mode Button

Give a brief about the bug ✍

Dark mode button is not fixed
At the middle of the screen user not able to change the theme

What is the expected behavior? 🤔

It should be fixed so that it would be easy for user to change theme when they are at the middle of the screen

Provide step by step information reproduce the bug 📄

I will made it fixed at there position using correct CSS property

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @parasss19 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

hey @kunjgit can you assign me this issue

/assign

Hey @parasss19 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


spin_echoes.ipynb references a removed optimizer

Description of the issue
docs/tutorials/google/spin_echoes.ipynb references cirq.MergeInteractionsToSqrtIswap which was removed in #5727.
Solution
Replace with a working transformer in the notebook, make sure this is rendered on cirq webpage.
Also search for and fix other invalid cirq.Something references in the repository.
Cirq version
1.5.0.dev at 2991e808bf62bb71da3fba9f1e83c79a01f57f17

Hello,
I'm interested in working on this issue if it's still available.
I've already started playing around with the notebook, cleaning it up and regenerating the output by replacing the old cirq.MergeInteractionsToSqrtIswap references with the cirq.optimize_for_target_gateset function transforming the operations to use the cirq.SqrtIswapTargetGateset.
I've been running the notebook locally using a Weber QVM, but I've not been able to notice a negligible difference in the final diagram survival probability between the optimized and unoptimized circuits. I'm not sure if running the tutorial on real Weber hardware would result in a more significant difference.
Any guidance would be really helpful. Thanks


More properties and patch for strand rendering

Let me know if you have questions.

Nice. FYI anecdotally spacefill is fastest with ris I've heard ;)


Create Environnement to work on redis cache

First version with redis support
Used the most popular library which in fact is not compatible: (unsafe dependancy)
will try with another: https://github.com/gomodule/redigo


Adding new PlanifNeige platform/sensor

Description:
This is a pull request for a new platform and sensor for Montreal's street-by-street snow removal APIs.
The component can be used to monitor snow removal operations on nearby streets and notify the user to move his car to avoid getting it towed, and/or receiving a ticket.
The dataset/API used for this component is from the City of Montreal's open data initiative (website in French).
Pull request in home-assistant.io with documentation: home-assistant/home-assistant.io#8654
Example entry for configuration.yaml (if applicable):
planifneige:
  api_key: YOUR_API_KEY
  database_path: /config/planifneige.db
  streets:
    - name: 2-22 Sainte-Cahterine (Right)
      streetid: 13812091
    - name: 1030-1116 Saint-Laurent (Left)
      streetid: 13811012

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
tox tests fail on typing tests (bold emphasis mine) - I tried to run them on a clean git clone of dev and get the same failure:

SKIPPED:  py35: InterpreterNotFound: python3.5SKIPPED:  py35: InterpreterNotFound: python3.5
  py36: commands succeeded
SKIPPED:  py37: InterpreterNotFound: python3.7
SKIPPED:  py38: InterpreterNotFound: python3.8
  lint: commands succeeded
  pylint: commands succeeded
**ERROR:   typing: commands failed**
  cov: commands succeeded

[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

I for one am happy to see this integration! 👍

Hello @kkr16,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

b850d71537d97f14e125d58c108830b4aefcd27d - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
1bee8090bf4042121af79c6d60dc99d2dc0bc563 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
d667a71c7b6598548f6779e43f373d5fc88e41e6 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
012bb4785a586d87e45e1e653e895ff64f54e580 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
23dde2874ff88cd11335b635aa2b6189362a6acc - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
921c0dfd715256a0e3d184d91b4baa26e1cfd75f - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
e074fad0fa57138d1e4b320304cde2567ddb2b8d - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
57883f7d78732a15e9afe1fc8f9bae2e72d08690 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
8c05b9a0bd41ca8a3f88cc4cea9d02b7d4405bd5 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
8152f948b972dbc0107b0644ce6adf71a4c10b43 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.
eb4b12e7452d18e1d6603fa87e6056bb1542a980 - This commit has something that looks like an email address (karim.roukoz@ericsson.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

I followed instructions below to correct a wrong email address in my previous commits, and clearly did something wrong.
Home Assistant team, please close this pull request. I'll rebase off a clean dev and re-submit a PR if that is OK.

2. You can use this script that GitHub provides to rewrite history


Do we need to prefer evicting pre-upgrade peers from the peer set across a network upgrade?

TODO

[ ] make Zebra disconnect from outdated peers after a network upgrade #1334
[ ] check that we don't need to reject new connections from outdated peers in the lead-up to a network upgrade
[ ] check that we don't need to close existing connections to outdated peers in the lead-up to a network upgrade
[ ] add a comment to the relevant file, listing the assumptions that are required for this design to be secure:

network convergence: Zebra knows the addresses of a large number of upgraded peers
starvation: Zebra attempts other addresses before re-trying rejected or disconnected addresses
connection flooding / starvation: Zebra rate-limits closing existing connections #1334

Motivation
Originally posted by @hdevalence in https://github.com/ZcashFoundation/zebra/pull/702
#702 added a comment that we should attempt to evict pre-upgrade peers from the peer set and prefer choosing post-upgrade ready peers for queries.
We should be careful to do everything we need to do to handle a network upgrade and it's good to consider doing this (I presume this matches zcashd behavior?).  But on the other hand, there are a couple of reasons to prefer not to do this:

in general, it's some additional code for us to write;
we would have to build additional plumbing to connect information about peer versions to the eviction logic;
we would have to change the details of the load balancing mechanism and build plumbing to connect peer versions to the load balancer;

I think that if we do nothing, we'll probably be fine:

we can close connections to peers that don't upgrade;
we connect to a very large number of peers, so we can afford to close many connections to peers that don't upgrade;
if all of the peers we were connected to didn't upgrade, we're in a network full of bad peers anyways, so trying to filter them would just prepone the failure.

Does this analysis seem right, or am I missing some important details?
Related Issues
#1334 disconnecting old peers after network upgrade activation
#337 ZIP-200: Network Upgrade Mechanism

cc @teor2345

The zcashd behaviour is specified in ZIP 201.

Thanks for the pointer! It looks like that spec is very specific to the details of the Zcashd network stack, which we don't replicate in Zebra, so I think we still need to figure out a good answer to these questions.

I think there are a few different questions here:

How can Zebra prefer new peers before the activation height?

How can Zebra disconnect old peers with existing connections at the activation height?

Do we need to prefer new peers before the activation height?

Can we guarantee that Zebra nodes always have a large number of peers?

Here's a quick design sketch for preferring new peers:

for every peer that connects, if it is an older version, reject the connection with some probability P. For example, we could set P to 0.5 just before the activation height (~1.5 days), and 1.0 at the activation height.

We might need to artificially increase the peer set churn, if long-lived connections to peers can last longer than 1.5 days. (We could lower the maximum peer connection time, or randomly disconnect peers.)

Here's a quick design sketch for disconnecting old peers:

for every peer, record its protocol version
at the activation height, disconnect old peers

if all of the peers we were connected to didn't upgrade, we're in a network full of bad peers anyways, so trying to filter them would just prepone the failure.

If we are in a network with a large proportion of old peers, we want to cleanly separate from them at the activation height. Preferring new peers makes us gradually discover new peers, via the old peers we are connected to. So we gradually increase our chances of being connected to new peers.

Let's assume:

We are in a large network (compared with our peer set size), with a large proportion of old peers
By chance, we happen to mostly be connected to old peers

Here are a few different scenarios:
A. 1. We struggle to download the activation block, because old peers won't validate it.
A. 2. The network partitions, and mines several different activation blocks, increasing the chance of chain reorgs across the activation height
B. 1. We abruptly disconnect from all the old peers, as soon as we validate the activation block
B. 2. Every Zebra node suddenly goes looking for new peers, leading to a spike in network load
C. 1. (same as B)
C. 2. There is a chain reorganisation across the network upgrade height, so we suddenly accept old peers again, leading to thrashing
D. 1. We disconnect from old peers, isolating ourselves in a small network of new peers.
D. 2. We are vulnerable to an eclipse attack:

https://zips.z.cash/zip-0201#network-coalescence
https://eprint.iacr.org/2015/263

we connect to a very large number of peers, so we can afford to close many connections to peers that don't upgrade;

I don't think we can guarantee that Zebra will always have a large number of peers.

Some users will want a small number of peers. For example, connecting to a single trusted node can be useful for:

wallet support,
network-level anonymity, or
sub-instances of a miner or exchange.

At the moment, we don't enforce having a large number of peers, users can set the peer limits as low as they like. (We don't seem to support a very low number of peers right now, but that's a separate issue, see #704.)
We could:

warn if the peer limit is low, or
warn if the actual number of peers is low.

But people don't read warnings, so we could:

shut down if the number of peers is very low, or
require users to configure a trusted peer mode to disable warnings or shutdowns.

I think this complicates the design, but for many users, a node shutdown is better than being attacked.
But I think it's easier for us to make sure Zebra has good peers.

Q 4. I don't think we can guarantee that Zebra will always have a large number of peers.
Some users will want a small number of peers. For example, connecting to a single trusted node can be useful for:
* wallet support,

* network-level anonymity, or

* sub-instances of a miner or exchange.

Supporting arbitrarily few peers was a non-goal of the architecture of the peer set (and the network stack generally), which aimed to treat peer connections as "cattle, not pets" (regrettable phrasing notwithstanding). So although some users may want a small number of peers, we should not seek to support that use-case.

Q 4. I don't think we can guarantee that Zebra will always have a large number of peers.
Some users will want a small number of peers. For example, connecting to a single trusted node can be useful for:
* wallet support,

* network-level anonymity, or

* sub-instances of a miner or exchange.

Supporting arbitrarily few peers was a non-goal of the architecture of the peer set (and the network stack generally), which aimed to treat peer connections as "cattle, not pets" (regrettable phrasing notwithstanding). So although some users may want a small number of peers, we should not seek to support that use-case.

Security arguments should be based on threat models, not supported use-cases.
"A Zebra node has a small number of peers" is something that can happen due to network conditions or a deliberate attack. (Even if we don't support configurations that deliberately seek to reduce the number of peers, they should be as secure as reasonably possible.)

I don't think we need to make any code changes for this ticket, as long as the changes in other tickets satisfy the following conditions:

network convergence: Zebra knows the addresses of a large number of upgraded peers

Zebra doesn't need to be connected to these addresses at the network upgrade, and they don't need to be a large proportion of its address book - they just need to be in the address book

starvation: Zebra attempts other addresses before re-trying rejected or disconnected addresses
connection flooding / starvation: Zebra rate-limits closing existing connections

These changes should be made in #1334 and various peer set security tickets.


New Atlas logos

 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.

Can we re-create this PR into 0.1.0-rc instead of into master?


Add link to introductory book of R

What does this PR do?
Add resource(s)
For resources
Add a introductory book for R language
Description
An introductory book in portuguese (Brazil) that teaches the R language.
Why is this valuable (or not)?
It is an introductory book for the R language, which teachs the basics of the language and its application to data analysis. The book covers:

language syntax and data structures available;
popular data frameworks used for various data analysis tasks;
how to draw plots;
how to debug R code;
more advanced concepts of the language, such as environments and functional programming patterns.

How do we know it's really free?
It is an open, free and static website based on a GitHub repo.
For book lists, is it a book? For course lists, is it a course? etc.
Is a book
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!
Thanks for making you book free! If it has an Open License, consider adding it to Unglue.it and Archive.org.


Don't hardcode style colors in widgets

Colors should not be inside the widgets themselves. Possibly use constants and/or integrate UI Style Lang to allow for styling via a stylesheet.

Resolved via using constants. Maybe in the future we could set styles via an external stylesheet.


[ListItem] Fix triggers onTouchTap when disabled

I think it was forgotten to send disabled to EnhancedButton as it has internal logic to not trigger onTouchTap if disabled is set to true.
Closes #7449.

nope :D

I removed onTouchTap event to the disabled div. Also i sent the disabled property to EnhancedButton to sent all items.

This is getting ridiculous. Pleas squash the merge.

are you sure it will work as expected? wont this remove the onTouchTap for all states?

you are right. Do you want me to open a new PR with only one commit?
regards

As you prefer, I will squash the commits if needed.

@jonashartwig Thanks

@oliviertassinari  EnchancedButton part is unreachable when disabled is true https://github.com/callemall/material-ui/pull/7486/files#diff-a3d0f303a92012b705e6df2b002e8a0eR722
This PR contains mistake.

@hwo411 True, I just thought it's inoffensive to have that logic.

yes, but then it does not change anything, it will send all values that are falsey as disabled then. Is it supposed to be fixed? I dont think it will break anything


Add support for python 3.5 and fix tests on 2.6.

Verified that @Jeff-Meadows has signed the CLA. Thanks for the pull request!


Accidentally cancelled inceptionV3 during install, now can't install at all

Hello,
i was setting up tensorflow for image classification, and after i ran :
python -m scripts.retrain
--bottleneck_dir=tf_files/bottlenecks
--model_dir=tf_files/models/"${ARCHITECTURE}"
--summaries_dir=tf_files/training_summaries/"${ARCHITECTURE}"
--output_graph=tf_files/retrained_graph.pb
--output_labels=tf_files/retrained_labels.txt
--architecture="${ARCHITECTURE}"
--image_dir=tf_files/flower_photos
It automatically started installing inception, i realized that i needed to change some options so i cancelled the install of inception.
Now i believe that i have a half install that doesn't let me install the full package or use the half package.
I may be wrong, but any suggestions would be appreciated.
FYI: i've run :
pip install inception, to which i receive a "python setup.py egg_info" failed with error code 1 in {my local/temp dir}
I also just tried running the scripts.retrain again, to which i receive a "EOFError: compressed file ended before the end-of-stream marker was reached"
Running on Windows 7

facing the same issue. Can you help me, how have you resloved it?


Signal failure when the URL is malformed

What:
Fix a bug where if the testBaseURL is malformed (or an invalid value is provided for port property, parameter, etc), no tests are executed but the test suite passes.
Checklist:

[ ] Documentation added to the README.md OR link to PR on https://github.com/specmatic/specmatic-documentation
[x] Tests
[ ] Sonar Quality Gate

I added below changes to this

Deleted isValidURL
ValidateURL should return success for URLs such as "http://test.com" where port is not mentioned
Parameterise tests to validate multiple valid and invalid testBaseURLs
Surrounding success tests with assertThatCode { ... }..doesNotThrowAnyException() because TestAbortedException does not cause test failure, instead quietly ignores the test that failed.


📃: Want to add Simon Say game under 'Vanilla JS' project

:red_circle: Title : Simon Say
:red_circle: Aim :  It implement all advance core logic of JS
:red_circle: Tech Stack Used : HTML, CSS, JS
Screenshots 📷

https://github.com/user-attachments/assets/14e49fb1-032e-47c9-809b-6b55971ef3ca

:white_check_mark: To be Mentioned while taking the issue :

Full name : Suvadip Sana
What is your participant role?  gssoc-extd, hacktoberfest

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

#108


Add multiline comments

#{ and #}. Also removes a few unused imports.
(The branch name and commit name, I just wasn't thinking)
Next we need to work on Vylight to add support for this.
Closes #876.

sighs I'll update Vylight next week

No, I'll update Vylight. :)