Install on LazyVim

Hello,
I found out about this plugin on [dotfyle.com(https://dotfyle.com/).
It looks like an amazing plugin. I wanted to try it out on my nvim setup, based on lazyvim.
I followed the docs and added the plugin:
return {
  {
    "cenk1cenk2/schema-companion.nvim",
    dependencies = {
      { "neovim/nvim-lspconfig" },
      { "nvim-lua/plenary.nvim" },
      { "nvim-telescope/telescope.nvim" },
    },
    config = function()
      require("schema-companion").setup({
        enable_telescope = true,
        formatting = true,
      })
    end,
  },
}

map("n", "<leader>kz", "<cmd>lua require('telescope').extensions.yaml_schema.select_from_matching_schemas()<CR>", { desc = "TEST" })

I tried to run the telescope yaml but I get an error:
 Error  14:01:45 msg_show.lua_error E5108: Error executing lua ...m/lazy/telescope.nvim/lua/telescope/_extensions/init.lua:10: 'yaml_schema' extension doesn't exist or isn't installed: module               'telescope._extensions.yaml_schema' not found:

Also i added the config option for the yaml-langauge-server but It seems nothing is working. Probably I am doing smth wrong.
Can you give any hints how to set this up for lazyvim?
Thank you

Dear @diskmanti,
Sorry for the little bit of a late answer, I have changed the plugin name since I do want to extend this to JSON if possible at some point.
So the telescope commands also changed a little bit.
map("n", "<leader>kz", "<cmd>lua require('telescope').extensions.schema_companion.select_from_matching_schemas()<CR>", { desc = "TEST" })

This would trigger only if there is a matching schema.
My yaml-language-server allows to look at the schema registry for matching schemas and something like below.
It is a bit abstracted so I will try to write it in the plain way which is as follows.
require("lspconfig").yamlls.setup(require("schema-companion").setup_client({
  filetypes = {
    "yaml",
    "!yaml.ansible",
    "!yaml.docker-compose",
  },
  settings = {
    flags = {
      debounce_text_changes = 50,
    },
    redhat = { telemetry = { enabled = false } },
    yaml = {
      hover = true,
      completion = true,
      validate = true,
      format = { enable = false },
      schemaStore = { enable = true, url = "https://www.schemastore.org/api/json/catalog.json" },
      schemaDownload = { enable = true },
      schemas = {
        -- kubernetes = {
        --   "templates/*!(.gitlab-ci).{yml,yaml}",
        --   "workloads/**/*!(kustomization).{yml,yaml}",
        --   "*.k8s.{yml,yaml}",
        --   "daemon.{yml,yaml}",
        --   "manager.{yml,yaml}",
        --   "restapi.{yml,yaml}",
        --   "*namespace*.{yml,yaml}",
        --   "role.{yml,yaml}",
        --   "role-binding.{yml,yaml}",
        --   "*onfigma*.{yml,yaml}",
        --   "*ingress*.{yml,yaml}",
        --   "*secret*.{yml,yaml}",
        --   "*deployment*.{yml,yaml}",
        --   "*service*.{yml,yaml}",
        --   "kubectl-edit*.yaml",
        -- },
        ["https://raw.githubusercontent.com/argoproj/argo-workflows/master/api/jsonschema/schema.json"] = { "*argocd*.{yml,yaml}" },
        ["http://json.schemastore.org/chart"] = { "Chart.{yml,yaml}" },
        ["https://gitlab.com/gitlab-org/gitlab/-/raw/master/app/assets/javascripts/editor/schema/ci.json"] = {
          ".gitlab-ci.yml",
        },
        ["https://json.schemastore.org/drone.json"] = { ".drone.yml" },
        ["https://raw.githubusercontent.com/ansible-community/schemas/main/f/ansible-playbook.json"] = {
          "deploy.yml",
          "provision.yml",
        },
        ["https://bitbucket.org/atlassianlabs/atlascode/raw/main/resources/schemas/pipelines-schema.json"] = {
          "bitbucket-pipelines.yml",
        },
        ["https://taskfile.dev/schema.json"] = {
          "Taskfile*.{yml,yaml}",
        },
        ["https://json.schemastore.org/pulumi.json"] = {
          "Pulumi.{yml,yaml}",
        },
        ["https://raw.githubusercontent.com/cenk1cenk2/docker-vizier/main/schema.json"] = {
          "vizier.{yml,yaml}",
        },
      },
    },
  },
}))

Where the plugin configuration for this is as follows.

require("schema-companion").setup({
        log_level = lvim.log.level,
        formatting = false,
        enable_telescope = true,
        -- Built in file matchers
        matchers = {
          require("schema-companion.matchers.kubernetes").setup({ version = "master" }),
        },
        schemas = {
          {
            name = "Kubernetes master",
            uri = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/master-standalone-strict/all.json",
          },
          {
            name = "Kubernetes v1.27",
            uri = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.27.16-standalone-strict/all.json",
          },
          {
            name = "Kubernetes v1.28",
            uri = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.28.12-standalone-strict/all.json",
          },
          {
            name = "Kubernetes v1.29",
            uri = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.29.7-standalone-strict/all.json",
          },
          {
            name = "Kubernetes v1.30",
            uri = "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.30.3-standalone-strict/all.json",
          },
          {
            name = "Gitlab CI",
            uri = "https://gitlab.com/gitlab-org/gitlab/-/raw/master/app/assets/javascripts/editor/schema/ci.json",
          },
        },
      })

I will try to update the documentation to reflect this at a convenient time.

hello @cenk1cenk2
Thank you for the feedback. Using your suggestions I was able to make the plugin work.
I am closing this issue


the procedure entry point _divmoddi4 could not be located in the dynamic link library D:\msys32\mingw32\bin\libglib-2.0-0.dll

hello
I have a problem when executing an application it tells me the above (in the title of the issue, in a dialogbox , and the application is terminated.

That's a problem in your GLib build. Please report this to where you got it from, which is probably msys?


Progress Tracker?

In this article, Nick Babich has written about vertical and horizontal progress trackers. Given the examples by material design, it'd be nice to implement as a new component.

Duplicate of #2414


ENG-1027: Redirect Temporal dev server log to file instead of stdout/stderr

Redo of #282, after #386

I was thinking we can just keep sending to stderr, but when the server goes down, close it for temporal.

I don't think that would work? The whole point of this change is to hide stdout and stderr instead of spamming the user's terminal with Temporal errors - which is at its worst when AK shuts down.

I was thinking we can just keep sending to stderr, but when the server goes down, close it for temporal.

I don't think that would work? The whole point of this change is to hide stdout and stderr instead of spamming the user's terminal with Temporal errors - which is at its worst when AK shuts down.

Yeah but we might not want to make ALL errors disappear, just those that happen when we close down the server. Anyhow, no biggy.


Add google-libphonenumber to Parsing

This adds google-libphonenumber, a library to parse, format  and validate international phone numbers.

All comments addressed.


Improved flow for uploading a photo

Aims to address #202, and also hopefully #201, #200, #203, #204.
Kind of hard to properly illustrate the flow with just screenshots, but ah well.
Before

After

If has photo, then:

else if no or not sure then:

If has camera, then

(Big green button on above screen routes to upload page)

else if no camera

After user uploads a photo, they see this screen:

Weirdness when clicking the "get help with uploading a file"

Loading page has the banner at the bottom about switching LA but none of the other pages do. Not sure if that was intentional or not?

clicking continue on http://localhost:3000/service-patterns/concessionary-travel/example-service/good-photo results in this error:

Fixed the Get help link
The council picker footer is on all the pages
The template not found error is already in master, it's a bug with the council picker. You need to refresh the page to get it to work.

@kaydale Ok that last thing should be properly fixed now.

[ ] Has this journey been researched yet? I'm wondering that when a user doesn't have a photo, do we need to add 'phone' or 'smartphone' whenever we talk about a 'digital camera'? Most users will have a phone that they can use to take a photo, but may not realise this as it sounds like they can only use a 'proper' digital camera.

[ ] When a user says they don't have a photo, the following 2 pages have the same H1, so 3 pages have the same H1. When it comes to accessibility, I don't think that's good practice, all H1s need to be different?

[ ] Change the H1 'You’ll need to get a digital photo from a shop' to 'You need to get a digital photo from a shop'

[ ] If a user goes through the non-Verify digital route, will they also see the 'You need to get a digital photo from a shop' page? If so, the Verify message on this page should be removed. Otherwise try this Verify message: 'You'll need to restart your application for an older person's bus pass. You do not need set up GOV.UK Verify again, just sign in with your certified company.'

[ ] Update 'If you need help doing this, you can contact the council.' to 'If you need help with getting a digital photo, contact Argleton County Council.'

This is being merged into the prototype, which is going into research. So all of these changes you're looking at are pre-research essentially.

I'll leave the point about the camera as it is for now and see how it tests.
I wasn't aware about the h1 point, makes sense though. What do you suggest for the h1s of those pages?

Cool, looking forward to seeing the research findings.
When the page has a question, the H1 can be the question, like on Register to vote: https://www.gov.uk/register-to-vote. So 'Do you have a digital camera' is the H1.
As for the other page, not sure what the H1 could be. Something about steps as there are 2 numbered steps on the page


Fix Falcon & MPT MatMul folding during ONNX export

Need this new pattern matching for Falcon/MPT during ONNX export to correctly fold MatMuls into MatMulInteger.
Also did some renaming for related funcs, hopefully making their intention clearly.
Qualification: manually tested with Falcon-7b pruned quantized.

Close the PR because it's been added as part of ONNXToDeepSparse


Sorting tags fails

Hi,
I noticed that sorting of tags fails.
If I lookup Circlator  and scroll down to the Docker Images section I can’t get the table sorted chronologically.
The following shows up as sorted by ‘Last Modified’, which it clearly isn't:
1.5.2--py35_0 | Wed, 25 Oct 2017 10:30:54 -0000 | 114.84851 MB
1.5.1--py36_0 | Tue, 09 May 2017 12:09:32 -0000 | 89.32478 MB
1.1.2--py35_0 | Wed, 07 Sep 2016 03:48:14 -0000 | 101.111517 MB
1.4.0--py35_0 | Wed, 02 Nov 2016 08:16:26 -0000 | 105.497783 MB
1.5.2--py36_0 | Wed, 25 Oct 2017 10:26:16 -0000 | 108.542889 MB

Over on quay circlator tags are sorted correctly.
Not sure if that plays any role but I'm on Firefox 47.0.1.
Cheers,
Rudiger

thanks @rbrauning for reporting this issue. We are working in a new version that would be released soon with this issue fixed. I will leave it open until is fixed, you will receive a notification when the issue is solve.


MySQL JSON column unsupported

Using:
MySQL: 5.7
(also tried MemSQL 5.8)
turbodbc: 2.7.0
Python: 3.5.2
Table definition:
Column     | Type
-----------+---------
data       | json

I'm able to write json data to the table using the execute method:
    cursor = conn.cursor()
    cursor.execute("INSERT INTO ExampleTable VALUES (?)", ['{"A": 1}'])

I then try to read the data I just wrote (after committing transaction and creating new cursor) :
cursor.execute("SELECT data FROM ExampleTable;")
And receive the following error:
RuntimeError: Error! Unsupported type identifier for column data @ NULLABLE UNKNOWN TYPE (precision<PHONE_NUMBER>3709551612, scale 0))

Hi! Thanks for the report. Could you try to cast the JSON field to VARCHAR as a workaround?

Hi! This works! Thanks for the suggestion.
It would be nice to have turbodbc support the JSON datatype explicitly to avoid such workarounds.

This works for me using the current master.
Using:
Driver: MySQL ODBC 8.0 Unicode Driver
turbodbc: 4.2.1
Python: 3.9.4
Table definition:
Column  | Type
--------+-----
data    | json (null)

>>> cursor.execute("select data from Table limit 1").fetchall()
[['{ my json data }']]


mktemp: fix PrefixContainsDirSeparator verification

I belive can resolve #4378 : ]

could you please add a test to make sure we don't regress in the future ?

could you please add a test to make sure we don't regress in the future ?

Well remembered, I'll do it as soon as I get back

@ZauJulio ping ? :)

@ZauJulio ping ? :)

