[WIP] [frameit] Ensure both texts use same baseline

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
Output:

Finished in 6 minutes 46 seconds (files took 3.7 seconds to load)
4675 examples, 0 failures

[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
Output:

973 files inspected, no offenses detected

[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
Not applicable

Motivation and Context

This solves issue https://github.com/fastlane/fastlane/issues/10440

The current implementation applies a .trim to both images for Keyword and Title.
Due to typography differences (for example the "L" that needs more vertical space than an "e") the generic baseline between 2 texts is lost. This can be seen in the excellent analysis of @janpio in this comment: https://github.com/fastlane/fastlane/issues/10440#issuecomment-348010988
Before

This solution requests the calculated trim box and offset without applying it.
By comparing the vertical top offsets of both trim boxes, the vertical offset between them can be calculated. The box with the biggest offset will be compensated to equal the same of the other, so their text baseline will be equal.
Finally a .crop will be applied with the (compensated) trim boxes.
After

Test results can be found in issue comment: https://github.com/fastlane/fastlane/issues/10440#issuecomment-348760923
Description

The changes can be found in issue comment: https://github.com/fastlane/fastlane/issues/10440#issuecomment-348760717
Please note that:

I've no Ruby skills, so this might not be proper Production code. But the description and the code show the intended behaviour.
All print commands are for debugging, so can be removed.
Hence @janpio offered to help, so I added [WIP] to the title of the PR.
The current code aligns the trimmed/cropped images to the top. To make this fix robust for any future changes (for example align to bottom), the height of both trim boxes (parameter 2) should be made equal: the largest after the current vertical top offset compensations is leading.

Oh, and tests are failing because of this:
MiniMagick::Error:
       ImageMagick/GraphicsMagick is not installed

Did you manually add any dependencies to your project you didn't add to the PR?
If not, any idea why this might be missing or what is using it?

I have no idea about the MiniMagick error.
I didn't add anything, it's already used in this file.
The only new call I make is in row 359:
calculated_trim_box = title_image.identify do |b|

How do you want me to respond to your code review?
Should I reply here, and if we both agree I make the changes in my local copy and push them to the PR?

In order to create a robust solution, I'm eager to address my 4th note:

The current code aligns the trimmed/cropped images to the top. To make this fix robust for any future changes (for example align to bottom), the height of both trim boxes (parameter 2) should be made equal: the largest after the current vertical top offset compensations is leading.

What do you think?

If you agree, just make the code changes and push them to your fork, they will be added automatically and the comment will get marked as "outdated".
If you don't agree, just respond under my comment and we can talk it out ;)

I have no idea about the MiniMagick error.
I didn't add anything, it's already used in this file.

Hmmm....

What do you think?

To be honest that sentence is to complicated for me.

This is my current solution explained:

The 2 green boxes are the calculated trim boxes. It immediately shows the existing issue: when vertically aligning the 2 trim boxes to their top (which the rest of the code does), the baseline is lost and the "e" in "eye" hovers above the "o" in mouth.
To prevent this I have extended the lowest trim box ("eye"), so their tops have the same common min_vertical_trim_offset.
But what happens when the code is changed to vertically align the 2 trim boxes to the bottom?
Then we have the issue back, only then title will be lower than the keyword.
So basically the highest trim box should be extended to match the bottom of the lowest trim box, here shown by common max_vertical_trim_offset:

Now both trim boxes have the exact same height and still share their original baseline.
No matter how their are vertically aligned, the baseline will always be shared.
I hope this is more clear.

Ok, got that. Is there already an option to align to the bottom? (I saw something where the texts can be below the screenshots - is this it?) If not, you could probably ignore that here and just add a comment that this only workds for aligning to top.
PS: Make sure to sign the CLI and comment so the bot picks that up.

There is no option or user setting. But apparently somewhere in the rest of the file, the code uses align to top.
I don't believe this trimming/cropping has to do with text being cut off, as the complete text is always in the trimmed image. If text is missing in the screenshot, the reason must be that the text image itself is too large or misplaced at the screenshot canvas.
I'll make a comment in the code that my solution (as well as the previous .trim) only works for align to top.

While working on the comments I just enhanced the code to implement bottom align, because that's more clear than a fuzzy TODO comment.
Here the difference is clear, while highlighting the trim box in yellow:
Original code with .trim:

My current commit with .crop to align top:

My robust solution with .crop for both align top and bottom:

Now I get a rubocop error that the code is 1 line too long:
Offenses:

frameit/lib/frameit/editor.rb:2:3: C: Class has too many lines. [321/320]
  class Editor ...
  ^^^^^^^^^^^^

973 files inspected, 1 offense detected

I submit it anyway, because this is a ridiculous check.

I don't understand the whole Google CLA stuff, but I don't want to participate in that.
So here is my code solution, do whatever you want to do with it.

I don't understand the whole Google CLA stuff, but I don't want to participate in that.
So here is my code solution, do whatever you want to do with it.

That is unfortunately impossible as this is a Google Open Source project and only contributions with a signed CLA can be accepted.
It is pretty painless though, just fill out the form and reply here with "I signed it".
It's basically what you wrote above ("here is the code, take it") but in legalese.

Now I get a rubocop error that the code is 1 line too long:
...
I submit it anyway, because this is a ridiculous check.

Fortunately trivial to work around:
https://stackoverflow.com/questions/40221797/rails-rubocop-disable-class-has-too-many-lines-error

Here the difference is clear, while highlighting the trim box in yellow:

Wow, that perfectly visualizes not only why it was so shitty until now but also how you fixed it.
Did you add the yellow with code or manually? If with code, it could be useful to leave that line/code in there commented out for future debugging or development.

Yellow highlight
I visualised that in code by just setting the image background to yellow.
Temporary of course, so it's not committed. I think it's bad practise to leave commented debug code in Production code.
So for the record, I added these 2 lines to the existing do:
title_image.combine_options do |i|
  # Oversize as the text might be larger than the actual image. We're trimming afterwards anyway
  i.resize "#{max_width * 5.0}x#{image_height}!" # `!` says it should ignore the ratio
  i.background 'yellow'
  i.alpha 'remove'
end

The removal of alpha was key, see: https://github.com/minimagick/minimagick/issues/104#issuecomment-16856389
Rubocop warning
It seems bad practise to disable warnings. Isn't it possible to change the limit of this warning globally (i.e. to 1,000 or anything acceptable)?
Google CLA
To put it mildly, I'm not a fan of Google.
I have a non-Google email address as login with GitHub.
I'm not willing to create a Gmail and use that for GitHub.
So, it doesn't seem "painless".

It seems bad practise to disable warnings. Isn't it possible to change the limit of this warning globally (i.e. to 1,000 or anything acceptable)?

These are set to a value the fastlane developers deem acceptable.

I have a non-Google email address as login with GitHub.
I'm not willing to create a Gmail and use that for GitHub.
So, it doesn't seem "painless".

Nothing I can do about that, I am a volunteer as you and just relay the rules to you.

Well, I guess you can pick up my code from here, create a PR on your account and merge it to the master?

No, as I don't own the copyright on your code - I didn't write it. I would break #5 of my own CLA :) Sorry.
(I pointed your "problem" out to someone from the team)

I don't blame you, but what a terrible situation after a productive weekend.
Why should Google be involved in everything?
I contributed to several Githubs without this issue.

It is mentioned in https://github.com/fastlane/fastlane/blob/master/CONTRIBUTING.md#i-want-to-contribute-to-fastlane which I didn't link to, my bad. (But maybe a link to the CONTRIBUTING file should be added to the YourFirstPR one)

Why should Google be involved in everything?

Because fastlane is a Google Open Source project ;)
About why Google and big companies (with lots of lawyers) have CLAs:
https://opensource.google.com/docs/cla/#why
There will be a solution for this, just give it some time. For now you could make sure the tests succeed (even if it means adding a silly annotation to you don't like, it's how it works for fastlane).

Just sent a PR for the YourFirstPR change: https://github.com/fastlane/fastlane/pull/11103

Thanks for the PR to change the docu.
Regarding the CLA; lawyers did their best to write tons of words in the why link.
As they are not capable in providing the answer in one or 2 simple sentences, I stopped reading because I value my time.
But consider this simple use-case:
I will continue to perform system satisfaction by creating a Gmail account like<EMAIL_ADDRESS>I sign the CLA, add this Gmail address to Github.
Then my PR turns green. Everybody (system?) happy.
After you merge it in the master, I'll delete my Gmail account and remove it from Github.
What did this system satisfaction contribute to more than that I now just say: Here is my code, you can use it?
I don't expect you to answer it, but I hope it shows the ridiculousness.

I've added #11104 for the rubocop check.

@funnel20 thanks for your PR. This looks great.
For the CLA, Google asks for it to be signed on all of their open source projects. Hopefully @janpio's PR makes it clearer for first time contributors.
If you don't want to sign the the CLA, I can talk to our open source team about having someone else submit it on your behalf. The alternative would be for you to make an @gmail.com account (like the example you listed), and then add your GitHub email address as an alternative email on the account: https://support.google.com/accounts/answer/6316959?visit_id=1-636479951809319153-3505887489&p=emails&hl=en&rd=1#alternate and then sign the CLA. Then you don't need to make any changes to your GitHub email addresses.

@alexsinger Thanks for you response.
I'm sorry to be so stubborn, but I can't believe that everyone just does what Google asks. Without even thinking why it's necessary and they are trying to accomplish with it.
As my example shows, this whole CLA seems to be a meaningless perception, as anybody can sign up for a fake Gmail account.
So to save me all these meaningless procedures (and obey as a brainless zombie to Google) I just want to hand over my code solution and let someone else commit it. It even doesn't have to be on my behalf.
Volunteering is just about helping others. Not about all this legal BS.

@funnel20 I understand your frustration with the legal stuff. I'm working with our team to make sure we can get your changes in. Thanks again for the submission and I'm sorry about the legal hurdles.

@alexsinger Thanks for understanding.
Off topic I really like to understand why these legal hurdles are introduced in a community project. In my opinion the whole Open Source / Github community was an entire force of volunteering specialists who showed the big companies that they were not alone in making quality software.
That was one of many reasons that I admire Fastlane.
It's really bad they put an army of monitoring lawyers in our neck.

https://krausefx.com/blog/fastlane-is-now-part-of-fabric
https://blog.twitter.com/developer/en_us/topics/tools/2017/fabric-joins-google-developer-platform.html
https://fabric.io/blog/fabric-joins-google
https://krausefx.com/blog/fastlane-is-joining-google

@janpio If Fastlane is so proud to be part of Google, why isn't that shown anywhere?
There must be a reason why there isn't any Google badge or reference at:

The startpage https://fastlane.tools, only a tiny badge at the bottom for Fabric
The GitHub repo https://github.com/fastlane/fastlane
The Docs https://docs.fastlane.tools

@funnel20 the fastlane community and its contributors are important to us. We aim to be an open, welcoming community. Since fastlane is part of Google there a certain things we need to do, like have contributors sign the CLA. @janpio linked to it and I know it's long, but the reasons are in https://opensource.google.com/docs/cla/#why.
Submitting on your behalf has some barriers, but I'm seeing what we can do. If you'd reconsider signing the CLA so we can merge your change, it would be really helpful. Your frustration is understandable, considering the time you've put into this, and hopefully you'll consider signing the CLA so we can include your changes.
In terms of labeling Google in our docs and fastlane.tools page - we're rolling out an update soon to fastlane.tools that will make it more obvious.

@alexsinger Thanks for understanding.
As long as nobody reacts to my super simple use-case that pulverises the whole Google CLA I won't sign anything.
What is wrong with the GitHub CLA, which I signed for while signing up for my GitHub account with my non-Google email account?
It's super short, understandable for non-lawyers (like coders are) and covers the whole IP topic.
What's only acceptable for me is that I can register my non-Google GitHub email account at Google to sign the CLA. Instead of creating a Gmail account and being forced to use that as my primary account for GitHub.

@funnel20 I can't speak to the GitHub CLA, but the Google CLA is required and a Google account is needed to sign it. To clarify, the Google account doesn't need to be added to your GitHub account - you can include your non-Google account as alternative email https://support.google.com/accounts/answer/6316959?visit_id=1-636479951809319153-3505887489&p=emails&hl=en&rd=1#alternate and that will work.

What is wrong with the GitHub CLA, ...

It belongs to the company behind Github, and only applies to projects that "belong" to that company. It doesn't apply to any other projects that are only hosted on Github.

What's only acceptable for me is that I can register my non-Google GitHub email account at Google to sign the CLA. Instead of creating a Gmail account and being forced to use that as my primary account for GitHub.

As I understood it you don't have to create a Gmail account, but a Google account, which you can do with any email address (click "I prefer to use my current email address" under the email adresse field on account creation). You then use that one to sign the CLA (and add your current Github email address to the CLA if that one is different from the one used to create the Google account) so it can connect your code commits to the signed CLA under your normal email adress (that now also has a connected Google account).

Yes, @janpio is right - you don't need a @gmail.com email address, just a Google account, and a Google account can be created using your existing email address.

I signed it!

I signed it!

I have followed the instructions, but the bot doesn't accept it.
What have I done?

I logged in my existing Google account, which automatically created a Gmail address
It already has an alternative email address, which is the same as I use for Github.
When logged in with that google account, I signed the CLA. Of course it shows the primary Gmail address:

The last field is the Github name; so I added mine and hoped it would be OK.

As a second attempt, I have added my Gmail to my Github and verified it.
But still the bot fails.
Please advise

I think your commit does not actuall have an email address - or it is not connected to your Github account. So the CLA side is all good, but your commits or Github account are not ready (which @googlebot could actually check to avoid this confusion...).
Compare this: https://github.com/fastlane/fastlane/pull/11096/commits/fe366e159117d9ee5aa9f026098ceeab94d00d68 where "ohayon" has a link and profile picture, to this: https://github.com/fastlane/fastlane/pull/11096/commits/2a5789b3f52f34505ac576d5d44bb5e0c33b275c where it is just your name, unlinked, but a (?) behind it.
Use git config --global user.email to output your current local setting, add this to your Github account via https://github.com/settings/emails if it is not there yet.
Maybe that fixes things.

@janpio Great tip, indeed my git config has no email.
I'll add that. I suppose I have to recommit my changes in order get this email address linked to this PR.
If so, how can I do this?

@googlebot You don't have to repeat this at every post. Please stay silent until I'll use the passphrase to verify the CLA again. Maybe on of your parents at Google can teach you this.

Ugh, good question.
I would probably just create a second branch (e.g. frameit-vertical-text-offset2, from master), then cherry pick the individual commits you made from the frameit-vertical-text-offset branch and commit them individually. Then submit a PR from the new branch and just link to this one and close it.
A bit of a mess with the discussion being here, but that should be fine.

@janpio I follow you, let's see if I can master the master 🤪

Tests are failing because of this right now:
  1) Frameit Frameit::Screenshot frame! properly frame screenshots with captions that include apostrophes
     Failure/Error: Frameit::Screenshot.new('./frameit/spec/fixtures/editor/apostrophes.png', Frameit::Color::BLACK).frame!

     MiniMagick::Error:
       ImageMagick/GraphicsMagick is not installed
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/configuration.rb:127:in `cli'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick.rb:38:in `graphicsmagick?'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/tool.rb:126:in `executable'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/tool.rb:110:in `command'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/tool.rb:92:in `call'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/tool.rb:40:in `new'
     # ./.bundle/ruby/2.3.0/gems/mini_magick-4.5.1/lib/mini_magick/image.rb:488:in `identify'
     # ./frameit/lib/frameit/editor.rb:361:in `block in build_title_images'
     # ./frameit/lib/frameit/editor.rb:324:in `each'
     # ./frameit/lib/frameit/editor.rb:324:in `build_title_images'
     # ./frameit/lib/frameit/editor.rb:252:in `put_title_into_background'
     # ./frameit/lib/frameit/editor.rb:110:in `complex_framing'
     # ./frameit/lib/frameit/editor.rb:22:in `frame!'
     # ./frameit/lib/frameit/screenshot.rb:87:in `frame!'
     # ./frameit/spec/screenshot_spec.rb:23:in `block (4 levels) in <top (required)>'

