mix ecto.migrate: "migration runner process"

Hello,
Just started to use Elixir and Phoenix. I'm currently doing the simplest of thing for a web app but look like my ecto is acting up. Trying to migrate to my database.
I've try to reinstall my ecto deps.
Environment

Elixir version (elixir -v): 1.2.2
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): pgsql 9.4
Ecto version (mix deps): 1.1.4
Operating system: Windows 10

Current behavior
mix ecto.migrate
(RuntimeError) could not find migration runner process for #PID<0.46.0>
(ecto) lib/ecto/migration/runner.ex:70: Ecto.Migration.Runner.prefix/0
(ecto) lib/ecto/migration.ex:665: Ecto.Migration.__prefix__/1
(ecto) lib/ecto/migration.ex:270: Ecto.Migration.create/1
(stdlib) erl_eval.erl:669: :erl_eval.do_apply/6
(elixir) lib/code.ex:321: Code.load_file/2
(ecto) lib/ecto/migrator.ex:221: anonymous fn/4 in Ecto.Migrator.migrate/4
(elixir) lib/enum.ex:1088: Enum."-map/2-lists^map/1-0-"/2
(ecto) lib/mix/tasks/ecto.migrate.ex:63: anonymous fn/4 in Mix.Tasks.Ecto.Migrate.run/2
(elixir) lib/enum.ex:604: Enum."-each/2-lists^foreach/1-0-"/2
(elixir) lib/enum.ex:604: Enum.each/2
(mix) lib/mix/cli.ex:58: Mix.CLI.run_task/2

Expected behavior
well run would be a good behavior.

Can you provide a simple app that reproduces the error? I am thinking you are using tasks or something of sorts in your migrations.

Please let me know if you can have a way to reproduce this because I cannot. :) I will gladly reopen the issue once we have a mechanism, thank you!

Hey Jose, I've just been speaking with gazler on irc about this, and I've had the same problem and resolved it.
It occurs when you try to do migration steps outside of the change function (or up/down functions i guess, I didnt try).
This migration replicates the issue:
defmodule Example.Repo.Migrations.Failure do
  use Ecto.Migration

  def change do
    create table(:testing) do
      add :name, :string
    end
  end

  create index(:testing, [:name])
end

Hope this helps you @Lemures

same here when a migration is created:
defmodule Rumbl.Repo.Migrations.CreateUser do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :name, :string
      add :username, :string, null: false
      add :password_hash, :string

      timestamps
    end
  end

  create unique_index(:users, [:username])
end

with model:
defmodule Rumbl.User do
  use Rumbl.Web, :model

  schema "users" do
    field :name, :string
    field :username, :string
    field :password, :string, virtual: true
    field :password_hash, :string

    timestamps
  end
end

throws backs:
mix ecto.migrate

** (RuntimeError) could not find migration runner process for #PID<0.71.0>
(ecto) lib/ecto/migration/runner.ex:70: Ecto.Migration.Runner.prefix/0
(ecto) lib/ecto/migration.ex:665: Ecto.Migration.prefix/1
(ecto) lib/ecto/migration.ex:270: Ecto.Migration.create/1
(stdlib) erl_eval.erl:670: :erl_eval.do_apply/6
(elixir) lib/code.ex:321: Code.load_file/2
(ecto) lib/ecto/migrator.ex:221: anonymous fn/4 in Ecto.Migrator.migrate/4
(elixir) lib/enum.ex:1183: Enum."-map/2-lists^map/1-0-"/2
(ecto) lib/mix/tasks/ecto.migrate.ex:69: anonymous fn/4 in Mix.Tasks.Ecto.Migrate.run/2
(elixir) lib/enum.ex:651: Enum."-each/2-lists^foreach/1-0-"/2
(elixir) lib/enum.ex:651: Enum.each/2
(mix) lib/mix/task.ex:296: Mix.Task.run_task/3
(mix) lib/mix/cli.ex:58: Mix.CLI.run_task/2

@BennyHallett  is correct:
doing this
defmodule Rumbl.Repo.Migrations.CreateUser do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :name, :string
      add :username, :string, null: false
      add :password_hash, :string

      timestamps
    end
    create unique_index(:users, [:username])
  end

  # create unique_index(:users, [:username])
end

fixes it and yields the expected:

10:53:44.183 [info]  create table users
10:53:44.464 [info]  create index users_username_index
10:53:44.510 [info]  == Migrated in 3.2s

I ran into a similar issue. The problem was user error — I had a malformed migration — but I think it would be helpful if the error message provided a better hint.

@codecakes, @BennyHallett  thank you guys, it solved my problem.

I had the same blasted problem.  Thanks for posting what was wrong!

The only fix I understand from the above is: make the index unique.

Is that correct?
what if I don't want a unique index?

@wdiechmann This issue doesn't have anything to do with a unique index, but rather that a migration didn't exist within a function.  Can't run migration steps at the model level.

@OvermindDL1
hmm - then how come when I changed create index to create unique_index the migration ran effortlessly?

Sounds like a different error then.  Should ask on the Elixir Forums.

I had forgot "def change" in my migration and that's the error I received - just as a hint.


SwaggerUI not showing endpoints

Hi there.  I have the Identity Server configured and I want to show the API details using Swagger (specifically using Swashbuckle).  How might we expose the existing APIs that the server offers, like token api, using Swagger?  Is this supported?
Thanks in advance.

I don't know anything about Swagger - but we don't use e.g. MVC for our endpoints. IOW - you would need to do that manually.
I don't see a lot of sense in doing that though. The documentation is the spec and if you need a tool to "play around", create a couple of scripts or use something like Postman.

There is a lot of value in providing swagger for your API. I highly reccomend you check out OpenApi. It serves as documentation as well as an api test harness which is very valuable when working with developers who dont understand the specification and just want to use the thing.
If the endpoints were created using an MVC controller then the framework will automatically provide documentation constructs for it for free, however it doesn't seem to be an actual controller.
I have a team of 30+ people (and growing) that are going to need to integrate with my identity server and having this sort of documentation and test harness is going to be very important. We are not all using dotnet core either, so all of the helper classes you have here won't help the teams using Java, Node, Python, etc...  That's where swagger comes in.
Thanks for the reply. I'll see what I can do about manually doing something to get it.

I think it might be big problem to just document OIDC/OAuth2 with OpenApi. If consumers first know OIDC/OAuth2, then there's no real need for OpenApi, and if there is then people should know where the specs are.

They just want/need to know how to integrate with it.

But that's what I mean by understanding the specs. This stuff is complicated, yes, but if folks don't appreciate the value and necessity in taking that time to learn some of if and how to do it the right way, then their software and companies will be the next front page story on how shitty our industry's security practices are.

I am having a similar issue.
Even do swagger-jsdoc does recognize the endpoints I declared in the JS files:
paths:     { '/api-gateway-express/health': { get: [Object] },      '/api-gateway-express/info': { get: [Object] },      '/api-gateway-express/version': { get: [Object] } },
when I pass it to the swagger-ui-express as a parameter those endpoints are not displayed on the page.
Its like swagger-ui-express ignores the parameter completely.
gatewayExpressApp.use('/api-gateway-express/api', swaggerUi.serve, swaggerUi.setup(swaggerJSDocJSON);
(For those who wonder, I am trying to do the same thing that can be done with restify-swagger-jsdoc)

FWIW, a recent blog post on supporting swagger in IdSvr: https://www.scottbrady91.com/Identity-Server/ASPNET-Core-Swagger-UI-Authorization-using-IdentityServer4

if you know a way to add Open API descriptions for the standard OAuth/OIDC endpoints, let us know then we can discuss this.

I doubt that there is a "nice" way of doing this, as the most popular swagger extensions rely on ApiExplorer, which in turn relies on the whole MVC routing / modelbinding to generate the Swagger doc.
The best bet is probably to just include a static OpenAPI spec with IdSrv. It won't change much anyhow, right? :)
On a side question: What's the reason you went with your own endpoint router? Were there technical limitations?

On a side question: What's the reason you went with your own endpoint router? Were there technical limitations?

Because any config we would need to do to MVC would affect any other MVC code in the same host. In short, ASP.NET Core is not designed to allow multiple instances of MVC (or other similar frameworks) due to the global DI/config architecture.

sounds good!

Exactly what I had in mind! Maybe I can make a static one and create a PR for it :-)
@leastprivilege What do you say?

@haseebahmed7 Did you work on this?  If not, I think I am going to work on it in the next week.

@amadard , @haseebahmed7 - did you implement this at the end? It would be really great to have

@guylevy I got started on it, but never finished it.   You asking is pushing me to think I should start a branch to build it out so incremental progress isn't lost.

@amadard were you able to make a dent in this? I'd love to help out if you need it.

Hoping someone will get annoyed by how bad this is and help me fix it up :) -
`
using Microsoft.OpenApi.Any;
using Microsoft.OpenApi.Models;
using Swashbuckle.AspNetCore.SwaggerGen;
namespace Utils.Swagger
{
public class IdServer4Fakes : IDocumentFilter
{
public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context)
{
var tokenPath = new OpenApiPathItem()
{
Description = "Token endpoint",
        };
        tokenPath.AddOperation(OperationType.Post, new OpenApiOperation()
        {
            Tags = new[] { new OpenApiTag() { Name = "Auth" } },
            RequestBody = new OpenApiRequestBody()
            {
                Content =
                {
                    ["application/x-www-form-urlencoded"] = new OpenApiMediaType()
                    {
                        Example = new OpenApiString("grant_type=password&client_id=js&username=blah&password=blah&client_secret=blah"),
                        Schema = new OpenApiSchema()
                        {
                            Reference = new OpenApiReference()
                            {
                                Id = "ref",
                                Type = ReferenceType.RequestBody
                            }
                        }
                    }
                },
            },
            Responses = new OpenApiResponses()
            {
                ["200"] = new OpenApiResponse
                {
                    Description = "OK",
                    Content =
                    {
                        ["application/json"] = new OpenApiMediaType()
                        {
                            Example = new OpenApiString("{\"access_token\":\"string\",\"expires_in\":86400,\"token_type\":\"Bearer\",\"refresh_token\":\"string\"}"),
                            Schema = new OpenApiSchema()
                        {
                            Reference = new OpenApiReference()
                            {
                                Id = "ref",
                                Type = ReferenceType.RequestBody
                            }
                        }
                        }
                    }
                }
            }
        }) ;
        swaggerDoc.Paths.Add("/connect/token/", tokenPath);
    }
}

}
`

@nickelbob I started building it out using Microsoft.OpenAPI to build a code first representation that could be delivered up using a RouteBuilder.MapGet.
What is everyone else's thoughts on this?
As I was writing this up, I saw @nickelbob responded, it seems we are on the same path!
E.g.

