Add files via upload

@nem0n I am sorry, the domain myseneca.ca is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Documentation for Holographic Remoting on HoloLens 2

TOC.md clean-up
Holographic Remoting Player page updated (holographic-remoting-player.md)
updated HoloLens 1 remoting page (add-holographic-remoting.md)
added pages for writing host, writing player, custom data channels, secure connection, and troubleshooting

Docs Build status updates of commit 0a0f9ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded

mixed-reality-docs/add-holographic-remoting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-host.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-custom-data-channels.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-secure-connection.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-troubleshooting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer-diag.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer.png
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 9ccc8e5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded

mixed-reality-docs/add-holographic-remoting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-host.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-custom-data-channels.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-secure-connection.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-troubleshooting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer-diag.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer.png
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 012604e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded

mixed-reality-docs/add-holographic-remoting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-host.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-custom-data-channels.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-secure-connection.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-troubleshooting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer-diag.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/holographicremotingplayer.png
:white_check_mark:Succeeded
View

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


Closes #87

🎯 Title and Issue Number

Closes #86

🗒 Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have gone through rules of when PR will get merged.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy-to-understand README.md.
[x] I have followed the proper file structure.
[x] I have gone through the repository guidelines.

Thank you for contributing! ❤
Don't forget to show some support by ⭐ the repository!

Hello @shuvadeepmondal, thank you for your pull request! We will review it shortly. 😇❣
Your contributions help us grow and improve our project. Please make sure you have read and followed our CONTRIBUTING GUIDELINES.
Your efforts are appreciated, and we value your participation in our community. Don't forget to ⭐ our repository!
Happy coding!✨

@Puskar-Roy Review It and merge it

Hey @shuvadeepmondal! We're excited to let you know that your pull request has been merged into the APIverse🚀🎁
Your commitment, effort, and expertise have greatly influenced our project, and we're incredibly thankful for your contributions.😇
Your code is now part of our expanding ecosystem, assisting developers worldwide in creating exceptional APIs.❣🔥
Happy Coding! ✨ We look forward to seeing you again soon!❣


Update officer-corps spirits.

Unclear why we have only two army and naval spirits, and no air ones. This just copies in the latest from vanilla.

Ok I think we're going to have to live with spelling errors from Paradox. 😀

Sorry, didn't think to check for country-specific stuff. I also removed some spirits that Pdox commented out.

The has_X_academy are all scripted triggers for ETH from BBA. They should be removed or they'll cause errors.


Added note to clarify XR plugin settings required for holographic remoting

Overview
Clarified relatively hidden step needed to ensure the XR plugin settings are correct to successfully start holographic remoting
Changes

Fixes: #8990

/azp run mrtk_docs


support test command line for windows

增加测试指令对 windows 的支持

Coverage remained the same at 8.734% when pulling ff8f9c3766054b102db7ef36966c49d4be33667a on SylvanasGone:devTestConfig into 6449832b4be08ab8d812dd891079d272104917a3 on ZhonganTechENG:dev.


Break username and domain from login on last @ instead of first @

When pulling apart the username and domain for domain checking and such, break on the last @ symbol instead of the first one.  Dovecot master usernames can look like username<EMAIL_ADDRESS>

Interesting


Unable to run licensee from user install (on Travis CI)

First of all, sorry for this kind of support request. I realize this might very well not be a bug in licensee, but as only an intermediate Rubyist I'm running out of ideas what's wrong.
I'm installing licensee version 9.9.0.beta.3 for the current user like
gem install --user-install licensee -v 9.9.0.beta.3

And them I'm trying to run licensee like
$(ruby -rubygems -e "puts Gem.user_dir")/bin/licensee version

