Update Release Notes for 1.9.0

I know that there is already version 1.10.0 out but nevertheless I would like tp propose a change to the release notes for 1.9.0. In the Bugfixes is the following mentioned:

A warning in logs saying APM server is not available when using 1.8 with APM server 6.x (no functionality impact)

The part "no functionality impact" is not correct: I found that the 1.8.0  agent silently skips numeric labels despite having the APM Server 6.8.0 which supports them. After switching to 1.9.0 the numeric labels work as described. I don't konw if there are other parts which do not work so I would like to propose to at least remove the hint about not impacting functionality.

Ahh, good point. Probably if agent 1.8.0 fails to parse the APM server version (ie <7.0), it defaults to assume that non-string labels are not supported, which would be true up to v6.7.0 but not thereafter.
Will remove that.
Thanks!

@wolframhaussig please take a look now, and either suggest a different text or close this issue.
Thanks for reporting!


Call to a member function getMethods() on null

Versions:

ide-helper Version: dev-master (2.8.1)
Laravel Version: 8.9.0
PHP Version: 7.4.8

Description:
I recently try to run the command below and I got this error:
$ artisan ide-helper:generate

   Error

  Call to a member function getMethods() on null

  at vendor/barryvdh/laravel-ide-helper/src/Macro.php:64
     60▕     {
     61▕         $enclosingClass = $this->method->getClosureScopeClass();
     62▕
     63▕         /** @var \ReflectionMethod $enclosingMethod */
  ➜  64▕         $enclosingMethod = Collection::make($enclosingClass->getMethods())
     65▕             ->first(function (\ReflectionMethod $method) {
     66▕                 return $method->getStartLine() <= $this->method->getStartLine()
     67▕                     && $method->getEndLine() >= $this->method->getEndLine();
     68▕             });

      +32 vendor frames
  33  artisan:37
      Illuminate\Foundation\Console\Kernel::handle()

Steps To Reproduce:

artisan ide-helper:generate

Can you debug which method this is about?
Also maybe the code is simply buggy as getClosureScopeClass can return null, but nevertheless would be good to understand what specifically triggers this.

This seems to be caused by invokable classes. This how to reproduce this issue:
Versions:

laravel/framework 8.18.1
barryvdh/laravel-ide-helper 2.8.2
PHP 7.4.10

Step 1.
Create class App\Foo:
<?php

namespace App;

class Foo
{
    public function __invoke($var)
    {
        return $var;
    }
}

Step 2.
Register the macro by passing a new instance of App\Foo:
<?php

namespace App\Providers;

use App\Foo;
// ...

class AppServiceProvider extends ServiceProvider
{
    public function register()
    {
        UrlGenerator::macro('foo', new Foo());
    }

    // ...
}

Step 3.
Run php artisan ide-helper:generate.
Result
   Error

  Call to a member function getMethods() on null

  at vendor/barryvdh/laravel-ide-helper/src/Macro.php:64
     60▕     {
     61▕         $enclosingClass = $this->method->getClosureScopeClass();
     62▕
     63▕         /** @var \ReflectionMethod $enclosingMethod */
  ➜  64▕         $enclosingMethod = Collection::make($enclosingClass->getMethods())
     65▕             ->first(function (\ReflectionMethod $method) {
     66▕                 return $method->getStartLine() <= $this->method->getStartLine()
     67▕                     && $method->getEndLine() >= $this->method->getEndLine();
     68▕             });

      +30 vendor frames
  31  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

Logging the value of $this->method at src/Macro.php:62 results in the following log entry:
[2020-12-10 10:26:38] local.DEBUG: Closure [ <user> public method Illuminate\Foundation\Providers\{closure} ] {
  @@ /Users/standaniels/Desktop/temp/vendor/laravel/framework/src/Illuminate/Foundation/Providers/FoundationServiceProvider.php 83 - 85
}

[2020-12-10 10:26:38] local.DEBUG: Method [ <user> public method __invoke ] {
  @@ /Users/standaniels/Desktop/temp/app/Foo.php 7 - 10

  - Parameters [1] {
    Parameter #0 [ <required> $string ]
  }
}

I don't use these kind of things 🤷♀
Can you propose a PR / test to fix this?

Fixed with https://github.com/barryvdh/laravel-ide-helper/pull/1124

Fixed with https://github.com/barryvdh/laravel-ide-helper/pull/1124

Released via https://github.com/barryvdh/laravel-ide-helper/releases/tag/v2.9.1

@mfn the PR seems to have broken some of my classes. ReflectionMethod::getClosureScopeClass() returns null for non closures. See example below:

I'm proposing this change as a fix. I'll submit a PR if you're happy with the change

But this change then will not support __invokeable classes?
Basically: yes, please make a PR but from a quick look just replacing it with isClosure might not be correct

@mfn thanks. I'll have another look at my proposed change and make changes if necessary and create the PR. Thanks.

I still have
Call to a member function getMethods() on null
at vendor/barryvdh/laravel-ide-helper/src/Macro.php
/** @var \ReflectionMethod $enclosingMethod */
        $enclosingMethod = Collection::make($enclosingClass->getMethods())
            ->first(function (\ReflectionMethod $method) {
                return $method->getStartLine() <= $this->method->getStartLine()
                    && $method->getEndLine() >= $this->method->getEndLine();
            });

in some cases $enclosingClass is null
I have 2.10.0 version installed

I also still have this issue. I enabled debugging and I can see that the problem stems from some other package I installed that adds a macro on the Request.
Request::macro(...)
This causes a problem in addLocationToPhpDoc()

I propose a quick, probably naive, fix here: https://github.com/barryvdh/laravel-ide-helper/pull/1239


⚠ SceneTime has degraded performance

In c74602e, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3580 ms

Resolved: SceneTime performance has improved in 8b1d99d after 6 minutes.


Unable to load VGG model's state dict on GPU

I am trying a generic code to load the state-dict of a model on CPU/GPU. It works fine for other vision type models but fails for VGG models on GPU machines
Here is a sample code I am trying out :
import torch
map_location = 'cuda' if torch.cuda.is_available() else 'cpu'
model_pt_path = 'vgg13-c768596a.pth'
state_dict = torch.load(model_pt_path, map_location=map_location)

The above code fails with following error
Traceback (most recent call last):
  File "test_vgg.py", line 4, in <module>
    state_dict = torch.load(model_pt_path, map_location=map_location)
  File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 593, in load
    return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
  File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 773, in _legacy_load
    result = unpickler.load()
  File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/torch/tensor.py", line 147, in __setstate__
    self.set_(*state)
RuntimeError: Expected object of device type cpu but got device type cuda for argument #2 'source'

I tried this same code with Densenet/Alexnet/Squeezenet models and it works fine on both GPU and CPU machines
Environment information :
OS : Ubuntu 18.0.4
PyTorch Version  : 1.5.1
TorchVision Version : 0.6.1

To narrow this down a little: the error only happens for the vgg11 and vgg13 weights.

This relates to https://github.com/pytorch/vision/issues/2068 although the error message is different, but the root issue is the same (very old serialization format).
The workaround on your side for now could be to avoid using the map_location for CUDA, and convert manually the tensors to GPU if needed (or maybe even just pass the weights on CPU to load_state_dict while the model is on CUDA, I think this will also work).

This now fails with
Traceback (most recent call last):
  File "/fsx/users/nicolashug/work/vision/lol.py", line 21, in <module>
    state_dict = torch.load(model_pt_path, map_location=map_location)
  File "/fsx/users/nicolashug/conda/envs/pt/lib/python3.9/site-packages/torch/serialization.py", line 607, in load
    return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
  File "/fsx/users/nicolashug/conda/envs/pt/lib/python3.9/site-packages/torch/serialization.py", line 786, in _legacy_load
    result = unpickler.load()
  File "/fsx/users/nicolashug/conda/envs/pt/lib/python3.9/site-packages/torch/_tensor.py", line 189, in __setstate__
    self.set_(*state)
RuntimeError: Attempted to set the storage of a tensor on device "cpu" to a storage on different device "cuda:0".  This is no longer allowed; the devices must match.

which is similar to https://github.com/pytorch/vision/issues/3767
I'll try to update the weights of VGG as done for the other models in https://github.com/pytorch/vision/pull/3620, since it seems to have solved the other issue https://github.com/pytorch/vision/issues/3767#issuecomment-831841164

BTW I can only reproduce the issue for inception_v3_google-9aff3935.pth,  vgg11-7de3a66a.pth,  vgg13-b20e5c87.pth
The rest of the VGG weights seem fine.


install idle state handler for query server

While doing performance testing in perf02, I noticed Repose keeps a lot of CLOSE_WAIT sockets to Blueflood on the Query nodes. This eventually generated 500s errors to the clients.
We encountered this same issue on Ingest nodes. We fixed it so that Blueflood didn't close the sockets connection, if HTTP_CONNECTION_READ_IDLE_TIME_SECONDS configuration property is set to 0. We set this property to zero on all of our environments, since Repose and Blueflood are running on the same box, we reduce IO churn by keeping connections open between Repose and Blueflood.
It turns out we never put the same code fix on Query servers. They were not using this configuration property.
This PR is to put the same code fix we have on Ingest servers into the Query servers.

Coverage increased (+0.01%) to 74.975% when pulling c4a334299603e24f083b03a03aa9741f74424658 on shintasmith:query_noclose into 502d2befe56cf6a11c7e737e509f2bade82c2629 on rackerlabs:master.


Artist cover images should sync based on cover set in volt

Similar to show cover images, artist cover images should sync based on the cover set in volt to avoid confusion from partners

[ ]  ensure cover images sync as per selections in volt
[ ] remove ability to set cover in folio

 Cover images for artists should sync from Volt to Folio


Add possibility to change theme

Site should have a possibility to swap between light and dark themes

Currently, there is no possibility to manually set schema. It is automatically adjusted to the user's current system theme.


text edited

I highlighted the words 'Open source' and gave some margin for spacing.

added the zoom out animation and golden color.

@Kruthardh11 Looks like this PR has merge conflicts. Please check.

i have resolved merge conflicts

Youve made a typo as well as the padding to the word open source isnt looking well.

it shows like this for me

 it shows like this for me

Thats because you havent made it responsive for all screens.

i sorted the the typo and made it responsive. let me know if any changes needed.

I resolved the conflicts. @ArslanYM please check it out

@Kruthardh11 Before I merge this PR , I want you to go through few documentation for further contributions. This way it would be easier for me to manage this project.
Links :

Commit
PR
merge conflicts

Surely.
I apologise for my mistakes. I'm considerably new to open source. I'm still learning. Thank you.

Surely. I apologise for my mistakes. I'm considerably new to open source. I'm still learning. Thank you.

I understand youre new to open source , thats why this is the best time to understand these practices. You dont owe an apology to anyone. Grow as you learn. Thanks


Fix #469, Addresses invalid CodeQL language option

Checklist (Please check before submitting)

[x] I reviewed the Contributing Guide.
[x] I signed and emailed the appropriate Contributor License Agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>
Describe the contribution
Fixes #469. CodeQL will no longer throw an error regarding valid language input options.
Testing performed
None.
Expected behavior changes
CodeQL will no longer throw an error regarding valid language input options.
System(s) tested on
n/a
Contributor Info - All information REQUIRED for consideration of pull request
@thnkslprpt

@thnkslprpt Hello! Your pull requests were discussed and approved in today's CCB meeting. Would you please confirm that you signed and emailed the appropriate Contributor License Agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>

Yes @chillfig - I've sent through both versions.
Thanks


opensubdiv: fix glew paths

Include and lib are not in ${glew} but in ${glew.dev}.
This changes what is found by the cmake of opensubdiv and some features
are now enabled, such as OpenGL 4.2 support.
Motivation for this change
Get opensubdiv right.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

How can I check that opengl 4.2 is enabled?

@Mic92 Well, OpenGL 4.2 was an example.
If I'm building opensubdiv with the current nixpkgs, I got this log:
...
-- Found GLEW: /nix/store/dxsxz9bh2n945viqpy82bx6m02ld1x46-glew-2.0.0-bin/include
...
CMake Warning at CMakeLists.txt:413 (message):
  OpenGL 4.2 was not found : support for GLSL transform feedback kernels will
  be disabled in Osd.  If you have an OpenGL SDK installed (version 4.2 or
  above), please refer to the FindOpenGL.cmake shared module in your cmake
  installation.

CMake Warning at CMakeLists.txt:428 (message):
  OpenGL 4.3 was not found : support for GLSL compute shader kernels will be
  disabled in Osd.  If you have an OpenGL SDK installed (version 4.3 or
  above), please refer to the FindOpenGL.cmake shared module in your cmake
  installation.

Observe how it complains about OpenGL 4.2 and 4.3 not enabled. See also how it has found GLEW in a totally wrong path:
$ ls /nix/store/dxsxz9bh2n945viqpy82bx6m02ld1x46-glew-2.0.0-bin/include
ls: cannot access '/nix/store/dxsxz9bh2n945viqpy82bx6m02ld1x46-glew-2.0.0-bin/include': No such file or directory

On the other hand, building with the fix:
...
-- Found GLEW: /nix/store/brs5lckc7rwq2njrlsqz0w9585xfjf4y-glew-2.0.0-dev/include
...

There is no error about OpenGL being disable AND the GLEW path is correct:
$ ls /nix/store/brs5lckc7rwq2njrlsqz0w9585xfjf4y-glew-2.0.0-dev/include
GL

But simply, we can check what's inside glew and glew.dev:
$ nix-build -A glew
/nix/store/dxsxz9bh2n945viqpy82bx6m02ld1x46-glew-2.0.0-bin
$ ls /nix/store/dxsxz9bh2n945viqpy82bx6m02ld1x46-glew-2.0.0-bin
bin
$ nix-build -A glew.dev
/nix/store/brs5lckc7rwq2njrlsqz0w9585xfjf4y-glew-2.0.0-dev
$ ls /nix/store/brs5lckc7rwq2njrlsqz0w9585xfjf4y-glew-2.0.0-dev
include  lib  nix-support

Thanks. Now I see the difference.

Does this also affect 17.09?

@Mic92 I think it also affect 17.09. I quickly checked and GLEW was already splitted in 17.09 AND building opensubdiv leads to the same errors.

fixed also there.


Show Vue School logo in free weekend banner mobile

This PR updates the free weekend banner to include the logo in the mobile viewport.
Screenshot
Related to https://github.com/vuejs/vue-router/pull/3737

@nicodevs can you check the conflicts?

@posva Thanks for the heads up, the conflicts are now solved.


Caption format issues

when I was trying to download some courses from udemy  I notice that some courses's captions have an error in its syntax of srt
for example in this course udemy.com/animationselling the caption file is like this
so it doesn't open by vlc

notice that all videos in this course captions aren't working

what is wrong with caption and it will be great if you can share the course url.

http://udemy.com/animationselling

@thesalah1212 i don't have this course in my account so i cannot test against this course but i have tested against many others and i didn't get any starting point from 0 so i will wait for more reports for now i 'm closing this.


SuppressWarning.hh missing

OS: Ubuntu 22.04
ROS Distro: Humble
Built from source or installed: Built from source
Package version:  1.0.0

** Expected behaviour
Successful build with colcon to install
Actual behaviour
Missing File Error
`
--- stderr: turtlebot4_ignition_gui_plugins
In file included from /usr/include/ignition/plugin1/gz/plugin/Register.hh:22,
from /usr/include/ignition/plugin1/ignition/plugin/Register.hh:18,
from /home/zenas/turtlebot4_ws/src/turtlebot4_simulator/turtlebot4_ignition_gui_plugins/Turtlebot4Hmi/Turtlebot4Hmi.cc:26:
/usr/include/ignition/plugin1/gz/plugin/detail/Register.hh:28:10: fatal error: gz/utilities/SuppressWarning.hh: No such file or directory
28 | #include <gz/utilities/SuppressWarning.hh>
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
gmake[2]: *** [Turtlebot4Hmi/CMakeFiles/Turtlebot4Hmi.dir/build.make:99: Turtlebot4Hmi/CMakeFiles/Turtlebot4Hmi.dir/Turtlebot4Hmi.cc.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:153: Turtlebot4Hmi/CMakeFiles/Turtlebot4Hmi.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
Failed   <<< turtlebot4_ignition_gui_plugins [1.13s, exited with code 2]
Summary: 1 package finished [1.36s]
1 package failed: turtlebot4_ignition_gui_plugins
1 package had stderr output: turtlebot4_ignition_gui_plugins
2 packages not processed
`
To Reproduce
Tried installing the package using the source installation steps provided in here
Other notes
Both Gazebo and Turtlebot4 are installed successfully and working, only this simulation environment is producing an error.

You might be missing some packages. Did you run rosdep install --from-path src -yi?

Yes, I did

It's now working. For some reason it started working after switch to this. Maybe specifying the ros distro when running rosdep helped. I can't say for sure.


Add command to switch active Netlify account

- Summary

Adds a new command (netlify switch) to allow you to switch between different Netlify accounts in CLI.
Fixes: https://github.com/netlify/cli/issues/105
- Test plan

netlify switch
Select "I would like to login to a new account". You'll be logged into the new account and your current account in CLI will be updated to reflect the latest.
netlify switch, you can now see both accounts and switch between them

- Description for the changelog

Adds a new flag to netlify login (--new), that allows you to login to a new Netlify account even if you're already logged in
Adds a new command netlify switch that allows you to switch between already logged in accounts and login to a new one

- A picture of a cute animal (not mandatory but encouraged)

i never knew we were working on this. imo its fine to run two commands logout and login again rather than support this

I personally think this will be useful since doing logout and login requires following the whole login flow each time. While this is simple and quick.
Looking for more opinions.

david clarified that its useful for "Work vs personal accounts. Also agencies
And support" :) up to you really i was just adding random not very helpful comments and i will stop now

Haha no problem. Your comments are always welcome 🙌🏽

Going to merge this since there's a general positive signal. Can revert if there's an objection.


Tutorial and resources about the Chrome DevTools Protocol?

This whole chrome remote debugging affair is hard to navigate. Is there a link to a proper tutorial?

The README and the Wiki are all that you need to get you started, also the issues contains a lot of useful use cases and solutions to problems.
Speaking of tutorials you may want to take a look at Getting Started with Headless Chrome.
Finally, additional information can be found in the Resources section of the README.
You have to consider that the Chrome DevTools Protocol is a pretty low-level solution, you may want to give Puppeteer a try which is a higher-level API.

Google really needs to hire some of the Java guys to write up a Chrome Trail!, There's no low-level vs high-level, there's only good tutorial vs messy tutorial.
In any case, I bookmarked some creams in the mess for heritors' sake:

https://stackoverflow.com/questions/28430479/using-google-chrome-remote-debugging-protocol
https://blog.phantombuster.com/web-scraping-in-2017-headless-chrome-tips-tricks-4d6521d695e8
https://minhcung.me/chrome-headless-a-whole-new-world-of-web-scraping/
https://chromedevtools.github.io/devtools-protocol/
https://chromium.googlesource.com/chromium/src/+/lkgr/headless/README.md
https://developers.google.com/web/updates/2017/04/headless-chrome
https://stackoverflow.com/questions/7848878/how-to-use-google-chrome-remote-debugging-protocol-in-http
https://github.com/auchenberg/devtools-remote
https://github.com/auchenberg/chrome-devtools-app


remove unused private function

Use of this function was removed in #10134, where I didn't realize it was the only use.

This passed macos_tests, which originally caught this error.  The remainder of the CI checks are, I assume, lost to AWS.


Update contributors.js

Description
Here my contribution to "The Duck Gallery".

You must respect the following conditions

Your contribution must respect the code of conduct
It's your first PR to this repository
You have added your image of the duck downloaded from https://pimpmyduck.zenika.com
You have not changed the image of the duck
The image filename must be your GitHub username (ex: your-username.png)
You are using your own GitHub user and it's the same as the PR author
Into the contributors.js file, the message must be short and respect the code of conduct

If the PR doesn't respect those conditions, it will be closed

Oh, @Kritiji you make correction in an other PR. I close this one.


grager, builds (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Question regarding VDI creation

Not really so much an issue, just a question.
The initial machine creates itself (no problem). However, when I try to create VDI machines from initial machine, the process is very slow.
For example, I tried to populate some VDI machines with a Windows 7 32-bit initial machine. Over the period of the population, the speed is going at under 1MB/sec.
Eventually, the copy process just fails after awhile.
Now, I was thinking it could be our disk performance (we currently have conventional 1TB drives in a RAID 10 array; we are going to upgrade to 8TB RAID 10 SSD storage). We are piloting this right now. If it works as intended, we are going to purchase an exceptionally beefy machine.
Just wondering if there is any advice on why the QCOW2 creation for VDI machines is slow.
Any help is much appreciated! Thanks!

This should not perform poorly like this. VDI machines are created from snapshots, so there is no big IO load. The only part where entire image is coped is when you copy initial image from source machine.
Perhaps you are creating too many VDI vms? All populated machines are started after new snapshot is created, this may overload your hypervisor resources if you overpopulate it.

Thank you for your assistance. I believe our resources are limited. However, I think we are going to proceed with upgrading our hardware.
Thank you again for all of your assistance! This project is amazing! :)


Unable to download older version of liqo

curl --fail -LS "https://github.com/liqotech/liqo/releases/download/v0.4.0/liqoctl-linux-amd64.tar.gz" | tar -xz
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404

It's not a bug. You should use this link https://github.com/liqotech/liqo/releases/download/v0.4.0/liqoctl-linux-amd64 (without .tar.gz, and then tar -xz). Everything should be fine.

Hi @sandyvanam, we modified the v0.4.0 release to include also the .tar.gz artifacts for the sake of compatibility with the installation command presented in the documentation.
Still, it is worth mentioning that liqoctl has undergone significant changes between v0.4.0 and v0.5.0, and most of the commands presented in the documentation will not work as is with v0.4.0. For this reason, we suggest upgrading to the latest version of Liqo.


Light status bar with dark icons

This is not really an issue but a question about how to do it in flutter.
I asked this question on SO but didn't get any answers.
https://stackoverflow.com/questions/49179175/how-to-make-android-status-bar-light-in-flutter
I want to change the status bar to light on Android which automatically turns icons dark. This was introduced in Android v23 with <item name="android:windowLightStatusBar">true</item>.
Please note that my question is NOT about changing status bar color or about removing translucency. I know how to do all that. I just want to change the color of icons which in Android is possible with windowLightStatusBar property.
Check image below for reference.

Hi @kalehv, right now flutter doesn't expose the necessary API to configure the status bar or other parts of the system chrome for Android.  I'm working on a PR to add the low level functionality in the engine here: https://github.com/flutter/flutter/pull/17171 - this should be done fairly soon!
After that I'll also need to update the framework and make a breaking change to the AppBar so that the low level API is useable - this will take a bit longer.

I assume this was fixed by #17171

During launch in splash screen the status and navigation bar still can't be set to the light mode. Changing the appearance after the splash screen is not desirable and interrupts the user experience. Why is it not possible to change the mode with <item name="android:windowLightStatusBar">true</item> in the main activtiy?

For making status bar transparent. In main.dart add
Future<void> main() async {   SystemChrome.setSystemUIOverlayStyle(     SystemUiOverlayStyle(       statusBarColor: Colors.transparent,     ),   );   ... }
Then in appbar add this
return AppBar(   brightness: Brightness.light,   ... )
Hope it helps


Use outputs instead of env vars

Seems like a better practice and may prevent issues with env var weirdness (#42)

This sounds a good idea in any case, although I'm still convinced things shouldn't break if an environment variable with a particular pattern happens to be defined, this could happen also outside of this workflow.


Remove stroke color on all icons

Removing this will allow coloring with CSS using stroke and fill properties like so:
.icon {
  color:blue;
  fill: currentColor;
  stroke: currentColor;
}

Removing this will allow coloring with CSS using stroke and fill properties like so:
.icon {
  color:blue;
  fill: currentColor;
  stroke: currentColor;
}

That's nice. Can you provide some after and before screenshot? I'm waiting to accept your PR.

Without this PR:

With this PR:

Same code used on both cases:
<x-ionicon-map-outline class="w-5 h-5 m-auto text-red-500 fill-current stroke-current" />


Upgrade from 1.6.6 to 1.7.1 and grid stopped working

Hi:
I've upgraded my project from 1.6.6 to 1.7.1 (refresh-dependencies, clean, easygrid-setup, compile).  When I now execute my project, any page with a grid comes up "crosshatched out" in Firefox and Chrome.  I also see "Warning:  Please select row" in dialog boxes at the end of the render.  Of course, it all worked fine before the upgrade.
I do see the grids rendering properly and loading with data.
Note that my pages have master/child grids.  It is behaving like the child grids are trying to load before any rows have been selected in the master grids.
What might have changed that could cause this?
Example code:

Click on a List to view the rental orders

		  	<grid:grid name="managerList">
		  		<grid:set altRows="true"/>
		  		<grid:set width="auto" height="auto" caption="Lists" />
		  		<grid:set col="counts" label="Counts" align="center" formatter="f:countsViewButton" />
		  	</grid:grid>

Click on the first column next to the PAS number to view the order details

	  	  <grid:grid name="managerListOrders" masterGrid="managerList" childParamName="listId">
		      <grid:set altRows="true"/>
	  	  	  <grid:set width="auto" height="auto" caption="Rental Orders"/>
	  	  </grid:grid>
	  	  <grid:exportButton name="managerListOrders" formats="['excel','csv','pdf']"/>

	def managerListGrid = {
		dataSourceType 'gorm'
		domainClass RentalList
		initialCriteria {
			def user = springSecurityService.currentUser
			manager {
				eq ('id', user.customerid)
			}
//			order("title")
		}
		enableFilter true
		inlineEdit false
		jqgrid {
			rowNum 10
			sortname 'title'
			sortorder 'asc'
		}
		columns {
			id {
				jqgrid {
					hidden true
				}
			}
			title {
				label "List"
				jqgrid {
					width "400"
				}
			}
			counts {
				value {
					def urlStr = "https://www.paslists.com/counts/" + URLEncoder.encode(it.lst_table,"UTF-8") + ".xlsx"
					def http = new HTTPBuilder(urlStr)
					def lst_table = it.lst_table
					def retval
					http.request(HEAD) {
						response.success = { resp ->
							retval = lst_table + ".xlsx"
						}
						response.failure = { resp ->
							retval = -1
						}
					}
					if (retval == -1) {
						urlStr = "https://www.paslists.com/counts/" + URLEncoder.encode(it.lst_table,"UTF-8") + ".pdf"
						http = new HTTPBuilder(urlStr)
						http.request(HEAD) {
							response.success = { resp ->
								retval = lst_table + ".pdf"
							}
							response.failure = { resp ->
								retval = -1
							}
						}
					}
					retval
				}
				enableFilter false
				jqgrid {
					width "100"
					align "center"
				}
				export {
					hidden true
				}
			}
		}
	}

	def managerListOrdersGrid = {
		dataSourceType 'gorm'
		domainClass PASOrder
		idColName 'ordnum'
		subGrid 'managerListOrderDetails'
		globalFilterClosure { params ->
			rentalList {
				eq('id', params.listId ? params.listId as long: -1l)
			}
		}
		initialCriteria {
			eq('ord_type','LRO')
			eq('cancelled','1')
		}
		jqgrid {
			sortname 'orddate'
			sortorder 'desc'
			rowNum 50
		}
		enableFilter true
		inlineEdit false
		columns {
			ordnum {
				type 'id'
				property 'ordnum'
				label "PAS Num"
				jqgrid {
					width "75"
					align "center"
				}
			}
			mgrnum {
				label "Mgr Num"
				jqgrid {
					width "100"
					align "center"
				}
			}
			bkrnum {
				label "Broker PO"
				jqgrid {
					width "100"
					align "center"
				}
			}
			mailer {
				label "Mailer"
				value { it.rentalMailer.company }
				sortProperty "rentalMailer.company"
				filterProperty "rentalMailer.company"
				jqgrid {
					width "175"
				}
			}
			orddate {
				label "Order Date"
				jqgrid {
					width "100"
					align "center"
				}
			}
			maildate {
				label "Mail Date"
				jqgrid {
					width "100"
					align "center"
				}
			}
			wantdate {
				label "Want Date"
				jqgrid {
					width "100"
					align "center"
				}
			}
			shipdate {
				label "Ship Date"
				jqgrid {
					width "100"
					align "center"
				}
			}
			statusDesc {
				label "Status"
				jqgrid {
					width "100"
					align "center"
				}
			}
			decoy {
				label "Decoy String"
				jqgrid {
					width "100"
					align "center"
				}
			}
		}
	}

Tried a "clean-all" instead of "clean".  Unfortunately, that did not help either.

Hi,
It could be the assets pipeline plugin.

Is there a particular version I need?  I have asset-pipeline:1.9.9
On 04/06/2015 02:49 PM, Tudor Malene wrote:

Hi,
It could be the assets pipeline plugin.
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90195016.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

Umm,
try a newer one. For ex 2.1.5

Will do.  Thanks.
On 04/06/2015 02:53 PM, Tudor Malene wrote:

Umm,
try a newer one. For ex 2.1.5
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90196522.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

Unfortunately, that did not fix it.
On 04/06/2015 02:53 PM, Tudor Malene wrote:

Umm,
try a newer one. For ex 2.1.5
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90196522.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

I had to add a dependency to get a production war file to build after
upgrading to asset-pipeline:2.1.5:
dependencies {
      ...
      compile 'com.google.guava:guava:18.0'  // required for asset pipeline
      ...
}

Then another refresh-dependencies, clean-all, compile.  Unfortunately,
that did not fix it either. :-(
On 04/06/2015 02:53 PM, Tudor Malene wrote:

Umm,
try a newer one. For ex 2.1.5
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90196522.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

You could try to compare the html of the page rendered with 1.6.6 and with 1.7.1
Maybe we spot something.

Good idea.  Trouble is I can't find any difference.  Any ideas on how to
do this comparison, other than manually on the Firefox Inspector
screen?  The pages are buried pretty deep in my application, behind
Spring Security logins.  It's hard to just do a wget or a curl. :-/
On 04/06/2015 03:33 PM, Tudor Malene wrote:

You could try to compare the html of the page rendered with 1.6.6 and
with 1.7.1
Maybe we spot something.
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90207834.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

Question:  How does Easygrid know when to render a child grid?  That
seems to be the problem here.  It seems to be trying to jump to
rendering the child grid before a master row is chosen.
On 04/06/2015 03:33 PM, Tudor Malene wrote:

You could try to compare the html of the page rendered with 1.6.6 and
with 1.7.1
Maybe we spot something.
—
Reply to this email directly or view it on GitHub
https://github.com/tudor-malene/Easygrid/issues/146#issuecomment-90207834.

--
Eric H. Raskin<PHONE_NUMBER> x120
Professional Advertising Systems Inc.<PHONE_NUMBER> fax
200 Business Park Dr Suite 304<EMAIL_ADDRESS>Armonk, NY 10504 	http://www.paslists.com

Hmmm... There is something that seems different upon closer inspection.  On 1.7.1 Easygrid, I am seeing "::before" and "::after" in the Web Console.  I don't see those things on 1.6.6.  What are they?

Version 1.7.1:

Version 1.6.6:

Note the ::before and ::after tags in the 1.7.1 version.  Where are they coming from?

So I dumbed down one of my pages to a single grid and it still displays incorrectly.  I see get a Warning box with "Please, select row" at the bottom of the age and the page is grayed out:

You can just make out the cross-hatching in the screen shot.  That should not be there.
Any idea where I can look now?   The GSP contains:

    <meta charset="utf-8">
    <meta name="layout" content="main"
    <title>Professional Advertising Systems Inc.</title>

Click on a Mailer to view their merge jobs

          <grid:grid name="brokerMailers" >
              <grid:set altRows="true"/>
              <grid:set width="500" height="auto" caption="Mailers" />
              <grid:set col="id" hidden="f:true" />
              <grid:set col="company" width="250" label="Mailer"/>
          </grid:grid>

I have tried commenting out portions of this, but I still get the same issue.  Seems like something is wrong with jqgrid itself?  How would I go about finding that?

Well, I've found the problem.  For some reason, the jqgrid CSS files are not being pulled in by the asset-pipeline.  When I load them directly from a CDN, everything works perfectly again.
I don't know enough about how asset-pipeline works with plugins to help you, but this is what you have in easygrid.jqgrid.css:
/*
*= require /jquery-ui/jquery-ui.css
*= require /jquery-ui/jquery-ui.theme.css
*= require /jqGrid/ui.jqgrid.css
*/

I'll bet it's the capital "G" in jqGrid.  The directory entry in your plugin is assets/stylesheets/jqgrid (not jqGrid).


Flip order of CXCancellation and 1QGateOptimization in default transpiler flow

What is the expected enhancement?
Flipping the order of CXCancellation and 1QGateOptimization in the default transpiler flow allows for the possibility of single-qubit gates in between cx gates to add up to identity, which in turn allows the cx gates to collected by the cx gate cancellation.  The current ordering does not allow for this, and is thus not as efficient at reducing depth.

Ideally one would want to do 1Q optim then cx cancellation in a loop until the depth reaches a fixed point.

Actually I take this back, it is not so clear which one to choose.

Actually it is more efficient.

The passmanager supports that. I can have a look.

I would also argue that cx cancellation should be before cx direction swap otherwise you cannot collection cx that have their directions swapped.  It also makes the cx cancellation pass more efficient as it is not swapping directions for cx gates that could be canceled.
So the new order should be 1Q optim, cx cancel, cx direction.

The worst case scenario in this modification (assuming everything is run once) is two leftover U3 gates, where as keeping things the same gives a worse case of 2 cx gates.  Since cx gates are 10x more costly, minimizing their count is the way to go.

we should do this in a loop until depth reaches fixed point.

PR #2086 fixes this.


Update documentation generate script

See https://github.com/apple/swift-markdown/pull/149#issuecomment-1734092183
Summary
Update the documentation generate script to use swift-docc-plugin.
The hand write script to call docc directly is tricky and may be out of date to enjoy the latest feature of docc. Instead we prefer to use swift-docc-plugin.
After we add SymbolKit as a dependency of swift-docc-plugin, we can't directly use swift-docc-plugin in this Package. So this PR add a new Package.swift to avoid such circle dependency issue.
Testing
Confirm the script build and work as expected.
git fetch
git worktree add --checkout gh-pages origin/gh-pages
export SWIFT_DOCC_SYMBOLKIT_ROOT=$PWD
swift package \
    --package-path Documentation \
    --allow-writing-to-directory "$SWIFT_DOCC_SYMBOLKIT_ROOT/gh-pages/docs" \
    generate-documentation \
    --target SymbolKit \
    --disable-indexing \
    --transform-for-static-hosting \
    --hosting-base-path swift-docc-symbolkit \
    --output-path "$SWIFT_DOCC_SYMBOLKIT_ROOT/gh-pages/docs"

Smart! I like this a lot – we should definitely offer this as a solution to folks who want to use the plugin for building docs but don't want to add an additional dependency.
For SymbolKit specifically – I'm concerned this may break in the future since the swift-docc-plugin itself depends on SymbolKit. I'm seeing this in the build logs:
warning: 'swift-docc-plugin': 'swift-docc-plugin' dependency on 'https://github.com/apple/swift-docc-symbolkit' conflicts with dependency on '/Users/ekusters/Developer/swift-docc-symbolkit' which has the same identity 'swift-docc-symbolkit'. this will be escalated to an error in future versions of SwiftPM.

What do you think?

Smart! I like this a lot – we should definitely offer this as a solution to folks who want to use the plugin for building docs but don't want to add an additional dependency.
For SymbolKit specifically – I'm concerned this may break in the future since the swift-docc-plugin itself depends on SymbolKit. I'm seeing this in the build logs:
warning: 'swift-docc-plugin': 'swift-docc-plugin' dependency on 'https://github.com/apple/swift-docc-symbolkit' conflicts with dependency on '/Users/ekusters/Developer/swift-docc-symbolkit' which has the same identity 'swift-docc-symbolkit'. this will be escalated to an error in future versions of SwiftPM.

What do you think?

Yes. I also noticed this warning locally. (Forgot to link it to the body of the PR)
Can we discuss this issue with folks in SwiftPM team to offer some API to fix it?
We have moduleAliases on Target.Dependency. And I'd to propose to add a new API for Package.Dependency
eg.
.package(name: "SymbolKit", path: "../."),
⬇
.package(name: "SymbolKit", path: "../.", id: "xx")

Another way to fix this is split snippet-extract to its own Package file so that swift-docc-plugin would not have a dependency on swift-docc-symbolkit anymore.
https://github.com/apple/swift-docc-plugin/blob/3aad4aabc42835f09f600bb17070f6a11bc84180/Package.swift#L75-L80


Recreate lock files to install packages properly

What kind of change does this PR introduce?
This PR is for #156.
Recreate yarn.lock and package-lock.json.
Checked

node v10.18.0 yarn v1.22.4 npm v6.13.4
node v12.10.0 yarn v1.22.4 npm v6.10.3

Also added a couple of yarn commands to README.md

Does this PR introduce a breaking change?
Nope

What needs to be documented once your changes are merged?
There is no need to update the documentation.

@koji - thanks for this! Just a few notes:

In the ideal case, we'd consistently use the defined node version in the .nvmrc while developing. We can do so by:

$ nvm use
$ npm install
$ npm run develop

It is important if you switch between node versions to remove your node_modules directory.
I can confirm that your PR works for both node 10 and node 12 as long as you delete your node_modules before installing.

@joeyklee
Oh, I missed .nvmrc completely.
Thank you for letting me know that. I'll keep in mind that.

no problem! thanks for troubleshooting this! <3


breaks if Cache-Control is array

curl -v https://www.reservaentradas.com/cines
< HTTP/1.1 200 OK
< Date: Wed, 05 Feb 2020 18:51:26 GMT
< Server: Apache/2.2.22 (Debian)
< Expires: Thu, 19 Nov 1981 08:52:00 GMT
< Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0
< Vary: Accept-Encoding
< Cache-Control: max-age=0, no-cache
< Transfer-Encoding: chunked
< Content-Type: text/html; charset=utf-8
<

Produces error:
TypeError: header.trim is not a function
          at parseCacheControl (/Users/gajus/Documents/dev/rayroute/rayman/node_modules/http-cache-semantics/index.js:62:26)
          at new CachePolicy (/Users/gajus/Documents/dev/rayroute/rayman/node_modules/http-cache-semantics/index.js:119:23)
          at action (/Users/gajus/Documents/dev/rayroute/rayman/src/interceptors/createCacheInterceptor/index.js:68:27)
          at Object.responseActions (/Users/gajus/Documents/dev/rayroute/rayman/node_modules/@rayroute/raygun/src/factories/createInterceptorActions.js:152:22)
          at processTicksAndRejections (internal/process/task_queues.js:97:5)
          at handleRequest (/Users/gajus/Documents/dev/rayroute/rayman/node_modules/@rayroute/raygun/src/factories/createRequestHandler.js:69:28)

Oops, indeed!
Could you make it detect non-string header value and run join(',') on it?

That is indeed how I have patched it locally.
+const normalizeHeaders = (headers: HeadersType): HeadersType => {
+  const normalizedHeaders = {
+    ...headers,
+  };
+
+  // @see https://github.com/kornelski/http-cache-semantics/issues/28
+  if (Array.isArray(normalizedHeaders['cache-control'])) {
+    normalizedHeaders['cache-control'] = normalizedHeaders['cache-control'].join(', ');
+  }
+
+  return normalizedHeaders;
+};
+


dts: stm32: u5: fixes dts warning

I am getting a warning if this is not set.
Fixes warning: /soc/interrupt-controller:
Missing #address-cells in interrupt provider

I get the warning when compiling, e.g.hello_world, for a stm32u5 board, e.g. b_u585i_iot02a or nucleo_u575zi_q:
west build -c -p -b nucleo_u575zi_q -d build-ssi-u575 zephyr/samples/hello_world

Actually I am getting some more warnings:

/zephyrproject/build-ssi-u575/zephyr/zephyr.dts:707.7-730.5: Warning (simple_bus_reg): /soc/can: missing or empty reg/ranges property
zephyrproject/build-ssi-u575/zephyr/zephyr.dts:625.30-654.5: Warning (spi_bus_bridge): /soc/octospi@420d2400: node name for SPI buses should be 'spi'
<stdout>: Warning (spi_bus_reg): Failed prerequisite 'spi_bus_bridge'
zephyrproject/build-ssi-u575/zephyr/zephyr.dts:100.39-109.5: Warning (interrupt_provider): /soc/interrupt-controller@46022000: Missing #address-cells in interrupt provider <- only this is fixed by this PR.

I am on Zephyr 3.3.99.

Strange, I've run the same command on my setup and I still do not see any warning.
I'm on Windows 10 and I have first used Zephyr SDK 0.15.2 and then 0.16.0 thinking it could be related to an SDK update.
What is your setup? OS? SDK version?

And DTC version too. I was on 1.4.7 on my side, but I just upgraded to 1.5.0 and I still have no warning.

Below is my complete log output, maybe you can read something from it. This is the information I have:

SDK version is 0.15.2
DT version is 1.6.1
Zephyr version is 3.3.99
CMake 3.24.2

What confuses me is this line, where the Zephyr 3.3.99 version and the build: zephyr-v3.3.0 diverge. Maybe I updated zephyr incorrectly and this could cause this issue?
[3/152] Generating include/generated/version.h
-- Zephyr version: 3.3.99 (/Users/user/devel/zephyrproject/zephyr), build: zephyr-v3.3.0-1142-g4a2eac72b0f9

Complete output:
$ west build -c -p -b b_u585i_iot02a -d build-ssi-u575 zephyr/samples/hello_world
-- west build: making build dir /Users/user/devel/zephyrproject/build-ssi-u575 pristine
-- west build: generating a build system
Loading Zephyr default modules (Zephyr base).
-- Application: /Users/user/devel/zephyrproject/zephyr/samples/hello_world
-- CMake version: 3.24.2
-- Found Python3: /Users/user/devel/zephyrproject/.venv/bin/python3.8 (found suitable exact version "3.8.16") found components: Interpreter
-- Cache files will be written to: /Users/z003a23c/Library/Caches/zephyr
-- Zephyr version: 3.3.99 (/Users/user/devel/zephyrproject/zephyr)
-- Found west (found suitable version "1.0.0", minimum required is "0.7.1")
-- Board: b_u585i_iot02a
-- ZEPHYR_TOOLCHAIN_VARIANT not set, trying to locate Zephyr SDK
-- Found host-tools: zephyr 0.15.2 (/Users/user/devel/tools/zephyr-sdk-0.15.2)
-- Found toolchain: zephyr 0.15.2 (/Users/user/devel/tools/zephyr-sdk-0.15.2)
-- Found Dtc: /opt/homebrew/bin/dtc (found suitable version "1.6.1", minimum required is "1.4.6")
-- Found BOARD.dts: /Users/user/devel/zephyrproject/zephyr/boards/arm/b_u585i_iot02a/b_u585i_iot02a.dts
-- Generated zephyr.dts: /Users/user/devel/zephyrproject/build-ssi-u575/zephyr/zephyr.dts
-- Generated devicetree_generated.h: /Users/user/devel/zephyrproject/build-ssi-u575/zephyr/include/generated/devicetree_generated.h
-- Including generated dts.cmake file: /Users/user/devel/zephyrproject/build-ssi-u575/zephyr/dts.cmake
/Users/user/devel/zephyrproject/build-ssi-u575/zephyr/zephyr.dts:707.7-730.5: Warning (simple_bus_reg): /soc/can: missing or empty reg/ranges property
/Users/user/devel/zephyrproject/build-ssi-u575/zephyr/zephyr.dts:625.30-654.5: Warning (spi_bus_bridge): /soc/octospi@420d2400: node name for SPI buses should be 'spi'
<stdout>: Warning (spi_bus_reg): Failed prerequisite 'spi_bus_bridge'
/Users/user/devel/zephyrproject/build-ssi-u575/zephyr/zephyr.dts:100.39-109.5: Warning (interrupt_provider): /soc/interrupt-controller@46022000: Missing #address-cells in interrupt provider
Parsing /Users/user/devel/zephyrproject/zephyr/Kconfig
Loaded configuration '/Users/user/devel/zephyrproject/zephyr/boards/arm/b_u585i_iot02a/b_u585i_iot02a_defconfig'
Merged configuration '/Users/user/devel/zephyrproject/zephyr/samples/hello_world/prj.conf'
Configuration saved to '/Users/user/devel/zephyrproject/build-ssi-u575/zephyr/.config'
Kconfig header saved to '/Users/user/devel/zephyrproject/build-ssi-u575/zephyr/include/generated/autoconf.h'
-- The C compiler identification is GNU 12.1.0
-- The CXX compiler identification is GNU 12.1.0
-- The ASM compiler identification is GNU
-- Found assembler: /Users/user/devel/tools/zephyr-sdk-0.15.2/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/user/devel/zephyrproject/build-ssi-u575
-- west build: building application
[1/152] Preparing syscall dependency handling

[3/152] Generating include/generated/version.h
-- Zephyr version: 3.3.99 (/Users/user/devel/zephyrproject/zephyr), build: zephyr-v3.3.0-1142-g4a2eac72b0f9
[142/152] Linking C executable zephyr/zephyr_pre0.elf

[146/152] Linking C executable zephyr/zephyr_pre1.elf

[152/152] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:       15300 B         2 MB      0.73%
             RAM:        4120 B       768 KB      0.52%
        IDT_LIST:          0 GB         2 KB      0.00%

Sure, will do!


Issue while create query in query builder page

Describe the bug
select build new query and click on edit and search then it shows message
Screenshots

#910


Fix type of SingleTileImageryProvider.fromUrl return type

Fixed the type of SingleTileImageryProvider.fromUrl.
The current type has a return type of void but the return type should be Promise<SingleTileImageryProvider>

(from Cesium.d.ts)

Thank you so much for the pull request @matthias-ccri! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

Hi @matthias-ccri, we'll need a signed CLA in order to review this PR. I don't see one from you in our records.
Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

Thanks, yeah I'm going through the process of getting the CLA signed by the proper people.

I was just told that the CLA was submitted!

Awesome, I can confirm the submitted CLA @matthias-ccri!
Everything looks great! I would ask that you add a note to CHANGES.md with a summary of the fix. Typically we don't include documentation updates, but we do generate a TypeScript definitions file from the documented types, which will be updated with this change.

Thanks @ggetz I've updated the CHANGES.md

Thanks @matthias-ccri!


Fence Rep Changing Unexpectedly

General information
Project issue is for:
SPT-NIGHTLY-3.8.0-29197-d3fdb08-20240323
Expected result:
Fence rep should remain the same
Actual result:
Fence rep reducing on scav death
I have noticed that if I take a vehicle extract as a pmc and get the fence rep gain, if I then go into the following raid as a scav and die it takes away the fence rep gained from the previous vehicle extract. Just recreated it on a new profile. Did EOD edition, started with .20 scav rep. Ran Ground Zero and took vehicle extract. Fence rep went up to .40. Entered a factroy raid as a scav and just ran around until I found a PMC to let kill me. After raid fence rep was back at .20.
Fresh install, with newest dev-core zip as well.
How to reproduce

Enter a raid as a PMC and take a vehicle extract
Note Fence rep after raid
Enter a raid as a Scav and die
Note Fence rep after raid

Attachments

server-2024-03-24.log (104 KiB)
LogOutput.log (74 KiB)
2024.03.24_19-40-17_<IP_ADDRESS>.29197 traces.log (144 KiB)

Originally written by Scavnagoodtime

it seems like we arent copying over the new Fence reputation data to the scav after running handleCarExtract. so the Scav data is now stale for the next time the player runs a scav.
pmcdata:
scavdata:

Attachments

image.png (20 KiB)
image.png (20 KiB)

Originally written by Kaeno

This is happening because we are now trusting the client for rep changes when we are a scav.
We use to only look at the PMCs fence reputation and calculate based on that.
https://dev.sp-tarkov.com/SPT-AKI/Server/commit/d5517d7fdf2cd57efd69a6e14e511d026937c0c8 is the change that made the server trust the client.

Originally written by Kaeno

Hopefully fixed with: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/eca6f4bd75eed75890e8a6e52acf4d0d69d758c9

Originally written by chomp

Re-tested on most recent zip (9fca68) and rep is now functioning as intended. Problem seems to be resolved.

Originally written by Scavnagoodtime


explaining ImageTupletfm

Hi,
I don't fully understand how to create the dataset. Can you clear up some confusing points please? In particular,

what does ImageTuple return? In imagetuple, does it return a figure from show() method or actual images?
in dataset, what is the function parent_label? is it a fastai function?
in dataset creation, the input "files" is a list of paths. each path is a folder containing images for that set. then to create a single (x,y) input to the model, the ImageTupletfm applies on the images from this path sampling them up to seq_len. the sampled set and the parent_label make the (x,y) result? Is this correct?

Hey, thanks for using this little project of mine:

I encourage you to check this:
https://tcapelle.github.io/pytorch/fastai/cv/2021/03/15/image_sequences.html
parent_label is a fastai func:

def parent_label(o):
    "Label `item` with the parent folder name."
    return Path(o).parent.name

-Yes

Thanks for information. One more question: what happens for videos of different lengths like if the video folder does not have enough samples? that is, if we want a seq_len of 20, but we have less than in the folder, how does the model deal with sequence lengths less that it expects?

it does not deal with different lenghts... please submit a PR if you make it work.

Thank you. I'm working on that now.


TRINITY-40: Add trinity templates, set default and cleanup Lessons

This PR:

Duplicates Morpheus templates to trinity
Sets trinity as default (but default value is overwritten to "morpheus" for now)
Makes some cleanup in lessons which is just removing code needed for pre Morpheus portals

Thanks for your work on this, @stetsche !

Ready!

Thanks @stetsche !


Storage Usage

hi,
we have several lookup rollup summaries running on schedules, some every day, some once a week.  They are working great, but i have noticed a potential issue.
Looking at my storage usage, Lookup Rollup Summary Schedule Items is currently using 611.00MB and growing.  With the way SF storage is, im keen to avoid something eating up this much space.
Is there a way to limit this, and delete old ones?  Is there a setting I've missed somewhere?
thanks in advance
james

These should be getting cleaned up as they are processed by your the RollupJob apex class you have scheduled? What i'm wondering though is if you have used the other scheduled mode, the full  recalc scheduled via the Scheduled Calculate button? And still have the Calculation Mode set to Scheduled. This i admit is a bit confusing, but if your using full scheduled re-calc you can you use some other Calculation Mode like Process Builder, this will avoid the DLRS triggers recording changes that are never processed by the RollupJob in your case. If this is your case, you can use the Truncate Object feature to clear the Lookup Rollup Summary Schedule Items out. I'll give some thought on how to better differentiate the two schedule modes better.

hi,
thanks for this.
So, if ive understood correctly....I have changed the Calculation Mode on each of the summaries to Process Builder.  (They were all set to scheduled before & i had scheduled them using the Schedule Calculate button)
I've also then deleted all Lookup Rollup Summary Schedule Items records from the object (did this via Jitterbit, as i couldn't see a truncate option for that object).
These summaries run weekly, so i will check them next week and make sure they are working ok.
thanks again.
james

Yep you got it.


Implementing multithreading

I took a music cover and ran the 2x example test, generating a 13 MB JXL.
This takes a VERY long time to load via jxlpy

In comparison for example with nomacs, which takes less than five times what jxlpy does due to using multiple threads.

Would it be feasible to make this library use multiple threads? I was trying to use an application with Pillow and this extension, and loading this image made the startup time very long.

Hi!
Firstly, sorry for late response, I've got a rather busy week.
Secondly, Thank you so much for adding my library to AUR :)
I also have some good news, your issue can be easily resolved by providing num_threads parameter to JXLPyDecoder.
import jxlpy
import time
import multiprocessing

CPU_COUNT = multiprocessing.cpu_count()

with open('cover_2x.jxl', 'rb') as f:
    data = f.read()

def decode_default():
    dec = jxlpy.JXLPyDecoder(data)
    output = dec.get_frame()
    dec.close()

def decode_multi():
    dec = jxlpy.JXLPyDecoder(data, num_threads=CPU_COUNT)
    output = dec.get_frame()
    dec.close()

for fun in (decode_default, decode_multi):
    old_time = time.time()
    fun()
    print('{}: {}'.format(fun.__name__, time.time()-old_time))

decode_default: 8.677569150924683
decode_multi: 0.7362372875213623

I've decided to one up your 3-day response with nearly a 3-year one.
I was initially asking because of Tauon added support for JXL through this library.
But it's literally just doing from jxlpy import JXLImagePlugin (in t_draw.py and t_main.py modules) and the images are loaded in other parts of the code automagically, is there a way to set the multithreading in the import?
And if not, would you know where else should it be set?


special character while reading rows or columns tag

I am completely newbie to DICOM and was assigned to parse DICOM tags ;) so please bear with me.
When I try to read "Rows" (x00280010) or "columns" (x00280011) data, it returns below characters:

and my code is:
let dicomFileAsBuffer = fs.readFileSync("path_to_dcm_file");
let ds = dicom.parseDicom(dicomFileAsBuffer);
let test = ds.string('x00280010');
console.log(test);

I am able to read most of the other required tags without any issue, it's just "rows" and "columns" returning those special characters instead of number.
I might be doing something wrong because when I try online tool it shows correct value of rows and columns (which is 400 and 400 respectively).
Though, I do have a work around:
var pixelDataElement = ds.elements.x00280010;
console.log(Buffer.from(ds.byteArray.buffer).readInt16LE(pixelDataElement.dataOffset));

which correctly returns number value (400). Still, why ds.string('x00280010') doesn't return "400" in the first place.
Usage:
npm module
node v8.11.3

@jaykumarthaker are you able to provide a minimal reproduction?

I guess it's because rows and columns are not string? ds.string('x00280010')

@swederik yes, ds.int16('x00280010') did the job! 👍
Thank you so much for your help! :)
Thank you @dannyrb for quick & kind reply.


