Engineering module changes

About the PR

Altered Content of Several Borg Modules for QoL:
Added Hand Radio to Mining Module
Added GPS to Grapple Module
Added Crew Monitor to Diagnosis Module
Exchanged Service Module Syringe with a Dropper
Added Dropper to Advanced Cleaning Module
Added Geiger Counter, Gas Analyzer, and Remote Signaller to Advanced Tools Module
Removed Gas Analyzer Module
Why / Balance

I feel like these changes will make it easier for Borgs going forward without upping their capabilities too much
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

The only issues are that the Crew Monitor in the Diagnosis Module needs a Cell which cannot be replaced, and that I can't find a way to make a Lingering Stack for Plasteel and Reinforced Glass
Changelog

:cl:

add: Added Hand Radio to Mining Module
add: Added GPS to Grapple Module
add: Added Crew Monitor to Diagnosis Module
add: Added Dropper to Advanced Cleaning Module
add: Added Geiger Counter, Gas Analyzer, and Remote Signaller to Advanced Tools Module
tweak: Exchanged Service Module Syringe with Dropper
remove; Removed Gas Analyzer Module

I had to fight my IDE to take these Screenshots.

The the syringe change just so service can't inject people?

test fails are real

"test fails are real"
Nuh Uh (Fixed it,)

The the syringe change just so service can't inject people?

I removed the syringe because a dropper works perfectly fine and I don't want a Budget Mediborg

dropper is 3x slower

Is there an actual difference between the BorgDropper and Regular Dropper?

Tested and the only difference is the BorgDropper can hold 10u and Dropper holds 5u

I'm still switching out the syringe because service borgs don't need to be able to inject people

Making the borg dropper have 15u capacity sounds like the perfect solution.

Gonna Mark as Draft for now while I modify some things.
Also Arendian I saw https://github.com/space-wizards/space-station-14/pull/23696/commits/466a5ae3ed18738e5cdde6c62be0faf3f0ab2057 and I'm basically gonna copy this code for the Crew Monitor to resolve the battery issue

Powercreep is real.
Remove crew monitor at least

Why do you feel it's powercreep?

Remove crew monitor at least

I have tripled the power use of the Borg Crew Monitor as a drawback

Crew monitoring is not supposed to be mobile it needs removing.

ok then get rid of the handheld crew monitor item

Crew monitoring is not supposed to be mobile

Handheld Crew Monitor go BRR

Crew monitoring is not supposed to be mobile it needs removing.

ok then get rid of the handheld crew monitor item

It has no bearing on this pr but https://github.com/space-wizards/space-station-14/pull/24761

Crew monitoring is not supposed to be mobile it needs removing.

ok then get rid of the handheld crew monitor item

It has no bearing on this pr but #24761

In that case I will get rid of the Crew Monitor, please lead with saying Handhelds are getting phased out.

I have addressed the requested Changes

Remove construction stacks #24792 this is just a bandaid and I don't want to increase tech debt.
that's free hands with a) extra steps and b) no way to know the whitelist

Do you mean construction stacks for the Module as a Whole or just my proposed Plasteel and Reinforced Glass Changes?

Whitelisted Free Hands requires More Code as a Whole and leads to more Tech Debt than adding a Lingering version of a Stack

Remove construction stacks #24792 this is just a bandaid and I don't want to increase tech debt.
that's free hands with a) extra steps and b) no way to know the whitelist

Do you mean construction stacks for the Module as a Whole or just my proposed Plasteel and Reinforced Glass Changes?

For now just don't make the problem worse.

Done

Changelog should be modified to be more concise. I don't think every change to every module has to be included outright.

Changelog should be modified to be more concise. I don't think every change to every module has to be included outright.

Done


Routes in ng build --prod

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
W 10

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.26

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.
Hi I have a simple app with two  components,  I make a router archive too, but  when I execute the command ng build --prod,  my routes stop working but in my developer version is working well , I move all the archive from inside the dist folder to my root of my domain can you help me ??

Thanks! We'll be in touch soon.

Can you share a simple app repository that we can use to reproduce this?

Heya, I think you have an issue with deployment. See https://angular.io/docs/ts/latest/guide/deployment.html for deployment information.


enh(cpp) support pack expansion in function declarations

Make dots legal in function declarations, which is necessary for variadic functions with parameter packs in C++: https://en.cppreference.com/w/cpp/language/parameter_pack
Resolves #2917
Checklist

[x] Added markup tests
[x] Updated the changelog at CHANGES.md
[x] Added myself to AUTHORS.txt, under Contributors

@mortie  Thanks so much!


Not sure why a south plugin is publishing data to a broker

It should be subscribing and ingesting into the gateway.

Hi @sudhirkshirsagar

It should be subscribing and ingesting into the gateway.

Indeed! South plugin is a subscriber only.

Not sure why a south plugin is publishing data to a broker #2

No, South plugin is not publishing any data. Its objective (obvious design) is to subscribe the topic and ingest as published on the subscribed topic.
Please check README / give a try to plugin. Let us know if need other clarifications.
Hope this helps!

When I saw the code yesterday, I saw a publisher. The code today is subscribing.

When I saw the code yesterday, I saw a publisher. The code today is subscribing.

Could you please point to the commit # of the code you saw yesterday? This repository has the subscriber code since its first commit.

I tried to look for it but I can't see that. I was looking at it along with Dr. Steve Mylroie and he had the same comment.


Concurrency with redis

Hi there
i re-implemented starwars example (actix) with redis:
https://github.com/salva-imm/starwars
I moved the models/data into Redis
I ran a few benchmarks, and it was terrible : )
my implementation is not the best one yet, I'm using clone and move heavily, for the sake of serde
but yet, I didn't expect to see this result:

400 req/s !!
is this how far we can go?

I ran some other benchmarks after changing response to hardcoded data(mostly None) to see if its Redis or my code that makes it slow
and I achieved 2k req/s , but yet its not what I'm getting while using REST and raw actix ( ~ 80k!)
I would appreciate it if you could give me some tips for performance and concurrency, I'm new in Rust : )
hardware:
Macbook m1/16G
query :
{ humans{ edges{ node{ id friends{ id name } name }}}}
(I chose this because its heavy, and I needed to retrieve friends as well)
benchmark tool:
https://github.com/hasura/graphql-bench + autocannon
redis data:

EDIT:
by using actix_redis rather than bb8+redis-rs, I improved the result up to 4k/s
just don't know how far I can go : )

Most of the time is spent fetching data from Redis.

@sunli829 indeed, if replacing redis driver(with actor model one) improved it a lot
but I got no more idea for going further
I wonder what is best I could do with async-graphql itself
I'm going to run this benchmark on your example directly, without any change : )


Unable to compile atom from source. Fails when creating mksnaphost

Prerequisites

