Is this the expected output?

Not so much an issue, as a discussion.
I used this script to generate my keyboard layout. I have a custom keyboard, that is not part of ZMK by default. I ran:
./zmk-viewer generate -u -f prototype-config/config/prototype.keymap -l zmk/prototype/info.json cradio -o ./keymap_pictures

I got this:

As you can see, it is not very readable. I do have 4 layers, but that isn't too much, is it?
My keyboard has nothing to do with cradio. I just used it as a placeholder. The generated image looks more like my keyboard than cradio, I believe, thanks to the info.json.

Hi, currently it is the expected output, I'm aware of the text overlay, but I'm not sure about how to solve it, maybe using the symbols instead of the name...
There's the option to generate a different image for each layer, "solving" this issue, but I agree this needs a real solution


undefined method ts_msec

Hey thanks so much for this gem!
I noticed that ts_msec has been defined for Time but not for ActiveSupport::TimeWithZone and that is why I got this error:
irb(main):002:0> test_ts = Redis::TimeSeries.new('test', redis: $redis_enterprise)
=> #<Redis::TimeSeries:0x000000010c1c7b88 @key="test", @redis=#<Redis client v4.5.1 for redis://<IP_ADDRESS>:6379/0>>
irb(main):003:0> test_ts.add(1234, Time.current)
/Users/thushara/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/bundler/gems/rails-fc4d83f43cd3/activesupport/lib/active_support/time_with_zone.rb:535:in `method_missing': undefined method `ts_msec' for Thu, 21 Jul 2022 17:41:05.380072000 UTC +00:00:Time (NoMethodError)

      wrap_with_time_zone time.__send__(sym, *args, &block)
                              ^^^^^^^^^
/Users/thushara/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/bundler/gems/rails-fc4d83f43cd3/activesupport/lib/active_support/time_with_zone.rb:535:in `method_missing': undefined method `ts_msec' for 2022-07-21 17:41:05.380072 UTC:Time (NoMethodError)

      wrap_with_time_zone time.__send__(sym, *args, &block)
                              ^^^^^^^^^
irb(main):004:0>

Wondering if there is an easy fix/work-around?

Hey, I took a quick look and wasn't able to reproduce on the latest version of the gem. Here's what my environment looks like right now:
[16] pry(main)> RUBY_VERSION
=> "3.0.2"
[17] pry(main)> Redis::VERSION
=> "4.7.1"
[18] pry(main)> Redis::TimeSeries::VERSION
=> "0.7.0"
[19] pry(main)> ActiveSupport.version
=> Gem::Version.new("<IP_ADDRESS>")
[20] pry(main)> @ts1
=> #<Redis::TimeSeries:0x00007f8afc23f148 @key="ts1", @redis=#<Redis client v4.7.1 for redis://<IP_ADDRESS>:6379/0>>
[21] pry(main)> @ts1.add(1234, Time.current)
=> #<Redis::TimeSeries::Sample:0x00007f8afd9e8768 @time=2022-07-21 14:00:50 449839/1048576 -0500, @value=0.1234e4>

Can you share your versions as well?

Never mind, I just reproduced - it's because the return value of Time.current changes depending on whether or not Time.zone is set (https://github.com/rails/rails/blob/6-0-stable/activesupport/lib/active_support/core_ext/time/calculations.rb#L39)
[38] pry(main)> Time.zone = nil
=> nil
[39] pry(main)> @ts1.add(1234, Time.current)
=> #<Redis::TimeSeries::Sample:0x00007f8afda03978 @time=2022-07-21 14:05:22 187695/1048576 -0500, @value=0.1234e4>
[40] pry(main)> Time.zone = 'UTC'
=> "UTC"
[41] pry(main)> @ts1.add(1234, Time.current)
NoMethodError: undefined method `ts_msec' for Thu, 21 Jul 2022 19:05:26.571629000 UTC +00:00:Time
from /Users/dzunk/.gem/ruby/3.0.2/gems/activesupport-<IP_ADDRESS>/lib/active_support/time_with_zone.rb:535:in `method_missing'
Caused by NoMethodError: undefined method `ts_msec' for 2022-07-21 19:05:26.571629 UTC:Time
from /Users/dzunk/.gem/ruby/3.0.2/gems/activesupport-<IP_ADDRESS>/lib/active_support/time_with_zone.rb:535:in `method_missing'

As a workaround you can call .to_time on the TimeWithZone object:
[50] pry(main)> @ts1.add(1234, Time.current)
NoMethodError: undefined method `ts_msec' for Thu, 21 Jul 2022 19:07:46.762628000 UTC +00:00:Time
from /Users/dzunk/.gem/ruby/3.0.2/gems/activesupport-<IP_ADDRESS>/lib/active_support/time_with_zone.rb:535:in `method_missing'
Caused by NoMethodError: undefined method `ts_msec' for 2022-07-21 19:07:46.762628 UTC:Time
from /Users/dzunk/.gem/ruby/3.0.2/gems/activesupport-<IP_ADDRESS>/lib/active_support/time_with_zone.rb:535:in `method_missing'
[51] pry(main)> @ts1.add(1234, Time.current.to_time)
=> #<Redis::TimeSeries::Sample:0x00007f8afda48910 @time=2022-07-21 14:07:48 2080375/2097152 -0500, @value=0.1234e4>

That should just work though - will get that fixed up!

Released this as 0.7.1 - try out that version and see if it's fixed 😄


Basic verification for responses

This PR adds a new crate called integration_tests which has a build script that generates tests from all response variants. The tests have two forms:
For variants that don't hold data:
  #[test]
  fn validate_terminatethreads_response() {
    let resp = Response {
      request_seq: 1,
      success: true,
      message: None,
      body: Some(ResponseBody::TerminateThreads),
    };
    let value = resp_to_value(&resp);
    assert!(SCHEMA.is_valid(&value));
  }

For variants that hold data:
  #[test]
  fn validate_threads_response() {
    let resp = Response {
      request_seq: 1,
      success: true,
      message: None,
      body: Some(ResponseBody::Threads(ThreadsResponse::default())),
    };
    let value = resp_to_value(&resp);
    assert!(SCHEMA.is_valid(&value));
  }

So this relies on the Default impl for each type. Obviously, this will mean that any Options will be None so many fields are not actually tested. But this is a good starting point. syn can be used to discover the struct fields and set them to generate more kinds of tests.
DRAFT: currently I cannot make the tests fail by renaming a field completely so I'm doing something wrong.

Also, feel free to add me as a maintainer 😀

I hope to look at it over the weekend 👍🏽 I was actually about to ask about making a new release with all the new fixes. I think after this is merged is a great time.
Regarding generating test data, would it be worth creating an example application that could talk to some of the most popular editors, so we could record the interaction and get some data that way? Then we could ensure that we were not only compliant with the spec, but that we also (sort of) works with those editors, in case they have some quirks or additions?


[Accessibility]A11y_VSCode_SourceControl_Keyboard: Tooltip is not visible to keyboard only users but only visible on mouse-hover.

Environment Details:
VSCode Version : 1.24.0
OS Version : Win10
Additional Details:
MAS Violated : MAS 1.1.1 , MAS 2.1.1
Repro Steps:

Launch VS Code.
Navigate to Activity Bar and select "Source Control"(Cntrl+Shift+G) button.
Navigate to "Commit","Refresh","More","Initialize repository","Stage/Unstage all changes","Stage/Unstage changes","Open File" etc. buttons using "Tab".
Also, mouse hover on all above mentioned buttons.

Actual:
Tooltip information is not available for keyboard only users for "Commit","Refresh","More","Initialize repository","Stage/Unstage all changes","Stage/Unstage changes","Open File" etc. buttons which is available on mouse hover.
Expected:
Tooltip information should be made available for keyboard only users which is available using mouse.
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
The keyboard only users will not be able to gain information regarding few controls present on screen which is only available using mouse hover.
MAS Reference:
MAS 2.1.1 - https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
MAS 1.1.1 - https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={d2d2051f-bdc8-4af7-8e18-38aae867e216}
Attachment For Reference:
A11y_VSCode_SourceControl_Keyboard_Tooltip.pptx
Does this issue occur when all extensions are disabled?: Yes

Dup of https://github.com/Microsoft/vscode/issues/52016


Use headless service for master pods in K8s

This fixes https://github.com/Alluxio/alluxio/issues/11562

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9999/
Test PASSed.

closed in favor of https://github.com/Alluxio/alluxio/pull/11593

closed in favor of https://github.com/Alluxio/alluxio/pull/11593


Warning - FanEntityFeature.TURN_OFF & FanEntityFeature.TURN_ON is not set but implemented

Version of the custom_component
Midea Air Appliance (LAN) 0.9.4
Configuration
HA Core 2024.9.1
2x Comfee Dehumidifier16L-32m²-DF3(WF)
Describe the bug
Warning appeared first time after updating HA from 2024.8.3 to 2024.9.1. The logger requests that this bug report be generated.
Debug log
Logger: homeassistant.components.fan
Quelle: components/fan/init.py:295
Integration: Lüfter (Dokumentation, Probleme)
Erstmals aufgetreten: 07:28:48 (4 Vorkommnisse)
Zuletzt protokolliert: 07:28:48
Entity None (<class 'custom_components.midea_dehumidifier_lan.fan.DehumidiferFan'>) does not set FanEntityFeature.TURN_OFF but implements the turn_off method. Please create a bug report at https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/issues
Entity None (<class 'custom_components.midea_dehumidifier_lan.fan.DehumidiferFan'>) does not set FanEntityFeature.TURN_ON but implements the turn_on method. Please create a bug report at https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan/issues

This is a duplicate of #163 and was fixed with #164. We're just waiting for a new version release.

THX a lot. I added the snipplet. Works great!


Permit transient frame to be deleted if no text fields contain text, nor children added

This could be implemented with another flag.
Alternatively, validityStatus, and constructionStatus could be two enums.

Covered by #32


adapt to new hardware interface read and write methods

There were some changes in https://github.com/ros-controls/ros2_control/pull/715, that added a time and period parameter to read and write methods. This adapts the packages to the new interface.
FYI, building without the changes causes:
/home/ijnek/tmp_workspaces/dynamixel_control_ws/src/dynamixel_control/dynamixel_hardware/include/dynamixel_hardware/dynamixel_hardware.hpp:81:15: error: ‘hardware_interface::return_type dynamixel_hardware::DynamixelHardware::read()’ marked ‘override’, but does not override
   81 |   return_type read() override;
      |               ^~~~
