Final Fixes Part 2

[ ] The "look at past PyCons" image viewer — as before — has a little vertical scroll bar next to it instead of simply filling its space completely.

In addition, as I hit right-arrow it does not scroll through as many images as it promises, and even seems to stop halfway between images.

[ ] Fix registration banner on home page. The ends get cut off on firefox at a variety of screen widths.

Registration banner ends get cut off in FF at 320x480:

However, the slideshow looks good:

I have just checked off the second item here as done, because it looks like more work was done on the banner in 0eb41879b44 and because when testing the site under Firefox the banner now seems to shrink correctly and remain an appropriate width even as the window becomes very narrow. So this issue now looks to be complete!


Revert "refactor: rewrite of .NET generator to TypeScript (#540)"

This reverts commit 9f62590, reversing changes made to 3c43be1.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I did the revert the naughty way.


Tests

Relies on #25
Fixes #18
Fixes #22

Looks great! 👏

I have no idea how it works, but the code looks nice! :space_invader:

Thank you!


Support connecting to multiple vCenters?

Does the CSI driver support multiple vCenter connections? It looks like the config has a generic cloud.conf defined. Whenever this config map is modified it's overridden with the interpolated values.
We're running on a stretch cluster and during a test cluster upgrade it's only connecting to one of our vCenters so far and can't find the VM UUID of other nodes in other vmware clusters.

In tree vsphere cloudprovider deprecated support for multiple vcenters long back - https://github.com/kubernetes/kubernetes/pull/98546 and vsphere CSI driver does not support multiple vcenters as well - https://github.com/kubernetes-sigs/vsphere-csi-driver/blob/master/docs/book/features/vsphere_csi_migration.md
Those are the reasons, this operator does not support multiple vcenters.


🛣 [FEAT]: Create the /ledgersync deeplink

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
Link ledgerlive://ledgersync to the ledger sync page.

❓ Context

LIVE-13757

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

@cgrellard-ledger you can use setLedgerSyncActivateDrawer now ;)


:sparkles: user can add generate key function (#9)

I think we should talk about this PR in live.
I believe this PR give to much features to the lib.

Closed in favor of #15


Packing the shiny app before DesktopDeploying it

hello everyone,
First, congratulations for this great idea!
I successfully ran the example in this repository that launched my default browser and rendered the app.
Now, I would like to packaging the shiny files before desktopdeploying it. For that, I included a launchApp function when packaging
launchApp <- function(){
  shiny::shinyApp(ui, server)
}

This package, let's say, MyPackage, could be attached and the function launcApp() can launch the app manually in the R terminal
Then I manually add this package into the app/library and changed the app.R function as
MyPackage::launchApp()
Once the appname.bat is launched, the browser opened and closed automatically.
What did I miss? Grateful if you could guide me on how to packaging first the shiny files before desktopdeploying it. Thank you

Suggest checking the log file for any errors.

Closing as this is over 1yr old. Feel free to reopen if still an issue.


[HOTFIX] Update to subgraph version 4.5.1

https://github.com/OlympusDAO/olympus-protocol-metrics-subgraph/pull/201

Can be merged once indexing is complete


Your module is not compatible with gtceu-1.20.1-1.1.3b, please update the module, thank you.

Your module is not compatible with gtceu-1.20.1-1.1.3b, please update the module, thank you.

I have already noticed about this issue, however I was extremely busy these days. I may try to fix that next week during my reading week.

Ok, could you please add the deleted function to speed up the processing speed of the single block machine in the next version? I really need it, thank you.

It is not hard tbh but I am encountering some technical issues and I cannot compile the code, because gtceu changed some of their dependency. I just need to copy the code they deprecated it's not difficult.

Ok,hope to see this feature in the next version of the module, thank you.

Already ported deprecated code from gregtech in 2.2.0


Write a test for CTRL+Click in REPL to ensure that blue links to web sites continue to work

We ran into a regression today because this was missing.

Nothing broke in a long time


Need to manually reload hyprland config after rebuilding on Nixos

Hyprland Version
 System/Version info
Hyprland, built from branch  at commit d92da7959a1d271c585d0a7de4c75a389bd1ac05  ().
Date: 2024-02-24
Tag:

flags: (if any)

System Information:
System name: Linux
Node name: theseus
Release: 6.7.5
Version: #1-NixOS SMP PREEMPT_DYNAMIC Fri Feb 16 18:14:32 UTC 2024

GPU information:
09:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev e7) (prog-if 00 [VGA controller])

os-release: ANSI_COLOR="1;34"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.05.20240223.cbc4211"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
IMAGE_ID=""
IMAGE_VERSION=""
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.05 (Uakari)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.05 (Uakari)"
VERSION_CODENAME=uakari
VERSION_ID="24.05"

plugins:
  hycov by DreamMaoMao ver 0.3

Bug or Regression?
Bug
Description
Ok, I am not sure if this is a real problem or I just have a bad memory, but now I have to manually reload the hyprland configuration with hyprctl reload after rebuilding my nix config with nixos-rebuild switch, and I do not remember having to do this in older days.
How to reproduce
For example:

I change value (e.g. border rounding) in nix config.
Build new generation and switch to it
Nothing changed, borders are the same, but in .config/hypr/hyprland.conf the rounding value changed to the new one.
Border only changed after manually running `hyprctl reload' in terminal. I don't remember having to do this, changes just instantly applied. But maybe I am wrong and it just Mandela effect or something.

Crash reports, logs, images, videos
No response

@fufexan for nix

I think the way HM reloads the config broke a while ago, due to some changes in Hyprland.
This patch should fix it, I'll try to open a PR today.
diff --git a/modules/services/window-managers/hyprland.nix b/modules/services/window-managers/hyprland.nix
index 168bab65..b280283b 100644
--- a/modules/services/window-managers/hyprland.nix
+++ b/modules/services/window-managers/hyprland.nix
@@ -247,10 +247,8 @@ in {
       onChange = lib.mkIf (cfg.package != null) ''
         ( # Execute in subshell so we don't poision environment with vars
           if [[ -d "/tmp/hypr" ]]; then
-            # This var must be set for hyprctl to function, but the value doesn't matter.
-            export HYPRLAND_INSTANCE_SIGNATURE="bogus"
             for i in $(${cfg.finalPackage}/bin/hyprctl instances -j | jq ".[].instance" -r); do
-              HYPRLAND_INSTANCE_SIGNATURE=$i ${cfg.finalPackage}/bin/hyprctl reload config-only
+              ${cfg.finalPackage}/bin/hyprctl reload config-only
             done
           fi
         )

I updated HM to latest rev 4ee704cb13a5a7645436f400b9acc89a67b9c08a (xscreensaver: add package option) after merge, issue is still here.
Also:
❱ hyprctl instances -j | jq ".[].instance" -r
terminate called after throwing an instance of 'std::invalid_argument'
  what():  stoull

Fixed, my skill issue


clean install failing laravel 5.7

I required twill package on clean laravel 5.7 installation but got this error:
`root@b16fe253e280:/var/www# composer require area17/twill:"^1.2.2"
Do not run Composer as root/super user! See https://getcomposer.org/root for details
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update
Package dimsav/laravel-translatable is abandoned, you should avoid using it. Use astrotomic/laravel-translatable instead.
Writing lock file
Generating optimized autoload files
Carbon 1 is deprecated, see how to migrate to Carbon 2.
https://carbon.nesbot.com/docs/#api-carbon-2
You can run './vendor/bin/upgrade-carbon' to get help in updating carbon and other frameworks and libraries that depend on it.

Illuminate\Foundation\ComposerScripts::postAutoloadDump
@php artisan package:discover --ansi