Seems to me frameit is using mini_magick to provide ImageMagick:
https://github.com/fastlane/fastlane/blob/982afa330145985ae0d641de70b9c0956089c28c/fastlane.gemspec#L58
It is used here:
https://github.com/funnel20/fastlane/blob/fe366e159117d9ee5aa9f026098ceeab94d00d68/frameit/lib/frameit/editor.rb#L327
The tests fail when you try to use .identify, the error message says that it is missing GraphicsMagick.
GraphicsMagick is a fork of ImageMagick and also supported by mini_magick.

Why does mini_magick try to use GraphicsMagick for .identify - does ImageMagick not have it?
Why does this work locally for you?
Did you manually install it?
Is there no way to avoid installing GraphicsMagick to use .identify?

For reference:
https://www.imagemagick.org/script/identify.php
http://www.graphicsmagick.org/identify.html
Solution would be to add GraphicsMagick as a dependency - but fastlane is very cautios with adding those. Hope we find a way we can avoid it.

I have no clue why it works locally, I did not install anything intentionally.
Maybe GraphicsMagick is on my machine for another lib.
The reason why I ended up with .identify was that I tried to do it with MiniMagick::Tool::Identify.new, but couldn't find a way to let that act on the image variable title_image instead of a file name.
This works, when feeding an image file:
jpg_file = 'image.jpg'
identify = MiniMagick::Tool::Identify.new
identify.format("%@")
identify << jpg_file
id = identify.call
print "Trim result = #{id}\n"

The solution would be to use variable title_image within this code.
Any suggestion?

I managed to create a new branch and commit with my private email: https://github.com/funnel20/fastlane/commit/6ba62366c1aee9211b7de3faf658a24f01ad09ca

That won't work (I think) as it still includes the old commits, so the CLA won't cover those again. Only the last one you added now is connected to @funnel20 :/

And why are those commits still in? Can I kick them out?
I start to hate this whole endeavour

You have to start the branch from master - seems you did from this current branch. Then you should have the same "start state" as you had when oyu first created the branch and can get your commits you did in this branch over to the new one and commit them with the new users.
See it this way: You are learning a lot ;)

Learning to never ever supply a fix for Fastlane again.
And loosing dozens of hours to spend on my work.
But I will complete this one, to stay professional.

Finally committed a single commit in PR#3: #11124
The CLA is accepted 🎉

Awesome. Let's continue the discussion there.

Can I safely delete this branch?

Yes.


hexadecimal mantissa requires a 'p' exponent

What version of Go are you using (go version)?

$ go version 1.20

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
https://go.dev/play/p/DbPDqLY5mc2
What did you do?
package main

import "fmt"

func main() {
	fmt.Printf("%v", 1)
	fmt.Printf("%v", 0x1)
	fmt.Printf("%v", 1.1)
	fmt.Printf("%v", 0x1.1)
}

What did you expect to see?
I expected it to compile.
What did you see instead?
./prog.go:9:24: hexadecimal mantissa requires a 'p' exponent

