Release 0.992.0 m6

Purpose

Release 0.992.0 m6

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1ca86c8). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #16643   +/-   ##
=========================================
  Coverage          ?   68.85%
=========================================
  Files             ?        5
  Lines             ?      122
  Branches          ?       21
=========================================
  Hits              ?       84
  Misses            ?       38
  Partials          ?        0

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 1ca86c8...302aceb. Read the comment docs.


Volumeof cube

Why:

Closes #323
What's being changed: Added code for volume of cube

Hey @ashankritwik07
Please update the linked issue for this and other PR(s) also

Closes #ISSUE_NUMBER

For example Closes #1

Done sir , do check

Done sir , do check

@ashankritwik07
The one you linked is this PR. It needs to be linked to the issue

@victoriacheng15 hi sir , please review the code.


Update Postgres "Create Server..." default version from 10 to 11

Support for Version 10 is ending on November 10, 2022.
https://www.postgresql.org/support/versioning/

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

Now I think it should be updated to version 14. That is what we're using for the official WebApp + DB template for Azure.


encoding error : input conversion failed

Hi,
I'm using mitmdump -T with inline script and parallel decorator. It's printing frequent error on console as "encoding error : input conversion failed due to input error, bytes 0x81 0x18 0x19 0xA0"
But the proxy works as normal with high cpu utilization(more than 100%) when processing requests.
Any idea what is it?

Is this related to #657?

what about the CPU utilization? It goes more than 100% when browse using only one connection. a quick spike of CPU when request comes. it's ok for one user. But what will happen when 100 users browsing same time?

It would be appreciated if you could answer @Kriechi's question when he's trying to help you. :wink:
Regarding CPU Utilization: mitmproxy is bound to a single CPU core, so 100% CPU utilization would be pretty surprising unless you're running on a very limited machine. In that case, 100% would not be surprising. Whatsoever, mitmproxy is not designed to proxy a large number of users.


Update fluent-plugin-cloudwatch-logs to 0.10.0

Update fluent-plugin-cloudwatch-logs to get fix for DescribeLogStreams erroneous calls:
https://github.com/fluent-plugins-nursery/fluent-plugin-cloudwatch-logs/pull/194
For testing, did these commands:
  vi templates/Gemfile.erb
  make gemfile    IMAGE_NAME=fluent/fluentd-cloudwatch VERSION=v1.11.0-debian-cloudwatch-1.0 DOCKERFILE=v1.11/debian-cloudwatch
  make dockerfile IMAGE_NAME=fluent/fluentd-cloudwatch VERSION=v1.11.0-debian-cloudwatch-1.0 DOCKERFILE=v1.11/debian-cloudwatch
  make image      IMAGE_NAME=fluent/fluentd-cloudwatch VERSION=v1.11.0-debian-cloudwatch-1.0 DOCKERFILE=v1.11/debian-cloudwatch
  docker run -it -e KUBERNETES_SERVICE_HOST=kubernetes.docker.internal -e KUBERNETES_SERVICE_PORT=6443 fluent/fluentd-cloudwatch:v1.11.0-debian-cloudwatch-1.0

@PenelopeFudd Thanks for the patch. Could you follow DCO?

Is there a way to modify the DCO message to say "Only maintainers have permission to sign off"?

I'd like to say:

Contributors sign-off that they adhere to these requirements by adding a Signed-off-by line to commit messages.

Please make sure you have the correct access rights
and the repository exists.

I confess, I'm not a git expert yet; perhaps the DCO message wanted me to check out the forked repository, amend the git log there, then git push, and some other command to make sure the pull request gets the change to the log message?

I've tried a few things, but my local repo is now hopelessly messed up.  Closing this PR and creating a new one.


Is there a way to have the data in the 2.0 chart node survive restarts?

Description
The V1 chart has a tested and working subflow that saves the data and reloads the data
after a restart. This is pretty vital functionality and needed for V2 charts.
Can someone have look at this when there is time please?
I tested the old working subflow with the V 2 chart and as expected it does not work.
It would be great if this subflow could be updated to work with 2.0.
or of course any other way to have the data survive reboots if possible?
This is the subflow that worked with V1 hooked up to a V2 chart for tests:
[
    {
        "id": "3b6a1337b69e1574",
        "type": "subflow",
        "name": "vvb d2",
        "info": "",
        "category": "",
        "in": [
            {
                "x": 50,
                "y": 30,
                "wires": [
                    {
                        "id": "0803c4cc5eb6de01"
                    }
                ]
            }
        ],
        "out": [
            {
                "x": 820,
                "y": 160,
                "wires": [
                    {
                        "id": "1d445f73ea677ec2",
                        "port": 0
                    },
                    {
                        "id": "de6cdd9bd32b3eb1",
                        "port": 0
                    }
                ]
            }
        ],
        "env": [],
        "meta": {},
        "color": "#DDAA99"
    },
    {
        "id": "1d445f73ea677ec2",
        "type": "file",
        "z": "3b6a1337b69e1574",
        "name": "",
        "filename": "",
        "appendNewline": true,
        "createDir": true,
        "overwriteFile": "true",
        "x": 660,
        "y": 140,
        "wires": [
            []
        ]
    },
    {
        "id": "006debb5e922bb46",
        "type": "file in",
        "z": "3b6a1337b69e1574",
        "name": "",
        "filename": "",
        "format": "utf8",
        "sendError": true,
        "x": 500,
        "y": 180,
        "wires": [
            [
                "de6cdd9bd32b3eb1"
            ]
        ]
    },
    {
        "id": "e37d519cbbd162d3",
        "type": "json",
        "z": "3b6a1337b69e1574",
        "name": "",
        "x": 500,
        "y": 140,
        "wires": [
            [
                "1d445f73ea677ec2"
            ]
        ]
    },
    {
        "id": "de6cdd9bd32b3eb1",
        "type": "json",
        "z": "3b6a1337b69e1574",
        "name": "",
        "x": 660,
        "y": 180,
        "wires": [
            []
        ]
    },
    {
        "id": "0803c4cc5eb6de01",
        "type": "function",
        "z": "3b6a1337b69e1574",
        "name": "LoadSave",
        "func": "var strSafe=msg.topic;\nif(strSafe)\n{\n    strSafe=strSafe.replace(/[^a-z0-9]/gi, '_').toLowerCase();\n    var loaded=context.get(strSafe)||0;\n    msg.filename =\"/home/hb19/.node-red/flowsave/vvbd2/\"+strSafe+\".dat\";\n    //node.warn(loaded);\n    if(0===loaded)\n    {\n        //load chart!! \n        msg.payload=\"load\";\n        context.set(strSafe,1);\n        \n        //node.warn(\"Loading \" +strSafe);\n    }\n    else\n    {\n        //node.warn(\"Writing \" +strSafe);\n    }\n    return msg;    \n}\nreturn null;\n\n",
        "outputs": 1,
        "timeout": "",
        "noerr": 0,
        "initialize": "",
        "finalize": "",
        "libs": [],
        "x": 200,
        "y": 160,
        "wires": [
            [
                "abd9aa795d47fbc6"
            ]
        ]
    },
    {
        "id": "abd9aa795d47fbc6",
        "type": "switch",
        "z": "3b6a1337b69e1574",
        "name": "Load data",
        "property": "payload",
        "propertyType": "msg",
        "rules": [
            {
                "t": "neq",
                "v": "load",
                "vt": "str"
            },
            {
                "t": "eq",
                "v": "load",
                "vt": "str"
            }
        ],
        "checkall": "false",
        "repair": false,
        "outputs": 2,
        "x": 350,
        "y": 160,
        "wires": [
            [
                "e37d519cbbd162d3"
            ],
            [
                "006debb5e922bb46"
            ]
        ]
    },
    {
        "id": "3dc8c1a1d345ef1d",
        "type": "ui-chart",
        "z": "98b31619.daed3",
        "g": "120d335300e6531b",
        "group": "f83d15d3a238accd",
        "name": "vvb chart week",
        "label": "",
        "order": 4,
        "chartType": "line",
        "category": "topic",
        "categoryType": "msg",
        "xAxisLabel": "",
        "xAxisProperty": "",
        "xAxisPropertyType": "property",
        "xAxisType": "time",
        "xAxisFormat": "",
        "xAxisFormatType": "auto",
        "yAxisLabel": "",
        "yAxisProperty": "",
        "ymin": "15",
        "ymax": "65",
        "action": "append",
        "stackSeries": false,
        "pointShape": "line",
        "pointRadius": "2",
        "showLegend": false,
        "removeOlder": 1,
        "removeOlderUnit": "604800",
        "removeOlderPoints": "",
        "colors": [
            "#ff9300",
            "#00fdff",
            "#ff7f0e",
            "#2ca02c",
            "#a347e1",
            "#d62728",
            "#ff9896",
            "#9467bd",
            "#c5b0d5"
        ],
        "textColor": [
            "#ffffff"
        ],
        "textColorDefault": false,
        "gridColor": [
            "#7a7a7a"
        ],
        "gridColorDefault": false,
        "width": "30",
        "height": "10",
        "className": "",
        "x": 4420,
        "y": 540,
        "wires": [
            [
                "2e85ceab3abb96f6"
            ]
        ]
    },
    {
        "id": "2e85ceab3abb96f6",
        "type": "subflow:3b6a1337b69e1574",
        "z": "98b31619.daed3",
        "g": "120d335300e6531b",
        "name": "",
        "x": 4430,
        "y": 600,
        "wires": [
            [
                "3dc8c1a1d345ef1d"
            ]
        ]
    },
    {
        "id": "f83d15d3a238accd",
        "type": "ui-group",
        "name": "vvb",
        "page": "40ab10feb1ddf2bb",
        "width": "30",
        "height": "20",
        "order": 1,
        "showTitle": false,
        "className": "",
        "visible": "true",
        "disabled": "false"
    },
    {
        "id": "40ab10feb1ddf2bb",
        "type": "ui-page",
        "name": "vvb",
        "ui": "cd62f850b848415c",
        "path": "/vvb",
        "icon": "home",
        "layout": "grid",
        "theme": "404b2b83a6155d6c",
        "breakpoints": [
            {
                "name": "Default",
                "px": "0",
                "cols": "30"
            },
            {
                "name": "Tablet",
                "px": "1080",
                "cols": "30"
            },
            {
                "name": "Small Desktop",
                "px": "1080",
                "cols": "30"
            },
            {
                "name": "Desktop",
                "px": "1080",
                "cols": "30"
            }
        ],
        "order": 1,
        "className": "",
        "visible": true,
        "disabled": false
    },
    {
        "id": "cd62f850b848415c",
        "type": "ui-base",
        "name": "flax 24",
        "path": "/dashboard",
        "includeClientData": true,
        "acceptsClientConfig": [
            "ui-notification",
            "ui-control"
        ],
        "showPathInSidebar": false,
        "showPageTitle": false,
        "navigationStyle": "default",
        "titleBarStyle": "default"
    },
    {
        "id": "404b2b83a6155d6c",
        "type": "ui-theme",
        "name": "dark jtm",
        "colors": {
            "surface": "#000000",
            "primary": "#0094ce",
            "bgPage": "#000000",
            "groupBg": "#000000",
            "groupOutline": "#000000"
        },
        "sizes": {
            "pagePadding": "12px",
            "groupGap": "12px",
            "groupBorderRadius": "4px",
            "widgetGap": "12px",
            "density": "comfortable"
        }
    }
]

