Fix command stuck loading

For configuration or updating questions please read the README and UPGRADE documentation,
or visit: https://gitter.im/PHP-CS-Fixer
When reporting an issue (bug) please provide the following information:
The PHP version you are using:
$ php -v
PHP 7.1.11-1+ubuntu16.04.1+deb.sury.org+1 (cli) (built: Oct 27 2017 13:49:56) ( NTS )
PHP CS Fixer version you are using:
$ php-cs-fixer -V
PHP CS Fixer 2.8.0 Round Numbers by Fabien Potencier and Dariusz Ruminski
The command you use to run the fixer (and the configuration file you are using if any).
./vendor/bin/php-cs-fixer fix --verbose

Hello,
I've updated an old project to phpcsfixer v2 and its configuration.
However, when running the phpcsfixer fix command, it's not doing anything and is stuck on:
You are running PHP CS Fixer with xdebug enabled. This has a major impact on runtime performance.
If you need help while solving warnings, ask at https://gitter.im/PHP-CS-Fixer, we will help you!
Loaded config default from "/Users/nicolaswidart/Sites/Freelance/projectname/website/.php_cs.dist".

When I run it on a specific file, it's working:
./vendor/bin/php-cs-fixer fix app/User.php

This same configuration file is working for other projects though, so I'm not sure why it's not here.
Did I miss something?
Thanks,

Hi,
Do you have a file with a lot of (anonymous) class definitions in it? That is the only performance issue we know of ATM. Might be hard to track down, but could you try fixing (--dry-run) different directories to see if you can isolate on which file the fixer hangs?

Could it also be that you are running it for the whole project, including the vendor?
Normally I directly tell it to run on ./tests and ./src to avoid it running on folders I don't want to.

It's also not running when I run it on the app/ folder only.
It's running for the whole project, but as you can see from the config file, a lot of folders are excluded like vendor, node_modules, etc.
There are indeed anonymous classes, trying to search where those are.

@nWidart
is it possible to somehow share access to project that error happen on ? (maybe it;s on github?)
that would really help us investigate what's going on, as lack of any CLI output is not helpful for debugging/fixing :(
(if it is not public but you are willing to share it temporary - please contact me directly)

@SpacePossum , anonymous classes are not the reason IMO, as we shall still start displaying progress indicator, and fixing single file, even with a lot of anonymous class, wont take minutes.

Sadly, closing due to lack of @nWidart presence

Well the issue is still present.
I cannot provide access to the repository as it's a private client project sadly.

oh, you are alive!
without any way to reproduce you are sadly on your own. if we cannot experience the issue, there is not much help we could provide. try to shrink down the issue to minimal case that you would be able to share. or consider granting temporary access to one of us as external experts.
also, upgrade to 2.8.1 and try out PHP_CS_FIXER_FUTURE_MODE=1 php-cs-fixer fix -vvv.
If it won't provide more details and you won't be able to shrink down the case to shareable one, try to debug FixCommand.php file to see in which moment it is freezing.

@keradus I think I have a similar issue. I let php-cs-fixer run on a file tree, and it progresses rapidly for a while and then it just gets stuck. I managed to figure out at least one file where it gets stuck. Unfortuantely, I can't share it. But the file is basically the export of a DB table in PHP array format. It's about 1400 lines with a structure like this:
$t_user = array(
    array('id' => '1','m_nr' => '7','status' => 'XXX','nachname' => 'XXX','vorname' => 'XXX','titel' => '','jahrgang' => 'XXX','ans_p' => 'XXX','ort_p' => 'XXX','land_p' => '','tel_p' => '','fax_p' => '','email_p' => 'XXX','firma' => 'XXX','ans_g' => '','ort_g' => 'XXX','land_g' => '','tel_g' => 'XXX','fax_g' => '','email_g' => 'XXX','username' => 'XXX','passwort' => 'XXX','geb' => 'XXX','mobil_p' => '','mobil_g' => '','abt_g' => '','pos_g' => 'XXX','branche_g' => '','hp_p' => '','hp_g' => 'XXX','prakt_firma' => 'XXX','prakt_abt' => '','prakt_bran' => 'XXX','prakt_ans' => 'XXX','prakt_ort' => 'XXX','prakt_tel' => 'XXX','prakt_hp' => 'XXX','prakt_auf' => '','prakt_tip' => '','dipl_stuhl' => 'XXX','dipl_thema' => 'XXX','dipl_betr' => 'XXX','dipl_tip' => '','ausl_uni' => 'XXX','ausl_ort' => 'XXX','ausl_land' => 'XXX','ausl_sem' => 'XXX','ausl_hp' => '','ausl_tip' => '','st_beginn' => 'XXX','ts' => 'XXX','gebname' => NULL),
    /* and so on..*/
);

This is the command I'm running:
php-cs-fixer fix -vvv --rules=no_spaces_after_function_name,elseif,braces,encoding,full_opening_tag,single_blank_line_at_eof,function_declaration,line_ending,method_argument_space,no_trailing_whitespace_in_comment,no_closing_tag,switch_case_semicolon_to_colon,switch_case_space,no_trailing_whitespace,whitespace_after_comma_in_array,whitespace_after_comma_in_array,no_whitespace_before_comma_in_array,no_empty_statement file.php

I let it run on this single file for something like 10mins on a Xeon E5-1620 3.5GHz machine. In top I see a PHP process that takes 100% CPU during that time and something like 200MB memory. I stopped with Ctrl-C after a while, because nothing was happening. Does this help?

P.S.: I would be very nice if -vvv would print out the filename it's currently processing, that would make it a lot easier to see where it chokes

@flack , please raise a new issue following the template - and for the example file you found problems with, simply anonymise the data, without example file, one won't be able to reproduce.
PS go ahead and create a proposal for 2.12 branch

@keradus I'm not exactly sure how to (non-manually) anonymize the data without destroying the wacky formatting that the file has (because I guess that's a factor here), but I'll take a look. If I find a way, I'll open a new ticket

@keradus filed as #3996


Access to API

Hi,
I can log in and create datasets and do what I need in builder. But I cannot access the sql or import api.
How can you access the APIs in the container?
Kind Regards.

The container is configured with subdomainless urls
The sql api is running at http://cartodb.localhost/user/{username}/api/v2/sql?api_key={api_key}&q={SQL query}
The import api is running at http://cartodb.localhost/user/{username}/api/v1/imports/?api_key={api_key}

Hi @sverhoeven , thanks for replying.
The SQL API works great thanks.
I can access the import api endpoint in the browser now, which is more promising, but I get an error with CURL.

Trying <IP_ADDRESS>...
TCP_NODELAY set
Connection failed
connect to <IP_ADDRESS> port 80 failed: Operation timed out
Failed to connect to <IP_ADDRESS> port 80: Operation timed out
Closing connection 0
curl: (7) Failed to connect to <IP_ADDRESS> port 80: Operation timed out

I am following the api information here https://carto.com/developers/import-api/reference/# and trying to import a file.

Hello @sverhoeven
How about accessing the Maps API?


Add acceleration to ProjectileComponent

The new addition should be accompanied by an associated test

Acceleration as force (with direction) or as desired constant speed change rate?

Acceleration as force (with direction)

Yes. So a vector with type Point2D in pixels per second, which should be applied to velocity each frame.


Task not firing on the next executor if app was killed

I think I have similar problem like this:
https://github.com/redisson/redisson/issues/2308
Expected behavior
I have 2 app instances with distributed executor services.
I expect, that if one instance will be killed another instance will start executing the task after specified time (taskRetryInterval property).
So it works sometimes. Sometimes it's not.
Actual behavior
If task had been submitted on executor and app instance was killed it doesn't works.
But after restarting the instance the feature starts working correctly. If the instance with submitted task was killed another one starts executing it.
Also sometimes I'm getting duplication of tasks. But this happens very rarely, I can't reproduce it.
Steps to reproduce or test case

Start first app instance
Start second app instance
Submit task
Stop app instance which executing the task
Awaiting that second app instance will start executing the task. But nothing happens
Start killed app instance
Random instance starts executing the task. After that case not reproducing and it's working correctly. But only for this task

Redisson
3.26.0
KeyDB
6.3.3
    // Spring config
    @Bean(destroyMethod = "")
    public RScheduledExecutorService executorService(RedissonClient redissonClient) {
        ExecutorOptions executorOptions = ExecutorOptions.name("test-1")
                .taskRetryInterval(Duration.ofSeconds(10));

        RScheduledExecutorService executorService = redissonClient.getExecutorService(executorOptions);

        WorkerOptions workerOptions = WorkerOptions.defaults();
        workerOptions.executorService(Executors.newFixedThreadPool(3));
        workerOptions.workers(3);
        workerOptions.taskTimeout(0, TimeUnit.SECONDS);

        executorService.registerWorkers(workerOptions);

        return executorService;
    }

    // Task execution
    public void run() {
        executorService.submitAsync(new Task(UUID.randomUUID()));
    }

    // Task
    @Data
    @AllArgsConstructor
    public class Task implements Callable<Long>, Serializable {

        private UUID uuid;

        @Override
        public Long call() {
            System.out.println("started");
            System.out.println(uuid);
            try {
                TimeUnit.HOURS.sleep(2);
            } catch (Exception e) {
                e.printStackTrace();
            }

            System.out.println("finished");

            return 1L;
        }
    }

Manual message in executor's scheduler channel starts task executing on another instance if first one was down.
E.g.
redissonClient.getTopic("{test-1:org.redisson.executor.RemoteExecutorService}:scheduler-channel", new LongCodec())
        .publish(System.currentTimeMillis());

