having issue with api server

I am making some experiments and have some troubles. I have changed some configurations of api server to enable cron resources. I checked docker ps and containers are appearing in the list with new process ids. I checked get nodes as well and seems normal.
$ kubectl get nodes
NAME                                           STATUS                     AGE
ip-10-0-10-10.xx-central-1.compute.internal    Ready,SchedulingDisabled   56d
ip-10-0-10-11.xx-central-1.compute.internal    Ready,SchedulingDisabled   56d
ip-10-0-10-115.xx-central-1.compute.internal   Ready                      56d
ip-10-0-10-12.xx-central-1.compute.internal    Ready,SchedulingDisabled   56d
ip-10-0-10-164.xx-central-1.compute.internal   Ready                      46d
ip-10-0-11-109.xx-central-1.compute.internal   Ready                      56d
ip-10-0-11-218.xx-central-1.compute.internal   Ready                      56d

But when I check endpoints I can only see a single endpoint. This seems problematic for me because I am using default configuration and there is 3 master.
$ kubectl get endpoints
NAME              ENDPOINTS        AGE
kubernetes        <IP_ADDRESS>:443   56d

When I try to access by browser I can see that there is connectivity for them as well. <IP_ADDRESS> and <IP_ADDRESS> seems up.

Can someone make a guess why my endpoints are not listed correctly?
Which part of the tack creates kubernetes service in default namespace?

Thanks

Probably this the k8s version
This script update the templates in manifests/addonsmove this to .addons and run 	kubectl create -f .addons/

You mean I am using an absolute version which is causing this error? I am using 1.5.1 right now might be better to upgrade latest 1.5.x?
I could not see relevant part in there? There is no add-on about it. What am I referring is this:

$ kubectl get service
NAME              CLUSTER-IP   EXTERNAL-IP   PORT(S)          AGE
kubernetes        <IP_ADDRESS>     <none>        443/TCP          56d

Thanks for your help

@cemo i'm seeing 3 endpoints with the latest drop on master.
$ kubectl get endpoints
NAME         ENDPOINTS                                      AGE
kubernetes   <IP_ADDRESS>:443,<IP_ADDRESS>:443,<IP_ADDRESS>:443   38m

@wellsie there is an issue at my side and see for details. I will close this issue since your setup is working properly at k8s 1.6.x.


Editor - HTML Card doesn't launch in edit mode

Issue summary
When created the HTML Card doesn't launch in edit mode. This is because codemirror has an issue with the cursor when focused this way.

Steps to reproduce

Add an Embed card

Expected Results
The card should be added in edit mode like the markdown card is.

I'll take a stab at this one unless you've already started @disordinary? As it's Codemirror related I'll see if I can spot anything that may help with random test failures we have with our Code Injection screen as well

Sure.
And no, I haven't started it.
The issue is that if we tell codemirror to focus the cursor appears but it doesn't accept any inputs. There are quite a few bugs relating to the cursor in and around the editor but I have yet to dig deep enough to know if it's mobiledoc related or not.


[BUG] Some icons are still displayed in black in dark mode

Describe the bug
Some icons display in black on grey background, instead of having the appropriate contrast.
To Reproduce
• Look at the mobile hamburger menu in dark mode.
• Also for example, add a conformance editor in dark mode.
Screenshots

Hm it looks like a lot of this comes from somewhat hacky treatment of theming, where the dark-theme CSS has some ad-hoc overrides of certain classes like .btn-light, and the .btn-light aren't replaced with .btn-dark and similar because that requires continuously replacing in the templates. I think solving this properly requires the theming architectural changes needed for #1627, but the biggest problems can be fixed directly.


[CosmosDB] Upgrade to version 3.0.0 and add support for NetworkAclBypass + Update Mongo ServerVersion + backup policy

Description

Change needed for ignite with support of NetworkAclBypass.
-Adds params network_acl_bypass and network_acl_bypass_resource_ids to az cosmosdb update/create
-Adds update serverversion
-Adds params backup_retention and backup_inteval to az cosmosdb update/craete
Testing Guide

az cosmosdb create --name accountName --resource-group rgName --network-acl-bypass AzureServices --network-acl-bypass-resource-ids subscriptions/subId/resourceGroups/rgName/providers/Microsoft.Synapse/workspaces/wsName
az cosmosdb update --name accountName --resource-group rgName --network-acl-bypass AzureServices --network-acl-bypass-resource-ids subscriptions/subId/resourceGroups/rgName/providers/Microsoft.Synapse/workspaces/wsName
az cosmosdb update --name <account_name>--resource-group <resource_group> --server-version 4.0
-az cosmosdb create --name accountName --resource-group rgName --backup-interval 240 --backup-retention 8
-az cosmosdb update--name accountName --resource-group rgName --backup-interval 240 --backup-retention 8

History Notes

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

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

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

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

CosmosDB

Hi @yonzhan @Juliehzl This change is needed for Ignite, could we change the label to S183? Thank you!

Hi @Juliehzl We have combined this change with 2 other CosmosDB changes. Please review

/azp run

/azp run

/azp run

Hi @kristynhamasaki , could you fix CI failures?

Hi @Juliehzl I reran the one test that hadn't been updated. Please queue the pipeline again. Thank you!

/azp run

/azp run

/azp run


Performance: Split bundle.js into separate components (bootstrap + UI)

Following from @jordwalke's comments on #933 - one thing we should look at is splitting up our bundle.js.

We have one giant bundle.js that handles all of our UI and Neovim interaction - but that's not really necessary, especially on first-load.
Ideally, we could isolate the minimum dependencies needed to get up-and-running (like NeovimInstance, CanvasRenderer, etc - bootstrapping stuff), and split out from the UI elements, so that we can minimize the startup time.

Need to spend some time with these docs: https://webpack.js.org/guides/code-splitting/

This is done with #1123  -

The entry point is 54.1 kb... hopefully we can continue to rev these and require less-and-less on startup. 0.bundle.js is needed relatively shortly after, but ideally we can trim that and load more in parallel while Neovim is starting.


feature: summarize implemented algorithms in stdlib and runtime

I think we need to declare the interested packages before we start digging more deeper. Feel free to add your preferences.

strings
sort
hash
image

I think this request is duplicated with the purpose of summarizing design history. An implemented algorithm usually mixes multiples algorithms with clever engineering, some of them don't even have a name.
One can always read the design history and figure out the actual detail. Thus, close.

I think this request is duplicated with the purpose of summarizing design history. An implemented algorithm usually mixes multiples algorithms with clever engineering, some of them don't even have a name.
One can always read the design history and figure out the actual detail. Thus, close.


Recall Preset Problem

When I set my Recall Preset command it seems to recall the next preset number instead of the one I set. Ex: when I recall Preset 1 the camera moves to the Preset 2 location, and so on....Preset 8 triggers Preset 9, etc.

It sounds that perhaps the preset id starts at zero instead of one.

It does seem to act like this. How could this get tested/fixed?

What camera and what firmware are you using ?

Using Birddog P200 cameras with the latest baseboard firmware and the recently available beta camera firmware.

it looks that BD changed the preset numbers.
I Programmed it with the 3.3 and there everything was right.
I take a look that i make a checkbox if you use version > 3.3 to be compatible for the next weeks until they have a stable release version from the firmeware

Thanks BenniM, I’m a bit unsure of what you mean, did you say you will revise your code when a stable released FW version is launched by BD, perhaps in the next few weeks?

Fixed. Thanks.

Fixed. Thanks.


fix: style download button on commits page

Thanks for you PR.
Could you add screenshots to describe what's changed, and add a link can test the changes? I did not see any difference😅

Actually I think the download button on commits list page should be removed, it makes no sense.

Yep. I also thought about but then I was able to style it pretty easily.

I've just fixed this issue, you can update to the latest version and have a try

install script via greasyfork, greasyfork home page
install script via openuserjs, openuserjs home page


Get model that was deleted in FlowContentObserver

DBFlow Version: 3.1.1, 4.0.0
Issue Kind (Bug, Question, Feature): Feature
Description:
FlowContentObserver notifies about delete actions after they've completed, so there is no way to get the original model's fields. It'd be nice if there was a way to get the model or have an event that fires before delete operation is complete.

If you use the OnModelStateChangedListener, you will receive a SQLCondition[] of Primary keys of the objects that were deleted.


reports that leaving rooms on matrix.org is very buggy.

you have to flush cache in order to get the app back but simply trying to leave a room leaves the app very confused, says aaron. https://matrix.to/#/!tDRGDwZwQnlkowsjsm:matrix.org/$15498943972616032JzCqy:matrix.org?via=matrix.org&via=disroot.org&via=riot.tenx.tech

Matthew: prior behavior was the app would spin for 1-5 minutes, then the room would be left, removed from the list of active rooms.

Currently: the app spins for 1-5 minutes, then dumps you back on the settings screen. You can click cancel, then return to the main view. The contents of the room are “you have left”, and I believe the server stops sending you room events.


MxNet -> MXNet

Job d2l-zh/PR-631/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-631/

Job d2l-zh/PR-631/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-631/


Security only supports "OR" logic, not "AND"