Have you provided an initial effort estimate for this issue?
I am no FlowFuse team member

Is there a way to up-vote this basic request?
"a way to have the data in the 2.0 chart node survive restarts"

Further investigation is required on our part on how we can support this. We can't support this natively in the module, as when node-RED restarts, all temporary data is cleared.
The flow shared utilises a file-in/file-out out, which we can investigate, and provide a sample flow for.

I think there will be ongoing requests for this feature.  The D1 node accomplishes it by providing an output which can can be saved in file or context and directly re-injected on startup, which is easy for the user to implement and does the job perfectly.

1) Retention of data points within v2.0 (possible to-do flowuse)
In my opinion, a perfect basis would be if the ui-graph in version 2.0 would generate an output like in version 1.0. Namely all >displayed< data points (older data points are obsolete) as an array, instead of just the last data point received. (The array must actually already be there internally, because how else does the graph display the data points for newly calling clients?)
2) Saving and injecting (to-do for someone else)
There is already a simple external option for permanently saving and then injecting the output array, e.g. with PERSIST.
Alternatively, you can adapt the flow mentioned in this thread.

Yes, I am considering the unified output approach, but we'd also then need to add support for injecting that data format too.

we'd also then need to add support for injecting that data format too.

Could it output data compatible with the existing config of the node, or would that be difficult to achieve?

Hello.
Just some positive feedback and also a question that I would assume I will not get an answer to as it is a bit of a "how long is a string" type of question. But here it goes:
We have finished migrating 3 offgrid systems  from Dashboard V1 to Dashboard V2
that are now only missing this feature. It has gone much better than expected.
D2 seems very stable and robust so far with much better performance than D1.
So very happy about that and kudos for great work and the migration tools and docs!
If at all possible. we would like to deploy these systems March-April next year, so is it realistic to hope for this feature by then by any chance?
Just planning ahead, and the V1 systems can certainly last for a while more so not vital but would be nice as maintenance for V1 has become a tad cumbersome and stale.
To be clear:
A way to have the data in the 2.0 chart node save the data and reload that data / survive restarts.


chore(deps): update devdependency @types/node to ^20.16.3

This PR contains the following updates:

Package
Type
Update
Change

