Great idea!

And support pull up load more?

Not exactly "pull up" to load more, but I have written another library (that works well with this library) to support loading more content as you scroll down. You can check it out here.
https://github.com/gh123man/ScrollViewLoader
Otherwise - I do not plan to add a "pull up" to load more to this library at this time.


Test against Pint 0.24

Description
Pint 0.24 support
Todos

[ ] Add Pint 0.24 to test matrix

Status

[ ] Ready to go

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

Project coverage is 33.86%. Comparing base (5197b17) to head (725a174).

:exclamation:  There is a different number of reports uploaded between BASE (5197b17) and HEAD (725a174). Click for more details.
HEAD has 15 uploads less than BASE

Flag
BASE (5197b17)
HEAD (725a174)

18
3

Additional details and impacted files


Upgrade to Font Awesome 4.3

40 New Icons in 4.3

Thanks for updating this. I am actually writing some Grunt tasks to do all of this automatically in the future. I will merge this one for now though!


Plugins can contribute to a formatting toolbar

Allow plugins to contribute icons to a formatting toolbar and to handle commands.

The video plugin contributes a button to the toolbar.


Ability to limit stars/likes to 1

Feature Request
As a user of RetroQuest
I need the ability to limit the number of stars/likes to 1
So that I can limit abuse of the functionality
Acceptance Criteria
Given I have limited the number of stars/likes
When I select a thought to star/like
Then then I cannot increment the star/like count beyond 1

Maybe with user accounts and some kind of permission system...

g

Limiting by MAC address might be better to slightly obscure votes and not have to handle account permissions, password resets, etc.

@MarkJRust - is there still interest in this?

This could be an interesting experiment to run 🤔

Duplicate issue.  I am closing this one


Overriding Identity Types + Overriding Property Types in queries.

Another hello from the F# world :)
In F# it is common to wrap identity types in Single Case discriminated unions such as:
type OrderId = Id of Guid

The goal and culture are to encode the domain and business rules in the type system as much as possible to leverage the power of compile-time checks to design and evolve models safely. Many folks are also starting to do it in C# (as part of DDD), but it is much more common in F# due to the ease and conciseness of designing such types.
The above type is not exactly a Guid, it is instantiated like so: OrderId.Id(Guid.NewGuid()). The most commonly used F# to JSON serializer unwraps such types and stores them as plain strings. Therefore the problem is not with the persistence format, but with the .NET type constraints introduced by Marten (ie. trying to use such a type as a primary key throws an exception, as explained in the documentation)
In this context, it would be very useful to be able to instruct Marten to override the type of primary keys, but also of properties in queries.
First of all by adding the following overload to the identity function to have something like the below to tell Marten that it should consider that the primary key has the provided generic type in the database:
storeOptions.Schema.For<Order>()
        .Identity<Order,Guid>(fun x -> x.Id)

Then another thing that could be useful is a Where<T1,T2> overload on IMartenQueryable so that we can do something like the below, for example when querying an Invoice document having an OrderId property (OrderId cannot be converted to Guid by default. Moreover, even if such a cast could be translated to SQL by the Linq provider, it would be quite cumbersome to define casts explicitly on all such types - especially since they are supposed to be one-liners):
documentSession.Query<Invoice>()
      .Where<Invoice,Guid>(x => x.OrderId, x => x == orderId)

The first lambda would be used to select the property, while the second lambda would be the one actually being turned into SQL using the type of the second generic parameter.
I think this would be useful beyond F#, as it would give users full flexibility to decouple the JSON representation of data from their representation in .NET. It would empower users to model their domain in the way making the most sense for them from a business perspective and use custom JsonConverters for serialization. These two methods would enable quite a lot of use cases.
What are your thoughts on this idea?

Seems very similar to the Typed-ID work from jeremy the last days.
See these discord messages and this branch.
But could also be different situation because of F#.

Duplicate with https://github.com/JasperFx/marten/issues/2487, and I'm unenthusiastic about the proposed solution (which isn't really simpler mechanically and would actually require more intrusive changes to the LINQ support).
We're going to accommodate value types such that Marten's LINQ support will see "SpecialValueTypeThatIsReallyWrapperAround Guid" and treat it as a Guid for the JSON & database parameter support. And we'll pull that off w/o having to require any marker interfaces or abstract classes or whatever else that makes F# devs get all snooty.


fixing the issue of select cast with invalid number of function argument

Description

Problem - cache lookup failed error when we run cast query with extensions like PgAudit, Valgrind enabled.
RCA - we blindly trying to call cast function with three arguments every time where as in postgres the number of arguments can vary from 1 to 3.
Fix - Adding the handling cases as per the number of arguments(nargs) passed via cast query.

Issues Resolved
BABEL-5272
Signed-off-by: Pranav Jain<EMAIL_ADDRESS>Test Scenarios Covered

Use case based -

Boundary conditions -

Arbitrary inputs -

Negative test cases -

Minor version upgrade tests -

Major version upgrade tests -

Performance tests -

Tooling impact -

Client tests -

Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

11 of 12 (91.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 74.481%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contrib/babelfishpg_common/src/sqlvariant.c
11
12
91.67%

Totals

Change from base Build<PHONE_NUMBER>0:
0.007%

Covered Lines:
45134

Relevant Lines:
60598

💛  - Coveralls


What is the mean of print?

Hi, thanks to your great work. During my optimization, it print that
iter: 0, pri_resid: 2.00879e+01, dual_resid: 3.45980e-05, mu: 2.15564e+00
iter: 1, pri_resid: 9.38095e-01, dual_resid: 2.89040e-05, mu: 1.54378e-01
iter: 2, pri_resid: 1.10201e-01, dual_resid: 1.68656e-05, mu: 3.06670e-02
iter: 3, pri_resid: 8.88062e-03, dual_resid: 1.45111e-05, mu: 3.15341e-03
iter: 4, pri_resid: 2.95888e-03, dual_resid: 1.42403e-05, mu: 1.35999e-03
iter: 5, pri_resid: 2.78638e-04, dual_resid: 1.49785e-05, mu: 2.77934e-04
iter: 6, pri_resid: 8.51825e-06, dual_resid: 1.65200e-05, mu: 5.20932e-05
iter: 7, pri_resid: 4.42947e-07, dual_resid: 1.60216e-05, mu: 1.87964e-06
iter: 8, pri_resid: 2.40871e-07, dual_resid: 1.67323e-05, mu: 3.06535e-08
iter: 9, pri_resid: nan, dual_resid: nan, mu: nan
iter: 10, pri_resid: nan, dual_resid: nan, mu: nan
I want to know what is the mean of pri_resid, dual_resid, and mu. In addition, does output nan normal? Does this represent divergence?

I confirm the same issue of NAN.

Same issue with "nan" here


setup.sh gcloud command flag order may be incorrect

I noticed in some places where gcloud is run that flags are not after the command:
For example:
export WEBHOOK=$(gcloud run --platform managed --region ${FOURKEYS_REGION} services describe event-handler --format=yaml | grep url | head -1 | sed -e 's/  *url: //g')
Running the above, I get this error:
`ERROR: (gcloud.run) unrecognized arguments: --platform (did you mean '--format'?)
To search the help text of gcloud commands, run:
gcloud help -- SEARCH_TERMS`
Modifications that seem to work:
  export WEBHOOK=$(gcloud run services describe event-handler --platform managed --region ${FOURKEYS_REGION} --format=yaml | grep url | head -1 | sed -e 's/  *url: //g')

Hi @davezen1 ! Thanks for the report. I think this may have been fixed with the changes in #57 -- can you try again and report back?

Hi @davezen1 ! Thanks for the report. I think this may have been fixed with the changes in #57 -- can you try again and report back?

Yes, ran it with the latest code and it is fixed.

Great, thanks for the update!


Passing model by GET append params instead of overwritting

I am passing model by GET and after first submit request look like this:

after second request:

I think params should be overwritten, but maybe I am missing something in code.
Form model
class TestForm extends Model
{
    public $text;

    public function rules()
    {
        return [
            ['text', 'safe'],
        ];
    }
}

Controller
public function actionTest()
    {
        $model = new TestForm();
        $model->load(Yii::$app->request->get());
        return $this->render('test', ['model'=>$model]);
    }

View
<?php $form = ActiveForm::begin(['method'=>'GET']); ?>
    <?= $form->field($model, 'text') ?>
    <?= Html::submitButton('OK') ?>
<?php ActiveForm::end(); ?>

PHP version: 5.5.9-1ubuntu4.14
Yii2 version: 2.0.6
OS: Ubuntu 14.04
Browser: Chrome 48.0.2564.97 (64-bit)

$model = new TestForm();        $filter->load(Yii::$app->request->get());
Use $model instead of $filter$?

@grin1 Before posting issue I was changing variables name and I did not notice this one. Now corrected, but issue is still valid.

You should specify form action explicitely to the URL without model parameters, otherwise it will just append input data to the current URL per each submit.


Fix CI badges and cleanup rosdoc documentation

Fix CI badges
Update information on CI
Remove unused rosdoc documentation

Thanks for noticing. I am adding it to this PR since it's a small typo.

Thanks for the help @tylerjw!
This looks good to merge @vatanaksoytezer.


Can´t uninstall solved

I have found the video of how to uninstall Fusion 360 in the repo. Although i don´t wan´t to remove just the extension but the whole programm. How do i go about that?

Hey @mmithoefer 👋🏻,
if you can wait then you can see this and another things tomorrow by the Tux-Day‘s 2021 where I have a Live-Presentation! 🙂

@mmithoefer Here can you see how it works: https://youtu.be/KTSdsOuuEmY?t=2058

Great. Thanks a lot


libipl: p10: ipl: Create PEL for empty guard file

libipl: p10: Create PEL for guard file exception
Currently whenever the guard file is empty or does not exist, IPLing is
terminated with an exception.
Fix : During ipl0, while trying to initialize the libguard if any
guard file related exception is triggered, call the registered callback with
IPL_ERR_GUARD_PARTITION_ACCESS to create the according PEL.
Adding the IPL_ERR_GUARD_PARTITION_ACCESS enables us to generate the PEL and "User
Data 1" provides more information regarding the PEL
Tested:
Point to dummy file instead of actual guard file
Using the above error in the ipl callback, able to generate the below PEL
{
"0x50000E0C": {
"SRC":                  "BD8D300B",
"Message":              "Guard partition access failure",
"PLID":                 "0x50000E0C",
"CreatorID":            "BMC",
"Subsystem":            "BMC Firmware",
"Commit Time":          "10/06/2022 12:33:57",
"Sev":                  "Predictive Error",
"CompID":               "0x3000"
}
}
...
...
"Error Details": {
"Message":              "Guard partition access failure"
},
"Callout Section": {
"Callout Count":        "1",
"Callouts": [{
"FRU Type":         "Maintenance Procedure Required",
"Priority":         "Mandatory, replace all with this type as a
unit",
"Procedure":        "BMC0001"
}]
}
},
...
"LOG011 2022-10-06 13:03:07": "Istep: updatehwmodel: started\n",
"LOG012 2022-10-06 13:03:07": "updatehwmodel: Genesis mode boot\n",
"LOG013 2022-10-06 13:03:07": "Guard file
/var/lib/phosphor-software-manager/hostfw/running/NOFILE is empty",
Signed-off-by: deepakala<EMAIL_ADDRESS>Change-Id: I8b9121a76d4dc3b8bc600f0f34ece271c481377b