return new OpenApiDocument
{
	Info = new OpenApiInfo
	{
		Version = version,
		Title = "IdentityServer4",
	},
	Components = new OpenApiComponents()
	{
		Schemas = new Dictionary<string, OpenApiSchema>
		{
			["DiscoveryDocumentType"] = new Microsoft.OpenApi.Readers.OpenApiStringReader().ReadFragment<OpenApiSchema>(
				"{\"type\":\"object\",\"properties\":{\"issuer\":{\"type\":\"string\"},\"jwks_uri\":{\"type\":\"string\"},\"authorization_endpoint\":{\"type\":\"string\"},\"token_endpoint\":{\"type\":\"string\"},\"userinfo_endpoint\":{\"type\":\"string\"},\"end_session_endpoint\":{\"type\":\"string\"},\"check_session_iframe\":{\"type\":\"string\"},\"revocation_endpoint\":{\"type\":\"string\"},\"introspection_endpoint\":{\"type\":\"string\"},\"device_authorization_endpoint\":{\"type\":\"string\"},\"frontchannel_logout_supported\":{\"type\":\"boolean\"},\"frontchannel_logout_session_supported\":{\"type\":\"boolean\"},\"backchannel_logout_supported\":{\"type\":\"boolean\"},\"backchannel_logout_session_supported\":{\"type\":\"boolean\"},\"scopes_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"claims_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"grant_types_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"response_types_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"response_modes_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"token_endpoint_auth_methods_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"id_token_signing_alg_values_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"subject_types_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"code_challenge_methods_supported\":{\"uniqueItems\":false,\"type\":\"array\",\"items\":{\"type\":\"string\"}},\"request_parameter_supported\":{\"type\":\"boolean\"}},\"required\":[\"issuer\",\"jwks_uri\",\"authorization_endpoint\",\"token_endpoint\",\"userinfo_endpoint\",\"end_session_endpoint\",\"check_session_iframe\",\"revocation_endpoint\",\"introspection_endpoint\",\"device_authorization_endpoint\",\"frontchannel_logout_supported\",\"frontchannel_logout_session_supported\",\"backchannel_logout_supported\",\"backchannel_logout_session_supported\",\"scopes_supported\",\"claims_supported\",\"grant_types_supported\",\"response_types_supported\",\"response_modes_supported\",\"token_endpoint_auth_methods_supported\",\"id_token_signing_alg_values_supported\",\"subject_types_supported\",\"code_challenge_methods_supported\",\"request_parameter_supported\"]}",
				Microsoft.OpenApi.OpenApiSpecVersion.OpenApi2_0, out var discoveryDiagnostic)
		}
	},
	Servers = new List<OpenApiServer>
	{
		new OpenApiServer { Url = authority}
	},
	Paths = new OpenApiPaths
	{
		["/.well-known/openid-configuration"] = new OpenApiPathItem
		{
			Description = "The discovery endpoint can be used to retrieve metadata about your IdentityServer - it returns information like the issuer name, key material, supported scopes etc.",
			Operations = new Dictionary<OperationType, OpenApiOperation>
			{
				[OperationType.Get] = new OpenApiOperation
				{
					Description = "Retrieve metadata about your IdentityServer",
					ExternalDocs = new OpenApiExternalDocs()
					{
						Description = "Discovery Endpoint",
						Url = new System.Uri("http://docs.identityserver.io/en/latest/endpoints/discovery.html")
					},
					Parameters = new List<OpenApiParameter>(),
					Responses = new OpenApiResponses
					{
						["200"] = new OpenApiResponse
						{
							Description = "OK",
							Content = new Dictionary<string, OpenApiMediaType>()
							{
								["application/json"] = new OpenApiMediaType()
								{
									Example = new OpenApiString("{\"issuer\":\"https://demo.identityserver.io\",\"jwks_uri\":\"https://demo.identityserver.io/.well-known/openid-configuration/jwks\",\"authorization_endpoint\":\"https://demo.identityserver.io/connect/authorize\",\"token_endpoint\":\"https://demo.identityserver.io/connect/token\",\"userinfo_endpoint\":\"https://demo.identityserver.io/connect/userinfo\",\"end_session_endpoint\":\"https://demo.identityserver.io/connect/endsession\",\"check_session_iframe\":\"https://demo.identityserver.io/connect/checksession\",\"revocation_endpoint\":\"https://demo.identityserver.io/connect/revocation\",\"introspection_endpoint\":\"https://demo.identityserver.io/connect/introspect\",\"device_authorization_endpoint\":\"https://demo.identityserver.io/connect/deviceauthorization\",\"frontchannel_logout_supported\":true,\"frontchannel_logout_session_supported\":true,\"backchannel_logout_supported\":true,\"backchannel_logout_session_supported\":true,\"scopes_supported\":[\"openid\",\"profile\",\"email\",\"api\",\"policyserver.runtime\",\"policyserver.management\",\"offline_access\"],\"claims_supported\":[\"sub\",\"name\",\"family_name\",\"given_name\",\"middle_name\",\"nickname\",\"preferred_username\",\"profile\",\"picture\",\"website\",\"gender\",\"birthdate\",\"zoneinfo\",\"locale\",\"updated_at\",\"email\",\"email_verified\"],\"grant_types_supported\":[\"authorization_code\",\"client_credentials\",\"refresh_token\",\"implicit\",\"password\",\"urn:ietf:params:oauth:grant-type:device_code\"],\"response_types_supported\":[\"code\",\"token\",\"id_token\",\"id_token token\",\"code id_token\",\"code token\",\"code id_token token\"],\"response_modes_supported\":[\"form_post\",\"query\",\"fragment\"],\"token_endpoint_auth_methods_supported\":[\"client_secret_basic\",\"client_secret_post\"],\"id_token_signing_alg_values_supported\":[\"RS256\"],\"subject_types_supported\":[\"public\"],\"code_challenge_methods_supported\":[\"plain\",\"S256\"],\"request_parameter_supported\":true}"),
									Schema = new OpenApiSchema()
									{
										Reference = new OpenApiReference { Type = ReferenceType.Schema, Id = "DiscoveryDocumentType" }
									}
								},
							},
						}
					}
				}
			}
		},
	},
};

And setting up the route in Startup:

private void ConfigureSwagger(IApplicationBuilder app, IHostingEnvironment env)
{
	Task swaggerJSON(HttpContext httpContext)
	{
		httpContext.Response.StatusCode = (int)HttpStatusCode.OK;
		return httpContext.Response.WriteAsync(
			IdentityServerSwagger.GetSwaggerDocument("https://demo.identityserver.io", "v1")
			.Serialize(OpenApiSpecVersion.OpenApi2_0, OpenApiFormat.Json));
	}
	var rb = new RouteBuilder(app);
	rb.MapGet("/swagger/v1/swagger.json", swaggerJSON);
	var routes = rb.Build();
	app.UseRouter(routes);

	if (env.IsDevelopment())
	{
		app.UseSwaggerUI(c =>
		{
			c.SwaggerEndpoint("/swagger/v1/swagger.json", "IdentityServer4");
		});
	}
}

@nickelbob I started a branch on my personal fork.  The OpenAPIDocument has the Discovery endpoint structured out, look it over and if you see any other properties that should be have detail, point it out!  I need to review the DiscoveryResponseGenerator to make sure my model matches the default implementation.
My thought is that IdentityServer would just generate the swagger.json file.  That way people can use whatever Swagger UI they want.


please add all google maps features to flutter

you know there are some things that are essential  to survive and for us maps is one such thing the plugin you have is no where near to real life use cases so just do it it is as important as the bin folder .. thank you for reading    it          .  ..    .............

It doesn't really make sense to keep this issue around because it's too generic.
Of course all features should be available.
If you need specific features please create specific issues or upvote existing ones. Thanks!


Use 8.0 embedder in Tizen 8.0 application

In Tizen 8.0, cbhm library has been deprecated.
So we need to use an embedder that does not depend on the cbhm library.
related issue: https://github.com/flutter-tizen/flutter-tizen/issues/538

This patch should be first. https://github.com/flutter-tizen/embedder/pull/63


Create CODEOWNERS file

Description

Add CODEOWNERS to the repo. The codeowners will be assigned to the PR when a new PR is made.
Followed instructions: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] Description of the pull request is clear and informative.
[x] I have added myself to the 'assignees'.
[x] I have added 'linked issues' if relevant.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@anaismiller can you also edit the PR template and add an extra space after #Description so the description has better formatting?


Fix for issue #33

This piece of code:
customPath = path.normalize(customPath);

If run on:  ${workspaceFolder}/../venv/bin/vale
would result in:  venv/bin/vale
The normalize needs to be run after the expansion to achieve the correct result.

Ok, thanks @hangie I'll do some testing. This isn't a feature I use, but I know others use the binary locations quite a bit.


Improve README: Better description of purpose and benefits of Result type

For #29
Draft, feedback welcome.

WIP, need to finish up first draft.

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #31   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          204       204
=========================================
  Hits           204       204

Dependent on #32 being merged

Closing for now, can be revisited in future


CB-2878 - Hive Needs to be in HTTP Mode for Knox to Proxy J/ODBC Access

Awaiting further input on parameters

Follow-up to #5848


No support for authentication with password

This library does not support specifying a password for the connection. MPD can be configured to only allow some actions with a password.

@auroraanna it does, try calling client.login("<your-password>") before issuing actions that require a password.

oh well, i only searched for "auth…"


Authentication through service-account

Please fill in the following fields:
Pre-built SDK from the website or open-source from this repo:
Firebase C++ SDK version: latest
Firebase plugins in use (Auth, Database, etc.): Auth
Additional SDKs you are using (Facebook, AdMob, etc.):
Platform you are using the C++ SDK on (Mac, Windows, or Linux):  Mac
Platform you are targeting (iOS, Android, and/or desktop): Desktop
Please describe the issue here:
Currently the way to authenticate api access is to set api-key in AppOptions. Is it possible that we could also have support for initialization based on service account like Java, Go and python sdk?
Please answer the following, if applicable:
Have you been able to reproduce this issue with just the Firebase C++ quickstarts ?
Yes
What's the issue repro rate? (eg 100%, 1/5 etc)
100%

The SDKs you mentioned are all admin SDKs intended to run in trusted execution environments (servers), so they have higher privileges than the client SDKs. This is why you cannot authenticate API access with a service account on the client SDKs and we recommend against using the admin SDKs in a client application.


[cli] connection refused by pgp.mit.edu when I request key for sift linux client install

I'm getting connection refused when I attempt:
gpg --keyserver pgp.mit.edu --recv -keys 22598A94

You can try
gpg --keyserver hkp://pgp.mit.edu --recv-keys 22598A94