In ServiceProvider.php line 173:
array_key_exists(): The first argument should be either a string or an inte
ger
Script @php artisan package:discover --ansi handling the post-autoload-dump event returned with error code 1
Installation failed, reverting ./composer.json to its original content.`
I found this issue:
https://spectrum.chat/twill/help/installation-failing~9616e158-f018-4ef3-8053-b1243b790c71
Which seems related but nothing new pops up.
Any suggestions?

did a fresh install of laravel 5.8, repeated the steps. seems to work fine now with 1.2 branch 🤷♂

Install of area17/twill failed
Installation failed, reverting ./composer.json and ./composer.lock to their original content.

same erroor


feat: suggest "privacy" commands

In response to https://t.me/BotNews/96, the right thing to do is to suggest developers to add a section about privacy.

I actually disagree to this. But it's not a big change. In the docs it's the right place for it. I get where you come from with this pr, but we actually don't know enough to make it such a recommendation, not every country is required to have such a policy in place or they have different regulations.

I actually disagree to this. But it's not a big change. In the docs is the right place for it. I get where you come from with this pr, but we actually don't know enough to make it such a recommendation, not every country is required to have such a policy in place or they have different regulations.

Tha announcement says every bot should respond to the command. It is a "standardized" command as much as /help or /start IMO

That is incorrect. According to the GDPR, every service that is accessible by citizens of an EU country must comply with it. Every bot is available in the EU, which means that every bot must comply.
Legally, there are only two types of bots: those with a privacy notice and those who break (at least) EU law. In practice, this means that nearly every new bot will implement a privacy notice.
Whether or not we should add an auto-complete suggestion for this is a different question, but my main motivation for adding this command is actually not a legal one—it is that BotNews encourages every bot developer to implement /privacy. Similarly, native apps have special handling for /start, /help, and /settings in their UI.

Tha announcement says every bot should respond to the command. It is a "standardized" command as much as /help or /start IMO

Your right, i read it multiple times and misread the "must make it"

Thanks for being thorough! I appreciate it :)

I actually disagree to this. But it's not a big change. In the docs is the right place for it. I get where you come from with this pr, but we actually don't know enough to make it such a recommendation, not every country is required to have such a policy in place or they have different regulations.

That is incorrect. According to the GDPR, every service that is accessible by citizens of an EU country must comply with it.

This is actually incorrect too. It needs to target EU Users, websites, speaking chinese and so on but not being blocked in EU, don't need to comply. Or i got something wrong seit GDPR. It's mainly the intent to process EU-Data.

https://gdpr.eu/companies-outside-of-europe/ seems to explain it fairly well.
You might be referring to targeting EU consumers, and I agree that this part does not apply. However, there is a second part: monitoring. If your bot is recording data about EU citizens (storing any kind of data in your database about them) then GDPR applies.
There seems to be an exception, though. If your bot is neither professional nor commercial, then this could be an exception. At this point I'm not entirely sure anymore.
Adding a short privacy note is trivial, though, so it's probably best to just do it.


how use script convert all en datasets to the format 'conll'

Sorry, is there a demo to show how to use your scripts?

No, there isn't a demo unfortunately.  I need to revist those scripts at some point so they are easier to use. Is there a particular dataset you want to convert to 'conll'?


How to add custom menus and submenus

how can I add more menus and submenus into list.

Please let me now the answer

any update on this

To achieve this you need to implement it manually. You can see the examples there are files named example02-useApiDirect.html and example03-mobile.html in these files you can get the idea.


Correct flag name

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10488

@JoanneHendrickson
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Fake Error indicatior

Looking for Uout=1.8V, Uref=0.6V I got many perfect results (as expected), however in such cases I received ERR[V]=-2.22044604925031E-16.

Thank you for detecting this issue. The error is fixed in commit 0341c08.


S3 file like objects - can they implement a flush method?

It would be useful if S3 file-like objects also implemented the "flush" method (which is available on local file like objects)
This would make it easier to use as a drop-in replacement for file-like objects.
This could be a no-op or could genuinely force a flush of data.
What do you think?

Is there an internal buffer in boto that needs to be flushed?

I don't think so... that's not quite what I meant sorry.
I'm trying to handle the situation where I'm passing the file object into a third-party library that I don't have control over. That library is calling the "flush" method. So I ordinarily I do:
f = open('test.csv', 'w')
library.library_method(f) # everything works ok - library_method calls flush - no errors

vs
f = smart_open('s3://bucket/blah/test.csv', 'w')
library.library_method(f) # fails - flush doesn't exist

I.e. it would be nice if smart_open could be used as a direct replacement for open.

I guess we can do that, sure. Any other methods of that sort? Can you add them all in a PR?
Plus add clear motivation & comments to those no-op methods too, so someone doesn't remove them in the future by accident.

@tmylk I have made a PR resolving this issue like you asked me to. Do review.

This request is similar to previous https://github.com/RaRe-Technologies/smart_open/issues/64#issuecomment-251632966 request to inherit or quack io.IOBase interface support. See python docs for the list of other functions that need to be added. They only have minor differences across Python versions.

Yes - I was thinking a sensible implementation might inherit from io.IOBase
I'll find some time to work on this and open a PR.

@menshikh-iv This is done.
(smartopen)sergeyich:issue152 misha$ python
Python 3.6.3 (default, Oct  4 2017, 06:09:15)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.37)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import smart_open
>>> f = smart_open.smart_open('s3://private/key.txt')
>>> dir(f)
['__abstractmethods__', '__class__', '__del__', '__delattr__', '__dict__', '__dir__', '__doc__', '__enter__', '__eq__', '__exit__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__iter__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__next__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '_abc_cache', '_abc_negative_cache', '_abc_negative_cache_version', '_abc_registry', '_buffer', '_buffer_size', '_checkClosed', '_checkReadable', '_checkSeekable', '_checkWritable', '_content_length', '_current_pos', '_eof', '_fill_buffer', '_line_terminator', '_object', '_raw_reader', '_read_from_buffer', 'close', 'closed', 'detach', 'fileno', 'flush', 'isatty', 'raw', 'read', 'read1', 'readable', 'readinto', 'readinto1', 'readline', 'readlines', 'seek', 'seekable', 'tell', 'terminate', 'truncate', 'writable', 'write', 'writelines']

Thanks @mpenkov
Works fine with current master fbc82cc04bf92e8c588c49c43ff5aff8234ea87d

Hello Everyone,
I see that @mpenkov 's comment shows that the f object has the 'fileno' method implemented.  However, after migration to the new open function, I no longer see 'fileno' as a method for this object anymore.  This is rather unfortunate as I would like to use the return value of smart_open.open() as the stdin parameter to a Popen call.  But as it is not implemented it throws this error:
[ERROR] UnsupportedOperation: fileno

@peter-t-kim I'm not sure at which stage we got rid of fileno. I don't think it's a product of the change to the open function (that change happened at a different abstraction level). You can confirm this by checking on the commit in my previous comment (fbc82cc) and then:
In [6]: fin = open('s3://bucket/key')  # redacted

In [7]: 'fileno' in dir(fin)
Out[7]: True

In [8]: fin.fileno()
---------------------------------------------------------------------------
UnsupportedOperation                      Traceback (most recent call last)
<ipython-input-8-63d0a20bf555> in <module>
----> 1 fin.fileno()

UnsupportedOperation: fileno

When reading from S3, smart_open does not create any new file descriptors. The file descriptors get created by the OS when opening local files: in the case of reading from S3, we bypass that abstraction level entirely. So, I don't think it's possible to even have a file descriptor in your situation.
If you want to pipe data from S3 into a subprocess, you have to do something like this.


Fixes to GPS permissions and default PID gains.

Added permissions request for Location permissions in main activity.
Added try-catch blocks to catch security exceptions in the vehicle service.
Changed safe vehicle thrust to full speed.
Updated default vehicle gains to new values.
Fixed bug in serialization of gains to log files.

Looks great, seems to work great. Merging.


Self-hosting

Hi;
I wanted to build riju locally, to test it (and it is my second time), to in the future host it myself but I don't wanna host it on AWS and rather do it on a VPS I'm already using.
I have two questions:

I have failed to build riju for the second time, and I'm not sure what I'm doing wrong whilst following the docs.

I have cloned the repo and ran make shell I=admin which gives me a root shell but I can't run latter commands inside it as it just returns me a make: *** No rule to make target '<make's arg>'.  Stop.
I've tried running make tmux outside of this shell, and it seems it works? I had to update the commander package, but then it throws
sudo chown root:riju system/out/riju-system-privileged
chown: invalid group: ‘root:riju’
make: *** [Makefile:160: system] Error 1
Error: command exited with code 2
    at runCommand (file:///home/maciej/archive/riju/tools/util.js:42:11)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async main (file:///home/maciej/archive/riju/tools/depgraph.js:678:3)

when I run dep image:runtime.

Even If I build it, is the project too AWS'y to run it not on AWS?

Hmm, can you show the full terminal session which leads to you getting make: *** No rule to make target '<make's arg>'. Stop.? I'm not sure how that happens.
You definitely don't want to try running Riju outside of Docker, as you'll run into a fair number of environmental quirks like the one you ran into there. (I wouldn't object to a pull request making the scripts more robust so that they're more likely to succeed outside of the standardized environment of the admin shell, but it's not really supported at present.)
In terms of running without AWS, you can definitely do it, it's just that you won't be able to make the same uptime guarantees during deployments. All you need to do is build all the Docker images (for the supported languages and for the app), and then docker run the app image. Everything should work fairly transparently, and there is still support for passing in TLS configuration to the app server so that it will use a Let's Encrypt cert rather than relying on AWS ACM. You can look at the supervisor script to see what arguments are used to run the app image.
We don't currently have explicit documentation on running it manually, so I filed https://github.com/raxod502/riju/issues/122 to track that.

The whole shell I can paste in now is I guess
maciej@w10-nevalicjus:~/archive/riju $ make shell I=admin
docker run -it --rm --hostname admin -v /home/maciej/archive/riju:/src -v /var/cache/riju:/var/cache/riju -v /var/run/docker.sock:/var/run/docker.sock -v /home/maciej/.aws:/var/cache/riju/.aws -v /home/maciej/.docker:/var/cache/riju/.docker -v /home/maciej/.ssh:/var/cache/riju/.ssh -v /home/maciej/.terraform.d:/var/cache/riju/.terraform.d -e NI -e AWS_REGION -e AWS_ACCESS_KEY_ID -e AWS_SECRET_ACCESS_KEY -e DOCKER_REPO -e PUBLIC_DOCKER_REPO -e S3_BUCKET -e DOMAIN -e VOLUME_MOUNT=/home/maciej/archive/riju  -e Z -e CI -e TEST_PATIENCE -e TEST_CONCURRENCY -e TEST_TIMEOUT_SECS -e FATHOM_SITE_ID -e RIJU_IMAGE_HASH="$(docker inspect riju:admin -f '{{ index .Config.Labels "riju.image-hash" }}')" --network host riju:admin
yarn install v1.22.5
info No lockfile found.
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
Done in 0.10s.
0 root@admin:/src# make tmux
make: *** No rule to make target 'tmux'.  Stop.

This one was done on WSL2, which I guess contributed to it not being able to 'make' itself.
I've tried doing it on the final VPS which runs linux natively and it also got the same tmux error at the first attempt but after stepping out of that admin shell and running it again it miracolously worked 🎉
Thanks for the notes as to running it on not-AWS, I was just worried.

Looks like there are two issues there:

For some reason you're running as root inside the container, so something may be wrong with how we drop permissions based on environment variables.
Unless your Makefile was somehow modified, it definitely does have a tmux target, which suggests that something is wrong with your volume mount (i.e. /src is empty). Not sure what could be going on there.

I don't have any objection to WSL support for local development, so feel free to open an issue to track that if you'd like.


Add Rails 4 support, validations and I18n support

Hello!
I happen to be working on a new Rails app and needed a few things in PublishingLogic. The codebase on master seems to be fairly outdated in many regards and I thought I'll take the time to modernise the gem codebase.
I hope the commits are fairly self-explanatory but here are the main points what this PR introduces:

Regenerates the gem strcture using modern skeleton by Bundler
Uses a modern ActiveRecord interface and promotes AR as an explicit dependency (from a dev dependency)
Removes Gemfile.lock and utilises the gem specification instead
Adds validation to publishing fields in relation to each other
Adds I18n support and includes examples on how to localise attributes and error messages
Re-enables the migration generator in modern Rails applications
Removes an old view partial generator (that used deprecated ERB syntax)
Adds a sample Code of Conduct (that was generated using Bundler)

I happen to be the gem owner on RubyGems and would like to release a new version soon but first I’d like to get these merged in if possible please. We should also work out how to push tags etc to the repository. If Unboxed are not keen on maintaining the gem, I’m more than happy to take care of it.
I also wonder whether the email in the gem specification is obsolete or still a thing.

Hey @liquid this mostly looks good.  We'll have a chat about maintenance and get back to you with further comments if we're still keen to maintain this, or we'll work out how to transfer ownership if we're not.

ping @ubxd

Oops! I meant to reply last week @liquid sorry!  We don't really use this gem anymore, and it would appear don't have anyone remaining with creds to release it to rubygems anyway if we did.  We think therefore it might be easiest if we transferred the repo to you and you can maintain it.  What do you think?  I guess how it'll need to work is that you'll need to delete your fork (but keep your branches locally), then we can transfer ownership to you and you can then repush your branches.  How does that sound?

@h-lame ok cool. I'll ensure i have the local copy then give you a heads up. Thanks!

@h-lame could you please transfer the app to the @rewire organisation? Thanks.

(polite ping @h-lame)

Oops.  Sorry!  Doing it now.

Actually, no I'm not.  I'd need admin permissions on @rewire which I don't think you'd want to give me.  @liquid - I've added you to a team that has admin on this repo so you should be able to move it away now.  Let me know if that doesn't work, and we can try something else.


RAP-4155 Removing values from a Reflex list

Fix list -= value to work correctly (previously it would add -value to the list if value was numeric, or fail otherwise)
Fix list = list - value to throw an exception if value is not in list

The test code changes look fine

I don't like the second premise. I don't agree with throwing an exception in that case.

It's the old dilemma - what to do if you're asked to delete something that isn't there.
To my mind if you try to remove something from a collection and the thing is not in that collection then you're doing probably something wrong and would want to know about it.
But I see your viewpoint and I'm happy to remove the exception.

can this be merged?

Not any more. I removed the exception but now there are conflicts. I will fix them.

Conflict resolved


feat: singing

When the voice is set to Alan Pope, whether via a Mimic plugin or Piper with alan-low or the default ovos-tts-plugin-server config, a request to sing will play the original Mimic voice songs from old Mycroft. If not, the assistant will say it's too shy to sing.
Also adds OCP/ovos-media compatibility for containerized environments. An internet connection to GitHub is required in that case, as the skill will send the GitHub raw file URL instead of the local file URL.
Closes #27
Closes #36

The skill doesn't always ask for confirmation when and when it asked for confirmation, the yes can't be heard and generates the trace below.
alize:35 - ERROR - Error in STT plugin: ChromiumSTT
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: Traceback (most recent call last):
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_dinkum_listener/plugins.py", line 30, in finalize
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     transcript = self.engine.execute(audio, self.language)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_stt_plugin_chromium/__init__.py", line 248, in execute
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     data = json.loads(result)["result"]
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/__init__.py", line 346, in loads
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     return _default_decoder.decode(s)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 337, in decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 355, in raw_decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     raise JSONDecodeError("Expecting value", s, err.value) from None
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

The skill doesn't always ask for confirmation when and when it asked for confirmation, the yes can't be heard and generates the trace below.
alize:35 - ERROR - Error in STT plugin: ChromiumSTT
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: Traceback (most recent call last):
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_dinkum_listener/plugins.py", line 30, in finalize
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     transcript = self.engine.execute(audio, self.language)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_stt_plugin_chromium/__init__.py", line 248, in execute
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     data = json.loads(result)["result"]
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/__init__.py", line 346, in loads
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     return _default_decoder.decode(s)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 337, in decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 355, in raw_decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     raise JSONDecodeError("Expecting value", s, err.value) from None
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

this was an issue with STT, are you on latest dinkum that solves that issue with short words? and on latest ovos-config?  (to see if instant_listen and remove_silence is on)

The skill doesn't always ask for confirmation when and when it asked for confirmation, the yes can't be heard and generates the trace below.
alize:35 - ERROR - Error in STT plugin: ChromiumSTT
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: Traceback (most recent call last):
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_dinkum_listener/plugins.py", line 30, in finalize
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     transcript = self.engine.execute(audio, self.language)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_stt_plugin_chromium/__init__.py", line 248, in execute
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     data = json.loads(result)["result"]
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/__init__.py", line 346, in loads
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     return _default_decoder.decode(s)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 337, in decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 355, in raw_decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     raise JSONDecodeError("Expecting value", s, err.value) from None
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

this was an issue with STT, are you on latest dinkum that solves that issue with short words? and on latest ovos-config? (to see if instant_listen and remove_silence is on)

I was running version 0.0.3a42...

The skill doesn't always ask for confirmation when and when it asked for confirmation, the yes can't be heard and generates the trace below.
alize:35 - ERROR - Error in STT plugin: ChromiumSTT
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: Traceback (most recent call last):
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_dinkum_listener/plugins.py", line 30, in finalize
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     transcript = self.engine.execute(audio, self.language)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/home/goldyfruit/.venvs/ovos/lib/python3.12/site-packages/ovos_stt_plugin_chromium/__init__.py", line 248, in execute
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     data = json.loads(result)["result"]
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/__init__.py", line 346, in loads
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     return _default_decoder.decode(s)
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 337, in decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:   File "/usr/lib/python3.12/json/decoder.py", line 355, in raw_decode
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]:     raise JSONDecodeError("Expecting value", s, err.value) from None
Jun 14 17:41:25 mark2 ovos-dinkum-listener[7277]: json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

this was an issue with STT, are you on latest dinkum that solves that issue with short words? and on latest ovos-config? (to see if instant_listen and remove_silence is on)

I was running version 0.0.3a42...

0.3a43 is where it's fixed! But what do you mean it doesn't always ask for confirmation?

@goldyfruit I can't replicate the issue you mentioned. If you have details on that please open up an issue and I'll put in a bugfix


Added Emrip number

Related Issue

Info about Issue or bug

fix #5072
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

@sairish2001 I have already added the same #4893


Widget management via GUI

Is your feature request related to a problem? If so, please describe.
Please add widget management via GUI , now we need to manually modify the yml file related to the container and is not very simple
Describe the solution you'd like
like in the add new item , add add new widget so we can include the necessary code without modify the yml file
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
None

Any news?

I would also love this :)


Pop all wrinklers option which excludes shiny wrinklers

I like popping wrinklers, but I don't like popping shiny wrinklers. 👍

Yea I know.  I prob should have added this in the last release since it is very simple.

Oh man. Sorry. Is there a way I can delete it?

Nah, it's fine, I'll just close both when it's added.  It happens.

Added to Version 2.0106.1


CocoaPods install issue

I just installed via Cocoapods into my project.  I used the exact code from the example project but I got a ton of errors in the TNImageSliderViewController.swift file.
The only way that I was able to get around this was to copy all of the code from the TNImageSliderViewController.swift from the example project and paste it into the same file in my project.
I'm still getting a handful of "unknown class" errors for TNImageSliderView and TNImageSliderViewController.
Also getting crash from a found nil optional on this line...
imageSliderVC.images = [image1, image2, image3]

I just updated the podspec with a new version which removes the errors for Xcode 7.3 & Swift 2.2


Add additional users to the exclude_users list in CopyItemToEachAccount.py

[x] We created two new accounts, Demonstration and ArcGISforDefense which will now contain the "Released" and "Demo" items. Add these two accounts to the exclude_users list so that the Tag excel spreadsheet item is not added to these two accounts.
[x] Redeploy script to production server.
[x] Remove tag items that exist in these portal accounts on the production server.
[x] Rerun the schedule task to verify that the script no longer adds the tag item to these accounts.

Made fix in v.next. Closing this issue.


Remove using redis proxy for redis protocol

What this PR does / why we need it:
Currently, envoy does not support CDS clusters for redis proxy. Redis services become unaccessible on Istio when redis proxy is used.
The code in envoy that produces an error when CDS cluster is used for redis proxy:
https://github.com/envoyproxy/envoy/blob/8fee0f11f1d06abb1dae820a388ffe6d785274c0/source/common/redis/proxy_filter.cc#L21
calls
https://github.com/envoyproxy/envoy/blob/6b2823da5006e92bc4b365e9e8804a4f6a2eba37/source/common/config/utility.cc#L47
where an exception is thrown, resulting in listener on the port and the cluster not being added.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1763
Special notes for your reviewer:
Release note:

@vadimeisenbergibm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
4cf09adda7cde14911b0040e568b05bc75462c7a
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-presubmit


Training Time

Hi Jie,
Many thanks for your awesome job!
May I inquire about the training time and the corresponding GPU device you using?

2hours, 2080ti


[PE-D][Tester D] What is a payroll and why do we need 2 integers for it?

I am sorry if I miss this detail in the UG, but I did not see if there is any explanation for what is a pay roll, and why I need 2 integers. What do these integers mean? I can guess the first integer is the salary, and the second integer is the day of payment? But this may not be intuitive for users.

It would be nice to have a table explaining some confusing or ambiguous terms.
A similar confusing term would be "leave". Should explain "leave" as day of leaves to avoid ambiguity!

Labels: severity.Medium type.DocumentationBug
original: rockman007372/ped#1

Duplicate issue


Get rid of /core for all

/Core was not meant to be in CDK V2.  Was bug they left in when we converted over.  Addressed and now just needs to be removed


Trimming old branches

When editing from the web editor there is getting a little bit of a long list https://github.com/MicrosoftDocs/sql-docs/branches/all
Could you delete the merged branches and those that got moved to your internal repo.
PS: thanks for opening the issue tracker again 👍

@nschonni Nick, I will look into this.

@nschonni Nick, Majority of branches deleted. Thanks. Closing this Issue 1227 now.

Closing.


Is there a way to set the font size?

I am looking for a way to increase the font size. I couldn't find anything related to this in the docs. Is there a way to do it?

Yes, you should be able to use CSS to customize the chat.
For example I changed the font in the demo project:
https://github.com/antoine92190/vue-advanced-chat/blob/master/demo/src/App.vue#L173-L176

You can change the font-size for all the text using the method above, otherwise you can point so a specific CSS class by inspecting the component and apply you font-size for each block/element

Cool thanks a lot


Build(deps): Bump @patternfly/react-table from 4.113.3 to 4.113.4 (#5116)

Pull request opened by the merge tool on behalf of #5116

Merged in 1a116e1bcf33c78ccaa7889ec17dd4f749e9bb8d


Add the functionality to insert new rows into the table

If the table doesn't have focus, a prompt will be given to the user.
If the table has focus, a new row will be inserted before the focused item.

非常感谢！


[WIP] Try fix windows tests

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1515   +/-   ##
=======================================
  Coverage   64.66%   64.66%
=======================================
  Files          83       83
  Lines        6461     6461
  Branches      992      992
=======================================
  Hits         4178     4178
  Misses       1984     1984
  Partials      299      299

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 d409c11...08b959d. Read the comment docs.


[Feature]:  Enable password keychain

Describe the solution you'd like
This would help users keep their password so they don't have to renew it all the time

Done, should make a 'v2' of this, though


refactor: RFC 334 Add NIST P and X25519 keys concrete examples

Signed-off-by: Baha Shaaban<EMAIL_ADDRESS>

@kdenhartog

What's the intent of description for this topic? Is it focused on DIDCommV1 pack/unpack style or DIDCommV2 stuff?

The intent is to advance RFC 0334 for use in DIDComm V1 with an eye on AIP 2.0.
This has nothing to do with DIDComm V2.

@kdenhartog

What's the intent of description for this topic? Is it focused on DIDCommV1 pack/unpack style or DIDCommV2 stuff?

The intent is to advance RFC 0334 for use in DIDComm V1 with an eye on AIP 2.0.
This has nothing to do with DIDComm V2.

Ahh interesting that may leave us with 3 potential formats that have to be considered at a DIDComm endpoint then while looking to migrate DIDCommV1 to DIDCommV2? E.g. RFC 0019 (old JWE derived pack/unpack), RFC 0334 (Includes Anoncrypt/Authcrypt with similar key types), and the DIDCommV2 version (albeit DIDCommV2 is just a subset of RFC 0334 from the looks of it)
I think this is going to make AIP 2.0 effectively compliant at the envelope layer at least, but probably still going to need mime type adjustments and such to indicate the change in version.

@kdenhartog @andrewwhitehead, I've updated this PR to include integrity checks as recommended by @kdenhartog's comment above.
The examples now include both X25519 and NIST P curved keys.
I've also used DIDComm V2's media types in the examples to be compatible.
This pr is ready for review/merge

Merged per discussion on Aries Working Group call.


client-ffi + dart-bindgen

The goal of this PR is to get an example working that uses dart-bindgen with the client code in this repo.
It was started by copying and pasting some code from sunshine-flutter/native/keystore-ffi and adding a Default implementation to the local keystore to get keystore_new working.
David has mentioned that bin/cli shows how to use the public API cc @shekohex

allo-isolate will make this much easier; better to start from scratch


Error occurred when converting nlohmann::json to std::any

Description
I would like to store objects in a std::map<std::string, std::any>. But converting nlohmann::json to std::any throws error type must be string, but is
I can only reproduce this issue on Windows. macOS seems working fine.
Reproduction steps
Minimal test case:
#include <any>
#include <nlohmann/json.hpp>

int main() {
    nlohmann::json j;
    std::any &&a = j;
    return 0;
}

Expected vs. actual results
Expected: Runs without error
Actual: Exception: Exception 0xe06d7363 encountered at address 0x7ffa855c466c
Minimal code example
No response
Error messages
<unknown> 0x00007ffa855c466c
<unknown> 0x00007ffa6dcfb660
nlohmann::detail::from_json<nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > >,std::any,0>(const nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > > &,std::any &) from_json.hpp:118
nlohmann::detail::from_json_fn::operator()<nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > >,std::any &>(const nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > > &,std::any &) from_json.hpp:480
nlohmann::adl_serializer<std::any,void>::from_json<nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > > const &,std::any>(const nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > > &,std::any &) adl_serializer.hpp:25
nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > >::get_impl<std::any,0>(priority_tag<0>) json.hpp:1613
nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > >::get<std::any,std::any>() json.hpp:1756
nlohmann::basic_json<std::map,std::vector,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,bool,__int64,unsigned __int64,double,std::allocator,nlohmann::adl_serializer,std::vector<unsigned char,std::allocator<unsigned char> > >::operator<std::any,0> std::any() json.hpp:1900
main() main.cpp:7
invoke_main() 0x00007ff7cb0d12c9
__scrt_common_main_seh() 0x00007ff7cb0d116e
__scrt_common_main() 0x00007ff7cb0d102e
mainCRTStartup(void *) 0x00007ff7cb0d135e
<unknown> 0x00007ffa872d54e0
<unknown> 0x00007ffa87e8485b

### Compiler and operating system

MSVC 19.31.31104 on Windows 11 21H2

### Library version

current develop 15fa6a342af7b51cb51a22599026e01f1d81957b

### Validation

- [X] The bug also occurs if the latest version from the [`develop`](https://github.com/nlohmann/json/tree/develop) branch is used.
- [X] I can successfully [compile and run the unit tests](https://github.com/nlohmann/json#execute-unit-tests).

Seems to work just fine for me with trunk:
https://godbolt.org/z/jcv3Ejsnd
(GCC 11, Clang trunk, MSVC latest)

Can I close this issue then?

But I didn't get it…… why std::any is a string type?

I cannot reproduce the runtime error.

I cannot reproduce the runtime error.

https://godbolt.org/z/T8fnsrKPz
This reproduces the issue.

Thanks! So this only occurs on MSVC? Very strange. I would need to see the stack trace from the exception and the state of the basic_json value.

Thanks! So this only occurs on MSVC? Very strange. I would need to see the stack trace from the exception and the state of the basic_json value.

I just tried again with Clang (trunk) and it works just fine with the rvalue ref. a.type() indicates it does contain a basic_json as expected. I can try to step through it with a debugger when built with GCC/Clang to see how it differs from the MSVC stack trace above. But that would have to wait until tomorrow.

Also, it only happens in the rvalue ref case

It seems on Apple Clang 13.1.6 and gcc 9.4.0 the copy constructor was called. The stack is also different from MSVC.

I would need to see the stack trace from the exception

I've posted the stack trace in the Error messages section.

I think this can be fixed by conditionally excluding std::any from being implicitly converted. Running it through CI right now.

@falbrechtskirchinger @nlohmann Thanks for your hard work!


Fixed import to local_planner

Description

Without adding agents.navigation I get an import error.  This is called from automatic_control.py for example.
Where has this been tested?

**Platform(s): Ubuntu 16.04
**Python version(s): 2.7 & 3.6
**Unreal Engine version(s):4.21

This change is

This is a duplicate pull-request.  Hence closing it.


Formkit playground is not rendering

Reproduction
https://formkit.com/plugins/zod
Describe the bug
Tried with chrome and edge

Environment
Windows10

Looks like actually all formkit playgrounds are broken at the moment🤔

@Rednas83 Thank you for reporting this, we're looking into it.

@Rednas83 Can you take a look at the playground for me and let me know if that issue is still persisting? You may need to hard reload/disable cache in dev tools.

We should be good — thanks for the report and thanks @sashamilenkovic for the quick resolution!


Batch Editing Mode to Speed Up Activities

A batch editing mode that does a couple things would speed up the adding of buildings.

Have an automatic "square corners" option when the "area" is finished
Preset the feature being edited.

For example, the map I am working in has few residential buildings marked. I hit shortcut "3" for area, click the area, then click the Residential Building feature, then click the area and select "square corners". This is done over and again.
Instead, it would be nice if upon completion of the area, the preselected feature (Residential Building in this case) is already selected, and the square corners option is applied.

It could be generalized to apply any field, like part of the address, state, and zip code.

just came here to :+1: this -- would be quite helpful for HOT tasks where tracing buildings is the primary task. Not sure about squaring automatically (there's much debate there) but could be an optional check-box.

I don't think squaring corners should be required, but one of the options one could set if they wish to. There may be other parameters that could automatically be set as well.
Or possibly, the square corners could be added as a shortcut. Would still make adding buildings easier.

Lots of these ideas are covered on existing issues.. See #528, #590, #839, #1761


Microsoft.Migrate - Split the existing readme(s) into separate files based on service

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[x] Other, please clarify:

For smooth onboarding of customers for respective service, we intend to take this change up where we split the existing readme files to respective service's readme. This is going to make the SDK generation process smooth.

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

My understanding is that Migrate wants to have separate SDKs for each of the sub services.
Can we keep the original readme files same and avoid modifications to the same in future considering that we dont want to release Migrate as a combined SDK?
You can add new project/RT specific readme files for the new API versions and reference your RT specific SDKs generation to those RT specific readme files?

My understanding is that Migrate wants to have separate SDKs for each of the sub services. Can we keep the original readme files same and avoid modifications to the same in future considering that we dont want to release Migrate as a combined SDK? You can add new project/RT specific readme files for the new API versions and reference your RT specific SDKs generation to those RT specific readme files?

Discussed offline and adding the old readmes as is. Will not add new versions to older readmes.

As discussed offline, please review the namespace for the packages. Remove the ARMChangesRequested tag post that.

@lirenhe @tadelesh can one of you please take a look? Do we have a template for the readme config files?

@lirenhe @tadelesh can one of you please take a look? Do we have a template for the readme config files?

sample file is here: https://github.com/Azure/azure-rest-api-specs/blob/main/documentation/samplefiles/readme.go.md. go do not need to set multi-api package config anymore. @srilipta-swain you could remove that part according to the template.

/azp run

@raych1 Do you know why SDK automation is skipping Java SDK?

/pr RequestMerge


Help .. increase gpt4 model maximum tokens more than 8192 PLEASE

Help .. increase gpt4 model maximum tokens more than 8192 PLEASE in https://nat.dev/ website . need more than 8192 please ... if can make it 16000

I ask long question and need answer to modify mo python code , the answer is stop and I get this message.  Please I pay without get full answer my question in coding , please increase memory side of nat.dev or increase the gpt4 tokens Maximum length.  So we can use

When using the API, the gpt-4 model has a limit of 8192 tokens, while gpt-3.5-turbo has a limit of 4096. Those are limited by the model itself, this project can do little about it.
It is said that there is a gpt-4-32k model which supports up to 32768 tokens, and that would be around 50000 words. If you have access to it, you can try it as an alternative.

ok let me access it .. this is my email in nat.dev<EMAIL_ADDRESS>

When using the API, the gpt-4 model has a limit of 8192 tokens, while gpt-3.5-turbo has a limit of 4096. Those are limited by the model itself, this project can do little about it. It is said that there is a gpt-4-32k model which supports up to 32768 tokens, and that would be around 50000 words. If you have access to it, you can try it as an alternative.

I'm a total newb, how do I access this?


text-decoration

Start using long-hand text-decoration properties, keeping old text-decoration shorthand for legacy support.

[ ] text-decoration-line
.-tdl-n     none
.-tdl-ol    overline
.-tdl-ul    underline
.-tdl-lt    line-through

[ ] text-decoration-style (abbrevs consistent with border-style)
.-tds-s     solid
.-tds-do    dotted
.-tds-da    dashed
.-tds-db    double
.-tds-wv    wavy

[ ] text-decoration-color (abbrevs consistent with border-color)
.-tdc-t    transparent
.-tdc-c    currentColor
.-tdc-b    #000000
.-tdc-w    #ffffff

 bangs v0.12.0

 bangs v0.13.0 (theme: fonts & text)

merging in text-decoration-{line,style,color} into v0.15.0 but leaving issue open for text-{decoration-skip,underline-position} into a later version.


feat(): $translatePartialLoader.addPart should return a promise

You could then add parts in the resolving process of a route and open the page as soon as the translation is available.
Currently it returns the $translatePartialLoader itself. The change would break chained calls.
An additional method addPartAndReturnPromise would not break this.

I didn't find the answer I was looking for when I got here. I came because I was trying to find out why $translatePArtialLoader.addPart('whatever') didn't return a promise. That just made total sense to me. I was ultimately trying to figure out how I can stop my view from loading until all of the required translations for that view loaded via ui-router's resolve. After reading what @jedmao came up, I thought I could put together a better solution. Overall, I think the problem with jedmao's code is that it's not returning a promise but just calling $translate.refresh() after calling addPart without guaranteeing that his translations loaded - after all, they are loading asynchronously. It was close so I just made the following adjustments:
In my resolve for my $stateProvider, I call loadParts on my translations service:
resolve: {
    languages: (translations) => {
        return translations.loadParts(
            'views/language-selection'
        );
    }
}

The key is that loadParts returns a promise (as I mentioned above), and it looks like this:
// My translations service
loadParts() {
        var parts = Array.prototype.slice.call(arguments), // convert arguments (parts) to a true array
            deferred = this.$q.defer(),
            totalLoaded = 0, // this represents how many parts/files have actually loaded
            total = arguments.length; // this is how many parts we're trying to load

        parts.forEach((part) => {
            this.loadPart(part).then(() => {
                totalLoaded += 1;

                if(totalLoaded === total) {
                    this.$translate.refresh();
                    deferred.resolve();
                }
            });
        });

        return deferred.promise;
    }

I take in however many "parts" I pass in, iterate over each of them and call my loadPart method for each. loadPart returns a promise as well and on then, I iterate the totalLoaded by 1 and then check to see if the totalLoaded matches the total passed in, and if it does, I resolve the promise for loadParts. This gives me a true accounting to make sure that everything has loaded before I resolve. Once that matches, I then call $translate.refresh() and resolve my promise.
Here's my loadPart method:
loadPart(part) {
        var deferred = this.$q.defer();

        var listener = this.$rootScope.$on('$translatePartialLoaderStructureChanged', () => {
            listener();
            deferred.resolve();
        });

        this.$translatePartialLoader.addPart(part);

        return deferred.promise;
    }

The listener var is so I can remove the listener from $rootScope once I'm done with it.

@jedmao maybe this is not part of  the issue here but I do not have any other site where I can make my question:
This code loads the translate by components, lets say I don't want to load "index", I need to load "home" because I need the user log-in.
.config(function($translatePartialLoaderProvider) {
  $translatePartialLoaderProvider.addPart('home');
});

My question is, where should I put this code? If I left this code on my config this will execute when my app load for the first time and it seems I can not put this inside a controller:
Any advice?

@napstercake $translatePartialLoader.addPart('home);` should do it for you.
@jedmao I'm having issues where the translation tables are not translated in my app even though I added a component during the config phase like @napstercake is doing and $translate.refresh() resolved. Any ideas?

