buttons nested inside of links

Description
The links to different social profiles are buttons wrapped inside of links which is not good semantically. Ideally they are just links and then they get styled according to the design. Alternatively they are buttons but given an aria role of link so that they are still accessible for screen readers.
Screenshots

Additional information
No response

No, the buttons don't appear to have an onclick handler currently
On Tue, 9 Nov 2021 at 15:52, Krish Gupta @.***> wrote:

I had a question do we still use the handle link function to send people
to sites?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/LinkFree/issues/649#issuecomment-964224812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANTHELR36V7LGNULNYVK73LULEYUVANCNFSM5HVKPHSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I can look into this, if no one else is.

For sure!


Fix subtuple check when replaying events

@Kelvin-M @inalgnu This appears to have broken the e2e tests. For some reason, the tuple is sometimes not found by find_training_step_tuple_from_key.
I fixed it in multi-channel by wrapping it in a try/except/pass, but there's probably a better way to fix it.

It may "break" e2e tests when launching them concurrently. Indeed, I think, when multiple jobs call the backend-server (and consequently invoke the ledger), blocks are not already received. And we the backend-worker launches a train/test after receiving an event, it may fail as the subtuple does not exist at this moment.
We can conclude that this tuple is new, and ignore the exception as it surely a 'todo' tuple

@AurelienGasser Can you add it as a fix for master branch ? :)


Allow invalid forward refs for the sake of future specification

Closes #5477 — @commonism is this the change you want?

@commonism the python 3.11 issue seems to be a bigger issue than I realized, because it seems to be actually evaluating the forward ref while building the model (as there is something in the namespace that actually resolves to that value).
I'm going back to leaning toward — I'm not sure this is sensible. I'm thinking maybe the right thing to do is for you to somehow "namespace" the forward refs more carefully to make sure they can't collide with field names.. looking into it more..

@commonism I don't see a good way forward here, so I'm going to close this for now, though could reopen or consider another PR to resolve this later.
That said, I do see a potential workaround for you that I'll comment on your original issue.


Feature/improve tests

Basically consolidates all the common middleware test server functionality into a common base class, which makes the tests a lot cleaner and consistent. Killed a few build warnings relating to the tests. Also adds a bit more test coverage, so I've upped the minimum coverage threshold.

Not sure why the build is failing in AppVeyor... maybe an issue with Appveyor or Cake.

Looks like a nuget issue on appveyor... https://appveyor.statuspage.io/

@binarymash thanks man, I'm on it!

Lets wait a wee bit before merging, see if appveyor fix things.

@binarymash building now! Thanks for the help!!!


Get build green by fixing Clerk

So seems like for the preview env in Vercel we need to use the env vars for Clerk Dev. Right now we are using the production API keys for both preview and Prod env which is why we are getting the 400 error message when trying to sign in.
Here's how to fix:
In clerk make sure you are in Development env
Copy these two API keys that should start with sk_test:
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=sk_test
CLERK_SECRET_KEY=sk_tests

Navigate to Shortest project in Vercel and update these env vars. Make sure only preview env is update for this.

I believe this should fix the issue we have with Clerk. Please let me know if you any help with this.

Sweet. We should also update the README so that anti-work devs can pull the dev keys and not get any errors locally

Updated the docs. Just an FYI I do not have permission to change Env vars in Vercel.

Updated the keys.


Edits to book #5

Minor suggestion: I'd write to "to ensure that these entities can't
[...]" to keep this part consistent with the "or that they're
completely [...]" part, which comes later in the same phrase.

Agreed.

This requires rephrasing: "the the compiler flags the use of [...]"

Duplicated word deleted.

Editorial comment: newlines in the ":ada:" role usually cause issues
when rendering the ReST file to its final format. At least, I had
this issue when using pandoc to generate a PDF file: the ":ada:"
roles containing newlines were not rendered correctly. Therefore,
I'd suggest to move this to the next line.

Good point.

Editorial comment: again, please avoid newlines within the ":ada:" role.

Done and I checked all of them.  This happened due to emacs autofill.

+Some intermediate assertions may not be proved by GNATprove
either because +it's missing some information or because the
amount of information +available is confusing to it. You can
verify these remaining assertions by
Editorial suggestion: because the phrase is long, I'd change
"confusing to it" to "confusing to GNATprove".

I just deleted "to it".  I think that's implied and makes the sentence
even easier to read.

 :ada'Old:

(misplaced colon at the end)

Fixed.

+represent this value and writing an assertion in both branches of an
+:ada:if statement that repeat the postcondition, but using X_Init.
*repeats

Got it.

+don't create new elements, that is, that all the elements present
+in A
*doesn't create

Done.

Editorial suggestion: the phrase starting with "This property isn't
[...]" is very long. It's also difficult to understand without
reading it twice (in my opinion, of course), so I'd suggest
splitting it into two phrases.

Yup.  Done.

+In the example below, calling Prove_P with X as an
+operand only +adds P (X) to the proof context instead of the
+larger set of assertions +required to verify it. In addition, the
+proof of P need only be done
Question: do we say "need only" or should it be "needs only"?

"need only": the proof need only

*let's look

Done.  And I checked for more.

+This is correct: Get_Model is used only in contracts.  Calls to
+Get_Model makes copies of the buffer's contents, which isn't
+efficient,
*make

Yup.


Update colourscheme

Codecov Report

Merging #71 (c50379d) into master (ed3b91c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #71   +/-   ##
=========================================
  Coverage     75.04%   75.04%
  Complexity      524      524
=========================================
  Files            80       80
  Lines          1643     1643
  Branches        189      189
=========================================
  Hits           1233     1233
  Misses          350      350
  Partials         60       60

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 ed3b91c...c50379d. Read the comment docs.


getKeys and getValues function for map type

Is your feature request related to a problem? Please describe.
I have a nested json that contains attributes field of type map. I want to transform that map into keys and values array.
ex.,
{
   "attributes":{
        "field1":"value1",
        "field2":"value2"
    }
}

can be processed and below values can be extracted from it
   "keys":["field1","field2"]
   "values":["value1","value2"]

Describe the solution you'd like
Expose getKeys and getValues function in Expression
Describe alternatives you've considered (Optional)
No alternatives found.
Additional context
Add any other context or screenshots about the feature request here.

This processor combination can achieve the above transformation:
  processor:
    - map_to_list:
        source: "attributes"
        target: "attributes-list"
        key_name: "keys"
        value_name: "values"
    - list_to_map:
        source: "attributes-list"
        use_source_key: true
        extract_value: true

map_to_list generates a list of objects separating keys and values:
{
  "attributes-list": [
    {
      "keys": "field1",
      "values": "value1"
    },
    {
      "keys": "field2",
      "values": "value2"
    }
  ]
}

then list_to_map gets you keys and values in a map:
{
    "keys":["field1","field2"],
   "values":["value1","value2"]
}

References on these two processors:
https://opensearch.org/docs/latest/data-prepper/pipelines/configuration/processors/map-to-list/
https://opensearch.org/docs/latest/data-prepper/pipelines/configuration/processors/list-to-map/

@oeyh Thank you 👍 It works out.


Rebase and resubmit PR for the lio iscsi procfs module

replace the old PR https://github.com/prometheus/procfs/pull/88

@pgier hi Paul, I try to rebase and merge with too much conflict. So I just create a new PR which is faster with all the remark update you send me from the #88 Thanks.
I don't think I can add you as the reviewer.

@pgier do you I should rebase squash all my commit?

@pgier thank you so much for all the comments, I'm looking forward to fix the lio module in node_exporter as well. Let me know if there is anything I should clean up.

Also, would probably be good to squash this down to one or just a few commits since the commit history is becoming pretty large.

@pgier I squash it down, should I change the comments? I'm worry it maybe too much as well. Maybe I can merge back to upstream master when the review is done?

Yes, would be good to clean up the commit comment as well.

Apologies, I'm busy for a couple of weeks and probably won't have time to review.


Select I2C and ADC libraries

We have 5 I2C sensors and 3 analog sensors on our sensor list. We are unsure how to approach finding and deciding which libraries to use.

Closing as done


group it!

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Summary
is that possible to add a category, so that makes management easy, I tried to add a category with / or -, and it works, but the problem is the filter function is not friendly, it will ignore / and -, thanks!
Motivation
makes management easy, sorry I do not know coding
Describe the solution
add a new column named category.

Every now and then, people request "tags" or "categories". See issue # 2610, 1922, 1763, 675, 13, and more.
Short answer : yes, some day.


for windows.

use NUL instead of /dev/null
don't call fork() on windows.

pager won't work on windows. but it's better than not working. :)

I've addressed both issues but in a manner somewhat different. I've kept all the logic inside Hub::Context rather than repeating it in commands.rb or Rakefile.


Error in async_http_request: 'str' object has no attribute 'get'

Hi, I'm trying to replace Cloud integration  with this one (LAN). I'm using the last one integration v2.0.1 on HASSIO.
Just for your info I still have installaed Meross Cloud IoT integration.
After added my two devices MSG100 cover garage I checked in the registry and there is this error:
Logger: custom_components.meross_lan
Source: custom_components/meross_lan/logger.py:29
Integration: Meross LAN (documentation, issues)
First occurred: 11:08:41 (2 occurrences)
Last logged: 11:08:41
MerossDevice(2008204790056336100448e1e92ceb3d) error in async_http_request: 'str' object has no attribute 'get'
MerossDevice(2008206126994936100448e1e92ce8b8) error in async_http_request: 'str' object has no attribute 'get'

Hello @krahabb , please could you leave a comment related to your action: closed? It will be appreciated

Hello @scannifn
Very precious hint here since I cannot really test the garagedoor since the code is just 'guessed'
This  error is of course related to a lack of safety checks when parsing responses and I'm already working on making it more robust but I would need some more logging in order to better sort it out.
We'll continue this topic on #29 if you're up to.
To enable debug logging please follow this one: https://github.com/krahabb/meross_lan/issues/12#issuecomment-846390799

I thought I was able to write before the GitHub UI close...

Like I said we'll contine on #29 in order to have a consistent flow :)

I'm really sorry!! I was too fast... whatelse can I say: SCUSA!
Ok, we can continue to troubleshooting on #29.
Thanks!
PS I f you need to have more details you are welcome!!! :)

FIGURATI!!!
I'm italian too :) You were very fast in replying but it was me dumb closing that way!
See you


Fixed permission issue with oradata

