Bug: token count not being saved

Token count is not being saved when a request is made to open ai.

Thanks for reporting @kashifali11! Can you provide some details as this works for most cases. What API are you using?

I am using latest version of the sdk and openai version 4.11.1
this is the span object that I get
{
'gen_ai.system': 'OpenAI',
'llm.request.type': 'chat',
'gen_ai.request.model': 'gpt-3.5-turbo-1106',
'gen_ai.request.max_tokens': 4096,
'gen_ai.request.temperature': 0.7,
'gen_ai.request.top_p': 1,
'gen_ai.prompt.0.role': 'system',
'gen_ai.prompt.0.content': '',
'gen_ai.prompt.1.role': 'user',
'gen_ai.prompt.1.content': 'Your name is naruto',
'gen_ai.response.model': 'gpt-3.5-turbo-1106',
'gen_ai.completion.0.finish_reason': 'stop',
'gen_ai.completion.0.role': 'assistant',
'gen_ai.completion.0.content': "I'm not Naruto, but I'm familiar with the character! Naruto is a popular anime and manga series about a young ninja with big dreams. If you have any questions about Naruto or anything else, feel free to ask!"
}

Thanks @kashifali11! Any chance you're using the streaming API? Can you provide a small example of how you're calling OpenAI?

Any update on this?

Sorry @kocobee this slipped between the cracks 🤦
We're on it now. Can you provide a full snippet? This code for example provides token usage attributes.

Sorry for the confusion @nirga , I was using streaming api. Here is the code snippet
async function create_joke() {   const responseStream = await traceloop.withTask({ name: "joke_creation" }, () => {     return client.chat.completions.create({       model: "gpt-3.5-turbo",       messages: [{ role: "user", content: "Tell me a joke about opentelemetry" }],       stream: true     })   });   let result = "";     for await (const chunk of responseStream) {       result += chunk.choices[0]?.delta?.content || "";     }   return result; }

Thanks @kashifali11! Fix is on the way!


Update radial progress chart

Styling fix for the alignment of icon in the RadialProgressChart
Added new parameter for setting the size of the icon iconSize

Changes look good @whitetiger1399 but the ci/circleci: unusedDependency is failing, please check what that is about?

Changes look good @whitetiger1399 but the ci/circleci: unusedDependency is failing, please check what that is about?

Known issue. It's mostly due to unused dependency which circle-ci is not able to detect. And missing dependency visx/brush for which I have created am issue with visx


Added option to configure private registries.

Hi and thanks for amazing project.
I needed to connect my cluster to existing registries, but I couldn't find an option to do so.
This is my attempt to create such option.
Q&A

Why don't you create this file on node startup?

Because if you change registries.yaml contents later we need to reload all k3s services on all nodes.

Why do new resources depend on kustomization resource?

Because otherwise, when we create cluster for the first time and you provide custom registry config, the error may occur.
Since we need to restart k3s services after updating the config, it may create a conflict where kustomization target and new targets try to interact with k3s service at the same time.
I would really appreciate a feedback for this PR.
Signed-off-by: Pavel Kirilin<EMAIL_ADDRESS>

@mysticaltech, sorry, I requested rereview from you and it accidentally removed @ifeulner from reviewers.

Hi @s3rius thanks for contributing  - awesome! Just to get it right - the idea is to add/adjust the registry settings AFTER the cluster has been created, right? And you're restarting the node after applying the new setting. As there is no "drain" before, there will be at least possibilities to corrupt data, I think...
Also the autoscaler nodes are not getting the new registry setting, this is another problem.
So another alternative would be to set the registry settings when a node is created, and only then. This setting could also be used when the cloud-init for the autoscaler nodes is prepared.
But no change afterwards possible anymore...  but with the creation of new nodes a new registry setting could also be applied.
So I see some drawbacks with both ways ... what do you think @mysticaltech?

That's a good take. But reboot is necessary after updating registry.yaml. Also, I'm not sure if it's possible to update autoscaled nodes. Maybe adding it to cloudconfig would be a nice solution. But still, I'm not sure how to drain and reboot nodes after updates.

@ifeulner Right on spot, I completely missed the need for draining.
But there's an easy way around that issue. @s3rius, just do the exact same thing for the registry file as is done for the k3s config file, which is first created in /tmp/config.yaml (see https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/b8dcab2bc7b5fc1a7b16b596364c9ecb4df76274/agents.tf#L52), and then moved to the right k3s folder in   common_commands_install_k3s (see https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/b8dcab2bc7b5fc1a7b16b596364c9ecb4df76274/locals.tf#L14).
That way, registries.yaml will follow the exact same path as config.yaml. And this will work as the k3s docs says that Upon startup, K3s will check to see if a registries.yaml file exists at /etc/rancher/k3s/ and instruct containerd to use any registries defined in the file..

@s3rius If you need any help, please let me know, this is normally quite straightforward.

Thanks for offering help. I'm just chilling. I'll do it next week.

Hi, everyone. I rewrote this pull request using cloud-config as you told me. It works.

@s3rius Even better! Perfect. @ifeulner Any final comments before we merge?

Just released, thanks @s3rius!


Increase delays in Dataset commit integration test

Should fix failing tests. There's a race condition in how quickly the dataset commit worker does its thing
Will find a better solution if this fails again in the future

I considered asking about the delays when I last reviewed this test code.  It's not ideal, but we can revisit this later based on how often we'll be hitting this condition in the wild.

In production code, we might just use a loop with a shorter delay to watch for the dataset repository master setting (or commit) to appear, if later code is gated on those conditions.

Yup. There are a few sleep()s sprinkled throughout the tests, definitely not best-practice. Would be good to introduce helper functions for waiting on these, as we have for media processing. If these features see any usage, or tests fail again we can add such helpers


[5.3] Consistent message building for database driver

Since we have:
return (new MailMessage)
         ->subject(........);

and
return (new NexmoMessage)
        ->from(........);

and same for slack, I thought it'd be more convenient to build the Database message like this:
return (new DatabaseMessage)
        ->data([........]);

Since database doesn't have any other methods I just prefer to keep it in the constructor.


As a part of issue #3,  Implement the ability to activate a spectator mode "automatic slow rotation of the camera". Then implement the ability to de-activate said spectator mode "normal camera rotation functionality".

Task Description: Implement the ability to activate a spectator mode "automatic slow rotation of the camera". Then implement the ability to de-activate said spectator mode "normal camera rotation functionality".

The ability to activate a spectator mode "automatic slow rotation of the camera" is implemented.


Auto bookwork and The solution button doesnt always show the answer

Sorry to  bother you again for the 2nd time but autobook work doesnt work on opera as well as the solution button only somtimes gives the answer ill try restarting the extention but if oyu could look into it that would be great !

Sorry to bother you again for the 2nd time but autobook work doesnt work on opera as well as the solution button only somtimes gives the answer ill try restarting the extention but if oyu could look into it that would be great !

For the solution button, sometimes it can't find a solution and will return a blank page, there's not really much I can do about that. But could you explain more about the autobookwork not working

That’s alr if you can’t do anything abt the solution not working it’s alr I’ll just use guath ai for that but for the auto book work not working basically I start my spars maths and I get my first book work check it’s says bookwork not found for where it’s ment to tell me which one I’m supposed to press nothing more just gives me a book work not found message and then I have to guess which one to press On 3 Oct 2024, at 07:59, davedude1011 @.***> wrote:
Sorry to bother you again for the 2nd time but autobook work doesnt work on opera as well as the solution button only somtimes gives the answer ill try restarting the extention but if oyu could look into it that would be great !
For the solution button, sometimes it can't find a solution and will return a blank page, there's not really much I can do about that. But could you explain more about the autobookwork not working
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Yea I might have done maybe like 150 but I’m not too sure but possibly Sent from my iPhoneOn 3 Oct 2024, at 16:21, davedude1011 @.***> wrote:
ok ill look into it, would you say you've done over 150 sparx maths questions since the start of this week? it might be a storage capacity problem
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

also it looks like this

ok the issue is probably from storage limitations, ill release a potential fix on the next update 👍

thanks mate could i ask when the next update will be ?

ill try and fix the issue this week 👍

thanks mate could i ask when the next update will be ?

Hello,
I love your AI but will the solution button ever work?

It gives me answers that aren't correct

or like nothing shows

read form the top he cant do much about it wait try using gauth on  your phone or on your computer by using the snipping tool on windows search if you search it up  just snip the question and ctrl V into gauth ai

ok ive released v3.4 which should fix this issue


[Bug]: react/jsx-filename-extension ignoreFilesWithoutCode option not working with eslint v8

Is there an existing issue for this?

[X] I have searched the existing issues and my issue is unique
[X] My issue appears in the command-line and not only in the text editor

Description Overview

 ignoreFilesWithoutCode flag is not working with eslint v8 8.57.0. Please check the below Screenshot.

Expected Behavior

This ignoreFilesWithoutCode should work when passed in the config.
   "react/jsx-filename-extension": [
      "error",
      {
        extensions: [".tsx", ".jsx"],
        ignoreFilesWithoutCode: true
      }
    ],

eslint-plugin-react version
7.33.2
eslint version
8.57.0
node version
20.10.0

That's because it was added in v7.34.0 of the plugin; you're using v7.33.2. https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CHANGELOG.md#7340---20240303

Thanks for prompt response @ljharb 🙏🏼


Don’t crash when UIViewLazyList exhausts its placeholders queue

