Variables in Kotlin: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-variables?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-variables#4
In which task and step of the codelab can this issue be found?
Step 5, "Create and use variables in Kotlin"
Describe the problem
Space is missing in string example
"Use the + symbol to add the string literal "at work" to the end of the reminder string."
should be
"Use the + symbol to add the string literal " at work" to the end of the reminder string."
to match output in sixth step under String headline
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hello @eiliftanberg,
Good catch! Updated and be published soon.


Release 0.5.1

Some confusion about what is contained in this release, but looks like it has the IE8 changes in it. Definitely needs testing before we release.


Drop the pre-release flag in the next release

Hello,
I am using your library to get TA for a trading bot project. The only thing that annoys me is that my IDE is not able to parse the requirements.txt because of the pre-release tag that is propagated towards PyPI.
Please consider in dropping the pre-release flag for the next release, unless you have any major stopping points.
Thanks!

Nice, thanks!


Script to translate the model file to PMML

This pull request is for issue Microsoft/LightGBM#31. I created a script pmml.py located inside the pmml directory. It takes the LightGBM_model.txt file generated by the lightGBM executable and translates each tree description inside to a pmml file. The script creates a directory if it does not exist and puts the files there. If the directory does exist then it asks the user to move it. The pmml files have been validated with it's xsd schema. More information on the schema can be found here. I also ran one of the pmml files as an input model to jpmml-evaluator. And it successfully took an input csv file and processed and scored each row inside. For more information how to use jpmml-evaluator, please look at the README.md file inside the pmml directory.

Hi @rmhasan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Hi @rmhasan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@rmhasan  Did you sign the CLA ?
@wxchan can you help to review this?

@rmhasan  Did you sign the CLA ?
@wxchan can you help to review this?

Shouldn't this extension somehow look like this?

Shouldn't this extension somehow look like this?

@guolinke I signed the CLA

@guolinke I signed the CLA

@wxchan, Im not sure how you mean it should look like that. Can you elaborate. When I was talking to @guolinke we decided on writing a python script that parses the LightGBM_model.txt file and outputs pmml files. You can everything we discussed at #31 .

@wxchan, Im not sure how you mean it should look like that. Can you elaborate. When I was talking to @guolinke we decided on writing a python script that parses the LightGBM_model.txt file and outputs pmml files. You can everything we discussed at #31 .

Sorry, I am not familiar with pmml, I thought it will be more like a (maven) package. Maybe @damienrj and @Alexkevin in #31 can help review it, is it what they want?
Anyway, to begin with, some simple suggestions:

clean your commit message, because it's a separated extension, people may read this pr in the future to understand what you do step by step. It'd better be merged all into one(it will count one commit anyway, just open a new local branch and pull with rebase)
your indent is too large, replacing it with 4 spaces will be better, use some text processor.

@wxchan You want me to rebase everything into one commit?

@wxchan  Hi I rebased the repository. Now its all in one commit.

Hi, I am closing and re-opening this PR to bump the CLA bot. Sorry for the inconvenience!

Hi @rmhasan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

@rmhasan what is the float precision in output pmml now?

@guolinke I read what ever is in the LightGBM_model.txt into a Decimal object, then I convert it into a string and print it out to the pmml file. It should maintain the precision of the variable printed to the LightGBM_model.txt file.

@rmhasan OK, I will merge it now.


Installation error

Unable to install on my Mac . Below is the error:
Unable to install 'Prisma.vscode-graphql'; there is no available version compatible with VS Code '1.20.1'.
Specifications

GraphQL for VSCode Extension Version:
VSCode Version: 1.20.1
OS Name: macOS Mojave
OS Version: 10.14

@gurpritsingh : I think you are on a very old version of VSCode. Can you please upgrade it, if possible?

I am closing this issue due to inactivity. Please re-open a new issue, if the problem persists.


New client library

Jenkins looks happy, except for the usual "not found" during migration...

The three code changes mentioned above have been done.


Need better field support.

Field support feels a bit weird right now. if you have  a *Frame, you can call Field(name) to get a field as you would expect, but there is no way to set any options on that field, and an error is returned as a member of the field struct rather than a second return value.
Based on the way everything else works, I'd expect to do something like
schema, err := client.Schema()
index, err := schema.Index(name)
frame, err := index.Frame(name)
field, err := frame.IntField(name, min, max)
err = client.SyncSchema(schema)
client.Query(field.Range(...))

Though, I think we could actually do away with the errors on everything but Schema and SyncSchema and do all the validation in those methods.
Currently, there is no easy way to "GetOrCreate" a field which is what is prompting this issue.

Closing this issue, since field stuff has changed quite a bit.


Link Aliucord Manager to Aliucord Installer

Description
Aliucord Manager is the upcoming replacement app for Aliucord Installer, which retains the same logo/app icon
Type of change
Icon link

Not sure why it went here, sorry


PrometheusBuilder::install: Build exporter in runtime context

Closes https://github.com/metrics-rs/metrics/issues/122

Tested this branch with a [patch.crates-io] in my local setup, and it resolves my issue.


RuntimeInformation.IsOSPlatform(OSPlatform.Windows) throws System.PlatformNotSupportedException on AWS Lambda