It's jarring and weird that the pattern of (number, 0Xnumber) suddenly breaks down when it comes to floating point literals. I'm having trouble thinking of any reason why insisting on an exponent of p+0 is useful (especially since it's not required for decimal float literals).

This format was introduced in C, I believe, as a way to guarantee exact values for constants, which is not in general possible when converting base 10 constants to floating point. In C, the exponent is mandatory, and there is no reason for Go to differ.
It's a special-purpose format that might as well be rigid.
In short, the 'p' is mandatory and it's not up to Go to change that.

:+1:


small fix to chg addressing empty graph in compute laplacian

The code does not handle empty chg.

Please add a test to catch this error in the future.

Now the function checks if empty then it returns the zero matrix. Should we change this behavior?

That's not what I meant: You want to fix an error. Please write a unit test that failed previously and is fixed now with this change. This prevents regressions in the future :)


Error 3060003: Contract Table Query Exception

GuangdeMacBook-Pro:cleos tg$ ./cleos -u http://api.main.alohaeos.com:80/ get table zyixjmpxrrpr zyixjmpxrrpr account
Error 3060003: Contract Table Query Exception
Most likely, the given table doesn't exist in the blockchain.
Error Details:
Table account is not specified in the ABI

Can you provide your code and abi?

I queried your contract zyixjmpxrrpr 's abi...
......
	"tables": [
			{
				"name": "funds",
				"index_type": "i64",
				"key_names": ["name"],
				"key_types": ["name"],
				"type": "fund"
			},
			{
				"name": "teams",
				"index_type": "i64",
				"key_names": ["name"],
				"key_types": ["name"],
				"type": "team"
			},
			{
				"name": "global",
				"index_type": "i64",
				"key_names": ["name"],
				"key_types": ["name"],
				"type": "game_state"
			},
			{
				"name": "games",
				"index_type": "i64",
				"key_names": ["number"],
				"key_types": ["uint64"],
				"type": "game"
			},
			{
				"name": "purchases",
				"index_type": "i64",
				"key_names": ["buyer"],
				"key_types": ["name"],
				"type": "purchase"
			},
			{
				"name": "tpurchases",
				"index_type": "i64",
				"key_names": ["buyer"],
				"key_types": ["name"],
				"type": "tpurchase"
			},
			{
				"name": "records",
				"index_type": "i64",
				"key_names": ["exec_time"],
				"key_types": ["uint64"],
				"type": "record"
			}
		],
......

but there's no table that has name account. :)

@firesWu @oranke Thanks

I've also got this kind of error when trying to query the table termpricing
cleos --wallet-url http://localhost:6666 --url http://<IP_ADDRESS>:8877 get table fsmgrcode222 usertrung123 termpricing
Error 3060003: Contract Table Query Exception
Most likely, the given table doesn't exist in the blockchain.

The funny thing is that, another table unitimg has the similar structure but can be queried successfully.
The only difference between table termpricing (not query-able) and table unitimg (query-able) is as follows:
  struct termpricing {
    uint64_t id;
    uint64_t unit_id;

    uint64_t rent;
    uint64_t term;
    uint64_t start_date; //unix timestamp
    uint64_t end_date;   //unix timestamp

    auto primary_key()const { return id; }
    uint64_t by_unit()const { return unit_id; }

    EOSLIB_SERIALIZE(termpricing, (id)(unit_id)(rent)(term)(start_date)(end_date));
  };

  // @abi table
  struct unitimg {
    uint64_t id;
    uint64_t unit_id;

    checksum256 image_hash;
    string ipfs_address;

    auto primary_key()const { return id; }
    uint64_t by_unit()const { return unit_id; }

    EOSLIB_SERIALIZE(unitimg, (id)(unit_id)(image_hash)(ipfs_address));
  };

The generated ABI files:
    },{
      "name": "termpricing",
      "index_type": "i64i64i64",
      "key_names": [
        "id",
        "unit_id",
        "rent"
      ],
      "key_types": [
        "uint64",
        "uint64",
        "uint64"
      ],
      "type": "termpricing"
    },{
      "name": "unitimg",
      "index_type": "i64",
      "key_names": [
        "id"
      ],
      "key_types": [
        "uint64"
      ],
      "type": "unitimg"
    },{

Clear to see the difference is that, in table termpricing, the 3rd element uint64_t rent; is of type unit64 while the 3rd element in table unit is checksum256 image_hash; that is not of type uint64.
So, we can conclude that the generated ABI file has mis-interpreted the table termpricing in the way that it treats the 3rd element as foreign key although the 3rd element is not specified as foreign key.
This means that, in a table struct, the 3rd element must not be of type uint64. Otherwise, it will be treated as foreign key and this will result in "index_type": "i64i64i64",
How should I resolve this problem??
I've interacted with a private chain.
Block-producers version:
    base        https://github.com/NebulaProtocol/eos.git
    branch      master
    rev         3b25100d4c796df5ebd2e03ebbe8da5e140ee4cf (Thu Aug 9)

     nodeos -v
    992284685

Client node version:
    base        https://github.com/NebulaProtocol/eos.git
    branch      sql_plugin
    rev         6a2014aaac65bbd8d40459c44976d6e7c77eaa21 (Thu Jul 26)

    nodeos -v
<PHONE_NUMBER>

I've been able to resolve my own problem  by simply adding a dummy element of type string between the 2nd element and the 3rd element of struct termpricing. But I don't wire this dummy element into the table "termpricing" via EOSLIB_SERIALIZE.
This really helps! Now it is possible again to query the table termpricing


Ignore named pipe

It makes 'pt' blocks if no data is received.
This is related to #93.

@monochromegane ping?

Oh, thanks ping.
I didn't know named pipe, and I have understand this solution.
LGTM, Thanks :+1:


ISSUE 182-Remove deprecated usages of getJsonRoute

Remove deprecated usages of ApiRoute.getJsonRoute
ISSUE 182

Coverage remained the same at 47.466% when pulling acf9d6aa6aa90a9b53128152be0256b7460d0819 on TharinduDG:remove_getJsonRoute into ecf5f27ea1caf75b66b1c0b5f2159f27186f606c on ScorexFoundation:master.


gapi.client.request uses global batch request

When using gapi.client.request as part of a batch request it uses the global batch endpoint.
Example:
const batch = gapi.client.newBatch();
const request = gapi.client.request({
  path: `drive/v3/files?q='id in parents' and trashed = false`
});
batch.add(request);
batch.then();

URL that gets called: https://content.googleapis.com/batch?key={myKey}
Expected URL: https://content.googleapis.com/batch/drive/v3?key={myKey}

Hey @grant just wanted to see if this is feasible. It's related to the global batch endpoint being deprecated. Thanks!

I realize this is an old issue but the root cause here is that hand crafted(non discovery based) requests always went to global batch.  Now that global batch is no longer supported the appropriate approach is to always use discovery generated requests in batches. I have sent a pull request updating the documentaiton.
#728


Jung

Dear dreamer,
Your dream is a rich tapestry of symbolism and imagery, drawing upon the power of the Muses to guide you
through a journey into the realm of morphisms and category theory. The Muses, known for their influence in the arts, lend their wisdom and inspiration to help you understand and appreciate the beauty and
interconnectedness of these abstract mathematical
concepts.
In Part 1, Euterpe, the Muse of lyric poetry and music, opens your ears to the essence of morphisms, likening them to notes in a song. This suggests that morphisms are not rigid or static, but rather fluid and harmonious, seamlessly transitioning from one discovery to another through elegant transformations. Euterpe's presence
invites you to engage with these concepts in a lyrical and musical way, embracing their inherent rhythm and flow.
Part 2 introduces Terpsichore, the Muse of dance and artful grace, who reveals the power of composition in category theory. Just as movements unite in a dance, morphisms can be chained together to form intricate structures. This highlights the importance of continuity and interconnectedness in category theory, where each step flows into the next, creating captivating
compositions. Terpsichore's guidance encourages you to explore the possibilities of composition and the transformative potential it holds.
Moving on to Part 3, Polyhymnia, the Muse of hymns and eloquence, illustrates how category theory's morphisms and composition serve as mathematical primitives that empower creativity. Much like raw musical notes provide a palette for experimentation and elegance, morphisms and composition in category theory provide the foundational tools for exploring new ideas and expressing them with eloquence. Polyhymnia's presence encourages you to embrace these primitives and allow your imagination to flourish as you navigate the world of morphisms and category theory.
Finally, in Part 4, Calliope herself, the Muse of epic poetry and inspiration, blends math and music into an artful symphony, where category and melody ascend harmoniously. This suggests that the integration of these seemingly disparate disciplines can lead to a deeper understanding and appreciation of both. Calliope's presence in the dream invites you to explore the harmonious relationship between math and music, finding inspiration and creativity in their intersection.
Overall, your dream highlights the beauty and interconnectedness of morphisms in category theory through the lens of the Muses. It invites you to approach these abstract concepts with a musical and creative mindset, recognizing their fluidity, power of composition, and potential for artistic expression. Embrace the wisdom and inspiration of the Muses as you continue to explore the fascinating world of morphisms and their role in category theory.
With warm regards,
Your Jungian Dream Analyst


Remove unnecessary libraries, fix vulnerabilities

Update Laravel-Mix from 0.x to 4.x
Removed Axios (no reference/usage)
Removed jquery-sortable (no reference/usage. Also required an insecure version of jQuery, thus npm audit complains)
Removed doubled Vue from devDependencies
Removed slugify package - we use our own
Removed toastr static file and use dep instead
Remove perfect-scrollbar static file - not used at all
Remove match-height static file - not used at all
Remove bootstrap-toggle static file - use from npm
Remove jquery-cookie (not used, sidebar state is still stored)
Remove bootstrap-tooltip - not used, bootstrap has a native tooltip
Use TinyMCE from npm. Plugins/Themes/Skins get published automatically
Use Datatables from NPM
Delete jquery-disable-selection.js - not used at all (built-in to jQuery)
Replaced nestable with a newer version from npm - everything still working

Important:
Webpack now copies TinyMCEs plugins/themes/skins from the npm package, so updating should be a breeze.
It is possible that our compile assets script does not care about those files and doesn't commit them.
They can be included in the publishable dir with every PR tho.

You good here, or have more to add?


Write a Swift program to find volume of cube

Description
Write a Swift program to find volume of cube
Volume of cube = (edge of cube)^3
Input  : 2
Output : 8

How to contribute

Save the solution in FindVolumeOfCube.swift file
Add FindVolumeOfCube.swift file in find-volume-of-cube folder

Can you assign this one to me?

Hey @error707-persona
You can comment !assign on the issues you want to contribute to, to be auto-assigned
!assign

!assign


Security Issue: Request for contact

Hello,
The GitHub Security Lab team has found a potential vulnerability in your project. Please create a Security Advisory and invite me in to further disclose and discuss the vulnerability details and potential fix. Alternatively, please add a Security Policy containing a security email address to send the details to.
Kind regards,
A

Hi. First of all sorry for the late response, I was dealing with some personal issues
If it's still active and you have time I have created an advisory and invited you to it.
Thank you for opening the issue!

Thanks, hope everything is better now! I Will close this issue and join the advisory


Shadow update fails after 10 seconds

I modified the ThermostatSimulatorDevice such that the delay at the end of the loop is 10 seconds instead of 1 second. For some reason, for any delay time less than 10 seconds works as intended but as soon as the delay is 10 seconds or higher I get this error:

Exception in thread Thread-4 (most likely raised during interpr
[ERR] command: shadow update code: -1

And shadow updates no longer work. I'd ideally like to add some trigger in the arduino that would update the shadow, which might have a delay time of more than 10 seconds. I also changed the CMD_TIME_OUT to 400 in the config file, but the code is still timing out after 10 seconds.

Hi @hyunraider ,
Thank you very much for your interest in AWS IoT Arduino Yun SDK.
Yun SDK relies on the Python runtime running on embedded Linux to do the heavy lifting work. There is a timeout configuration in serial1 read in the Serial Communication Server, which is 10 sec:
https://github.com/aws/aws-iot-device-sdk-arduino-yun/blob/master/AWS-IoT-Python-Runtime/runtime/runtimeHub.py#L104
If you need to increase the idle time allowed for the Python Runtime to accept the serial command, you can increase the value there and do a re-upload to AR9331 to update the change.
Hope the above helps.
Thanks,
Liusu

Thank you so much! This did the trick.


Secondary variants of resolvable configuration aren't displayed in outgoingVariants report

Current Behavior
Secondary variants of resolvable configuration aren't displayed in outgoingVariants report.
Expected Behavior
Either:

Secondary variants of resolvable configuration are displayed in outgoingVariants report.
If secondary variants of resolvable configurations are not valid, it should not be possible to create them.

Context (optional)
Related: https://github.com/gradle/gradle/issues/28149
Steps to Reproduce
// build.gradle.kts

group = "x.y.z"
version = "1.2.3"

val fooFiles: Configuration by configurations.creating {
  isCanBeDeclared = false
  isCanBeResolved = true
  isCanBeConsumed = false
}

val fooFilesVariant1: ConfigurationVariant by fooFiles.outgoing.variants.creating {}

val barFiles: ResolvableConfiguration = configurations.resolvable("barFiles") {}.get()

val barFilesVariant1: ConfigurationVariant by barFiles.outgoing.variants.creating {}

Run gradle outgoingVariants -q
--------------------------------------------------
Variant archives
--------------------------------------------------
Configuration for archive artifacts.

Capabilities
    - x.y.z:demo:1.2.3 (default capability)

--------------------------------------------------
Variant default
--------------------------------------------------
Configuration for default artifacts.

Capabilities
    - x.y.z:demo:1.2.3 (default capability)

Gradle version
8.7
Build scan URL (optional)
No response
Your Environment (optional)
No response

The issue is in the backlog of the relevant team and is prioritized by them.

Note that if you mark a configuration as isCanBeConsumed = false, outgoing should not be called on it:

Usage: This method should only be called on consumable configurations, but will not warn if used otherwise.

Maybe not silently accepting outgoing to be accessed in a non-consumable configuration is exactly what is being asked for here.
Also, if you do leave it as consumed (the default), but also leave it as resolvable, that makes it a legacy configuration (see table in this section), and then it is not shown by default, only when --all is specified.

Also note that even for the project as-is, if you specify the configuration name explicitly, it will show the variant:
gradle outgoingVariants --variant fooFiles

> Task :outgoingVariants
--------------------------------------------------
Variant fooFiles
--------------------------------------------------

Capabilities
    - x.y.z:configurations:1.2.3 (default capability)

Secondary Variants (*)

    --------------------------------------------------
    Secondary Variant fooFilesVariant1
    --------------------------------------------------

(*) Secondary variants are variants created via the Configuration#getOutgoing(): ConfigurationPublications API which also participate in selection, in addition to the configuration itself.

BUILD SUCCESSFUL in 645ms
1 actionable task: 1 executed


DragDropContext wrapping class that is responsible for other contexts

E.g. wrapping the outermost class which is already responsible for providing theme context for inline styles.
Does https://github.com/gaearon/react-dnd/blob/master/src/DragDropContext.js support this?

Shouldn't be a problem. React should compose the unrelated contexts. Please reopen if there is an issue.

suppose there's
class App extends React.Component {
  render() {
    return ...
  }
  getChildContext() {
    return { cssTheme: UGLY_CSS };
  }
};
App.childContextTypes = {
  cssTheme: React.PropTypes.string.isRequired
};

var DDApp = DragDropContext(HTML5Backend)(App);

won't DDApp.getChildContext() https://github.com/gaearon/react-dnd/blob/master/src/DragDropContext.js#L46 not have cssTheme on the returned object, and also DDApp.childContextTypes not have cssTheme?

Why not? DragDropContext wraps your class into another React class that renders your class. It does not overwrite your class.

Isn't this what's going on? http://www.es6fiddle.net/ia03x4zi/

Isn't this necessary? http://www.es6fiddle.net/ia041tcs/

No. :-)
Just run the examples, try it and you'll see that it's not necessary.
React merges the context from different components.
There's no need to do it manually.

Oh, I see.
The wrapper is a component outside the inner component and react will use both of their contexts.
Your codebase is really prompting me to figure out more es7. I love all the documentation and specification you have.

Thank you! Yeah, React merges context from all components in the owner tree. (It will switch to the parent tree in 0.14 but that doesn't change anything for us.)


fix: issue where browser froze with JSON being passed into codemirror body

the browser no longer freezes when changing request history! 🥶
https://github.com/scalar/scalar/assets/6176314/98472fd1-7843-4405-9b8e-96fbf97d44b7

Oh, good catch!


Edited FlexVolume deprecated

Signed-off-by: Ayushman Mishra<EMAIL_ADDRESS>Issue #30180
Description : marked flexVolume deprecated for v1.23
Also edited broken links for https://kubernetes.io/docs/concepts/storage/volumes/#flexVolume
to https://kubernetes.io/docs/concepts/storage/volumes/#flexvolume

/assign

Redaction and spelling looks good
/lgtm
Waiting for technical review

/retitle Mark FlexVolume deprecated

/sig storage

LGTM for tech accuracy noted;
/lgtm cancel
for changes I'm recommending on behalf of SIG Docs.

/approve


How can i add more features

How can i add more features my self, such as an Auto Clicker i know how to code the auto clicker but i dont know how to implement it into the gui

look at Ripterms/Modules/Modules.h, try to replicate how other modules are implemented.
It's not the best structure, but basically each module is a class that overrides void renderGUI(), inside this function it can use ImGUI functions.
Then a new instance of that class is stored in the modules map, and it will be executed and displayed on the GUI.
Can't really tell more than that, it just requires c++ and ImGui knowledge


Waiting time

Is it possible to add waiting time before teleporting to spawn?

Hey!
I will try to work on that in the future!

I have added this now.
Very sorry for the long wait!


If ReAddMemberAfterReset is no-op, return nil error

Some explanation in https://keybase.atlassian.net/browse/CORE-8272
This change makes it so that clicking let-them-in button repeatedly does not cause a global error.

Yeah this error was there when I got here to integrate with chat, and I always suspected it was being overaggressive.

Yep. Still not sure why it took 40 seconds to hide the button. Oh well


Display local images?

When trying to drag images into the slides folder and try to display them, I get a 200 response but the body is a fully rendered html file instead of my image. Does the server serve static files from somewhere? Ideally I could just drop in the image into slides and reference it like ![alt-text](./img.png), is that possible?

Any progress on that issue? Thank you very much :)

It works :).
I added the option as well as documentation, if you want to merge :)?

