Hoisting String/Substring extensions to StringProtocol

@swift-ci Please smoke benchmark

Build comment file:
Build failed before running benchmark.

@swift-ci Please benchmark

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - ba0352cfafbbcf799b3ccf6ec991c416203449e1
Test requested by - @dabrahams

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - ba0352cfafbbcf799b3ccf6ec991c416203449e1
Test requested by - @dabrahams

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 2b648959035e6fa1ca8ad5b73391b890d83fefa4
Test requested by - @dabrahams

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 2b648959035e6fa1ca8ad5b73391b890d83fefa4
Test requested by - @dabrahams

Build comment file:
Build failed before running benchmark.

@swift-ci Please benchmark

@swift-ci Please smoke test

@swift-ci Please benchmark

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 2b648959035e6fa1ca8ad5b73391b890d83fefa4
Test requested by - @dabrahams

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 2b648959035e6fa1ca8ad5b73391b890d83fefa4
Test requested by - @dabrahams

@swift-ci Please test

@swift-ci Please test

@swift-ci Please benchmark

@swift-ci Please benchmark

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit -<PHONE_NUMBER>c0169d62fb46ad66caadd7f627c9b4
Test requested by - @dabrahams

@swift-ci Please test

@swift-ci Please benchmark

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit -<PHONE_NUMBER>c0169d62fb46ad66caadd7f627c9b4
Test requested by - @dabrahams

@swift-ci Please test

@swift-ci Please benchmark

Build comment file:
Build failed before running benchmark.

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 71812b94c380eb8c394ed229629f2058c62f20a0
Test requested by - @dabrahams

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 71812b94c380eb8c394ed229629f2058c62f20a0
Test requested by - @dabrahams

Build comment file:
Build failed before running benchmark.

@swift-ci Please clean test

@swift-ci Please clean test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 71812b94c380eb8c394ed229629f2058c62f20a0
Test requested by - @dabrahams


Create github_secrets.py

its missing so i added this

It's missing because it contains the GitHub API key which cannot be shared. You will have to create the github_secrets.py file in your local project directory.


[TEST] Test auto hide

Summary

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Fixes #

@miina can this be closed?


fix code example in readme

Hello,
there is a small fix for code example in README.md
Please, look here and here.
Have a nice one :)

Thank you, good catch


VCD: Fix compute and sizing policies

What this PR does / why we need it:
This PR fixes propagation of compute and sizing policies for VMware Cloud Director.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind bug
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Fix assignment of compute and sizing policies for VMware Cloud Director

Documentation:

NONE

/cherry-pick release/v1.54

/cherry-pick release/v1.56

/cherry-pick release/v1.57


ItemBin's list state changed won't trigger GridView update.

Hi there!
When this plugin migrate with images choosing, it seems that then List<ItemBin> updated, the GridView didn't update, which cause _initItemPositions won't triggered. And drag item would failed because positions didn't update.
Hope this can be solve~
作者你好~
这个插件在图片选择应用中，当ItemBin的List被更新时，GridView没有被更新，导致_initItemPositions没有被触发，List的索引没有更新到数据中，此时拖动会造成数组越界（因为原数组为空）。
希望这个问题可以解决~

Try using the new version

Try using the new version

Thanks, I fix this by adding itemBinUpdated to controller, but I'll try the new version to see if the problem has been solved.
Code below:
class EditSwitchController{
  DragAbleGridViewState dragAbleGridViewState;

  void editStateChanged(){
    dragAbleGridViewState.changeDeleteIconState();
  }

  void itemBinUpdated () {
    dragAbleGridViewState._initItemPositions();
  }
}

Verified. Thanks a lot 😄

New version has better performance


Unbreak all features test

Fixes #1987

I've tested this locally on rav1e 0.2.0 and src/api/context.rs tests still segfaults when tracing is enabled.

I would land this and document that using all-features in general is discouraged.

I landed the useful commit on the other PR, this one can be closed pending upstream decision regarding the right cfg tag.

Agreed, we have some features that don't necessarily play nicely together. Many of our features are specifically for debugging. If you want a fully featured binary, the default feature set gives you that.

Thank you I will listen to that advice.


function getContract() not loading contract

Describe the bug
getContract() seems to not be able to load the deployed contract, showing the following error:
Error: No Contract deployed with name EventLog
      at Object.getContract (node_modules\@nomiclabs\hardhat-ethers\src\internal\helpers.ts:447:11)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at Context.<anonymous> (test\unit\EventLog.test.js:12:20)

To Reproduce
Steps to reproduce the behavior:

Deploy a contract, in my case I deployed with:

const { networkConfig, developmentChains } = require("../helper-hardhat-config")
const { network, ethers } = require("hardhat")
const { verify } = require("../utils/verify")

module.exports = async (hre) => {
    const { getNamedAccounts, deployments } = hre
    const { deploy, log, get } = deployments
    const { deployer } = await getNamedAccounts()

    const eventLog = await deploy("EventLog", {
        contract: "contracts/EventLog.sol:EventLog",
        from: deployer,
        log: true,
    })
}
module.exports.tags = ["all", "eventLog"]

Test the contract using getContract() --> run "yarn hardhat test"

The test file:
const { inputToConfig } = require("@ethereum-waffle/compiler")
const { assert } = require("chai")
const { deployments, ethers, getNamedAccounts } = require("hardhat")

describe("EventLog", async function () {
    let eventLog, deployer
    beforeEach(async function () {
        deployer = (await getNamedAccounts()).deployer
        deployments.fixture(["all"])
        eventLog = await ethers.getContract("EventLog", deployer)
    })

    describe("constructor", async function () {
        it("sets the number of events correctly", async function () {
            const numberOfEvents = await log.getNumberOfEvents()
            assert.equal(numberOfEvents, 0)
        })
    })
})

Expected behavior
Retrieve the contract EventLog by using getContract()
versions
"@nomiclabs/hardhat-ethers": "npm:hardhat-deploy-ethers",
"@nomiclabs/hardhat-etherscan": "^3.0.0",
"@nomiclabs/hardhat-waffle": "^2.0.0",
"chai": "^4.2.0",
"ethereum-waffle": "^3.0.0",
"ethers": "^5.6.8",
"hardhat": "^2.9.7",
"hardhat-deploy": "^0.11.10",
Additional context
The contract was deployed and showed up as a log on the hardhat node terminal:
deploying "EventLog" (tx: 0x354af893172d75a1289de2b61f271e55902f65757e99efc56dd183f15cb62e40)...: deployed at 0xe7f1725E7734CE288F8367e1Bb143E90bb3F0512 with 2558179 gas

looks like you are missing an await for deployments.fixture(["all"])

Right, solved!


Installation instructions are not clear

Is the image downloaded with a docker command?
if so which one?
docker run --rm -it --ulimit stack=100000000:100000000 trailofbits/manticore bash
?

Hi,
This command is from the manticore README. Docker downloads the image if it cannot find it in its context.
This is NOT the image of my container. You can build it with the build.sh script.


Fix email validation

mail.ParseAddress return an reformated email if the format is not really an email (with a space after of before for example).
A check was added to compare the input and the output from mail.ParseAddress.

Hello @johandorland, do you think you could review and merge this PR? Thank you!


Update BurpSuite to 2024.7.6 and add missing dependency for built-in browser

Description of changes

Fixed an issue where Burp Suite's integrated browser failed to start due to a missing libgbm dependency by adding mesa, see https://github.com/NixOS/nixpkgs/issues/338013.
Updated the version to 2024.7.6.

Note: The Burp menu's browser health check successfully identified the missing dependency.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@cryptoluks could you amend the commit message to eg.
burpsuite: 2024.6.6 -> 2024.7.6, add missing dependency for built-in browser?


Update testobject capabilities

testobject-appium-java-api also had a TestObjectCapabilities class that was out of sync with this one. I've deleted the one in testobject-appium-java-api so that we don't duplicate any code and added missing capabilities to this class.

👍


Memory size is greater than image sizes? how to free memory on navigating back?

I am displaying 300+ items in a listview and each having image defined as below. Images are retrieved from aws s3 bucket. I profiled memory usage with xamarin profiler and I can see that the more I scroll in the list, memory usage increases which makes sense as each images are cached. But I am going above 300 mb cache. total size of all images physically are not more than 15 mb.

How can it be cached 20 times more than actual size? If I use loading.png for loadingplaceholder, is it also cached for 300 times for each item in the list?

When i navigate back from the list, my memory size is not reduced. this is probably expected as some people may want to keep them in memory. Is there an option free the memory in build? a parameter or something?

<ffimageloading:CachedImage  RetryCount="1"
DownsampleToViewSize="true"
 TransparencyEnabled = "false"  CacheDuration = "30"
LoadingPlaceholder="loading_dark.png"    ErrorPlaceholder="error.png"
Source="{Binding Image, Converter={StaticResource thumbnailConvertor}}"
  Aspect="AspectFit"    />

In my program, approximately 10,000 images are displayed in ListView or FlowListView, after viewing 500pic - 207mb in the Xamarin profiler.
Are you reading
https://github.com/luberda-molinet/FFImageLoading/wiki/Xamarin.Forms-Advanced#usage-in-listview-with-listviewcachingstrategyrecycleelement-enabled
?
https://github.com/luberda-molinet/FFImageLoading/wiki/Xamarin.Forms-API#clear-cache
Clears the cache for a specified entry (Memory, Disk, All). Could be an url or filename / file path. If removeSimilar is set to true then it also removes all image cache variants
Using an ImageSource
CachedImage.InvalidateCache(ImageSource source, Cache.CacheType cacheType, bool removeSimilar = false)

@3-50

after viewing 500pic - 207mb in the Xamarin profiler. that is still a lot isnt it unless your avg. size of your images are 500 kb. I am new in xamarin profiler as well. I check the image attached and there i see bytes and I assume those are images downloaded since  I dont have anything else as byte. Do you know if I can dig into it more specifically ? I just changed to cachetype as disk and performance is still good but when I am still getting high memory consumption. I guess even I use disk, it is sill putting into memory (XF behavior probably)

Just for your info, images loaded into memory may take much more space than on disk (since they aren't compressed anymore). However if they are downscaled, they may take less memory than on disk, it really depends on what are your images and what you do with them.

@molinch I can understand this but this is quite a lot. much much greater than original total size. you can see around 337 mb is huge.

@EmilAlipiev
You can control memory cache size by yourself :) Just use custom FFImageLoading config:
var config = new FFImageLoading.Config.Configuration()
{
    MaxMemoryCacheSize = 9999999,
};
ImageService.Instance.Initialize(config);