Merged


submitting new form on mobile doesn't load success message

success message doesn't show up

actually for all versions but only on azure site


FormBuilderDateTimePicker & FormBuilderDropdown: add adaptative feature

Is there an existing issue for this?

[X] I have searched the existing issues

Package/Plugin version
9.1.1
What you'd like to happen
First of all, thank you so much for your work and this great package!

That would be nice to add the adaptative feature for possible fields such as FormBuilderDateTimePicker and FormBuilderDropdown to automatically show either Material components on android or Cupertino ones on iOs.

Example on some basic widgets:
https://docs.flutter.dev/platform-integration/platform-adaptations#widgets-with-adaptive-constructors

Alternatives you've considered
No response
Aditional information
No response

Will be nice but currently we only support Cupertino and Material separated.
See https://github.com/flutter-form-builder-ecosystem/form_builder_cupertino_fields for Cupertino widgets


Add an AppData XML data file to Linux builds

The AppData files provide descriptive data about an application, and is typically used in application managers to display additional information to the user. Having an AppData file is especially important for some graphical application installers: without one, an application manager may not event show an entry for the application. This happens for example in GNOME Software.
For the sake of completeness, this commit not only adds the AppData file to the Flatpak build, but also to the Linux .deb and .rpm packages.

This is follow-up to PR #16169 and a complement to issue #7112.

Hi @aperezdc, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Updated!

Thanks for the quick turn around @aperezdc, this is great 😄

@Tyriar: You're welcome! Also, it is being nice contributing to VS Code — kudos for caring about keeping the community healthy :smile:


Update README.md for allowExternalErrors options

In the README, it writes that the options allowExternalErrors  is settings at true by default but it's not:
https://github.com/maticzav/graphql-shield/blob/9428a7a5e32cd66d13faea0e7da7db711be2d739/src/index.ts#L312-L318

:tada: This PR is included in version 2.2.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remote change directory

I can't change the remote directory.
remote.cd('/usr/local/folder');

Or
remote.exec(cd '/usr/local/folder');

Neither of them worked for me.

I'm using 0.6.10 version of Flightplan.

You probably want to use transport#with(). Docs: https://github.com/pstadler/flightplan#transportwithcommandoptions-options-fn


fixing issue #23

kenzanlabs/keystone#7
fixes #23

Can one of the admins verify this patch?

This looks nice @cxoltero !

ok to test

@cxoltero
Shouldn't the README be updated as well?


🚧Add switches to toggle protocols

Protocols sholud be defined by network.
Mainnet:

[ ] Soroswap
[ ] SDEX
Testnet:
[ ] Soroswap
[ ] Phoenix

Github workflow randomly failing navigation tests but they are passing in local instance & manual testing

workflow approved!

@esteblock I will test frontend tests yar ci run

please run: yarn cy:run


Return total voting power on shutter proposals

For ongoing proposals that have shutter enabled, we need to provide the total voting power. This will allow us to display the quorum on the UI along with a warning that the quorum may be inaccurate until the proposal is closed and choices are validated.

@bonustrack can we get someone on this?

Completed in https://github.com/snapshot-labs/snapshot-sequencer/pull/23


[Addition] Rhodium (Forge 1.16.x-1.19.x)

Checklist

[X] I've checked that my mod suggestion hasn't already been added to one of the lists.
[X] I've checked the latest issues to make sure that this mod hasn't already been suggested.

What is your mod's name?

Preformance
What side dose your mod need to run on?

Optimizes DataFixerUpper, based on LazyDFU
(OPTIONAL) Give some extra information about the mod.
"In comparison to other mods, Rhodium has the primary advantage of being simple, safe and having a low impact on the game."

What dose "low impact to the game" mean exactly?  it dose not any game mechanics? it has very little performance impact?

I Don't know exactly, I'm not the mod owner
I Just noticed this mod makes my game load 10 seconds faster than LazyDFU

I Don't know exactly, I'm not the mod owner I Just noticed this mod makes my game load 10 seconds faster than LazyDFU

is that with mods that add stuff like AE2 or Mekenism, or just with performance mods?

Will not add as its a direct copy of Lazy DataFixerUpper


Support disabling CMake options through the makefile

CMake will remember the options and we currently ignore values of anything but 1. Instead I've set it to just pass the argument of RUNTIME_CHECKS, LTO and WERROR directly to cmake, and added options for disabling ASAN, TSAN and UBSAN by setting them to zero.

I didn't actually test every option, but I tested it enough to make sure that it can be used to both enable and disable the options, and that the ifdefs work correctly.


Simplify ranges for month and week pickers

Is your feature request related to a problem? Please describe.
When using the datepicker in range-mode, it would be great to have a simple, consistent modeltype between the different pickers (days/weeks/months).
Right now the different pickers behave differently in determining what the 'end time' of a range is.
E.g. the week and month pickers end on the start of the last week/month, instead of it's end.
The day picker currently returns the last timestamp of the last day.
Describe the solution you'd like
It would be nice to have a simple [startTimestamp, endTimestamp] as model for any range, similar to the way the day-picker currently functions.
Describe alternatives you've considered
I suppose an alternative would be to do a lot of conditional logic and transforming the model types accordingly.
Additional context

Here's an overview of the current behavior of the different pickers:
https://stackblitz.com/edit/vuepic-vue-datepicker-rm8tdk?file=src%2Fcomponents%2FPlayground.vue

This is too much tweaking just to have hours, by design you don't show hours in these modes, if needed, you can just manually update the end value. You can also provide custom format function and display the value you want.


[Bug] GraphQL Query variables, are market as not appear in any GraphQL query, but they are!

Insomnia Version: 7.0.3
Operating System: Windows 10 x64
Describe the bug
When access stored graphql request, all Query variables are marked as not appear in any graphql query. But they are, because were used before and stored in a request.
To Reproduce

Create a new GraphQL request with some variables
Run query, it need to work with variables.
Now, close insomnia or go to another query
Return to request created before, and check the variables. You will see all variables marked as invalid
Just retype the same variable name, and all variables will works again

Expected behavior
As the stored request has valid query variables, so the query works fine. BUT... all queries are marked as invalid
Screenshots

Desktop (please complete the following information):

OS: Windows 10 64 bits
Installation Method: Windows installer
App Version: 7.0.3

Additional context
Everything works... just the variables are marked as does not appear in any query. But if you check in my screenshot, the updatePost query contains the variables, none of them are missing.

Same problem here.
Did you find a solution?

same problem here, but the mutation is not being executed

same problem here, this problem has been around for a year now I think

same problem here, this problem has been around for a year now I think

similar issue

Is this issue still reproducible in the latest version of Insomnia?

Yes. I can get you a screenshot tomorrow if you need it. Thx.

Same issue: I've got the error despite everything working fine.
Though, I've noticed something strange: the error disappears when the schema is loaded ("schema fetched just now"). When I switch tab and switch back again, the error comes back.
See the video:
https://user-images.githubusercontent.com/47944500/212424831-2bb1a3d8-63b0-41d2-a3a8-5b8faf9ac121.mov

I've only just started experiencing this, and I also found that manually fetching the schema wasn't working and I couldn't view the documentation.
I've always had "Automatic Fetch" turned off for schema introspection.
I turned "Automatic Fetch" back on, and it recognised the variables were there and the documentation started to work again.

Same issue here, I first noticed it in version 2022.7.0 and the issue persists in version 2022.7.5

I thought I'm doing something wrong. I new to Insomnia and just dowloaded the app, have same issue

This is the work around I found that worked
https://www.appsloveworld.com/graphql/13/passing-variable-in-insomnia-graphql-request

Same here, and I can not clear variables. After removed it, I send request and it show again


fix(cli): fix Chinese display garbled in windows

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 remained the same at 82.894%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5340

Relevant Lines:
6442

💛  - Coveralls


Swapping the next.js  component ?

Hi,
Since next  doesn't support data prefetch, I want to swap it with https://github.com/scaleapi/data-prefetch-link which itself extends next.js Link component.  So, is it possible to pass what might be considered as next.js  Link  interface implementations?

It's quite weird that i missed such an obvious mention in the documentation. Although, as of now, I am focused on slightly different things for a month or two, i will try this after moving to the task related to this again. Thanks for helping me with the only issue that was bothering me while deciding to use this library.

Since jsx tags are just js functions, did you tried passing both the arguments with identical values?


catalog: Exporting errors so these are available outside pkg/catalog as well

