Project Creation, Listing, Editing, Membership

Background
bcdevexchange.org is experimenting with a public/private collaboration framework called Code With Us where government program owners are able to post coding and other digital based opportunities.
During this iteration the BC Developers' Exchange is creating a better way for government program owners to create Code With Us opportunities, provide work context and facilitate collaboration.
Related to issues design 1, design 3 and design 4
Story
As a government program lead (owner) I need to be able to create a program project that will provide background information (context) for opportunities that are listed on the Code With Us page of the BC Developers' Exchange.
How to contribute
We are offering this issue as a paid opportunity through our experimental procurement model, Code-with-Us.
To work on this issue, we are accepting proposals until 5:00 PM PDT on Monday, October 31, 2016. After that time, we will evaluate the proposals we’ve received, and assign the work to one person by 5:00 PM PDT on Wednesday, November 2.
How assignment works
What to include in your proposal
We will evaluate and score proposals by the following criteria:

Your ability to dedicate the time needed to complete the issue by November 21, 2016 (Mandatory).
The estimated date you think you can show us your first iteration (Mandatory).
A short (no more than 200 words) summary of your proposed approach (60 points).
References to your relevant experience and demonstrated ability to do the work (40 points). For example, a link to your GitHub projects.
With your proposal, you must attach a copy of the Code-with-Us Terms, with the required information asked for in the “Acceptance” section of the Terms inserted into the document (Mandatory).

The highest scoring proposal will get the assignment.
How to submit your proposal
Please submit your proposal to<EMAIL_ADDRESS>before the deadline stated above. Reference "Project Creation, Listing, Editing, Membership" in the subject line.
Acceptance Criteria
To be paid for this opportunity we need you to:

Use the BC Developers' Exchange bootstrap theme to deliver a process for government program users to create, publish and list program projects.
Work with the BC DevEx UX team to refine their design.
Create wizard that takes government users through the steps to completion.
Develop a process that enables the simultaneous creation of a GitHub repo project.
Assume that the user does not have a Github account.
Assume that an existing repo does not exist.
Project will be displayed on the DevEx web app.
Project Listing to be added to the program profile page.
Add project button on the program profile page.
Listing alphabetically with no filtering or pagination.
project creator becomes owner.
Project membership to work in the same fashion as program membership.
Project profile also works in the same fashion as the program profile page.

How payment works
Questions, Comments, Suggestions?
Please post your questions in the comments section below.

Is there an existing data model available to follow, or did I just commit the sin of RTFM?

Hi dewolfe001, there will be an initial model there once I get round to it, however we will want to review this for completeness once work begins, and I expect it will change of course :)  The sections for the new work are simply stubbed out at the moment.

Would putting forward a proposal to integrate an existing FOSS MongoDB / Node.js CMS into this role be something considered as a valid approach; or would it be considered to be an undesirable approach?

Hi dewolfe001. I think I answered this in another post, sorry.  Not this time is the answer, this approach has been examined already.

thanks: yes, you did.

Hello Subscribers: Just a friendly reminder to submit your proposal before 5:00PM today. Thanks for your interest!

Thank you to everone who submitted a proposal. We are assigning this issue to @dewolfe001.

@ccoldwell @paulroberts68 - Fantatastic news! with these three current issues having to inter-operate, can we talk about next steps, meeting and collaboration?

Hi Shawn,
We want to set up a meeting for tomorrow morning at 9:30 to kick things off
and talk about everything.  We are going to meet at my office:
#302 - 1221 Broad Street
The name on the office is Three Trees, Cisco and a couple of others.
Looking forward to getting the ball rolling!
Chris
On Thu, Nov 3, 2016 at 9:47 AM, Shawn DeWolfe<EMAIL_ADDRESS>wrote:

@ccoldwell https://github.com/ccoldwell @paulroberts68
https://github.com/paulroberts68 - Fantatastic news! with these three
current issues having to inter-operate, can we talk about next steps,
meeting and collaboration?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BCDevExchange/design/issues/5#issuecomment-258201706,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEtDDa_GBkXTypGMPOkff6Xxq4U3iQgks5q6hAqgaJpZM4KilES
.

If this all goes offline and away from github/open communities, how will those not currently 'assigned' influence the tech outcome or understand the architectural choices ultimately adopted?
I realize its early days on this new procurement model, but was hoping the "Province on GitHub" model would be closer to the opensource community-driven approach rather than private contracting approach... can I suggest this be moved to a hangout, slack channel, or similar for wider participation?
I'm constrained by my current client load and the shortness of the deadlines here, but I would like to work on this site in the future and have an interest in how this procurement model works out vs the BC Bid approach -- as such, broader community participation in the project governance and technical choices would be a 'really good thing' imo.

Hi Chris,
I can't attend in person tomorrow, as I have a previous commitment at UVic. I could attend via Skype or do a meeting at any time on Monday.

Shawn

Hmm,
I think its pretty important that we do this in person.  How about Monday
morning? The earliest I can meet is 9:30 that day.  Can you make that work?
Chris
On Thu, Nov 3, 2016 at 12:10 PM, Shawn DeWolfe<EMAIL_ADDRESS>wrote:

Hi Chris,
I can't attend in person tomorrow, as I have a previous commitment at
UVic. I could attend via Skype or do a meeting at any time on Monday.

Shawn

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BCDevExchange/design/issues/5#issuecomment-258244520,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEtDEc2Meqb7rI5-4dS1tWngY7JWeR8ks5q6jG3gaJpZM4KilES
.

So are we still on with me for tommorrow?  Thanks

Hi Wayne,
Yes, let's you and I still make a start tomorrow.  We will roll Shawn into
things on Monday, but there's no sense in holding you back unnecessarily.
Chris
On Thu, Nov 3, 2016 at 2:38 PM, Wayne Vo<EMAIL_ADDRESS>wrote:

So are we still on with me for tommorrow? Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BCDevExchange/design/issues/5#issuecomment-258282003,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEtDM7bW2_eTeDug4ZfEbpoaSdse8zAks5q6lRSgaJpZM4KilES
.

I agree that doing this in person is what we need to do.
If we can do Monday, I would appreciate that. I can do 9:30. I have an 11AM appointment, but if we need to run this past 11AM (I feel we do), I can push that appointment to another date / time.

@StormTide  we understand your concerns and we're working hard to engage and collaborate openly and, with the help of our community, get better at doing it. A key consideration in the design of the new Code With Us process was to keep development and design in the open. We have no intention of taking it all offline (please see https://github.com/BCDevExchange/devex/issues/5#issuecomment-258507576). Our interest in developers getting together is not meant to signal that work will done out of sight; simply direct interaction is sometimes preferable. @lmullane (our UX Researcher) is designing creative ways to include community involvement in every aspect of our work. He’s very keen to receive advice and help as moving into an open development model is very new for us.

@paulroberts68 'assign and go offline' seems to me to be the standard BC Bid process... community is still trying to figure out how this is going to work too. So if thats the intent, fair enough, but I guess I was hoping for a more F/LOSS'y approach.

The first cut of this was committed yesterday. There are a few pieces I need to work through to close off this issue and before it could be considered ready to any degree

the projects/:ProjectID path that should display the project is not working
the projects:/ProjectID/edit path is also not working. It should bring up the edit UI. It is failing because the data that is requested comes back as an array when an object is expected.
the Programs dropdown on the projects edit / create forms is not working. I am missing a piece to connect to the Programs model, find the _id and Name of the Program and use that as a reference to populate a select / option in the form.
I need to pull the reference to the user and the user's secret key out of the code and put that into a config that can be used by the Projects module.

For anyone wanting to follow this issue, the commits are happening over in this repo: https://github.com/BCDevExchange/devex/commits/master

This issue is now closed


[Feature Request] Enabling phase_took_enabled as a Search request parameter

Is your feature request related to a problem? Please describe
I came across this setting phase_took_enabled which I liked very much, but this is a cluster level setting and once it is added the setting is enabled for all searches, which is not my usecase. My usecase is I want to get this breakdown of phase level took time for few of my expensive queries hence I want to control this setting via URL parameter of search api.
Similar thing is available for default_allow_partial_results where either I can set this as a setting or pass this as a URL parameter in search.
Ref docs: https://opensearch.org/docs/latest/install-and-configure/configuring-opensearch/search-settings/
Describe the solution you'd like
Make the setting phase_took_enabled also a URL parameter for Search api.
Related component
Search
Describe alternatives you've considered
NA
Additional context
Ref docs: https://opensearch.org/docs/latest/install-and-configure/configuring-opensearch/search-settings/

Good news! It is available as a search request parameter.
You just need to set &phase_took=true in your URL:
https://github.com/opensearch-project/OpenSearch/blob/ca3796e8dde8830f21ec090cebe287d862abb264/server/src/main/java/org/opensearch/rest/action/search/RestSearchAction.java#L187-L191
It looks like we did remember to include it in the list of URL parameters in the search API documentation: https://opensearch.org/docs/latest/api-reference/search/#url-parameters

Thanks this was helpful. Closing this GH issue.


Display interfaces in asc order