@dwilt I haven't touched Angular in over a year now, nor do I have access to my eventual configuration; however, @prabinv might.

@dwilt if I use $translatePartialLoader inside of a controller I always have this error:Unknown provider: $translateProviderProvider <- $translateProvider <- myController also if y use just use $translatePartialLoader.addPart('home') where should I specify my URL to consume my rest that will retrieve the translation table ONLY for home. :(

You cannot use providers in non config phase contexts, like a controller.

@knalli From here: http://angular-translate.github.io/docs/#/guide/12_asynchronous-loading (about midway down the page)

translatePartialLoaderProvider configures $translatePartialLoader which means you're able to inject $translatePartialLoader into your controllers and services to make use of it. But why do you want to? As you can add a translation part to the loader via the provider, you can also do so with the provided service to manipulate loader state at runtime. So what does that mean? It means, you just have to inject the loader and use that one to add additional translation parts.

Let's say we have a ContactCtrl within our contact module, things could look like this:

angular.module('contact')
.controller('ContactCtrl', function ($scope, $translatePartialLoader) {
  $translatePartialLoader.addPart('contact');
});

That wasn't hard, right? So what happens here is we manipulate our loaders state during runtime. This makes sure that angular-translate only loads specific translation data when we really want to. Once you added a new part, $translatePartialLoader fires an $translatePartialLoaderStructureChanged event.

Yeah. $translatePartialLoader and $translate are regular services and available in the regular context (services, controllers, directives, filters). But the providers like $translatePartialLoaderProvider or $translateProvider are only available in the config phase.

Where is the call to the REST service that retrieve me the translation table if I use only
$translatePartialLoader.addPart('contact');

In the config? If I call to my REST from the config it will fire everytime the page refresh right? So the concept of partialLoader is to load translation table on demand. I'm so confuse.

You use $translatePartialLoaderProvider.addPart() in the config phase, and this will load all registered parts. After you have added additional parts after the config phase via $translatePartialLoader.addPart(), you will have to invoke $translate.refresh().


[Bug]: Upgrade from 1.0.8 to 2.0.x fails via  Homebridge UI

What happened?
I have been trying to upgrade this plugin via Homebridge UI but it gives me an error message on every attempt.
Please see the attached log output. It seems to have something to do with permissions on the ffmpeg-for-homebridge package but I am not sure.
Any help?
Thanks,
Device Type
Doorbell (Supported)
Plugin Version
other (UnSupported)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian 10+ (Supported)
Relevant log output
USER: homebridge
DIR: /var/lib/homebridge
CMD: pnpm install --save homebridge-eufy-security@latest

Packages: +24 -5
++++++++++++++++++++++++-----
Progress: resolved 513, reused 511, downloaded 2, added 23, done
<EMAIL_ADDRESS><EMAIL_ADDRESS>Running install script, failed in 1s
.../node_modules/ffmpeg-for-homebridge install$ node install.js
│ Downloading ffmpeg from https://github.com/homebridge/ffmpeg-for-homebridge/release
│ node:events:505
│       throw er; // Unhandled 'error' event
│       ^
│ Error: EACCES: permission denied, open '/home/homebridge/.npm/_ffmpeg_for_homebridg
│ Emitted 'error' event on WriteStream instance at:
│     at emitErrorNT (node:internal/streams/destroy:157:8)
│     at emitErrorCloseNT (node:internal/streams/destroy:122:3)
│     at processTicksAndRejections (node:internal/process/task_queues:83:21) {
│   errno: -13,
│   code: 'EACCES',
│   syscall: 'open',
│   path: '/home/homebridge/.npm/_ffmpeg_for_homebridge/.download'
│ }
└─ Failed in 1s
 ELIFECYCLE  Command failed with exit code 1.

Operation failed. Please review log for details.

Hi @eulisef
this issue might be a duplicate of #47
Try the steps described in this issue, but likely it will be resolved by updating manually via cli: sudo npm install -g homebridge-eufy-security@latest
For some reasons the installer seems to have a problem with certain environment variables on some systems. Therefore the used option -E through the homebridge UI causes this error.
Let me know if this resolves your issue. I will add a section to common issues, if this helps you too.

Hi Thieren,
I tried following issue #47 , i tried the common issues section and going via cli for both ffmpeg-for-homebridge and your suggestion... both went through without any error message but i am still on 1.0.8...
pi@raspberrypi:~ $ sudo npm i ffmpeg-for-homebridge -g
changed 16 packages, and audited 17 packages in 9s
4 packages are looking for funding
run npm fund for details
found 0 vulnerabilities
pi@raspberrypi:~ $ sudo npm install -g homebridge-eufy-security@latest
changed 145 packages, and audited 146 packages in 48s
23 packages are looking for funding
run npm fund for details
found 0 vulnerabilities

And when i tried via Homebridge UI now I got this error:
USER: homebridge
DIR: /var/lib/homebridge
CMD: npm install --save<EMAIL_ADDRESS>npm ERR! code EACCES
npm ERR! syscall open
npm ERR! path /home/homebridge/.npm/_cacache/tmp/caff1a1e
npm ERR! errno -13
npm ERR!
npm ERR! Your cache folder contains root-owned files, due to a bug in
npm ERR! previous versions of npm which has since been addressed.
npm ERR!
npm ERR! To permanently fix this problem, please run:
npm ERR!   sudo chown -R 999:996 "/home/homebridge/.npm"
npm ERR! Log files were not written due to an error writing to the directory: /home/homebridge/.npm/_logs
npm ERR! You can rerun the command with --loglevel=verbose to see the logs in your terminal
Operation failed. Please review log for details.

ok... it worked.
i followed the recommendation from the latest error and ran this from cli:
sudo chown -R 999:996 "/home/homebridge/.npm"
afterwards i tried again and now it updated properly

Ok. Good to know that this is caused by a bug in npm.
Didn't knew this error message until now.


Get list of ALL gateways

Hello
What is the best way to list ALL gateways including the unofficial ones?
\Omnipay\Omnipay::getSupportedGateways() only returns an array of predefined "official" gateways
So far I did it like so:
    /**
     * Retuns an array of registered namespaces from composer's autoload file
     * @return array
     */
    protected function getGatewayNamespaces()
    {
        $file = __DIR__ . '/vendor/composer/autoload_psr4.php';
        if (is_readable($file)) {
            $namespaces = include $file;
            return array_keys($namespaces);
        }
        return array();
    }

    /**
     * Returns an array of gateway IDs (short names) extracted from the registered namespaces
     * @return array
     */
    public function getGatewayIds()
    {
        $gateways = array();

        foreach ($this->getGatewayNamespaces() as $namespace) {
            if (strpos($namespace, 'Omnipay') !== 0) {
                continue;
            }
            $matches = array();
            preg_match('/Omnipay\\\(.+?)\\\/', $namespace, $matches);

            if (isset($matches[1])) {
                $gateways[] = trim($matches[1]);
            }
        }

        return $gateways;
    }

    /**
     * Returns an array of ALL registered gateway instances
     * @return array
     */
    public function getGatewayInstances()
    {
        foreach ($this->getGatewayIds() as $id) {
            $class = \Omnipay\Common\Helper::getGatewayClassName($id);
            if (class_exists($class)) {
                \Omnipay\Omnipay::register($id);
            }
        }

        $registered = array();
        foreach (\Omnipay\Omnipay::find() as $id) {
            $registered[$id] = \Omnipay\Omnipay::create($id);
        }

        return $registered;
    }

Is there a better method?

That's pretty genius TBH. I've been looking for a way to automatically get a list of the installed drivers for the example application. Since they all use PSR-4, and all use composer, this is a great way to get a list of installed drivers.
The application does need to know where the vendor directory is. I don't believe there is a general way to discover it from anything in the global space. So your function here would need to know in advance where it is in relation to the vendor directory. If using a framework of some sort, then that may have some configuration settings you can query.
I believe getSupportedGateways() is gone or going anyway, since it was only used by the example application, and was very difficult to maintain.

So your function here would need to know in advance where it is in relation to the vendor directory.
Do you mean
$file = __DIR__ . '/vendor/composer/autoload_psr4.php'; ?
Yes, you're right, but in my case it will always work. I'm worring about namespaces. Can we rely on them?
For example, will GatewayID in namespace  \Omnipay\GatewayID\... always point to the corresponding gateway GatewayID

Hmm, probably not. There isn't - so far as I know - a way to ask a base class if it happens to be an OmniPay gateway and what services it supports. I feel like a discovery metadata class is needed for each gateway so it can be queried as to what it is.
Unless there are any other bright ideas I've overlooked, it seems that you need a list of known OmniPay gateways. The top-level classes will extend Omnipay\Common\AbstractGateway, which is the only thing I can see they have in common, but how you get that list of classes I don't know. You may be able to see that \Omnipay\AuthorizeNet\ is PSR-4 autoloaded, but how do you then discover that \Omnipay\AuthorizeNet\AIMGateway (or CIMGateway or whatever) exists so you can check it extends AbstractGateway? Maybe composer has a way to give you this list?

https://packagist.org/packages/omnipay/common/dependents

Outsourcing some of the metadata capture :-) That's handy.
How about the API details once you have the gateway package name? With Authorize.Net you have AIM, CIM and DPM, but where should these be listed so they can be discovered? I'm not sure if this is the kind of detail that OP is looking for, but it seems essential to me.


