Flow caches and fails to purge errors

Flow can sometimes cache errors and requires a killall flow to show that in fact there are no errors.
See closed issue at:
https://github.com/facebook/flow/issues/1901

Just got this funny mistake where the error printer was aware of the update but the error checker wasn't.
12:08 $ yarn run flow
yarn run v0.23.2
$ flow
Error: ../rylodb/src/database.js:518
518:         const promiseUser = this.queryAsync(deleteQuery, [userUuid]);
                                 ^^^ identifier `thi`. Could not resolve name

Found 1 error
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
✘-1 ~/rylo/backend/packages/api [master ↑·3|✚ 6…6]

I guess you need an image to appreciate this one:

Can you give some more information on your environment: flow version, OS, etc.?

Darwin yuvmac.local 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64
15:45 $ flow --version
Warning: `flow --version` is deprecated in favor of `flow version`
Flow, a static type checker for JavaScript, version 0.51.1

I have the same problem with a similar setup:
$ uname -a
Darwin Maximilians-MacBook-Air.local 16.5.0 Darwin Kernel Version 16.5.0: Fri Mar  3 16:52:33 PST 2017; root:xnu-3789.51.2~3/RELEASE_X86_64 x86_64
$ flow version
Flow, a static type checker for JavaScript, version 0.51.0
``

A colleague of mine is facing the same issue. He's using Ubuntu 17.04.
However, he is still having the same problem where some errors from before are showing up.
In his specific case, it's a "module not found" error.

Same problem on Mac OS (10.13.6) using Webstorm 18.1.4.

Ran into this as well on macOS 10.3.3, Flow 0.80. Added // @flow strict, Flow complained about one of the modules not being strict, made that module strict, Flow still complains. Curiously, flow check seems to work whereas flow doesn't.

I'm seeing this on Mac 10.13.6 High Sierra. Running rm -rf ~/.flow-typed clears the cache.

My Flow version is 0.82.0 and I had exactly the same issue as @kangax described. Running rm -rf ~/.flow-typed helped to remove the errors.

A colleague of mine is still facing the issue. He's using Ubuntu 19.04 with Flow version 0.98.1.

Using $ flow caches errors.
Using $ flow check always shows correct/expected/uncached results. But of course it's way slower.
Using $ flow stop and then $ flow shows expected result once and then caches again.

I might have found a solution.
I just realised that my colleague's Linux machine (Ubuntu 18.04 with Flow 0.98.1) had default watch limit for inotify. His PhpStorm was warning us about the limit as we have many files to watch in our project. We increased the limit as IntelliJ docs suggested and voila! After restarting the Flow server both flow and flow check work as expected. I've already increased it months ago on my Linux machine, so I wasn't able to reproduce the problem :smiley:.
Here are the steps we followed:

Add the following line to either /etc/sysctl.conf file or a new *.conf file (e.g. idea.conf) under /etc/sysctl.d/ directory:
fs.inotify.max_user_watches = 524288
Then run this command to apply the change: sudo sysctl -p --system
(If you use your terminal inside your IDE) Restart your IDE.
Stop Flow server: flow stop
Start Flow again: flow (also flow check should work)

:warning: To debug if your files are being watched by Flow server, I recommend you to run it in the foreground: flow server. This way you'll be able to see whether the server "watches" for changes:
flow server
May 20 14:29:39.809 [info] argv=/project/node_modules/flow-bin/flow-linux64-v0.98.1/flow server
May 20 14:29:39.809 [info] Creating a new Flow server
May 20 14:30:12.065 [info] File watcher reported 3 files changed
May 20 14:30:27.078 [info] File watcher reported 6 files changed

I never remember where flow caches to and googling it never turns anything up, so I'm going to leave this here in case it helps anyone else and my future self that forgets...
You can remove the flow cache and re-run flow to purge errors that seem cached/irrelevant.
rm -rf /tmp/flow
yarn flow

I never remember where flow caches to and googling it never turns anything up, so I'm going to leave this here in case it helps anyone else and my future self that forgets...
You can remove the flow cache and re-run flow to purge errors that seem cached/irrelevant.
rm -rf /tmp/flow
yarn flow

Whatever entity you prefer, bless you!

This should be fixed a while ago.


Make the SoS feature gesture based

Maybe some accelerometer based gesture could be implemented to make the sos feature more accessible.

Feature idea turned down in community poll, so this wouldn't be added


Pausing and resuming topics subscription while subscribing topic with RegExp

Is your feature request related to a problem? Please describe.
I am subscribing to multiple Kafka topics with RegExp. But when it comes to pause and resume consumer, I am able to provide the topic as a string only. That's a problem as I have not idea what exactly topics I subscribed to, so how can I list them here? And I didn't found a way to list subscribed Kafka topics, so I couldn't pause them separately.
Describe the solution you'd like
I would like for consumer to:

methods pause and resume being able to take a topic name as RegExp (similar as method subscribe)
or just provide a method to list all subscribed topics, so I could pause & resume them separately
or/and allow methods pause and resume to be called without arguments, so then all topics would be paused or resumed

when I use  subscribing to multiple Kafka topics with RegExp ,I can't get any topic


Add a Gitter chat badge to README.md

ghpabs/angular2-seed now has a Chat Room on Gitter
@ghpabs has just created a chat room. You can visit it here: https://gitter.im/ghpabs/angular2-seed.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Done in #8.


[D000620] Rep. John Delaney

Rep. John Delaney:
http://delaney.house.gov
Likely contact form URL to start with:
http://delaney.house.gov/contact/email-me

YMAL file not found. I am working on it.


Phase2 scanning table5 takes lots of time when parallel plotting with 2 plots and the process always getting disappeared

3990X
256G
P4510 3.7t nvme
Always appear in phase2 when the plotter begin to scan table5

this will not happen when i only do 1 plot once a time, which will reach the result within 19min, but things get weird when i do 2 plots once a time, the plotter will scan table5 in phase2 for a longggggg time (always > 250s). I use top command and find that the process just taking a little bit cpu usage (0%)

must be slow temp 1 drive

When I only run one task at a time, I can obtain an excellent result (1100s one plot). However, when I run two tasks on two different NVME disks, I would get a poor result in phase 2 (the time for scanning table 5, 4, 3, 2 will be extremely long, usually 300s). Nevertheless, when I put these NVME disks on other machines(2680 v4), their performance is normal. The temp 1 drive isn't slow as you suggested, but very fast and good.

Threadripper threads: https://github.com/madMAx43v3r/chia-plotter/issues/626, https://github.com/madMAx43v3r/chia-plotter/issues/513, https://github.com/madMAx43v3r/chia-plotter/issues/351, etc. Perhaps your bottleneck is the bandwidth to the drives. Did you check iotop? Or use nmon and select d or D?


Performance Issue Fine-Tuning D-FINE S with custom dataset

Hi everyone,
I’m fine-tuning the D-FINE S model initialized with Objects365 weights using a custom dataset, but I’m encountering significantly lower performance than expected.
Performance Comparison
In just a single epoch:

D-FINE S achieves<EMAIL_ADDRESS>YOLOv11 S (initialized with COCO weights) achieves<EMAIL_ADDRESS>
1. Dataset Preparation:

Converted custom dataset annotations (x1,y1,x2,y2) to COCO format (x1, y1, w, h).
Since the dataset includes truncated images, I added in the train.py:
ImageFile.LOAD_TRUNCATED_IMAGES = True
This dataset contains only one class, category_id=0, so I set num_classes: 1 and remap_mscoco_category: False.

2. Configuration Changes:

Using dfine_hgnetv2_s_obj2custom.yml as the base configuration.
Modified only the dataset path and adjusted the batch size to 8 for both training and validation.

3. Training Command:
CUDA_VISIBLE_DEVICES=0 torchrun --master_port=7777 --nproc_per_node=1 train.py \
  -c configs/dfine/custom/objects365/dfine_hgnetv2_s_obj2custom.yml \
  --seed=0 -t dfine_s_obj365.pth

What could be causing the poor performance?
I’d appreciate any insights or suggestions for debugging this issue. Thanks in advance!
PS: I tried fine-tuning for 20 epochs, and the mAP@50 only improves slightly to 0.4. Using COCO weights instead of Objects365 weights does not lead to any improvement either.

Can you try training from scratch without any pre-training weights?

Can you try training from scratch without any pre-training weights?

I trained the model from scratch but obtained an mAP@50 of -1.000. I suspect the issue may lie in the annotation format not aligning with the requirements of D-FINE. Could you clarify the correct format? Should the annotations follow COCO format (x1, y1, w, h), YOLO format (x_center, y_center, w, h), or another standard? Additionally, should the values be normalized or not?
Thank you in advance.

Same here - I tried these 2 variants :
dfine_hgnetv2_${model}_obj365.yml and dfine_hgnetv2_${model}_custom.yml
and the mAP50 never goes above 0.35 while even YOLOv9c gets to 0.7

@sergiosanchoasensio Could you please tell me how to use the pretrained model checkpoint file. Please share codes if you are no issues in sharing


Add HTML video player

Summary
Video hosted on Dropbox and Localhost

Test plan

Coverage remained the same at 93.368% when pulling b971301a4847b9a6dbe32f73b976b2a9c473a85a on danactive/html5-video-player into a4360c34df08efeeff7a041589ad3e6a1be3fcb7 on master.

Closes #424


feat: cell height based on font size

dynamically calcualte cell height based on font size (from theme) and row height we get from styling panel
https://user-images.githubusercontent.com/29652890/234535027-0b541fb9-7d3a-47ef-8e09-7cd761f357fe.mov

You should set lineHeight: calc(4/3) on all cells so that they render with the line height we are using for calculating the cell height

Just fixed it
Since we using an extension build for rendering tests, my build was a bit old and based on my previous debugging steps.


What folder to put LLMLoader chkpts?

Im having issues with finding the checkpoints.. ive put them in models/llm .. is that correct or am I missing somtehing?

yes, that is wrong. you need to put them in models/LLavacheckpoints.

thank you


create proper logo animation

 🤜       🤛
  🤜     🤛
   🤜   🤛
    🤜 🤛
     🤜🤛

https://www.fiverr.com/pro_animator93 This guy seems cool
Actually there are not too many results for css logo on fiver :D https://www.fiverr.com/search/gigs?query=css logo

