Address DM21 mismatch: accept 1 minute difference between queries

Address DM21 mismatch: accept 1 minute (1 bit state) difference between queries
14:07:00.8905 Destination Specific DM21 Request to Engine #1 (0)
14:07:00.8951 18EA00F9 [3] 00 C1 00 (TX)
14:07:00.8981 18C1F900 [8] 00 00 00 00 00 00 02 00
DM21 from Engine #1 (0): [
 Distance Traveled While MIL is Activated:     0 km (0 mi)
 Time Run by Engine While MIL is Activated:    0 minutes
 Distance Since DTCs Cleared:                  0 km (0 mi)
 Time Since DTCs Cleared:                      2 minutes
]
FAIL: <IP_ADDRESS>.a - Difference compared to data received during global request from Engine #1 (0)

Need text proposal

Work with #1257

<IP_ADDRESS>	Actions2.
a.	DS DM21 to each OBD ECU.
<IP_ADDRESS>	Fail Criteria2
a.	Fail if any difference compared to data received from global request.
i.	Allow 1 minute difference between responses for SPN 3296
same as #1257

PgnDefinition already has a isVariableBroadcast flag.  Development, verify that this flag is correctly intergpretted for the SPNs in question.  No DA parsing changes should be needed.

The interpretation (implementation) will change as timing is to be ignored.
Best Wishes,
Eric Swenson
Sent from my iPad

On Oct 20, 2023, at 1:42 PM, Joe Batt @.***> wrote:

PgnDefinition already has a isVariableBroadcast flag. Development, verify that this flag is correctly intergpretted for the SPNs in question. No DA parsing changes should be needed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.


Update README

Issue Description
🔑
The README.md file does not have listings for the scan and supply tools. A bit of sorting/organization might be nice as well for iOS/Android pairings. The issue(s) call to action isn't linked.
Environment
fastlane 1.83.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)? no
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)? no

Working on this currently in a fork for a PR. Will submit once I've completed a thorough pass.

Nice catch, thanks for submitting a PR 👍


[Question] Change string color in MS Teams?

Is there a way to change the color of a string that's shown in ms teams? How about the text inside a stringbuilder?

@meganlcooper Hi Megan,
What type of object are you hoping to colorize? You can set the optional TextFormat property to control how your message's text content will be rendered. Here's some more info about activities, cards and actions. Does that answer your question? Please post again if not!

Going to close this issue, feel free to reopen to continue the discussion.


Unable to connect to keosd, if keosd is running please kill the process and try again.

Hey guys so when i try to create a wallet from cleos it always says
 "/usr/local/bin/keosd" launched Unable to connect to keosd, if keosd is running please kill the process and try again."
I tried to kill keosd by using pkill keosd but its not working, i even tried to re-download the code and re-installed it, the problem still persists. Appreciate it if anyone could help me

same:
Unable to connect to keosd, if keosd is running please kill the process and try again.

https://www.bcskill.com/index.php/archives/105.html

thank you very much, the code was very useful, problem solved


[Docs] Documentation on how to port existing apps from create-react-app

Hi, I'm one of the many developers who have an existing project created with create-react-app and would like to port over and adopt react-static. However, short of great documentation that I was able to find on how to do this, I'm at a lost on how to proceed.
I understand that it is fairly easy to start a new project from scratch with the react-static CLI. However, this is not an option for existing apps, and the current documentation do not paint a whole picture on how to port existing projects to adopt the features of react-static.
Perhaps I'm just not seeing it, but after reading through all the current documentations + Googling for a while, I see many face the same conundrum.
Thanks!

@tetrachrome Is there a reason you closed this?

Why was this cloed? This is very important to be in the documentation.


[RFC]

What's the problem? 🤔
Check if PR has label / test-label-applied (pull_request), how to solve this？
What are the requirements? ❓
No response
What are our options? 💡
No response
Proposed solution 🟢
No response
Resources and benchmarks 🔗
No response

@cccEric only maintainers can add labels to PR's, you need to worry about it.


Update documentation for pnpm to npm

Description
I have update README.md and CONTRIBUTIONG.md file and make some work easy for developers to add their profile
Changes Proposed
If you are using react then use Vite. It is more faster than create-react-app. If you are thinking to migrate to to NextJS then don't need to do this.
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[ ] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

If you have any work for me. please let me know

@sahilatahar Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


is pydantic dependency needed?

pydantic is installed along django-generate-series, but it seems is not used in the code.

Thanks for the report. Fixed in 4b4b041de58b7141ffbb6cc9456079492ec053dc


Switch to mv3 - part 2

This omnibus issue is a continuation of #175 as the switch to mv3 is not finished yet. This issue organizes our approach and the open issues to resolve.
Going forward, @OliverWang13 will be our product manager keeping us organized and on track timewise with assistance by @kalicki1.
0. Current Status
The #175 mv3 branch is merged into main. But there are at least three open questions:

When can we close out the various open mv3 issues?
How do we address the switch to mv3 for Firefox?
How can we start the usability study while we are in the process of switching to mv3?

In detail:
1. Various Open mv3 Issues
Here are the various mv3 issues and the order in which they need to be addressed:

[ ] #259 is the main problem; we have a timing problem for attaching the GPC signal via the DOM
[ ] #262 covers various clean-up work for the domain list that does not seem too problematic
[ ] #283 once all functional mv3 changes have been made
[ ] #213 pretty quick update of documentation
[ ] #273 release of the final mv3 product

2. Switch to mv3 for Firefox
It seems to me we have largely worked on switching to mv3 from the Chrome perspective. It seems to me that the various open issues cover Chrome. It is far less clear what we do about Firefox. In fact, it seems that Firefox's own timeline for switching to mv3 is not clear at this point. Here is their most recent blog post on the topic. Two highlights:

Mozilla will maintain support for blocking WebRequest in MV3. To maximize compatibility with other browsers, we will also ship support for declarativeNetRequest.

So, we can leave our Firefox version as is?

We’ve found Service Workers can’t fully support various use cases we consider important, especially around DOM-related features and APIs. ... In Firefox, we have decided to support Event Pages in MV3, and our developer preview will not include Service Workers (we’re continuing to work on supporting these for a future release).

