Running with many CSV traces is causing Segmentation Fault

Running HARM with more and more CSV traces increases the probability of a segmentation fault occurring during execution.
With <150 traces, HARM might crash 20% of the time.
With >1000 traces, HARM might crash 98% of the time.
This is likely related to the multi-thread processing functionality of the program and involve race-conditions or deadlocks because the following execution will not cause a crash:
harm --csv-dir {my_csv_dir} --clk clk --conf {my_harm_config_dir}/harm_config.xml --max-threads 1
Removing the --max-threads limit causes possible crashes to occur again.
harm --csv-dir {my_csv_dir} --clk clk --conf {my_harm_config_dir}/harm_config.xml

Hi @michaelpekala-gtri
The number of traces used as input likely has nothing to do with race conditions. Threads share only read-only memory, and everything else is duplicated to avoid issues such as false sharing, which can significantly reduce performance. Could you share your input so I can reproduce the error? Also, what version of harm are you using—was it compiled from source, or are you using the Docker image?

csv.zip
harm_config.txt
I built on version 0511e0a.
I ran with the following command: harm --csv-dir csv/ --clk clk --conf harm_config.xml
I have attached a .txt file since Github will not let me upload .xml files.

You are using an extremely old version of harm. I recently released v2, in which several bugs were fixed. I suggest upgrading to the latest commit. I tested your example on 10 runs with the latest version, and none of them crashed. Please read the README.md to see the changes of v2: some options have changed; for example, in your XML, you are using dtLimits option 'R' that became 'U' in v2.


Unclear on which driver is required

The example works fine and a device called 'PIC32MZ USB' enumerates in Windows, however, I am unable to determine which driver needs to be installed in order to perform data transfer. Testing through terminal, so probably a serial port?

The device enumerates with a "vendor specific" interface (0xFF, // bInterfaceClass - Vendor Specific) and is intended to use with libusb or WinUSB.
You can use Zadig or the Windows dialog to install the driver.

I then use ZorkLibUSB as a wrapper around libusb to access the device.

Is there an easy way for it work as a standard virtual serial device, even the driver is manually installed?

It requires the implementation of the usb-cdc protocol, similar to this example:
https://github.com/majbthrd/pic16f1454-bootloader/tree/master/example-apps/minimalCDC
The descriptors have to be modified/added and a number of control-transfer callbacks have to be implemented.
Then a "USB Serial Device" and tools like PuTTY would work.

Thank you


openlayers 6 not working with vue web component

Version
2.6.11
Reproduction link
https://codesandbox.io/s/bold-matsumoto-tpxsn?file=/src/App.vue
Steps to reproduce
First you can see the link to minimal production that work very well.
To produce the bug:
Download the project using or execute npm run build-component. This will run:
vue-cli-service build --target wc --name openlayers-vue ./src/components/MapContainer.vue
Go to dist, then open demo.html. As you can see, openlayer not working any more.
What is expected?
I must see the map works in dev mode and prod mode (web component)
What is actually happening?
The map is working only in dev mode .

Probably related to #10932: with ol version 6.3.1, ol.css must be included for the interaction to work.

@fredj  thanks but  the pb is openlayers not mount the html element as you can see here, the dev is empty.

I can't help with Vue but I've loaded dist/demo.html and when the OpenLayers Map constructor is called, the target element is not present in the page (document.getElementById('mapOL') is null)

@fredj, thanks. Now i am using another method in vue that allow us to get html element. So what is have now  is mounted openlayers with zoom control but without canvas.
#DOM image:

#browser result (only zoom control is showing)

@fredj, thanks. Now i am using another method in vue that allow us to get html element. So what is have now is mounted openlayers with zoom control but without canvas.
#DOM image:

#browser result (only zoom control is showing)

I can not understand why openlayers is not putting canvas inside html element.

Try to remove the style 'display: none' of 'ol-unselectable ol-layers' element.

@Kiyoaki-w same thing. No map canvas in the html DOM.

https://codesandbox.io/s/vibrant-kepler-6wvjb seems to work fine for me. Can you confirm?

@ahocevar, thank to your participation in this ticket. So the example that you indicated is  dev mode or in dev version, in this case, openlayers works.  But in web component prod version,  like this demo.html example https://github.com/bnping/openlayers6-web-vue, Openlayers is not putting canvas in the html dom, so their is no map. I see only the zoom control buttons.
I asked the vue core team about this, they said that this is a pb in Openlayers and for this reason, i am here try to find a solution.

thanks you all, i found the solution. Here is the working example
https://codesandbox.io/s/adoring-worker-i9k3d?file=/index.html

I've been using Vue successfully with OpenLayers in production mode. But I do not use web components.

@ahocevar  Now we have a working example that  can used by the community.


Bot name issues in creating a new bot

Version
master
Describe the bug
Bot Name could not be show up during bot creating.
Go to home page , create a new bot, 2 issues.

go to default myBot directory, there is no bot name show up but actually I do have bot created

check dup bot name should check for the location loaded but not "myBot"

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

addressed in https://github.com/microsoft/BotFramework-Composer/issues/1445


is this the os with onered square or a cube it may have two but i cant remember

i want to know to take a look at the project i think it is skylight but i cant remember i was just messing around with it a bit

@austanss can you send me a picture of the os please

Yea I think I found the one I was thinking about it’s all a blur to me rn

But if I had a USB to sata cable I could get the source code for it but I don’t have one no more mine broke


lsp: Add preStart/postStop hooks to run custom command when start lsp

Language server
golang
Requested feature
  Is possible supplied a way to inject custom command into vim.lsp?  like
vim.lsp.start({
   name="gopls",
   cmd= {'name-of-language-extensions'},
   root_dir=str,
   pre_start={'custom command'} or function or nil,
   post_stop = {'custom command'} or function or nil,
})

Other clients which have this feature
No response

:help LspAttach, :help LspDetach
More generally: this repo is a collection of configs. More "features" will not be added to this repo.


Update Readme.md

It works on Linux, updated line that previously said only works on Mac OS.

I know it works on Linux but it doesn’t work for Slack when downloaded from the store (SNAP)


No promotion page found

Request type
Correction or update
Details
https://wiki.developer.mozilla.org/en-US/docs/MDN/About/Promote
This link shows not found

I deleted this because it was out of date. Did you find a link to it somewhere?

@chrisdavidmills  Actually  on mdn web docs  page under the Help improve MDN section there is a link to this page which shows Not Found

@heysujal oops ;-)
I'll transfer this issue to the Kuma repo and ask one of the devs help me to locate this link and remove it. Thanks for reporting!

Issue moved to mdn/kuma #7644 via ZenHub


OpenApiInfo, TermsOfService have to be uri not string

Description
OpenApiInfo, TermsOfService have to be Uri not String
Page URL
https://learn.microsoft.com/en-us/aspnet/core/tutorials/getting-started-with-nswag?view=aspnetcore-8.0&tabs=visual-studio
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/tutorials/getting-started-with-NSwag.md
Document ID
25f7fb5b-92b0-a45b-57ab-fdceab571884
Article author
@zuckerthoben

Thanks, I took a look at the latest spec to see if something changed.  Per the spec it should be a URL as type string.  Looking at the code example on this doc and it uses an example of a valid URL as a string.
 TermsOfService = "https://example.com/terms"
I will go ahead and close, but if I am missing something here let me know and I am happy to reopen.
The latest spec (3.1.0):
3.1.0
termsOfService
Type: String
Description: A URL to the Terms of Service for the API. This MUST be in the form of a URL.


Locale is returning en_ if country code is empty string

Steps to Reproduce

Create two locales one = Locale("en") and two = Locale("en", "")
Log both
See that one is "en" and two is "en_"

Additional Info and thoughts
This happens on an iOS simulator with the default locale.
I did some research and in this code: https://github.com/flutter/engine/blob/master/lib/ui/window.dart#L511
There should be an additional check if the scriptCode or the _countryCode  is an empty string because "en_" is not a valid Unicode BCP47 Locale Identifier

I did some research and in this code: https://github.com/flutter/engine/blob/master/lib/ui/window.dart#L511
There should be an additional check if the scriptCode or the _countryCode  is an empty string because "en_" is not a valid Unicode BCP47 Locale Identifier

Thanks for the investigation!  Pull requests are always welcome, if you are interested.
See https://github.com/flutter/engine/blob/master/CONTRIBUTING.md.

Historically, empty string as country code is valid, so adding an assert there is not correct. However, what we can do is check for empty string in the toString to not produce the extra underscore.

This was a quick change, so I made https://github.com/flutter/engine/pull/17280


NullPointerException during the docker tutorial