gpg --keyserver hkp://pgp.mit.edu:80 --recv-keys 22598A94

Thanks... the 2nd one worked for me:
gpg --keyserver hkp://pgp.mit.edu:80 --recv-keys 22598A94
without :80 it still refused connection...

Thanks again...

Even trying port 80 isn't currently working for me.

Hi @jwilczek sorry you are having issues. I've listed the commands that do and don't work for me.
Are you using a proxy? Have you checked your network in general?
This works
gpg --keyserver pgp.mit.edu --recv-keys 22598A94

This does not:
gpg --keyserver hkp://pgp.mit.edu:80 --recv-keys 22598A94

This works
gpg --keyserver hkp://pgp.mit.edu --recv-keys 22598A94

You can always go to http://pgp.mit.edu/ search and download the key from there.


XA mode go scheme design/XA模式GO方案设计

What would you like to be added:
Why is this needed:
XA mode go scheme design/XA模式GO方案设计（输出文档），待后续开发使用

@
chuntaojun

i will start work this weekend


[Fixes #5496] Added config schema to the Splunk On Call plugin.

Signed-off-by: Tavis Aitken<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Added config schema to the Splunk On Call plugin to allow the frontend to see the configs.
Fixes #5496
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@taitken-splunk Hi! Just a small poke that the file has the wrong name. Renaming it to config.d.ts should fix the problem

@taitken-splunk Hi! Just a small poke that the file has the wrong name. Renaming it to config.d.ts should fix the problem

Right this has been updated now.

@taitken-splunk Ping! This one is really close and we're eager to merge; just a small nit remaining!


Issues with contrast in some places

The "Mocha Mauve" theme has some really bad spots where I cannot read or see stuff because the contrast is missing.
Here some examples:

(this is also present in other themes not just mocha mauve)


Add a copyright / notice file

Please answer these questions before submitting a bug report.
What version of OpenCensus are you using?

N/A
What did you expect to see?
This project doesn't appear to include any copyright information or a NOTICE file. This is desired to comply with the open source license conditions.
What did you see instead?
LICENSE file contains the placeholder for year and copyright owner ("Copyright [yyyy] [name of copyright owner]")
Additional context
Add any other context about the problem here.

@davidmateos I would like to take this up. Just a heads up, I will like to know that do I need to update the placeholder for year and copyright owner or do I need to do something else.

It's enough if you update the placeholder for year and copyright owner


RFindUnit is incompatible with Delphi Berlin

Incompatibility on helpers level and get access to private fields (SimpleParser.pas).

http://stackoverflow.com/questions/37351215/how-to-access-a-private-field-from-a-class-helper-in-delphi-10-1-berlin

This error is fixed in the updated version of DelphiAST

I merged Jkour branch!
Thank you,


Backoff reconnect

ztunnel should be in consistent with envoy to reconnect with backoff policy
https://github.com/envoyproxy/envoy/blob/a4053922d7311e11c18e764ff5e5bfa8f71cd334/source/common/config/grpc_stream.h#L21 and there are efforts to make it configurable https://github.com/envoyproxy/envoy/pull/24701

I will try to implement this


Last version not available in deno.land/x

Hi. I just realized the last version (v0.1.22-alpha, with support for document fragment) is not available in deno.land/x/deno_dom.

That is confusing... I'll investigate why it's not propagated to deno.land/x
Thanks for bringing it up

Apparently it's some caching issue on deno.land/x, it should be there now

Well, it's still not visible on the main page, but you can go there directly with<EMAIL_ADDRESS>

Okay, it shows on the main page now

Great. Thank you!


Local data don't work on non-secure environments

Describe the bug
When following the instructions available in the documentation about the emojibase data without a CDN, sometimes it doesn't work.
To Reproduce
Follow the instrutions in the documentation site about not using the CDN for the emojis data, and check it out on an untrusted context.
To show it more easily, I created a proof of concept of the bug:
git clone https://github.com/andreslucena/picmo-bug-emojibase-local-data
cd picmo-bug-emojibase-local-data
npm install
npm run build-inline
cd dist/
# open a webserver
python3 -m http.server 9000
# open the webpage with a browser
xdg-open http://localhost:9000/inline.html
# check out the same page but with a domain that resolves to localhost
xdg-open http://localtest.me:9000/inline.html

Expected behavior
It should work in the same environments as the default solution (CDN) works
Stacktrace
The error it gives is the following:

Uncaught (in promise) TypeError: crypto.subtle is undefined

What is happening is that the browser detects that this is an insecure context and doesn't provide the crypto API. See more details at stackoveflow: How to use SubtleCrypto in chrome (window.crypto.subtle is undefined)

I'd want to know if:

This is indeed considered a bug. If not, I think we should add a warning in the documentation that this feature only works in secure contexts (localhost and web servers with HTTPS)
Would you be open for me to make a PR solving this issue.

As the problem is with this line: https://github.com/joeattardi/picmo/blob/5bb82fd47a2262f71af8786e6731e833feed74b9/packages/picmo/src/util.ts#L102
Then I'm afraid that the solution would be to introduce a new dependency, such as js-sha256, that seems a well maintained and small library.
(Let me know if there's another approach for solving this!)

Hi, thanks for reporting this.
I am most likely not accepting PRs right now, as I am in the midst of deciding the future of this library. I don't have the free time I used to have, and usage of the library has declined, so I'm considering winding it down and deprecating the library.
Given that, I'd hate to waste your time having you submit a PR for a library that might not be maintained much longer.

Thanks for the quick answer and the honesty!
For what is worth, as I know that most of the times a developer don't have visibility on the usage of their library, I work with Decidim, an open source web platform for citizen participation that's being used in Barcelona, NYC, Brazil, France, the European Commission, Helsinki, City of Mexico and other places. Since 2021 we've been using PicMo (when it was called Emoji Button). Thanks for your work!


Add custom attributes to SQLCommenter comments in pg instrumentation

Is your feature request related to a problem? Please describe

Support for attaching SQLComment comments to SQL statements handled by the pg module was recently added in https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1286. This issue proposes adding support for custom attributes to the SQL comments, besides tracestate and traceparent, which are now automatically generated by pg instrumentation (if the addSqlCommenterCommentToQueries config is enabled).
Examples of the custom attributes (as suggested by the spec) are action, controller, framework and route.
Describe the solution you'd like to see

As suggested in https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1286#issuecomment-1375196941, one approach would be to allow the instrumentation to extract the custom attributes from the active context when a query is run (perhaps controlled by another config option). With this approach, one question that comes to mind is whether or not the attributes should be changeable from span to span.
Describe alternatives you've considered

Not using pg instrumentation to handle SQLCommenter comments, and instead appending the comments myself, where I can add whichever attributes I'd like.
Additional context

I'm creating this issue to get some thoughts on:

whether or not this makes sense for pg instrumentation
if 1, what approach would be best for implementing this
if 1 and 2, pointers on where to get started so I or someone else can get started on a PR

Don't close.


Changed the way to rename variable identifiers and references

Fixed https://github.com/javascript-obfuscator/javascript-obfuscator/pull/564

Coverage decreased (-0.02%) to 98.714% when pulling d229c8ffa1625c42ae9dfe46d1b313b280d392c7 on issue-563 into 4860db799fc983948c2d12281d5a97674ea326fe on master.

Coverage decreased (-0.02%) to 98.714% when pulling d229c8ffa1625c42ae9dfe46d1b313b280d392c7 on issue-563 into 4860db799fc983948c2d12281d5a97674ea326fe on master.


update readme

@faizanahmad055 Image is available for testing. docker pull stakater/workshop-excercise:SNAPSHOT-PR-5-7a5d2112


Building produces "cannot find module" error

Building and attempting to run the resulting binary produces the following error on start-up:
"Cannot find module ./etc/javascripts/mainwindow'"

https://github.com/alexkim205/G-Desktop-Suite/pull/30


Fixed failed to parse sourcemaps error

This fixes the issue surfaced in https://github.com/jakubroztocil/rrule/issues/522 that generates console warnings for anyone who imports rrule into their project.
I've tested it locally in my forked version and it appears to remove the errors.

Thanks, for the contribution! The problem with this approach is it points to the src/ folder in the production output, which ideally we shouldn't be even publishing.
What I propose we do instead is not minify the published output, then you won't even need sourcemaps. Most end-stage build tools minify the whole bundle including node_modules in application code, so I don't think this will have too much downside.

I've create two PRs which fix the situation for now:

https://github.com/jkbrzt/rrule/pull/637
https://github.com/jkbrzt/rrule/pull/634

@davidgoli I'd invite you to pick the one that makes more sense. Just having sourcemaps which point to non-existing source files makes not much, I'd say :call_me_hand:


update es.json

Thanks for marking the new lines with [EN], it made them much easier to find!

awesome! gonna hold this PR for at most a day to allow other translations to trickle in.
Thanks!


MDM does not seem to install driver extension classes

I have noticed that driver packages do not seem to install the extension class, for example the HP EliteBook 840 14 inch G9 and G10 have a Realtek sound controller and a Cirrus Logic Awesome Speaker Amps controller, but also require the Cirrus Amp Driver Extension for the sound to work.
For example when the driver pack is downloaded for the G10 the extension exist in the structure HP\EliteBook 840 14 inch G10 Notebook PC\Windows10-22H2-x64-2.00 A 1\StandardPkg\Audio\REALTEK\P014HB-B2C\src\driver\CsAudioExt
Does anyone know of a work around for this.
We are using SCCM alongside to the Automated Driver Tool

Hi
same issue here.
I worked around it creating an Application in my Task Sequence to Install the CsAudioExt drivers after the ADT install step.
The Application contains only the CsAudioExt folder from the ADT package and I use
"%windir%\system32\pnputil.exe /add-driver ".\driver\Csaudioext*.inf" /subdirs /install" to install them.
I also set a WMI Condition on the step to only run on a specific hardware.
maybe this helps.


"[download] Limit reached for max number of failed attempts`"

Hi,
I met a problem "[download] Limit reached for max number of failed attempts`" sometimes. I think it is because the RSS source is not stable enough. Is there any way to increase the max number of attempts? Or any other method to deal with this error?
Thank you very much,
Xilock
[get-cache] Use remote cache: https://molakirlee.github.io/feed/cache.json
[discover-cache] Pre-build cache restored: https://molakirlee.github.io/feed/cache.json
[enrich] 0.16s |  30 cached |   0 new | https://rsshub.app/geekpark/breakingnews
[enrich] 3.85s |  36 cached |   1 new | https://sspai.com/feed
[download] 2 attempts left, error download https://rsshub.app/wallstreetcn/news/global ECONNABORTED
[download] 2 attempts left, error download https://rsshub.app/bbc/world-asia ECONNABORTED
[download] 2 attempts left, error download https://rsshub.app/wallstreetcn/live ECONNABORTED
[download] 1 attempts left, error download https://rsshub.app/wallstreetcn/news/global ECONNABORTED
[download] 1 attempts left, error download https://rsshub.app/bbc/world-asia ECONNABORTED
[download] 1 attempts left, error download https://rsshub.app/wallstreetcn/live ECONNABORTED
[download] 0 attempts left, error download https://rsshub.app/wallstreetcn/news/global ECONNABORTED
/home/runner/work/feed/feed/node_modules/@osmoscraft/osmosfeed/bin/main.js:31614
      throw new Error(`[download] Limit reached for max number of failed attempts`);
            ^

Error: [download] Limit reached for max number of failed attempts
    at downloadHelper (/home/runner/work/feed/feed/node_modules/@osmoscraft/osmosfeed/bin/main.js:31614:13)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at runNextTicks (node:internal/process/task_queues:65:3)
    at listOnTimeout (node:internal/timers:526:9)
    at processTimers (node:internal/timers:500:7)
    at async enrichInternal (/home/runner/work/feed/feed/node_modules/@osmoscraft/osmosfeed/bin/main.js:31664:21)
    at async Promise.all (index 2)
    at async run (/home/runner/work/feed/feed/node_modules/@osmoscraft/osmosfeed/bin/main.js:31875:27)
Error: Process completed with exit code 1.

Hi @molakirlee and @philippnagel, I've moved your issue from the template repo to the main repo. I'm sorry I didn't see this earlier. I don't track the issues in the template repo as often.
This is a duplicate of #67, I have fixed it with PR #72 and is released for v1.13.0
Note that some network errors are impossible to recover, no matter how many retires we have. So the latest logic will just fallback to the cached content, instead of crashing the build.
Let me know if it works as expected. Thank you again for helping out.


Fix possible infinite loop in message template parser + better handling incorrect templates

See https://github.com/NLog/NLog/issues/2557#issuecomment-365200102

@snakefoot thanks for reviewing, I will react in this PR to make the discussions a bit more separated.

No need to use short (16 bit) since new processors prefer 32 bit and 64 bit variables.

This is a side effect hole.Index, which is already a 'short' (see https://github.com/NLog/NLog/blob/19de4feb0a6f1292e9d0c69da5c75b73efda3ea1/src/NLog/MessageTemplates/Hole.cs#L66)

Also no need to perform "safe" lookup, as invalid templates should trigger exceptions (just not unhandled exceptions)

What is the expected output of  MessageTemplateParameters.Count with " {3} {4} {9} {8} {5} {6} {7}" ? It was 2, which was really unexpected for me. Also if we could make (partly) a good result, that really better then no result. e.g. in  " {3} {4} {9} {8} {5} {6} {7}" we could maybe render 6 out of 7, which is better then 2 (and 0).

I prefer strict mode, but I can see Serilog has undocumented behavior when using invalid templates. Mixing strict with loose.
P.S. Please don't make these constructors internal. See #2581 (Maybe remove one of the struct constructors, but the one with CaptureType is used by NLog.Extensions)
Sent from my Samsung device
-------- Original message --------
From: Julian Verdurmen<EMAIL_ADDRESS>Date: 19/02/2018 20:04 (GMT+00:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Fix isPositional (#2576)
@snakefoothttps://github.com/snakefoot thanks for reviewing, I will react in this PR to make the discussions a bit more separated.
No need to use short (16 bit) since new processors prefer 32 bit and 64 bit variables.
This is a side effect hole.Index, which is already a 'short' (see https://github.com/NLog/NLog/blob/19de4feb0a6f1292e9d0c69da5c75b73efda3ea1/src/NLog/MessageTemplates/Hole.cs#L66)
Also no need to perform "safe" lookup, as invalid templates should trigger exceptions (just not unhandled exceptions)
What is the expected output of MessageTemplateParameters.Count with " {3} {4} {9} {8} {5} {6} {7}" ? It was 2, which was really unexpected for me. Also if we could make (partly) a good result, that really better then no result. e.g. in " {3} {4} {9} {8} {5} {6} {7}" we could maybe render 6 out of 7, which is better then 2 (and 0).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/pull/2576#issuecomment-366790852, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnA9shj7DSmkg46tVrhnSq4I42Vchks5tWdO9gaJpZM4SDq-c.

I prefer strict mode, but I can see Serilog has undocumented behavior when using invalid templates. Mixing strict with loose.

I prefer strict mode when:

it's a compile error (not run-time)
and/or it could be easily fixed (read: without recompile)
if the non-strict mode has undefined behavior.


filter causing "Uncaught SyntaxError: Unexpected identifier" error, feed doesn't load

I've got this very basic feed, want to filter for a date range:
<script type="text/javascript">
    var feed = new Instafeed({
        get: 'tagged',
        tagName: 'tag_name',
        clientId: 'my_client_id',
        filter: function(image) {
            var minDate = (new Date('June 1 2010')).getTime();
            var maxDate = (new Date('October 12 2016')).getTime();
            var imageDate = parseInt(image.created_time, 10);

            return imageDate >= minDate && imageDate <= maxDate;
        }
    });
    feed.run();
</script>

When I load the page, I get "Uncaught SyntaxError: Unexpected identifier" about filter. I'm sourcing instafeed.min.js in the same directory; without the filter condition, this feed loads perfectly fine.
Any advice on what I'm missing? Thank you for making this, by the way!

(  -__-) I was missing a comma, hah. Sorry about that. Thanks again for making this!


Remove redondant python file

Salut,
Un fichier redondant que tu as laissé passé, à supprimer ;)

Hi, I understand that you thought 04_birthday_manager/birthday_countdown.py (1) was redundant with 03_birthday_countdown/program.py (2). But look, I don't think so. In the current state, (1) is indeed a copy-paste of (2), but you have to remark light differences:

(1) is documented (because docstrings are only introduced in lesson 04, they are for example heavily used in 04_birthday_manager/container.py)

(1) has two commented lines (30, 31), that are used during lesson 04 to show the interest of __name__ and __main__; besides in (1) main() is called within a if __name__ == '__main__': (otherwise, when you run 04_birthday_manager/program.py, the banner BIRTHDAY COUNTDOWN is displayed)

More importantly, (1) is used to be imported in 04_birthday_manager/program.py via import birthday_countdown in line 2.
Do you think of a better way to do things there?

Yes, forget what I said yesterday ! I ran the wrong command, and I thought it was the same file. (git diff program.py birthday_countdown.py instead of diff program.py birthday_countdown.py).
However, I should send you another PR soon.

Happy to hear that :) can't wait!


Enabling Multi Factor Authentication for User Association via Dashboard

Multi-Factor Authentication for User Association is not available for associating federated accounts or for associating local accounts using the dashboard association gadget.
There is a possibility of security attacks if an attacker already has an account in the system with login permission (if self-signup is enabled) due to not having MFA with User Association.
Therefore, the need of having MFA for User Association via the Dashboard has been raised.

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Segment Cost computation

Hello,
Thank you for open open sourcing  your implementation.
In polynomial_optimization_linear_impl,
the segment cost is computed as 0.5 * c^T * Q * c
Can you please explain reason for multiplication factor of 0.5 ?
Thanks

As far as I know this is just for convenience. If you compute the gradient you get a fixed scalar free expression, i.e., d/dc (0.5 * c^T * Q * c) = Q * c


Tensorflow-GPU processing fit on CPU

System information

Haven't written custom code.
OS Platform and Distribution: Windows 10
TensorFlow installed from: Anaconda
TensorFlow version: 2.0.0
Python version: 3.7
CUDA/cuDNN version: CUDA: 10.0.130, cuDNN: 7.6.5
GPU model and memory: GTX 1060 3Gb

Describe the current behavior
Tensorflow GPU is using the CPU instead of the GPU to fit. If I use tf.compat.v1.debugging.set_log_device_placement(True) when trying to fit the model I get:
2020-01-04 21:00:20.616161: I tensorflow/core/common_runtime/eager/execute.cc:574] Executing op RangeDataset in device /job:localhost/replica:0/task:0/device:CPU:0
2020-01-04 21:00:20.616622: I tensorflow/core/common_runtime/eager/execute.cc:574] Executing op RepeatDataset in device /job:localhost/replica:0/task:0/device:CPU:0
2020-01-04 21:00:20.637944: I tensorflow/core/common_runtime/eager/execute.cc:574] Executing op MapDataset in device /job:localhost/replica:0/task:0/device:CPU:0
But for example on other ocations (not fitting):
2020-01-04 21:00:19.201662: I tensorflow/core/common_runtime/eager/execute.cc:574] Executing op VarHandleOp in device /job:localhost/replica:0/task:0/device:GPU:0
Code to reproduce the issue
import gym
import numpy as np
from tensorflow import keras
from collections import deque
import random
import tensorflow as tf
import gc
import time
print(tf.test.is_gpu_available())
tf.compat.v1.debugging.set_log_device_placement(True)

