updated docs to include webpack config

Was getting errors like this when trying to integrate with the Angular 2 Webpack Starter:
WARNING in<EMAIL_ADDRESS>Cannot find source file '../../lib/index.ts': Error: Cannot resolve 'file' or 'directory' ../../lib/index.ts in /home/select/Dev/shoutr-srv/client/node_modules/@ngrx/router
Found the solution on gitter here: https://gitter.im/ngrx/store/archives/2016/04/26, and thought I should share with everyone else.

Will you update to only include specific dependencies for ngrx/router? ngrx/router-store isn't required. Rebase on top of master also.
Thanks

@brandonroberts think it should be good to go but please double check that I did the rebase correctly

LGTM. Thanks!


MapValueAttribute does not support mapping non-nullable constant values, to nullable properties of the same type

Describe the bug
The following
enum TargetEnum
{
    Bar,
}

class Source
{
    string Quux { get; set; }
}

class Target
{
    string Quux { get; set; }
    TargetEnum? TheTarget { get; set; }
}

[MapValue(nameof(Target.TheTarget), TargetEnum.Bar))
static partial Target Map(this Source source);

incorrectly yields
RMG077: Cannot assign constant value TargetEnum.Bar of type TargetEnum to Target.TheTarget of type TargetEnum?

but should result in no error and valid generated code, because a non-null value of a type is always assignable to a nullable value of the same type.
The below cannot be used as a workaround, as typecasts are not permitted in attributes.
[MapValue(nameof(Target.TheTarget), (TargetEnum?)TargetEnum.Bar))

The below is a valid workaround, but undesirable.
[MapValue(nameof(Target.TheTarget), Use = nameof(GetNullableTargetEnumValue)))
static partial Target Map(this Source source);

static TargetEnum? GetNullableTargetEnumValue() => TargetEnum.Bar;

Environment (please complete the following information):

Mapperly Version: 4.0.0
Nullable reference types: enabled
.NET Version: 8.0.403
Target Framework: .net8.0
Compiler Version: 4.11.0-3.24460.3 (5649376e)
C# Language Version: 12.0
IDE: VS2<IP_ADDRESS>
OS: Windows 11 Pro 23H2 (22631.4169)

Additional context
#631

Thanks for reporting. Feel free to contribute 😉
Hint: Riok.Mapperly.Descriptors.MappingBodyBuilders.SourceValueBuilder.TryBuildConstantSourceValue

While reading the code I already fixed it: https://github.com/riok/mapperly/pull/1536

Fix released with 4.1.0-next.1.

@latonz 4.1.0-next.1 fixes the case whereby
[MapValue(nameof(Target.TheTarget), TargetEnum.Bar))

raised RMG077, which is good... but that version now causes RMG078 to be raised for the previously-working
[MapValue(nameof(Target.TheTarget), Use = nameof(GetNullableTargetEnumValue)))
static partial Target Map(this Source source);

static TargetEnum? GetNullableTargetEnumValue() => TargetEnum.Bar;

which is incorrect.

@IanKemp My bad... https://github.com/riok/mapperly/pull/1538 should fix it definitely (hopefully).

@latonz thanks for your hard work - will wait for next .next version to drop and report back with verification.


Changing the behaviour of ios Datetimepicker to work like android's.

@mbastian93 could you please test this on android emulator?

I already tested the changes on android @mbastian93. You don't need it anymore.


Fix req object placement

Fixes https://github.com/pinojs/pino-http/issues/307#issue-1992665710

Unfortunately creating a child logger with the given request set is critical to keep attributing all logs to the given request.

Not sure if I follow. I believe it does keep the request set to all logs having it this way. req is present for all logs unless this condition is meet  https://github.com/pinojs/pino-http/blob/77191aedae7d0fb0370d0b5761613ec97be29941/logger.js#L191
that why Im not sure about that one test. If the req should be added to this false evaluated condition then all logs will contain req/res objects

I've run the example.js file in both master and this PR. It's different.
master
➜  pino-http git:(master) node example.js
{"level":30,"time":1706779462620,"pid":27894,"hostname":"mcl","req":{"id":1,"method":"GET","url":"/","headers":{"host":"localhost:3000","user-agent":"curl/8.4.0","accept":"*/*"},"remoteAddress":"::1","remotePort":56915},"msg":"something else"}
{"level":30,"time":1706779462624,"pid":27894,"hostname":"mcl","req":{"id":1,"method":"GET","url":"/","headers":{"host":"localhost:3000","user-agent":"curl/8.4.0","accept":"*/*"},"remoteAddress":"::1","remotePort":56915},"res":{"statusCode":200,"headers":{}},"responseTime":4,"msg":"request completed"}

This PR:
➜  pino-http git:(fix-nexted-key) node example.js
{"level":30,"time":1706779498790,"pid":28809,"hostname":"mcl","msg":"something else"}
{"level":30,"time":1706779498793,"pid":28809,"hostname":"mcl","req":{"id":1,"method":"GET","url":"/","headers":{"host":"localhost:3000","user-agent":"curl/8.4.0","accept":"*/*"},"remoteAddress":"::1","remotePort":56932},"res":{"statusCode":200,"headers":{}},"responseTime":3,"msg":"request completed"}


Update Core/Route.php

I have added single line to check if the url ends with '/' or not. If it is not then line appends '/' at the end. In this way, there is no need to add 2 route with slashed and nonslashed one.
They both identical and one of them work for 'echo' and 'echo/' route.
$router->add('echo', ['controller' => 'Home', 'action' => 'echo']);
$router->add('echo/', ['controller' => 'Home', 'action' => 'echo']);

Thank you for the request - this is something that I will be including in an updated version of the framework, but for now I don't want to change it so that it's in line with the associated course. Thanks anyway

Oh I am looking forward to the new version. Thank you.


Application Run by Reply-Button

When a dialog is displayed ModeLess I wonder wheter is may be possible to invoke the service of a VB-Project via Application Run. If possible, a Button may be provided with an attached "Run-String" which is performed when the buttons is pressed. This wolud offer an enormous flexibility because by default there is no way back to the application which displayed a modeless message.

Implemented! The service mMsg.ButtonAppRun allows to collect Application.Run information attached to a buttons in a Dictionary which is passed on to the message form with the services mMsg.Box and mMsg.Dsply (the UserForm remains completely encapsulated and is not actively used by the caller).


compile error Smart Mailbox

Hi,
thanks for this cool Smart-Mailbox project, but i constantly get an compilation error:
Compilation error: 'GPIO_SEL_25' was not declared in this scope; did you mean 'GPIO_NUM_25'?
I tried to rename, but no success.
All Libs are present and Board Support URL is also ok.
Using Arduino 2.3.2 / Firebeetle ESP 32 E
Any Ideas ?
Thanks

Hi,
the code is pretty old and it seems the bitselectors for getting the wakeup reason are no more defined.
Instead, there's now a function for getting the pin like there is one for getting the wakeup reason, called esp_sleep_get_ext1_wakeup_status()
If you give me a day or two, i'll find out how.

Hi McSlow,
that would be awsome !
Cheers


Can we provide NUXT style support just use layout in script block?

In nuxt we use layouts in options API in script block
{
  layout: 'name'
}

I have a project migrated from nuxt to vite which contains a lot  pages, if this plugin support nuxt like style, it will reduce a lot replace work for me.
Thanks a lot!

You can change line 11 ...
      component: layouts[route.meta?.layout || 'default'],

... to point to whatever you want to.
If you use something else than vite-plugin-vue, I would encourage you to find out how your route object is created. But, the preferred Vue Router way of adding extra infos to routes, is putting them in route.meta

You can change line 11 ...
      component: layouts[route.meta?.layout || 'default'],

... to point to whatever you want to.
If you use something else than vite-plugin-vue, I would encourage you to find out how your route object is created. But, the preferred Vue Router way of adding extra infos to routes, is putting them in route.meta

thanks for reply!


Working example with javascript

Hi @valendres, can add working example of Javascript. >

Hey @ram5607, theres some working examples within the example package: https://github.com/valendres/playwright-msw/tree/main/packages/example
Please let me know if this helps :)


i18n(fr): Update guides/server-side-rendering.mdx from #8011

Description (required)
Update guides/server-side-rendering.mdx from #8011

Related issues & labels (optional)

Closes #
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
guides/server-side-rendering.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[5.2] Rename "lists" to "pluck" in the query builder

...to be consistent with Collection@pluck, Arr::pluck and array_pluck.
Keeping lists as an alias.

Maybe we should mark the old function as deprecated then?

Maybe we should mark the old function as deprecated then?

I thought we're all about consistent in GB vs Eloquent, Eloquent\Builder::lists() is not deprecated, however Eloquent\Builder::pluck() is at the moment. https://github.com/laravel/framework/blob/5.1/src/Illuminate/Database/Eloquent/Builder.php#L203

lol

LOL. We'll get there in the end.

@JosephSilber fix Eloquent pluck too.

Wow. Just so that you know, just had an hard time working on an app built for Laravel 5.1, but which should have looked at Laravel 5.2 documentation at some point. It was calling pluck thus was getting the first row only.
Changing method roles is really tricky. Please, strongly emphasize this in upgrade guides if you do this again! Cheers.


Add data flow accordion and modal

https://www.figma.com/file/dmEwdK3xZwjKfGVQ9t66xe/Fides-v.2-Master-Working-Files?node-id=3892-123818&t=deCDCLuWBOGAFzeL-0