[ ] No redmine Issue =[
[ ] Ready for review

We've been down this road a few times in the past. The order of the interfaces is significant as they are. They cannot be alphabetized in these lists.
When it comes to things like HA it is critical that the order match exactly, and this will only compound confusion.

Hi @jim-p,
We have a large infrastructure, with many, many VLANs. The order that was created represents nothing to us. But ok, thank you for the explanation.


⚠ Bofrost.de has degraded performance

In a3c27c1, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1293 ms

Resolved: Bofrost.de performance has improved in 686b3b4 after 15 minutes.


add bell state kata

Adding Bell State Kata
This resolves part of #1185 .

The CI build failures are caused by case mismatch in solution.md files in other tasks of Superposition kata. When we did refactoring of infrastructure for https://github.com/microsoft/qsharp/pull/1174/, we renamed Solution.qs files to start with capital S, but didn't update the references to the solutions in solution.md, they're still spelled with lower case. (And, since the Superposition kata was not on the list of published katas, the tests didn't catch that.)
Could you please update this as part of your PR? Will be easier than opening a new one and then merging things

The CI build failures are caused by case mismatch in solution.md files in other tasks of Superposition kata. When we did refactoring of infrastructure for #1174, we renamed Solution.qs files to start with capital S, but didn't update the references to the solutions in solution.md, they're still spelled with lower case. (And, since the Superposition kata was not on the list of published katas, the tests didn't catch that.)
Could you please update this as part of your PR? Will be easier than opening a new one and then merging things

Sure thing!


[zephyr] Added support for few GeneralDiagnostics attributes

Problem
Platform doesn't support a few optional attributes and also
mandatory BootReason event.
Change overview

Added support for attributes: UpTime, TotalOperationalHours
Added support for event: BootReason

Testing
Performed manual testing with nrfconnect lock example and Python CHIP controller

Adding fast-track label, as change is only about adding support for few attributes for a specific platform. No generic changes done.


take advantage of relx's --dev-mode

this is amazing

Nice! Last time (quite some months ago) I tried to use this, it didn't work and I didn't have time to track it down.

@seth let me know if works for you. oc_erchef is doing great with it.

:+1:, a whole lot more sane (now that relx is apparently up to the task...)


Linked frames

fix #22 , support linked frame in merged mode, tested in 1.4 and 1.5.1. feedback is welcome.

error.


Support for download from Azure

Support to download the CD and pre-requisites from Azure File or Blob storage.

@beutepa are we keeping the [object]Context parameter?

Let's keep it for now.
The root commands Get-AzureStorageBlobContent/Get-AzureStorageFileContent need this Context. They don't support StorageAccountName/StorageAccountKey parameters.
I will investigate if there is a 'ProfileName' in Azure like in AWS S3 to align.
After the investigation, we can decide at which level we do the 'conversion' to the Context.


Allow abstentions

It should be possible to abstain from a vote. An abstention does not affect the vote positively or negatively. But it does end up being recorded...

You would abstain by using this emoji: :zipper_mouth_face:


Add LICENSE

What is the opensource license for this project? Is it MIT?

Yes.

MIT license added. :+1:


[AutoPR containerregistry/resource-manager] [Do not merge] Swagger and Examples for the new Scoped tokens feature of Azure Container registry

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5543

(message created by the CI based on PR content)

Codecov Report

Merging #4772 into restapi_auto_containerregistry/resource-manager will decrease coverage by 0.01%.
The diff coverage is 13.79%.

@@                                 Coverage Diff                                 @@
##           restapi_auto_containerregistry/resource-manager    #4772      +/-   ##
===================================================================================
- Coverage                                            49.51%    49.5%   -0.02%
===================================================================================
  Files                                                 6333     6332       -1
  Lines                                               177235   177196      -39
===================================================================================
- Hits                                                 87763    87714      -49
- Misses                                               89472    89482      +10

Impacted Files
Coverage Δ

...erregistry/container_registry_management_client.py
62.82% <13.79%> (-11.2%)
:arrow_down:

...re/graphrbac/operations/applications_operations.py
35.98% <0%> (-4.72%)
:arrow_down:

azure-graphrbac/azure/graphrbac/models/__init__.py
56.25% <0%> (-2.34%)
:arrow_down:

...phrbac/models/application_create_parameters_py3.py
100% <0%> (ø)
:arrow_up:

...rbac/models/service_principal_object_result_py3.py

.../mgmt/containerservice/container_service_client.py
60.52% <0%> (+1.96%)
: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 62ebd23...b4518e4. Read the comment docs.


Initialize an empty rails app

#20
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[x] I included unit tests that cover my changes
[x] I added/updated the documentation about my changes
[x] I made my own code-review before requesting one

Description of the changes
I just initialized an empty rails application, trying to follow the node template as an root.
Technical highlight/advice
I want to highlight that for unit tests I opt in for rspec instead of the default minitest. The reason I made that change is because is more popular in web applications.

@shiipou When you have the time you can please see if this PR is ok?

@shiipou I will do that, sorry for the late response, this week is being kind busy, I'll work on it!

@shiipou I will do that, sorry for the late response, this week is being kind busy, I'll work on it!

No problems, take your time.
Thank you for this contribution that help us a lot.

Reworked the application base to be more minimal, only loading what will be used now, hope this do the trick :)


Always go build in CI generate steps

With the recent cgo changes, bugs can sneak through if we don't make sure to go build all the permutations

CI passed with the dummy commit to force all the steps - https://github.com/ollama/ollama/actions/runs/9813737088


Meta tile and meta descriptions missing in category object

Steps to reproduce the issue

