Leak resulting from RxBleClient.scanBleDevices()

I've been pulling my hair out trying to debug a leak in my app when using RxBleClient.scanBleDevices().  My subscribe onNext lambda holds a reference to the Activity, however I am unsubscribing when the Activity is destroyed, so I shouldn't be leaking.  The leak's GC root is a thread, which references com.polidea.rxandroidble.internal.radio.FIFORunnableEntry.emitter.
Maybe I'm just missing something here?  In that case, any insight is greatly appreciated.
I've created an example that demonstrates the issue:
import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;

import com.google.common.base.Strings;
import com.polidea.rxandroidble.RxBleClient;
import com.polidea.rxandroidble.scan.ScanSettings;

import com.example.R;
import com.example.MyApplication;
import rx.Observable;
import rx.Subscription;
import timber.log.Timber;

public class DebugLeakActivity extends AppCompatActivity {

    RxBleClient bleClient;
    Subscription subscription = null;
    String someActivityField;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_debug_leak);

        // in my app, RxBleClient is provided by dependency injection and has the lifetime of the app.
        // simulating here.
        bleClient = ((MyApplication)getApplication()).bleClient;

    }

    @Override
    protected void onStart() {
        super.onStart();

        subscription = scan()
                .subscribe(
                        // reference a field in the activity to create a reference to the Activity from the lambda
                        deviceString -> someActivityField = deviceString,
                        throwable -> Timber.e(throwable));
    }

    @Override
    protected void onDestroy() {

        if (subscription != null) {
            subscription.unsubscribe();
            subscription = null;
        }
        super.onDestroy();
    }

    public Observable<String> scan() {
        ScanSettings scanSettings = new ScanSettings.Builder()
                .setScanMode(ScanSettings.SCAN_MODE_LOW_LATENCY)
                .build();

        return bleClient.scanBleDevices(scanSettings)
                .doOnUnsubscribe(() -> Timber.d("scanBleDevices() unsubscribed"))
                .distinct()
                .doOnNext(scanResult -> Timber.d(String.format("Found BLE device: %s, %s", scanResult.getBleDevice().getMacAddress(), Strings.nullToEmpty(scanResult.getBleDevice().getName()))))
                .map(rxBleScanResult -> rxBleScanResult.toString());
    }
}

Here is the reference chain from LeakCanary:
In com.example:0.20.0-SNAPSHOT:200000.
* com.example.presentation.DebugLeakActivity has leaked:
* GC ROOT thread java.lang.Thread.<Java Local> (named 'Thread-8')
* references com.polidea.rxandroidble.internal.radio.FIFORunnableEntry.emitter
* references rx.internal.operators.OnSubscribeCreate$NoneEmitter.actual
* references rx.internal.operators.OperatorUnsubscribeOn$1.val$subscriber (anonymous subclass of rx.Subscriber)
* references rx.internal.operators.OnSubscribeMap$MapSubscriber.actual
* references rx.internal.operators.OperatorMerge$InnerSubscriber.parent
* references rx.internal.operators.OperatorMerge$MergeSubscriber.child
* references rx.observers.Subscribers$5.val$subscriber (anonymous subclass of rx.Subscriber)
* references rx.observers.Subscribers$5.val$subscriber (anonymous subclass of rx.Subscriber)
* references rx.observers.Subscribers$5.val$subscriber (anonymous subclass of rx.Subscriber)
* references rx.internal.operators.OperatorDistinct$1.val$child (anonymous subclass of rx.Subscriber)
* references rx.internal.operators.OnSubscribeDoOnEach$DoOnEachSubscriber.subscriber
* references rx.internal.operators.OnSubscribeMap$MapSubscriber.actual
* references rx.observers.SafeSubscriber.actual
* references rx.internal.util.ActionSubscriber.onNext
* references com.example.presentation.DebugLeakActivity$$Lambda$1.arg$1 (anonymous implementation of rx.functions.Action1)
* leaks com.example.presentation.DebugLeakActivity instance

I've attached the full LeakCanary details for more information:
In 2.txt

Are you sure that the activity is destroyed by the time LeakCanary fires? (I have not used id so I do not know).
One thing you can do is to put a log in .onDestroy() and check if it is available in the logcat before the LeakCanary statement.
The other possibility would be to move unsubscription to an opposite event of .onStart() -> .onStop().
And as a matter of fact even calling .scan() makes a reference to your Activity. :)

Ping?

Thanks for the suggestions.  I did some memory dumps and it seems that in some cases some of the Activities do eventually get garbage collected, but never all.  I'm going to experiment with moving the subscription / unsubscription to .onStart() and onStop() and will fill you in on the results.

Impatiently waiting. :) 👍

Looking at this code snippet:
    @Override
    protected void onStart() {
        super.onStart();

        subscription = scan()
                .subscribe(
                        // reference a field in the activity to create a reference to the Activity from the lambda
                        deviceString -> someActivityField = deviceString,
                        throwable -> Timber.e(throwable));
    }

    @Override
    protected void onDestroy() {

        if (subscription != null) {
            subscription.unsubscribe();
            subscription = null;
        }
        super.onDestroy();
    }

The onStart can be called multiple times before the onDestroy is finally called.
Instead of unsubscribing in the onDestroy, override the onStop instead and unsubscribe there.

I think that @streetsofboston comment may be the cause of the observed leak. Closing for now. If new information will be available—feel free to reopen.


x/tools: optimise runtrybot logic

CUE developers trigger the trybots (GitHub workflows) using the cueckoo command. However it is very basic.

When triggering trybots/unity, we should only do so if we haven't previously triggered them for the same patchset.
When a new patchset is submitted, we should reset the labels on non-trivial rebases.

I believe this has been solved by https://review.gerrithub.io/c/cue-sh/tools/+/1169892, and any further work in this space will be solved by switching to using labels to trigger trybots/other, and webhooks hanging off GerritHub to "see" the edge of a label being added.
cc @mvdan to confirm one way or the other, and close if required please!


Add ONNX runtime inference adapter

What does this PR do?
This PR adds ONNX runtime -based inference adapter. Currently tested only with multilabel classification model
onnx_adapter = ONNXRuntimeAdapter("model.onnx")
model_onnx = ClassificationModel.create_model(onnx_adapter)
prediction = model_onnx(image)

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

This PR misses a lot of details, but it showcases the approach: use python preprocessing and onnx metadata.
@Wovchena @AlexKoff88 please have a look if non-OV adapter is acceptable from the overall conception.
Unfortunately, some OV stuff leaked to implementation of particular models (like add softmax for classification), and this should be addressed in a modelwise manner.

What's the reason to have onnx backed? You've already enabled inference by https://github.com/openvinotoolkit/model_api/pull/106

What's the reason to have onnx backed? You've already enabled inference by #106

Some of the Geti customers prefer to use ONNX runtime rather than OV for inference on their side

Can you add a note that people shouldn't expect it to work in every case, so I could ignore bugs related to this.

Can you add a note that people shouldn't expect it to work in every case, so I could ignore bugs related to this.

Sure, I'll add this.

For my reference, who needs ORT inference and for which workloads?

Need a couple of tests that verify that it works as expected to the supported cases and throughs an exception for the unsupported model types.

Need a couple of tests that verify that it works as expected to the supported cases and throughs an exception for the unsupported model types.

The tests were added, similar to other adapters. Unfortunately, the inference results are slightly different than ones obtained with OV. Since we can't change anything in the inference process itself, the next step would be to compare OV vs python preprocessing. We already faced that problem when upgraded from old OMZ model wrappers in OTX. I'm planning to add testcases that measure the exact difference.
@Wovchena it seems like we can't add implicit assertions to prohibit usage of ORT adapter with untested wrappers, because those model wrappers should know about ORT or any other adapters (in case of a whitelist). Maybe I overlooked an obvious solution though.

Model's __init__() takes inference_adapter. You can try verifying the type for each subclass.

Model's __init__() takes inference_adapter. You can try verifying the type for each subclass.

Yep, that's a possible option, but we have to import all wrappers in model.py. Is that fine?

You can perform the check in each subclass

@AlexKoff88 @Wovchena the tests & fixes are ready. Please have a look one again if this can be merged

I am ok with the changes


why using delta_RS

Hello
In the file ranking_losses.py, I am a little confused about the following codes.
if delta_RS > 0:
fg_relations=torch.clamp(fg_relations/(2delta_RS)+0.5,min=0,max=1)
bg_relations=torch.clamp(bg_relations/(2delta_RS)+0.5,min=0,max=1)
else:
fg_relations = (fg_relations >= 0).float()
bg_relations = (bg_relations >= 0).float()
I do not quite understand why you used delta_RS and what is the bias number 0.5 used for. It seems to me that when you add 0.5, it will make some negative logits to be positive. Does it make sense?
Thanks

Hi,
Thanks for your interest in our work!
Basically, $\delta_{RS}$ aims to ensure a margin among examples for better generalization. To provide more insight based on your example, when we add $0.50$ to a negative example with a logit of $s$, then the network should assign a logit to a positive at least $0.50+s$ in order to have zero error originating from that pair  (i.e. the network should ensure that the score of this positive is at least larger than the negative with logit of $x+0.50$ to minimize error.). For the same purpose, it has been adopted by AP Loss and aLRP Loss as well. In addition to these works, with RS Loss, we also want to enforce a margin among positives with $\delta_{RS}$.
Regarding the code snippet you shared; "if" part implements the smoothed step function (please see Fig. 4 and Eq. (21) in https://arxiv.org/pdf/2008.07294.pdf) and the "else" part basically aims to prevent the division-by-zero error when $\delta_{RS}=0$ (i.e. $\delta_{RS}$ is a multiplier of the denominator). Also, note that the "else" part corresponds to step function without any smoothing (dashed curve in Fig. 4 of https://arxiv.org/pdf/2008.07294.pdf).
Hope it is more clear now.

Thanks for your prompt reponse. I have one more question. I think the logit is the input the sigmoid funtion but I wonder if it makes any difference if we use the output of the sigmoid function as input to the ranking loss.

Yes, you are correct about what I mean by "logit".
In fact, I have also thought about using the normalized logits after sigmoid (I will call them "probabilities" considering that they are between [0,1]) but never tried. That's why, I can only make some "rational" guesses:

In that case, I think you have to, at least, tune $\delta_{RS}$ (i.e.  $0.50$ seems to be a large margin for the [0,1] range of probabilities).
There should not be any difference between ranking logits or probabilities. However, I am not perfectly sure because when we use sigmoid then the range of the raw values is squeezed within [0,1]. Maybe, this limited range can have an impact.

So, by validating $\delta_{RS}$, I think it can be worth to try. If you do so, please let me know as well :)

