Add itemized feedback for dep resolution in init

Compares manifest, lock and packageTree to separate direct and transitive
dependencies. Based on the constraints, logs them as feedback.
Creates ConstraintFeedback struct to hold dep constraint data.
Adds GetUsingFeedback() and GetLockingFeedback() functions to generate
feedback text.

Fixes #503

Added the feedback implementation in internal/internal.go for now. Not sure if it should be moved to a separate internal/ file or combined with some existing module ?

Output of init with this change:
$ dep init -no-examples
Cached golang.org/x/sys
Cached github.com/darkowlzz/openurl
Cached github.com/fsnotify/fsnotify
dep: Locking in master (ca83bd2cb9abb47839b50eb4da612f00158f5870) for transitive dep golang.org/x/sys
dep: Using master as constraint for direct dep github.com/darkowlzz/openurl
dep: Locking in master (673a04d14da017f965f0b97f87ab369ec4a1fd08) for direct dep github.com/darkowlzz/openurl
dep: Using master as constraint for direct dep github.com/fsnotify/fsnotify
dep: Locking in master (fd9ec7deca8bf46ecd2a795baaacf2b3a9be1197) for direct dep github.com/fsnotify/fsnotify

The revision is in its long form. Would the default git short SHA length (7) be good enough? Shorten with rev[0:7].

Cached golang.org/x/sys
Cached github.com/darkowlzz/openurl
Cached github.com/fsnotify/fsnotify

Let's also get rid of this feedback as part of the PR. We added it initially to provide some feedback to the user while work was being done, but the feedback we're adding here is much better.

The revision is in its long form. Would the default git short SHA length (7) be good enough? Shorten with rev[0:7].

Generally yes, but we can't actually rely on only having git SHA1s - there's also hg, bzr, svn, and whatever else we come up with in the future.
hg also has SHA1 revs, but svn has simple integers, and bzr has a weird thing. I'd say that it's fine to trim if you can positively identify something shaped like a SHA1, but in the other cases, leave it as-is. deduceConstraint() in ensure.go has some logic you could follow on that.

Changed the algo.
Check revision to for valid SHA1 digest and trims to 7 characters.
Remove "Cached" message.

This is how the feedback looks now:
$ dep init -no-examples
dep: Locking in v1.0.0 (ff2948a) for transitive dep github.com/sdboyer/deptest
dep: Using master as constraint for direct dep github.com/fsnotify/fsnotify
dep: Locking in master (fd9ec7d) for direct dep github.com/fsnotify/fsnotify
dep: Locking in master (ca83bd2) for transitive dep golang.org/x/sys
dep: Using ^2.0.0 as constraint for direct dep github.com/sdboyer/deptestdos
dep: Locking in v2.0.0 (5c60720) for direct dep github.com/sdboyer/deptestdos
dep: Using master as constraint for direct dep github.com/darkowlzz/openurl
dep: Locking in master (673a04d) for direct dep github.com/darkowlzz/openurl

Guess it's ready?

 So instead of just reviewing the code, I finally actually set this up and ran it locally. And I now realize there was a basic miscommunication - I'm sorry 😢
All of the output itself is looks good, but the timing is off. This provides the itemized list of choices after solving has already occurred. That was not what I had in mind; rather, I wanted this output to replace the existing output that's generated during the discovery process. So, that's all of the:

Found import of <project>, analyzing...
Could not determine version for <project>, omitting from generated manifest
Analyzing <project>...

This is why the spec has those header sections, e.g. Searching GOPATH for remaining projects... - I was imagining this output happening concurrently with each search process. Now, I realize that those headers will need to change a bit as the other parts get implemented (hopefully i'll get to #497 today), but unless you think it would be too difficult/cause a bunch of unnecessary work to achieve these incremental output changes until after the logic of init changes as well, then I'd prefer that all this output be moved up from the post-solve segment into the discovery segment.

@sdboyer I thought about it when I started implementing this. The feedback should be incremental. I realized that it would require changes in gps for instant feedback while resolving. But then, I thought these feedback should not be part of gps because they are dep specific. Hence, continued with this post-solve feedback.
Maybe the gps solver could be modified to attach some function that would be called to log these feedback at every solving iteration and keep gps generic?
About moving the output from post-solve segment to discovery segment, again I'm not sure how to get the solution data before running a solve.
On another thought, this is a network mode specific issue, I guess, which is the only mode at the moment 😬. VersionInWorkspace() could be used to fetch details of ondisk projects and feedback for those at the time of discovery is feasible. And only for the projects that require network, code in this PR (with some changes) can be used. Thoughts?
Not sure how to proceed from here. Maybe put this on hold until the other parts are implemented 🙂

oops! it can still be done. My bad. Right now, we do search $GOPATH and go for network for not on disk deps. Which means, feedback can be logged for them, no need to wait for -gopath PR 🤦♂ silly me 😅.

Separated network and $GOPATH based dep's feedback. $GOPATH based deps show up incrementally as they are discovered but network based deps show up together because they are received together from the solver. Still wondering if it would be a good idea to attach a function to the solver to make network deps discovery incremental 🤔
Feedback output still looks the same, with $GOPATH and network headers:
$ dep init -no-examples
dep: Searching GOPATH for projects...
dep: Using master as constraint for direct dep github.com/fsnotify/fsnotify
dep: Locking in master (fd9ec7d) for direct dep github.com/fsnotify/fsnotify
dep: Locking in master (ca83bd2) for transitive dep golang.org/x/sys
dep: Using network for remaining projects...
dep: Using master as constraint for direct dep github.com/darkowlzz/openurl
dep: Locking in master (673a04d) for direct dep github.com/darkowlzz/openurl
dep: Using ^2.0.0 as constraint for direct dep github.com/sdboyer/deptestdos
dep: Locking in v2.0.0 (5c60720) for direct dep github.com/sdboyer/deptestdos
dep: Locking in v1.0.0 (ff2948a) for transitive dep github.com/sdboyer/deptest

This obviously needs refactor after #525 (sorry 😞 ), but two nits for when you get around to it:

The dep:  prefix doesn't really help anything, let's drop it (which is the default anyway, after incorporating #525)
Let's prefix the individual discovery items with a two space indent, but leave the section "headers" (e.g. Searching GOPATH for projects...) unindented, so it's easier to visually scan and see when the different discovery modes are being used.

I'm prepping that feature branch - rebase, when you can? (plus those last couple nits).

Will rebase once #544 is merged. Import cycle issues even if I rebase now.

all merged up :)

@sdboyer squashed, rebased and made some adjustments to fit with the new code.

Created new internal package internal/feedback/.
Removed "dep" and added 2 space indent for feedback.

$ dep init
Searching GOPATH for projects...
  Using 673a04d as hint for direct dep github.com/darkowlzz/openurl
  Using master as constraint for direct dep github.com/fsnotify/fsnotify
  Locking in master (fd9ec7d) for direct dep github.com/fsnotify/fsnotify
  Locking in master (ca83bd2) for transitive dep golang.org/x/sys
Using network for remaining projects...
  Using ^2.0.0 as constraint for direct dep github.com/sdboyer/deptestdos
  Locking in v2.0.0 (5c60720) for direct dep github.com/sdboyer/deptestdos
  Locking in v1.0.0 (ff2948a) for transitive dep github.com/sdboyer/deptest

OK, new internal/feedback package seems fine enough for now. Thanks for keeping at this - in we go!!!

So, I just ran dep init -v, and realized that what we've put together here meshes really poorly with that output 😄  We need a follow-up to basically scrub out all of the pre-existing output (with the dep:  prefix), and have the only change with dep init -v be the trace output.
b/c yeah, this is super confusing:
dep: Finding dependencies for "github.com/Masterminds/glide"...
dep: Found 33 dependencies.
Searching GOPATH for projects...
dep: Building dependency graph...
dep: Found import of "github.com/Masterminds/semver", analyzing...
  Using range-prerelease as constraint for direct dep github.com/Masterminds/semver
  Locking in range-prerelease (8dec626) for direct dep github.com/Masterminds/semver
dep: Found import of "github.com/Masterminds/vcs", analyzing...
  Using master as constraint for direct dep github.com/Masterminds/vcs
  Locking in master (6abbba9) for direct dep github.com/Masterminds/vcs
dep: Found import of "github.com/codegangsta/cli", analyzing...
dep: Could not determine version for "github.com/codegangsta/cli", omitting from generated manifest
dep: Found import of "github.com/sdboyer/gps", analyzing...
  Using master as constraint for direct dep github.com/sdboyer/gps
  Locking in master (4d9c485) for direct dep github.com/sdboyer/gps
dep: Found import of "gopkg.in/yaml.v2", analyzing...
  Using 53feefa as hint for direct dep gopkg.in/yaml.v2
dep: Analyzing transitive imports...
dep: Analyzing "github.com/Masterminds/vcs"...
dep: Analyzing "github.com/codegangsta/cli"...
dep: Analyzing "github.com/sdboyer/gps"...
dep: Analyzing "github.com/Masterminds/semver"...
dep: Analyzing "github.com/armon/go-radix"...
dep: Analyzing "github.com/sdboyer/constext"...
  Locking in master (836a144) for transitive dep github.com/sdboyer/constext
dep: Analyzing "gopkg.in/yaml.v2"...
Using network for remaining projects...
dep: Solving...
Root project is "github.com/Masterminds/glide"
 17 transitively valid internal packages
 5 external packages imported from 5 projects
(0)   ✓ select (root)
(1)	? attempt github.com/Masterminds/semver with 1 pkgs; at least 1 versions to try
(1)	    try github.com/Masterminds/semver@range-prerelease
(1)	✓ select github.com/Masterminds/semver@range-prerelease w/1 pkgs
(2)	? attempt github.com/Masterminds/vcs with 1 pkgs; at least 1 versions to try
(2)	    try github.com/Masterminds/vcs@master
(2)	✓ select github.com/Masterminds/vcs@master w/1 pkgs
(3)	? attempt github.com/sdboyer/gps with 1 pkgs; at least 1 versions to try
(3)	    try github.com/sdboyer/gps@master
(3)	✓ select github.com/sdboyer/gps@master w/4 pkgs
(4)	? attempt github.com/sdboyer/constext with 1 pkgs; at least 1 versions to try
(4)	    try github.com/sdboyer/constext@master
(4)	✓ select github.com/sdboyer/constext@master w/1 pkgs
(5)	? attempt gopkg.in/yaml.v2 with 1 pkgs; at least 1 versions to try
(5)	    try gopkg.in/yaml.v2@53feefa2559fb8dfa8d81baad31be332c97d6c77
(5)	✓ select gopkg.in/yaml.v2@53feefa2559fb8dfa8d81baad31be332c97d6c77 w/1 pkgs
(6)	? attempt github.com/armon/go-radix with 1 pkgs; 1 versions to try
(6)	    try github.com/armon/go-radix@master
(6)	✓ select github.com/armon/go-radix@master w/1 pkgs
(7)	? attempt github.com/codegangsta/cli with 1 pkgs; 47 versions to try
(7)	    try<EMAIL_ADDRESS>(7)	✓ select<EMAIL_ADDRESS>w/1 pkgs
  ✓ found solution with 10 packages from 7 projects

Solver wall times by segment:
         b-list-pkgs: 2.558107961s
              b-gmal: 2.376535693s
         select-root:   1.541794ms
             satisfy:   1.206161ms
         select-atom:   1.109764ms
            new-atom:    158.866µs
     b-list-versions:     49.901µs
  b-deduce-proj-root:     23.482µs
               other:     21.256µs
     b-source-exists:      9.749µs

  TOTAL: 4.938764627s

  Using master as constraint for direct dep github.com/armon/go-radix
  Locking in master (4239b77) for direct dep github.com/armon/go-radix
  Using ^1.19.1 as constraint for direct dep github.com/codegangsta/cli
  Locking in v1.19.1 (0bdedde) for direct dep github.com/codegangsta/cli

oops! 😅 that's really a mess. Will create a follow-up PR.

cool, thank you :) i'm also inclined to revisit the enumerating of the items "over the network" - that's just listing out what ended up in the solution. this creates misleading output - what's read from GOPATH (and later, other projects' metadata) are hints/inputs that might change, whereas the "from network" output is what's actually been selected.
i think it may instead be better to enumerate the list of dependencies for which the local search method did not turn up a result, enumerate them, and indicate that the dep will instead simply rely on its "default most recent version" logic. Only, yknow, wordsmithed in a way that will be understandable for users 😄