We should instead render some kind of spacer with the same height as our placeholders.

Duplicate of https://github.com/cashapp/redwood/issues/1500


[Direct Line] API removes "separator: true" property from Adaptive Card attachment?

Bot Info

App ID: 49790b66-712b-4338-8044-0bdb7099c7f0
SDK Platform: Node.js
SDK Version: 3.11.0
Active Channels: Direct Line
Deployment Environment: Azure App Service

Issue Description
When an adaptive card schema such as Flight Itinerary is sent to a client which is using directline, separator: true property is removed from directline.botframework.com before sending it to the client. I'm using botbuilder Node.js SDK on the backend and I've verified by stepping through the debugger until the POST request in made to DirectLine endpoint that separator: true property exists. This appears to be an issue at the Directline endpoint rather than the SDK.
Reproduction Steps

Create a bot and setup a dialog to return an adaptive card schema which has separator: true property.
Create a client using Botframework directlinejs to listen for activities.
Check the returned message object for separator: true property.

Expected Behavior
Directline 3.0 endpoint should not remove any properties from attachments object. Along with separator: true, I've found that it also removes width property.
Actual Results
Here's the JSON which is received on the client using directlinejs websocket which is missing separator: true property.
{
  "activities": [
    {
      "type": "message",
      "id": "BqY3Oa1a4sVGwDhw6bbq8C|0000006",
      "timestamp": "2017-11-07T21:05:14.3393491Z",
      "localTimestamp": "2017-11-07T21:05:14.746+00:00",
      "channelId": "directline",
      "from": {
        "id": "customBot",
        "name": "customBot"
      },
      "conversation": {
        "id": "BqY3Oa1a4sVGwDhw6bbq8C"
      },
      "inputHint": "acceptingInput",
      "attachments": [
        {
          "contentType": "application/vnd.microsoft.card.adaptive",
          "content": {
            "type": "AdaptiveCard",
            "body": [
              {
                "type": "TextBlock",
                "weight": "bolder",
                "text": "Passengers"
              },
              {
                "type": "TextBlock",
                "text": "Sarah Hum"
              },
              {
                "type": "TextBlock",
                "text": "Jeremy Goldberg"
              },
              {
                "type": "TextBlock",
                "text": "Evan Litvak"
              },
              {
                "type": "TextBlock",
                "weight": "bolder",
                "text": "2 Stops"
              },
              {
                "type": "TextBlock",
                "weight": "bolder",
                "text": "Fri, October 10 8:30 AM"
              },
              {
                "type": "ColumnSet",
                "columns": [
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "isSubtle": true,
                        "text": "San Francisco"
                      },
                      {
                        "type": "TextBlock",
                        "size": "extraLarge",
                        "color": "accent",
                        "text": "SFO"
                      }
                    ],
                    "style": null
                  },
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "text": " "
                      },
                      {
                        "type": "Image",
                        "size": "small",
                        "url": "http://messagecardplayground.azurewebsites.net/assets/airplane.png"
                      }
                    ],
                    "style": null
                  },
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "isSubtle": true,
                        "text": "Amsterdam",
                        "horizontalAlignment": "right"
                      },
                      {
                        "type": "TextBlock",
                        "size": "extraLarge",
                        "color": "accent",
                        "text": "AMS",
                        "horizontalAlignment": "right"
                      }
                    ],
                    "style": null
                  }
                ]
              },
              {
                "type": "TextBlock",
                "weight": "bolder",
                "text": "Non-Stop"
              },
              {
                "type": "TextBlock",
                "weight": "bolder",
                "text": "Fri, October 18 9:50 PM"
              },
              {
                "type": "ColumnSet",
                "columns": [
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "isSubtle": true,
                        "text": "Amsterdam"
                      },
                      {
                        "type": "TextBlock",
                        "size": "extraLarge",
                        "color": "accent",
                        "text": "AMS"
                      }
                    ],
                    "style": null
                  },
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "text": " "
                      },
                      {
                        "type": "Image",
                        "size": "small",
                        "url": "http://messagecardplayground.azurewebsites.net/assets/airplane.png"
                      }
                    ],
                    "style": null
                  },
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "isSubtle": true,
                        "text": "San Francisco",
                        "horizontalAlignment": "right"
                      },
                      {
                        "type": "TextBlock",
                        "size": "extraLarge",
                        "color": "accent",
                        "text": "SFO",
                        "horizontalAlignment": "right"
                      }
                    ],
                    "style": null
                  }
                ]
              },
              {
                "type": "ColumnSet",
                "columns": [
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "size": "medium",
                        "isSubtle": true,
                        "text": "Total"
                      }
                    ],
                    "style": null
                  },
                  {
                    "type": "Column",
                    "items": [
                      {
                        "type": "TextBlock",
                        "size": "medium",
                        "weight": "bolder",
                        "text": "$4,032.54",
                        "horizontalAlignment": "right"
                      }
                    ],
                    "style": null
                  }
                ]
              }
            ],
            "speak": "Your flight is confirmed for you and 3 other passengers from San Francisco to Amsterdam on Friday, October 10 8:30 AM",
            "version": "1.0"
          }
        }
      ],
      "replyToId": "BqY3Oa1a4sVGwDhw6bbq8C|0000005"
    }
  ],
  "watermark": "6"
}

Hi,
Any resolve for this issue? I'm having a similar issue.

Thank you for your submission. The Microsoft Bot Framework team prefers that bug fixes and feature requests were opened as new issues on the official Bot Framework Github repo. The Stack Overflow botframework tag is a place to ask how to questions and get support from the community. Please feel free to submit an issue on GitHub, and the dev team will look into your request.


find() should work like find(1)

I expected the following to work... but took a look at the docs and require('shelljs').find quite a bit different than I expected.
Here is what I'm trying to do:
// find all directories in the cwd that have a package.json
var sh = require('shelljs');
var packages = sh.find('*/package.json').map((f) => {
 return path.dirname(f)
});

This is obviously not possible with sh.find. You have to do something like
sh.find('.').filter((p) => {
  return path.dirname(p) === cwd
      && path.filename(p) === 'package.json'
});

Would be nice if it supported the same functionality as find(1).

The parameter to find is a path, not a pattern. This functionality should be provided via the -name parameter instead, see https://www.gnu.org/software/findutils/manual/html_mono/find.html#Base-Name-Patterns


No pricing information for query - Evaluating the legacy Per Node pricing tier

Under "Evaluating the legacy Per Node pricing tier"
the query example for comparing pricing has a link to
https://azure.microsoft.com/en-us/pricing/details/monitor/
to gather details.
That link does not contain information on per-node or per-GB pricing, so the query cannot be utilised without this data.

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

ID: fed35a55-0db9-4afa-9a80-fe9adac1e7d7
Version Independent ID: 67af640b-9a6c-bf24-3a99-14ad8dab849f
Content: Manage usage and costs for Azure Monitor Logs - Azure Monitor
Content Source: articles/azure-monitor/logs/manage-cost-storage.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@darrenjoycenz Thanks for your comment. We will review the issue and get back to you shortly.

Assigning to content author for doc enhancement

#assign: GuyWi

The following users don't have valid GitHub IDs, or are not collaborators on this repo: guywi

#reassign: @guywi-ms

@darrenjoycenz Thanks very much for your inquiry!
For Pay-As-You-Go (per-GB) pricing details, see https://azure.microsoft.com/pricing/details/monitor/.
You can see your per-node costs in your Azure usage and charge data. For more information, see https://docs.microsoft.com/en-us/azure/cost-management-billing/understand/download-azure-daily-usage.
We've submitted a documentation change to clarify this issue, and the updated article should be live by tomorrow.
Please let us know if there's anything more we can do for you.
To follow up on this issue, leave us a comment with an @mention.
#please-close


integration start

#13 Relating to this issue.
Changed a bunch of files, please go through the changes, will have to add clear after each screen is displayed and need to ensure all functions are working as intended.

The fixes on atm_operator.py seem fine.


ILB ASE certificates

the documentation on this page should be updated to reflect that uploading a certificate for an ILB ASE is only for ASE's created before May 2019 as specified here:  https://docs.microsoft.com/en-us/azure/app-service/environment/create-ilb-ase#ilb-ases-made-before-may-2019

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

ID: 5f3a8caa-796a-18ab-480c-161e6c806782
Version Independent ID: 9b8988f6-79d9-4558-5c0c-983ceb857a76
Content: Certificates bindings - Azure App Service Environment
Content Source: articles/app-service/environment/certificates.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@mapoitras Thank you for your feedback! We will review and update as appropriate.

@Thanks again for the feedback! The PR has been created for this, once verified by author would be visible.
We will now close this out but if you feel you need more information please just let us know


Cannot read property 'getBoundingClientRect' of null on your examples page.

✔ BUGS → This form is required:
Steps
I could not track it down.
It is available on your site:
https://react.semantic-ui.com/layouts/homepage
Expected Result
Responsiveness
Actual Result
Crashing between 768px and 992px width and less than 320 px.
 I think it's because the source is only considering mobile and desktop and crashes on every resolution that falls between all other categories.