Thanks a lot. I will definitely let you know after I get some results.


Crashes at Basic Protocol 3 step 6 "Save/restore detections"

Hello, I go through the basic protocols right now and it crashes when I try to reload the ROI I set before:
Started Mag with:
python run.py --img sample_region.czi --roi_profile lightsheet --offset 30,35,30 --size 25,25,12
I tried to save a second ROI, so the ROI list would be longer, so maybe the "out-of-range" doesn't match but it gave me the same result.
Full output:
Set filenames to ['/home/tgottsch/Atlas_project/magellanmapper/examples/sample_region/sample_region.czi'], current filename /home/tgottsch/Atlas_project/magellanmapper/examples/sample_region/sample_region.czi
Set ROI offsets to [(30, 35, 30)], current offset (30, 35, 30) (x,y,z)
Set ROI sizes to [(25, 25, 12)], current size (25, 25, 12) (x,y,z)
Updating ROI profile for channel 0
Set default (channel 0) ROI profile: default,lightsheet
Set atlas profile to default
setting up Matplotlib style default
applying theme DEFAULT
Loaded database from magmap.db
set multiprocessing start method to fork
Loading main image
filename: /home/tgottsch/Atlas_project/magellanmapper/examples/sample_region/sample_region.czi
filename_base: /home/tgottsch/Atlas_project/magellanmapper/examples/sample_region/sample_region
Reading image metadata from /home/tgottsch/Atlas_project/magellanmapper/examples/sample_region/sample_region_meta.npz
unable to load scaling from archive, will ignore
unable to load plane from archive, will ignore
loaded image5d version number 14
names: ['sample_region']
sizes [[  1  51 200 200   2]]
set resolutions to [[5.488412   1.13827774 1.13827774]]
magnification: 5.0
zoom: 0.8
set near_min to [236. 198.]
set near_max to [4374.  902.]
Set vmax_overview to [4811.4  992.2]
image5d shape: (1, 51, 200, 200, 2)
set border to zeros
set border to [0. 0. 0.]
setting up Matplotlib style seaborn
applying theme DEFAULT
stale {<ViewerTabs.ROI_ED: 1>: True, <ViewerTabs.ATLAS_ED: 2>: True, <ViewerTabs.MAYAVI: 3>: True}
got None selected
no roi found
Changed channel to [0, 1]
x: 30, y: 0, z: 0
x: 30, y: 35, z: 0
x: 30, y: 35, z: 30
camera: {'view': (0.0, 0.0, 1.0, array([0., 0., 0.])), 'roll': 0.0}
view: (75.00000000000001, 140.0, 100.0, array([0., 0., 0.]))
roll: -175.0
using ROI offset [30, 35, 30], size of [25 25 12] (x,y,z)
Initiating ROI Editor
margin: None, savefig: png
zoom_levels: [1.         1.69947907] max_zoom: 2.6666666666666665
2D border: [0. 0. 0.]
z_overview: 30
plane xy downsampling factors by image: [1, 1, 1]
plane xy downsampling factors by image: [1, 1, 1]
2D plot time: 0.4261758327484131
findfont: Font family ['sans-serif'] not found. Falling back to DejaVu Sans.
findfont: Font family ['sans-serif'] not found. Falling back to DejaVu Sans.
got offset (30,35,30) of size (25,25,12)
got None selected
no roi found
Changed channel to [0]
Exception occurred in traits notification handler for object: <magmap.gui.visualizer.Visualization object at 0x7fb3f97138e0>, trait: rois_check_list, old value: None selected, new value: offset (30,35,30) of size (25,25,12)
Traceback (most recent call last):
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 522, in _dispatch_change_event
    self.dispatch(handler, *args)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 484, in dispatch
    handler(*args)
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/gui/visualizer.py", line 2306, in load_roi
    blob_matches = config.db.select_blob_matches(roi_id)
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/io/sqlite.py", line 905, in select_blob_matches
    return self._parse_blob_matches(self.cur.fetchall())
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/io/sqlite.py", line 887, in _parse_blob_matches
    df = df_io.dict_to_data_frame(rows, records_cols=rows[0].keys())
IndexError: list index out of range
Traceback (most recent call last):
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traitsui/qt4/check_list_editor.py", line 129, in update_object
    self.value = value
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traitsui/editor.py", line 731, in _set_value
    self.ui.do_undoable(self.__set_value, value)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traitsui/ui.py", line 658, in do_undoable
    action(*args, **kw)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traitsui/editor.py", line 685, in __set_value
    method(self.ui.info, self.object, name, value)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traitsui/handler.py", line 286, in setattr
    setattr(object, name, value)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 476, in __call__
    self.notify_listener(self, object, trait_name, old, new)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 550, in _notify_method_listener
    object, trait_name, old, new, listener
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 530, in _dispatch_change_event
    handle_exception(object, trait_name, old, new)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 148, in _handle_exception
    raise excp
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 522, in _dispatch_change_event
    self.dispatch(handler, *args)
  File "/home/tgottsch/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 484, in dispatch
    handler(*args)
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/gui/visualizer.py", line 2306, in load_roi
    blob_matches = config.db.select_blob_matches(roi_id)
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/io/sqlite.py", line 905, in select_blob_matches
    return self._parse_blob_matches(self.cur.fetchall())
  File "/home/tgottsch/Atlas_project/magellanmapper/magmap/io/sqlite.py", line 887, in _parse_blob_matches
    df = df_io.dict_to_data_frame(rows, records_cols=rows[0].keys())
IndexError: list index out of range
Aborted

Update: I tried the same procedure on another OS (Ubuntu 20 instead of Debian 8) using a fresh installation. Result stays the same but the error changes:

traits - ERROR - Exception occurred in traits notification handler for object: <magmap.gui.visualizer.Visualization object at 0x7f0e7dd15eb8>, trait: rois_check_list, old value: None selected, new value: offset (30,35,30) of size (25,25,12)
Traceback (most recent call last):
  File "/home/nrd/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 524, in _dispatch_change_event
    self.dispatch(handler, *args)
  File "/home/nrd/anaconda3/envs/mag/lib/python3.6/site-packages/traits/trait_notifiers.py", line 486, in dispatch
    handler(*args)
  File "/home/nrd/magellanmapper/magmap/gui/visualizer.py", line 2598, in load_roi
    self.detect_blobs(segs=blobs, blob_matches=blob_matches)
  File "/home/nrd/magellanmapper/magmap/gui/visualizer.py", line 2158, in detect_blobs
    self.redraw_selected_viewer(clear=False)
  File "/home/nrd/magellanmapper/magmap/gui/visualizer.py", line 1937, in redraw_selected_viewer
    self._launch_roi_editor()
  File "/home/nrd/magellanmapper/magmap/gui/visualizer.py", line 2374, in _launch_roi_editor
    *stack_args, **stack_args_named, zoom_levels=2)
  File "/home/nrd/magellanmapper/magmap/gui/roi_editor.py", line 905, in plot_2d_stack
    cmap_labels=cmap_labels)
  File "/home/nrd/magellanmapper/magmap/gui/roi_editor.py", line 1309, in show_subplot
    self.blobs.blob_matches.get_blobs(1),
  File "/home/nrd/magellanmapper/magmap/cv/colocalizer.py", line 106, in get_blobs
    if col.value not in self.df:
TypeError: argument of type 'NoneType' is not iterable
Unhandled Python exception
Aborted (core dumped)

Thanks for finding this, @SaibotMagd ! I reproduced it on my end as well, a regression from a new feature added since the Current Protocols. I had partially fixed it (the error in your first stacktrace) but evidently not completely...thanks for reporting it.
I pushed a fix in PR #12 . If you are on Git, you can access it by grabbing the branch:
git fetch; git checkout fix_absent_blob_matches
Please let me know if it works for you and I can merge it.

It seems to work pretty well. I checked the reloading of cell detection after switch to another roi and the reload after complete restart of mag. Both works great. Thanks alot!

Great, thanks for testing it and glad it worked for you!


Updated Broken Link for Skeleton screen blog

The link to skeleton screens blog was broken and no longer available. The new resource link provided has more helpful and reliable information.
Note: I have no affiliation with the associated link. I just found it to be better than everything else I could search for.

I signed it!

I've reached out to @HannahAtkin on Twitter and will try to get it fixed there first. If it's not fixed in the next week or so, I'll switch it out.

https://twitter.com/HannahAtkin/status/897912190778212352 - will check next week.

Hi Pete,
The link should be working now. Please let me know if you have any trouble
with it.
Hannah
On Thu, Aug 17, 2017 at 11:11 AM, Pete LePage<EMAIL_ADDRESS>wrote:

https://twitter.com/HannahAtkin/status/897912190778212352 - will check
next week.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/WebFundamentals/pull/4947#issuecomment-323088055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALcKLUho-oSzbCoj-86rnjlTmMMoki5yks5sZEw_gaJpZM4O5TPL
.

Thanks @hannahatkin - looks like the URL changed though. I'll push another PR with the updated url.


Build fails with "as.exe" file not found

I've tried tried several ways to compile a firmware for a ESP32-C6 ("Super Mini" ali board with 4MB flash), however it constantly fails with the following error:
Executing task: C:\Users\juerg\.platformio\penv\Scripts\platformio.exe run

