Using C++14 and C++17 features

I have noticed several changes in CMakeLists.txt related to C++14 and C++17. If I'm not mistaken we can write C++ code in Pistache based at least on C++14 standard (even more simply, we can use features from C++14). Am I right?

As far as I know, the answer you your question is "no".  Pistache should still compile on an older Debian distro that only supports c++11.  Another contributor added a "string_view" that compiles on c++11 and implements the c++14/17 spec.
Unfortunately, I don't recall who is using the older compiler.  We should probably announce a deprecation date for pure c++11, and allow contributions that only support c++17 and beyond.
I've been holding off on making an official release b/c we still have lots of bugs and interface-level (eg, things exposed via header files) code changes.  But before we drop c++11 support, we should probably have an official release that does support c++11 so that if someone really needs it, we can point them to it.
I don't want to be tied down by c++11 forever, but I don't have a solid plan w/ firm dates to move past it.  I'm open to suggestions!

I think it was me with the issue of an older cmake on debian stable.
For us it would be ok of you move forward, we will adjust.

Hello again, I have a question, why we run building project on travis using gcc 4.9 ? It was added several weeks ago. What's the reason? More than one month ago we decided to go to C++14, but as I have already written here only since gcc 5.0 there is C++14 full support.

knowledge4igor@: Good point.  I do not have a good mental map of which GCC versions support which C++ features, and which Ubuntu distros ship with which compilers.  Arthur refactored the travis test to get better compiler coverage.
We are a lot closer to declaring an official release now that the memory leaks have been fixed.  I'm still concerned that the unit tests are flakey though.
I propose that once we fix the tests so that they are stable (or the code, as needed), we cut a 0.1 release.  Anyone who wants c++11 only (and not > c++11) can fork from there, which this project moves forwards to an eventual c++17 only code base.  Thoughts?
Sadly, my time is highly constrained until mid-April.  I can review and approve PRs, but I won't have much time for authoring them myself.

@dennisjenkins75 Ok, l hope all active contributors will be concentrated on fixing all defects and writting tests and after this stabilization we will move to C++17.

This issue could be closed because currently we moved to C++14.

For better or worse, we are on c++14 now.  c++11 support was dropped.


Implement EIP-2124: Fork identifier for chain compatibility checks

I'd rather hold back on the newsfragment until we are actually including the fork-id in our ENR (#1415) and validating the ones we receive

I'd rather hold back on the newsfragment until we are actually including the fork-id in our ENR (#1415) and validating the ones we receive

Fair enough!


Location name changes post r26631

On loading TourGuide prints the following to the gCLI and the main window never loads.
Bad location value: "Frat House In Disguise" (relay_TourGuide.ash, line 38570, char 83 to char 105)
Bad location value: "Hippy Camp In Disguise" (relay_TourGuide.ash, line 38572, char 84 to char 106)
Bad location value: "Frat House In Disguise" (relay_TourGuide.ash, line 38570, char 83 to char 105)
Bad location value: "Hippy Camp In Disguise" (relay_TourGuide.ash, line 38572, char 84 to char 106)
See https://kolmafia.us/threads/ash-location_accessible-function.27836/ for details.

this has been fixed in #62 -- thank you! :-)


@conda-forge-admin, please add user @cmutel

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @cmutel in conda-forge/bw2analyzer-feedstock#4.


vllm video inference by frame exceeds limit_mm_per_prompt

[rank0]: ValueError: You set video=10 (or defaulted to 1) in --limit-mm-per-prompt, but found 26 items in the same prompt.

manually converted pils to tensors


Prep for HAL 0.5.0 release

Update the changelog with all HAL changes, bump version number