As you can see in this Travis CI build, it fails with
/home/travis/.rvm/rubies/ruby-2.4.1/lib/ruby/site_ruby/2.4.0/rubygems.rb:271:in `find_spec_for_exe': can't find gem licensee (>= 0.a) (Gem::GemNotFoundException)
    from /home/travis/.rvm/rubies/ruby-2.4.1/lib/ruby/site_ruby/2.4.0/rubygems.rb:299:in `activate_bin_path'
    from /home/travis/.gem/ruby/2.4.0/bin/licensee:23:in `<main>'
    from /home/travis/.rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `eval'
    from /home/travis/.rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `<main>'

It works on a local Linux machine I have access to, however.
Any idea what's going wrong on Travis CI?

@sschuberth have you tried using bundler, adding it to your Gemfile, and prepending bundle exec?  If you look at the backtrace above, it seems to be installing Licensee to Ruby 2.4.0, but then trying to run it in 2.4.1.

Well, I sort of know it works with bundler, but I'd like to avoid using it as my use case is different: I'm not using licensee as a dependency in another Ruby project, but straight as command line tool. Thus I'd like to be able to install it straight via gem instead of creating an "artificial" Gemfile and be using bundler.
Good point about the Ruby versions, though, I'll dig into that!

@benbalter looks like the 2.4.1 vs 2.4.0 is normal. The Gem.user_dir seems to zero-out the patch level so Ruby versions that only differ in the patch level can use the same gems. Any more ideas?

Haven't seen this before, but two other ideas from Googling:

sudo gem install licensee
gem update --system before the install command

The latter I also already tried, it has no effect as gem itself is already up-to-date. The former basically works, but for portability reasons I need to install licensee per user, not in the system.


crate db docker container crashes

I tried running './service start' command from the main directory to run the services. However, I observed that the Crate DB image crashes almost immediately.
Things I tried to sort out -

Changed image tag to latest
Removed the command section to see if the image runs correctly or not.
Changed image tag to previously stable version.
However, it is observed that the Crate DB image runs correctly in stand alone. Please help me sort out the issue.
Thanks and Regards

If just pulled and ran as follows - I can see no errors:
 git status

On branch NGSI-v2
Your branch is up to date with 'origin/NGSI-v2'.

nothing to commit, working tree clean

 git rev-parse HEAD

678bed961be0069db4a5b64beca10dd2e29d62f1

Check that you have the right git hash, then pull the containers:
./services create

Pulling Docker images
Pulling mongo-db    ... done
Pulling iot-agent   ... done
Pulling orion       ... done
Pulling crate-db    ... done
Pulling quantumleap ... done
Pulling tutorial    ... done
Pulling grafana     ... done

Now start:
 ./services start

Stopping containers
Starting seven containers Orion, QuantumLeap, IoT-Agent, Tutorial, a Grafana metrics dashboard and CrateDB and MongoDB databases.
- Orion is the context broker
- QuantumLeap will write to CrateDB
- Grafana will read from CrateDB
- IoT-Agent is configured for the UltraLight Protocol
- Tutorial acts as a series of dummy IoT Sensors over HTTP

Creating db-mongo ... done
Creating db-crate         ... done
Creating fiware-iot-agent   ... done
Creating fiware-orion       ... done
Creating fiware-quantumleap ... done
Creating grafana            ... done
Creating fiware-tutorial    ... done
Adding appropriate MongoDB indexes for Orion  ... done
Adding appropriate MongoDB indexes for IoT-Agent  ... done

⏳ Waiting for Orion to be available

⏳ Loading context data  done

⏳ Waiting for IoT-Agent to be available

IoT Agent HTTP state:  000  (waiting for 200)
⏳ Provisioning IoT devices  done

NAMES                STATUS                             PORTS
fiware-tutorial      Up 17 seconds (health: starting)   <IP_ADDRESS>:3000-3001->3000-3001/tcp
fiware-quantumleap   Up 19 seconds (health: starting)   <IP_ADDRESS>:8668->8668/tcp
fiware-iot-agent     Up 19 seconds (health: starting)   <IP_ADDRESS>:4041->4041/tcp, <IP_ADDRESS>:7896->7896/tcp, 4061/tcp
fiware-orion         Up 19 seconds (health: starting)   <IP_ADDRESS>:1026->1026/tcp

Now open http://localhost:3000/device/monitor

When running these statements the docker containers are starting fine for me.
To diagnose further you'll need to post a log from the failing docker container. e.g. docker logs db-crate

Also, sometimes docker needs clean up to work, try:
docker system prune
docker network prune
docker volume prune

to free up resources.

I followed the instructions as per your answer. Everything goes fine as shown....
However, when I try to insert time-series data into Crate DB, (QuantumLeap API - List the first N Sampled Values - https://github.com/FIWARE/tutorials.Time-Series-Data#quantumleap-api---list-the-first-n-sampled-values).
The request returns the error -
"Something went wrong with QL. Error: No more Servers available, exception from last server: HTTPConnectionPool(host='crate-db', port=4200): Max retries exceeded with url: /_sql?error_trace=true (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f9d99f222b0>: Failed to establish a new connection: [Errno -2] Name does not resolve'))"
Then when I saw the running containers
docker ps -a
Then I found that the Crate DB crashed -
CONTAINER ID        IMAGE                               COMMAND                  CREATED             STATUS                             PORTS                                                      NAMES
60a26b8307be        fiware/tutorials.context-provider   "/nodejs/bin/node ./…"   17 seconds ago      Up 16 seconds (health: starting)   <IP_ADDRESS>:3000-3001->3000-3001/tcp                           fiware-tutorial
49e25d830dd6        smartsdk/quantumleap:0.7.6          "/bin/sh -c 'python …"   18 seconds ago      Up 16 seconds (health: starting)   <IP_ADDRESS>:8668->8668/tcp                                     fiware-quantumleap
f100670d221e        grafana/grafana:6.1.6               "/run.sh"                18 seconds ago      Up 16 seconds                      <IP_ADDRESS>:3003->3000/tcp                                     grafana
4e2ba6836fe1        fiware/orion:2.4.2                  "/usr/bin/contextBro…"   18 seconds ago      Up 17 seconds (health: starting)   <IP_ADDRESS>:1026->1026/tcp                                     fiware-orion
7da6d403a80a        fiware/iotagent-ul:1.14.0           "docker/entrypoint.s…"   18 seconds ago      Up 17 seconds (health: starting)   <IP_ADDRESS>:4041->4041/tcp, <IP_ADDRESS>:7896->7896/tcp, 4061/tcp   fiware-iot-agent
d4fcbd8e5690        mongo:3.6                           "docker-entrypoint.s…"   19 seconds ago      Up 18 seconds (health: starting)   <IP_ADDRESS>:27017->27017/tcp                                   db-mongo
f37a7e509f4c        crate:3.1.2                         "/docker-entrypoint.…"   19 seconds ago      Exited (78) 5 seconds ago                                                                     db-crate

Thus the container is created, but its crashing immediately.
PS : I did docker prune network, system and volume before proceeding. Still the error persists.
Please help me out. Thanks

Well, that QuantumLeap response error message looks fine, because there is no CrateDB running. :smile:
Since CrateDB is failing for you,  you'll need to diagnose the log from the failing docker container. e.g. docker logs db-crate. However that would be a Crate-DB error, not a tutorial failure.

Figured out the issue from the logs
ERROR: [1] bootstrap checks failed
[1]: max virtual memory areas vm.max_map_count [65530] is too low, increase to at least [262144]

sorted out the by using the command -
sudo sysctl -w vm.max_map_count=262144

Closing the issue here.
Thanks a lot 👍 :)

Dear Utkarsh and Jason,
thanks for sharing this. https://github.com/crate/crate/issues/6293, https://github.com/crate/crate/issues/8227, https://github.com/crate/crate/issues/8338 and https://github.com/crate/crate/issues/8930 talk about the same issue.
Unfortunately, this setting has to be applied to the host system, so there is no way to fix that within the Docker container itself. Others have been struggling with this as well, see for example https://github.com/docker-library/elasticsearch/issues/133, https://github.com/sonarsource/docker-sonarqube/issues/279 and https://github.com/docker/for-win/issues/5202.
The respective documentation for Elasticsearch and CrateDB can be found at [1] and [2]. If the limit cannot be adjusted on the host system, the memory map limit check can be bypassed by passing the -Cnode.store.allow_mmapfs=false option to the crate command as documented at [3]. However, this will result in degraded performance.
@jason-fox and @fisuda: Can I humbly suggest to put a short comment about this specific issue into the README.md and README.ja.md files and link to the respective documentation of CrateDB [2,3] in order to improve the experience for users not aware about these technical details?
Apart from that, I would like to suggest to add the CRATE_HEAP_SIZE=2g environment variable setting to the crate-db section within docker-compose.yml, see also [3].
With kind regards,
Andreas.
cc @seut
[1] https://www.elastic.co/guide/en/elasticsearch/reference/current/docker.html#_set_vm_max_map_count_to_at_least_262144
[2] https://crate.io/docs/crate/howtos/en/latest/admin/bootstrap-checks.html#bootstrap-checks
[3] https://crate.io/docs/crate/howtos/en/latest/deployment/containers/docker.html#troubleshooting


Bluetooth device list shows duplicates in IoTCoreDefaultApp

Connect using Windows IoT Remote Client (Preview) to IoTCoreDefaultApp in system using CSR 4.0 Bluetooth adapter.
Click on Settings and then on Bluetooth. Turn Bluetooth on. Observe list of devices able to pair.
Click the <- (back) button. Click on Settings again. Observe that the same devices are listed, but now they are listed twice.
Repeat #3 multiple times. Each time it is repeated more duplicates of the same devices are shown.

Hi mjmeans,
Can you please confirm the devices involved?
Sincere thanks,
IoTGirl

Raspberry Pi 3 with CSR 4.0. The Bluetooth devices in question are: "[TV] Samsung LED46" which is actually a Samsung TV; and the other device is a laptop (Dell Inspiron 7537) which has built-in Bluetooth. I am available for a remote screen connection if you need to actually see it happen.

Hi mjmeans,   I have asked a member of the IoTCore Team to take a look. Sincerely, IoTGirl

Fixed


GEODE-9712: genericize netcore

This brings in support for our existing IRegion<TKey, TValue> interface with the following restrictions:
TKey currently limited to string, short, and int.
TValue currently limited to string only.
SessionState tests currently failing. Not sure why yet.
Note:
Leaving as draft at least until SessionState issue figured out.

Please disregard my previous PR for GEODE-9712. It had merge conflicts due to erroneous branch being created on apache.

With the decision to go with a pure C# client for .NET, we no loner need this interop based approach.


Documentation link on homepage is incorrect

Checklist

Please ensure the following tasks are completed before filing a bug report.

[x] Read and understood the Code of Conduct.
[x] Searched for existing issues and pull requests.

Description

When the documentation link in the header of the homepage is clicked it takes the user to https://stdlib.io/develop/docs/api/@stdlib/ , which is a directory index not an actual web page.

Related Issues

Does this issue have any related issues?

No
Questions

Any questions for reviewers?

No.
Other

Any other information relevant to this issue? This may include screenshots, references, stack traces, sample output, and/or implementation notes.

Demo

If relevant, provide a link to a live demo.

For a live demo of the issue, see

https://stdlib.io/ and click documentation in the top menu bar
It will take you here: https://stdlib.io/develop/docs/api/@stdlib/

Reproduction

What steps are required to reproduce the unexpected output?

In order to reproduce this bug, do the following:

Visit https://stdlib.io/
click documentation

Expected Results

What are the expected results?

I think it should direct the user to this url: https://stdlib.io/develop/docs/api/@stdlib/stdlib
Actual Results

What are the actual results?

It will take you to a directory listing: https://stdlib.io/develop/docs/api/@stdlib/
Environments

What environments are affected (e.g., Node v0.4.x, Chrome, IE 11)? If Node.js, include the npm version, operating system, and any other potentially relevant platform information.

The following environments are affected:

Chrome Version 72.0.3626.109 (Official Build) (64-bit)

same

We recently launched new API docs! Still a couple bugs to fix which we are working on at the moment, but should be accessible now. See https://stdlib.io/docs/api.


Running on a Mac

Hello,
I've been using the scan repo binary on a Linux box but now have a requirement to run it on a Mac as well.
Will the code run on a Mac or is it Linux only?

Yes it will. If downloading a precompiled binary then make sure you choose the -darwin-amd64.tar.gz version from the releases page.
If installing with go then a go get github.com/UKHomeOffice/repo-security-scanner will work on any platform.

fasntastic thank you


get_offset: more robust casting to int

Expands the validation of integers to include (for instance) numpy.int64.  Raises a ValueError if the type cannot be cast

Not switching to try/except for this library. You can use on your local copy if preferred.


soc: posix: inf_clock: add C++ support to NATIVE_TASK

Explicitly define function pointer as the correct type (void*)() and not (void*). In C this cast is done implicitly, but C++ does not allow it.
A minimal recreation for reference:

C: https://godbolt.org/z/cjvnoWxc5
C++: https://godbolt.org/z/v63Mvqh6s

Signed-off-by: Hein Wessels<EMAIL_ADDRESS>

Note the clang failure. (& for the edtt_stop change, you need to leave the return in the if, and would  need to change the proto in the header)

I think clang is about the const being in the wrong place.  @andyross  you added it to apease clang for sanitising, but should it have read static const void (*__native_task_fun)() instead

I think clang is complaining about the const being in the wrong place.

Nice catch. I've fixed it as you proposed for now as it does seem to be the problem. Let's see if clang still complains.


RFC: README.md, WIP: REQUIRE

Updated README.md. Added REQUIRE file, QuBase needs to be added, which needs to be registered. Any pointers on this would be helpful.
Edit : Related PR https://github.com/JuliaLang/julia/issues/7737, I guess we could register QuBase.jl as I guess there is no support for links in REQUIRE.

I guess it will take some more work before we register QuBase.jl. I would just state in the README that one needs to clone this repo and QuBase.jl to get started.

@acroy I hope this is fine. Please do let me know if this can be merged.

I think it is fine and I guess you can merge it yourself, right?

Yeah I can merge this

Go for it :-)


[BUG] Installing 7-zip 64bit (automatically picked by WingetUI) after restart of system changes to 32bit as I do not see context menu

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
Installing 7-zip 64bit (automatically picked by WingetUI) after restart of system changes to 32bit as I do not see context menu
Steps to reproduce the issue

WingetUI Log
Operation: installation - Perform date 2023-09-28 20:48:34.266505
 Package: <Package: 7-Zip;7zip.7zip;23.01;Winget: winget;<PackageManagers.winget.WingetPackageManager object at 0x00000192DB42FD90>;<Interface.CustomWidgets.GenericWidgets.TreeWidgetItemWithQAction object at 0x00000192DC5E40C0>>
 Command-line call: "C:\Users\reddy\AppData\Local\Programs\WingetUI\PackageManagers\winget-cli_x64\winget.exe install --id 7zip.7zip --exact --accept-source-agreements --disable-interactivity --accept-package-agreements" Output code: 0
 Console output:
   | -
   | \
   | |
   | ███▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  1024 KB / 7.98 MB
   | ███████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  2.00 MB / 7.98 MB
   | ███████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  3.00 MB / 7.98 MB
   | ███████████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒  4.00 MB / 7.98 MB
   | ██████████████████▒▒▒▒▒▒▒▒▒▒▒▒  5.00 MB / 7.98 MB
   | ██████████████████████▒▒▒▒▒▒▒▒  6.00 MB / 7.98 MB
   | ██████████████████████████▒▒▒▒  7.00 MB / 7.98 MB
   | ██████████████████████████████  7.98 MB / 7.98 MB
   | Found 7-Zip [7zip.7zip] Version 23.01
   | This application is licensed to you by its owner.
   | Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
   | Downloading https://www.7-zip.org/a/7z2301-x64.msi
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | -
   | \
   | |
   | /
   | ████████████████▒▒▒▒▒▒▒▒▒▒▒▒▒▒  1024 KB / 1.84 MB
   | ██████████████████████████████  1.84 MB / 1.84 MB
   | Successfully verified installer hash
   | Starting package install...
   | The installer will request to run as administrator, expect a prompt.
   | -
   | \
   | Successfully installed
   |

Package Managers Logs
Name                                          Id                                            Version             Source
-----------------------------------------------------------------------------------------------------------------------
7-Zip 23.01 (x64 edition)                     7zip.7zip                                     <IP_ADDRESS>

Relevant information
No response
Screenshots and videos
No response

Sorry, False alarm! Closing the ticket


Support for system-assigned and user-assigned managed identities

Description of your changes

Fixes #164, #292, #319
This PR adds support for system-assigned and user-assigned managed identities, formerly known as Managed Service Identities (MSIs) via the Azure Identity client module.
An example ProviderConfig that uses a system-assigned managed identity is as follows:
apiVersion: azure.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
  name: example
spec:
  credentials:
    source: InjectedIdentity
  subscriptionID: <Azure subscription ID>

Another example using a user-assigned managed identity:
apiVersion: azure.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
  name: example
spec:
  credentials:
    source: InjectedIdentity
  subscriptionID: <Azure subscription ID>
  clientID: <Client ID of the user-assigned managed kubelet identity>

I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

I have successfully provisioned a ResourceGroup and an AKSCluster using a system-assigned managed identity and also a user-assigned managed identity with the ProviderConfigs given above.

I tested the PR with user-assigned managed identity with following steps:

Created an identitiy

az identity create --name ezgiidentity --resource-group ezgi --location westeurope

Assigned Managed Identity Operator and Contributor roles to it. Seeing these role assignments' effects took some time.
Created an AKS cluster and set its identity and its kubelet's identity to the managed one above

 az aks create --resource-group ezgi --name ezgi-cluster --enable-managed-identity --assign-identity $ID --location westeurope

Installed crossplane

up uxp installed -n crossplane-system

Installed provider built from this branch

kc install provider ulucinar/provider-azure:build-3eacbce2

Created a provider config with user-managed identity ID

apiVersion: azure.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
  name: example
spec:
  credentials:
    source: InjectedIdentity
  subscriptionID: <sub-id>
  clientID: <client-id>

Successfully created a resourcegroup with example manifest
Successfully created subnet, virtual network with example manifets
Failed to create an AKS cluster with following error

Warning  CannotCreateExternalResource  3m16s (x340 over 28m)   managed/akscluster.compute.azure.crossplane.io  cannot create AKSCluster: containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="MustDefineAtLeastOneSystemPool" Message="Must define at least one system pool."

While preparing a cluster with user-managed identity, I noticed that kubelet's identity should also be set. Following curl command helped me to troubleshoot. I created a pod with curl binary in the cluster and executed it inside it:
curl 'http://<IP_ADDRESS>/metadata/identity/oauth2/token?api-version=2018-02-01&resource=https%3A%2F%2Fmanagement.azure.com%2F' -H Metadata:true -s

@turkenh I have a client who would like to use Manage Identities.  I would be happy to try and help fix whatever CodeCov is saying is wrong but I do not have access to see what the problem is.  Can you let us know what it is or help to get it addressed so that we can all start using these amazingly helpful changes from @ulucinar?

any estimate on this feature delivery? it would be really useful

I would also love to see this PR merged because it would simplify our current setup using Crossplane with Service Principals a lot!

Any chance we can get this merged?

Any ETA here? Hoping this helps me automate giving the role "DNS Zone Contributor" to my AKS clusters so external-dns will work.

I'm looking at implementing this provider. The feature you've introduced aligns perfectly with our current needs. Thank you for your valuable work @ulucinar !
I see this was approved on May 9, 2022 but is still not merged because of codegov check.
@turkenh / @ezgidemirel if possible, could you revisit this matter for a second evaluation? The report is no longer available, and a previous user mentioned that external users don't have access to it. Your assistance in this matter would be greatly appreciated.

Any updates? Would love to see this feature implemented


ImportError: cannot import name 'inference_detector_huge_image' from 'mmdet.apis' (/home/avantika/workspace/OBBDetection/mmdet/apis/init.py)

This error is coming when i use the following command for inference :
python huge_image_demo.py demo/demo.jpg configs/obb/oriented_rcnn/faster_rcnn_orpn_r50_fpn_1x_dota10.py  checkpoints/faster_rcnn_orpn_r50_fpn_1x_dota10_epoch12.pth BboxToolkit/tools/split_configs/dota1_0/ss_test.json
I am using python 3.9 and mmcv version 1.6.2 and getting this error.
How can I resolve this?

请问你有解决吗？

I want to know if you have solved this problem

The mmdet that the author uses is the mmdet that he added and modified, not the official mmdet module. So you need to download the author's mmdet source from github and install it. I hope it helps.


Feature: add a ghost mode

Hi,
Im migrating my ui to your plugin 👌
One missing feature is ghost mode, having a chat in ghost mode delete all his messages after being seen..
Thx

Im migrating my ui to your plugin 👌

Congrats! As of today it's by far the most complete chat kit, I was thinking of using https://pub.dev/packages/dash_chat but sadly they didn't deliver their v2.

One missing feature is ghost mode, having a chat in ghost mode delete all his messages after being seen..
Can you elaborate? I didn't quite understand how it works.

Im migrating my ui to your plugin 👌

Congrats! As of today it's by far the most complete chat kit, I was thinking of using https://pub.dev/packages/dash_chat but sadly they didn't deliver their v2.

One missing feature is ghost mode, having a chat in ghost mode delete all his messages after being seen..
Can you elaborate? I didn't quite understand how it works.

Sure.
When creating a new chat, creator can choose a ghost mode (inspired by snapchat).
In that mode, when a message is posted by someone, and read by all other members: the message is deleted forever
.
Writing this, i realized, its not the responsibility of the lib to handle this feature: its not UI, its a business case :)
forget about this feature

Sure.
When creating a new chat, creator can choose a ghost mode (inspired by snapchat).
In that mode, when a message is posted by someone, and read by all other members: the message is deleted forever
.
Writing this, i realized, its not the responsibility of the lib to handle this feature: its not UI, its a business case :)
forget about this feature


set tracking url template as pointer, as we may need empty string to …

…remove the field on update

OK


Enable auto-provision of Kafka when using "streaming" strategy

When provisioning the Elasticsearch storage, if the server url is not defined (and running on OpenShift), the Jaeger operator will attempt to auto-provision the Elasticsearch storage using the elasticsearch-operator.
The same capability should be provided when the "streaming" strategy has been selected, and the Kafka topic (and/or broker) has not been configured for both consumer and producer.
Unlike the ES auto-provisioning - it may be possible to also perform this when running on K8S - we need to confirm that the same CRD is used in upstream strimzi and downstream AMQ.

AMQ documentation shows this as a CR example:
apiVersion: kafka.strimzi.io/v1alpha1
kind: Kafka
metadata:
  name: my-cluster

Kafka topic (and/or broker) has not been configured for both consumer and producer.

My preference would be just based on the broker - as this is equivalent to the ES server URL. We should still enable the user to specify the topic they wish to use, regardless of how the kafka cluster is provisioned.

Implemented in https://github.com/jaegertracing/jaeger-operator/pull/713


docs: fix build

At some point "private": true became "private": "true" and whether or not a package is private means that pnpm seems to do different things when installing, running scripts, etc.
When we move back to true boolean (rather than "true" string), this issue becomes apparent in Vercel (see the failed preview deploy after the first commit in this PR).
To work around this, I've added a blank pnpm-workspace.yaml to the docs directory so that pnpm treats it as its own workspace (with no sub-packages) and installs work properly again.
I also had to add missing dev deps (since they were inferred from the root originally).
I opted to take this path instead of including this in the root workspace because I encountered issues when trying to figure out the build pipelines, especially for other monorepo-like directories (templates, examples), but I think we can safely return this to the workspace in the future once we finish sorting out the build clean up.

ooc would it help to add docs to the top level pnpm-workspace.yaml instead of making it its own workspace? (Noticed it's not listed there)
If not, happy to merge this

I left a note about this in the PR description:

I opted to take this path instead of including this in the root workspace because I encountered issues when trying to figure out the build pipelines, especially for other monorepo-like directories (templates, examples), but I think we can safely return this to the workspace in the future once we finish sorting out the build clean up.

because I encountered issues when trying to figure out the build pipelines, especially for other monorepo-like directories (templates, examples)

Do you have some more context on this?
Should we add this to https://github.com/latticexyz/mud/issues/673?


SVGs for mobile buttons

i dont know how to make pull requests so i'll just submit this issue and see if you'll add this.
the svg buttons are for every theme that doesn't use a pixel art-style.
if requested, there will be a follow-up comment for other buttons that will be vectorized.

All right, I'll add these, what name do I put in the credits for making these?

wrong button

All right, I'll add these, what name do I put in the credits for making these?
just put "wdymLmao_" in the credits

for the bottom left

I already did those

the high quality ones correct?

yeah

is it drastically different than the ones i'm proposing?

I mean they're already in the mod so you can look for yourself

they're pixelated

what im proposing are for the non-retro themes

Indeed they are for non-retro themes

i guess it'll do

Updated designs for Insert Channel Below, Duplicate Selected Pattern, and Select All


Move koji_promote to main package

Coverage remained the same at 87.478% when pulling 7bb90f4f8aafa8b24d036650cf6505229498b04b on koji-promote-packaging into b360fd57b18b65546338b837a900dffe999de138 on master.

Coverage decreased (-0.7%) to 86.79% when pulling ebeea780762f87dd3df103269b26157783b5462d on koji-promote-packaging into b360fd57b18b65546338b837a900dffe999de138 on master.


[Migration] Support importing XSK projects from a zip

Currently, the migration perspective supports only connecting to a HANA 1 instance and migrating delivery units from it. In some cases, there might be no way to connect to such an instance and we should support manually importing a zip having an XS project. This would also make it easier for local development.
Frontend changes:
We should implement some additional screen before the one for entering Neo credentials where the user would decide if he wants to migrate a project from a HANA 1 instance or from a local zip. It would probably be sufficient to have two radio buttons:

Migrate a project from a local ZIP archive
Migrate a project from a live HANA instance
Based on the user input, we would either go to the current Neo credentials screen or a new one having a drag n drop file chooser similar to the one used in the Dirigible for importing projects in a workspace.
The drag n drop screen should also have the same workspace selection dropdown as in the other migration screens.

Backend changes:
The Flowable process should also be modified in order to support this functionality. We would need to "bypass" the tasks for connecting to the HANA and fetching content from it.
Please, contact me when the frontend changes are done in order to discuss thoroughly the backend changes and split the work if necessary.

Please, assign it to me


Reorganize bug template for internal reporting (closes cwa-documentation#470)

Adds 'app version' to the list of requested technical details (https://github.com/corona-warn-app/cwa-documentation/issues/477) and restructures bugreport sections to be more in line with EXPOSUREAPP-3710.

Internal Tracking ID: EXPOSUREAPP-3710

@marcussc Thanks. Fixed in https://github.com/corona-warn-app/cwa-app-ios/pull/1666 . Closing this PR then.


fix: resolve issues with yarn (remove npm)

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


[BUG]

Stuck on Grant Permission screen on Samsung Galaxy S9 running Android 10 security patch level June 1, 2021

Possible duplicate of #16 #9 and #7.


[5.7] Add command to generate controllers from routes list

The idea behind the command is to make the development as fast as possible.
Having in the following routes
Route::get('/', 'HomesController@index');

After the command
php artisan generate:controller
A App\Http\Controllers\HomesController Controller will be created for you directly based on the routes you have.
The same as the event:generate command.

Hello @taylorotwell, Thank you for your kind feedback, Actually I don't think it corrects a bug since its new feature if I can say that.
I still think that it touches nothing around the framework, and to be honest I don't think its a good idea to make it as a package.
Dev will be happy to have a command that can help them generate the Controllers.
Thanks in advance.

There are a few existing Laravel generator packages that this world be a perfect addition to.

Hello @taylorotwell Can you consider a merge :D

@akiyamaSM Taylor usually doesn't replies to old prs. It's best that you create an issue in the ideas repo first or ping him on discord when he's online.

This kind of functionality is great for a package. IMO it doesn't belong in the core.

@devcircus agreed

@driesvints @devcircus Thanks for the guid, I've just used it as a package
https://github.com/akiyamaSM/laravel-controllers-generator


Remove rebuild

It's about time

I may be too late to the party, but one useful usage of rebuild that I've taken advantage of in the past was to build unikernels in Reason.
I didn't really use reuild itself, but pasted the contents of the myocamlbuild.ml file in the auto-generated unikernel file with the same name.

@anmonteiro Does dune support unikernel builds yet?

@hcarty It's been a year since I last tried; I haven't kept up with Dune, so can't answer that one.

I think the mirage folks have put a fair amount of testing and effort into it. Dune also has Reason support out of the box so it may provide a simpler build environment overall.


Do not throw when there are less unique positions than required for rendering.

For #2375. Geometry.createGeometry can now return undefined. If it does return undefined, it adds nothing to the batched geometry/draws nothing.

Update CHANGES.md.

@mramato This is ready for another look. I updated CHANGES.md, but I wasn't sure if this should go in the Breaking Changes section or not.

This all looks good to me, I wouldn't count this as a breaking change; more like an improvement in expected behavior. @pjcozzi did you want to look at this again?  If not, feel free to hit merge.

I agree with @mramato about the breaking change.


Use getCanGoBack selector in ModalSearchNavBar

We don't need to use the getNav selector in ModalSearchNavBar
as we use it only to check if we can go back which is the purpose
of getCanGoBack selector.
This is good for maintainability but also helps with a potential
future removal of the Redux-React Native integration. This is the
last component that directly relies on the whole nav state.
The other is AppWithNavigation but it serves only for Redux
integration.

Thanks, merged!
I made just small tweaks in the commit message; the biggest was that it's nearly the "last component" that did this, as the next paragraph points out there is one more. :wink:


Storybook sass compile error

I'd look at services/web-app/.storybook/main.js. We'll probably have to add a config.resolve to handle the ~ in @use statements. Or, don't use ~ and use a path to that file in the use statements.

@alisonjoseph ah! Nevermind. I see you just did the later in that PR.


Bug: Agent tests fail ocassionally due to a missing mock

What were you trying to achieve?
Run unit tests
What are the expected results?
Unit tests pass
What are the received results?

Steps To Reproduce
No response
In what environment did you encounter the issue?
github
Additional information you deem important
No response

add test clean up on algos as well

and there is a race condition as well on Data method


DASHBUILDE-96: DB2 does not always support the LIMIT clause (0.3.x)

Backport of the DASHBUILDE-96 fix
@romartin Can you review & merge please?

+1

0.3.x: c92bfbdd3a7bdd9fdda9f3b7067ca282789483e6


fixing comments for doc build

@tdunn please review/merge. A couple of small fixes to address some doc build warnings

All good!


[WIP] De-sing

git ls-files | xargs -n1 gsed -i 's/Desing/Design/g'
git ls-files | xargs -n1 gsed -i 's/desing/design/g'
git mv desing-tokens.sketchplugin design-tokens.sketchplugin

Oops, just a sec, noticed some paths are messed up here...


Add a new command to add a composer repository.

Add a new cdev command that will wrap the
composer config repositories.repo-name vcs https://github.com/<orgname or username>/repo
This is so that new packages are allowed to define use repos that are not on packagist.

This has been added and pushed up.


Add GUI for Cloud Service enable/disable

Add button into Host view to enable/disable CloudService having it's SystemService.
GUI for Openstack nova scheduler enable/disable functionality introduced in #7996.

Screenshots

@miq-bot remove_label wip

Hi, may I ask for review @martinpovolny @mzazrivec?

@aufi, @tzumainn is this for darga?

@blomquisg I think it's a 'nice to have' but maybe not required?  Doing this helps prevent the corner case where a node is being scaled down but nova schedules an instance onto it at the same moment.

@blomquisg : looks good for me, please merge and mark for darga as you see fit


fix(web/css): add success css for showing AC

Add css for class success.
The AC problems are shown in the problemset page now.

OK, I will close this.


re:library

Source name
Re:library
Source link
https://re-library.com
Language
English
Other details
Biggest translator of Gender Bender works.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

done in https://github.com/LNReader/lnreader-plugins/pull/936


Add name to property schema object

Adding name to property schema object for this update.

@pllnk what do you mean required in requests?
PropertyBase is used in Property which is part of the response.


Help on stack trace error when no log specified

Hi,
Probably pursuant to an update, I am.now getting the below error:
[~] # goaccess --no-global-config -o /share/Web/goaccess/index.html

==19743== GoAccess 1.3 crashed by Sig 11
==19743==
==19743== STACK TRACE:
==19743==
==19743== 0 goaccess(sigsegv_handler+0x13e) [0x40d90e]
==19743== 1 /lib/libpthread.so.0(+0x106f0) [0x7f217b56a6f0]
==19743== 2 /lib/libc.so.6(fclose+0x4) [0x7f217b21ef54]
==19743== 3 goaccess(get_config_file_path+0x6c) [0x41d68c]
==19743== 4 goaccess(parse_conf_file+0x32) [0x41d952]
==19743== 5 goaccess(main+0xa6) [0x4091e6]
==19743== 6 /lib/libc.so.6(__libc_start_main+0xf0) [0x7f217b1d7810]
==19743== 7 goaccess(_start+0x29) [0x40a949]
==19743==
==19743== Please report it by opening an issue on GitHub:
==19743== https://github.com/allinurl/goaccess/issues

GoAccess is compiled as follows:
[~] # ldd /opt/GoAccess/bin/goaccess
        linux-vdso.so.1 (0x00007ffc2c52b000)
        libnsl.so.1 => /lib/libnsl.so.1 (0x00007fd53d77d000)
        libtinfow.so.6 => /opt/GoAccess/lib/libtinfow.so.6 (0x00007fd53d550000)
        libncursesw.so.6 => /opt/GoAccess/lib/libncursesw.so.6 (0x00007fd53d310000)        libmaxminddb.so.0 => /opt/GoAccess/lib/libmaxminddb.so.0 (0x00007fd53d10b000)
        libcrypto.so.1.1 => /opt/GoAccess/lib/libcrypto.so.1.1 (0x00007fd53cc44000)        libssl.so.1.1 => /opt/GoAccess/lib/libssl.so.1.1 (0x00007fd53c9b5000)
        libpthread.so.0 => /lib/libpthread.so.0 (0x00007fd53c798000)
        libc.so.6 => /lib/libc.so.6 (0x00007fd53c3f5000)
        libm.so.6 => /lib/libm.so.6 (0x00007fd53c0f0000)
        libz.so.1 => /opt/GoAccess/lib/libz.so.1 (0x00007fd53bed5000)
        libdl.so.2 => /lib/libdl.so.2 (0x00007fd53bcd1000)
        /lib64/ld-linux-x86-64.so.2 (0x00007fd53d995000)

The permissions on index.html are the following:
[/share/Web/goaccess] # stat -c "%a %n" index.html
644 index.html

What could it be? Could someone please help me out with this error?
Thanks

It seems to be linked to the fact that I did not specified a log file in this case.

Thanks for reporting this. Are you able to reproduce it? I tried with the command you posted above but I do not get any segfaults.

I think it was because I was not pointing to any log to chew. '-)


Added isIPRange validation class

Currently this class only supports IPv4.
This class does not convert ip ranges to the base number (i.e. <IP_ADDRESS>/25 does not become <IP_ADDRESS>/25 by extension <IP_ADDRESS>/24 does not become <IP_ADDRESS>/24).
This class is just intended to validate the IP range format.  It is not intended to validate that the range matches your network configuration.  That should be checked against your IPAM solution.

I believe there is a bug in here when subnet will get passed in and cause a ReferenceError.  I'm trying to write a test that exposes that bug.  I will re-request when I have that problem solved.


Viewing hazelcast map causes memory overflow

Hello, we are using your open source hazelcast-monitor and encountered an OutOfMemory error. We have a very large IMap cache with a data volume of about 10 million. In hazelcast we have configured a maximum capacity of 2 million, but when querying the map data through hazelcast-monitor, it will cause memory overflow. .
After reading the code, I found that all the data was loaded during the query. I want to confirm whether this is the reason.

Hello,
yes the monitor will indeed pull each entry of the Map and apply the filtering predicate.
In your case I suspect that there is no filter so all 2 million entries are added in the resulting list, causing the OutOfMemoryError.
At the time of writing, I was using Hazelcast 3.9.3 which didn't have any facility to do this type of things, so I chose to implement this manual filter. However, even with enough memory and/or a restrictive filter, I would expect very poor performance on such a large Map, let alone the fact that the query is executed periodically.
A possible improvement would be to upgrade Hazelcast to at least version 3.12 which introduces paginated distributed queries with SQL syntax (see Hazelcast Docs: Distributed Query, and Hazelcast Docs: Querying in Collections and Arrays), which I expect to perform much better.
Last but not least, please keep in mind that hazelcast-monitor is intended as an aid during development rather than a battle-tested tool for production monitoring.

Thank you very much for your answers！
The monitor tool uses true by default to filter data. It is best to add a limit to the number of existing implementations to avoid possible problems


Fix error when radio title is missing

This fixes an assumption that there will always be a title element when parsing radio-like stations (e.g., anything without a specific duration).
Example traceback:
    track_info = self.soco.get_current_track_info()
  File "/usr/local/lib/python3.9/site-packages/soco/core.py", line 1813, in get_current_track_info
    track.update(_parse_radio_metadata(metadata))
  File "/usr/local/lib/python3.9/site-packages/soco/core.py", line 1803, in _parse_radio_metadata
    if title in track["uri"] or title in urllib.parse.unquote(track["uri"]):
TypeError: 'in <string>' requires string as left operand, not NoneType

LGTM -- thanks. This fix is now released in v0.25.3.


391 seed for saksdokumenter

Adds seed script for saksdokumenter
Based on branch from #257, merge this first to get rid of all the commits by mathias
Currently does not link to a document, could add this if needed just to see if download is working.
Er altså bare backend/root/management/commands/seed_scripts/documents.py og backend/root/management/commands/seed_scripts/__init__.py som er changes her.

Gjør om til draft inntil #257 er merget


[release/v1.28.x] backport okio-bom version bump to otel-java

Issue #, if available: Patch was made by checking out v1.28.0 tag and performing this git operation.
git cherry-pick 8e9f04eba000533201985727e0d8d1e6cf19e59f
PR #485 was using a protected branch name
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #486 (87f76fc) into release/v1.28.x (eaf370a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@                Coverage Diff                 @@
##             release/v1.28.x     #486   +/-   ##
==================================================
  Coverage              85.71%   85.71%
  Complexity                19       19
==================================================
  Files                      3        3
  Lines                     49       49
  Branches                   5        5
==================================================
  Hits                      42       42
  Misses                     3        3
  Partials                   4        4

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add ipv6 mapped ipv4 addresses to built in ip ranges

Been using uniquelocal as a trust proxy for expressjs to ignore rfc1918 addresses that trusted proxies are deployed into to get to the true client up.
All had been working well with this until ipv4 ips started being prefixed with ::ffff: e.g. ::ffff:<IP_ADDRESS>. Some cursory research suggest that these are ipv6 mapped ipv4 ips when running in an ipv6 complaint network.
https://stackoverflow.com/questions/29411551/express-js-req-ip-is-returning-ffff127-0-0-1
Given that it would seem to make sense to include these in the built in sets of uniquelocal, loopback, and linklocal
var IP_RANGES = {
  linklocal: ['<IP_ADDRESS>/16', '::ffff:<IP_ADDRESS>/16', 'fe80::/10'],
  loopback: ['<IP_ADDRESS>/8', '::ffff:<IP_ADDRESS>/8', '::1/128'],
  uniquelocal: ['<IP_ADDRESS>/8', '::ffff:<IP_ADDRESS>/8', '<IP_ADDRESS>/12', '::ffff:<IP_ADDRESS>/12', '<IP_ADDRESS>/16', '::ffff:<IP_ADDRESS>/16', 'fc00::/7']
}

Ah interesting. Yeah this must not be very common for it to not have come up as a suggestion here for this long. That said, seems likely to be a good addition. Would you mind opening this as a PR so we can discuss details in that form as necessary?


Fix function number datatype from char to uint16_t

Hi, @areusch
Here is the bug I mentioned before.
I cast the first two byte to uint16_t to store the function number.

@kparzysz-quic thanks for pointing out the missing pieces. For the discussion about putting number in the name buffer, please refer to the discussion: https://discuss.tvm.apache.org/t/byoc-limited-function-numbers-on-crt-runtime/11952

@kparzysz-quic i agree it can be a bit misleading. if you feel strongly about it, i'm okay with us adopting the more explicit approach. it would waste a few bytes, but we do now provide a more compact solution for the standalone deployment workflow, so perhaps that's okay now. i am curious on your take on my rationale presented in the thread Cody links above. Regardless of the strategy taken around the C struct, it's going to be a bit dangerous since we generate this structure from LLVM and C source codegen, and then try to re-interpret the structure using a different compiler.
@A1245967 you'll also need to update the LLVM codegen and C source codegen to generate the proper structure.

I agree that accessing C structures via LLVM codegen may be tricky (due to pointer sizes in particular, and alignment rules), we're already running into a similar issue with enums in Device (IIRC).  I should be ok to have the number of functions "hidden" in the buffer, but we should have separate functions to (1) read it from the buffer, (2) store it to the buffer, (3) get the beginning of the actual payload.  This is what I meant in my first comment---this way we would avoid things like for (char *p = names + 1; ...), where it's not clear what that + 1 means, and instead have something like for (char *p = some_better_name_for_the_beginning_of_the_actual_names(); ...).  This would localize the encoding and decoding of the function count to specialized function, where the situation can be explained in comments.  This is all to make it harder for people to make an accidental mistake.

@kparzysz-quic ah i understand. sure, that's totally fine with me.
@A1245967 , if you're up for it, would you mind introducing a few helper functions e.g. TVMFuncRegistry_GetNumFuncs and TVMFuncRegistry_Get0thFunctionName and updating the remaining places to match? i think we can merge with that.

@areusch okay, I try to use some helper functions instead of accessing the buffer directly.

@A1245967 that looks pretty good--i think you just need to update the C++ unit tests in tests/crt/func_registry_test.cc

@kparzysz-quic merged this now


Don't break backward compatibility with older PHP versions.

Composer.json does not have a php version contraint, so it would be a shame to break the backward compatibility just for array notation.

Hi Anton,
Thanks for the pull request. Yes that is true. The PR contains an error, you did the change as well in a javascript file and that will break. Could you revert that part ?
Regards,
Rik
Adyen

Thanks, good one.

Hi Anton,
Thanks for the change I have merged your pull request.
Regards,
Rik
Adyen


Interdependent editable packages

First of all, thanks for all the work on this great package!
I am working on a project updating some old packages that are not distributed via PyPI but via our own package index. Using pipenv with our own package index works great. However, in my dev environment, I'd like to install them editable from github, checking out the correct branch. I can do so by calling the pipenv install -e git+git@github.com:<>/<>.git@<>, which works fine.
The dependency graph of the relevant packages is as follows:
lizard-blockbox==0.30.dev0
  - lizard-ui [required: Any, installed: 4.40]
  - lizard-management-command-runner [required: Any, installed: 0.2.dev0]
    - lizard-ui [required: Any, installed: 4.40]

Pipenv version: `'11.10.1.dev4'