class Agent:
    def __init__(self, state_size, action_size):
        self.LEARNING_RATE = 0.001
        self.EPSILON = 1.0
        self.EPSILON_DECAY = 0.97
        self.MIN_EPSILON = 0.01
        self.DISCOUNT = 0.95

        self.model = keras.Sequential()
        self.model.add(keras.layers.Dense(state_size, activation="relu"))
        self.model.add(keras.layers.Dense(24, activation="relu"))
        self.model.add(keras.layers.Dense(48, activation="relu"))
        self.model.add(keras.layers.Dense(action_size, activation="linear"))
        self.model.compile(optimizer=keras.optimizers.Adam(learning_rate=self.LEARNING_RATE),
                                                           loss="mse")

        self.memory = deque(maxlen=2000)

    def act(self, state):
        action = 0

        if state is None:
            action = 1
        else:
            if np.random.rand() <= self.EPSILON:
                action = random.randint(0, 2)
            else:
                action = self.model.predict(state)
                keras.backend.clear_session()
                gc.collect()
                action = np.argmax(action)

        return action

    def remember(self, state, reward, action, next_state, done):
        self.memory.append((state, reward, action, next_state, done))

    def retrain(self, minibatch_size):
        if minibatch_size < len(self.memory):
            minibatch = random.sample(self.memory, minibatch_size)
            now = time.time()
            for i in range(100):
                print("Start time:", str(now))
            for state, reward, action, next_state, done in minibatch:
                target = reward

                if not done:
                    target = reward + self.DISCOUNT*np.amax(self.model.predict(next_state)[0])
                    keras.backend.clear_session()
                    gc.collect()

                if not (state is None):
                    target_f = self.model.predict(state)
                    keras.backend.clear_session()
                    gc.collect()
                    target_f[0][action] = target
                    self.model.fit(state, target_f, epochs=1, verbose=0)

            print("Elapsed:", str(time.time() - now))

        if self.EPSILON > self.MIN_EPSILON:
            self.EPSILON *= self.EPSILON_DECAY

    def load(self, name):
        self.model.build(input_shape=(1, 2))
        self.model.load_weights(name)

    def save(self):
        self.model.save_weights(str(episode + 6700) + ".h5")

