fix: Reduce VM recovery times

What this PR does / why we need it:
It aims to reduce the VM recovery time when an operator such as NodeHealthCheck is used to remediate unhealthy nodes, i.e. nodes no longer responsive.
In such scenario, the launcher pod is deleted, and the VMi needs to be rescheduled if the spec runStrategy is configured as Always or RerunOnFailure. The VM's Phase is Running but VM has the following condition:
Status:
  Conditions:
    Last Probe Time:       2023-12-18T16:31:31Z
    Last Transition Time:  2023-12-18T16:31:31Z
    Message:               virt-launcher pod has not yet been scheduled
    Reason:                PodNotExists

Eventually, the VMi is rescheduled and the virt-launcher pod recreated.  However, it takes over than 100 seconds in best case. In the following figure, the total VM recovery time break down can be seen where:

Vm_not_ready: Time for the VMi to be READY=False.
Launcher_terminating: Time until launcher pod changes state to terminating.
Launcher_deleted: Time until the launcher pod is deleted from the cluster.
Launcher_respawned: Time until the launcher pod is respawned (respanwed: in any state pending, running, …).
Vm_ready: Time for the VMi to be READY=True.

Please note that those measures are taken from the previous step until the step is finished. Regarding the vm_not_ready measurement, this time starts from the point where kubelet is stopped (this is the way to force a node to be unhealthy) until this VMi state is reached.
For instance, launcher_terminating measurement starts when vm_not_ready, i.e.  the VMi READY field is False, until the launcher pod updates its status to terminating.

As can be seen, the vm_launcher_respawned speed-up is approximately 100x.
The issue is: if a VMi in Phase Running on a given node, and the node becomes unhealthy, VMis are not marked as Failed. This is due to the fact that the main responsible for this task is the virt-handler pod, which own the VMi. However, the virt-handler pod will be unresponsive, unable to be redeployed and unable to mark the VMi as Failed.  The secondary responsible for this action is the node controller. Nevertheless, the node controller requires that heartbeats from the virt-handler have not been received for a certain period of time (5 minutes).
Therefore, in order to transition the VMi to a FAILED state, it is required to wait until:

The node to get healthy again, the virt-handler to be recreated, the virt-handler to detect that VMi is on Running phase without a launcher-pod, and to mark the VMi as Failed.

https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-handler/vm.go#L3280-L3283

The node controller to detect that the virt-handler has been unresponsive for a given amount of time (5 minutes by default), to detect VMis without virt-launcher pod and to mark the VMis as Failed.

https://github.com/kubevirt/kubevirt/blob/d4e5e11c84d0630c1c688b1bfffe7a0133f9675d/pkg/virt-controller/watch/node.go#L462-L477
It reduces the virt-launcher pod respawn time by short-circuiting the feedback loop that the node controller would already handle and the virt-handler recreation. Therefore, the virt-controller will mark the VMi as Failed and it will force the VMi to be rescheduled on another node faster.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes # CNV-36137
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

Reduced VM rescheduling time on node failure

/cc @enp0s3

/cc

@jcanocan Do we have a function test simulating this? It seems to me as easily reproducible in the test.

@jcanocan Do we have a function test simulating this? It seems to me as easily reproducible in the test.

I didn't have the chance to try out. However, the test should work without any external dependencies such as NHC, right?

@jcanocan Do we have a function test simulating this? It seems to me as easily reproducible in the test.

I didn't have the chance to try out. However, the test should work without any external dependencies such as NHC, right?

Right.

@jcanocan Do we have a function test simulating this? It seems to me as easily reproducible in the test.

I didn't have the chance to try out. However, the test should work without any external dependencies such as NHC, right?

Right.

Ok, I will try my best. If you have an idea or an example, that would be helpful. Right now, I'm trying to solve the failures.

v1:

Fixed unit test to adjust new status updates since if the pod is not present, it will be created in a single sync call. This causes new updates and patches on the pods and VMis no previously considered.

v2:

Added a check to only mark VMis as failed when the virt-handler pod is down or gone.

With this additional check, the launcher_pod are not respawned as fast as the original approach. However, it does not break current functional test and the VMi behavior are identical. Check:

The main source of delay has been spotted when this error appears on virt-controller logs:
{"component":"virt-controller","level":"info","msg":"reenqueuing VirtualMachineInstance default/fedora-test-snr","pos":"vmi.go:325","reason":"patching of vmi conditions and activePods failed: Internal error occurred: failed calling webhook \"virtualmachineinstances-update-validator.kubevirt.io\": failed to call webhook: Post \"https://virt-api.kubevirt.svc:443/virtualmachineinstances-validate-update?timeout=10s\": context deadline exceeded","timestamp":"2024-01-05T13:53:44.490347Z"}
I0105 13:53:44.490389       1 trace.go:236] Trace[945221119]: "virt-controller VMI workqueue" Workqueue Key:default/fedora-test-snr (05-Jan-2024 13:53:34.481) (total time: 10008ms):
Trace[945221119]: ---"updateStatus" VMI Name:fedora-test-snr 10008ms (13:53:44.490)
Trace[945221119]: [10.008402325s] [10.008402325s] END

/cc @0xFelix

/retest-required
The failing test works locally, giving it another shoot.

The failing test works locally, giving it another shoot.
/retest-required

/retest-required