Execute task with timeout

Something like https://github.com/dotnet/orleans/blob/73e8ae362675d3d94738ab170ed2bc222a6ee3f3/src/Orleans/Async/TaskExtensions.cs#L232 could be useful for async I/O operations

AsyncEx already has equivalent functionality.

@StephenCleary Sorry, which method are you referring to?

WaitAsync, with a timeout-based CancellationToken.

The proposed method differs a bit - 1) simpler method signature for this specific case - you don't have to construct CancellationTokenSource every time 2) in referenced WaitAsync the executed task can throw unobserved exception (https://stackoverflow.com/questions/7883052/a-tasks-exceptions-were-not-observed-either-by-waiting-on-the-task-or-accessi) if the method returns on timeout

Agreed, but using CancellationTokens for timeouts is standard. It also allows the same behavior to allow any kind of "cancel this await" logic (not just timeouts).

Yes. But that is desired behavior, IMO. Note that with both solutions (WaitAsync and WithTimeout), the task still runs to completion. That is, the task is not canceled; only the wait is canceled. Thus, I prefer the semantics that such tasks will have unobserved exceptions.


Chained values with following function calls don't get split into lines

When taking a look at a reproduction of https://github.com/jlongster/prettier/issues/200 I noticed that if some really long chain of nested objects occurred and then a following function call, it does not split the nesting into new lines.
https://jlongster.github.io/prettier/#{"content"%3A"request.hello.world.function('foo').some.really.really.really.really.really.really.really.really.really.really.really.really.really.really.really.really.really.really.really.long.constant.post('%2Fversion')  %2F%2F comment 1\n  .set('Prefer'%2C 'plurality%3Dsingular') %2F* comment 2 *%2F\n  .send()\n  \nrequest\n  .post.version  %2F%2F comment 1\n  .set.Prefer.plurality.singular %2F* comment 2 *%2F\n  .send()\n"%2C"options"%3A{"printWidth"%3A40%2C"tabWidth"%3A2%2C"singleQuote"%3Atrue%2C"trailingComma"%3Afalse%2C"bracketSpacing"%3Afalse}}
I would love to work on this issue myself, but it might take me some time to wrap my head around the code if it's alright that the fix won't be ready within a day or two 😄