class Environment:
    def __init__(self):
        self.env = gym.make("MountainCar-v0")
        self.reward = 0
        self.max_height = -999
        self.record_pos = -9

    def make_move(self, action):
        state, reward, done, info = self.env.step(action)

        if state[0] > self.max_height:
            if state[0] > self.record_pos:
                self.record_pos = state[0]
                self.max_height = self.record_pos
                self.reward = 50
            else:
                self.max_height = state[0]
                self.reward = 10

        if state[0] == 0.5:
            self.reward = 100

        return state, reward, done

    def get_state_size(self):
        return self.env.observation_space.shape[0]

    def get_action_size(self):
        return self.env.action_space.n

    def reset(self):
        self.env.reset()
        self.reward = 0
        self.max_height = -999

    def close(self):
        self.env.close()

    def render(self):
        self.env.render()

    def get_max_position(self):
        return self.record_pos

environment = Environment()
agent = Agent(environment.get_state_size(), environment.get_action_size())
agent.load("6700.h5")
for episode in range(1000):
    i = 0
    state = None
    environment.reset()
    done = False
    previous_height = 0
    while not done:
        if episode % 100 == 0:
            environment.render()

        action = agent.act(state)
        next_state, reward, done = environment.make_move(action)

        next_state = np.array(next_state)
        next_state = np.reshape(next_state, [1, environment.get_state_size()])

        if state is not None:
            agent.remember(state, reward, action, next_state, done)

        state = next_state

    print("Max height:", environment.get_max_position(), "episode:", episode)
    if episode % 100 == 0 and episode != 0:
        agent.save()
    agent.retrain(32)

environment.close()

Other log that might be useful is:
print(tf.test.is_gpu_available()) ----> True
Also, the GPU load is 0%.

@PBB-as
Request you to share colab link or minimal standalone code with proper indentation to reproduce the issue in our environment. It helps us in localizing the issue faster. Thanks!

@PBB-as
Request you to share colab link or minimal standalone code with proper indentation to reproduce the issue in our environment. It helps us in localizing the issue faster. Thanks!

Done!

@PBB-as
I tried reproducing the issue with TF 2.0. Please, find the gist here. Is this the expected behavior?. Thanks!

@ravikyram
That was logging me at first too, but if you keep running the code (train a few samples) you'll note that Tensorflow will use the CPU instead of GPU as I mentioned before.

2020-01-04 21:00:20.637944: I tensorflow/core/common_runtime/eager/execute.cc:574] Executing op MapDataset in device /job:localhost/replica:0/task:0/device:CPU:0

I think that the problem is solved using:
config = tf.compat.v1.ConfigProto()
config.gpu_options.allow_growth = True
config.gpu_options.per_process_gpu_memory_fraction = 0.95 #Or any other fraction != 1
session = tf.compat.v1.Session(config=config)

And then running Tensorflow as follows:
with session:

@PBB-as
Please, let me know if we can close this issue since it looks to be fixed. Thanks!

@ravikyram
Yes, close it. Thank you!

@santiagopardal  I am having the same issue. Could you elaborate a bit more on how you use the session to solve it? (Attaching the modified code would help). Thanks.


basePath does not work with urlMappingStrategy: "rewrite" in app router

Describe the bug
When using the next-app example, using basePath with urlMappingStrategy: "rewrite" redirects to not-found page when navigating to http://localhost:3000/base.
To Reproduce
Steps to reproduce the behavior:

Set basePath in locales/client.ts to /base
Set basePath in next.config.js to /base
Set urlMappingStrategy in middleware.ts to rewrite
Start dev server
Go to http://localhost:3000/base
not-found screen is shown

Expected behavior
From the description of rewrite:

rewrite: https://example.com/products -> rewrite to https://example.com/en/products -> client doesn't see the locale in the url

I would expect that navigating to http://localhost:3000/base would rewrite to http://localhost:3000/base/en.
Screenshots

About (please complete the following information):

next-international 1.1.3
Next.js 13.5.4

Good catch, we'll probably need to introduce a new basePath option to createI18nMiddleware. In the meantime, you could use the default urlMappingStrategy. Not ideal but I'll try to fix this bug asap.

I came across the same Issue. I believe this library is one of the few libraries that meet the need for the Next.js app router users to "not want to change the url structure". So I hope this Issue will be fixed as soon as possible.

@QuiiBz
I came across the same Issue. I believe this library is one of the few libraries that meet the need for the Next.js app router users to "not want to change the url structure". So I hope this Issue will be fixed as soon as possible.


Gradle dependency

Hi,
This is something quite useful for us and we are looking for a more Annotation based handling of Error Codes. Is there a Gradle dependency for this that we could use? We are using Feign with Hystrix, plugged in using OkHttp client.
Regards,
Sri

Hi Sri -
I need to get this released and into maven central and then you should be able to use it (apologies it's taken me so long!)

@srikumarcs @saintf +1 here to get this on maven central let me know if/how I can help out for this !

@srikumarcs @scoquelin Sorry guys - took a while - and still taking - but - almost there!
I had to update credentials and get a few bits and pieces going in the orchestration for releases - but... I seem to finally have all the appropriate rights and such. A release version of this is sitting in bintray - waiting to be manually reviewed by one of the moderators, and as soon as they approve we should be able to push to maven central. When that's done - I'll ping over the coordinates.
Sorry and thanks for your patience!

@srikumarcs @scoquelin released!
https://search.maven.org/#artifactdetails|io.github.openfeign|feign-annotation-error-decoder|1.0.3|jar
Try it out and enjoy!


create pars template for calibration

Brief description
Create an example template for external/internal partners who wish to calibrate. Template can be adjusted as we go, I've only added in 'context-specific --REPLACE' and commented out areas that are context-specific while leaving in universal, e.g. fecundity. Just a start of some additional resources for end users.
Type(s) of change

[ ] Bugfix
[ ] Refactor
[ ] New feature
[x] Other

Checklist

My code follows the style guide

[ ] Yes
[x] N/A

I've commented my code

[x] Yes
[ ] N/A

I've incremented the version number

[ ] Yes
[x] N/A

I've updated the changelog

[ ] Yes
[x] N/A

I've added tests and checked code coverage

[ ] Yes
[x] N/A

Tagging @avictorious @mzimmermann-IDM @cliffckerr for an approval before merging this simple template file. Will likely replace with the automation work, but I wanted to have something available for new users.


[Feat] Expand howto-control-rpi-poweredup.md

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Next step: Try Hub2Hub Communication

https://hackaday.io/project/183741-livestock-and-pet-geofencing/log/203261-hub2hub-communication
https://github.com/bricklife/LEGO-Hub2Hub-Communication-Hacks
https://hubmodule.readthedocs.io/en/latest/hub2hub/


Update for wildcard entries in hosts file param

Added support for wildcard entries in the hosts file, using * as a prefix:
<IP_ADDRESS>   *.example.com

Only one wildcard can be used, and it must be on the beginning of the domain. Name resolution is implemented on a single level, so the example will resolve api.example.com, but will not resolve v2.api.example.com.

Thanks for the PR and especially for fixing and adding the tests! :thumbsup: :smiley:

Could you squash the commits?

If you insist, I can, but to be honest, I don't want to. I'd rather send you a postcard from the Azores, while I'm enjoying some wine with my lovely wife. Merge it and let's continue. I'd rather make some usable feature than concern myself with commit history.

Don't forget the postcard!

Need an address for that one ;)
On Sat, Mar 12, 2016 at 4:20 PM, Jan B<EMAIL_ADDRESS>wrote:

Don't forget the postcard!
—
Reply to this email directly or view it on GitHub
https://github.com/janeczku/go-dnsmasq/pull/5#issuecomment-195759973.


read dicom with jpeg compress data

hi, i want to read a dicom with data compressed as jpeg. i find the daikon.js, and use it to get uncompress dicom image, but then, how to use itk-js to read the image, because i use vtk.js too, i know vtk.js depend on itk.js to read such as image, dicom etc.
main code logic is below:
import itkreadImageDICOMFileSeries from 'itk/readImageDICOMFileSeries';
import vtkHttpDataAccessHelper from 'vtk.js/Sources/IO/Core/DataAccessHelper/HttpDataAccessHelper';

vtkHttpDataAccessHelper.fetchBinary(url).then((arrayBuffer) => {
    const image = daikon.Series.parseImage(new DataView(arrayBuffer));
    if (image === null) {
        console.error(daikon.Series.parserError);
    }

    const rawData = image.getRawData(); // ArrayBuffer

    const filename = `test.dcm`;

    const file = new File([rawData], filename);

    itkreadImageDICOMFileSeries(null, [file])
        .then(({ image: itkImage, webWorker }) => {
            // code not exec here
            // get a Error: Could not read file: /work/test.dcm
        })
})

update:
because the raw image data have got, i think i have to create a Image instance, but need a correct ImageType.
https://github.com/InsightSoftwareConsortium/itk-js/blob/f2b16ac4b9097367969dc1ff961037bc43441753/src/Image.js#L4-L23
https://github.com/InsightSoftwareConsortium/itk-js/blob/f2b16ac4b9097367969dc1ff961037bc43441753/src/ImageType.js#L4-L12
but i don't know how to determine the componentType componets and other input args of constructor ImageType.
thanks for your suggestions!

@chenkaiC4 you do not need to use daikon -- use itk.js directly\ and pass it the array buffer.
Here is an example:
https://github.com/Kitware/itk-vtk-viewer/blob/ddc38926657cadfa38280ab4da4e45ca7c482a61/src/processFiles.js#L15-L24

@chenkaiC4 I reproduced the read failure with the daikon.js test dataset -- I will see about fixing the ITK build so JPEG2000 compression is supported.