Cleanup devilflippy's embedded mods

Prefer to release a zip on the concerned repository and add a direct link in the README.md to this zip.
Only the config should stay in The_Escape repository (for easier maintainance)

Better sales to Traders
Tarkov_Insurance&Repair_Services


Optionally enable antimalware scanning when done

With this change, the script offers to enable malware scanning when it is complete. Note that we always prompt before enabling. This is so the user can verify the script actually succeeded before enabling.
The prompt can be disabled by passing -DontEnableAntiMalwareScanning.

This is what the experience looks like with this change.

Do not merge this. This is still under discussion.

Closing for now. Will keep the branch alive in case we decide to proceed with this.


Big assets forming on build

Hi, I'm having a problem with the assets generated by the plugin. I don't know if it is a bug or if there is a way to stop this from happening.
You can see here that when I finish the building process I end up with this huge 12M asset file inside build/assets/js:

When I tried to look inside of it I realized that it had a bunch of names and other data:

So I took a look at the code to see where it is coming from and it seems that the postman-collection or postman-code generators dependencies are using faker to generate fake data, causing this asset to be huge. It this a bug? Is there a way not to include this unnecessary data?
If we cannot avoid this, can we configure the webpack to limit asset sizes?
Thank you in advance. (:

I'm not sure how to get rid of this as a dependency, open to PRs though if you have an idea


Issue when running

When I run the module I am getting the following error:
============= RESTART: C:/Users/thomas/Documents/Depop/depop.py =============
Traceback (most recent call last):
File "C:/Users/thomas/Documents/Depop/depop.py", line 1, in
from selenium import webdriver
ModuleNotFoundError: No module named 'selenium'

Any ideas?

Selenium is a python package. You need to have it installed on your system in order to run the script. I updated the instructions to include this step.
So try running pip install selenium in a terminal/command prompt.
Or with conda: conda install selenium
Here's some info about PIP and python packages.
Also FYI, if you search this error on google (i.e. ModuleNotFoundError: No module named 'selenium') the first result is exact what you need to do
Stack overflow is your friend.


no-dupe-class-members error

Hey 👋
As you know it's possible to overload functions and methods in TypeScript.
I receive no-dupe-class-members error when I try to overload class method:
class Foo {
  bar(arg0: string): void;
  bar(arg0: string, arg1: string): void;
  bar(arg0: string, arg1?: string) {
    // implementation
  }
}

Also TypeScript throws an error if we have duplicate identifiers in our code:
// Duplicate identifier 'bar'.
class Foo {
  bar = 1;
  bar = 1;
}

// Duplicate identifier 'bar'.
class Foo {
  get bar() {
    return 1;
  }
  get bar() {
    return 1;
  }
}

Maybe this rule can be disabled? Or there might be some unexpected consequences 🤔
Here are some links that might be useful:

ESLint no-dupe-class-members rule
Rule in Airbnb config

@iamturns wdyt?

:tada: This issue has been resolved in version 6.3.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hey @snacqs - thanks for the issue and PR! Very well written, I appreciate it all the info 👍
Rule is disabled now in 6.3.2 🎉


Add Heroku deploy button

Want to add the Heroku deploy again, since it seems easy enough to have.
TODO: Try to actually deploy this on Heroku.

https://devcenter.heroku.com/articles/heroku-button
https://devcenter.heroku.com/articles/app-json-schema

Issue ref: #326

Seems like I need to work in master to test this. b761e55


Shell connection eval statement does not propagate exit status

The Docker Machine docs say to do the following to connect your shell to a machine:
$ eval "$(docker-machine env default)"

However, one deficiency with this is that it doesn't propagate the correct exit status if the docker-machine env invocation fails. To see this for yourself, you can execute the following:
$ eval "$(docker-machine env bad_name)"; echo $?
Error trying to get host "bad_name": Loading host from store failed: Host does not exist: "bad_name"
0

One possible solution would be to recommend the following variation instead:
$ eval 'out="$(docker-machine env default)"'

Then you would have--
$ eval 'out="$(docker-machine env default)"'; echo $?
0
$ eval 'out="$(docker-machine env bad_name)"'; echo $?
Error trying to get host "bad_name": Loading host from store failed: Host does not exist: "bad_name"
1

This work-around is documented here, for example, where it says, "a way to take the exit status into account is to use the command substitution in a simple assignment: then the exit status of the assignment is the exit status of the last command substitution in the assignment(s)."
I think this would help because this assignment work-around isn't so obvious and easy to find. It would be especially helpful in a scripting context.

Hi @cjerdonek, thanks for the issue.
While I agree with the principle of the idea, in my experience having to remember and type correctly eval $(docker-machine env name) has already proven to be at the limit of what people are willing to tolerate.
In general, if the command in the subshell is successful, it will not print any output (to STDERR), but otherwise it will.  This is historically how we indicate this to the user.
You can, of course, still use your method in your own scripts.


Update gnark-crypto to latest develop

Felt.ToRegular was removed, so had to refactor some of our existing code as well.

In the pedersen_hash file, after these changes we won't be needing the lowPartMask and lowPartBits constants that is present currently. Those unused constants can also be removed in this PR.


Some weird parts in the code

Hi, All. First, thanks for the great work and the efforts to make it open-source!
Now, I'm trying to reproduce the results with the provided code.
While I was reviewing and running each file step-by-step, I observed some weird lines as denoted below (3rd block in ./autocast_experiments/data/process.ipynb).
negated_questions = json.load(open(f'negated_tf_questions.json'))
autocast_questions = json.load(open(f'autocast_questions.json'))
qid_to_question = {q['id']: q for q in autocast_questions}
qid_to_negation = {q['id']: q for q in negated_questions}
all_questions = []
for q in data:
    q['qtype'] = qid_to_question[q['question_id']]['qtype']
    if q['question_id'] in qid_to_negation:
        negated_q = copy.deepcopy(q)
        negated_q['question'] = qid_to_negation[q['question_id']]['negated']
        for day in q['targets']:
            day['target'] = 1 - float(day['target']) # flip the forecast probabilities
        if q['answers'][0] == 'yes':
            negated_q['answers'] = ['no']
        else:
            negated_q['answers'] = ['yes']
        all_questions.append(negated_q)

    all_questions.append(q)

This part is for modifying the values of negated_q from the original q.
However, the current code flips the probability of the original, not the negated one:
for day in q['targets']:
    day['target'] = 1 - float(day['target']) # flip the forecast probabilities

Hence, I think that it should be modified as follow:
for day in negated_q['targets']:
    day['target'] = 1 - float(day['target']) # flip the forecast probabilities

Do I miss something, or is the suggested modification right?
Thanks.

Thanks for catching this. (Note that this blurb of code was written after the experiments so the reported results are still valid.)


[Feature] 希望支持 ilogtail 收集日志输出至 Doris

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
https://github.com/alibaba/ilogtail/issues/935
Use case
No response
Related issues
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

give me

Is there any progress on this issue?


fixes  functions mock  when dependsOn resource

*Issue
Amplify mock function fails when it depends on another resources and resources not pushed yet.
Description of changes:

Added a condition to check if the resources pushed yet.

Added tests for the same.

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

Closing as its already fixed

Closing as its already fixed


Add dependabot configuration file

This automatically enables Dependabot to:

Submit pull requests for security updates and version updates for Composer dependencies.
Submit pull requests for security updates and version updates for GH Action runner dependencies.

For Composer dependencies, a preference is given to widen the version restrictions instead of updating them to a new minimum.
This is a deliberate choice as this package is a library, not an application.
The configuration has been set up to:

Run once a week.
Submit a maximum of 5 pull requests at a time.
If additional pull requests are needed, these will subsequently be submitted the next time Dependabot runs after one or more of the open pull requests have been merged.
The commit messages for PRs submitted by Dependabot will be prefixed according the unofficial conventions used in this repo up to now.
The PRs will automatically be labelled with an appropriate label as already in use in this repo.

Refs:

https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#versioning-strategy

Love it when the commit message is longer than the actual change 🎉


[Feature][UI Next] Project.

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

[x] List
[x] Project Overview
[x] Workflow Relationship
[x] Workflow Definition
[x] Task Definition
[x] Workflow Instance
[x] Task Instance
[x] Form Component
[ ] Dag
[x] SHELL @Amy0104
[x] SUB_PROCESS @labbomb
[ ] PROCUDURE @calvinjiang
[ ] SQL @calvinjiang
[x] SPARK @Amy0104
[ ] FLINK @devosend
[x] MR @Amy0104
[x] PYTHON @calvinjiang
[ ] DEPENDENT @Amy0104 something wrong
[ ] HTTP
[ ] DATAX @labbomb
[ ] PIGEON @labbomb
[ ] SQOOP
[ ] CONDITIONS
[ ] DATA_QUALITY
[ ] SWITCH
[ ] SEATUNNEL

Use case
No response
Related issues
#7332
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I will develop the form of python task.

I will develop the form of flink task.

I will develop the form of SUB_PROCESS task.

I will develop the form of SQL task.

I will develop the form of PROCEDURE task.

I will develop the form of SUB_PROCESS、 DATAX、PIGEON task.

YYDS

HTTP,SQOOP and CONDITIONS.

I will develop the form of seatunnel task.

I will develop the form of the switch task.

I will develop the form of the switch task.

thanks

I'll develop the form of the DATAX task.

I'll develop the tree of the workflow definition.


Update log-analytics-cross-workspace-search.md

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

@yorac - I have reviewed your proposed change and am fine with it.  Thanks for making the time to submit it.  #sign-off


Update additional-office365-ip-addresses-and-urls.md

Microsoft Azure data center IP as De[recating. so added the new information

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3796

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3796

@JohanFreelancer9 Please copy review

@JohanFreelancer9 Please copy review

@JoeDavies-MSFT Please merge this PR. Thanks.

@JoeDavies-MSFT Please merge this PR. Thanks.


The units-scooter right camera zip download link is 'corridors_walk1.synced.right_camera.zip'

Fixed. You might need to reset your cookies on your browser :)
Thanks for pointing that out!


Implement -N (--no-follow-symlinks).

Implement a -N flag which causes yara to not follow any symlinks when scanning,
regardless of the recursive option.

This fixes #1347 - but it does so in a manner which I think is better than the suggestion. With the -N flag you can turn off symlinks when scanning, regardless of the recursive option.

Here's an example of the behavior.
wxs@wxs-mbp yara % cat rules/a.yara
import "hash"

rule a {
  condition:
    // This is the sha256 of /bin/ls on my system.
    hash.sha256(0, filesize) == "43969ad2a7eaf877f6852423f6ad803d7d89183ac152983d8d71e34d91c78079"
}
wxs@wxs-mbp yara % ls -l ~/tmp
total 104
lrwxr-xr-x  1 wxs  wxs      4 Sep  3 15:47 foo@ -> /bin
lrwxr-xr-x  1 wxs  wxs      7 Sep  3 15:48 ls@ -> /bin/ls
-rwxr-xr-x  1 wxs  wxs  51888 Sep  3 22:28 ls.copy*
wxs@wxs-mbp yara % ./yara rules/a.yara ~/tmp
a /Users/wxs/tmp/ls.copy
a /Users/wxs/tmp/ls
wxs@wxs-mbp yara % ./yara -r rules/a.yara ~/tmp
a /Users/wxs/tmp/ls.copy
a /Users/wxs/tmp/foo/ls
a /Users/wxs/tmp/ls
wxs@wxs-mbp yara % ./yara -rN rules/a.yara ~/tmp
a /Users/wxs/tmp/ls.copy
wxs@wxs-mbp yara %

The CIFuzz check is failing to build due to missing flex. This looks like it has nothing to do with my changes.


support upstream tls

Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

[optional Relevant Links:]

Any extra documentation required to understand the issue.

Plz provide more infos, then we can have a deep understanding on the background of this issue.

the current eg version can not support upstream HTTPS, when a request  call the upstream with HTTPS the following error occurs
Client sent an HTTP request to an HTTPS server.
@Xunzhuo

Moving to the backlog since the dependent upstream issue, kubernetes-sigs/gateway-api#91, is not fixed.

above issue is reg downstream TLS, https://github.com/kubernetes-sigs/gateway-api/issues/1067 tracks upstream TLS from gateway to backend

using annotation or labels like upstream/tls: true supports upstream tls before gwapi support?

this is being addressed in upstream https://github.com/kubernetes-sigs/gateway-api/pull/1906 right now, lets wait for upstream to define this feature

closing in favor of https://github.com/envoyproxy/gateway/issues/2033


Fix: the problem that the li shows 2 incomplete numbers #103

Description
Fix the problem that the li shows 2 incomplete numbers
修复列表显示2位数不全的为题

Linked Issues
#148
Additional context
Maybe DIu missed the merge(〃'▽'〃)

Please remove the extra code and resubmit the PR, Thanks.

Maybe using list-style-position: inside; is a better choice. Then setting left padding to a smaller value may have a good-looking.

Maybe using list-style-position: inside; is a better choice. Then setting left padding to a smaller value may have a good-looking.

Maybe using list-style-position: inside , this adjustment is correct.

list-style-position: inside will cause overflow, as the marker occupied space for paragraph. That's why use 2em padding, #148

I noticed that situation, but it's better than the list item number partially displayed.

And using 2em padding doesn't solve the problem perfectly. I found another way to solve it: delete the overflow-hidden class from the parent element of the ol. But I'm not exactly sure what this class does. It seems that the ChatGPT official website does not have such a situation, and it does not use styles like overflow: hidden.

@Hime-Hina Be careful of overflow caused by long lines of code.
https://github.com/ddiu8081/chatgpt-demo/commit/89a8a2732caf13bec37ac17673c1206a23d188ed the overflow-hidden seems used to limit the code block.

The message block needs to inherit the remaining space of the flex container, ChatGPT WebUI has set a fixed width to achieve this.


Add nacos dependencies to dubbo-demo

What is the purpose of the change
Add nacos dependencies to dubbo-demo
Brief changelog
Add nacos dependencies to dubbo-demo
Verifying this change
Maven test passed
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4447 into 2.7.3-release will decrease coverage by 0.04%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             2.7.3-release    #4447      +/-   ##
===================================================
- Coverage             63.5%   63.46%   -0.05%
+ Complexity             452      451       -1
===================================================
  Files                  765      765
  Lines                32690    32696       +6
  Branches              5149     5151       +2
===================================================
- Hits                 20761    20751      -10
- Misses                9542     9556      +14
- Partials              2387     2389       +2

Impacted Files
Coverage Δ
Complexity Δ

...onfig/spring/extension/SpringExtensionFactory.java
78.04% <0%> (-7.32%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

...e/dubbo/remoting/transport/netty/NettyChannel.java
57.64% <0%> (-4.71%)
20% <0%> (-1%)

...pache/dubbo/registry/support/AbstractRegistry.java
78.54% <0%> (-3.07%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

...n/java/org/apache/dubbo/common/utils/NetUtils.java
67.05% <0%> (-1.62%)
0% <0%> (ø)

...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java
80.14% <0%> (-0.74%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
67.39% <0%> (+4.34%)
0% <0%> (ø)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
71.42% <0%> (+9.52%)
0% <0%> (ø)
:arrow_down:

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
56.81% <0%> (+11.36%)
0% <0%> (ø)
:arrow_down:

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 a229d87...e213204. Read the comment docs.


MENDER_DISK_IMAGE should not be in the input/ directory

Signed-off-by: Drew Moseley<EMAIL_ADDRESS>

No.  Just general confusion.


03/27/2018 06:57 : Create Pull Request for branch master -> prTesting1_2018032706

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


tickets/SITCOM-528

Add Butler secret to UWS system.

The only other deployment is the int cluster which I believe was at NCSA. Are there supposed to be others? I can fix this to  make things a bit more flexible for SQLite vs Postgres. I'll need to work on the server as well. When are we going to get someone from DM to handle this system or this basically a telescope only thing now?

KT, please checkout the last commit as this shifts the config to something more flexible.


Update New-CsTeamsCallingPolicy.md

We are in the process of integrating two new settings under TeamsCallingPolicy. While the policy settings are available via the Powershell cmdlets, these settings are not yet ready to be consumed via Teams customers. Adding "reserved for internal Microsoft use" to avoid any confusion related to public use of these two settings.

Docs Build status updates of commit 8839d47:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

@dariomws @get-itips @tseward

Docs Build status updates of commit 2ced54f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

Docs Build status updates of commit 6221ed2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

Docs Build status updates of commit 6221ed2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

Docs Build status updates of commit 578eeff:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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

@tiburd Please merge this PR. Thanks


Getting a console error in Safari

Live site (click on 'Get in Touch' at the top right)
The icon is supposed to morph but it throws out a console error:
TypeError: 'undefined' is not a function (evaluating 'u(this._rafid)')
Works okay in Chrome, FF, and IE, though. Btw, I'm on Windows.

@lakeebs Does that occur on Safari for Windows? That's a dead browser. Please re-open the issue, if that appeared on iOS or OSX.


Show name of city/region being displayed

It would be great if we can see the name of the city/region shown somewhere on the screen.

Duplicate of #58.


Can not connect to Neo4j.

I use neo4j/neo4j to login. It returned Error : Neo.ClientError.Security.AuthenticationRateLimit: The client has provided incorrect authentication details too many times in a row.

This is expected. The default lock out is 5s:
https://neo4j.com/docs/operations-manual/current/reference/configuration-settings/#config_dbms.security.auth_lock_time

I tried to log in with correct username and password a few hours later,but it still return this error.

This looks to be something related to authentication/ username/ password rather than anything specific to Neo4j's official Docker containers.
If you have a support contract, I'd open a can with Neo4j Support. If not, you could reach out to the Community forum.

Came across this issue after a similar experience. I have neo4j running on the neo4j:5.11.0-community-bullseye image.

Use the incorrect credentials a few times to trigger the error
Wait longer than 5s, or several hours
Attempt to sign in with the correct credentials
See the Neo.ClientError.Security.AuthenticationRateLimit: The client has provided incorrect authentication details too many times in a row message


Use open source quantum chemistry software instead of Gaussian

What should we add?
Unless there is a compelling reason, please use open source quantum chemistry software for quantum chemistry.
This page has a partial list of open source quantum chemistry software.

The answer is here.
Thank you.

Nature support various drivers - see https://qiskit.org/documentation/nature/getting_started.html#optional-installs This includes open-source quantum chemistry software too. Ok you closed it just as I was answering, so I'll leave it as-is. Yes, similar answer to your other issue.

@woodsp-ibm
A quick followup question: You support PyQuante. But PyQuante seems to be dead. The older SourceForge-hosted version requires EOLed Python-2.7 And the newer GitHub-hosted fork https://github.com/rpmuller/pyquante2 seems to be dead as well.
Do you have a recommended fork that is supported?

Yes, it seems to no longer be actively supported. Like the other drivers its an optional install - it does work still but as you say is not actively maintained. In the early days its pure python version, was the only driver option for Windows platform besides the commercial Gaussian16. Nowadays PSI4 supports Windows so there is that option, though PyQuante has the more direct interface and perhaps is a little easier to use albeit its more limited function. Certainly PSI4 is a much more performant/capable software. The optional install instructions refer to and point at Pyquante2 link you show, I am not aware of any active fork of that code.


No volumes were detected when running postgres on Docker

Describe the bug
First of all, it is really great starter repo when doing a kickstart backend development on Nest.js but I have something for improvements, It's not really a bug, it's mostly about improvements when running the app locally on docker and sometimes we need to restart everything when a new change comes in. Although I'm fully aware that this repository is not running on docker for development purposes more just for testing or utilizing it.
I'm just gonna correct the docker-compose file to add the volumes correctly

https://github.com/brocoders/nestjs-boilerplate/pull/1132


chore: nextjs 13

Closes #1170
@elitan any possible breaking change introduced by Next 13?

Codecov Report

Merging #1172 (53bdc29) into main (dafc581) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1172   +/-   ##
=======================================
  Coverage   94.76%   94.76%
=======================================
  Files          48       48
  Lines        6912     6912
  Branches      445      445
=======================================
  Hits         6550     6550
  Misses        362      362

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

These are the breaking changes:

https://nextjs.org/blog/next-13#breaking-changes

The minimum React version and the minimum Node version bumps may be the most important.

These are the breaking changes:

https://nextjs.org/blog/next-13#breaking-changes

The minimum React version and the minimum Node version bumps may be the most important.

but using the right version of Node and React with Next 13 is not the Nhost SDK problem right?
I meant breaking changes specific to the interaction between Next 13 and the SDK

One note though: Maybe we want to set the required peer dependency version to ^12.0.10 || ^13.0.0 so that any Next.js 13.x version would be good for the SDK as a peer dependency.

yes, done


Background colors for IESG agenda documents

What happened?
Agenda documents I'd balloted on used to have background colors -- light green for NoObj, light pink for DISCUSS, etc. These are gone now. They were genuinely useful, I hope they can be brought back.
What browser(s) are you seeing the problem on?
Not Applicable
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

(my recollection is that the background color functionality was only on the "documents" agenda page (https://datatracker.ietf.org/iesg/agenda/documents/) but not the main agenda (https://datatracker.ietf.org/iesg/agenda/), and indeed was one of the differentiating features that cause the "documents" page to be widely preferred.  Not that we need to go for bug-for-bug compatibility with the previous state, of course...)

I loved that feature too!

There is a colored bar to the right of a document indicating what you balloted - too subtle?

There is a colored bar to the right of a document indicating what you balloted - too subtle?

It took me a while to spot it and it is enough for me (having such a bar on the left as well would be a plus)

@larseggert any objection to the color on both sides?

The PR has a different fix, i.e., restores the colored backgrounds.


Prove that any theory can be extended to a complete theory

This should be a simple Zorn's lemma/ultrafilter lemma argument; also needed in completeness.

Note: use exists_ultrafilter in mathlib.order.filter


Prevent two different processes from opening the same sync'ed realm

Right now there is nothing preventing it but it will cause undefined behavior..

It's essentially impossible for this to happen on iOS/tvOS/watchOS due to Realm managing where the synced Realm is stored on the file system, but this could feasibly happen on macOS. I don't know about Android.

It looks like realm/realm-core#2313 is laying the groundwork for this.

Thanks for spotting the connection @bdash :-)

realm/realm-core#2313 was in the realm-core v2.2.0 release, which should eliminate any potential for undefined behavior.


Write documentation for revad.toml

Various sections, what they mean and do, how to edit them to achieve various results

Reva should be a black box to sysadmins, but it's not. You can currently only use it and configure it if you know how to debug the code.

Opened as https://github.com/cs3org/reva/issues/3276

Thanks, yes, the two issues make exactly the same point! Closing this one as a duplicate and also as being in the wrong repo. :)


Do not add data to POST body in Container::start method.

Starting from v1.24, the API does not anymore accept data in the request body.

Thanks for the PR @jsimonet , maybe we must check api version globally and take some actions in consecuence.


Error 400 - Bad request

Getting the following in the logs:-
[3/3/2021, 9:11:52 AM] [homebridge-hyundai-bluelink] Client Error Response code 400 (Bad Request)
VIN, Pin, Username and password all correct.
Any ideas?
HB - V1.3.1
Plugin - latest

Can you run in debug mode to see if anything else is logged

Will do!
I'll upload what I find.

Here is a debug log:-
[3/3/2021, 2:42:08 PM] Loading 4 platforms...
[3/3/2021, 2:42:08 PM] [Config] Initializing config platform...
[3/3/2021, 2:42:08 PM] [Config] Running in Service Mode
[3/3/2021, 2:42:08 PM] [Ring] Initializing Ring platform...
[3/3/2021, 2:42:08 PM] [Govee] Initializing Govee platform...
[3/3/2021, 2:42:08 PM] [Hyundai] Initializing Hyundai platform...
DEPRECATED user supplied a custom 'mdns' option. This option is deprecated and ignored. Please move to the new 'bind' option.
Preparing Advertiser for 'Homebridge 175A 10CE' using ciao backend!
Setup Payload:
X-HM://0024IENMZPDMF
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:
┌────────────┐
│ XXXXXXXXX
└────────────┘

Starting to advertise 'Homebridge 175A 10CE' using ciao backend!
[3/3/2021, 2:42:08 PM] Homebridge v1.3.1 (Homebridge 175A) is running on port 51882.
Initializing HAP-NodeJS v0.9.2...
Initializing HAP-NodeJS v0.9.2...
Initializing HAP-NodeJS v0.9.2...
[3/3/2021, 2:42:09 PM] [homebridge-ring] Launched external bridge with PID 73533
[3/3/2021, 2:42:09 PM] [homebridge-hyundai-bluelink] Launched external bridge with PID 73535
[3/3/2021, 2:42:09 PM] [Govee] Launched external bridge with PID 73534
[3/3/2021, 2:42:09 PM] Registering platform 'homebridge-govee.Govee'
[3/3/2021, 2:42:09 PM] [Govee] Loaded homebridge-govee v2.13.0 successfully
[3/3/2021, 2:42:09 PM] Loaded 2 cached accessories from cachedAccessories.0E4C385F1E42.
DEPRECATED user supplied a custom 'mdns' option. This option is deprecated and ignored. Please move to the new 'bind' option.
Preparing Advertiser for 'Govee 00AB' using ciao backend!
[3/3/2021, 2:42:09 PM] [Govee] [v2.13.0] initialised. Syncing with Govee.
Starting to advertise 'Govee 00AB' using ciao backend!
[3/3/2021, 2:42:09 PM] Homebridge v1.3.1 (Govee) is running on port 31965.
[3/3/2021, 2:42:09 PM] Registering platform 'homebridge-hyundai-bluelink.Hyundai'
[3/3/2021, 2:42:09 PM] [homebridge-hyundai-bluelink] Loaded homebridge-hyundai-bluelink v1.2.11 successfully
[3/3/2021, 2:42:09 PM] Loaded 0 cached accessories from cachedAccessories.0E6376EC5A25.
DEPRECATED user supplied a custom 'mdns' option. This option is deprecated and ignored. Please move to the new 'bind' option.
Preparing Advertiser for 'homebridge-hyundai-bluelink 2165' using ciao backend!
[3/3/2021, 2:42:09 PM] [homebridge-hyundai-bluelink] Executed didFinishLaunching callback
[3/3/2021, 2:42:09 PM] [homebridge-hyundai-bluelink] Hyundai config: {
credentials: {
username<EMAIL_ADDRESS>password: 'xxxxxxxx',
region: 'EU',
pin: 'xxxxxx'
},
vehicles: [ { vin: 'XXXXXXXXXXXXXX', maxRange: 280 } ],
remoteStart: {
cool: true,
heat: true,
defrost: true,
temperature: 73,
igniOnDuration: 15
},
refreshInterval: 20,
platform: 'Hyundai'
}
Starting to advertise 'homebridge-hyundai-bluelink 2165' using ciao backend!
[3/3/2021, 2:42:09 PM] Homebridge v1.3.1 (homebridge-hyundai-bluelink) is running on port 36118.
[3/3/2021, 2:42:09 PM] Registering platform 'homebridge-ring.Ring'
[3/3/2021, 2:42:09 PM] [homebridge-ring] Loaded homebridge-ring v9.15.3 successfully
[3/3/2021, 2:42:09 PM] Loaded 2 cached accessories from cachedAccessories.0EBCAADCC62D.
[3/3/2021, 2:42:09 PM] [homebridge-ring] Configuring cached accessory b17088ea-0267-418f-974b-bfae9e597f1b Front Door
[3/3/2021, 2:42:09 PM] [homebridge-ring] [Circular]
[3/3/2021, 2:42:09 PM] [homebridge-ring] Configuring cached accessory 98104e03-906b-4993-a1d0-cf09ec3d4b58 Downstairs
[3/3/2021, 2:42:09 PM] [homebridge-ring] [Circular]
DEPRECATED user supplied a custom 'mdns' option. This option is deprecated and ignored. Please move to the new 'bind' option.
Preparing Advertiser for 'homebridge-ring 2D05' using ciao backend!
[3/3/2021, 2:42:09 PM] [homebridge-ring] didFinishLaunching
Starting to advertise 'homebridge-ring 2D05' using ciao backend!
[3/3/2021, 2:42:09 PM] Homebridge v1.3.1 (homebridge-ring) is running on port 45422.
[3/3/2021, 2:42:10 PM] [homebridge-hyundai-bluelink] Client Error Response code 400 (Bad Request)
[3/3/2021, 2:42:10 PM] [Govee] [Strip Light] initialised with id F1:EA:A4:C1:38:21:AB:04.
[3/3/2021, 2:42:10 PM] [Govee] [Bed Strip] initialised with id FE:9D:A4:C1:38:46:32:A6.
[3/3/2021, 2:42:10 PM] [Govee] ✓ Setup complete. This plugin has been made with ♥ by bwp91 from the UK!
[3/3/2021, 2:42:11 PM] [homebridge-ring] Found the following locations:
[3/3/2021, 2:42:11 PM] [homebridge-ring]   locationId: gcp386-2n5lu-0 - 19 Woodcutters Dr, Waterlooville PO8, UK
I only run four plugins so hopefully the log isn't too hard to read....
Cheers!

Thanks! I was hoping Bluelinky would provide more information in the debug logs, but it seems that's all we have to work with :(. From looking at their past issues, it seems there might be something with your Hyundai account needing to be upgraded? https://github.com/Hacksore/bluelinky/issues/89
Hope that helps!

Great! I'll take a look, thank you very much :)

@marksthomas curious what the current status is if you don't mind sharing

Hey.
I had it uninstalled. Just tried again and log full of this:-
[4/26/2021, 10:24:47 PM] [Hyundai] Client Error K [ManagedBluelinkyError]:
@EuropeController.login: [400] Bad Request on [POST]
https://prd.eu-ccapi.hyundai.com:8080/api/v1/user/signin -
{"errId":"24fe0b63-1f9d-4ccf-b888-35cd22e24f53","errCode":"4003","errMsg":"Invalid
values","errBody":{"userId":"","remainCount":3,"remainTime":300}}
at d

(/usr/local/lib/node_modules/homebridge-hyundai-bluelink/node_modules/bluelinky/dist/index.js:17:144947)
at Y.<anonymous>

(/usr/local/lib/node_modules/homebridge-hyundai-bluelink/node_modules/bluelinky/dist/index.js:17:169022)
at Generator.throw (<anonymous>)

at P

(/usr/local/lib/node_modules/homebridge-hyundai-bluelink/node_modules/bluelinky/dist/index.js:17:136)
at processTicksAndRejections (node:internal/process/task_queues:94:5) {

source: GotError [HTTPError]: Response code 400 (Bad Request)
  at EventEmitter.<anonymous>

(/usr/local/lib/node_modules/homebridge-hyundai-bluelink/node_modules/got/source/as-promise.js:74:19)
  at processTicksAndRejections (node:internal/process/task_queues:94:5)

{
host: 'prd.eu-ccapi.hyundai.com:8080',

hostname: 'prd.eu-ccapi.hyundai.com',

method: 'POST',

path: '/api/v1/user/signin',

socketPath: undefined,

protocol: 'https:',

url: 'https://prd.eu-ccapi.hyundai.com:8080/api/v1/user/signin',

gotOptions: {

  path: '/api/v1/user/signin',

  protocol: 'https:',

  slashes: true,

  auth: null,

  host: 'prd.eu-ccapi.hyundai.com:8080',

  port: '8080',

  hostname: 'prd.eu-ccapi.hyundai.com',

  hash: null,

  search: null,

  query: null,

  pathname: '/api/v1/user/signin',

  href: 'https://prd.eu-ccapi.hyundai.com:8080/api/v1/user/signin',

  retry: [Object],

  headers: [Object],

  hooks: [Object],

  decompress: true,

  throwHttpErrors: true,

  followRedirect: true,

  stream: false,

  form: false,

  json: true,

  cache: false,

  useElectronNet: false,

  method: 'POST',

  body: '{"email":"XXXXXXX","password":"XXXXX"}',

  cookieJar: [CookieJar]

},

statusCode: 400,

statusMessage: 'Bad Request',

headers: {

  'ccsp-request-id': '24fe0b63-1f9d-4ccf-b888-35cd22e24f53',

  'ccsp-span-id': '4',

  'content-length': '149',

  'content-type': 'application/json',

  date: 'Mon, 26 Apr 2021 21:24:47 GMT',

  vary: 'Origin',

  'x-ratelimit-limit': '0',

  'x-ratelimit-remaining': '0',

  'x-ratelimit-reset': '0',

  connection: 'close'

},

body: {

  errId: '24fe0b63-1f9d-4ccf-b888-35cd22e24f53',

  errCode: '4003',

  errMsg: 'Invalid values',

  errBody: [Object]

}

}
}
This is different than what happened before.
Cheers.
Mark
On Mon, 26 Apr 2021 at 21:53, Andrew Thal @.***> wrote:

@marksthomas https://github.com/marksthomas curious what the current
status is if you don't mind sharing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/8#issuecomment-827139850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYIHSRYG65UXURREORPVPLTKXHFLANCNFSM4YQ3IEWQ
.

--
Mark Thomas

Weird. Bluelinky had closed that issue, I guess it's not fully fixed

No worries.
It's disabled atm but if you need me to test again just let me know.
Mark
Mark Thomas
On Tue, 27 Apr 2021 at 15:45, Andrew Thal @.***> wrote:

Weird. Bluelinky had closed that issue, I guess it's not fully fixed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/8#issuecomment-827663536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYIHSTKGCKDLTKDL3J5ED3TK3EZ3ANCNFSM4YQ3IEWQ
.

Hey,
So it just started working. I can see the car in the Home app. A bit slow
to react but it's a great start :)
Any way to have it show the range in miles remaining instead of Km?
--
Mark Thomas
On Tue, 27 Apr 2021 at 15:54, Andrew Thal @.***> wrote:

Closed #8 https://github.com/athal7/homebridge-hyundai-bluelink/issues/8
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/8#event-4651370609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYIHSRMW56RR5VRPI2PTSDTK3F33ANCNFSM4YQ3IEWQ
.

Glad to hear it! Show the range where in km?

It’s fine. Answered my own question.
In the logs it’ll show max range. I just put “m”i after the max range in
the settings and it reports correctly.
On Wed, 5 May 2021 at 00:12, Andrew Thal @.***> wrote:

Glad to hear it! Show the range where in miles vs km?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/8#issuecomment-832306207,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYIHSXGDZ7J3BQYO7LTZBTTMB5NXANCNFSM4YQ3IEWQ
.

--
Mark Thomas

Oh in the logs. I think that's based on the settings you have setup with bluelink, nothing. I wouldn't recommend putting letters in the settings, that value is used for calculating % distance left

Ah ok. Got you.
On Wed, 5 May 2021 at 12:42, Andrew Thal @.***> wrote:

Oh in the logs. I think that's based on the settings you have setup with
bluelink. I wouldn't recommend putting letters in the settings, that value
is used for calculating % distance left
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/athal7/homebridge-hyundai-bluelink/issues/8#issuecomment-832621447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYIHSS4XKKCD357SPNPFZ3TMEVIVANCNFSM4YQ3IEWQ
.

--
Mark Thomas


Update label color

Summary by CodeRabbit

Style

Updated label color for dynamic placeholders to enhance visual consistency with new design specifications.

Wrong solution : We do not need a colo for label/placeholder. Use filter instead.


Introduce resources_err! error macro

Which issue does this PR close?

Closes #.
Related to https://github.com/apache/datafusion/issues/7360
Rationale for this change

Expand error macros series in DataFusion to include a ResourcesExhausted error
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

It would be better to replace the following but also be done on a follow-on.
https://github.com/apache/arrow-datafusion/blob/37428bb2034abc1aba5d4cd52cf8f2885623bada/datafusion/physical-plan/src/aggregates/mod.rs#L1881-L1884
https://github.com/apache/arrow-datafusion/blob/37428bb2034abc1aba5d4cd52cf8f2885623bada/datafusion/physical-plan/src/aggregates/row_hash.rs#L606-L610

Thanks folks for the review


Cannot use pub with --preview-dart-2

> DART_VM_OPTIONS=--preview-dart-2 pub
Invalid isolate state - Unable to make it runnable

On Dart VM version: 2.0.0-dev.26.0 (Thu Feb 15 03:06:13 2018 +0100) on "macos_x64"

CC @a-siva @nex3 @zanderso

in the dart sdk builds pub, analyzer, dart2js and ddc use an application snapshot to run, these snapshots are built in dart 1 mode. When preview-dart-2 is specified it is not compatible with these snapshots and hence the failure. The error message is not very helpful and we will fix that.
To get these command with --preview-dart-2 first the pub, dart2js, analyzer code needs to be made strong mode clean and these application snapshots need to be generated with that option turned on. I think  we planned to do this in Q2.
The issue that the original bug wanted addressed was a way for folks to run tests in preview-dart-2 mode using pub. For this to work 'pub run' needs to implement the --preview-dart-2 option so that it starts the test process using this option, somewhat similar to the '-c' or '--checked' option that it currently supports.
sivalinuxmach[sdk]>out/ReleaseX64/dart-sdk/bin/pub run --help
Run an executable from a package.
Usage: pub run  [args...]
-h, --help            Print this usage information.
-c, --[no-]checked    Enable runtime type checks and assertions.
--mode            Mode to run transformers in.
(defaults to "release" for dependencies, "debug" for entrypoint)

Who needs to own this issue of implementing the option '--preview-dart-2' in 'pub run' ?
Not sure who owns pub these days.

We have no plans, or bandwidth, to work on pub client in q1.

related to dart-lang/pub#1807

@nex3 @grouma @natebosch Can you all figure out what's needed here?

/cc @kmillikin - there are likely changes need to CFE to make it run on Dart 2.

/cc @kmillikin - there are likely changes need to CFE to make it run on Dart 2.

FYI, I was able to locally run DDC and DDK with --preview-dart-2. All tests were passing. Also DDC/DDK are now building themselves as part of their snapshot process (selfhost for training data), and that compile succeeds with no errors. Based on that I believe Analyzer/CFE are pretty Dart 2 clean at this point. (There might be some runtime cast failures on code paths that DDC/DDK does not exercise).

To get these command with --preview-dart-2 first the pub, dart2js, analyzer code needs to be made strong mode clean and these application snapshots need to be generated with that option turned on. I think we planned to do this in Q2.

I haven't figured out how to build the DDC snapshots yet with --preview-dart-2. Would appreciate help with that. I've been chatting a bit with @a-siva

Once https://github.com/dart-lang/pub/pull/1871 lands, we won't need dart2js or DDC to be Dart 2-compatible anymore. We do still use the analyzer, though—is that Dart 2-clean?

I'll work on checking that pub run code paths work in Dart 2 runtime.
I'll likely need some help getting it set up in the pub wrapping script in the SDK.

Is test Dart 2 clean? If so, we should be able to verify pub's cleanliness by running its tests.
If not, we can get part of the way by spawning the pub subprocesses the tests run with --preview-dart-2.

pub run seems to work when I fix the static errors.
https://github.com/dart-lang/pub/pull/1874
However it's drastically slower.
Set up a package which has a single dependency:
dependencies:
  pub:
    path: ../pub

Invoke pub with no arguments - just prints usage and exits.

pub run pub - 1.36 s
dart ../pub/bin/pub.dart - 0.87s
dart --preview-dart-2 ../pub/bin/pub.dart - 11.46s
dart --preview-dart-2 ../pub/bin/pub.dart run pub - 18.35s

It's possible that using a snapshot in Dart 2 semantics will be an improvement, and that this will carry over when using a hosted dependency rather than a path dependency...

I can't get timing data with snapshots - https://github.com/dart-lang/sdk/issues/32911

Here is the list of work that I think is blocking the end to end pub run --preview-dart-2 test scenario.

[ ] Sync pub into the SDK. Blocked on https://github.com/dart-lang/sdk/issues/32900
[ ] Set up the flags in the shell and bat script to get the --preview-dart-2 flag passed to the dart vm and not pub itself. (could skip this if we're ok with DART_VM_OPTIONS=--preview-dart-2 ux)
[ ] Ensure that package:test is Dart 2 runtime clean
[ ] Give pub an understanding of whether the run command is Dart 2 or not. Today it wants to run from snapshots when it can, and Dart1/Dart2 snapshots are incompatible (and today not possible for Dart2, blocked on #32911). The hacky thing to do is generate only 1 snapshot and run from source when using the "other" mode, the full solution would involve generating2 snapshots and picking the right one at runtime.

Given that the running from source is so slow with the CFE right now, how do we verify that a given chunk of code is in fact Dart 2-clean?

@a-siva - note Nate's Dart 1 vs Dart 2 perf comment above.

This continues to cause friction when we work on making packages dart2 compatible. @a-siva is someone looking into the perf issues?

We'll need to patch to make the SDK run in Dart 2 by default first.
On Fri, Apr 20, 2018 at 7:21 AM vsmenon<EMAIL_ADDRESS>wrote:

@nex3 https://github.com/nex3 - my understanding from @dgrove
https://github.com/dgrove is that you're the primary driver on this.
Are you able to make progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/32188#issuecomment-383111727,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACAsW2n94T7pB2TLPiPhO5vwLKETtWPMks5tqe7NgaJpZM4SHkwH
.

Is that blocking? My understanding from @natebosch 's comment is that we can run from source to make progress on the rest - we'll need the snapshot for reasonable perf though.

We should start testing by running from source with --preview-dart-2 and in parallel look at dart2 snapshot generation so that we can have a dart 2 sdk build.

@vsmenon According to https://github.com/dart-lang/sdk/issues/32188#issuecomment-382179856, it looks like pub itself is in an okay state here, modulo the snapshot problem pointed out https://github.com/dart-lang/sdk/issues/32188#issuecomment-382526880.
Is there a way to detect at runtime if the current VM is running in Dart 2 mode?

it looks like pub itself is in an okay state here

There is work we need to do in pub around invoking the right thing based on the Dart mode. If we're in Dart 2 we can't use the Dart 1 snapshot so we'll need to run from source (for now).

Is there a way to detect at runtime if the current VM is running in Dart 2 mode?

bool isDart2() => (<T>() => T)<String>() == String

I'll see if I can take a crack at the snapshot issue on Monday.

When I disable running from snapshots in Dart 2, it looks like Isolate.spawnUri() isn't working properly. I've filed https://github.com/dart-lang/sdk/issues/32950 to track the SDK issue.

#32950 has been fixed.

So trying to run pub directly fails – guessing it's because of the pub snapshot.
Tried this:
$ dart --preview-dart-2 ~/source/github/pub/bin/pub.dart run test
Unable to spawn isolate: file:///Users/kevmoo/.pub-cache/hosted/pub.dartlang.org/front_end-0.1.0-alpha.9/lib/src/fasta/source/outline_builder.dart:736:49: Error: A value of type 'dart.core::List<dynamic>' can't be assigned to a variable of type 'dart.core::Iterable<fasta.formal_parameter_builder::FormalParameterBuilder<fasta.type_builder::TypeBuilder>>'.
Try changing the type of the left hand side, or casting the right hand side to 'dart.core::Iterable<fasta.formal_parameter_builder::FormalParameterBuilder<fasta.type_builder::TypeBuilder>>'.
        formals.setRange(count, formals.length, last);
                                                ^

#33030 is closed. Is there any more work to do here, @natebosch ?

@dgrove – need to update pub_semver package – just told @natebosch

Darn. Still not fixed. I patched locally. Still getting
$ DART_VM_OPTIONS=--preview-dart-2 pub
Missing root library

@dgrove - there is much more to do.

[ ] At a minimum we need to have special handling for this flag in the shell script in the SDK.
[ ] To shave off ~10 seconds of the slowdown from fasta we need to generate a Dart 2 or kernel snapshot in the SDK.
[ ] Ideally we need to completely overhaul the way snapshots are generated in pub so that we can generate app-jit snapshots on first run rather than static snapshots on pub get. In today's implementation we fall back on running from source for every run which is slow.

Still not fixed. I patched locally. Still getting

As you discovered, the reason is that we only ship the snapshot with the SDK. It looks like my 2nd bullet point from the message above is a requirement, not a nice-to-have.

@a-siva you might need to sync past https://github.com/dart-lang/sdk/commit/4da831fd3ad72e4590cf007814d7b52c11d2a6d6

Pending CL - https://dart-review.googlesource.com/c/sdk/+/54743

@a-siva - @natebosch 's cl is breaking on the bots with:
C:/b/s/w/ir/cache/vpython/1bda7b/Scripts/python.exe ../../build/gn_run_binary.py compiled_action dart.exe --deterministic --packages=C:/b/s/w/ir/cache/builder/sdk/.packages --snapshot=gen/pub2.dart.snapshot --snapshot-depfile=C:/b/s/w/ir/cache/builder/sdk/out/ReleaseIA32/gen/pub2.dart.snapshot.d --preview-dart-2 --snapshot-kind=script C:/b/s/w/ir/cache/builder/sdk/third_party/pkg/pub/bin/pub.dart
Command failed: ./dart.exe --deterministic --packages=C:/b/s/w/ir/cache/builder/sdk/.packages --snapshot=gen/pub2.dart.snapshot --snapshot-depfile=C:/b/s/w/ir/cache/builder/sdk/out/ReleaseIA32/gen/pub2.dart.snapshot.d --preview-dart-2 --snapshot-kind=script C:/b/s/w/ir/cache/builder/sdk/third_party/pkg/pub/bin/pub.dart
output: c:/b/s/w/ir/cache/builder/sdk/.packages: Error: Unable to read '.packages' file:
  FileSystemException(uri=c:/b/s/w/ir/cache/builder/sdk/.packages; message=File c:/b/s/w/ir/cache/builder/sdk/.packages does not exist.).

I'm seeing similar with my DDC CL:
Command failed: ./dart --deterministic --packages=/Users/vsm/dart/sdk/.packages --snapshot=gen/dartdevc.dart.snapshot --snapshot-depfile=/Users/vsm/dart/sdk/xcodebuild/ReleaseX64/gen/dartdevc.dart.snapshot.d --preview-dart-2 --snapshot-kind=app-jit /Users/vsm/dart/sdk/pkg/dev_compiler/bin/dartdevc.dart --dart-sdk /Users/vsm/dart/sdk/sdk --dart-sdk-summary /Users/vsm/dart/sdk/xcodebuild/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /Users/vsm/dart/sdk/pkg/dev_compiler -o dartdevc.js /Users/vsm/dart/sdk/pkg/dev_compiler/bin/dartdevc.dart
output: Error while initializing Kernel isolate

Any ideas?

@vsmenon, are you running from xcodebuild/ReleaseX64/ directory?

Yes.

This CL https://dart-review.googlesource.com/c/sdk/+/55589 fixes the .package not found error but now an assertion in the optimizing compiler is being triggered which I am investigating.

Spoke to @JekCharlsonYu, this was moved to Stable by mistake.

https://dart-review.googlesource.com/c/sdk/+/55589 is being reviewed

Spoke too soon. Waiting on https://dart-review.googlesource.com/c/sdk/+/54743 from @natebosch

@a-siva I'm still unable to build locally.

Error while initializing Kernel isolate

Were you able to reproduce this error, did your CL fix it for you?

the build error I'm seeing now might be flaky - it was for @vsmenon as well. I was able to get a successful build locally.

Hopefully fixed in https://github.com/dart-lang/sdk/commit/2ff2af79213b150f57799a69d2b7757fa71daa42

We want https://github.com/dart-lang/pub/issues/1683 to fix the performance issues, but we're unblocked to run things (from source) now.


NIOTypedWebSocketServerUpgrader hangs when connection is opened and closed without sending any data

Expected behavior
Given a server setup with WebSocket upgrade using NIOTypedWebSocketServerUpgrader and allowRemoteHalfClosure set to true on the childChannel. If I open a connection to this server and close it immediately without sending any data the negotiation future returned from configureUpgradableHTTPServerPipeline on the server should throw a connection closed error.
Actual behavior
The negotiation future never completes leaving the child channel dangling.
If allowRemoteHalfClosure is set to false on the server then the negotiation future throws a inappropriateOperationForState error which isn't ideal but at least I can catch that the client has closed the connection
Steps to reproduce

Using the NIOWebSocketServer.swift example
Add .childChannelOption(ChannelOptions.allowRemoteHalfClosure, value: true) before bind call in run
Use a client to connect and close connection immediately

try await ClientBootstrap(group: MultiThreadedEventLoopGroup.singleton)
    .connect(host: "localhost", port: 8888)
    .flatMap { $0.close() }
    .get()

Note that negotiation future never completes

SwiftNIO version/commit hash
e5a216ba89deba84356bad9d4c2eab99071c745b
System & version information
swift-driver version: <IP_ADDRESS> Apple Swift version 5.10 (swiftlang-<IP_ADDRESS> clang-15<IP_ADDRESS>)
Target: x86_64-apple-macosx14.0

Yeah, this is a really good catch. We'd welcome a fix into the server upgrader here to make it aware of half closure.


Update AUR package

Feature/enhancement description:
It would be great to have the aur package updated with release as well http://aur.archlinux.org/packages/lorien-bin
You can use github actions to do that.

I don't maintain that AUR package. You have to ask the person who packaged it to update.


Conflicting channel services

Describe the bug
Same as https://github.com/knative/eventing/issues/5023
Need to fix in both KafkaChannel implementations we have.
Expected behavior
See https://github.com/knative/eventing/issues/5023
To Reproduce
See https://github.com/knative/eventing/issues/5023
Knative release version
0.21
Additional context
See https://github.com/knative/eventing/issues/5023

/remove-lifecycle stale


Thanos Receive container not starting w/ v0.32.x

Object Storage Provider:
minIO
What happened:
Changed version of container to v0.32.0 from v0.31.0 and the receive container no longer starts with following output
ts=2023-10-12T21:38:00.31138432Z caller=factory.go:53 level=info name=receive component=receive msg="loading bucket configuration"
ts=2023-10-12T21:38:00.311810444Z caller=receive.go:750 level=info name=receive component=receive msg="default tenant data dir already present, not attempting to migrate storage"
ts=2023-10-12T21:38:00.312061938Z caller=receive.go:277 level=debug name=receive component=receive msg="setting up TSDB"
ts=2023-10-12T21:38:00.312107869Z caller=receive.go:599 level=debug name=receive component=receive msg="removing storage lock files if any"
ts=2023-10-12T21:38:00.312209623Z caller=main.go:135 level=error name=receive err="open /data/remote-write-receive-data: permission denied\nremove storage lock files\nmain.startTSDBAndUpload\n\t/app/cmd/thanos/receive.go:601\nmain.runReceive\n\t/app/cmd/thanos/receive.go:279\nmain.registerReceive.func1\n\t/app/cmd/thanos/receive.go:99\nmain.main\n\t/app/cmd/thanos/main.go:133\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1598\npreparing receive command failed\nmain.main\n\t/app/cmd/thanos/main.go:135\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1598"

What you expected to happen:
receive container to start and receive incoming metrics
Anything else we need to know:
changing image version back to v0.31.0 allows container to start as expected.

It's not running as root anymore for security purposes so you'll have to make adjustments: https://github.com/thanos-io/thanos/pull/6107. I think this is unfixable from our end.

That change kick me too. Prometheus in docker run as nobody so...
Any way, isn't that user thanos an overhead? Simple user nobody is in every linux distro.
Use that and Dockerfile will be simpler.


404 on "edit this page" static site button

sets to reproduce :
go on this page
click on "edit this page" button
404 ! :)

There is a misconfiguration in the static generator path configuration :
Actual Button path :
https://github.com/mui-org/material-ui/blob/master/docs/src/docs/src/pages/components/cards/cards.md

Real path :
https://github.com/mui-org/material-ui/blob/master/docs/src/pages/components/cards/cards.md

Here is the double copy typo in the button path : docs/src/docs/src
Regards.

@florent-andre Thank you for the feedback. I should have been more careful in a recent refactorization. What do you think of this fix? Do you want to submit a pull request? :)
diff --git a/docs/src/modules/components/useMarkdownDocs.js b/docs/src/modules/components/useMarkdownDocs.js
index 87d8210d6..5f50b206f 100644
--- a/docs/src/modules/components/useMarkdownDocs.js
+++ b/docs/src/modules/components/useMarkdownDocs.js
@@ -143,7 +143,7 @@ ${headers.components
               key={content}
               demo={demos[name]}
               demoOptions={demoOptions}
-              githubLocation={`${SOURCE_CODE_ROOT_URL}/${name}`}
+              githubLocation={`${SOURCE_CODE_ROOT_URL}/docs/src/${name}`}
             />
           );
         }
diff --git a/docs/src/modules/constants.js b/docs/src/modules/constants.js
index 855110bb2..ff0526878 100644
--- a/docs/src/modules/constants.js
+++ b/docs/src/modules/constants.js
@@ -52,7 +52,7 @@ const LANGUAGES_LABEL = [
   },
 ];

-const SOURCE_CODE_ROOT_URL = 'https://github.com/mui-org/material-ui/blob/master/docs/src';
+const SOURCE_CODE_ROOT_URL = 'https://github.com/mui-org/material-ui/blob/master';

 module.exports = {
   CODE_VARIANTS,
diff --git a/docs/src/modules/utils/generateMarkdown.js b/docs/src/modules/utils/generateMarkdown.js
index 39d9306c5..538a25cff 100644
--- a/docs/src/modules/utils/generateMarkdown.js
+++ b/docs/src/modules/utils/generateMarkdown.js
@@ -3,9 +3,8 @@ import * as recast from 'recast';
 import { parse as docgenParse } from 'react-docgen';
 import { Router as Router2 } from 'next/router';
 import { pageToTitle } from './helpers';
-import { LANGUAGES_IN_PROGRESS } from 'docs/src/modules/constants';
+import { SOURCE_CODE_ROOT_URL, LANGUAGES_IN_PROGRESS } from 'docs/src/modules/constants';

-const SOURCE_CODE_ROOT_URL = 'https://github.com/mui-org/material-ui/blob/master';
 const PATH_REPLACE_REGEX = /\\/g;
 const PATH_SEPARATOR = '/';
 const DEMO_IGNORE = LANGUAGES_IN_PROGRESS.map(language => `-${language}.md`);

hi! I take this issue! thanks 👍🏻

I have done this PR to fix the issue!

I'm quite new to open source and contributing, I'm sorry if something is wrong. I will be better with time.. haha

Thank you too Olivier! @oliviertassinari

This should be re-opened as the static site was not deployed or the fix has missing part. There is still a 404 on the button.

@florent-andre An issue is closed when the associated PR is committed. That's just how GitHub works (but also how issues are handled in general).


feat: German texts/translations (#187)

This PR adds the German texts/translations for eslint.org
Couple of notes/remarks:

I could not come up with a good/propper word for reviewer, so I left it in English for now. Everything that came to my mind, e.g. Gutachter or Prüfer didn't feel right. Maybe someone else has a good idea?! If not, I think we can stick with reviewer for now.
I'm not very good at punctuation when it comes to commas, so there might be some errors there 🙈

Resolves #187

Thanks! Please be sure to sign the CLA (see comment). I’ll see if we can get a few more eyes on this.

Sorry I lost track of this! Building the site now.

No worries, glad I could be of help and cool to see it went live

@stefankolb We'd like to pay you for this contribution but I couldn't find an email address to contact you. Can you please shoot us an email at contact (at) eslint (dot) org? Thanks!

That's awesome, I just sent an email. Thank you!


Pane: 8 - Time Distribution always blank with default Squid access log format

There is an issue when parsing the decimal dates found within the standard Squid access logs.  It was working in GoAccess v1.2 but broke in v1.3 and is still broken in v1.4.
To reproduce this you can create a small log file called squid.log containing the following three lines:
1591919639.503    271 <IP_ADDRESS> TCP_TUNNEL/200 3951 CONNECT outlook.office.com:443 - HIER_DIRECT/<IP_ADDRESS> -
1591919643.857   3578 <IP_ADDRESS> TCP_TUNNEL/200 24531 CONNECT outlook.office.com:443 - HIER_DIRECT/<IP_ADDRESS> -
1591919648.857   4211 <IP_ADDRESS> TCP_TUNNEL/200 13631 CONNECT outlook.office.com:443 - HIER_DIRECT/<IP_ADDRESS> -
One this file has been created you can duplicate the issue using the following command (with either v1.3 or v1.4):
goaccess --no-global-config --log-format='%x.%^ %~ %L %h %^/%s %b %m %U %^' --date-format=%s --time-format=%s squid.log
You will find that the "Time Distribution" pane is empty, when it should have three hits.
I've not got time to look into the root cause, however I was able to workaround the issue for this use-case by adding “logitem->time=tkn;” just before the “break;” within the “case 'x':” clause located in the “parse_specifier” function found at line 1063 in the “src/parser.c” code.
Obviously this is a hack as the parser library was refactored during the v1.3 release, but it did resolve the issue I was seeing.  Somebody with more knowledge of the libraries might be able to come up with a more legitimate fix.
The bottom of the clause now reads as follows:
1060:    set_numeric_date (&logitem->numdate, logitem->date);
1061:    set_tm_dt_logitem (logitem, tm);
1062:    set_tm_tm_logitem (logitem, tm);
1063:    logitem->time=tkn;
1064:    break;

Thanks @Kevin-NZ for reporting this. It has been fixed upstream. Feel free to test it by building from development and let me know if that fixes the issue on your end. Otherwise it will be pushed out in the upcoming version.


Patch #227: Output path defaults to xlsxfile.xml

makes the output_path cli command optional
output path defaults to the xlsx file input directory but with a filename extension of '.xml'
i.e.

xls2xform /some/path/to/file.xlsx  # output-path is not defined

will write the file converted file to -> /some/path/to/file.xml
this fixes #227

Codecov Report

Merging #300 into master will increase coverage by 0.22%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
+ Coverage   81.19%   81.42%   +0.22%
==========================================
  Files          23       23
  Lines        3106     3111       +5
  Branches      729      730       +1
==========================================
+ Hits         2522     2533      +11
+ Misses        450      439      -11
- Partials      134      139       +5

Impacted Files
Coverage Δ

pyxform/xls2xform.py
72.3% <83.33%> (+28.97%)
:arrow_up:

pyxform/validators/util.py
75.53% <0%> (-2.13%)
:arrow_down:

pyxform/survey_element.py
87.43% <0%> (-1.01%)
:arrow_down:

pyxform/survey.py
89.09% <0%> (-0.52%)
:arrow_down:

pyxform/xform2json.py
79.78% <0%> (-0.43%)
:arrow_down:

pyxform/xls2json.py
76.35% <0%> (-0.34%)
:arrow_down:

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 1b10580...add2a2a. Read the comment docs.

issue resolved in pr #302 whose commit history is much cleaner


Adding Toggle Button

Is your feature request related to a problem? Please describe.
We need to toggle between two things.
Describe the solution you'd like
I have created a toggle button.
Describe alternatives you've considered
I used HTML,CSS.
Additional context
https://user-images.githubusercontent.com/98695632/182358189-81ad5336-812a-4f6b-8b9e-f3c7e88ec611.mp4

Assigning it to you🥳🥳


[FR] Derpibooru search support

Ripme version: 1.7.70
Java version:  java version "1.8.0_191" Java(TM) SE Runtime Environment (build 1.8.0_191-b12) Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)
Operating system:  Win10 Enterprise x64 1803 (OS Build 17134.345)

Exact URL you were trying to rip when the problem occurred: The resulting link, given in parts below, although with the default filters on the site only showing SFW, contains NSFW underage material.
Please include any additional information about how to reproduce the problem:
None.

Expected Behavior
Rip the resulting content of the search.
Actual Behavior
Can't rip this URL: No compatible ripper found

It shouldn't be much harder to support ripping the search feature.
For example, my favs given with the search sentence faved_by:rautamiekka, which is encoded into faved_by%3Arautamiekka and appended into https://derpibooru.org/search?q=.

Works in 1.7.84, at least.


Wrong commit hash icon on awesome-fontconfig mode

Describe Your Issue
I've got the symbol for female instead of the commit hash icon in left prompt elements

Have you tried to debug or fix it?
It's quite a normal installation with the font config file as follows:
  <alias>
    <family>monospace</family>
    <prefer>
            <!--<family>Pomodoro</family> -->
            <!-- <family>FontAwesome</family> -->
            <!-- <family>Octicons</family> -->
            <!-- <family>Icomoon</family> -->
             <family>PowerlineSymbols</family>
    </prefer>
  </alias>c

</fontconfig>

and the config override i got on .zshrc are as follow
source ~/.fonts/*.sh
# Path to your oh-my-zsh installation.
  export ZSH="/home/nakama/.oh-my-zsh"

# Set name of the theme to load --- if set to "random", it will
# load a random theme each time oh-my-zsh is loaded, in which case,
# to know which specific one was loaded, run: echo $RANDOM_THEME
# See https://github.com/robbyrussell/oh-my-zsh/wiki/Themes
#ZSH_THEME="robbyrussell"
POWERLEVEL9K_MODE='awesome-fontconfig'
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(dir rbenv vcs)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(status root_indicator background_jobs time docker_machine)

POWERLEVEL9K_SHORTEN_DIR_LENGTH=2
POWERLEVEL9K_SHOW_CHANGESET=true
ZSH_THEME="powerlevel9k/powerlevel9k"

Environment Information
This information will help us understand your configuration.

What version of ZSH are you using? You can use zsh --version to see this. : zsh 5.4.2
Do you use a ZSH framework (e.g., Oh-My-ZSH, Antigen)? : Oh-My-ZSH
How did you install P9k (cloning the repo, by tarball, a package from your OS, etc.,)? : cloning into oh-my-zsh theme folder
What version of P9k are you using? current master branch (as of now, [this commit])(https://github.com/bhilburn/powerlevel9k/commit/22a761208d24027a26daa947a7b4034365fefc16)
Which terminal emulator do you use? : tilda? does it break it?

Issues with Fonts & Icons
You may delete this section if your issue is not font / icon related.

Which font do you use?  : awesome-terminal fonts
Which font configuration mode are you using? You can check this with (echo $POWERLEVEL9K_MODE). : awesome-fontconfig
Please share the contents of $P9k/debug/font-issues.zsh. : can't found this file
If this is an icon problem, does the output of $ get_icon_names look correct? nope, it shows the "female icon"

What do you get when you do echo ${CODEPOINT_OF_OCTICONS_BOOKMARK}?

In awesome-fontconfig we make use of ${CODEPOINT_OF_xxx} which is the codepoints specified in source ~/.fonts/*.sh. If a font is not available on the system for that codepoint, or if another font takes president and has a different glyph for that number, it will display. Thus, you would have to ensure the order of president has glyph fonts before powerline fonts. Alternatively, you could use a nerd font, which already has all of the other fonts built into it.

Hi, thanks for answering
the echo comand that you gives me return nothing, it's just a newline
there might be something wrong with my setup, though i manage to replicate it on another machine
I'll check the .sh file that you mentioned
thanks

Yeah, given up on awesome-terminals and used the nerdfont instead
now it works everywhere
closing this issue now


Avoid reporting exit codes when user does not care to see them

The lumina-open program will usually launch an application and then stick around and catch error codes, displaying an error message whenever the target program returns a non-zero exit value. Some programs misbehave and always return non-zero, causing an error to be reported whenever it closes.
In order to avoid having extra lumina-open processes running and error reports from these
misbehaving applications, I have added an option to turn off process monitoring. With this patch, lumina-open will check for the existence of a file. If the ~/.config/lumina-desktop/nowatch file exists, then lumina-open will launch the given process and immediately exit. If the nowatch file
is not present, lumina-open follows its default behaviour.
This basically gives the user (even a non-admin who cannot change .desktop files) the ability to shut off lumina-open's error reporting when working with programs that givee bad return values.

I noticed that your pull request only works if there is no alternate location for the users config files (XDG_CONFIG_HOME is set to default value of "~/.config"). I just re-implemented[1] this pull request a moment ago to fix this and add in that functionality for you.
[1] https://github.com/trueos/lumina/commit/f2f3c6847da220af56f436c5f3c3d02b39477aa3

Looks good. Thank you.


[docs] Demo SearchIcons crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/components/material-icons/?query=leave&theme=Two+tone
ExitToApp icon crashes
demo SearchIcons crashes

Your Environment

Tech
Version

MUI
v5.2.5

netlify deploy
https://61c098e520eae9000736c0df--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0

Probably due to browser issues.. Reloaded and it worked


libuuid.so.1: cannot open shared object file: No such file or directory

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
There is docs for this: https://nixpacks.com/docs/providers/node
This was solved after adding
"optionalDependencies": {
    "node-canvas": "*"
  }

Problem
This should not be the case, it should be looking at canvas because the name of the package is canvas and not node-canvas
https://www.npmjs.com/package/canvas
To reproduce
No response
Expected behavior
No response
Environment
Any

See this: https://discord.com/channels/713503345364697088/1135937739217449072/1136038578468683818

Thanks for the bug report. Do you have an example repo where this bug is happening? Or could you provide the package.json file.

@coffee-cup I think this is happening because canvas does not exist in the root package json. I have a mono repo and I have canvas inside the server directory.
Maybe you should scan all package.json in the project folder and include libuuid if canvas exist in any of those
{
  "name": "server",
  "version": "0.0.1",
  "description": "",
  "author": "",
  "private": true,
  "license": "UNLICENSED",
  "scripts": {
    "prebuild": "rimraf dist",
    "build:prod": "yarn prisma generate && yarn prisma migrate deploy && yarn build",
    "build": "nest build",
    "format": "prettier --write \"src/**/*.ts\" \"test/**/*.ts\"",
    "start": "nest start",
    "dev": "nest start --watch --preserveWatchOutput",
    "start:debug": "nest start --debug --watch",
    "start:prod": "node dist/main",
    "lint": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix",
    "test": "jest",
    "test:watch": "jest --watch",
    "test:cov": "jest --coverage",
    "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
    "test:e2e": "jest --config ./test/jest-e2e.json",
    "migrate:dev": "yarn prisma migrate dev"
  },
  "dependencies": {
    "@nestjs/axios": "^3.0.0",
    "@nestjs/common": "^9.3.9",
    "@nestjs/config": "^2.3.1",
    "@nestjs/core": "^9.3.9",
    "@nestjs/event-emitter": "^1.4.1",
    "@nestjs/platform-express": "^9.3.9",
    "@nestjs/serve-static": "^3.0.1",
    "@nestjs/swagger": "^7.0.3",
    "@prisma/client": "^4.11.0",
    "@shopify/shopify-api": "^6.2.0",
    "aws-sdk": "^2.1239.0",
    "canvas": "^2.11.2",
    "class-transformer": "^0.5.1",
    "class-validator": "^0.14.0",
    "cookie-parser": "^1.4.6",
    "dayjs": "^1.10.6",
    "jsbarcode": "^3.11.5",
    "liquidjs": "^10.8.3",
    "nodemailer": "^6.9.3",
    "prisma": "^4.11.0",
    "qrcode": "^1.5.3",
    "reflect-metadata": "^0.1.13",
    "rimraf": "^4.4.0",
    "rxjs": "^7.8.0",
    "shopify-api-node": "^3.12.5"
  },
  "devDependencies": {
    "@nestjs/cli": "^9.2.0",
    "@nestjs/devtools-integration": "^0.1.3",
    "@nestjs/schematics": "^9.0.4",
    "@nestjs/testing": "^9.3.9",
    "@types/cookie-parser": "^1.4.3",
    "@types/express": "^4.17.17",
    "@types/jest": "29.4.0",
    "@types/mime-types": "^2.1.1",
    "@types/multer": "^1.4.7",
    "@types/node": "^18.15.0",
    "@types/supertest": "^2.0.12",
    "@types/uuid": "^8.3.4",
    "@typescript-eslint/eslint-plugin": "^5.54.1",
    "@typescript-eslint/parser": "^5.54.1",
    "eslint": "^8.35.0",
    "eslint-config-prettier": "^8.7.0",
    "eslint-plugin-prettier": "^4.2.1",
    "jest": "29.5.0",
    "prettier": "^2.8.4",
    "source-map-support": "^0.5.21",
    "supertest": "^6.3.3",
    "ts-jest": "29.0.5",
    "ts-loader": "^9.4.2",
    "ts-node": "^10.9.1",
    "tsconfig-paths": "4.1.2",
    "typescript": "^4.9.5"
  },
  "jest": {
    "moduleFileExtensions": [
      "js",
      "json",
      "ts"
    ],
    "rootDir": "src",
    "testRegex": ".*\\.spec\\.ts$",
    "transform": {
      "^.+\\.(t|j)s$": "ts-jest"
    },
    "collectCoverageFrom": [
      "**/*.(t|j)s"
    ],
    "coverageDirectory": "../coverage",
    "testEnvironment": "node"
  }
}


change updateMTOShipment description to match implementation

Draft PR
Description
This ticket fixes a bug on the description of the updateMTOShipment endpoint, that mistakenly tells the user that an agent can not be added but can instead be updated. The opposite behavior is true. An agent can be added using the updateMTOShipment endpoint, but an agent can not be added using this endpoint. Instead, the user should be pointed to the updateMTOAgent endpoint to update any of the subfields of an existing agent.
Reviewer Notes
Please make sure all the validations for this fix are covered.
Setup
Add any steps or code to run in this section to help others prepare to run your code:
echo "Code goes here"

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 9ab5992ba1b634d65fb86b974b1c22d63569cfc4