network: Fix security groups for CentOS

On RHEL/CentOS/Fedora the bridge related sysctl rules are enabled
in kernel by default but can only be disabled. Enabling those keys
will fail, causing iptables/ebtables tables to not be created
and fails SG on CentOS. Therefore, SG does not work on CentOS.
This also fixes an integration test case, which assumes first few
tests complete within 3 minutes. In nested env the value may be large,
this increases the value to 20 minutes.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

GitHub Issue/PRs

Screenshots (if appropriate):
How Has This Been Tested?

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
Testing
[ ] I have added tests to cover my changes.
[ ] All relevant new and existing integration tests have passed.

I've updated docs here:
https://github.com/apache/cloudstack-docs-admin/commit/ea1edc31b25c94471a7abb15385ae3b33c45a69b

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Thanks, agree @wido.

I'll merge this based on travis results, manual testing (trillian tests don't do sg+centos) and code review.

Trillian test result (tid-2533)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 106241 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2590-t2533-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_network.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Smoke tests completed. 66 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_03_vpc_privategw_restart_vpc_cleanup
Failure
210.03
test_privategw_acl.py


ImportError: No module named CGATPipelines.PipelineTimeseries

Hi,
Several tests fail in:
https://travis-ci.org/CGATOxford/cgat/jobs/57975046
with the same error message:
ImportError: No module named CGATPipelines.PipelineTimeseries

Could somebody please have a look?
Thanks!

Thanks Mike and Andreas!
Fixed with commit 85cc5d0


Make kill objectives more common

About the PR
This PR adjusts the weight of traitor objectives, reducing the frequency of steal and social objectives.
Also, the difficulty level of escape to centcomm, steal rd-hardsuit, keep alive, help progress is lowered slightly so traitors dont just get 1-2 of those.
Why / Balance
Currently it's quite common to get only 1-2 steal/social objectives and then escape to centcomm. This PR will throw in more kill objectives in the mix.
Also, by reducing the social objectives it'll be harder for the syndies to group up.

Please no.

Hmm ok, maybe we need new objectives then instead. Maybe a coerce / hurt someone but not killing them?

Hmm ok, maybe we need a new objective type then instead. Maybe a coerce / hurt someone but not killing them?

 They can be revived afterwards and the objective doesn't count. Some forks have this objective for killing jobs like janitors and scientists, but for higher risk jobs like security officers and heads, it's an explicit kill or maroon.

Hmm ok, maybe we need a new objective type then instead. Maybe a coerce / hurt someone but not killing them?

 They can be revived afterwards and the objective still counts. Some forks have this objective for killing jobs like janitors and scientists, but for higher risk jobs like security officers and heads, it's an explicit kill or maroon.

Teach [character name] a Lesson.
"[character name] owes a lot of money to [syndicate company]. Teach them a lesson and rough them up severely, so they know not to mess with [syndicate company]"

Please no.

The exact opposite, IMO.  Get an emag, thief gloves, and a storage implant, and you get what you need every time. The only exceptions to this are the nuke disk and RD’s hardsuit, but those are so rare that they are barely rolled once in the entire server every 5 syndie shifts. By the nature of how stealthy theft is, the round is barely effected by it, so a round where every syndie has a steal objective is straight boring. We have survival shifts and the actual thieves if you want a break from PvP. For example, a syndie stealing the hypo is annoying as a CMO, but you can still do your job just as effectively. It adds effectively no meaningful gameplay as sec, a competent syndie will have hidden it off their person or used a storage implant, and all sec can do is random searches that serve to annoy the rest of the crew. The round isn’t progressed in any meaningful way, and now that the syndie has completely their objective, the round stalls for them. They have 0 incentive to continue progressing the round as an antag, as any further action by them could lead to getting caught, so, they do nothing. Once enough syndies have done this, the round devolves into a boring extended shift until someone decides that evac is needed because nothing is happening.
This isn’t just conjecture, I see it all the time on Lizard of all places. You will play sec or med, and realize that the shift has been incredibly boring because NOTHING of note has happened. Some minor mobs, a wave or two of people, but no urgent revivals, no desperately searching maints for dead officers, basically 0 healing at all for a full medbay of doctors. The round ends, and you realize it’s because the syndicates silently completed all of their objectives, and had no reason to go loud.
The game NEEDS antags to be willing to kill or damage the station to be make a majority of the departments worth playing at the moment. Pure steal objective syndicates go against that. Now, even though I’ve said all this, there is a place for steal objectives with syndicates, especially with accompanying kill and escape objectives, as it makes the syndicate actually think about what they want to prioritize, and not fall into the same boring pure thief meta.
So overall, I agree with this PR. A balance should be struck, where most syndicates have to escape with 1 kill and 1 steal objective, some rarely having only thief, and some rarely having only kill. Both are equally boring after long enough, and should not be the norm.

This isn’t just conjecture, I see it all the time on Lizard of all places. You will play sec or med, and realize that the shift has been incredibly boring because NOTHING of note has happened

Whuh? Where did Lizard Sel Fantags go? 😕

please dont incentivise round removal even more

please dont incentivise round removal even more

Could always make cloning more common to compensate. :trollface:
Not saying to make cloning as trivial and/or cheap as it used to be. Then there was also the idea with centcom and mercenary reinforcements to generate more non-antags to replace dead crew.

Ah yes, exactly what SS14 needs, more round removal! /s

Please keep it constructive, folks. The Nth rendition of "don't do this" is not adding anything to the discussion, we heard it the first time.

Murder is fun for sec and a challenge for the syndicate, I find the most boring syndicate shifts when everyone has steal objectives. Another sec report of a head who lost an item due to thief gloves is so boring for security and does not produce anything interesting in shift. Oh no… my qm digigiboard… time to tell sec who can't do anything nor affects the shift at all. The purpose of antag is to push the round forward by creating drama. There is no drama in something most people don't know is stolen, on the other hand your engi buddy got stabbed in maints and was barely saved by the security team is much more dramatic.

Murder is fun for sec and a challenge for the syndicate, I find the most boring syndicate shifts when everyone has steal objectives. Another sec report of a head who lost an item due to thief gloves is so boring for security and does not produce anything interesting in shift. Oh no… my qm digigiboard… time to tell sec who can't do anything nor affects the shift at all. The purpose of antag is to push the round forward by creating drama. There is no drama in something most people don't know is stolen, on the other hand your engi buddy got stabbed in maints and was barely saved by the security team is much more dramatic.

On the other hand I offer you
"Yo did the captain get gibbed? By who?"

Get an emag, thief gloves, and a storage implant, and you get what you need every time.

I think that speaks more to how powerful those items are more than anything else. The thieving gloves and storage implant are easily two of the most powerful syndicate items in the game! 4 tc for basically free AA from capt or gun from a cadet is really powerful and there is basically zero reason NOT to buy the storage implant because of how powerful it is.
The thieving gloves can lead to fun situations so them being very powerful isn't a super big deal in my mind, but the storage implant is really not a fun item. It basically just nullifies the steal objectives! I think a good way to fix it would be to blacklist all steal objectives from it so you can't just hide whatever you want in there, but also significantly reduce its TC. Something like https://github.com/DeltaV-Station/Delta-v/pull/2286

DeltaV also has what I'd consider to be a better implementation, and that is to change kill objectives for non-heads to "teach them a lesson" which turns green if the person is just killed.  Heads still need to be round removed however. https://github.com/DeltaV-Station/Delta-v/pull/2184

I feel like this could work if the "Teach them a lesson" goes through, yeah

Well that's not what this PR is about tho, overall o do agree with his pr, rounds with a ton of steal objectives tend to be ultra boring, and going back to the teach them a lesson, that only works with MRP, because people on LRP remember who killed them.Also about "captain was gibbed an hour ago" well why does no one know who dis that to capi, thats just a skill issue, did capi go into maints alone with coords off?

Well that's not what this PR is about tho, overall o do agree with his pr, rounds with a ton of steal objectives tend to be ultra boring, and going back to the teach them a lesson, that only works with MRP, because people on LRP remember who killed them.Also about "captain was gibbed an hour ago" well why does no one know who dis that to capi, thats just a skill issue, did capi go into maints alone with coords off?

Not just captain, who's gonna think anything's wrong when janitor no. 3 suddenly goes missing? Nothing. People outside of sec shouldn't be particularly concerned with these things anyways unless they are a target or a witness. And dismissing a valid criticism of a specific scenario as a skill issue is denying the premise of the question as a whole.

Well i was talking about the captain's case, if you are talking over crew as a whole, i woulnt really mind making kill objs  more likely to target security/command rather than john station #4 passenger that is nowhere to be seen. If you are just a random crew, well you can have coords on and try to call on radio before dying if you are able, you are not the protagonist, is a multiplayer game, sometimes its your turn to die and other times is your turn to kill, its part of the game. Round removal is not an issue. (When its justified)