@thewtex hi, i have create a Image instance manually which declared at https://github.com/InsightSoftwareConsortium/itk-js/blob/6377e1890d524fa9cf688c21698cfa4cfb6ca02b/src/Image.js#L4 , the raw arraybuffer is get from the daikon.js's concatenateImageData method which will reorder the 2D images of dicom and concat them.
relative code shows as follow, it works. But the slicer mode display is not correct. see the two picture.
      const itkImage = new Image(new ImageType(3, dataType));
      itkImage.size[0] = row;
      itkImage.size[1] = col;
      itkImage.size[2] = dataLength;
      itkImage.spacing[0] = pixelSpacing[1];
      itkImage.spacing[1] = sliceThickness / 2;
      itkImage.spacing[2] = pixelSpacing[0];
      itkImage.direction.setElement(0, 0, 0.0);
      itkImage.direction.setElement(0, 1, -0.0);
      itkImage.direction.setElement(0, 2, -1.0);
      itkImage.direction.setElement(1, 0, 1.0);
      itkImage.direction.setElement(1, 1, 0.0);
      itkImage.direction.setElement(1, 2, 0.0);
      itkImage.direction.setElement(2, 0, 0.0);
      itkImage.direction.setElement(2, 1, -1.0);
      itkImage.direction.setElement(2, 2, 0.0);
      itkImage.data = new Uint16Array(file);

      const imageData = vtkITKHelper.convertItkToVtkImage(itkImage);

Follow picture is well display, it display a dicom series not compressed. but the second picture is wrong, the dicom series is the same with the first picture's. i do this just want to test the combination with daikon.js and itk.js. from the two pictures, the spacing and direction seemd to be wrong. but i don't know how to fix it.

@thewtex thanks, i will have a try to fix it. 👍

Hi @chenkaiC4 can I email you to ask a lot things about dicom reader?
I need your help

@LabbaAwwabi<EMAIL_ADDRESS>i'm not sure i can solve your problems, just have a try.

Hi!
I just found this thread via google!
What I am trying to accomplish: I have JPEG-LS compressed DICOM images and want to show them in a browser with a certain windowing, which I apply to it.
Do you have - by any chance - any simple code which does that in JS. I. e., start with a DICOM image (e. g. by link) and result in the canvas image with the correct windowing applied? Also stating, exactly which JS libraries to include? I do not need JPEG2000, only JPEG-LS.
Best regards and thanks a lot!
Martin

@martinweihrauch Did you get any solution for JPEG-LS?

Workaround using gdcm-js
const fs = require('fs');
const gdcm = require("gdcm-js");
const readImageLocalFileSync = require('itk/readImageLocalFileSync.js');

var dicomoriginal = new Uint8Array(fs.readFileSync('compressed.dcm'));
var memfs = [{name: "input.dcm", data: dicomoriginal}];
var args = ["-i", "input.dcm", "-o", "output.dcm", "-w"];

var result =  gdcm.gdcmconv({MEMFS:memfs,arguments:args});

fs.writeFileSync('temp.dcm', result.MEMFS[0].data)

var in_img = readImageLocalFileSync('temp.dcm');
console.log(in_img)

This reader available in npm med-img-reader reads compressed DICOM images and returns in the ITK format so it is compatible with VTK.js as well.

itk/readImageFile and itk/readImageDICOMSeries use GDCM under the hood since 12.1.0 (not. that readImageDICOMSeries's interface changed recently), so these issues should be addressed.


Improving code and documentation.

The current python codes are poorly documented and written, as I wrote them while learning python myself. I aim to improve and document them well so that any python beginner can understand them easily.

Can I contribute?

Can I contribute?

Yes ofcourse! No need to ask. Thanks.


pydantic.dataclasses not plumbing metadata for schema generation (namely, field description)

Checks

[x ] I added a descriptive title to this issue
[ x] I have searched (google, github) for similar issues and couldn't find anything
[x ] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
...             pydantic version: 1.7.3
            pydantic compiled: True
                 install path: $VENV_PATH/lib/python3.8/site-packages/pydantic
               python version: 3.8.5 (default, Jul 28 2020, 12:59:40)  [GCC 9.3.0]
                     platform: Linux-5.8.0-43-generic-x86_64-with-glibc2.29
     optional deps. installed: ['typing-extensions']

from pydantic import BaseModel, Field
from pydantic.dataclasses import dataclass
from dataclasses import field

class DescriptionFromBasemodel(BaseModel):
    descr_present: int = Field(
        42,
        title='my title',
        description='descr text',)

print(DescriptionFromBasemodel.schema_json(indent=4))

@dataclass
class DataclassWithDescription:
   descr_missing: int = field(metadata=dict(
       title="my title",
       description="descr text"))

print(DataclassWithDescription.__pydantic_model__.schema_json(indent=4))

outputs as expected for raw basemodel:
{
    "title": "DescriptionFromBasemodel",
    "type": "object",
    "properties": {
        "descr_present": {
            "title": "my title",
            "description": "descr text",
            "default": 42,
            "type": "integer"
        }
    }
}

but for the model generated from the dataclass, no attrs are preserved:
    "title": "DataclassWithDescription",
    "type": "object",
    "properties": {
        "descr_missing": {
            "title": "Descr Missing",
            "type": "integer"
        }
    },
    "required": [
        "descr_missing"
    ]
}

Hi @m10d
This is not a bug but a feature request related to https://github.com/samuelcolvin/pydantic/issues/470.
There is no current support of what you're asking for pydantic dataclass
But I agree using metadata kwarg of field would make sense


add running percent correct

see also #164
here's what i have so far:
def calc_running_percent_correct(events, timestamps, start_events=[b"TARGET_ON"], end_events=[b"REWARD", b"TRIAL_END"], success_events=b"REWARD",
                                window_size=60., window_step=None):
    '''
    A wrapper around get_trial_segments which counts the number of trials with a reward event
    and divides by the running number of trials to calculate success rate. Trials are counted at
    the end of each segment, for example on a reward event.

    Args:
        events (nt): events vector, can be codes, event names, anything to match
        timestamps (nt): time at which each event occurs
        start_events (list, optional): set of start events to match
        end_events (list, optional): set of end events to match
        success_events (list, optional): which events make a trial a successful trial
        window_size (float, optional): the running window size, defaults to 60 s
        window_step (float, optional): the step between betweens, if none, calculate the event rate with NON-OVERLAPPING windows

    Returns:
        tuple: tuple containing:
            percent_correct (nt'): number of successful trials along time
            time (nt'): time vector to use for plotting the above timeseries

    '''
#     if events is not np.ndarray: events = np.array(events)
#     if timestamps is not np.ndarray: timestamps = np.array(timestamps)

    segments, segment_times = aopy.preproc.get_trial_segments(events, timestamps, start_events, end_events)
    correct = np.array([np.isin(success_events, x).any() for x in segments])
    if window_step is None:
        window_step = window_size

    # Search over segments to find correct trials
    start_timestamp = segment_times[0][0]
    last_timestamp = segment_times[-1][1]
    stop_timestamp = start_timestamp + window_size
    correct_rate = list()
    start_timestamp_list = list()
    trial_idx = 0
    while stop_timestamp <= last_timestamp:

        # Find correct trials
        in_window = [start_timestamp < x[1] <= stop_timestamp for x in segment_times]
        n_trials = np.count_nonzero(in_window)
        n_correct = np.count_nonzero(np.logical_and(correct, in_window))
        if n_trials > 0:
            window_rate = n_correct / n_trials
        else:
            window_rate = 0
        correct_rate.append(window_rate)
        start_timestamp_list.append(start_timestamp)

        # Shift the window
        start_timestamp = start_timestamp + window_step
        stop_timestamp = stop_timestamp + window_step

    # Calculate the window timstamps
    time = np.array(start_timestamp_list) + 0.5 * window_size

    return correct_rate, time

def test_calc_running_percent_correct():
    events = [b'TARGET_ON', b'REWARD', b'TARGET_ON', b'TRIAL_END', b'TARGET_ON', b'REWARD']
    timestamps = [1, 2, 3, 4, 5, 6, 9]
    correct, time = calc_running_percent_correct(events, timestamps, window_size=1)
    print(correct)
    print(time)
    plt.plot(time, correct)

I've written a version of this for my analysis. Will look at blending them together.


Primitive type narrowing doesn't respect re-assignment in function body

Bug Report
🔎 Search Terms
narrowing inside function
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about type narrowing

⏯ Playground Link

Playground link with relevant code
💻 Code

let myBool = false; // on hover, myBool is typed as boolean

myBool // on hover, myBool is now typed as false

// change myBool from false to true
runCode(() => {
    myBool = true;
})

myBool // on hover, myBool is still typed as false

console.log(myBool) // correctly prints true

function runCode(run: () => void) {
    run();
}

🙁 Actual behavior
myBool is a boolean and declared with let. It is allowed to be changed to from false to true, and indeed I do! But TypeScript doesn't realize that I've changed it, so it narrows the type (on hover) to false even after I've changed it.
🙂 Expected behavior
Either: (1) TS shouldn't narrow the type to false at all, or (2) TS should realize that the value may have changed on the runCode(...) line, and therefore not restrict the type to false after that function call.

Duplicate of #9998.


Add dependency libs

Add dependencies on LLVM libs.
Fixes building with shared libraries on.
Also move some common code out of the if.

Is that actually common code? I remember there are some issues with using add_llvm_library out-of-tree.

Oh, I misread the add_library for add_llvm_library…


CI Testing Matrix: BeforeOrAt

Feature Request
Description of Problem:
We have produced a testing matrix which documents all scenarios we want to produce to confirm that our constraints work as expected. We now need to revisit our current tests with the matrix in mind, remove any duplicate tests and add/update missing scenarios.
Testing Matrix: https://docs.google.com/spreadsheets/d/1t4GJxZNIIYVdSGsx6B8KxsCJOwIRvaFX1g6b8lMIMFc/edit#gid=0
Acceptance Criteria:

[ ] BeforeOrAt tests match the desired testing matrix
[ ] Issues are raised for any failing tests with a clear "ignore" tag and a comment mentioning the bug raised with a short description of the test failure

Duplicate of #308

Closing as duplicate of #308


Handover: new fields requested in the views

The following fields have been requested a long time ago. It would be good to get this ready for the new serverside version of the application.

[ ] open PA
[ ] open PB
[ ] not signed MCCRs
[ ] open work orders (< 100%)
[ ] not signed CPCLs
[ ] open SWCRs
[ ] not signed tasks
[ ] not signed actions
[ ] open queries
[ ] remaining mhrs on open work orders

@aman-riaz @BenSvend @espenkalle What is going on here? These new fields needs to be added to the FAM SQL custom view right?

@ken-mellem Fields are in FAM but need more info. What do we do with them? Filters? Grid? Sidesheet etc

This one is based on #13. I am not quite sure how the data should be displayed. @atlewee perhaps you should qualify if this need is real - of if it is simply a "I want it all"-left over from the early days. If so, then I propose we close this for now.

@ken-mellem : leftovers/nice2have/not sure if makes sense :)
Agree, lets close this.


Conformer yaml is missing several required hparams

When running the recipes/LibriSpeech/ASR/transformer/train.py with the conformer.yaml the training errors with several missing hparam errors.  Adding the following to the yaml allows training to proceed:
skip_prep, tokenizer, pretrainer, stage_one_epochs, Adam (renamed from optimizer)
Can submit as a PR if you want.