Well this is fun.
If I don't bump the HAL version in the BSPs, I get a CI error from the BSPs (since the in-tree HAL is a different version).
If I do, I get a type conflict (because the PIO crate pulls in 0.4.0's rp2040_hal::pio::UninitStateMachine).

Those circular dependencies really get annoying.
For i2c-pio I already suggested moving the example https://github.com/ithinuel/i2c-pio-rs/pull/6.
But we have the same issue with ws2812-pio. And that one's worse, because there are several boards with ws2812 LEDs, so having those examples for the board crates is really nice.
As those examples already depend on a git version of ws2812-pio, it's easy to work around by pointing to an updated git version, eg.:
-ws2812-pio = { git = "https://github.com/ithinuel/ws2812-pio-rs", rev = "fd6b6604d65a66242b52ccf7f24a95ca325991dd" }
+ws2812-pio = { git = "https://github.com/jannic/ws2812-pio-rs", rev = "b6d27e8f3715c2b4d0fc0737c366314beb1f18c7" }

(Just an example - of course @ithinuel could include the updated dependency in his own git, no need to point to my fork.)
We could also include a vendored version of ws2812-pio somewhere inside https://github.com/rp-rs/rp-hal so we could keep the versions in sync. As ws2812-pio is basically a single small source file, that wouldn't cause much overhead.

Yeah, I agree. I've started down the vendoring approach too.
The other option is to have BSPs not in this repo, so they have their own CI, can pin to a specific version of the HAL and {i2c,ws2812}PIO drivers that agree with each other and not break everything.

Instead of vendoring inside of the rp-hal repo we could also have a downstream fork of those repos under rp-rs.
That makes it easier to update them if the main repos change.

If we start adding new repos I think I'd prefer a separate repo for the BSPs, as you suggested above.

I was planning on having a discussion on how we maintain BSPs after this release was out, so if we can find a clean solution that will get us past this issue that would be my preference.
Having the PIO drivers support a range of versions (like https://github.com/ithinuel/ws2812-pio-rs/compare/main...9names:rp-hal_0.5.0) will get us over the line for CI, but users will have to "cargo update" at least for a few packages when they update their hal/bsp to avoid them trying to compile against 0.4.0 while the HAL itself is 0.5.0
That's unfortunate.
We'll want a new release of the drivers themselves anyway for folks that are using the crates.io versions of the HAL and BSP

Hmm no, if I select a range of versions for ws2812 that would be locking us out of finishing the PIO fifo changes during 0.5, which I absolutely do not want.
I'll try to get that resolved first.

As a reminder I'm ok to move the ws2812b-pio-rs & i2c-pio-rs to the rp-rs org instead of maintaining a fork or having to vendor it.
I believe we should start using the crates' version rather than a git-hash.
Having each BSP in their respective repo (or 1 repo to hold them all) seem like a sensible solution to me since the circ dep in on the repo not on the crates.

Thanks, I forgot you said that it was okay.
Would you be able to add rp-rs/admins as a crate owner for each of them as well, so that we can push version bumps to crates.io if necessary?

What about this sequence of changes?

Push branches refering to rp-hal version 0.5.0 to https://github.com/rp-rs/ws2812-pio-rs and https://github.com/rp-rs/i2c-pio-rs
Change the examples to use those, by their git revision
Release rp-hal version 0.5.0
Do releases of ws2812-pio-rs and i2c-pio-rs depending on rp-hal 0.5.0
Change the examples to refer to the released versions

Step 5 isn't a breaking change, as those are just dev-dependencies for the examples. And I guess most people copying the examples are getting them form github anyways, so it doesn't matter which versions the release points to.

I still want to get the FIFO changes across the line for 0.5.0.
Now that we have the pio repos under rp-rs i'm okay with linking to a git branch.
Moved my i2c-pio-rs branch into rp-rs/i2c-pio-rs, updated #352 to use that branch.
I'll rebase this on top of main once that is merged, then create branches for ws2812-pio-rs and i2c-pio-rs that depend on rp-hal 0.5.0.

How shall we handle the board support packages?
As they re-export the hal, a breaking change of the hal is also a breaking change of the bsp.
Do we release them separately, or shall we bump their versions at the same time?

Do we release them separately, or shall we bump their versions at the same time?

When releasing new versions of the BSPs, we should also update their changelogs, and I noticed some of them contain broken links to non-existing tags, e.g. in https://github.com/rp-rs/rp-hal/blob/main/boards/adafruit-feather-rp2040/CHANGELOG.md
To not delay the HAL release, I'd prefer to do the BSP releases later.

I've usually left BSP release stuff until after the HAL is out.   Some of them have not even had a release yet.
All the links will need to be updated if we move the BSPs, that won't be fun.


Training Code

I wanted to inquire if it would be possible for you to provide the training code for the method mentioned in your paper.

Due to my current research focus on the field of object detection, I haven't released the complete training code for this project. However, I will provide a project that includes the training code with CE in the future.
If you urgently need the training code to complete your task, you can try replacing the backbone network and loss function using the works at  and
Furthermore, this is our earlier work. Perhaps the recent work of Meta AI, SiLK (https://github.com/facebookresearch/silk), could be more helpful to you, as we have noticed that it covers a comprehensive range of content.
Thank you for your attention, and if you have any further questions, please feel free to continue the discussion.

Due to my current research focus on the field of object detection, I haven't released the complete training code for this project. However, I will provide a project that includes the training code with CE in the future.
If you urgently need the training code to complete your task, you can try replacing the backbone network and loss function using the works at and
Furthermore, this is our earlier work. Perhaps the recent work of Meta AI, SiLK (https://github.com/facebookresearch/silk), could be more helpful to you, as we have noticed that it covers a comprehensive range of content.
Thank you for your attention, and if you have any further questions, please feel free to continue the discussion.

Due to my current research focus on the field of object detection, I haven't released the complete training code for this project. However, I will provide a project that includes the training code with CE in the future.
If you urgently need the training code to complete your task, you can try replacing the backbone network and loss function using the works at and
Furthermore, this is our earlier work. Perhaps the recent work of Meta AI, SiLK (https://github.com/facebookresearch/silk), could be more helpful to you, as we have noticed that it covers a comprehensive range of content.
Thank you for your attention, and if you have any further questions, please feel free to continue the discussion.

Due to my current research focus on the field of object detection, I haven't released the complete training code for this project. However, I will provide a project that includes the training code with Cross-Entropy loss in the future.
If you urgently need the training code to complete your task, you can try replacing the backbone network and loss function using the works at (https://github.com/eric-yyjau/pytorch-superpoint) and (https://github.com/rpautrat/SuperPoint).
Furthermore, this is our earlier work. Perhaps the recent work of Meta AI, SiLK (https://github.com/facebookresearch/silk), could be more helpful to you, as we have noticed that it covers a comprehensive range of content.
Thank you for your attention, and if you have any further questions, please feel free to continue the discussion.


remaining issue 9/24-9/31

[ ] plot lineage of different states for figure 9.

[x] try different starting points.

[x] Run profiling.

[ ] write more concise text in the manuscript with a focus on the big picture.

I already looked through the profiling. What do you mean by different starting points? Like I've mentioned before, I don't think something like k-means initialization will be better. I think what we have is good.

I think we also talked about initialization for parameter estimation? Maybe I'm mistaken, I remember you said starting from different places in the plane may result differently because it is dependent on initial conditions. I believe we were looking at figure 5. I remember vaguely so it is likely that I'm wrong. @aarmey

I think that's where I said we should run the fitting twice and take the better results. We do that now.

Oh yes, I will take that off the list.


[Designer] Make it possible to share cards through the URL

Make it possible to have a URL that populates the designer with a design and sample data

Same idea than #8533
Although can be a good idea, big cards cannot be shared, it overflows the explorer


Update admin

Issue that this pull request solves
Closes: #693
Proposed changes
Admin update route added together with validation
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)

Checklist
Put an x in the boxes that apply

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[x ] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

@Kajol-Kumari  , please check it once, i have tested this codes on my local projects, it worked for them

@AyushSingh22 , @Kajol-Kumari  , I think , you should merge this request because i had already tested these files on my local system

@Kajol-Kumari maam, actually ,Ip

@Kajol-Kumari  maam, i did it just because of the fact that reset password ,change password and forgotten password have similar type of codes.So, I do not want to make it complex by using existing schema

Adding level 1 to this pr due to the extra effort that you had to put raising the pr again..


kubectl v1.29.7 is built with go1.22.5 but expected goVersion go1.21.*

What happened:
kubectl v1.29.7 is built with go1.22.5 but expected goVersion go1.21.*
What you expected to happen:
kubectl in v1.29 line is expected with goVersion go1.21.*
How to reproduce it (as minimally and precisely as possible):
Download kubectl from https://dl.k8s.io/release/v1.29.7/bin/linux/amd64/kubectl, and check version details
It's saying "goVersion: go1.22.5", but the expected is goVersion go1.21.* as set in https://github.com/kubernetes/kubectl/blob/kubernetes-1.29.7/go.mod#L5
# ./kubectl version --client -o yaml
clientVersion:
  buildDate: "2024-07-17T00:06:19Z"
  compiler: gc
  gitCommit: 4e4a18878ce330fefda1dc46acca88ba355e9ce7
  gitTreeState: clean
  gitVersion: v1.29.7
  goVersion: go1.22.5
  major: "1"
  minor: "29"
  platform: linux/amd64
kustomizeVersion: v5.0.4-0.20230601165947-6ce0bf390ce3

Anything else we need to know?:
Also checked kubectl v1.29.6 with "kubectl version --client -o yaml" command, which is with goVersion: go1.21.11
So it maybe build issue that goVersion bump in kubectl v1.29.7
Environment:

Kubernetes client and server versions (use kubectl version):v1.29.7
Cloud provider or hardware configuration: linux
OS (e.g: cat /etc/os-release): CentOS Stream 9 / amd64

Thanks for raising this issue, is it crashes? @caixiangibm
If it doesn't crash it might be something related to release team and i think we can compare our go version with main kk version and upgrade our too if everything is fine.

@ah8ad3 , not sure it crashed or not, we need kubectl v1.29.* with goVersion go1.21.* to be compatible our integration development.

/transfer kubernetes


[BUG] Prisma model generated by Ts.ED is incompatible with the one generated by Prisma

Information

Version: 7.x
Packages: @tsed/prisma

@eliectrlshift made a fork of the production ready template and, created a Prisma schema with a one-to-many relationship and when seeding the database gets the following error:
Type 'ProjectDriveModel' is not assignable to type '(Without<ProjectDriveCreateInput, ProjectDriveUncheckedCreateInput> & ProjectDriveUncheckedCreateInput) | (Without<...> & ProjectDriveCreateInput)'.
  Type 'ProjectDriveModel' is not assignable to type 'Without<ProjectDriveUncheckedCreateInput, ProjectDriveCreateInput> & ProjectDriveCreateInput'.
    Type 'ProjectDriveModel' is not assignable to type 'Without<ProjectDriveUncheckedCreateInput, ProjectDriveCreateInput>'.
      Types of property 'id' are incompatible.
        Type 'number' is not assignable to type 'undefined'.

Prisma schema:

model Project {
  id                Int             @id @default(autoincrement())
  uuid              String          @unique @db.VarChar(1000)
  publicName        String          @db.VarChar(255)
  privateName       String          @db.VarChar(255)
  slug              String          @db.VarChar(255)
  description       String          @db.VarChar(500)
  projectDrive      ProjectDrive?
  createdAt         DateTime?       @default(now()) @db.Timestamptz(3)
  createdBy         String?         @db.VarChar(255)
  updatedAt         DateTime?       @default(now()) @updatedAt @db.Timestamptz(3)
  updatedBy         String?         @db.VarChar(255)
  deletedAt         DateTime?       @db.Timestamptz(3)
  deletedBy         String?         @db.VarChar(255)
}

model ProjectDrive {
  id                        Int                   @id                                   @default(autoincrement())
  uuid                      String                @unique                               @db.VarChar(1000)
  folderId                  String?               @unique                               @db.VarChar(500)
  name                      String?               @unique                               @db.VarChar(500)
  url                       String?               @db.VarChar(500)
  project                   Project?              @relation(fields: [puid], references: [uuid])
  puid                      String?               @unique
  createdAt                 DateTime?             @default(now())                       @db.Timestamptz(3)
  createdBy                 String?               @db.VarChar(255)
  updatedAt                 DateTime?             @default(now())                       @updatedAt                    @db.Timestamptz(3)
  updatedBy                 String?               @db.VarChar(255)
  deletedAt                 DateTime?             @db.Timestamptz(3)
  deletedBy                 String?               @db.VarChar(255)
}

ProjectDriveModel generated by Ts.ED :

export declare class ProjectDriveModel implements ProjectDrive {
    id: number;
    uuid: string;
    folderId: string | null;
    name: string | null;
    url: string | null;
    project: ProjectModel | null;
    puid: string | null;
    createdAt: Date | null;
    createdBy: string | null;
    updatedAt: Date | null;
    updatedBy: string | null;
    deletedAt: Date | null;
    deletedBy: string | null;
}

ProjectDriveCreateInput generated by Prisma:

export type ProjectDriveCreateInput = {
    uuid: string
    folderId?: string | null
    name?: string | null
    url?: string | null
    project?: ProjectCreateNestedOneWithoutProjectDriveInput
    createdAt?: Date | string | null
    createdBy?: string | null
    updatedAt?: Date | string | null
    updatedBy?: string | null
    deletedAt?: Date | string | null
    deletedBy?: string | null
}

ProjectDriveUncheckedCreateInput generated by Prisma:

export type ProjectDriveUncheckedCreateInput = {
    id?: number
    uuid: string
    folderId?: string | null
    name?: string | null
    url?: string | null
    puid?: string | null
    createdAt?: Date | string | null
    createdBy?: string | null
    updatedAt?: Date | string | null
    updatedBy?: string | null
    deletedAt?: Date | string | null
    deletedBy?: string | null
  }

How to reproduce the issue

git clone https://github.com/eliectrlshift/express-typescript-skeleton-306
yarn install
cp .env.exemple .env
Change values for PostgreSQL database
yarn run prisma:migrate
yarn run prisma:seed

The original issue can be found at the following link: https://github.com/borjapazr/express-typescript-skeleton/issues/306
Thank you in advance 🙏

Hello @borjapazr ,
I already read the issue. But I haven't time to works on it. Making PR is totally possible!
The code generator is here:

https://github.com/tsedio/tsed/tree/production/packages/orm/prisma/src

You can start by opening a PR, with a integration test that reproduce the issue:

https://github.com/tsedio/tsed/tree/production/packages/orm/prisma/test

You can duplicate the mongo or postgres integration test and update the prisma schema and then run the unit test on @tsed/prisma package.

Then we'll try to add some tests to cover your usecase.
Also, you reporting a typescript issue, but you don't give where the problem occurs. I expect at least to have this kind of information :) (is it the generated repository?)
See you
Romain

Hello, @Romakita 😊.
Sorry for not specifying enough information. Anyway, I think the problem is easy to see. In fact, it can be reproduced by following the steps indicated in the issue. Basically, when you run the database seed yarn run prisma:seed, the mentioned error occurs.
As you say, it is a TypeScript error, but it is related to the model generated by Ts.ED. In this particular situation, the problem occurs when performing the insertion of an element through the PrismaClient or through the repositories generated by Ts.ED.
At the moment I have not been able to check what could be the solution to this problem either, I simply limited myself to report this issue since several people reported it in my project. If I have time this weekend I will try to check it, but no promises. Sorry.
I will keep you updated.
Thank you for your work and effort.
Regards.

Hi again, @Romakita!
Maybe it's a Prisma problem? 🤔
https://github.com/prisma/prisma/issues/15946
https://github.com/prisma/prisma/issues/8797

Yes the second seems to be similar ^^


Remove plane feature

As of PR #581 and PR #582, nothing uses the plane feature any more.

[Current coverage][cc-pull] is 76.28%

Merging [#583][cc-pull] into [master][cc-base-branch] will increase coverage by 2.24%

@@             master       #583   diff @@
==========================================
  Files            83         80     -3
  Lines          7316       7160   -156
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           5417       5462    +45
+ Misses         1899       1698   -201
  Partials          0          0

Powered by Codecov. Last updated by [878ecc7...0413397][cc-compare]
[cc-base-branch]: https://codecov.io/gh/OpenGeoscience/geojs/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/OpenGeoscience/geojs/compare/878ecc7751dd5d1c8f4f82ba4fca49a259774073...0413397fe795be22e23e36e2be80d8b079493e36
[cc-pull]: https://codecov.io/gh/OpenGeoscience/geojs/pull/583?src=pr

LGTM 👍


Request for information on how to report exceptions.

Hi, thank you for the fantastic library, this is very useful!
This isn't really an issue so much as a question, sorry I wasn't sure where to ask. Please feel free to close this issue immediately.
When I get an exception inside a test, it gets swallowed like so:
ERROR in (defrule-test) (TypeError:NaN:NaN)
Uncaught exception, not in assertion.
expected: nil
  actual: #object[TypeError TypeError: Cannot read property 'name' of null]

The easy workaround is I put the code outside a test and run it again to get info on where the problem occurred.
Conversely, sometimes I get exceptions thrown from spec that would really benefit from some interpretation. Is there a way for me to include figwheel style error formatting in my command-line test command?

Hmm, yeah I think this is just something cljs.test does I'm afraid, maybe someone who's dealt with this before has a better trick but I'll sometimes wrap the failing code in a (try (catch ...)) that logs out the entire error. You could even wrap that up in a macro or function to make it a little easier to write.
If you want nice formatting of your spec errors, I'd suggest using expound to do your checking. You can use it alongside orchestra to get full instrumentation with human readable errors! It works really well, kind of Elm inspired? I love using it at network boundaries personally.
I hope this helps at least a little bit! Even if it's not exactly what you were hoping for. I hope you're having a great weekend :smile:

Thinking about it, I would imagine it's the cljs.test/is macro that's swallowing these, maybe there's some config in there like something you can bind or you could write your own is that catches and displays errors properly?

Good ideas, thank you!


Match es.** doesn't work

Hello,
it seems like <match es.**> doesn't work and in fact the only match that works is **.
Here's what I have in my config:
<source>
  type tail
  format apache2
  time_format "%d/%b/%Y:%T %z"
  path /var/log/httpd/error_log
  pos_file /var/log/td-agent/apache.error.pos
  tag es.apache.error
</source>

<match es.**>
  type "aws-elasticsearch-service"
  type_name "error_log"
  logstash_format true
  include_tag_key true
  tag_key "@log_name"
  flush_interval 5s

  log_level debug
  buffer_chunk_limit 512K
  max_retry_wait 300

  <endpoint>
    url "https://es-link..."
    region "us-west-1"
    access_key_id "...."
    secret_access_key "....."
  </endpoint>
</match>

Any input appreciated,
Thanks

Hi @egulik ,
Thank you for the feedback.
Can you publish the log of fluentd?
thanks,
atomta

ran into the same things es.** doesn't work, only ** would work


fix(scotia): seperated the files into 3 .py files

I have seperated the init.py file into three files to make it more modular.

Thank you for your time, good luck friend.


🐞 Kubernetes integration docs + helm chart need to be updated to the latest version

What is the issue?
Coming from here: https://discord.com/channels/707636530424053791/1229798907362414673
Looks like our helm chart doesn't really work with the latest version of Dagger and it's pinned to a v0.9.x engine. We should try to keep it updated every time we release a new version.
cc @gerhard @matipan
Dagger version
dagger v0.11.1
Steps to reproduce
Follow the guide here: https://docs.dagger.io/integrations/104820/kubernetes
Try updating the engine image from v0.9.x to the latest dagger version.  The Dagger engine won't start after that.
Log output
No response

Hehe... theoretically, your CD automation should be able to take care of this (with Dagger??? :grin: ). :P
Scott

Hehe... theoretically, your CD automation should be able to take care of this (with Dagger??? 😁 ). 😝

yep :P

I'm currently going through the upgrade to v0.11.1 for our internal pipelines so I'll take a look at this one! The helm chart is a bit forgotten at the moment but with the new dynamic CI we can definitely make some tests that raise this for us since we can simply target runs-on: dagger-v0-11-1. Maybe we could integrate some of this as part of the follow ups for the release? cc @gerhard @jedevc

:wave: just sent a PR to bump the chart for the moment here https://github.com/dagger/dagger/pull/7176

Partly addressed in:

https://github.com/dagger/dagger/pull/7205

The follow-up is to automate this.

we have improved the release process and we're already bumping the helm chart on each release. Thx @gerhard


{Core} Remove uuid workaround code

This code was introduced in https://github.com/Azure/azure-cli/pull/5644 to fix https://github.com/Azure/azure-cli/issues/5184.
Since it's fixed in upstream https://github.com/python/cpython/pull/5254 a long time ago, we don't need it anymore.

Core

This PR will simplify https://github.com/Azure/azure-cli/pull/29888.


fine-tuning

Is it possible to fine-tune a gpt4free model with langchain-gpt4free?

you can find model and provider in g4f repo and replace to change


"no roms were found in gba folder", even if there's GBA roms inside the folder.

As the title implied, I have the gba folder on the root of my micro SD card (using an action replay MAX gamecube memory card), I have roms in there that are smaller than 256KB, but the program is still throwing me a "No .gba files found in gba folder". Am I missing sometihng here?

Did you ever find a solution to this?


Added turn indicators

Fixed EOL problem. now only shows proper changes

@dcoollx recently, a pull request was merged that allowed more than 2 players. Therefore a few conflicts will be there and you will have to modify the logic to support more than 2 players as well.
You can either close this pull request, update your fork with the latest ketanhwr:master and make another pull request with the new logic, or you can rebase your pull request over my current master. I'll suggest the first since you will have a nicer commit history.

will create a new one with multiplayer logic per your request


isValidNumber is failing some valid numbers.

"(366) 522-8999" parsed with "US" and "+84 74 883313" parsed with "ZZ" both fail isValidNumber:
Changing the area code to something like "(818) 522-8999" for the first one passes.
"+84 74 883313" is a valid number for Vietnam - from wikipedia:
Area codes starting with 7[edit]
With effect from 5 October 2008, all telephone numbers in this area have 7 digits.
74 – Trà Vinh Province
http://en.wikipedia.org/wiki/Telephone_numbers_in_Vietnam
Debugging a little it looks like the problem is somewhere in this returning nil
NSString *regionCode = [self getRegionCodeForNumber:number];
I'm guessing this has something to do with metadata rules?

According to this site http://www.computerhope.com/areacode.htm
366, 367, and 368 are unassigned area codes
So it looks like expected validation result

I have saved number "6375273023" in my Contacts. When I validate number it returns false. Can anyone help.
NBPhoneNumberUtil *phoneUtil = [NBPhoneNumberUtil sharedInstance];
NSError *anError = nil;
NBPhoneNumber *myNumber = [phoneUtil parse:mobileNumber
defaultRegion:@"IN" error:&anError];
BOOL isValidNumber = [phoneUtil isValidNumber:myNumber];
isValidNumber returns NO.
Can anyone help.


CMD_OVERRIDE_GOTO does not work

It seems to me that current implementation of CMD_OVERRIDE_GOTO does not work.
https://github.com/PX4/Firmware/blob/master/src/modules/commander/commander.cpp#L957-L964
When I send the command in order to pause processing of the mission, the commander writes
WARN  [commander] Pause mission cmd
and positive acknowledge is returned, but the vehicle continues.
I suppose that changing of status_local->nav_state does not change the control state.
As many other commands use main_state_transition function, is it possible that this handling of CMD_OVERRIDE_GOTO is obsolete?

Yes that's pretty bogus and should be deleted. Try MAV_CMD_DO_REPOSITION instead.

Thanks! :)
MAV_CMD_DO_REPOSITION with all values NAN pauses the mission and restart is possible with MAV_CMD_MISSION_START, I just need to keep actual mission item index.


Handle "data not available here" in a scalable way

We know that some datasets won't have data for a place (permafrost, etc).  Right now there's a sort of default "Not available here" blurb which is a good start, but how could we handle this when 2 or 3 datasets are missing?  Possibly we include a block in the header (by the mini-map) that is its own "Data not available" block that explains what's not available, and why (coverage doesn't include this area, or it's not possible to include for some technical reason).

This issue is mostly done. All that is left to do, I think, is to come up with less technical text that explains why data is missing (where possible).


Change linux to posix

Closes #21

A patch release will be required.

AFAICT os.name == posix isn't equivalent to being on a Linux OS; for example it could also be MacOS. The sys.platform gives a more fine-grained result.
Since I don't have the context, I don't know which one would be appropriate here.

Hi @greschd! You are totally right, 'posix' refers to both types of OS (Linux and MacOS). However, the pexpectlibrary works fine on all 'posix' OS and thus, it should be represented in all of them (if available)
https://pexpect.readthedocs.io/en/stable/install.html#requirements


fix: add /provider directory for dependabot

Hi,
This PR fixes the directory for the dependabot, when looking for go modules.

@ringods ofc! Done!


Connection to existing cluster (created with kubeadm)

When I try the Kubernetes Create command I hit the below error. So I assume I need to connect to a cluster first? Is there a way to configure the extension to use a kubeadm cluster?

Correct.  The extension uses your active kubectl context (in other words, the cluster to which kubectl is currently connected).  To activate a particular cluster you could either:

Open the Kubernetes tree view in the explorer pane, select your kubeadm cluster, right-click and choose "Set as Current Context"
In the Terminal, run kubectl config use-context <your_context_name>

If your cluster isn't currently in your kubeconfig file, then you'll need to add it.  The extension doesn't currently provide a way to do this; you'll need to use cluster-specific tools or kubectl config sub-commands.

Hi,
So I've connected to my cluster in the terminal

But my cluster doesn't appear in the explorer

And I can only configure Azure clusters

Does vs-kubernetes expect the kubeconfig to be in a certain folder? For instance mine resides in %HOMEPATH%\.kube\dev\config

Does the extension expect the kubernetes config file in specific location. Mine is under %USERHOME%\.kube\dev\config
Connected to my kubeadm k8s cluster now usin terminal

Nothing appears in explorer config k8s pane

And only Azure options in

We don't expect a particular kubeconfig location, we just invoke kubectl.  But clearly we're not getting any output from kubectl in your case.    Does your terminal have a KUBECONFIG environment variable set up?  If so, is that same environment variable set up in the VS Code process?  If kubectl is using a different (and empty) config file when we call it, that would explain the behaviour you're seeing I think...
Re the Configure pane - yeah, we only have configure from Azure built in at the moment.  Would love to talk about how we can support configuring from a kubeadm cluster too.  But if your kubectl is working then you shouldn't need to do the configure step.
Thanks for your patience in investigating this!

@dkirrane would you be able to run an echo $env:KUBECONFIG in a powershell window? We're considering making the kubeconfig location configurable in the extension, but need to confirm why the extension isn't picking up on the location of your kubeconfig.

@itowlson: based on feedback in Beijing and several sporadic reports, we should be able to load a config visually along with "Add Existing Cluster". Can we promote this to a feature request, and review whether it's required?

@squillace Yes, absolutely.  Can you expand on what you mean by "load a config visually"?  Do you mean having a command (or right-click option) to say "I want to use a nondefault kubeconfig file" and have it prompt you for the kubeconfig?  What is the scope, e.g. does the selected config remain in force from session to session, or only for the current session?

As a temporray workaround, note you can merge multiple kubeconfigs with (slightly hacky)
KUBECONFIG=`/path1/config:/path2/config`

And then run `kubectl config view`

that will give you the merged kubeconfig that you can place in a standard location.

But I will also implement this for real.


Fix the utils.prefixPointerEvent function

You're calculating char positions for "MSPointerDown", "MSPointerMove"... instead of "pointerdown", "pointermove" ect.

+1

Actually I've fixed that. Now the zoom works on Windows surface. There were also issues with the touch events.
Here the patch for that fix.
0001-Add-zooming-support-for-Microsoft-Surface.zip

Many thanks !


npm run add-dev <DISCORD_USER_ID> gives error

~/Bot$ npm run add-dev <DISCORD_USER_ID>

<EMAIL_ADDRESS>add-dev
node src/dev.js "<DISCORD_USER_ID>"

[ERROR] >> Developer Badge >> Failed to connect to the database!
MongooseError: The uri parameter to openUri() must be a string, got "undefined". Make sure the first parameter to mongoose.connect() or mongoose.createConnection() is a string.
at NativeConnection.Connection.openUri (/home/runner/Bot/node_modules/mongoose/lib/connection.js:695:11)
at /home/runner/Bot/node_modules/mongoose/lib/index.js:417:10
at /home/runner/Bot/node_modules/mongoose/lib/helpers/promiseOrCallback.js:41:5
at new Promise ()
at promiseOrCallback (/home/runner/Bot/node_modules/mongoose/lib/helpers/promiseOrCallback.js:40:10)
at Mongoose._promiseOrCallback (/home/runner/Bot/node_modules/mongoose/lib/index.js:1270:10)
at Mongoose.connect (/home/runner/Bot/node_modules/mongoose/lib/index.js:416:20)
at Object. (/home/runner/Bot/src/dev.js:21:10)
at Module._compile (node:internal/modules/cjs/loader:1099:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
I replaced <DISCORD_USER_ID>with my discord id but I keep my discord id as anonymous. Pls help @CorwinDev

Update your mongoose to ^6.8.0

@TheForgottenOne5982 be sure that you have set your mongo url in .env/secrets


Working my way through the demo

README driven development XD

Whoops, meant to propose this change to my own fork! Just ignore me XD


xml example

Codecov Report
Patch and project coverage have no change.

Comparison is base (396196d) 87.23% compared to head (12b762a) 87.23%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #90   +/-   ##
=========================================
  Coverage     87.23%   87.23%
  Complexity       16       16
=========================================
  Files             9        9
  Lines            47       47
=========================================
  Hits             41       41
  Misses            6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@rultor merge

@rultor merge

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

@rultor merge

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


Binding position and size seems to break in certain cases

In my app, the bug can appear without a window resize, when both widgets are created and binded together before adding them to the widget tree.
For this example, try to maximize the window 2 seconds after it appears, and you'll likely see this:

Tested on Windows.
from kivy.app import App
from kivy.lang import Builder
from kivy.core.window import Window
from kivy.factory import Factory
from kivy.uix.boxlayout import BoxLayout
from kivy.clock import Clock

kv = """
<WG@Widget>:
    col: 0.2, 0.2, 0.2, 1
    canvas:
        Color:
            rgba: self.col
        Rectangle:
            size: self.size
            pos: self.pos

<Test>:
    bg: bg
    Widget

    WG:
        id: bg
        col: 0.6, 0.6, 0.6, 1

    Widget
"""

Builder.load_string(kv)

class Test(BoxLayout):
    def __init__(self, *args, **kwargs):
        super(Test, self).__init__(*args, **kwargs)
        Clock.schedule_once(self.add, 2)

    def add(self, *args):
        fg = Factory.WG(size_hint=(None, None))
        self.bg.bind(size=fg.setter('size'))
        self.bg.bind(center=fg.setter('center'))
        fg.size = self.bg.size
        fg.center = self.bg.center
        Window.add_widget(fg)

class TestApp(App):
    def build(self):
        return Test()

if __name__ == '__main__':
    TestApp().run()

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/3962610-binding-position-and-size-seems-to-break-in-certain-cases?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

Fixed in master with sdl2.


refactor(transformer): remove unreachable code from TS namespace transform

#3475  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Add custom DNS settings to service definition

- What I did
This fix tries to fix the issue raised in #24391 about allowing custom DNS settings to service definition.
- How I did it
This fix adds DNS, DNSOptions, DNSSearch to service definition, as well as --dns, --dns-opt, and --dns-search to service create.
- How to verify it
An integration test has been added to cover the changes in this fix.
- Description for the changelog
Add --dns, --dns-opt, and dns-search to service create.
- A picture of a cute animal (not mandatory but encouraged)
This fix fixes #24391.
NOTE: A PR in swarmkit has been created separately:
https://github.com/docker/swarmkit/pull/1681

ping @aluzzardi @mrjana @dnephin PTAL

Design LGTM

Moving this back to design review because the UX for the flags for create and update need to be decided on.

Discussing this; we probably want this on docker service update as well, can you add that to this PR? (will be 6 flags; --dns-add , --dns-rm, etc.)

Thanks all for the review. The PR has been updated with DNS flags for service update. The docs has not been updated yet but will update the PR if we are OK with the implementation.

oh, needs a rebase already 😞

moving to code review

Thanks @thaJeztah The PR has been rebased and updated.

Ping @thaJeztah, any feedback for the PR?

ping @mrjana @sanimej

@yongtang does this still needs vendoring? Also, need rebase.

Thanks @LK4D4. The PR has been rebased and the vendoring has been done already.

@yongtang Need another rebase.
PR LGTM (not a maintainer).

@thaJeztah @yongtang  DNS configs of a running container/task can't be modified. Given that, I think we should remove the dns add/rm options for service update.

Thanks @sanimej. Let me take a look and remove related fields in service update.

@sanimej won't service update create new tasks? Or is it still an issue then?

I tried again. The service update recreated new tasks. Both <IP_ADDRESS> -> <IP_ADDRESS> and <IP_ADDRESS> -> <IP_ADDRESS> works.

@yongtang @thaJeztah If the service update creates recreates the tasks with the updated config then it shouldn't be an issue. Thanks for confirming.

@yongtang before I forget to ask; don't forget to document the API changes in this as well 😅

Thanks @thaJeztah. The PR has been updated with doc changes.

ping @aaronlehmann @mrjana can you review this?

Thanks for making the changes. LGTM

Design, Code and Docs look good.
LGTM
@yongtang can you please rebase?

LGTM

@yongtang I talked with @stevvooe, and it looks good to him after the flag is changed to --dns-options (instead of --dns-opt)

@yongtang As a follow up PR, we can add an alias to docker run / docker create, like we did for --network. i.e., for docker run, the "preferred" option should be --dns-options as well, but we keep --dns-opt (possibly hidden)

Thanks @aluzzardi @stevvooe @thaJeztah. The PR has been updated with --dns-options.

docs LGTM

Thanks so much @yongtang!

using 1.13 for windows, for some reason the container doesn't try to send query to my host's dns now. It won't resolve my private network dns.


AnimationTree blend position not  as expect

Godot version
master
System information
debian testing
Issue description
blend animation in AnimationTree (xfade, add, blendspace, etc.) not work as expect
https://user-images.githubusercontent.com/12198203/159103660-1b03029b-a5e1-469e-8e96-cff9442fb89a.mp4
Steps to reproduce

open project.
change blend value
see position change

Minimal reproduction project
animtree_blend_err.zip

@ywaby Which commit hash did you use? I think most of blend bug has been removed latest master, please test it.

OK, It's fixed.


Adding unique id to server config and passing to Kafka #279

See https://github.com/odpi/egeria/issues/279

Please validate this pull requests works with your Kafka setup.  One change that is important to note - the name of the topic in the connection endpoint was being ignored.  It should now use the proper value.  The connector is producing Audit log messages so you should see something like:
OCF-KAFKA-TOPIC-CONNECTOR-0001 Connecting to Apache Kafka Topic /open-metadata/repository-services/cohort/cohort1/OMRSTopic with a server identifier of fed729d6-c1d7-4270-8437-23c258d25017
The other change is to add the server Id to the key of each message and ignore any message with a key that matches the local server Id.  This is a performance enhancement.
Two questions:
Does the new configuration work in your environment?
Are all of the messages being transmitted correctly.

Server id is added in messages. Should this also be added in Member Registration class. Such that we can extract the server id of local server while persisting registry store?

@guptaneeru

Server id is added in messages. Should this also be added in Member Registration class. Such that we can extract the server id of local server while persisting registry store?

The simple answer is no - the server Id is not currently needed to persist the registry store.   The registry store relates to the membership of the cohort, not the config of the server.

Testing this fix revealed that the URL type structure for topic names that we have defined in OMRS and the OMASs do not work as Apache Kafka does not support "/" in a topic name.  So the topic names have been changed to this format: CP.open-metadata.access-services.AssetConsumer.inTopic

Worked through my GE omas scenario (configure event bus, enable access services, cohort, start instance, get classifications, add a few assets, get governed assets) - all worked fine.
Will review logs further and any additional notes, but so far behaviour seems the same with the PR as before.
my //current// config is
{
"producer":
{
"bootstrap.servers":"{{kafkaep}}",
"acks":"all",
"retries":"0",
"batch.size":"16384",
"linger.ms":"1",
"buffer.memory":"33554432",
"max.request.size":"10485760",
"key.serializer":"org.apache.kafka.common.serialization.StringSerializer",
"value.serializer":"org.apache.kafka.common.serialization.StringSerializer",
"kafka.omrs.topic.id":"{{cohort}}"
},
"consumer":
{
"bootstrap.servers":"{{kafkaep}}",
"zookeeper.session.timeout.ms":"400",
"zookeeper.sync.time.ms":"200",
"fetch.message.max.bytes":"10485760",
"max.partition.fetch.bytes":"10485760",
"key.deserializer":"org.apache.kafka.common.serialization.StringDeserializer",
"value.deserializer":"org.apache.kafka.common.serialization.StringDeserializer",
"kafka.omrs.topic.id":"{{cohort}}"
}
}

I was also able to run the same scenario with a simpler kafka config
{
"producer":
{
"bootstrap.servers":"{{kafkaep}}",
"kafka.omrs.topic.id":"{{cohort}}"
},
"consumer":
{
"bootstrap.servers":"{{kafkaep}}",
"kafka.omrs.topic.id":"{{cohort}}"
}
}

(Note those variables are defined by my postman environment - the endpoint URL, and a name - in my case the same as cohort name)

Where is the value "localhost:9092/OMRSTopic" coming from?  I think it is coming from the event bus URLRoot parameter ?

It is using  the endpoint.address as topic ConnectorConfigurationFactory.getDefaultEventBusConnection
which is concatenated with topicURLRoot
if (topicURLRoot == null)
{
endpoint.setAddress(topicName);
}
else
{
endpoint.setAddress(topicURLRoot + "." + topicName);
}

If you look at the code above, the URLRoot and the topic name are connected with a ".".  This is the code I see in master.   The URL root is passed (or should be passed) as a request param on the event bus command - it is not passed in the properties from the request body.  In the original post, the topic name is coming out as "localhost:9092/OMRSTopic.open-metadata.repository-services.cohort.cohort1.OMRSTopic". This suggests to me that the URLRoot is set to "localhost:9092/OMRSTopic" which is an invalid value.  What is the URLRoot passed on the event bus command in your scenario?  I think this is a case of passing incorrect parameters on the configuration call but I want to be sure.

Indeed, bad value for topicURLRoot query parameter in OMAGServerAdminResource.
However, we need validations for parameters, and return bad request if they are not compatible.
I have created a bug for it:
https://github.com/odpi/egeria/issues/302

Maybe rename parameter name from topicURLRoot to topicRoot or topicPrefix


Load CSV data for same table name (different structure) in different schema

Description
We have a need to load the database via CSV for the same table name (with different structure) for different schema.  When I am trying to use the CSV file name as Schema_TableName.csv but it does not work.  Sometimes, it tries to load the data intended for one schema into another and causes error.

Hello @midway500 ,
Do you think you could provide a project sample with this issue?
It will help my developer investigate the issue more efficiently.
We now always ask for a project sample even if the issue is easy to reproduce. As a free product, we must find some way to save time to offer an overall better experience for everyone and release fixes faster.
You can send it to<EMAIL_ADDRESS>if you need to keep the source private
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Will provide a project sample.

Hello @midway500 ,
I believe we never got the project.
We will close the issue but feel free to provide the project anytime and we will re-open it.

Please re-open the ticket.
I am sending the a sample project to<EMAIL_ADDRESS>Go to the Sample.Effort.UnitTest project and run the TestMethod in UnitTest1.  The test will fail with error:

Message: Test method Sample.Effort.UnitTest.UnitTest1.TestMethod1 threw exception:
Effort.Exceptions.EffortException: Unhandled exception while trying to initialize the content of 'Table<dbo_Table1>' table ---> Effort.Exceptions.EffortException: An unhandled exception occurred while trying to assign value '1/1/2000 12:00:00 AM' to Property 'Column1' of type 'System.Int32' during entity initialization for table 'Effort.Internal.DbManagement.Schema.TableName' ---> System.InvalidCastException: Specified cast is not valid.

Hello @midway500 ,
Sorry for the delay, my developer finally got time to look at it.
Could you try the v2.2.4 and let us know if everything works as expected in this version?
Best Regards,
Jonathan

Will give it a try.

Version 2.2.4 corrects my issue.  Thank you so much.


Updates webcrypto and karma dependencies

Description
Updates the versions of @trust/webcrypto and karma to avoid high/medium dependency vulnerabilities.
Noticeable dependency updates:

webcrypto:

Updated base64url dependency to 3.0.0 with the release of<EMAIL_ADDRESS>

karma:

Updated log4js to remove npm audit warnings here. Original report here.

Awesome, thanks for the contribution! Let me pull this down to test. I'll set up CI so this process is faster in the future.


Explore Directed Acyclical Graph (DAG) creation using  Workflow Orchestration Packages

Explore the feasibility of adding DAG creation to GraphVIPER.graph_tools for each of the following WOPs:

Airflow
Luigi
Prefect (e.g., concept docs, deployment pathways, use case recipes)
Beam
Argo
Dask + Prefect (e.g., task runners for multiple workflows)
Dask + Coiled (e.g., use case demos)

@dklyons81 I have updated graphviper by separating the dask code and added the airflow workflow creation code:
src/graphviper/graph_tools/generate_dask_workflow.py
src/graphviper/graph_tools/generate_airflow_workflow.py
docs/airflow_tutorial.ipynb

The viper_graph format is (the input to generate functions):
'map' :
'node_task': python_callable
'input_params': list of dicts (len equal to the number of mapping nodes)
'reduce' (optional):
'mode': 'tree'/'single_node'
'node_task': python_callable
'input_params': dict


TypeError: int() argument must be a string, a bytes-like object or a real number, not 'NoneType'

$ inventree-part-import NUD3124LT1GOSTR-ND
(only 3 of 5 available supplier modules are configured)
warning: category 'Electronics - uncategorized' on host is not defined in categories.yaml
warning: category 'Furniture' on host is not defined in categories.yaml
successfully setup categories!

searching for NUD3124LT1GOSTR-ND ...
(no results at TME)
(no results at Mouser)
Traceback (most recent call last):4LT1GOSTR-ND ...
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree/base.py", line 51, in __init__
    pk = int(pk)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/.local/bin/inventree-part-import", line 8, in <module>
    sys.exit(inventree_part_import())
  File "/home/h/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree_part_import/cli.py", line 21, in wrapper
    func(*args, **kwargs)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree_part_import/cli.py", line 171, in inventree_part_import
    last_import_result = importer.import_part(part, supplier, only_supplier)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree_part_import/part_importer.py", line 76, in import_part
    import_result |= self.import_supplier_part(supplier, api_part)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree_part_import/part_importer.py", line 116, in import_supplier_part
    manufacturer_part = ManufacturerPart(self.api, supplier_part.manufacturer_part)
  File "/home/hb/.local/pipx/venvs/inventree-part-import/lib/python3.10/site-packages/inventree/base.py", line 53, in __init__
    raise TypeError(f"Supplied <pk> value ({pk}) for {self.__class__} is invalid.")
TypeError: Supplied <pk> value (None) for <class 'inventree.company.ManufacturerPart'> is invalid.```

This is an edge case I didn't think was possible. Should be fixed on master, would be great if you could give it a try ^^

Test working, though it should be noted that I've since ensured my DB has a manufacturer part for all parts, which was not the case at the time of the error.
Closing issue at any rate...


[CLI] rancher apps -o json doesn't produce valid JSON

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Install two apps.
Run 'rancher apps -o json'

Result:
Output starts:
{"ID":"p-c8sbj:phpbb","App":{"id":...
This should surely be an array of objects, so I'd expect it to be:
[{"ID":"p-c8sbj:phpbb","App":{"id":...
Not having an array makes it hard to process in a CI environment.
Environment information

Rancher version: 2.2.3
Installation option: Single install
CLI version: 2.2.0

This is not a bug. @ap1969 it is valid JSON, it's called line delimited (or streaming) JSON. https://en.wikipedia.org/wiki/JSON_streaming
There are several tools including jq that support and understand how to parse it.


BUG: environment variables don't work in terminal

name: Bug report
about: Create a report to help us improve
title: ''
labels: ''
assignees: ''

Describe the bug
Type in terminal:
/projects/nextjs-hde56y
❯ echo $CHROME_PATH
Cannot read property 'replace' of undefined

/projects/nextjs-hde56y
❯ echo $ASDWQE
Cannot read property 'replace' of undefined

/projects/nextjs-hde56y
❯ echo $HOME
Cannot read property 'replace' of undefined

/projects/nextjs-hde56y
❯ echho $PATHH
Cannot read property 'replace' of undefined

/projects/nextjs-hde56y
❯ echho $PATH
Cannot read property 'replace' of undefined

Version of webcontainer
Hash: 7286dbe8d47d3473cde7be488b7d048fb59b05f2
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 90.0.4430.212
Major version = 90
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.212 Safari/537.36
performance.memory = {
totalJSHeapSize: 84237848,
usedJSHeapSize: 79113420,
jsHeapSizeLimit:<PHONE_NUMBER>
}

Hi @connorjclark! You are right, this does not work right now. We have the env command where you can see the environment variables. Setting environment variables work though, like HELLO=world env will show you that HELLO is set.
As in the other issue for the time command, we are planning to add more features to the shell implementation, as well as this one.

FWIW we've haphazardly fixed the spurious errors like Cannot read property 'replace' of undefined.

Support for shell variables has now been added to webcontainer - you should be able to run things like echo $HOME and echo $PATH and have it work!


Fix secondsSinceLastSwitch param in networkInfo JSON object

Task/Issue URL: https://app.asana.com/0/1199371158290272/1201752109049663/f
Description
The secondsSinceLastSwitch JSON field in the networkInfo object was not sent properly. This PR fixes it.
The PR changes the shared preferences key to ensure a proper migration.
Steps to test this PR

filter logcat by New network info

Test fresh install

[ ] fresh install from this branch
[ ] enable AppTP
[ ] Verify currentNetwork is correct and that secondsSinceLastSwitch is set to 0
[ ] Force-stop the app and re-launch it
[ ] Verify currentNetwork is correct and that secondsSinceLastSwitch is set to ~ amount of seconds that passed
[ ] Switch network (WIFI -> CELLULAR or CELLULAR -> WIFI)
[ ] Verify currentNetwork and previousNetwork are correct and secondsSinceLastSwitch is set to 0
[ ] Force-stop the app and re-launch it
[ ] Verify currentNetwork is correct and that secondsSinceLastSwitch is set to ~ amount of seconds that passed
[ ] Toggle airplane mode ON
[ ] Verify currentNetwork is set to NO_NETWORK and previousNetwork is correct and secondsSinceLastSwitch is set to 0
[ ] Toggle airplane mode OFF
[ ] Verify currentNetwork is correct, previousNetwork is set to NO_NETWORK and secondsSinceLastSwitch is set to 0

Test app update

[ ] Fresh install from develop
[ ] Enable AppTP
[ ] Verify New network info message appears
[ ] Switch network
[ ] Verify New network info message appears
[ ] Update app from this branch
[ ] Verify the fresh install test above also passes

Current dependencies on/for this PR:

develop

PR #1722   (this PR)

This comment was auto-generated by Graphite.


GC process may be skipping deletion of some unreferenced files.

I have found "old" files on a system that are not referenced in the metadata table that are not being removed by the default GC process.

I have found "old" files on a system that are not referenced in the metadata table that are not being removed by the default GC process.  So far I have found two common "types" that are not being removed.
a) Files in a bulk import directory - this may be from a failed ingest, but the failure would have had to of occurred after the files were moved / renamed - they have the expected directory structure and file naming convention.   Because of the age of the files, I need to find more recent occurrences where information for the import would be available in the logs to help determine what may have occurred with the ingest.
b) Files that seem to be from a failed / interrupted compaction - the files are the .tmp_rf files that are used during the compaction process.  There were no running compactions, and running a compaction over the range of the file completed.  It did not appear that this was enough to trigger the file being gc'ed (but I may have been impatient with the check and need to resist if the file has since been removed) However, the file "should" have been removed long before I took any action.
The may be a few other files that don't fit these criteria, and more checking is required to see if there are recent occurrences with tserver and master log files available for the times of the file creation to narrow the scope.

This isn't a bug. This is the intended design. The original garbage collector used to crawl all of HDFS, look for referenced files, and delete everything referenced. This was unsafe in the case of failure (some files were too aggressively deleted prior to a reference being added), and a big burden on the name node.
The new garbage collector tries to only delete things that have been explicitly identified as a candidate for deletion, and are provably safe to delete. It errs on the side of leaving things behind, rather than deleting them. Of course, this means that system administrators need to watch their clouds, especially in the case of failures, for anything left behind unreferenced, but that is an intentional trade-off.

Understood - I didn't realize this was intended - I'm working on tooling that would help in identifying these "unreferenced" candidates - we can discuss the best mechanism that would make the utility available to the wider community if that is desired.

Removed bug and making this an enhancement instead. It seems that the best course of action will be to provide a mechanism for these files to be discovered / reported to administrators.

No activity in over 3 years so closing out, can be reopened if still an issue or planned to be worked.

Also related: as similar issue: #1006


Redux type def

We'll need type definitions in order to avoid these state not exists on type DefaultRootState errors on the UI screens.

I have discussed this with @cakesoft-shashank. We are skipping this for now

This is done in keeper with redux-toolkit

Fixed cc @cakesoft-shashank


Trojan reported on Windows 10 (Trojan:Win32/Wacatac.G!ml)

Describe the bug
Windows reports Tauri app .exe as a threat, detecting Trojan:Win32/Wacatac.G!ml
More info
To Reproduce
Create a basic sample app with create-react-app and follow the tauri guide :

npx create-react-app myapp
cd myapp
yarn add tauri
add "tauri": "tauri" to package.json
yarn tauri init
yarn build
yarn tauri build
copy .exe file from target/release to another folder (without WebView2Loader.dll)

-> If I execute the app, I usually get an error because WebView2Loader.dll cannot be found, which is expected, but sometimes Windows Defender blocks and quarantines the app. I have not had this behaviour without moving the .exe file, but as it seems to happen randomly, I don't know if it related.
Expected behavior
No Trojan/virus warning
Screenshots

Platform and Versions (please complete the following information):
OS: Windows_NT(10.0.18363) - win32/x64
Node: 14.15.4
NPM: 6.14.10
Yarn: 1.22.10
Rustc: 1.49.0 (e1884a8e3 2020-12-29)
Additional context
Similar to #1060

hmm. It seems like this is a report from an atypical install approach. i would like to request that you build a proper MSI and install that. would you mind?

Hi, I rebuilt the app, installed it with msi, and run it with no warning.
As the problem seemed to happen randomly, I uninstalled it and tried again, with the exact same msi file.
When running the app after the second installation, Windows Defender blocket it, showing this treat: Behavior:Win32/Execution.IF!ml

Hmm. I am wondering if we have to sign artefacts now for windows. I am not a windows expert, unfortunately. Perhaps you could attach the file here (or send it via wetransfer) and then we could run some analysis. I am a bit surprised though that we don't get this report all the time...

Please join our discord and ping me there so we can triage properly

Thanks, this is likely due to the DLL problems faced by webview/webview. Will be resolved when WRY lands with the next release.


Created .htaccess to route to 404.html

Description
This PR fixes #48
Created .htaccess to route to 404.html

Good job


Ghost image shown when dragging an element with a modal box contained inside

Dragging an element with a modal box button and class modal in it, shows a draggable ghost image that contains all draggable elements.
The following is Svelte code for such a modal, but it could be any HTML code.

<slot name="button">Remove</slot>

<input type="checkbox" {id} class="modal-toggle" />

<slot name="title">Remove</slot>

<slot>Are you sure you want to remove?</slot>

			 {
					removeFcn();
				}}
			>
				<slot name="action">Remove</slot>

				<slot name="cancel">Cancel</slot>

I don't understand... this is the expected behavior of browsers, whatever element you drag, you can see them as draggable ghost in case you want to drop them somewhere else.


Angular CLI App doesn't run after CLI update.

Please provide us with the following information:

OS?
linux 4.4.0-62-generic
Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

$ ng --version

                             _                           _  _
  __ _  _ __    __ _  _   _ | |  __ _  _ __         ___ | |(_)
 / _` || '_ \  / _` || | | || | / _` || '__|_____  / __|| || |
| (_| || | | || (_| || |_| || || (_| || |  |_____|| (__ | || |
 \__,_||_| |_| \__, | \__,_||_| \__,_||_|          \___||_||_|
               |___/
@angular/cli: 1.0.0-beta.31
node: 7.4.0
os: linux x64
@angular/common: 2.4.7
@angular/compiler: 2.4.7
@angular/core: 2.4.7
@angular/forms: 2.4.7
@angular/http: 2.4.7
@angular/platform-browser: 2.4.7
@angular/platform-browser-dynamic: 2.4.7
@angular/router: 3.4.7
@angular/compiler-cli: 2.4.7

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

Old app created using CLI doesn't start after CLI update and Angular update to v2.4.7
npm update --save
You can clone this sample-repo and update the app under folder input-output.
I've followed the steps on upgrading from the CLI site.
The log given by the failure.

Normally this include a stack trace and some more information.

input-output: master$ ng s
Cannot find module '@angular/compiler-cli'
Error: Cannot find module '@angular/compiler-cli'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/node_modules/@ngtools/webpack/src/plugin.js:5:24)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/node_modules/@ngtools/webpack/src/index.js:5:10)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/models/webpack-configs/typescript.js:4:19)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/models/webpack-configs/index.js:9:10)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/models/webpack-config.js:4:27)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/tasks/serve.js:10:26)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Class.run (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/commands/serve.js:74:27)
    at Class.<anonymous> (/home/yadav/.nvm/versions/node/v7.4.0/lib/node_modules/@angular/cli/ember-cli/lib/models/command.js:134:17)
    at process._tickCallback (internal/process/next_tick.js:103:7)

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

i also face this issue try to install npm i --save @angular/compiler-cli and it works for me

@rajinder-yadav If I understand correctly your app was created with a previous version of angular-cli, and now you've updated the global angular-cli version.
This issue happened to me, I fixed it by updating my local version of the cli:
rm -rf node_modules dist # use rmdir on Windows
npm install --save-dev @angular/cli@latest
npm install
ng update
Good luck.

I have also problems on app build with previous version !
I try to do :
rm -rf node_modules dist # use rmdir on Windows
npm install --save-dev @angular/cli@latest
npm install
ng update

I am using Windows 10 (32bits)
node -v = v7.5.0
npm -v = 4.2.0
ng -v =
__ _  _ __    __ _  _   _ | |  __ _  _ __         ___ | |(_)
/ _` || '_ \  / _` || | | || | / _` || '__|_____  / __|| || |
| (_| || | | || (_| || |_| || || (_| || |  |_____|| (__ | || |
\__,_||_| |_| \__, | \__,_||_| \__,_||_|          \___||_||_|
              |___/
angular-cli: 1.0.0-beta.26
node: 7.5.0
os: win32 ia32
@angular/common: 2.4.7
@angular/compiler: 2.4.7
@angular/core: 2.4.7
@angular/forms: 2.4.7
@angular/http: 2.4.7
@angular/platform-browser: 2.4.7
@angular/platform-browser-dynamic: 2.4.7
@angular/router: 3.4.7
@angular/cli: 1.0.0-beta.31
@angular/compiler-cli: 2.4.7

when i type ng serve :
You have to be inside an angular-cli project in order to use the serve command.
ng update :
The specified command update is invalid. For available options, see ng help.

@bvaissade make sure you dont have angular-cli in devDepenecnies in your package.json, there should be only @angular/cli and then reinstall all packages with:
rm -rf node_modules
npm install

OK I did the steps today on two older projects cli project and things are working fine.
My initial issue has been resolved, I may have pooched something along the way last night!

Closing as resolved.

thanks @mikeeus you saved me


Sockets leak

A socket opened in OrientSocket.__init__ method is replaced for nothing with a new one in OrientSocket.connect method. So the socket opened in OrientSocket.__init__ is wasted and remains unclosed.
Moreover, in OrientDB class there is no way to close underlying OrientSocket (it is only closed in case of some error).

Created the fix for this issue:
https://github.com/mogui/pyorient/pull/234

I'm excited by this fix. This could potentially fix all my OrientDB issues. I'm using pyorient with celery and rabbitmq. We could not run celery with more than 1 worker because for some reason OrientDB would go ape-shit if we did and start locking the records and not release them. I initially suspected celery was not closing the connections and flooding OrientDB's connection pool.
We found that in distributed mode, when using celery, OrientDB would struggle with concurrency.
Could this fix point to the illusive double connection we've been experiencing?

@anber500, the PR #234 eliminates the redundant double connection which leaded to sockets leak. Also close method is added to OrientDB class. It should be called when the OrientDB class instance is not needed anymore in order to close the underlying socket.

Create a PR with the same fix as in #234 but for master branch:
#235

@TropicalPenguin, when is it planned to merge recent changes from develop to master and release the next version?

I'd imagine fairly promptly after the following unit tests are passing again for the develop branch:
tests.test_Factory_and_usage.CommandTestCase
tests.test_new_Iface.CommandTestCase
tests.test_raw_messages_3.RawMessages_5_TestCase

Unfortunately I'm not familiar with the area of the code covered by these. I suspect if anyone is to fix them, it would then be a trivial decision for @mogui to merge to master and do a new release.

Hi all,
i'm checking for tests failures, these failures could be related with some internal changes in OrientDB and the tests must be changed accordingly.
@anber500 as @spivachuk said this only remove a redundant initialization. Probably you have to close the no more used sockets by yourself.
Tnx @spivachuk for your PR, the close method will be useful.

The failures

Merged


feat: added GitLab secret detection report parser

Added a feature to accept GitLab secret detection reports
Ref: https://docs.gitlab.com/ee/user/application_security/secret_detection/

Hi @natebwangsut this is a great start! Please follow our how to guide  to make it perfect. There has been a lot of changes around creating parsers recently!

@damiencarol let me know what do you think

@Maffooch could you take another look?

@damiencarol @Maffooch any update ?

@Maffooch do you have more things to say for this PR? The contributor fixed your comment.


Window & Dynamic Size List Virtualization Causes Mobile Safari IOS Scroll Momentum Interruption

Describe the bug
By using the useWindowVirtualizer hook to apply virtualization for list items with different heights in relation to window scrolling, when you scroll, the virtualization behavior will occur.
Items that no longer need to be rendered will disappear from the DOM, and the list items corresponding to the visible indices will begin to appear in the DOM.
On Mobile iOS Safari, momentum scrolling triggered by user gestures works, but when combined with virtualization, the momentum effect fails to end smoothly and instead stutters or breaks abruptly. Is there a way to resolve this issue?
import "./styles.css";
import { useWindowVirtualizer } from "@tanstack/react-virtual";

const createRandomHeights = (
  min: number,
  max: number,
  count: number = 1000
) => {
  const heights = [];
  for (let i = 0; i < count; i++) {
    heights.push(Math.floor(Math.random() * (max - min + 1) + min));
  }
  return heights;
};

const randomHeights = createRandomHeights(100, 400);
const colors = ["red", "yellow", "blue"];

export default function App() {
  const virtualizer = useWindowVirtualizer({
    count: 1000,
    estimateSize: () => 400,
    overscan: 5,
    scrollMargin: 0,
  });

  const virtualItems = virtualizer.getVirtualItems();

  return (

      <div
        style={{
          height: `${virtualizer.getTotalSize()}px`,
          position: "relative",
          width: "100%",
        }}
      >

          {virtualItems.map((virtualItem) => {
            return (

            );
          })}

  );
}

Your minimal, reproducible example
https://codesandbox.io/p/sandbox/qhkvyz
Steps to reproduce

Go to CodeSandbox Link
Scroll down Preview Page
As you scroll, the momentum scroll stops occurring and the scrolling becomes increasingly choppy.

Expected behavior
I hope to maintain a seamless momentum scroll experience.
How often does this bug happen?
Every time
Screenshots or Videos
https://github.com/user-attachments/assets/2b27fcd6-e3de-449b-ac20-2633d76619d3
https://github.com/user-attachments/assets/166be2b0-cab1-495a-a9e1-9b770e31c1fc
Platform

OS: IOS
Browser: Safari
Version: 18.1, 17.6.1

tanstack-virtual version
v3.10.6
TypeScript version
v5.4.5
Additional context
It seems that the CodeSandbox link is difficult to run on Mobile iOS Safari at the moment. However, you can reproduce the issue by running the attached source code.
Terms & Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I'm no expert here, so take my findings with a pinch of salt
When an element is first "mounted" the size is measured, if the element is a different size from expected it will adjust the scroll position to hide this. Safari appears to only preserve a momentum scroll when doing this if it is the same direction, e.g. if currently scrolling down and I set the scroll to an earlier position, it will cancel it.
The result is that if scrolling down when seeing an element for the first time, the estimated size needs to be larger than elements actually are. This is called out in the docs - https://tanstack.com/virtual/latest/docs/api/virtualizer#estimatesize

If you are dynamically measuring your elements, it's recommended to estimate the largest possible size (width/height, within comfort) of your items. This will ensure features like smooth-scrolling will have a better chance at working correctly

The key for me was that I was implementing a message view which scrolls up by default, this flips the logic round and instead the estimate needs to be smaller than possible. Then when the user scrolls back down it will have already measured the elements and everything will work "correctly".


[Bug] The engine was not killed due to the operation timeout

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
Due to the operation timeout, the session close failed, resulting in the engine not being killed.
Affects Version(s)
1.4.0
Kyuubi Server Log Output
2022-04-24 08:11:51,888 WARN  [KyuubiSessionManager-timeout-checker: Thread-8
7]: org.apache.kyuubi.operation.KyuubiOperationManager(60) - Operation Operat
ionHandle [type=UNKNOWN_OPERATION, identifier: e3d47534-072b-4970-952b-3929e7
107259] is timed-out and will be closed

2022-04-24 10:30:06,699 ERROR [KyuubiThriftBinaryFrontendServiceHandler-Pool:
 Thread-2060109]: org.apache.kyuubi.server.KyuubiThriftBinaryFrontendService(
72) - Error closing session:
org.apache.kyuubi.KyuubiSQLException: Invalid OperationHandle [type=UNKNOWN_O
PERATION, identifier: e3d47534-072b-4970-952b-3929e7107259]
        at org.apache.kyuubi.KyuubiSQLException$.apply(KyuubiSQLException.sca
la:69)
        at org.apache.kyuubi.operation.OperationManager.removeOperation(Opera
tionManager.scala:88)
        at org.apache.kyuubi.operation.OperationManager.closeOperation(Operat
ionManager.scala:103)
        at org.apache.kyuubi.session.AbstractSession.$anonfun$closeOperation$
1(AbstractSession.scala:180)
        at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java
:23)
        at org.apache.kyuubi.session.AbstractSession.withAcquireRelease(Abstr
actSession.scala:75)
        at org.apache.kyuubi.session.AbstractSession.closeOperation(AbstractS
ession.scala:178)
        at org.apache.kyuubi.session.KyuubiSessionImpl.close(KyuubiSessionImp
l.scala:143)
        at org.apache.kyuubi.session.SessionManager.closeSession(SessionManag
er.scala:93)
        at org.apache.kyuubi.service.AbstractBackendService.closeSession(Abst
ractBackendService.scala:49)
        at org.apache.kyuubi.service.ThriftBinaryFrontendService.CloseSession
(ThriftBinaryFrontendService.scala:227)
        at org.apache.hive.service.rpc.thrift.TCLIService$Processor$CloseSess
ion.getResult(TCLIService.java:1397)
        at org.apache.hive.service.rpc.thrift.TCLIService$Processor$CloseSess
ion.getResult(TCLIService.java:1382)
        at org.apache.thrift.ProcessFunction.process(ProcessFunction.java:39)
        at org.apache.thrift.TBaseProcessor.process(TBaseProcessor.java:39)
        at org.apache.kyuubi.service.authentication.HadoopThriftAuthBridgeSer
ver$TUGIAssumingProcessor.process(HadoopThriftAuthBridgeServer.scala:162)
        at org.apache.thrift.server.TThreadPoolServer$WorkerProcess.run(TThre
adPoolServer.java:286)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecut
or.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecu
tor.java:617)
        at java.lang.Thread.run(Thread.java:745)

Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

this should be fixed by https://github.com/apache/incubator-kyuubi/pull/1983

this should be fixed by #1983

Thanks, it seems so.

update lastAccessTime in getStatus

This change seems necessary, can you help me review #2452?


server functions are a bit restrictive

I'd like to be able to create a regular REST api using axum and derive the client side actions/resources. I realize I can do this manually using create_local_resource which I'll probably do for now, but would be nice to also integrate with ssr somehow.

There's nothing stopping you from doing this, is there? You can just create a REST API use it. If you use create_resource, you'll even get streaming data from the server and suspense support. In fact, server functions are just an abstraction over a REST API.

ok thanks


androidx.credentials.TYPE_SECURITY_ERROR when using example app

Hey there, so I'm running in to an issue when running the example passkey native app in the repo on an android simulator. This is also the same as the error returned in my own application. Steps to reproduce:

From packages/passkeys/passkeys directory, run melos run example-passkeys-native as instructed in the README
Type in email address in the email address section
click sign up

Returns error after a moment saying TYPE_SECURITY_ERROR The incoming request cannot be validated.
I have not modified any parts of the example application.

This happens both on physical device and on emulator

Having exactly the same issue on Android device.
OnePlus9, Android 14

@briancorbin @eimjustas
Hey guys, thanks for bringing this up. We fixed it now => examples should work again now.
In case you are interested about the root cause of the error:
There was a misconfiguration on our side. The relying party ID got changed from flutter.corbado.io to corbado.io. This meant that Android (but also iOS devices) could no longer fetch assetlinks.json (or apple-app-site-association) files and the OS (in that case Android throws an exception).


Bash script mode

Hi,
Simple question about style guide for Bash:
Is there a reason not to recommend what’s usually called “Bash strict mode”? (I’d call it “Bash script mode” myself)
Which is the fact of writing any Bash script with those four settings (declared in the top of the file):
set -euo pipefail
IFS=$'\n\t'

It implies to change a few habits one may have, but I find this very useful once used to.
A detailed explanation: http://redsymbol.net/articles/unofficial-bash-strict-mode/

we shouldn't encourage people to think turning on set -e means they don't need to add error checking
https://mywiki.wooledge.org/BashFAQ/105
set -u seems a matter of taste. I don't know that it catches bugs that really matter. for typod vars, shellcheck is better imo, especially because it is a static analyzer.
pipefail can be a little insidious. consider tools that exit early like grep -q -- if you were piping a large file to it, it could trigger a write to a closed ("broken") pipe, but not consistently. this is a fun bug to track down.
might be worthwhile adding an appendix on these options and why we don't require them all the time.

we shouldn't encourage people to think turning on set -e means they don't need to add error checking

The errexit option (set -e) clearly doesn’t prevent the need for proper error checking. I would even say it forces people not letting some silent errors here and there. I do not see how it could represent such an encouragement.
More I think about it, not exiting on error only makes sens for an interactive use of Bash, but not for a script. But it is indeed probably not always a good idea to set this option when the script is used. But during the development phase at least it highly useful.
You’re right for the pipefail option. In my opinion it’s not as important as the errexit and nounset options.
The nounset (set -u), imho, forces user not to let “dead” code that could make the code harder to (re)read, and to always choose default value for any variable.
Thank you for your answers.

we shouldn't encourage people to think turning on set -e means they don't need to add error checking

The errexit option (set -e) clearly doesn’t prevent the need for proper error checking. I would even say it forces people not letting some silent errors here and there. I do not see how it could represent such an encouragement.
More I think about it, not exiting on error only makes sens for an interactive use of Bash, but not for a script. But it is indeed probably not always a good idea to set this option when the script is used. But during the development phase at least it is highly useful imo.
You’re right for the pipefail option. I don’t think it is as important as the errexit and nounset options.
The nounset (set -u), imho, forces user not to let “dead” code that could make the code harder to (re)read, and to always choose default value for any variable.
Thank you for your answers.


BUG: xxx.log is close

When we open the same log file when we sealos run at the same time, and when an app prioritises the end, the file will be closed, causing subsequent apps to fail to read logs.

Origin issue related: #1293

Closed dur to # #1300


Compare personas

It is not plugged into data access or app builder because missing latest version of gptDAO on my branch.

CA is respected, and as you said the only thing left is to link it.
Well done!


Level 1>Backup on GoogleDrive button not working

Level 1>Backup on GoogleDrive button not working after skip automated cloud backup on "New Hexa Wallet" screen.

-Verified this issue on staging app v1.6.1(279)


[Bug]: Regression in the copy_to_directory rule on Windows

What happened?
❯ bazel build //appinventor/appengine:MakeWar
INFO: Analyzed target //appinventor/appengine:MakeWar (1 packages loaded, 656 targets configured).
INFO: Found 1 target...
ERROR: D:/github/pavi2410/appinventor/appinventor/appengine/BUILD.bazel:179:18: Copying files to directory failed: (Exit 1): copy_to_directory.exe failed: error executing command (from target //appinventor/appengine:MakeWar) external\copy_to_directory_windows_amd64\copy_to_directory.exe bazel-out/x64_windows-fastbuild/bin/appinventor/appengine/MakeWar_config.json
2023/01/15 21:20:08 open bazel-out/x64_windows-fastbuild/bin/appinventor/appengine/MakeWar/extra/ode\soycReport\splitPoints21.xml.gz: The system cannot find the path specified.
Target //appinventor/appengine:MakeWar failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.662s, Critical Path: 0.18s
INFO: 3 processes: 3 internal.
FAILED: Build did NOT complete successfully

I'm using copy_to_directory rule to copy some build files to a different file under the bin directory. The path in the error above is the dest dir. Why does it attempt to open it? Where do I look for the problem?
Version
Development (host) and target OS/architectures: Windows 11 64-bit
Output of bazel --version: bazel 6.0.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:

1.19.2 working
1.20.0 not working
1.21.3 not working

Language(s) and/or frameworks involved:
Java, GWT, AppEngine
How to reproduce
https://github.com/pavi2410/appinventor-sources/blob/0a96526fe572e7e50e6382dcb5ce2da7b55168d6/appinventor/appengine/BUILD.bazel#L184-L203
Any other information?
No response
Fund our work

[ ] Sponsor our open source work by donating a bug bounty

Should be fixed in the next release by https://github.com/aspect-build/bazel-lib/commit/ce043b299df03f40e360cfe9ab37d89f3d552b56 and https://github.com/aspect-build/bazel-lib/commit/74caa5c09776303ea1d1ed7a9c74aecb6778127d.

However, I'm not able to confirm this particular case is fixed since I'm hitting a JDK toolchain issue on my Windows machine before being able to build this particular target:
ERROR: C:/users/gmago/_bazel_greg/ilojnvke/external/local_jdk/BUILD.bazel:2:10: in fail_rule rule @local_jdk//:jdk:
Traceback (most recent call last):
        File "C:/users/gmago/_bazel_greg/ilojnvke/external/bazel_tools/tools/jdk/fail_rule.bzl", line 19, column 13, in _fail_rule_impl
                fail("%s %s" % (ctx.attr.header, ctx.attr.message))
Error in fail: Auto-Configuration Error: Cannot find Java binary bin/java.exe in C:/users/gmago/_bazel_greg/install/c9ea1a9caee100e72d651d25da00df4d/embedded_tools/tools/jdk/nosystemjdk; either correct your JAVA_HOME, PATH or specify Java from remote repository (e.g. --java_runtime_version=remotejdk_11
ERROR: C:/users/gmago/_bazel_greg/ilojnvke/external/local_jdk/BUILD.bazel:2:10: Analysis of target '@local_jdk//:jdk' failed

@pavi2410 Could you please close this issue if the 1.23.3 release resolves this?

It's working fine again! Thanks for the quick fix!


v0.1.16 - fix deprecation warnings from CSV

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/20622
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.16 -m "<description of version>" cf303235c297b74a408b8da5033bc1739cfa1964
git push origin v0.1.16


Please provide EddystoneTLM

I just checked your examples which is https://github.com/sandeepmistry/arduino-BLEPeripheral/tree/master/examples/Eddystone
There are only UID and URL examples.
I would like to get TLM example.
Please make it for me.
Thanks.

@iamdenny it would be great if you had time to submit a pull request for this!
A few pointers:

https://github.com/sandeepmistry/arduino-BLEPeripheral/blob/master/EddystoneBeacon.cpp
https://github.com/google/eddystone/tree/master/eddystone-tlm

In EddystoneBeacon::loop(), a check would need to be added to switch the Eddystone data advertised from UID/URL to TLM at a regular interval. For example, current advertisement interval is 100ms, so TLM data could be interleaved every 10 packets or so.

@sandeepmistry  @iamdenny  Please check this and let me know if it helps.
https://github.com/arun271/arduino-BLEPeripheral/blob/eddystone/examples/Eddystone/EddystoneTLM
Thanks.


Fix handling of filter expressions in parquet pyarrow-dataset engine

On pyarrow master, we disallowed the "boolean evaluation" of Expression objects (see https://github.com/apache/arrow/pull/9352, because Python doesn't allow us to override the meaning of not expr or expr1 and expr2, those do not what one would expect from it. So better to raise an error similar to how numpy arrays raise).
The main consequence is that you explicitly need to check is not None to check if the expression has a value.

Looks good to me! Thanks for opening this @jorisvandenbossche.


Add some bulletproofing to deconstructComponentInternal.

Fixes issue with recent closure compiler versions.
@Garbee PTAL!

I'm not sure of any case where this would get called without a component. However should it occur we should throw an error since that is an exceptional case. Doing an if wrapper would just mask a problem instead of allowing it to be handled.
Then again, without a case where a problem is being caused it is hard to tell how a developer would handle it.
Let me think on this one some. I see where you are trying to go with just letting things continue to operate as normal in case of error.

Actually, I think you're right, @Garbee. I think we should just be updating the notation to indicate the object will always be present, instead. There's only one use of this method and it relies on a component existing.
Let me know what you decide and I'll make the change.

Considering if a component isn't even given then clearly there is nothing to splice from the array, no memory leak would occur from adding in the if check. So, I'd say merge since it doesn't really matter much at this point. We can deal with any fallout if it happens to come up later.
For 2.x I'd like us to try and apply a more rigid approach to components. Have expectations and throw errors when they aren't met. But, that is a discussion for after another time (probably next week after I have some time to explain how it helps in a design doc.)

Sweet, thanks @Garbee!


Widgets do not show on mobile device

Hi,
I can't get widgets to appear on mobile phone (Android 7.0) in any browser (Chrome, Firefox, Fully Kiosk Browser). The page looks like on the attached screenshot (screenshot1). On the desktop everything looks well.

Regards
Adam

call it absolut!
http(s)://[IP]:[PORT]/fhem/tablet/index.html
Do not use HTTPSRV!


Implement from_reader

This PR implements reader support for deser-hjson. People coming from other serde-based deserializers (serde_json, serde_yaml, etc.) expect this crate::from_reader(io::Read) function to exist.
Breaking API changes:

The Error enum got a new variant for representing I/O errors. This is a breaking change because the enum was not marked as non-exhaustive.
Had to drop the Clone and PartialEq derives from the Error enum because the new wrapped std::io::Error error does not implement these traits.

Thanks. Error implementing Clone and PartialEq made little sense anyway.
With the breaking change, I'll release that as 2.0.


[e2e]: delegator-lowperms create main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.11
Run: https://github.com/slsa-framework/example-package/actions/runs/5680941373
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.11
Date: Thu Jul 27 13:12:43 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.12
Run: https://github.com/slsa-framework/example-package/actions/runs/5684680455
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.12
Date: Thu Jul 27 19:35:59 UTC 2023
Tests are passing now. Closing this issue.