Node Error

Hi,
i am running ioBroker on my Synology NAS with Node.js: v18.19.0.
I couldn't find my H5179 devices like mentioned here. So i tried the latest GitHub Release but those errors came up:

2024-01-18 13:38:38.917 - error: host.buanet-iobroker-1 Caught by controller[0]: node:internal/modules/cjs/loader:1137
--
2024-01-18 13:38:38.917 - error: host.buanet-iobroker-1 Caught by controller[0]: throw err;
2024-01-18 13:38:38.917 - error: host.buanet-iobroker-1 Caught by controller[0]: ^
2024-01-18 13:38:38.917 - error: host.buanet-iobroker-1 Caught by controller[0]: Error: Cannot find module './tools/hexTool'
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: Require stack:
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: - /opt/iobroker/node_modules/iobroker.govee-local/build/main.js
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module._resolveFilename (node:internal/modules/cjs/loader:1134:15)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module._load (node:internal/modules/cjs/loader:975:27)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module.require (node:internal/modules/cjs/loader:1225:19)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at require (node:internal/modules/helpers:177:18)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Object. (/opt/iobroker/node_modules/iobroker.govee-local/build/main.js:22:22)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module._compile (node:internal/modules/cjs/loader:1356:14)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module.load (node:internal/modules/cjs/loader:1197:32)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Module._load (node:internal/modules/cjs/loader:1013:12)
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12) {
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: code: 'MODULE_NOT_FOUND',
2024-01-18 13:38:38.918 - error: host.buanet-iobroker-1 Caught by controller[0]: requireStack: [ '/opt/iobroker/node_modules/iobroker.govee-local/build/main.js' ]
2024-01-18 13:38:38.919 - error: host.buanet-iobroker-1 Caught by controller[0]: }
2024-01-18 13:38:38.919 - error: host.buanet-iobroker-1 Caught by controller[0]: Node.js v18.19.0
2024-01-18 13:38:38.919 - error: host.buanet-iobroker-1 instance system.adapter.govee-local.0 terminated with code 1 (JS_CONTROLLER_STOPPED)

Which Node Version is required?

The link that you provided refers to the other (cloud dependent!) Govee adapter.
As of the documentation, this device does not support the LAN API.


AttributeError: type object ‘AnnotationFile’ has no attribute ‘annotation_shape’

Hi teams,
I tried folking 2024_czii_mlchallenge_notebooks
/overview.ipynb.
I encountered an AttributeError perhaps when using the copick library. The error message indicates that the AnnotationFile class does not have an attribute named annotation_shape.
Error Traceback:
---> 11 pick_set = example_run.get_picks(object_name=object_name)[0]
File /opt/conda/lib/python3.10/site-packages/copick/impl/cryoet_data_portal.py:619, in CopickRunCDP._query_static_picks(self)
614 client = cdp.Client()
615 go_map = self.root.go_map
616 point_annos = cdp.AnnotationFile.find(
617     client,
618     [
--> 619         cdp.AnnotationFile.annotation_shape.annotation.run_id == self.portal_run_id,
620         cdp.AnnotationFile.annotation_shape.shape_type._in(["Point", "OrientedPoint"]),  # noqa
621         cdp.AnnotationFile.annotation_shape.annotation.object_id._in(go_map.keys()),  # noqa
622     ],
623 )
625 return [
626     CopickPicksCDP(
627         run=self,
(...)
631     for af in point_annos
632 ]
AttributeError: type object 'AnnotationFile' has no attribute 'annotation_shape'

Hi @yxyyxy93, thanks for bringing this to our attention!
Could you tell me the output of
import cryoet_data_portal
print(cryoet_data_portal.__version__)

in the environment you're running this in? Could you also check the version of copick?
We recently released cryoet-data-portal 4.0.0. The error you're describing sounds like you may have installed an older version of the API client.

Hi @yxyyxy93, if you set up your environment using the overview.ipynb notebook or by following the instructions in tomotwin_notebook_inference.ipynb, this issue should be fixed by installing cryoet-data-portal>=4.0.0 and copick>=0.6.0. You can also compare the fix in #8.

@yxyyxy93 Closing this as a fix has been merged. If the fix does not solve your issue, please feel free to re-open.


[atip] Update atip to adapt if there is no data.conf

If there is no data.conf file, the test tool will work OK.
Save the result pictures together with the baseline

I have test the new atip on tct-webstorage-w3c-tests/tct-extra-html5-tests/css3-color-app/css3-backgrounds-app ,  and the result is OK.


Typo in ERC20 Gas Payments

https://docs.biconomy.io/docs/supportedchains/supportedTokens
In the above link, tokens supported on Ethereum Mainnet are not mentioned.
In the tables and in the Pair column, it is mentioned "AAVE/USD", USD doesnt imply anything, it should have been USDT, and USD is mentioned everywhere.

Also in the below page
https://docs.biconomy.io/docs/tutorials/nextjs/sdkintegration


Building with JsPsych

Can we build an experiment using JsPsych that functions using Psiturk and Heroku with a similar structure?

Hello @Sahaj09, have you had a chance to run the project and take a look at it? At present the example code we have is for jsPsych. You can modify the experiment by changing the experiment.js file (link here). You can see the jsPsych plugins that are currently loaded on the exp.html template (link here). The documentation is a bit sparse, so do let us know if you have any issues getting things running.


The filter mapping configuration can not be  saved in V1.2.0 +

1、set the param DASHBOARD_CROSS_FILTERS = True
2、click "Edit Dashborad"
3、set filter mapping
4、config filter scopes
5、click “save” to save the filter mapping info.
6、click “save” to save the dashboard info.
7、refesh the current page or relogin the system
8、The filter scopes configured a momen ago are invalid...
I have fixed the BUG...

You'd better turn on DASHBOARD_NATIVE_FILTERS as well
And in addition I would upgrade to the latest version bc filters are still not stable in 1.2


how to run it?

is there a user guide to connect to each other?

I updated the README with further information. Thanks for your interest in this project!

I updated the README with further information. Thanks for your interest in this project!

thank you for update so quickly. it is good example to study libp2p.


singularity pull a github container registry image

Hi,
I was not sure if I should put this under issue or feature request.
I am trying to use a docker image that is not on dockerhub, but on github's container registry (openms-executables). I know there is the option to do the following for dockerhub images:
rule library:
    input:
          "input.tsv"
    output:
          "output.pqp"
    singularity:
          "docker://myimage/ondockerhub:latest"
     shell:
          "command -in {input} -out {output}

How can I get snakemake singularity to pull a GitHub container registry? Is this possible?
For docker, I can usually do the following:
docker pull ghcr.io/openms/openms-executables:latest
Best,
Justin

Nevermind, realized I could just do this:
singularity:
         "docker://ghcr.io/openms/openms-executables:latest"


Failed to Sync project

Failed to notify project evaluation listener.
org.gradle.api.tasks.compile.CompileOptions.setBootClasspath(Ljava/lang/String;)V

@ahmadnav  could you give us more information about what you meet?


Make unit tests tests fewer implementation details

Some minor refactorings, mostly of our test helpers. I have a bunch more local changes that aren't ready to be committed just yet, but these are the ones that should be able to land already. This changes the testing setup a bit, so it's more obvious where we're testing implementation details. Most of the machine's context is now hidden away already, the goal is to make it entirely opaque to unit tests and only use publicly observable properties and behavior in those tests.

Closing this in favor of an upcoming PR that contains broader refactorings of the internals.


swift-inspect: ensure that we eject any injected code

We would previously fail to eject the injected code on a failure.  This would prevent a future introspection into the process as well as leave the file open with an incremented retain count in the kernel space which would prevent the file from being deleted.
In the future, when the application is able to treat the injected code as a resource, this resource would be temporarily extracted, but would no longer be possible to delete until a reboot (with a registration of the deletion) due to the retained code.

CC: @hjyamauchi

@swift-ci please test


Rename

Hold on this for a minute. I think I will finish mine soon. So we merge it before this. (And change any remaining Inflecto reference altogether).

Sure

@Ptico I have rebased this and solved a small conflict.


Unbreak builds with old C++ compilers

Cadence xcc can't handle some of the recent C++11 code.  (There are third party out of tree SOF modules using C++ too that aren't ready for xt-clang).  Let's slow walk the transition a bit and recompatibilize things for older compilers.

Back link to a bunch of discussion in #53405 which is relevant

The one thing I would ask if you are willing to do another rev, would be to add a few test permutations to tests/lib/cpp/cxx/testcase.yaml specifying the STD_CPP98, STD_CPP11, STD_CPP14, STD_CPP98, ... that simply include all of the mission critical headers.
cc @stephanosio

Just stumbled on #55204 with more relevant discussion

add a few test permutations

That seems like a great idea.  See final commit, unless I messed things up.

GCC 4.2.0 is at this point, 16 years old. So my question is what problems or inefficiencies are being introduced by rolling back the standard level we are using and having to support an older version? And for how much longer do we need to support such an old compiler?

Until in-tree projects aren't using it, basically.  I don't like xcc either, but it's what existing source code using current Zephyr is being built with.   If we don't want to support it we should remove the existing integration and tell SOF to fork the project or whatever.
And FWIW: in this case I think the justification was pretty thin for the features we were using.  The constexpr template wasn't providing much value, and the other stuff was just bitrot.
I do see that the new C++98 test variant pulled up more failures though, so I need to take a look at those.

Two more patches to clean up issues detected with CONFIG_STD_CPP98=y.

Ugh, one more.  Pretty sure this is it.

Yet more stuff turned up that works on the SDK (and my local gcc 12.2.1) but not the CI host compiler.  May just have to keep banging on this until they all fall out.

sizeof() only works on lvalues or type names.  You can "sizeof(int)" or "sizeof(*foo)", but you can't "sizeof(*foo+2)" or "sizeof(atan(dy, dx))"

It's often hard to get things "done" while getting them "right", especially in this area.
I'm impressed by how this PR does both - with a negative diff on top.
Thanks for the free class.


Plugin does not work as documented with Kotlin DSL

I have set up a new IntelliJ plugin project from the JetBrains/intellij-platform-plugin-template. When I try to add the GrammarKit plugin to my build.gradle.kts as described in README.md, I get the following output from Gradle:
* What went wrong:
Script compilation errors:

  Line 57:     jflexRelease.set("1.7.0-1")
                            ^ Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
                                public inline operator fun kotlin.text.StringBuilder /* = java.lang.StringBuilder */.set(index: Int, value: Char): Unit defined in kotlin.text

  Line 60:     grammarKitRelease.set("2021.1.2")
                                 ^ Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
                                     public inline operator fun kotlin.text.StringBuilder /* = java.lang.StringBuilder */.set(index: Int, value: Char): Unit defined in kotlin.text

  Line 63:     intellijRelease.set("203.7717.81")
                               ^ Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
                                   public inline operator fun kotlin.text.StringBuilder /* = java.lang.StringBuilder */.set(index: Int, value: Char): Unit defined in kotlin.text

3 errors

I have added the following to my build.gradle.kts:
plugins {
    // ... the other plugins from the plugin project template ...
    id("org.jetbrains.grammarkit") version "2021.1.3"
}