@rsilvery Does clicking Confirm in the modal save and send a request to the backend?

Or does it just populate what shows up in the accordion and saving ( request to the backend) only happens when save is clicked on the main form?

Please see Simon's response in the design channel. https://ethyca.slack.com/archives/C0114BAK160/p1680799872228429?thread_ts=1680799659.657109&cid=C0114BAK160
If possible, let's post these questions there since he doesn't have access to this repo for some reason :)


fix(runloop): use upstream status code in passive health check (#10325)

We expect the upstream passive unhealthy healthcheck on HTTP status code should only mark the targets as "down" (not reachable or proxiable) using the returned HTTP status code from the target, and it should not be affected by the final HTTP status code set by Kong plugin, which is returned to the client.
So we change the passive health check implementation to use nginx var upstream_code, this will not be changed by any plugin.
FTI-4841
Fix #10281
(cherry picked from commit dbe8d947e7d6a6c12f8464fc2f08276be1cd392f)

Summary

Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference
FTI-4802

@attenuation please review and approve if this PR matches your fix in https://github.com/Kong/kong/pull/10325.


pysisyphus is broken on unstable

I think I broke pysisyphus when I reverted the HDF5 patch (no idea how these two are connected).
It now fails a test:
FAILED tests/test_cos_dimer/test_cos_dimer.py::test_hcn_neb_dimer_irc - assert 4 == 6

@sheepforce

This test keeps failing:
=========================== short test summary info ============================
FAILED tests/test_cos_dimer/test_cos_dimer.py::test_hcn_neb_dimer_irc - assert 4 == 6
= 1 failed, 750 passed, 189 skipped, 355 deselected, 14 xfailed, 342 warnings in 3856.47s (1:04:16) =

It seems to be a bit of a gamble if it passes or fails. It fails almost every other evaluation of the master branch, when nixpkgs is updated.
Is there a way to disable this test?

Ok, I disabled the test. It builds fine again.


Update homeScreen.dart

I remember specifically we were told that the bottom menu bar no longer needs a "search" function.

This was addressed in PR #35, thanks for remembering!


f.robyn() syntax error

Hi,
When running the exec code. The modelling part.
model_output_collect <- f.robyn(set_hyperBoundLocal
                                ,optimizer_name = set_hyperOptimAlgo
                                ,set_trial = set_trial
                                ,set_cores = set_cores
                                ,plot_folder = getwd()) # please set your folder path to save plots. It ends without "/".

I'm recieving this error with traceback right away:
Error in py_module_import(module, convert = convert) :
  SyntaxError: invalid syntax (typing.py, line 254)
5.
stop(structure(list(message = "SyntaxError: invalid syntax (typing.py, line 254)",
    call = py_module_import(module, convert = convert), cppstack = NULL), class = c("Rcpp::exception",
"C++Error", "error", "condition")))
4.
py_module_import(module, convert = convert)
3.
import("nevergrad") at fb_robyn.func.R#708
2.
f.mmm(set_hyperBoundLocal, set_iter = set_iter, set_cores = set_cores,
    optimizer_name = optmz) at fb_robyn.func.R#1156
1.
f.robyn(set_hyperBoundLocal, optimizer_name = set_hyperOptimAlgo,
    set_trial = set_trial, set_cores = set_cores, plot_folder = getwd())

Hello, it looks like you can't load nevergrad? This update requires some new libraries, including a python library "nevergrad" that has to be installed through conda. In the new .exec script, line 42-45, there's simple guide to install it.

there's been couple of fixes pushed. please try out the latest version and let us know if everything works.


Proposal to provide a dispose function in class Animation which can easily remove listeners in animation

Use case
Animation provides removeStatusListener and removeListener functions to clean up any callbacks. but when using an anonymous method, these two methods can not work. This question occurs in the official example :

But in the source code of animations.dart, removeStatusListener is called frequently.

Why doesn't Flutter provide a dispose function in Animation?
Proposal
Provide a dispose function in class Animation which can easily remove listeners in animation.

I am not 100% sure what this is asking for. The Animation object doesn't hold any resources of its own so it doesn't need to offer a dispose method to clear those resources.
In the example given the animation object returned by Tween.animate actually adds the listeners directly to the controller.
Could you elaborate what your concern is? Or why you need a dispose functionality on Animation?


Run presto-admin tests on an arbitrary cluster

Add ConfigurableCluster, which is used in place of a docker cluster
if a .yaml file is detected
Updated tests to make them more generic

Testing: make test-all on vagrant; ran the tests that were changed on
docker as well

There are some tests that are annotated with @docker_only -- namely, they only run when you're running tests on docker clusters. Some of them need to be like that -- like the running tests on a different OS -- but others are like that because running them on another cluster would be pretty intrusive (the ones that change authentication, for example, or require other non-sudo users).  I figured it wasn't worth it to make them work, but I can try longer if that seems appropriate.

looks good


CI generates warnings according custom walks ...

Wrapping a custom walk function as an AnonymousWalk. Future versions will only support custom walks that explicitly subtyle AbstractWalk.

Julia 1.8.5 when running test FMIFlux


a sounddevice callback swallows all errors

Hi,
My issue is that just by importing sounddevice, any posterior error in the code is "swallowded", meaning not thrown by the terminal and the terminal just exits without throwing anything.

The image above is the closest i could get to having some logs of the error. To get this error i had to import sounddevice inside a function so as to have my code run before because any other way the error was just suppresed.

it seems issue #468 has the same problem and might be due to import sounddevice.

If the import of sounddevice crashes your Python interpreter, then there may be a problem with the Python environment in which you execute the code. You seem to be on Windows, but please describe how you set up Python and installed the sounddevice package (maybe pip or conda?).

This is the info:

pip show sounddevice
Name: sounddevice
Version: 0.4.6
Summary: Play and Record Sound with Python
Home-page: http://python-sounddevice.readthedocs.io/
Author: Matthias Geier
Author-email<EMAIL_ADDRESS>License: MIT
Location: c:\users\gescolano\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages
Requires: CFFI
Required-by:

The problem is that it doesn't crash the python interpreter. After an error occurs, instead of it being thrown to the terminal and stopping execution it just stops executing without giving any feedback. On my other computer with python 3.10 this doesn't happen...
For example:
import sounddevice as sd
stream = sd.InputStream()
stream.start()
print("Recording...")

output: "Recording..."
import torch
vad = torch.hub.load("snakers4/silero-vad", "silero_vad")
print("Recording...")

output: "Recording..."
import sounddevice as sd
import torch
vad = torch.hub.load("snakers4/silero-vad", "silero_vad")
stream = sd.InputStream()
stream.start()
print("Recording...")

output: (nothing). An error must have been thrown, for some reason and the execution just stops with no error thrown.


libtorrent: 20161212 -> 0.13.7, rtorrent: 20161023 -> 0.9.7

Motivation for this change
Things done

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

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

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

@GrahamcOfBorg build rtorrent

Success on x86_64-darwin (full log)
Attempted: rtorrent
Partial log (click to expand)
make[2]: Nothing to be done for 'install-exec-am'.
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-rtorrent-0.9.7.drv-0/source'
make[1]: Leaving directory '/private/tmp/nix-build-rtorrent-0.9.7.drv-0/source'
post-installation fixup
gzipping man pages under /nix/store/5vv33amnpcyjvd02vcw9bbbcimgy1grg-rtorrent-0.9.7/share/man/
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/5vv33amnpcyjvd02vcw9bbbcimgy1grg-rtorrent-0.9.7/bin
patching script interpreter paths in /nix/store/5vv33amnpcyjvd02vcw9bbbcimgy1grg-rtorrent-0.9.7
/nix/store/5vv33amnpcyjvd02vcw9bbbcimgy1grg-rtorrent-0.9.7

Success on x86_64-linux (full log)
Attempted: rtorrent
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7
shrinking /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7/bin/rtorrent
gzipping man pages under /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7/share/man/
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7/bin
patching script interpreter paths in /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7
checking for references to /build in /nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7...
/nix/store/x3q0il54q917ccyq3cnbbq298glmk97l-rtorrent-0.9.7

Success on aarch64-linux (full log)
Attempted: rtorrent
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7
shrinking /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7/bin/rtorrent
gzipping man pages under /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7/share/man/
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7/bin
patching script interpreter paths in /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7
checking for references to /build in /nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7...
/nix/store/jjy57azsaryjhs4zw7y0pwci6z0yc230-rtorrent-0.9.7


[BUG] Start Preview (QTGL) loads 'xcb' module from cv2 path vs QT path

Describe the bug
On start_preview(QTGL), QT throws an error stating -
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/usr/local/lib/python3.9/dist-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: xcb, eglfs, linuxfb, minimal, minimalegl, offscreen, vnc.
To Reproduce
Use code that imports/utilizes opencv2 and picamera2. Opencv-python (with GUI bindings) should be installed.
Expected behaviour
Preview windows launches without error
Console Output, Screenshots
export QT_DEBUG_PLUGINS=1
...
Got keys from plugin meta data ("vnc")
QFactoryLoader::QFactoryLoader() looking at "/usr/lib/aarch64-linux-gnu/qt5/plugins/platforms/libqxcb.so"
Found metadata in lib /usr/lib/aarch64-linux-gnu/qt5/plugins/platforms/libqxcb.so, metadata=
{
"IID": "org.qt-project.Qt.QPA.QPlatformIntegrationFactoryInterface.5.3",
"MetaData": {
"Keys": [
"xcb"
]
},
"archreq": 0,
"className": "QXcbIntegrationPlugin",
"debug": false,
"version": 331520
}
Got keys from plugin meta data ("xcb")
QFactoryLoader::QFactoryLoader() checking directory path "/usr/bin/platforms" ...
loaded library "/usr/local/lib/python3.9/dist-packages/cv2/qt/plugins/platforms/libqxcb.so"
QObject::moveToThread: Current thread (0x7f40020840) is not the object's thread (0x7f40223430).
Cannot move to target thread (0x7f40020840)
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/usr/local/lib/python3.9/dist-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: xcb, eglfs, linuxfb, minimal, minimalegl, offscreen, vnc.
Hardware :
Raspberry 4 (4GB) with Arducam 16MP

Hi, thanks for reporting this. I'm not managing to see this problem, could you attach a small self-contained code snippet that will reproduce it? Perhaps also explain whether you're logged directly into the Pi or are (for example) connected via ssh.  Thanks!

Just to add that a quick google search (for "python qt plugins opencv") reveals that OpenCV installs the QT plugins for its own use, and it seems like the ones you have are not compatible with the PyQt5 ones. Does that sound plausible in your case? If the official installation instructions on a clean image lead to a failure please let us know, otherwise those various pages thrown up by the search suggest various remedies...

Try this script (Be sure to verify that opencv-python is installed)

import argparse
import sys
import time
import cv2
from picamera2 import Picamera2, Preview
picam2 = Picamera2()
picam2.start_preview(Preview.QTGL)
picam2.start()
image = picam2.capture_array()
image = cv2.resize(image, (320, 240))
image = cv2.cvtColor(image, cv2.COLOR_BGR2RGB)
cv2.imshow('test', image)

I resolved it initially by going to headless opencv. Which is ofcourse not ideal.
However I found another workaround and that is to include this -
import os, sys
ci_build_and_not_headless = False
try:
from cv2.version import ci_build, headless
ci_and_not_headless = ci_build and not headless
except:
pass
if sys.platform.startswith("linux") and ci_and_not_headless:
os.environ.pop("QT_QPA_PLATFORM_PLUGIN_PATH")
if sys.platform.startswith("linux") and ci_and_not_headless:
os.environ.pop("QT_QPA_FONTDIR")
This however breaks opencv2.imshow or any opencv QT based function from working.
Ask - would it be possible to limit scope for all picamera2 based QT windows to only refer to plugin loaded from pyqt folder vs any other location?

I honestly have no idea whether it's possible to "limit the scope" of Picamera2's Qt windows in any way - is this something you have seen before and could point me at?
I'm also not able to make anything go wrong on my system which makes it difficult to try anything! Can you explain what you did to get your system into this state where it doesn't work properly?

hi!
i have the same bug when try to start on Raspbian 64 app_capture2.py:
[0:13:13.452996081] [3436]  INFO Camera camera_manager.cpp:293 libcamera v0.0.0+3700-f30ad033
[0:13:13.495844114] [3445]  INFO RPI raspberrypi.cpp:1368 Registered camera /base/soc/i2c0mux/i2c@1/ov5647@36 to Unicam device /dev/media0 and ISP device /dev/media2
[0:13:13.500441611] [3436]  INFO Camera camera.cpp:1029 configuring streams: (0) 800x600-XBGR8888
[0:13:13.500943755] [3445]  INFO RPI raspberrypi.cpp:759 Sensor: /base/soc/i2c0mux/i2c@1/ov5647@36 - Selected sensor format: 1296x972-SGBRG10_1X10 - Selected unicam format: 1296x972-pGAA
QObject::moveToThread: Current thread (0x12473eb0) is not the object's thread (0x132981f0).
Cannot move to target thread (0x12473eb0)

qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/home/pi/.local/lib/python3.9/site-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: xcb, eglfs, linuxfb, minimal, minimalegl, offscreen, vnc.

I think may be it is qt5 issue because when i use this:
export QT_QPA_PLATFORM=vnc
The output changed to:
[0:19:06.261125967] [4199]  INFO Camera camera_manager.cpp:293 libcamera v0.0.0+3700-f30ad033
[0:19:06.300875238] [4206]  INFO RPI raspberrypi.cpp:1368 Registered camera /base/soc/i2c0mux/i2c@1/ov5647@36 to Unicam device /dev/media0 and ISP device /dev/media2
[0:19:06.304477303] [4199]  INFO Camera camera.cpp:1029 configuring streams: (0) 800x600-XBGR8888
[0:19:06.304855186] [4206]  INFO RPI raspberrypi.cpp:759 Sensor: /base/soc/i2c0mux/i2c@1/ov5647@36 - Selected sensor format: 1296x972-SGBRG10_1X10 - Selected unicam format: 1296x972-pGAA
Traceback (most recent call last):
  File "/home/pi/picamera2/examples/app_capture2.py", line 36, in <module>
    qpicamera2 = QGlPicamera2(picam2, width=800, height=600, keep_ar=False)
  File "/home/pi/.local/lib/python3.9/site-packages/picamera2/previews/q_gl_picamera2.py", line 102, in __init__
    self.init_gl()
  File "/home/pi/.local/lib/python3.9/site-packages/picamera2/previews/q_gl_picamera2.py", line 138, in init_gl
    self.create_surface()
  File "/home/pi/.local/lib/python3.9/site-packages/picamera2/previews/q_gl_picamera2.py", line 132, in create_surface
    surface = eglCreateWindowSurface(self.egl.display, self.egl.config,
  File "/home/pi/.local/lib/python3.9/site-packages/OpenGL/platform/baseplatform.py", line 415, in __call__
    return self( *args, **named )
  File "/home/pi/.local/lib/python3.9/site-packages/OpenGL/error.py", line 230, in glCheckError
    raise self._errorClass(
OpenGL.raw.EGL._errors.EGLError: EGLError(
	err = EGL_BAD_ALLOC,
	baseOperation = eglCreateWindowSurface,
	cArguments = (
		<OpenGL._opaque.EGLDisplay_pointer object at 0x7f8a1e6ec0>,
		<OpenGL._opaque.EGLConfig_pointer object at 0x7f8a8596c0>,
		c_void_p(1),
		None,
	),
	result = <OpenGL._opaque.EGLSurface_pointer object at 0x7f8254cbc0>
)

I think it is qt5 issue. To quick fix just add:
import os
os.environ.pop("QT_QPA_PLATFORM_PLUGIN_PATH")

or
import os
os.environ.update({"QT_QPA_PLATFORM_PLUGIN_PATH": "/usr/lib/aarch64-linux-gnu/qt5/plugins/xcbglintegrations/libqxcb-glx-integration.so"})

see here: https://stackoverflow.com/questions/63829991/qt-qpa-plugin-could-not-load-the-qt-platform-plugin-xcb-in-even-though-it

Thanks for the info.
@ankitb82 Can you comment on whether that works for you? If this seems to be a generally good workaround then it might be worth adding it to the documentation. Thanks!

This works the same way the workaround I posted above works - it allows picamera2 UX to function but opencv UX functions don't (imshow, etc).

This works the same way the workaround I posted above works - it allows picamera2 UX to function but opencv UX functions don't (imshow, etc).

d u see stackoverflow link i posted ? thats your question. rebuild opencv if you want cv funcs.

I did look at the stackoverflow link. But think about it, expecting folks to recompile opencv just to work with picamera2 seems like a stretch right?
Opencv is such a common library and those window functions are super commonly used (During development and otherwise).

Rewinding this discussion a bit, can you say exactly what image you're running and how you installed PyQt5 and OpenCV? So far as I know, if you start with the standard Raspberry Pi OS image and install according to the instructions (use "sudo apt install -y python3-pyqt5 python3-opencv") then everything just works. Was there a particular reason that required you to do something different? Thanks!

Hi, just wondering if there's anything else to add on this issue?
My feeling is that it's not really a Picamera2 issue, it's down to Qt, OpenCV and Python how these things co-exist. In fact the latest Picamera2 on PyPI by default installs neither Qt nor OpenCV, so I'm thinking it's not something for Picamera2 to fix.
As far as I know, everything works fine for folks using the standard installation instructions. But if there are particular circumstances where Qt and OpenCV start to fight then I'd be very happy to add an "Application Note" to our documentation that explains how to deal with this. Having not encountered this myself I don't think I'm really qualified - would anyone else on this thread be able to supply a paragraph or two that I could use for this purpose?
Thanks!

I think I'm going to close this issue at this point, but if anyone would like to return to it - especially if we feel we know what guidance to give users - then please do open another issue. Thanks!

I've been trying to find anyway to get opencv to work with Picamera2 for the past 2 months with no solution. I did not do anything outside of the normal process for install. It simply never works. I've installed over 20 times from scratch and this issue persists. @davidplowman do you have some specific set of instructions of exact commands and order of running them which gets this working? I have Picamera2 working fine, everything works until I try to use any cv2 command which involves preview at which point I constantly get the common "Could not find the Qt platform plugin "xcb" error over and over again.  I would incredibly grateful for any way past this.

@trezero Hi, I start from a fresh Raspberry Pi OS image. I always download and install the latest using the Raspberry Pi Imager. Once that's booted and applied all its updates, I run
sudo apt install -y python3-opencv
sudo apt install -y opencv-data

as described in section 2.5.1 of the manual. After that everything seems to work for me, please let us know if it doesn't for you. Thanks!

I still have the same issue. I was attempting to port a software for DIC called RealPi2dDIC to PiCamera2 and got the said error "Could not find the Qt platform plugin "xcb". The link as follows for the original code for your suggestion on where it might have gone wrong:
https://github.com/ElsevierSoftwareX/SOFTX-D-20-00054

One thing to watch out for is mixing software installed with pip and installed with apt.
Python modules are placed in different locations and depending on the python path the wrong version gets picked up.
xcb (X C bindings) is an x-windows library for easy access (simpler than direct Xlib calls).
Qt might also have been installed in more than one place.

I had the very same problem (and I could not remember anymore, that I had installed pip install opencv-python
), but then I uninstalled it and with sudo apt install -y python3-opencv everything worked fine! Thank you!


Add image support

openai now has a new query which takes image as an extra_query, please consider support it

similar to #97


Multi-currency support

only tested on USD currently

Do you have any plans to improve the project? We are very interested but it lacks support for CHF and EUR.


pull request for e2e test at 11/05/2016 04:56:01

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Any good example of JSQVideoMediaItem

Hello,
Thanks for your efforts on this framework.
I googled to find any good example of JSQVideoMediaItem, but I could not find any good one. Could you please demonstrate some example?
I used the demo with the following code:
- (void)addVideoMediaMessage
{
    NSBundle *bundle = [NSBundle mainBundle];
    NSString *moviePath = [bundle pathForResource:@"splash" ofType:@"m4v"];
    NSURL *videoURL = [NSURL fileURLWithPath:moviePath];

    JSQVideoMediaItem *videoItem = [[JSQVideoMediaItem alloc] initWithFileURL:videoURL isReadyToPlay:YES];
    JSQMessage *videoMessage = [JSQMessage messageWithSenderId:kJSQDemoAvatarIdSquires
                                                   displayName:kJSQDemoAvatarDisplayNameSquires
                                                         media:videoItem];
    [self.messages addObject:videoMessage];
}

I am sure the address is OK. There is no error in message box. But when I click on video message, nothing happened. (something like start video).
In addition, I need to know the supported video formats for JSQVideoMediaItem.
Thanks man

The framework does not supply the mechanics to actually play the view, simple create & display unique bubble's (video types, image types, text types, etc). What you need to do is look at the didTapMessageBubbleAtIndexPath methods, implement your own video player solution and pass the url found from the didTapMessageBubbleAtIndexPath method after getting the message and play the video from the url you get.

Thanks @ryanphillipthomas ! :smile: That's correct.

how to get videoUrl in didTapMessageBubbleAtIndexPath?Also how to play?


respondWith should no-op/throw when called outside context of fetch event

addEventListener('fetch', async event => {
  await Promise.resolve();
  // In production, this is a no-op because respondWith was not called immediately.
  // cloudworker is returning the response. Perhaps it should throw to help folks catch issues quickly.
  event.respondWith(new Response('hello world'));
});

thanks! I'll take a look at this sometime soon.


Update to 1.19.3 and 1.19.4

The aquatic torch item is now found in the functional block tags right after the redstone torch. This version works for both Minecraft 1.19.3 and Minecraft 1.19.4.

Thanks a lot!


Classify Brainfuck as a programming language and include samples

The search results of usage on GitHub:
https://github.com/search?q=language%3A+brainfuck&ref=searchresults&type=Repositories&utf8=✓

@CodingAnarchy The Travis build is failing because of this test.
@josh Is there a reason for Brainfuck to be classified as nil?

I understand that Brainfuck isn't a popular language, as esoteric as it is, but to have the files just be ignored in the repositories they are a part of is frustrating, at least in my perspective.

I've brought your commits into https://github.com/github/linguist/pull/2153, which I opened around the same as this PR. #2153 is, like, a superset of this PR, because it turns out many languages are lacking types.

Works for me.  Thanks!


fix: #108 Visual feedback for settings checkbox

Improves visual feedback for checkboxes on the settings page, fixes #108
Minor changes to styling only.

Thanks for the contribution!


TimeZone: Editing a date column with timestamps+seconds triggers an empty state in calendar picker

Description
When you go into edit mode, a date column that contains seconds and timezone doesn't retain its value.
Note:

Tested this without seconds and the calendar picker retains its value when you go into edit mode.

This is reproducible in the demo with a slight tweak, change the display format and dateFormat to include :ss (In addition to the current HH:mm).
Version
Data-grid:origin/feature/timezones
Browser: Chrome
Screenshots

After investigating further, I've found this issue to be invalid.  The issue here is that you've specified the dateFormat.format to include seconds (ss) but the actual data does not include seconds.  The actual data format must match the dateFormat.format, that is the entire point of the dateFormat.format property.
Closing this, works as expected.

Thanks @drewharvey . Verified adding seconds to <dateFormat> as well as the actual data resolves the issue.


Add a generic install.sh and update README.md.

Adds a generic install.sh utility.
Updates README.md to use the new installer.
Adds tests.

OSX install example:
om:~ kw$ curl -LSsf https://raw.githubusercontent.com/kwlzn/lift/kwlzn/installer/install.sh | /bin/bash
Download URL is: https://github.com/a-scie/lift/releases/latest/download/science-fat-macos-aarch64
######################################################################## 100.0%
######################################################################## 100.0%
Download completed successfully
Download matched it's expected sha256 fingerprint, proceeding
Installed https://github.com/a-scie/lift/releases/latest/download/science-fat-macos-aarch64 to /Users/kw/.local/bin/science
WARNING: /Users/kw/.local/bin is not detected on $PATH
You'll either need to invoke /Users/kw/.local/bin/science explicitly or else add /Users/kw/.local/bin to your shell's PATH.
om:~ kw$ /Users/kw/.local/bin/science -V
0.7.0

I'm not sure what you were gunning for here @kwlzn. A step towards that sort of thing maybe?

yeah, a pre-1.0 rough edged step in that direction. essentially, just aiming to make it trivial to link to and have a complete newcomer running science -h (as docs) in <1 minute in case it comes up in integration discussions with other OSS projects.
I'm happy to refine this a bit more tho and port get_pants.sh's approach/logic + add some CI coverage with 1.0 in mind.

the salient part of the latest CI failure seems to be a failure in the pwsh curl mode call:
pwsh -c \'Invoke-WebRequest -OutFile /c/actions-runner/_work/lift/lift/.nox/f83bbc49/tmp/tmp.LGZBCkLskN/science-fat-windows-x86_64.exe.sha256 -Uri https://github.com/a-scie/lift/releases/download/v0.7.0/science-fat-windows-x86_64.exe.sha256\'
 Invoke-WebRequest:
 Could not find a part of the path \'C:\\c\\actions-runner\\_work\\lift\\lift\\.nox\\f83bbc49\\tmp\\tmp.LGZBCkLskN\\science-fat-windows-x86_64.exe.sha256\'.

my intuition says the /c/... in /c/actions-runner/... implies the C:\ bit in alternate form, but pwsh is prepending to that to arrive at C:\c\.... need to do some more digging.

I think probably not. It seems highly likely dirname / basename don't play well with .

I think probably not. It seems highly likely dirname / basename don't play well with .

afaict, basename is used exactly once, with a URL and not a filepath, so that's natively using / always and spits out a single path part sans any path separators. and dirname is only ever called on OSX as a x-platform way to net install -D behavior. almost all of the logic leading up to this point is lifted directly from get-pants.sh, which presumably is known to work on Windows - which is the most baffling part.
I did notice one mixed \ + / path construction in the script in debug logging (due to -d receiving a windows temp dir path from the TempPathFactory fixture), but already fixed that in baba4f4.
there is one other similar path separator join (directly adjacent to where basename is used), but that's only ever joined with a mktemp -d output which appears to use / exclusively on windows fwict.

explicitly converting the /c/x/y/z mktemp-returned MSYS POSIX format to C:\x\y\z via cygpath -w in https://github.com/a-scie/lift/pull/88/commits/5c1a74e6c91c078c54a1b8124fe7f6494d09d13f definitely got past the initial pwsh iwr failure fwiw.
but based on https://devblogs.microsoft.com/commandline/tar-and-curl-come-to-windows/ it seems windows does have native curl since windows 10 (2018-present - not sure if support needs to go back further than that?) - it definitely works as expected at least in CI, so I moved to that as a unification.
with the broken downloads fixed, we're now onto a shasum failure:
sha256sum: \'science-fat-windows-x86_64.exe\'$\'\\r\\r\': No such file or directory

which seems to indicate some trailing carriage returns that may be messing things up:
om:tmp kw$ cat -e science-fat-windows-x86_64.exe.sha256
010cfa5ddf8d50482dada3624c044d74dc8e699d7b218d3d8d74a89204a4029c *science-fat-windows-x86_64.exe^M^M$

(with shasum checking removed, the tests pass E2E on the windows shards)

sanitizing the downloaded sha256 file with tr on windows seems to have done the trick and all tests now pass.
so, I think this is good to go - assuming you're ok with the switch to curl. there appear to be other failures from test_exe.py on the windows shard tho. is that expected?

Thanks for wading through this Kris.

so, I think this is good to go - assuming you're ok with the switch to curl.

Absolutely - simpler. I was forced to pwsh 2 years ago when getting scie-jump green in CI. I'm not sure what is different this much time passed.


Huge image filesizes and caching

From @emilhem on August 12, 2016 8:31
The header image on desktop should be a JPG and compressed a lot! 4 MB is a lot!
The other images should be compressed as well to improve delivery.
Also caching should be used on all the images hosted on dnollk.se
A tip is to look at Google Pagespeed: https://developers.google.com/speed/pagespeed/insights/?hl=en&url=dnollk.se
Copied from original issue: dtekcth/DNollK.se#23

This issue was moved to Jassob/DNollK.se#21

There are definitely room for improvements on both the amount data we push to clients and the number of requests needed to open for each client.
I think it would be nice if we could merge some javascript files so that they would be transferred in one blob instead of opening multiple connections.

Fixed header image with b9122744a9bbf8407f4d7b55741ad1f8373d2ef6
Now we have a new problem though! The dn0k.png is 11 MB large, so I resized it (the old one is named dn0k.old.png) and now it's only 1.7 MB. So we should warn the user when uploading to not upload so f***ing large images and use compression on PNGs.

Also I worked on the server to fix caching. It's better now.
 I ls -Al:ed the media/2016/ folder.

Fixed logo image with 0da2efe0e22507995b0d1d6fd4daea80b1837355
I have compressed a lot of the images on dnollk now. Data savings are huge. Now I can load dNollK without caching 200 times with a 2GB dataplan instead of just 20.
When the commits b9122744a9bbf8407f4d7b55741ad1f8373d2ef6, 0da2efe0e22507995b0d1d6fd4daea80b1837355 are applied then this issue can be closed.

Good work!
You are right in that we should inform the administrators of the site not to upload huge images as that leads to a poor experience for the viewer..
I'll check in on these changes tomorrow just so I understand what you've done (so that I might learn something in the process).


Our scan is terribly slow

It must end soon.  I think the solution is "sliding window scan", as implemented here: https://github.com/Oblomov/cldpp/tree/master/src

I am implementing this in the faster-scans branch.  Speedup seem to be around 50%.  I had hoped for more, but I think we can get there with some further tweaks.


CRD name changes in cleanup for quickstart docs

Small changes needed for crd cleanup in the quickstart docs...
WAS:
schedulingspecs.mcad.ibm.com appwrappers.mcad.ibm.com quotasubtrees.ibm.com
but now is:
quotasubtrees.quota.codeflare.dev
appwrappers.workload.codeflare.dev
schedulingspecs.workload.codeflare.dev

Closed in https://github.com/opendatahub-io/distributed-workloads/pull/119


Support for .org files?

Thanks for writing and extensively explaining this project on Reddit!
I am a LogSeq user and have all my files in .org format.
I contributed an org parser for unstructured a while ago.
Would this project automatically support org through unstructured, or does sth. else need to be done (e.g. adding .org to some whitelist)?
Thank you!

It should be relatively easy to add. Can you point me to some examples of .org files?

Support added in 0.6.2


Necessary changes that were brought in by API changes in Kubernetes c…

…lient
Relevant XTF changes [1]. Kubernetes Client migration guide [2].
[1] https://github.com/xtf-cz/xtf/pull/532
[2] https://github.com/fabric8io/kubernetes-client/blob/v6.0.0/doc/MIGRATION-v6.md
Description

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change
[ ] Chore (non-breaking change which doesn't affect codebase;
test, version modification, documentation, etc.)

Checklist

[x] Code contributed by me aligns with current project license: Apache 2.0
[x] Pull Request contains a description of the changes
[x] Pull Request does not include fixes for multiple issues/topics
[x] Code is self-descriptive and/or documented
[ ] I have implemented unit tests to cover my changes
[ ] I tested my code in OpenShift

/test all


Single support

This will fix #40
Because in Java it is not possible to have the same Signature but a different return value I've splittet the RxTiPresenterUtils into three parts:

RxTiObservableUtils
RxTiUtils
RxTiSingleUtils
RxTiPresenterUtils is deprecated now.

I've removed the Presenter in the class name because you can use it in Activtiys as well..
It is really not recommended but not mandatory...
However. I've also changed these "pattern" in Rx2.
But at this time, because we have no deliver*View methods, only the RxTiUtils class exist.
We need in Rx2 also a RxTiMaybeUtils :)
But that will be done in the future...
I've also added a simple example in our sample.

Because that solution is not acceptable (we have talked about that internally already) I will close that for now...


[Mobile] ERR_UNKNOWN_URL_SCHEME on launching links

Tested on Android
Flutter 3.0.2
Dart 2.17.3

flutter_quill: ^5.0.3
After adding a link to a text, tapping the link gives an error:
ERR_UNKNOWN_URL_SCHEME

It happened that the first tap it works, then it doesn't anymore.
In this case, the URL is https://open.spotify.com/...  but it shows only from spotify.com

Doesn't seem an issue where Quill can do anything about it. The ERR_UNKNOWN_URL_SCHEME error is commonly because of your browser issue . There's no application on your device which can handle that particular action. more info

Then please share the full link so we can test it. What I see on screen is different than what you claim. Cheers.

An example with an URL that happen is
https://open.spotify.com/playlist/74DWNKPKo2VbJtD510pe3t?si=mEaILnGaSxmYiZB7vW9ZsQ&utm_source=copy-link
This link should be open by the app if you have the Spotify app installed, otherwise it should open the website.
Instead, in both cases, it gives the errors ERR_UNKNOWN_URL_SCHEME
If on the same device you open this link outside the app, it will work fine! Even if you launch it from a note of Google Keep

At some point I'll have a look and relay back information. can't promise a precise date.

Any update on this or an ETA?
It is an essential feature for my use of this library.

Several months if it's hanging on me. I've got a lot of backlog to go trough.

The problem is localized on launchUrl in lib/src/widgets/text_line.dart:
  Future<void> _launchUrl(String url) async {
    await launchUrl(Uri.parse(url));
  }

That as default launch with LaunchMode.platformDefault that Leaves the decision of how to launch the URL to the platform implementation.
Change this could be considered a breaking change I guess, but as a workaround, you can override onLaunchUrl in QuillEditor
QuillEditor(
  onLaunchUrl: (url) async {
    await launchUrl(Uri.parse(url), mode: LaunchMode.externalApplication);
  },
  ...
)

Even if you have to change it everywhere and you can't use QuillToolbar.basic anymore.
launchUrl is from an external dependency, maybe the bug should be passed to them(?). I am not completely satisfied, but for now, this would do.


CLOUD-48190 upgrade to swarm 1.1.0

Images are missing except OpenStack

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2119/


Cannot use tailwind config on app.config

Version
@nuxtjs/tailwindcss: v6.10.4
nuxt: v3.9.1
Reproduction Link
https://stackblitz.com/edit/github-aowrfw?file=nuxt.config.ts
Steps to reproduce
Run the application npm run dev and check the terminal.
What is Expected?
The application should start and the tailwind configuration accessible on app.config file.
What is actually happening?
The application failed to start and the error below is thrown. This only applies to app.config and not when importing from vue components.

Valid issue. Thanks for reporting and providing a reproduction; let me investigate and get back to you. 🙂

I want to investigate this further, but I've added a workaround if you'd like to test on nightly release, set exposeConfig.write to true

Hi @ineshbose, I have tested the nightly release however the issue still exists. I pasted the linux and windows logs below to help. Thanks for your time!
Linux
[08:06:08]  ERROR  [worker reload] [worker init] Directory import '/home/projects/github-aowrfw/.nuxt/tailwind.config' is not supported resolving ES modules imported from /home/projects/github-aowrfw/.nuxt/dev/index.mjs
at _node_internal (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:36:5406)
at new  (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:36:4168)
at https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:114:11863
at https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:114:12159
at defaultResolve (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:114:12167)
at nextResolve (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:248:2120)
at ESMLoader.resolve (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:248:7556)
at ESMLoader.getModuleJob (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:248:4387)
at https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:181:932
at _0x560a95.link (https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:352:381444)
at https://githubaowrfw-fuki.w-credentialless.staticblitz.com/blitz.babcaeac.js:181:893
Windows
[08:05:24]  ERROR  [worker reload] [worker init] Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
at __node_internal_captureLargerStackTrace (node:internal/errors:496:5)
at new NodeError (node:internal/errors:405:5)
at throwIfUnsupportedURLScheme (node:internal/modules/esm/load:136:11)
at defaultLoad (node:internal/modules/esm/load:87:3)
at nextLoad (node:internal/modules/esm/loader:163:28)
at ESMLoader.load (node:internal/modules/esm/loader:603:26)
at ESMLoader.moduleProvider (node:internal/modules/esm/loader:457:22)
at new ModuleJob (node:internal/modules/esm/module_job:64:26)
at #createModuleJob (node:internal/modules/esm/loader:480:17)
at ESMLoader.getModuleJob (node:internal/modules/esm/loader:434:34)
at async ModuleWrap. (node:internal/modules/esm/module_job:79:21)

I forgot to mention that after you use exposeConfig.write: true, also update the import statement to import theme from '#tailwind-config/theme.mjs'

update the import statement to import theme from '#tailwind-config/theme.mjs'

This works well, thanks!


Cannot link pony on Ubuntu 18.04

When I try building Ubuntu 18.04 with LLVM-6 as make it errors in the linking stage as
Makefile:289: WARNING: LLVM 6 support is experimental and may result in decreased performance or crashes
Linking libponyc.tests
Linking libponyrt.tests
/usr/bin/x86_64-linux-gnu-ld: build/release/libponyrt.a(except_try_catch.o): relocation R_X86_64_32 against symbol `ponyint_personality_v0' can not be used when making a PIE object; recompile with -fPIC
/usr/bin/x86_64-linux-gnu-ld: final link failed: Nonrepresentable section on output
collect2: error: ld returned 1 exit status
Makefile:814: recipe for target 'build/release/libponyrt.tests' failed
make: *** [build/release/libponyrt.tests] Error 1
make: *** Waiting for unfinished jobs....
/usr/bin/x86_64-linux-gnu-ld: build/release/libponyrt.a(except_try_catch.o): relocation R_X86_64_32 against symbol `ponyint_personality_v0' can not be used when making a PIE object; recompile with -fPIC
/usr/bin/x86_64-linux-gnu-ld: final link failed: Nonrepresentable section on output
collect2: error: ld returned 1 exit status
Makefile:814: recipe for target 'build/release/libponyc.tests' failed
make: *** [build/release/libponyc.tests] Error 1

. What's wrong?

This should solve your problem.
https://www.ponylang.org/faq/#pic-compile-error
try
make default_pic=true

Thanks!

You're welcome. For future reference nordlow, these sort of questions where you aren't sure if its a bug or not should be directed to the IRC channel or mailing list.
https://github.com/ponylang/ponyc/blob/master/SUPPORT.md#should-i-open-an-issue

Got it, thanks.


Change ovpn path

How can I set the default path of all the generate ovpn? (instead of /root but something like /root/config)
Thanks

Replace $homeDir to /root/config in the script https://github.com/angristan/openvpn-install/blob/2950fd445752fd0515cdbf37b28ab8a494bc8299/openvpn-install.sh#L1083-L1090

Is possible to do in command line without edit the SH? Since can be updated

The script doesn't allow customizing that path yet, but you can run sed to edit the line.
sed -i 's|homeDir="/root"$|homeDir="/root/config"|' openvpn-install.sh


Take a look at minio for testing S3 provisioning

https://github.com/minio/minio

Looks like we're going with s3proxy with tool backup at least


Bus is not always 1

When building powercape.c and ina219.c for Beaglebone Black running a Resin.io Debian container, I had to change int i2c_bus = 1; to int i2c_bus = 2; as the devices were on the i2c-2 bus instead of the i2c-1 bus.
I have a patch if you want to make the fixes programatically (git apply fix_powercape.patch).

Whoops, I forgot to put the "bus" option in the powercape utility ala the ina219 util.  And then I went and used "-b" for "enter bootloader".  Hmmm....


Starfinder (simple): Correct 'Perception' spelling in NPC sheet

Changes / Comments
The 'Perception' button for NPCs was misspelled.
Roll20 Requests

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Thank you for finding this minor error. I approve of the changes.
As the author and curator of this sheet I would appreciate a heads up when errors are found so that corrections can be tracked.

@Acyd69 No problem! I wasn't sure of who to contact.
Maybe the repo needs a top-level CODEOWNERS file, Roll20 folks?

Maybe the repo needs a top-level CODEOWNERS file, Roll20 folks?

It's sufficient to look at the sheet.json to find that out, or to look at the edit history if someone have edited it a lot or frequently.


Explain Unfolding step

Had a few questions about what is going on here in the sampling stage in the Refiner:
if self.patch_crop_method == 'unfold':
               return x.permute(0, 2, 3, 1) \
                    .unfold(1, size + 2 * padding, size) \
                    .unfold(2, size + 2 * padding, size)[idx[0], idx[1], idx[2]]

https://github.com/PeterL1n/BackgroundMattingV2/blob/4a56223a1cd9b2c2678582513c573debbfc12cae/model/refiner.py#L205

x is (bs,c,h,w). Why is it being permuted to (bs,h,w,c) before the unfold?

Generally one unfold across the channel dimension should be able to extract the patches. Why are there two unfolds here?

What is the logic behind size + 2 * padding?

We want to make the eventual tensor becomes [batch, n_patch_y, n_patch_x, channel, patch_size_h, patch_size_w]. This allow us to use index [batch, patch_idx_y, patch_idx_x] to select out tensor patches of shape [n_patch, channel, patch_size_y, patch_size_x].
After the first permute, it becomes [batch, height, width, channel]. The first unfold works on height dimension and leave the width dimension unchanged, making the tensor [batch, n_patch_y, width, channel, patch_size_h]. The second unfold works on the width dimension, making the tensor [batch, n_patch_y, n_patch_x, channel, patch_size_h, patch_size_w]. You need two unfold to work on two spatial dimensions.
The pytorch unfold function takes patch size, and stride size two parameters to allow cropping of overlapping patches. In this method, we define size to be the central non-overlapping area of each crop, and padding as the overlapping extension. so size + 2 * padding becomes the actual width and height of the crop.

Some additional thoughts.
The implementation in this repository sticks with the primitive functions PyTorch offers. This means in order to perform convolution only on selective patches, we always need to crop out tensor, convolve, then replace in tensor. This involves additional memory copy overhead.
I think it is possible to write a custom Cuda kernel, given the target patch locations, and the Cuda kernel only convolves the patch region without making any additional tensor copies. It would be interesting if someone wants to experiment it.

Thanks @PeterL1n for your reply. Will spend some time on this in the coming week and revert if any more doubts.

Was able to understand how the unfolding works however the idx we have here is the pixel indexes on the original grid. Once we change the grid via unfold to folded regions(with the particular step size) that idx scheme no longer applies.
eg)
size=3
padding=1
x=torch.rand((1,3,320,320))
out=x.permute(0, 2, 3, 1).unfold(1, size + 2 * padding, size) \
                    .unfold(2, size + 2 * padding, size)
out.size()

In [9]: torch.Size([1, 106, 106, 3, 5, 5])
So nonzero points on the original grid of x of 320X320 no longer match to the new grid of 106 X 106.
According to this the indexing with idx is not correct. Is something being missed here?

First of all, you missed a padding op. See https://github.com/PeterL1n/BackgroundMattingV2/blob/4a56223a1cd9b2c2678582513c573debbfc12cae/model/refiner.py#L203
Second, 106 is right because 320 / 3 roughly equals 106. 3 is a bad choice because 320 is not really divisible by 3. Let change it to 2 for example. It should be:
size = 2
padding = 1
x = torch.randn((1, 3, 320, 320))
x = torch.nn.functional.pad(x, (padding, padding, padding, padding))
x = x.permute(0, 2, 3, 1) \
        .unfold(1, size + 2 * padding, size) \
        .unfold(2, size + 2 * padding, size)
print(x.shape) # torch.Size([1, 160, 160, 3, 4, 4])

To select patches, you will need a index of 160x160:
idx = torch.rand((1, 160, 160)).gt(0.5) # bool tensor
x = x[idx] # shape: (num_patches, 3, 4, 4)

The above code calculates nonzeros internally at x[idx], if you need to use idx to select multiple times, it is faster to calculate nonzeros yourself ones and reuse it elsewhere.
idx = torch.rand((1, 160, 160)).gt(0.5) # bool tensor
idx = torch.nonzero(idx) # shape: (num_patches, 3)
x = x[idx[:, 0], idx[:, 1], idx[:, 2]] # shape: (num_patches, 3, 4, 4)

Thanks @PeterL1n, yes totally missed the importance of the padding step. Will go through this more throughly and reply with further questions.


[No QA] Configure MapBox separately from node

Details
The current configuration of MapBox in setupNode works fine for internal employees. However, for contributors working from forks, it does not work because they do not have access to the MAPBOX_SDK_DOWNLOAD_TOKEN repo secret.
Therefore setupNode (and in turn tests, lint, etc...) are failing for all PRs from forks for now.
This PR doesn't address the root problem – we can't configure MapBox or run native builds from forks – but it does reduce the impact of the problem substantially.
After we merge this PR, the main limitation will be that we can't run AdHoc test builds of PRs from forks. This is common workflow, so is something we'll want to fix as soon as we reasonably can. In the meantime, Expensify engineers can work around this by:

Checking out the PR from a fork. I use the gh cli to do this like gh pr checkout 12345
Create a local branch from the forked branch, and push it. git switch -c my-copy-of-12345 && git push origin my-copy-of-12345
Then create a PR from that branch gh pr create --web
Then you can add the Ready to Build label to your copy PR, and run AdHoc builds from there.

Fixed Issues
$ n/a – all contributor PRs are currently borked
Tests
We've gotta merge + live-test this.

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

Offline tests
None.
QA Steps
None.

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

PR Author Checklist

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

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

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

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

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

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

I think we also need to call the setup script in .github/actions/composite/buildAndroidAPK/action.yml

This is the PR that introduced Mapbox and modified GHworkflows https://github.com/Expensify/App/pull/25506/files
I should have shared this earlier 😓

Reviewer Checklist

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

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

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

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

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Yes, very strange since everything passed here

I see, these are failing because they use the setupNode action from main, which expects the secret token, but this PR stops passing that token to setupNode. You can see this unrelated test PR has tests passing fine, because the secret is accessible from PRs in the base repo.

I see, these are failing because they use the setupNode action from main, which expects the secret token, but this PR stops passing that token to setupNode. You can see https://github.com/Expensify/App/pull/25653 has tests passing fine, because the secret is accessible from PRs in the base repo.

wow good find! is there a way to merge this with tests failing or do we have a way to make tests succeed?

I think we need to merge it with tests failing.

Technically we might be able to fix this type of error more generally by using actions/checkout and relative paths to actions rather than using actions from @main, but sometimes its useful that actions are taken from @main so I'm not sure what's really a best practice.
For now I think we should just merge + live-test

I think we also need to call the setup script in .github/actions/composite/buildAndroidAPK/action.yml

I think we have to do this ⬆

Oh, I just ran the scripts before calling that action:

here, and
here

That way we don't have to worry about passing the secret to the composite action as an input. I can change this if you feel I should, either way works

Ok, lets do this. Merging through failing tests as explained above

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.56-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/roryabraham in version: 1.3.56-24 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.58-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


So we already have the proto file in https://github.com/census-instrumentation/opencensus-experiments/tree/master/integration/src/main/proto, it was moved there because the Java code needed it in that location. No need to commit it again :)

I added the generated Go file

I added the generated Go file

@ramonza it already exists in https://github.com/census-instrumentation/opencensus-experiments/blob/master/integration/src/main/proto/defs.pb.go.
Did you use the Makefile generation directive or was this by hand?

I ran "make test" and it was generated to this location

That's weird, I think the generated pb.go should be under src/main/proto/? (https://github.com/census-instrumentation/opencensus-experiments/blob/master/integration/Makefile#L5)
Did you run against the latest Makefile?

@songy23 the proto directive is broken but in deed, @ramonza it shouldn't generate that file in anyway, unless that's using older code before @songy23's commit.
@ramonza, please remove that protofile from your commit, I'll send a separate PR to fix that directive and the latest file.

I see - BTW all the go files are using the pb.go under src/main/proto, e.g https://github.com/census-instrumentation/opencensus-experiments/blob/master/integration/go-grpc-client/interop_test.go#L31.

I've sent https://github.com/census-instrumentation/opencensus-experiments/pull/7, sorry for the noise @ramonza.


Add more info for failed scenarios

When there are scenarios mismatch, this will now show the expected and actual results in the output.
This PR is for merging with the performance-updates-on-top-of-v906-release.
Changes

Added expected and actual results for message output
same code fixes as https://github.com/cfpb/owning-a-home-api/pull/90 but on different branch

Review

@fna

Closing this PR as these changes have already been merged into master in 9ab3e70b.


Feature: remove the existing orderbook when receiving a signal from the relayer

Description
This change allows the broker to reset its orderbook when indicated by the Relayer. This avoids problems where old events get stuck in the broker's orderbook with no way to remove them.
To accomplish this, the Orderbook itself needed to be rebuilt from the event store on Broker start up to avoid stale orders from accumulating.
The Relayer's associated change should be deployed before this one, since it depends on changes to the Relayer proto definition.
Todos

[x] Tests
[ ] Documentation
[ ] Link to Trello

@treygriffith these changes LGTM.


oauth2 with Mobileclient

Hi
I am using two step authentication for my google account.
Using the documentation I successfully configured Musicmanager.
I am trying to also setup Mobileclient with oauth2, but I am lost in the process of creating my credentials.
Is there any way to create the oauth token for Mobileclient the same way I created it for Musicmanager?
Thank you

Unfortunately, the mobileclient does not support oauth right now. https://github.com/simon-weber/gmusicapi/issues/426 is the ticket to watch for that. Instead, you'll need to set up an application password and use that.

Thank you @simon-weber
Meanwhile I managed to create an application password and it works.


Link to "The Guide" is broken

Would be nice if that worked instead :grinning:

Yes, indeed. :smiley: Fixed it now, thanks for reporting!


Code-splitting with require.ensure: wrong path

How to make webpack write in the code the real chunk path? I mean this line:
https://github.com/designeng/base-webpack/blob/multiple/public/build/bundle.js#L76
I did not found any documentation about it.
If my index.html and bundles in the same folder, it's ok. The problems begin when I try to build in another folder.
My example here:
https://github.com/designeng/base-webpack/tree/multiple

Sorry, it works with publicPath option:
http://webpack.github.io/docs/configuration.html#output-publicpath


Add update guide

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Current Behaviour
There is no update guide and no information how to upgrade CVAT in installation guide.
Possible Solution
Need to add Update Guide (or corresponding section to installation documentation)
Steps to Reproduce (for bugs)
N/A
Context
N/A
Your Environment
N/A
Next steps
N/a

Hi, any news on this one?

@Shravan1804 https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/faq.md#how-to-update-cvat

@azhavoro , thanks for the hint.
Regarding backups, the FAQ only says

Before upgrading, please follow the official docker manual and backup all CVAT volumes.

Maybe they are related to each task?

~/cvat$ docker volume inspect fefbb2c80c666ea2e82b3217b35b7f1d78675ffda865ef4ced91320674c56b57
[
    {
        "CreatedAt": "2020-06-03T14:00:49Z",
        "Driver": "local",
        "Labels": null,
        "Mountpoint": "/var/lib/docker/volumes/fefbb2c80c666ea2e82b3217b35b7f1d78675ffda865ef4ced91320674c56b57/_data",
        "Name": "fefbb2c80c666ea2e82b3217b35b7f1d78675ffda865ef4ced91320674c56b57",
        "Options": null,
        "Scope": "local"
    }
]

A bit offtopic: all the volumes are pointing to "/var/lib/docker/volumes/... would be enough to backup the whole "/var/lib/docker/volumes/ folder and replace it in case of a loss? The Docker manual is not clear to me yet, but I'm new to the topic so maybe I have to keep on learning.

@aguadopd did you manage to take the backups? I also don't know which volumes we should backup.

@kraster010 No, and I guess we are not going to do it soon since we have a lot of valuable data now :rofl: .
I gues it's safe to update, anyway, as stated in https://github.com/openvinotoolkit/cvat/issues/2114#issuecomment-686397693  .
We have done it before without problem when we were starting with CVAT. Now it's a harder decision maybe.

@aguadopd did you manage to take the backups? I also don't know which volumes we should backup.

@kraster010 all volumes associated with CVAT are prefixed with cvat, you should backup *cvat_cvat_data and cvat_cvat_db at least. Also yes, you can backup it by copying folder, but stop all cvat containers before.

@Shravan1804 https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/faq.md#how-to-update-cvat

Hi, how do I update it? This link is 404 error and doesn't work anymore.

@Shravan1804 https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/faq.md#how-to-update-cvat

Hi, how do I update it? This link is 404 error and doesn't work anymore.

Check out new docs at https://openvinotoolkit.github.io/cvat/docs/for-users/faq/#how-to-update-cvat

Thanks, will it work if I’ve been using the docker file version 2.3 (cvat projects not available in that version) - with cvat repo I cloned about a year ago.. and then update to the latest cvat repo with dockerfile version 3.3.
Will the data backup and restore still work?
Cheers

From: Pablo Aguado @.>
Sent: Tuesday, June 22, 2021 4:06:57 AM
To: openvinotoolkit/cvat @.>
Cc: Sam Davidson @.>; Comment @.>
Subject: Re: [openvinotoolkit/cvat] Add update guide (#1557)
@Shravan1804https://github.com/Shravan1804 https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/faq.md#how-to-update-cvat
Hi, how do I update it? This link is 404 error and doesn't work anymore.
Check out new docs at https://openvinotoolkit.github.io/cvat/docs/for-users/faq/#how-to-update-cvat
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/openvinotoolkit/cvat/issues/1557#issuecomment-865159700, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AK7R4HFYUA3K7USTXMEKSWTTT5PSDANCNFSM4NE2FURA.

@Shravan1804 https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/faq.md#how-to-update-cvat

Hi, how do I update it? This link is 404 error and doesn't work anymore.

Check out new docs at https://openvinotoolkit.github.io/cvat/docs/for-users/faq/#how-to-update-cvat

The link is not working... 404 not found

Now you can find docs at https://openvinotoolkit.github.io/cvat/docs/faq/ :+1:

Here is the current working link:
https://cvat-ai.github.io/cvat/docs/faq/#how-to-update-cvat

its not working unfortunately

@nikste , https://opencv.github.io/cvat/docs/faq/#how-to-update-cvat


Refactor and update verification details on quickstarts

Rancher (verify --only)
Openshift (verify --only and ServiceExport)

Thanks for all the comments and suggestions

Sorry I have additional comments after testing the command.

Note, this needs [SQUASH] on merge.


some clarifications and whitespace fixy-fix

small tweaks to the readme that i think would be helpful

This change is

Thanks! Merged to master as of 0dd44707fa25


new db model for color storage with light/dark mode pairs

[ ] new activityset schema
[ ] db migration
[ ] add new default colors
[ ] update color chip display to support dual colors
[ ] update activity token display to support dual colors
[ ] update timeline/minitimeline/richtimeline to support dual colors
[ ] update activity slat to support dual colors
[ ] update activity toggle to support dual colors

I'm including the activity slat appearance update into this - cause the changes to this will affect that a fair bit too.


Consider removing LLVM dependency

If I edit CMakeLists.txt to read:
cmake_minimum_required(VERSION 3.22)

and then run
cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo -GNinja ..
ninja

I see
/z/ctbench/grapher/lib/grapher/plotters/debug.cpp:1:10: fatal error: llvm/Support/raw_ostream.h: No such file or directory
    1 | #include <llvm/Support/raw_ostream.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
/z/ctbench/grapher/grapher-utils.cpp:1:10: fatal error: llvm/Support/CommandLine.h: No such file or directory
    1 | #include <llvm/Support/CommandLine.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

Is it possible to remove this LLVM dependency in favour of using the C++ STL instead? If not, is it possible to make the LLVM linking mechanism more robust?
https://github.com/openjournals/joss-reviews/issues/5165

Hello,

Is it possible to remove this LLVM dependency in favour of using the C++ STL instead?

The LLVM can't be dropped so easily as the CLI code relies heavily on LLVM's CommandLine support library, which doesn't have any equivalent in the STL. It is also required for symbol demangling in the compare_by plotter.

If not, is it possible to make the LLVM linking mechanism more robust?

I'm not sure what you mean by more robust? The find_package command is called with the REQUIRED tag (here: https://github.com/JPenuchot/ctbench/blob/main/cmake/dependencies.cmake) and the grapher target is being linked against the export target exposed by LLVM (here: https://github.com/JPenuchot/ctbench/blob/main/grapher/cmake/grapher-target.cmake). I'm very surprised that CMake didn't catch any error and I'm suspecting that Ubuntu's way of splitting packages might be responsible for that.
It seems like most of your issues are due to the project not being built on Ubuntu as I use Arch for development, so I will likely set up an additional GitHub Action to check that ctbench compiles well on a recent Ubuntu system. I'll investigate that next week and come back to you, thanks for bringing that up.

Hello, this issue was not encountered on the Ubuntu 23.04 CI and might be related to a missing package on your system, maybe libllvm15 or llvm-dev? I'll be waiting for your feedback.

I'm closing the issue as it was resolved for Ubuntu 23.04, feel free to reopen if I missed something.


Using flow with withRouter and other higher order components

Hi,
I was trying to use flow simply as a replacement for react proptypes as a start. When my component is wrapped in 'withRouter' hoc flow doesn't throw errors as it should. I've searched for hours and haven't found a solution/tutorial that I could simply apply within a short amount of time I've had. Could anyone please help me? It drives me crazy... :) Thanks
Cheers,
Chris
Here is very simplified snippet:
https://flow.org/try/#0JYWwDg9gTgLgBAJQKYEMDG8BmUIjgcilQ3wChSB6CuAGyRgGc5QUBzYAOyThgAtgmA5uGgwkAEzjZcBIuhgBaHAFcxUBeNxk68AO7A+CCKqRQ4AXjgAKAMK5IXDjACUFgHzWwOMA1fmPADx2Io7wAN4AdFFeED4AvhRu5DAAnmDcAAreTJZhpHBw+uJ8AFxwHMogAEampHHkOnB2Tiicphae2WVZsb7ucHkFaBAcDOH5BYXAxbwdAAwTcR0xPuQFRDDKUBzWEwUB4sAAbnBjKXTmYWFFfHFxbmIAHjABFIdHSQXOdQ30TSMwVpcKAAdQMvCMJjMln0hmMalsAKBpm+pEaYjGAEYOkEkW0oIk4FQeLwcLomCgdqYcFA0X8MTAAEw45qA-FguFQwnEhi8Yw0SR8MlwSlwak00hAA

The HoCs need to be typed, otherwise they are typed implicitly as any. Here is the documentation on typing HoCs. Here is the flow-typed definition for withRouter, as part of a larger definition file of React Router v4.

I used flow-typed before and now after your response I tried it again. Turns out flow wasn't picking up those definitions unless i explicitly put './flow-typed/*' under [libs] in .flowconfig - is that a normal behaviour?
From docs:

NOTE: Using the /flow-typed directory for libdefs is a convention that enables Flow to JustWork™

Anyway, after finally adding those definitions I'm getting this error -

Which tries to point out that my component doesn't fullfill this definition[react.js:122]:
declare type React$StatelessFunctionalComponent<Props> = {   (props: Props, context: any): React$Node,   displayName?: ?string,   propTypes?: $Subtype<{[_: $Keys<Props>]: any}>,   contextTypes?: any };
My code is pretty much like the flow try snipped in the first post.
Do you have any idea why flow is throwing this error? If I replace '(props: Props)' with '(props: any)' error disappears but I lose all my type definitions...


Great tool for linux users!  I need a way to open two separate instances with different profiles

Thanks again, I use this daily.  The only issue is: I need a way to run o_ne instance_ to hit my 20 or so o365 admin accounts  (this would be set to target the same window when clicking on stuff so I don't get kicked out of the PWA window when going to the Exchange admin portal for example.)
Then another instance that uses the my actual o365 user account for accessing word/excel documents in sharepoint etc. Ideally they would be a different color icon.  I temporarily got this working by installing a much older version that has a different icon, .7 I think, but it errors out on opening files in new windows for some reason.
This never was an issue before as I used to use Ubuntu, and there was a snap that must have been older (or a different developer, as it had a different icon, and then I also ran this version as a Deb and had two functional versions with different icons and sessions. But now I have switched to Debian 12, and won't be installing snap :).  I have tried other electron versions available on github and elsewhere and normal PWAs. But they all have shortcomings. I even used nativfier to make my own, but when I did that the .desktop icon launches the right electron app, but is not connected to the favorited icon, and as such makes 2 icons per instance which just rubs my ocd the wrong way.  Or they target a new window in chrome in a tab which is also annoying.  Your version is perfect, other than the ask herein.
Let me know if you can think of a way
Thanks again!
Bryan.

Interestingly enough I just installed a deb for the last 0.7.3 over 0.7.0 and I can now open in new windows without an error.  There is still an error screen when first opening, but works after that.
Probably not the best way since it is older, but at least it is working for now.
I'm guessing it is working because it has a different name.  I suppose another way I could approach this is to fork it and make one specificaly for "admin.microsoft.com" website instead, with a different icon.

Hi! I am sorry for not replying to this issue, but I need to prioritize my studies at the moment due to personal reasons, and my exams are approaching as well. I'll still try my best to find some time and fix this. I hope you understand.

Hi! Actually, since I do not have much time to spend on this app, I came up with a temporary and quick solution. Basically, in the menu bar, I'll be adding two options, like this:

The first option would open Microsoft 365 Web with your personal account (https://microsoft365.com/?auth=1), and other one would open MS 365 Web with your Work/School account (https://microsoft365.com/?auth=2), which is basically what I do in the "Open Enterprise Version" or "Open Normal version", the difference is that these options change the default page, while this opens them for the current session only.
Would this work?

Little update: Instead of just loading different URLs, I have created two different sessions, for work/school account and personal account, which would totally separate both accounts, even while normal usage. That should do the work. Is that fine?


Fix equating comparisons between MPANs

I'm undecided as to whether this should be a bump from 1.0.2 to 1.0.3 or to 1.1.0 on account of the API sort-of-change.

I noticed today that MPAN(string) ≠ MPAN(string) on account of the fact that I never implemented __eq__() on the MPAN class.  This PR fixes that.

It's a good practise to have a changelog, and for smallish projects, it's common to have the changelog in the readme, especially since it means that the changelog appears on PyPI.
Before I go ahead and merge this, what're your thoughts on:
MPAN(mpan_str) ≠ mpan_str ?
MPAN(mpan_int) ≠ mpan_int ?

and
Should this be a patch bump (1.0.3) or minor bump (1.1.0)?

Should this be a patch bump (1.0.3) or minor bump (1.1.0)?

Minor seems fine to me, this is the sort of thing you would reasonably expect to be implemented already, so it doesn't feel like a new feature per se.


Allow Envoy::OptionsImpl to take a Bootstrap proto object directly

Description:
I'd like to add a method to Envoy::OptionsImpl to specify a Bootstrap proto directly as opposed to using the setConfigYaml/setConfigPath fields, i.e. add something like:
void setConfigProto(const envoy::config::bootstrap::v2::Bootstrap& config_proto)

This would allow clients to directly specify a Bootstrap proto object without having to deal with yaml serialization and would simplify our configuration process since we construct the Bootstrap proto programmatically as a protocol buffer.
Another option might be to have Envoy::OptionsImpl take a text proto, e.g. configTextProto.
While configPath does allow one to provide a text proto via the command line, I'd prefer a route that avoids the file system if possible.
Do any of these changes sound reasonable?

@alyssawilk @htuch what do you think?

This seems fine to me.

Please do the direct typed proto, this is a great idea. I would skip text proto, unless there is a product need for this, since it is not a stable format and we're trying to encourage folks to stick to YAML as the canonical text format in Envoy.

SG, I will send a PR soon.

+1, please expose the bootstrap protobuf from OptionsImpl as well, as in our user story, there are components which read from bootstrap config.


Implement Floating Shortcut Panel for Android

This PR implements a Shortcut Panel for Android. The toolbar floats above the UI, with buttons that trigger key events (e.g., undo and redo actions).

It includes a drag handle for repositioning.
Initially only undo and redo buttons are available. However it gives an option to edit and add extra shortcut button.

I really like it!
There needs to be something to prevent you from having the same setting multiple times.

We should also not forget to translate the shortcuts

There needs to be something to prevent you from having the same setting multiple times.

Restricted assigning the same shortcut to multiple slots and added an advice panel.

@MewPurPur I updated the shortcut selection panel to use the shortcut description instead of the shortcut name and aligned it to the left. Also, updated the translation files.

Clear button is needed.
Used shortcuts should be disabled in the popup and also highlighted in red when being typed.
The new icons should use the "#def" fill color which is standard for GodSVG.
The config option shouldn't be the gear with a black outline - that one is used in color swatches because they can have arbitrary colors, but is not appropriate here.

Now it's pretty good. I think the options should be disabled rather than removed from the popup.
Now there's the issue that the panel gets hidden when you open menus. We can leave it for later.
Make the dragging icon bigger to match its actual size and avoid upscaling artifacts (I'd suggest having 4x2 dots also)

@MewPurPur Options are now disabled instead of being removed entirely. I also increased the icon size and eliminated upscaling.