On the long run, our Chrome and Firefox releases should be as similar as possible to keep the workload as small as possible. It  seems difficult to tell how our Firefox release will look like in the end given the current uncertainty of Firefox's concrete mv3 switch. Or can we? On the short run, do we keep the Firefox release essentially as is and switch to mv3 for Chrome only? Do we role back our mv3 release of Chrome? (see Usability study question below) Something else ...?
So what do we do?
3. How to Usability Study in the Midst of mv3 Switch?
The problem is that we are halfway in the switch (see #175) but also fairly far off from finalizing it. This is partly due to hitting difficult problems, especially, #259, partly because the final situation for Firefox is not known, and partly because our pace is too leisurely.
The big question is, can we push this through fairly quickly to move on to our research questions or do we need to roll back to mv2 and tackle mv3 again later? Or something in between, e.g., mv3 for Chrome and leave Firefox as is.
@OliverWang13 will take it from here and with @kalicki1's help organize and implement the switch to mv3.

Excellent!

This is not something I am extremely familiar with yet so I do not know how long this would take.

I'd say, take two days, @OliverWang13, and try to gauge then how long this would take. Then, we can revisit.

This has been completed.


how to disable console window when game runs?

Non-console Windows applications use the WinMain() entry point.
So this:
int main(int argc, char* argv[])
should be replaced with:
int CALLBACK WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
Don't forget to include <windows.h>
You may also want to add the following line to CMakeLists.txt:
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")

Non-console Windows applications use the WinMain() entry point.
So this:
int main(int argc, char* argv[])
should be replaced with:
int CALLBACK WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
Don't forget to include <windows.h>
You may also want to add the following line to CMakeLists.txt:
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")

awesome reply! thanks heaps.


manifest yml fixes

logging_level addition to manifest yml
disable cf incubation routing_api for diego
uaa authority & permissions fixes

@tchughesiv Thanks a lot!
This is just a tip: you can use 'force -f' when you want to update your PR, without closing an existing one.

Thanks for the tip, I'm still a rookie.

On Sep 14, 2015, at 1:20 PM, Iwasaki Yudai<EMAIL_ADDRESS>wrote:
@tchughesiv Thanks a lot!
This is just a tip: you can use 'force -f' when you want to update your PR, without closing an existing one.
—
Reply to this email directly or view it on GitHub.


Update the write-up for Fall, 2020

I've done a full editing pass of the README, and I need to enter those changes. Some of these are just improvements and fixes, but other are driven by the change to bats-core.

I've made processed everything through "Saying what language you're using", which is essentially the first 8 "pages" of the write-up. I'll need to pick up at "Making the test fail (and error status codes)".


chore: Remove BOM for YAML frontmatter

@nschonni : Thanks for your contribution! The author, @tamram, has been notified to review your proposed change.

@nschonni - Thanks for your contribution!
#sign-off


Improve inference graph example

"Fixes #issue-number" or "Add description of the problem this PR solves"
Proposed Changes

Made some improvements to inferencegraph example

/lgtm
/approve


How can the gulped data be used for Action Anticipation

I ran 'snakemake -p gulp_all', and got the gulp directories. I am wondering how do I set the anticipation time (Ta) = 1 second?
For one segment: ('start_frame', 262), ('stop_frame', 370),
And then the next segment: ('start_frame', 418), ('stop_frame', 569)
In such a case how do we set Ta=1 second, for the anticipation time? Since for many segments, the difference in time between 2 segments is more than 1 second?

Thanks for your question. This repo is primarily designed to help you with the action-recognition challenge. For questions related to the action anticipation or object detection challenges, please email the dataset's general info<EMAIL_ADDRESS>This will allow you to direct the question to the relevant researcher.


Remove debug = true for release builds

It looks like this was added when when doing some profiling. Not including debug info makes the binary size a lot smaller for Linux users

Thanks for the PR!
Yep definitely forgot to take this out after debugging.


Closing app

How can I make scrolling down a ListView type not close the application?

solved https://github.com/flutter/flutter/issues/77883#issuecomment-1996986904

@mrleesin How did you solve the issue? I

For anyone having similar issue. This sample html shows how to fix the problem. Ref -> https://github.com/flutter/flutter/issues/77883#issuecomment-1996986904
<!DOCTYPE html>
<html>
<head>
  <style>
    body {
      height: 10000vh !important;
      position: static !important;
      overscroll-behavior: none;
    }

    flutter-view {
      position: fixed !important;
      max-width: 100vw !important;
      max-height: 100vh !important;
    }
  </style>
  <!--
    If you are serving your web app in a path other than the root, change the
    href value below to reflect the base path you are serving from.

    The path provided below has to start and end with a slash "/" in order for
    it to work correctly.

    For more details:
    * https://developer.mozilla.org/en-US/docs/Web/HTML/Element/base

    This is a placeholder for base href that will be replaced by the value of
    the `--base-href` argument provided to `flutter build`.
  -->
  <base href="$FLUTTER_BASE_HREF">

  <meta charset="UTF-8">
  <meta content="IE=Edge" http-equiv="X-UA-Compatible">
  <meta name="RaqamliDokon" content="Telegramda Online Raqamli Do'kon">

  <!-- iOS meta tags & icons -->
  <meta name="apple-mobile-web-app-capable" content="yes">
  <meta name="apple-mobile-web-app-status-bar-style" content="black">
  <meta name="apple-mobile-web-app-title" content="RaqamliDokon">
  <link rel="apple-touch-icon" href="icons/Icon-192.png">

  <!-- Favicon -->
  <link rel="icon" type="image/png" href="favicon.png"/>

  <title>RaqamliDokon</title>
  <link rel="manifest" href="manifest.json">
</head>
<body>
<script src="flutter_bootstrap.js" async></script>
<script src="https://telegram.org/js/telegram-web-app.js" defer=""></script>
<script src="https://unpkg.com/pica/dist/pica.min.js" ></script>
<script>window.scroll(0, window.document.body.scrollHeight);</script>
</body>
</html>


Facing multiple issues.

I am getting an error when luis entity is mapped to a dynamic field with terms in a form. I have created a repo and would try my best to explain it here.
Repo: https://github.com/ankitbko/Bot-Issue355
Issue 1
Run the bot and send text "Where is London". A Null Reference exception is thrown at Step.cs Match method at line  if (stepState.State == FieldStepStates.SentPrompt) because stepState is null.
This case works fine if you would comment Line 32 and 33(below) of CityLuisDialog.cs and send the same input.
if (result.TryFindEntity("builtin.places.absolute_location", out loc))
                entities.Add(new EntityRecommendation(type: nameof(CityForm.Location)) { Entity = loc.Entity })

Issue 2
In CityLuisDialog.cs L83 field.AddTerms(airport, Language.GenerateTerms(Language.CamelCase(airport), 3)); change 3 to 5.
Send text "Where is London". The following exception is thrown at Recognize.cs AddExpression method at _expressions.Add(new ValueAndExpression(value, new Regex(expr, RegexOptions.IgnoreCase), maxWords));
"exceptionMessage": "parsing \"(\\b(?:(?:(\\s+meet\\s+\\s+&\\s+\\s+greet\\s+)\\s+\\s+liverpool)|(?:greets?\\s+)\\s+\\s+liverpools?)|(?:meets?\\s+\\s+&\\s+\\s+greets?)|(?:liverpools?)|(?:greets?)|(?:meets?))\\b)|(qqqq)\" - Too many )'s.",
"exceptionType": "System.ArgumentException",

On further digging, it seems Issue 2 is caused only when "(Meet & Greet) Liverpool" is present in the string. It seems parenthesis is the culprit.
Issue 1 only happens if the bot is unable to map the LUIS entity to just one enum option. It should, as next step, prompt user to clarify the ambiguity from the subset of matches as it happens with non-LUIS flow.

Handling disambiguation from LUIS entities was something I left as a TODO when we first released.  I had started down the path of implementing it, but it was complicated so I left it.  I'm going to take a look at again this weekend and try to get it done.  I'll look into the other issue as well.  Thanks for the nice repros!

@chrimc62 Thanks for the quick response. I didn't knew it was not implemented. I thought since the LUIS entity is just another way to input to the form, it would converge and follow the same path as if the user was entering the input. This is a very required feature, if possible please implement this.
Unrelated to this, what happens if you return false from field validation delegate?
Also is it possible to display an enum field in form as buttons with images? I see there is a FormButton class, but I didn't get how do I use it.

The meet&greet issue is by design.  The terms are regular expressions so you need to make sure you escape the ()'s if you don't want them to be regular expressions.  If you don't then with the n-grams you can end up with invalid regular expressions.  Your strings also have double spaces which meant the matching was not working right and I've fixed that.
The issues around initial validation have to do with the way the internal state machine works--for example navigation doesn't really work if you are doing validation of initial inputs.  Agree the feature should be there and I'm working on it.

If you return false from the field validation, the value is not set in the form. @msft-shahins can comment on the button/image issue.  I don't think there is an easy way to do it in the current SDK, but it might be possible through manipulating the buttons after they come out of FormFlow.

@chrimc62 Thanks for quick response. I intended to ask what happens if we return false from SetDefine not validation, since now we return ValidationResult from validation delegate. My bad. I believe it would not define the field, so how does the field behave in this situation and is there any case when we would want to return false?

If you return false while in a form, the step is considered complete and will be skipped.  When processing the LUIS entities the return value is currently ignored, but I will probably be changing that when I take care of your issue.

I have a fix for this and will check-in tomorrow and it should be in our next release.

This is just great. Thanks for quick resolution of the issue. Looking forward to it in action.


does not respect right padding of bootstrap column

I saw this issue mentioned in a discussion about another issue but could not tell if one was specifically opened up for this.  If I give my grid a class of col-sm-12 the left is aligned correctly but the right side extends past the page.  I have tried doing a window resize and that will make it a little better but it still ignores the actual bootstrap column styling.

As you can see the grid hits the right border of the container and there should be a 15px margin.

I'm wondering about whether we (someone) should create a bootstrap feature, which adds styles and formatting for bootstrap, and perhaps uses the bootstrap editors for the edit feature as well.  It seems like a lot of people would use the grid with bootstrap, so if it doesn't play nice we should deal with it.


add uuid() function

Which issue does this PR close?

Closes #4045
Rationale for this change
add uuid() function

What changes are included in this PR?

Are there any user-facing changes?

Thanks @Jimexist I still recommend a sql based test, but I think it is not strictly necessary

thanks for the comment @alamb - test added


Overriding of Sub Chart Name in the _helpers.tpl in the Parent Chart

Hi,
Is it possible to override .Chart.Name object of the Subchart from the _helpers.tpl in the parent chart. Basically, we have Chart A which has 2 dependencies Chart B and  Chart C. Each of the chart has its own template functions in their own _helpers.tpl. These template functions uses .Chart.Name which is used to update the common Labels. According to the Helm Best Practices, when the chart is deployed using umbrella chart,  the dependency chart should have the name of umbrella chart. We are unable to override the .Chart.Name or any other Chart object in the parent chart (chart A) as they are only pointing to the Sub Chart's Chart.yaml information.
{{- define "commonLabelsWithoutChartVersion" -}}
{{- if .Values.common_labels }}
app.kubernetes.io/name: {{ .Chart.Name }}                       //When overrided from Chart A (Parent Chart), it gives the Chart name of subchart (B)
app.kubernetes.io/instance: {{ .Release.Name }}
app.kubernetes.io/component: "Example"
app.kubernetes.io/part-of: {{ .Values.partOf }}
app.kubernetes.io/managed-by: "helm"
{{- end }}
{{- end }}
So, Is it possible to override the .Chart.Name or any other .Chart built in object from the parent _helpers.tpl ?

No, it is not possible to “rewrite” built-in variables or values passed into the template engine. They act similar to constants.

Closing this issue as answered. Please feel free to comment here and let us know if you have further questions. Thanks!

For more info see https://pkg.go.dev/text/template


Background is not rendered correctly

On Linux, the (transparent) background for the lowest screen is not black.

I've fixed this issue.


TOC can be sorted alphabeticaly, to behave like an index

Hi,
first of all, thank you for your work, it helped me a lot!
I am suggesting this patch, because of a specific use case I encountered:
I needed to create a index of the document, where items are sorted alphabeticaly, followed by their corresponding page (for this I use flying-saucer and @page css instructions, but it's unrelated to this patch).
As far as I understood, the behavior I want is very similar to the TOC extension.
So, instead of duplicating the code into a new extension, I started hacking a bit the TOC code.
I added a new option, to allow sorting headers by their titles.
Please let me know if I this approach is incorrect, and what I should be changed in this patch to make it mergeable.
Thank you again

@rems, I will take a look at it but as far as I am concerned if the option is useful and does not break existing functionality I see no reason why it would require a whole new extension.
Not seeking pity, but I just upgraded my iMac to Sierra and haven't resolved "all the improvements" yet to have my development environment fully functional. I expect to be done with most of it today and take a look at your changes. If they are close I will probably merge them and make adjustments for the next release. I was hip deep into cleaning up code when something possessed me to upgrade the OS so will have to get back in the zone once the system is back to fully functional.

@rems, I took a look at the code and it is fine except for heading sorting will not be what you would expect if HTML rendering is used for headings. I would make a change to always use pure text for sorting the headings irrespective of the rendering setting. That way the final result will always make intuitive sense.
If you have some test cases just post them here. I will be merging the code today and add the tests after the merge.

@rems, thank you for the code changes and a great idea. It spawned others which I think add to the TOC extension.
The upcoming release will have your changes with some mods. I changed the code to sort on heading text without the inline decorations, added symmetrical options that were begging to be added since sorting made them obvious alternatives. Your desired option in the new code would be: increasing, sorted in increasing order either in markdown source in the [TOC increasing] or by setting TocExtension.LIST_TYPE key in builder options to TocOptions.ListType.SORTED.

Add: added Toc and SimToc options:

hierarchy: as before hierarchical list of headings
flat: flat list of headings
reversed: flat reversed list of headings
increasing: flat, alphabetically increasing by heading text
decreasing: flat, alphabetically decreasing by heading text

Extension changes:

new key TocExtension.LIST_TYPE takes a TocOptions.ListType enum for list generation
type.

TocOptions now supports Immutable interface allowing to get a mutable copy whose
fields can be modified directly with toMutable(), and back to TocOptions with
toImmutable().

SimTocOptions have been removed. TocOptions holds all the options for both the
regular TOC and simulated TOC.

@rems, repo updated 0.9.0. Maven release, might take a few hours. Last release 0.8.0 never made it to central.

Thank you a lot, 0.9.0 works perfectly.
This new TOC extension looks great, and merging SimTOC into TOC definitively makes sense.

@rems, you are welcome and thank you for the new feature.


/end command should end to the winning team by default

If you do /end b , end will win, if you do /end r, red will win, but if you do /end, it should end towards the team that was winning, instead of ending it as a tie

I don't think /end should end the match with the winner. It should have a new flag, something like /end -w

https://github.com/twizmwazin/CardinalPGM/pull/463#issuecomment-85784155

in tm.oc.tc if you /end it ends towards the winning team


Switching to mySQL storage

Hello, I did a bunch of stuff in h2 storage before realizing I needed to use MySQL storage.  I switched over to that and now when I try to join it says "Permissions data could not be loaded"
Is there a way to transfer the data over or do I need to restart?

https://github.com/lucko/LuckPerms/wiki/Switching-storage-types :)

God, I don't know how I missed that, thanks.

So I did that but when I import the file, it get's a 'failure' on everything:
Relevant part of config: https://i.gyazo.com/416dd4750e68da238e7ec15b46fb20af.png (Password is there, just whited out)
Section of console after doing import: https://i.gyazo.com/6e8c375aa553bc951b18881f8ba7fa97.png

LP cannot connect to your database.

Hm, I don't really know much about MySQL honestly, do I need to create a database for it?

Yes

Ok I spent some time learning to use MySQL and creating that database did indeed fix it, thanks SO much for the help, I really appreciate it!

I do have one more question though...
Is it possible to make it so the rank names in /who show up with caps?
IE:  when you do /who it says
duke:  Name
instead of
Duke:  Name

https://github.com/lucko/LuckPerms/wiki/Default-Groups#keep-using-the-default-group-but-just-modify-the-display-name

Thanks!
I am having an issue though since changing that, when I have people in the default group (Not 'default' but the one it makes them a member of, 'commoner') it will not let me 'lp user name promote track' says
It says they aren't in any groups in the track despite being a member of commoner, the first member of the track.  When I try to promote them after it says it is "adding them to commoner, the first group in the track" it says the same thing.


tutorial update for primitives v2

Summary
This PR updates the VQE, QAOA, CHSH, and Grovers tutorials for compatibility with PrimitivesV2.
Details and comments
Fixes #

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: nbronn:x: ashsakiYou have signed the CLA already but the status is still pending? Let us recheck it.

Fixes: https://github.com/Qiskit/qiskit-ibm-runtime/issues/1497

@pandasa123 @ashsaki Updates LGTM, fixed some typos and spelling errors. I have not had a chance to run these on a real backend either, and I'd like to do so before merging

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.284%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5764

Relevant Lines:
7005

💛  - Coveralls

Ran all the tutorials, they look good to me


Rails 4.2 support

Hi guys,
I was wondering; is support for Rails 4.2 coming? And when will the new gem version be pushed to rubygems.org?
Cheers!

Rails 4.2 will be supported as soon as someone does the work and submits a pull request. There's no one actively supporting this gem right now. I'm happy to merge PRs and cut releases, but I don't have time to develop this.

Contact with author github.com/div/mailboxer

fixed by #359, released as 0.13.0

Thanks @jcoyne for the heads up!


README-to have more details about the repository

we could add some more brief details about the repository in which it could be included in the README File.

@diego3g  what do you think and any suggestion?

@adarshsaraogi i'm working on creating a project with all the tasks that should be done and the layout. Probably will do that until tomorrow night.

I started designing the Rocket Redis UI inside Figma today (https://www.figma.com/file/YRor24p0TmTxcjl5L86jGb/Rocket-Redis?node-id=0%3A1), i'll try to get it done until tomorrow. Today i'll also create the project here inside Github with every task.

I've created the project with the main functionalities i can think now for the application MVP.
Project: https://github.com/diego3g/rocketredis/projects/1
If you find anything that i could include there, feel free to open an issue and point me to add it to the project if we believe that it can be included in this first version.


Bluetooth: controller: Initial Periodic Advertising and Sync support

Pending:

Generation of LE Periodic Advertising Report

@andrzej-kaczmarek please add yourself as reviewer to this PR (I am not able to add you).

@ppryga FYI

DNM, would like rebase over https://github.com/zephyrproject-rtos/zephyr/pull/28950


RequestBasedStickySessionServiceInstanceListSupplier with blocking client

Describe the bug
Version:
artifactId=spring-cloud-loadbalancer
groupId=org.springframework.cloud
version=3.0.1

In RequestBasedStickySessionServiceInstanceListSupplier instance selection based on checking that context instanceof RequestDataContext, but for blocking client context is not wrapped in RequestDataContext and it is DefaultRequestContext.
How could we overcome this issue? It seems like for not reactive client we can't get HttpRequest information from context to filter instances by url or cookies or another infromation from http request.

@karlovskiy yes, as specified in the docs, it is only supported for the reactive @LoadBalanced Webclient, ReactorLoadBalancerExchangeFilterFunction and RetryableLoadBalancerExchangeFilterFunction.
Will mark this as enhancement and we'll work on it in the future.


Support for unshallow routing when language has changed

Hi,
it would be great if we could provide an option to allow a reload of getInitialProps of the active page. Currently, shallow routing is used.
The problem I'm trying to solve is really common. I have static translation which works great but my product data is fetched from an external source. How should I fix it without to trigger a real browser reload?
This is also useful if you use routeChangeComplete for analytics. Right know, routeChangeComplete has to be fired twice if you want to trigger a refetch of getInitialProps after a language change.
Related:
https://github.com/isaachinman/next-i18next/issues/299
Proposal:
const NextI18NextInstance = new NextI18Next({
  shallowRouteChange: false
})

Hi @StarpTech @isaachinman
I agree the proposal with a flag. This will let the previous and classic behaviour as expected and allow us (people concerned by full refresh) accessing a way to avoid calling Route in parallel to i18n.changeLanguage

@lipoolock There would never be a "full refresh" from a browser perspective. We're talking about a route re-render at the NextJs level.

@StarpTech I've added this behaviour to the core functionality without a flag as I believe it's the right way forward. Feel free to review, and let me know what you think.


[proposal] move memory qos plugin to qos-manager

What is your proposal:
move memory qos plugin from resmgr to qos-manager
Why is this needed:
Is there a suggested solution, if so, please add it:

/milestone v1.2
/assign zwzhang0107

/area koordlet

consider some container may have ignore_resource property

/reopen


Exit After Dump

I'm using mongo-connector to index a large (140M documents) in Elasticsearch.
The initial dump process takes about 24hours to complete. My issue is that my oplog only covers about 15 hours of data at any given time (the database sees a lot of traffic). So when mongo-connector finishes dumping the collection, it tries to read the oplog entry from when it first started and errors out because it doesn't exist.
I feel like the best thing would be to have an option that will kill the mongo-connector when it finishes its dump process. That way you could run two processes one with no-dump to read only the oplog data and one with only-dump to just dump the collection.

@aherlihy
If you have a chance, your input would be appreciated. ^^


Support for conditional path filtering

Implements support for conditional path filtering, e.g.:
"ActionCondFilterPaths": {
    "CondAllOf": [
        {
            "CondBool": true
        },
        {
            "CondNot":
              {
                 "CondPathPredicate": { "PP": "2-0#0" }
             }
        }
    ]
}

Applying the above action to an AppPathSet will yield a new AppPathSet containing the paths that satisfy the condition.

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

go/lib/pktcls/action.go, line 29 at r1 (raw file):

	Act(interface{}) interface{}
	GetName() string
	setName(name string)

Make this public as well.

go/lib/pktcls/action.go, line 37 at r1 (raw file):

// Filter only paths which match the embedded PathPredicate.
//
// DEPRECATED, use ActionCondFilterPaths instead.

As discussed offline, don't deprecate and just make ActionPathFilter what now is ActionCondPathFilter

go/lib/pktcls/action_map_test.go, line 91 at r1 (raw file):

}

func TestMarshalJSONCondActions(t *testing.T) {

Doesnt' apply to this test file, but I think you could unit test the path predicate semantics by programmatically creating some sciond.PathEntryReply (and members thereof) and then compare AppPathSets

go/lib/pktcls/action_map_test.go, line 94 at r1 (raw file):

	Convey("Initialize path predicates", t, func() {
		ppA, err := pathmgr.NewPathPredicate("1-11#18,1-11#87")
		SoMsg("ppA err", err, ShouldBeNil)

These are semantically not unit test, so what we really would like to do here is to panic. Leaving this here as a reference, because I'd need to look into how to do that best with goconvey.

Comments from Reviewable

Reviewed 3 of 5 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

Comments from Reviewable

Freezing this until https://github.com/scionproto/scion/pull/1443 is merged because I want to use it for some testing stuff.

Comments from Reviewable

Reviewed 22 of 22 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

go/lib/pktcls/action_test.go, line 52 at r2 (raw file):

		{
			Name:     "AllOf(AllOf(true))",
			FileName: "act_allof_allof_true",

lets just call these files act_<#>

go/lib/pktcls/action_test.go, line 319 at r2 (raw file):

	t.Helper()

	g := graph.NewFromDescription(graph.DefaultGraphDescription)

There is NewDefaultGraphx

go/lib/pktcls/common_test.go, line 1 at r2 (raw file):

// Copyright 2018 ETH Zurich

As agreed offline, make the write function a bit more general and move this out to some xtest package, because it will be used in many places.xx

Comments from Reviewable

Review status: 9 of 22 files reviewed at latest revision, 3 unresolved discussions.

go/lib/pktcls/action_test.go, line 52 at r2 (raw file):
Previously, shitz wrote…
lets just call these files act_<#>

Done.

go/lib/pktcls/action_test.go, line 319 at r2 (raw file):
Previously, shitz wrote…
There is NewDefaultGraphx

Done.

go/lib/pktcls/common_test.go, line 1 at r2 (raw file):
Previously, shitz wrote…
As agreed offline, make the write function a bit more general and move this out to some xtest package, because it will be used in many places.xx

Done.

Comments from Reviewable

Reviewed 15 of 27 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 10 of 32 files at r2, 14 of 27 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

go/lib/pktcls/cond_test.go, line 115 at r3 (raw file):

				&layers.IPv4{
					SrcIP: net.IP{192, 168, 1, 1},
					DstIP: net.IP{192, 168, 1, 2},

It would be better to use a src IP that is outside the expected network, so that it's clear which one is being matched.

go/lib/xtest/helpers.go, line 74 at r3 (raw file):

// FailOnErr causes t to exit with a fatal error if err is non-nil.
func FailOnErr(t *testing.T, err error) {

t.Helper()?

go/lib/xtest/helpers.go, line 80 at r3 (raw file):

}

// MustMarshalToFile marshals v and writes the result to file

This should probably mention JSON in the name.

Comments from Reviewable

Reviewed 4 of 5 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Feat/test-github-action

Summary
Adds two github actions:

test (on matric of versions)
publish to ghcr on release publish events

Modifies test setup:

uses newer version of setupenv-test and kubebuilder to allow for in ci testing

Docs:

updates docs to cover new commands in makefile

Details:
When running in github, we obviously don't have etcd or other tooling installed by default. The script that was being used to download all of the env setup wasn't working properly in github (I have some failed actions demonstrating that).
In April of this year, controller-runtime switched to a new method: https://github.com/kubernetes-sigs/controller-runtime/commit/782807d8d4fb287e86f6c91862529574ac42cc0a#diff-051375546db9782e3debc25e0241edf1d5e5e2ec0f183dd8634ca5b2c8968bb8
I mimic'd that method with a few modifications to match what was currently happening. I left everything as a makefile script for now, but we may want to move the test-setup command to a bash script at some point to make it easier.
You'll note the addition of an ENV variable, that's based on: https://github.com/kubernetes-sigs/kubebuilder/issues/326
I've currently dropped the kube-builder tools into /tmp/kube-builder but they could theoretically be anywhere. This is really only relevant to people working on the operator.
If there is a different method wanted, then I can explore that route.
I've also configured it so it can run a matrix of tests. This will allow us to test the operator on a variety of linux platforms, with a variety of golang versions, and a variety of k8s versions. I've only included one of each for now for demonstration,
but we can obviously add more.
works towards #20
Successful example here: https://github.com/KnechtionsCoding/operator/runs/2765262646
Next Steps
@simskij will probably need to follow: https://docs.github.com/en/packages/managing-github-packages-using-github-actions-workflows/publishing-and-installing-a-package-with-github-actions#upgrading-a-workflow-that-accesses-ghcrio to allow the actions/workflow to publish to ghcr

Added the permissions needed to ghcr.io. Once you've sorted out @dgzlopes suggestion, I'm happy to merge it. :)


Sanitize multiple spaces in display names to protect against some security concerns

Adds a display name sanitation rule which reduces runs of spaces into a single space.

Before
After

With apologies to rem.

It's possible to bypass this regex by interleaving zero-width codepoints (e.g. U+200B) with whitespace, for example: "              "
const CHECK_MARKS_RE = /[\u2705\u2713\u2714\u2611]/gu
const CONTROL_CHARS_RE =
  /[\u0000-\u001F\u007F-\u009F\u061C\u200E\u200F\u202A-\u202E\u2066-\u2069]/g
const MULTIPLE_SPACES_RE = /[\s][\s]+/g
console.log("hello              world".replace(CHECK_MARKS_RE, '').replace(CONTROL_CHARS_RE, '').replace(MULTIPLE_SPACES_RE, ' ').trim())

@DavidBuchanan314 appreciate it https://github.com/bluesky-social/social-app/pull/5729

What was the security concern?

Using whitespace to push the domain handle off screen so you can impersonate people

Since this is a change to the social-app frontend, usernames with tons of spaces will still be stored on the backend, no?
Meaning other clients using the service (or using the at protocol at large) would still have to face the same sorts of concerns?
if that's the case the api docs for app-bsky-actor-get-profile already state that the username could be 640 characters long, which should be enough of a warning for username length to be truncated appropriately, but it might be a good idea to have a style guide (if I missed one), that tells third party client developers to always have the domain handle visible (given that impersonation is a concern).

@pfrazee still managed to bypass it, btw: test-acc.bsky.social
I think trying to validate like this is not the best idea since there are so many damn characters that can be used to do this (here's a regex I made to validate spaces on Picosky, although not every character might fit this scenario since our use case was slightly different, albeit similar: ensureMultilineValid.ts#L2)
Display names are so problematic... my suggestion is follow what we're doing at Pico, as done by @notjuliet: https://github.com/psky-atp/client/commit/dd48b6aa19db2a5cde5f69a89de92e08e609dc7a
For reference, the character I used in the bypass is Hangul Filler, but I bet there are other characters such as Mongolian Vowel Separator which might also work.

Thanks so much @oestradiol. The only reason I haven't gone with that approach is because that kind of layout control is wildly finicky across all the react-native platforms, but it sounds like we just have to accept reality and attack it from that direction.
Really appreciate the pointers. You win again, unicode.

You're welcome, Paul. Good luck everyone, you got this! <3

@pfrazee

Really appreciate the pointers. You win again, unicode.

Please bear with me to the end.
The design of Javascript purposefully excludes whitespace and control characters outside of a certain scope, as outlined in ECMA-262 12.2 "White space", Note 2:

NOTE 2 Other than for the code points listed in Table 36, ECMAScript WhiteSpace intentionally excludes all code points that have the Unicode “White_Space” property but which are not classified in general category “Space_Separator” (“Zs”)

NB also the following paragraph from 12.3 "Line Terminators":

Line terminators are included in the set of white space code points that are matched by the \s class in regular expressions

ECMA-262 Annex F notes:

12.2: In ECMAScript 2016, Unicode 8.0.0 or higher is mandated, as opposed to ECMAScript 2015 which mandated Unicode 5.1. In particular, this caused U+180E MONGOLIAN VOWEL SEPARATOR, which was in the Space_Separator (Zs) category and thus treated as whitespace in ECMAScript 2015, to be moved to the Format (Cf) category (as of Unicode 6.3.0). This causes whitespace-sensitive methods to behave differently. For example, "\u180E".trim().length was 0 in previous editions, but 1 in ECMAScript 2016 and later. Additionally, ECMAScript 2017 mandated always using the latest version of the Unicode Standard

The conclusion then would be that there are intentional limitations of what \s is intended to capture, which a pathological user uploading arbitrary unicode strings may take advantage of, which makes attempt to stop them futile.
However
Javascripts regexp engine, in unicode aware mode (eg. /.+/u) has a different tool we can use to address this: the character class escape \p{}, (ECMA-262 22.2.1 Patterns, in the bnf under CharacterClassEscape).  For a list of categories one can consult https://www.unicode.org/reports/tr44/#GC_Values_Table (though the paragraph above it is salient and a good reference).   A more interactive version that more easily presents which characters can be seen in those categories is https://www.compart.com/en/unicode/category .
Given that the code already attempts to strip out control characters with a rather daunting capture class range:
https://github.com/bluesky-social/social-app/blob/725e8c1805393ddf87ffb64c35d5787aabfffcad/src/lib/strings/display-names.ts#L9
And that all of the above problematic characters are in the unicode control class category, If no control characters are desired, lines 8-9 could be replaced with:
const CONTROL_CHARS_RE = /\p{C}/gu

if some control chars are desired I would highly recommend whitelisting those in:
const CONTROL_CHARS_RE = /\p{C}/gu  //node automatically orders these gu, which is cute, so I pronounce it guh
const CONTROL_CHARS_WHITELIST_RE = /[\u0600-\u08E2]/ //e.g. Arabic format characters, g is not needed for now, but can be used
const MULTIPLE_SPACES_RE = /\s{2,}/g //two or more captures of whitespace
///[...]
    return str
      .replace(CHECK_MARKS_RE, '')
      .replace(CONTROL_CHARS_RE, (reMatchStr) =>
              reMatchStr.match(CONTROL_CHARS_WHITELIST_RE) ? reMatchStr : '') )
      .replace(MULTIPLE_SPACES_RE, ' ')
      .trim()

If \s, being defined by ecma language needs, and the unicode separator categories differ, /\s{2,}/g above could be replaced with /\p{Z}{2,}/gu to showcase explicit intention, though it is harder to immediately understand intent and should have a comment.
The above replace function argument can be defined elsewhere to improve readability and formatting.  The basic structure is that every control char match is checked to see if it's permitted, and if it is, passed through.  Because it breaks down to a ternary, the truthy String.prototype.match call could be replaced with a Set.prototype.has call, where a set of what control characters are permissible could be outlined as follows:
const controlCharsWhitelist = new Set([
 "\uXXXX", //a reason why this is allowed
 // ...
]);

As all the zero width and nonprinting spaces listed in the ticket's feedback are all found in the control character category, the rough solution here provided would likely fit your needs.
Any questions are welcome, or feedback on the presentational or design aspects outlined here.
I personally disagree with the approach of truncation in this PR and in the above code, and think you should use the layout control option, however.

@JamesKoenig wow, thanks for the valuable input! I didn't know about that, and it's also gonna help me in my application.
I managed to clean up my regex a lot by replacing most of it with simply \p{C}\p{Z}\p{M}. That being said, I still think this is not enough for Bluesky, because for some reason some characters such as U+3164, U+115F and U+1160 are classified as... letters..? While doing my research, I found this source, if that helps at all.


Normal severity alerts

Describe the bug
alerts are switched to severity normal but not closed directly, is there a way to specify which severity to show in the list so i can hide the normal ? or timeout for normal alerts ?
Steps to reproduce the behavior:
wait to receive a resolved or closed status alert
Expected behavior
a week ago alerts used to close automatically, now instead my list is full of "normal" alerts
Screenshots

Desktop (please complete the following information):

OS: Debian Stretch
Browser: any

Your screenshot doesn't show the status of the alerts in the list so I can only assume they are all "open".

alerts are switched to severity normal but not closed directly, is there a way to specify which severity to show in the list so i can hide the normal ? or timeout for normal alerts ?

The web UI is not designed to filter by severity but to filter by status and sort by severity because that's how the alerts are normally managed. The normal alerts should automatically close and disappear from the main console when using the "active" filter.
You say things were working a week ago. What changed? Did you upgrade, reboot, restart after a config change? And please let me know exactly what version you are using. Thanks.

hello, thank you for your reply, there are many people working with me on it so i don't know what was changed... can you tell me please how to set the active filter ? i have prometheus monitoring server, when alertmanager is sendin the "ok" or "resolved" status it stays open

Are you using Prometheus Alertmanager webhook integration? https://github.com/alerta/prometheus-config
Can you share your prometheus and alertmanager configs? Or at least the parts relevant to Alerta inegration?

name: 'alerta'
webhook_configs:

url: 'http://tls-prometheus-tsg:8080/webhooks/prometheus'
send_resolved: true

I'd need to see an example rules file, as well. Thanks.
I can see straight away that the status is "open" when it should be "closed". What version of Alerta are you using?

alerta 6.7.5
alerta client 6.5.0
requests 2.21.0
click 7.0

all my alert rule files have severity critical,warning,notice.. but no normal.. so im sure prometheus doesnt have a 'normal' severity

Can you copy/paste the prometheus rule below that generated the alert in question? Thanks.

alert: highCpuLoad1
expr: (node_load1{subtype=~'.postopia.', job!~".-(AWS|AWS-.|ana|sad)", environment!="alpha"}  > 40)
for: 2m
labels:
rulefile: postopia.yml
severity: critical
iris_plan: iris_plan
annotations:
summary: "high cpu load1 {{$value}} on  {{$labels.node}} ({{$labels.ip}} {{$labels.unifiedlocation}})"
description: "High CPU Load1 {{$value}} on {{$labels.node}} for more than 2 minutes ({{$labels.ip}} in {{$labels.unifiedlocation}})."
(edited)

found this too :
Feb 27 17:25:32 tls-prometheus-tsg gunicorn[99827]: INFO:flask.app:State Transition: Rule #ALL-* STATE=open     ACTION=         SET=open     SEVERITY=critical     -> normal   HISTORY=open    -> open     => SEVERITY=normal  , STATUS=open
Feb 27 17:25:32 tls-prometheus-tsg002 gunicorn[99827]: INFO:flask.app:State Transition: Rule #ALL-* STATE=open     ACTION=         SET=open     SEVERITY=normal       -> critical HISTORY=open    -> open     => SEVERITY=critical, STATUS=open

i fixed it... i noticed that the auto close rule has a condition if state=normal.. and the default normal was 5 before i was adding more severities to alerta and i changed it in my alertad.conf to 8 which was causing the alert not to match the condition, when i put it back at 5 all normal alerts are closing
the code i found was in /usr/local/lib/python3.5/dist-packages/alerta/models/alarms/alerta.py line 115


Please auto-merge pre-1.0 rust dependencies

While semver itself allows any breaking changes in 0.x.y release, cargo treats 0.x.y as if x was the major version and y was the minor version. This means that in the rust ecosystem, 0.5.2 is considered to be compatible to 0.5.1.
Currently, pre-1.0 rust dependencies are not automatically merged, even though most crates are 0.x.y versions and behave according to the cargo semver rules. Please enable auto-merging for pre-1.0 crates.

Automerge is no longer supported in GitHub-native Dependabot, so closing this issue.


App won't launch after installing it with Enterprise installer

User needed a Toggl Desktop which would install in Program Files (or a different location) so all users on the Desktop could use it when logged in. We recommended Enterprise installer but after the installation, the app won't launch and it gives Appcrash errors:

Problem signature:
Problem Event Name: APPCRASH
Application Name: TogglDesktop.exe
Application Version: <IP_ADDRESS>
Application Timestamp: 589c712a
Fault Module Name: MSVCR120.dll
Fault Module Version: 12.0.21005.1
Fault Module Timestamp: 524f7ce6
Exception Code: 40000015
Exception Offset: 000a7676
OS Version: 6.1.76<IP_ADDRESS>.256.4
Locale ID: 1061
Additional Information 1: dd93
Additional Information 2: dd93ef6d3211883182d4a88a8c8fe85f
Additional Information 3: a482
Additional Information 4: a4827a939525efa4460f70a46490d84c

Case #434827

case/434134

will be fixed with #2955


Grid border not closed at left-top corner (no z in SVG path)

Version
v5.1.1
Reproduction link
https://qp9zl.csb.app/
Steps to reproduce
render with option
const option = {
        grid: {
          show: true,
          borderWidth: 3,
          borderColor: "#000000"
        }
      };

What is expected?
corner closed
What is actually happening?
corner not closed

I'm not sure why this commit https://github.com/ecomfe/zrender/commit/da94b645863f50d6975c4c56e3c97f08bd5edd4f removed ctx.closePath(). @pissang

@plainheart I thought the first point and the last point has been connected in the roundRect.ts helper so there is no need to do closePath again, which is an extra cost in the case that drawing lots of rect(like in benchmark.html). But it seems to be a mistake in this case.
Also, only the SVG renderer that drawing a rectangle without border-radius will have this issue. Because canvas renderer will use ctx.rect command, which is closed underlying automatically. And drawing rectangle with border-radius also has no issue because there is an arc command connecting the first point and last point, which is like:

So I think what we need is adding an Z command when rendering rect command in the SVG renderer https://github.com/ecomfe/zrender/blob/da94b645863f50d6975c4c56e3c97f08bd5edd4f/src/svg/graphic.ts#L229

@pissang Yes. I agree with you. That's right.


Fix deprecation warning for v::BasicTracedReference::operator->()

Replace uses of operator->() with Get() per the deprecation guidance.
Test: bazel build //...

Thanks Dom, thanks Felix.


Disabling authentication & signups does not, in fact, disable authentication / sign-up

I've disabled authentication in my config file, by including:
:authentication:
    :enabled: false
    :signup: false
    :signin: false
    :autoverify: false

However, when I visit the site, I still see a sign-up link, as seen below.

When I click on the link, I am able to enter my email address and a password, and it tries to send the mail in the background:

I'd expect those config settings to disable authentication altogether, however all it seems to do is remove the top right buttons for sign-up and sign-in.

This feature was added in #451 (from an older request #170). It disables session authentication as well but the signup routes and logic should be disabled :signup: false. I'll be pushing a fix for #512 today so I'll see if I can get this in too for v0.16.1.

These changes are in the v0.16.1-RC1 release and develop. Now, when authentication is disabled, all auth-related web and api endpoints return a 404 like they don't exist at all.
If you get a chance to have a look, let me know it works for you.

I've just tested it with v0.16.1-RC1, and it does indeed disable the sign-up. Thanks for the quick response!


Removing apiextensions.k8s.io/v1beta1 references

Replacing them by apiextensions.k8s.io/v1.

@kleesc Can I have an approval to run the workflow?

/assign @kleesc

@ricardomaraschini Thanks for this! We aren't currently using the features of Kubebuilder that utilize these files, but it doesn't hurt to merge this.


ns-viewer: command not found

Describe the bug
I am using the CLI to run nerfstudio scripts.  All of the CLI commands seem to work for me, EXCEPT when I try to use ns-viewer, I get the following error: ns-viewer: command not found
To Reproduce
Steps to reproduce the behavior:
1.) In a terminal, activate nerfstudio conda environment.
2.) cd nerfstudio
3.) ns-viewer
Expected behavior
I expect the nerfstudio viewer to be run, and to be able to then access it via my browser.
Screenshots

Additional context
I am on a Linux machine using Ubuntu 20.04.

I pulled the latest from this repo and re-installed.  The problem is resolved.


Calculated attribute values

We need to be able to designate an attribute as calculated.
This would imply that it is readonly, and that it executes some code to determine its value automatically.
It could involve code and values of other attributes in the model.
This concept was the ValueProvider concept from NuPattern.

We can define that an attribute takes code instead of a value. That code can be scriban code that is executed when running any automation.
We need to define a time and place to execute these kinds of functions.
Right now that is only when CommandLaunchPoint is executed.
We probably need to define other LaunchPoints, like when a property changes.


BUG: pd.eval with numexpr engine coerces 1 element numpy array to scalar

Closes #10546. Used assert_numpy_atray_equivalent to guarantee array comparison, not scalar. This should be prior to #10542 and being changed to use assert_numpy_array_equal in #10542.

@jreback Can you take a look for this?

looks ok. just a suggestion I put above (but may not be exactly correct)


optimize CASE WHEN expr for no-nullable int8 type

support SIMD multi select if implemnts in int8 more defail in (#967)

Wait to test a suitable value to remove the configuration item

run starrocks_be_unittest


Make all tables in elapsed time dashboard sortable on Roslyn Dashboard.

Copied from TFS work item #234594.
Make all tables in elapsed time dashboard sortable

Completed.


Rubocop warning removal

This PR works to remove a few warnings RuboCop warns about.

We're green!

🎉


[FEATURE] Clear local image cache via cache clear service and command

Simply renames the cache dir, deletes it as a whole and creates a new one. The command name is sulu:media:cache:clear
tasks:

[x] Move cache clear into the dedicated format caches and execute them
[x] Add entry to CHANGELOG.md

informations:

q
a

Fixed tickets
fixes #808

Related PRs
#1052

BC Breaks
none

Documentation PR
n/a

Reopens #1052 as the branch was deleted by accident and GitHub no longer allows reopening it after rebase :dizzy_face:

@danrot ACK from me

@patkar The rest looks also good for me, if you could fix my few minor comments I would give it a test run, and merge it right afterwards. Will do that quite fast, since it is for sure not only your fault that this PR is so long running :confused:

@danrot fixes are done

Are you living two lives? :open_mouth:
Will merge as soon as tests are passing.

yeah, amending a commit with a different mail address than the original xD

Build is green: https://travis-ci.org/sulu-io/sulu/builds/108780636 :smile:

I am really sry to bother you again, but is it possible that the new folder is created with the wrong permissions? Have to correct again, before the generation is working again. Why do you first move the folder, before deleting? If we would just delete the content of the existing one it would make no problems for sure.

Sounds odd, because the code
$this->filesystem->rename($realCacheDir, $oldCacheDir);
$this->filesystem->mkdir($realCacheDir);
$this->filesystem->remove($oldCacheDir);

is exactly the same as symfony deletes his cache folder if it would do no warmups. This is the reason why I simply move the the cache folder and generate a new one to atomically delete the old one in silent. IMHO this should prevent race conditions with file generations and deletions.
As said, it's inspired / copied from the symfony clear cache command and thus the $this->filesystem->mkdir($realCacheDir); call should create the folder with the default webserver permission of new folders.

@danrot maybe we change the permission to be set on /uploads instead of /uploads/media. so the uploads folder can be used for custom bundles.

I see. In an deployment environment, where the write permission is only in the web/uploads/media folder this is bad since the system can't create a new folder within web/uploads :/

@alexander-schranz Are you talking about web/uploads? I think that's a good solution anyway. Then the only thing we would have to do is to mention this in the UPGRADE.md file in this PR, and update the documentation at http://docs.sulu.io/en/latest/book/getting-started/setup.html#complete-the-installation

@danrot yes the web/uploads

@danrot UPGRADE.md updated and documentation PR created.

@patkar Thank you! :tada: It's incredible how fast things are going here :wink:

yaaaay :tada:


Inconsistent naming of (PowerToys) Awake

On some places (like the issue label or the settings) it's named Awake. On other places (like the PR or named by @dend) it's named PowerToys Awake.
A consisted naming (before release) would be good. I prefer PowerToys Awake, becauseall the PowerToys which haven't a standalone name (f.i. FancyZone) or "PowerToys" in their name (f.i. PowerToys Run) are getting translated. And personally I don't think it sounds good, if Awake gets translated

This should be PowerToys Awake. Valid feedback

This would solve my issue of not recognizing awake as non-malware and linking it to PowerToys directly. Since the update, I had to figure out what this extra piece of software was on my taskbar.

#11978 should be addressing the tray issue.

This would solve my issue of not recognizing awake as non-malware and linking it to PowerToys directly. Since the update, I had to figure out what this extra piece of software was on my taskbar.

same here, thought i got some unwanted bloatware or virus

FYI: docs are done, labels for this issue need update (Area-UI)

Hey @Jay-o-Way can you send me an email<EMAIL_ADDRESS>

Hey @Jay-o-Way can you send me an email<EMAIL_ADDRESS>
I did. The 14th. What did you want to talk about? @crutkas

Can you do me a solid and send it again?  I can’t seem to find it. Want to chat :). (It is a good thing)

@Aaron-Junker can you say what exactly is left to adjust?

@Aaron-Junker can you say what exactly is left to adjust?

The title in the settings is still only Awake

The title in the settings is still only Awake

Honestly and personally, I don't think this is a problem. It makes total sense that "Awake" is a part of PowerToys, when working with the PowerToys Settings window. Having that said: "PowerToys Run" is a similar case. But I guess that this module could be mistaken with Windows Run...

Closing this issue - will work on bringing consistency within Awake, but will leave it to the team to make sure that the right branding policy is followed.


remove Revision series for stat functionality

Current functionality is not complete and will be reimplemented later

Radio buttons for the Revision stats are commented out


Node fails to sync on QA / Dev

Need to investigate why

created ticket here: https://substrate.stackexchange.com/questions/3846/lightsyncstate-is-removed-from-default-client-chainspec

Can sync a node on devnet using version 1.4.0

Following options are still viable:
1: Initially sync the chain using the old version of the network (before upgrade to polkadot 0.9.24) and run the new binary after the node stops syncing
2: Sync from an existing data dir (a dump needs to be created in order to start from a certain height)

1: is working for devnet, but not for qa-net. Tried images: tfchainnode:1.8.0, dylanverstraete/tfchain-qanet & dylanverstraete/tfchain-qanet2

fixed in https://github.com/threefoldtech/tfchain/pull/368

@coesensbert I will create a new image that can be used for qanet/devnet, with this new image you don't need to specify bootnodes anymore! :-)

I also suspect that the issues lies within the kubernetes cluster where both qa/dev are deployed. Adding more bootstrap nodes fixed the issue but it's still not ideal behaviour.


Access to Cache.db is denied

Ive installed HeliumCat on a web app service in azure. My first query using any of the commands (front, radius, etc.) I get results output.
Subsequent commands though result in the following error:
Unhandled exception. System.TypeInitializationException: The type initializer for 'HeliumCat.Services.CacheService' threw an exception.  ---> System.UnauthorizedAccessException: Access to the path 'D:\home\site\wwwroot\cache.db' is denied.    at System.IO.Strategies.FileStreamHelpers.FlushToDisk(SafeFileHandle handle)    at System.IO.Strategies.BufferedFileStreamStrategy.Flush(Boolean flushToDisk)    at System.IO.FileStream.Flush(Boolean flushToDisk)    at LiteDB.StreamExtensions.FlushToDisk(Stream stream)    at LiteDB.Engine.DiskService.Write(IEnumerable 1 pages, FileOrigin origin)    at LiteDB.Engine.WalIndexService.CheckpointInternal()    at LiteDB.Engine.WalIndexService.Checkpoint()    at LiteDB.Engine.LiteEngine.Rebuild(RebuildOptions options)    at LiteDB.LiteDatabase.Rebuild(RebuildOptions options)    at HeliumCat.Services.CacheService..ctor()    at HeliumCat.Services.CacheService..cctor()    --- End of inner exception stack trace ---    at HeliumCat.Services.HotspotService.GetHotspotByName(String name)    at HeliumCat.Commands.Commands.GetHotspotByIdentifier(String identifier)    at HeliumCat.Commands.Commands.RadiusBeaconStats(RadiusOptions options)    at Program.<>c.<<<Main>$>b__0_1>d.MoveNext() --- End of stack trace from previous location ---    at CommandLine.ParserResultExtensions.WithParsedAsync[T](ParserResult1 result, Func2 action)    at Program.<Main>$(String[] args)    at Program.<Main>(String[] args)
If I delete cache.db, rerun my query \ cmd, then I get results again.
FWIW, i'm using AzureDevOps to perform a deployment to the app service using zip deploy

Thanks for reporting this.
Do you upload cache.db file as well ?
It might be a good idea to run a command first on your local machine so that it creates the cache.db file. Then upload all the files (including cache.db) to azure.


Worker fix

Awesome, with this simple fix (my bad!) worker.html works like a charm, thanks Alon!
However... now simple.html fails :)
Exception... "Illegal operation on WrappedNative prototype object"
nsresult: "0x8057000c (NS_ERROR_XPC_BAD_OP_ON_WN_PROTO)"
location: "JS frame :: http://localhost.org:8123/j2k.js/openjpeg.js :: <TOP_LEVEL> :: line 297"
data: no

simple.html is now fixed in a4abb8e0538d68f54211f8f441ee5a8c99886b19


Mark internal FFT functions as static

This results in a slightly smaller overall FFT function when the compiler inlines the stage/merge functions, when it might not otherwise.
This also resolves the GCC warnings:
warning: no previous declaration for 'merge_rfft_f16' [-Wmissing-declarations]
warning: no previous declaration for 'merge_rfft_f32' [-Wmissing-declarations]
warning: no previous declaration for 'merge_rfft_f64' [-Wmissing-declarations]
warning: no previous declaration for 'stage_rfft_f16' [-Wmissing-declarations]
warning: no previous declaration for 'stage_rfft_f32' [-Wmissing-declarations]
warning: no previous declaration for 'stage_rfft_f64' [-Wmissing-declarations]

It seems some projects are calling stage_rfft_f* from outside CMSIS, prototypes should probably be moved to a header, not marked static.


TypeError: Cannot read properties of undefined (reading 'slice')

Please include some context and the contents of the console in your browser's developer tools.
JavaScript Error
TypeError: Cannot read properties of undefined (reading 'slice')
    at parseKey (<anonymous>:8:51708)
    at e.value (<anonymous>:8:52335)
    at e.value (<anonymous>:8:51601)
    at e.value (<anonymous>:8:51423)
    at e.value (<anonymous>:8:50407)
    at e.value (<anonymous>:8:50053)
    at m.value (<anonymous>:8:40009)
    at t.format (<anonymous>:8:112140)
    at xc (<anonymous>:8:113557)
    at bf (<anonymous>:3:186649)

Reproduction Steps
Please tell us what you were doing when this error occurred, so we can more easily debug it and find a solution.

…

User Agent
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Saya mengklik tombol kembali


Migrate to new ESLint configuration file format

https://eslint.org/docs/latest/use/configure/configuration-files-new

Should address this:
https://github.com/vezel-dev/celerity/blob/5eefb514778bedfc8fa2c1ac3ab03bd2efe01c01/src/extensions/vscode/.eslintrc.json#L4

This is needed to update to ESLint 9.0.0.

Also, the configuration file should maybe be synchronized between the Celerity and TON repos as part of this.


[6.0] Move public from extension decl to each decl in the extension in generated files

Explanation: In the original PR, we refactored code in the CodeGeneration package to not contain public extensions. To reduce the risk of merging conflicts in cherry-picks into release/6.0, we are merging this refactored code into this branch as well.
Scope: Generated code.
Issue: N/A
Original PR: #2609
Risk: Low.
Testing: Passes current test suite.
Reviewer: Alex Hoppen (@ahoppen)

@swift-ci Please test


Use AS_HELP_STRING instead of obsolete AC_HELP_STRING macro

AC_HELP_STRING has been obsolete since at least Autoconf 2.53, which was released in 2003.

Merged. Thanks @hughmcmaster


adding User Agent to the HttpClient

This change adds the User Agent, using the Plugin Version, Dalamud Version and OS Version to give more insight into the actual usage of the Universalis API, which just added User Agents to their Dashboard.

Good idea


[BEAM-960][BEAM-8033][BEAM-1675] Improvements to JdbcIO

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PostCommit

Run Java PostCommit

Run Java PostCommit

Run Java PostCommit

Postcommit run: https://ci-beam.apache.org/job/beam_PostCommit_Java_PR/834/

@pabloem Why not three different PRs since they are related to different Jiras?

Mainly laziness....  : ) @aromanenko-dev
These three commits+two jiras are pretty related to each other:

[BEAM-960] support coder inference for JdbcIO
[BEAM-1675] Deprecate withCoder in JdbcIO
Update JdbcIO integration test to not rely on withCoder

I'll make another PR for the other one ([BEAM-8033] Throwing an error if a dataSourceProviderFn is defined tw…)

opened https://github.com/apache/beam/pull/15963 for the unrelated change


sql/parse: fix != parsing.

Current coverage is 55.44% (diff: 100%)

Merging #80 into master will increase coverage by 0.29%

@@             master        #80   diff @@
==========================================
  Files            33         33
  Lines          1291       1295     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            712        718     +6
+ Misses          534        532     -2
  Partials         45         45

Powered by Codecov. Last update 92c1449...f7a944a


Is there support for IOP modules?

Wondering if the extension supports them, if not, perhaps you know a way to load them into Ghidra?

This extension is mainly intended for use with the MIPS R5900 core in the EE. The IOP is a MIPS R3000 core, so you could try using the MIPS:LE:32:default processor included by Ghidra. If you are in fortunate position of having an IRX module with mdebug symbols, the symbol table parser included with ghidra-emotionengine-reloaded should still work with it even when not using the R5900 processor.
IOP modules are ELF files so they should load normally. I know Ziemas has done some stuff with IOP modules in Ghidra, although I haven't used his extension myself and it's probably outdated at this point:
https://github.com/Ziemas/ghidra_irx
It looks like its main feature is to name functions referenced by the import/export tables (which are used to dynamically link the IOP modules together using stubs).

and it's probably outdated at this point:

big issue is that due to the many versions of IRX modules around its kinda hard to get a complete and decent list...
fortunately it has lots of imports from system modules. wich are great to quickly identify thread creation, RPCs and so on...


start_appinventor do not work

Describe the bug

Affects

[ ] Designer
[ ] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior

Steps to reproduce

Can you describe more about the issue?


Add GET, PUT and DELETE methods to events controller

For record lookups:
Decided to use find_by(id: id) instead of find_by_id(id) (being phased out) and where(id: id).take (same performance).

We should also note that the reason for using find_by(id: id) instead of just find is that find throws an exception if the event is not found, while find_by(id: id) simply returns nil, which is what we want.


Freeze colorama to version >=0.4.6

This solves and issue where older versions of colorama do not work. This can also serve as a guide for packagers.
Closes #165

To be specific, just_fix_windows_console was added in 0.4.6

Sorry for late review of this pull request, didn't have enough time during the summer.
Merging now.


Load models on boot

Currently SQL is run from various gems (like SchemaPlus) when a model instance is first loaded. This usually happens on first request, which generates misleading data for various test tools and production monitoring.
e.g.
SELECT a.attnum, a.attname FROM pg_attribute a WHERE a.attrelid = ? AND a.attnum IN (?)
SELECT distinct i.relname, d.indisunique, d.indkey, pg_get_indexdef(d.indexrelid), t.oid, pg_catalog.obj_description(i.oid, ?) AS comment, (SELECT COUNT(*) FROM pg_opclass o JOIN (SELECT unnest(string_to_array(d.indclass::text, ?))::int oid) c ON o.oid = c.oid WHERE o.opcdefault = ?)  AND d.indisprimary = ? AND t.relname = ? AND n.nspname = ANY (current_schemas(false)) ORDER BY i.relname

Load models on boot so this SQL is executed before first web request / job run.

Still not resolved.. also need to check Scout after 24 hours


Roll Flutter Engine from 67a7fe18c014 to e4423702930b (3 revisions)

https://github.com/flutter/engine/compare/67a7fe18c014...e4423702930b
2024-08-22<EMAIL_ADDRESS>vulkan_glfw validation layer logging (flutter/engine#54607)
2024-08-22<EMAIL_ADDRESS>Roll Skia from 4c66b7e42027 to 7611984dc27b (1 revision) (flutter/engine#54710)
2024-08-22<EMAIL_ADDRESS>Roll Dart SDK from 025bf8d376d3 to ce160bf13347 (1 revision) (flutter/engine#54709)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Loss function of run_classifier.py takes in 2 inputs of different dimensions.

I am having an error here https://github.com/huggingface/pytorch-pretrained-BERT/blob/master/examples/run_classifier.py#L836
In this line, loss = loss_fct(logits.view(-1, num_labels), label_ids.view(-1)), suppose we have 2 labels (entailment vs. not_entailment like QNLI task), then,
logits is already in dimension batch_size x num_labels. Using logits.view(-1, num_labels) will convert logits into an array of length (2 times batch_size) x 1.
So, this logits does not match label_ids.view(-1) which is batch_size x 1.
Does anyone else see this error when running the code ?
Thanks.

Closing issue, because I pass in the num_labels as 1 instead of 2 for the QNLI task. I was thinking that giving 1 label is enough because the 2nd label can be inferred from the 1st one.


Refactor member input for add member command

Changes:

Refactored member input for add member command to take in member index instead of member name
Updated and fixed user guide

Codecov Report
Base: 39.82% // Head: 39.40% // Decreases project coverage by -0.42% :warning:

Coverage data is based on head (a37c05a) compared to base (3c747b4).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #115      +/-   ##
============================================
- Coverage     39.82%   39.40%   -0.43%
- Complexity      452      453       +1
============================================
  Files           137      138       +1
  Lines          2541     2576      +35
  Branches        323      325       +2
============================================
+ Hits           1012     1015       +3
- Misses         1463     1496      +33
+ Partials         66       65       -1

Impacted Files
Coverage Δ

...seedu/address/logic/commands/AddMemberCommand.java
13.33% <0.00%> (+2.22%)
:arrow_up:

...eedu/address/logic/commands/AssignTaskCommand.java
5.88% <ø> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/TaskListPanel.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/RootCommand.java
50.00% <0.00%> (ø)

...a/seedu/address/logic/commands/TasksOfCommand.java
11.76% <0.00%> (ø)

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

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


feat(bridge-ui-v2): Claim & Release

TODO: unit testing will come in a second PR, after this gets reviewed

Successful bridging and transactions on taiko