Expected result
Correct installation of three packages from github.
Actual result
Pipenv starts installing lizard-blockbox, which has a dependency on lizard-management-command-runner. This is not on Pypi and installation fails.
Steps to replicate
Generate the Pipfile as follows (the order is important -- see dependency graph):
pipenv --two
pipenv install django==1.8.19
pipenv install -e git+git@github.com:lizardsystem/lizard-ui.git@casper-django18#egg=lizard-ui
pipenv install -e git+git@github.com:lizardsystem/lizard-management-command-runner.git@casper-django18#egg=lizard-management-command-runner
pipenv install -e git+git@github.com:lizardsystem/lizard-blockbox.git@casper-django18#egg=lizard-blockbox

Resulting Pipfile
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[packages]
django = "==1.8.19"
lizard-ui = {git = "git@github.com:lizardsystem/lizard-ui.git", editable = true, ref = "casper-django18"}
lizard-management-command-runner = {git = "git@github.com:lizardsystem/lizard-management-command-runner.git", editable = true, ref = "casper-django18"}
lizard-blockbox = {git = "git@github.com:lizardsystem/lizard-blockbox.git", editable = true, ref = "casper-django18"}

[dev-packages]

[requires]
python_version = "2.7"

Then remove the environment and install again: pipenv --rm; pipenv --two; pipenv install.
The following pip error is logged:
Looking in indexes: https://pypi.python.org/simple00
Obtaining lizard-blockbox from git+git@github.com:lizardsystem/lizard-blockbox.git@a85e3d9c417df52dbfb8c05ada7a7915f55aa3a5#egg=lizard-blockbox
  Skipping because already up-to-date.