Thats another thing though, security and command are already massive targets for murder or round removal in any case so it would be a redundant change thats will only make people want to play them less. Round removal is certainly an issue and thats why its so rare to see and why its actually worthy of exeuction. steal objectives currently are fine because it gives security and command work to do while not causing chaos amongst the whole crew which is ideal.

The whole point of the game is chaos, its not a job simulator

I shouldnt have to tell you how wrong this statement is from anyones perspective

The point is a CERTAIN amount of chaos. Thus, it can not be said that more chaos is always objectively good

Along with the fact that the game also quite literally is a job simulator, pick a social deduction game at random and I'm sure it will have some form of job or town-sided tasks. This does not change for ss13.  Related issue is that some departments don't have enough to offer for some people, causing validhunting/powergaming/job abandonment.

Not just captain, who's gonna think anything's wrong when janitor no. 3 suddenly goes missing? Nothing. People outside of sec shouldn't be particularly concerned with these things anyways unless they are a target or a witness. And dismissing a valid criticism of a specific scenario as a skill issue is denying the premise of the question as a whole.

tbf if sec doesn't worry about why nobody has seen John Janitor in the last 20 minutes, they're not doing their job. Sec is supposed to protect NT assets, and that includes staff, so staff disappearing should be investigated. The problem of course is that this would involve talking to people instead of shooting their guns.

finally something to prove robustness as a syndicate (stealing/saving is boring)

As a result of the maintainers' discussions, it was decided not to merge this PR
we don't need to increase the count of round-remove situations. The described problem with the boredom of non-killing tasks should be solved by increasing the interest of existing non-killing tasks, or adding new ones.


German translation

German translation for Spot

You might want to wait with merging this PR until the Strings are updated as proposed in #149.

Thank you for the good advice.

No problem! :)
I spotted a typo in the last commit - it should be "Es" and not "ES" :see_no_evil:

What do you think about "Playlists" vs "Wiedergabelisten"?

I can live with both translations. Maybe it makes sense to take the wording from Spotify... So I'll leave the translation as it is for now. Or do you have bigger problems with it?

Nope, all good 👍🏼

Again: Thank you!

Thank you both for working together, this is great!!
I've pushed some updated msgids to master; if you're used to meson/gettext, you may run (in a configured project)  ninja spot-update-po -C target to update your po file automatically -- it will try to reconcile old msgids with new ones, watch out for changes (and for lines mentioning "fuzzy")

So, ich habe die Übersetzung mit der neuen po-Datei überarbeitet. Leider arbeite ich im Browser, sodass ich kein automatisches Update vornehmen kann (zumindest weiß ich nicht wie es geht - bin kein Programierer). Ich hoffe, dass das so auch in Ordnung ist.
@daPhipz: Hast du Zeit noch mal drüberzuschauen?

No worries about the automatic update, I'll add more documentation in the future. Thanks for this!
I'll let @daPhipz have a look then :)

Looks good to me! :+1:

Merging, thank you both once again!


docs: :pencil2: fix typo in readme

Closes #31

Hey @niteshseram,
Thanks a lot for taking the issue and making the app documentation better. Please feel free to pick up more, contribute more. I wish you a great #Hactoberfest ahead. You will be added to the contribution list right after we merge this PR. Don't forget to share about your contribution publicly and provide your support with a star ⭐.
Great work 🔥 !!!!

@all-contributors please add @niteshseram for Bug

Hey @niteshseram,
Thanks a lot for taking the issue and making the app documentation better. Please feel free to pick up more, contribute more. I wish you a great #Hactoberfest ahead. You will be added to the contribution list right after we merge this PR. Don't forget to share about your contribution publicly and provide your support with a star ⭐.
Great work 🔥 !!!!

Thank you so much. I love the project idea. I always struggle thinking about one. Also, the project uses the tech stack that I am really interested. Next.js + tailwindcss ❤


G3 325 api genset endpoint update

Prefer to see more tests for failure but I you know more about this than I :)

Thank yo for the review Matt. We have additional failure tests for this endpoint. This PR was mainly only adding additional parameters to an existing endpoint.


Exported var Arrays  in one object are connected/identical when initialized

Godot version
3.5.rc6 official
System information
Windows 10
Issue description
When defining a Reference/Node with multiple "export (Array)" vars it seems that its either the same Array gets set in all the exports or they are linked with each other, because when you just go and append to one of the arrays that get exported, suddenly all the arrays receive the value[s]. I can circumvent this by setting the array instead, but this seems like unintended behaviour.
Steps to reproduce
Open the Reproduction project and press Run and check the console output.
Minimal reproduction project
Array_Reproduction_Test.zip

Duplicate of https://github.com/godotengine/godot/issues/20436.


Forward algorithm issue - models/model.py

Hi,
I am running your code with some custom-created splits for the FSC dataset. Running lines 123 and 135 of forward() and forward_text(), respectively, in the models/model.py file causes the following error message:
Traceback (most recent call last):
File "train.py", line 35, in <module>
 runner.train()
File ".../alexa-end-to-end-slu/experiments/experiment_base.py", line 68, in train
train_loss, train_acc = self.train_step(batch)
File ".../alexa-end-to-end-slu/experiments/experiment_base.py", line 121, in train_step
 metrics = self.compute_loss(batch)
File ".../alexa-end-to-end-slu/experiments/experiment_triplet.py", line 111, in compute_loss
output_pos = self.model(input_text=batch['encoded_text2'],
File "/home/aostapen/miniconda3/envs/cs2/lib/python3.8/site-packages/torch/nn/modules/module.py", line 532, in __call__
result = self.forward(*input, **kwargs)
File ".../alexa-end-to-end-slu/models/model.py", line 108, in forward
return **self.forward_text(input_text, text_lengths)**
File ".../alexa-end-to-end-slu/models/model.py", line 137, in forward_text
**text_logits = self.classifier(text_embedding)**
File "/home/aostapen/miniconda3/envs/cs2/lib/python3.8/site-packages/torch/nn/modules/module.py", line 532, in __call__
result = self.forward(*input, **kwargs)
File "/home/aostapen/miniconda3/envs/cs2/lib/python3.8/site-packages/torch/nn/modules/linear.py", line 87, in forward
return F.linear(input, self.weight, self.bias)
File "/home/aostapen/miniconda3/envs/cs2/lib/python3.8/site-packages/torch/nn/functional.py", line 1368, in linear
**if input.dim() == 2 and bias is not None:**
**AttributeError: 'str' object has no attribute 'dim'**
This is because text_embedding is actually saved as the key 'pooler_output' returned in the BertModel forward(). Perhaps these lines should be:
_, text_embedding = self.bert(input_ids=input_text, attention_mask=attn_mask)[:2]
Thanks in advance.

Hi, do you get the complete.csv file? I meet the same problem as you.Could you please share your Snips dataset?


Incorrect behaviour

Steps -
Fields on my page {email, username, password, password_confirmation} & submit button

I have added validation rules on Registration page
Submitting form all correct entries but password confirmation
Got appropriate error message
from same page, removed all values
submit page

Expected - Required fields validation should be displayed and all input fields should be empty as submitted.
Actual - Got correct messages, however username & password field is showing old values.
I even tried refreshing page, now error messages are not displayed but username&password is as is.
Please confirm the behavior. Thanks!
Regards,
Anand Chavan

You might need to remove, presuming you're using the withInput() declaration:
From:
return Redirect::back()->withInput()->withErrors($e->getErrors());

To:
return Redirect::back()->withErrors($e->getErrors());


Add an additional timeslot for the poolpump

To allow the pool pump to run serveral times a day we need an additional pool pump timeslot.

Implemented timeslots as an array, so that we can add as many slots as we like
new desired property in device twin that defines the timeslot
New PoolPumpConfigSchema

Property
Type
Description
Example value

enabled
bool
pool pump enabled, used for stopping the pump at any time
true/false

timeslots
array
Timeslots for enabling pool pump

Timeslot Schema

Property
Type
Description
Example value

startTime
string
Time when the pump starts (format hh:mm:ss)
10:12:59

stopTime
string
Time when the pump stops (format hh:mm:ss)
10:20:00

@wlumetsberger  please use branch feature/issue-3-timeslot for implementation.


Update User Guide and Developer Guide

Update the description of Find Command in User Guide to improve clarity.

Update Activity Diagram of Find Command, and update the description of Find Command implementation.

Update the description of Find Command implementation, with relevant UML diagrams.

Codecov Report

Merging #268 (5126137) into master (9a18739) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #268   +/-   ##
=========================================
  Coverage     66.22%   66.22%
  Complexity      667      667
=========================================
  Files           107      107
  Lines          2203     2203
  Branches        259      259
=========================================
  Hits           1459     1459
  Misses          637      637
  Partials        107      107

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 9a18739...5126137. Read the comment docs.


west.yml: update to pick up the latest changes in the NXP HAL

Recent changes in the NXP HAL git repo require an update to the west.yml file.
Signed-off-by: Yves Vandervennet<EMAIL_ADDRESS>

Please add some information in the commit message of what changed in the HAL


Unsupported type using make_fself

I'm trying to code an homebrew for PS4 and I have succesfully set and compiled an example. But when I try to "fake sign" with make_fself by flatz I have this message: "error: unable to load elf file: ('Unsupported type.')
How can I fix this?
Thanks for all your work!

it seems the type for hito elfs is DYN = 0x0003, so we need to do appropriate changes to have it working as one of the above :)