/home/ijnek/tmp_workspaces/dynamixel_control_ws/src/dynamixel_control/dynamixel_hardware/include/dynamixel_hardware/dynamixel_hardware.hpp:84:15: error: ‘hardware_interface::return_type dynamixel_hardware::DynamixelHardware::write()’ marked ‘override’, but does not override
   84 |   return_type write() override;

I'm not sure if replacing the read()call with read(rclcpp::Time{}, rclcpp::Duration(0, 0)); is good, (and vice-versa with write). What do you think about that part?

https://github.com/youtalk/dynamixel_control/pull/24#issuecomment-1247998660


Incremental product update

Is their a way to tigger a single product update from Pimcore to ES? Or do i need to run this indexer every 15 mins?

Or a single product update.


Sciex Wiff2 files converted to MZML looses intensity

Hello,
My name is Cagakan from SCIEX, UK support team.
One of our customer is asking:
"When converting the wiff2 files to mzml the peak areas and intensities are much lower than when looking at them in SCIEXOS."
It would be great if you have an answer for that,
Thanks for your time.
Best.
Cagakan

Is this customer Ms. Panzenboeck? I've responded to her emails about this issue recently.


Multiple versions of GraphQL?

Detailed Description
When adding Kimera to an Apollo Server project the server raises an error because of multiple versions of the graphql dependency being present.
Expected Behavior
It'd be nice if this didn't happen , it is a bit of a roadblock for new users.
Current Behavior
The server crashes when trying to serve mocks
Workaround
I've worked around this in a yarn-based project by using the resolutions field in package.json, I don't know what I'd have done if I'd been in npm.

This should help: https://indepth.dev/npm-peer-dependencies/


[sdl2pp] Fix exported pc file, cleanup

What does your PR fix?
Fixes SDL2main target appearing in pc file, https://github.com/microsoft/vcpkg/pull/23898#issuecomment-1085792419
Rectifies and trims the dependencies patch.
Moves the installed find module to share/${PORT}.
Rectifies the usage instructions.

Which triplets are supported/not supported?  Would it be fine for you to include this PR in #25803?

@dg0yt This would break my sdl2-ttf PR (again). Would it be fine for you to include this PR in #25803?

I'm aware of the overlap, but I think the PRs are too different in scope. For review and traceability, it is better treat them individually IMO.

Thanks!


ROX-24374: add profile result response

Description
add profile result response
Add control data to endpoints:

GetComplianceScanResults

curl --insecure https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results -u admin:XXX | json_pp
allresults.json

GetComplianceScanCheckResult works

curl --insecure https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/result/4195f7a4-4525-4d8d-a5ae-f4917ed965e9 -u admin:XXX | json_pp
specificcheckuid.json

GetComplianceScanConfigurationResults works

curl --insecure https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results/ocp4-cis -u admin:XXX | json_pp
config-all.json

GetComplianceProfileResults works

curl --insecure "https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results/profiles/ocp4-cis-node-1-5/checks" -u admin:XXX | json_pp
profile-all.json

GetComplianceProfileCheckResult works

curl --insecure "https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results/profiles/ocp4-cis-node-1-5/checks/ocp4-cis-node-1-5-worker-kubelet-eviction-thresholds-set-hard-nodefs-inodesfree" -u admin:XXX | json_pp
profile-specific-check.json

GetComplianceProfileClusterResults works

curl --insecure "https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results/profiles/ocp4-cis-node-1-5/clusters/6e34af90-077f-4105-88cd-4e81f347043b" -u admin:XXX | json_pp
profile-cluster.json

GetComplianceProfileCheckDetails works

curl --insecure "https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/results/profiles/ocp4-cis-node-1-5/checks/ocp4-cis-node-1-5-worker-kubelet-eviction-thresholds-set-hard-nodefs-inodesfree/details" -u admin:XXX | json_pp
profile-specific-check-details.json

GetComplianceProfileCheckStats works

curl --insecure "https://central-stackrox.apps.ds-06-10-scrawny-cream-snow.ocp.infra.rox.systems:443/v2/compliance/scan/stats/profiles/ocp4-cis-node-1-5/checks/ocp4-cis-node-1-5-worker-kubelet-eviction-thresholds-set-hard-nodefs-inodesfree" -u admin:XXX | json_pp
profile-specific-check-stats.json

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~- [ ] Evaluated and added CHANGELOG entry if required~
~- [ ] Determined and documented upgrade steps~
~- [ ] Documented user facing changes (create PR based on [openshift/openshift-docs]~(https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

QUery both endpoints without any data in central-db
Query endpoints with CO installed
Query endpoints with CO installed + results

Testing Results

Add 404 Error if profile was not found in stats endpoint

# RESULT: SHOULD RETURN 404?
❯ roxcurl /v2/compliance/scan/results/profiles/ocp4123/checks
{"profileResults":[],"profileName":"ocp4123","totalCount":0}%                                                                                ❯ roxcurl /v2/compliance/scan/results/profiles/ocp4123/checks

❯ roxcurl /v2/compliance/scan/results/profiles/ocp4-cis/checks
{"profileResults":[],"profileName":"ocp4-cis","totalCount":0}%

❯ roxcurl /v2/compliance/scan/results/profiles/ocp4/checks/test
{"checkResults":[],"profileName":"ocp4","checkName":"test","totalCount":0}%

====================================================

# with check and profile not existing
❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4/checks/test
{"profileResults":[],"profileName":"ocp4","totalCount":1}%

# with non existing check
❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis/checks/test
{"profileResults":[],"profileName":"ocp4-cis","totalCount":1}%

# with no checks
❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis
{"scanStats":[],"totalCount":0}%

# with existing check/rule not part of ocp4-cis
❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis/checks/rhcos4-usbguard-allow-hid
{"profileResults":[],"profileName":"ocp4-cis","totalCount":1}%

# with check part of ocp4-cis
❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis/checks/ocp4-file-permissions-multus-conf
{"profileResults":[],"profileName":"ocp4-cis","totalCount":1}%

❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis/checks/ocp4-cis-api-server-admission-control-plugin-alwaysadmit | jq

{
  "profileResults": [
    {
      "checkName": "ocp4-cis-api-server-admission-control-plugin-alwaysadmit",
      "rationale": "Enabling the admission control plugin AlwaysAdmit allows all requests and does not provide any filtering.",
      "ruleName": "ocp4-api-server-admission-control-plugin-alwaysadmit",
      "checkStats": [
        {
          "count": 0,
          "status": "FAIL"
        },
        {
          "count": 0,
          "status": "INFO"
        },
        {
          "count": 1,
          "status": "PASS"
        },
        {
          "count": 0,
          "status": "ERROR"
        },
        {
          "count": 0,
          "status": "MANUAL"
        },
        {
          "count": 0,
          "status": "INCONSISTENT"
        },
        {
          "count": 0,
          "status": "NOT_APPLICABLE"
        }
      ],
      "controls": [
        {
          "standard": "CIS-OCP",
          "control": "1.2.10"
        },
        {
          "standard": "NERC-CIP",
          "control": "CIP-003-8 R6"
        },
        {
          "standard": "NERC-CIP",
          "control": "CIP-004-6 R3"
        },
        {
          "standard": "NERC-CIP",
          "control": "CIP-007-3 R6.1"
        },
        {
          "standard": "NIST-800-53",
          "control": "CM-6"
        },
        {
          "standard": "NIST-800-53",
          "control": "CM-6(1)"
        },
        {
          "standard": "PCI-DSS",
          "control": "Req-2.2"
        }
      ]
    }
  ],
  "profileName": "ocp4-cis",
  "totalCount": 1
}

❯ roxcurl /v2/compliance/scan/stats/profiles/ocp4-cis | jq

{
  "scanStats": [
    {
      "checkStats": [
        {
          "count": 9,
          "status": "FAIL"
        },
        {
          "count": 0,
          "status": "INFO"
        },
        {
          "count": 59,
          "status": "PASS"
        },
        {
          "count": 0,
          "status": "ERROR"
        },
        {
          "count": 21,
          "status": "MANUAL"
        },
        {
          "count": 0,
          "status": "INCONSISTENT"
        },
        {
          "count": 0,
          "status": "NOT_APPLICABLE"
        }
      ],
      "profileName": "ocp4-cis",
      "title": "CIS Red Hat OpenShift Container Platform 4 Benchmark",
      "version": "1.5.0"
    }
  ],
  "totalCount": 1
}

/test ocp-4-15-compliance-e2e-tests

Empty query:
❯ roxcurl /v2/compliance/scan/results
{"error":"Unable to process compliance rule for query pagination:\u003climit:1000 \u003e : invalid arguments","code":3,"message":"Unable to process compliance rule for query pagination:\u003climit:1000 \u003e : invalid arguments","details":[]}%

/retest


why some places is not suggested as autocompleteService

seems this is not an issue I just need to define the type that I want to use the default is geocode for all i need to add more type https://developers.google.com/places/supported_types#table3
why when I write place, some place it not shown for autocomplete,
but when I try to write it in maps.google.com, google have autocomplete for that place
is typeahead-addresspicker use different autocompleteService?
can I use the autocomplete that maps.google.com have?
so I have do some research
and i found it theres is two type autocomplete that google have
https://developers.google.com/maps/documentation/javascript/examples/places-autocomplete
https://developers.google.com/maps/documentation/javascript/examples/places-autocomplete-addressform
there is type parameter in googleautocomplete the first one use all and the second one is 'geocode'
It seems typeahead use the second one
I assume that because the resullt that the second one give
is same as when I use typeahead addresspicker

having the same issue .what if i want to use getQueryPredictions() instead of getPlacePredictions() ?


MSBuild Android support

Add enable *-android targets integration with VS Android projects:
Here is simple patch which enabled compilation:
 scripts/buildsystems/msbuild/vcpkg.targets | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/scripts/buildsystems/msbuild/vcpkg.targets b/scripts/buildsystems/msbuild/vcpkg.targets
index 8807befd8..ddd3f6c0a 100644
--- a/scripts/buildsystems/msbuild/vcpkg.targets
+++ b/scripts/buildsystems/msbuild/vcpkg.targets
@@ -49,6 +49,30 @@
     <VcpkgTriplet Condition="'$(VcpkgTriplet)' == ''">arm64-uwp</VcpkgTriplet>
   </PropertyGroup>

+  <PropertyGroup Condition="'$(Platform)|$(ApplicationType)|$(ApplicationTypeRevision)' == 'x86|Android'">
+    <VcpkgEnabled Condition="'$(VcpkgEnabled)' == ''">true</VcpkgEnabled>
+    <VcpkgTriplet Condition="'$(VcpkgTriplet)' == ''">x86-android</VcpkgTriplet>
+    <VcpkgAutoLink Condition="'$(VcpkgAutoLink)' == ''">false</VcpkgAutoLink>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Platform)|$(ApplicationType)|$(ApplicationTypeRevision)' == 'x64|Android|3.0'">
+    <VcpkgEnabled Condition="'$(VcpkgEnabled)' == ''">true</VcpkgEnabled>
+    <VcpkgTriplet Condition="'$(VcpkgTriplet)' == ''">x64-android</VcpkgTriplet>
+    <VcpkgAutoLink Condition="'$(VcpkgAutoLink)' == ''">false</VcpkgAutoLink>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Platform)|$(ApplicationType)|$(ApplicationTypeRevision)' == 'ARM|Android|3.0'">
+    <VcpkgEnabled Condition="'$(VcpkgEnabled)' == ''">true</VcpkgEnabled>
+    <VcpkgTriplet Condition="'$(VcpkgTriplet)' == ''">arm-android</VcpkgTriplet>
+    <VcpkgAutoLink Condition="'$(VcpkgAutoLink)' == ''">false</VcpkgAutoLink>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Platform)|$(ApplicationType)|$(ApplicationTypeRevision)' == 'ARM64|Android|3.0'">
+    <VcpkgEnabled Condition="'$(VcpkgEnabled)' == ''">true</VcpkgEnabled>
+    <VcpkgTriplet Condition="'$(VcpkgTriplet)' == ''">arm64-android</VcpkgTriplet>
+    <VcpkgAutoLink Condition="'$(VcpkgAutoLink)' == ''">false</VcpkgAutoLink>
+  </PropertyGroup>
+
   <PropertyGroup Condition="'$(VcpkgEnabled)' == 'true'">
     <VcpkgConfiguration Condition="'$(VcpkgConfiguration)' == ''">$(Configuration)</VcpkgConfiguration>
     <VcpkgNormalizedConfiguration Condition="$(VcpkgConfiguration.StartsWith('Debug'))">Debug</VcpkgNormalizedConfiguration>