A recent change has introduced volumes to docker-oracle-xe-11g that broke the initialization of the oracle instance.
This is due to a defect with the recursive copy of the original oradata: cp -r does not retain the original attributes of the files such as owner and group. When oracle initializes, it doesn't have permission to access the oradata. Instead of cp -r, cp -a does the same as cp -r but preserves the attributes. This effectively retains the permission settings of the original oradata.
Thank you Wei-Ming for posting this. :) It's been quite valuable for prototyping some stuff.

Thanks for the fix!


Warn if added kubeconfig contains relative paths

Users are having issues on connecting to cluster if Kubeconfig (namely exec) contains non-absolute binary paths. Some shell environments might add something to PATH and that is not available when app is started from app menu. Lens could analyse added kubeconfigs and warn users if relative paths are found in Kubeconfig.

Fixed in #1013.


Add title case state name column to table

Is your feature request related to a problem? Please describe.
We now need full text state name
Describe the solution you'd like
I need a CSV with state name in title case for all rows
Describe alternatives you've considered
using uszipcode
Additional context
n/a

capitalized all state names


Create built-in Type-safe Mappers to facilitate the Result/Output params mapping (depends on 3rd party)

Create annotation @AutoMapper that will depend on optional dependency (mostly commons-dbutils) and do the typesafe auto mapping from db to user object
If the use need to use this functions, then he will need to import the 3rd part library (using mvn, gradle, etc..) and then use the Auto Mapping feature

Reading javadoc here and here and asking a question on SO trying to understand what if I use getObject(int) to just do the mapping..


Add unified binary proposal

cc @zhouqiang-cl
This is a proposal to unify the TiKV binaries into a single tikv binary. This would be part of a major version change since it would have some backwards incompatibilities.

@zhangjinpeng1987 @AndreMouche @siddontang @winkyao @nolouch How did you think about this proposal

Friendly ping

Ping @zhangjinpeng1987 @AndreMouche @siddontang @winkyao @nolouch

I prefer sub command like tikv-server ctl xxxx and tikv-server server xxx.

@zhangjinpeng1987 So you'd prever to see tikv ctl xxx instead of tikv xxx, so that the tikv command can only be called with a subcommand?

@Hoverbear Yes.

@zhangjinpeng1987 That's definitely a great option, let's see if others have some opinion on this?
The original reason I suggested this is for two reasons:

It's not idiomatic to start daemons and services like TiKV directly from command line. They should be run via something like systemctl enable tikv --now or rc update add tikv or otherwise using an init system. So most users would very rarely, call tikv server, the majority of the time they would likely call tikv ctl.
This would create a big complexity difference between tikv server, which just accepts some flags, and tikv ctl which has many subcommands and options. The downside of this is that tikv server kind of gets lost in the subcommands, but as noted, most users should be using provides init service scripts.

Here's an interaction with ctl as a separate command:
$ tikv --help
TiKV (tikv)
Release Version:   4.0.1-alpha
Git Commit Hash:   e54b2eb06d1560b64cf23160f28fa74ff7a394ee
Git Commit Branch: rpm
UTC Build Time:    2019-12-13 08:44:16
Rust Version:      rustc 1.39.0-nightly (c6e9c76c5 2019-09-04)

A distributed transactional key-value store.

USAGE:
    tikv [FLAGS] [OPTIONS] [SUBCOMMAND]

FLAGS:
    -h, --help                    Prints help information
    -V, --version                 Prints version information

SUBCOMMANDS:
    ctl                           Interact with a TiKV instance.
    server                        Begin a TiKV Server Daemon.

---

$ tikv ctl --help
TiKV Control (tikv)
Release Version:   4.0.1-alpha
Git Commit Hash:   e54b2eb06d1560b64cf23160f28fa74ff7a394ee
Git Commit Branch: rpm
UTC Build Time:    2019-12-13 08:44:16
Rust Version:      rustc 1.39.0-nightly (c6e9c76c5 2019-09-04)

A distributed transactional key-value store.

USAGE:
    tikv ctl [FLAGS] [OPTIONS] [SUBCOMMAND]

FLAGS:
    -h, --help                    Prints help information
        --skip-paranoid-checks    Skip paranoid checks when open rocksdb
    -V, --version                 Prints version information

OPTIONS:
        --ca-path <ca_path>              Set the CA certificate path
        --cert-path <cert_path>          Set the certificate path
        --cipher-file <cipher_file>      set cipher file path
        --config <config>                Set the config for rocksdb
        --db <db>                        Set the rocksdb path
        --decode <decode>                Decode a key in escaped format
        --encode <encode>                Encode a key in escaped format
        --to-hex <escaped-to-hex>        Convert an escaped key to hex key
        --to-escaped <hex-to-escaped>    Convert a hex key to escaped key
        --host <host>                    Set the remote host
        --key-path <key_path>            Set the private key path
        --pd <pd>                        Set the address of pd
        --raftdb <raftdb>                Set the raft rocksdb path

SUBCOMMANDS:
    bad-regions           Get all regions with corrupt raft
    cluster               Print the cluster id
    compact               Compact a column family in a specified range
    compact-cluster       Compact the whole cluster in a specified range in one or more column families
    consistency-check     Force a consistency-check for a specified region
    diff                  Calculate difference of region keys from different dbs
    dump-snap-meta        Dump snapshot meta file
    fail                  Inject failures to TiKV and recovery
    help                  Prints this message or the help of the given subcommand(s)
    metrics               Print the metrics
    modify-tikv-config    Modify tikv config, eg. tikv-ctl --host ip:port modify-tikv-config -m kvdb -n
                          default.disable_auto_compactions -v true
    mvcc                  Print the mvcc value
    print                 Print the raw value
    raft                  Print a raft log entry
    random-hex            Generate random bytes with specified length and print as hex
    raw-scan              Print all raw keys in the range
    recover-mvcc          Recover mvcc data on one node by deleting corrupted keys
    recreate-region       Recreate a region with given metadata, but alloc new id for it
    region-properties     Show region properties
    scan                  Print the range db range
    size                  Print region size
    split-region          Split the region
    store                 Print the store id
    tombstone             Set some regions on the node to tombstone by manual
    unsafe-recover        Unsafely recover the cluster when the majority replicas are failed

---

$ tikv server --help
TiKV (tikv)
Release Version:   4.0.1-alpha
Git Commit Hash:   e54b2eb06d1560b64cf23160f28fa74ff7a394ee
Git Commit Branch: rpm
UTC Build Time:    2019-12-13 08:44:16
Rust Version:      rustc 1.39.0-nightly (c6e9c76c5 2019-09-04)

A distributed transactional key-value store.

USAGE:
    tikv server [FLAGS] [OPTIONS]

FLAGS:
        --config-check
            Check config file validity and exit

    -h, --help
            Prints help information

        --print-sample-config
            Print a sample config to stdout

    -V, --version
            Prints version information

OPTIONS:
    -A, --addr <IP:PORT>
            Set the listening address

        --advertise-addr <IP:PORT>
            Set the advertise listening address for client communication

        --capacity <CAPACITY>
            Set the store capacity to use. If not set, use entire partition

    -C, --config <FILE>
            Set the configuration file

    -s, --data-dir <PATH>
            Set the directory used to store data

        --labels <KEY=VALUE>...
            Set the server labels. Uses `,` to separate kv pairs, like `zone=cn,disk=ssd`

    -f, --log-file <FILE>
            Set the log file path. If not set, logs will output to stderr

    -L, --log-level <LEVEL>
            Set the log level [possible values: trace, debug, info, warn, warning, error, critical]

        --metrics-addr <IP:PORT>
            Sets push address to the Prometheus Pushgateway, leaves it empty will disable Prometheus push

        --pd-endpoints <PD_URL>...
            Set the PD endpoints to use. Use `,` to separate multiple PDs

        --status-addr <IP:PORT>
            Set the HTTP listening address for the status report service

And now as the default:
$ tikv --help
TiKV (tikv)
Release Version:   4.0.1-alpha
Git Commit Hash:   e54b2eb06d1560b64cf23160f28fa74ff7a394ee
Git Commit Branch: rpm
UTC Build Time:    2019-12-13 08:44:16
Rust Version:      rustc 1.39.0-nightly (c6e9c76c5 2019-09-04)

A distributed transactional key-value store.

USAGE:
    tikv [FLAGS] [OPTIONS] [SUBCOMMAND]

FLAGS:
    -h, --help                    Prints help information
        --skip-paranoid-checks    Skip paranoid checks when open rocksdb
    -V, --version                 Prints version information

OPTIONS:
        --ca-path <ca_path>              Set the CA certificate path
        --cert-path <cert_path>          Set the certificate path
        --cipher-file <cipher_file>      set cipher file path
        --config <config>                Set the config for rocksdb
        --db <db>                        Set the rocksdb path
        --decode <decode>                Decode a key in escaped format
        --encode <encode>                Encode a key in escaped format
        --to-hex <escaped-to-hex>        Convert an escaped key to hex key
        --to-escaped <hex-to-escaped>    Convert a hex key to escaped key
        --host <host>                    Set the remote host
        --key-path <key_path>            Set the private key path
        --pd <pd>                        Set the address of pd
        --raftdb <raftdb>                Set the raft rocksdb path

SUBCOMMANDS:
    bad-regions           Get all regions with corrupt raft
    cluster               Print the cluster id
    compact               Compact a column family in a specified range
    compact-cluster       Compact the whole cluster in a specified range in one or more column families
    consistency-check     Force a consistency-check for a specified region
    diff                  Calculate difference of region keys from different dbs
    dump-snap-meta        Dump snapshot meta file
    fail                  Inject failures to TiKV and recovery
    help                  Prints this message or the help of the given subcommand(s)
    metrics               Print the metrics
    modify-tikv-config    Modify tikv config, eg. tikv-ctl --host ip:port modify-tikv-config -m kvdb -n
                          default.disable_auto_compactions -v true
    mvcc                  Print the mvcc value
    print                 Print the raw value
    raft                  Print a raft log entry
    random-hex            Generate random bytes with specified length and print as hex
    raw-scan              Print all raw keys in the range
    recover-mvcc          Recover mvcc data on one node by deleting corrupted keys
    recreate-region       Recreate a region with given metadata, but alloc new id for it
    region-properties     Show region properties
    scan                  Print the range db range
    server                Start a TiKV Daemon

    size                  Print region size
    split-region          Split the region
    store                 Print the store id
    tombstone             Set some regions on the node to tombstone by manual
    unsafe-recover        Unsafely recover the cluster when the majority replicas are failed

---