Processing ESP32C6 (framework: arduino; platform: https://github.com/pioarduino/platform-espressif32/releases/download/stable/platform-espressif32.zip; board: esp32-c6-devkitm-1)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/esp32-c6-devkitm-1.html
PLATFORM: Espressif 32 (53.3.10) > Espressif ESP32-C6-DevKitM-1
HARDWARE: ESP32C6 160MHz, 320KB RAM, 4MB Flash
DEBUG: Current (cmsis-dap) External (cmsis-dap, esp-bridge, esp-builtin, esp-prog, iot-bus-jtag, jlink, minimodule, olimex-arm-usb-ocd, olimex-arm-usb-ocd-h, olimex-arm-usb-tiny-h, olimex-jtag-tiny, tumpa)
PACKAGES:
 - framework-arduinoespressif32 @ 3.1.0
 - framework-arduinoespressif32-libs @ 5.3.0+sha.083aad99cf
 - tool-esptoolpy @ 4.8.5
 - tool-mklittlefs @ 3.2.0
 - tool-riscv32-esp-elf-gdb @ 14.2.0+20240403
 - tool-xtensa-esp-elf-gdb @ 14.2.0+20240403
 - toolchain-riscv32-esp @ 13.2.0+20240530
LDF: Library Dependency Finder -> https://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 42 compatible libraries
Scanning dependencies...
Dependency Graph
|-- PubSubClient @ 2.8.0
|-- ArduinoOTA @ 3.1.0
|-- WiFi @ 3.1.0
Building in release mode
Compiling .pio\build\ESP32C6\src\common-functions.cpp.o
Compiling .pio\build\ESP32C6\src\main.cpp.o
Compiling .pio\build\ESP32C6\src\mqtt-subscriptions.cpp.o
Compiling .pio\build\ESP32C6\src\setup.cpp.o
Compiling .pio\build\ESP32C6\src\user_setup_loop.cpp.o
Building .pio\build\ESP32C6\bootloader.bin
Generating partitions .pio\build\ESP32C6\partitions.bin
Compiling .pio\build\ESP32C6\lib94a\PubSubClient\PubSubClient.cpp.o
esptool.py v4.8.5
Creating esp32c6 image...
Merged 2 ELF sections
Successfully created esp32c6 image.
Compiling .pio\build\ESP32C6\libab8\Network\NetworkClient.cpp.o
Compiling .pio\build\ESP32C6\libab8\Network\NetworkEvents.cpp.o
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\src\mqtt-subscriptions.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\lib94a\PubSubClient\PubSubClient.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\libab8\Network\NetworkEvents.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\src\user_setup_loop.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\src\main.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\src\common-functions.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\src\setup.cpp.o] Error 1
riscv32-esp-elf-g++: fatal error: cannot execute 'C:/Users/juerg/.platformio/packages/toolchain-riscv32-esp/bin/../lib/gcc/riscv32-esp-elf/13.2.0/../../../../riscv32-esp-elf/bin/as.exe': CreateProcess: No such file or directory
compilation terminated.
*** [.pio\build\ESP32C6\libab8\Network\NetworkClient.cpp.o] Error 1
========================================================================== [FAILED] Took 12.58 seconds ==========================================================================

Environment    Status    Duration
-------------  --------  ------------
ESP32C6        FAILED    00:00:12.585
===================================================================== 1 failed, 0 succeeded in 00:00:12.585 =====================================================================

This is especially strange as i can find the file as.exe in the mentioned path.
My relevant parts of the platformio.ini file:
[env:ESP32C6]
framework = arduino
platform = https://github.com/pioarduino/platform-espressif32/releases/download/stable/platform-espressif32.zip
; Chip type ESP32-C6FH4 (4MB flash on-chip)
; board config file in projects "boards" directory!
board = esp32-c6-devkitm-1
; Use partition table with maximized app space (for 4MB flash); reduces SPIFFS partition to 132kB
board_build.partitions = min_spiffs.csv

I am using this board file in my local boards directory.
I've tried building with the original PlatformIO VSC extension installed as well as PIOarduino. Same result.
Any help highly appreciated.
Thanks,
Juergen

Known limitation of Windows. Caused by Windows path length limit.
Workarounds can be found here https://github.com/arendst/Tasmota/discussions/21964

Oh great, i thought in 2024 we're past such BS.. 🙄
Thanks for the hint, closing case.

In case anyone else stumbles across this issue, i was able to work around it with a relatively simple solution:
a.) Issue the following command in Windows terminal (not elevated, same user you run VSC, adopt to your needs):
subst p: C:\Users\Your_Username\.platformio
b.) add the following to your [platformio] section of platformio.ini:
core_dir = p:\
Compilation for my ESP32-C6 works now.
yours,
Juergen


Google cwebp is not always accessible although downloaded

Actual behaviour
Webserver has no webp-support using google cwebp (therefore the media manager does not convert on upload)
CLI has webp-support using google cwebp
Expected behaviour
CLI and webserver have webp-support using google cwebp
Environment

Server: Plesk + Cloudlinux
PHP 7.2

Steps to reproduce

Have configured php option open_basedir for php-fcgi to reduce access rights
The symfony finder component uses different directories to search
Snap

Checklist

[x] I have checked all issues and haven't found a similar report.
[x] I am using the latest stable version of this plugin.

I don't see here any potential to fix that inside the Plugin. Feel free to reopen when you have a Fix or working configuration for that Plesk + Cloudlinux mixup.


Impossible to disconnect / add / revoke users from the server without restarting it. Statistics on users are also not synchronized without restarting the server

It is not possible to add a new user / disconnect a user / remove a user without restarting the server. When adding / deactivating / deleting, only the web interface works, but the apology does not take effect. Those to the disconnected / removed peer can still be connected, but to the newly added peer, on the contrary, cannot be connected. It only helps to restart the server via guard with the command systemctl restart wg-quick@wg0

User statistics are not synchronized with the user list apology.

How can this problem be solved?

It’s clearly explained in readme


strict-type-predicates incompatible with type unkown

Bug Report

TSLint version: 5.11.0
TypeScript version: 3.1.3
Running TSLint via: CLI

TypeScript code being linted
export const x = (params: unknown) => {
  if (typeof params === 'object') {
    return params
  }
}

with tslint.json configuration:
{
  "extends": [
    "tslint:recommended",
    "tslint-react",
    "tslint-config-standard",
    "tslint-config-prettier",
    "tslint-plugin-prettier"
  ],
  "rules": {
    "member-access": [true, "no-public"],
    "no-floating-promises": false,
    "object-literal-sort-keys": false,
    "prettier": true
  }
}

Actual behavior
Error: strict-type-predicates  Expression is always true.
Expected behavior
No Eror

Agreed! Duplicate of #4107.


Main page gui show detail

Resolve #63
A quick implementation of showing task from GUI.
Hope can help you in implementing your features.
Feel free to overwrite the code and design in this pr.

Thanks for the help, will look through it when coding the GUI aspect

LGTM


Missing  import _tkinter and more

Traceback (most recent call last):
  File "/home/zezen/Downloads/network-analysis-hackathon/network_analysis_gui.py", line 9, in <module>
    import PySimpleGUI as sg
  File "/home/zezen/.local/lib/python3.10/site-packages/PySimpleGUI/__init__.py", line 2, in <module>
    from .PySimpleGUI import *
  File "/home/zezen/.local/lib/python3.10/site-packages/PySimpleGUI/PySimpleGUI.py", line 134, in <module>
    import tkinter as tk
  File "/usr/lib/python3.10/tkinter/__init__.py", line 37, in <module>
    import _tkinter # If this fails your Python may not be configured for Tk

I have tried
pip3 install tkinter
sudo apt-get install python-tk
sudo apt-get install python3-tk
pip3 install Tk interface

but none helped.
Only this seems to work:
sudo apt-get install python3.10-tk
Do also add the missing:
ModuleNotFoundError: No module named 'plotly_express'
I have tried the obvious:
pip3 install plotly_express
but it did not help, so I needed:
pip uninstall plotly_express && python -m pip install plotly_express

not sure why so.
Do add also these to your requirements.txt, after some troubleshooting why it happens.

Even after jumping all these hoops, I am getting:

Hi, sorry for the late reply. My answer might still help you anyway. Just as a heads-up, I don't think any of the people here currently have the time/priorities to maintain this project (it was built over a weekend as a proof of concept).
Anyway, it seems to me that you are installing dependencies in your main python installation on your machine, this is generally discouraged since you end up with a lot of packages that might cause conflicts. For example maybe you want to use this project and it requires numpy version X but pandas only works with numpy version Y.
Maybe try using installing our package using virtual environments (https://docs.python.org/3/library/venv.html).
In case that does not work. Feel free to comment here again in case the virtual environment does not resolve the issues.


Permission of grub/menu.lst is not persistent

Description
/boot/boot/grub/menu.lst permissions are 0755, and setting it to 0644 or 0600 is gone on reboot.
This could also be fixed by enforcing the permission by whatever is resetting it on boot
Impact
Low impact, just that the file is world readable, and doesn't need to be executable
Environment and steps to reproduce

Set-up: tested on 2605.9.0 in qemu
Action(s):
a. chmod 0600 /boot/boot/grub/menu.lst
b. systemctl reboot
c. stat /boot/boot/grub/menu.lst

It's a FAT filesystem and won't store permissions, or?

/boot is a FAT filesystem, it doesn't support permissions. It has to be FAT because that's the EFI boot partition. So we can't fix this.

Reopening: we should mount with the umask=0077 option. The unit is here: https://github.com/flatcar-linux/init/blob/flatcar-master/systemd/system/boot.mount.


exif: Log warning for metadata decode error

The error is logged as a WARNING and nil is returned.
see #8519

@IveGotNorto Is there anyway to include the full path to the offending image? I have updated the minimal example for https://github.com/gohugoio/hugo/issues/8519 to include an additional assets mount (see config.toml). The resulting warning from this PR is:
WARN 2021/06/29 10:32:46 Unable to decode Exif metadata from image: /images/broken.jpg
WARN 2021/06/29 10:32:46 Unable to decode Exif metadata from image: /images/c.jpg

It would be helpful if the warnings were:
WARN 2021/06/29 10:32:46 Unable to decode Exif metadata from image: /foo/images/broken.jpg
WARN 2021/06/29 10:32:46 Unable to decode Exif metadata from image: /assets/images/c.jpg

I don't know is this is possible. What you have done so far is great. Thank you.


semeval16 trial data

Hi, I would like to ask what is the trial in semeval16 in your data, because I can't find trial in the link you gave, only gold test and phase A test and phase B test, is it one of the test as trial or is it something else? I look forward to hearing from you, thank you very much!

Hi,
Below are the links to trial data for Semeval14 and Semeval16
Semeval14:
https://alt.qcri.org/semeval2014/task4/data/uploads/laptops-trial.xml
https://alt.qcri.org/semeval2014/task4/data/uploads/restaurants-trial.xml
Semeval16:
https://alt.qcri.org/semeval2016/task5/data/uploads/trial-data/english-trial/restaurants_trial_english_sl.xml
https://alt.qcri.org/semeval2016/task5/data/uploads/trial-data/english-trial/laptops_trial_english_sl.xml

the difference between phase A and B is in providing annotation in phase B to predict polarity only.

you can use gold annotations

you can use gold annotations
Okey, thank you


macvim: Update to 7.4-75

Weird. Github seems to have screwed up something right now. It looks as if this wasn’t merged and now has merge conflicts, but the cask is indeed updated, and the commit is present.
Well, apart from this weird occurrence with the web interface, everything seems to have gone right, so closing this manually.
Thank you.

Thanks


pythonPackages.ablog: init at 0.8.4

added also the necessary package
pythonPackages.invoke: init at 0.17.0
Motivation for this change
I need to use the package in production, so I added and tested it.
Things done

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

[ ] NixOS
[ ] macOS
[X] Linux

[ ] 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/)
[ ] Fits CONTRIBUTING.md.