Merged


eval $(weave env) not working with fish

eval $(weave env) not working with fish
fish using "set VAR VALUE"
to set a global variables

This came up before, in #1269.
Would you like to submit a pull request to make it work?

Sorry, i did not have  so much experience on coding.


name 'LoadFirmwareImage' is not defined

I get this error when I try to dump a partition:
./esp32_image_parser.py create_elf flash.bin -partition app0 -output app0.elf
Dumping partition 'app0' to app0_out.bin
Traceback (most recent call last):
  File "softs/esp32_image_parser/./esp32_image_parser.py", line 281, in <module>
    main()
  File "softs/esp32_image_parser/./esp32_image_parser.py", line 264, in main
    image2elf(dump_file, output_file, verbose)
  File "softs/esp32_image_parser/./esp32_image_parser.py", line 41, in image2elf
    image = LoadFirmwareImage('esp32', filename)
NameError: name 'LoadFirmwareImage' is not defined

This can be fixed in esp32_image_parser.py if you add from esptool.bin_image import *:
import sys
import json
import os, argparse
from makeelf.elf import *
from esptool import *
from esptool.bin_image import *
from esp32_firmware_reader import *
from read_nvs import *
...

thx

can you make a PR for this? just so everyone's got the fix

thank


auto-cpufreq: 1.6.4 -> 1.6.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for auto-cpufreq is: "Automatic CPU speed & power optimizer for Linux"
meta.homepage for auto-cpufreq is: "https://github.com/AdnanHodzic/auto-cpufreq"
meta.changelog for auto-cpufreq is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9/bin/auto-cpufreq passed the binary check.

/nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9/bin/cpufreqctl.auto-cpufreq passed the binary check.

/nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9/bin/.auto-cpufreq-wrapped passed the binary check.

3 of 3 passed binary check by having a zero exit code.

2 of 3 passed binary check by having the new version present in output.

found 1.6.9 with grep in /nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9

found 1.6.9 in filename of file in /nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9

directory tree listing: https://gist.github.com/f17674ddfae6a2f5ed8c7e8dea787bfe

du listing: https://gist.github.com/4a17232e5d74eca87b7fcdc0d9c27a62

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
auto-cpufreq

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A auto-cpufreq https://github.com/r-ryantm/nixpkgs/archive/e35d881ac3cad7d9a52f5bbf6504bacf8aebfcfd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9
ls -la /nix/store/fqbhlvgp81pk8b1lxilxgc3i1djnl7m1-python3.9-auto-cpufreq-1.6.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

auto-cpufreq

Maintainer pings
cc @Technical27 for testing.

Result of nixpkgs-review pr 139082 run on x86_64-linux 1

1 package built:

auto-cpufreq


Firmware download fails - IndexError: list index out of range

pethublocal fails to download firmware with the following error. Does not seem linked to DNS / network issue as I tried from two different hosts on different networks.
Download Credentials and Firmware for Hub (highly recommended)? Y/N y
INFO:pethublocal:Current Hub Firmware 233.364
INFO:pethublocal:Downloading Current Firmware for H008-xxxxxxx
INFO:pethublocal:SureHub Host hub.api.surehub.io IP Address: <IP_ADDRESS>
INFO:pethublocal:Downloading first firmware record to get header information
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "POST /api/firmware HTTP/1.1" 200 0
Traceback (most recent call last):
  File "/home/ubuntu/test/bin/pethublocal", line 8, in <module>
    sys.exit(cli())
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/ubuntu/test/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/ubuntu/test/lib/python3.10/site-packages/pethublocal/cli/__init__.py", line 72, in setup
    pethubconfig = config_load(True, force)
  File "/home/ubuntu/test/lib/python3.10/site-packages/pethublocal/functions.py", line 379, in config_load
    log.info(download_firmware(serial_number, force))
  File "/home/ubuntu/test/lib/python3.10/site-packages/pethublocal/functions.py", line 104, in download_firmware
    recordcount = int(byte[2], 16) + 6
IndexError: list index out of range

Same issue here, did anyone find a solution for this or is it the end of the road for this project?

Also seeing the same thing, seems to return a 500 response

This seems to be because SureFlap are returning a 0 byte response based on the request. The new firmware 233.364 must have bumped the bootloader and I am worried they have moved over to HTTPS for retrieving the firmware which will break everything unfortunately. I don't yet have my hands on a hub that has the new firmware to confirm, but am contacting a friend today to see if I can borrow theirs.

Thanks @plambrechtsen, really appreciate your effort to try to bring this amazing project back to life. I guess getting your hands in one of these hubs is the best idea as you can play around with it and see what the new firmware is doing, but just in case if there is anything we can do to help just shout.

Last week sureflap decided to block the surepy library by its user agent and bricked everyones integrations - I think that has renewed a desire for local control (it has for me at least!). Hopefully it can be fixed.

Last week sureflap decided to block the surepy library by its user agent and bricked everyones integrations - I think that has renewed a desire for local control (it has for me at least!). Hopefully it can be fixed.

I did not notice an issue in my PHL integration - or are you talking about their own Sure Pet app (another shiny outage for all their users)?

Neither in this case, surepy is a open source library someone wrote to interact with the Sure Pet api without needing to use the Sure Pet app.
This thread explains it better - https://github.com/home-assistant/core/issues/93814
Sure Pet wont engage with the community in any way and have now shown they are willing to take additional measures to block 3rd party access meaning the solution most people are using is looking like it might not have much of a future.

@Ryanmt2 Read through the issue ... that's insane :-(
I really have no clue why they're fu**ing up their user base.
Does anybody know what wireless protocol is used between the hub and the flaps (@plambrechtsen )?
In case it's some standard (i.e. Zigbee) I hope someone will create or start funding a project of an independent hub replacement on a Raspberry ...
In case I need to replace a flap one day I'm really, really concerned to even shortly connect my hub back to their cloud to get a proper new config. And I hope my hub never dies. PHL is running so great here (apart from the message-of-death issue of the hub, but my watchdog gets it back to life then after max 70 minutes), we'll never want to miss the local installation anymore!

Here's hoping all of us on the new firmware aren't screwed, but I'm guessing we are.

Does anybody know what wireless protocol is used between the hub and the flaps (@plambrechtsen )?

According to some comments in the thread @Ryanmt2 mentioned (i.e. https://community.home-assistant.io/t/local-deployment-for-sureflap-surepetcare-connect-using-only-local-mqtt-broker/274494/295) it seems the communication between the hub and the flap is zigbee, but I am not sure if it is standard zigbee and with a known key so it can be reproduced. Obviously being able to connect the flap to our zigbee network so the hub is not required anymore would be a dream. @plambrechtsen might have more details about this.

https://github.com/plambrechtsen/pethublocal
The original repo details some alterative hub options that were investigated before parking the idea in favour of the current solution. It is indeed zigbee
The wemos d1 hub sounds like a dream if there is ever any suitable firmware created!

If I recall correctly, it is essentially ZigBee but with enough of a change to the packet headers (or something like that) so that it's not recognised as ZigBee and thus needs it's own transceiver to receive the data.
Obviously previously the docker software stack and the firmware downgrade was easier than setting up custom transceiver hardware, but if sure pet are actively trying to prevent that then that may no longer be the case.
I guess the advantage of a hardware solution would be that in theory the OEM hub and app could work alongside the home-made one and just sniff the data.
As long as they can't remotely update device firmware.....

You are right, I have done a small test disconnecting the sure hub, then enabling the "permit join" in my zigbee2mqtt and then putting the flap in pairing mode (which according to the app just require to click the settings button located at the left), and unfortunately the flap has not joined the zigbee network (1st step of the guide https://www.zigbee2mqtt.io/advanced/support-new-devices/01_support_new_devices.html), so it seems they have changed the protocol enough so that it is not recognised by the zigbee coordinator, which is a pity because it would have been the easiest solution.

Hello,
I can't find the link back but at some point people mentioned a microchip
implementation of this 2.4GHz zigbee-like protocol. There is probably some
documentation available for the communication protocol but figuring the
binary representation of states and commands would probably be a
challenge....
I also think bypassing the hub is the way to go though...
Another (challenging) option is to go lower level and plug an esp32 right
to the sensors/actuators... When mounting the cat flap I could see a motor
to control the lock, a magnetic sensor on the door hinge (probably to infer
the cat direction), and an antenna coupled to a chip-reader chip (?)... Not
sure how difficult (probably a lot) but depending on the people that are
motivated to give it a try, there are different layers to attack.
Not complaining about the physical build, which is a good base, otherwise
it might be simpler to just start a DIY project from scratch ;)

It's going to be a heavy lift to replace the hub. I doubt that's likely to happen. And it may not even matter if we can't downgrade the firmware. It's possible they've locked us out of everything.

Another (challenging) option is to go lower level and plug an esp32 right to the sensors/actuators... When mounting the cat flap I could see a motor to control the lock, a magnetic sensor on the door hinge (probably to infer the cat direction), and an antenna coupled to a chip-reader chip (?)... Not sure how difficult (probably a lot) but depending on the people that are motivated to give it a try, there are different layers to attack. Not complaining about the physical build, which is a good base, otherwise it might be simpler to just start a DIY project from scratch ;)