Website Logo

I would like to add fabicons to your website which best fit for Portfolio-Collector, kindly give approval to proceed

I would like to add fabicons to your website which best fit for Portfolio-Collector, kindly give approval to proceed

Do it and put some screenshot of your work in the pull request comment


Updated TextFieldBoxes.java for opening keyboard on single tap on edi…

…ttext
Updated TextFieldBoxes.java for opening keyboard on single tap on edittext. Now keyboard opens on clicking anywhere on EditText view.

Please pull the keyboard fix as it's a major issue in terms of user experience..


$user variable empty in authenticated method when using the AuthenticatesUsers trait

Laravel Version: 5.6.8
PHP Version: 7.2.2
Database Driver & Version: mysql-5.7-19

Description:
When creating a controller for example called ApiLoginController, and creating an authenticated method using the AuthenticatesUsers, it seems it is impossible to access the $user variable passed usually in parameter. Same goes for Auth::user();
Steps To Reproduce:

Installing an empty Laravel project
Installing Laravel Passport
Installing Guzzlehttp/guzzle
Configurating the framework to make Laravel PassPort working
Creating a roule in api.php such as :

Route::post('login', 'Auth\ApiLoginController@authenticated');

Then in your web.php route file :

`Route::get('/', function () {
$http = new GuzzleHttp\Client;
$response = $http->post('http://hyvefivity.local/api/login', [
    'form_params' => [
        'grant_type'    => 'password',
        'client_id'     => '10',
        'client_secret' => 'eilJdC1Op3iA1yfCOJaAjYydnxdX67waTxuFPUKM',
        'username'      => 'my-email',
        'password'      => 'my-password',
    ],

]);
return json_decode((string) $response->getBody(), true);

});`

Then finally creating that ApiLoginController.php for example like this :

`namespace App\Http\Controllers\Auth;
use App\Models\User\User;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;
use Illuminate\Support\Facades\Route;
use Illuminate\Foundation\Auth\AuthenticatesUsers;
class ApiLoginController extends Controller
{
use AuthenticatesUsers;
protected function authenticated(Request $request, $user)
{
}

}`
Each time you'll try to fetch the / url, it will try to make a request to that method which will result in an ERROR 500 even if you don't put anything in the method, just because the $user parameter is not accepted.
I've created a topic on StackOverflow with all the problem description : topic

I'm not sure exactly what you are trying to achieve here. Api authentication is different from normal session authentication. The user is never really "logged in" to your application. In every request they have to send a token to verify their identity.
I don't see this as a framework related issue. You might get help a lot faster if you post your question in a forum like Laracasts here

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. Instead please try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.


Support Local Python Installations

On the version of rye I published there is currently no support for local Python installations. This is pretty much intentional because I really want the experience to not involve manually downloading Pythons. However there are also situations where this is necessary. In the original implementation I had, the way non automatically managed Python installations worked was with registering them.
Link a Python:
ln -s /path/to/python<EMAIL_ADDRESS>
And to pick a default distribution for what gets picked up when you write 3.11.0 another symlink does:
ln -s<EMAIL_ADDRESS>~/.rye/py/3.11.0

I however never really fleshed out the details and it was pretty brittle. In particular the only way I found to reliably pin a project to pypy for instance was to write pypy@madeupversion into .python-version which obviously is not very portable.

rustup's implementation might be a good reference here:
rustup toolchain link system /usr
rustup default system

I guess making the default configurable is enough to cover this issue; I can already register the system one.

find a good way to pin to specific installations without throwing stuff into the .python-version file that other toolchains cannot work with

What about doing something like
[tool.rye]
managed = true
standalone-toolchain = false

With a predictable resolution strategy for this setting. Could take inspiration from py.

Closing this for now. It's possible now to register custom toolchains, for improvements in the handling individual tickets should be opened.


Improve compliance with CFEP-18

We currently ship also static libraries in the libosqp package. This goes against CFEP-18 ( https://github.com/conda-forge/cfep/blob/master/cfep-18.md ). We should remove the static version of the library.

As pointed out in https://github.com/osqp/osqp/issues/309 and https://github.com/ros-industrial-consortium/tesseract_planning/issues/86, the issue is actually more serious: as osqp does not properly export symbols, the osqp.lib that we are currently shipping with the package it is not the import library of the osqp.dll shared library, but rather a static library, so all the downstream projects on Windows are actually linking the static library osqp.lib instead of the shared library osqp.dll.


internal version not bumped in v0.1.1

You forgot to bump the version for v0.1.1 inside libs/Version.cs which means the version check within the application fails and always wants to update even though it is already the newest version
Perhaps the release strategy should be improved a little and, for example, a check should be built into the release CI to check whether the version matches the git tag or insert the git tag version into the code :)

This is fixed in 0.1.2.  Will be releasing this soon (tm).


Iceberg REST configurable OAuth Endpoint

Enhancement
Recent changes in Iceberg REST deprecated the oauth2 endpoint as part of the REST spec. Instead, the OAuth2 endpoint of the IdP should be used.
Extract from the updated spec:

The oauth/tokens endpoint is DEPRECATED for REMOVAL. It is not recommended to implement this endpoint, unless you are fully aware of the potential security implications.
All clients are encouraged to explicitly set the configuration property oauth2-server-uri to the correct OAuth endpoint.

It would be great to make the oauth2 server uri configurable by adding a iceberg.catalog.oauth2.server-uri configuration for the Iceberg Rest Catalog.
While on it, it might make sense to also rename the credential configuration property from iceberg.catalog.credential to iceberg.catalog.oauth2.credential as oauth2 might not be the only auth mechanism in the future. trino, presto and spark all use a nested oauth2 prefix, which might be a good orientation. They are also more precise in naming the properties iceberg.rest.xxx.xxx, as iceberg also supports other catalogs than rest.

You are right, we should use oauth2 prefix in the future.
Did trino implement oauth2.server-uri parameters? I didn't have seen this parameters in their doc.

@Smith-Cruise no, trino is also missing the parameter currently but an issue is also open: https://github.com/trinodb/trino/issues/23086

This issue could probably be implemented together with https://github.com/StarRocks/starrocks/issues/50518

fixed in https://github.com/StarRocks/starrocks/pull/53748


[Question]: Is T8502 Firmware version <IP_ADDRESS> supported?

Ask your question
Device: Eufy Security lock T8502
Firmware version: <IP_ADDRESS> (2023-06-10)
Hi. I got a new security lock T8502, but in the "Eufy Security" integration, I do not see the device whereas my eufy app shows it. There is one device in the integration that says "UNSUPPORTED", but not sure if that is the lock.
Is there support for this version yet?
Thanks,
Vivek Vakkalanka.

@bropat Can you please take a look at this when you get a chance?

I am having the same issue as well. I have 2 T8502 (Branded as SmartLock E110) with firmware E110

This device is not supported as specified here.
There is already a feature request here.


[Bug]: Import of file containing OAuth API DS behaviour is incorrect

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I try to import a json file containing an OAuth DS, the import flow is not consistent and throws errors.
Steps To Reproduce

Create a new workspace on appsmith
Import the attached file [rename to json]
Try to configure the OAuth DS and authorize it with valid credentials.
Issue:1. It takes user to oauth review page, and gives an error "Something went wrong"
Issue:2. It does not redirect user back to the home page import modal, to complete configuring other DS
Issue:3. On manually clicking the Homepage, the import modal comes up with authorization failed error though authorization has succeeded as per the green check mark

Note the issues in the video below:
https://ddpa7mfuvv.vmaker.com/record/uAnVNacDguldMSwM
OldApp_DSTesting1.9.24 (1) copy.txt
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud

If the DS modal shows up, Import-Export is done.


How to hook on preexisting monaco editor

Hello,
Thanks for the project,
I'm currently using it for a small Web Editor: https://draco-lang.github.io/
As you can currently see, the loading time right now is very slow due to blazor (and me shipping untrimmed DLLs, but that another problem).
One easy fix to remove the loading time I want to do is to display the text editors, before blazor is loaded.
The issue is that now my editors are not created by BlazorMonaco, and I don't see a way to wrap an existing editor.

I dropped this dependency and wrote more logic in JS instead.


napalm: init at 3.3.1

Motivation for this change
Adds Python library and client for vendor-independent configuration of network devices. These can help nix-based network infrastructure projects.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@IvarWithoutBones Thank you for reviewing. I've enabled all the tests and added myself to maintainers.


Image editor doesn't close when exiting a skillmap activity

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Go to a skillmap, open an activity, open the image editor
Draw something, exit the skillmap (using the back arrow)
Open a different activity
See that the image editor is still open

Still repros. Good to fix by next release.