As @molinch said:
downloaded file size != bitmap size in memory
1px = 4 Bytes (Android ARGB_8888)

1024px x 1024px = 4 Megabytes

2048px x 2048px = 16 Megabytes


Publish 1.2.1 to bower

Please, last version to bower it is 1.0.0.

It should already be published as it's based on the GitHub tags automatically.

I had to put exactly the 1.2.1 version able to download the exact bower
bower install ngInfiniteScroll#1.2.1

+1


foreach index is string

I'm just wondering why index is string in forEach loop????????????????????????????????????????????????

@ahmedashrafdevv
Please ask support question on stackoverflow or flutter's reddit help page. If you have specific issue against forEach, you'll need to open it in dedicated repo: https://github.com/dart-lang/language/issues
closing from here


Obscure configuration error

My first experience with this gem was:
E, [2021-03-18T04:14:42.145080 #24298] ERROR -- : OpenTelemetry error: unexpected configuration error - OpenTelemetry::SDK::ConfigurationError

There was an error in my configuration, which was roughly copied from an example in an issue within this repository:
OpenTelemetry::SDK.configure do |config|
  config.service_name = "..."
  config.add_span_processor(
    OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(
      exporter: OpenTelemetry::Exporter::OTLP::Exporter.new(
        endpoint: "ingest.lightstep.com:43/api/v2/otel/trace",
        insecure: false.
        headers: { "lightstep-access-token" => ENV.fetch("LIGHTSTEP_TOKEN") }
      )
    )
  )
end

It was revealed when I wrapped it to output more information about the exception:
OpenTelemetry::SDK.configure do |config|
  begin
    config.service_name = "..."
    config.add_span_processor(
      OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(
        exporter: OpenTelemetry::Exporter::OTLP::Exporter.new(
          endpoint: "ingest.lightstep.com:43/api/v2/otel/trace",
          insecure: false.
          headers: { "lightstep-access-token" => ENV.fetch("LIGHTSTEP_TOKEN") }

      )
    )
  rescue
    puts "#{$!.class.name}: #{$!}", *$!.backtrace
    raise
  end
end

(it was unknown keyword: :insecure)
Perhaps whatever is logging this error should provide more context?

The problem is that we're rescuing any errors during configuration and instead raising a ConfigurationError (to allow users to supply their own error handling, and to distinguish config errors from errors that occur elsewhere), but in doing so, we discard the original error. It is available from e.cause, but we should probably just report information from it directly.
Thanks for raising the issue - I'll fix this today.

Thanks fbogsany, I see the adjustments! 🥳
It's not quite what I had in mind, though. The new implementation still avoids displaying a stack trace, or the causing error class.
Instead I'd propose appending the exception's #full_message in opentelemetry's default error_handler which would reveal the full exception's details including class, message, backtrace, and the chain of causal exceptions' details as well:

The default error_handler is intended to be called in a wide variety of contexts. We need to balance verbosity with clarity, and we want to avoid spewing tons of stack traces into logs if someone uses the APIs incorrectly. It may be useful to provide a verbose error_handler as an option in the configuration or in opentelemetry-common, but I don't see us making that the default.


removing conversion-gen=false for BuildVolumeSource.Type field

Need to remove the +k8s:conversion-gen=false so that openshift/openshift-apiserver can auto-generate the conversions.
Tested this in the openshift/openshift-apiserver repo locally and it worked.

/assign @adambkaplan @bparees

/lgtm


some problem of demo_cpm_body.py!!

How to use the demo_cpm_body.py(or demo_cpm_hand.py) and get a better result?
However   I get a bad result as follows:
body:

hand:

body:

body:

look for help!!!thx

original hand pic:

in demo_cpm_hand.py
this code 'test_img = cpm_utils.read_image(FLAGS.DEMO_TYPE, [], FLAGS.input_size, 'IMAGE')' will resize and crop the image, put your hand in the center of the image will get a better  performance.


Fix documentation of localization related editor settings

interface/editor/localize_settings:

It's not dedicated to the Editor Settings dialog. It's used by most EditorInspector-based editor UI. Mainly Project Settings and Editor Settings, but also used in places like the advanced import settings.
It's not disabled by default.

interface/inspector/default_property_name_style

Raw is snake_case only, not related to GDScript vs C#.
Capitalized is not simply String.capitalize. Extra care is taken, like capitalization of macOS and special handling of articles and conjunctions.

Also did a bit rewording. Previously, it sounds like localize_settings and default_property_name_style are a base setting and an override. But they control different parts of the editor although both related to EditorInspector (the control).

Thanks!


[hotfix] Fix wrong Java doc comment of BroadcastStateBootstrapFunction.Context

What is the purpose of the change
This pull request fixed wrong Java doc comment of BroadcastStateBootstrapFunction.Context
Brief change log

Fix wrong Java doc comment of BroadcastStateBootstrapFunction.Context

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @aljoscha


Add frontend form to import content from DEV or Hackernoon

Hi @nilandev, I would like to work on the issue. Could you please assign me this issue.


Keyword highlighting is missing a number of tokens

The list of keyword tokens seems incomplete. The full list can be found here:
https://github.com/rubberduck-vba/RubberduckWeb/blob/master/RubberduckWeb/RubberduckWeb/VBA/FormattedCodeBlockBuilder.cs#L121-L290
(note, operators get keyword highlighting there)

closed with #9


Database code refactoring

Add an EntityManagerTemplate so we have more fine-grained control over database transactions rather than container-managed.
This PR is part of #1093 and most of the refactoring was written by @melowe . This PR attempted to be more specific to the database code refactoring, addressed a couple of issues and added tests to address them.

Codecov Report

Merging #1099 into master will decrease coverage by 0.02%.
The diff coverage is 98.61%.

@@             Coverage Diff              @@
##             master    #1099      +/-   ##
============================================
- Coverage     99.39%   99.37%   -0.03%
- Complexity     2264     2294      +30
============================================
  Files           350      352       +2
  Lines          6298     6391      +93
  Branches        360      363       +3
============================================
+ Hits           6260     6351      +91
  Misses            5        5
- Partials         33       35       +2

Flag
Coverage Δ
Complexity Δ

#unittests
99.37% <98.61%> (-0.03%)
2294.00 <41.00> (+30.00)
:arrow_down:

Impacted Files
Coverage Δ
Complexity Δ

...m/quorum/tessera/data/EncryptedRawTransaction.java
96.77% <ø> (ø)
18.00 <0.00> (ø)

.../com/quorum/tessera/data/EncryptedTransaction.java
100.00% <ø> (ø)
12.00 <0.00> (ø)

...com/quorum/tessera/data/EntityManagerTemplate.java
94.73% <94.73%> (ø)
4.00 <4.00> (?)

...orum/tessera/data/EncryptedTransactionDAOImpl.java
98.33% <98.27%> (-1.67%)
19.00 <17.00> (+12.00)
:arrow_down:

...um/tessera/transaction/TransactionManagerImpl.java
100.00% <100.00%> (ø)
38.00 <8.00> (+4.00)

...m/tessera/data/EncryptedRawTransactionDAOImpl.java
100.00% <100.00%> (ø)
11.00 <6.00> (+4.00)

...m/quorum/tessera/data/EntityManagerDAOFactory.java
100.00% <100.00%> (ø)
6.00 <6.00> (?)

...om/quorum/tessera/partyinfo/ResendManagerImpl.java
97.50% <100.00%> (ø)
8.00 <0.00> (ø)

.../java/com/quorum/tessera/config/EncryptorType.java
100.00% <0.00%> (ø)
1.00% <0.00%> (ø%)

... and 1 more

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 8ebfce7...74d3062. Read the comment docs.


Export to schematic doesn't work on KiCAD v8

Export to schematic doesn't work on KiCAD v8.  No error is displayed, but the LCSC field doesn't show up in the schematic.
I notice that the log loads my schematic as if it were kicad v6, not even v7, which is obviously incorrect.  I believe this is caused by the is_nightly check in schematicexport.py.
Thanks for your great work on this plugin!  Other than that one issue, it has basically worked perfectly on v8.
KiCad Version
Application: KiCad Schematic Editor x86_64 on x86_64

Version: 8.0.1, release build

Libraries:
	wxWidgets 3.2.4
	FreeType 2.13.2
	HarfBuzz 8.3.0
	FontConfig 2.15.0
	libcurl/7.79.1 (SecureTransport) LibreSSL/3.3.6 zlib/1.2.11 nghttp2/1.45.1

Platform: macOS Monterey Version 12.5.1 (Build 21G83), 64 bit, Little endian, wxMac

Build Info:
	Date: Mar 14 2024 14:07:38
	wxWidgets: 3.2.4 (wchar_t,wx containers)
	Boost: 1.84.0
	OCC: 7.7.2
	Curl: 7.87.0
	ngspice: 42
	Compiler: Clang 14.0.3 with C++ ABI 1002

Build settings:

I tried replacing if is_nightly(GetBuildVersion()): with if True: and while that got it to use the KiCad V7 routine, my file was still not updated.  So it looks like a new regex will be needed for kicad v8.

@computergeek1507 Can you look into this, please?

Is it okay if I drop v6 support, what are the versions you would like to support?

It will be amazing to keep v6 if possible (even if it is not supported anymore) as I have lot of old design with KiCad v6 that I do not want to switch to KiCad v7 or v8 as KiCad v6 is rock stable and to avoid some issues ... (I think I'm not the only one)

@bvernoux I totally understand and would not delete code that works for v6 if it not stands in the way of any future support, at least for a while.

Fixed in #433

😬 oh No, I didn't test the function myself before merging.
Hopefully you can fix the issue.
If you want you can email me the file<EMAIL_ADDRESS>so that I can take a look tomorrow.

send it to me at scooter_seh(at)yahoo(dot)com

@Bouni @computergeek1507 thank you so much for your work on this!
I just tested it, and unfortunately it corrupted my schematic file.  I want to help debug this. I'd be happy to privately send the schematic file to either of you to see what went wrong. In the mean time hopefully I will be able to manually edit the file to get back in business.

Wow, thank both for replying so quicky!!  I was able to restore the file from an auto-backup.  🎉   I sent the full project file to both of you.  The non-working schematic is suffixed BROKEN in the filename.
Attached below is the diff created when I assign the number from a single part, as well as a screenshot of an error when KiCAD tries to open the resulting schematic.  The only other "change" my difftool can detect is an extra newline at the end of the file.  But I doubt that matters at all.
Staring at that diff, isn't there just a missing parenthesis?

the plugin also makes a copy of the schematic file and adds "_old" before editing it. If it gets corrupts, delete the file and rename the "_old" file to get the original file back

Looks like this is fixed in 3959138.  Thanks!


Hide required markers on login page

Codecov Report

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

@@           Coverage Diff           @@
##           master     #329   +/-   ##
=======================================
  Coverage   58.99%   58.99%
=======================================
  Files        1029     1029
  Lines       32078    32078
  Branches     4392     4392
=======================================
  Hits        18925    18925
  Misses      13153    13153


Feature : How about showing links to github project  -  twitter profile of the author?

If the user wants to donate he would hit the command thanks requirements.txt donate which would show links to the authors patreon pages. While thanks requirements tweet would show the authors twitter profile links using which user can tweet the author expressing his love for the library.
The same for github as well - it would show github project links which helps people star the project.
I was planning to put this up in the package that I'm building but thought of chiming in my thoughts here instead of building yet another thanks library :smile:

Hey! I think something like thanks tweet requirements.txt is pretty reasonable, but I'm still thinking about it.
I definitely worry about too much information or information overload, so if you have thoughts on what the output would look like that would be amazing to see.

By output if you meant to say cli UI - I don't have it.
I mean to say it would similar to the output of the command thanks requirements.txt as of now.

I was indeed writing a small script to automatically star dependencies of a project in GitHub (heavily inspired by cargo-thanks). Right now, I have small pieces of code for fetching github repositories of the dependencies, which should be enough right now to fit in the current script (just listing the repositories and let the user manually star the ones he is interested in).
I think it could be nice to include the functionality using a simple --github flag to the script to show the links to the repositories instead the donations: not overloading the input, and offered as an alternative of "real" support (as donations are). @phildini, if this sounds good to you I can start working on a PR!

Hey @DiegoVicen! Yes, I will happily accept a PR. There are some changes incoming that might mean the PR goes through a couple rounds, but I'd still love to have some code-thoughts on what this looks like. 😄

Hey @DiegoVicen you might want to keep an eye on #7. I'm thinking of trying to pull project funding info from the setup.py, and pulling the github info from there as well seems ideal.

The prototype I mentioned just parses the output of pip show <package> and hopes to find a github link (which is not the case in many big packages, like numpy for instance), so that actually sounds promising. If not, at least using both methods and try to complement each other.


[FEAT]: Identify a suitable method to attach a OAK-D-Lite to LEGO Technic parts

Description
We need to find a way for ensuring that the OAK-D-Lite devices are securely fixed to the ARNEIS mechanical assembly
Additional Information
A detailed mechanical drawing of the OAK-D-Lite is available inside the official documentation

Please also review the OAK Design Guide - Camera Placement and Mechanical Stiffness

The distance between the two M4 threaded holes in the OAK-D-Lite enclosure is 75 mm - a VESA mount size.
Unfortunately 75 mm is not an exact multiple of 8.0 mm (1M i.e. the distance between the centerpoints of two studs of a LEGO brick), therefore my initial idea of securing a 11M beam with the help of two M4 threads does not work 😢

... well, unless we use this tool 👿

Another option could be to use a chunk of this product - available commercially as "Striscia Brick"

The item is listed on Amazon, although it is currently out of stock:
https://www.amazon.it/Flessibile-Costruzioni-mattoncini-riutilizzabile-Multicolore/dp/B0773MCWPK

Just read on channel "opencv-competition" of "Luxonis Community" in Discord

Update from Discord channel "opencv-competition" on 2022-01-24 11:23 CET:

Hi guys,
quick update on the adapter. The newest version is now opensource on github (https://github.com/muelleradam/OLLA).


[azure-kinect-sensor-sdk] Add new port

Too many files needs to be fixed...
Related: #8747.
Needs to wait #8769 #8779 merge.

/azp run

/azp run

/azp run

Is this PR ready to merge? I really want to see this in vcpkg.

LGTM @jasjuang We are waiting for the MacOS CI pipeline to be back up.


add support for mintbase-wallet

Description
Please include a summary of the wallet and how users can install it. Be sure to take a look at our guide on Custom Wallets to understand core concepts when integrating with Wallet Selector.
Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings

Summary
Mintbase Wallet integration
PR summary
This PR contains the integration with Wallet selector. To achieve it, a package has been added to package.json file (@mintbase-js/wallet)

Hi @rubenmarcus  , thanks for submitting this PR. You can find the criteria for including new wallets for Wallet Selector in our CONTRIBUTING.md file, and please confirm that the criteria are met.

Hi, @rubenmarcus I checked the Mintbase repo:  and below are some comments/suggestions:

You can resolve the walletUrl based on moduleOptions.options.network (see my-near-wallet in our repo):
https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/setup.ts#L32

This value should not be hard-coded, you can get it from options.network (see my-near-wallet):
https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L70

Please throw for verifyOwner if not supported: https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L212
The error message can be The verifyOwner method is not supported for ${metadata.name}

I am unable to understand the need for setting the callbackUrl in localStorage unless it's needed in some specific case on your codebase, signAndSendTransaction(s) can optionally take a callbackUrl if not provided you can just default to location.href.

Will the wallet support passing a contractId on signIn to create a FunctionCall access key, for the given contractId: https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L119

Is there any plan to add support for signMessage NEP413:
https://github.com/near/NEPs/blob/master/neps/nep-0413.md

Hi, @rubenmarcus I checked the Mintbase repo: and below are some comments/suggestions:

You can resolve the walletUrl based on moduleOptions.options.network (see my-near-wallet in our repo):
https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/setup.ts#L32
This value should not be hard-coded, you can get it from options.network (see my-near-wallet):
https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L70
Please throw for verifyOwner if not supported: https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L212
The error message can be The verifyOwner method is not supported for ${metadata.name}
I am unable to understand the need for setting the callbackUrl in localStorage unless it's needed in some specific case on your codebase, signAndSendTransaction(s) can optionally take a callbackUrl if not provided you can just default to location.href.
Will the wallet support passing a contractId on signIn to create a FunctionCall access key, for the given contractId: https://github.com/Mintbase/mintbase-js/blob/beta/packages/wallet/src/mintbase-wallet.ts#L119
Is there any plan to add support for signMessage NEP413:
https://github.com/near/NEPs/blob/master/neps/nep-0413.md

sorry for takin lot of time we were busy with NEARCON
THIS P/R fixes :

1 ✅

2 ✅

3 ✅

4 we default to location.href, but at mintbase we do tons of wallet.signTransactions check execute method of our sdk, and we use a different url for the callbackUrl, so we had a need to set this globally once, and not on every execute call with signTransaction ( imagine a codebase with + then 200 wallet calls )

5 that was the problem why on the example showed by kujtimprenkuSQA the guest book demo didnt worked. We dont get the contractId on our signIn, because we do differently, we just need to know the username, if you test the multiple transactions it will work look this one was made by me. Because we get the receiverId all the times on the call. We don't create access keys when connecting accounts.

6 Yes, we have plans. Is it required to get this merged?

5 that was the problem why on the example showed by kujtimprenkuSQA the guest book demo didnt worked. We dont get the contractId on our signIn, because we do differently, we just need to know the username, if you test the multiple transactions it will work look this one was made by me. Because we get the receiverId all the times on the call. We don't create access keys when connecting accounts.

6 Yes, we have plans. Is it required to get this merged?

Can you please add a check on the signAndSendTransaction if there is not contractId or a receiverId to throw and inform the dApp that they need to pass a receiverId?

Something like this:
const { contract } = store.getState();

if (!receiverId && !contract) {
     throw new Error("No receiver found to send the transaction to");
}

No, signMessage is not a blocker for merging this PR.

5 that was the problem why on the example showed by kujtimprenkuSQA the guest book demo didnt worked. We dont get the contractId on our signIn, because we do differently, we just need to know the username, if you test the multiple transactions it will work look this one was made by me. Because we get the receiverId all the times on the call. We don't create access keys when connecting accounts.

6 Yes, we have plans. Is it required to get this merged?

Can you please add a check on the signAndSendTransaction if there is not contractId or a receiverId to throw and inform the dApp that they need to pass a receiverId?

Something like this:
const { contract } = store.getState();

if (!receiverId && !contract) {
     throw new Error("No receiver found to send the transaction to");
}

No, signMessage is not a blocker for merging this PR.

this p/r fixes this
did the changes on the last commit
is anything missing?

Hi, @rubenmarcus thank you again for submitting this PR and addressing the comments, this looks good to me now.

Hi, @rubenmarcus we have just published a new version of the Wallet Selector that includes support for the Mintbase Wallet:
https://www.npmjs.com/package/@near-wallet-selector/mintbase-wallet


⚠ kibana has degraded performance

In 33094c1, kibana (kibana.daotrunghieu.com) experienced degraded performance:

HTTP code: 200
Response time: 2369 ms

Resolved: kibana performance has improved in 1f549e4.


[bot] Update to version 3.14.1

Auto-update version number to 3.14.1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12237242827. Examine the logs at this URL for more detail.

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: passed
rerender called: passed

Thus the PR was passing and merged! Have a great day!


Decode latency spikes randomly.

Unsure why this happens. I'm using OVR Space Calibrator in continuous calibration normally and suddenly everything stutters and decode latency spikes. I thought it could be my network but from the statistics view in the server, network latency and encoding are normal.
I'll add that I'm using HEVC.
I'll attach some pictures of the statistics screen taken from inside VR. The last picture is of normal conditions.
I also have a video of it happening, I'll try cutting the relevant parts and uploading it later if it'll help identify the issue.

Forgot to mention I increased the requested bitrate to 250mbps. Maybe that's the issue? Will do some tests with the default value later and report back.

I've noticed a bug recently with the streamer where a lot of games with mirror windows, for whatever reason, hang the client until the windows are minimized. Does it go away if you minimize the app? Or is it only with continuous calib?

It only stops if I wait it out or reset the connection by closing and re-opening the client. Continuous calib on or off doesn't matter, it will still spike and stay like that for a seemingly random amount of time.

Hmmm, what's your PC GPU? fwiw I've been pushing 350Mbps HEVC out of my 3060 with no issues on decode time, but the increasing slope makes me think it's something weirder

I’m on older drivers, I could try updating.

I was assuming it was something on the Vision Pro as this seems like an issue with decode.

Wait, what is it doing when the decode ramps up? like is it still drawing fine, is it headlocked (frame stuck to head), or is it timewarping (quad stuck in space, but you can walk around it)? Is there a lot of latency?

Timewarping mostly. Video lags and tries to catch up. Screens take a while to follow the head. Some artifacts on the video as well.

I have a 4090 w/ up-to-date drivers and OS running Windows 11. Getting over 200mbps starts to get really laggy (video has to catch up), but I also have Resolution set to Extreme (width: 6080). If I set it to Medium I can push it a little higher but definitely not 350. This varies greatly by game too. Above ^ is all for Alyx. If I play Legendary Tales I set it down to ~100 otherwise it's laggy.
Also, I'm on Adaptive Mode with Minimum set to 20 and max network latency set to 8ms.

So I guess first thing: Extreme is mostly pointless, the Vision Pro VR mode is limited to 1920*2, 3840. I've got my fingers crossed that Apple will enable foveated rendering and higher res so we can get crisper images, but yeah. You'll also want to ensure fixed foveation is on.
But I've also been working on disgnosing a strange quirk I've noticed where if I have my Mbps above ~150 on Medium with HEVC, it will give me network spikes only when I duck down and the view changes significantly. I have yet to see the decoder go above 5ms but it's possible you accidentally found extreme conditions which trigger some kind of Apple bug instead of odd networking behavior.

Ok actually I tested Extreme on my 3060 with HEVC and it actually works, maybe it's only H264 which fails. But also, the other thinfs I found improve smoothness are

Disable decoding latency limiter
Disable optimize game render latency

HEVC seems strange bc I can never get it to actually boost the encode Mbps, not sure if it actually can go higher than around 200

I think I’ve managed to replicate this consistently. Seems like it’s tied to bitrate as I suspected.
Here’s a video of me recording from the Vision Pro as it’s happening.
In the video you can see that decode latency spikes happen when bitrate also spikes. It’s also completely unusable above 300mbps and seems stable around 100mbps. Further in the video I show how it looks inside Steam VR when latency is super high.
I wouldn’t think it’s got to do with my network setup. I’m using WiFi 6 and sitting right next to my router. I don’t have issues when streaming a 4k@120Hz feed with Sunshine+Moonlight for instance.

I have recently updated to the latest nightly release of the server and used the test-flight version.
The problem is mostly solved. If I set the bitrate manually I still get lag spikes but they are no longer related to Decode according to the statistics tab. By setting the bitrate to Adaptive the lag spikes are almost completely gone.
One thing I realized is that bitrate plummets if I move the head too fast. Is this expected behavior?
Considering this question is unrelated to the original issue I'll close this one.


Clean up css code

@marisawadsworth  contribute here https://github.com/Ishaan28malik/Get-Dev-Resources


Add CSS Color Module Level 5 (color-mix(), color-contrast(), Relative color)

Spec: https://www.w3.org/TR/css-color-5/

color-adjust() "Removed color-adjust(), keeping relative color syntax"
color-mix() => https://caniuse.com/mdn-css_types_color_color-mix
color-contrast() => https://caniuse.com/mdn-css_types_color_color-contrast
Relative color

https://twitter.com/equinusocio/status/1253646505900212226

Virtually following up color-mod #2864
The color-adjust() function is not to be confused with the property of the same name https://caniuse.com/#search=color-adjust
We could consider creating an issue for each of the functions

Two are coming as experimental with Safari TP 122:

Added experimental support for CSS Color 5 color-contrast() (r273683)
Added experimental support for CSS Color 5 color-mix() (r273244)

color-mix() is available via MDN:

https://caniuse.com/mdn-css_types_color_color-mix
https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-mix()

color-contrast() as well:

https://caniuse.com/mdn-css_types_color_color-contrast
https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-contrast()

color-adjust() has been removed, per https://www.w3.org/TR/css-color-5/#changes-20210601:

Removed color-adjust(), keeping relative color syntax

I hadn't listed relative color syntax aka. "CSS Relative Colors", added now.
More links:

https://www.w3.org/TR/css-color-5/#relative-colors
https://blog.jim-nielsen.com/2021/css-relative-colors/
https://dev.to/fabiogiolito/create-a-color-theme-with-these-upcoming-css-features-4o83

Support bugs, stolen from https://www.bram.us/2021/12/27/css-in-2022/#the-hotlist--color-functions:

Chromium: Issues #1152772, #1092638, and #1274133
Firefox: Meta Issue #1679227
Safari: Issues #222530, #222258, and #221880

With color-adjust() removed from the spec and color-mix() and color-contrast() being covered by MDN data (via mdn/browser-compat-data#9489, mdn/browser-compat-data#12039), that appears to leave this issue only covering relative colors.
The spec also defines @color-profile and device-cmyk(), but both already have an MDN page, so they're likely to be covered by MDN data in the future:
https://developer.mozilla.org/en-US/docs/Web/CSS/@color-profile
https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/device-cmyk()

that appears to leave this issue only covering relative colors.

Check!

The spec also defines @color-profile and device-cmyk(), but both already have an MDN page, so they're likely to be covered by MDN data in the future

Good to know!
Those have also separate issues:

device-cmyk(): #1314
@color-profile: #5958

Please give us the ability to be able to see the opacity of a custom property that has a hex color code in it. This is propably the most common use case.
If you have a theme, each color is one variable (a hex color code), not 3 x RGB values one has to keep track of.
188K designers have sought for a fix for this on stackoverflow:
https://stackoverflow.com/questions/40010597/how-do-i-apply-opacity-to-a-css-color-variable
:root {
  --color: #f0f0f0;
}

#element {
  background: (somehow use var(--color) at some opacity);
}

@mangelozzi I think you should voice this pain point at https://github.com/w3c/csswg-drafts/labels/css-color-5
More generally, methodically looking at the most popular CSS stackoverflow questions that have unsatisfying answers, and reporting them on the CSS specs working group would be a way to yield dramatically significant, reality-aligned improvements to CSS.

Links for relative colors:

Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1274133
Firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1701488
Implemented in Webkit behind a flag: https://bugs.webkit.org/show_bug.cgi?id=221880 (Safari TP 122)

Links to issues for color-contrast:

Chromium: https://bugs.chromium.org/p/chromium/issues/detail?id=1111385
Firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=1682439
Implemented in Safari already behind a flag

https://developer.apple.com/documentation/safari-release-notes/safari-16_4-release-notes:

Added support for CSS relative color syntax.

Let's get these pages on Can I Use!

Safari 16.2 added support for color-mix() back in December 2022.

Safari 16.4 beta has support for Relative Color Syntax.

yes, please add relative color syntax to caniuse! i've been checking for it periodically and end up slapping my forehead each time thinking i wasn't putting in the right incantations into the search box, to then finally realize it's not in there at all yet.

Safari 16.4 shipped with support for relative color at the end of March

Someone needs to make three pull-requests.
What is the current state of all the other browser support? Does Chrome/Edge/other Chromium mobile browsers or Firefox support any of these three new features? Even in partial experimental state?
Can anyone find issues for Chromium and Gecko so we can link to them?
Are there any MDN pages yet?

@jensimmons To my understanding the only thing missing from caniuse covered by this issue is relative color, with the rest being available via MDN:

color-mix()

https://caniuse.com/mdn-css_types_color_color-mix
https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-mix

color-contrast()

https://caniuse.com/mdn-css_types_color_color-contrast
https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-contrast

Issues for relative color (comment above):

Links for relative colors:
Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1274133
Firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1701488
…

Relative color syntax now available at https://caniuse.com/css-relative-colors
Closing this ticket now as all appear to be accounted for now, thanks all for helping to provide info. :)


How to change drag dropEffect based on drop targets?

The current dropEffect example shows how to change dropEffect on the drag source based on the drag source itself (https://react-dnd.github.io/react-dnd/examples/customize/drop-effects).  Is there a way to dynamically change the dropEffect based on the target?  For example, I have 2 drop target boxes.  If I drag an item to Box 1, it should show Add effect.  If I drag an item to Box 2, it should show Copy effect.

It doesn't look like this is supported out of the box, but I was able to get something working with this hack (loosely inspired by this hack)
export const HTML5BackendWithModifierKeys = (manager, context) => {
	const result = HTML5Backend(manager, context);
	result.dropTargetOptions = {};

	result.handleTopDragOver_ = result.handleTopDragOver;
	result.handleTopDragOver = (e) => {
		const { dragOverTargetIds } = result;
		const bestDropTarget = (dragOverTargetIds ?? []).find(
			(x) => result.monitor.canDropOnTarget(x) && x
		);

		const dropTargetDropEffect =
			bestDropTarget &&
			result.dropTargetOptions[bestDropTarget]?.dropEffect;

		result.handleTopDragOver_(e);
		result.altKeyPressed = Boolean(dropTargetDropEffect);
		e.dataTransfer.dropEffect =
			dropTargetDropEffect || e.dataTransfer.dropEffect;
	};

	result.connectDropTarget_ = result.connectDropTarget;
	result.connectDropTarget = (targetId, node, options) => {
		result.dropTargetOptions[targetId] = options;
		const unsubscribe = result.connectDropTarget_(targetId, node, options);

		return () => {
			delete result.dropTargetOptions[targetId];
			unsubscribe();
		};
	};

	return result;
};

Then you can specify the dropEffect in an options object in the useDrop hook like so
useDrop({
	...,
	options: {
		dropEffect: 'copy'
	},
})


tz-cyber-dev-day notebook updates

Description
Updates to the dev day notebook content. Adds codes skeleton exercise.
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

This was incorporated in separate PRs. Closing


Remove bindig DataContext to DateTimePickerModel

                <local:DateTimePicker
                    Margin="0,0,6,0"
                    DataContext="{Binding DateTimePickerModel}" />

Not priority, close for now.


Trying to use slots with template tag in an inline template breaks syntax highlighting

The problem in a nutshell is this, using my dropdown component  example:
Vue.component('test', {
  template : `
  <dropdown label='My Button contents'>
    <my-actual-menu-component></my-actual-menu-component>
  </dropdown>
  `,
   beforeCreate() {
      console.log('whatever');
    }
})

Above everything is highlighted correctly, but as soon as I use the template tag in there like so:
Vue.component('test', {
  template : `
  <dropdown>
    <template #btn>
      My Button Contents<i class='icon></i>
    </template>
    <my-actual-menu-component></my-actual-menu-component>
  </dropdown>
  `,
   beforeCreate() {
      console.log('whatever');
    }
})

The template string and everything above it is highlighted properly, but everything after the backtick enclosed string gets highlighted as if it is a string. If I remove the  #btn from the <template #btn> tag so that there are no spaces between template and its opening and closing angle brackets, only the template tags themselves get highlighted as strings, and the rest of the document is fine. However, in order to use slots properly we must necessarily use the name sometimes, as in this case where the default slot is already being used for the dropdown menu, not the toggle button. I should mention that <template v-slot:btn> does not work either. It seems to be the space, even <template > gives the same bug.
For now my workaround is to put the template at the bottom of the definition object, since it only messes with the highlighting after it. Thank you for this plugin by the way! This is the best way to write components, I've wanted it since I started using angularjs 5 years ago.

I think I posted this in the wrong place, dunno how I got here.

I believe this is for my Javascript-Vue extension.

Closing this as it pertains to another extension.

Moved to Javascript-Vue #4


Date fields accepted by zod schema are not accepted by JSON schema generated from it

Hello @matthew-kaye, thanks for opening an issue.
The behavior you describe makes sense to me. z.date() represents the JS Date class, which by default is serialized to JSON as a date-time string, thus matching z.string().datetime().
Feel free to reopen the issue if you don't agree!

I'm not sure I do agree actually sorry 😅 Since a zod schema for z.date() would allow a data field like "2024-01-01" but a JSON schema requiring a date-time would not. It seems normal to me to expect that data which would pass the zod schema would also be accepted by the JSON schema generated from it

Hope that makes sense! If alternatively you know of a way to ensure the generated schema accepts a date field then do let me know - I'm keen to actually check it's a valid date with a format field rather than attempt a long regex
Original issue

Hmm, ok, I understand the issue. I'm usually more focused on describing the output of data validated through Zod, which is ironic given we can only serialize the input expectations... I guess in this case another(!) strategy option might be useful to be able to generate something like
{
  "type": "string",
  "anyOf": [
    {
      "format": "date",
    },
    {
      "format": "date-time"
    },
    {
      "format": "time"
    }
  ]
}

Hmm, ok, I understand the issue. I'm usually more focused on describing the output of data validated through Zod, which is ironic given we can only serialize the input expectations... I guess in this case another(!) strategy option might be useful to be able to generate something like
{
  "type": "string",
  "anyOf": [
    {
      "format": "date",
    },
    {
      "format": "date-time"
    },
    {
      "format": "time"
    }
  ]
}

Thanks for understanding, that would certainly solve all my issues if that was the output!

@matthew-kaye Mind checking out #111?

Ah sorry went straight to theatre after 5:30, will test it out though!


Move interaction with ladders

When casting a move and looking up or down when in a ladder block then the magic move doesn't work. When you look forward the move does work.

May be a vanilla issue, no interact or animate events are fired when left clicking towards the air block here. They are fired when clicking on the block. No arm movement animation is shown when clicking into the air as well.

Bummer :(


what's the most robust way of importing the default CSS from an html static page?

📚 Documentation
It's unclear what would be the preferred way of importing the default CSS from an html static page.
I have the need to host static .html pages in the /static folder and would like to style them like the rest of the docusarus  website.
In docusaurus v1 it was possible to access it at ${baseUrl}css/main.css.
How should it be done in v2?
Have you read the Contributing Guidelines on issues?
yes

Hi,
Everything you put in /static will be hosted
So if you have /static/myPage.html, and /static/css/myCss.css
You can load "/${baseUrl}/css/myCss.css", or preferably "css/myCss.css"
We don't process anything from this folder, content is published as-is, Docusaurus does not really have anything to do with that

thanks for the reply @slorber
i was not referring to custom css, but rather the default css from @docusaurus/theme-classic

The CSS file loaded by the classic theme is infima/dist/css/default/default.css (in node_modules)
For the docusaurus website, it is loaded through webpack css loader.
You could just put this file in /static and load it in HTML pages


Cannot add Data Links

Hello,
Thanks for your plugin !
I am currently replacing the Angular Traffic Lights plugin in our dashboards by yours. I am able to display the data I want, the way I want, from an OpenSearch Data Source:

I would like to add data links to redirect to another dashboard.
When I click on the "Add link" button I get this formular

But when I click on "Save" I get this error:

I am running the dockerised Grafana 11.1.4 version.
I gave an example data link I would like to create but what I am looking for is to have a link for each traffic light that a viewer could click on to be redirected to a specific dashboard depending on the value of the data which is displayed.
Thanks for your help !

@curvaton thanks for raising this.
I tried to reproduce your issue locally and I cannot. The feature isn't custom to my panel, it's part of the Grafana codebase. You can find the docs here.
As for the error - could you pass me the panel JSON and panel data please? They can be accessed by clicking inspect in the panel menu. If I have those I might have a better chance of reproducing the issue.

Hello,
Thanks for the investigation. Since this happens in a production environment, to give the panel json and panel data I have to prepare some mock data and reproduce in a local environment and then send it to you. I'll try to do this as fast as I can.
Thanks again.

Hello,
I could finally spend some time to produce mock data for reproducing the issue. I could reproduce it running in a local docker-compose environment with an opensearch datasource and grafana services (version 11.1.4).
Here are the data and panels in json format as requested.
data.json
panel.json
Hope this helps.

Here are the data and panels in json format as requested.

Thanks for taking the time @curvaton. I'll try and repro it and get back to you soon.

@curvaton A fix for this should now be available in version 0.5.1. 🚀
Please reply here if you that's not the case.


.Net: Assistants implementation still in the code, while "almost" identical to the new Agents implementation being used..

Describe the bug
A clear and concise description of what the bug is.
Seemingly duplicate code still is in the code base
To Reproduce
Steps to reproduce the behavior:

Go https://github.com/microsoft/semantic-kernel/tree/main/dotnet/src/Experimental/Assistants
look at the code
go https://github.com/microsoft/semantic-kernel/tree/main/dotnet/src/Experimental/Agents
look at the code
compare

Expected behavior
A clear and concise description of what you expected to happen.
There should be no /Assistants ;)
Of course, maybe you left that there for a reason but the code is mostly similar-identical for what I saw, so... It will always be in the git history, so I'd delete it. It can always be retrieved.
Unless I am missing something, which can pretty well be...

Should be removed in linked PR.  Thanks for callilng this out.


Mem_OutputUsage calling out_fnct from inside a critical region

I ran into the following issue: the function Mem_OutputUsage is calling the provided out_fnct from within a critical region. The out_fnct I was providing was writing to the serial port. The STM32 BSP implements the serial code using a semaphore that gets signalled in an interrupt. This code doesn't work if it is called from inside a critical region, causing weird issues like OSSemPend returning without an error when the semaphore was not signalled yet, eventually causing the code to hang.
Since using a semaphore to synchronise with an interrupt is a common pattern for implementing IO drivers, I feel the implementation in Mem_OutputUsage should not do IO from within the critical region.

This sounds like a BSP implementation issue. Which STM32 BSP are you referring to?


Remove global config from auth

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Remove global configuration from auth classes.
Other references
Part of: #2758

LGTM, thanks!


Is there a way of changing the animation of the button icon?

Hello;
I want to change the animation of the button icon. Normally it returns counter-clockwise a little bit. but what I need is just move up, or stay without animation. Is it possible?

I solved it by writing:
[fab setRotationDegrees:0];


Cannot invoke 'bind' with an argument list of type '(to: Binder<Resource?>)'

Hi there, I'm just following the doc and receive "Cannot invoke 'bind' with an argument list of type '(to: Binder<Resource?>)'" error. Please help me check if my code have any issue
viewModel.user.map { user -> NSURL in        return NSURL.init(string: user.avatar) ?? NSURL.init()  } .bind(to: avatarImageView.kf.rx.image()) .disposed(by: rx.disposeBag)

NSURL doesn't conform to Resource type. Use URL instead:
viewModel
    .user
    .map { user in URL(string: user.avatar) }
    .bind(to: avatarImageView.kf.rx.image())
    .disposed(by: rx.disposeBag)

NSURL is deprecated, please use URL instead.

Instance method 'bind(to:)' requires the types 'URL?' and 'Resource?' be equivalent

I think you have to unwrap your URL.

Instance method 'bind(to:)' requires the types 'URL' and 'Resource?' be equivalent

I unwrap the URL but the problem has always existed


Add entry point for exercise information

Don't merge this pull request i find some display error. When the exercise has a title and a description the aesthetics are horrible.

What is the status of this PR? @leunam217

Hi @yurug AFAICT, this PR is almost-ready, but the layout should probably be made more responsive (so that for example, if the browser window is resized to fit the half of the screen, the metadata will be hidden and available in a tab…)

the layout should probably be made more responsive (so that for example, if the browser window is resized to fit the half of the screen, the metadata will be hidden and available in a tab…)

I plan to address this soon / this week.

What is the status of this PR? @erikmd

Hi @yurug, sorry for the delay… this PR is still work-in-progress.
But I can try to finalize it this WE (otherwise feel free to ping me then!)

I've just fix the conflict, extended and polished this PR,
but as I don't have access to @leunam217's source repo, I'll open a new PR.
@yurug please close this PR #297, I don't have the rights to do so.


Update chart repository URL to new location

Helm is losing their old chart repository location and have had to move to a new URL. We should update our hard-coded URL for the stable repository to the new location:
OLD: https://kubernetes-charts.storage.googleapis.com/
NEW: https://charts.helm.sh/stable
Announcement here: https://helm.sh/blog/new-location-stable-incubator-charts/

/assign

Thanks for fixing this @Evalle! :)


Parse terraform local module source

What this PR does / why we need it:
This PR changes to show URL for terraform local module.
ref: local module https://www.terraform.io/language/modules/sources#local-paths
Which issue(s) this PR fixes:
A part of https://github.com/pipe-cd/pipecd/issues/3303
Does this PR introduce a user-facing change?:

Be able to show URL for terraform local module.

Nice
/lgtm

@ffjlabo Thanks for your great work!
Please feel free to transfer this PR to me. 👍
And let me hold this PR to take a deeper look at this feature.
/hold

/hold


feat: Return slices instead of Vec

This PR might need to be double checked. I tried to avoid any regressions, but the  bench marks gave mixing results

I have added a note for the failing test case. I am actually not 100% sure what is happening because I don't remember what the test looks like and the output is...confusing, lol. But that is the only change I can see that might have caused that.
You also committed .dstore in src.
I will need to test this against the application I am using downstream to see what happens before merging, it looks like it shouldn't require many changes though.
The perf benchmarks can be quite mixed, you can get very different results running the full backtest repeatedly. Anything that reduces clones will help performance.

@calumrussell Fixed the issue.
I also managed to squeeze more perf out of the dividend function(1-2%). Will add it in the next PR.

@calumrussell Fixed the issue. I also managed to squeeze more perf out of the dividend function(1-2%). Will add it in the next PR. Edit: Multiple perf-increases. Should equate to a >4% (Optimistic estimate) increase in perf, perhaps even more

Okay....39% perf gains???!

The trade test seems to regressed slightly.
I thought it was pay_dividends too (there should be a greater than zero test before paying, for some reason there are no checks for zero values in the payment functions) but couldn't find anything.
I will have to pull this in my other project to see what needs to be changed and then merge tomorrow. Appreciate your time and effort.


Add warning to ASDK content regarding out of capacity on the local C:\ of the ASDK host may render the env unrecoverable

@knithinc, is this the same issue as #450?


Failed to create terminal for task when running automatic tasks

VSCode Version: 1.32.3
OS Version: macOS Mojave 10.14.3

Steps to Reproduce:
Automatic tasks seem to interfere with extensions that need to run tasks.
I use a PHPUnit extension (https://github.com/elonmallin/vscode-phpunit) and I find that whenever I launch a workspace that runs automatic tasks, the phpunit commands fail with an error message saying Failed to create terminal for task phpunit: run.
Turning off the automatic tasks seems to fix the issue.

Does this issue occur when all extensions are disabled?: Yes/No
No

This is duplicate of #70557. It should be fixed in insiders, and the fix should be available in the next release.


CI to generate binary database test datasets

We should test our multi-platform portability using binary databases created on each platform, which ideally would be generated by our CI runners.
As well, this could prevent the need to update and regenerate the binary test cases every this a storage update is made.

Also see #2944.


Kibana 5.6.10 support

Hey,
We're using ElasticSearch 5.6.10. Will you plan to support it officially?
As a workaround, I'm trying to install kibana 5.6.9 instead of kibana 5.6.10.

I'll port and test this weekend, look for a release Sunday or Monday morning.

An Update:

Workaround works. Kibana 5.6.9 supports ElasticSearch 5.6.10 (a warning message is shown, though). We don't want to support 2 version of Kibana, so 5.6.10 release would be great. Thank you!
I did noticed that document view called ‘Structured’ WASN'T added to the discover application. I don't know if it has any correlation with the versions conflict.

Ports completed:
5.6.10
5.6.11
6.3.2

Upgraded to 5.6.10 today, worked like a charm

Hi @ppadovani , your plugin looks really promising to what I'm trying to do, any plans on giving support for 6.4.0 yet? As my project is new I chose to use the latest ELK version.
I just tried the last version of your plugin, modified the package.json settings and give it a go but Kibana broken while trying to "optime" the assets of your plugin. Let me know if there's anything I could do to help you, I'm very new to ELK but I'm familiar with Web stuff.
Thanks.


Resolve a crash issue when entering a non parsable date

add a question mark to dateTime.months[monthData.month] to avoid the date picker to crash when a wrong date is selected. Instead a NAN and relatively empty date picker is shown.
The demo page has been slightly changed to allow the selection of a time. When entering for example 2015-01-0100:00 (blank space is missing), it was crashing before.

I never knew there is syntax like this.
dateTime.months[monthData.month]?.fullName


process.exit() inside switch/case gives invalid lint/suspicious/noFallthroughSwitchClause - conflicting with TypeScript

Environment information
e.g.
switch (getStringProperty(error, 'code')) {
  case 'EACCES':
    console.error(`${bind} requires elevated privileges`);
    process.exit(EXIT_ERROR);
  case 'EADDRINUSE':
    console.error(`${bind} is already in use`);
    process.exit(EXIT_ERROR);
  default:
    throw error;
}

Gives error
error.ts:86:7 lint/suspicious/noFallthroughSwitchClause ━━━━━━━━━━

  ✖ This case is falling through to the next case.

    84 │     // handle specific listen errors with friendly messages
    85 │     switch (getStringProperty(error, 'code')) {
  > 86 │       case 'EACCES':
       │       ^^^^^^^^^^^^^^
  > 87 │         console.error(`${bind} requires elevated privileges`);
  > 88 │         process.exit(EXIT_ERROR);
       │         ^^^^^^^^^^^^^^^^^^^^^^^^^
    89 │       case 'EADDRINUSE':
    90 │         console.error(`${bind} is already in use`);

  ℹ Add a `break` or `return` statement to the end of this case to prevent fallthrough.

However, adding a return or break gives a typescript error saying unreachable code detected.
Rule name
lint/suspicious/noFallthroughSwitchClause
Playground link
https://biomejs.dev/playground/?indentStyle=space&quoteStyle=single&importSortingEnabled=false&code=cwB3AGkAdABjAGgAIAAoAGcAZQB0AFMAdAByAGkAbgBnAFAAcgBvAHAAZQByAHQAeQAoAGUAcgByAG8AcgAsACAAJwBjAG8AZABlACcAKQApACAAewAKACAAIABjAGEAcwBlACAAJwBFAEEAQwBDAEUAUwAnADoACgAgACAAIAAgAGMAbwBuAHMAbwBsAGUALgBlAHIAcgBvAHIAKABgACQAewBiAGkAbgBkAH0AIAByAGUAcQB1AGkAcgBlAHMAIABlAGwAZQB2AGEAdABlAGQAIABwAHIAaQB2AGkAbABlAGcAZQBzAGAAKQA7AAoAIAAgACAAIABwAHIAbwBjAGUAcwBzAC4AZQB4AGkAdAAoAEUAWABJAFQAXwBFAFIAUgBPAFIAKQA7AAoAIAAgAGMAYQBzAGUAIAAnAEUAQQBEAEQAUgBJAE4AVQBTAEUAJwA6AAoAIAAgACAAIABjAG8AbgBzAG8AbABlAC4AZQByAHIAbwByACgAYAAkAHsAYgBpAG4AZAB9ACAAaQBzACAAYQBsAHIAZQBhAGQAeQAgAGkAbgAgAHUAcwBlAGAAKQA7AAoAIAAgACAAIABwAHIAbwBjAGUAcwBzAC4AZQB4AGkAdAAoAEUAWABJAFQAXwBFAFIAUgBPAFIAKQA7AAoAIAAgAGQAZQBmAGEAdQBsAHQAOgAKACAAIAAgACAAdABoAHIAbwB3ACAAZQByAHIAbwByADsACgB9AA%3D%3D
Expected result
It should not display an error
Code of Conduct

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

TypeScript treats code section below a never expression unreachable, and the signature of process.exit is exit(code?: number): never.
To properly support this we'll need some basic type inference.

I would argue that this is a Node.js thing, and process.exit is a side effect that the rule shouldn't handle.
Plus, this could be valid for every function that works as process.exit.
Sure, this could be solved by some type-inference, although we are still far from it.
My suggestion would be to refactor the code and place process.exit somewhere that is safer to handle.
Probably we should extend the documentation of the rule and explain that it doesn't handle side-effect functions.

For anyone who wants to help, we need to update the rule's documentation to explain that process.exit isn't considered by the rule.


Fixing problem of referencing the right video path

Small fix regarding the issues raised here: #32

closing with the change implemented in #36

Also added this PR to allow python >= 3.8 (although =3.8 is still required for linux)


Add a working call-in number

Signed-off-by: Doug Davis<EMAIL_ADDRESS>

Fixes #168 by copying opencontainers/runtime-spec#512.

LGTM

LGTM


lint(quickstart): Invoke command with ./

installer does not install in PATH
Closes #361
How was this PR tested?
To be honest, it hasn't. Looked for a make doc or some kind of way to run the documentation localy without taking too much time and stopped here :)  I checked the output md was correct.
Checklist

[x] included documentation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Merci Antoine!


log('Your string'); Not print full data android studio console

log('Your string');  //Big Data Full print problem
import  nb_utils package any page .. than not add import 'dart:developer';
example ---
import 'package:nb_utils/nb_utils.dart';
import 'dart:developer'; // import problem
log('data: $data');

Do not use import 'dart:developer';

but not print api response big data nb utils package log(e)

Log method uses the default print method of dart. The difference is only that log method will not be called when the app is in release mode.

Print method can print specific characters in console.


Supported Kubernetes Versions should be updated

Description
Supported Kubernetes Versions in documentation are 1.21.2, 1.20.8, 1.19.12, but actually 1.22.8,1.21.11,1.20.15.
Location

https://tanzucommunityedition.io/docs/v0.12/support-matrix/#supported-kubernetes-versions
https://tanzucommunityedition.io/docs/edge/support-matrix/#supported-kubernetes-versions

Additional Context

Thanks @hidekazuna - we're going to track this in #4088


Grpc PHP extension causing latency

What version of gRPC and what language are you using?
1.54.0 (occurs on other versions too)
What operating system (Linux, Windows,...) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
php (installed using pecl install grpc)
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
When I try to fetch large number of docs (20K) from Firestore server using a Firestore::runQuery or Firestore::listDocumentscall, it takes unusually high amount of time (100+ seconds). nodejs performance for the same is <10 seconds.

Code to get time elapsed in seconds Firestore documents
require_once __DIR__ . '/vendor/autoload.php';

use Google\Cloud\Firestore\FirestoreClient;

$client = new FirestoreClient();
$collection = $client->collection(uniqid());

function insertDocuments($client, $collection, $count)
{
    $bulkwriter = $client->bulkWriter();
    for ($i = 0; $i < $count; $i++) {
        $newDoc = $collection->newDocument();
        $bulkwriter->create($newDoc, [
            'foo' => 'bar',
            'index' => "{$i}"
        ]);
    }
    $bulkwriter->flush();
}

// insert 20K docs
const DOCS_ADDED = 20000;
insertDocuments($client, $collection, DOCS_ADDED);

// validate count equals 20K
$count = $collection->count();
print ("{$count} documents found the collection.".PHP_EOL);

function calcTimeElapsedInMillis($callable)
{
    $startTime = microtime(true)*1000.0;
    $callable();
    $endTime = microtime(true)*1000.0;
    $timeElapsedInMillis = $endTime - $startTime;
    return $timeElapsedInMillis;
}

function callGetDocuments($collection, $expectedCount)
{
    $documents = $collection->limit($expectedCount)->documents();
    $actualCount = count(iterator_to_array($documents));
    if ($actualCount != $expectedCount) {
        print("Found {$actualCount} expected {$expectedCount}".PHP_EOL);
    }
}

function callListDocuments($collection, $expectedCount)
{
    $documents = $collection->listDocuments([
        'pageSize' => $expectedCount,
        'resultLimit' => $expectedCount
    ]);
    $actualCount = count(iterator_to_array($documents));
    if ($actualCount != $expectedCount) {
        print("Found {$actualCount} expected {$expectedCount}".PHP_EOL);
    }
}

function runTests($collection, $testCaseCount)
{
    echo "| Documents Count | Fetch Seconds in documents() | Fetch Seconds listDocuments() |".PHP_EOL;
    echo "|-------------|---------------------|--------------------|".PHP_EOL;
    $testCases = generateTestCases($testCaseCount);
    foreach ($testCases as $docCount) {
        $timeElapsedGetDocs = calcTimeElapsedInMillis(function () use ($collection, $docCount) {
            return callGetDocuments($collection, $docCount);
        });
        $timeElapsedGetDocs = ceil($timeElapsedGetDocs / 1000.0);
        $timeElapsedListDocs = calcTimeElapsedInMillis(function () use ($collection, $docCount) {
            return callGetDocuments($collection, $docCount);
        });
        $timeElapsedListDocs = ceil($timeElapsedListDocs / 1000.0);
        print ("|{$docCount}|{$timeElapsedGetDocs}|{$timeElapsedListDocs}|".PHP_EOL);
    }
    echo "Test completed successfully.".PHP_EOL;
}

function generateTestCases($testCase)
{
    $testCases = [];
    for ($i = 0; $i < $testCase; $i++) {
        $testCases[] = DOCS_ADDED / pow(10, $i);
    }
    return $testCases;
}
runTests($collection, 4);

It gives the following log:

Documents Count
Fetch Seconds in documents()
Fetch Seconds listDocuments()

20000
106
104

2000
11
11

200
2
2

20
1
1

What did you expect to see?
The time range here should not linearly increase with doc count. It seems the grpc extension is not able to handle the large doc count properly.
What did you see instead?
I expect the time for fetching 20K documents to be <10 seconds.
I profiled the php process running the above code and it showed this flame which shows that most of the time is spent in Grpc\call::startBatch and Grpc\ServerStreamingCall::responses(you need to download & open  file in chrome and hover over the largest bars in red. This was generated by a tool called phpspy.
Anything else we should know about your project / environment?
This is actually a sub-issue that I am reporting on behalf of an external user who reported this issue on https://github.com/googleapis/google-cloud-php/issues/4429. Surprisingly, the user reported (and I validated) that a similar nodejs script fetches all 20K documents in under 10 seconds.
Let me know if I need to do further debugging, I dont know how to profile grpc or node.

I couldn't reproduce this issue, using the same code, I got the following test results:
From GCP VM:

Documents Count
Fetch Seconds in documents()
Fetch Seconds listDocuments()

20000
7
7

2000
1
1

200
1
1

20
1
1

From my laptop:

Documents Count
Fetch Seconds in documents()
Fetch Seconds listDocuments()

20000
4
4

2000
1
1

200
1
1

20
1
1

Confirmed that this is indeed the timing that I can see on a clean setup.
Maybe it's a transient issue that i saw. 🥀


extra740 scans showing public ebs snapshots

This scan shows all ebs snapshots that are public and there can be a lot that come up. Is there a way to ignore the public snapshots shared with an account?

Hi @caseybecking, I think yuo are referring to extra72 7.2 [extra72] Ensure there are no EBS Snapshots set as Public

by default it uses --max-items 100 unless you run prowler with -m <higher value>.  Also that check runs this command aws ec2 describe-snapshots --region REGION --owner-ids ACCOUNTID --output text --query 'Snapshots[*].{ID:SnapshotId}' --max-items 100 hence it shows only snapshots own by the account where the check is being ran.
If you are really talking about check extra740 it works the same way.

reading doc, I see that --owner-ids can a combination of AWS account IDs, self , and amazon. Not sure if in your case self would make the trick. Can you try and let me know? Thanks.

@caseybecking are you using a different default output in your .aws/config file? I noticed a similar issues in my scans when the default output was modified. I just submitted issue #696 related to this.

Interesting yes it may have been config related. After removing the AWS_DEFAULT_OUTPUT="text" from my env variables and leaving it blank i no longer get the notification of failed snaps due to encryption.

Can you please confirm that? I can confirm that both checks extra740 and extra72 use --output text explicit in their code so I doubt it is related to output.

I just tested a couple version it did not have the issue in RC2 but has the same issue in RC4 & RC5 and it looks to be something with jq possibly. If i add the environment variable of AWS_DEFAULT_OUTPUT="text" to zsh it breaks the output, if i unset it and run the same command the output looks to be fine.
With variable -
./prowler -c check740 -p REMOVED -r us-east-1
jq: error (at <stdin>:1): Cannot index number with string "Account"
parse error: Invalid numeric literal at line 1, column 17
jq: error (at <stdin>:1): Cannot index number with string "Arn"
parse error: Invalid numeric literal at line 1, column 17
jq: error (at <stdin>:1): Cannot index number with string "UserId"
parse error: Invalid numeric literal at line 1, column 17
                          _
  _ __  _ __ _____      _| | ___ _ __
 | '_ \| '__/ _ \ \ /\ / / |/ _ \ '__|
 | |_) | | | (_) \ V  V /| |  __/ |
 | .__/|_|  \___/ \_/\_/ |_|\___|_|v2.3.0RC5
 |_| the handy cloud security tool

 Date: Tue 24 Nov 2020 05:57:30 AM PST

 Colors code for results:
 INFO (Information), PASS (Recommended value),  WARNING (Ignored by whitelist),  FAIL (Fix required),  Not Scored

 This report is being generated using credentials below:

 AWS-CLI Profile: [REMOVED] AWS API Region: [us-east-1] AWS Filter Region: [all]
 AWS Account: [] UserId: []
 Caller Identity ARN: []

 7.40 [extra740] Check if EBS snapshots are encrypted (Not Scored) (Not part of CIS benchmark)
       INFO! Looking for EBS Snapshots in all regions...
       FAIL! eu-north-1: snap-066d68b937472d8f2 is currently not encrypted!
       FAIL! eu-north-1: snap-037c7cf91e3e4390d is currently not encrypted!

After unseting the variable -
./prowler -c check740 -p REMOVED -r us-east-1
                          _
  _ __  _ __ _____      _| | ___ _ __
 | '_ \| '__/ _ \ \ /\ / / |/ _ \ '__|
 | |_) | | | (_) \ V  V /| |  __/ |
 | .__/|_|  \___/ \_/\_/ |_|\___|_|v2.3.0RC5
 |_| the handy cloud security tool

 Date: Tue 24 Nov 2020 05:59:38 AM PST

 Colors code for results:
 INFO (Information), PASS (Recommended value),  WARNING (Ignored by whitelist),  FAIL (Fix required),  Not Scored

 This report is being generated using credentials below:

 AWS-CLI Profile: [REMOVED] AWS API Region: [us-east-1] AWS Filter Region: [all]
 AWS Account: [REMOVED] UserId: [REMOVED]
 Caller Identity ARN: [arn:aws:iam::REMOVED:user/REMOVED]

 7.40 [extra740] Check if EBS snapshots are encrypted (Not Scored) (Not part of CIS benchmark)
       INFO! Looking for EBS Snapshots in all regions...
       INFO! eu-north-1: No EBS Snapshots found

AWS cli takes output preferences via environment and command line, the question here is that when both are specified, which takes precedence? It sounds like environment does, so maybe we need to unset AWS_DEFAULT_OUTPUT in the prowler startup.

AWS cli takes output preferences via environment and command line, the question here is that when both are specified, which takes precedence? It sounds like environment does, so maybe we need to unset AWS_DEFAULT_OUTPUT in the prowler startup.


Update frameworks in PowerDNS

At the moment PowerDNS has admin-lte 2.4.9 and bootstrap 3.4.1 as framework. These are both released in february 2019. Meanwhile stable major versions of admin-lte (current 3.05) and bootstrap (current 4.5.0) have been released. I would like to see these to be updated. If help is needed, i can help.
Especially since bootstrap 4 runs the styling in scss in stead of less, which makes it easier to customize colors etc.

Sorry for the late response. I didn't have much time for this project. Yes, It is nice to keep the versions up-to-date. Feel free to submit your contribution with a PR. Thanks.

@the-missing-pink Maybe you could help @RGanor with #1173 ?
The new maintainer team is not well versed in UI things.

Hey @the-missing-pink, i started working on updating those, but as @jbe-dw said we are not really an UI experts and it giving me some problem.
I’ll be greateful for any help in #1170

@RGanor i would help, but i have only limited time (some hours per week). I've started on my local dev to adapt the header and menu.


sbt-extras: init at 77686b3

Fixes #21783
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

We use this at work a lot.

@GrahamcOfBorg sbt-extras

Thanks!


Move RPC Feature into separate projects

The main objective of this PR was to create two new projects and move the majority of files into the new projects:

Stratis.Bitcoin.Features.RPC
Stratic.Bitcoin.Features.RPC.Tests

Created an "Interfaces" folder in Stratis.Bitcoin.
Created feature agnostic interfaces to deal with FullNode dependencies:

IBlockStore. Block store methods required by FullNode RPC Controller. This may be temporary until we find a better way to extend or add to RPC methods...
INetworkDifficulty
ITransactionInfo

Could comment out some unused code to avoid dependency due to seemingly not being used. Flagged with "TODO" and indicated that re-factoring may be required.

More work may be required but limiting the size of this PR to these changes.

Generally the idea is that every controller lives in its feature, we will only uses interfaces when controllers are sharing a resource.
This is fine for now but controllers need to be pulled out to features at some point.


Bug in ColorSpectrum, fix attached

Hi! I've just found bug that still present even in dev branch.
Aurora.EffectsEngine.ColorSpectrum.GetColorAt(float position, float max_position = 1.0f)
calculates wrong colors if max position is not 1.0f
Here is code with fix;
       public Color GetColorAt(float position, float max_position = 1.0f)
        {
            position = CorrectPosition((position / max_position) + shift);

	    max_position = 1.0f; // Add this line here to fix the bug

			float closest_lower = 0.0f;
            float closest_higher = 1.0f;

            foreach (KeyValuePair<float, Color> kvp in colors)
            {
                if((kvp.Key * max_position) == position)
                {
                    return kvp.Value;
                }

                if((kvp.Key * max_position) > position && kvp.Key < closest_higher)
                {
                    closest_higher = kvp.Key;
                }

                if ((kvp.Key * max_position) < position && kvp.Key > closest_lower)
                {
                    closest_lower = kvp.Key;
                }
            }

            return Utils.ColorUtils.BlendColors(colors[closest_lower], colors[closest_higher], ((double)( (position / max_position) - closest_lower ) / (double)(closest_higher - closest_lower)));
        }

I don't understand what this fix is for? The color spectrum is designed so it has at least two points (0.0f and 1.0f). The parameters refer to "current position" out of "maximum position". The spectrum itself can be considered a line of different colors from 0% to 100%. So if the position is 0.4 and max_position is 2.0, then it will return a color from spectrum at 0.2 or 20%.
I guess I need to throw more exceptions :P

bugged version it will work correctly only if you use normalized points (0.0-1.0) like you do almost every where. but if you try to use for example GetColorAt(56, 100) it will return wrong color.
look here:
 if((kvp.Key * max_position) > position && kvp.Key < closest_higher)
you are comparing already normalized position  with key multiplied on not normalized max_position.
here you normalizing position:
position = CorrectPosition((position / max_position) + shift);
but you forgot to set max_position to 1f after that.
That's why I added this line:
max_position = 1.0f;
Hope now it's more clearly.

I see where you're coming from now... Yeah... I will look into it. Thanks for bringing this up!

Np!) I've discovered when my script was not working correctly. Anyway, thanks for you amazing program!


[BUG] Force delete option on nodes is not working when the cluster deletion errors out

Rancher Server Setup

Rancher version: v2.7-head 9f6a39e
Installation option (Docker install/Helm Chart): Docker install

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): k3s

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.26.4+k3s1
Cluster Type (Local/Downstream): Downstream Infrastructure provider RKE2, 2 nodes - 1 etcd + cp and 1 worker node

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: Admin

Describe the bug
Following the issue https://github.com/rancher/rancher/issues/41635, if we try to delete the nodes by using a force delete option on an rke2 node driver cluster, the nodes were never deleted from the cloud provider.
To Reproduce
to manually make the cluster go into an error state, these steps can be used to reproduce the error state https://github.com/rancher/rancher/issues/34945#issuecomment-974546498.

On rancher server v2.7-hea, create an rke2 node driver cluster with 2 nodes - 1 etcd + cp and 1 worker
Invalidate the cloud credentials used to provision the cluster by deleting the access token on the cloud provider side (e.g. Digital Ocean / AWS API token).
Delete the cluster
The cluster is stuck in deleting state
From the cluster explorer >> select the force delete option on the nodes.

Result
The request sent  https://<rancherIP>/v1/rke-machine.cattle.io.linodemachines/fleet-default/anurke2fleet2-ec-c7ba3540-lz7f7 returns 200 status but the nodes are not deleted from the backend.
Since the status is a 200, the annotation provisioning.cattle.io/force-machine-remove is set to true
Expected Result
Expected the force delete operation to go through and the nodes are deleted from the backend.
Reference issue:
https://github.com/rancher/rancher/issues/41635
Additional context:
Also, if we try to delete by using kubectl -n fleet-default edit linodemachines.rke-machine.cattle.io anurke2fleet2-ec-c7ba3540-lz7f7 in the local cluster and set the annotation provisioning.cattle.io/force-machine-remove: "true", the nodes are not removed.

QA TEST PLAN
Scenarios

Scenario
Test Case

1
As a standard user, provision an RKE2 node driver cluster (1 etcd, 1 cp, 1 worker), force delete each node after invalidating cloud credentials, validate they're removed in Rancher and on the infrastructure backend

2
As a standard user, provision an RKE2 node driver cluster (1 etcd/cp, 2 workers), force delete each node after invalidating cloud credentials, validate they're removed in Rancher and on the infrastructure backend

3
As a standard user, provision an K3S node driver cluster (1 node, all roles), force delete the node after invalidating cloud credentials, validate the node is removed in Rancher and on the infrastructure backend

Validated that this is addressed in v2.7-head https://github.com/rancher/rancher/commit/5334be362e291576c8f32b4432762399729832e6. See details below:
ENVIRONMENT DETAILS

Rancher install: Docker
Rancher version: v2.7-head https://github.com/rancher/rancher/commit/5334be362e291576c8f32b4432762399729832e6
Browser: Chrome

TEST RESULT

Scenario
Test Case
Result

1
As a standard user, provision an RKE2 node driver cluster (1 etcd, 1 cp, 1 worker), force delete each node after invalidating cloud credentials, validate they're removed in Rancher
:white_check_mark:

2
As a standard user, provision an RKE2 node driver cluster (1 etcd/cp, 2 workers), force delete each node after invalidating cloud credentials, validate they're removed in Rancher
:white_check_mark:

3
As a standard user, provision an K3S node driver cluster (1 node, all roles), force delete the node after invalidating cloud credentials, validate the node is removed in Rancher
:white_check_mark:

VALIDATION STEPS
Scenario 1

As a standard user, provisioned an RKE2 v1.26.4+rke2r1 node driver cluster (1 etcd, 1 cp, 1 worker).
On the cloud provider side, invalidated the cloud credentials that were used to provisioned the cluster.
Logged out and logged in as an admin user.
Deleted the cluster and noted that as expected, the cluster is stuck when attempting to delete itself.
Force deleted the worker node in the cluster and validated that it was removed from Rancher.

Scenario 2

Repeated scenario 1, but provisioned an RKE2 v1.25.9+rke2r1 node driver cluster (1 etcd/cp, 2 workers).

Validated that the results were the same.

Scenario 3

Repeated scenario 1, but provisioned an K3S v1.26.4+k3s1 node driver cluster (1 node, all roles).

Validated that the results were the same.


about updateButton

I want to click on the Update button to pop up a confirmation box, and When I selected "Yes"  let it do update.

Here is code using saltarelle which i have done in my project. You can do it in similar way using TypeScript.
         jQueryEventHandler vSaveClick; //Declare event variable
        jQueryEventHandler vApplyClick;
        protected override List<ToolButton> GetToolbarButtons()
        {
            var buttons = base.GetToolbarButtons();
            vSaveClick = buttons[0].OnClick; //Assign event variable
            vApplyClick = buttons[1].OnClick;

            buttons[0].OnClick = e => { ConfirmBeforeSave(vSaveClick, e); };
            buttons[1].OnClick = e => { ConfirmBeforeSave(vApplyClick, e); };

            return buttons;
        }
        void ConfirmBeforeSave(jQueryEventHandler ClickEvent, jQueryEvent e)
        {
            Q.Confirm("This will also changes status from Approved to Pending. Do you want to update the fields? ", () =>
            {
                ClickEvent(e);
            });
        }

thank you, I'll give Hello message

This method is feasible, I also realized this effect TypeScript . thank you

You are welcome @thisXYH

@ramveersgh
Have you ever tried to click a button pops up a Grid box? I discovered how to generate a Grid box, but do not know how to show it in the form of bomb box


Video remoting was disconnected

I downloaded fresh Ubuntu 18.04 iso from website, followed the instructions on the wiki to the letter and after I restart, I see the Xorg login. I enter in my username and password and then it abruptly disconnects with message "Video remoting was disconnected". If I turn off Enhanced session services, I can see the Ubuntu desktop, but any attempts to launch any app besides Settings gets a spinning circle and then killed. Username and password are fine because it gives me an error if I intentionally enter wrong password.
Running on Windows 10 Enterprise 1803, all updates applied, Hyper-V gen 2 VM. Using commit 11be5b1 (42 minutes old). Wiki instructions are also the newest instructions, less than an hour old. Also tried the jterry75 install.sh as well, same results. Even tried an old post saying disabling UDP/TCP checksum offloads.

Hey @shadowx360, if you can connect onto the VM with a normal connection (non-enhanced) a few things to check:

sudo systemctl status xrdp - What is printed here?
/var/log/xrdp.log - What is printed here?
/var/log/xrdp-sesman.log - What is printed here?

You wouldn't happen to have autologin enabled? Then it's related to #22, I had a similar experience (including inability to launch terminal)

I am getting the exact same issue as @shadowx360, with the current release of Ubuntu Server 18, and the most recent "Insider" version of Windows 10. Also tried with the most recent version of Windows 10 Pro, but no difference. Hyper-V gen2
Granted I'm no Linix guru, but I've been trying this for days to no avail. Also tried Ubuntu 16, same thing.
The xrdp service is up, and I even made user part of sudu, thinking that it was perhaps a permissions issue.
Any thoughts on next steps, @cwilhit ?
I checked:
systemctl status xrdp
/var/log/xrdp.log
/var/log/xrdp-sesman.log
And perhaps I don't what I'm looking at/what to look for, but answering the "What is printed here?" question is not simple, as there is a lot of verbage in those files.
Thanks in advance for any guidance you might have.

Check your /etc/xrdp/xrdp.ini file. Is it blank?

Chris - I checked that /etc/xrdp/xrdp.ini file, and it is definitely not blank - loads of stuff in there.

One thing I'm not clear on, is if this functionality still requires a pre-release version of Windows 10. I'm currently running on Windows 10 Pro, version 1803.

@nedotter 1803 supports the feature

Thanks, @sundhaug92  - very frustrating beating my head against the screen for about a week now, trying to get this to work.

@nedotter Understandable. Again, you wouldn't happen to have autologin enabled?

@sundhaug92 I've never enabled it - is it enabled by default? How can I best confirm if it's enabled?

@nedotter Then it should not be enabled. If you disable enhanced session support, do you have toi enter a username and password or does it just automatically log you in to the desktop?

@sundhaug92 When I disable enhanced session support, I must supply user/pass to login.

Have you tried with Ubuntu Desktop? This is the image that we've been using and I wonder if Ubuntu Server doesn't carry some bits which might be contributing to your issue.

Thanks, Chris. Will try that and report back.

@cwilhit I tried with Ubuntu desktop, and it worked - thanks for the suggestion. Most humble apologies for not getting your name right in my earlier comments (now corrected).

Not an issue :) it also took me a moment to realize you were trying with Ubuntu server instead of desktop. With your issue resolved, I will close this.

You wouldn't happen to have autologin enabled? Then it's related to #22, I had a similar experience (including inability to launch terminal)

That was my problem :) Thank you!

Make sure you are logged out of Ubuntu Desktop when you use the Enhanced Session.
If you are already logged in, there seems to be some problem with logging in again with the Xorg login.
This was the only thing that I needed to do to fix the problem- ensure I was logged out before connecting with an Enhanced Session.