test: enable codecov to be able to see the updated coverage

this commit enable codecov to be able to see the updated coverage
this issue https://github.com/codecov/feedback/issues/354 was fixed

This hangs if there is an exceed in limit of requests to GH, I think using https://github.com/codecov/codecov-action v4 solves it. Could we use that?

@schuellerf are you good here? :) I don't want to dissmiss your change request, but looks resolved :)

Sidenote: It was pretty hard to find (as it's not documented) and I hope that's the correct implementation used, but considering the wide pattern it's really ok to explicitly specify the file to upload
https://github.com/codecov/uploader/blob/main/src/helpers/files.ts#L161


⚠ Bofrost.fr has degraded performance

In c3ff39d, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Bofrost.fr performance has improved in cea669b after 6 minutes.


Fix issue #53700 - Have no sound for notification received

Explanation of Change
This PR adds sound to browser notifications for new messages in New Expensify. Previously, notifications were playing silently by default. The changes implement proper sound playback when notifications are triggered and ensures the silent flag is set correctly.
Fixed Issues
$ https://github.com/Expensify/App/issues/53700
PROPOSAL: $ https://github.com/Expensify/App/issues/53700#issuecomment-2537406732
Tests

Send a message to yourself from another account/browser while the app is in the background
Verify that a browser notification appears with sound
Verify notifications work when the app is:

In a background tab
Minimized
In focus

Offline tests

Not applicable for this feature as notifications require an active connection

QA Steps

Login to New Expensify on two different browsers/devices
Send messages between the accounts while the receiving browser has the app in background
Verify notifications appear with sound on the receiving browser

// TODO: These must be filled out, or the issue title must include "[No QA]."

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos (sound on)

Android: Native
https://github.com/user-attachments/assets/90d18fab-e66d-43c0-866c-5d072bfb9183

Android: mWeb Chrome

https://github.com/user-attachments/assets/54f8341c-a5ac-4bbb-ae1d-a3a09c776613

iOS: Native

https://github.com/user-attachments/assets/b525ac63-34a1-4e3c-91ac-43e3a974382d

iOS: mWeb Safari

https://github.com/user-attachments/assets/235f7f81-43ba-4e70-b3bc-b56983aed0fd

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/79e89aad-c2b2-4d64-acde-42d7b9dc2be4

MacOS: Desktop
https://github.com/user-attachments/assets/220b2045-997c-4828-acf7-c752e3bbff6c

I have read the CLA Document and I hereby sign the CLA

recheck

@JoshIri360 Please include testing videos for all platforms. Even if the issue is specific to one platform, we still need to upload a video and have to test on all the platforms listed on checklist.

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] For any bug fix or new feature in this PR, I verified that sufficient unit tests are included to prevent regressions in this flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/688bdc18-9cfb-44e1-b450-c95c1836d974