Not able to get on-prem K8S logs in LogAnalytics Workspace

I am trying to get the logs from on-prem K8s cluster. The cluster is succefully on-boarded to Azure portal but I cannot see the logs though the script is successfully executed at my end:

The cluster is visible in the Azure portal, however no logs are available:

I tried to check in the Log Analytics workspace as well, it showing the logs from my IBM Cloud cluster only:

Please let me know how to get the on-premise K8S logs in the Log Analytics workspace.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 82ed00b7-74dc-7d9d-d9a4-11b0b9c8932c
Version Independent ID: 4368e324-40e1-6e28-bc61-63b555e0218d
Content: Configure Azure Arc enabled Kubernetes cluster with Azure Monitor for containers - Azure Monitor
Content Source: articles/azure-monitor/insights/container-insights-enable-arc-enabled-clusters.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@sormita Thanks for your comment. We are actively investigating and will get back to you shortly.

Is there an update to this issue?

Is there an update to this issue?

@sormita ,

Can you please share  o/p of following commands to understand what’s going on here
kubectl get pods -n kube-system
kubectl  logs  -n kube-system
exec into one of the pod
kubectl exec -it  -n kube-system  bash
cat /var/opt/microsoft/omsagent/log/omsagent.log  # check if you see any errors

@sormita Can you please provide requested information as mentioned in previous comment?

@sormita Just following up to check if you had chance to check previous comments?

@SwathiDhanwada-MSFT my on-prem cluster is currently down, I am trying to create another on-prem cluster, once it is available I will try out the suggestion and let you know.

I am getting error like below for logs of api-server pod of kube-system namespace:
W0713 04:33:13.786790       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.792382       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.793442       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.802782       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.852681       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.856110       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.904550       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:13.921209       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
I0713 04:33:14.003902       1 clientconn.go:882] blockingPicker: the picked transport is not ready, loop back to repick
W0713 04:33:14.017763       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:14.022062       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
W0713 04:33:14.038608       1 clientconn.go:1208] grpc: addrConn.createTransport failed to connect to {https://127.0.0.
:2379   0 }. Err :connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:2379: connect:
onnection refused". Reconnecting...
output.txt
Attached is the full output of the logs. Do you think this is the reason for the error?

Hi, @sormita, looks like you have issues configuring etcd as described here https://github.com/kubernetes/kubernetes/issues/19235.
If you are planning to create dev/test on-prem cluster you can use kind https://kind.sigs.k8s.io/docs/user/quick-start/

@ganga1980 Thank you for your valuable insights.
@sormita Checking if you need further assistance on the issue.

No you can close the issue.

Thanks for your response @sormita.We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


CI for Node

game.test.js

Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not


OG Image generator

User story
As a content editor or marketer,
I want to be able to easily create and add OG images to a web page,
so my audience sees a preview / inspiring image when a link to my page is shared somewhere.
Possible solution
Use vercel/og Pages Plugin, og-worker package or something similar to generate OG images for web pages. During build- and/or run-time. With one or more templates for the images.

@jbmoelker would you consider using bannerify.ai instead? Its underlying is also powered by Satori (which powers astro-og-canvas)
Would love to chat more if you're interested x.com/duc__an

Btw I have this open source that you might find useful https://github.com/ducan-ne/ogcool

Stumbled across a perfect reference: https://kevinkipp.com/blog/on-the-fly-open-graph-images/ generating OG Images using Cloudflare browser rendering (Puppeteer) and Cloudflare R2 (for caching) within an Astro project. The blog also links to all the code in a public repositories ⭐


LambdaCacheRefreshTask does not seem to refresh the cache

LambdaCacheRefreshTask does not seem to refresh the cache, this causes problems downstream if the AWS Lambda Deployment stage is followed by a AWS Lambda Route stage where the lookup for Newest version returns the old cached version.
Looking at the clouddriver logs I cannot see a on-demand cache refresh being triggered. I switched to calling clouddriver directly in this branch and added a cache refresh after publish. After this change I do see on-demand cache refresh being triggered in clouddriver logs. Not sure why cacheService.forceCacheUpdate does not trigger an update.
However it does not seem like the cache is refreshed in time for the AWS Lambda Publish in some cases, even with the 20s wait in LambdaCacheRefreshTask.
My current workaround is to add a LambdaWaitForCachePublishTask which simply waits until the published version is returned from clouddriver. The changes are available in this branch. This solves the problem for my usecase, I'd be happy to make a PR if my changes makes sense.

Sure, send me a PR...

Sure, send me a PR...

Marking as closed after merging changes to master...

Marking as closed after merging changes to master...


Api diff between preview8 and RC1

cc  @pgovind @eerhardt @dotnet/ncl  @tommcdon @krwq @ajcvickers @cheenamalhotra @david-engel @safern @tarekgh @stephentoub @bartonjs @tannergooding @CarolEidt @echesakovMSFT @carlossanlop @Anipik @steveharter @ericstj
@pranavkm @Tratcher @Pilchie @davidfowl

System.Globalization, System.Diagnostics, System.Resources and System.Text LGTM

Some of these changes are breaking, with parameters to public methods being renamed.  Looks like these came in https://github.com/dotnet/runtime/pull/39741/files#diff-6581d2a62a9b3fdecfc846a0178f5e2174d6bff734f2cb830412a3c35df4f551R13
and that the breaking change process was followed, but I'm a little surprised we decided to do that.


NoMethodError: undefined method `fetch' for #Page:0x000055d0b9436d60

https://sentry.io/starkast/wiki/issues/492301922/
NoMethodError: undefined method `fetch' for #<Page:0x000055d0b9436d60>
  tilt/template.rb:170:in `call'
    method.bind(scope).call(locals, &block)
  tilt/template.rb:170:in `evaluate'
    method.bind(scope).call(locals, &block)
  tilt/haml.rb:24:in `evaluate'
    super
  tilt/template.rb:109:in `render'
    evaluate(scope, locals || {}, &block)
  sinatra/base.rb:823:in `render'
    output          = template.render(scope, locals, &block)
...
(80 additional frame(s) were not displayed)

NoMethodError: undefined method `fetch' for #<Page:0x000055d0b9436d60>

I think this change broke this. Sounds like we were relying on a "bug".


KeyError: 's3prl.upstream'

Hi,
I am working on notebook - ESPNET Noisy SLU Demonstration
https://colab.research.google.com/drive/14nCrJ05vJcQX0cJuXjbMVFWUHJ3Wfb6N?authuser=1
for the cell 13 I am getting error - KeyError: 's3prl.upstream'
This part of code -

from espnet2.bin.asr_inference import Speech2Text
speech2text_fsc = Speech2Text.from_pretrained(
asr_train_config="/content/fsc_model/exp/asr_train_asr_hubert_transformer_adam_specaug_raw_en_word/config.yaml",
asr_model_file="/content/fsc_model/exp/asr_train_asr_hubert_transformer_adam_specaug_raw_en_word/valid.acc.ave_5best.pth",
# Decoding parameters are not included in the model file
nbest=1
)
##########
Can you please help. Thanks.

Sure, let me check

Sure, let me check

Thanks a lot, Updated the URL.

Hey, when I want to reproduce the issue and click the run all button, I encounter this error which I think is not related to S3PRL. Could you help to adjust to colab to directly reproduce the S3PRL error?
Many thanks!!

Oh never mind, after I run the colab again it works now. It seems the first run all might crash somehow. Might be an useful information to you

I think it will work after switching to an older version S3PRL. Since S3PRL now depends on fairseq, hence you will need the correct fairseq version (fixed in our setup.py) to make S3PRL work. This fix also appear in the latest ESPNet installation:
https://github.com/simpleoier/espnet/blob/74965106a6ca561545f4821812e35eca0dd7e4b4/tools/installers/install_s3prl.sh#L54-L57
Either switching to an older commit of S3PRL, or upgrade the fairseq to align with the commit in our setup.py. I think both ways work. In your colab I try to use an older S3PRL and now it works

Thanks a lot, this works. Closing the issue.


Support max_items query parameter

Motivation
SM list API must handle the max_items query parameter to determine the maximum number of items to return.
This max_items parameter must not be greater than an internally configured value.
Approach
Parameter max_items introduces which means maximum number of items to return in the response.If the client sets max_items to 0, it is assumed that the client is only interested in the total number of items.

Coverage increased (+0.01%) to 89.638% when pulling 21024db18d7c5cc3c3348625006a29a4960add98 on max-items-param into f479b10d3d8570afeef586a0aaab276de5084e43 on paging-topic.


Option of relative double precision for doubles