Carnival is finally over here in Brazil, I'll upload the test and send it today : ]

@ZauJulio ping ? :)

done, I believe that is enough.
I loved contributing to the project, I'm going to start my studies in Rust and intend to continue here 🧡

There's a rustfmt issue, but otherwise it's good

show, I just uploaded the fmt correction

Hi @tertsdiepraam , I was wondering about the current state of PR, do you need anything else? If yes, how can I help you?


Add assertThat method in the list of recommended imports

Hmm, I don't remember what this one was about. Maybe it was was about
recommending "assertThat" if the test has no assertions or something? Do we
do it?
Maurício Aniche
Tech Academy Lead at Adyen
Assistant Professor in Software Engineering at TU Delft
Author of Effective Software Testing: A Developer's Guide
https://www.effective-software-testing.com
https://www.mauricioaniche.com
On Tue, Oct 11, 2022 at 11:29 AM Martin Mladenov @.***>
wrote:

This issue was moved here when we switched from Basecamp. It is related to
assignment configuration and not Andy itself, so I guess we can close it?
@mauricioaniche https://github.com/mauricioaniche
—
Reply to this email directly, view it on GitHub
https://github.com/cse1110/andy/issues/80#issuecomment-1274400625, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYTTEQI4XZ6NQK6ZPRLBDWCUXP7ANCNFSM5JU2PPQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@mauricioaniche

Hmm, I don't remember what this one was about. Maybe it was was about recommending "assertThat" if the test has no assertions or something? Do we do it?

I think it refers to the list of imports in the solution templates of exercises. We already do that in all (new) exercises.
We don't do anything special if a student's test has no assertions - we simply run tests as usual. We only have a code check that can detect that and fail if there are no assertions.

Then, let's close this one!
Maurício Aniche
Tech Academy Lead at Adyen
Assistant Professor in Software Engineering at TU Delft
Author of Effective Software Testing: A Developer's Guide
https://www.effective-software-testing.com
https://www.mauricioaniche.com
On Tue, Oct 11, 2022 at 11:46 AM Martin Mladenov @.***>
wrote:

@mauricioaniche https://github.com/mauricioaniche
Hmm, I don't remember what this one was about. Maybe it was was about
recommending "assertThat" if the test has no assertions or something? Do we
do it?
I think it refers to the list of imports in the solution templates of
exercises. We already do that in all (new) exercises.
We don't do anything special if a student's test has no assertions - we
simply run tests as usual. We only have a code check that can detect that
and fail if there are no assertions.
—
Reply to this email directly, view it on GitHub
https://github.com/cse1110/andy/issues/80#issuecomment-1274422961, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYTTFOQWQ2GQIGXYWG4WTWCUZPPANCNFSM5JU2PPQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add additional props passing method

This solution is an improvement over the createElement method in that it passes eslint-plugin-react problem https://github.com/yannickcr/eslint-plugin-react/issues/512, and that conceptually, I think a middleware is the place to do these kind of manipulations.

Maybe it's a good idea to mention that you can access the route object from routeComponentProps here? Since you could store extra props for specific routes there.

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