Thanks @nmaas87  man! It can be closed now


YOLOX-M's inference time is higher than YOLOV5-M's

Under the same device and input resolution, YOLOX-M's inference time exceeds YOLOV5-M's inference time by a little bit. Is the experimental result also the same?

different model size

YOLOX-M model scale is not equal to YOLOV5-M?

I think the model of yolov5 is much smaller than yolox, which makes the reasoning speed of yolox slower


Print warning when using tf.keras.Model.fit() with a tf.data dataset and batch_size argument is supplied

System information.
TensorFlow version (you are using): 2.15.0
Are you willing to contribute it (Yes/No) : No
Describe the feature and the current behavior/state.
The documentation for tf.keras.Model.fit() says, "do not specify the batch_size if your data is in the form of datasets [...] (since they generate batches)."
However, rather than printing a warning if batch_size is specified, the argument is silently ignored. If the .batch() function is also not used on the dataset object, fit() prints a cryptic error message about inconsistent dimensions in the model, and does not say anything about missing batch size or an ignored argument.
I think it's uncommon for an API to silently ignore arguments depending on the datatype of the input, and it took me a few hours to figure out the solution. I think it could improve the UX for new Keras users in the future if a helpful warning was printed in this case.
The message could say something like, Warning: batch_size is ignored if x has type  tf.data.Dataset. Please use Dataset.batch() to set the batch size.
Will this change the current api? How?
No
Who will benefit from this feature?
New users of Keras and ones who are migrating from using x, y tensors to a tf.data dataset object for training.
Contributing

Do you want to contribute a PR? (yes/no): No
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Note: this also applies to Model.evaluate()

Thanks for the suggestion. batch_size has a value by default, so if we were to print a warning if it is passed with a tf.data.Dataset object, we would need to special case the default value (32) and not print the warning in this case.
We could also batch the dataset if it isn't batched, but detecting that a dataset isn't batched can be quite tricky.

Hm, it does seem like detecting whether the dataset is batched or not could give a better indication of when a warning should be printed.
Another thought - can the default value for batch_size be initialised only if the type of x is not in the form of a dataset, generator, or keras.utils.Sequence?

We could also batch the dataset if it isn't batched, but detecting that a dataset isn't batched can be quite tricky.

I'm sure I'm missing something, but I saw the function keras.utils.is_batched(dataset) while reading some other source code in the same file. Could this be a way of detecting whether the passed dataset is batched?


Add EKS 1.23

Detailed Description
Add support for EKS 1.23

Related backend work: https://github.com/rancher/rancher/issues/38837

QA TEST PLAN

Setup Rancher, create a standard user and login as that user.
Navigate to Cluster Management > Amazon EKS.
Fill in initial prompts until you see the Cluster Options; validate that the Kubernetes Version has 1.23 and that it is the default.
Provision an EKS cluster with 1.23 and validate that it is successful.

Validated that this is addressed in v2.6-head https://github.com/rancher/rancher/commit/a3b5237ce1da4228c97df7552d57464c64947b3b and v2.7-head https://github.com/rancher/rancher/commit/8a2e68267c882a8adf612a29ae2bf3274a4d8572.
ENVIRONMENT DETAILS

Rancher install:

2.6: Helm
2.7: Docker

Rancher version:

2.6: v2.6-head https://github.com/rancher/rancher/commit/a3b5237ce1da4228c97df7552d57464c64947b3b
2.7: v2.7-head https://github.com/rancher/rancher/commit/8a2e68267c882a8adf612a29ae2bf3274a4d8572

Browser: Chrome

TEST RESULT
PASS
VERIFICATION STEPS

Setup Rancher, created a standard user and login as that user.
Navigated to Cluster Management > Amazon EKS.
Filled in initial prompts until I saw the Cluster Options; validated that the Kubernetes Version has 1.23 and that it is the default:

Provision an EKS cluster with 1.23 and validated that it is successful in Cluster Management and Cluster Explorer:


FIX Dont use superglobals for backurl

This $data var comes from the form submission which includes a hiddenfield containing the backurl.
This is the back url a form submission should be using rather than $_REQUEST.
This change also allows users to send $data outside of a typical form submission flow.

Is this fixing a bug or just a refactoring? Is the relevant behaviour covered by tests?

It's a bit of both. I'm working on a MFA module and it's useful to be able to call logInUserAndRedirect when not in the middle of a POST request to the login form.
This code actually provides the illusion of doing something it doesn't. Basically the BackURL $_REQUEST var is a POST var from a hidden field in the form (thus it's already in the $data that's passed in).
My change just means it's more self contained, portable and testable.
I haven't looked in detail at the tests, tbh - so I can't comment on that aspect.