First, thanks for a great tool!
We have some objects we want to compare where the order of magnitude of the double values can vary by quite a significant amount. One property might be in the order of magnitude of e+11 and another property e-6. For some applications we want to check that the values are within an acceptable tolerance, say within 6 significant digits, rather than exact. With the current
https://github.com/GregFinzer/Compare-Net-Objects/blob/f578c6158983365a4b580dab43b847aee4e9a6aa/Compare-NET-Objects/ComparisonConfig.cs#L537
and
https://github.com/GregFinzer/Compare-Net-Objects/blob/f578c6158983365a4b580dab43b847aee4e9a6aa/Compare-NET-Objects/TypeComparers/DoubleComparer.cs#L33-L48
This seems not possible for a single object carrying both of those properties.
I take it from https://github.com/GregFinzer/Compare-Net-Objects/issues/148 and https://github.com/GregFinzer/Compare-Net-Objects/commit/d86e74fe58f50642ec90de11fb4f58cb3da41533 that it was an intentional change, which I can understand as well.
Wonder if it would be possible to get a boolean setting added, like RelativeDoublePrecision added to the config that could allow for the  behaviour previous to https://github.com/GregFinzer/Compare-Net-Objects/commit/d86e74fe58f50642ec90de11fb4f58cb3da41533 to be used?
Also related to https://github.com/GregFinzer/Compare-Net-Objects/issues/182

I think the best course of action would be for your project to use a custom comparer.  That way current double functionality stays in tact.
See:
https://github.com/GregFinzer/Compare-Net-Objects/wiki/Custom-Comparers

Thanks, will have a look at the custom comparers, think that will resort our issues!


WIP: Mute/unmute the user

Closes #2214
The icons have become more interactive

Replaced mute icon with blocked eye icon

I closing the PR. I think it's better to wait for new fields to appear in the DB. https://github.com/FAForever/downlords-faf-client/issues/2214#issuecomment-1027266196


feat!: Has* game mixins return bool

Description
HasTappables, HasHoverables and HasDraggables now returns a bool in their methods so that the user can know whether their components marked the event as handled.
Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples.

Breaking Change
The breaking change is at the method signature. All methods overridden have to add the boolean return value.
Previously:
void onTapDown(int pointerId, TapDownInfo info) {}

Now:
bool onTapDown(int pointerId, TapDownInfo info) {
  return true:
}

[x] Yes, this is a breaking change.

Related Issues

@spydon API details aside, I guess the big question is this: How do we even know that an event was handled?
Say, HasDraggables.onDragStart returns the result of propagating handleDragStart(). However, the handleDragStart returns true/false indicating whether or not the event ought to continue being propagated to other components. Which is not the same as "being handled".

That's true, maybe we should separate those two things. Then you'd just set the handled flag in your methods instead then, and the booleans would just be for propagation. I still don't like having the mutable events, but I don't see a better option for passing that information through the flow.


Unique table identifier?

I am writing some code to convert a sol::table into a C++ structure, and I want to prevent infinite recursive loops from forming when a table has itself somewhere in one of its own keys, or in the sub-structure.
What can I use as a unique identifier for the table? I tried registry_index() but I thing this value changes as I am traversing through the data...

Not 100% sure, but fairly sure there is no unique value you can get from the lua API.
You would need to compare the values using lua equality operation to determine if two tables are equal (lua_rawequal). You can then generate a running number yourself to keep track of the references.
Another way is to instead keep track of the data in lua. With a lua table you can store the table itself as the key and then later when you want to check if a table was already seen, you just look it from the table (example).
A quick and dirty alternative might be to use tostring() on the table. This will return a string similar to table: 0x1b85730 where the value is the address of the table I recon. However, a table can override the tostring operation with a metafield __tostring .


Explorer: local file view

It is slightly inconvenient to have to make a remote to view local files...
In the explorer remote selection, could we make one which opens a local backend? Or this could be a different button?
Local backends can be specified by path, so "/" will work.  We could get rclone to return the path of the users home directory if we want.  Or maybe we could use a browser choose a directory function - I'm not sure.
So in remote terms you can use fs="/" remote="" to view the root directory.

We can prompt the user to create a local config if it is not present in the config list. Or we can provide a button to open local config, and upon click, we can ask the user to select a local path, or take "/" as default. Taking "/" might not be convenient for everyone though.

We can prompt the user to create a local config if it is not present in the config list.

That isn't the way rclone is normally used - people don't create local remotes normally, they just use the path directly.

Or we can provide a button to open local config, and upon click, we can ask the user to select a local path, or take "/" as default. Taking "/" might not be convenient for everyone though.

I was thinking you'd probably want a few options in the list...
So

/ (or C:, D: etc on Windows)
the users home directory
the current directory that rclone was started in if different to the above
the temporary directory (maybe)

I don't know whether you can work these out from the browser - I don't think so.  However it would be trivial to create an api to return a list of interesting local paths.
Asking the user to choose with / is the default is not a bad solution to be getting on with.

With the new box to type in, could we allow local paths to be typed in there too?  Eg

It seems like a good idea. But it won't be intuitive.
On Mon 10 Jun, 2019, 4:49 PM Nick Craig-Wood<EMAIL_ADDRESS>wrote:

With the new box to type in, could we allow local paths to be typed in
there too? Eg
[image: image]
https://user-images.githubusercontent.com/536803/59192309-f210c600-8b79-11e9-815e-74c69fcb29df.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/negative0/rclone-webui-react/issues/6?email_source=notifications&email_token=AC5LNYB5HIC3RYU27I2ITRLPZY2DDA5CNFSM4HVBS3MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXJTRTQ#issuecomment-500381902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC5LNYHOCBB5TZBN3RTVZLDPZY2DDANCNFSM4HVBS3MA
.

It seems like a good idea. But it won't be intuitive.

In rclone terms remote:path and /tmp/localdir are the same things.  /tmp/localdir is a special way of writing a local remote with a root of "/tmp/localdir".  So I think that would make sense to existing rclone users.
So you can do
rclone sync remote:path /tmp/localdir
rclone sync /tmp/localdir remote:path
rclone sync /tmp/localdir  /tmp/localdir2
rclone sync remote:path remote2:path2

And all of those are rclone sync source_remote destination_remote

Hello - how do i access a local path in the rclone web UI? I'm running the rclone plugin for unraid and accessing via "http://<IP_ADDRESS>:5572/#/remoteExplorer"

Hello - how do i access a local path in the rclone web UI? I'm running the rclone plugin for unraid and accessing via "http://<IP_ADDRESS>:5572/#/remoteExplorer"

Bump - can someone help me with this?

Hello - how do i access a local path in the rclone web UI? I'm running the rclone plugin for unraid and accessing via "http://<IP_ADDRESS>:5572/#/remoteExplorer"

Bump - can someone help me with this?

Hello - how do i access a local path in the rclone web UI? I'm running the rclone plugin for unraid and accessing via "http://<IP_ADDRESS>:5572/#/remoteExplorer"

Bump - can someone help me with this?

Hello - how do i access a local path in the rclone web UI? I'm running the rclone plugin for unraid and accessing via "http://<IP_ADDRESS>:5572/#/remoteExplorer"

Bump - can someone help me with this?

if you create local "remote" in config like c:\ = hd1:  you can open hd1: in web and get access to local files. Is it what you need?


DAPR 1.6 uses Spring Boot Starter Web with vulnerability

Ask your question here
DAPR 1.6 uses spring-boot-starter-web/2.3.5.RELEASE which has vulnerability. Any plans to upgrade the Spring Boot Starter Web version without vulnerability as many organization don't allow  any artifacts with open vulnerability ?

@sujitp149 Thanks for reporting this. We can try bumping the version to 2.7.3. Please feel free to submit a PR, we can see if that is breaking anything.

Out of curiosity: why is spring-boot-starter-web included and not spring-boot-starter-webflux, as all methods seem to be non-blocking?

Great point, I think we should offer webflux but it should probably be a new artifact so it does not break existing users.

I have added the PR above to the release. I will keep this open to confirm that it will really remove the vulnerability. If not, we will need to upgrade to a new major version in the next release.


Running Kubeaudit as a Cronjob and Kubeaudit's Contianer image

ISSUE TYPE

[x] Bug Report

BUG REPORT
SUMMARY
I don't seem to find a kubernetes manifest file over here. I would like to run Kubeaudit as a Cronjob for example. Is this documented somewhere?

Hi @ramesh-ramani,
We don't currently publish a docker image, but you can make your own as follows:

Clone the repo
Build kubeaudit: make
Create a docker image: docker build . -t kubeaudit

You can then either push this image to Dockerhub or your own registry and set it as the image in the Cronjob template, something like
apiVersion: batch/v1beta1
kind: CronJob
spec:
  jobTemplate:
    spec:
      template:
        spec:
          containers:
          - image: <your_registry>/kubeaudit
            args: ["all"]

I'm going to see about publishing an official kubeaudit image.

Hi 👋
Tried this out and worked nicely with the docker pull request.
One thing missing from the YAML above is the RBAC config. Without any pointers I've settled on the config listed below. (For a single namespace audit)
Is there a way to reduce the permissions to the actually needed rules from the code? Having a * on both apiGroups and resources makes me feel a bit uneasy 😬
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kubeaudit
  namespace: default
---
kind: Role
apiVersion: rbac.authorization.k8s.io/v1beta1
metadata:
  name: kubeaudit
  namespace: default