Android: mWeb Chrome
https://github.com/user-attachments/assets/63525412-8d06-483b-af63-14c1f2950fad

iOS: Native
https://github.com/user-attachments/assets/fd6986a9-1803-445f-a0ea-45eb34df1792

iOS: mWeb Safari
https://github.com/user-attachments/assets/b25741f4-42ed-4e29-8437-4b0ee91d6d8e

MacOS: Chrome / Safari
Chrome
https://github.com/user-attachments/assets/c946d5a8-ff0c-4db6-8868-cda6698299cc
Safari
https://github.com/user-attachments/assets/8714ad90-40e0-4906-8477-e6eb608dee25

MacOS: Desktop
https://github.com/user-attachments/assets/bfdf6316-654a-49dd-99bb-bb73a4229951


Profiling

What's the correct way to measure run-times of SYCL code? If I have something simple
vector<int> data(...);

queue q;
{
  buffer b(data.data(), data.size());

  // computation
  q.submit(...);
}

How can I measure how long it takes to move data to the device, then to run the computation, and then to move data back? I.e., I want three separate times. Is there a clean way to do this?

With hipSYCL you can use NVIDIA and AMD vendor profiling tools, e.g. nvprof. The SYCL spec also includes a queue profiling feature which will only allow you to measure kernels though and is unimplemented in hipSYCL. The restriction of only measuring kernels is because data transfers are implicit in SYCL and therefore, there's no way for the user to query them unless the explicit copy() functions are used.

