Zsh: add color - to switch back to previous color

If I am checking out a bunch of color schemes, I often find myself
switching to a theme for a bit and then wanting to switch back to the
one I was previously using. This commit adds this feature to the color
script by teaching it to respond to the special "-" argument. This works
similarly to how other commands like cd - and git checkout - work,
so it feels pretty natural.
I considered writing the previous color as metadata to my existing
base16 config file to avoid adding another file, but I decided that this
approach would be a little easier to implement so I am rolling with it
for now. But in the future, that might be a possible avenue for
improvement.
I originally did this in
https://github.com/lencioni/dotfiles/commit/249de701 and thought you
might be interested.

Neat idea!
(Over-engineering instincts kicking in here make me want to implement this as a full "reflog" style thing with the ability to switch back to @{1}, @{2} etc as well. I would never use this, but keeping one previous entry seems a bit arbitrary when you could just keep a log of them, and maybe you could justify it in terms of being able to generate interesting stats on color scheme usage.)

If you bring this in, you'll want this fix: https://github.com/lencioni/dotfiles/commit/81f2ff6900a3ed41c9d844323d09b177dd52b9f1


Facing issue on integrating pdf.js with Backend NodeJs

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: Using Backend Node.Js
Operating system and its version: Mac
PDF.js version: Latest
Is a browser extension: NO

Steps to reproduce the problem:

Create an Express Js App
Install pdfjs-dist
import the pdfjs-dist, as done in https://github.com/mozilla/pdf.js/blob/0725b6299f122632a7d5b97f82fe584ae1584c5d/examples/node/getinfo.js
Run npm run dev

What is the expected behavior? (add screenshot)
Import the pdfjs-dist and start using the APIs. I want to fetch the Document Outline of the PDF and found an api getOutline() as given in https://github.com/mozilla/pdf.js/blob/0725b6299f122632a7d5b97f82fe584ae1584c5d/src/display/api.js#L951-L972.
What went wrong? (add screenshot)
npm run dev throws error. TypeError: Cannot read properties of undefined (reading 'prototype')

Here is the code
app.js

package.json

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): NA

PDF.js version: Latest

Please remember to provide a version number, since "latest" isn't really helpful.

Here is the code

Please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.


Add Inputs.form

Fixes #73. Demo: https://observablehq.com/d/4108c4771279ad1a

Any thoughts on this one, @Fil? (I’ll get to that Legends PR soon…)

I like the fact that it's hierarchical, like the good old dat-gui; however in that case, we could want a way to pass styling or fieldset+label options to the children, if we want the inputs structure to be visible? This would be useful in general but particularly so for nested inputs. But maybe that's for another project.
Another idea is that we could also support a Map of inputs, that would return a Map of values (or maybe a key.value object). It would seem more consistent with Inputs.select, for example, and lists of options generated by, say, d3.rollup.

I'd love see an option for a template callback added that simply receives the inputs array/object as passed to Inputs.form(), and returns the outermost element.

Nice idea. Added.

When will the new release get deployed to Observable?

In the near future.

Is it possible for the properties of an input in a form to depend on the value of another in the same form?


Issue #119: Added basic details around launchpad install script

@cmoulliard let me know what you think

Merged: https://github.com/openshiftio/appdev-documentation/commit/f526ba3a0a75099ffca679598ba2ca8e9137dff4


[fix][Modal | Dialog]: Add dismissible layer component

What does it do?
Add a new component to the design-system that handles "Dismissible Layers" commonly found with modals / dialogs etc. This component handles global listeners for escape and pointerDown the latter is important because a "click" event causes a false closure, see https://github.com/strapi/strapi/issues/13044#issuecomment-1225900907
Why is it needed?

Modals shouldn't close on a click event you want to know where the mouse pressed down not where it finished.

Additional thoughts
There's more you can do in this area, but I see this as a temporary fix whilst we decide on the progression of this lib.
Related issue(s)/PR(s)

partially https://github.com/strapi/strapi/issues/13044

Also should this be added to other components e.g. Popover? IIRC those menus behave similarly, they're supposed to have a focus capture and the same dismissible properties...?

Also should this be added to other components e.g. Popover? IIRC those menus behave similarly, they're supposed to have a focus capture and the same dismissible properties...?

Sounds good indeed 👀

Also should this be added to other components e.g. Popover? IIRC those menus behave similarly, they're supposed to have a focus capture and the same dismissible properties...?

Sounds good indeed 👀

I agree @ronronscelestes, @gu-stav what do you think?

yep I agree, made this change.


"handlePastedFiles" trigger conditions?

Ctrl + V does not trigger

HACK: Monkey-patch
(function () {
    const PhotosMimeType = require('fbjs/lib/PhotosMimeType');
    const DataTransfer = require('fbjs/lib/DataTransfer');
    DataTransfer.prototype.isImage = function () {
        const isImage = this.types.some(type => type.indexOf('application/x-moz-file') !== -1);
        if (isImage) return true;

        const items = this.getFiles();
        for (let i = 0; i < items.length; i++) {
            const type = items[i].type;
            if (!(new PhotosMimeType(type)).isImage()) return false;
        }

        return true;
    };
}());

The fix will be pushed to fbjs soon. :)

@hellendag when will repair?

https://github.com/facebook/fbjs/commit/b907f914994dce1e814838240c2af3ddb88d59fc. We'll ship a new version of fbjs with that soon.

Passing to @zpao now that this has landed. Once we've got a new fbjs version out, this can be closed. :)

We published alpha.2 so new install should be getting it already but also just added https://github.com/facebook/draft-js/pull/193 which will make sure

Merged.


Items background color not rendering

Describe the bug
A clear and concise description of what the bug is.
style: {
backgroundColor: 'fuchsia'
},
Style backgroundColor added in items not rendering.
To Reproduce
Please provide one or both of the following:
Code (If you have a branch somewhere that shows your issue) or link to codesandbox example. Here's a base template for this library:
OR
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Library Version
Please provide version of library in which this bug is occurring.
Desktop (please complete the following information):

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

Smartphone (please complete the following information):

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

Additional context
Add any other context about the problem here.

you need to do background not backgroundColor

you need to do background not backgroundColor

But in the documentation - https://github.com/namespace-ee/react-calendar-timeline#items it is given that it will be backgroundColor within style tag.

@himadriganguly have you checked this example?
https://codesandbox.io/s/k0wn41y0o7
if it doesn't help you please provide a codesandbox an I will reopen this issue


Commented out copy of app for iOS

This change is

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


Add importvalue tests

Coverage increased (+9.3%) to 100.0% when pulling 33df7736f3810a003ff01eb117f78175ff779161 on yuce:add-importvalue-tests into d4a5ac82132176849cf868c8adf893bbd78cbfcc on pilosa:master.

I've restored the travis config to the current release of Pilosa. The builds won't succeed meanwhile.

Merging this, it only includes tests.


Update tests for upstream changes in rustc

It looks like the restrictions around calling proc-macro APIs outside of proc macros have been tightened, so our previous test no longer compiles.
The no_implicit_prelude seems to interact strangely with 2018-style modules, but our generated code is ok.

It looks like the restrictions around calling proc-macro APIs outside of proc macros have been tightened

I hoped the opposite might happen soon :D
Thanks for catching this!


Feature: Shows asterisk automatically when input has RequiredValidator in ReactiveForms

Bug, feature request, or proposal:
Feature Request.
What is the expected behavior?
Actually there's an property in input called required. I'm wondering if is there any chance to have it built-in in reactive forms? I mean, if an input has the Validators.required it will automatically shows the asterisk indicating that's required. To disallow this behavior people could use [hideRequiredMarker ]="true".
What is the current behavior?
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Material 2.0.0-beta.6
Is there anything else we should know?
It would help a lot since you wouldn't need to put Validators.required in component and also [required] in template.

Looks like dupe of https://github.com/angular/material2/issues/2574

Woops, I didn't said this issue. Thanks @willshowell


New invoice from project details page

Notion card
https://www.notion.so/saeloun/Admin-and-owner-should-be-able-to-generate-a-new-invoice-from-the-project-details-page-afb77ebb3d234b2f87801932b29dbbce
Summary

Admin/Owner can generate an invoice for that client from the project details page.