rules:
  - apiGroups: ["*"]
    resources: ["*"]
    verbs: ["get", "list"]
---
kind: RoleBinding
apiVersion: rbac.authorization.k8s.io/v1beta1
metadata:
  name: kubeaudit
  namespace: default
subjects:
  - kind: ServiceAccount
    name: kubeaudit
    namespace: default
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: kubeaudit
---
apiVersion: batch/v1
kind: Job
metadata:
  name: kubeaudit
spec:
  template:
    spec:
      restartPolicy: OnFailure
      containers:
        - name: kubeaudit
          image: shopify/kubeaudit:v0.11.0
          imagePullPolicy: IfNotPresent
          args:
            - "all"
            - "--namespace"
            - "default"
      serviceAccountName: kubeaudit

Hi @J12934 , glad it worked for you! Given the resources kubeaudit currently supports, I believe you'll need these RBAC permissions:
rules:
  - apiGroups: [""]
    resources:
      - pods
      - podtemplates
      - replicationcontrollers
      - namespaces
    verbs: ["get", "list"]
  - apiGroups: ["apps"]
    resources:
      - daemonsets
      - statefulsets
      - deployments
    verbs: ["get", "list"]
  - apiGroups: ["batch"]
    resources:
      - cronjobs
    verbs: ["get", "list"]
  - apiGroups: ["networking"]
    resources:
      - networkpolicies
    verbs: ["get", "list"]

Note that we are planning to add support for more resources that may break your job if you update kubeaudit, specifically

serviceaccounts in the core API group
jobs in the batch API group

come to mind.
I haven't had a chance to test this if you wouldn't mind letting me know if it works for you!

This seems to be working great, thank you very much 👍 @genevieveluyt

Opened a PR to publish a Docker image during CI/CD: https://github.com/Shopify/kubeaudit/pull/295

@J12934 Full instructions on how to run kubeaudit as a job are available now! https://github.com/Shopify/kubeaudit/blob/master/docs/cluster.md

@genevieveluyt Awesome work, thank you 🙏🚀


Add ability to manage groups

The recent update to the Bots API introduced ability to manage members of group chats: currently you can kick people from a group and unban them from a supergroup.
The API I want to use:
chat.ban(user_id)
chat.unban(user_id)

Good!


Add Vector read support

Requested by @elfwine in #13

Committed the Vector read in its current state.
Decided to add Vector properties to Array, so I can recognise a Vector and write one.
Will post more details in a bit ...

Done for now. Might look at ArrayBuffer for fixed vectors in the future.


Adjust samples to tag with "Microsoft 365 Copilot" to align with Sample Gallery

Need to use this tag under products, then scripts should appear under the Copilot for Microsoft 365 section.
"Microsoft 365 Copilot"
Refer to my sample in Teams DEV for reference; https://github.com/pnp/teams-dev-samples/blob/main/samples/msgext-copilot-message-centre/assets/sample.json

Closing as complete


{AzureContainerInstances} fixes Azure/azure-cli#24074

fixes Azure/azure-cli#24074
The az container attach command description is not detailed and its not clear. This PR includes the detailed description for the command.
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AzureContainerInstances

/azp run


Refactor

A proposal to a refactor of the existing code.

Splits up the functionality (methods, logic etc)

More readable (?)

Improvements to some logic

Now, mod.ts still calls the required logic, but the proper logic is split up
New File Structure
/
    src/
        commands/
            ...
        options/
            ...
        utils.ts
...

Address all of your comments @crookse with the above new commits

Alos stripped out utils.ts completely

Just a little fyi. Since I come from Java, you can only have classes, interfaces, enums, etc, in each file. That's why there are some conventions of naming IModule when referring to the interface and Module when referring to the class. Since in TypeScript we can have multiple things inside the same file, you don't require to use that convention.
I'm fine with having interface+types+enums+class definition in the same file if they refer to the same thing. Your choice 😃


Make for loop in InputEvent more consistent and modernized

Summary of the Pull Request
In lInputEvent, there are two for loops. One is a range based loop operating on "records", and the other is a classic for-loop doing the same. For consistency, the for-loop was changed into the more modern variation via a compiler refactoring, which has the exact same behavior as the other for-loop in the main function.
Yes, of course this was tested manually and with the unit tests.

References

PR Checklist

[ ] Closes #xxx
[X] CLA signed. If not, go over here and sign the CLA
[X] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Unit testing passed. In addition, for the manual validation tests, I compared the output for sample values between the two loops ensuring the same results.

Thanks!


chore: release 0.110.0

:robot: I have created a release *beep* *boop*
0.110.0
google/cloud-bigquery 1.11.0
Features

Update BigQuery's default endpoint (#2305) (ee34ce2)

google/cloud-bigtable 1.1.0
Features