Requirement already satisfied: Django<1.9,>=1.7 in ./.venv/lib/python2.7/site-packages (from lizard-blockbox) (1.8.19)
Requirement already satisfied: django-celery in ./.venv/lib/python2.7/site-packages (from lizard-blockbox) (3.2.2)
Collecting lizard-management-command-runner (from lizard-blockbox)

  Could not find a version that satisfies the requirement lizard-management-command-runner (from lizard-blockbox) (from versions: )
No matching distribution found for lizard-management-command-runner (from lizard-blockbox)

Proposed fix
A nodeps=True option in the Pipfile

Just don't install them in editable mode, it's the same thing

That was fast! Thanks. Setting editable = false gives me precisely the same issue (after removing env and lockfile)

Please reopen! @techalchemy

I found the issue, I misnamed the egg, the dashes should have been replaced with underscores. This is solved.

@techalchemy Please ignore my previous comment. I would like to reopen this issue, it keeps popping up at random. If I deploy the specified Pipfile.lock (well, with different branch names now) sometimes it fails, sometimes it does not.
The issue is that lizard-blockbox depends on the other two editable packages. These are not available on PyPI, so if Pipenv happens to install lizard-blockbox first, it fails, but if it had installed the other to already, there is no issue. Luckily pipenv retries failed installs, but for 3 interdependent packages, this is not always sufficient.
Best solution would be to “blacklist” other specified VCS packages when installing a VCS package. But pip does not provide this option.
It seems to be a use case for the sequential install option; but how would I determine the installation order then?
Other solution might be (bit hacky) convincing pip that VCS dependencies already exist before installing (and then force-reinstall at the actual pip install.
Now I just retry deployment until it does not fail. Mostly it works on the second try.

Sequential installs are installed... welll... sequentially. There is a fix for VCS. Installs in the pipeline. Pr is up.

@techalchemy  Right, that fix for VCS ordering would help indeed, so the order is determined by the Pipfile. Thanks.

@techalchemy this is still failing for me.
My Pipfile:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
"com.example.a" = {ref = "master", git = "https://<private>.git", editable = true, subdirectory = "a"}
"com.example.b" = {ref = "master", git = "https://<private>.git", editable = true, subdirectory = "b"}
"com.example.b" = {ref = "master", git = "https://<private>.git", editable = true, subdirectory = "c"}

[dev-packages]

[requires]
python_version = "2.7"

This gives me
> pipenv install
Creating a virtualenv for this project...
Pipfile: <snip>/Pipfile
Using /usr/local/bin/python2.7 (2.7.15) to create virtualenv...
⠋Running virtualenv with interpreter /usr/local/bin/python2.7
New python executable in /Users/<snip>/.local/share/<snip>/<snip>/bin/python2.7
Also creating executable in /Users/<snip>/.local/share/virtualenvs/<snip>/bin/python
Installing setuptools, pip, wheel...done.
Setting project for <snip> to <snip>

Virtualenv location: <snip>
Pipfile.lock (f8f229) out of date, updating to (cfb2eb)...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...

CRITICAL:notpip._internal.index:Could not find a version that satisfies the requirement com.example.a>=1.0.9 (from com.example.b->-r /var/folders/4k/zb_bkrkx5pvgnhsvfz59mw100000gn/T/pipenv-GWr0N2-requirements/pipenv-GQQKDK-constraints.txt (line 15)) (from versions: )
Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  You can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
No matching distribution found for com.example.a>=1.0.9 (from com.example.b->-r /var/folders/4k/zb_bkrkx5pvgnhsvfz59mw100000gn/T/pipenv-GWr0N2-requirements/pipenv-GQQKDK-constraints.txt (line 15))

/usr/local/lib/python2.7/site-packages/pipenv/_compat.py:113: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/var/folders/4k/zb_bkrkx5pvgnhsvfz59mw100000gn/T/pipenv-nsR755-requirements'>
  warnings.warn(warn_message, ResourceWarning)

Note that this works in pip, if you use dependency_links like so:
setup(
  name="com.example.b",
  version="1.0.0",
  install_requires = [ "com.example.a==1.0.0"],
  dependency_links = [<EMAIL_ADDRESS>]
)```
`pip install -e /path/to/monorepo/b`

This correctly installs `com.example.a`, but pipenv throws the same error as above


Implementation of PCA added

congratulations on your hactoberfest contribution. if you like this repo then give it a star. Happy hacking..

Hello sir,
I have contributed to your repository for the Hactoberfest 2021 . But in
the Hactoberfest portal it is showing "Excluded project". I don't
understand what is this and why is this happening??? I would be grateful if
you could clear it a little bit.
Thank you!!!
On Tue, 5 Oct, 2021, 11:51 AM Vijay, @.***> wrote:

congratulations on your hactoberfest contribution. if you like this repo
then give it a star. Happy hacking..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vijaygupta18/Hacktoberfest-2021/pull/368#issuecomment-934102837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARMLZTBN2JNL33QM4F663NTUFKKOLANCNFSM5FKGK73A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


HTTP Error 403: Forbidden while downloading videos

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.09.15. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2017.09.15

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--username', u'PRIVATE', u'--password', u'PRIVATE', u'--verbose', u'https://www.pluralsight.com/courses/javascript-module-fundamentals']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.09.15
[debug] Python version 2.7.12 - Linux-4.4.0-93-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: none
[debug] Proxy map: {}
[pluralsight:course] javascript-module-fundamentals: Downloading course JSON
[pluralsight:course] javascript-module-fundamentals: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/home/youtube/dev/programs/youtube-dl-2017.09.15/youtube-dl/youtube_dl/extractor/common.py", line 503, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/home/youtube/dev/programs/youtube-dl-2017.09.15/youtube-dl/youtube_dl/YoutubeDL.py", line 2177, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

...
<end of log>

Description of your issue, suggested solution and other information
I was trying to download a video for my future reference, which I normally do, and in between the download process was interrupted. On command line a HTTP Error 403 was shown. Further I was asked to raise a bug request. Any help is appreciated.
Note: I have verified that I am on the latest version of youtube-dl.

You can download PluralSight videos by finding the video link in the page source. Not sure if that helps you at all.

I think the issue is being caused by "I'm not a robot" mechanism.
the behavior and error of this issue seems to be identical to when you try downloading without authentication at all (--username and --password flags)

Facing same issue for https://www.raywenderlich.com/977854-data-structures-algorithms-in-swift/lessons/4


Support mediaQueryList event listener in older Safari

Overview
Older Safari and Mobile Safari versions have addListener instead of addEventListener and removeListener instead of removeEventListener.
Reference:
https://github.com/chakra-ui/chakra-ui/pull/5474
Connects #2294
Demo
Using an iOS 13 iPhone via BrowserStack
Before

After

Testing Instructions

Compare this branch to ogr/develop when browsing on iOS 13

Checklist

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

I also saw it working in iOS 15. I could replicate it in iOS 13 using BrowserStack and the simulator running the older iOS.

Thanks for the review.


Dependency Graph cannot be generated

Hi I think I ran into the same issue as described here: https://github.com/CycloneDX/cdxgen/issues/243
When generating a bom.json with "cdxgen -o bom.json" and uploading the file into Dependency Track (https://docs.dependencytrack.org/) it does not create a dependency graph.
I used this project as a sample: https://github.com/acemilyalcin/sample-node-project
If I try "cyclonedx-npm --output-file bom.json" it works.
Here are both bom files:

Thank you in advance!

@mariusgiesen, please retest with 8.1.2

The issue was fixed in 8.1.2
but while working with projects containing multiple sub projects for example if a project has a backend and the frontend then cdxgen just uses one of them as bom-ref which leads to the tree containing only half of the components,
So it will either have only frontend components or only backend components.
@prabhu any fix regarding that possible?

@phoenix-aditya, do you have an example project to replicate this issue?

For example in a project like this which contains maven and npm packages,
https://github.com/darecoder/Polling-App
only the npm packages are part of the dependency tree on dependency tracker.
The issue i can see is that as the npm packages and maven packages are not dependent on each other therefore 2 separate trees are being generated and cdxgen picks the root of one of those trees.
The solution as far as i can see would be to either add a false root and combine all the roots together or render multiple trees in dependency tracker

@phoenix-aditya could you file an enhancement request for dependency-track to render multiple subtrees?

@prabhu
is this feature addition possible?
https://github.com/DependencyTrack/dependency-track/issues/2572#issuecomment-1458013850

@phoenix-aditya

I guess the feature is not required in cyclonedx Plugins as the problem wont exist as they are tech stack dependent

Even under same tech stack (Java/gradle), I'm facing the same issue if you have subprojects under subdirs with different gradle files. The superproject and the subproject are only related in terms of superdirectory and subdirectories and do not have any parent-dependency relation and thus not rendered in DT.

I'm also having issues seeing a Dependency Graph in Dependency Track when using cdxgen version 9.3.1.  I've tested across npm, maven and nuget repositories with the same outcome

@setchy could you kindly share any test projects and generated sbom to reproduce the issue? If the projects are private, you can share them via discord.
https://discord.gg/pF4BYWEJcS

Raised https://github.com/CycloneDX/cdxgen/issues/447 after doing some further debugging locally

@setchy could you kindly share any test projects and generated sbom to reproduce the issue? If the projects are private, you can share them via discord.
https://discord.gg/pF4BYWEJcS

Raised https://github.com/CycloneDX/cdxgen/issues/447 after doing some further debugging locally


Add support for CSP Nonce to the PrerenderTagHelper

CspNonce property is added into PrerenderTagHelper class. nonce attribute is rendered if this property is not empty. https://github.com/aspnet/AspNetCore/issues/9416

@anurse @Tratcher I see that some checks are failed, but I can't understand how they related with my changes. Can you help me?

@dima117 you have to regenerate the reference sources for  src/Middleware/SpaServices/src/Microsoft.AspNetCore.SpaServices.csproj: https://github.com/aspnet/AspNetCore/blob/master/docs/ReferenceAssemblies.md#when-changing-public-api

Closing as there has been no activity on this PR for quite a while.


New Android Auto app doesn't allow to start GMaps

The new version of Android Auto mobile app allows the execution of phone apps only from car screens. (see screenshot).
The side effect is that now GMaps cannot be launched within Android Auto and, consequently, when in screen mirroring it has the same FPS cap as any other screen casting.
This is the same also when AAIdrive is in USB mode.
Is there any possibility to lift the FPS cap for GMaps only? Can AAIdrive detect which app is opened from the background?
Thanks for all the hard work!

Good morning! That is disappointing to hear!
Android doesn't provide a good way to tell what app is in the foreground :( The Android Auto detection checks a global Car UI Mode flag, and doesn't provide any further options.
I did update Screen Mirroring to go at 1fps instead of 0.5fps while in motion a few months ago, are you using that version?

I'm using 1.3.2-114-52ea6bd-bmww_xl
I am not sure when I last got this nightly build

Screen Mirroring is a separate app with its own versions. I believe it will still include the git hash in there, to help you figure out what code version you have installed.

I wasn't able to fetch the file I downloaded with the version. I will try to install the most recent version and re-test

Tested the latest version in USB mode. It can only work smoothly when the car is stationary. When in movement, the FPS cap is lowering sensibly the quality of the experience, which was not the case before when Maps was running under Android Auto. I guess there is not much else we can do here...

Unfortunately there isn't any good way to detect the running Maps app :(


[Joy UI] Add CircularProgress component

[x] I have followed (at least) the PR section of the contributing guide.

Done:

Initial implementation
Playground demo
Props: color, size, variant: 'indeterminate' | 'determinate', etc; To be more discussed with Jun and Danilo

UPDATE:

✅ Iterate on demos/implementation with Jun and Danilo's feedback
✅ Add unit tests
✅ Add typescript test

Preview: https://deploy-preview-33869--material-ui.netlify.app/joy-ui/react-circular-progress/

@danilo-leal

A question: we mention determinate and indeterminate in the introduction－how would I go about it to controlling the component so I have one or the other? Is that done through a prop?

Yes, through a prop called variant at the moment, but I think we should change the name to something else to reduce confusion since variant is used often in other Joy components for theme values. Maybe a prop called determinate that can be either true or false? It can be false by default.

Also, maybe we should document size and color (even though they're on the Playground there already) just for the sake of consistency?! Other components have those sections too.

I agree. Will do that!

I dig the idea of a boolean determinate prop! 👌

@danilo-leal I dig the idea of a boolean determinate prop! 👌

Done! Can you suggest some ideas for demos?

@danilo-leal @hbjORbj Does it make sense to not follow the Material Design look and feel?
Here is a simpler version using div + border:
https://user-images.githubusercontent.com/18292247/184580113-7d48b19a-1af3-4bd2-aa2a-78f6554135e7.mov
https://mui.com/joy-ui/react-link/#common-examples

Variant
Let's add the global variant to CircularProgress. I think it is useful for composition:

CSS variables

let's add --CircularProgress-size that can be customized by the parent{
   width: 'var(--CircularProgress-size, 32px)',
}

let's add --CircularProgress-thickness that can be customized by the parent as well.
I think --CircularProgress-speed to control how fast it moves is worth trying.

Hey, Jun @siriwatknp , I addressed all your feedback except one: I am not sure how to go about implementing determinate prop. Can you help me with that?

@hbjORbj Looks like my proposed solution, using a border, would not work for Determinate progress. Seems like using svg with stroke-dasharray and stroke-dashoffset is the best approach so far.
I will create a quick POC on this first.

@hbjORbj Here is a working POC using SVG https://codesandbox.io/s/poc-circular-progress-3dffpd?file=/index.js

@hbjORbj Another review
implementation

The track or progress is cut off when thickness is not the same value:

I suggest mixing CSS min or max to the r properties of the track and progress slots.

The --CircularProgress-speed is not used. Maybe changing it to --CircularProgress-indeterminateDuration is more direct?

Playground

add determinate switch

Demos
Please add these demos

determinate progress similar to Material UI
Children (text in the middle of the circle)
CSS variables section

@siriwatknp
Thanks, Jun, I addressed your feedback.
Let me know!

This is looking great @hbjORbj and @siriwatknp! Awesome work!
Pushed a couple of refinement changes and the only thing that's standing out to me is the icon children being slightly misaligned－I've tried adding justifyContent: 'center' but that didn't appear to solve 🤔

Thanks @danilo-leal ! It looks much better now :)
I aligned the icons!

Sweet, thanks @hbjORbj, it looks good now! I see we're centralizing them using specific values for each CircularProgress size. Have you verified if that works out if you change the icon's size?

@danilo-leal
I verified that it works out if we change the component (circular progress)'s size, but not the icon's size. Hmm, I am not sure how to best about this. Any idea @siriwatknp Jun?

@danilo-leal
I verified that it works out if we change the component (circular progress)'s size, but not the icon's size. Just confirmed that it breaks with varying the icon's size. Hmm, I am not sure how to best about this. Any idea @siriwatknp Jun?

FYI, I am trying alternative way, will share soon.

Changes from me

fix the rotation degree to let the progress starts from the top

rename the slot to be more meaningful from circle1, circle2 to track, progress (similar to Switch, Slider)

Reduce the default size values to match other components, e.g. IconButton, Button

Make the svg absolute so that the children of the CircularProgress can stay at the center using flexbox.

update the outline styles to be consistent with other components (using border):

add some more useful variables: --CircularProgress-circulation, --CircularProgress-linecap.

fine-tune the track and progress stroke:
https://user-images.githubusercontent.com/18292247/189306590-36282e70-d0a1-4bf7-b3dc-1e5e62480893.mov

Brilliant Jun 🤩🤩🤩 Thanks!!!

What do you think about slowing down the default speed? 0.5s seems a bit frenetic. 1s feels much more calm. This could perhaps be configurable by a prop?

No problem!

Outlined feels like it should have an inner outline, so that the track itself is outlined.

Agree, that's a better outlined.

The color of the spinner doesn't appear to match the color selected in the Playground.

Yeah, the swatch should follow the variant. Will fix it in a separate PR.

It might be nice to have (an option of?) a linear gradient on the spinner so that it fades towards the tail, along the lines of this:

Good suggestion but I think it will be a customization example, not a built-in style. Will try to add it soon.

It isn't possible to adjust CSS variables with the keyboard. (On which note, when can we expect @siriwatknp's number input to be migrated to MUI? 😁 It's the most requested component! )

Yeah, I think we should start the NumberInput on MUI Base soon. cc @michaldudak

There is a growing list of components we should create in MUI Base :) Let's set priorities on the next product meeting.


Route the questions to the correct previous/next places

This isn't 100% yet and there are way more questions to route, but
I've started...

Pausing this momentarily to work on the last question (entering NHS number). Routing will be easier when all teh questions are done.

Starting this up again now...

Superseded by #66. 😬


buffer overflow fix buf is sized 24 (ary member of RString)

Can you post Ruby code and if necessary any other steps that are needed to reproduce the bug?
Also a string returned by mrb_str_buf_new() never uses the embedded array to store its content. So your fix seems incorrect to me.


HS and POT macro work for cat but not bear

I have a macro
#showtooltip
/click dmhStart
/dmh cd hs pot
/use Charged Crystal Focus
/use Healing Potion Injector
/click dmhEnd

If i use it in cat it works as intended, but dire bear pops me to human form.
Running /dmh debug I get the following doing both cat and dire bear
Slash command called:  start
Slash command called: cd hs pot
Slash command called: end
Enabling autoUnshift again...
Resetting SpellQueueWindow to 400
Edit:  Think i found the issue.  On line 329 of DruidMacroHelper.lua it reads
spellBear = L["FORM_BEAR"];
I changed it to
spellBear = L["FORM_DIRE_BEAR"];
And it seems to work.

Sorry should have included the build, its DruidMacroHelper-1.2.1-10-gdaa2a67-bcc.zip.  And client is in English.  I'm sure I broke your regular bear form vs dire bear form check, and I don't think there is more to it because I wasn't getting that behavior on my ironskin pots or seeds, just the hs / health pot macro.  I did try and remove the healing pot part of so its just healthstone  and still got the same behavior.

Hmm.. for me everything looks okay. Can you check the output of /dmh debug in the latest version? It should look like this:

If it does not (you have [bar:2]!Bear Form; instead of [bar:2]!Dire Bear Form;) can you check what the following scripts return?
/script print(not GetSpellInfo("Dire Bear Form"))
/script print(not GetSpellInfo("Bear Form"))

The first should return false and the second true. If that is the case can you check whether /script DruidMacroHelper:UpdateForms() fixes the issue?

It shows [bar:2]!Dire Bear Form; and all seems to be working correctly in this version.

Strange, I did not change anything related to that issue in the current version. I only added some debugging information. Maybe it is an edge case (race-condition when loading the interface for example) that causes it. Calling DruidMacroHelper:UpdateForms() on PLAYER_ENTERING_WORLD probably is a good idea.

I made those changes, so hopefully that issue should not occur anymore with the latest version

Awesome, i'll grab the new version.  Thanks for all the work!


[Bug]: /playlist command does not show note for truncated items

Describe the bug
If you add more than 10 tracks to your playlist, the /playlist command won't show any hint, that items were truncated.
To Reproduce

Add more than 10 items to the playlist (eg. Playlist or Album)
Use /playlist

Expected behavior
The embed has a message that items were truncated, if there are more than 10 items.
Screenshots

Implemented in #106


Fixes downloader error on iOS

Sets the CACHE_DIR variable to be in a folder where the user has write permissions when targeting the iOS platform. Without the fix, python keeps printing "downloader error" when trying to save new map tiles.

Build is failing with:
======================================================================
ERROR: test_init_simple_map (tests.test_mapview.TextInputTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/app/tests/test_mapview.py", line 12, in test_init_simple_map
    mapview = MapView(**kwargs)
  File "/app/mapview/view.py", line 541, in __init__
    self.map_source.cache_dir = self.cache_dir
AttributeError: 'MapView' object has no attribute 'cache_dir'

And we're migrating to the new flower structure https://github.com/kivy-garden/mapview
Let's make a follow up PR on there. Otherwise I'll try to merge and fix myself a bit later


[Question] When i use hql on linkis There are some errors  help me please

Before asking

[X] I have read the How to ask for help (我已阅读如何寻求帮助指引文档).

Your environment

Linkis version used: 1.1.0
Environment name and version:

hdp-2.7.7
hive-2.3.3
spark-2.5.4
jdk 1.8.0_121

Describe your questions
Q1. I install as https://linkis.apache.org but I still some errors
Eureka service list
Uploading image.png…
Some logs info or acctch file
logs.zip

Hello, is there any missing jars under the convenient update to deal with this problem?
你好,方便更新下缺少那些jar处理掉这个问题的吗?

It's not the same error

org.apache.linkis.orchestrator.ecm.exception.ECMPluginErrorException: errCode: 12003 ,desc: hadoop105:9101_48 Failed  to async get EngineNode AMErrorException: errCode: 30002 ,desc: ServiceInstance(linkis-cg-engineconn, hadoop105:42033) ticketID:33ef7680-6019-495c-96da-e8ca7aa928b7 初始化引擎失败,原因: Failed to start EngineConn, reason: ECMErrorException: errCode: 11102 ,desc: Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=250m; support was removed in 8.0
Whether the version is incompatible?

org.apache.linkis.orchestrator.ecm.exception.ECMPluginErrorException: errCode: 12003 ,desc: hadoop105:9101_48 Failed to async get EngineNode AMErrorException: errCode: 30002 ,desc: ServiceInstance(linkis-cg-engineconn, hadoop105:42033) ticketID:33ef7680-6019-495c-96da-e8ca7aa928b7 初始化引擎失败,原因: Failed to start EngineConn, reason: ECMErrorException: errCode: 11102 ,desc: Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=250m; support was removed in 8.0 Whether the version is incompatible?

这个在新的版本里已经修复了

about dss1.0.0 and linkis 1.1.0

This question has not been answered for a long time, I will close it first, if necessary, you can re-open it.


bug: playground undeclared variable

Bug Report
Description
Bug Summary:
Uncaught (in promise) ReferenceError: assignment to undeclared variable currentRequestId
    ee Playground.svelte:84
    T Playground.svelte:214
    Ct Playground.svelte:380
    St dom.js:361
    m Playground.svelte:384
    m Playground.svelte:395
    _t Component.js:44
    m Playground.svelte:389
    _t Component.js:44
    p root.svelte:51
    wt utils.js:203
    p +layout.svelte:67
    at scheduler.js:119
    ut scheduler.js:79
    promise callback*lt scheduler.js:20
    ht Component.js:81
    ctx Component.js:139
    $$set root.svelte:62
    $set Component.js:507
    ce client.js:1129
    _start_router client.js:1603
    _start_router client.js:1521
    jt start.js:27
    <anonymous> playground:80
    promise callback* playground:79
Playground.svelte:84:5

Steps to Reproduce:
Use the Playground in completion mode with a text model (as you should, not an instruct/chat model). Do anything. You will get one opportunity to generate a completion, then the UI will break. The "cancel" button will not work, and it will not ever revert to a "submit" button.
I am using the latest docker image, and I checked before submitting this. It is very clearly just an untested piece of code. Probably just need to remove this line of code.

Good catch, PR welcome!

I don't have dev environment set up for this repo, sadly, and anything javascript related could take an hour or two to get fully set up... which is more than I want to spend on this bug at the moment. Instead, I'll just use ollama's completion API directly for this experiment. If there were a devcontainer in the repo, I might be able to easily test and submit a PR, but I'm not planning to blindly submit a PR without being able to test it.

I was about to post this issue 2 minutes ago when I realised I was about to post in the ollama repo...

The "currentRequestId is not defined" is triggered when you clic on "cancel".
stopResponse
Playground.svelte:84 Uncaught (in promise) ReferenceError: currentRequestId is not defined
    at ee (Playground.svelte:84:6)
    at async T (Playground.svelte:214:11)

After that the button stay in cancel stay, and doesn't revert back to "Submit". Cuda activity does stops when cancelling, suggesting this is mostly a UX bug.

Fixed on dev!

@tjbck nice! thanks!


please add an option to enable black-like formatting of trailing comments

in the section of intentional deviations from black, this is mentioned:
Expressions with (non-pragma) trailing comments are split more often#
please add a way to retain black behaviour. or at least for "task" (TODO, XXX, ...) comments.
our team usually tries to stick to short trailing comments but one big exception are TODO and FIXME comments which are used as kind of a "conversation" between the submitter and reviewer within the code itself. as such, they tend to be pretty long (containing explanations, questions, etc.) and we are fine with that as they will be ironed out before the final merge.
ruff constantly reformatting this is kind of anoying for everyone involved and is often a reason to be reluctant to switch from black to ruff.
please consider an option to leave trailing comments (as long as they start within the line-lenght limit) untouched. that would be the best solution for us. the second best would be to make the formatter honor the "ignore-overlong-task-comments" for the formatter.
thank you.

@rpodgorny I understand that moving comments is annoying and we try to minimise the cases where this is happening.
However, the comment placement logic is one of the most complicated parts of the ruff formatter and supporting multiple placement logic would significantly increase complexity. That's why I prefer better understanding your use case and see if we can improve the comment placement overall.
Could you share a specific example where adding / removing the TODO/FIXME comment changes formatting and how you're ideal formatting would look like?

I'm closing this because it is out of scope for now. @rpodgorny feel free to re-open if you have a follow up question.

i don't really know if i have enough privileges to reopen, but anyway - i can't provide exact code but i've created this example that should show our use case:
def func(args):
        code
        code code code

        # TODO: case 1 - the entire logic in following block needs rework becase such and such. now follows dozen lines of explanation of how such change is meant and how it should be implemented. individual lines do not really matter, the block as a while is commented here. this comment being split automatically to multiple lines is acceptable.
        block code code
        block code
        block code code code

        # this is some very long, possibly multi-line comment explaining what the following block does.
        code code code
        code
        code code
        code code  # TODO: case 2 - this line's performance is sub-optimal becase such and such. follows a very long explanation how it should be changed. the rest of the block is alright. it's just this line that's to be taken care of. splitting this comment to multiple lines is not acceptable. also, moving this comment to line above is not acceptable as it "lessens" the intent of being local only to this line. splitting and/or moving may be misinterpreted as having the meaning of "rest of block" or something...
        code
        code code code

        code
        code code
        code code code
        # TODO: case 3 - drop the rest of this because such and such. if case 2 gets reformatted, there could be confusion with this case.
        code code code
        code code

line wrapping in editor ensures all todo items are fully visible. the exact placement of comments carries meaning and any shuffling would be counter-productive.
...anyway, isn't this exactly what ignore-overlong-task-comments is meant for?


Fixing run bug fo ApplyMosaicRuleToRasters

Description
The sample was unable to run and failed upong execution, simple fix.
Checklist:

[x] Set target branch to main (current release) or v.next (next release)
[x] Request a reviewer
[x] Assign a reviewer
[x] Tag reviewer in comment

@D-R-Smith , could you please review?


MyMacbook

I want to use Notable on my Macbook retina.

Refer to the README.md file for how to build, the run npm run release to build your package to the dist folder


bugfix for the light modelers

the custom param coef_GPP was getting sorted so that its indices started at [6,3] instead of [1,1] in fit. the logic for this case in previous versions of the code is a mystery to me - no idea what i was thinking - so i've replaced it with the assumption that things with two indices can parse those indices as [time,date].

appveyor issue may be due to tidyr 0.7.0: https://github.com/tidyverse/tidyr/issues/350


Add instructions for running from source to README

Add instructions for running directly from source code to README

Good idea to document how to run it from source. The approach you suggest doesn't actually work for me. I usually do this:
pip3 install -e /path/to/checkout/of/aider

After that you can invoke aider as normal, but it will be running from the working directory of your checkout.
Let me know if you'd like to update the PR, or I can.

@paul-gauthier thanks for the reply. I'll can update the change later today

@paul-gauthier updated

Thanks for the PR!


[pylint] List-return-type checker

Client methods that start with List_ should return an AsyncItemPaged/ItemPaged, or a custom class that then either inherits from ItemPaged or creates its own custom paging class with by_page()
Fixes #3232

Are there violations of this? Should this be added to the global CI suppressions for a time?

My plan is to initially add them disabled and then see the impact/ give SDKs time to fix the warnings and then enable them.


Extend n2.mediaDisplay library to support inline media

Currently media files are shown within a dialog box. This library needs to be extended to support inline media as well.
Note: Once this is done, the insert media player show service will need to be updated (remove any code duplication)

This issue was resolved with the n2s_insertMediaPlayer show service, added with #749.
See wiki for further details.


add security IT

Signed-off-by: Yaliang Wu<EMAIL_ADDRESS>Description
Add security IT.
./gradlew integTest -Dtests.rest.cluster=localhost:9200 -Dtests.cluster=localhost:9200 -Dtests.clustername="docker-cluster" -Dhttps=true -Duser=admin -Dpassword=admin --tests="org.opensearch.ml.rest.SecureMLRestIT"
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Yes, will add more test cased in next PRs.


Hide rustls certificate types and parsing methods

This decouples an important part of the API from rustls, and greatly simplifies the task of robustly reading a certificate from disk.
Thanks to @est31 for working out how to wrangle this properly.

Thanks @Ralith for making this PR! It's a much more general solution than my own PR #167.

The approach of parsing twice feels a little ugly/fragile, maybe we should have different methods for these, or an enum thing?

The user doesn't typically know or care what type of key is inside their .pem file, and I think it's important to support that gracefully. This is ultimately just TLS file formats being extremely strange.

The approach of parsing twice feels a little ugly/fragile, maybe we should have different methods for these, or an enum thing?

The user doesn't typically know or care what type of key is inside their .pem file, and I think it's important to support that gracefully. This is ultimately just TLS file formats being extremely strange.

They do care, because the private key must match the public key in the certificate that is being used. In general, to be safe, one must always know the public key when loading a private key, and that means knowing which algorithm to use.
Another reason ring's API is the  way it is (and thus Rustls's API is the way it is): In ring we avoid supplying any function that references more than one algorithm. This way, if the calling code never references an algorithm, no code for that algorithm will be linked in, without any configuration.

They do care, because the private key must match the public key in the certificate that is being used. In general, to be safe, one must always know the public key when loading a private key, and that means knowing which algorithm to use.

To be clear, I was referring to PKCS#1 vs PKCS#8, not different algorithms. None of the code in this PR makes any assumptions about or references to cryptographic algorithms.


add version message to fe heartbeat and system page

Proposed changes
This commit adds git commit message of fe to the system page, just like be.
To achieve this, fe heartbeat message has now been carrying git commit message.
Types of changes

[] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation Update (if none of the other choices apply)
[] Code refactor (Modify the code structure, format the code, etc...)

Checklist

[] I have create an issue on (Fix #ISSUE), and have described the bug/feature there in detail
[] Compiling and unit tests pass locally with my changes
[] I have added tests that prove my fix is effective or that my feature works
[] If this change need a document change, I have updated the document
[] Any dependent changes have been merged

Further comments
This is my first public pr so plzzzzzz be gentle, thx=.=

@ccoffline  Hi, Thanks for your contribution.

If we only want to show a git commit message on FE web, why do we need to add a commit message to the heartbeat?

@ccoffline Hi, Thanks for your contribution.

If we only want to show a git commit message on FE web, why do we need to add a commit message to the heartbeat?t

FE web shows status of each BE and FE by represent the message from the heartbeat.
The BE Version has been shown on the web and it is carried by the BE heartbeat.
I use the same implementation as BE.