return roles=null for api keys with user-level permissions

@btel if you have a sec for review, that would be great :)

@btel if you have a sec for review, that would be great :)

Codecov Report

Merging #309 (4ad5cb4) into master (304d17c) will increase coverage by 0.88%.
The diff coverage is 86.95%.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
+ Coverage   84.60%   85.48%   +0.88%
==========================================
  Files          55       56       +1
  Lines        4377     4390      +13
==========================================
+ Hits         3703     3753      +50
+ Misses        674      637      -37

Impacted Files
Coverage Δ

quetz/main.py
87.97% <83.33%> (-0.10%)
:arrow_down:

quetz/dao.py
92.68% <87.50%> (+0.05%)
:arrow_up:

quetz/rest_models.py
99.37% <100.00%> (ø)

...s/versions/53f81aba78ce_use_biginteger_for_size.py
56.25% <0.00%> (-18.75%)
:arrow_down:

...ns/versions/30241b33d849_add_task_pending_state.py
80.00% <0.00%> (-13.34%)
:arrow_down:

...ons/a3ffa287d074_case_insensitive_channel_names.py
68.75% <0.00%> (-6.25%)
:arrow_down:

...ions/versions/8dfb7c4bfbd7_new_package_versions.py
82.35% <0.00%> (-5.89%)
:arrow_down:

...s/versions/8d1e9a9e0b1f_adding_download_metrics.py
72.22% <0.00%> (-5.56%)
:arrow_down:

quetz/database_extensions.py
56.36% <0.00%> (-3.64%)
:arrow_down:

