[ML Text Recognition] Drawing shapes for slightly rotated images (On captured image)

Hello!
Background
I am using Firebase text recognition to recognise text in images. However I have the following problem:

As you can see in the image if the target image is slightly rotated then the frames are being overlapped. What I would like to achieve is for the blue rectangles to follow the text rather than having a horizontal shape. Currently I do drawing in the following manner, inside of my processText(...) method:
// Paragraphs
for block in text.blocks {
    // Lines
    for line in block.lines {
        drawFrame(line.frame, transform: transform)
    }

The drawFrame(...) method does the following:
private func drawFrame(_ frame: CGRect, transform: CGAffineTransform) {
    let transformedRect = frame.applying(transform)
    UIUtilities.addRectangle(transformedRect, to: self.annotationOverlayView)
}

So, once the line element is transformed I do the drawing:
public static func addRectangle(_ rectangle: CGRect, to view: UIView){
    let rectangleView = UIView(frame: rectangle)
    rectangleView.layer.borderColor = UIColor(hex: Constants.colorDarkBlue).cgColor
    rectangleView.layer.borderWidth = Constants.lineWidth
    rectangleView.bounds = rectangleView.frame.insetBy(dx:-3, dy: -3)
    view.addSubview(rectangleView)
}

What did I try?
I tried looking at the addShape(...) method in this repository: https://github.com/firebase/quickstart-ios/blob/5b752734233d4be625ece2747385d4974f2fc07f/mlvision/MLVisionExample/UIUtilities.swift#L52
I am 100% confident that `addShape(...) is the method to use but I I can see in the example during the live run example it is capable of adjusting the shape of the highlight regardless of how much I rotate the image.
So I tried to do the following:
This is inside my processResults(...) method. The captureImageView is the view in which I put the captured image on and then do processing on it.
// Paragraphs
for block in text.blocks {
    // Lines
    for line in block.lines {
        let points = self.convertedPoints(from: line.cornerPoints, width: capturedImageView.bounds.width, height: capturedImageView.bounds.height)
        print("Points: \(points)")
        UIUtilities.addShape(
             withPoints: points,
             to: self.annotationOverlayView,
             color: UIColor.orange)

Similar to the Repository I link above, I copied over the addShape(..) into my class. I also coppied convertedPoints and normalizedPoint
You can see in my  convertedPoints(...) I am not actually using the normalizedPoint constant (as I am not sure what do there).
private func convertedPoints(
    from points: [NSValue]?,
    width: CGFloat,
    height: CGFloat
    ) -> [NSValue]? {
    return points?.map {
        let cgPointValue = $0.cgPointValue
        let normalizedPoint = CGPoint(x: cgPointValue.x / width, y: cgPointValue.y / height)
        //let cgPoint = previewLayer.layerPointConverted(fromCaptureDevicePoint: normalizedPoint)
        let cgPoint = capturedImageView.frame.origin
        let value = NSValue(cgPoint: cgPoint)
        return value
    }
}

And this is my normalizePoint(...) method
private func normalizedPoint(
    fromVisionPoint point: VisionPoint,
    width: CGFloat,
    height: CGFloat
    ) -> CGPoint {
    let cgPoint = CGPoint(x: CGFloat(point.x.floatValue), y: CGFloat(point.y.floatValue))
    var normalizedPoint = CGPoint(x: cgPoint.x / width, y: cgPoint.y / height)
    normalizedPoint = capturedImageView.frame.origin
    return normalizedPoint
}

To Lazy to Read
I would like to adjust the shape of the highlight to adapt to slightly rotated images. Similar to the example mlvision application (With live AVFoundation camera run) in this repository.

The mlvision sample doesn't do any slanted highlighting as far as I know. The highlighting logic is here. You may have better luck asking how to draw slanted rectangles on StackOverflow.

@morganchen12 It does! The following image illustrates that https://imgur.com/vTd8LqE This is using the MlVision example from this repository. At the top right there is a camera icon. The difference is, it does such detection for AVPreviewLayer, so the image is constantly being buffered. But what I would like to do is to only use it with an image I have captured.
Additionally, I have already asked my question on StackOverflow as well.

Any update on this, or should I just close the issue?


Cannot build portus on mac using docker-compose

I am following the instructions at https://github.com/SUSE/Portus/wiki/Docker-Compose-Environment.
I cloned the portus repo to a local directory on my mac.  I change directories into that directory and I issue the command:
docker-compose up
It indicates that it installs 77/77 componenets and then I get the following output:
(77/77) Installing: patterns-openSUSE-devel_basis-20170518-6.1.x86_64 [.....done]
Output of kbd-2.0.3-3.14.x86_64.rpm %posttrans script:
Please run mkinitrd as soon as your system is complete.
Don't run Bundler as root. Bundler can ask for sudo if it is needed, and
installing your bundle as root will break this application for all non-root
users on this machine.
Warning: the running version of Bundler is older than the version that created the lockfile. We suggest you upgrade to the latest version of Bundler by running gem install bundler.
Your Ruby version is 2.4.1, but your Gemfile specified >= 2.4
ERROR: Service 'portus' failed to build: The command '/bin/sh -c zypper ref &&     zypper -n in --no-recommends ruby2.4-devel ruby2.4-rubygem-bundler            libxml2-devel nodejs libmysqlclient-devel postgresql-devel libxslt1 &&     zypper -n in --no-recommends -t pattern devel_basis &&     bundle install --retry=3 &&     zypper -n rm wicked wicked-service autoconf automake            binutils bison cpp cvs flex gdbm-devel gettext-tools            libtool m4 make makeinfo &&     zypper clean -a' returned a non-zero code: 18
I see that these commands are in the Dockerfile and it looks like they were executed just fine in the beginning of the process unless it is not actually being run at that point.  I saw the following output in the beginning after I first issued docker-compose up:
Step 7/8 : RUN zypper ref &&     zypper -n in --no-recommends ruby2.4-devel ruby2.4-rubygem-bundler            libxml2-devel nodejs libmysqlclient-devel postgresql-devel libxslt1 &&     zypper -n in --no-recommends -t pattern devel_basis &&     bundle install --retry=3 &&     zypper -n rm wicked wicked-service autoconf automake            binutils bison cpp cvs flex gdbm-devel gettext-tools            libtool m4 make makeinfo &&     zypper clean -a
Is there an issue with the zypper command?  Just to repeat I am using docker-compose on a mac.

Did a little more debugging by breaking down the long RUN command into parts to see what part actually is causing the error.
I found that it is the bundle portion:
ERROR: Service 'portus' failed to build: The command '/bin/sh -c bundle install --retry=3' returned a non-zero code: 18

I modified the Docker file RUN command to get a better idea of which part of the piped command is causing the error.  Modified the RUN command by splitting it up to look like the following:
RUN zypper ref &&
zypper -n in --no-recommends ruby2.4-devel ruby2.4-rubygem-bundler
libxml2-devel nodejs libmysqlclient-devel postgresql-devel libxslt1 &&
zypper -n in --no-recommends -t pattern devel_basis
RUN bundle install --retry=3 &&
zypper -n rm wicked wicked-service autoconf automake
binutils bison cpp cvs flex gdbm-devel gettext-tools
libtool m4 make makeinfo &&
zypper clean -a
I still received the same ERROR as expected, but I think it is safe to say that the portion of the RUN command that is failing is the bundle portion.  Here is the error message:
Step 8/9 : RUN bundle install --retry=3 &&     zypper -n rm wicked wicked-service autoconf automake            binutils bison cpp cvs flex gdbm-devel gettext-tools            libtool m4 make makeinfo &&     zypper clean -a
---> Running in 042795f84a95
Don't run Bundler as root. Bundler can ask for sudo if it is needed, and
installing your bundle as root will break this application for all non-root
users on this machine.
Warning: the running version of Bundler is older than the version that created the lockfile. We suggest you upgrade to the latest version of Bundler by running gem install bundler.
Your Ruby version is 2.4.1, but your Gemfile specified >= 2.4
ERROR: Service 'portus' failed to build: The command '/bin/sh -c bundle install --retry=3 &&     zypper -n rm wicked wicked-service autoconf automake            binutils bison cpp cvs flex gdbm-devel gettext-tools            libtool m4 make makeinfo &&     zypper clean -a' returned a non-zero code: 18
Is there an issue with running bundle the way it is being run as:
/bin/sh -c bundle install --retry=3

Hi @dkeidel397, sorry for the delay. The problem is because ruby2.4-rubygem-bundler package in openSUSE has an old version of bundler that doesn't support the constraint ruby >= 2.4.
So, for now what you can do is just remove that line from the Gemfile and run docker-compose build portus again. That should work. Sorry for the inconvenience.

No problem.  Thanks for getting back to me.  While I was waiting for a response I changed the line in the Gemfile to ruby "2.4.1" (and also removed the line completely from the Gemfile) which got me past the ERROR, but now when I run docker-compose up, I am getting ERROR messages that complain of things like:
Fetching gem metadata from https://rubygems.org/........
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rubygems.org/..
Gem::RemoteFetcher::FetchError: SSL_connect SYSCALL returned=5 errno=0 state=SSLv3 read server session ticket A (https://rubygems.org/gems/rake-12.2.1.gem)
An error occurred while installing rake (12.2.1), and Bundler cannot continue.
Make sure that gem install rake -v '12.2.1' succeeds before bundling.
The weird thing is that the gem it mentions is different every time I run docker-compose up OR docker-compose build portus.
Seems others have complained about this issue here:
http://railsapps.github.io/openssl-certificate-verify-failed.html
I am a little fearful of modifying the ssl setting on my mac, so wanted to see what you or others thought before I go down that road.

That one I never saw. 😕


Clarified minimal Android version in readme

Only the android target requirement is 8.1.  You can use this library as far back as Android v5.  I'll update the docs at some point in the future.  Thanks


Only send image loading chunk events from the worker isolate if there are listeners registered.

fyi @tvolkert, this is I believe how sending of chunk events can be made dependent on whether they are any listeners or not.

Nice!  I was originally thinking of plumbing it all the way down to whether we passed a non-null value to onBytesReceived, but this method is way less invasive.
Let's keep this isolated in its own change so you can land https://github.com/flutter/flutter/pull/34188, then apply this as a follow-on change.


Allow search to filter by title

Since this is not merged in yet. we need to open up a card before searching otherwise there will be an error
https://github.com/user-attachments/assets/2a53ddee-60d7-499f-8d0a-716c6a2b6cec

@burieberry I removed "every" and took your question to remove card-type


look up signing key via AffiliateService when erroring envelope

The ol' different signing/encryption keys strikes again!
Homepoint originator in prod has different signing/encryption/auth keys, and when the servicing affiliate rejected an envelope, the error failed to propagate back to the Homepoint originator, since the incoming mail public key is the encryption key, while the envelope table key is the signing key. This ensures the correct key is used when processing incoming mail

ensures correct key is used for envelope lookup, preventing error from being dropped

Codecov Report

Merging #102 (75ab1e2) into main (4c2af7a) will increase coverage by 0.06%.
The diff coverage is 45.45%.

@@             Coverage Diff              @@
##               main     #102      +/-   ##
============================================
+ Coverage     12.07%   12.13%   +0.06%
  Complexity       37       37
============================================
  Files            66       66
  Lines          3471     3477       +6
  Branches        368      368
============================================
+ Hits            419      422       +3
- Misses         2991     2994       +3
  Partials         61       61

Impacted Files
Coverage Δ

...in/io/provenance/engine/service/EnvelopeService.kt
41.39% <45.45%> (+0.21%)
:arrow_up:

Continue to review full report at Codecov.

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


NullPointerException can occur when Riot API has issues

java.lang.NullPointerException
at main.java.riotapi.RiotApi.getSummonerByName(RiotApi.java:888)
This should probably be prevented by checking if the collection is null, and returning a null object rather than trying to iterate it.

{code}
Map<String, Summoner> summoners = SummonerApi.getSummonersByName(getEndpoint(), getRegion(), getKey(), summonerName);
for(Summoner summoner: summoners.entrySet()) {
return summoner.getValue();
}
return null;
{code}

That would also throw NullPointerExceptions, if SummonerApi.getSummonersByName() returns null.

I don't think getSummersByName should ever return a null, If it can't find anything, it should return an empty map. Going to look at the code a bit more.

You are talking about the getSummonerByName methods that return a Summoner rather than a Map with a single Summoner correct?

Also -
Do you know the specifics of when this occurs?
If the API returns any error codes, it should throw a RiotApiException.
So I'm guessing the API is returning a 200 response code, with no data?

Yes, the mentioned issue was found in the method to receive a single summoner.
I know about the RiotApiException, but sometimes there are some other issues with the connection, for example:
java.net.ConnectException: Connection timed out
or
javax.net.ssl.SSLHandshakeException: Remote host closed connection during handshake
or
java.io.EOFException: SSL peer shut down incorrectly
I get tons of exceptions every day, and it's hard to say which exact exception leads to which exact behavior, especially since I have a pretty big backend application using this API, running billions of requests every month.

If adding the null checks solve the issues, I'll go ahead and add them since it wouldn't really effect anything else.
I was just wondering about what the root cause is, because it might be better to just handle these cases in the request code than adding null check to everything.

True. It seems that Riot has some really major issues for the Korean region lately: https://developer.riotgames.com/status
Maybe some of the KR requests did throw some kinds of ConnectExceptions and SSLExceptions which ended up with the summoner object being null?
In Request.java you currently catch IOException (ConnectException and SSLException extend IOException) to log them, and then return null. Maybe we shouldn't return null here, but throw a new kind of RiotApiException instead? Let's call it RiotApiException.IOEXCEPTION and throw it, instead of returning null. (This idea was inspired by this pull request: https://github.com/rithms/riot-api-java/pull/26)
I'll see if I can do some more tests with this issue.

I have tested this in a live environment and it turns out that those IOExceptions are much easier to handle, when the API throws a proper Exception instead of returning a random null object (which can actually break the API, as shown in the example mentioned in the original post).
I have added a pull request (https://github.com/rithms/riot-api-java/pull/32) to change this behavior, aswell as adding a new constant Season.CURRENT.
That would also mark pull request https://github.com/rithms/riot-api-java/pull/26 as obsolete. (Thanks for the idea though, dshen6!)

PR https://github.com/rithms/riot-api-java/pull/32 has been merged, adding a new RiotAPIException, RiotAPIException.IOEXCEPTION.
This issue should be fixed now, without the need to add additional null checks.
Let me know if you run into any further issues.

Even though this was closed, I still don't see how the code would result in a null object being returned from having the execute method return an Exception. I think rithms might be correct that the request did execute correctly and GSON returned a null for some reason, but if that is the case this is not fixed and might still have random nulls being returned in places where they are not supposed to appear.

I added null checks to every Gson result in this Pull Request: https://github.com/rithms/riot-api-java/pull/33
If Gson returns null for whatever reason, I throw a RiotException.PARSE_FAILURE.

Linnun's PR https://github.com/rithms/riot-api-java/pull/33 should now handle any cases in which GSON returns a null.
I'm going to close this issue again. If you run into anymore problems, go ahead and comment here or open a new issue.


Speech Recognition fails on Blazor Server

The service works fine on Web Assembly but not on Server. When I try to use Speech Recognition on Server, the code throws an error "aborted". This error is coming from the browser's API and I get no other information.
I am using the package Blazor.SpeechRecognition for the Server implementation.

Hi @EdCharbeneau!
Luckily, I have a sample app written that verifies all this functionality. See the https://github.com/IEvangelist/blazorators/blob/main/samples/BlazorServer.ExampleConsumer/Pages/ListenToMe.razor.cs. You could clone the repo and just set this solution as the startup and verify it yourself - I just did locally.
I do know that there's lots of issues with server-side and @rendermode. Is it trying to do SSR, like is prerendering enabled? Similar to this issue: https://github.com/IEvangelist/blazorators/issues/59#issuecomment-2418329584

I'll take a look, thanks 👍

I'll take a look, thanks 👍

Just let me know what you find out.

It appears that when you use cancel, that it may be normal for the "aborted" error to be triggered by the API. For some reason I'm only getting it on Blazor Server and not Client.
await SpeechRecognition.CancelSpeechRecognitionAsync(true) // error "aborted"

I also get an "error" when the service times out vs. being cancelled by the user.
"no-speech"

Here is what I ended up with
    public Task OnError(SpeechRecognitionErrorEvent args)
    {
        switch (args.Error)
        {
            case "audio-capture":
            case "network":
            case "not-allowed":
            case "service-not-allowed":
            case "bad-grammar":
            case "language-not-supported":
                throw new Exception($"Speech: {args.Error}: {args.Message}");
            case "no-speech":
            case "aborted":
                break;
        };
        StateHasChanged();
        return Task.CompletedTask;
    }

Ah, you're passing in true for the isAborted... so that literally translates into the abort being called. Instead, pass false which will simply stop recognition.


fix(auth): bump tgtg to 0.18.0 to fix auth issue

https://github.com/ahivert/tgtg-python/issues/325

Once I also removed the user_id argument from the api call, the integration works again. Removing all the other lines referencing user_id required changing configuration.yaml, which I guess is a breaking change.
https://github.com/Chouffy/home_assistant_tgtg/blob/915afebb39fc390d0c767cee1ae6c247d2b85744/custom_components/tgtg/sensor.py#L70-L75

Thanks, I'll release an update later today.

@ThanhCN0 The new release should work both with as without user_id in the configuration.
The user_id is now optional (but can only be removed after updating).
Feel free to test it out.


Old images on Movies example app listings

To reproduce, search for a movie and browse into it, then back out of the movie view to the search, then delete your search terms by holding down the delete key (on your laptop while running in simulator). It seems to be timing dependent but with a few tries I was able to generate the screenshots below.

Maybe this is throttling from the data API? Maybe caching would help?

The "keep a" experience is probably better at an infrastructural level because you can always get the "blank" way by adding a key to the Image equal to the src.

@vjeux I've tried using <Image defaultSource=... /> to show some kind of a loading placeholder image, but it shows the old image indefinitely. I'll get a minimal repro set up for you.
Meanwhile is there workaround we can use to blank the image and force loading the correct one?

workaround
/* @flow */
'use strict';

var React = require('react-native');
var { Image } = React;

// Hack to work around https://github.com/facebook/react-native/issues/89
//
// Render an <Image /> normally.
// When react reuses the component with a new url, force react to render a
// blank image once before rendering the correct url later.

var DelayedImage = React.createClass({
  propTypes: Image.propTypes,
  getInitialState(): { showImage: boolean } {
    return { showImage: true };
  },
  componentWillReceiveProps: function(nextProps: any) {
    if(this.props.source.uri != nextProps.source.uri) {
      this.setState({ showImage: false });
      setTimeout(() => this.setState({ showImage: true }), 0);
    }
  },
  render: function(): React.Component {
    return <Image {...this.props} source={{uri: this.state.showImage ? this.props.source.uri : null}} />
  },
});

module.exports = DelayedImage;

The issue here now is that the images change but the text does not

@vjeux - who is the owner of ListView?

Probably an Image issue; I can't imagine it being specific to ListView.

This is pretty busted - any updates, @nicklockwood?

This is a tough one to catch since the results are never consistent. I was still able to reproduce the issue, maybe 1 out of every 4-5 attempts, following @jchris explanation. See demo below...
I think the catch here is to make sure each <Image /> has a key property set.
In the case of the Movie Example, add a key to each image inside of our MovieCell seems to resolve the issue.
I've noticed some other people have issues with Images rendering properly within ListView and this did the trick.
<Image
  source={getImageSource(this.props.movie, 'det')}
  style={styles.cellImage}
  key={this.props.movie.id}
/>

Not sure if having to add a key  a bug or just a requirement to get the images to re-render properly.
Thoughts?
![Image of Demo]
(http://d.pr/i/16bcY)

My thought is that a "loadingImage" property? would force the state machine to represent the that the image isn't ready.

@jchris - we can do this now by leveraging the onLoad callbacks :smile:

Quick observation on this. I've been seeing this bug for sometime now elsewhere. I've tried keying the image, setting it in state, then calling form there. I've dropped debug points and can see the url that was being assigned to the image wasn't what I was seeing. It only happened on one image in my render row though which got me thinking...
Through elimination the thing that got it working for me was to remove the <TouchableOpacity/> and replace with <TouchableWithoutFeedback />. I have no idea why this works and it may be a Red Herring (maybe something was cached?) but mine has worked consistently through 15 builds now, and I can trigger the bug again as soon as I swap back.

@christopherdro fixed this in the commit referenced above :smile:


Internal Error with iOS project

when I use it with iOS project,
it always prompts error:
Internal Error:   /usr/local/Cellar/infer/0.15.0/lib/infer/infer/bin/../lib/python/infer.py
  -j 4 --project-root
...
  exited with code 65
Error backtrace:
Raised at file "base/Die.ml" (inlined), line 25, characters 6-36
Called from file "base/Logging.ml", line 314, characters 58-80
Called from file "integration/Driver.ml", line 159, characters 2-16
Called from file "integration/Driver.ml", line 280, characters 6-420
Called from file "infer.ml", line 20, characters 2-36
Called from file "infer.ml", line 130, characters 8-54

I have the same error while analyze my project...

I met this problem, too...

What command do you run? The recommended way to run infer on iOS projects is to generate a compilation database using xcpretty.

I have the same error while analyze my project...

I also met this problem, how to resolve it?

This error just means that some part of infer failed. We need more information to investigate:

the version of infer used
the command used to run infer that causes the crash
the full output of infer

Thanks!

Make sure you have disabled COMPILER_INDEX_STORE_ENABLE build setting in all your targets (and PODs)

My project showed up in Chinese on the project file path and was finally replaced with English to complete the solution, but the third party introduced by pod file at compile time could not find the error.😓

I have the same error while analyze my project...

I met the same problem

I also run into a similar error. It is not an iOS project either. It's a very large Java and Groovy  project. Also error only occurs when --reactive option is used. Thanks for any suggestion.
Internal Error:   /opt/infer-osx-v0.17.0/lib/infer/infer/bin/../lib/python/infer.py
  --force-integration gradle -j 8 --project-root /Users/hmusavi/app
  --out /Users/hmusavi/elis/apps/infer-out --
  /Users/hmusavi/elis/tools/gradle-2.13/bin/gradle
  assemble:
  exited with code 1
Error backtrace:
Raised at file "string.ml", line 142, characters 16-31
Called from file "src/string.ml", line 407, characters 12-33
Called from file "src/string.ml", line 416, characters 11-33
Re-raised at file "base/Die.ml" (inlined), line 26, characters 8-56
Called from file "base/Die.ml", line 32, characters 6-39
Called from file "integration/Driver.ml", line 171, characters 2-16
Called from file "integration/Driver.ml", line 272, characters 6-409
Called from file "integration/Driver.ml" (inlined), line 323, characters 2-29
Called from file "integration/Driver.ml", line 328, characters 62-89
Called from file "base/Utils.ml", line 398, characters 16-20
Called from file "scuba/ScubaLogging.ml", line 66, characters 29-44
Called from file "infer.ml", line 20, characters 2-36
Called from file "base/Utils.ml", line 398, characters 16-20
Called from file "scuba/ScubaLogging.ml", line 66, characters 29-44
Called from file "infer.ml" (inlined), line 25, characters 22-94
Called from file "infer.ml", line 137, characters 8-54


Hugo New resources adds to root resource dir rather than content dir

If I have content of type "resources" and I then try to create a new piece of content for that type (ie. hugo new resources/foo.md) this content will be added to the root resources dir, not the content/resources dir.
I'm not sure if this was an intentional design decision, but in my mind the main resources dir should only be hugo generated content (resized images, bundled js, etc.) and not be included in the new command.
To work around this I can do hugo new content/resources/foo.md but that doesnt seem ideal. Additionally, an archetype of resources doesn't appear to work at all.

Hi! I am new to all of this but I looked into the code and played around a little bit with the CLI and found out that hugo will look if there is a folder with the same name as your wished content type.
So if you have a folder "hello/" and you use "hugo new hello/somecontent.md" the new file will be created as "PATHTOYOURSITE/hello/somecontent.md" and not under the content folder.


[nnx] Export missing symbols

What does this PR do?

Export missing symbols from the attention module
Exports Initializer
Adopts JAX's Initializer definition

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (008c3cc) 56.32% compared to head (03ffeac) 56.33%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3583      +/-   ##
==========================================
+ Coverage   56.32%   56.33%   +0.01%
==========================================
  Files         100      100
  Lines       11951    11952       +1
==========================================
+ Hits         6731     6733       +2
+ Misses       5220     5219       -1

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


[bug] memory leak, freezing, lagging since 0.57.0 using PowerToys Run

Microsoft PowerToys version
0.57.0
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Since update to 0.57.0 there is a giant memory leak using PowerToys Run, rendering the PC unusable. Also doesnt find entries anymore it did before, like "control panel" for example.

✔ Expected Behavior
No regression or memory leak.
❌ Actual Behavior
Regression and memory leak.
Other Software
No response

@makedir
Is it fast again when you disable the system plugin and do you have many network adapters?
/bugreport

cc: @jaimecbernardo

I can reproduce this easily when toggling back and forth the "use windows search" option

Yes I have 2 open network connections at the same time, a WireGuard VPN and Wifi.

The issue does seem to happen with every togle regardless you do, not only Windows search on/off, with every toggle, it uses 100% for a longer time and use more and more memory.

https://user-images.githubusercontent.com/6721083/161436836-76c96e69-ebc6-426d-a6b0-c8cb621e2338.mp4

This is a duplicate of a known bug. /dup #17390

Also the memory leak? Not sure how it happened, that it had 10GB of memory usage yesterday. Maybe it is related.

Also the memory leak? Not sure how it happened, that it had 10GB of memory usage yesterday. Maybe it is related.

Damn. My mistake. Sorry.

Also it doesnt find system elements anymore since 0.57.0, like "control panel".

Also it doesnt find system elements anymore since 0.57.0, like "control panel".

Can you please send us a bug report and exactly describe what you search and can't find.

Before update to 0.57.0 you could search for example for "control panel", which though is related also to this issue, that you cant search for other than English system elements:
https://github.com/microsoft/PowerToys/issues/17308
But now after 0.57.0 it doesnt even show control panel anymore:
Before 0.57.0:

Now:

The fix for all the CPU usage should be the same fix as https://github.com/microsoft/PowerToys/issues/17390 indeed, which should also make the memory issue much better.
Regarding the Control Panel, we have to understand why is it not appearing.
From what I see here, we're still waiting for the /bugreport

I dont want to send a bug report, because I dont know what information will be send maybe including private data from my computer.

@jaimecbernardo

The cpu usage/memory usage bug is this settings thing you fixed.
The "Control Panel" result is the shortcut bug you fixed.

So I guess this issue can be closed as duplicate of the two original issues.

Not fixed, happening still/again

Not fixed, happening still/again with 0.57.2

Tracked with #17390


fix bugs in google travel time

Description:
fixed some bugs in google travel time
@robbiet480
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

@Danielhiversen Thanks!!! 🍪


Are twitch videos encrypted?

Why does it need to download parts then merge and finally convert?

There's no encryption, at least as for standard VoDs. Might be that 'Sub only' streams are being encrypted, but as I never stumbled across one, I'm not sure. Also this does not seem very likely for me. There's other solutions to provide a content protection as required for 'Sub only' streams.
VoDs are being provided in chunks by Twitch. That's the way HTTP-Streaming works (https://en.wikipedia.org/wiki/HTTP_Live_Streaming).
As you usually do not want to open a playlist with ~300+ Files, TwitchLeecher merges the chunks into one single file.
The conversion is being done for portability and to minimize disk space usage.

Codereaders is basically right. A 3 hour stream might consist out of 3000 parts where each one has 3mb or something. A web player loads the playlist (containing links to all 3000 files with according timestamps). So if you jump to a location in the VOD, the webplayer knows which part to load for the selected time. It then starts to play back that 3mb file. While playing back that file it loads the next few files over the net and seamlessly plays those one after another. If the player cannot load the needed parts in time, the video starts buffering (aka waiting for new files).
And yea... since you don't want to click a new file every 10s TL is nice enough to merge all those little files into a single one on a binary level. A web player is not capable of loading "a little part" of a giant VOD file and buffering from there. Would be cool, but that's not the case.


gRPC Module exit code 107 fails to log an error

Brief summary
This was found while trying to configure gRPC tests with the Operator + Runner model.
If the importPath for the loaded .proto file does not exist, K6 will exit with code 107 (script exception).
When running K6 locally - k6 run test.js - the error is easy to see and debug
Ex.
k6 run test.js

          /\      |‾‾| /‾‾/   /‾‾/
     /\  /  \     |  |/  /   /  /
    /  \/    \    |     (   /   ‾‾\
   /          \   |  |\  \ |  (‾)  |
  / __________ \  |__| \__\ \_____/ .io

ERRO[0000] GoError: stat /Users/nema/kneemaa/hello.proto: no such file or directory
        at reflect.methodValueCall (native)
        at file:///Users/kneemaa/core/test.js:5:13(35)
        at native  hint="script exception"

But when ran in Kubernetes with the Operator model, the pods end up in an Error status and have no logs to indicate what the error is. When you describe the pod you can see that the exit code is 107
    Port:          6565/TCP
    Host Port:     0/TCP
    Command:
      sh
      -c
      k6 archive --log-output=none /test/grpc.js -O ./grpc.js.archived.tar  && k6 inspect --execution-requirements --log-output=none ./grpc.js.archived.tar
    State:          Terminated
      Reason:       Error
      Exit Code:    107
      Started:      Tue, 15 Nov 2022 22:26:20 -0800
      Finished:     Tue, 15 Nov 2022 22:26:21 -0800
    Ready:          False
    Restart Count:  0

No logs available
➜ kubectl -n chaos grpc-stress-initializer-bsppv

k6 version
k6 v0.41.0 ((devel), go1.19.3, darwin/arm64)
OS
macOS 12, kubernetes 1.24.1
Docker version and image (if applicable)
No response
Steps to reproduce the problem

Setup a gRPC test following the steps in this article - https://k6.io/blog/performance-testing-grpc-services/
Confirm you can successfully test
Update the importPath of client.load(['definitions'], 'hello.proto');  to be something invalid
Re-run the test
Observe error but no log output.

Expected behaviour
Helpful output for exit code 107 to logs for debugging
Actual behaviour
No log output for exit code 107

@na-- any ideas on where to set this for initializer? We are attempting to build our own debug images locally and use that image to test but the --log-output is staying  at none

Hi @kneemaa, thanks for opening the issue! This happened due to some peculiarities of k6 logging requiring workarounds. I've discussed this case with k6 team and just added a fix in PR #174, please see.
Related: https://github.com/grafana/k6-docs/issues/877


feat: Send SMSes using Twilio API

Pr should come from a feature branch like feat/send-sms-with-twilio and not from the dev branch.
There should also be a description of what was done in the ticket. See other prs for reference.


PR: Fix visibility of status bar widget that checks for Spyder updates (Application)

Description of Changes

The widget was not actually hidden when Spyder is not installed through our Windows and Mac apps (this is a follow-up of #21784, which tried to do the same).
Avoid updating the widget status when it's not visible.
Also, make worker that checks for updates emit sig_ready when it can't find the conda channel from which Spyder was installed. That allows the slots connected to that signal to do its job as expected.
And change the text of check_updates_action when checking for updates to give better feedback to users about why that action is disabled.

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12

Two additional comments about this PR:

I tested the new Linux artifact and the update status widget is not shown anymore.
I think some of these changes will need to be ported to PR #21836.

I think some of these changes will need to be ported to PR PR: Only show UpdateManager statusbar widget while updating and when updates are available #21836.

I think the desired behavior is already in spyder-ide/spyder#21836, but we should certainly confirm that.

I think the desired behavior is already in https://github.com/spyder-ide/spyder/pull/21836, but we should certainly confirm that.

I ported these two items because they are not part of that PR:

Also, make worker that checks for updates emit sig_ready when it can't find the conda channel from which Spyder was installed. That allows the slots connected to that signal to do its job as expected.
And change the text of check_updates_action when checking for updates to give better feedback to users about why that action is disabled.


The currency is not read correctly from .xls file

I have a currency format in my .xls file, and I tested on my project and the demo website. The currency is not parsed correctly. It parsed as NaN. The currency format that I am using is like $1,123.87.
Is this a known issue? Will this be fixed in the future?
Thanks a lot.

This is also an issue for me...

Did you succeed in ?
I have the same problem, I don't read the good value.
Have you got a solution ?

@mca22120 do you have a sample file you can share?

I think we may have addressed this in a recent commit.  If the file came from LibreOffice, it may have generated an oddball format like [$$-409]#,##0.00;[RED]\\-[$$-409]#,##0.00.  We have since addressed the localization of currencies, so it should work.
We are going to close for now, but if you can share a file we can investigate further


copied v0.2 from exploratory branch

Codecov Report

Merging #13 (43fb8a2) into main (db2fc7c) will decrease coverage by 5.11%.
The diff coverage is 22.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   29.05%   23.95%   -5.11%
==========================================
  Files           3        3
  Lines         117      167      +50
==========================================
+ Hits           34       40       +6
- Misses         83      127      +44

Flag
Coverage Δ

unittests
23.95% <22.66%> (-5.11%)
:arrow_down:

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

Impacted Files
Coverage Δ

scripts/focus.sh
0.00% <0.00%> (ø)

scripts/menu.sh
0.00% <0.00%> (ø)

scripts/functions.sh
76.92% <87.17%> (+22.95%)
:arrow_up:

Codecov Report

Merging #13 (ba446ef) into main (db2fc7c) will decrease coverage by 29.06%.
 Consider uploading reports for the commit d807186 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #13       +/-   ##
==========================================
- Coverage   29.05%   0.00%   -29.06%
==========================================
  Files           3       1        -2
  Lines         117     109        -8
==========================================
- Hits           34       0       -34
- Misses         83     109       +26

Flag
Coverage Δ

unittests
0.00% <0.00%> (-29.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

scripts/focus.sh
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes


Login icon not visible properly

Description

The login icon in the navbar of the chapter page is not visible properly.
Expected Behavior

I can solve this issue. Can you please assign this task to me?
Screenshots

Enviroment:

OS: Mac Linux Windows
Browser: Chrome Safari Firefox
Version:
Device: Desktop Mobile

Can you please add labels to this PR?


Getting  suddenly Received status code 403 from server: Forbidden

Hello,
I m getting suddenly the following error
Could not GET 'https://dl.bintray.com/hummatli/maven/com/mobapphome/library/android-app-updater/1.3.3/android-app-updater-1.3.3.pom'. Received status code 403 from server: Forbidden
Any advice?

Fixed


docs: autogenerate doc from source code

autogenerate https://spinti-software.github.io/cpcdosos2.1/ from source code

Nice


chore(Dependencies) : update elastic-apm-agent from 1.36.0 to 1.38.0

About this PR
📦 Updates co.elastic.apm:elastic-apm-agent from 1.36.0 to 1.38.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.elastic.apm", artifactId = "elastic-apm-agent" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "co.elastic.apm", artifactId = "elastic-apm-agent" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #614.


Typing in a path that resolves to a directory creates a file called untitled

It would be nice if a path matched an existing directory, that directory was opened as a project. (Kind of like what happens when executing atom [DIR_PATH] from the command line, or if a directory is selected from the Open... file dialog.)

why do you need this feature here? this plugin is about easy way of creating files in project, not opening projects

why do you need this feature here? this plugin is about easy way of creating files in project, not opening projects

Opening a new, unsaved, but named buffer is a pretty specific feature. Opening things seems to me to be a more useful super set that is naturally grouped together. Opening a project is just a suggestion. It's also consistent with the way that Atom behaves now.
In any event, opening a "untitled" buffer seems to me to be the wrong behavior. (Although if opening a directory as a project is off the table, I'm not sure what is the right behavior.)

Sorry, maybe misunderstanding take place. My point is that I’m usually working with projects and don’t need to open existing folder (inside this project) as a new project — otherwize, I will greatly annoyed. Second point is that opening folder as project is out of scope plugin named "*-new-file". Third point: it is duplicating Open… dialog, isn’t it?

Sorry, maybe misunderstanding take place. My point is that I’m usually working with projects and don’t need to open existing folder (inside this project) as a new project — otherwize, I will greatly annoyed.

Ah, excellent point. I'm not sure what the behavior would be in this case. Maybe navigating to (or expanding) the folder in the first open project in which it appears? You're right. This doesn't have an obvious "correct" behavior.

Second point is that opening folder as project is probably out of scope plugin named "*-new-file". Third point: it is duplicating Open… dialog, isn’t it?

I agree that the name of the project would be too narrow, but opening files is (perhaps unintentionally) already supported by this package. Try entering a path that already exists as a file, and this package will open that file.
I suspect that if there were two packages: one that only allowed opening new files and nothing else; and one that behaved (by analogy) like Emacs' C-x C-f, the latter would be generally more popular. (That's just a guess though.)
Right now, this package sits somewhere in between, which creates some confusion about what it does vs. what it's supposed to do.

I see there's already a branch that suggests adding behaviors around existing directories is something worth experimenting with?

Hi guys, first of all thanks for great discussion! Really helps me a lot. Posita you are right about opening untitled although I don't personally like opening new project instead. Maybe simple message that this folder already exists could work?

Maybe simple message that this folder already exists could work?

Perhaps. It depends on which direction you want to take. I hate to keep coming back to C-x C-f, but it's a pretty powerful tool. If you want to support opening (which you already do, perhaps by accident, but "creating" an existing file happens to open that file), then an error message seems inconsistent. However, if this package is really only about creating new things, then maybe an error message for existing directors and files is appropriate. I think the latter is much less useful.
That being said, directories might be complicated where an error message is appropriate for now. Just for fun, I tried:
% .../atom.sh [sub_directory_of_an_already_open_project]

Atom's default behavior is to open a new window with the sub-directory as a new project. I agree with @iamstarkov that this is probably the worst alternative.

Atom has announced Multi-Folder Projects. It gives us another option here: to open folder in existing atom window (project) instead of creating new one.
There are still some questions though. For example, it's unclear to me what to do with a folder inside already opened project. Or what if I just want to create a directory and not to open it? Maybe there should be several commands that handles each case?

Closed as wontfix?

https://github.com/Trudko/advanced-new-file/issues/37#issuecomment-136403491


Hotfix/gpu docker

ML API works on local now (with or without gpu) and using our own base image again.

@rha930 I think I need "Write" access on the repo to allow this PR approval to go through


Does not build using make

GOPATH=/home/tperson:/home/tperson/src/github.com/whyrusleeping/gx/pkg
go install
main.go:10:2: cannot find package "QmR5FHS9TpLbL9oYY8ZDR3A7UWcHTBawU1FJ6pu9SvTcPa/cobra" in any of:
	/home/tperson/.go/src/QmR5FHS9TpLbL9oYY8ZDR3A7UWcHTBawU1FJ6pu9SvTcPa/cobra (from $GOROOT)
	/home/tperson/src/QmR5FHS9TpLbL9oYY8ZDR3A7UWcHTBawU1FJ6pu9SvTcPa/cobra (from $GOPATH)
Makefile:8: recipe for target 'install' failed
make: *** [install] Error 1

To get around for now you need to append the vendor directory to your GOPATH.
env GOPATH=$GOPATH:(pwd)/vendor make install

Or $(pwd) for those not in fish

hrm, i'll try and fix this

(in go 1.5 you can just go get -vendor github.com/whyrusleeping/gx)

(in go 1.5 you can just go get -vendor github.com/whyrusleeping/gx)

We'll, that is just the best thing I've heard all day.

also, it should work with the makefile now...

setting the var like that only has the effect in one line in some versions of make.
this worked for me:
CWD = $(shell pwd)

all:
	GOPATH=$(GOPATH):$(CWD)/vendor go build

install:
	GOPATH=$(GOPATH):$(CWD)/vendor go install


My game is about destroying bricks using a ball. As you play, there's a 10% chance to get a power-up that makes your player wider, giving you an advantage. The game has four levels, and you can even customize the code to create your own levels for a personalized challenge.

How do you play your game?
To play my game, you use the 'A' and 'D' keys to move left and right. After you win a level, you can click 'K' to move on to the next level.

Code
Check off the items that are true.

[X] The game was made using the Sprig editor.
[X] The game is placed in the in the /games directory.
[X] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[X] The game runs without errors.
[X] The name of the file/game contains only alphanumeric characters, -s, or _s.
[X] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Great job with this game! I loved the power-ups mechanic where the paddle size would be expanded. I'm approving this, and you'll see it in the gallery soon- congrats!
If you're a teenager, you can request a free Sprig console at this link and join our online Slack community to chat with other people making Sprig games.

omg thank you very much <3


RELEASE: v0.1.0-alpha.41

Coverage remained the same at 100.0% when pulling d08fa9651c8eb80e1b6c2ff13e41f1ca2f6f2bce on dev into a58388f27173055cc60ae7cd949a39ec64ffd891 on next.


false positive ignore

If we are confirmed that there are some false positive lints for specific line. Is there any way to mark it as ignored for lint again?

You can add //nolint: nilaway to the specific line to indicate that the nilaway error on the particular line should be ignored.
Note that this ignore logic is implemented by the analyzer driver of your choice. Bazel/nogo and golangci-lint should have support for it, the standalone checker (which under the hood uses x/tools/go/analysis/singlechecker) currently does not. However, the standalone checker is really only for evaluation purposes such that it's not recommended to be used in production anyway.
I'm closing this issue, but feel free to re-open it if the above doesn't answer your question :)


HW 05 Link opens HW 01

Similar problem to the Lab 05 link.

Hi @Charles-CCSF. I'll delete this issue and message you separately since this is not related to the students.


Non-underscore syntax supported for ActionView form helpers?

Hello,
The Theo README lists the following <form> helpers syntax:
<_form-with model%="widget" data-turbo-confirm="Are you sure?">

    <_label name="name" />
    <_text-field name="name" />

    <_label name="size" />
    <_select name="size" options%="['Big', 'Small']" />

  <_submit value="Create" />
</_form-with>

However, as I noted in this post the <_some-partial> syntax breaks the standard HTML parser of Prettier.
Question, does Theo support a more HTML-compatible syntax for Rails ActionView helpers?
Basically, I would like to use Theo if I can totally avoid the <_some-tage-name> underscore syntax.
Note, I am not saying to get rid of the underscore syntax, just wondering if a Prettier-compatible syntax is available?
Best regards.

Yeah, the JSX-style PascalCase syntax is fully supported and equivalent of _underscore syntax, you can call partials with it. When I'll find the time, I will make it clearer in the README.
Does it answer your question? Or are you asking to have a new way of calling helpers (and not partials)?

So PascalCase is supported for Form Helpers?
For example ‘FormWith‘?


Collapsing Header With Scroll freezes

after implementing the example : CollapsingHeaderWithScroll , i noticed the following issue:

drag the Intractable view up , all works fine, the header scales down
scroll the content of the Intractable little bit (NOT TO THE END)
then the Intractable drag down is NOT active , the screen just cannot be dragged down, to get it working, i had to scroll the content of the scroll view Intractable to the end, and ONLY after that, i can expand the header

Update for this example screen will be available soon.
In the mean time, I am closing the issue as a part of an effort to cleanup and revive the project. If this issue persists after v1.0.0, you are more than welcome to reintroduce the issue.
Thanks for your contribution.

This issue still persists in latest interractable version


Update to simplecov 0.10

Changelog can be found here: https://github.com/colszowka/simplecov/blob/master/CHANGELOG.md#0100-2015-04-18-changes
No breaking changes as far as I can see.

Coverage remained the same at 90.94% when pulling be03c74f49fc4479ab470b843ced9319e29ea488 on nTraum:update_simplecov into bb40b396095763fad73bdfb8dc9bddd14f57945f on lemurheavy:master.

Coverage remained the same at 90.94% when pulling be03c74f49fc4479ab470b843ced9319e29ea488 on nTraum:update_simplecov into bb40b396095763fad73bdfb8dc9bddd14f57945f on lemurheavy:master.

Coverage remained the same at 90.94% when pulling be03c74f49fc4479ab470b843ced9319e29ea488 on nTraum:update_simplecov into bb40b396095763fad73bdfb8dc9bddd14f57945f on lemurheavy:master.

:+1:

Should be good. I’ve also included this change in #83, so you could also just merge that.

Thanks @nTraum, this change came through on @sferik's PR.
Releasing a new version now. Cheers!

Thanks for updating! :)

thanks!


doc: format update

Codecov Report

Merging #70 (c08095b) into main (cf8492f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #70   +/-   ##
=======================================
  Coverage   31.57%   31.57%
=======================================
  Files          15       15
  Lines         627      627
=======================================
  Hits          198      198
  Misses        429      429

Continue to review full report at Codecov.

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

recheck

recheck

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


feat(learn): Add keyboard nav with up and down keys in challenge description

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #37158, I have tested it on my local site.

@scissorsneedfoodtoo I've checked this out in gitpod and it works perfectly. If you're happy that it doesn't damage accessibility, I think this can be merged.

:sparkles: :sparkles: :sparkles: Congratulations on your first contribution to freeCodeCamp! :sparkles: :sparkles: :sparkles:
Thank you for reporting this issue submitting a PR to fix it, @rickysyamsi. We're all really looking forward to your next contribution. Also thank you for testing this and confirming it won't decrease accessibility, @ojeytonwilliams.


New issues are not getting added to the hugrverse project

The latest permission changes broke hugrbot's token access to the project.
We need to give the add-to-project action access to both this repository and the project in a different organization, which may not be supported by fine-grained access tokens.

This is now fixed. I manually added the previous issues too.


PowerToys Run: Being able to bind Windows key.

Summary of the new feature/enhancement
I would like to be able to bind PowerToys Run to windows key + f.
Description of the new feature/enhancement
I would like this because ctrl F is most other ' normal' searches. The windows system wide equivalent should be able to be bound to windows key + f in my opinion.
What should be changed
Allow the windows key to be mapped as a PowerToys Run command starting button e.g. windows key + f, to mimmic ctrl + f.

hi@bartvanlaar, issue #2325 will enable this in our 0.19 release tomorrow!
closing this as duplicate!


Cloudedit

Fixed typo's in require statements.   May need to do a clean install.  Also cloud be differences in the way Mac's and ubuntu (my test server on Cloud9.io) handle case sensitivity.

I also added a "taggable" sub class for resources per Chri's suggestion.  Not sure I like the inhertence model 100% but it seems to work.  Easy tagging becomes important for ops.


Suggestion: Give some form of feedback when trying to applyedits no changes instead of no feedback at all

Generally speaking a message at the start or the end of the command should do the trick too.


Command line option for Visual Studio version

As the title says, add a --vs CLI option to set the wanted VS version for compilation.

I can take this

Shall we have a finished set of option (like 14.0, 13.0, 12.0 and latest <- default)?

I was thinking keying it on VS version (and not the compiler version) as that's how I've historically handled it. so something like --vs=2015.

Fixed by https://github.com/mono/Embeddinator-4000/commit/e12ece26843579feeeba83b739c7bf69455f9bfe.


Port local mqtt pub sub pysys tests to rust

Port the pysys tests that are present in the folder tests/Pysys/local_mqtt to rust.
The tests include

local_mqtt_pub_fail_noqos
local_mqtt_pub_sub
local_mqtt_pub_fail_qos0
local_mqtt_pub_sub_retain
local_mqtt_pub_fail_qos1
local_mqtt_pub_sub_with_sudo
local_mqtt_pub_fail_qos2
local_mqtt_sub_fail

Describe the solution you'd like
Once ported successfully, these tests should run as part of cargo test
Describe alternatives you've considered

Additional context
spawn a mqtt server
Use config-dir option to set the mqtt.port for the temporary config, on which the mqtt server runs.

Remove tests with sudo as we got rid of using the separated config file depending on user.


Clean up output on error

Before this change, when an error occurred, the error would be printed out, followed by the usage information, and then followed again by the error message. Seeing the usage information tends to be misleading since it traditionally suggests a problem parsing the program arguments. This disables that behavior and removes the redundant error message.

I'm not quite sure why GH isn't flagging a merge conflict, but I did almost exactly this in #28: https://github.com/edgebitio/edgebit-cli/commit/d469b912347c0beb313af6ddce8b68d8115be526


fix problem with UI datetimepicker

https://github.com/refactorthis/xtform/issues/40

I don't see how the checks are related to the code I wrote :/
Are they old?


Readme files are misleading with respect to Firebase plans.

I see in most of the projects Readme.md, (eg. Full-text search via Algolia, moderate-images) there is something like "Enable Billing on your Firebase project by switching to the Blaze or Candle plans".
Is there a plan called Candle exists in firebase? At least, I don't see here.
Also, why do we need to enable billing for testing cloud functions? Especially when you are talking between Google Cloud components?

The Candle plan is the older name of the fixed cost plan (now Flame). Thanks for pointing this out. We'll update these READMEs.
The reason you need to enabled billing for these two samples is:

Full-text search via Algolia: In this sample you'll need to do requests to a non-Google service (Algolia) and this currently requires to have billing enabled for abuse reasons.
moderate-images: This sample uses the Cloud Vision API which requires to have billing enabled.

Thank you very much. I appreciate the acknowledgement Nicolas.

Fixed in the READMEs


502 and 400 error while sending chunk data

I  get 502(Bad gateway) and 400 (Bad request) when i tried to send chunk to backend (node js ). Backend service was still running doesn't receive any request.

And also with this  net::ERR_INCOMPLETE_CHUNKED_ENCODING meesage


metadata template_version type

Is there anywhere saying that the {metadata: template_version} needs to be a string and not allowing decimals (ie: '1.1' vs 1.1)? I'm allowing integers or strings at the CSAR metadata file level but noticed it's forced string-only for inline metadata.

Yes, in the latest commit we are being stricter by default. I believe this is a good idea: while YAML can allow you to write "1.1", there's no guarantee for things like "1.0.0". It's best for users to avoid potential errors and stick to the correct format.
This behavior can be changed by setting context.presentation.allow_primitive_coersion to True, in which case primitives will be coerced (if possible) to their types, which is the old behavior.


Performance

Hello
I noticed that the performance of the below code is prolonged compared to https://platform.openai.com/tokenizer
   protected static $encoding_name = 'cl100k_base';

    public function countTokens($content){
        $provider = new EncoderProvider();
        $encoder = $provider->get(self::$encoding_name);
        $tokens = $encoder->encode($content);
        return count($tokens);
    }

The above code will take 3 seconds to return total tokens compared to  https://platform.openai.com/tokenizer (less than a second) when I run to use it with this text MSGSphere.txt
Is there any way to optimize the speed?

Hello All
for anyone looking for a solution, the problem we didn't set a caching directory, @yethee Kindly, can you mention this in the README file
    public function countTokens($content){
        $provider = new EncoderProvider();
        $encoder = $provider->setVocabCache(storage_path('encoders'));
        $encoder = $provider->get(self::$encoding_name);
        $tokens = $encoder->encode($content);
        return count($tokens);
    }

Thanks for feedback!
I've updated the README. And since 0.2.0 the cache is enabled by default.


Make e2ebasic file uploads viewable

Description
When running milmove in local dev, files associated with the test cases in e2ebasic.go were not viewable because, for local storage we were using storage.Memory instead of storage.Filesystem in e2ebasic.go. This PR updates e2ebasic.go to use storage.Filesystem.
In making this update another issue was discovered that was preventing all locally uploaded / served pdfs from being viewed on local dev. The security headers X-Frame-Options and Content-Security-Policy in security_headers.go, were blocking locally served files from being embedded within our own site. In order to fix this issue we created a bare muxer, which has no middleware attached and is used exclusively for serving local files. The site muxer and its accompanying middleware is now attached to bare.
Reviewer Notes
On master run make db_dev_e2e_populate. Open the office app and navigate to the document uploader. None of the documents created by db_dev_e2e_populate should be viewable. Also, any pdf uploaded from local dev (not just those created by db_dev_e2e_populate), should not be viewable in the document uploader. Check out this branch and rerun make db_dev_e2e_populate. Now the documents should be viewable and pdfs uploaded from local dev should be viewable as well.
Setup
make db_dev_e2e_populate
make office_client_run

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

I'd like to confirm that the test containers for e2e are still read-only so that we aren't introducing problems with this change.

Could you clarify a bit about the type of problems that we could be introducing? One thing is, right now if an e2etest were to try to interact w/ a document, it would fail b/c the documents being served by the inmemory file system aren't viewable. But I'm not sure how that weighs against the rational for introducing the inmemory storage.
I guess to maintain the status quo I could maybe add a "local" flag in cmd/generate-test-data/main.go.


How to locate /images/ for the animation

Hey there,
So we've got a data.json full of our animation data and an /images/ directory including the images that will be animated, given to us by our designer. However, I didn't see any mention of how the /images/ paths can be found through react-lottie? Unsure if this is the correct place to ask, but if anyone has come across this issue (as I myself don't know enough about After Effects - which it was exported with), could you advise?
I have seen this issue crop up on lottie-web itself and people suggest a way so that the /images/ folder doesn't appear on export, or does react-lottie have something like lottie-web, when it comes to picking up assetPaths for the .json? To ensure it finds the images.
Apologies if this seems vague, I'm just not very well versed in the design process itself.

@sgript hey let me know if you find any solutions for this:) struggling with the same problem

Hey @sgript
I solved it by creating my own json withing a javascript file.
Instead of using the animation.json directly, create a animation.js file where you import the image and replace the [image_in_question].png occurrences for your image_in_question_name. Remember to fix the folders path and delete it.
 This way, your image paths will be correct.
import image_in_question_name from './images/[image_in_question].png'

export default {"v": ...... ANIMATION JSON ....... };

This worked for me and I am using gatsby.
Hope it helps!

I'm building a chrome extension and in that case images get a special path. I can access that path from javascript. How would I change the JSON so that it uses the correct path?

@wahlforss look at this example:
import img_0 from './images/img_0.png'

export default {
  v: '5.2.1',
  fr: 25,
  ip: 1,
  op: 74,
  w: 500,
  h: 500,
  nm: 'pulseloop',
  ddd: 0,
  assets: [
    { id: 'image_0', w: 701, h: 632, u: '', p: img_0, e: 0 },
    {
      id: 'comp_0',
      layers: [
        {
          ddd: 0,
          ind: 1,
          ty: 2,
          nm: img_0,
          cl: 'png',
          refId: 'image_0',
....

I change the u:'path-to-folder' to u: '' and all the occurrences where you use the image name to img_0(my imported image) directly.

@rageofflames
Thank you man! That fixed it :)

@rageofflames @sgript
Hey! it seems you can also just place it in the public folder with the same pathing ('./images/img_0.png) and it will work.
had a quick read through the animation.json and found that the structure of the json is different from the one @rageofflames posted, so because versions, YMMV.
*tested on react-lottie 1.2.3

oh sorry about that @rageofflames , I might not have been answering the main thread directly. I manually edited them because i was only dealing with a small number of animations.
I only mentioned it in case others came along (like me) looking for the easiest way to make lottie pngs work in react and didn't know that we could just place it in the public folder.
your method looks really good for automating image imports, I'll do that from now on!

Argh I can only get @rageofflames method to work with one image... The rest of the images are not found.
Any ideas?

@wahlforss look at this example:
import img_0 from './images/img_0.png'

export default {
  v: '5.2.1',
  fr: 25,
  ip: 1,
  op: 74,
  w: 500,
  h: 500,
  nm: 'pulseloop',
  ddd: 0,
  assets: [
    { id: 'image_0', w: 701, h: 632, u: '', p: img_0, e: 0 },
    {
      id: 'comp_0',
      layers: [
        {
          ddd: 0,
          ind: 1,
          ty: 2,
          nm: img_0,
          cl: 'png',
          refId: 'image_0',
....

I changed the u:'path-to-folder' to u: '' and all the occurrences where you use the image name to img_0 which represents the image direct path.
Hope it helps

Thank you!

The images aren't visible outside public folder, move the png's to the public folder (you can put them wherever in "public/animations/[animationName]/") and in the json, change u:"images/" for the folder where your images are ( using the last example should be u:animations/[animationName]/ )

@wahlforss look at this example:
import img_0 from './images/img_0.png'

export default {
  v: '5.2.1',
  fr: 25,
  ip: 1,
  op: 74,
  w: 500,
  h: 500,
  nm: 'pulseloop',
  ddd: 0,
  assets: [
    { id: 'image_0', w: 701, h: 632, u: '', p: img_0, e: 0 },
    {
      id: 'comp_0',
      layers: [
        {
          ddd: 0,
          ind: 1,
          ty: 2,
          nm: img_0,
          cl: 'png',
          refId: 'image_0',
....

I changed the u:'path-to-folder' to u: '' and all the occurrences where you use the image name to img_0 which represents the image direct path.
Hope it helps 👍

Thanks dude, this worked!

I understand the solution that is given in previous answers, but I can't seem to figure out how to import and use this as animationData for the Lottie component.
My snippet:
import * as React from 'react';
import Lottie from 'react-lottie';
import animationData from '../../assets/animations/background-animation/animation.js';

export default class BackgroundAnimation extends React.Component {

    constructor(props) {
      super(props);
      this.state = {isStopped: false, isPaused: false};
    }

    render() {
      const defaultOptions = {
        loop: true,
        autoplay: true,
        animationData: animationData,
        rendererSettings: {
          preserveAspectRatio: 'xMidYMid slice'
        }
      };

      return <Lottie options={defaultOptions}/>
    }
  }

import animationData from '../../assets/animations/background-animation/animation.js';

Try dropping the .js extension and if it doesn't work can you share the animation.js file?

Make sure the path to the animationData is correct and then make the json file a js file and import the pictures into js file you are trying to use. Then in assets in the js file change the “p” to the imported picture variable name. I used “img_0” and so on
Sent from my iPhone

On Apr 3, 2019, at 7:05 AM, Sacha<EMAIL_ADDRESS>wrote:
I understand the solution that is given in previous answers, but I can't seem to figure out how to import and use this as animationData for the Lottie component.
My snippet:
import * as React from 'react';
import Lottie from 'react-lottie';
import animationData from '../../assets/animations/background-animation/animation.js';
export default class BackgroundAnimation extends React.Component {
constructor(props) {
  super(props);
  this.state = {isStopped: false, isPaused: false};
}

render() {
  const defaultOptions = {
    loop: true,
    autoplay: true,
    animationData: animationData,
    rendererSettings: {
      preserveAspectRatio: 'xMidYMid slice'
    }
  };

  return <Lottie options={defaultOptions}/>
}

}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for replying.
@rowgregory, what you described is exactly what I did. The issue I'm facing tho is that the import of the animation.js file throws Uncaught Error: Cannot find module '../../assets/animations/background-animation/animation.js'
@rageofflames, removing the .js extension results in the same error Uncaught Error: Cannot find module '../../assets/animations/background-animation/animation' but without the .js ofcourse. This is a snippet from my animation.js file.
import img_0 from './images/img_0.png'
import img_1 from './images/img_1.png'
...

export default {
    "v": "5.4.4",
    "fr": 30,
    "ip": 0,
    "op": 930,
    "w": 1920,
    "h": 1080,
    "nm": "TV - landscape",
    "ddd": 0,
    "assets": [
        {
            "id": "image_0",
            "w": 2349,
            "h": 1134,
            "u": "",
            "p": img_0,
            "e": 0
        },
        {
            "id": "image_1",
            "w": 1138,
            "h": 1139,
            "u": "",
            "p": img_1,
            "e": 0
        },
        ...
     ]
}

@Skagoo your code appears to be similar to mine, the problem seems to be related to the paths and it's hard to help from my side 🤔Check the import path, recompile, manually check if the file is at the expected path...

Yeah you probably need to get rid of ../../
Sent from my iPhone

On Apr 3, 2019, at 8:42 AM, Sacha<EMAIL_ADDRESS>wrote:
Thanks for replying.
@rowgregory, what you described is exactly what I did. The issue I'm facing tho is that the import of the animation.js file throws Uncaught Error: Cannot find module '../../assets/animations/background-animation/animation.js'
@rageofflames, removing the .js extension results in the same error Uncaught Error: Cannot find module '../../assets/animations/background-animation/animation' but without the .js ofcourse. This is a snippet from my animation.js file.
import img_0 from './images/img_0.png'
import img_1 from './images/img_1.png'
...
export default {
"v": "5.4.4",
"fr": 30,
"ip": 0,
"op": 930,
"w": 1920,
"h": 1080,
"nm": "TV - landscape",
"ddd": 0,
"assets": [
{
"id": "image_0",
"w": 2349,
"h": 1134,
"u": "",
"p": img_0,
"e": 0
},
{
"id": "image_1",
"w": 1138,
"h": 1139,
"u": "",
"p": img_1,
"e": 0
},
...
]
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@rageofflames after digging a bit deeper I found the following issue Module not found: Error: Can't resolve 'source-map-loader' in '/Users/sacha/Documents/dev/DocFoundry/af-doc-fe/src'
@rowgregory I've tried that as well.. Been playing around with the path but that doesn't seem to do the trick.
This is the relevant project structure.
.
└── src
    ├── assets
    │   └── animations
    │       └── background-animation
    │           ├── images
    │           │   ├── img_0.png
    │           │   └── img_1.png
    │           └── animation.js
    └── modules
        └── Waldo
            └── BackgroundAnimation.tsx

Thanks for helping me out, I will keep you posted if anything changes.

Update: If I place the contents of animation.js in the same file as BackgroundAnimation.tsx It works perfectly fine. So it is indeed an issue in the import reference.. Can't seem to find out what is wrong with the statement though.

My issue was fixed by placing the json in a .tsx file instead of .js.

I've found another way and I guess this is easier than converting JSON to JS.
If you are using Gatsby, you should move your assets for the animation to the static folder.
The static folder should be at the root of your project.
For example.
-static
--animations
---img_0.png
---img_1.png
---img_2.png
After that in your JSON file you should only change the path parameter - "u": "animations/" and that's it.
Gatsby automatically copies the static folder into the public folder, and here you will see new folder animations from the static folder with your images, that you created.

@wahlforss look at this example:
import img_0 from './images/img_0.png'

export default {
  v: '5.2.1',
  fr: 25,
  ip: 1,
  op: 74,
  w: 500,
  h: 500,
  nm: 'pulseloop',
  ddd: 0,
  assets: [
    { id: 'image_0', w: 701, h: 632, u: '', p: img_0, e: 0 },
    {
      id: 'comp_0',
      layers: [
        {
          ddd: 0,
          ind: 1,
          ty: 2,
          nm: img_0,
          cl: 'png',
          refId: 'image_0',
....

I changed the u:'path-to-folder' to u: '' and all the occurrences where you use the image name to img_0 which represents the image direct path.
Hope it helps 👍

thanks a lot !!!!!!

I found myself processing a lot of Lottie files this way so I wrote a really hacky node script if anyone's interested - it takes a command line argument of the path of the .json file and the images/ folder and writes an index.js file there with the correct image imports and asset "p" names:
const fs = require('fs');
const util = require('util');
const path = require("path")

const readdir = util.promisify(fs.readdir);

const filePath = process.argv.slice(2)[0]

async function getFileNames(pathToDir) {
    let names;
    try {
        names = await readdir(pathToDir);
    } catch (err) {
        console.log(err);
    }
    if (names === undefined) {
        console.log('undefined');
    } else {
        return names
    }
}

// provide the location of .json file and /images w/ path

let jsonFile
let imageNames

getFileNames(filePath).then(allFiles => {
    allFiles.forEach(file => {
        if (path.extname(file) === ".json") {
            jsonFile = file
        }
    })

    let json = JSON.parse(fs.readFileSync(filePath + "/" + jsonFile, 'utf8'))

    json.assets.forEach(asset => {
        if (asset.id.slice(0, 5) === "image") {
            asset.u = ''
            const assetString = asset.p.replace(".png", "")
            asset.p = "REPLACE" + assetString + "REPLACE"
        }
    })

    getFileNames(filePath + "/images").then(res => {
        imageNames = res
        imageNames = imageNames.map(image => "import " + image.replace(".png", "") + " from './images/" + image + "'")
        let finalFile = imageNames.join("\n") + "\n" +
            // then read json file and remove "u":"images/","p":"img_2.png" => "u":"","p":img_2
            "export default " + JSON.stringify(json).replace(/"REPLACE|REPLACE"/g, "")
        fs.writeFile(filePath + "/" + "index.js", finalFile, err => err && console.log("error", err))
    })
})

So you could invoke it like:
node <this-scripts-name>.js /Users/YOURUSERNAME/Downloads/lotties/my-lottie-folder

I've found another way and I guess this is easier than converting JSON to JS.
If you are using Gatsby, you should move your assets for the animation to the static folder.
The static folder should be at the root of your project.
For example.
-static
--animations
---img_0.png
---img_1.png
---img_2.png
After that in your JSON file you should only change the path parameter - "u": "animations/" and that's it.
Gatsby automatically copies the static folder into the public folder, and here you will see new folder animations from the static folder with your images, that you created.

Hope this works for NextJS, but sadly it doesnt.
Anybody working with image lottie on NextJS?

In addition to the solution above, for the NextJS the static folder is currently deprecated and you can move the your animation folder into your public folder directly. In your JSON file  only change the path parameter - "u": "animation's directory name/" like the solution above

Hey @sgript
I solved it by creating my own json withing a javascript file.
Instead of using the animation.json directly, create a animation.js file where you import the image and replace the [image_in_question].png occurrences for your image_in_question_name. Remember to fix the folders path and delete it.
 This way, your image paths will be correct.
import image_in_question_name from './images/[image_in_question].png'

export default {"v": ...... ANIMATION JSON ....... };

This worked for me and I am using gatsby.
Hope it helps!

Thanks
This worked :smiley:

animations
@necheporenko
It really works perfectly from me. Thanks a lot. :)


Windows在安装最后一步时总是不成功怎么办

错误代码：

devtools::install_github("xiaoleiLiubio/MVP")
Installation failed: Failed to connect to raw.githubusercontent.com port 443: Timed out
devtools::install_github("xiaoleiLiubio/MVP")
Downloading GitHub repo xiaoleiLiubio/MVP@master
from URL https://api.github.com/repos/xiaoleiLiubio/MVP/zipball/master
Installation failed: transfer closed with outstanding read data remaining

No problem with other packages, only this package is always interrupted.

This is due to the instability of network transmission. now that we have reduced the size of the package, this problem should no longer occur.


actionItems only show if drawer is modal

I'm submitting a:

[ x] bug report
[ ] feature request

What OS are you using?
kubuntu 19.10
What browser(s) is this bug affecting:
chromium, firefox. (probably all browsers)
Current behavior:
When using a top-app-bar with a drawer, the icons in the "actionItems" slot only show if the drawer has type="modal"
Expected behavior:
The drawer type should not affect the actionItems in the top-app-bar
Steps to reproduce:
In the drawer demo, remove the type="modal" from the last example and see the icons on the right side of the top-app-bar disappear

The issue is that the top app bar is staying as wide as the viewport when the drawer opens, and the action items are off screen to the left.

Examples
current behavior:

desired behavior:

The issue is that mdc-top-app-bar is styled with position: fixed, which essential means that it has a width 100% of the window now.
For the mdc-top-app-bar a styling option with position: absolute might be viable. However I dont know what side effects there might be, especially with mdc-top-app-bar-fixed

I maintain a customized fork of this (internal to my company) and its been a while but i'm pretty sure i just did a hack fix where the drawer will toggle a class in any slotted top-app-bars and then the top app bar does a calc to reduce width by drawer width. I think position: absolute had other issues

fixed in #1508 you need to size top-app-bar to be 100% - the width of the drawer which can be adjusted with --mdc-drawer-width
e.g.
mdc-drawer[open]:not(type="modal") {
  --mdc-top-app-bar-width: calc(100% - var(--mdc-drawer-width, 256px));
}

What's about this issue actionItems slot disappear on desktop media for standard bar with header ?
It looks similar one, but different issue. I'm using standard header for desktop version and modal type for mobile version. The actionItems slot disappear for standard with header.

Hello @mahdiridho, please read my comment above; it should solve your issue as well

I left a question above due to your solution doesn't work on my case, I have tried.

I had a typo in the solution. The :not selector should be selecting the type attribute. Does that work?

Here, I created a demo to simulate my issue :
mwc-top-app-bar slot actionItems disappear on desktop media

it is not yet released; please install @material/mwc-top-app-bar@canary to see the changes

it is not yet released; please install @material/mwc-top-app-bar@canary to see the changes

Well, It works but give me a new bug for the drawer style. It has about 10px margin now

For temporary solution, I adjust the drawer style to be :
mwc-drawer {   margin: -10px; }
But, I think you will fix this part later?
Thanks

That looks like the body's default margin. Looking at your code, it doesn't look like you have anything to remove it. Try adding this to your index.html file:
<style>body { margin: 0; }</style>

Since you're setting the width to 100% of the parent container, and the body has a margin, that's why the drawer isn't expanded to the full width of the viewport.

Ah I see, you're right @asyncLiz
I have updated the demo repo for the final sample
Thanks everyone

it is not yet released; please install @material/mwc-top-app-bar@canary to see the changes

Hi,
I get back my problem, either on my above demo or other projects. The above solution doesn't work anymore. You can try my demo link above. What happens?

Anybody knows why do @material/mwc-top-app-bar either @material/mwc-top-app-bar@canary have same problem now? Previously @material/mwc-top-app-bar@canary solved the issue.

I unfortunately cannot repro. But in your example all mwc components should have the same version. Please update mwc-drawer's version to be the same as mwc-top-app-bar.
Note that our demos on github.io run at @canary in which i cannot reproduce your issue:
https://material-components.github.io/material-components-web-components/demos/

I unfortunately cannot repro. But in your example all mwc components should have the same version. Please update mwc-drawer's version to be the same as mwc-top-app-bar.
Note that our demos on github.io run at @canary in which i cannot reproduce your issue:
https://material-components.github.io/material-components-web-components/demos/

Well, I have updated the mwc-drawer to be work @canary :
    "dependencies": {       "@material/mwc-drawer": "^0.18.0-canary.f6a6ca38.0",       "@material/mwc-icon-button": "^0.17.2",       "@material/mwc-top-app-bar": "^0.18.0-canary.f6a6ca38.0",       "lit-element": "^2.3.1"     }
But same problem :

Strange.. previously it worked?

I am still unable to reproduce this issue. Can you please provide a live reproduction case? You can fork / remix and edit https://mwc-demos.glitch.me/ with the link at the top of the page (don't forget to set all the dependencies to @canary).
Additionally, this application already uses mwc-top-app-bar-fixed. You just need to add the drawer and styling

I provided the demo on separate repo, you can just clone, install, and serve the demo app to reproduce the issue :
https://github.com/mahdiridho/actionItems-issue
I'm implementing property binding for the mwc-drawer type and the mwc-top-app-bar size

Previous one (about a week ago), my dependencies work like so :
"dependencies": {     "@material/mwc-drawer": "^0.17.2",     "@material/mwc-icon-button": "^0.17.2",     "@material/mwc-top-app-bar": "^0.18.0-canary.efdfbc21.0",     "lit-element": "^2.3.1"   }
And it solved my issue. This week I was implementing the solution to another project but doesn't work. Then I back to my demo repo to compared, strange.. It didn't work as well with my original demo.
The update dependencies doesn't work as well, now I stuck how to resolve it.

I am still unable to reproduce this issue. Can you please provide a live reproduction case? You can fork / remix and edit https://mwc-demos.glitch.me/ with the link at the top of the page (don't forget to set all the dependencies to @canary).
Additionally, this application already uses mwc-top-app-bar-fixed. You just need to add the drawer and styling

Well, I just remix the glitch demo here :
https://glitch.com/~unmarred-pewter-mayflower
Now, you can see and reproduce the issue

This is due to your use of semantic versioning. Here is a working example:
https://glitch.com/edit/#!/quasar-fearless-magazine?path=package.json%3A12%3A30
in your package.json you have:
"@material/mwc-top-app-bar": "^0.18.0-canary.f6a6ca38.0",
The ^ means to go up to the latest major version which in this case resolves to:
0.18.0-canary.fd970958.0
which is in fact an older canary commit. You need to pin it to that version using
"@material/mwc-top-app-bar": "=0.18.0-canary.f6a6ca38.0",
or if you don't mind the uncertainty
"@material/mwc-top-app-bar": "canary",
which will always be the latest version. Additionally, all components in our component set should be on the same version to prevent code duplication and possible web component name registry clashes. You should also align the version of @material/mwc-icon-button

I see
I did npm i -s @material/mwc-top-app-bar@canary and it set to semantic version.
It works back now, thanks


how to export to Onnx model for faster inference

this is a very useful project, would be great if it can be used in production with onnx support

Thank you very much for your constructive suggestions! Could you please specify which part we need to provide onnx support first? Should it be the inference part of our multimodal encoder?

Thank you! 👍
shared encoder for Text, Image, Audio, Video is the most useful one, those embedings can be used for LLM app that does audio, video retrieval;
searching task would be: image to audio search, image to video clip search, text to audio search, etc.


Make paddedBuffer less error-prone

Make paddedBuffer less error-prone by strictly checking the input address length & copying it to a pre-created 32-byte-long null buffer instead of using .padStart()
+ some code deduplication & cleanup

ty, good pr


[VSCode][DEBUG] Listener is not getting shutdown at the debug stop

Description:
$subject
Steps to reproduce:
Start a service on debug mode in VSCode and stop the debug. Service is not stopped and no way to shut them except killing the process
Affected Versions:
Ballerina v1.1.0
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Closing since this is fixed with #https://github.com/ballerina-platform/ballerina-lang/pull/28549


InternVid pretrained ViCLIP model

Hi,
The link to the pre-trained ViCLIP model in this page seems to be invalid. Could you please update it?
Thank you.

We update it here.


Getting error when deployed in Azure Service Fabric cluster

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.

[ ] A link to the published documentation article that you have feedback about.
https://docs.microsoft.com/en-in/azure/service-fabric/service-fabric-get-started-containers

[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

I followed the article and when I try to deploy to the Azure Service Fabric Cluster, I got this error message, that could not download the image and later, after adding isolation, I get this error message. CreateContainer: failure in a Windows system call: No hypervisor is present on this system. (0xc0351000)

@mstechies Please help with the link of document on which above document feedback request is created.

https://docs.microsoft.com/en-in/azure/service-fabric/service-fabric-get-started-containers

Closing as Duplicate of #13737


rclone backup writes to root directory instead of mounted emptyDir

Context
This chart creates an emptyDir (or, if backup persistence is enabled, a PVC) mount at the location given by destDir. Kubernetes sets the filesystem permissions automatically so that tar backups can be written to this directory.
The destDir option is doing double duty; for rclone backups, it also specifies the location within the rclone bucket where backups will be uploaded.
Issue
When rclone is selected as the backup method, the backup tar file is temporarily written to disk before being uploaded. However, the file is written in the current working directory, which defaults to /. On some (all?) Kubernetes providers, the root directory is not writable to containers that have dropped privileges, so unless the container has been set to run as root, the temporary file cannot be written, and the backup fails.
There's a few options for fixes:

Either this chart or the mc-backup Dockerfile should be updated to set the working directory to destDir; or
The script should be updated to write rclone temporary files to destDir; and/or
A new parameter should be added to disambiguate "backup working directory" from "rclone destination path".

It feels cleanest to decouple the two uses of destDir, but doing so could also be a breaking change for people who have rclone backup already working via a root-privileged container.

Hacky workaround: the BACKUP_NAME environment variable (default: minecraftServer.worldSaveName or "world") is normally used to set the prefix for the backup filename - but you can put slashes in it, which allows you to also control which directory it's in. Nothing else in the backup script has any problems if you do this (at least for rclone backups), so you can just set that environment variable to the same path as destDir to put it into the emptyDir/PVC mount that the chart already creates.
    mcbackup:
      destDir: same/destdir/as/before
      extraEnv:
        BACKUP_NAME: /same/destdir/as/before/world

The latest image for itzg/mc-backup now sets the working directory to /backups, so perhaps that will help with this scenario.

That makes things less hacky, but a workaround is still required if destDir is set non-default (which it will be if users need to put backups in a particular folder on the rclone remote). That workaround is now (untested):
extraVolumes:
- volumeMounts:
  - name: realBackupDir
    mountPath: /backups
  volumes:
  - name: realBackupDir
    emptyDir: {}  # or some other volume config, if you prefer

i.e. users still have to manually bind their backup volume, because this chart currently puts it at destDir (which makes sense for how tar backups use that parameter, but not for how rclone uses it).
Not sure what our tolerance is for breaking changes to the rclone integration (it's pretty new), but I suspect that if we continue using destDir to mean different things for different backup schemes, it's going to result in some confusing logic in this chart.

The backup sidecar support was only added 25 days ago, so I think it's reasonable to make a breaking change now rather than perpetuate an awkward configuration.
If you could PR something, then that would be great.


server/coordinator: fix context pitfall

According to golang package initialization, we may meet some
mysterious bugs if we do not use the same context package.
That is:
context.Canceled != context.Canceled

Because the first Canceled may comes from "context" and the
later one comes from "golang.org/x/net/context".
PTAL @siddontang @huachaohuang

Change all to context, remove golang.org/x/net/context

No, we can not, unless all packages in vendor change to "context" too.

Does etcd use context now? do we already meet this bug?