Add IAM management methods. (#2298) (8858563)

google/cloud-speech 0.27.0
Features

Add SpeakerDiarizationConfig to v1 client. (#2300) (ad69110)

google/cloud-web-risk 0.3.0
⚠ BREAKING CHANGES

Add threatTypes to searchHashes request signature. (#2309)

Bug Fixes

Add threatTypes to searchHashes request signature. (#2309) (8ed8b72)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/google-cloud-php/releases/tag/v0.110.0


The 'Emulator & Attached' node is not localized

Storage Explorer Version: 1.25.0-dev
Build Number: 20220628.2
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.4 (Apple M1 Pro)
Language: All languages
How Found: From running test case
Regression From: Previous release (1.24.3)
Steps to Reproduce

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '中文（简体）' -> Restart Storage Explorer.
Check whether the 'Emulator & Attached' node is localized.

Expected Experience
The 'Emulator & Attached' node is localized.
Actual Experience
The 'Emulator & Attached' node is not localized.
Additional Context
The nodes (Attached Containers) and (Emulator-Default Ports) (key) are not localized.

Due to moving of strings. I will keep an eye on this.

@v-kellyluo it looks like these strings are now localized. I'm going to close this issue.

@MRayermannMSFT, verified this issue on the build main/20220719.8:

The 'Emulator & Attached' node is localized.
The nodes (Attached Containers) and (Emulator-Default Ports) (key) are not localized.

The strings are now translated and I've updated the package.

Verified this issue on the build main/20220719.17. Fixed.


Is this still being maintained?

If not, would you be willing for me (or someone else) to pickup the project?

https://chrome.google.com/webstore/detail/github-vscode-icons-updat/amefmmaoenlhckgaoppgnmhlcolehkho


Jomar ppc long form update

@Katiemoser  kindly see attached for the updates

Missing icons on features can be fixed on CMS
Footer copy needs to be updated as well as logo to give contrast on logo also can be adjust on CMS

@ardeay  @Katiemoser
This is complete and ready for merge once approved.


Add test verifying checks are run in parallel

This adds a test which covers the functionality added with #3. I verified the test by cherry-picking it onto a branch off master before #3 was merged and watching it fail.
https://jira.mesosphere.com/browse/DCOS_OSS-2239

@gpaul @amitaekbote PTAL, thanks!


BAU: Remove unnecessary exceptions from Certificate Chain Validation filter

Exceptions are only for error reporting and handling and should not be used for other purposes like control flow.
Authors: @adityapahuja

This seems to be creating some quite large and complex methods - it's good to get rid of exceptions as control flow, but I'm not sure about inlining too much, we still want to keep methods small - see https://ncrcoe.gitbooks.io/java-for-small-teams/content/style/300_keep_methods_small_and_simple.html

Sorry, that comment may have been overly negative - I quite like it overall, a definite improvement.

The methods seem fairly self-contained and appropriately named. I think this is just the sort of thing you can't really split up any further without introducing some cognitive overhead. I think it's more important to encapsulate behaviour over minimising LOC per method.


python3Packages.wandb: 0.15.11 -> 0.15.12

Motivation for this change
Upgrades python3Packages.wandb from 0.15.11 to 0.15.12
This PR was automatically generated by nixpkgs-upkeep.

CI workflow that created this PR.
Internal tag: dac6d39fb3b6f8b5661553866d14d3a0.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Closed by #280986.


Make connect buttons keyboard accessible

Making the two connect elements traditional buttons will make them keyboard accessible :)
https://user-images.githubusercontent.com/46655/156001609-acbca746-f4f1-42eb-b026-471f81342e64.mp4

Created a separate PR in order to be able to generate the build preview
https://github.com/matter-labs/zksync-wallet-vue/pull/236


--dryRun option of julieOps does not detect schema changes & compatability with schema registry.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I am trying to manage schema in schema registry through julieops.
As I continued testing, I realized that the dryrun option for schema changes didn't work as I expected.
Perhaps julieops is a problem caused by not tracking the schema state.

Even though the schema has already been applied, it looks like the schema is being recreated.

Apply schema to topic
Without changing anything, add the --dryrun option again to check whether the schema is applied.
The schema has already been applied, but I can see com.purbon.kafka.topology.actions.topics.RegisterSchemaAction log

Schema compatibility cannot be tracked.

It would be nice to be able to track schema compatibility in dryrun .
I found that it aws discussed at https://github.com/kafka-ops/julie/issues/223

Describe the solution you'd like
A clear and concise description of what you want to happen.

Track the schema state, so that can track what changes are made during --dryrun option.
could check the compatibility of the schema, maybe compatibility can be checked with https://docs.confluent.io/platform/current/schema-registry/develop/using.html#sr-test-compat-latest api.

This is the type of functionality that was proposed in an old PR (254).
The organisation that I work for is using an old fork that includes that PR.
@purbon this is my explanation for not just upgrading to the latest release, that you asked about on another issue that seems to have been deleted.

Hi @Sounie that is fine, but for this purpose, backporting will not really help. I would suggest you can grab the related commits from here and just use it in. What do you think?


Make name, price and what to export configurable

fixed

Hi,
at first glance, your implementation seems to work correctly. So far I just found the typing error, which you have already fixed.
I will let you know when the tests are successful and I am ready to merge your pull request.
Thank you for contributing to the source code of our shopware connector.
Best regards,
Ioana

Hi,
Wenn der Import von Name und  Beschreibungen des Artikels konfigurierbar sein müsste, dann müsste die Übersetzungen auch dementsprechend angepasst werden.
Schöne Grüße
Ioana

@ioanamoldovan wir setzen diesen patch jetzt erfolgreich in unserem Produktivsystem ein. Was müssten wir noch tun damit er gemergt wird?

Hi,
i really would like to merge your changes for the next major release. That release will include Shopware 5.2.x compatibility. Could you check your changes against the development branch and a Shopware 5.2 installation if they still work as expected?
Best regards,
Jochen

@jochenmanz unfortunately we are not in the position to test this change against a 5.2 system as we would need a 5.2 test system. We currently do not dispose neither of time nor resources to setup a 5.2 system. This may be possible in Q1 2017, but as all the functionality is only dependent from the plugin in itself and not the shopware version it should be working as expected at least against the API 115.


Correctly sync all cores

Closes https://github.com/hermitcore/libhermit-rs/issues/506.
This requires uhyve 0.1.3 for SMP.

bors try

bors r+


Refactor concat tests.

small cleanup in test_concat.py

Oh nice! I didn't mean you should to change it for this PR, hope that was clear. Thank you!

Yeah it was. I just thought it was a good idea :)


Increase machine size for RBE prod pools

Type of Request
infra task
Infrastructure Environment
rbe, devicelab
What is happening?
The performance associated with current RBE machines are not at par with goma.
@zanderso has requested that the RBE prod project machine type be upgraded to e2-highmem-8 instead of the current e2-standard-2.
Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

New machine types have rolled out.
FYI, @zanderso we are now at our quota limits. So if you foresee any additional requests in terms of machine types or quantities, please let me know, so that we can setup the necessary quota limits.

Thanks!


Rewrite float-array addition/subtraction to use a single function

Currently, subtraction is done as https://github.com/apytypes/apytypes/blob/5055aa33857ba29afecbfc8551f9db6a9b125b35/src/apyfloatarray.cc#L454
However, this causes a copy of the array and therefore a performance penalty for subtraction compared to addition, see top plot below.
It would be useful to have a single method called with a bool for add or subtract to alleviate this.

This issue is fixed with #574.


Documentation: remove bash-test framework references

Also do some minor fixups of grammatical errors, and some rewording to make sentences more clear.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>Fixes: #2546

test-docs-please

@cilium/contributing please review.

Merging, as no feedback has been received as requested, but have received approval, and because we need the doc fixes for 1.0.


Display each key that is typed in a corner of the screen

Description of the new feature / enhancement
Every time any key or key combination is pressed, the key (or key combination) should be shown in a little semi-transparent window in one corner of the screen. It should fade away after a configured amount of time.
Scenario when this would be used?
This would be very helpful when showing someone something in a screen share video call. Or when recording tutorials. For example, explaining a user how to use a specific software. Then the other user would instantly see which keys I'm pressing.
This would work great together with Mouse Higlighter. Mouse Higlighter makes it obvious when I'm clicking something. My feature would make it obvious which keys I'm pressing.
Supporting information
Like in this video at 7:55 (watch the bottom left corner)

Duplicate of #981

Thanks for your suggestion! This issue is a duplicate.
I'm linking this issue to #981 to centralize the discussion thread, please go to that thread for discussion.
Thanks for your help to make PowerToys a better piece of software!


Undefined method 'load_path' for Ceedling:Module

I have a ceedling project or setup that have been working. For some reason (after updating to Ubuntu 24.04) I had to update ruby. After the Ruby update I run in to this exakt issue: undefined method 'exists?'
After installing (what I think is) latest pre-release 1.0.0-2e94ec6 I get the following error:

The Project.yml file have the same lines for the load_path as the example file, looking like this

I believe the line that you've included from a previous example is no longer necessary and has been removed from Ceedling's usage. If you don't have any additional load_paths to add, you can simply give it an empty array:
:plugins:
  :load_paths: []

When I try to give it an empty array, I get this error instead:

Yeah, error reporting is not the best, but are you sure it fails on the line where empty load_paths: [] is?
Reason I am asking is because there are few more changes to project.yml and I suggest you rather generate a freshly started one for 1.0.0 and then compare with your existing. That way it is a bit more manageable.

Yes, you are right, it seems to fail somewhere else. By generating a new project, Ceedling runs fine with that one. I haven't tried to convert the old project (I don't have the time right now, and it will affect other users).
Thanks for the support!

@magnushedin So sorry about this. If you run with --verbosity debug you will get a full stack trace that explains what the actual issue is. There have been lots of changes in the newest Ceedling. The load_paths entry you have must be from a very old project configuration example. It's referencing something that never should have been exposed in a project configuration. That said, an empty [] should not be causing a bad reference. I would appreciate knowing the offending line of code from the debug stack trace so we can fix that. In the meantime, you should not need load_paths at all for your plugin configuration if you're not loading your own plugins from outside of Ceedling's built-in options.

@magnushedin I've tried to reproduce your issue and so far am unable to do so. Certainly, the "#{Ceedling.load_path}" reference is unneeded. And, in fact, you can entirely omit :load_paths if you are not providing custom plugins from outside of Ceedling's built-in options. If you could kindly report back with the results of running with --verbosity debug that would be most helpful.
Please reopen this issue when you are able to report back.
Incidentally, in the prerelease version of Ceedling you are using, some plugins have been renamed and some have been combined. In my tests, Ceedling is reporting unrecognized plugin names without issue. Once you're past the originally reported problem, Ceedling will complain about xml_tests_report and stdout_pretty_tests_report. Please see the list of plugin name changes in BreakingChanges.

@mkarlesky Sorry for the late responce.
Using my old project.yml file, omitting the complete "#{Ceedling.load_path}" entry and running with --verbosity debug, I get this output. Hope it helps.
🚧 Loaded project configuration from working directory.

Using: /home/magnus.hedin/workspace/220_02_190/unit_tests/project.yml
Working directory: /home/magnus.hedin/workspace/220_02_190/unit_tests
Set which Ceedling from config :project ↳ :which_ceedling => gem
Launching Ceedling from /var/lib/gems/3.2.0/gems/ceedling-1.0.0/

🚧 Application & Build Frameworks
Ceedling => 1.0.0-2e94ec6
CMock => 2.6.0
Unity => 2.6.0
CException => 1.3.4
🚧 Validating configuration contains minimum required sections...
🚧 Project Configuration Handling
Processing Unity configuration...
🧨 EXCEPTION: undefined method <<' for nil:NilClass Backtrace ==> /var/lib/gems/3.2.0/gems/ceedling-1.0.0/lib/ceedling/configurator.rb:216:in populate_unity_config': undefined method <<' for nil:NilClass (NoMethodError) 	/var/lib/gems/3.2.0/gems/ceedling-1.0.0/lib/ceedling/setupinator.rb:81:in do_setup'
/var/lib/gems/3.2.0/gems/ceedling-1.0.0/lib/ceedling/rakefile.rb:65:in <top (required)>' 	<internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb>:86:in require'
internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb:86:in require' 	/var/lib/gems/3.2.0/gems/ceedling-1.0.0/bin/cli_helper.rb:133:in load_ceedling'
/var/lib/gems/3.2.0/gems/ceedling-1.0.0/bin/cli_handler.rb:223:in build' 	/var/lib/gems/3.2.0/gems/ceedling-1.0.0/bin/cli.rb:346:in build'
/var/lib/gems/3.2.0/gems/thor-1.3.2/lib/thor/command.rb:28:in run' 	/var/lib/gems/3.2.0/gems/thor-1.3.2/lib/thor/invocation.rb:127:in invoke_command'
/var/lib/gems/3.2.0/gems/thor-1.3.2/lib/thor.rb:538:in dispatch' 	/var/lib/gems/3.2.0/gems/ceedling-1.0.0/bin/cli.rb:93:in start'
/var/lib/gems/3.2.0/gems/ceedling-1.0.0/bin/ceedling:125:in <top (required)>' 	/usr/local/bin/ceedling:25:in load'
/usr/local/bin/ceedling:25:in `'

@magnushedin Ah ha! Thank you. Now I know what's wrong. You found a configuration handling bug following a significant refactoring in recent months. This will be fixed in the next prerelease build. It appears you are using paramaterized tests. As a workaround, please add :defines: [] to your  :unity configuration.

@mkarlesky Yes, that solved the problem and now I need to change names for plugins.
Thank you!

@magnushedin The latest prerelease (1.0.0-13d0e3d) should have this bug fixed and is available for testing. Just for sake of housekeeping I am going to close this issue. If you are able to try the new prerelease, please report back in this issue. If :unity or :cmock project configuration bugs persist for you, please reopen this issue.


PiFace does not work

PiFace was working well in combination with ResinOS, but I can't get it to working after upgrading to HassOS.
config.txt:
kernel=u-boot.bin
disable_splash=1
dtparam=audio=on
dtparam=spi=on

Log:
Error during setup of component rpi_pfio
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/homeassistant/setup.py", line 148, in _async_setup_component
    component.setup, hass, processed_config)  # type: ignore
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.6/site-packages/homeassistant/components/rpi_pfio.py", line 25, in setup
    pifacedigital = PFIO.PiFaceDigital()
  File "/usr/local/lib/python3.6/site-packages/pifacedigitalio/core.py", line 82, in __init__
    self.init_board()
  File "/usr/local/lib/python3.6/site-packages/pifacedigitalio/core.py", line 107, in init_board
    h=self.hardware_addr, b=self.bus, c=self.chip_select))
pifacedigitalio.core.NoPiFaceDigitalDetectedError: No PiFace Digital board detected (hardware_addr=0, bus=0, chip_select=0).

https://github.com/home-assistant/hassos/blob/dev/Documentation/boards/raspberrypi.md

Looks like this PR has solved my issues. I will close this one.


optimize nms

Port nms from python to c++ and use parallel for to optimize it.

@XiaobingSuper Please help to review it.
Do I need to remove as well the .to("cpu") in the decoding part of the calibration?
https://github.com/XiaobingSuper/inference/blob/9c6c40367fd4652ce35d5eb87357611cc5c10f2f/others/cloud/single_stage_detector/pytorch/infer.py#L148


[Feature] More control over moving and resizing

Is it possible to have more control over moving and resizing of items?
My use case is a "window manager" where each item has a <Titlebar>. I want the contents of the "windows" to be fully interactive (not draggable), and to move a "window" you drag its titlebar. I also want to be able to resize any of the four corners.
Should I fork this great project, or could it be done through configuration and tricks?

That's not possible right now. But it's a cool feature to add.

I'll try forking and having a look at it then. Thanks again for the quick reply!

Adding custom dragger is not difficult.
Styles related to moving must be set in move Start event handler.
https://github.com/cuire/svelte-grid-extended/blob/481843ce6adbef15d09bdfb4fa692204c38a55f2/src/lib/GridItem.svelte#L238
And the handler need to be passed to slot like so:
https://github.com/cuire/svelte-grid-extended/blob/481843ce6adbef15d09bdfb4fa692204c38a55f2/src/lib/GridItem.svelte#L241
<slot {moveStart} />

Same must be done in Grid component
https://github.com/cuire/svelte-grid-extended/blob/481843ce6adbef15d09bdfb4fa692204c38a55f2/src/lib/Grid.svelte#L230-L233
	on:previewchange={updateGridDimensions}
	let:moveStart
>
	<slot {item} {moveStart} />
</GridItem>

And finally it can be used
<Grid {items} cols={10} rows={10} let:moveStart>

Title

</Grid>

Unfortunately I can't offer a better solution, since <slot> cannot have directives

Nice! Thanks for doing most of the work :)
I only had to add the option to override moveStart()'s if (!movable) return guard. Because I had to set movable: false on the items so that the whole item wasn't draggable. I'm sure there's a cleaner way, but it worked.

Done in 1.0.0


(Octal Number System)

The Octal Number System is very important in digital computer work. The Octal Number System has a base of eight,meaning that eight unique symbols:0,1,2,3,4,5,6 and (7).
3721.2406⁸=(3×8³).
ANS-?

(Octal Number System Systematic)
Welcome to Gboard Clipboard, any text you copy will be saved here.
Q1...


Define a way to synchronize state between all node type.

Node type are defined in this discussion: #390
State sync definition are described in this discussion: #398
State sync verification is described in this discussion: #468
Initial state synchronization are described in these discussions: #469 #471

Marking as decentralized. For centralized launch, we only need to ensure node operators can download trusted state from files.


Move ownership of loaded binary to Boomerang's core

Loader plugins should get the name and QByteArray containing the binary data of the file, this should allow us to:

[ ] properly handle the ownership of data, and remove the redundant memory management from all loaders
[ ] use test-contained binaries to verify loader correctness
[ ] in the future, store binaries inside serialized project file

@nemerle Perhaps a tasklist would work best instead of those bullets.

ya, might as well

Note automatic milestone progress.


Rectify the modifier order in namesrv

Which Issue(s) This PR Fixes

Fixes #issue_id
Brief Description
Adhere to Java's modifier order specification.

How Did You Test This Change?

Only modifier order changed.

@oopooa It would be better to create an issue associated with the pull request.

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

Project coverage is 42.94%. Comparing base (3c27bfe) to head (575387b).

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7921      +/-   ##
=============================================
+ Coverage      42.92%   42.94%   +0.02%
- Complexity      9910     9920      +10
=============================================
  Files           1190     1190
  Lines          86042    86042
  Branches       11081    11081
=============================================
+ Hits           36930    36953      +23
+ Misses         44537    44520      -17
+ Partials        4575     4569       -6

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

@RongtongJin The issue link has been added.


Wiki Cookbook -> Vec(elts) is deprecated, use VecInit(elts) instead

Type of issue: bug report

Impact: no functional change

Development Phase: request
Other information
Documentation wiki

If the current behavior is a bug, please provide the steps to reproduce the problem:
I don't know how to «pull request» a patch for the wiki, but in cookbook page I think the example for vec of registers initialisation is obsolete (I tested it and chisel3 complain about it) :
on chapter «how-do-i-create-a-reg-of-type-vec» the line :
  val initRegOfVec = RegInit(Vec(Seq.fill(4)(0.U(32.W))))
Should be
  val initRegOfVec = RegInit(VecInit(Seq.fill(4)(0.U(32.W))))
No ?

Fixed. Thanks for the catch!


Text Encoding

The tool doesn't understand ANSI encoding or microsoft.

Could we change encoding or find a way to detect it ?
Using the windows msi installer.
Thank you for your tool !

@mathieugrasland Could you try changing the Encoding of the text file in the "Save As" dialog and see if that helps? I don't spend much time on developing this anymore and I can't add encoding detection. Sorry.

Oh understood.
It won't work because there letters like éè.
Thanks for the tool by the way 👍


Missing IdentityClaims

Hi, nice implementation! Unfortunatelly it's missing the IdentityClaims implementation, did you happen to implement it?
Thanks for the great piece of work!

No I haven't, if I end up doing that I will add to this project.


fix coverage reports

Closes https://github.com/liancheng/scalafix-organize-imports/issues/54

@liancheng here we go! failure ... expected - I guess it should be merged to master anyway and things will turn green in the next commit/PR?

Hmm, I still didn't see OrganizeImports.scala included in the report? The newly included file is:
shared/src/main/scala/fix/Implicits.scala

The shared project is used for testing purposes, and we don't really care about the coverage of this SBT project.

Maybe this issue is related to this one: https://docs.codecov.io/docs/fixing-paths

oops indeed, I saw code coverage decreased, so I didn't even check... I'll have another look, sorry for the noise

Lol, I would be pretty sad if the test coverage is only around 70%. The last time I checked it, it was above 95%.

Looking at the raw upload https://codecov.io/codecov/v4/raw/2020-05-29/6B959D8FB7518A1CFF0DF84B926C2924/e9379fbc51d62d1217d50ee4fd3254cd8ea840aa/0f970c5e-e729-4037-a1da-2c3da3158b11.txt, it's clearly related to the filename, which is merely fix/OrganizeImports.scala, and exists in other (test) sub-projects. So we could rename the test package to avoid that. I am trying to ignore the test paths to see if that helps.

I tried ignore and fixes in codecov.yml (see below), but it didn't help (even though codecov.yml is detected according to the logs). FWIW, this is what the github action seems to wrap: https://codecov.io/bash.
So I just went for the rename route, which seems to work at least. Another option would be to rename the test input/output package from fix to test to make sure someone does not introduce a conflict again in the future.
ignore:
  - "input*/**/*"
  - "output/**/*"

fixes:
  - "::rules/src/main/scala/"

Awesome! Thanks for looking into this and get it fixed! 🎉

This is the first time I feel glad to see the test coverage dropped 😆