Add meta_title and meta_description to local.json in vue-storefront-api:

  "entities": {
    "category": {
      "includeFields": [
        "meta_description",
        "meta_title",
        "children_data",
        "id",
        "children_count",
        "sku",
        "name",
        "is_active",
        "parent_id",
        "level",
        "url_key",
        "description",
        "image"
      ]
    },

Add meta_title and meta_description to local.json in vue-storefront:

    "category": {
      "includeFields": [ "meta_description", "meta_title", "children_data", "id", "children_count", "sku", "name", "is_active", "parent_id", "level", "url_key", "product_count", "p
    },

Rebuild vue-storefront and vue-storefront-api.
Reindex via mage2vuestorefront:

node --harmony cli.js fullreindex
node --harmony cli.js categories --removeNonExistent=true --extendedCategories=true

Clear browser data

Current behavior
Visit a frontend category and check the category computed property. The meta title and description are missing.

Expected behavior
After adding the (category) meta_title and meta_description these attributes should be available in the category object
Can you handle fixing this bug by yourself?

No, i'm also not sure that this is a bug. Maybe i'm doing something wrong?
Environment details

Vue Storefront (API): 1.6
Mage2vuestorerfront: master branch.
Browser: Chrome Mac v70
OS: Ubuntu 16
Node: v8.14.0

Additional information
Related issue: https://github.com/DivanteLtd/vue-storefront/issues/1383

Currently the Magento meta_* fields are not used (we do generate meta from description and title fields) in Category.js (either in Product.js) but I belive this can Ben easily fixed - @patzick can you take a look at it?

@pkarw i've checked with Kibana, and I've noticed that some of the categories indeed have meta_title and meta_description fields now, and they are also available in the category object. However, this isn't the case for all the categories, in some categories the data is still missing. Maybe I will have to completely destroy and rebuild the elastic indexes? What would be the best way to do this?


support vulgar fractions

Supports strings like "1½ cups flour", "¼ dl milk".

great fix :+1: will mention my previous bad fix in the other PR


[occm] use yaml list instead of string to pass extra controller arguments

Action required: the controllerExtraArgs field in the occm helm chart now takes a list of string arguments. Users should remove the |- string quote operator from controllerExtraArgs.

fixes #1637

@ouroboros8 Thanks for the PR. Few things to take care
cla/linuxfoundation — ouroboros8 needs CLA  - CLA needs to be signed first
hashtag(#) mentions are not allowed in commit messages.

@ouroboros8 And also, please follow the PR template as defined here https://github.com/kubernetes/cloud-provider-openstack/blob/master/.github/PULL_REQUEST_TEMPLATE.md . This would come automatically before you submit Pull Request. Edit the required fields and submit the PR.

/ok-to-test

Thanks @ramineni, I have signed the CLA and will take care of your other feedback shortly :)


F# options missing from Fonts and Colors

In the latest nightly release (<IP_ADDRESS>41402), the F# options are missing from Fonts and Colors. It looks like the mutable variables are still highlighted as before, but functions are not, see:

Not sure when this was broken, or what could have caused it.

@saul Interesting.  I've got them in <IP_ADDRESS>41701 (latest today).

The settings are there on current master.

Closing for now, as I've not seen this missing in nightlies either.  Perhaps it was a bad build.


Return value of Spatie\TwitterLabs\FilteredStream\FilteredStream::setRules() must be an instance of Spatie\TwitterLabs\FilteredStream\Responses\Rules\AddRulesResponse, null returned

Hi Freek and Team Spatie,
First of all, thank you for all you do for the Laravel community!
I tried to follow the docs to install this and I get the following when running the listen command.
My dashboard.php config
'tiles' => [
        'twitter' => [
            'configurations' => [
                'default' => [
                    'access_token' => env('TWITTER_ACCESS_TOKEN'),
                    'access_token_secret' => env('TWITTER_ACCESS_TOKEN_SECRET'),
                    'consumer_key' => env('TWITTER_CONSUMER_KEY'),
                    'consumer_secret' => env('TWITTER_CONSUMER_SECRET'),
                    'listen_for' => [
                        '@_CPotter',
                    ],
                ],
            ],
            'refresh_interval_in_seconds' => 5,
        ],
    ],

I run
 php artisan dashboard:listen-twitter-mentions
Get the following:
vagrant@homestead:~/dashboard$ php artisan dashboard:listen-twitter-mentions
Listening for mentions for configuration named `default`...

   TypeError

  Return value of Spatie\TwitterLabs\FilteredStream\FilteredStream::setRules() must be an instance of Spatie\TwitterLabs\FilteredStream\Responses\Rules\AddRulesResponse, null returned

  at vendor/spatie/twitter-labs/src/FilteredStream/FilteredStream.php:114
    110|     }
    111|
    112|     public function setRules(Rule ...$rules): AddRulesResponse
    113|     {
  > 114|         return $this->await($this->asyncSetRules(...$rules));
    115|     }
    116|
    117|     public function asyncGetRules(): PromiseInterface
    118|     {

      +15 vendor frames
  16  artisan:37
      Illuminate\Foundation\Console\Kernel::handle()

Any suggestions ? I tried to clear config cache just in case my keys were not present but I have tested with tinker.

Hi! This is because your Twitter account is either not a developer account, not enrolled in Twitter Labs or the Twitter application isn't connected to Twitter labs.
Have a look at https://developer.twitter.com/en/account/labs and make sure your application is registered for the Filtered Stream preview like this:

Sorry this wasn't clear, we'll include these steps in the docs soon :)


feat: Test scene for animation reset

Test scene for visibilty model: https://github.com/decentraland/unity-client/issues/233
What?
Scene resetting animations
...
Why?
To simplify testing process
...

Waiting for https://github.com/decentraland/unity-client/pull/306


APIv2: How to compile proto file that imports descriptor.proto?

I’m playing with API-v2. When compiling a proto file that imports the descriptor.proto file, I get a compile error on this line (in the generated code):
// This is a compile-time assertion that a sufficiently up-to-date version
// of the legacy proto package is being used.
const _ = proto.ProtoPackageIsVersion4

The command I used to compile (gives no errors while generating):
protoc --go_out=Mgoogle/protobuf/descriptor.proto=github.com/golang/protobuf/protoc-gen-go/descriptor:. gorums.proto

PS: The descriptor.proto file that I’m mapping to here is in the v1.3.3 folder. However, the new API-v2 does not seem to contain this file…
The file I’m trying to compile is gorums.proto:
syntax = "proto3";

package gorums;

import "google/protobuf/descriptor.proto";

option go_package = "github.com/relab/gorums";

extend google.protobuf.MethodOptions {
  bool qc = 50000;
  bool correctable = 50001;
  bool correctable_stream = 50002;
  bool multicast = 50003;
  bool qc_future = 50004;
  bool qf_with_req = 50005;
  bool per_node_arg = 50006;

  string custom_return_type = 51000;
}

Any pointers would be much appreciated.

Thanks for trying out APIv2!
The error is because the generated .pb.go is using a version of the github.com/golang/protobuf/proto package that's too old. Unfortunately, the version you need isn't released yet.  Assuming you're using modules, you can add the necessary prerelease version to go.mod with:
go get -u github.com/golang/protobuf/proto@api-v1

Just to clarify; did you mean v1 or v2 in the go get command? (Not at computer; will test tomorrow...)

I mean @api-v1--this specifies a branch that includes a version of APIv1 that uses APIv2 under the hood.

Promoting this to blocks-v2, since it raises an interesting question:
Should google.golang.org/protobuf have a dependency on github.com/golang/protobuf, at least in the short term, to ensure that anyone using APIv2 gets an appropriately new version of APIv1?
We've wanted to avoid that dependency, but perhaps it's worth it at least for a transitional period to avoid confusion.

Just wanted to report back that the go get command above did solve the problem for me.
About the short term dependency issue: It could save people a fair amount of initial debug time for the transitional period. Obviously, once the dependency is removed, people may run into problems then. But maybe it would smooth out the pain.
Another alternative is of course to write up a migration/troubleshooting guide, and place it visibly on the github page and elsewhere... Maybe also write a blog post at golang.org about the release. Let me know if you plan to do so; I'd be happy to review it from an external perspective, prior to publication.

Okay, so the first part worked, but now I want to also build a proto with a gRPC service, and I run into a weird problem with Go modules (probably VSCode) updating the go.mod file.
Here is the file I'm compiling:
syntax = "proto3";
package goproto.protoc.gorums;
option go_package = "github.com/relab/gorums/cmd/protoc-gen-gorums/testdata/basic";

service Storage {
  rpc Read(ReadRequest) returns (ReadResponse) {}
}

message ReadRequest {}
message ReadResponse {}

I have this in ...gorums/go.mod (project root):
module github.com/relab/gorums

go 1.13

require (
	github.com/golang/protobuf v1.2.1-0.20200212180908-0fd14f96108a
	google.golang.org/protobuf v0.0.0-20200213061406-709e7c8474b1
)

And this in .../gorums/cmd/protoc-gen-gorums/testdata/go.mod:
module github.com/relab/gorums/cmd/protoc-gen-gorums/testdata

go 1.13

require (
	google.golang.org/grpc v1.28.0-pre.0.20200205181625-597699c0ef29
	github.com/golang/protobuf v1.2.1-0.20200212180908-0fd14f96108a
	google.golang.org/protobuf v0.0.0-20200213061406-709e7c8474b1
)

When I run:
protoc --go_out=paths=source_relative:. --go-grpc_out=paths=source_relative:. basic/basic.proto

It generates the expected .pb.go and _grpc.pb.go files. Soon thereafter, I assume that VSCode detects the two files, and updates the go.mod file in the project root as follows:
module github.com/relab/gorums

go 1.13

require (
	github.com/golang/protobuf v1.3.2
	github.com/google/go-cmp v0.3.1 // indirect
	google.golang.org/grpc v1.27.1 // indirect
	google.golang.org/protobuf v0.0.0-20200213061406-709e7c8474b1
)

I should point out that I have no other .go files in the entire project. I also tried to close down VSCode and run go build in the directory with the generated files. In this case, the go.mod file in testdata folder gets updated in a similar way to the above (though some different versions; I did recompile the v2 protoc-gen-go and protoc-gen-grpc between to ensure I was actually using v2, which may or may not be related to the different versions):
module github.com/relab/gorums/cmd/protoc-gen-gorums/testdata

go 1.13

require (
	github.com/golang/protobuf v1.3.3
	github.com/google/go-cmp v0.3.1 // indirect
	google.golang.org/grpc v1.28.0-pre.0.20200205181625-597699c0ef29
	google.golang.org/protobuf v0.0.0-20200213061406-709e7c8474b1
)

And resulting in the following compiler error:
# github.com/relab/gorums/cmd/protoc-gen-gorums/testdata/basic
./basic.pb.go:26:11: undefined: "github.com/golang/protobuf/proto".ProtoPackageIsVersion4

I would very much appreciate any input towards resolving these issues. Thanks!
PS: Maybe this is related to my use of two go.mod files and the special directory testdata, but I'm following the template from the protobuf@api-v2 repo. Maybe that's a bad idea?

The problem turns out to be that the pseudoversion for api-v1 sorted earlier than the version depended on by the gRPC module, leaving you in a state where it's impossible to satisfy all the dependencies at the same time.
I've just tagged<EMAIL_ADDRESS>which contains the proper revision at an appropriately high (but unreleased) version. I believe everything will work properly now if you go get -u<EMAIL_ADDRESS>Sorry for the trouble, and thank you for trying this out!

And I just submitted a change to add a dependency from APIv2 to APIv1, which means you should no longer need to manually update github.com/golang/protobuf: Any program using APIv2 will also automatically use a sufficiently new version of APIv1.

Thanks a lot! It now seems to build correctly on command line... I only tried to run go build -a -v -x without any "users" of the generated code yet, so no binary is produced in the local folder, but the last line of the output is cp $WORK/b001/_pkg_.a /Users.../Library/Caches/.... So I assume this is ok.
Below I provide a description of behaviors experienced...
For some weird reason VSCode / gopls is unable to detect the types ReadRequest and ReadResponse (the message types) from the grpc file, even though both files are in the same folder, and have the same package name. I get:
undeclared name: ReadRequest
undeclared name: ReadResponse
...

(maybe this is a VSCode/gopls issue, and I should file an issue there, but let me know if this is a well-known problem... maybe this is related to the treatment of the testdata folder? Will try more tomorrow.)
I've tried to restart the language server and reload the VSCode window to no avail. And after restarting VSCode entirely, I get:
Package github.com/golang/protobuf/proto is deprecated: Use the "google.golang.org/protobuf/proto" package instead.  (SA1019)

(seems this last problem may be the staticcheck linter. I changed to golint and the problem seems to have gone away.)
Thanks for all your help so far! Much appreciated.

I updated the comment above to reflect my latest findings. Seems that following the template of golang/protobuf with the testdata folder for building a new protobuf generator is unwise, at least if you want to have tests inside testdata that can compile in VSCode (or with other go tooling).

Thanks again for trying out APIv2!


MoE example for COPA/score fails in the moe branch

🐛 Bug
Trying to run the COPA example for a dense LM on this page (https://github.com/pytorch/fairseq/tree/main/examples/moe_lm) and it fails with the following stack trace.
Also, there are no instructions to load an MoE checkpoint (en_moe_lm_52b.tar.gz) for this COPA example. Is that possible? I can create a separate issue/question for it too.
To Reproduce
Steps to reproduce the behavior (always include the command you ran):

Save the COPA example in a file named copa.py
Run python copa.py

Code sample
Contents of my copa.py
from fairseq.models.transformer_lm import TransformerLanguageModel
model_dir = '/home/amawa/downloads/en_dense_lm_125m'
lm = TransformerLanguageModel.from_pretrained(model_dir, bpe='gpt2')
lm = lm.eval();  # disable dropout
lm = lm.half();  # use FP16 for evaluation
lm = lm.cuda();  # move to GPU

def get_logprobs(prompt):
    import re
    prompt = re.sub('\n+' , '\n', prompt)  # collapse repeated newlines, which indicate separate documents
    return lm.score(prompt, replace_newlines_with_eos=True)['positional_scores']

# Zero-shot evaluation for the Choice of Plausible Alternatives (COPA) task.
# A return value of 1 indicates that the first alternative is more plausible,
# while 2 indicates that the second alternative is more plausible.
def COPA_eval(prompt, alternative1, alternative2):
    lprob1 = get_logprobs(prompt + "\n" + alternative1).sum()
    lprob2 = get_logprobs(prompt + "\n" + alternative2).sum()
    return 1 if lprob1 > lprob2 else 2

COPA_eval("The man broke his toe. What was the CAUSE of this?", "He got a hole in his sock.", "He dropped a hammer on his foot.")
COPA_eval("I tipped the bottle. What happened as a RESULT?", "The liquid in the bottle froze.", "The liquid in the bottle poured out.")
COPA_eval("I knocked on my neighbor's door. What happened as a RESULT?", "My neighbor invited me in.", "My neighbor left his house.")

Stack trace:
Traceback (most recent call last):
  File "copa.py", line 21, in <module>
    COPA_eval("The man broke his toe. What was the CAUSE of this?", "He got a hole in his sock.", "He dropped a hammer on his foot.")
  File "copa.py", line 17, in COPA_eval
    lprob1 = get_logprobs(prompt + "\n" + alternative1).sum()
  File "copa.py", line 11, in get_logprobs
    return lm.score(prompt, replace_newlines_with_eos=True)['positional_scores']
  File "/home/amawa/fairseq-moe/fairseq/hub_utils.py", line 170, in score
    return self.score([sentences], **kwargs)[0]
  File "/home/amawa/fairseq-moe/fairseq/hub_utils.py", line 175, in score
    for hypos in self.generate(
  File "/home/amawa/fairseq-moe/fairseq/hub_utils.py", line 201, in generate
    generator = self.task.build_generator(self.models, gen_args)
  File "/home/amawa/fairseq-moe/fairseq/tasks/fairseq_task.py", line 356, in build_generator
    return SequenceScorer(
TypeError: __init__() got an unexpected keyword argument 'compute_vocab_dist'

Expected behavior
Expected output of the three COPA_eval lines are 2, 2, and 1 getting printed. This is the case when you run it for the main branch of fairseq but not for moe branch.

Environment

fairseq Version (moe):
PyTorch Version (1.10)
OS (e.g., Linux):
How you installed fairseq (source):
Build command you used (pip install -e . ):
Python version: 38
CUDA/cuDNN version: 11.3
GPU models and configuration: dense_lm_125m
Any other relevant information:

Additional context

I ran into the same problem and fixed it (?) by commenting out this line: https://github.com/facebookresearch/fairseq/blob/ebea0072062e2e8f4563644f27546df355357f5e/fairseq/tasks/fairseq_task.py#L359


Remove selection

Uncaught TypeError: Cannot read property 'call' of undefined

Ok, closing for now, this is due to the new dude-graph-toolbar


Open reveal View controller when click a button

Hello
I want to open the main view controller when click on a button in a ViewController.
When my application starts I open a ViewController with a button.
I want to click on that button and open the main view with the menu.

could you perhaps tell how to do that in swift ?
tried this, but doesnt work :
@IBAction func sideBarShow(sender: AnyObject) {
    let revealController = SWRevealViewController()

    revealController.rightRevealToggleAnimated(true)

}

I have a pangesturerecognizer added like this:
self.view.addGestureRecognizer(self.revealViewController().panGestureRecognizer())
and it works perfectly


Support webauthn MFA factor

Looks like webauthn (yubikey 5c nano in this case) isn't supported. It would be great if it was.
DEBU[0022] Step: 2
INFO[0022] Requesting MFA. Please complete two-factor authentication with your second device
DEBU[0022] 00uMrQE7-7jbCWJZz61wCH5d4PKf7V8YVp2IgMb0S4
DEBU[0022] Failed to validate credentials: factor webauthn not supported
Failed to validate credentials```

I don't have access to a webauthn device here, so don't look at me :D Happy to accept a PR.

How about if I buy you one? ;)

How about if I buy you one? ;)

Haha, yeah, I can't really do that; I answer only to the crystalline entity that runs Segment.

@nicgrayson If you're running on a Mac you have a WebAuthn device in the form of TouchID in Chrome

https://www.yubico.com/product/security-key-by-yubico

Well I actually bought a Yubikey 5c myself. I believe this would give us TouchID support as well, which would be super cool.
That said, there are no plans to implement this as yet. Happy to accept a PR though, and I can even test it now

I believe this would give us TouchID support as well

Right now I'm pretty sure only Chrome can use TouchID for WebAuthn, so you'd need to open a browser for the auth request.

Even if it's not being tackled right now, I think it makes sense to keep this issue open

Yeah, I'm happy to buy you one as well!  We're looking to shift from YubiKey + Seedfile to the U2F / WebAuthN implementation in Okta.  For us, this would help us to manage the growing list of YubiKeys we have.  It's getting difficult to manage seed files at scale.

@bennumo fair enough, but like I said, no plans to implement this by maintainers at present. PRs encouraged (after #236 is lifted)

I really don't like this definition of closed personally. I'm more in favor of a label indicating that an issue is low-priority but that PRs are welcome


[WIP]:feat(cloud_firestore): v1 rework

Cloud Firestore
Along with the below changes, the plugin has undergone a quality of life update to better support exceptions thrown. Any Firestore specific errors now return a FirebaseException, allowing you to directly access the code (e.g. permission-denied) and message.
Firestore:

BREAKING: Calling settings() now accepts a Settings instance.
DEPRECATED: Calling document() is deprecated in favor of doc().
DEPRECATED: Calling Firestore(app: app) is now deprecated. Use Firestore.instance or Firestore.instanceFor instead.
BREAKING: enablePersistence has now been removed, see settings() instead.
NEW: Add clearPersistence() support.
NEW: Add disableNetwork() support.
NEW: Add enableNetwork() support.
NEW: Add snapshotInSync() support.
NEW: Add terminate() support.
NEW: Add waitForPendingWrites() support.

CollectionReference:

BREAKING: Getting a collection parent document via parent() has been changed to a getter parent. @ehesp confirm
DEPRECATED: Calling document() is deprecated in favor of doc().

Query:

BREAKING: The internal query logic has been overhauled to better assert invalid queries locally.
NEW: Query methods are now chainable.
NEW: It is now possible to call same-point cursor based queries without throwing (e.g. calling endAt() and then endBefore() will replace the "end" cursor query with the endBefore).
NEW: Added support for limitToLast.

QuerySnapshot:

DEPRECATED: documents has been deprecated in favor of docs.
DEPRECATED: documentChanges has been deprecated in favor of docChanges.
NEW: docs now returns a List<QueryDocumentSnapshot> vs List<DocumentSnapshot>. This doesn't break existing functionality.

DocumentReference:

BREAKING: setData/set has been updated to accept an instance of SetOptions (see below, supports mergeFields).
BREAKING: Getting a document parent collection via parent() has been changed to a getter parent. @ehesp confirm
DEPRECATED: documentID has been deprecated in favor of id.
DEPRECATED: setData() has been deprecated in favor of set().
DEPRECATED: updateData() has been deprecated in favor of update().

DocumentSnapshot:

BREAKING: Getting a snapshots data via the data getter is now done via the data() method.
DEPRECATED: documentID has been deprecated in favor of id.
NEW: Added support for fetching nested snapshot data via the get() method. If no data exists at the given path, a StateError will be thrown.

WriteBatch:

BREAKING: setData/set now supports SetOptions to merge data/fields (previously this accepted a Map).
DEPRECATED: setData() has been deprecated in favor of set().
DEPRECATED: updateData() has been deprecated in favor of update().

Transaction: @ehesp: confirm method naming (e.g. set)

BREAKING: Transactions have been overhauled to address a number of issues:

Values returned from the transaction will now be returned from the Future. Previously, only primitive values (e.g. String) were supported. It is now possible to return values such as a DataSnapshot.
When manually throwing an exception, the context was lost and a generic PlatformException was thrown. You can now throw & catch on any exceptions.
The modify methods on a transaction (set, delete, update) were previously Futures. This has been updated to better reflect how transactions should behave - they are now synchronous and are executed atomically once the transaction handler block has finished executing.

FieldPath:

NEW: The constructor has now been made public to accept a List of String values. Previously field paths were accessible only via a dot-notated string path. This meant attempting to access a field in a document with a . in the name (e.g<EMAIL_ADDRESS>was impossible.

GetOptions: New class created to support how data is fetched from Firestore (server, cache, serverAndCache).
SetOptions: New class created to both merge and mergeFields when setting data on documents.
GeoPoint:

BREAKING: Add latitude and longitude validation when constructing a new GeoPoint instance.

Related Issues

[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/833
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/774
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/2081
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/211
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/857
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/1311
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/855
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/1223
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/2101
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/2291
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/185
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/2338
[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/issues/2293

Related PRs

[x] CLOSES: https://github.com/FirebaseExtended/flutterfire/pull/2110

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: Ehesp:white_check_mark: kirstywilliams:white_check_mark: Salakar:x: greghespYou have signed the CLA already but the status is still pending? Let us recheck it.

Closing in favour of #28 - had to manually merge with main repo master changes - see conflicts above.


[Hotfix] [Seatunnel-Web][seatunnel-server] Null Pointer Exception when try to create new user #5549

Purpose of this pull request
Null pointer exception is not thrown when creating a new user on seatunnel ui
Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs

@dijiekstra can you please review

@EricJoy2048 thanks for approving, all checks have passed can you also please merge this.


sequencer: rename sourceActionIndex to positionInTransaction

We should rename all instances of sourceActionIndex to positionInTransaction to be clearer about what this field means.
┆Issue Number: ENG-950

Is this still an open issue? I believe you closed it with #1718?

Is this still an open issue? I believe you closed it with #1718?

@SuperFluffy there are many instances of source_action_index that I think should still be changed to match, though I realize the issue said sourceActionIndex before. I've adjusted the description accordingly.


Adding GetMetricsData support

Add GetMetricData Cloudwatch API
Added MaxRetries for Resourcegrouptaggingapi, Autoscalingapi
Added Counter
resourceGroupTaggingAPICounter
autoScalingAPICounter,
cloudwatchGetMetricStatisticsAPICounter
cloudwatchGetMetricDataAPICounter

Removed code for GetmetricStatistics for discovery Job

Hi @Deepak1100. I deployed a new exporter with your code. But even with enableMetricData: true I see that it still uses getMetricStatistics API calls. I see that in ASW cost explorer, and in the metrics from the exporter itself:
yace_cloudwatch_getmetricdata_requests_total - all zeros,
yace_cloudwatch_getmetricstatistics_requests_total - still growing
Could you please advise how to switch to getMetricData properly?

Currently in hospital. Will be back soon and check that :) thanks for your contribution!

updated the PR

sample config:
discovery:
  jobs:
  - type: "elb"
    region: ap-south-1
    length: 60
    enableMetricData: true
    delay: 120
    searchTags:
      - Key: kubernetes.io/service-name
        Value: .*
    metrics:
      - name: HealthyHostCount
        statistics:
        - 'Minimum'
        period: 60
  - type: "alb"
    region: ap-south-1
    length: 60
    delay: 120
    enableMetricData: true
    searchTags:
      - Key: kubernetes.io/ingress-name
        Value: .*
    metrics:
      - name: RequestCount
        statistics:
        - 'Sum'
        period: 60
        nilToZero: true
      - name: Latency
        statistics:
        - 'Average'
        period: 60
        nilToZero: true
      - name: SpilloverCount
        statistics:
        - 'Sum'
        - 'Maximum'
        period: 60
        nilToZero: true

Verified with my current configuration, modified with the new settings, that these changes appear to be working as expected.

Hey, so back at work again :)
I love this PR. Would like to set it live. Is there any reason why we keep GetMetricStatistics. From my point of view we should not support both but only GetMetricData.
This would reduce complexity. What are your thoughts?

the option is given so that the existing config should work for all the people after the upgrade as it is.
they can change config for getmetricdata latter(we can deprecate it few versions later).
plus some people can use both the call in same config for different job
But I don't mind removing GetMetricStatistics  call now as well

@thomaspeitz  Removed code for GetmetricStatistics for discovery Job and
Added MaxRetries for Resourcegrouptaggingapi, Autoscalingapi

Thanks a lot <3 - Will release it this week.


sysutils/smart: add word break to ident in widget

Add word-breaking to the "Ident" column of the SMART Dashboard widget, to prevent long device names from causing the table to extend outside of the widget's div.
See the following example:

With this PR, this will get corrected to look as follows:

This style of word breaking is in line with the table in the "interface_list" widget. I had originally considered truncating the long "Ident" values with ellipsis; however, since there was already precedent in other widgets to use word-break, I wanted to stay consistent.

Merged, thanks!


[move-prover] Fix some type and other problems in boogie prelude, generated boogie

I fixed some problems that I ran into when trying to port the specs for libra_coin.move.
1. Address was defined redundantly as type int and as a constructor of values.

2. Fixing that exposed some other problems where int Addresses and Value Address were getting mixed up.  Fixed those (I think).

3. Fixed a few random problems:  $ExistsResource vs. $ResourceExists,
typo: $ResoureValue, calling $ResourceExists instead of $ResourceValue and
vice-versa.

4. Commented out apparently unused function InitTransaction, instead of fixing
a type error.

Motivation
Debugging Move Prover
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
cargo test
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra r=wrwg

:pushpin: Commit 3ce0861 has been approved by wrwg

@bors-libra r=wrwg

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #2924

:pushpin: Commit 3ce0861 has been approved by wrwg

:hourglass: Testing commit 3ce08617d079a044d6527773b04f3bfa74dc5465 with merge 600370ad8d36755f9373da5e26884b7c569b49f5...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wrwg
Pushing 600370ad8d36755f9373da5e26884b7c569b49f5 to master...


en-US/installation: update to run directly as a windows service

Updates gogits/gogs#630

Thanks!


Set a cursor to pointer when hovering over rows in CSmartTable

Hi,
I want to make it more clear to users that the rows in the table are clickable, so I need to make a cursor a pointer when hovering over rows.
How can I achieve that?
Thanks

@asirre you can add custom styles
tbody tr:hover { cursor: pointer }


Support for manually setting the target IP

We deploy our Traefik as a DaemonSet with direct hostPort binding. This means that there is no service that switchboard could retrieve the external IP from.
Since our external IP never changes, it may be a simple fix to allow the configuration of a fixed IP that switchboard uses for all targets.

Makes sense, will add this feature!


Update tools in preparation for load test timeouts.

Support multiline strings in template generation (as already supported in config generation, supporting roundtrip).
Ignore substitution of variables that are set by the controller at runtime (DRIVER_PORT, KILL_AFTER, POD_TIMEOUT).

See also https://github.com/grpc/grpc/pull/26516.

@wanlin31 will use this as a prerequisite for generating configurations that include timeouts. These configurations now include environment variables that are set by the controller, so should not be substituted when generating configs from templates. The templates are generated from examples by loadtest_template.py and now include multiline strings, so loadtest_template.py should format multiline strings in the same way as loadtest_config.py (to allow round trip between templates and examples).

Should we we include the update of loadtest_examples.sh here too?

Should we include the update of loadtest_examples.sh here too?

No, because no change is needed. loadtest_config.py now ignores the three environment variables that are used by the controller, so there is no need to do anything special in loadtest_examples.sh.


Real time dashboard updates from the API

WIP on #BM-242

There's a botched commit in there (6a6f1f4) that contains nothing.


chore: improve theme-management cookbook

What is it?

Docs

Description
Previous version of this set the theme to light if no theme was found in local storage. Updated version checks for preferences and sets accordingly.
That is: if nothing saved in local storage and prefers dark mode, local storage is set to dark etc....
Checklist

[x] Tested locally and seems to work just fine on a new qwik app.

@Shane-Donlon your change is pretty minimal but it the description you are mentioning more things.
Am I missing something?

@Shane-Donlon your change is pretty minimal but it the description you are mentioning more things. Am I missing something?

Thanks Giorgio. It is a really small change, I was trying to describe the current bug in the code and what the PR does.
Example before the PR .

Checks local storage.
if theme is true then adds theme
else adds light theme

Problem: This would automatically add light as the theme regardless of dark mode preferences.
The updated function does the below

Checks local storage.
if theme is true then adds theme
else checks for dark mode preferences
if prefers dark mode
Sets theme to dark
Else sets to light.

Thanks @Shane-Donlon :clap:


[beta] fix(eslint): messages severity

This is more a question than a proposal:
Originally, the react-hooks/exhaustive-deps eslint rule throws a warning in the React Docs implementations, and actually whatever you use this rule (vscode, cli, etc). However, currently, this rule is leading to a blocking state in the preview component, even showing a Lint error title there.
So, taking into account the educational purposes of the documentation should we treat all rules as an error or leave the exhaustive-deps rule as a warning and update the preview error?
Current

This PR

Let's just change the lint configuration to treat it as an error.


Integrate Sonarcloud for SB-RC

What is the feature request for?
Registry Core
What problem/inconvenience (if any) will this feature solve?
Sonarcloud should analyse each PR and help in reducing bugs,code smells, duplications and help in maintaining overall code

Let's assume this will be enabled for RC. What's the plan after that? Have we, as a team, decided on

Analysis we will configure in SonarCloud?
Actions we will take on the Analysis reports?
Benchmark we are aiming for?

Let's put down a plan for Automated PR checks, Code Quality, Test Coverage, and other related benchmarks. @tejash-jl @Pratikshakhandagale @chaitrali-r @suvarnakale @varadeth

@chaitrali-r Any updates on this task?


Using progressbar and echo

Is there a nice way of using both the progressbar and echoing output. Right now it looks like a mess, but i might be doing something wrong.
Ideally i would like the output to appear just above the progressbar when using a TTY. For non-tty environments (i.e. batch processing) i don't really need the progressbar at all.
I couldn't find anything regarding their interaction in the documentation.

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


ArcGIS Pro Patches get reinstalled: Not idempotent

The QFE ID returned by DSC code from "HKLM:\SOFTWARE\ESRI\ArcGISPro\Updates*"  does not match the one retreived from the msp file and so the Pro patches get reinstalled every time.
Snippets from my log file:
[[ArcGIS_InstallPatch]ProInstallPatch] Comparing QFE ID ArcGIS_Pro_241_170796 against ID QFE-24-PRO-Update241
[[ArcGIS_InstallPatch]ProInstallPatch] Comparing QFE ID ArcGIS_Pro_241_170796 against ID QFE-24-PRO-Update242
[[ArcGIS_InstallPatch]ProInstallPatch]  > Patch not installed - installing
....
[[ArcGIS_InstallPatch]ProInstallPatch] Comparing QFE ID ArcGIS_Pro_242_170797 against ID QFE-24-PRO-Update241
[[ArcGIS_InstallPatch]ProInstallPatch] Comparing QFE ID ArcGIS_Pro_242_170797 against ID QFE-24-PRO-Update242
[[ArcGIS_InstallPatch]ProInstallPatch]  > Patch not installed - installing

@pcsswamin
The PowerShell DSC ArcGIS Module will attempt to extract a QFE ID from the .msp file and check to see if it exists in the Windows registry, if it finds a match then the patch is skipped. If no match found, then the patch is installed. If the QFE ID isn't found in the .msp or there is a mismatch then this problem you are reporting occurs.
I recommend calling Esri Technical Support to report this issue.

@cameronkroeker - I disagree. This is not specific to my environment or organization. So shouldn't calling esri tech support directly be the last option here for an esri customer here? I am not trying to subvert due process here.
The esri automation team should be to reproduce this issue very quickly and route it internally to the team that does patch releases across the arcgis platform. In addition to Pro, This QFE ID mismatch issue also manifests in 10.6.1 ArcGIS Server patches (blank or empty QFE IDs) and so this is more systemic problem which one of you should be able to route through internally in Redlands much more efficiently than me calling tech support and explaining everything from scratch.

@pcsswamin
The challenge here is that the issue isn’t with the PowerShell DSC ArcGIS Module, but rather the patches (.msp). It would be ideal to be able to find all existing patches that are missing the QFE ID’s (or incorrect QFE ID’s), that have already been released and replace them with an msp that contains the correct QFE ID. However, this will still remain an issue for customers who have already downloaded and installed a patch that didn’t contain a QFE ID.
With that said we will follow up internally with the patch release team, but still encourage all customers to reach out to Esri Technical Support as this is the best avenue.  Additionally, GitHub isn’t really the proper channel or forum to report patches missing QFE ID’s, but rather these should be reported to Esri Technical Support. This way Esri Technical Support can reproduce the issue and submit an official bug report that contains a bug ID that will allow the status to be tracked via the Support site.
If you would like to discuss further, please feel free to contact me directly via email<EMAIL_ADDRESS>Thanks,
Cameron K.

@cameronkroeker
I am willing to start a tech support case to start tackling this. Taking it all the way to register a bug requires considerable investment of time and that is what I was balking at. Happy to take this offline. I will drop you an email on this soon.
Thanks


INSTALL_PATH was dropped from Makefile in 075a35a6d390167b77b687e067dd0ba593e7f624

What steps will reproduce the problem?
1. Make leveldb on OS X
2. Note lack of absolute install path in `-install_name`
3. Attempt to build against leveldb dylib on OS X and fail

What is the expected output? What do you see instead?

g++  -dynamiclib -install_name libleveldb.dylib.1

instead of

g++  -dynamiclib -install_name /path/to/current/directory/libleveldb.dylib.1

What version of the product are you using? On what operating system?

Anything after SHA 075a35a6d390167b77b687e067dd0ba593e7f624

Please provide any additional information below.

It looks like when

$(SHARED3):
	$(CXX) $(LDFLAGS) $(PLATFORM_SHARED_LDFLAGS)$(SHARED2) $(CXXFLAGS) $(PLATFORM_SHARED_CFLAGS) $(SOURCES) -o $(SHARED3)

was made, the $(INSTALL_PATH) param was dropped completely.

The attached patch corrects this.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 21 Aug 2012 at 5:44
Attachments:

install_path.patch

Issue 112 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 22 Aug 2012 at 6:08

Fixed in 1.7.0.

Original comment by<EMAIL_ADDRESS>on 16 Oct 2012 at 11:29

Changed state: Fixed


.Slug is empty if not present in front matter, can it be inferred from the name of the file ?

I'm trying to create a list page that includes the pages inside it, I'd like to use something like .Slug to generate the #anchor tags. But .Slug is blank if not present in front matter. Do I have to set a slug in the front matter, or can it default to the name of the file the content was loaded from ? Is there an alternative way to do this ?

{{ with .Slug }}{{ . }}{{ else }}{{ .BaseFileName }}{{ end }}

It might be worth to read the (little bit too complex) document about cross-references:
http://gohugo.io/extras/crossreferences/
By default, the Blackfriday Markdown engine creates some crazy #anchor tags that is a combi of a filename hash and the title and a sequence if duplicates.
You can turn this off by putting this into your config.toml (or similar in yaml):
[blackfriday]
  plainIdAnchors = true

Thanks for at, is .BaseFileName mentioned in the Variables page?

On 12 May 2015, at 22:05, Bjørn Erik Pedersen<EMAIL_ADDRESS>wrote:
{{ with .Slug }}{{ . }}{{ else }}{{ .BaseFileName }}{{ end }}
It might be worth to read the (little bit too complex) document about cross-references:
http://gohugo.io/extras/crossreferences/
By default, the Blackfriday Markdown engine creates some crazy #anchor tags that is a combi of a filename hash and the title and a sequence if duplicates.
You can turn this off by putting this into your config.toml (or similar in yaml):
[blackfriday]
plainIdAnchors = true
—
Reply to this email directly or view it on GitHub.

No, see #1132


Releasing 0.4.0

Coverage decreased (-5.3%) to 22.337% when pulling 493d4baa3fc1fc63eb27c9ac5ce90d57867507b9 on develop into ca0aae38e7b345867b324e42038a5620fbdbb829 on master.


LayoutLmv3 |  ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/2.4.0/metrics/seqeval/seqeval.py

I tried to fine-tune layoutLMv3 Chinese model on XFUND, and I input those in terminal
 python examples/run_xfund.py \
    --data_dir data --language zh \
    --do_train --do_eval \
    --model_name_or_path microsoft/layoutlmv3-base-chinese \
    --output_dir output

At first, it returns error AttributeError: module 'huggingface_hub' has no attribute 'hf_api'](https://github.com/huggingface/datasets/issues/4117#top)
Then I update some packages including datasets, huggingface_hub, transformers, etc.
it returns a new error now:
ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/2.4.0/metrics/seqeval/seqeval.py (ConnectionError(MaxRetryError("HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /huggingface/datasets/2.4.0/metrics/seqeval/seqeval.py (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fbc58fd64d0>: Failed to establish a new connection: [Errno 111] Connection refused'))")))
How to fix it?
P.S the cuda version in my linux server is 10.2 and here are some parts of my packages list:
| Name |                   Version  |                 Build  Channel |
_libgcc_mutex             0.1                        main    defaults
cudatoolkit               10.2.89             h713d32c_10    https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge
datasets                  2.3.2                    pypi_0    pypi
deepspeed                 0.5.7                    pypi_0    pypi
detectron2                0.6+cu102                pypi_0    pypi
evaluate                  0.1.2                    pypi_0    pypi
huggingface-hub           0.8.1                    pypi_0    pypi
layoutlmv3                0.1                       dev_0
python                    3.7.13               h12debd9_0    defaults
python-dateutil           2.8.2                    pypi_0    pypi
python_abi                3.7                     2_cp37m    https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge
pytorch                   1.10.0          py3.7_cuda10.2_cudnn7.6.5_0    pytorch
pytorch-mutex             1.0                        cuda    pytorch
response                  0.5.0                    pypi_0    pypi
responses                 0.18.0                   pypi_0    pypi
sentencepiece             0.1.97                   pypi_0    pypi
seqeval                   1.2.2                    pypi_0    pypi
tensorboard               2.7.0                    pypi_0    pypi
tensorboard-data-server   0.6.1                    pypi_0    pypi
tensorboard-plugin-wit    1.8.1                    pypi_0    pypi
torch                     1.9.0                    pypi_0    pypi
torchaudio                0.10.0               py37_cu102    pytorch
torchvision               0.11.0               py37_cu102    pytorch
transformers              4.12.5                   pypi_0    pypi

Have you solved the problem? It looks like a network issue. Can you reach the address on your server (e.g., through wget https://raw.githubusercontent.com/huggingface/datasets/2.4.0/metrics/seqeval/seqeval.py)?

I am closing this issue for now, since it is inactive.


Use /usr/bin/env for scripts

Proposed changes
/usr/bin/env bash is more flexible than /bin/bash when using different OSes.
Also found some files and script with wrong permissions.
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

I'm assuming the results markdown files get their permissions from the code that generates them, so we'll probably need to update that as well.

That was updated already with the recent changes to the NFR tests unless I missed something. It’s strange that the edge files still had wrong permissions but maybe it’s because they were created before and an append doesn’t change file permissions?


Chaining AutoMigrate seems to have an effect on soft-deletes

What version of Go are you using (go version)?
$ go version
go version go1.10.2 darwin/amd64
$ uname -a
Darwin violet.local 17.7.0 Darwin Kernel Version 17.7.0: Thu Jun 21 22:53:14 PDT 2018; root:xnu-4570.71.2~1/RELEASE_X86_64 x86_64

Which database and its version are you using?
$ sqlite3 version
SQLite version 3.19.3 2017-06-27 16:48:08

Please provide a complete runnable program to reproduce your issue. IMPORTANT
The following example creates a SQLite3 database, auto-migrates a simple table, inserts an entity and then tries to soft-delete that entity. This works fine when you do db.AutoMigrate(...) (1), but not when you use chaining like so: db = db.AutoMigrate(...) (2).
With (1) it generates:
UPDATE "companies" SET "deleted_at"='2018-07-18 15:39:03'  WHERE "companies"."deleted_at" IS NULL AND "companies"."id" = '1'
[1 rows affected or returned ]

With (2) it generates:
DELETE FROM "companies"  WHERE "companies"."id" = '1'

To reproduce, please comment the line marked #1 below and uncomment #2 instead:
package main

import (
	"log"
	"os"
	"time"

	"github.com/jinzhu/gorm"
	_ "github.com/mattn/go-sqlite3"
)

type Company struct {
	ID        int
	Name      string
	CreatedAt time.Time
	UpdatedAt time.Time
	DeletedAt *time.Time
}

func main() {
	log.SetFlags(log.LstdFlags | log.Lshortfile)

	db, err := gorm.Open("sqlite3", "/tmp/gorm.db")
	if err != nil {
		log.Fatal(err)
	}
	defer func() {
		db.Close()
		os.Remove("/tmp/gorm.db")
	}()

	db = db.Debug().LogMode(true)

	// #1 this works
	db.AutoMigrate(&Company{})
	// #2 this doesn't. See below
	// db = db.AutoMigrate(&Company{})

	// Create
	c := &Company{Name: "Google"}
	if err := db.Save(c).Error; err != nil {
		log.Fatal(err)
	}

	// This should soft-delete, but it doesn't do so
	// when you uncomment #2 (see above)
	if err := db.Delete(c).Error; err != nil {
		log.Fatal(err)
	}

	var count int
	if err := db.Unscoped().Model(&Company{}).Count(&count).Error; err != nil {
		log.Fatal(err)
	}
	if want, have := 1, count; want != have {
		log.Fatalf("expected %d companies, have %d", want, have)
	}
}

I encountered the same issue when chaning CreateTable
This works:
	db.
		DropTable(&Test{}).
		CreateTable(&Test{})
	db.
		Debug().
		Create(&t).
		Delete(&t) // soft deletes

This doesn't
  	db.
		DropTable(&Test{}).
		CreateTable(&Test{})
		Debug().
		Create(&t).
		Delete(&t) // deletes the record


cache comparison for mp3 files not working (re-uploads all mp3s each deploy)

When deploying a lektor site that hosts a podcast each mp3 file is re-uploaded each time:
$ AWS_PROFILE=lektor-deploy lektor deploy
Deploying to S3
  Build cache: /Users/krosaen/Library/Caches/Lektor/builds/47d71c9cd6c3bea470ee10d73758dde5
  Target: s3://brosaen.com
  adding podcasts/pistons/27/.__transQy8kFO
  adding podcasts/pistons/27/brosaen-episode-27.mp3
  adding podcasts/pistons/27/index.html
  updating index.html
  updating podcasts/pistons.xml
  updating podcasts/pistons/1/brosaen-episode-1.mp3
  updating podcasts/pistons/1/index.html
  updating podcasts/pistons/10/brosaen-episode-10.mp3
  updating podcasts/pistons/10/index.html
  updating podcasts/pistons/11/brosaen-episode-11.mp3
  updating podcasts/pistons/11/index.html
  updating podcasts/pistons/12/brosaen-episode-12.mp3
...

I'm guessing this is due to, as the code states,

MD5s can be stored in the 'ETag' field of S3 objects. The
field doesn't store the MD5 in two cases: objects uploaded
with Multipart Upload and objects encrypted with SSE-C or
SSE-KMS. In those cases, we'll just return an empty string.

so perhaps this is a feature request—is there a way to set the etag field for larger objects?

Amazon's docs say that, for multipart uploads, the ETag "will not necessarily be an MD5 hash of the object data." They don't actually say what it will be, just that it isn't the md5.
So, we don't have a confident way to know that the mp3s have not changed.
One option is to just look at file size and last-modified time for files that don't have usable ETags. This seems a little dangerous because the file contents could change without changing the size and we wouldn't know to push the update. Maybe we could disable it by default, but let users enable it through config if they understand the risk.

Thanks for the additional details, Spencer, makes sense.


Catalog should play nice with native dependencies too

Catalogs help with normal binary dependencies, but with runtime natives only is still quite verbose
Expected Behavior
I'd like the possibility to have something like this:
implementation(jogamp.gluegen)
runtimeOnly(jogamp.gluegen, classifier = joglNative)

Current Behavior
this is what the current implementation requires instead:
implementation(jogamp.gluegen)
val gluegen = jogamp.gluegen.get()
runtimeOnly(gluegen.module.group, gluegen.module.name, gluegen.versionConstraint.toString(), classifier = joglNative)

Context
Dealing extensively with dependencies for native libraries
I also tried
runtimeOnly(jogamp.gluegen) {
    artifact {
        name = "joglNative" // also `jogamp.gluegen.get().module.name
        classifier = joglNative
    }
}

but I get

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':runtimeClasspath'.

Closing as duplicate of #16768


Add binding to open debug info

it'd be nice to be able to open up the F3 menu (to view coordinates) without needing to hit a button on my keyboard or if someone's playing split screen and the controller player is lost and needs to be able to see them.
also, a virtual keyboard if possible for signs and chat would also be nice for the aforementioned reason (laziness or split screen with the other player using the keyboard)
it probably won't be easy to do, since you'd need to possibly make custom textures, but it'd be really valuable for those kinds of people.
Also, why do some bindings make a section go red when nothing else is bound to it? (left and right on dpad) i tried binding screenshot to dpad right and it just went red and essentially broke the button.
P.S: Switch pro controller works when using bluetooth, when wired windows is screwy and reads motion controls data as buttons, making the controller go crazy.

I can add a new binding to open the debug info.
Virtual keyboard is something on the todo list, it might be a while before it's added.
Screnshots are something that can be taken from anywhere (In-game/main menu/inventory). It's considered a universal bindings and will conflict with everything else if it has the same button assigned. The entire DPad is used to navigate the inventory and menus, so that's why binding screenshot to something on the Dpad will conflict.
There are very limited options on a controller compared to keyboard. Even the keyboard faces the same problem once you start adding a lot of mods. There is not a lot I can do to make everything work. I could make it so screenshots only work in-game (no inventories or menus) and that would fix the issue.

Added binding in https://github.com/MrCrayfish/Controllable/commit/cd6f9b61d99e4ab431ae9fa93fb556aa6cb23ac2


Update bower-asset/inputmask, bower-asset/punycode.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Could we have it in ascending order?

It seems that this PR broke MaskedInputAsset - https://github.com/yiisoft/yii2/issues/19988#issuecomment-1746617337, https://github.com/yiisoft/yii2/issues/19986


Sending complex objects over the network and deserializing them

I'm currently using this implementation of MessagePack for serializing/deserializing somewhat complex structures in a project I've been developing.
I want to be able to send and receive serialized MessagePack Objects that can then be parsed and constructed in order to build the "complex structure" on the receiving end.
The complex structure in question is a Blockchain. My Blockchain class consists of an array of Blocks (Block class instances) which themselves contain an array of Transactions (Transaction class instances).
I was able to serialize the Blockchain class by implementing  serialize methods in each of the classes: Blockchain::serialize(), Block::serialize(), Transaction::serialize().
After serializing the Blockchain, I would send the dump() output of that object to the receiving end, in order for it to construct it's own Blockchain from the incoming packet.
The problem is that, after serializing the Blockchain, I can access it's elements on the Sender Node (the one where the serialized blockchain originated from); however, I cannot to this on the Receiver Node. I get the MessagePack dump string but I am not able to parse the incoming buffer. I've checked the parse error  and it's populated with the message "end of buffer".
I'm not sure if this problem has to do with the way I try to serialize and construct the classes, or if it has to do with how I send the MessagePack dump() output to the receiver Node.
This issue might not have been very explicit but I am not sure how I can explain it better. I'm just looking for help to figure out what's causing this. I can get into further detail if someone's interested in helping out.
Also, keep in mind that I am a young dev, and I'm quite new to C++. The code may look like crap, be advised.

Hi @JoaoAJMatos
Thanks for your kind sample codes and information. I think that this trouble is caused by mishandling received data.
My guess is the following.
As you know, MessagePack is a binary serialization format. This 'binary' means that its serialized data contains null characters whose value is '\0'. So we cannot handle these values as a C language string that ends with a null character.
But this line constructs std::string instance from C language string. So if response_buffer contains null characters in the middle of itself, the constructor of std::string will stop reading the contents of response_buffer and construct an instance whose value is incompleted. I think this problem will be solved by adding the length of the content to the constructor of std::string.
The following code is a simulation to reproduce the above situation.  To mimic network communication, this code converts serizelizedCompany into a C language string and then creates a std::string instance.
int main()
{
    using namespace msgpack11;
    std::string err;

    Person person1("John", 30);
    Person person2("Rita", 0);
    Person person3("Pedro", 34);

    Company myCompany({person1, person2, person3});

    std::string serializedCompany = myCompany.serialize().dump();
    std::cout << "Serialized company: " << serializedCompany << std::endl;

    const char* recvData = serializedCompany.c_str();
    const int  recvDataLength = serializedCompany.length();

    std::string recvSerializedCompany(recvData);  // this line doesn't work
    //std::string recvSerializedCompany(recvData, recvDataLength); // this line works
    std::cout << "Received Serialized company: " << recvSerializedCompany << std::endl;

    Company myNewCompany(recvSerializedCompany);
    myNewCompany.printEmployees();

    return 0;
}

The result of the above code is the following. We can see that deserialization fails.
$ ./a.out
Serialized company: employeesagenameJohnagenameRitaage"namePedro
Received Serialized company: employeesagenameJohnage

After switching the construction of recvSerializedCompany, we get the following result. It works correctly.
$ ./a.out
Serialized company: employeesagenameJohnagenameRitaage"namePedro
Received Serialized company: employeesagenameJohnagenameRitaage"namePedro
John 30
Rita 0
Pedro 34

I'm happy that this will be useful for you.

Thank you once more for the reply.
You are completely right, I totally missed that!
I will try your fix and edit this reply if it works.
I cannot thank you enough for your time and effort :)
(Btw most of the code you see there is for debugging purposes and to try to find a work-around. Everything will be clean in the final project)

I have been testing out some stuff, and I realized something.
How can I send the serialized data through the socket, if I can't get the full string with c_str()?
Everything works as expected on the receiving end, the only problem is that I cannot send the full buffer to the client.
Do you have any idea of how I could solve this?


KAFKA-16707: Kafka Kraft : using Principal Type in StandardACL in order to defined ACL with a notion of group without rewriting KafkaPrincipal of client by rules

Default StandardAuthorizer in Kraft mode is defining a KafkaPrincpal as type=User and a name, and a special wildcard eventually.
The difficulty with this solution is that we can't define ACL by group of KafkaPrincipal.
There is a way for the moment to do so by defining RULE to rewrite the KafkaPrincipal name field, BUT, to introduce this way the notion of group, you have to set rules which will make you loose the uniq part of the KafkaPrincipal name of the connected client.
The concept here, in the StandardAuthorizer of Kafka Kraft, is to add  the management of KafkaPrincipal type:
Regex
StartsWith
EndsWith
Contains
(User is still available and keep working as before to avoid any regression/issue with current configurations)

This would be done in the StandardAcl class of metadata/authorizer, and the findresult method of StandardAuthorizerData will delegate the match to the StandardAcl class (for performance reason: precompile regex in ACL).
*I added tests in metadat, and run ./gradlew test from kafak:trunk and my fork: no more failed test on my branch than kafka:trunk
Committer Checklist (excluded from commit message)

[ x ] Verify design and implementation => thanks to spell checker in gradle process
[ x ] Verify test coverage and CI build status => adding few tests in metadata, an run gradlew test without more failed test thant kafka:trunk
[ x ] Verify documentation (including upgrade notes) : added few lines in doc, no upgrade info as the previous behaviour should still work as before.

Link to the JIRA-16707

Hello, When I'm running "./gradlew test" on my side from apache/kafka/trunk clone, I have failed tests.
So Is there a way to know (an easy one?) to know which failed test in "continuous-integration/jenkins/pr-merge" I have to look ?


Feat: add --cli-path flag support

I'm thinking, as we may make changes to the destination on Hookdeck if the CLI path doesn't match, I wonder if we should communicate it to the user more clearly to avoid unintended behavior.
I'm taking inspiration from Terraform, so we can have something along the line of:
$ hookdeck listen 3000 my_source --cli-path /new/path

Source "my_source" has a connection to "my_cli_destination" with a CLI path of "/old/path". Would you like to update it on Hookdeck? ...yes/no

This way, it's a bit interactive and we can avoid making changes that the users don't expect. To skip the interactivity part, we can do --auto-approve or something like that to make it more straightforward.
What do you think? I could be overthinking a behavior that doesn't warrant this level of caution?

Superseded by https://github.com/hookdeck/hookdeck-cli/pull/92


Use a user-specified/modified cmorize_obs_*.ncl/py file

Is your feature request related to a problem? Please describe.
Currently, the default cmorize_obs_*.ncl/py file installed under ESMValTool_root/cmorizers/obs is used for cmorization observational datasets.
For example:
cmorize_obs -c config_user.yml -o NCEP

will use the cmorizers/obs/cmorize_obs_ncep.ncl
As sometimes the esmvaltool is installed with root privilege, the users can not directly modify the cmorizer script. Therefore, is it possible to specify the cmorizer script to a user-modified version?
Would you be able to help out?
not sure I am able to implement this feature.
Thanks!
Yanchun

@valeriupredoi I think it is currently not possible to run a cmorizer from an arbitrary location outside the tool, is it?
@YanchunHe I would recommend installing the tool locally to add new features (see the user guide).

Thanks a lot @mattiarighi
If this is not possible, then I will have to install a personal copy.


Update browser data for display:contents bug

The severe display:contents bug was fixed in Chrome 89, see also Good news about display: contents and Chrome
This is my first contribution, hope this is the correct way.
I have verified it in Chrome 89, and also updated the data from 90-92, assuming it will still be fixed in those releases.

Looking good!
How have you verified it? Do you have a testcase? Maybe are even able to share it?
I'm asking because then we (you) could check Edge 89 as well... :)

I used

for tables https://codepen.io/aardrian/pen/xxGEKKJ
for lists https://codepen.io/aardrian/pen/abOBWKa and https://codepen.io/rachelandrew/pen/oNYapVb
for headings https://codepen.io/aardrian/pen/WNvoOpM
for buttons https://codepen.io/aardrian/pen/NWqbggX

Have just verified they also work in Edge 89, will update the PR.

Have just updated, thanks for the nudge!

Thanks!


Update logic handling failures for kapp-controller custom resources

What this PR does / why we need it:
It ensures that kapp shows the usefulErrorMessage rather than a message pointing towards it when there is a failure in a kapp-controller custom resource
Does this PR introduce a user-facing change?

Improve failure messages for kapp-controller resources

Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

Old output:
Target cluster 'https://<IP_ADDRESS>:54877' (nodes: minikube)

Changes

Namespace  Name      Kind  Age  Op      Op st.  Wait to    Rs  Ri
default    dev-test  App   -    create  -       reconcile  -   -

Op:      1 create, 0 delete, 0 update, 0 noop, 0 exists
Wait to: 1 reconcile, 0 delete, 0 noop

Continue? [yN]: y

4:28:15PM: ---- applying 1 changes [0/1 done] ----
4:28:15PM: create app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:28:15PM: ---- waiting on 1 changes [0/1 done] ----
4:28:15PM: ongoing: reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:28:15PM:  ^ Waiting for generation 1 to be observed
4:28:18PM: fail: reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:28:18PM:  ^ Reconcile failed:  (message: Deploying: Error (see .status.usefulErrorMessage for details))

kapp: Error: waiting on reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default:
  Finished unsuccessfully (Reconcile failed:  (message: Deploying: Error (see .status.usefulErrorMessage for details)))

vs
New output:
Target cluster 'https://<IP_ADDRESS>:54877' (nodes: minikube)

Changes

Namespace  Name      Kind  Age  Op      Op st.  Wait to    Rs  Ri
default    dev-test  App   -    create  -       reconcile  -   -

Op:      1 create, 0 delete, 0 update, 0 noop, 0 exists
Wait to: 1 reconcile, 0 delete, 0 noop

Continue? [yN]: y

4:34:13PM: ---- applying 1 changes [0/1 done] ----
4:34:13PM: create app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:34:13PM: ---- waiting on 1 changes [0/1 done] ----
4:34:13PM: ongoing: reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:34:13PM:  ^ Waiting for generation 1 to be observed
4:34:16PM: fail: reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default
4:34:16PM:  ^ Reconcile failed:  (message: kapp: Error: Expected to find kind '/v1/ConfigMaps', but did not:
- Kubernetes API server did not have matching apiVersion + kind
- No matching CRD was found in given configuration)

kapp: Error: waiting on reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default:
  Finished unsuccessfully (Reconcile failed:  (message: kapp: Error: Expected to find kind '/v1/ConfigMaps', but did not:
- Kubernetes API server did not have matching apiVersion + kind
- No matching CRD was found in given configuration))

@praveenrewar this would also affect the information fields in diff summary, how do you feel about that?
Example:
Target cluster 'https://<IP_ADDRESS>:54877' (nodes: minikube)

Changes

Namespace  Name      Kind  Age  Op  Op st.  Wait to    Rs    Ri
default    dev-test  App   55s  -   -       reconcile  fail  Reconcile failed:  (message: kapp:
                                                             Error: Expected to find kind
                                                             '/v1/ConfigMaps', but did not:
                                                             - Kubernetes API server did not
                                                             have matching apiVersion + kind
                                                             - No matching CRD was found in
                                                             given configuration)

Op:      0 create, 0 delete, 0 update, 1 noop, 0 exists
Wait to: 1 reconcile, 0 delete, 0 noop

We could start truncating here, but we do not do that for other resources (to the best of my knowledge)

4:34:16PM:  ^ Reconcile failed:  (message: kapp: Error: Expected to find kind '/v1/ConfigMaps', but did not:
- Kubernetes API server did not have matching apiVersion + kind
- No matching CRD was found in given configuration)

kapp: Error: waiting on reconcile app/dev-test (kappctrl.k14s.io/v1alpha1) namespace: default:
  Finished unsuccessfully (Reconcile failed:  (message: kapp: Error: Expected to find kind '/v1/ConfigMaps', but did not:
- Kubernetes API server did not have matching apiVersion + kind
- No matching CRD was found in given configuration))

Hmm, that seems like a lot of error messages, should we try to avoid printing the error in one of the places? Probably not a good idea, but just thinking out loud.

Namespace  Name      Kind  Age  Op  Op st.  Wait to    Rs    Ri
default    dev-test  App   55s  -   -       reconcile  fail  Reconcile failed:  (message: kapp:
                                                             Error: Expected to find kind
                                                             '/v1/ConfigMaps', but did not:
                                                             - Kubernetes API server did not
                                                             have matching apiVersion + kind
                                                             - No matching CRD was found in
                                                             given configuration)

Ouch!! Yeah, truncating does seems like a good option, but how would users view the complete error message then 🤔

That makes sense to me, merging this away!

kctrl should surface information for failed PackageInstallations that are a part of a meta package better for versions of kapp-controller that are using kapp with these changes 🚀


Vertical line mouse move

Context
Look I use this code https://jsfiddle.net/56s9av1j/ for add vertical line. But I have problem in core.helpers.js var boundingRect = canvas.getBoundingClientRect();  -> error Uncaught TypeError: Cannot read property 'getBoundingClientRect' of undefined
may be you can help me how I can add vertical line on mouse move ? Some example https://jsfiddle.net/56s9av1j/
Environment

Chart.js version: 2.7.3
Browser name and version: chrome, last version

The fiddle you shared works okay for me. Can you create a fiddle that has the problem you describe?

Fiddle works fine for me. There is also a plugin for this https://github.com/abelheinsbroek/chartjs-plugin-crosshair


Encoder class kwarg is ignored on json serialization

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
The encoder class is ignored when passed through the cls kwarg of the json method of BaseModel. It is a valid json.dumps kwarg that expects an encoder class (example), but since a default encoder is used when json's encoder kwarg is None, the encoder class' default method is "overwritten" by it.
Example Code
from pydantic import BaseModel
from json import JSONEncoder

class MyExoticType:
    pass

class Model(BaseModel):
    x: MyExoticType

    class Config:
        arbitrary_types_allowed = True

class CustomEncoder(JSONEncoder):
    def default(self, o):
        if isinstance(o, MyExoticType):
            return 'exo'
        raise TypeError('not serialisable')

# Raises 'TypeError' instead of properly encoding.
Model(x=MyExoticType()).json(cls=CustomEncoder)

Python, Pydantic & OS Version
pydantic version: 2.0.0.dev0
                 install path: /Users/gustavokrieger/PycharmProjects/pydantic/pydantic
               python version: 3.10.4 (main, Jun 28 2022, 21:27:37) [Clang 13.1.6 (clang-13<IP_ADDRESS>.5)]
                     platform: macOS-12.5.1-arm64-arm-64bit
     optional deps. installed: ['devtools', 'email-validator', 'typing-extensions']

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[X] Data serialization - .dict() and .json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

I have an idea of a solution. If you guys agree this is an actual issue, please let me know what I need to do before opening a PR.

Thanks @gustavopcassol for the issue.
We are working on V2 and probably json() will be replaced by model_dump_json().
So, I would suggest waiting for V2

Oh, ok. Anyway, if you guys ever look back at this, what I had in mind for a solution was this code:
if encoder or 'cls' not in dumps_kwargs:
    encoder = cast(Callable[[Any], Any], encoder or self.__json_encoder__)

Replacing this line:
https://github.com/pydantic/pydantic/blob/60cad30b2cd1dc3aa1858ddf907666e6f3429182/pydantic/main.py#L487

In v2, we've removed the ability to pass json dumps kwargs directly into model_json_dumps, so I think this is no longer relevant.
But I'll note that you should be able to achieve everything that could be achieved with JSON encoders through the new @field_serializer or @model_serializer decorator. I'm not sure whether we have much documentation for these yet, but either way, there are many tests you can look at using these decorators on the main branch.
If there is functionality that you need but can't replicate, please let us know; I think anything that is still not possible should be doable by adding context to serialization (if we haven't already? I'm sure we'll need to eventually..)
Because of the above, I'm going to close this issue now, but feel free to create a new issue if the new approach in v2 doesn't work for you.