alright, i will request an animation from him.

fyi: hired the pro_animator93 for the job

While we wait for Mr. Fiver, maybe we can monkeypatch some animation with the new scrolly thing: https://developer.mozilla.org/en-US/docs/Web/CSS/animation-timeline

Hmm please have a look, i think as a first rough version this could be a good starting point https://github.com/hendriknielaender/double-trouble/pull/234


Intermittent black screen on initial load

I and other users have noticed that sometimes there is a black screen on activating the lights that is only fixed by invalidating the cache and refreshing.
I need to track down the cause, but I suspect our dependance upon keys coming from cache in OSD or Mirador.  I need to investigate a more reliable way to get these keys potentially.  However, I think first we should try and pinpoint the conditions upone when this happens so we can make sure our implementation will actually fix it.

Caused by a bug in Google Chrome https://issues.chromium.org/issues/40876300 which means content negotiation doesn't work on URLs that are the same making it (the bug) incompatible with IIIF.  Turning off browser caching on info.json responses fixes this, by adding the no-cache header to all info.json responses.


Add optional width/height to Scene.drillPick()

Duplicate of https://github.com/AnalyticalGraphicsInc/cesium/pull/6363.  Spec is modified from the 'picks a primitive with a modified pick search area'.

Works great, thanks @chris-cooper!


Support all tokens from OpenSea API output

For #780.
This PR

Makes the wallet tab display all tokens supported by OpenSea
Adds some special handling for displaying more popular tokens like HyperDragons, MyCryptoHeroes, MLBCB, CryptoSaga
Renames most types and symbols which refer to "cryptokitties" to "opensea non-fungible tokens"
Throttles calls and cache results from OpenSea for wallets that hold many token (IDs) — A JSON "record" is about 4kb so 1500 HyperDragons is 6MB. Throttling starts when the total number of token IDs in a wallet is 25 and when that happens, OpenSea results are cached for 1 minute. Note that 1 OpenSea call is made for each wallet, not for each contract.

It's bit too slow if we are showing many token IDs for a token, e.g. tapping HyperDragons in the wallet tab and waiting for the list of 1500 HyperDragons to show up takes ~4 seconds. Will see if we can improve this in a separate PR. Probably an edge case though.
Looks like we can create a separate tab (alongside ALL, CURRENCY and ASSETS) in wallet called COLLECTIBLES displaying tokens returned by OpenSea (which means ERC721 for now). We might be able to have a nicer display then just a list because we can probably display an icon for each one. What do you think @james-sangalli?
Is transfer supposed to be working for all the tokens on OpenSea? Shall we disable transfer for all of them and only enable for those that we manually verify works @james-sangalli?

Sidenote: I'll work on a PR to change how transactions are fetched. Otherwise the app stalls when testing with wallets that have many collectibles (because they naturally have many transactions).

@hboon looks great!
Transfer is standardised in erc721, if opensea can return the token ids then it shouldn't be an issue support transfer for all the collectables in opensea.


Remove non-blocking builds stability % from SQ

The stability of non-blocking builds is always 100% because they are
never blocking the submit-queue. As this is useless, let's just not
display it.

LGTM


Disable ingress per default

Deploying ingress resource per default doesn't make much sense, since not everyone has an ingress controller deployed. We decided to disable it per default.
Like its done in other charts:
Grafana
https://github.com/grafana/helm-charts/blob/main/charts/grafana/values.yaml#L183-L184
Kibana:
https://github.com/elastic/helm-charts/blob/main/kibana/values.yaml#L142-L143
Elasticsearch
https://github.com/elastic/helm-charts/blob/main/elasticsearch/values.yaml#L250-L251
BTW @menski I saw that the previous full chart deployed an ingress nginx controller https://github.com/camunda-community-hub/camunda-cloud-helm/blob/main/charts/zeebe-full-helm/Chart.yaml#L19-L21 but tbh I'm not sure whether we really want that. Plus it didn't work https://github.com/camunda-community-hub/camunda-cloud-helm/issues/146
related to #124

I approve! Who wants to deploy an nginx bundled with a helm-chart anyway? :D
@Zelldon have you found how I get on the list of reviewers?


General redesign of ImageVisualizer

Still in progress, includes the imzML fix and the spectraPlot fix.
Now uses the datasets and has a ImagingPlot that is reusable and allows to set new datasets like the SpectraPlot

Ready for now. THe visualizer needs more attention later

All done


[5.6] alpha_num validation bug

I don't know if i am the one who's experiencing this kind of bug in Laravel.
Either i used any of these:
   Validator::make($data, [        'password' => 'required|alpha_num',    ]);
or
PasswordRequest.php
public function rules() {     return [        'password' => 'required|alpha_num',     ]; }
The alpha_num validation doesn't work.
Laravel Version: 5.6.*
PHP Version: 7.2

What does "doesn't work" mean? What $data are you validating?

$data represents data from a request

I'm asking about the actual password values. Are there invalid values that pass the validation? Or valid values that get rejected?

the password value pass was 12345678, the other validation are good.

From the documentation:

alpha_num
The field under validation must be entirely alpha-numeric characters.

So of course 12345678 passes that as all characters in the string are numeric.

so what should be the validation?

If you want to validate that the password must have both alphabetic and numeric characters you'll have to write a custom regex for that. btw, as this obviously isn't a Laravel bug please close the issue.


Invalid VNode  error with v-for

Is there anything one needs to be aware of with v-for?
<compiler compiler="vugel">
    <container>
      <text v-for="a in 10" :key="a" :x="110" :y="10">{{a}}</text>
    </container>
</compiler>

I have spent some time trying to get above working. It works without v-for, but with v-for I get the following error:
Invalid VNode type: Symbol(Fragment) (symbol)
  at <TestComp>
  at <Anonymous>

Edit: I guess this is related to custom-block implementation, rather than Vugel itself. With codepen example v-for works fine, but when I try with vite and the custom block implemented here I get above exception.
Custom block is discussed here #2

Yeah, the custom block impl may not work great. If you can reproduce this in codepen there would be an actual bug. v-for etc should just work normally.


Problem with uwsim catkin isolated make

I have a problem when running the command catkin isolated make, in the installation in the uwsim package. The system installs all the packages up to the uwsim correctly, however when it reaches the uwsim the computer freezes and does not recover again. Could someone help me with this problem?

hello，have you sloved this problem?

No, I still have the problem


catchThrowable for expected Exceptions only

so you can differ between test who have unexpected exception (Test Flag is typically Red) and those who throws normal assertionsErrors  (Test Flag is typically Yellow/Orange)

I'm not sure to understand this PR, you have just added a private method that is not used (?)
What is the problem you are trying to solve ?
Could you add a unit test showing how this works ?
Thanks


✨ feat: add support for datacite v4.5

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

Project coverage is 21.34%. Comparing base (89cad84) to head (4a23548).
Report is 2 commits behind head on staging.

Files
Patch %
Lines

pyfairdatatools/generate.py
19.04%
12 Missing and 5 partials :warning:

pyfairdatatools/validate.py
55.88%
8 Missing and 7 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           staging      #13       +/-   ##
============================================
- Coverage    40.30%   21.34%   -18.97%
============================================
  Files            6       12        +6
  Lines          521      970      +449
  Branches       133      202       +69
============================================
- Hits           210      207        -3
- Misses         249      706      +457
+ Partials        62       57        -5

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


Any ue4-docker command not works ,really need some help.

Myt system is Ubuntu 16.04 ,After install pip3 ,i use any docker command that i get the info like this :
nick@NickLinux:~/Downloads$ ue4-docker version
Traceback (most recent call last):
File "/usr/local/bin/ue4-docker", line 7, in
from ue4docker import main
File "/usr/local/lib/python3.5/dist-packages/ue4docker/init.py", line 1, in
from .build import build
File "/usr/local/lib/python3.5/dist-packages/ue4docker/build.py", line 2, in
from .infrastructure import *
File "/usr/local/lib/python3.5/dist-packages/ue4docker/infrastructure/init.py", line 2, in
from .CredentialEndpoint import CredentialEndpoint
File "/usr/local/lib/python3.5/dist-packages/ue4docker/infrastructure/CredentialEndpoint.py", line 1, in
import logging, multiprocessing, os, platform, secrets, time, urllib.parse
ImportError: No module named 'secrets'
i also pinned pip3 -V info :
nick@NickLinux:~/Downloads$ pip3 -V
pip 20.0.2 from /usr/local/lib/python3.5/dist-packages/pip (python 3.5)
Hope someone can give me some advice , thx : )

That's odd that pip actually allowed you to install the ue4-docker package using Python 3.5, since the package's metadata sets Python 3.6 as the minimum required version. Your output lists the pip version as 20.0.2, and pip has supported the python_requires flag since version 9.0.0. Very strange indeed.


[v6][Feature Request] Maybe we eliminate CrudTrait?!

Feature Request
What's the feature you think Backpack should have?
Do we really need the CrudTrait to be added to a Model, in order to work with it? It's one extra step when creating an admin panel... and I don't really like it. It feel like you're bloating up your model. And the stuff it adds... isn't exactly stuff that should be on the Model anyway:
trait CrudTrait
{
    use HasIdentifiableAttribute;
    use HasEnumFields;
    use HasRelationshipFields;
    use HasUploadFields;
    use HasFakeFields;
    use HasTranslatableFields;

    public static function hasCrudTrait()
    {
        return true;
    }
}

It's admin panel stuff, not Model stuff.
Have you already implemented a prototype solution, for your own project?
No, but I've just thought of a way to possibly eliminate it. What I don't know is... how much of a breaking change this would be. But if I'm right... it could even be a non-breaking change... 🤯 Or close to it.
What if when we do
        CRUD::setModel(\App\Models\User::class);

that method doesn't just do
        $this->model = new $model_namespace();

but instead... passed that model... to passthrough Model? So we'd have something like:
<?php

namespace Backpack\CRUD\app\Library\CrudPanel;

class CrudModel
{
    use HasIdentifiableAttribute;
    use HasEnumFields;
    use HasRelationshipFields;
    use HasUploadFields;
    use HasFakeFields;
    use HasTranslatableFields;

    public $model;

    public function __construct(Model $model)
    {
        $this->model = $model;
    }

    public function __call($method, $parameters)
    {
        return $this->model->{$method}(...$parameters);
    }
}