Please read the header of the file you modified.

About travis error: I will fix the merge request within few days, applying this time all the necessary test commands, as listed in the error messages. So I hope to be 100% correct the next time. Sorry!
BTW I followed the instructions on http://nixos.org/nixpkgs/manual/#chap-submitting-changes but they are not in par with what I understood here/now. Maybe I will try to update the manual too.

Please read the header of the file you modified.

ABlog is an application / Python cli tool so (if I understood correctly) I need to move away from this module that contains only libraries, and build with buildPythonApplication in the proper module for stand-alone applications.
BTW as partial excuse, in this module there is also Sphinx, from which ABlog is derived, and so I considered (faulty) as the right place. Sorry.
I will update within few days, and I will integrate also the other change requests.

@FRidh I decided to abort the PR because I will not use ABlog in production, and probably this PR is not useful to others.
Many thanks for the review, and sorry for the waste of time. In any case I understood better something about Nix.


post-processing: improve performance

Closing this. It was divided into several recent PRs that have now been merged.


Migrate to Compose: Display map from mobile map package

Description
Migrate Display map from mobile map package to compose

Thanks @shubham7109 -- ready for re-review!

@TADraeseke Seems like the checker is failing, could you fix the issue? Also, I'm able to run the sample without the need for MapViewProxy here.

Thanks @shubham7109 -- my mistake on the proxy! That's been removed and the style checker passing. Thanks!


can't get 'ebooks archive' to authenticate

I was having this problem with an older 2.x version as well. I thought it might have been a ruby 1.9 problem so I upgraded to ruby 2.0, but the problem still persists:
(21:42:56) tape@charles:~/twitter_ebooks/tape> ebooks archive tape
New archive for @tape at corpus/tape.json
/var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/response/raise_error.rb:15:in on_complete': Could not authenticate you (Twitter::Error::Unauthorized)     from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/response.rb:9:inblock in call'
from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/response.rb:57:in on_complete'     from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/response.rb:8:incall'
from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/request/url_encoded.rb:15:in call'     from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/request/multipart.rb:14:incall'
from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/request/multipart_with_file.rb:17:in call'     from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/rack_builder.rb:139:inbuild_response'
from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/connection.rb:377:in run_request'     from /var/lib/gems/2.0.0/gems/faraday-0.9.0/lib/faraday/connection.rb:140:inget'
from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/request.rb:33:in perform'     from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/utils.rb:51:inperform_request'
from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/utils.rb:96:in perform_request_with_objects'     from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/utils.rb:140:inobjects_from_response_with_user'
from /var/lib/gems/2.0.0/gems/twitter-5.13.0/lib/twitter/rest/timelines.rb:49:in user_timeline'     from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/lib/twitter_ebooks/archive.rb:77:inblock in sync'
from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/lib/twitter_ebooks/archive.rb:74:in loop'     from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/lib/twitter_ebooks/archive.rb:74:insync'
from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/bin/ebooks:152:in archive'     from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/bin/ebooks:324:incommand'
from /var/lib/gems/2.0.0/gems/twitter_ebooks-3.0.4/bin/ebooks:341:in <top (required)>'     from /usr/local/bin/ebooks:23:inload'
from /usr/local/bin/ebooks:23:in `'
currently have twitter_ebooks 3.0.4.

Hello,
I have the same issue but can't seem to find the ~.ebooksrc file. Where could it be ?

@Zerhariel it's in Users/YOURNAME/.ebooksrc

In case you've found your way here after creating a new Twitter account and can't figure out why the keys from the app's page don't work—make sure your email is confirmed, then re-generate the token and secret. Then, as mentioned, drop them into ~/.ebooksrc and your Heroku app's environment variables.


Quick Start Guide needs updating

The Quick Start Guide instructions for Client Configuration ( http://aperturetiles.com/docs/development/quickstart/#tile-client-configuration ) need to be updated.  For example, it references the crossplot-maps.json file under WEB-INF/classes/map which doesn't exist in the template.  The tile client template here http://aperturetiles.com/download/#tile-client-template also seems to have layers for both the julia set example and the twitter layer (the twitter layer should probably be removed), and the app.js file loads the world base map, not the extents of the julia set demo.

Cherry picked doc changes into 0.4.0-maintenance branch, and posted the updated Quick Start, Configuration and Download pages to aperturetiles.com along with the new Tile Quick Start Template ZIP download.


Add sensor naming convention

As per title
cc @traversaro

Just a small caveat @pattacini :

Where link is the name of the part the sensor is attached to (e.g., head), type is the type of sensor (e.g., imu), and nr is a number starting from 0.

I would avoid to use part in this context, as part is typically either head, left_arm, right_arm, torso, while in this context link is the link (i.e. either head, chest, etc etc). The fact that head is both a link and a part does not help. :D

Correct!
I'll amend it straight on master.

See https://github.com/icub-tech-iit/documentation/commit/ff2f34a5fb2efad8d27a36c716ffc083fc557dc6.


Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true

Describe the bug
The Kubernetes deployment sets PF_pulsarFunctionsCluster but this doesn't appear to get applied to conf/functions_worker.yml.
Looking at the source, it appears that the script gen-yml-from-env.py is supposed to update functions_worker.yml, but I think the script is missing the value pulsarFunctionsCluster in INT_KEYS  here: https://github.com/apache/pulsar/blob/60d2bfa389815b1769b36a0158059bfc3372a9a0/docker/pulsar/scripts/gen-yml-from-env.py#L30-L38
This is a followup to issue https://github.com/apache/pulsar/issues/2328.

@rocketraman INT_KEYS are used for the settings whose value is Integer. So I don't think pulsarFunctionCluster is an INT key.
You need to specify the function settings by prefixing the settings with "PF_". That says, you need to specify the system variable "PF_pulsarFunctionCluster", so that gen-yml-from-env.py can work as expected.

@rocketraman expanding on @sijie's comment, I can confirm that if you set it as an environment variable, it gets applied. I have added this to my broker configmap for k8s:
apiVersion: v1
data:
  PF_pulsarFunctionsCluster: <clustername>

Then gen-yml-from-env.py  picks it up and applies it to functions_worker.yml.

Hmm, for some reason it did not work for me. I have exactly that (based on the sample in the repo) and yet the broker was still starting up with an error looking for cluster "standalone".  Will try again tomorrow.

I spent more than 12 hours on this...
There are 2 problems here:

The functions worker needs to know the cluster name.
The public/functions namespace needs to have that cluster set.

The list of public/functions clusters is set when the ZooKeeper service is created.
Here is my solution:

zookeeper.yaml

apiVersion: v1
kind: ConfigMap
metadata:
  name: zookeeper-config
data:
...
  PF_pulsarFunctionsCluster: rings
  PF_numFunctionPackageReplicas: "2"
...
      containers:
        - name: zookeeper
          image: apachepulsar/pulsar:2.4.1
          command: ["sh", "-c"]
          args:
            - >
              bin/apply-config-from-env.py conf/zookeeper.conf &&
              bin/apply-config-from-env.py conf/pulsar_env.sh &&
              bin/generate-zookeeper-config.sh conf/zookeeper.conf &&
              bin/gen-yml-from-env.py conf/functions_worker.yml &&
              bin/pulsar zookeeper
...

broker.yaml:

apiVersion: v1
kind: ConfigMap
metadata:
  name: broker-config
data:
...
  functionsWorkerEnabled: "true"
  PF_pulsarFunctionsCluster: rings
  PF_numFunctionPackageReplicas: "2"
...
      containers:
        - name: broker
          image: apachepulsar/pulsar:2.4.1
          command: ["sh", "-c"]
          args:
            - >
              bin/apply-config-from-env.py conf/broker.conf &&
              bin/gen-yml-from-env.py conf/functions_worker.yml &&
              bin/apply-config-from-env.py conf/pulsar_env.sh &&
              bin/pulsar broker
...

I hope this helps you.

@stupidusername  None of the zookeeper conf files have anything related to function worker cluster names in them (nor would I expect them to). I'm not certain why setting PF_pulsarFunctionsCluster for the zookeeper container would solve the issue. I don't even use the zookeeper that comes with Pulsar.

I'm not adding that value to the ZooKeeper configuration. It is being added to functions_worker.yml. Both for the broker and ZooKeeper services.
After reading (https://github.com/apache/pulsar/issues/2328#issuecomment-411013973)[this] I got the idea that maybe there was an error during the creation of the ZooKeeper data.
I tried this workaround and it looks like it's working.
Keep in mind thatPF_ env variales are only applied if you add this line:
bin/gen-yml-from-env.py conf/functions_worker.yml &&

I'd be very surprised if the container running ZooKeeper was responsible for reading functions_worker.yml, and initializing any data inside zookeeper from that file. If that is the case, I'd say that's a major bug. Nothing in the Pulsar docs indicates you need to initialize ZK this way, or indeed, even use the ZooKeeper provided by Pulsar.

My ZK, which is a standalone ZK rather that the one started by bin/pulsar zookeeper does have this key /admin/policies/public/functions, and it does have the wrong value in it:
"replication_clusters":["standalone"]

so this means two things:

This value is being initialized from somewhere, but it isn't the functions_worker.yaml in the ZK container, because my ZK container does not have functions_worker.yaml in it.

It isn't one of the broker containers, because my broker container has this line in functions_worker.yaml:

# grep pulsarFunctionsCluster conf/functions_worker.yml
pulsarFunctionsCluster: local

which is the correct value, and does not match what is in ZK.

How did you initialize you cluster metadata?

@rocketraman you were right. My solution does not make any sense. When I tried it for the first time I was working on a new cluster. Before that, I tried many other solutions but none of them work (including setting the PF_ env variables in broker.yaml). And I think that's the issue here, once the clusters of /admin/policies/public/functions are set, they can't change.
Check this code from pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/Worker.java:
// getting namespace policy
log.info("Initializing Pulsar Functions namespace...");
try {
    try {
        admin.namespaces().getPolicies(workerConfig.getPulsarFunctionsNamespace());
    } catch (PulsarAdminException e) {
        if (e.getStatusCode() == Response.Status.NOT_FOUND.getStatusCode()) {
            // if not found than create
            try {
                Policies policies = new Policies();
                policies.retention_policies = new RetentionPolicies(-1, -1);
                policies.replication_clusters = new HashSet<>();
                policies.replication_clusters.add(workerConfig.getPulsarFunctionsCluster());
                admin.namespaces().createNamespace(workerConfig.getPulsarFunctionsNamespace(),
                        policies);
            } catch (PulsarAdminException e1) {
                // prevent race condition with other workers starting up
                if (e1.getStatusCode() != Response.Status.CONFLICT.getStatusCode()) {
                    log.error("Failed to create namespace {} for pulsar functions", workerConfig
                            .getPulsarFunctionsNamespace(), e1);
                    throw e1;
                }
            }
        } else {
            log.error("Failed to get retention policy for pulsar function namespace {}",
                    workerConfig.getPulsarFunctionsNamespace(), e);
            throw e;
        }
    }
    try {
        internalConf = admin.brokers().getInternalConfigurationData();
    } catch (PulsarAdminException e) {
        log.error("Failed to retrieve broker internal configuration", e);
        throw e;
    }
} finally {
    admin.close();
}

It looks like the values are only set if they are not found. What do you think?

This may be the solution...

If you are starting from fresh you need to make sure to configure PF_pulsarFunctionsCluster right before deploying the worker.

If you already started the worker with a wrong configuration, you can try this out:
$ bin/pulsar-admin namespaces set-clusters --cluster=<cluster> public/functions

And then redeploy the worker with a fixed configuration.

yeah I think @stupidusername already provided the right solution

Would like to close this issue, since there was already a solution


@tsed/socketio bugs

Informations
I can't import  @SocketMiddlewareError() and @SocketMiddleware() decorators (missing import statements).
In examples are @SocketErr() but it should be @SocketErr.
Compilation error (TS version=2.6.2):
node_modules/@tsed/socketio/lib/services/SocketIOServer.d.ts(9,45): error TS1005: '=' expected.

9 export declare const SocketIOServer: unique symbol;

Is there any way to inject  to @SocketMiddlewareError() event name on which the error occurred?
Bug | 4.8.1

@kalu111 Fixed :)
I've added a new decorator SocketEventName. Check the updated documentation ;)

Thanks,
What tag should be now?

v4.9.0

I found an another issue.
Error thrown in middleware is not catched by @SocketErrorHandler().
Can you also chek it, pls :)

Normally if you catch the error, the process will not be exited. I don't remember who asked before about that, but is a requested feature. Normally, in Node.js when an error is thrown the process should be down (and free all allocated memory). The developer must put a process manager like PM2 to restart the process automatically.
But I understand the problem with this instruction (because this behavior is not the same with the HTTPException in the express server part).

Normally. If you handle correctly the error (when the fix will be release XD), the catch will not triggered.

Can you send me the configuration of Websocket Service with the event where you have the problem. Because I've tried different use case, and all work fine.

@SocketService('/')
@SocketUseBefore(MySocketMiddleware)
@SocketUseAfter(ErrorHandlerSocketMiddleware)
export class MySocketService {}

@SocketService('/')
export class OtherSocketService {
    @Input('request')
    public async handle() {
        // some code
    }
}

@SocketMiddleware()
export class MySocketMiddleware {
    public async use( ) {
        throw new Error('Error');
    }
}

@SocketService('/')
@SocketUseBefore(MySocketMiddleware)
@SocketUseAfter(ErrorHandlerSocketMiddleware)
export class MySocketService {}

@SocketService('/')
export class OtherSocketService {
    @Input('request')
    public async handle() {
        // some code
    }
}

@SocketMiddleware()
export class MySocketMiddleware {
    public async use( ) {
        throw new Error('Error');
    }
}

Thanks

For me it. Works. The ErrorHandlerSocketMiddleware catch the error correctly.
Here my middleware:
@SocketMiddlewareError()
export class ErrorHandlerSocketMiddleware {
    async use(@SocketErr err: any, @Socket socket: SocketIO.Socket) {
        $log.error("===", err);
        socket.emit("eventErr", {message: "An error has occured"});
    }
}

How have you declared your ErrorHandlerSocketMiddleware ?

Another issue :)
When I'm injecting @SocketSession to another SocketService or Middleware I'm getting empty Map.
@SocketService('/')
export class MySocketService {
    @Input('request1')
    async handle(@SocketSession session: SocketSession) {
        session.set('test', 'test');
    }
}

@SocketService('/')
export class OtherSocketService {
    @Input('request2')
    async handle(@SocketSession session: SocketSession) {
        const test = session.get('test'); // undefined
    }
}

Pls, check it.

OK, issue with @SocketMiddlewareError resolved :)
I injected a class to the middleware constructor described without @Service() decorator.
Sorry :)
But strange, that there was no warning about that?

yes it's strange. Normally Injector must fail when a Service is unknown

About @SocketMiddlewareError - I found the issue.
I set the error handler for SocketService where there was no event handlers.
@SocketService('/')
@SocketUseBefore(MySocketMiddleware)
@SocketUseAfter(ErrorHandlerSocketMiddleware)
export class MySocketService {}

When I set error handler to SocketService with event handlter, then it works correctly
@SocketService('/')
@SocketUseBefore(MySocketMiddleware)
@SocketUseAfter(ErrorHandlerSocketMiddleware)
export class OtherSocketService {
    @Input('request')
    public async handle() {
        // some code
    }
}

I was thought that middlewares are inherited :)
About @SocketSession
Yes, I can use my own implementation to store session data between SocketServices.
But I think, when we have opportunity to create multiple SocketService with the same namespace so it would be seems to can use session between them. SocketId is not change until disconnect - so access to the session data is not a problem. Right?

Eventually store session per socket namespace?

Per namespace, I thinks it's possible.

Next release will let you to use the session per namespace :)

v4.10.0

It works great !
Thank you for your time :)

I have also a few new topics but I will create separate threads for that :)

Thanks @kalu111 :)
yes I'll prefer a separate topic for each demand. Is easiest for me to track the issue :)


aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

the navbar color isn't right there's a white border around everything and the https is fricked aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
i cant fix any of it
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

i was dum - peep the TO_DO

We should move the TODO into separate issues


Rails 6, Production only issue, works fine in dev, tripped up assets:precompile build process on Heroku, compiled fine locally.

/app/vendor/bundle/ruby/2.6.0/gems/zeitwerk-2.1.10/lib/zeitwerk/loader/callbacks.rb:17:in `on_file_autoloaded': expected file /app/vendor/bundle/ruby/2.6.0/gems/solidus_auth_devise-2.3.0/lib/controllers/backend/spree/admin/orders/customer_details_controller_decorator.rb to define constant Spree::Admin::Orders::CustomerDetailsControllerDecorator, but didn't (NameError)