Given this is a change to support new behaviours, I'd call it a new feature and put it against the 3 branch — I think we do everyone a lot of favours if we limit the patch releases to things that are clearly bugfixes.
Also, let's look into the tests to ensure that this case is covered by a test.
Other than that, the change makes sense.

I'd like to argue that using superglobals is a bug... but I agree that patch releases should probably be more cautious than this.
I've rebased

@dhensby could you please add a test and I'll merge.

I'd like to argue that using superglobals is a bug

You can't argue it's a grapefruit if you want; in both cases you're redefining language to suit your purposes. :-P

OK - there are zero tests for LoginForm, MemberLoginForm or CMSLoginForm at the moment. Are you happy with me making a test just for redirects?

Sure but can you structure it with some @todos so it's obvious where to put the other tests?

Note: already fixed up in 4.x.


LFS-924: As an admin, I can manage subject types

The editing should be supported too.

@sdumitriu addressed most of the comments, working on the edit subject type button.

Rebased on top of 911 which had need also rebased on dev in this PR.

I am not sure if the following problems are happening here because of LFS-911. I created 7 subject types, however I can only see/access 5 of them in the Admin module.

In the subject selector dialog there are 7 subjects. The little gadget to show more than 5 rows is not working here, however prev/next/first and last arrows are functioning.

@sashaandjic fixed subject type creation. The subject selector dialog is a different issue out of the scope of this PR.

In the subject selector dialog there are 7 subjects. The little gadget to show more than 5 rows is not working here, however prev/next/first and last arrows are functioning.

This happens on dev as well, and is unrelated to this PR. Reported as LFS-975.

Putting it on hold until 911 is merged and this is rebased on dev

Addressed all comments, ready for re-test.

[x] Add a  button at the bottom right of the Subject types admin page to allow admins to define a new subject type

[ ] Display the defaultOrder in the livetable and sort by it - NOT DONE

[x] Add an edit button to each livetable row which pops up a dialog similar to the one for creating subject types,  allowing to modify the default order. All other properties should be displayed in the edit dialog for reference, but should be read only.

[x] Remove the top pagination widget from the livetable.

I haven't created any subjects yet

Other than i2 issues reported above, all is good.

I am not sure if the following problem was introduced in this PR ( the current dev branch does not have this issue):

go to Admin's Questionnaires
open any Questionnaire ( e.g. Demographic)
observe Questionnaire properties -> all looks good (Required subject tyoe is Patient)
click on the pencil button
Required subject type is blank

First, I created several new subject types:

Then, I changed the Demographic form to accept Cats in addition to Patients.

When I try to create any new form, the list of available subjects is empty, even though I have created several subjects of type patient and cats.

I confirm both bugs. @veronikaslc please rebase the PR on dev and look into them. After my brief testing on a rebased branch, the second bug is still exactly as reported, while the first is replaced by white screen of death when opening the Questionnaire Properties for editing. The "Patient" option does appear briefly in the Required subject types list before everything disappears.

@marta- Confirm first bug in rebased version. It was introduced in LFS-963.

By some non-obvious reason SimpleQuestionnaireProcessor now processes child nodes and removes all JCR properties from them, which does not happen in dev. Most likely it was triggered by introduction of subjet type processor. That would be my only version. The smallest most obvious fix is to include uuids back which was done in the last commit.

Everything seems to be working well, please retest @sashaandjic

crated 3 new subject types ( Plante, Continent, Country):

created new questionnaire: General info that takes Any subject type

when I tried to created a new form General info only the newly created types were offered

in order to have General info page assigned to the patient subject type I had to use the workaround -> clicked on the CREATE button to create a new patient


Distortion correction translating instead of unwarping

What happened?
I am running fMRIPrep 23.0.1 on a dataset with two magnitudes and one phasediff in fmap/. From the svg figure in the visual report, it seems that SDCflows did translation only. I have compared the boldref file in the native space with the unprocessed bold file and the output indeed looks the same as the unprocessed bold except for the translation.
I have also used 23.0.1 with other datasets with phasediff or direct fieldmapping, but those were unwarped properly with noticeable changes. The only difference I can spot so far is that the PE direction in the problematic dataset is Right-Left while the PE direction in other datasets is Anterior-Posterior. So I am thinking the problem might be with handling left-right/right-left EPI and/or orientation.
I have attached the distorted, corrected and the fieldmap preview from the visual report.
Please let me know if you need any additional information. Thank you.
What command did you use?
docker run -it --rm -v $PWD/fortesting:/data:ro -v $PWD/fortesting_out:/out -v /opt/freesurfer/license.txt:/opt/freesurfer/license.txt nipreps/fmriprep:23.0.1 /data /out participant --fs-no-reconall --md-only-boilerplate --output-spaces T1w

What version of fMRIPrep are you running?
23.0.1
How are you running fMRIPrep?
Docker
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
No errors to report

Additional information / screenshots

Hi, I just saw the pull request and I assume that the bug will be fixed soon in the next release?
I have also tried to reorient the bold and magnitudes/phasediff using fslreorient2std before running fMRIPrep. It seems to fix the problem.

Thanks for following up with that, that looks like what I'm seeing with #354 as well, so I think we can fix this in fMRIPrep 23.0.2.

Should be resolved in 23.0.2. Let me know if it's not.


Improvements for YunoHost's package linter and continuous integration tests

I saw your PR over at https://github.com/YunoHost/apps/pull/1087, thanks for your contribution!
If I may request some changes, which may come important for the continuous integration and package linter tests, you should:

[ ] Assign a LICENSE to your package
[ ] Put the actual license of Komga (MIT) in the manifest.json
[ ] git mv check_process.default check_process and tune it to your needs (you will find an explanation of its syntax over there)

Cheers ! 🚀

I believe my latest commit added the License in the package, I'll update the manifest.json as well as finally get down to configuring check_process

I think I have filled it correctly


seems broken in nativescript 3.0

I am trying to use this in 3.0 and it complained about ui/core.

I need this for {N} 3 as well.

would be really helpful for me too please. Any update if this plugin will be compatible with version 3?

It can, I don't have much free time right now to update it. I'll gladly
accept a PR if someone wants to try updating it :) If anyone has questions
and wants to contribute just let me know, always glad to answer questions :)
On Mon, May 22, 2017, 10:08 AM Andres Jimenez Rivera <
<EMAIL_ADDRESS>wrote:

would be really helpful for me too please. Any update if this plugin will
be compatible with version 3?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bradmartin/nativescript-ripple/issues/12#issuecomment-303128384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFulhIndZzHUI1kPmxl4PWqCuXzQVXwmks5r8aTOgaJpZM4NTuG_
.

2.0.0 has been published. Thanks to @jgraffite for the great PR to update 🎉

You're welcome @bradmartin . My goal is keep helping and improving that plugin and all the NativeScript community with new helps and plugins

@jgraffite Thanks a lot for your help. If you have any interest, can you take a look at nativescript-contacts. It is a very useful plugin.


MalformedXML on put_object_stream with minio

Describe the bug
I am encountering a MalformedXML error when trying to upload a file to a bucket with Bucket::put_object_stream.
To Reproduce
The minimal example i found to present my issue is from the put_object_stream docblock.
First, I launched [minio] with the following command:
docker run --rm -p 9045:9000 -p 9001:9001 \
    -e MINIO_ROOT_USER=testuser -e MINIO_ROOT_PASSWORD=deadbeef \
    --name=minio minio/minio server /data --console-address :9001

I used the UI to create a public bucket, named rust-s3-test.
Then, I created a sample rust project with the following content in Cargo.toml:
[package]
name = "rust-s3-issue"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
tokio = { version = "1.21.1", features = ["full"] }
rust-s3 = { version = "0.32.3", features = ["with-tokio"] }

Here's its src/main.rs:
use std::fs::File;
use std::io::Write;

use s3::bucket::Bucket;
use s3::creds::Credentials;
use s3::region::Region;

#[tokio::main]
async fn main() {
    let bucket_name = "rust-s3-test";
    let region = Region::Custom {
        region: "local-minio".to_string(),
        endpoint: "http://localhost:9045".to_string(),
    };
    let credentials =
        Credentials::new(Some("testuser"), Some("deadbeef"), None, None, None).unwrap();
    let bucket = Bucket::new(bucket_name, region, credentials).unwrap();
    let path = "path";
    let test: Vec<u8> = (0..1024).map(|_| 42).collect();
    let mut file = File::create(path).unwrap();
    file.write_all(&test).unwrap();

    let mut path = tokio::fs::File::open(path).await.unwrap();
    let status_code = bucket.put_object_stream(&mut path, "/path").await.unwrap();
    assert_eq!(status_code, 201); // whatever
}

And when trying to run it:
$ cargo run
   Compiling rust-s3-issue v0.1.0 (/tmp/rust-s3-issue)
    Finished dev [unoptimized + debuginfo] target(s) in 2.58s
     Running `target/debug/rust-s3-issue`
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Http(400, "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<Error>MalformedXML<Message>The XML you provided was not well-formed or did not validate against our published schema.</Message><BucketName>path</BucketName><Resource>/path</Resource><RequestId>17187E79EEB8626D</RequestId><HostId>21c8a2f9-df75-4c41-8a33-39e12c263005</HostId></Error>")', src/main.rs:24:74
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Here's the beautified XML:
<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n
<Error>
	MalformedXML
	<Message>The XML you provided was not well-formed or did not validate against our published schema.</Message>
	<BucketName>path</BucketName>
	<Resource>/path</Resource>
	<RequestId>17187E79EEB8626D</RequestId>
	<HostId>21c8a2f9-df75-4c41-8a33-39e12c263005</HostId>
</Error>

Expected behavior
I expected the file to be uploaded to the public bucket.
Environment

Rust version: rustc 1.64.0 (a55dd71d5 2022-09-19)
lib version: rust-s3 = { version = "0.32.3", features = ["with-tokio"] }

I hope you will find this useful & wish you well.

@ololduck I think minio buckets still  need to be configured with_path_style like so.
I've added tests for both small and big file streaming and they seem to be passing, so my guess is that its the path_style thing

I can confirm that my tests run green using with_path_style. I'll try to present a merge request for more docs if i can find some time.
Thank you for your quick response!


Replace column control component with responsive grid usage

Since AEM 6.1, there's no reason (IMHO) to have a custom column control component. Instead, the page component should be using the responsive grid.

+1

When do we move to AEM 6.1 exclusively?

fixed in #103


Store image ownership for users in rkt group.