This PR renames (capitalizes) the errors in pkg/catalog
Exporting the pkg/catalog errors makes it possible for users of catalog to understand what went wrong (coming in PR https://github.com/openservicemesh/osm/pull/2625).
No functional code changes in this PR.

Codecov Report

Merging #2629 (e9ce23c) into main (bf9b96a) will increase coverage by 0.18%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #2629      +/-   ##
==========================================
+ Coverage   63.47%   63.66%   +0.18%
==========================================
  Files         152      152
  Lines        7059     7059
==========================================
+ Hits         4481     4494      +13
+ Misses       2562     2549      -13
  Partials       16       16

Flag
Coverage Δ

unittests
63.66% <50.00%> (+0.18%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/catalog/routes.go
80.33% <0.00%> (ø)

pkg/catalog/traffictarget.go
88.31% <0.00%> (ø)

pkg/catalog/service.go
90.38% <50.00%> (ø)

pkg/catalog/xds_certificates.go
91.39% <80.00%> (ø)

pkg/envoy/route/config.go
95.45% <0.00%> (-0.76%)
:arrow_down:

pkg/configurator/validating_webhook.go
80.00% <0.00%> (+9.65%)
:arrow_up:

Continue to review full report at Codecov.

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

@snehachhabria

Is there a plan to use these errors outside of catalog ?

Yes - starting w/ this test:  https://github.com/openservicemesh/osm/pull/2625   (which I'll resurrect shortly)

@shashankram Here is one example where the catalog error makes sense to be bubbled up (instead of rewriting it): https://github.com/openservicemesh/osm/pull/2627/files#diff-7a1f450fbabb0088828083ed90ad3ccc49ed4edd9ae6f58ad5bdcdd158b2dda8R141
And run tests against these error conditions: https://github.com/openservicemesh/osm/pull/2627/files#diff-f4d146b2f432d283619be84d448017b08e2dafd4a489ddf306d629f48d6f5366R214-R217

@shashankram Here is one example where the catalog error makes sense to be bubbled up (instead of rewriting it): https://github.com/openservicemesh/osm/pull/2627/files#diff-7a1f450fbabb0088828083ed90ad3ccc49ed4edd9ae6f58ad5bdcdd158b2dda8R141
And run tests against these error conditions: https://github.com/openservicemesh/osm/pull/2627/files#diff-f4d146b2f432d283619be84d448017b08e2dafd4a489ddf306d629f48d6f5366R214-R217

Good stuff! I guess we should always export errors being returned across package boundaries?


Fix connected accounts (plug session back)

Coverage increased (+0.02%) to 67.2% when pulling 17b8fe6dd445ed49f505f1936fe9c0e87ea22efe on fix/connected-accounts into 07492dac1535f95a7d2a6e961d438403242375b4 on master.


refactor: registryless oci_image

fixes #484  and closes https://github.com/bazel-contrib/rules_oci/issues/410, https://github.com/bazel-contrib/rules_oci/issues/285, https://github.com/bazel-contrib/rules_oci/issues/176, https://github.com/bazel-contrib/rules_oci/issues/170, https://github.com/bazel-contrib/rules_oci/issues/148, https://github.com/bazel-contrib/rules_oci/issues/107
TODO:

[ ] update oci_image documentation, now that we also support zstd with this change. @alexeagle should zstd support go separately?
[ ] detailed description
[ ] add the layer with correct media type when base image is docker + add a test for it.
[x] use hermetic zstd
[ ] run against distroless to check breakages, ask some of our customers to see if breaks anything.
[ ] @alexeagle you wrote the most of the stampable attributes, i need your eyes there, i might be missing something there.
[ ] run a benchmark

Did a quick test here. Everything builds/tests fine, but running an oci_tarball gives an error like this:
open /var/lib/docker/tmp/docker-import-1867629837/blobs/sha256/a74371ebe00e8fad96ff8f7ec8421e280fd0828ecdd38457c8a2f7c1ceb1b870.tar.gz: no such file or directory

yeah, there's an unrelated change in this PR. thanks for the info!

@mortenmj did you see any significant speed-up/slow-down, do you have an before-after?

@mortenmj did you see any significant speed-up/slow-down, do you have an before-after?

It seemed fast, but I didn't do any formal testing

Benchmark: distroless building all images 5m 17s -> 3m 59s

Breaking changes are observed here: https://github.com/GoogleContainerTools/distroless/pull/1586#issuecomment-2097035728 🌮

PTAL again, this is now ready.

Great work!

There seems to be an error in the description, which should be pkg_tar#extension.


Add link to demo apps for testnet paymaster

Adding link to demo apps

Updated PR to reflect the below


DATAJDBC-138 -Improve package structure and naming.

Moved SimpleJdbcRepository into support package.
Moved mapping package under core.
Renamed JdbcEntityOperations to JdbcAggregateOperations.

Merged by Oliver.


style changes for zipcode form, button and placement

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #420
Test URLs:

Before: https://main--sling--aemsites.aem.live/programming/sports/pro-football
After: https://420-zip--sling--aemsites.aem.live/programming/sports/pro-football

@kmurugulla there are a few small suggestions:

width of ours is bit less (110px) and theirs is 127px.
padding between location icon and 0 is more in our case
Background color seems different in zipcode block (they are using  rgb(41, 41, 56) as background color ).

Tried to get the styles as close as possible for zipcode selection experience . Please review @rusmeenkhan1 and @helms-charity


Missing License

Hello,
Would it be possible to add a license to this repo? Not having a license restricts users from using this code. Please refer to the GitHub's licensing help page.

@saprmarks, @aaronmueller
Many thanks.

Just did this—sorry it took so long to get to it.


Function HasHeader() checks for field named "header" instead of a field of type "Header"

Description
In received_message_age.hpp, the HasHeader() function checks if the msg contains a field "header". In this situation one cannot have a custom message custom/Header and call the field "header" in other messages. It should instead check if the message contains a field "header" with type "Header"
Expected Behavior
Return  true if a message contains a field named "header" with type "Header" (for foxy, .
Actual Behavior
Returns true if a message contains a field named "header" regardless of the type.
To Reproduce
System (please complete the following information)

OS: Ubuntu 20.04
ROS 2 Distro: Foxy

** For ROS 2 bugs - please attach a VCS.repos or Dockerfile if possible to help us reproduce the environment **
Additional context
** Add any other context about the problem here **

Thanks for the report! This does indeed sound like a problem - prs always welcome of course if you have an idea how to fix it :)

I was a bit hesitant to make a pr because I don't have too much experience with c++ but I can give it a try!

Is this issue still being tackled? I kept receiving compilation errors because a field in my custom message definition is named "header" and I believe the compiler was trying to use my message type instead of std_msgs/msg/Header when calling HasHeader(). I think this is the case because the error explained that the compiler kept trying to find "m.header.stamp" in my message definition instead of the std_msg's header definition because "stamp" is not a subfield of my header field.

It became a bit of an rabbit hole changing it to look for a msg of type std_msgs/msg/Header because some tests made up different header msgs and didn't want to depend on std_msgs.

So I ran into this issue today. Still valid!


[PR BOT] Update CHECKSUMS files and Fix checksums

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

incorrect


Staging > Production

Includes:

New coordinator profile section
Adds "Coordinators" level between "Leaders" and "Teachers" on chapter pages
Section only visible if any chapter members are set to title in admin side
Program Manager job position
Reorganized jobs pages into partials for easier editing

Leaving this for now...going to start fresh with a new PR


uv_open

I was looking into the code and one thing stood out in my eyes: we have functions like uv_write and uv_fileno which basically accept any kind of handle and dependent on the type of it (which is a basically a variable in the struct) it determines what to do.
Yet we have multiple functions for opening an already existing file descriptor: uv_tcp_open, uv_udp_open.
The most annoying part IMO is that there is no uv_tty_open, even though the constructor of uv_tty_init requires a file descriptor. I think we could move the functionality of opening a file descriptor in a tty to open.
I think creating uv_open would make writing bindings easier, because currently it kinda is a lot of code.

Ok, after further investigation, the signatures are a bit different:
UV_EXTERN int uv_udp_open(uv_udp_t* handle, uv_os_sock_t sock);
UV_EXTERN int uv_tcp_open(uv_tcp_t* handle, uv_os_sock_t sock);
UV_EXTERN int uv_pipe_open(uv_pipe_t*, uv_file file);

On windows uv_os_sock_t is SOCKET. Don't know what to make of it.

This would indeed be nice, specially on Unix, there everything is a fd. On Windows, however, we have a problem, which I don't know how to solve elegantly. AFAIK all SOCKET and HANDLEs can be stored as 32bit integers, because even on 64bit Windows, handles only use the lest signifficant 32bits, but it sounds too much like a hack.

We could use IntPtr and cast to Int on unix only.

That's just reversing the hack :-S

But there is no possible way we loose precision that way

I still think we're trying to force it here. Not to mention intptr_t is a C99-ism, IIRC.
While I like the idea, I don't like how the implementation would result, so lets stick with the different functions we have for now.

I agree, this is the way C libs work


HDX related documentation

Feature selection guide:
http://exports-staging.hotosm.org/en/help/feature_selections

Github development documentation has been updated.


Update root NS record on zone creation rather than attempt to create it

This is my crack at fixing the issue described by https://github.com/octodns/octodns-ns1/issues/48 for octodns-ultra. We observed the same issue and saw it resolved with this fix.
I'm new to this codebase, so open to suggestions on alternative solutions. I have a fix queued for octodns-ns1 as well, but it's ~identical so I'll wait for feedback on this before sending that one out.

Thanks!


Get python dlls from Python directory

Fixed #143

Download:
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python35
installer/trik-studio-installer-master-i686.exe
trik-studio-master_qt5.12_i686.zip
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python35-x64
installer/trik-studio-installer-master-x86_64.exe
trik-studio-master_qt5.12_x86_64.zip


Is there any way to clear the "listbox" in the dom?

Hello, I'm using the library and I was wondering if there's anyway to clear out the Listbox in the dom that has the list of li/ul results? I want to be able to clear it once I pick a suggestion from the dropdown or when im doing a new search.
The reason for this request is sometimes for fast typers OR users that would select all and start typing, they would get a glimpse of the old suggestions in the drop down before it overwrites itself with the new suggestions. Also I'm having a rare bug where sometimes for some reason I'm not sure of yet (I can't reproduce it constantly) two dropdowns overlap, 1 having an old suggestion that is stuck there and the new suggestions in a dropdown below the first one. Maybe if I'm able to destroy the whole listbox of li and it's elements, I would eliminate those problems most probably.
I tried to add an on.('input') even that gets the text and checks if the length is <= 1, I select $("role=listbox") by JQuery and call empty: $("role=listbox").empty(), which doesn't seem to do it for me as I still get glimpses of the old suggestions before destruction.
It's driving me crazy and I thought maybe someone here would be able to help with a clean and consistent way that I'm not aware of.
I'm getting my suggestion list from an ajax call so everytime I wait like 150 ms for user input then trigger the call which would get me all the needed values and set them in the awesomplete.list. So before this 150 ms is done waiting, the box would be displaying the old suggestions.

Any chance you could record a quick video of the behavior you're trying to avoid? Sounds like it may be related to transitions on the listbox. If you set transition: none on it temporarily (via .css() since you're using jQuery), that might help.

Hello Lea,
Transition is already set to none as that helped us make the control I'm building way more responsive.
This is a link of the video that shows what I described in my post above.
https://vimeo.com/288698483
Note that on keyup event I'm trying to do: $('#awesomplete_list_1').empty(); just to clear the available options maybe that would help, and that's triggering, still you get a glimpse of the previous options as you start writing and I have absolutely no idea where's it's bringing them from, as I inspected the dom to see if there's still any residue.
Also note that i'm using the keyup event to trigger an timeout so the control would actually wait the user for 150 ms to give a buffer for fast typers to finish their word before going to the API I'm calling to bring suggestions.

I have been struggling with that too.


Requesting public domain for CDF best practices site

We're thinking bestpractices.cd.foundation would do well, open to alternative suggestions. The content repo is at https://github.com/cdfoundation/best-practices-site and we're currently we're posting private drafts via the org netlify account.

We managed this through main LF ticketing system


Unbound does not remove pid file outside chroot directory when terminated

If I run unbound chrooted in /opt/local/etc/unbound/ and the pidfile is /opt/local/var/run/unbound/unbound.pid then unbound can write the pidfile, but not remove it.
nsd does this.
When the pidfile is inside the chroot hirearchy, unbound behaves properly
Version 1.9.2

Thanks for you report. Removing the pidfile is not possible as privileges are dropped after the creation of this file.
NSD does not allow a pidfile outside of the chroot directory. Adding this requirement to Unbound is not desirable as this will break current installations. Note that Unbound should not have a problem on startup with an old pidfile present. If no process is running with the PID from the file Unbound will only log the "did not exit gracefully last time" message when the pidfile is in chroot, for exactly this reason.
If have added text to the ubound.conf man page to reflect this limitation.


Add extra parameter for trigger insert or removal-callback

Hi,
I want to use your plugin in my application but I've got one problem. I need to use the node in trigger for other action (update some other data in relation with node).
I see in jquery api that trigger can take an extra parameter : trigger( eventType [, extraParameters] ). Perhaps it's possible to pass node argument in extra parameters. The code could be :
function trigger_removal_callback(node) {
node.parent().parent().trigger('removal-callback', node);
}

I'd also like to know which node was removed because I need to renumber what is left and the callback is triggered before the nodes are removed, so I don't know how to number them.


missing constructor for scipy.sparse.csr_matrix

https://docs-cupy.chainer.org/en/stable/reference/sparse.html#sparse-matrices
says

CuPy supports sparse matrices using cuSPARSE. These matrices have the same interfaces of SciPy’s sparse matrices.

However, the constructor

csr_matrix((data, (row_ind, col_ind)), [shape=(M, N)])

from https://docs.scipy.org/doc/scipy/reference/generated/scipy.sparse.csr_matrix.html#scipy-sparse-csr-matrix

is missing in cupyx.scipy.sparse.csr_matrix in
https://docs-cupy.chainer.org/en/stable/reference/generated/cupyx.scipy.sparse.csr_matrix.html#cupyx-scipy-sparse-csr-matrix

To add another example: the constructor coo_matrix(D) (initializing a COO matrix with a dense matrix) is also missing.


Don't pay vote Debug logging cost in normal operation

Feature Request
Summary
Currently we spend lots of RAM allocation in Debug logs across votes. Here are some profiles of a query serving full node in its operation over 1-2 hours. We see:
addVote's StringShort calls taking 5GB of allocation (compared to the state machines total 48GB):

These are both debug line calls:

https://github.com/cometbft/cometbft/blob/d27732ad9a992e321e440417a2a8220f90fe0044/internal/consensus/state.go#L2289
https://github.com/cometbft/cometbft/blob/d27732ad9a992e321e440417a2a8220f90fe0044/internal/consensus/state.go#L2378

Peerstate.setHasVote taking 15.4GB, this is entirely in debug logging:
https://github.com/cometbft/cometbft/blob/d27732ad9a992e321e440417a2a8220f90fe0044/internal/consensus/reactor.go#L1444-L1449
Problem Definition
This is certainly adding notable load to the system, and its not even providing utility most of the time!
Proposal
I'm not sure what the most well abstracted way to do the logic for "if debug mode, run this log logic" is. Just wanted to point out that there is a lot were paying right now

I think this can do the job: https://github.com/cometbft/cometbft/blob/d27732ad9a992e321e440417a2a8220f90fe0044/libs/log/lazy.go#L14-L19

Heres a screenshot for SetHasVote, sorry for not including that in OP:

But for the first 2 examples, we can do what we already have for block hashes, with LazyBlockHash.
In both cases it is a vote set. The logic is that we pass the vote set (reference), and only call the StringShort() if the logger produces the log message.

I agree that there are too many allocations in this logic, especially on consensus/state. Unfortunately the current logging logic allocates heap even if debug mode is not enabled.
I've played around and implemented some logic on a branch to replace that, adding a new method on the Logger called DebugOn that returns a bool if the debug mode is enabled. We have many loggers implemented on this interface and I implemented the logic only on filter logger.
And then I replaced all the logger.Debug logic in the consensus/state to check first if logger.DebugOn() then you log. It works, the logger.Debug() is not invoked and hopefully these allocations won't happen. But I didn't have time to profile it.
What do you think about this approach @ValarDragon and @cason ?
My branch can be found here and the commit is -> https://github.com/cometbft/cometbft/commit/d6dfcf1bb439f1f944d21c39c2e33e07cdf59004

This sounds great to me, I really like it! Can you add it to these two spots:

https://github.com/cometbft/cometbft/blob/d27732ad9a992e321e440417a2a8220f90fe0044/internal/consensus/reactor.go#L1444-L1449
https://github.com/cometbft/cometbft/blob/main/p2p/conn/connection.go#L361-L362

Then can help with getting a mainnet profile for it! With the cumulative changes here. I think this will be a big performance difference!

Cool, I'll look into these other places too. If you can do a profile that would be amazing to validate the performance improvement

Hi @ValarDragon I've added the additional places you mentioned above, I've also added a few more that might help.
Not sure how you are going to profile this but I pushed a draft PR that it's easier to visualize the changes #2989
Please let me know if there's anything else you need in order to be able to test this. Thanks again !

Commented out the spot instances in Osmosis fork for noticeable performance gains / reductions in gossip routines and consensus mutex!

Here is an idea: during our release process, we produce a separate binary without all debug calls (find /path/to/folder -type f -exec sed -i '/^log\.Debug/d' {} +).

This is a good idea, but only works for single-line debug commands. In any case, we use logger.

Where is our good old #define ? 😞

There is a super expensive (for who is doing it) but simple idea: if logger is debug, then code for the debug calls.

Maybe we can encapsulate the debug log and use build tags to disable it? Something along the lines
// +build !debug

package main

import (
    log "github.com/sirupsen/logrus"
)

func debugLog(logger *log.Logger, msg string) {
    // Do nothing, debug logging is disabled
}

got some ideas on how this might work. When I have a bit of time, will try to implement that

Not sure if we should close this issue, although @andynog's solution is a way to go. The option to move to another provider (e.g., slog) is still in the table.

Hi @ValarDragon, should we leave this open ? We've implemented a solution in #4365 that in certain ways is a solution to the problem.
Please let us know. Thanks !


定时任务不执行 需要有人访问页面才能触发任务

如果长时间没人访问网站，任务就不执行，需要再次访问下网站才能触发任务。

It's hard to say what could be causing this without having sight of a little more implementation details - could you please elaborate? (English preferable, regret that I cannot read or understand Chinese).


bug fix for componentType is null

There is an error for the inner class handling.
class A {
@JvmField var b: B? = null
}
class B {
@JvmField var value: String? = null
}

Thank you.


Challenge workflows - Stage 4

This updates the Dashboard to use the new email verification workflows
to:

Re-send email verifications (for new users).
Create an email verification challenge (when updating the user's email).

For context, see: https://github.com/aptible/auth.aptible.com/pull/255
--
Note: this includes https://github.com/aptible/dashboard.aptible.com/pull/777, but that PR is expected to ship earlier (see the rollout plan in https://github.com/aptible/auth.aptible.com/pull/255). The relevant commit is the other one (Stage 4).
cc @sandersonet @fancyremarker @blakepettersson

Moved to https://github.com/aptible/dashboard-private/pull/21


Update azip-integration.md

added this line to the Note:

In the label configuration, permissions should be assigned to any authenticated user or all users in your organization for Defender for cloud apps to be able to read label information.

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

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

@dcurwin
Thanks for reviewing the proposed changes. IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

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


LESSPlugin: change filename path from relative to absolute

This patch fix @import resolving inside the less file, if main less file isn't in root directory.

Ok, I will merge it now. Thanks,  Please take a look at the 2.0 branch. The release is soon!
There will be many changes to the API https://github.com/fuse-box/preview-1.4.1/tree/master/examples/react-code-splitting


Support alternate group names in get_*_extensions

These functions are brief, but it would be nice not to have to duplicate them in other colcon packages which re-use the same extension frameworks.

Example of such duplication: https://github.com/colcon/colcon-installed-package-information/blob/18a770ee6e6f07d06300568660e9c6b62d17d4b7/colcon_installed_package_information/package_identification/init.py#L8


Add HTML format to the book_export test and fix the format to work properly

The export_book method in the _html.py format file was not being called by tests.  Adding it causes this error:
ERROR: test_book_export_no_exceptions (__main__.TablibTestCase)
Test that various exports don't error out.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test_tablib.py", line 395, in test_book_export_no_exceptions
    book.html
  File "/home/vagrant/development/tablib/tablib/formats/_html.py", line 63, in export_book
    stream.write('<%s>%s</%s>\n' % (BOOK_ENDINGS, title, BOOK_ENDINGS))
TypeError: 'str' does not support the buffer interface

This PR also fixes that error largely by copying the way that export_set method does.

Thanks!


Fix function comments based on best practices from Effective Go

Every exported function in a program should have a doc comment. The first sentence should be a summary that starts with the name being declared.
From effective go.
PR generated by CodeLingo. Install here to drive Continuous Higher Standards.

Hi

Hi there!
@BlakeMScurr from CodeLingo here. This PR is testing out a new service. I'd love to hear your feedback or answer your questions. We're keen to learn how we can best help dev teams without getting in your way.
@BlakeMScurr & the CodeLingo Team

Hi, @BlakeMScurr thanks for the PR.
Not sure what kind of feedback your looking for, due to the small nature of the change. It seems like a cool free service though. I guess the best feedback I can give you is to perhaps add spell checking capabilities, so that the bot could have caught the typos mentioned in #418

No problem @gbolo 😄
Thanks for the feedback, I'll put spell checking in the roadmap. Installing on GitHub is also free by the way, which will check all your new PRs.
Is it a rather humble little PR isn't it? 😆 It's small and and looks like it could almost be done with grep. However, this is actually the start of a growing library of ever higher code standards. So you can expect some much more interesting AST heavy PRs soon - assuming that wouldn't bother you!


Unable to biuld px4 v 1.9.0 on beagle bone blue native

Describe the bug
A clear and concise description of the bug.
To Reproduce
Steps to reproduce the behavior:

Drone switched on '...'
Uploaded mission '....' (attach QGC mission file)
Took off '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Log Files and Screenshots
Always provide a link to the flight log file:

Download the flight log file from the vehicle (tutorial).
Share the link to a log showing the problem on PX4 Flight Review.

Add screenshots to help explain your problem.
Drone (please complete the following information):

Describe the type of drone.
Photo of the IMU / autopilot setup if possible.

Additional context
Add any other context about the problem here.

Here is a log of error massage;
debian@beaglebone:/var/lib/cloud9/test/Firmware$ make beaglebone_blue_native
-- PX4 config file: /var/lib/cloud9/test/Firmware/boards/beaglebone/blue/native.cmake
-- PX4 config: beaglebone_blue_native
-- PX4 platform: posix
-- PX4 lockstep: disabled
-- PX4 version: v1.9.0-rc0
-- cmake build type: RelWithDebInfo
-- The CXX compiler identification is GNU 6.3.0
-- The C compiler identification is GNU 6.3.0
-- The ASM compiler identification is GNU
-- Found assembler: /usr/bin/cc
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found PythonInterp: /usr/bin/python (found version "2.7.13")
-- Found PY_jinja2: /usr/lib/python2.7/dist-packages/jinja2
DF Drivers: mpu9250;bmp280
Adding DF driver: mpu9250
Adding DF driver: bmp280
-- PX4 ECL: Very lightweight Estimation & Control Library v0.9.0-676-gc4492b1
-- Configuring done
-- Generating done
-- Build files have been written to: /var/lib/cloud9/test/Firmware/build/beaglebone_blue_native
Scanning dependencies of target df_bmp280
Scanning dependencies of target df_mpu9250
Scanning dependencies of target df_driver_framework
Scanning dependencies of target uorb_headers
[  0%] Building CXX object src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/drivers/mpu9250/CMakeFiles/df_mpu9250.dir/MPU9250.cpp.o
[  0%] Generating uORB topic headers
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DriverFramework.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/WorkMgr.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/WorkItems.cpp.o
/var/lib/cloud9/test/Firmware/src/lib/DriverFramework/drivers/bmp280/BMP280.cpp: In member function ‘int DriverFramework::BMP280::loadCalibration()’:
/var/lib/cloud9/test/Firmware/src/lib/DriverFramework/drivers/bmp280/BMP280.cpp:124:58: error: cast from ‘uint8_t* {aka unsigned char*}’ to ‘uint16_t* {aka short unsigned int*}’ increases required alignment of target type [-Werror=cast-align]
uint16_t *calib_value_ptr = (uint16_t *) &calib_values[0];
^
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/build.make:62: recipe for target 'src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o' failed
make[3]: *** [src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o] Error 1
CMakeFiles/Makefile2:1484: recipe for target 'src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/all' failed
make[2]: *** [src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....

debian@beaglebone:/var/lib/cloud9/test/Firmware$ make beaglebone_blue_native
-- PX4 config file: /var/lib/cloud9/test/Firmware/boards/beaglebone/blue/native.cmake
-- PX4 config: beaglebone_blue_native
-- PX4 platform: posix
-- PX4 lockstep: disabled
-- PX4 version: v1.9.0-rc0
-- cmake build type: RelWithDebInfo
-- The CXX compiler identification is GNU 6.3.0
-- The C compiler identification is GNU 6.3.0
-- The ASM compiler identification is GNU
-- Found assembler: /usr/bin/cc
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found PythonInterp: /usr/bin/python (found version "2.7.13")
-- Found PY_jinja2: /usr/lib/python2.7/dist-packages/jinja2
DF Drivers: mpu9250;bmp280
Adding DF driver: mpu9250
Adding DF driver: bmp280
-- PX4 ECL: Very lightweight Estimation & Control Library v0.9.0-676-gc4492b1
-- Configuring done
-- Generating done
-- Build files have been written to: /var/lib/cloud9/test/Firmware/build/beaglebone_blue_native
Scanning dependencies of target df_bmp280
Scanning dependencies of target df_mpu9250
Scanning dependencies of target df_driver_framework
Scanning dependencies of target uorb_headers
[  0%] Building CXX object src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/drivers/mpu9250/CMakeFiles/df_mpu9250.dir/MPU9250.cpp.o
[  0%] Generating uORB topic headers
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DriverFramework.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/WorkMgr.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/WorkItems.cpp.o
/var/lib/cloud9/test/Firmware/src/lib/DriverFramework/drivers/bmp280/BMP280.cpp: In member function ‘int DriverFramework::BMP280::loadCalibration()’:
/var/lib/cloud9/test/Firmware/src/lib/DriverFramework/drivers/bmp280/BMP280.cpp:124:58: error: cast from ‘uint8_t* {aka unsigned char*}’ to ‘uint16_t* {aka short unsigned int*}’ increases required alignment of target type [-Werror=cast-align]
uint16_t *calib_value_ptr = (uint16_t *) &calib_values[0];
^
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/build.make:62: recipe for target 'src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o' failed
make[3]: *** [src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/BMP280.cpp.o] Error 1
CMakeFiles/Makefile2:1484: recipe for target 'src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/all' failed
make[2]: *** [src/lib/DriverFramework/drivers/bmp280/CMakeFiles/df_bmp280.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DevMgr.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DevObj.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/drivers/mpu9250/CMakeFiles/df_mpu9250.dir/MPU9250_mag.cpp.o
[  0%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/SyncObj.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DFList.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/Time.cpp.o
[  1%] Linking CXX static library libdf_mpu9250.a
[  1%] Built target df_mpu9250
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/DFDiag.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/I2CDevObj.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/SPIDevObj.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/ImuSensor.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/BaroSensor.cpp.o
[  1%] Building CXX object src/lib/DriverFramework/framework/src/CMakeFiles/df_driver_framework.dir/MagSensor.cpp.o
[  2%] Linking CXX static library libdf_driver_framework.a
[  2%] Built target df_driver_framework
[  2%] Built target uorb_headers
Makefile:160: recipe for target 'all' failed
make[1]: *** [all] Error 2
Makefile:190: recipe for target 'beaglebone_blue_native' failed
make: *** [beaglebone_blue_native] Error 2

It's the cast align warning (-Werror=cast-align). What we really need to do is update this build and get it in the CI system. I'd like to either bring librobotcontrol in as a submodule or via cmake ExternalProject. We should also be able to dump the native/cross configs. A single default can work for both with the architecture specified.

You can work around a warning by removing -Werror for now here:
https://github.com/PX4/Firmware/blob/b35d0485669fd47173eb742bc0a08cb0dbac81f0/cmake/px4_add_common_flags.cmake#L66


buildPatchFromRevert

Better alternative to https://github.com/JSON8/patch/pull/10
In #10 the revert property would be a valid JSON patch but it comes with a cost.
With this, we don't break the API, the revert property is still not a valid JSON patch but can be transformed to it easily using buildPatchFromRevert
var doc = {foo: 'bar'}
var patch = [...]
var patching = ooPatch.apply(doc, patch, {reversible: true})
doc = patching.doc

//
// know you can revert 2 different ways
//

// the old way
doc = ooPatch.revert(doc, patching.revert)

// OR

// the new way - ideal for packing/unpacking, saving, ...
var revertPatch = ooPatch.buildPatchFromRevert(patching.revert) // valid JSON Patch
doc = ooPatch.apply(doc, revertPatch)

This looks like a big patch it can be resumed to rename foo method to 'buildPatchFromRevert` and export it. The rest is documentation and moving code around for better clarity and smaller builds.

idea for next PR:
apply applies the patch only, no options
patch === curent apply(doc, patches, {reversible: truc}

this is not a breaking change so merging into develop, please feel free to comment and I may address in an other PR


UI Framework

This is really important to start using Cosmic for UI in visual applications. Essentially it will be a declarative/reactive framework that can be easily used in js. It should also be usable within Zig code as well for dev hud and other cool stuff yet to come. Details about what this looks like will come later when I revisit this. Even though some work has already been done, much work is needed to develop a standard library of widgets. This ticket will track the progress.

#21 Adds a ui library in zig. It compiles to desktop and wasm/webgl2.
Leaving this ticket open for an equivalent JS API.


Add support for numbers 0-9

This can be really easily done by just adding box 4 in the letter configuration. Digit 0 will sit in the last square of box 3 (_>...), whilst the other 9 numbers will spread out evenly across box 4.
An example of the number 101 would be:
^<...._>...^<....

This has been resolved by #3


Add node selector to identify a sub-cluster for batch job

Currently, we cached all node in kube-bached and assign node to pod accordingly. But for batch job, the conflict is so heavy :(. We'll use 'static' zone for batch job before finding a "good" way to work together with kube-scheduler.
List some tasks (including but not limited to)

[x] Filter pods with scheduler name for kube-batchd (#170)
[x] Filter nodes with labels for kube-batchd (#203)
[ ] Filter nodes with NodeAffinity for kube-batchd

No one is working on it, right?
Is this the requirement? "Nodes, having a particular label X, will be considered by kube-batchd"

The feature was done; close this.


Add support for overriding non-selected Khronos vendor options

This PR is broken into two commits:

The first commit cleans up some of the validation logic and ensures that a vendor plugin is enabled prior to executing logic against it (e.g: showing warnings)
The second commit showcases how we can support multiple vendors under the Khronos loader while keeping a clean user interface. To do so, it makes use of the _get_export_options_overrides(...) method introduced in Godot 4.3. An alternative would be to use _get_export_option_visibility(...) instead, but that method is not yet available to the EditorExportPlugin.

Note that the approach used in the second commit is up for discussion, and may not be merged in if we decide to go a different route.

This indeed works well, though I'm guessing that once _get_export_option_visibility becomes possible that would be a better solution.
It would also be nice that if the Khronos option isn't selected, all entries related to Khronos aren't shown. The same with the other selections. But that is a nice to have.
I'm happy if we merge this as is with a note that we improve upon this when we can. I'll rebase my MagicLeap PR and add the needed changes in.

I think we should merge this as it is now, and then improve it over time.

Sounds good!
@dsnopek Any thoughts on this approach?


[client + contracts] Roads

Overhaul Roads

Hi @ponderingdemocritus can you provide more details on this

sorry @kateberryd ! this is a very involved task. I may have other tasks, can you reach out to me in the discord https://discord.gg/DuXRzPJa


Tune http.Client's Transport's MaxIdleConns/MaxIdleConnsPerHost

In doing some work on another project, i've learnt far more than I ever expected to about go's http.Client, and how it's defaults may not be ideal. We're already setting timeouts, which is good, but the defaults around idle connection reuse are pretty woeful:
This is our current settings:

https://github.com/OJ/gobuster/blob/master/libgobuster/http.go#L51-L59

Some refs:

https://medium.com/@valyala/net-http-client-has-the-following-additional-limitations-318ac870ce9d
https://hackernoon.com/avoiding-memory-leak-in-golang-api-1843ef45fca8#e81a

Looking at the source for net/http/transport.go we can see that the MaxIdleConnections is 100, but DefaultMaxIdleConnsPerHost is only 2:
var DefaultTransport RoundTripper = &Transport{
	Proxy: ProxyFromEnvironment,
	DialContext: (&net.Dialer{
		Timeout:   30 * time.Second,
		KeepAlive: 30 * time.Second,
		DualStack: true,
	}).DialContext,
	MaxIdleConns:          100,
	IdleConnTimeout:       90 * time.Second,
	TLSHandshakeTimeout:   10 * time.Second,
	ExpectContinueTimeout: 1 * time.Second,
}

// DefaultMaxIdleConnsPerHost is the default value of Transport's
// MaxIdleConnsPerHost.
const DefaultMaxIdleConnsPerHost = 2

What this means in practical terms is that for every 100 connections we open to a webserver, we close 98 of them, which leads to a lot of sockets left in our kernel in the TIME_WAITstate, and also means we need to pay the overhead of establishing a new TCP (and/or TLS) handshake on those 98 closed sockets.
I'm not 100% on this, but I think the best setting would be to set MaxIdleConns == MaxIdleConnsPerHost == number of goroutines/'threads' specified.

https://github.com/golang/go/blob/master/src/net/http/transport.go#L191-L216

Then if we make sure that we set keep-alive on requests to the server, if supported, it should mean that we get a lot more reuse per socket, which should mean faster gobusting.
For this to work, you also have to ensure you fully read/close the response body:

Just be sure to fully read (and/or close) the Response.Body, otherwise that TCP connection won't be re-used.

It looks as though the entire body is already being read, in these areas, so this should be fine:

https://github.com/OJ/gobuster/blob/master/libgobuster/http.go#L111
https://github.com/OJ/gobuster/blob/master/libgobuster/http.go#L119-L121

There are a LOT more little things that can be tuned in the http.Client's Transport, but not sure if/what would make as much difference as this.
Making use of keep-alive was one of the techniques discussed in this research:

https://portswigger.net/blog/turbo-intruder-embracing-the-billion-request-attack
https://youtu.be/vCpIAsxESFY?t=211

If anyone wants to benchmark the speed/resource difference before/after, i'd be pretty interested to see just how much of an improvement it makes, but from the above video, it claims about a 400% increase from reusing the connections (and like 6000% if you want to use http pipelining..)

I'll try to get to this very soon. Nice research @0xdevalias thank you!

came here to ask about this, i am looking for a simple way to control the amount of connections i am establishing via the command line.

Did some quick testing on windows and there is indeed a speedup.
Default values from the optimization branch:
PS C:\Users\firefart\Documents\code\gobuster> Measure-Command { .\v3.exe dir -u https://firefart.at -w .\wordlist_small.txt }
2019/04/17 21:12:23 Starting gobuster
2019/04/17 21:12:39 Finished%)

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 16
Milliseconds      : 702
Ticks             : 167029543
TotalDays         : 0,000193321230324074
TotalHours        : 0,00463970952777778
TotalMinutes      : 0,278382571666667
TotalSeconds      : 16,7029543
TotalMilliseconds : 16702,9543

PS C:\Users\firefart\Documents\code\gobuster> Measure-Command { .\v3.exe dir -u https://firefart.at -w .\wordlist_small.txt }
2019/04/17 21:12:42 Starting gobuster
2019/04/17 21:12:59 Finished%)

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 16
Milliseconds      : 918
Ticks             : 169185885
TotalDays         : 0,000195816996527778
TotalHours        : 0,00469960791666667
TotalMinutes      : 0,281976475
TotalSeconds      : 16,9185885
TotalMilliseconds : 16918,5885

And after setting the following values:
	client.client = &http.Client{
		Timeout:       opt.Timeout,
		CheckRedirect: redirectFunc,
		Transport: &http.Transport{
			Proxy:               proxyURLFunc,
			MaxIdleConns:        100,
			MaxIdleConnsPerHost: 100,
			TLSClientConfig: &tls.Config{
				InsecureSkipVerify: opt.InsecureSSL,
			},
		}}

PS C:\Users\firefart\Documents\code\gobuster> Measure-Command { .\v3.exe dir -u https://firefart.at -w .\wordlist_small.txt }
2019/04/17 21:13:12 Starting gobuster
2019/04/17 21:13:27 Finished%)

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 14
Milliseconds      : 976
Ticks             : 149768399
TotalDays         : 0,000173343054398148
TotalHours        : 0,00416023330555556
TotalMinutes      : 0,249613998333333
TotalSeconds      : 14,9768399
TotalMilliseconds : 14976,8399

PS C:\Users\firefart\Documents\code\gobuster> Measure-Command { .\v3.exe dir -u https://firefart.at -w .\wordlist_small.txt }
2019/04/17 21:13:29 Starting gobuster
2019/04/17 21:13:44 Finished%)

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 14
Milliseconds      : 804
Ticks             : 148046087
TotalDays         : 0,000171349637731481
TotalHours        : 0,00411239130555556
TotalMinutes      : 0,246743478333333
TotalSeconds      : 14,8046087
TotalMilliseconds : 14804,6087

So with default parameters it takes around 17 seconds for a 5000 entry wordlist and with both parameters set to 100 we are around 15 seconds.
So I think we should use this optimization and hope we will not kill many servers with it :)

I created a PR over here:
https://github.com/OJ/gobuster/pull/140


Problem with mapping of Artist / Object 9895

--Update, tested this for Object 34753 too which has an individual artist and the same problem persists (not specific to institutions as the example cites below).
Our constituents file is split into 6 subfiles...not sure why this is but I think it's preventing the validator from reading our artists names.
I expected to see:
Artist name "Tiffany and Company" -- Note there is a separate model for institution/individual/culture in our data (I can understand why but this freaks me out).
but instead I saw:
Nothing/Blank
The current query was:
PREFIX owl: <http://www.w3.org/2002/07/owl#>
PREFIX rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX crm: <http://www.cidoc-crm.org/cidoc-crm/>
PREFIX skos: <http://www.w3.org/2004/02/skos/core#>
PREFIX xsd: <http://www.w3.org/2001/XMLSchema#>
PREFIX foaf: <http://xmlns.com/foaf/0.1/>
PREFIX dct: <http://purl.org/dc/terms/>
PREFIX dc: <http://purl.org/dc/elements/1.1/>
PREFIX schema: <http://schema.org/>

SELECT ?actor_name ?actor_type ?actor_class
WHERE {
<http://data.thewalters.org/object/9895> a crm:E22_Man-Made_Object ;
   crm:P108i_was_produced_by ?production_event .

?production_event a crm:E12_Production ;
  crm:P14_carried_out_by ?actor_class .

OPTIONAL {
  ?actor_class a ?actor_type .
}
OPTIONAL {
  ?actor_class rdfs:label ?actor_name.
}

}

Yup—this is a valid issue.  The production (http://data.americanartcollaborative.org/page/wam/object/9895/production) does not have any Actors associated with it.

Fixed it


Add docker support for ghidra

It is essential to have the docker support and this will ensure to run the tool in a sandbox environment and easy to use.

Can you be more specific?  Is there some aspect of Ghidra that is preventing it from working in a docker container or are you asking for official pre-made containers to be generated and distributed?  Thanks!

Note also:
https://hub.docker.com/r/0xf15h/ghidra
https://github.com/kjcolley7/docker-ghidra-server

Thanks @d-millar.  It looks like the community is already on top of the docker support.

Update - also:
https://github.com/bskaggs/ghidra-docker
https://github.com/blacktop/docker-ghidra


not getting git icon on jup notebook

i have did all the steps for installing and set the id_rsa.pub in my git account.
but when i run the jupyter notebook i didnt see any of the git icon for commiting and pushing
btw im hosting my notebook on aws ec2 ubuntu instance
plz helppppppp

Ishaq4466, check to see where your Anaconda is installed. I was able to make this work on Windows and also RHEL 7. Make sure your Anaconda is installed in ROOT. If its installed on /OPT then I know for a fact it wouldn't work. Let me know of that works for you. Please update this post when done. Thanks again .

@ishaq4466 please make sure the extension is enabled by running the commands -
jupyter serverextension enable --py githubcommit
jupyter nbextension install --py githubcommit

The resultant of these command will be validating - Ok


Implement and test login with the anonymous user for SSL enabled iRODS

From the TODO comment in session.py.

More information:
Currently we implemented Login for regular users to iRODS and SSL enabled iRODS.
The anonymous user is a special user which can authenticate with iRODS and then use iRODS tickets to get the authorisation to a certain collection or data object.
The commented code implements the authentication for the anonymous user to an iRODS instance which does not use SSL encryption. This needs to be extended to SSL enabled iRODS instances.


[Github] File Size

New badge with specific Github file size information.
Related issue : #730

So this is displaying the git repo size? This could be confused with some binary package size. Maybe this would be more appropriate?

I used lightgrey here because I don't really see why the size should be displayed in green. The size isn't positive or negative, it's just informational.

@olivierlacan hi,
Not repo size. File size.
For client-side libraries ( lodash , underscore , phaser , etc ...)
Its easier to lib display if it is lightweight on readme.md

Hi, I'd like to get this merged. Would you be willing to add tests using the project's new testing framework?

@paulmelnikow done
Awesome job on this new testing setup. 👍
I might create a new PR populating this new github service-test

Thanks, I'm glad to hear that!
Testing more of the github services would be great! Feel free to open an issue to coordinate, if you'd like. Another open PR (#756) has tests of the downloads endpoint, though the rest of the badges are wide open. There is an issue that is causing github to be flaky in CI, which will also need to be fixed at some point (observed here, see this log).
Will leave you a few minor comments.

Moved to #976

Thanks for jumping on those tests!
It would be easier to me to finish up the review I started here. Small and atomic PRs are easier, and plus we've already begun the conversation. Hopefully we can merge this in the next day or two, and then proceed with the rest of the tests in #976.

@paulmelnikow ok i'll organize this

@paulmelnikow this is done.
Atomic PRs make sense. Make things more splited and organized.
I'll reorganize #976 now

@paulmelnikow done
Lmk if you need any changes.
I'll keep my eyes on the #976

After merge this we can close #730

I have no clue what's this travis error right now

I don't see the error.
Thanks for the changes!  Do you have a clue what happens by any chance? @Daniel15 We can't enable that to be part of npm test unless all of it passes.
1) Github File size
        [ GET http://localhost:1111/github/size/webcaetano/craft/build/craft.min.js.json ]:
     ValidationError: child "value" fails because ["value" with value "repo or file not found" fails to match the required pattern: /^[0-9]*[.]?[0-9]+\s(B|kB|MB|GB|TB|PB|EB|ZB|YB)$/]
      at Object.exports.process (node_modules/joi/lib/errors.js:181:19)
      at Object._validateWithOptions (node_modules/joi/lib/types/any/index.js:651:31)
      at Object.root.validate (node_modules/joi/lib/index.js:121:23)
      at Object.pathMatch.matchJSONTypes (node_modules/icedfrisby/lib/pathMatch.js:299:9)
      at node_modules/icedfrisby/lib/icedfrisby.js:721:10
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1200:33)
      at node_modules/icedfrisby/lib/icedfrisby.js:1179:29
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:561:11)
      at Request.self.callback (node_modules/request/request.js:188:22)
      at Request.<anonymous> (node_modules/request/request.js:1171:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1091:12)
      at endReadableNT (_stream_readable.js:975:12)
      at _combinedTickCallback (internal/process/next_tick.js:80:11)
      at process._tickDomainCallback (internal/process/next_tick.js:128:9)

@espadrine yes i do.
That was my fault. I changed the file name of my repo recently.
@espadrine i'll make a PR fixing that

@webcaetano OK thanks a lot! Don't worry, there's no rush, I won't look at it until next Saturday (but I will definitely look at it on Saturday if the PR is ready ☺)

@espadrine #1066

Hi, thanks for your question. Sorry this isn't clear.
If you go to https://shields.io/#/examples/size and click on the badge you'll see a page for the badge, which shows an example badge URL:
https://img.shields.io/github/size/webcaetano/craft/build/phaser-craft.min.js.svg
webcaetano/craft is the repo and build/phaser-craft.min.js is the filename. Hope that helps!


fix(sol): sol connect and command window

Codecov Report

Merging #353 (9700ce7) into master (8b81d5e) will decrease coverage by 0.42%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #353      +/-   ##
==========================================
- Coverage   55.83%   55.40%   -0.43%
==========================================
  Files          39       39
  Lines         969      971       +2
  Branches       88       90       +2
==========================================
- Hits          541      538       -3
- Misses        395      400       +5
  Partials       33       33

Impacted Files
Coverage Δ

src/app/devices/kvm/kvm.component.ts
61.76% <0.00%> (-2.42%)
:arrow_down:

src/app/devices/sol/sol.component.ts
54.90% <0.00%> (-3.10%)
:arrow_down:

src/app/shared/add-device/add-device.component.ts
86.36% <0.00%> (-2.28%)
:arrow_down:

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 e70a04c...9700ce7. Read the comment docs.


Refine any type

Current generated document not accurately.

this PR make those more accurately.

What's the problem? I don't see any inaccuracy. Any within the YAML/JSON namespaces refers to YAML::Any/JSON::Any, respectively.

What's the problem? I don't see any inaccuracy. Any within the YAML/JSON namespaces refers to YAML::Any/JSON::Any, respectively.

newbie user for use YAML.parse don't know what is the Any means, tell the user exactly #as_a will return Array(YAML::Any) is better than only Array(Any), right?
Analysis from another angle, if you want return a JSON::Any from some convert methods, what is the Any stand for?

You can click Any and you will know.

You can click Any and you will know.

Okay, click jump to same position.
So, Do we consider give uniform format is more neatly? (check following screenshot)

I mean, if you want the fully qualified name then it's a doc generator change. But types lookup in code works the same as in docs. If you know one you should understand the other.

if you want the fully qualified name then it's a doc generator change

@asterite, although titled as document improvement, but that not the only reason:

Use same style(all qualified or all not qualified) is better, for std-lib, if let me choice, i always prefer later, why not all use same form?
qualified name will result in a better performance? not sure, i means, ::YAML::Any is fairly straightforward, don't need lookup Any belong to which namespace, right?

why not all use same form?

Writing out the fully qualified name of every constant adds a lot of clutter and makes the code hard to read. In the context of JSON namespace, it should be sufficiently clear that Any refers to JSON::Any, not any of the other anys out there.

qualified name will result in a better performance? not sure, i means, ::YAML::Any is fairly straightforward, don't need lookup

Even a fully qualified name needs lookup of the name spaces for validation. I won't comment on any performance consideration of either variant because it's absolutely negligible. It doesn't make any difference.
This is entirely about developer ergonomics, not compiler performance.


Support extra args for tests

Supports passing final arguments (after --) that will be forwarded to the test runners.
Example use case: specifying a test filter:
cargo flaky -i 100 --release -- bug_4640

will pass bug_4640 to each test binary that gets run

Looks good to me, I believe this superseed #12 right?
@SabatierBoris since this PR also updates a bunch of stuff and use the more common format in rust cargo flaky -- extra-args-here I'm going to merge this one instead of yours sorry.

@irevoire You're right, this one is better that mine, but it do not fix my issue : set "feature flag".
With this PR, no "extra argument" is sent to "cargo build".
Il will rebase my PR and try to add this feature.
BTW, thanks @dureuill, I was looking to add this feature, very usefull when you want debug 1 flaky test

@SabatierBoris Oh ok my bad! 🙈

Il will rebase my PR and try to add this feature.

Yes please! Thanks a lot


Add content/pt/docs/concepts/architecture/cloud-controller.md

Hey @jcjesus . Could you please have a look this PR?
you can review the page here as well: https://deploy-preview-13979--kubernetes-io-master-staging.netlify.com/pt/docs/concepts/architecture/cloud-controller/
/assign @jcjesus

@femrtnz in cloud-controller.md file, you translated Service Controller to Portuguese, but Node Controller and other not. I will suggest you translate all or neither.

@femrtnz I will suggest to you translate "Design de Arquitetura" to "Projeto de Arquitetura (Design)"

Hi @femrtnz!
I will suggest to you translate "Controlador de Node" to "Controlador de Nós", to not confusing with "NodeJS", hehehe

/assign @femrtnz

Hi @femrtnz!
I will suggest to you translate "Controlador de Node" to "Controlador de Nós", to not confusing with "NodeJS", hehehe
In this file, where contains the word "Node" I think we need translate to "Nó" and keep the pattern to translation , 'cause we have "Node Controller" and "Controlador de Node" in the same file.

@jcjesus Thanks for the review. I'm just worried to translate a specific Kubernetes name where we can lose the meaning within the context. e.g, using kubectl get nodes


Broken profile for Saurabh Thakur

Description
Saurabh Thankur's profile on the meshery.io site does not appear.

I popped in a quick fix for @thakursaurabh1998's profile.


Default installation will not display images in page blocks

Hi,
Firstly great product, really looking forward to getting to use it & know it more.
I've noticed that when I upload images to a page block they do not display & they upload my images to public/storage/public/blocks/(myimage).jpg
Here is the actual filepath of an upload. http://mysite.test/storage/public/blocks/sj6CJ28kOWxdbBuHX4QWrh5N5pvE7zDlbWllXl12.jpeg
My filesystems.php is the default settings for laravel 7 & I can upload and view images perfectly fine for Laravel Voyager.
I have the following dependencies...
        "php": "^7.3",         "ext-bread-fields-fix": "dev-master",         "ext-json": "*",         "algolia/algoliasearch-client-php": "^2.7",         "bugsnag/bugsnag-laravel": "^2.0",         "fideloper/proxy": "^4.2",         "fruitcake/laravel-cors": "^1.0",         "guzzlehttp/guzzle": "^6.3",         "laravel/framework": "^7.0",         "laravel/tinker": "^2.0",         "laravel/ui": "^2.0",         "pvtl/voyager-blog": "^0.2.9",         "pvtl/voyager-forms": "^0.4.7",         "pvtl/voyager-frontend": "^0.9.8",         "pvtl/voyager-page-blocks": "^0.9.0",         "tcg/voyager": "^1.4"
This is the filesystem driver set in .env file
FILESYSTEM_DRIVER=public
Any support or guidance would be much appreciated!
Adam

remove this: FILESYSTEM_DRIVER=public

This worked perfectly thanks! Not as default installation as I thought.


issues with auto-complete in REPL

Describe the bug
Members get properly imported, and work, the function name is used, however, the auto-complete does not work.
For example, for Subprogmrams module (o1 auto-imported by the plugin), the function o1.play works (as has a shortcut reference via the extended object), whereas the function average works as well, but does not show in auto-complete.
Overall seems like an issue with REPLs internal functionality.
To Reproduce
Steps to reproduce the behavior:

Start a REPL for SubProgramms module
type o1.play (auto-complete works)
start typing o1.avera... (no auto-complete)
type o1.average and hit "send" - REPL complains about the function lacking parameters

Expected behavior
Auto-completes in p. 3
Screenshots

@jsorva is it something, that you've encountered already, by any chance?

Yes, I’ve seen similar things. My not-terribly-well-informed current understanding is that the IJ Scala plugin’s REPL’s auto-complete (and red-color-warning) are limited in how they handle Scala’s package objects and/or type aliases. I haven’t looked into it deeply. In any case, since we’re looking to transition to Scala 3.0 (which discards package objects as a concept), perhaps this can remain as a low-priority issue. Good to be aware of it though.

does not seem like something the plugin team needs to handle, so made an issue out of it just in case: https://youtrack.jetbrains.com/issue/SCL-18125

Closing this for now, as we have reported the issue to JetBrains and it should hopefully go away with 3.0 anyways.


Auto-scale title in a shape

basically adapt the fontSize, probably you can calculate that based on the number of characters in the title

@bjdmeest,
Could you elaborate a bit more?

How would you want the fontSize to be different in example 5? Solid Pod is a title.

Well, let's say the title is This is a Solid pod that stores many different RDF blocks nicely next to each other: decrease the font size so that that still fits within the with of that rectangle :)

I also auto-scaled labels.


[Feature request] High Score Card : link to settings in High Score area

button in menu to quickly jump to the Settings would be nice.

I'm not a 100% sure which settings do you mean. Another button to open the template editor for the selected table?

yes, a button that directly goes to the settings instead of multiple clicks out to get there.

Solved, 2.18.3


Update the CM MLPerf inference docs for CUDA device running on host

We need to update the MLPerf inference docs for native CUDA runs

Add a remark that unless CUDA, cuDNN and TensorRT are available in the environment it is recommended to use the docker option
In the run options specify the flags to pass in the cuDNN and TensorRT run files

Hi @arjunsuresh , this PR adds the first point.


fails

kubectl graph all -o cypher | cypher-shell -u neo4j -p secret
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x1da90d8]

goroutine 1 [running]:
k8s.io/api/core/v1.(*ObjectReference).GroupVersionKind(...)
        /home/runner/go/pkg/mod/k8s.io/api@v0.18.8/core/v1/objectreference.go:30
github.com/steveteuber/kubectl-graph/pkg/graph.(*CoreV1Graph).Endpoints(0xc00064c0f8, 0xc0000e0640, 0xc0000a8010, 0xc000450414, 0xa)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/core.go:160 +0x278
github.com/steveteuber/kubectl-graph/pkg/graph.(*CoreV1Graph).ServiceTypeClusterIP(0xc00064c0f8, 0xc0002d6900, 0x2126000, 0x1, 0xc0002d6900)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/core.go:199 +0x1f0
github.com/steveteuber/kubectl-graph/pkg/graph.(*CoreV1Graph).Service(0xc00064c0f8, 0xc0002d6900, 0xc0002d6900, 0x0, 0x0)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/core.go:178 +0x7d
github.com/steveteuber/kubectl-graph/pkg/graph.(*CoreV1Graph).Unstructured(0xc00064c0f8, 0xc0005d6018, 0x1, 0x1)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/core.go:64 +0x275
github.com/steveteuber/kubectl-graph/pkg/graph.(*Graph).Unstructured(0xc000721830, 0xc0005d6018, 0x3783308, 0xc00036c0a0)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/graph.go:200 +0x152
github.com/steveteuber/kubectl-graph/pkg/graph.NewGraph(0xc0000d8420, 0xc00064c0f0, 0x1, 0x1, 0x0, 0x0, 0x0)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/graph/graph.go:185 +0x157
github.com/steveteuber/kubectl-graph/pkg/cmd.(*GraphOptions).Run(0xc00031a8f0, 0x23d2ae0, 0xc0004bd6b0, 0xc0000e2dc0, 0xc0004bd8c0, 0x1, 0x3, 0x0, 0x0)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/cmd/cmd.go:164 +0x2cc
github.com/steveteuber/kubectl-graph/pkg/cmd.NewCmdGraph.func1(0xc0000e2dc0, 0xc0004bd8c0, 0x1, 0x3)
        /home/runner/work/kubectl-graph/kubectl-graph/pkg/cmd/cmd.go:80 +0x159
github.com/spf13/cobra.(*Command).execute(0xc0000e2dc0, 0xc0000a6050, 0x3, 0x3, 0xc0000e2dc0, 0xc0000a6050)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:846 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0xc0000e2dc0, 0x7, 0xc000142580, 0x238bce0)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
main.main()
        /home/runner/work/kubectl-graph/kubectl-graph/cmd/kubectl-graph/main.go:15 +0xda
Unable to connect to localhost:7687, ensure the database is running and that there is a working network connection to it.

Can you please add documentation if there is additional steps for running this plugin?

Hey @yashbhutwala,
Thank you for opening this issue. It seems like you have created a Service without a selector and added an Endpoint object manually. I have not tested that particular use case, but I will take a look into it and yes the documentation needs to be improved!

Hey @yashbhutwala,
I have updated the documentation with a usage example.
Tomorrow i'll fix the panic: runtime error: invalid memory address or nil pointer dereference.

I think this issue should be fixed now. Please upgrade the plugin to a newer version:
$ kubectl krew upgrade graph
Updated the local copy of plugin index.
  Upgrades available for installed plugins:
    * graph v0.1.0 -> v0.1.1
Upgrading plugin: graph
Upgraded plugin: graph

@yashbhutwala?

failure no longer happens, but the docs are still not clear to me to make use of this tool.  I spun up the neo4j container and I ran kubectl graph all -n kube-system -o cypher | cypher-shell..  now what?  I don't see a graph in the neo4j ui...

OK I see the results now.  2 things:

can you improve docs on how to explore within neo4j?  as a total noob to neo4j, it was kinda difficult for me.
can you add proper logging in your tool for failures, success, and long running processes?  since it takes a while to run and there is no log, it's easy to assume something's wrong or the program is hung...

Thanks for your feedback, I will improve these things!


Raxmlng - replace * with - for input file

Describe the bug
In the raxmlNG step, "*" in the fasta file need to be replaced with - or else it fails.
Impact
Fail to run
To Reproduce
Any alignment which has a *
Expected behavior
Complete tree file
Screenshots
Logs
-[nf-core/mycosnp] Pipeline completed with errors-
Error executing process > 'NFCORE_MYCOSNP:MYCOSNP:CREATE_PHYLOGENY:RAXMLNG (1)'
Caused by:
Process NFCORE_MYCOSNP:MYCOSNP:CREATE_PHYLOGENY:RAXMLNG (1) terminated with an error exit status (1)
Command executed:
raxml-ng
--all --model GTR+G --bs-trees 1000
--msa combined_vcf-to-fasta.fasta
--threads 2
--prefix output
cat <<-END_VERSIONS > versions.yml
"NFCORE_MYCOSNP:MYCOSNP:CREATE_PHYLOGENY:RAXMLNG":
raxmlng: $(echo $(raxml-ng --version 2>&1) | sed 's/^.RAxML-NG v. //; s/released.$//')
END_VERSIONS
Command exit status:
1
Command output:
ERROR: Invalid character in sequence 2 at position 1020: *
ERROR: Invalid character in sequence 2 at position 1600: *
ERROR: Invalid character in sequence 2 at position 1703: *
ERROR: Invalid character in sequence 2 at position 2043: *
ERROR: Invalid character in sequence 2 at position 3069: *
ERROR: Invalid character in sequence 2 at position 8679: *
ERROR: Invalid character in sequence 2 at position 11069: *
ERROR: Invalid character in sequence 2 at position 20803: *
ERROR: Invalid character in sequence 2 at position 21924: *
ERROR: Invalid character in sequence 2 at position 21925: *
ERROR: Invalid character in sequence 2 at position 25064: *
ERROR: Invalid character in sequence 2 at position 28986: *
ERROR: Invalid character in sequence 2 at position 29056: *
ERROR: Invalid character in sequence 2 at position 31209: *
ERROR: Invalid character in sequence 2 at position 36528: *
ERROR: Invalid character in sequence 2 at position 37319: *
ERROR: Invalid character in sequence 2 at position 37967: *
ERROR: Invalid character in sequence 2 at position 39231: *
ERROR: Invalid character in sequence 2 at position 39880: *
ERROR: Invalid character in sequence 2 at position 42175: *
ERROR: Invalid character in sequence 2 at position 42176: *
ERROR: Invalid character in sequence 3 at position 5206: *
ERROR: Invalid character in sequence 3 at position 9832: *
ERROR: Invalid character in sequence 3 at position 12678: *
ERROR: Invalid character in sequence 3 at position 14401: *
ERROR: Invalid character in sequence 3 at position 14402: *
ERROR: Invalid character in sequence 3 at position 14430: *
ERROR: Invalid character in sequence 3 at position 28941: *
ERROR: Invalid character in sequence 3 at position 28942: *
ERROR: Invalid character in sequence 3 at position 31911: *
ERROR: Invalid character in sequence 3 at position 32935: *
ERROR: Invalid character in sequence 3 at position 33868: *
ERROR: Invalid character in sequence 3 at position 36010: *
ERROR: Invalid character in sequence 3 at position 42164: *
ERROR: Invalid character in sequence 3 at position 42165: *
ERROR: Invalid character in sequence 4 at position 3069: *
ERROR: Invalid character in sequence 4 at position 8679: *
ERROR: Invalid character in sequence 4 at position 11069: *
ERROR: Invalid character in sequence 4 at position 20799: *
ERROR: Invalid character in sequence 4 at position 21924: *
ERROR: Invalid character in sequence 4 at position 21925: *
ERROR: Invalid character in sequence 4 at position 25064: *
ERROR: Invalid character in sequence 4 at position 29056: *
ERROR: Invalid character in sequence 4 at position 36528: *
ERROR: Invalid character in sequence 4 at position 37319: *
ERROR: Invalid character in sequence 4 at position 37967: *
ERROR: Invalid character in sequence 4 at position 39880: *
ERROR: Alignment check failed (see details above)!
Command wrapper:
ERROR: Invalid character in sequence 2 at position 1020: *
ERROR: Invalid character in sequence 2 at position 1600: *
ERROR: Invalid character in sequence 2 at position 1703: *
ERROR: Invalid character in sequence 2 at position 2043: *
ERROR: Invalid character in sequence 2 at position 3069: *
ERROR: Invalid character in sequence 2 at position 8679: *
ERROR: Invalid character in sequence 2 at position 11069: *
ERROR: Invalid character in sequence 2 at position 20803: *
ERROR: Invalid character in sequence 2 at position 21924: *
ERROR: Invalid character in sequence 2 at position 21925: *
ERROR: Invalid character in sequence 2 at position 25064: *
ERROR: Invalid character in sequence 2 at position 28986: *
ERROR: Invalid character in sequence 2 at position 29056: *
ERROR: Invalid character in sequence 2 at position 31209: *
ERROR: Invalid character in sequence 2 at position 36528: *
ERROR: Invalid character in sequence 2 at position 37319: *
ERROR: Invalid character in sequence 2 at position 37967: *
ERROR: Invalid character in sequence 2 at position 39231: *
ERROR: Invalid character in sequence 2 at position 39880: *
ERROR: Invalid character in sequence 2 at position 42175: *
ERROR: Invalid character in sequence 2 at position 42176: *
ERROR: Invalid character in sequence 3 at position 5206: *
ERROR: Invalid character in sequence 3 at position 9832: *
ERROR: Invalid character in sequence 3 at position 12678: *
ERROR: Invalid character in sequence 3 at position 14401: *
ERROR: Invalid character in sequence 3 at position 14402: *
ERROR: Invalid character in sequence 3 at position 14430: *
ERROR: Invalid character in sequence 3 at position 28941: *
ERROR: Invalid character in sequence 3 at position 28942: *
ERROR: Invalid character in sequence 3 at position 31911: *
ERROR: Invalid character in sequence 3 at position 32935: *
ERROR: Invalid character in sequence 3 at position 33868: *
ERROR: Invalid character in sequence 3 at position 36010: *
ERROR: Invalid character in sequence 3 at position 42164: *
ERROR: Invalid character in sequence 3 at position 42165: *
ERROR: Invalid character in sequence 4 at position 3069: *
ERROR: Invalid character in sequence 4 at position 8679: *
ERROR: Invalid character in sequence 4 at position 11069: *
ERROR: Invalid character in sequence 4 at position 20799: *
ERROR: Invalid character in sequence 4 at position 21924: *
ERROR: Invalid character in sequence 4 at position 21925: *
ERROR: Invalid character in sequence 4 at position 25064: *
ERROR: Invalid character in sequence 4 at position 29056: *
ERROR: Invalid character in sequence 4 at position 36528: *
ERROR: Invalid character in sequence 4 at position 37319: *
ERROR: Invalid character in sequence 4 at position 37967: *
ERROR: Invalid character in sequence 4 at position 39880: *
ERROR: Alignment check failed (see details above)!
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Commit 1b71bac closes this issue


Multi quic rebase.revert reuseaddr

Problem
test revert reuseaddr changes
Summary of Changes
Fixes #

Closed for testing purpose. It will be driven by the main PR.


Write unit tests for bot/cogs/doc.py

Write unit tests for bot/cogs/doc.py.
Implementation details
Please make sure to read the general information in the meta issue and the testing README. We are aiming for a 100% branch coverage for this file, but if you think that is not possible, please discuss that in this issue.
Additional information
If you want to work on this issue, please make sure that you get assigned to it by one of the core devs before starting to work on it. We would like to prevent the situation that multiple people are working on the same issue. To get assigned, leave a comment showing your interesting in tackling this issue.

Internal discussions deemed these tests to be unnecessary.


Status code: 400, status: undefined.

bloxy returns a status code 400 for a bad request when using any command that requires a roblox-username input target

Hi @unfetchable,
Sorry for the delay here, I missed the issues on this repo entirely due to me being busy.
Were you able to resolve this issue, or did you need to bypass something in order to get things to work?
Please let me know what you were doing, so as for the new version we can avoid it in its entirety.
I'm closing this issue for now due to the time I spent getting to this.


[wagumi-sbt]add wagumi-sbt method

Additional snapshot strategies for Wagumi sbt holders

I may also use api-v2, as it is currently a viable alternative.
However, we will need to add a dedicated strategy in the future and will contact you again at that time.
@ChaituVR

For sure :) Feel free to create new one when ever you are ready