Preview
https://www.loom.com/share/8368d4dcaeaa4331aee05a7bf20527cf
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactor (non-breaking and retains the same functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[ ] I have annotated changes in the PR that are relevant to the reviewer
[ ] I have added automated tests for my code

@abinashpa Please update the branch.


About Error Handling in Server

Description:
As I implementing some features, I noticed that it is quite cumbersome to add and handle error. I thought that there should be I best practice for go based gRPC server’s error handling, so I have researched few.
I’m leaving this issue to compare Yorkie’s error handling with other go based open source projects like etcd, kubernetes, promethous, grafana, etc.
First, I will compare Yorkie’s error handling with etcd, which also uses gRPC server as it’s modules.
In summary, etcd’s error handling structure looks like this:

api

v3rpc/rpctypes

error.go : Creates custom error type variables using gRPC status and code.ErrGRPCRootUserNotExist = status.Error(codes.FailedPrecondition, "etcdserver: root user does not exist")

server

auth

store.go : ErrRootUserNotExist error is defined and used.ErrRootUserNotExist = errors.New("auth: root user does not exist")

etcdserver/api/v3rpc

util.go : Handles error in server(togRPCError()). Imports several modules(store.go) in server, and maps modules’ error(ErrRootUserNotExist) to rpctypes.go's custom gRPC error(ErrGRPCRootUserNotExist) to convert to gRPC error.
var toGRPCErrorMap = map[error]error{
    auth.ErrRootUserNotExist:     rpctypes.ErrGRPCRootUserNotExist,
    ...
}

In Yorkie, error handling is done like this:

server

backend

database.go : ErrProjectNotFound error is defined and used.  ErrProjectNotFound = errors.New("project not found")

grpchelper

status.go : Handles error in server(toStatusError()). Imports several modules(database.go) in server, and maps modules’ error(ErrProjectNotFound) directly to gRPC error to convert to gRPC error.
var errorToCode = map[error]codes.Code{
    database.ErrProjectNotFound:  codes.NotFound,
    ...
}

The differences between the two is that there are custom gRPC error type variable defined in etcd’s api/types using status.error(). It looks like the overall error handling structure is almost the same except there are several api versions in etcd so it is nested with api versions.
To conclude, etcd seems to redefine error in api to clarify and organize all the errors that needs to be sent to client.
(I will add comparison on other open sources like K8s and Prometheus in further updates)
Why:
It is quite cumbersome to add and handle error on implementing new features.
Therefore, it will be good to introduce best practice for go based gRPC server’s error handling by referencing other go based open source projects like etcd, kubernetes, promethous, grafana, etc.

Related to https://github.com/yorkie-team/yorkie/issues/199.

Currently, the server delivers detailed error codes that occur to the client(#927). I'll close this issue for now and reopen it as needed.


sink deletes k8s objects when json path returns a value

Which Issue(s) this Pull Request Fixes

Fixes #191
Release Note

NONE

Notes for Reviewers

The operator grants the sink's ServiceAccount permissions to delete the resource type(s) being watched. When deploying kubearchive, the user can specify a JSON Path that, if it returns a value when applied to a kubernetes object, will cause the sink to delete the kubernetes object from the cluster.

Instructions to test:

install kubearchive following instructions from DEVELOPMENT.md including database initialization
run kubectl create ns demo-tekton
run kubectl apply -f https://storage.googleapis.com/tekton-releases/pipeline/latest/release.yaml
run kubectl apply -f https://raw.githubusercontent.com/skoved/kubearchive/raw/first-demo/conf-tekton.yaml
in a separate terminal run `kubectl logs -n kubearchive -l app=kubearchive-sink -f
run kubectl apply -f https://raw.githubusercontent.com/skoved/kubearchive/raw/first-demo/tekton.yaml

After ~ 10-20 seconds, you should see log that says successfully deleted kubernetes object <uid>

run kubectl apply -f https://raw.githubusercontent.com/skoved/kubearchive/raw/first-demo/conf-tekton.yaml

the URL has an extra "/raw" on it, its
https://raw.githubusercontent.com/skoved/kubearchive/first-demo/conf-tekton.yaml
The second raw URL also has an extra "/raw" on it, it should be:
https://raw.githubusercontent.com/skoved/kubearchive/first-demo/tekton.yaml

I get the following logs:
2024/07/23 07:20:34.129580 received CloudEvent:  ffe9b483-f469-4bd2-94e8-fac6c2ac23f9
2024/07/23 07:20:34.129693 cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 contains all required fields. Attempting to write it to the database
2024/07/23 07:20:34.129701 writing resource from cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 into the database
2024/07/23 07:20:34.137091 successfully wrote cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 to the database
2024/07/23 07:20:34.137098 checking if resource from cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 needs to be deleted
2024/07/23 07:20:34.137111 attempting to delete kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.163339 successfully deleted kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.164397 received CloudEvent:  0d8b37d1-7956-4ba2-802a-ac336d01889e
2024/07/23 07:20:34.164530 cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e contains all required fields. Attempting to write it to the database
2024/07/23 07:20:34.164538 writing resource from cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e into the database
2024/07/23 07:20:34.171759 successfully wrote cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e to the database
2024/07/23 07:20:34.171765 checking if resource from cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e needs to be deleted
2024/07/23 07:20:34.171777 attempting to delete kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.179396 could not delete resource 0d69c727-b2fc-4c27-916a-e4822bb772c3: pipelineruns.tekton.dev "hello-goodbye-run" not found

So when its deleted, I also receive an event about it and try to delete it again. I think its fine for now, but probably we want to track it, so we do something about it.

kubearchive

my bad, I will fix

run kubectl apply -f https://raw.githubusercontent.com/skoved/kubearchive/raw/first-demo/conf-tekton.yaml

the URL has an extra "/raw" on it, its
https://raw.githubusercontent.com/skoved/kubearchive/first-demo/conf-tekton.yaml
The second raw URL also has an extra "/raw" on it, it should be:
https://raw.githubusercontent.com/skoved/kubearchive/first-demo/tekton.yaml

oops, my bad. I fixed the link for that

I get the following logs:
2024/07/23 07:20:34.129580 received CloudEvent:  ffe9b483-f469-4bd2-94e8-fac6c2ac23f9
2024/07/23 07:20:34.129693 cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 contains all required fields. Attempting to write it to the database
2024/07/23 07:20:34.129701 writing resource from cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 into the database
2024/07/23 07:20:34.137091 successfully wrote cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 to the database
2024/07/23 07:20:34.137098 checking if resource from cloudevent ffe9b483-f469-4bd2-94e8-fac6c2ac23f9 needs to be deleted
2024/07/23 07:20:34.137111 attempting to delete kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.163339 successfully deleted kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.164397 received CloudEvent:  0d8b37d1-7956-4ba2-802a-ac336d01889e
2024/07/23 07:20:34.164530 cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e contains all required fields. Attempting to write it to the database
2024/07/23 07:20:34.164538 writing resource from cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e into the database
2024/07/23 07:20:34.171759 successfully wrote cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e to the database
2024/07/23 07:20:34.171765 checking if resource from cloudevent 0d8b37d1-7956-4ba2-802a-ac336d01889e needs to be deleted
2024/07/23 07:20:34.171777 attempting to delete kubernetes object: 0d69c727-b2fc-4c27-916a-e4822bb772c3
2024/07/23 07:20:34.179396 could not delete resource 0d69c727-b2fc-4c27-916a-e4822bb772c3: pipelineruns.tekton.dev "hello-goodbye-run" not found

So when its deleted, I also receive an event about it and try to delete it again. I think its fine for now, but probably we want to track it, so we do something about it.

yeah, the sink deletes an object, ApiServerSource ends up creating a delete event for that resource. We need to process that event to get the timestamp for when the object was deleted, but you're right that the sink needs skip trying to delete the resource in that case. See #232

So when its deleted, I also receive an event about it and try to delete it again.

Shouldn't we configure ApiServerSource to not send events for deletions?


Fix misspelled param names in SpecAn.

What does this Pull Request accomplish?
Fixes two misspelled param names for SpecAn ("measurerment" => "measurement").
Why should this Pull Request be merged?
SpecAn has the misspellings in their API but we can fix it on our end in the proto file.
What testing has been done?
Build succeeds.

Note: Filed AB#1871307 to RFmx to fix it on the driver side of things.


Rename bitcoind_ibd to bitcoin_ibd in Prometheus Script

breaks expression: "bitcoind_ibd" in
https://github.com/digitalbitbox/bitbox-base/blob/c2e3a698799c3593270f4dea53ac41a488e68571/tools/bbbsupervisor/bbbsupervisor.go#L283

@0xB10C renamed in bbbsupervisor as well now.

LGTM, at least for the supervisor.


Fix errors loading when target bar is disabled

Disabling the target bar module breaks the addon quite horrifically; you can still get to the settings page to re-enable it however.
This fixes the errors by adding a few key if TRP3_API.target checks in areas where it was initially failing.
One issue that I still need to look at after this fix is with the module disabled, it seems the glance portion doesn't quite take the hint:

Nevermind on that secondary glance bar issue as it seems to be intended; the target frame just adopts the glance bar and re-positions it from what I can tell.


Global suppression for namespace is ignored

We use EF migrations on one of our projects and there are a lot of auto generated files (one per migration) that result in lots of RECS0070 warnings.
I've attempted to suppress them in GlobalSuppressions.cs using the following line of code but the warnings remain in the Error List window even after a rebuild.
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Redundancies in Code", "RECS0070:Redundant explicit argument name specification", Justification = "Auto-generated code by Entity Framework Migrations", Scope = "namespace", Target = "Portal.Data.Returns.Migrations")]

Sorry, looks like a restart of VS2015 has cleared those persisting warnings.


Font size label and value

Would be great if you could provide a list of labels and values on fontsize plugin, so I can do values as "2rem" but the user see "32px" or "32pt" I have to do accessibility on this website, so the paragraphs need to be in em or rem

You can already do this by using translations :)
I admit, its a little bit hacky, but you can put some labels like this:
// 				   v--- 'en' or any language you are using
$.trumbowyg.langs.en.fontsizes = [
	'1rem': '14px',
	'1.2rem': '17px',
	'1.5rem': '21px'
};

$('#editor-custom-font-sizes')
.trumbowyg({
    btns: [
        ['fontsize']
    ],
    plugins: {
        fontsize: {
            sizeList: [
                '1rem',
                '1.2rem',
                '1.5rem'
            ],
            allowCustomSize: false
        }
    }
});


Specify charset when generating tokens

Ensures the token is URL-safe even if the system charset is not a superset of ASCII.

Thanks for the update @OrangeDog. This is now in master.

What's the reason for switching to Charset.forName("US-ASCII")?

There was a build failure:

[INFO] --- animal-sniffer-maven-plugin:1.6:check (enforce-java-6) @ spring-security-oauth2 ---
[INFO] Checking unresolved references to org.codehaus.mojo.signature:java16:1.0
[ERROR] Undefined reference: java/nio/charset/StandardCharsets.US_ASCII#Ljava/nio/charset/Charset; in /Users/jgrandja/workspace/spring-security-oauth-fork/spring-security-oauth2/target/classes/org/springframework/security/oauth2/provider/token/DefaultTokenServices.class

Ok, I didn't realise this was supposed to still support Java 6.


JAVA-42138 upgrade or disable modules that fail due to restlet error

New PR: https://github.com/eugenp/tutorials/pull/18035 to add JIRA id's for modules disabled in this PR.


Merge Microsoft.app 2022 06 01 preview to main

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

All changes in this PR are from our feature branch https://github.com/Azure/azure-rest-api-specs/tree/Microsoft.App-2022-06-01-preview, which are reviewed by ARM

@tadelesh @qiaozha @msyyc please check the sdk breaking changes

/azp run

@Juliehzl Could you help to provide the previous PR review and the breaking change review item?

/azp run

The breaking is from this PR and has been approved yet.


specify the entry point for pub build to use

Currently, pub build acts on all entry-points in the web/ directory. In our application we have several entry-points in the web/ directory, but we only want to deploy a specific entry-point. I'd like the ability to indicate which entry-point to deploy, and only have pub deploy that specific one.

This issue has been moved to dart-lang/pub#739.


Add flutter app

Create new Flutter Increment Number App #80

Hey @krishna-raz can you please assign me this

Hey @krishna-raz
Flutter application you're referring is a default application, this app won't be considered in this poject! If you have new idea feel free to raise a new issue
Thank you!


Work with MySQL group replication

Error log:
2022-08-24 02:14:35,802 ERROR [c.c.u.d.Merovingian2] (main:null) (logid:) cleanupForServer:Exception:The table does not comply with the requirements by an external plugin.
2022-08-24 02:14:35,909 ERROR [c.c.u.d.Merovingian2] (main:null) (logid:) cleanupForServer:Exception:cleanupForServer:Exception:The table does not comply with the requirements by an external plugin.

seem caused by op_lock and op_nwgrp_work use MEMORY engine:
| def           | cloud              | op_lock                                              | MEMORY             |
| def           | cloud              | op_nwgrp_work                                        | MEMORY             |
+---------------+--------------------+------------------------------------------------------+--------------------+

any solution?
in ./engine/schema/src/main/resources/META-INF/db/schema-490to4910.sql:, engine will be set to innodb：
ALTER TABLE cloud.op_nwgrp_work ENGINE=INNODB;

So, can I use InnoDB from the start？
ISSUE TYPE

Bug Report

COMPONENT NAME

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

@itsmdev thanks for reporting, I think they all should be INNODB unless there is any reason not to. Can you check latest main or 4.17 branch incase this is already fixed those cc @nvazquez if you remember we fixed them?

@rohityadavcloud
https://github.com/apache/cloudstack/blob/c58118148e1a579efc1d59dba27f54463222c843/setup/db/create-schema.sql#L401
https://github.com/apache/cloudstack/blob/c58118148e1a579efc1d59dba27f54463222c843/setup/db/create-schema.sql#L1796

according to https://github.com/apache/cloudstack/blob/4.17/engine/schema/src/main/resources/META-INF/db/schema-490to4910.sql#L30-L31 this has long been changed and will be automatically be updated on every installation after 4.9

@itsmdev I am closing this issue. If you feel this is invalid please reopen or create a new one

This can be replicated on new install, when bootstrapping with cloudstack-setup-databases.  I guess the script uses an older schema which is expected to be upgraded during the first run. However, CloudStack fails to start due to the error above.


Feature request: Web browser notification

It would be nice to have mailhog capable to display notifications upon receiving messages. This would save clicking on mailhog tab every time to see if a message is received.

Which browser? Here my experience on a Mac:

Firefox: It seems to work. It even plays a bing sound.
Safari: Shows the notifcation, but no sound. (dunno why not playing sound)
Chrome: No notification, no sound. (dunno why)

Screenshot: Mailhog is not the active tab, but receives the notification.

@peterdd Chrome, Linux, Version 72.0.3626.121 (Official Build) (64-bit)

Maybe I have the reason: https://developer.mozilla.org/en-US/docs/Web/API/notificationSecure context
"This feature is available only in secure contexts (HTTPS), in some or all supporting browsers."
So in the latest Firefox 67 it is not working anymore - unless I change in about:config the dom.webnotifications.allowinsecure to false when you use mailhog just local for testing without TLS.
(notification works again, with sound)
In Chrome 74 I fiddled with chrome://flags
#unsafely-treat-insecure-origin-as-secure (enabled and insert address of your mailhog running)
and
#enable-message-center-new-style-notification (enabled)
to get them working for a local http ip address. ( http://10.0.0.x:8025 )
(notifications shown, no sound, don't know why)

Hi,
With Firefox 74, even this does not seem to work.

So in the latest Firefox 67 it is not working anymore - unless I change in about:config the dom.webnotifications.allowinsecure to false when you use mailhog just local for testing without TLS.

Any other way to make this work now?

Sorry, I meant set
dom.webnotifications.allowinsecure to true
With Firefox 74 was able to get the notification alerts on mac again. New seems the little speech bubble icon in the address bar. Click and allow it in the popup dialog and you get the other icon as shown in screnshot.

@peterdd Perfect! Both changes done and it works on Windows 10 & Firefox 74! Thanks a lot!


Adds to readme.md

I saw your youtube video on making the first contribution to Github.

Can't say no to that, can I? ;-)


Treasury Advisory Service Meeting -  Monday, 13th February, 2023

Attendees

[x] Andre
[x] Stephen
[x] Miro

Action Notes

Don't close the issue after meeting as the GitHub action needs an open issue with the label 'meeting' in order to make the new meeting issue.
Keep notes in the body of each issue for these auto-generated meeting issues.

Meeting Notes

Issue with the GitHub Action that produces this meeting issue. Produces NULL result. Possible problem caused by lack of previous issue may resolve itself when the Action runs again for the next meeting.

Previous in series: #96

Next in series: #100


Gracefully handle pre-skull-stripped T1w images

A number of subjects in ds000223 (e.g. sub-07) have pre-skull-stripped T1w images. ANTs skull-stripping on these datasets clips most superficial cortex, resulting in very poor preprocessing.
I propose adding a --pre-skull-stripped flag (or similar) to indicate that the anatomical workflow should assume the incoming image is already skull-stripped.
Alternately, we could do a test for a given percentage of zero voxels as evidence for skull-stripping, though this would need to be tested against defacers, as well.

Related to NeuroStars #4091, another thing we could consider is allowing users to supply their own desc-brain_mask.nii.gz, similar to how they currently supply lesion masks. In the existing dataset cases, we could simply filter T1w images with T1w_data > 0 to produce a mask, that would then bypass the skullstripping workflow.

This would increase the number of datasets we support in NeuroScout by at least 1, and possibly more, so if there's any way to prioritize this (especially given that it's tagged as low effort), it would be much appreciated.

I tested out on a limited set of images (skullstripped, defaced, raw), Chris's suggestion of seeing if all extreme planes are all zeros. In my limited test case this worked.
I also looked into where I could stick this into the workflow. It seems like niworkflows.anat.ants.init_brain_extraction_wf is the right place.
I'm guessing, if this test indicates the image is skullstripped, we'd skip the ant normalization step, and potentially the following guassian interpolation, and thresholding. Instead, we could just threshold > 0 as @effigies suggested.
Seem correct? This ants brain extraction workflow is a bit confusing as its not clear if "normalization" = "extraction"

Brain extraction and normalization are two different things. We would just skip extraction, and pass the skull stripped brain in two places for normalization.

Makes sense, I just have no clue where the extraction happens then (is it map_gaussian with the templates? antsAI?), so if you could point me to which steps would be skipped, I'd be happy to give implementation a go and test w/ the datasets I have.

https://github.com/poldracklab/smriprep/blob/5b519b1e2446ec8b69848575d7bc67800474f035/smriprep/workflows/anatomical.py#L226-L231
https://github.com/poldracklab/smriprep/blob/5b519b1e2446ec8b69848575d7bc67800474f035/smriprep/workflows/anatomical.py#L259-L260
https://github.com/poldracklab/smriprep/blob/5b519b1e2446ec8b69848575d7bc67800474f035/smriprep/workflows/anatomical.py#L269-L270
I guess the question is whether to INU correct a skull-stripped image or assume that it already is. I think in the past we've assumed that a second application is generally not harmful, so as long as it doesn't depend on the presence of skull, it probably just makes sense to swap brain_extraction_wf with an N4 node.


Source Control not working

Issue Type: Bug
Trying to view the Source Control tab (on the left) in a freshly cloned repo with .git directory in place (or even a new clean repo with git inited) vsCode only offers a button with 'Initialize Repository'
When I push that button, it says 'Git: spawn usr/local/bin/git ENOENT' and buttons 'cancel' and 'Open Git Log'.
Open Git log says (in terminal)
Validating found git in: usr/local/bin/git
Using git 2.23.0 from usr/local/bin/git
Opening repository for path='/Users/esgerjellema/Workspace/frontend-aurelia' failed; ex=spawn usr/local/bin/git ENOENT {
"gitErrorCode": "NotAGitRepository"
} .. and more of such.
I tried many things like disabling extensions, reinstalling xcode command-line-tools, accept licence, many other solutions I could find on the web, but to no avail.
VS Code version: Code 1.62.3 (Universal) (ccbaa2d27e38e5afa3e5c21c1c7bef4657064247, 2021-11-17T07:59:13.865Z)
OS version: Darwin x64 21.1.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-9600K CPU @ 3.70GHz (6 x 3700)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 4, 3

Memory (System)
16.00GB (0.85GB free)

Process Argv
--crash-reporter-id 2d6f446a-c6c8-4899-a3d5-4999669b5c7c

Screen Reader
no

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

vscode-caniuse
aga
0.5.0

atlascode
atl
2.10.0

aurelia
Aur
2.0.7

vscode-style-formatter
dwe
1.0.0

gitlens
eam
11.7.0

prettier-vscode
esb
9.0.0

auto-rename-tag
for
0.1.9

beautify
Hoo
1.5.0

git-graph
mhu
1.30.0

vscode-autoprefixer
mrm
3.0.1

es6-css-minify
olb
3.3.2

vscode-yaml
red
1.2.2

vscode-sort-json
ric
1.20.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30384386
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30391933
pythonvspyt187:30373474
vsaa593cf:30376535
vscexrecpromptt2:30404948
vscop804cf:30404767
vs360cf:30404996
vsrem710cf:30405999

@Esger, sorry for not getting back to you on this until now.
Can you please confirm that you are unable to initialise a repository using the following steps:

Create a new folder foo
In terminal navigate inside the foo folder and type code-insiders .
Open the SCM viewlet and click on the "Initialize Repository" button.
If it fails, can you please share the contents of the git output?

In the terminal could you also please run whereis git and send me the output.

Hi, thank you for reaching out.
I tried what you suggested and I get this error message when I type
code-insiders . :
zsh: command not found: code-insiders
With bash shell I get the same error.
So I install code-insiders first and open the foo folder from within
code-insiders - started manually (not from terminal)
Then I could press Initialize Repository and that works.
Not sure about the git output you ask for, but I included the zipped .git
folder here.
Should I use code-insiders i.s.o. vsCode then?
Kind regards,
Esger Jellema
Voermanhaven 19
9742 VR  Groningen
M 06 49 68 68 93
http://nl.linkedin.com/in/esgerjellemaProfiel van Esger Jellema
weergeven http://www.linkedin.com/in/esgerjellema
Op vr 10 dec. 2021 om 11:29 schreef Ladislau Szomoru <
@.***>:

@Esger https://github.com/Esger, sorry for not getting back to you on
this until now.
Can you please confirm that you are unable to initialise a repository
using the following steps:

Create a new folder foo
In terminal navigate inside the foo folder and type code-insiders .
Open the SCM viewlet and click on the "Initialize Repository" button.
If it fails, can you please share the contents of the git output?

In the terminal could you also please run whereis git and send me the
output.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/138278#issuecomment-990824053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARDH3SLIN4EYFD5UIQOG3UQHJAFANCNFSM5JE6BCZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Are you saying that everything is working as expected in VS Insiders?
What about if you upgrade to the latest version of VS Code (1.63 which was released this week)?

I already had the newest version of VS Code
Version: 1.63.0 (Universal)
Commit: 7db1a2b88f7557e0a43fec75b6ba7e50b3e9f77e
Date: 2021-12-07T05:15:48.091Z (3 days ago)
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.1.0
Which is not working as aspected... it's not recognizing the git repo.
VS Insiders is indeed working as expected.. source control, git-lens
extension and others all fine.
Kind regards,
Esger Jellema
Voermanhaven 19
9742 VR  Groningen
M 06 49 68 68 93
http://nl.linkedin.com/in/esgerjellemaProfiel van Esger Jellema
weergeven http://www.linkedin.com/in/esgerjellema
Op vr 10 dec. 2021 om 13:19 schreef Ladislau Szomoru <
@.***>:

Are you saying that everything is working as expected in VS Insiders?
What about if you upgrade to the latest version of VS Code (1.63 which
was released this week)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/138278#issuecomment-990926362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARDH3YLRMOHDECDNTJWNLUQHV4HANCNFSM5JE6BCZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

While its great that things are working in Insiders I really want to get to the bottom of why things are broken in Stable.

Could you start VS Code Stable using code --disable-extensions and see if that resolves the problem?
Could you start VS Code Stable using code and see if invoking the Git: Show Git Output from the command palette resolves the issue?

Hi, good morning,
When I tried disabling extensions (manually) before, it did not work. Now
with the terminal (code --disable-extensions) command source control seems
to work fine in VS Code Stable.
Strangely with terminal (code) command it seems to work as well. I'm
riddled here. Git output:
Validating found git in: usr/local/bin/git
Validating found git in: /usr/local/bin/git
Using git 2.23.0 from /usr/local/bin/git

git rev-parse --git-dir
Open repository: /Users/esgerjellema/Workspace/frontend-aurelia
git status -z -u
git symbolic-ref --short HEAD
git for-each-ref
--format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)
refs/heads/develop refs/remotes/develop
git for-each-ref --sort -committerdate --format %(refname) %(objectname)
%(*objectname)
git remote --verbose
git config --get commit.template
git check-ignore -v -z --stdin
git show --textconv :src/slides/slide-edit-settings.html
git ls-files --stage --
/Users/esgerjellema/Workspace/frontend-aurelia/src/slides/slide-edit-settings.html
git show --textconv :src/resources/bnc-base/bnc-form.less
git ls-files --stage --
/Users/esgerjellema/Workspace/frontend-aurelia/src/resources/bnc-base/bnc-form.less
git cat-file -s 4397d6978a63c1e499f89747bf1276b211418019
git cat-file -s ed8866e0e61eebfbad167fbcbbf14e620c645835
git status -z -u
git symbolic-ref --short HEAD
git for-each-ref
--format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)
refs/heads/develop refs/remotes/develop
git for-each-ref --sort -committerdate --format %(refname) %(objectname)
%(*objectname)
git remote --verbose
git config --get commit.template
git ls-files --stage --
/Users/esgerjellema/Workspace/frontend-aurelia/src/slides/slide-edit-settings.html
git ls-files --stage --
/Users/esgerjellema/Workspace/frontend-aurelia/src/resources/bnc-base/bnc-form.less
git cat-file -s 4397d6978a63c1e499f89747bf1276b211418019
git cat-file -s ed8866e0e61eebfbad167fbcbbf14e620c645835
git show --textconv :src/slides/slide-edit-settings.html
git show --textconv :src/resources/bnc-base/bnc-form.less

Then when quitting VS Code and starting it from the dock it doesn't work
anymore and gives this output:
Validating found git in: usr/local/bin/git
Using git 2.23.0 from usr/local/bin/git
Opening repository for
path='/Users/esgerjellema/Workspace/frontend-aurelia' failed; ex=spawn
usr/local/bin/git ENOENT {
"gitErrorCode": "NotAGitRepository"
}Error: spawn usr/local/bin/git ENOENT
at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
at onErrorNT (internal/child_process.js:465:16)
at processTicksAndRejections (internal/process/task_queues.js:80:21)
Quitting and starting it from terminal with code command it works again.
So that's what I can do for now, start code from the terminal.
Kind regards,
Esger Jellema
Voermanhaven 19
9742 VR  Groningen
M 06 49 68 68 93
http://nl.linkedin.com/in/esgerjellemaProfiel van Esger Jellema
weergeven http://www.linkedin.com/in/esgerjellema
Op vr 10 dec. 2021 om 20:26 schreef Ladislau Szomoru <
@.***>:

While its great that things are working in Insiders I really want to get
to the bottom of why things are broken in Stable.

Could you start VS Code Stable using code --disable-extensions and
see if that resolves the problem?
Could you start VS Code Stable using code and see if invoking the Git:
Show Git Output from the command palette resolves the issue?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/138278#issuecomment-991236779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARDH3MCOL2K242OL4TOQTUQJH77ANCNFSM5JE6BCZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

whereis git says:
@.*** foo % whereis git
/usr/bin/git
Met vriendelijke groet,
Esger Jellema
Voermanhaven 19
9742 VR  Groningen
M 06 49 68 68 93
http://nl.linkedin.com/in/esgerjellemaProfiel van Esger Jellema
weergeven http://www.linkedin.com/in/esgerjellema
Op vr 10 dec. 2021 om 11:29 schreef Ladislau Szomoru <
@.***>:

@Esger https://github.com/Esger, sorry for not getting back to you on
this until now.
Can you please confirm that you are unable to initialise a repository
using the following steps:

Create a new folder foo
In terminal navigate inside the foo folder and type code-insiders .
Open the SCM viewlet and click on the "Initialize Repository" button.
If it fails, can you please share the contents of the git output?

In the terminal could you also please run whereis git and send me the
output.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/138278#issuecomment-990824053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARDH3SLIN4EYFD5UIQOG3UQHJAFANCNFSM5JE6BCZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I was experiencing the same problem with 1.64.0-insider (Universal). Running from the command line, with and without --disable-extensions resolved the issue. Executing the Git: Show Git Output command also resolved the issue. Other related issues seem to be 138862 and 138754.
By disabling and enabling my extensions, I narrowed the suspect extension to Azure Repos. This extension is deprecated, and disabling or removing it resolved the issue.

@TrevorKarjanis thanks for sharing. Will be closing the issue for now. Please reopen if the issue reoccurs.


performance improvement for ModelSupport.getElementInfoMap

While investigating startup performance, I found that ModelSupport takes
a long time (~2 seconds) to initialize.
The most obvious spot for improvement was ModelSupport.getElementInfoMap
which is:

reading a file that contains class names for all model classes
finding those using Class.forName
iterating this list twice, and
reflecting over each class and its parents (even if they've already
been visited in a previous iteration)

I addressed much of that, but we're still reflecting over hundreds of
classes and so its still not fast. In my local testing, the method is
about twice as fast now, but it still takes over a second to initialize.
Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

I think the rationale behind going with a modelClasses file was that we could potentially generate it from fhir-tools and then when the model changed (FHIR version rev) we could be mostly up-to-date. Since model revisions are not very frequent this optimization makes sense to me.


Binary versions are showing wrong in neovim :checkhealth for providers

Version 10.0.19559.1000
Running :checkhealth on neovim causes the issue
Binary version numbers have "^[[H^[[J" characters in front of them. I tried using both packages from repository (for the binaries and neovim) and a version manager (like asdf). It should report the correct version number (e.g. v10.19.0). For nodejs I think the code that queries the version uses the command like this
https://github.com/neovim/neovim/blob/master/runtime/autoload/health/provider.vim#L625

That line of code isn't going to work if node emits is CRLF. Which is how win32 node behaves by design.

Educated guess you don't have node installed in WSL.

I tried both nodejs from the repositories and with asdf version manager. Same result. I know node is installed on wsl because I am currently using it.

Can't repro here (made me look). I assume the output is supposed to look like this.
 Whatever the reason it is almost certainly going to be a userspace thing. Termcap related possibly.

Can you try after installing npm neovim package with 'npm install -g neovim' ? Because the main version issue is with node version, python seems ok.

No effect. It wouldn't. That ESC H ESC J is a cursor positioning sequence. Try export TERM=vt220 for the sport of it. Or export TERM=linux, which is roughly the same thing. Going to make a wild guess (maybe incorrect) you are not running an Ubuntu install from the Store, is that about right?

Yes, I'm using Ubuntu from the store now it became "^[[3^[[H^[[J". And I realized, anything I echo in neovim command like ":!echo $PATH" has "^[[3^[[H^[[J" in front of it.

A simpler repro than nvim(1) might be:
$ printf "\033[H\033[Jhello\n"

Which should print hello in the upper left corner of your screen. [For you, probably, it won't.]
What was the result (if anything) of setting TERM=vt220 before running nvim(1) or the above printf(1).

Yes, I'm using Ubuntu from the store now

Thanks; that's a useful data point. Now it is down to guessing your diverge versus a clean install. You could try moving your .bashrc out of the way (mv ~/.bashrc ~/bashrc.bak, start another shell) to see if the culprit is in there. It might not, but it is a good usual suspect. That or the problem is somehow on the terminal emulator (read: console) side, although absent externalities the Windows Console will interpret those escape sequences fine.

printf worked as normally (hello on the upper corner). After you said to start a clean shell I realized I was using tmux. I suspected it might be the problem so running without tmux the problem was solved. I was using someone else's tmux config(https://github.com/gpakosz/.tmux) on a clean tmux there is also no problem. So somewhere in that guys conf there is a line that breaks it for wsl. That tmux conf is not so important for me so for now I can say the problem is solved. Thanks

Trying to figure out what was wrong with the conf. I realized that in my config.fish file (i am using fish as default shell) I have these lines so that the session always start with tmux
if status is-interactive
and not set -q TMUX
    exec tmux
end
clear

if i comment those lines, even if I use the previous tmux conf everything is fine. Those lines were the culprit.

always start with tmux

Heh; I had guessed you failed to mention tmux earlier, but it worked under clean tmux for me too, so I assumed that wasn't it. Couldn't account for a tmux config tho.

So somewhere in that guys conf there is a line that breaks it for wsl.

It isn't a "for WSL" thing. Neither WSL nor a Real Linux kernel has any idea what those escape sequences mean, or why tmux or fish (or anything else) writes what it writes down a pty.
It might be the terminal emulator, possibly (maybe,hypothetically,in theory). You could experiment with other emulators. MSFT has a new Terminal. I use ttyd (based on xterm.js) sometimes.
Allowing for possibility "something else".

so for now I can say the problem is solved. Thanks

Glad you're solved!

It is weird that when I start tmux myself it works but with those lines it doesnt. Anyways its not a big deal to write 4 characters everytime I start the emulator. After that next tabs/panes start with tmux anyways.
I am already using Microsoft Terminal btw. I'll try other emulators as well.


[v1.3.0] Allow messages to be animated (Closes #7)

This PR adds a very basic animation to the NotificationMessage add/remove operations. I added options for making a message animate (defaults to false) as well as the animation duration (defaults to 0.25).
The one tricky thing here that I had to do was modify the Generic.xaml layout for the container such that the animations looked good on a dark background as well as a light background. Otherwise, on a light background, the background for the container would show up first while the message animated in, and that looked bad. I'm open to alternatives.
This does adjust the INotificationMessage interface, so this should probably be considered a breaking change in case anyone out there is implementing that interface.
Closes #7.

Looks great.
Couple of question though:

Could we extracted the animation properties from INotificationMessage to eg. INotificationMessageAnimation? We would then implement both interfaces in NotificationMessage class to keep backward compatibility.
Can we seperate the current DoubleAnimations you are using to another file and add AnimationIn and AnimationOut properties so that user can hook up other custom animations (slide for example), what do you think about that?

@AleksandarDev How's that? I changed INotificationMessageAnimation to INotificationAnimation since it technically could be used (for whatever reason) by non-messages, plus it's a little less verbose. I also wasn't sure what to name the properties for the animation in/out DependencyProperty items, so I went with AnimationInDependencyPropProperty, which is better than PropertyProperty or something 😁.

Yeah, this looks great.
Can you please add summary comments to the interface and methods you added.
Also, we're using this  for all class level methods, properties and fields when accessing (for example this.RemoveMessage(args.Message); instead of RemoveMessage(args.Message);) so if you could update that too?
I'll update the readme, changelog, version and nuspec once you do these changes and we're ready to publish.

@AleksandarDev Everything should be commented, and I added the missing this accessors. The only other change I made was grabbing the animation before checking if it was null in case it's dynamically generated on get like ours is so that it's not created twice.

@ghost1372 This is not really related to the animation feature. Can we continue discussion here #12?

@Deadpikle Thank you for the contribution. The v1.3.0 was published 🥂


Bug: Critical Error Report Blank

Issue:
Select the Evaluation option for configuration below, received Critical Errors after running the evaluation but not showing on the report. The report only shows system/component has an informational message.
Related to issue https://app.zenhub.com/workspaces/ecmps-prod-team-64623d2269329d0c96ba04f3/issues/gh/us-epa-camd/easey-ui/5799
Steps to Recreate

Check out Kyger Creek (2876) 1,2,CS12
Select QA & Cert / Test Data
Click on Import Data
Select Import Historical Data
Select 2022 Q4 Reporting Periods
Click Preview
Check all boxes on RATA test
Click Import
Select Evaluation
Click on Critical Errors Eval Status

Bug remediated. The system no longer displays critical error message. The system displays informational message


IRGen: inline the initialisers for members

Rather than use the explicit constructor initialiser list, use the inline initialiser. This makes it more obvious which values are initialised and which are not as well as avoids a huge block of initialisation. This also makes it easier to adjust the options without having to reflow text.

@swift-ci please test

This is a C++20 feature, the windows build will likely fail as we do not pass /std:c++20 currently.


Update RSpec::Mocks.with_temporary_scope to return the result of the passed block.

This implements #1328.

Thanks!


Specifying a user

It appears it is now possible to specify a "User" when creating a subscription. What does this do, create a pre-approved subscription for that user specifically?
Please update the doc screenshot and cover this option.

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

ID: a9b18e22-18d0-f82d-cbd0-e9ca300ee655
Version Independent ID: 2737fc2f-cb47-c881-e2ca-1eba4efa26d2
Content: Create subscriptions in Azure API Management
Content Source: articles/api-management/api-management-howto-create-subscriptions.md
Service: api-management
GitHub Login: @miaojiang
Microsoft Alias: apimpm

@dcbrown16 Thanks for reaching out. We will review and update as appropriate.

@dcbrown16 Could you elaborate more on your ask where do you see the option specify a "User" when creating a subscription.

Here are the steps to get there - #3 is the menu item in question.

@dcbrown16 I have assigned this to content author to review and update as appropriate.

@dcbrown16 it creates a subscription for this particular user.

Updated the doc in PR #72839. #please-close

Thanks @miaojiang !

@DixitArora-MSFT reopening for follow-up.
@miaojiang there is a somewhat confusing related UI area that it would also be nice to address, here or in another document.
Under Products > (select one) > Subscriptions, we have both "Add subscribers" and "Add subscription" buttons, and the "Add subscription" button lets you specify a user in the same way as the path addressed earlier in this thread.
Are these understandings based on testing correct?
Add subscribers - lets you select multiple users, but auto-generates subscription name. If you select multiple users, it makes one subscription for each user.
Add subscription - you can select 0 or 1 users (presumably 0 would be with the intent of offering it as an approval-based subscription). You can specify whatever name you want for the subscription.
I know this is not the place for feature requests, so in addition to asking for confirmation and documentation of the two UI elements above, I'll just suggest that it would seem best if you could both specify a subscription name and choose any number of users at the same time, from one button.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: Microsoft Q&A. If you feel this is not the case we can re-open the issue. #please-close


add production env

Fix for https://github.com/Shopify/slate/issues/83
Partial fix for https://github.com/Shopify/slate-tools/issues/39
@macdonaldr93 @cshold @t-kelly Any concerns about this?  Alternatively, I just remove "live" as the initial value.

Merging this as a band-aid fix for the issues reported in https://github.com/Shopify/slate-tools/issues/39.
I recognize this requires an extra step when testing, but it's worth it to avoid having devs accidentally wipe their live site.  You can always write the env flag if you don't want to go looking for the theme id: slate deploy -e production
Also, now the config.yml has an example of how to create multiple environments in your config file. 😎


Add shape to clusters

Currently we can add shape only to nodes:
Node A = Node.builder().label("A").shape(shape).build();

However, it would be very nice if it were possible to add shapes to clusters. Although it is not a standard graphviz feature it is possible to do - https://forum.graphviz.org/t/clusters-outline-geometry-attribute/313/2
So, I suggest to add something like this
Cluster.builder().shape(shape)...

or like this (to show it is not standard) :
ShapedCluster.builder().shape(shape)...

The reason is that it is very often necessary to add shapes not only to nodes but also for containers: For example, in UML:

If this feature is implemented then I think it is also necessary to add possibility to add edges between clusters and between clusters and nodes.

Good idea, Maybe I can try and only need small changed for current code architecture, but the hardest part is that considered the overlap between clusters if have multi cluster shapes, because per shape which not rectangle should have outer and inner container size and should detected the outer container size as the original rectangle in anticipation of enough space to avoid overlap with other clusters.
For the cluster edges, both the Graphviz compound and graph-support Example GraphAttrTest#compoundCases are supported line cut by cluster border, but PlantUML used another heuristic way to implemented similar request by standard Graphviz, first split the original graph to multiple graphs without cluster and rendered one by one with nested order and detected rendered graph size as the size of parent node size to continue parent graph rendered, so maybe have another way to implement this request and without change original cluster feature.
We can adds Graph-Label feature learned from PlantUML heuristic method to perfect implemented the multi-shapes cluster and cluster edges request like following, and cluster concept transfer to node as the similar result but I think it is enormous challenge.
Node n = Node.builder()
    .shape(NodeShapeEnum.TRIANGLE)
    .graphLabel(
        Graphviz.digraph()
            ...
            .build()
    )
    .build()

If nested graph in node we can not connect "child" node to "parent" node, this is meaning of cluster existence. Cluster and nest graph complementary each other, so the criterion when choose nest graph or cluster is whether needed connect the inner node to outside node.
Because recursive graph it means node only care the child graph size rather than the layout, but cluster is a part of original graph and participated all layout phases like rank assigned/mincross/position, there are totally difference but seems like similar in some scenarios.
Following case show a different result when use "nested" and "cluster" respectively:

@PavelTurk Just a example with dot:
digraph G {
    compound=true
    rankdir=LR
    node[shape=rect]
    a->d[dir=back label="requires" ltail="cluster_0" lhead="cluster_2"]
    b->e[label="exports" lhead="cluster_2"]
    b->e[dir=back label="opens" ltail="cluster_0"]
    subgraph cluster_0 {
        label="ModuleA"
        subgraph cluster_1 {
            label="coo.foo"
            a[label="api"]
            b[label="internal"]
        }
    }

    subgraph cluster_2 {
        label="ModuleB"
        subgraph cluster_3 {
            label="com.bar"
            d->e[minlen=0 style=invis]
            d[style=invis fixedsize=true height=0]
            e[label="impl"]
        }
    }
}

First I will try multi cluster shapes and graph nested in the future, then I can give you some heavier solutions.
graphviz-java
If you want to use original graphviz in java (by the way graph-support is just use the same algorithm re-implementation but do not have any relationship with graphviz) you can try graphviz-java, it is compile the graphviz to Javascript and try to use V8 javascript engine (a js engine of java) to run the js version of graphviz, and it is a more proven solution rather than create JNI interface by yourself, it is obviously heavier but more close to graphviz than graph-support
PlantUml
PlantUML is a versatile component that enables swift and straightforward diagram creation, and it write by java and provide scripts and java api for users. PlantUML use the graphviz as the layout engine to caculated the node coordinate but have independent rendered engine. I think PlantUML it is good choice because it have richer features and already used multi ways to integrate graphviz and do downgrade by your enviroment:

If download graphviz on local environment and will caculate by local (same as graphviz-java);
If not download graphviz, try use another project smetana(a crazy project try to translate c code to java code, but not done as I know) if this project can cover the functions;
If detected the java js engine and use the Vizjs(same as graphviz-java).

Finally, only the PlantUML + smetana is not have other dependency seems like graph-support, so you can try all these solutions to integrate to your project without create JNI by yourself and hope helpful for you.

All shapes consists of inner and outer boxes, all elements in shape should within inner box. graph-support use ShapePropCalc#minContainerSize to describing size relationship between inner and outer boxes and auto calculate right size to node.
So more safe that not overflow elements if more less gap of size between inner and outer boxes for specific cluster shape.
The shape STAR more easy occurred overflow issue because of large gap between inner and outer boxes liking following example:

@PavelTurk Latest version supported multi cluster shape but have limitation I listed below, Cluster shapes except ClusterShapeEnum#RECT no guarantee that cluster container will surround all nodes under Layout.DOT engine but will try best estimated the container size.
Both node shape and cluster shape can extension by CustomizeShapeRender to customized the specific shape by yourself.


Fix connect to third party services flexbox

Fixes #5471

CR @xavijam

An image in the description would be nice

Also, what is the staging server to test this thing :dancer: ?

When there is an error appears like:

All problems fixed! :sparkles:

:+1:


migrate typechain to have a single file comprising all the fns available for a contract

at the moment these are duplicated over "queries" and "tx". It should just be one file, call as appropriate. fns which are a tx should have some kind of gas associated in the params, whereas queries shouldn't

What is the status of typechain problems like this?

@forgetso Please add this under the epic for typechain proposal

Refinement ^^^


Unable to compile | Invalid operands error

Hi,
I am trying to compile the codebase as instructed by using g++ --std=c++11 src/cpp/main.cpp src/cpp/DeslantImgCPU.cpp pkg-config --cflags --libs opencv -o DeslantImg. However I run into the below mentioned error to which I am unable to find a solution.
In file included from src/cpp/DeslantImgCPU.cpp:1: In file included from src/cpp/DeslantImgCPU.hpp:2: In file included from /usr/local/Cellar/opencv/3.3.0_3/include/opencv2/core.hpp:52: In file included from /usr/local/Cellar/opencv/3.3.0_3/include/opencv2/core/cvdef.h:424: In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/array:109: /Library/Developer/CommandLineTools/usr/include/c++/v1/algorithm:719:71: error: invalid operands to binary expression       ('const htr::Result' and 'const htr::Result')     bool operator()(const _T1& __x, const _T1& __y) const {return __x < __y;}
Having already overridden the '<' operator for the struct Result, I am not sure how to rectify the same. Looking forward to your help on resolving this. TIA.
System details
OS X 10.13.4 | OpenCV 3.3 | Apple LLVM version 9.1.0

I'll come back to you next week.

please try to make the operator < method (DeslantImgCPU.cpp, line 17) constant:
bool operator < (const Result& rhs) const { return sum_alpha < rhs.sum_alpha; }

@githubharald : Worked like a charm. Thanks.


Edge Agent - Docker snapshots should use timeouts

Issue
When the Agent tries to build a snapshot of a Docker environment there is no timeout for those requests, this could potentially lead to the Agent getting stuck forever (or too long) if the Docker daemon is unresponsive.

PR -> https://github.com/portainer/agent/pull/522


JEST - Cannot find module 'single-spa-react/parcel' from 'root.component.js

My application is running well and no errors are ocurring.
But when I run the test suite, I got this:

I tried to import directly from single-spa-react/lib/esm/parcel but with no success... :/
"devDependencies": {
    "@babel/core": "^7.12.9",
    "@babel/plugin-transform-runtime": "^7.12.1",
    "@babel/preset-env": "^7.12.7",
    "@babel/preset-react": "^7.12.7",
    "@babel/runtime": "^7.12.5",
    "@testing-library/jest-dom": "^5.11.6",
    "@testing-library/react": "^11.2.2",
    "@types/jest": "^26.0.16",
    "@types/systemjs": "^6.1.0",
    "babel-eslint": "^11.0.0-beta.2",
    "babel-jest": "^26.6.3",
    "concurrently": "^5.3.0",
    "cross-env": "^7.0.3",
    "eslint": "^7.15.0",
    "eslint-config-prettier": "^7.0.0",
    "eslint-config-react-important-stuff": "^3.0.0",
    "eslint-plugin-prettier": "^3.2.0",
    "husky": "^4.3.5",
    "identity-obj-proxy": "^3.0.0",
    "jest": "^26.6.3",
    "jest-cli": "^26.6.3",
    "prettier": "^2.2.1",
    "pretty-quick": "^3.1.0",
    "single-spa-react": "^3.2.0",
    "webpack": "^5.8.0",
    "webpack-cli": "^4.2.0",
    "webpack-config-single-spa-react": "^2.0.0-beta.1",
    "webpack-dev-server": "^4.0.0-beta.0",
    "webpack-merge": "^5.4.0"
  }

Please upgrade to single-spa-react@4, which fixed all the types. See https://github.com/single-spa/single-spa-react/releases/tag/v4.0.0. This test shows that parcel imports are working, and I also confirmed it locally in a sample project.

We have been running into this issue as well when updating from 2.14.0 to 4.1.1. We are able to run the tests when we change the import to single-spa-react/lib/umd/parcel but then the production build fails.
It is also possible to reproduce this problem using a brand new project created using<EMAIL_ADDRESS>Options:

react
app/parcel
yarn
typescript

After that has run all we have to change is the root component and run yarn test.
import React from "react";
import Parcel from 'single-spa-react/parcel'

export default function Root(props) {
  return
<Parcel config={()=>{return System.import('')}}/>
;
}

P.S. When running create-single-spa we get this message A patch update of create-single-spa is available: 2.3.1 → 2.2.2 but a 2.3.x version is not available for download.

Using 2.2.3 of create-single-spa fixes the warning but the issue remains.

It's possible jest's module resolution algorithm does not yet support nodejs package entry points, which are required for single-spa-react/parcel to correctly resolve to the node_modules/single-spa-react/lib/cjs/parcel.cjs file that should be used, as specified in this code.
See https://github.com/facebook/jest/issues/9771 where this issue with jest is being discussed.
To workaround, you can use moduleNameMapper
// jest.config.js
module.exports = {
  moduleNameMapper: {
    "single-spa-react/parcel": "single-spa-react/lib/cjs/parcel.cjs"
  }
}

That config will get you close, but might not work exactly. If it doesn't, try using node_modules/single-spa-react/lib/cjs/parcel.cjs as the file path in the moduleNameMapper.

If you get it working, a contribution adding this to create-single-spa would be welcome.

I will give it a shot.


Fix environment specific header termination on Unix (issue #483)

Doneness:

[x] Build is okay - I made sure that this change is building successfully.
[x] No Bugs - I made sure that this change is working properly as expected. It doesn't have any bugs that you are aware of.
[ ] Branching - If this is not a hotfix, I am making this request against master branch

:white_check_mark: Build Titanium-Web-Proxy 3.0.649 completed (commit https://github.com/justcoding121/Titanium-Web-Proxy/commit/4fc8dccd5c by @)

:white_check_mark: Build Titanium-Web-Proxy 3.0.650 completed (commit https://github.com/justcoding121/Titanium-Web-Proxy/commit/6509cb721a by @)


Organization identifiers use deprecated fields

For example, buyer_summary.buyer_identifier is constructed from buyer -> 'identifier' -> 'scheme' and `buyer -> 'identifier -> 'id'.
Similarly, procuringEntity_summary.procuringentity_identifier, tenderers_summary.tenderer_identifier and award_suppliers_summary.suppliers_identifier are also constructed using the identifier block which has been deprecated outside the parties section.
This means that these fields are NULL for OCDS 1.1 data, e.g (from @camilamila):

Using collection 193 (from Chile) with ocid = ocds-70d2nz-5413-181-L117:
views.parties_summary.identifier= CL-MercadoPublico-6326
views.parties_summary.party.id= 70.772.100-6
views.procuringEntity_summary.procuringentity_parties_id = 70.772.100-6
views.procuringEntity_summary.procuringentity_identifier = NULL

@kindly I'm not sure if this was planned (for OCDS 1.0 support) or unintentional, but given that we upgrade all data to 1.1 in the process pipeline, I think these fields should be constructed from the organization identifier in the parties section.
Edit: This also applies to the unique_identifier_attempt fields for organization references.

award_suppliers_summary.supplier_additionalidentifiers_ids, award_suppliers_summary.supplier_additionalidentifiers_count and likely others also read additionalIdentifiers, etc. from the OrganizationReference in awards.suppliers rather than from the Organization in parties.

I think the solution is to join with parties_summary.

 Views: Add new aggregates/fields

 Views: Several queries use deprecated fields instead of parties array

Yep we need to join to parties and we do just not getting the data from correct place.  We should probably only join if the id join is correct not the unique_identifier_attempt.

@kindly Please re-read the create table staged_award_suppliers_summary statement, viz my comments in https://github.com/open-contracting/kingfisher-views/issues/45#issuecomment-593769081. My reading is that supplier is a OrganizationReference object in awards.suppliers, not an Organization object from parties.

@jpmckinney  sorry last message not clear.  I agree that it is currently wrong.
The information should probably got from parties_summary table.  There is a join to that table (in order to get parties_index) in those queries but the information is not got from there and is using the 1.0 deprecated fields.  This was by design.
The above point was about how we join to the parties_summary table.  Currently we are using the id field and but it is possible to join using unique_identifier_attempt. I still think we should just use the id field so the above changes should be easy as just referring to the already created fields in the parties_summary table instead.
I think the original idea was to leave it up to the analysts to see how they wanted to join to the parties_summary table (using either parties_index or unique_identifier_attempt).   As that was the main use of unique_identifier_attempt in the first place.   This was when lots of people were doing parties badly.

It looks like it's going to be fairly easy to maintain 1.0 compatibility (fall back to the old query), is this worth doing? The kingfisher live server upgrades data to 1.1, but it looks like our vagrant dev setup doesn't out of the box (an alternative could be to change that).

I don't think we need to support 1.0. I think it's fine to require that data be upgraded, which users can either do via Kingfisher Process's transforms, or by using OCDS Kit or Toucan before local-loading into Kingfisher Process.


remove coroutine in access phase

We introduce this in
https://github.com/Kong/kong/commit/744d3b32e47323301bdbe402f9bf1cb2acbf88e8
to persist the behaviour that kong.response.exit can shortcut current
processing. Though this is also documented in PDK document, our code
doesn't seem to rely on it at all.
There are some patterns mentioned https://github.com/Kong/kong/pull/8951#issuecomment-1173592847
that may affect current behaviour.
Originally opened in #8951, needs more work to finish backward compatibility.
Proposed to include in 4.0 miletone.

Dear contributor,
We're closing this issue as there hasn't been any update to it for a long time. If the issue is still relevant in the latest version, please feel free to reopen it. We're more than happy to revisit it again.
Please have a look at our pledge to the community for more information.
Sincerely,
Kong Gateway Team


Implement read_many

When serializing to datacite, having a read_many to retrieve multiple subjects (vocabularies) at once would be great.

read many is available since invenio-vocabularies uses invenio-records-resources

@ppanero that's exactly what I mean. read_many is different for vocabularies because the ids are actually ('<type>', <id>) . So a vocabulary-specific read_many is needed.


HTTPS tests are inaccurate for Firefox

On the website https://privacytests.org, Firefox is listed as not upgrading HTTP connections to HTTPS or displaying a warning for insecure sites (see the included screenshot), but it has had that ability since 2020, with the addition of HTTPS-Only mode in version 83, and it is still present today in version 131 (see the included screenshot from Firefox 131's settings page).  Documentation about the feature can be found here.
https://privacytests.org screenshot:

Firefox 131 screenshot:

Hi Jesse, thanks for the comment. The reason you are seeing these test failures is that PrivacyTests looks at the behavior of a browser when it is set to default settings only.
It's true that Firefox offers HTTPS-Only Mode, but unfortunately that feature is disabled by default, which means it fails the Insecure website warning test in both Normal and Private windows.
Firefox Private mode does have HTTPS-by-Default enabled in Private Windows, which means that it passes the Upgradable address test for Private Windows, but this feature is also not enabled in Normal windows, so Firefox fails that test in the case you point out above.
Hopefully Firefox will upgrade its default HTTPS handling in the near future so that it passes these tests!

I see.  That does make sense.  I wasn't aware that the tests only looked at default, and thought it looked at whether or not a browser was capable of protecting against each type of leak.


example of running with a config

an area showing that you can run oragono with a different config (which is great, thank you!) and that you can configure the yaml as well to change paths (I didn't want to enumerate all the settings, but the awareness of such is hopefully useful to others!)

Between this an #113 - I figured you'd have some specific formatting but it's easier to know what you want once your bootstrapped with maybe something you don't want as much!

Definitely, all good! I tend to be a stickler for formatting and all so I go a bit mental with the revisions sometimes. I really appreciate the work either way, it's awesome to have contributions :)


Checks if resource exists in datastore before running validation

What this PR accomplishes
Works together with https://github.com/ongov/ckanext-ontario_theme/pull/619.
Because existing CSV resources on the catalogue have never been through validation, some may not actually be in the datastore. If these existing but not-existing-in-datastore CSV resources get sent for validation, an error will occur since validation calls the resource from the datastore (e.g. https://github.com/ongov/ckanext-validation/blob/develop/ckanext/validation/logic.py#L88).
The solution is to check if an existing CSV resource exists in the datastore before sending it for validation.
Issue(s) resolved
DATA-1630
What needs review

Run tests on existing CSV resources that are not in the catalogue (these are usually remote WEB or remote CSV resources)
Check that regular sanity tests pass (on local and remote file types)

looks good1 👍


Unable to connect to MariaDb

Component
server
Describe the bug
Using the configuration from the docs, I'm unable to start woodpecker, using MariaDB.
Other containers on the same network can reach the database container without problems.
System Info
devops_woodpecker_ci:
    image: woodpeckerci/woodpecker-server:${WOODPECKER_TAG}
    restart: unless-stopped
    ports:
      - ${WOODPECKER_PORT}:8000
      - 9000
    networks:
      - devops_network
    environment:
      - WOODPECKER_DATABASE_DRIVER=mysql
      - WOODPECKER_DATABASE_DATASOURCE=DBUSER:DBPASSWORD@tcp(DBHOST:3306)/DATABASE?parseTime=true
      - WOODPECKER_OPEN=true
      - WOODPECKER_HOST=${WOODPECKER_URL}
      - WOODPECKER_GITEA=true
      - WOODPECKER_GITEA_URL=${GITEA_URL}
      - WOODPECKER_SECRET=${WOODPECKER_SECRET}
    depends_on:
      - devops_woodpecker_db

Additional context
Logs:
time="2021-10-10T10:59:20Z" level=error msg="unable to open database file: no such file or directory",
time="2021-10-10T10:59:57Z" level=fatal msg="database ping attempts failed"

Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server.

@Flying--Dutchman what WOODPECKER_TAG did you use?!!


blas/gonum: generate complex64 implementation

Please take a look (per commit).

What do you envision for the tests? Write them manually or try to generate them from float64/complex128 tests?

I was hoping to do it mechanically. If we square root the tolerances we probably have a pretty good approximation of what will work. I did this for some testing of OpenBLAS a while back and it was OK. The problem is how much stuff there is.


Add log section to webhook show page

Closes #364

Coverage remained the same at 88.246% when pulling a74d5af3af3669976afd77a6076d0f0355d27338 on feature/show-webhook-logs into 73d9c386904aa92901e604205a089a7ec0e4cbdc on main.


Delete user coordinates after he leaves trip

When a user leaves a trip - the server should delete his GPS coordinates from the DB.

opened on backend repo


Tag Task Azure DevOps Server 2019 Compatibility?

I added the Tag Build or Release Task to my build pipeline but it fails with the following error:
Could not find token for autheniticating. Please enable OAuth token in Build/Release Options.
I did a bit of digging and it seems like I might need to enable "Allow scripts to access the OAuth token" in my build options? Although, that option doesn't appear on my build options tab. In any case do you have any insight as to what could be going on? Is the task compatible with Azure DevOps Server 2019?
Thanks!

Hi @Celaires - I'm guessing you're using YAML build and not the Classic (UI), right? The "OAuth token" is something you enable in the UI. To enable it in YAML, you need to add persistCredentials to the checkout task, like so:
steps:
- checkout: self
  persistCredentials: true

The persistCredentials: true places the OAuth token into a system variable, SYSTEM_ACCESSTOKEN which is what the Tag Build or Release Task uses internally.
See this issue.
Let me know if that unblocks you!

Ok, that's kind of what I stumbled across this morning too. However, I am using the Classic (UI) and there doesn't appear to be an option persist credentials or "Allow scripts to access the OAuth token".

Ha, of course, as soon as I commented I found the the "Allow scripts to access the OAuth token" setting (it's kind of hidden in the UI). Things are working now. Thanks for your help!


Missing "each" in modifications in MSL 2.2.1 and MSL 3.0

Reported by stefanv on 17 Oct 2008 13:34 UTC
In both MSL 2.2.1 and MSL 3.0 in Modelica/Blocks/Sources/KinematicPTP, there is the following declaration (and a few more like it):
parameter Real qd_max[:](final min=Modelica.Constants.small) = {1};
It should be:
parameter Real qd_max[:](each final min=Modelica.Constants.small) = {1}

Migrated-From: https://trac.modelica.org/Modelica/ticket/113

Comment by dietmarw on 17 Oct 2008 13:49 UTC
Taking into account the new procedure of default values this shoul probably be changed to
parameter Real qd_max[:](each final min=Modelica.Constants.small, start={1});

right? And I would suggest fixing this for MSL 3.0.1 Milestone

Modified by dietmarw on 17 Oct 2008 13:53 UTC

Comment by otter on 25 Oct 2008 06:59 UTC
Blocks.KinematicPTP, Blocks.KinematicPTP2:
Added missing "each" as
parameter Real qd_max[:](each final min=Modelica.Constants.small)

in trunk (9c2d1594a71ddb8321ed968e2182e56f06519fcf) and in maintenance 3.0 (5d2f67a0b5869768413f4164b17972bdc1806df8) and as
parameter Real qd_max[:](each final min=Modelica.Constants.small) = {1}

in maintenance 2.2.2 (64a9491f8438e8329c653a903c413a6a2065bcbd) and 2.2.1 (9d545ff9f2c7f6e14fc2c505da5317a60e3dccfb)

Modified by otter on 21 Jan 2009 11:08 UTC


Added better 'successful clone' message.

I think we are done with the clone stuff.  This final pull request has better checking to confirm that all subjects in the triples no longer contain the previous resourceURI.  If the checking passes, then we get a nice blue info message telling the user the newly minted ID of the instance.  If there's a problem, then we get an old fashioned alert-- but this will never happen.

I will await Kirk's approval before merging.


Added policy for KMS and "enable" flag

Brought master branch in line with 0.11/master branch

/codefresh run test

addressed in https://github.com/cloudposse/terraform-aws-kms-key/releases/tag/0.3.0


Error with spacial functions in projection

OrientDB Version: 3.0.15
Java Version: 1.8
OS: Windows 8
Expected behavior
Some result with the distance value
Actual behavior
java.lang.NullPointerException
Steps to reproduce
Code similar to #8051
CREATE class Restaurant;
CREATE PROPERTY Restaurant.name STRING;
CREATE PROPERTY Restaurant.location EMBEDDED OPoint;

INSERT INTO  Restaurant SET name = 'London', location = St_GeomFromText("POINT (-0.1277583 51.5073509)");
INSERT INTO  Restaurant SET name = 'Trafalgar', location = St_GeomFromText("POINT (-0.1280688 51.5080388)");
INSERT INTO  Restaurant SET name = 'Lambeth North Station', location = St_GeomFromText("POINT (-0.1120681 51.4989103)");
INSERT INTO  Restaurant SET name = 'Montreal', location = St_GeomFromText("POINT (-73.567256 45.5016889)");

CREATE INDEX bla ON Restaurant (location) SPATIAL ENGINE LUCENE;

/* should return values*/
SELECT  ST_Distance_Sphere(location, St_GeomFromText("POINT (-0.1277583 51.5073509)")) as dist from Restaurant;

Works well without using fields in the function
SELECT  ST_Distance_Sphere(St_GeomFromText("POINT (-0.1277583 51.5073509)")), St_GeomFromText("POINT (-0.1277583 51.5073509)")) as dist from Restaurant
Similar probleme with ST_Distance but response is different
SELECT  ST_Distance(location, St_GeomFromText("POINT (-0.1277583 51.5073509)")) as dist from Restaurant
Response

org.locationtech.spatial4j.exception.InvalidShapeException: can't make Geometry from: null

Stacktrace

java.lang.NullPointerException
at com.orientechnologies.spatial.functions.OSTDistanceSphereFunction.execute(OSTDistanceSphereFunction.java:52)
at com.orientechnologies.orient.core.sql.parser.OFunctionCall.execute(OFunctionCall.java:136)
at com.orientechnologies.orient.core.sql.parser.OFunctionCall.execute(OFunctionCall.java:90)
at com.orientechnologies.orient.core.sql.parser.OLevelZeroIdentifier.execute(OLevelZeroIdentifier.java:60)
at com.orientechnologies.orient.core.sql.parser.OBaseIdentifier.execute(OBaseIdentifier.java:70)
at com.orientechnologies.orient.core.sql.parser.OBaseExpression.execute(OBaseExpression.java:120)
at com.orientechnologies.orient.core.sql.parser.OExpression.execute(OExpression.java:110)
at com.orientechnologies.orient.core.sql.parser.OProjectionItem.execute(OProjectionItem.java:147)
at com.orientechnologies.orient.core.sql.parser.OProjection.calculateSingle(OProjection.java:115)
at com.orientechnologies.orient.core.sql.executor.ProjectionCalculationStep.calculateProjections(ProjectionCalculationStep.java:66)
at com.orientechnologies.orient.core.sql.executor.ProjectionCalculationStep.access$000(ProjectionCalculationStep.java:13)
at com.orientechnologies.orient.core.sql.executor.ProjectionCalculationStep$1.next(ProjectionCalculationStep.java:41)
at com.orientechnologies.orient.core.sql.parser.OLocalResultSet.next(OLocalResultSet.java:69)
at com.orientechnologies.orient.core.sql.parser.OLocalResultSetLifecycleDecorator.next(OLocalResultSetLifecycleDecorator.java:56)
at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommand.execute(OServerCommandPostCommand.java:97)
at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommandGraph.execute(OServerCommandPostCommandGraph.java:36)
at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.service(ONetworkProtocolHttpAbstract.java:223)
at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.execute(ONetworkProtocolHttpAbstract.java:702)
at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:69)

I get same results with:
OrientDB Version: 3.0.17
Java Version: 1.8
OS: Windows 10

Same here:
OrientDB Version: 3.0.23
OS: macOS 10.14.6 (18G95)


error out when comment at the head of first line

If I comment at the head of first line with the comment plugin registered:
// log(11)
log(22)

jsep will throw error when parsing:
Error: Unexpected "l" at character 11
    at Jsep.throwError (jsep.js:327)
    at Jsep.gobbleExpressions (jsep.js:424)
    at Jsep.parse (jsep.js:385)
    at jsep (jsep.js:1040)
    at evaluate.js:50
    at __WEBPACK_DEFAULT_EXPORT__ (evaluate.js:49)
    at exec (inityxy.js:263)
    at HTMLUnknownElement.callCallback (react-dom.development.js:188)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:237)
    at invokeGuardedCallback (react-dom.development.js:292)

but it works fine if I swap the lines:
log(22)
// log(11)

parsed as:
{
    "type": "CallExpression",
    "arguments": [
        {
            "type": "Literal",
            "value": 22,
            "raw": "22"
        }
    ],
    "callee": {
        "type": "Identifier",
        "name": "log"
    }
}

Thanks @zclc - I guess my unit test was not written correctly, which is why I couldn't see it! 🤦🏻♀


feat: add suggestions in FeedbackRecord

Description
As of the recent addition of the suggestions in the Argilla API at https://github.com/argilla-io/argilla/issues/3277, we make sure that's covered in both the SDK and the Python client.
So on, this PR adds the following:

suggestions included in FeedbackRecord
Update FeedbackRecord to "listen" for updates (for the moment just applies to suggestions, but may eventually be extended to external_id, metadata, and responses)
Add add_suggestions function in the SDK
Add include=responses and include=suggestions as default in get_records
add_question and add_field now return the schema, QuestionSchema and FieldSchema, respectively (to get the Argilla ID of those, required to log suggestions as we use the Argilla Question ID)
Extend pydantic.BaseModels to add the id attribute for anything Argilla-related
Constrain pydantic.BaseModels to disallow extra fields and have more control over those
Handle suggestions on both pull and push from Argilla
Handle suggestions on both pull and push from HuggingFace

Additionally, I've also included some missing type-hints, improved question_by_name and field_by_name methods, fix some return type-hints, rename "FeedbackTask dataset" occurrences to "FeedbackDataset", and some other minor fixes.
Closes https://github.com/argilla-io/Team/issues/115
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested

[X] Add unit tests for the newly included suggestions attribute in FeedbackRecord
[X] Add unit tests for the add_suggestions function in the SDK
[X] Add unit tests for FeedbackDataset.records related functions/methods

Checklist

[ ] I added relevant documentation
[X] follows the style guidelines of this project
[X] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[ ] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

This PR should be merged right after https://github.com/argilla-io/argilla/pull/3391