$ tikv server --help
TiKV (tikv)
Release Version:   4.0.1-alpha
Git Commit Hash:   e54b2eb06d1560b64cf23160f28fa74ff7a394ee
Git Commit Branch: rpm
UTC Build Time:    2019-12-13 08:44:16
Rust Version:      rustc 1.39.0-nightly (c6e9c76c5 2019-09-04)

A distributed transactional key-value store.

USAGE:
    tikv server [FLAGS] [OPTIONS]

FLAGS:
        --config-check
            Check config file validity and exit

    -h, --help
            Prints help information

        --print-sample-config
            Print a sample config to stdout

    -V, --version
            Prints version information

OPTIONS:
    -A, --addr <IP:PORT>
            Set the listening address

        --advertise-addr <IP:PORT>
            Set the advertise listening address for client communication

        --capacity <CAPACITY>
            Set the store capacity to use. If not set, use entire partition

    -C, --config <FILE>
            Set the configuration file

    -s, --data-dir <PATH>
            Set the directory used to store data

        --labels <KEY=VALUE>...
            Set the server labels. Uses `,` to separate kv pairs, like `zone=cn,disk=ssd`

    -f, --log-file <FILE>
            Set the log file path. If not set, logs will output to stderr

    -L, --log-level <LEVEL>
            Set the log level [possible values: trace, debug, info, warn, warning, error, critical]

        --metrics-addr <IP:PORT>
            Sets push address to the Prometheus Pushgateway, leaves it empty will disable Prometheus push

        --pd-endpoints <PD_URL>...
            Set the PD endpoints to use. Use `,` to separate multiple PDs

        --status-addr <IP:PORT>
            Set the HTTP listening address for the status report service

Closing. I am not longer involved with the project and have no stake in this merging. Feel free to create a new PR.


fix: asset import cli call

I don't really know why "label_asset_id": None is now output when label_asset_external_id is called, but that fixes the issue.

I don't really know why "label_asset_id": None is now output when label_asset_external_id is called, but that fixes the issue.

kili/cli/project/label.py
kili.append_to_labels(
label_asset_external_id=external_ids[i],
json_response=json_response,
project_id=project_id,
label_asset_id=None,  <---
)
label_asset_id=None has been added to the function.
@crsegerie any particular reason ?

I don't really know why "label_asset_id": None is now output when label_asset_external_id is called, but that fixes the issue.

kili/cli/project/label.py kili.append_to_labels( label_asset_external_id=external_ids[i], json_response=json_response, project_id=project_id, label_asset_id=None, <--- ) label_asset_id=None has been added to the function. @crsegerie any particular reason ?

Had a look, I don't think this is necessary to call this argument (type-wise). So I'm removing this argument.


Support access the tracer context in spring gateway filter

[ ] If this is non-trivial feature, paste the links/URLs to the design doc.

[x] Update the documentation to include this new feature.

[x] Tests(including UT, IT, E2E) are added to verify the new feature.

[ ] If it's UI related, attach the screenshots below.

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. https://github.com/apache/skywalking/issues/10862

[x] Update the CHANGES log.

Hi, this feature should be good. But you instrument a lot of things, could you try to put a doc about why you are instrumenting? And why there are so many things have to be done?

Hi, this feature should be good. But you instrument a lot of things, could you try to put a doc about why you are instrumenting? And why there are so many things have to be done?

I add a [Feature] issues linked to this.
The source is mainly based on the relevant source of apm-toolkit-trace, with some simple modifications.

I am not sure why, but this PR doesn't drive CI running.

I am not sure why, but this PR doesn't drive CI running.

I also see the same problem in #541

No, it isn't. That PR is waiting for maintainers' approval to run. But your PR doesn't.
Yours

That PR

You need to resolve conflicts before we can run the CI

You need to resolve conflicts before we can run the CI

There was no conflict, still not working.

You need to resolve conflicts before we can run the CI

There was no conflict, still not working.

My Github Actions now works normally...

Could you rebase your commits, 14 commits for now...

BTW , If the feature of https://github.com/apache/skywalking/issues/10862 is implemented with minimal modifications, it seems that only the putCorrelation method needs to be implemented, as the current WebFluxSkyWalkingOperators + TraceContext can support read operations.
e.g.
public Mono<Void> filter(ServerWebExchange exchange, GatewayFilterChain chain) {

        ServerHttpRequest buildRequest = exchange.getRequest()
                .mutate()
                .headers(httpHeaders -> WebFluxSkyWalkingOperators.continueTracing(exchange, () -> {
                    httpHeaders.add("x-trace-id", TraceContext.traceId());
                    httpHeaders.add("x-segment-id", TraceContext.segmentId());
                    httpHeaders.add("x-span-id", String.valueOf(TraceContext.spanId()));
                }))
                .build();
        return chain.filter(exchange.mutate().request(buildRequest).build());
    }

BTW , If the feature of apache/skywalking#10862 is implemented with minimal modifications, it seems that only the putCorrelation method needs to be implemented, as the current WebFluxSkyWalkingOperators + TraceContext can support read operations.
e.g.
public Mono<Void> filter(ServerWebExchange exchange, GatewayFilterChain chain) {

        ServerHttpRequest buildRequest = exchange.getRequest()
                .mutate()
                .headers(httpHeaders -> WebFluxSkyWalkingOperators.continueTracing(exchange, () -> {
                    httpHeaders.add("x-trace-id", TraceContext.traceId());
                    httpHeaders.add("x-segment-id", TraceContext.segmentId());
                    httpHeaders.add("x-span-id", String.valueOf(TraceContext.spanId()));
                }))
                .build();
        return chain.filter(exchange.mutate().request(buildRequest).build());
    }

In this context WebFluxSkyWalkingOperators.continueTracing will add a meaningless LocalSpan

In this context WebFluxSkyWalkingOperators.continueTracing will add a meaningless LocalSpan

Make sense. You are correct. Keep what you have built. But please update others accordingly.


Add shared implementation of dispatching logic

Fix https://github.com/aspnet/EntityFramework/issues/5182
cc @NTaylorMullen @Eilon

I dont see use of the IsDispatcher method. How do you determine that?
:watch:

"IsDispatcher" is implied by the separate assemblies that "Ms.EfCore.Tools" contains dispatcher and "Ms.EfCore.Tools.Cli" contains the inside-man.

Updated

:shipit:


Support for MSVC environment

updated whisper-cpp to v1.3.0 because the build in MSVC is ok after update whisper-cpp to v1.3.0

whisper-cpp's MSVC build had problem from Japanese character "『" and "』" encoding problem
MSVC missleaded the file has SJIS but after v1.3.0, CMakeList.txt configure it is UTF-8 explicitly so now MSVC can build it.

And I had tested it with MSVC clean environment. Just following is reportable:

we need to add three dependencies for this build. We need to add these dependencies for PATH.

MSVC itself

CMake

LLVM(clang)

It seems buildable it by cargo so we don't need to build it from git repository.

So I had rewrite BUILDING.md

I'll work on adding the new features from whisper.cpp v1.3.0 later tonight.

Expect new versions of both whisper-rs and whisper-rs-sys sometime in the next few hours.


CI: fslock test fail

https://app.travis-ci.com/github/juicedata/juicefs/builds/246281417#:~:text=--- FAIL%3A TestExec (10.70s,2735

Not reproduced for a long time.


100k Remote User Connections Document update

@nthewara : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @nthewara! #sign-off


[HOTFIX] (Re)build by default in start script