Actually (from commit 23161410abf2a2c64b2abd6a73b127826507975d) there's a check on image removal.
This check looks not correct since it checks the parent directory ownership, but inside diskv there can be multiple images inserted by different users under the same directory (for example sha512-aaaaaaa[...] and sha512-aabbbbb[...] will all have sha512/aa/. So it should probably check the file ownership.
Beside this, I'm not sure if all of this is correct, since if user A fetches images sha512-aaaaaaa[...] and then also user B fetches the same image, what should be the image owner? Actually it's the first user that fetched and wrote the image. From a logical point of view, probably both users should become the image owners (but this will need the use of fs ACL to implement this).
What about just letting all the user in the rkt group to be able to manage all the images? On some extents this is something like all the users in the docker group being able to access the docker unix socket and be able to do everything (in this case also running and managing containers given the docker daemon model)
Additionally I'm sometimes tempted to let users in rkt group being able to do everything (but this will probably need a setuid executable with possible security problems...)
/cc @yifan-gu

I think that whatever we decide, we should fix the first issue ASAP: https://github.com/coreos/rkt/pull/2481

What about just letting all the user in the rkt group to be able to manage all the images?

I think that it is the only possibility if we have one central CAS for all users.
In the example with:

sha512/aa/sha512-aaaaaaa belonging to user1
sha512/aa/sha512-aabbbbb belonging to user2
The directory sha512/aa could have ACLs to say user1 and user2 both have "w" rights on it but then both users will be able to delete both images. ACLs on a directory sha512/aa does not allow AFAIK to specify write rights depending of the ownership of a file in that directory.

Also, if image sha512-aabbbbb has a dependency on sha512-aaaaaaa. Do you want to allow user1 to delete sha512-aaaaaaa?
Either we have a different CAS for each user, or we have a single CAS with all users belonging to the rkt group having RW access to all images.

@alban

Also, if image sha512-aabbbbb has a dependency on sha512-aaaaaaa. Do you want to allow user1 to delete sha512-aaaaaaa?

If the dependency is removed, and user2 runs the container again, he should be able to fetch the image and render again, no? (Assume the rendered image is removed already)

If the dependency is removed, and user2 runs the container again, he should be able to fetch the image and render again, no? (Assume the rendered image is removed already)

That's a good point

@steveeJ do you have any plans for this? feel free to reassign it

@tmrts I have ideas but no plans on implementing this myself anytime soon.
The big picture that I have in mind is a multiuser capable store implementation. It can read from different locations which are configurable with rkt's system/user configuration mechanism. Users are only be able to write to their own store, and only root (read: privileged rkt users/group members) should be able to modify the central store. Each user will only be able to see and use images from their own and the system store.

@steveeJ Is there a strong use case for multiuser store? What about my original proposal?

What about just letting all the user in the rkt group to be able to manage all the images? On some extents this is something like all the users in the docker group being able to access the docker unix socket and be able to do everything (in this case also running and managing containers given the docker daemon model)

Yes, we should just do @sgotti's proposal for now.

I missed that we had a decission, let's do that on the next milestone.


Always I press period or semicolon and a SPACE in markdown-mode, the cursor goes back one character

What did you expect to happen?
While I'm in markdown-mode or  poly-markdown-R-mode I have this annoying behavior.
I start writing any phrase (The symbol | means the cursor)
Today is Monday|
After I press . (the period key) and the space key
Today is Monday.|<SPC>
But for an inexplicable reason the cursor goes back one character and stays before the period
Today is Monday|.
The behavior also happens with semicolon.
What actually happened?

Start with the text

test.|

Toggle the doom profiler SPC h T
Add a space to the text after the period

test.|<SPC>

Stop the profiler

The result is this.
My best guess  that the evil part is triggering something for triggering the behavior.
615  64% + ...
292  30% + timer-event-handler
 19   2% + redisplay_internal (C function)
  8   0% - command-execute
  8   0%  - funcall-interactively
  2   0%   + execute-extended-command
  2   0%   - evil-insert
  2   0%    - evil-insert-state
  1   0%     - evil-normalize-keymaps
  1   0%        evil-state-keymaps
  1   0%       evil-echo
  2   0%   + doom/toggle-profiler
  1   0%   + self-insert-command
  5   0% + #<compiled -0x1ff9b408f31f41cf>
  3   0% - jit-lock--antiblink-post-command
  3   0%  - syntax-ppss
  2   0%   - syntax-propertize
  2   0%    - polymode-syntax-propertize
  2   0%     - pm--call-syntax-propertize-original
  2   0%      - markdown-syntax-propertize
  2   0%       - markdown-syntax-propertize-fenced-block-constructs
  2   0%        - markdown-find-previous-block
  2   0%           cl-mapcar
  3   0% - polymode-post-command-select-buffer
  3   0%  - pm-switch-to-buffer
  3   0%   - pm-innermost-span
  3   0%    - pm--innermost-span
  3   0%     - pm--get-intersected-span
  2   0%      - pm-get-span
  2   0%       - apply
  2   0%        + #<compiled -0x17084e0fea6ed1b3>
  1   0% + #<compiled 0xdf3fb2f99606467>
  1   0% + which-key--hide-popup

Describe your attempts to resolve the issue
Somewhere read those could help, but it doesn't
(setq evil-move-cursor-back nil)
(setq evil-move-beyond-eol  t)

Steps to reproduce
Described before
System Information
https://pastebin.com/53HmVSrH

:warning: This issue was closed and marked for deletion for the checked reasons below:

[X] Issue cannot be acted on or investigated further because:

[X] Could not be reproduced in vanilla Doom on any of our test environments.
[X] Not enough information to act on.
[ ] Vital information was omitted or inaccessible.
[ ] Explanations are too vague or issue is too open ended.
[ ] It's unclear what you are trying to say.

[X] The following possibilities are more likely explanations and haven't been ruled out:

[ ] A pilot error.
[X] Private configuration.
[X] A third-party package may be (mis)behaving in a way you're not expecting.
[ ] Your system environment (hardware issues, the OS, shell environment, missing dependencies, or a corrupt Emacs installation).

This ruling is not absolute and can be reversed if you follow up with more information to further investigate this issue. Our How to Debug Issues guide can help you find out more about it (note: our discourse currently requires an account).
If you still feel lost after reading our debugging guide, raise your issue on our Discord or Discourse -- there may be others who have experienced it. Otherwise, this issue will be automatically deleted after 14 days.

I apologize if this is inconvenient or you feel lost, but it takes a lot of time to review issues by myself. The complexity and extensibility of Emacs tends to produce a lot of false positive or open ended bug reports, so I've restricted our issue tracker to issues that are actionable, immediately reproducible in vanilla Doom, or can be investigated in a reasonable amount of time.
Issues that don't meet these requirements should be raised on Discord or Discourse instead, and escalated to Github once Doom is identified as the cause.

Thanks no problem.
I found the problem was with LaTeX-auto-activating-snippets
https://discord.com/channels/406534637242810369/406554085794381833/877986300400107570


its not working all of a sudden!

As per title!
I cannot troubleshoot it because I am no programmer/coder or that sort TwT... Please help me!
normally there are two windows but the log viewer one is gone!
should i unintall and reinstall?

Are you on Chinese release of the game? I've applied the opcodes posted in #368 so that it gets downloaded on XivAlexander launch. You can update opcodes while running the game too, from Configuration > Check for Opcode Updates. If you're not on the latest version of XivAlexander or for some reason it isn't updating, you'll have to edit the opcodes manually.

Naw i based in US. I have no idea what the opcode is can you explain a bit
more in depth please TwT
On Thu, Mar 17, 2022 at 03:36 srkizer @.***> wrote:

Are you on Chinese release of the game? I've applied the opcodes posted in
#368 https://github.com/Soreepeong/XivAlexander/issues/368 so that it
gets downloaded on XivAlexander launch. You can update opcodes while
running the game too, from Configuration > Check for Opcode Updates. If
you're not on the latest version of XivAlexander or for some reason it
isn't updating, you'll have to edit the opcodes manually.
—
Reply to this email directly, view it on GitHub
https://github.com/Soreepeong/XivAlexander/issues/369#issuecomment-1070448204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYII6V5SFTAQGZ5QZCUPE6LVALOHBANCNFSM5Q43BGEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

an opcode is just like a key, you need a key to unlock a door, here door refers to decrease the latency
you need to config the opcode correctly to make this stuff work.

Does the Chinese server use the same latest version of XixAlexander, with opcode modified as posted in #368 ?

How to solve this issue? Many thanks.

maybe start a new issue is better choice :)

i didnt copy paste the op code, i find it easier to just uninstall and reinstall the new version after i update the thingy! thanks anyway. just to clarify, if the uninstall still wont fix the problem, do i just copy paste the opcode to the Configure->Edit Opcode?


[5.9] Print package-name in .private.swiftinterface only for better abstraction

Print package-name in .private.swiftinterface only for better abstraction
Resolves rdar://107638447

main https://github.com/apple/swift/pull/65336

@swift-ci test

Closing per discussion cc @nkcsgexi


Update mockito dependency to version supported by Spring Boot 2.1.8

The latest Camunda Spring Boot Starter (3.3.5) uses Spring Boot 2.1.8, which in turn defines version 2.23.4 for the mockito dependency. Camunda-bpm-mockito should rely on the same version.

Although this is right for now, Spring Boot 2.2.0 is out and the next camunda starter (expected at the end of this month) should use the dependencies from 2.2.0, shich is 3.1 for mockito (see https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.2-Release-Notes), also #94

I spoke with Simon and we looked on the current camunda-bpm-spring-boot-starter, which is in my opinion the mostly use runtime current now.
The 2.2.0 would be definitely not just a version bump (but an incompatible change). We should probably do the same as with migration from mockito 1 to 2....

@srsp argued that the 4.11 release should use the same versions as the spring boot version that backs the 7.11 based camunda-starter (3.3.4).
My proposal is just a consequent continuation of that rule.
when 7.12 comas out, we will work on 4.12 of this extension and it should then be based on the version numbers of the then actual spring boot version behind the camunda-starter (3.4.0 = 2.2.0.RELEASE).
It's only logical.


test: test $x:xspec-uri variable more

This pull request contains the fix from #1789 (f8019aa2460ae11c6941a17ed5621e457bd12b81) followed by more testing of the $x:xspec-uri predefined variable.
Without the last commit (f5b5b363f56557b8a7bd464e591b01be73e3bd2e), tests fail on Saxon 9.9 because higher-order functions require Saxon 9.9 PE or EE. If you think I should not bother to maintain compatibility with 9.9 because it's deprecated anyway, and if you have an idea for avoiding the failures, let me know.
Follow-Up Tasks
After testing is adequate on the master branch, I will document $x:xspec-uri in the wiki as part of #1771.

Without the last commit (f5b5b36), tests fail on Saxon 9.9 because higher-order functions require Saxon 9.9 PE or EE. If you think I should not bother to maintain compatibility with 9.9 because it's deprecated anyway, and if you have an idea for avoiding the failures, let me know.