[x ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Unable to compile atom from source. Fails creating mksnaphost
Generating startup blob with mksnapshot
Moving generated startup blob into "/home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/v8_context_snapshot.bin"
Error: ENOENT: no such file or directory, rename '/home/alex/atom/out/v8_context_snapshot.bin' -> '/home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/v8_context_snapshot.bin'
Steps to Reproduce
script/clean
script/build --create-rpm-package
Expected behavior: [What you expect to happen]
Complete the build
Actual behavior: [What actually happens]
Unable to build
Reproduces how often: [What percentage of the time does it reproduce?]
all the time
Versions
cat /etc/redhat-release
Fedora release 26 (Twenty Six)
apm  2.1.3
npm  6.2.0
node 8.9.3 x64
atom 1.34.0
python 2.7.15
git 2.13.6
Additional Information
Downloading electron-v3.0.14-linux-x64.zip
[============================================>] 100.0% of 51.5 MB (4.04 MB/s)
Packaging app for platform linux x64 using electron v3.0.14
Changing permissions for node files in /home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64
Copying non-ASAR resources to /home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/resources
Writing LICENSE.md to /home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/resources
Application bundle created at /home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64
Generating snapshot script at "/home/alex/atom/out/startup.js" (3986)
Minifying startup script
Verifying if snapshot can be executed via mksnapshot
Generating startup blob with mksnapshot
Moving generated startup blob into "/home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/v8_context_snapshot.bin"
Error: ENOENT: no such file or directory, rename '/home/alex/atom/out/v8_context_snapshot.bin' -> '/home/alex/atom/out/atom-dev-1.36.0-dev-c79716659-amd64/v8_context_snapshot.bin'
at Error (native)
at Object.fs.renameSync (fs.js:819:18)
at electronLink.then.err (/home/alex/atom/script/lib/generate-startup-snapshot.js:128:10)

A couple questions to help diagnose the issue:

In your Atom repo's script/ folder, what does npm ls electron-mksnapshot return?
What are the top-level contents of the out/ folder after building Atom?

@daviwil  thank you!
<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>[alex@dell atom]$ np
npm    nproc
[alex@dell atom]$ ls out/
app  atom-dev-1.36.0-dev-c79716659-amd64  startup.js  symbols
Thanks for the ls cmd. That gave it away. There was a pb with the fc26 npm installation. I was unable to upgrade to the latest version.
This was my solution. After this compilation and rpm installation was successful .
git clone git://github.com/creationix/nvm.git ~/nvm
echo "source ~/nvm/nvm.sh" >> ~/.bashrc
source ~/.bashrc
nmv
nvm_install_latest_npm

Awesome, glad that helped you uncover the issue!  I'll go ahead and close this out, let us know if you run into any other issues.


[Licensing] Make clear licensing of Python code and downloaded plugins

As mentioned by @jsspencer in hande-qmc/hande#2, it is not clear by skimming through Autocmake-related files what their license is. I suggest to prepend the Python code and the Autocmake plugins with the BSD-3-Clause license header. In addition, it might also be a good idea to generate or download a brief README.md in the cmake directory that makes clear that the Autocmake project is used to generate parts of the CMake infrastructure. I propose the following as content:
The CMake infrastructure for this project is generated using [Autocmake] by Radovan Bast, Jonas Juselius and contributors.
The `update.py` Python script and the contents of the directories `autocmake` and `downloaded` are licensed under the terms of the [BSD-3-Clause license], unless otherwise stated.

[Autocmake]: http://autocmake.readthedocs.io/en/latest/
[BSD-3-Clause license]: https://tldrlegal.com/license/bsd-3-clause-license-(revised)

Thanks. Very good suggestion. Working on this.


Why are we patching phpunit, since this makes it a lot harder to use a recent phpunit package?

I'm using a recent phpunit package, and the patches from this repo don't apply on that version.
I have to add this to my composer.json file for a composer install to work.
        "patches-ignore": {
            "yiisoft/yii2-debug": {
                "phpunit/phpunit": {
                    "Fix PHP 7 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch",
                    "Fix PHP 8 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php8.patch"
                }
            },
            "yiisoft/yii2-faker": {
                "phpunit/phpunit": {
                    "Fix PHP 7 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch",
                    "Fix PHP 8 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php8.patch"
                }
            },
            "yiisoft/yii2-gii": {
                "phpunit/phpunit": {
                    "Fix PHP 7 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch",
                    "Fix PHP 8 compatibility": "https://yiisoft.github.io/phpunit-patches/phpunit_php8.patch"
                }
            }
        }

But every time I try to bump my dependencies to a latest version, there is an extra yii package that adds the patches. Recently, yii2-swiftmailer started to include these patches.
Without the patches-ignore, I get this when doing a composer install.
  - Installing phpunit/phpunit (8.5.20): Extracting archive
> post-package-install: cweagans\Composer\Patches->postInstall
  - Applying patches for phpunit/phpunit
    https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch (Fix PHP 7 compatibility)
patch '-p1' --no-backup-if-mismatch -d '/opt/app/vendor/phpunit/phpunit' < '/tmp/626fc89f174ff.patch'
patching file src/Util/Getopt.php

Hunk #1 FAILED at 35.
Hunk #2 FAILED at 94.
Hunk #3 FAILED at 139.

3 out of 3 hunks FAILED -- saving rejects to file src/Util/Getopt.php.rej

patch '-p0' --no-backup-if-mismatch -d '/opt/app/vendor/phpunit/phpunit' < '/tmp/626fc89f174ff.patch'
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/src/Util/Getopt.php b/src/Util/Getopt.php
|index ba21be3..96931a3 100644
|--- a/src/Util/Getopt.php
|+++ b/src/Util/Getopt.php
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.

3 out of 3 hunks ignored

patch '-p2' --no-backup-if-mismatch -d '/opt/app/vendor/phpunit/phpunit' < '/tmp/626fc89f174ff.patch'
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/src/Util/Getopt.php b/src/Util/Getopt.php
|index ba21be3..96931a3 100644
|--- a/src/Util/Getopt.php
|+++ b/src/Util/Getopt.php
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.

3 out of 3 hunks ignored

patch '-p4' --no-backup-if-mismatch -d '/opt/app/vendor/phpunit/phpunit' < '/tmp/626fc89f174ff.patch'
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/src/Util/Getopt.php b/src/Util/Getopt.php
|index ba21be3..96931a3 100644
|--- a/src/Util/Getopt.php
|+++ b/src/Util/Getopt.php
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.

3 out of 3 hunks ignored

   Could not apply patch! Skipping. The error was: Cannot apply patch https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch

In Patches.php line 326:

  [Exception]
  Cannot apply patch Fix PHP 7 compatibility (https://yiisoft.github.io/phpunit-patches/phpunit_php7.patch)!

I have a "phpunit/phpunit": "^8.0" in my require-dev section. Is Yii2 only supported with phpunit 4.8, as this repo seems to indicate?

That's a bit weird. These are dev dependencies so the are installed only when you install the packages as a root and not otherwise. If you don't need patches, remove them from your root package along with cweagans/composer-patches.


Add support for .Net Core

Related https://github.com/reactive-streams/reactive-streams-dotnet/issues/18

@alexvaluyskiy looks like there was some sort of failure with getting the NUnit binaries running nicely on .NET Standard, judging from the error message

@Aaronontheweb I run tests only for net45 target, because NUnit does not support NetCore+msbuild at the moment

@viktorklang looks like this change is good to go. Adds support for .NET Standard 1.4, which should allow Reactive.Streams to be consumed in .NET Core going forward. Preserves the existing .NET 4.5 runtime also.

@alexvaluyskiy makes sense.

@Aaronontheweb Actually for netstandard1.0

but only Reactive.Streams package, all others (Unicast and TCK) will target 1.4 right?

@Silv3rcircl3 yes


Flutter Engine must have a consistent native thread stack size for the root isolate thread.

Currently, the engine launches the thread that hosts the root isolate. The stack sizes are left at the defaults used by the platform. This is currently 512KB on iOS and 1MB on Android. While Flutter provides no guarantee of a specific stack size available to Dart applications, it might be desirable to have some consistency. Since iOS uses 1MB for the main thread, this seems like a fine default.

I tried nested containers to see the Dart behavior across these platforms/build modes:
On Android emulator (x86): ~4200 nested containers with stack frame count 17936.
On Android Device (arm64) debug mode: ~4100 nested containers with stack frame count: 16718.
On Android Device (arm64) release mode: ~6100 nested containers with stack frame count: 24724.
On iOS simulator (x64): ~700 nested containers with stack frame count: 3254.
On iOS device (arm64, iOS 14.1) debug mode: ~800 nested containers with stack frame count: 3369.
On iOS device (arm64) release mode: ~1200 nested containers with stack frame count: 5075.
 Widget _buildContainers(int n) {
    if (n == 0) {
      return Container(child: Text('Testing'));
    }
    return Container(child: _buildContainers(n - 1));
  }

This should get higher priority. on macOS main thread stack size is 8MB. With 512kb on a moderately complex application the widget inspector will often fail with a stack overflow.
Unfortunately fml::Thread right now is backed by std::thread, which does not allow configuring the stack size, so we'll likely need per platform thread implementation (at least for platforms where non-default stack size is desirable).


Removing a few more non-trivial struct memsets.

This structure (tsi_ssl_client_handshaker_options) has an explicit constructor that sets everything to zeroes.

Failed tests are unrelated.


redid documentation and added spicetify

Merged in https://github.com/KatieUmbra/rose-pine-dots/pull/3


invalid translation from xmm0 to zmm0 breaking decompilation

Discussed in https://github.com/Vector35/binaryninja-api/discussions/4343

Originally posted by sunbo2001 May 25, 2023

The 'chart_course' function is supposed to take only one argument, arg1, but the parsing results in a second argument, zmm0. Now I Edit the Function with 'Edit Function Properties' :

I unchecked zmm0,but it still appears：

another question is function 'atof’,:

as you see, the return register of this function should be xmm0，and I also used the same way to edit function properties,but nothing happen.

in next picture, the arg2 should be replaced by var_78.

is it a bug?
the binary file is here:
https://github.com/guyinatuxedo/nightmare/blob/master/modules/08-bof_dynamic/fb19_overfloat/overfloat
TL;DR

At offset 0x40094a, in LLIL there is the correct return value of atof into xmm0, but in MLIL it's translated into zmm0 which isn't tracked as being the return value and thus incorrectly shows as a parameter to chart_course instead.

Looks like we also have https://github.com/Vector35/binaryninja-api/issues/4027 which this is a duplicate of so you can track when that is resolved and this should also be fixed.


API documentation for AuthenticateAsync is missing Exception information

Type of issue
Missing information
Description
[Enter feedback here]
When the user cancels out of the OAuth process this method throws a TaskCanceledException. This is not currently documented.
Page URL
https://learn.microsoft.com/en-us/dotnet/api/microsoft.maui.authentication.webauthenticator.authenticateasync?view=net-maui-8.0&source=docs#microsoft-maui-authentication-webauthenticator-authenticateasync(microsoft-maui-authentication-webauthenticatoroptions)
Content source URL
https://github.com/dotnet/dotnet-maui-api-docs/blob/live/docs/xml/Microsoft.Maui.Authentication/WebAuthenticator.xml
Document Version Independent Id
4706e5e5-fdb0-11ea-4755-c16525a1b538
Article author
@dotnet-bot
Metadata

ID: 4706e5e5-fdb0-11ea-4755-c16525a1b538
Service: dotnet-mobile
Sub-service: dotnet-maui

I think this one is added in code now too @peterfoot ?
Here: https://github.com/dotnet/maui/blob/8cd9239c283c1ba7398a8f6f3b063089582db3ff/src/Essentials/src/WebAuthenticator/WebAuthenticator.shared.cs#L21
Just needs publishing probably?

Thanks, happy to close this now. Why is it tagged Windows though? - the TaskCanceledException is used on iOS and Android too.

@peterfoot is that right? The only evidence could find in our code was this this is only happening on Windows? But happy to be proven wrong :)

We're using the Xamarin Essentials WebAuthenticator and if the user exits the flow (tapping X or Close) in either iOS or Android it throws this exception. We were initially surprised by it as it wasn't documented and I don't think TaskCanceledException is normally used for user interactions.

Interesting. A search for that exception also on Essentials only shows UWP code: https://github.com/search?q=repo%3Axamarin%2FEssentials TaskCanceledException&type=code
Maybe it doesn't originate from .NET MAUI/Xamarin.Essentials then? Anyway, I will open another PR to take off the Windows specific part. I'd rather have people be extra careful on all platforms with this and document too much than too little. Thanks for this!

Looking at the iOS code it is:-
else if (error.Domain == asWebAuthenticationSessionErrorDomain && error.Code == asWebAuthenticationSessionErrorCodeCanceledLogin)
tcsResponse.TrySetCanceled();
else if (error.Domain == sfAuthenticationErrorDomain && error.Code == sfAuthenticationErrorCanceledLogin)
tcsResponse.TrySetCanceled();
else
tcsResponse.TrySetException(new NSErrorException(error));
Setting the task canceled here results in the TaskCanceledException as the consumer of the API. I haven't looked at the Android equivalent but I suspect it is using TrySetCanceled somewhere too.

Ah I see yeah thats why it didn't show up in my simple search. Anyway, merged a PR for changing this on MAUI API docs so that should be all good then now!


Exclude Mac OS .DS_Store files

👎
This ignore is in the macOS specific file. It is unrelated to node.


[FEATURE REQUEST] Add option to resize  EXCALIDRAW embeddings via "| size" syntax via drag-resize or scroll-wheel

example
original
! [[Drawing 2024-01-18 13.27.33.excalidraw]]
Complete the resizing
! [[Drawing 2024-01-18 13.27.33.excalidraw|200]]

Excalidraw is not an image, thus seems redundant to support such feature in this plugin. Probably better place would be to contact Zsolt himself to directly implement it in his plugin.


Update file change

The schema didn't match with other rest api specs available. Azure portal Iaasexp team uses these to generate models for their UI.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

You don't have permission to trigger SDK Automation.
Please add yourself to Azure group from opensource portal if you are MSFT employee,
or please ask reviewer to add comment *** /openapibot sdkautomation ***.
Please ask<EMAIL_ADDRESS>(or NullMDR in github) for additional help.

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

Can one of the admins verify this patch?

@joybroto Can you please check CI errors whether you need to update the examples?
##[error]Validating "examples" and "x-ms-examples" in  specification/servicefabric/resource-manager/Microsoft.ServiceFabric/stable/2018-02-01/cluster.json

Close this now as no response. Please feel free to reopen when you plan to continue your work.


Extend gltf viewer example to showcase PBR

perhaps IBL as well

Implemented in FF: https://github.com/JuanDiegoMontoya/Frogfood


Feature request: Specify a version to update to

In order to reproduce an environment at a specific version of CoreOS for stability / debugging purposes, it is currently necessary to upload the base image @ that version into our image store (OpenStack Glance, ESX, etc). Due to the security setup of our environment, this is a prohibitively difficult and expensive task that involves many manual steps.
If the update_engine_client could be given a parameter --update-to=XXX.X.X, then we could boot from older images and use the cloud-config file to direct update_engine to download a specific CoreOS version.

I second this feature request. To be able to specify a version number for update is required in deployments where a specific update needs to be selected based on compatibility with the Docker, etcd and other software packaged in that particular update.

@utyagi24 assuming you are using the daemons provided with the OS, you can be sure that they are compatible with one another. That portion of the OS is updated atomically.

When update engine grabs an update from the server, the client is not the one to determine which version gets downloaded. The server tells the client where to find the update payload and it's version. Reversing this role would require significant changes to the Omaha protocol, invalidating any security vetting done up to this point. It's just not feasible for us to make this change.
If control is truly needed over the exact version of CoreOS your cluster is running, it's a heck of a lot easier and more powerful to use CoreUpdate.

@crawford My applications use Docker remote API, depend on etcd API etc.  With any CoreOS update all these API version are subject to change and many enterprise applications demanding high reliability are not going to be comfortable with shifting grounds underneath.

@utyagi24 I see. Then in that case, CoreUpdate is essential.


chore(docker): add -DFORCE_CUDA=1 option to strictly build CUDA related packages

Description

[ ] https://github.com/autowarefoundation/autoware/pull/5446

Following https://github.com/autowarefoundation/autoware/pull/5446#pullrequestreview-2441317201, this PR adds the -DFORCE_CUDA=1 option when performing the colcon build for universe-sensing-perception-devel-cuda, ensuring that the build is not skipped and always executed. This guarantees that the -devel-cuda image has the CUDA development libraries installed.
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@youtalk
Thanks for this PR! DId you start refactor universe packages? If so, please close this PR in favor of your changes.
Otherwise let's merge this PR and then we can check if CI fails in linked PR.

I mistakenly thought the FORCE_CUDA option was already supported by default. When I tested it on a local container, the option was ignored. I will try a different method to enforce the build of CUDA packages.


Resources with i18n in chinese breaks controllers & routes generation

The routes and controllers are generated using the i18n translations. This approach fails when translation have non safe characters (such as accents, or are written in Chinese).
Naming should use a "safe_name" for underscored and camelcased strings.

Pull request #800.

+1, route generation is broken with spanish translations that have "tilde" characters (á,é,í...)

@bbonamin Can you confirm that it works fine using https://github.com/pcreux/active_admin/tree/patch-issue-799 ?

I changed the reference to the git repo in my Gemfile: gem 'activeadmin', :git => 'https://github.com/pcreux/active_admin.git'.
Then I used the command bundle update activeadmin
And finally I tried to generate the assets: rails g active_admin:assets but the error is still present: https://gist.github.com/1422767
If I remove the tildes from the model translations in my yml file, it works again. Did I miss anything?

Yup! You missed the branch! :)
Add this to the gem file: :branch => "patch-issue-799"
Thanks for testing this!