quetz/testing/fixtures.py
97.01% <0.00%> (-2.23%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #309 (4ad5cb4) into master (304d17c) will increase coverage by 0.88%.
The diff coverage is 86.95%.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
+ Coverage   84.60%   85.48%   +0.88%
==========================================
  Files          55       56       +1
  Lines        4377     4390      +13
==========================================
+ Hits         3703     3753      +50
+ Misses        674      637      -37

Impacted Files
Coverage Δ

quetz/main.py
87.97% <83.33%> (-0.10%)
:arrow_down:

quetz/dao.py
92.68% <87.50%> (+0.05%)
:arrow_up:

quetz/rest_models.py
99.37% <100.00%> (ø)

...s/versions/53f81aba78ce_use_biginteger_for_size.py
56.25% <0.00%> (-18.75%)
:arrow_down:

...ns/versions/30241b33d849_add_task_pending_state.py
80.00% <0.00%> (-13.34%)
:arrow_down:

...ons/a3ffa287d074_case_insensitive_channel_names.py
68.75% <0.00%> (-6.25%)
:arrow_down:

...ions/versions/8dfb7c4bfbd7_new_package_versions.py
82.35% <0.00%> (-5.89%)
:arrow_down:

...s/versions/8d1e9a9e0b1f_adding_download_metrics.py
72.22% <0.00%> (-5.56%)
:arrow_down:

quetz/database_extensions.py
56.36% <0.00%> (-3.64%)
:arrow_down:

quetz/testing/fixtures.py
97.01% <0.00%> (-2.23%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


svg-transform-loader stopped honoring -svg-height after upgrade from 1.0.2 to 1.2.1

I've noticed that after upgrading svg-transform-loader from 1.0.2 to 1.2.1 SVGs have wrong sizes.
I'm using svg-transform-loader with postcss-loader (3.0.0) and postcss-move-props-to-bg-image-query (0.2.5).
Here's a relevant part of my webpack.config.js:
{
  test: /\.svg(\?.*)?$/,
  use: [
    'url-loader',
    'svg-transform-loader'
  ]
},

postcss.config.js:
module.exports = {
  plugins: {
    'autoprefixer': {},
    'postcss-move-props-to-bg-image-query': {},
    'cssnano': {},
  }
};

and here's example CSS (using @mdi/svg):
.demo {
  height: 100px;
  background: #ffffe0 center no-repeat;
  background-image: url("~@mdi/svg/svg/alert.svg");
  -svg-fill: #ff0000;
  -svg-width: 50px;
  -svg-height: 50px;
}

Now, with 1.2.1 fill is applied but width and height are ignored. If I downgrade to 1.0.2, width and height are correct again.
What other info/samples would be helpful here?

@iwaszko could you please provide the alert.svg content?

Sure, please find it below. On the other hand, this was just a demonstration as the problem seems to happen for all SVGs from https://www.npmjs.com/package/@mdi/svg that I've used.
<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd"><svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1" width="24" height="24" viewBox="0 0 24 24"><path d="M13,14H11V10H13M13,18H11V16H13M1,21H23L12,2L1,21Z" /></svg>

Ah, it was changing of default behaviour of posthtml-transform. Okay, I've published new major version 2.0.0 of svg-transform-loader which processes all nodes, including  tag. As it behaves so, you should update you code in this way:
.demo {
  background-image: url("~@mdi/svg/svg/alert.svg");
  -svg-fill: #ff0000;

  /* you should specify selector to transform as second value to process only svg */
  -svg-width: 50px svg;

  /* and here too */
  -svg-height: 50px svg;
}

@iwaszko by the way, why you're not using background-size prop instead of transforming entire SVG? :)

Thanks a lot, that was fast! And that's a fair question too... ;)


need some help with save editor.

So my problem is when I open Save editor and go to appearance I can edit it just fine and so forth, but I can't actually see the changes in the save editor itself i am making the changes and then starting the game to see what it does. I looked at videos on youtube and see that others see the images of the character in appearance in save editor but mine does not. So I was wondering what I might have done wrong since all the changes I make actually do happen in game, but I just can't see the actual changes in the save editor itself. if someone can help please let me know.

Appearance previews are in a separate download. (So that they don't need to be re-downloaded with every revision.)


Add PP3 - HOMER SIMPSON VS PETER GRIFFIN

Song Name
-API- Paradise on E [REMIX]
Artist Name
apimusic
Source
Direct file download
Youtube Link
No response
Direct File Link
https://i.supa.codes/BzlSS
Song ID
32772
Start Offset [ms]
0
Extra
Used in the level Plasma Pulse III - 20159880

https://i.supa.codes/BzlSS

Is this your cdn? We need permissions to put this in auto nong from the owner of the url
Link isn't working


update http to https for documentation tutorial images

Updated examples/tutorials files from http to https to work with docs output. Testing output with this PR...

For [Docs issue#856)(https://github.com/CartoDB/docs/issues/856), problems with github latency...

retest this please

retest this please


Added issue label api

Hi,
During using of this library I found missed API's:

createLabel,
updateLabel
deleteLabel

This endpoint's need to create a custom label for repository, for example, specify color.
For this reason I must to change domain of Label, this changes a breaking, so maybe it's reasonable to create other model like  LabelData.

Hi @juanpedromoreno,
Thx, sure will do today.

Ooops the CreateLabel integration test failed :( , I forgot running the integration tests (manually), as they are not running in forks

https://github.com/47degrees/github4s/pull/550 should fix the test :)

Oops, sorry, forget to change the status.
Thanks you for great library.


Bug Fixed: Navbar items Overflowing

Description
Updated the Navbar to not allow overflowing for normal screen size, which means navbar items will be inline, making it look appealing and smooth.
Fixes:  #690
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have made this from my own
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Screenshots

heyy @huamanraj
Made the PR for my issue. Please review and merge it.
Thanks a lot for this opportunity,
Aarush

heyy @huamanraj
Made the PR for my issue. Please review and merge it.
Thanks a lot for this opportunity,
Aarush

hello @aarushroyy there are some conflicts resolve that!

Conflicting files
assets/css/style.css

Solve this conflicitng issues & raise PR again


fix: Array#shuffle(!) result distribution

previously the shuffle algorithm would leave the first element untouched about 1/3rd of the time

I can mathematically confirm these changes are correct- there are "n!" ways to order a list of "n" elements, however the original functionality would randomly pick one of "n^n" elements.
This necessarily would produce lopsided behavior, as n^n is not an integer multiple of n!, except in the case where n=0, 1, or 2 (where n^n is (effectively) 0, 1 and 4 respectively, and n! is (effectively) 0, 1 and 2 respectively), in all other cases, the "n is never divisible by n-1" rule comes into play and makes some permutations appear more often than others.
This appears to manifest as the 0th element of an array moving around significantly less often than other elements, which this PR fixes.

Thank you.


Load order error causing spec failures

System Info
 ❯ uname -a                                                                                [12:40:26]
Darwin Matthews-MacBook-Pro.local 14.5.0 Darwin Kernel Version 14.5.0: Wed Jul 29 02:26:53 PDT 2015; root:xnu-2782.40.9~1/RELEASE_X86_64 x86_64

Rails 3.2.22
Ruby 2.1.7
Zeus 0.15.4
Description of the Issue
Running bundle exec rspec spec allows me to run my test suite just fine.
However, when I try to run with Zeus, I get a load order issue:
 ❯ zeus rspec spec                                                                                [12:35:40]
/Users/mec/Sites/zipmark/service/spec/lib/rack/report_request_metrics_spec.rb:6: warning: already initialized constant App
/Users/mec/Sites/zipmark/service/spec/lib/rack/content_type_default_spec.rb:6: warning: previous definition of App was here
/Users/mec/Sites/zipmark/service/spec/services/vendor_integrations_creation_service_spec.rb:3:in `<top (required)>': uninitialized constant VendorIntegrationsCreationService (NameError)
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus/load_tracking.rb:50:in `load'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus/load_tracking.rb:50:in `load'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus/load_tracking.rb:43:in `load'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/rspec-core-2.11.1/lib/rspec/core/configuration.rb:780:in `block in load_spec_files'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/rspec-core-2.11.1/lib/rspec/core/configuration.rb:780:in `map'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/rspec-core-2.11.1/lib/rspec/core/configuration.rb:780:in `load_spec_files'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/rspec-core-2.11.1/lib/rspec/core/command_line.rb:22:in `run'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/rspec-core-2.11.1/lib/rspec/core/runner.rb:69:in `run'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus/rails.rb:216:in `test'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus.rb:148:in `block in command'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus.rb:135:in `fork'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus.rb:135:in `command'
        from /Users/mec/.rbenv/versions/2.1.7/lib/ruby/gems/2.1.0/gems/zeus-0.15.4/lib/zeus.rb:50:in `go'
        from -e:1:in `<main>'

To try and fix this, I directly require'ed the uninitialized class's file in the complaining spec:
require 'spec_helper'
require 'vendor_integrations_create_service'  # <--- added this line

describe VendorIntegrationsCreationService do

While that solves this particular load order issue, I now end up with lots of failing specs - they seem to all be failing because my various classes which rely on ActiveRecord::Base don't know about their superclass:
  2272) UserUpdateService ignoring keys should call developer interest service with the new params
     Failure/Error: let(:user) { User.create(:email =><EMAIL_ADDRESS>}
     NoMethodError:
       undefined method `create' for User:Class
     # ./spec/services/user_update_service_spec.rb:4:in `block (2 levels) in <top (required)>'
     # ./spec/services/user_update_service_spec.rb:173:in `block (3 levels) in <top (required)>'
     # -e:1:in `<main>'

Theory about what's causing this: models' superclasses are not right!
Not sure how, but my User class is not right when I run the tests via Zeus (the output below was generated using binding.pry in spec/models/user_spec.rb as part of the command zeus rspec spec/models_user_spec.rb:
[2] pry(#<RSpec::Core::ExampleGroup::Nested_1::Nested_27>)> defined? ActiveRecord::Base
=> "constant"
[3] pry(#<RSpec::Core::ExampleGroup::Nested_1::Nested_27>)> defined? User
=> "constant"
[4] pry(#<RSpec::Core::ExampleGroup::Nested_1::Nested_27>)> User.superclass
=> Object

Since, for example, my User class doesn't know about it's class-level methods like .create and .transaction, I suspect that this is a load order issue in which Zeus is executing the spec files before pulling in all the Rails dependencies like ActiveRecord::Base.
Does anyone have experience with an issue like this by chance? Please help!

Zeus does not do anything magical as far as

pulling in all the Rails dependencies

Zeus pre-loads your application by requiring application.rb just the same way that booting up a rails server/console/rake task would. There's no way for me to debug this issue without seeing an app that reproduces it.
I suspect the root of your issues is that your spec helper is not loading the full Rails env.

OK thank you @latortuga, I will check if full Rails environment gets loaded in spec_helper.rb and report back. Thanks again :smile:

@latortuga you were right! Turns out, we have a spec/support/stubs.rb file which was stubbing out User. Now to figure out why this breaks with Zeus behind the wheel but not with the normal rspec executable...

I'm sorry but I can't debug code I can't see. Trying to make a simple example app that exhibits the problem may lead you to a solution.


VBA converts from locale dependant decimal separator to point

Hello,
This is a suggestion to handle writing decimal numbers using VBA in a locale independent way to correct issue #1. I commited only the VBA file.
Best regards,
Leonardo

Thank you for providing this fix for users outside of the US.
Regards,
Erik


Sometimes the integration tests fail

  Integration
    python-spake2
      Generates the same SPAKE2 session key (symmetric):                 OK (0.12s)
      Generates the same SPAKE2 session key (asymmetric, we are side B): OK (0.10s)
      Generates the same SPAKE2 session key (asymmetric, we are side A): FAIL (0.10s)
        expected: "840e135b2d8a621c84a8d2e074f87af2c308d5e563e1a2eb32109b738fd65671"
         but got: "be0505bfd32cf55f9745b0d6be1104bf86b3c2dea04d6c9e17cdd229edce04f6"

On my system (x86_64-linux, using tools from the flake-supplied devShell).  it's intermittent.  It's not always the "side A" case.  I've seen all three fail.  The frequency of some failure is perhaps 1 run in 20.

Also, the expected and actual values aren't always the same.  For example, on a later run where "side A" failed:
      Generates the same SPAKE2 session key (asymmetric, we are side A): FAIL (0.10s)
        expected: "1d3084932a5993f4012f13f0afb87c6a1731a5be14b927168626f2efd0714af8"
         but got: "de367397b6fd6f8fa8609085325144d42bb30e4a9d325f1c58350523acd589a0"

The Ed25519 elements used by the tests are always the same.  src/Crypto/Spake2/Groups/Ed25519.hs defines an AbelianGroup instance for Ed25519 and the generateElement function of that class is used to generate a random keypair for the Haskell side of the exchange.  If I steal a randomly generated value from this function from a failed test run and hard-code it so it is always the generated element, it does not make the test failure any more deterministic.  This seems to suggest the failure comes entirely from the Haskell interpretation of the blinded keypair received from the Python implementation.

It's also possible it's the python-spake2 side that is producing the incorrect (or at least different) shared secret, right?

Yes, indeed.  I got as far as thinking about that and then decided not to go further on this today.


Option to reverse %n naming based on Sketch's logic

When duplicating layers, they move up in the sketch layer tree so to speak.
User must then move the layers in reverse order so that the rename plug will properly order with %N.
Could we have an option to reverse that naming so the top most layer isn't #1 it's actually the last number in the list?

Step 1: Duplicating an artboard 7 times moves the layers upwards on the layers list.

Step 2: Then when using %n re-name it, the artboards are reversed.

Desired result:

Has this been solved?

Not to my knowledge!

Lowercase %n rename the layers ascending order.
Uppercase %N rename the layers in descending order.


[WIP] Probabilistic two-stage detection

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
This is sota for detection in 2021.
Modification
Added a fuxian.py file that describes the paper I want to reproduce.
Checklist

only test for Qualification registration.

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


Update README.md

Proposed notice about difference between Android and iOS functionality. I learned this the hard way...

Thank you for the clarifiaction. I´ll merge it in.


Add 2nd slide file

Add 2nd slide file
This pull request is intended to merge the changes made in adding the 2nd file to the slide project.

Merged the 2nd slide into the main branch.


Support for replay

It would be nice to have a way of repeating a request, like chrome dev tools' "Replay XHR".

That's definitely a good idea, but meanwhile you can use "Copy as cURL" feature to replay requests from your terminal.


Bot not merging, is it not running anymore?

Is the bot down? Would love a quick message if this is decommissioned.

Yes, it looked like it did not pick up the PR you just submitted. I'll restart it when I get a chance but that will be early next week.
Thanks for the report.

Accidentally started the bot with the wrong user at one point. Fixed permissions and merged in #237. Thanks.


idle connection close

[REQUIRED] Step 2: Describe your environment

Operating System version: ubuntu
Firebase SDK version: firebase-admin-java 9.4.0
Library version: jdk 17
Firebase Product: fcm (auth, database, storage, etc)

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
I am testing with firebase-admin-java 9.4.0.
Since communication is via HTTP/2, I expected the connection to remain open if there were no issues. However, after making a request, if there are no additional requests, the connection is dropped after about 3 minutes.
The strange part is that the entity sending the FIN packet is not the client, but the FCM server.
Is there any specific reason why the server is sending a FIN packet and closing the connection, even though it's using HTTP/2?

https://github.com/firebase/firebase-admin-java/issues/1028#issuecomment-2437267762


Extending the Mobile Elements

The project looks quite promising; I'm hoping that it is eventually made part of an official API.
After looking at the source code for the various elements I realized that only a few of the many properties for the Xamarin.Forms elements are exposed, and due to the fact that the underlying native control is not itself exposed as a Blazor parameter, there is no way to set those properties.
In my case I was attempting to use the FlowDirection property to use RTL functionality (I work on some Hebrew language apps from time to time).
I suppose I could write my own components and expose the FlowDirection property as a parameter myself, but I wonder if there is a "better" way to do this.
BTW, I noticed that someone else complained about the error messages indicating ambiguity between the mobile blazor binding elements and the Xamarin Forms ones.  I simply substituted the line @using XF = Xamarin.Forms in _Imports.razor for the @using Xamarin.Forms and then applied the prefix XF. to those items that derive from the forms classes.  It's not a "perfect" solution, but it eliminated the error messages and didn't require a lot of extra text in the razor files.  I found that I could specify the @code{} segment of my files in partial classes to better separate the UI and logic using this technique quite easily.

Hi @rlebowitz most missing properties have been added now, as well as several missing controls. You can build the source of this repo to see the latest, or otherwise we plan to release an update to nuget.org very soon that will have these fixes.
We have a little bit of info on how to write custom components (which can involve extending a built-in component) here: https://docs.microsoft.com/mobile-blazor-bindings/advanced/custom-components
So if you found another missing property on a built-in components, first, let us know here, but you can create a derived component that adds the missing property by following the guidance above.


Look at moving the base image for Docker to an Ubuntu based one to more closely match a production environment.

@dostler, if you can get me a list of packages that you install in most cases, I can add those to such a base image.


[SelectField/Performance] Select field is slow when it contains 100+ items

Problem description
Select field is remarkably slow with for example 179 items inside. It opens and closes slowly.
Is there anything that I could do to make it faster?
Link to minimally-working code that reproduces the issue
http://www.webpackbin.com/VJ5Yv0MWM
Versions

Material-UI: 0.16.1
React: 15.3.2
Browser: 54.0.2840.87 (64-bit)

@alekbarszczewski Thanks for the feedback. I'm closing it for #2787.


Create a binary crate for the executor

Implemented in https://github.com/logos-co/nomos-node/pull/766


test: cleanup CNI network used by the tests

issue introduced with:
https://github.com/containers/libpod/pull/1871
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

/approve
LGTM

/lgtm
On Tue, Nov 27, 2018, 09:11 OpenShift CI Robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rhatdan
https://github.com/containers/libpod/pull/1878#issuecomment-442071682
The full list of commands accepted by this bot can be found here
https://go.k8s.io/bot-commands.
The pull request process is described here
https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process
Needs approval from an approver in each of these files:

OWNERS https://github.com/containers/libpod/blob/master/OWNERS
[rhatdan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/1878#issuecomment-442071699,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHYHCI67MOkN-vuOFz-oTGgYKzANtfLFks5uzUgjgaJpZM4Y1j31
.

/lgtm

bot, retest this please

bot, retest this please

@mheon @rhatdan @baude finally tests are passing

/lgtm


[SPARK-40384][INFRA] Only do base image real in time build when infra dockerfile is changed

What changes were proposed in this pull request?
Do base image real in time build only when infra dockerfile is changed.
Why are the changes needed?
Recently github ghcr is unstable, so we better use infra static image when no infra dockerfile changes.
After this change even github ghcr image write has some issue, only infra dockerfile PR is blocked, other PR don't have any impact.
Does this PR introduce any user-facing change?
No
How was this patch tested?

CI passed, and use the ghcr.io/apache/spark/apache-spark-github-action-image-cache:master-static.
local test infra changes trigger CI: https://github.com/Yikun/spark/pull/170
test ./dev/is-changed.py -m infra-image

# previous hash of dockerfile is changed
$ export APACHE_SPARK_REF=871152bda69a5d5db714eecd42fdd3e7d2e04557
$ ./dev/is-changed.py -m infra-image
true
# hash of dockerfile is changed
$ export APACHE_SPARK_REF=0830575100c1bcbc89d98a6859fe3b8d46ca2e6e
$ ./dev/is-changed.py -m infra-image
false
# next hash  of dockerfile is changed
$ export APACHE_SPARK_REF=e6c58c1bd6f64ebfb337348fa6132c0b230dc932
$ ./dev/is-changed.py -m infra-image
false

cc @HyukjinKwon

@HyukjinKwon Thanks, I will revisit today, and mark it ready for review.

Ready to go

Merged to master.

Test on only dockerfile changes: base trigger and only lint job triggered based on base image, need to enable pyspark/sparkr in followup https://github.com/apache/spark/pull/37865.
Test on apache/spark merge commits: base skip and use static-master directly in sparkr, lint and pyspark
Test on root dockerfile changes: https://github.com/Yikun/spark/pull/172
Test of schedule job: TBD
Test on branch job (3.2 or 3.3): TBD

Offline discussed with @Yikun. Let's revert it out for now (since this isn't a problem if ghcr isn't broken, and to make our CI implementation simple)

@HyukjinKwon Thanks!


Add "rebalance account" function to conviction voting pallet

A user can participate in votes and also participate in staking.
While staking, a user can be slashed, and thus their balance can decrease.
However, the conviction voting pallet will not know that a users balance has decreased, and will still give them voting representation of their old balance.
There should be a simple, permissionless call which allows anyone to be able to ping a user account, and the system will check their latest balance, and rebalance the users votes appropriately.

I will be working on this issue @shawntabrizi

Possible duplicate of #10761

haha yes, a duplicate indeed


Add -Wait Switch parameters to the New-CaasVlan and New-CaasNetworkDomain

cc @manishkp @BenObermayer

hold the phone. this needs refactoring

ok please review @manishkp
p.s. @tintoy can you recommend a better way of having a shared "wait for thing to complete" base class/interface without having to use generics in this crazy way. This implementation feels horrible.

@tonybaloney yeah, just define an interface with required properties common to all entity data-contracts. Look at the DMS powershell Cmdlets for an example of how to do this.

@tonybaloney or check out how I did it in the Go client.

All the progress comes as public ProgressType from the API, we treat them the same, and have logic behind it.
We can add the wait logic in the base class.

sadly VlanType and NetworkDomainType don't share a common base class, so even though they both have a state field, that's just a convention. Our types are all auto-documented from the XSD, so any changes I make will be overridden in the next API update.

@tintoy @manishkp this is how I implemented it
https://github.com/tonybaloney/DimensionData.ComputeClient/blob/34641c830814b9850f2aca02ca1666ddeb47c3a8/PowershellModule/CloudCompute.Powershell/Contracts/WaitableCmdlet.cs
And an example implementation:
https://github.com/tonybaloney/DimensionData.ComputeClient/blob/34641c830814b9850f2aca02ca1666ddeb47c3a8/PowershellModule/CloudCompute.Powershell/Mcp20/DeployCaasNetworkDomainCmdlet.cs#L103-L113

That's why I used an interface :)

But you can't put the SwitchParameter field in an interface? Nor can the implementations share any kind of repeat-catch-fail logic.
You cheated :-)
Get Outlook for iOS
On Fri, Aug 5, 2016 at 11:14 AM +1000, "Adam Friedman"<EMAIL_ADDRESS>wrote:
That's why I used an interface :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hard-coded property on interface returns enum.

@tonybaloney I've emailed you a couple of links to TFS so you can see how we did it for the Aperture powershell module (Identity or DMS are good examples).

ok fine that solution is much better :-)

I have changed the yml file, so the build should not fail for pull requests

@manishkp ok i rebased off develop and also implemented that logic (will kinda but the closest without having tons of nested if statemets)


Ignore coverage for unknown package version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
1.1%

Covered Lines:
133

Relevant Lines:
133

💛  - Coveralls


Is there a schedule for code release？

估计还早...

Thanks for your attention. We will release codes before June. Please stay tuned.

@cshizhe When the code will be released?

@cshizhe Could you release your codes?

@cshizhe Could you release your codes?

I am preparing the codes, data and instructions and will release them this week.

@cshizhe Could you release your codes?

I am preparing the codes, data and instructions and will release them this week.

Thank you very much!

Hi all,
I have released our codes and instructions. The datasets will also be released soon, which are delayed a little bit due to network issues.

Hi cshizhe,
The link of features is invalid now, could you check it?


Enhancement: Boss laser limb does not have "charge up" in context menu

Need to add a context menu item for "charge up" so that a sound fx can be triggered when the boss starts charging up the laser

Fixed with #118


estree_ast::flavor evaluates eager to the default

Describe the bug
Not sure if this is a bug or simple misconfiguration: When there's a property requires evaluation to flavor, it doesn't seem to be applied. See below examples.
Input code
let cm = std::sync::Arc::new(SourceMap::new(FilePathMapping::empty()));
let fm = cm.new_source_file_from(FileName::Anon, std::sync::Arc::new("foo".to_string()));
let comments = swc_node_comments::SwcComments::default();

let context = swc_estree_compat::babelify::Context { cm, fm, comments };

let id = Expr::Ident(Ident::new("foo".into(), DUMMY_SP));
let flavor = swc_estree_ast::flavor::Flavor::Acorn {
    extra_comments: false,
};

// Here, run `babelify` inside of custom flavor context
let e: swc_estree_ast::Expression = flavor.with(|| id.babelify(&context).into());

e.serialize(&Serializer::json_compatible().serialize_missing_as_null(false))
    .expect("")

Config
No response
Playground link
https://github.com/kwonoj/test_flavor
Expected behavior
// Expect skip_none_and_false and skip_none_and_false applied as it matches to the flavor Acorn:
{
  type: 'Identifier',
  start: undefined,
  end: undefined,
  loc: undefined,
  name: 'foo',
  //decorators: undefined,
  //optional: false,
  //typeAnnotation: undefined
}

Actual behavior
// Properties are still serialized

{
  ...
  decorators: undefined,
  optional: false,
  typeAnnotation: undefined
}

Version
latest
Additional context

Possible misconfiguration
or wasm plays some weird to scoped_tls?

You should serialize as json within with

Cool, thx for the clarification.


Bot Framework Composer server start is looping (unexpected end of JSON)

Describe the bug
You can see "Server starting" screen after that nothing happens. Opening the Bot Framework Composer with terminal only shows the Syntaxerror : "UnhandledPromiseRejectionWarning: SyntaxError: Unexpected end of JSON input".

Version

Bot Framework Composer 2.1.3-nightly.364675.1a93f21
node.js 14.17.3 LTS
npm 9.6.5

Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Download Bot Framework Composer nightly build
Start Bot Framework Composer

(there are no more information to reproduce)
Expected behavior
Bot Framework Composer should show all Bots.

Screenshots

Additional context
(node:11276) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at migrateStore (C:\ProgramData\Bot Framework Composer\resources\app.asar\node_modules\@bfc\server\build\store\store.js:17:31)
    at Object.<anonymous> (C:\ProgramData\Bot Framework Composer\resources\app.asar\node_modules\@bfc\server\build\store\store.js:22:1)
    at Module._compile (internal/modules/cjs/loader.js:1130:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1160:10)
    at Module.load (internal/modules/cjs/loader.js:987:32)
    at Module._load (internal/modules/cjs/loader.js:828:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Module.require (internal/modules/cjs/loader.js:1011:19)
    at require (internal/modules/cjs/helpers.js:88:18)
(Use `Bot Framework Composer --trace-warnings ...` to show where the warning was created)
(node:11276) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:11276) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
[16124:1109/113309.015:ERROR:gpu_init.cc(441)] Passthrough is not supported, GL is disabled

@neis-lotte Is this still happening for you? I am not able to reproduce the issue.

@neis-lotte try uninstalling both, "Bot Framework Composer" and "Bot Framework Composer Test", then install the fresh Nightly from here and see if it helps.
The default location for the nightly is C:\Users\%USER%\AppData\Local\Programs\Bot Framework Composer Test and from the screenshots provided it seems you're running regular composer build.

@neis-lotte try moving the file located in C:\Users\%USER%\AppData\Roaming\BotFrameworkComposer\data.json where %USER% is your windows computer username, somewhere else and see if it fixes the issue.
Looks like the file got corrupted which prevents Composer from start

So my Pc got a shotdown while running the program, after that it did not work. Now after resetting my PC everything works fine. But Thank You, Next Time I'll try.


Random Color Generator

Project title
(Write project title over here)
Random Color Generator
Project Description
(Describe the project over here)
I would like to create a Random Color Generator using Html , Css and Javascript. It will generate random colors when users clicks on the generate button .Please assign it to me.
Doc Domain

[] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ?

@Tusharjoshi1703  start workijg on this .

Are u still working on this, if not please close this issue.

Yes ,by tomorrow I'll raise the Pr.Sorry for the inconvenience
On Sun, 30 Jul, 2023, 21:14 Vikash Yadav, @.***> wrote:

Are u still working on this, if not please close this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Vikash-8090-Yadav/Future.WebNet/issues/358#issuecomment-1657204934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZJSIUBO4QK5DGISCS3U7DXSZ6PLANCNFSM6AAAAAAZRW2R6Y
.
You are receiving this because you were assigned.Message ID:
@.***>


QA on article-related events

Following up on https://github.com/artsy/force-private/issues/5375.
cc: @kanaabe @owendodd @anipetrov

Article share [related article] doesn't fire from related articles (under "What to read next").

Additional notes for superarticles:

Article impression [table of contents] doesn't fire for either top or bottom TOC.

also TODO: Follow button image set

@joelauerbach - I set up impressions for the related box, but didn't include the URL's because there are 10 and it is a very long list -- let me know if this seems practical or you'd like to get the slugs.

@kanaabe - looks like the follow button clicks are already being tracked. Tried to add impressions as well, but the current setup is dependent on scrolling (not possible inside an imageset). Do you think click-tracking is sufficient for this case?

@eessex Thanks! I think we'd like to at least get the Positron IDs of the related articles, if possible.


Change NORMALIZEWHITESPACE to NORMALIZE_WHITESPACE

Python uses the latter.

Let's do this after #77, which simplified the directive parsing. Dont' forget to update NEWS.

oops, yes +1.  I can do this if you want.

That would be great.


[TS] Additional built-in container styles

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
TS implementation of #1886

Implemented with 46f9ab8


⚠ RPC server - Harmony has degraded performance

In d944af4, RPC server - Harmony ($HARMONY_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Harmony performance has improved in 8a80fe8.


[ISSUE #4108] Field may be 'final'[EventMeshRebalanceService]

Fixes #4108
Motivation
Field may be 'final'[EventMeshRebalanceService]
Modifications
make the three field final
Documentation
Refactoring of this part of the code

please follow the issue received to fix it.

Hello, issue https://github.com/apache/eventmesh/issues/4108 is already fixed by PR https://github.com/apache/eventmesh/pull/4149. Is there any more modifications you wanna make in this PR?


Feat: features to subcircuit

Extension to the feature/subcircuit branch.
Makes feature/subcircuit compatible with new UI

@abstrekt @satu0king This is ready for review 🎉

Not compiling for me :(

@gr455 please can you add screenshots.

Not showing anything for me

Not showing anything for me

Fixed in #1849


bugfix/carrousel-landing

change position to info description in carrousel and fix button-lockdown's size
Button-lockdown:

info carrousel center:

Please dont just copy and paste the css from zeplin, it is not optimized to work inside the site.
remove all the rules that set normal as these dont do anything.
and replace the runes like color: var(--black); with the correct color or remove them if they are not needed.


[BUG] [macOS] Eww window is only visible on one space

Describe the bug
When on macOS, the eww window is only visible on the one space it's  been started from.
Reproducing the issue
Use eww on macOS.
Expected behaviour
The window should be sticky, appearing on all spaces or following the user around
Additional context
I guess that could either be fixed by actually making the window visible on all spaces, or it could also be solved by making the window a proper detectable window according to macOS, which would emable the user to make it sticky through their WM

@Rougemoot can you try setting focusable="true" on your window definition?
as in:
<window name="main_window" selectable="true">
  ...
</window>

That would tell the WM to treat eww more like a normal window, specifically also making it focusable. This may make it possible for you to go with your option 2, making it sticky in the WM config. Note that that is likely to also make eww focusable by the WM, which is something you'd then also need to tell the WM to ignore in some other way

I'll definitely try that, although is is focusable or selectable?

focusable, sorry xD Just noticed my mistake

Added the focusable attribute to the docs, among a couple others in 0e97847a3ae4614cc1dcb7cf51ece2e63745c97f

Unfortunately this doesn't solve the problem. The window still isn't detected by my WM (yabai) so I can't make it sticky.
Two things have changed from adding the selectable property: a border appeared around the window, and all text becomes grey unless I click on the window, as if to indicate that the window isn't currently focused.

i guess the border and focus stuff was to be expected, lol. It's kinda weird that it isn't detected by the WM tho,... do you have any way of seeing what makes the WM ignore it?

Nope, I can just ask the WM what it sees, and it just doesn't see it as a proper window

not as a proper window (if so, what exactly does that mean? what does it see it as?) or just not at all?

Oh yeah sorry it just doesn't see it at all.

This might be what we need?
https://wiki.gnome.org/Projects/GTK/OSX

@Rougemoot Do you mean the widgets...

...start in a place, are visible there, and then if you launch a window it gets covered up?
...always just keeps getting started behind windows?

Because if it's the first one, that's what the default behavior is on my window manager as well.

@Rougemoot Do you mean the widgets...

...start in a place, are visible there, and then if you launch a window it gets covered up?
...always just keeps getting started behind windows?

Because if it's the first one, that's what the default behavior is on my window manager as well.

may be a result of xmonad not respecting the window stacking hints, that may or may not be fixed in the gtk4-version, or may require some xmonad shit that i'll need to investigate.

Vermoots issue here is workspaces, eww on macos is currently not "sticky", and cannot be made such as it isn't seen by yabai at all. That, too, may be fixed by gtk4.

may be a result of xmonad not respecting the window stacking hints, that may or may not be fixed in the gtk4-version, or may require some xmonad shit that i'll need to investigate.

Vermoots issue here is workspaces, eww on macos is currently not "sticky", and cannot be made such as it isn't seen by yabai at all. That, too, may be fixed by gtk4.


Python3 compatibility is broken

There are two old-style print statements in __init__.py that break compatibility with Python 3.
I added brackets to turn them into function calls and that seemed to fix it, but I have not done extensive testing to see if there are any other compatibility issues.

I just ran into this issue too.

Running the example on the README file just gives this error:
Types [2, 2, 1] and maxint [24, 24, 0] detected
--- Evolve in 625 possible combinations ---
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-2-9446b1aeccef> in <module>()
     28                                    generations_number=5,
     29                                    n_jobs=4)
---> 30 cv.fit(X, y)

/usr/local/lib/python3.5/site-packages/evolutionary_search/__init__.py in fit(self, X, y)
    286         self.best_params_ = None
    287         for possible_params in self.possible_params:
--> 288             self._fit(X, y, possible_params)
    289         if self.refit:
    290             self.best_estimator_ = clone(self.estimator)

/usr/local/lib/python3.5/site-packages/evolutionary_search/__init__.py in _fit(self, X, y, parameter_dict)
    346         pop, logbook = algorithms.eaSimple(pop, toolbox, cxpb=0.5, mutpb=0.2,
    347                                            ngen=self.generations_number, stats=stats,
--> 348                                            halloffame=hof, verbose=self.verbose)
    349
    350         current_best_score_ = hof[0].fitness.values[0]

/usr/local/lib/python3.5/site-packages/deap/algorithms.py in eaSimple(population, toolbox, cxpb, mutpb, ngen, stats, halloffame, verbose)
    145     # Evaluate the individuals with an invalid fitness
    146     invalid_ind = [ind for ind in population if not ind.fitness.valid]
--> 147     fitnesses = toolbox.map(toolbox.evaluate, invalid_ind)
    148     for ind, fit in zip(invalid_ind, fitnesses):
    149         ind.fitness.values = fit

/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/multiprocessing/pool.py in map(self, func, iterable, chunksize)
    258         in a list that is returned.
    259         '''
--> 260         return self._map_async(func, iterable, mapstar, chunksize).get()
    261
    262     def starmap(self, func, iterable, chunksize=None):

/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/multiprocessing/pool.py in get(self, timeout)
    606             return self._value
    607         else:
--> 608             raise self._value
    609
    610     def _set(self, i, obj):

/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/multiprocessing/pool.py in _handle_tasks(taskqueue, put, outqueue, pool, cache)
    383                         break
    384                     try:
--> 385                         put(task)
    386                     except Exception as e:
    387                         job, ind = task[:2]

/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/multiprocessing/connection.py in send(self, obj)
    204         self._check_closed()
    205         self._check_writable()
--> 206         self._send_bytes(ForkingPickler.dumps(obj))
    207
    208     def recv_bytes(self, maxlength=None):

/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/multiprocessing/reduction.py in dumps(cls, obj, protocol)
     48     def dumps(cls, obj, protocol=None):
     49         buf = io.BytesIO()
---> 50         cls(buf, protocol).dump(obj)
     51         return buf.getbuffer()
     52

TypeError: can't pickle dict_items objects

Same problem here. Is there any solution?

I just fixed the compatibility with pyhthon3. Just do a pip install sklearn-deap --upgrade


Build time is slow

Perhaps it's time to consider another GUI framework. Suggestions?

Could also consider dynamically linking the peer library.


Not working with IE11

Hi @fooloomanzoo ,
I have used the datetime-picker and customized it as follows, custom-datetime-picker. The es5-bundled build throws errors on IE11 (by webcomponent-lite.js). Polymer lint does not show any errors either! I would really appreciate it if you could spare a bit of your valuable time to help me out in any way you can.
Thank you!

I hope the following information would be useful in some way.

I am using prpl-server to serve the build
I tried removing code line by line until I got a working build on IE11. Following is the status,
a. If I import a file from datetime-picker the build is not working.
b. After commenting out my custom code( for ex: method and template overrides of DatePicker) and commenting out the html imports from datetime-picker, the build works.
c. After commenting out my custom code and leaving the html imports from datetime-picker, the build does not work.

(I imported only the calendar-button.html in date-time-picker.html and I commented the rest of it as follows. Also only the html imports from datetime-picker was left inside the calendar-button.html )
<link rel="import" href="../../bower_components/polymer/polymer-element.html">

<link rel="import" href="calendar-button.html">
<!--<link rel="import" href="time-button.html">
<link rel="import" href="customized-date-input.html">
<link rel="import" href="micro-seconds-input.html">-->

<dom-module id="date-time-picker">
    <template>
            <style is="custom-style">
                :host {
                    display: block;
                    width: auto
                }

               calendar-button,
               time-button{
                --iron-icon-fill-color: #1C5397;
                --input-style_-_padding:6px;
                --iron-icon-width:16px;

                background: #ffffff;
                height: 27px;
                width: 28px;
                position: relative;
                top: 7px;
                /* left: 5px; */
                border: 1px solid rgba(0, 0, 0, .25);

                --datetime-background:#32495d;
               }

               customized-date-input,
               micro-seconds-input{
                   background-color: #ffffff;
                   border: 1px solid rgba(0, 0, 0, .25);
                   height: 27px;
                   font-size: 0.9em;
                   --input-focus_-_color: #E9A75F;
                   --input-style_-_color: #444444;
               }

            </style>

        <div class="container">
           <!--<calendar-button no-overlap
                            auto-confirm
                            tabindex="-1"
                            year="{{year}}"
                            month="{{month}}"
                            day="{{day}}"

            ></calendar-button>
            <customized-date-input locale="[[_localeForDateInput]]"
                                   _seperator="[[_seperator]]"
                                   year="{{year}}"
                                   month="{{month}}"
                                   day="{{day}}"
            >
            </customized-date-input>
            <time-button no-overlap
                         clamp=""
                         tabindex="-1"
                         hours="{{hours}}"
                         minutes="{{minutes}}"
                         seconds="{{seconds}}"
                         milliseconds="{{milliseconds}}"
                         microseconds="{{microseconds}}"
            >
            </time-button>

            <micro-seconds-input clamp=""
                                 hours="{{hours}}"
                                 minutes="{{minutes}}"
                                 seconds="{{seconds}}"
                                 milliseconds="{{milliseconds}}"
                                 microseconds="{{microseconds}}"
            >
            </micro-seconds-input>-->
            why you do this??
        </div>
    </template>

    <script>
        /**
         * `mrs-datetime-picker`
         * A polymer element by incorporating datetime-picker webcomponent by fooloomanzoo
         *
         * @customElement
         * @polymer
         * @demo demo/index.html
         */

        class DateTimePicker extends Polymer.Element {
            static get is() {
                return 'date-time-picker';
            }

            static get properties() {
                return {
                    /**
                     * protected property to set date format
                     * en => MM/DD/YYYY
                     * fr => DD/MM/YYYY
                     * ja => YYYY/MM/DD
                     */
                    _localeForDateInput: {
                        type: String,
                        computed: '_computeDateFormat(dateFormat)'
                    },
                    /**
                     * protected property to set date seperator
                     * seperator from the date format is given by _computeDateFormat(dateFormat)
                     */
                    _seperator: {
                        type: String,
                        value: '/'
                    },
                    /**
                     * property to set date format
                     * MM/DD/YYYY
                     * DD/MM/YYYY
                     * YYYY/MM/DD
                     * MM-DD-YYYY
                     * DD-MM-YYYY
                     * YYYY-MM-DD
                     */
                    dateFormat: {
                      type: String
                    },
                    year: {
                        type: Number,
                        notify: true
                    },
                    month: {
                        type: Number,
                        notify: true
                    },
                    day: {
                        type: Number,
                        notify: true
                    },
                    hours: {
                        type: Number,
                        notify: true
                    },
                    minutes: {
                        type: Number,
                        notify: true
                    },
                    seconds: {
                        type: Number,
                        notify: true
                    },
                    milliseconds: {
                        type: Number,
                        notify: true
                    },
                    microseconds: {
                        type: Number,
                        notify: true
                    },
                    dateTime: {
                        type: Date,
                        computed: '_computeDateTimeObject(year, month, day, hours, minutes, seconds, milliseconds)'
                    }

                }
            }

            _computeDateFormat(dateFormat){
                switch(dateFormat) {
                    case 'MM/DD/YYYY':
                        this._seperator = '/';
                        return 'en';
                        break;
                    case 'MM-DD-YYYY':
                        this._seperator = '-';
                        return 'en';
                        break;
                    case 'DD/MM/YYYY':
                        this._seperator = '/';
                        return 'fr';
                        break;
                    case 'DD-MM-YYYY':
                        this._seperator = '-';
                        return 'fr';
                        break;
                    case 'YYYY/MM/DD':
                        this._seperator = '/';
                        return 'ja';
                        break;
                    case 'YYYY-MM-DD':
                        this._seperator = '-';
                        return 'ja';
                        break;
                    default:
                        return 'en';
                }
            }

            _computeDateTimeObject(year, month, day, hours, minutes, seconds, milliseconds){
                let computedMonth = month -1;
                return new Date(year, computedMonth, day, hours, minutes, seconds, milliseconds);
            }

        }
        window.customElements.define(DateTimePicker.is, DateTimePicker);
    </script>
</dom-module>

the build with the above code behaves as I explained in 2. above.

I have made a branch in my repo with the changes I made to get a working build. (working build does not mean the issue is fixed)

I opened some issues myself on polymer's and polymer-build's github repro about beeing not transpilable to IE11. I tried for this element to reduce limit the usage of too new techniques, but es-6 classes are the core of extending each element. And that is the problem. There are a couple of tools that are better than the prpl-server of polymer for transpiling, because it also just uses a babel transpiler under the hood. There is like said babel, roolup and webpack for example. But they all have some difficulties and there is also the question of your purpose, if you f.o. dynamically serve or statically and your server.

Oh! This is a really good web component, pity that IE is not ready to upgrade (no pun intended). I am studying vaadin elements (works with IE11), I'll post a workaround if I come across any. Thank you @fooloomanzoo . Cheers!


Install service Lastpass

Hi,
I'm new trying Ferdi, but I need Lastpass, yes or yes.
I saw the service in Ferdi website, that's one of the main reasons why I chose Ferdi, but when I try to install it, I cannot find it.
Could you help me, please?

@artesan0 LastPass is only available in the new nightlies releases
Unfortunately the website is showing automatically all the new recipes creating this confusion for some users.

Can you please try to upgrade to the nightlies available here AND then upgrading the recipes by following the steps outlined here?
If you are on macos or linux, I have shared some newer builds of the latest source code here


Dota 2 Api Schema not working?

Dota 2 Api Schema not working
https://api.steampowered.com/IEconItems_570/GetSchema/v0001/?key=xxxxxxxxxxxxxxxxxxx
steam had changed, bcz its not showing anymore result.
any information about that?

We only support the IEconItems API for TF2.  Use GenericInventory for Dota 2

DotA2 schema is down for few months.
Look here for more informations.


benchmark/agents/common  function cal_lane_its_info seems to have errors

Even if there are many cars in intersections, the flag intersection_distance and closest_its_nv_rel_speed  is no different
I think line 624  has errors,
line_heading = (
                (np.arctan2(nv_to_ego_vec[:, 1], nv_to_ego_vec[:, 0]) * 180 / np.pi)
                - 90
            ) % 360
 nv_to_line_heading = (nv_heading - line_heading) % 360
 ego_to_line_heading = (ego_heading - line_heading) % 360

I replace -90 to +90, the flag intersection_distance and closest_its_nv_rel_speed can have a effect
I think there may be other angle calculation errors in this file

Taking a quick look, it seems like the heading_to_degree and heading_to_vec conversions have an errors converting from north facing heading to east facing degrees. Counter clockwise rotation should have the conversions subtracting 0.5 pi.

Taking a quick look, it seems like the heading_to_degree and heading_to_vec conversions have an errors converting from north facing heading to east facing degrees/radians. Counter-clockwise rotation should have the conversions subtracting 0.5 pi.
There is someone looking into it.

In line 68, it says heading_to_degree function takes 0 as up direction , but actually 0 means down direction.See code and picture blow:
def heading_to_degree(heading):

### +y = 0 rad. Note the 0 means up direction ###

return np.degrees((heading + math.pi) % (2 * math.pi))

So line 301 minus 90 degree to convert heading_to_vec from east to north because it thinks heading_to_degree is north facing。Actually heading_to_degree is south facing, is should add 90.

@feidieufo @Gamenot @Hyperion-shuo yes, this is a bug.

@mg2015started  I found another problem,in common.py,when I use cruising.yaml, in get_reward_adapter function，
    def cruising(last_env_obs, env_obs, env_reward):
        global lane_crash_flag
        global intersection_crash_flag

lane_crash_flag and intersection_crash_flag is always false
         # if to collide in 3s, make it slow down
         if ttc < 2 or ego_closest_its_nv_distance < 6:
             intersection_crash_flag = True

intersection_crash_flag has been modified before
but in the reward function the intersection_crash_flag is still false

@feidieufo


ERD foreign keys referencing the same table are shown incorrectly

System information:

Operating system (distribution) and version Win10
DBeaver version 21.3.0
Additional extensions

Steps to reproduce, if exist:

Create a table with a foreign key referencing the same table (Sample SQLite db, table employee)
Open a diagram

Fixed (the fix was included in 21.3.0)

Verified
Minor issue created based on this one
#14724


Update default data node.

Issue:
When attempting to set the value of a child without first defining mock data a TypeError is thrown.
Sample code:
import { AdminRoot, constants as Constants  } from 'firebase-admin-mock'

admin = new AdminRoot()
admin.initializeApp({
    databaseUrl: Constants.DEFAULT_DATABASE_URL,
})

// Without the below line.
// admin.setMockData({ foo: 'bar' })

admin.database().ref('test_key').set('here_is_a_val', () => { }).then(() => {
    console.log('Success!')
}).catch((error) => {
    console.log('Error occurred:' + error.message)
})

Expected behavior:
test_key is set successfully with value here_is_a_val and the promise resolves with Success!
Actual behavior:
The following TypeError is thrown:TypeError: Cannot read property 'setIn' of null
Proposed changes:
The issue is due to the initial data being set to null. The default root was updated to be {} as opposed to null.

As far as I know Firebase initializes with null, so we should leave the initial state as it is right now. The solution should be to do some sort of check in the .set() method or maybe elsewhere to avoid these kinds of errors. I'm a bit busy at the moment, would you consider doing the changes yourself?
If you do the Reference class is in the database-accesors.js file. This project was hacked together in a rush and it may be a little bit contributor unfriendly. It you have any questions don't hesitate to ask.

Sounds good! I'll close this PR and reopen with an alternative solution!


Invalid repetition range should give an error

Describe the bug
The following melody code should not compile, but it does and generates an invalid regex because the repetition range cannot be from 2 to 0.
2 to 0 of match { "test"; }

crates/melody_compiler/src/ast/source_to_ast.rs
Rule::quantifier_range => {
    let (start, end) = first_last_inner_str(kind)?;

    // maybe check here that end >= start?

    Node::Quantifier(Quantifier {
        kind: QuantifierKind::Range {
            start: start.to_owned(),
            end: end.to_owned(),
        },
        lazy,
        expression: Box::new(expression),
    })
}

To Reproduce
$ echo '2 to 0 of match { "test"; }' | melody /dev/stdin
> (?:test){2,0}

Expected behavior
An error.

Right you are, will look into fixing this soon. Thanks!

Fixed in https://github.com/yoav-lavi/melody/commit/c856f06640aa70f5b37498dcc57d992c39e840e6, will release a version with the fix soon


Options descriptions are cut off on windows with display scaling 150%

I am running Windows 10 with a display scaling of 150%.
On Windows 10 with a display scaling of 150%, the descriptions in the options window are cut off:

If I switch display scaling to 100% and relaunch the launcher, the options text shows correctly. I didn't try any other display scale settings but I'm assuming it's going to be off for other values as well.

Thanks for the report, I'll look into it when I have some free time.


Json output

I've tested a JSON sample like this with the 0.4.2 release:
public class PippoController extends Application{
    @Override
    protected void onInit() {
        GET("/shop", new RouteHandler() {
            public void handle(RouteContext routeContext) {
                Shop shop = createShop();
                routeContext.json().send(shop);
            }
        });
    }

    private Shop createShop(){
        Shop shop = new Shop();
        shop.setName("KFC");
        shop.setStaff(new String[]{"messi","ronaldo"});
        return shop;
    }
}

The output was:
curl localhost:8338/shop

Cannot find a route for 'GET /shop'

Available routes:

GET /shop

</b

And the error was:
ro.pippo.core.PippoRuntimeException: You must set a content type engine for 'application/json; charset=UTF-8'
	at ro.pippo.core.Response.send(Response.java:818)
	at ro.pippo.core.Response.send(Response.java:809)
	at ro.pippo.core.route.DefaultRouteContext.send(DefaultRouteContext.java:223)
	at ro.test.pippo.PippoController$1.handle(PippoController.java:17)
	at ro.pippo.core.route.DefaultRouteContext.handleRoute(DefaultRouteContext.java:339)
	at ro.pippo.core.route.DefaultRouteContext.next(DefaultRouteContext.java:249)

XML is working well. I will add the Shop class as well
@XmlRootElement
public class Shop {
    private String name;
    private String staff[];

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public String[] getStaff() {
        return staff;
    }

    public void setStaff(String[] staff) {
        this.staff = staff;
    }
}

You need to add one of the Pippo JSON engines to your classpath.  Add a dependency on either pippo-gson or pippo-fastjson.  pippo-jackson is coming in the next release.

Great,
Now it works, but I think a clearer error will be more appropriate
Thanks

The error is clear enough.


Allowing for external sizing of images

Status
READY
Description

Using external image size and fit properties so that I can force images to have same size and aspect ratio.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Hi! @MattCrist, I'll have a look at this during the day so that I can understand the reason behind the PR.
Thanks for the PR :)

@cgutierr-zgz If you need some code samples or screenshots to explain the issue I can do that.
Basically I have a set of reference images all 1:1 aspect ratio and I need to force the second image to stretch to the same size so the comparison works properly.  I definitely need to pass in BoxFit.fill to get the stretching to happen, I was surprised that I also need to explicitly set the size but it seemed to be the only way to make it work.


Running commands as tasks - proper support

I am trying to give a user ability to create a task that will transfer file to remote target. There is no such command in shell, however there is command I created in my extension, but it needs arguments (and no prompting as it suppose to be scripted).
The issue #11396 was closed as implemented saying it supports execution of command as tasks, but it does not really...
Specifying "command" in the "command" property only works if command returns void and has no arguments (and I think it just works by accident because vscode will ignore evaluation of "empty" command).
And specifying command in input section is clumsy and only works if something uses that input.
Also I failed to locate any documentation describing on how to create task that runs a command.
I would like a proper support such as

support "type": "command"
the "args" of the task pass properly as args of the command
if command returns anything print it as string on a terminal as for shell and not attempt to evaluate result as command to be executed (as it is now)

Here is a test
register command:
     subscriptions.push(
      commands.registerCommand("qnx.echo", async (...args: any[]) => {
        return JSON.stringify(args);
      })
    );

in instance of vscode with this command registered in tasks.json
    {
      "label": "command_echo",
      "command": "${command:qnx.echo}",
      "args": ["xxx"],
      "problemMatcher": []
    }

current output (observed with vscode 1.73.1):
    *  Executing task: [["${command:qnx.echo}","/home/user/pathtomyfolder"]] xxx
    *  The terminal process failed to launch: Path to shell executable "[["${command:qnx.echo}","/home/user/pathtomyfolder"]]" does not exist.

expected output (assuming we can add "type": "command" also to task):
    *  Executing task: ${command:qnx.echo} xxx
    [xxx]
    *  Terminal will be reused by tasks, press any key to close it.

Note - I can "hack" tasks.json to achieve kind of the same result by using input and combining with "echo" shell command, but its not proper command support as "Running commands as tasks" should be - as claimed to be implemented in #11396
Current hack:
  {
  "version": "2.0.0",
  "tasks": [
   {
      "type": "shell",
      "label": "command_echo",
      "command": "echo ${input:echo_xxx}",
      "problemMatcher": []
    }
  ],
  "inputs": [
    {
      "id": "echo_xxx",
      "type": "command",
      "command": "qnx.echo",
      "args": "xxx"
    }
  ]
}

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

I would like to re-classify it as default, because its not really a feature, its a bug in current implementation of feature #11396


Cannot do Near.call to thirdparty contracts using FastAuth-created account

Describe the bug
FastAuth-created account cannot sign transactions to third-party contracts on near.org, so if users go with the cool feature of near.org, they become hostages unable to interact with DevHub or any other BOS app that uses a custom contract (not social.near) even if they fund the account as signing with full-access key does not work.
To Reproduce

Sign into FastAuth-created account
Go to DevHub: https://near.org/devgovgigs.near/widget/gigs-board.pages.Feed
Try to like or post something
You won't be able to submit the transaction

Expected behavior
I should be able to sign a transaction to third-party contracts (non-social.near), and get the transaction relayed or prompted to fund my account with some NEAR to cover the fees.
Screenshots
Here is the video recording: https://www.loom.com/share/22744a79b3b54e56a9eae1ba7264977b

Thanks for filing this issue @frol ! I ran into the same paint points when onboarding ecosystem contributors to DevHub recently, as they could not interact with the platform via FastAuth accounts. 😞
It would be ideal if we could prioritize this issue as it's a critical developer engagement piece for the NEAR ecosystem.
@trechriron should this issue be transferred to fast-auth-signer or mentioned in a newly created one in that repo?
cc  @DavidM-D @diogoortega

I have added to the Fast Auth project board in Triage status. We regularly look through defects and determine priority, severity, and assignment.


Raid is not running

Tryied to run as administator, but result not changes. Have message "Raid is not runnig" when running app.
Using Blue Stacks to run raid

unfortunately, this tool will only work with the Plarium Play version for PC, not bluestacks. Sorry 😞


soc: esp32: Fix compile warnings when redefining BIT() macro

Include soc.h first, which will include the ESP-IDF headers -- which
will define the BIT() macro without checking if they're already
defined, like the Zephyr headers do.
Signed-off-by: Leandro Pereira<EMAIL_ADDRESS>

Codecov Report

Merging #6278 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6278   +/-   ##
=======================================
  Coverage   53.15%   53.15%
=======================================
  Files         412      412
  Lines       40139    40139
  Branches     7732     7732
=======================================
  Hits        21337    21337
  Misses      15667    15667
  Partials     3135     3135

Continue to review full report at Codecov.

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


MT5 Metatester hang

During testing sometimes the process is hanging.
In Event viewer I can see such information:
The program metatester64.exe version <IP_ADDRESS>1 stopped interacting with Windows and was closed. To see if more information about the problem is available, check the problem history in the Security and Maintenance control panel.
Process ID: 2808
Start Time: 01d62647d8cb3887
Termination Time: 10
Application Path: C:\Program Files\ForexClub MT5\metatester64.exe
Report Id: 27129896-87ab-46f2-9b46-d9856858c050
Faulting package full name:
Faulting package-relative application ID:
Hang type: Cross-thread
Any workaround?

Looks like I fixed this closing active Testing agents (using Manager of testing agent)


Suggestions to tweak before integrating into main

Hey @nesk
as discussed here the minor tweaks before bringing this back in. The last commit contains the result of running it, if this causes issues we can take it out easily.
Otherwise looks good!
Cheers,
Peter

Thank you so much for taking time reviewing this. You saw nothing weird in the generated phpdoc?

I looked only over a few and the changes from re-running it. Anything in particular to look at @nesk?

I'm merging this PR because its valid. However, as you know, we will have to fix the documentation generator first.

Yeah :/ That's going to be piece of work too...


fix: rm checksum for dependencies from local path and fix symlink

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

35 of 51 (68.63%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 43.33%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/client/client.go
10
16
62.5%

pkg/utils/utils.go
25
35
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1939

Relevant Lines:
4475

💛  - Coveralls


Text: Indicate that letterSpacing works on Android

The docs say that letterSpacing is iOS-only but it also works on Android.
Why do we document the Text styles in both text.md and text-style-props.md?
Adam Comella
Microsoft Corp.

Cool, thanks for the update! I guess it didn't work back when this was initially written.


issues adding notification

Hey, this notification looks great but running into some issues when adding to my own site.

Any idea?
The code is at patrickziegler.se/fitnessbuddy

ok,it seems to have to do where in the html I drop the directive.. If I have it at the stat of the body it seems to work, but then a) No text is shown in the box and b)the box is hidden under some other buttons I use..
I would like to have the notifications come out at the bottom right of the page, I tried moving the directive but then it always get the above error

also I can't get it to show up on mobile browser :/
ill update if I can fix it

ok, I figured out the z-index should be on the top class ng-atomic-notify for it to hover above all my other content


Backport #6674: Prevent bare metal machine config references from changing to existing machine configs

Manual backport of #6674.

/retest

/approve


Update chat.rs

Signed-off-by: Michael Yuan<EMAIL_ADDRESS>The chat_request.max_tokens is never None. Even if the request does not contain this field, it will default to the server default (i.e., 1024).
That means the else block is never executed.
If a request sets a low max_tokens, all subsequent requests will "inherit" that low n_predict and never recover.
In this PR, I am moving this else block so that subsequent requests can reset the n_predict value.

Hello, I am a code review agent on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


System.get_env

I may be totally looking at this wrong and would love to be corrected.
It looks like if you did the config like this:
config :my_app, mailgun_domain: "https://api.mailgun.net/v3/mydomain.com",
                mailgun_key: System.get_env("MAILGUN_API_KEY")

It's going to pull the environment variable at compile time since the config happens in a macro? The problem, I'm running into is that at runtime  the env variable may be different than the compile time one (if there's even a compile time one there).

I've been battling this off and on for a few days too. I've been trying to follow this with the Heroku deploy guide. But the environment variables don't work there. Everything seems to work okay locally though? I don't even know how I can configure the Mailgun client without using the keys directly since the environment variables don't seem to be available at compile time.

I have the same issue with Heroku, if the config is set during compile time it cannot be changed by changing ENV variable, the app must be recompiled.
Will be glad to help with this issue.

System.get_env("MAILGUN_API_KEY") is a compile time value. That gets converted by Mix.Config into a sys.config file at compile time and the value of the environment variable is used.
On top of that, the config macro in this libray is expanded at compile time as well, and caches the value from the Mix.Config/sys.config. So the value that will be in it is the one that is set at compile time only.
Short answer: The configuration of this app doesn't support changing the configuration at runtime which is kind of broken.

@geofflane My little patch should help with that.


seed not used in shuffle

It appears that the seed is not used in shuffle?
https://github.com/webdataset/webdataset/blob/f11fd66c163722c607ec99475a6f3cb880ec35b8/webdataset/filters.py#L210-L212

Ah, duplicate of #347


Since getting to 1.4.1 task order changes when completing a task

Since updating to 1.41. the order of my open tasks keeps changing when I complete a task.
Is this a new setting I missed or a bug?
Thanks,
George

Maybe I should add some more info :)
It happens in a note where I use the following code to get my tasks:
due today
not done
hide backlink
hide due date
hide recurrence rule
hide task count

It is really annoying especially when quickly completing a bunch of tasks as they keep jumping around making me complete wrong tasks.
I hope there is a solution
George

Thank you for the report @George20730. Something similar was reported here: #383
I was not yet able to reproduce it. But I agree that it’s a bug that must be fixed.

Hi @schemar if it could help, I could send you my task related files.

Thank you very much, I appreciate it. For now, I'll try and reproduce it myself. I have to have a testcase covering this either way.

Hi @schemar.
I'm wondering whether the issue reported by @George20730 -- which I've also experienced -- may be related to the algorithm driving the new "Sort by Urgency" default that was introduced in version 1.4.0 (which was released earlier on the same day as version 1.4.1 -- which @George20730 identified as the source of the problem).
What does Tasks use as a tie-breaker when multiple items have the same Urgency score?
(Asking because It occurs to me that if it were the file's last modified date, then that value could change the order -- relative to other tasks with the same Urgency score -- of any not yet completed tasks that are contained in the same note file as the just-completed task.)

It is related to that change. From the implementation, it should not alter the original behavior if everything else is the same. But somehow it does (non-deterministically). The only influencers are the ones listed in the docs here: https://schemar.github.io/obsidian-tasks/advanced/urgency/
After that, no other "tie breaker" is used. It baffles me a bit, but we'll get to the ground of this eventually :smile:

This is happening primarily because status is included in the list of default comparators. We shouldn't sort by status unless the user explicitly requests it. Also, the way urgency was calculated yields a different value every time this could cause sort order to change depending on the initial state of the list. See PR above.

Right. I included urgency scores and got 1.95 on all my tasks. But changing to in_progress or scheduled moves the task.
## 5.08 Monday
- Monday thing 1
- Monday thing 2
- [ ] Monday task 1/1

## 5.09 Tuesday
- [x] Tuesday task 1/1 (DONE)

## 5.10 Wednesday
- [ ] Wednesday task 1/1

## 5.11 Thursday
- [x] Thursday task 1/4 (DONE)
- [x] Thursday task 2/4 (DONE)
- [/] Thursday task 3/4 (IN PROGRESS)
- [ ] Thursday task 4/4 (NOT DONE)

## 5.12 Friday
- [ ] Friday task 1/1

Shows up as:
tasks
not done
path includes planner/2023.05.Wk19.md
short mode
sort by heading
hide task count
show urgency

If I don't have the Thursday task 3/4 as in progress and change it back to not done [ ] then it gets the order right.

Right. I included urgency scores and got 1.95 on all my tasks. But changing to in_progress or scheduled moves the task.

@alxgoldstn I’m happy to help, but your question is not related to this topic.
Please start a new Q&A Discussion so we can discuss it there without spamming all the people who have contributed to this.
Thanks!

@alxgoldstn My apologies - I misread this as being about the 1.95 score - but on re-re-reading, see it really is about sorting.
I will have a closer look - don't re-post in Q&A....

Ah, true. I was meaning to highlight that the Urgency score was the same, so therefore any other sorting should only come from the rules I included.
Also, thank you! You're doing massively amazing work and this plugin is mind-boggling! :)

OK I have done some experimenting and have a better understanding of how sorting actually works in practice, and it differs somewhat from the docs.
I'm struggling to write it up well in the docs, so for now I will place notes here, in case it helps anyone else....
The 5 default sorts, that are appended to every query

The default order is:
sort by urgency
sort by status
sort by dueDate
sort by priority
sort by path

Those 5 lines are always appended to the end of any sort by instructions supplied by the user.

sort by status is a particular problem for people using custom statuses, and for searches that include done tasks - as for backwards compatibility reasons, it puts [ ] first, and all others afterwards, regardless of your custom status types.

So doing sort by status.type instead is more useful.

sort by path is important, as it prevents tasks in the most-recently edited file jumping to the end of the list after editing due to an implementation detail of how the Tasks caching code works

Others have pointed out that because urgency does not take account of status, this default sort order can put urgent-but-done tasks ahead of not-yet-done tasks.

What I think the default sort order should be
So, a better/safer default order would probably be:
sort by status.type
sort by urgency
sort by dueDate
sort by priority
sort by path

Ordinarily I don't like changing existing behaviour for fear of breaking existing searches, but I suspect that the current behaviour is sufficiently broken that in this case it is justified.
How to force your own sort order
If you don't like the default sort orders, your only choice is to add enough sort lines to your query that uniquely sort all tasks in to a unique order, so that the default 5 do not have any effect on the final task order.

But looking back to the original subject of this issue:

Since getting to 1.4.1 task order changes when completing a task

The reason that competing a task changes the sort order is because the status is included in the default sort ordering....
As already pointed out in  https://github.com/obsidian-tasks-group/obsidian-tasks/issues/439#issuecomment-1062906678 above.

Wouldn't it be better to change the default so that it can be overridden? I would think that the minimum requirement should be to alter the user's tasks list as little as possible, with query options adding alterations.
Also, I'm new to the plugin, so how would changing the default sort order break existing searches? Sure, it might alter users' search result order, but they could either:

add in sort by status (or status.type, as proposed) to the task list
add in sort by status to the Global Query to get "pre-current" functionality

The biggest problem with this default is that there isn't (afaik) any way to remove the existing sort order. A cooking analogy: Why am I trying to remove salt from a meal? That's hard / impossible. Instead, just don't pre-salt the food, and let me add it in if I want.

@schemar Hallo, Any thoughts you have on todays messages would be very welcome.

Also, I'm new to the plugin, so how would changing the default sort order break existing searches?

For people who were happy with the previous default sort order for many but not all of their queries, any change in the default will require them to update all their queries where they didn't need to add any sort instructions, to now manually set an order.
So that's 'break' in the sense of:

Creating work for users who were previously happy
"There will be user complaints..." - you only have to look at the start of this ticket for an example - and user support emails == time-consuming effort for me!

Wouldn't it be better to change the default so that it can be overridden? I would think that the minimum requirement should be to alter the user's tasks list as little as possible, with query options adding alterations.

I do take the point about being able to override the defaults.
I didn't write this code  - from reading it, I had always assumed from the docs that if the user provided any sort lines, then the defaults would not be used.
TIL.
So my initial inclination is:

Only apply the default 5 sorts if the user doesn't supply any sort instructions.
Make the 2 adjustments to the default sort order that I mentioned above.

One important thing though: without a rewrite of the internals, if sort by path isn't in the list, the order of tasks will change randomly for all tasks in any file that is edited. People aren't seeing that now because path is always in the list.
So maybe it should be:

If the user supplies any sort instructions, append only sort by path
Otherwise, apply the updated 5 default sorts, so minimise changes to existing tasks blocks.

@schemar Thanks for a very insightful and helpful reply.

The upcoming 6.0.0 release has a few small breaking changes/bug fixes in it, so I'm looking at fixing the default sort order so that searches that include historical DONE and CANCELLED tasks appear lower down the task results.
Specifically, I'm changing the always-on sort order like this:

It basically means that if the user does not add any sort by, tasks will appear in the order:

All IN_PROGRESS tasks
Then all TODO tasks
Then all DONE tasks
Then all CANCELLED tasks
Then all NON_TASK tasks

And within each of those 5 groups, tasks will be sorted by urgency, then due date, then priority, then file path.
It's a localised change - and from experimenting with it, it gives a much obviously correct sort order.
I have no idea whether it fixes the issue originally described in this ticket, as it does not have a a set of tasks that I can copy in order to reproduce the problem...
So I will mark this issue as closed when I have done the above - and if there is still a problem with unreliable sorting after the 6.0.0 release, then I will require a full reproduction is given in a new issue.

Just released in 6.0.0


Add rolfedh Rolfe Dlugy-Hegwer as a reviewer.

I'd like to request review privileges for the website.

I've set up a new Team in GitHub so reviewers can manage pull requests on this repo. Once the invites are accepted I'll approve this PR.

/approve


Updates to config files for next major version

And adding a index.js to expose all of shell's libraries via an npm entry.

shell?


Things still do do

[ ] Post visibility - need ability to set author level permissions. Back-end: we need an extra field in the Post model called "visibleTo". Getting posts needs to query for these. Front-end: Mostly hooked up, need a list friends.
[ ] Post deletion. Author should be able to delete a post. Front-end: Show delete button only for users own post, send a DELETE request to posts//. Wait for response and remove post.
[ ] Add friend - Front-end: show a list of users that the current user is not friends with, allow them to follow other users. If we aren't changing the back-end this means we need two network calls: one to get friends and one to get all users. Then we need to filter these into two different groups: friends and others.

That should be enough to get more than half the requirements, don't even need unfriend (and likely don't need post deletion).
Miscellaneous:

[ ] API is inconsistent with requirements.txt. Probably should eventually change it. At the very least we NEED to document it.
[ ] Add more tests
[ ] Need to deploy to Heroku and make sure everything works

I'm currently working on Post Visibility


Chore: Bump android compileSdkVersion to 31

Codecov Report

Merging #20 (0b20bd1) into main (253b94b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           79        79
=========================================
  Hits            79        79

Continue to review full report at Codecov.

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


🐛 [BUG] - z-index issue

🐛 Bug Report Form

Description
When we scroll down the images goes on top of the navbar .

Reproduction URL
Please provide your GitHub URL to reproduce the issue

Screenshots

https://github.com/ZeroOctave/resource-gallery/assets/85991449/ef82bf8b-3233-4172-a9d7-9d76eade7db3
|
| Logs | Relevant log output goes here |
| Additional Context | Please provide any other relevant information about the problem |
🌐 Browsers
Which browsers are you experiencing the problem on? (Select all that apply)

[x] Chrome

💻 OS
What is the impacted environment (operating system)? (Select all that apply)

[x] Windows

📝 Sections Completed
Check the box of the sections you have completed:

[x] Description
[ ] Reproduction URL
[ ] Reproduction Steps
[x] Screenshots
[ ] Logs
[x] Browsers
[x] OS
[ ] Additional Context

🔍 Priority
Please select the priority of the bug:

[x] High
[ ] Medium
[ ] Low

📜 Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct:

[x] I follow Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme.

Can you assign this issue under GSSOC'23.

Hey👋🏼, @parthpathakpp1, thank you for opening an issue. You need to update or make PR in under 7 days
Soon the maintainers/owner will review it and provide you with feedback/suggestions.If you think it's something urgent, feel free to reach out to Anand on Twitter.Give us a ⭐ to show some supportKeep Contributing 🚀


Added ability to deploy cuncurrent pull requests

This will enable us to add AppSet name as suffix to GitHub and GitLab App names, and renamed GitLab file from pr to mr as suffix

:robot:
Key changes in the patch are:

The version of the application has been updated from "0.9.1" to "0.10.1".
The ability to deploy concurrent pull requests has been added.
The AppSet name has been added as a suffix to GitHub and GitLab App names.
The GitLab file has been renamed from "pr" to "mr" as a suffix.
The annotations have been updated to include a link to the artifacthub documentation.

There are no potential problems identified in the modified lines of the patch. However, an optimization suggestion could be to add more detailed descriptions to the annotations, especially for the changes made to the application. For example, instead of "change ability to deploy services to different namespaces", a more detailed description could be added to explain what specific changes were made to enable this ability. Similarly, a more detailed description could be added to explain how the AppSet name was added as a suffix to GitHub and GitLab App names. This would make it easier for other developers to understand the changes made to the application.

:robot:
The patch introduces changes to the helm chart for ArgoCD ApplicationSets for Kubernetes. The most significant change is the ability to deploy concurrent pull requests. Additionally, the patch adds the ability to include the AppSet name as a suffix to GitHub and GitLab App names, and renames the GitLab file from pr to mr as a suffix.
Minor changes include adding platform annotations to services, updating HELM chart keywords, updating Chart annotation artifacthub changelog, updating chart documentation, and updating chart values example.
The patch also updates the version of the helm chart from 0.9.1 to 0.10.1.
One potential problem is that the patch does not provide enough information about the changes made to the platform annotations to services. It would be helpful to provide more details about the changes made to ensure that they do not cause any issues.
An optimization suggestion would be to provide more information about the changes made to the helm chart values example. This would help users understand how to use the helm chart more effectively. For example, providing more details about the values that can be customized would be helpful.


OU-407: Add monitoring-plugin uiplugin

This PR looks to add an ACM specific version of the monitoring plugin into obo. It is reliant on changes in this PR which add the ACM specific flags.
This plugin should be deployable alongside the monitoring plugin deployed by CMO to allow for the local-cluster to still have the observability pages, while enabling the observability pages under ACM using the obo deployment.

Updates to the monitoring-plugin PR have been completed which sync feature names and parameters with changes in this PR

/hold newest changes fail when ACM isn't installed

/unhold

/ok-to-test

/retest


validate format for SettingsChange#key

This is probably for developers only, but this puts validation on the value that goes into the key

update:

added \A, \z to ensure the regular expression matched the key completely

update:

added specs around the validation


/usr/bin/env: ‘python’: No such file or directory

Following your tutorial on LinkedIn after entering to your docker container, when trying to clone the meta-information by using the repo tool the following error pops up:
imx6ull@aa92169034b9:~/oe-core$ repo init -u git://git.toradex.com/toradex-manifest.git -b kirkstone-6.x.y -m tdxref/default.xml
/usr/bin/env: ‘python’: No such file or directory

@sajjadsef Thanks for report the problem.
I fixed the dockerfile and uploaded the Docker Hub image. Please, try again with the latest Docker image version.


Use typescriptin vue, when overide one method in child component, in this method cannot call parent method?

Version
2.5.16
Reproduction link
https://jsfiddle.net/50wL7mdz/443179/
Steps to reproduce
a. setup vue-cli
b. create a typescript project
d. accord the https://github.com/Realytics/fork-ts-checker-webpack-plugin/issues/111 , create vue.config.js file and write content.
g. split HelloWorld.vue to HelloWorld.ts and HelloWorld.vue ,
accord   https://jsfiddle.net/50wL7mdz/443279/    modify HelloWorld.ts and HelloWord.vue.
h. run yarn run serve, then access the http://localhost:8080, click the Test Button, you can find the error in console.
What is expected?
console should log the "before_value_changed in base"
What is actually happening?
some error happened.

I create two vue class , and one extends other and override one method,
in this method of child vue class,  i use super.xxx  to call this method of parent class,
it will error.
It works fine for typescript.  (followed code at http://www.typescriptlang.org/play/index.html,  )
class BaseComponent {
public before_value_changed() {
}
public on_value_changed() {
this.before_value_changed();
return "Hello, ";
}
}
class HelloWorld extends BaseComponent {
public before_value_changed() {
console.log('beforeGreet');
super.before_value_changed();
}
}
let hello = new HelloWorld();
let button = document.createElement('button');
button.textContent = "Say Hello";
button.onclick = function() {
alert(hello.on_value_changed());
}
document.body.appendChild(button);

See https://github.com/vuejs/vue-class-component/issues/93#issuecomment-299383043


Add multigraph support to networkx executor

Thank you @celiibrendan!
This also addresses #101, @hpawa. Shortly I will add support for the GrandIsoExecutor as well!

Going to adapt this in #107, closing for now!


Implemented common traits for LogLevel, LogLevelFilter, LogMetadata, …

Added traits to common structs and enums, according to #144. My first pr--did I do this right? :)
Ottavio

Thanks! Looks good to me.
One tip: next time if you put "fixes #144" in the comment when you make the PR, it will auto-close the issue when your PR gets merged.

Added traits to common structs and enums. Fixes #144. My first pr--did I do this right? :)


Link subjects

Obviously, this needs to be de-hacked for internationalization

The whole page has to be de-hacked ;)
On Sun, Jun 14, 2015 at 7:14 PM, arekkas<EMAIL_ADDRESS>wrote:

Merged #111.
Reply to this email directly or view it on GitHub:
https://github.com/serlo-org/athene2/pull/111#event-330484166


Introduced new Exporter system

See #763
So I've splitted my proposal into 2 contributions:

this one which introduce a refactoring of export system (with BC) to allow the implementation of new exporters

-> allow the exporters MUST have render() method which return the correct output.
As a sample, I will - after this contribution to be merged - add a new HTML/PHPUnit-like exporter

Hello @mickaelandrieu, @n1k0. Do you have any update on this PR? It seems like the work on exporter (like #647) is a bit frozen. Can I give a hand anywhere?

@nicolashefti the PR is actually done, and could be merged.
But @n1k0 want to make a complete review before :)

Closed.
Feel free to get the code if needed.


Train on ImageNet?

Hi! Thanks for providing the code! I wonder did you pretrain it on ImageNet and could you please share it?

Hi @fzyzcjy!
We are planning to share pretrained weights, but currently, we do not have them.
Cheers,
Martin

Thanks very much! Waiting :)

Looking forward to the weight of ImageNet. @martinkersner

Hi @martinkersner!
If possible, besides the weights, would you please share the loss curve and param setting?
Such curve for example the one below would be helpful to monitor our training
https://github.com/PengBoXiangShang/MobileNetV3_PyTorch/raw/master/figures/training_loss.png