Option to "align" text like curved but displayed straight

Is there an option to display text horizontally but similar to "curved"'s alignment?

Hi, no there is currently no option for this and I don't see how any combination of the existing parameters could make this happen.

@zarocknz thanks for confirming.

Same issue. And problem with curved – only monospaced Fons looks good on curve. Any other fonts looks ugly.

Same issue. And problem with curved – only monospaced Fons looks good on curve. Any other fonts looks ugly.


[feature request] Unix sockets

I would really like to use libasync for IPC on Linux (specifically I would like to support it in i3ipc-d as the event dispatcher), but that does require support for unix sockets. Would it be possible to get support for those in libasync?

Of course, will add within a month or so (once I have time!)

Would it be possible for you give me an overview on what would need to be implemented?
Since I really need this for a project of mine (for which I wrote i3ipc-d in the first place), I tried to judge whether I would be able to implement this myself, but duplicating the tcp implementation and modifying it for unix did not appear to me to be very straight forward, as egrep -r revealed extensive references to tcp, i.e. even if I implemented unix.d, I cant easily make out how to plug in into the rest of libasync.

That'll be a pre-initialized file descriptor, you could probably do it by passing a custom fd to AsyncTCPConnection but I think AsyncEvent would be more of a fit for it. You'd need to create the unix socket beforehand using c-style examples. http://troydhanson.github.io/network/Unix_domain_sockets.html