It works perfectly!
2011/12/2 Philippe Creux <
<EMAIL_ADDRESS>

Yup! You missed the branch! :)
Add this to the gem file: :branch => "patch-issue-799"
Thanks for testing this!

Reply to this email directly or view it on GitHub:
https://github.com/gregbell/active_admin/issues/799#issuecomment-2990560

Perfect! Thanks for confirming this! :)

Works great with i18n pluralization as well (:one, :few, :other). Without this patch, internationalized and pluralized models will have named routes that are improperly generated.

fatal: remote error:
https://github.com/pcreux/active_admin.git is not a valid repository name


XMake stucked with C++20 modules

Xmake Version
v2.7.2+HEAD.d88193698,
Operating System Version and Architecture
Windows 11 Version 22H2 22623.875
Describe Bug
When building a large project with C++20 modules, XMake stuck indefinitly as you can see on this vcperf

vcperf.txt
(you need to rename to .json and use a chrome trace viewer like ui.perfetto.dev)
i got the callstack and XMake seems to stuck on
xmake b -j1
<subprocess: cmd>: cmd /c ver
<subprocess: git>: git rev-parse --abbrev-ref HEAD
<subprocess: git>: git rev-parse --verify HEAD
[xmake]: [engine]: stack traceback:
        @programdir\core\base\hashset.lua: in for iterator 'for iterator'
        @programdir\modules\private\async\jobpool.lua:189: in method '_genleafjobs'
        @programdir\modules\private\async\jobpool.lua:189: in method '_genleafjobs'
        @programdir\modules\private\async\jobpool.lua:185: in for iterator 'for iterator'
        @programdir\modules\private\async\jobpool.lua:189: in method '_genleafjobs'
        @programdir\modules\private\async\jobpool.lua:189: in method '_genleafjobs'
        @programdir\modules\private\async\jobpool.lua:189: in method '_genleafjobs'
        @programdir\modules\private\async\runjobs.lua:183: in function <@programdir\modules\private\async\runjobs.lua:162>
        @programdir\core\base\scheduler.lua:520: in function 'base/scheduler.co_group_begin'
        @programdir\modules\private\async\runjobs.lua:183: in function <@programdir\modules\private\async\runjobs.lua:162>
        ...     (skipping 2 levels)
        @programdir\actions\build\main.lua:108: in global '_do_build'
        [C]: in function 'xpcall'
        @programdir\core\base\utils.lua:280: in function 'base/utils.trycall'
        @programdir\actions\build\build.lua:257: in function <@programdir\actions\build\build.lua:245>
        (...tail calls...)
        @programdir\core\sandbox\modules\try.lua:117: in global 'try'
        @programdir\actions\build\main.lua:147: in function <@programdir\actions\build\main.lua:113>
        (...tail calls...)
        [C]: in function 'xpcall'
        @programdir\actions\build\main.lua:147: in function <@programdir\actions\build\main.lua:113>
        (...tail calls...)
        @programdir\core\sandbox\modules\try.lua:117: in global 'try'
        [C]: in function 'xpcall'
        @programdir\actions\build\main.lua:155: in function <@programdir\actions\build\main.lua:149>
        [C]: in function 'xpcall'

so maybe here there is a infinite loop (maybe something is wrong with my project, but XMake should not be stuck)
Expected Behavior
XMake build or fail, but not stucked
Project Configuration
you can try with my projet
> git clone https://github.com/TapzCrew/StormKit.git
> cd StormKit
> xmake f "--kind=static" "--mode=release" "--plat=windows" "--arch=x64" "--enable_wsi_x11=no" "--enable_applications=no" "--enable_wsi_wayland=no" "--enable_engine=no" "--enable_entities=yes" "--enable_log=yes" "--enable_examples=no" "--unity_build=no" "--enable_wsi=yes" "--enable_image=yes" "--enable_gpu=yes"

> xmake b

Additional Information and Error Logs
No response

oh i think it's because i have a circular dependency in my project

xmake f "--kind=static" "--mode=release" "--plat=windows" "--arch=x64" "--enable_wsi_x11=no" "--enable_applications=no" "--enable_wsi_wayland=no" "--enable_engine=no" "--enable_entities=yes" "--enable_log=yes" "--enable_examples=no" "--unity_build=no" "--enable_wsi=yes" "--enable_image=yes" "--enable_gpu=yes"

invalid options for --enable_applications=no, --enable_engine=no, ...
main branch?

I checkout modulify branch. It does not stuk, but it has other errors.

I checkout modulify branch, it does not stuk for me, but it has some other errors.

I checkout modulify branch, it does not stuk for me, but it has some other errors.
msvc has only std.core, not std.