You should try to fix it, the code that's responsible for it is https://github.com/jlongster/prettier/blob/d8c51483fc3c27233c40e99953487ef31966b38e/src/printer.js#L1985
Can't guarantee no one will try and fix it in parallel as there are quite a few issues raised about this part of the code, but worst case, you may have learned something :)

Yeah, this is a great part of the code to dive into if you're up for a challenge! I really need to write better documentation than commands.md but that's all we have for now.
The main reason is because we don't break on property accesses. The only time we do is when we think you are trying to use promise/observables/or some library that does a lot of function chaining:
observable
  .map(x => { ... })
  .switchWhat(x => { ...})

That's a really common pattern in JS and that syntax is desirable.
But for generic property accesses like foo.bar.bar you don't want to break. At least, usually you don't. I don't think it's usually a problem that we don't break on generic property accesses. When you're looking up a deeply nested object, you usually want foo.bar.prop("baz").biz to stay as one thing just like a variable would.
Where should it break? Should it split the entire lookup across lines (one lookup per line?) or be more like binary expressions where we try to fit as many as we can on one line? The former is too aggressive imho for simple lookups and the latter technically complicated.
Before you spend a ton of time on this, I just want to make sure you are hitting this on real code and it's not just an abstract use case. If you want to to play around with the code and show us various formats we can output with this case, by all means do! But note that if it introduces a lot of complexity, we'll probably only merge it if it solves a pattern people use in the real-world.