I think this is a duplicate of https://github.com/solidusio/solidus_auth_devise/issues/174 - thanks for reporting anyway!

Of course, you're right. Thank you!


make exec command run asynchronous output

It is observed that the current implementation print out the exec output in shell func when the whole script finishes. It's lagging when shell func invokes a long running script, eg a lengthy Ansible task

Fixed
However the same async need to be applied to GOSH: https://github.com/mvdan/sh/issues/592
For whoever may be concerned, please raise this agin to follow it up


Add ztoc v1.0

This needs a lot of work to be mergeable, but I wanted to discuss the direction before putting more effort into it. What do you all think?
Issue #, if available:
N/A
Description of changes:
This change copies our existing ztoc flatbuffer definition to ztoc09.fbs and removes build_tool_identifier from ztoc.fbs.
The problem I'm trying to solve is that build_tool_identifier is a versioned string (e.g. AWS SOCI CLI v0.1) which means that different versions of the tool produce different outputs for the same inputs. Removing the field is a breaking change so the thought was to make that breaking change for ztoc 1.0, but keep the original definition in SOCI so that existing v0.9 ztocs still work. The only way to do that is to duplicate the ztoc serialization logic.
Alternatively, we could keep the field and remove the version (which still produces different output from different tools, even if they use the same library) or we could mark the field deprecated and just ignore it.
If we want ztoc to eventually live outside of SOCI, I think it makes sense to try to have a clean v1.0 that doesn't have deprecated fields. If we accept this change, then when we eventually split off ztoc, the v0.9 definitions would stay in SOCI and only the v1.0 logic would be split. That way we get backwards compatibility in SOCI and a clean ztoc package. That might artificially restrict the ztoc API because we would need a way for an external package (in this case SOCI) to serialize and deserialize a ztoc.
Testing performed:
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The only way to do that is to duplicate the ztoc serialization logic.

Would it be possible for the non-SOCI implementation to simply provide a way to hook custom pre/post serialization functions that could handle extra fields? Then the 0.9 implementation would just be a few functions long and wrap the 1.0 implementation, rather than be a copy.

also we need to have necessary unit (or integration) tests to test those expected conversions.