i know, std module is mine not msvc one so it should work as it work on my machine 🤔 (https://asciinema.org/a/CVvpPlLRijP3GPYnqmGBWQ9bR)

ah maybe it's because i'm using VS 17.4 preview 6, i'll try with latests not preview msvc

https://asciinema.org/a/CVvpPlLRijP3GPYnqmGBWQ9bR)

Have you fixed the stuck problem? I saw it work in your video.

https://asciinema.org/a/CVvpPlLRijP3GPYnqmGBWQ9bR)

Have you fixed the stuck problem? I saw it work in your video.

at the end, xmake is stuck in an infinite loop (with the  print table)
i made a small example simple_test.zip

OK, I'll improve it by providing more friendly error messages, and you can modifying the module code to avoid circular dependencies first.

try this xmake update -s github:xmake-io/xmake#moduledeps
please remove build first.
$ rm -rf build/;xmake
[  0%]: generating.cxx.module.deps Foo.mpp
[  0%]: generating.cxx.module.deps Foo2.mpp
[  0%]: generating.cxx.module.deps Foo3.mpp
[  0%]: generating.cxx.module.deps main.cpp
error: circular modules dependency(Foo2, Foo, Foo3, Foo2) detected!
  -> module(Foo2) in Foo2.mpp
  -> module(Foo) in Foo.mpp
  -> module(Foo3) in Foo3.mpp
  -> module(Foo2) in Foo2.mpp

it work fine !


Add builds for missing architectures.

Compared to InfluxDB, Telegraf and Chronograf, Kapcitor is missing some build targets. We need to get Kapacitor up to parity with them.

This has been fixed, it was an issue unrelated to the build process in this repo.


Add cli execution

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #4   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      8    +4
  Lines         100    156   +56
=====================================
+ Hits          100    156   +56

Impacted Files
Coverage Δ

output/writer.go
100% <ø> (ø)
:arrow_up:

algorithm/factory.go
100% <100%> (ø)

output/factory.go
100% <100%> (ø)

input/factory.go
100% <100%> (ø)

transform/map_reduce.go
100% <100%> (ø)
:arrow_up:

input/reader.go
100% <100%> (ø)
:arrow_up:

algorithm/sort.go
100% <100%> (ø)
:arrow_up:

transform/factory.go
100% <100%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


MGMT-16687 Use additional partition for /var/lib/containers

@javipolo note that 4.15.2 is available and have this fix:
MGMT-16980: Change the default value of ENABLE_SKIP_MCO_REBOOT to false (#6005) #6005
So please update the default target version and set the default behavior for preaching to use the additional partition.

Please update the default target version and set the default behavior for preaching to use the additional partition.

Done
I also added the ability to select whether to use the partition or directory method for preaching with an environment variable, defaulting to partition :)
I separated it in a different commit so it's easy to review the new changes
https://github.com/rh-ecosystem-edge/ib-orchestrate-vm/pull/39/commits/b7d434e9689afc70ff85436798bcda7732d52ac4


Updating Device Firmware using Windows Update: sample code and questions.   If  SSD WU is recommended by microsoft?

[Enter feedback here]

BUG 17170213
If  SSD firmware WU is recommended by microsoft?

The last 3rd bullet:

In the diagram immediately above, the firmware driver can look for the parent device to find the primary device with which it needs to communicate

How to look for the parent device to find the primary device with which it needs to communicate and how to communicate?  ( sample code?)

How to get/scan disks’ instance? How to send command to them?  (can use UMDF2 ? or need to useKMDF? )

After finish install firmware update driver, there are 2 devices ( please check the attached pic): is it expected?

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

ID: 07d25e12-2213-9c84-2fe9-ee2f32e82a27
Version Independent ID: 6ff8eec2-25d4-f1af-a155-74cfa99ad7df
Content: Updating Device Firmware using Windows Update
Content Source: windows-driver-docs-pr/install/updating-device-firmware-using-windows-update.md
Service: unspecified
Product: windows-hardware
GitHub Login: @tedhudek
Microsoft Alias: windowsdriverdev

This was handled separately.

Sorry， I do not understand your comment: could you please elaborate? any direct answer?
Thanks!
发件人: Ted Hudek<EMAIL_ADDRESS>发送时间: 2018年6月14日 13:41
收件人: MicrosoftDocs/windows-driver-docs<EMAIL_ADDRESS>抄送: Jing Cai<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>主题: Re: [MicrosoftDocs/windows-driver-docs] Updating Device Firmware using Windows Update: sample code and questions. If SSD WU is recommended by microsoft? (#534)
This was handled separately.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-driver-docs/issues/534#issuecomment-397431350, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Al7uZrHs4zAScGko_Rn3jSHK4GmPuQl-ks5t8spigaJpZM4UTvzl.

cai6jing,
A bug was filed on Microsoft to address this privately. Please refer to that bug for further communications on this issue.
Thanks,
Nabil


deal with the UNK token

Solve the issue #2 if OOV exists.

This change would break using pre-trained BERT because the token ids will be off-by-one from the ones BERT sees during pre-training.


Closing the write and watch stream after 60s of idleness

This is port of the Android CLs cr/167321880 and cr/168294709.
This change is unfortunately pretty large:

It ports the Idle timeout logic in RemoteStore and AbstractStream to the iOS equivalent classes.
It moves the initialization of the stream delegate from the stream constructor to 'start'. This allows us to set a new delegate every time we call start.
It allows us to abandon both the delegate as well as the stream. This is useful for streams that we consider closed but that still might send messages.
It adds a new common delegate type that is shared between both the write and the watch stream and forwards calls to the remote store. This allows us to use the delegate in the base class (FSTStream).
To make testing easier and to not have to expose even more of the private API, it gets rid of the beta serializers.
It moves the StreamTest to integration tests and uses the actual streams to tests the handshake semantics.
It adds a new test-only dispatch queue that schedules the idle timeout after one second.

I've split up this change into the test rewrite (https://github.com/firebase/firebase-ios-sdk/pull/391) and this, which is now set to merge against #391 branch (for easier review).
I also split up the delegate handling for GRPC callbacks and Remote Store callbacks and reverted the porting the Stream callback structure from Android. By renaming the handleStreamOpen/handleStreamClose to notify*, I think was able to overcome the part of the design that I did not really fancy.
Let me know what you think!

@wilhuff I merged in the recent changes and updated this PR.


Fix wrong parameter name reference for test timeout in live tests

Fix for issue introduced in https://github.com/Azure/azure-sdk-for-net/pull/29544

/azp run net - template - tests


Support for disabling private messages

We have had Zulip running for some time and we love it, however, we are developing a serious problem with Group PM's and Private Messages
We noticed our productivity dropping a little bit, so a bit of sleuthing showed that we had a large number of Group Pm's going and a huge number of Private Messages, which circumvents teh whole point of Team communication via Group Messages ..
We work in a culture where it is Ok to do this in a hidden manner, whilst we have company polices in place that do not allow this and to limit it to breaks and personal time, we find it is being severely abused by most of the staff ..
Can anyone suggest a quick and easy way to disable both of these features ??

Interesting.  My experience is that in most teams, (group) private messages make up a significant fraction of communication on Zulip, and it's often about work (e.g. discussing the details of a problem with a colleague), so I'm not sure disabling these features is the right solution, but I also think that's your judgement to make, not mine :).
If you're willing to deploy a hack (not removing the UI for private messages, just making sending them not work), this commit should work, but clunkily (The users won't get a clear error message): https://github.com/timabbott/zulip/commits/disable-private-messages-hack.
If you try that out you can build your own zulip release using tools/build-release-tarball and then install it using upgrade-zulip.
I'd be interested in hearing back whether you find this to solve the problems on your team.

Tim, I agree, with your sentiments and I am sure there is a percentage that is work and project related, but it appears that these discussions are not shared with the rest of the team, so we end up, apparently, with redundant discussions which does waste a lot of time. (Apart form teh high level of personal discussions)
We are attempting to engender a culture where all discussions are done in streams so that all team members realize a benefit and can contribute as well to a solution or offer useful information
When we moved from Slack to Zulip, our ratio of Slac Dm's to Group Messages was 90 to 1, ie: 90% of messages were Dm's. We found the issue after noticing a direct impact on the team productivity and some issues with duplicated effort.
Initially with Zulip we noticed a huge improvement, but over past weeks, our teams are moving more and more away from Stream messaging  to PM's. This was evident in the volume of stream messages visibly dropping.
Our thought at the moment is to disable it and see what the result is.
Thanks for the Hack, we will give it a shot in the next week or so when I get a chance to redeploy the server
I am happy to let you know the results, I will try and see if we can get some form of meaningful stats done as well so we can assess the impact ..
ie:
Total Stream Messages
Total one - one PM's
Total group PM's
Regards
Mark

Makes sense.  I definitely understand the problem -- it's really great when you can get most discussions happening in streams, but it can be somewhat tricky to train users to do so.  Good luck!

It will be an interesting exercise .. One question, is there any documentation about how the messages are structured in the database so I can dry and dig out some useful stats with out a huge effort. I have been poking around . Specifically how to identify Group Pm's, 1-1 Pm's, Streams and users

https://github.com/zulip/zulip/blob/master/zerver/models.py#L545 has some helpful exposition.
Basically you should be able to do something like this (in ./manage.py shell):
from zerver.models import *
Message.objects.filter(recipient__type=Recipient.STREAM).count()
Message.objects.filter(recipient__type=Recipient.PERSONAL).count()
Message.objects.filter(recipient__type=Recipient.HUDDLE).count()

to get the totals.  I imagine you'll also want to filter on when the message was sent, which is stored in pub_date.

Thanks heaps Tim ..

Hi tim, some updates on this ..
(I just finished upgrading zulip - note on this the script apt-get update upgraded postgresql which upgrade the con file and broke the zulip upgrade. I had to reset the listening addresses in postgresql.conf and manually change the zulip password in zulip-secrets and in postgres to get it working. Not exactly sure WHY this happened, though, I thought postgres update would keep all the configs current)
Current Stats FYI are
In [2]: Message.objects.filter(recipient__type=Recipient.STREAM).count()
Out[2]: 10127
In [3]: Message.objects.filter(recipient__type=Recipient.PERSONAL).count()
Out[3]: 34031
In [4]: Message.objects.filter(recipient__type=Recipient.HUDDLE).count()
Out[4]: 14276
So we will definitely try and restrict the DM capability. Will keep you posted
PS: This is a slow process as I have to do this over a weekend, and I only work weekends once a month usually
Cheers
Mark

Cool, thanks for the update and stats!
I'd love to get more detail (maybe in another issue) on the Zulip upgrade issues -- I'm sure other folks have similar configuration needs to you folks, and it'd be great to have your configuration be well-supported by the Zulip upgrade process.

Will do on the upgrade issues .. I am trying to build the tarsal, but I think I need a full dev environment for it. I figured out I needed nam .. but I guess there is more .. Im trying to steal this in so I don't run out of time ..

Yeah, unfortunately you need the development environment right now to build a release tarball (basically because it needs dependencies like Django installed in order to build the static assets)

ok, thanks, will see if I can get time in to do that now

A couple of questions before I pursue my organization's initiative.
Did the hack work for the OP?
In order to deploy the hack does the full dev environment need to been installed first?
What steps can be performed to change this from a hack to an admin option that can be turned off and users do not see the PM menus as well?
Thanks!

For turning it into an admin option, this tutorial: http://zulip.readthedocs.io/en/latest/new-feature-tutorial.html should cover everything one needs to know technically for adding the option itself.
Hiding the PM menus from the UI is probably pretty doable and would basically take the form of adding if statements in the templates in a few places.  Doing that would require a development environment.
The hack I described is most cleanly done via build-release-tarball from a development environment, but you can just go in and patch the Python code in an installation and then restart the server to have it take effect (or use upgrade-zulip-from-git to deploy a git commit, though I'd recommend rebasing that super-old branch of mine first).

(I'd probably tag this option as experimental in the text on the /#administration page at first since it seems like the kind of thing where we'll discover various things we missed)

@timabbott Thank you! Zulip and this thread helped us move away from HipChat. We've done some selective private messaging permissions using something like...
message_type_name == "private"
and "Full Name" not in user_profile.full_name
However, we're trying to allow messages to specific people (i.e. HR) from anyone. Is there any chance you can point us in the right direction? We're having trouble figuring out how to identify the recipient.

@timabbott nm just found it. Thanks again. For anyone else looking for a solution like this, add something like...
and<EMAIL_ADDRESS>not in message_to
to the condition. So the extended "hack" would look something like...
if  (
    message_type_name == "private" #disable private messages
    and "Full Name" not in user_profile.full_name #allowed to send pm to anyone
    and<EMAIL_ADDRESS>not in message_to #anyone is allowed to send pm to this recipient
    ):
    return json_error("Private messages have been disabled by the administrator.")

OK.  It's interesting that more folks are interested in this feature.  I definitely encourage anyone else who comes across this issue and wants it to post.  It would likely be easy to implement and maintain an official "disallow private messages" setting using roughly that approach, so if someone's willing to do the work (including adding tests), we could potentially add this as a supported feature.

Is there any news about this feature? I'm very much interested in. This is the only thing missing for me to use Zulip.

This would be a VERY useful enterprise feature

Thanks for all the feedback folks.  Closing in favor of the duplicate #6617.


More than one file was found with OS independent path 'META-INF/core.kotlin_module'.

Please perform the required changes

I've never seen this, how does it appear? When you implement a module?

It happens when I'm building/running the project with the dependencies


enh: modify version output

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 68.15%. Comparing base (ef52a35) to head (b58bdc3).

Files
Patch %
Lines

config/config.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##              3.0     #299   +/-   ##
=======================================
  Coverage   68.15%   68.15%
=======================================
  Files          85       85
  Lines        9787     9787
=======================================
  Hits         6670     6670
  Misses       2432     2432
  Partials      685      685

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


modify lldb test results formatter for upstream

Adapt build-script-impl LLDB test suite runner launch code
for the LLDB package name changes that occurred upstream.
Without this change, running the LLDB test suite will fail out with an unknown module name.

@swift-ci Please test

@shahmishal I thought we have PR checking enabled for master-next?

@shahmishal I thought we have PR checking enabled for master-next?

Nm, it took a few minutes for the UI to update and show it.

Hmm something isn't right with the PR checking here.  I'm going to merge and address.


Feature request: Prefix devices

I want to use the plugin on two different homebridge instances so I can give my daughter access to her AC. This is currently not possible as once you startup the second instance it will clash due a service that is already registered on the network with the same name. It would be nice if we could either override the name or have some sort of prefix we can configure.

Can you post the crash log, I will look what I can do

Here you go:
homebridge-mia    | [6/20/2024, 10:25:20 AM] Got SIGTERM, shutting down Homebridge...
homebridge-mia    | [6/20/2024, 10:25:20 AM] [Meross] Child bridge process ended
homebridge-mia    | [6/20/2024, 10:25:21 AM] [Meross] Process Ended. Code: null, Signal: SIGTERM
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] ---- Air Conditioner: Schlafzimmer Klima ----
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Account: Klima
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Indoor: MSZ-AP25VG
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Serial:<PHONE_NUMBER>
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Firmware: 37000
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Manufacturer: Mitsubishi
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] ----------------------------------
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Power: OFF
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Operation mode: AIR CONDITIONER OFF
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Room temperature: 24°C
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Target temperature: 20°C
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Heating threshold temperature: 20°C
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Cooling threshold temperature: 20°C
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Fan speed mode: QUIET
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Vane swing mode: AUTO
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Temperature display unit: °C
homebridge-mia    | [6/20/2024, 10:25:21 AM] [melcloudcontrol] Air Conditioner, Schlafzimmer Klima, Lock physical controls: UNLOCKED
homebridge-mia    | [6/20/2024, 10:25:21 AM] Schlafzimmer Klima 3798 is running on port 42539.
homebridge-mia    | [6/20/2024, 10:25:21 AM] Please add [Schlafzimmer Klima 3798] manually in Home app. Setup Code: 032-45-154
homebridge-mia    | [6/20/2024, 10:25:21 AM] Error: Service name is already in use on the network
homebridge-mia    |     at Registry._onProbeComplete (/homebridge/node_modules/homebridge/node_modules/bonjour-hap/lib/Registry.js:108:27)
homebridge-mia    |     at Prober.done (/homebridge/node_modules/homebridge/node_modules/bonjour-hap/lib/Prober.js:67:10)
homebridge-mia    |     at Prober.onMDNSresponse (/homebridge/node_modules/homebridge/node_modules/bonjour-hap/lib/Prober.js:61:102)
homebridge-mia    |     at EventEmitter.emit (node:events:518:28)
homebridge-mia    |     at Socket.<anonymous> (/homebridge/node_modules/homebridge/node_modules/multicast-dns/index.js:49:43)
homebridge-mia    |     at Socket.emit (node:events:518:28)
homebridge-mia    |     at UDP.onMessage (node:dgram:942:8)