Hmm... then all the methods in those traits would need to call $this->model instead of $this or self or static. So I guess it would be a breaking change... But... maybe it's worth it?
What would the upgrade guide say? Probably:

remove the CrudTrait from your Model, that no longer exists;
if you're intentionally using any of the methods in CrudTrait, instead of $this->methodName() please do (new CrudModel(self))->methodName(); here's a list of all methods in that trait; please note that if you're using uploadFileToDisk() or uploadMultipleFilesToDisk()... we have a better way to do that now, see X;

Do you see this as a core feature or an add-on?
Core but it's a SHOULD/COULD.

@pxpm let me know what you think about this idea, if it has any merit in your eyes... if it's worth doing. Feel like a good change from OUR perspective (maintainers), but a pretty selfish one because it doesn't really bring any benefit to the developer. Well I guess:
PROs:

the model stays 100% clean;

CONs:

much more difficult to override one of the methods provided by CrudTrait (or CrudModel in this case);
breaking change for v5 developers;


Fix false positive results from the ARCHIVE

Also, a bugfix entry in the changelog would be helpful.


On Ubuntu 24.04, clearing the scrollback with tput fails with the bundled/built binary from Github

Here is what I see in my console when the call is made with the regular terminal:
tput: /tmp/_MEI3OZe8M/libtinfo.so.6: version `NCURSES6_TINFO_6.2.20211010' not found (required by tput)

Simply calling tput reset in the same terminal does work for some reason.
That's the result I get from using the binary from this run https://github.com/eth-educators/ethstaker-deposit-cli/actions/runs/10925219471 .
Have you tried this on Ubuntu 24.04 @yorickdowne ? Maybe it's the binary bundle that links to a static lib that isn't present on my system?

The clearing part does work when I call it from my dev environment on the same machine with uv run ethstaker_deposit/deposit.py new-mnemonic for instance. So it seems to be related to building/bundling part of the binaries.

Always something … only tried it directly, not as a binary.

I’m pondering what’s going on - it should check for presence of tput and then just call the system tput directly. Unclear what it’s doing with the ncurses library, and how that relates to the binary.

Do you get the same issue when you try to run the bundled binary? Maybe bundling adds a version of tput or ncurses that causes this issue. I'll try to explore this on my side too.

Ubuntu 20 has libtinfo.so.5
Let’s see what’s in the bundled file. We may need to exclude libtinfo in particular when using pyinstaller, though the docs don’t say how. See https://pyinstaller.org/en/stable/spec-files.html#posix-specific-options on how to exclude everything and include some … we want the opposite, exclude some.

I can reproduce this
If I manually remove libtinfo.so.6 from the extracted dir in /tmp before running ./deposit, the issue doesn't occur

-> We need to figure out how to exclude libtinfo from the binary package

I have a PR with exclusion. That should work on all versions of Debian (11, 12; Ubuntu 20, 22, 24), but we need to check that now excluding libtinfo.so.6 doesn't hinder us in another way.


Matomo issues

There are currently two issues with our Matomo setup (introduced in #419):

We don't have an opt-out button (this is recommended by Matomo for GDPR compliance). I can't get the code for this to work in Docsify (it is included in the privacy page, but commented out for now).
More importantly: page visits are not properly logged. In particular, direct visits inside one browser session (tab/window/whatever) are logged, but any subsequent page visits within that same session are not. This is probably because Docsify does something "smart", capturing the sub-page clicks, changing the address in the address bar and refreshing the page to contain the content you clicked on, but not actually doing any HTTP requests. The page just changes content through client-side Javascript. Apparently (and not surprisingly) this does not jive with Matomo's logging script.

We could investigate how to fix this inside Docsify, but we should not spend too much effort on this (e.g. writing a Docsify Matomo plugin would not be sustainable for us, but maybe there's an easy trick to make Matomo work with Docsify or Docsify's page update system could be slightly tweaked so that it emulates HTTP requests better...).
Also related to #405: with mkdocs I think we wouldn't have any of these problems, so that would also be a solution.

@ewan-escience thoughts? Or @sverhoeven who I think set up the Docsify system originally.


added @package and @subpackage to a blacklist for class annotations

will add an error if @package or @subpackage is used on class level
as recorded in #69

Thanks @wickedOne!


Use interlocked.h for interlocked operations in interlocked_hl

Use interlocked.h for interlocked operations in interlocked_hl
Note: this does not yet use sync.h as that API has to be fixed :-(

#include "windows.h"

can me removed

Refers to: interfaces/reals/real_interlocked_hl.h:9 in b5abe36. [](commit_id = b5abe366d078ad2c658c175c53490158cfcd030a, deletion_comment = False)

SRS_INTERLOCKED_HL_02_018: [ InterlockedHL_SetAndWake shall call WakeByAddressSingle. ]

wake_by_address_single

Refers to: win32/devdoc/interlocked_hl_requirements.md:156 in b5abe36. [](commit_id = b5abe366d078ad2c658c175c53490158cfcd030a, deletion_comment = False)

#include "windows.h"

My pleasure.

In reply to: 703949195 [](ancestors = 703949195)

Refers to: interfaces/reals/real_interlocked_hl.h:9 in b5abe36. [](commit_id = b5abe366d078ad2c658c175c53490158cfcd030a, deletion_comment = False)

SRS_INTERLOCKED_HL_02_018: [ InterlockedHL_SetAndWake shall call WakeByAddressSingle. ]

Will be done as a separate PR.

In reply to: 703949795 [](ancestors = 703949795)

Refers to: win32/devdoc/interlocked_hl_requirements.md:156 in b5abe36. [](commit_id = b5abe366d078ad2c658c175c53490158cfcd030a, deletion_comment = False)


Add support for custom decoder and encoder

Parent: https://github.com/braverhealth/phoenix-socket-dart/pull/59
Adding support to add your own encoder and decoder.
For example:
Using BERT decoder for incoming messages
https://pub.dev/packages/bert
https://github.com/Youimmi/bert.git
import 'package:bert/bert.dart as bert';
import 'package:phoenix_socket/src/message_serializer.dart';

socket = PhoenixRawSocket(
  socketUrl,
  socketOptions: PhoenixSocketOptions(
    serializer: MessageSerializer(decoder: bert.decode),
  ),
);

@matehat please check it

@Pacane any chance to review and merge this ?

Thanks for your contributions @rsmything ! And sorry for the delay ..!


Cilium v.1.0.0-rc8

Change CI to use Coreos for cilium testing. Changed cilium to ver 1.0.0-rc8

ci check this

Instead of removing Ubuntu can we keep both?

@melkosoft It's the gce ubuntu-1604-lts image, which I believe is on kernel version 4.13

manually kicked off both jobs

@melkosoft Looks like Ubuntu succeeded but CoreOS failed when waiting for the cillium pods to be ready

@chadswen What will be the next step? -)

Rerunning CoreOS to check if it's just flaky. If it fails consistently we may need to get more data from the pods.


Question: why is DISTINCT ignored in the query AST

Your SQL parser parses the distinct keyword (when used in the context of a list of columns being selected), but does not retain it in the purposes of the AST.
As an example, take the following test where the query
SELECT DISTINCT username, user_id FROM users

is parsed into the following list of columns, losing the distinct keyword in the process:
[Expr.Ident "username"; Expr.Ident "user_id"]

Is this an artefact of your analyzer not needing the distinct keyword in order to do its type checking? In other words, is this a conscious omission?

Hi @sebastian, so about the parser. It is really a very slimmed down version of SQL and not a full parser. This is because I only use the parser for some very specific cases when the database doesn't give me enough information (such as input parameter nullability).  To circle back to your question:

Is this an artefact of your analyzer not needing the distinct keyword in order to do its type checking?

Yes

In other words, is this a conscious omission?

Yes and no. I just don't have any use case for the distinct keyword because using it doesn't change the static types of the query. However, if you need it for some project you are working on, I would happily accept PRs to improve the parsers and publish it as a library. Though to be honest, it is currently very far from being a full-fledged SQL parser of postgres and I doubt it will be


Dictionary

Description of the new feature / enhancement
Hello contributors
First of all, thank for your dedication.
Enhanced search feature.
Thank you.
Daniel

Scenario when this would be used?
My mother language is not English but, friendly, I need to find some word meaning .
So, I suggest extended search feature.
For example, if I want to know 'shortcut',
type "dictionary shortcut" or "chrome shortcut" and push enter
then, this input call a program with the first argument with the second argument.
It will be great!
Supporting information
No response

Duplicate of #3309

Thanks for your suggestion! This has been raised in #3309 for the dictionary feature, and #3245 for the search feature. To centralize the discussion thread, please go to that thread for discussion.


3d physics support

does it has future features to have 3d physics support? thanks

Does it support DAE files with 3d physics?


enable hadoop web cross-origin and log web service

With this PR, other service could get plain-text job log, at
http://{worker_ip}:8042/ws/v1/node/containers/{container_id}/logs/stdout

and
http://{master_ip}:8188/ws/v1/applicationhistory/containers/{container_id}/logs/stdout?redirected_from_node=true

Coverage increased (+0.01%) to 52.649% when pulling 5b03b70803babccb6d62f4de91854619232f6ef8 on zimiao/cors_test into 6e0a52aeae1d6ca668b0ec85796af5841b464406 on master.

Coverage increased (+0.01%) to 52.649% when pulling 5b03b70803babccb6d62f4de91854619232f6ef8 on zimiao/cors_test into 6e0a52aeae1d6ca668b0ec85796af5841b464406 on master.

Coverage increased (+0.01%) to 52.649% when pulling 5b03b70803babccb6d62f4de91854619232f6ef8 on zimiao/cors_test into 6e0a52aeae1d6ca668b0ec85796af5841b464406 on master.

Coverage increased (+0.01%) to 52.649% when pulling 5b03b70803babccb6d62f4de91854619232f6ef8 on zimiao/cors_test into 6e0a52aeae1d6ca668b0ec85796af5841b464406 on master.


Add Andy's scripts

@boshek @BonnieJRobert I've started this folder with my scripts in it. You can push yours to this branch, and once they're all there we can send them to the participants!

Done!

@boshek @BonnieJRobert is this done now?


ACT-659 Re-implement Indicators using Vue: Create modal

GIVEN I am a logged in user
AND I navigate to the Indicators > Indicator list page
AND I click + Indicators
THEN I should see an create modal that should be using Vue

@chelliah Here's another one that could convert to Vue


Filter enrollments/courses by organization code

Description
OSPR-1438
Added ORGANIZATION_CODE config var and logic to filter displayed courses and enrollments respectively.
Notes

Dependent on mobile API changes in edx-platform PR.

Reviewers

[ ] Code review: @BenjiLee

Thanks for the pull request, @renevatium! I've created OSPR-1438 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

@@             master       #829   diff @@
==========================================
  Files           486        486
  Lines         19871      19874     +3
  Methods           0          0
  Messages          0          0
  Branches       2853       2853
==========================================
- Hits           3772       3762    -10
- Misses        15661      15676    +15
+ Partials        438        436     -2

Review #829 on Codecov

Powered by Codecov. Last update 2c5344f...2ed6539

@renevatium This is also a pretty minor change as well and we may choose to include this. I thought it may have been a bigger change that we may have rejected for inclusion in the main app.  Let me review with the team.

@renevatium In your use case, how does "Find Courses" work?

@BenjiLee we are using native, avoiding webviews as much as possible. Find courses filters already as the org query param was already an option on the course API endpoint.

@renevatium Here is your time to shine! The server side changes have been merged. We've recently made a few folder structure changes so I would recommend merging/refactoring.

@BenjiLee Will fix conflicts today. There is some confusion with the server side changes as they have been reverted and I can't create a new PR for them. Hopefully we can get this sorted out this week.

@renevatium Any plans on making a whitelist of orgs?

@BenjiLee whitelist? As in allow for multiple organizations? Problem there is that the mobile API endpoints don't allow for it. Could update them but with the deprecation coming, maybe that's something to include in the newer courses/enrollment APIs? If you could provide a use case, I can certainly look into it.

@renevatium Just a thought :) No plans from our end yet.

Looks good so far. I just need to spin up an instance with your changes in the server side, so I can manually test it on the app.

@BenjiLee sorry, coding in too many languages today. Fixed the quotes. Should be good now.

@renevatium So some good news and bad news. We haven't really sorted out our APIs in the android repo so it can get pretty confusing. We want to apply the new query parameters here https://github.com/edx/edx-app-android/blob/master/OpenEdXMobile/src/main/java/org/edx/mobile/user/UserService.java#L51
in stead of in the RestApiManager. Sorry for the confusion, we haven't had a chance to properly refactor all the API mayhem :/
The good news, I've verified that the server changes worked.

@renevatium When ORGANIZATION_CODE: '', it is returning no courses.

@BenjiLee damn, sorry about that, blind coded the changes. Dealing with lots of issues today trying to migrate our apps to the new assets structure. Will get that sorted out asap.

@renevatium I kind of jumped on this without the OK to review. Was excited to get this PR through.

@BenjiLee no worries, appreciate it! Will get fix for empty code up in a few minutes.

@BenjiLee damn, I think the problem is actually in platform, check looks for None org but not zero length string value. Can't be fixed on mobile as the Call dependence doesn't allow for Path validation (that I know of). Have to run into a meeting now but will submit PR as soon as I'm done.

I think that if you keep the default code as null instead of an empty string, and pass that as a query parameter to the Retrofit interfaces (e.g. CourseService, UserService, OauthRestApi) as the default value, then it would skip adding the parameter. The Api class is using the old Apache client API though (which I'm soon about to remove), and it constructs the query parameters itself via the toGetUrl() method, so that method would need to have a null check added.

@1zaman good call, defaulting to null in Config seems to have sorted it out. @BenjiLee let me know if there are any other issues. R

@1zaman assumed you meant the toGetUrl Bundle in the Api class getEnrolledCourses method. Have added the check 👍

I actually meant modifying the toGetUrl() method itself generically, so that it checks for null values and ignores them. But I suppose doing it in just this method is good enough too; this whole class is soon about to be nuked anyway.

👍
Good to merge!
Hopefully we can start organizing our APIs a bit better so we can be more open source friendly.

@BenjiLee will get squash done today. Hopefully once the older APIs are deprecated properly, it'll be easier to have all calls in a single place. Will start working on that if I can find time between everything else 😁

@BenjiLee squashed. Hope I didn't break anything, can never seem to rebase smoothly 😆

@renevatium You get used to it :) Then eventually you will become a git ninja.
Also, you need to merge or rebase against master, seems like there are merging conflicts.

@BenjiLee 😫 rebased against origin/master instead of upstream. Fixing now.

@renevatium Ah, that makes sense.  I can't touch your fork.

Maybe you can:
https://github.com/blog/2247-improving-collaboration-with-forks
On Thu, Oct 6, 2016 at 6:37 PM, Christopher Lee<EMAIL_ADDRESS>wrote:

@renevatium https://github.com/renevatium I just remembered that you
don't have merge access >.<
Could you update your commit message and rebase again? I can't touch your
fork.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edx/edx-app-android/pull/829#issuecomment-251963469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDCjTvcJ7a3IAsTi7I07MdTnGOVP0axks5qxPmSgaJpZM4J8kfJ
.

Renamed commit message and started a new PR here since I can't touch your fork. https://github.com/edx/edx-app-android/pull/862


Add link to job details and tooltip to commit status in repo list in dashboard

Tooltip:

Link to the target job:

Ideally this would have the same popover that is visible on repo commit pages, but that will likely require rewriting that one to Vue so it can load on-demand on mouseover.


{AKS} Fix test case test_aks_create_and_update_web_application_routing_dns_zone

This checklist is used to make sure that common guidelines for a pull request are followed.
{AKS} Fix test case test_aks_create_and_update_web_application_routing_dns_zone. Follow-up of PR #6512.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS


Switch back to the official vnu.jar for HTML validation

I might have a look myself when I have some time.
Note that alternatively we could use https://github.com/zrrrzzt/html-validator-cli which shouldn't require Java installed.

No Java dependencies, please :).

Sure, I don't like it either, believe me!
But what we have now is pretty much useless. It's like we don't have a validator. Only the official one can guarantee we are doing things right.

BTW the cli I list above should work, I'll give it a try when I have some time and ping you guys.

But what we have now is pretty much useless. It's like we don't have a validator. Only the official one can guarantee we are doing things right.

Why is it useless?

Well, I shouldn't have said useless, but it's bad. The HTML specs exist for a reason. And vnu.jar is the official tool to do this job. I mean no disrespect for htmllint, but it just doesn't make sense to me to not use the official tool which follows the specs since they make the specs.

Anyway, I had a look at html-validator-cli. It works fine, except it doesn't accept glob patterns. So we need a node script helper to do this job.

html-proofer might be worth a look, as suggested by Jekyll's continuous integration advice.

That isn't the official validator either. html-validator-cli is as close as
we can get to the official validator and without Java. We just need a
custom wrapper since it doesn't accept wildcards.
On Sun, Jul 30, 2017 at 1:03 PM, Bardi Harborow<EMAIL_ADDRESS>wrote:

html-proofer https://github.com/gjtorikian/html-proofer might be worth
a look, as suggested by Jekyll's continuous integration advice
https://jekyllrb.com/docs/continuous-integration/travis-ci/.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/23171#issuecomment-318891231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtWBygjy-uB5jQduTewhlB9v-J6x3ks5sTFUDgaJpZM4OmhNs
.

I see html-validator-cli also has the ability to ignore specific warnings which is very useful e.g. when using some workarounds for certain Mozilla Firefox bugs and oddities.

One example of broken HTML not caught by the current solution https://github.com/toshimaru/jekyll-toc/issues/25

Another one... https://github.com/twbs/bootstrap/pull/23184

The toc issue was a problem with me having two toc generators on that page.

Yup, still the current solution didn't warn us, did it? :P

@mdo: here are the remaining issues minus the ones we'd ignore anyway about img missing src.
https://gist.github.com/XhmikosR/9e417efb51b3b7b86c8502c8bb36b7a0
Until we have a proper HTML validation solution again.


Logout and session improvements

This PR introduces code for server side logout. It also improves the session code by only encoding the session id within access tokens to avoid leaking any user information. Also the resume session code is now handled by the refreshTokens method since these are essentially the same thing.

These changes will come in as part of the Lerna PR.


Add IgnoreFieldsExcept

For larger structs and when the API is potentially evolving it would be handy to have the inverse of IgnoreFields, i.e. cmpopts.FilterFields(typ interface{}, names ...string) cmp.Option.
And for symmetry I guess there should also be FilterInterfaces and FilterTypes.
An alternative naming could be Include… if this clashes to much with the root packages concept of filters.

It looks like filterField that private and unused now could be a part of solution of this problem with something like IncludeFilter.

I could really use something like this or #53. I have a struct with a bunch of fields and I only want to compare, say, 3-5 of them.
I got something that works for one simple use case by doing this:
func cmpFields(fields ...string) cmp.Option {
	filter := func(p cmp.Path) bool {
		s := p.String()
		if s == "" {
			return false
		}
		for _, f := range fields {
			if s == f {
				return false
			}
		}
		return true
	}
	return cmp.FilterPath(filter, cmp.Ignore())
}

but I'm guessing that there are lots of corner cases that this doesn't handle judging by how much more complicated the cmpopts.IgnoreFields code is. (I haven't had a chance to study it closely yet.)

Suppose we had IgnoreFieldsExcept, what is the user expectation when they see something like the following?
cmp.Diff(x, y, cmp.Options{
    cmpopts.IgnoreFieldsExcept(MyStruct{}, "FooField"),
    cmpopts.IgnoreFieldsExcept(MyStruct{}, "BarField"),
})

I may be mistaken, but I'm suspect many users will expect that either MyStruct.FooField or MyStruct.BarField will be compared, and that everything else is ignored. However, what actually happens is that no field ends up being compared. Will such a behavior be surprising to users?

How about Include* rather than Ignore*Except and make the default AND not OR to match Ignore* if that's how IgnoreFields functions currently? That's what I'd expect (granted I've never tried to pass multiple IgnoreFields).
// Both FooField and BarField are ignored.
cmp.Diff(x, y, cmp.Options{
    cmpopts.IgnoreFields(MyStruct{}, "FooField"),
    cmpopts.IgnoreFields(MyStruct{}, "BarField"),
})

// Only FooField and BarField are included.
cmp.Diff(x, y, cmp.Options{
    cmpopts.IncludeFields(MyStruct{}, "FooField"),
    cmpopts.IncludeFields(MyStruct{}, "BarField"),
})

I've been following this ticket for a while but I think my use case at the time was I wanted an easy way to diff only fields given by a field mask.

I'm not sure how that'd be implemented. The options framework of cmp assumes that everything is included except for the things that are explicitly ignored. The implementation of cmpopts.IncludeFields would still need to be implemented under the hood in terms of cmp.Ignore options. We could introduce another primitive option called cmp.Include, but that would open yet another can of worms, such as what happens when both a cmp.Ignore and a cmp.Include is applicable for the same node.

@dsnet

I may be mistaken, but I suspect many users will expect that either MyStruct.FooField or MyStruct.BarField will be compared, and that everything else is ignored. However, what actually happens is that no field ends up being compared (since the intersection of two disjoint sets is the null set). Will such a behavior be surprising to users?

Personally the behavior is what I'd expect, though perhaps others would be confused.
I can see what you're angling at in wanting options to be coherently composable, but in practice I want to compare a limited subset of struct fields pretty often and I've never once composed multiple ignore options. I think there's a case for being pragmatic here.
Perhaps a note warning the reader about stacking ignore options together on the doc comment for IgnoreFieldsExcept would be sufficient.
Or, consider exporting more primitives in cmpopts so I can build this (and #53) myself outside of cmp/cmpopts without copy-pasting large swaths of struct_filter.go.

\cc @neild, do you have an opinion about whether to add IgnoreFieldsExcept?
I personally have also needed something like IgnoreFieldsExcept a few times. My fears about users being surprised by composability of this option may be entirely unfounded.

Or, consider exporting more primitives in cmpopts so I can build this (and #53) myself outside of cmp/cmpopts without copy-pasting large swaths of struct_filter.go.

I've been wanting to export exportField for a long time, but the composability questions around that are even more complex than simply the composability of IgnoreFieldsExcept. @rogpeppe and I had a relatively long discussion on Slack about it. I don't think we came to any clear consensus on what to do.

I've been wanting to export exportField for a long time,

(Do you mean filterField?)

Yep, fixed.

I don't think the composability of IgnoreFieldsExcept is particularly surprising: It isn't hard to understand that the fields which remain are the union of all fields that were not ignored.

I am interested in this feature too, would be really helpful in comparing only the required fields. Any timelines when this might be expected to be taken up?

Any update?

I am interested in this feature too.

Check this out.
func IgnoreAllUnexported() cmp.Option {
    return cmp.FilterPath(func(p cmp.Path) bool {
        sf, ok := p.Index(-1).(cmp.StructField)
        return ok && unicode.IsLower(rune(sf.Name()[0]))
    }, cmp.Ignore())
}


Revert "Keep UTC time zone attached to datetime objects when API retu…

@badrpc FYI - have to revert this so we can safely release a fix for the user-agent
This reverts commit 02c9c844899247d24fb5a85780d80df4edd8a629.
Throws the following exception on Enyaq for Webspider:
Traceback (most recent call last):
  File "/home/nivo/dev/skodaconnect/example/me.py", line 358, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/home/nivo/dev/skodaconnect/example/me.py", line 247, in main
    print(f'\tstr_state: {instrument.str_state} - state: {instrument.state}')
  File "/home/nivo/dev/skodaconnect/skodaconnect/dashboard.py", line 123, in str_state
    return f'{self.state}'
  File "/home/nivo/dev/skodaconnect/skodaconnect/dashboard.py", line 127, in state
    val = super().state
  File "/home/nivo/dev/skodaconnect/skodaconnect/dashboard.py", line 58, in state
    if hasattr(self.vehicle, self.attr):
  File "/home/nivo/dev/skodaconnect/skodaconnect/vehicle.py", line 1542, in last_connected
    last_connected = datetime.fromisoformat(last_connected_utc)
ValueError: Invalid isoformat string: '2023-12-11T15:34:24.425Z'

Oh, I'm really sorry for the troubles.
However I must say that I have asked about this on the original pull request:

Important: this will only work on Python 3.11 and higher because prior to 3.11 this method only supported formats that could be emitted by date.isoformat() or datetime.isoformat(). In Python 3.9 for example this code will fail to parse the timestamp returned by the API. I can replace this with datetime.strptime(last_connected_utc,'%Y-%m-%dT%H:%M:%SZ').astimezone(timezone.utc) which works in older versions. I'm not sure what is the expectation of Python version for this library.

I'm quite certain that problem is not with a model but with python version:
Python 3.9:
% python3.9
Python 3.9.14 (main, Sep 20 2022, 01:16:05)
[Clang 13.0.0 (git@github.com:llvm/llvm-project.git llvmorg-13.0.0-0-gd7b669b3a on freebsd13
Type "help", "copyright", "credits" or "license" for more information.
>>> from datetime import datetime
>>> datetime.fromisoformat("2023-12-11T15:34:24.425Z")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Invalid isoformat string: '2023-12-11T15:34:24.425Z'
>>>

Python 3.11:
% python3
Python 3.11.5 (main, Aug 29 2023, 15:31:31) [GCC 13.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from datetime import datetime
>>> datetime.fromisoformat("2023-12-11T15:34:24.425Z")
datetime.datetime(2023, 12, 11, 15, 34, 24, 425000, tzinfo=datetime.timezone.utc)
>>>

Shall I implement this in a way that's compatible with Python before 3.11?

@WebSpider did you get an error on a Python 3.9 env?
If that's the case and it all works fine on 3.11 then I think you can just resubmit the exact same fix. HA Core depends on 3.11 now anyway (https://github.com/home-assistant/core/blob/dev/pyproject.toml#L24). Sorry we weren't quite sure yesterday and had to get a fix out for the updated user-agent.

I will double check, I don't know what my python version in the venv was at that point.

No worries, I can totally understand it.
FYI: 3.10 is also expected to have the same error - just checked. And according to https://docs.python.org/3/library/datetime.html#datetime.datetime.fromisoformat 3.11 is the first version which support the required format.

Yep. It was python version related. The exception is solved when using 3.11.
However, the date conversion does not seem correct to me:
DEBUG:skodaconnect.connection:Request for "https://api.connect.skoda-auto.cz/api/v2/vehicle-status/MYVIN" returned with status code [200], response: {'remote': {'capturedAt': '2023-12-12T09:22:05.583Z', 'status': {'open': 'NO', 'locked': 'YES'}, 'mileageInKm': 60000.1, 'doors': [{'name': 'BONNET', 'status': 'CLOSED'}, {'name': 'TRUNK', 'status': 'LOCKED'}, {'name': 'REAR_RIGHT', 'status': 'LOCKED'}, {'name': 'REAR_LEFT', 'status': 'LOCKED'}, {'name': 'FRONT_RIGHT', 'status': 'LOCKED'}, {'name': 'FRONT_LEFT', 'status': 'LOCKED'}], 'windows': [{'name': 'SUN_ROOF', 'status': 'CLOSED'}, {'name': 'ROOF_COVER', 'status': 'UNSUPPORTED'}, {'name': 'SUN_ROOF_REAR', 'status': 'UNSUPPORTED'}, {'name': 'FRONT_LEFT', 'status': 'CLOSED'}, {'name': 'FRONT_RIGHT', 'status': 'CLOSED'}, {'name': 'REAR_LEFT', 'status': 'CLOSED'}, {'name': 'REAR_RIGHT', 'status': 'CLOSED'}], 'lights': {'overallStatus': 'OFF', 'lightsStatus': [{'name': 'RIGHT', 'status': 'OFF'}, {'name': 'LEFT', 'status': 'OFF'}]}}, 'local': None, 'errors': []}

Results in the dashboard in:
MYVIN Request results - (request_results)
        str_state: N/A - state: N/A
        attributes: {'latest': 'N/A', 'state': 'N/A', 'departuretimer': 'N/A', 'departuretimer_timestamp': '1970-01-01T00:00:00', 'batterycharge': 'N/A', 'batterycharge_timestamp': '1970-01-01T00:00:00', 'air-conditioning': 'N/A', 'air-conditioning_timestamp': '1970-01-01T00:00:00', 'refresh': 'N/A', 'refresh_timestamp': '1970-01-01T00:00:00', 'lock': 'N/A', 'lock_timestamp': '1970-01-01T00:00:00'}

MYVIN Door locked - (door_locked)
        str_state: Locked - state: True
        attributes: {'last_result': 'N/A', 'last_timestamp': '1970-01-01T00:00:00'}

So the time for some reason is set to the beginning of the epoch, and not to the value of capturedAt?

On the surface this looks completely unrelated to my change. I only updated two specific sensors: last_connected and parking_time - these should be visible in Home Assistant as sensors, not as attributes of entities. I don't know where this attributes are coming from, is this not even something maintained internally by Home Assistant? I only have these attributes on a few entities and N/A in latest or last_result attributes makes me think that maybe integration has never provided values for these entities?

I looked a bit more into these "N/A" and "1970-01-01T00:00:00". At least some of them seem to be related to actions initiated from Home Assistant. For example I have just triggered "Force data refresh" and now for request_results I have
latest: Refresh
state: Success
batterycharge: N/A
batterycharge_timestamp: 1970-01-01T00:00:00
refresh: Success
refresh_timestamp: 2023-12-13T09:56:12
lock: N/A
lock_timestamp: 1970-01-01T00:00:00
model: Škoda Scala Ambition/2019-11-12
assumed_state: true
icon: mdi:chat-alert
friendly_name: My Skoda Request results

Before triggering "Force data refresh" I had "N/A" in latest and refresh and "1970-01-01T00:00:00" in refresh_timestamp. I'm not sure how exactly this works, but I don't think this is in any way related to last_connected and parking_time sensors I was changing in https://github.com/skodaconnect/skodaconnect/commit/02c9c844899247d24fb5a85780d80df4edd8a629.
I will prepare another PR identical to https://github.com/skodaconnect/skodaconnect/commit/02c9c844899247d24fb5a85780d80df4edd8a629 later today.


Retain filtered committee when searching from Q & R page

@longhotsummer

That's it? Does it work? :) Pleasantly surprised.

Unless I'm missing something. I was quite surprised too.

As long as it works.


Add StringViewArray and BinaryViewArray (#4253)

Draft as not yet standardised and needs a LOT more testing
Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Related mailing list discussion: https://lists.apache.org/thread/w88tpz76ox8h3rxkjl4so6rg3f1rv7wt

See https://github.com/apache/arrow-rs/issues/5374 for implementation discussions

Thanks for the comments @sundy-li . Is there any chance you or someone at DataBend are interested in / planning on working on this feature?

Yes, we are working on it at https://github.com/datafuselabs/databend/pull/14662
In databend, our memory array is still based on arrow2. But reading/writing is using arrow-rs, we are planing to work on it after  https://github.com/datafuselabs/databend/pull/14662 is finished.
@ariesdevil

Hi @alamb, I'm willing to work on this feature after https://github.com/datafuselabs/databend/pull/14662 is finished.

Hi @alamb, I'm willing to work on this feature after datafuselabs/databend#14662 is finished.

Thank you @sundy-li  and @ariesdevil
I left a comment on https://github.com/apache/arrow-rs/issues/5374 https://github.com/apache/arrow-rs/issues/5374#issuecomment-1968772760 about how we can potentially work on this together

I'm going to close this PR as I believe others are picking this up

Add StringViewArray implementation and layout and basic construction + tests #5469

Indeed -- we are tracking the work in https://github.com/apache/arrow-rs/issues/5374


Face Liveness Detection System Using Profile and Blink Detection

Pull Request Title:
Add Face Liveness Detection System Using Profile and Blink Detection
Issue Number - #904
Description:
This pull request implements a Face Liveness Detection System that leverages profile detection and blink detection techniques. The goal is to verify whether a user is physically present during the authentication process by analyzing their facial features and eye blinks.
Changes Made:

Implemented profile detection using Haar Cascade classifiers for frontal and profile face detection.
Integrated blink detection using Dlib's shape predictor to calculate the Eye Aspect Ratio (EAR).
Developed a user interface that prompts users to turn their faces and blink.
Added a README file explaining the methodology, how to run the main file, and project structure.
Included a requirements.txt file with necessary dependencies.

Methodology:

Profile Detection: Utilizes Haar Cascade classifiers to detect frontal and profile faces.
Blink Detection: Employs Dlib's shape predictor to monitor eye blinks based on the EAR threshold.
User Interaction: Prompts the user for specific actions (turning and blinking) to confirm liveness.

How to Test:

Ensure the necessary XML and DAT files are placed in the dataset/ directory.
Run the main.py file to initiate the detection process.
Follow the prompts on the screen for liveness detection.

Hello @abhisheks008 please mege my PR


PR: Close websocket if process was stopped on Windows

Fixes #90
It also updates pywinpty dependency to version 0.1.2

@ccordoba12 Is there any way to prevent blocked tests on AppVeyor? Sometimes this one gets stuck spyder_terminal/tests/test_terminal.py::test_new_terminal

Nop, I was looking for a way too, but found none. At least Travis and Circle only run for 10 mins and if they don't see any output, they terminate the process.
But Appveyor runs for 1h!

Maybe I can reduce the number of test environments required on Appveyor's matrix?

Yeah, I think we could test only on Windows 64 to reduce the chances of Appveyor being stalled in a test.


Make JS module, that will handle the UL events

As stated in https://github.com/nordnet/cordova-universal-links-plugin/issues/3 - we need a module, that does something like this:
function onDeviceReady() {
  console.log('deviceready');

  // set handler for the event
  universalLinks.on('event_name', onULOpen);
};

function onULOpen(event) {
  // handle the event
}

Internally, it will store the events, received from the native side; and then universalLinks.on(...) is called - send them to the callback. Basically, you are subscribing to the event through the plugins module.

This would definately be very useful! I am now working around the timing issue by setting event details in localstorage from index.html and then acting on it when the app has finished loading.


Visibility problem

I was using version 3.2.0 and never had problems, now with 3.4.0 I experience following problem:
Sometimes I have to scale the image with my fingers before it's shown... onReady is called, but the screen stays empty until I do a scale gesture, this makes the image visible
The other thing is, that in my ViewPager using a FragmentStatePagerAdapter the views to the left/right sometimes to load the images as well until I swipe to them. The fragments are created but onReady of the image does not get called. Touching the current page does help and results in onReady for the left and right image to get called...

I can't reproduce this. There's a possibility that release 3.4.1 has fixed it, please try that and let me know if you're still seeing this problem. Please also include the code you're using to set up the view.

I can post the adapter if you want, but I think I've found something:

I'm using activity transitions, if I disable them, it works fine... onReady/onImageLoaded seems to work always now as I can see, just the image stays empty until i swipe a bit on it...
it happens more often with big images (4000x4000 and upwards)

Any ideas on that?
I want to change this anyway to use a preview image on top of your view for the transitions. This may solve this problem as the transitions won't mess around with your views visibility...

I'm also facing the problem with the ViewPager, the image of a fragment is empty until a bit of swiping is done to that fragment.
I have tried with a big offScreenPageLimit, but it still happens. Fragments already cached (when going back) doesn't have this issue.
I thought it was related to #117, but I'm not using previews. I've also tried setting image dimensions.
Disabling tiling fixes it, but I would like to keep it enabled. The images used are around ~1000x1600

I still can't reproduce anything like this, with or without custom page transitions, and with small or large images. With tiling enabled, it's normal for the image to not be displayed immediately when you start the swipe to the next page, because it has to load. It can't be loaded until the view's dimensions are known, and the dimensions may not be known until the view is partly on screen.
I have never seen the view get to calling onReady without displaying the image, or need extra touch gestures to prompt the image to be displayed.

Actually, the second thing is not reproducable anymore... So I would say that was something different. Probably something with transitions, I'm not sure anymore.
If the other thing can't be solved currently, I would suggest something like following: a constructor for the image that takes a size as well, so the image could be loaded before the view is rendered? Or something similar

It's the view size that's needed, not the image size. I'm considering an option to override the default handling and assume the view will be full screen then subsample the image accordingly, but misuse of this flag would cause crashes and it adds yet another loading path to the many already present.

Of course, the view size... But in a gallery app for example, the view size normally is full screen and therefore is already known, so this would probably be quite useful for many people

With these changes I was able to load the initial tile before a fragment is partially visible. The view dimensions of non visible fragments from the viewpager are known beforehand (getWidth() and getHeight() return the view size), but the initial tile isn't loaded until the first onDraw call.
The problem is that we lose access to the canvas when initializing outside onDraw, so getMaximumBitmapWidth and getMaximumBitmapHeight can't be used.
That's the reason I added the setMaxDimensions method. Then in the onCreate of the activity I stored the value obtained from this method (the activity is declared hardware accelerated in the manifest) and assign that value to every image view with setMaxDimensions.
In my tablet and phone the method returns 4096 instead of the 16384 returned by the canvas method, but I haven't tried with bigger images so I don't know if there will be any issue.
I'm also not sure if this causes any other problem in the library, but it works fine in my case.
Are these changes safe enough @davemorrissey ?

I've implemented @inorichi's suggestion and it works well. The next image in a view pager is loaded while off screen.
I'm not certain whether this was the cause of the original issue, which I never reproduced, but will close this now. Please reopen if there are other cases where the image is not loaded.


howler.js sprite demo not working

howler.js sprite demo does not seem to be working.
OS: MacOS Mojave.
Browser: Chrome Version 74.0.3729.108 (Official Build) (64-bit)

Thanks for noticing that. Fixed.


fix: Allow reconciliation of resource overrides

Signed-off-by: jannfis<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What does this PR do / why we need it:
Perform reconciliation of deployments and statefulsets when resource requests or limits have changed.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

Create ArgoCD CR
Set resource specifications on controller, repo or server components
Before this fix: Neither application controller, repo server or server objects will get reconciled (keep old resource spec)
After this fix: All three workloads should get updated resource spec from ArgoCD CR

We also need to fix keycloak workloads.  See keycloak.go  (We should probably move some of the deployment stuff to deployment.go).  It looks like resource request/limits are not read from CR.

We also need to fix keycloak workloads. See keycloak.go (We should probably move some of the deployment stuff to deployment.go). It looks like resource request/limits are not read from CR.

@wtam2018 Should we consider this change when we move out of OpenShift Templates and use the kubernetes manifests for Keycloak Installation ? I believe that is the correct time to make this fix.
P.S:
I have seen Keycloak does not come up with with anything less than the current request/limits

I don't think we can wait.   Users may need to tweak these settings if sso enabled.

We also need to fix keycloak workloads

Yes, I agree. But this should come with another PR imho.

I have seen Keycloak does not come up with with anything less than the current request/limits

Valid concern, since Keycloak is a Java application which probably sets minimum JVM heap size as well and most likely is pretty hungry for RAM. Anyway, I agree with William that we should let users override the resource limits. I can think of users wanting to tweak it for CPU being available at scheduling time.


do not wrap white space

Description
This PR resolves 2 minor issues

Simple css fix to not wrap white spaces in the label in sequence viewer
Renaming Expression tab to Regulation in GB

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-1606
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-1585
Deployment URL(s)
http://white-space-wrap.review.ensembl.org/
Views affected
Sequence viewer in GB drawer

@jyothishnt there is one more element that needs wrapping in the drawer, its the the "Blast whole sequence button", sorry this was added to the ticket 1569 description in the morning. This also mean that you will have to wait for the other ticket that Andrey is working on to be merged and then do this.
Or another way to do it is create a separate ticket for wrapping the blast whole sequence.

@jyothishnt there is one more element that needs wrapping in the drawer, its the the "Blast whole sequence button", sorry this was added to the ticket 1569 description in the morning. This also mean that you will have to wait for the other ticket that Andrey is working on to be merged and then do this.
Or another way to do it is create a separate ticket for wrapping the blast whole sequence.

This will be done in #748


New register flow

#217

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jbesraa:x: nadavmorenonadavmoreno seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@florpor nice1!


feat: check savedSearch query field

The type of this PR is: FEATURE

This PR resolves FX-2967
Description
As an admin with the saved search feature flag enabled
When I apply filters on the artist artwork screen
A query is executed that checks if I have a saved search with those filters
Acceptance criteria:

Every time a user with the saved search flag applies filters, a query is executed that checks if that user has an existing saved search with those filters
End result of this ticket could just be logging out the result of that query

Demo
https://user-images.githubusercontent.com/3513494/121049697-9e650180-c7c0-11eb-9c11-e93e46673b2e.mp4
https://user-images.githubusercontent.com/3513494/121050503-4c70ab80-c7c1-11eb-8feb-832316e97550.mp4

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

We should eventually add specs that exercise both GraphQL mutations

@dblandin If it is not difficult, can you describe in more detail what is meant by this?

Sure! I'm thinking of something like this which sets up some mock mutation data, does something in the UI (like tap the enable/disable) button, and asserts on some kind of response like a toast appearing or the banner switching to the other state.
There should be some examples in Eigen but here's one example in Force that I'm familiar with: https://github.com/artsy/force/blob/8aa26fc8215cbb679ba7d14f81d25d23fc6337e9/src/v2/Components/UserSettings/TwoFactorAuthentication/tests/TwoFactorAuthentication.jest.tsx#L124

@dblandin

createSavedSearch mutation spec
deleteSavedSearch mutation spec


[Narwhal] rename Core to Certifier

Description
The functionality of Core is more restricted now, only covering the Header to Certificate logic. So renaming Core to Certifier to better represent its purpose.
Test Plan
Existing unit tests.
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

that seems like a good idea


Slice Testing with just the Autoconfigurations related to stubrunner loaded in

As a consumer of a service with stubs published, it would be great to load up only the spring configuration and components related to stub runner without needing to load up the entire application context. For a complex app with its own set of auto-configurations and with external database dependencies, loading up an entire context is difficult.
Just to provide a little more context, a consumer test looks like this:
@RunWith(SpringRunner::class)
@SpringBootTest
@AutoConfigureStubRunner(ids = arrayOf("samples.contract:sample-producer:+:stubs"), workOffline = true)
class PassThroughContractTest {

The @SpringBootTest annotation causes the entire boot application context along with any auto-configurations to be loaded up which is costly for na app with likely external dependencies.
Junit rule works well as a substitute. It would be great to have a more Spring Boot native solution also.
Here is the project where I was trying it out - https://github.com/bijukunjummen/s-c-contract-sample

I will try and send you a PR @marcingrzejszczak

Done via https://github.com/spring-cloud/spring-cloud-contract/pull/496


Address RSpec deprecation warnings

Deprecation Warnings:

Use of rspec-core's `its` method is deprecated. Use the rspec-its gem instead. Called from /Users/temikus/Code/vagrant-dev/vagrant-google/test/unit/common/config_test.rb:33:in `block (2 levels) in <top (required)>'.
Use of rspec-core's `its` method is deprecated. Use the rspec-its gem instead. Called from /Users/temikus/Code/vagrant-dev/vagrant-google/test/unit/common/config_test.rb:34:in `block (2 levels) in <top (required)>'.
Use of rspec-core's `its` method is deprecated. Use the rspec-its gem instead. Called from /Users/temikus/Code/vagrant-dev/vagrant-google/test/unit/common/config_test.rb:35:in `block (2 levels) in <top (required)>'.
Too many uses of deprecated 'Use of rspec-core's `its` method'. Pass `--deprecation-out` or set `config.deprecation_stream` to a file for full output.