grammarKit {
    // Version of IntelliJ patched JFlex (see the link below), Default is 1.7.0-1
    jflexRelease.set("1.7.0-1")

    // Release version, tag, or short commit hash of Grammar-Kit to use (see link below). By default, the latest available is used.
    grammarKitRelease.set("2021.1.2")

    // Optionally provide an IntelliJ version to build the classpath for GenerateParser/GenerateLexer tasks
    intellijRelease.set("203.7717.81")
}

IntelliJ also highlights the calls to the set() method as errors. If I navigate to the source code of jflexRelease in IntelliJ, it shows me some Groovy code:
package org.jetbrains.grammarkit

class GrammarKitPluginExtension {
    def grammarKitRelease = '2020.3.1'
    def jflexRelease = '1.7.0-1'
    String intellijRelease = null
}

Since the current version of the plugin is implemented in Kotlin, this is definitely confusing. I have checked the contents of org.jetbrains.grammarkit:gradle-grammarkit-plugin:2021.1.3 from Maven Central. It definitely looks like it was compiled from Groovy code.
Has the plugin actually been published incorrectly?

Thanks for clearing this up! I have successfully updated my build.gradle.kts.
I'll leave to you whether you want to keep this issue open for documentation purposes.


Format issue on elasticsearch_user_config.index_patterns.max_index_count

Hi, since few days I'm facing the following error when trying to update the max_index_count property on an Elastic cluster:
Error: cannot set `elasticsearch_user_config` : elasticsearch_user_config.0.index_patterns.0.max_index_count: '' expected type 'string', got unconvertible type 'float64'

My tf code is the following:
terraform
resource "aiven_service" "elasticsearch" {
    project = data.aiven_project.myproject.project
    cloud_name = var.elasticsearch_cluster.cloud_name
    plan = var.elasticsearch_cluster.plan
    service_name = var.elasticsearch_cluster.service_name
    service_type = "elasticsearch"
    termination_protection = true

    elasticsearch_user_config {
        elasticsearch_version = var.elasticsearch_cluster.elasticsearch_version

        dynamic "index_patterns" {
          for_each = var.environments
          content {
            pattern = "access-${index_patterns.value}-????.??.??"
            max_index_count = var.elasticsearch_cluster.index_max_count_access
          }
        }

        dynamic "index_patterns" {
          for_each = var.environments
          content {
            pattern = "monitoring-${index_patterns.value}-*"
            max_index_count = var.elasticsearch_cluster.index_max_count_monitoring
          }
        }

        ip_filter = var.ip_filter
    }
}

The index patterns are applied but the "apply" command ends with the above error. And then if I run a "plan", the errors comes back. It seems that the provider succeeds in writing the property but not in reading it.
I'm using Terraform 0.12.28 and Aiven provider 1.3.0.
This was working fine some days/weeks ago with older versions of both terraform and provider. Was there any change in the Aiven API regarding the max_index_count property type?
Thanks!

Hi, I will investigate and come back to you ASAP

A quick update, this is a bug in Aiven Terraform Provide and not related with Aiven API, we are working on the solution which will be available in the next release.


cuda RuntimeError

I run this code : python main.py --config  config/st_gcn/nturgbd-cross-subject/test.yaml
but I recieved this error:
RuntimeError: cuda runtime error (35) : CUDA driver version is insufficient for CUDA runtime version at torch/csrc/cuda/Module.cpp:107
How should do  if I want to verify the pretrained model only in CPU?

CUDA driver version is insufficient for CUDA runtime version

I think the message is clear here. You could try to install newer CUDA driver and CUDA toolkit.


sourcecode in blog article?!?

It seems that it is not possible to insert sourcecode (highlighted with pygments) in a blog article, isn't it?

...should be possible via placeholders and "CMSPygmentsPlugin", isn't it?

Yes, "CMSPygmentsPlugin" do the job... But has some problems: https://github.com/nephila/djangocms-blog/issues/87


Safari: File system view context menu does not work on iOS devices

This affects all browsers (since they're all just WebKit under the hood) on iOS devices. See https://github.com/source-academy/frontend/pull/2242#pullrequestreview-1145046059 for more information.

I managed to get hold of an old iPhone for testing. It seems that Safari on iOS doesn't even fire the context menu event. Unfortunately, it seems that Safari on iOS straight up does not support context menus.

Not working on this issue because the Source Academy does not officially support Safari (see https://github.com/source-academy/frontend/pull/2242#issuecomment-1284022558). Leaving this issue open for reference should support be added in the future.


termite not recognized

I am using termite as my terminal. My TERM variable is set to termite.
When I try to start, for example, wtf, it just errors out:
ERROR exec: "infocmp": executable file not found in $PATH

(original issue here: wtfutil/wtf#562)
However, setting the TERM variable to xterm-termite works.
Since ncurses 6.1, there is actually an entry for termite, called "termite". So, this is actually the correct value one should use for the TERM variable. See https://invisible-island.net/ncurses/NEWS.html#index-t20180127:

add "termite" entry (report by Markus Pfeiffer) -TD

Please fix tcell to make "termite" an alias for "xterm-termite" (or the other way round).

Sorry for the noise, I just realised that this (probably?) was already done in 1922b3786eaefecbba6fd8e11dbfd3531fad7a54#diff-e4713edd220ab6844ec6056e261b9ef8, AFAICS. But wtf is still using an old version.

My initial problem now has been resolved in regard to wtf. One just has to
import ( _ "github.com/gdamore/tcell/terminfo/extended") in order for TERM=termite to work.
The remaining point is that the old "xterm-termite" does not work anymore, but unfortunately, this is what termite itself uses as the corresponding PR thestinger/termite#591 is still open.
Would it make sense to keep xterm-termite supported for users running termite without changing TERM by themselves (like I do). Otherwise, feel free to close this ticket.

See #302 for my plans to go ahead and merge the extended definitions into tcell.

This was fixed in #302.


Enhancement Route.py

Hi Joost,
I have build a plugin with enhanced fms functionality. Implementing the plugin I had the problem that I wanted to store additional information for the fms functionality in the route dataset. In this way I wouldn't need to keep track of a separate list of waypoints next to the list already available in Bluesky. I made this work by extending the route class after runtime.
The only way I could make this work, was if I made a small change to the route.py that would be backwards compatible, but also would allow for the enhancement of the route class through the plugin.
So my question to you is if you add the following change to the master Bluesky such that the enhanced fms functionality becomes compatible with future Bluesky versions. The change I am suggesting is setup such that it should be backwards compatible (I only take a number of steps from a module and put them in a separate module that is called from the original module). With this addition I am able to make my plugin available for general use.
Route.py row 821
# NEW LOCAL DEL MODULE THAT IS CALLED BY DELWPT
def _del_wpt_data(self, idx):
	del self.wpname[idx]
	del self.wplat[idx]
	del self.wplon[idx]
	del self.wpalt[idx]
	del self.wpspd[idx]
	del self.wptype[idx]

def delwpt(self,delwpname,iac=None):
        """Delete waypoint"""

        # Delete complete route?
        if delwpname =="*":
            return self.delrte(iac)

        # Look up waypoint
        idx = -1
        i = len(self.wpname)
        while idx == -1 and i > 0:
            i -= 1
            if self.wpname[i].upper() == delwpname.upper():
                idx = i

        # Delete waypoint
        if idx == -1:
            return False, "Waypoint " + delwpname + " not found"

        self.nwp =self.nwp - 1
	# REPLACED SEPARATE DEL FUNCTIONS WITH SINGLE DEL FUNCTION THAT CALLS ALL DELLS
        #del self.wpname[idx]
        #del self.wplat[idx]
        #del self.wplon[idx]
        #del self.wpalt[idx]
        #del self.wpspd[idx]
        #del self.wptype[idx]
		self._del_wpt_data(idx)  # NEW LINE

        if self.iactwp > idx:
            self.iactwp = max(0, self.iactwp - 1)

        self.iactwp = min(self.iactwp, self.nwp - 1)

        # If no waypoints left, make sure to disable LNAV/VNAV
        if self.nwp==0 and (iac or iac==0):
            bs.traf.swlnav[iac]    =  False
            bs.traf.swvnav[iac]    =  False
            bs.traf.swvnavspd[iac] =  False

        return True

Once this is available in Bluesky I am able to do the following through the plugin to enhance the Route class:
old_route_init = Route.__init__
def new_route_init(self, *k, **kw):
    old_route_init(self, *k, **kw)
    self.wprta = []  # [s] Required Time of Arrival to WPT

Route.__init__ = new_route_init
old_route_addwpt_data = Route.addwpt_data

def new_route_addwpt_data(self, overwrt, wpidx, *k, **kw):
    old_route_addwpt_data(self, overwrt, wpidx, *k, **kw)
    if overwrt:
        self.wprta[wpidx] = -1.  # negative indicates no rta
    else:
        self.wprta.insert(wpidx, -1.)  # negative indicates no rta

Route.addwpt_data = new_route_addwpt_data
old_route_del_wpt_data = Route._del_wpt_data

def new_del_wpt_data(self, wpidx):
    old_route_del_wpt_data(self, wpidx)
    del self.wprta[wpidx]

Route._del_wpt_data = new_del_wpt_data

In this way the stack functions for adding, updating and deleting waypoints remain functional for the standard waypoint data and also works likewise on the enhanced waypoint data.

Hi Joost,
Lijkt me prima voorstel, ik verwacht inderdaad geen probleem of interferentie met bestaande code op deze manier.
Met vriendelijke groeten,
Jacco Hoekstra
On 1 May 2019, at 11:11, rjdverbeek <notifications<EMAIL_ADDRESS>wrote:
Hi Joost,
I have build a plugin with enhanced fms functionality. Implementing the plugin I had the problem that I wanted to store additional information for the fms functionality in the route dataset. In this way I wouldn't need to keep track of a separate list of waypoints next to the list already available in Bluesky. I made this work by extending the route class after runtime.
The only way I could make this work, was if I made a small change to the route.py that would be backwards compatible, but also would allow for the enhancement of the route class through the plugin.
So my question to you is if you add the following change to the master Bluesky such that the enhanced fms functionality becomes compatible with future Bluesky versions. The change I am suggesting is setup such that it should be backwards compatible (I only take a number of steps from a module and put them in a separate module that is called from the original module). With this addition I am able to make my plugin available for general use.
Route.py row 821
NEW LOCAL DEL MODULE THAT IS CALLED BY DELWPT
def _del_wpt_data(self, idx):
del self.wpname[idx]
del self.wplat[idx]
del self.wplon[idx]
del self.wpalt[idx]
del self.wpspd[idx]
del self.wptype[idx]
def delwpt(self,delwpname,iac=None):
"""Delete waypoint"""
    # Delete complete route?
    if delwpname =="*":
        return self.delrte(iac)

    # Look up waypoint
    idx = -1
    i = len(self.wpname)
    while idx == -1 and i > 0:
        i -= 1
        if self.wpname[i].upper() == delwpname.upper():
            idx = i

    # Delete waypoint
    if idx == -1:
        return False, "Waypoint " + delwpname + " not found"

    self.nwp =self.nwp - 1
    # REPLACED SEPARATE DEL FUNCTIONS WITH SINGLE DEL FUNCTION THAT CALLS ALL DELLS
    #del self.wpname[idx]
    #del self.wplat[idx]
    #del self.wplon[idx]
    #del self.wpalt[idx]
    #del self.wpspd[idx]
    #del self.wptype[idx]
            self._del_wpt_data(idx)  # NEW LINE

    if self.iactwp > idx:
        self.iactwp = max(0, self.iactwp - 1)

    self.iactwp = min(self.iactwp, self.nwp - 1)

    # If no waypoints left, make sure to disable LNAV/VNAV
    if self.nwp==0 and (iac or iac==0):
        bs.traf.swlnav[iac]    =  False
        bs.traf.swvnav[iac]    =  False
        bs.traf.swvnavspd[iac] =  False

    return True

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/TUDelft-CNS-ATM/bluesky/issues/190, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABWT2BAORHQQ2YMDBC52P7DPTFNCVANCNFSM4HJSPVLQ.

Hi René,
I like the idea of making more parts of bluesky (such as the route implementation) more extensible, but I'd rather use inheritance than this kind of monkey-patching, which, in my opinion, is not the most beautiful 'feature' of Python ;)
Basically my plan is to enable plugins to extend basically any part of bluesky. I will think on this a bit more to get to a flexible approach! I'll keep you posted.

Hi Joost,
You are right. This is not the most beautiful solution. But it helped me
not changing your original code.
Good luck with finding a way to make also the route implementation
extensible.
Wkr. René
Op vr 3 mei 2019 om 15:38 schreef Joost Ellerbroek<EMAIL_ADDRESS>
:

Hi René,
I like the idea of making more parts of bluesky (such as the route
implementation) more extensible, but I'd rather use inheritance than this
kind of monkey-patching, which, in my opinion, is not the most beautiful
'feature' of Python ;)
Basically my plan is to enable plugins to extend basically any part of
bluesky. I will think on this a bit more to get to a flexible approach!
I'll keep you posted.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TUDelft-CNS-ATM/bluesky/issues/190#issuecomment-489098451,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD35BDBRTVZLUEAXCE3QRJTPTQ567ANCNFSM4HJSPVLQ
.

Hi René,
In b3d103b and several following commits I've implemented the possibility for plugins to subclass most classes in BlueSky. If that plugin is loaded the newly defined implementation becomes visible, and can be selected via a stack command. This way, all functions can be adapted through regular inheritance. For your example this would be something like:
from bluesky.traffic import Route

class MyRoute(Route):
    def __init__(self):
        super().__init__()
        # my initialisation here

    def delwpt(self,delwpname,iac=None):
        success = super().delwpt(delwpname, iac)
        # do something with your own data
        return success

Load this plugin, and select your new implementation via the stack as follows:
IMPL route myroute


feat(.gitignore) add gitignore parameter

add an optional paramater to create a gitignore
This feature will allow an optional ignore parameter. It expects and
array, and will automatically add the dist folder, node_modules,
bower_components, and the ole .DS_Store.

    });
});