I had a go at this before I came to this project.
I've got a regular dual scan cat flap that I pulled apart. As you say, the lock, magnetic sensor, and the IR are all fairly easy. But I couldn't get anywhere with the RF coil. The apparently have to be carefully tuned and I couldn't get close.
I think the way forward is with working on intercepting the packets in flight with an appropriate ZigBee sniffer that will recognise the modded packets. At the moment (thanks to this project) we can see read the actions that each transmission performs so hopefully it's a just a case of process of elimination.
As far as I know, the end devices can't be updated remotely, so whatever they do with the hub won't matter, it will still have to speak unencrypted to the devices.

In case anyone may have missed it, @plambrechtsen's response about rolling your own hub is towards the bottom of this post. It lists some of the things that would need to be done.
https://community.home-assistant.io/t/local-deployment-for-sureflap-surepetcare-connect-using-only-local-mqtt-broker/274494/429

I was planning to buy a Cat Flap Connect + hub soon, but I'm pretty dedicated to local-only control.  Would a new device likely be impacted by this if it's never been connected to the cloud/can I set it up without having to connect it?  Does this mean I should wait and see how this resolves?
The thread linked below indicates the issue is resolved for the cloud-based HA integration.
https://github.com/home-assistant/core/issues/93814

My understanding: it is quite a risky endavaour for two reasons:

The hub you are buying might already have a newer firmware that won't allow the downgrade
You must ensure that the hub will never ever get an internet connection


Adding a couple of project ideas

Hello !
Here are 2 project proposal for the Hackathon, happy to work on something else as well !
Best,

@oliche Thanks for the projects!
FYI I started porting some of the destriping code in SI! https://github.com/SpikeInterface/spikeinterface/pull/508
Also, I think we could merge the IBL benchmark into a more general project to improve QM and add them to SF? See https://github.com/catalystneuro/spike-sorting-hackathon/pull/8 proposal from @samuelgarcia


PostgreSQL repository sources removed - unable to install extensions

Description
Is removal of postgresql apt sources really necessary at https://github.com/actions/virtual-environments/pull/3091 ?
Postgresql integration tests rely on installing extensions to the database. PostGIS is a common one.
But trying to install them without original repository downloads not only incompatible versions from distribution's main repository, but also conflicting lower version of postgres-12 is installed in parallel with existing working version 13.
As first step in all builds I have to readd the sources and rerun repository synchronization, which, besides being not obvious to do, adds time and bandwidth to the build run:
#####
## This is to undo removal of postgres repository sources that
## https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/installers/postgresql.sh
## end of the script removes
######
REPO_URL="https://apt.postgresql.org/pub/repos/apt/"
echo "deb $REPO_URL `lsb_release -cs`-pgdg main" | sudo tee /etc/apt/sources.list.d/pgdg.list
sudo apt-get update

sudo apt-get -y install postgresql-13-postgis-3
sudo -u postgres psql -c "create extension postgis" integration_test_db

Virtual environments affected

[ ] Ubuntu 16.04
[X] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20210816.1
Is it regression?
No response
Expected behavior
Can install postgres extensions to complete integration tests.
Actual behavior
Because sources for currently installed postgres version are removed, not only incompatible versions are downloaded from main repository, conflicting lower version of postgres is installed in parallel with existing working version.
Repro steps
sudo apt-get -y install postgresql-13-postgis-3
sudo -u postgres psql -c "create extension postgis" integration_test_db

Hello @alextech,
Using apt repositories brings a bunch of problems and potential risks for the GitHub Actions CI environment so we have reduced the number of apt sources. For more information, please see this issue.

I'm going to close this issue. Feel free to contact us in case of any questions.


sort: invalid option -- V

list_hooks_in_dir causes an error with GNU sort version 5.93 (as installed in OS X 10.5) due to the sort -V option on line 45
I'm not sure what the -V option does so I haven't submitted a patch.
Cheers,
Sean.

While there is a darwin now, I am removing the -V completely so this will be resolved.

09b39381ecff895d4b3d6f340390ae71af727f2a


Integrating with yii2-yee-post & yii2-yee-page

Is possible integrate the yii2-yee-seo with yii2-yee-post & yii2-yee-page? (like yoast in wordpress)

@msoa currently there no such feature. You need to create all seo records by yourself. Maybe this feature will be implemented within 0.2.x release.

Release is near?


Why use os matrix?

Expected Behaviour
Code is executed once on ubuntu-latest.
Actual Behaviour
os matrix is used, implying that other os' could be added, which will cause the code to run more than once.
The actual action code checks if it is running in ubuntu and will 'do less' if it is not, but it should just not happen.

should this issue be in @adobe/generator-aio-app repo instead?

There are no Github Actions files in this repo. Closing.


Clarify the guarantees of strongly consistent queries

Please clarify if the strongly consistent queries mode provides only a "read-your-write" guarantee for a specific client session, or a stronger guarantee such that a write performed by some session is immediately visible to all other sessions.
The current documentation uses the term "you", which is ambiguous.

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

ID: 93aec2dc-c6c4-766d-421f-50e4905b4e8b
Version Independent ID: 92192f5c-e7cc-5d6a-adcb-1b4969dfe113
Content: Query consistency - Azure Data Explorer
Content Source: data-explorer/kusto/concepts/queryconsistency.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@Spoonbender -
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

Thanks @Spoonbender for reaching out.
Response from @zivcaspi:
There are no “sessions” as such in Kusto today. Once a “write” transaction completes successfully, any subsequent strongly-consistent “read” transaction will “see” its results.