@normj  Added breaking check https://github.com/aws/aws-sdk-net/commit/a4d2348b25865195e43e9448ec5655bc473d5dcc
Unknown error responding to request: PlatformNotSupportedException:
System.PlatformNotSupportedException: Operation is not supported on this platform.
at System.Runtime.InteropServices.OSPlatform.get_Windows()
at Amazon.Runtime.ClientConfig.get_HttpClientCacheSize()
at Amazon.Runtime.HttpRequestMessageFactory.CreateConfigUniqueString(IClientConfig clientConfig)
at Amazon.Runtime.HttpRequestMessageFactory.CreateHttpRequest(Uri requestUri)
at Amazon.Runtime.Internal.HttpHandler`1.CreateWebRequest(IRequestContext requestContext)
at Amazon.Runtime.Internal.HttpHandler`1.<InvokeAsync>d__9`1.MoveNext()

Code Line Link

This error usually happen when one of your dependencies requires .NET Core 1.1. Would you be able to add your project file to the issue?

<PackageReference Include="Amazon.Lambda.AspNetCoreServer" Version="0.10.2-preview1" />
<PackageReference Include="Amazon.Lambda.Logging.AspNetCore" Version="1.1.0" />
<PackageReference Include="AWSSDK.Extensions.NETCore.Setup" Version="3.3.3" />
<PackageReference Include="AWSSDK.SQS" Version="<IP_ADDRESS>" />
<PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.3" />
<PackageReference Include="Microsoft.AspNetCore.Routing" Version="1.1.2" />
<PackageReference Include="Microsoft.AspNetCore.Server.IISIntegration" Version="1.1.3" />
<PackageReference Include="Microsoft.AspNetCore.Server.Kestrel" Version="1.1.3" />
<PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="1.1.2" />
<PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="1.1.2" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="1.1.2" />
<PackageReference Include="Microsoft.Extensions.Configuration.UserSecrets" Version="1.1.2" />
<PackageReference Include="Microsoft.Extensions.Logging" Version="1.1.2" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="1.1.2" />
<PackageReference Include="Serilog" Version="2.5.0" />
<PackageReference Include="Serilog.Enrichers.Memory" Version="1.0.4" />
<PackageReference Include="Serilog.Enrichers.Process" Version="2.0.1" />
<PackageReference Include="Serilog.Enrichers.Thread" Version="3.0.0" />
<PackageReference Include="Serilog.Exceptions" Version="2.5.0" />
<PackageReference Include="Serilog.Extensions.Logging" Version="2.0.2" />
<PackageReference Include="Serilog.Formatting.Compact" Version="1.0.0" />
<PackageReference Include="Serilog.Sinks.Console" Version="3.1.1" />
<PackageReference Include="Swashbuckle.AspNetCore" Version="1.0.0" />

Your project file is pulling in .NET Core 1.1 dependencies like Microsoft.AspNetCore.Mvc version 1.1.3. If you use our tooling to deploy it should warn you when you pull in .NET Core 1.1 references. If you look at the *.deps.json file that is in your publish folder look for the NuGet packages that are causing NETStandard.Library 1.6.1 to be pulled in. You need it to be using version NETStandard.Library 1.6.0.


Importing a dump from v0.22 doesn't work on v1.0.0-rc.2

I tried to import this dump, and Meillisearch wasn't able to import it.
20230130-162609246.tar.gz

Fixed by #3441, will be released in v1.0.0-rc.3


Query output value mapping

I'm using your sql_exporter...its great!!!However  i'm having hard time mapping the query output back to prometheus..Can you please share a document if you have any..for example..i want to check disabled user my query is like :
 - metric_name: mssql_account_locked
    type: counter
#    type: gauge
    help: 'Account locked status'
    key_labels:
      - username
    value_label: status
    values: [cntr_value]
    query_ref: account_locked_out

- query_name: account_locked_out
    query: |
      SELECT
        name as username, is_disabled as cntr_value
      FROM sys.sql_logins

I should see in prometheus a metrics with name "mssql_account_locked" ..right ?
However i can't see it

Well, I don't know what to say. Apart from a couple of quirks (i.e. the type should be gauge, not counter; and you should drop the value_label if your query only returns one value per row) everything else looks fine. Maybe apart from the YAML indentation, but that could simply be the copy-pasta.
BTW, there is a very thoroughly commented configuration example at https://github.com/free/sql_exporter/blob/master/documentation/sql_exporter.yml . You may use it as a reference, or source for copy-pasting bits and pieces.
Anyway, if you actually started SQL Exporter correctly and it actually loaded your configuration (you can check that in the console output, it should say something about Loading configuration from and Loaded collector from), then any errors should also be output to the console. Do you see anything there?


String Body Parameters

Hi,
I'm using wiremock and it's helping a lot with my project development.
I have a problem when I try to send not exact string parameters through request data. It works OK when I use:
"url": "/policy/search",
"bodyPatterns" : [
{ "equalToJson" : "{"session_id":"23232", "results_type": "0"}"}
]
But what happens if I want that results_type accepts ANY incoming string parameter?
Let's say: { "equalToJson" : "{"session_id":"23232", "results_type": "any string"}"}
How can I achieve this?
Thanks!

Please ask questions  like this on the mailing list:
https://groups.google.com/forum/#!forum/wiremock-user


Bump nerves_system_br to 1.27.2

Bump nerves_system_br to 1.27.2
Remove no longer needed i-k3-boot-firmware

I didn't test this, but it compiles.

Hey @fhunleth, thanks for the contribution. I had intentionally left it on the nerves_system_br that wasn't latest because of the same thing you saw 😅: the pkg name conflict with ti-k3-boot-firmware. For the time being I would like to continue to use our managed pkg mainly because there have been a lot of improvements to the FW since <IP_ADDRESS>.
I do have a branch waiting for me when I get back home which brings in latest BR and renames our pkg to ti-k3-boot-firmware-next (I was between -next or -latest suffixes), but I probably won't be able to get to this till maybe mid-week of 05/13. I'm more than happy to approve the PR if you make those changes before I can get to it.
Thoughts?

I just did this quickly to make it possible to easily use this system with Nerves projects. No need to merge. It sounds like the next step for you or Alex is to send updates to Buildroot.


Testing the pullrequest resource

After debugging your CI pipeline, it is because you are on an older version of the resource for the pull-request. Please bosh recreate your workers. Thanks.


Fix code example in README

This won't compile if it isn't pub!
Fixes https://github.com/rustwasm/wasm-bindgen/issues/309

Thanks @steveklabnik! :D


Same issue as #336 -  TRADFRI bulb GU10 WS 400lm Not Supported

Updated to the latest version, Running v3.0 version of the plugin. Receiving the error;
2021-01-19 18:55:13.659 (Zigbee2MQTT) This plugin does not support zigbee device with model "LED1537R6/LED1739R5" yet
2021-01-19 18:55:13.659 (Zigbee2MQTT) If you would like plugin to support this device, please create ticket by this link: https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin/issues/new?labels=new+device&template=new-device-support.md

This Tradfri bulb used to work before, so i'm quite confused..

Most likely you are not running v3.0 of the plugin as such message has been removed in 3.0
This device is supported in the latest plugin version

Most likely you are not running v3.0 of the plugin as such message has been removed in 3.0
This device is supported in the latest plugin version

CHANGELOG.txt
This is the changelog file, which states that version.
Can you tell me how to certainly check the version?

CHANGELOG.txt
This is the changelog file, which states that version.
Can you tell me how to certainly check the version?

Just follow these steps: https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin#plugin-update
The version will be in your Domoticz log, something like:
2021-01-18 09:45:09.374 Status: (Zigbee2MQTT) Started.
2021-01-18 09:45:09.712 Status: (Zigbee2MQTT) Initialized version 3.0.0

Just follow these steps: https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin#plugin-update
The version will be in your Domoticz log, something like:
2021-01-18 09:45:09.374 Status: (Zigbee2MQTT) Started.
2021-01-18 09:45:09.712 Status: (Zigbee2MQTT) Initialized version 3.0.0

Hi Stas,
I have followed the update steps. While booting Dotomitcz (or Zigbee2MQTT) there is no message as you mention.
I have set the plugin in Domoticz to "verbose" logging mode

Hi Stas,
I have followed the update steps. While booting Dotomitcz (or Zigbee2MQTT) there is no message as you mention.
I have set the plugin in Domoticz to "verbose" logging mode


Align header to main page

I've got pretty close to the appearance of the main page, but not quite pixel perfect, so there are a few little jumps when switching back and forth. I'll sort those out in a follow-up work item.

Partial resolution of #47.


Add security attributes

Summary | Résumé
Set a security attribute  that will allow an administrator to take the appropriate actions when retrieving erred data.
closed #701
deps
Security Attribute Badge:

@falila It is almost good to go but @bryan-robitaille added a comment to the ticket https://github.com/cds-snc/platform-forms-client/issues/701#issuecomment-1112551569

@falila  Did you plan on adding integration tests to make sure the badge is displayed depending on form definitions?


Getting 215 from Twitter half of the time

Hi,
Thanks for the lib, working great for our project.
I have a really weird issue though: we have twitter/fb signin buttons and link buttons if you're logged in and haven't linked your account. The signin buttons work perfectly fine and so does the fb link one.
However, the twitter linking button always fail with the following error:
{"errors":[{"code":215,"message":"Bad Authentication data."}]}

This is really odd as I wrapped my use of ng-cordova-auth in a service and so both buttons calls the exact same method with the same params, I checked the keys they are identical in both calls so the issue must be something to do with the generated params for the call to https://api.twitter.com/oauth/request_token.
Here are the signature obj for both requests
// works
{"signature_base_string":"POST&https%3A%2F%2Fapi.twitter.com%2Foauth%2Frequest_token&oauth_callback%3Dhttp%253A%252F%252Flocalhost%252Fcallback%26oauth_consumer_key%3Dsb5Q5NWvtyRmNjDX3QBZQfqOI%26oauth_nonce%3DsobE3HKFkP%26oauth_signature_method%3DHMAC-SHA1%26oauth_timestamp%3D1426710378%26oauth_version%3D1.0","authorization_header":"OAuth oauth_consumer_key=\"sb5Q5NWvtyRmNjDX3QBZQfqOI\",oauth_nonce=\"sobE3HKFkP\",oauth_signature_method=\"HMAC-SHA1\",oauth_timestamp=\"1426710378\",oauth_version=\"1.0\",oauth_signature=\"NxLNWpif%2FZe16%2BOzCzV99MjAy0Q%3D\"","signature":"NxLNWpif%2FZe16%2BOzCzV99MjAy0Q%3D"}

// fails
{"signature_base_string":"POST&https%3A%2F%2Fapi.twitter.com%2Foauth%2Frequest_token&oauth_callback%3Dhttp%253A%252F%252Flocalhost%252Fcallback%26oauth_consumer_key%3Dsb5Q5NWvtyRmNjDX3QBZQfqOI%26oauth_nonce%3DFvblLjrY1j%26oauth_signature_method%3DHMAC-SHA1%26oauth_timestamp%3D1426710389%26oauth_version%3D1.0","authorization_header":"OAuth oauth_consumer_key=\"sb5Q5NWvtyRmNjDX3QBZQfqOI\",oauth_nonce=\"FvblLjrY1j\",oauth_signature_method=\"HMAC-SHA1\",oauth_timestamp=\"1426710389\",oauth_version=\"1.0\",oauth_signature=\"vcAxp1IjgmtdMy7UyZx86A0aOUQ%3D\"","signature":"vcAxp1IjgmtdMy7UyZx86A0aOUQ%3D"}

Any clues ?
Cheers

Nevermind I was being silly !
For the people finding that issue, make sure you don't change the Authorization header in your angular config otherwise that will not work (obviously).


Update Release Notes

Getting closer...
@farfilli I went through the commit history to try to remember everything. If you have time, please take a look at the Release Notes and let me know if you notice anything I messed up/left out.

@farfilli
To make it less confusing for people who download the previous release, I changed the names of README.md and release_notes.md to README_NEW.md and release_notes_NEW.md.  I also added README.md and release_notes.md from v<IP_ADDRESS>.
I'll undo the changes when I get the release candidate ready.
In the meantime, if you want to see my recent changes, please look at *_NEW.md.

They looks good to me

Good deal.  FYI, I'll be out of the office next week.  To be responsive to questions/comments I'm going to wait until the following week to post to the SE Forum.  I'm still working on the text of that post.  Below is what I have so far.  I might take out the Question part and save it for later.
==================================
Solid Edge Housekeeper: Some Big News, a Call for Beta Testers, and a Question
Big News
Housekeeper has its first GitHub Contributor -- our very own @Fiorini!  I can't wait for you to see what he has done with the place!
Beta Testers
To be among the first to see the new release in action, volunteer to be a Beta tester!  Beta testing is nothing more than performing your own workflow on your own files and telling me if you run into problems.  It isn't meant to be a lot of work.  The big idea is to make the program better for you and me and everyone else!  To sign up, message me or respond to this post.
A Question
For a future release, I'm looking at how we find files related to a top-level assembly on a server.  The way it works now, the program finds all internal links to the assembly, which is fast.  After that, the only thing left is to find the drawings.
For that, Housekeeper uses where used, which is slow, at least on non-indexed drives.  The thing is, there is an implicit assumption that we don't know the drawing file names or where they are located.
At my company we know exactly.  The draft has the same name as the model and is in the same directory.  We don't need where used for that.
Here's the question (even if you don't use Housekeeper): How about you?  If you had the name of a model file, would you know the drawing's name and where it is located?  What about revisions, or other complexities?
Please respond to this post with your company's method.  The hope is to come up with a file name "formula" and eliminate the need for where used altogether.
If someone has already posted the method you use, to minimize work for you, please feel free to simply "Like" their response.  Or maybe respond and say "we do it like @SoAndSo, except this tiny difference".
Looking forward to your input!

Done for now...


Feature Request: Copy Clipboard Icon for Clarity

Select one ...  (check one with "x")
[ ] bug
[x] feature request
[ ] enhancement

Expected behavior

Actual behavior

Reproduction of behavior

Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

Environment details

Angular version: 4.x.x

Clarity version:

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Completed as part of epic #1787


Adding php static analisys

This PR is in order to add a static analyzer, getting errors before run phpunit.

Hello @reisraff,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @reisraff,Thanks again for the PR!                                         We noticed that you have earned 3 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold.                                         Also, please be sure that you have officially registered here.                                         Thank you and Happy Hacktobering!                                         Team SendGrid DX

Hello @reisraff,Thanks again for the PR!                                         You have earned 3 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


Can you change the Rocket to custom icon?

I am interested in notifying Instana on releases but also on new commits or even pull requests. However the rocket icon could be misleading if I am using a GitHub action to push and notify Instana that I had made a new change.

You can see here I committed some changes but it displays a rocket

It appears that you are submitting the issue to an unrelated repository.


BROWSER CAN'T START WITH SECOND COLAB RUN

EVERYTHING WORKS FINE UNTIL I BACK TO COLAB NEXT DAY AND WANT TO RUN IT, EVERY TIME WHEN I REINSTAL WEBUI INFINITY BROWSER IT CAN'T START ON NEXT NEW START OF COLAB NOTEBOOK.
No module named 'dotenv'
*** Error loading script: iib_setup.py
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 319, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/script_loading.py", line 10, in load_module
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib_setup.py", line 1, in
from scripts.iib.api import infinite_image_browsing_api, send_img_path
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib/api.py", line 50, in
from functional import seq
ModuleNotFoundError: No module named 'functional'

pip install -r requirements.txt

You can try restarting sd-webui or manually installing the dependencies to see if that helps.

It did't help at all.

the same problem at my own linux server

Same problem. After manually installing functional, throws error ImportError: cannot import name 'seq' from 'functional'.
*** Error loading script: iib_setup.py
    Traceback (most recent call last):
      File "/workspace/sd/stable-diffusion-webui/modules/scripts.py", line 274, in load_scripts
        script_module = script_loading.load_module(scriptfile.path)
      File "/workspace/sd/stable-diffusion-webui/modules/script_loading.py", line 10, in load_module
        module_spec.loader.exec_module(module)
      File "<frozen importlib._bootstrap_external>", line 883, in exec_module
      File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
      File "/workspace/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib_setup.py", line 1, in <module>
        from scripts.iib.api import infinite_image_browsing_api, send_img_path
      File "/workspace/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib/api.py", line 50, in <module>
        from functional import seq
    ImportError: cannot import name 'seq' from 'functional' (/usr/local/lib/python3.10/dist-packages/functional/__init__.py)

What version of SDWebUI are you running on? I have been unable to reproduce this issue.

Same problem. After manually installing functional, throws error ImportError: cannot import name 'seq' from 'functional'.
*** Error loading script: iib_setup.py
    Traceback (most recent call last):
      File "/workspace/sd/stable-diffusion-webui/modules/scripts.py", line 274, in load_scripts
        script_module = script_loading.load_module(scriptfile.path)
      File "/workspace/sd/stable-diffusion-webui/modules/script_loading.py", line 10, in load_module
        module_spec.loader.exec_module(module)
      File "<frozen importlib._bootstrap_external>", line 883, in exec_module
      File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
      File "/workspace/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib_setup.py", line 1, in <module>
        from scripts.iib.api import infinite_image_browsing_api, send_img_path
      File "/workspace/sd/stable-diffusion-webui/extensions/sd-webui-infinite-image-browsing/scripts/iib/api.py", line 50, in <module>
        from functional import seq
    ImportError: cannot import name 'seq' from 'functional' (/usr/local/lib/python3.10/dist-packages/functional/__init__.py)

It's not 'functional,' it's 'pyfunctional.' Please refer to this link: https://github.com/zanllp/sd-webui-infinite-image-browsing/blob/main/requirements.txt.

It's not 'functional,' it's 'pyfunctional' Please refer to this link: https://github.com/zanllp/sd-webui-infinite-image-browsing/blob/main/requirements.txt.

My SDWebUI is not current but this fixed my problem. Thanks!

Same issue.  I had to install manually install pyfunctional with python -m pip install pyfunctional.
requirements.txt listed it but it


docs: Added auto-formating and link localizing for all docs.

cc @saswatamcode
Finally some unified and consistent tool for checking docs and local links. More additional later!
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

FYI: Some discussion on Slack: https://cloud-native.slack.com/archives/C022HUPSK7S/p1623492399007900

Are you shepharding this PR now @saswatamcode ?

PTAL @kakkoyun @yeya24 @saswatamcode  lot's of automated formatting, moved proposal into 3 categories and nothing more, no content added!

Do you mind double checking again @onprem ? We need your  amazing eyes ;p

I think I did not kill anything important, but who klnows


basic RMA handling?

Does it make sense to add a feature/tool to do some basic RMA handling? I'd say this is one of the daily duties of someone managing a data center.

Probably addressed by #182


chore: add github issue, pr template

What type of PR is this?
/kind cleanup

What this PR does / why we need it:

When participants are involved, it leads to more standardized issue and pull request submissions, helping reviewers better understand the meaning behind the issues and pull requests.

Which issue(s) this PR fixes:

Fixes # None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/assign @danielvegamyhre
thanks!

/lgtm
/approve
Thanks!


possible to pass username and password for login?

Hi, possible to pass username and password for login?

so , dump question


[Question] What images were used?

Question
COCO provides lots of datasets in terms of different versions. I want to know which dataset is utilized. Where can I download the caption and bounding box files? Please provide guidance If you are free.

Hi, please check out the data here: https://github.com/haotian-liu/LLaVA#visual-instruction-tuning
Thanks.


add arpeggio

the idea is to show the chord tone on fretboard that can be use to play arpeggio, something similar like the scale, but only show the chord tone
e.g. for major arpeggio (semitone : 0-4-7, degree : 1-3-5)
music-module.ts (not sure you want to implement this on the same module or create new)
    { name: "Major", intervals: new mod.Mod([true, false, false, false, true, false, false, true, false, false, false, false]), modes: [{ name: 'Major', index: 0}], defaultModeIndex: 0 },

thanks

Not sure I understand? Guitar dashboard already does that when you select a chord?

Q: Guitar dashboard already does that when you select a chord?
A: hm 50-50 i think, the current code show the chord tone with another tone that is part of the scale, the advantage to add arpeggio are :

guitarist can focus just the chord tone on fretboard
it more support of the chord, let say, augmented, diminished, suspended, dominant, major, minor and it's variation (6th, 7th)
with current code, it just support either major or minor

the idea is base on https://fretboarder.app
when you click it shows major arpeggio on fretboard that is consist R (root note) -3-5 and it's note (A-C#-E)
the author created more complex, can combine between (scale + scale), (scale + chord), etc (more than 2 elements combination e.g. (chord+scale+chord) ), which is enlarge the idea to make improvisation
base on experience it's rarely use all of the tone that provide by scale, some used, some not
here's my local modification for chord, if you interest to implement, i'll send the code, thanks

My intention with Guitar Dashboard was to provide a "music theory explorer for guitarists", not necessarily a catalogue of chords etc. Showing the chord as part of a scale is exactly what GD is meant to do. The numbered buttons in the very centre are there so you can add and remove chord degrees. Here's is it displaying what you asking for above:

Thanks for showing me fretboarder.app, it looks really nice, especially the feature that allows you to combine scales/chords. I think their intention is very much to act as a catalogue. You can see this in the different approaches to how we have coded our apps. fretboarder.app simply hard codes chords as interval sets, as you can see in this code file: music.js. There's is nothing like that in GD because the chords are generated by an algorithm.


[Refactor] OpenSearchException streamables to a registry

OpenSearchException uses an enumerator to create a handler for each serializable Exception implementation. The enumerator is to make it easy to store in an immutable map. The problem with this Hack is that it makes it difficult to unwind the tight coupling of StreamInput with the OpenSearchException class. This commit switches from using an enumerator to a registry in the opensearch-core library in order to support serverless and cloud native implementations outside of the server module.
This commit also refactors base primitive serialization readers and writers from StreamInput and StreamOutput to BaseStreamInput and BaseStreamOutput, respectively.
relates #5910

Not reproducible, rebasing and refiring
./gradlew ':server:test' --tests "org.opensearch.index.IndexServiceTests.testAsyncTranslogTrimTaskOnClosedIndex" -Dtests.seed=4212AD3BAE883BE9 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ar-BH -Dtests.timezone=CST6CDT -Druntime.java=19

Oye!! Another unrelated failure? Looks like PRs are becoming awfully volatile due to test failures not being regularly fixed.
This one was not reproducible but has already been documented as a "flaky test" and caught on multiple PRs.
./gradlew ':qa:smoke-test-http:integTest' --tests "org.opensearch.http.SearchRestCancellationIT.testAutomaticCancellationMultiSearchDuringQueryPhase" -Dtests.seed=82B60F546F2663DE -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ru-RU -Dtests.timezone=Asia/Hebron -Druntime.java=19

Unreal.. two more (https://github.com/opensearch-project/OpenSearch/issues/3590)
Neither are reproducible locally.
./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.allocation.ClusterRerouteIT.testDelayWithALargeAmountOfShards" -Dtests.seed=86C4F0E32ABC37F8 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=cs -Dtests.timezone=Atlantic/Jan_Mayen -Druntime.java=19

./gradlew ':server:internalClusterTest' --tests "org.opensearch.search.SearchWeightedRoutingIT.testSearchAggregationWithNetworkDisruption_FailOpenEnabled" -Dtests.seed=86C4F0E32ABC37F8 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-CR -Dtests.timezone=Pacific/Pitcairn -Druntime.java=19

testDelayWithALargeAmountOfShards

Sure will mute this test.

testDelayWithALargeAmountOfShards

Sure will mute this test.

That works.. let's try to resolve soon, though so @AwaitsFix doesn't turn into an alias of /dev/null :)

Do you want this in 2.x?


ci(lint): fix ignite version

Fix the installed ignite cli version when checking generated sources.
To avoid any inconsistencies between local generation the CI generation, use the --clear-cache flag when proceeding locally to prevent ignite from checking sources checksums before generating the spec:
ignite generate openapi --yes --clear-cache

Does not fix the issue of the generated openapi.yml file which is different from the one in the branch. 😞 That's weird...
Can we add a step in the workflow to output the diff for debugging?

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


Document Scanner not working.

On single button click two instance of camera and Gallery are opening.

After clicking of image it is not moving to image cropper screen. Also its show on log isDirectoryCreated=false, and onTruncateError

Any Update?


Plugin disables f-movement in macros

When I use default settings, they are re-mapping [FfTf] keys to better-n functions.
However, in case of recording macros, recorded 'f' were not reproduced when macro is called.
Here is a workaround to disable keys in config:
          ["f"] = {cmdline = true},
          ["t"] = {cmdline = true},
          ["F"] = {cmdline = true},
          ["T"] = {cmdline = true},

p.s. Maybe it's worth to find another option to make clear how to disable better-n calls and propagating original nvim calls. I've only managed to figure this out after digging into code.
Thanks

Hey, thanks for the issue!
I'll think about of to fix the "mandatory" default overrides, I agree it's a bit problematic.
Regarding f-movements not working, I'll look into this and see if it's something that's fixable.

This should be solved with #10, although you will need to modify your configuration to work with the new version. (Sorry about that!)


Cleanup recurring event creation

Cleanup recurring event creation to move inside of the recurring event model.

Codecov Report

Merging #132 (0064037) into main (784a95a) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   96.54%   96.56%   +0.01%
==========================================
  Files          43       43
  Lines        2403     2413      +10
==========================================
+ Hits         2320     2330      +10
  Misses         83       83

Impacted Files
Coverage Δ

ical/event.py
97.38% <100.00%> (+0.30%)
:arrow_up:

ical/timeline.py
98.14% <100.00%> (-0.29%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Use an IPC call to show the dialog for choosing a directory

Use an IPC call to show the dialog for choosing a directory.
That way we don't need enableRemoteModule; which is more secure.

Meanwhile this has been implemented in #79


Fix error responses in Tracking endpoint

Fixes point 1 mentioned in UPS-API/api-documentation#12

Hi, thank you for the PR. We have updated the error responses and merged the updated code.


go.cd replaced by gocd.io ?

Installing package repos is currently broken due to go.cd being down

Released 1.3.2 with all domain name changes.

Thanks!


Request for bundle fails under Windows when accessing root page

Request for bundle fails under Windows when accessing root page (/).
Request URL:http://localhost:60274/__bundle//index.js.bundle.js
Request Method:GET
Status Code:404 Not Found

In this example I tried creating a simple index.js file and GET /. When I tried to GET /index.js it worked just fine. My guess is that it has to do with Windows different directory separator \.
node -v && run-js --version
v4.3.0
2.0.1

My guess is that it has to do with Windows different directory separator .

Maybe, but  on some underlying shit (express) package/middleware, i dont think it is run-js problem - i mean, in first glance it looks good, it uses path module which as i know dont have problems for such windows shits :D (note: damn, sorry for my aggressive ton, i hate windows ;d)
Cheers :beer: :)

Or the double slash thing (__bundle//) setting here lib/routes/bundle-file.js#L19

Indeed I would prefer to be working on linux, but since I have to communicate with an C# API in a Windows project, It would be a pain to be switching between operational systems.
Since I bumped into this, I'm using prot for now. If I have the time, maybe I'll come back to this issue and try to solve it (since prot is not being actively maintained).

Thanks for catching this! This should be fixed with v2.1.0, which I've just published on npm. Let me know if you run into any other issues on Windows.

Works like a charm. Great Job!


Freeze containerd pods

Changes

Allows the container-freezer service to freeze containers using the containerd runtime.

Fixes #6
(will merge this after #2 )
/assign @julz @markusthoemmes

/hold
Adding some tests

/unhold

I'll defer to @julz on this.
/unassign

Woot !


Footnote numbers can get split across lines

If a footnote number falls the end the line, the brackets and numbers can get split across lines. For example:
Lorem ipsum dolor sit amet, consectetur adipiscing elit.[
1]

This happens because the footnote number is three separate fragments and Prawn has no way to correlate them.
Furthermore, if the footnote number is the last text in the paragraph, it will be placed too high on the line, making it overlap the preceding line.

One way to solve this problem is to put the square brackets around the number inside the link. This appearance is consistent with how footnotes are done on Wikipedia and thus a reasonable style to use as a default.
The other solution is to implement wrap avoidance (i.e., nowrap) in the arranger.

Since we're going to proceed with a style change, I'm scheduling this fix for the 2.3.x release line.

I'm going to apply the style change in a separate issue that will be schedule for 3.0.0.


Error after 6.01, definitions.json not found

Probably aware of this issue, but just in case, when I start this, I receive the following output that may be related to a change in the XIVAlex source:

Try giving -u argument when running the script for now.

No problem!  Just wanted to make sure this was accounted for.

Hello, a friend told me xivalex was up so I decided to try and run this again too, however I'm finding the same issue posted above.
I don't understand much about coding so I don't get what it means to give the -u argument when running the script, is this done in the iptables line?

@BloodletterXIV Try running rm definitions.json, and ignore the result. After that, run the last line curl ... | python again. It should work now, but I couldn't test it yet.

I do not believe it is working for me, I get this output, but the lag in game persists nor am I getting a log of my actions:

It is connecting, but the actual latency mitigation feels like it is not going through.  I am no longer seeing a "DelayAdjusted" value on my log, and can still barely double weave without clipping, suggesting it is not picking up the latency correction.  Here is what my log is reporting when I am queueing and performing actions:

im having the exact same issue as YOxidized, its connecting but not mitigating

@Soreepeong I did exactly as instructed and I'm having the same problem as the users above, I can get into the game but double weaving is impossible. I noticed there was an error and a "shutdown" the moment I got past the queue and loaded into the game, which I've never seen before at that part.

@YOxidized You might want to try fiddling with -e [fixed extra delay in seconds] to set different extra delay, or -m to turn ping measuring on again. Measured ping tend to be largely inaccurate, so I turned it off by default.
@BloodletterXIV That's probably an another problem. Screencap of the error messages would help.

These changes seem to have a very adverse effect on the program's effectiveness, and the new log reporting makes it very difficult to really measure the discrepancy.  I can try these fixes, but I may just fork the version before yesterday's changes instead.

@Soreepeong Alright, here's what's happening exactly:
After the iptables line, I start the script and it gives me this (the opcode downloading stuff is totally new, this never happened in 3 months and when EW came out I just did rm definitions.json and everything worked perfectly).

Once I did that I stopped the program with Ctrl+C and it gave me this "cleaning up" log which is also something new. The rm definitions.json seemed to work fine after.

Finally, I ran the curl line again and got into the game. The D->U Read Finish below is when I loaded in and the very last part was just an example of what it shows when I perform an OGCD action.

@BloodletterXIV Your issues seem to mirror my own, so I do not think it is anything to do with your VM; I appreciate you thoroughly explaining them in this manner, and your screenshots are exactly what I am seeing in my VM.

my log is the same as BloodletterXIV

thats what appears when i input the curl line (which never appeared like that)

and thats me logging in and using gcds and ogcds in a  dummy

Okay, I think I figured out why it didn't do anything. I was misunderstanding that slicing a bytearray will return a partial view, but it actually returned a copy. It should correctly mitigate stuff now.

I can confirm, that at least for me, it is working properly now, thanks and keep the amazing work!

It does work, but it still lacks the log reporting that lets the user know exactly what is going on in the background, and though I have nothing to substantiate this claim, feels slightly less effective.  Thank you for your hard work as always, but may I ask what benefits this new version has over the previous?

Previous version by default tended to overcompensate in a lot of cases, due to inaccurate latency measurement, so it's off by default. You can turn it on by giving it a flag.
Instead of doing curl http://.... | python, try:
curl http://... -o mitigate.py
python mitigate.py -m

Ah, if it overcompensated, then that makes sense to change.  I will experiment with -m toggled on, but this version still does a fine job of achieving the desired results.  I may experiment with toggling the -m option, and thank you so much once again for your hard work, dedication, and for developing this for PS4 players so that we may share a raid comfort than many PC players have access to.

If you ever noticed skills being refunded not because of distance nor enemy becoming untargetable, that is a symptom of overcompensation. If it never happened to you, -m is probably fine to use for your case.

This is really good to know; I have heard of people on XIVAlex experiencing this in the past, but it has not once happened to me, so I will utilize the -m flag for now and disable it if I see any of this behavior as a result of its usage.


log the error for redirection of request

current, when moved/asked happen (such as fail over happen), no any message hint the event.
so suggest log it as warn to show the event.
test result example:
WARNING: should request to : <IP_ADDRESS>:7380
BTW： the tests are not stable.

@jiafu1115 In case of cluster reconfiguration, node repair like scenarios user can be bombarded with this kind of logs. These scenarios may be rare but expected. So, IMHO, we should not log these exceptions.

@marcosnils WDYT?

node repair should only happen in control limit time period. it shouldn't take too much time. Take normal case, for example, I take manual fail over and no any message for the application to know the event. so I raise this PR.

@marcosnils @HeartSaVioR Close?

@sazzad16 change to fine from warning to let user have more choice to decide if track the change.

will open new one due to the PR is to older branch.


Entity could not be transformed

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0-dev20190317 (gpu nightly)
TensorFlow Addons installed from (source, PyPi): source
TensorFlow Addons version: master
Python version and type (eg. Anaconda Python, Stock Python as in Mac, or homebrew installed Python etc): anaconda (conda-forge) python 3.7.1
Bazel version (if compiling from source): 0.20.0
GCC/Compiler version (if compiling from source): 7.3.0
Is GPU used? (yes/no): yes
GPU model (if used): RTX 2070

I'm seeing the following warning appear when trying to use the transform op.  The simplest way to reproduce is to simply run the tests via python:
python ~/src/addons/tensorflow_addons/image/transform_test.py
[...]
W0317 18:03:00.753756<PHONE_NUMBER>68544 tf_logging.py:161] Entity <function image_projective_transform_v2 at 0x7f3e73a05a60> could not be transformed and will be staged without change. Error details can be found in the logs when running with the env variable AUTOGRAPH_VERBOSITY >= 1. Please report this to the AutoGraph team. Cause: Unexpected error transforming <function image_projective_transform_v2 at 0x7f3e73a05a60>. If you believe this is due to a bug, please set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output when filing the bug report. Caused by: Unable to locate the source code of <function image_projective_transform_v2 at 0x7f3e73a05a60>. Note that functions defined in certain environments, like the interactive Python shell do not expose their source code. If that is the case, you should to define them in a .py source file. If you are certain the code is graph-compatible, wrap the call using @tf.autograph.do_not_convert. Original error: could not get source code

export AUTOGRAPH_VERBOSITY=10
python ~/src/addons/tensorflow_addons/image/transform_test.py
[...]
I0317 18:11:38.742995<PHONE_NUMBER>01152 ag_logging.py:132] Error transforming <function image_projective_transform_v2 at 0x7fa2ea6b2a60>
Traceback (most recent call last):
  File "~/miniconda3/envs/tf2/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/parser.py", line 51, in parse_entity
    source = tf_inspect.getsource_no_unwrap(entity)
  File "~/miniconda3/envs/tf2/lib/python3.7/site-packages/tensorflow/python/util/tf_inspect.py", line 408, in getsource_no_unwrap
    lines, lnum = _inspect.findsource(obj)
  File "~/miniconda3/envs/tf2/lib/python3.7/inspect.py", line 786, in findsource
    raise OSError('could not get source code')
OSError: could not get source code

Is this expected?  I expect this is likely an artifact of a broken source build on my end, but I imagine others may experience similar issues until the CI/CD system is building nightly versions for all platforms and python versions.

Hi Kyle, without looking too closely I see you're running py37 and said you installed from binary. As seen in #75 we don't have a working py37 build yet.
Are you sure you're in the right environment?

Oh, it seems you're running the test from the source code. The binaries found in tensorflow_addons/custom_ops are not built until you compile. For that reason it's probably recommended you use a pip install (or conda in the future). Alternatively you could build and test using bazel

I pip-installed TF from the wheel and compiled tf-addons from source using the following:
 conda activate tf2

python --version
Python 3.7.1

./configure.sh
bazel build build_pip_pkg
bazel-bin/build_pip_pkg artifacts
pip install artifacts/tensorflow_addons-*.whl

I should also note that I had to make some bazel config changes to get the build working on my system (namely, setting D_GLIBCXX_USE_CXX11_ABI=1 throughout).

Last note: I've seen the same warning in very basic non-test calculations, but I figured executing the test script manually is a pretty good way to reproduce the issue initially.

@kyleabeauchamp Thanks for reporting this problem, Kyle! Would you like to fix it? I'm kind of busy this week, so I might have time to look into the issue next week.

I'll certainly open a PR if I end up figuring out the solution, but it's not obvious that I'll figure it out before you, as I only recently started ramping up on 2.0.

@alextp Alexandre, do you know what the warning means?

@seanpmorgan @jharmsen Hi, is it related to #90?

I tried a git revert f36d04f5f65573f1c88c5b7110cf939b1f654e99 and the issue was still present.
I can poke around a little more.

A lot of the test coverage we moved over from contrib is incomplete and I think this bug is an example of something that should be caught (It's on the agenda to review the tests of the ported code). If I were to guess it seems like it's related to decorating the function with autograph, but I haven't dug in past that stack trace. Also a little busy, but I can take a look in a day or two if the error isn't immediately obvious to @alextp

@mdanatg I think the warnings from autograph need to be a little clearer. Can you comment here?

Yes, we're reviewing the warning messages, and planning to adjust this one as well. But I'm encouraged that the instructions in the warning allowed you to re-run the code with the diagnostic flag - that helps a lot.
To be clear, this is just a warning and probably doesn't affect the functionality of your code. So we probably want to include a flag that suppresses it for now.
Digging deeper for the image_projective_transform_v2 function that causes the error, it looks like a custom op - these won't have any Python source code attached, so autograph should not attempt to convert them in the first place. We'll look into that.

So just a recap:

Regarding what I said about test coverage catching this... probably unlikely because its difficult to assert no warnings when addons depends on TF which depends on many other packages. I apologize for reading through the issue too quickly and not seeing the error came after the AUTOGRAPH env variable. That said we still need to re-evaluate the coverage for tf.contrib ports.
@mdanatg Can you update this issue when we can expect custom ops to be ignored by autograph?
@kyleabeauchamp Thanks again for reporting this. We'll leave it open until a fix because the unable to locate source code warning does sound quite damning

Will do! Feel free to add me to the Assignees list but we linked an internal bog to the issue.

@mdanatg Just wondering if you could give an update on this issue? I checked today's nightly and looks like the issue is still there. With TF2 release candidate somewhere on the horizon I thought it's worth revisiting.

It seems that the issue did not update automatically, probably because this is a separate repository. The issue should be resolved at head, but if it still appears then we need to look a bit closely.
Unfortunately I wasn't able to build addons from source. This is the error I'm getting when running python tensorflow_addons/image/transform_ops_test.py:
...
File "...tensorflow/python/framework/load_library.py", line 61, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError:
.../tensorflow_addons/custom_ops/image/_image_ops.so: undefined symbol: _ZN10tensorflow8internal21CheckOpMessageBuilder9NewStringEv

Alternately, if you run the script instead, with AUTOGRAPH_VERBOSITY=10 and link the complete log output, that might be enough for me to figure out what's wrong.

Ah, sorry thats an ugly error that comes from compiling with gcc >=5. We're waiting on a new build toolchain that will fix this see #119 .
A temporary fix should be to build in this docker container which will make it compatible with the tensorflow pip package:
https://github.com/tensorflow/addons/blob/master/CONTRIBUTING.md#development-environment

Thanks, the docker container worked, and I found the culprit as well. It seems that our own op loader generates some dynamic code using exec, which discards source code information. I'll have a fix shortly to handle those properly.
The warning is otherwise absolutely benign, but sorry for the annoyance it created!

Thanks for looking into this! No real annoyance... most of our users at the moment understand we're still working out some stuff. After TF2 release I'd imagine this be more problematic.

I just checked the test with the nightly and the warnings should go away now - can you confirm?

Confirmed. Thank you for your help!

Hello
I am trying to get started with tensorflow-addons, and currently receiving the following error message:
 from tensorflow_addons import seq2seq
  File "/home/john_tukey/miniconda3/envs/tf20py36/lib/python3.6/site-packages/tensorflow_addons/__init__.py", line 75, in <module>
    from tensorflow_addons import image
  File "/home/john_tukey/miniconda3/envs/tf20py36/lib/python3.6/site-packages/tensorflow_addons/image/__init__.py", line 22, in <module>
    from tensorflow_addons.image.distance_transform import euclidean_dist_transform
  File "/home/john_tukey/miniconda3/envs/tf20py36/lib/python3.6/site-packages/tensorflow_addons/image/distance_transform.py", line 26, in <module>
    get_path_to_datafile("custom_ops/image/_image_ops.so"))
  File "/home/john_tukey/miniconda3/envs/tf20py36/lib/python3.6/site-packages/tensorflow/python/framework/load_library.py", line 61, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: /home/john_tukey/miniconda3/envs/tf20py36/lib/python3.6/site-packages/tensorflow_addons/custom_ops/image/_image_ops.so: undefined symbol: _ZTVN10tensorflow14kernel_factory17OpKernelRegistrar18PtrOpKernelFactoryE
libgcc_s.so.1 must be installed for pthread_cancel to work

Packages installed from pip
$ conda list
# packages in environment at /home/john_tukey/miniconda3/envs/tf20py36:
#
# Name                    Version                   Build  Channel
absl-py                   0.7.1                    pypi_0    pypi
artifacts                 20190320                 pypi_0    pypi
astor                     0.7.1                    pypi_0    pypi
ca-certificates           2019.1.23                     0
certifi                   2019.3.9                 py36_0
gast                      0.2.2                    pypi_0    pypi
google-pasta              0.1.6                    pypi_0    pypi
grpcio                    1.20.1                   pypi_0    pypi
h5py                      2.9.0                    pypi_0    pypi
keras-applications        1.0.7                    pypi_0    pypi
keras-preprocessing       1.0.9                    pypi_0    pypi
libgcc-ng                 8.2.0                hdf63c60_1
markdown                  3.1                      pypi_0    pypi
numpy                     1.16.3                   pypi_0    pypi
openssl                   1.0.2r               h7b6447c_0
pip                       19.1.1                   py36_0
protobuf                  3.7.1                    pypi_0    pypi
python                    3.6.0                         0
readline                  6.2                           2
setuptools                41.0.1                   py36_0
six                       1.12.0                   pypi_0    pypi
sqlite                    3.13.0                        0
tb-nightly                1.14.0a20190511          pypi_0    pypi
tensorflow-addons         0.3.1                    pypi_0    pypi
tensorflow-estimator-2-0-preview 1.14.0.dev2019051100          pypi_0    pypi
termcolor                 1.1.0                    pypi_0    pypi
tf-nightly-2-0-preview    2.0.0.dev20190511          pypi_0    pypi
tk                        8.5.18                        0
werkzeug                  0.15.2                   pypi_0    pypi
wheel                     0.33.2                   py36_0
wrapt                     1.11.1                   pypi_0    pypi
xz                        5.2.4                h14c3975_4
zlib                      1.2.11               h7b6447c_3

Same story with other calls to
tf.load_op_library(     get_path_to_datafile("custom_ops/ [...] ))
Any advice @seanpmorgan @mdanatg ?
Thank you

Hi! Could you create a separate issue for this? I believe it is related to an ABI version mistmatch but we'll need some more information. Such as was addons and tensorflow both installed from pip? It'll be easier to troubleshoot in a separate thread because this isn't related.

Sure, creating a separate issue right now.

feature_extractor_url ="https://tfhub.dev/google/tf2-preview/mobilenet_v2/classification/2" #@param {type:"string"}

feature_extrator_layer = hub.KerasLayer(feature_extractor_url, input_shape=IMAGE_SHAPE+(3,))

model = tf.keras.Sequential([
  feature_extractor_layer,
  layers.Dense(image_data_train.num_classes, activation='softmax')
])

model.summary()

WARNING:tensorflow:Entity <bound method KerasLayer.call of <tensorflow_hub.keras_layer.KerasLayer object at 0x13180d390>> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method KerasLayer.call of <tensorflow_hub.keras_layer.KerasLayer object at 0x13180d390>> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: module 'gast' has no attribute 'Num'
WARNING: Entity <bound method KerasLayer.call of <tensorflow_hub.keras_layer.KerasLayer object at 0x13180d390>> could not be transformed and will be executed as-is. Please report this to the AutoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: module 'gast' has no attribute 'Num'

@ruifengma could you open a separate bug for these warnings? They're due to a different cause, happy to elaborate on a separate thread.


Andi downcases column names

From smartcitiesdata/andi 56
Per @rajones-hntb
When presented a dataset definition containing a schema, andi should downcase the column names to ensure a proper match when compared to results from Presto.
AC:

Andi downcases names in the schema

TN:

Make sure that the schema will work with forklift, reaper, valkyrie, and flair

Using this bad boi as a test case
Ohio Worker Classification - Census Tract 2006 - 2010 -- Total Workers (Workers 16 years and over)
https://discovery.dev.internal.smartcolumbusos.com/dataset/aashto/15c31c63_151a_4f6d_97e3_2a2d97621a4c

The migration of existing dataset definitions will take place in https://github.com/smartcitiesdata/smartcitiesdata/issues/238


Design database using Sequelize

Design the database for the project using Sequelize. This includes the following data:
Customers:

Company Name
Contact Name
Address
Phone Number
email address

Inventory Items:

Item name
Item inventory quantity
Item cost
Item selling price

Invoices:

Customer ID
Items purchases
Quantity purchased
Price
Total for Invoice

Database format is designed, and seed data is provided.


Add material/patternfly ui for function workflow

create a custom webview to guide function workflow. Use patternfly for the forms and components creation.
Patternfly: https://www.patternfly.org/v4/components/form/

vscode-wizard is the recommended way to built forms in VSCode. It has been used by several of our plugins (Kafka, RSP, Tekton). It should be the way to go unless there are good reasons to deviate


builder-client observe-latest doesn't exit when there are errors in the build

we observed some errors, such as: uploading failed and external package installation failed. these errors didn't show up in the builder-client, and it didn't exit (the server didn't close the connection).

I checked the code, and this is likely related to the use of Lwt_condition.broadcast. Not sure how to properly deal with this, maybe use a Lwt_stream.t?

Hmm. I have thought about this before but for other reasons. When builder-server is busy receiving many output lines from a builder-worker lines to the client are dropped if the client can't keep up. To me this has been annoying because it's often long multi-line error messages that are dropped. In a similar manner the "end" message may be dropped.
For the output lines I understand that we may want to drop output lines so as to not have an indefinitely(*) growing queue of output lines in memory, but it seems crucial that we don't drop the "end" message. Maybe we can use a queue that is bounded for output lines but "unbounded" for "end" messages (of which there will be at most one in a well-behaving program). I'm not sure how best to implement that.

fixed by #48 \o/


feat: re-triggering the build

dependency: none
PR summary

Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.29.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Automate opensha rupture set testing

As SRM team
we want to automate repetitive testing processes
so that they require less manual work to complete
Done when:

[x] wrap existing java rupture generator class to support setting, retrieving interesting attributes from test runs (e.g. number of subsections)
[x] output data is written to unique, temporary location
[x] provide a py4j so that we can do automation in a python driver script
[x] provide a simple python example to demonstrate use in test automation (separate project)

2 example python scripts added under src/python/automation.

[ ] add shasum to the build/libs/*.jar as check on the build artefact


CLOUDSTACK-8845: Fixed list snapshots command throwing exception

https://issues.apache.org/jira/browse/CLOUDSTACK-8845
Fixed list snapshots command throwing exception if any one of snapshot is not present on primary.
Bug got introduced because of KVM specific code.
Testing done
List snapashots working fine.

@anshul1886 Thanks for the fix! Could you please make this PR against 4.6? Once merged in 4.6, it will be fwd-merged to master. Thanks!

Created PR https://github.com/apache/cloudstack/pull/1166 for 4.6.

@anshul1886 Thanks, you can close this PR as the other one will go to both 4.6 and master.


Unable to read xml files under « partition directory »

Hello,
I'm trying to read XML files with your API and I'm get stuck with a problem I don't find any solution.
I would like to read XML files like other CSV files with basePath option.

I have some input files under /input/dhi=20201029120000/data.csv and /input/dhi=20201029130000/data.csv. data.csv is as follow:

Click to see data
id1
id2

When i'm running this command :

val paths = Seq("/input/dhi=20201029120000/data.csv", "/input/dhi=20201029130000/data.csv")
spark.read.option("basePath", "/input").csv(paths: _*).show(false)

I get the following output

+---+--------------+
|id |dhi           |
+---+--------------+
|id1|20201029120000|
|id2|20201029120000|
|id1|20201029130000|
|id2|20201029130000|
+---+--------------+

As long as I know, your API can't be called like spark.read.xml(paths: _*) (may be causing by #4 ?)
Now if i'm adapting this to my XML files :

I have some input files under /input/dhi=20201029120000/data.xml and /input/dhi=20201029130000/data.xml. data.xml is as follow :

Click to see data
<datas>
    <data>
        <id>id1</id>
    </data>
    <data>
        <id>id2</id>
    </data>
    <data>
        <id>id3</id>
    </data>
</datas>

When I'm running this command :

val paths = Seq("/input/dhi=20201029120000/data.xml", "/input/dhi=20201029130000/data.xml")
spark.read.options(Map("basePath" ->"/input", "rootTag" -> "datas", "rowTag" -> "data")).xml(paths.mkString(",")).show(false)

I'm getting the following ouput (i.e i'm missing the « partition » directory dhi)

+---+
|id |
+---+
|id1|
|id2|
|id1|
|id2|
+---+

When I'm running this command :

val paths = Seq("/input/dhi=20201029120000/data.xml", "/input/dhi=20201029130000/data.xml")
spark.read.options(Map("basePath" ->"/input", "rootTag" -> "datas", "rowTag" -> "data")).format("xml").load(paths: _*).show(false)

I'm getting an Exception with message 'path' must be specified for XML data.

I'm already using this method with CSV and JSON files (I suppose this works with all spark-native datasources) and it could be very cool to have it work on XML files too !
Please tell me if I'm missing something, if you have a workaround or any other questions !
Thanks !

I think you're right, it doesn't work, and yes probably has to do with DSv2 support. That I don't know how to change/fix but would be open to anyone trying. Do wildcards work in a single path? .../**/** or .../*/*?

No wilcards doesn't work :(
I'll have to wait that someone gets interestes into this problem unfortunately :(

I am provisionally closing this as a likely duplicate of #4 as you note. Yes it would be good to allow that to work.


How to draw the line on 2D image using the coordinate position data of this image?

Dear all,
I want to draw some lines on a image plotted by Pseudocolor, but seems hard to make. Does anyone know how to do that?
Maybe we could use the 2D line objects in Annotation instead, but it's a troublesome operation when I need draw many lines.

Maybe we could use the 2D line objects in Annotation instead, but it's a troublesome operation when I need draw many lines.

Yes, 2D annotation objects seem like the right choice. If you have many to create, consider doing so with Python Scripting


Update About Page

Description
Update the About page like it is in the Figma project.

The About page has been updated, and the accordion needed to be fixed as well as a change in terms of logic.


Modified Visual Basic's Yara rule

The ThunRTMain function is the one needed to startup a Visual Basic app (can be imported via ordinal), so if it's a VB executable is required to import it.
In addition, the use of ? wildcard may slow down the scanning.

Samples escaping previous rule:

7b8e1fcceefcff278937265d150e5216925aa4d6efc9b0a9c135ea14fae19b5e
98d3f5ce1cd8cdea7522e455c3aea6b71e3f63c953f2169d0791ececd0c97bfd

Lets run TC tests.

Thank you for the PR. Changes look good and all TC tests passed. Merging.


[DDW-511] Implement Trezor device passphrase handling

This PR introduces passphrase handling on Trezor devices.
JIRA
Improved design with a passphrase label: https://zpl.io/aRJ8xlK
There is an issue with Trezor-Connect. Sometimes the device does not receive a signal to request a passphrase on device. This is a know issue developers faced at the official Trezor Connect Popup: https://github.com/trezor/connect/issues/689
We are not using official popup so I posted a new issue on Trezor Connect repository: https://github.com/trezor/connect/issues/783 cc @darko-mijic @nikolaglumac
Dedicated QA:  @ManusMcCole
Todos

[x] Handle passphrase on wallet restore
[x] Handle passphrase  on TX send immediately after wallet restore
[x] Handle wallet actions when passphrase already entered in the session and cached
[x] Improve Trezor change events using constants from lib
[x] Prepare feature flow diagram
[x] Write testing checklist regarding to feature flow diagram
[x] Add additional label to Hardware Wallet status "Enter passphrase if needed"
[x] Add Japanese translations
[x] Check Trezor-Connect glitch / issue when passphrase not requested on the device
[ ] Add Screenshots with additional passphrase message

Testing Checklist

Slack QA thread
[ ] Restore a wallet with an empty passphrase and another one with a passphrase you set
[ ] Send transaction with passphrase protected hardware wallet
[ ] Delegate with passphrase protected hardware wallet

NOTE:
- This is Trezor ONLY feature.
For restoring /sending tx / delegating you can follow Flow Diagram I prepared.
Update your Trezor firmware to the latest version and set a passphrase on your device.
To set a Trezor passphrase please follow this documentation https://wiki.trezor.io/Passphrase
The passphrase will create another "account" you can use. To use / restore "original / old" HW just leave the passphrase empty and confirm public key export after that. Also, pay attention to the additional label added to HW status box ONLY for Trezor interactions
Trezor Passphrase Flow PDF
Trezor Passphrase Flow (1).pdf

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes
[ ] PR has default-sized Daedalus window screenshots or animated GIFs of important UI changes:

[ ] In English
[ ] In Japanese

[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

Test Cases
Scenario 1 - Create Blank passphrase wallet
Given that I am on Daedalus
And I have configured my Trezor device to passphrase enabled
When I pair the device to Daedalus
And I input a BLANK passphrase
Then a new wallet is created that is the same as the original wallet would have been with this device if passphrase was not enabled

Scenario 2 - Create passphrase protected wallet
Given that I am on Daedalus
And I have configured my Trezor device to passphrase enabled
When I pair the device to Daedalus
Then I am asked to provide a passphrase
And a new wallet is created that differs to the original wallet created with this device with no passphrase

Scenario 2 - Create transaction with passphrase protected wallet
Given that I have a passphrase protected wallet in Daedalus already restored
When I attempt to sign a transaction to send some tx
Then I am asked to provide passphrase
And it is successful

Scenario 3 - Delegate/redelegate with passphrase protected wallet
Given that I have a passphrase protected wallet in Daedalus already restored
When I attempt to delegate/redelegate
ThenI am asked to provide passphrase
And it is successful

Scenario 4 - Verify session must be disconnected to switch between passphrase protected and blank passphrase wallets
Given that I have BLANK passphrase protected(wallet1) wallet in Daedalus and a passphrase protected(wallet2) already restored
When I successfully sign a transaction with wallet 1
And I do not end the session by disconnecting device
And I attempt to sign a transaction/delegate with wallet2
Then Daedalus will give an error message “Disconnect and reconnect your hardware wallet to restart the process.”
And when I reconnect and follow the transaction steps the transaction will be successful

@DominikGuzei can you please help us fix the Netlify deployment issue: https://app.netlify.com/sites/daedalus-storybook/deploys/5ff6d4ff3d71d400071e129b 🙏

@DominikGuzei can you please help us fix the Netlify deployment issue: https://app.netlify.com/sites/daedalus-storybook/deploys/5ff6d4ff3d71d400071e129b 🙏

@tomothespian please merge latest develop 🙏

Any idea when this will be released?
At the moment, I would strongly discourage using Trezor with Daedalus due to this feature missing.

Any idea when this will be released?
At the moment, I would strongly discourage using Trezor with Daedalus due to this feature missing.

No clear ETA yet.

Hi @tomislavhoracek Tomislav. Good work.Here is the testing report for build 17175
For case 4 in your diagram when the user does not kill prevous HW session before interacting with the other wallet. Do we plan to have some text outlining this to users "To Interact with another HW device must be reconnected and session killed" . Or just show the error message See screenshot
When a user 1.opens Daedalus with no device connected, 2.Goes to send transaction and is asked to connect device, 3.Device is connected. Only for this scenario the user is asked on the Trezor device to "Please type your passphrase on the connected host" with no option on Daedalus to do so.screenshot. This message stays on the Trezor screen even after the transaction is cancelled.All other scenarios the user is asked to input on the Trezor device
Also when the user is being asked on the Trezor device to input passphrase on Daedalus is still says "Export the public key on your device". Do we plan to add a text step for "Please enter passphrase" ?

Hi @tomislavhoracek. I added some extra details to the report above via an edit

@ManusMcCole Thanks
1. For case 4 in your diagram when the user does not kill previous HW session before interacting with the other wallet. Do we plan to have some text outlining this to users "To Interact with another HW device must be reconnected and session killed" . Or just show the error message

For now we will use this message as I know.

When a user 1.opens Daedalus with no device connected, 2.Goes to send transaction and is asked to connect device, 3.Device is connected. Only for this scenario the user is asked on the Trezor device to "Please type your passphrase on the connected host" with no option on Daedalus to do so.screenshot. This message stays on the Trezor screen even after the transaction is cancelled.All other scenarios the user is asked to input passphrase on the Trezor device

I saw this just once (after like 50 tries) and it seems like "trezor-connect" glitch. Even if an event to enter passphrase ON DEVICE is sent, a device just ignores that.

On windows
When user follows steps 1-3.The same message flashes across the screen "Please type your passphrase on the connected host" and then the correct message asking user to input passphrase on the device appears. After the user inputs the passphrase a Transport is missing error appears.See screenshot

This is weird. We can check this one together

Also when the user is being asked on the Trezor device to input passphrase on Daedalus is still says "Export the public key on your device". Do we plan to add a text step for "Please enter passphrase" ?

I'm not sure that we can get a signal after passphrase is entered but I can check. Also, I'm not sure if we need to show that message?

cc @nikolaglumac @darko-mijic

Hi @tomislavhoracek
I found another issue on Windows only
When a user attempts to send a transaction or redelegate a wallet. On the confirm transaction screen the message "export the public key on your device" shows up for 1-2 seconds and if a device is not connected in that time exporting the public key fails. See screenshot

@ManusMcCole More details please 🙏 https://input-output-rnd.slack.com/archives/GGKFXSKC6/p1616076338013400?thread_ts=1616076249.013300&cid=GGKFXSKC6
Also, can you reproduce that on the latest develop branch build?

Hi @tomislavhoracek. The issues became very difficult to reproduce and pin down. I am investigating further now. Many attempts yesterday evening and I could no reproduce. I will send you an update soon

Hi @tomislavhoracek. Here is my update
For the error linked below I had yesterday on Mac I found that it was caused by using my personal Trezor Model One inbetween testing this PR on Daedalus. Trezor Suite must remain open for it to cause issues. Perhaps we should make it clear to users that Trezor Suite must be closed before using Trezor with Daedalus by adding it to the Connecting Documents How to use Ledger and Trezor HW with Daedalus and How to pair Trezor T with Daedalus

For Windows I cannot reproduce the issue after reinstalling 5 times and testing the scanarios many times.(No other USB devices were introduced yesterday when this error arose). However it is not reproducible either so not an issue right now. (I will keep an eye out for it however). Would it be possible the error on Mac somehow affected the device in someway that it caused an issue on Win ?
Testing now and it is working really good with fast flow almost ready to approve. Still doing some more tests

On Mac
When a user 1.opens Daedalus with no device connected, 2.Goes to send transaction and is asked to connect device, 3.Device is connected. Only for this scenario the user is asked on the Trezor device to "Please type your passphrase on the connected host" with no option on Daedalus to do so.screenshot. This message stays on the Trezor screen even after the transaction is cancelled.All other scenarios the user is asked to input passphrase on the Trezor device
On windows
When user follows steps 1-3.The same message flashes across the screen "Please type your passphrase on the connected host" and then the correct message asking user to input passphrase on the device appears. After the user inputs the passphrase a Transport is missing error appears.See screenshot

@ManusMcCole I updated a PR description with notes about Trezor Connect issue. Please take a look
Also, I have one more challenge for you 😄
A New label "Enter passphrase if needed" is added to the hardware wallet status box. This label should be visible on "Public key export" and "Transaction signing" ONLY when you are interacting with Trezor device. For Ledger is the same as before.
Please try to next:

Restore Ledger / Trezor
Sign and send TX with Ledger and Trezor
Delegate with Ledger and Trezor
Undelegate Ledger and Trezor from settings

For 1-4 Check if the new (additional) label is correctly displayed!

@alexander-rukin @daniloprates please review this one 🙏

@tomislavhoracek please add screenshots 🙏

@daniloprates please review this PR 🙏

@alexander-rukin @daniloprates please review this one 🙏

Can I test it through netlify ?

Hi @tomislavhoracek. Nice work. Here is the testing report. I have a Ledger Nano S Hardware wallet present in Daedalus and also 2 Trezor Model T Hardware Wallets(One with a blank passphrase W1(blank) and W2(passphrase protected). I only ever have one hardware device connected at any one time
On all OS.I found one scenario where the new additional banner "Please enter passphrase if needed" does not show up. When a user signs a transaction with one wallet(blank passphrase or passphrase protected wallet) and does not disconnect the device to end session then trys to sign a transaction/delegate with the other wallet (Case 4 in UML and test scenario 4). The correct error message appears and the user is asked to reconnect device. When the user is then asked to enter passphrase the new banner is not shown but the message from before only asking user to "Export Public Key"
On Mac about 30% of the time the new banner is not visible only the "Export Public Key message". This happens with the same likelihood on scenario 5 and 6 equally as posted in the test scenarios

@alexander-rukin @daniloprates please review this one 🙏

Can I test it through netlify ?

@alexander-rukin Yes you can. Let me write paths to netlify components

@alexander-rukin you can check PAIR dialog in storybook
Hardware Wallets -> Hardware wallet connect Trezor step 3
For tx send (confirmation dialog) only Hardware Wallet status box with "connect hardware wallet" is presented.

@ManusMcCole Recognizing Trezor devices is improved. Can you please take a look again 🙏

@daniloprates I made the improvements you mentioned. Can you take a look?

Amazing work 💯✨

you did not address hidden wallet passphrase issue

@europound can you explain?

In your report i did not find a resolve to main issue
Hidden wallet passphrase prompt is not popping up

The device itself will prompt you to enter the passphrase (on it’s own display). No prompts will pop up in Daedalus. This is the way.

and yet again you missed the bug
very sad your lack of attention to details

I am afraid I don't understand what you are trying to point out...
We have implemented the support for https://wiki.trezor.io/Passphrase which means you will be able to use your hidden wallet on Trezor Model T. There is no and there will be no support for Trezor Model One in Daedalus (nor any other Cardano wallet).

@europound this feature has not yet released so you won't be able to give it a go just yet. Thanks for the patience!

If your Trezor Model T device has the passphrase set then Daedalus will trigger it to ask you for it on the device itself:

my bad Trezor LCD T i meant
have you actually physically with your own hands took trezor lcd T, connected to pc, launched daedalus wallet then tried to stake with hidden wallet?
If not then you will never understand the issue and everything else is pointless chit chat
and thus we pay tax 3% and some and give up stake control to adalite 3rd paties

@europound I understand what you are pointing out - so far it was impossible to pair your Trezor Model T with Daedalus in case you had a passphrase (hidden wallet) set on the Trezor device.
With the change we implemented in this pull-request this limitation has been removed and you will be able to use your Trezor Model T with hidden wallets in the next Daedalus version (it has not been released yet).
Thanks for the patience and sorry for the trouble! 🙏

looking forward to it

Hi, I was excited to see this added to 4.0.3 but this feature still isn't working correctly. When you try to pair your Trezor + Passphrase with the Daedalus wallet - a prompt now ask you to enter your passphrase, but no browser instance is launched that allows you to input the passphrase. The instructions too are not very helpful, and don't address the passphrase either. I saw a post somewhere else advising the user to remove all wallets first, but that isn't really a solve more of a workaround. Would suggest re-opening this ticket.

There is no browser prompt. You should be asked to enter the passphrase on the device itself.

Thanks @nikolaglumac - I tried again and this time it worked!

Bad. Add keyboard input in case Hidden password is long and complicated


Blip feature extractor API

Thanks for the great work. I have some questions about the BLIP feature extractor interface.

In the example code, you wrote

# torch.Size([1, 12, 768]), use features_multimodal[:,0,:] for multimodal classification tasks

What are the other channels [:, 1:12, :] useful for?

In the example code of the API, there is another attribute called image_features (link), but they are not available. Can you comment on the difference between image_embeds and image_features, and how to access the latter?

print(features_image.image_embeds.shape)
print(features_image.image_features.shape)

Thanks!

Hi @jhwang7628 , thanks for your interest.

The features with index 0 are of the CLS token. The rest correspond to other text token positions. Although we use [CLS] by default, you may also use others for your applications, e.g. taking mean of them.
It turns out the naming of in-line example code is deprecated. Can you see README for example as of now? In the meantime, we will update the naming accordingly. Re your question, embeddings are directly obtained from feature networks, while embed_proj are normalized embeddings, obtained by further projecting embeddings on the low-dimensional space. embed_proj are useful for calculating feature similarities in a normalized space.

Hi @dxli94 ,
Thanks for getting back to me.
I am doing some testing and it seems that the full feature [CLS] channel (features.image_embeds[:,0,:]) does not have on-par performance with the projected features (features.image_embeds_proj[:,0,:]). Using it for video retrieval task on MSRVTT, the performance is 0.2% compared to 31%. That makes me wonder if I am doing anything suspicious. Any guess?


[6.0.2] Migrations, ->default() for tinyInteger/boolean fails

Laravel Version: 6.0.2
PHP Version: 7.3.8-1+ubuntu18.04.1+deb.sury.org+1
Database Driver & Version: MySQL 5.7.27-0ubuntu<IP_ADDRESS>

Description:
When trying to set default value for boolean/tinyInteger i get syntax error (1064).
Works fine in previous versions (<= 6.0.1) .
Steps To Reproduce:
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;

Schema::create('test', function (Blueprint $table) {
$table->boolean('test')->default(true);
});

Produces error
Illuminate/Database/QueryException with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '('1')) default character set utf8mb4 collate 'utf8mb4_unicode_ci'' at line 1 (SQL: create table `test` (`test` tinyint(1) not null default ('1')) default character set utf8mb4 collate 'utf8mb4_unicode_ci')'

@taylorotwell / @driesvints Confirmed, downgrading to 6.0.1 made my migrations / tests work again

I also have this problem on my project. This is due to https://github.com/laravel/framework/pull/29878
ping @browner12

can you guys respond with your MySQL versions?

I am running MySQL 5.7.25

seems like this is due to MySQL 5.
I'm hoping we can find a way to make this change backwards compatible over MySQL versions, so we don't have to start adding checks for the version.

My project is running MySQL 5.7.19.
@browner12 What about instead of adding checks for MySQL versions, add checks so that the parentheses are added only when they are really required?

Same problem here. I just updated the homestead box to the very last 8.1.0 version which came with MySQL 5.7.27 and installed the last release (6.0.2) of Laravel.
According to the MySQL 5.7.27 documentation explicit default should not be in parentheses and the BLOB, TEXT, GEOMETRY, and JSON data types cannot be assigned a default value.
https://dev.mysql.com/doc/refman/5.7/en/data-type-defaults.html#data-types-defaults-explicit
According to the MySQL 8.0 documentation the BLOB, TEXT, GEOMETRY, and JSON data types can be assigned a default value only if the value is written as an expression, even if the expression value is a literal:
CREATE TABLE t2 (b BLOB DEFAULT ('abc'));
This is permitted (literal default specified as expression):
CREATE TABLE t2 (b BLOB DEFAULT 'abc');
This produces an error (literal default not specified as expression):
https://dev.mysql.com/doc/refman/8.0/en/data-type-defaults.html
I think the placement of hard coded parentheses in the MySqlGrammer::modifyDefault (Laravel 6.0.2) was a mistake. Parentheses should be set according to the column type.
    protected function modifyDefault(Blueprint $blueprint, Fluent $column)
    {
        if (! is_null($column->default)) {
            return ' default ('.$this->getDefaultValue($column->default).')';
        }
    }

@X-Coder264 the problem is that v5 cannot use the parentheses at all, so it's dependent on the version, not the field type.
I've asked Taylor to revert the commit, and proposed another solution in my original PR.

We're gonna revert this.

We released v6.0.3 which reverts this

It still doesn't work with v6.0.3. Below is the code
Schema::table('product_variants_exports', function (Blueprint $table) {
      $table->boolean('is_variant_product')->default(false)->after('product_variant_id');
});

The error from MySQL 5.7.25 is
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '('0') after `product_varian
  t_id`' at line 1 (SQL: alter table `product_variants_exports` add `is_variant_product` tinyint(1) not null default ('0') after `product_variant_id`)


Improve GHSA-fjq5-5j5f-mvxh

Updates

Affected products

Comments
Several other components are also affected as a result of cloning (copying source code) or shading (copying source code and renaming packages). Proof-of-Vulnerability projects with tests to verify the presence of the CVE can be found here: https://github.com/jensdietrich/xshady-release/.
See https://github.com/github/advisory-database/pull/2258, especially https://github.com/github/advisory-database/pull/2258#issuecomment-1569073245, for more details.

For org.apache.servicemix.bundles:org.apache.servicemix.bundles.commons-collections it appears that there are three versions on maven
https://central.sonatype.com/artifact/org.apache.servicemix.bundles/org.apache.servicemix.bundles.commons-collections/3.2.1_3/versions
All of which look to have a copy of the commons collections to me.

Thanks @darakian, I'll look into those other versions today and try to flesh this PR out a bit more 👍

Hi @darakian,
You were right -- those 2 versions of org.apache.servicemix.bundles:org.apache.servicemix.bundles.commons-collections are also vulnerable. Thank you for spotting that! They weren't picked up automatically by our tool because their source code jars were missing from Maven Central Repo. I've added pom.xmls to repro these vulnerabilities in the xshady-release repo, and added extra versions in fb36249. Gory details in https://github.com/jensdietrich/shadedetector/issues/78.
In 6fd3b9e I've also added some more source repo metadata, and a reference to the paper preprint.
Is this PR now good to go do you think?

Hi @darakian,
To get past the "Explicitly listing more than one affected version is not currently supported. Use range events instead" error, 5c57656 removes the complete list of explicit versions -- this seems less misleading than leaving any one of the 3 there.
It would be great if we could somehow specify explicitly that we know these 3 versions are definitely vulnerable, because I think merging the PR in its current state will only make the GHSA suggest this, based on:

these 3 versions exist at the time of this GHSA update
I didn't include a fixed or last_affected entry
the assumption that I would have tested all versions available at the time of the GHSA update

Is there a way to explicitly specify this? I didn't see anything that does exactly this at https://ossf.github.io/osv-schema/. I could just add "last_affected": "3.2.1_3", but https://ossf.github.io/osv-schema/#last_affected-vs-fixed-example says

The last_affected field is typically assigned at the time of discovery and assumes the vulnerability will be addressed in the following version.

but I don't know if that assumption is reasonable.
Otherwise I think this PR is ready?

Given the range you removed I could manually add back >= 3.2.1_1, <= 3.2.1_3

Please do! (For reference is that something I could have done myself in this PR, by making new affected products for each single version as you suggested in the previous sentence? Or something only you can do?)
Thanks @darakian!

For reference is that something I could have done myself in this PR, by making new affected products for each single version as you suggested in the previous sentence? Or something only you can do?

It is, though I can also change it and as I'm trying I'm now remembering that our version logic can't handle underscores 🤦.
I went ahead and bounded the versions you specified though. Let me know if you disagree and we can edit them 👍

Thanks @darakian, the version ranges for org.apache.servicemix.bundles:org.apache.servicemix.bundles.collections-generic and org.apache.servicemix.bundles:org.apache.servicemix.bundles.commons-collections look good to me given the underscore restrictions, but I noticed a couple of other issues on the published advisory page:

None of the new references (e.g., https://github.com/jensdietrich/xshady-release/tree/main/CVE-2015-7501) appear
net.sourceforge.collections:collections-generic has both "= 4.0.1" and "= 4.01" listed -- the former should not be there

Welp, that's a fat finger on my end 🤦
I culled the https://svn.apache.org/... links since those can be inferred from the package and version info, but otherwise hopefully good 👍

Looks good now, thanks!