Okay, thank you for the pointers, I'll try may hand at it.

First attempt of it is here, though I did have to rewrite some parts of AsyncEvent to support stateful sockets (I need connect and close events to be dispatched for unix domain sockets, but AsyncEvent did not support that yet): There is new argument in the constructor with which the caller tells AsyncEvent whether the given file descriptor is a stateful socket (like UDS) or not.

Looks good so far, it seems to fit in nicely with the rest. Does this work in your tests?

Cool, I was concerned that you may object to changing AsyncEvent like that.
Regarding to your questio: I'm currently using a minimal test like this:
auto loop = getThreadEventLoop;

auto unix = new AsyncUDSConnection(loop);
unix.peer = new UnixAddress("/var/run/user/1000/i3/ipc-socket.1234");
unix.run((code) {
	writeln(code);
});

while (true) {
	loop.loop(-1.seconds);
}

and the connect/read/close events seem to be dispatched correctly. I'll follow up with a more complete test soon.

https://github.com/Calrama/libasync/commit/f756d97e74daff281ef1c126fdade80f42d91025 seems to be working as expected (complete UDS connection). Up next are a UDS listener, the promised test as an example (using both) and then making sure that this doesn't break either build or tests on non-UDS platforms

Awesome

https://github.com/Calrama/libasync/commit/1b80b608ac05ee02a015be7fce1aa39798c171c1 implements a UDS listener that seems to work fine. I followed the API for TCP Listener mostly, though I do require for the delegate that received the new UDSConnection to be nothrow (I could not think of any reason to allow exceptions here).