Are You already try to add in MELCLoud service quest Access and use this credentials data on other instance?

@grzegorz914 yes... Is this not possible ?

This is possible but wil break all current paired account, i will prepare beta for test and publish for You, be patient.

@grzegorz914 actually I can also just create a new guest account for that. Maybe its good enough to document this ?

Please try 0.21.9-beta and let me know

Do I need to do any changes in the config ?

No need any changes just in other instance create account for quest with different name, and the all accessory need to be paired again in Home app

@grzegorz914 yes! This works :)

Nice, i will publish it today.

1.0.0 released


update CI to test release candidate

I removed the part we commented out for the release candidate test and also added some cases to the matrix, let me know if this looks correct @jaimergp :smile:

LGTM!
I posted a comment in the OpenMM issue tracker with an idea on how to automate this comment/uncomment manual switch. It involves using the Anaconda CLI to compare timestamps and only act if RCs are recent enough.
We could try it here!

I will give it a try! I think what you posted should work just fine. I'll let you know when I'm ready either for you to take a look or need some more sage advice :smile:

OpenMM 7.5.2 has been released now, so unless we have fancy logic that only tests active RCs to test, we can close this!

I'm going to keep this PR open until I add in that logic :)

See https://github.com/openmm/openmm/issues/3098#issuecomment-838224141

Also remove testing "latest" in the build matrix since that is the same as conda-forge now. So we will be testing:

latest openmm release
openmm rc (if there is one)
openmm nightly

This also drops python 3.7 testing according to NEP 29
@ijpulidos ready for review!

Oh and adds openmm 7.7 to the testing matrix

I just have some non-blocking question about why the rc-check and test have to be done that differently compared to the nightly, 7.6 and 7.7 tests. I don't pretend to understand the whole thing, since I'm not that familiar with github actions, but if you can summarize a bit of it that would be really helpful for me to understand and also learn :). Thanks!!

So 7.6 and 7.7 we just install from conda-forge channel.
For the nightly we use the omnia-dev channel to install the nightly build.
To test a release candidate we need to install openmm from the conda-forge/label/openmm_rc channel (which is technically just a label). Now we only want to install the release candidate when there is a new one, so the rc-check looks to see if there is a new one in the last 30 days, then if there is, runs our CI that installs the RC. That way we don't just keep testing the same RC until there is a new one.

So 7.6 and 7.7 we just install from conda-forge channel. For the nightly we use the omnia-dev channel to install the nightly build. To test a release candidate we need to install openmm from the conda-forge/label/openmm_rc channel (which is technically just a label). Now we only want to install the release candidate when there is a new one, so the rc-check looks to see if there is a new one in the last 30 days, then if there is, runs our CI that installs the RC. That way we don't just keep testing the same RC until there is a new one.

Ah great, I get it now, thanks for explaining this to me!


Add support for serialization of type bytearray

For all preview API versions since 2022-09-02-preview, a new property named ManagedClusterSecurityProfileCustomCATrustCertificates is added in containerservice(AKS). The property is declared with type array, where the value of each item is of type string and format byte. Thus, the generated SDK declares the property as type [btyearray]. It could be seen that the serialization helpers that comes with the SDK could handle properties of type bytearray. But the serialization processing in knack is not competent, so opened this PR to add support for serialization of type bytearray.

/azp run

/azp run


Add Issue template for Component Library

This PR adds an issue template that we can use for Sage Component Library cards

We will have to see how the templates display but the Overview h1 is not descriptive of the the contents. Maybe Overview should be the first h2.
Consider "Name: <ComponentName /> for h1"
Change:
Spec questions
To:
Spec Questions (h2 and capital Q)
And move the Spec Questions heading to the bottom
Change:
investgiation
To:
investigation (spelling)
Change States and Props to h3 (sub-headings of Details)

We will have to see how the templates display but the Overview h1 is not descriptive of the the contents. Maybe Overview should be the first h2.
Consider "Name: <ComponentName /> for h1"
Change:
Spec questions
To:
Spec Questions (h2 and capital Q)
And move the Spec Questions heading to the bottom
Change:
investgiation
To:
investigation (spelling)
Change States and Props to h3 (sub-headings of Details)

Good calls!
I'll have to see how to review this since I did it from the UI instead of CLI!

References https://github.com/HausDAO/daohaus-monorepo/issues/34 -- will need to refine this and add Checklists that we determine

Added this in directly with feedback. Going to close this PR as it's not relevant.


Request for Dataset

Hi AsteriosPar ,
I hope you are doing well. I have been exploring your repository and am very impressed with your work.
I noticed that the dataset used in your project is not included in the repository. Could you please provide access to the dataset? It would be very helpful for my research and understanding of the project.
Thank you for your assistance!
Best regards.

Hi cddc22,
The dataset is indeed not included yet. I have been working on improving it and balancing the data and hopefully it will be publicly available soon.
The repository has a pre-trained model attached that should work for cases captured with the same sensor in a range 1m - 7m.
Best,
Asterios


Cannot get daemon status

