Break out Stdin processing to its own package

The process of wrapping the stdin for ghci is useful, and I would like to use it for the ghci-ng plugin for haskell-ide-engine
I can either copy and amend the code, or break it out into a separate package.
What is best?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28533753-break-out-stdin-processing-to-its-own-package?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github).

I would be happy with either solution. The code you're looking for is in src/IHaskell/IPython/Stdin.hs. It probably won't just be a simple copy/paste b/c this depends heavily on IPython-specific stuff.
Let me know if you have any questions.

I assume this does a global swap-out of stdin. In other words, once this is in place anything that does stdio in the running process will be routed through these pipes.
Which would obviously mean that any other protocol making use of stdio would be unable to run, or would get strange results.

Hmmm... I am not sure of that, actually, for two reasons.
First of all, it does a swap out of stdin in the interpreted context, not the compiled one. These probably share the raw file handles at the lowest level, but use different Handle values. So I'm not sure how it interacts with the compiled code's stdin. But my best guess is that it applies to the compiled code as well, so you would be right.
Also, either way, you could undo the change. In Ihaskell, we have to do the same thing to stdout in order to capture GHCi output. However, unlike witih stdin, we swap out stdout every time we need to run something, and then swap it back immediately afterwards. The same should be possible with stdin, although since there's a thread running to feed stdin, you would also need to clean up this thread.

Ok


Connection Query Callback has no Rows

I am struggling to get rows back from the mysql package.
Can you tell me what I am doing wrong?
I tried with 2.7.0 and 2.8.0
=== npm-mysql-tests.js ===
var expect = require('expect.js')
    , mysql = require('mysql');

describe('Npm-MySql-Testing', function() {

    it('Query', function() {

        var pool =  mysql.createPool({
            host: 'localhost'
            , port: 3306
            , user: 'root'
            , password: 'PASSWORD'
            , database: 'DATABASE'
            , waitForConnections: true
            , connectionLimit: 100
            , queueLimit: 10
        });

        pool.getConnection(function(err, connection){
            connection.query('SELECT (1+1) AS solution', {}, function(err, rows, fields) {
                if( err) {
                    console.log('Error while performing Query.');
                }

                // Never Shows
                console.log(rows);

            });

            connection.release();
        });

    });

});

Hm, I'm not sure. I see a few possibilities:

You are calling connection.release() too early--you should be waiting until you're done with the connection, i.e. the call should be inside the callback to connection.query instead of outside it.
I'm not familiar with the testing library you are using, but how does the test know you are performing an async action? Typically your testing library should be requiring you to call done() or something within the connection.query callback.

First, let me say thanks for responding.  I really appreciate you getting
back to me.  My company is experimenting with the node stack.  The fact
that you contacted me on a Sunday with suggestions is amazing.  You are
representing the node community in an extremely positive way.
I didn't realize that testing async in mocha required the done function as
the second parameter in the "it" section.  My code is at work; I will hit
it first thing in the morning.   You may have just solved my problem.  I
really appreciate you taking the time to help me.  I am so close to getting
my persistence service to work. Thank you for the assist.
On Sun, Aug 16, 2015 at 4:39 PM, Douglas Christopher Wilson <
<EMAIL_ADDRESS>wrote:

Hm, I'm not sure. I see a few possibilities:

You are calling connection.release() too early--you should be
waiting until you're done with the connection, i.e. the call should be
inside the callback to connection.query instead of outside it.
I'm not familiar with the testing library you are using, but how
does the test know you are performing an async action? Typically your
testing library should be requiring you to call done() or something
within the connection.query callback.

—
Reply to this email directly or view it on GitHub
https://github.com/felixge/node-mysql/issues/1181#issuecomment-131629891
.

It's no problem! Now that I know it's mocha, I'll try to run your example there to see what happens on my machine and see what I can find out for you :)

So some testing looks like what is happening is mocha is force shutting down the Node.js app if it thinks all the testing is complete (which is usually not how Node.js works, where usually all pending actions have to be completed before the app shuts down). The changes for the simple test above would be as follows:
var expect = require('expect.js')
    , mysql = require('mysql');

describe('Npm-MySql-Testing', function() {

    it('Query', function(done) { // <-- the done argument tells mocha this is an async test, not completed until done() is called

        var pool =  mysql.createPool({
            host: 'localhost'
            , port: 3306
            , user: 'root'
            , password: 'PASSWORD'
            , database: 'DATABASE'
            , waitForConnections: true
            , connectionLimit: 100
            , queueLimit: 10
        });

        pool.getConnection(function(err, connection){
            connection.query('SELECT (1+1) AS solution', {}, function(err, rows, fields) {
                connection.release(); // <-- call release here

                if( err) {
                    console.log('Error while performing Query.');

                    done(err); // and then, you usually want to "handle" the error, which mocha lets you pass to done
                    return; // and stop executing this function; you can alternatively omit this and put everything after this in an "else"
                }

                console.log(rows);

                pool.end(done); // because this makes a new pool, you'll also want to clean it up when done by calling pool.end and just give "done" as the callback for pool.end
                // done(); // if you didn't want to end the pool, you'd just call done() here
            });

            //connection.release(); // <-- rather than here
        });

    });

});

Created a new droplet, installed MySql 5.7, and built a new test case with
your code. Your solution works flawlessly.  I really appreciate the quick
turn around.    Thanks again.
On Sun, Aug 16, 2015 at 7:18 PM, Douglas Christopher Wilson <
<EMAIL_ADDRESS>wrote:

So some testing looks like what is happening is mocha is force shutting
down the Node.js app if it thinks all the testing is complete (which is
usually not how Node.js works, where usually all pending actions have to be
completed before the app shuts down). The changes for the simple test above
would be as follows:
var expect = require('expect.js')
, mysql = require('mysql');
describe('Npm-MySql-Testing', function() {
it('Query', function(done) { // <-- the done argument tells mocha this is an async test, not completed until done() is called

    var pool =  mysql.createPool({
        host: 'localhost'
        , port: 3306
        , user: 'root'
        , password: 'PASSWORD'
        , database: 'DATABASE'
        , waitForConnections: true
        , connectionLimit: 100
        , queueLimit: 10
    });

    pool.getConnection(function(err, connection){
        connection.query('SELECT (1+1) AS solution', {}, function(err, rows, fields) {
            connection.release(); // <-- call release here

            if( err) {
                console.log('Error while performing Query.');

                done(err); // and then, you usually want to "handle" the error, which mocha lets you pass to done
                return; // and stop executing this function; you can alternatively omit this and put everything after this in an "else"
            }

            console.log(rows);

            pool.end(done); // because this makes a new pool, you'll also want to clean it up when done by calling pool.end and just give "done" as the callback for pool.end
            // done(); // if you didn't want to end the pool, you'd just call done() here
        });

        //connection.release(); // <-- rather than here
    });

});

});
—
Reply to this email directly or view it on GitHub
https://github.com/felixge/node-mysql/issues/1181#issuecomment-131644267
.

It's no problem! I hope the comments I added to the code helped as well :) Please feel free to open issues whenever you have a question/issue :)


[DNM][MINOR] Add Github Actions to automatically add issues and PRs to support projects

Change Logs
As above.
Impact
Makes the project management easier.
Risk level
none
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Closing this PR as the functionality requires write permissions and has to be implemented in a different way given Apache's restriction on the GITHUB_TOKEN for the PRs from forks.


Error in Creating New Topic Without Any Post

Branch: rails4
When I tried to create a new topic, I got the error
undefined method `user=' for nil:NilClass/Users/echui/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/bundler/gems/forem-c3b763b5fb28/app/models/forem/topic.rb:143:in `set_first_post_user'

The root cause is that the filter before_save  :set_first_post_user is expecting a post
https://github.com/radar/forem/blob/rails4/app/models/forem/topic.rb#L141-L144
    def set_first_post_user
      post = posts.first
      post.user = user
    end

Is this a bug or a design issue?

How are you trying to create this new topic? Please provide some steps to reproduce the issue on a new app.

On 25 Apr 2015, at 03:02, edmondchui<EMAIL_ADDRESS>wrote:
Branch: rails4
When I tried to create a new topic, I got the error
undefined method user=' for nil:NilClass/Users/echui/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/bundler/gems/forem-c3b763b5fb28/app/models/forem/topic.rb:143:in set_first_post_user'
The root cause is that the filter before_save :set_first_post_user is expecting a post
https://github.com/radar/forem/blob/rails4/app/models/forem/topic.rb#L141-L144
def set_first_post_user
  post = posts.first
  post.user = user
end

Is this a bug or a design issue?
—
Reply to this email directly or view it on GitHub.

Here is how I create a topic
# creates a valid Forem::User
user = User.create!
forum = Forem.create!
topic = Forem::Topic.create(subject: "New Topic", forum_id: forum.id, user_id: user.id, state: :approved)

You are not including post text with create and since there is no post text, there will be no post and you'll get the exception. This works:
topic = Forem::Topic.create(subject: 'New Topic', forum_id: forum.id, user_id: u.id, posts_attributes: {'0' => {text: 'Post Text'}})

What Yan-Hoose said.

Thanks @yan-hoose.
My intention is to create an empty topic to start with. That's why I didn't provide a post text. Is that a valid use case?

But I'm not sure if a topic with only a subject and without content is useful. A topic should have something in it or it would be meaningless. That is what Forem assumes, that you have a topic with a first post as the topic text.


Impossible to write

Hello, I m trying to use your ACDDokanNet but I Have some issues:

Download folder to my computer works
Upload file Don t work. It say me I haven't permission to write (Windows Message)

Can you help me ? (I m starting the program as an administrator)
Regards

Are you uploading just in Windows Explorer or other application?

Only using windows explorer. Mount works great but I Haven t rights to write :/

Issue is fixed. The cache folder was set to read only !!!


Download mosquitto certificate on the fly

moved from https://github.com/beerfactory/hbmqtt/pull/232

Closing in favor or #10 as force push after rebase seems to update only the original MR.


Use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter

When reviewing your template we've noticed that it is using parameters to build a url. It's a best practice to use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter.

@yeeha567 Hi! Thank you for you feedback!
mate you working for Google?
We are not using the any of them for URLs we are using them to encode the cookie value.


temp: remove comment-appeal from front-door