But I see following in Build console:
1>Target AppLocalFromInstalled:
1>  dumpbin : The term 'dumpbin' is not recognized as the name of a cmdlet, function, script file, or operable program.
1>  Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
1>  At D:\dev\vcpkg\scripts\buildsystems\msbuild\applocal.ps1:60 char:12
1>  +     $a = $(dumpbin /DEPENDENTS $targetBinary | ? { $_ -match "^    [^ ...
1>  +            ~~~~~~~
1>      + CategoryInfo          : ObjectNotFound: (dumpbin:String) [], CommandNotFoundException
1>      + FullyQualifiedErrorId : CommandNotFoundException

$(Platform)|$(ApplicationType)|$(ApplicationTypeRevision)

do you have any experience in integrating this platform under cmake ?

@Voskrese  no I don't, I have no experience in custom toolchains with MS toolsets. I am interested in using vcpkg built libraries in VS Android via vcxproj/msbuild at the moment, arm64/clang. This setup as I understand uses NDK compilation toolchain.

strange, as far as I know, compilation under ANDROID in VCPKG does not affect MSBUILD
why then do these edits?
I thought that you have some experience using PLATFORMTOOLSET ANDROID MSBUILD

This issue is not about Android compilation(worked out of the box for me). Its about vcpkg integration with VS/msbuild. I just reverse engineered those properties from msbuild output, trying to integrate.

https://docs.microsoft.com/en-us/visualstudio/extensibility/visual-cpp-project-extensibility?view=vs-2019
https://github.com/microsoft/vcpkg/blob/master/scripts/cmake/vcpkg_build_msbuild.cmake
https://github.com/microsoft/vcpkg/blob/master/scripts/cmake/vcpkg_install_msbuild.cmake

Please note changes are done to scripts/buildsystems/msbuild/vcpkg.targets.
This script is used for INTEGRATION.
Please see https://vcpkg.readthedocs.io/en/latest/about/faq/#i-cant-use-user-wide-integration-can-i-use-a-per-project-integration
As I understand it is installed when vcpkg integrate install is run. I tried my changes on project locally first in order to not break my system wide integration.

D:\dev\vcpkg\scripts\buildsystems\msbuild\ applocal.ps1
$( dumpbin.exe /DEPENDENTS $targetBinary
this script with dumpbin looks for and copies dependencies

so did your edit help you avoid further errors and what was the port?

you're writing code in MSVC for the ANDROID platform and trying to integrate VCPKG into your project, right?
open PR https://github.com/microsoft/vcpkg/pulls

Yes this issue is about adding Visual Studio integration for Android projects. My patch is just demonstration that this is possible and should be relatively straightforward, but I am not an expert in MSBuild and vcpkg, and I had troubles with copying part - that is why I used static linkage in my triplets.
So I have opened issue to add that support by more knowledgeable engineer. And left my preliminary findings in order to simplify things.

Another hint for person responsible for fix.
Thinking on discussion here, I checked global integration. And it does not work. Problem seems to be in MSBuild files.
It seems Microsoft.CppCommon.targets relies on Microsoft.Cpp.Common.props but it is not imported for Clang toolset. So my fix was to add VCLibPackagePath property to that file. After that I was able to import android libraries.
...
  <PropertyGroup>
    <VCLibPackagePath Condition="'$(VCLibPackagePath)' == ''">$(LOCALAPPDATA)\vcpkg\vcpkg.user</VCLibPackagePath>
  </PropertyGroup>
  <Import Condition="'$(VCLibPackagePath)' != '' and Exists('$(VCLibPackagePath).targets')" Project="$(VCLibPackagePath).targets" />
...


URLLint: make URL checks concurrent

What this PR does / why we need it

The urllint checks find all instances of a URL in our repo, then tries
to perform a GET on each link to check that it is a valid link. This was
being done sequentially, resulting in a lot of time waiting for the GET
request to respond before continuing on to the next URL.
This updates the urllint code to perform these requests with some
concurrency so we can perform some operations while waiting to get a
response.
With local testing, this changed the make urllint job go from just
under 60 seconds to check all links, to just under 4.
Describe testing done for PR

Prior to code changes, ran time make urllint to get a benchmark of the sequential operations:
58.175 total

After making the changes, ran time make urllint again several times. Results would fluctuate +/- a second or two, but much faster:
3.434 total

To make sure things were still operating as expected, introduced a typo in one of the urls in MAINTAINERS.md and verified it gave the appropriate error message:
-------------------------------  Fail Summary  ---------------------------------------------
File Path: /Users/smcginnis/src/vmware-tanzu/community-edition/MAINTAINERS.md
URL: https://github.com/ssssstmcginnis
Url Position: 9 : 31
Message: HTTP Status Code: 404

Total  Fail            : 1
---------------------------------------------------------------------------------------
exit status 1

Special notes for your reviewer

Awesome!


Multiple Validators for the same input field.

I believe that it would be very useful to have a note about multiple validators for the same field.
I was able to accomplish this with:
this.myForm = fb.group({
  'cardNumber': new FormControl('', Validators.compose([Validators.required, CustomValidators.creditCard]))
});

@phixMe In addition, the view is like this
<input type="text" formControlName="cardNumber" placeholder="cardNumber"/>
<p *ngIf="myForm.controls.cardNumber.errors?.creditCard || myForm.controls.cardNumber.errors?.required">invalid card</p>

Validation works but the ngIf doesn't


move env files to root path

Move the .env files to the root react path ./react. .env files must be in this path for them to pick up custom variables.

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


Update CNI to v1.3.0 for v1.12

Manual backport of https://github.com/cilium/cilium/pull/25400. Need to update image tags after build completes.
Update CNI (loopback) to 1.3.0

sorry for the noise :sweat_smile: wrong base


Reuse call stack when invoking filter

What type of PR is this?

/label tide/merge-method-squash
What this PR does / why we need it:
Preallocates a call stack for wasm invocation to reduce allocations during the filter hot path. This improves performance of function invocation a little bit.
Which issue(s) this PR fixes:

Special notes for your reviewer:
This project may not be at a stage for such small optimizations so no need to proceed if so - I saw some perf work happening so figured it may help to send it
Does this PR introduce a user-facing change?

NONE

What are the benchmark results of this change?

goos: darwin
goarch: arm64
pkg: sigs.k8s.io/kube-scheduler-wasm-extension/internal/e2e
                                            │   old.txt   │              new.txt              │
                                            │   sec/op    │   sec/op     vs base              │
PluginFilter/noop/params:_small-10            201.1n ± 3%   190.6n ± 3%  -5.17% (p=0.002 n=6)
PluginFilter/noop/params:_real-10             200.9n ± 3%   189.5n ± 2%  -5.67% (p=0.002 n=6)
PluginFilter/filter-simple/params:_small-10   8.588µ ± 3%   8.249µ ± 4%  -3.95% (p=0.041 n=6)
PluginFilter/filter-simple/params:_real-10    58.85µ ± 3%   59.30µ ± 2%       ~ (p=0.589 n=6)
geomean                                       2.126µ        2.050µ       -3.54%

/cc @codefromthecrypt

thanks for lighting turn-around as this drifts my local stuff and I was a bit anxious when it would land. All good!


Cold staking with pi issues

Hi
I have recently accessed my particl wallet which  was running staking via raspberry pi from aroud 2018.
The last time the staking transactions were updated were in june 2018. Trying to get this updated but forgotton the public key for my staking pool.
I have plugged in my pi which I set up for staking a few years ago and trying to update. None of the commands from  github are working and I keep getting the same error message.

partyman requires updating. lastest version is : 0.16
do "git pull" manually or download the lastest version.

I have tried the gitpull command line but had no success

Hi @tommygunn123 .
If you go into the "partyman" directory and type "git pull" it should update to the latest.
If that fails, you can rename "partyman" to "partyman_old" and run the "git clone" command again.


Changed the README after some of its contents have been moved to the wiki

The build failures are totally unrelated with this change. @mssola please take care of them


docs: Add installation instructions via Copr repository

To easen installation and maintenance of Warden on Fedora systems, I initiated a Copr repository.
For now the spec file is hosted on a separate Git repository on Pagure.io, if you'd like to host the RPM spec file in the main Warden repository, let me know :).
EDIT
The Copr package now also places the systemd-resolved file (as described by #360) and restarts systemd-resolved after installing/upgrading.

Great idea! Would like to have similar thing for Ubuntu

@davidalger, what do you think?


Export DecoratingTrait

Fix #18

Codecov Report

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

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   96.78%   96.78%
=======================================
  Files           9        9
  Lines         778      778
=======================================
  Hits          753      753
  Misses         25       25

Impacted Files
Coverage Δ

src/compgraph.jl
94.28% <ø> (ø)
:arrow_up:

src/NaiveNASlib.jl
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e90b9d...80d3fb1. Read the comment docs.


disable coloring when NO_COLOR env var is set

A person @jcs suggests the standard way of disabling color. See http://no-color.org/ https://twitter.com/matsuu/status/962252252508340224
If colo supports this disabling switch, the dependent tools (like eater, eatest, agreed, saku etc) automatically conform the above standard and probably that makes everyone happy!

-1, NO_COLOR environment option should not be supported by the colorize module.
colorize module feature is simple: colorize strings on console . (it is a unix philosophy right?)
NO_COLOR env is the feature of those CLI environment, like eater, saku, agreed.

OK. That's a design decision.


Any way to get packages strategy to work with merge module plugin?

Hi,
I'm attempting to use the the packages entry strategy along with typedoc-plugin-merge-modules. However, as the docs state, plugins do not run on individual packages, only in the global typedoc config. I'm attempting to merge at the module level with custom @module tags.
If I run each project's docs individually and output to json, and then merge manually, the plugin works - however cross-package links are not resolved.
Perhaps this is an issue for the plugin repo, but I suppose my question here is: are there plans for using plugins on sub-projects? Or should plugins applied globally work on each sub-project?
Thanks

Those docs should probably be refined a bit -- plugins can affect child packages, but they won't be loaded for every single child. They'll be loaded once at the root level, then each child will be converted with that same application instance (though there will be a different Options instance for each package). However the merge module plugin is set up apparently doesn't work with this... I'm not familiar with its implementation, but I can't think of any reason it wouldn't work out of the box right now.

It looks like it is set up to only listen to one of the events for conversion, rather than merging when converters are finished, and when merging is finished.
https://github.com/krisztianb/typedoc-plugin-merge-modules/blob/master/src/plugin.ts#L55-L66
The comment there is wrong, as TypeDoc won't create a new Converter instance, but it looks like this is intentional..

Appreciate the investigation! I'll take a closer look when I have a chance.


Philips Hue Aurelle (model 3216331P6) measurements added

Can you rebase your PR, as this contains commits/code now (wiz profiles) which is already on master.

Ah, my apologies, I am not familiar enough of git (and all attempts to rebase failed :). I'll close this pull request until I figure out how to do this properly.


add clues.Hide to replace observe.PII

The observe PII handlers are getting replaced by
the more standardized clues secrets.

Does this PR need a docs update or release note?

[x] :no_entry: No

Type of change

[x] :broom: Tech Debt/Cleanup

Issue(s)

#2024

Test Plan

[x] :muscle: Manual
[x] :green_heart: E2E

Not the hash on the user name
Connecting to repository:      3s done
Connecting to M365:         done
Backing Up ∙ 9be2915234bb5525
Discovering items to backup:   2m26s done

After injecting clues.SetHasher(clues.NoHash()) into the cli executor:
Connecting to repository:      3s done
Connecting to M365:         done
Backing Up ∙<EMAIL_ADDRESS>
And with clues.SetHasher(clues.HashCfg{HashAlg: clues.Flatmask})
Connecting to repository:      3s done
Connecting to M365:         done
Backing Up ∙ ***


add coredev april 2023

FYI: I'm not seeing these on https://btctranscripts.com/bitcoin-core-dev-tech/ , but maybe I'm looking in the wrong place?

Fixed

follow-up: is there a reason we don't have all of the "Core Dev Tech" notes bucketed by date? We do for the 2015 meetup but not the others. If there are no objections, I'll open a PR to bucket the others. Having them bucketed would make it much easier to reference "here is what was covered at the last core dev" , etc

follow-up: is there a reason we don't have all of the "Core Dev Tech" notes bucketed by date? We do for the 2015 meetup but not the others. If there are no objections, I'll open a PR to bucket the others. Having them bucketed would make it much easier to reference "here is what was covered at the last core dev" , etc

It seems reasonable. You'll just need to account for the url change in order to avoid broken links. To do that you can create aliases, see https://github.com/bitcointranscripts/bitcointranscripts/pull/219

If there are no objections, I'll open a PR to bucket the others.

no objections


fix(stylis): ensure rules are split coming out of stylis

When vendor prefixing is enabled, the default behavior in stylis is for them to come out as a clump. Unfortunately, this cases issues with CSSOM because some browsers reject rules containing styles with unknown selectors, e.g. Chrome might reject a -moz prefix or vice versa.

Going to bring this in for the next rc as a partial fix.


Core is removed when delete it in title edit page

repro steps:
•	Navigate to https://pxt.microbit.org/beta
•	Click button “More…”->”Project Setting”
•	Click “Edit Settings As text”
•	remove "core": "", "radio": ""
{
"name": "test",
"dependencies": {},
"description": "",
"files": [
"main.blocks",
"main.ts",
"README.md"
]
}
•	Click Blocks
Expect result: core should not be deleted
Actual result: core is deleted and simulator disappear

Shared script page: https://pxt.microbit.org/61648-03705-46752-45529

@gujenkin when applying the logic about adding missing packages, we should always ensure that the core package of the target is present first.

We have decided to not fix any explorer bugs for now as they are not used by students. Closing as Won't Fix.

Even though this is project settings, it is related to explorer


[Feature] peripheral.wrap for Immersive Engineering's Redstone Wire Connector

Describe your feature idea
Allow the Redstone Wire Connector from Immersive Engineering to be used as a peripheral, with an API to control it's input/output status and its redstone colour.
Additional context
https://ftb.gamepedia.com/Redstone_Wire_Connector_(Immersive_Engineering)
For some reason, CC:Tweaked does not properly read the colour of IE's redstone signals through it's redstone wires and connectors. (getBundledInput() does not return a colour). This would allow a computer to still send a coloured redstone signal in IE by changing the colour of the connector before sending a signal.
I've tried talking to the IE mod team, and they don't seem interested in finding out why it's not working properly. This is a bit of an indirect, but functional, fix. Not really your problem, but would be massively appreciated!

Sounds good and useful, I will implement this in the future.

Sounds good and useful, I will implement this in the future.

new release of IE already adds that feature

First 1.16.5 release
Allowed CC:Tweaked computers to emit bundled redstone signals using interface connectors (Malte)

here
can be closed

new release of IE already adds that feature

First 1.16.5 release
Allowed CC:Tweaked computers to emit bundled redstone signals using interface connectors (Malte)

here
can be closed

new release of IE already adds that feature

First 1.16.5 release
Allowed CC:Tweaked computers to emit bundled redstone signals using interface connectors (Malte)

here
can be closed

Sort of. It allows the Redstone Interface Connector to read an output from a computer, but input is still not implemented. I was playing around with this the other day to see if it was worth deleting this suggestion.
It would still be kind of cool to be able to alter the configuration of attached Redstone Connectors as peripherals.

new release of IE already adds that feature

First 1.16.5 release
Allowed CC:Tweaked computers to emit bundled redstone signals using interface connectors (Malte)

here
can be closed

Sort of. It allows the Redstone Interface Connector to read an output from a computer, but input is still not implemented. I was playing around with this the other day to see if it was worth deleting this suggestion.
It would still be kind of cool to be able to alter the configuration of attached Redstone Connectors as peripherals.

Hey, i want to add this feature in 0.5b. What functions do you want?
Something like:
getChannel()
getSignalForChannel(int color)
setChannel(int color)

Yeah, those sound good! If I can think of something else I'll comment!

Hey, I have a lil' question.
I currently work on an Immersive Engineering integration but I saw https://github.com/BluSunrize/ImmersiveEngineering/commit/17e8a953412d08a7baefa7926b2297ed6ec62bbe
Does IE already have cc integration?

They didn't when I made the request.
I think now they've added some compatibility. There's not a lot to it though.
I haven't played in a while now so that also may have changed.

Fixed in 0.6r


Update calc engine to remove existence of negative 0

The current calc engine has many cases that result in a calculation of negative 0. While there are a couple spots where this is checked and resolved (2, to be exact), this should be improved to prevent cases like "not(1-1) = -1".

I'm not sure I follow - isn't "not(1-1) = -1" the correct result?


feat(package.js): support dependency specification by logical OR

Description

Supports specifying dependencies by logical ORs; by specifying multiple ids in dependency separated by |, one of the ids will be required.  This is necessary to correctly specify the dependency packages for CacheText, InputPipePlugin and SigContrastFastAviUtl after implementing multiple LSMASHWorks.
This is a breaking change. This PR by itself will not cause any problems, but after the corresponding xml release, the current release of apm will not be able to find the packages it depends on.
Related Issue

Motivation and Context

How Has This Been Tested?

packages_list.zip
InputPipePlugin included in the attachment depends on multiple LSMASHWorks as follows. It should work the same way no matter which LSMASHWorks is installed.
	<package>
		...
		<dependencies>
			<dependency>LSMASHWorks|VFRmaniacLSMASHWorks|HolyWuLSMASHWorks|MrOjiiLSMASHWorks</dependency>
		</dependencies>
		...
	</package>

Screenshots (if appropriate):
Types of changes

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

Checklist:

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

I merged hal-shu-sato/apm-data#40 so I will release apm including this PR right soon.

I haven't submitted the corresponding xml PR yet (it is not apm-data#40), so no need to rush if you don't have time. Just in case.

Due to adding the continuous attribute, the current apm shows [Object Object] in the current version column and record the object in apm.json after installing.
It should be fixed as soon as possible.


Add an example of acronym to the "Rule 11 - JSON properties format"

Let's look if we already have a good example in the current version of the Schema...

We actually have a counter example with coordinatesWGS84: where WGS stands for "World Geodetic System". I will correct the schema renaming coordinatesWGS84: into ``coordinatesWgs84:, and then use it as an example within the papiNet-JSON-Style-Guide.md` file.

Done, see commit 5df79ca.


fix(spinner): RTL fix for Spinner

Short description of what this resolves:
This will fix bubbles and circles spinner on RTL
Changes proposed in this pull request:

as spinners are only clockwise we should not use position mixin and instead use multi-dir mixin

Ionic Version: 3.x
Fixes: #11211

any update for this?

@kensodemann It shouldn't be hard to merge this both in master and in core. Can you please see if this can be done for the next version?

LGTM, I am fine merging it if @brandyscarney is.
Question: would this be more maintainable?
@include multi-dir() {
  @include position(0, null, null, 0);
}

I wasn't completely sure if that would work once I started following the chain of mixins down.

@kensodemann but putting position mixin inside multi-dir mixin is not make any sense. let me explain the problem here.
The whole problem with bubbles and circles is that on spinner.ts we use left style (regardless of site's direction RTL or LTR) for placing the dots SVGs on the correct position, now when we use position mixin it would result in placing right: 0; in the css instead of left style on RTL so the left style on SVG element won't replace the inheritance hence they always position at right: 0; on RTL regardless of different left style on SVG element itself.
Bottom line is we have two options:

modify spinner.ts file and change that it checks for direction (RTL and LTR) before putting left style on SVG elements and use right style in case of RTL.
modify spinner.scss file and hard-codedly put left: 0; regardless of site's direction the same as .ts.
As sass modifications is always preferred, I think changing sass and putting hard-coded left value is not a total bad idea.

@kensodemann It was a bit before your time, so just to let you know, I am the one who wrote the aces Nixon’s for directional support, signed off by brandy
This looks good to me

@sijav - thank you for the explanation, that was helpful

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


adds btc_usd currency pair

const tickerStream = new TickerStream();
const tickerTopic = tickerStream.subscribe();

I see, then no need for this PR!


Get free from global symbol table on Linux as we do for macos.

Coverage remained the same at 85.273% when pulling 438ac6ee7b1a1dcbeb51966b1deb48f9fbbdf33e on no-load-c into fd68d0da1523e13c4680955fd96bf2ab404e9775 on master.

Disregarding the coverage report.


Running on the background

Excellent tool, congrats for hn # 1!
Would it be possible to opt-in always running the extension on the background, so that you don't need to worry about profiling a specific website, and just passively collect all API's you've accessed?
Might be relevant when you've lost access to a front (or if the front gets taken down) and you'd like to directly communicate with the API.

Thank you! This makes use of the chrome.devtools.network API which is only available when Chrome DevTools is open. This isn't possible with the approach this tool takes. There is a tool that could achieve that (I think) though called Akita.


Fix compatibility with Python 3.12.

Fixes: #829

Thanks for that, was just on it to fix it by myself :-)


Refactoring

Codecov Report
Base: 69.50% // Head: 69.42% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (b9583cd) compared to base (121ec46).
Patch coverage: 97.22% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #320      +/-   ##
==========================================
- Coverage   69.50%   69.42%   -0.09%
==========================================
  Files          65       65
  Lines       12157    12124      -33
  Branches     1619     1619
==========================================
- Hits         8450     8417      -33
  Misses       2110     2110
  Partials     1597     1597

Impacted Files
Coverage Δ

src/background/csa/socket.ts
22.00% <ø> (ø)

src/background/encrypt.ts
22.72% <ø> (ø)

src/background/environment.ts
40.00% <ø> (ø)

src/background/security.ts
69.35% <ø> (ø)

src/common/helpers/datetime.ts
89.47% <ø> (ø)

src/common/helpers/string.ts
36.84% <ø> (ø)

src/common/helpers/testing.ts
65.71% <ø> (ø)

src/common/helpers/time.ts
81.81% <ø> (ø)

src/common/log.ts
80.00% <ø> (ø)

src/common/settings/research.ts
66.66% <ø> (ø)

... and 56 more

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

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


Typos on Readme.md

There are some typo in the readme, in Driver definition and Step definition:
onNext: (Element) => {},                    // Called when moving to next step on any step
onPrevious: (Element) => {},                // Called when moving to next step on any step

Should be:
onNext: (Element) => {},                    // Called when moving to next step on any step
onPrevious: (Element) => {},                // Called when moving to previous step on any step

Thanks for this library, works like a charm!

@Kangcor Are you proposing a PR or I can take it? 😃

If you don't mind let me do it, I could use the experience of the process
in this easy PR. Thanks!
El jue., 21 feb. 2019 a las 18:24, Islam Attrash<EMAIL_ADDRESS>escribió:

@Kangcor https://github.com/Kangcor Are you proposing a PR or I can
take it? 😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kamranahmedse/driver.js/issues/147#issuecomment-466088468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFm9fvmiXh_o6aWjRf8RdOMYqjOHAAPTks5vPtZrgaJpZM4bHlsp
.

Created pull request PR

Thanks


Target not respected when using runtime package built for 17.2 running on 17.3

Describe the bug
I found a bug with Continia Document Capture, and they believe it is caused by a runtime package built against BC 17.2 being run on BC 17.3, as it seems not to occur on BC 17.2. Thus, I am posting here in case this is a general and unknown issue with runtime app use.
I haven't retested personally but trust Continia probably know what they're talking about!
To Reproduce
Opening the Document Capture Setup card and clicking the action Related > About brings up an Error() window with the following text:

You cannot make this call on table ID<PHONE_NUMBER> from a RecordRef data type when you are using a compilation target other than OnPrem."

The stack trace from "View the last known error" is:
"CSC Onprem Licn. Mgt."(CodeUnit 6192854).GetLicenseeInfo line 12 - Continia Core by Continia Software
"CSC Core Mgt."(CodeUnit 6192832).GetLicenseeInfo line 12 - Continia Core by Continia Software
"CSC Core"(CodeUnit 6192833).GetLicenseeInfo line 5 - Continia Core by Continia Software
"CDC License Mgt. Subscr."(CodeUnit 6085861).OnGetLicenseeInfo line 4 - Continia Document Capture 365 by Continia Software
"CDC Continia License Mgt."(CodeUnit 6192771).OnGetLicenseeInfo line 2 - Continia Document Capture 365 by Continia Software
"CDC Continia License Mgt."(CodeUnit 6192771).LicenseeInfo line 2 - Continia Document Capture 365 by Continia Software
"CDC About Document Capture"(Page 6085605).OnOpenPage(Trigger) line 2 - Continia Document Capture 365 by Continia Software

I have also seen errors in the Event Viewer, during normal usage, not reported via GUI:

Message Cloud -> OnPremise call security violation. Caller App=Continia Document Capture 365 by Continia Software <IP_ADDRESS> Func=OnRun, Callee App=Base Application by Microsoft 17.3.20469.20605 Func=OnAfterSalesHeaderDelete

Message Cloud -> OnPremise call security violation. Caller App=Continia Document Capture 365 by Continia Software <IP_ADDRESS> Func=OnRun, Callee App=Base Application by Microsoft 17.3.20469.20605 Func=SetCRMSalesOrderIdOnSalesHeaderDeletion

Message Cloud -> OnPremise call security violation. Caller App=Continia Document Capture 365 by Continia Software <IP_ADDRESS> Func=OnRun, Callee App=Base Application by Microsoft 17.3.20469.20605 Func=RemoveCouplingToCRMSalesOrderOnSalesHeaderDelete

a Continia developer said:

I have just tested in my own environment and there seems to be an issue when using runtime packages.
DC 6.50.2 has been build for BC17.2, but if you install the app on a BC 17.3 then the target is not respected and then you get the error.

Expected behavior
No error / no difference in behaviour whether the runtime package is run on 17.2 or 17.3
5. Versions:

AL Language: n/a
Business Central: GB Business Central 17.3 (Platform 17.0.20458.20517 + Application 17.3.20469.20605)

I have same problem when trying to change a Customer record using RecordRef. When I am changing the customer manual, I don't have problems.
The error is:
Cloud -> OnPremise call security violation. Caller App=XXX by ZZZ <IP_ADDRESS> Func=CreateTableRecord, Callee App=Base Application by Microsoft 17.3.20469.20605 Func=RunWorkflowOnCustomerChanged
Application 17.3.20469.20605

The event viewer issue is just noise for now and will be removed in the 17.5 release. The runtime package issue needs to be investigated. I guess it was compiled OnPrem and then it is a bug. Else it is not.

Hello again,
I think, I found my problem - service was 17.2, but the database is 17.3.
I upgraded the service to 17.3 and now I don't have this issue.

Ok then I close the issue.

Closing my issue based on a report from someone else, which I don't think applies to my situation and I haven't verified yet in any case... seems wrong to me.

I have not seen any bugs. Runtime packages have to follow the runtime. By design.


feat: allow disabling/enabling input

Proposed changes:
Export the function toggleInputDisabled for consumption and allow passing a boolean to toggle on/off the input field.
Status (please check what you already did):

[X] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

Great! Thank you

Hey @fparga, looks like a great addition and is a feature I could definitely make use of -- is there a way to configure this from the HTML code defining the widget?


WIP: Fix/stammw/clippy fixes

fixes #59

This probably needs a rebase, seems to include all the changes from #67, but with clippy fixes making it conflict.

Ho no! :man_facepalming: Didn't think of that...
There, it's rebased  now.


false positive: missing EIP association resources associated with NAT Gateways

Description
https://github.com/snyk/driftctl/blob/5f863c7d643f5eaa63399ea11e847349cb87feb4/pkg/middlewares/aws_nat_gateway_eip_assoc.go#L44
Results in missing resources being reported:
Scanned states (1)
Found missing resources:
  - eipassoc-00000000000000001 (aws_eip_association)
  - eipassoc-00000000000000002 (aws_eip_association)
  - eipassoc-00000000000000003 (aws_eip_association)
  - eipassoc-00000000000000004 (aws_eip_association)
  - eipassoc-00000000000000005 (aws_eip_association)
  - eipassoc-00000000000000006 (aws_eip_association)
  - eipassoc-00000000000000007 (aws_eip_association)
  - eipassoc-00000000000000008 (aws_eip_association)

If I stop managing the corresponding eips, then the missing eipassocs go away, and instead I get aws_eip resources reported as not managed by IaC:
  aws_eip:
    - eipalloc-00000000000000001
    - eipalloc-00000000000000002
    - eipalloc-00000000000000003
    - eipalloc-00000000000000004
    - eipalloc-00000000000000005
    - eipalloc-00000000000000006
    - eipalloc-00000000000000007
    - eipalloc-00000000000000008

If I manage both the eip and the eipassoc, I just get a more detailed missing resource report:
Scanned states (1)
Found missing resources:
  - eipassoc-00000000000000001 (module.ngw_eip_assoc)
  - eipassoc-00000000000000002 (module.ngw_eip_assoc)
  - eipassoc-00000000000000003 (module.ngw_eip_assoc)
  - eipassoc-00000000000000004 (module.ngw_eip_assoc)
  - eipassoc-00000000000000005 (module.ngw_eip_assoc)
  - eipassoc-00000000000000006 (module.ngw_eip_assoc)
  - eipassoc-00000000000000007 (module.ngw_eip_assoc)
  - eipassoc-00000000000000008 (module.ngw_eip_assoc)

Environment

OS: Any
driftctl version: Latest
terraform version: Any
terraform providers versions: Any

How to reproduce
Allocate an EIP
Create a nat gateway and associate the EIP.
Possible Solution
Don't ignore EIP associations to nat gateways.
Additional context
The issue is probably caused by aws_eip_association_expander.go expanding the association from the eip after they were already ignored.
DEBU[0012] Starting middleware middleware=middlewares.AwsNetworkACLExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsNatGatewayEipAssoc
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000001 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000002 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000003 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000004 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000005 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000006 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000007 type=aws_eip_association
DEBU[0012] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000008 type=aws_eip_association
DEBU[0012] Starting middleware middleware=middlewares.AwsBucketPolicyExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsSQSQueuePolicyExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsDefaultSQSQueuePolicy
DEBU[0012] Starting middleware middleware=middlewares.AwsSNSTopicPolicyExpander
DEBU[0012] Starting middleware middleware=*middlewares.AwsRoleManagedPolicyExpander
DEBU[0012] Starting middleware middleware=middlewares.TagsAllManager
DEBU[0012] Starting middleware middleware=middlewares.EipAssociationExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsRDSClusterInstanceExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayDeploymentExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayResourceExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayApiExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayRestApiPolicyExpander
DEBU[0012] Starting middleware middleware=middlewares.AwsConsoleApiGatewayGatewayResponse
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayDomainNamesReconciler
DEBU[0012] Starting middleware middleware=middlewares.AwsApiGatewayBasePathMappingReconciler
DEBU[0012] Starting middleware middleware=middlewares.AwsEbsEncryptionByDefaultReconciler
DEBU[0012] Starting middleware middleware=middlewares.AwsALBTransformer
DEBU[0012] Starting middleware middleware=middlewares.AwsALBListenerTransformer
DEBU[0012] Starting middleware middleware=*middlewares.GoogleIAMBindingTransformer
DEBU[0012] Starting middleware middleware=*middlewares.GoogleStorageBucketIAMPolicyTransformer
DEBU[0012] Starting middleware middleware=*middlewares.GoogleComputeInstanceGroupManagerReconciler
DEBU[0012] Starting middleware middleware=middlewares.AzurermRouteExpander
DEBU[0012] Starting middleware middleware=middlewares.AzurermSubnetExpander
DEBU[0012] Starting middleware middleware=*middlewares.AwsS3BucketPublicAccessBlockReconciler
DEBU[0012] Starting middleware middleware=middlewares.AwsDefaults

Moving the AwsNatGatewayEipAssoc middleware after EipAssociationExpander has not resolved the issue.
DEBU[0013] Starting middleware middleware=middlewares.EipAssociationExpander
DEBU[0013] Starting middleware middleware=middlewares.AwsNatGatewayEipAssoc
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-040a274c7dc13e17a type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-0010c265d3ee8241c type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-0bb6aeedc311d8667 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-09d9be0f883f70331 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-01ff746e1ecd991a3 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-0f45e35a020949658 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-05f444c1d1c6f94ae type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-0003ab6d0574c00be type=aws_eip_association

Moving the AwsNatGatewayEipAssoc middleware after EipAssociationExpander has not resolved the issue.
DEBU[0013] Starting middleware middleware=middlewares.EipAssociationExpander
DEBU[0013] Starting middleware middleware=middlewares.AwsNatGatewayEipAssoc
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000001 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000002 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000003 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000004 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000005 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000006 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000007 type=aws_eip_association
DEBU[0013] Ignoring aws_eip_association as it is associated to a nat gateway id=eipassoc-00000000000000008 type=aws_eip_association

fixed by https://github.com/snyk/driftctl/pull/1619

This issue has been referenced in the v0.39.0 release.

This issue has been referenced in the v0.39.0 release.

This issue has been referenced in the v0.39.0 release.


Automation doesn`t work

Automation doesn't work for me, what is wrong?
I created a file automations-extension.js with a raw from WebUI
Put code in the configuration.yaml
======================
automations:
  automation_by_action:
    trigger:
      platform: action
      entity: 0x70ac08fffec5cd06
      action: single
    action:
      entity: 0x7cb03eaa0a002bab
      service: toggle
======================
> 0x70ac08fffec5cd06 - Smart button Tuya ERS-10TZBVB-AA
(https://www.zigbee2mqtt.io/devices/ERS-10TZBVB-AA.html#tuya-ers-10tzbvb-aa)
> 0x7cb03eaa0a002bab - OSRAM Smart+ plug AB3257001NJ
(https://www.zigbee2mqtt.io/devices/AB3257001NJ.html#osram-ab3257001nj)
>>>>>>>>>>>>>>>>>>>>>>
LOG:
Info 2023-12-02 17:49:43MQTT publish: topic 'zigbee2mqtt/0x70ac08fffec5cd06', payload '{"action":"single","battery":100,"linkquality":147,"operation_mode":"event","voltage":3000}'
Info 2023-12-02 17:49:43MQTT publish: topic 'zigbee2mqtt/0x70ac08fffec5cd06', payload '{"action":"","battery":100,"linkquality":147,"operation_mode":"event","voltage":3000}'
Info 2023-12-02 17:49:43MQTT publish: topic 'zigbee2mqtt/0x70ac08fffec5cd06/action', payload 'single'
Info 2023-12-02 17:50:09MQTT publish: topic 'zigbee2mqtt/0x7cb03eaa0a002bab', payload '{"linkquality":141,"state":"OFF","update":{"installed_version":16909577,"latest_version":16909577,"state":"idle"},"update_available":false}'
Info 2023-12-02 17:50:09MQTT publish: topic 'zigbee2mqtt/0x7cb03eaa0a002bab', payload '{"linkquality":147,"state":"OFF","update":{"installed_version":16909577,"latest_version":16909577,"state":"idle"},"update_available":false}'
Info 2023-12-02 17:50:10MQTT publish: topic 'zigbee2mqtt/0x7cb03eaa0a002bab', payload '{"linkquality":144,"state":"ON","update":{"installed_version":16909577,"latest_version":16909577,"state":"idle"},"update_available":false}'
Info 2023-12-02 17:50:10MQTT publish: topic 'zigbee2mqtt/0x7cb03eaa0a002bab', payload '{"linkquality":149,"state":"ON","update":{"installed_version":16909577,"latest_version":16909577,"state":"idle"},"update_available":false}'
<<<<<<<<<<<<<<<<<<<<<<

Answer by myself)
Automation start working when I changed standart format of Friendly name to Abc😧
@Anonym-tsk could you add the ability to use standard device names
Thanks!

use standard device names

Please try to add quotes around device name, e.g.
entity: '0x7cb03eaa0a002bab'


changing the pw of the keystore

How does one do that?

@chrisfranko You cannot change the password to a keystore, if you want a new password you need to create a new keystore with the new password.

Thats what I figured.
On 4 May 2016 at 15:47, Christian Lundkvist<EMAIL_ADDRESS>wrote:

@chrisfranko https://github.com/chrisfranko You cannot change the
password to a keystore, if you want a new password you need to create a new
keystore with the new password.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ConsenSys/eth-lightwallet/issues/91#issuecomment-216980717


Ability to "pre-migrate" and/or replicate a container

I'm not sure, "pre-migrate" is the right word.
It would be great to be able to

Create container replicas.
In case of LXC it's easy, just zfs snap & transfer container to a remote machine and that's it.
With LXD it's different, because uid is not fixed and container configurations are not stored with the container.
NB: this way it's easy to create clones of online containers too.
Create a preliminary container (dataset) migration.
In case of LXC it's easy because either via rsync or zfs send data could be transferred to a remote machine. When it's done even after a long time container can be stopped, transfer deltas and finish migration easily.

lxc snapshot source:container1 pre-migrate
lxc copy container1/pre-migrate target:container2
Doesn't that do what you want?

Serge,
Sorry, somehow I missed emails from github.
No, I don't think that's what I want or I don't get the point.
This is the workflow I usually follow:

replicated containers (poor men's HA)
Container is time to time zfs replicated to/from a remote machine.
If the source is off for any reason, the container on destination can be easily started.

replicated containers (dev instances)
Container is time to time zfs replicated to/from a remote machine.
On the remote machine container is started, used as an exact copy of the production environment. Development changes can be implemented, tests are running fine on production data. On next time of replication changes are dropped on the development instance and delta is copied from the production one.

preliminary container migration
Container is zfs replicated to/from a remote machine. The original system is still running, the target not yet.
When it's done the the replication process can be repeated number of times and if it's needed for some reason the last step can be done much later (basically the same as ##1).

Backup (bonus)
Container get's snapshotted and replicated via a backup tool (script).
It's must be restorable, there is a database of container settings, not just the dataset itself.

AFAIK a copy of a container is independent of the source, it's not like zfs snapshots and send/recv.
In case of zfs if there a common snapshot, the replication 'stream' works.
Let me know, if I'm not clear.

@stgraber if the above makes sense, please reopen.

@stgraber
Thanks so much, much appreciated!

@stgraber
Do you want me to separate this to smaller issues?
Specifically ##4 looks a bit different.

We can keep it in one place for now and we'll split it once someone has time to work on it.

So looking through this issue, I think we actually have all that these days through support for --refresh on lxc copy.
So a container can be copied from a system to another, then changes further changes to the source can be copied without requiring a full copy. This effectively resets the destination to once again match the source.
We also have the backup API which allows for downloading tarballs of containers (including ZFS optimized blobs) and storing/restoring them which should fit the needs of the last point.


LSOF url is broken

The current URL on nixpkgs/pkgs/development/tools/misc/lsof/default.nix is broken.  It seems the the ualberta.ca had some url updates.
Old link: ftp://sunsite.ualberta.ca/pub/Mirror/lsof/lsof_4.88.tar.bz2
I've got it working with a HTTP mirror at
http://mirrorservice.org/sites/lsof.itap.purdue.edu/pub/tools/unix/lsof/OLD/lsof_4.88.tar.bz2

The point is that the mirrors are both good, but the tarball changes location because they keep the newest one on a different path than the others (add OLD/). Now we try both locations.


[Python Typechecking] CI workflow pipenv run mypy src\ci_workflow tests\tests_ci_workflow

See full instructions at https://github.com/opensearch-project/opensearch-build/issues/1237
Acceptance criteria:

Run pipenv run mypy src/ci_workflow tests/tests_ci_workflow passes without error

Keeping the issue open for remaining fixes - https://github.com/opensearch-project/opensearch-build/pull/1582#pullrequestreview-870834996

Closing this issue after additional fixes.


Expected passes/fails for advanced testing

I ran the following commands under the section, Advanced testing, of the README.
source $ENVROOT/atomsci/bin/activate # activate your pip environment. `atomsci` is an example here.
cd atomsci/ddm/test
pytest

It's not clear to me what is expected to pass or fail. This is what I got when running the pytest using the Docker container.
The pytest (in /AMPL/atomsci/ddm/test) resulted in:======================= 51 failed, 112 passed, 266 warnings in 704.19s (0:11:44) =======================

Is this to be expected? It would be helpful to have more documentation in this section on expected behavior, dependencies required to set up the pytest for more passes, and why one would need to do advanced testing.
@estahlberg
@SeanTBlack

Most of the errors are FileNotFoundError because most of the data files were defined using relative paths. For example:
FAILED integrative/ad_index/test_ad_index.py::test - FileNotFoundError: [Errno 2] No such file or directory: 'H1_RF.json'
FAILED integrative/balancing_trans/test_balancing_transformer.py::test_balancing_transformer - FileNotFoundError: [Errno 2] ...
FAILED integrative/compare_models/test_filesystem_perf_results.py::test_RF_results - FileNotFoundError: [Errno 2] No such fi...
FAILED integrative/compare_models/test_filesystem_perf_results.py::test_NN_results - FileNotFoundError: [Errno 2] No such fi...
FAILED integrative/compare_models/test_filesystem_perf_results.py::test_XGB_results - FileNotFoundError: [Errno 2] No such f...

Will update the tests and provide more info with the different tests. Currently, there's a README  that provides basic info on the tests that require special resources.

Per Naomi, this can be closed.


[[FIX]] Consistently lint evaluated code

Previously, upon encountering certain forms of dynamic code evaluation
(namely the browser's setTimeout and setInterval functions), JSHint
would attempt to lint the code being evaluated. Due to what appears to
to be an unintentional typo on the JSHint source code, this behavior did
not extend to code evaluated via the Function and eval functions.
Extend this behavior to all forms of evaluated code.

@rwaldron: This is about 3 bugs separated from a more important issue, but I
want to tackle each part on its own. What do you think about this? It's
backwards-breaking, but the current behavior is clearly inconsistent and clearly
the result of a typo.

Coverage increased (+0.03%) to 97.751% when pulling 35671ad22d7d28d44f564e575d37e27609d943ad on jugglinmike:lint-eval-code into bec152c81aa1a1b5ff21f56589363619494a7d24 on jshint:master.

Rick and I have discussed this offline, and we've decided that unfortunately,
we cannot include this fix outside of a major release of JSHint.
While it is a bug fix, and Semantic Versioning supports fixing bugs (even those
that are backwards-incompatible) with patch release, the important detail here
is that the underlying behavior is not documented. As far as I can see, there
is no wording in JSHint's documentation which explains how JSHint is expected
to lint any dynamically-modified code it encounters. Because of that, enabling
it in any new context (even though doing so would be more logical/consistent
overall) is effectively introducing a new feature.
In light of that, I'm going to label this fix for inclusion in the next major
release.


Crash when trying to render/open GUI of a Grid

Issue description:
game crashes when trying to render the GUI of a Grid, after looking on the Issues one person has had the same problem but he had to much stuff in his network, problem one that has over 1 Million iron (i was bourd one time) works and opens, the one that has 200-300 items in it crashes the game
i have also noticed that it can happen when you load up your game and in the world for about 10-15 seconds then it crashes but it produces the same crash log
What happens:
opening the grid sometimes causes a crash
What you expected to happen:
the GUI to open and allow access to my storage
Steps to reproduce:
it's realy hard to reproduce and when i tryed some times it crashed, other times it didn't
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>3
Refined Storage: 1.2-Beta3

Does this issue occur on a server? [yes/no]
unknown
If a (crash)log is relevant for this issue, link it here:
crash-2016-10-27_13.15.03-client.txt
[pastebin/gist/etc link here]

slimeknights.tconstruct.library.client.RenderUtil It seems to be something related to TiC
Name: tconstruct.tank // slimeknights.tconstruct.smeltery.tileentity.TileTank
This might be on our end tho. Cause of the java.lang.IllegalStateException: Already building!

Not an RS issue at all, RS uses normal item rendering functions MC provides. Report this to TConstruct.


checkbox background is missing, and its not showing

Hi,
if I add checkbox mouse cursor recognize it, but its hidden and can't be checked. I found that in admin-lte.css on row 2862 is background image that is missing (icheck/miniminal/minimal.png).
Is there way to show checkbox, or I just have to set my own image in my custom css?
thx for help

:+1:

Sorry for too late reply!
PR https://github.com/shine60vn/adminlte-rails/pull/4 fixed this issue already, we can close this now!


[ocaml5-issue] Out_channel.flush can cause Syserror when used in parallel

A Cygwin 5.2 run triggered when merging #443 to main found an unexpected counterexample to STM Out_channel parallel:
https://github.com/ocaml-multicore/multicoretests/actions/runs/8361686850/job/22890359851
random seed: 292949458
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test sequential (generating)
[✓] 1000    0    0 1000 / 1000     2.9s STM Out_channel test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test parallel
[ ]  620    0    0  542 / 1000    57.1s STM Out_channel test parallel
[ ] 1116    0    0  966 / 1000   117.5s STM Out_channel test parallel (shrinking:    1.0011)
[✗] 1117    0    1  966 / 1000   129.8s STM Out_channel test parallel

--- Failure --------------------------------------------------------------------

Test STM Out_channel test parallel failed (1 shrink steps):

                                                              |
                                     Output_substring ("\225\185!\188Q\142\138", 0, 96)
                                                       Output_byte 36
                                     Output_bytes "\158\199e\220\253\148\... (truncated)
                                                            Flush
                                                            Close
                                                   Output ("Z\197j", 3, 3)
                                                          Open_text
                                                         Is_buffered
                                                    Set_binary_mode false
                                              Output_substring ("{8)g", 18, 1)
                                                           Length
                                                         Close_noerr
                                                          Open_text
                                                      Set_buffered true
                                                              |
                                   .----------------------------------------------------.
                                   |                                                    |
                              Close_noerr                                     Set_binary_mode true
                               Open_text                                               Pos
                Output_substring ("\to\128N\186", 7, 6)                       Set_binary_mode true
                            Output_char '9'                                      Output_byte 54
                         Set_binary_mode true                                        Length
                         Output_bytes "\182U"                                         Flush
                            Output_char '5'                    Output_substring ("\014\236\187\146^... (truncated)
              Output ("(|$\217\203\206\162t\235", 18, 4)                      Set_binary_mode true
                                  Pos                                              Is_buffered
                         Set_binary_mode false                                    Output_byte 1

+++ Messages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Messages for test STM Out_channel test parallel:

  Results incompatible with linearized model

                                                                                |
                                                       Output_substring ("\225\185!\188Q\142\138", 0, 96) : Error (Invalid_argument("output_substring"))
                                                                                Output_byte 36 : Ok (())
                                                                         Output_bytes "\158\199e\220\253\148\... (truncated) : Ok (())
                                                                                Flush : Ok (())
                                                                                Close : Ok (())
                                                                         Output ("Z\197j", 3, 3) : Error (Invalid_argument("output"))
                                                                                Open_text : Ok (())
                                                                                Is_buffered : Ok (true)
                                                                                Set_binary_mode false : Ok (())
                                                                Output_substring ("{8)g", 18, 1) : Error (Invalid_argument("output_substring"))
                                                                                Length : Ok (0)
                                                                                Close_noerr : Ok (())
                                                                                Open_text : Ok (())
                                                                                Set_buffered true : Ok (())
                                                                                |
                                                     .--------------------------------------------------------------------------------------------------.
                                                     |                                                                                |
                                           Close_noerr : Ok (())                                                                         Set_binary_mode true : Ok (())
                                            Open_text : Ok (())                                                                                Pos : Ok (0)
          Output_substring ("\to\128N\186", 7, 6) : Error (Invalid_argument("output_substring"))                                         Set_binary_mode true : Ok (())
                                         Output_char '9' : Ok (())                                                                          Output_byte 54 : Ok (())
                                      Set_binary_mode true : Ok (())                                                                             Length : Ok (0)
                                      Output_bytes "\182U" : Ok (())                                                            Flush : Error (Sys_error("Bad file descriptor"))
                                         Output_char '5' : Ok (())                                                        Output_substring ("\014\236\187\146^... (truncated) : Ok (())
              Output ("(|$\217\203\206\162t\235", 18, 4) : Error (Invalid_argument("output"))                            Set_binary_mode true : Error (Sys_error("Bad file descriptor"))
                                               Pos : Ok (4)                                                                                Is_buffered : Ok (true)
                                      Set_binary_mode false : Ok (())                                                                        Output_byte 1 : Ok (())

================================================================================
failure (1 tests failed, 0 tests errored, ran 2 tests)
File "src/io/dune", line 40, characters 7-16:
40 |  (name stm_tests)
            ^^^^^^^^^
(cd _build/default/src/io && ./stm_tests.exe --verbose)
Command exited with code 1.

AFAICS, the failure can be explained by Flush in the "right leg" causing Sys_error("Bad file descriptor").
This goes against the specification, which says:
       val close : t -> unit

       Close the given channel, flushing all buffered write operations.  Output functions raise a Sys_error exception when they are  applied  to  a  closed  output
       channel, except Out_channel.close and Out_channel.flush , which do nothing when applied to an already closed channel. [...]

This is currently captured in the STM Out_channel test as:
    | Flush, Res ((Result (Unit,Exn),_), r) ->
      (match s,r with
       | Closed, Error (Sys_error _) -> false (* should not generate an error *)
       | Closed, Ok () -> true
       | Open _, Ok () -> true
       | _ -> false)

Just observed this again - now on FreeBSD 5.1 trying out the latest QCheck memory usage improvement
https://github.com/ocaml-multicore/multicoretests/tree/try-qcheck-mem-improvement
https://ocaml-multicoretests.ci.dev:8100/job/2024-06-28/125000-ci-ocluster-build-caa2d4
random seed: 362318939
generated error fail pass / total     time test name

[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test sequential
[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test sequential (generating)
[✓] 1000    0    0 1000 / 1000     1.2s STM Out_channel test sequential

[ ]    0    0    0    0 / 1000     0.0s STM Out_channel test parallel
[✗]  103    0    1   90 / 1000     4.7s STM Out_channel test parallel

--- Failure --------------------------------------------------------------------

Test STM Out_channel test parallel failed (1 shrink steps):

                                                              |
                                       Output ("\011\230\197\204\173\164'\024", 9, 1)
                                     Output_bytes "\159\238+\152]\006*\24... (truncated)
                                                         Close_noerr
                                                          Open_text
                                                            Flush
                                                            Flush
                                                              |
                                   .----------------------------------------------------.
                                   |                                                    |
                                Length                                             Close_noerr
                            Output_char 'i'                    Output ("Dc\015\194*x\159YlA\b\247\1... (truncated)
                                Length                                              Open_text
                             Output_byte 8                                       Output_char 'G'
                                 Flush                                        Set_binary_mode false

+++ Messages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Messages for test STM Out_channel test parallel:

  Results incompatible with linearized model

                                                                                |
                                                          Output ("\011\230\197\204\173\164'\024", 9, 1) : Error (Invalid_argument("output"))
                                                                     Output_bytes "\159\238+\152]\006*\24... (truncated) : Ok (())
                                                                                Close_noerr : Ok (())
                                                                                Open_text : Ok (())
                                                                                Flush : Ok (())
                                                                                Flush : Ok (())
                                                                                |
                                                   .-----------------------------------------------------------------------------------------------.
                                                   |                                                                                |
                                            Length : Ok (0)                                                                              Close_noerr : Ok (())
                                       Output_char 'i' : Ok (())                                     Output ("Dc\015\194*x\159YlA\b\247\1... (truncated) : Error (Sys_error("Bad file descriptor"))
                                            Length : Ok (0)                                                                               Open_text : Ok (())
                                        Output_byte 8 : Ok (())                                                                        Output_char 'G' : Ok (())
                            Flush : Error (Sys_error("Bad file descriptor"))                                                        Set_binary_mode false : Ok (())

================================================================================
failure (1 tests failed, 0 tests errored, ran 2 tests)
File "src/io/dune", line 40, characters 7-16:
40 |  (name stm_tests)
            ^^^^^^^^^
(cd _build/default/src/io && ./stm_tests.exe --verbose)
Command exited with code 1.

I've created a standalone reproducer for this one:
let test () =
  let path = Filename.temp_file "stm-" "" in
  let channel = Atomic.make (Out_channel.open_text path) in

  (* First, a bit of one-domain channel activity *)
  Out_channel.output_byte (Atomic.get channel) 1;
  (try Out_channel.flush (Atomic.get channel) with (Sys_error _) -> assert false);

  let wait = Atomic.make true in

  (* Domain 1 closes-opens-outputs repeatedly *)
  let d1 = Domain.spawn (fun () ->
      while Atomic.get wait do Domain.cpu_relax() done;
      for _ = 1 to 50 do
        Out_channel.close (Atomic.get channel);
        Atomic.set channel (Out_channel.open_text path);
        Out_channel.output_byte (Atomic.get channel) 1;
      done;
    ) in

  (* Domain 2 calls flush and pos repeatedly *)
  let d2 = Domain.spawn (fun () ->
      Atomic.set wait false;
      (*
       "Output functions raise a Sys_error exception when they are applied to a  closed  output  channel,
        except  Out_channel.close  and Out_channel.flush , which do nothing when applied to an already closed channel."
      *)
      for _ = 1 to 50 do
        (try Out_channel.flush (Atomic.get channel)
         with (Sys_error msg) -> Printf.printf "Out_channel.flush raised Sys_error %S\n%!" msg; assert false);
        ignore (Out_channel.pos (Atomic.get channel));
      done;
    ) in

  let () = Domain.join d1 in
  let () = Domain.join d2 in
  (* Please leave the torture chamber nice and clean as you found it *)
  (try Out_channel.close (Atomic.get channel) with Sys_error _ -> ());
  Sys.remove path

let _ =
  for i = 1 to 5_000 do
    if i mod 250 = 0 then Printf.printf "#%!";
    test ()
  done

With it, locally on Linux I am able to trigger the assertion failure on 5.1.0+fp and 5.2.0+fp switches.

The branch https://github.com/ocaml-multicore/multicoretests/tree/flushtest-repro-focus triggers a focused test of it on the CI.
Across all runs, this reproducer only triggered in multicoretests-ci where flush may trigger a Sys_error in

linux-arm64-5.1 https://ocaml-multicoretests.ci.dev:8100/job/2024-08-09/141442-ci-ocluster-build-d1fac4
linux-s390x-5.1 https://ocaml-multicoretests.ci.dev:8100/job/2024-08-09/141442-ci-ocluster-build-96b82e

In trying to understand this better, I ran dune exec src/io/stm_tests.exe -- -v locally under a 5.2.0+fp switch, where it fails spectacularly - even the sequential one:
$ dune exec src/io/stm_tests.exe -- -v
random seed: 242921126
generated error fail pass / total     time test name
[✗]   20    0    1   19 / 1000     0.1s STM Out_channel test sequential
[✗]    8    0    1    7 / 1000     5.9s STM Out_channel test parallel

--- Failure --------------------------------------------------------------------

Test STM Out_channel test sequential failed (19 shrink steps):

   Close_noerr
   Output_char 'H'
   Output_char ']'

+++ Messages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Messages for test STM Out_channel test sequential:

  Results incompatible with model

   Close_noerr : Ok (())
   Output_char 'H' : Error (Sys_error("Bad file descriptor"))
   Output_char ']' : Ok (())

--- Failure --------------------------------------------------------------------

Test STM Out_channel test parallel failed (45 shrink steps):

                            |
                          Close
                      Output_byte 4
                     Output_char 'E'
                            |
                 .---------------------.
                 |                     |

+++ Messages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Messages for test STM Out_channel test parallel:

  Results incompatible with linearized model

                                                             |
                                                      Close : Ok (())
                                  Output_byte 4 : Error (Sys_error("Bad file descriptor"))
                                                 Output_char 'E' : Ok (())
                                                             |
                                .---------------------------------------------------------.
                                |                                                         |

================================================================================
failure (2 tests failed, 0 tests errored, ran 2 tests)

The same test however completes under the following two which seems to indicate a fp-related bug that may have since been fixed on trunk:

5.2.0 (no fp)
5.3.0+fp

A standalone reproducer:
let test () =
  let path = Filename.temp_file "stm-" "" in
  let channel = Out_channel.open_text path in

  Out_channel.close_noerr channel;
  (try Out_channel.output_char channel 'A'; assert false with (Sys_error _) -> ()); (* Error (Sys_error("Bad file descriptor")) *)
  (try Out_channel.output_char channel 'B'; assert false with (Sys_error _) -> ()); (* Succeeds *)

  Sys.remove path

let _ =
  for i = 1 to 10_000 do
    if i mod 250 = 0 then Printf.printf "#%!";
    test ()
  done

To summarize

the Cygwin 5.2.0 failure was actually in 5.2.0~alpha1. It hasn't shown up since #12678 was merged (into 5.2.0~beta1)
the FreeBSD 5.1 failure was in 5.1.1 - not including #12678
linux-arm64-5.1

I ran into this too when running the test suite with Jane Street's branch of the OCaml compiler. My explanation seems to apply equally to the most recent head of ocaml/ocaml, so I thought I'd share it here.
Explanation: the STM model expects that Out_channel.flush will never raise if interleaved with Out_channel.close. However, Out_channel.flush can raise if you get an unlucky concurrent interleaving with Out_channel.close. In particular:

Out_channel.flush is called before the close happens, and this guard is satisfied (given the channel is still open): https://github.com/ocaml/ocaml/blob/fb5e30cb40603c75b5d19987fcd7911cc003d077/runtime/io.c#L822-L823
check_pending in caml_flush_partial relinquishes Out_channel.flush’s lock on the channel: https://github.com/ocaml/ocaml/blob/fb5e30cb40603c75b5d19987fcd7911cc003d077/runtime/io.c#L118-L132
Out_channel.close is called and completes, setting channel->buffer + 1 = channel->curr (https://github.com/ocaml/ocaml/blob/fb5e30cb40603c75b5d19987fcd7911cc003d077/runtime/io.c#L721-L730)
The execution of flush continues. caml_flush_partial is called, and this check indicates that there is exactly 1 byte to write (given the previous bullet point).
The write fails, and so this code raises Sys_error

If my explanation is correct, then maybe caml_flush_partial should check if the channel is closed before attempting to write to it.

(The issue is probably more widespread than suggested by my last message: I see that caml_flush is called by lots of functions, not just caml_ml_flush. e.g. outputting a string. All of these operations can thus relinquish the channel lock midoperation.)

Thanks for sharing! :pray:
Your explanation makes sense to me. I can't reproduce the error ATM though... I'll need to look more into it.


Increment versions for healthdataaiservices releases

Increment package versions for healthdataaiservices releases

API change check
API changes are not detected in this pull request.


OpenShift pipeline for building code when a pull-request is opened

Is your task related to a problem? Please describe.

Describe the solution you'd like

Add e2e test script for https://github.com/redhat-developer/kam/tree/master/docs/journey/day1#your-first-ci-run
Describe alternatives you've considered

Additional context

/area testing