@JianyuanZhong

Also, some issues when doing eval with the recipe yaml:
might be an incorrect case of Transformer for default conformer.yaml has:
modules:
  Transformer: *id003

Which seems to run on training, but when running an eval with TransformerASR.from_hparams errors out.
Also... had to add modules: compute_features, normalize, pre_transformer, beam_searcher to get the eval to run.  Might be a good idea to have these in the default yaml.

Could you open a PR?

Right, let's see if we shouldn't simply remove this YAML for now. THe recipe isn't respecting the correct pretraining loading scheme and might induce peoples in the wrong direction.

Sorry about not responding re: the PR.  I'm not confident about what the correct config should be, and don't want to introduce new issues.
Running a training now with a YAML pieced together from the github version, and the https://huggingface.co/speechbrain/asr-transformer-transformerlm-librispeech/blob/main/hyperparams.yaml version to get training/inference to run.  (Getting long gibberish inference results so far, coming up on end of epoch 1, with ~16k hours of training data.)

Wow, that's a lot of training data. Note: You might want to work with the transformer rather than the conformer as transformers are way more stable (and faster) in the current version.

In general, the conformer part is still in beta version. We are working on that, but still not stable. @JianyuanZhong @popcornell

Being fixed in #535.


Http Methods

Creates the ability to add HTTP/REST methods to an SDK. Would be used by Extension SDKs
Creates a new module called Http that looks like a Transport Layer.
Methods that have the "http" tag will:
import Transport from '../Http'

So that existing method templates will just work over Http.
The "http" tag supports several extensions:

x-http-endpoint - The baseUrl for the REST API
x-http-path - the path to append to the baseUrl
x-http-headers - any headers to include
x-http-parameters - any query parameters to include

Additionally, all of these extensions will support the following macros:

${param.} - insert the value of param
${token} - insert the value of the Extension SDK's token

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-openrpc/53/rdkcentral/firebolt-openrpc

Commit: e7843303f7bb3912a29ef1fd27c08967c2c2771f

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-openrpc/53/rdkcentral/firebolt-openrpc

Commit: 7f55ab5a20453d2da3e8fcb6b64468936b8c576e

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-openrpc/53/rdkcentral/firebolt-openrpc

Commit: f29fa0c3c0db3e8aea88c5c5820ea62582ac35a8

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Thanks Jeremy

Commit: f29fa0c3c0db3e8aea88c5c5820ea62582ac35a8

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/firebolt-openrpc/53/rdkcentral/firebolt-openrpc

Commit: caad894c27ff62a408988d03f874634857ae4d3e

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.


make sure stack trace shows up in errors

With the current error implementation, I don't get any stack trace when tests fail in Mocha as shown below.

This makes debugging very difficult. With this change, I get a stack trace:

Plus I think it is better to report a StargateError or AstraError than an AxiosError.

Also made the stack trace show up in Winston:


feat: Remove individual product from the scan carousel

What
Added a clear button to remove an individual product from the list of scanned products.
Video
https://user-images.githubusercontent.com/47862474/165375927-c3694f96-7e11-4950-b780-5fa64499ed41.mp4
Fixes bug(s)

#1295

Part of

#655

Not sure why we put the weight on the right (vs the Mockups) and thus can't put the cross on the left.
If in the red bar, it might send the message it will remove the red bar only.
Also, I expect it to be at the same distance from the right as the product name/brand are from the left.

Not sure why we put the weight on the right (vs the Mockups) and thus can't put the cross on the left.
If in the red bar, it might send the message it will remove the red bar only.
Also, I expect it to be at the same distance from the right as the product name/brand is from the left.

I'll place the cross icon on the left.
I'm not sure about the second point.
I'll fix this and change the distance as suggested.

@bhattabhi013 i was tired, cross on the right, see the original bug with mockup for reference

Hi everyone,
Here is the screenshot of the updated UI as suggested by @teolemon.

merged, thanks @bhattabhi013


docker: set user etcd-operator

fix https://github.com/coreos/etcd-operator/issues/806

With this change, etcd operator and backup sidecar won't be user root by default.

Encounter a problem:
Pod's volumes is mounted as root and non-root user couldn't access it.
Relevant upstream issue: https://github.com/kubernetes/kubernetes/issues/2630

@hongchaodeng I've been doing the init-container chown workaround for a while, seems to be problem free unless there's a hostPath volume and the user expects different ownership at the host level.
https://github.com/kubernetes/kubernetes/issues/2630#issuecomment-275118375