Merging [#3389][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master      #3389   diff @@
==========================================
  Files            30         30
  Lines           921        921
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            875        875
  Misses           46         46
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by 431a549
[cc-base-branch]: https://codecov.io/gh/reactjs/react-router/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/reactjs/react-router/pull/3389?src=pr

That can be covered in other docs. It's not relevant for the purposes of this discussion and will e.g. fail most linting rules.

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

Merging [#3389][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master      #3389   diff @@
==========================================
  Files            30         30
  Lines           921        921
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            875        875
  Misses           46         46
  Partials          0          0

Powered by Codecov. Last updated by 3e693e7
[cc-base-branch]: https://codecov.io/gh/reactjs/react-router/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/reactjs/react-router/pull/3389?src=pr

Thanks!


[contribution] make test

Is your feature request related to a problem? Please describe.
I'd like to be able to run the tests very easily and quickly, without knowing what's behind, pytest or anything else.
Describe the solution you'd like
make test
Describe alternatives you've considered, if relevant
No response
Additional context
No response

@augustebaum i remember you had a target in your local Makefile to do so?

yes, I meant quickly accessible.

Just a thought regarding Makefile: it does not work on Windows.
Some alternatives to know about:

spin: https://github.com/scientific-python/spin. The idea is to provide consistent developer commands across the scientific ecosystem that works and all platforms. It is used in NumPy, SciPy, scikit-learn as examples
pixi: https://pixi.sh/latest/. This is more generic tools. In addition to manage environment for instance for the CI, it provides a way to define tasks that is exactly what you need here. We give it a "spin" in skrub.

Usage as a user
spin tests

pixi run tests

I don't think it is a priority right now but it could be great to evaluate those solutions. Feel free to open a specific issues particularly for this feature.

The makefile is only for contributors use.

Contributors can still have Windows :). The tools mentioned are really crafted for contributors/developers tool not end-user.

That's an issue for sphinx as well, since we build it with make html. Maybe we could just use sphinx-build directly, like they explain in the docs

For now, we all use unix-like OS compatible with Makefile. IMO, keep moving, and come back to your suggestion when we have to.


Complete TODOs

Could @embark or @nerdrew PTAL?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Feature request: show a diff of changes to config, ask for confirmation

Otherwise LGTM.

Added #19


Ener dev50 add comment  to code

After some docmentations I will take time to Comment all Code.

Thank you for the PR, but it is not needed, so I'm closing it.


Is there a way to change color in real time on the hair segmentation android?

I was wondering if there is a way to input RGB values dynamically in the RecolotCalculatorOptions in the graph

Hi,
Currently, the Recolor Calculator reads in a Color from the options , and reads those values into color_ during the calculator's Open() funtion, and sends the values to the GPU during InitGPU() .
To enable changing the color every frame, a couple things need to happen:

Create an input_stream of Color values (or an array, or your own custom struct) . This would look similar to the "IMAGE_GPU" packet stream (except with a different datatype).
Read those values into color_ during ProcessGPU (again similar to how it's done here and here )
Send the values to the GPU, via adding
glUniform3f(glGetUniformLocation(program_, "recolor"), color_[0], color_[1], color_[2]);
right after the call to glUseProgram(program_); inside of GlRender() .
Also don't forget to register the new stream in GetContract()

After all that is done, you would need another calculator to send in the new color values, and that would be up to you on the logic.
These changes to Recolor Calculator are something we can consider adding in the future, but right now there is no timeline for it.
Hopefully that provides some insight for you.
Cheers,
~Chris

Hi,
I am not sure how to follow step 2. I made a input_stream  "RGB_ARRAY". I added the following lines,
const Packet& rgb_packet = cc->Inputs().Tag("RGB_ARRAY").Value();
const auto& rgb_buffer = rgb_packet.Get<mediapipe::GpuBuffer>();
I intend to make a slider(seekbar) for each of the RGB channels in the app. Do I need another calculator for inputting values from the sliders and output an array to the Recolor Calculator? If so ,how do I send the slider value to the new calculator?

Hi,
What data type is the "RGB_ARRAY" ? Let's say it is std::vector, then you should be accessing the data of the Packet via
const Packet& rgb_packet = cc->Inputs().Tag("RGB_ARRAY").Value();
const auto& rgb_array = rgb_packet.Get<std::vector<int>>();
(instead of GpuBuffer).
For the sliders, you could have them all in one calculator, and output the std::vector in one packet stream... or you could have separate calculators with separate streams "RED_VALUE","BLUE_VALUE"...etc, and then modify the Recolor Calculator to accept each color value stream separately
const Packet& red_packet = cc->Inputs().Tag("RED_VALUE").Value();
const auto& red_value = rgb_packet.Get<int>();
....
If you already have some sliders that output values, then I would suggest creating one ColorSliderCalculator that outputs the RGB_ARRAY stream as vector/ , and have the color slider calculator accept an InputSidePacket to a pointer to the slider values, then output those values as a packet (rgb vector<>) from the calculator.
InputSidePackets are packets that get sent once at the beginning of the graph run. They are usually used for initialization values or connecting data structures across calculators or other parts of the code (in your case pointers to rgb sliders).

Hi,
I am having trouble sending the packets from sliders to mediapipe graph. I have made the ColorSliderCalculator which takes in 3 input stream and outputs a vector from calculator. As for the slider logic , this,thisand this holds the slider values . I suppose the next thing I need to do is to create int packets like this . How do I send the packets to the input stream. ? Also let me know if there is any corrections needed in my code.

Hi,
your cc->Outputs().Tag('RGB_OUT).Set<std::vector>(); is missing a ' after OUT , but also i would use double quotes for strings.
You will probably want to use the FrameProcessor to get the packet creator, instead of creating a new one. processor.getPacketCreator() that way you associate with the graph.
You can also get access to the graph via the processor, in which you would add your packets to the input stream. See what onNewFrame is doing at addConsumablePacketToInputStream  .
In this case, the input stream is defined at the top of the graph as input_video , you would need to add another 3 streams there to accept your color packets.
To keep things simple, i would recommend sending in your custom packets each frame, regardless if the values change, so that calculators don't wait on missing inputs (there is a way to change this behavior later if desired).

Hi
I have made the changes in BUILD , activity_main.xml ,MainActivity.java , hair_segmentation_mobile_gpu.pbtxt , and created ColorSliderCalculator. Bazel build is working fine . But the apk crashes once the app is launched. Can you please check for possible bugs?

Hi,
based on adb logcat:
08-20 21:54:43.104 32312 32333 E AndroidRuntime: RecolorCalculator::GetContract failed to validate:
08-20 21:54:43.104 32312 32333 E AndroidRuntime: For input streams ValidatePacketTypeSet failed:
08-20 21:54:43.104 32312 32333 E AndroidRuntime: Tag "RGB_ARRAY" index 0 was not expected.

I don't see any modification to the Recolor calculator to accept the new rgb array packet stream.
you will need to add this in GetContract to that calculator
  if (cc->Inputs().HasTag("RGB_ARRAY")) {
    cc->Inputs().Tag("RGB_ARRAY").Set<vector::<int>>();
  }

then, in GlRender(), right after the glUseProgram() update
 glUniform3f(glGetUniformLocation(program_, "recolor"), my_colors[0], my_colors[1],               my_colors[2]);  with my_colors being from your new packet stream
also, you are really close in the java side. but putting everything in surfaceChanged is not what you want though, as that is called only once!
you want something like this
 private class RGBHandler implements TextureFrameConsumer {
    @Override
    public void onNewFrame(TextureFrame frame) {
       <blah> sliders update<blah>
   }
}

to receive callbacks each new camera frame
see HERE where i did this in your MainActivity.java with RGBHandler
also, you need to use PacketClonerCalculator in your graph to dupe the slider values each frame, otherwise (like i said previously), the graph will hang waiting for all input streams in the recolor calculator.
see HERE where i modified your graph with PacketCloner
BTW
i tested everything i linked to compiles at least, but you need to implement the recolor calculator logic to handle the new slider value streams
^note the above links expire after a month
hope that helps

Hi
I made the suggested changes.Build the app without errors. This is my latest commit.
However the camera doesn't open and the app looks like this .

Likely something is not correct when starting the graph.
At a quick glance, I don't see anything too wrong, except the output of ColorSliderCalculator...
Can you use a unique pointer for the output packet of the ColorSliderCalculator ,
(ex: absl::make_unique<std::array<int>> )  ? See the letterbox padding output as reference.
What does logcat say?

Logcat outputs:
adb logcat |grep com.google.mediapipe.apps.hairsegmentationgpu 08-24 23:08:15.684  2672  2672 I Timeline: Timeline: Activity_launch_request time:146732 intent:Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity bnds=[238,909][439,1241] } 08-24 23:08:17.091  6907  6907 D AccessibilityManager: current package=com.google.mediapipe.apps.hairsegmentationgpu, accessibility manager mIsFinalEnabled=false, mOptimizeEnabled=false, mIsUiAutomationEnabled=false, mIsInterestedPackage=false 08-24 23:08:17.114  6907  6907 I art     :   at void com.google.mediapipe.apps.hairsegmentationgpu.MainActivity.onCreate(android.os.Bundle) (MainActivity.java:87) 08-24 23:08:17.115  6907  6907 I art     : Caused by: java.lang.ClassNotFoundException: Didn't find class "android.view.View$OnUnhandledKeyEventListener" on path: DexPathList[[zip file "/data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/base.apk"],nativeLibraryDirectories=[/data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/lib/arm64, /data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/base.apk!/lib/arm64-v8a, /system/lib64, /vendor/lib64]] 08-24 23:08:17.115  6907  6907 I art     :   at void com.google.mediapipe.apps.hairsegmentationgpu.MainActivity.onCreate(android.os.Bundle) (MainActivity.java:87) 08-24 23:08:17.349   729  1591 I CameraService: CameraService::connect call (PID -1 "com.google.mediapipe.apps.hairsegmentationgpu", camera ID 1) for HAL version default and Camera API version 1 08-24 23:08:17.467  1555  1576 I ActivityManager: Displayed com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity: +1s733ms 08-24 23:08:17.467  1555  1576 I Timeline: Timeline: Activity_windows_visible id: ActivityRecord{4c14eb8 u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49739} time:148515 08-24 23:08:57.722  1555  1570 W ActivityManager: Activity pause timeout for ActivityRecord{4c14eb8 u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49739 f} 08-24 23:09:00.648  2672  2672 I Timeline: Timeline: Activity_launch_request time:191696 intent:Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity bnds=[238,603][439,935] } 08-24 23:09:08.209  1555  1570 W ActivityManager: Activity destroy timeout for ActivityRecord{4c14eb8 u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49739 f} 08-24 23:10:01.520  1555  1570 W ActivityManager: Activity pause timeout for ActivityRecord{7a6db9e u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49740} 08-24 23:10:11.522  1555  1570 W ActivityManager: Activity stop timeout for ActivityRecord{7a6db9e u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49740} 08-24 23:10:15.131  1555  2059 D ProcessManager: remove task: TaskRecord{aecdf46 #49740 A=com.google.mediapipe.apps.hairsegmentationgpu U=0 StackId=1 sz=1} 08-24 23:10:41.070  2672  2672 I Timeline: Timeline: Activity_launch_request time:292117 intent:Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity bnds=[238,1569][439,1901] } 08-24 23:10:41.361  7682  7682 D AccessibilityManager: current package=com.google.mediapipe.apps.hairsegmentationgpu, accessibility manager mIsFinalEnabled=false, mOptimizeEnabled=false, mIsUiAutomationEnabled=false, mIsInterestedPackage=false 08-24 23:10:41.384  7682  7682 I art     :   at void com.google.mediapipe.apps.hairsegmentationgpu.MainActivity.onCreate(android.os.Bundle) (MainActivity.java:87) 08-24 23:10:41.385  7682  7682 I art     : Caused by: java.lang.ClassNotFoundException: Didn't find class "android.view.View$OnUnhandledKeyEventListener" on path: DexPathList[[zip file "/data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/base.apk"],nativeLibraryDirectories=[/data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/lib/arm64, /data/app/com.google.mediapipe.apps.hairsegmentationgpu-1/base.apk!/lib/arm64-v8a, /system/lib64, /vendor/lib64]] 08-24 23:10:41.385  7682  7682 I art     :   at void com.google.mediapipe.apps.hairsegmentationgpu.MainActivity.onCreate(android.os.Bundle) (MainActivity.java:87) 08-24 23:10:41.597   729  1591 I CameraService: CameraService::connect call (PID -1 "com.google.mediapipe.apps.hairsegmentationgpu", camera ID 1) for HAL version default and Camera API version 1 08-24 23:10:41.710  1555  1576 I ActivityManager: Displayed com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity: +619ms 08-24 23:10:41.710  1555  1576 I Timeline: Timeline: Activity_windows_visible id: ActivityRecord{c5822d0 u0 com.google.mediapipe.apps.hairsegmentationgpu/.MainActivity t49742} time:292758
I also tried following the letterbox padding example. But the app crashed with those modifications. Here is the gist with code modification and log.

Logcat outputs  with this commit
 08-27 12:54:30.661  2931  2950 D ExternalTextureConv: Created output texture: 2 width: 1080 height: 1440
08-27 12:54:30.695  2931  2950 D ExternalTextureConv: Created output texture: 5 width: 1080 height: 1440
08-27 12:54:30.751  2931  2949 I tflite  : Initialized TensorFlow Lite runtime.
08-27 12:54:30.752  2931  2949 I tflite  : Created TensorFlow Lite delegate for GPU.
08-27 12:54:44.556  1968  2024 W WindowManager: Unable to start animation, surface is null or no children.
08-27 12:54:54.029  1968  1968 W WindowManager: removeWindowToken: Attempted to remove non-existing token: android.os.Binder@e188363
The last 2 lines  (Unable to start animation and removeWindowToken) occur when I move one of the sliders

to make the black screen away, you need to publish the packets of RGB at least once outside of the callback, for example at the end of onCreate().
the ColorSliderCalculator is not really needed. you could publish an array directly or just work with 3 inputs.
the actual problem here is retrieving the packets in the recolor_calculator. it seems that those packages never arrive. i checked with the following:

if (cc->Inputs().HasTag("RGB_ARRAY") && !cc->Inputs().Tag("RGB_ARRAY").IsEmpty()) {
const Packet& rgb_packet = cc->Inputs().Tag("RGB_ARRAY").Value();
const auto& rgb_buffer = rgb_packet.Get<std::array<int,3>>();
my_color.push_back(rgb_buffer[0] / 255.0);
my_color.push_back(rgb_buffer[1] / 255.0);
my_color.push_back(rgb_buffer[2] / 255.0);
}

this way, everything is fine, but the color is not changing.
however, when i remove the check with HasTag and IsEmpty, then the app will crash with this error:
2019-08-27 11:28:11.710 19931-19931/? A/DEBUG: pid: 19869, tid: 19887, name: mediapipe_gl_ru  >>> com.google.mediapipe.apps.hairsegmentationgpu <<< 2019-08-27 11:28:11.710 19931-19931/? A/DEBUG: signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
this error doesn occur with this line only:

const Packet& rgb_packet = cc->Inputs().Tag("RGB_ARRAY").Value();

in fact, it is this following line that leads to the error:

const auto& rgb_buffer = rgb_packet.Get<std::array<int,3>>()

i've tried different types of packets but it is always the same error.

i forgot to mention that you should use System.currentTimeMillis() instead of packet.getTimestamp() inside the RGBHandler, because packet.getTimestamp() will output some invalid negative number.

@gitunit  I am getting the same error

@gitunit  I am the getting the same error as you when I send the packets at the end of onCreate()

@sandipan1 so the black screen is gone for you as well?
it looks like the packets are empty inside the recolor_calculator. im not sure why.

finally! i figured it out. the timestamp of the rgb packets was way ahead of the image, thus there were never packets ready to be synced with the images. you can debug those timestamp inside of Graph.java in the addConsumablePacketToInputStream method.

@gitunit In order to sync with the frame , I have added frame.getTimeStamp(). Logcat is this
However the black screen returns.I am not sure how to debug inside Graph.java

easy, instead of the TextureFrameConsumer, you need to set the interface for OnWillAddFrameListener. this is an interface where you can get the timestamp before the image frame is being send.

Now the video frame is displaying. However, the segmented hair is inverted vertically and the color is only black. Nothing happens when I change the slider positions.I have tried changing setting processor.getVideoSurfaceOutput().setFlipY(FLIP_FRAMES_VERTICALLY)   but it doesnt help
These are the  changes

you can try removing the
flip_vertically: true
in the graph (or setting it to false)
if you are in sync with the v0.6 release, you shouldn't need it, and that may be why things are flipped
...
can you control the color if you manually set the r/g/b packets to hardcode a single color when calling addConsumablePacketToInputStream() ? (instead of the slider values)

@sandipan1 Any updates from your end? We will close this for now. Pls reopen if u have updates

Hello
I Try to replicate all the steps to run this example, but I  have some complications.
What I do?

I clone the MediaPipe repository
I change the files that @SwatiModi recommended here

ColorSliderCalculator.cc
recolor_calculator.cc
MainActivity.java
hair_segmentation_mobile_gpu.pbtxt

Build the app without errors
the app run but I have the black screen

On the logcat I have this
12-07 10:00:25.112  1468  1553 W WindowManager: preserveSurfaceLocked: failed, no surface, w=Window{65604df u0 com.google.mediapipe.apps.hairsegmentationgpu/com.google.mediapipe.apps.hairsegmentationgpu.MainActivity}
And when scroll the color slide have this on the logcat
12-07 10:18:17.610  1468  1616 W WindowManager: Unable to start animation, surface is null or no children. 12-07 10:18:27.093  1468  1468 W WindowManager: removeWindowToken: Attempted to remove non-existing token: android.os.Binder@d1fad5d
The custom mediapipe git is here
How can I resolve this ?
thanks a lot!

Hello
I Try to replicate all the steps to run this example, but I have some complications.
What I do?
* I clone the MediaPipe repository

* I change the files that @SwatiModi recommended [here](https://github.com/google/mediapipe/issues/420#issuecomment-694376175)

  * [ColorSliderCalculator.cc](https://github.com/pulpoec/mediapipe_custom/blob/new_branch/mediapipe/calculators/image/ColorSliderCalculator.cc)
  * [recolor_calculator.cc](https://github.com/pulpoec/mediapipe_custom/blob/new_branch/mediapipe/calculators/image/recolor_calculator.cc)
  * [MainActivity.java](https://github.com/pulpoec/mediapipe_custom/blob/new_branch/mediapipe/examples/android/src/java/com/google/mediapipe/apps/hairsegmentationgpu/MainActivity.java)
  * [hair_segmentation_mobile_gpu.pbtxt](https://github.com/pulpoec/mediapipe_custom/blob/new_branch/mediapipe/graphs/hair_segmentation/hair_segmentation_mobile_gpu.pbtxt)

* Build the app without errors

* the app run but I have the black screen

On the logcat I have this
12-07 10:00:25.112 1468 1553 W WindowManager: preserveSurfaceLocked: failed, no surface, w=Window{65604df u0 com.google.mediapipe.apps.hairsegmentationgpu/com.google.mediapipe.apps.hairsegmentationgpu.MainActivity}
And when scroll the color slide have this on the logcat
12-07 10:18:17.610 1468 1616 W WindowManager: Unable to start animation, surface is null or no children. 12-07 10:18:27.093 1468 1468 W WindowManager: removeWindowToken: Attempted to remove non-existing token: android.os.Binder@d1fad5d
The custom mediapipe git is here
How can I resolve this ?
thanks a lot!

Hi i have the same black screen problem. How did you solve this problem?

anyone with working solution with the new version of mediapipe?
i got this errors after trying what is suggested here check this comment


feat: wrappers refactor

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove "with native extensions" message (UI::RGProxy)

Using version 1.10.1, I noticed the console output was a little different. There are these with native extensions output:
Installing rake 0.9.2
with native extensions Installing RedCloth 4.2.8
Using posix-spawn 0.3.6
Installing albino 1.3.3
Installing chunky_png 1.3.0
with native extensions Installing fast-stemmer 1.0.0
Installing classifier 1.3.3
Installing fssm 0.2.10
Installing sass 3.2.14
Using compass 0.12.2
Installing directory_watcher 1.4.0
Installing haml 3.1.2
with native extensions Installing iconv 1.0.3
Installing kramdown 0.13.3
...

After some digging around, I believe the output we would want is something like: Installing iconv 1.0.3 with native extensions. However, since the install_message is output after the gem in installed, it doesn't seem like it would be a straightforward fix.
I also think that UI::RGProxy was not used in some of the previous versions and was reintroduced by this fix. This means that previous versions did not display with native extensions at all.
That's why I decided to remove UI::RGProxy entirely. If I am mistaken and we should keep that class, I could revise this PR to only include the documentation (.ronn) changes.
Edit: I saw that tests were failing when I removed UI::RGProxy. I've updated this PR to remove the say overriding method

We actually want this message to print (and we should add a test to make sure that it prints on a single line the way it's supposed to. Thanks for pointing out that it's not working, though!


allow prepare_ml_data.py to run without GSP data source after locations CSV files have been created

Detailed Description
GSP data source is required to sample geo locations from.  But, once the locations CSV files have been created, it should be possible to run prepare_ml_data.py with any subset of DataSources (e.g. just 'satellite').
But, if gsp isn't included, then prepare_ml_data.py complains because it always checks whether the data source that defines the geospatial regions is in the list of datasources.
We should re-order the code a little so that the code is happy with any subset of DataSources (after the CSV files specifying the locations have been created)

I just got this,
got round it by using
python scripts/prepare_ml_data.py --data_source=gsp --data_source=pv
for the moment

PR #492 fixes this issue


Stable URL for latest asset downloads

We'll always want to just pull the latest phenio into monarch-ingest. I'm not sure if there's an existing pattern for how to implement this, but it might work to use a GitHub action like this to create a "latest" tag?
https://github.com/marketplace/actions/latest-tag#:~:text=Automatically creates %26 updates a latest,need to use a branch

Thanks for the suggestion! I'll just do it manually for now while releases are lest frequent, so the most recent phenio.owl will be at https://github.com/monarch-initiative/phenio/releases/latest/download/phenio.owl


Is it any way to know how long it has taken the user to respond back a message?

I am using Skype channel and C#.
Specifically I would like to know:
A) The period time that passed between the user wrote the first character of a message and then sent it (this probably doesn't exist but..) This is usually known as typing cadence.
OR
B) The time between the last message by the bot and the arrival of the response by the user.
Thanks!

@piffarettig
IAwaitable result ;
var message = await result;
var time =message.LocalTimestamp;

@piffarettig, I'm assuming @Lipa9963's comment helped you out, so I'm closing this. Feel free to reopen it if this is not the case. Thanks!


Verify anchor design and update styling if necessary

🧹 Tech Debt
There are a couple differences between our anchor styling and the Figma design:

Visited links have a distinct color (purple)
Keyboard focus is indicated by an outline rather than double underline

note: we should push back on this visual design since the text could flow across multiple lines, which would look weird if we try to draw a box around it
An outline will work fine –

These were not the designs when we implemented the anchor. We should confirm whether they are what we want (with Brandon) and implement if so.

We decided yesterday to go forward with the box approach. Moved to Current Iteration since the table keyboard navigation feature revealed that the current focus styling isn't prominent enough.

After further discussion, we've decided:

Visited links will not have a different color. This decision may be revisited in the future based on specific use cases or client requests.
Keyboard focus will continue to use a double underline, but the link color will change to make it more visually prominent. There is a technical issue with having an outline that goes outside of the bounds of the link (and remain visible) and being able to ellipsize truncated text, because of the need to use overflow: hidden.


MM-35332 Cypress/E2E: Not extending session when setting is disabled

Summary
Related PR - https://github.com/mattermost/mattermost-server/pull/17581
Test cases:

MM-T4045 - session via Email login
MM-T4046 - session via LDAP login
MM-T4047 - session via SAML login

Note:

Separated into its own spec file since they require different auth mechanism and to make it easier to move individually into production test.
Test will fail if your server doesn't include the recent fix - https://github.com/mattermost/mattermost-server/commit/a668479992047548e6d40eed4e5cc1d408161447

Ticket Link
https://mattermost.atlassian.net/browse/MM-35332
Screenshots

Release Note
NONE

@saturninoabril I enabled the keycloak but it's now complaining about missing fixture


how to install for windows??

this looks like a great tool, but I really can't figure out how to get it install it!
Ive downloaded the .exe file. ran it, ran it via my bash cli, ran it via powershell, downloaded rust and cargo, tried to install that way but then i got an error saying i needed some visual studio tools...
There must be a simple step that I dont know about here?

Thanks for the kind words! There are 2 ways to install Doctave on Windows: download the precompiled binary, or install via Cargo. Admittedly I don't develop on Windows so testing on that platform so far has not been as extensive as on Linux/Mac (tested on a clean Windows 10 VM).
The precompiled .exe published as a zip here should be a "static binary" you can run in a terminal (e.g. Powershell). Do you get an error when you run it in? Could you paste the message here so I could debug the error?
The other option is to install Doctave via Cargo, in which case it is compiled locally on your machine. It looks like you may need the Visual Studio C++ build tools installed to get Rust/Cargo to work. Here's a blog post describing how to do that. Rust's own documentation for installing Rust/Cargo on Windows can be found here.
Let me know if either of those options work.


About the BN part

Hi, thanks for your great work. I am wondering about the BN part, it seems that the methods like "cRT" and "DRW" do update the running mean and variances, right? I can not find the code segment which aims to freeze this part.

Hello! Actually, in the final framework of "cRT", "LWS",  "DRS", and "DRW", they do update the running mean and variances in BN. While, in the "tau-norm" and "KNN" decoupling models, they don't update them. In our paper, we focus on the over-confidence issue. Due to the different class number composition ratios for the two stages, we believe the shifted BN would have some effects on the recognition performance and the calibration, which hasn't been discussed in previous two-stage methods. According to our experiment, it's proved.

Hello! Actually, in the final framework of "cRT", "LWS", "DRS", and "DRW", they do update the running mean and variances in BN. While, in the "tau-norm" and "KNN" decoupling models, they don't update them. In our paper, we focus on the over-confidence issue. Due to the different class number composition ratios for the two stages, we believe the shifted BN could have some effects on the recognition performance and the calibration, which hasn't been discussed in previous two-stage methods. According to our experiment, it's proved.

So, acturally, the shifted BN is a widely used mehotd for "CRT", "LWS" also fixed the affine part?

Hello! Actually, in the final framework of "cRT", "LWS", "DRS", and "DRW", they do update the running mean and variances in BN. While, in the "tau-norm" and "KNN" decoupling models, they don't update them. In our paper, we focus on the over-confidence issue. Due to the different class number composition ratios for the two stages, we believe the shifted BN could have some effects on the recognition performance and the calibration, which hasn't been discussed in previous two-stage methods. According to our experiment, it's proved.

So, acturally, the shifted BN is a widely used mehotd for "CRT", "LWS" also fixed the affine part?

Is there something more details in the BN part of this paper?

Yes, both "cRT" and "LWS" fix the affine part and update the running means and variances in Stage-2. For more details about the shifted BN, please refer to the experiment part. We also think a novel normalization to effectively deal with long-tailed recognition should be discussed and explored in the future.

Yes, both "cRT" and "LWS" fix the affine part and update the running means and variances in Stage-2. For more details about the shifted BN, please refer to the experiment part. We also think a novel normalization to effectively deal with long-tailed recognition should be discussed and explored in the future.

Thanks, look forward to your new work


EnqueueProgram test hangs on 1x N300

Confirm that we indeed see different behavior between 1x N300 and T3000 (4x N300).
Investigate if a specific commit caused a regression

Narrowed down regression to 67238f6dde6b9e9ea319ca935c1c7f586440a45d
With fix implemented in c6f06f8e5f3997f975bc086cab86d2f2a19c1725


TypeError: scopesArray.map is not a function on v7.0

Issue summary

@shopify/shopify-api version: 7.0.1
Node version:v14.21.3
Operating system: macos

it seem like a bug on<EMAIL_ADDRESS>, I meet a error that UnhandledPromiseRejectionWarning: TypeError: scopesArray.map is not a function in the AuthScopes when update this version<EMAIL_ADDRESS>
but the first time is ok it break on second time reload the page on localhost dev

log below
(node:69582) UnhandledPromiseRejectionWarning: TypeError: scopesArray.map is not a function
    at new AuthScopes (web/node_modules/.pnpm/@shopify+shopify-api@7.0.0/node_modules/@shopify/shopify-api/lib/auth/scopes/index.js:15:14)
    at Session.isActive (web/node_modules/.pnpm/@shopify+shopify-api@7.0.0/node_modules/@shopify/shopify-api/lib/session/session.js:74:79)
    at/web/node_modules/.pnpm/@shopify+shopify-app-express@1.2.2/node_modules/@shopify/shopify-app-express/build/cjs/middlewares/validate-authenticated-session.js:58:21

Expected behavior

Actual behavior

Steps to reproduce the problem

@schapicool Please advise how you're reproducing this?  What database/database adapter are you using?


Boolean coercion has undesired effects on command-line argument processing

Expected behavior
Specifying the option --deploy-dev-tools or --deploy-dev-tools true should do the same thing
Actual behavior
Specifying the option --deploy-dev-tools leaves the deployDevTools value set to false
Consequence
If a user does this by accident they will not get an indication they've done something wrong, the value will be the opposite of what they expect, and there will be no messages in the output indicating that the command-line tools will or won't be installed.
This makes troubleshooting the issue challenging. Normally when the developer tools are not installed but the user requested them to be installed there is a warning that indicates the tools won't be installed because previous deployments exist.
Background
I was trying to set up a new Greengrass system today and my command-line looked like this:
sudo -E $JAVA -Droot="$GREENGRASS_ROOT" -Dlog.store=FILE \
  -jar ./GreengrassInstaller/lib/Greengrass.jar \
  --aws-region us-east-1 \
  --thing-name "$THING_NAME" \
  --thing-group-name "$THING_GROUP_NAME" \
  --thing-policy-name GreengrassV2IoTThingPolicy \
  --tes-role-name GreengrassV2TokenExchangeRole \
  --tes-role-alias-name GreengrassCoreTokenExchangeRoleAlias \
  --component-default-user ggc_user:ggc_group \
  --provision true \
  --setup-system-service true \
  --deploy-dev-tools

--deploy-dev-tools was missing the true option after it due to a copy and paste issue on my side.
I would expect this flag to either set the value to true or to report an error that a value needs to be specified. Normally when a flag can be specified the user expects the affirmative version --deploy-dev-tools to set the value to true and the negative version --no-deploy-dev-tools to set the value to false.
This line of code appears to be the culprit that is setting the value to false:
https://github.com/aws-greengrass/aws-greengrass-nucleus/blob/34c2fe952486bf8cb171d06619e428fc8434ea4c/src/main/java/com/aws/greengrass/easysetup/GreengrassSetup.java#L451
It appears that the Coerce.toBoolean method is used outside of just the Greengrass setup so I don't know if its behavior can be changed easily.
Instead it might be easier to rework the command-line processing code to use a library that handles this automatically like picocli.

It looks like adding picocli as a dependency appears to increase the output JAR size by ~400k.

Thank you for pointing out the issue. We will look into possible solutions.


Added files related to Compose page

I have completed the assigned work. This PR closes the issue #5  .

This is the UI of the Composing a Complaint page.
After typing the complaint and adding image, This page look like this.

Good job @jainsid2305 ... one suggestion : I guess white color for title would look better and if that card for filling complaints, if shaped a bit rounded at the top right corner may go well with the design.

Thanks .
I will make the suggested changes.


preload_data is only for RawFIF

I was just working with a RawEDF and I went to call raw.preload_data, but I noticed that this method didn't exist for this Raw class. It's currently only implemented for RawFIF. I also saw that there is a new (to me at least) parameter in _read_segment called data_buffer that is being used whereas the other modules don't have this. should we use this method's use?

as you can now load edf data without preloading yes we could add this
method. It means all raw classes should properly implement it. Clearly not
an issue for v0.9 but maybe a rainy sunday afternoon...

@agramfort AFAIK only EDF and FIFF allow not preloading data. If that's the case then preload_data should automatically work for the other backends because they automatically set self._preloaded = True or whatever the property is, and raw.preload_data() checks that (or should check for it) and returns if data are already preloaded. So it should just be a matter of moving the method to Base and making sure it works with EDF.

can you give it a try?

Yeah, maybe today or later this week


Add capability for special UDP packet handling during hot restart

Commit Message: Add capability for special UDP packet handling during hot restart
Additional Description: Adds a hook for a callback to UDP listeners, and uses that callback in ActiveQuicListener and EnvoyQuicDispatcher. This is a no-op in production since the registration function isn't yet called, but is a first step towards allowing us to catch new connections on a QUIC listener and forward them to a new instance during hot restart. An important part of https://github.com/envoyproxy/envoy/issues/19454
Risk Level: Low, essentially a no-op. Adds one nullptr check in the common path per nonroutable quic packet; zero touch to the common routable packet path.
Testing: Added unit test in both EnvoyQuicDispatcher and ActiveQuicListener.
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Assigning Ryan for HTTP/3 expertise.
/assign @RyanTheOptimist

Can you take a look @RyanTheOptimist , and @danzh2010? Thank you

@danzh2010 I'm happy for you to do the first pass on this. Let me know when you're happy.

/wait

I think UDP proxy would also be affect by hot restarting. So ActiveUdpListener probably also need to handle onHotRestarting(forwarding_func) appropriately.
cc @mattklein123

I think UDP proxy would also be affect by hot restarting. So ActiveUdpListener probably also need to handle onHotRestarting(forwarding_func) appropriately.

I don't think regular UDP cares that much for the most part, because it makes no difference which instance handles the packet (unless the set of filters has changed). It's only the connection context of QUIC that makes it instance-dependent.

It also keeps a state based on downstream 4 tuple and upstream 4 tuple. So I think it is affect. But Lyft might not need to use it? @mattklein123

(I definitely don't want to be also changing the behavior of regular UDP, so if it is relevant, it can be relevant later.)

You don't need to provide a solution for that. But it's worth to add a ENVOY_BUG or comment along the code path to indicate there is a gap.

/retest


Control.py broken?

I seem to have problems with control.py.
I expect to have done some mistake while installing.

This is an Error i get for running something like 'ctl.ground([("step",[42])])' in the python API:

File "/home/zimne/anaconda3/envs/tel/lib/python3.9/site-packages/clingo/control.py", line 335, in ground
c_part.params[i] = sym._rep
AttributeError: 'int' object has no attribute '_rep'

And this is an error if I am running the Asprilo Visualizer:

File "/home/zimne/anaconda3/envs/tel/lib/python3.9/site-packages/visualizer-0.2.2-py3.7.egg/visualizer/parser.py", line 206, in parse
AttributeError: 'Control' object has no attribute 'builder'

*The same programs do run on 5.4 though

I think the API interface has slightly changed in the latest update. Wrap the 42 inside Clingo.Number().-------- Originalnachricht --------Betreff: Re: [potassco/clingo] Control.py broken?  (#323)Von: ZimneJonas An: potassco/clingo Cc: Subscribed
*The same programs do run on 5.4 though
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/potassco/clingo/issues/323#issuecomment-881929492",
"url": "https://github.com/potassco/clingo/issues/323#issuecomment-881929492",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi @ZimneJonas ! Thanks for pointing this out to us! In fact, the paper at archive will ultimately align with 5.5 (have you checked the recent version ?), once it is published. We just pre-published its preliminary version that only ran with an intermediate version. Sorry for any inconvenience!

I am going to open an issue for the asprilo visualizer.

Hi, I tried to run the examples published in 'Multi-shot ASP solving with clingo' section 5.1, Listing 5, 6 and 7 https://arxiv.org/abs/1705.09811 and encountered the same error as Jonas. I followed the advice to wrap the parameters inside the number method and now I'm getting :
File "/home/niko/anaconda3/lib/python3.9/site-packages/clingo/symbol.py", line 248, in Number
_lib.clingo_symbol_create_number(number, p_rep)
TypeError: an integer is required
Did I make a mistake or is there also an updated version for this paper?

Please do not comment on long closed issues and rather use a new one. Please open a new one and also show us the code you are trying to run.


fix: json field formatting

:\ it's not formatted on my side...
with unfold ^0.39.0

If you have a problem, create an issue. Thanks


incorrect optional/required parameters

for az cdn endpoint show the CLI states that either the Id is required or name, resource group, and profile name.
The documentation doesn't mention the same.

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

ID: 17e2b3b2-3e26-a47f-ee80-5bd5c3b065c4
Version Independent ID: 72ef6ca0-b8f6-4952-732d-34aefa414bb6
Content: az cdn endpoint
Content Source: src/azure-cli/azure/cli/command_modules/cdn/_help.py
Service: cdn
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

routing to appropriate team

The id, name, resource group, and profile name are all listed under optional parameters in the documentation because there is at least one situation where each of these is optional, (ie. --ids is optional when specifying the endpoint name, resource group, and profile name individually, and vice-versa). However, the requirement to provide one of the two sets is stated in the description of --ids. If you're trying to get all available endpoints, please use the command az cdn endpoint list.


[BUG] Impetus filling issue

If you a fill an impetus from nothing with charged amethyst it caps at 200,000 media and then stops accepting media via hopper. However, after casting with the impetus it can be filled past 200k, and the media value can overflow. I suspect it has to do with non-whole number media values, but that's just a guess.

Further investigation reveals that impetus's will only lock if they hit 200k exactly, so if they're on 199,996 and you hopper in a shard they will pass 200k and continue to fill and eventually overflow.


benchmark test

Benchmark Testing Implementation for Various Node.js Frameworks
This pull request introduces a comprehensive benchmarking test that compares the performance of several popular Node.js frameworks. The benchmarks were designed to measure each framework’s:
Requests per second: The number of requests the framework can handle per second.
Transfer rate: The amount of data transferred per second.
Latency: The average time taken to respond to requests.
Latency Stdev: The standard deviation in the response time across all requests.
Max Latency: The maximum latency observed during the tests.
Frameworks Tested:

uWebSockets.js (uws)
Node.js HTTP (http)
Fastify
Koa
Restify
Hapi
Express

Results Summary:
The table below highlights the key metrics recorded during the benchmarking:

(index)
Framework
Requests/sec
Transfer/sec
Latency
Total Requests
Transfer Total
Latency Stdev
Latency Max

0
uws
34801.87
3.48MB
29.22ms
349784
35.03MB
2.53ms
72.43ms

1
http
24739.38
4.20MB
41.16ms
248928
42.26MB
7.45ms
185.33ms

2
fastify
24002.04
4.03MB
42.57ms
241340
40.51MB
8.77ms
249.95ms

3
koa
20874.8
3.48MB
48.37ms
210834
35.19MB
10.52ms
276.27ms

4
restify
13881.54
2.45MB
73.55ms
139624
24.63MB
17.21ms
363.28ms

5
hapi
13473.97
2.84MB
75.93ms
135722
28.61MB
20.09ms
484.66ms

6
express
6865.61
1.56MB
147.68ms
69122
15.69MB
29.83ms
740.75ms

Why this matters:
The benchmarking test helps in identifying the most efficient frameworks in terms of raw performance under heavy load. It can guide developers in choosing the right framework depending on the specific needs of their projects. For instance, if maximum throughput is a priority, uWebSockets.js (uws) significantly outperforms the others. Conversely, more feature-rich but slower frameworks like Express may be suitable for different use cases where development speed and ease-of-use are more important than raw performance.
Next Steps:
Further Analysis: We could continue adding more frameworks and analyze them under different load conditions.
Optimization: Based on the results, optimizations for specific frameworks may be proposed to improve their performance.
Please review the implementation and let me know if any improvements are needed. Thank you!

The only benchmark that matters is uWS vs. Node.js. These "frameworks" are irrelevant and pointless to track, as they all build on Node.js.

I disagree with your opinion, friend, because if your proposal is to have a project that supports HTTP, comparing it with the most used projects on the market is of fundamental importance, Express is much more used than the native implementation of HTTP in Node.js.

It's not my proposal it's my decision.


Error in one line cause the suggestions to stop

Description:
Having an error in the next line cause the suggestions to stop. This is not the case for java in intellij.
Steps to reproduce:
Consider the following code:

This is the behavior of similar scenario in intellij for java code:

Code completion is supported even if there's an error.
Affected Versions:
Swan Lake Preview1
OS, DB, other environment details and versions:
Kubuntu 20.04

Fixed with the latest syntax tree API migration. Hence closing the issue.

Fixed with the latest syntax tree API migration. Hence closing the issue.


Open Window doesn't appear unless you a window in atom already open

Description
Open Window doesn't appear unless you a window in atom already open
Steps to Reproduce

Open Atom
Close all windows, do not quit the application
Go to the File Menu and click on Open

Expected behavior: A window to open up allowing me to select a file/folder to open in Atom
Actual behavior: Nothing
Reproduces how often: Always
Versions
Atom    : 1.12.8
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
macOS: 10.12.2 Build 16C67
Additional Information
This is on a fresh install of macOS. Also the Open Window does appear but only when you have a window in Atom already open.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/12508. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Personalizer Client Library for .NET page doesn't have the correct links

Can someone please refresh/update the doc (Azure Personalizer client library  ) to point to the latest NuGet package (Azure.AI.Personalizer 2.0.0-beta.2) in the install and key concepts sections of this doc? Current links/wording in these sections are confusing/outdated with this release.
Install section:
Current: dotnet add package Azure.AI.Personalizer --version 2.0.0-beta.1
Desired ( @Mohit-Chakraborty to confirm) : dotnet add package Azure.AI.Personalizer --version 2.0.0-beta.2
Key concepts section: add link to the nuget package page
Current: Azure.AI.Personalizer is built on top of ...
Desired ( @Mohit-Chakraborty to confirm) : Azure.AI.Personalizer is build on top of ...

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

ID: b04f689b-ba64-a2b5-400b-b9bc935c014f
Version Independent ID: f9dd7c84-caad-ae65-d7a4-2919d56ecba1
Content: Azure Personalizer client library for .NET - Azure for .NET Developers
Content Source: api/overview/azure/ai.personalizer-readme-pre.md
Service: personalizer
Product: azure
Technology: azure
GitHub Login: @orenmichaely
Microsoft Alias: ormichae

Install section:
Current: dotnet add package Azure.AI.Personalizer --version 2.0.0-beta.1
Desired ( @Mohit-Chakraborty to confirm) : dotnet add package Azure.AI.Personalizer --version 2.0.0-beta.2
Key concepts section: add link to the nuget package page
Current: Azure.AI.Personalizer is built on top of ...
Desired ( @Mohit-Chakraborty to confirm) : Azure.AI.Personalizer is build on top of ...

Sidebar: Why is the bot adding Azure.Identity to this. This is an AI SDK not an Identity one

The bot makes that determination based on an ML model trained against repository issues; in this case, it had enough confidence to believe Identity was the correct label.   It was obviously mistaken for this one.

We should update to dotnet add package Azure.AI.Personalizer --prerelease rather than including a specific version.

This is the backing GitHub page - https://github.com/Azure/azure-docs-sdk-dotnet/blob/main/api/overview/azure/ai.personalizer-readme-pre.md

That page is generated from the README in the SDK repository.   The source for that is here.

Ah, thanks for the info Jesse.

@johnhuang01 more info in this comments thread around the source to change FYI. Please feel free to add any other comments/questions here.

@achandmsft I have sent the PR: https://github.com/Azure/azure-docs-sdk-dotnet/pull/2156

@jsquire, I don't see an update to the docs page even though we have updated the README in the SDK. I guess changes are only picked up upon release.
If we want to force an update, I wonder if it is okay to update the page in azure-docs-sdk-dotnet directly.

They're picked up on release and at scheduled intervals.    @danieljurek would be the person to ask about the specifics.

@johnhuang01 -- please merge the PR if you're ready for the change to go out: https://github.com/Azure/azure-docs-sdk-dotnet/pull/2156
@Mohit-Chakraborty -- These pages update by two means:

Updates to the page in the azure-docs-sdk-dotnet repo which are later pushed live (on Wednesday, most weeks)
Package releases from our CI system which pick up the README.md file from azure-sdk-for-net repo for the package, add some metadata, and update the azure-docs-sdk-dotnet repo with those changes.

A new release will pave completely over whatever changes have been made to the markdown file in azure-docs-sdk-dotnet since the last release so make sure whatever content is in the azure-docs-sdk-dotnet repo is also reflected in your README.md in azure-sdk-for-net source repo if you intend to keep it in the next release.

Understood. Thanks for the info @danieljurek.

@danieljurek I still don't see the personalizer documentation under - https://learn.microsoft.com/en-us/dotnet/api/overview/azure/?view=azure-dotnet-preview.
The PR that had the doc changes were merged https://github.com/Azure/azure-docs-sdk-dotnet/pull/2156. Am I looking at the wrong place?

@pallavit -- The documentation is live here: https://learn.microsoft.com/en-us/dotnet/api/overview/azure/ai.personalizer-readme?view=azure-dotnet-preview ... It has the changes merged from the PR https://github.com/Azure/azure-docs-sdk-dotnet/pull/2156
The top-level ToC to which you link is a list of services, not packages. In the case of the Azure.AI.Personalizer package the package lives under the "Cognitive Services" service. If the ToC needs to change then the value of ServiceName should be updated in the metadata: https://github.com/Azure/azure-sdk/blob/main/_data/releases/latest/dotnet-packages.csv#L63
I'm closing this issue as the concerns have been addressed.


Fix Parent EMS Cloud Title on Storage Pages

Six pages added or modified recently for Cloud Storage and Storage Manager pages
mis-titled the link to the Cloud Manager.  It defaulted to "Parent ems cloud" and
should be "Parent Cloud Provider".
Here is an image of the relevant relationship field for a Cloud Object Store Object.

The pages changed were:

Cinder and Swift Storage Managers
Cloud Volumes
Cloud Volume Backups
Cloud Volume Snapshots
Cloud Object Store Containers
Cloud Object Store Objects

@h-kataria @dclarizio @roliveri please review and merge.  Thanks.

@dclarizio @h-kataria changed to use ui_lookup to obtain the title string.

@dclarizio @h-kataria fixed to now do the ui_lookup on the model instead of using a table lookup based on the parent object.  The parent object results in a string that is too long - "Parent Cloud Provider (Openstack)".  Using the model "ManageIQ::Providers::CloudManager" results in the string "Parent Cloud Provider".  This is ready to rock and roll.

looks good

Euwe Backport details:
$ git log
commit 53e1abad1c3e74578022d961c2f5a79fc18e6352
Author: Harpreet Kataria<EMAIL_ADDRESS>Date:   Thu Oct 6 13:01:34 2016 -0400

    Merge pull request #11726 from jerryk55/Fix_Parent_Ems_Cloud_Title_on_Storage_Mgr_Pages

    Fix Parent EMS Cloud Title on Storage Pages
    (cherry picked from commit 2ebb35fb7aba59fb11e96a3d1b9b33c44782d449)


[Rest Server] Refactor storage by leveraging persistent volumes

Refactor storage by leveraging kubernetes pv and pvc.
TODO:

[x] backward compatibility
[x] samba and azure storage tests
[x] documents

Could we mark deprecated to the storageConfig, storageServer API and the teamwise_storage plugin, considering

they works only in internal release and for a short while,
we actually want users to use the new API and k8s pv/pvc to mount

Could we mark to-be-deprecated to the storageConfig, storageServer API and the teamwise_storage plugin, considering

they are only released internally and for a short while,
we actually want users to use the new API and k8s pv/pvc to mount

yes, those apis have been removed from api docs in this PR, @Binyang2014 will delete runtime plugin after this PR merged.
user will only see docs on new api and use new, existing code uses old api won't be effected.

This change only keep get and list API.
Do we need add some doc about how to create or delete storage?

pls refer to this document

And previous tools such as storagectl will not work.

yes, I have added a deprecated notice for it

Maybe we need to fix this tool to let admin add storage for certain group. Otherwise, we need to modify configMap manually.

admin could use paictl or group api to create/update/delete storage for a given group, as described in this document

Test case:

follow docs to setup nfs/samba/azurefile/azureblob persistent volume separately
check whether they're able to be mounted in PAI.


Coding: Is't possible to separate javascript and component like Angular4+ instead into one .vue file?

Coding: Is't possible to separate javascript and component like Angular4+ instead into one .vue file?

https://vue-loader.vuejs.org/spec.html#src-imports


inline formula notation issue

I wonder why the inline formula notation $$...$$ is different with math mode - Inline equation in latex with text - TeX - LaTeX Stack Exchange $...$

[not a gitbook dev, just providing context]
The problem everybody runs into with $ is false positives — texts like It costs between $20 and $37 occur naturally and having them turn into math unless escaped would be annoying to users who don't need math, and incompatible to the original markdown (well, any syntax extension is technically incompatible but  practically the question is whether many or few existing documents change meaning).
There are many solutions to this, eg. constraining $ with heuristics (e.g. closing $ not followed by digit and not under full moon), using \( / \[ (which interfere with escaping some markdown syntaxes) or \\( / \\[ instead, etc.
Using $$ for both inline and display was I believe an invention of kramdown, and is one of the prettier solutions.  There is interesting critique on https://github.com/GitbookIO/kramed/issues/3 however that it prevents conveying when display math is part of a larger "semantic paragraph" (a concept mostly ignored by Markdown but easily expressed in LaTeX).
And {% math %}...{% endmath %} is of course modeled on Gitbook templating constructs.
FWIW, I'm cataloging math in markdown syntaxes at https://github.com/cben/mathdown/wiki/math-in-markdown

FWIW, I'm cataloging math in markdown syntaxes at https://github.com/cben/mathdown/wiki/math-in-markdown

@cben that's very useful! Thanks for sharing this.

@cben Awesome answer :+1:
With GitBook, inline math are differentiated from block math using the first character, example:
Some inline math $$a = b$$

And a block of math:

$$
a = b
$$

I like that!  2 late questions:

If there are trailing spaces between the opening $$ and the newline it becomes inline.  IMHO surprising, should be still block.

It seems that's specific to GitBook plugins, kramed itself still uses same regex as kramdown, essentially /^ *$$...$$ *\n/ (sounds like should be alone on a line, but I tested kramdown and it actually must be alone in paragraph, as documented, there is probably extra logic...).  Right?
Given that kramed is "your" fork, any plans to match gitbook rule, or will you keep the kramdown rule?

For the inline math, does there exist any way to reset it to $ .... $ in gitbook?  I find  $$ ... $$ annoy me very much when I want to copy markdown to latex or other place.

For the inline math, does there exist any way to reset it to $ .... $ in gitbook? I find $$ ... $$ annoy me very much when I want to copy markdown to latex or other place.

Have you found the right way? @weihuayi

我尝试解决过这个问题，但一直没有成功。
---Original---
From<EMAIL_ADDRESS>Date: Sat, Aug 10, 2019 02:39 AM
To<EMAIL_ADDRESS>Cc: "Huayi Wei"weihuayi<EMAIL_ADDRESS>Subject: Re: [GitbookIO/plugin-mathjax] inline formula notation issue (#15)
For the inline math, does there exist any way to reset it to $ .... $ in gitbook? I find $$ ... $$ annoy me very much when I want to copy markdown to latex or other place.
Have you found the right way? @weihuayi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

可以先把 markdown 转成 latex 试试
---Original---
From<EMAIL_ADDRESS>Date: Sat, Aug 10, 2019 02:39 AM
To<EMAIL_ADDRESS>Cc: "Huayi Wei"weihuayi<EMAIL_ADDRESS>Subject: Re: [GitbookIO/plugin-mathjax] inline formula notation issue (#15)
For the inline math, does there exist any way to reset it to $ .... $ in gitbook? I find $$ ... $$ annoy me very much when I want to copy markdown to latex or other place.
Have you found the right way? @weihuayi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Trying to update a DS that doesn't exist returns an Internal Server Error

I'm submitting a ...

bug report

Traffic Control components affected ...

CDN in a Box
Traffic Ops

Current behavior:

Try making an API call to PUT (update) a delivery service that doesn't exist.
You will get back an internal server error (status code 500).
If you tail the logs, you'll see that this happens because the error is sql: no rows in result set (which is expected)
Expected behavior:

This action should return a 404 (not found) if the DS was not found, or maybe a 400 (bad request).
Minimal reproduction of the problem with instructions:

Same as mentioned above
Anything else:

I haven't checked if the other resources show a similar kind of behavior, maybe we need to check everything for this.

yep, should be 404 Not Found


[Linaro:ARM_CI] Add clang support to aarch64 toolchain

Add the ability to use clang instead of gcc in the aarch64 toolchain

Hi @MichaelHudgins Can you please review this PR ? Thank you!

@elfringham let me know when you want to get this merged.

Hi @elfringham Any update on this CL? Please. Thank you!

@MichaelHudgins We would like to have this PR merged now please. It adds the toolchain but does not use it by default. This will allow use and testing by other CI systems such as Linaro and ARM.

Sorry on the delay, was OOO last week.  Can you please rebase?

@MichaelHudgins I cannot see what the failure on the feedback/copybara check is about. Is it something I need to fix?

Its from a broken windows build from the last LLVM integrate, so not on your end.


question: how are tests run using this action?

Hi 👋
I was seeing this action being used, instead of previous actions, across multiple repositories, where some of them also have tests. I had a look at build/action.yml but did not see any step that is running the tests?
I think it would be important to also run all tests in CI if they are available to avoid regressions and have confidence the package is actually working as expected.

Currently, tests aren't run using any of the actions listed.  I think some have not been run for a while which doesn't help.  I think the best way to handle this is to have an action here that runs the tests if a tests folder is detected.  Is that something you're interested in helping with?

I think the best way to handle this is to have an action here that runs the tests if a tests folder is detected

This can be done in 1 workflow using multiple jobs in the called workflow:
jobs:
  get-lib-info:
    runs-on: ubuntu-latest
    outputs:
      has-tests: ${{ steps.lib-info.outputs.has-tests }}
    steps:
      - name: get repo info
        id: lib-info
        run: |
          has_tests=false
          if [[ -d 'tests' ]]; then
              has_tests=true
          fi
          echo "has-tests=$has_tests" >> $GITHUB_OUTPUT

  run-tests:
    needs: [get-lib-info]
    # output vars are always strings, not booleans
    if: needs.get-lib-info.outputs.has-tests == 'true'
    steps:
      - run: coverage -m run pytest

somewhat off-topic comment
This tactic could also be used to consolidate the release-pypi workflow and the release-gh workflow with some extra bash script:
        run: |
          deploy_pypi=false
          if [[ -f 'setup.py' || -f 'pyproject.toml' ]]; then
              deploy_pypi=true
          fi
          echo "deploy-pypi=$deploy_pypi" >> $GITHUB_OUTPUT

Although, I think you can just check if pypi credentials are provided (as optional inputs in this scenario).
on:
  workflow_call:
    inputs:
      pypi-username:
        required: false
        default: '__token__'
        type: string
    secrets:
      pypi-password:
        required: false
        default: '?!'

jobs:
  release-pypi:
    if: secrets.pypi-password != '?!'
    # ...

Oops! I just realized these workflows are meant to be composite actions and not reusable workflows. While there's only a few subtle differences, my above suggestion (excluding the off-topic part) could still apply. @tekktrik please disregard my other comment about using secrets; I'm so sorry.


separate vivado tools

restructure project to begin splitting vivado impl and synth
finished basic structure of separate vivsynth and vivimpl classes
added parent class for vivado tools, changed flows accordingly
basic refactoring of unittests due to split of vivado synth and impl tools
refactored so build directories are created correctly on start of flow
fixed bug where vivado unit tests were failing by making sure static variable allowing rules to be written was reset correctly after each test
pylint ignore duplicate code warnings for flows' create build snippets methods

Closes #313


Help users when they need to use pg_dump: when a baselining is needed for a brownfield project.

We mention pg_dump in https://github.com/prisma/docs/blob/99c5858661dc5284fd9e565476b41993f271bf7d/content/300-guides/025-migrate/100-developing-with-prisma-migrate/050-add-prisma-migrate-to-a-project.mdx?plain=1#L93
In a brownfield project when a user wants to manage their database with Prisma Migrate, a baseline migration is required (because there is an existing database).
pg_dump can be used to create a reliable baseline migration, example
pg_dump --file=migration.sql --schema-only --dbname=mydb --host=localhost --username=myuser --password

⚠ The migration.sql dump needs to be edited and this following line removed.
SELECT pg_catalog.set_config('search_path', '', false);

If it is not removed, the migration will fail with the following error message:
Applying migration `20220510114355_baseline`
Error: P1014

The underlying table for model `_prisma_migrations` does not exist.

We could mention this somewhere in our docs.

What is the explanation behind this?

If it is not removed, the migration will fail with the following error message:
The underlying table for model _prisma_migrations does not exist.

How does that 1 SQL command lead to this error message?

Migrate relies on the search_path which is set to the current schema behind the scenes when connecting to the db.
Since the search path is modified during the migration, here to nothing, it causes this.

Doesn't that imply a potential bug in our logic reading _prisma_migrations, that we should specify the schema explicitly instead of relying on the search paths? (That would still work, correct?) Or alternatively, that we should detect such search path modifications and warn about them and maybe even prevent running migrate on such SQL files?

I'm getting similar errors. It was complaining that _prisma_migrations doesnt exist, it created it in the correct schmea, but then in created all my tables in the public schema instead of the one specified in my database url.
export DATABASE_URL=postgresql://${DB_USER}:${DB_PASS}@${DB_HOST}:${DB_PORT}/kick_the_can?schema=prisma_raw_events
npx prisma migrate deploy --schema=./apps/thred-events-inserter/prisma/schema.prisma
also in the migrations.sql file, I have to go public.create_hypertable, as it doesn't find create_hypertable otherwise. It's not really documented but i assumed if i specified a specific schema in the database url, everything would just work and be restricted to that schema.

Detecting a change to the search path, as @janpio suggested, would be great. I had exactly the problem described in this issue, and it took a while to find the cause. It might be worth detecting since this is the default behavior of pg_dump.
That said, migrations could probably do a number of things that screw up Prisma. However, changes from pg_catalog.set_config are only valid in the current session. So a safer (albeit slower) way would be to run each migration in its own session. That is, create a new database connection for each. This makes migrations more isolated, and thus there are fewer assumptions about what they can and cannot do.


postinstall.js fails due to write_ambient.js on arm architecture

Describe the bug
The import { write_ambient } from './write_ambient.js'; line is causing an error when running postinstall.js.
I have never had this error come up using Svelte, but am trying to use it on a Raspbian OS installation for the first time.  Something about this OS is not support in the write_ambient.js code.
Reproduction
Install Node.js on Raspbian Bookworm.
Create a new Svelte Kit project.
Run npm i or run the postinstall.js script directly.
Error: illegal instruction
Comment out the write_ambient.js import in the file: './src/core/sync/sync.js'
Error goes away.
I can't figure out what code in the write_ambient.js file is causing the error, but the error goes away as soon as you comment that import out.
Logs
Illegal Instruction

System Info
Operating System: Raspbian GNU/Linux 12 (bookworm)
          Kernel: Linux 6.1.0-rpi8-rpi-v6
    Architecture: arm

  System:
    OS: Linux 6.1 Raspbian GNU/Linux 12 (bookworm) 12 (bookworm)
    CPU: (1) arm unknown
    Memory: 306.51 MB / 428.89 MB
    Container: Yes
    Shell: 5.2.15 - /bin/bash
  Binaries:
    Node: 21.6.1 - ~/.nvm/versions/node/v21.6.1/bin/node
    npm: 10.2.4 - ~/.nvm/versions/node/v21.6.1/bin/npm
    pnpm: 8.15.1 - ~/.nvm/versions/node/v21.6.1/bin/pnpm

Severity
serious, but I can work around it
Additional Information
Can't use Svelte Kit on this OS.  Always used Mac OS or Fedora in the past.

I ran into the same error trying to install another library with a postinstall.js script as well.  So maybe this is related to the codebase that produces these postinstall files?  Where do these come from, or how do they get implemented in @svelte/kit?

I've tracked this down I believe.  Seems to be an issue with Node.js, where the cpuinfo detection fails to pick up arm6 architecture and builds for arm7.  One solution if you're running Node.js on something using arm6 is to just buy an arm7 device, for example, Pi Zero => Pi Zero 2.
If anybody experiences this problem with npm/node and other libraries, read this:
https://raspberrypi.stackexchange.com/questions/24059/node-js-v0-11-14-exits-with-illegal-instruction


Is there an example about regression task for texts with keras?

https://github.com/fchollet/keras/blob/master/examples/imdb_lstm.py
This one is for binary classification (sentiment analysis), but it should be quite similar for multiple categories if you use a softmax instead of the sigmoid layer and the categorical_crossentropy loss function.

@jrieke Thanks for your reply! I am afraid that my description is ambiguous. I want to know if there is an example to do regression task not classification.

Use a linear activation function at the last layer and MSE as your loss function. Most people also seem to use rmsprop as optimizer for regression tasks.

@rpinsler Thanks for your reply!

@rpinsler  I try what you said. But the result of model.predict() is same to result of model.predict_proba() like [ 0.08436067  0.06973539 -0.03013037 ...,  0.06828094  0.06003802
0.29175624] which is a output of a test sample. The range of regression is [0,9], apparently the result is not we want. So what's wrong with that we have done?and how to get correct output.


Bug: ImportError: cannot import name 'update_products_discounted_price' from 'saleor.product.utils.variant_prices'

What are you trying to achieve?
I recently upgraded my Saleor core from version 3.16.8 to 3.17.1 and encountered an ImportError issue. The specific error message is as follows:
ImportError: cannot import name 'update_products_discounted_price' from 'saleor.product.utils.variant_prices'
Upon investigating the issue, I found that the problem lies in the file saleor/discount/migrations/tasks/saleor3_16.py This file appears to have an import error, as it's referencing functions and models that may have been removed in the latest release (3.17.1).
Additionally, I noticed that the SaleChannelListing, SaleTranslation, and Sale models have also been removed in the latest release (3.17.1). However, these models are still being imported and used in the saleor3_16.py file.
Steps to reproduce the problem

Upgrade the Saleor platform from version 3.16.8 to 3.17.1.
Attempt to run the platform or execute relevant management commands.
Observe the ImportError and traceback message mentioned above.

What did you expect to happen?
I would like to raise this issue to bring it to the attention of the Saleor development team. The desired outcome is to have the saleor3_16.py file updated to reflect the changes in the latest release and to resolve the ImportError issue. This will ensure that users can successfully upgrade to version 3.17.1 without encountering this problem.
Logs
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/app/saleor/discount/migrations/tasks/saleor3_16.py", line 11, in
from ....product.utils.variant_prices import update_products_discounted_price
ImportError: cannot import name 'update_products_discounted_price' from 'saleor.product.utils.variant_prices' (/app/saleor/product/utils/variant_prices.py)
Environment
Saleor platform version: 3.16.8 (source) and 3.17.1 (target).
The issue appears to be located in the saleor/discount/migrations/tasks/saleor3_16.py file.
The issue is related to import errors and the use of deprecated models and functions.
This issue is preventing a successful upgrade to the latest version and is impacting the platform's functionality.
Environment Information:
Operating System: Ubuntu 22.04.3 LTS
Python Version: 3.9.18
Saleor Version: 3.16.8 (source) and 3.17.1 (target)

Hi @albinaj,
The file that is causing your problem saleor/discount/migrations/tasks/saleor3_16.py is not part of the version 3.17.1. It's only the part od the 3.16 version.
This file is required for proper data migration and any of the function from there shouldn't be run on version 3.17.
Are you sure that you have up to date changes from 3.17.1?

There is a chance, that when you were upgrading from 3.16.8 your celery was still processing the migration task, the one you mentioned to be precise. And this migration task stops being valid from 3.17 version onward.
When doing upgrade from version to version+1 we are doing zero-downtime, but that came with some downsides, please check: https://docs.saleor.io/docs/3.x/setup/upgrading#upgrading-with-zero-downtime
For now I would recommend to check if you have all Sales models correctly migrated to new Promotions, to verify if there is corrupted data and to confirm if my guess is correct.

Hi @kadewu @IKarbowiak
After removing saleor/discount/migrations/tasks/saleor3_16.py everything worked as expected.
I initially downloaded your latest release zip file (version 3.16.1) and have been actively working on adding features and customizing Saleor without altering any Saleor-specific files. This approach allows me to seamlessly incorporate future Saleor releases.
To the best of my knowledge, this process should go smoothly when I pull from Saleor branches. However, when I recently pulled version 3.17.1, I noticed that saleor/discount/migrations/tasks/saleor3_16.py was not removed, and new saleor/discount/migrations/tasks/saleor3_17.py files were added.
Manually checking and removing release-specific files is not a practical solution for my workflow. Do you have any suggestions for handling this situation in my use case?

Could you explain in details your process?
What do you exactly mean by

I recently pulled version 3.17.1

Does it means that you download the zip again and merging it by you own?
The best way is to use git pull and handle the merge with the use of git, otherwise you might have the problems with the files that were deleted.

@IKarbowiak I appreciate your interest in understanding my process. Let me clarify the steps I took in detail:
Git Pull: I did use git pull to update my codebase.
Separate Branch: At the time, Saleor's version was 3.16.8. I maintain a separate branch specifically for incorporating Saleor releases without any of my modifications.
Upstream Repository: I added Saleor's official repository as an upstream remote.
Git Pull from Upstream: To update to version 3.17.1, I executed the command git pull upstream 3.17.1 This action fetched the changes made in the official repository.
Merge: Following the pull, I merged the changes into my separate branch. I understand that this version had a significant number of changes.
Testing: Although I didn't thoroughly check every change, I did conduct testing. During this process, I noticed that saleor/discount/migrations/tasks/saleor3_16.py not removed as expected.
Feel free to correct me if I'm mistaken, but could you confirm if this is the correct method to obtain both release changes and minor updates?

The flow sounds right.
Maybe try to search or ask people for such case in a public forums like Stack Overflow, as it looks like that something went wrong during the merge process.
I hope you won't have such problems anymore in the future! 🤞


Make it possible to play direct uploaded/attached media content.

There is a method to do this with just getting the link to the file, but this shouldn't be needed.

This has been added to the planned features list


Fix documentation for generate_{service}_shared_access_signature

Hi @jofriedm-msft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jofriedm). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Tooltip auto position

Issue Number
fix #801
Description
It was required that the tooltip adjusted its position depending on the space it has avaliable, so I created a service with a subject to facilitate the comunication between the the directive and the component, so when the component verify the space around the host it notify the service so the directive knows if it has to call again the setPosition to fix the tooltip position.
How to Test
yarn test tooltip
Compliance

[X] I have verified that this change complies with our code and contribution policies.
[X] I have verified that this change does not cause regressions and does not affect other parts of the code.

@Thiago-Pio-Brisa