Replace ID flags with arguments [CLI-11]

Description
This PR replaces ID flags with positional arguments whenever a single ID is required.

Also:

Cleaned up the help text in actions
Added apps show

Testing

[ ] This change adds test coverage for new/changed/fixed functionality

Checklist

[ ] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
[X] All active GitHub checks for tests, formatting, and security are passing
[ ] The correct base branch is being used, if not master

can we allow using audience apart from allowing id?
eg
auth0 apis scopes list http://test-api-6

Done: https://github.com/auth0/auth0-cli/pull/183


Data normalization

Hi,
I saw that data is normalized for the val split, but not for the train split:
https://github.com/ziplab/LITv2/blob/b0a35dec6d7d1244401bf428cccaa82df9ccd813/classification/data/build.py#L134
Shouldn't normalization be applied to both val and train splits?
Thank you for your help again.

That's a good question. In general, we have different aims for training and testing.

During training, we aim to train a model to learn from the input data and learn the underlying patterns and representations within the data distribution. If we normalize the train set, we are artificially altering the data distribution and may introduce biases into the model's training process. This is unwanted.
During testing, we aim to evaluate the models'performance. It is important to normalize the validation set to ensure consistency and fair evaluation.

The following discussions may help,

https://stackoverflow.com/questions/58151507/why-pytorch-officially-use-mean-0-485-0-456-0-406-and-std-0-229-0-224-0-2
https://discuss.pytorch.org/t/discussion-why-normalise-according-to-imagenet-mean-and-std-dev-for-transfer-learning/115670

Thank you!


Footer GitHub accelerator image pixelates on larger screens

Description
A higher quality image can be added and a bg-contain classname for smaller devices?

Screenshots
No response
Additional information
No response

Hi, I would like to work on this issue. Thank You.

Hi, I would like to work on this issue. @SaraJaoude Can you please Assign me this task. Thank You.

@dhawansolanki I will assign this to you. In future please avoid tagging a specific maintainer to request assignment - all maintainers receive notifications and can assign issues.  This makes the project run more smoothly and reduces unnecessary notifications.

No activity for 2 weeks, unblocking by unassigning, so others can move it forwards

Hi, I'd love to contribute in resolving this issue. Can anyone please assign it to me.

Hi, I'd love to contribute in resolving this issue. Can anyone please assign it to me.

@heyrishabh13 I will assign this to you

@heyrishabh13 how are you getting along with raising a Pull Request for this Issue?

@heyrishabh13 thank you for submitting a Pull Request for this Issue.  As mentioned by @kumarsonsoff3 in the comments for #7537 please remove any updates to your profile (and raise a new Pull Request for these).

i can work on this

i can work on this

@plbstl I have assigned this to you

As the rebranding has been started this will be fixed as part of that process so I will close this Issue for now.


Cluster Op Infra

Prototype something to launch ops on multiple devices. This is only needed for slow dispatch, which it looks like we may not try to support in the short term.

no longer relevant


Rename RawQuery.tables and Changes.tables to affectedTables

Closes #141
@nikitin-da please review

Sorry, forgot to change README


2.7.6 latest plugin version next , previous button in datepicker not working

When press previous or next button it shows error

Thank you so much for your report.
I fixed this and released 2.7.7.

thanks for quick reply.
On Wed, Apr 25, 2018 at 5:02 PM, Misha Tarnorutsky<EMAIL_ADDRESS>
wrote:

Thank you so much for your report.
I fixed this and released 2.7.7.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/misha130/ion-datepicker/issues/73#issuecomment-384255179,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZathi1SF1eJb_w9A2Rt_Oedu-zHctodks5tsF7RgaJpZM4TiyM3
.


[fix][sec]Upgrade jackson to 2.17.2

Motivation

jackson-1.14.2 has the vulnerability.
https://security.snyk.io/vuln/SNYK-JAVA-COMFASTERXMLJACKSONCORE-7569538
@JsonIgnore in latest version of jackson works well.
https://github.com/apache/pulsar/pull/20182
https://github.com/FasterXML/jackson-databind/commit/4537f09d3419440554c7cf6f2d8130205fc6be6e

I run the following code:

JsonIgnore test code
class Foo {
    private int a = 10;

    @JsonIgnore
    private transient int b = 11;

    public int getA() {
        return a;
    }

    public void setA(int a) {
        this.a = a;
    }

    public int getB() {
        return b;
    }

    public void setB(int b) {
        this.b = b;
    }

    public static void main(String args[]) throws JsonProcessingException {
        ObjectMapper ob = new ObjectMapper();
        System.out.println(ob.writeValueAsString(new Foo()));
    }
}

result:
# expected
 {"a":10}

# 2.14.2
{"a":10}

# 2.15.2
{"a":10,"b":11}

# 2.17.2
{"a":10}

Modifications

Upgrade jackson to 2.17.2

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.59%. Comparing base (bbc6224) to head (e60bff4).
Report is 521 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23174      +/-   ##
============================================
+ Coverage     73.57%   74.59%   +1.02%
- Complexity    32624    34172    +1548
============================================
  Files          1877     1921      +44
  Lines        139502   144480    +4978
  Branches      15299    15809     +510
============================================
+ Hits         102638   107775    +5137
+ Misses        28908    28460     -448
- Partials       7956     8245     +289

Flag
Coverage Δ

inttests
27.99% <100.00%> (+3.40%)
:arrow_up:

systests
24.72% <100.00%> (+0.40%)
:arrow_up:

unittests
73.94% <100.00%> (+1.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

...ava/org/apache/pulsar/common/util/FieldParser.java
87.40% <100.00%> (ø)

... and 503 files with indirect coverage changes


please re-open closed issue, more info has been provided (late sorry)

please re-open this issue
https://github.com/laravel/framework/issues/24005
sorry that i took a few days to respond to the request for info

Please don't open another issue, just comment on the closed one.


adding additional logging for autocomplete feature

Context
Summary:  The PR adds an additional field inlineCompletionItemContext only for dotCom users and public github Repos in the private metadata field, in the cody.completion/suggested event. In addition, we add a check recordsPrivateMetadataTranscript: isDotComUser ? 1: 0 here in the metadata of telemetry-v2 to send the events in private gcs buckets. The struct of inlineCompletionItemContext is:
"inlineCompletionItemContext": {
    "gitUrl": string,
    "commit": string,
    "prefix": string,
    "suffix": string,
    "triggerLine": number,
    "triggerCharacter": number,
    "context": []
}

In the current telemetry, we only log the insertText field which is the completion. Addition of the additional context items such as prefix, suffix and contextSnippets retrieved by various retrievers would help us a in a number of offline tasks & analysis such as offline repro of completions & comparison across several LLMs, offline-online correlation, context & LLM evaluation on real user data, real-world dataset which can be leveraged for training, fine-tuning, evaluation etc.
Each string field in the structure is truncated to 4000 characters (~1000 tokens), and the maximum number of context items are 15. On an average case we get ~2-3 context items, and this should contribute to ~16kb of logging data.
We only log this data for dotCom users and public github Repos and in the cody.completion/suggested events and not for the other events, so the payload size of other events will not affected & we avoid duplication

Test plan

Unit Tests updated as per new logs to check we are only logging for DotComUsers.
Manual testing: Start the debugger from this branch. For any new autocomplete request, the telemetry response for the "cody.completion/suggested" should have an additional field inlineCompletionItemContext for only public repos:

Example log (Note: Since the example log can be big, have replaced actual field value with  at some places ):
█ telemetry-v2: recordEvent: cody.completion/suggested  {
  "parameters": {
    "version": 0,
    "interactionID": "65350db8-2b23-4827-be18-41713d0d2eeb",
    "metadata": [
      {
        "key": "multiline",
        "value": 0
      },
      {
        "key": "testFile",
        "value": 0
      },
      { ...... <other_fields> .........}
    ],
    "privateMetadata": {
      "triggerKind": "Automatic",
      "providerIdentifier": "fireworks",
      "providerModel": "starcoder-hybrid",
      "languageId": "typescriptreact",
      "traceId": "554a89e584cce3562bf3aff42f481500",
      "multilineMode": null,
      "id": "65350db8-2b23-4827-be18-41713d0d2eeb",
      "contextSummary": {
        "strategy": "jaccard-similarity",
        "duration": 2.1568756103515625,
        "totalChars": 6070,
        "retrieverStats": {
          "jaccard-similarity": {
            "suggestedItems": 1,
            "positionBitmap": 1,
            "retrievedItems": 2,
            "duration": 1.9716243743896484
          }
        }
      },
      "source": "Network",
      "inlineCompletionItemContext": {
        "gitUrl": "github.com/sourcegraph/cody",
        "commit": "1b4b0f8fb16e9081383d59f82d9d6c662643086c",
        "prefix": " <string> ",
        "suffix": " <string>",
        "triggerLine": 175,
        "triggerCharacter": 22,
        "context": [
          {
            "content": " <string>",
            "startLine": 0,
            "endLine": 49,
            "filePath": "/Users/hiteshsagtani/dev/bulletproof-react/src/components/layouts/dashboard-layout.tsx"
          }
        ]
      },
      "items": [
        {
          "lineCount": 1,
          "charCount": 12,
          "stopReason": "streaming-truncation",
          "parseErrorCount": 0,
          "nodeTypes": {
            "atCursor": "jsx_element",
            "parent": "jsx_element",
            "grandparent": "jsx_element",
            "greatGrandparent": "jsx_element",
            "lastAncestorOnTheSameLine": "jsx_element"
          },
          "nodeTypesWithCompletion": {
            "atCursor": "jsx_element",
            "parent": "jsx_element",
            "grandparent": "jsx_element",
            "greatGrandparent": "jsx_element",
            "lastAncestorOnTheSameLine": "jsx_expression"
          },
          "insertText": "{user.email}"
        }
      ],
      "otherCompletionProviders": []
    }
  },
  "timestamp": "2024-06-08T13:06:00.618Z"
}

@akalia25 Is the PR looking for the logging perspective and is it okay to merge this ?

When this sensitive data is being collected, we'll also need to update the metadata on the event cody.completion/suggested to include recordsPrivateMetadataTranscript:1. This ensures that the event is being sent to our ML pipeline. More info here :)

Thanks for pointing this out Aditya. Added this change along with logging only for v2 telemetry as discussed in the offline sync.


[DEV] Add progress logger for grid execution

Let's use something like tqdm instead and customize a style
Let's make progress bar an option but not a default feature

and 2. are done in the latest commit.


Add OTel Operator auto-instrumentation troubleshooting page

Add troubleshooting tips for OTel Operator's auto-instrumentation capability.
Ref issue: #4723
Related PR: #4708

@jaronoff97 thanks for taking a look, is this good to go now or any other requests for change?

Another common issue that we have seen is that the user hardcoded JAVA_TOOLS_OPTIONS (or equivalent for python and node) in docker image or it consumes it from a config map. In this case the operator does not see it and incorrectly overrides it

Added new section to address this.

@avillela @pavolloffay is this good to go?

@avillela @pavolloffay is this good to go?

I've made the requested changes. Just waiting on feedback from @pavolloffay on the changes. 😁

I am merging this now, if there are any open concerns, please follow up in a new issue


[ZEPPELIN-3320] upgrade the flink interpreter from 1.1.3 to 1.4.0

What is this PR for?
upgrade the flink interpreter from 1.1.3 to 1.4.0，the LocalFlinkMiniCluster default haService is EmbeddedHaServices in 1.4.0, so use HighAvailabilityServicesUtils.createHighAvailabilityServices to create StandaloneHaServices
What type of PR is it?
Improvement
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3320
How should this be tested?

no tested

Screenshots (if appropriate)
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

could you close / open this PR to run tests - please check if there is any error. thanks


Multiple DropDown usage in One ViewController

I am having a weird issue,
here is what i am doing
 self.maxPricePicker = [[DownPicker alloc] initWithTextField:self.maxPrice withData:self.mainPricesArray];
    [self.maxPricePicker setPlaceholder:@"Max. Price"];

    self.minPricePicker = [[DownPicker alloc] initWithTextField:self.minPrice withData:self.mainPricesArray];
    [self.minPricePicker setPlaceholder:@"Min. Price"];

    self.sizePicker = [[DownPicker alloc] initWithTextField:self.sizeUnit withData:@[@"Sq. Feet", @"Marla", @"Kanal"]];
    [self.sizePicker setPlaceholder:@"Select Size Unit"];

However it seems to me that only MinPrice picker works. When i click on it, suddenly all the picker values change to initial value? Also when i try to pick through other pickers i get a crash at
- (void)pickerView:(UIPickerView *)pickerView didSelectRow:(NSInteger)row inComponent:(NSInteger)component
{
    self->textField.text = [dataArray objectAtIndex:row];

    if([self.delegate respondsToSelector:@selector(pickerView:didSelectIndex:)])
        [self.delegate pickerView:pickerView didSelectIndex:row];
}

at  self->textField.text = [dataArray objectAtIndex:row];

textField and pickerView are nil. Not sure why the MinPrice picker is working and others not? and why their value change first time when i click on minPrice

The problem in my case was IQKeyboardManager library. I removed this folder from my project and DownPicker works fine.

I solved it like below.
in .h file
@Property (nonatomic) DownPicker *picker1;
@Property (nonatomic) DownPicker *picker2;
in .m file
NSArray *array1 = @[@"data 1", @"data 2", @"data 3"];
self.picker1 = [[DownPicker alloc] initWithTextField:self.priceTierTF withData:array1];
NSArray *array2 = @[@"data 1", @"data 2", @"data 3"];
self.picker2 = [[DownPicker alloc] initWithTextField:self.priceTierTF withData:array2];


feat(changelog): breaking changes are never skipped

Hi there! Trying to contribute to this project first-time.
Description
Trying to resolve #106 by option to protect breaking changes, even when matched by a skipping commit parser. The option protect_breaking_commits is introduced in the config.
Note that during testing, one caveat was discovered: If a commit is matched by one parser, it can't be matched by any other. That means if a breaking commit was prevented from being skipped, it won't become part of any other group. Maybe it is worth to think about allowing multiple parsers for commits in an hierarchical order?
Motivation and Context

Breaking changes are highly relevant for stakeholders, therefore should not be skipped in a changelog, even when matched by commit parsers.

Fixes #106
How Has This Been Tested?

Extended the test cases by breaking change commits. Added a usual breaking change commit and one that matches a skipped commit parser regex. The latter is not skipped even it matches a commit parser that has skip set to true, if protect_breaking_commits is true.

nightly-aarch64-apple-darwin / rustc 1.66.0-nightly (8ce3204af 2022-09-30)

Backward compatibility is provided.
Screenshots / Output (if appropriate):
See tests in src/changelog.rs
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report
Base: 64.06% // Head: 63.82% // Decreases project coverage by -0.24% :warning:

Coverage data is based on head (ba5f844) compared to base (f9d4b88).
Patch coverage: 87.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
- Coverage   64.06%   63.82%   -0.23%
==========================================
  Files          14       14
  Lines         790      796       +6
==========================================
+ Hits          506      508       +2
- Misses        284      288       +4

Flag
Coverage Δ

unit-tests
63.82% <87.50%> (-0.23%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

git-cliff-core/src/commit.rs
94.48% <80.00%> (-0.38%)
:arrow_down:

git-cliff/src/changelog.rs
69.88% <100.00%> (-1.28%)
:arrow_down:

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

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

I added test fixtures for this PR in 4e61f30

Thanks @orhun for the review! I think I addressed everything now. Feel free to add more comments if unsure about something.


Fix spelling mistakes

Thanks!

:tada: This PR is included in version 6.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dataset Viewer issue for rajistics/indian_food_images

Link
https://huggingface.co/datasets/rajistics/indian_food_images/viewer/rajistics--indian_food_images/train
Description
I have a train/test split in my dataset

t
The dataset viewer works for the test split (images of indian food), but does not show my train split.  My guess is maybe there is some corrupt image file that is guessing this.  But I have no idea.
The original dataset was pulled from here: https://www.kaggle.com/datasets/l33tc0d3r/indian-food-classification?resource=download-directory
Owner
Yes

Hi, thanks for reporting. I triggered a refresh of the preview for this dataset, and it works now. I'm not sure what occurred.


✨ Add the ability to forward events from web views to mobile sessions

What and why?
Customers want the ability to track the full journey of a use, which may include interacting with WebViews which are already tracked by Datadog. This allows them to forward events from their WebViews to Mobile sessions.
This requires the use of the webview_flutter package, which is the main recommended package for Flutter.
Right now, this is embedded into the main datadog_flutter_plugin package, but I'm thinking about moving it to its own package prior to releasing it (or even before merging the PR).
How?
We're utilizing the bridging interfaces Datadog has already written for WKWebView and Android's WebView. By utilizing an interface provided by webview_flutter, we're able to get the underlying platform controls and attach Datadog's observers.
Review checklist

[ ] This pull request has appropriate unit and / or integration tests
[ ] This pull request references a Github or JIRA issue

CI Configuration (optional)

[ ] Run unit tests
[ ] Run integration tests

After review, we're going to need to pull this into its own package to maintain Flutter 2.8 support in the main package. Closing this and will reopen with the new package.


Vernemq can't start

Hello,
I'm trying Vernemq cluster, testing environment is Ubuntu14.04 in vmware.
I have 2 questions about this.

I edit /etc/vernemq/vernemq.conf and only change nodename (ex<EMAIL_ADDRESS>then restart vernemq but failed. I use sudo vernemq console and find the error:
{error_logger,{{2016,7,22},{18,21,4}},"Protocol: ~tp: the name<EMAIL_ADDRESS>seems to be in use by another Erlang node",["inet_tcp"]}

I change nodename again (ex. VerneMQ1@<IP_ADDRESS>), run "sudo vernemq console" and get the following error:

config is OK
-config /var/lib/vernemq/generated.configs/app.2<IP_ADDRESS>.31.54.config -args_file /var/lib/vernemq/generated.configs/vm.2<IP_ADDRESS>.31.54.args -vm_args /var/lib/vernemq/generated.configs/vm.2<IP_ADDRESS>.31.54.args
!!!!
!!!! WARNING: ulimit -n is 1024; 65536 is the recommended minimum.
!!!!
Exec:  /usr/lib/vernemq/erts-7.3/bin/erlexec -boot /usr/lib/vernemq/releases/0.13.1/vernemq               -config /var/lib/vernemq/generated.configs/app.2<IP_ADDRESS>.31.54.config -args_file /var/lib/vernemq/generated.configs/vm.2<IP_ADDRESS>.31.54.args -vm_args /var/lib/vernemq/generated.configs/vm.2<IP_ADDRESS>.31.54.args              -pa /usr/lib/vernemq/lib/erlio-patches -- console
Root: /usr/lib/vernemq
{error_logger,{{2016,7,22},{18,31,55}},"Protocol: ~tp: the name<EMAIL_ADDRESS>seems to be in use by another Erlang node",["inet_tcp"]}
{error_logger,{{2016,7,22},{18,31,55}},crash_report,[[{initial_call,{net_kernel,init,['Argument__1']}},{pid,<0.23.0>},{registered_name,[]},{error_info,{exit,{error,badarg},[{gen_server,init_it,6,[{file,"gen_server.erl"},{line,344}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,240}]}]}},{ancestors,[net_sup,kernel_sup,<0.10.0>]},{messages,[]},{links,[#Port<0.502>,<0.20.0>]},{dictionary,[{longnames,true}]},{trap_exit,true},{status,running},{heap_size,610},{stack_size,27},{reductions,837}],[]]}
{error_logger,{{2016,7,22},{18,31,55}},supervisor_report,[{supervisor,{local,net_sup}},{errorContext,start_error},{reason,{'EXIT',nodistribution}},{offender,[{pid,undefined},{id,net_kernel},{mfargs,{net_kernel,start_link,[['VerneMQ0@<IP_ADDRESS>',longnames]]}},{restart_type,permanent},{shutdown,2000},{child_type,worker}]}]}
{error_logger,{{2016,7,22},{18,31,55}},supervisor_report,[{supervisor,{local,kernel_sup}},{errorContext,start_error},{reason,{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}}},{offender,[{pid,undefined},{id,net_sup},{mfargs,{erl_distribution,start_link,[]}},{restart_type,permanent},{shutdown,infinity},{child_type,supervisor}]}]}
{error_logger,{{2016,7,22},{18,31,55}},crash_report,[[{initial_call,{application_master,init,['Argument__1','Argument__2','Argument__3','Argument__4']}},{pid,<0.9.0>},{registered_name,[]},{error_info,{exit,{{shutdown,{failed_to_start_child,net_sup,{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}}}},{kernel,start,[normal,[]]}},[{application_master,init,4,[{file,"application_master.erl"},{line,134}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,240}]}]}},{ancestors,[<0.8.0>]},{messages,[{'EXIT',<0.10.0>,normal}]},{links,[<0.8.0>,<0.7.0>]},{dictionary,[]},{trap_exit,true},{status,running},{heap_size,376},{stack_size,27},{reductions,164}],[]]}
{error_logger,{{2016,7,22},{18,31,55}},std_info,[{application,kernel},{exited,{{shutdown,{failed_to_start_child,net_sup,{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}}}},{kernel,start,[normal,[]]}}},{type,permanent}]}
{"Kernel pid terminated",application_controller,"{application_start_failure,kernel,{{shutdown,{failed_to_start_child,net_sup,{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}}}},{kernel,start,[normal,[]]}}}"}

Crash dump is being written to: /erl_crash.dump...Kernel pid terminated (application_controller) ({application_start_failure,kernel,{{shutdown,{failed_to_start_child,net_sup,{shutdown,{failed_to_start_child,net_kernel,{'EXIT',nodistribution}}}}},{k

If I change nodename to default<EMAIL_ADDRESS>vernemq will start ok. Did I miss some configuration for cluster setting?

Hi @landy5661
Thanks for your problem description!
The most important thing here, is to define what you actually are trying to do and what your expectations are.
How did you set up the cluster? if you have a cluster, renaming and rebooting single nodes in it won't just work like that.
It looks like you are editing the same config file in /etc/vernemq/vernemq.conf and try to start different nodes from that? (this won't work)

Hi ioolkos,
Thanks for your reply, I try to describ what I want to do.
Im looking for a MQTT broker which support cluster. I do some search google and find this url: https://github.com/mqtt/mqtt.github.io/wiki/server-support, then I select Vernemq and try it.
I want to create 2 VM with Ubuntu 14.04, install VerneMQ on both, and then set up a cluster.
I edit vernemq.conf and change the nodename > vernemq restart > make the new nodename work > but failed. I want to make each nodename is unique, so I change the nodename in vernemq.conf.
After some research I found this issue: https://github.com/erlio/vernemq/issues/134.
I delete the folder "/var/lib/vernemq/" and restart vernemq > still failed.
Then I reinstall vernemq and use the same config file > restart vernemq > it works.
Did I do something wrong at first step?
https://vernemq.com/docs/clustering/

You most probably created a cluster, and then changed nodenames. Do it the other way around ;)
(VerneMQ registers the cluster nodenames in its metadata, and then complains, if you want to boot a cluster node with a new name)
Hope this helps!

Hi ioolkos,
Thanks for your explanation. I change my steps and the 2 clusters are all ok.
in single node
old:  node is running -> change nodename -> vernemq restart -> fail.
new: vernemq stop -> change nodename -> vernemq start -> success.


Deploy to pypi on merge into master

This PR

adds functionality to deploy to pypi when the master branch is built.
bumps the minor version number.
fixes some links in the README.

@SergeyPirogov, as I'm not a maintainer on pypi, I can't add my own encrypted password to deploy. Two options: (a) you can add me as a maintainer and I can deploy using my account, or (b) you can follow the instructions here to encrypt your password and add the password in the .travis.yml file.
This should also resolve #27.

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   76.07%   76.07%
=======================================
  Files          17       17
  Lines         326      326
  Branches       21       21
=======================================
  Hits          248      248
  Misses         64       64
  Partials       14       14

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 6f06fbe...da3dff0. Read the comment docs.

@tillahoffmann, please give me your pypi id, so I will be able to add you as a maintainer

My pypi id is tillahoffmann (profile here).

Hi @SergeyPirogov, just wanted to follow up on being added as a maintainer.

@tillahoffmann added you as collaborator

Great, thanks for setting that up. I think this is ready to be merged.


[BUG]: (Not really a bug) I'm not sure where i'm supossed to put my token.

Program version
5.0.0
Operating System
Windows
Type
GUI
Service
AnimeOnegai
Command used
Login
Show ID
1234
Episode
1
Console Output
[WARN] Private key missing

=== Multi Downloader NX GUI 5.0.0 ===

GUI server started on port 3000
[WS] Connection to public ws from '::1'
[WS] Connection from '::1'
[ERROR] Authentication not possible, manual authentication required due to recaptcha. In order to login use the --token flag. You can get the token by logging into the website, and opening the dev console and running the command "localStorage.ott_token"
[ERROR] Authentication not possible, manual authentication required due to recaptcha. In order to login use the --token flag. You can get the token by logging into the website, and opening the dev console and running the command "localStorage.ott_token"
[ERROR] Authentication not possible, manual authentication required due to recaptcha. In order to login use the --token flag. You can get the token by logging into the website, and opening the dev console and running the command "localStorage.ott_token"

Additional Information
Hi!
I got the token but i'm not sure where i have tu put it.
I tried opening the "cmdhere.bat" and the command "--tokenflag [token]" but it didn't recognice it.
Help :,^

Hi!
I managed to log in with the CLI version but now i got this error:
Requesting: [E.7201] ME ENCANTAN LAS MAGICAL GIRLS - 13 - Ep. 13 [WARN] Decryption not enabled! [ERROR] You don't have access to download this content Skip muxing since no vids are downloaded
But I should have acces, i got Tier 2 subscription... I'm kinda sure i'm doing something wrong but i'm not sure what...

You need to place a CDM in the widevine directory


Can we provide a way to create a new stealth window。

I want to use an existing browser instance to create a new context,
In puppeteer：
browser.createIncognitoBrowserContext() // Target.createBrowserContext

Sure, we can. Just a simple function call: https://chromedevtools.github.io/devtools-protocol/tot/Target#method-createBrowserContext

If it can provide more convenient APIs, it should be able to attract more users.

Thanks for your feedback. Added in 9e1ace46d9e5d02d00a1f8a65eda48d873365b81, released in v0.23.1
Example code
Please close this ticket if you feel the implementation is what you want.


Check challenge before presenting it

With async challenge Check, it is often happens,
that solver.Check() fails on first run after solver.Present()
Cert-manager then tries again, but starts with solver.Present(),
which not being idempotent right now fails on certain DNS providers.
This change swaps order of solver.Check() and solver.Present().
Check is not returning error if propagation not happened, it then
allows Present() to run.
In the current form, Present() will be spamming with errors,
but this doesn't stop Check from happening on every attempt,
so eventually Challenge can be verified and accepted. In the future,
Present() should be made idempotent.

Thanks for picking this up - I was slightly nervous about how all the different DNS providers behave to be honest. Namely:

whether calling Present more than once is okay
whether calling CleanUp will error if the challenge hasn't been presented yet

I've updated the HTTP01 solver to make both Present & CleanUp idempotent.
Do you know of any misbehaving DNS providers? I'm keen to update those to help resolve this issue longer term 😄

we are using CloudFlare and it definitely not idempotent, POST request to create an entry returns "Records already exist"

can confirm that I was having a similar issue with Google Cloud Provider

grr. Okay, thanks for the feedback. Looks like more work to do!


Evaluate codeQL on e.g. on https://github.com/arburk/fishbone

https://github.com/github/codeql

what is cQL ?

CodeQL
QL is the query language that powers CodeQL. QL is an object-oriented logic programming language. GitHub, language experts, and security researchers create the queries used for code scanning, and the queries are open source. The community maintains and updates the queries to improve analysis and reduce false positives. For more information, see CodeQL on the GitHub Security Lab website.
https://docs.github.com/en/github/finding-security-vulnerabilities-and-errors-in-your-code

see also
https://semmle.com/codeql
https://lgtm.com/

conducted the tutorials https://help.semmle.com/QL/learn-ql/beginner/ql-tutorials.html and played around a little. Would like to discuss with someone also intereseted in this topic.

Let‘s do that!

current status & ideas

https://owasp.org/www-project-juice-shop/ (https://github.com/bkimminich/juice-shop/actions) is already using cQL
https://github.com/arburk/fishbone trying to reproduce very simple cQL issues

next steps

Next step(s)

[ ] execute / run checks locally on dev-machines
[ ] publish results within GitHub scope

Postponed

[ ] execute / run checks on Jenkins CI

The generated sarif can be added as artifact by the following workflow step
- name: Archive sarif files
    uses: actions/upload-artifact@v2
    with:
      name: Sarif file from codeQL scan
      path: /home/runner/work/fishbone/results/java-builtin.sarif

The path needs to be adjusted to related repo of course 😉

agenda for security show & tell

agenda for security show & tell

Evaluation is completed so far. Followup story to be created for an integrated usage with Jenkins/Sonarqube


Add VmOfToday utility

@ader1990 updated

@juhlee-microsoft  "If this is in LISAv2, then anyone can remove Azure RG with Secret file. I suggest this should be under another authentication before execution."
Could you help me understand this comment? Anyone has a secret file or has admin access to the subscription would be able to remove an RG or a lock on RG, it's not any issue related with the script.

@juhlee-microsoft "If this is in LISAv2, then anyone can remove Azure RG with Secret file. I suggest this should be under another authentication before execution."
Could you help me understand this comment? Anyone has a secret file or has admin access to the subscription would be able to remove an RG or a lock on RG, it's not any issue related with the script.

If anyone know our subscription and secret file, any accident unlocks the locked RG. It will remove RG eventually. Again, this script is good one, but I am not sure if we keep it in public.

@juhlee-microsoft  As I commented above, RGs would hard to be unlocked by accident using this script, but rather unlocked on purpose.


Daily Diary Entry WebApp

Project title
Daily Diary Entry Website
Project Description
A website with minimalist eye soothing frontend where you can log your daily life, like a diary entry.
The logs will be stored in the form of json and will be shown when you come back.
Web stack
HTML,
CSS
JS
Nodejs
Screenshot(mandatory)

Additional Context(mandatory)
Are you contributing under any open-source program ?
Yes, Codepeak 2023

how i can contribute on this project ???

how i can contribute on this project ???

You can enhance the project or fix the bugs after this PR is properly merged in this repo


Slow operation for retry when queued list is large, using LREM

Seeing an issue with a slow operation in the retry path.  I see that the code is using LREM for removing an item from the queued list when retrying.
https://github.com/tobymao/saq/blob/main/saq/queue/redis.py#L416
This command is marked as slow in Redis as it's O(N+M) to the size of the queue.
This can be particularly an issue when the queued list is large and there's a
a lot of tasks retrying / failing (which could happen during a degraded system state especially).
For example, trying this locally, if the queue size is 500k, it would slow down to a few hundred total operations per second.
The other issue is that it could also lock up other operations on the Redis server (for example other queues, caching, etc).
This is especially worse if the item is not in the list - which seems the expected case? Not sure on when it's expected
that the job being processed would also be in the queued list.
Fan of Saq so far, wanted to see if any there could be any potential improvements to the performance of this code path. Thanks!

i'm not sure if this is possible because the core data structure is to use lists for the main queue using blmove to _queued and _active
primary development on saq has moved to postgres because it's much easier to build a robust queue on there. so i'm going to mark this as not planned, but happy to discuss or open to contribution if you want to fix this.
but i'm not planning to put much effort into redis as postgres provides much more capabalities.

Question on this - for the retry use case, the item would already be move from queued to active using blmove, what's the
case for scanning the self._queued in addition to removing from active?
async def _retry(self, job: Job, error: str | None) -> None:
        job_id = job.id
        next_retry_delay = job.next_retry_delay()

        async with self.redis.pipeline(transaction=True) as pipe:
            pipe = pipe.lrem(self._active, 1, job_id)
            pipe = pipe.lrem(self._queued, 1, job_id)

my guess is that this was added to ensure that everything is wiped clean. even if it’s not in the active list. so retrying something that’s still queued. if you can prove that is unnecessary, we can remove it


Inversion of the icon

I use a garage door. In Homekit the icon is inverted: when I open the icon - bridge sent code to the closing, when I close the icon - bridge sent the code to the opening!

Hey @Maxwell70 can you end me your config please.

Hi @lprhodes
{
"platform":"BroadlinkRM",
"name":"Broadlink RM",
"hideScanFrequencyButton":false,
"hideLearnButton":false,
"accessories":[
{
"name":"NiVorota",
"host":"<IP_ADDRESS>",
"type":"garage-door-opener",
"openCloseDuration":"3",
"autoCloseDelay":"3",
"data":{
"open":"c2222c00.......",
"close":"b2071c....."
}
},

Please update to the latest version

@lprhodes What is the command to update?

@lprhodes After the update, the problem was solved! Thank You!!!!!!!


Glimpse does not permit open connection with azure tables

When I installed glimpse for MVC5 project I can not save info into azure tables, because does not permit to execute this line of code "  CloudStorageAccount maccount = CloudStorageAccount.Parse(CloudConfigurationManager.GetSetting("cxxxxxxxx"));".

Unfortunately there isn't much I can do at the moment. That said, the progress is being tracked at #226


Address pr feedback in black demo pr

What was wrong?
Addresses PR feedback on some minor formatting stuff in #581.
How was it fixed?
Addresses @pipermerriam 's feedback.
Cute Animal Picture

pinging @pipermerriam @carver @cburgdorf @lithp as this mostly touches stuff yall are closer to


[deployment-service] make update-bindings notable changes

:nut_and_bolt: Description: What code changed, and why?
while running make update-bindings for automate-deployment service getting notable changes due to updates in lib go-bindata

:chains: Related Resources
:+1: Definition of Done

As per the commit https://github.com/kevinburke/go-bindata/commit/1f34fcd85389344f29592b7efdefef286c76d7c0 fixes provided for all: do not capitalize error message.
Better comments and minor fixes for var canonicalName typo

:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist

[ ] Tests added/updated?
[ ] Docs added/updated?

:camera: Screenshots, if applicable
Signed-off-by: Vivek Singh<EMAIL_ADDRESS>

@stevendanna waiting for review. any comment?


Add linting target to Makefile to enforce code style for Python and JavaScript sources

Hey @ckadner! I would like to work on this issue.

Hey Phu, glad to see you back. Please go ahead!


Add Node 12 support

Description
This PR will make it possible to build this project in Node.js 12. Both deprecated and removed V8 features have been replaced with new equivalent features. Deprecated Nan features have also been replaced. Besides being able to build in Node.js 12, all build warnings about deprecated functions are now fixed.
Motivation and Context

Node.js 12 will probably very soon be the next LTS. The changes in this PR are critical to make it possible for Node.js 12 projects using this library.

Fixes issue #153
How Has This Been Tested?

Built in Travis CI
unit tests
integration tests
manual testing, using the code in the examples folder
added new component tests: testing new C++ code (converters.h)

npm test
total:     108
passing:   108
duration:  12ms

npm run integrationtest
(verbose output, skipped pasting here)
npm run test-components
total:     8
passing:   8
duration:  8ms

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read the code of conduct.
[x] I have read the contributing guide.
[ ] I have updated the documentation accordingly (if applicable).

I would very much appreciate to get feedback on this PR. My C++ skills are not good, and I am not sure about the new code written follow guidelines or handles memory issues correctly.
Maybe @kuebk can have a look?
Also @jbienkowski311 ?


Kiali Console opens up twice

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
We have rancher-istio 1.7.100 chart installed on Rancher 2.5.1.
When going to Istio on Cluster Explorer and clicking on Kiali in the Overview it opens Kiali on 2 tabs. It is expected to open only on 1 tab.
Result:

Other details that may be helpful:
Environment information

Rancher version: 2.5.1
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Kubernetes version (use kubectl version): 1.19.3

@tompatib Hello, what browser version are you using ?


Switch with bitmask option

Hi Mosibi great job, I have a question for the switch with the bitmask option. When I turn on the switch the bitmask goes on. When I turn off the switch the entire register goes to zero. I tested it both on the PC and on the heatpump. Do you have the same behavior?

Hi,
Can you tell with exact register that is, then I will have a look at it.

I tried with Register 5, I turned on Silent Mode and it worked. When I turned it off, it also set the climate curve to 0. I then connected the esp to the computer via a usb-rs485 adapter and configured it in slave via modbus slave software. When I turn on individual switches, the individual bitmasks are activated, when I turn off a switch it sets the entire register to 0.

And then you are referring to this register (5) and bit (7) which you change?? In the current configuration this is still a binary_sensor, so not configurable yet.
  # Bit: 7
  - platform: modbus_controller
    modbus_controller_id: "${devicename}"
    name: "${entity_prefix} Function Setting Silent Mode Level"
    id: "${devicename}_function_setting_silent_mode_level"
    register_type: holding
    address: 0x5
    bitmask: 0x80

Did you change the code yourself to make it possible to set this specific register/bit?

Yes, I changed the settings to switch.
switch:
 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "curve_on_off"
   id: curve_on_off
   register_type: holding
   address: 0x5
   bitmask: 0x1000

 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "silent_level"
   id: silent_level
   register_type: holding
   address: 0x5
   bitmask: 0x40

 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "echo"
   id: echo
   register_type: holding
   address: 0x5
   bitmask: 0x400

I analyzed better, when I activate one switch set all the others to 0. It's my mistake?

For the moment I use a Select sensor with lambda function to change only the bitmask, without changing the others, but it is complicated.
First I read the register and then I use the Select sensor with lambda to change the bitmask
sensor:
  # Registro 5 - Function setting
  - platform: modbus_controller
    modbus_controller_id: "${devicename}"
    name: "Function setting"
    id: function_setting
    register_type: holding
    address: 5
    value_type: U_WORD

select:
    # Registro 5 Bit 12 - Curve enable
  - platform: modbus_controller
    use_write_multiple: true
    modbus_controller_id: "${devicename}"
    name: "Curve enable"
    id: curve_enable
    address: 5
    value_type: U_WORD
    optionsmap:
      "Spenta": 0
      "Attiva": 4096
    lambda: |-
      //ESP_LOGE("main","Modbus Number incoming value = %d",x);
      //ESP_LOGE("main","Modbus eval value = %d",(x & 0x1000));
      if ((x & 0x1000) == 0)
        return  std::string("Spenta");
      if ((x & 0x1000) == 4096)
        return  std::string("Attiva");
      return {};
    write_lambda: |-
      //ESP_LOGE("main","Modbus write gets = %d",value);
      uint16_t unmodified =  id(function_setting).state;
      //ESP_LOGE("main","Modbus write unmodified = %d", unmodified);
      uint16_t modified = ((unmodified & ~0x1000) | value);
      //ESP_LOGE("main","Modbus write to write = %d", modified);
      return modified;

The lambda code makes it indeed a lot complexer. What you have to keep in mind is that you need to update the complete register in the lambda_write function.
If you look in the current heatpump.yaml in the master branch at register 270, https://github.com/Mosibi/Midea-heat-pump-ESPHome/blob/55c8d2bc704c80531018a7ab5d129a7eb80cdecb/heatpump.yaml#L2593, you see that I use "unmasked_value_register_270" which holds the original 16 bits value. That value is updated by the regular lambda (read) function on each run and in the write part, I update the specific bits and send that back to the heat pump.
When I develop such code, I often have the real write not enabled and use ESP_LOGI to show the incoming values and the outcome of the new value, until I am very sure that it delivers the right value.

Unfortunately I don't know the code that well.  I found that solution in a shared config file on another forum for another heatpump.  I just adapted it to my use, but it's still complicated.  With the sensor switch I have the behavior that I described in the previous post.  Does it work correctly for you?  Thank you

In the end I will make everything configurable, it will be just a matter of time. I am now working on other registers, but I only work on it in the weekends.
So continue trying and learning, or just wait 😄

You must use globals for it -i personally extend this bits. if you use switch tan modbus controller don't klnow other bits settings -so it write whole uint16 register with only set bit you send 0like silent mode and disable others ;(
Like in this project registers 270, 272, and t1s and curve is saved in globals and then used to get original value, changed to our specific change and saved to heatpump.
Also there is missing tbt and tbt2 values -other description than main midea clone -York Monoblock
So I modified processing for registers:

id: unmasked_value_register_210
type: uint16_t
restore_value: True
id: unmasked_value_register_211
type: uint16_t
restore_value: True
id: unmasked_value_register_0
type: uint16_t
restore_value: True
id: unmasked_value_register_5
type: uint16_t
restore_value: True
and modified in code with moved some sensors to text to get description, PMW value coded to % not position.
I've also add automatically turn on DHW or Heat after manually disable to automatically start eg after 3.5hour
I use cooling so i change by month from heating to cooling
Added own calculation for t1s as my curve temp which automatically set temp for heating
I use WANAS recuperation to get romms temp from Homeassistant ;)
Some of them is still in progress ;) -licke calculation start time DHW etc -i just moved from my code to esphome
I've add also (in progress) support to start external pump if external temp is below 3deg and PUMP_I don't run in last hour -i've got freezed pipes when heatpump was not turn on for 3 days in outside temp -20deg -i've got also other heat source tconnected to buffer ;)
Take a look for my fork ;) -i trying update manualy from this github -i've more changed  :) -especially added automation in direct esp -i preffer this way in case homeassistant server is unavalable for while ;)

Yes, I changed the settings to switch.
switch:
 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "curve_on_off"
   id: curve_on_off
   register_type: holding
   address: 0x5
   bitmask: 0x1000

 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "silent_level"
   id: silent_level
   register_type: holding
   address: 0x5
   bitmask: 0x40

 - platform: modbus_controller
   modbus_controller_id: "${devicename}"
   name: "echo"
   id: echo
   register_type: holding
   address: 0x5
   bitmask: 0x400

I analyzed better, when I activate one switch set all the others to 0. It's my mistake?

No this doesn't work as expected ;(

In the end I will make everything configurable, it will be just a matter of time. I am now working on other registers, but I only work on it in the weekends.
So continue trying and learning, or just wait 😄

I've got all configurable ;)
@Mosibi in my service manual for york (midea clone) heatpump registers 120 is for TBT1 temp and reg 121 for TBT2 -in your yaml there is Hydraulic Module Current 1 and 2 -is correct?
I've also updated decode hp appliance -with my available data from SM
# Register: 200 (High byte)
  - platform: modbus_controller
    modbus_controller_id: "${devicename}"
    name: "${entity_prefix} Home Appliance Type"
    id: "${id_entity_prefix}home_appliance_type"
    icon: "mdi:information-box-outline"
    register_type: holding
    address: 201
    response_size: 2
    raw_encode: HEXBYTES
    #value_type: U_WORD
    lambda: |-
         int idx = item->offset;
         std::string z = "";
         uint16_t rawdata = (uint16_t(data[idx]) << 8) + uint16_t(data[idx + 1]);
         //uint16_t rawdata = (data[idx ] );
         ESP_LOGI("main", "The current version is %s", String(rawdata,HEX).c_str());
         if ((rawdata >> 8) == 7) {
           z = "Air to water heat pump";
         } else {
           z = String(data[idx]).c_str();
         }
         return {z};
    #   - map:
    #     7 -> Air to water heat pump
  # Register: 200 (Low byte, first 4 bits)
  - platform: modbus_controller
    modbus_controller_id: "${devicename}"
    name: "${entity_prefix} Home Appliance Sub Type"
    id: "${id_entity_prefix}home_appliance_sub_type"
    icon: "mdi:information-box-outline"
    register_type: holding
    address: 201
    response_size: 2
    raw_encode: HEXBYTES
   # bitmask: 0xF000
    lambda: |-
         int idx = item->offset;
         std::string z = "";
         uint16_t rawdata = (uint16_t(data[idx]) << 8) + uint16_t(data[idx + 1]);
         ESP_LOGI("main", "The current version is %s", String(rawdata,HEX).c_str());
         if (((rawdata & 0x00F0) >> 4) == 2) {
           z = "R32";
         } else {
           z = String((rawdata & 0x00F0) >> 4).c_str();
         }
         return {z};
      # - map:
      #   2 -> R32
  # Register: 200 (Low byte, second 4 bits)
  - platform: modbus_controller
    modbus_controller_id: "${devicename}"
    name: "${entity_prefix} Home Appliance Product Code"
    id: "${id_entity_prefix}home_appliance_product_code"
    icon: "mdi:information-box-outline"
    register_type: holding
    response_size: 2
    raw_encode: HEXBYTES
    address: 201
    #bitmask: 0x0F00
    lambda: |-
         int idx = item->offset;
         std::string z = "";
         uint16_t rawdata = (uint16_t(data[idx]) << 8) + uint16_t(data[idx + 1]);
         ESP_LOGI("main", "The current version is %s rawdata ", String(rawdata, HEX).c_str());
         if (((rawdata & 0x0F) ) == 4) {
           z = "4";
         } else {
           z = String((rawdata & 0x0F) ).c_str();
         }
         return {z};
      # - map:
      #   2 -> R32

And a little more ;)

@Mosibi in my service manual for york (midea clone) heatpump registers 120 is for TBT1 temp and reg 121 for TBT2 -in your yaml there is Hydraulic Module Current 1 and 2 -is correct?

That is the annoying part of these registers, not every Midea (clone) has exact the same usage/description for a register. I assume that the registers depend on the software version used by the heatpump and that Midea uses a specific software version for each model/type. The Midea clones are probably based on a specific Midea type/model and thus get that software version.
At some point I think I have to create multiple versions of the heatpump.yaml, based on the software version or model/type name.

I've also updated decode hp appliance -with my available data from SM

I would be nice if you could create a pull request for this.
If scrolled through your code and noticed that you are doing things like setting the leaving water temperature, based on the outside temperature, so that you have more control than which is originally present. I do the same, but I keep that outside the yaml/controller and use Home Assistant for that.
My goal is to keep the configuration as dumb as possible and only add "enriched" sensors like your sliding time window for the outside temperature.
If you have such sensors that add information that can be used for logic/automations (in Home Assistant) which are useful for others, I gladly accept a pull request for them!
You have been busy!! 😉

@Mosibi Ye you right. The code should be simple and universal for others so it isn't neede to include -especially sensors from outside of heatpump -i think only things useful for others and working as standalone from this device is good to apply ;) -
I put commit with added functionallity to be able to read/write registers 0, 5, 210, 211 -i think thats all R/W registers (0 and 5 is corrected to remember original value -in other case like in this issue other values are reseted), ive added calculated COP, and human readable uptime of sensor,
I think that removed !secrets isn't good choice -it's easier to apply editing secrets than running with phone to do captive portal (personally i don't even know what it looks), and after do factory reset -it is needed to run again with phone to this controller... ;)
I think functionality to reenable heat pump DHS and zones to automatically start after defined time would be also great -especially when you change some parameyters -heatpump automatically turn off and you have to manually turn on. I think with added it with configurable switch to enable/disable this funcyion and defined by user time -configurable from www or hjomeassistant ;)
I also noticed that when heatpump monoblock (like mine) in winter with lower temps, and when other heat source work also and heat pump don't run for some time eg few days -then water in pipes can freeze -so i put own external pump to be energized for few minuttes if pump_i doesn't run for more than some time eg 1-2 hours -driven by separate pin on ESP connected to Relay or SSR and this connected to external pump to make some water flow inside pipes in heatpump ;)

So @Mosibi you can close this PR after last commit pull request -there was started with register 5. or you can wait to apply rest register 210 and 211 -then you get whole config enabled by www or homeassistant or other ;)

I will close this as soon as I merged your PR for register 5


track and output clock delay

Feature request:
Per-track delay option to offset a track by N milliseconds, also available for the output clock.
This would help solve an issue for me that has nothing to do with the Performer, but with other digital modules in my rack that suffer from latency. Being as Performer is a "brain" type of sequencer, being able to solve this here would be the most elegant solution in my opinion, but of course I'm biased as this is a big problem for me.
Some context to more fully illustrate the issue I have:
I am using the Performer as my master clock via clock out.
I am using 4 tracks on the Performer to trigger 4 analog voices.
The remaining 4 tracks are going into a digital module that has 9ms latency (an ER-301, but pretty much all digital modules / samplers are going to have some latency).
To work around this issue I:

added the minimum gate offset to all the tracks that are triggering analog voices (12.5% I believe it is)
sacrificed an entire track with the same gate offset to act as master clock for the rest of the system (this is the one that hurts a lot)
used the remaining 3 tracks without a gate offset to trigger drums on the ER-301
the minimum offset value actually pushes everything just a tiny bit too far from the non-offset triggers after they go through the 9ms latency, but fortunately the ER-301 has a micro-delay so I compensate with that (this is also dependent on tempo so I have to readjust)

So again being able to apply a static millisecond offset to individual tracks as well as the clock out would be amazing as I would be able to keep everything super tight and wouldn't have to use a track for the output clock. Kind of edge request perhaps I really don't have an idea how big of an ask this is implementation wise, but thanks for your consideration.

I noticed this problem when sending clock out from performer to Grids. Grids just consistently seemed to be off by a few ms. Perhaps this could fix that issue?

Grids (at least my grids) requires the reset sent to it to come slightly ahead of the clock (otherwise you get a weird stutter sometimes)


Ubuntu 14.04 LTS Connection Failure

Hi Yeri
After installing you hotspotd and running it two things I observe

ubuntu won't connect to internet anymore
device connects but data not downloading

@udoyen Can you paste the console log (the text that appears on the console) since hotspotd is run?

Closing as there has been no activity.


Is a licence required?

@Mosaddeq107 did you add the license because of a requirement or you just wanted to?

I think we need a license. I have seen group 1 has already made it.

@Mosaddeq107 did you add the license because of a requirement or you just wanted to?

It's not mandatory. However, Dr. Tim encouraged us to add a license for each homework.

Okay, thanks.


Circular dependency issues related to Python packages residing at epanet-python repo

Circular dependency issues are introduced when moving python wrapper packages from the EPANET repo to the epanet-python repo. This issue documents the circular dependency and hopefully through discussion will result in a sound strategy for addressing them.

@samhatchett thanks for the reminder yesterday. I had missed this in Issue #123
The Python package scikit-build integrates CMake with python setup tools.


Superfluous entry

Already covered by ECHO, %OUT.

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

ID: 95d3ef20-3495-450e-21bb-c7fe9f82092c
Version Independent ID: 83ab427b-96e5-7a4f-23a7-641b52a78ff0
Content: %OUT
Content Source: docs/assembler/masm/percent-out.md
Product: visual-cpp
Technology: cpp-masm
GitHub Login: @corob-msft
Microsoft Alias: corob

@Player1UK thanks for the feedback! However, after considering pros and cons, the product team has decided that the entries should not be merged.


How to impl scan in this db ?

Scan will cross shards, on which thread should scan be executed, and how should iterators organize it

Scan needs to go to the threads that contain the keys you want to scan.
The iterator should live in the client, rust has an AsyncIter only in nightly, so I would probably implement a trait that has a next function that the user needs to call manually, like I did in iter_filter.
To make this performant you would probably need to implement a new response type that streams the results from the server, instead of closing after a single response.
Not a simple feature, but it basically comes down to this.


[Question]: How to use chat stores along with chat memory buffer for saving contextual memory for different users?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
How to use chat stores and chat memory buffers for persisting and loading contextual chat message history for differnt users?
I have explored the chat stores and understood that we can configure different chat stores for different users with a unique user id.
from llama_index.core.storage.chat_store import SimpleChatStore
from llama_index.core.memory import ChatMemoryBuffer

chat_store = SimpleChatStore()

chat_memory_user1 = ChatMemoryBuffer.from_defaults(
    token_limit=3000,
    chat_store=chat_store,
    chat_store_key="user1",
)

chat_engine = index.as_chat_engine(
    chat_mode="context",
    memory=chat_memory_user1,
    system_prompt=(
        "You are a chatbot, able to have normal interactions, as well as talk"
        " about an essay discussing Paul Grahams life."
    ),
)

We can have a different chat memory buffer object per user id and hence give the appropriate context to our chat engine.
My question is regarding persisting these memory objects and retrieving the appropriate object when an old user invokes the application and have the chatbot remember their context. Is there any way defined in llama index to do this? Are they allowing persistant storage for chat memory buffer objects?
If anyone is aware about how to solve this issue, it will be of great help. Thanks!

You can load/persist the simple chat store to/from disk, but its up to you to pull the appropriate history using the key
For example,
chat_store.persist("./chat_store.json")

chat_store = SimpleChatStore.from_persist_path("./chat_store.json")

chat_memory_user1 = ChatMemoryBuffer.from_defaults(
    token_limit=3000,
    chat_store=chat_store,
    chat_store_key="user1",
)

You just need to keep track of which chat_store_key maps to which user

@logan-markewich okay! Just another question, when we persist the chat_store object, it would replace the existing keys in the file. Is this the expected behaviour? Shouldn't it append the chat_store.json file with new user id as the key?

@logan-markewich okay! Just another question, when we persist the chat_store object, it would replace the existing keys in the file. Is this the expected behaviour? Shouldn't it append the chat_store.json file with new user id as the key?  In case where there are multiple chat store objects to be persisted within the same file, this method will simply overwrite each persist as it is in write mode.

Having multiple chat entrypoints for chat data management doesn't look good from a design perspective, unnecessary complexity


verifying reported disk/memory usage test appears to be checking disk usage only once

On this line of the lifecycle test it looks like the intention was to check disk over and over until the CF_PUSH_TIMEOUT is reached.  However, because the anonymous function is called immediately it does not check repeatedly.  Instead, it will check only once and then repeatedly check the float return value of the function (which won't ever change, since it was set the first and only time the function was called) until the timeout is reached.

No longer calling the anonymous function immediately: https://github.com/cloudfoundry/wats/commit/39de616021f16ddea9984b3299839780ae50476c
Tests passed and merged to master: https://greenhouse.ci.cf-app.com/teams/main/pipelines/greenhouse/jobs/pesto-bosh-deploy-windows/builds/95


JIRA-4693: add a test after bug fix

@krystal can you please review?
http://<IP_ADDRESS>:8080/job/Framework-Regression/638/console

LGTM +1


Supporting the evolving Rider Plugin

Whatever changes we need to support the completion of the base functionality of the Rider plugin project.

Runtime support:

[ ] Handling draft upgrades

Authoring support:

[ ] Building toolkits
[ ] Adding/editing elements
[ ] etc..


missing corelib_binds.rs

we were trying to run the test's for the library and continue to get theses errors we even tried to regen_binds but nothing changes the error  the header files are present in /usr/include/mono-2.0/mono as the build script expects
OS Debian GNU/Linux 12
Mono JIT compiler version <IP_ADDRESS>
apt mono-complete
Compiling wrapped_mono v0.3.1 (/workspaces/New folder/wrapped_mono)
error: couldn't read src/corelib_binds.rs: No such file or directory (os error 2)
--> src/bindgen.rs:58:35
|
58 |             self.target.write_all(include_bytes!("corelib_binds.rs"))?;
|                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: this error originates in the macro include_bytes (in Nightly builds, run with -Z macro-backtrace for more info)
warning: unused imports: Method, TypeDefinition
--> src/bindgen.rs:1:23
|
1 | use crate::metadata::{Method, TypeDefinition};
|                       ^^^^^^  ^^^^^^^^^^^^^^
|
= note: #[warn(unused_imports)] on by default
warning: unused import: MethodTable
--> src/bindgen.rs:2:23
|
2 | use crate::metadata::{MethodTable, TypeDefOrRef, TypeDefinitionTable, TypeReferenceTable};
|                       ^^^^^^^^^^^
warning: unused import: std::path::PathBuf
--> src/bindgen.rs:8:5
|
8 | use std::path::PathBuf;
|     ^^^^^^^^^^^^^^^^^^
warning: unused import: MonoReflectionType
--> src/reflection_type.rs:1:32
|
1 | use crate::binds::{MonoObject, MonoReflectionType, MonoType};
|                                ^^^^^^^^^^^^^^^^^^
warning: unused imports: InteropRecive, InteropSend
--> src/reflection_type.rs:5:27
|
5 | use crate::{InteropClass, InteropRecive, InteropSend};
|                           ^^^^^^^^^^^^^  ^^^^^^^^^^^
warning: wrapped_mono (lib) generated 5 warnings
error: could not compile wrapped_mono (lib) due to previous error; 5 warnings emitted

It seems you have pulled directly from GitHub? This file was indeed missing, and should now be present after 5e39dbcdd4f4cba03c290972e3ac97ae12e07801. I should however note that you would probably be better off using the stable version from crates.io. The GitHub version does have slightly more features, but they are mostly unfinished, as you can see from your issues. The GitHub version should not be used for anything else than development of wrapped_mono.
The missing file comes from an experimental C# to Rust binding generator, something not yet included in stable versions. It is also why regen_binds did nothing - it is not about any binding to mono, but something only relevant to generating Rust bindings for C# libraries. Did the fix resolve your issues?

Closed because of inactivity.


[BUG] Page Pattern layout - /layouts/pattern

Informations

Type
Version

Bug
2.x

Description
Property 'addContext' does not exist on type 'Logger'. Did you mean 'context'?ts(2551)
Logger.d.ts(20, 9): 'context' is declared here.

Todos

[ ] Example (log, usecase, etc...)
[x] Ts.ED version (bug/question)

Hello @karnavpargi
You’ve right, it’s logger.context.add :)
Thanks for the issue!


[WIP] Relay node interface

Coverage remained the same at 90.431% when pulling 87fb4880197d618722675f3adfd10ff04aab0c73 on relay-node-interface into 2a5c7bda92efdf24e8e79b06f542d090ea0c7b4f on admin-graphql.

please, write tests for middlewares


There is a problem with the website address

Hello, http://bvisionweb1.cs.unc.edu/licheng/referit/data/images/saiapr_tc-12.zip and Download MSCOCO from mscoco
Are these two websites invalid?

Hi, can you try these two links?
http://images.cocodataset.org/zips/train2014.zip
https://drive.google.com/file/d/1R6Tm7tQTHCil6A_eOhjudK3rgaBxkD2t

mscoco is train2014.zip. But the second one should be  a subset of saiapr_tc-12? You mentioned in refer/data/README.md

Both of these links can be used, but are both the original dataset and not the subset you extracted?

The subsets will be extracted in the data processing and loading codes.

Okay, thank you. I'll give it a try when I have time. And your ideas are great.

Thank you!

Hello, I wanted to bring to your attention that I have noticed the Visual Genome images dataset link is currently invalid. When trying to access it, I encountered an error message indicating that the link is not working.

Hello, I wanted to bring to your attention that I have noticed the Visual Genome images dataset link is currently invalid. When trying to access it, I encountered an error message indicating that the link is not working.

Thank you for letting us know. Indeed the visual genome website is down. Alternatively, you can use the following link to download the visual genome images:
https://cs.stanford.edu/people/rak248/VG_100K_2/images.zip
 https://cs.stanford.edu/people/rak248/VG_100K_2/images2.zip


Transforming Data During Dump

Is it possible to use this tool to transform data during the dump? For example to hide a name or email field?

Nope - the code is essentially just a pure JS implementation of the mysqldump binary.
If this is important to you, happy to accept a PR


Trouble loading mod with Forge MDK

I was trying to alter this mod for 1.12.2, just some minor changes for myself. I got the workspace all set up and there was an error that showed up instantly without me changing anything, and I was unable to find another solution. Would you know anything about this?

It seems pretty important, and hovering over it says the field GuiIngame.persistantChatGUI is not visible.

Looks like the access transformers did not load correctly, try running setupDecompWorkspace again

Nothing changed.

No idea what's wrong, check the gradle logs, maybe they have some info.


Investigate the difference between rendering pipelines of ManualDocumentRenderer and SpecialistDocumentRenderer

As per @h-lame's comment on PR #798.

It feels like it might be a bug that this and the ManualDocumentRenderer you deal with in the next few commits are subtly different; this one doesn't do FootnoteSomethingSomething in its pipeline, but the ManualDocumentRenderer does. We should check obviously, but I think we can get rid of this one and replace it's single use with an instance of the ManualDocumentRenderer instead.

@chrislo mentioned that he's addressing this in his current branch.

I fixed this in d88b3891 as part of the renaming from ManualDocument to Section


Cassandra timeout at start of titan.sh

Hi,
I just wanted to try out Titan with Cassandra and Rexster but I get a timeout when trying to start it. I just execute bin/titan.sh start -v of Titan 0.5.4 on an Ubuntu 14.04 x64 VM.
Here are the last lines of output I get on the terminal:
 INFO 16:20:56,030 Enqueuing flush of Memtable-local@854850358(168/4635 serialized/live bytes, 7 ops)
 INFO 16:20:56,030 Writing Memtable-local@854850358(168/4635 serialized/live bytes, 7 ops)
 INFO 16:20:56,031 Compacting [SSTableReader(path='db/cassandra/data/system/local/system-local-jb-7-Data.db'), SSTableReader(path='db/cassandra/data/system/local/system-local-jb-8-Data.db'), SSTableReader(path='db/cassandra/data/system/local/system-local-jb-5-Data.db'), SSTableReader(path='db/cassandra/data/system/local/system-local-jb-6-Data.db')]
 INFO 16:20:56,037 Completed flushing db/cassandra/data/system/local/system-local-jb-9-Data.db (174 bytes) for commitlog position ReplayPosition(segmentId=1433168455193, position=174230)
 INFO 16:20:56,056 Node localhost/<IP_ADDRESS> state jump to normal
........................... timeout exceeded (60 seconds)
See /home/user/tmp/titan-0.5.4-hadoop2/bin/../log/cassandra.log for Cassandra log output.

and the last line of the log file log/cassandra.log:
 INFO [FlushWriter:1] 2015-06-01 16:20:56,037 Memtable.java (line 403) Completed flushing db/cassandra/data/system/local/system-local-jb-9-Data.db (174 bytes) for commitlog position ReplayPosition(segmentId=1433168455193, position=174230)
 INFO [main] 2015-06-01 16:20:56,056 StorageService.java (line 1556) Node localhost/<IP_ADDRESS> state jump to normal
 INFO [CompactionExecutor:3] 2015-06-01 16:20:56,173 CompactionTask.java (line 275) Compacted 4 sstables to [db/cassandra/data/system/local/system-local-jb-10,].  1.131 bytes to 594 (~52% of original) in 107ms = 0,005294MB/s.  4 total partitions merged to 1.  Partition merge counts were {4:1, }
 INFO [main] 2015-06-01 16:20:56,197 Server.java (line 155) Starting listening for CQL clients on localhost/<IP_ADDRESS>:9042...
 INFO [main] 2015-06-01 16:20:56,273 ThriftServer.java (line 99) Using TFramedTransport with a max frame size of 15728640 bytes.
 INFO [main] 2015-06-01 16:20:56,275 ThriftServer.java (line 118) Binding thrift service to localhost/<IP_ADDRESS>:9160
 INFO [main] 2015-06-01 16:20:56,282 TServerCustomFactory.java (line 47) Using synchronous/threadpool thrift server on localhost : 9160
 INFO [Thread-2] 2015-06-01 16:20:56,283 ThriftServer.java (line 135) Listening for thrift clients...

Since Cassandra seems to be running (which is what bin/titan.sh status tells me) and I cannot see any errors except for the timeout, I cannot figure out why Titan fails to start. Any help is highly appreciated.

Perhaps you can look at #790 as this issue has been reported before.  Perhaps you could comment with respect to the questions @dalaro posted here:
https://github.com/thinkaurelius/titan/issues/790#issuecomment-67065775
and here:
https://github.com/thinkaurelius/titan/issues/790#issuecomment-67194187
as I suspect his questions would largely be the same on this issue.

Thanks, I just tried some suggestions in #790 and noticed a problem with my Java path when running nodetool. After resolving this, Titan started successfully :-)
Sorry for the inconvenience but I really couldn't figure out where the problem was with the error messages I got. Maybe, you could print / log errors reported by nodetool at the end of the timeout?


Removing the switch between Linux and Windows containers on Docker Desktop for Windows

Tell us about your request
For Docker Desktop for Windows, the switch between Windows and Linux Containers should be removed on Windows version supporting both type of containers simultaneously.
Which service(s) is this request for?
Docker Desktop for Windows
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Single control plane for both Windows and Linux Docker containers on Windows
Are you currently working around the issue?
Switching between them is a story of the past.

Thanks, this is a duplicate of #79. Feel free to go and upvote that one.

No, it's the same. If you read the original ticket, it says "The option to work streamless with Linux and Windows container is hard, because of the switch context option."


New design for Autocomplete in TimerBar

⚠ Warning

This PR is branched off from #2844, so it's critical to merge it before reviewing in this PR 🍰

📒 Description

This PR dedicatedly introduce new design for Timer Bar and keep maintaining the original logic for AutoComplete in Edit Time Entry Popover. As the design is totally different and should be in separate ticket 🎫

🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] Refactor how AutoComplete feature works (Refactor code, re-use ProjectLabel components, fix bug and improve the logic to calculate the height of AutoCompleteContainer, ...)
[x] Implement new design

👫 Relationships
Closes #2814
🔎 Review hints

Test fully all case of AutoComplete and make sure it works 💯

Rebased this PR from #2844

I fixed the issue where the color of project name dot was wrong. Also there was an issue that client dot was visible when there was no client added to the time entry.

Currently the focus ring looks very strange and doesn't fit the timer design. Is it possible to make the focus ring to be similar to the input field design (bigger corner radius). Might be possible with someting like https://stackoverflow.com/a/10569801/1015296

Autocomplete does not have a bottom constraint and when window has smaller height the bottom of the listing is outside of the view.

It's ready to review in second phrase. Please make sure to check out new branch from origin since I've rebased it 👍


Add methods to recalculate and toggle static geometry

SteamAudioGeometry wasn't working properly for my procedurally generated geometry as my actual geometry is being globally positioned after SteamAudioGeometry::_ready is called. So I needed a way to "recalculate" the static audio geometry.
I figured I'd also add an easy way to enable and disable the audio geometry.
https://github.com/stechyo/godot-steam-audio/assets/46084870/cb2d45b3-1ac6-40f1-848b-b6989899cd95
I had some issues getting the GDExtension to build properly on Windows (msvc).
scons platform=windows target=template_debug dev_build=true built and worked fine. Omitting the dev_build=true flag builds fine, but the Godot project would instantly crash.
I did notice quite a size difference between my dll (~500KB) and the one provided in project/addons (~1.8MB). I tried building with use_mingw=true and after about 30-40 minutes it finally finished (subsequent builds weren't much faster). In any case, it did end up producing a dll around the same size and the GDExtension worked fine. The build time is quite the hindrance though. I'll probably find a solution with a fresh pair of eyes tomorrow (likely related: https://github.com/godotengine/godot/issues/30892).

Hi, thanks for this! I hadn't thought of this use-case and your code looks good (just had some nitpicks).
I'll try to test your PR this week, but I'll only merge it after #33 and after releasing 0.1.0. I did not want to introduce any more changes in this release, hope you understand.
Semi-unrelated things:
Omitting dev_build=true creates a different binary name, so it might be the reason for your crashing.
I'm building the Windows binary with mingw mostly because I don't use Windows except for testing this extension's releases. I wasn't aware that MSVC binaries were that much smaller, so I'll probably make a Windows/MSVC step when I get around to doing #31. Just a check, size and compilation time is the only difference between MSVC and MinGW builds, right?
Feel free to keep building on MSVC for now, since there doesn't seem to be a difference.

I did not want to introduce any more changes in this release, hope you understand

Totally fine! I'm more than okay with running my own builds for the time being.

Just a check, size and compilation time is the only difference between MSVC and MinGW builds, right?

I couldn't get MSVC builds to run without dev_build=true. I'm quite certain it's more than just naming as I did ensure the binary paths matched up in libgodot-steam-audio.gdextension.

Is it possible that it has something to do with the fact that my fork of Godot is compiled on MinGW?

(Will test your changes this weekend and merge if all the review comments are solved and it looks good. Great contrib, thanks!)

Did you leave review comments? Missing repo setting?
https://github.com/stechyo/godot-steam-audio/assets/46084870/f2216c23-0436-4fc1-a162-10c2d17a6715

... sorry, new to having other people's PRs on my own repos. I think they should show up now.

Just tested, works fine and no crashes. Merging, thanks!


CORS headers are reset after deploy

The Gateway AWS interface currently has a streamlined interface for setting up CORS. The problem is that deploying from fluct resets these response headers. There is currently no way to specify a CORS headers such as Access-Control-Allow-Origin in the configuration.

+1


Unable to register new Project when Okta enabled on my domain

When trying to create a project on app.staging.medplum.com, I was unable to create a new project from my medplum.com email. The I tried both logged in and logged out of Medplum, using the Google Auth button. The Staging Environment has a DomainConfiguration set for the medplum.com domain, so  suspect that this might be causing the issue.
In both cases, I am logged in to Okta
Repro when I'm logged in to Medplum:

Navigate to https://app.staging.medplum.com/register
Redirected to https://app.staging.medplum.com/signin?project=new
Presented with a sign-in form
Click on Google Auth button
Redirected to https://app.staging.medplum.com/signin?login=xxx&scope=openid&nonce=xxx&code_challenge=xxx&code_challenge_method=S256
Presented with a profile chooser dialog
If I click on a project, am logged into that project

Repro when I'm logged out of Medplum:

Navigate to https://app.staging.medplum.com/register
Presented with "Create new account" form
Click Google Auth button
POST request goes out to https://api.staging.medplum.com/auth/google. 200 Response comes back with payload:
{"domain":"medplum.com","authorizeUrl":"https://dev-71569655.okta.com/oauth2/v1/authorize?client_id=xxx&redirect_uri=https%3A%2F%2Fapi.staging.medplum.com%2Fauth%2Fexternal&response_type=code&scope=openid+profile+email"}, but nothing else happens

Confirmed


Fix random seeding in LanguageModel.generate

In the current implementation of LanguageModel.generate, the random number generator is re-seeded (in fact re-instantiated) before every word:
https://github.com/nltk/nltk/blob/66f56ead003e64c5bd215427c8d09b4a5fd7fd78/nltk/lm/api.py#L89-L92
This is especially unfortunate when random_seed is not None, because it makes the model always predict the same token for a given context.
This PR fixes this issue by seeding the generator only once.

@cifkao thanks for the finding and fixing this. A neat little bug :)
:+1: from me!

Thanks @cifkao


[urllib3 Warning ] -  SubjectAltNameWarning: Certificate for xyz.example.com has no subjectAltName, falling back to check for a commonName for now.

While running ACSTIS today on the JS app hosted on heroku, I happen to get some warnings. Will this affect the scan (I am guessing no) is there a way we could ignore them in the acstis script.
I tried scanning other websites and I feel the scanner runs fine, Just that I am getting these warnings now. Somehow I did not get these errors the first time I tested this new version (Surprise !!)
_PS Angular-CSTI-Scanner\angularjs-csti-scanner-master> python .\extended.test.py -c -d "https://owaspjuiceshop221b.herokuapp.com/#/search" -tc "Burp_CA_Cert.pem"
Version 3.0.1 - Copyright 2017 Tijme Gommers<EMAIL_ADDRESS>[INFO] Looking for AngularJS version using a headless browser.
[INFO] Waiting until DOM is completely loaded.
[INFO] Found AngularJS version 1.5.11.
[INFO] Angular CSTI scanner started.
[INFO] Scanning https://owaspjuiceshop221b.herokuapp.com/#/search
C:\Python27\lib\site-packages\urllib3\connection.py:344: SubjectAltNameWarning: Certificate for owaspjuiceshop221b.herokuapp.com has no subjectAltName, falling back to check for a commonName for now. This feature is being
removed by major browsers and deprecated by RFC 2818. (See https://github.com/shazow/urllib3/issues/497 for details.)
SubjectAltNameWarning
C:\Python27\lib\site-packages\urllib3\connection.py:344: SubjectAltNameWarning: Certificate for owaspjuiceshop221b.herokuapp.com has no subjectAltName, falling back to check for a commonName for now. This feature is being
removed by major browsers and deprecated by RFC 2818. (See https://github.com/shazow/urllib3/issues/497 for details.)
SubjectAltNameWarning
C:\Python27\lib\site-packages\urllib3\connection.py:344: SubjectAltNameWarning: Certificate for owaspjuiceshop221b.herokuapp.com has no subjectAltName, falling back to check for a commonName for now. This feature is being
removed by major browsers and deprecated by RFC 2818. (See https://github.com/shazow/urllib3/issues/497 for details.)
SubjectAltNameWarning_

@jovyn Thank you for reporting this issue. I will make sure these warnings will only be visible if debug mode is enabled.

I just fixed this issue in the crawler develop tree. Issue #6 needs to be fixed in the crawler as well. I will release a new version of the crawler as soon as I fixed #6, then I'll update ACSTIS with the new version of the crawler.

@jovyn I fixed this issue on the develop branch by disabling urllib3 warnings in N.Y.A.W.C. Could you test if this works for you?


Dependency deadlock in vehicle_school

I wanted to reproduce your results for the vehicle benchmark, however, while trying to install your requirements.txt file, I have run into a dependency issue.
At the first installation, (pip install -r requirements.txt) I get:

To solve these issues, I run a force reinstall for the packages with the mentioned versions in the Error.
pip install --force-reinstall -v "numpy==1.21" pip install --force-reinstall -v "jax==0.4.6" pip install --force-reinstall -v "typing-extensions==4.2.0"
Now, when I try to install jaxlib 0.4.7, I get the following error:

Clearly, optax 0.1.0 which is a dependency for jaxlib 0.4.7 requires typing-extensions 3.10.0 but chex 0.1.7 which a dependency for another package, requires typing-extensions>=4.2.0
Can you help me with this issue? I would suggest that you update the repo with a pip freeze output in the requirements.txt file so that the dependencies are taken care of automatically. That way, the user can simply do a  pip install --no-dependencies -r requirements.txt to install everything flawlessly.

Hi @prarabdhshukla
Thanks for your interest! I think the dependency requirements for JAX related packages are flexible and any version you can get working should generally be fine. The requirements.txt file was written a while ago and some of these packages are updated frequently.
You could probably just to forget the specified versions in requirements.txt and install the newest versions of these packages, and I can help you with any runtime errors that may occur. Alternatively you could also try the set of JAX related dependencies here which I wrote more recently.
I'll try to update the requirements.txt here hopefully soon. Feel free to follow up here if you have any additional questions.

Hey @kenziyuliu !
Thanks for your response. Sure, I will try out these things and let you know if I face any problems.
Thanks again!


Provide a way to adapt StreamWriter into OutputStream

Related Issue #3092
Motivation:
StreamMessageWriter<HttpData> reactive = ,,,;
OutputStream out = reactive.asOutputStream();
objectMapper.writeValue(out, myJsonObject);

I want to provide a way to adapt StreamMessage into InputStream and OutputStream

#4059

Modifications:

Add StreamMessageOutputStream to adapt StreamMessage to OutputStream
Add StreamMessage.toOutputStream()

Result:

You can now use StreamMessage.toOutputStream() to adapt StreamMessage to OutputStream
✅ Close #3092 issue

I was meaning to add StreamMessageWriter.toOutputStream() rather than StreamMessage.toOutputStream()

oh, I have to add toOutputStream on StreamMessageWriter! it's seems more easier.
I'll update PR based on your comments! thanks for checking 🙇

Sorry for late 😢, I'll update this PR until 04/17!

Don't be in a hurry. Take your time. :-)

📌 Sorry for late, I'll update this PR until this Wednesday

We have discussed the backpressure design of OutputStream.
The OutputStream created by StreamWriter.toOutputStream() has some downsides on backpressure.

OutputStream.write() is blocked infinitely if the StreamMessage spawned the OutputStream is not subscribed.
Potentially, OutputStream.write() can unexpectedly block an event loop.

So I'd like to change the design of this API to safely write data to OutputStream 1) on a blocking task executor 2) after the StreamMessage is subscribed.
interface StreamMessage {
  static ByteStreamMessage fromOutputStream(Consumer<OutputStream> outpuStreamWriter) { ... }

  // The executor is used to invoke outpuStreamWriter
  static ByteStreamMessage fromOutputStream(Consumer<OutputStream> outpuStreamWriter, Executor executor) { ... }
}

The APIs above could be used like:
HttpResponse serve(...) {
    var stream = StreaMessage.fromOutputStream(this:startService);
    return HttpRsponse.builder(headers)
                      .content(stream)
                      .build()
}

private void startBlockingService(OutputStream os) {
    // perform blocking operation
    var data0 = readDataFromSomeWhere0();
    // os.write() will be blocked if the buffer is full and the subscriber of the returned HttpResponse hasn't consumed data0 yet.
    os.write(data0);
    var data1 = readDataFromSomeWhere1();
    os.write(data1);
    os.close();
}

https://line-armeria.slack.com/archives/CRQ5NNZQU/p1660663862501869

📌 (for history) I leave question on slack about review comment! PTAL.

https://github.com/line/armeria/pull/4186#issuecomment-1211699740, So I'd like to change the design of this API ..
Slack, Let's close this PR and introduce StreamMessage.fromOutputStream() with another PR ..

📌 (For history) Due to change of API design, we decide to close this PR and continue with another PR!
So I close this PR. thanks to all reviewers!


Not able to change config storage value

There doesn't seem to be a get/set function to change the value of the config storage.  I'm not able to use session storage with $authprovider.storage = 'sessionStorage';, but it works if I hard code it into the module.
storage: {
  get: function() { return config.storage; },
  set: function(value) { config.storage = value; }
}

Should be at ~line 190

Noticed this right before you submitted it. Here is a PR for it
https://github.com/sahat/satellizer/pull/381


readdirp related circular dependency detect issue

Describe the bug
chokidar seems to use readdirp 3.5.0
I don't know why this happens. On github, chokidar 3.5.1 uses readdirp 3.6.0
But, after npm download, chokidar uses readdirp 3.5.0
readdirp 3.5.0 has circular dependency detect issue on Windows 10 (it was fixed on 3.6.0)
So, I think I need chokidar to use readdirp 3.6.0 on Windows 10 to avoid this issue.
Thank you.
Versions (please complete the following information):

Chokidar version: 3.5.1 (latest)
Node version: 12.17
OS version: Windows 10

To Reproduce:
yarn
Expected behavior
package.json dependency includes readdirp 3.6.0
Actual behavior
package.json dependency includes readdirp 3.5.0

Closed due to cannot reproduce