@bzub
I feel that is too hacky :(

fixed

the self hosted etcd will touch bind mounted on host path.

etcd pod touches the host path, not operator.

oh. ok. this is just for the operator. then looks fine to me.

@philips

Can we put the runAsUser into the kubernetes deployment?

No for two reasons:

We create user "etcd-operator", but runAsUser takes int64.
The container will be run as "etcd-operator" user by default. There is no need to specify it.


localized keyboard layouts

I am under the impression that the plugin has issues with local keyboard layouts. Tried recording a macro on a german keyboard layout using quotes " which caused the macro to create @ during replay.
Haven't investigated the code yet but german layout has " on shift+2, which is where the @ is located on US layouts.
Any ideas how to fix this?

Same problem on japanese localized keyboard. (Win10)
It can work if modify the WindowsAndLinuxCharCodeTranslations table appropriatly.
Even it is not good solution, (and maybe for Windows and Linux only)
Additionally, in japanese keyboard, the hotkeys 'ctrl-x (' and 'ctrl-x )' also deviated to US keyboard position. This may be a problem of Atom itself.

I am having the same issue on a system with a Spanish (ES) keyboard under Windows 10.
Whenever I press [ctrl]-x (, all I get is the opening and closing brackets with the cursor in between, as if I hadn't typed [ctrl]-x before it.
I am new to atom, so the issue could be that I didn't configure the package correctly or that I somehow mistype the sequence, but it does seem to be enabled as the key binding resolver does show that Ctrl-x is for this atom-keyboard-macros.
Any hints?

as I don't use Atom anymore and noone seems to care I'll close this issue and leave it open for someone else to keep tracking this


Large JS/JSON Files Lock Up VS Code

VSCode Version: 1.4
OS Version: 14905.rs_prerelease.160811-1739

Steps to Reproduce:

Drop in a large JS/JSON file, say a large array of objects.

VS Code can't process such a file and locks up. Also tested in Sublime 3 and there were no issues with the same data set, was able to move around the file with no problem.

@BitwiseGuy Can you please attach an example file (real or generated) that can help us reproduce?

Here is the attached file. It was originally a JSON file, converted to TXT for upload here. It is a list of worldwide airports and various properties. The data will ultimately be stored in a database, but for testing was pulled into the project as a JSON file. When opening the file in the editor, it locks up VS Code. I am unable to navigate in any way and I eventually get a not responding error.
I tried the same project in Sublime 3 and had no issues and also opened the file in Notepad++ with no issues.
I just ran with project with all extensions disabled. Initially I was able to navigate to the file and move around a bit (an improvement), but after trying to scroll up and down through the data I encountered the error again.
airports.txt

Can you please try out with VSCode 1.7+ and let me know if this issue still repos for you. I just tested this on 1.7.1 and did not notice any real slowdown (although syntax highlighting takes a moment to kick in for the whole file once you tell VSCode that the file is json):

If you are still seeing performance problems, please also let us know what platform you are on so that we can better attempt to reproduce this issue. Thanks.


Steps for automation tests

Hello. Could you add decorators to the automation test steps for Python?
For example from Qase:
with qase.step("Step 1. Switch to Seat", "Seat selected"):
search.select_seat()

I can add them in form of a decorator and even context manager, but they would not be set automatically

Will add this request to the development plan

Could I know about estimate time?

I would also like to clarify about the lack of automatic adding - what does this mean? If I write steps in the test code, will I still have to add them manually to the TMS for this test? And when changing steps in the code, they will not be updated in TMS?

When you run the command pytest --testomatio sync the info about test will be send to the testomatio. But without steps. There is even no API to sync test steps.
So yes, you have to update them manually both in the code and on the TMS.
During the reporting pytest --testomatio report there is an API to transfer test steps (never used it before, not sure how does it work)
The most probably will have time during June to implement this

Thanks for response


Use users avatars in Mattermost incoming webhook

It would be useful to use real avatars in "icon_url" parameter for Mattermost. That way you can visually distinguish which user wrote the message.
The avatars should probably follow rules for the media, e.g. get reuploaded to MediaServer if needed.

This is already implemented for mattermost, slack and discord.
I guess you're talking about telegram ?:)

Yes, for Telegram→Mattermost messages.

This was a rather big patch, could you please test this and give feedback ?
Thanks!

Unfortunately doesn't work as expected. Debug log shows that Telegram avatars are downloaded but in the message the avatar argument is blank, and no avatar is added to Telegram→Mattermost relayed message.
I've send debug log to your email.

Sometimes Matterbridge adds avatars to Mattermost message, but MediaServerUpload URL is used instead of MediaServerDownload.

Seems like the first messages are getting relayed without avatar, until the avatar is downloaded, and after it's downloaded, everything work fine if https://github.com/42wim/matterbridge/commit/7886f05e881e45303087a5ae116657f0f6164d81#diff-b113bff04f00d1b657baf156f81621e2R60 is changed to general.MediaServerDownload.

Oops, indeed thanks for debugging this!
So after this change it works like you expect it works ? (except the first message of each user)

So after this change it works like you expect it works ? (except the first message of each user)

Yes.


📝 Add direction to work from a branch of main repo rather than a fork

Per @danishswag's recent contribution – this needs to be made more obvious in CONTRIBUTING.md

As a follow-up to this, if we're updating CONTRIBUTING.md anyway, another issue I ran into is that the tests it recommends running also requires the packages in docs/requirements.txt too since the first test script (tests/test_local_spec) builds the docs locally.
If we want to keep the test suite as is, we could maybe just merge the two requirements files?

Must have been an issue with my Python install, please feel free to ignore


Bug in brew remove?

This isn't a case of normal usage, but:  brew remove "" results in:
/usr/local/Cellar/autoconf is not a valid keg

There's nothing special about autoconf; testing reveals that whatever formula is the alphabetically earliest that's installed will be the one that appears in the message.
This kind of thing is probably supposed to be blocked by line 13 of uninstall.rb, but the KegUnspecifiedError isn't getting raised AFAICT.  I suspect that the named method insufficiently strict to use as the guard here.

Yeah I can reproduce, it tries to uninstall whatever keg that comes first alphabetically, and fails.

Fix in #1139.

Thanks!


Address Geolocation issues

This was written to resolve issue #881  . Please check it out and let me know what you think.
All worked as intended from what I can tell  on my machines; let me know if that's not the case for others. I added a slight modification of the rake task from the geocoder docs for the back-generation of article coordinates. The command is to run that is:
rake geocode_data:all CLASS=Article SLEEP=0.25 BATCH=100

Hey @IsaacBell , it looks like the Travis build failed for the PR. Considering that master is still green, would you mind looking into why the build is failing on your PR?  Thanks.

Whoops, thought I'd commented already. Same issue with AWS environment
variables (aws_access_key_id, etc.) as pops up in the Issues for
development, because I don't have production key env variables set.
On May 17, 2017 6:25 PM, "Mark Nyon"<EMAIL_ADDRESS>wrote:

Hey @IsaacBell https://github.com/isaacbell , it looks like the Travis
https://travis-ci.org/BOWiki/BOW/builds/233431544 build failed for the
PR. Considering that master is still green, would you mind looking into
why the build is failing on your PR? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BOWiki/BOW/pull/896#issuecomment-302258633, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACffpV38P81kuOIOUkai5bJgAewOYhHwks5r64HzgaJpZM4NeigX
.

I think that you may need to be a member of the organization, @IsaacBell. I've just sent an invite. Once you accept, Travis should run the build with the environmental variables that you need.

@IsaacBell BTW, due to the way Github does permissions, you might need to reconnect your Github account to Travis in order for it to pick up the organization permissions.

So here's some additional information. Due to the way that Travis handles pull requests from forks, the environmental variables will not get copied over to your fork, @IsaacBell.
 Does that make sense?

Yes, makes sense, I'll do that.


Skip api version checks when install action is configured in client mode only

What this PR does / why we need it:
Hi there,
With helm 3, when an install action is run in client-mode only (install without --validate) and if your chart contains custom resources you end up hitting error that look like that.
apiVersion "example.com/v1" in my-app/templates/cutoms.yaml is not available

This PR attempt to workaround the issue by skipping entirely the check if we are in client mode as we can't be aware of any custom resource definition present on the remote cluster in advance as we don't interact with it in such mode.
Thoughts?
Special notes for your reviewer:
The error hit is similar to the one described in #6680
If applicable:

[ ] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

I believe this should close https://github.com/helm/helm/issues/6505.

Hmm. I don't think passing a boolean across several API layers just to disable a check is a great design... The validation is in place to ensure that there aren't CRDs being installed which will not render correctly given the available APIVersions.
Have you considered the proposal in #6680? What's your current use case, and would that approach solve it for you?

I believe the issue and solution described in #6680 is a little different. That proposal would fix template rendering when a chart also creates each CRD that it uses, however that won't work for charts that simply create custom resources and not their definitions (this is probably the more common of the two use cases).

We still want to catch validation errors with malformed templates to ensure the user won't run into validation errors when they invoke helm install.

This PR appears to only skip returning the specific error for a missing apiVersion:
https://github.com/helm/helm/blob/b57c8967f33df85330c9e7d0007f8b3663178885/pkg/releaseutil/manifest_sorter.go#L132-L134
If clientOnly is explicitly set to false via the --validate flag, then we get the full error checking. This seems like the best of both worlds from an end-user perspective (and gives us the same behavior as helm template in v2).

That said #6680 is also valid and would require some additional work to allow template to succeed when --validate is passed.

Apologies for the automatic closure. We just switched the dev-v3 branch to master, and the old master branch is now dev-v2. Feel free to re-open a new PR with this change. Thanks again and sorry for the hassle!

Reopened. No problem Matthew!

@bacongobbler FYI it is possible to switch the target branch of an existing pull request and retain the existing review comments/discussion.

Thank you for the heads up. I found out about that new feature after we deleted the branch. :disappointed:
Once the branch has been deleted, there is no way to switch the target branch and re-open the PR.


App bar pin cluster

Summary
Fixes #9587

Users can pin a cluster by clicking it from the menu.
Users can pin a cluster even if it is not ready.

Screenshot/Video

====================

======== Pnined cluster list ===========

======== Pnined cluster list in close App bar ===========

The smallIdentifier functionality will be extracted from the component and tested separately as part of https://github.com/rancher/dashboard/issues/9658


Allow for disabling innerItemsWidget

After I've selected my items, I don't necessarily want them populated in innerItemsWidget. Instead, I'd rather handle those selections a different way (in a separate list below the search widget for example).
It'd be great if after selecting, the searchHint remains in the search bar and nothing is populated.
if (list.isEmpty && hintIndex != null) {
  innerItemsWidget = items[hintIndex];
} else {
  innerItemsWidget = widget.selectedAggregateWidgetFn != null
      ? widget.selectedAggregateWidgetFn!(list)
      : Column(
          children: isPopulatingSearchBar ? list : [searchHint], // Something like this perhaps
        );
}

Well, well, @chrallard , I have good news for you, it is already possible with no change thanks to the selectedAggregateWidgetFn parameter as with the below example:
SearchChoices.single(
        items: items,
        value: selectedValueSingleDialog,
        hint: "Select one",
        searchHint: "Select one",
        onChanged: (value) {
          setState(() {
            selectedValueSingleDialog = value;
          });
        },
        isExpanded: true,
        selectedAggregateWidgetFn: (List<Widget> list) {
          return (Column(children: [
            Text("Reselect one"),
          ]));
        },
      )

Cheers.


Many To Many Query (Filter)

I am having an issue querying a many to many relationship...
My relationships are as follows
type User struct {
ID string
Roles        []Role gorm:"many2many:user_roles;"
}
type Role struct {
ID string
Users    []User gorm:"many2many:user_roles;"
RoleName string      gorm:"not null"
}
I have a table user_roles which has been created. When I create a user with a list of Role, those all get saved accordingly.
What I am curious about is, how can I retrieve a list of users, based on a specific RoleName.
eg.
Select u.Id
From users  u
Join user_roles ur on ur.user_id = u.id
Join roles r on ur.role_id = r.id
where r.role_name = 'xxxx'
I understand that GORM does it somewhat different. How would I accomplish essentially what this query returns?

https://gorm.io/docs/advanced_query.html
OR
https://gorm.io/docs/many_to_many.html

???

I believe I have figure this out...

@doppl-neal How have you accomplished this? I've facing the same issue


[Experimental] Add getInstance() to NativeDgnDb

Performance of getElement() vs getInstance()
getInstance() use InstanceReader which does not use ECSqlStatement and have far less moving parts (it does not do any prepare per class. it just prepare one sqlite stmt per table) then when using ECSqlStatement. InstanceReader also get called when we run SELECT $ FROM BisCore.Element.

DateTime::ToString() now use similar alog as sqlite which is very performant. This function as hot spot in profiler when generating JSON.
Added ExecuteSql() to help run pragmas for testing quickly without preparing and holding statements.
Added GetInstance() allowing bulk read of ecinstances from iModel.

% change : $(V_2 - V_1) / |V_1| * 100 = ?$

Allowing to pass on InstanceReader options would seem to be greatly beneficial.

/azp run imodel-native

/azp run imodel-native

Azure Pipelines failed to run 1 pipeline(s).

@DanRod1999 any idea what's wrong?

Azure Pipelines failed to run 1 pipeline(s).

@DanRod1999 any idea what's wrong?
Looks like an error in the yml file. I'll take a look

found the issue caused by a recent change in imodel-native-internal. Will fix now

where are the performance tests used to get the table comparing performance?

where are the performance tests used to get the table comparing performance?

Its out of here and likely to go some where else. I was thinking more of itwin-cli where we can write performance test and run it against a given dataset (files in directory) and create standard report out of it that can be compare over runs.


Add Oracle package PKG_REPLICATE_UTIL for replicating sources from one schema to another.

Should work for source tables and views (non-remote first).
The idea is to check privileges on the source object, create a synonym to the source object and then create a view as the destination with a custom set of columns.
Scripts:

db/app/ddl/src/full/R__09.ORACLE_TOOLS.PACKAGE_SPEC.PKG_REPLICATE_UTIL.sql
db/app/ddl/src/full/R__14.ORACLE_TOOLS.PACKAGE_BODY.PKG_REPLICATE_UTIL.sql

See 3.4.0


syncthing: 0.12.19 -> 0.12.21 + nixos module user support

Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[X] NixOS

[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

More
This patch only applies against master due to changes merged recently.
We are basically doing a few things in this, which I can of course break up in case people feel that makes sense:

Upgrade to 0.12.21
Add syncthing-inotify (init at 0.6.8)
Changes to the NixOS module to allow specifying:
If syncthing should run as a system service, and
If we should be using inotify
A stub package for syncthing 0.12 as the upcoming 0.13 will again change the protocol.
User services for both syncthing and syncthing-inotify so they will run for normal users on a multi-user machine.

Defaults have been set, so that behaviour is the same as before, so there should be no surprises.
Cherrypicking along with 7c8c578 makes it run against nixos-unstable.

cc @grwlf @rnhmjoj @davidak @ctheune

syncthing-inotify runs but can't communicate with syncthing (the log says it may be missing the API key). Is this supposed to work out of the box or should be configured manually?

It ran perfectly for me when testing it, so I think I may have pushed the wrong commit. Don't waste any more time on this - I'll check it tomorrow and push the proper commit/branch. I'm sorry you wasted your time on this.

Ok, no problem.

Updates?

I am breaking this PR into multiple, separate PRs so we can deal with them one at a time.

@peti you raised the issue of making a derivation with support for inotify. What would be the proper way to achieve that instead of how my PR had it as a separate package?

@peterhoeg, nevermind, I misunderstood the nature of synthing-inotify. I hadn't realized that it's a separate package entirely.


Fixed liquibase issue in kork-sql-test

Summary
Project Jira : https://devopsmx.atlassian.net/browse/OP-20333
Project Doc : NA
Issue : SqlTestUtilTest TCs are failing bcoz there is issue in setting physicalFilePath
Solution : Added changelog-master.yml file and passed physicalFilePath in DatabaseChangeLog constructor
How changes are verified
Liquibase testcase passing in intellij
Documentation Updates
Do we need to update dashboards? No
Do we need to update SOP, new hire wiki or other documents? No
Rollback, Deployment Details
Can this change be rolled back automatically without any issue?  Yes
Is this a backwards-compatible change in your opinion ?  Yes
Pre deployment steps : NA
Post deployment steps : NA

@aman-agrawal - a comment would help to understand why this change was required. Other than that it lgtm.

Initially we are getting changeLog as Null in initDatabase(String jdbcUrl, SQLDialect dialect, String dbName) method and it is causing issue for Liquibase.update(dbName). See update() method in Liquibase.java.
With this change, it is set to "db/changelog-master.yml" and Liquibase.update(dbName) is successful.