@YaelSchuster thank you for the clarification!
I suggest clarifying in the document itself as well, since the doc uses the term "read-my-changes" which resembles the well-known "read your write" (a.k.a "read my write") consistency model, which provides weaker consistency than what Kusto provides (See https://jepsen.io/consistency/models/read-your-writes and https://en.wikipedia.org/wiki/Consistency_model#Read-your-writes_consistency )... for your consideration.
Either way this issue can be closed as far as I'm concerned :-)

@Spoonbender thanks for the references. I'm going to close the issue at this time.
#please-close


Malformed WorkflowTemplate crashes whole UI

Checklist:

[x] I've included the version.
[x] I've included reproduction steps.
[x] I've included the workflow YAML.
[x] I've included the logs.

What happened:
Deployed a WorkflowTemplate with kubectl (really helm) that was malformed, but still accepted by K8s API server (fit CRD apparently). This broke the WorkflowTemplate UI list view (http://localhost:2746/workflow-templates), see screenshot below.
argo cli would have caught it:
# argo template create -n argo src/test/resources/workflow-template.yaml
020/08/04 17:32:17 Failed to parse workflow template: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal object into Go struct field Arguments.spec.templates.dag.tasks.arguments.parameters of type []v1alpha1.Parameter

but we will be deploying our workflow (-template) together with our application as a single helm chart because we want the declarative resource management, so argo cli isn't really an option.
What you expected to happen:
The WorkflowTemplate UI's list view should as least load and show me any WorkflowTemplates that are indeed syntactically correct. Put another way, I expect not to be prevented from working with my valid WorkflowTemplates, just because yesterday's newly deployed WorkflowTemplate is broken.
Particularly troubling is that the entire UI, including sidebar, disappears. Since it's only an ajax api request (http://localhost:2746/api/v1/workflow-templates/argo) that fails, I would expect to at least get a sensible error within the WorkflowTemplate UI about what has gone wrong.  Ideally I would even be allowed to see the valid WorkflowTemplates and be informed about the broken one separately, maybe with a warning icon on the list item. While this could presumably be fixed by making the api and frontend smarter, another 500 will crop up in the future and break the UI somewhere else, and I would love not to be kicked out of the UI entirely whenever the server fails to provide some data.
On that note, the error page seems to have a double-redirect which breaks the browser back button, so I can't even get back to the main page that I was on previously. Should I post a separate issue about that?
How to reproduce it (as minimally and precisely as possible):

start with blank docker for mac kubernetes environment (haven't tried other setups, but I suspect any kubernetes environment should work)
setup argo

kubectl create ns argo
kubectl -n argo apply -f https://raw.githubusercontent.com/argoproj/argo/stable/manifests/quick-start-minimal.yaml
kubectl port-forward -n argo deploy/argo-server web

deploy a WorkflowTemplate that Kubernetes accepts (fits CRD spec), but is technically malformed in one place (maybe CRD spec isn't strict enough?)

# workflow-template.yaml
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: some-template
spec:
  entrypoint: main
  templates:
    - name: main
      dag:
        tasks:
          - name: print
            template: print
            arguments:
              parameters:
                # this should be
                # parameters:
                #   name: someParam
                #   value: "Hello, world!"
                someParam: "Hello, world!"
    - name: print
      script:
        image:  busybox
        command: [sh]
        script: |
          echo input was: {{inputs.parameters.someParam}}

kubectl -n argo apply -f workflow-template.yaml

visit http://localhost:2746/workflow-templates
watch the entire ui disappear behind an unfriendly API 500 error page:

Anything else we need to know?:
Environment:
Docker for Mac <IP_ADDRESS> (46911) with Kubernetes enabled

Argo version:

argo: v2.9.4+20d2ace.dirty
  BuildDate: 2020-07-25T08:30:49Z
  GitCommit: 20d2ace3d5344db68ce1bc2a250bbb1ba9862613
  GitTreeState: dirty
  GitTag: v2.9.4
  GoVersion: go1.14.5
  Compiler: gc
  Platform: darwin/amd64

Kubernetes version :

clientVersion:
  buildDate: "2019-12-13T11:51:44Z"
  compiler: gc
  gitCommit: 70132b0f130acc0bed193d9ba59dd186f0e634cf
  gitTreeState: clean
  gitVersion: v1.17.0
  goVersion: go1.13.4
  major: "1"
  minor: "17"
  platform: darwin/amd64
serverVersion:
  buildDate: "2020-01-15T08:18:29Z"
  compiler: gc
  gitCommit: e7f962ba86f4ce7033828210ca3556393c377bcc
  gitTreeState: clean
  gitVersion: v1.16.6-beta.0
  goVersion: go1.13.5
  major: "1"
  minor: 16+
  platform: linux/amd64

Logs

server logs:

surprisingly, the server doesn't log the stacktrace seen in the UI.
time="2020-08-04T15:49:34Z" level=info authModes="[server client]" baseHRef=/ managedNamespace=argo namespace=argo secure=false
time="2020-08-04T15:49:34Z" level=warning msg="You are running in insecure mode. Learn how to enable transport layer security: https://github.com/argoproj/argo/blob/master/docs/tls.md"
time="2020-08-04T15:49:34Z" level=info msg="config map" name=workflow-controller-configmap
time="2020-08-04T15:49:34Z" level=info msg="SSO disabled"
time="2020-08-04T15:49:34Z" level=info msg="Starting Argo Server" version=v2.9.3
time="2020-08-04T15:49:34Z" level=info msg="Argo Server started successfully on http://localhost:2746"

workflow-controller logs:

but the workflow-controller does (partly at least)
...
E0804 15:50:42.669594       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20191225075139-73fd2ddc9180/tools/cache/reflector.go:98: Failed to list *v1alpha1.WorkflowTemplate: v1alpha1.WorkflowTemplateList.Items: v1alpha1.WorkflowTemplates: v1alpha1.WorkflowTemplate.Spec: v1alpha1.WorkflowTemplateSpec.WorkflowSpec: Templates: []v1alpha1.Template: v1alpha1.Template.DAG: v1alpha1.DAGTemplate.Tasks: []v1alpha1.DAGTask: v1alpha1.DAGTask.Arguments: v1alpha1.Arguments.Parameters: []v1alpha1.Parameter: decode slice: expect [ or n, but found {, error found in #10 byte of ...|ameters":{"someParam|..., bigger context ...|es":[{"dag":{"tasks":[{"arguments":{"parameters":{"someParam":"Hello, world!"}},"name":"print","temp|...
...

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Looking into this

Was able to reprodcue

Seems like this error is propagated directly from the Kubernetes client when we make our List request here:
https://github.com/argoproj/argo/blob/dbb39368295cbc0ef886e78236338572c37607a1/server/workflowtemplate/workflow_template_server.go#L67
This might limit our ability to respond gracefully to this error, such as being able to display non-malformed Workflow templates or provide a more graceful explanation of the issue. Investigating more.

The UI fails to this error page. Think is a known issue, see #3454.

If the 500 error originates in the Kuberenetes API , then the 500 is probably not fixable. I. do think we should be more graceful in handling these errors. I.e. we shouldn't dump the user to  and unusable UI.

Haven't used the k8s go-client myself, so I don't know if the CRD yaml is generated from go classes or something, but is it possible to make the CRD spec stricter, so it isn't possible to submit a malformed workflow template in the first place? It seems to me this particular 500 arises because what the K8s API server accepts and what the go client accepts, by way of its json parser, doesn't quite match. To be honest, I wouldn't expect to be allowed to insert someParam: "Hello, world!" under parameters in the first place, if the spec doesn't say that's a valid key.

@SebastianGoeb in theory - yes -in practise - no.
We provide CRDs with validation, but we don't use them by default.
Why?

We need to be able to run multiple installations of Argo Workflows in a single cluster.
CRD validation cannot pick up on many common errors (e.g. missing template names, invalid parameters).

Essentially, if you need a robust system, you must lint your manifests.
The argo template lint commands allow you to do this.

If it isn't fixable by reconfiguration, maybe it's something that should be bumped up to the go-client itself? Because I would expect it to deserialize anything that the server accepts, i.e. anything that is allowed by the CRD's OpenAPI spec. Or, conversely, if it refuses to deserialize a CR, then I would expect the server to refuse it too.

Ah, I see. That's a shame. But maybe that could be built into our CI workflow.

Available for testing in v2.10.0-rc1.


metabase: 0.38.0 -> 0.41.4

Motivation for this change
Fixes CVE-2021-44228.
Note that I'm aware this is marked as agpl3 (not Only or Plus). The license in Metabase repository is vague about specific version of AGPL (albeit it's clear at this point point that it's AGPL3 because it's the only version of AGPL). This shouldn't block merging this update. I may try asking upstream this later on.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

nixpkgs-review happy for me, macos 10.15 & nixos x86_64. Passes nixos test.
In theory this could be fixed for stable by just switching out its internal dependency, but because we just redistribute the upstream jar that's not an option :(

I think it makes sense to update Metabase for all channels due to state migrations involved.
However, looking at changes between 0.38.0 and 0.41.4 there technically is a breaking change - Internet Explorer 11 is no longer supported by Metabase. This may be acceptable however - many applications already don't support IE11 already, so organizations were already pretty much forced to not use IE11.

Alternatively, it's possible to upgrade stable NixOS channels to https://github.com/metabase/metabase/releases/tag/v0.38.6 (in order to avoid breaking change due to IE11), but I think an update would make sense.

Oh I didn't know such a release existed. Why would we not upgrade stable to 0.38.6 ? This is exactly what such releases are created for.

We could, albeit it's worth noting that releases seem to have about a year of support, which means that Metabase 0.38 released Feb 17, 2021 would have its support end before NixOS 21.11 support ends.
In a similar scenario, GitLab 14 was backported to stable (#128964), despite having a lot of potentially breaking changes: https://about.gitlab.com/releases/2021/06/22/gitlab-14-0-released/#release-removals (and in fact, one large one - dependency on PostgreSQL version which before backport wasn't even available in NixOS 21.05).
In this case, the breaking changes are pretty much just IE11 incompatibility.

Releases don't explode once upstream deems them "unsupported", it simply means that upstream won't make backport releases for vulnerabilities found during the unsupported period. In that sense, the majority of software in nixpkgs has zero "supported" period as they don't do any stable releases. If a vulnerability were found during the unsupported period and we were unable to patch it ourselves, then we could consider bumping. As it is, in all likelihood it will be fine and we can leave users to have their stable software.
On top of this, I'd really hate for what the gitlab maintainers do (at the protestations of others) to become a template for other packages.

Fair enough. This should be fine to merge now.


Bruno-query : License file not included in package on npm

Hi Team,
I see licence file is not included in package Bruno-query available on npm. This is blocking me to use Bruno cli in my organization. The current version available on npm was last updated 2 years ago.
https://www.npmjs.com/package/@usebruno/query
Request you to please push latest package to npm including licence file.
Thanks

@sreelakshmi-bruno This should be fairly straightforward. Please have the LICENSE updated to MIT in bruno-query in the next npm version update.


Basic Sample generated vcxproj file appears to be null

Generated a project using the Basic Sample project type

Opened the resulting projects/VS2022/project_name.sln file in Visual Studio 2022

IDE displays an error that there was an issue opening the .vcxproj file

Double checked the actual file in a text editor and it just displays (null)

I love this project idea though!

@erikyuzwa Oh... thanks for reporting! Afaik it worked on my last test but I've been doing many many tweaks today for the release, maybe I broke something...
I'll review it asap but in any case testing and improvements are very welcome for the project! 😄

@erikyuzwa did you figure out why? I see you managed to generate the project in the other issue, I have the same problem now

not yet @kagenash1 - it only happens for the Basic Sample - the other project types generates the .vcxproj file properly

not yet @kagenash1 - it only happens for the Basic Sample - the other project types generates the .vcxproj file properly (for me)

oh that's an insight already

@kagenash1 You can check the function SetupProject() in the source code, everything is updated there and is very organized.

@kagenash1 - I figure it's gotta be around here - https://github.com/raysan5/raylib_project_creator/blob/main/src/raylib_project_creator.c#L1113 - something is null which is causing the project file to null out? I don't know if that helps, but good luck!

@erikyuzwa I think it's fixed, I'm releasing an update in a while...

confirmed fixed in https://github.com/raysan5/raylib_project_creator/releases/tag/1.1 👍 🔥


(PUP-7192) Implement state machine classes for SSL initialization

This pull request defines logic for building, composing, and running state machines. The intent is to use this for reimplementing how Puppet agents submit CSRs and fetch certificates.

CLA signed by all contributors.

Per @branan's request, this is an example of how one would use this state machine logic.
BOOTSTRAP_CA = Puppet::StateMachine.build("bootstrap ca") do |m|
  m.start_state(:start)

  # TODO
  #   * Lock and unlock SSL directory
  #   * Enable HTTP verification after CA cert has been downloaded
  #

  m.state :start,
    action: -> { Puppet::SSL::Host.ca_location = :remote },
    event: ->(_) { :read_ca_cert },
    transitions: {read_ca_cert: :read_ca_cert}

  m.state :read_ca_cert,
    action: -> { File.readable?(Puppet[:localcacert]) },
    event: ->(result) { result ? :ca_cert_present : :ca_cert_absent },
    transitions: {
      ca_cert_present: :read_ca_crl,
      ca_cert_absent: :fetch_ca_cert
    }

  m.state :fetch_ca_cert,
    action: -> do
      http = Puppet::Network::HttpPool.http_instance(Puppet[:ca_server], Puppet[:ca_port], true, false)
      resp = http.get('/puppet-ca/v1/certificate/ca?environment=production')
      if resp.code == '200'
        File.open(Puppet[:localcacert], 'w:ASCII') { |fh| fh.write(resp.body) }
      end
    end,
    event: ->(result) { result ? :ca_cert_fetched : :ca_cert_absent },
    transitions: {
      ca_cert_fetched: :read_ca_crl,
      ca_cert_absent: :ca_cert_unretrievable
    }

  m.state :read_ca_crl,
    action: -> { File.readable?(Puppet[:hostcrl]) },
    event: ->(result) { result ? :ca_crl_present : :ca_crl_absent },
    transitions: {
      ca_crl_present: :complete,
      ca_crl_absent: :fetch_ca_crl,
    }

  m.state :fetch_ca_crl,
    action: -> do
      http = Puppet::Network::HttpPool.http_instance(Puppet[:ca_server], Puppet[:ca_port], true, false)
      resp = http.get('/puppet-ca/v1/certificate_revocation_list/ca?environment=production')
      if resp.code == '200'
        File.open(Puppet[:hostcrl], 'w:ASCII') { |fh| fh.write(resp.body) }
      end
    end,
    event: ->(result) { result ? :ca_crl_fetched : :ca_crl_absent },
    transitions: {
      ca_crl_fetched: :complete,
      ca_crl_absent: :ca_crl_unretrievable
    }

  m.state :complete, type: :final
  m.state :error, type: :error
  m.state :ca_cert_unretrievable, type: :error
  m.state :ca_crl_unretrievable, type: :error
end

I can include another example for composing state machines later.

A general comment - I personally find the explicit required namespacing of machines during a composition a little unintuitive. The namespaces are a new set of metadata I have to keep track of that are otherwise not relevant to the ultimate outcome, being internal machinery of the composition.
Puppet::StateMachine::Machine#name is a required attribute for creating a state machine (though admittedly not required to be globally unique, we could do collision detection in composition) - would it be simpler to use the #name as an implicit namespace, or would that just muddy the waters?

@MosesMendoza @branan since I changed the diff that the namespacing conversation was associated with I can't comment there, so I'll continue the conversation here.
So yes, there is some weirdness with namespacing. Right now a simple name is just Symbol; namspacing that yields Array<Symbol>; namespacing that yields another Array<Symbol>. This makes simple state machines simple, is consistent when all states are namespaced, but gets odd otherwise.
One option is to always use arrays for state names. This makes things more consistent, with and without namespaces, but declaring states and transitions becomes more cumbersome.
m.state([:run_agent], # namespaces to [:ns, :run_agent]
  transitions: {
    event1: [:transition1], # namespaces to [:ns, :transition1]
    event2: [:transition2], # namespaces to [:ns, :transition2]
  }
)

The other option is to use strings, and use double colons or some other separator for namespacing.
m.state("run_agent", # namespaces to "ns::run_agent"
  transitions: {
    event1: "transition1", # namespaces to "ns::transition1"
    event2: "transition2", # namespaces to "ns::transition2"
  }
)

This unifies the object types in use, but I don't know if this is any clearer.
Anywho. I'm happy to change this, I don't have a lot of opinions. Namespacing is probably the least interesting/valuable part of the work I'm doing in terms of SSL, so I'm happy to implement whatever is clearest so that I can get to the really exciting things involving large prime numbers and elliptic curves.

Can you see any future benefit to splitting transition into its own class (ie such that a Puppet::StateMachine::State instance may have/reference one or more Puppet::StateMachine::Transition instances) ?

@adrienthebo this is pretty cool - nice work on this PR. I see us being able to leverage this quite successfully.

There are some cases where we could extract a Transition class, such as Henrik's interest in being able to add guards to transitions. However I don't foresee an immediate need for this, and if we need that functionality we should be able to add it later as the implementation of transitions (thus far) is internal only.

@adrienthebo thought more about the namespacing - I agree requiring arrays would be laborious and pendantic. And a quick glance suggests there's nothing that would prohibit a user from using either a symbol OR an array to name a "simple" non-namespaced state if one so desires? I'm 👍  on implementation as-is.

will merge pending CI going green again

@branan @MosesMendoza are we still blocked on CI for this? It looks like integration tests are passing but internationalization tests are failing; is the latter a blocker for merging this?

Actually waiting to get a green run on the puppet-agent/master puppet tests.

puppet/master looks green. I'm merging this


nsolve conversion to Float64

Before anything else thanks @jverzani for this package I really like the effort.
I was trying to use nsolve yesterday and found that for different starting points, there three different errors thrown. Two of them seemed normal as they were SymPy errors of convergence and division by zero, but the other one was a Julia's InexactError. By looking at your code I found that you are doing conversions to Float64 (I imagine for type stability reasons).
The problem is that in some cases nsolve returns a Complex, and in many of those cases, the imaginary part is extremely small. But your code is trying to do the conversion anyway. This actually was impeding me to find a root in an interval.
I used the following code to workaround that
function nsolve(ex::Sym, x0::Number)
    α = sympy_meth(:nsolve, project(ex). x0)
    typeof(α) <: Complex && imag(α) < eps() && return convert(Float64, real(α))
    return convert(Float64, α)
end

I think there might be other alternatives. One of them is restructure SymPy.jl's nsolve so it you can define the return type to be Complex. Another one would be to look at the SymPy code, rewrite it in Julia in a way that is type stable, as far as I understand SymPy uses mpmath.findroot to find roots. But for now I'm using the code above.
I do not have to much time to do those changes by my own, but I sure can help.

Thanks for pointing this out! That function was written before I defined N to do basically what you are doing here. So the return value is now (with bac241a) just a symbolic number or vector. Calling N on either will convert to a floating point or complex number.

I see. Thanks again for the package!


Add methods for operating on Acls

The current implementation only allows setting an Acl when a Znode is created, but does not also allow retrieving an Acl for inspection, or updating it after the fact.

Looks like this was resolved in #13!


Docs.MS Release Updates for azure-template

Update docs metadata and targeting for release of azure-template

Docs Build status updates of commit 5b34b56:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 5b34b56:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/preview/template-readme.md
:warning:Warning
View (azure-node-preview || azure-node-legacy || azure-node-latest)
Details

ci-configs/packages-preview.json
:white_check_mark:Succeeded

docs-ref-services/preview/template-readme.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Color topdown map

Hello,
I have a question about top down map. We can receive a black and white top down map using function "get_topdown_view".
Can I get color topdown map like this? And how?

Thank you!

Thank you~

Thank you @aclegg3
But when I edit the hm3d .glb file with Blender, there are only meshes. Textures are not shown in the Blender (the mode of Blender is changed to texture mode, and I have test other .glb files, and the textures were displayed well.) . Also I tried 3dmax, micrsoft 3D viewer, the .glb files could not be opened by these softwares. Are the .glb formats  in the HM3d datasets specialized or different with normal .glb files or the version of the software should be a some special one?

@jingjing00kk, there are multiple downloads for HM3D. The habitat download is basis compressed for more efficient use in simulation. Blender won't load those. The other formats should contain uncompressed textures for the same scenes which can be loaded in Blender.


Fix language toggle in docs

Summary
The language toggle was broken for two reasons:

Sphinx 6 does not activate jquery by default anymore, so the interactivity was broken
The versionutils.py file should be setting language_label, not version_label

This PR also makes some other docs config improvements:

Pin the sphinx theme version for more stability
Move more docs dependencies from tox.ini to requirements-dev.txt to follow the convention of other Qiskit projects.
Remove unnecessary pip install lines in tox.ini.
Don't install sphinx-rtd-theme, which is superseded by qiskit-sphinx-theme
Remove override of the versions.html template. We should use qiskit-sphinx-theme instead.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.336%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3453

Relevant Lines:
5053

💛  - Coveralls


Fix deprecated node-js actions breaking release pipeline

Our release pipeline is breaking, due to some some sort of deprecation in supported github actions: https://github.com/informalsystems/apalache/actions/runs/3261446509

it does not seem to be breaking after 1.5 years, probably, fixed


Generalized Sphinx code to display inputs and outputs across Examples.

For issue #1851

This doesn't include **kwargs inputs, but it does include a dict for the return values. This was necessary for it to work with both HelloWorld and SimpleGaussian. It can still use a little tweaking, but it "works" and should be somewhat extensible.

On the *args, **kwargs change, this will break any existing code, including any tests, that depend on the current input method. Not sure how much that matters, but I'll need to figure out how to update the tests.

The changes needed for args kwargs can be done in a separate PR.


[fix] undefined sanitize_to_id

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.

Coverage increased (+17.5%) to 76.991% when pulling eb20ceb3383877a9e0e39b51839544d837e27479 on fix/captcha into 6fbc1250d8eb565c46ce0ff3fa2254940a1a7929 on master.


Update to the latest kubevirtci which deploys the sr-iov webhook

What this PR does / why we need it:
This pr updates to the latest kubevirtci version which deploys the sr-iov mutating webhook for adding resource requests to the first container of the launcher pod. This allows us to test our compatibility with it (and on the long run remove our custom code when it will be shipped as part of the CNAO)
Release note:

NONE

Close in favour of https://github.com/kubevirt/kubevirt/pull/2836


Modify: WestWind.MarkdownMonster version <IP_ADDRESS>

Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

#161623 (comment)

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


isLightDescendant should return false for self

Fixes https://github.com/Polymer/polymer/issues/2610

LGTM, thx :+1: :100:


clicking bin in control panel for first item causes crash

Steps to reproduce the problem:
Please fill the below with the necessary steps to reproduce the problem
open http://v2-dev.virtualflybrain.org/org.geppetto.frontend/geppetto?id=VFB_00017894&i=VFB_00017894,VFB_00030901,VFB_00030624,VFB_00030623,VFB_00030840,VFB_00030616,VFB_00030633 then try to remove an item in the control pannel.
Error Information:
Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.
Invariant Violation: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.
at eval (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:23115:28)
at checkForNestedUpdates (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:23118:7)
at scheduleUpdateOnFiber (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:21389:3)
at Object.enqueueSetState (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:13100:5)
at Object.Component.setState (webpack:///./node_modules/react/cjs/react.development.js?:325:16)
at Object.handleInstances (webpack:///./node_modules/@geppettoengine/geppetto-client/js/components/widgets/stackViewer/StackViewerComponent.js?:1257:16)
at Object.componentWillReceiveProps (webpack:///./node_modules/@geppettoengine/geppetto-client/js/components/widgets/stackViewer/StackViewerComponent.js?:1174:16)
at callComponentWillReceiveProps (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:13388:14)
at updateClassInstance (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:13598:7)
at updateClassComponent (webpack:///./node_modules/react-dom/cjs/react-dom.development.js?:16992:20)
  Adding 8 instances to 423
  Passing 424 instances
  Adding 8 instances to 424
  Passing 425 instances
  Adding 8 instances to 425
  Passing 426 instances
  Adding 8 instances to 426
  Passing 427 instances
- The above error occurred in one of your React components:
    in Unknown (created by StackViewer)
    in StackViewer (created by Tab)
    in div (created by Tab)
    in div (created by Tab)
    in Tab (created by Layout)
    in div (created by Layout)
    in Layout (created by VFBMain)
    in div (created by VFBMain)
    in VFBMain (created by Route)
    in Route
    in Switch
    in Router (created by BrowserRouter)
    in BrowserRouter
    in ErrorCatcher (created by WithStyles(ErrorCatcher))
    in WithStyles(ErrorCatcher)

React will try to recreate this component tree from scratch using the error boundary you provided, ErrorCatcher.
  USER: <IP_ADDRESS>  Penryn, CA, United States

waiting for https://github.com/openworm/geppetto-client/pull/72 to be merged but it's good to be closed.


worked on cart checkout bug

Fixed product price update (increases by 100 every update) when checkout
Fixed checkout bug with confirmation page.


Refactor numerical/non numerical column

Is your feature request related to a problem? Please describe.

I noticed an inconsistency in the current codebase (mostly due to the uns entry of (non) numerical columns got introduced later on). Sometimes, there is custom code checking for numerical columns in different ways, which is now obviously obsolete.
Describe the solution you would like

Refactor code to use uns entries.

Seems like thats already fixed.


docs(npmignore): trying to fix npmignore to get README.md to publish

Codecov Report

Merging #47 into master will not impact coverage.

@@          Coverage Diff          @@
##           master    #47   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         290    290
=====================================
  Hits          290    290

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 5cb2af9...9aa2f55. Read the comment docs.


add collatz conjecture exercise

Add the collatz conjecture exercise in #803

This looks good, thanks for making those changes. :heart:

I like this new exercise and I am reminded of the version that I solved here a couple of years ago.  It would be nice to see a test that addresses the unproven aspect of the Collatz Conjecture that the Project Euler exercise touches on.  Just a suggestion, this exercise is fine otherwise.  If there is any interest in this I can provide the highest number, under 1 million, that produces the longest chain; I'd have to review my solution to determine how long is the longest chain.

Thanks for the exercise. Everyone feel free to add on more aspects.

+1 this is a nice exercise! I think adding a test for a much higher input (the stated example of the longest chain starting at a number below 1 million would be a great candidate) would be a nice way to cap the suite and really run an implementation through its paces.


Jump in version of Node.js required to run

Describe the bug
This plugin requires Node 16 after 1.32.9, but it wasn't a major or even minor version number increase.  Seems risky to change the runtime environment with much of an indication in the change logs or release notes.  Also prevents deploying to Node14 runtimes where you want to build your dependencies using the same version of Node.
To Reproduce

Using Node 14.
Run sls package.

Expected behavior

It to compile.

Versions (please complete the following information):

OS: Mac
Serverless Framework Version: 3.23.0
Plugin Version: v1.32.9

We got bitten by this too.
Node 14.x is kind of old now but we'd been holding back due to encountering other issues with serverless-offline. This kind of forces the issue but a heads-up would have been cool.


Reflect migration details on both the source and target pods

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3472
In this PR, we have exposed more detailed migration information in kubevirt and exposed this information on annotations of the source and destination pods.
Special notes for your reviewer:
Release note:

In my opinion, we can create a new branch for this pr. This will make changes more readable.


phoebemirman/ch15454/when-orphaned-order-is-the-only-order-ui

fix for bug that if user's only open order is an orphaned one the ui is messed up. and if they have no positions or open orders, just the one orphaned one, the market doesn't show up in their portfolio.
https://app.clubhouse.io/augur/story/15454/when-orphaned-order-is-the-only-order-ui-isn-t-right

Coverage decreased (-0.04%) to 66.625% when pulling 256abd5cc02d8193d54b167a5677ed5830b3f59e on phoebemirman/ch15454/when-orphaned-order-is-the-only-order-ui into d9cbce6b516675990fbc1273d2ead0a0f41272db on master.

Coverage decreased (-0.04%) to 66.625% when pulling 256abd5cc02d8193d54b167a5677ed5830b3f59e on phoebemirman/ch15454/when-orphaned-order-is-the-only-order-ui into d9cbce6b516675990fbc1273d2ead0a0f41272db on master.


Disabled select text does not appear lighter

I add the disabled property programatically : the select is disabled, but the weight of the text does not change.

Please provide a JSBin, JSFiddle or similar isolated test case showing the problem.

I have created a fiddle and  an example on the web.  The code is the same; however, the appearance is very different.  In jsfiddle () things behave as you would expect.  In the online example, I see no difference between the disabled/enabled select.  Hope this is helpful.

Hi @lenberman!
You appear to have posted a live example (https://fiddle.jshell.net/zdkdvjvz/2/show/light/), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 31, column 7 thru column 54: No space between attributes.
line 31, column 7 thru column 64: Quote " in attribute name. Probable cause: Matching quote missing somewhere earlier.
line 31, column 7 thru column 258: Bad value <link rel= for attribute src on element script: Illegal character in path segment: < is not allowed.
line 31, column 7 thru column 258: Attribute stylesheet" not allowed on element script at this point.
line 31, column 7 thru column 258: Attribute href not allowed on element script at this point.
line 31, column 261 thru column 269: The text content of element script was not in the required format: Expected space, tab, newline, or slash but found " instead.
line 35, column 7 thru column 60: No space between attributes.
line 35, column 7 thru column 67: A slash was not immediately followed by >.
line 35, column 7 thru column 68: A slash was not immediately followed by >.
line 35, column 7 thru column 89: A slash was not immediately followed by >.
line 35, column 7 thru column 94: A slash was not immediately followed by >.
line 35, column 7 thru column 99: A slash was not immediately followed by >.
line 35, column 7 thru column 106: A slash was not immediately followed by >.
line 35, column 7 thru column 112: A slash was not immediately followed by >.
line 35, column 7 thru column 115: A slash was not immediately followed by >.
line 35, column 7 thru column 128: Quote " in attribute name. Probable cause: Matching quote missing somewhere earlier.
line 35, column 7 thru column 129: Bad value     <script src= for attribute src on element script: Illegal character in path segment: < is not allowed.
line 35, column 7 thru column 129: Attribute https: not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute cdnjs.cloudflare.com not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute ajax not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute libs not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute tether not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute 1.1.1 not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute js not allowed on element script at this point.
line 35, column 7 thru column 129: Attribute tether.min.js" not allowed on element script at this point.
line 35, column 141 thru column 149: Stray end tag script.
line 39, column 7 thru column 60: No space between attributes.
line 39, column 7 thru column 67: A slash was not immediately followed by >.
line 39, column 7 thru column 68: A slash was not immediately followed by >.
line 39, column 7 thru column 92: A slash was not immediately followed by >.
line 39, column 7 thru column 102: A slash was not immediately followed by >.
line 39, column 7 thru column 116: A slash was not immediately followed by >.
line 39, column 7 thru column 119: A slash was not immediately followed by >.
line 39, column 7 thru column 135: Quote " in attribute name. Probable cause: Matching quote missing somewhere earlier.
line 39, column 7 thru column 244: Bad value     <script src= for attribute src on element script: Illegal character in path segment: < is not allowed.
line 39, column 7 thru column 244: Attribute https: not allowed on element script at this point.
line 39, column 7 thru column 244: Attribute maxcdn.bootstrapcdn.com not allowed on element script at this point.
line 39, column 7 thru column 244: Attribute bootstrap not allowed on element script at this point.
line 39, column 7 thru column 244: Attribute 4.0.0-alpha.2 not allowed on element script at this point.
line 39, column 7 thru column 244: Attribute js not allowed on element script at this point.
line 39, column 7 thru column 244: Attribute bootstrap.min.js" not allowed on element script at this point.
line 39, column 256 thru column 264: Stray end tag script.
line 43, column 3 thru column 25: Element style not allowed as child of element body in this context. (Suppressing further errors from this subtree.)
line 47, column 3 thru column 9: Element title not allowed as child of element body in this context. (Suppressing further errors from this subtree.)
line 74, column 1 thru column 7: Stray end tag head.
line 76, column 1 thru column 6: Start tag body seen but an element of the same type was already open.
line 77, column 3 thru column 8: Start tag body seen but an element of the same type was already open.
line 82, column 5 thru column 13: Element formset not allowed as child of element form in this context. (Suppressing further errors from this subtree.)
line 95, column 1 thru column 7: Saw an end tag after body had been closed.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Deleted the redundant <body>: https://jsfiddle.net/cvrebert/zdkdvjvz/4/

Hi @cvrebert!
You appear to have posted a live example (https://fiddle.jshell.net/cvrebert/zdkdvjvz/4/show/light/), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 80, column 5 thru column 13: Element formset not allowed as child of element form in this context. (Suppressing further errors from this subtree.)

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Assuming <fieldset> was intended: https://jsfiddle.net/cvrebert/zdkdvjvz/5/

Hi @cvrebert!
You appear to have posted a live example (https://fiddle.jshell.net/cvrebert/zdkdvjvz/5/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 77, column 5: W007: Found one or more <button>s missing a type attribute.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Revised: https://jsfiddle.net/cvrebert/zdkdvjvz/6/

There seems to be another issue with the height of select when trying to use control sizing...
http://codeply.com/go/fy1CWmz03e

This was addressed in #19735.


Add js event to represent different state of scheduler

onRender
onRendered
onEventRender
onEventRendered

Pro feature! 👍
Really like real plugin 😮

lol, just noobing around 😮

I only is noobing around 😞


Update Xcode schemes for the package

These are used when we open the package folder in Xcode; they contain configuration settings (such as enabling code coverage collection) that is missing from Package.swift.
This update makes these stop referring to nonexistent targets and commit some changes that Xcode adds on its own every time we open the project.
Checklist

[X] I've read the Contribution Guidelines
[X] My contributions are licensed under the Swift license.
[ ] I've followed the coding style of the rest of the project.
[ ] I've added tests covering all new code paths my change adds to the project (if appropriate).
[ ] I've added benchmarks covering new functionality (if appropriate).
[ ] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[ ] I've updated the documentation if necessary.

@swift-ci test

True


feat(chat-models): Tool calling feature for Ollama (#288)

Resolves #288

Could you take a look at my changes?
Pending things:

Port the documentation
Streaming support (we can leave it outside the scope of this PR)

Refactored changes for formatting prompt logic with tool choice option , parsing result, handling multiple tool result and updated folder structure looks good.

I've added the documentation. It's good to go 🚀


Fix padding between line # & code in native editor

For #9201
Restores changes lost as part of redux refactor.
Old code here https://github.com/microsoft/vscode-python/issues/9201#issuecomment-567249280

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9202   +/-   ##
=======================================
  Coverage   57.45%   57.45%
=======================================
  Files         551      551
  Lines       29388    29388
  Branches     4450     4450
=======================================
  Hits        16886    16886
  Misses      11555    11555
  Partials      947      947

Impacted Files
Coverage Δ

...science-ui/native-editor/redux/reducers/effects.ts
15.94% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Image needs to have a fixed aspect ratio

This is being worked on in the GUI branch.

Solved by 451f635


pom.xml:472-475: This limits are set according to current...

The puzzle 1088-ac9afd75 from #1088 has to be resolved:
https://github.com/yegor256/takes/blob/72f09dd348a86f566fd627681e7d30489f83c148/pom.xml#L472-L475
The puzzle was created by @LightPro-Inc on 12-Mar-21.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1088-ac9afd75 has disappeared from the source code, that's why I closed this issue.

Job gh:yegor256/takes#1112 is not assigned, can't get performer


Can anyone help

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

To get help, please fill out the information requested

Closing this issue due to lack of update/activity - Feel free to re-open if appropriate.


Spike: Integrate Superset with Clickhouse storing xAPI events

With https://github.com/openedx/data-wg/issues/21, we created a Tutor plugin to deploy Superset integrated to the Open edX SSO and MySQL database, and worked out how to restrict access to data based on course roles.
Ultimately, the reference implementation of Open edX Analytics will use an LRS (e.g. Ralph) to store xAPI events in Clickhouse. So we need Superset to read xAPI events from Clickhouse.
This ticket involves these different repositories:

Tutor Superset plugin -- Apache Superset, with customisations for Open edX
Tutor Clickhouse plugin -- installs Clickhouse
OARS -- glue that ties all of the analytics plugins together.

Acceptance criteria

Plugin(s) add a Clickhouse Data Source to Superset,  ref https://superset.apache.org/docs/databases/clickhouse/
Configurable option to pre-populate Clickhouse with a large xAPI data set, containing Open edX events, ref https://github.com/bmtcril/xapi-db-load
Plugin(s) add query templates to Superset to enforce course role permissions when reading xAPI events from Clickhouse.
Plugin(s) add a Chart to Superset that creates a simple demonstration of the above xAPI events and course access permissions (can be basic).
Update OARS plugin README / docs to point to all of these plugins.

Looks great, I've moved it to the OARS board and set it ready for work so if you get a chance to pick it up you're good to go.

assign me

@bmtcril Github wouldn't assign me :( Maybe because I created this ticket?

Could be that the action isn't installed on this repo. I'll round up and try to figure it out, but have assigned it to you in the meantime. Thanks for picking it up!

Ideas about what goes where:

Superset plugin needs to hold the python dependencies and custom code, because it needs to be loaded when Superset runs.
Clickhouse plugin should just load Clickhouse into the Tutor environment, with whatever minimal customizations are needed to access it from within Tutor.
OARS plugin should store any Open edX-specific scripts that invoke the Superset API, or data that can be loaded into Clickhouse.
All dev data should not be loaded by default; enable a Tutor env variable in order to trigger loading.

@bmtcril You mentioned that you'd like the OARSy stuff that sits across multiple technologies (like superset and clickhouse) to live in an OARS repo.
Would you like me to turn this repo (openedx-oars) into a tutor plugin (by adding tutoroars/plugin.py and files like that), or should I create a new tutor-contrib-oars plugin repo?

Sorry I missed this one. I agree with you about what goes where, in terms of specifics I think we'll probably run into some fun snags along the way but I'd start with:
tutor-contrib-superset:

All you said
Configuration to talk to a shared Tutor mysql (where are creds created / stored?)
Configuration to talk to a shared Tutor cache (redis, right? here are creds created / stored?)
Presumably uses database info from the LMS settings?
Set up SSO into LMS w/custom code and config to support our default permissions scheme
Any additional customized code / templates that can't be installed via OARS?

tutor-contrib-clickhouse:

All you said
Permissions for admin user, an LRS user, and reporting user

tutor-contrib-ralph (new repo, doesn't exist yet):

Configuration needed to stand up the LRS backended on ClickHouse
Creates the "xapi" database and any generic table(s) for accepting xAPI events

tutor-contrib-oars (new repo, doesn't exist yet):

Creates any necessary downstream tables / views in ClickHouse to support reporting

It seems like Ralph is going to start with a very sparse table structure, so we may need to make at least one materialized view to get the data in a performant state

Creates any Superset datasources needed to talk to Tutor ClickHouse
Optionally bootstraps Superset with reports
Optionally bootstraps ClickHouse with test data via xapi-db-load with the ralph_clickhouse backend, providing end-to-end testing of the configuration

This currently lives in your tutor-contrib-clickhouse PR, which is good for now, but should probably move here once it exists

Updates LMS settings to point event-routing-backends to Tutor Ralph
Eventually updates LMS settings to point any deep links or API URLs to Superset

I think that's everything, only a couple of open questions about which systems own some credentials

@bmtcril Cool, that clarifies things for me.

tutor-contrib-superset:
...

Configuration to talk to a shared Tutor mysql (where are creds created / stored?)

Yep, Superset's MySQL settings default to the LMS's settings here: host, port, table name, username + password

Configuration to talk to a shared Tutor cache (redis, right? here are creds created / stored?)

Yep, Superset is using Open edX's redis, connection details passed in here (though looking at that, we might need a REDIS_USERNAME too?)

tutor-contrib-oars (new repo, doesn't exist yet):
...

Creates any Superset datasources needed to talk to Tutor ClickHouse, using the "reporting" user from tutor-contrib-clickhouse

Cool, yes, that makes sense.
Do you want to cookie-cutter a repo under openedx for this? I can start it under open-craft, but means we have to move it later.

Optionally bootstraps ClickHouse with test data via xapi-db-load ...
This currently lives in your tutor-contrib-clickhouse PR, which is good for now, but should probably move here once it exists

Ah ok yes, that's what I was thinking too.
We're on the same page!

About the materialized view.
You can get inspired about what we are doing on NAU.
We have a couple of materialized views that we use for our analytics. The majority of the queries (except the joins w/ nau_openedx_extensions_nauuserextendedmodel table) are compatible with Open edX.

https://github.com/fccn/nau-database-exporter/blob/master/nau.py

@pomegranited I think we can call this one done and move on to the more specific issues in the repo. What do you think?


Please cut python 3.8 wheel for 0.3.3

the python version was constrained in #533 to >=3.8 but no 3.8 wheel was published for 0.3.3. Please publish a 3.8 wheel. I also took a quick glance at the CI configs and see that you're still building against 3.7. It might be time to update that as well

Close?

Yes


⚠ Mobitopup has degraded performance

In 4e6c394, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 16489 ms

Resolved: Mobitopup performance has improved in 3b5a6c3.


SpanId not copied to ParentSpanId (Istio service mesh + Feign calls)

Context:

Kubernetes cluster
Istio service mesh
Java microservices all using:

Spring Cloud Feign 1.4.4.RELEASE
Spring Boot 1.5.12.RELEASE
Sleuth 1.3.5.RELEASE

The trace IDs are correctly being propagated in subsequent Feign calls but for some reason, the parent span ID is always new and is not copied from the caller's span ID.
To use Sleuth, I just added the POM dependency and no other special configuration were done.
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-sleuth</artifactId>
			<version>${sleuth.version}</version>
		</dependency>

Can you please check if it works fine with a more recent version of Sleuth? The one you're using will soon be outdated and we have fixed a lot of issues in the newer versions

@marcingrzejszczak I was able to upgrade it until 1.3.6.RELEASE but it has the same issue. I am encountering errors when use 2.0.0 or higher probably because of my Spring Boot version. Is this an expected version incompatibility? Or is there a Sleuth version higher than 1.3.6 that should work with Spring Boot 1.5.12?

Please read about the compatibility matrix of Boot vs Cloud versions over here https://spring.io/projects/spring-cloud.

Release Train | Boot Version
-- | --
Greenwich | 2.1.x
Finchley | 2.0.x
Edgware | 1.5.x
Dalston | 1.5.x

If you need to check out Spring Cloud Sleuth versions you can for example visit the project website https://spring.io/projects/spring-cloud-sleuth#learn
You could go to maven central and search for a given artifact and check out its version.
Last but not least since you're in the Sleuth project you could just check out the releases https://github.com/spring-cloud/spring-cloud-sleuth/releases

@marcingrzejszczak Looks like we need to start much sooner our planned upgrade to Spring Boot 2. Thank you for your time!


Poof not able to rename a file somehow?

I did a demo today and poof was not able to rename temp files in an extremely basic project.  Sounds like a bug.

Turned out relative includes had a bug; the filenames included the quotes.  Not sure if it failed on Linux (I think it worked), but it definitely failed on Windows.


[GOBBLIN-1721] Give option to cancel helix workflow through Delete API

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-1721] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1721

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Currently when we receive a job restart(handleUpdateJobConfigArrival), GobblinHelixJobLauncher will firstly call  helixTaskDriver.waitToStop to stop the workflow, then initiate the new one. We observe the behavior of Helix taking exceptionally long to stop the workflow, making the job state staying in STOPPING status. This will make waitToStop timeout and throw exception all the time, making the new flow never be able to launch.

We can utilize Delete API in this case since our job is stateless for Helix, to avoid job hanging.
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Tested in the cluster.

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3580 (d024c60) into master (fafd40b) will increase coverage by 1.78%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##             master    #3580      +/-   ##
============================================
+ Coverage     46.87%   48.66%   +1.78%
+ Complexity    10640     7851    -2789
============================================
  Files          2113     1467     -646
  Lines         82798    57816   -24982
  Branches       9220     6653    -2567
============================================
- Hits          38813    28134   -10679
+ Misses        40425    27063   -13362
+ Partials       3560     2619     -941

Impacted Files
Coverage Δ

...bblin/cluster/GobblinClusterConfigurationKeys.java
0.00% <ø> (ø)

...ache/gobblin/cluster/GobblinHelixJobScheduler.java
53.67% <40.00%> (-0.93%)
:arrow_down:

...a/org/apache/gobblin/util/limiter/NoopLimiter.java
40.00% <0.00%> (-20.00%)
:arrow_down:

...lin/util/filesystem/FileSystemInstrumentation.java
85.71% <0.00%> (-7.15%)
:arrow_down:

.../java/org/apache/gobblin/cluster/SleepingTask.java
39.39% <0.00%> (-6.07%)
:arrow_down:

...a/management/copy/publisher/CopyDataPublisher.java
74.17% <0.00%> (-1.33%)
:arrow_down:

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
63.63% <0.00%> (-0.32%)
:arrow_down:

...anagement/copy/replication/ConfigBasedDataset.java
68.87% <0.00%> (ø)

...gobblin/service/monitoring/FsFlowGraphMonitor.java

.../apache/gobblin/http/ApacheHttpResponseStatus.java

... and 647 more

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


bug: Android canvas vertical scrolling issue

Bug Report
Ionic version:

[x] 5.1
Current behavior:
I am developing an angular/ionic5 app to work with map data. I am using ArcGIS JS API. Overall structure of my home page looks like that
<ion-header>
    <app-header></app-header>
</ion-header>
<ion-content>
    <app-map></app-map>
</ion-content>

where app-map is a component with div which is populated by ArcGIS JS API, and is structured as follows:

.map {
    height: 100%;
    width: 100%;
    margin: 0;
    padding: 0;
}

the resulting generated map component DOM looks like that:

         <canvas style="width: 411px; height: 613px; display: block;" width="1079" height="1609"></canvas>

The issue i'm facing is that swiping in vertical direction on Android doesn't move the map... On IOS it works perfectly. On Android moving map is only possible by dragging it but only in horizontal direction. Is it possible to move the map similarly on android as on IOS.
IOS:

Android:

Please help, I spent almost 3 days on the issue, and couldn't find any solution...
Thank you in advance.
Expected behavior:
Vertical moving of map on Android works similarly as on IOS
Steps to reproduce:
Described above
Related code:
Provided above

i solved it, it looks like i had to additionally load some CSS from ArcGIS JS API. Sorry for bothering...


[Bug][Manager] The error message is not right when deleting a successful inlong group

What happened
It prompts "Current status=config_failed was not allowed to delete" when deleting a successful inlong group, and this group's status changed to "Configuration failed".

What you expected to happen
No error throws.
How to reproduce
Just delete one inlong group whose status was config successful.
Environment
No response
InLong version
master
InLong Component
InLong Manager
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

On the way.


implement live update, remove convert button

Update output when selection is changed, or
When input is changed
convert button not required

resolves #2

It's a better UX. Thanks @sayandipdutta


feat(instance): add getServerUserData method

Codecov Report

Merging #405 (10877d5) into main (01b33a7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #405   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           44        44
  Lines          636       636
  Branches       129       129
=========================================
  Hits           636       636

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


slurm: <IP_ADDRESS> -> <IP_ADDRESS>, pyslurm: 20180908 -> 18.08.3

Motivation for this change
Update and slurm and and pyslurm 20180908 -> 18.08.3
CC @veprbl
Things done

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

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

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

Success on x86_64-linux (full log)
Attempted: slurm
Partial log (click to expand)
/nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>

@GrahamcOfBorg test slurm

@GrahamcOfBorg build python36Packages.pyslurm

Success on x86_64-linux (full log)
Attempted: python36Packages.pyslurm
Partial log (click to expand)
Successfully installed pyslurm-<IP_ADDRESS>
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3
shrinking /nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3/lib/python3.6/site-packages/pyslurm/pyslurm.cpython-36m-x86_64-linux-gnu.so
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3/lib
patching script interpreter paths in /nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3
checking for references to /build in /nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3...
/nix/store/a7i307xc98czxkw5dc5g2f54z4d7z0ps-python3.6-pyslurm-18-08-3

Success on aarch64-linux (full log)
Attempted: slurm
Partial log (click to expand)
gzipping man pages under /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/lib  /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/bin  /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/sbin
patching script interpreter paths in /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>
checking for references to /build in /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>...
moving /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/sbin/* to /nix/store/w4r261rrgz7wn9b39ajzxww0rvvc4gzx-slurm-<IP_ADDRESS>/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/d3phdx60nw0xvmpkgmi7a7s2ah3zklpk-slurm-<IP_ADDRESS>-dev
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/d3phdx60nw0xvmpkgmi7a7s2ah3zklpk-slurm-<IP_ADDRESS>-dev
checking for references to /build in /nix/store/d3phdx60nw0xvmpkgmi7a7s2ah3zklpk-slurm-<IP_ADDRESS>-dev...

Success on aarch64-linux (full log)
Attempted: python36Packages.pyslurm
Partial log (click to expand)
Successfully installed pyslurm-<IP_ADDRESS>
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3
shrinking /nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3/lib/python3.6/site-packages/pyslurm/pyslurm.cpython-36m-aarch64-linux-gnu.so
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3/lib
patching script interpreter paths in /nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3
checking for references to /build in /nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3...
/nix/store/bpysl05rzg2mz55swngcrd7s9b9vcwzi-python3.6-pyslurm-18-08-3

Success on aarch64-linux (full log)
Attempted: tests.slurm
Partial log (click to expand)
test script finished in 198.51s
cleaning up
killing control (pid 631)
killing node3 (pid 643)
killing node1 (pid 657)
killing submit (pid 670)
killing node2 (pid 683)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/456939yk19xwvpwiywm9yhz44bc9n849-vm-test-run-slurm


View an artists profile

An artists profile will show there top songs + a list of all there albums, official tracks and user uploaded tracks?

Current designs for artist pages made in penpot

Completed in v0.4.0


Object specifies the type of printer (Software/Hardware) | Get-Printer cmdlet

Summary of the new feature / enhancement
Microsoft Windows operating system comes with preinstalled virtual printers such as Microsoft Print to PDF, Microsoft XPS Document Writer, Fax, etc.
Sometimes it is required to select only the hardware printers (Cannon, HP, etc.).
The Get-Printer cmdlet does not have an object that differentiates between hardware and software printers.
I'm suggesting to introduce a new object to specify the type of printer.
Proposed technical implementation details (optional)
No response

The cmdlet is not in the repository. Please use Windows Feedback tool or Microsoft forums to share your suggestion.