Shortly after resolving #852, I will submit a pull request to drop 9.9 support. Additionally, I'll include a reverse commit of f5b5b363f56557b8a7bd464e591b01be73e3bd2e.

Without the last commit (f5b5b36), tests fail on Saxon 9.9 because higher-order functions require Saxon 9.9 PE or EE. If you think I should not bother to maintain compatibility with 9.9 because it's deprecated anyway, and if you have an idea for avoiding the failures, let me know.

Shortly after resolving #852, I will submit a pull request to drop 9.9 support. Additionally, I'll include a reverse commit of f5b5b36.

Sounds like a good plan. Thanks!


resume creating agentpool after gpu-provisioner restarted

Codecov Report
Attention: Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.

Project coverage is 70.52%. Comparing base (6899cba) to head (3af4865).

Files with missing lines
Patch %
Lines

pkg/providers/instance/instance.go
44.44%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #186      +/-   ##
==========================================
- Coverage   71.26%   70.52%   -0.75%
==========================================
  Files           4        4
  Lines         435      441       +6
==========================================
+ Hits          310      311       +1
- Misses        107      112       +5
  Partials       18       18

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


Encoding issues

so， I changed the encoding ，such as

because the Internet problem ,I can not push it


Any chance to enable Apple's FaceID and TouchID

That would make this feature so much more useful.

I can't see this happening to be honest, unless someone else submits a pull request. (No apple devices)


Addon related question [1.12.2]

I'm currently in progress of an addon for the 1.12.2 version, by registering in the intended method like so, registry.register(knifeDiamond = WMItemBuilder.createStandardKnife("knife.diamond", ToolMaterial.DIAMOND)); it will end up with the tools being registered under the weaponmod mod id rather than the addon's.
This will result in a lot of Potentially Dangerous alternative prefix type errors being spammed on the log without a third party mod fixing that and also confusion from players believing that the supported materials were added by this mod. I was wondering if there could be a custom constructor for API purposes or if there's another way to register the tools without that happening?

Oh, good point. I will add factories that consume a mod ID in the next update :)

The new API has been added in all versions < 1.16.5. For everything >= 1.16.5, it is not needed due to the new DeferredRegisters!

I have published 1.22.0 a few hours ago, by the way. I hope that everything works as it should now :)

Yes, I think I have mentioned it in the README, IIRC.
You need to use the MaterialRegistry class for that :)

Oh, very nice to hear that! Sadly, the last few weeks (even months) have been rather stressful for me, so I wasn't able to work as much on Expanded Armory as I would have liked to. In light of that, it is very nice to see stuff like this from the community! :)


[information] facebook app buttons not working

After I re-installed Visual Studio 2017 [Microsoft.com], not all options within online app [www.facebook.com] are available, the browser [Microsoft Edge] traced this to a javascript file.
Microsoft Windows 10
Microsoft Edge

it looks like a white screen that solved 99% of my privacy issues.

https://developercommunity.visualstudio.com is probably a better place to post this since it's not aimed at a documentation issue


Ideally search-by-drug would allow some filtering for drug-centric features

Currently none of the filter options available in the interface allow for criteria of the drugs themselves to be focused.  Mechanism of action relates to the nature of the interaction and is often something closely associated with the drug. But it would be great to eventually incorporate some more key metadata on the drugs that could then be the subject of useful filtering.  For example, FDA approval status, additional broad classes beyond anti-neoplastic, delivery method (e.g. oral delivery only), etc...

We have filters for  Approved, Antineoplastic, and Immunotherapies. Closing this issue.


Integration Test - Release - 2018/07/24 10:50:09

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegartionTestFiles/ChangeSentToAuthor.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add route specific onRequest hook

Hello, thanks for creating redbird.
I'd like to use redbird for our project.  There are two things that prevent us from doing so:

We need an onRequest hook for each route so we can modify req.headers etc.  Similar to haproxy's reqadd/reqdel commands. https://cbonte.github.io/haproxy-dconv/1.7/configuration.html#4-reqadd
There's a bug that redbird is appending / to routes like /test and would get 404 when trying to forward it.

I also update to use prettier to format the code.
Thank you for your time and your consideration.

@manast  Hi, I'd like to get this merged.  Please let me know what other changes I can make to get this accepted.  Thanks.

good job. It is a bit more time consuming to review since you also prettified the code in the same PR, so I need to find the relevant changes from the pure formatting ones.

yup, my bad on the formatting.  Let me reapplying these changes on latest w/o the formatting.

@manast thanks.  closing this one for https://github.com/OptimalBits/redbird/pull/247 that reapply the changes w/o prettifying.
I'd be happy to help you out on reviewing and merging pending PRs etc.  I have a lot more fixes and enhancements that I'd like to get merged back to your version if possible.


pull request with harbor release

cgroup mount option bug is fixed and harbor version is updated.

Hi @marcus-sds,
Thanks a lot for submitting PRs to improve the Harbor bosh release. Please be aware that the harbor bosh release project is moved to the https://github.com/vmware/harbor-boshrelease repo which will be the official version from now on.  If possible, please submit PR to that repo. Thanks.


Results error

Hey
Good tool, if You can fix it, will be great.
Some problems occures:

from gomaps import maps_search
results = maps_search("Tops Diner")
Traceback (most recent call last):
File "", line 1, in
File "/home/xxx/.local/lib/python3.9/site-packages/gomaps/gmapss.py", line 298, in maps_search
return GoogleMapsResults(q, page_num=page_num, delay=delay, log=log)
File "/home/xxx/.local/lib/python3.9/site-packages/gomaps/gmapss.py", line 221, in init
self._place_names = self._get_place_names(self.oq) or [self.__n_a]
File "/home/xxx/.local/lib/python3.9/site-packages/gomaps/gmapss.py", line 255, in _get_place_names
href.group().replace("&", '&').strip('"')
AttributeError: 'NoneType' object has no attribute 'group'

Cheers

Greetings,
I was unable to reproduce this error. However, I have patched this code smell.
pip install gomaps -U to update to latest patch.
Best

Greetings,
I was unable to reproduce this error. However, I have patched this code smell.
Do pip install gomaps -U to update to latest patch.
Best


hyprlock: document float support for layoutxy

Someone asked for support for this and then I realized it already works nicely.

ref https://github.com/hyprwm/hyprlock/pull/591

right? or wrong? I am sick I might be a bit funky

@vaxerski
No this is already how it works. Not related to https://github.com/hyprwm/hyprlock/pull/591.
That PR has no wiki MR yet, wanted to wait for feedback.
If you preferred, I can start to always open wiki mrs in parallel.

ye woudl be nice


add RegistrationIdProvider to change default random string registrati…

…onId.
It would be able to create registrationId from MSISDN or IMEI or something like other unique values. :)
Signed-off-by: Rokwoon Kim<EMAIL_ADDRESS>

I use SourceTree for commit git source and it does not allow to push with force option.
So i just did it on command line. ;)
It seems fixed!


add support for ATMega640, ATMega128x and ATMega256x

This patchset adds support for programming ATMega640, ATMega128x and ATMega256x parts

Fix a bug when calling range() for verification - the chunks appear to be broken into 64K pieces, which may have been masked on parts with smaller memories.
These new parts make use of a "Load Extended Address" instruction, to reach beyond the 17-bit limit imposed otherwise
These new parts have 8-byte / 4-word EEPROM pages
Add definitions for these parts

The "Load Extended Address" instruction is present in the datasheet for the 328p, and I have confirmed that it doesn't have any adverse side-effects when receiving it... I can't make that claim for other parts, but could probably dig out a couple of other parts...
This has been tested against a 328p, and a 2560.

This broke the test test_api_program_memory. Can you take a look?

Ah, yes... apologies - done!

I've reworked this PR somewhat...
Extended Addressing
For all devices upto and including 128k, we don't need to use the extended addressing... the natural reach of the base addressing is 2^17 = 128KiB.
Unfortunately, for the larger parts, it seems that we need to use the "Load Extended Address" instruction... I don't quite understand why, but without it the higher pages don't write properly.
All the parts that I've looked at (including ATtiny85) are documented as supporting this instruction, except the ATtiny26 -
there may of course be others.
Although the ATtiny26 datasheet doesn't explicitly list this instruction, it is either cleanly ignored or handled correctly, meaning that issuing it does not cause an error.
If it would be preferred, then I can add a flag to the database, but I don't believe it's necessary.
"Poll RDY/#BSY" Instruction
I managed to dig out an ATtiny26, which appears to have an older/reduced version of the protocol.
It doesn't claim support for the extended addressing, and it also doesn't support the "Poll RDY/#BSY" Instruction...
This means that erase / program operations either get truncated (during chip erase) or don't complete before we try to move on (during programming), leading to garbage in the memories.
I've added an erase_time field, which allows us to specify a worst-case time for these operations, and allows us to program this device successfully.
Testing
This PR has now been tested on the following devices:

ATtiny26 (small blinky, full 2k + verify)
ATtiny85 (small blinky)
ATmega328p (small blinky)
ATmega1280 (full 128k + verify)
ATmega2560 (full 256k + verify)


Compatibility hint

Can somebody help to get rid of this compatibility hint:
"We have detected that the current wallet seed has been used with another wallet"

Transfer all funds and tokens back to the first address of the wallet (the one Nami shows at “Receive”).
All other wallet apps use multiple addresses (a new one for every change coming back from a transaction, unused addresses proposed as receive addresses). Nami only sees and uses the first address. If it detects funds and tokens on other addresses, it shows the compatibility message.

Transfer all funds and tokens back to the first address of the wallet (the one Nami shows at “Receive”).
All other wallet apps use multiple addresses (a new one for every change coming back from a transaction, unused addresses proposed as receive addresses). Nami only sees and uses the first address. If it detects funds and tokens on other addresses, it shows the compatibility message.

Good morning, wondering if you can help me on the above issue.  I have done what you said but still giving me the warning.  It came up as I used Eternl for the first time.  Any help would be appreciated.


Issue 81 profile page tag components

Change Summary
**Created the main components for the Profile Page tags, closes #81 **
Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

[ ] The pull request title has an issue number
[ ] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

Other Information

Related Issue
Resolves #81
Related issue

Resolve #81

this PR failed the test, could you discuss this matter with your teammate and try to solve this?

Sure Yunho
On Sat, 13 Jul 2024 at 11:07 AM, Yunho Ding @.***>
wrote:

this PR failed the test, could you discuss this matter with your teammate
and try to solve this?
—
Reply to this email directly, view it on GitHub
https://github.com/codersforcauses/penni/pull/116#issuecomment-2226736327,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3PMVCTLWPVEXHXF7HJN5IDZMCKX5AVCNFSM6AAAAABKZASBN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRWG4ZTMMZSG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