@types/node (source)
devDependencies
patch
^20.16.2 -> ^20.16.3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Hywax Assistant.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (b9d280b) to head (49097a7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #64   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          146       146
  Branches        26        26
=========================================
  Hits           146       146

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


test: Add some code coverage

Adding some tests to contribute to code coverage.
Submitting for PR to see if the tests run ok in the pipeline.

/azp run

/azp run

Codecov Report

Merging #1392 (1acac3e) into master (52b440c) will decrease coverage by 84.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1392       +/-   ##
==========================================
- Coverage   84.77%   0.60%   -84.18%
==========================================
  Files         287     258       -29
  Lines       14250   12989     -1261
  Branches      719     719
==========================================
- Hits        12080      78    -12002
- Misses       2170   12911    +10741

Impacted Files
Coverage Δ

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/scala/org/apache/spark/ml/param/MapParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/scala/org/apache/spark/ml/param/UDPyFParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...cala/org/apache/spark/ml/NamespaceInjections.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...cala/org/apache/spark/ml/param/DataTypeParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/ByteArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/EvaluatorParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/ParamSpaceParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/com/microsoft/azure/synapse/ml/stages/udfs.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 270 more

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 52b440c...1acac3e. Read the comment docs.


Create an optional packrat optimization

A packrat parser is a PEG parser like Parser that memoizes parts of the parse, so as to make backtracking much cheaper.  It guarantees linear parse time.  This comes at the cost of space, and potentially a lot of it.  It should therefore be an optional strategy, perhaps controlled via a parameter to *parse().

This turns out not to be bpossible, because of actions.  It took me a long time to realize this.  An action may have side effects outside of the parse, or inside of the parse.  By this, I mean that the side effect could be completely remote, like logging or launching a missile.  It could also be technically a side effect, but still contribute to some attribute, like:
#include <boost/parser/parser.hpp>

#include <iostream>
#include <string>

namespace bp = boost::parser;

int main()
{
    std::cout << "Enter a list of doubles, separated by commas. ";
    std::string input;
    std::getline(std::cin, input);

    std::vector<double> result;
    auto const action = [&result](auto & ctx) {
        std::cout << "Got one!\n";
        result.push_back(_attr(ctx));
    };
    auto const action_parser = bp::double_[action];
    auto const success = bp::parse(input, action_parser % ',', bp::ws);

    if (success) {
        std::cout << "You entered:\n";
        for (double x : result) {
            std::cout << x << "\n";
        }
    } else {
        std::cout << "Parse failure.\n";
    }
}

In a packrat parser, nonterminals' parse results are recorded.  So for a parser like action_parser % ',' above ("P" hereafter), were it part of a larger parse, we would memoize its result the first time we used it at location L within the parse.  Then, later on, we would re-use that memoized result if trying to use P at that same location L.  That works in this case; even though we don't re-run the actions within action_parser, the result is correct, because we captured the result of those actions in an external variable.  However, if something else in the code were to reset the external variable result, we would inappropriately not have re-run the actions within P.  There is no way for a memoization system to know whether to re-execute the actions or not; the right answer depends on the user's particular use of actions.
A similar problem exists for "external" (that is non-attribute-affecting) side effects.  If the user writes actions to count successful parses, re-running them is an error.  If the user writes actions to count all parses, not re-running them is an error.  Either convention will turn out to be wrong for some users.
In short, the packrat parser change would not be an optimization, since it changes the library's semantics.


Project maintenance state

I can see that @ivy is still active on GitHub, but still, nothing has been committed on this repository for almost one year.
As this fork has been described as "Actively maintained fork of gomail", I would like to know if this project is still going to be maintained.
If not, is there any maintained and viable fork?
Sincerely,
Charles

To my knowledge, the project is stable. It's true, I haven't a lot of time recently to give serious focus to developing new features or solving some outstanding bugs caused by various edge-cases. I will, however, gladly give attention to anyone putting forth an effort to step up to the task of making an improvement in the health of this codebase.

Dear all, it is possible to change your code?
Change from here:

https://github.com/go-gomail/gomail
https://github.com/go-gomail/gomail/issues/188
https://github.com/go-mail/mail
https://github.com/Shopify/gomail
https://github.com/Shopify/gomail/issues/59
https://github.com/ory/mail

It has been unmaintained and maybe one or more vulnerabilities :/
There are:

https://github.com/xhit/go-simple-mail from @xhit
https://github.com/wneessen/go-mail from @wneessen

Linked to:

https://github.com/go-gomail/gomail/issues/182
https://github.com/go-gomail/gomail/issues/163
https://github.com/go-gomail/gomail/issues/142
https://github.com/go-gomail/gomail/issues/108
https://github.com/go-gomail/gomail/issues/104
https://github.com/go-gomail/gomail/issues/105
https://github.com/go-gomail/gomail/issues/94
https://github.com/go-mail/mail/issues/78
https://github.com/go-mail/mail/issues/73
https://github.com/go-mail/mail/issues/61


Optional EOS for file-based sources

When file-based adapters are used, they must support mandatory passing EOS or ignore it. It is necessary to support making it possible to use trackers in processing pipelines that use tracking, and source-adapter feeds the framework on a file-by-file basis infinitely.

We have EOS_ON_LOCATION_CHANGE parameter in the media files source adapter, but currently set it automatically, depending on the adapter type (image or video). We can allow users to set it.
https://github.com/insight-platform/Savant/blob/develop/adapters/gst/sources/media_files.sh#L31


USB serial interface malfunction

I am using USB serial intefaces P2P link with simple config as this:
[[Serial Interface]]
type = SerialInterface
interface_enabled = True
# Serial port for the device
port = COM18
speed = 115200
databits = 8
parity = none
stopbits = 1
While messages do work, sending files STUCKs forever. Same for Win app and Python app ....
I have a gess that my serial interface buffer is limited to 240bytes packets but no idea how add this in inteface's settings.

Hey there! Thanks for the report.
I do find that larger files don't send at all, usually over 1MB. I am yet to investigate this...

Have you tried sending a smaller file? Something around 20KB?
Have you tried using Sideband with the same config? Does the file that fails in MeshChat work in Sideband?

The default incoming file limit for LXMF is 1MB, but it can be managed by the user / application with LXMRouter.delivery_per_transfer_limit. Sideband has a user option to manage this, but it's up to the individual application how to handle it.
Apparently, I forgot to add an API function for setting it, but just using setting LXMRouter.delivery_per_transfer_limit (in kilobytes) directly should be fine for now.

I updated MeshChat a few versions back to increase the default limit to 10MB.

117b02cc67f86ba04a3d3d8f56d0e9cf8619a3fe
https://github.com/liamcottle/reticulum-meshchat/blob/c92ddac00dbc5521ee59ce41d8c572a5a8f175ef/meshchat.py#L114

This will likely become a configurable option in the settings UI down the line.
@markqvist RE files/messages failing to send when over the recipients receiving limit, do you have any thoughts on adding a response (in the link) from LXMF with a "message rejected" or "message exceeds size limit" status in this case? Currently, the LXMF router will just keep trying to send the message again, up until the max attempts, even though this message will never succeed in sending.

Yes, but I just haven't gotten around to designing the best way to handle it yet. Ideally, it would be something the sender knew beforehand, so it doesn't have to even try delivering a large message if the recipient won't even accept it anyways. There's a few considerations to go through there, though, both implementation and privacy-wise.

Closing out. This shouldn't be an issue with MeshChat itself. If you still see this behaviour feel free to follow up.


Version 3.0

In reviewing the most recent download of CDM, labeled as version 1.76, I noticed that a number of classes are marked as 'Deprecated in version 3.0'. Are there different versioning systems or is 1.76 'on the road' to 3.0?

The description you see referring to "version 3.0" comes from descriptions in the Dynamics365 entities where these entities originate from. They do not refer to the CDM version, so the description can safely be ignored.


Removed login dialog when installing InApps for Huawei

Removed login dialog when installing InApps for Huawei. Changed to call handleInstallError when user is not logged in.
There was a problem when installing InApps when user is logged out. HuaweiPurchaseManager was opening login window - handleInstallError was not called and InApp products was still not installed after successful login. I think that it will be better to call handleInstallError and leave it to a programmer - he can open login window or show text dialog.

Thanks for your contribution. How can the user login, is it possible via an other app?

Yes, user can login via Huawei AppGallery app. If logged once he will be logged all the time unless he click to log out.

Good to know!


No link to godot 3

Hi, I was reading your tutorial : http://kidscancode.org/godot_recipes/3.x/audio/audio_manager/
but link doesn't work and this situation is the same for every other tutorial with .zip file. Are you planning to focus only on godot 4?

No, that was not intentional, but a side-effect of the re-organization to support 3 & 4. I fixed most links, but there are still a few I need to replace.
Thanks for the note!


Light suppression

Hi，Sorry to bother you， Looks like you didn't publish the demo. py  ， So I would like to ask if your light suppression work can also play a good role in suppressing glare in the daytime scene

While our framework is designed for night image enhancement, the unsupervised light-effects suppression network can also work for daytime flare removal since it learns from the unpaired flare/flare-free data shown in Fig.2.
Using constraints specific to daytime to handle remaining problems will be part of our future work.

Thank you for your reply, I wonder if you will publish the relevant code for light effect suppression，thank you

released


.gitignore the .terraform.lock.hcl file

need to ignore file .terraform.lock.hcl

https://www.terraform.io/docs/language/dependency-lock.html

this can be added to branch gitignore to prevent writing to source

cancel


bigtcp: Modularize and use the devices table

As part of the runtime device reconfiguration work, modularize bigtcp implementation and
switch use of option.Config.GetDevices() over to a query on Table[*Device].
The  reconciliation on device changes is not yet fully implemented (WIP in https://github.com/joamaki/cilium/tree/pr/joamaki/bigtcp-reconciliation) as it requires also reconciling the GSO/GRO on all lxc and cilium_ devices.
The changes in this PR were manually tested with test/bigtcp in a kind cluster.

/test

/test

/test


raidboss only working for half of suzaku?

I'm not sure if I'm doing something wrong but when I'm using raidboss it'll show the timeline and the "tankbuster on " trigger for the first half of Suzaku, up until she disappears for the phase change - after the phase change the timeline is gone and the name specific triggers won't show up
it suddenly stopped working a few days ago. I didn't change anything about the files or anything
any chance you'd know something about it?

Did you figure this out?

Kind of! I had to do a fresh install of ACT (for reasons other than cactbot) and it worked after, it was probably just ACT being funny! Thank you!


[mgx-68] Feature/compound

auto compound for XYK
proxy pallet

blocked until #342 is merged
todo: integration test for proxies


Unsigned int constructors too generous

xref https://github.com/numpy/numpy-stubs/pull/70
The code
import numpy as np

class A:
    def __float__(self):
        return 4.0

np.uint8(A())

fails at runtime, but passes type checking. The reason is here:
https://github.com/numpy/numpy-stubs/blob/master/numpy-stubs/__init__.pyi#L402
where the constructor of number is:
class number(generic):
    def __init__(self, value: Union[SupportsInt, SupportsFloat] = ...) -> None: ...

Would you kindly let me know what exactly needs to be done so that I can try working on the issue?

Hey @rajatdiptabiswas, so what will need to be done is adding more specific constructors for things in the signed integer hierarchy:
https://github.com/numpy/numpy-stubs/blob/master/numpy-stubs/__init__.pyi#L420
and the unsigned integer hierarchy:
https://github.com/numpy/numpy-stubs/blob/master/numpy-stubs/__init__.pyi#L444
The constructors will be similar to the generic one here:
https://github.com/numpy/numpy-stubs/blob/master/numpy-stubs/__init__.pyi#L402
but will not longer include SupportsFloat. And of course tests will need to be added!

Thanks for the clarification @person142
I have an exam coming up but I shall try my best to get it done as soon as possible!

Hi @person142
I have tried looking into the issue and these are the things I believe I should be doing.
For each of the classes int8, int16, int32, int64, uint8, uint16, uint32, and uint64 in numpy-stubs/core/__init__.pyi I should create constructors similar to the one in class number(generic) and remove SupportsFloat from the definition.
Hence each of the int8, ..., uint64 functions will be updated to something like:
class uint64(unsignedinteger):
    def __init__(self, value: SupportsInt = ...) -> None: ...

instead of the current implementation
class uint64(unsignedinteger): ...

Would you kindly confirm if these are correct?
I also needed some help writing the tests. I believe I should edit the file numpy-stubs/tests/pass/scalars.py. However, I can't figure out exactly how or what to write.
I am new to writing tests and anything that might help me learn and complete this task would be amazing.
I shall start working on these as soon as you confirm. Thanks in advance!

Would you kindly confirm if these are correct?

Yes, that looks correct.

I also needed some help writing the tests. I believe I should edit the file numpy-stubs/tests/pass/scalars.py. However, I can't figure out exactly how or what to write.

Since we are allowing less in the constructor, I would add cases to tests/fail/scalars.py (as more things should fail now). That would probably look something like
class A:
    def __float__(self):
        return 1.0

np.uint8(A())  # E: <whatever-error-mypy-gives>
...

Note that you can run the tests by doing python runtests.py from the repo root.

I have made the changes you mentioned @person142.
Kindly take a look as to whether I have made them correctly. If you find something wrong please let me know. I'll make the necessary updates!
These are the changes that I have made:

Removed SupportsFloat from the classes int8, int16, int32, int64, uint8, uint16, uint32, and uint64
Removed a test from tests/pass/scalars.py
Added several tests to tests/fail/scalars.py

Looks like the checks have failed for the PR. I'll look into what's wrong.


Wsteth connector

Added WSTETH connector with deposit and withdraw functions.

Connector details -
Name - WSTETH-A
Address - 0xa1040eaaB1D5ba731aAD6D9b59C70E1E1a5BEd57


Git blame format list the filepath too long.

for me, the most information of bit blame is sha, commiter and date, but now the git blame list sha filepath first, and the filepath always is very long, I couldn't get my concerned info in my first eye.

I agree. Most of all, the file path is redundant when running git blame on a single file.
It should be shown at the top and not in every single line

+1


azure-mgmt-digitaltwins failing basic import

install the package -> import * from azure.mgmt.digitaltwins
INFO:root:Importing all modules from namespace [azure.mgmt.digitaltwins] to verify dependency
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/__init__.py", line 13, in <module>
    from ._azure_digital_twins_management_client import AzureDigitalTwinsManagementClient
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/_azure_digital_twins_management_client.py", line 16, in <module>
    from .operations import DigitalTwinsOperations
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/operations/__init__.py", line 12, in <module>
    from ._digital_twins_operations import DigitalTwinsOperations
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/operations/_digital_twins_operations.py", line 17, in <module>
    from .. import models
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/models/__init__.py", line 30, in <module>
    from ._models import CheckNameRequest
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/models/_models.py", line 88, in <module>
    class DigitalTwinsResource(Model):
  File "/home/vsts/work/1/s/sdk/digitaltwins/azure-mgmt-digitaltwins/.tox/depends/lib/python2.7/site-packages/azure/mgmt/digitaltwins/models/_models.py", line 128, in DigitalTwinsResource
    sku = DigitalTwinsSkuInfo()
NameError: name 'DigitalTwinsSkuInfo' is not defined

I will note that while the above error is from the py27 env. The failure occurred on all platforms.

Yeah this is a known issue - I believe the SDK in master is T1, and the currently published SDK is the T2 release that's being maintained in this branch:
https://github.com/Azure/azure-sdk-for-python/tree/release/v3/sdk/digitaltwins/azure-mgmt-digitaltwins
I'm not sure what the timeframe is for merging the T2 SDK into master.

Thanks for the heads up Anna! Do you think we could just merge the T2 one into master then?
The reason I ask, is you're saying that the currently published SDK is the T2 release, so we wouldn't be losing anything right?
All that being said I know there is a plan that the mgmt folks are working off of, and I don't want to mess with that if it incurs additional effort.

Yeah that would be my preference too - I don't like the master is broken. Though I will leave that to @changlong-liu  :)

Hi @scbedd , @annatisch ,
The master branch code for digitaltwins is old and there are errors.
Digitialtwins is currently released as Track1,  so the latest code is in release/v3 branch.
I tend to delete the digitaltwins code in master branch to avoid this confusing. And adding @msyyc  to know this.

Closing this issue since new digitaltwins has been released.
CC to @msyyc


Interoperability Bounties List

NIP-23 Add long form to RSS readers
by @fiatjaf

If you are a bored developer maybe you will be interested in implementing simple NIP-23 Nostr sources support on some open-source RSS reader apps out there for something like $800 each.

The goal is to make it work, record some screencasts and open PRs. If they will merge or not that's their decision and we don't care.

The feature is just to support pasting npub, nip05 or nprofile URIs, use relay hints if available or fetch write relays from kind:10002, then fetch new kind:30023 articles from these relays periodically just like the app already fetches RSS feeds from URLs. No need to bother with comments, likes, anything for now.

Eligible apps:

[ ] - https://gitlab.com/news-flash/news_flash_gtk
[ ] - https://github.com/Ranchero-Software/NetNewsWire
[ ] - https://github.com/nkanaev/yarr
[ ] - https://github.com/Athou/commafeed
[ ] - https://github.com/RSSNext/Follow
[ ] - https://github.com/samuelclay/NewsBlur
[ ] - https://github.com/miniflux/v2
[ ] - https://github.com/Ashinch/ReadYou
[ ] - https://github.com/stringer-rss/stringer
[ ] - https://github.com/newsboat/newsboat
[ ] - https://github.com/spacecowboy/Feeder
[ ] - https://github.com/msasikanth/twine
[ ] - https://github.com/lwindolf/liferea
[ ] - https://github.com/martinrotter/rssguard
[ ] - https://github.com/nextcloud/news

https://njump.me/nevent1qvzqqqqqqypzqwlsccluhy6xxsr6l9a9uhhxf75g85g8a709tprjcn4e42h053vaqqsqqqqns8wf9ukah2ynpzdxgur30dwzn6er4msrqnge35pzhdqn0ns7t9een

@fiatjaf mentions
"2 other reserved, another 1 interested"


Some minor additions

Maybe adding zinc, because you added brass but not zinc.
And also maybe add crystal as Product type.

Thnaks :D


ServiceBase.GetCharacteristicAsync() throws NullReferenceException

About 1% of users experience a crash when fetching characteristics.
Current workaround is just try catching it and trying once more.
Steps to reproduce

Request characteristics

Expected behavior
Retrieve characteristics
Actual behavior
Application crashes
Crashlog
ServiceBase+<>c__DisplayClass12_0.b__0 (Plugin.BLE.Abstractions.Contracts.ICharacteristic c)
System.NullReferenceException: Object reference not set to an instance of an object
Plugin.BLE.Abstractions
ServiceBase+<>c__DisplayClass12_0.<GetCharacteristicAsync>b__0 (Plugin.BLE.Abstractions.Contracts.ICharacteristic c)

System.Linq
Enumerable.TryGetFirst[TSource] (System.Collections.Generic.IEnumerable`1[T] source, System.Func`2[T,TResult] predicate, System.Boolean& found)

System.Linq
Enumerable.FirstOrDefault[TSource] (System.Collections.Generic.IEnumerable`1[T] source, System.Func`2[T,TResult] predicate)

Plugin.BLE.Abstractions
ServiceBase.GetCharacteristicAsync (System.Guid id)

Configuration
Version of the Plugin: 2.1.3
Platform: Android 10, 11, 12
Device:

HUAWEI P30 lite
P30 Pro
Galaxy S9
Galaxy A50
Mate 20 Pro
Galaxy A22
Galaxy A41
V2050
Galaxy S10e
Redmi Note 8 pro

Hi @TomasMalcanek,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

Some actual code for reproduction would be very helpful.


Update cabal.project

They don't necessarily have to be in sync, but its probably a good idea if they were in sync.
Unfortunately some people prefer stack and some cabal so we need to keep both. Yes it would be good to have a tool to keep these in sync.

bors r+


async fn query should take a reference to a query, not the query instance itself

https://github.com/aprimadi/influxdb2/blob/11b8b9d23568d09672cf8ed85b7d878c0deaa8e4/src/api/query.rs#L100C57-L100C62
I think it's useless for it to take ownership of the query. A reference should work just fine. This way i can lazily initialize my query object and reuse a reference to it over and over.

I think that's a valid point, we'd greatly appreciate a PR which makes that change.
That would be a backwards incompatible API change, which warrants a new minor version (since we're still at 0.x) instead of a patch bump

@Boudewijn26 would it be better to accept an Option<impl Borrow<Query>> to maintain a certain level of compatibility for the API or Option<&Query>?


AutoSizeGroupBuilder widget would be a good convenience feature

Every time someone is new to AutoSizeText, they initially make this mistake:
class MyWidget extends StatelessWidget {
  final _autoSizeGroup = AutoSizeGroup();
  @override Widget build() {
    return Column(
      children: [
        AutoSizeText('hello', autoSizeGroup: _autoSizeGroup),
        AutoSizeText('goodbye', autoSizeGroup: _autoSizeGroup),
      ],
    );
  }
}

The non-obvious mistake being that the AutoSizeGroup will keep being reconstructed any time the widget rebuilds, and lose its state, which can cause flickering and performance issues.
The addition of a standard AutoSizeGroupBuilder to capture the AutoSizeGroup in its own stateful widget would make it much simpler to use it correctly, like:
class MyWidget extends StatelessWidget {
  @override Widget build() {
    return AutoSizeGroupBuilder(
      builder: (_, autoSizeGroup) => Column(
        children: [
          AutoSizeText('hello', autoSizeGroup: autoSizeGroup),
          AutoSizeText('goodbye', autoSizeGroup: autoSizeGroup),
        ],
      ),
    );
  }
}

I think that's a great idea 👍


New package: iFileSpace.iFile version 2.5.5

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Add a gpsctl.conf file for more control

Add a gpsctl.conf file for more configurability.
Rather than multiplying command line options, get options from a .conf file.
This is a work in progress.
The plan is to set program defaults in a [gpsctl] section of gpsctl.conf.  This will always be obeyed.
Other sections will be used to configure the u-blox device only if --configure_for_timing or --galileo are selected.

Current status: --port parameter can now be specified in /etc/gpsctl.conf
The commandline parameter will take precedence, if present.
For example:
[gpsctl]
port = /dev/ttyAMA0

Release 0.7 supports specification of nmea version and satellites via the config file.
Enjoy!

All done - every parameter which was configured in the original now has a corresponding gpsctl.conf entry
For completeness, we probably need and nmea:enabled flag and process as the current -n option

nmea:enabled now implemented
gpsctl:verbosity also implemented
The baud rate stuff can stay on the command line for now, it's simpler and clearer


E2E Test

I am hoping to at least build the driver, create kind and maybe run a few of those pods as a e2e test.
Surprising this works. I don’t really understand where mine came from in the GitHub runner but it is found.

/hold
Test seems flaky.

/hold cancel
added kubectl waits on the pods which may help with flakiness.

/lgtm

/assign @klueska
for approval

Last outstanding item is https://github.com/kubernetes-sigs/dra-example-driver/pull/56#discussion_r1740080606. We can resolve this or we can do it as a follow up. I don't care either way. I'll maybe find some time next week to finish that up.

Let's merge as-is for now since running something is better than running nothing. But let's definitely follow-up on that.
/approve


mitosheet import error in Vs code

Environment data

VS Code version: XXX
Jupyter Extension version (available under the Extensions sidebar): XXX
Python Extension version (available under the Extensions sidebar): XXX
OS (Windows | Mac | Linux distro) and version: XXX
Python and/or Anaconda version: XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Jupyter server running: Local | Remote | N/A

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

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

XXX

Hi @ridhanya99, sorry to hear you're having issues. Could you please fill out the issue template for us to help you?

Thank you for your response..
Error: Module mitosheet3, semver range ^0.3.70 is not registered as a widget module.
Unable to load a compatible version of the widget 'mitosheet3'. Expected behavior may be affected.

@ridhanya99 could you please upload the output from the Jupyter output panel? As well as the Jupyter extension version, the python extension version and your OS (windows, mac, linux)? Also, could you describe what you're trying to do? Are you running a notebook?
Thanks.

Thank you for your reply sir,
I was trying to use mitosheet in vscode jupyter notebook.
import mitosheet
import pandas as pd
mitosheet.sheet()
I faced the following error:
Error: Module mitosheet3, semver range ^0.3.70 is not registered as a
widget module.
Unable to load a compatible version of the widget 'mitosheet3'. Expected
behavior may be affected.
Then I found that it can be run in jupyter lab, so I am currently using it
there..
Thanks,
@ridhanya99
On Wed, Jun 23, 2021 at 6:13 AM David Kutugata @.***>
wrote:

@ridhanya99 https://github.com/ridhanya99 could you please upload the
output from the Jupyter output panel? As well as the Jupyter extension
version, the python extension version and your OS (windows, mac, linux)?
Also, could you describe what you're trying to do? Are you running a
notebook?
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/6336#issuecomment-866434392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOPNEC3VD2QKNXLLPDZ5NTTUEU2DANCNFSM46726IXA
.

Sorry for the trouble @ridhanya99, I'll gladly try to help you resolve this.
Could you upload the output from the Jupyter output panel when the error reproes? That would help a lot.

Dupe of #6320.

Thanks a lot!
On Fri, Jun 25, 2021 at 1:40 AM Jim Griesmer @.***>
wrote:

Closed #6336 https://github.com/microsoft/vscode-jupyter/issues/6336.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/6336#event-4936634061,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOPNEEEU3LQXKTGVCUNQSTTUOGMNANCNFSM46726IXA
.


Add note about EnableNETAnalyzers when NuGet packaged is installed

Contributes to #23010

Looks great with minor comments.
I have to say that I love how the code-analysis docs have evolved and how you manage to make it clear, informative, and to the point. This is really great ❤ 🎉

Thank you for the nice feedback :D


Logical Partition Size

I do not believe under V1 the statements made that:
-there's no upper limit to how many posts a user can create,
-posts can be arbitrarily long,
-there's no upper limit to how many comments and likes a post can have,
are true. If the posts are logically partitioned by postid then there is a limit of 20GB. The same I believe woudl apply to the userid logical grouping as well.

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

ID: 669528a7-38dc-a6a9-f98f-4cbb91fd44e0
Version Independent ID: 5e5fcb52-86e8-7e08-6209-79f9ce5e2a91
Content: Model and partition data on Azure Cosmos DB with a real-world example
Content Source: articles/cosmos-db/how-to-model-partition-example.md
Service: cosmos-db
GitHub Login: @ThomasWeiss
Microsoft Alias: thweiss

@susanlinsfu thanks for your comment. These statements were made to illustrate the choice of embedding vs. referencing. If we embedded in the same doc all comments with their post, we could be easily limited by the max doc size of 2 MB. By referencing (keeping posts and comments in separate docs):

we make sure that each post or comment can have up to 2 MB (which is more than enough in terms of text length)
each post and its comments can occupy up to 20 GB; with an average doc size of 128 KB (again, probably more than enough) this means we could have up to 20 x 1024 x 1024 / 128 = 163,840 comments per post - which is more we would ever need.

Thanks Thomas for such a quick reply. I completely agree, reaching 20GB is this scenario is a long shot, but I just wanted to make aware that there is a limit (either for someone who is making a similar system or mimicking the design pattern for another scenario).  This may be a bad example, but for instance if an application wanted to allow images/videos in comments and the images/videos were stored in base64 encoded format (even though they should probably be stored in a CDN) in an item, the logical partition size may continue to grow quickly.

@susanlinsfu Thanks for the feedback. We will go ahead and close this thread for now.

@susanlinsfu Logical and Physical partition discussions with Cosmos DB are super helpful here as they assist other customers who read the documentation along with these issues to gain more insight than what the doc might be able to convey. As for the topic of embedding content in a document or referencing external content in a hybrid scenario, the following doc is helpful: Data modeling in Azure Cosmos DB
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

@susanlinsfu Storing binary data in Cosmos DB is indeed an anti-pattern; the recommended alternative is to store that type of data (pictures, videos etc.) in Azure Blob Storage and include the link to the file in the document stored in Cosmos DB.


Error updating or installing plugin

Previously working sfdx wry commands are no longer working, and an attempt to update (or install yields  -
sfdx wry:file:replace
›   Warning: wry:file:replace is not a sfdx command.
Did you mean plugins:update? [y/n]: y
warning "sfdx-wry-plugin ><EMAIL_ADDRESS>has unmet peer dependency "@oclif/plugin-help@^2".
sfdx-cli: Updating plugins... done
sfdx commands still not recognized.
Any ideas?
Thanks!

Morning,
I'm having the same issue.
Any thoughts or comments?
Cheers

Good Morning,
I am having the same issue. It worked before, but now it says Warning: wry:file:replace is not a sfdx command. I removed all the references and installed again, still the same issue.
Any suggestions please?
Thanks

Same for me :( Install went OK, attempt to use wry:file:replace results in "not a sfdx command"

Add me to the list.

+1
Has anyone found a solution for this?

pulling source and using sfdx plugins:link sfdx-wry-plugin works.  Not the best but, it works.

Thanks @squattingdog that worked for me

git clone https://github.com/billryoung/sfdx-wry-plugin.git
sfdx plugins:link sfdx-wry-plugin


getGrants of specific role

I have use case similar to https://github.com/onury/accesscontrol/issues/31
I don't  want expose all roles and action to client , just roles and actions that granted
In the following
ac.grant('user').read('profile', ['*', '!id'])
   .grant('admin').extend('user');

{
    "admin": {
        "$extend": ["user"]
    },
    "user": {
        "profile": {
            "read:any": ["*", "!id"]
        }
    }
}

If the client's role is user , the admin role is exposed too

This is not specific to AccessControl.js. You should never expose grants / policies to client side.
(Even if you only expose the policy of the required role and no other role; it's still bad practice.)
You should handle the request and grant/deny access on server side. And if granted; filter data if needed (using permission#filter() if using this lib) and return the response. Client side should have no knowledge of your policies.

I don’t agree that you should not expose permissions. Your clients or users knows what they can do and they can’t do anyway (because you document this somewhere on app website).
For example, on medium anonymous users can’t leave applause and it’s clear for everybody (because you just can’t do this on ui).
So, what’s the difference whether you share permissions as JSON object or as human readable text or as UX?
If you don’t share permissions, your clients will need to duplicate or even hardcode permission logic. Later it will be very hard to change.
I consider it to be safe to share permissions with client for currently logged in user. Some time ago I wrote an article about this:
https://medium.com/dailyjs/casl-and-cancan-permissions-sharing-between-ui-and-api-5f1fa8b4bec

A good restful design requires to return 403 Forbidden status response for actions which are not allowed. So, anyway it’s quite easy to understand or get a list of permissions for a particular user credentials.
Side note: I think the issue with permission sharing in this library exists because it relies on user roles and not on what user can do (e.g., create post, read post, leave comment). If you could share user actions instead of roles, it would not expose any internals of underlying permission logic


RefinementListWidgetOptions.showMore should conditionally render

Bug: What is the current behavior?
RefinementListWidgetOptions.showMore template always renders, regardless of # of facets or RefinementListWidgetOptions.limit.
It should only render when the requested # of facets is greater than RefinementListWidgetOptions.limit (v1 behavior), or some sort of disabled class should be applied so it can be hidden with CSS.
https://community.algolia.com/instantsearch.js/v2/widgets/refinementList.html#struct-RefinementListWidgetOptions-showMore
Looks like it was brought up here, but I'm not seeing it fixed in v2.0.0-beta.2: https://github.com/algolia/instantsearch.js/issues/1455.
This works as expected in v1. The show more block isn't rendered at all if it is not applicable.
What is the version you are using? Always use the latest one before opening a bug issue.
v2.0.0-beta.2

Thanks for the report @timkelty :)
Indeed we're missing an API in the connector that expresses the ability to actually do a toggle of the show more button. On it!

I created a new build with the fix for this issue 2.0.0-beta.3<EMAIL_ADDRESS>


Add support for DNSCrypt

Hi please add support for DNS service for DNSCrypt it will be making VPN even more secure.
https://dnscrypt.org/

Wouldn't this be more relevant? https://github.com/Angristan/OpenVPN-install/issues/5

You'd likely need to add the client to the host which adds complexity to this install which may be out of scope.

Also, as DNScrypt is abandoned

What about now? https://github.com/jedisct1/dnscrypt-proxy last update about 4 days ago or I missing something?
Maybe existing another way for DNS encryption on onw OpenVPN server?

There's also an interesting WIP here: http://odns.cs.princeton.edu/ but it's only at an early stage.

I think that our eventual goal should be to decide if we want this to prevent or at least help eliminate unintentional leakage.  Then maybe publish a section stating what will likely still leak regardless of our efforts here.
DNS queries should be encrypted entirely so that only you and your DNS server knows what's happening there should be no simple path between eavesdroppers and your traffic.  My own goals are to have next level privacy protections without significant cost to my convenience then maybe one day I'll write a blog or something about it.
There is an RFC for this https://tools.ietf.org/html/rfc7858 which may eventually be possible; I think that Quad9 is primed to provide it as part of their service asap if not already.  I could be out of date with this information.

@chamunks as some people aren't aware of, encrypting DNS queries doesn't improve privacy. Why? Because you are relaying on an ISP, or vps provider. So in fact they can't get your browsing history by looking at the dns queries but looking at the traffic port 80/443. From this point of view encrypting dns queries only makes things relatively harder but not impossible.
Again, DNS over TLS only protects queries from being modified by an attacker, kind of DNSSEC.
Regarding quad9, from their faq:

We do support DNS over TLS on port 853 (the standard) using an auth name of dns.quad9.net.

Sounds like DNS over TLS its just signing the queries.  Ideally, you'd want full obfuscation of the DNS queries to third parties.  It's quite frustrating that DNS queries are being used against you these days, but I guess it was inevitable.
Caching your DNS queries through the local unbound process is going to help decrease DNS leaks significantly.  However, you still expose whatever DNS queries are coming out of the VPN machine to wherever you're getting your upstream information.  Ideally, you want the relationship from your device to wherever you're getting the information entirely secured and assured against tampering.

Sounds like DNS over TLS its just signing the queries.

In theory is more than that: a secure channel is established and all the data passed can't be eavesdropped...

Ideally, you'd want full obfuscation of the DNS queries to third parties.

...In practice encrypting dns queries does not prevent from seeing people's browsing history. You can still watching for traffic on ports 80/443 and have all the visited sites ( dns queries encryption / obfuscation is useless here ). So this was what I tried to explain.

True enough thanks for the clarity and yes that's what I'm aware of as well.  The situation, however, is it removes a vector for proving intent.  The encrypted https packet would go a different route as well then the DNS packet so the DNS packet would be encrypted so that you can't see where you're going to be sending packets.  DNS is just a point of concentrated data collection ability.  If you can take steps to make that difficult, you remove a vector.
It is indeed not a 100% fix.  The situation is that basically running your own VPN is to choose exactly where your data egresses and helps decentralize this sort of function.

Ah, I see why you're saying that now.  I stopped using my own ISP's DNS over a decade ago due to reliability issues, but yes you're right all it takes is for someone to sit at my ISP and listen in.
Which is why I personally would love to be able to encrypt my data over the pipe to the DNS provider.  I don't want to have to trust my ISP basically.

DNSCrypt is not necessary at the client-level when no leaks are occurring; DNS requests are forwarded to the VPN server and cannot be intercepted by the clients' ISP.
On the other hand, DNSCrypt could be used at the host-level to add another layer of obfuscation, making it harder for hosting providers from watching/hijacking DNS queries.


[BUG]: drizzle-zod errors with latest version 0.31.0

What version of drizzle-orm are you using?
0.31.0
What version of drizzle-kit are you using?
0.22.1
Describe the Bug
After upgrading to the newest version, all of my schema files are littered with errors:
[IsDrizzleTable] is declared here

Everything still works and compiles as expected, despite the errors.
If found this previous thread, but restarting the TS server does not correct my errors:
https://github.com/drizzle-team/drizzle-orm/issues/1665
Expected behavior
I would expect no errors, or remediation steps to correct them.
Environment & setup
VSCode
Windows

Clearing out all node_modules and package-lock.json files, then running npm i seems to have sorted it out.


yarn app:start:local-api does not use local running launcher-creator-backend

When I use yarn app:start:local-api the list of supported backends are always visible even when the service is down in http://localhost:8081

I see this error when opening the Backend screen:
index.js:1446 Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.
    in DataLoader (created by OpenshiftClusterLoader)
    in OpenshiftClusterLoader (created by Overview)
    in Overview (created by HubOverviewCard)
    in div (created by HubOverviewCard)
    in div (created by GridItem)
    in GridItem (created by HubOverviewCard)
    in HubOverviewCard (created by HubNSpoke)

Apparently it caches things. I click on Applications -> Clear Storage and the problem is  gone


[Bug] List Widget shows Unexpected error

Description
When refreshing the page with a checkbox group within a list widget, the application throws an error.
Steps to reproduce the behaviour:
Add steps to reproduce this behaviour, include console / network logs & screenshots

Add a list widget
Add a checkbox group within the list widget
Refresh the page after converting the options control to JS
See error

Important Details

Version: [Cloud]
OS: MacOS
Browser Chrome
Environment production

No error occurs.
This has been fixed
https://user-images.githubusercontent.com/31691737/170880680-ca634c1e-40ab-4143-a727-ae9c43c0825f.mov


zombienet: sketch of warp-sync test for validators

Sketch of warp-sync test for validators. Not tested.
warp-sync is now failing with babe-skip-epochs, the reason is here: https://github.com/paritytech/substrate/pull/12751#issuecomment-1328032781
Follow-up of: #12769

Dumb question. Can't we merge this test (0002) with 0001 by warp syncing two nodes there?
I.e. by adding a warp sync for a validator there?

Dumb question. Can't we merge this test (0002) with 0001 by warp syncing two nodes there? I.e. by adding a warp sync for a validator in 0001?

For me those scenarios are slightly different, and different failures are observed. I would prefer to keep them separate. Just to summarize:

0001: full nodes only: three synced from db, one warp-synced,
0002: two validators warp-synced,  three full nodes synced from db
0003: validators synced from db, one full node synced from db, full node warp-synced

Yeah. Your are right, better to separate the scenarios.

So as we've discussed in the chat we have 2 issues to resolve.

(test case 0002) : validator fails after being warp synced
(test case 0003) : full node fails after being warp synced

Both the cases are due to epoch skipping feature not handling two edge cases.
Case 3 has been already deeply discussed https://github.com/paritytech/substrate/pull/12751#issuecomment-1328032781
and we need a fix for the gap. I may attempt to relax the gap checks when searching for epoch data.

Case 1.
**TL;DR: the issue here is that the oracle is_major_syncing() returns false (sync state is Idle) but the blocks were not all imported yet by the backend **
Long explanation:
Here the failure is due to a validator trying to build a new block after being warp synced.
This is the execution stack triggering the failure (less depth first)

next_slot() is called by the block authoring worker when is time to attempt block production
create_inherent_data_provider() closure called with the last best block (i.e. the one the sync leaves us with) here
create_uncles_inherent_data_provider() called here
Client::uncles() called here
...
Starting from current best block (0x5448..., i.e. the point where the warp sync positioned us)
we begin collecting uncles here.

(hash=0x5445..., height=12133) => OK   (THIS IS THE BEST BLOCK according to warp sync)
(hash=0x9e94..., height=12132) => OK
(hash=0xaea8..., height=12131) => OK   (THIS IS THE LAST FINALIZED BLOCK according to warp sync)
(hash=0x2e3b..., height=12130) => FAIL (block header not found error)

That is a bit strange since when the oracle returns is_major_syncing() == false the headers
should have been downloaded (i.e. we don't attempt block authoring if we've not downloaded the blocks)
So here
instead of return I've inserted a continue.
And indeed it fails 2 times and then at the third attempt it works.
If we print the logs of babe we see that when the next_slot() is invoked the block_import for old blocks has not finished to
import all the blocks yet.
When all the blocks are imported, 0x2e3b is now present.
(Note that the blocks from last finalized (12131) to best (12133) are imported first... then the import begins from 1, 2 ,3 ...)
To wrap-up, looks like in the next_slot() it is not sufficient to check if the network has finished downloading the blocks. We also need to check if we imported the one below the last finalized.

Test case 3 (https://github.com/paritytech/substrate/pull/13078#issuecomment-1374021848) should be fixed by https://github.com/paritytech/substrate/pull/13106

bot rebase

Test case 2 (https://github.com/paritytech/substrate/pull/13078#issuecomment-1374021848) should be fixed by https://github.com/paritytech/substrate/pull/13155

That is a bit strange since when the oracle returns is_major_syncing() == false the headers
should have been downloaded (i.e. we don't attempt block authoring if we've not downloaded the blocks)

Dumb question: why cannot is_major_syncing be changed to reflect the state of import?

bot rebase

Requires https://github.com/paritytech/substrate/pull/13135

bot rebase

Test are currently not passing due to: https://github.com/paritytech/zombienet/issues/700


Update android_lib_build.sh

Two small fixes:

The Android directory in the repo starts with a capital-A. This changes the path definition for ARM64 and ARMv7a to match, so they will succeed on case-sensitive filesystems
My script was failing because I didn't have an Android/app/libs directory. This Adds the -p flag to mkdir, so that intermediate directories are created automatically if they don't exist

Thank you :)


Audio input animation?

Is it possible to get some sort of stream or indication of input volume while recording to be able to generate an animation similar to the one below?  Often it is helpful for the user to be able to see the animation correlate to their voice to know that everything is working as it should.  Thanks!

@n8stowell82  Have you work on this feature?

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as "For Discussion" or "Good first issue" and I will leave it open. Thank you for your contributions.

I think this falls under For Discussion. While not a bug, I would like to see this feature added.


Auto Size Reduce Lity window as per window resize

Is there is any way to resize automatically lity window as per window resize, seems here size of lity frame is fixed..

how to resize iframe of lity as per window resize?

iFrame are only resizing its width at the moment.


"Notify für folgende Rufnummern aktivieren" is ignored for outgoing calls

Usecase:
I use Zammad with placetel control notify api to show my caller log in Zammad. I have many phone devices in my company and 2 phone devices in my support department. I the caller log I only want to see my 2 support related phone devices (because I dont want to see sales activity an other stuff).
Problem:
I only activated the 2 support phone numbers in the placetel admin interface ->
Weitere Einstellungen -> APIs -> externe APIs -> Notify für folgende Rufnummern aktivieren
But placetel always pushs all putbound call informations of any phone device (even all sales acitvity) to Zammad.
Solution:
Only push call informations for checked numbers in "Notify für folgende Rufnummern aktivieren"

For context, here's a screencast that actually shows the issue we believe to have found:

Incoming data is highlighted (as soon as it arrives) on the UI for the ease of recognition.
During the first try, you'll see us calling out to any number without selected extensions 0* and 00*.
During the second try you'll see how the Notify-API behaves when having selected the extensions 0* and 00*.
* Please note: Extension 0 is configured as default outgoing number - the caller is part of this configuration (and also member of the call group that's assigned to it).
99 is the fax extension which we ignored (but shouldn't really count).

Hi @MrGeneration and @martini
right now, this is the intended behavior and stated in the documentation:

Currently there are notifications for all outgoing calls.

We will discuss our options here and I will come back to you!

@MrGeneration we made the following change: Outgoing calls are only notified if the caller ID of the used SIP user is set and the used number is enabled for notifications.

Amazing news - thank you very much for your effort! :)


updated QML intro based on Alex's edits

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/libraries/machine-learning/intro.md
75
52
2
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 1bbe69f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/user-guide/libraries/machine-learning/intro.md
:bulb:Suggestion
View
Details

articles/user-guide/libraries/machine-learning/intro.md

Line 2, Column 8: [Suggestion-duplicate-titles] Title 'Quantum machine learning library - Microsoft Quantum' is duplicated in: 'user-guide/libraries/machine-learning/glossary-qml.md(2,8), user-guide/libraries/machine-learning/intro.md(2,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/libraries/machine-learning/intro.md
75
54
2
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 4b2a107:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/user-guide/libraries/machine-learning/intro.md
:bulb:Suggestion
View
Details

articles/user-guide/libraries/machine-learning/intro.md

Line 2, Column 8: [Suggestion-duplicate-titles] Title 'Quantum machine learning library - Microsoft Quantum' is duplicated in: 'user-guide/libraries/machine-learning/glossary-qml.md(2,8), user-guide/libraries/machine-learning/intro.md(2,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/user-guide/libraries/machine-learning/intro.md
75
54
2
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 75ffafb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/user-guide/libraries/machine-learning/intro.md
:bulb:Suggestion
View
Details

articles/user-guide/libraries/machine-learning/intro.md

Line 2, Column 8: [Suggestion-duplicate-titles] Title 'Quantum machine learning library - Microsoft Quantum' is duplicated in: 'user-guide/libraries/machine-learning/glossary-qml.md(2,8), user-guide/libraries/machine-learning/intro.md(2,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Feat: Adding 404 not found page

here we can add some gits or some pics to show the not found route

@sunny0625 can i work upon it


Add view json download

Checked for duplicates
Yes - I've already checked
Alternatives considered
No - I haven't considered
Related problems
I'm frustrated when [...] happens as documented in issue-XYZ
Describe the feature request
Since there is a button to upload a view file there should also be one to download it. I have created some useful views using the UI but it is a pain to retrieve them through the graphql interface

Views can be downloaded from the Saved Views modal:

Open up the Saved Views modal

Click on the download icon

That said we may want to offer a "Download View" in the main view menu since this functionality is a bit hidden otherwise?

I think adding the option to the modal would be good


Error in example

In the example:
sds s1 = sdsnew("abcd");
sds s2 = sdsempty();
s[1] = 1;
s[2] = 2;
s[3] = '\n';
s2 = sdscatrepr(s2,s1,sdslen(s1));
printf("%s\n", s2);
output> "a\x01\x02\n"
We are editing indices of s but that is not declared, s should be s1.

See pull request #108 for a fix.


Best-effort fix of non-functioning Properties menu

Describe your changes
PR turns Relations > Properties  menu into a page that visualises the types of relationships possible/allowed between entity classes.
Some code comments suggest there are plans to rework how Triples are handled, so as not to interfere with that, no attempt at consolidating code to handle Triples and Properties was made; instead, the behaviour for Triples was mimicked for Properties.
Summary:

create new class PropertyFilter
create new class PropertyTable with columns Subject, Predicate, Object, Reverse predicate
add attributes to tables (requires fixing of templates to come into effect)
explicitly specify fields/table columns to initially sort by in views
add "id" column to Triple view (which is what Triple tables are default sorted by anyway, but isn't a sort order users can currently go back to once they sort by a different column)
reformat files with black, clean up imports

Screenshot of resulting table

Caveats
Relationships with more than one subject or more than one object result in duplicate rows even though each row already lists all subjects/objects. Attempts on my part to split them out into separate rows weren't successful.

Counting of rows also is buggy/seems affected negatively by the same underlying issue: in the app I work on, sorting by Subject yields a different number (18) than sorting by Object (21). Some rows also appear to be missing when sorting in one direction, while others are missing when sorting in the other.
This solution is probably still better than not having a functioning Property page at all, so I'd suggest merging the PR and making sure everyone's aware of these bugs. (Hence: "best-effort fix".)
Related issues and PRs
Resolves issue:

#23

Checklist (Replace the space in square brackets with a lowercase x for all that apply)

[x] My changes don't generate new warnings or errors
[x] My changes follow the project's code formatting rules and style guidelines
[x] I have commented my code with Docstrings and code comments, particularly complex, unusual or hard-to-read code
[ ] I have updated the project documentation to reflect the changes I introduce
[ ] I have added new unit tests or updated existing ones to demonstrate my changes works

Actually, I forgot about something that I'd stashed away, so will set this PR to Draft for another day to revise it.

Ok, updated, incl. with Docstrings & code comments that had been stashed away, now ready to merge.
To expand on earlier notes:
I haven't been able to figure out how to get at the verbose names of entities and sort them alphabetically. My research suggests there's a way to retrieve more info about a model used in a ManyToMany field (subj_class, obj_class in Property) by using a through table, but that is not compatibel with limit_choices_to. + The  Property class is also used to build triples, and I didn't know how messing with the fields in the class would/could potentially affect those.
I'm also not familiar enough with Django Filters to know if those would make it possible to get at the verbose names (hence also why PropertyFilter is a 1:1 copy of TripleFilter). Same goes for the filter attribute of the django_tables2 ManyToManyColumns, obviously.
This revision now results in the correct number of rows being output for the properties themselves, in any case.

Looks good. The triples and properties table are certainly an improvement. Thank you!


there's a bug in app.js

when I define a variable called users and set it to an object in app.js the whole app blows up.
what gives?

Hi! I've never heard of such a thing. Can you provide a copy of the app.js that blows up and what "blows up" means? I assume it means there is an error, and if so, please paste what that is as well.

Hello.
I think, what s/he was trying to say is defining a variable named users will conflict with the one you use for routers.
app.js : line 9:
var users = require('./routes/users');
app.js : line 26
app.use('/users', users);
If you define another thing with the same name, it'll 'blow up' at line 26. And that's totally normal because of the nature.

Thanks, @gokaygurcan ! I'm going to close this, but @lifeiscontent , I can re-open if you can provide further explanation on the nature of your issue :)


[Bug] Fix a bug that when check whether a view is hidden or user interaction not enabled.

When check whether a view is hidden or user interaction not enabled, we should check its super view recursively.

Thank you!

You are welcome. This is a special case I encountered in my project. So I think maybe others will also need this.


docs: Add docs for LightGBM execution mode

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
80
93
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Codecov Report

Merging #1779 (06a7c95) into master (8dc4a58) will decrease coverage by 79.56%.
The diff coverage is n/a.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
80
93
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
97
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
97
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/lightgbm/about.md
98
100
93
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx


fix: add unsupported status condition for filters within BackendRef

What type of PR is this?

highlight unsupported BackendRef filters using a status condition
introduce BackendRefContext in translator

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes

https://github.com/envoyproxy/gateway/issues/2572

/retest

/retest

/retest

sure, found the context in https://github.com/envoyproxy/gateway/issues/1252

/retest

/retest

/retest


Add dist builds

This project uses semver, please check the scope of this pr:

[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Added dist builds.

Changes Unknown when pulling b1d78d2d66dc9f22ac8cfce75a3ab3512e9132e5 on sandersky:master into ** on ciena-blueplanet:master**.

Changes Unknown when pulling 136de144a4d353bf2c2712f6071cd52dbbbb65ea on sandersky:master into ** on ciena-blueplanet:master**.


char_lookup failing on zKill keys

requested character info for the user Pech Tor
 [15/06/2018 08:34:40] ERROR events on_command_error 100: CommandInvokeError
 Traceback (most recent call last):
   File "/firetail/firetail/extensions/char_lookup/char_lookup.py", line 75, in _char
     danger_ratio = zkill_stats['dangerRatio']
 KeyError: 'dangerRatio'

 During handling of the above exception, another exception occurred:

 Traceback (most recent call last):
   File "/usr/local/lib/python3.6/site-packages/discord/ext/commands/core.py", line 60, in wrapped
     ret = await coro(*args, **kwargs)
   File "/firetail/firetail/extensions/char_lookup/char_lookup.py", line 165, in _char
     await ctx.message.delete()
   File "/usr/local/lib/python3.6/site-packages/discord/message.py", line 545, in delete
     await self._state.http.delete_message(self.channel.id, self.id)
   File "/usr/local/lib/python3.6/site-packages/discord/http.py", line 210, in request
     raise Forbidden(r, data)
 discord.errors.Forbidden: FORBIDDEN (status code: 403): Missing Permissions

char_lookup extension needs to guard more against zKill returning out of spec results. Command also seems to take quite long to process, and occasionally produces double output. I'll look into submitting a PR to correct this.

char lookup is slow, I need to debug it a bit and figure out where the slowdown is occuring.


⚠ Fortnite has degraded performance

In db2ba08, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 4483 ms

Resolved: Fortnite performance has improved in 3c838af.


Fix JsonServiceRegistryDao serialization/deserialization of the salt

Closes #1339

Perfect. :+1:

I am assuming this should be backported to 4.1 as well. Right?

Yes, @JsonIgnore will also probably need to be added to the deprecated setSalt method here: https://github.com/Jasig/cas/blob/4.1.x/cas-server-core/src/main/java/org/jasig/cas/authentication/principal/ShibbolethCompatiblePersistentIdGenerator.java#L81

Sure thing. I'll cherry-pick the commit back once it's merged. Thanks again.


Some small fixes

It seems the style like _application/config/mimes.php* does not work.
See http://www.codeigniter.com/user_guide/installation/upgrade_300.html#step-3-replace-config-mimes-php

It's supposed to be italic, not bold ... Please change that to a single asterisk and revert the rest of the changes.

Oh, sorry. I changed to a single asterisk.
And I found that a line break is needed between two file paths:
http://www.codeigniter.com/user_guide/installation/upgrade_300.html#step-5-move-your-log-class-overrides-or-extensions

revert the rest of the changes.

Where is wrong?

What's wrong is that you're fixing things that aren't broken ... This is far from your first PR, you should know, at the very least, not to batch multiple changes in a single patch. Here's why:

While an acceptable change, the lack of a full stop at the end of one sentence isn't necessarily wrong.
Changing "in order to avoid backwards compatibility" to "for backward compatibility" is good, but I insist on the "s". :)
Changing "any character" to "any characters" is wrong - "any" for singular, "all" for plural.

If you've found that a linebreak is missing, why are you converting that thing to a list instead of adding the linebreak? While it is obviously a list of a kind, we use different indentation level for bullet-point lists (which this was not intended to be in the first place) and there are at least two similar instances that are now inconsistent with this one (but admittedly also broken, due to missing linebreaks).
It's not that big of a deal - I'm sure you'll now fix these issues. However, your description only says that you're addressing the _foo/bar* thing and if you really only did that, it would've been merged by now and we wouldn't have to now go back and forth in a conversation that neither of us enjoys.

Okay, I posted another PR #4411 only containing changes formatting of paths.
I close this PR, but I still have something to fix.

If you've found that a linebreak is missing, why are you converting that thing to a list instead of adding the linebreak?

Because I don't know how to do so. I searched it but I was not able to find how to put linebreak in RST.

we use different indentation level for bullet-point lists (which this was not intended to be in the first place) and there are at least two similar instances that are now inconsistent with this one

I don't understand the format you want. Do you mean to add one more tab?

If you've found that a linebreak is missing, why are you converting that thing to a list instead of adding the linebreak?

Because I don't know how to do so. I searched it but I was not able to find how to put linebreak in RST.

I just found this: http://docutils.sourceforge.net/FAQ.html#how-to-indicate-a-line-break-or-a-significant-newline
But as a general rule, if you don't know how to fix something - just note that it is broken and leave it for someone else to fix. Treat it as real programming code - completely changing it is likely not welcome.
In this particular case, I don't mind switching it to a bullet-point list, but make it consistent and do that for other lists in the document as well. :) Although, changing it to a literal code block may be more readable.

we use different indentation level for bullet-point lists (which this was not intended to be in the first place) and there are at least two similar instances that are now inconsistent with this one

I don't understand the format you want. Do you mean to add one more tab?

No ... See "Step 6", which uses a bullet-point list already - the dashes are indented by just 2 spaces further.


chore: Added appId and orgId parameters to git analytics events

Description
This PR adds two additional parameters to the analytics events for git - appId and orgId
PR fixes following issue(s)
Fixes #23347
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test sha=9ade04f


typo in image name

In docker-compose.yml and README.md, hw_server:2021.2 -> hw_server:v2021.2 (v omitted)
In build.sh, --tag ghcr.io/stv0g/hw_server:${VIVADO_VERSION} -> --tag ghcr.io/stv0g/hw_server:v${VIVADO_VERSION}  (v omitted)
In docker-compose.yml, aptman/qus -> aptman/qus:latest (error without tag in docker 20.10.5)
In README.md, ghcr.io/sst/hw_server -> ghcr.io/stv0g/hw_server (sst to stv0g)
In Dockerfile, cd $(basename ${VIVADO_TAR_FILE} -> cd /xilinx/$(basename ${VIVADO_TAR_FILE} (/xilinx/ omitted)
I wannt to pull requests, but but due to company security policy, I can't :(

I wannt to pull requests, but but due to company security policy, I can't :(

Okay, please tell your company that your security policy is shit if you can contribute to tools and projects which you use.


otel: instrument observability (metrics + traces) on the database

From https://github.com/TBD54566975/ftl/issues/2194#issuecomment-2274154832:
Immediate Needs
For metrics in DD, the well-lit path appears to be DD's official Postgres integration. That comes with not only all the core metrics we want, but also a nice dashboard. For immediate use cases, this is definitely the best solution. Next step on that is talking to @bradleydwyer
If we cannot do that^, then we'll need to instrument custom metrics right now, which is possible but not ideal, both because of the implementation time and also because of the added complexity to the sql code (much of which is generated by sqlc, so we can't just do whatever we want to make the interface nice/intuitive).
Long Term
That said, FTL is not tied to DD, so in the long run, we’ll likely want some integration for this outside of the DD world as well. The first reasonable avenue for this was otel's zero code instrumentation for Go. Unfortunately, that integration only supports traces right now, not metrics. The whole project is marked as a work in progress, but there doesn't appear to be any immediate plans to add metric support.
The urgency for supporting non-DD database metrics is fairly low. Our current customer team and next customer team are both internal, so they should both use DD. Most likely, the next one after that will be as well. We will want to lift this instrumentation out of the DD-only world before significant amounts of open source development, but perhaps otel will also provide better built-ins for instrumenting database metrics by then. This seems like something we can afford to wait for.
Parent doc link for zero-code: https://opentelemetry.io/docs/concepts/instrumentation/zero-code/

A simpler path is probably to switch to using the standard library's sql API and use OTEL's instrumentation for that.
We're currently using sqlc's pgx5 driver because it supports bulk inserts (IIRC?), but we can likely work around that.

Niiiiice that looks fantastic! From some brief grepping, I couldn't find any INSERTs in the current code that would cause issues, and it looks like there are plenty of ways to COPY without relying on sqlc's :copyfrom (which we don't even use now).
It seems like the biggest thing we'd be losing is the codegen. What do you think would be the best way to move forward with the existing queries.sql files, if we went down that route? Start manually modifying the files that were generated by sqlc, and delete the queries.sql files they were generated from?

Notes from standup: we don't need to remove sqlc, just switch out the driver. :)

reopening until we can unrevert the revert


Use In-cluster Devfile registries for integration tests

/kind user-story
User Story
As an odo developer, I want to use stable Devfile registries (but not the default Production one) for the integration tests So that any changes to Staging Registry (what we're currently using) do not break or block us from continuing odo development.
Acceptance Criteria

[ ] It should setup an in-cluster Devfile registry. See https://odo.dev/blog/deploying-and-using-in-cluster-devfile-registry#deploying-a-devfile-registry-in-the-cluster
[ ] Tests should be configured such that they use the staging registry by default, because it is still great that odo can detect and report such issues to the Devfile Services team.
[ ] But if the tests fail due to the Staging Registry, it should fallback to using in-cluster registry. This can be done quite easily now that odo can automatically detect and use registries declared in the cluster. See https://odo.dev/blog/deploying-and-using-in-cluster-devfile-registry#declaring-the-devfile-registry

Links

Related Epic (mandatory):

/kind user-story

We have succesfully installed a Devfile Registry on the Kubernetes cluster on IBM Cloud.
The only tag available for images is next, which is the one where the problem happens.
We have been able to use a previous image saved by Parthvi but which is customized for specific tests, we will need to build an image from original sources.
The PR #6705 makes the tests accept a DEVFILE_REGISTRY env variable to point to this specific devfile registry; this env variable has been added to all CI jobs.

Marked as release-blocker as the tests are failing on all PRs and all nightlies due to errors on Staging Devfile Registry