[KYUUBI #1764] Reuse the workingDir in ProcBuilder

Why are the changes needed?

Reuse the workingDir in ProcBuilder for Trino
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[X] Run test locally before make a pull request

You may need to remove useless import, IIUC.

Codecov Report

Merging #1765 (41152da) into master (3881469) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1765      +/-   ##
============================================
- Coverage     59.52%   59.43%   -0.09%
  Complexity      241      241
============================================
  Files           273      273
  Lines         13415    13397      -18
  Branches       1695     1691       -4
============================================
- Hits           7985     7963      -22
- Misses         4746     4752       +6
+ Partials        684      682       -2

Impacted Files
Coverage Δ

...ache/kyuubi/engine/trino/TrinoProcessBuilder.scala
44.82% <ø> (-2.55%)
:arrow_down:

...he/kyuubi/engine/flink/operation/GetCatalogs.scala
83.33% <0.00%> (-4.17%)
:arrow_down:

...ache/kyuubi/engine/flink/operation/GetTables.scala
84.00% <0.00%> (-2.49%)
:arrow_down:

...kyuubi/engine/flink/operation/FlinkOperation.scala
53.96% <0.00%> (-1.04%)
:arrow_down:

...a/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
27.27% <0.00%> (-0.83%)
:arrow_down:

.../kyuubi/engine/flink/operation/GetTableTypes.scala
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

thanks, merged to master


python311Packages.pyocd: 0.35.1 -> 0.36.0

https://github.com/pyocd/pyOCD/releases/tag/v0.36.0
Description of changes
Fixes the build. Converts it to a PEP517 style build. Some light refactoring to bring it in line with the python package set.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

12 packages built:

nitrokey-app2
nitrokey-app2.dist
pynitrokey
pynitrokey.dist
pyocd (python311Packages.pyocd)
pyocd.dist (python311Packages.pyocd.dist)
python310Packages.pyocd
python310Packages.pyocd.dist
python310Packages.spsdk
python310Packages.spsdk.dist
python311Packages.spsdk
python311Packages.spsdk.dist


Tests: Make it easier to run unit tests locally

Running phpunit tests locally has been a bit cumbersome for me, frequently having to look up the right docker command (docker compose -f docker-compose-test.yml up). This PR adds and documents npm and composer scripts that run unit tests in wp-env.
Running Tests
You can now run the test suite using either npm or composer:
# Using npm
npm run env-start
npm run env-test

# Using composer
wp-env start
composer run test:wp-env

PHPUnit Arguments
Both commands support additional PHPUnit arguments. Add them after --:
# Run a specific test
npm run env-test -- --filter=test_migrate_to_4_1_0

# Run tests in a specific file
npm run env-test -- tests/includes/class-test-migration.php

# Run tests with a specific group
npm run env-test -- --group=migration

# Run tests with verbose output
npm run env-test -- --verbose

# The same works with composer
composer run test:wp-env -- --filter=test_migrate_to_4_1_0

Common PHPUnit arguments:

--filter: Run tests matching a pattern
--group: Run tests with a specific @group annotation
--exclude-group: Exclude tests with a specific @group annotation
--verbose: Output more verbose information
--debug: Display debugging information

Proposed changes:

Adds tests specific config for wp-env.
Adds scripts for npm and composer.

Agreed! Once they're settled I assume we can mostly copy/paste the PR description for it


Error constructing the URL for django-storages when using django-configurations and other

Hi,
When using django-configurations it happens that URL are malformed
<script src="https://s3.amazonaws.com/<Value default: None>/ckeditor/ckeditor/ckeditor.js" type="text/javascript"></script>

This happens because when using django-configurations usually the bucket name is set as
AWS_STORAGE_BUCKET_NAME = values.SecretValue()

Then inside widgets.py the injected media is
try:
            js += (
                settings.STATIC_URL + 'ckeditor/ckeditor/ckeditor.js',
                settings.STATIC_URL + 'ckeditor/ckeditor-init.js',
            )

I think a better approach is to use
from django.contrib.staticfiles.templatetags.staticfiles import static
url = static('ckeditor/ckeditor/ckeditor.js')

I think this way more cases with the storage can be handled.
When you mention on the docs
NOTE: django-ckeditor will not work with S3 through django-storages without this line in settings.py:

AWS_QUERYSTRING_AUTH = False

Isnt it an issue with CORS and this ?
best,

Yes, definitely there shouldn't be any settings.STATIC_URL there.

New version released with the latests commits. I'm not using AWS now so I can't say much about the config.

Temporarily settings.STATIC_URL can't be replaces with static() as it causes some problems with cached storages. That would have to be solved first.

Thanks! We're in the process of fixing this. See https://github.com/django-ckeditor/django-ckeditor/pull/377 and many many other issues and pull requests :-)


build: more bad obj dependency issues fixed [v2]

The fixes here were mostly spotted by make -j runs of some targets (like
samples), which would sometimes reach use of some generated dependencies
before they were fully made, leading to errors that were recoverable
from a subsequent make invocation.
First of all, the inclusion of the .dep files was totally broken, so the
files listed in them were not being checked for dependencies at all! We
now moved the inclusion to a place where "all-objs" is already declared
and we're fine.
Some files were including "sol_config.h" by hand, and by fixing some
dependency declarations this will not only be unecessary but impossible
-- only the automatic compiler line inclusion will pass it forward.
"extra-headers" usage in "parse-common-module" was taken off -- it was
passing forward wrong objects and had only one user, that now users the
general "headers-$(whatever)" syntax.
"make samples" invocation right away after kconfig is working again,
though we still have samples depending on the whole library objects
set (we'll fix that later if we feel like declaring the individual
sample dependencies by hand). On the other hand, now that .dep
files do work, changes in headers, modules or any dependency of
another module will trigger its build again.
One other thing is that explicit module dependency declaration had to
move back to our Makefiles, since at least on static builds we only have
that info to stall module builds till their dependencies are met on
parallel builds.
Thanks a lot for more help of Dorileo here :)
Signed-off-by: Gustavo Lima Chaves<EMAIL_ADDRESS>
Changes since #1047:

fixed issue raised on static builds

Lots of build runs here and all going beautifully. It's Friday, what a better day to merge this? Doing that now.


feature: Add a prerequisite in the installation docs to first install Neovim

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
So I'm a complete Neovim newbie - never touched it before.
I heard about LazyVim as a great way to try out Neovim, so I thought I'd give it a go.
I followed the installation docs, however the last step, running nvim did not work.
It took me an embarrassingly long while to work out that I first needed to actually install Neovim.
While this may seem ridiculous, it seems to me the entire appeal of LazyVim is to introduce people to Neovim, and I actually blame the Neovim docs for this, because like I said, I'm a complete newbie!
I'm aware that Neovim is mentioned as a prerequisite in the 'Requirements' section, but this is underneath the 'Features' and IMHO is a terrible spot for this.
Describe the solution you'd like
I think the 'Requirements' section within 'Getting Started' should be moved to the 'Installation' page.
Describe alternatives you've considered
Include a check in the LazyVim installation steps for a Neovim installation, and if it's missing, point this out to the user.
Additional context
No response

You are welcome to make a PR at the website's repo, although in my opinion it is mentioned and there's no further need for that. But your PR might introduce some new light for newbies like yourself, so who knows?

LazyVim is a Neovim setup
Transform your Neovim into a full-fledged IDE

These are the first lines and i'm not sure how you missed them. Under requirements it's just the required version of Neovim.

Neovim is literally under the requirements https://www.lazyvim.org/#-requirements


hotkeys: Add arrow key navigation in streams/subscriptions menu.

Fixes #4198

@timabbott Do you think that the motion is a bit jerky? Also, when you initially press down, the whole list goes down with it. I think you told Jonathan that the list should go down when you reach the bottom of the list.

Yeah, ideally the list would only scroll when you're scrolling off screen (I guess I commented on this above).  I'm potentially OK with saving that for a part 2 after Jonathan's branch is merged so we can reuse his approach.

Reviewed and merged.  I think we should open a few follow-up issues for the items I mentioned above, though (jerky scrolling, etc.).  Can you take care of it @synicalsyntax?


module 'nmap' has no attribute 'PortScanner'

Checklist for submitting an issue to KickThemOut:

[x] I have carefully read the README file and haven't managed to resolve my issue.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I am running the latest version of KickThemOut.

OS name & version:  Debian 9
Python version:  Python 3.5.3
Scapy version:  Scapy (3.0.0)
Nmap version:  Nmap 7.60SVN

Description: python3 kickthemout.py

Scanning your network, hang on...
ERROR: Network scanning failed. Please check your requirements configuration.
If you still cannot resolve this error, please submit an issue here:
https://github.com/k4m4/kickthemout/issues
Details: module 'nmap' has no attribute 'PortScanner'

Hi there. Can you please uninstall and re-install all dependencies following the process stated in the README file, pasting the results of all of your steps here? Thanks.

I solved this problem on my parrot security os.
try 'pip3 install nmap'
if it installed allready it will print out the version.
if version is'nt 0.6.1 you have to install it mannually
follow my step:

hg clone https://bitbucket.org/xael/python-nmap
cd python-nmap
python3 setup.py install

I hope it will work....:)

Closing due to inactivity. If you are still experiencing this problem, please re-open this issue. Thanks!

i have a problem named "module 'nmap' has no attribute 'PortScanner'"
can someone help me please


Adding ML module and example use

Adding machine learning module to geemap based on #124 .
Additional files included:

geemap/ml.py: ml module which includes functions to convert sklearn ensemble/decisiontree estimator objects to strings for use with ee
examples/notebooks/local_rf_training.ipynb: example notebook use that highlights functionality
examples/data/rf_example.csv: csv file containing data used in example notebook.

Updated files:

requirements.txt: added additional requirements used by the ml model, specifically numpy and pandas

@KMarkert Thanks a lot for the excellent contribution! Merging now.


download video couldn't generate thumbnail

hi
when I pick download video.
below error occurs on iOS ( iPhone7 )
android works fine
flutter: /private/var/mobile/Containers/Data/Application/09F51573-CA6E-49BA-A792-E7FEBF4A5F74/tmp/_talkv_woYgOXv7ml_zSU8dBVSYS00f6uxummnuk_talkv_high.mp4 couldn't generate thumbnail, error:Error Domain=AVFoundationErrorDomain Code=-11832 "Cannot Open" UserInfo={NSLocalizedFailureReason=This media cannot be used., NSLocalizedDescription=Cannot Open, NSUnderlyingError=0x283f9c690 {Error Domain=NSOSStatusErrorDomain Code=-12431 "(null)"}}
Image
https://helloworld43943.tumblr.com/image/649523710269767680
thx for your effort 😇

I have the same error. But it seems to be video-dependent.
E.g. this video works fine https://file-examples-com.github.io/uploads/2017/04/file_example_MP4_480_1_5MG.mp4

Hi All,
have the same issue on iOS. Seems like an issue with video_thumbnail plugin. It cannot produce the last duration time thumbnail.
Do you have any progress?