https://github.com/Calrama/libasync/commit/9eb2d2533672e62224447a002c928744da6c4b5a has usage examples for UDS connection and UDS listener

Whelp, it works.

Since the above PR has been merged, I'll close this.

Thanks for the addition. I'm sure a lot of future users will also be thankful the great work


Add in the extra component to the Velvet tutorial

From Simon's tutorial in the Perth workshop.
fork from github and integrate

had already done 18 Nov.


How to Deploy A Smart Contract Using Web3Dart

In Javascript We Deploy Smart contract something like that
let contract = new web3.eth.Contract(DonactionsAbi);
let deploy = await contract.deploy({ data: DonactionsByteCode, arguments: [address, Name] });
I just want to know how to Deploy smart contract in Dart

perhaps try something like that. I hope it's a lead
import 'dart:convert';
import 'dart:developer';
import 'package:flutter/services.dart';
import 'package:http/http.dart';
import 'package:web3dart/web3dart.dart';

class Contract {
  final String apiUrl = 'http://<IP_ADDRESS>:7545';

  Future<void> deploy(
    String privateKey,
    String address,
    String name,
  }) async {
    final privateKey = 'xxxxxxx';

    final web3Client = Web3Client(apiUrl, Client());
    final credentials = EthPrivateKey.fromHex(privateKey);

    final abiStringFile =
        await rootBundle.loadString('truffle/build/contracts/HTLC_ETH.json');
    final jsonAbi = jsonDecode(abiStringFile);
    final contract = DeployedContract(
      jsonAbi,
      EthereumAddress.fromHex(address),
    );

    final transaction = Transaction.callContract(
      contract: contract,
      function: contract.function('constructor'),
      parameters: [
        EthereumAddress.fromHex(address),
        name,
      ],
    );

    final transactionHash = await web3Client.sendTransaction(
      credentials,
      transaction,
    );
    final receipt = await web3Client.getTransactionReceipt(transactionHash);

    log('Contract deployed at ${receipt!.contractAddress!.hex}');
  }
}


Rematch: post release fixups

closes #70
closes #69
closes #68

Also cc @gizzmo since it touches Rematch stuff.


Using prototype.js hack/fix and wrapping the digits

Hey currently using the prototype fix listed here by Ronpeled #44. How could I wrap the digits using this format?

Check out the examples: http://hilios.github.io/jQuery.countdown/examples/legacy-style.html