I'd be happy to use an explicit copy. Can you give me an idea how I'd do that in the example above?

Your overall code could look something like this:
vector<int> data(...);
queue q;
{
  buffer b(data.size());
  // You need to force the sycl runtime to move data to the device now so that it won't be measured later
  // This can either be done with explicit `copy()` functions or a dummy kernel. With copy():
  q.wait_and_throw(); // Make sure queue is empty before measuring
  double transfer_time1 = timer();
  q.submit([&](sycl::handler& cgh){
    auto acc = b.get_access<sycl::access::mode::discard_write>(cgh);
    cgh.copy(data.data(), acc);
  });
  q.wait_and_throw();
  double transfer_time2=timer();

  // computation
  double t1 = timer();
  q.submit(...);
  q.wait_and_throw()
  double t2 = timer();

  // Copy data back to host, analogously to first copy
  double transfer_time1 = timer();
  q.submit([&](sycl::handler& cgh){
    auto acc = b.get_access<sycl::access::mode::read>(cgh);
    cgh.copy(acc, data.data());
  });
  q.wait_and_throw();
  double transfer_time2=timer();
}

Note that I also have not initialized the buffer with the pointer in order to prevent it from doing the automatic write-back implicitly when it goes out of scope. This way, we can also measure the final device->host data transfer.