Hello,
Have the same issue  on IOS. Please assist, really need to fix it. :(
couldn't generate thumbnail, error:Error Domain=AVFoundationErrorDomain Code=-11832 "Cannot Open" UserInfo={NSLocalizedFailureReason=This media cannot be used., NSLocalizedDescription=Cannot Open, NSUnderlyingError=0x283f9c690 {Error Domain=NSOSStatusErrorDomain Code=-12431 "(null)"}}

@yoonjiyong @char-lance-flamme @amag2511  Have you added the proper permissions to the Info.plist file as stated here
Also, let me know which iOS version are you using.

Hi All,
@amag2511
I don't have any progress with it.
@sbis04
ios/Runner/Info.plist
<key>NSCameraUsageDescription</key> <string>Used to get user profile image</string> <key>NSMicrophoneUsageDescription</key> <string>Used to capture audio for video upload</string> <key>NSPhotoLibraryUsageDescription</key> <string>Used to get user profile image</string>
Device - iPhone 8
Softvare version - 14.4.2
As I mentioned above - with some videos it works cool, but with others, it returns the error.

@Vitaly-V can you provide some videos using which you are facing this issue? I will try to reproduce the error and then look into it.

@sbis04
It is my Info.plist
NSCameraUsageDescription
Used to demonstrate image picker plugin
NSMicrophoneUsageDescription
Used to capture audio for image picker plugin
NSPhotoLibraryUsageDescription
Used to demonstrate image picker plugin
my Device Iphone XR
IOS 14.4.02(18D70)

Guys, any updates?

I also experience the same problem. Here are more details in case they can help:

Here is the error in console:
Flutter: Attempted to composite external texture sources using the software backend. This backend is only used on simulators. This feature is only available on actual devices where OpenGL or Metal is used for rendering.
Flutter: Attempted to composite external texture sources using the software backend. This backend is only used on simulators. This feature is only available on actual devices where OpenGL or Metal is used for rendering.
flutter: ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
flutter: The following _CastError was thrown building:
flutter: Null check operator used on a null value
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      ThumbnailViewer.build.<anonymous closure>.<anonymous closure> (package:video_trimmer/src/thumbnail_viewer.dart:63:58)
flutter: #1      SliverChildBuilderDelegate.build (package:flutter/src/widgets/sliver.dart:455:22)
flutter: #2      SliverMultiBoxAdaptorElement._build (package:flutter/src/widgets/sliver.dart:1213:28)
flutter: #3      SliverMultiBoxAdaptorElement.createChild.<anonymous closure> (package:flutter/src/widgets/sliver.dart:1226:55)
flutter: #4      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2535:19)
flutter: #5      SliverMultiBoxAdaptorElement.createChild (package:flutter/src/widgets/sliver.dart:1219:12)
flutter: #6      RenderSliverMultiBoxAdaptor._createOrObtainChild.<anonymous closure> (package:flutter/src/rendering/sliver_multi_box_adaptor.dart:349:23)
flutter: #7      RenderObject.invokeLayoutCallback.<anonymous closure> (package:flutter/src/rendering/object.dart:1889:59)
flutter: #8      PipelineOwner._enableMutationsToDirtySubtrees (package:flutter/src/rendering/object.dart:912:15)
flutter: #9      RenderObject.invokeLayoutCallback (package:flutter/src/rendering/object.dart:1889:14)
flutter: #10     RenderSliverMultiBoxAdaptor._createOrObtainChild (package:flutter/src/rendering/sliver_multi_box_adaptor.dart:338:5)
flutter: #11     RenderSliverMultiBoxAdaptor.insertAndLayoutChild (package:flutter/src/rendering/sliver_multi_box_adaptor.dart:484:5)
flutter: #12     RenderSliverList.performLayout.advance (package:flutter/src/rendering/sliver_list.dart:239:19)
flutter: #13     RenderSliverList.performLayout (package:flutter/src/rendering/sliver_list.dart:282:19)
flutter: #14     RenderObject.layout (package:flutter/src/rendering/object.dart:1779:7)
flutter: #15     RenderSliverEdgeInsetsPadding.performLayout (package:flutter/src/rendering/sliver_padding.dart:137:12)
flutter: #16     RenderSliverPadding.performLayout (package:flutter/src/rendering/sliver_padding.dart:371:11)
flutter: #17     RenderObject.layout (package:flutter/src/rendering/object.dart:1779:7)
flutter: #18     RenderViewportBase.layoutChildSequence (package:flutter/src/rendering/viewport.dart:510:13)
flutter: #19     RenderViewport._attemptLayout (package:flutter/src/rendering/viewport.dart:1574:12)
flutter: #20     RenderViewport.performLayout (package:flutter/src/rendering/viewport.dart:1483:20)
flutter: #21     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1636:7)
flutter: #22     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:881:18)
flutter: #23     RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:462:19)
flutter: #24     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:876:13)
flutter: #25     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:328:5)
flutter: #26     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1144:15)
flutter: #27     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1082:9)
flutter: #28     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:998:5)
flutter: #32     _invoke (dart:ui/hooks.dart:163:10)
flutter: #33     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:259:5)
flutter: #34     _drawFrame (dart:ui/hooks.dart:126:31)
flutter: (elided 3 frames from dart:async)
flutter: ════════════════════════════════════════════════════════════════════════════════════════════════════

flutter doctor (iOS only):
[✓] Flutter (Channel stable, 2.2.1, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Connected device (2 available)

Thanks. Happy to provide more details if it helps.

@Gicminos I see that the above screenshot is from an iOS Simulator, so did you try it on a physical device? Also, did you try with any different video to check if the error persists in that as well?

@sbis04, on the real devices' error looks pretty the same.
Yes, it's video dependent, - some videos are trimming well, some with this error.

@Vitaly-V Can you please share a video file or it's link which is producing this error?

@sbis04, please try this video file.
https://user-images.githubusercontent.com/4039270/122532393-1f29c600-d029-11eb-9289-7ead7b4f3c99.mp4

Or this one
https://user-images.githubusercontent.com/4039270/122536481-3e2a5700-d02d-11eb-8d19-5604ea96654f.mp4

I confirm what Vitaly-V is saying. The error on the real device is the same (while on release mode you see the gray band instead of the red error) and it is yes video dependant.

This PR https://github.com/sbis04/video_trimmer/pull/112 seems to be claiming to fix this error. Happy to test it myself in the next days, but will probably have to fork it and test it with just a cherry-pick of https://github.com/sbis04/video_trimmer/pull/112/commits/82898fa13affa984aa6a1790d4ab8d3feef51504 because I am not sure about the other changes.

Try using the latest version of the plugin:
video_trimmer: ^0.5.2

Let me know if the thumbnail error is fixed.

We haven't tested widely yet, but we do indeed see improvements. I will let you know in case of any issues. Thanks!

Closing this issue. Feel free to open, if the issue still persists.

I am getting this error
Runner[3759:312601] couldn't generate thumbnail, error:Error Domain=AVFoundationErrorDomain Code=-11832 "Cannot Open" UserInfo={NSLocalizedFailureReason=This media cannot be used., NSLocalizedDescription=Cannot Open, NSUnderlyingError=0x281948270 {Error Domain=NSOSStatusErrorDomain Code=-12431 "(null)"}}

I am getting this error
Runner[3759:312601] couldn't generate thumbnail, error:Error Domain=AVFoundationErrorDomain Code=-11832 "Cannot Open" UserInfo={NSLocalizedFailureReason=This media cannot be used., NSLocalizedDescription=Cannot Open, NSUnderlyingError=0x281948270 {Error Domain=NSOSStatusErrorDomain Code=-12431 "(null)"}}

I have the same error

@BanidAzin @nisargdoshi is the error occurring with all the videos? On which platform are you testing (Android/iOS)?

@BanidAzin @nisargdoshi is the error occurring with all the videos? On which platform are you testing (Android/iOS)?

@sbis04 Yes, it's happening for me all the time. I am trying to create a thumbnail after capturing a video using the camera.
I am testing it on iOS.

Any progress on this issue? I have the same error on iPhone SE second gen with iOS 15.1

I am also getting this issue. It does not prevent me from trimming the video, but it does result in a not-so-ideal UX. Looking forward to updates on this. I am testing on a real device iOS (15)


Create overload without parameter to replace parameter with default null

The DbContextScopeFactory class has a constructor with an optional argument. Optional arguments are not well supported by IoC libraries. In particular, SimpleInjector does not support them at all.
This commit refactors that constructor by splitting it into overloads with and without arguments.
Resolves #37

Is this project still being maintained?


Add support to allow multiple IAP audience claims

Currently autoconfiguration supports a single audience value. With this commit
now is possible to configure multiple allowable IAP audience claims.
Fixes gh-1468

@acmarcel Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

One more thing, @acmarcel -- could you update the refdoc to indicate support for multiple allowable audiences?

@acmarcel Thank you for signing the Contributor License Agreement!

thanks!


DFS and DFW don't work as anti-dragon shield

🐛 Bug Report
Write a description of the bug here, give all the information you can about what it is, how it happened.
Dragonfire Shield and Dragonfire Ward do not work as anti-dragon shield for purposes of killing blue dragons (presumably anything else that requires an anti-dragon shield as well).

Should be fixed in next update. Added all the dragonfire protection shields to count as anti-dragon shield.


Python 3.6 invalid escape sequence deprecation fixes

Can you please add a changelog entry? 0.0.1 bump is good.

Done.


Fixer portion of the Go fixchain & preload tool

The Fixer element takes chains and tries to create a valid chain for each
certificate in the given chain.  Errors are reported to one channel, and
successfully fixed chains are pushed to another.
Simplified intended usage of this element:
fixer := NewFixer(...)
fixer.QueueChain(...)   // Repeat this line for as many chains as you have.
fixer.Wait()

Sorry for the delay.
A sync wrapper around an async API is needlessly complex. Synchronous logic is much easier to write and debug. A sync wrapper is not complicated, but then neither is an async wrapper.
The problem with sync wrapping async is that to understand the actual logic you have to at the same time understand the asynchronousness.
With async wrapping sync the logic is decoupled from concurrency. And the concurrency wrapping becomes (in Go) so simple that often it's not even worth adding this second API.
"Expose synchronous APIs, calling them concurrently is easy."
-- http://talks.golang.org/2013/bestpractices.slide#26

Sync wrapper is no more, and exposed now is a synchronous Fix() function that works with the internal fixing functions (which have all been made fully synchronous too) to fix a chain.  This application logic is actually quite simple, and this division will make it really easy to improve on the core fixing algorithm in the future :D
The async wrapper bit - the Fixer - is code we need for the overall fixchain tool.  So we need that code to still be around too.

Sorry for the delay in reviewing. Looks good now.


to update

Hi, thank you for your PR.
These changes are not relevant to the course content, since they are user-specific, and would break the learning material. Consequently, the changes would break the learning process for future students.
Please keep the changes in your fork, and continue the course with your fork. I am going to close this PR.
If you believe this to be an error, feel free to reach out, or open a new PR with further explanation.


chore(esitmate): component to estimate resource

Signed-off-by: wanglei4687<EMAIL_ADDRESS>What type of PR is this?

[ ] API-change
[ ] BUG
[ ] Improvement
[ ] Documentation
[ ] Feature
[ ] Test and CI
[ ] Code Refactoring

Which issue(s) this PR fixes:
Fixes #
What this PR does / why we need it:
Not Available
Special notes for your reviewer:
Not Available
Additional documentation (e.g. design docs, usage docs, etc.):
Not Available

did you mean benchmark?

did you mean benchmark or evaluation?

did you mean benchmark or evaluation?

yep


Improvements to "Authoring Maintainable Build Scripts" docs

Context
Most used internal APIs: https://gist.github.com/eriwen/ccf04b44cb9b0cb28193a8bc9aec01d9#file-results-20170614-csv — perhaps you could come up with more alternatives for common ones @bmuschko?
Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

perhaps you could come up with more alternatives for common ones

I think a lot of the APIs listed are only in the list because people forked the Gradle core repository. For example there's no way somebody is actually using org.gradle.integtests.fixtures.AbstractIntegrationSpec which is listed very high in the list. To get a better idea on the actual usage of internal APIs we should somehow filter out the Gradle core forks.
Some alternatives:

org.gradle.internal.reflect.Instantiator: Already has a public API equivalent.
org.gradle.api.internal.project.ProjectInternal: There's likely no good reason why people use the class. Project should just be fine expect if you want to access other internals again.
org.gradle.util.GradleVersion: I am wondering if we should make this class public. I have seen many users ask about it. Plus we control the versioning scheme so it doesn't come with a big risk.
org.gradle.util.CollectionUtils: Just don't use it. There are better alternatives e.g. Guava, Commons Collections or Java 8.
org.gradle.api.internal.ConventionMapping: I'd suggest replacing all usage in the Gradle core code base with Provider and deprecating it.
org.gradle.logging.ProgressLogger: This a feature a lot of plugin developers asked for. We should think of some sort of public API for it.


Improve the CMake scripts

This PR leverages the work done on core and sync.

@jedelbo Please assess if anything from here is useful, or close the PR if not.

This will become obsolete with the monorepo. We can salvage things piecemeal as needed.


Enable chaining for middlewares and afterwares in the network interface - Issue 564

TODO:

[ ] If this PR is a new feature, reference an issue where a consensus about the design was reached (not necessary for small changes)
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass
[x] Update CHANGELOG.md with your change
[x] Add your name and email to the AUTHORS file (optional)
[x] If this was a change that affects the external API, update the docs and post a link to the PR in the discussion

Hi,
I'm trying to solve #564. Have a look at this PR and tell me what you think.
Remarks :

If I'm not mistaking, it should also possible to chain use() and useAfter() in the same call. I might need to also add a test for that. Right now, my patch only tests use(...).use(...) and useAfter(...).useAfter(...). Please tell me if such test is relevant.
As this PR adds a new feature, I need to update the docs. I will, but first I'd like to know if the PR looks good.

Regards

Just added a new commit that includes the require change + two new tests for testing use(...).useAfter() and useAfter().use() chaining.
I will now work on a PR on the docs. Stay tuned ...

Here is a link to a PR on apollostack/core-docs which documents this feature :
https://github.com/apollostack/core-docs/pull/218

For me, this PR is ready to be merged but tell me if some changes need to be done.

Looks good to me. Thanks a lot @oricordeau !


Video either muted or not automatically played

Hi,
first of all thanks for your amazing library guys.
I just have one question for you.
When one opens the video library, for example with the following YPImagePickerConfiguration :
config.library.mediaType = .video
config.screens = [.library]

the first video selected in the library is automatically played with sound.
My question is : is there a config option to change that default behaviour to either playing the video muted or just not playing the video at all upon selection ?
Thanks for your time and assistance.

Interested in hearing about this too. The automatic playback is something I'd like to disable if possible.

Don't think there is an option, I +1 on implementing this too.

Is this feature is implemented or yet to be implemented ?

Too old issue. Feel free to open new.


Salesforce NPSP Add-on Package

I am developing a NPSP add-on package these are version of different NPSP packages: Relationships - 3.8
Nonprofit Success Pack - 3.162
Households - 3.11
Affiliations - 3.7
Contacts & Organizations - 3.11
Recurring Donations - 3.13
And I am trying to upload my package. It is giving me this error [There are around 100+ NPSP test classes in the below list]:

All the above classes are from NPSP packs. After recompiling this page is seen:
enter image description here

I cannot save these NPSP classes since they are part of managed package. I saved all of my packs classes but it still gives the same error. Any suggestions or workaround?
In my developer org technically I was able to make release before Winter 20 release which happened on gone by Saturday, after than I have not been able to upload package.
Thank you

This issue has been resolved by Salesforce. Thank you.


Defender for IoT Sample CLI SPAN port configuration (Cisco 2960) command fix

On our documentation there is a sample that for CLI SPAN port configuration (Cisco 2960). On that sample set of commands, at line 5, we have a command: Cisco2960# show monitor 1. This command is missing the key 'session'. It should be:
Cisco2960# show monitor session 1
The suggested fix adds the missing 'session' keyword on line 5.
Here is the link to the documentation page that has missing 'session' keyword:
https://learn.microsoft.com/en-us/azure/defender-for-iot/organizations/traffic-mirroring/configure-mirror-span#sample-cli-span-port-configuration-cisco-2960

#sign-off


feat(client): stop margin from pushing outline out of the container

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Help camper with this PR #49579
because logo CSS is pushing the outline outside its container while its container is smaller than the parent container, so there is no outline at the bottom and top of the logo
The final desired look.

the goal of the PR to have this focus outline, but because it is changing things, which I needed to change, it won't hurt to check the layout in mobile and laptop as well.

Just a heads up, I know this will break RTL, just don't want to add RTL fix until this is approved

Whelp, you have found another 1px bug
rec-tab (18).webm


KubernetesEndpointResolver works with the name "endpoint" only

Describe the bug
KubernetesEndpointResolver works with the name endpoint only. An override doesn't work.
To Reproduce
Following configuration doesn't work:
      ---
      apiVersion: ambassador/v1
      kind: KubernetesEndpointResolver
      name: my_endpoint
      ---
      apiVersion: ambassador/v1
      kind: Module
      name: ambassador
      ambassador_id: ambassador-80
      config:
        service_port: 8080
        resolver: my_endpoint

But this works:
      ---
      apiVersion: ambassador/v1
      kind: KubernetesEndpointResolver
      name: endpoint
      ---
      apiVersion: ambassador/v1
      kind: Module
      name: ambassador
      ambassador_id: ambassador-80
      config:
        service_port: 8080
        resolver: endpoint

Expected behavior
KubernetesEndpointResolver should work with name override.
Versions (please complete the following information):

Ambassador: 0.60.3
Kubernetes environment: AWS KOPS
Version 1.14.1

This worked with annotations, but CRDs had some issues around what was a legal name (see also #1562) and around Resolver CRDs with no spec.


[Teams] Add reply to channel bot

Fixes https://github.com/microsoft/botbuilder-js/issues/1210

Pull Request Test Coverage Report for Build 81999

1 of 4 (25.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 86.396%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder/src/teamsActivityHelpers.ts
1
4
25.0%

Totals

Change from base Build 81991:
-0.1%

Covered Lines:
4593

Relevant Lines:
5111

💛  - Coveralls


Updated Shell Script for Raspbian Stretch Lite

I have updated the install.sh to install properly with Raspbian Stretch Lite. Updated script to use PHP7.0 over PHP5.
https://rpguy.com/code/install.sh

See pull request https://github.com/urlgrey/hsmm-pi/pull/131 for updated script. Changes are similar to those listed on the site listed above but it should install the latest version of PHP regardless of the version.


Auto-update Python documentation

Woops.

Any idea whats up with the .doctree files?  Are those supposed to be added?

Closing; obviated by pull request 9565.


VSCodium based editor.

Create a extension to open a obs file. Which is an .md file.
vscode currently has a preview mode. we need to have an editor that will allow editing in the preview mode.
01.md


On Macbook mono process goes to 400% of the CPU at regular interval

VS Code Version: 1.55
OS Version: Catalina 10.15.7

Steps to Reproduce:

Open a dotnet core 5.0 / angular project
run project in debbug mode
the fan of the computer will become crazy / it is hard to work
solution is to kill mono process but then you loose the intellicente and it is simply not possible to work

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabled gpu_compositing: enabled metal: disabled_off multiple_raster_threads: enabled_on oop_rasterization: enabled opengl: enabled_on protected_video_decode: unavailable_off rasterization: enabled skia_renderer: disabled_off_ok video_decode: enabled webgl: enabled webgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.09GB free)

Process Argv
--crash-reporter-id ef0c5ab3-61ea-482a-9968-7dd422fe0185

Screen Reader
no

VM
0%

Extension
Author (truncated)
Version

ng-template
Ang
11.2.11

npm-intellisense
chr
1.3.1

es7-react-js-snippets
dsz
3.1.1

vscode-npm-script
eg2
0.3.21

ionic2-vscode
jgw
1.0.0

vscode-webpack
Kni
1.1.0

csharp
ms-
1.23.11

vscode-typescript-tslint-plugin
ms-
1.3.3

vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30291499
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstry914:30276682
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
pythonvspyt639:30291489
pythontb:30283811
pythonvspyt551:30291414
vspre833cf:30267465
pythonptprofiler:30281270
vshan820cf:30276953
pythondataviewer:30285071
vscus158:30286553

Does this issue occur when all extensions are disabled?: Yes

Please open an issue against the C# extension. Thanks!

Okay. Thank you for your reply.
Best regards
On Sun, 25 Apr 2021 at 18:20, VS Code Issue Triage Bot <
@.***> wrote:

This issue is caused by an extension, please file it with the repository
(or contact) the extension has linked in its overview in VS Code or the
marketplace https://aka.ms/vscodemarketplace for VS Code. See also our issue
reporting https://aka.ms/vscodeissuereporting guidelines.
Happy Coding!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/121942#issuecomment-826350201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHR4JAQPNYCDXXSN4LQJ6LTKQ6NXANCNFSM43MP7STA
.

In fact the problem comes from C# for Visual Studio Code (powered by
OmniSharp) extension
Both are not a Microsoft product ? Don't you have to test Visual Studio
Code with Microsoft basic extension lille C# one?
Anyway I cannot use Visual Studio Code, great news.
Best Regards
On Mon, 26 Apr 2021 at 22:39, Sebastien Dubos @.***>
wrote:

Okay. Thank you for your reply.
Best regards
On Sun, 25 Apr 2021 at 18:20, VS Code Issue Triage Bot <
@.***> wrote:

This issue is caused by an extension, please file it with the repository
(or contact) the extension has linked in its overview in VS Code or the
marketplace https://aka.ms/vscodemarketplace for VS Code. See also our issue
reporting https://aka.ms/vscodeissuereporting guidelines.
Happy Coding!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/121942#issuecomment-826350201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHR4JAQPNYCDXXSN4LQJ6LTKQ6NXANCNFSM43MP7STA
.


ci: implement auto labeler for new PR and issue

Closes: #24
Changes
ci: implement auto labeler for new PR and issue

It works fine now:


Support WNS for UWP (Windows 10) apps

Can we add support for Windows Push Notification Services (WNS)? This will support pushing to Universal Windows Platform apps.

Is it different from MPNS? If it is, of course we can add that feature. PRs are always welcome and I'll help you as much as I can if you need help to do it.

Yep, different from MPNS.  And, awesome thanks for the offer for help!

@Shayon, @calonso I'm added WNS support in this PR: https://github.com/calonso/ruby-push-notifications/pull/21

Amazing @krim, thank you very much!! I'll review it ASAP.
Thanks again for your work!!

Closing this as #21 was merged adding it