I tried to fix this as well. I added a conditional group for the member lookup, so it breaks if necessary. Though I'm not sure this is the best idea. I'll open a PR today.

@jlongster the real code that I'm trying to fix is this.
I also thought about breaking after the assignment operator, which I generally prefer, but I know that's unconventional (writing Elm, one gets used to this):
function* sendMail() {
  try {
    const pdfStream =
      yield contractsService.getLandlordContractStreamFromBooking(booking);
  } catch (error) {
    report(error);
  }
}```

I opted for breaking after the member lookup instead:

```js
function* sendMail() {
  try {
    const pdfStream = yield contractsService
      .getLandlordContractStreamFromBooking(booking);
  } catch (error) {
    report(error);
  }
}

@arnarthor yours would be formatted as:
request.hello.world.function("foo").some
  .really.really.really.really.really
  .really.really.really.really.really
  .really.really.really.really.really
  .really.really.really.really.long
  .constant
  .post("/version")
  .set("Prefer", "plurality=singular")
  .send();

// comment 1
request.post.version.set.Prefer
  .plurality.singular /* comment 2 */
  .send();

This allows for newlines after a member lookup if really necessary to adhere to the max-width.


Installing bionic arm on healthy arm won't drop arm after surgery

Describe the bug
This mod patches natural body parts in the game to drop after installing an artificial replacement. This will only happen if the body part is not damaged. The 'drop healthy limb after surgery' mechanic works for all organs except for arms.
Expected behavior
After installing a Bionic Arm, the biological arm should be spawned if it was healthy on the patient.
To Reproduce
Steps to reproduce the behavior:

Debug in a bionic arm and debug 'Finish all research'
Find a pawn with a completely healthy arm. No negative hediffs
In patient's Health tab, queue up Install bionic arm on that arm
Observe that the biological arm will not be spawned after surgery

Additional context
This mechanic seems to work fine for legs, eyes, hearts...etc. It's only arms that are having an issue. The patch method for Arms is the same as other body parts, so it's not immediately obvious what the problem is.
Technical notes
MedicalRecipesUtility.IsCleanAndDroppable() might be returning false or the PatchOperation for Arm is failing. Debug logging might be needed.

There's no BodyPartDef named Arm in the vanilla game. That's why the PatchOperation is failing...yikes.


Font size

Hello I use Reacticz on a touchpad hanging on the wall. The problem is that we can not see the text. So I submit an idea for developers: add an option to enlarge or reduce the font size of cells

I did some tests, one simple and quick solution would be to use the css property zoom in the body but it's not supported in Firefox. Using the css property transform:scale gives weird behaviors. I keep working on it.

Awesome, thanks @davidlb!

Fixed!


TCP flow behavior for probe down scenario.

Document states that if a backend instance's health probe fails, established TCP connections to this backend instance continue. It also states that the Load Balancer is a pass through service (does not terminate TCP connections). Before this line though, it specifically states that the Basic Load Balancer will terminate all existing TCP flows to the backend pool.
This is a bit confusing to read, will the Basic SKU Load Balancer terminate established TCP flows to the backend pool if the health probes fail or will it allow the connection to persist and only block new TCP connections?

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

ID: 0e1e7cb2-c5a7-7122-c729-dc259db99a40
Version Independent ID: 45c6fd72-5114-fcda-0bab-2d80ac6bed33
Content: Use Azure Load Balancer health probes to scale and provide high availability for your service
Content Source: articles/load-balancer/load-balancer-custom-probe-overview.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

@david-frazee , Thank you for your feedback. i will investigate and get back to you.

I think the below snippet will answer your question. In Standard LB, if the probe is down, the TCP connections are not terminated and new connections will be terminated.
For Basic LB, if the probe is down, TCP connection and the new connection are terminated.

Let me know if you have any further questions.

@david-frazee , Do you have any update on this issue?

@david-frazee , I will go ahead and close this issue. Please tag me if you have any further questions.


Get RestDocs working and add gradle task to generate them

This task should be run as part of the unbreakable build

#21


use of checkers on docker_cli_exec_unix_test.go

Signed-off-by: weiyan<EMAIL_ADDRESS>

Seems good to go :smile:

LGTM

LGTM 🐸


Add stat on panic recovery in HTTP middleware.

Coverage decreased (-0.05%) to 97.304% when pulling 4b924e5b5601198d7491b410bbf3a26d64d09916 on panic_recovery_stat into 7ac86cc9731317010855541b7e7db83ad61c7e7a on master.


Typescript classes not defined?

eg. class in my client code:
    class A {
    }
    this.A = A;

I can use A in other files, but when I use it in integrated tests, jasmine complains not defined.

Resolved, classes can not be used except in steps or befores.

I think that has something to do with the loading order. I've created a new issue for this: https://github.com/Sanjo/meteor-jasmine/issues/225. I fixed that case in server integration mode some time ago. But probably it still exists in client integration mode.


Body::wrap_stream question

https://docs.rs/hyper/0.13.6/hyper/struct.Body.html#method.wrap_stream
I'm wondering if this API requires a bit more documentation in terms of how this affects the underlying networking, and also regarding the requirements on the stream.
I tried using this in Servo to support streaming uploads(using this as the body of a request), and this didn't seem to work, although I don't know if it's the server that didn't like the request or if I did something wrong using the API.
I noticed the following output from the server:
 0:04.54 INFO STDERR: <IP_ADDRESS> - - [13/Jun/2020 18:22:08] code 400, message Bad request syntax ('4')
 0:04.54 INFO STDERR: <IP_ADDRESS> - - [13/Jun/2020 18:22:08] "4" 400 -

When sending a request whose body would be the string "Test".
So I'm guessing this actually matches the concept of https://en.wikipedia.org/wiki/Chunked_transfer_encoding#Encoded_data, where the 4 is the first line indicating that the chunk is of size 4?
I have to say that I did end-up setting the "Transfer-encoding" chunked header, although I think I was seeing this error even before, which would seem to indicate that Hyper itself switches to that protocol when using a Stream as a request body? If so, I think it is worth documenting in more details.
Also, are there any requirements on how to use the corresponding Sink, for example the need to gracefully close it and so on?

If the body's size_hint method does not return an exact size, the hyper will check if you've included a specific header to describe the body. If that doesn't exist either, hyper will resort to a streaming body, which over HTTP/1.1, will use chunked transfer-encoding. It should automatically add the header.

thanks for the info!


Option to change Petrarch version to Petrarch2

Update the pipeline so it works with Petrarch2. The Petrarch version is set by a new option in the config file. The code is pretty simple and I've tested all the way through with both and everything seems to work fine. @johnb30 or @philip-schrodt, want to double check that it looks okay?

@johnb30 The latest commit should have fixed everything you pointed out. Once you give me a 👍  I'll merge to master.

👍


'module' is undefined

Hi Team after i debug i am getting this error.
'module' is undefined

Sorry, but its more likely that is a problem with your code. Please provide a more descriptive stack trace next time.


Fix: Do not set the cookie domain by default