(Also note that the measured times here may also include some small runtime overhead since you are also measuring scheduling of operations etc which will not be contained in measurements from tools such as nvprof)

Brilliant. Thanks!

Closing, this seems solved to me - reopen if this is not the case.


Cannot cast Dolby Atmos content via Chromecast

Have you read the FAQ and checked for duplicate open issues?
Yes
If the problem is related to FairPlay, have you read the tutorial?

What version of Shaka Player are you using?

v4.12.5
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Google Chrome version 131.0.6778.140
Windows 11
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Chromecast with Google TV (HD) / android os 12
What are the manifest and license server URIs?

https://d2pbyw961c6y8f.cloudfront.net/toshi/naturesfury_dash/manifest.mpd
What configuration are you using?  What is the output of player.getNonDefaultConfiguration()?

What did you do?

Create a DASH stream of Dolby Atmos content with ffmpeg.
Load it on Shaka Player v4.12.5 on Chrome browser on Windows.
Here's the ffmpeg command I used:
ffmpeg -i input.mp4 -codec copy -f dash -seg_duration 10 -dash_segment_type mp4 -movflags faststart -strict -2 manifest.mpd
What did you expect to happen?
The content cannot be played on Shaka Player itself as expected (not supported), however, casting it to STB such as "Chromecast with Google TV (HD) " should work.
What actually happened?