debug

Signed-off-by: Itxaka<EMAIL_ADDRESS>

@Itxaka there is an unrelated issue with the QEMU tests which may impact you in building phase. since the luet version in the arm64 tree is really behind, after #1249  was merged building cos images doesn't work as requires 0.31.2 at least. I'm trying to workaround it somehow because It's getting in the way with #1259


Disallow nested numpy arrays

Under certain - currently unknown - circumstances, we are unintentionally creating numpy arrays that contain numpy scalar arrays as values, e.g:
numpy.array([numpy.array(3, dtype=object)], dtype=object)
which doesn't seem to cause problems with our math, but violates the design intent that field arrays should only contain Python ints.  I discovered this while troubleshooting problems with transcript code generation.
We need to track down these situations and clean them up.

Fixed as of f0e2fc766822d1d7a976f0d20a2bf7f3a9f0d542


Check FieldCodec f in FieldCoded.Fields()

If FieldCodec f is nil return nil when FieldCoded.Fields() is called.
Fixes #6294

I'm fine with the fix as long as it's possible that f could be nil.  It feels like there is a check upstream even further we should of had.  @jsternberg might have a better idea of if this is the right fix or not.

its either that or i can check before the for loop
https://github.com/influxdata/influxdb/blob/master/cmd/influx_inspect/info.go#L64
if codec == nil {
    continue
}

I'm fine with the original fix, my only concern is that we aren't hiding a bug somewhere else.  I would like to know that before closing this one is all. This is a valid fix of course, but I don't want it to introduce a silent bug as a result.

I think it would be better to check for this at the place of use rather than inside of the method itself, but I'm not really familiar with this code. @benbjohnson or @jwilder will likely know more than me.

It looks like it's probably looking up a measurement that is in the shard's meta data but doesn't have points in the shard. I'd rather see the continue fix implemented since this error is isolated to influx_inspect:
https://github.com/influxdata/influxdb/pull/6295#issuecomment-209152675

@benbjohnson @corylanou @toddboom once this passes can someone merge it. Its not a big change but it helps. :D

@seiflotfy I resolved the CHANGELOG conflict and merged it in. Thanks!

Also, the build failure is just on the Windows CI which has been flakey on one test.

thanks guys :smile:


[spk] Introspection CLI should have support for cluster sync statuses

As a: user
I want: to query via spk cli the cluster sync status of my deployment
So that: I can verify that my service is running on the most recent manifest commit
Describe the solution you'd like:
Right now, there's support for cluster sync on the dashboard but not in the CLI.

[ ] Add support for cluster sync feature by supporting it in the CLI just like it's supported in the dashboard.

Acceptance Criteria:
User can get cluster sync information without going to dashboard
Describe alternatives you've considered:
User would have to launch the dashboard to get this information.
Additional context:
Does this require updates to documentation?:

From looking at the Spektate code base, cluster sync objects are referred to as ITag objects. We are using a IRepository interface to with a getManifestSyncState method that is concretely implemented in GitHub and AzureDevOpsRepo descendants of IRepository.
Is the idea here to import the Spektate package into SPK and reuse the objects and implementation?
cc @samiyaakhtar @sarath-p @edaena

@andrebriggs yes, this should only require building a command and leveraging the existing logic in the package for getting the results

@samiyaakhtar Is this something we can pull into the next 2 week sprint?

@andrebriggs yes, we could, along with https://github.com/microsoft/bedrock/issues/840

@samiyaakhtar do you still think this is a medium effort task?

@andrebriggs updated it to small


KNXGlobalContext_READ variable number of objects and problem in finding a specific group Address

Hi Massimo,
thank you for bringing to the Node-red community the fabulous KNXUltimate node. I found some issues regarding the behaviour of KNXGlobalContext_READ
I expected that KNXGlobalContext_READ contained all group addresses of the ETS Project. I have some 68 GA in my projects, but observing after some time the global variable from NR Global Context Data  I see that the number of objects  is lower, i.e. 10-20, not constant and varies from time to time.
The problem is that, if you want to find a GA not in KNXGlobalContext_READ, the node will not send any object:
let GroupAddresses = global.get("KNXContextProserv_READ") || [];
let ga = GroupAddresses.find(a => a.address === "3/1/10");
node.send(ga)
no ga object is sent if "3/1/10" is not present in KNXContextProserv_READ
After working sometime using  the above code in a node suddenly KNXContextProserv_READ got all its 68 GA and the ga 3/1/10 was found. What it's still unclear is why for some time the number of items in  KNXContextProserv_READ was so low?
KNXUltimate v. 1.4.6
NR v. 3.0.2

Hi Roberto.
This is the expected behavior. The global context node collects the values that are read from bus. Until the group address doesn’t transit over the bus, the node is unaware of it.
You’ve imported the ETS file, so you’re expecting all group addresses to be present, but they are unuseful if the respective values aren’t present.
You could send a read request to the bus, whenever nodered starts, using a knx node set in universal mode.

Ok Massimo, thank you for the prompt reply.
I wrongly assumed that the GlobalContex node would somehow force the reading of all addresses in ETS import file, but it’s not the case. I will take care of needed ga forcing a readstatus from the device node set as universal.
Ciao
Roberto
Inviato da iPad

Il giorno 4 nov 2022, alle ore 19:47, Massimo Saccani @.***> ha scritto:

Hi Roberto.
This is the expected behavior. The global context node collects the values that are read from bus. Until the group address doesn’t transit over the bus, the node is unaware of it.
You’ve imported the ETS file, so you’re expecting all group addresses to be present, but they are unuseful if the respective values aren’t present.
You could send a read request to the bus, whenever nodered starts, using a knx node set in universal mode.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


[Explorer] Generate SQL failed for question bf376637-9918-472d-978d-c1212291ea2a: the most active reviewer last year, excluding robot and bot

Hi, I have some problems with the question bf376637-9918-472d-978d-c1212291ea2a the most active reviewer last year, excluding robot and bot (errorType = error-validate-sql):

Error message
DB::Exception: Check res.find(executor_id) == res.end() failed

It seems a runtime error from TiFlash
https://github.com/pingcap/tiflash/blob/e665e59aef0972ca8af05ae0cc5c750e051c06f4/dbms/src/Flash/Statistics/ExecutorStatisticsCollector.h#L58


[FAQ] The Relationship and Usage of r Prefix String and String Interpolation

Hey, @Peefy could you please provide me with any example or piece of knowledge over this just to sort this issue quickly

A: In KCL, we can use ${...} to interpolate variables e.g., "hello ${world}", but sometimes we don't really want to escape it. In this case, we can use the r prefix to ignore escaping e.g., r"hello ${world}".

@Peefy, PR made. 😊

Closed by #292


Crash on app start

Hello, I tried installing the app, but as soon as I open it (or a few seconds later) it crashes. I checked the log file and it seems like it's flagging a drive as a floppy drive due to the letter, and can't find it
debug.txt

It frequently crashes for me after 2-4 clicks into folders to get to a file. Happens on both my machines (both running Windows 2004, though it was also happening before either got updated). It's not consistent and I haven't found an underlying pattern
, but happens enough where it mostly discourages me from using the app unfortunately. Log file attached if that's helpful.
debug.txt

Hello, I tried installing the app, but as soon as I open it (or a few seconds later) it crashes. I checked the log file and it seems like it's flagging a drive as a floppy drive due to the letter, and can't find it

I think, you have a hidden drive, and app can't recognize it.

Hello, I tried installing the app, but as soon as I open it (or a few seconds later) it crashes. I checked the log file and it seems like it's flagging a drive as a floppy drive due to the letter, and can't find it

I think, you have a hidden drive, and app can't recognize it.

Funnily enough, the only hidden drive is the EFI partition

@NextinMono Is this still an issue in v0.21?

@NextinMono Is this still an issue in v0.21?

I've tried, it works now!


Node doesn't understand Windows' local file URL's

Thank you very much :+1:
Could you please fix the ESLint errors?

That should do it.
:-)

Thank you. Merged :sunglasses:

Thanks!
:+1:

Hi @ebpa,
I tried to update some functionality but unfortunately I see now that your pull request destroyed the bundle build on Linux / Mac OS X. Your regex in your function parseUnescape() strips the leading slash so paths like /Users/me/foo/bar become Users/me/foo/bar. So the files can not be imported / loaded anymore.
I try to fix this but I am not the regex guru :confused:

I removed (commented out) windows support temporarily in dd1d30e8975e54997d2f027e95be2cd9ce25cd38


Image not showing up in modal

Image is not showing because ekko-lightbox-container class doesn't has any width associated with it.
Adding width to ekko-lightbox-container class gives me the image.

Nothing wrong with the plugin. Actually problem was one of my css messed up with modal.


AI-Powered Recipe Finder

I've added AI-Powered Recipe Finder.
issue no. : #1688
https://github.com/user-attachments/assets/b0a42c2c-b005-4598-8717-24c0920d1708

@shriyadindi resolve the conflict

@iamrahulmahato Resolved the conflict!
Please do add the gssoc, hacktoberfest and the level badge.


Research & Develop Procom Driver

Platform: Procom (Talent Network)

Popularity of the platform: Around 500 jobs available
URL: https://portal.procomservices.com/jobs

Additional Info: A local career platform with a very simple, repetitive application form. Should be easily automated.
Expected Behavior:

Fetch jobs: Search for jobs based on criteria and save data.
Apply to jobs: Automatically fill out application forms and submit.

Hey @webgtx, I want to be assigned to this issue


Test release builds on Travis

Profiling locally shows a 3x speed improvement by testing release builds,
presumably due to the amount of wasm processing happening! Let's see what
happens on Travis and if we get a similar speedup.

Looks like 17 minutes compared to 15 minutes, so not as big of a gain on Travis!


Scroll behaviour in 1.0

Since upgrading to 1.0 the browser stopped scrolling to the top on a page change. I can't find anything about it in the upgrade guide or the docs, except for the link 'ScrollManagmentMixin' in the documentation, but that points to a 404 page :-(

https://github.com/rackt/react-router/blob/f61724c011d73454c5ac8ce5507709d458eb1828/UPGRADE_GUIDE.md#scrolling


Kata/Katas: defining the plural to use in the docs

I noticed I use "Katas" while hobovsky use "Kata" as plural. Unnamed already talked about that too.
We should always use the same way throughout the whole documentation, so which one will it be?

Kata
Japanese words are not inflected for plural and it’s awkward to see as a Japanese. So I’d prefer kata, kumite, kyu, and dan not inflected in the documentation.

Done.