Confirming this. fixed by adding  minWidth to the `Responsive' component.


Updated Grammar

Capitalized the word "princess" 😉

Can you resubmit this, but make the merge request to the dev branch?


Context.writeline for array or List

Can we have an override that accepts a string array or List and prints the collection out, one line per item?

You can write your own extension method for whatever kind of arguments you need. It is not even required to be defined in the same assembly.
But I feel that it is a quite taste-specific case. One would like to print them one line per item, the other prefers comma- or semicolon-separated values, and someone would like a bullet list at all.
So I’d rather add neither of those, leaving anyone the opportunity to write it the way they want, using the existing basic overloads..


Add UI keyboard navigation

Objective
Add keyboard navigation to bevy_ui.
Solution
Add a Focused resource that contains the currently focused entity, which is changed by Tab, Shift+Tab or clicking on an entity. Entities that are visible and have FocusPolicy::Block can be navigated to. To aid styling, a new FocusedState component has been added to each button that is updated when it is focused.

Changelog
Added

Focused resource containing the currently focused entity and the focus visibility (similar to the :focus-visible css pseudo-class).
FocusState component containing the current focus state of the entity to aid with styling.

@ndarilek I'd like your opinion on how this can integrate with AccessKit :)

I'd like your opinion on how this can integrate with AccessKit :)

It looks like bevy would need to send the relevant focused entity with the TreeUpdate event.

I have added access kit integration.

We shouldn't hard-code keybindings for this. Especially Tab is a common keybinding used in games, so this might collide with other controls. We can keep these values as defaults, but this should be configurable.

I can't really think of any other keybindings for keyboard navigation (other than arrow keys - but they are seperate), but I agree that we would need some API for stopping the tab key from modifying the UI if it has modified the game. How would you suggest allowing this to be configured @TimJentzsch?

For example, using the arrow keys and gamepad input. I'm not sure if we should have these relatively common use cases built-in as well, but at some point we should at least add the possibility for the user to add this. Otherwise, we might get some interactions that don't properly update the focus state and screenreader info if the devs don't know about the details.

I agree that arrow key navigation is very important for lists or for a row of buttons in a dialogue menu. In order to handle arrow key navigation, the game would have to declare a node as being a navigable row or column. Do you have any thoughts on what the API could look like?

How would you suggest allowing this to be configured?

Maybe a FocusSettings resource could work. It could store the key bindings for navigation (if we want them to be configurable) and perhaps a bool that can be used to disable the behavior if necessary. I'm curious to hear what other people think about this.

In order to handle arrow key navigation, the game would have to declare a node as being a navigable row or column. Do you have any thoughts on what the API could look like?

This is probably hard to solve for the general case... although it would be nice if we could somehow provide this functionality if needed. We also have to keep internationalization in mind -- in the future, the locale could affect how the UI is rendered, e.g. in some languages the text is layed out vertical so menus might be horizontal and the other arrow keys would make more sense for navigation.
I can't think of a nice API of the bat, but maybe we can at least make it easy for the devs (or plugins) to implement their own system on top of this. E.g. triggering an event that can change the focus to a given entity.

Can confirm it's hard to solve for the general case. This is what https://lib.rs/crates/bevy-ui-navigation does. And there even was An RFC (RFC41) which I had to close due to feature creep on my side.

I didn't realise that navigation could be so complex!
I'm sorry that I duplicated some of your code; I managed to completely miss it when searching for keyboard navigation. But perhaps some very basic built in tab navigation is still a useful stopgap measure until the more complete implementation is added?

I'm taking a look at your PR and I think it solves a few of the problems in my impl! :sweat_smile: Given the time I spent on this, I'm a bit ashamed…
I'll take a more careful look at the code tomorrow, but it seems like a good start. and not victim of the feature creep bevy-ui-navigation has.

Yeah, my view is that now that we've taken a look at the problem space more broadly we're well equipped to get an incremental start in and build from there :)

It's been a while since I did the AccessKit work but I believe I added a Focus resource which stores the entity of the widget with keyboard focus, which should correctly update the tree with the focused widget.
Note that this is keyboard focus. I.e. mousing over a widget should not update this resource. I suspect keyboard and gamepad are interchangeable here.
Thanks for caring about accessibility.

I suspect keyboard and gamepad are interchangeable here.

They are :) The element focused should always be the same, the only difference is how you navigate between focused elements.

i dont know how feasable this actually is, but the way i would solve this is ordering the buttons by the global transform, larger global transform goes at the end of the list and as you press tab it slowly goes through the list of buttons, top too bottom, left too right, as far as im aware most ui goes this way, if buttons are on the same Y plane then the X plane is used for ordering, buttons shouldnt be on top of each other yes?

Might be worth looking at Godot's APIs/handling of this.
I believe, and it's been a while since I've used Godot, that it has both an auto-placement API as well as a way to explicitly tag up/down/left/right/next/previous focus from any given widget. So if the auto layout algorithm does something that makes no sense, you can explicitly tag widgets to receive focus when d-pad/tab/shift-tab controls are used on a widget-by-widget basis. That's what I'd suggest here. An auto-layout algorithm will probably get it right some large amount of time, but there will always be corner cases it misses or handles poorly.

I would suggest also taking a glance at RFC 41, the discussion on the RFC and the ui-navigation API. There is a lot of edge cases involved, and it would be a shame to miss the ones we already uncovered.
I'd be happy to open a pr on 0HyperCube/bevy. I can cook up something tomorrow. Please tell me if you'd like that.

Thanks for the code review @nicopap. I've had a read through of the RFC and I can't see anything that is directly related to pitfalls in the current, very basic document flow based tab navigation (other than a lot of missing features); although I would be very happy for you to add commits / PRs for anything.

@0HyperCube Looks like cargo fmt is failing in CI

Could the existing component Interaction and the new Focusable be merged into one? Focus seems like a kind of interaction

not a fan of mixing keyboard and mouse navigation / focus / interaction with the same components, this often have surprising side effects.
In the example, if I click on one of the button, then move my mouse to another and press space, the first button gets clicked with no indication it's going to be the one selected

not a fan of mixing keyboard and mouse navigation / focus / interaction with the same components, this often have surprising side effects.
In the example, if I click on one of the button, then move my mouse to another and press space, the first button gets clicked with no indication it's going to be the one selected

This is the standard behaviour on the web @mockersf; although I'm open to suggestions to improve this behaviour.

Could the existing component Interaction and the new Focusable be merged into one? Focus seems like a kind of interaction

We could potentially do something like:
struct Interactable {
    focus_state: FocusState,
    hovered: bool,
    clicked: bool,
}

However, note that not all interactable UI nodes should be keyboard focusable. In HTML this can be done with tabindex="-1" which is used on dialogues (before the <dialog> tag) and to provide keyboard navigation for complex custom widgets where focus needs to be controlled by code.

Any chance that this could make it to 0.14?

Any chance that this could make it to 0.14?

I would love to see this get added

Trying to use some of the design from this in the lastest stable bevy 0.14.2, and it would seem that setting Focus() to some UI node (such as a text box UI element, or button) is causing some sort of race condition with the a11y accesskit node tree (Focused node id not in node list panic)
myapp::ui::navigation: setting focus state to Focusable { focus_state: Focused { visible: true } } for ent Some(Entity { index: 4, generation: 1 })
thread 'main' panicked at C:\Users\conno.cargo\registry\src\index.crates.io-6f17d22bba15001f\accesskit_consumer-0.22.0\src\tree.rs:36:13:
Focused id #4294967300 is not in the node list

Do you have a minimal repro? Hoping to do an accesskit improvement pass this cycle and would like to address this.
Thanks.


Error while paying with crypto: value "4586059860" is out of range for type integer

Describe the bug
Error while paying with crypto: value "4586059860" is out of range for type integer
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/monitorcontrol
Click on Crypto Contribution
Proceed with any amount, either personal or org
See error

Screenshots

@SudharakaP I can reproduce this one. The full error message is:

Corresponding sentry issue: https://open-collective.sentry.io/issues/3301058358/.
I suppose it's happening around that place, maybe The Giving Block returns a broken fx rate?


Do we need to stop gradient when computing alpha_loss and policy_loss?

Hi Young,
I noticed that in the dopamine's implementation, it stops gradient for alpha when computing the policy loss to avoid the gradient changing the value of alpha.
alpha_value = jnp.exp(jax.lax.stop_gradient(log_alpha))
policy_loss = jnp.mean(alpha_value * action_log_prob - no_grad_q_value)

https://github.com/google/dopamine/blob/1cbed6c7c35163e73b0ee2f2d2bf032b057570dd/dopamine/jax/agents/sac/sac_agent.py#L170
Similarly, it stops gradient for the policy when computing the alpha loss to avoid affecting the policy.
alpha_loss = jnp.mean(log_alpha * jax.lax.stop_gradient(entropy_diff))

https://github.com/google/dopamine/blob/1cbed6c7c35163e73b0ee2f2d2bf032b057570dd/dopamine/jax/agents/sac/sac_agent.py#L175

We don't need to do it here because we never add up the loss from different parts of our model. Instead, we keep a dictionary of all the individual losses, and take gradient for each one separately (in line 281). This paradigm allows us to freely compose different networks without having to worry about extra gradient accumulated coming from other losses.

Thank you. That's a nice design.


[AutoPR resources/resource-manager] [Test Only] Add 404 for HEAD APIs.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6912

(message created by the CI based on PR content)


Connection issue

[2019-05-19 20:51:29.285528 - ERROR] Connection failed while loading product on https://www.ebay.com/itm/

I am also receiving the same error message as of today 06/06/2019. I am investigating and If i find a solution I will post.

I am getting a different type of connection error.
2019-06-08 00:25:44.895044 - ERROR] Connection failed while creating an account with email<EMAIL_ADDRESS>If I need to configure anything else for this to work please do let me know!
Thanks!


ModuleNotFoundError: No module named 'fcntl' on Windows 10

Os: Windows 10
Python: Python 3.11
Terminal: Powershell (Windows terminal)
How to reproduce : type 'jpterm' in prompt
Hi,
I wrote in Twitter but i think its more appropiate here.
Your library use the module 'fcntl' that is not available for Windows platforms

Thanks for reporting the issue, #50 should fix it.

I confirm but I had to manually uninstall the txl dependencies since after uninstaling jpterm old ones were still there and install did not detect they were obsoletes
What i did:

pip install -U jpterm => still the same issue
pip uninstall + restart terminal + pip install git+https://github.com/davidbrochart/jpterm => still the same problem
pip uninstall + manually uninstall all txl-* packages + pip install git+https://github.com/davidbrochart/jpterm => no problem


dataSources() DSN truncation

Environment

Python: 3.7.6 (64-bit)
pyodbc: 4.0.30
OS: Windows 10
DB: MS SQL Server 2012
Driver: ODBC Driver 17 for SQL Server

Issue
Observed behavior:

dataSources() truncates DSN to 31 characters if DSN is 32 characters long

Expected behavior:

dataSources() does not truncate DSN if it's 32 characters long

Suspected cause:
szDSN array in mod_datasources() doesn't take into account extra space needed for null-terminator, therefore DSN names that utilize maximum available space of 32 characters (defined by SQL_MAX_DSN_LENGTH) get truncated.
Proposed solution:
Declare szDSN array using SQL_MAX_DSN_LENGTH + 1 to accommodate extra space needed for null-terminator.

PR submitted #719


How do you get the "Maximum" possible TED ?

How do you get the "Maximum" possible TED ?
The TED is integer.
I want to get in some way the "maximum" TED between two trees.
The reason is I need a score with value between 0 and 1 i.e.  score = distance/max
Is this feasible ?

Is this a good way ?
nc1 = tree1.node_count()
nc2 = tree2.node_count()
score = ted /  max([nc1, nc2])

I don't know how to get the actual maximum, but I would guess it should be something like nc1 + nc2 instead of max(nc1, nc2). If you have two completely distinct trees, the TED would probably be bigger than the size of one of them.


Creating Persistent Volumes

I had to manually create persistent volumes in order to get this example to work. Is that expected, or is there likely something wrong with my configuration of minikube?
If it's expected that the user should manually create the persistent volumes, I'd be happy to update the instructions and share my example. Here's what I did:
kubectl create -f pv.yaml

# pv.yaml
kind: PersistentVolume
apiVersion: v1
metadata:
  name: pv0
  labels:
    type: local
spec:
  storageClassName: default
  capacity:
    storage: 10Gi
  accessModes:
    - ReadWriteOnce
  hostPath:
    path: "/tmp/data/pv0"
---
kind: PersistentVolume
apiVersion: v1
metadata:
  name: pv1
  labels:
    type: local
spec:
  storageClassName: default
  capacity:
    storage: 10Gi
  accessModes:
    - ReadWriteOnce
  hostPath:
    path: "/tmp/data/pv1"
---
kind: PersistentVolume
apiVersion: v1
metadata:
  name: pv2
  labels:
    type: local
spec:
  storageClassName: default
  capacity:
    storage: 10Gi
  accessModes:
    - ReadWriteOnce
  hostPath:
    path: "/tmp/data/pv2"

@blachniet how did you tie back the PVs to the stateful set? Since even after creating the PVs I am getting the same error as before i.e.

SchedulerPredicates failed due to PersistentVolumeClaim is not bound: "data-consul-0", which is unexpected., SchedulerPredicates failed due to PersistentVolumeClaim is not bound: "data-consul-0", which is unexpected., SchedulerPredicates failed due to PersistentVolumeClaim is not bound: "data-consul-0", which is unexpected.`