The casting fails and the STB does not start to play the Dolby Atmos content.
Are you planning to send a PR to fix it?

Hi @Tsuruoka-569 Please keep in mind Dolby Atmos on Chromecast is supported only via passthrough, so your TV Chromecast is connected to needs to be able to decode Atmos.
Do you have any logs from failure?

We are testing STBs connected to a Denon AV receiver that is compatible with Atmos decoding.
I tested further and found that the Atmos casting could succeed with some STB devices such as NVidia Shield Pro. But still fails on Chromecast with Google TV (HD).
From a shaka demo player page, the casting only succeeds when the "play" button (outside of the player window) is pressed after casting connection is established. It does not succeed by clicking the play button inside the player window.
When trying to cast to Chromecast with Google TV (HD), I see this error : shaka.util.Error {severity: 2, category: 4, code: 4032, data: Array(0), handled: false, …} .
I wonder if this is a STB-side implementation issue.
Thanks in advance,
TT

@joeyparrish can you review it? Thanks!


BoundsError: attempt to access 2-element Array

Details

Atom version: 1.33.1
Julia version: 1.0.1
OS: Mac OS
Package versions:

Atom.jl: v0.7.11
julia-client: 0.7.11
ink: 0.9.13

Steps to reproduce

run the following

ex = :(function foo(::Int) end)