Problem statement
When specifying a security block in the AND format (described here: https://swagger.io/docs/specification/authentication/ ), having only one pass allows requests to continue without a 401.
The documentation states:
security:
  - A
  - B

 # A OR B
security:
  - A
    B

 # A AND B

Swagger specification
I've included the relevant bit here.  I've put a full test case at https://github.com/gavbaa/go-swagger-auth-test if you want to clone and review directly.  It's the TodoList Swagger file, with only the following security bit added.
securityDefinitions:
  AppTokenAuth:
    type: apiKey
    in: header
    name: App-Token
  TokenAuth:
    type: apiKey
    in: header
    name: Token

security:
  - AppTokenAuth: []
    TokenAuth: []

Steps to reproduce
I implemented only the AppTokenAuthAuth function and left TokenAuthAuth returning an error:
	api.AppTokenAuthAuth = func(token string) (interface{}, error) {
		return true, nil
		//return nil, errors.NotImplemented("api key auth (AppTokenAuth) App-Token from header param [App-Token] has not yet been implemented")
	}

	// Applies when the "Token" header is set
	api.TokenAuthAuth = func(token string) (interface{}, error) {
		return nil, errors.NotImplemented("api key auth (TokenAuth) Token from header param [Token] has not yet been implemented")
	}

However, upon cURLing with just an App-Token, it moves onto the method handler instead of rejecting the request with a 401.
➜  ~ curl -H "App-Token: x" "http://<IP_ADDRESS>:58282"
"operation todos.Get has not yet been implemented"

Environment
swagger version: 2.0.0
go version: 1.8.1
OS:  OSX 10.12.5

New to go-swagger but I've also run into this issue and had a quick look into the code.  Long story short, the problem ultimately is with the go-openapi router + middleware collapsing the path/route security []map into just an [] of unique requirements, go-swagger then implements this / has to.
The go-openapi which go-swagger depends on defines a struct model for a swagger spec, take a look at spec.Operation which has a spec.OperationProps which seems to define the properties on the route path.
spec.OperationProps defines security prop as an "[]map[string][]string"
This matches with the swagger spec / documentation, i.e there are a list of acceptable maps of security definitions, the list is the OR part and the map is the AND part.
Where this seems to go wrong / get lost is in the go-swagger analyzer, take a look at analysis.SecurityRequirementsFor:
// SecurityRequirementsFor gets the security requirements for the operation
func (s *Spec) SecurityRequirementsFor(operation *spec.Operation) []SecurityRequirement {
////
schemes = operation.Security
////
unique := make(map[string]SecurityRequirement)
for _, scheme := range schemes {
for k, v := range scheme {
if _, ok := unique[k]; !ok {
unique[k] = SecurityRequirement{Name: k, Scopes: v}
}
}
}
var result []SecurityRequirement
for _, v := range unique {
	result = append(result, v)
}
return result

}
It takes the []map and squashes it down into a flat array of "unique" security requirements, destroying the grouping.  Really here it should return a [][]SecurityRequirement.  However, worse still go-swagger depends on the go-openapi runtime / middleware / router and that to my eye seems broken / explains why it is doing it.
In go-openapi middleware package router.go, the middleware.AddRoute function, it takes the go-swagger analazyer implementation from above and gets the security definitions as an array, and then goes on to match Authenticators for them which I suspect are created by the securityDefinitions:.
func (d *defaultRouteBuilder) AddRoute(method, path string, operation *spec.Operation) {
////
////
definitions := d.analyzer.SecurityDefinitionsFor(operation)
////
record := denco.NewRecord(pathConverter.ReplaceAllString(path, ":$1"), &routeEnt
////
Authenticators: d.api.AuthenticatorsFor(definitions)
////
}
That seems just wrong / it should be supporting the list of map here. Having a look at the middleware context.go, middleware.Authorize:
for scheme, authenticator := range route.Authenticators {
	applies, usr, err := authenticator.Authenticate(&security.ScopedAuthRequest{
		Request:        request,
		RequiredScopes: route.Scopes[scheme],
	})
	if !applies || err != nil || usr == nil {
		if err != nil {
			lastError = err
		}
		continue
	}
	if route.Authorizer != nil {
		if err := route.Authorizer.Authorize(request, usr); err != nil {
			return nil, nil, errors.New(http.StatusForbidden, err.Error())
		}
	}
	rCtx = stdContext.WithValue(rCtx, ctxSecurityPrincipal, usr)
	rCtx = stdContext.WithValue(rCtx, ctxSecurityScopes, route.Scopes[scheme])
	return usr, request.WithContext(rCtx), nil
}

You can see it loops through the Authenticators and if one applies and is valid, the authorizer is then questioned with the principal and the route continues or errors.  So its doing the OR part on a flattened security definition.
This isnt just broken, its actually dangerous as if any sub part of the auth passes, the route will pass / be hit.
So looks like a fix to go-openapi is needed to handle the []map and then a fix to go-swagger is needed to implement it.  First one seems a bit tricky as the router / routes are fairly involved, the second one is simple enough apart from you need to change the codegen which looks fairly involved / steep looks learning curve.

Thanks for the analysis. Is this for openapi 2.0 or 3.0. In the first comment the specification linked is 3.0 which is not what we currently support.

https://swagger.io/docs/specification/2-0/authentication/
Yeah 2.0, you can see in the go-openapi spec.OperationProps object has an [] of map[], so the spec object can handle a list of grouped requirements with scopes, it just the rounter fails to implement it and the knock on API to that go-swagger has to implement.
The change looks fairly reasonable except for the fact one principal to rule them doesnt really cut it.  A   a typed map[string]interface[] to map security definition id -> principal for whichever security group passes map be a reasonable strategy, could simplify if there was only 1 requirement / return just the principal .  Seems though this doesn't interact well with the -P option / defined principal / something clever with codgen may work.

I'm looking into this, however I don't really see how you can have 2 principals.
Once you authenticate you should have a single identity, because you're interacting with a single entity which is proving its identity.
Under which circumstances would you need to return a principal that can't be represented by a single object? Whether it's through a union like type or otherwise.
I can fix the AND vs OR logic that's all fairly doable but I'm having trouble understanding why you'd ever have multiple identities represented in your application.

This isn't about having two principals, it's about the single principal system requiring two auth tokens to validate access.  In this case, "ApiKeyAuth" as the Swagger Authentication documentation describes it.

I understand that piece, it's this comment: https://github.com/go-swagger/go-swagger/issues/1089#issuecomment-337518939 from @mmmguitar that I'm unclear about

This functionality has been merged to master

@casualjim, just noticed this sorry,  may well be my lack of understanding but the principle returned relates to what auth'd it doesnt it?  For example, if you do an API key it returns the key as the principle, or if you do a JWT token it returns the JWT token as the principal from which you can extract things.
So where you have the AND logic, API key and JWT token -> what gets returned as the principal there? Wouldn't the principle be a combination of the two?

I think the simplest solution here would be to have a final catch all method that would take the array of principals returned from the various methods that could combine then and return the final principal.

But the proposals here are return a map or return an array.
I don't see how those are different from what I propose, your security principal is a single struct with properties. Presumably you know what auths your application is doing and which fields should be filled out.
You can say the security principal is a map or a slice, the thing doesn't care much about that type.
However what isn't here is the migration to using context based authenticators (which can be fixed fairly easily). At that point you should have everything to you need to capture these use cases.


added memory fallback for titanium android

@wei-lee need to add this on android as it doesn't have a window name

@wei-lee yes this is for titanium, well dom is obvious as it's a native front-end and 'maybe' that is why there is no webkit-sql as well as it's only a javascript runtime

@edewit ok, but if we use memory, all the data will not be persisted for Titanium. Is this ok?

guess it was no different if you are using 'window-name', I'm not sure what is persisted with Lawn chair.

@wei-lee so good to merge?

I think perhaps better use this? https://github.com/feedhenry/fh-js-sdk/blob/master/libs/lawnchair/lawnchairTitanium.js. Turns out we already have a data adapter for Titanium!

@wei-lee good to merge?

@edewit Looks like it needs a rebase from master

@nialldonnellyfh rebased

@edewit It needs a version bump.

@wei-lee @nialldonnellyfh it needs a rebase again, can't we merge this one?

I agree w/ @edewit, lack of frequent merges cause pain to maintain PRs
ship it :smile:
On Wednesday, 18 May 2016, Erik Jan de Wit<EMAIL_ADDRESS>wrote:

@wei-lee https://github.com/wei-lee @nialldonnellyfh
https://github.com/nialldonnellyfh it needs a rebase again, can't we
merge this one?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/feedhenry/fh-js-sdk/pull/161#issuecomment-219973694

--
Sent from Gmail Mobile

LGTM. merge it


Small website fixes

Fixed app insights in the Designer not firing when "Templating Docs" is clicked
Updated some homepage copy

Microsoft Reviewers: Open in CodeFlow

PS: @matthidinger, the fabricbot comment around website testing only gets added when a PR is created.


Dropdown component and image format selection

I've added a new dropdown component that allows users to select different image formats (JPEG, PNG for now) for export. This should give users more flexibility in how they save their gradients. Check it out and let me know what you think!

Screenshot:

The new dropdown component for selecting image formats looks great! Please create a separate branch for your changes before submitting the pull request.


PaintingContext creates level that is never disposed.

PR: https://github.com/flutter/flutter/pull/133940
final OffsetLayer layer = child.updateCompositedLayer(oldLayer: null);

@goderbauer and @chunhtai , can you help to triage it?

Adding this to triage to get it triaged.

Hi @polina-c, can you provide a complete minimal reproducible code sample?
PS: after adding the team-framework label the issue is now in the incoming list for the framework team so it will be picked up by secondary triage. in triage is usually used by primary triage.

Steps:

Convert the test 'InkFeature skips painting if intermediate node skips' to testWidgetsWithLeakTracking, the same way as the linked PR.
run flutter test test/material/material_test.dartand and see failures

Reproducible using the steps outlined above.

test logs
To run this test again: /Users/nexus/dev/sdks/flutters/bin/cache/dart-sdk/bin/dart test /Users/nexus/dev/sdks/flutters/packages/flutter/test/material/material_test.dart -p vm --plain-name 'Border painting (tearDownAll)'
00:09 +41 -4: LookupBoundary (tearDownAll) [E]
  Expected: leak free
    Actual: <Instance of 'Leaks'>
     Which: contains leaks:
            # The text is generated by leak_tracker.
            # For leak troubleshooting tips open:
            # https://github.com/dart-lang/leak_tracker/blob/main/doc/TROUBLESHOOT.md
            notDisposed:
              total: 4
              objects:
                PictureLayer:
                  test: InkFeature skips painting if intermediate node skips
                  identityHashCode:<PHONE_NUMBER>
                ContainerLayer:
                  test: InkFeature skips painting if intermediate node skips
                  identityHashCode: 643197407
                PictureLayer:
                  test: InkFeature skips painting if intermediate node skips
                  identityHashCode: 892199783
                ContainerLayer:
                  test: InkFeature skips painting if intermediate node skips
                  identityHashCode: 925766089

  package:matcher                                                  expect
  package:flutter_test/src/widget_tester.dart 458:18               expect
  package:leak_tracker_flutter_testing/src/test_widgets.dart 81:5  _tearDownTestingWithLeakTracking
  ===== asynchronous gap ===========================
  dart:async                                                       _CustomZone.registerBinaryCallback
  package:leak_tracker_flutter_testing/src/test_widgets.dart 59:9  configureLeakTrackingTearDown.<fn>

flutter doctor -v
[✓] Flutter (Channel master, 3.14.0-14.0.pre.137, on macOS 13.4.1 22F770820d darwin-arm64, locale en-GB)
    • Flutter version 3.14.0-14.0.pre.137 on channel master at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 83c24274bb (2 hours ago), 2023-09-07 01:20:06 -0400
    • Engine revision 187c5b3c5f
    • Dart version 3.2.0 (build 3.2.0-134.0.dev)
    • DevTools version 2.27.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode-14.3.0.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/232.8660.185/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.1)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/232.9559.62/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.72.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.4.1 22F770820d darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.179

[✓] Network resources
    • All expected network resources are available.

• No issues found!


entgql: syntax error when there's no field in the schema type.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
with this code:
func (User) Fields() []ent.Field {
	return nil
}
func (User) Mixin() []ent.Mixin {
	return []ent.Mixin{
		BaseMixin{},
	}
}

I am seeing this error for this selection of code when I have no field in a schema type
./gql_collection.go:1005:16: undefined: fieldSeen
./gql_collection.go:1006:29: undefined: selectedFields
./gql_collection.go:1007:5: undefined: fieldSeen

The error is happening possibly because of this piece of code
https://github.com/ent/contrib/blob/b00990bedaf062d407b56294e526ffb4adde5aa8/entgql/template/collection.tmpl#L43-L57
expected Behavior:
I want the code generation to be successful.

Tech
Version

Go
1.20.4

Ent
0.12.3

Database
Postgres

Driver
github.com/jackc/pgx/v5

Hello @hsblhsn, can you create a repo to reproduce the issue? I tried with the latest version of Entgql but can't create it yet.

Actually, fields in mixin also can not see in api, like
client.User.Create().SetSeen().....
SetSeen() won't generate by go generate ./...


Add PBR tutorial - updated

This replaces https://github.com/KhronosGroup/glTF-Tutorials/pull/46 , now that the CLA handling should be set up correctly.
As soon as this is merged, the link at https://github.com/KhronosGroup/glTF/blob/master/README.md#tutorials can be updated to point to the Khronos version.

@Khronoswebmaster I thought (based on https://github.com/KhronosGroup/glTF-Tutorials/pull/46#issuecomment-813432804 ) that it should now no longer show the CLA as a pending check - did I misunderstand something here?

I will contact cla-assistant and see how we fix this. This repository is not linked to cla-assistant and there are no webhooks, but still it is attempting to approve this. Clicking the Details link beside cla-assistant in this repository shows this error:
There is no CLA to sign for KhronosGroup/glTF-Tutorials

 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.0 out of 3 committers have signed the CLA.:x: Mohamad Moneimne:x: pjcozzi:x: moneimneMohamad Moneimne seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.0 out of 3 committers have signed the CLA.:x: Mohamad Moneimne:x: pjcozzi:x: moneimneMohamad Moneimne seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

If this is the easiest solution, I can also close this PR and open it as a new one (that is: if the current issue might only be a glitch to some settings being changed while this PR was open, and if this issue does not affect future PRs). Just drop me a note if I should try it.

If you create a new PR, does it still show the CLA. This would be interesting to find out. The fact these PRs are stuck with still having the CLA enabled worries me that new PRs will also have the CLA enabled.


Use custom theme even on RTD

Now that we've tested it locally we want it on `production' RTD
I've tested this by using a custom version on RTD, and you can see the link here

LGTM


Proposed addition of “price” to “terms” during an etch

Hey! I wanted to know if we were to submit a PR to ord altering the indexer to do the following if it’s something that has a possibility of being approved.
The addition of a “price” field
Currently an etches “terms” looks like the following:
struct Terms {
amount: Option,
cap: Option,
height: (Option, Option),
offset: (Option, Option),
}
I propose an OPTIONAL extra field “price” to be added to these terms, expressed as the following:
price: Option<u128>
How the “price” field would function
Price is an integer expressed in Satoshi. During an etch, IF the price field is present the following would be added as a requirement:
For the following definition, “Etch Output” refers to the following:
The address owning the first NON-OP_RETURN transaction output of an Etch transaction or, in the case it’s provided, the address owning the transaction output specified by “pointer” of an Etch transaction:
A transaction including a runestone that is minting MUST include a transaction output transferring a MINIMUM of “price” amount of satoshi to ”Etch Output”
If there are no transaction outputs in vout that transfer a MINIMUM of “price” to “Etch Output”, the mint is ignored BUT the transaction is not considered a cenotaph and edicts are still processed.
If no price field is provided, all functionality above is ignored.
Usecase
This would create a decentralized way to create ICOs natively on Bitcoin - allowing the fungibility of Rune to be used as a mechanism to fund innovative projects on the space
Final thoughts
I leave my proposal here for feedback and if there’s interest / probability of merging or being added, I would be happy to implement in ord through a PR.
Thanks for reading :)

This would be a great addition to the protocol.
There are massive risks when it comes to token sales but the mechanism described is much more efficient than a project premining and then selling their runes.
It could be worth checking out OrdinalsBot's approach to rune etching/minting. When setting the details of your Rune you can set a price and revenue address.

Also had the idea of adding an asset field, which takes in a RuneID. That way, mint can be priced in other tunes rather than satoshi.
this would have other usecase a too, for example, wrapped runes that represent other assets (like BTC, USD) which can only be minted by a custodian that has the asset that it costs to mint the wrapped rune (because they would be the only ones that have it)
Runes could even be used to stable coin natively on Bitcoin with this mechanism, with institutions (like tether) being the only ones authorized to mint since they are the only holder of the rune described in RuneID (assuming they premined that rune).
TLDR: granular access to a runes mint function could be granted by leveraging the fungibility of the Runes protocol.


Suggestion: attachment ease of use

Evertything makes sense. I thought today that it would be nice to associate the datasheet to a part and behold there is already an attachments area. Perfect.
My suggestion applys to both attaching picture, images, and whatever may be attached via the attachment areas. Allow pulling via hyperlink. Ex:
If I need a datasheet for a part say a resistor it seems like several extra steps to download the pdf, save the pdf, attach the pdf to InvenTree, then delete the local copy of the pdf.
It would be amazing if we could paste the link to the pdf into the attachment area and have it automatically download and link it.

I actually started looking into the pull-via-hyperlink approach (e.g. drag-and-drop an image or link from a browser).
However, two major issues:

The javascript required to do this is really complicated and seems to be vastly different per browser. And there's no guarantee it would remain supported.
It (IMO) could introduce security issues - at the very least this needs to be carefully considered

I think that the addition of drag-and-drop support for file attachment would be really useful, and relatively easy to achieve. The hyperlink stuff requires some further thought - and at this stage would require too much dev time which I think is better spent elsewhere (e.g. key functionality).

LTS is important.
As I am adding parts to my own inventory from suppliers I am often just borrowing their pictures for the part. I have now come to realize they do not have a unique picture for everything. EX: 0805 Thick File Resistors will mainly use just one image. Sometimes there is a unique image per manufacture. Anyway, have you thought about that? Linking 1 image to multiple parts?

@SchrodingersGat Hey, I found a few sites like imgur, github, gitlab, ect. that have drag and drop capabilities. On all of these sites I can drag an image from one webpage and drop it into the drag and drop area... I don't know what that means but I was excited. If you did drag and drop maybe it would just magically allow drag and drop from other sites.
This is my test image: https://www.mouser.com/images/kemet/hd/SMD_MLCC_new_t.jpg
and I just draged it below
![](https://www.mouser.com/images/kemet/hd/SMD_MLCC_new_t.jpg)

@a7hybnj2 I suggest you take a look at InvenTree's python interface: https://github.com/inventree/inventree-python
With a few lines of Python code you can download a picture/file from the supplier's website (wget for instance) and upload it to a specific part on InvenTree (Thumbnail or attachment, you have the options). If you have any issue or question, you can post them here: https://github.com/inventree/inventree-python/issues

This is basically the same request as #1306
I was thinking of an easier method: copy-paste the hyperlink instead of drag-and-drop. @a7hybnj2 do you think that this approach would work for you?

@ivladz That would certainly be an improvement. I was thinking about it from the perspective of not just pulling images from websites. I will often take my own picture of the inventory which help identify it better than some of the generic pictures you find for parts online. I take a batch of photos then view my pictures in thumbnail view to be able to select the correct one. In certain application/websites (like github) you can drag and drop. Like ⇩when your making a comment (that arrow is pointing to the box that you drag them into).

@a7hybnj2 Good point. I think overall several quality of life changes are needed for images/attachments:

Adding via hyperlink (copy paste method)
Drag and drop local file
Take photo from camera directly from webpage
Add photo directly from phone APP (gallery or new from camera)

In short, any one of them brings something to the table.

@ivladz I concur. Also the order you have listed seems like a good order of implementation because it seems to also be easiest to most difficult.￼￼


xy stage- not homing on open

On open xy stage doesn't return to home and sets previous settings as (0, 0)
you can input negative values to return to original home then set as (0, 0) but doesn't go automatically

My mistake. I shut off homing temporarily to speed up open to test code.  I've re-added home on start-up and threaded it so that it does not freeze loading.


Filepath issue on Linux

Unfortunately, the file path that is set isn't compatible with Linux (see picture below). It should be cd /home/.. instead of cd home/..
I'm pretty sure there is a method in the Obsidian API that handles this, but I can't find it just now (I'll update the issue later if I can).

@MichaBrugger you can fork the repo and fix the bug.

@MichaBrugger
https://github.com/yekingyan/obsidian-open-in-other-editor/pull/5
my branch has the fix. I wouldnt mind someone testing this on linux for me. Cuz i only detecting macs right now.


react-native-permissions: NativeModule.RNPermissions is null

react-native-permissions: NativeModule.RNPermissions is null error
im working with android version but i get this error when i emulate using expo.
thanks

I have mocked react-native-permissions in my own jestSetupFile.
I'm getting this error when running tests with jest.

I'm getting this error too when I try to run. I think it has a problem with the podspec file.

Just add
jest.mock('react-native-qrcode-scanner', () => jest.fn());
before your failing import (within your jest test).

that problem is actually because react-native-permissions is not compatible with expo as expo already has their own way to manage permissions by himself natively. For tests, the suggestion above works.
So, I'm closing this.


[stdlib] Generalize array bridge/cast machinery

Now we support casting and bridging to/from [Any], not just [AnyObject]
Note that the typechecker still doesn't allow all the casts we'd like;
see the FIXMEs in test/1_stdlib/ArrayBridge.swift.gyb.

@swift-ci Please test and merge

Sweet! @rjmccall has most of the type checker changes in place, IIRC

@swift-ci Please test and merge


custom marker icon size control

Why there is no option to control custom icon maker size in MarkerOptions changing image size is not good practice?

There's a similar open issue with crowd label, so closing this in favor of https://github.com/flutter/flutter/issues/34657.


Frontier - Add MOBIX token

Token image is already existing
coingecko id need the pool to be created

Scam project. Why not listing?

We are no longer deploying frontier from frontier branch


Article on transition-timing-function doesn't say how users can protect ourselves.

Cross-filed from https://bugzilla.mozilla.org/show_bug.cgi?id=1512826
https://developer.mozilla.org/en-US/docs/Web/CSS/transition-timing-function
Since "ease-in-out" animations are popular, and can hurt, and can trigger severe migraines and severe nausea, it can be important for users to be able to protect ourselves against these. This article only currently discusses how developers can inflict these. If another article discusses how we can protect ourselves, then this article should include a link.

added a11y concerns to https://developer.mozilla.org/en-US/docs/Web/CSS/transition-timing-function page


Change buffer requires refresh

environment
vim: VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Jun 24 2020 23:42:02) macOS version Included patches: 1-1050 Compiled by Homebrew
vim-airline: latest as today
OS: macOS High Sierra 10.13.6 (but also happens under Mojave on MBA)
Have you reproduced with a minimal vimrc: no
What is your airline configuration:
Plugin 'vim-airline/vim-airline'
Plugin 'vim-airline/vim-airline-themes'
" airline cfg options begin here
let g:airline#extensions#tabline#enabled = 1
let g:airline_powerline_fonts = 1
let g:airline#extensions#tabline#formatter = 'default'
let g:airline_theme='durant'
let g:airline#extensions#syntastic#enabled = 1
let airline#extensions#syntastic#error_symbol = 'E:'
let airline#extensions#syntastic#stl_format_err = '%E{[%fe(#%e)]}'
let airline#extensions#syntastic#warning_symbol = 'W:'
let airline#extensions#syntastic#stl_format_warn = '%W{[%fw(#%w)]}'
if you are using terminal:
terminal: iterm2 build 3.3.12
$TERM variable: xterm-256color
color configuration (:set t_Co=256
actual behaviour
When I have a (v|h)split and change buffer, the theme loses all of its colours, and I have to refresh.

Have you tried with a different terminal or the gui?

I have tried with MacOS Terminal and MacVim, both exhibit the same issue. If I toggle between full-screen and back again, airline refreshes and momentarily, it's all good. If I then change buffer either using kbd or mouse, issue returns, text is visible in statusline, but only as plain unformatted text. Let me know if there is anything I can do to help debug this (assuming it is an issue, and not user error...)

This happens on a single :vsp? Can you provide a minimal reproducible example?

I have tried with MacOS Terminal and MacVim, both exhibit the same issue. If I toggle between full-screen and back again, airline refreshes and momentarily, it's all good. If I then change buffer either using kbd or mouse, issue returns, text is visible in statusline, but only as plain unformatted text. Let me know if there is anything I can do to help debug this (assuming it is an issue, and not user error...)

Yes, one single :vsp, for example my .vimrc and a single small perl file. When I start, airline is clearly OK. Soon as I change to the other split, the issue occurs. Here's a vid: https://www.dropbox.com/s/d20hg203muxwafs/Airline issue example video.mov?dl=0 which I hope helps

How did you change to another Window? It wasn't :vsp. Can you reproduce only with a minimal example without any other plugins installed? Perhaps a workaround:
augroup MyAirlineAutogroup
  au!
  au WinEnter * :AirlineRefresh
augroup END

The way I usually changed windows was after a :vsp, and by using either mouse click or l to invoke :bnext. I think from memory I did just proper :bn and :bp to see if that made any difference but it didn't.
I stripped my .vimrc to:
===================================================================
set termguicolors
set mouse=a
set nocompatible              " be iMproved, required
filetype off                  " required
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
Plugin 'VundleVim/Vundle.vim'
Plugin 'vim-airline/vim-airline'
Plugin 'vim-airline/vim-airline-themes'
" airline cfg options begin here
let g:airline#extensions#tabline#enabled = 1
let g:airline_powerline_fonts = 1
let g:airline#extensions#tabline#formatter = 'default'
let g:airline_theme='random'
let g:airline#extensions#syntastic#enabled = 1
let airline#extensions#syntastic#error_symbol = 'E:'
let airline#extensions#syntastic#stl_format_err = '%E{[%fe(#%e)]}'
let airline#extensions#syntastic#warning_symbol = 'W:'
let airline#extensions#syntastic#stl_format_warn = '%W{[%fw(#%w)]}'
set noshowmode
" All of your Plugins must be added before the following line
call vundle#end()            " required
filetype plugin indent on    " required
"
filetype plugin on
filetype indent on
let g:airline_theme='durant'
===================================================================
This resolved the issue for me, works great every time/way I switch, so it looks like the culprit is another plugin or cfg option, will strip back bit by bit and see if I can track it down. For me, if you are content, this can be closed as an issue. Thanks for the workaround suggestion too, will have a go with that.
Many thanks for your help with this one, much appreciated.

I had the same problem, every time I change buffer the status bar appear for half a second and then dissapear, even sometimes this happened when I change mode.
I did something similar like you I took my airline config and put it in the very first line of my .vimrc then when reset vim some warning messages appeared, but the plugins works flawlessly, then I moved the config to it original position and now work perfectly with no warning at all.
I know, its weird but maybe this info helps.

Hi same problem here, I move the config to the top of my .vimrc and some warnings appears but it works flawlessly, then I put the config on it original position and everything work great, even I restart my PC and it keep working great,but after a while the problem came back I try to reproduce the "solution" I just wrote but never worked again.
Your work around doesn't work for me because when I change buffer the airline appear for half a second (or less) but then dissapear.
I have this problem with me from a long time ago, anyway its starting to bothering me that I have to refresh everytime, btw my way to refresh it's with nerdtree or tagbar anything even splits work to restore the status bar.
I also thing is a plug-in conflict because I have the same problem in my laptop (Ubuntu-KDE, Konsole, zhs) than in my desktop computer (Ubunto, Gnome-terminal,bash)
I will try to keep writing, if I discover more.

Hi again, its look like the quickfix window is the problem.
When I open it (no matter how), and return everything its ok, but then if I change buffer the airline disappear.
Also if I enter in insert mode and return to normal mode the airline disappear too.
If I reset the session everything its ok, so the moment I activate the quickfix my session get corrupted so to speak.
In session which I don't invoke the quickfix everything its ok.

Please provide exact reproduction steps

Am 29.07.2020 um 12:58 schrieb Cristo García<EMAIL_ADDRESS>
Hi again, its look like the quickfix window is the problem.
When I open it (no matter how), and return everything its ok, but then if I change buffer the airline disappear.
Also if I enter in insert mode and return to normal mode the airline disappear too.
If I reset the session everything its ok, so the moment I activate the quickfix my session get corrupted so to speak.
In session which I don't invoke the quickfix everything its ok.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

You have at least one of these options activated:
let g:ale_open_list = 1 and let g:ale_set_quicfix = 1
Open the quickfix, close it, and then change buffer with :bp or :bn

So this requires the ale plugin?

Am 30.07.2020 um 00:26 schrieb Cristo García<EMAIL_ADDRESS>
You have at least one of these options activated:
let g:ale_open_list = 1 and let g:ale_set_quicfix = 1
Open the quickfix, close it, and then change buffer with :bp or :bn
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

It looks like, thats why I dont know it this issue must be here or in the ALE's issues

possibly some autocommands need the nested flag. good reproduction steps would help tracking this down.

Does this mean you couldn't reproduce it?
Well I open vim with a minimal vimrc:
vim -Nu minimal <anyfile>

My minimal vimrc is the next:

call plug#begin('~/.vim/plugged')
  Plug 'dense-analysis/ale'

  Plug 'vim-airline/vim-airline'
call plug#end()

" ALE
let g:ale_set_quickfix = 1

packloadall

I suppose there is no problem with vim-plug.
Now open quickfix with :copen and if you want close it with :cclose, edit a new file and now airline is gone.
You can also create a new buffer before open the qickfix and when you already opened the quickfix use :bp, the result its the same.
Now I can refresh the airline, what I usually do its create a new split, the problem its, that from this point it doesn't matter if I open the quicfix or not, everytime I change the buffer I lost the airline.

I cannot reproduce. Does Ale actually need to generate a quickfix buffer? Or should it work with an empty window by just using :copen?
Here is my .vimrc:
set nocp
let g:ale_set_quickfix=1

I have cloned ale and vim-airline (and the themes repository) below ~/.vim/pack/minpac/start, so using packages.
Going through your mentioned steps, I am basically running the following commands:
vim vim-airline/README.md -c ':copen' -c ':cclose' -c ':e ~/.vim/vimrc'

The quickfix doesnt need to be populated.
I did the next attempts:

2 cpp files. Result: Bug.
1 .yaml and one Makefile. Bug
2 .txt files. Everything its ok and it work.
My personal .vimrc and a Readme.md. Bug

Everything with my minimal vimrc.
Please try with different projects you got over there, if you can't still reproduce it give me a couple of days so I can give you a good report.

exact same steps using putty (with and without tmux) and $TERM=xterm-256color does not show the problem for me

Well, I can work without the quickfix, for my part you can close the issue.

can't you work around it with an additional bufenter autocommand?

How did you change to another Window? It wasn't :vsp. Can you reproduce only with a minimal example without any other plugins installed? Perhaps a workaround:
augroup MyAirlineAutogroup
  au!
  au WinEnter * :AirlineRefresh
augroup END

The thing is I tried your workaround, but as I said in one of these comments the airline work for a very briefly moment and then dissapear so if I manage to create a workaround it would be too shabby.
Anyway I had this problem since a long time ago, and I decided to talk about it when I saw other people could have the same problem (I didn't even opened this issue), but it look is not the case, and I already change my workflow and get used to it, so there is no problem, so in my opinion this look too complicated and not worthy, but of course you choose what to do.
Thanks you anyway.

Yeah, this is sometimes disappointing that terminals are not deterministic (?). Or perhaps this is a mac issue (for which I cannot test). I don't really know why this works for some and doesn't work for others.


Add support for credential_process from profiles

Motivation and Context

Fixes: https://github.com/awslabs/aws-sdk-rust/issues/261
Description
Starting to add support for credential_process in the default credentials chain. I'm just pushing this up as a draft to get early feedback before continuing implementation.
Testing

I tested this manually by using patching it into http://github.com/vectordotdev/vector but will follow through with tests here.
Checklist

[x] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[x] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Ah, also, I integrated this into the profile file provider rather than creating a new one that is part of the default chain. I think it makes sense as part of the profile file provider (this is how rusoto modeled it too) rather than a separate provider that also tries to load profile files, but open to opinions here.

Thanks for taking a look @rcoh ! I pushed some more changes and think this is ready for full review.

any update on this functionality?

any update on this functionality?

I'll try and get this wrapped up and merged this week.

Thank you for taking this across the line @jdisanti ! Thanks to @rcoh @Velfi for the review too. Apologies that I didn't get to circle back to address the comments yet. I'm happy to see this merged in though 😄


Login page title is "My UI Tasks"

cc @dclarizio @h-kataria

@Fryguy What is this?  I don't see this in my env . . . you have a screen shot?  Does it happen all the time or just after you visit that page and logout?

well now I'm not sure...I logged out and now it says "ManageIQ: Dashboard" (which is arguably wrong also, because I'm not on the Dashboard.  Restarted server and now it says "ManageIQ: Dashboard".  I've seen this numerous times though, so maybe it's session related or something.  I'll close for now and reopen if I see it again.

@Fryguy i am reopening this, i was able to recreate this issue, if i am on My Tasks screen and logout, login screen shows ManageIQ: My UI Tasks as title.

Oh phew...thought I was losing it :)

@Fryguy doesn't mean you're not losing it, just not cuz of this.

@dclarizio should the title on Login screen say ManageIQ: Login instead of ManageIQ: Dashboard

@h-kataria Yes, xxx: Login makes sense.

BZ ticket https://bugzilla.redhat.com/show_bug.cgi?id=1222667

@h-kataria Is this closed, then?

@fryguy yes this is fixed.


build(nodejs): update version to >=12.0.0

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

v3.8.3
GitHub release

Your semantic-release bot :package::rocket:


works fun only for gmail

Not yet.
<EMAIL_ADDRESS>equivalent<EMAIL_ADDRESS>but check result false in the first case.

yes, for me also, only it is working with gmail not for other providers.

So is it gmail checker or email checker?

What email providers are supported doesn't depend from this library's code. It depends on whether the email service answers the HELO request or not.

What email providers are supported doesn't depend from this library's code. It depends on whether the email service answers the HELO request or not.

Already const EMAIL_REGEX is bad, because the<EMAIL_ADDRESS>is a valid email string, even if no one is using it yet.  Since the first test is a string test with EMAIL_REGEX constant, we do not get further tests.

@nova76 the + sign issue is already mentioned in https://github.com/pensierinmusica/email-check/issues/11


Delete package.json from unmonitored branch

Integration settings -> Pull request status checks -> Only fail when the PR is adding a dependency with issues


When I send a POST request to "/api/request" with files:

It seems to be useful to test requests with files included.
I use https://github.com/danielm/uploader in my projects.
Dmuploader posts files to my REST API ex. /api/order/attachment-file
API responses success or false information.
Very precursory scenario format ...
Scenario: Privileged user attaches the file to the order
When I send a POST request to "/api/order/attachment-file" with files:
"""
        {
            "files" : [{"fixtures/test-file1.png", "fixtures/test-file2.png"}],
        }
 """
Then the response status code should be 200
And the response should be in JSON
And the JSON should be equal to:
      """
        {
          "success": true,
          "data":
              {
                  "success_message": "Files attached succesfully"
              }
      }
      """

What do you think?

Why not !
But I just guess we should use table notation instead of string notation, like
When I send a POST request to "/api/order/attachment-file" with files attached:
    | files                   |
    | fixtures/test-file1.png |
    | fixtures/test-file2.png |

It would be more simple to use and to code !

I wonder, how to make a mixture with posting files and body using existing "I send a POST request to "echo" with body:". ?
Wnen I send a POST request to "echo" with body:
"""
        {
            "username" : "pablo",
            "password": "money"
        }
        """
     with files attached:
    | files                   |
    | fixtures/test-file1.png |
    | fixtures/test-file2.png |

Not sure if Behat syntax accepts it. Did you test it ?

I haven't test it yet.
I'm looking for notation with only one "send request to".
It would be very ackward if I would sent it as following...
Wnen I send a POST request to "echo" with body:
"""
 {
     "username" : "pablo",
     "password": "money"
 }
"""
And When I send a POST request to "echo" with files attached:
 | files                   |
 | fixtures/test-file1.png |
 | fixtures/test-file2.png |

We can consider this:
Wnen I send a POST request to "echo" with:
    | body                   | {.........}
    | files[0]                | ....
    | files[1]                | ....

Have anyone checked if and how it is implemented at other extensions?

The only option we get I guess is to have 2 steps with a first one that push state (like headers)

@tyx could you write an example?

Given I attach files:
 | files                   |
 | fixtures/test-file1.png |
 | fixtures/test-file2.png |
When I send a POST request to "echo":
"""
 {
     "username" : "pablo",
     "password": "money"
 }
"""

I'm working on the issue today.
The problem is to send payload with json and files(multipart/form-data) together.
When I'm sending payload using "Content-Type: multipart/form-data; boundary=..." header, the json payload does not exists in a request body - "Wnen I send a POST request to "echo" with body:".

https://stackoverflow.com/questions/48009305/how-to-post-http-request-with-json-and-files-payload

indeed ! And it makes sense as we should use a different content type.
I just had a look and in one app we used base_64 workaround.

I think that If files are included, json attributes should be placed in multipart/form-data body.
But a big quession is if most restful frameworks will handle this like ZendFramework does
https://github.com/zendframework/zend-mvc/blob/master/src/Controller/AbstractRestfulController.php#L471

But how could you guess you add json in body without correct content type?


dimensional order of operations / moderately complex formulas

I suspect this may be a difficult fix for the fixed-dimension-type design model, but I like the intent of this module, and ran into the problem pretty quickly, so I'll let you know.
I could also be wrong --- and it's an easy fix --- or wrong in that I'm using the module incorrectly.
The problem seems to be related to order of operations and comparison to the pre-defined types. It seems like if any part of an operation doesn't match one of the predefined types --- even if the dimensions will later cancel to match ---  the units get dropped and calculation proceeds without units.
Impractical illustrative simple test case
my $a = 1kg;
my $b = 1kg;
my $c = ♎<1 m/s^2>;
this works fine:
my $d = $a * $c;
say $d.in('N');
output: 1N
something like this should also work, but it doesn't. Seems to choke on the
idea of a square kilogram before the dimension-canceling takes place.
my $e = $a * $b * $c / $b ;
say $e.in('N');
output: No such symbol in method multiply at (Physics::Measure) line 254
say $e;
output: 1
Practical test case
Calculation of hydraulic power
my Flow $Q = Flow.new( value => 1, units => 'm^3/s' );
my Length $h = Length.new( value => 10, units => 'm' );
my Density $d = Density.new( value => 1000, units => 'kg/m^3' );
my Acceleration $g = Acceleration.new( value => 9.81, units => 'm/s^2' );
my $P = $Q × $h × $d × $g;
say $P;
output: 98100
say $P.in( 'kW' );
output: No such method 'in' for invocant of type 'Rat'.`
Other program examples
gnu units, udunits2, orpie, hp calculators can usually do this type of thing, as long as the end result is dimensionally consistent with whatever unit you're trying to express/convert after the operation is complete.
gnu units
units -t -v "(1 m^3/s)*(10 m)*(1000 kg/m^3)*(9.81 m/s^2)" kW
output: 9.81 kW
mixed-unit systems also work fine, as long as result is consistent
units -t -v "(100 gallons/min)*(10 ft)*(9.81 m/s^2)*(62.4 lb/ft^3)" horsepower
output: 0.25286413 horsepower
udunits2
udunits2 -H "(1 m^3/s)*(10 m)*(1000 kg/m^3)*(9.81 m/s^2)" -W kW
output: 9.81 kW

@td002 - many thanks for trying this module and for the feedback. I did indeed have an early version of the code that synthesized "artificial" units for intermediate results where there is no predefined unit to be found, this did not make it to the current release
I agree that this is a worthwhile new feature and therefore will leave this issue open and aim to implement in the next major release
I am struggling a bit with tuits on this right now, so please feel welcome to make the change yourself and submit a PR
Since you have knowledge of the wider range of unit libraries, perhaps I should consider refactoring this module around a more widely used lib - which one would you say is the best target (mainly for linux type systems for now)?

In terms of a well-tested pure unit library that has an API that could be accessed through NativeCall, AFAIK there's only UDUNITS2.
GNU Units is also very good, but no official API.
I'm a practicing civil engineer and haven't the skill to make even a partial NativeCall port of this system ---  but I do use the XCAS/GIAC computer algebra system all the time. GIAC has an API. It also supports a wide array of units that you can tag to values.
In a lot of ways unit management is a problem for a CAS. So with XCAS/GIAC you can supply an equation, and then substitute values with whatever units you have without the need for conversion, then solve for the missing variable on either side of the equation, and get the answer in whatever (dimensionally consistent) unit you need. e.g.
hyd_power := P = Q*h*d*g
comment("note solve returns a list, have to use the [0] subscript")
convert( solve( subst( hyd_power, [ Q = 100_(galUS/mn), h = 10_ft, g = 9.81_(m/s^2), d = 62.4_(lb/ft^3) ] ), P )[0], _W )
output:<PHONE_NUMBER>13_W
comment( "now solve for head in feet")
convert( solve( subst( hyd_power, [ Q = 100_(galUS/mn), P =<PHONE_NUMBER>_W, g = 9.81_(m/s^2), d = 62.4_(lb/ft^3) ] ), h )[0], _ft )
output: 9.99999999933_ft
There is a python interface for GIAC. If I had the time to try to understand how they make the bridge from python  variables to giac variables, and I also somehow suddenly acquired the skill to do that in Raku, I'd give it a go with NativeCall.


Revert naming convention from template to component

Part of #386

821 file changes. Turning out to be largest changes in any PR in OSS ever :D


Update dbt + data mesh quickstart guide to use provision app

One major piece of feedback we received about our Quickstart Guide is how many manual steps one has to do in order to use the guide. And so, this change removes the manual steps and replaces it with this Streamlit app to make the participant experience much smoother.
⚠ Please merge this PR in ASAP! This change is required for a Snowflake webinar on 30 October. We plan to do the dry run on 28 October.

cc @iamontheinet @dpguthrie


Cartographer worse than Gmapping

Launch file, Bag ,Configuration is here.
I tested the same bag file in gmapping and cartographer.  The two results are listed as follows,
Cartographer,

Just like I mentioned in #190 , the loop closure detection did not work.
Gmapping(which is very similar to the real environment),

Since I had already combined imu data into odom(/odometry/filtered in the ros bag), I did not provide the raw imu data(/costly_imu in the ros bag) to both slam.
Would you @SirVer  please tell what should I do next?
Thanks very much.

Your statement is that default settings are better than default settings of Cartographer? That seems unsurprising. What did you try to tune already? Have you tried disabling your IMU/odom merged topic and only use scan matching as I suggested in https://github.com/googlecartographer/cartographer_ros/issues/249?

Thanks for replying so fast.
I tracked the odom trajectory and it was accurate in most time.
After comparing the mapping progress between Gmapping and Cartographer, I found the problem was caused by what you alerted me in #190 ,i.e. my laser is not always level, it is fixed on the robot and the floor has tiny pitches in the following part,

Is that possible to fix this by tunning any parameters (e.g. increasing confidence of odom)?

Is that possible to fix this by tunning any parameters (e.g. increasing confidence of odom)?

You can tune of course, but it is hard to say if this is fixable from just the image. If your sensors are strongly disagreeing, there will be no real way to fix the issue. If you trust odom so significanlty more than the laser and it has a bias (which it likely has), you will mess up other parts of the map.
I tried looking at your bag and configuration. But I do not have access to the drive elements. Also, make sure that your bag already contains the always-level laser changes.

Sorry, I forgot to enable others to access the drive. It can be downloaded now.

I am running into a similar problem where the loop closure is poor in indoor environments. I am using a laser, odom, and an IMU. The floor I am mapping also has small bumps and the robot does not always stay level. If I am providing IMU data should cartographer handle this okay? Or do I need to do some pre-processing on the laser data?

@alexbaucom17 It probably also just comes down to tuning. Sometimes IMU jumps over bumps are extreme and throw off the SLAM - then it might be better to disable IMU and use correlative scan (see closed issues labeled tuning for ideas.)

Got it. I will do some more testing and tuning then. Thanks.

@nangong I found some time to look at this now. Unsorted comments:

Your launch file refers to demo_husky.lua which you did not include. I used demo_myrobot.lua instead.
You also didn't include demo_myrobot.rviz.
Your configuration files have gone out of sync with HEAD, i.e. some options have vanished or have been renamed.
You did not include a URDF and your bag contains TF information from another SLAM run - so I could not use that either. This means I cannot spatially relate your odom or IMU data to your laser data - I therefore disabled both odometry and IMU.

It would have been nicer to have a proper git fork with all files so I could reproduce on exactly your conditions - you it pretty hard for me to help you :(.

Your outdoor (?) area that connects the two rooms the robot drives into seems to violate the 2D assumptions (i.e. flat floor, mostly orthogonal objects). Even to me as a human the laser clouds seem very different on the way back than they look on the way there.
Sometimes your laser only sees very far away objects. Your configuration before threw away everything farther away than 30m, so there was no information for SLAM.
With your laser's poor angular resolution and high noise you cannot expect to SLAM at 5cm if your objects are so far away - your points visible jump by 15-30cm. I set the resolution to 15cm and that seemed to be sufficient to capture this.

My final settings drift by ~1.5m after the long loop (comping back to the first building) which is fixed by loop closure. Image attached below. I think adding IMU + odometry might increase performance (i.e. require less CPU) since you might get away without the correlative_scan_matcher, but this will probably  not improve quality.
The middle part of the map is still wrong. But I cannot blame SLAM there, the data is not looking consistent as mentioned above. This claim is supported that after loop closure both building parts are very good, but the middle part is still inconsistent - this is really only possible if the data is inconsistent there because everything is the same optimization problem.
My changes are in this branch in the last few commits.

Got it..
Thanks very much for your help... : )

@nangong  @SirVer I am using RPlidar which have a 6m range only .. and i am getting visual odometry from ZED camera, and the odom is very accurate.. how can i increase the odom confidence as you were saying in the upper comments ?

@ahmdsamy They showed me how to change odometry confidence in this issue. You can look at the comments in the config file of my fork.


Don't re-scope the arguments of almost all CSS pseudo-classes

Problem:
Follow on from #4604
Some CSS pseudo-classes should have their arguments re-scoped, some should not.
Fix:
Prevent traversal of all CSS pseudo-class arguments, with the explicit exception of :is and :where, using css-tree's walk.skip. This means that we would rescope the arguments of :is(h1) to :is(#canvas-container h1), but not the arguments of :has(h1), or the arguments of a pseudo-class nested inside :has(:is(h1))

Closing in favour of #4613


Add export const uuid = v4

Is your feature request related to a problem? Please describe.
For all libraries I use I don't have to write imports manually in WebStorm. All I have to do just write code like const x = doSomething<ALT + Enter> and it shows me hint Add "import doSomething from XXX'?
With export const v4 I have to write manually import as in manual import { v4 as uuidv4 } from 'uuid';. It's frustrating.
Describe the solution you'd like
Change v4() name or add one more export as export const uuid4. So after import it would be very clear what is it about and why.
Describe alternatives you've considered

Writing manually :(
Adding somewhere utils/uuid.ts with

export { v4 as uuid4} from 'uuid'

Type const x = v4<ALT + Enter> instead?

@broofa then I would have hard to understand function name usage in code, which doesn't make any sense unless developer checks the import. It makes code harder to read and support.
Also since your recommendation in readme is import v4 as uuid you kinda realize it too.
So why not implement naming that would be more developer-friendly by default?

Thank you for raising this @syabro
I do have to admit that personally I'm also not super happy with the current named exports. They have mostly historic reasons:

When this library was initially released in 2011 for Node.js there was no notion of named exports and not even object destructuring, and in addition this library only supported v4 UUIDs, so the API surface was:
var uuid = require('uuid');
uuid();

Then at some point additional algorithms were implemented and the default export merely became an alias for uuid.v4(). IIRC that was still before Node.js 6.0 so object destructuring assignments like const { v4 } = require('uuid') still weren't really a thing:
var uuid = require('uuid');
uuid.v4();
uuid.v1();
// etc…

Then uuid started to be used in browsers as well and treeshaking became an issue, hence we started encouraging deep imports. But since these deep imported files still only exported a default export that local import names weren't really an issue either:
var uuidv4 = require('uuid/v4');
uuidv4();

So only with the migration to ECMAScript Modules (ESM) and named exports the import names became somewhat predetermined.
import { v4 } from 'uuid';
v4();

The reason we kept the v4, v1 etc. names when moving to ESM was the idea of keeping the API change as small as possible.
I do admit though that in the ESM world we live in by now requiring aliasing when importing one of the algorithms is really not ideal and at least I didn't really consider that aspect enough when doing the migration to ESM.
All that said I believe this might be a good chance to get rid of the confusing v1, v4, … naming entirely. After all it seems to be causing more harm than good, see discussions in https://github.com/tc39/proposal-uuid/issues/3 and this analysis.
So instead of sticking to these names I suggest to export the algorithms with more telling names, like discussed in https://github.com/tc39/proposal-uuid/issues/3#issuecomment-544190284. Something like:
v1() -> timeUUID()
v4() -> randomUUID()
v3() -> md5UUID() // (or nameUUID()?)
v5() -> sha1UUID() // (or nameUUID()?)

An alternative would obviously be to export uuidv4() instead of v4(), this naming would be similar to what Python does
Thoughts?

@ctavan thanks for clarifying.
randomUUId() / uuidv4() would be great since it's after reading it's obvious what function does.
PS As soon it's not default export I will be happy to see it :)

"There are only two hard things in Computer Science: cache invalidation and naming things"
I suspect this is exactly what Phil Karlton had in mind with the latter part of that quote.   There's no clearly "right" naming scheme here.  Developers are going to have to remember what the name is regardless of what we choose.  The current scheme at least has the advantage of not having to remember how UUID/uuid/Uuid/UUId is capitalized. 😝
I don't see the "more harm than good" you mention of the current scheme.... at least not to an extent that warrants (again) breaking the API for 1M's of users.  My vote?  Stick with the current scheme until there's broader consensus on what the names should be.  I.e. Wait until the standard uuid work settles out, and then go with whatever API scheme is decided there.

"There are only two hard things in Computer Science: cache invalidation and naming things"

I've always known that one as:
"There are only two hard things in Computer Science: cache invalidation, naming things, and off by one errors"
😄

Wait until the standard uuid work settles out, and then go with whatever API scheme is decided there.

I'm leaning towards this also, even if I also feel the frustration every time I import the function. If we change to something like timeUUID() now, and then TC39 settles on another name, we'll probably want to change again.

This is great PR, for god sake why I would always name function before importing it. Just name it uuidv4. uuidStringify and so on.

Angular compilation warning: application.service.ts depends on 'uuid'. CommonJS or AMD dependencies can cause optimization bailouts. For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

Angular compilation warning: application.service.ts depends on 'uuid'. CommonJS or AMD dependencies can cause optimization bailouts. For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

@gizm0bill This is not related issue. Please create new one if there is a bug.

@gizm0bill This is not related issue. Please create new one if there is a bug.

@gizm0bill This is not related issue. Please create new one if there is a bug.

Actually it is...

@gizm0bill This is not related issue. Please create new one if there is a bug.

Actually it is...

@gizm0bill From the warning it looks like you use some very old version without es modules support.

@gizm0bill From the warning it looks like you use some very old version without es modules support.

FYI, you can just do this:
import * as uuid from 'uuid';

uuid.v4()

I quite like this syntax, actually. I think it should be recommended in the README. The main caveat is that treeshaking generally doesn't work when importing the whole namespace. I don't personally feel that matters too much here, though.

@sholladay rollup and webpack 5 does not have the problem with namespaces. They are treated as named imports unless used as value.

@sholladay writing it manually is the same as import {v4 as uuidv4} from 'uuid'

Sort of, @syabro. It's the same in that I showed an alternative way to use the v4 function, via namespace import instead of as a named import. It's the same function, but accessed a different way. In my opinion, the namespace import is much less ugly.
It could be simplified further by attaching the functions as properties of the default export object. Then you'd be able to do this:
import uuid from 'uuid';

uuid.v4();

That would be the cleanest syntax.


fix: CLI docs

The CLI docs were outdated. I've updated them by running pixi run generate-cli-docs.
No idea why so much changed here.

Probably because @ruben-arts fork of clap-markdown was commented out in the Cargo.toml. Unfortunately I don't remember what the fork did...

@wolfv I added the patch again


Update documentation for usage maintenance.sh

I did the minimal change here.  It didn't really seem like the place to more fully document maintenance.sh

agreed. thanks!


Feature Request: Filter by IP Address

Hi,
The filter tool is very useful, it would be improved further if we could also filter hosts by IP address and/or system hostname (as set on the client) if possible.

I'll look into it.
You can currently filter by IP in the command palette if you just want to jump to a system.


failure to extract accelerometer features

mhealthtools::accelerometer_features(accelerometer_data)
fails because there is no way for the user to specify the extract parameter, which gets used downstream in sensor_features, and we are not doing a check to see if it is NULL and then set the default parameters
I assume similar problem is there in tremor, walk and gyroscope as the logic is the same
@philerooski

Even the simple mhealthtools::get_walk_features(datAccel, datGyro) fails because the user friendly version (plug and play) should have a default setting. Interested parties can go and research further in the documentation to see how to use the modular functionality of the package, because as is the package has that modular functionality, we should impose it on the users from the start.


COM-50: Run Angular build on all commits in CircleCI

Previously this only ran when we were ready to actually publish to NPM. Running for all commits will help avoid merging anything that breaks the angular build

This failed and then passed on the same commit. I think what's happening is that the angular project doesn't get built with the local version of components inside the repo, it gets built with whatever version was most recently published to npm (uses the version specifier *. So if particularly if we last published a version with stencil v3 and then have a commit with v2, or vice-versa, this is likely to fail.
On main it should be fixable by just running the build a second time.


RequirementMachine: Fix some problems in concrete contraction and a couple of other issues

@swift-ci Please smoke test


Unraid problem 404

Hello, I am using an UNRAID server on which I have installed "epicgames-freegames-node" from the UNRAID store.
I have modified and pasted the "config.json" file in the right place.
I don't know much about it, and I tried to shorten the JSON file as much as possible because I didn't need all the other notifications.
So, I'm not sure if the file is correct.

{
"runOnStartup": true,
"cronSchedule": "0 0,6,12,18 * * *",
"logLevel": "info",
"webPortalConfig": {
"baseUrl": "https://store.epicgames.com",
},
"accounts": [
{
"email"<EMAIL_ADDRESS>},
],
"notifiers": [
// You may configure as many of any notifier as needed
// Here are some examples of each type
{
"type": "email",
"smtpHost": "smtp.free.fr",
"smtpPort": 587,
"emailSenderAddress"<EMAIL_ADDRESS>"emailSenderName": "Epic Games Captchas",
"emailRecipientAddress"<EMAIL_ADDRESS>"secure": false,
"auth": {
"user"<EMAIL_ADDRESS>"pass": "xxxxxx",
},
},
],
}

When I start Docker, I receive an email with a link. When I click on the link, I see this image.

Avez-vous supprimé des choses importantes dans le fichier .json qui pourraient causer cette erreur ?

Make sure base url is pointing at your server and port for the container.  Like mine.  Various parts redacted appropriately except my base url.  It's internal anyways.
{
  "cronSchedule": "0 0,6,12,18 * * *",
  "blacklistedGames": [],
  "searchStrategy": "all",
  "runOnStartup": false,
  "runOnce": false,
  "timezone": "America/New_York",
  "intervalTime": 60,
  "accountConcurrency": 3,
  "logLevel": "info",
  "testNotifiers": false,
  "skipVersionCheck": false,
  "browserLaunchTimeout": 15,
  "browserLaunchRetryAttempts": 5,
  "browserNavigationTimeout": 30000,
  "deviceAuthClientId": "REDACTED",
  "deviceAuthSecret": "REDACTED",
  "deviceAuthPollRateSeconds": 10,
  "webPortalConfig": {
    "baseUrl": "http://<IP_ADDRESS>:3055"
  },
  "errorsDir": "config/errors",
  "accounts": [
    {
      "email"<EMAIL_ADDRESS>    }
  ],
  "notifiers": [
    {
      "type": "discord",
      "webhookUrl": "https://discord.com/api/webhooks/0/xxx"
    }
  ]
}

Make sure base url is pointing at your server and port for the container. Like mine. Various parts redacted appropriately except my base url. It's internal anyways.
{
  "cronSchedule": "0 0,6,12,18 * * *",
  "blacklistedGames": [],
  "searchStrategy": "all",
  "runOnStartup": false,
  "runOnce": false,
  "timezone": "America/New_York",
  "intervalTime": 60,
  "accountConcurrency": 3,
  "logLevel": "info",
  "testNotifiers": false,
  "skipVersionCheck": false,
  "browserLaunchTimeout": 15,
  "browserLaunchRetryAttempts": 5,
  "browserNavigationTimeout": 30000,
  "deviceAuthClientId": "REDACTED",
  "deviceAuthSecret": "REDACTED",
  "deviceAuthPollRateSeconds": 10,
  "webPortalConfig": {
    "baseUrl": "http://<IP_ADDRESS>:3055"
  },
  "errorsDir": "config/errors",
  "accounts": [
    {
      "email"<EMAIL_ADDRESS>    }
  ],
  "notifiers": [
    {
      "type": "discord",
      "webhookUrl": "https://discord.com/api/webhooks/0/xxx"
    }
  ]
}

Thank you very much, you’ve helped me out! I no longer have any problems.


Namespace/Name are reverted to defaults when version is selected

Namespace resets when chart changes version (because of page refresh). People work from left to right, so it’s unexpected that a value already chosen would change back to something else. Chart version should be on its own line in top left corner, and perhaps with nothing selected - no fields can be edited until the user chooses a version
Since version is on the top row, the expectation is the user would pick version before picking namespace/name. Some charts require specific namespaces or names for the chart being deployed. Namespace could be preserved, name is set to the name of the chart so you wouldn't want it to be.
The defaults should be less dependent upon the version, so we need a UX story that would allow admins to pick a version without selecting a namespace. Perhaps this could be addressed with a move of the button from the right to the left.

We will be improving the overall UX in 2.6, so will address this then

@richard-cox This is related to the Helm install process - I think with the changes you have made, this is no longer an issue, since you can't change the version once you enter the install stepper

Correct (ish), there's no where these two overlap fields overlap.
For new apps... Version is selected in the chart detail page and namespace set in step 1 of the install wizard
For existing apps.. Version can be updated in step 1 of the install wizard... but namespace cannot be updated for existing apps


Something wrong about accuracy.

Dear author
Hello, I am currently reproducing the relevant experiments in your paper. According to the guidance information on the README, when running the file of "pipeline_PlanEnc.sh" , the accuracy is always zero. I have checked a lot of relevant information, but I still can't find a solution. May I ask if this effect is caused by the omission of some important steps.
The following is the relevant information in the period of running:
Loading train dataset from data/rnn_notdelex_exp.train.1.pt, number of examples: 18071
Epoch  5,    50/  283; acc:   0.00; ppl:   3.76; xent:   1.32; 20891 src tok/s; 16978 tgt tok/s;      5 s elapsed
Epoch  5,   100/  283; acc:   0.00; ppl:   3.72; xent:   1.31; 20883 src tok/s; 17391 tgt tok/s;      9 s elapsed
Epoch  5,   150/  283; acc:   0.00; ppl:   3.74; xent:   1.32; 21349 src tok/s; 17339 tgt tok/s;     14 s elapsed
Epoch  5,   200/  283; acc:   0.00; ppl:   3.43; xent:   1.23; 20920 src tok/s; 17546 tgt tok/s;     18 s elapsed
Epoch  5,   250/  283; acc:   0.00; ppl:   3.36; xent:   1.21; 20454 src tok/s; 17024 tgt tok/s;     23 s elapsed
Train perplexity: 3.58627
Train accuracy: 0.0000
Loading valid dataset from data/rnn_notdelex_exp.valid.1.pt, number of examples: 2262
Validation perplexity: 3.36388
Validation accuracy: 0.0000
Loading train dataset from data/rnn_notdelex_exp.train.1.pt, number of examples: 18071
Epoch  6,    50/  283; acc:   0.00; ppl:   3.25; xent:   1.18; 2007 src tok/s; 1631 tgt tok/s;     48 s elapsed
Epoch  6,   100/  283; acc:   0.00; ppl:   3.25; xent:   1.18; 1344 src tok/s; 1119 tgt tok/s;    118 s elapsed
Epoch  6,   150/  283; acc:   0.00; ppl:   3.24; xent:   1.18; 1388 src tok/s; 1127 tgt tok/s;    189 s elapsed
Epoch  6,   200/  283; acc:   0.00; ppl:   3.08; xent:   1.12; 1348 src tok/s; 1131 tgt tok/s;    257 s elapsed
Epoch  6,   250/  283; acc:   0.00; ppl:   3.05; xent:   1.12; 1317 src tok/s; 1096 tgt tok/s;    326 s elapsed
Train perplexity: 3.17113
Train accuracy: 0.0000
Loading valid dataset from data/rnn_notdelex_exp.valid.1.pt, number of examples: 2262
Validation perplexity: 3.08522
Validation accuracy: 0.0000
I am looking forward to your reply at your earliest convenience. Thank you very much.

Hello, thanks for your interest in our work. I do not remember the details of the log, but it seems the ppl is decreasing as expected, indicating that the decoder is becoming better and better. The unexpected acc value might be caused by a bug from the evaluation script. I guess you can safely ignore that. We do not use acc to evaluate the quality of a decoder.

Dear @zhaochaocs
Thank you very much for your timely reply.  Can the problem of "acc=0"  be solved? In addition, does this issue affect the final model that generated during the training process?

Yes, it can be solved once you've decided how to evaluate the accuracy of the generated texts. One straightforward way is to check the token overlap between the predicted text and the reference, but I don't think it can give us better guidance than ppl. I don't think it will affect the final model.

Dear @zhaochaocs
Thank you very much for your kind help. I will have a try on this experiment.


[Edison Arduino] Illegal arguments for construction of _exports_Pwm

A related discussion here: https://communities.intel.com/thread/60203
For some reasons it's impossible to use any of the pins as PWM, trying to use servo/pwm will leads to the same error message. However, the same code runs fine a day after.
root@edison:~# node

var m = require("mraa")
undefined
var p = new m.Pwm(3)
Error: Illegal arguments for construction of _exports_Pwm
at repl:1:9
at REPLServer.self.eval (repl.js:110:21)
at repl.js:249:20
at REPLServer.self.eval (repl.js:122:7)
at Interface. (repl.js:239:12)
at Interface.EventEmitter.emit (events.js:95:17)
at Interface._onLine (readline.js:202:10)
at Interface._line (readline.js:531:8)
at Interface._ttyWrite (readline.js:760:14)
at ReadStream.onkeypress (readline.js:99:10)

this works here, could you do a m.getVersion() to give me an idea of the version/platform you are using?

It's v0.6.1
Now the problem happens when I'm using the Edison mini-breakout board. The same Edison and code would work on a Arduino extension board(which had the same problem previously, but disappeared after night). I've tried with 2 mini-breakouts.

So on a minibreakout board pin 3 does nothing, it's J17-4, see docs here http://iotdk.intel.com/docs/master/mraa/edison.html - you can also use the gpio.c sample from the examples to list all the pins and their capabilities, that can be helpful.

OK thanks! It's quite confusing for me though, when the same pin number refers to different hardware pins when using different platforms. It would also be better for hardware migration, if I don't have to change the values in my code.
Now it makes sense for the issue on Arduino module. It's possible that I deleted some journey files(to free up space, a bug in Edison makes it so each time one re-flashes the system the journey becomes larger, ending up to hundreds of Mbs) making Edison forgot it was on the Arduino module.

Yes, it's not as simple as I'd like, however there's not really a good way of doing this, this is why there is getPlatformType and you can change your pins depending on your hardware.
If you look at types.h you can use the MRAA_INTEL_EDISON_GPxx enum which means you could use the kernel GPIO numbering (or the MRAA_INTEL_EDISON_MINIBOARD_Jxx_yy miniboard enum if you wished).


--trace_model performance

Hi, I have tested using the --trace_model mode on a small batch of sequences of the same length; I get an 80s tracing time followed by 20s inference for each sequence. If I just fold them without --trace_model it takes 18-19s for inference of each. Am I doing something wrong? There doesn't seem to be much documentation about this feature.

This feature is under active development and will be improving soon, but you should still see a speedup using the current version. Which version of torch are you using? To get the most out of tracing it's important that you use torch >= 1.12.0.

Thanks, I'm using 1.12.0+cu116.

What kind of GPU?

I'm using an RTX 3060. I just placed the separate fasta files in the input directory and ran run_pretrained_openfold.py on that directory with --trace_model enabled. Is there anything else I need to do? I also had --use_precomputed_alignments enabled, btw. Thanks for the help.

Sit tight until I upload the new version of tracing (should be fairly soon).

Sure thing, thanks!

Hi @gahdritz just curious, any updates on this? Is the trace_model usable now?

Whoops forgot to close this. It should work fine now, especially for shorter proteins (< 1000 residues).


Add zsh completions for fzf flags

The fzf executable itself did not have any completions so I've added these based on the man page. Some descriptions have been abbreviated to maintain short lines.
The colorscheme flags needs further work to get proper completion of all its options working. Currently it only works to complete the base scheme. I'm still learning zsh completion so I wasn't able to figure it out yet.
This script could be installed into the $fpath on install or manually moved to a completion directory by the user. I can try an auto install script if this looks good or add the installation steps into the README.

Even my grandma's knitting is more complicated than this PR.

@EnginnerDog is that for or against the validity of the PR?


The WMS boundary is missing again because the GIS server is offline

Change the WMS from GIS to Cube.

The new URL is used and the boundary is back online now.


Add static assertions for iterator concept satisfaction of code unit view iterator

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

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
468

Relevant Lines:
468

💛  - Coveralls


ImgeSet / FactSet / Code Clean-Up

ImageSet added
FactSet added
UI tweaked to better align with the rest of platforms' feel and look.
Code cleanup (Refactored) -- work in progress

@nesalang,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Unknown separator type when using where in particular way

Hi there,
I have a redis collection, I want to write where like regular Linq:
var roomBoxes = RedisConnectionProvider.RedisCollection<RoomBoxCacheDto>(); roomBoxes = roomBoxes.Where(p => p.PersonCapacity+ p.ExtraPersonCapacity >= roomFilter.PersonCount.Value);
but it throws an "Unknown separator type" exception when I want to use .ToList() .
I tried to use an aggregation set for solving this issue and it worked, but I have many filters in my search that I don't need to use an aggregation set.
How can I write this query on rediscollection?
Thank you

@imansafari1991 There may be other answers out there, but the way I would do this would be to create an indexed computed property on your model and then query that.
i.e. public int TotalCapacity => PersonCapacity+ExtraPersonCapacity;
Then query like you are expecting, just against the computed field Where(p=>p.TotalCapacity <= roomFilter.PersonCount.Value)

What a great trick, this worked for me.
Thank you so much @VagyokC4

Either a computed field like @VagyokC4 points out, or an Aggregation with a filter would do the trick.

Going to close this out to tidy up :)


Notifi Amplitude events & feature flag

What is the purpose of the change

Adds amplitude events and feature flag for notifi notifications dropdown.
Also adds some minor cleanup
ClickUp Task
ClickUp Task URL
Brief Changelog

Amplitude vents for the panel, from this list
Added "Notifications" feature flag in LD

Testing and Verifying
For testing, I recommend toggling the "DEBUG" flag in use-amplitude-analytics and watching the console to ensure the events log as expected.

looks good to me once we figure out whatever the deploy error is

It appears to be a build error in generated code cc @JoseRFelix


Add a Pre-Transition Signal

This allows for the code calling Transitions.change_scene(...) to connect to the signal and perform some setup or modifications to the scene before the transition starts.
Example Usage:
func setup_player():
    pass # Player setup here. Set its position, change its colour, whatever!

func transition_scene(next_scene_path: String):
    var next_scene = load(next_scene_path).instance()
    Transitions.connect("pre_transition", self, "setup_player")
    Transitions.change_scene(next_scene, Transitions.FadeType.Blend, 1.5, transition)
    # Optional disconnecting from signal here.

Very minimal change. Might be my Godot Newbie Naïvety, but I can't imagine how these two lines could break anything.
Also open to different naming of the signal and/or passing the new scene along with it.

Just curious, can you call setup_player before calling Transitions.change_scene? Is there something in Player.gd that will break if you do that?

It crashes for me because when I set the player's position, it calls a method of an onready variable (which is null because it's not ready?). Weirdly, if I use the node path instead of the onready variable, it works.

That ... doesn't make sense. Using the node path instead of onready should crash equally because you haven't added the player to the scene yet. It could be a race condition in Godot or something platform-specific (crashes in release mode, crashes on Linux, etc.)
I see exactly the problem now. In my RPG, I solved this by modifying the source to allow, essentially, a less clean version of what you're doing (instead of a signal, I directly find/set the player's position, which is an input to the fade method).
Looks good to me! Sorry for all the questions, thanks for your contribution!

Thanks for accepting (I agree it doesn't make much sense to me - I think something like a race condition sounds plausible), and thanks for the addon in the first place! It's a great one.
Good luck with your game, and sorry in advance for any more pull requests I make >.< It's only because I think the addon is worth the effort! Feel free to tell me to shove off and make my own plugin ;)

No worries, I hope you didn't get the impression that I'm unfriendly to changes :sweat_smile: thanks for the contribution, hopefully you have more useful ones to contribute in the future!


[AIRFLOW-2562]Add Google Kubernetes Engine Operators

JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2562

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Add Google Kubernetes Engine create_cluster, delete_cluster operators
This allows users to use airflow to create or delete clusters in the
google cloud platform via an operator
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
tests.contrib.operators.test_gcp_container_operator.py
tests.contrib.hooks.test_gcp_container_hook.py

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3477 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3477      +/-   ##
==========================================
- Coverage   77.04%   77.03%   -0.01%
==========================================
  Files         203      203
  Lines       15119    15119
==========================================
- Hits        11648    11647       -1
- Misses       3471     3472       +1

Impacted Files
Coverage Δ

airflow/models.py
87.32% <0%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks for this. This is certainly going to be very helpful.

Can you please rebase against the master? The CI was failing for unrelated tests which are now fixed.

Rebased, looks like it's passing now.
Thanks!

@fenglu-g Any more comments on this?

Good catch -- fixed merge conflicts
Thanks

@kaxil feel free to merge if it looks good to you as well, thank you for your help!


clamp: auto in parent elements with many childs

When the vertical ellipse is placed in an element with a parent with many childs the 'auto' clamp dont works. Its unable to calculate the correct height.
If you replace the line 92 of clamp.js (in the function getMaxLines) with this other one it ill do it well
var availHeight = height || (element.parentNode.clientHeight-element.offsetTop),

I hope it works now :)


Simple updates to prod urls in datasets notebooks

Small batch of simple notebook updates: new production api urls and modifications for pgstac titiler v1 breaking changes.
notebooks/datasets

air quality covid
ocean npp timeseries analysis
volcano so2 monitoring

notebooks/quickstarts

hls-visualization

What about `nceo-biomass-statistics.ipynb"?

Ignore comments above, read in the spreadsheet that you can't run in the hub.