I can successfully start the daemon, but cannot get a status once it's running:
➜  ~ hyperdrive start
** Beta Notice: **

  The daemon's been started with simple telemetry enabled. To disable, restart with --telemetry false.

  Telemetry allows us to collect statistics such as uptime and the number of active connections.
  We do not send confidential information, such as drive keys, to our telemetry server.

✔ Hyperdrive daemon listening on localhost:3101
➜  ~ hyperdrive status
Could not get the daemon status:
Error

node v12.13.1
hyperdrive-daemon @beta

Just double checked hyperdrive setup:
➜  ~ hyperdrive setup
Configuring FUSE...
Successfully configured FUSE:

  * FUSE is already configured.
  * Successfully created the root Hyperdrive directory.

Restarted my machine and everything's working as expected. I'll close for now and reopen if I encounter this again.

Hmm the daemon's not giving an informative error message there eh ;) If it happens again, there are a few things you could try that'd help debugging:

Install PM2 globally (npm i pm2 -g) then do a pm2 list and see if the process called hyperdrive is running or is errored.
If it's errored, maybe see if there's anything interesting at the end of the log file at ~/.hyperdrive/log.json. Stack traces should be printed there if there's a crash.

Thanks for the issue!


Opening the browser always showing itself

Hi,
Seems the docker starts with no errors, on SSL enabled.  However, each time I try to use it, it will open the adds-on in a kiosk mode (although it is set to 0) and with the HA default dashboard in it.
Removing the SSL back no a none secured (setting it back to 0) opens it just fine.
Thanks,
Tal.

Hi,
Did you ever change FF_KIOSK?
Can you try if you have the same behavior with the upstream container?
https://github.com/jlesage/docker-firefox
Most of the environment variables are use as-it by the original container.


FinishTransaction not working on TestFlight build - iOS 15.1

Version of react-native-iap
7.5.1
Version of react-native
0.63.4
Platforms you faced the error (IOS or Android or both?)
IOS
Expected behavior
FinishTransaction call should allow consumable product on iOS to be purchasable again.
Actual behavior
FinishTransaction allows consumable product on iOS to be purchasable again in sandbox/debug modes, but not on testflight release.
Tested environment (Emulator? Real Device?)
Real device running from Xcode, testflight on real device.
Steps to reproduce the behavior
Launch app on testflight as regular QA build, finishTransaction won't work and will not allow receipts to be removed from queue.

I think this can be disregarded, seemed to be an issue on Apple's end and only occurred on some our our external tester accounts.

Cannot finish a purchasing transaction [SKPaymentQueue finishTransaction:
App crash.
RNIap.requestPurchase(
productId,
false,
userDetail_Id,
userDetail_Id
).then(async (purchase) => {
(api call ).then(()=>{
RNIap.finishTransaction(purchase, true);
// RNIap.clearTransactionIOS(); with this also tested. still app getting crashed.
}).catch(error=>{
RNIap.finishTransaction(purchase, true);
})
}).catch(error=>{
RNIap.clearTransactionIOS();
})

]

@codal-mpawar did you manage to solve this?

we're also getting crashes on ios when calling finishTransaction(product, true/false) with the following native error:
+[NSInvocation _invocationWithMethodSignature:frame:]: method signature argument cannot be nil


multistorage -  map storageclass to flashSystemCluster

multistorage feature  - map storageClass to FlashSystemCluster using secret

 Multi storage- add storageClass to storageSystem configmap mapping by comparing secret host address


Fix slowness of html loading

Fix issue https://github.com/google/syzkaller/issues/2495

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Nice!


GitHub icon redesign for "more" sidebar

GitHub has updated their icons:
New version

Old version

Example URL: https://github.com/sindresorhus/refined-github/
But the "More" dropdown's icons haven't changed:

https://github.com/primer/octicons/pull/370

So GitHub is using the new icons even though they haven't been merged yet?

@fregante this can get closed

Thanks for opening this issue! I noticed and updated the icons in #3147 and now I saw this 😄


Intent classification using Ludwig

Issues handled: #58  #57

@TeAmP0is0N This is the folder I added in which the notebook as well as the dataset used is present.
Please review those and let me know.

@TeAmP0is0N This is the folder I added in which the notebook as well as the dataset used is present.
Please review those and let me know.

@TeAmP0is0N Did u check?

@SubhradeepSS Take more datasets like this and work on those datasets.

Will the PR be merged after working on all datasets?

Yes, your PR will merge.

Yes, your PR will merge.
@TeAmP0is0N
Upto how many datasets do I need to work on?Shouldn't the PRs be separately merged for every dataset as this will count towards gssoc score?Please correct me if I am wrong

@SubhradeepSS No, you should not create diff PR. You can add upto total 5 datasets in a PR.

Issues handled: #58 #57

You are handling only issue no #58 right now. So mention only #58.

Issues handled: #58 #57

You are handling only issue no #58 right now. So mention only #58.

@TeAmP0is0N Did u check my notebook.
?I have also worked on #57

@SubhradeepSS yes, I checked your PR. in related to issue no #57 there will be more issue. #57 is the base for intent classification

Needs more work to integrate with the bot. Can't be merged as-is.


R Engine refresh update

@j-martens : Thanks for your contribution! The author, @rastala, has been notified to review your proposed change.


Option to disable line padding for intraline differences

I want to not shift lines when there are just changes on it. Example, Currently is like this:

I want to be like this:

May be added a option to disable the Minimap automatically when on diff mode?

I had experimented with this before, but ran into some issues. Definitely a good feature to have though. I'll take another crack at it as soon as I have the time. Thanks!

Added in #23


WIP: FUTURE: Default default-constructor of SymmetricSecondRankTensor

Follow-up to pull request https://github.com/InsightSoftwareConsortium/ITK/pull/4469 commit 755cd10e5421d41f4004be101c888d5ba24be8e4
"PERF: FUTURE: Default default-constructors of RGBPixel and RGBAPixel"

I assume this PR is similar to the RGBPixel improvement with the goal of improving the performance of the Image::Allocate method. Perhaps this is a case there the optimization should go into the Allocate method. In SimpleITK and when interfacing with other libraries and buffers, the Image container buffer is considered a raw buffer. Perhaps if certain conditions, c++ traits or itk PixelTraits then a special optimization in this method could occur where the buffer is initialized as some type of raw buffer, then case to the component type.
These initialized objects can be an insidious thing to track down as the behavior is not deterministic and varies with debug and release builds. So I have some concern with the proposed future changes.

I assume this PR is similar to the RGBPixel improvement with the goal of improving the performance of the Image::Allocate method.