`be_false` is deprecated. Use `be_falsey` (for Ruby's conditional semantics) or `be false` (for exact `== false` equality) instead. Called from /Users/temikus/Code/vagrant-dev/vagrant-google/test/unit/common/config_test.rb:45:in `block (3 levels) in <top (required)>'.

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
`config.raise_errors_for_deprecations!`, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

Adressed in #178


ApprovalNotification INPUT fc-port requires a change

Please find the attached 221213 ApprovalNotification_INPUTChange.docx that describes the scenario with example.
Proposal :
To change the INPUT of the forwarding-construct “ApprovalNotification” from
ro-2-0-0-op-s-is-003 (/v1/regard-updated-approval-status)
to
ro-2-0-0-op-s-is-010 (/v1/relay-server-replacement)
Note :
This problem is observed for the applications that subscribed for “notify-approval” (lets say for the TAC applications except TAR)
Problem :
Let’s take RO and EATL. RO is upgrading from version 1 to version 2.
Step 1 : EATL application has version 1.0.0 of RO
Step 2 : EATL receives request /v1/regard-application for “version 2.0.0” of RO and creates new LTP instance group
Step 3 : RO version 1.0.0 sends “v1/register-application” to RO version 2.0.0. So , RO version 2.0.0 sends /v1/embed-yourself to EATL which creates few more operation-clients and modifies the corresponding forwarding-constructs too.
Step 4 : RO version 1.0.0 sends “relay-server-replacement”. So , EATL will receive “v1/update-client” which updates the version 1.0.0 of RO to version 2.0.0.
Problem :
1.	Two instances of the same application and version is now available in the CONFIGFile of EATL.
2.	ForwardingConstruct mappings are scattered across both the application instances.
Step 5 : deregister application version 1.0.0 sends disregard-application for version 1.0.0. But version 1 will not be available , so no deletion happens.
Solution :
This problem will occur in applications that subscribed for “notify-approval”. This problem shall be resolved by notifying the new application approval after the “relay-server-replacement” procedure.
Currently , after receiving an “approval” from “TAR” , RO sends “regard-application” to all the applications that subscribed for notification.
Changing the INPUT of the forwarding-construct “ApprovalNotification”
from
ro-2-0-0-op-s-is-003 (/v1/regard-updated-approval-status)
to
ro-2-0-0-op-s-is-010 (/v1/relay-server-replacement)
Details as follow ,
Step 1 : EATL application has version 1.0.0 of RO
Step 2 : RO version 1.0.0 sends “v1/register-application” to RO version 2.0.0. So , RO version 2.0.0 sends /v1/embed-yourself to EATL which creates few more operation-clients and modifies the corresponding forwarding-constructs too.
Step 3 : RO version 1.0.0 sends “relay-server-replacement”. So , EATL will receive “v1/update-client” which updates the version 1.0.0 of RO to version 2.0.0.
Step 4 : RO sends "regard-application". Already version updated. But forwardings will be triggered as the service is idempotent.
Step 5 : If any operation gets upgraded,  new version sends  “relay-operation-update” which triggers “update-operation-client” to update the operation-name.
Step 6 : deregister application version 1.0.0 sends disregard-application for version 1.0.0. But version 1 will not be available , so no deletion happens.

An excessive analysis of the problem has been made in a comment in ALT issue#158.
To double check, whether problem is solved with /v1/update-client not to take effect, if application subscribed at RO://v1/notify-approvals.

First of all thanks for going through the high level steps and understanding the problem.
Also thanks for the descriptive details of the /v1/update-client logic expectation.
From the table , could able to understand that

If already a http-client for a (application-name + new-release-number) is available ,  then
no need to perform any http-client update operation and only the linked tcp-client shall be updated.

This will solve the problem of having two instance of the same application-name + release-number in the CONFIG file.
In this new proposal , we have to resolve one more problem(in a special scenario) to make the solution complete. Kindly please find the below details,
So , after “update-client” , both the instance “http-client_1 with old release” and “http-client_2 with new release” will be available in the CONFIGfile. (The “http-client_1” will be deleted as a part of the “disregard-application”.)
Problem :
Similar to the https://github.com/openBackhaul/OperationKeyManagement/issues/141 issue , in the TAC applications EATL, OL , ALT , OKM  there are 3 operation-clients of RO which are not created based on a subscription.
Please view this issue https://github.com/openBackhaul/ExecutionAndTraceLog/issues/155 for details.
In case , if an operation-client is not created from a subscription and is available in the CONFIGfile as a part of the initial deployment ,
then ,
we will lose these operation-clients once it is disregarded.
For example :
in EATL’s CONFIGfile ,
while upgrading RegistryOffice to a newer version ,
RO's old version will have 8 operation-clients ,
whereas the new version will have only 4 operation-clients.
When we disregard version 1 ,
their corresponding operation-clients will also be lost. (highlighted in yellow in the below table)

Proposal :
Proposal#1 :
One proposal to solve this problem shall be adding individual subscription operations for creating operation-clients and fc-ports for this operations.
Proposal#2 :
In the server side , we can introduce a small logic to avoid losing such operation-clients ,
While receiving “update-client” , (the below flow chart only concentrates on the operation-client part)

It seems we have the same discussion in two issues.
Please find my comment in EATL issue#155

Issue will be solved by adding a description to /v1/update-client in the ApplicationPattern (AP issue#467).


Pin knative.dev/{pkg,networking,test-infra} to release-0.16

/assign @ZhiminXiang

/test pull-knative-net-certmanager-unit-tests

@ZhiminXiang : I think the unit test breakage is legitimate. Can you please help take a look?  thanks

This unit test is fixed by https://github.com/knative/net-certmanager/pull/37
/close


Module function naming

We should probably be more consistent with the naming conventions used for the functions in the module. For example, all "exported" functions in the os.sh module should probably start with primer_os_.
Given this convention, may be should implementations have an external function starting with primer_step_ instead, see #2

Fixed by 2b679605889266c8a38881b63a3527b14f33b9a5


Starting a navigation

User story

As a User
I want to simulate the start of a navigation
So that {}

There should be a button that simulates the start of a navigation immediately. There should also be a possibility to simulate a delayed start (delay 30 seconds or one minute).

As a User
I want to be able to see, whether the app is currently simulating a navigation
So that {}

Acceptance criteria

In the navigation context there is a possibility to simulate the start of a navigation immediately.
In the navigation context there is a possibility to simulate the start of navigation after 30 seconds or after one minute.
The delayed start of the navigation must also be done when the app is running in the background.
The start of a navigation is correctly requested from the corresponding AOSP service.
It should be visible in the app whether it currently has the focus of the navigation type.

Notes
The necessary AOSP service is the CarAppFocusManager: https://developer.android.com/reference/android/car/CarAppFocusManager
The start of a navigation is simulated by requesting the focus at the CarAppFocusManager.
A listener can be registered at the CarAppFocusManager, which receives information about the current status of the app focus (addFocusListener()).
Definition of done (DoD)

Added only after week 5
The same for all features
Here goes the project specific part

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Duplicate


Update the QGB branch to use cosmos-sdk v0.46.x and latest celestia-app master

Update the QGB branch to use cosmos-sdk v0.46.x and latest celestia-app master.
Closes: https://github.com/celestiaorg/celestia-app/issues/499, https://github.com/celestiaorg/celestia-app/issues/517

Codecov Report

Merging #600 (08fa37d) into qgb-integration (195da78) will increase coverage by 1.20%.
The diff coverage is 57.66%.

:exclamation: Current head 08fa37d differs from pull request most recent head 319cf5e. Consider uploading reports for the commit 319cf5e to get more accurate results

@@                 Coverage Diff                 @@
##           qgb-integration     #600      +/-   ##
===================================================
+ Coverage             9.76%   10.96%   +1.20%
===================================================
  Files                   55       57       +2
  Lines                 9036     9240     +204
===================================================
+ Hits                   882     1013     +131
- Misses                8075     8125      +50
- Partials                79      102      +23

Impacted Files
Coverage Δ

e2e/qgb/Service.go
0.00% <ø> (ø)

e2e/qgb/qgb_network.go
0.00% <0.00%> (ø)

x/payment/keeper/keeper.go
62.50% <0.00%> (+6.94%)
:arrow_up:

x/payment/types/events.go
0.00% <ø> (ø)

x/qgb/module.go
3.57% <ø> (+0.06%)
:arrow_up:

x/qgb/types/codec.go
0.00% <ø> (ø)

x/payment/types/builder_options.go
41.26% <30.18%> (-58.74%)
:arrow_down:

x/payment/types/verify_sigs.go
36.84% <36.84%> (ø)

x/qgb/keeper/hooks.go
36.11% <40.00%> (+1.49%)
:arrow_up:

x/payment/types/builder.go
40.10% <47.05%> (-1.76%)
:arrow_down:

... and 10 more

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

@evan-forbes Sure, but the thing is, if I remove the cosmetics and the fixes, the CI will not pass.
So, I guess we have the following options:

Squash all  the last 4 commits into 1 to have a clean history
Create a PR  targeting this branch having only those CI fixes so you can review them. Then, proceed  to review  this one.
Merge this one without the CI passing and the project not building.

For the merge, we need to sit down and see what we want to do with the QGB. Whether we want a separate repo for orchestrator and relayer or keep them here.

Ahh I see I didn't know that it wouldn't pass. Hmm, ok then it's probably fine. Maybe just clean them up slightly like rename them or something.
I'm definitely fine with keeping them (or squashing them)

@evan-forbes I just separated them into 2 chained PRs:

This one should be reviewed first, contains QGB changes: https://github.com/celestiaorg/celestia-app/pull/602
This one contains the changes from main: https://github.com/celestiaorg/celestia-app/pull/601


Added possibility to load PNG images from a data buffer.

The appveyor failure seems like appveyor flaking out.  I'm rerunning that now.

Looks good, thanks for the PR :)


How to remove the add tag option in tagging?

Hi, I'm using your tagging feature. Thanks for that.
In that, when I'm typing a tag which is not existing, it asks me to add a new tag. I want to hide it and only show the available tags to select. How do I achieve this?
Thanks in advance

Just remove the :taggable="true". It’s the default behavior of a multiple select.
https://vue-multiselect.js.org/#sub-multiple-select


generate heap profile to data dir

Signed-off-by: 5kbpers<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #11265
Problem Summary:
Currently we generate heap profile to /tmp, then the files may be lost after pod restarting in docker environment.
What is changed and how it works?
Proposal: xxx
What's Changed:

generate heap profile to data dir

Check List
Tests

Unit test
Integration test

Release note
None

/merge

/merge

/merge

/merge

/run-all-tests

/merge


Show information from both sections in the DLP landing page

The DLP landing page should show keys from both sections 1 (human entered) and sections 3 (from dandi organize) of the schema.
This should also show sections that are empty but required with an alert icon, so that dataset creators can see this.

related issue #103

@satra by the 3rd section, do you mean these lines?
https://github.com/BICCN/BICCN-Infrastructure-Draft/blob/ce0df09b310bbcdd0a7a63cc51a0955e3cae97b8/BICCN-policies/dataset-metadata.yaml#L66-L82
Do you have an example of a dandiset with this data?

Do you have an example of a dandiset with this data?

https://dandiarchive.org/dandiset/000004/draft
most of the new datasets in draft that have data should have this information.

@satra by the 3rd section, do you mean these lines?
https://github.com/BICCN/BICCN-Infrastructure-Draft/blob/ce0df09b310bbcdd0a7a63cc51a0955e3cae97b8/BICCN-policies/dataset-metadata.yaml#L66-L82

yes.

This should also show sections that are empty but required with an alert icon, so that dataset creators can see this.

In the Landing Page itself, or just the editor?

In the editor. the landing page can focus on available sections with data for now. it would be nice if the text on the landing page fields could be selected, and if it responsively covered two columns.


Change NovelCoronaAPIv1 to CovidAPIv1 for consistency

I would like to change the name of APIv1's model from NovelCoronaAPIv1 to CovidAPIv1 to make it consistent with APIv2.
Tasks

Change the model's name from NovelCoronaAPIv1 to CovidAPIv1
Make sure that all methods' names used by this model are fixed accordingly

Example:

Old

novel_corona_api = NovelCoronaAPIv1()

New

COVID_API_V1 = CovidAPIv1() # new - global scope

I'd like to work on this issue.


Notification actions (buttons in notifications) do nothing

Steps to reproduce

Run an app that displays notifications with actions OR create a notification with an action button as in the example here:

val snoozeIntent = Intent(this, MyBroadcastReceiver::class.java).apply {
    action = ACTION_SNOOZE
    putExtra(EXTRA_NOTIFICATION_ID, 0)
}
val snoozePendingIntent: PendingIntent =
    PendingIntent.getBroadcast(this, 0, snoozeIntent, 0)
val builder = NotificationCompat.Builder(this, CHANNEL_ID)
        .setSmallIcon(R.drawable.notification_icon)
        .setContentTitle("My notification")
        .setContentText("Hello World!")
        .setPriority(NotificationCompat.PRIORITY_DEFAULT)
        .setContentIntent(pendingIntent)
        .addAction(R.drawable.ic_snooze, getString(R.string.snooze),
                snoozePendingIntent)

Post the notification
Click the button in the notification
The notification gets dismissed and does nothing else, instead of firing the intent snoozePendingIntent.

✔ Expected Behavior
Clicking on the notification button should do what it was programmed to do, like firing the intent in the example above
❌ Actual Behavior
The notification gets dismissed instead, even when it is not programmed to and nothing else happens.
Other Software
All Android apps that show notifications with buttons in them, are affected.
Please specify the version of Windows Subsystem for Android
2304.40000.10.0

I can reproduce this problem, which is somewhat historical, the Action button on the notification has never worked properly since the first release of WSA, and can only be clicked to perform the default action on the notification.

Hello! We have the same problem in our application.
All notification actions do nothing and do not display any errors in Logcat from Android side. They just dismissed silently.
Please, fix this as it blocks our application from normal usage.


Create Annapoornaaradhya.md

Hello GSSOC team these are a few PR that are not merged.
They are level 2.
@MastanSayyad could you please review these and merge my PR
https://github.com/PriyaGhosal/BuddyTrail/pull/1856
https://github.com/beRajeevKumar/Frontend_Mentor/pull/484
Thank you

https://github.com/PriyaGhosal/BuddyTrail/pull/1856 level 1
https://github.com/beRajeevKumar/Frontend_Mentor/pull/484 level 2


Is there a command to preview the preprocessed code?

I tried gcc -E corne.keymap but nothing happened

Give pcpp config/corne.keymap a try.

Give pcpp --passthru-unfound-includes corne.keymap a try (install with pip if needed).
The option prevents it from complaining about missing zmk includes

It works. Thank you so much!


[Launch] Next Build Analytics

Description
There has been a switch to GA4 tracking which is different than what was previously done in Next Build.
All tracking needs to be updated to mimic the work that was done in Content Build.
Additional Notes
Most of this work seemed to be done at the component level but we need to verify that items are tracking as expected. Contact Randi Mays to discuss what frontend work was done. There are chrome extensions that will allow you to see what data is being sent via google analytics.
Acceptance Criteria

[ ] GA4 tracking is implemented and we are getting accurate reports to what we currently see in next build

Following steps

For any template with recordEvent, look at its equivalent in content-build
check that equivalent for any recent updates, with an eye for changes relating to analytics

Content File / Next File / Same
Content-Build
Next-Build
Last changed for  content-build

PressRelease
<va-link>
<a> with recordEvent
1 month - Randi   Mays

Event
<va-link>
<va-link> with recordEvent
3 months - Randi   Mays

News_story / StoryListingLink
<va-link>
<a> with recordEvent
3 months - Josh   Mills

How_do_you_rate / rateYourExperience
Conditional  RecordEvent
Conditional  RecordEvent
3 years - Kelson   Adams

linkTeaser
Two <va-link>
Two <li> with recordEvent
6 months - Randi   Mays, Chris Kim

imageAndStaticMap
<div> with  recordEvent
<div> with  recordEvent
3 years - Napoleon   K

Default_contact / contactInfo
<a> with recordEvent
<Link>  with recordEvent
1 Year - Randi   Mays

benefitHubsLinks
<va-link>
<a>  with recordEvent>
6 months - Randi   Mays

Fullwidth_banner_alerts / facilitybanner
<va-link> with  recordEvent>
Conditional  RecordEvent
3 months - Randi   Mays

404.tsx
Not Found
useEffect w/   record Event
N/A

Using Adswerve and Content-Build PR as reference to test if the Google Analytics is working and no duplications.
Testing done on dev.va.gov
Replicate Testing in Content-Build's PR

Disability compensation accordion: "Learn how to manage disability compensation benefits online"
Matched Prod

Expand Image
![Screenshot 2024-09-04 at 12 51 31 PM](https://github.com/user-attachments/assets/013ac097-f65f-4798-bf5e-47d488d0e46f)

Rank of fixing

rateYourExperience [ fix dev to send the page-view like prod ]
Fullwdith_banner_alerts /facilityBanner [ Update dev to be a nav-link-click and not a nav-alert-boc-link-click ]
News_story /storyListingLink [ Update to va-link ]
pressRelease [ Update to va-link ]
(Maybe) ImageAnStaticMap [ Possibly don't change since they both are behaving the same ]