type the following

ex.args[1].args[2].

Seems like a bug in Base:
julia> ex = :(function foo(::Int) end)
:(function foo(::Int)
      #= REPL[6]:1 =#
  end)

julia> ex.args[1].args[2].┌ Error: Error in the keymap
│   exception =
│    BoundsError: attempt to access 2-element Array{Any,1} at index [3]
│    Stacktrace:
│     [1] getindex at ./array.jl:731 [inlined]
...


[Bug]: RuntimeError: Error while calling cudaStreamCreateWithFlags(&cstream, cudaStreamNonBlocking) in file ..../dlib/cuda/gpu_data.cpp:231. code: 800, reason: operation not permitted

What Operating System(s) are you seeing this problem on?
Linux (x86-64)
dlib version
19.24.2
Python version
3.8
Compiler
pip 23.2.1
Expected Behavior
No response
Current Behavior
RuntimeError: Error while calling cudaStreamCreateWithFlags(&cstream, cudaStreamNonBlocking) in file /tmp/pip-install-bngevxx6/dlib_192b1240c0634738a0a187ed106e5d89/dlib/cuda/gpu_data.cpp:231. code: 800, reason: operation not permitted
Steps to Reproduce
I use dlib in my python project which on docker ,  part of the code is as follows, sometimes errors will occur
`
self.face_shape_predictor = dlib.shape_predictor('shape_predictor_68_face_landmarks.dat')
self.face_recognition_model = dlib.face_recognition_model_v1('dlib_face_recognition_resnet_model_v1.dat')
face_landmarks = self.face_shape_predictor(image, face_detection)
self.face_recognition_model.compute_face_descriptor(image, face_landmarks, 0, 0.25)
`
Anything else?
No response

Not sure what the deal is.  Probably cuda isn't installed on your system
correctly.  Hard to say.  Try reinstalling cuda and check that stuff like
nvidia-smi reports things are in order.
On Fri, Mar 8, 2024 at 3:07 AM rd-fhl @.***> wrote:

What Operating System(s) are you seeing this problem on?
Linux (x86-64)
dlib version
19.24.2
Python version
3.8
Compiler
pip 23.2.1
Expected Behavior
No response
Current Behavior
RuntimeError: Error while calling cudaStreamCreateWithFlags(&cstream,
cudaStreamNonBlocking) in file
/tmp/pip-install-bngevxx6/dlib_192b1240c0634738a0a187ed106e5d89/dlib/cuda/gpu_data.cpp:231.
code: 800, reason: operation not permitted
Steps to Reproduce
I use dlib in my python project which on docker , part of the code is as
follows, sometimes errors will occur
self.face_shape_predictor =
dlib.shape_predictor('shape_predictor_68_face_landmarks.dat')
self.face_recognition_model =
dlib.face_recognition_model_v1('dlib_face_recognition_resnet_model_v1.dat')
face_landmarks = self.face_shape_predictor(image, face_detection)
self.face_recognition_model.compute_face_descriptor(image, face_landmarks,
0, 0.25)
Anything else?
No response
—
Reply to this email directly, view it on GitHub
https://github.com/davisking/dlib/issues/2927, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPYFR2ZMGJIIZTFRFJC523YXFWTXAVCNFSM6AAAAABEMMMJYKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TKNJQGQYDSNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Not sure what the deal is. Probably cuda isn't installed on your system correctly. Hard to say. Try reinstalling cuda and check that stuff like nvidia-smi reports things are in order.
…
On Fri, Mar 8, 2024 at 3:07 AM rd-fhl @.> wrote: What Operating System(s) are you seeing this problem on? Linux (x86-64) dlib version 19.24.2 Python version 3.8 Compiler pip 23.2.1 Expected Behavior No response Current Behavior RuntimeError: Error while calling cudaStreamCreateWithFlags(&cstream, cudaStreamNonBlocking) in file /tmp/pip-install-bngevxx6/dlib_192b1240c0634738a0a187ed106e5d89/dlib/cuda/gpu_data.cpp:231. code: 800, reason: operation not permitted Steps to Reproduce I use dlib in my python project which on docker , part of the code is as follows, sometimes errors will occur self.face_shape_predictor = dlib.shape_predictor('shape_predictor_68_face_landmarks.dat') self.face_recognition_model = dlib.face_recognition_model_v1('dlib_face_recognition_resnet_model_v1.dat') face_landmarks = self.face_shape_predictor(image, face_detection) self.face_recognition_model.compute_face_descriptor(image, face_landmarks, 0, 0.25) Anything else? No response — Reply to this email directly, view it on GitHub <#2927>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABPYFR2ZMGJIIZTFRFJC523YXFWTXAVCNFSM6AAAAABEMMMJYKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TKNJQGQYDSNI . You are receiving this because you are subscribed to this thread.Message ID: @.>

nvidia-driver515, cuda11.7 has been installed, this is not a problem that will definitely occur, so I don’t know how to eliminate it