Hello,
I'm following the docker tutorial, the first steps go well without problems but when I try to launch the Mongo NS I got "an uncaught exception. Message is: java.lang.NullPointerException" from the NFVO.
I can't figure out what I did wrong. I'd like some help.
Log and explanations :
openbaton.log
2018-03-28 08:33:30.693  WARN 911 --- [http-nio-8080-exec-10] o.o.n.a.i.AuthorizeInterceptor           : AnonymousUser requesting method: GET on /error
2018-03-28 08:44:11.150  INFO 911 --- [http-nio-8080-exec-3] o.o.n.c.a.NetworkServiceRecordManagement : Removing NSR with id: dc621619-401a-4739-adb2-ec138de43e85
2018-03-28 08:44:11.164  INFO 911 --- [OpenBatonAsyncTask-30] o.o.nfvo.vnfm_reg.state.VnfStateHandler  : Executing Task releaseresourcesTask for vnfr MongoDB. Cyclic=false
2018-03-28 08:44:11.164  INFO 911 --- [OpenBatonAsyncTask-12] o.o.n.v.tasks.abstracts.AbstractTask     : Release resources for VNFR: MongoDB
2018-03-28 08:44:30.706  INFO 911 --- [http-nio-8080-exec-10] o.o.n.c.a.NetworkServiceRecordManagement : Looking for NetworkServiceDescriptor with ID: 2f7861f7-fbba-4ad0-8fa0-d6881801f882
2018-03-28 08:44:30.709  INFO 911 --- [http-nio-8080-exec-10] o.o.n.c.a.NetworkServiceRecordManagement : Checking if all vnfm are registered and active
2018-03-28 08:44:30.715  WARN 911 --- [http-nio-8080-exec-10] o.o.n.c.a.NetworkServiceRecordManagement : Quota check is disabled... Please enable for comprehensive grant operation
2018-03-28 08:44:30.733  INFO 911 --- [OpenBatonAsyncTask-26] o.o.nfvo.vnfm_reg.state.VnfStateHandler  : Executing Task grantoperationTask for vnfr MongoDB. Cyclic=false
2018-03-28 08:44:30.733  INFO 911 --- [OpenBatonAsyncTask-13] o.o.n.v.tasks.abstracts.AbstractTask     : Executing task: GrantOperation on VNFR: MongoDB
2018-03-28 08:44:30.743  WARN 911 --- [OpenBatonAsyncTask-13] o.o.n.v.tasks.abstracts.AbstractTask     : Checking quota is disabled, please consider to enable it
2018-03-28 08:44:30.749  INFO 911 --- [OpenBatonAsyncTask-13] o.o.n.core.core.VnfPlacementManagement   : Chosen VimInstance: Optional[BaseVimInstance{name='vim-instance', location=Location{n
ame='Berlin', latitude='52.525876', longitude='13.314400'} BaseEntity{id='506a52e5-c9c6-4b0f-b4e1-1e153d07ac10', version=2, projectId='', shared=false, metadata={}}, type='docker', active=tr
ue} BaseEntity{id='c74402fa-c0b7-4ba9-87c1-f3acb6aa6684', version=3, projectId='2e01fe52-fead-43d5-8adc-346db6c845a4', shared=false, metadata={}}]
2018-03-28 08:44:30.753  INFO 911 --- [OpenBatonAsyncTask-29] o.openbaton.nfvo.core.api.VimManagement  : Refreshing vim
2018-03-28 08:44:30.798  INFO 911 --- [OpenBatonAsyncTask-13] o.o.nfvo.core.core.NetworkManagement     : Creating network new-network on vim vim-instance
2018-03-28 08:44:30.859  INFO 911 --- [OpenBatonAsyncTask-13] org.openbaton.vim_impl.vim.GenericVIM    : Created Network with name: new-network on VimInstance vim-instance
2018-03-28 08:44:30.861  INFO 911 --- [OpenBatonAsyncTask-13] o.o.nfvo.core.core.NetworkManagement     : Created Network new-network_7168
2018-03-28 08:44:30.873  INFO 911 --- [OpenBatonAsyncTask-13] o.o.n.v.tasks.abstracts.AbstractTask     : Choose all Vim Instance for vnfr: MongoDB
2018-03-28 08:44:30.893  INFO 911 --- [OpenBatonAsyncTask-26] o.o.nfvo.vnfm_reg.state.VnfStateHandler  : Executing Task allocateresourcesTask for vnfr MongoDB. Cyclic=false
2018-03-28 08:44:30.894  INFO 911 --- [OpenBatonAsyncTask-14] o.o.n.v.tasks.abstracts.AbstractTask     : Executing task: AllocateResources for VNFR: MongoDB
2018-03-28 08:44:30.927  INFO 911 --- [OpenBatonAsyncTask-29] o.o.nfvo.core.core.ResourceManagement    : Checking if Flavor m1.small exists...
2018-03-28 08:44:30.927  WARN 911 --- [OpenBatonAsyncTask-29] o.o.nfvo.core.core.ResourceManagement    : Flavor creation is supported for OpenStack only at the moment
2018-03-28 08:44:30.930  INFO 911 --- [OpenBatonAsyncTask-29] o.o.nfvo.core.core.ResourceManagement    : FloatingIp chosen are: {new-network=}
2018-03-28 08:44:30.932 ERROR 911 --- [OpenBatonAsyncTask-14] o.o.n.v.tasks.abstracts.AbstractTask     : There was an uncaught exception. Message is: java.lang.NullPointerException
2018-03-28 08:44:30.936  INFO 911 --- [OpenBatonAsyncTask-14] o.o.n.v.tasks.abstracts.AbstractTask     : Saved the VNFR MongoDB with status error after an exception
2018-03-28 08:44:30.954  INFO 911 --- [OpenBatonAsyncTask-30] o.o.nfvo.vnfm_reg.state.VnfStateHandler  : Executing Task errorTask for vnfr MongoDB. Cyclic=false
2018-03-28 08:44:30.954  INFO 911 --- [OpenBatonAsyncTask-30] o.o.nfvo.vnfm_reg.state.VnfStateHandler  : Sent INSTANTIATE to VNF: MongoDB
2018-03-28 08:44:30.954 ERROR 911 --- [OpenBatonAsyncTask-15] o.o.n.v.tasks.abstracts.AbstractTask     : ERROR from VNFM: Unable to allocate Resources
2018-03-28 08:44:30.956 ERROR 911 --- [OpenBatonAsyncTask-15] o.o.n.v.tasks.abstracts.AbstractTask     : Received ERROR message from VNFM related to VNFR: MongoDB
2018-03-28 08:44:30.960 ERROR 911 --- [OpenBatonAsyncTask-15] o.o.n.v.tasks.abstracts.AbstractTask     : There was an uncaught exception. Message is: Object of class [org.openbaton.catalogue
.mano.record.VirtualNetworkFunctionRecord] with identifier [79675b3d-7437-43be-bd8e-be7a2ce4e8fb]: optimistic locking failed; nested exception is org.hibernate.StaleObjectStateException: Row
 was updated or deleted by another transaction (or unsaved-value mapping was incorrect) : [org.openbaton.catalogue.mano.record.VirtualNetworkFunctionRecord#79675b3d-7437-43be-bd8e-be7a2ce4e8
fb]