The code looks good, but can you share a little more about how you're using this module?
The way it's meant to work is that this module creates a new git repo in dist/ (by default) whose responsibility is to manage the contents of that folder. (Commits from that repo are then copied into the repo at /).
So it would seem unusual to have a dist/dist or a dist/node_modules, which seems to be what you're guarding against here. Am I missing something?


The New-AzVirtualHubVnetConnection Routine Creates Invalid Network Peers.

This guidance is flawed first because it does not work (the network peers created are alarmingly incorrect, in fact they are completely non-functional), and second because it creates references to an unknown azure subscription (182c56c8-59a8-4b38-9513-7967d114dcc3). The New-AzVirtualHubVnetConnection routine creates invalid network peers such that the "remote" peer in either spoke networks references an invalid/unknown azure subscription (in our case, ID = 182c56c8-59a8-4b38-9513-7967d114dcc3). We have no idea where this subscription comes from. It is not one of ours. It is not acceptable that this sample code creates network peers belonging to a completely alien subscription.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 5fe43b05-6297-1bcc-a7d2-6e3ffac5e828
Version Independent ID: b9d85d1e-be9f-ce4e-b3cf-89ba62942b9a
Content: Tutorial: Secure your virtual hub using Azure PowerShell
Content Source: articles/firewall-manager/secure-cloud-network-powershell.md
Service: firewall-manager
GitHub Login: @jomore
Microsoft Alias: victorh

@klawrawkz Thanks for your feedback. We shall review this and get back to you soon

@klawrawkz
I am not able to repo your issue.
The vHUb of vWAN is deployed in a Product Group Subscription. Hence, the Peering created is referencing to a resource in a different subscription.
This is an expected behavior.
Are you facing any connectivity issues from the Spoke VNet to the Hub or vWAN?

Hi @KapilAnanth-MSFT, thanks for the quick response. Still, I'm confused about your explanation.
"The vHUb of vWAN is deployed in a Product Group Subscription"
When I follow along with this tutorial using my subscription and my own settings, why would  New-AzVirtualHub create a hub in your "product group subscription?"
When I use settings germane to my environment, a hub is created in my subscription. See below for example.
$ThisRG = "MY_BUNCH_OF_SERVICES_RG"
$ThisLocation = "centralus"
$ThisVwanName = "bunch-of-svcs-wan"
$ThisHubName =  "bunch-of-svcs-hub"
$ThisHubAddressPrefix = "<IP_ADDRESS>/24"
 $ThisSpokeAddressPrefix = "<IP_ADDRESS>/21"
$ThisSubnetConfigName = "cfg-snet-config-001"
$ThisSNetName = "bos-spoke-001"
$ThisOtherSubnetConfigName = "cfg-otherbos-snet-config-001"
$ThisOtherSNetName = "otherbos-spoke-001"
$ThisSpokeName = "snet-spoke-name-001"
$ThisVMPiPName = "logic-vm-pip-001"
$ThisVMCredential = New-Object System.Management.Automation.PSCredential ($ThisVMLocalAdminUser , $ThisVMLocalAdminUser);
$ThisVMSize = "Standard_B2ms"

$ThisVwan = New-AzVirtualWan -Name $ThisVwanName -ResourceGroupName $ThisRG -Location $ThisLocation -AllowVnetToVnetTraffic -AllowBranchToBranchTraffic -VirtualWANType Standard -Verbose  -Confirm:$false

$ThisHub = New-AzVirtualHub -Name $ThisHubName -ResourceGroupName $ThisRG -VirtualWan $ThisVwan -Location $ThisLocation -AddressPrefix $ThisHubAddressPrefix -Sku Standard -Verbose -Confirm:$false

$ThisSpokeConnection = Get-AzVirtualNetwork -ResourceGroupName $ThisRG -Name $ThisSpokeName -Verbose
     Add-AzVirtualNetworkSubnetConfig -Name $ThisSNetName -VirtualNetwork $ThisSpokeConnection -AddressPrefix $ThisSpokeAddressPrefix -Verbose -Debug
     $ThisSpokeConnection | Set-AzVirtualNetwork

# Here's the issue....
Add-AzVirtualNetworkSubnetConfig -Name $ThisSubnetConfigName -VirtualNetwork $ThisSpokeConnection -AddressPrefix $ThisSpokeAddressPrefix -Verbose -Debug

$ThisVM = New-AzVM -Name "SomeVM" -ResourceGroupName $ThisRG -Location $ThisLocation `
                 -Image $ThisLogicVMImage -credential $ThisVMCredential `
                 -VirtualNetworkName $ThisSpokeName -SubnetName $ThisSNetName -PublicIpAddressName $ThisVMPiPName -Size $ThisVMSize -Verbose

# Here's where the error is pointed out and it is not clear that "some other subscription" is involved as peer in this networking configuration. What is going on here, and why is the foreign subscription involved? I'm using values pertinent to my environment.
#     DEBUG: AzureQoSEvent:  Module: Az.Compute:5.3.0; CommandName: New-AzVM; PSVersion: 7.3.3; IsSuccess: False;
#     Duration: 00:02:09.7759432;
#     Exception: Cannot update address space on vnet Vnet address space <IP_ADDRESS>/24 overlaps with address space on peering
Please perform sync operation on the peering /subscriptions/<my GUID>/resourceGroups/<MyRG>/
#     providers/Microsoft.Network/virtualNetworks/cfg-snet-config-001/virtualNetworkPeerings/RemoteVnetToHubPeering_a42be7cc-0c9f-41de-b652-ecf94f86885d
#     and then try updating address space. For more information please refer to https://aka.ms/VnetPeeringInfo;

As noted in the code snippet above, when I create the subnet config and set the virtual network, the "Product Group Subscription" you mention is involved in the operation. How is this Product Group Subscription included in the config when I'm using only settings related to my subscription, and my services? There is no reference that I know of in the POSH running these operations that would reach out to some mysterious subscription () and attempt to peer with network services therein.
Yours in confusion,
klaw

@klawrawkz
Azure vHub is not deployed in the Product Group subscription. Apologies for the confusion.

The important thing to note here is that a VNet Peering and vHub Connection are not the same and are different.
A vHub Connection is a VnetPeering under the hood and is masked to the customers.
This masked resource is why you would see a reference to a different subscription/RG on the Spoke's Peering section

I hope I was able to explain why you are seeing a reference to a different subscription.
Now, I am not exactly sure what you are trying to do with your commands,
But I believe you are trying to update a subnet of a Spoke VNet that is connected to a vHub.

From the error message, it appears that there is a overlap between the VNets connected to the vHub.
Perhaps, there is a different Vnet connected to the vHub that is overlapping with the subnet range you are trying to add.
That is why the error message is pointing to the vHub's Peering (as explained above, this would display a reference)

@klawrawkz
I am glad you have successfully created the vWAN and vHub
I understand your concern wrt vHub Connections.
However, users will face an issue only when they consider this "Connection" as Peering and try to run the commands.
Using "AzVirtualHubVnetConnection" commands should lead to success.
I shall suggest internally to avoid exposing this subscription details in the verbose messages.


PSScriptAnalyzer: "The variable  is assigned but never used" false error within ForEach-Object scope

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all open and closed issues to ensure it has not already been reported.
[X] I have read the troubleshooting guide.
[X] I am sure this issue is with the extension itself and does not reproduce in a standalone PowerShell instance.
[X] I have verified that I am using the latest version of Visual Studio Code and the PowerShell extension.
[ ] If this is a security issue, I have read the security issue reporting guidance.

Summary
When I declare a local variable and try to set it within the ForEach-Object scope.

PowerShell Version
PS C:\Users\rguilbault\Documents\Projects\MTDSCLib> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.1778
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.1778
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Visual Studio Code Version
code --version
1.79.2
695af097c7bd098fbf017ce3ac85e09bbc5dda06
x64

Extension Version
code --list-extensions --show-versions | Select-String powershell