Setting ./bin/start to rebuild by default. Hotfix for Issue #291.
[Issue: #291]
Signed-off-by: jbobo<EMAIL_ADDRESS>

@adamgering So what should we do? We've confirmed that our documented start method, running ./bin/start produces errors.

@adamgering It looks like the problem remains where deploying the project in non-dev mode results in errors accessing the compiled protobuf files.
Regarding your comment/request for changes: Please recommend an alternative solution to the problem if you have one.

Let's remove non-dev mode then, it's a feature built for nobody, no one uses it and no one is blocked.

It can be addressed in a PR to cleanup the DockerFiles. Until then, it's low priority.

This is only low-priority to those who are up to speed on whatever undocumented process builds without error. The build takes a long time to carry out, and if the documentation leads people to follow these steps then more time is wasted in reattempts and escalation than it would be with the additional build step.
This issue has already left a number of contributors blocked trying to figure out what they did wrong. It needs to be addressed even if the fix is sub-optimal.

I agree that this is sub-optimal, but it's a valid issue that @nadiabahrami brought up a couple days ago and has been dealing with. We know the issue involves containers finding the protobuf files but we don't have a permanent solution yet. Since we know running -b with the start script fixes the issue for now and causes no immediate harm, let's just push this as a hotfix to prevent impacting new contributors and leave Issue #291 open to be solved with a permanent fix when we have time.

@adamgering if you can run script and show us that running in dev-mode without -b fixes the issue then I'm down to rebase my commit and go with that instead. At the moment tho we've got proof that this works and I'd like to merge it now to minimize the impact to new contributors.

Replaced by PR #351

Ok, will pull and test out PR #351 later. Thanks @jbobo, @adamgering, @nadiabahrami for sorting this out!


generator ignores a foreign key name when imports table

In case of importing tables from the database, the generator ignores an actual foreign key column name and always supposes that it’s equal to concat('tableName', '_id').
It’s quite cumbersome to fix all the foreign key column names throughout the generated API if they don’t comply to the default Eloquent behavior.

Reference of: #475

fixed.


API Update Spot

API Request
Story

As a developer
I need an API route to update a spot's information
So that I can change its information

Acceptance Criteria

Route /spots/:id
Method: PUT
Request Object:

{
	definition from PR #16
}

Response

{
	definition from PR #16 // send back the request body for now
}

written in a spots-api/api/put-spot.js file
Include an spots-api/api/put-spot.spec.js file with a supertest for the new endpoint

Approach

Since we have no database yet, just send back example data.

@bmanzella Sprint 2

Completed


SignalR Core - Hub Context

How can I get the Hub context in another class other than a Controller. I need to send back a response to the user when some other function finish.

Exact same concept as sending a response from a Controller. Either use DI to get the context or pass it into your class via someone else that got created from DI.
#884 for reference


Testing a PR

Stuart Loria testing the creation of a PR for the Cloud course from MIT

Closing pull request


Laplace Subnetwork with timm library model

Hi,
I would like to apply the Laplace Subnetwork approach to a timm library model (standard resnet18). I think the problem I am encountering is not unique to timm models per se, but to inplace operations?  I have made a small reproducible example in this google colab. The error
Output 0 of BackwardHookFunctionBackward is a view and is being modified inplace. This view was created inside a custom Function (or because an input was returned as-is) and the autograd logic to handle view+inplace would override the custom backward associated with the custom Function, leading to incorrect gradients. This behavior is forbidden. You can fix this by cloning the output of the custom Function.
only occurs when trying to do a subnetwork approach, and not with the default Laplace parameters. I have also tried to change the timm resnet activation functions to not be inplace, but maybe it is also related to skip connections? Even though the error does not occur inside the Laplace library immediately, I was wondering if you had any suggestions or pointers to make this approach work.

I suppose this is related: https://docs.backpack.pt/en/1.4.0/use_cases/example_resnet_all_in_one.html#any-resnet-with-backpack-s-converter

Thanks for the recommendation. I believe I installed as you suggested @runame, however, I get
laplace/curvature/asdl.py", line 132, in diag     fisher_maker = get_fisher_maker(self.model, cfg, self.kfac_conv) TypeError: get_fisher_maker() takes 2 positional arguments but 3 were given .

Ah right, can you try to remove the argument self.kfac_conv to get_fisher_maker?

Thank you for the help, for some models I do get the following pytorch warning and I am not sure what it implies:
UserWarning: Using a non-full backward hook when the forward contains multiple autograd Nodes is deprecated and will be removed in future versions. This hook will be missing some grad_input. Please use register_full_backward_hook to get the documented behavior.
From this trace:
lib/python3.9/site-packages/laplace/baselaplace.py:377: in fit
    loss_batch, H_batch = self._curv_closure(X, y, N)
lib/python3.9/site-packages/laplace/baselaplace.py:777: in _curv_closure
    return self.backend.kron(X, y, N=N)
lib/python3.9/site-packages/laplace/curvature/asdl.py:164: in kron
    f, _ = fisher_maker.forward_and_backward()
lib/python3.9/site-packages/asdl/fisher.py:116: in forward_and_backward
    self.call_model()
lib/python3.9/site-packages/asdl/grad_maker.py:258: in call_model
    self._model_output = self._model_fn(*self._model_args, **self._model_kwargs)
lib/python3.9/site-packages/torch/nn/modules/module.py:1571: in _call_impl
    self._maybe_warn_non_full_backward_hook(args, result, grad_fn)


Is it possible to read environment variable directly in the TOML config file?

Hi @guangie88 , Is it at all possible to fetch environment variable directly in the TOML file other than the tag-based approach?

Nope, most templating engine e.g. Jinja2 doesn't do this by default either.


feat: add new GitLab tokens

Closes #1401
Description:
Add additional GitLab tokens
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

@bufferoverflow super! I agree with @rgmz, adding an extra entropy check helps limit the FPs

Thanks for the feedback @rgmz and @zricethezav , I just added Entropy: 3  to all the tokens.

@bufferoverflow my bad on not merging this earlier. Would you mind resolving conflicts? Sorry!

thank you @bufferoverflow! Merged


False positive on the PostgreSQL JDBC client JAR

CVE-2016-7048 was updated on 2018-10-12 and is now breaking our builds.  Upon further inspection, we see that this is a vulnerability to those using the interactive installer - clearly those of us downloading this artifact from Maven Central are not affected
False positive on library postgresql.jar - reported as cpe:2.3:a:postgresql:postgresql:9.3 -  CVE-2016-7048
<dependency>
     <groupId>org.postgresql</groupId>
     <artifactId>postgresql</artifactId>
</dependency>

At least versions >=  42.2.1 are affected.

This was fixed recently - see https://github.com/jeremylong/DependencyCheck/issues/1488 - but apparently pops up again.
There are multiple problems with this:

The CVE does not affect the JDBC driver in the first place.
The version compare algorithm is somehow broken: the CVE affects (<< 9.3.15 | (>> 9.3 && << 9.4.10) | (>> 9.4 && << 9.5.5)) which means PostgreSQL 10.x and PostgreSQL-JDBC 42.x are both not affected.

I’d definitely look at the latter in any case, since that shows that the Maven plugin somehow does not work correctly!
[INFO] --- dependency-check-maven:3.3.2:aggregate (default) @ veraweb-parent ---
[INFO] Central analyzer disabled
[INFO] Checking for updates
[INFO] starting getUpdatesNeeded() ...
[INFO] NVD CVE requires several updates; this could take a couple of minutes.
[INFO] Download Started for NVD CVE - 2004
[INFO] Download Started for NVD CVE - 2003
[INFO] Download Started for NVD CVE - 2002
[INFO] Download Complete for NVD CVE - 2003  (2312 ms)
[INFO] Download Started for NVD CVE - 2005
[INFO] Processing Started for NVD CVE - 2003
[INFO] Download Complete for NVD CVE - 2004  (2643 ms)
[INFO] Download Started for NVD CVE - 2006
[INFO] Processing Started for NVD CVE - 2004
[INFO] Download Complete for NVD CVE - 2005  (3008 ms)
[INFO] Download Started for NVD CVE - 2007
[INFO] Download Complete for NVD CVE - 2006  (3024 ms)
[INFO] Download Started for NVD CVE - 2008
[INFO] Download Complete for NVD CVE - 2002  (6238 ms)
[INFO] Download Started for NVD CVE - 2009
[INFO] Download Complete for NVD CVE - 2008  (3288 ms)
[INFO] Download Started for NVD CVE - 2010
[INFO] Download Complete for NVD CVE - 2009  (3037 ms)
[INFO] Download Started for NVD CVE - 2011
[INFO] Download Complete for NVD CVE - 2007  (4040 ms)
[INFO] Download Started for NVD CVE - 2012
[INFO] Download Complete for NVD CVE - 2012  (3255 ms)
[INFO] Download Started for NVD CVE - 2013
[INFO] Processing Complete for NVD CVE - 2003  (11752 ms)
[INFO] Processing Started for NVD CVE - 2005
[INFO] Download Complete for NVD CVE - 2011  (6216 ms)
[INFO] Download Started for NVD CVE - 2014
[INFO] Download Complete for NVD CVE - 2010  (7629 ms)
[INFO] Download Started for NVD CVE - 2015
[INFO] Download Complete for NVD CVE - 2013  (3967 ms)
[INFO] Download Started for NVD CVE - 2016
[INFO] Download Complete for NVD CVE - 2014  (3969 ms)
[INFO] Download Started for NVD CVE - 2017
[INFO] Processing Complete for NVD CVE - 2004  (17555 ms)
[INFO] Processing Started for NVD CVE - 2006
[INFO] Download Complete for NVD CVE - 2015  (3619 ms)
[INFO] Download Started for NVD CVE - 2018
[INFO] Download Complete for NVD CVE - 2016  (3524 ms)
[INFO] Download Started for NVD CVE - Modified
[INFO] Download Complete for NVD CVE - 2017  (6490 ms)
[INFO] Download Complete for NVD CVE - 2018  (8951 ms)
[INFO] Processing Complete for NVD CVE - 2005  (18546 ms)
[INFO] Processing Started for NVD CVE - 2002
[INFO] Download Complete for NVD CVE - Modified  (12096 ms)
[INFO] Processing Complete for NVD CVE - 2002  (8339 ms)
[INFO] Processing Started for NVD CVE - 2008
[INFO] Processing Complete for NVD CVE - 2006  (21055 ms)
[INFO] Processing Started for NVD CVE - 2009
[INFO] Processing Complete for NVD CVE - 2009  (19782 ms)
[INFO] Processing Started for NVD CVE - 2007
[INFO] Processing Complete for NVD CVE - 2008  (22850 ms)
[INFO] Processing Started for NVD CVE - 2012
[INFO] Processing Complete for NVD CVE - 2007  (53492 ms)
[INFO] Processing Started for NVD CVE - 2011
[INFO] Processing Complete for NVD CVE - 2012  (77745 ms)
[INFO] Processing Started for NVD CVE - 2010
[INFO] Processing Complete for NVD CVE - 2011  (60389 ms)
[INFO] Processing Started for NVD CVE - 2013
[INFO] Processing Complete for NVD CVE - 2010  (41067 ms)
[INFO] Processing Started for NVD CVE - 2014
[INFO] Processing Complete for NVD CVE - 2013  (28461 ms)
[INFO] Processing Started for NVD CVE - 2015
[INFO] Processing Complete for NVD CVE - 2014  (26197 ms)
[INFO] Processing Started for NVD CVE - 2016
[INFO] Processing Complete for NVD CVE - 2015  (12701 ms)
[INFO] Processing Started for NVD CVE - 2017
[INFO] Processing Complete for NVD CVE - 2016  (14855 ms)
[INFO] Processing Started for NVD CVE - 2018
[INFO] Processing Complete for NVD CVE - 2018  (39958 ms)
[INFO] Processing Started for NVD CVE - Modified
[INFO] Processing Complete for NVD CVE - 2017  (48260 ms)
[INFO] Processing Complete for NVD CVE - Modified  (6681 ms)
[INFO] Begin database maintenance.
[INFO] End database maintenance.
[INFO] Check for updates complete (278493 ms)
[INFO] Analysis Started
[INFO] Finished Archive Analyzer (2 seconds)
[INFO] Finished File Name Analyzer (0 seconds)
[INFO] Finished Jar Analyzer (2 seconds)
[INFO] Finished Assembly Analyzer (2 seconds)
[INFO] Finished Dependency Merging Analyzer (0 seconds)
[INFO] Finished Version Filter Analyzer (0 seconds)
[INFO] Finished Hint Analyzer (0 seconds)
[INFO] Created CPE Index (4 seconds)
[INFO] Skipping CPE Analysis for npm
[INFO] Finished CPE Analyzer (6 seconds)
[INFO] Finished False Positive Analyzer (0 seconds)
[INFO] Finished NVD CVE Analyzer (0 seconds)
[INFO] Finished Vulnerability Suppression Analyzer (0 seconds)
[INFO] Finished Dependency Bundling Analyzer (0 seconds)
[INFO] Analysis Complete (14 seconds)
[WARNING]

One or more dependencies were identified with known vulnerabilities in org.evolvis.veraweb (parent):

postgresql-42.2.5.jar (org.postgresql:postgresql:42.2.5, cpe:/a:postgresql:postgresql:42.2.5, cpe:/a:postgresql:postgresql_jdbc_driver:42.2.5) : CVE-2016-7048
javax.servlet-3.0.0.v201112011016.jar (org.eclipse.jetty.orbit:javax.servlet:3.0.0.v201112011016, cpe:/a:eclipse:jetty:3.0.0.v20111201, cpe:/a:jetty:jetty:3.0.0.v20111201) : CVE-2017-7656, CVE-2017-7658, CVE-2017-7657, CVE-2017-9735
logback-core-1.1.2.jar (cpe:/a:logback:logback:1.1.2, ch.qos.logback:logback-core:1.1.2) : CVE-2017-5929
jetty-util-9.0.7.v20131107.jar (cpe:/a:eclipse:jetty:9.0.7.v20131107, cpe:/a:jetty:jetty:9.0.7.v20131107, org.eclipse.jetty:jetty-util:9.0.7.v20131107) : CVE-2017-7656, CVE-2017-7658, CVE-2017-7657, CVE-2017-9735

See the dependency-check report for more details.

It appears that the CVE was updated recently to add cpe:/a:postgresql:postgresql:- - and currently, ODC treats the "-" as any. There are plans to change this behavior as part of the move from the NVD XML data feeds to the JSON data feeds.
Additionally, we have taken the stance on DB related findings that we would only suppress specific CVEs as they are reported/found as opposed to just suppressing the CPE. The reason is that suppressing by CPE will remove all future CVEs where as just suppressing by CVE will still match on new/un-reviewed findings and it is possible that one of these would affect the driver and still be listed under the database server's CPE.

@jeremylong That all sounds reasonable ... we've suppressed it locally for now but let me know if we can help with the conversion to JSON, etc.

On Wed, 24 Oct 2018, Jeremy Long wrote:

opposed to just suppressing the CPE. The reason is that suppressing by
CPE will remove all future CVEs where as just suppressing by CVE will
still match on new/un-reviewed findings and it is possible that one of
these would affect the driver and still be listed under the database
server's CPE.

I doubt: the JDBC has cpe:/a:postgresql:postgresql_jdbc_driver
and you could ask the PostgreSQL team whether they intend to
keep them separated.

Additionally, we have taken the stance on DB related findings that we
would only suppress specific CVEs as they are reported/found as

Then please explain to me how this is a “DB related finding” when
your plugin is unable to realise that version 42.x is not affected
by something affecting only some 9.x releases?


Upload choking internet speed

System Info
hf_transfer==0.1.6, Windows 11, Python version 3.10.9, huggingface_hub==0.23.4
While having the environment variable 'HF_HUB_ENABLE_HF_TRANSFER' set to 1 the upload speeds attempted by hf_transfer completely chokes my connection and pretty much causes Denial Of Service.
My internet connection is rated at 250/250 Mbit/s yet it still seems to attempt to push far beyond that and all other connections on Local Network stops working properly due to it.
In fact when setting the the variable to 0 I end up uploading faster than with it set to 1 due to it not choking the connection repeatedly
Reproduction
Unsure if it is reproducible that easily however there might be some ways to mitigate it by pre-checking speed or manually setting some form of upper limit
Expected behavior
Being able to upload faster with HF_HUB_ENABLE_HF_TRANSFER set to 1 compared to HF_HUB_ENABLE_HF_TRANSFER set to 0 and not having my entire internet connection getting choked

Unfortunately this is working as intended.
hf_transfer doesn't aim to be a silver bullet solving everyone's problem (which is why it's not ON by default and has to be opted in).
hf_transfer works by saturating the network over multiple connections. It's only useful for bypassing some S3 limitations over single connections and going beyond 500MB/s in certain circumstances. If it saturates your router or any kind of hardware on the line, it's unfortunately normal.
Adding any form of saturation detection would slow down those use cases while attempting to solve the case your doing here.
Not using hf_transfer is the solution you should be using.


CVE-2021-43045 on Apache.Avro < 1.10.2 .NET Dependency

Description
There is a new CVE on Apache.Avro < 1.10.2 .NET. Solution: Upgrade to Apache.Avro 1.11.0.
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-43045

PR exists: https://github.com/confluentinc/confluent-kafka-dotnet/pull/1757


Inline editing of HTMLText etc.

@sheadawson, I've been using this module by @milkyway-multimedia for inline editing inside GridFields on a few occasions now. Thought I'd leave a quick note here; I think this would be a good addition to the Blocks module because it allows editors to edit a block's content in the context of the parent page. It supports inline HTMLEditor just fine (Uploadfield may need a little work).

BTW; we could also look into gridfieldextensions/GridFieldAddExistingSearchButton as it would allow editors to search existing blocks by more than just title/name.

Only thing is, I don't think EditableRow supports versioning (yet).

I've been working on this a bit. @sheadawson, what do you think; better to publish as a 'block-enhancements' module or could we include this work in the Blocks module?

Have everything working now: sorting to assign block area, block-preview images, inline-editing.

Wow this is looking pretty slick dude! How did you get on with the publishing side of things? Are there any limitations as to what fields can be used in the summary edit views? Can I have a play with it?

Yes, you can :) https://github.com/micschk/silverstripe-block_enhancements


Update and xit existing cypress tests

I want to merge this change because...
I have updated and xited existing cypress tests

Screenshots

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Changes are mentioned in the changelog.

CI cypress run has failed: https://github.com/mirumee/saleor-storefront/pull/801/checks?check_run_id=848725199


Update fields.mjs

resources/app/common/data/fields.mjs

[ ] Add new typedefs
[ ] Add Form Field integration methods
[ ] Add Active Effect application methods
[ ] Add DocumentUUIDField
[ ] Add AnyField
[ ] Add DocumentTypeField
[ ] Add TypedSchemaField
[ ] Remove deprecated fields

@LukeAbby DocumentTypeField is the biggest blocker of all of these because it's a StringField extension that's used by the big document types like Actor and Item.

After toInput is implemented, update the formInput and formGroup handlebars helpers in src/foundry/client/apps/templates.d.mts

TODO after 0f58b4f3f1a15b12dd431a2385d6f0faede1514a

[ ] Consider extracting the callback on DataFieldOptions["validate"] to a DataFieldValidator callback as defined by Foundry (possibly DataField.Validator?)
[ ] Individual properties of DataField.Context should have undefined as a valid option, consider using InexactPartial
[ ] Implementation of Form Field helpers needs improvement


Add deprecation notice

(FWIW, this is an unfortunate reflection of reality, not a RFC -- note that there has not been an official release of this project since 19.03.5 and the latest Docker release is 19.03.9.)

Just to be extra clear here -- this isn't something I "don't have time for" or anything like that.  It was an official request that this project finally be laid to rest (months ago, so this PR even suffers from delay).
Additionally, the "maintenance mode" note has been in our repository description since March of 2018, so we're really into two years of this project being publicly stated to be on limited life support (let alone however long it was on life support before that was made clear in the documentation).
For further (explicit) context, Boot2Docker was only really still maintained for Windows 7 and Windows 10 Home users.  The former should be off Windows 7 now that it's EOL, and the latter are now supported by Docker Desktop thanks to WSL2 (as noted in the changes in this PR).  Docker Toolbox, which is the main way Windows 7 and Windows 10 Home users would've been consuming this hasn't seen a release since 19.03.1 (Jul 31, 2019).  Docker Machine is in a similar boat with the last release being Sep 2, 2019.
We've done our best to try and actively encourage folks who need something like this to find other projects like RancherOS, especially for use cases that are outside the narrow focus of a personal machine development VM that this project has had.  Hopefully this PR helps give anyone still relying on Boot2Docker the push they need to find something more suitable. :heart:

SGTM

For anyone who can't or does not want to use Docker Desktop for Mac:
mobymac allows for an easy setup of a Docker VM using Virtualbox and NFS mounts.
I previously used docker-machine/boot2docker but after this announcement I switched the project over to use vagrant to install docker into a vanilla debian box. Performance is equal to boot2docker in my measurements, it just takes up a bit more disk space.
Feel free to link to mobymac in the boot2docker README if you like.

So sad to see this deprecated. Docker-Toolbox is still way faster and way more reliable than docker for windows.
Using hyper-v on a windows 10 decreases whole system performance  so much. (As the whole windows10 runs as a vm).

@tekook are you using the new WSL2 backend? https://docs.docker.com/docker-for-windows/wsl/ we're working closely with Microsoft to improve performance issues; if you have more details about the performance issues you're seeing (and steps to reproduce), feel free to open a ticket in the Docker for Windows issue tracker; https://github.com/docker/for-win/issues

@thaJeztah wsl2 is unusable while sharing data between host and vm (see https://github.com/microsoft/WSL/issues/4197).
The "old" backend (docker) utilizes a hyper-v vm, thus turning the whole system into a vm thus slowing down the whole system and makes it impossible to create x64 vm with (e.g) vmware workstation.
This makes docker-for-windows impossible for my use case (since I need to boot multiple x64vms via workstation while deving)

@tekook thanks; yes, performance in sharing between the Windows filesystem (compared to sharing from Linux / WSL2) is something currently being looked at by the Windows team.

I wonder if this had some interest for running Docker from a USB key without admin privileges using Qemu to provide a local Docker server for things like Codewind. When you come to something like a school where they are used to swap your lab randomly this is just what your teacher  needs.

For me a showstopper would be https://github.com/docker/for-mac/issues/180 where the real client ip is never passed to the container. Serious problem when validating htaccess rules, modsecurity  or the like

I wonder if this had some interest for running Docker from a USB key without admin privileges using Qemu to provide a local Docker server for things like Codewind. When you come to something like a school where they are used to swap your lab randomly this is just what your teacher needs.

@TWINGSISTER thanks for the feedback; the primary goal of this project was to have a desktop environment to run Docker on non-Linux environments. While there could be other use-cases, those were not the main focus, and with Docker Desktop maturing, maintaining both projects couldn't really be warranted. That said; the code in this repository is open-source, so Codewind could still make use of it (or perhaps a solution based on https://github.com/linuxkit/linuxkit would fit their needs).
Also, feel free to open a ticket in Docker's public roadmap: https://github.com/docker/roadmap to describe your use-case and why Docker Desktop cannot address that use case. Having a ticket there can help getting it being considered and prioritised if there's interest.

For me a showstopper would be docker/for-mac#180 where the real client ip is never passed to the container. Serious problem when validating htaccess rules, modsecurity or the like

@logopk thanks for bringing that up. The Docker Desktop team is aware of that limitation and I was told there's an existing internal ticket. To help getting that prioritised, I would also recommend opening a ticket in the public roadmap (if none exists yet) https://github.com/docker/roadmap. If you do, I'd recommend describing your use-case, include a link to the docker for mac issue (https://github.com/docker/for-mac/issues/180) (and perhaps "announce" it there as well, so that others can add their use-case and/or up-vote the ticket).

@tianon : thanks for being open about the project future, wish something similarly straight-forward could be added to the docker-machine repository too (https://github.com/docker/machine) as well - instead of some undefined "maintenance mode"...

Docker Machine is in a similar boat with the last release being Sep 2, 2019.

And thanks for maintaining! So long, and thanks for all the fish

Just FYI, I've pushed out a final-final release at https://github.com/boot2docker/boot2docker/releases/tag/v19.03.12 (especially to get https://github.com/boot2docker/boot2docker/pull/1403 in a released version). :heart:

Why would any sane person move to platform specific backend when it prohibits moving containers from Any OS to Any OS.

Just to be extra clear here -- this isn't something I "don't have time for" or anything like that. It was an official request that this project finally be laid to rest (months ago, so this PR even suffers from delay).
Additionally, the "maintenance mode" note has been in our repository description since March of 2018, so we're really into two years of this project being publicly stated to be on limited life support (let alone however long it was on life support before that was made clear in the documentation).
For further (explicit) context, Boot2Docker was only really still maintained for Windows 7 and Windows 10 Home users. The former should be off Windows 7 now that it's EOL, and the latter are now supported by Docker Desktop thanks to WSL2 (as noted in the changes in this PR). Docker Toolbox, which is the main way Windows 7 and Windows 10 Home users would've been consuming this hasn't seen a release since 19.03.1 (Jul 31, 2019). Docker Machine is in a similar boat with the last release being Sep 2, 2019.
We've done our best to try and actively encourage folks who need something like this to find other projects like RancherOS, especially for use cases that are outside the narrow focus of a personal machine development VM that this project has had. Hopefully this PR helps give anyone still relying on Boot2Docker the push they need to find something more suitable. heart

I would not recommend switching to RancherOS, as it is also being phased out, (See: https://github.com/rancher/os/issues/3000).
Disclaimer: I recently started working for Rancher Labs. However, I've also been following this space for some time as an open source maintainer for various projects.

We've done our best to try and actively encourage folks who need something like this to find other projects like RancherOS, especially for use cases that are outside the narrow focus of a personal machine development VM that this project has had. Hopefully this PR helps give anyone still relying on Boot2Docker the push they need to find something more suitable. heart

I would not recommend switching to RancherOS, as it is also being phased out, (See: rancher/os#3000).
Disclaimer: I recently started working for Rancher Labs. However, I've also been following this space for some time as an open source maintainer for various projects.

FYI for those who have think like me that RancherOS was best OS ever made and was very disappointed to see it to get deprecated. I decided to fork it to name BurmillaOS and we recently released our first version which continues directly from there where RancherOS ended. That community is currently still small but growing.


Add Debug trait implementations for all structs and enums

This PR implements the Debug trait for all structs and enums that were missing it. Changes are minimal and only add #[derive(Debug)] where needed.
Modified files:

src/exchange/modify.rs
src/exchange/order.rs
src/exchange/cancel.rs
src/exchange/exchange_client.rs
src/req.rs

Testing:

All 8 unit tests passed successfully
CI script (ci.sh) completed with no errors
All compilation checks passed
No warnings or errors related to Debug implementations

Resolves: #52
Created by Devin: https://preview.devin.ai/sessions/26449cc89c9b4b9886ba30da44fd6bb9

Thanks Devin

thanks devin


CDN link in NPM README is out of date

The CDN link example in @justifi/webcomponents is not up to date with the latest version, and will not stay up to date automatically.

update the link
See if there is a way to import the package.json and pull the version in dynamically, possibly by making the file an .mdx file instead of .md?

@colganmeanor I can't find a PR for this, can you attach one? I'd like to pull your branch down and add a couple of other README changes to the other packages and push out a patch version so that we don't have to force re-publish the current version


Recommendation on how to handle importing a world.

Discussed in https://github.com/itzg/docker-minecraft-bedrock-server/discussions/260

Originally posted by chaoscode July  2, 2022
I am attempting to move off a VM to this docker image for my daughters mc servers. The issue I am running into is important her world levels into the docker and have it be priested. I was thinking about forking the docker but I wanted to use your image to stay up to date. priested volumes don't help because I can not tell the game where the worlds folder is located. What is the best way to handle this?

also, every time the docker restarts the world is reset. How do you make the world levels priest?

So I wanna share how I solved this with everyone as I was doing this in TrueNAS Scale and I figured out how to make it work. This took me hours to nail down.
1: Create a new dataset called like "minecraft-bedrock-servername"
2: Copy world directories to the new dataset "minecraft-bedrock-servername"
2: create a defined volumn for the docker:
a: Host Path = "/mnt/PoolName/minecraft-bedrock-servername"
b: Mount Path = "data/worlds"
3: Add Server Container Environment Variable
a: Environment Variable Name = "LEVEL_NAME"
b: Environment Variable Value = "levelname"
Start the Application in TrueNAS Scale.


Update sbt to 1.5.0

Updates org.scala-sbt:sbt from 1.4.9 to 1.5.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-minor

Codecov Report

Merging #175 (95f7a75) into master (994f65f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   37.80%   37.80%
=======================================
  Files           3        3
  Lines          82       82
=======================================
  Hits           31       31
  Misses         51       51

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 994f65f...95f7a75. Read the comment docs.


How do you change time format on the clocks to 12 hour? Question

Available in JaxCore settings. (Updated)


perf(plugin/prometheus): use ngx.re.match to replace string.match

Summary
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference

Fix #[issue number]

Do we have exsting tests for this piece of code? If not, this would be a good time to add some. What do you think @chronolaw ?

It seems that it is the responsibility of unit tests, but spec/03-plugins/26-prometheus can not cover this situation.

How about factoring out the piece that is supposed to match to a separate function that can be tested via unit tests?

These functions are used internally, they are not exposed for testing.
And Prometheus module need to be inited in init and init_worker phase,
this makes it hard to do unit test for internal small functions.
A way may be exposing these functions only for testing, such as Prometheus._check_metric_and_label_names,
but is it worthy?

PR #11137 will add the tests for this PR and #11115.

Good suggestions. I will change these regex in CE PR.


language es-ES overwrites es

See this line which is exactly like the one used for es.
Obviously the same is true within the unified languages file at this line
I guess it should be:
this.numeral.language('es-ES', language);

At the moment, version 1.5.3, if you call
numeral.language('es-ES'):

you get an error saying Unknown language : es-ES

+1

+1 for nl-NL. Did you find a solution?

all files names are standardized as lowercase in 2.0 so this should be fixed.


How to use it with v-for directive?

Looks like in case of v-for, all repeated items get same id. What am I doing wrong?
Thanks,
Alex

Hi @berlaga!
I didn't actually think of it. Did you find a solution?

You can use item specific value to make those IDs unique again:
<li v-for="user in users" v-uni-id="user.id"> ... </li>


php short_open_tags should not be required

check all php view files to make sure short open tags is not a system requirement.

search in your IDE for:

replace them all with:

verify each changed result before commit. add changelog


Correctly persists Release upgrade failure

When release upgrade fails, updatedRelease is already created
in a storage by *ReleaseServer.UpdateRelease, therefore we should
be updating it's status, not creating it again.

Here is how it looks like without a fix:
[tiller] 2017/09/22 11:00:02 creating updated release for prometheus-operator
FIRST >>>>>> [tiller:storage] 2017/09/22 11:00:02 creating release "prometheus-operator.v2"
[tiller] 2017/09/22 11:00:03 performing update for prometheus-operator
[tiller] 2017/09/22 11:00:03 executing 1 pre-upgrade hooks for prometheus-operator
[tiller] 2017/09/22 11:00:03 hooks complete for pre-upgrade prometheus-operator
[tiller:kubeClient] 2017/09/22 11:00:03 building resources from updated manifest
[tiller:kubeClient] 2017/09/22 11:00:03 checking 4 resources for changes
[tiller:kubeClient] 2017/09/22 11:00:03 Looks like there are no changes for ServiceAccount "prometheus-operator-prometheus-operator"
[tiller:kubeClient] 2017/09/22 11:00:04 Looks like there are no changes for ClusterRoleBinding "prometheus-operator-prometheus-operator"
[tiller] 2017/09/22 11:00:04 warning: Upgrade "prometheus-operator" failed: no resource with the name "prometheus-operator-prometheus-operator" found
[tiller:storage] 2017/09/22 11:00:04 updating release "prometheus-operator.v1"
SECOND >>>> [tiller:storage] 2017/09/22 11:00:05 creating release "prometheus-operator.v2"
[tiller] 2017/09/22 11:00:05 warning: Failed to record release prometheus-operator: release: "prometheus-operator.v2" already exists

cc'ing @fibonacci1729 to review this since this package was his baby

Any way you can write a unit test under release_update_test.go to verify this fix as well? Thanks!

I'll add tests shortly

Tests were in place, but Memory storage driver was storing pointers passed to it as arguments, so updates to them were implicitly updating stored Release. I have changed Memory storage driver to store deep copy,  so more store operation errors are visible

@bacongobbler all tests are green now and actually test the thing, can you have a look? Change is fairly straightforward

@redbaron If @fibonacci1729 can't get around to this, I'll try to get to it on Friday or Monday (and if we decide to do a bug fix release, we can pull this in). Thank you for catching all of these bugs as of late


I hope this helps some
Thank you so much for your hard work.
Hopefully once I fully understand the framework I will be able to assist you in this project.
This is very close to how Blizz Servers are structured.

Do not know why it double posted so im closing this one


[core-http] Remove unused dev dependency "ts-loader"

Dev dependency ts-loader appears to be unused.  The only package which depends on it is core-http, and there are no other references to ts-loader in package.json or the source code.
https://github.com/Azure/azure-sdk-for-js/search?q=ts-loader&unscoped_q=ts-loader

Looks like this was missed when removing webpack as a dependency of core-http.  Looks good, thanks!


Updated Testimonials Section

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy

Update Testimonials Section
Summarize what this PR fixes:-
Update Testimonials Section
Fixes #458
Checklist
(Add an x between the brackets to check the items)

[x] I have summarize what this PR fixes above
[x] I have provided the issue ID/URL above
[x] I have not added Bootstrap or jQuery and used correct styling guide.
[x] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[x] I was the first one to claim the issue or I created the issue

I resolved the conflicts this time, be sure you are running git pull upstream master before pushing your changes and checking for conflicts. This issue is really a duplicate of #367, but the styling looks better, so I'm going to let it slide.


How Can I setup dataset and .pth path?

I git clone this git page and download datasets and .pth file suessful, and then I want to know how to setup dataset and pth path.
I got a path error.

You need to modify the dataset loading path and the .pth loading path in the https://github.com/JerryX1110/RPCMVOS/blob/main/configs/resnet101_rpcm_ytb_stage_1.py and https://github.com/JerryX1110/RPCMVOS/blob/main/configs/resnet101_rpcm_ytb_stage_2.py.
I add some descriptions about the dataset path configs in these two config files. If you still have problems, please do not hesitate to let me know~ : )

You need to modify the dataset loading path and the .pth loading path in the https://github.com/JerryX1110/RPCMVOS/blob/main/configs/resnet101_rpcm_ytb_stage_1.py and https://github.com/JerryX1110/RPCMVOS/blob/main/configs/resnet101_rpcm_ytb_stage_2.py.
I add some descriptions about the dataset path configs in these two config files. If you still have problems, please do not hesitate to let me know~ : )

Thanks, I will try it!


fix(firestore): fix an issue that would cause FieldValue.increment to be interpreted as double

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
closes #12436
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Hi @Lyokone,
This didn't seem to fix the issue with the int increment as it is still resulting in a double stored in firestore.
increment(1) should result in a stored value of a int
increment(1.0) should result in a stored value of a double.

Hi @Lyokone, This didn't seem to fix the issue with the int increment as it is still resulting in a double stored in firestore. increment(1) should result in a stored value of a int increment(1.0) should result in a stored value of a double.

I can confirm this as an ongoing issue still. Since the value type changes from int to double in Firestore whenever FieldValue.increment is invoked, I have to patch the document using onDocumentWritten from Firebase Functions until a fix is in place.
Can this be looked at @Lyokone please?


[BUILD] Improve Makefile

Add the following improvements to the Makefile:

Massively improve build speed by letting 'make' use more CPU cores.

Added rule to automatically detect which features of Make can be used depending on which version is installed.

Separate build artifact files (*.o/*.d) into a separate directory.

Add dependency file creation to Makefile.

Now, whenever something in a header file changes, all source files that include this header file will recompile, even if nothing in the source files themselves changed.
This works by telling the compiler to separate the compilation process into 3 stages:
i.   Pre-processing:
Creates .d files which keep track of all header file dependencies.
ii.  Compiling:
Compiles the source files into object files.
iii. Linking:
Links the object files into an executable.

Add a print rule for Makefile debugging.

Let the project recompile when the Makefile changes.

Let the project recompile when any library files change.

Replace default print-outs with custom messages (🌊).

Improve readability of the Makefile by changing the layout and adding more whitespace and comments.

@LeaYeh It is now ready for review and merge 🌊


Nova Scotia English School Board

@Bombardier-C-Kram Please tell us more about using this email domain. Which schools have email addresses in this domain? How many schools are there? How many students and teachers (approximately) use the email addresses? What is official website of the Nova Scotia English Schools? Is this board email domain is common for all the schools in Nova Scotia?

This is the domain for all english schools in nova scotia. I really dont know how many people use the email. The website is: http://ns-schools.ednet.ns.ca/ but there are many "regional centers" in nova scotia.

@Bombardier-C-Kram https://en.wikipedia.org/wiki/List_of_schools_in_Nova_Scotia suggests that there are dozens of them. JetBrains does not generally provide the tools access to umbrella organizations, but you say that all the schools share the same domain, i.e. the domain is for students' and teachers' use.
Let's consider this case as an exception, and add this domain to the repository. Pull request merged.
Please notice that free educational subscription for JetBrains tools is for students and teachers only, and is valid for using the tools for educational purpose only. If employees of the schools who are not students, nor teachers, also need a license for JetBrains tools, they are eligible for an academic discount for the tools. They can get the discount by email to<EMAIL_ADDRESS>Thank you!


fix: accessing __asyncLoader property of component options

resolves #10744

The linked issue doesn't include a reproduction, so it's difficult to tell whether this is the correct fix. It's likely that this is just hiding the root cause of the problem rather than actually fixing it.
#6908 attempted a very similar fix, and hasn't been merged for the same reason.
If you're able to provide a reproduction it would be a massive help in getting this resolved.


Save Canvas content to png file, and anothers features.

With the Canvas release, it needs this features:

function to save the canvas content as a PNG file.
function to_bytes(get content as image bytes).
function to_imgb64(get content as base 64).
cv.Image to render images in the canvas.
image parameter for Paint, for drawing textured shapes.

https://gist.github.com/OPY-bbt/a5418127d8444393a2ef25ad2d966dc0

I wouldn't mind seeing a click through transparent canvas

I've also been waiting for the feature to save the drawn canvas shapes to a png file.  Doesn't seem like it'd be hard to do, I needed it for an image mask creator to draw in the areas to Inpaint within my AI software (DiffusionDeluxe.com), and I'd be implementing it throughout my project.  I already got it setup to load an init image to the canvas stack and drawing on top of it, but it's useless if I can't export a black & white image to use.  Right now I have to tell my users to create it in Photoshop and bring the mask file in manually, or use the Gradio based Mask Maker which is lame, and I'd really like to have it part of the builtin workflow.  If help is needed in the code I know how to do it in regular Flutter, but looks like most of it is figured out with that customPainter.dart using PictureRecorder...

With the Canvas release, it needs this features:随着 Canvas 版本的发布，它需要以下功能：

function to save the canvas content as a PNG file.函数将画布内容保存为 PNG 文件。
function to_bytes(get content as image bytes).函数 to_bytes（获取图像字节内容）。
function to_imgb64(get content as base 64).函数 to_imgb64（获取以 64 为基数的内容）。
cv.Image to render images in the canvas.cv.Image 在画布中渲染图像。
image parameter for Paint, for drawing textured shapes.Paint 的 image 参数，用于绘制纹理形状。

With the development of AI drawing, we also need this mask function and urgently need this canvas save function

I found this content about saving canvas to an image:
https://stackoverflow.com/questions/50320479/flutter-how-would-one-save-a-canvas-custompainter-to-an-image-file
https://github.com/rxlabz/flutter_canvas_to_image
Note: But I don't know if transparency is preserved.


[Netcode] Other players position updates are not "100%" smooth

Currently the previous and current positions are being kept track of client side. The client lerps between the previous and current positions with a value of T where T depends on the packet send rate.
Perhaps if we increase the number of positions we keep track of from 2 to 3 or maybe even 5 we can have more of a smooth effect client-side.

I just noticed (and I may be wrong about this) that the T value is sometimes not in sync and the player has to rejoin to get it in sync. The player has a rubberband effect when the T value is not in sync.

Resolved by 107b3f1064bb2e368f5e97c8b13eab8cf0f7d2a4. Additionally jitters have been fixed by 3b2af5c4b51a593d193b9cf7e2f2ee57dc1aedd4.


[gtest] provided find of gtest library by cmake find_package command.

Hello.
There is a build in FindGTest at CMake package, so it probably better to use it rather than manual link of this library.
Not sure that in this commit best practice of using this command, so let it be as some start point to discuss this theme.
Thanks.

I don't think it's a good idea since built-in FindGTest.cmake only searches for compiled library
It's not recommended to use precompiled gtests library since it can lead to undefined behaviour if the library and the project were compiled with different compiler flag (https://code.google.com/p/googletest/wiki/FAQ#Why_is_it_not_recommended_to_install_a_pre-compiled_copy_of_Goog)
Lantern's CMakeLists.txt searches for sources and compiles them every time

Yes. I know that

About gtest, I read some documents, that unfortunately I can not find right now, that idea that lib must have same compile flags like in full project. >>

Well similar documents founded.
But I even do not know where to get binary package, rather than make it by own. So code in this commit test with source package.
P.S.
Some proposals to discuss: it seems that for Linux checking of environment variable should not be and instead find must work on known place that provided at libgtest-dev, personally I use /usr/src/gtest/
For all platforms: if gtest library not provided do not add tests target.

Those are both good proposals (except that env variable should be checked on Linux anyway in case of fallback) and I'll merge a pull request with them if it will be implemented carefully and tested on different platforms

One of proposal prepered.
About testing: my list of different platforms consist only from Windows and Linux, so Mac OS X unable to check by me for now.


Making webtorrent stream using standalone script

https://github.com/feross/webtorrent/wiki/FAQ
I'm not getting this answer right, do you mean that if nobody seeds a torrent file, I can't upload it?
Can I stream and seed torrent using standalone webtorrent.min.js script?
I'm trying to make small site to stream video, just copying magnet link, do you have any example of that, if do, can you send it to me? Thank you very much! :)

Why does browser downloading not work?

It does work! But you can't just paste in any random magnet uri or .torrent file. Is the torrent being seeded by a webrtc-capable client, i.e. webtorrent-hybrid or instant.io?

In the browser, WebTorrent can only download torrents that are explicitly seeded to web peers via a WebTorrent capable client. Desktop torrent clients need to support WebRTC to peer with web browsers.

With WebTorrent, you can download any torrent from the network as long as there are seeders. This is how regular BitTorrent works too.
The point that the FAQ answer is trying to make is that you can't download any old torrent. There have to be "web peers" seeding it. Peers that support WebRTC and WebTorrent.

So, can I be seeder by using your script?

Yes, you can be a seeder with the WebTorrent script.


Creating a export named "scale" will change the actual scale of the Control node

Godot version
v4.1.1.stable.mono.official [bd6af8e0e]
System information
Windows 11, RX 6600, 10400 CPU
Issue description
If you create a [Export] named "scale" then the actual scale of the Control node will be reset.

Steps to reproduce

Create a new ColorRect node
Attach a script to it
In the script add the following

public partial class ColorRect : Node
{
    [Export] float scale;
}

Build the project
Notice in the inspector how the scale of the control node is now (0, 0)
Try commenting out the [Export] line and rebuild the project
Notice how the scale has gone back to normal (1, 1)

Minimal reproduction project
ScaleTest.zip

Can you reproduce this with GDScript?

Related:

https://github.com/godotengine/godot/issues/75554
https://github.com/godotengine/godot/issues/45961

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/75554 and https://github.com/godotengine/godot/issues/45961.


Fix invalid byte sequence in UTF-8 for sendgrid

There is an error when the filename for sendgrid contains an invalid character, for example "test\xc3\x28.pdf".

Can you add a spec for this please?

Hi, I just updated the PR. I think it's the best to convert all strings to UTF-8 before sending the data to rails. What I changed:

charset is automatically set to utf-8
convert all string values to utf-8 (check validity first, replace invalid characters if needed)

What do you think?

Closing this as this library has been deprecated: https://github.com/thoughtbot/griddler-sendgrid/pull/41


Fix estimate_read_length and add a test for it

The est_read_length function divides by the number of requested reads (500) to compute the average, but it should divide by the number of actually returned reads. This makes a difference in the test dataset because it has fewer than 500 reads.
This changes the mapping results for the test dataset since previously, different presets for the index were selected. I have updated the expected single-end SAM file, but not the paired-end one because this makes one of the "rescuable.42" reads unmappable. I don’t know why at the moment.

I replaced the sequence of the "rescuable.42" read with one that can be rescued again, so this is now ready.
Interestingly, using the different index parameters (read length 300), we no longer have any mapping qualities equal to zero. But figuring out why MAPQs work the way they do and/or fixing them is a different issue.


Update libconnman-qt and libqofono to latest upstream versions

Update to latest upstream versions.
Signed-off-by: Herman van Hazendonk<EMAIL_ADDRESS>

In master-next now, thanks
b61ea2c1..5144a99a  master-next -> master-next

Merged, thanks


Warning: Failed child... type number, expected string (ActionSheet)

react-native, react and native-base version
react-native 0.53
react 16.2.0
native-base 2.3.8
Expected behaviour
Work just fine without warnings (on Debug)
Actual behaviour
When activating the ActionSheet, the warning its displayed.
Steps to reproduce (code snippet or screenshot)

Screenshot of emulator/device
Is the bug present in both ios and android or in any one of them?
both platforms. iOS AND android
Any other additional info which would help us debug the issue quicker.
On the ActionSheet.js file, the keyExtractor returns the index, but it should return it converted to string
            <FlatList
              style={{ marginHorizontal: -15, marginTop: 15 }}
              data={this.state.items}
              keyExtractor={(item, index) => index}

should look:
            <FlatList
              style={{ marginHorizontal: -15, marginTop: 15 }}
              data={this.state.items}
              keyExtractor={(item, index) => String(index) }

Fixed with 2.3.9

The same is happening on Picker in ios

We're experiencing this with the Picker on iOS, too.

The problem is unresolved on iOS in the Picker
"native-base": "2.3.9",
"react": "16.2.0",
"react-native": "0.53.3",

@flindenberg @CristiTr this will be fixed in upcoming release

@akhil-geekyants, that's great news. Thank you.

Fixed with NB 2.3.10

Thanks! I can confirm that this issue was resolved.


Reflect the move of TokenTree

In the latest rust nightly TokenTree was moved out of syntax::ast into syntax::tokenstream
This changes the ioreg code accordingly.

LGTM, but I'm super hesitant to pull stuff without building it myself until the CI is fixed. If you want to work on getting that fixed up a bit too that would be awesome.

I'll have a look at the CI integration. On the meantime there are some compilation errors related to lifetimes in ioreg. I am new to rust so these are really obscure to me. Can you give it a look ?

ioregs broken again 😞 LGTM, but we still need a green build on at least one platform to merge.


SelectInput: Better Examples in Documentation for Custom Label on Opts

SelectInput should have an example of passing in a custom, jsx label for options.
Currently, the only examples show each option as:
{
  label: 'Muffin',
  value: 'muffin-01'
}

example should show something more like:
const opt = {
  label: (

          Muffin

  ),
  value: 'muffin-01'
}

Unfortunately, this implementation violates browser DOM nesting rules. When I try this, I get the following console error:
warning.js:33 Warning: validateDOMNesting(...):  cannot appear as a child of <option>.

@slamflipstrom That's definitely an issue, but doesn't prevent us from using this method for custom labels.
Created #15 to address this.


Project Admin / Project Viewer API access for audit events of a certain project

Currently, API access to audit events (audits table) is limited to users with the sitewide role of Administrator.
It could be useful to expose the audit events in the scope of a project to Project Admins and Project Viewers of that project, starting with the API, perhaps eventually used in a presentation on the Frontend.
One thing that this would enable is the processing/representation of submission deletions. Currently when one deletes a submission, I think there is no way for a Project Admin/Viewer to uncover any trace of that action, even if they did it themselves.

Hello, I tried to solve the issue.
This is what I did:
Modified the audits resource to allow Project Admins and Viewers to access audit events within their project scope. Added a new endpoint for project-specific audit events and updated the existing endpoints to check for project-level permissions.
You can review changes in this commit: https://github.com/lattaai13/getodk-central-backend-1300/commit/d07b40934273bf07b8141ad76786befd13072c3f.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


NPC invaders

Could you please add a section for the NPC invaders? https://eldenring.wiki.fextralife.com/NPC+Invaders
Note that some of them invade multiple times, at different locations.

Hello
Thank you for your suggestion but I believe this may be too niche for this project. However, feel free to fork this project and add it yourself to your own version.
This repo https://github.com/RoundtableHold/roundtablehold.github.io may be more willing to add such a feature you are looking for if that interests you as well.
Good luck


Update README for Swift Package Manager installation

Updated README by adding Swift Package Manager to Installation.

Thank you for the contribution.


Style for anchors

From @heruan on September 1, 2017 16:34
There's no style for anchor elements like Awesome link!. It would be great to have them style with the theme colors instead of the default browser ones!
Copied from original issue: vaadin/vaadin-themes#49

Thanks, that’s a very valid point. Should be included in the valo/material-colors style module.

Great, thank you! I would also remove the underline style when the link is not hovered, to differentiate the hover state (like here in GitHub).

Yeah, I briefly thought about that, but for now decided just to affect the colors. If we want to remove the underline, I wasn’t completely sure where I should put those rules. My gut feeling is that they would belong to typography.html, part of the valo-typography style module.

Darn, just noticed it’s not possible to change the opacity of link color between unvisited and visited links due to privacy reasons: https://developer.mozilla.org/en-US/docs/Web/CSS/Privacy_and_the_:visited_selector
I’ll try to think of another way to set the correct color. Maybe we need to introduce another color just for this :/

What about not differentiate visited from unvisited links, at all? In my opinion, it's no more what the user expects. I think there are three key factors about styling links:

they should be immediately recognizable, so go for the theme primary color;
underline them on hover, to express the fact the pointer is ready to trigger the action;
be consistent: if the user changes the device, or clears the history, links should not change the color, so it's better to just ignore the visited state and treat them as in 1.

Well, I never thought about links that way. But I do agree, the visited style is mostly a relic of a by-gone era nowadays. So I ended up doing just as you proposed, and links are always using the primary color, and only have the underline when hovered. Thanks for the suggestions!


Small Holocene fixes

Description & motivation 💭

Adds a loading slot for the ApiPagination component so that a custom loading element may be used in cases where it might not be a table
Adds stopPropagation to the on:click on the IconButton component (we use an IconButton on the Modal component and this prevents a button from being triggered if it is behind the Modal component)
Adds text-nowrap to the Badge component so that text does not wrap on small screens

Screenshots (if applicable) 📸
Design Considerations 🎨
Testing 🧪
How was this tested 👻

[x] Manual testing
[ ] E2E tests added
[ ] Unit tests added

Steps for others to test: 🚶🏽♂🚶🏽♀
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
Docs
Any docs updates needed?

why the button snaps changed when we didn't update that component

@rossedfort it's the Badge component we use for the count on the Button component (it now has the text-nowrap).


Release storage 1.10.0

This pull request was generated using releasetool.

@theacodes What was the reason for closing the PR?

I did it to demonstrate to our new engineering manager how releasetool
works.
On Wed, May 16, 2018 at 11:32 AM Tres Seaver<EMAIL_ADDRESS>wrote:

@theacodes https://github.com/theacodes What was the reason for closing
the PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/5330#issuecomment-389620706,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc1qatazIIKOq9zmfnP19hPQ3VA3wks5tzHCzgaJpZM4T_0ac
.


Project Page - block staff can also change period and status of project in project page

Describe the bug
https://www.berrycast.com/conversations/2b7bec99-145b-55e8-b9fa-2b33b2cfcf19

https://www.berrycast.com/conversations/2f67fb5e-4632-56fd-9ad5-5958adfaca16


Real-time IBKR mode

Support opening a live subscription to IB portfolio and trade data, and automatically updating bankroll with changes.

We should expand the interfaces of bankroll-marketdata as part of this work.


Remove Builtin.UnknownObject completely

The runtime now provides prebuilt value witnesses for AnyObject (mangled name yXl).

The demangler and remangler continue to support "BO" because it's convenient to be able to demangle old symbols.

Reflection still uses the notion of "unknown object" to mean an object with unknown refcounting. That's different from AnyObject because AnyObject is still an existential, meaning it contains an object with unknown refcounting.

The reflection tests are broken at the moment.

Type-based alias analysis for Builtin.UnknownObject was incorrect, so it's a good thing we weren't using it.

Same with enum layout. (This one assumed UnknownObject never referred to an Objective-C tagged pointer. That certainly wasn't how we were using it!)

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 238215cb8f770b719c161e38382e8c01a50713f2

@jrose-apple This is awesome, but I'm worried we're going to break reflection when binaries built with Swift 4.1 are used with an older version of the library.
The way it works is for "primitive" types, we emit "builtin type descriptors" which just state the size, alignment and extra inhabitants of the type.
You changed it to no longer emit this descriptor for UnknownObject, which makes sense, but I think it should now emit a descriptor for AnyObject, but probably under the old name so that the old library can find it.

Build failed
Swift Test OS X Platform
Git Sha - 238215cb8f770b719c161e38382e8c01a50713f2

@jrose-apple Take a look at lldb too, thankfully most of the usages look trivially removable:
source/Symbol/SwiftASTContext.cpp:    case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:                       m_ast_context_ap->TheUnknownObjectType.getPointer());
source/Symbol/SwiftASTContext.cpp:            referent_type->is<swift::BuiltinUnknownObjectType>() ||
source/Symbol/SwiftASTContext.cpp:    if (can_type == GetASTContext()->TheUnknownObjectType.getPointer())
source/Symbol/SwiftASTContext.cpp:      type_kind == swift::TypeKind::BuiltinUnknownObject)
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:    case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:    case swift::TypeKind::BuiltinUnknownObject:
source/Symbol/SwiftASTContext.cpp:  case swift::TypeKind::BuiltinUnknownObject:

Oops, thanks for the LLDB reminder! I'll probably need to talk to you about yXl vs BO, but this isn't high-priority.

Closing in favor of #27378.