@davidvossel @vladikr
Initially, I thought it could be better to check for the handler as well but now when I think about it, this could be as simple as
if !vmiPodExists {
   vmi.Status.Phase = v1.Failed

as I don't see any negatives of doing it. WDYT?

Without your patch, what eventually transitions the VMI to phase=Failed. Are we dependent on virt-handler eventually coming back online on the same node that the VMI is scheduled to?
It seems like without your patch here, we could be in a situation where if a node is abruptly killed and never comes back online, that vmi's with phase=Running might be stuck indefinitely in that phase.

From what I've understood, right now we are dependent on virt-controller eventually picking up that the virt-handler on a node died (no heartbeat). In @jcanocan's test scenario he kills a node, that comes back after a while. The node coming back online triggers the node controller in virt-controller, which eventually marks the VMI as Failed. Until then the VMI remains in phase Running, even though the virt-launcher Pod does not exist. This PR tries to improve that behavior, by marking the VMI as Failed, when virt-controller sees, that it is running but there is no Pod. (basically what @xpivarc suggested in his last comment)
See https://github.com/kubevirt/kubevirt/blob/d4e5e11c84d0630c1c688b1bfffe7a0133f9675d/pkg/virt-controller/watch/node.go#L186

@davidvossel @vladikr Initially, I thought it could be better to check for the handler as well but now when I think about it, this could be as simple as
if !vmiPodExists {
   vmi.Status.Phase = v1.Failed

as I don't see any negatives of doing it. WDYT?

I've spotted some issues with VMReplicaSets.

Initially, I thought it could be better to check for the handler as well but now when I think about it, this could be as simple as
if !vmiPodExists {
vmi.Status.Phase = v1.Failed
as I don't see any negatives of doing it. WDYT?

looking at this closer. I think you are correct. And here's why

This is same logic that the node controller is doing today when the virt-handler heartbeat times out https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-controller/watch/node.go#L248
Virt-handler already knows how to handle cleaning up local cache even after the vmi is removed from the api server.  the domain informer has a local cache it uses to know if a vmi used to exist https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-handler/vm.go#L1624

Essentially what we're doing by moving a vmi.Status.Phase == Running to vmi.Status.Phase == Failed is shortening the feedback loop that the node controller would already handle.
I think we can take this a step further even and move the VMI to failed if the vmi is Scheduled or Running when no vmi pod exists.

v3:

Added a functional test.

Initially, I thought it could be better to check for the handler as well but now when I think about it, this could be as simple as
if !vmiPodExists {
vmi.Status.Phase = v1.Failed
as I don't see any negatives of doing it. WDYT?

looking at this closer. I think you are correct. And here's why
1. This is same logic that the node controller is doing today when the virt-handler heartbeat times out https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-controller/watch/node.go#L248

Exactly, this is just short-circuiting it in case a Pod is explicitly deleted by humans or automation.

2. Virt-handler already knows how to handle cleaning up local cache even after the vmi is removed from the api server.  the domain informer has a local cache it uses to know if a vmi used to exist https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-handler/vm.go#L1624

+1 I believe this is covered by tests as well.

Essentially what we're doing by moving a vmi.Status.Phase == Running to vmi.Status.Phase == Failed is shortening the feedback loop that the node controller would already handle.

+1

I think we can take this a step further even and move the VMI to failed if the vmi is Scheduled or Running when no vmi pod exists.

@jcanocan feel free to drop the check for virt-handler.

v4:

Dropped the virt-handler is alive check.
VMi will be marked as Failed also if the VMi is Scheduled but the virt-launcher pod is not found.
Updated commit description.

v4:

Dropped the virt-handler is alive check.
VMi will be marked as Failed also if the VMi is Scheduled but the virt-launcher pod is not found.
Updated commit description.

Initially, I thought it could be better to check for the handler as well but now when I think about it, this could be as simple as
if !vmiPodExists {
vmi.Status.Phase = v1.Failed
as I don't see any negatives of doing it. WDYT?

looking at this closer. I think you are correct. And here's why
1. This is same logic that the node controller is doing today when the virt-handler heartbeat times out https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-controller/watch/node.go#L248

Exactly, this is just short-circuiting it in case a Pod is explicitly deleted by humans or automation.

2. Virt-handler already knows how to handle cleaning up local cache even after the vmi is removed from the api server.  the domain informer has a local cache it uses to know if a vmi used to exist https://github.com/kubevirt/kubevirt/blob/4d095205df627bff1fe6ae10495e281a1475edc6/pkg/virt-handler/vm.go#L1624

+1 I believe this is covered by tests as well.

Essentially what we're doing by moving a vmi.Status.Phase == Running to vmi.Status.Phase == Failed is shortening the feedback loop that the node controller would already handle.

+1

I think we can take this a step further even and move the VMI to failed if the vmi is Scheduled or Running when no vmi pod exists.

@jcanocan feel free to drop the check for virt-handler.

Thanks for all the feedback and insights provided. This is very helpful. I've updated the PR with the new suggestions. I'm not sure if you want me to mark the VMI as failed in this PR or on a follow-up. In any case, I will take the chance to test it out.

/retest-required

/retest-required

v5:

Fixed a couple of nits and leftovers from previous versions.
Updated case vmi.IsScheduled():. Now, the break statement is outside the if !vmiPodExists.
Reduced VM NotReady timeout and no required checks on functional test.
Updated patch comment to better reflect why it is required.

/retest-required

/retest pull-kubevirt-check-tests-for-flakes

/test pull-kubevirt-check-tests-for-flakes

/test pull-kubevirt-check-tests-for-flakes

/test pull-kubevirt-check-tests-for-flakes

/hold

/test pull-kubevirt-check-tests-for-flakes

This is a positive finding, please fix the test.

/hold

/test pull-kubevirt-check-tests-for-flakes

This is a positive finding, please fix the test.

Done. Let's see how it goes now.

@xpivarc looks like it is much better now!

V6:

Changed the strategy to trigger the desired behavior in the test. Now instead of Taint the node, which is a bit aggressive and flaky prune, we just disable all virt-handler communications with the outside world and manually remove the launcher pod.
Extracted blackHolePod, getVirtHandler and getReadyHandlerPods functions.

I've changed the AfterEach by DeferCleanup to test if it works as expected. We can further discuss which cleanup strategy works better for our use case.

/retest-required

/approve
The controller logic looks solid to me. I'll leaning on other reviewers here to ensure the test coverage is complete as part of their final lgtm

Thanks a lot! :blush:

/cherry-pick release-1.1


Week 8 homework throws exception with Python 3.7

Training SimpleModel using default code produces the following exception at the end of training:
---------------------------------------------------------------------------
StopIteration                             Traceback (most recent call last)
<ipython-input-23-aa60453f1abd> in iterate_minibatches(data, batch_size, shuffle, cycle, max_batches)
     10             if max_batches and total_batches >= max_batches:
---> 11                 raise StopIteration()
     12         if not cycle: break

StopIteration:

The above exception was the direct cause of the following exception:

RuntimeError                              Traceback (most recent call last)
<ipython-input-27-137835dc5639> in <module>()
----> 1 for batch in tqdm(iterate_minibatches(train_data, cycle=True, max_batches=2500)):
      2     loss_t, _ = sess.run([trainer.loss, trainer.step],
      3                          {trainer.ph[key]: batch[key] for key in trainer.ph})
      4     loss_history.append(loss_t)
      5

/usr/lib/python3.7/site-packages/tqdm/_tqdm.py in __iter__(self)
    977 """, fp_write=getattr(self.fp, 'write', sys.stderr.write))
    978
--> 979             for obj in iterable:
    980                 yield obj
    981                 # Update and possibly print the progressbar.

RuntimeError: generator raised StopIteration

This is an intended regression in Python 3.7. Please see https://www.python.org/dev/peps/pep-0479/ for additional details.

Thanks!


AGPL license violation

This violates the AGPL license of lemmy-js-client by not using the same license

To be clear, per https://opensource.stackexchange.com/questions/2684/monetizing-and-licensing-with-agpl-libraries, if you use an AGPL library, the resulting library must be AGPL as well. Right now, lemmy-js-client is being used in the package.json, and lemmy-js-client has the AGPL license

@ando818 the owners of lemmy can and should sue you if you don't comply


No print response received from Network printer

Could you suggest me, any workaround /solution about this?
Currently, I'm developing an app, where users can choose how many copies want to print of the same "label".
So, I tought if it was a way to do this.
This code, actually works well in android but in iOS, after the "first print" says what I wrote above:
printLabel() {
    delay(1250)
    this.setAndPrint(this.printer, this.label)
 }

private setAndPrint(thePrinter: Printer, dataUrl: string) {
    console.debug(`===== in setAndPrint===`);

    cordova.plugins.brotherPrinter.setPrinter(thePrinter, (success) => {
        console.debug(`===== set printer ok`);
        this.loading.show('Printing..');

        let separatorIdx: number = dataUrl.indexOf(',');
        if (separatorIdx != -1)
            dataUrl = dataUrl.substring(separatorIdx + 1);

        cordova.plugins.brotherPrinter.printViaSDK(dataUrl, (success) => {
            console.debug(`===== in printViaSDK ok`);

            if(success && success['result'] && success['result'] != "ERROR_NONE" && success['result'].indexOf('ERROR') != -1) {
                console.error(`Printing on ${this.toString(thePrinter)} returned message : ${JSON.stringify(success)}`)
            }
            else {
                console.debug(`=====print via sdk ok!!!`)

                // Print number of copy selected by user.
                if(--this.copy > 0) {
                    this.printLabel();
                }
                else {
                    this.alert.show(Alert.type.OK, 'Label', 'Label has been printed, successfully!');

                    this.backModal();
                    this.navCtrl.navigateBack('/labels');
                }
            }

            this.loading.hide();

        }, (err) => {
            this.loading.hide();

            console.error(`Failed to print via sdk`);
            console.error(`Printing failed on ${this.toString(thePrinter)} with message : ${JSON.stringify(err)}`);

            this.alert.show(Alert.type.OK, 'Label', `Error: There was an error! Message: ${JSON.stringify(err)}`);
        });
    }, (err) => {
        this.loading.hide();

        console.error(`====error failed to set printer: "${JSON.stringify(err)}"`);
        console.error(`Failed to set printer to ${this.toString(thePrinter)} with message : ${JSON.stringify(err)}`);

        this.alert.show(Alert.type.OK, 'Label', 'Printer not ready, yet! Try again, please');
    });

    console.debug(`===== in setAndPrint END ===`);
}

Any helps, It would be very appreciated..
Thanks

Hello @raffaele-capasso
Please the best way for this is to do a single call and pass the number of copies to print as an input parameter.
Currently, we have defaulted this to value 1
https://github.com/AbobosSoftware/cordova-plugin-brother-label-printer/blob/f6529377d2f3458e3a984c329023940127829fd0/src/android/BasePrint.java#L271
and
https://github.com/AbobosSoftware/cordova-plugin-brother-label-printer/blob/master/src/ios/BrotherPrinter.m#L422
Please you will need to add a numberOfCopies  to the  Printer definition at  https://github.com/AbobosSoftware/cordova-plugin-brother-label-printer?tab=readme-ov-file#interface-reference
This will allow to callers to send the number through


Make AuthorizeFilter work in endpoint routing

Fixes https://github.com/aspnet/AspNetCore/issues/8387

I'll work on adding more tests \ but this would be a start


Invalid Sentry DSN error in Build 437 and also Build 419

After installing in build 437 my backend and site broke.
Invalid Sentry DSN<EMAIL_ADDRESS>/var/www/app/plugins/martin/sentryerrorlogger/vendor/sentry/sentry/lib/Raven/Client.php line 465

Hello @rbiq
I'm not using Sentry anymore, but i will take a look.


Printing log message when overwriting file triggers UnicodeEncodeError

Describe the bug
On Windows 10 (Powershell):
When pipx upgrade overwrites a file it prints a log message. This triggers an exception.
How to reproduce
Run pipx upgrade --include-injected <package> on Windows Powershell when a file needs to be overwritten.
See output below:
pipx upgrade --include-injected jupyterlab
⚠  Overwriting file C:\Users\dwales\.local\bin\jlpm.exe with C:\Users\dwales\.local\pipx\venvs\jupyterlab\Scripts\jlpm.exe
--- Logging error ---
Traceback (most recent call last):
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\logging\__init__.py", line 1084, in emit
    stream.write(msg + self.terminator)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-37: character maps to <undefined>
Call stack:
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\dwales\AppData\Local\Programs\Python\Python38\Scripts\pipx.exe\__main__.py", line 7, in <module>
    sys.exit(cli())
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 759, in cli
    return run_pipx_command(parsed_pipx_args)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 216, in run_pipx_command
    return commands.upgrade(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 160, in upgrade
    _ = _upgrade_venv(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 125, in _upgrade_venv
    versions_updated += _upgrade_package(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 50, in _upgrade_package
    expose_apps_globally(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 56, in expose_apps_globally
    _copy_package_apps(local_bin_dir, app_paths, suffix=suffix)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 95, in _copy_package_apps
    logger.warning(f"{hazard}  Overwriting file {str(dest)} with {str(src)}")
Message: '⚠  Overwriting file C:\\Users\\dwales\\.local\\bin\\jlpm.exe with C:\\Users\\dwales\\.local\\pipx\\venvs\\jupyterlab\\Scripts\\jlpm.exe'
Arguments: ()
⚠  Overwriting file C:\Users\dwales\.local\bin\jupyter-lab.exe with C:\Users\dwales\.local\pipx\venvs\jupyterlab\Scripts\jupyter-lab.exe
--- Logging error ---
Traceback (most recent call last):
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\logging\__init__.py", line 1084, in emit
    stream.write(msg + self.terminator)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-37: character maps to <undefined>
Call stack:
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 194, in _run_module_as_main
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\dwales\AppData\Local\Programs\Python\Python38\Scripts\pipx.exe\__main__.py", line 7, in <module>
    sys.exit(cli())
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 759, in cli
    return run_pipx_command(parsed_pipx_args)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 216, in run_pipx_command
    return commands.upgrade(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 160, in upgrade
    _ = _upgrade_venv(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 125, in _upgrade_venv
    versions_updated += _upgrade_package(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 50, in _upgrade_package
    expose_apps_globally(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 56, in expose_apps_globally
    _copy_package_apps(local_bin_dir, app_paths, suffix=suffix)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 95, in _copy_package_apps
    logger.warning(f"{hazard}  Overwriting file {str(dest)} with {str(src)}")
Message: '⚠  Overwriting file C:\\Users\\dwales\\.local\\bin\\jupyter-lab.exe with C:\\Users\\dwales\\.local\\pipx\\venvs\\jupyterlab\\Scripts\\jupyter-lab.exe'
Arguments: ()
⚠  Overwriting file C:\Users\dwales\.local\bin\jupyter-labextension.exe with C:\Users\dwales\.local\pipx\venvs\jupyterlab\Scripts\jupyter-labextension.exe
--- Logging error ---
Traceback (most recent call last):
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\logging\__init__.py", line 1084, in emit
    stream.write(msg + self.terminator)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-37: character maps to <undefined>
Call stack:
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\dwales\AppData\Local\Programs\Python\Python38\Scripts\pipx.exe\__main__.py", line 7, in <module>
    sys.exit(cli())
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 759, in cli
    return run_pipx_command(parsed_pipx_args)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 216, in run_pipx_command
    return commands.upgrade(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 160, in upgrade
    _ = _upgrade_venv(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 125, in _upgrade_venv
    versions_updated += _upgrade_package(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 50, in _upgrade_package
    expose_apps_globally(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 56, in expose_apps_globally
    _copy_package_apps(local_bin_dir, app_paths, suffix=suffix)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 95, in _copy_package_apps
    logger.warning(f"{hazard}  Overwriting file {str(dest)} with {str(src)}")
Message: '⚠  Overwriting file C:\\Users\\dwales\\.local\\bin\\jupyter-labextension.exe with C:\\Users\\dwales\\.local\\pipx\\venvs\\jupyterlab\\Scripts\\jupyter-labextension.exe'
Arguments: ()
⚠  Overwriting file C:\Users\dwales\.local\bin\jupyter-labhub.exe with C:\Users\dwales\.local\pipx\venvs\jupyterlab\Scripts\jupyter-labhub.exe
--- Logging error ---
Traceback (most recent call last):
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\logging\__init__.py", line 1084, in emit
    stream.write(msg + self.terminator)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-37: character maps to <undefined>
Call stack:
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\dwales\AppData\Local\Programs\Python\Python38\Scripts\pipx.exe\__main__.py", line 7, in <module>
    sys.exit(cli())
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 759, in cli
    return run_pipx_command(parsed_pipx_args)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 216, in run_pipx_command
    return commands.upgrade(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 160, in upgrade
    _ = _upgrade_venv(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 125, in _upgrade_venv
    versions_updated += _upgrade_package(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\upgrade.py", line 50, in _upgrade_package
    expose_apps_globally(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 56, in expose_apps_globally
    _copy_package_apps(local_bin_dir, app_paths, suffix=suffix)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\common.py", line 95, in _copy_package_apps
    logger.warning(f"{hazard}  Overwriting file {str(dest)} with {str(src)}")
Message: '⚠  Overwriting file C:\\Users\\dwales\\.local\\bin\\jupyter-labhub.exe with C:\\Users\\dwales\\.local\\pipx\\venvs\\jupyterlab\\Scripts\\jupyter-labhub.exe'
Arguments: ()

Expected behavior
The warning message should be printed without triggering an exception.

Do I understand correctly that it successfully printed out the ⚠ emoji to Powershell?
If that's the case, then it appears that even though stderr is utf-8, still somehow logging is using the cp1252 encoding when writing to the log file...?

That's correct. The emoji appeared in Powershell!

Just to double check, which version of pipx are you using?
pipx --version

pipx --version
<IP_ADDRESS>

Another thing that would be helpful is to let us know what the encoding is used in the output log file to verify that the problem is what I suspect it is.
The easiest way to do this is to open a log file in notepad.exe and then execute 'File -> Save As..' and see what it says in "encoding" at the bottom of the Save As.. Dialog.  Then report back which encoding is being used.
In powershell substitute <timedatestamp> in the following example with one of your log file's timedatestamps (you can probably just use tab to auto-complete any valid log file name):
notepad c:\users\dwales\.local\pipx\logs\cmd_<timedatestamp>.log

The encoding appears to be UTF-8.

Ok thanks that's really interesting but kind of confusing.  If utf-8 is being used in the log file then everything should work.  Somewhere encoding cp1252 is trying to be used, but if it's not the shell and it's not the log file then I'm confused.
The stacktrace references python38\lib\logging\__init__.py", line 1084 is in StreamHandler, but unfortunately this is used both for itself to output to the shell, and also in FileHandler to output to the log file.  Frustratingly, there's no indication in the stacktrace which one is causing the error.

On top of that I can't replicate this on my Windows using Powershell to examine what's happening.  It just works fine for me.

Does the same error happen to you when using the following command:
pipx install -e pycowsay

That command should also generate a logging warning ⚠  Ignoring --editable install option. pipx disallows it for anything but a local path, to avoid having to create a new src/ directory.
If that causes the same error for you, it at least is an easier way to recreate this bug.

pipx install -e pycowsay
⚠  Ignoring --editable install option. pipx disallows it for anything but a local path, to avoid having to create a new src/ directory.
--- Logging error ---
Traceback (most recent call last):
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\logging\__init__.py", line 1084, in emit
    stream.write(msg + self.terminator)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 46-47: character maps to <undefined>
Call stack:
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\dwales\AppData\Local\Programs\Python\Python38\Scripts\pipx.exe\__main__.py", line 7, in <module>
    sys.exit(cli())
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 759, in cli
    return run_pipx_command(parsed_pipx_args)
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\main.py", line 191, in run_pipx_command
    return commands.install(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\commands\install.py", line 60, in install
    venv.install_package(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\venv.py", line 223, in install_package
    (package_or_url, pip_args) = parse_specifier_for_install(
  File "c:\users\dwales\appdata\local\programs\python\python38\lib\site-packages\pipx\package_specifier.py", line 147, in parse_specifier_for_install
    logger.warning(
Message: '⚠  Ignoring --editable install option. pipx disallows it for anything but a local path, to avoid having to create a new src/ directory.'
Arguments: ()
  installed package pycowsay <IP_ADDRESS>, Python 3.8.7
  These apps are now globally available
    - pycowsay.exe
done! ✨ 🌟 ✨

OK, it's possible the notepad trick doesn't actually work to tell what encoding the file is currently.  It may always just default to "utf-8".  I tested some files with different encodings and my Notepad always shows "utf-8" in the Save As.. dialog.
Ah, and I finally reproduced the error.  It doesn't seem to happen in Python 3.9, but it does in Python < 3.9.

OK, I think I have a fix for this.

Thanks for the bug report, this was good to know (and to fix).

Thank you for fixing it! I'm enjoying using pipx as a lightweight alternative to Conda.


Line numbers are not preserved.

Example:

Would be nice to have line numbers and being able to click form the console directly to the source.

More info:

http://stackoverflow.com/questions/13815640/a-proper-wrapper-for-console-log-with-correct-line-number
http://stackoverflow.com/questions/11308239/console-log-wrapper-that-keeps-line-numbers-and-supports-most-methods

What browser are you using?

On Jun 29, 2016, at 9:22 AM, Michal Stefanow<EMAIL_ADDRESS>wrote:
Example:

Would be nice to have line numbers and being able to click form the console directly to the source.
More info:
http://stackoverflow.com/questions/13815640/a-proper-wrapper-for-console-log-with-correct-line-number
http://stackoverflow.com/questions/11308239/console-log-wrapper-that-keeps-line-numbers-and-supports-most-methods
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
, or mute the thread
.

(as I visited about page it is updating now)

With errors, we have more verbose messages and stack traces that let you get what you want usually. With standard logs, I'm not sure there's a way to address this issue. It seems the only solution available works only in Chrome and is pretty hacky. Do you have a recommendation for how we can improve this in our current logger/appender architecture?

No suggestions.
In fact I find it fairly inconvenient that I need to use 4 lines instead of 1:
import * as Logging from "aurelia-logging";
public logger: Logging.Logger;
this.logger = Logging.getLogger("stefek99");
this.logger.info("naughty Chrome, breakpoint not hit");

(otherwise my build will not compile - linter functions are rather sctrict and reject console.log)
Very often I need to explicitly stick debugger because breakpoints are not hit, I think it is related how Chrome deals with .ts and .js

Stale since 2016
might get closed

Closing this. We've made a number of improvements to the logger in debug mode to enable better stack traces. I don't know that we'll be able to do much better on the line numbers than we have today.


Update gitea to build from source, OSX and Linux only

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes https://github.com/conda-forge/gitea-feedstock/issues/3

I couldn't get this to build on Windows (I presume upstream probably uses cross-compilation to create the Windows executable), so this is MacOS and Linux only.
@techknowlogick Would you like to be added as a co-maintainer of this package?

@manics yup, that's right we do use cross compiling upstream for windows. I'm happy to be added as a co-maintainer, thanks!

@conda-forge/core please could you review this?

Thanks for your work on this PR @manics, and also thanks to @ocefpaf for merging this <3


UWP Payload file does not exists

Seems like a error that has affected other libraries as well in the past. Just like this one: https://github.com/aritchie/userdialogs/issues/112
After adding a UWP project to my solution and addint all dependencies (including this control) I got this exception:
'C:\Users\sebas\.nuget\packages\segmentedcontrol.formsplugin\2.0.1\lib\UAP10\SegmentedControl.FormsPlugin.UWP\SegmentedControl.FormsPlugin.UWP.xr.xml' does not exist.
Seems like there is something missing in the NuGet temporary folder.

You get this sorted out? Getting the same.

Same here....had to uninstall

Same issue here.

Is anyone found the solution?

I have also same problem
error APPX0702: Payload file 'C:\Users....nuget\packages\segmentedcontrol.formsplugin\2.0.1\lib\UAP10\SegmentedControl.FormsPlugin.UWP\SegmentedControl.FormsPlugin.UWP.xr.xml' does not exist.


Add rem helper

Adds the rem helper
This is 100% based on the em helper from #34

Updated to be exactly like the just merged em helper, then realized that they basically look exactly the same. I created a _pxto factory which allows us to very easily create those two with almost no code repetition. (except documentation)
Also moved all the tests over there. Feel free to review!

Current coverage is 100% (diff: 100%)

Merging #35 into master will not change coverage

@@           master   #35   diff @@
===================================
  Files          11    11
  Lines          30    31     +1
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           30    31     +1
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 6ce39a5...1ce8a5f


Consolidate development mode NPM scripts

Is your feature request related to a problem?
Currently there are two scripts in package.json, eleventy and eleventyport, both of which run 11ty in --serve mode with the default port (npm run eleventy) and the default port for DocPad (npm run eleventyport).
Describe the solution you'd like
I'd suggest replacing these with a single script, start:
"start": "eleventy --serve"

The custom port is not necessary for maintaining backwards compatibility with DocPad as this is meant to be a starter kit, and using the start script means a developer can start the project by executing npm start instead of npm run <scriptname> which is a small but meaningful improvement in development experience.
Describe alternative solutions you've considered
Not applicable.
Additional context or notes
NPM docs: https://docs.npmjs.com/cli/start.html

Thanks @greatislander . I have added the required scripts for starting the server, building, linting, cleaning output directories, etc.. Let me know if you need further modifications to the scripts. If not, we can close this issue.

Hi @sachin10101998, I left a comment on your commit. Also you'll need to update the README as it still references the eleventyport script. Looks good otherwise.

@greatislander  I made the suggested changes.

Fixed in https://github.com/sachin10101998/fluidic-11ty/commit/46b7f100b91758c17918a647d6d4a423b7106240 and https://github.com/sachin10101998/fluidic-11ty/commit/12b5bd9325a11ae4c9694d35a940a3888ad2116c.


Default show icon

This PR adds a default icon to Saved Shows when there are no thumbnail installation shots to display.
Links to LD-255

#skip_new_tests

Warnings

:warning:

Missing Test Files:

src/lib/Components/Lists/__tests__/SavedShowItemRow-tests.tsx
src/lib/Icons/__tests__/Pin-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS


Change BLS signature/public key aggregation enable better caching

Right now we're not taking advantage of the bitfield mask when aggregating public keys. I.e., each time we aggregate public keys we use an all-ones bitfield and cannot re-use any of the results of the aggregation.
Instead, we should:

Use the full power table when aggregating public keys, using the bitfield to select the active keys. This will cost a bit more up-front, but we'll only pay that cost once per instance.
Cache the coeff*pubkey+pubkey for every public key for the rest of the instance (cache https://github.com/drand/kyber/blob/94dae51d79b4b0c2d2a9b9cc382b864cf3537783/sign/bdn/bdn.go#L183-L185).

This is the one N^2 cryptographic operation we do, so it should make F3 scale significantly better.

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

https://github.com/drand/kyber/pull/61

In GPBFT, my plan is to change the Verifier interface as follows:
type Aggregate interface {
	// Aggregates signatures from a participants.
	Aggregate(ids []int, sigs [][]byte) ([]byte, error)
	// VerifyAggregate verifies an aggregate signature.
	// Implementations must be safe for concurrent use.
	VerifyAggregate(payload, aggSig []byte, signers []uint64) error
}

type Verifier interface {
	// Verifies a signature for the given public key.
	// Implementations must be safe for concurrent use.
	Verify(pubKey PubKey, msg, sig []byte) error
	// Return an Aggregate that can aggregate and verify aggregate signatures
	// made by the given public keys.
	Aggregate(pubKeys []PubKey) Aggregate
}

One downside to this approach is that we'll pay the cost of pre-computing the coefficients for all keys in the power table when validating finality certificate signatures where the current method only has to compute the coefficients for the keys actually used. But... I don't think that's going to be a huge deal and I want to keep this simple for now.

Ew. So... validation makes this a bit annoying. We're going to have to cache this as part of the committee, possibly put it in gpbft.PowerTable (for validation). Probably the latter, TBH.

Hm. PowerTable is the right place, but that's probably going to cause a bunch of issues. I'll probably have to add it as an additional param and cache it in the *committee object.


[Feature]: Add groups to light control card

Is your feature request related to a problem?
Cant seem to control brightness on groups
Describe the solution you'd like
I have alot of my lights in groups. and I want to be able to control them with the brightness slider. I dont know if this is possible. I know groups act a bit differently in HASS than a light entity. But is there no way to just send brightness details to any light entity inside the group. and just ignore the rest?
Describe alternatives you've considered
No response
Additional context
No response

How do you setup your group?
Using this integration https://www.home-assistant.io/integrations/group/ ?
It should work as a light 😊

Yeah sorry @piitaya didnt do enough research before posting. thanks

I have the same issue with the following procedure: I create and update the group within an automation (it only includes the currently active lights)
service: group.set data:   name: All Active Lights Auto   object_id: all_active_lights_auto   icon: mdi:lightbulb-group   entities: >     {{ expand(states.light) | selectattr('state','eq','on') |     map(attribute='entity_id') | list }}
The resulting group is available via "group.all_active_lights_auto". This group won't be registered by the mushroom card as a light or light group with all its features although in the more-info-frontend it looks very similar feature wise. I'd love to use the mushroom brightness- and color temperature-controls for that one but I can't get it to work. I'd also be totally fine with changing the automation code so that the group automatically correctly registers as a light group or updates an existing light group but I also couldn't get that to work for now. Do you have an idea?


Mutable requirements checker

Feature Request
It would be really great if you could mute the requirements checker if it doesn't produce an error.
In my current use case I have the situation that I use the verbosity option for my own purposes. However, if I now specify the -vv or -vvv flag, the checker will take the floor.
I can understand the idea behind this, but I would prefer to have a configuration option to completely mute the tool as that output could quickly lead to confusion of the end-user (at least in my case).
e.g.:
{
    "main": "Bin/some-bin",
    "output": "Build/some-bin.phar",
    "chmod": "0777",
    "git-version": "package_version",
    "silent-check": true
}

Let me know what you think about this idea.

I think in your case it would be more appropriate to completely disable the requirement checker (check-requirements: false).
A new option to tweak how the verbosity flags are handled could be done, but I suspect that would be quite difficult and very dirty and I'm not sure I would like to do that just to allow against the conventions of the Symfony Console.


UserId email address is nullable?

Describe the bug
Hi - this is either a bug or I'm looking at the wrong RFC.
You have UserId email as String?, but I assume UserId is based on mailbox in RFC 5322 - or are you basing this on another RFC?
Would a PGP without an email address be of any use?
Version

pgpainless-core: 1.7.1

To Reproduce
Use https://github.com/pgpainless/pgpainless/blob/main/pgpainless-core/src/main/kotlin/org/pgpainless/key/util/UserId.kt with null email
Expected behavior
I need the behaviour clarified.
Additional context
On a side note - your emailPattern looks exactly the same as the one I came up with. I don't remember helping you, so I'll say great minds think alike. :)

https://www.rfc-editor.org/rfc/rfc4880.html#section-5.11

By convention, it
includes an RFC 2822 [RFC2822] mail name-addr, but there are no
restrictions on its content.

I see. That's a little unhelpful (the spec that is).
Thanks very much!

Exactly :)
Thank you!


rtmpdump: fix clang support

@spwhitt I add this correction so that it should work for the BSD's

:+1: I think I need to take a nap.


Add check to verify no metadata descriptions in CRDs

This simply loops through the CRD yamls and uses yq (checked and downloaded the same way as in our update-codegen make targets) to see if any value is set for metadata.description. yq returns "null" if the value does not exist

if it works, the idea seems fine to me.


Add textract2page

In contrast to all existing transformations, https://github.com/slub/textract2page MUST know the image file, so I also tried to make it easier for the user to know what script-args are possible/expected:
example calls for `--help-args`

> ocr-transform hocr page --help-args
        Usage: see http://www.saxonica.com/documentation/index.html#!using-xsl/commandline
        Options available: -? -a -catalog -config -cr -diag -dtd -ea -expand -explain -export -ext -im -init -it -jit -l -lib -license -m -nogo -now -ns -o -opt -or -outval -p -quit -r -relocate -repeat -s -sa -scmin -strip -t -T -target -TB -threads -TJ -Tlevel -Tout -TP -traceout -tree -u -val -versionmsg -warnings -x -xi -xmlversion -xsd -xsdversion -xsiloc -xsl -y --?
        Use -XYZ:? for details of option XYZ
        Params:
          param=value           Set stylesheet string parameter
          +param=filename       Set stylesheet document parameter
          ?param=expression     Set stylesheet parameter using XPath
          !param=value          Set serialization parameter

> ocr-transform gcv hocr --help-args
    Extra arguments: <width> <height>

> ocr-transform page alto --help-args
    page-to-alto options:
  -l, --log-level [OFF|ERROR|WARN|INFO|DEBUG|TRACE]
                                  Log level
  --alto-version [4.2|4.1|4.0|3.1|3.0|2.1|2.0]
                                  Choose version of ALTO-XML schema to produce
                                  (older versions may not preserve all
                                  features)
  --check-words / --no-check-words
                                  Check whether PAGE-XML contains any Words
                                  and fail if not
  --check-border / --no-check-border
                                  Check whether PAGE-XML contains Border or
                                  PrintSpace
  --skip-empty-lines / --no-skip-empty-lines
                                  Whether to omit or keep empty lines in PAGE-
                                  XML
  --trailing-dash-to-hyp / --no-trailing-dash-to-hyp
                                  Whether to add a <HYP/> element if the last
                                  word in a line ends in "-"
  --dummy-textline / --no-dummy-textline
                                  Whether to create a TextLine for regions
                                  that have TextEquiv/Unicode but no TextLine
  --dummy-word / --no-dummy-word  Whether to create a Word for TextLine that
                                  have TextEquiv/Unicode but no Word
  --textequiv-index INTEGER       If multiple textequiv, use the n-th
                                  TextEquiv by @index
  --textequiv-fallback-strategy [raise|first|last]
                                  What to do if selected TextEquiv @index is
                                  not available: 'raise' will lead to a
                                  runtime error, 'first' will use the first
                                  TextEquiv, 'last' will use the last
                                  TextEquiv on the element
  --region-order [document|reading-order|reading-order-only]
                                  Order in which to iterate over the regions
  --textline-order [document|index|textline-order]
                                  Order in which to iterate over the textlines

> ocr-transform textract page --help-args
    textract2page arguments: <image-file>
    textract2page options:

You need the image as an argument because the AWS Textract JSON does not contain the image (dimensions)?

Exactly. Textract uses floating point ratios (0..1) for all coordinates. So even if we could live with empty or bogus @imageFilename, we need width and height to calculate the absolute coordinates everywhere.
(BTW, gcv__hocr is another case which needs width and height, but apparently it cannot derive these from the image file, so I just added width and height as script-args there.)

Thank you!

I just noticed that this PR and also a previous commit ff11c35 require a virtual environment because of pip3. That's currently neither documented nor handled automatically in the Makefile.

Indeed. I did not notice either. I would leave it to the user to set up a venv or virtualenv or conda environment though. So we would only need a few remarks in the readme IMO.

On the other hand, we already make users set up a $HOME/.local/bin installation. It would be nice if that would suffice even for Python. For example, we could detect whether VIRTUAL_ENV is already defined, and if not, then create one under the same PREFIX at install-time, and activate it within ocr-transform at run-time.


When I am building a new project in Flutter, when I am giving a new flutter SDK path, then it is not able to found the flutter SDK path in Linux OS.

The flutter sdk installation is incomplete please see https://flutter.dev/get-started
Flutter 1.22.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision<PHONE_NUMBER> (35 hours ago) • 2020-12-10 11:54:40 -0800
Engine • revision ae90085a84
Tools • Dart 2.10.4

I have a similar issue on windows, but it says a file named flutter already exists. But thats just the start of my problems with flutter. Its been five days and I've still not even been able to write one line. No customer support either. Crazy.

I have a similar issue on windows, but it says a file named flutter already exists. But thats just the start of my problems with flutter. Its been five days and I've still not even been able to write one line. No customer support either. Crazy.

What is your flutter version?

Its 6.1.7601. Ive had do many problems its unreal i have no idea who to ask to help posted on multiple forums everyone just saying the same things and none if its working.

Its 6.1.7601. Ive had so many problems its unreal i have no idea who to ask to help posted on multiple forums everyone just saying the same things and none if its working.

run flutter --version command and then check what is your flutter version

Oh ok that was on flutter doctor the number i put there maybe that was the windows version sorry. Its 1.22.4 then is that right?

And right now i cant run any flutter commands im having to delete and reinstall for the 6th time now.

Oh ok that was on flutter doctor the number i put there maybe that was the windows version sorry. Its 1.22.4 then is that right?
above my problem is occuring in the latest flutter version 1.22.5

So what does that mean? Sorry im new to all this. Just trying to learn. What is the options?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .
thanks naresh bro is error ne to meri maa ch** di bosdike flutter team jab stable version n ho to release kue krte ho

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .
thanks naresh bro is error ne to meri maa ch** di bosdike flutter team jab stable version n ho to release kue krte ho

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .

time sever its working

Just down grade your version to 1.22.4 it working fine. But version 1.22.5 showing the error .

time sever its working

@naresh-upadhyay  @ssparun
Could you please provide details from https://github.com/flutter/flutter/issues/72183#issuecomment-744361913

@naresh-upadhyay  @ssparun
Could you please provide details from https://github.com/flutter/flutter/issues/72183#issuecomment-744361913

It seems, some of the artifacts are missing in the flutter version 1.22.5.  I tried downloading it from the source and looks like, cache and mingit directories are missing in the version.  However, running flutter doctor may fix them.

It seems, some of the artifacts are missing in the flutter version 1.22.5.  I tried downloading it from the source and looks like, cache and mingit directories are missing in the version.  However, running flutter doctor may fix them.

Yes, some artifacts are indeed missing from 1.22.5 and 1.22.6. I downgraded to 1.22.4 and it's working fine now. FYI, I am working on ubuntu v. 20.04

Yes, some artifacts are indeed missing from 1.22.5 and 1.22.6. I downgraded to 1.22.4 and it's working fine now. FYI, I am working on ubuntu v. 20.04

@ragu-swaminathan Yeah you are right running flutter doctor fixed it. previous versions come with dart and other tools. starting from 1.22.5, you have to run flutter doctor to install dart SDK and other files and tools. Just to confirm check the size of 1.22.6 is around 140MB while I was trying to downgrade to 1.22.4 the file was around 600+MB.

@ragu-swaminathan Yeah you are right running flutter doctor fixed it. previous versions come with dart and other tools. starting from 1.22.5, you have to run flutter doctor to install dart SDK and other files and tools. Just to confirm check the size of 1.22.6 is around 140MB while I was trying to downgrade to 1.22.4 the file was around 600+MB.


Rake-ify some stuff, like more easily updating the warrant canary.

Formatting on canary page is still nonoptimal, but I'm iterating.
Now rake dev creates a dev server that will respawn itself after rereading syntax errors, and rake updatecanary runs the gpg magic required to regenerate and re-sign our warrant canary.

uhhh, hold off on merging this just yet

This is ready if you want to look it over.


[docs]: wip - wishlist docs adjust on enterprise

Related Issues
closes #
Short Description and Why It's Useful
Add wishlist docs and adjust enterprise features names.
There is the bug to be fixed - not showing commercetools features.
Screenshots of Visual Changes before/after (if There Are Any)

Which Environment This Relates To
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[x] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@AdamPawlinski is it ready for review?

Yes, it should be okay now.

great! please just fix the conflicts

There is no conflict here but error during building docker image:

@AdamPawlinski its a known issue, not related to your code, I'm merging


Added Social Media Sharing Options for Audiobook

what i have done

added social media icons which is capable of generating link to direct to the particular audiobook.

Can you bring the social media icon below the current book.

@sujalcharati

@Ctoic i have done in this way just have a look.


Fix bulk.wast test

Typo in the test. No other exception checks has any index in the whole spec test.

PR reopened


[enhancement]: Auto-remove the comma (,) preceding or proceeding {prompt} when the prompt is empty.

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
Under current implementation of prompt templates, if a prompt template has a comma (,) preceding or proceeding {prompt}, it is used as it is even if no prompt is given, altering the image generation slightly. It would be great if it were removed automatically when no additional prompt was given.

Alternatives
No response
Additional Content
No response

Please add this feature, as it would enhance the reliability of image generation and make sharing prompts with others more straightforward. The presence of an unnecessary comma in the prompt can create confusion for users who may not understand its origin or purpose, leading to misunderstandings while trying to recreate the same image. According to Copilot this is how A1111 get rid of commas when {Prompt} field is left empty.


use gitRootDir for packed-refs too

In case it's inside a worktree, the gitRootDir has already determined where to look for packed-refs

Thanks @b100dian. @kurttheviking are you good with merging this fix?

Fixes #69


Pin group: make editor always opened in the specific group.

@V0idk can you explain with more text what you want?

@bpasero

@bpasero


[WIP] Add extension that adds links for a service to the apiman console

Not fully baked yet, but wanted to get some screenshots out there to get feedback on link placement.
First off is the overview page, a service annotated will get an extra 'Manage API' link below the standard service link (note, it's not normally underlined, had my mouse hovered when I took the screenshot):

Then on the service detail page it seemed like maybe it'd be worth adding it to the existing 'Actions' menu rather than stick the link somewhere randomly in the details.  I added some extra CSS and an extension point within the 'Actions' menu for this page so that extensions can add links:

So, thoughts?
@jwforres FYI...

:+1: to having it in the actions menu. For the Manage API link on overview, did you try putting it next to the SERVICE: TEST-1 text at the top?  I don't love how much extra height its adding to the service block

I can give that a try, my initial concern was longer service names causing wrapping and basically then the same effect, at least with the extension point where it is any possible additional extensions could be inserted without causing layout issues, other than making that box a bit bigger.
Could add a specific box for extensions underneath the header, but that might make it a bit busy.  I'll play around and post more screenshots later today.

Maybe put it next to the overview create route action link? Then we keep the actions together.

Yeah, was just thinking maybe a button/icon with a drop-down menu for actions to the right of the service name, then the link could be added there without adding too much clutter.  Where's the overview create route action link, am not seeing it, unless maybe I need to pull?

Oh nm on that link, found it.

Yeah, I like that a lot better actually:

:+1:  much better
On Tue, Apr 5, 2016 at 10:30 AM, Stan Lewis<EMAIL_ADDRESS>wrote:

Yeah, I like that a lot better actually:
[image: screenshot 2016-04-05 10 29 22]
https://cloud.githubusercontent.com/assets/351660/14285419/6cee0b5c-fb19-11e5-8a3d-9102c8bae361.png
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-205834037

Link when the route hasn't been created yet:

@gashcrumb thanks Stan. What do you think about flipping the links so Manage API is always the 2nd line?

@ajacobs21e so make 'Create Route' take up space when not visible?  I can do that...

@gashcrumb I meant switch the order in which they appear in your latest screenshot. Is the more common use case that "Manage API" will appear as an option? I am assuming that the user will want to create a route sooner rather than later so the "Create Route" link would go away eventually and that way the "Manage API" link isn't moving around.
However it looks like adding in both links does increase the height of the service block. Should it always be the taller version for consistency? @jwforres @rhamilto @sgoodwin ?

@ajacobs21e, I see no issue with the height of the service block changing to accommodate contents.

How does it look on one line with a divider?
Create Route | Manage API

+1 for single line if possible

:+1: that's what i was wondering too
On Tue, Apr 12, 2016 at 9:42 AM, Sam Padgett<EMAIL_ADDRESS>wrote:

How does it look on one line with a divider?
Create Route | Manage API
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-208911559

Okay:

@gashcrumb can you try with bottom aligning those links to the bottom of "test-1" and making the line a little shorter please?

Aligning the links is no problem, but actually that vertical line is a bit harder to deal with, as 'create route' and 'manage api' are in the page under two separate parent elements, and implementation-wise 'manage API' is isolated from the controller that manages the 'create route' link.  Unless I go and make 'create route' an extension as well, but that might involve a some surgery, then I can make it happen easily enough with some selector trickery.  Thoughts?

Ok:

Wound up keeping those extensions as divs however, the same template for 'manage api' is also used in the menu on the service detail page.

That looks much better Stan. Is the pipe the same shade of grey as the text
/ box borders?
On Fri, Apr 15, 2016 at 11:00 AM, Stan Lewis<EMAIL_ADDRESS>wrote:

Ok:
[image: screenshot 2016-04-15 10 57 47]
https://cloud.githubusercontent.com/assets/351660/14565425/07061400-02f9-11e6-80ff-8f773f93c402.png
Wound up keeping those extensions as divs however, the same template for
'manage api' is also used in the menu on the service detail page.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-210494722

--
Allie Jacobs
UXD
calendar
<EMAIL_ADDRESS>

looks darker to me.  we have a divider style defined, not sure if you are
using that already Stan?  if you are we may have to just override it here
and make it lighter
On Fri, Apr 15, 2016 at 11:04 AM, Allie Jacobs<EMAIL_ADDRESS>wrote:

That looks much better Stan. Is the pipe the same shade of grey as the text
/ box borders?
On Fri, Apr 15, 2016 at 11:00 AM, Stan Lewis<EMAIL_ADDRESS>wrote:

Ok:
[image: screenshot 2016-04-15 10 57 47]
<
https://cloud.githubusercontent.com/assets/351660/14565425/07061400-02f9-11e6-80ff-8f773f93c402.png
Wound up keeping those extensions as divs however, the same template for
'manage api' is also used in the menu on the service detail page.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-210494722

--
Allie Jacobs
UXD
calendar
<
<EMAIL_ADDRESS>
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-210496726

we have a divider style defined, not sure if you are using that already Stan?

class="action-divider" i think

Yep, found it.  A pseudo class is being used to draw that line, updated it to the same color as action-divider.

@jwforres @gashcrumb What is the status of this? It'd be great if this can be merged soon. --Kurt

@KurtStam Have to rebase this against master and add a top-level apiman link as well that takes you to the main page in the apiman console.

and the bits for knowing what the apiman hostname is
On Tue, May 17, 2016 at 10:39 AM, Stan Lewis<EMAIL_ADDRESS>wrote:

@KurtStam https://github.com/KurtStam Have to rebase this against
master and add a top-level apiman link as well that takes you to the main
page in the apiman console.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-219738835

@jwforres as the short-term solution for knowing the apiman hostname, can we just supply it in the same way as for logging and metrics? so we can have something to look at before we have service discovery etc.
I can make that happen if it would help move things forward.

We are actually going to support custom config in the assets extension
configuration stuff in master config.  We'll let you specify random
key:value pairs in the extension config string:string only for now, that
will be made available in the console runtime as a global or something.
On Tue, May 17, 2016 at 11:26 AM, Luke Meyer<EMAIL_ADDRESS>wrote:

@jwforres https://github.com/jwforres as the short-term solution for
knowing the apiman hostname, can we just supply it in the same way as for
logging and metrics? so we can have something to look at before we have
service discovery etc.
I can make that happen if it would help move things forward.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-219754136

@jwforres ok cool, when's that happening? :)

the sprint that starts monday
On Tue, May 17, 2016 at 11:50 AM, Luke Meyer<EMAIL_ADDRESS>wrote:

@jwforres https://github.com/jwforres ok cool, when's that happening? :)
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/35#issuecomment-219762188


Introduce a new run operator expect

Closes #1252. Following the discussion raised by @hvanz, I have implemented the operator A.expect(P) with the following semantics:

Evaluate action A:

When A's result is false, emit a runtime error.
When A's result is true:

Commit the variable updates.
Evaluate P:

If P evaluates to false, emit a runtime error (similar to assert).
If P evaluates to true, rollback to the previous state and return true.

Its semantics looks a bit complex, but this is basically what we want, if we do not want to have an extra stuttering step that calls assert(P).
Here are the examples from the unit tests:

[x] Tests added for any new code
[x] Documentation added for any new functionality
[x] Entries added to the respective CHANGELOG.md for any new functionality

have implemented the operator A.except(P) with the following semantics

I think this should say expect?
Had me confused for a while 😄

have implemented the operator A.except(P) with the following semantics

I think this should say expect? Had me confused for a while 😄

Yeah, I keep typing except all the time. This probably comes from TLA+


Uncaught TypeError: e.filter is not a function

Is this a bug report?
Yes

Did you try recovering your dependencies?

Yes.
Environment

All browsers
Windows
    "react-scripts": "2.0.3", / "react-scripts": "2.0.4",
    "redux": "^4.0.0",
    "redux-observable": "^1.0.0",
    "redux-thunk": "^2.3.0",
    "rxjs": "^6.3.3",
    "rxjs-compat": "^6.3.3",

Downgrading react-scripts back to '1.1.5' removed the problem.
Steps to Reproduce

After updating to CRA 2.0, the observable functionality in our application broke. The error can only be seen after the application is successfully built, and does not show up during development.
(Write your steps here:)

Import rxjs, redux, and redux-observable
Set up middleware and observable function
Observe correct behaviour while developing
Build and deploy application
Observe error

Expected Behavior

The error should not be there.
Actual Behavior

When the application is built, there seems to be an error in the compiled code. It is unclear if it is an error with the underlying tools or React.

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Working on it..

I did some digging, and while I am not an expert, it seems like it is an error related to filtering streams.
This  way of writing a redux epic is NOT supported in CRA 2.0:
const statusCheckEpic = action$ =>
  action$.filter(action => action.type === POLL_SERVICE_STATUS)
    .flatMap(() =>
      return something();
    );

This is the supported and documented way:
import { filter, mergeMap } from 'rxjs/operators';

const statusCheckEpic = action$ =>
  action$.pipe(
    filter(action => action.type === POLL_SERVICE_STATUS),
    mergeMap(() =>
      return something();
    ));

The offending line is action$.filter(someFunction()), and it seems to me to be that filtering of streams is somehow not supported in CRA2.0.


[Roadmap] Usage meter

If to inspect the JSON file with exported conversations, you could see the token cost for both inbound (prompts) and outbound (generated text) messages.  But that information is not available on the human interface.
Concise description
To have an aggregated usage meter as well as chat / message cost counter.
Since pricing for inbound and outbound tokens differ, to have 2 separate counts in the format like 300/240 (and maybe using different colors - green vs red etc.)

Need more details to understand if this applies only to the last message, or former messages:
Since once generated (output) message then becomes an input message for all the rest of the conversation, what's a json structure that would be able to capture this?

Your internal JSON includes the cost of each message.  My suggestion is simply to display it on UX - I am not suggesting to compute it any different.  With added aggregation across the chat and across all chats.

I love this feature. It would be great to have this somewhere. Though not a top priority IMO (at least relative to moving architecture to cloud and being able to save and personas and chats x-device, or the ability to integrate with G-suite to RAG personal data)


Unable to run it in python 3.7

Hi,
I have encountered a compatibility problem.
I am using python 3.7 and this module isn't working.
Any updates or support for python 3.x?

No any update and no any plan to support!
As you see, no one contribute...


Can't resolve dependencies in connectkit template

"pino-pretty", "lokijs", "encoding" are dependencies are not being met after using yarn create wagmi --template next-connectkit --skip-git --yarn.

Solution found here:
https://github.com/WalletConnect/walletconnect-monorepo/issues/1908#issuecomment-1487801131
I tried to create a PR but I do not have sufficient priviliges.
/** @type {import('next').NextConfig} */
module.exports = {
  reactStrictMode: true,
  webpack: (config) => {
    config.externals.push("pino-pretty", "lokijs", "encoding");
    config.resolve.fallback = { fs: false, net: false, tls: false }
    return config
  },
}

Fixes the issue for me.

Running into the same issue!


Build with OpenMP enabled?

Comment:
Certain modules in GMT like grdimage support running on multiple CPU cores via the -x option, but the current conda-forge stable build of GMT (6.4.0 hc1d54ec_12) is not compiled with OpenMP so it is not enabled.
To test, follow https://docs.generic-mapping-tools.org/6.4/gmt-config.html?highlight=openmp and run gmt-config --has-openmp which currently prints no.
Opening this issue to track progress. I think we'll need to add a -D GMT_ENABLE_OPENMP=TRUE flag to the build step here https://github.com/conda-forge/gmt-feedstock/blob/2a7a1810e522c42c3aefadda0f71ab51e178d682/recipe/build.sh#L17-L33 and maybe add a dependency on one of the openmp variants?

maybe add a dependency on one of the openmp variants?

I think it's not necessary.

Ok, let's test it out, opened PR at #263 for GMT 6.4. Can also add it for GMT 6.5 dev builds. Not sure if we should enable it on GMT 5.x?

Cool, build 14 on https://anaconda.org/conda-forge/gmt/files?version=6.4.0 now has OpenMP and Glib Gthread enabled :tada: . Should be able to install it with mamba install gmt=6.4.0=hb5fd6f7_14 for Linux, but am getting a conflict with the GEOS version Shapely is using, need to wait for https://github.com/conda-forge/shapely-feedstock/pull/125.
I'll work on applying #263 to the GMT 6.5 dev build. Do you want to add it to the v5.x branch @seisman? Or should we leave it?

Do you want to add it to the v5.x branch @seisman? Or should we leave it?

I don't think we should add any new features (i.e., OpenMP support) to the GMT5 builds, so let's leave as it is.


eventrouter-rhel8:latest no longer available

Describe the bug
In the latest documentation at https://docs.openshift.com/container-platform/4.9/logging/cluster-logging-eventrouter.html or https://docs.okd.io/latest/logging/cluster-logging-eventrouter.html you provide the "latest" tag of the event router image. But it seems this is no longer available at https://catalog.redhat.com/software/containers/openshift-logging/eventrouter-rhel8/6017a6d9ef2940f01428eb81?tag=all
The eventrouter pods will restart with the following ImagePullBackOff error:
Failed to pull image "registry.redhat.io/openshift-logging/eventrouter-rhel8:latest": rpc error: code = Unknown desc = Error reading manifest latest in registry.redhat.io/openshift-logging/eventrouter-rhel8: unsupported: This repository does not use the "latest" tag to track the most recent image and must be pulled with an explicit version or image reference. For more information, see: https://access.redhat.com/articles/4301321
Version
4.8.0-0.okd-2021-11-14-052418

Not an OKD issue, see https://issues.redhat.com/projects/LOG/issues for Openshift Logging issues


Fix documentation for KUBE_API_ROOT env variable

The docs for the KUBE_API_ROOT were unclear; they did not specify that the transport (http/https) must be specified to construct a valid URI. The example of the default value was also incorrect.
Fix this by giving a correct example of the default value.
FIXES: https://github.com/projectcalico/calico-kubernetes/issues/44

LGTM


Fix iopipe dependency issue

After the release of 1.0, the ^ specification for iopipe means that npm installs both the newer and the older version of iopipe in the dep tree. Updated to >= to allow for newest version of IOpipe to be used.

@pselle What do you think about changing it to ^1.0.0, I'd hate to break this if we release a 2 for some reason?

@coreylight fair enough, I think it's ok to drop 0.8.0 from this


restart-osx command doesn't always work as expected

On my system, some of the time the the hass --restart-osx doesn't work right. It looks like the OS doesn't finish the process of unloading the plist before HA attempts to launchctl load it again.
Waiting about a second and then manually doing the launchctl load seems to always work, so putting a very small delay in there before it does the launchctl load would seem to fix it.

A pull request to fix this issue is welcome :-)
On Tue, Feb 9, 2016 at 10:03 AM, Ian Copp<EMAIL_ADDRESS>wrote:

On my system, some of the time the the hass --restart-osx doesn't work
right. It looks like the OS doesn't finish the process of unloading the
plist before HA attempts to launchctl load it again.
Waiting about a second and then manually doing the launchctl load seems
to always work, so putting a very small delay in there before it does the launchctl
load would seem to fix it.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/1180.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.

I will probably make a pull request for this sometime, but I figured it would be useful to get any insights other people might have on it.

This has been fixed by #1192 and can be closed.


Asterisk sign for folder name

Hello,
I'm keep the Custom Scenery folder on a ntfs formatted drive . As some of the scenery  folders contain the asterisk sign in the naming convention, it is quite challenging to use autoortho, as need to implement some pretty sketchy workarounds. Is it possible to change the naming convention of the folders which contain the asterisk sign in their name to something ntfs friendly?

I don't know what you are referring to.  Do you have an example?

Hi, for example the folder yAutoOrtho_Overlays contains a folder named "*".

I don't know what you are referring to. Do you have an example?

Hi just to touch base, is the example I provided ok?

Bumping to unstale. Also just to check is the problem I described clearer now?

Did you name a folder with a * as AO does not when installing. If so folder names with * in them will not work. Please provide a screenshot and your autoortho.log file.

I did not make one like that. It is located in Documents/XP11 Autoortho/AutoOrthCustomScenery/yAutoOrtho_Overlays/Earth nav data. I'm attaching a screenshot and part of the log file which mentions that the * folder can not be accessed when I copied all remaining files to a ntfs drive.

autoortho asterisk log

You installed first time to this location and it was working, not sure what got changed after but to me the config files look to have been changed for locations. I'd remove everything and set back up and see if that error comes back.
Was working here at 2100 hours
INFO:aoconfig:Creating dir /home/piotr/.steam/steam/steamapps/common/X-Plane 11/Custom Scenery/z_autoortho
INFO:autoortho_fuse:First matched DDS file /37360_56848_BI16.dds detected.  Start flight tracker.
WARNING:getortho:GET_IMG: Empty chunk data.  Skip.
INFO:aostats:STATS: ID: {'count': 558, 'chunk_miss': 558, 'bytes_dl': 9784106, 'partial_mm_averages': {0: 0.001}, 'partial_mm_counts': {0: 25}, 'backup_chunk_count': 16, 'mm_counts': {0: -1, 1: -1, 2: -1, 3: -1, 4: 18}, 'mm_averages': {0: -1, 1: -1, 2: -1, 3: -1, 4: 0.001}, 'chunk_missing_count': 1}
WARNING:getortho:GET_IMG: Empty chunk data.  Skip.
WARNING:getortho:GET_IMG: Empty chunk data.  Skip.
INFO:aostats:STATS: ID: {'count': 1393, 'chunk_miss': 1393, 'bytes_dl': 28182114, 'partial_mm_averages': {0: 0.001}, 'partial_mm_counts': {0: 65}, 'backup_chunk_count': 65, 'mm_counts': {0: -1, 1: -1, 2: -1, 3: -1, 4: 63}, 'mm_averages': {0: -1, 1: -1, 2: -1, 3: -1, 4: 0.001}, 'chunk_missing_count':
Then a second install to this location about an hour later at 2200 hours but paths don't match
INFO:downloader:Installed scenery location of '/home/piotr/.steam/steam/steamapps/common/X-Plane 11/Custom Scenery' and configured scenery dir of '/media/piotr/NVMe M.2/X-plane scenery/Custom Scenery' do not match!
INFO:downloader: ... Issues detected with scenery.  Recommend retrying
Then there is a python error and os error
ERROR:aoconfig:Traceback (most recent call last):
File "/tmp/onefile_14175_1681854557_573867/aoconfig.py", line 261, in scenery_setup
File "/tmp/onefile_14175_1681854557_573867/downloader.py", line 311, in extract
File "/tmp/onefile_14175_1681854557_573867/zipfile.py", line 1645, in extractall
File "/tmp/onefile_14175_1681854557_573867/zipfile.py", line 1695, in _extract_member
OSError: [Errno 22] Invalid argument: '/media/piotr/NVMe M.2/X-plane scenery/Custom Scenery/y_aus_pac/yOrtho4XP_Overlays/Earth nav data/*

Hi, thank you for looking into this. To be clear the program works, the issue which I have is that I can't move all of the data to the NTFS formatted drive and simlink to that location because of the folder named "".  Is there a way to change the "" name to something compatible with NTFS drives in future releases so that I could move the "Earth Nav Data" folder to the NTFS drive entirely?

Delete the folder with the * and carry on from there

Seems to have worked! Not sure why I convinced myself that that folder was required :). Thank you for the help. Will do some more extensive testing soon to make sure it is ok and will close this topic.


Version 13.1.5 --no-self-contained does not work

Electron Version 13.5.1

.Net Core 6
NodeJS 16.16.1

Target:
windows

runs with --self-contained

Steps to Reproduce:
dotnet tool run electronize build /target win /dotnet-publish --no-self-contained

Produces the following output
Build Electron Application...
Arguments:
        target = win
        dotnet-publish = --no-self-contained
Build ASP.NET Core App for win-x64...
Executing dotnet publish in this directory: C:\_src\myapp\obj\desktop\win
Build ASP.NET Core App for win-x64 under Release-Configuration...
dotnet publish -r win-x64 -c "Release" --output "C:\_src\myapp\obj\desktop\win\bin" /p:PublishReadyToRun=true /p:PublishSingleFile=true --self-contained

I just noticed what is going on.  By downloading the source code and building myself, it seems that it works.  The latest release that is installed when running dotnet tool install electronnet.cli  installs 13.5.1  which is before /dotnet-publsh was added.
when will a new build be released?

Hi @drowhunter, here is the issue that talks about the new version -> #710

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!


Kinetic release?

Greetings,
Is Kinetic release of this package planned ?
Rgds

Hi,
I recently moved to Ubuntu 16.04 + ROS Kinetic.
The "catkinized" package compiles without problems now.
Regards


Update i18n-zh work base branch in PULL_REQUEST_TEMPLATE.md

Relates to #14143
/cc @kubernetes/sig-docs-zh-owners
/cc @SataQiu @xichengliudui @tengqm @markthink @zhangxiaoyu-zidif

/cc @SataQiu @xichengliudui @tengqm @markthink @zhangxiaoyu-zidif

/lgtm
/approve

/assign @zacharysarah
We are now working on release-1.14 and need your approval :laughing:

/lgtm
/approve


Request: Add --write-comments option

Comments, if available, are currently written to the .info.json-file, but, especially for cases with large amounts of comments, it would be useful to store them in a separate file.

I don't see much difference. What is the concrete use case for separate file for comments?

E.g. when doing analysis of comments, it would be simpler to have them stored in a separate file instead of accessing them from the .info.json. I agree that it wouldn't be much of a difference; it is still easy to get the comments from the .info.json file. But with respects to sorting, it would be more practical, much like --write-description.

For analysis of comments using .info.json is just fine. --write-description has concrete use case of using in 3rdparty software that support .description files.

@dstftw OK. Thanks for answering


Sequence Flow not connected with the element edge

Given

I want to reconnect a sequence flow to another element

When

I drop the sequence flow tip on the outline of the element

Then

The sequence flow is visually not attached to the element. There is a gap.

Expected

The sequence flow tip is connected to the element edge. Without gap.

Needs fix in https://github.com/bpmn-io/bpmn-js/issues/529.

Closing this in favor of https://github.com/camunda/camunda-modeler/issues/1624.


Better doc on action(s)

I think the doc re Action could be improved:
https://pydoit.org/tasks.html#actions
The actions of a task are always run sequentially. There are 2 basic kinds of actions: cmd-action and python-action. The action “result” is used to determine if task execution was successful or not ..
In the first line there is "actions". Second line only mentions "action".
This leaves to the reader to figure out and test:
"If I have more than one action, will only the last action determine success (and result output into db)
like a bashscript or is this different? "
Also it is unclear if "enviroment and $PWD" is passed on between CmdAction(s)

This leaves to the reader to figure out and test:

Yes. can be improved..
If an action fails, subsequent actions are not event executed. similar to shell -e

Also it is unclear if "enviroment and $PWD" is passed on between CmdAction(s)

This is documented, although you need to be familiar withPopen.
If action is a string, the command will be executed through the shell. (Popen argument shell=True).


friendlier hostname for cloud instances

Using the IP/hostname for cloud contacts is pretty ugly.
Two suggestions:

change the contact name to something like "cloud instance at IP".
a separate section of the contacts list for cloud friends.

I suspect the latter is a little tricky technically, given that we don't currently translate contact names. Off the bat, I prefer the second option.

Can we give them nicknames?

Related: In the contact list, cloud instances get a little blue cloud icon superimposed on their icons, so we have some notion of "instance type" for each contact. Showing this only in the icon is too subtle for many users. Might be worth an additional text label displaying the instance type. Then we could also use this for e.g. "mobile" and "desktop", to complement uProxy for Android.
@trevj also had the good suggestion of indicating a contact's region, perhaps with a flag.