By setting the domain attribute on the cookie we were allowing the
cookie to be applied to subdomains where it may not be valid and may
interfere with other services protected by keycloak-gatekeeper.
(For example, a gatekeeper running on https://domain.com could break a
gatekeeper running on https://sub.domain.com .)
Instead, we should simply not set the attribute unless there is
a specific domain configured.
For more information please see section <IP_ADDRESS> of RFC 6265.

@daniel-ac-martin maybe you missed this, but please read https://github.com/keycloak/keycloak/blob/master/CONTRIBUTING.md and follow these guidelines when contributing to Keycloak

@abstractj: Thanks for letting me know. I've raised the issue on the mailing list. Hopefully that will lead to a JIRA ticket being created.

Oh, it seems my e-mail to the list was blocked. You guys make it quite hard to contribute!

@daniel-ac-martin not sure what happened with your e-mail, or why it was blocked. Have you subscribed to the mailing list?
About your change, please follow the contribution guidelines and if there's something we can improve to make the process better. Let us now.

@abstractj: I wasn't subscribed. - Didn't realise that was a requirement.
Just subscribed and re-sent the e-mail. Hopefully, it will get through this time!

@abstractj: JIRA created at https://issues.jboss.org/browse/KEYCLOAK-10668 and this PR has been updated to reference the JIRA.

@abstractj: Do you think you will have time to look at this soon?

@daniel-ac-martin as far as I know, setting host as default domain is a sensible default.
I think your feature should rather be explicit, e.g. "DisableCookieDomain".

@fredbi: Why would you want the cookie to be applied to subdomains by default? Surely it is safer to limit the cookie to just the hostname?

Howdee! Any chance of this being merged and fixed anytime soon?

@daniel-ac-martin

@fredbi: Why would you want the cookie to be applied to subdomains by default? Surely it is safer to limit the cookie to just the hostname?

Yes. I concur. You made a good point. Looks good to me.

@daniel-ac-martin @fredbi one thing concerns me about this change, is this part of RFC:
WARNING: Some existing user agents treat an absent Domain
      attribute as if the Domain attribute were present and contained
      the current host name.  For example, if example.com returns a Set-
      Cookie header without a Domain attribute, these user agents will
      erroneously send the cookie to www.example.com as well.

Correct me if I'm wrong, but that would open the possibility to malicious attackers to setup a subdomain in the internal network and get the cookies for example.com for example.
One alternative to make this change possible is to provide something like:
--cookie-domain=www.example.com where users explicitly tell which hosts to the cookie will
be sent
@fredbi @daniel-ac-martin does it make any sense to you?

Correct me if I'm wrong, but that would open the possibility to malicious attackers to setup a subdomain in the internal network and get the cookies for example.com for example.

Is that not the status quo in anycase?
i.e. This change fixes the problem for modern, RFC-compliant clients but not for non-compliant ones.
Also, if --cookie-domain=www.example.com was a thing then a bad actor might still be able to stand up foo.www.example.com to steal the cookies.

@abstractj: See above. (Forgot to tag you!)

Thanks @daniel-ac-martin, the way how it works today I believe bad actors can do this, at the same time, it is bad for the reasons you mentioned in this PR. But there must be something in between that can make this change possible to keep us safe and at the same time fix the issue you mentioned.
I was thinking about get some inspiration from https://tools.ietf.org/html/rfc4592 about wildcards (we do something similar on Keycloak server for Web Origins):

--cookie-domain=www.example.com the cookie will be strictly applied to the domain specified
--cookie-domain=*.example.com we permit cookies to be applied to subdomains

Would that make sense for you?

@abstractj: I think the question is how do we get the bad clients to honour any of that?
This change should make things safe for RFC-compliant clients.
For other clients I would think you would need to:

Understand what incantation is required to coax out secure behaviour (if any)
Employ the incantation based on the presence of a match against a user agent string

Or am I missing something?

@daniel-ac-martin thank you, let's merge it for now. I don't see any issues with this and nobody raised any concerns on the mailing list. So we are good to go.
Thanks for this change.


New version: OnlineStatsBase v1.1.1

Registering package: OnlineStatsBase
Repository: https://github.com/joshday/OnlineStatsBase.jl
Created by: @joshday
Version: v1.1.1
Commit: d96aaf89c189bcfa88ce6998eeaf024f599de2eb
Reviewed by: @joshday
Reference: https://github.com/joshday/OnlineStatsBase.jl/commit/d96aaf89c189bcfa88ce6998eeaf024f599de2eb#commitcomment-37859712

My apologies, this can be closed in favor of https://github.com/JuliaRegistries/General/pull/11052


Pointcloud to laserscan dev

@vmayoral Hi! This benchmark is now complete and ready to be ran on your hardware. Please take a look and let me know if any improvements should be made. This benchmark evaluates the performance of the PointCloudToLaserScanNode within the pointcloud_to_laserscan package which converts a point cloud to a 2D laserscan. An example is illustrated here using the r2b data:

A view of the computational graphs can be found within the /imgs directory or here
This pull request also incorporates the changes that I needed to make to the tracetools_benchmark package in order to create custom tracepoints (i.e. robotperf_pointcloud_input_cb_init and robotperf_pointcloud_input_cb_fini)

Thanks a lot @jasonjabbour! Thanks for the contribution. Looking good on a first look. Let us book some time to try and reproduce it.
We'll take a look at this ASAP.


explain always true when invoked from commandline

Looking here:
parser.add_argument('--explain', default=True, action='store_true', help='explain')

Since default=True, args.explain will always be True, I think, even if --explain is not given.
I noticed this because of a discrepancy between running popper from the command line vs. running from another python program. The former was much faster than the latter, and it was because explain was true in the former but not the latter.
Btw, what does explain mean?

fixed by enabling explain by default https://github.com/logic-and-learning-lab/Popper/commit/1c69be11ff5313f28fec6a57520029589fd6d278
the explain setting tells Popper to find the minimal unsatisfiable core of a totally incomplete program (a program that does not cover any positive examples)


WHERE functionalities are slightly changed.

I usually used Where method like this.
builder.Select().From().Where(checkFilterFunction())

And inside of WHERE, the array can have condition or not. Based on the internal functions. If the return of checkFilterFunction() is nil, previously there was no WHERE in the query. But now, even when the condition is null, it is placing WHERE, and there are no following query, so it causes syntax error on latest version.
When I check the function,
// Where sets expressions of WHERE in SELECT.
func (sb *SelectBuilder) Where(andExpr ...string) *SelectBuilder {
	if sb.WhereClause == nil {
		sb.WhereClause = NewWhereClause()
	}

	sb.WhereClause.AddWhereExpr(sb.args, andExpr...)
	sb.marker = selectMarkerAfterWhere
	return sb
}

I think it can be changed like this,
// Where sets expressions of WHERE in SELECT.
func (sb *SelectBuilder) Where(andExpr ...string) *SelectBuilder {
	if andExpr == nil {
		return sb
	}

	if sb.WhereClause == nil {
		sb.WhereClause = NewWhereClause()
	}

	sb.WhereClause.AddWhereExpr(sb.args, andExpr...)
	sb.marker = selectMarkerAfterWhere
	return sb
}

Any thought?

It should be a bug. I'll fix it. Thanks for reporting this to me.

The fix is in tag v1.27.2.

@huandu Thanks for the quick fix!


WIP Dst 456

Users will not be able to jump ahead in tasks even if they have the steps url. Additionally, for users still within the Your Details task, they will not be able to jump ahead to the final two steps in that task until they have verified their email.

Codecov Report
Attention: Patch coverage is 86.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 80.58%. Comparing base (1ceea5c) to head (4c55687).
Report is 7 commits behind head on main.

Files
Patch %
Lines

django_app/report_a_suspected_breach/views.py
69.23%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   80.49%   80.58%   +0.09%
==========================================
  Files          64       64
  Lines        1994     2024      +30
==========================================
+ Hits         1605     1631      +26
- Misses        389      393       +4

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

@jenriordan I would take advantage of the def can_start method of each of the tasks in the TaskList. In the get method of the view, you can check if the current task (retrieved with self.tasklist.current_task) has can_start True or False, if it's False then we raise the error view


not recognizing lon, lat

Context
Yesterday I imported the dataset and lon, lat was identified and so the the_geom column was populated. Today I run the same query and the the_geom column is not populated with the lon, lat.
Steps to Reproduce
1.Create new table
2. run sql query : INSERT INTO {table name} (lat,lon,description,combined_key,confirmed,deaths,recovered) VALUES ('34.22333378','-82.46170658','location_summary','Abbeville South Carolina US','4','0.0','0.0')
3. tried also changing column names to latitude and longitude.
4. tried uploading csv directly but didn't work either. Yesterday I ran the same query and it worked.
Current Result

I created a new account and it worked.
Expected result
I expect lat lon to be recognised
Browser and version
chrome Version 80.0.3987.132 (Official Build) (64-bit)

@CartoDB/rt-managers

@dandevmsc Running INSERT operations on a table doesn't populate the_geom column unless you explicitly insert data to that column.
If you want to populate the_geom column with an INSERT operation, then you would need to apply a query like the next one (using ST_Point() and ST_SetSRID() PostGIS functions):
INSERT INTO {table name} (the_geom, lat,lon,description,combined_key,confirmed,deaths,recovered)
VALUES (ST_SetSRID(ST_Point(34.22333378, -82.46170658'),4326)'34.22333378','-82.46170658','location_summary','Abbeville South Carolina US','4','0.0','0.0')

The column the_geom is only populated during the import process when the next conditions are met:

You have the import type guessing option activated.
The longitude and latitude columns are number type and not text types.
The longitude and latitude columns have one of the next names: longitude,lon,lng,longitudedecimal,longitud,long,decimallongitude,decimallong,point_longitude,latitude,lat,latitudedecimal,latitud,lati,decimallatitude,decimallat and point_latitude

Please, use of the<EMAIL_ADDRESS>email address to ask technical questions or problems or if the problem remains after checking the previous steps.

@oriolbx  thanks! that is clear now.
there was just a minor syntax error in the above code, this works:
INSERT INTO {table name} (the_geom, lat,lon,description,combined_key,confirmed,deaths,recovered)
VALUES (ST_SetSRID(ST_Point(34.22333378, -82.46170658),4326),'34.22333378','-82.46170658','location_summary','Abbeville South Carolina US','4','0.0','0.0')


Evaluate cryptonite library

We should evaluate the cryptonite library, and consider if there are other options we might want to rely on instead.
Candidates:

https://github.com/Yubico/libfido2 (C library)

@arianvp also thought about not depending on cryptonite in https://github.com/arianvp/haskell-fido2/issues/25

Using libfido2 is more than acceptable. Creating the bridge from Haskell to the C library might be hard, but we get most of this back in not having to implement our own logic for the crypto part. We still have to decide on the right API, but we will use libfido2.


How to use marzipano in vue

Is there any example of how to use this library on VUE?

in template you can add simple div:

than use it in script like this:
var element = this.$refs.test;
var viewer = new this.$marzipano.Viewer(element);

Thank you, you are the bes

Thank you

Do you know why i get this error? ERROR: 0:1: '.' : syntax error