<EMAIL_ADDRESS>
Steps to Reproduce
I get an error on the $success = $true line here:
function Test-LinterError {
$success = $false
@(1) | ForEach-Object {
$success = $true
}
return $success
}
and here:
function Test-LinterError {
$success = $false
{
$success = $true
}
return $success
}
I do not get such lint errors for any of these blocks:
function Test-LinterError {
$success = $false
if(1) {
$success = $true
}
return $success
}
function Test-LinterError {
$success = $false
foreach($item in @(1)) {
$success = $true
}
return $success
}
in the first situation (the one of interest), $success is properly side effected from within the ForEach-Object block such that returning it is meaningful use, i.e. the function returns $true.
contrarily, the second situation (the raw block/scope), I get that $success there is local to that scope and it test out as expected (e.g. the function returns $false).
I have scoured documentation looking for any explanation to justify the linter error, but I think it is simply a bug and perhaps not a commonly enough used coding pattern for anybody to have noticed/cared about it. I am willing to concede the point if someone can show me the documentation that explains why my expectation is flawed, though.
Visuals
No response
Logs
No response

This is a known issue (see e.g. #1163) as this rule is limited to per scriptblock analysis, which is a hard problem to solve and I don't expect that effort to be made.


Doc improvements: palette usage, manual install instructions, JSX

I had to install the package manually, and found a couple of issues in the docs. Not sure if this is the correct place to report that, but this was my best guess :sweat_smile:

plugin page and docs only mention tsx files and TypeScript stuff. It looks like the plugin does not support plain JSX projects.
I couldn't get the preview to run (https://github.com/react-buddy/create-react-app/issues/13). Is that a blocker to use the palette? I'm not sure how it works, if it needs configuration (since there's a palette file injected in my project) or other details. I guess that section needs improvements?
it seems the IDE installer doesn't include the react-script replacement, nor the index add-on, which I only found when reading about the manual install.

furthermore, the manual sample code doesn't explain what's useInitialHook or what should I feed it, if any.
I also guessed the ComponentPreviews was the one injected in my project, not the one from the package (Storm suggested both)
lastly, Storm complained that <ComponentPreviews/> is incorrect, since it's expecting a function, not a component.

Hi! Thanks for the feedback, we are currently working on complete doc rewrite!

I couldn't get the preview to run (https://github.com/react-buddy/create-react-app/issues/13). Is that a blocker to use the palette? I'm not sure how it works, if it needs configuration (since there's a palette file injected in my project) or other details. I guess that section needs improvements?
We'll deprecate this package since RB now works with pure create-react-app (as long as with other popular starters)

Cool! I'll wait for those changes to try the extension again, congratz for the persistence and good work!


when snapping to bottom (and right?) allow a couple of pixels to still be grabbed

it annoys me that once a controller is snapped to the bottom it's fiddly to get it unsnapped

done on dev


refactor: remove cli

[ ] remove cli rocket-scripts
[ ] update test and fixtures
[ ] add API comments that can looks up good in the VSCode

Pull Request Test Coverage Report for Build 201040860

0 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 82.07%

Totals

Change from base Build 200233669:
-0.8%

Covered Lines:
779

Relevant Lines:
867

💛  - Coveralls

Pull Request Test Coverage Report for Build 201040860

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 82.07%

Totals

Change from base Build 200233669:
-0.8%

Covered Lines:
779

Relevant Lines:
867

💛  - Coveralls

[ ] remove duplicated params with webpackConfig and devServerConfig (https, proxy...)
[ ] block user input to webpackConfig and devServerConfig with Omit

[ ] change composition design devserverstart({...start({ ... })}) ???


Alias MultiViews on Ubuntu 14.04

Default recipe causes a config error on Ubuntu 14.04:
AH00526: Syntax error on line 18 of /etc/apache2/mods-enabled/alias.conf:
Either all Options must start with + or -, or no Option may.

Changing "MultiViews" to "+MultiViews" (or "-MultiViews", for that matter) in /etc/apache2/mods-enabled/alias.conf resolves the issue.

thanks @rgormley As far as I understood the issue, the problem is located here: https://github.com/svanzoest/apache2-cookbook/blob/cd03a52e1e937889ce510e2aff492642a0b8ba3f/templates/default/mods/alias.conf.erb
Am I right?

This is the template/line that's causing the problem on apache2.4 installs:
https://github.com/hardening-io/chef-apache-hardening/blob/master/templates/default/mods/alias.conf.erb#L18

Thanks @cmjosh we definitely need a plus there. Would you like to create a PR? Maybe we should also use attributes for this config.

:+1:


Add Period type

@alexander-yevsyukov commented on Thu Jul 27 2017
... similar to Period type from Java 8.

Let's postpone adding this type until we see a real demand when describing domains related to time.
For calculations we can use Java Time or Ranges from Guava.


fix sync update in distributed mnist demo

Can one of the admins verify this patch?

I signed it!

@caisq could you take a look?

Looking

Can one of the admins verify this patch?

Adding @jmchen-g

https://github.com/tensorflow/tensorflow/issues/3457 provided more information

@caisq / @jmchen-g should we go ahead to merge this?

This fix is not right. The chief queue runner can only run by the chief. This seems to expose a bug somewhere but we have not figured it yet. We are in the middle of changing the sync_rep_optimizer so will come back to this after that change.

Any updates on the sync mode?

@jmchen-g any updates on this?

Most of the updates are implemented internally and after they are verified, we will push it outside. That will take some time though. We can close this PR for now as the fix is not a real fix but happens to get around some issues in the current implementation. The new implementation should be more robust and easier to debug.

hi @jmchen-g any updates on this?

@caisq, any update on this please?
On Thu, Jul 6, 2017 at 12:22 PM, Renato Marroquin<EMAIL_ADDRESS>wrote:

hi @jmchen-g https://github.com/jmchen-g any updates on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/3458#issuecomment-313493621,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APD49u1OCLjX_k7VO3cUzt-YTckqhiXLks5sLTP8gaJpZM4JSh4c
.


Segmentation/masking temporarily removed

Have you made anyone progress on re-adding segmentation/masking? Preferably with multi style image support?

For anyone needing to use segmentation masks, you can use:
git checkout 262a825 to change the version of your directory back to the old version with the working masks.

I decided to do it in two parts - adopt Johnson's new code without sorting out the masking part, and then after sort that out. It didn't look too hard, so am hopeful that i can do it next time i have a few hours - unless someone beats me to it!

@martinbenson Going beyond the scope Deep Photo Style Transfer research paper, another somewhat recently released research paper addresses some issue with segmentation in regards to style transfer. Because these two papers seem to have been published within a short period of time of each other, the work done in either could not have been used in the other's code.
I don't know how much of an issue the receptive field problem in regards to Spatial Control (style transfer with segmentation) is with deep-photo-styletransfer, but Leon Gatys' NeuralImageSynthesis solves the issue via adding in place dilation to the style loss function. Specifically, the relevant code is in the LossLayers.lua file. In the research paper, I believe that the in-place dilation is called "eroded guidance channels", where "guidance channels" refers to inputs and settings that effect your output.

Wow, thats why I cannot use  "-content_seg" option

I have now re-implemented segmentation/masks!


Patch for "7839bbf2: Fix language-change not applied to all components immediately"

Patch for 7839bbf2: Fix language-change not applied to all components immediately

Missed the "Account Password" card that is used for authenticated case.
For cards that are only used in Settings, listen to the language change in "select" instead of passing it in as a prop. I think this is cleaner.

Agreed this is cleaner.


Why do you suggest to put presentational stuff in the App container?

I'd like to know what's behind this decision.
Shouldn't we have an App component which get rendered by the App container?

Yes, but it was easier at the time to simply keep it in the App container for our small example!

I see, thanks as usual 🙂

No worries, keep em coming!


Fix weight url in resnest

Replace the url in resnest.py cause the original one has expired,

This should be merged, I was having the same url issue


Fixed #35056 -- Fixed system check crash on reverse m2m relations with related_name in ModelAdmin.filter_horizontal/vertical.

ticket-35056
Thanks Thomas Feldmann for the report.
Regression in 107865780aa44914e21d27fdf4ca269bc61c7f01.

@hramezani Thanks for checking :+1:


Massive console spam: "Attempt to attach world capability before mod is inititalized - silly fake worlds" repeated 1500+ times

Mod Version:
DynamicSurroundings-1.12.2-<IP_ADDRESS>
Forge Version:
<IP_ADDRESS>47
Link to client log:
latest-3.log
Description:
Description of the problem.  In the case of crashes, it is good to know things like the following:
The lines
[Client thread/INFO] [dsurround]: Attempt to attach world capability before mod is inititalized - silly fake worlds.
[Client thread/INFO] [dsurround]: Creating default SeasonInfo for dimension overworld

are spammed a lot in the console and log

Yep.  Exactly what the message says.    Newer version of DS hides it behind a flag setting, so update the the latest.

Ah, Twitch never showed there was an update. Sorry about that.

No worries.  The latest is a BETA and Twitch may be configured to only show released.


[feature request] v4 missing one click "empty trash" button

:wave:
You have an option, inside the dropdown Advanced Options on the right side of the toolbar.
cf ->

It's well hidden. Thanks for pointing me to this :+1:


Angular CLI build fails because typeClasses and iconClasses have no index signature

Bug report
When I create a new project using angular-cli (v1.7.2) and Angular2-Toaster (v5.0.0), ng build returns the following error:
ERROR in
node_modules\angular2-toaster\src\toaster-container.component.d.ts.ToasterContainerComponent.html(8,17):
  Element implicitly has an 'any' type because type 'Object' has no index signature.
node_modules\angular2-toaster\src\toaster-container.component.d.ts.ToasterContainerComponent.html(5,17):
  Element implicitly has an 'any' type because type 'Object' has no index signature.

This is because toaster-config.ts has the following definitions:

typeClasses?: Object;
iconClasses?: Object;

and toaster-container.component.ts has the following HTML template:

[ngClass]="toasterconfig.typeClasses[toast.type]"
[iconClass]="toasterconfig.iconClasses[toast.type]"

I'll provide a PR to fix this.

@andrewpmontgomery I've actually failed to reproduce this now in several attempts.  Can you please describe the base configuration required to reproduce this?  i want to make sure the project is properly configured to highlight these issues going forward.

You can reproduce the problem in your latest branch simply by changing:
"suppressImplicitAnyIndexErrors": false,
to
"suppressImplicitAnyIndexErrors": true,
in the tsconfig-aot.json file.
I can't reproduce this in a sample consuming application, so perhaps something has changed in angular-cli or another package.


Add a new API for moving a sprite with a controller

We want to transition away from controller.dx() and controller.dy() to a simpler API

Current proposal is for this to be an API in the controller category:
controller.control(sprite, Axis.x);

The wording of the block would be:
 _    __________________________________________________
| \__/                                                  |
|       control `sprite` in direction `x (horizontal)`  |
|_    __________________________________________________|
  \__/

And the block would have a dropdown of axes with these descriptive names:

x (horizontal)
y (vertical)
x+y (both axes)

@mmoskal @pelikhan @eric-camplin how's this?

Don't you need speed? I would have add two arguments defaulting to 100 for both axes

From: Richard Knoll<EMAIL_ADDRESS>Sent: Tuesday, June 5, 2018 3:04:59 PM
To: Microsoft/pxt-32
Cc: Michal Moskal; Mention
Subject: Re: [Microsoft/pxt-32] Add a new API for moving a sprite with a controller (#128)
Current proposal is for this to be an API in the controller category:
controller.control(sprite, Axis.x);
The wording of the block would be:

| _/                                                  |
|       control sprite in direction x (horizontal)  |
|    __________________________________________________|
__/
And the block would have a dropdown of axes with these descriptive names:

x (horizontal)
y (vertical)
x+y (both axes)

@mmoskalhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmmoskal&data=02|01|Michal.Moskal%40microsoft.com|c47add60078e4b55fc8c08d5cb3061bf|72f988bf86f141af91ab2d7cd011db47|1|0|636638331020443622&sdata=mdvf%2BiY%2B6FjY5asDA3MVhQ0t5jEJuFr7E7G7kf%2FA0OY%3D&reserved=0 @pelikhanhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpelikhan&data=02|01|Michal.Moskal%40microsoft.com|c47add60078e4b55fc8c08d5cb3061bf|72f988bf86f141af91ab2d7cd011db47|1|0|636638331020453632&sdata=g89v66X20ihUorVUN%2BY77ZRurEx5vAVIPbdqwPHwVEM%3D&reserved=0 @eric-camplinhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feric-camplin&data=02|01|Michal.Moskal%40microsoft.com|c47add60078e4b55fc8c08d5cb3061bf|72f988bf86f141af91ab2d7cd011db47|1|0|636638331020453632&sdata=DRbVz6Hy4h232b%2FCsEfe3wae8flbTjnxBbkhLYFrBCU%3D&reserved=0 how's this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-32%2Fissues%2F128%23issuecomment-394875493&data=02|01|Michal.Moskal%40microsoft.com|c47add60078e4b55fc8c08d5cb3061bf|72f988bf86f141af91ab2d7cd011db47|1|0|636638331020463637&sdata=8MDagT6iPQ%2BCRwn5It%2BUJX4fZONGKkZ70NWrSv7RQ9U%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKLfOOqo1kImAmRS_d5UFS53-K0SCtFFks5t5wCLgaJpZM4ULX56&data=02|01|Michal.Moskal%40microsoft.com|c47add60078e4b55fc8c08d5cb3061bf|72f988bf86f141af91ab2d7cd011db47|1|0|636638331020463637&sdata=enqu8j8iIEgLvh2w4BmFTS3oBE7C2%2BHhUmoX9kjcOIs%3D&reserved=0.

we already have speed on sprites so I guess it should be familiar
control [sprite] with speed vx [100] vy [100]

instead of axis you could have "direction x", "direction y", "speed x", "speed y", ....

@mmoskal are you proposing a separate block or just using two number arguments instead of a dropdown? Personally, I'm not sure we need the speed for this block. We could choose one "good enough" value and leave dx and dy around for those that want finer-grained controls.
@Jaqster thoughts?

I'm thinking one block, with two values. Maybe both can be optional.
I think 50 and 200 will make for a very different game, while being conceptually cheap, so I would be inclined to have the speed.

I like a controller option for speed, but be sure to label as "velocity" for STEM purposes seeing that it pertains to an axis direction

It should align with the name of sprite property whatever it is

From: Eric Camplin<EMAIL_ADDRESS>Sent: Wednesday, June 6, 2018 8:30:34 AM
To: Microsoft/pxt-32
Cc: Michal Moskal; Mention
Subject: Re: [Microsoft/pxt-32] Add a new API for moving a sprite with a controller (#128)
I like a controller option for speed, but be sure to label as "velocity" for STEM purposes seeing that it pertains to an axis direction
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-32%2Fissues%2F128%23issuecomment-395111441&data=02|01|Michal.Moskal%40microsoft.com|b880a5de513e4453e5bf08d5cbc2727b|72f988bf86f141af91ab2d7cd011db47|1|0|636638958362076210&sdata=AOa4f9C%2FqBPB43QJseGzNA6ksDWDisGux40NgmyIsbI%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKLfOCmGqGlZIVJxZsDX_14OzEzY8dnCks5t5_WagaJpZM4ULX56&data=02|01|Michal.Moskal%40microsoft.com|b880a5de513e4453e5bf08d5cbc2727b|72f988bf86f141af91ab2d7cd011db47|1|0|636638958362076210&sdata=ExXtAov8d6WipTciTSqF3VCWtQzgmef4c9xX1jcKdwc%3D&reserved=0.


Learning Tuesday: Create dev container

What does this PR change?

setup dev container
create dev container during PR

Example: https://github.com/mbussolotto/uyuni-tools/pull/4: clicking on Create Codespace it's possible to create a golang container environment where compile, run unit test and change the code. Container timeout and retention it's a user setting https://github.com/settings/codespaces.
All personal GitHub.com accounts include a monthly quota of free usage each month. GitHub will provide users in the Free plan 120 core hours, or 60 hours of run time on a 2 core codespace, plus 15 GB of storage each month.

Test coverage

No tests: add explanation

No tests: already covered

Unit tests were added

[ ] DONE

Links
Issue(s): #

[ ] DONE

Changelogs
Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository
If you don't need a changelog check, please mark this checkbox:

[ ] No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)
Before you merge
Check How to branch and merge properly!

@cbosdo , @rjmateus @juliogonzalez what do you think? I had this test with uyuni-tools, but my idea is to try to do something similar for uyuni: the goal would be have a quick online environment that can be used e.g. in cause of unit-test and/or linting test failures

Is it possible to preload the go dependencies? they take a while to download. But that seems cool for newcomers.

closing the browser tab is not enough to stop the created codespace...

IIUC when you close the tab, the container will be idle and you can set Default idle timeout here https://github.com/settings/codespaces . But I need to do more test on it
`

it looks really cool :)

Is it possible to preload the go dependencies? they take a while to download. But that seems cool for newcomers. Also I would add a link to the codespaces pricing and getting started doc: closing the browser tab is not enough to stop the created codespace...

I am testing this... how can we stop the codespace? :-D

Is it possible to preload the go dependencies? they take a while to download. But that seems cool for newcomers. Also I would add a link to the codespaces pricing and getting started doc: closing the browser tab is not enough to stop the created codespace...

I am testing this... how can we stop the codespace? :-D

Not sure if I did it somehow, or I was just idle for enough time, but...


Add Digest and Bearer Auth modes to http request node

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
As discussed on the forum, this PR introduces Digest and Bearer authentication for the HttpRequest node.
Following flow can be used to test the 3 authentication methods:

[{"id":"4b680d8d.3dce74","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":320,"wires":[["f97dcb19.a8faf8"]]},{"id":"4f3f587b.9fb988","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":320,"wires":[]},{"id":"a6c3b589.da0e48","type":"comment","z":"dc82a538.a81ba8","name":"Digest authentication without credentials => 401 (Unauthorised)","info":"","x":550,"y":280,"wires":[]},{"id":"8c9ec024.38585","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":420,"wires":[["881fe2e2.7ece7"]]},{"id":"3e0812f7.a99f2e","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":420,"wires":[]},{"id":"7d22c59f.eb794c","type":"comment","z":"dc82a538.a81ba8","name":"Digest authentication with credentials => 200 (Success)","info":"","x":520,"y":380,"wires":[]},{"id":"5406fbb0.bb9ad4","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":80,"wires":[["72aa922d.2a18ec"]]},{"id":"4c1678ed.1b7fd8","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":80,"wires":[]},{"id":"3169ca53.c12056","type":"comment","z":"dc82a538.a81ba8","name":"Basic authentication without credentials => 401 (Unauthorised)","info":"","x":540,"y":40,"wires":[]},{"id":"f8475555.ddd588","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":180,"wires":[["4e9922b0.a168dc"]]},{"id":"5b2bd581.f7be4c","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":180,"wires":[]},{"id":"74a30db0.2b16a4","type":"comment","z":"dc82a538.a81ba8","name":"Basic authentication with credentials => 200 (Success)","info":"","x":520,"y":140,"wires":[]},{"id":"e7e6355a.9fcef8","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":560,"wires":[["2c956c46.61fa44"]]},{"id":"3697ab96.bc85f4","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":560,"wires":[]},{"id":"3670c9ce.cc0e96","type":"comment","z":"dc82a538.a81ba8","name":"Bearer authentication without credentials => 401 (Unauthorised)","info":"","x":550,"y":520,"wires":[]},{"id":"992b6a57.21c7a8","type":"inject","z":"dc82a538.a81ba8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":400,"y":660,"wires":[["5332b98c.86f338"]]},{"id":"e413a9f2.988888","type":"debug","z":"dc82a538.a81ba8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","x":730,"y":660,"wires":[]},{"id":"9e267fc6.9edf","type":"comment","z":"dc82a538.a81ba8","name":"Bearer authentication with credentials => 200 (Success)","info":"","x":520,"y":620,"wires":[]},{"id":"72aa922d.2a18ec","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/basic-auth/myuser/mypasswd","tls":"","proxy":"","authType":"basic","x":570,"y":80,"wires":[["4c1678ed.1b7fd8"]]},{"id":"4e9922b0.a168dc","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/basic-auth/myuser/mypasswd","tls":"","proxy":"","authType":"basic","x":570,"y":180,"wires":[["5b2bd581.f7be4c"]]},{"id":"f97dcb19.a8faf8","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/digest-auth/auth/myuser/mypasswd","tls":"","proxy":"","authType":"digest","x":570,"y":320,"wires":[["4f3f587b.9fb988"]]},{"id":"881fe2e2.7ece7","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/digest-auth/auth/myuser/mypasswd","tls":"","proxy":"","authType":"digest","x":570,"y":420,"wires":[["3e0812f7.a99f2e"]]},{"id":"2c956c46.61fa44","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/bearer","tls":"","proxy":"","authType":"bearer","x":570,"y":560,"wires":[["3697ab96.bc85f4"]]},{"id":"5332b98c.86f338","type":"http request","z":"dc82a538.a81ba8","name":"","method":"GET","ret":"txt","paytoqs":false,"url":"http://httpbin.org/bearer","tls":"","proxy":"","authType":"bearer","x":570,"y":660,"wires":[["e413a9f2.988888"]]}]

All 6 tests in this flow make use of httpbin.org, which offers endpoints for all 3 authentication methods.

Test basic authentication without credentials, which results in 401 (Unauthorised).
Test basic authentication with credentials, which results in 200.  Don't forget to specify username 'myuser' and password 'mypasswd'.
Test digest authentication without credentials, which results in 401 (Unauthorised).
Test digest authentication with credentials, which results in 200.  Don't forget to specify username 'myuser' and password 'mypasswd'.
Test bearer authentication without credentials, which results in 401 (Unauthorised).
Test bearer authentication with credentials, which results in 200.  Don't forget to specify some (random) token.

I have tried to have no impact on existing http-request nodes, i.e. nodes that have been created with a previous version.  I have tested this by adding a few http-request nodes to my flow (with the previous version), and afterwards I have installed this new version:

When the config screen is opened, then basic authentication will be displayed automatically (when credentials are available and authType is undefined):

When the config screen hasn't been opened, and an input message arrives (but no credentials are available):

No credentials will be set (i.e. we won't arrive at line 183), since there is no username.  This is the same logic as in the previous version.
When the config screen hasn't been opened, and an input message arrives (and credentials are available):

The credentials will be set (i.e. we will arrive at line 183): indeed I treat empty authType (in case credentials are available) as 'basic' authentication.

Thanks for reviewing this PR !!!!
Bart
Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.04%) to 76.392% when pulling cd85ed97481c165431ea9dbc8b1c54ce3af0825a on bartbutenaers:dev into 9d673a213ed1cdc2d2ce4388fae7c6cffb46aacc on node-red:dev.

@knolleary - Looks all good to me. Can you give the once over and merge ?


Bump Version For NPM Release With Leak Timer Fix

Summary:
I'm still seeing the issue #2 occasionally.
Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the Pulse component.

I noticed that the timer leak #3 is not released via npm. This bumps
the version in order to make a release easy. :)
Thanks for creating this component!

Ahh good catch, thanks for using and contributing! I will update the npm module tonight.


Generate different graphql file for different proto

We have a scenario where we have a PB folder.
Inside PB folder we have 2 proto file. ie: A.proto, B.proto . Now suppose we want to generate graphql file for a and b proto inside PB folder.
Both the proto has the same option package as well.
Currently, only one file Is generated.
I need to generate graphql file for A.proto as well as for b.proto inside PB folder.
How can I achieve this?
Thank you.

Hi @atul161 , thank you for creating the issue.
it may be supported as well but I'll look into it.


set dates(start and end date) in input box and don't show the range the calendar (like new calendar) so how to do this , any help?

invalid


Could not connect to the endpoint URL: "https://coiled-runtime-ci.s3.None.amazonaws.com/"

Tests have started failing a few hours ago, with the message
botocore.exceptions.EndpointConnectionError: Could not connect to the endpoint URL: "https://coiled-runtime-ci.s3.None.amazonaws.com/"

e.g. https://github.com/coiled/benchmarks/actions/runs/6253354999/job/16978575620?pr=999
@ntabris could you look into this?

CC @phofl I believe you experienced the same in your own tests

coiled-runtime-ci.s3.none.amazonaws.com

I could be wrong but I don't think there's supposed to be a none in that url. Any idea where that's coming from?

Our code is asking boto3 to write a file to
s3://coiled-runtime-ci/test-scratch/cluster-dumps/{client.cluster.name}/<unique test name>
where client.cluster is an instance of coiled.Cluster. Its name attribute is the only thing that could turn to None in that URI.

This is happening again. The result is that the whole CI is K.O.
INFO     coiled-runtime:conftest.py:645 Cluster state dump can be found at: s3://coiled-runtime-ci/test-scratch/cluster_dumps/test_repeated_merge_spill-63fbf224/stability.test_deadlock.py.test_repeated_merge_spill.msgpack.gz

E   botocore.exceptions.EndpointConnectionError: Could not connect to the endpoint URL: "https://coiled-runtime-ci.s3.none.amazonaws.com/test-scratch/cluster_dumps/test_repeated_merge_spill-63fbf224/stability.test_deadlock.py.test_repeated_merge_spill.msgpack.gz"

Some code is trying to build a regional S3 endpoint, which has the form https://<bucket>.<region_name>.amazonaws.com, and the region name is None.
I have no idea where that's happening, or where None is happening. If anyone has a consistent reproducer, then it would be possible to dig in, but without this I'm not sure how to debug.
Anyone know how to get this consistently? (Also, does it only happen in CI?)
We are setting DEFAULT_REGION on the cluster. That's shipped from local env var (local in this case means on the CI runner). It might be worth checking that this is set to a valid value.
As a workaround, it might work to explicit set region in your code that uses s3.

It turns out that all tests that read or write data on s3://coiled-runtime-ci/ are broken. Disabling cluster dumps only fixes the subset of tests that doesn't perform any S3 I/O - which is to say, not many.

@crusaderky We'd like to help but it's hard without an example (either a reproducer or a fails-to-reproduce...) we can run outside your CI.
If you can give us that, one of us will happily dig in. Is that reasonable?

@dchudz yes just open a blank PR.
I'm failing to reproduce just with a notebook, even if I use the account dask-benchmarks.
Notably, though, on my notebook I don't have the following environment variables set:
DASK_COILED__TOKEN: ${{ secrets.COILED_BENCHMARK_BOT_TOKEN }}
AWS_ACCESS_KEY_ID: ${{ secrets.RUNTIME_CI_BOT_AWS_ACCESS_KEY_ID }}
AWS_SECRET_ACCESS_KEY: ${{ secrets.RUNTIME_CI_BOT_AWS_SECRET_ACCESS_KEY }}

@crusaderky is there anything special I need to do to run locally?
it's OK if it doesn't repro.

@crusaderky what do I need to do to print output I can see in the CI run?
I added some prints ("boto default region_name is") in https://github.com/coiled/benchmarks/pull/1029. The test they're in is running (and failing) but I don't see my output.
https://github.com/coiled/benchmarks/actions/runs/6341971500/job/17226775013?pr=1029

@crusaderky is there anything special I need to do to run locally?
it's OK if it doesn't repro.

import coiled
import distributed

cluster = coiled.Cluster(
    # kwargs from cluster_kwargs.yaml
    package_sync=True,
    wait_for_workers=True,
    scheduler_vm_types=["m6i.large"],
    backend_options={
      "spot": True,
      "spot_on_demand_fallback": True,
      "multizone": True,
    }
    n_workers=1,
    worker_vm_types=["m6i.large"],
    account="dask-benchmarks",
)

client = distributed.Client(cluster)
# The workers will use fsspec to write to s3
client.dump_cluster_state("s3://coiled-runtime-ci/test-scratch/foo-DELETEME.msgpack.gz")

@crusaderky I'm struggling b.c it seems like after the first couple changes, my tests stop running on PRs
https://github.com/coiled/benchmarks/pull/1032
any advice?

Found a fix!
If I export the env variable
AWS_DEFAULT_REGION: us-east-2
before I invoke pytest, the problem disappears. That hack was already there elsewhere in the CI script to invoke the aws cli; no idea why boto3 worked until now.

@crusaderky I'm struggling b.c it seems like after the first couple changes, my tests stop running on PRs
#1032
any advice?

it's still busy running the previous iteration of your tests; there are limited slots. You have to manually cancel the runs you don't care about in the Actions tab

Closed by #1029

Well, turns out that this is my fault—sorry!
Specifically, there was an unintended change in coiled client (made by me) such that if default boto session region was None locally, it then shipped "None" to the cluster. In most cases there is a default session, but that wasn't the case for the way creds were set in benchmarks CI.
I expect we'll get fix out in next client release.


Update glossary in UG

Add glossary for the following terms:
Attendance
Classes
Command
Index
Lessons
Students
Tag
Telegram
Utility
Venue

Will not include Student and Utility as no description/alternative explanation that adds value could be found.


Initial stab at removing aws-sdk-v1

Tests don't work, and I don't have time to continue ATM, but this removes nokogiri.

Fab :100: We can merge and open an issue to fix CI.

I'm not even sure the api is right yet, not worth doing that


find_files.ps1 instantly closes on Find It Faster: search file

The script exits immediately after the terminal opens with the following error:
→ C:\Users\Grindle\Documents\Other\Godot\Projects\Fish-Game [main ≡ +4 ~5 -0 !]› c:\Users\Grindle\.vscode\extensions\tomrijndorp.find-it-faster-0.0.30\find_files.ps1  'c:\Users\Grindle\Documents\Other\Godot\Projects\Fish-Game'
unknown option:
canceled
→ C:\Users\Grindle\Documents\Other\Godot\Projects\Fish-Game [main ≡ +4 ~5 -0 !]›

For some reason $QUERYPARAM =  "" is parsed as an actual parameter, hence unknown option:.
I've never used PowerShell before but here is an inelegant solution that constructs the command in a more explicit fashion:
Replace
if ($PREVIEW_ENABLED -eq 1) {
    $result = Invoke-Expression "rg --files --hidden $USE_GITIGNORE_OPT --glob '!**/.git/' $GLOBS $TYPE_FILTER_ARR $PATHS" | fzf --cycle --multi $QUERYPARAM "${QUERY}"  --preview "$PREVIEW_COMMAND" --preview-window "$PREVIEW_WINDOW"
}
else {
    $result = Invoke-Expression "rg --files --hidden $USE_GITIGNORE_OPT --glob '!**/.git/' $GLOBS $TYPE_FILTER_ARR $PATHS" | fzf --cycle --multi "$QUERYPARAM" "${QUERY}"
}

With
  $fzf_array = @("fzf", "--cycle", "--multi")
  if ($QUERYPARAM -ne "") {
      $fzf_array += $QUERYPARAM
      $fzf_array += "${QUERY}"
  }
  if ($PREVIEW_ENABLED -eq 1) {
      $fzf_array += "--preview"
      $fzf_array += "'$PREVIEW_COMMAND'"
      $fzf_array += "--preview-window"
      $fzf_array += "$PREVIEW_WINDOW"
  }
  $fzf_command = $fzf_array -join " "

  $rg_array = @("rg", "--files", "--hidden", "--glob", '!**/.git/')
  if ((VGet "env:USE_GITIGNORE" 0) -eq 0) {
      $rg_array += "--no-ignore"
  }
  if (($GLOBS.Length -gt 0)) {
      $rg_array += "$GLOBS"
  }
  if (($TYPE_FILTER_ARR.Length -gt 0)) {
      $rg_array += "$TYPE_FILTER_ARR"
  }
  if (($PATHS -ne "")) {
     $rg_array += "$PATHS"
  }
  $rg_command = $rg_array -join " "

  $result = Invoke-Expression ($rg_command + " | " + $fzf_command)

EDIT: Note there is already an old, now-closed pull request that tried to solve this problem but it didn't really solve it and more so hacked it out; see https://github.com/tomrijndorp/vscode-finditfaster/pull/30
EDIT 2: This seems to also be an issue in the other scripts.

Having the same problem

I personally can't help with this as I don't own a Windows machine, and rely on community contributions for the experimental Windows support. So I'm leaving this open and hopefully it'll be picked up at some point :)

#71 has some proposed changes as a fix for this issue. Please let me know if changes are required.

@tomrijndorp would it be possible to merge that PR to fix this issue?

Tried out the changed locally on my windows machine. This fixed the instant close for me.

Apologies, I haven't been able to focus much on this extension lately due to other commitments. I just published version 0.0.31 which incorporates @Luiggi33's PR :)


test: split log gathering goroutine

If Cilium pods are in crash loop backoff, kub.DumpCiliumCommandOutput
blocks for whole context duration, which causes subsequent kub.GatherLogs
call to fail immediately.
This change splits these two calls to separate goroutines.

test-4.9


Support Sentry.Context.set_http_context/1

It seems that the :request key is supported for Sentry.capture_exception/2, but it can't be set ahead of time. The SDK documentation seems to indicate that this should be supported: https://docs.sentry.io/clientdev/context/#common-context-methods
Essentially, I'm trying to do the same thing as Sentry.Plug but setting it at the time the plug is called so that Sentry.capture_exception/2 reports will have it.

It looks supported to me:
Here is where the context is set ahead of time, it works per process like Logger.metadata
https://github.com/getsentry/sentry-elixir/blob/141e7a58b7b10e2907d121f11711b58ce3777daf/lib/sentry/context.ex#L30
Here is where it is accessing the process context and sending it with the Event, which is what an exception gets transformed to:
https://github.com/getsentry/sentry-elixir/blob/141e7a58b7b10e2907d121f11711b58ce3777daf/lib/sentry/event.ex#L43
e.g.
Sentry.Context.set_extra_context(%{request: request_id})
Sentry.capture_exception(...) # correctly pulls context from Process dictionary


Remember last radio, playlist,...

If I'm listening a radio or playlist and close the application, when I started it again it doesn't remember the radio, playlist,... So I need to find what I was listening.
It would be nice if it could remember what I was listening and start at the same song.

I was thinking of creating this functionality, currently with the separate node process it makes it hard to reload windows once they are loaded due to the way it is managed, I will work on seeing if I could make some kind of save system. I was thinking of linking this into some kind of URI linking system so that you can open Spotify uri'/links in the player locally - I'm not too sure on how that works on Linux at the moment though...

I have added a basic lastURL to the preferences in the latest commit - it doesn't currently work (it gets overwritten when it's loading) so I will make sure this gets overridden and working for the next commit (probably in the next few days)

Added in https://github.com/Quacky2200/Spotify-Web-Player-for-Linux/releases/tag/1.0.0


docs: add AbortIncompleteMultipartUpload to docs for addLifecycleRule

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@ddelgrosso1 should we label this do not merge until the feature is released?


Movemodules issue

My upgrade hangs this time on "moving aside incompatible modules". It is hard to say from the error message what is going wrong.
One possibility is that this concerns a module that is still in the database but no longer on the disk.
Jqxhr messages are a horror that only tell initiates that something unspecified went wrong with an ajax call. In this case at least the name of the module should have been mentioned. It might be considered to provide the guilty ajax call with all its arguments.
note 1: in this case debug mode was off.
note 2: I tried several times but always with the same result. Finally I got out of the situation by upgrading the shop to Prestashop <IP_ADDRESS> first. After that psonesixmigrator worked ok.

Hi, I stumbled on the very same error at the same step with the latest psonesixmigrator module  v2.0.1, Prestashop 1.6.23 so the latest 1.6.x line release.
This is a blocking issue that prevents the upgrade.
In my case I found the culprit by activating the Pand HP error log. There was a stack trace of the fatal error in module blockmyaccountfooter.
" PHP Fatal error:  Uncaught Error: Call to undefined method stdClass::createData()"
I deinstalled the blockmyaccountfooter module and the migration went smoothly.
Hopes this helps and maybe you could improve the psonesixmigrator to handle this correctly.

Probably can't help much with the error display problem. These 500 errors usually mean a PHP problem, so one should find something in log/ in the shop root folder.

To remove the showstopper, I wrapped module uninstallation in try ... catch, hopefully replacing this Exception with an error message and accordingly, these type 500 errors going away. See 8be7f2ad09c957637d2e819a805bfb1130bc0514.
A module release will follow before too long.

The new module release happened: https://github.com/thirtybees/psonesixmigrator/releases/tag/2.1.0
I hope this removes these nasty showstoppers. If it doesn't, please don't hesitate to re-open this issue or open another one.

Thanks !

Unfortunately, the issue is still there.
Tried to migrate a PS <IP_ADDRESS> to tb 1.0.8
Got error:
[Ajax / Server Error for action deleteModules] textStatus: "error " errorThrown:"Internal Server Error " jqXHR: " "


Context menus aren't removed when a package is removed.

Based on information from https://github.com/atom/tree-view/issues/131 and https://github.com/atom/tabs/issues/29

This has been fixed.


depend on phpcs stable version

now stable version of phpcs is 3.0.2

3.0 has a different autoloading mechanism. Are you planning to fix the breaking changes?

@felixfbecker sorry, i just wonder why keep this version, i don't have enough ability to fix it, sorry again


flash algorithm execution speed

I've been trying to optimize pyOCD flash write speed with my debug probe.  After configuring trace debug logs (thanks Chris!), I noticed less than half the time is spent doing block writes.  Most of the time seems to be spent generating the AP read and write commands in between block transfers.  Here's an excerpt from the logs:
0002482:DEBUG:ap:_write_block32:001014 }
0002482:DEBUG:ap:read_mem:001018 (ap=0x0; addr=0xe000edf0, size=32) {
0002482:DEBUG:ap:write_ap:001019 cached (ap=0x0; addr=0x00000000) = 0x23000012
0002482:DEBUG:dap:write_ap:001020 (addr=0x00000004) = 0xe000edf0
0002482:DEBUG:dap_access_cmsis_dap:get_request_space(1, 05:w)[wc=7, rc=0, ba=1->0] -> (sz=1, free=5, delta=-247)
0002482:DEBUG:dap_access_cmsis_dap:add(1, 05:w) -> [wc=8, rc=0, ba=0]
0002482:DEBUG:dap_access_cmsis_dap:get_request_space(1, 0f:r)[wc=8, rc=0, ba=0->0] -> (sz=1, free=15, delta=-246)
0002482:DEBUG:dap_access_cmsis_dap:add(1, 0f:r) -> [wc=8, rc=1, ba=0]
0002483:DEBUG:dap:read_ap:001021 (addr=0x0000000c) -> ...
0002483:DEBUG:dap_access_cmsis_dap:New _Command
0002485:DEBUG:dap:read_ap:001021 ...(addr=0x0000000c) -> 0x01030003
0002485:DEBUG:ap:read_mem:001018 (ap=0x0; addr=0xe000edf0, size=32) -> 0x01030003 }
0002485:DEBUG:ap:read_mem:001022 (ap=0x0; addr=0xe000edf0, size=32) {
0002485:DEBUG:ap:write_ap:001023 cached (ap=0x0; addr=0x00000000) = 0x23000012
0002485:DEBUG:dap:write_ap:001024 (addr=0x00000004) = 0xe000edf0
0002485:DEBUG:dap_access_cmsis_dap:get_request_space(1, 05:w)[wc=0, rc=0, ba=1->1] -> (sz=1, free=14)
0002485:DEBUG:dap_access_cmsis_dap:add(1, 05:w) -> [wc=1, rc=0, ba=1]
0002485:DEBUG:dap_access_cmsis_dap:get_request_space(1, 0f:r)[wc=1, rc=0, ba=1->0] -> (sz=1, free=15, delta=-253)
0002485:DEBUG:dap_access_cmsis_dap:add(1, 0f:r) -> [wc=1, rc=1, ba=0]
0002485:DEBUG:dap:read_ap:001025 (addr=0x0000000c) -> ...
0002486:DEBUG:dap_access_cmsis_dap:New _Command
0002488:DEBUG:dap:read_ap:001025 ...(addr=0x0000000c) -> 0x00030003
0002488:DEBUG:ap:read_mem:001022 (ap=0x0; addr=0xe000edf0, size=32) -> 0x00030003 }
0002488:DEBUG:ap:write_mem:001026 (ap=0x0; addr=0xe000edf4, size=32) = 0x00000000 {
0002488:DEBUG:ap:write_ap:001027 cached (ap=0x0; addr=0x00000000) = 0x23000012
0002488:DEBUG:dap:write_ap:001028 (addr=0x00000004) = 0xe000edf4
0002488:DEBUG:dap_access_cmsis_dap:get_request_space(1, 05:w)[wc=0, rc=0, ba=1->1] -> (sz=1, free=14)

It takes until the 2504ms timestamp to get a full packet to write.  In total from the end of one write_block32 until the start of the next write_block32 is 45ms.  pyOCD seems to be doing some sort of caching of the parsed flash algorithm, as the time between the 1st and 2nd block writes is about twice as long (~100ms).  Is there any easy way to speed this up?  The host CPU is a 3.2GHz Intel Core 64-bit.

I did a comparison between pyOCD and openocd flashing a 20kB file to a stm32f103.  pyOCD runtime was 3.9s vs 2.45s for openocd.

Obviously a big part of that is simply that Python is slower than C. Are you using Python 3.9? It's noticeably faster than 2.7, or even earlier 3.x versions. Another option is pypy3.
There are probably multiple optimizations that could be implemented within pyocd, but I'm not aware of anything that's low hanging fruit. Profiling like you're doing is the best way to start.

My tests were with Python 3.7.9.  I'll try 3.9.1 to see if it is any faster.  Do you know if pyOCD works with pypy3?  My experience with it was not great in the past.
I've noticed pyOCD only lightly loads 2 of 4 cores.  If my python programming skills were better I'd consider modifying the AP code to use a separate thread for generating the sequence of commands required for the flash algorithm.

Python 3.9 doesn't run on Win7.  I just tried upgrading to 3.8.7, and pyOCD is a bit faster now, taking 3.6s instead of 3.9s.

I haven't tested a ton with pypy3, but I didn't run into any problems. Ymmv. (And I've had plenty of trouble with it in other cases.)
The problem with multithreading in CPython is the GIL: Global Interpreter Lock. That about says it. Performance is only improved for multithreaded CPython if your threads are mostly IO-bound, or call into C-based libraries that release the GIL while they run. The multiprocessing package makes it easy to use separate processes for performance instead of threads, but the overhead probably would make any gains disappear. …… There are benefits to using Python, but also some serious drawbacks.
One option we've talked about is to have sort of a meta-flash-algorithm, code that runs on the target to orchestrate calls to the algo. For instance, instead of having to call-and-wait for every sector to be erased, drop an array of sector addresses in RAM and call the orchestrator, which will then call the algo for each address. The question is, would it really improve performance when double-buffering is being used? Usually the flash operations are the real bottleneck.
It's been several years since I looked at flash programming performance in detail, and there may very well be areas that could be improved that I'm not thinking of.

Well, that's something at least…
One thing that may improve pyocd performance is how it handles data buffers. Instead of using an efficient type like array.array or bytebuffer, it uses lists of ints. This unfortunate design choice was deeply ingrained from the beginning. I've had fixing it on my todo list for a long time (probably waiting til after dropping Python 2 would be a good idea). I wonder if it would even make a measurable difference, since for the most part the buffers that pyocd works with are quite small.

I also don't think the data buffer handling is a performance problem.  If it were, then I think the write_block32 would be slow.  It's too bad the flash algorithms weren't simpler (and therefore more efficient).  The way I'd do it is have the code that is loaded into SRAM loop through a chunk of data to write to flash, and jump back to the start.  I'd have the debug probe set a breakpoint at the branch instruction, and continue execution when the next chunk of data is transferred to SRAM.
As for simpler ways of getting pyOCD to run faster, I just tried nuitka with a small program that failed with pypy on Windows 7.  The nuitka-compiled version works fine.  I'm not sure how to get it to compile a whole package like pyOCD, but from skimming the docs, I think nuitka can do that.

@flit pyOCD is really slow if flashing a large application. Set keep_unwritten=False it can improve the flashing performance and we used the way for NXP IMXRT series. I do not know why but it is useful.

@Hoohaha That's a really good point. By default, pyocd will try to a) only erase and program sectors that change, b) not lose data in a partially-programmed sector. These behaviours are controlled by the smart_flash and  keep_unwritten options, respectively. smart_flash requires reading existing flash contents. Whether it really improves performance, or slows it down, depends a lot on how you're using pyocd, i.e. writing entirely new code versus frequent updates of mostly the same code in a development cycle. I'm not sure exactly why keep_unwritten would noticeably affect performance (it's been a while since I wrote that code), but the large sector sizes for the QSPI devices used with RT series probably plays a role in what you're seeing.
@nerdralph I didn't know about nuitka, very cool! I had been thinking about using cython.

The way I'd do it is have the code that is loaded into SRAM loop through a chunk of data to write to flash, and jump back to the start. I'd have the debug probe set a breakpoint at the branch instruction, and continue execution when the next chunk of data is transferred to SRAM.

That's basically how it works. First the algo is loaded to RAM and inited. (Sectors erased first.) Then a buffer of data is copied to RAM, and the algo started running. It loops over the data, writing to flash in whatever size the flash word is (usually 4, 8, or 16 bytes, though more modern ~40nm parts have 256- or 512-byte words). When done it returns to a bkpt instruction. While the algo is running, a second buffer is being filled with the next page to write, and the buffer ping-ponging begins. On almost all devices, pyocd is finished writing the next buffer before the algo completes for the previous.


Carla Library for ARM64

Hello everyone,
I'm trying to use Jetson Nano as a client and my desktop pc as the server. But I couldn't get carla library for jetson. When i try to use "make PythonAPI"  i receive error. Also linux_x86_64 egg file doesnt work for ARM64. Can anybody provide me the egg file for aarch64 or give suggestions create my own egg file ? Thank you.

As far as i see you have to compile everything from scratch for arm architecture. Everything around here is using x86. The Ubuntu and Windows builds are x86. arm is a completely different Instruction set architecture (ISA).
https://en.wikipedia.org/wiki/Comparison_of_instruction_set_architectures

@jokolaco Yes, I think so too.  I searched for aarch64 egg file, but couldn't find anywhere. In one post, someone managed to create the eggfile with "make PythonAPI" command. But I couldn't make it. I want to connect the jetson to my PC with an ethernet cable but this problem stalled the process.

I am not sure if arm is supported. But if this is the case, you should use on your Jetson Nano the same Ubuntu version like the server. Also you need all the libs. Try to understand the compiler errors and follow the script. There a many dependencies of external sources(e.g. libosm2odr.a, libxerces-c.a) which also needs to be compiled on arm. I would suggest just to use another client on x86, besides the nano is not very powerful so you can easily use a cheap old standard pc, laptop as a client or just your server ;-)

@jokolaco thank you for the responses. I've seen some implementations with ROS and jetson. I'll experiment a little bit.

Hello,
Is there any prebuilt Python libraries(.egg files etc.) that you can share for aarch64 platform?

@jokolaco thank you for the responses. I've seen some implementations with ROS and jetson. I'll experiment a little bit.

Have you successfully deployed Carla on Jetson? Can you tell me how it is achieved?


install --force should delete existing gems before installing

@segiddins that's a bug in force, then. we need to nuke the directory, since it's possible that a malicious gem could install files that don't get overwritten when the non-malicious gem is installed. :/

@indirect Can you provide more details? I'd like to help fix this bug. Thanks!

@karlo57 sure, no problem.
Right now, when you run bundle install --force, we download each .gem file again (even if we already have a copy downloaded), and we install each gem again (even if that gem is already installed). The additional thing we need to do is uninstall each gem before we download and install it again. Or if not uninstall, at least delete the directory containing the installed/unpacked files, to guarantee that the only files that are left afterward came from the gem that was just installed with --force.

@indirect  I'm deleting the bundle path on first run of the install process if the --force flag is ticked. Is this correct? And should I also delete the user's directories for instances that the user has defined a different install path?
Thanks!
https://github.com/karlo57/bundler/commit/1f1688ae8440827342f0dd54aaf1e5a7ac104470

I think we should close this now? I'll create a new issue for the new bug found out.

@karlo57 Resolved by which commit?

@indirect Should we close this now that https://github.com/bundler/bundler/commit/13c445680f65293ee50f11718e8a86bfa87abd2e is merged? It's not clear if this is an issue fixed by that commit or it's a non-issue.

I guess it was a non-issue? Either way, seems resolved by a test now.


Add support for importing types within the same package

The Travis build has failed.

I also pushed up the version to 1.2 and added the modification to the CHANGELOG (I was not sure if you had any pending changes planned).

Thanks ! Do you have an ETA to release it to OPAM ? I have a project waiting to be released that depends on that feature. Nothing urgent though, just curious.

https://github.com/ocaml/opam-repository/pull/9020

Thanks !


LuckPerms support

It is the best permission manager you'll find and that's why I would love the support of it on this addon.

Surely will add this! :smile:  it has been added to my todo list.

+1


vegeta report -reporter json fails with 1 request

What version of the project are you using?
6.1.1
What operating system and processor architecture are you using?
macOS Sierra
Version 10.12.1 (16B2555)
MacBook Pro (Retina, 15-inch, Mid 2014)
Processor: 2.5 GHz Intel Core i7
What did you do?
I have a vegeta attack output file with only 1 request.
cat attack_output.data | vegeta report -reporter json
What did you expect to see?
a json report
What did you see instead?
> cat attack_output.data | vegeta report -reporter json
2016/12/14 19:38:29 json: unsupported value: +Inf

> echo $?
1

Looks like go's json formatter can't handle +Inf. In python, json.dumps(float('inf')) == 'Infinity'.

Hey @chriswatrous, thanks for reporting this edge case. Just for me to understand where you're coming from, do you use Vegeta for single requests?

If I have a call with a complicated body or set of headers, I sometimes use 1 call until I get everything right. I have a microservice that actually runs vegeta attack and a script that calls that microservice and generates the reports. My script fails if vegeta exits with a nonzero code.


WQX output incorrect AQL value spelling

In tabMWRwqx(), the Result Detection Condition logic should be "If input Result Value = BDL then 'Not Detected', else if input Result Value = AQL then 'Present Above Quantification Limit', else blank."  Currently, "Present Above Quantification Limit" is output as "Present Above Quantitation Limit", which is not accepted by WQX.  This needs to be corrected, but be aware not to change anything in the Result Detection/Quantitation Limit columns.  In these columns it should be spelled as "quantitation".

This looks good.


Autoload shared libraries

When a conifer JSON model is loaded conifer is not able to load also the conifer_bridge.so shared libraries (if they were already produced).
This implies:

The user must run .compile() every time
Every time that the model is re-compiled a new shared library with a different timestamp is created and a new dict is appended in the metadata list in the model JSON

This PR tries to fix this behavior by looking for the shared library with the last timestamp present in the metadata. It looks for it in the same folder of the JSON if the is not specified by the user.
It works for both the xilinx and cpp backends.

Ok, I have applied all the comments.

I created a load_shared_library in ModelBase that does nothing but is overloaded in the backends that need to load a shared library.
Now shared_library can be None, a dir path, or the shared library path. If None or dir string, a .so file is searched in the JSON or the specified directory starting from the last available timestamp to the oldest.
If shared_library is the path to the .so file, it is automatically loaded

I think the code changes for the library reloading are now good, but I'd like to see more tests:

split the test into some test functions that aim to test only this functionality
test some of the expected behaviour explicitly, not only the prediction results (e.g. check the name of the loaded library)
test different combinations of the arguments


rewrite tests to be compatible with pytests; no main and test data needs to be in root, not in the package directory

the pytests need to be run via python:
python -m pytest tests/
Not sure why simple pytest tests won't work

Additionally pytest does not see the path of the other model.
Maybe there is a cleaner way than appending it to the sys.path.


TextField with type date - intended or error when using label

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Using the component <TextField type="text" label="myLabel" placeholder="myPlaceholder" /> leads to the having the label and a date mask to overwrite each other.

Expected behavior 🤔
I would expect the element only to show the label and eventually the date format when an invalid date has been used or as a tooltip.
Steps to reproduce 🕹
https://codesandbox.io/s/useformcontrol-material-demo-forked-270g0?file=/demo.tsx:230-306
import * as React from "react";
import TextField from "@mui/material/TextField";
import Box from "@mui/material/Box";

export default function UseFormControl() {
  return (
    <Box component="form" noValidate autoComplete="off">
      <TextField type="text" label="myLabel" placeholder="myPlaceholder" />
      <br/>
      <TextField type="date" label="myLabel" placeholder="myPlaceholder" />
    </Box>
  );
}

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
 System:
    OS: macOS 12.1
    CPU: (8) x64 Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz
    Memory: 463.25 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
    npm: 8.3.1 - ~/.nvm/versions/node/v16.13.2/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.9.1 - /usr/local/bin/pod
    Homebrew: 3.3.10 - /usr/local/bin/brew
    pip3: 21.3.1 - ~/.pyenv/shims/pip3
    RubyGems: <IP_ADDRESS> - /usr/bin/gem
  Utilities:
    CMake: 3.22.1 - /usr/local/bin/cmake
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.32.0 - /usr/bin/git
    Clang: 13.0.0 - /usr/bin/clang
    Subversion: 1.14.1 - /usr/local/bin/svn
  Servers:
    Apache: 2.4.51 - /usr/sbin/apachectl
  Virtualization:
    Docker: 20.10.12 - /usr/local/bin/docker
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
  IDEs:
    Nano: 2.0.6 - /usr/bin/nano
    VSCode: 1.63.2 - /Applications/Visual Studio Code.app/Contents/Resources/app/bin/code
    Vim: 8.2 - /usr/bin/vim
    Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Java: 11.0.10 - /usr/bin/javac
    Perl: 5.34.0 - /usr/local/bin/perl
    Python: 3.10.0 - /Users/doberkofler/.pyenv/shims/python
    Python3: 3.10.0 - /Users/doberkofler/.pyenv/shims/python3
    Ruby: 2.6.8 - /usr/bin/ruby
  Databases:
    SQLite: 3.36.0 - /usr/bin/sqlite3
  Browsers:
    Chrome: 97.0.4692.99
    Safari: 15.2

I want to fix it
I can just hide the date format placeholder and show it when the focus is on the input

Sounds good to me

@doberkofler I've discovered that this problem was described in the documentation
https://mui.com/components/text-fields/#shrink
Can it help you for a while?

I need help. @mnajdova
We cannot detect the existence of a default placeholder.
We only can detect it by type. So, I see two ways to close this problem, but you can see more.
The first one is ignoring this problem. The second one is hardcoding all of the types which has a default placeholder.
What is the best solution?