Pull Request Template
Describe your changes
Not to merge - need to fix this properly and conditionally add comment appeal to front door per environment.
This is to get around an issue deploying to PROD as we have no URL for the comment-appeal front end.
Useful information to review or test

Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[ ] My code follows the style guidelines of this project
[ ] My code changes do not include any hardcoded secrets or passwords
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] My Tech Lead is aware of any infrastructure changes that will cost money

Worked around with a temporary URL:
https://github.com/Planning-Inspectorate/infrastructure-environments/pull/581


[RSDK-3404] Add comment to camera proto regarding intrinsic and distortion parameters for transform cameras

This PR adds a short comment to the camera proto file because users seemed to be confused when transform cameras errored if they were passed intrinsic and distortion of the underlying camera. These underlying parameters are not used for transforms and they should be left empty. The transforms will then use an empty cameraModel for initialization and function properly.

another thing you'll want to do it make all the derived proto files, too

Yeah, I think I get it a little bit better now, I was under the assumption that users were only running into this for transform cameras, but the error can occur any time that the camera's parameters are set to zero, right?


rubygems has 1.3.24, this repo has only 1.3.23

https://rubygems.org/gems/kramdown-rfc2629 shows 1.3.24 as the latest version, but the master branch here  is at 911495f5b3db820aa2e090e3ba0e5e6227de8914, which is only 1.3.23
maybe 1.3.24 hasn't been pushed?

Fixed.  Sorry about that, and thanks for the reminder.

Fixed.  Sorry about that, and thanks for the reminder.


Never include the current page in breadcrumb nav

What is the context of this PR?
GDS guidance on breadcrumbs states:

The breadcrumbs should start with your ‘home’ page and end with the parent section of the current page.

So, we need to remove our conditional inclusion of the current page, and instead never include it as the final item.

Normally I'd expect the template logic to hide the current breadcrumb page - but I guess it is possible in the model code because of the strict page hierarchy. This works fine for the prototype.

ignore this - I misread the template change


MKV to MKV, reducing size but leaving interlaced?

Hi Don,
This may sound strange, but what I'm getting at is trying to solve the problem that some interlaced DVD rips just don't deinterlace as well as I'd like. I have a little Perl script that produces a 3 minute cut of an MKV produced by MakeMKV for each deinterlace method offered: default, deinterlace, detelecine and decomb. It also outputs the same cut as the original MKV. The output goes to a share Plex sees, so I can easily compare all 5 on a TV screen.
In some cases, usually TV shows, it's easy to see the loss of quality compared to the original MKV. I'm not an expert on all the possible transcode-video & Handbrake options, but I have experimented with --target big in combination with --cvbr, and still no go.
What I usually do in a case like this is just forget about transcode-video and stick the MKV right on my Plex server. My Plex clients are a Raspberry PI 2 and 3 running Kodi with a beta plugin called Plex for Kodi. This client displays the MKV beautifully without requiring a Plex transcode.  Of course now I have extra large MKVs on my server.
So I was thinking. If Kodi & Plex & Raspberry do such a nice job deinterlacing, why not let them do that against a reduced size MKV? I may be completely off-base here, but does it make any sense to have an option for transcode-video to apply his marvelous algorithms without deinterlacing, producing either an MKV or MP4? My hope is that I'd get the benefit of reduced size, and let the client handle deinterlacing.
My apologies if this boils down to a dumb question, I don't know if in my ignorance I'm asking for something counter to the design.
Thanks,
Mark M

@markmarz There's no option in HandBrakeCLI to preserve interlaced input when transcoding. You could pass options through to the x264 encoder and enable interlaced transcoding mode, but that really only works well if the input is actually progressive, i.e. not interlaced.
So, your feature request is, unfortunately, not really practical. Sorry about that.
I realize you experimented with the various filters already, but it's likely that "detelecine" would have worked for you if you also forced a frame rate. This is what I do problematic videos like the ones you describe:
transcode-video --filter detelecine --force-rate 23.976 "/path/to/Movie.mkv"

That's assuming, of course, that the original or correct frame rate is 23.976 FPS.

@donmelton Oh, well, too bad on the interlacing option. OTOH I'm looking forward to experimenting with the frame rate in combination with detelecine. I've had good luck with detelecine in general, I'm confident your frame rate suggestion will help a lot.
Thanks!
Mark

@markmarz If it doesn't, open another issue and let me know. Good luck, sir!

I've successfully been able to preserve interlaced video by using the -E switch to add bff or tff tag for the x264 encoder.  it does have some drawbacks as I get a warning about interlaced and weightp not being compatible, and the need to determine field order. In addition I made sure to specify --filters for no telecine, no decomb, no comb detect, no deinterlace.

@donmelton I've successfully been able to preserve interlaced video by using the -E switch to add bff or tff tag for the x264 encoder.  it does have some drawbacks as I get a warning about interlaced and weightp not being compatible, and the need to determine field order. In addition I made sure to specify --filters for no telecine, no decomb, no comb detect, no deinterlace.

Hi Don,
Thanks very much for your diligence!
Unfortunately I'm not clear on the -E switch. I've checked the Handbrake
CLI guide and the only -E option I can find is for Audio, although I did
find an -e switch for Video. I've played around with -Ebff and -Etff, -ebff
and -etff and they're all invalid.
Here's one attempt:
transcode-video --target big --filter no-detelecine --filter no-decomb
--filter no-comb-detect --filter no-deinterlace --force-rate 29.970 -Ebff
"E:\dvd_input\Monk\Season 01/Monk - S01E04.mkv" -o
"E:\dvd_output\Monk\Season 01\Monk - S01E04 bff.mkv"
Please correct my attempt when you get a chance. Thank you very much!

Mark M.

On Fri, Mar 8, 2019 at 12:45 PM docmcfreckles<EMAIL_ADDRESS>wrote:

@donmelton https://github.com/donmelton I've successfully been able to
preserve interlaced video by using the -E switch to add bff or tff tag for
the x264 encoder. it does have some drawbacks as I get a warning about
interlaced and weightp not being compatible, and the need to determine
field order. In addition I made sure to specify --filters for no telecine,
no decomb, no comb detect, no deinterlace.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/donmelton/video_transcoding/issues/126#issuecomment-471033577,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb7beI8_i7OYE8KXqgAzhAOsmk0Vj6xks5vUq_MgaJpZM4L_37S
.

@docmcfreckles Yeah, the hard thing is knowing whether to use the top or bottom field first. :) Also, you can disable weightp with --encoder-option weightp=0 or -E weightp=0 so you don't get the warning.
@markmarz The -E option that @docmcfreckles mentioned is not from HandBrakeCLI, it's from transcode-video. And it's a shorthand for --encoder-option. And you need a space between the option and the argument, plus a value for that argument key, like this: E bff=1
Here's a corrected version of your command line:
transcode-video --target big --filter no-detelecine --filter no-decomb \
--filter no-comb-detect --filter no-deinterlace --force-rate 29.970 -E bff=1 \
"E:\dvd_input\Monk\Season 01/Monk - S01E04.mkv" -o \
"E:\dvd_output\Monk\Season 01\Monk - S01E04 bff.mkv"

Does that help?

Thank you!
On Fri, Mar 8, 2019 at 3:30 PM Don Melton<EMAIL_ADDRESS>wrote:

@docmcfreckles https://github.com/docmcfreckles Yeah, the hard thing is
knowing whether to use the top or bottom field first. :) Also, you can
disable weightp with --encoder-option weightp=0 or -E weightp=0 so you
don't get the warning.
@markmarz https://github.com/markmarz The -E option that @docmcfreckles
https://github.com/docmcfreckles mentioned is not from HandBrakeCLI,
it's from transcode-video. And it's a shorthand for --encoder-option. And
you need a space between the option and the argument, plus a value for that
argument key, like this: E bff=1
Here's a corrected version of your command line:
transcode-video --target big --filter no-detelecine --filter no-decomb
--filter no-comb-detect --filter no-deinterlace --force-rate 29.970 -E bff=1
"E:\dvd_input\Monk\Season 01/Monk - S01E04.mkv" -o
"E:\dvd_output\Monk\Season 01\Monk - S01E04 bff.mkv"
Does that help?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/donmelton/video_transcoding/issues/126#issuecomment-471082716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb7bYouzumwDFY047Xl_CqBUm2bodOfks5vUtZzgaJpZM4L_37S
.

@markmarz  as don mentioned the tricky bit is determining whether you have bff or tff in your interlaced content. the metadata that the encoder gives isn't always accurate. One shortcut I've used is to have ffmpeg give you it's best determination for a section of the video
ffmpeg -filter:v idet \ # use the video filter idet
-ss 00:10:00 \ # fast forward 10 minutes into the source
-frames:v 1000 \ # sample 1000 frames
-an \ # strip audio
-f rawvideo \ # output the frames as raw video for the idet filter to operate on
-y /dev/null \ # dump the output to /dev/null (*nixism) after we're done with it
-i title01.mkv```

I used this for my box set of Married with Children, where the opening credits were shot on film, telecine'd with a 3:2 pulldown but the body of the show was shot video, necessitating the jump ahead in the source.

The authoritive way (that the internet told me) to determine bff/tff is to split the fields with something like avisynth and then view them by hand to determine order but I don't have the patience and the whole thing confused me as I didn't know what I was looking at. hence the "good enough" solution by getting ffmpeg to guess given above.

Ah, Don's top/bottom comment went right over my head. Thank you for
clarifying this and offering a workable method to figure it out.
The output I get from your ffmpeg invocation includes:
Stream #0:0(eng): Video: mpeg2video (Main), yuv420p(tv, top first),
720x480 [SAR 32:27 DAR 16:9], SAR 186:157 DAR 279:157, 29.97 fps, 29.97
tbr, 1k tbn, 59.94 tbc
top first = TFF, right?
On Sat, Mar 9, 2019 at 12:45 AM docmcfreckles<EMAIL_ADDRESS>wrote:

@markmarz https://github.com/markmarz as don mentioned the tricky bit
is determining whether you have bff or tff in your interlaced content. the
metadata that the encoder gives isn't always accurate. One shortcut I've
used is to have ffmpeg give you it's best determination for a section of
the video
ffmpeg -filter:v idet \ # use the video filter idet
-ss 00:10:00 \ # fast forward 10 minutes into the source
-frames:v 1000 \ # sample 1000 frames
-an \ # strip audio
-f rawvideo \ # output the frames as raw video for the idet filter to operate on
-y /dev/null \ # dump the output to /dev/null (*nixism) after we're done with it
-i title01.mkv```
I used this for my box set of Married with Children, where the opening credits were shot on film, telecine'd with a 3:2 pulldown but the body of the show was shot video, necessitating the jump ahead in the source.
The authoritive way (that the internet told me) to determine bff/tff is to split the fields with something like avisynth and then view them by hand to determine order but I don't have the patience and the whole thing confused me as I didn't know what I was looking at. hence the "good enough" solution by getting ffmpeg to guess given above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/donmelton/video_transcoding/issues/126#issuecomment-471151866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb7bVovF3HSStkoMZmIQ0P9C_fqdaqEks5vU1h2gaJpZM4L_37S
.

