Added ability to upload full prebuilds folder.

see https://github.com/mafintosh/prebuild/pull/64

see https://github.com/mafintosh/prebuild/pull/64

2.8.0

2.8.0


Bug 1947402: Add policyv1 scheme

This is necessary in order to add PDB helpers in library-go (https://github.com/openshift/library-go/pull/1056).
/assign @deads2k
CC @openshift/storage

/assign @deads2k


Ignore empty lines when uncommenting code

Fixes #81474

I did some test with vscode and have updated my code a little bit according to vscode's default behaviour, the code works this way now:
When commenting/uncommenting code, empty lines do not affect godot's decision of whether the selected code is commented or not,  unless they are all empty lines,  in that case I treat them as uncommented code.

Thanks!


Fix 3 broken unit tests

Running the unit tests with npm test consistently produces 3 errors. We should fix them so that the tests pass cleanly.

npm tests are gone


refactor(workspace): convert the project in a workspace

Still a draft, I want to see if the CI whould run correctly


Unknown device info type 21

Ahoj, v logu mám neustále toto:
`Logger: custom_components.jablotron100
Source: custom_components/jablotron100/jablotron.py:2030
Integration: Jablotron 100 (documentation, issues)
First occurred: 13. dubna 2022 15:03:50 (26 occurrences)
Last logged: 17:09:35
Unknown device info type 21: 0f00d52823d2af0101
Unknown device info type 21: 0f00d5282300000101`
Tušíš prosím, jak to odstranit?

Try version 3.10.1 - There's should be probably more info in the log.

Hi guys,
same here.
HACS JA100 Integration
Logger: custom_components.jablotron100
Source: custom_components/jablotron100/jablotron.py:2030
Integration: Jablotron 100 (documentation, issues)
First occurred: 19:02:29 (1 occurrences)
Last logged: 19:02:29
Unknown device info type 21 (device 234): 900cea0a090f00d5640000000000
core-2022.5.4
Home Assistant OS 8.0
supervisor-2022.05.2
HACS
Integration version: 1.24.5
Jablotron 100+ 3.1.10
DEBUG:
2022-05-16 19:34:28 DEBUG (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Incoming (device 233): 900be90a080f00a60400000000
2022-05-16 19:34:28 DEBUG (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Incoming: d0089630843ff08ab34c
2022-05-16 19:34:29 DEBUG (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Incoming (device 234): 900cea0a090f00d5640000000000
2022-05-16 19:34:29 ERROR (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Unknown device info type 21 (device 234): 900cea0a090f00d5640000000000
Any suggestions? Everithing is working fine only this message apears periodicaly every hour in the log.
Thanks in advance.
Martin

It will be not reported in next version.

Many thanks for response and your information. Can I ask you, just to satisfy my curiosity what is the reason for this error message?
Pěkný den. 😉
Martin

On 19. 5. 2022, at 19:19, Jaroslav Hanslík @.***> wrote:
It will be not reported in next version.
—
Reply to this email directly, view it on GitHub https://github.com/kukulich/home-assistant-jablotron100/issues/41#issuecomment-1131980595, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYIHDCVAX4NMW23MG67QCY3VKZZ2RANCNFSM5TOCZI4Q.
You are receiving this because you commented.

Released in 3.11.0.


edited

:star: PR built and available in a preview environment ashu1019singh-demo-node-2-pr-1 here


Use _ScaffoldLayout to position scroll-resizable appbars

Use _ScaffoldLayout to layout the appbar, even if it's going to be resized when the app is scrolled. As a consequence, the drawer is always stacked on top of the appbar.
Fixes https://github.com/flutter/flutter/issues/2923

LGTM
LGTM

LGTM


Add Repl() to ValueKey.

fixes #735

Codecov Report

Merging #737 (34f2ff3) into main (e3b2ba4) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #737      +/-   ##
==========================================
- Coverage   81.15%   81.07%   -0.09%
==========================================
  Files          49       49
  Lines        6539     6543       +4
  Branches     1569     1571       +2
==========================================
- Hits         5307     5305       -2
- Misses       1039     1042       +3
- Partials      193      196       +3

Impacted Files
Coverage Δ

amaranth/hdl/ast.py
89.59% <0.00%> (-0.38%)
:arrow_down:

amaranth/_toolchain/cxx.py
95.23% <0.00%> (-4.77%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


DateTime fields are not editable if data format is TOML

It seems like when editing entries the UI is hiding any field named "date" by default even if it's in the collection list. It will show up in the preview pane on the right but not in the edit pane. This happens with both a top level key and a nested key. More specifically the fields are visible when "creating" but not when "editing".
Example post:
+++
title = "Test"
date = 2024-10-18T00:00:00

[nested_test]
date = 2024-10-17T00:00:00
+++

Example collection:
collections:
  - name: "posts"
    label: "Posts"
    folder: "content/posts"
    format: "toml-frontmatter"
    extension: "md"
    fields:
      - label: "Title"
        name: "title"
        widget: "string"
      - label: "Date"
        name: "date"
        widget: "datetime"
      - label: "Nested"
        name: "nested_test"
        widget: "object"
        fields:
          - label: "Nested Date"
            name: "date"
            widget: "datetime"

Example of a the nested date field missing in the edit pane on the left, but showing above "price" in the preview pane:

There's also strange behaviour with the summary field: summary: "{{fields.date}}" works fine but shows the massive datetime string, yet summary: "{{fields.date | date('L')}}" will simply be blank where the date is when the object is collapsed.

I can see the basic intention of this functionality, in certain SSG it is not best practice to edit the "date" field since it's interpreted as a "created_date" and you should instead change the "updated_at" field instead, but having this enforced with no way to opt out is a little strange; especially when it affects all fields with the name including nested fields.
If someone wants to replicate this functionality there should simply be a hidden: true for widgets, or maybe even a disabled: true to just toggle the input's "disabled" property and make it uneditable instead.
As far as I can tell the workaround is to just rename all of the fields to be something else (like post_date or nested_test.nested_date) but that's kind of strange.

Thanks for the heads-up! Will check.

Looks like nesting doesn’t matter; It’s due to a difference between the TOML parser and the JSON/YAML parsers. Fixinig.

Fixed in v0.41.1 🙇🏼

Thanks for looking into it and the quick fix 👍


chore: add env docs for xcuitest

Description

Add docs for configuring env vars for xcuitest + simulators
Motivation and Context

Document new capabilities.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

@mhan83
Thanks for this, will you also update https://docs.saucelabs.com/dev/cli/saucectl/run/#--env?

@mhan83
Thanks for this, will you also update https://docs.saucelabs.com/dev/cli/saucectl/run/#--env?

Sure, can do.


Fix completions of exports elsewhere in same file

Fixes #43289, along with another bug I found on the way:
module.exports = { a/**/ } // has a completion for itself

and
const c = 0;
export { c as something };
export * as somethingElse from "./other";

s/**/ // has completions for 'something' and 'somethingElse'

@sandersn I realized I couldn’t really fix the primary bug (#43289) without addressing the contextual typing thing head-on, which I think I’ve done now. getContextualTypeForThisPropertyAssignment is now only used for this (and aliased-this) property assignments. We now group AssignmentDeclarationKind.ExportsProperty, .Prototype, .PrototypeProperty, and .ModuleExports together as being the simpler cases where they can have a contextual type only via a JSDoc type annotation, whereas .Property and .ThisProperty might be new declarations, might be immediately annotated with JSDoc, or might have a contextual type from the original declaration of the object. Some of the simpler cases were incorrectly being fed through that more complicated logic.

@typescript-bot user test this


Update release process and release script to add new features

The library starter project has a new release script (first version). It's similar to the ponyc one but takes 2 extra args and posts the release notes on GitHub.

PR opened: https://github.com/ponylang/ponyc/pull/3368


Strings autocompleted as paths

Type: Bug

Behaviour
Expected vs. Actual
Expected: The autocomplete dialogue doesn't interrupt my attempts to type every time I press ".
Actual: VS Code figures it knows all possible string contents and can autocomplete them, and it is wrong.
Steps to reproduce:

Create a new .py file
Type "

Rant
Most irritatingly, typing """ and then enter inserts .git/. I would prefer it insert a newline, and perhaps a closing triple quote, without having to train myself to mash escape. That said, it's extremely frustrating in all circumstances. Typing a string doesn't mean I won't need to glance at the code above the current line anymore. And I have literally never a single time used this feature, because literally not even a single time has the string I want to type been a relative path from the workspace root. Even when I want to type a path, it's relative to the package root or current file, so it's still wrong!
This can be worked around, at the cost of removing all filepath completions, by adding this to your config:
    "[python]": {
        "editor.suggest.showFiles": false
    }

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.11.3
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

(you don't want to look at 28k of unrelated log lines, so I'm not pasting it here.)

User Settings

languageServer: "Jedi"

experiments
• enabled: false

Extension version: 2023.8.0
VS Code version: Code - OSS 1.78.2 (b3e4e68a0bc097f0ae7907b217c1119af9e03435, 2023-05-12T07:29:20.595Z)
OS version: Linux x64 6.3.2-arch1-1
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
AMD Ryzen 9 7900X 12-Core Processor (24 x 5067)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 1, 1

Memory (System)
30.50GB (26.53GB free)

Process Argv
. --enable-proposed-api ms-python.python

Screen Reader
no

VM
0%

DESKTOP_SESSION
i3

XDG_CURRENT_DESKTOP
i3

XDG_SESSION_DESKTOP
i3

XDG_SESSION_TYPE
x11

This is coming form Jedi completions, I have asked to see if there is a setting that we can use to turn this off: https://github.com/pappasam/jedi-language-server/issues/274
[Trace - 11:01:16 AM] Received response 'textDocument/completion - (18)' in 17ms.
Result: {
    "isIncomplete": false,
    "items": [
        {
            "label": ".devcontainer\\",
            "kind": 17,
            "sortText": "v00",
            "filterText": ".devcontainer\\",
            "insertText": ".devcontainer\\",
            "insertTextFormat": 1
        },
        {
            "label": ".git\\",
            "kind": 17,
            "sortText": "v01",
            "filterText": ".git\\",
            "insertText": ".git\\",
            "insertTextFormat": 1
        },
        {
            "label": ".github\\",
            "kind": 17,
            "sortText": "v02",
            "filterText": ".github\\",
            "insertText": ".github\\",
            "insertTextFormat": 1
        },
        {
            "label": ".gitignore",
            "kind": 17,
            "sortText": "v03",
            "filterText": ".gitignore",
            "insertText": ".gitignore",
            "insertTextFormat": 1
        },
        {
            "label": ".mypy_cache\\",
            "kind": 17,
            "sortText": "v04",
            "filterText": ".mypy_cache\\",
            "insertText": ".mypy_cache\\",
            "insertTextFormat": 1
        },
        {
            "label": ".nox\\",
            "kind": 17,
            "sortText": "v05",
            "filterText": ".nox\\",
            "insertText": ".nox\\",
            "insertTextFormat": 1
        },

Once we have a setting from jedi-language-server we can pass it through to the server here:
https://github.com/microsoft/vscode-python/blob/713007f035b77d2724f4a471b97f53db1a9a12e3/src/client/activation/jedi/analysisOptions.ts#L58-L85


Remove whitespaces

Coverage increased (+0.02%) to 76.609% when pulling a60b76b18cb80e02ae6d7beb799081e411446b0e on amarcine:v1.0 into 321f33723bc9ecdd7f1475e5899657b67efc96a2 on contiv:v1.0.


Issues with default build task flow with multiroot

Testing #34953
If I don't have a task marked as the default build task, I can configure one as in #34953 and run it, and it works fine.
But if I open two folders in a multiroot workspace that each have a default build task configured, then press ctrl+shift+b, then it just shows all the tasks with the option to select one. I would expect it to let me choose between my different default build tasks.
And another problem that I only see on my debug workspace on Mac -

I start watch task A through the menu from cmd+shift+b
I start watch task B also through cmd+shift+b
A notification appears saying "The task B [...] is already active and in background mode". But it isn't active, only task A is active.

Also, I should be able to configure multiple build tasks, but when I run "Configure default build task", it shows a message that there is already a task marked as the default build task.

Investigating for September the problem

A notification appears saying "The task B [...] is already active and in background mode". But it isn't active, only task A is active.

To mark a task as a default build task for the workspace would require task support for workspaces which (to be in line with Debug) we decided not to do for now. You can't define launch configs nor tasks for a workspace. Only for folders.
@egamma we might need to rethink that decision.

What I'd like to have is a build task per folder, then when I press cmd+shift+b, I see all the available build tasks. My workspace has 3 folders and each needs to be built.

@roblourens that works but currently you can't start them all at once. You need to start them one by one. What I clould add is an "All" entry with starts them all. I was thinking about an "All" entry for terminate and relaunch as well.

Fixed the initial reported problem minus the Confogure default Build Task improvement mentioned. Tracked in https://github.com/Microsoft/vscode/issues/35312


Cannot create a portable library targeting previous versions

Build: 26114.2
This is with the desktop workload installed.

This issue was moved to dotnet/roslyn#16607


Demo - DDIC seachhelp generator

Hi,
I created a new demo class "Z2UI5_CL_APP_DEMO_102" for the DDIC searchhelp generator.
I added the two static method GENERATE_DDIC_SHLP and SELECT_DDIC_SHLP to the class in order to run it standalone.
Thanks,
Axel

thank you a lot for the example! i will take a look to it.


Events captured on macOS/iOS missing default UE context values.

During the plugin initialization on Apple platforms Sentry initialization failed message pops up in logs which means that sentry-cocoa initialization wasn't completed by the time USentrySubsystem tried to set some default context values, tags, etc.
Apparently, native SDK initialization is performed asynchronously so some mechanism allowing to get a valid init result is required in order to guarantee proper UE plugin initialization.

This should help: https://github.com/getsentry/sentry-cocoa/pull/3376

This doesn't seem to be an issue after switching to UEngineSubsystem.


Adding twitter link in footer section

Is your feature request related to a problem? Please describe.
I observed that there is no twitter link in footer section
Describe the solution you'd like
I'll add twitter link in footer section
Additional context
I am SSOC'24 Contributor.
Please assign me to this task.

I can add Twitter link can you please assign me this issue under SSOC'24


One point oh

When do we hit it? We should plan what it will take.

The first tag 0.1.0 has been made


Be able to recognize time zones with fractional hours

Coverage remained the same at 94.433% when pulling 5988e988f5882300e55316abadce0a443474e458 on usrenmae:master into 756bc687df560959aa5c0282a11ed496acbc046b on gweis:master.

thanks


close method doesn't work

Hi,
I was exercising easywsclient using the test code and one of my use cases requires a short-lived websocket where the client will issue a close.  When I tested it, I noticed that my server never received the close message prior to the client exiting.  In looking at the source, it appears that the buffer with the close message is constructed but never sent.  I am not a C++ dev, but it looked to me like I needed to add the send, etc. (int ret = ::send(sockfd, (char*)&txbuf[0], txbuf.size(), 0);)  I did so in my test program and it worked - the server now sees the close message come through.  Could you please incorporate this/a fix in for this issue when you have a chance as I am not sure that my fix implementation is complete?
Regards,
Chris Whelan
void close() {
if(readyState == CLOSING || readyState == CLOSED) { return; }
readyState = CLOSING;
uint8_t closeFrame[6] = {0x88, 0x80, 0x00, 0x00, 0x00, 0x00}; // last 4 bytes are a masking key
std::vector<uint8_t> header(closeFrame, closeFrame+6);
txbuf.insert(txbuf.end(), header.begin(), header.end());
int ret = ::send(sockfd, (char*)&txbuf[0], txbuf.size(), 0);
if (ret <= 0) {
closesocket(sockfd);
readyState = CLOSED;
fputs(ret < 0 ? "Connection error!\n" : "Connection closed!\n", stderr);
}
}

This appears to have been a problem with a sleep call in my test program.  My updated code works fine on both AIX on Linux

Hi Chris, thank you for taking the time for this issue (and also to diagnose/close). If you notice anything else, you are very invited to make a report again. Thanks!

No problem David!  Will do.
Regards,
Chris

On Apr 17, 2016, at 5:56 PM, David Baird<EMAIL_ADDRESS>wrote:
Hi Chris, thank you for taking the time for this issue (and also to diagnose/close). If you notice anything else, you are very invited to make a report again. Thanks!
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub


Support manual transaction and make sure only one transaction active

upps.. sorry it seems duplicate with #102

@swznd: wouldn't it be a better idea to use booleans instead of 0 and 1 to check if a transaction is active?

hmm, if using booleans, when we call commit in another function/module it will disturb the previous transaction, because it just check true or false, without knowing how much transaction has been started.
For example, we meet the condition like this:
\QB::transaction(function($qb) {
    $insertId = $qb->table('table')->insert(['column' => 'data']);

    $anotherId = anotherFunction($insertId);

    \QB::table('table_3')->where('another_id', $anotherId)->update(['column' => 'data']);
});

function anotherFunction($id)
{
    try {
        \QB::beginTransaction();
        $insertId = \QB::table('table_another')->insert(['table_id' => $id, 'column' => 'data']);
        \QB::commit();
    } catch (\Exception $e) {
        \QB::rollback();
        throw $e;
    }
}

when commit / rollback in anotherFunction executed it will disturb the transaction outside, so the next query will not run inside transaction as expected.
for reference:
https://github.com/laravel/framework/issues/1686
https://github.com/laravel/framework/issues/1823

I had similar problem.
Its my solution below (pseudocode):
function firstQuery(){
   $qb->table('table_1')->insert($data1).....
}
function secondQuery(){
   $qb->table('table_2')->update($data2).....
}
function thirdQuery(){
   $qb->table('table_3')->delete().....
}

Main class/function:
$inTransaction = $qb->pdo()->inTransaction();
try {
    if (!$inTransaction) {
        $qb->pdo()->beginTransaction();
    }
    firstQuery();
    secondQuery();
    thirdQuery();
    if (!$inTransaction) {
        $qb->pdo()->commit();
    }
} catch (Exception $e) {
    if (!$inTransaction) {
        $qb->pdo()->rollBack();
    }
}

Of course for all sub-function/methods and main function/method instance of $qb have to by the same - only one connection to DB. In each method we should check if we are in transaction, then we can stack and nest methods/functions.


Getting rid of all_opensource_files

This ticket is for tracking the progress of removing the all_opensource_files bazel target.
See the discussion in #15368.

[x] Turn check_futures_test into a sanity check #15671
[x] De-Bazel check_load_py_test sanity check #15677
[x] De-Bazel pip_smoke_test sanity check #15678
[ ] Rewrite //tensorflow/contrib/makefile:build_all_linux #16491

Either we get rid of the dependency or we remove the boilerplate with bazelbuild/bazel/issues/4577.

FYI we have removed all our dependencies on all_opensource_files.

@yifeif Great, could you take a look at #16491 then?

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @Androbin: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Build a predictive model which can predict the size of cloth for a customer

*What is the awesome project you're thinking of?
The most important motivation behind this was to get the size predicted to avoid any problems while buying clothes.
Deployment with flask. (web app)
Additional context
Dataset

Hello @Montekkundan, I would really like to work on this project. Would you kindly assign me this project?

@muhammadanas0716 will this count in Hacktoberfest??

Nope.

If you still want to contribute, I'll be ever so greatefulSent from my phone-------- Original message --------From: Arghya Banerjee @.>Date: Tue, Sep 27, 2022, 6:28 PMTo: muhammadanas0716/Machine-Learning-Projects-101 @.>Cc: Muhammad Anas @.>, Mention @.>Subject: Re: [muhammadanas0716/Machine-Learning-Projects-101] Build a predictive model which can predict the size of cloth for a customer (Issue #17)
Nope.
Ok no probs :)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

If you still want to contribute, I'll be ever so greateful

Is this project still going on, like can I contribute to it ?
I am new to open source but have knowledge of Machine learning deep learning


WebSocket "Connect to" nodes display "disconnected" while being connected

Current Behavior
I created two websocket nodes with the type "Connect to". One websocket in and one websocket out. They are both connected to the same WebSocket server that use the new Subprotocol feature (since release 2.2.0).
They connect successfully as I can both send and receive messages. However both nodes are marked as disconnected with a red mark. Restarting the flow shows them as connected very briefly.
Expected Behavior
Connected websocket nodes should have a connected status instead of a disconnected status.
Steps To Reproduce

Create a websocket in/out node to a websocket server and deploy.

Example flow
No response
Environment

Node-RED version: 2.2.2
Node.js version: 14.18.2
npm version: 6.14.15
Platform/OS: Docker/Linux
Browser: Safari and Edge

Hi, thank you for raising an issue.
Please provide a demonstration flow so we can replicate your setup.
```
paste exported flow between 3 back ticks
```

I uploaded the flow in a gist
This is how it looks: you can click on inject and you will start to receive messages. Notice also that the status node sends a connected event.

Interesting, can you restart the flow to see what happens ?
I ran the container using containerd in Kubernetes in a very different environment and I had the same behaviour. I could try to reproduce the issue outside Docker if you don't see the bug when you restart the flow.

Interesting, can you restart the flow to see what happens ?

In what way?  Restart node-red?  Press the inject again?  Use the deploy button menu "restart flows"?

The deploy button menu "restart flows", sometimes the first deploy works but the following deploys are inconsistents.

@fungiboletus
I have managed to re-create this and it happens fairly regularly (enough to debug it).
I have added a 2nd output so I can monitor the internal readyState of the socket (temporary for debugging - this will NOT be in the update).  As you can see, the WS nodes status is jumping quickly past connected to disconnected BUT the 2nd output (which is basically a 200ms interval monitoring socket.readyState) shows the internal state of the socket.  also, nodte how the output of the status says "connected"

As you can see, this issue is *mostly a visual problem. (some internal state is modified by the old sockets disconnect event)
I can see why this happens ...

restarting flows re-creates the node
the newly created websocket is opened - causing the CONNECTED status
then the previous websocket reports it has closed  - causing the DISCONNECTED status

but I am so far struggling with the best approach.
@knolleary @dceejay
Is there any reason we cannot move to ws v8 (currently NR is on V7.5.6, latest ws is V8.5.0
Of course I would expect this update  is targeted to the next NR major release.
Please let me know your thoughts, either way? ASAP would be good so I can either begin work on the upgrade to ws v8 or work on some kind of workaround for delayed close events.

would be a sensible v3 thought.
Re delayed close... should it/can it just not call done() until closed ?

Re delayed close... should it/can it just not call done() until closed ?

I've looked at the we.close method - there is no callback in ws v7 on the close function.
Of course we could somehow hook the closed event to call done but I have already seen the ws node has internal state to prevent the opened, closed connecting events triggering more than once - I suspect we would end up, at some point with the node-red editor "waiting on the ws close event until the internal node-red close timeout" (long deploys)

callback ...... and in v8 ?
or could go for the good old just add a 25mS timeout and call done :-)

@dceejay so, after some trials, I see the ws close event is taking between 280 ~ 490ms (for this particular ws connection) to complete the close handshake.
Would it be acceptable to monitor the readyState using a 10ms interval & calling done() once  readyState == closed OR  a MAX of 1000ms (whichever comes first)?
e.g...
        node.on("close", function(done) {
            node.closing = true;
            node.server.close();
            //wait 100*10 (1000ms max) for ws to close
            const closeMonitorInterval = 10;
            let closeMonitorCount = 100;
            let si = setInterval(() => {
                if(node.server.readyState === ws.CLOSED || closeMonitorCount <= 0) {
                    clearInterval(si);
                    return done();
                }
                closeMonitorCount--;
            }, closeMonitorInterval);
        }

Looks very sensible to me. Maybe 20ms. No need to rush. But hey.


Is there a way to completely disable Pushy for a device by calling a single method?

Hi, I'm looking for a way to disable Pushy and Push notifications programmatically through the App.
For Android, I found:
Pushy.toggleNotifications(Boolean enabled, Context ctx);
Is there something like this for iOS? I found:
Pushy.toggleAPNs( _value: Bool)
but it seems to not work. I have tried calling toggleAPNs with both True and False value, but I receive the same results.
Tested on 1.0.38 Pushy version, iOS 16

Hi @zvonedev,
Thanks for your patience. This has now been resolved in https://github.com/pushy/pushy-sdk-ios/commit/045c7b62e5d93ac6658d211bfe934e788c448199 with the addition of pushy.unregister().
Invoking this new method will unregister the device from remote notifications. Calling pushy.register() on this device in the future will restore the device token which was previously assigned to the device, and make it possible to receive notifications again.

Import version 1.0.39 of the Pushy iOS SDK by updating the import statement in your project's Podfile within the target declaration:

pod 'Pushy', '1.0.39'

Save the file and run the following command to install the SDK:

pod install

Call pushy.unregister() from your app code to unregister the device. Please make sure not to call pushy.register() until you're ready for the device to receive notifications again:

// Unregister from push notifications
pushy.unregister()

Please let us know if you face any issues.


Default value for 'name' variable prevents route creation

The default value for the variable 'name' is an empty string. If a name is not provided to the module, the route that is attempted to be created is named "-nat-zone", which is an invalid name for a route.
This issue surfaced by the addition of a '-' prefix to the route name in commit here.
Either the name variable needs to be a required variable or the generated name with an empty name needs to be a valid name for a route.

Fixed with #23


Fix Interface not updating when switching class

Also when you add after switching class, it adds all previous Persons from the unupdated interface.
Most likely to do with the selectedClassAddressBook

fixed


AI not accounting for heat cost when rollup occurs

From Harkonnen in closed-3437 on RT discord:
It looks like that evalution identical weapons as one has a nasty side effect - if firing all of them causes overheat CG does not evalute the possibility of firing obly some of them

clever_girl.log.2.zip
clever_girl.log.1.zip


Ubuntu - Cannot find optical drive

I have 1 blu-ray drive located at /dev/sr0
 *-cdrom
       description: DVD writer
       product: BD-ROM  BDC-207D
       vendor: PIONEER
       physical id: 0.0.0
       bus info: scsi@7:0.0.0
       logical name: /dev/cdrom
       logical name: /dev/cdrw
       logical name: /dev/dvd
       logical name: /dev/dvdrw
       logical name: /dev/sr0
       version: 1.00
       capabilities: removable audio cd-r cd-rw dvd dvd-r
       configuration: ansiversion=5 status=ready
     *-medium
          physical id: 0
          logical name: /dev/cdrom

Starting the container and navigating to the web interface shows that no optical disk drive found.
Ran sudo docker logs makmkv
app_ccextractor = "/usr/bin/ccextractor"
Checking for new beta key...
Registration key already up-to-date.
[cont-init.d] 90-makemkv.sh: exited 0.
[cont-init.d] 95-check-optical-drive.sh: executing...
[cont-init.d] 95-check-optical-drive.sh: looking for usable optical drives...
[cont-init.d] 95-check-optical-drive.sh: found optical drive [/dev/sr0, /dev/sg0], but it is not usable because:
[cont-init.d] 95-check-optical-drive.sh:   --> the host device /dev/sg0 is not exposed to the container.
[cont-init.d] 95-check-optical-drive.sh: no usable optical drive found.
[cont-init.d] 95-check-optical-drive.sh: exited 0.
[cont-init.d] 96-install-autodiscripper-srv.sh: executing...
[cont-init.d] 96-install-autodiscripper-srv.sh: no usable optical drive available.
[cont-init.d] 96-install-autodiscripper-srv.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] starting s6-fdholderd...

Also ran sudo docker exec -ti makemkv /opt/makemkv/bin/makemkvcon -r --cache=1 info disc:99991
MSG:1005,0,1,"MakeMKV v1.15.1 linux(x64-release) started","%1 started","MakeMKV v1.15.1 linux(x64-release)"
MSG:5074,0,0,"Automatic checking for updates is enabled, you may disable it in preferences if you don't want MakeMKV to contact web server.","Automatic checking for updates is enabled, you may disable it in preferences if you don't want MakeMKV to contact web server."
MSG:3338,0,2,"Downloading latest SDF to /root/.MakeMKV ...","Downloading latest %1 to %2 ...","SDF","/root/.MakeMKV"
MSG:5042,0,0,"The program can't find any usable optical drives.","The program can't find any usable optical drives."
DRV:0,256,999,0,"","",""
DRV:1,256,999,0,"","",""
DRV:2,256,999,0,"","",""
DRV:3,256,999,0,"","",""
DRV:4,256,999,0,"","",""
DRV:5,256,999,0,"","",""
DRV:6,256,999,0,"","",""
DRV:7,256,999,0,"","",""
DRV:8,256,999,0,"","",""
DRV:9,256,999,0,"","",""
DRV:10,256,999,0,"","",""
DRV:11,256,999,0,"","",""
DRV:12,256,999,0,"","",""
DRV:13,256,999,0,"","",""
DRV:14,256,999,0,"","",""
DRV:15,256,999,0,"","",""
MSG:5010,0,0,"Failed to open disc","Failed to open disc"
TCOUNT:0

I'm at a loss of what else I'm missing

Nm I found it. The dive was hiding in another (castle) position.
ls -la dev/sg*
crw-rw----+ 1 root cdrom 21, 0 Jun 26 22:29 /dev/sg0
crw-rw----  1 root disk  21, 1 Jun 26 22:29 /dev/sg1
crw-rw----  1 root disk  21, 2 Jun 26 22:29 /dev/sg2
crw-rw----  1 root disk  21, 3 Jun 26 22:29 /dev/sg3

Changed docker run command to --device /dev/sg0 and it's found now.


tests: bench: chmod after reflink on macos

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (5e37397) 64.51% compared to head (ebfd4a0) 63.40%.

Files
Patch %
Lines

tests/benchmarks/test_fs.py
62.50%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #236      +/-   ##
==========================================
- Coverage   64.51%   63.40%   -1.12%
==========================================
  Files          27       27
  Lines        2046     2052       +6
  Branches      324      325       +1
==========================================
- Hits         1320     1301      -19
- Misses        666      692      +26
+ Partials       60       59       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

In #237, I have _reflink that does not do any chmod, and it's usually faster than pyreflink.


Python error when --sim is used

Describe the bug
Error :
2022-03-10 23:45:00 | JASMYUSDT | 15m | Close: 0.0169 | BUY
Resistance level of 0.0172 formed at 2022-03-10 21:15:00
Fibonacci Retracement Levels:{'ratio0_286': 0.01, 'ratio0': 0.01}
TypeError("cannot concatenate object of type '<class 'dict'>'; only Series and DataFrame objs are valid")
when i run simulation
To Reproduce
Steps to reproduce the behavior:
python3 pycryptobot.py --market JASMYUSDT --live 0 --sim fast
Expected behavior
Result of the simulation
Desktop (please complete the following information):

OS: Ubuntu 20.04.4 LTS
pycryptobot beta 6.2.0
python 3.9.5
pip 22.0.4

Hello. I have the same problem. I fixed it by updating the matplotlib libraries

Thank you t00be, it work perfectly with matplotlib 5.3.1 !
Need to update requirement.txt please :-)

With matplotlib 5.3.1 :
python3 pycryptobot.py --market HOTUSDT --sim fast
2022-04-04 09:23:38 INFO     2022-03-24 15:15:00 | HOTUSDT | 15m | 0.004786 | BUY
2022-04-04 09:23:38 INFO      Resistance level of 0.004823 formed at 2022-03-24 10:45:00
2022-04-04 09:23:38 INFO      Fibonacci Retracement Levels:{'ratio0_286': 0.0, 'ratio0': 0.0}
2022-04-04 09:24:08 CRITICAL Restarting application after exception: TypeError("cannot concatenate object of type '<class 'dict'>'; only Series and DataFrame objs are valid")

I have the same issue, in my case it is happening when using Binance.
I tried updating matplotlib but I already have the latest version.
Error:
2022-04-22 07:45:00 | BTCUSDT | 15m | Close: 40709.59 - Ready Chg: 0.00%/0.0% | ^ EMA12/26: 40650.7580 > 40649.3900 ^ | ^ MACD: 1.3680 > -12.0639 ^ | ^ OBV: 14640.0308 (4.1600%) ^ | ERI: buy | BUY | Last Action: SELL | DF HIGH: 42819.99 | DF LOW: 38795.19 | SWING: 10.37% | CURR Price is -4.93% away from DF HIGH | Range: 2022-04-19 05:00:00 <--> 2022-04-22 07:45:00
2022-04-22 07:45:00 | BTCUSDT | 15m | Close: 40709.59 | BUY
Support level of 39751.0 formed at 2022-04-21 22:30:00
Fibonacci Retracement Levels:{'ratio1': 40290.8, 'ratio0_768': 40877.57}
TypeError("cannot concatenate object of type '<class 'dict'>'; only Series and DataFrame objs are valid",)
OS: Ubuntu 20.04.4 LTS
pycryptobot  6.3.0
python 3.6.9
pip 21.1.2

same issue here:
running:
python3 pycryptobot.py --market BTC-USDT --sim fast-sample --simstartdate 2022-01-01 --simenddate 2022-04-01
leads to:
2022-01-14 20:00:00 | BTCUSDT | 1h | Close: 43100.94 - Ready Chg: 2.42%/0.5% | ^ EMA12/26: 42861.8433 > 42848.7487 ^ | ^ MACD: 13.0945 > -90.0369 ^ | BUY | Last Action: SELL | DF HIGH: 51853.1 | DF LOW: 40679.82 | SWING: 27.47% | CURR Price is -16.88% away from DF HIGH | Range: 2022-01-02 09:00:00 <--> 2022-01-14 20:00:00
2022-01-14 20:00:00 | BTCUSDT | 1h | Close: 43100.94 | BUY
Resistance level of 44500.0 formed at 2022-01-13 14:00:00
Fibonacci Retracement Levels:{'ratio0_768': 42329.91, 'ratio0_618': 43396.78}
TypeError("cannot concatenate object of type '<class 'dict'>'; only Series and DataFrame objs are valid")

A new major version was released this year. Please upgrade to v7.0.34 or later and if you are still having a problem, raise an issue.


hack/gofmt: allow go1.4

Otherwise it is skipped with:
Unknown go version 'go', skipping gofmt.

LGTM

Perhaps futile, but I will wait for one green.

PTAL, fix some of the file that are simplified further with 1.4

Ah, good, thanks. Will wait for green.

Oups, actually scratch that.
We want gofmt to be limited to 1.3 because we don't want to validate 1.4 only syntax that breaks the 1.3 build.
Sorry for the noise closing.


default homepage is wrong

I think the reason is that the same homepage is probably used for both package.json and the github repository. but I think these should be different. The package.json homepage is used by NPM to link back to the project, but when you're already on the project, you don't need that link.

@doowb I think we should just use the org's github url for homepage if it's an org, otherwise the user's


NOTIFICATIONS: the notification must be displayed only if its support-status is = 100

REQUESTS SERVICE: created getSnapshotLastConversation() that makes a query for the conversations with support-status == 100


Add biflatmap

Simple idea to add FlatMap the same way we have Map to connected streams to allow us to perform both filtering and mapping at the same time.

sbt scalafmt is needed

Please squash commits to a single "Implement biFlatMap"


Update Invoke-SqlCmd2.ps1

-LiteralPath is required on Test-Path to allow for files names with square brackets
Example[filename].sql

premature


Fix Reset to default to work with boolean/checkboxes

Add field name to each config value
Check field name and use checked for checkbox and value otherwise

Fix #189

Thanks for the comments. I updated the PR accordingly.

@rvernica Thanks!


Add separate route for author page and code optimisation

Signed-off-by: Pallavi-PH<EMAIL_ADDRESS>In this PR we:

Adding separate route for author page instead of query params
Add blog card to components
Modify image paths in header, footer, sponsors,  avatar, and error page and direct them to point to the image folder at the root of the current sub directories so that we can call them in nested paths without having broken images
Conditionally display breadcrumbs on the author page. We don't want to display breadcrumbs in the case of 404
Conditionally showing cursor pointer for tag button wherever required

URL structure:

Full page:

URL structure:

Full page:

Please check for linting errors,


DP-1.8: Adding test support for Nokia

Following changes are made in this PR -
1. Updating ConfigureQoS for QoS OC configuration for Nokia
2. Adding function SetForwardingGroupWithFabricPriority to configure ForwardingGroup with Fabric-priority in qoscfg.go
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Functional Test Report for #2003 / 1c985bdae1d1f8d002f9dc8692a099179a3f23db
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

DP-1.8: Two strict priority queue traffic test

Cisco 8000E

DP-1.8: Two strict priority queue traffic test

Cisco XRd

DP-1.8: Two strict priority queue traffic test

Juniper cPTX

DP-1.8: Two strict priority queue traffic test

Nokia SR Linux

DP-1.8: Two strict priority queue traffic test

Openconfig Lemming

DP-1.8: Two strict priority queue traffic test

Help


Ref/replace sequencer with replica url

Urgent change for Chronicle Timeout issues
Urgent change to replace our Chronicle RPC URL with a replica endpoint rather than using the sequencer url for all operations. We will now only use the sequencer url for transactions while using the replica in all other cases.

Should we also add that to the constants package, LIT_CHAINS > chronicleTestnet > rpcUrls array?

yes please


Getting latitude and longitude 0.0

If the location is previously on while running the application first time, it's picking the latitude and longitude.Then if I close the app and turn off the location, while opening the app for the second time its navigating to settings to turn on location but after location is on it is unable to pick the latitude and longitude. I am getting 0.0 for both.

Did you follow all steps from the README?
Can you log something in the onPositionChanged callback?
Can you show your call to the SimpleLocation constructor?

my activity code :```
public class MainActivity extends AppCompatActivity {
Button btn_location;
private SimpleLocation location;
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    btn_location = (Button) findViewById(R.id.btn_location);

    // construct a new instance of SimpleLocation
    location = new SimpleLocation(this);
    // if we can't access the location yet
    if (!location.hasLocationEnabled()) {
        // ask the user to enable location access
        SimpleLocation.openSettings(MainActivity.this);
    }

    btn_location.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            if (!location.hasLocationEnabled()) {
                // ask the user to enable location access
                SimpleLocation.openSettings(MainActivity.this);
            }
            final double latitude = location.getLatitude();
            final double longitude = location.getLongitude();

            if (latitude != 0.0 && longitude != 0.0) {

                Log.d("LOCATION", "LATITUDE :" + latitude);
                Log.d("LOCATION", "LONGITUDE :" + longitude);

                Toast.makeText(MainActivity.this, "Latitude is :" + latitude + " longitude is :" + longitude, Toast.LENGTH_SHORT).show();

            }else{
                Log.d("LOCATION", "LATITUDE :" + latitude);
                Log.d("LOCATION", "LONGITUDE :" + longitude);
                location.beginUpdates();
            }
        }
    });
}
@Override
protected void onResume() {
    super.onResume();
    if (!location.hasLocationEnabled()) {
        // ask the user to enable location access
        SimpleLocation.openSettings(this);
    }

    // make the device update its location
    location.beginUpdates();
}
@Override
protected void onPause() {
    // stop location updates (saves battery)
    location.endUpdates();
    super.onPause();
}

}

First, you should be able to safely remove the if (!location.hasLocationEnabled()) branch and its contents from onResume because you have it in onCreate already.
Second, the additional call to location.beginUpdates() in the OnClickListener in onCreate is probably superfluous.
Otherwise, this looks good. But why don’t you implement the onPositionChanged callback as shown in the README and try to log something there. That way, you would see if and when location updates are available.
Last but not least, you should try the extended arguments of the constructor, again, as shown in the README. For example, try replacing
location = new SimpleLocation(this);

with
location = new SimpleLocation(this, true, false, 5 * 1000, true);

and see if that helps. It should attempt to update the location every 5 seconds.


[RFE] make custom auth.json location possible.

What happened?
I am using a custom location for my authentication file, by setting a REGISTRY_AUTH_FILE env variable.
podman login picks this up, but oc-mirror does not. This leads to confusing situations where podman login succeeds, but you then get a authentication when running oc-mirror.
So consider making oc-mirror aware of the REGISTRY_AUTH_FILE variable.

I second this.  The same --authfile parameter used by podman and skopeo would be nice...

I third this. I lost several hours because of this today... This is not very intuitive...

I'm also missing the feature to configure the auth.json for oc-mirror. It would make automation around oc-mirror much more simple and clear. Thanks for considering this feature!


feat(i18n): import locales outside the component

You can import a specific locale from dayjs.
It doesn't build all the locales:

How it works now: https://vue-tailwind-datepicker-git-ontheflylocaleimport-tomoe-gozen.vercel.app/advanced-features.html#localization-i18n

Very keen to see this merged in.  Having unused locales included in the build output has bothered me for a while

Works a treat, thank you @elreco !

@Jarrod-Stone nice!! thank you :)
It doesn't work on codesandbox I don't know why :'(

I will revert this because it doesn't work


Is it possible to remove ?page=2&per-page=10 from URL?

Is it possible to remove the page number and per page option from URL?
As it seems to be conflicting with my ajax (alternatively how should I change my ajax)
$(document).ready(function() {
$('a.btnsolid-sm').click(function() {
    var Id = jQuery(this).attr('id');
    $.ajax({
        type: 'POST',
        data: {
            'modal_id' : Id,
        },
        url : 'http://localhost:8888/blog/viewmodal?id='+Id,
        success: function(response) {
            if(response) {
                $('.modal_blog_wrapper').html(response);
                $('#modal_'+Id).modal('show');
                $(document).on('hidden.bs.modal', modal_id, function (event) {
                    $(this).remove();
                });
            } else {
                alert('Error');
            }
        }
    });

});

});

Just disable EXTENSION_HISTORY

It seems like question is answered. Thanks @usualdesigner.

how to disable EXTENSION_HISTORY ?


As a user, I want to see all the ranked proposal beneficiaries

Currently we have top 11 proposal beneficiaries, but that's not enough, I want to see the list contains all the Beneficiaries, ranked by the estimated fiat value.

Figma link
Preview


Create NCRcombination.c

NCR formula is used to find the number of ways where r objects chosen from n objects and the order is not important.

This formula is used to find the combination


Custom folders for assets (css, js, fonts)

If i move the either content or scripts folder SiteInitialization.cs throws an exception.
Can the client-side static content folders (/content, /fonts & /scripts) be moved to custom locations so that the following project folder structure would be possible?
\Assets
\Assets\Site
\Assets\Site\Css (currently in /content/site/ folder)
\Assets\Site\Fonts (currently in /fonts folder)
\Assets\Site\Js (currently in /scripts/site/ folder)
\Assets\ExternalLibs
\Assets\ExternalLibs\Bootstrap
\Assets\ExternalLibs\JQuery\
\Assets\ExternalLibs\JQueryUi
etc..
The \Assets\ExternalLibs{package}\ are currently mixed across the /content/, /fonts/ & /scripts/ folders.
Perhaps application settings could be used to point to the default location for each:
Site-Content
Site-Fonts
Site-Scripts
External-Packages (where each external package would have its own sub-folder)

This is where ASP.NET MVC and NuGet packages places them by default. I don't see any advantages to moving them.

What about projects that use packet mangers other than NuGet (like NPM, Bower)?

You can configure those paths to install to the default content folders using package.json and/or .bowerconfig.

We are currently not using Bower/NPM, as it means some additional dependency, which i'd like to avoid. When we switch to ASP.NET Core (looks far away), we'll probably make use of Bower.

Just wanted to say that Bower looks to be no longer maintained. If anyone is worrying about the npm dependency tree, npm 3 looks to have introduced de-duping in the dependency graph for all libraries installed and flattens the dependency structure to only 1 level deep in /node_modules. I think if we do move to npm it should force use of npm 3.

Hmm, sad to hear, they are just recently integrated into VS. NuGet works kinda fine for now, though not all packages are available, or up to date.


[proposal] koordinator-descheduler has ability to limit the total number of pod evictions

What is your proposal:
koordinator-descheduler would like to has the ability to control the total number of pods rescheduled in each round so that we can avoid situations where too many pods are evicted due to other anomalies occurring.
koordinaotr-descheduler currently has total limits for per node and each namespace, and now it needs to add a total limit for all pods as well.
Why is this needed:
Is there a suggested solution, if so, please add it:

/assign

@songtao98


Release QBit to public repo

Geoff, please cut a release at your earliest convenience.

+1
I had a dickens of a time getting it all loaded in and built :)

Now it is getting built by travis. :) Thanks to sailor geoff..

It builds now in travis again.
I added the maven stuff to qbit-servlet and qbit-jetty so those get built  and put into public repo.
Let's pull that trigger.

Two releases so far 0.61! :)


How i can change my repository language?

I made a file with this name: .gitattributes.
In this file i have:
$ cat .gitattributes *.rb linguist-language=C#
But is not working.

Do you have a link to your repository or is it a private one?

To change the language i made an .cs file. :))


Update README.md

Fixes #34
Added contributors section in Readme.md

You can also add a bot named @add-contributors which automatically adds the names and details of contributors @Ctoic

Thanks for suggestion @naikmubashir can you add the bot ?

I can only if i have access to your repo @Ctoic

You cannot by forking it ?

I'm saying if I add bot to the forked repo it won't add the bot to this repo.

Okay I will add the bot once I get time , thanks for your suggestion @naikmubashir 👍🏼


Fix disabling client signups / email changing

The config key was being unset when these config options were set to the disabled state. (So if "Allow new signups" was disabled, the configuration wasn't unset internally)
To fix this, I just inverted the logic so the toggle is now to disable signups rather than enabling them.
Now it works correctly:

As a bonus, I think logically speaking it makes more sense for the toggle in the settings to disable these options rather than enabling them, especially since by default these will appear unchecked in the setting.
Closes #1343

I do not know why it is still not working for me. Above, it is said that it has been fixed. When I set "Disable new signups" to on, visitors can still access to registration page.

About FOSSBilling
FOSSBilling version
0.5.5
PHP version
8.0.27
[Apache 2.0]

I do not know why it is still not working for me. Above, it is said that it has been fixed. When I set "Disable new signups" to on, visitors can still access to registration page.

About FOSSBilling
FOSSBilling version
0.5.5
PHP version
8.0.27
[Apache 2.0]

Did you try actually creating an account?
It shows a message when someone attempts to register an account. The page itself is still accessible

Thanks for your quick response. Yes it was a confusing to have the link still active and the registration form is also available. But after entering the user details, there is a warning message that tells registration is not allowed. Ok, but I think that it will be a good idea to not let user waiting until reaching all these steps to tell him that registration is closed. Maybe to deactivate the registration link/form it more practical.


Don't clone org when built-in version will work

ox-hugo lists org 9.0 as a dependency. Emacs 26.1 ships with org 9.1.9. However when I try and use straight to install ox-hugo it attempts to clone org as an external package (which fails because of an SSL certificate problem, but that is irrelevant). If the built-in version of org meets the package requirements, There is no need to clone the external package (and some users, including myself, can't get it.)

You may not realize this, but straight.el completely ignores all version numbering and does not include a dependency resolver. The reasoning for this is that such features would add extraordinary complexity without really improving user experience in all but the most remote of circumstances.
The fact that it always tries to install all packages, rather than relying on the ones shipped with Emacs, is an explicit design choice. The reasoning for this is that users should receive the most recent versions of packages by default, and versions shipped with Emacs are always outdated.
If you want to use the built-in version:
(straight-use-package '(org :type built-in))

Thanks, that worked. I understand your reasoning for not want to add a version resolver.

Great :)
See also #328 for documenting the fact that this feature exists.


Few ideas

Hi. Good job on vue-disqus.
I have 2 suggestions though.
Can you please follow the standard and create /dist folder where you put you ready to use generated files?
vue-disqus/dist/vue-disqus.js
And second, can you include minified version in dist as well?
vue-disqus/dist/vue-disqus.min.js

Sorry for my delay.
I will refactor and create dist files for ES6, AMD and commonJS.

@maxorlovsky
Late, but left.
It is supported for (AMD, UMD, ES6, CJS)
Thanks for idea.


Fix typo in publish-status workflow

Description of changes: Fix typo in publish-status workflow
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #416 (6e9fa1d) into main (f763147) will not change coverage.
The diff coverage is n/a.

: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            @@
##               main     #416   +/-   ##
=========================================
  Coverage     85.71%   85.71%
  Complexity       19       19
=========================================
  Files             3        3
  Lines            49       49
  Branches          5        5
=========================================
  Hits             42       42
  Misses            3        3
  Partials          4        4

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


confusion in implementation of decoded network

Hi,
Thanks for your implementation!
I have some confusion in the implementation of the decoded network.
I knew that each cell has two inputs from prev_prev_cell and prev_cell. But what the case in the decoded network. I mean that: If layer_i has downsampling rate 16, but layer_{i-1} has downsampling rate 4. In this case, prev_prev_c is None and only has prev_c's output as input, right? (from the paper, I saw that prev_prev_c's output should have the same scale as the current layer).
Could you give me some hints? Thanks in advance.

If lay_i havs downsampling 16 then it will only get information from lay_{i-1} with sample 8 and 16 not 4

Thank you for your quick answer!
I have another question about afine of BN in train_search phase and retrain phase. From darts, the learnable parameters of BN are disabled in train_search phase. But in some other paper like proxylessNAS, affine of BN is not set specifically. What effect do you think affine mode of BN might have?
Looking forward to your reply!

In our experiment, we set affine to False in search and True in retrain
False in search is accroding toi darts
True in retrain is accroding to deeplab v3+


The bottom nav is invisible

The bottom bar is hidden through scrollback, so nobody sees it. Discovered the fact only because was looking how to deal with #2850.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11029449-the-bottom-nav-is-invisible?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Can you be more specific? I thought maybe you were referring to the footer, but I can see that just fine:

Yes. I am referring to footer. It is shown only when you scroll down no matter how small the page content is.

Ah, gotcha.

Addressed in #3718.


clears the map on certain prop changes

@btmills
Allows multiple props to be checked for diffs on componentWillRecieveProps to check if the map should be redrawn.
I swear this is the final PR! sorry for opening / closing the others...

Good idea, thanks!


Administration Panel

The admin panel should be simple and easy to navigate
Probably just use SBAdmin (https://startbootstrap.com/themes/sb-admin-2/)
HomePage

[x] Widgets
[x] Total Games
[x] Total Users
[x] Votes In
[x] Votes Out
[x] Graph of games added over time (30 days)
[x] Any games needing a review

Categories

[x] Create Categories
[x] Update Categories
[x] Delete Categories
[x] Read Categories

Manage games

[x] Edit Games
[x] Easily Approve Games
[x] View game info
[x] Votes in/out
[ ] Screenshots

Manage users

[x] View User Profile
[x] Edit User
[x] Create User
[ ] View users and games associatedwith them
[ ] Ban user, consequently also bans all games
[x] Promote to administrator
[x] Userlevels should be simple, either admin or not

Same as #2 regarding the screenshots and banner for listings


Cargo book occasionally resets theme when using Pale Moon browser

Problem
When reading the Cargo book, bundled with the installation and accessible by calling rustup docs --cargo, navigating between partitions via ToC occasionally resets the theme to the default light one.
Steps

Install Pale Moon browser.
Download rustup.
Install the default toolchain (maybe it can be skipped).
Use rustup docs --cargo to open the book in the browser.
Navigate to 1.1.
Switch the theme to "Rust".
Navigate to other partition, for example, to 4.2.7. — the theme resets to the default one.
Navigate back to the original partition — the theme is back to "Rust".

Possible Solution(s)
As a workaround, the other browser can be used, I cannot reproduce the issue using Microsoft Edge.
Idk, but, based on the behavior, it looks like the theme is somehow attached to the partition.
Notes
The bundled Unstable book, Edition guide, Rustc book, and Embedded book exhibit the same behavior.
The bundled Rust book,  Rustonomicon, Rustdoc book, and Rust by Example do not suffer from this issue.
Version
cargo 1.58.0 (f01b232bc 2022-01-19)
release: 1.58.0
commit-hash: f01b232bc7f4d94f0c4603930a5a96277715eb8c
commit-date: 2022-01-19
host: x86_64-pc-windows-msvc
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.80.0-DEV (sys:0.4.51+curl-7.80.0 vendored ssl:Schannel)
os: Windows 10.0.22000 (Windows 10 Pro) [64-bit]

Pale Moon Version: 29.4.4

I feel like this issue should go to rust-lang/mdBook. Apart from that, if my memory still fresh, Pale Moon is (was?) a Firefox fork. What does Firefox behave on your machine?
For the code itself, the theme in mdbook is a simple item stored in localStorage. Each time when the page is loaded, a script tries to get the item mdbook-theme from localStorage and set the style accordingly.
Refs:

https://github.com/rust-lang/mdBook/blob/601ebc5499ac87/src/theme/book.js#L309-L363
https://github.com/rust-lang/mdBook/blob/a72d6002b78388/src/theme/index.hbs#L65-L78

@weihanglo, I decided to open the issue here because the behavior differs between the offline books on Pale Moon.
I installed Firefox and it seems to behave in the third way: it keeps the theme in all of the online books (like all three browsers I tried), but it resets the theme in all of the offline books.
So, to sum up:
Edge works both online and offline.
Pale Moon works online, and works offline for some of the books included with the Rust installation.
Firefox works online and does not work with offline books.
I suppose, I'd open an issue at mdBook, but keep this issue opened for now.

I found out, thanks to @weihanglo, that the issue seems to be rooted in the fact that local storage access for file:/// is implementation defined and covered by non normative section of the spec.


validation puzzle

Reviewing the problem from a higher level, if a candidate is not familiar with Node, using npm to find and install al library might be a bit too much. Also, the expectation to read documentation for probably a completely unfamiliar library and implement against it is probably also pretty tough.

Good start. Will propose some changes in a separate PR.


added imap move

Hi, I added the possibility to move emails into a imap subfolder, since I needed it. I hope you find it usefull too.
greetings
Thorsten

Cool, Thorsten! This'll need test coverage to demonstrate its behavior.

I added the test for my changes, please have a look...


asus g513

can i use it?

Hey @izachuynh in the Asus Rog Strix G513QM it worked for me! Thx @wroberts for this amazing work hehe I was getting crazy trying to patch Linux kernel for installing the OpenRGB.


Gulp --production command problem with jquery library

I am using jquery library and in my gulpfile.js and scripts function jquery added for minifing js sources.
after run gulp --production the jquery library exploded and not worked.
plead help us and any guidlines make be usefull.
Tnx

This is not a support forum. Please direct support questions to StackOverflow.


Metadata information on Share page

It might be nice to have a way to view some of the metadata for a brew from the Share page.   For example, the authors, created date, last updated date, and tags.
A "i" inside a circle icon (the same as the icon used for the metadata editor) in the Brew Title section of the navbar could either pop up a modal on hover or click to display this information.  Further, the author names could be links to the authors user page.
This would basically be the first step toward a more "browsable" experience without really having to add a "home" or "search" page.

Some of this is available from the Source </> menu, where the metadata is in a codefence.

Not all metadata though, and for some brews not even at all.

This is not ideal as UX, of course, and I second the motion for a UI widget to reveal this.

A separate but relevant question though: we have up to now not disclosed the author/owner of a brew from any share page, not even hidden in the source. If all you have is the /share/mumble url, then you can't find out.
Maybe a quick question to the sub-reddit to poll the community as to whether they would be upset if their username got revealed on their shared brews? I don't think there's a problem, some might well welcome it, but it's an assumption that might have some negatives. Best to check first.

Going through old issues and found that this is a duplicate of #1679 .  Closing this in favor of the original, but will pull @ericscheid's comment (particularly about disclosing authors) to that one.


Fully qualified function names in @type! guards

When defining a guarded type with @type!, if the type is used in an @spec! in other module, the functions used in the guard need to be fully qualified. This generates a compilation error:
defmodule A do
  use TypeCheck

  @type! f :: integer() when is_f(f)

  def is_f(_f), do: true
end

defmodule B do
  use TypeCheck

  @spec! f() :: A.f()
  def f(), do: 1
end

== Compilation error in file lib/examples.ex ==
** (CompileError) lib/type_check/spec.ex:4: undefined function is_f/1 (expected B to define such a function or for it to be imported, but none are available)

But this doesn't:
defmodule A do
  use TypeCheck

  @type! f :: integer() when A.is_f(f)

  def is_f(_f), do: true
end

defmodule B do
  use TypeCheck

  @spec! f() :: A.f()
  def f(), do: 1
end

If the @spec! that uses the type is defined in the same module that the guarded @type! this error doesn't appear.

The fix is part of the now-published release v0.12.4.


Install TagBot as a GitHub Action

You're receiving this pull request because the now-deprecated Julia TagBot GitHub App is installed for this repository. This pull request installs TagBot as a GitHub Action.
If this PR does not belong in this repository for any reason, feel free to close it.
See this Discourse thread for more details and to ask questions.
Please do not comment on this pull request, because it is unlikely that I (@christopher-dG) will see it.

Codecov Report

Merging #108 into master will decrease coverage by 2.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   93.88%   91.61%   -2.28%
==========================================
  Files           9        9
  Lines         687      763      +76
==========================================
+ Hits          645      699      +54
- Misses         42       64      +22

Impacted Files
Coverage Δ

src/euler_types.jl
91.59% <0%> (-5.18%)
:arrow_down:

src/angleaxis_types.jl
92.77% <0%> (-1.06%)
:arrow_down:

src/quaternion_types.jl
97.43% <0%> (-0.81%)
:arrow_down:

src/derivatives.jl
100% <0%> (ø)
:arrow_up:

src/util.jl
100% <0%> (ø)
:arrow_up:

src/core_types.jl
86.79% <0%> (+1.49%)
:arrow_up:

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 031a678...14080cf. Read the comment docs.


Parallax Style Fixes

I noticed something similar to this.
https://www.concrete5.org/developers/bugs/5-7-5/parallax-style-doesnt-fill-container./
The parallax style resizer method doesn't always fill the full DIV on large monitors. But sometimes it does. If you make something with a parallax background in 5.7.5

So I'm not able to replicate this using google chrome's device viewer, can someone provide an example that breaks at a specific resolution?


Merge Master and AsyncUDP in one branch

Merge Master and AsyncUDP in one branch
In the esp-knx-ip.h file you can select which UDP library to use:

WIFIUDP (for esp8266 lib v2.3.0 and up)
AsyncUDP (for esp8266 lib v2.4.1 and up)

Hi Nico,
How are you?
In this PR, I propose to merge both branches and add a new key in the esp-knx-ip.h file to select which Multicast Library to use, due to the problems on the esp8266 board libraries.
This is the version of esp-knx-ip library that is being used in Sonoff-Tasmota for KNX communication.
Hope you consider it.
Thanks!

No longer needed.


Broken notification when used with multiple reverse proxy

In case when site is located behind more then one reverse proxy validation of X-Forwarded-For for exact match of one of ip is not sufficiant, there should be ability to use more then one ip in X-Forwarded-For or just validate x-real-ip

All the extra attributes like x-real-id, x-forwarded-for can be tricked easily.
The option to validate x-forwarder-for should be used only for testing purposes. On production, you should configure your proxy to pass the original visitor address like described here https://support.cloudflare.com/hc/en-us/articles/200170786 and disable forwarded ip validation in this library.


[#145907723] Updated legacy SDK

Please review, but do not merge.

This depends on https://github.com/leanix/leanix-eam/pull/591


Update mpv.json

Update urls for mpv-0.34.0.
Closes #7270

The URL is unreachable. Probably due to some problem in SourceForge's CDN. Will check later.
wget.exe https://downloads.sourceforge.net/project/mpv-player-windows/stable/mpv-0.34.0-x86_64.7z
--2021-11-09 15:59:10--  https://downloads.sourceforge.net/project/mpv-player-windows/stable/mpv-0.34.0-x86_64.7z
Resolving downloads.sourceforge.net (downloads.sourceforge.net)... <IP_ADDRESS>
Connecting to downloads.sourceforge.net (downloads.sourceforge.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2021-11-09 15:59:11 ERROR 404: Not Found.


README.md is innacurate

The getPath() command now supports randomized or chosen paths. Both of these should be mentioned in the README.
There are now about 7000 characters in the database; the stats section should be updated accordingly.
Also, a better description of the path-finding algorithm should be given under "How It Works".

Fixed as of commit d810d2f.


Drop --rpc-url and --network-passphrase.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

@leighmcculloch only soroban e2e tests are failing. I believe this requires changing https://github.com/stellar/system-test, right? What's the process? First I update stellar/system-test then merge this, or is it the other way around?

@leighmcculloch only soroban e2e tests are failing. I believe this requires changing stellar/system-test, right? What's the process? First I update stellar/system-test then merge this, or is it the other way around?

We don't have a process. We haven't broken it much. 🫢


⚠ Bofrost.be has degraded performance

In 1ea3ced, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2975 ms

Resolved: Bofrost.be performance has improved in d74b453.


Remove code duplication normalizing timestamp in a dedicated method

@aricooperdavis following our talking I made an attempt to make the code clearer when normalizing.
Moreover I moved the code in a separate method so there's no more code duplication.

Nice improvement, thank you!

I tested the script with Honor Band 4 file and was almost are ok. Only the timestamp '.000Z' string add that i need adjust to 'Z'.
Great work.
Best regards.

Thanks @ronison - could you just confirm that the Honor Band 4 requires line 456 to read :
line[0] = dt.utcfromtimestamp(line[0]).isoformat()+'.Z' #time
rather than
line[0] = dt.utcfromtimestamp(line[0]).isoformat()+'.000Z' #time?
If so I will implement a command line argument/flag for the Honor Band 4. Also, if you could send over an example HiTrack file for me to have a look at that would be excellent!
Cheers

Hi @aricooperdavis,  In fact you need to change lines 347 and 456 as follows:
347: stats['start_time'] = dt.utcfromtimestamp(stats['start_time']).isoformat()+'Z'
...
456: line[0] = dt.utcfromtimestamp(line[0]).isoformat()+'Z' #time

I think this change would work for all devices and you would not need to add a parameter to it.
HiTrack.txt

This file was generated with the exercise started directly from the band, without the application. The first line was a problem.
HiTrack_1557178410000155718218700030001.txt


Add check for screen and nssm presence on the system

There should be check if screen (Linux, Mac) or nssm is present (Windows).

Addressing screen check in https://github.com/piotr-iohk/cardano-up/pull/5. nssm check will not be done for now.


[TASK] Mandatory def input types

Goal
Constrain the type system such that all defs have a known input type.
Benefits
More robust type inference is the major goal here. By knowing the set of concrete types that a def is expecting, only that set of types has to be tested.
Concerns
Agnostic defs
Many defs are input agnostic. This will still be a requirement.
The proposed solution is to force the input to be Nil. Unfortunately, this will break how the current type inference flows types.
The convention will break certain patterns, for example:
# This would previously work, and list out along folder `foo-bar`.
\ 'foo' | ls { + '-bar' }

# Now the concatenation would have to occur sooner, be bound to a variable, and then passed as an argument.
\ 'foo' | + '-bar' | let $x | ls $x

Since this pattern becomes unwieldy with the change, it might be worthwhile adding a definition which can handle this:
\ 'foo' | + '-bar' | zog ls $z
                     ^^^ --- ^ a def which stores input in $z and calls `ls` with Nil input

Another approach would be to consider defs with Nil input types to be agnostic. It is rare to actually switch input to Nil to call a def, so rather than having a distinction between Nil and anything, combine the two and add a special variable (or literal?) which can be populated with the output of the previous block.
This approach may grate with the goal of reducing type inference complexity. The issue is the coercion to any output type, it makes the type inference unconstrained.
def foo Nil (len:Num) { \ $len | + 2 }

ls | foo { \#i | len }

Maybe a combination?
To keep the input type as a constraint, a defining characteristic of the call needs to be present.
The input def (\) would be a good candidate.
Parsing would need to be altered to such that trailing characters are taken as a def identifier.
def foo Nil (len:Num) { \ $len | + 2 }
                         ^ a space is now necessary

ls | \foo { \ $prev | len }
     ^^^^     ^^^^ store output of previous block in `$prev`
     `foo` is known to be called with `Nil`.

# Another way to write this
ls | len | \foo $prev

The let def
The let def is special, since it is required to be general.

[ ] #109
[ ] #110
[ ] #111
[ ] #112

A show stopping concern is how general definitions will be constrained by this change.
For example, the min definition is intended to be generalised in #83, which would be something like: def min (b) { {> $b} $b #i }.
Introducing an input requirement would place a burden on introducing generics to handle this polymorphism (eg def min 'a:'a (b:'a) { {> $b} $b #i }).
This detracts from the goal, reducing the set of types to infer.
Instead, the solution should combine some ideas initially proposed, and undertake a rework of the inferencer. The \ overload is potentially useful for arithmetic operators, and better set management could be employed on the type graph.


bug: fix sorting of release versions

Changes Proposed

Sort release version in Install modal by semver

Check List

[x] The title of my pull request is a short description of the changes
[x] This PR relates to some issue: #326
[ ] I have documented the changes made (if applicable)
[x] I have covered the changes with unit tests

@undera

Thanks!
We need to make sure that this change will remain in React version

i will migrate it to that branch as well once the code reaches that stage


Introduce useDefaultValueIfUndefined optional feature to ease passing arguments through

We are wrapping components provided by Ember Bootstrap in custom addons. So we have many code like this:
// addon/components/my-form.hbs

<BsForm
  @formLayout={{@formLayout}}
  @model={{@model}}
  @onSubmit={{@onSubmit}}
>
  {{yield form}}
</BsForm>

We noticed that the current implementation of defaultValue decorator does not use the defaults if the argument is passed but being undefined. So we would need to redeclare all default value:
// addon/components/my-form.hbs

<BsForm
  @formLayout={{if @formLayout @formLayout "vertical"}}
  @model={{if @model @model (hash)}}
  @onSubmit={{if @onSubmit @onSubmit this.noop}}
>
  {{yield form}}
</BsForm>

Not great solution. So we thought we might want to change the defaultValue decorator.
I also noticed that arg decorator used for the components, which have been refactored to @glimmer/component seem to fall back to default value if undefined is passed: https://github.com/kaliber5/ember-bootstrap/blob/c740df9f818a8081e523e265a8e438418b78648c/addon/utils/decorators/arg.js#L5-L8 So I think we can consider the current implementation of defaultValue to be a bug.
I wasn't sure how to test this as it affects so many places. Just added one test as a showcase.
Disclaimer: Decorators are still scary to me. I'm definitely not an expert on that topic. Came up with that fix by try and error rather than by understanding what I'm doing. Please let me know if it's totally wrong.

This does not resolve the issue for onBefore, onSubmit and onInvalid as they do not use the defaultValue decorator. Need to address them differently.

So we would need to redeclare all default value:

We needed to do similar things, so yes, I see you!

So I think we can consider the current implementation of defaultValue to be a bug.

I don't think so actually. At least depending from which angle you see it...
It was introduced to not cause breaking changes when refactoring to native classes, where we define default values als class fields. But those always live on the instance, and are not part of the prototype chain, as was the case for properties in Ember's object model (.extend()). So @defaultValue makes these live on the prototype again (as CPs), as they were before. Maybe it could have been named differently as the primary intent again was to keep the behavior the same, so you could (jokingly) call the decorator @makeComputedOnPrototypeWithAdditionalDefaultValueSupport.
The intent was not to change the treatment of undefined when passed as an argument. That said, I am pretty sure that any 2.x version (i.e. without native classes and no @defaultValue) shows the exact same behavior you are "complaining" about here! See this quick demo: https://ember-twiddle.com/3dca6060033641818c24965a7468bd57
So, as @defaultValue is an implementation detail, and not used by any users, the implementation seems to do exactly what it is meant to do (preserve compatibility)!
That doesn't mean that it wouldn't be desirable to change that. However that seems to be a potentially breaking change. Though not sure if anybody would really intentionally pass @someArg={{undefined}} to "unset" some property - which would not work with the new semantics anymore.
Having said that...

I also noticed that arg decorator used for the components, which have been refactored to @glimmer/component seem to fall back to default value if undefined is passed

This discussions seems to have uncovered that the replacement of @defaultValue (for Ember.Components) with @arg (for Glimmer.Components) could be seen - again in a very edge case scenario - as a breaking change! Not sure if this means we have to rethink this, e.g. change the @arg behavior to match what we had before, and only do what we do now as a later explicitly breaking change?
FWIW, I just had a very similar topic in a PR to embroider yesterday, when I realized that Object.assign() treats a missing property differently than one with an undefined value:
Object.assign({}, { foo: 'bar' }, { baz: true }); // {foo: "bar", baz: true}
Object.assign({}, { foo: 'bar' }, { foo: undefined, baz: true }); // {foo: undefined, baz: true}

That's pretty much the same...
So TL;DR, I am in favor of changing this, but I am not sure if we should do that ignoring any potential (but assumingly very rare) breaking changes, or be totally correct Semver-wise and only introduce this change in a major bump?

So TL;DR, I am in favor of changing this, but I am not sure if we should do that ignoring any potential (but assumingly very rare) breaking changes, or be totally correct Semver-wise and only introduce this change in a major bump?

I see two potential path forwards:

Explore for which arguments undefined is a supported value. There are three cases I currently see:

In some cases (e.g. formLayout of <BsForm>) Ember Bootstrap currently throws if undefined is passed. Changing these should be safe.
In some other cases it may not throw but passing undefined is not documented as a supported value and the component does not behave as expected if doing so. Not sure how we should handle these case. Strictly speaking it may not be considered as a breaking change if undefined is not documented as a supported value.
And of course there might be cases in which undefined is a supported value but not the default one. If we have these cases we would need to drop undefined as a supported value and introduce null as a replacement (if it's not yet supported). These will be a breaking change for sure.

Introduce an optional feature to change the behavior from compatibility mode to apply default if undefined is passed. This will allow us to move forward without waiting for a new major. @embroider/macros should provide a great infrastructure for that one. It should only affect @defaultValue and @arg decorators. So the increased complexity should be fine.

The optional feature also has the benefit that we can provide a path forward for @arg decorator. As we already released the breaking change in v4.1 rolling it back would resolve the breaking change if compared to v4.0 but introduce a breaking change if compared to v4.1. Supporting both through an optional feature seems to be the only path forward here in my opinion.

As we already released the breaking change in v4.1

We didn't release anything related to this at all, did we?
The Glimmer component stuff is still unreleased!
But I like the idea of that optional feature anyway! Indeed exposing this flag as our ownConfig enables to easily support different builds using  embroiders babel macros!

As we already released the breaking change in v4.1

We didn't release anything related to this at all, did we?
The Glimmer component stuff is still unreleased!

Oh. You are right. I thought it was released as part of v4.1. Good that we catched this one before releasing!

But I like the idea of that optional feature anyway! Indeed exposing this flag as our ownConfig enables to easily support different builds using embroiders babel macros!

Will implement in this pull request.

@simonihmig I implemented a useDefaultValueIfUndefined optional feature. Changed @defaultValue and @arg decorators to use it. I also added an additional ember try scenario to test it.
Will writing some tests just to have one I directly run into another case, which wasn't covered by changing one of the decorators: model argument of <BsForm>. So #1250 wasn't a rare case and I expect to face more issues like that. But I'm a little bit lazy to write tests for all of our public API. Would it be fine for you if we merge this one without additional tests and than fix bug by bug?

Would it be fine for you if we merge this one without additional tests and than fix bug by bug?

Yes.

I would recommend that we consider every difference between not passing an argument or passing it with undefined value as a bug if useDefaultValueIfUndefined optional feature is enabled.

Makes sense.

Not sure if we should document that optional feature just right now. Maybe we can treat it as unstable API for right now to gather some more experience without blocking a new minor release?

Agree.
Hope the embroider issue get's sorted out. Already spent most of last Friday on another one. I may be able to look at tis new one some time, but no guarantees...
Thanks for working on this!


Migrating to Hyperforce

Hardcoded URL preventing step verification for migrating to Hyperforce.
This is caused on Viusalforce Page:' Welcome', the hardcoded URL is "http://na1.salesforce.com/help/doc/en/configuring_remoteproxy.htm"

@Eman-Bass, this shouldn't be a problem in the latest release (2.19).
This is a false positive problem by the scanner utility, it won't break on Hyperforce even if you don't upgrade.


Direct V8 support via it's d8 runner.

On FreeBSD, d8 is kinda faster than node. Plus it does not fail in strange ways on 32bits architectures.
I stole the javascript runner from the SpiderMonkey one.

This repository has been deprecated and has moved to https://github.com/rails/execjs.


youtube player instead of Twitch player

we need yotube player instead of twitch player

https://www.youtube.com/embed/live_stream?channel=CHANNEL_ID

https://stackoverflow.com/questions/39275889/how-to-embed-new-youtubes-live-video-permanent-url


Always log DeltaConnectionFailureToConnect

When analyzing sessions that fail to connect, we don't have the luxury of the "MultipleDeltaConnectionFailures" telemetry event.  We are already logging JoinSession_end over and over again (only once every 8 seconds!) so also logging "DeltaConnectionFailureToConnect" every time (instead of just once) feels ok.
This also adds additional context of which step failed (JoinSession, GetWebsocketToken, or initializing socket.io).
Also fixes #9185

Related https://github.com/microsoft/FluidFramework/issues/9185 (+@chensixx, who has it assigned in March).
This will result in a ton of events in error table. I think we should keep error table clean (i.e., any error should ideally have a bug to get rid of it one way or another).

@vladsud thanks for the review!  Made some updates.  This should resolve #9185 now I think (FYI @chensixx )


ImportBase fails with not-null fields even if defined with default value

When loading a JSON data file with ImportBase, looks like a not-null field without a supplied value is failing import even if the model defines a default value.

This was not fixed entirely. Need to reopen.

Fixed and verified in latest client load.


feat: make learning interface mobile responsive

What does this PR do?
Make learning interface mobile responsive
Description of Task to be completed?
Make learning interface content and editor stack on each other in case viewport width < 768px
When in mobile view

Remove lessons header
only show icon for button action

How should this be manually tested?
check the preview link generated by netlify build
Any background context you want to provide?
Screenshots(optional)
If applicable, add screenshots to help to explain the problem/solution.
Additional context(optional)
Add any other context if any.

The code snippets in the content are overflowing. Check this screenshot.


Detect when GNU coreutils is installed on Mac OS X and use the correct date functions

As some users may have installed GNU coreutils on Mac OS X, e.g. brew install coreutils, it's possible that
the date command uses the GNU version, instead of the standard BSD version.

Detect if GNU coreutils is installed on Mac and if it is, use the GNU variants of date functions
Reduce some of the duplication in the file, which resolves a bug where the cygwin version of how_many_days_from_today()
had the operands switched around, leading to a positive result instead of negative
Add test_tcp_connectivity function for cygwin (uses the GNU variant)

Fixes #534
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Tested by running check13 in debug on standard Mac OS X. Confirmed that date was standard - does not support --version argument:
$ date --version
date: illegal option -- -
usage: date [-jnRu] [-d dst] [-r seconds] [-t west] [-v[+|-]val[ymwdHMS]] ...
            [-f fmt date | [[[mm]dd]HH]MM[[cc]yy][.ss]] [+format]

$ bash -x ./prowler -b -c check13 2>&1 | grep -E 'gnu|bsd'
++ '[' darwin18.7.0 == linux-gnu ']'
+ bsd_decode_report
++ bsd_how_older_from_today 2020-04-01
++ bsd_how_older_from_today 2020-03-10
++ bsd_how_older_from_today 2020-03-29
++ bsd_how_older_from_today 2020-04-09

Created a symlink in the prowler directory to gdate (have coreutils installed but not set to Mac default):
$ ln -s /usr/local/bin/gdate date
$ ls -ltrah
...
date -> /usr/local/bin/gdate
$ date --version
date (GNU coreutils) 8.32

$ bash -x ./prowler -b -c check13 2>&1 | grep -E 'gnu|bsd'
++ '[' darwin18.7.0 == linux-gnu ']'
+ gnu_decode_report
++ gnu_how_older_from_today 2020-04-01
++ gnu_how_older_from_today 2020-03-10
++ gnu_how_older_from_today 2020-03-29
++ gnu_how_older_from_today 2020-04-09

This works like a charm! Thanks again for fixing this!!!

Thanks @marcjay


Embed python katacoda

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b433e02279a97484cd6f41989f3181b3595dce8c/
Edited pages:

tutorials/build-a-python-app-with-cockroachdb-interactive.md
v20.2/build-a-python-app-with-cockroachdb.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/32a646d3e718f2697ee6415c28ee055afac6dab8/
Edited pages:

tutorials/build-a-python-app-with-cockroachdb-interactive.md
tutorials/katacoda/build-python-app/step1.md
tutorials/katacoda/build-python-app/step3.md
v20.1/build-a-python-app-with-cockroachdb.md
v20.2/build-a-python-app-with-cockroachdb-django.md
v20.2/build-a-python-app-with-cockroachdb-pony.md
v20.2/build-a-python-app-with-cockroachdb-sqlalchemy.md
v20.2/build-a-python-app-with-cockroachdb.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a98a1c80e6d7e7c592a489751d7666c4256ef811/
Edited pages:

tutorials/build-a-python-app-with-cockroachdb-interactive.md
tutorials/katacoda/build-python-app/step1.md
tutorials/katacoda/build-python-app/step3.md
tutorials/katacoda/build-python-app/step4.md
v20.1/build-a-python-app-with-cockroachdb.md
v20.2/build-a-python-app-with-cockroachdb-django.md
v20.2/build-a-python-app-with-cockroachdb-pony.md
v20.2/build-a-python-app-with-cockroachdb-sqlalchemy.md
v20.2/build-a-python-app-with-cockroachdb.md


Temperature Sensor only displays in increments of 0.5°C in Google Home

In the addon's ingress page (nice that that's now implemented btw!), my temperature sensors read as follows:

Which is in line with what Home Assistant reports. However, in the Google Home app (and thereby also when using Google Assistant), it only ever shows in increments of 0.5°C:

We are reporting the value exactly as you see it on the UI to all controllers. It seems to be google home which just displays rounded values.

We are reporting the value exactly as you see it on the UI to all controllers. It seems to be google home which just displays rounded values.

Well, that's annoying. Must be yet another symptom of their poor implementation of Matter. These same sensors, when connected directly via their cloud integration to Google Home, report accurate values. Thanks for your quick response.
Closing as issue is with Google, not this addon.

I think not that is has anything to do with a "poor matter implementation" but maybe with the definitions of their understanding of Smart home devices and usage ... But yes ... should be reported to Google


Find/Remove-DbaLongRunningBackup

Is this a feature OR bug:
https://trello.com/c/YP93MDWy
Additional comments
Needs to be fleshed out a bit more or closed.

done!

people can use stop-process to "remove"


Memory leaks

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)
na

Platform
PC

SDK Version
2.11.1

Language
C#

Issue Description
The Device class (and more) doesn't contain a destructor or dispose.
Calling QueryDevices continuously creates new Device objects which live on after. When using this method numerous times it ultimately causes a OutOfMemoryException.

[Realsense Customer Engineering Team Comment]
@sennevanoppen
I did find the memory leak of this function, and will let engineer knows about this issue. Can you let me know why you need to query device often? To check if device exist?

Should be resolved on the development branch and going to be part of 2.16 release

I believe this is resolved now, please open a new issue if you encounter any problem


kata-env: support JSON output

Description of problem
kata-runtime kata-env only outputs in TOML right now. It would be great if it supported some other formats - for instance, the metrics system currently parses some of that TOML into JSON, and it would be great if we could just take a complete JSON output and store it away with the test results.
Expected result
An option such as kata-runtime kata-env --json to generate JSON formatted output.
One might consider YAML as well - but I did not find an obvious YAML encoder to use, so will stick to JSON additions for now.

I thought I already had an issue for this, either in this repo or in the tests repo or even over in the old CC repos - but I cannot find it. It has been mentioned (by me) a couple of times in the past. Time to 'make it so'.
I'll try to get some runtime size diffs as well, for reference.

OK, a very quick size between the base master commit and the json version:
   text    data     bss     dec     hex filename
11924802         321417<PHONE_NUMBER>2147         bd16d3 kata-runtime (json)
11924506         321257<PHONE_NUMBER>1691         bd150b kata-runtime (b2bec33)

That looks like a very minimal size increase to me - anybody feeling concerned?


Some data in the State variable may be corrupt

As identified by this issue https://github.com/NCAR/MUSICA-Performance-Comparison/issues/62, if we reuse the State variable between different iterations of solve function, the results are corrupt and we get fail to converge error message later.
Kyle commented that this was also the reason why he had to initialize the LU matrix to zero originally (https://github.com/NCAR/micm/issues/587).
This issue indicates that the LU matrix in State variable is not overwritten correctly between iterations (or we should not expect it to be overwritten correctly at all?).

The state should be completely reusable. The fact that it isn't is a problem

Thanks Kyle for the clarification. I looked at these two lines https://github.com/NCAR/micm/blob/main/include/micm/solver/lu_decomposition.inl#L201 and https://github.com/NCAR/micm/blob/main/include/micm/solver/lu_decomposition.inl#L214. Could the Uvector use the values from the previous time step without a zeroing?

We set them just before those, but it's in an if block so maybe not all of the values are zeros. Perhaps we should add back a zero of L and U

Thanks @K20shores for your comments. Hmm, interesting. For the first time when the State variable is generated, are the L and U matrices zeroed?

I've learned some things. Thanks to @mattldawson's suggestion, I set the L and U matrices to Inf and found out that sometimes we end up with infs after the LU decomposition.
The issue seems to be related to the interplay between block size (which are grid cells) and the number of species.
Sizes leading to valid L and U matrices

Species (size)
Blocks (grid cells)

1
1

2
1

2
2

3
2

Sizes leading to L and U matrices with infs

Species (size)
Blocks (grid cells)

3
1

3
2

3
3

3
4

Also, the math on geeks for geeks for the doolittle algorithm that we use indicates that U should always take a value from the input matrix, but I can see in the debugger that on some cases it doesn't. I think this means we are missing some do_aik_ values which are made in the initialize function

https://github.com/NCAR/micm/blob/94c22da20e387f55dc0ccdb5c582668894b60e07/include/micm/solver/lu_decomposition.inl#L56-L66

Thanks to @K20shores , we track down the issue further. Given an initial Jacobian matrix with size 4x4 and nonzero elements marked as X below:
X 0 0 0
0 X X X
X X X X
0 X 0 X

The expected L matrix should be:
X 0 0 0
0 X 0 0
X X X 0
0 X X X

While we are constructing the third element of the fourth row,L[3][2] is not overwritten by the corresponding Jacobian matrix element first. https://github.com/NCAR/micm/blob/94c22da20e387f55dc0ccdb5c582668894b60e07/include/micm/solver/lu_decomposition.inl#L211
Since we are initializing the L matrix with inf now, this leads to a inf - some value operation later and corrupts the result.
https://github.com/NCAR/micm/blob/94c22da20e387f55dc0ccdb5c582668894b60e07/include/micm/solver/lu_decomposition.inl#L214
After discussion with Kyle, the lines https://github.com/NCAR/micm/blob/94c22da20e387f55dc0ccdb5c582668894b60e07/include/micm/solver/lu_decomposition.inl#L56 and https://github.com/NCAR/micm/blob/94c22da20e387f55dc0ccdb5c582668894b60e07/include/micm/solver/lu_decomposition.inl#L85 can be problematic. The nonzero index of lower and upper matrices should be set by checking the lower and upper matrices themselves, rather than the Jacobian matrix unless we have a misunderstanding of what is going on here.

@jian helped me pin down the problem.
The issue is that the sparsity pattern of the LU matrices doesn't necessarily match the sparsity pattern of the jacobian matrix. In the algorithm, the LU matrices need to be filled with the value from the A matrix when the LU matrices are nonzero.
The first change we needed to make was to accurately record when to set the LU matrix values based off of when the LU matrices are zero, not the jacobian as was happening before.
The second change is to record a sentinel value. This value indicates when the L or U matrix is nonzero and the jacobian is actually zero. In this case, we set the LU matrix to zero rather than copying from the jacobian matrix.


[icd] fix DefaultICDClientStorage::LoadFabricList

DefaultICDClientStorage::LoadFabricList mistakenly uses TLV::ScopedBufferTLVReader reader(std::move(backingBuffer), len);
Instead of TLV::ScopedBufferTLVReader reader(std::move(backingBuffer), length);
Resulting garbage data after the TLV payload and failure of reader.VerifyEndOfContainer()
This PR fixes this issue.
Also, we can make kMaxFabricListTlvLength a constexpr.
Note: Ignore whitespace changes when reviewing changes to ut.

please also fix TLV::ScopedBufferTLVReader on https://github.com/project-chip/connectedhomeip/blob/39593141dec42eea75506f9bba3a9b80e7e9ca6c/src/app/icd/client/DefaultICDClientStorage.cpp#L191 and https://github.com/project-chip/connectedhomeip/blob/39593141dec42eea75506f9bba3a9b80e7e9ca6c/src/app/icd/client/DefaultICDClientStorage.cpp#L227?

https://github.com/project-chip/connectedhomeip/pull/30767