Just for the record, we've been suffering from the same (or at least a very similar) issue for more than a year already:
Error while calling cudaStreamCreateWithFlags(&cstream, cudaStreamNonBlocking) in file /app/src/3rdparty/dlib/dlib/cuda/gpu_data.cpp:231. code: 800, reason: operation not permitted
It just happens randomly in production, and is very difficult to reproduce, especially on one's own machine. Mostly things are working great, so I don't think CUDA is installed incorrectly or anything. The problem has occurred on different CUDA versions (v11.7 and v12.3 at least), different GPU types (Tesla and Turing at least), and in both training and inference. I'm not 100 % certain, but I believe it happens on Docker only (which is how we run our prod systems; but it does sometimes happen in CI tests as well, and so far only on Docker and never without – but as said it's uncommon, so this is not conclusive just yet).

Running out of ideas, I'm thinking if the limit discussed here should be bumped up again. 🤔
Might just as well try...

Some tentative results from running on containerd suggest that this problem might be specific to Docker (and its Nvidia driver, I guess).

Alternatively, disabling systemd cgroup management in Docker may help (it did in our case, it seems).

(Found that workaround from here.)


soak test plan

Why?
We are going to do another release. Need soak testing.
What?


invalid container.env

@bodsch thanks for your great work!
I have a problem, with the generated "container.env" file, which looks like
# generated by ansible
-Xmx1gSPRING_BOOT_EXPLODED_APP=true

and causes the JVM to fail at startup: Invalid maximum heap size: -Xmx1gSPRING_BOOT_EXPLODED_APP=true
The inventory looks like:
environments:
  JAVA_HEAP: 1g
  SPRING_BOOT_EXPLODED_APP: "true"

The porblem seems to be the non existent line break between the entries. Can you have a look? Thx!

Hello @timolemke!
Can you please tell me which version of the Ansible role you are using?

Hi @bodsch  I am using the latest version 2.4.0

Thanks
I still need the output of the following command line:
ansible-galaxy collection list | grep bodsch

I have adapted the behaviour when creating the properties and environment files in version 2.4.1. however, this requires the new version of the collection bodsch.core !
Please update the collection and the role.

Works with Version 2.4.2
Thx!


Backend for the state of the regulation editor

This PR implements the backend for tracking and validating the state of the model edited in the regulations editor. It is synchronized with the frontend using events.
The events supported by the main ModelState struct are listed in the src/aeon_events.ts file.
Before reviewing and merging this, finishing at least some of the following things would be nice. However, (except for the first one?), they can be addressed via some subsequent PR.

[x] Divide the current ModelState events into "two groups" - (A) those for adding new objects (regulations, variables) and (B) those for modifying/removing the existing objects. This would enable us to fix some issues with paths and payloads. Events of group A would have a more general path, such as ['model', 'regulation', 'add'], and carry a JSON payload fully encoding some prepared structure (such as RegulationData). Events of group B would have more specific paths encoding the specific object, such as ['model', 'regulation', 'regulator_id', 'target_id', 'remove']. The payload of these events could then be just a single value or even be empty sometimes (now, all the IDs are also part of the payload, making it "bloated").
[x] Finish some remaining minor events for modifying the ModelState and add tests for the some remaining cases.
[x] Update the API src/aeon_events.ts reflecting the previous points.
[ ] Utilize the front-end event API from the TS code (emit the events to the backend reacting to user inputs and react to backend events adjusting the state).
[x] Finish the refresh part of the SessionState API for  ModelState.  There is a placeholder currently, and all the "getter" events should be added.
[ ] Refactor the code :)

The main issues have been (in some way) resolved. The current event API might need some updates/polishing in the future.
The API with events supported by the model's backend is provided in the src/aeon_events.ts. In some following PR, we should connect the front-end and back-end functionality, utilizing this event API in the TS code. For more details on the implementation of these events, see the ModelState struct on the Rust side, mainly the file src-tauri/src/sketchbook/_model_state/_impl_session_state.rs.


Collecting feedback

It'd be good to think about how we incorporate feedback into the open process.
Being open is great - but you should open yourself to criticism, congratulations, and feedback.

Im open to it all :)

😮

#525 for on-site feedback collector
Email address on Next


Provide one-way links

Sometimes it's useful to put something in a guide without linking back up to it, or to put a link into a seealso without the target linking back. Reciprocal links are awesome and one of Mallard's defining features, but sometimes it's nice to block them.
Maybe an attribute that specifies whether the link goes in, out, or both, with both the default?

+1


python3Packages.importlib-metadata: 4.8.1 -> 4.8.2

Motivation for this change
Update to latest upstream release 4.8.2
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)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

really wish they wouldn't deprecate items in a patch release: https://github.com/python/importlib_metadata/blob/main/CHANGES.rst#v481
and 4.8.2 is empty: https://github.com/python/importlib_metadata/blob/main/CHANGES.rst#v482

@cole-h ofborg didn't go off here. Is this any help for you or should I just start it manually?

@ofborg eval
Hm, I'm seeing a common thread with these that aren't starting... I'll have to do some more research, though. Thanks.

https://github.com/NixOS/nixpkgs/pull/146489

1 package built:
azure-cli


Home always active bug

Active styling is now only applied to home when it is active. I also removed the bootstrap style tags and applied custom style classes. These can later be fine tuned to fit dark and light themes.

Couple things keep me from approving/merging this:

Please update this PR to avoid the merge conflict (because I merged-in prior PRs).
When I try to resolve the merge conflict in my local, the NavBar links are all basic hyperlink underlined text instead of clickable (finger pointer) words.

I accepted the changes from both branches and that seems to have resolved it


2.2.1 update quirks

A couple of quirks in the install/update:

When I updated, Mailchimp became an unlisted addon named Akismet. I removed the previous version and installed from scratch.
On /cp/mailchimp/config/edit I got Undefined index: id , which points at MailchimpAudience.php line 31. Setting the Audience ID through the config file fixed this.

A couple of quirks in the install/update:

When I updated, Mailchimp became an unlisted addon named Akismet. I removed the previous version and installed from scratch.

@websmyth remove the existing addon then re-add w/ composer require silentz/mailchimp. I changed the namespace, I'll update the instructions.

On /cp/mailchimp/config/edit I got Undefined index: id , which points at MailchimpAudience.php line 31. Setting the Audience ID through the config file fixed this.

Thanks, will fix

fixed in 2.2.2


v1.10: Branch stability check

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

/test-backport-1.10

/ci-eks-1.10

@kaworu this branch is a bit out-of-date wrt the v1.10 branch now, I was thinking to just close it. If you are still using it, please close whenever you're done. Though it may make more sense to open a fresh similar PR against the tip of this branch and trigger CI from there.


chore: Cleanup of oio::Read, docs, comments, naming

Signed-off-by: Xuanwo<EMAIL_ADDRESS>

Thanks!


fix include_src deprecation

fix WARNING: $include_src is deprecated and will be removed in the next major release, please use $include => { 'src' => false } instead

Yeah, we need to do this, but it would shift the lower bound of the apt requirement to >= 2.0.0 and that is a breaking change I think. It's not technically specified as part of semver about how dependency changes impact versions, though.
What do you think? Also, this PR will need rebasing as it includes two merge commits.

This is also not complete fix of problem:
Warning: Scope(Apt::Source[apt.postgresql.org]): $key_source is deprecated and will be removed in the next major release, please use $key => { 'source' => https://www.postgresql.org/media/keys/ACCC4CF8.asc } instead.
I know it's breaking change for APT dependency, but as many repos are switching to new module version, this should follow as well I think.


FR: Add native Snap Kit integration to Firebase Auth

Feature proposal

Firebase Component: Auth

Describe your use case and/or feature request here.
Auth currently supports many identity solutions with different platforms. Snap Kit (https://kit.snapchat.com/) is a hugely popular platform and it would be great to have native, out-of-the-box support for login via Snap through Firebase.

Thanks for the request, @nhurwitz. We have an open issue already tracking this (https://github.com/firebase/firebase-ios-sdk/issues/3195), so I'm going to close this one in favour of it. Please add a thumbs up reaction to the original issue so we can better track how many people are interested in the feature. Thanks!


 Right now there is a status for each matrix job.

ci is now required check to merge a PR (not including administrators)


postembryonic neuronal lineage paper (FBrf0224164) [sf#28]

@gm119
Reported by gm119 on 2014-03-28 11:20 UTC
I am curating a paper that describes "most postembryonic neuronal lineages in the Drosophila thoracic central nervous system".
FBrf0224164 == Lacin et al., 2014, Development 141(5): 1011--1021

Title:
Transcription factor expression uniquely identifies most postembryonic neuronal lineages in the Drosophila thoracic central nervous system.

PMID: 24550109
They say they have 'constructed a gene expression map that enables the rapid, unambiguous identification of 23 of the 25 postembryonic lineages based on the expression of 15 transcription factors'.
I thought you might want to be alerted to this (!) as hopefully this can be used to make a nice set of terms for postembryonic neurons, plus I need a couple of specific anatomy terms to record phenotype from the paper.
The terms I need are for:
'3il axon bundle'
'3id axon bundle'
'lineage 3 postembryonic neuroblast clone'
I will attach my curation record so you can see where I need the terms.

Updated by mmc46 on 2014-06-02 14:13 UTC

Milestone: FB_2014_03 --> FB_2014_06

Commented by gm119 on 2014-07-29 15:24 UTC
I'm going to submit the main part of my record now, minus the new FBbt terms.  Am happy to make the .edit record to add the FBbt terms once they are made.
This is what needs adding:
FBrf0224164
HGTX[D25]
3il?? | third instar larval stage
HGTX[Scer\UAS.cBa]
3id?? | third instar larval stage { Scer\GAL4[elav-C155] }
unc-4[Scer\UAS.cLa]
lineage 3 postembryonic neuroblast clone?? | third instar larval stage { Scer\GAL4[elav-C155] }

primary and secondary lineages added in #750 (but not hemilineages or clones)

appropriate term for this is probably 'lineage 3 secondary neuron' FBbt:00048918

appropriate term for this is probably 'lineage 3 secondary neuron' FBbt:00048918