Ah, Don's top/bottom comment went right over my head. Thank you for
clarifying this and offering a workable method to figure it out.
 :) Seriously, I forgot all about the idet filter in ffmpeg. Thanks!
@markmarz Yes, that means "top first" so you would use -E tff=1 on your transcode-video command line instead.

@markmarz actually, that is the information that comes from the encoder metadata, not what ffmpeg thinks of the input... at the bottom you should see a report:
video:506250kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.000000%
[Parsed_idet_0 @ 0x557144f17600] Repeated Fields: Neither: 18980 Top:     2 Bottom:     1
[Parsed_idet_0 @ 0x557144f17600] Single frame detection: TFF:     0 BFF: 17429 Progressive:   143 Undetermined:  1411
[Parsed_idet_0 @ 0x557144f17600] Multi frame detection: TFF:     0 BFF: 18965 Progressive:     4 Undetermined:    14

if most of the frame detection falls in the TFF: column, ffmpeg thinks it's tff, if most falls in BFF: then ffmpeg thinks it's bff, and if it Progressive: wins, congrats you don't have to bother with interlaced...
@donmelton thanks for the kudos, but, I can't claim credit, i'm a lowly hacker who can just google better than the average bear ;-)

Thanks for the clarification! The upshot for me is to favor alternatives to
preserving interlacing if possible. Too much uncertainty.
In my use case I can live with interlaced video even though I can't take
advantage of transcode-video, because my clients do a good job handling
interlaced video.
On Tue, Mar 12, 2019 at 4:02 AM docmcfreckles<EMAIL_ADDRESS>wrote:

@markmarz https://github.com/markmarz actually, that is the information
that comes from the encoder metadata, not what ffmpeg thinks of the
input... at the bottom you should see a report:
video:506250kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.000000%
[Parsed_idet_0 @ 0x557144f17600] Repeated Fields: Neither: 18980 Top:     2 Bottom:     1
[Parsed_idet_0 @ 0x557144f17600] Single frame detection: TFF:     0 BFF: 17429 Progressive:   143 Undetermined:  1411
[Parsed_idet_0 @ 0x557144f17600] Multi frame detection: TFF:     0 BFF: 18965 Progressive:     4 Undetermined:    14
if most of the frame detection falls in the TFF: column, ffmpeg thinks
it's tff, if most falls in BFF: then ffmpeg thinks it's bff, and if it
Progressive: wins, congrats you don't have to bother with interlaced...
the ugly bit is that just because the encoder metadata says something's
interlaced a certain way, it can contain mixed content, some parts
progressive, some interlaced, some shot on film and telecined with
pulldown... it's an ugly business. I try to pick a spot in the middle (of a
tv show) that is representative, so the main body will at least be right.
that doesn't necessarily help with shows that are montage shows where the
clips haven't been cut together with the most delicate care, and you aren't
really sure what you'll get from scene to scene. but I digress.
@donmelton https://github.com/donmelton thanks for the kudos, but, I
can't claim credit, i'm a lowly hacker who can just google better than the
average bear ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/donmelton/video_transcoding/issues/126#issuecomment-471910649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb7bRnhhwvnsF3DZIb_ZPEprPdYtTdgks5vV20agaJpZM4L_37S
.


QuickInput controller triggers layout even when not visible?

While working on https://github.com/microsoft/vscode/pull/160103 I noticed two things:

we seem to be creating the QuickInputController anytime you access QuickInputService.controller which imho is often an unwanted side effect, esp. for the onShow and onHide events that only care about quick input actually being used (for those, my PR addresses this, but its unclear to me in other cases, e.g. shouldn't focus method return when controller is undefined?
we layout the underlying widget but the widget is not visible at that point? unclear to me the purpose:

https://github.com/microsoft/vscode/blob/a27fc69ce51852e47e0d3f1ef5297f4551f4cac5/src/vs/platform/quickinput/browser/quickInput.ts#L87
//cc @chrmarti

@mjbvz actually fixed this recently and now we only layout when the quickpick is displayed:
https://github.com/microsoft/vscode/commit/11a7e20a9a6a1b7e49b719a48a45b0cd4ac5eeab


Search paths not populated in container resolv.conf

Rancher Version:
v0.59.1
Docker Version:
1.10.2
OS:
Ubuntu 14.04.4 LTS
Steps to Reproduce:

Add a new Digital Ocean Host  (has Docker 1.10.2 preinstalled)
Add a new service (e.g. using redis:3.0 image) to that host
Start the service and execute a shell
Issue the command cat /etc/resolv.conf

Results:
# Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
#     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
# nameserver <IP_ADDRESS>
# nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

Expected:
# Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
#     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
# nameserver <IP_ADDRESS>
# nameserver <IP_ADDRESS>
# nameserver <IP_ADDRESS>
search local rancher.internal
nameserver <IP_ADDRESS>

@janeczku this PR added support for adding rancher-internal domain to search section:
https://github.com/rancher/host-api/pull/58

A bit confused what this PR is supposed to do. Right now even without this patch the search domains get added to resolv.conf on my Docker v1.9 hosts as they always have. Just not on the Docker v1.10 hosts.

@janeczku before the fix, rancher.internal was added to the search path only when the search path wasn't empty - I see you have "local" prepended, it might have been pre-set somewhere (through search domain, or may be automatically by docker 1.9).
With the fix I've mentioned above, default search domains - rancher-internal is one of them - is being set no matter if the search already had been set with some initial values.
I've tested it on latest Rancher RC and docker 1.10.2 for redis:3.0 image. Search domain is set to:
search default.rancher.internal redis.default.rancher.internal rancher.internal

first 2 domains are service's/stack's namespaces.

before the fix, rancher.internal was added to the search path only when the search path wasn't empty

Oh that explains it! The Digital Ocean hosts indeed don't have any search prefix preconfigured. Thanks for the clear up - i'd never have guessed this to have caused it.

Tested with latest build from master:

Add a new Digital Ocean Host (has Docker 1.10.2 preinstalled)
Add a new service (e.g. using redis:3.0 image) to that host
Start the service and execute a shell
Issue the command cat /etc/resolv.conf

I see rancher.internal being appended to the search path.

seems like this pr caused another bug, container's resolv.conf will be flushed periodically even container's resolv.conf is not empty.


Refactor and implement JWT auth

Closes #19 and #49. Going to need everyone's approval for this because a lot was changed and/or refactored.

Typo found in users.js that could lead to runtime error. Apart from that the application runs well on my end.

Thanks. The issue should be fixed.

Looks good now, except for the use of _id in the backend vs id in the frontend. It can be resolved by renaming all the ids in frontend to _id.

Done. Thanks

Merged to "chat-conversation" branch


Unity 2022.3

This Mesh Collider is attached to GameObject at path 'Chunk Pool/Pooled Chunk(Clone)/Main Mesh' with Mesh 'Terrain' in Scene 'SampleScene', but the mesh doesn't have any vertices.
UnityEngine.StackTraceUtility:ExtractStackTrace ()
Chunks.MeshGeneration.ChunkMeshGenerator:Awake () (at Assets/Scripts/Chunks/MeshGeneration/ChunkMeshGenerator.cs:159)
Pooling.GameObjectPool:Initialize (UnityEngine.GameObject,UnityEngine.Vector3) (at Assets/Scripts/Pooling/GameObjectPool.cs:16)
Pooling.PoolBase`1<UnityEngine.GameObject>:GetObject (UnityEngine.Vector3) (at Assets/Scripts/Pooling/PoolBase.cs:14)
Chunks.World:SpawnNearChunks (Unity.Mathematics.int2,bool) (at Assets/Scripts/Chunks/World.cs:90)
Chunks.World:UpdateChunks (bool) (at Assets/Scripts/Chunks/World.cs:71)
Chunks.World:Start () (at Assets/Scripts/Chunks/World.cs:153)

I no longer maintain the project so you might have to try and fix this on your own by forking the repo.


A copy of ProductImagePresenter has been removed from the module tree but is still active!

Hi,
I am currently facing a really weird problem and I hope one of you is able to help me solving it.
I have 2 Presenter Classes inside classes/. Every time I change the content which includes the shortcodes I get a error message like this:
A copy of ProductImagePresenter has been removed from the module tree but is still active!
If I restart the rails server it is working, but this is not the solution.
Files:
/app/classes/product_image_presenter.rb
/app/classes/product_name_presenter.rb
/app/views/shortcode_templates/product_image.html.erb
/app/views/shortcode_templates/product_name.html.erb
Example Content:
[product_image id="4482698"]
[product_name id="4482698"]

Hi, I assume you are using Shortcode within a rails app. This error happens because of the way rails reloads classes in development mode, you will need yo reload your rails app each time you make a change to a presenter.

The point is, I am not changing the presenter. I am changing the content.E.g.
From:
[product_image id="4482698"]
[product_name id="4482698"]
To:
[product_image id="1"]
[product_name id="1"]

You haven't provided enough information to help. I would suspect the issue is somewhere in the way you have coded your application but it's impossible to say without seeing all of the code relating to how you are invoking the gem, the presenters and templates.

Do you put you presenters in the same folder? Maybe thats the problem?

 It's impossible to help without seeing your code.

That's easy to solve. I have the feeling the reason is the Modul called Helper.
Path /app/classes/product_image_presenter.rb
class ProductImagePresenter
  include HelperModule

  def self.for
    :product_image
  end

  def initialize(attributes, content, additional_attributes)
    @content = content
    @attributes = attributes
    @id = attributes[:id]
  end

  def content
    @content
  end

  def attributes
    {
        id: @id,
        name: name
    }
  end

  private

  def name
    product = Product.where("id = ?", @attributes[:id].to_i).first
    product.name unless product.nil?
  end
end

Controller:
Path /app/controllers/compare_controller.rb
class CompareController < ApplicationController
  def index
    # variable to test if the shortcodes work
    @description = '[product_image id="4482698"]'
  end
end

View:

Compare#index

Find me in app/views/compare/index.html.erb

<%= shortcode_with_html @description %>

Shortcode Template (Just to test it):
<%= @attributes[:name] %>
Compare Helper:
Path: app/helpers/compare_helper.rb
module CompareHelper
  def shortcode(c)
    Shortcode.process(c)
  end

  def shortcode_with_html(c)
    raw shortcode(c)
  end
end

Helper Module
Path: app/classes/helper_module.rb
module HelperModule
  def image_host
    if Rails.env.production?
      "..."
    else
      "localhost:3000"
    end
  end
end


Previous/Next buttons aren't fully accessible

I'm wrapping your excellent component inside a Lit element and providing override icons for the next/previous buttons. The Storybook a11y checker is identifying accessibility issues with the buttons as they don't have an aria-label or a tooltip. I'm not sure how this can be fixed in both an a11y and an i18n friendly way though, so don't have any suggestions or a PR I'm afraid :(

It should just be a case of adding an aria-label to your icons. See how it's done here with shoelace icons https://wicky.nillia.ms/cally/guides/usage/#calendar-components

This is something that needs solving your end. The buttons are labelled by default in Cally. So I will close, but happy to discuss further

This is something that needs solving your end. The buttons are labelled by default in Cally. So I will close, but happy to discuss further

Of course ! I was so focused on the buttons I didn't think of this, sorry for the wasted issue :)

No worries, easily done 👍


Search SDK: Prep work for enabling custom serialization

There are a few issues in the Azure Search .NET SDK related to serialization:
https://github.com/Azure/azure-sdk-for-net/issues/1259
https://github.com/Azure/azure-sdk-for-net/issues/1363
This PR is the first step to addressing them. See individual commits for details. Note that this work is about enabling custom serialization of model classes to documents. It affects the data plane of the SDK, which is currently all custom code.

I'm using an older version of TestFramwork that refers to JSON.NET 6. I'm looking for a workaround since the latest version isn't in NuGet yet, and I can't use a project reference since I can't port the Search SDK to DNX yet (long story).


Implement validation

Link to the original specification. Some of the requirements:

Queries and mutations share the same namespace. Therefore their names must be guaranteed to be unique.
Anonymous queries/mutations should not be allowed in documents having more than one operation.
Fragments defined on some types are not allowed to introduce names of fields, that don't exists in referred type - aliases are allowed.
Scalars must be leaf nodes of the operation.
Object types, interfaces or unions must not be leaf nodes of the operation - query/mutation must specify all necessary fields in that case.

Closing this one. More details can be found in #24


Inconsistent action/actions argument name (Runner / Environment.execute)

In a recent commit (fd30c0fd7faa64f5df9232bfa75907b09dc377e9), the parameter name "actions" was changed to "action" in most environments. In execution/runner.py (line 104) "actions" is still used, which means most environments don't work. If "action" is to be the new name, I think the following environments will have to be changed too, because they still use "actions":

DummyEnv
OpenAIUniverse
MazeExplorer
MinimalTest

@daggertye : Maybe you already have the changes locally but they were missing from the commit?

Yeah, I forgot this. The fix is in #451.


Updated README.md

Corrected the documentation link. Old one was not working.

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

Looks like it's duplicate of that PR: https://github.com/dotintent/react-native-ble-plx/pull/868


Update mix.lock to be compatible with hex v0.20.2+

Yesterday’s hex 0.20.2 release requires us to regenerate mix.lock checksums.

🦀 Requesting reviewers for this pull request:

@garno (contributor with 0 commits in the last 90 days and 11 commits overall)
@MathieuLegault1 (expert reviewer for the elixir stack)

@ericmj has an interesting comment on the forum thread I started on the forum!
https://elixirforum.com/t/docker-build-started-failling-today/28844/6

This happens when you have a manifest file that is newer than your mix.lock file. That’s because you run mix deps.get which updates your manifest and lock file but then your dockefile’s COPY command introduces the old lockfile again. I would recommend against doing COPY . . in your dockerfile, you should instead only copy what is needed for the next build step.
I will publish a new release of Hex that mitigates this issue.

I will explore on this avenue tonight!

Alright, I’ll close this PR since it only provides a short-term solution to the current situation — it doesn’t protect against future breaking changes that might occur with hex.

Sorry for interjecting (I was mentioned above). It is still a good idea to update the lockfile to the new format since it is more secure and most mix deps.* tasks will try to update to it anyway.

@ericmj Yes, I spoke with @gcauchon and his next pull request (based on your COPY . . comment) will do that as well 😄

^this 🍻


Stack templates wont deploy due to deprecated Lambda Python runtime version being referenced

Problem
CentralizedArchitectureApplianceVpc2Az.yaml stack template wont deploy due to deprecated Lambda Python runtime version being referenced.
2024-09-04 13:27:08 UTC+0100

VpceServiceName

CREATE_FAILED

-

Resource handler returned message: "The runtime parameter of python3.7 is no longer supported for creating or updating AWS Lambda functions. We recommend you use a supported runtime while creating or updating functions. (Service: Lambda, Status Code: 400, Request ID: obfuscated)" (RequestToken: obfuscated, HandlerErrorCode: InvalidRequest)

Testing with Runtime: python3.12 worked fine.
Solution

Replace the deprecated Python runtime reference with the latest 3.x version.
Before:
Runtime: python3.7
After:
Runtime: python3.12
Code path - https://github.com/aws-samples/aws-gateway-load-balancer-code-samples/blob/main/aws-cloudformation/centralized_architecture/CentralizedArchitectureApplianceVpc2Az.yaml#L968

Update in Git repo.

Update the files in s3://tech-content-us-west-2.s3-us-west-2.amazonaws.com to make sure that the 1-click solution works as well.

Due to this issue, the 1 click deployment is also broken.
https://us-west-2.console.aws.amazon.com/cloudformation/home?region=us-west-2#/stacks/create/template?stackName=GwlbCentralizedDemo&templateURL=https://tech-content-us-west-2.s3-us-west-2.amazonaws.com/aws-gwlb-cloudformation-samples/centralized-architecture/nested-stack/CentralizedArchitecturePrimary.yaml
Now I need to demo to customers, but host the files myself which is a bit of a pain.

Fixed. Thanks.


Documentation for literals

Fixes #3078

I think it would have been more semantically correct to replace `{false, true}` with `{false}`, `{true}`, and similarly for other lines that have multiple examples.

@MDLC01 feel free to send a followup PR


Design implementing

NYC logo follows you down the screen (May be able to remove this?)
Title section that doesn't dispay on mobile
Image section
Story panel
Inputs for size of family

@cweems


Added tf2pickup.rs

Added tf2pickup.rs branding. Logo is a WIP and will be changed later.

I updated the domain and fixed the typos. Im waiting for bobair to do the graphics so we should wait for her until we merge.
How about a version bump aswell?

I removed them now. Thanks for the heads up!


Rename or remove cilium cleanup

Cilium cleanup sounds pretty innocent, many people like things to be clean.
Unfortunately the actual functionality of this is dangerous and invasive, and occasionally users stumble onto the option while trying to solve another problem. Rather than cleaning things up and helping to get Cilium into a better state, this leads the user into further problems.
https://github.com/cilium/cilium/blob/5ab0230bb9f0566bf36375e9640a6afa802b681b/cilium-dbg/cmd/cleanup.go#L30-L33
We should consider at least renaming the option to make it clearer that this is some sort of "force-destroy" of state (flush also works, but flush doesn't sound dangerous enough).
We may also consider dropping the feature, however maybe it's still useful for something like uninstall.

@joestringer I can work on this issue, can you please assign it to me

@shardulsrivastava sure, do you have a proposal or PR to discuss?

@joestringer could assign it to me?

@hemanthmalla mentioned he's still using the command, so we probably want a renaming rather than a removal.


chore(main): release 1.4.9

:robot: I have created a release beep boop
1.4.9 (2024-02-13)
Miscellaneous Chores

deps-dev: bump phpstan/phpstan from 1.10.57 to 1.10.58 (#85) (e889d57)
deps-dev: bump phpunit/phpunit from 11.0.2 to 11.0.3 (#84) (76938af)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.4.9 :sunflower:


It would be even greater if I could backup or share my library online, as my library is now probably a file only a couple of megabytes.

I have submitted a feature request for better library integration as well.
It is the one thing holding me back from using pulsar regularly. If you are
interested I coded a set of bash scripts that basically do what you want
for the older xbmctorrent plugin, they are a bit cludgy, but they work and
build a set of strm files that can be added to your library, look up
torrentscrape on github if you want to take a look. At the very least it
will give you the functionality you want until pulsar has the proper
support.

It would be even greater if I could backup or share my library online, as
my library is now probably a file only a couple of megabytes.
—
Reply to this email directly or view it on GitHub
https://github.com/steeve/plugin.video.pulsar/issues/79.

I saw your post but felt I should create a separate feature request because your implementation and my request differ in a significant way. Your project does automatic searching, and my request is more for saving things I have searched for once and want to keep.
I agree that for video series/podcasts, some kind of process that goes and looks for new episodes is definitely valuable, but my specific request is for just a subset of that functionality, for example, saving a single standalone public domain movie.

isnt this library integration something that mancuniancol has done with a third part aplication provider for pulsar?


Hosted Service fails to start subgraph

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
When attempting to start the subgraph after deploying to Hosted Service, I get the following error:
Failed to start subgraph, code: SubgraphStartFailure, error: expected triggers adapter that matches deployment Qm... with required capabilities:
traces: A matching Ethereum network with NodeCapabilities { archive: false, traces: true } was not found.

I've tried rinkeby, avalanche and aurora-testnet, all of which produce this error.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
graph init --product hosted-service
(graph -v prints 0.26.0)
Protocol: ethereum
Subgraph name: {USER}/decoy
Directory: decoy
Ethereum network: rinkeby
Contract address: 0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D
Contract Name: Contract

cd decoy && yarn deploy
Might need to fix subgraph.yaml a bit. When I deploy this I get the same error and it never starts or syncs.
What is the expected behavior?
No errors.

Hey @giraffekey - I assume your subgraph is using callHandlers, these require trace filtering by Ethereum nodes, which isn't supported by the networks you mention, so the subgraph will need to use event or blockHandlers only. Closing as not a graph-node issue. Please do re-open if you are still having issues!


(7.1.0 branch) Enabling the legacy property view

Documentation the impact of enabling the legacy property view in Integration Studio.
Fixes: https://github.com/wso2/docs-ei/issues/1698

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: samgnaniah:white_check_mark: RukshiW:white_check_mark: sajithshn:white_check_mark: nilminiwso2:white_check_mark: senthuran16:x: 10669450You have signed the CLA already but the status is still pending? Let us recheck it.


Add rules: scss-function-pattern, scss-mixin-pattern (updated branch)

Implements #12
An updated (confilcts-free) version of #15

Updated the branch and rebased it to another one. Some commits already in the repo got into th PR, but git seems ok with that :) Will that go fine?

Some commits already in the repo got into th PR, but git seems ok with that

Hmm... I'm not sure. I think that merging this will cause the same commits to show twice in commit history. At least that's what has happened before.
How did you rebase?
I usually do it this way:

Pull the latest changes (git checkout master before if you need to)
git checkout mybranch (you should use a custom branch instead of master)
git rebase master
Fix conflict(s)
git add file-that-conflicts
git rebase --continue (now there should be no conflicts left)
git push origin mybranch -f use force to overwrite history (the conflicting commits in your branch)

I did almost the same except for not including -f flag in 7) and pushing to my fork's master. Now I did exactly as you suggested, yet the commits not from my fork are still shown here...

Ok #18 should work.

👍

Closing this, let's continue with #18


Bidirectional Tilted Blinds should start at 50% not 0% for some devices

System Health details
System Information

version
core-2023.10.2

installation_type
Home Assistant Container

dev
false

hassio
false

docker
true

user
root

virtualenv
false

python_version
3.11.5

os_name
Linux

os_version
6.1.57-1-lts

arch
x86_64

timezone
America/Los_Angeles

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.33.0

Stage
running

Available Repositories
1321

Downloaded Repositories
39

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Dashboards

dashboards
3

resources
21

views
13

mode
storage

Recorder

oldest_recorder_run
October 20, 2023 at 12:05 AM

current_recorder_run
October 26, 2023 at 2:07 PM

estimated_db_size
379.03 MiB

database_engine
sqlite

database_version
3.41.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
Thank you for creating this integration, it makes it as easy as possible to deploy these calculations to a sensor!
Currently, when using tilted blinds in Bidirectional mode with my SwitchBot Tilt devices, they start at 0% (Closed down) and move to a peak of 50% (fully open) when the sun is the strongest. After that, they move back down to 0% (fully closed down) as the sun moves off of them.
What I would like to have an option for is to have this occur in the upper range of the bidirectional mode. So start at 50%, peak at 100% when the blinds need to be fully closed (facing up) as they are getting a lot of sun, and then move back to 50% (open) as the sun is not hitting them as much.
Would it be possible for you to implement this as a setting if it is not a bug for all users?
Thank you!
Reproduction steps

Setup a Bidirectional Tilted blind
Assign the integration to a SwitchBot blind tilt device.
Observe (Image shows me changing configuration settings, see after 6 AM on purple line or the descent of the teal line around 12pm on the previous day)

Debug logs
N/A

Diagnostics dump
No response

This is already possible with the inverse state option that changes the range from 0-100% to 100-0%

This is correct. Sorry


Migrate /raid command to Helix API

As part of Twitch's announced deprecation of IRC-based commands (see here for more info, the /raid command needs to be migrated to use the relevant Helix API endpoint.
Helix API reference: https://dev.twitch.tv/docs/api/reference#start-a-raid
Split from #3956

As per @iProdigy's analysis on #3956 (comment)

/raid & /unraid are now broadcaster only commands

As per @iProdigy's analysis

Relevant uservoice: https://twitch.uservoice.com/forums/310213-developers/suggestions/45281644-allow-channel-editors-to-use-helix-raids-api


Add language support to DatePicker

Fix for #343
Adds localisation to the Log's DatePicker.
Tested in all currently supported languages + incomming french.
Defaults to english if no AspNetCore cookie found.
Support for chinese is a bit janky as the picker supports two different chineses variants but doesn't have a neutral one (no "zh", only "zh-CN" and "zh-TW").
GetCookie function could be replaced by a jQuery plugin but i did not want to alter this project too much, as i am a new contributor.

Great work, I am happy that you made these all changes in PRs. I will review/merge everything at the weekend - I hope. 😊👍🏼
Feel free to continue.
Thank again for your work!


[request] change copy to clipboard for color picker to pure RGB values

Description of the new feature / enhancement
Following issue:

The Copy to clipboard button copies the values as:
"rgb(204, 255, 51)"
This is annoying and counterproductive, because you cant use that in most applications or programing languages. Change it to:
"204, 255, 51"
Scenario when this would be used?
Mostly in every scenario, where there doesnt exist a "rgb()" procedure.
Supporting information
No response

/dup #8305

Im confused. So you made it worse over time? As I understand, it was possible in the past to define your own rules and output format, but that seems to have been removed. Why? Give it back please.

Give an edit mode to define the output format for each please.

@makedir: No this feature is never implemented yet. But community pull requests are welcome. Thank you.

Oh. That other post was from 2000 and even showed an example as a gif, thought that was the previous state.


Bug in error message from snapshot listener

Query Listeners in the combination of whereGreaterThan and whereNotEqualTo filters is throwing an exception as expected but with a small bug in the error message where it says com.google.cloud.firestore.FirestoreException: Backend ended Listen stream: Cannot have inequality filters on multiple properties: name with only mentioning one field property whereas with other combinations the message was correct mentioning exactly what all the multiple properties were.

Hi @shiva-6  , thank you so much for reporting this issue. I will look into this and keep you updated.

Query Listeners in the combination of whereGreaterThan and whereNotEqualTo filters is throwing an exception..
... whereas with other combinations the message was correct mentioning exactly what all the multiple properties were.

hi @shiva-6 , I was able to reproduce the issue on snapshot listener. It is only listing the second inequality property as you mentioned, but this issue is not limited to the "whereGreaterThan" and "whereNotEqualTo" combo. https://github.com/googleapis/java-firestore/pull/1093/files
While we look into this, could you please help try other combination and double check the error messages? It would be super appreciated if you could provide us with your reproducible code and/or screenshot of the error messages with different inequality combinations.

Another Query Listeners combination with an improper error message
db.collection("restaurants_p/6gaKC7syRE1qUGOD0cZZ/reviews").whereGreaterThan("rating",4.4).whereLessThan("author","ramesh");
Error message: Backend ended Listen stream: Cannot have inequality filters on multiple properties: author

Hi @partapu, thank you for sharing this. Seems like this is not only limited to specific combinations.


First pass at unprocessed batch items with recursive backoff

I haven't reviewed this with John yet, I was hoping to do that next week assuming he has time.
I was initially hoping to have the BatchWriteItem case class take a schedule instead of just a number of retries and the time to wait. I had some problems with using the schedule however, primarily that the schedule helper on ZIO returns the schedule's value rather than the ZIO effect's return value.
Another option I was thinking on was using streams and having a sink that has the unprocessed items as "leftovers".

I'm still going to wait on a review with John to see if he has other suggestions before marking this PR as ready to merge/review; my guess is he will/


lighbulb.nvim doesn't work with custom statuscolumn

Could anyone share a working config of lightbulb integration working with default sign column. It works for virtual_text setup but doesn't work for default sign column setup.
my config https://github.com/kapral18/config.nvim/blob/ffdb572892a01f74b7328cc8279008136258baf0/lua/plugins/improve-code-actions.lua
Originally posted by @kapral18 in https://github.com/LazyVim/LazyVim/discussions/1562

While debugging, I don't see any defined signs from lightbulb. It seems they are defined using extmarks instead.
https://github.com/kosayoda/nvim-lightbulb/blob/8f00b89dd1b1dbde16872bee5fbcee2e58c9b8e9/lua/nvim-lightbulb/init.lua#L127

Should be fixed now. Check on LazyVim dev or wait for the new release (tomorrow).
Be aware that only the highest priority sign will be shown in the left column.
The default sign priority for lighbulb is 10, while warnings (12) and errors are 13.
So on lines with diagnostics, the lightbulb will not be shown in favor of the diagnistics.
lightbulb.nvim has a priority setting that you can change to always show the lighbulb if available.


Invalid status if apibara fails to acquire persistence lock

If you submit an invalid etcd url, apibara fails to acquire the ertcd lock for 30 seconds and then fails. I beleive the exit code in this case isn't something that reflects an error because we get the ExitStatus as success and the node doesn't move to FailedRunning state.
For reference, I got this when etcd was behind the VPC and ecs couldn't access it

Apibara plans to add this feature this week.


Fixed broken link in README

The link http://gogs.io/docs/intro/ produces a 404 error and is used in the README updated it to http://gogs.io/docs/intro which correctly links to the introductory documentation.
I opened issue #1855

Thanks, but can you re-PR to develop branch?


Bugfixes + extra validation + tenant support on api calls

Closes #34 + Added extra validation on client id if you use client_credentials grant

What does the tenant support on api calls do exactly? Can you please provide a sample where this is used?

If you have an OAuth2 application that does api calls for the user instead of the user itself.
For example if you have an OAuth2 application setup with the client_credentials grant you request oauth tokens like this:
`POST /2337dcc6-3a21-4d95..../oauth2/token HTTP/1.1
Host: login.microsoftonline.com
Cache-Control: no-cache
Content-Type: multipart/form-data; boundary=----WebKitFormBoundary7MA4YWxkTrZu0gW
------WebKitFormBoundary7MA4YWxkTrZu0gW
Content-Disposition: form-data; name="client_id"

------WebKitFormBoundary7MA4YWxkTrZu0gW
Content-Disposition: form-data; name="scope"
https://graph.windows.net/2337dcc6-3a21-4d95..../.default
------WebKitFormBoundary7MA4YWxkTrZu0gW
Content-Disposition: form-data; name="client_secret"

------WebKitFormBoundary7MA4YWxkTrZu0gW
Content-Disposition: form-data; name="grant_type"
client_credentials
------WebKitFormBoundary7MA4YWxkTrZu0gW--`
And you get back:
{     "token_type": "Bearer",     "expires_in": "3599",     "ext_expires_in": "0",     "expires_on": "1512658264",     "not_before": "1512654364",     "resource": "00000002-<truncated>",     "access_token": "eyJ0eXAiOiJK<truncated>" }
If then you want to do an api call to get for example all the users you need to include the tenant in the uri:
https://graph.windows.net/2337dcc6-3a21-4d95..../users?api-version=1.6
If you don't include the tenant information you'll get the following response:
`
https://graph.windows.net/users?api-version=1.6
{
"odata.error": {
"code": "Request_InvalidRequestUrl",
"message": {
"lang": "en",
"value": "Request url was invalid. The request should be like /tenantdomainname/Entity or /$metadata. Tenant domain name can be any of the verified, unverified domain names or context id."
}
}
}`
I'll change the commit to remove the 'common' and only add the tenant if it's set as an option if that's ok for you. If you need more info feel free to ask or decline PR.

I've updated the file to make sure it stays as it is by default and only add the tenant if it's set as an option on the provider.

Let's do it a bit different then: Here's a detection whether the nextLink is URL - https://github.com/NoScopie/oauth2-azure/blob/b031aac960bc334debaac8141b5fe03188cb889e/src/Provider/Azure.php#L181 I would defintiely start there. So if the nextLink is not URL, check if it is meant for Azure Graph, then do the magic with tenant. Also, I am not sure if common is going to work with Azure Graph, the common equivalent there is myorganization. Could you please check on that?

Hi, sorry for the late reply. Atm I don't have time to look into this further due to a large project I'm working on. I'll try to get back to this asap.

I am not going to merge this PR, instead I reworked the changes in https://github.com/TheNetworg/oauth2-azure/commit/3163106aaa5edd0e537d1c86f00e80c13f040cf1 If this is alright with you @NoScopie, I can release this as a new version.


build: changed publish action to be triggered when release is published

This PR updates the publish action in our GitHub Actions workflow to be triggered when a new release is published. This change ensures that the package is published to PyPI only when a new release is created, providing better control over our release process and improving the overall project maintainability.

Codecov Report
Patch and project coverage have no change.

Comparison is base (a5a31c1) 99.40% compared to head (38efe71) 99.40%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   99.40%   99.40%
=======================================
  Files           3        3
  Lines         167      167
  Branches       45       45
=======================================
  Hits          166      166
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Ogit : Project Attribute required in ogit/itsm/Ticket.

I think in ogit/ITSM/Ticket we need an optional attribute ogit/Project.Here we should save the Project id for which the issue belongs to. As shown in the attachment, a company can have many projects. Each project has a number of issues. These issues are mapped to tickets in Graphit. For reverse mapping (From Graphit to the issue ticketing system) we we need to scan through all the issues to find the right project. Eg. "ticket 5" in Graphit has no information about the Project. Currently, to find that ticket 5 belongs to Project 3, we need to scan through all the issues.
This attribute is also necessary since in many (if not all) issue-ticketing systems we have the element of "Product", "Project".

@ipeca optional attribute ogit/project will be added to entity ogit/ITSM/Ticket when executing the changes on the OGIT cleanup, till then please use it as free attribute (Note that Ticket already allows this - see last commit - #9e4a0826ff33ae4cc7c16e20f70f5ce55e4c619a

Hi Iulian, please close the ticket if no further action is needed, as the needed attribute was already defined in the new incoming changes and you can use it as free attribute meanwhile.
Tks.

Closed.


Add support for Deberta models

Also add example for zero-shot-classification

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


in builder/test/main.c the main() **argv

Change to **argv or *argv[] . This was flagged as an error by clang.

Thank you for your comment, I've updated libraries to solve it!


Sign out should sign me out and not remember me if I don't explicitly ask to be remembered

Steps:

Sign in to Azure
Do NOT choose "keep me signed in"
Sign out
Sign in again: my credentials are remembered and I'm automatically signed in, which I don't want.

The "keep me signed in" checkbox is ignored and assumed always true perhaps?

Thank for opening this discussion.
The purpose of remember me is not to control whether or not the user is prompted for credentials the login tool. Its purpose is to prevent the user from having to login again if the emulator is closed and reopened at a later time. To implement what you describe, we would need two checkboxes, one to remember credentials in the sign in tool and another to remember the user's azure login at the emulator. The two are mutually exclusive and we currently implement only the latter.

The current state doesn't feel right because if I sign in with my work account by mistake, then I sign out with the intention of signing in with my personal account, I cannot - the emulator simply chooses to sign me in to an account that i did not ask to be remembered and I don't want to sign in to.
I got frustrated because of this and I imagined our users might feel equally frustrated. Whether it meets the bar to get it fixed or not, I'm not sure! Lets dicuss in standup :)

If we leave the checkboxes as they are, but sign out reprompts for which account you want to sign in, I would be fine with that. That is a correct behavior, and seems easy enough to fix.

Yes, setting the prompt=consent seems to work. I can implement this

@justinwilaby Sure! However when I find these things, I can help fix them too, no need for you to jump in, we can help :) I file it as issue so we get to discuss what the right thing is 👍

Fixed with #977

Reopening since our fix for this unfortunately was not what we need. We need to figure out what to do (probably will take a bit) and do it (probably a few lines of code, jsut hard to know which!).

after we make ours a first party app, we may be able to have consent work and re-apply


Add legend label style option

This feature was proposed several times in the past (#4163, #4496, #4811 and #4890) in order to use a line or a custom-sized box as a legend for a line in line charts, but no PR has been merged yet. I would like to try a bit different approach.

Deprecate the usePointStyle legend label option
Instead, introduce the style legend label option, which can have 'box', 'line' and 'point' value

'box': The same appearance as the current implementation
'line': The line style is used. Border width, border color, line cap style, line join style and line dashes are inherited from the corresponding dataset
'point': The same appearance as the current usePointStyle option.
If not set, the 'line' style is used for line elements, and the 'box' style for other elements.

As it detects the dataset type and choose a suitable legend label style, mixed charts are also supported.

See https://jsfiddle.net/nagix/d86rvwn5/

Note that the chart with style: 'point' shows a dashed circle, but this should not be a dashed line. I'm trying to fix this with #5621.
The existing tests are fixed more tests are added. Also, document is updated.
Fixes #4727

I'm not sure we should deprecate usePointStyle. IMO, this is 2 different features: labels.usePointStyle allows to pick the dataset point options (instead of the dataset line options) while labels.style allows to control the shape of the label.
I think the following use cases should be valid for a line or radar chart:

style: 'point' and usePointStyle: false: draw points with the line color/border/...
style: 'point' and usePointStyle: true: draw points with the point color/border/...
style: 'box' and usePointStyle: true: draw boxes with the point color/border/...
style: 'line' and usePointStyle: true: draw lines with the point color/border/...
...

@simonbrunel usePointStyle: true doesn't mean using the point color/border/..., but using pointStyle shapes such as 'circle' and 'triangle'. So, style: 'box' vs usePointStyle: true and style: 'point' vs usePointStyle: true are exclusive. style: 'line' and usePointStyle: true can be used together and useful, though.

I thought usePointStyle was also using the point color/border/... instead of the line ones.

Note that the chart with style: 'point' shows a dashed circle, but this should not be a dashed line. I'm trying to fix this with #5621.

I think it should be a dashed line in this case and I would not do special cases based on the chart type. If we want the labels to use the point color/border/ ... instead of the line options, then we should introduce a new option (if usePointStyle is not this one).

In the current implementation, usePointStyle: true doesn't use point color/border/... but use line color/border/..., and that causes inconsistency in appearance between legend and chart elements when they have different styles. But, this is for other PR.
In this PR, I'm just focusing on "shapes". usePointStyle only switches between a box and point shape, but this proposal is trying to give more options including a line.

Really appreciate this.
Only I would expect:
 datasets: [{
      type: 'line',

to be:
 datasets: [{
      labelType: 'line', //box, line, circle

And that the labelType property in the dataset, overrides whatever is set as default at options.legend.labels.style.

@nagix I totally get that this PR is not about color/border/etc. but #5621 uses usePointStyle to switch between the dataset (line) and element (point) colors/border/etc. I don't think style: 'point' should also change the color/border/etc.: the shape and the color/border/etc. should be independent IMO.
I don't really like the term style because it's too confusing. We don't know if we are talking about the shape, the colors, the opacity, ... or everything. We may prefer to call this new option: labels.shape or labels.symbol instead of labels.style.
So I would rather keep usePointStyle to make the legend label match the point style (shape/color/border/etc.). labels.shape (if defined) would override the point 'symbol' if usePointStyle: true while using the point color/border/etc.
Finally, labels.shape: 'point' would mean: use the current point shape (not the other styling options). Ideally, we should support all other shapes, especially circle since it's a wanted feature for any type of charts.

So, style: 'box' and usePointStyle: true are exclusive

It's not exclusive, it allows to use the point color/border/etc. while displaying a box, which I'm sure is a valid use case. The other way is also valid: shape: 'point', usePointStyle: false, meaning I want to use the line color/border/etc. while displaying the current point shape.
What do you guys think? (sorry for the long comment)

I agree that we keep usePointStyle to make the legend label symbol match the point style while we introduce symbol to control the type of the symbol. As the value 'point' doesn't represent the shape, I'd prefer the term symbol rather than shape. As @simonbrunel said, style is definitely confusing.

labels.shape (if defined) would override the point 'symbol' if usePointStyle: true while using the point color/border/etc.

I don't see the necessity of the box or line symbol in point style, so I think symbol doesn't need to override the point symbol. But, the point symbol on a line symbol is quite useful. So, I propose this:

Any comments?

As the value 'point' doesn't represent the shape, I'd prefer the term symbol rather than shape

I would call the point shape (triangle, circle, etc.) a symbol (per #4811)

I don't see the necessity of the box or line symbol in the point style

I still think it's a valid use case, why enforcing such restriction?
--
Actually, #4811 is closer to what I'm thinking about customizing the legend labels: allow the user to pick any available symbol as legend labels (whatever the usePointStyle value). I'm not fan of complex / rigid option logic and prefer to keep things simple and flexible. At some point, someone will ask for circle or triangle in a bar chart.
So I think I prefer the new option to select / override the label symbol (any of this list) while usePointStyle switches between dataset/element style (I would maybe not support point since it doesn't make sense in all charts).

Ok, in that case, I can wait for #4811.

@nagix I'm not completely understanding the conclusion that you and Simon came to. Is this PR a duplicate of https://github.com/chartjs/Chart.js/pull/4811 and should be closed? Or only partially a duplicate and still adds some new functionality in which case it should be updated to add only the new functionality?
I'm hoping we can either update or close the PR. I'd like to make sure all the open PRs are in a reviewable state. Otherwise it gets really hard to keep track of which we need to review and which we shouldn't

@nagix should this PR be updated or closed?

@nagix I'm going to close this PR as inactive since there hasn't been any response and it's not clear to me from the comments that it's still needed. Please feel free to reopen if I'm wrong about that


Add ansi and bgAnsi to TypeScript declarations

Here are remaining methods, after I researched the meaning of the argument:

Add 2 type tests to index.test-d.ts which failed first
Add 2 declarations to index.d.ts
Add link and example to readme.md

Thank you very much for quick turn-around on #368 ❤

What do y’all think about removing these 2 methods from chalk public interface?
Through research and discussion in #370 I see internal use of ansi to convert down to lower-level format, but it seems like SGR color code as argument is a limited use case for chalk callers:

first I jumped to assumption that argument is 4-bit unsigned color index, like 1 for red
then I jumped to assumption that at level 1 the argument could be any SGR code, like 1 for bold, but the close code does not match: default color not normal weight, or whatever else

@pedrottimark Can you move the above into a new issue? The Chalk v3 release is imminent, but we can discuss whether to remove them for the next major version.


Bug: the arrow moves after the popover is visible

The initial position of the arrow is correct, but it moves a little up soon. Please fix. Thanks!

can you please record a screencast of this? Firt time I hear about this

@escamoteur
Actually, I found that this happens only on flutter's dev channel. After switching back to the stable channel, it works. so this is a bug of flutter. Closing


What path to put emacsclient at?

Hi @elizagamedev thanks for your work on this project. What path do you have emacsclient at. I suspect my code.app shim or Unity itself doesn't have access as they may be in a different path or env than where my emacsclient is

On macOS unity is probably not inheriting your bash PATH when running the shim. The simplest solution would be to change emacsclient to an absolute path.

@elizagamedev that was indeed the issue and it works as expected now. I also suspect this was maybe what the user at #2 was seeing. I opened a PR to add a note to the readme. #8 Thanks for the help!


Added Slider

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #308

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system:
Version:
Text-editors used:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS

https://rohank2502.github.io/SFC-foundations/    @himanshu007-creator  Check the slider have added 3 dummy events will create template for that event in separate issue !! Please let me know any working issue if any ! Just there is slight delay of 2-3 sec after completing the slideshow to restart it I will create one issue and make it don't for now I don't have any solution but I will look few ideas to manage this and will solve that issue but overall the slider works well !!

hi @rohank2502 , there have been changes in design. I will share you the design here. Wait

Coloring/border shadowing is up to you but one suggestion, use gray because it will be visible in both themes. And make this whole container somewhat transparent (~ 40%)
When can expect the changes?

Hey, @himanshu007-creator  For Now I have created the new slider event page and added in button but till 10-11 am tomorrow I will make the same page redesign as you proposed !!

@himanshu007-creator  Is slider ok ?

rest everything is good, just fix position of live button and remove the hover effect. It should simply be blinking live, on hover there should me no changes

rest everything is good, just fix position of live button and remove the hover effect. It should simply be blinking live, on hover there should me no changes

OKOK got it will do that


Update windows-virtual-desktop.md

Changed the sentence on Azure subscription limit to recommendations - as you could fill up more VMs if necessary. No real limit so word is incorrect.

update Github link.

@christiaanbrinkhoff : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ckittel: I'm sorry - only the author of this article, @doodlemania2, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


First draft of manual page for cht

Not intended for commit right now, but suggestions (especially on TODO items) are welcome.

I should note that on OpenBSD I found no programs naming "cht" neither in base system or in Ports. Thus I think "cht.sh" could be renamed to "cht" (actually I'm using it this way all the time :) ).

Actually, the idea behind the cht.sh client is that its called exactly as the server itself.
The cheat.sh project was developed as a part of a bigger project, where console services at the end of the day can be called directly as programs (we are working on that in curl; it is not yet fully implemented).
That will mean that any console service could be called directly by its name, and its arguments will be rewrite just in time (by curl) into the query string.
To use curl in this mode, it should be called with the name of the service (which should be a hardlink of a symlink to curl).
Say:
ln -s /usr/bin/curl /usr/bin/cht.sh
ln -s /usr/bin/curl /usr/bin/wttr.in
ln -s /usr/bin/curl /usr/bin/rate.sx
cht.sh python copy file
wttr.in Moscow
rate.sx btc

Of course, that's true, that cht.sh is currently a standalone script, but we are trying to emulate the look and feel of this future service-as-a-command thing.

Hm, okay. I really need to get the whole picture in my head first then.

@grayed Vadim, your man is a great thing anyway, the only change that we should do here is rename it to cht.sh.1 and replace cht with cht.sh. Please reopen the pull request


[BEAM-4005] Futurize tools subpackage

This pull request prepares the tools subpackage for Python 3 support. This PR is part of a series in which all subpackages will be updated using the same approach.
This approach has been documented here and the first pull request in the series (Futurize coders subpackage) demonstrating this approach can be found at #5053.
R: @aaltay @tvalentyn

run python precommit

LGTM. Thank you.


interactive_markers: 1.11.3-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository interactive_markers to 1.11.3-0:

upstream repository: https://github.com/ros-visualization/interactive_markers.git
release repository: https://github.com/ros-gbp/interactive_markers-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.22
previous version for package: 1.11.1-0

interactive_markers
* The ``processFeedback`` function of the menu handler no longer catches the ``KeyErrors`` of the feedback_cb.
  See: #29 <https://github.com/ros-visualization/interactive_markers/issues/29>
* Added the ``empty()`` and ``size()`` members to ``InteractiveMarkerServer`` interface.
  See: #30 <https://github.com/ros-visualization/interactive_markers/issues/30>
* Contributors: Blake Anderson, Guglielmo Gemignani

Hold this merge.


Allow #DEFINE like @orlof uses

This change is

To actually handle the syntax @orlof is using, we also need to make the comma in defines optional:
#DEFINE THING 1234


Closes #1484: Add a little padding on top of the add photo message

Description
Added padding to the cardView and textView in the card in fragment_offline_summary_product.xml
Related issues and discussion
#1484
Screen-shots, if any

@N00845855  Lets wait for reviews from other members as this a design related issue. In the meantime i am fixing your travis build.

Merged, thanks @N00845855


amplify add codegen not generate API in AppSync

amplify add codegen  not asking API file name  it asking in ios API.swift but in android API not generated.
when try to configure amplify codgen command
Error  "No AppSync API configured. Please add an API"
Any one have solution please help .
Log after  amplify add codegen --apiId XXXXXX
Getting API details
Successfully added API AppSyncDemo to your Amplify project
? Enter the file name pattern of graphql queries, mutations and subscriptions app/src/main/graphql/**/*.graphql
? Do you want to generate/update all possible GraphQL operations - queries, mutations and subscriptions Yes
? Enter maximum statement depth [increase from default if your schema is deeply nested] 2
✔ Downloaded the schema
✔ Generated GraphQL operations successfully and saved at app/src/main/graphql/com/amazonaws/amplify/generated/graphql

Are you looking at generating types from a API built on the AppSync console?
Below are some guides in android and iOS.
https://docs.amplify.aws/sdk/api/graphql/q/platform/android
https://docs.amplify.aws/sdk/api/graphql/q/platform/ios

I have tried same steps ,
I can not pull from appsynch backend.
$ sudo amplify pull
Pre-pull status:
Current Environment: dev

Category
Resource name
Operation
Provider plugin

Api
AppSyncDemo
No Change

✔ Successfully pulled backend environment dev from the cloud.
✖ There was an error initializing your environment.
ENOENT: no such file or directory, open '/home/kaushal/AndroidStudioProjects/AppsyncDemo2/amplify/backend/backend-config.json'
Error: ENOENT: no such file or directory, open '/home/kaushal/AndroidStudioProjects/AppsyncDemo2/amplify/backend/backend-config.json'
at Object.openSync (fs.js:465:3)
at Object.readFileSync (fs.js:368:35)
at AmplifyToolkit.readJsonFile [as _readJsonFile] (/usr/local/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/read-json-file.js:14:33)
at initEnv (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-api/src/index.ts:61:33)
at /usr/local/lib/node_modules/@aws-amplify/cli/src/initialize-env.js:43:52
at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/promise-sequential/index.js:16:18
at processTicksAndRejections (internal/process/task_queues.js:93:5)
I tried creating file also but still getting error.
after creating file it can not pull anything it showing nothing in pull data.just blank pull.
I also tried creating 3-4 new android project and try to create with Integrate with your app steps
if i create api with add command and push data to appsync it working but i want to use existing appsync data from server and generate api in android project that is not working.

I have learned that while you use the amplify CLI to add the existing API, you cannot use the Amplify SDK or gradle plugins to handle it: https://github.com/aws-amplify/amplify-cli/issues/5069#issuecomment-673683606 . You need to use the classic com.amazonaws.appsync gradle plugin to generate the types on build and the classic aws-android-sdk-appsync library to make the AppSync calls.

@RandomEngy thanks for jumpin in and answering. Closing this issue.

@RandomEngy thanks for jumpin in and answering. Closing this issue.


In socket mode, python outputs come delayed and not immediate

When i execute the below python program in socket mode (via /connect)
from time import sleep

sleep(3)
print('1')

sleep(3)
print('2')

sleep(3)
print('3')

The output all comes at once, rather than at 3 sec intervals

Looks like adding this env variable PYTHONUNBUFFERED = 1  makes stdout/stdin streams to be unbuffered. Reference
Adding -u flag to request doesn't work as we need -u <file-name> but in the api app the args are set the other way <file-name ...flags> Reference
Possible to add PYTHONUNBUFFERED = 1 to the package env file of python?
https://github.com/engineer-man/piston/blob/master/packages/python/3.12.0/environment

For now, you can use print('1', flush=True). I think this should cause the output to appear immediately.

While what you mentioned works, it would be better if it was added as part of the env file so the user won't have be informed to set flush while executing code.
For now I have forked the repo and added a post build script that installs python package and updates the env file.


Add underscore prefix to the template that not extend base one

It seems we add "_" prefix to the name of all templates that not extend the base one.

The _ is added to any template fragment which is included in other templates. So I'm :+1: to this change.

Yes, it make sense. Ryan use it in his screencasts too

Thanks for all your work on this project!

I love Symfony! :)


Add an inner class to the Card Component

Sometimes using the bootstrap grid system is diffecult without an inner div element holding all the internal card structure (card-top, card-content, card-bottom). For example to surround the Chart with panel like border.

This Pull Request wraps the Card's element inside a div with the card-inner class and add support for a new Card setting option named "cardInnerClasses" to append classes to the div.

@rhabbachi  Very sorry I haven't merged this! I'm not sure how this effects our production builds so I'm being very cautious...

@rhabbachi is this still needed?


Add oleg_nenashev to release-plugin

I'm not a maintainer, but I want to release the version with SECURITY-170 fix (https://github.com/jenkinsci/release-plugin/pull/17) and several other pretty important fixes. Using my Jenkins Core and Security team membership as a justification

CC @daniel-beck @batmat @rtyler @avalanche123 @petehayes @fredg02

If somebody has concerns, I can take the ownership of the plugin

@oleg-nenashev Have you tried to contact the maintainer via email? Or did they publicly abandon the plugin?

I've sent the email today. I assume that it has been previously sent by @amuniz. At least we were discussing it at some point IIRC

Let's wait a bit for a response.

Did you send a public email on the dev ML @oleg-nenashev, with the maintainer in copy @oleg-nenashev ? Would seem sensible IMO.

Yes, you have to go through the developer mailing list to get added to the repositories. I personally don't have any reservations and would be happy if you could perform maintenance/release of this plugin

@avalanche123 Thanks a lot!


Recognizing files from the desktop and displaying them in checkboxes

Create a page that takes the directory path and displays all the files in that directory as checkboxes
user selects the files and clicks submit and , the selected files list would be stored in a variable and displayed when clicked submit

Check boxes added and the files selected were also being displayed

Check boxes updated and displayed, issue resolved


Let retry policy to decide about non idempotent queries

This PR:

Makes Conn.exec to include a flag into error that signals that query could have been actually executed, despite error been recieved.
Picks up this error in queryExecutor.do and if query is not idempotent wrap it with user-facing error that would signal to retry policy and to user that this non idempotent query was failed but could be actually executed.

Fixes: #326, #331

@dkropachev ping

@dkropachev I applied changes suggested by you and split everything into separate commits

@sylwiaszunejko , I have found one more issue and fixed it in the code, pushing one more commit, please review it.

@sylwiaszunejko , I have found one more issue and fixed it in the code, pushing one more commit, please review it.

I incorporated this change into commits, but in this case I think simple comment to consider changing return type to RetryType instead of bool would make perfect sense and I could actually implement that myself and save you some time

@sylwiaszunejko , I have found one more issue and fixed it in the code, pushing one more commit, please review it.

I incorporated this change into commits, but in this case I think simple comment to consider changing return type to RetryType instead of bool would make perfect sense and I could actually implement that myself and save you some time

Having this change in a separate commit makes it confusing because reaction on say, context.Canceled and on ErrHostDown are not preserved in between.

@dkropachev does it look better now? Or did I miss some parts when trying to separate the changes into small separate commits?


exclude usage examples which don't match our author/data file

Lookup for audi then go to usage word examples. the first line does not have the work and author's name, only the line number. it should be Pseudo-Varro, Sententiae + line number.
obviously it cannot be searched by author (and work).

QA-<IP_ADDRESS>

pseudo-varro is gone (tested that Varro, Marcus Terentius is still in the authors list)


Problem reaching Go app

I am trying to deploy a Go application that uses Echo. Here is the repo for reference - https://github.com/hasit/healthyrepo.
This is my boxfile.yml:
run.config:
  engine: golang
  engine.config:
    runtime: go-1.8
    package: 'github.com/hasit/healthyrepo'
    fetch: 'go get'
    build: 'go build'

web.api:
  start: healthyrepo

I see my component on the dashboard as shown in the screenshot below:

When I try to access it, I get the follow log message repeatedly:
2017/05/25 05:48:54 http: proxy error: dial tcp <IP_ADDRESS>:8080: getsockopt: connection refused

What am I doing wrong?

At this point I would try to verify a couple of things:
1 - Verify the app is actually running in the log viewer (dashboard > app > logs)
2 - Ensure the app is bound to 8080 on ALL interfaces, not just <IP_ADDRESS>
If you need to help with either of these, feel free to hop in the slack channel and we can walk through it with you.

@tylerflint
My DB cluster had to allow connections from nanobox IP.
I will close this issue now since it has been resolved.


highlighted important text in the doc

highlighted important steps in the doc
Description of the change:
Reason for the change:
Link to original source:

Closes #

I appreciate what you were trying to do here, however, according to our style guide, we only highlight actual UI elements or text that the user will be clicking on. Please check out our open issues for changes that we’re looking for in this repo! We’re adding new ones from easy to difficult every day.


Don't throw for XMLPullParserException

On some devices, this would happen
https://android.googlesource.com/platform/frameworks/base/+/master/services/core/java/com/android/server/fingerprint/FingerprintsUserState.java#222
and there is not much we can do here so no throw and return false;

Stack trace says this is wrong, see #36


Rename [host] to [@metadata][ip_address]

This aligns the field name with logstash-input-tcp.
Closes #40.

I decided not to implement DNS resolving into the [host] field as it is likely to have a big performance impact without additional caching.

Closing in favour of #43


Home page card does not trigger animation

The .buildlearngrow-card does not trigger a pretty animation like the links in the navbar. I don't know how big of a deal this is, just wanted to let you know.

Yes was gonna take care of it. Should be a quick fix thanks

Commit in registration PR takes care of this - will auto-close when registration is merged


Bump mimemagic 0.3.9

What
Bump mimemagic 0.3.9
Why
see https://www.theregister.com/2021/03/25/ruby_rails_code/
Previous versions to 0.3.6 yanked
plus 0.3.9 seems to just work.
We do not actually use it anyway, but is a rails
dependency that we are currently pulling in,
even though it should be possible not to by removing
require 'rails' from config/application.rb

Closing as superceded by https://github.com/ministryofjustice/laa-court-data-ui/pull/522


[rapids][ml] Fix RAPIDS and MLVM init actions on Dataproc 2.0

Fixes after Anaconda component removal from 2.0 images.

/gcbrun


config: node with a single config file

This renamed the Endpoint structs, to make it clear they are not reusable
Added a endpoint method to render the corresponding Endpoint url. (an URL is needed because of the protocol)
Added a top-level config for each server. This gives a namespace for the individual serve types, allowing them to be in the same file. - Configs are under store, block_producer, and rpc namespaces.
Added tests for the above.
Now ENV variables are separated by __ instead of a _. This allow configuration names with _ to be set via environment variables.

fixes #100

Not sure how much complexity this adds, but I think it would be cleaner from the UX perspective.

I think this is for sure true for node config, but not so sure about component configs (they may actually be a bit less clear because there is not clear identification of the main component).

The above info should be sufficient and we don't need to duplicate store_endpoint settings under block_producer and rpc etc..

Those are not the same things. A server socket is open against an interface and a port, the interface is represented by a ip address, and <IP_ADDRESS> is a valid value to represent all interfaces. This value can not be used to open a connection to.
A client definition is different in multiple ways:

it uses protobuf, which runs on top of a protocol like http and https, so this bit of information is necessary on top of the (ip, port) pair.
the client peforms DNS resolution before opening the connection. This allows the client to refer to another node by name
DNS resolution resolves to the public ip, this is NOT necessarily the same the server will have bound to, the server can be behind a firewall, network group, reverse proxy, etc.

IMO the settings should not be collapsed.

The endpoint setting in the server and the client are not the same. They should not be collapsed.

Ah - makes sense. I think what confused me is that all of them are endpoints. Maybe there is a way to make them more distinct?

Ah - makes sense. I think what confused me is that all of them are endpoints. Maybe there is a way to make them more distinct?

For example, maybe something like this:
[block_producer]
endpoint = { host = "<IP_ADDRESS>" port = 8080 }
store_url = "http://store:8000"

[rpc]
endpoint = { host = "<IP_ADDRESS>" port = 8080 }
store_url = "http://store:8000"
block_producer_url = "http://block_producer:8001"

[store]
endpoint = { host = "<IP_ADDRESS>" port = 8080 }
sqlite = "local.sqlite3"

Ah - makes sense. I think what confused me is that all of them are endpoints. Maybe there is a way to make them more distinct?

I have renamed it to IpPort already


Travis CI: Tests fail on Node.js >= v12

https://travis-ci.org/github/dlmanning/gulp-sass/pull_requests
https://nodejs.org/api/fs.html

Thanks you but we should not fail on these node versions so instead we need to fix the root cause of the failures.

We need to fix the root cause of the failures.

Once that is done, the last 5 lines could be deleted.

Apologies I thought there were two separate PRs. Lets remove the allowed failures and I'll look into the build failures.

I can't reopen the PR because the branch has been deleted

Recreated in #790


Change FindPythonLibs to ask the interpreter for its config (master).

This askes the python interpreter what the location of libs and headers is instead
of relying on finding the libraries/headers on the file system.

meant to open this in my fork. sorry for the noise.