no log in the plugin directory for docker (only for plugin-openstack and plugin-test)
dockerVNFM
08:44:11 [DEBU] docker-vnf -> handleNfvM ▶  Received Message RELEASE_RESOURCES
08:44:11 [NOTI] docker-vnf -> (*VnfmImpl ▶  Remove container for vnfr: MongoDB
08:44:11 [DEBU] docker-vnf ->  getConfig ▶  Getting config with id: 46d50827-43fa-4915-bd59-6f17dacb10c9
08:44:11 [ERRO] docker-vnf -> (*VnfmImpl ▶  Error while getting config: EOF
08:44:11 [ERRO] docker-vnf -> (*VnfmImpl ▶  Probably not found
08:44:30 [DEBU] docker-vnf -> handleNfvM ▶  Received Message INSTANTIATE
08:44:30 [DEBU] docker-vnf -> (*worker). ▶  received extensions: map[brokerPort:5672 monitoringIp: nsr-id:9df2b168-2897-47a0-8355-4dccc7030770 brokerIp:<IP_ADDRESS> timezone:CET]
08:44:30 [DEBU] docker-vnf -> (*worker). ▶  received keys: []
08:44:30 [INFO] docker-vnf ->        Rpc ▶  Executing RPC to queue: vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Getting Channel for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Got Channel for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Declaring Queue for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Declared Queue for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Registering consumer for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Registered consumer for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Publishing message to queue vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Published message to queue vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Received Response
08:44:30 [DEBU] docker-vnf -> (*worker). ▶  Received VNFR after GRANT_OPERATION
08:44:30 [DEBU] docker-vnf -> (*worker). ▶  allocating resources for the VNFR
08:44:30 [DEBU] docker-vnf -> (*worker). ▶  will send to NFVO UserData
08:44:30 [INFO] docker-vnf ->        Rpc ▶  Executing RPC to queue: vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Getting Channel for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Got Channel for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Declaring Queue for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Declared Queue for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Registering consumer for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Registered consumer for RPC
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Publishing message to queue vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Published message to queue vnfm.nfvo.actions.reply
08:44:30 [DEBU] docker-vnf -> handleNfvM ▶  Received Message ERROR
08:44:30 [ERRO] docker-vnf -> (*worker). ▶  received an error from the NFVO
08:44:30 [ERRO] docker-vnf -> (*VnfmImpl ▶  Recevied Error for vnfr: MongoDB
08:44:30 [DEBU] docker-vnf ->        Rpc ▶  Received Response
08:44:30 [ERRO] docker-vnf -> (*worker). ▶  exchange error
08:44:30 [ERRO] docker-vnf -> handleMess ▶  &{Unable to allocate Resources 0xc420267d40 9df2b168-2897-47a0-8355-4dccc7030770}
08:44:30 [DEBU] docker-vnf -> handleNfvM ▶  Received Message ERROR
08:44:30 [ERRO] docker-vnf -> (*worker). ▶  received an error from the NFVO
08:44:30 [ERRO] docker-vnf -> (*VnfmImpl ▶  Recevied Error for vnfr: MongoDB

docker driver
08:44:30 [INFO] docker-dri -> PluginImpl ▶  Listed 20 images
08:44:30 [INFO] docker-dri -> PluginImpl ▶  Listed 5 networks
08:44:30 [DEBU] docker-dri -> PluginImpl ▶  Networks are [host bridge none ob_default new-network_4612]
08:44:30 [DEBU] docker-dri -> PluginImpl ▶  Received DockerNetwork &{BaseNetwork:{ID: HbVersion:0 ProjectID: Shared:false Metadata:map[] Name:new-network ExtID:} Scope: Driver: Gateway: Subnet:}
08:44:30 [DEBU] docker-dri -> PluginImpl ▶  Creating network [new-network_7168] with config {false bridge  false <nil> false false false false <nil> map[] map[]}
08:44:30 [INFO] docker-dri -> PluginImpl ▶  Created ob network [new-network_7168] with ext id [e8750ba5df89581cc41bb2da270bb25991d286755b8af68e14d9adf9079eba33]
09:32:22 [ERRO] docker-dri -> PluginImpl ▶  Error while retrieving the image by id
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x757ee6]

goroutine 125 [running]:
github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-docker-driver/handler.PluginImpl.ListServer(0xc42016fb00, 0x897ea0, 0xc420024018, 0xc4203236e0, 0x0, 0x8581b6, 0x2a, 0x793120, 0xc4203f62a0, 0x0, ...)
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-docker-driver/handler/handler.go:487 +0x806
github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk.(HandlerVim).ListServer-fm(0x793120, 0xc4203f62a0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk/worker.go:208 +0x46
reflect.Value.call(0x7bcb60, 0xc42011ece0, 0x13, 0x8496ae, 0x4, 0xc42011ed00, 0x1, 0x1, 0x0, 0x793120, ...)
	/usr/local/go/src/reflect/value.go:447 +0x969
reflect.Value.Call(0x7bcb60, 0xc42011ece0, 0x13, 0xc42011ed00, 0x1, 0x1, 0x0, 0xc420432000, 0xa)
	/usr/local/go/src/reflect/value.go:308 +0xa4
github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk.worker.handle(0xc42016fb00, 0x89bd60, 0xc4202cc680, 0x7edec0, 0xc4203023f0, 0x812d80, 0xc4202e4820, 0xc420432000, 0xa, 0xc4204465a0, ...)
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk/worker.go:86 +0xc2e
github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk.handlePluginRequest(0xc420392000, 0x265b, 0x2800, 0x83a2a0, 0xc4202cc680, 0x0, 0xc42034c140, 0x812d80, 0xc4202e4820, 0x7edec0, ...)
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/pluginsdk/handler.go:28 +0x2f9
github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/sdk.(*Manager).Serve.func1.1(0xc420320200, 0xc4200be640)
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/sdk/commonsdk.go:311 +0xb6
created by github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/sdk.(*Manager).Serve.func1
	/go/src/github.com/golang/openbaton/go-docker-driver/vendor/github.com/openbaton/go-openbaton/sdk/commonsdk.go:310 +0x10a

All my docker images are listed and I can run manually the mongo:lastest docker
I use those command to launch my environement :
docker run --name openbaton -d -p 8080:8080 -p 5672:5672 -p 15672:15672 -p 8443:8443 -e RABBITMQ_BROKERIP=<IP_ADDRESS> openbaton/standalone

docker run -v /var/run/docker.sock:/var/run/docker.sock -v /mypath/Openbaton:/root/ openbaton/vnfm-docker-go  -conf /root/config.toml

docker run -v /var/run/docker.sock:/var/run/docker.sock -v /mypath/Openbaton:/root/ openbaton/driver-docker-go  -conf /root/config.toml

with this config.toml
type        = "docker"
workers     = 10
username    = "openbaton-manager-user"
password    = "openbaton"
logLevel    = "DEBUG"
brokerIp    = "<IP_ADDRESS>"
brokerPort  = 5672

(I needed to increase the "workers" value to avoid having an error "JSON malformated" )
The vnfpakage files are the same I just change the image upload parameter to "false"
I created the NSD using the webui and just by selecting mongodb from the  VNFDs Catalogue menu

There is a nil pointer in the docker driver so this issue should be moved to the docker driver repo. i suppose from the logs, the problem is while retrieving the image id from the tag defined in the vnfd.json. Most probably because was never pulled in the docker engine. Although the pull should be done while uploading the package, please try to pull manually the mongodb image and try again, but this time using the right docker images. You do not need the full standalone openbaton, but the nfvo, vnfm and plugin.
you can use this compose file:
version: '3'
services:
  nfvo:
    image: openbaton/nfvo:latest
    depends_on:
      - rabbitmq_broker
      - nfvo_database
    restart: always
    environment:
      - NFVO_RABBIT_BROKERIP=rabbitmq_broker
      - NFVO_PLUGIN_INSTALLATION-DIR=/dev/null
      - NFVO_VIM_ACTIVE_CHECK=false
      - SPRING_RABBITMQ_HOST=rabbitmq_broker
      - SPRING_DATASOURCE_URL=jdbc:mysql://nfvo_database:3306/openbaton
      - SPRING_DATASOURCE_DRIVER-CLASS-NAME=org.mariadb.jdbc.Driver
      - SPRING_JPA_DATABASE-PLATFORM=org.hibernate.dialect.MySQLDialect
    ports:
      - "8080:8080"
  vnfm-docker:
    image: openbaton/vnfm-docker-go:latest
    depends_on:
      - nfvo
    restart: always
    command: -ip rabbitmq_broker
    volumes:
        - /var/run/docker.sock:/var/run/docker.sock:ro
  driver-docker:
    image: openbaton/driver-docker-go:latest
    depends_on:
      - nfvo
    restart: always
    command: -ip rabbitmq_broker
    volumes:
        - /var/run/docker.sock:/var/run/docker.sock:ro
  nfvo_database:
    image: mariadb
    environment:
      - MYSQL_RANDOM_ROOT_PASSWORD=true
      - MYSQL_DATABASE=openbaton
      - MYSQL_USER=admin
      - MYSQL_PASSWORD=changeme
  rabbitmq_broker:
    image: rabbitmq:3-management-alpine
    hostname: openbaton-rabbitmq
    environment:
      - RABBITMQ_DEFAULT_USER=admin
      - RABBITMQ_DEFAULT_PASS=openbaton
    ports:
      - "5672:5672"
      - "15672:15672"

the workers are completely unrelated with the json format issue you hit. would be good to have a way to replicate it

Thank you  for your help !
I had already pulled the lastest mongo docker before starting OpenBaton and the image is in the list with the same tag ( ["mongo:latest"] ) as the config file :
image:
    upload: "false"
    names:
        - "mongo:latest"
    link: "mongo:latest"
image-config:
    name: "mongo:latest"

Furthermore with this compose file it works perfectly (thank you again) and I use the same pop.json and the same .jar for my VNF package.
My problem is sloved, but if you think there is a real bug and it's not just a misconfiguration in the dockers I used to run OpenBaton I can pursue the investigation.
I will try to reproduce the misconfiguration json and open an issue if I can do it.

Hello,
I am following the same tutorial as you @JohanPy that is docker-tutorial .
I have already installed:

Openbaton (as a docker container) link here
Docker VIM driver link here
Docker VNFManager for OpenBaton link here
With this installed and launched I can access to openbaton web and list my docker images, networks....

My problem is that when I try to launch a NSD, this mongo db for example, the VNFM driver sais
[ERRO] docker-vnf -> handleMess ▶  ERROR: &{**Image with name or id** [mongo:latest] **not found** 0xc4200c78c0 88263e7d-eacd-4226-8cb9-0225e9aa1fd3}
Why not found? In the Point of Presence information it appears (see screen shot)
Do you have any idea about what could be happening? @JohanPy @lorenzotomasini @raj2569 @mcilloni
Thanks in advance.
Regards.

Hi @prom450 , Could you solve your problem ?I have a similar issue like yours. I created the issue here. I want to to run Iperf NSD tutorial in two different docker containers as in this tutorial. But, the VNFM says  [ERRO] docker-vnf -> handleMess ▶ ERROR: &{Image with name or id [iperfserver:latest] not found 0xc4200cf200 d0f4c0cc-ad9a-4a8e-81c5-630590db75a2}  although In the Point of Presence information the image appears. Thanks !


KfProducer.java:17: it for data consumption

The puzzle 30-6ca3f2f5 from #30 has to be resolved:
https://github.com/eo-cqrs/eo-kafka/blob/7a0719636e94595eab07e113a2031b87d9838e1e/src/main/java/org/eocqrs/kafka/KfProducer.java#L17-L17
The puzzle was created by @h1alexbel on 23-Feb-23.
Estimate: 60 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 30-6ca3f2f5 has disappeared from the source code, that's why I closed this issue.


Added Tensorflow Dataset example

Added a comprehensive guide to use Tensorflow Dataset colab example

Hey, @lamberta  I have contributed my first colab example. I'd love to hear feedbacks from you.

Hey, @MarkDaoust would you please review it.

Hey, I'm thinking about making a colab example for custom callbacks using Tensorflow 2.0.
Will it work and can I contribute here?

Hi @abhinavsp0730
There's a custom callback guide here: https://www.tensorflow.org/guide/keras/custom_callback
Feel free to raise an issue of submit a PR is you think something is misisng.
The community examples directory is really meant for things we don't intend to cover in the official guides and tutorials and would like to avoid duplication, where possible---since there are not enough reviewers to maintain both.

Could you add the license and colab/github buttons at the top?

Yes, I'm gonna  add the colab/github at the top.

Hey, @MarkDaoust  I'm going to put the link(github/colab) of the .ipnyb which present in my forked repo.


Error while installing packages

i am getting the below error when installing package. can someone guide me to solve this
**npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm ERR! Unexpected end of JSON input while parsing near '...5f3ca35761e47e05b206c'
npm ERR!      [E:\Users\tarun.atom.apm_logs\2021-07-05T07_44_18_428Z-debug.log]**
2021-07-05T07_44_18_428Z-debug.log

@TarunVuta Which package are you trying to install?

@sadick254 linter-eslint, atom-beautify, atom-ternJS etc. since it was failing through installer i tried to do it manually(PowerShell, npm). it worked but with a bunch of warnings.

@TarunVuta what node version are u using

@kkanekii v16.4.2

Try re installing


'duration' params didn't work in Tween.to methods

'duration' params didn't work in Tween.to methods but 'time' works well.

@yangjunjun  can u paste your code snippet？

You can see this demo

@picacure  sorry, I get this problem with the "hilo-standalone.js" file from Flappy Bird tutor which was  '0.0.1' version.  It's ok with the latest verson. so I will close this issue. and which you will update the tutor.

 Flappy Bird tutor  is updated to the latest verson

@06wj  thanks.


Vertical bar in regex rule

Can't use vertical bar in regex rule
Example:
$regex_rule = [ 'name' => 'regex:/(aaa|bbb)/'];

Result:
Fatal error: Uncaught Rakit\Validation\RuleNotFoundException: Validator 'bbb)/' is not registered

@MohammedElmsellem you have to use the array syntax for complex regexes instead of the inline string. For example:
$regex_rule = [
    'name' => [
        'regex' => '/(aaa|bbb)/',
    ]
];

@dave-redfern Another error occurred: Fatal error: Uncaught Rakit\Validation\RuleNotFoundException: Validator '/(aaa|bbb)/' is not registered.
I don't think it works like that, when you pass an array it take only values as list of rules.

Ah! Whoopsie, my bad ☺ Try instead:
[
    'name' => [
        'regex:/(aaa|bbb)/',
    ],
]

I just had to do this the other day myself.

@dave-redfern it works thanks 👍👍

Here is my working example
tobh is time of birth in hours
  "tobh" => [                 "required:true",                 "regex:/^([0-1]?[0-9]|2[0-3])$/"             ],


Verson 3.3.0: time.h declares mrb_time_at. Function changed to time_at (no mrb_)

The API function mrb_time_at was changed in time.c to time_at. The time.h file still has mrb_time_at.
Added
extern "C" {
  MRB_API mrb_value time_at(mrb_state *mrb, time_t sec, time_t usec, enum mrb_timezone zone);
}

as a temporary hack.

Ah, time_at is not a static function. 5265e35 was wrong.


⚠ ClamAV has degraded performance

In 714af39, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 96 ms

Resolved: ClamAV performance has improved in 4762c6a.


Connection required IP address

Currently, the connections between devices require an IP address.
We have interfaces that are either part of MLAGS or LAGS or are just plain layer2 links between switching and routers and thus do not have address space assigned.
Can we have a feature where we can make a device interface part of a LAG or LACP or just select that it's a layer2 trunk or access port with no address space assigned?
Cheers
Will

Currently, the connections between devices require an IP address.

You can connect interfaces regardless of whether they have an IP address assigned. The button with a plus sign adds an IP address; the button with two arrows facing each other connects interfaces.


Documentation fix

Adding some documentation

Codecov Report
Patch and project coverage have no change.

Comparison is base (6caa639) 85.03% compared to head (5a811f1) 85.03%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #488   +/-   ##
=======================================
  Coverage   85.03%   85.03%
=======================================
  Files         198      198
  Lines       26053    26053
=======================================
  Hits        22153    22153
  Misses       3900     3900

Files Changed
Coverage Δ

unified_planning/io/pddl_reader.py
79.00% <ø> (ø)

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


udf_schema should not track frequent items

In versions 1.2.0-1.2.2 of whylogs, udf_schema would include frequent items in core metrics when attaching UDFs.
This would change behavior in LangKit profiling with respect to logging frequent items when using llm_metrics.init() to wire in the udfs.
Suggested fix is to depend on whylogs 1.2.3+ with the fix for udf_schema

fixed in v0.0.6


Difference between Paneron extension template repository and registry extension template repository

Hi @strogonoff, this is HSU student who is working with ISO-19160 project.
As the documentation mentioned, we can choose to use Paneron or registry extension template repository. May I know what is the difference between them?
Also, may I know how to pass and save the data (e.g. YAML file) to Paneron? Can you please provide some example code for us?
Thank you sooo much!

There are two templates, basic and registry. The first one represents a basic extension, the other one represents an extension that works with ISO 19135-1 registry data.

There are two templates, basic and registry. The first one represents a basic extension, the other one represents an extension that works with ISO 19135-1 registry data.

@strogonoff in my understanding, paneron will handle the data passed by the extension, however, I can't find any documentation about the how can we pass the data to paneron, can we have some more information about it? thanks.

@strogonoff in my understanding, paneron will handle the data passed by the extension, however, I can't find any documentation about the how can we pass the data to paneron, can we have some more information about it? thanks.

@szehang you should use Registry Kit (the registry template), it provides you with functionality to manage the data structures and for tracking.

@szehang you should use Registry Kit (the registry template), it provides you with functionality to manage the data structures and for tracking.


Update fideslang to 1.1.0

Closes #824
Code Changes

[x] Include tags from fideslang==1.0.0
[x] update data_categories to new defaults from fideslang==1.1.0
[x] Update docs to match fideslang documentation (autogenerate these in the future or point directly to fideslang?)
[x] Update all tests to use the new default taxonomy as required
[x] Add sorting to the visualize endpoint to handle potential failures where a parent key doesn't exist yet

Steps to Confirm

[x] Run through commands utilizing the new taxonomy

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[x] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
This started as a test bed for fideslang#58.
Initially being unable to test this, the scope expanded to cover tests and documentation as well that are impacted by the change.
Not sure if there is a better way to reduce some of the difficulty in reviewing unfortunately :(

@allisonking @ssangervasi it looks like there may be some cypress tests or other areas on the front end that could be impacted by these fideslang changes, do we have an easy way of running the frontend tests via a nox command or anything to see what shakes out? I don't think I have triggered the checks within the PR yet

We don't have a nox command for running frontend tests yet, but the best thing to do might be to regenerate the openapi types on the frontend side. If those change, then that would trigger the CI run. If those don't change, then our frontend tests might not catch anything for just fideslang changes, since we don't have real end to end tests yet (we mock out our backend responses) 😬
Buuuut for now, to retrigger the openapi types:

cd clients/admin-ui
npm install
npm run openapi:generate
And that should directly modify the types which should be committed

Thank you @allisonking !! Will go through those tonight 🙌🏽

@SteveDMurphy @conceptualshark this is definitely another issue, but it doesn't make sense to me to maintain fideslang docs in fidesctl anymore now that fideslang has its own repo
Are y'all ok with me opening up a follow-on issue to remove the fideslang-related docs from fidesctl? Just one more place to forget to update 😁


Bugs are not being reported correctly

Description
Bug from sentry may not be being reported correctly. Investigate to check if this is the case and see why

Dupe


Create tinkerbell stack during cluster create

Signed-off-by: Abhinav Pandey<EMAIL_ADDRESS>Description of changes:
This PR does the following:

create the tinkerbell stack locally on the bootstrap cluster, with boots running on docker
uses this temporary tinkerbell stack to provision a workload cluster
moves the tinkerbell stack to this workload cluster

Testing (if applicable):
Ran create successfully and verified the workload cluster
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold

/lgtm


Minitng error on Asset Hub Kusama

What happened?
Two separate artists sent me this notification they got while mining on AssetHUb Kusama - production - on laptop not phone.

Please reproduce in steps
Mint NFT to an existing collection.
Expected Behavior
Mint without error
What browsers are you seeing the problem on?
No response
At which address did you encounter bug?
No response
Are you logged in?
None
Which wallet you are using?
No response
Screenshots
added
Relevant log output
No response
Payment link for reward
https://kodadot.xyz/ksm/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The first is telling that user does not have balance to mint (tx fee + fee + deposit)
The second is similar to

https://github.com/kodadot/nft-gallery/issues/7128

@JustLuuuu please provide artists addresses.

NFT already exist error: this guy with this collection: https://kodadot.xyz/ahk/collection/114 he managed to finish minting later that day. Still getting this error can be discouraging.
First error was this collection: https://kodadot.xyz/ahk/collection/115, I just sent her more funds

Thanks @JustLuuuu , ill check it out


Configutation management

How can we realize the configuration of the host and the individual apps?

What do you think about the latest addition to dockapp: setting-management?
Does it pass for a Configutation management?
One thing that is still remains to be done is storing/restoring settings into/from a file...  i will probably generate a bash script to be sourced by bash (and regenerated laster on).

I think the current implementation suffices for now.


Atari 800 configuration RG353M

Hi everyone,
thanks for all the work done in this project!
I have recently switched from RG351MP to RG353M and I have a problem with the configuration provided for the Atari 800 emulator (basically with the controls).
I am using latest updated version of ArkOS (12302022).
With the setup provided for RG351MP I was able to use both dpad and left analog as the joystick, also the common combination select+x to enter the Retroarch menu was possible.
With the RG353M provided config setup only left analog is working as joystick, the onscreen keyboard is reached via dpad down and what I've just figured out is that R3+X is now the combination for the Retroarch menu.
When I reset the controller configuration in the Retroarch settings it seems to be fine again. However, when I save the config my changes are reverted with the next start of any Atari 800 game.
In general it looks like that the Atari 800 emulator is the only one that comes with a predefined override of the controller configuration and perhaps something is preventing me from changing this manually.
Could anybody please assist?
Thanks in advance :-)

I have a fix coming for this soon.

Hi @christianhaitian, many thanks, sounds great!
I've just donated some money for your awesome work ;-)

This should be addressed with today's update. Thanks for your donation.


Feature/asb 926 change link email address

:tada: This PR is included in version 2.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Passing a URI to Zotero.File.getContentsAsync() is deprecated -- use Zotero.HTTP.request() instead

The use of Zotero.File.getContentsAsync() in the following file seems to cause the deprecation warning as mentioned in the title.
https://github.com/windingwind/zotero-plugin-toolkit/blob/3ac678de032b866c11f0412a74877e4796401949/src/managers/preferencePane.ts#L223

Thx. This is a known issue.
The uri of plugin resources can be a http uri or a local file uri, depending on how the plugin is loaded. This file API handles automatically.
If we don’t use this API, we’ll have to save a copy of it in the lib. Considering this API is only used in the Z6 compatibility code and there’s no evidence that it will be removed before Z6 comes to its end of support, it’s safe to continue using it.

Okay. Thanks for the explanation!


Fixed #25170 -- Made assertXMLEqual ignore leading and trailing whitespace

Made assertXMLEqual() and assertXMLNotEqual() ignore whitespaces outside of tags.

Buildbot, add to whitelist.

merged in 2085d8d5bc2625076983d3be14b30ea684c3f0d5, thanks!


how to generate code for dependent protos   ?  e.g., google types:  date/datetime

Is there a way to generate TS code when I use import "google/type/datetime.proto";  dependency?
Repo for reproduction: https://github.com/xmlking/entity-resolution

Proto File :https://github.com/xmlking/entity-resolution/blob/main/proto/entityapis/er/schema/entity/v1/entity.proto
Generated TS file: https://github.com/xmlking/entity-resolution/blob/main/gen/ts/er/schema/entity/v1/entity_pb.ts#L8-L10
Other TS plugins have options generate_dependencies to generate code for google types
  - remote: buf.build/timostamm/plugins/protobuf-ts:v2.4.0-1
    out: gen/ts
    opt:
      - generate_dependencies
      - ts_nocheck
      - eslint_disable

Hey Sumanth,
the generate_dependencies option would help in this case, but it is not a good solution overall because every plugin has to implement it. Instead, it should be solved in the compiler.
I see you are using the buf CLI, which does have a solution for this case:
      --include-imports        Also generate all imports except for Well-Known Types.

This will look up the imports, and ask the plugin to generate those files as well. Well-known types (like google.protobuf.Timestamp) will not be generated. They will be imported from @bufbuild/protobuf instead.
I'm closing this, but feel free to reopen in case I missed something.

Hi @xmlking sorry for the delayed response.  Your suggestion of allowing a more fine-grained way of generating imports and WKT is definitely valid.  I filed an issue against our CLI repo for handling this a bit better at the plugin level.  See https://github.com/bufbuild/buf/issues/1407 for more information.
Until then, unfortunately the immediate workaround I can suggest is to use separate templates and run buf generate once for each and use --include-imports for the JS one only.  Apologies for the inconvenience.


Use arm with python

Hi! I want to build and use openvino in mac os with m1 CPU. So, I use this instruction: https://github.com/openvinotoolkit/openvino_contrib/wiki/How-to-build-ARM-CPU-plugin. After building i run setupvars.sh in install_ie folder. But it is not founded python:
[setupvars.sh] WARNING: Can not find OpenVINO Python binaries by path /Users/rizhiy/Downloads/openvino_builds/install_ie/python
[setupvars.sh] WARNING: OpenVINO Python environment does not set properly
[setupvars.sh] OpenVINO environment initialized

Сan anyone help with this?

@andriyrizhiy the instruction you're referring to has been significantly changed recently.
Could you please try a new version of instructinon?
Be the way, python environment is required only if you need to develop python applications. If you develop or run C++ applications you may skip this warning.


Idnits generates false positive for figures with Unicode characters

I have draft that uses Unicode characters in drawing (https://www.ietf.org/archive/id/draft-smn-idr-inter-domain-ibgp-03.txt). Running this draft through https://author-tools.ietf.org/idnits results in 151 errors as follows:
"Line appears to be too long, but this could be caused by non-ascii characters in UTF-8 encoding"
Similar issue was opened in the past (https://github.com/ietf-tools/datatracker/issues/2574), which suppose to be fixed. However, false positive are still there.

@kszarkowicz, The new version of idnits is called idnits v3. It's still in alpha stage of the development.
It's available on author tools from https://author-tools.ietf.org/idnits3
Seems like this issue is not occurring in v3: https://author-tools.ietf.org/api/idnits3?url=https://www.ietf.org/archive/id/draft-smn-idr-inter-domain-ibgp-03.txt
BTW idnits v3 works better with xml files.
There are few unrelated errors when you submit an XML file: https://author-tools.ietf.org/api/idnits3?url=https://www.ietf.org/archive/id/draft-smn-idr-inter-domain-ibgp-03.xml

OK. That is good. When idnits3 is expected to be officially released and used during shepherd's reviews?

It will still be some months before that is generally available. In the meantime, you the warning is not a blocking warning.
(Modifying the older idnits is quite difficult - it's a collection of sed-like things in a bash script, and character encoding nuance may be beyond it).

OK. Let's close this issue.


Add explicit Cache-Control headers

See https://github.com/rust-lang-nursery/rustup.rs/issues/1358

@alexcrichton S3 metadata are used by CloudFront to send the header.
If possible I would suggest to monitor the impact of that on the next nightly build.
One of my concern is that I am wondering if CloudFront will reread the files the S3 bucket for each request.  I would expect CloudFront to have tight integration with S3 but that's not what Amazon's documentation suggests.

Thanks! To confirm these cache headers indicate that the contents should not be cached, right?
If so, I'd actually imagine that all archived content could be pretty heavily cached as it in theory never changes!

Exactly, that's why for archiving I set the Control-Cache to public, which basically means "Cache all the time for everyone".

Oh! Sorry I misread that, looks great to me, thanks so much!

As a side note, I don't think S3/ Cloudfront is a good long term strategy and I think we should think about building an API. We could still use S3/Cloufront to store/send files but not to determine appropriate versions etc...
Several reasons for that:

Right now theres some logic duplication where rustup needs to know about S3. An api could abstract that away, minimising how coupled things are and allowing to change backend, optimising cache, etc.
Invalidating Cloudfront is known to be flakey. (I had bad experience myself and I have heard several time the same feedback). You can potentially ends up with really weird issues like a stable,toml being invalidate but not stable.toml.sha256, resulting in rustup thinking it got a corrupted download.
You could a LOT of optimizations with an API that you can't do at the moment. For example you wouldn't need stable to be a folder, just a soft link, avoiding copying distributions; You could dynamically change what are the recommended default components; etc.

@thibaultdelor heh I'd definitely believe it! We've mostly optimized for our own sanity (aka simplicity) historically rather than other aspects, but I can certainly see that shifting as Rust is much larger than when rustup was originally written!


Derive Debug for Schema and SchemaInner

Wouldn't it be more helpful to rely on a prettified json serialization?

Yeah, probably. I mainly added to have some representation at all.


fix(cloudevent): adjust cloudevents resource scope

Change cloudevents resource scope from system to project.
这个 PR 实现什么功能/修复什么问题:
修正：更正cloudevent的资源范围从system改为project

是否需要 backport 到之前的 release 分支:

release/3.6

/cc @zexi
/area cloudevents

Codecov Report

Merging #9652 (7daa123) into master (ef30d23) will increase coverage by 2.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9652       +/-   ##
==========================================
+ Coverage    6.00%   8.08%    +2.07%
==========================================
  Files         845     107      -738
  Lines      131233   12955   -118278
==========================================
- Hits         7879    1047     -6832
+ Misses     122653   11788   -110865
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.08% <ø> (+2.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/keystone/models/passwords.go

pkg/hostman/hostinfo/hostinfo.go

pkg/monitor/models/metric_field.go

pkg/monitor/tsdb/query_endpoint.go

pkg/compute/models/cloudproviders.go

pkg/util/rbacutils/policyset.go

pkg/baremetal/tasks/resetbmc.go

pkg/util/printutils/printgetter.go

pkg/util/stringutils2/stringutils.go

pkg/compute/models/guest_actions.go

... and 728 more

Continue to review full report at Codecov.

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


[Question] Documenting exceptions

Hello,
this is more of a question than an issue report, but we are working on a library using (python) SimpleITK and it seems that there is a general lack of documentation regarding raised exceptions in the (python) API. What is the status in this regard?

Hello @akiross,
SimpleITK is a C++ library on top of ITK. The core API documentation is thus for C++, published using doxygen and relying on the ITK documentation.  The SimpleITK documentation uses the core documentation to provide language specific conventions, e.g. docstrings in Python, and similar functionality in all eight languages SimpleITK is wrapped for. The ITK documentation does not include exception information.
Unfortunately, at this time we don't have the resources to update all of the ITK documentation (major task). Not out of the question that we look into it sometime in the future.
As ITK and SimpleITK are open source projects we always welcome contributions from the community, so if you are willing to improve the ITK documentation by documenting the exceptions that would be a great contribution to the community, both for ITK and SimpleITK users.


Submit button alignment fixed

_ No description provided. _

Looks good , I am merging it , Thank you @Rishav13773 for your contribution


Serialization of Lists of Strings

I found the existing behavior when serializing length one strings to be surprising.
s = Series(["One","Two","3"])
s.meta = MetaData(qtype=QSTRING_LIST)

k.sync('type each',s)
    0    10
    1    10
    2   -10
    dtype: int1

k.sync('{x like "Two"}', s)
    `type

I realize that QSTRING_LIST == QGENERIC_LIST but as far as I can tell there is no way to serialize a list of strings without the risk of some ending up as characters.
This fork contains a pretty straightforward change - QWriter will now always serialize str as a QSTRING regardless of its length.
The bytes type retains old behaviour so a bytestring will serialize as a QSTRING while a length-one bytes will serialize to the QCHAR type.  This change is carried over to QReader, so QSTRING is converted to str and QCHAR is converted to bytes.
I'm not sure if this is the best approach to the problem, or if other people even consider it an issue but I figured I'd toss it out as an option.

Thanks for pointing this out. This default encoding is based on behavior of q console, e.g.:
q)type each ("one";"two";"3")
10 10 -10h

I agree that this might become surprising and limiting in some cases. In order to minimize impact on existing code base, I would propose a different approach based on parser configuration mechanism. The QConnection would accept a single_char_strings argument which would control the encoding for single character length scripts:
q = qconnection.QConnection(host='localhost', port=5000, single_char_strings = True)
q.open()

s = pandas.Series(["One","Two","3"])
s.meta = MetaData(qtype=QSTRING_LIST)

r = q('{[x] d::x; type each x}', s)
print (r)

s = qlist(['One', 'Two', '3'], qtype = QSTRING_LIST)
r = q('{[x] n::x; type each x}', s, single_char_strings = True)
print (r)

s = ['One', 'Two', '3']
r = q('{[x] p::x; type each x}', s)
print (r)

q.close()

Would yield:
[10 10 10]
[10 10 10]
[10 10 10]

Does this solution match your use case?

That would probably be a better option.  It would work well for me.

The enhancement has been applied to master and 1.1 branch.
Check the documentation for 1.1 for details.


Support knowledge_base views generation.

One of my favorite features of devise is that it lets you generate views for your own customization.
They do it by running the command:
rails generate devise:views

It would be great to have
rails generate knowledge_base:views

This is no longer an issue for me. Closing. Feel free to make a new issue if this is going to be addressed in the future.


"Internal server error" upon password reset attempt

I just made my first attempt at logging into LW 2.0, and after some trial and error realized that I would need to reset my password from LW 1. So I attempted to do so, using the email associated with my LW 1 account. Rather than getting a password reset, I got the following error:

I don't know much about what's causing the error, but I'm pretty sure it's not a problem with my browser, because the same error occurred both in Firefox and in Chrome.

We fixed a bunch of bugs related to this, so I think it should be fixed. Happy to reopen if it's still an issue.


Dead links in fabric readme: https://github.com/hyperledger/fabric/blob/master/README.md

Writing Chaincode

... Follow the instructions on the Sandbox Setup page, which allows you to run your chaincode off the blockchain. <== sandbox set up points to https://github.com/hyperledger/fabric/blob/master/docs/api/SandboxSetup.md which produces error 404

Working with CLI, REST, and Node.js

When you are ready to start interacting with the peer node through the available APIs and packages, follow the instructions on the API Documentation page.  << API Documentation page points to https://github.com/hyperledger/fabric/blob/master/docs/api/CoreAPI.md  which produces error 404.

links work now

links are not working for me, 404 is still there.

The disconnect on this issue might be that the title references the README file, but the cited problems are in files in /API ... /dev-setup also has some issues, I'm looking at these now.

Also finding a number of references to "OPENCHAIN_" and obc-peer, various .yaml files that appear to have been renamed. So potentially more to do than just correcting the links.

The problem with some of the /API links appears to be the old directory was /api and the new one is /API -- so links that haven't been updated with capital /API are not found. I'd like to check with the dev. on which change they'd like to make - directory name or links.

Anya confirms /API - so all links with /api need to be changed to /API. An example of where link URLs defined in one place, as single common entities, would really help; GitHub doesn't support this, to my knowledge.

docs have been refactored. closing this for now as no longer relevant,


Update licenses for 27 casks

Three minor notes:

little-snitch was committed as :commercial.  Their site states:

Little Snitch offers a free, built-in demo mode that provides the same protection and functionality
as the full version. The demo runs for three hours, and it can be restarted as often as you like.
The Network Monitor expires after 30 days.

I don't consider that to be enough consistent functionality to qualify as :freemium but I recognize that someone might think so.

seamonkey was stylized as :oss as it is tri-licensed under MPL, GPL, and LGPL.  See link in commit and http://www.seamonkey-project.org/legal/ for more info
The murasaki available to us is :gratis, it's just the MAS version that's commercial.

Thank you, as always.


Added unification support for operator application

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

closes #2618

Codecov Report

Merging #2620 (dba43ec) into main (dd5fff8) will increase coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Current head dba43ec differs from pull request most recent head 5ab6245. Consider uploading reports for the commit 5ab6245 to get more accurate results

@@            Coverage Diff             @@
##             main    #2620      +/-   ##
==========================================
+ Coverage   78.48%   78.50%   +0.01%
==========================================
  Files         445      445
  Lines       15645    15658      +13
  Branches     2543     2575      +32
==========================================
+ Hits        12279    12292      +13
  Misses       3366     3366

Impacted Files
Coverage Δ

...alache/tla/typecomp/unsafe/UnsafeBaseBuilder.scala
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


AudioFrame.copyfrom() should move the internal used_size marker

Using the latest version (at the time of writing) of the recording & playback branch: https://github.com/microbit-foundation/micropython-microbit-v2/commit/0b06914c71c18533da90df85230ac198578669bf
Hex: https://github.com/microbit-foundation/micropython-microbit-v2/actions/runs/8416237764?pr=163
>>> a = microphone.record(100)
>>> b = audio.AudioFrame(100)
>>> b.copyfrom(a)
>>> # This plays a short audio
>>> audio.play(a)
>>> # This does not
>>> audio.play(b)
>>> # And the  sound data is there
>>> for i in range(len(a)):
...     assert a[i] == b[i]
...
>>>

If we manually edit the last byte in b it then does play a short clip:
>>> b[len(b) - 1] = 255
>>> audio.play(b)
>>>

Copying comment https://github.com/microbit-foundation/micropython-microbit-v2/pull/163#issuecomment-2068519700 here:
This needs discussion. copyfrom() actually allows copying from any object with the buffer protocol (eg bytes, memoryview) and they won't necessarily have the used_size attribute.
This method should probably just set used_size to the total bytes copied fro the buffer-like object. The only issue there is copying from another AudioFrame it will copy the total allocated size, not the used_size, of the other AudioFrame.

Yes, I think AudioFrame.copyfrom() should set used_size to to the last byte copied from the input argument (any buffer-like object).
So, if we start with a long AudioFrame that has data all the way to the end, and then we copy a short buffer into the AudioFrame, then we would end up with the used_size marker at length of the small buffer.
But I think this makes sense only if we also provide accessors to the used_marker (https://github.com/microbit-foundation/micropython-microbit-v2/issues/196), otherwise there is "magic" that cannot be changed without odd workarounds (like assigning to itself the last indexable value, e.g audio_frame[len(audio_frame) - 1] = audio_frame[len(audio_frame) - 1] vs audio_frame.set_marker(len(audio_frame))).
The question would be, do we special-case copying from another AudioFrame?
We'd have a few options, for all of these we can consider the following set up code:
af_long = audio.AudioFrame(duration=5000)
af_short = audio.AudioFrame(duration=2000)
microphone.record_into(af_short, wait=False)
sleep(1000)    # Record for only 1 second, af_short is only half filled
microphone.stop_recording()

The AudioFrame is fully copied, including the used_size value

af_long.copyfrom(af_short)
audio.play(af_long) # Plays for 1 second

The AudioFrame is fully copied, the used_marker placed at size of the input argument

af_long.copyfrom(af_short)
audio.play(af_long) # Plays for 2 second, 1st second audio data, 2nd second empty
af_long.set_position(af_short.get_position())
audio.play(af_long) # Plays for 1 second

The AudioFrame is only copied until the used_marker

af_long.copyfrom(af_short)
audio.play(af_long) # Plays for 1 second

In my opinion, for a user option 1. and 3. make most sense for easy-of-use. Option 2. is not too bad, as it makes sense if you already know about the marker, however I think it's a lot more likely that the users would want 1 second playback instead of 2, so making that default makes sense.
From options 1. and 3. , I'm a lot more inclined on option 1., because option 3. limiting how much data is copied doesn't feel quite right.

This might also need to be discussed with https://github.com/microbit-foundation/micropython-microbit-v2/issues/194 in mind, as one of the suggestions is to add an extra argument to have AudioFrame.copyfrom(buffer, index).
af_long = AudioFrame(duration=5000)
af_short = AudioFrame(duration=2000)
while True:
    if button_a.is_pressed():
        microphone.record_into(af_short, wait=False)
        while button_a.is_pressed():
            pass
        microphone.stop()
        af_long.copyfrom(af_short, index=af_long.get_position())
    if button_b.is_pressed():
        audio.play(large_af)
    sleep(200)

So in these cases we would need to make sure that when calculating the user_size position, it also needs to be offset from the index point (from AudioFrame.copyfrom(buffer, index)).

We've changed the implementation and the new audio objects do not have an "internal marker":

https://github.com/microbit-foundation/micropython-microbit-v2/issues/205#issuecomment-2221716206


Add Maven Publish Action

GitHub Action for automatically publishing Maven packages

@samuelmeuli your changes have been merged, thanks for your contribution 👍

Thanks @gep13!


How to allow microphone permission in neutralino js -react app

I am creating a calling desktop app for which I require microphone permission before call gets connected.

@rakeshkk16  since its a browser running, you can use navigate to get access to the permission Use this link for reference

Should the microphone permission be an alert message droping down?


Add orderBy to meetingNotes query so action items show up last

Proposed changes
Changes the Meeting Notes query to order agenda items with agenda_owning_participant_id asnull last.
Resolves #316.

Checklist

[X] Are the issues being addressed linked to this PR?
[X] Do all commit messages start with the issue number?
[X] Are all code changes sufficiently tested?
[X] Are there screenshots for UI changes?

Why agenda owning participant id is not in backticks?

The change works without the back-ticks.


[Sync status] Improve contents and layout of sync status

@coreyleamon I hacked together a sync status screen, but it's just the barebones of what's needed to check the sync status for features and observations. Would you like to have a go at a proper design for this interface? The known follow-ups at the moment are:

[ ] Show icon for each status (PENDING, IN_PROGRESS, COMPLETE, FAILED)
[ ] Show related feature caption/name on each line if available (@scolsen)
[ ] Add date/time completed/failed
[ ] Add number of retries
[ ] i18n: Make text translatable
[ ] Create custom layout instead of using multiline string

Screenshot of current UI:

@kdyson FYI

Is this a status screen that conveys the status of uploading data once connected to a network? If so, I have a few questions:

Is it important for these data lines to be viewed together in a screen, which disassociates them from the geographical representation of the point or polygon?
Does the user need to manually re-initiate when there is a failure, or could the system make x attempts before being considered "failed"?
If a data point continues failing to upload, what action can a user take? Will they need to re-submit/recreate the data point?
Do uploaded data points need to be archived in any way? How long and in what contexts might the user need to maintain visibility of the status even if successful?

Hi @coreyleamon sorry I missed these:

Is this a status screen that conveys the status of uploading data once connected to a network? If so, I have a few questions:

Yes, that's correct.

Is it important for these data lines to be viewed together in a screen, which disassociates them from the geographical representation of the point or polygon?

I think both may be useful. This list is useful because data collectors want to know which changes were recently made without needing to search for possibly modified features on the map.

Does the user need to manually re-initiate when there is a failure, or could the system make x attempts before being considered "failed"?

It already does this, we just don't show it yet in the UI. But after X attempts we don't have a well-defined story for how the user either retries or abandons the change.

If a data point continues failing to upload, what action can a user take? Will they need to re-submit/recreate the data point?

Currently none; see previous point.

Do uploaded data points need to be archived in any way? How long and in what contexts might the user need to maintain visibility of the status even if successful?

Good q - one option would be a CTA to clear completed items from the list. Wdyt?

@ngosue FYI

@scolsen FYI, latest designs:

@gino-m the only outstanding tasks left in this are number of retries (which may not be in the mocks anymore), internationalization, and the custom layout
WDYT about filing new issues for these outstanding tasks, for task management before IDF announcement? Particularly, custom layout can be filed as code health, i18n as a new FR (are there additional i18n tasks? Is i18n blocking for IDF announcement?), and potentially dropping the retries if the mocks don't have them anymore.

@jcqli  The current impl still shows the enum keys instead of label strings, and the job name doesn't show as a formatted string (just the ID shows). Separate tasks sgtm! Can you create? @scolsen Any chance you could patch these today?

https://github.com/google/ground-android/issues/2319 for translations
https://github.com/google/ground-android/issues/2320 for custom layout
Unclear what the number of retries should look like so we can file that later as the need arises

@scolsen Can we just show user readable labels for the status instead of constants?

@shobhitagarwal1612 Would you be able to help close out this issue with the UI polish still needed?

Sure. I can take this one up.


Resumable retriable connection for Bootstrap

@tombentley , is there a way I could convince you to add the resumability and retriability you implemented for the CMR to the Bootstrap class?
There's only one "but": that bootstrap jar must stay as small as possible (it's only 20KB right now), so nothing fancy, quick and dirty is good here. But it would be nice to have because the distributions are getting close to 20MB, so on connections like yours that's bound to give trouble)
The code that does the downloading is pretty small and can be found here: https://github.com/ceylon/ceylon/blob/master/compiler-java/src/com/redhat/ceylon/launcher/Bootstrap.java#L552

Using something similar to the RetryingSizedInputStream I used in the CMR the ceylon-bootstrap.jar comes out at 26k. Do you think this is acceptable?

Sure, it's still small. We can always see if we can shave bits off later.

PS: if you make it an inner class of the Bootstrap you don't have to worry about fixing the build, it will pick up anything called Bootstrap$

Pushed, but untested. The one thing this doesn't cope with properly if the remote file gets changed during a retry. Does the script always request a specific version of the distribution (i.e. the remote file never changes) or is the remote file always just the most recent version of ceylon (and thus changes every time we release a new version)?

It always requests a specific version yes

Hmm, I wonder how we test this? It would be easiest if Bootstraps members weren't all static, then I could at least override the URL to point to a test http server. Or are there other tests for it @quintesse?

Well the obvious way would be to make things non-static. The current "design" if you can call it that was just to make it work quickly following what was already there.

Oh, suddenly after my first coffee I realized that you can specify the distribution it has to download, and therefore the server it connects to, yourself in the ceylon-bootstrap.properties file ^^
We'd want that anyway for any tests because we don't want a test polluting a user's home directory.
So the properties file could be something like:
distribution=http://localhost:9090/dist.zip
installation=../test-dists

Thanks @tombentley !


Overlay bug

When an overlay is active, the timing of the video in PGM shows "overlay ..." and not the timing of the video.

Related to https://github.com/jensstigaard/vinproma/issues/11


dbt Tasks got randomly killed (sigterm) in Cloud Composer

Describe the bug
The issue is tasks in Cloud Composer got randomly killed (sigterm).
Logging query to show the errors:
resource.type="cloud_composer_environment" resource.labels.location="" resource.labels.environment_name="" timestamp>="2021-09-25T19:00:45.396Z" severity>=ERROR " Received SIGTERM. Terminating subprocesses."
Steps To Reproduce

Create a  Cloud Composer environment
install https://pypi.org/project/dbt/ and run dtbt task

Expected behavior
No error like "ERROR - Received SIGTERM. Terminating subprocesses." received
Screenshots and log output
NA
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
<output goes here>

The operating system you're using:
The output of python --version:
Additional context
Add any other context about the problem here.

@stevenzhang-support It looks like that error and the associated log message are coming from Cloud Composer, not from dbt Core. Is that right?
Given that, I'm not sure how I can help here. It looks like this error message could be the result of insufficient memory, or of subprocesses being cleaned up too quickly (especially if they're long-running):

https://groups.google.com/g/cloud-composer-discuss/c/nhMC0AhqugA?pli=1
https://stackoverflow.com/questions/54313601/airflow-signals-sigterm-to-subprocesses-unexpectedly

I'm going to close this issue, since I don't believe the resolution for this bug will come in the form of a code change to dbt-core. I'd be happy to reopen if you're able to identify a particular behavior of dbt-core that's not working as it should, or not working the way you expect, thereby making it difficult to deploy via Cloud Composer.

thanks


Unable To Change Mode On Zwave Thermostat Via Google Assistant

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.65.4
Python release (python3 --version):
3.6.3
Component/platform:
zwave
google_assistant
Description of problem:
I am  unable to ask google assistant to set the thermostat to Heat or Cool or even turn it off.  I am able to query the device and see if it is set to Heat, Cool or off.
Expected:
I should be able to turn on the heater or cooler
Problem-relevant configuration.yaml entries and steps to reproduce:

Install latest version of home assistant
Setup zwave with a Honeywell zwave thermostat
Setup google assistant with the thermostat
Sync the device
Ask "Hey google is the thermostat on?"
Observe proper response
Say "Hey google turn on the heater."

Google responds with that mode isnt available for the thermostat. However it will see the heat or cool modes are on if I turn them on manually.
Traceback (if applicable):
n/a
Additional info:

Same Problem appears now with homematic thermostats. It worked before.

I've hid your screenshots as they consume too much vertical space.
Homematic too does not use the STATE_ constants as defined by the climate component.

Possibly related to this I've found that the EphEmber climate device is no longer appearing in google assistant. After upgrading to 0.65.3 I did a resync and they disappeared from the home app.
I see no errors in the logs.

There is indeed a hard requirement on SUPPORT_OPERATION_MODE. Without it, code should not call the set operation mode service (because it's not supported).
If Ephember supports set operation mode, it should add the support flag.

But if you want your climate device to be read only and still report the state and temperature then this isn't possible now. Previously I could query the state of the device without being able to change it. I think a change state was just a empty call that did nothing but am not 100% certain on that.

I think we need better documentation of how components of a certain domain have to be designed to work as expected and to not break in other places if something unrelated gets changed. I also think, that we might need some type of subsystem maintainer who can overlook new code and changes of a certain domain to keep it in line with the bigger picture.

@ttroy50 you can mimic this with the generic thermostat.
@janLo this was not something unrelated changing, we did change Google Assistant. We made Google Assistant follow our exact implementation. I did a call for subsystem maintainers but did not get a lot of response. We're open source and we got to play with the cards we're dealt. Feel free to open a PR for Homematic to address the current operation issue.

@balloob That doesn't make much sense to me. You then have to configure each thermostat twice. Once in the main climate component and once in the generic thermostat component.
Different thermostats may support different things and I think we should be aiming to expose any available functionality when we can. Preventing reading / setting the temperature because it can't change on / off in operation mode seems to be making a hard line in the wrong place.

Google Assistant is built around traits. Currently Google Assistant is limited to reporting temperature on devices that implement the TemperatureSetting trait. This trait requires the thermostat to support setting mode and temperature. If you want to deviate from this, feel free to run an adjusted Google Assistant component locally. Home Assistant won't deviate in it's integration from any of Googles requirements.
Nest launched temperature sensors for rooms today so I would not be surprised if some sort of temperature sensor trait would be made available at some point.

Didn't see any fixes come in but just wanted to mention the issue still exists in 0.66

This also seems like an issue with HomeKit. I can update setpoint (temperature) but mode changes don't send or display.

I'm in support of consistent code, even at the cost of functionality but I don't really understand this change since voice control seems a very common use of controlling climate. My limited understanding of the thread so far is that the homeassistant system maintainers have passed the requirement for a fix to the z-wave component maintainers with no response. From an integration view it seems unusual to force a change that way, but not something I can help fix. 2 options then:

Possibly use Generic thermostat. Don't think that will work for me since trying to control the local setting and the HA commands will cause conflicts.

As balloob suggested "If you want to deviate from this, feel free to run an adjusted Google Assistant component locally". I tried this:
a) Download an older version of the Google Assistant component from Git (10 Feb 2018)
b) Create a custom_components folder in config and drop the Git folder there renamed "google_assistant_old".
c) change "google_assistant" to "google_assistant_old" in configuration.yaml and restart.

No luck so far. I assume the references to "google_assistant" might need to be changed to "google_assistant_old" in the individual files? Since it's a set of files rather than one .py it's hard to see the structure. Any other suggestions? Thanks

I think the main problem is, that home assistant is "move fast and break things". The new release model is a step in the right direction but before it has found maintainers for its subsystems it will not get any better. The Homematic and zwave climate components got introduced with the climate component itself and get still blamed that they do not implement the methods the the climate component expect. I doubt that anyone in mentioned components is even aware of the necessary changes that came at some point. And so I doubt it will get fixed by them.
I would expect that the change that broke things (Google assistant change) would get reverted for a stable version until the fixes are made and applied in the broken components.

You would imagine a popular component like zwave (that has its own discord channel) would have a device maintainer who would get notified about this.  If I knew who it was I would tag them in here but I don't and don't see a codeowner listed either so its hard to imagine how they get notified about changes like this.

@janLo you're wrong. The climate component used to be the thermostat component. Z-Wave was not at the beginning of the thermostat component, the component got introduced together with the Nest integration.
If no developer is stepping up to fix a broken component, I can only assume that it's not that big of a deal? It's easy to post here in the comments and hoping other people spend their time to fix your problems, but last time I checked, that's not how open source works.

If no developer is stepping up to fix a broken component, I can only assume that it's not that big of a deal?

Thats probably because the component that is broken is the google-assistant component. Sorry, but from a users perspective its clearly a regression in this component. That its caused by implementation details of another component is just a detail. A proper solution would have been to hold the change in the google-assistant back until the issues in the components it depends on are fixed.

The Google Assistant component is made to work with the Home Assistant abstraction of climate devices. It's not broken, it works exactly as intended. Last comment I am going to make in this thread.

@andrey-git I saw several PR's and zwave bug fixes come from you lately, are you considered the zwave codeowner?  If so just wanted to make sure this bug was on your radar :) if not apologies for tagging you.

I am a zwave codeowner, but I don't have a zwave climate device, so I doubt I'll get to checking this integration.

@audity-git can I donate one? I have a CT100 🙂

I have a climate device, but not Google home.

I do think our best bet is going to be implementing this once if we can get it PR'd, the CT100 Thermos are a bit awkward anyway because of the multiple climate devices it creates. https://github.com/home-assistant/home-assistant/pull/11472

A similar issue was reported in the forum: https://community.home-assistant.io/t/homekit-remote-access/57532
On the very basic level it seems like the ZWave operation modes are mostly/all capitalized whereas Home Assistant expects lowercase. Although this isn't an issue for the frontend, since we render the Operation Modes from the list, it breaks all secondary components, like Google Home, Amazon Alexa or HomeKit (and the frontend translations).

I can confirm that. I modified the climate/zwave to return lower case for both "state" and "operation modes" properties. (i know this is a hack, but wanted to try it).
Now it seems i can use the Alexa app on the phone and I can control the thermostat temperature through the app. It is functional. I can also ask "Alexa, set thermostat to 75", and she sets it. However I get a response from her "Thermostat is not responding" .
So setting up lowercase, kind of works, but i think something else is missing in addition to lower case.
Does anyone know what type of confirmation is Alexa expecting, so she confirms the operation with "ok"?

@kdjordjev The isssue is a bit more complex. Whereas Home Assistant wants to use lowercases, ZWave expects uppercases. That's the reason why your thermostat was not responding.
The solution I could think of would be some kind of mapping dictionary: HA values -> list of possible Z-Wave values. That would also take care of some special cases, e.g. Heat (Default) instead of Heat.
Unfortunately I don't have a Z-Wave climate device, so fixing it myself would be quite difficult.

I don't have a Z-Wave climate or google home device myself.
But anyways, are these the mode strings we're seeing from the device? https://github.com/OpenZWave/open-zwave/blob/0f8736bd7d77aaf97f4748a7fa65ada4dd487da8/cpp/src/command_classes/ThermostatMode.cpp#L51-L68
As long as they match up to what we're seeing we should be able to just use that list to make a mapping to normalized hass modes.

@armills These are the values that I see for my thermostat.  I only use Off Heat and Cool though, the others are not used on my thermostat.  I think it depends on the installation done for programming.  I have a honeywell zwave thermostat TH8320ZW

@cdce8p
I wanted to clarify.. I did a hack on climate/zwave device level, not within zwave stack.
The communication between the device and zwave has not been changed and is as before.
The only change is that i did mapping of the climate/zwave device properties to the required STATE_ names, and returned the state of the device.
Everything with home assistant is functional, and Alexa is recognizing the device, i can control it through the Alexa app. The only issue is that she responds with "Not Responding" (rather than "Ok", while actually she changed the temperature.
In any case... I understand this is not the right way to do it, but i am not familiar with the zwave. I hope this helps

state_mapping = {"Heat":STATE_HEAT, "Cool":STATE_COOL,"Auto":STATE_AUTO,"Off":STATE_OFF,"Cool
 Econ":STATE_COOL,"Heat Econ":STATE_HEAT}

@property
def state(self):
    """Return the current state."""
    return self.current_operation

@property
def current_operation(self):
    """Return current operation."""
    return state_mapping[self._current_operation]

@property
def operation_list(self):
    """List of available operation modes."""
    if self._operation_list:
        return [state_mapping[k] for k in self._operation_list]
    else:
        return None

@kdjordjev The solution you provide is working like you say. But there are other modes that are in use with zwave, but I'm not really sure they do other things than heat, cool, auto or idle. Maybe we can map them to what wwe think they might do and use that as a starter, and adjust if someone has another functionality.
Is this also a working fix for google assistant?
What climate component are working with these two platforms, maybe we can find the "Not responing" fix in them.

I think it's OK to also include the modes that don't correspond to standard hass modes, but for the ones that do we need to make sure we're using the standard hass constants.

@kdjordjev I think you might need to reverse the mapping in set_operation_mode, since otherwise the ZWave device doesn't know what to do with the lowercase modes.
@armills Although the list seems to be the official attributes, I have already seen some different ones as well: https://github.com/OpenZWave/open-zwave/blob/0f8736bd7d77aaf97f4748a7fa65ada4dd487da8/config/thermofloor/heatit021.xml#L5-L11
I agree that we should only touch those modes that have directly correspond a HA one.
All: I can prepare a PR with what I think might work. Would just need someone who is then able to test and verify the changes.

@cdce8p I would be more than happy to test :)

I am not familiar with all modes, so my mappings above reflect what my thermostat supported.
I have 2GIG CT80
Thank you @cdce8p . I think not mapping back  set_operation_mode might be my issue.
In any case I will also try to test your changes

FYI, my thermostat (Linear GC-TBZ48) shows Auto, Off, Aux Heat, Heat, and Cool as the modes listed under Operation.

@dshokouhi, @kdjordjev Who's up for some testing? I just opened the PR.
The Link to the raw file: download. It's a first draft, but, unless I missed something, should work.

Thanks to @dshokouhi for debugging and testing! My PR works.
If you want to try yourself:

Inside the config folder create a dictionary custom_components.
Inside this folder, create climate and then zwave.
Download the new file (link) and place it inside the zwave folder. The path should look like this: .homeassistant/custom_components/climate/zwave/zwave.py
Restart

I will test it tonight with Alexa
Can you also add the following mappings
"Cool Econ":STATE_COOL,"Heat Econ":STATE_HEAT

@kdjordjev Unfortunately that won't be possible. My fix is desinged for a 1:1 mapping and Cool is already mapped to STATE_COOL.
The only options are to either wait for the redesign of the climate component or change the mapping dict manually in your custom_components folder, although you would remove mapping for Cool then.

To report on the test:
The Alexa behavior is the same as before.

HA working
Alexa App is functional
Alexa replies with: "Hm, Heating Thermostat is not responding", but sets the T as requested

Interesting in my testing Google Assistant responded as expected "changing thermostat to cool" and it changed the mode accordingly

@kdjordjev Can you open a new issue for that? That needs to be solved independently from this one.

I just merged the fix onto the dev branch, so it will be included in the beta for 0.73.
@Everyone
Remember to delete the file in the custom_components folder to receive future updates.

I just want to report that the uppercase/lowercase problem on google assistant component (cloud in my case) exists on MQTT HVAC climate devices too and it was causing resposes like "  does not support this mode" ("La modalità non è disponibile per ").
In my configuration.yaml i had to change all modes to lower case and handle them on my node-red instance to revert them to uppercase.

Hi, any solution for google home? I tried to use eq3 Buetooth Thermostat with google assistant, but it replies "can't set this temperature for this device", alexa works

@Krid86a - I have the same issue.
Appropriate issue ( https://github.com/home-assistant/home-assistant/issues/13712 ) was closed and marked as duplicate with this issue.
As @awarecan wrote in this issue (hiden comments): https://github.com/home-assistant/home-assistant/issues/17302 ,eq3btsmart.py do not support STATE_COOL, STATE_HEAT which are supported by google assistant:
https://github.com/home-assistant/home-assistant/blob/3f87d413813de84935ea67b5212c55348524447f/homeassistant/components/climate/eq3btsmart.py#L64-L71
https://github.com/home-assistant/home-assistant/blob/3f87d413813de84935ea67b5212c55348524447f/homeassistant/components/google_assistant/trait.py#L408-L415

@Krid86a @reaper7 please track eq3 issue in #13712, I reopened it.
I am going to lock this issue since z-wave thermostat already fixed. Please open new issue for other platform.


Add ray[default] to wget to run distributed inference out of box

part of https://github.com/vllm-project/vllm/issues/11137
Why do we need such PR?

User wants to leverage vllm default image to manage RayCluster version instead of managing separate ray images and vLLM distributions which frequently surface version compatibility problems
ray[default] package provides more capabilities like submission api and dashboard which is essential to run vLLM distributed inference in cluster environment
wget is used in kubeRay prob.https://github.com/ray-project/kuberay/blob/e595ee4c6297fb6b385421f7ca34fbd7c1c0b49f/ray-operator/controllers/ray/common/pod.go#L253 and https://github.com/ray-project/kuberay/blob/e595ee4c6297fb6b385421f7ca34fbd7c1c0b49f/ray-operator/controllers/ray/utils/constant.go#L178.  This can be definitely changed to curl but I feel adding a new package won't take that many room here.
Adding wget and ray[default] won't be risky. I notice some other files already include this change.

https://github.com/vllm-project/vllm/blob/f9ecbb18bf03338a4272c933a49a87021363b048/requirements-tpu.txt#L11
https://github.com/vllm-project/vllm/blob/f9ecbb18bf03338a4272c933a49a87021363b048/Dockerfile.arm#L11

FIX

this change is not directly related to above issue. I did verify the testing on my end and it's working fine.
https://github.com/vllm-project/vllm/blob/2d1b9baa8f57fc59912c7bcd07fd630fb9d72c9d/tests/entrypoints/openai/test_chat.py#L469-L503


Issue 238 non empty option persistence

Addressing issue #238:

the enable_empty_option  and enable_non_empty_option options in checklist filter type now get persisted as can be programmatically selected
added test case
Simplified CheckList properties naming and added comments for auto-generated documentation ( #197 )

:+1:


feat: Poll for packet acknowledgements

Description, Motivation, and Context
Further solidifies the pattern to make ibc.Chain simple. Essentially, query some state from the chain; other functions poll, wait, or take other action on the state.
The (hopefully quick) upcoming PR is polling for timeouts.
Timeouts was not included in this PR because it was already sizable.
The hope is polling for state may improve test suite run times.
Known Limitations, Trade-offs, Tech Debt

I could not reduce the ibc.Chain interface. I explored a few designs (even with generics) but having 2 separate methods for acks and timeouts will probably be easiest.
Reminder: The package test is a "sibling" package and probably not the right place. I'd prefer the root ibctest, but an import cycle currently prevents it. It's not worth fighting the import cycle yet.

@mark-rushakoff Thanks for the review! I will address the PR comments (all great suggestions) in the next PR.


adds unit test for models/members.js

#539
Adds unit tests for models/members.js file
New Coverage

LGTM :+1:

Closing this PR, as /members route is being deprecated


[AppService] fix #11412: az webapp create is removing existing application settings

History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)
[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

@panchagnula - FYA
@ThejaChoudary - FYI

@panchagnula - need your help here. Build is failing and I am not sure the exact issue. Could you please suggest me here.

This is has some unwanted changes and unable to revert them. closing and will create a new request


Library blocks Typescript compilation

🐛 The bug
When Trying to build a Typescript Project using magic-regexp tsc hangs indefinitely.
It just hangs and does nothing.
The Problem occurred in different projects (finding the problem the first time did cost me a good amount of time).
🛠 To reproduce
https://stackblitz.com/edit/github-dmq6xe?file=package.json
🌈 Expected behaviour
The build Process should go through or at least throw an error.
ℹ Additional context
When using ts-node to run the Script it works for whatever reason.
I've added a build script & a start:ts-node script to the example.

Hi @akowi-sknobloch,
can you try adding "skipLibCheck": true in tsconfig and see if this resolve the issue?
thanks

Alright seems like "skipLibCheck": true does the trick.
Does now compile without any issues.
Thank you!


Plugin prevents scrolling down the page when document mode toggled

Info:

OS: Windows 11
Logseq ver: 0.8.17
Theme ver: Default Light Theme
[ ] ⚠ Do i have issue with DISABLED my custom.css? I haven't created/edited any custom.css.

Description:
When attempting to scroll down the page in the rendered document mode (no dots visible to the left of blocks), Logseq struggles to scroll down at all.
To Reproduce:
Enable Sticky Headers plugin v1.4.3. Disabling the plugin fixes the issue.
Expected behavior:
Unhindered scrolling regardless of the current document mode.

this plugin some kind of experimental )
not 100% work in complex cases (((

i've tested and it scrolls. style of headers little broken in doc mode, but scrolls...
have you tried to reload Logseq (Ctrl + R)?

⚠ Experimental! Will work 99% only with AweomeUI + Awesome Styler nstalled! Other thems and Tabs plugin support not garanted!

i've tested and it scrolls. style of headers little broken in doc mode, but scrolls... have you tried to reload Logseq (Ctrl + R)?

I have tried with all plugins disabled except for logseq-sticky-headers and it still gets stuck when strolling.
I think the problem is somehow related to the width of the document's editing area. Here is a video (notice that when the right side bar is opened, the document editing area is reduced and the scroll bug returns):
https://user-images.githubusercontent.com/25013746/226200081-fe37d9b5-97df-43c2-9e60-76f6eb549adb.mp4

@yoyurec Any ideas?

Please reopen this issue, thanks.


Get CSS styles programatically

As  @petargyurov advised, added the ability to fetch the CSS styles programmatically.
This should allow easier handling for new styles of books :)

Thanks @Tranquility2 ! Looks good, I only have some minor comments/questions:

is the function getAllCssRules necessary provided it's only being used in getRootCssStyles?
things might get messed up if we add other root variables that are used for something else; might be better to prefix all patterns vars... 🤔 ...I think it's fine for now, we can deal with that later if we come to it.
I don't think we need to console.log anything
the file css_utils.js should probably be called cssUtils.js

@petargyurov Thanks for the feedback!

absolutely right, I usually keep the logic separated for future use, but I agree this will be simpler just to merge.
good catch, I update all the patterns vars to avoid any future issues.
commented out the console print (didn't deleted it as it can be useful for debugging issues), hope its ok.
total agree, update the file name accordingly.

TL;DR added/changed everything as you recommended.
Please note I didn't squash all the commits (with the fixes) so it will be (hopefully) easier to review :)
(If it looks ok don't forget to squash merge)

Oh well, I assume you wanted the full history?😅

Nope, just trigger finger! Morning coffee hadn't kicked in yet 😅 Oh well.


Tab Spacing hover

If you switch tab spacing within the status bar, the hover indicator will stay on the item after the interaction. It should go away after the user changes tab spacing.

Closed by #100


Resolve Failing Documentation Build for azure-mgmt-core

The install_requires wasn't properly replacing to a<buildid>. It was looking for .dev still.
This means that it's not properly installing the azure-core-1.8.2aXX version that hasn't released yet. This update ensures that azure-mgmt-core will require a 1.8.2a so it'll match the unreleased version of the package.
@praveenkuttappan take a look? This will turn our build green, but isn't repeatable like the .dev one was. We should take another look at this and dynamically recreate the rec, which will allow us to get away from the string-replacement that is currently happening.

Hey @praveenkuttappan really 'preciate the signoff. I just cleared it because I caught an issue.
When we install the version from the wheel directory, the version will be alpha. However, we get the expected version from the version on disk!. My latest commit ensures that when we don't erroneously throw mismatch errors for "hey we expected 12.0.0b2 instead of the installed 12.0.0a2020100201.
We get the 12.0.0b2 value from the local version on disk, so ensuring we set dev build the same way as when the packages were built ensures that everything lines up AND we know we're installing the "latest dev" version for regression checks.


Logic

your_code_here
{ title: "How did we do today?",
	questions: [
	{
		type: "dropdown", isRequired: true, name: "choosepicture", renderAs: "imagepicker", title: "People satisfaction degree!",
		choices: [
					  {value: "Awesome", text: "./img/1.png"},
					  {value: "Good", text: "./img/2.png"},
					  {value: "NotGreat", text: "./img/3.png"},
					  {value: "Awful", text: "./img/4.png"}
				]
	}
]},

Example I have 4 survey question. It is possible if I click Awesome it will directed to end page "Thank you for feedback etc etc" but when you click on value Good-Awful it will directly to next page which is my questionnaire.
Thank you.

You can use "Complete Trigger" to accomplish your task. Please check corresponding example.

Thank you @tsv2013, this is what I'm looking for.


Pipeline service validation error missing service

when running add_pipeline_status and instantiating Pipeline object we get the following error for missing service in the response.
Seen after this PR
1 validation error for Pipeline
service

Fixed in #6657


Kconfig circular dependency test PR

[DNM] test PR

Achieved the goal of tests. The PR is closed.