Indeed, I have indeed seen huge ("future") performance improvements of Image::Allocate from the similar RGBPixel PR (#4469), and I would expect similar ("future") performance improvements for SymmetricSecondRankTensor, with this commit. But it's not just that. It's also a matter of consistency between pixel types:
    TPixel pixel1; // May not be initialized, after this change
    TPixel pixel2{}; // Properly zero-initialized.
    TPixel pixel3 = TPixel{}; // Properly initialized.
    TPixel pixel4 = TPixel(); // Properly initialized.

As I mentioned at https://github.com/InsightSoftwareConsortium/ITK/pull/4469#discussion_r1496080480
Note that std::make_unique<TPixel>() and std::make_shared<TPixel>() will zero-initialize the created pixel(s) anyway, also with this PR.
Slightly off-topic, I think it would help if itk::Image would have an extra member function, AllocateZeroInitialized(), which would be equivalent to Allocate(bool initializePixels) with true as argument. It seems cleared to me to explicitly write image.AllocateZeroInitialized(), instead of image.Allocate(true).  What do you think?


SCF initial guess by superposition of atomic charge densities

Dear all,
VASP and other softwares use the "superposition of atomic charge densities" as the initial guess of SCF calculation.
Is this available for PySCF?
I suppose SAP is something similar?
Best,
Zhihao

It is. Just set init_guess to atom.


feat: Add MySQL connector session and integration files

feat(dozer-ingestion): Add MySQL connector module

Added a new mysql_connector module to the dozer-ingestion library for connecting to a MySQL database.
This module provides the necessary functionality to capture data changes using MySQL CDC and send them to the Dozer channel.
The mysql_connector module includes functions for connecting to MySQL, capturing data changes, and translating change events to Dozer operations.
Also added tests for capturing data changes, translating change events, and sending operations to the channel.

need review


Annotations ref entity

This reworks how reference entities are done.

I realized that for reference entities, it doesn't matter if they're a workflow, a launch plan, or a task, the behavior we want is roughly the same.  The simplest thing to do is to mimic what a task does.  As such, I've basically copied the task's logic into a new class called ReferenceEntity, the only difference is that I've renamed dispatch_execute to unwrap_literal_map_and_execute.  There's definitely some refactoring to do in the future, perhaps even between tasks and workflows.  We know that the create_and_link_node behavior is already common across all three.
Added an execute function to the Workflow - this allows us to simplify the mocking/patch functions.

Note that the mock functions don't handle LaunchPlans yet.  This is because LaunchPlans don't have an execute function, the __call__ is delegated to the workflow.

All three reference objects now derive from the same base. Note that the constructor for ReferenceEntity does not call super() and that the Workflow/LaunchPlan/Task is always second in the inheritance order. This means that the constructor for the Workflow/LaunchPlan/Task is not called for ReferenceEntities.
Because of circular dependencies, I couldn't put ReferenceTask alongside PythonFunctionTask so I had to make a separate file for it.  The separate file needs to be loaded though for the plugin to register so I added the import to the init file.

A few comments

Looks good, but lets wait for @honnix to reply?


sophie adding admin main menu, author menu, and log menu. also added …

adding admin main menu, author menu, and log menu. also added  some changes to initial user main and user account menus

Great job Sophie1


Custom player model not rendered in portals when gravity is non-default

According to your testing, does the issue occur with ONLY Immersive Portals mod and Farbic API?
No
Describe the issue
I use a popular custom player model mod, MineLittlePony, that only affects the client side. When I disable the mod with its ingame menu, this issue disappears. It happens both in single player and on my server.
When my player gravity is set to anything but the default "down", my player model stops rendering in portals, with some odd edge cases. I will do my best to document them.
I am playing on the latest 1.19.2 version and using the Quilt version of Gravity API. I'm willing to update to 1.19.3 but only if I know it's going to solve the issue.
Video [edge cases shown starting at 0:40]:
https://www.youtube.com/watch?v=UknbwzhNAu8
Post the log here
https://pastebin.com/kb0dKhnq

It may be an issue of gravity api

It may be an issue of gravity api. Is the third person view normal when the gravity is not default?

yes, when gravity is default everything works perfectly. The strange part is that the portal does render the player model in nondefault gravity, just only some of the time.

Is the third person view normal when the gravity is not default? I need to know whether it's related to immptl.

Sorry I didn't see the video

It's a complex 3-mod compatibility issue


Alternative to weak mode?

Weak mode allowed you gradually convert a file. Now you can gradually convert the project, one file at a time, but you can't work on a file here and a file there, not without fixing them completely. If there is no alternative I believe it defeats the entire purpose of flow..and people would just use TS instead, if the hassle is almost he same.

On a related note, I'd love to be able to ignore a type of warning/error for the whole of a file in a single directive. At the moment this doesn't seem possible at all.
Use case: I have a file that generates dummy/demo data objects. For flexibility and extensibility, the ID field is generated in the main processing. It's tedious to have to declare $FlowIgnore[missing-prop] on every object. But I want to know about those errors in every other file in the project, so I can't turn it off at the project level.


Add lost in sound by ROY KNOX

Song Name
lost in sound
Artist Name
ROY KNOX
Source
Youtube
Youtube Link
3
Direct File Link
No response
Song ID
15
Start Offset [ms]
No response
Extra
Created from Auto Nong.
Created for level: 85592420

reject // YouTube link must be a direct link to a YouTube video or the string of characters following watch?v= in the URL.
i.e. https://www.youtube.com/watch?v=jNQXAC9IVRw, or simply jNQXAC9IVRw
I believe you meant to link this video, however you simply put the number 3 in place of a URL.


Seating translations

The seating app does not support language change

Uncompiled messages (developing on windows) are added in 9aefc1329759bc0302a4c7ebcd23996bab065fa0


How to retrieve the users data out of unzipped config file ?

i checked this https://github.com/osixia/docker-openldap-backup/issues/5
But i dont see any data file to restore. i see only config gz files  in format of 20201118T184501-config.gz

This answer https://github.com/osixia/docker-openldap-backup/issues/5#issuecomment-547437923 solved the problem. Although it's not clear why creating a new server image with already existing database and config mounted locations fail.


Bring CW721 pointer contract up to spec

Describe your changes and provide context

Switched CwERC721 contract to have all queries that exist on CW721 Base contract
Add support for royalties standard (ERC2981 / CW2981)

Testing performed to validate your change

Added unit tests to x/evm/client/wasm/query_test.go to test out two new EVM queries ("TotalSupply" and "RoyaltyInfo")
Added tests to contracts/test/CW721toERC721PointerTest.js that tests out new query responses

Codecov Report
Attention: Patch coverage is 33.33333% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 60.72%. Comparing base (045251b) to head (dd0a0c1).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1668      +/-   ##
==========================================
+ Coverage   60.70%   60.72%   +0.02%
==========================================
  Files         367      367
  Lines       26784    26809      +25
==========================================
+ Hits        16258    16280      +22
+ Misses       9429     9420       -9
- Partials     1097     1109      +12

Files
Coverage Δ

x/evm/client/wasm/query.go
39.16% <33.33%> (+3.23%)
:arrow_up:

... and 1 file with indirect coverage changes

Closing. https://github.com/sei-protocol/sei-chain/pull/1673 is a copy and was merged in.


Potential incompatibility with jest-lua

I'm working on a PR to migrate Flipbook over to Pesde (https://github.com/flipbook-labs/flipbook/pull/293) and when running unit tests I'm finding that Jest is erroring for all of them with the following:
  ● Test suite failed to run

    unable to find environment 'jest-environment-roblox'

      ReplicatedStorage.Packages..pesde.wally#jsdotlua+jest-runner.3.6.1-rc.2.jest-runner.runTest:160
      ReplicatedStorage.Packages..pesde.wally#jsdotlua+promise.3.5.2.promise:172 function runExecutor
      ReplicatedStorage.Packages..pesde.wally#jsdotlua+promise.3.5.2.promise:181
      ReplicatedStorage.Packages..pesde.wally#jsdotlua+promise.3.5.2.promise:1245
      ReplicatedStorage.Packages..pesde.wally#jsdotlua+promise.3.5.2.promise:172 function runExecutor
      ReplicatedStorage.Packages..pesde.wally#jsdotlua+promise.3.5.2.promise:299

It seems like Jest's traversal up its ancestry may be getting tripped up by Pesde's index structure. Here's some relevant line's from the runTest function that's failing:
https://github.com/jsdotlua/jest-lua/blob/main/src/jest-runner/src/runTest.lua#L151-L162

Good catch! The issue stems from the fact Wally nests their dependencies a level less than pesde, since they name their _Index entries as {scope}_{name}@{version} while pesde uses a subfolder such as {scope}+{name}/{version}. This will be fixed in the next release.


Add mtgfetch

https://github.com/monteluka/mtgfetch
Also put the miscellaneous section in alphabetical order.

Thanks for contribute.


[Bug] select is empty

Bug Line

Where is the bug? (line URL)
when sending an select it is not shown in discord.

Excpected Action

What did you expect the module to do?
showing selection in discord.

Actual Action

What actually happened?
nothing shows, just the content of the message is displayed.

Reproducing the action

How could you reproduce the action?
using following code nothing is display in discord except for the Message Content "Content"

options = [ Option(label='Ja',value='1',default=True), Option(label='Nein',value='0',default=False) ]
    await msg.channel.send(
        "Content",
        components=[
            Select(options=options,id=5019)
        ],
    )

Environment

What is the version of the python?
Python: 3.9.4
OS: Windows 10 (Pro)
Module: discord-components == <IP_ADDRESS>

Check

[True] Are you using the newest version?
[False] Does this bug occurs security issues?

Currently, only selected accounts can see dropdowns. Just like buttons in the past.

You can't use select in normal messages currently.
It is not released.
Although you can use it on ephemeral messages.


Issues wrapping lines with ' and "

Passing  -w causes lines with single and double quote characters to be wrapped incorrectly - both with additional spaces, and not fitting to the expected line width.
Using consolemd 0.5.0, I ran consolemd -w 40 with these two input lines:
I don't know what kind of monster we've made here, but it's definitely
of the "three arms, no legs" variety.

The output was
I don ' t know what kind of monster we ' ve made here, but it ' s definitely
of the " three arms, no legs "  variety.

... the first line of which is 76 characters long. The spaces around quotes are also unexpected.

Thank you for using consolemd and for reporting this bug.
Unfortunately I'm not sure if this is fixable...
If you run consolemd -d -w40 with that input you'll see that at each quote character a new text tag is generated. Each of the text tags is wrapped individually so in this case no text block is longer than 40 chars so wrapping won't actually occur. These tags come from commonmark and I'm definitely not opening that can of worms.
Hmm... maybe the raw text needs to be wrapped and then outputted with --no-soft-wrap. I'll give it some thought and testing.

So good news is that per-processing the text works but the simple solution merges all paragraphs into a single text blob. So each paragraph needs to be individually processed but getting that text out of commonmark is... not the best.
The good news is that extra spaces will get fixed either way.

Fixed and pushed version 0.5.1 to pypi


[BUGFIX] Corrected np.random.randint upper limit in data.stream.py

Description
Since default np.int_  is int32 for Windows, an upper limit for random seed of 2**32 rises a ValueError: high is out of bounds for int32.
with np.iinfo(np.int_).max, np.random.randint is platform-independent
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

very small fix to allow Windows users to use module (otherwise random seed is bugged)

cc @dmlc/gluon-nlp-team

Job PR-935/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-935/1/index.html

It seems that the test_dataloader test is consistently hanging in the CI. For example: http://ci.mxnet.io/blue/organizations/jenkins/GluonNLP-py3-cpu-unittest/detail/PR-935/3/pipeline#step-159-log-1116

Could be due to using 64 bit instead of 32 bit range used before. I added a commit so that np.int32 is used

np.int_ can be 64 bit, and it is 64 bit on our CI

Job PR-935/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-935/2/index.html


Cleaner approach to pointers

From what I can tell, all three fields of what makes IPFS providers have been abused for OpenBazaar pointers. Specifically,

The provider target key is replaced by a peer ID prefix key
The provider peer ID is replaced by a random magic ID
The provider addrs are replaced by the ciphertext hash

Moreover, it seems the sole purpose of the magic ID is to distinguish pointers from providers so as to give them a longer TTL.
As it stands this is a puzzling and somewhat poorly documented hack. @jackkleeman and I have been banging our heads with OpenBazaar pointers. Since OpenBazaar is forking IPFS, why not create two new DHT messages ADD_POINTER and GET_POINTERS to handle pointers?

That wouldn't change how the data is stored. It's still going to be key -> peerInfo.
The code pointers.go code handles everything that is needed to set and get pointers.
The only difference custom methods would make would be it would remove the reliance on the magic number but then that field would just be filled with random bits anyway.
Creating and maintaining a third data structure in the DHT seems unnecessary and potentially a source of problems.

It's still going to be key -> peerInfo.

Why use peerInfo? We control how data is stored on the DHT. A pointer is not a peer info.

that field would just be filled with random bits

Why? The magic number can be removed—completely. No need for random bits. Just map prefix hash -> ciphertext hash. ADD_POINTER adds a ciphertext hash. GET_POINTERS gets all ciphertext hashes.

potentially a source of problems

Why?

You have to build out an entirely different storage interface just to basically replicate 90% of the functionality that IPFS already has.
And you have to build out separate IPFS network messages and handlers and then all that needs to be maintained. It wont be as easy as a simply rebase to the latest IPFS code. You'd have to keep all these changes in line with the changes going on in IPFS.

You have to build out an entirely different storage interface just to basically replicate 90% of the functionality that IPFS already has.

The storage interface is 90% there. It's a small tweak to the existing providers storage interface.

you have to build out separate IPFS network messages and handlers

The job is 90% done. The messages and handlers are almost identical to existing messages and handlers for providers.

You'd have to keep all these changes in line with the changes going on in IPFS.

The OpenBazaar-specific changes will be minimal given the storage, messages and handlers for pointers would be almost identical to those for providers. It's almost a copy-paste job; keeping in sync should not be hard. It would be a shame to knowingly introduce technical debt at the protocol level before OB 2 is even launched.


Drop the Queue module

This pull request drops the queue module. To the best of my knowledge, the only case where it was being used was via the massmailer module and it was used to queue emails. Problem with that is our email module already does email queuing so we had two separate modules being used to queue emails, each with different behavior.
Pros

Less code to maintain.
Massmailer will now actually have limited number of tries for emails, like the default email system. Otherwise it would end up trying indefinitely
No possibility of someone being confused when the massmailer email queue doesn't have / follow the same settings as the normal email settings

I've also added the ability to see the email queue from the email settings:

Cons

Having a generic queue module is actually useful and technically that's what the existing queue module does, however it's imo quite poor at that as it seems somewhat incomplete, is disabled by default, and is entirely separate from the core software. I also believe there are existing packages we can use which would allow us to build a better version than what currently exists.

It's not important for this PR, but I can't help feeling the functionality provided by the Massmailer module should really just be an option in the Client module. Perhaps a further merging of functionality could be done down the line?

Massmailer module should really just be an option in the Client module

Eh.. I'd probably disagree. It's both a way to communicate info on mass and also a tool that can be used for sending marketing / promotional emails. In my opinion, it makes sense to have it separate & on it's own.

You likely know the functionality better than me - I based my opinion on it only being used to communicate with clients. That said, I don't feel too strongly either way.

Honestly I didn't have a use for it until today, which is why I ended up sitting down to clean things up. First time I have used it outside of just verifying the test email will send, lol.
I also don't feel strongly about where it is, but personally I do think that it's best in it's own module.

This makes sense to me and the changes look good.
Regarding...

In my opinion, it makes sense to have it separate & on it's own.

I share this opinion.


Register Package

JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/2308
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" b87d9460c1eb319979b5140eade7ffba4f0084cb
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/2420
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" f9a0732c71fa70e8541c91293f055a89f1baed15
git push origin v0.1.0


Automatically create role for reading routine DB in WebApplication

See https://github.com/intersystems/ipm/pull/562#issuecomment-2344012005 and https://github.com/intersystems/ipm/pull/562#issuecomment-2343662304
We should allow a syntax (such as ${codeReadRole}) in WebApplication resource. IPM should create this role (if nonexistent) and set add this role to the WebApplication.

@isc-shuliu thanks for spinning this out as a new issue - I agree that's most appropriate.

After discussion with @isc-kiyer , it's best to create a <Role> resource instead of using a ${codeReadRole}. Will close this one.


website not running locally on windows 32 bit machine

Expected behavior:
Actual behavior:
Steps to reproduce:
Screenshots (if applicable):
OS and Browser:

@swekshajha12 is this still an issue?

@swekshajha12 I'd close this issue since it lacks the details. If you're still facing it, please reopen.


Changing onyo.assets.name-format while having assets does not rename assets

When changing the keys that the asset filename is constructed of, the change is not applied to all assets. Furthermore, it now becomes impossible to edit assets using the old filename scheme.
For example, the .owned suffix was changed to .basegame (i.e., the onyo.asset.filename changed from {shortname} ({year}).{status} to {shortname} ({year}).{type}) resulting in:
❱ onyo edit "7-wonders (2010).owned"
ERROR: No such asset: /home/niels/onyo/boardgames/7-wonders (2010).basegame
Cancel command (y) or continue editing asset (n)? n

It is furthermore not possible to set the filename by changing the type key:
❱ onyo set -k type=base -p "7-wonder (2010).owned" --rename                                                                                1 !
ERROR: The following paths aren't assets:
/home/niels/onyo/boardgames/7-wonder (2010).owned

onyo.assets.filename -> onyo.assets.name-format (#632 #572)


Expose max_id in user_followers API

Getting all of the followers of a user in one go is not practical for users with a lot of followers. Exposing the max_id used in the internal user_followers would make it easier to continue from where it was left at a later point. This would be useful for getting different followers each time the code is run.
That being said I am not sure if this would work, or how long the max_id is valid.

@flatplate I have implemented user_followers_v1_chunk which will give a list of users and max_id
However, Instagram via the private API (mobile app) stopped giving out all followers, it gives out a maximum of 200 users, it is considered that you no longer need to give
But the web api still gives all followers, I will try to implement the method based on it

@flatplate added user_followers_gql_chunk and user_followers_gql, which give out a full list of followers through the web api, while user_followers_gql_chunk also gives out a cursor (alternative of max_id), you can set some max_amount (for example, 100) and get a cursor, later use it to continue the selection


Make host status persistent across restarts

Traffic Server doesn't retain the host status ( up/down) across Traffic Server restarts. This PR uses stats(records.snap) to retain the host status. Although stats is not the right place to store this information as HostDB is the process of being rewritten, records.snap seems to be a temporary alternative. This is a PR to fix issue #5248.

The PR breaks SocksProxy.
The defaults socks server is set to UP (status: 2) during loadSocksConfiguration() which called by netProcessor.init():
Traffic Server 9.0.0 Apr 29 2019 20:13:46 bogon
traffic_server: using root directory '/usr/local/ats'
[Apr 30 00:15:33.600]  DEBUG: <DNS.cc:1785 (ink_dns_init)> (dns) ink_dns_init: called with init_called = 0
[Apr 30 00:15:33.600]  DEBUG: <Socks.cc:474 (loadSocksConfiguration)> (Socks) Socks Version 5
[Apr 30 00:15:33.600]  DEBUG: <Socks.cc:484 (loadSocksConfiguration)> (Socks) server connect timeout: 10 socks respnonse timeout 15
[Apr 30 00:15:33.600]  DEBUG: <Socks.cc:495 (loadSocksConfiguration)> (SocksProxy) Read SocksProxy info: accept_enabled = 1 accept_port = 10800 http_port = 80
[Apr 30 00:15:33.600]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_manual, id: -1
[Apr 30 00:15:33.600]  DEBUG: <HostStatus.cc:249 (createHostStat)> (host_statuses) stat name: proxy.process.host_status.<IP_ADDRESS>_active, id: 1
[Apr 30 00:15:33.600]  DEBUG: <HostStatus.cc:249 (createHostStat)> (host_statuses) stat name: proxy.process.host_status.<IP_ADDRESS>_local, id: 2
[Apr 30 00:15:33.654]  DEBUG: <HostStatus.cc:249 (createHostStat)> (host_statuses) stat name: proxy.process.host_status.<IP_ADDRESS>_manual, id: 3
[Apr 30 00:15:33.654]  DEBUG: <HostStatus.cc:249 (createHostStat)> (host_statuses) stat name: proxy.process.host_status.<IP_ADDRESS>_self_detect, id: 4
[Apr 30 00:15:33.654]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_manual, id: 3
[Apr 30 00:15:33.654]  DEBUG: <HostStatus.cc:168 (setHostStatus)> (host_statuses) set status up for :  name: <IP_ADDRESS>, status: 2, reason_stat: proxy.process.host_status.<IP_ADDRESS>_manual
[Apr 30 00:15:33.654]  DEBUG: <HostStatus.cc:177 (setHostStatus)> (host_statuses) name: <IP_ADDRESS>, status: 2
[Apr 30 00:15:33.673]  DEBUG: <Socks.cc:502 (loadSocksConfiguration)> (Socks) Socks Config File: /usr/local/ats/etc/trafficserver/socks.config
[Apr 30 00:15:33.676]  DEBUG: <Socks.cc:530 (loadSocksConfiguration)> (Socks) Socks Turned on

The defaults socks server is set to DOWN (status: 1) by HostStatus::instance().loadHostStatusFromStats().
[Apr 30 00:15:33.677]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_active, id: 1
[Apr 30 00:15:33.677]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_active, id: 1
[Apr 30 00:15:33.677]  DEBUG: <HostStatus.cc:172 (setHostStatus)> (host_statuses) set status down for :  name: <IP_ADDRESS>, status: 1, reason_stat: proxy.process.host_status.<IP_ADDRESS>_active
[Apr 30 00:15:33.677]  DEBUG: <HostStatus.cc:177 (setHostStatus)> (host_statuses) name: <IP_ADDRESS>, status: 1
[Apr 30 00:15:33.678]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_local, id: 2
[Apr 30 00:15:33.678]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_local, id: 2
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:172 (setHostStatus)> (host_statuses) set status down for :  name: <IP_ADDRESS>, status: 1, reason_stat: proxy.process.host_status.<IP_ADDRESS>_local
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:177 (setHostStatus)> (host_statuses) name: <IP_ADDRESS>, status: 1
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_manual, id: 3
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_manual, id: 3
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:172 (setHostStatus)> (host_statuses) set status down for :  name: <IP_ADDRESS>, status: 1, reason_stat: proxy.process.host_status.<IP_ADDRESS>_manual
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:177 (setHostStatus)> (host_statuses) name: <IP_ADDRESS>, status: 1
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_self_detect, id: 4
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:267 (getHostStatId)> (host_statuses) name: proxy.process.host_status.<IP_ADDRESS>_self_detect, id: 4
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:172 (setHostStatus)> (host_statuses) set status down for :  name: <IP_ADDRESS>, status: 1, reason_stat: proxy.process.host_status.<IP_ADDRESS>_self_detect
[Apr 30 00:15:33.742]  DEBUG: <HostStatus.cc:177 (setHostStatus)> (host_statuses) name: <IP_ADDRESS>, status: 1

1808   netProcessor.init();                     // ---> socks config loaded at here
1809   prep_HttpProxyServer();
...snip...
1862   } else {
1863     remapProcessor.start(num_remap_threads, stacksize);
1864     RecProcessStart();
1865     initCacheControl();
1866     IpAllow::startup();
1867     HostStatus::instance().loadHostStatusFromStats();    // ---> HostStatus persistent data loaded at here
1868     ParentConfig::startup();          // ---> parents proxy config loaded at here
1869 #ifdef SPLIT_DNS
1870     SplitDNSConfig::startup();
1871 #endif

@jrushford Does the HostStatus persistent data have to be loaded before the records ?

This has merge conflicts, and we'll need a new 8.1.x PR. Also, feel free to recommend / suggest other PRs that this depends on, such that it makes the back porting easier.