I don't think that there was anything else I needed to do in order to tie the stateful set to the PVs. The persistent volumne claims in the stateful set should match to the PVs created by this configuration.

@blachniet yes, I met this problem as well, and manually created 3 PVs to make it work
@tankchintan agree with @blachniet , PVC will automatically select from the PV pools (make sure you have 3 available, also I don't set  storageClassName: default


Update depth_and_complexity.md

I think this sentence in the doc is grammatically wrong.
Updated based on the Chinese version.

Thanks! 🙂


[Future] Option to get decoded urls in log

Hello! Flask logs urls as encoded strings. For example if I have route with cyrillic characters: @app.route('/привет') (https://gist.github.com/dima74/5e025da90ee90a78b65bc3e3c69623ea), than flask will log "GET /%D0%BF%D1%80%D0%B8%D0%B2%D0%B5%D1%82 HTTP/1.1". It would be very nice if there is option to get that Flask logs urls decoded. (Thanks for Flask!)

Please report this against Werkzeug. Flask does not provide the dev server.

OK, thank you a lot.


21BCAB35

Issue/Feature Description:
Authentication Error
Why this issue to fixed / feature is needed(give scenarios or use cases):
When trying to login it shows no proper internet connection.
How to reproduce, in case of a bug:
Try refreshing or change the network connection.
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)
@kalaiselvi76

@kalaiselvikks76


:bug: Improve delete flow

Move from AlertDialog to Dialog so we have more control over the modals (and don't close them prematurely)
Move Replication update context into an async Task

#271  👈
main

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

Join @acco and the rest of your teammates on  Graphite


#2 build landing page

Fixes #2 (Build landing page)
Changes

implement routing in App.js
change folderstructure
build home, modeler and error pages

Screenshots
Before: No Frontend (only "first-react-app" template")
After:
Screenshot Welcome Page:

Screenshot Modeling Page:

Checklist before merge
Developer's responsibilities

[ ] Approved by at least one developer: enter reviewer here
[ ] Pull request build has passed
[ ] Production backend has been updated to reflect new features and fix bugs
[ ] tested locally (in all browsers if frontend)
[ ] updated the docs
[ ] added tests where necessary

Product Owner's responsibilities

[ ] No issues have been found on staging environment
[ ] Product Owner has approved the release

Seems to not work. Please see the attached picture. Additionally there are still the node modules present on the branch

Both problems should be fixed, please review again


Ensure repo is back at master, and clean

Fixes #390
Supersedes https://github.com/fastlane/ci/pull/391

Merging even with failure because I need to investigate failure using a test branch


SCVMM - VMs without a VM XML config file cause parser error

This occurs in VMs without a VHD.
The static data we use for SCVMM refresher rspec testing does not contain virtual switch/logical network data therefore no spec test can be added for this PR. I have created a separate github issue for updating the static inventory file: #2820
https://bugzilla.redhat.com/show_bug.cgi?id=1217195

@blomquisg @Fryguy @roliveri  - I am setting the location property for diskless VMs to "unknown" like we do for templates. Can either of you think of any repercussions of doing so?

I think that should be fine.


Disable frameguard using frameguard(false)

Hi @EvanHahn, I'm using helmet for other modules and by default frameguard is always set to 'SAMEORIGIN'. I can see in the documentation here : https://helmetjs.github.io/docs/ that we should be able to disable frameguard, but I wasn't able to do it.

frameguard is both a standalone module and one of the pieces of the helmet package included by default.
If you're using the helmet package, you can do it like this:
app.use(helmet({
  frameguard: false
}))

If you're just using frameguard (in other words, you ran npm install frameguard), then you can just not include it. You can remove any line like app.use(frameguard()). This package doesn't have a way to disable it this way.
Does that help?

I haven't heard back on this in a few months so I'm going to close. Let me know if you want me to reopen!


Adding content to "Average cycle page"

In this page we want a horizontal number slider something like this

I would like to work on this

Alright then, GO GO GO !!

Hey, @Samriddhi25 what is the status, are you still working on it?

Hey, @Samriddhi25 what is the status, are you still working on it?

yes


OV2640 functions example - upside down and left to  right

Hello
Could someone (or Lee) please explain?   When I run ArduCAM_Mini_2MP_OV2640_functions   with the windows host and set to  320_240 and jpeg,   I get quite a nice single shot photo.
However, if I tick "Set ToBMP"  (because I want  RGB565 output) ,   I still get a picture  but it is  upside down and left to right?
Its not a big  problem as I can write code to reverse it, but would lie to understand why, before delving deep into the code.
Thanks

Hi,
Are you able to get a valid BMP image from the host app? In my case, when I inspect the BMP image, its just a JPG stored with BMP extension.
Can you inspect your BMP images in hex encoding and validate?
Thanks!

Hi,
I used the host app only to verify things were working   visually as I always  intended to display to a TFT.
So I did not investigate the host app too much
As the BMP image was clean although wrong orientation,  I proceeded to develop my code (and also got no response here).
However when I started to read the BMP image from the FIFO (buffer)  and display it on my TFT,  I found that the image was only left to right and not upside down.
So pixel (0,0)  was stored in the FIFO at  (319,0)  as I read out the pixels.
and  pixel (0,1) was at (319,1) and so on.....
So instead of displaying to the TFT at (x , y )  it is displayed  at   ( 319-x ,  y ).
So the image on the TFT was exactly how I expected it....   not upside down not left to right.
So I just remember to put (319-x)  where I expected to only have x !!
Back to the host app  ......  at   320x240....

a jpeg image is about   17,000 bytes

a BMP image is about     153,600 bytes   (320x240x2)   plus the header data that the Host app needs of about 160 bytes   and some terminating (I  think about 2)  bytes.

so if the host app byte counter is   incrementing about 154,000 bytes each image,  you are getting a full BMP at 320x240 pixels.
Hope all that made sense.


Release to pypi

The current release is v1.8 but on PyPI it's still 1.5.2.

Hi @Changaco, thanks for the reminder, and sorry for the delay getting back to you. We've just updated the PyPi pacage too.

Thanks, 10 days isn't that long a delay (especially in august), and I had a workaround so it was a minor inconvenience.


oath of thunder paladin interactions: mjolnir

I was trying out the oath of thunder paladin subclass because it sounded like a fun Thor spinoff, and I found an interaction with my other abilities that I can't really figure out if its intentional or not.
Mjolnir has a weapon requirement of war hammer type weapons that reads: "You have disadvantage to attacks with any  other weapon type different from war hammers"
which makes sense, because thor, mjolnir, hammer, etc... however as per the text reads this should only apply to weapon attacks, and also I still have a war hammer equipped anyway, but the interaction seems to also be affecting some spells/abilities that cause damage to enemies like the paladn's own channel divinity: divine bolt, as well as the sacred flame spell he has. Here's some screenshots that show the disadvantage modifier showing up on the target preview on the left side.

Please share your save to simplify debug, fix, qa

here's a copy of the save file, not sure if you only require this or something else as well.
solasta save fle.zip

It looks like Odin messed up. Will check.


Update CHANGELOG for v2.36

I've compiled the list of changes and updated our CHANGELOG.md accordingly.

Nice! I'll update these.
The intention was to give more updates so that we can easily delete unnecessary ones. :+1:

Great VSCode auto formatted the entire CHANGELOG.md. Fixing.

/prombench v2.35.0

/prombench cancel


This crate will increase the size of wasm by about 100KB

My whole program is 33 KB after compilation, but after adding this crate, it becomes 130 KB
before
-rw-r--r--  1 axetroy  staff    33K  5 14 16:31 wasm_hasher_sha1_bg.wasm

after
-rw-r--r--  1 axetroy  staff   128K  5 14 16:34 wasm_hasher_sha1_bg.wasm

I think it is acceptable to increase within 10KB, but why increase so much
Here is my Cargo.toml
[package]
name = "wasm-hasher-sha1"
version = "0.1.2"
authors = ["Axetroy<EMAIL_ADDRESS>edition = "2021"
repository = "https://github.com/axetroy/wasm-hasher"
readme = "README.md"
description = "computed data hash by webAssembly"
license = "Anti-996"
keywords = ["sha1", "wasm", "webassembly"]

[lib]
crate-type = ["cdylib", "rlib"]

[features]
default = ["start", "wee_alloc"]
start = ["console_error_panic_hook", "tracing-wasm"]

[dependencies]
console_error_panic_hook = { version = "0.1.7", optional = true}
tracing-wasm = { version = "0.2.1", optional = true }
# The `wasm-bindgen` crate provides the bare minimum functionality needed
# to interact with JavaScript.
wasm-bindgen = "0.2.85"
sha1 = "0.10.5"
js-sys = "0.3.62"
digest = "0.10.6"
wasm-bindgen-futures = "0.4.35"
# `wee_alloc` is a tiny allocator for wasm that is only ~1K in code size
# compared to the default allocator's ~10K. However, it is slower than the default
# allocator, so it's not enabled by default.
wee_alloc = { version = "0.4.2", optional = true }
wasm-hasher-lib = { path = "../lib" }

# The `web-sys` crate allows you to interact with the various browser APIs,
# like the DOM.
[dependencies.web-sys]
version = "0.3.61"
features = ["console", "FileReader", "Blob", "AbortSignal"]

# These crates are used for running unit tests.
[dev-dependencies]
wasm-bindgen-test = "0.2.45"
futures = "0.1.27"
js-sys = "0.3.62"

@axetroy - Did you have any other discoveries about what was going on here? I'm personally unsure how I can debug this issue.

@axetroy - Did you have any other discoveries about what was going on here? I'm personally unsure how I can debug this issue.

Sorry, I can't help you with this, I don't know much about it


Rules Search Improvement

Would we be able to add a link to the search itself when using $search? So I can click through and be taken to the NRDB search page with those parameters.
EDIT: This feature is currently available but only on searches with more than 5 results.

Thanks for the suggestion! It's been patched in.


CloudWatch events are not handled by Zappa

CloudWatch events triggering lambda functions are not being handled by Zappa. The events contain a json with the following format: {'awslogs': {'data': 'Dmt.....EA'}} which fails to match any of the conditions in the LambdaHandler.handler function.
Context

When configuring CloudWatch to trigger a Lambda function deployed by Zappa, I can see Zappa receiving the event and calling LambdaHandler.handler():
START RequestId: 850...e9a Version: $LATEST
Instancing..
[DEBUG] 2019-08-30T18:02:42.685Z 850...e9a Zappa Event: {'awslogs': {'data': 'H4sIA...AAA='}}
END RequestId: 850...e9a
REPORT RequestId: 850...e9a Duration: 2.53 ms Billed Duration: 100 ms Memory Size: 512 MB Max Memory Used: 67 MB

but my configured handler function is not called.
Expected Behavior

My configured function in zappa_settings.json should be executed 🌟
Actual Behavior

My function is not executed ⛈
Possible Fix

Add a new if condition in the handler function to look for the awslogs key in the event object/dict and trigger the configured handler function.
Steps to Reproduce

Create and deploy a lambda function with Zappa. Add a simple print statement in the handler to confirm that the function is triggered.
Create a CloudWatch log subscription filter that triggers the lambda function (see here)
Push data to the log that triggers the subscription filter:  aws logs put-subscription-filter \  --log-group-name myLogGroup \  --filter-name demo \  --filter-pattern "" \  --destination-arn arn:aws:lambda:region:123456789123:function:helloworld
The lambda function will be executed, but the handler function will not.

Your Environment

Zappa version used: 0.48.2
Operating System and Python version: Python 3.6.5
The output of pip freeze:

argcomplete==1.9.3
boto3==1.9.195
botocore==1.12.195
certifi==2019.6.16
cfn-flip==1.2.1
chardet==3.0.4
Click==7.0
docutils==0.15.1
durationpy==0.5
future==0.16.0
hjson==3.0.1
idna==2.8
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.20.0
placebo==0.9.0
python-dateutil==2.6.1
python-slugify==1.2.4
PyYAML==5.1.1
requests==2.22.0
s3transfer==0.2.1
six==1.12.0
toml==0.10.0
tqdm==4.19.1
troposphere==2.4.9
Unidecode==1.1.1
urllib3==1.25.3
Werkzeug==0.15.5
wsgi-request-logger==0.4.6
zappa==0.48.2

Link to your project (optional):
Your zappa_settings.py:

APP_MODULE='notify'
APP_FUNCTION='handler'
EXCEPTION_HANDLER=None
DEBUG=True
LOG_LEVEL='DEBUG'
BINARY_SUPPORT=True
CONTEXT_HEADER_MAPPINGS={}
DOMAIN=None
BASE_PATH=None
ENVIRONMENT_VARIABLES={'AWS_REGION': 'us-east-1'}
API_STAGE='prd'
PROJECT_NAME='test-zappa-production'
SETTINGS_FILE=None
DJANGO_SETTINGS=None
AWS_EVENT_MAPPING={}
AWS_BOT_EVENT_MAPPING={}
COGNITO_TRIGGER_MAPPING={}
ASYNC_RESPONSE_TABLE=''

I just noticed that updating via zappa removes the Cloudwatch logs trigger too.


Cannot start crouton with sudo startxfce4

sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
ln: failed to access '/var/run/crouton/mnt/stateful_partition/crouton/chroots/xenial/etc/resolv.conf': Permission denied
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Please describe your issue:
Crouton fails to load since ChromeOS updated recently to 67.0.3376.
Attemptiong to run an update on my install gives:
Preparing chroot environment...
Installing brightness into the chroot...
Installing croutonpowerd into the chroot...
Installing croutonversion into the chroot...
Installing host-dbus into the chroot...
Installing host-wayland into the chroot...
Installing volume into the chroot...
Installing pulseaudio-default.pa into the chroot...
Installing croutonclip into the chroot...
Installing croutonnotify into the chroot...
Installing croutonurlhandler into the chroot...
Installing startxiwi into the host...
Installing croutoncycle into the chroot...
Installing croutonfindnacl into the chroot...
Installing croutontriggerd into the chroot...
Installing croutonxinitrc-wrapper into the chroot...
Installing setres into the chroot...
Installing xinit into the chroot...
Installing xiwi into the chroot...
Installing xbindkeysrc.scm into the chroot...
Installing xiwi.conf into the chroot...
Installing xorg-dummy.conf into the chroot...
Installing xserverrc into the chroot...
Installing xserverrc-xiwi into the chroot...
Installing xserverrc-local.example into the chroot...
Installing startxfce4 into the host...
Installing crouton-noroot into the chroot...
Installing startxfce4 into the chroot...
cp: failed to access '/var/run/crouton/mnt/stateful_partition/crouton/chroots/xenial/usr/local/bin/startxfce4': Permission denied
If known, describe the steps to reproduce the issue:
Reproducible by running sudo startxfce4 from shell. Output received:
chronos@localhost / $ sudo startxfce4
Entering /mnt/stateful_partition/crouton/chroots/xenial...
ln: failed to access '/var/run/crouton/mnt/stateful_partition/crouton/chroots/xenial/etc/resolv.conf': Permission denied
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Issuing ls command in /mnt/stateful_partition/crouton/chroots/xenial/etx shows broken symlinks for the following files/folders:
debian_chroot
localtime
resolv.conf
vtrgb

A fix has been pushed, update your chroot(s).


Config Option to set the DB name

I run enterprise edition, a small feature would be the ability to set the name of the database in config file with a default to 'neo4j'.
Runs fast, awesome tool :)

whoops see it in the code


X-Forwarded-User is not getting forwarded to upstream

Hi,
I don't see X-Forwarded-User and X-Forwarded-Email forwarded to upstream, I have captured the request/response and don't see them anywhere. I have also tried passing "-pass-basic-auth=true -pass-host-header=true" on the cmd line as part of the oauth2_proxy startup but still no luck.
Am i missing something here ?
My stack is ELB -> nginx -> oauth2_proxy - > App server , I am looking for these forwarded headers on App server.

I found the issue, didn't realize that we have set " pass_basic_auth = false" in oauth.cfg and we were trying testing out by passing cmd line param "-pass-basic-auth=true". Looks like the config from .cfg file is taking the precedence.


Optimize CI performance

Currently CI takes a long time
https://github.com/apache/pulsar-client-node/actions/runs/4380814694/jobs/7668271679

This is a common issue that using QEMU to emulate ARM64 instructions on AMD64 machines is slow.
I don't think we have good solutions for it, but waiting for GitHub supports ARM64 runner. My personal project uses buildjet runners with a paid subscription. But for pulsar-client-node managed by the Apache org, you may file a ticket on https://issues.apache.org/jira/projects/INFRA.


Exposing error in form of thiserror instead of anyhow

Is your feature request related to a problem? Please describe.
The error trait is not exposed in the library, which makes it difficult to unwrap and bubble up.
Describe alternatives you've considered
An alternate would be if you could expose a custom RustS3Error trait for error handling

Yeah, this was a bit surprising for me! The anyhow crate explicitly states that it should be used for application code and libraries should use thiserror instead.
Also my upstream library is using thiserror as well, but because of this crate I had to import anyhow as well, just so I could encapsulate it with thiserror.
This looks like a good first contribution, I'll see if I'll have the time to work on a PR.

In the past, I already tried to add this to rust-s3: https://github.com/durch/rust-s3/pull/166. Maybe this could serve as a useful basis if you desire to implement it and want to get it merged (something I failed to do, given that my PR was closed without explanations?).
In any case, I second the fact that this would be a very useful feature!

Sorry, wrong link, the most recent PR on the matter is https://github.com/durch/rust-s3/pull/189.


Unable to use variable in ADO pipeline azureSubscription

Please provide us with the following information:

This issue is a: (mark with an x)

[x ] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
I am unable to use a variable for the azureSubscription parameter in my ADO pipeline.  When used I get the following error.
"The pipeline is not valid. Job Deploy: Step AzureContainerApps input connectedServiceNameARM references service connection $(connectionName) which could not be found. The service connection does not exist, has been disabled or has not been authorized for use. For authorization details, refer to https://aka.ms/yamlauthz."
all of the other parameters using variables are okay
Steps to reproduce
the pipeline code
stages:
    - stage: Deploy
    # dependsOn: Build
    # condition: succeeded()
    jobs:
    - job: Deploy
      strategy:
        matrix:
          chwashPipeline:
            buildinfo: $(chwashPipelineVar)
          # dkingPipeline:
          #   buildinfo: $(dkingPipelineVar)
      pool:
        # vmImage: 'ubuntu-latest'
        name: Default
      steps:
      - task: PowerShell@2
        name: GetUserVariables
        inputs:
          targetType: 'inline'
          script: |
            $json = $env:BUILDINFO
            $obj = ConvertFrom-Json -InputObject $json
            $connectionName = $obj.connectionName
            $containerAppName = $obj.containerAppName
            $resourceGroup = $obj.resourceGroup
            $name = $obj.name

            Write-Host "Connection Name: $connectionName"
            Write-Host "Container App Name: $containerAppName"
            Write-Host "Resource Group: $resourceGroup"
            Write-Host "Name: $name"

            Write-Host "##vso[task.setvariable variable=connectionName]$connectionName"
            Write-Host "##vso[task.setvariable variable=containerAppName]$containerAppName"
            Write-Host "##vso[task.setvariable variable=resourceGroup]$resourceGroup"
            Write-Host "##vso[task.setvariable variable=name]$name"

      - task: AzureContainerApps@1
        displayName: 'ContainerAppDeploy-$(name)'
        inputs:
          appSourcePath: '$(Build.SourcesDirectory)'
          azureSubscription: '$(connectionName)'
          acrName: 'akscbweb'
          containerAppName: '$(containerAppName)'
          resourceGroup: '$(resourceGroup)'

it only works when the azureSubscription value is hard coded.
Expected behavior [What you expected to happen.]
Expected that i would be able use a variable in the azureSubscription paremeter
Actual behavior [What actually happened.]
receive error above
Screenshots

Thanks for reaching out @cwash05. There's been some changes to opt-in vs. opt-out on the connection front. Would you mind checking that the connection your are trying to use is opted into being used by the pipeline please.

@simonjj the service connection is good.  if i replace the variable with the service everything works.  even the other variables.  it just seem for some reason it wont do the interpolation for the  azureSubscription feild

I think this is related to when these variables are resolved, explained here:

It also answers another common issue: why can't I use variables to resolve service connection / environment names? Resources are authorized before a stage can start running, so stage- and job-level variables aren't available. Pipeline-level variables can be used, but only those variables explicitly included in the pipeline. Variable groups are themselves a resource subject to authorization, so their data is likewise not available when checking resource authorization.

Not sure if you can get around this by setting the variable in a prior stage.


Update geku.md

one typo
added correct files for raspberry pi ubuntu

This is the first PR I've ever received on something I wrote myself! Thank you!

This is the first PR I've ever received on something I wrote myself! Thank you!

thank you for your awesome guide! it was very easy to follow
I only had to add a password file in the validator_keys directory, otherwise teku wouldn't run


Adding 3rd part using Option 1

Hello Guys,
Kindly approve the update.
Thanks.

Test of pull request creation, no action needed here.
Thanks.

main code


Github CI: Use v1.7.0 of UCX

Spurious MPI-Linux SMP failure:
make -C startupTest smptest OPTS='-optimize -production  -g -Werror=vla -optimize -production  -g -Werror=vla' TESTOPTS='+setcpuaffinity'
make[4]: Entering directory '/home/travis/build/UIUC-PPL/charm/mpi-linux-x86_64-smp/tests/charm++/startupTest'
../../../bin/testrun  ./startupTest 200 10 0.0001 1  +p2 ++ppn 2 +setcpuaffinity
Running on 1 processors:  ./startupTest 200 10 0.0001 1 +ppn 2 +setcpuaffinity
charmrun>  /usr/bin/setarch x86_64 -R  mpirun -np 1  ./startupTest 200 10 0.0001 1 +ppn 2 +setcpuaffinity
Charm++> Running on MPI version: 3.1
Charm++> level of thread support used: MPI_THREAD_FUNNELED (desired: MPI_THREAD_FUNNELED)
Charm++> Running in SMP mode: 1 processes, 2 worker threads (PEs) + 1 comm threads per process, 2 PEs total
Charm++> The comm. thread both sends and receives messages
Converse/Charm++ Commit ID: b0f4684
CharmLB> Load balancer assumes all CPUs are same.
Charm++> cpu affinity enabled.
Charm++> Running on 1 hosts (1 sockets x 1 cores x 2 PUs = 2-way SMP)
Charm++> cpu topology info is gathered in 0.000 seconds.
Charm++> Warning: Running with more SMP threads (3) than physical cores (2).
         Use +CmiSleepOnIdle (default behavior) or +CmiSpinOnIdle to silence this message.
Nodes 1 Cpus 2 Using 200 10 from 200 10 0.0001 from 0.0001
Setup Complete for arrSize 200 WasteUnits 0.0001
Create Report for 4
Create Report for 5
Create Report for 6
Create Report for 7
Done Report for 4
Done Report for 5
{4} on [0] gets reduction from 5, recv 200
Done Report for 6
{3} on [1] gets reduction from 4, recv 200
{5} on [1] gets reduction from 6, recv 2000
{1} on [1] gets reduction from 2, recv 200
Done Report for 7
{6} on [0] gets reduction from 7, recv 2000
Create Report for 0
Create Report for 1
Done Report for 1
{0} on [0] gets reduction from 1, recv 200
Create Report for 2
Done Report for 2
Create Report for 3
Done Report for 3
{2} on [0] gets reduction from 3, recv 200
Done Report for 0
All Done 8
[Partition 0][Node 0] End of program
real	0m0.483s
user	0m0.745s
sys	0m0.180s
../../../bin/testrun  ./startupTest 200 10 0.0001 1  +p4 ++ppn 2 +setcpuaffinity
Running on 2 processors:  ./startupTest 200 10 0.0001 1 +ppn 2 +setcpuaffinity
charmrun>  /usr/bin/setarch x86_64 -R  mpirun -np 2  ./startupTest 200 10 0.0001 1 +ppn 2 +setcpuaffinity
Charm++> Running on MPI version: 3.1
Charm++> level of thread support used: MPI_THREAD_FUNNELED (desired: MPI_THREAD_FUNNELED)
Charm++> Running in SMP mode: 2 processes, 2 worker threads (PEs) + 1 comm threads per process, 4 PEs total
Charm++> The comm. thread both sends and receives messages
Converse/Charm++ Commit ID: b0f4684
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated


colorbar for heatmap

add colorbar below each heatmap
rasterized image to reduce file size  when output vector graphic files

hi, @simonvh , I've create a PR to add colorbar, if you find colorbar is needed for heatmap. refer to #68

Thanks for this! Looks good to me. However, it would indeed be necessary to make the colorbar optional. Can you make the colorbar code dependent on a flag/variable? Should be an argument of heatmap_plot, but can be set to True as default.

Add --no-colorbar option in the command line to turn off colorbar.

Sorry, this took way too long. Thanks @BioNinja !


Use  the converter from doltgresql to do mysql -> duckdb sql conversion

The Dolt developers have implemented a converter to translate MySQL CREATE TABLE DDLs to Postgresql DDLs. Their use case is to translate the test cases for Dolt to test their new product, Doltgres. We may use this converter as a temporary alternative to #13.

Since SqlGlot works exceptionally well, there is no need to replace it. If any incompatibilities are found, we can contribute to SqlGlot to address them. Let's leverage the power of the open-source community!


[#77] Add SwiftLint job for test workflow

Summary
Add a SwiftLint step into jobs which are including tests.
Miscellaneous
Implements #77.
Removed deprecated SwiftLint rules & added a rule for line length.

I had to merge your master into my own, hence the review reset.

@Ethenyl Thank you again, I requested the review from @curquiza.

@ppamorim you're welcome 😉 I will check the other issues when I have some free time.

Thank you @Ethenyl for your work, and @ppamorim for your review!
bors merge


Adjust sync release commit message bugzilla inclusion

The text about bugzillas needs to be prefixed with dash
so that this info gets into autogenerated changelog when using %autochangelog,
see https://docs.pagure.org/Fedora-Infra.rpmautospec/autochangelog.html#changelog-entries-generated-from-commit-messages
RELEASE NOTES BEGIN
Packit now supports automatic ordering of ☕ after all checks pass.
RELEASE NOTES END

Just add release notes, please..;)


Suggestion - Integrate MobileSAM into the pipeline for lightweight and faster inference

Reference: https://github.com/ChaoningZhang/MobileSAM
Our project performs on par with the original SAM and keeps exactly the same pipeline as the original SAM except for a change on the image encode, therefore, it is easy to Integrate into any project.
MobileSAM is around 60 times smaller and around 50 times faster than original SAM, and it is around 7 times smaller and around 5 times faster than the concurrent FastSAM. The comparison of the whole pipeline is summarzed as follows:

Best Wishes,
Qiao

Awesome! Thanks for sharing very much.
This will be definitely on our to-do list.


Attempt to fix login with GitHub Enterprise

Since the GitHub Enterprise upgrade on 2016-01-20 to Version 2.4.3, several
users (possibly all users) are seeing an error at login time that seems to be
related to the GitHub OAuth interaction. The deploy Jenkins do not have the
same problem and are using this earlier version of the github-oauth plugin.
Here are two snippets from the stack trace that Jenkins shows to users:
java.io.IOException: {"message":"You need at least read:org scope or user scope to list your organizations.","documentation_url":"https://developer.github.com/enterprise/2.4/v3/orgs/#list-your-organizations"}
	at org.kohsuke.github.Requester.handleApiError(Requester.java:342)

and:
Caused by: java.io.IOException: Server returned HTTP response code: 403 for URL: https://github.gds/api/v3/user/orgs
	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1628)

Here is the changelog for the plugin: https://wiki.jenkins-ci.org/display/JENKINS/Github+OAuth+Plugin#GithubOAuthPlugin-Releasenotes
Here are the release notes for GitHub Enterprise: https://enterprise.github.com/releases

Seems to be https://github.com/jenkinsci/github-oauth-plugin/pull/39

Actually https://github.com/jenkinsci/github-oauth-plugin/pull/35 added scopes at all.


Camp Harmony: ability to change lodging assignment

Hi WIll,
It's going really well!  I don't seem to have the ability to change someone's housing request, for example from semi-private to economy. Can you add this to the back end?

I think this is fixed.


Updating cntk to use maven repo

Updating cntk to use maven repo

@imatiach-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Move content from feature preview document to user guide

To make it sustainable and easy to discover for users. The content
is reworded to fit into the existing documentation and with the new
terminology definitions. Some explanations are moved to the release
notes, which previously linked to the preview document, to better
illustrate the features there.
Features documented:

optional dependencies
BOM support
compile/runtime scope separation
metadata sources (was already added to documentation earlier,
therefore it can also be removed from the preview doc)

@ljacomet I assigned you as additional reviewer. If you have some time, maybe you can have a look at the documentation sections that are touched in this PR and give feedback from a "user perspective" if you have any.

Thanks for your reviews @ljacomet and @bmuschko. I addressed your comments. Let me know if there is anything else.

👍 LGTM

All in all, I think whole section on Java Libraries should be checked and improved. But I won't do that as part of this PR. (I think it's on you are planing to work on that Ben, right?)

It needs to be worked on but we didn't exactly schedule it yet.


Global.json schema requires property projects

having this simple globa.json
{
	"sources": [ "src", "tests"],
	"sdk": {
		"version": "1.0.0-beta3"
	}
}

visual studio says : Missing required property projects.
schema is : http://json.schemastore.org/global
which clearly says that projects is required. I don't know if this is good or not, I can't get to find a documentation about what projects is, but if I provided as empty or with some globing combinations, projects are not discovered at referencing so I guess something is going on, probably property shouldn't be required on the schema.

Oh, looks like sources is now deprecated and was renamed to projects ? then that will make sense.


Create req/resp classes only generated API methods

Hi.
Thank you for update of my issues "Generate not all API methods". It's work good.
But classes are create for all method, not just generated ones.
Please update to generated only needed req/resp classes.
Thank you very much for your job.

Hi,
I will look into it.

Hey.
I would like to suggest asking question ONLY via Github.
We are not going to add such feature, what I can recommend is removing unneeded swagger parts via script and then generate it.
If you want to introduce custom behavior feel free to fork project.

Okay.


Add configuration option for removing mandatory json {success: true} from traditional endpoint upload

Based on previous (conversation)[https://github.com/FineUploader/fine-uploader/issues/1025#issuecomment-126341926), add configuration option to remove mandatory json response with {success: true} for traditional endpoint response, so we can go just with HTTP 201 response like success.

Note that we will still need to require IE9 & IE8 to return a 200 response with {"success":true} due to issues with handling of non-200 responses in those browsers when an iframe is involved.
I don't want to make this a configuration option. Instead, 6.0 would mandate that endpoints declare the status of the upload request via the response status code, unless the request comes from IE9/IE8. All of this becomes a bit more complicated in a cross-origin environment.
Honestly, I don't expect to to implement this given the scope of the change and the potential edge cases that may suffer.

Ok, I understand this. In our specific case IE9&IE8 are not important, so we have much simpler case. Thanks for accept this.

Duplicate of #1325.


Typo on PANOS Document Page

There is a Type on the page: https://splunk.github.io/splunk-connect-for-syslog/main/sources/vendor/PaloaltoNetworks/panos/
Attaching Screen shot to show location.  I know this is small, but thought it might help other people out

.

Hi @kdiasSPL SC4S is a community project. Please fee free to directly create a PR for fixes like this. Thank you :)

Closed by #1770


Hide "Schedule action for VM" on VM deployment for "cloudadmin" view

Description
It is necessary to add new option to hide "Schedule action for VM" on VM deployment for "cloudadmin" view.
Use case
Cloud administrators who want to avoid configuring scheduled actions for virtual machines by cloud users.
Interface Changes
Sunstone.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Please consider all sunstone views

@tinova PRs to be approved:

Code
Docs


docs: Small typo in factories.rst

Description
Small typo in factories.rst. Change all "full qualified classname" to "fully qualified classname".
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

@kenjis on security.rst page A6 Sensitive Data Exposure and A7 Missing Function Level Access Control sections have the same description. I would like to change it but I cannot find what it was before. In fact this page needs a complete update. When you follow the OWASP Top Ten Cheat Sheet link on top of the page you can see that this sheet is changed. Do you have any idea or suggestion on how to update the CodeIgniter Security Guidelines page? (I had to write this comment here, I don't know how to comment on security.rst page without creating a pull request for it.)

@obozdag See #7390


Did some styling on Wildlife Conservation forum now it looks perfect #1556

styled the wildlife conservation forum, Changed the colour of the submit button which now looks compatible with the theme used in the website.

@vishwas343 Great contribution to Hacktoberfest 2024! 😊 Please include before and after images or video of the changes from next time to improve clarity. Thanks for your effort! 🙌


enable beter confusion matrix display

as requested I also PR here to add new dynamic calculation of the first coloumn width and other columns width base on content, enabling better display when there are several classes and correct the relevant test.

Codecov Report

Merging #9 (fb7c95a) into dev (1afefc4) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##              dev       #9      +/-   ##
==========================================
- Coverage   91.94%   91.83%   -0.11%
==========================================
  Files          14       14
  Lines         695      698       +3
==========================================
+ Hits          639      641       +2
- Misses         56       57       +1

Files Changed
Coverage Δ

src/confusion_matrices.jl
96.55% <100.00%> (+0.06%)
:arrow_up:

... and 1 file with indirect coverage changes


Support for only using nuget v3 api

Can be used by setting source https://api.nuget.org/v3/index.json in paket.dependencies
Adds a reference to NuGet.Versioning so that we can reuse there parsing logic for the new version range syntax.
I wrote this so that paket could be used against nuget servers that do not support the v2 api. The v3 api, while still crazy in places, is much easier to implement.

I removed the dependency on NuGet.Versioning, it looked like the dependency range syntax had changed radically in v3, but it hasn't changed at all. The v3 api is just handing out valid, but odd, version ranges.

One thing I didn't do, which could be a target for a future pr, is call the v3 apis whenever a v2 api is handed in. That might speed up the typical dependency resolution.

Wooooo!


[SP] avoid uploading cached shared library

Description
Currently, we will download from the S3 and use that to upload. It is not performing correctly and just make the shared library not updating.

Codecov Report

Merging #916 (be1f426) into master (a5b4b30) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #916      +/-   ##
============================================
- Coverage     70.31%   70.29%   -0.03%
  Complexity     5085     5085
============================================
  Files           501      501
  Lines         22431    22431
  Branches       2334     2334
============================================
- Hits          15773    15768       -5
- Misses         5417     5428      +11
+ Partials       1241     1235       -6

Impacted Files
Coverage Δ
Complexity Δ

...gine/src/main/java/ai/djl/dlr/engine/DlrModel.java
53.12% <0.00%> (-3.13%)
4.00% <0.00%> (ø%)

api/src/main/java/ai/djl/util/Platform.java
43.47% <0.00%> (-2.18%)
8.00% <0.00%> (ø%)

...g/src/main/java/ai/djl/serving/util/Connector.java
55.40% <0.00%> (-1.36%)
21.00% <0.00%> (ø%)

...ngine/src/main/java/ai/djl/mxnet/jna/LibUtils.java
42.23% <0.00%> (-1.25%)
15.00% <0.00%> (ø%)

...rc/main/java/ai/djl/paddlepaddle/jni/LibUtils.java
51.14% <0.00%> (ø)
14.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


fix(dropdown): remove hover state background color change #34

Alaska Airlines Pull Request
Removed hover state background color change.
Fixes: #34  (issue, if applicable)
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


App.synthesize methods are marked as deprecated with no replacement

In latest version (0.25.3) the App.synthesize methods are marked as deprecated but don't offer a replacement or pointer as to why. Please provide guidance for the "new" sythesize.

From your question I can tell you are running/synthesizing CDK apps not in the way we intended them to.
The intended way is to let the cdk toolkit drive your application by launching it as a subprocess, not to run CDK apps in-memory. In other words, you are using an unsupported part of the API, and those synthesizeStack methods were never supposed to have been there or public in the first place (they were there for the benefit of our unit tests). Please don't do that, and if you do, be aware that we can break you at any time.
Now, the slightly better news is that you can probably keep on doing what you are doing today by calling app.run() and inspecting the object that is returned to you from it. It will be an InMemoryStore and contain the templates of the stacks that got synthesized. But again, these APIs are not intended to be called by users directly.

A bit tangential, but I have a need for adding business-specific validation to Stacks.  A contrived example would be something like "in dev always use on-demand billing for dynamodb".  Right now (0.24.1) I've extended Stack and overridden toCloudFormation to validate against the synthesized stack object.  I worry that this falls into the same category of "not intended to be called by users directly" and will break in a future upgrade.
@digitalsanctum what is your use case for relying on synthesize directly?
@rix0rrr depending on his answer, what's your guidance on the best place to hook in?  You mention app.run but then say don't do that ;)  Perhaps this is a feature request?

I would consider it a feature request at the moment.
As a practical matter, you should be able to achieve the same effect by having a class implement IAspect, like this:
class DynamoValidator implements IAspect {
  public visit(construct: IConstruct) {
    if (Resource.isResource(construct) && construct.resourceType == 'AWS::DynamoDB::Table') {
      if (!whatIWant(construct)) {
        throw new Error('oops');
     }
    }
  }
}

app.node.apply(new DynamoValidator());

There might be some details wrong with the code above (around type checking), but the pattern should hold.

The short version is I'd like to use CDK (without the CLI bits) in concert with the SDK to easily do things like integration tests. Yes, I could just use the SDK but I think there's a huge opportunity to be able to just use the CDK and the benefits that come with it like automatic IAM role creation, reusable constructs, etc. Being able to use the CDK synth and deploy without worrying about installing the CLI would be ideal.
For the purposes of this issue, it would be nice to have your explanation noted in the code comment along with the deprecation for other folks that are using the public synth methods.

It seems the feature I describe above has already been proposed and voted for by several people here: https://github.com/awslabs/aws-cdk/issues/601


Not able to "rails g spree_recently_viewed:install"

I am using spree version 4.1
When I try to rails g spree_recently_viewed:install it gives following errors. Can someone help me please?
/Users/tashi/.rvm/gems/ruby-2.6.5/gems/spree_core-4.1.5/app/models/spree/preferences/store.rb:9: warning: already initialized constant DB_EXCEPTIONS
/Users/tashi/.rvm/gems/ruby-2.6.5/gems/spree_core-4.1.5/app/models/spree/preferences/store.rb:9: warning: previous definition of DB_EXCEPTIONS was here
/Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:710:in block (2 levels) in skip_callback': Before process_action callback :set_current_order has not been defined (ArgumentError)         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:706:in each'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:706:in block in skip_callback'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:628:in block in __update_callbacks'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:626:in reverse_each'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:626:in __update_callbacks'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:705:in skip_callback'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/actionpack-<IP_ADDRESS>/lib/abstract_controller/callbacks.rb:215:in block (3 levels) in module:ClassMethods'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/actionpack-<IP_ADDRESS>/lib/abstract_controller/callbacks.rb:96:in block in _insert_callbacks'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/actionpack-<IP_ADDRESS>/lib/abstract_controller/callbacks.rb:95:in each'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/actionpack-<IP_ADDRESS>/lib/abstract_controller/callbacks.rb:95:in _insert_callbacks'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/actionpack-<IP_ADDRESS>/lib/abstract_controller/callbacks.rb:214:in block (2 levels) in module:ClassMethods'
from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/app/controllers/spree/products_controller_decorator.rb:5:in prepended'         from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/app/controllers/spree/products_controller_decorator.rb:28:in prepend'
from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/app/controllers/spree/products_controller_decorator.rb:28:in <main>'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in block in require_with_bootsnap_lfi'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in register'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in require_with_bootsnap_lfi'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:31:in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/zeitwerk-2.3.0/lib/zeitwerk/kernel.rb:16:in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/app/controllers/spree/products_controller_decorator.rb:1:in <main>'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:55:in load'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:55:in load'         from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/lib/spree_recently_viewed/engine.rb:16:in block in activate'
from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/lib/spree_recently_viewed/engine.rb:15:in glob'         from /Users/tashi/.rvm/gems/ruby-2.6.5/bundler/gems/spree_recently_viewed-85f2e6870a7e/lib/spree_recently_viewed/engine.rb:15:in activate'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:429:in instance_exec'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:429:in block in make_lambda'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:201:in block (2 levels) in halting'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:607:in block (2 levels) in default_terminator'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:606:in catch'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:606:in block in default_terminator'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:202:in block in halting'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:514:in block in invoke_before'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:514:in each'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:514:in invoke_before'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:134:in run_callbacks'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/reloader.rb:88:in prepare!'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/application/finisher.rb:113:in block in <module:Finisher>'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in instance_exec'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in run'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:61:in block in run_initializers'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:228:in block in tsort_each'         from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:350:in block (2 levels) in each_strongly_connected_component'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:431:in each_strongly_connected_component_from'         from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:349:in block in each_strongly_connected_component'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:347:in each'         from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:347:in call'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:347:in each_strongly_connected_component'         from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:226:in tsort_each'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/2.6.0/tsort.rb:205:in tsort_each'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:60:in run_initializers'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:363:in initialize!'         from /Users/tashi/DragonCoder/btmade-app/config/environment.rb:7:in '
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in require'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in block in require_with_bootsnap_lfi'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in register'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in require_with_bootsnap_lfi'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:31:in require'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/zeitwerk-2.3.0/lib/zeitwerk/kernel.rb:23:in require'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:325:in block in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:291:in load_dependency'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:325:in require'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application.rb:106:in preload'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application.rb:157:in serve'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application.rb:145:in block in run'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application.rb:139:in loop'
from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application.rb:139:in run'         from /Users/tashi/.rvm/gems/ruby-2.6.5/gems/spring-2.1.0/lib/spring/application/boot.rb:19:in <top (required)>'
from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/site_ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in require'         from /Users/tashi/.rvm/rubies/ruby-2.6.5/lib/ruby/site_ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in require'
from -e:1:in `'

Facing the same issue. Following along.


Do not send too many reports in one API request.

Currently all check monitoring reports are sent to Mackerel in one API request, but it may contain too many reports in one API request, if the host so many check monitoring.

Thanks!