interface members can't have a defination

public interface IControl
{
void Paint() => Console.WriteLine("Default Paint method");
}
interface members can't have a defination, please remove above code lines.

They can, starting with C# 8.0/.Net Core 3.0, thanks to default interface methods. Though I think the article could be clearer that this is a new feature and provide a link with more information.

cc @BillWagner (who added this code to the article in question)

Thanks for writing this @VilasMahalle
As @svick points out, interfaces may define a default implementation for some member types starting in C# 8.
I've add this issue to the current sprint to clarify that this feature is only available in C# 8, and to link to the other articles where this is discussed.


Admin - Edit Project

Hey, first of all, I want to thank you, what an awesome project.
Now - I've found an issue, when you want to edit project, you open the form for edit, but you don't actually change anything at all and you click save again, you get an error:
Line 84:                                 @Html.DropDownList("ReassignTo", (System.Web.Mvc.SelectList)ViewBag.ProjectReassignList, new { @class = "form-control", data_val = "true", data_val_required = "Please select a project.", })
Thank you for your attention

I forgot to post the error message:
"There is no ViewData item of type 'IEnumerable' that has the key 'ReassignTo'."


Change switch PNG to SVG

I have changed the switch PNG to a SVG file. It looks better on retina displays and is a smaller file size.

Here is the same SVG but with bettercompression:


Publisher listener methods

Change set_listener to follow the DDS Standard
Add new unit test for PIM listener methods
PSM implementation
Add new unit test for PSM listener methods

NOTE: This PR must be merged after #1089

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows


Startup script fails: "Could not find or load main class org.apache.vysper.spring.ServerMain"

In https://github.com/apache/mina-vysper/blob/685a690b3aa141f7ab9f275c8c29b0b043232dd4/README.md:

A ready-to-run setup is created by the build in
  dist/target/appassembler/
Under
  dist/target/appassembler/bin
you'll find start scripts for Unix/Mac and Windows.

When I run mvn install and then try to run the startup script, here's the result:

Error: Could not find or load main class org.apache.vysper.spring.ServerMain
Caused by: java.lang.ClassNotFoundException: org.apache.vysper.spring.ServerMain

Indeed, I cannot find a class called org.apache.vysper.spring.ServerMain in the repository. There are several ServerMain classes but not one in package org.apache.vysper.spring.
Am I doing something wrong? Or is there an easy fix to get Vysper running?

This is no longer urgent for me, as I discovered the Apache Vysper Embedded in a WAR demo. But it would be nice if the startup script worked.

i got the same issue Error: Could not find or load main class org.apache.vysper.spring.ServerMain


v0.17.0 : hdfs name node can not access

v0.17.0 : hdfs name node can not access:
When I use the HDFS data in PAI, it seems that the hdfs server could not be accessed

The hadoop-node is online:

and I can access http://<IP_ADDRESS>:5070/explorer.html
The port of 5070 statues:
$ curl -i "http://<IP_ADDRESS>:5070/webhdfs/v1/?op=LISTSTATUS"
HTTP/1.1 200 OK
Cache-Control: no-cache
Expires: Fri, 20 Mar 2020 08:24:20 GMT
Date: Fri, 20 Mar 2020 08:24:20 GMT
Pragma: no-cache
Expires: Fri, 20 Mar 2020 08:24:20 GMT
Date: Fri, 20 Mar 2020 08:24:20 GMT
Pragma: no-cache
Content-Type: application/json
X-FRAME-OPTIONS: SAMEORIGIN
Transfer-Encoding: chunked

{"FileStatuses":{"FileStatus":[
{"accessTime":0,"blockSize":0,"childrenNum":2,"fileId":16400,"group":"supergroup","length":0,"modificationTime":1578626572479,"owner":"root","pathSuffix":"Container","permission":"777","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":79,"fileId":16399,"group":"supergroup","length":0,"modificationTime":1584692436570,"owner":"root","pathSuffix":"Launcher","permission":"755","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":4,"fileId":16736,"group":"supergroup","length":0,"modificationTime":1582269363897,"owner":"dr.who","pathSuffix":"datasets","permission":"777","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":4,"fileId":169490,"group":"supergroup","length":0,"modificationTime":1584330081066,"owner":"dr.who","pathSuffix":"out","permission":"755","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":1,"fileId":16419,"group":"supergroup","length":0,"modificationTime":1576920533097,"owner":"root","pathSuffix":"sys","permission":"755","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":1,"fileId":16393,"group":"supergroup","length":0,"modificationTime":1576919312644,"owner":"root","pathSuffix":"tmp","permission":"770","replication":0,"storagePolicy":0,"type":"DIRECTORY"},
{"accessTime":0,"blockSize":0,"childrenNum":2,"fileId":16386,"group":"supergroup","length":0,"modificationTime":1576919291641,"owner":"root","pathSuffix":"yarn","permission":"755","replication":0,"storagePolicy":0,"type":"DIRECTORY"}
]}}

but the port 50700 is not accessable:
$ curl -i "http://<IP_ADDRESS>:50700/webhdfs/v1/?op=LISTSTATUS"
curl: (7) Failed to connect to <IP_ADDRESS> port 50700: Connection refused

OpenPAI Environment:

OpenPAI version: v0.17.0
OS: Ubuntu 16.04.6 LTS
Kernel: Linux ubuntu 4.4.0-142-generic
Hardware:
machine-sku: GENERIC:   mem: 250   gpu:     type: 2080ti     count: 8   cpu:     vcore: 64   os: ubuntu16.04

Adding 0.17 release manager to triage the issue.

@shuangjiexu Can you show us the pylon deamonSet spec?
You can get it from k8s-dashboard

@scarlett2018 This problem is related the yarn version PAI. What's our plan about yarn version. Seems we bring some regression issue when developing pure k8s version. And not fully tested the yarn version in the latest release.

@scarlett2018 This problem is related the yarn version PAI. What's our plan about yarn version. Seems we bring some regression issue when developing pure k8s version. And not fully tested the yarn version in the latest release.

if we can provide quick fix for yarn version, I'd prefer to have a fix PR to the yarn branch and release a 0.17.1 for a stable yarn version. if the change is big, we might have to suggest user to roll back to previous version. @fanyangCS  @sterowang to comment.

There’s no benefit to upgrade to 0.17 for yarn version. I suggest users to stick to 0.14 if they prefer yarn version.

There’s no benefit to upgrade to 0.17 for yarn version. I suggest users to stick to 0.14 if they prefer yarn version.

@fanyangCS @Binyang2014  -
In this case, at least 2 things need to take to avoid user confusions:

update 0.17.0 release note with the above statement. Clarify the major changes in 0.17.0 is for the upcoming K8S version, ask user to wait for v-next version or stay with Yarn version
share the v0.17 downgrade to v0.14 guidance to users who had already upgrade
comments?

There’s no benefit to upgrade to 0.17 for yarn version. I suggest users to stick to 0.14 if they prefer yarn version.

@fanyangCS @Binyang2014 -
In this case, at least 2 things need to take to avoid user confusions:

update 0.17.0 release note with the above statement. Clarify the major changes in 0.17.0 is for the upcoming K8S version, ask user to wait for v-next version or stay with Yarn version
share the v0.17 downgrade to v0.14 guidance to users who had already upgrade
comments?

For #1, I updated the release note as below.

There’s no benefit to upgrade to 0.17 for yarn version. I suggest users to stick to 0.14 if they prefer yarn version.

@fanyangCS @Binyang2014 -
In this case, at least 2 things need to take to avoid user confusions:

update 0.17.0 release note with the above statement. Clarify the major changes in 0.17.0 is for the upcoming K8S version, ask user to wait for v-next version or stay with Yarn version
share the v0.17 downgrade to v0.14 guidance to users who had already upgrade
comments?

For #1, I updated the release note as below.

@fanyangCS @Binyang2014 - do you think we should do something on #2 issue I mentioned.

@scarlett2018 I don't think we need to provide a way for customer about downgrade v0.17 to v0.14. Since v0.17 is a breaking change. All job history will lost after upgrade to v0.17. If user want to downgrade to v0.14. The job history will also be lost, so he/she can clean the bed, then reinstall v0.14 to finish downgrade.

@scarlett2018 I don't think we need to provide a way for customer about downgrade v0.17 to v0.14. Since v0.17 is a breaking change. All job history will lost after upgrade to v0.17. If user want to downgrade to v0.14. The job history will also be lost, so he/she can clean the bed, then reinstall v0.14 to finish downgrade.

Sorry to hear that, but seems like the only approach. @shuangjiexu - do you have further questions?


Build and Deploy to PyPi is failing

Describe the bug
The build packages and deploy to PyPi is failing because it can't find requirements.txt for some reason
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

ill have a look at this


Consolidate json parsing into one implementation

Describe the bug
Currently we have two ways of doing json parsing, one in vertex.py and one in data_parsing.py.
The latter one should be made to be used in production. They both do the same thing, but there's tests written for the second and it works in some edge cases.

Looks like @c-j-johnston fixed this in 9c38a6ba30a91fa058fbec1dd86c84e854a4fd97.


import downstream cluster via access + secret token

This replaces #176 and allow the programmatical import of clusters by the use of access + secret token.
Furthermore it will allow to use the rancher2-provider to install more rancher-addons from their catalog (e.g. monitoring, logging-charts..).
(By using hdns for the racher dns-records + a local-exec  "while .. nslookup rancher-ui" it would even be possible to do both with 1 terraform-plan).
On destroy this will obviously fail  if the rancher-ui server is non-responsive/has already been deleted. But this usually shouldn't happen.
@mysticaltech as already mentioned, please check. Especially regarding the provider-setup - i'm not sure i have the new layout already internalized. Hopre this one goes smoother than the previous one.;)

Thanks @strowi , I appreciate you taking the time to submit this, but I do not understand why it is needed. Everything is already working great with the present setup without the need of a new provider!
Can you explain to me exactly what is the added value? I want to understand.

The ability to programmatically install rancher addons is interesting, but it's not a scope I think we should support. Folks csn do that from the rancher ui already.

I'm sorry @strowi , I will close that PR for now, but it you can help me understand why we really need it, we can reopen it later. Thanks for your understanding!


Please add a mention that a project must be explicitly set to allow access to other submodules in the same organization

Under Authenticated Submodules (https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/pipeline-options-for-git?view=azure-devops#authenticated-submodules)
"This requires that the job access token has access to the repository in the second project. If you restricted the job access token as explained in the section above, then you won't be able to do this."
It should also be mentioned that the project must be explicitly set to allow access to other project git repos.
Not sure exactly which of the following settings allowed me to finally get this to work:

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

ID: 8b846aea-82f7-a988-67d8-1b5e74b61ed2
Version Independent ID: 2d632729-bb33-c0a0-c996-e1d8e86c2e23
Content: Build Azure Repos Git repositories - Azure Pipelines
Content Source: docs/pipelines/repos/azure-repos-git.md
Product: devops
Technology: devops-cicd-repos
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thanks for calling this out. We are updating the doc to clarify this. You should see the update soon.


npm ERR! missing script: emulate

yo firebase-functions:function uppercaser
npm run functions:emulate```

> npm run emulate
> npm ERR! missing script: emulate

I am not sure I understand. Are you trying to run both commands? I had mapped functions emulation to npm functions:start before, not sure where npm run functions:emulate comes from

https://github.com/prescottprue/generator-firebase-functions/blob/8205b02d5eb14e6b6e41baefcbd86aed00bdda82/generators/app/templates/package.json#L10

@qalqi That is no longer part of the template which is generated, and the link you provided actually points to an old commit. Are you generating a new project or is this for an existing project? Did you try updating?

I was trying to use generator-firebase-functions instead of  generator-react-firebase

Ahh I understand - I thought this was actually on the main generator repo. Thanks for clarifying

https://github.com/prescottprue/generator-firebase-functions/blob/8205b02d5eb14e6b6e41baefcbd86aed00bdda82/generators/function/templates/_httpsFunction.js#L16
This should be response.write('Hello from <%= name %>')

@qalqi Since firebase http cloud functions are express wrapped functions, response.send() should also work right? Have you experienced it not working?
Either way, thanks for the suggestions

Yep. got error  500, Error: Can't set headers after they are sent.. after executing writeHead, we cant response.send it seems.  instead response.write or response.end
#https://stackoverflow.com/a/7086621/2560946

Agreed - response.status().send() is the right way to do it, so I'll just switch to response.end

Waiting for PR :)


[PoC] Add AVX2 version of get_sad

While playing around with rav1e I noticed it was spending a lot of time in the get_sad function, and vTune said further vectorization might help, so here's a rough hack to help rustc a little.
This should produce the same output as get_sad, and I think I've handled non-multiples of 8 sizes correctly, but this is definitely unsafe and worth double-checking!
As for results, rav1e now runs ~10% faster on my machine (encoding 24 frames at 1080p):
$ time ./target/release/rav1e -l 24 input.y4m -o output_short.ivf

User
System
Real time

Before
128,33s
0,37s
2:08,73

After
113,17s
0,30s
1:53,52

I've tried to factor out the common code in a get_sad_generic.
sse2 shouldn't be too hard to write, but rustc should already be able to autovectorize the iter/zip/map loop, it's just not the most efficient implementation.

I prepare this to make easier to reason about get_sad.
Probably we should see if making specialized version helps. I'll try to come up with something the next days.

I don't think I'll have time to rebase this right now, feel free to close if it's too much of a bother to fix up.

If nobody beats me at it I'll rebase it during the next weekend.

SSE assembly was added and the plan now is to do everything in assembly so I'm closing this for now.


"match" matcher doesn't support spaces in the right parameter

After I looked at the examples it seems to be a known and expected issue.
Hence I updated README with the notes about this behaviour which looks like a bug for new users.
If you think I should put that to a different section of a README please let me know and I'll fix it.

@timurb I think that, in general, my preference would be to incorporate this as a (self-documenting) unit test rather than just documentation.
However, since #97 is still outstanding and may obviate this documentation, let's wait and see how that shakes out first.
Regardless, we appreciate your contribution!

@timurb I'm going to close this as a duplicate of #103. We have an outstanding PR that attempts to address the issue, and I'd rather continue down that path. I see this more as a bug than as incorrect documentation.


simulations: Add scrollbar to all listviews

This will make it more obvious that the sliders list is scrollable.
Wrap all list views with an always-on scrollbar, you'll need a ScrollController to make it always on.

Sorry, this one is duplicate of #104.


GEP 1709: ConformanceReports prototype API

What type of PR is this?
/kind feature
/kind gep
/kind test
/area conformance
What this PR does / why we need it:
This supports #1784 by adding an experimental version of the ConformanceReport as described in GEP 1709. Because the GEP is currently considered to be in the Prototyping status, the rules we've set for that status indicate content must not be distributed in a release, so this PR introduces the idea of using Golang build tags as the mechanism to ensure the prototype content is not directly distributed as part of any upcoming release.

/retest all

/test all

/test all

/test all

/retest


.#x shorthand for private properties

This is a compile error:
foo ||> .#x = 1

The . is interpreted as the partial placeholder rather than member access.
Without the dot, it is not interpreted in the intended way:
foo ||> #x = 1
// vvv
(this.#x = 1)(foo), foo

It looks like .#x just generally doesn't work as a private member access helper. Should be an easy fix.


Cannot install to Angular 15.2.0 using rxjs 7.8.0

Bug Description
Conflicting Versions, Angular 15.2.0 uses rxjs 7.8.0 while ui5-webcomponents-ngx tries to use rxjs 7.5.0. Using --legacy-peer-deps or --force does not help either.
Expected Behavior
It should be able to be installed.
Steps to Reproduce

Make an Angular App using version 15.2.0
try ng add @ui5/webcomponents-ngx

Context

UI5 Web Components for Angular version: 0.1.0
Angular Version:15.2.0
OS/Platform: Windows

Log Output / Stack Trace / Screenshots
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/rxjs
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ^7.4.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@ui5/webcomponents-ngx
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     2 more (@angular/forms, @angular/platform-browser)
npm ERR!   2 more (@angular/core, @angular/forms)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@ui5/webcomponents-ngx
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!

Priority

[ ] Low
[] Medium
[ ] High
[x] Very High

Fixing, thanks for the report


New lines are escaped now

Hi!
With a recent update I've noticed that my debug data dumps in logs get fully escaped. Previously, I had nicely formatted data structs in output (using spew.Sdump), so I could actually read them.
Code:

data is a complex data structure with arrays, pointers, many custom types, optional fields etc.
log.Debug(spew.Sdump(data))

And now I get:
(*pkg.Data)(0xc423e8cb00)({\n UUID: (string) \"\",\n Property:...
I've browsed through recent updates, but can't find a way to turn this off other then fixing version of logrus to some previous one. Please advise. The data dumps are unreadable now.

This is probably because we hardened the string escaping with %q instead of the previous unsafe escaping.
See #580
I'm afraid there isn't much we can do here. This could actually be considered as a bug which has been fixed. Being able to send new lines in your log could make them harder to read, and actually reduce the purpose of structured logging, where you would only log specific meaningful fields not dump a whole json object.

This makes sense when logging to a log file, but when logging for development this is really annoying. Stack traces that are logged are now unreadable.

Yes please on this one. Having a switch of some sort would be very nice.

Another use case I have where I am printing json serialized objects in the debug logs. In the printed json, all double quotes are escaped too making it totally unreadable.

For human consumption, I am using https://github.com/antonfisher/nested-logrus-formatter which does not escape the text content.
For example:
Mar 18 09:32:26.740 [INFO]  [Content:## Generated by my-tool
# Read the policy
path "sys/policies/acl/a-policy" {
  "capabilities" = [ "read" ]
}

path "secret/*" {
  "capabilities" = ["read","list"]
}
path "auth/token/create/a-role" {
  "capabilities" = ["update"]
  "denied_parameters" = {"*"=[]}
}
] [Environment:dev]

I hope this helps.

A switch would definitely be handy...
For example LaTeX can use \n as a start of custom command very often which makes it quite hard to convert these escaped new lines back to, you know, just new lines. And debugging dynamically generated LaTeX templates without being able to actually look at the generated template is, you know, just sad.

Another vote for being able to decide whether the log should escape or not-escape newlines. Come on, this is common sense.

Why logrus keep line number and new line escape so unconvenient？ This is golang way?

Vote for an option to decide if newline should be escaped in log.
Newline is really more readable when log stack trace.

To be clear everyone is asking for an option that produces this:
time="2009-11-10T23:00:00Z" level=info traceField=line 1
line 2
line 3 otherLogField=bar

^ assuming a trace that is:
line 1
line 2
line 3

I managed to do that approximately in this way:
    if log.GetLevel()==log.DebugLevel {
        strSlice:=strings.Split(str, "\n")
        for _, s := range strSlice {
            log.Debug(s)
        }
    }

It's not ideal solution but at least it works somehow... Also it will be called only in development when you set level to DebugLevel. I think there is no need to use debug messages in production. So no need to use multiline output in production. Hope, this code helps someone;)

Need ability to enable or disable such behavior and ability to configure striping/replacing new lines in message fields.

reasonale to enable newline...

For all kinds of local testing, this option would be extremely useful.

Re-pinging on this to prevent it from getting closed. Right now, the only option that I can tell is fmt.Fprintf(logrus.StandardLogger().Out, theStr)

We solved this with a custom formatter that looks for traces in a specific field and appends them to the log. Replace stack with your field of choice.
func TestMain(m *testing.M) {
        testLogger.SetLevel(logrus.WarnLevel)
        testLogger.SetFormatter(&StackFormatter{})
        os.Exit(m.Run())
}

type StackFormatter struct {
        logrus.TextFormatter
}

// Render a log entry with a TextFormatter - except if there is a `stack` field, print it (with newlines) after the log message.
func (f *StackFormatter) Format(entry *logrus.Entry) ([]byte, error) {
        stack, ok := entry.Data["stack"]
        if ok {
                delete(entry.Data, "stack")
        }
        res, err := f.TextFormatter.Format(entry)
        if stack, ok := stack.(string); ok && stack != "" {
                res = append(res, []byte("stack trace:\n"+stack)...)
        }
        return res, err
}

It is not stale!

same to me .俺也一样

I set ForceColors true to resolve this problem, BUT I don't know why
logrus.SetFormatter(&logrus.TextFormatter{
    ForceColors: true,
})

logger.SetFormatter(&logrus.TextFormatter{
DisableQuote: true,
})
https://github.com/sirupsen/logrus/pull/1134/commits/c7455de10af6bbe2b2bdafee2cb682e813817ed2

logger.SetFormatter(&logrus.TextFormatter{
    DisableQuote: true,
})

c7455de

It's worked for me! Thanks

为什么不提供一个开关，比如我把日志写入日志文件中，在查看日志的时候，换行缩进显示 JSON 格式的日志将更加可读。Why not provide a switch, for example I write the log to the log file, when viewing the log, it will be more readable to display the log in JSON format with newline indentation.


Cadence: Queries aren't working

@johncburns1: I've implemented the first simple workflow query unit test: Test_EndToEnd.Workflow_Query_Simple.
I fixed a few related issues on the .NET side but when I make a query I get this error back from the cadence-proxy:
Message: Neon.Cadence.CadenceCustomException : QueryFailedError: Message: unknown queryType query. KnownQueryTypes=[__stack_trace]

NOTE: You can temporary increase maxWaitSeconds value in the unit test to prevent the workflow from timing out while you're debugging.

Jack fixed this.


[Feature Request] allow metrics expressions in the metricsProvider

We are interested in use datadog complex metrics expressions in the HPA in order to base our scale target in metrics arithmetics, something like this HPA metrics configuration example:
[...]
metrics:
    - type: External
      external:
        metricName: >-
          ewma_10((max:docker.cpu.usage{account:staging,kube_container_name:app,kube_service:serviceX}|max:kubernetes.cpu.requests{account:staging,kube_container_name:app,kube_service:serviceX}))
        targetValue: 80

that will base the scale on the ratio between the cpu usage and the requested cpu.
It is possible to query expressions like that (that are available through the API) into the metricsProvider? If not it would be a great feature to implement.
We're currently using a custom metrics provider that queries the datadog API and serves those metrics queries as an external metrics provider for the HPA, but we're interested on using the official datadog-agent for doing that. If you think that this feature is interesting to implement in the agent, we will gladly adapt our code into the datadog-agent.

Hi @aleixrm-scopely,
Thanks for your interest! We're actually planning to release a feature to allow arbitrary Datadog queries in the next version of Datadog Cluster Agent.
It will be through  a DatadogMetric CRD where you'll have a free text query field dedicated to that.

Great! Thanks for the quick response. We will be attentive to the next version. Is there an estimated release date?
Thanks!

It should be released in July (though, no strong commitment here).

Hi @vboulineau I'll close the issue because we tested the datadog custom metrics server and is working as expected (https://docs.datadoghq.com/agent/cluster_agent/external_metrics/). Awesome job guys! 👍🏽


this.send is not a function

Issue description

Install package from https://pkg.pr.new/riffy-team/riffy@c2f70bc
Try to play any song

Code sample
To realize that the problem is not in my code, I changed the package.json in the repository with the example (https://github.com/riffy-team/riffy-music-bot).
“dependencies”: { “riffy”: “https://pkg.pr.new/riffy-team/riffy@c2f70bc” }

Versions
riffy - https://pkg.pr.new/riffy-team/riffy@c2f70bc
djs - 14.14.1
windows 11
lavalink v4
Issue priority
High (immediate attention needed)

Issue description

Install package from https://pkg.pr.new/riffy-team/riffy@c2f70bc
Try to play any song

Code sample
To realize that the problem is not in my code, I changed the package.json in the repository with the example (https://github.com/riffy-team/riffy-music-bot).
“dependencies”: { “riffy”: “https://pkg.pr.new/riffy-team/riffy@c2f70bc” }

Versions
riffy - https://pkg.pr.new/riffy-team/riffy@c2f70bc djs - 14.14.1 windows 11 lavalink v4
Issue priority
High (immediate attention needed)

Fixed in https://pkg.pr.new/riffy-team/riffy@bcaaf04


Copy edits for analysis blog

Closes #5

Also changed the plot title font sizes to look less like the subheadings of the blog
From:

To:

looks good to me @lizgzil I can't help but this we should change the title - which one do you prefer?

Exploring the skill demand landscape
Exploring skill demand
Exploring UK skill demand

Or do you have any other suggestions?


More InfoBarPanel API udpates

Updating the API to match API spec one more time. Update naming, and don't ignore some of the values in padding/margins.

/azp run

/azp run


Content is Cutoff in Expense Items

This is related to the comment https://github.com/opencollective/opencollective/issues/4872#issuecomment-958578164
It seems that after implementing https://github.com/opencollective/opencollective-frontend/pull/7016 for expense items which has only two lines we have our collapse text behavior which is undesirable. Ideally the second line should only be blurred if a third line exists. 🤔

It's not only about blurring, we arguably don't need the "Read full description" in those cases. I would advocate simply changing the min-height to collapse to the double.

It's not only about blurring, we arguably don't need the "Read full description" in those cases. I would advocate simply changing the min-height to collapse to the double.

@Betree : Isn't that what was done in the above PR? So here we make sure only if there's a third line content is cut off. The content if cut off always at the second line, but previously we were not checking if it goes beyond the second line. So now if there's only two lines we don't show the "Read full description". 🤔


It should be possible to build TDLib for Tizen too.

It should be possible to build TDLib for Tizen too.
First you need to run native build to generate some source files:
mkdir build
cd build
cmake ..
cmake --build . --target prepare_cross_compiling

Than you can copy all source files from directories td, tdactor, tddb, tdnet, tdutils, tdutils/generate/auto, td/generate/auto into a single td/ folder, add the directory to a project in Tizen Studio and build it using the latest available compiler.

About C# .NET Tizen apps? In the build instructions there doesn't seem to be an option for Tizen.
This kind of apps need to be made in Visual Studio 2019 using a Samsung extension
Originally posted by @MarcAnt01 in https://github.com/tdlib/td/issues/908#issuecomment-580809062

It should be possible to build TDLib for Tizen too.
First you need to run native build to generate some source files:
mkdir build
cd build
cmake ..
cmake --build . --target prepare_cross_compiling

Than you can copy all source files from directories td, tdactor, tddb, tdnet, tdutils, tdutils/generate/auto, td/generate/auto into a single td/ folder, add the directory to a project in Tizen Studio and build it using the latest available compiler.

About C# .NET Tizen apps? In the build instructions there doesn't seem to be an option for Tizen.
This kind of apps need to be made in Visual Studio 2019 using a Samsung extension
Originally posted by @MarcAnt01 in #908 (comment)


Doc glossary

What this PR does / why we need it
Add a description of package repository to the glossary
Move the glossary into a separate topic
Details for the Release Notes (PLEASE PROVIDE)

Which issue(s) this PR fixes

Fixes: #2490
Describe testing done for PR
Run hugo server as per instructions in readme
Special notes for your reviewer

ya good suggestion, will do


⚠ Secret Website 2 has degraded performance

In f48d577, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 5290 ms

Resolved: Secret Website 2 performance has improved in f437708.


Extended Intraday Data Merged with

data, meta_data = ts.get_intraday_extended(symbol=ticker, interval=timeframe, slice='year'+str(year)+'month'+str(month))

"Note": "The TIME_SERIES_INTRADAY_EXTENDED API has been merged with the regular TIME_SERIES_INTRADAY API. To query current and 20+ years of historical intraday data', ' please see the latest documentation and examples at https://www.alphavantage.co/documentation/#intraday. Thank you!"

get_intraday_extended needs to be dropped, and the new functionality of getting extended intraday data incorporated into the get_intraday function.

Tracked by https://github.com/RomelTorres/alpha_vantage/pull/348.


chore: contents

ref #259
closes #263

PR-Codex overview
Focus of this PR:
This PR focuses on updating the website's content and structure, including changes to the navigation, pages, and layout.
Detailed summary:

Deleted static/pets.md, static/whoami.md, and static/fav-books.md files.
Updated the title of static/years.md to "Years".
Added a link to the "Years" page in static/tags.md.
Updated the social media icon for Twitter in _data/social.yml.
Renamed static/movies.md to static/to-watch.md and updated the title and permalink.
Updated the navigation links in _data/navigation.yml.
Updated the configuration in _config.yml, including the title, description, avatar, and logo.
Added a new page static/products.md with information about the author's products.
Updated the font-awesome library version in _includes/head.html.
Added a new page static/programming-books.md with a list of recommended programming books.
Added a new page static/about-me.md with information about the author.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 3min)


demo documents are very heavily pre-annotated

At least for letter 0 the document is very heavily annotated - should the demo have just a few annotations only to give the user an idea of what they can try themselves?

Good catch. The issue stems from the fact that the public demo session is globally accessible / shared across all users; will either update it so demo annotations are local to the user, or setup a schedule to wipe them every X hours.

I've setup a cron job that will periodically drop the demo annotations.


Feature Request: Output a full-size screenshot of the site to the right of the generated one.

As the title says, it'd be good to have the reference to the right, that's the next thing I want to do unless I want to alt-tab or use a separate screen.
This could be an option or the default, I think either way works because of the nature of the plugin. It's not meant to be 1:1 because that's too complicated (TM), so until it somehow is, perhaps the reference is useful.
I'd like this a lot!
Thoughts?

Interesting, that would require creating another API to run screenshots in a headless browser which can be a bit costly if used a lot, but I'll leave this open in case it becomes a common request I can consider adding it as an option


Space suffix after match only in non-min JS

Spent a very long time trying to figure out why the AtWho's demo page was adding a space after each inserted match while my code wasn't.
I realized that the non "min" JS file successfully adds a space after every match (like it's supposed to), while the "[...].min.js" file does not.
Thanks.

👍 This issue tripped me as well when I used non-minified version.
Then I came across this reported issue by @jeekca and it saved me from having to debug.

@jeekca can You please look into the issue #439


Update Github PR status after verification run

Adds a buildkite hook that sets job metadata for whether or not tests are passing.
Updates Girthub release PR status based on build metadata.

Related PR: https://github.com/fusionjs/probot-app-workflow/pull/75
Fixes #32

!merge


Update exercises.ml

it's useless to write if statement for boolean expressions,

if (a boolean expression) then true else false is equivalent to (a boolean boolean expression)
if (a boolean expression) then false else true is equivalent to (not (a boolean boolean expression))

Have fun ;)

Thank you very much for the feedback @morelvictor, much appreciated!


Put common autonomous code in separate java classes & files so there aren't 6+ copies

AutoCommon.java gathers up all our drive functions.
FindPropPipeline.java contains the OpenCV vision / prop detection code (used by AutoCommon).
FindPropVisInitData.java is a tiny all-data class for tweaking the prop detection setup.
Each of the .java autonomous programs now create a 'lib' object from this AutoCommon class instead of calling common functions.

@lcdnbl Let's do all the merging at the meeting tonight.


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #140   +/-   ##
=======================================
  Coverage   35.71%   35.71%
=======================================
  Files           3        3
  Lines          28       28
=======================================
  Hits           10       10
  Misses         18       18

Continue to review full report at Codecov.

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

Coverage remained the same at 34.286% when pulling 492b5c4333171c226c782c9ec8fd10b9643faf7d on template-sync-1 into 621603bc93b32fb42b5cbd540ed7ab1370bd8ba1 on master.


STAGING_AUDIT

Create a staging_audit schema.
the SOURCE table should live here.  Lee's processing can automatically update this table.  Erica's CEM processing should pull from there.
Current CSV
https://github.com/OHDSI/CommonEvidenceModel/blob/master/evidenceProcessingClean/extras/source.csv

new staging_audit.source schema/table created and loaded with current rows in evidence.source table

re-opened for @ericaVoss to update CEM process to pull from the new table

@leeevans could you open up permissions to this table?  Thank you!  I think the owner is ecmadmin, could this be altered to rw_grp?
SELECT *
FROM STAGING_AUDIT.SOURCE

ERROR:  permission denied for relation source
********** Error **********

ERROR: permission denied for relation source
SQL state: 42501

@ericaVoss try it now

@leeevans works now!  Thank you!


lb-rs: prevent overlapping syncs

@tvanderstad can you do a sanity check on the code, visually here?


when i try to train model gets below error----hdf5

akshata@akshata-Inspiron-3551:~/torch/seq2seq-attn-master$ th train.lua -data_file data/demo-train.hdf5 -val_data_file data/demo-val.hdf5 -savefile demo-model
/home/akshata/torch/install/bin/lua: /home/akshata/torch/install/share/lua/5.2/trepl/init.lua:389: module 'hdf5' not found:No LuaRocks module found for hdf5
no field package.preload['hdf5']
no file '/home/akshata/.luarocks/share/lua/5.2/hdf5.lua'
no file '/home/akshata/.luarocks/share/lua/5.2/hdf5/init.lua'
no file '/home/akshata/torch/install/share/lua/5.2/hdf5.lua'
no file '/home/akshata/torch/install/share/lua/5.2/hdf5/init.lua'
no file '/home/akshata/.luarocks/share/lua/5.1/hdf5.lua'
no file '/home/akshata/.luarocks/share/lua/5.1/hdf5/init.lua'
no file '/home/akshata/torch/install/share/lua/5.1/hdf5.lua'
no file '/home/akshata/torch/install/share/lua/5.1/hdf5/init.lua'
no file './hdf5.lua'
no file '/home/akshata/torch/install/share/luajit-2.1.0-beta1/hdf5.lua'
no file '/usr/local/share/lua/5.1/hdf5.lua'
no file '/usr/local/share/lua/5.1/hdf5/init.lua'
no file '/home/akshata/.luarocks/lib/lua/5.2/hdf5.so'
no file '/home/akshata/torch/install/lib/lua/5.2/hdf5.so'
no file '/home/akshata/torch/install/lib/hdf5.so'
no file '/home/akshata/.luarocks/lib/lua/5.1/hdf5.so'
no file '/home/akshata/torch/install/lib/lua/5.1/hdf5.so'
no file './hdf5.so'
no file '/usr/local/lib/lua/5.1/hdf5.so'
no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
[C]: in function 'error'
/home/akshata/torch/install/share/lua/5.2/trepl/init.lua:389: in function 'require'
train.lua:3: in main chunk
[C]: in function 'dofile'
...hata/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: in ?

I have the same question , have you figured out it?

You just need to install the hdf5 package:
luarocks install hdf5

thank you，i have figured out it with your reply☺

在 2017年3月24日，下午4:44，Guillaume Klein<EMAIL_ADDRESS>写道：
You just need to install the hdf5 package:
luarocks install hdf5
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

You just need to install the hdf5 package:
luarocks install hdf5

I met a new problem when I was doing this. It shows that "Missing dependencies for hdf5:
logroll "


Problem with "pub get" of "sky"

IO  : Spawning "C:\dart-sdk\lib_internal\pub\asset\7zip\7za.exe e data.tar.gz"
in C:\Users\Ross\AppData\Local\Temp\pub_a5d5d995-3e25-11e5-826d-303a64246ad9
IO  : Finished C:\dart-sdk\lib_internal\pub\asset\7zip\7za.exe. Exit code 0.
| stdout:
| |
| | 7-Zip (A) 9.20  Copyright (c) 1999-2010 Igor Pavlov  2010-11-18
| |
| | Processing archive: data.tar.gz
| |
| | Extracting  data.tar
| |
| | Everything is Ok
| |
| | Size:       98856960
| | Compressed: 30545368
| Nothing output on stderr.
IO  : Spawning "C:\dart-sdk\lib_internal\pub\asset\7zip\7za.exe x C:\Users\Ross
\AppData\Local\Temp\pub_a5d5d995-3e25-11e5-826d-303a64246ad9\data.tar" in C:\Use
rs\Ross\AppData\Roaming\Pub\Cache_temp\dira5d5d994-3e25-11e5-826d-303a64246ad9
ERR : Timed out after 30.0s while downloading https://pub.dartlang.org/packages/
material_design_icons/versions/0.0.2.tar.gz.
| This is likely a transient error. Please try again later.
FINE: Exception type: TimeoutException
ERR : package:pub/src/io.dart 831  timeout.
| dart:isolate                 _RawReceivePortImpl._handleMessage
| ===== asynchronous gap ===========================
| dart:async                              Timer.Timer
| package:pub/src/io.dart 820             timeout
| package:pub/src/source/hosted.dart 194  HostedSource._download..
| dart:isolate                            _RawReceivePortImpl._handleMessage
| ===== asynchronous gap ===========================
| dart:async                              _Future.then
| package:pub/src/source/hosted.dart 193  HostedSource._download.<fn>
| dart:async                              Future.Future.sync
| package:pub/src/source/hosted.dart 184  HostedSource._download
| package:pub/src/source/hosted.dart 93   HostedSource.downloadToSystemCache

.
| dart:isolate                            _RawReceivePortImpl._handleMessage
| ===== asynchronous gap ===========================
| dart:async                             _Future.then
| package:pub/src/source/hosted.dart 86  HostedSource.downloadToSystemCache
| package:pub/src/source/cached.dart 51  CachedSource.get
| package:pub/src/entrypoint.dart 365    Entrypoint._get.<fn>
| dart:async                             Future.Future.sync
| package:pub/src/entrypoint.dart 357    Entrypoint._get
| dart:async                             Future.wait
| package:pub/src/entrypoint.dart 133    Entrypoint.acquireDependencies.<asy

nc>
| dart:isolate                           _RawReceivePortImpl._handleMessage
| ===== asynchronous gap ===========================
| dart:async                           _Future.then
| package:pub/src/entrypoint.dart 115  Entrypoint.acquireDependencies.<async

| dart:isolate                         _RawReceivePortImpl._handleMessage
| ===== asynchronous gap ===========================
| dart:async                               Future.Future.microtask
| package:pub/src/entrypoint.dart          Entrypoint.acquireDependencies
| package:pub/src/command/get.dart 30      GetCommand.run
| package:args/command_runner.dart 178     CommandRunner.runCommand.<fn>
| dart:async                               Future.Future.sync
| package:args/command_runner.dart 131     CommandRunner.runCommand
| package:pub/src/command_runner.dart 121  PubCommandRunner.runCommand.<asyn

c>.
| dart:async                               Future.Future.sync
| package:pub/src/utils.dart 103           captureErrors.
| package:stack_trace/src/chain.dart 78    Chain.capture
| package:pub/src/utils.dart 117           captureErrors
| package:pub/src/command_runner.dart 121  PubCommandRunner.runCommand.
| dart:isolate                             _RawReceivePortImpl._handleMessag
e
---- End log transcript ----

Everything goes fine until "material_design_icons" try to download. At that point I get a timeout.

Thanks for the report. What version of pub are you using? The timeout was removed in 1.12 (dev channels).

This issue was moved to dart-lang/pub#1310


Fix return type of spl_autoload_register's callable

spl_autoload_register returns a boolean, its callable should return void https://www.php.net/manual/en/function.spl-autoload-register.php
@scrivo

You're welcome.


Using event messages in real time

I would like to use the events of the DAVIS sensor for processing in real real time. Can someone please point me towards how I can go about this?
Thanks

Hello, I also want to use the event camera for real-time processing.  (Real-time detection, tracking, obstacle avoidance)
What should I do with the event camera ? Can you tell me how you use the event camera for real-time processing？
Thanks

Did you find a solution for this?


if TextArea is multiRow then Horizontal Scroll should be visible

Spark or MX TextArea?

Spark TextArea

This seems to be working for me with mx:TextArea.


Failed to launch chrome! spawn

Actual behavior
Chrome browser not launched on exeuctin openBrowser(). Error '[FAIL] Error: Failed to launch chrome! spawn C:\Program Files (x86)\Google\Chrome\Application ENOENT'
I set the env var setx TAIKO_BROWSER_PATH "C:\Program Files (x86)\Google\Chrome\Application"
Versions
Version: 1.0.11 (Chromium:84.0.4143.0)
Node Version: 12.18.0
OS: Windows 10

Please point to the full executable path.
refer https://github.com/getgauge/taiko/issues/875#issuecomment-540389202

It worked thank you.


Observing 500 response for API "http://xxx:xx/v1/translate-equipment-sequence-ids-to-ltp-uuids "

Issue: Observing 500 response while trying the API "http://xxx:xx/v1/translate-equipment-sequence-ids-to-ltp-uuids "
Package: accessportltptranslator_v1.0.0_impl
API: http://xxx:xx/v1/translate-equipment-sequence-ids-to-ltp-uuids "
Response:
{
"code": 500,
"message": "ReadingEquipmentDataOfTopLevelEquipment - Unknown error occurred"
}
Docker Logs:
08:30:26.919] ERROR (1): error occurred step2: {}
[08:30:26.919] ERROR (1):
http_status: 500
message: "ReadingEquipmentDataOfTopLevelEquipment - Unknown error occurred"

For the analysis of this issue we would need more information what was the content of the request, for example which request was made internally by ATLT before the error was thrown. Directly before the error, equipment info is requested from MWDI.
The most likely issue would be that either part of the equipment information which is required to be present by ATLT is missing, or there happens an unknown error while requesting this data from the MWDI.

We tried for all 3 vendors
curl -X POST "http://xx:xx/v1/translate-equipment-sequence-ids-to-ltp-uuids" -H "accept: application/json" -H "user: User Name" -H "originator: Resolver" -H "x-correlator: 550e8400-e29b-11d4-a716-446655440000" -H "trace-indicator: 1.3.1" -H "customer-journey: Unknown value" -H "operation-key: Operation key not yet provided." -H "Content-Type: application/json" -d "{"mount-name":"513250004","string-of-concatenated-sequence-ids":"h20600.c10000"}"
Body:
{
"mount-name": "513250004",
"string-of-concatenated-sequence-ids": "h20600.c10000"
}
Response:
{
"code": 500,
"message": "ReadingEquipmentDataOfTopLevelEquipment - Unknown error occurred"
}
Logs: 513250004_Equipment_logs.txt
docker_logs_a2ltp.txt

The request that causes the error is described in the Equipment_logs.txt from line 366. I cannot reproduce the same error in our testing environment.
Please try the following: Execute this request as described in the line 378 of the log directly on MWDI. It should throw an error and you can compare what is missing in the request structure.


Remove 1.19, 1.20 and 1.21 warnings and update backup/restore

Remove information about 1.19, 1.20 and 1.21 kubernetes versions as they are super old.
Update backup/restore info on the "Restoring a Snapshot to New Nodes" section (token was missing)

Rebase and the test deployment should pass now.


Making sure that JMX images are available for e2e tests to run

What does this PR do?
Updates the gitlab pipelines to make sure the JMX Docker image is published for QA.
Motivation
These images are needed for running e2e tests that check JMX working correctly.
Describe how to test/QA your changes
Check pipelines worked as expected.

/merge


The MetaEvent Optimization method doesn't properly optimize output data

Description:
The reduceMetaComplexEvent method in QueryParserHelper class doesn't properly reduce the output data attributes. In certain instances (Such as for Incremental Aggregation) we can see the same attribute repeated several times
Suggested Assignees:
@suhothayan

fixed in https://github.com/wso2/siddhi/pull/682/


Remove unnecessary setters and tests for them from DefaultProject

These methods and tests for them aren't used.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Fix issue #34

Thanks @zuoxingdong ! Could you add the same to the Lua API? It is pretty straight forward if you use: https://github.com/torch/image/blob/master/doc/gui.md#res-imagetodisplaytensorinput-
Otherwise, I can add it after your PR...

@ajabri I've tried to make a Lua version. Could you help if it is correct, since I've never used Lua before.

@ajabri Does it pass the test check ?

Hi @zuoxingdong , sorry I've been swamped in the past few days but I will pull this in and test later today! Apologies!


Extract strings for translation from theme.json

Closes https://github.com/wp-cli/i18n-command/issues/224
Setup

See https://make.wordpress.org/cli/handbook/contributions/pull-requests/#setting-up
To run the behat tests you also need a working mysql locally:

Install the packages if you don't have them. In linux: sudo apt install jq mysql-server mysql-client php-mysql
composer install && composer behat => it didn't setup the database for me (mysql 8) so I did;

sudo mysql -u root
CREATE DATABASE IF NOT EXISTS wp_cli_test;
CREATE USER 'wp_cli_test'@'localhost' IDENTIFIED WITH mysql_native_password BY 'password1';
GRANT ALL PRIVILEGES ON wp_cli_test.* TO "wp_cli_test"@"localhost"

How to test

Check out this PR locally and cd to the directory.
composer install
Create a new directory called foo-theme and create a theme.json file within with the following contents:

      {
        "version": "1",
        "settings": {
          "color": {
            "palette": [
              { "slug": "black", "color": "#000000", "name": "Black" }
            ]
          }
        }
      }

Run vendor/bin/wp i18n make-pot foo-theme
Verify that there's a foo-theme/foo-theme.pot file and that the "Black" string is present as well as its context "Color name". You should see something like:

msgctxt "Color name"
msgid "Black"
msgstr ""

Stress test the network connection
Switch to offline and verify that the file is still generated properly using the local file. Upon executing the command, you should see the following message:
PHP Warning:  file_get_contents(): php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution in /home/andres/src/i18n-command/src/ThemeJsonExtractor.php on line 106
PHP Warning:  file_get_contents(https://raw.githubusercontent.com/WordPress/gutenberg/wp/trunk/lib/experimental-i18n-theme.json): failed to open stream: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution in /home/andres/src/i18n-command/src/ThemeJsonExtractor.php on line 106
Success: POT file successfully generated!

@swissspidy @schlessera do you think this is ready? The only bit under discussion is whether the theme-18n.json can be bundled with this command until we find a better path forward.

@swissspidy @schlessera what would be the next steps for this? Anything I can do to move things forward?

I'd love to get some additional feedback from @schlessera
Once merged, it would be good to follow up with another PR to improve the block.json extractor in similar fashion, re-using some of the code.

@schlessera @swissspidy I wanted to share the last fortnight of July (15th onwards) I'll have very limited availability. JFYI so you know what to expect from me in terms of time to respond/help/feedback/etc. You are very welcome to own this PR, should I not be around.

Hi folks, what's the status for this one, do you think we're still on track to land this in 5.8?

Hi folks, what's the status for this one, do you think we're still on track to land this in 5.8?

Just awaiting final review from @schlessera


Flutter tools should be more robust with paths, shells, multiple versions of installed deps

Numerous users reported issues with flutter tools not working as expected. We identified a few root causes:

a user's shell was different than what our tools expected
paths were not configured
multiple versions of installed dependencies were identified, caused confusion

We can look into making a more robust tooling experience that can "survive the wild". Flutter tools should work no matter what shell the user uses, or if they have their environment's paths set correctly, or if they have multiple versions of the Android SDK installed.

cc @devoncarew for ideas

Some notes on the current situation (and I'll speak to macos specifically, since that's where much of the problem is):

when the user installs flutter, they set up the path to flutter/bin for the shell instance they're currently in
if they start Atom in that shell, then it will inherit the paths for that process, and will be able to see flutter, as well as anything else on the path, like brew, adb, ...
if they start Atom from the mac Dock, it will run in the /bin/sh shell. The default shell on the mac is bash, so it's highly unlikely sh will have any of the proper paths for the user. And specifically, they won't have the path to the flutter script from the above shell instance
Atom will not be able to auto-locate the flutter script (done by shelling out to which). The user can set an explicit path for flutter_root, in which case we will be able to run the flutter tool. However, the tool will inherit the paths for the shell we're in, which has none of the user's settings. So flutter will not be able to locate adb, any of the brew installed tools, jarsigner, ...
We can (and currently do) try and detect the user's default shell (zsh, tcsh, bash) and shell out first to that shell, and then run the flutter tool (/bin/bash -l -c 'flutter -d sdfsdf -run). This picks up the path variable that the user has set up.  I don't think the current getting started guide has users making their path changes permanent though.
We need to shell out slightly differently depending on the specific shell the user is using. And in some cases it seems like we can't reliably determine the specific shell to use.

Other options are to make paths and locations very explicit. We'd create and read from a flutter config file, something like ~/flutter.config. This would operate similarly to .gitconfig. flutter doctor or flutter config would be able to automatically create this file w/ auto-discovered info, and the flutter tool would let the user read and write values to it. It would contain info like the path to the flutter repo, the path to the android sdk, and the paths to any other necessary tools like jarsigner, the 304 brew installed utils we use. Atom will then be able to locate flutter, and the flutter process that we spawn off will be able to know explicitly where the other apps are w/o having to have good env vars or a PATH set up. This option is more precise but requires some setup on the user's part, and perhaps a bit more knowledge about how our tools work.

Note that we also have a similar issue w/ a launcher on linux (issue files by @qchong).

The more I see this come up, the less I believe we can depend on the shell's config (especially from a GUI app!) and the more I believe we need to be explicit about our own config and provide tooling to validate/manipulate such.

This is largely fixed.  If there are remaining issues, we should file more specific bugs.


AB#8580 Athlete Detail screen restyling

Please review after merging PR#92

@sc-nelikostadinova do you get this in iOS?


Failed to start remote client due to: dial tcp :9876: getsockopt: connection refused

So how to debug this?
Tried to run this:
./apm bin apm --source="github.com/topfreegames/apm" --keep-alive

Did you try ro un ./apm serve before running ./apm bin?

Hmm I tried now, but I can't really make it.
FATA[0000] Failed to reborn daemon due to open /go/src/github.com/topfreemgames/apm/main.pid: no such file or directory.
I believe a clear example is needed.
I might write it for you the moment I figure out how to write that config file.

Can you please describe me the exact steps you're running? Also I don't understand why you're trying to run APM which is a Process Manager with another APM instance.

You don't need to write that config file. Just run ./apm serve and then ./apm bin and it should work fine. APM will create both the directory (which will be called '.apmenv' inside the folder you're running apm) and the config file for you.

Ok I'm a bit confused.
I've written a http API in Go, and all I'm trying to do is to run it forever.. what exactly do I need to pass to ./apm serve?

Nothing. Just run ./apm serve, then you can run you http api in go with './apm bin app-name --source="github.com/yourhttpapi/ingo".

I guess I'm just retarded.. I can suggest writing an example to the Readme.md file


Thoth.Json.Net depends on Fable.Core and Fable.Core shouldn't be on the server

Hey!
Thoth.Json.Net depends on Fable.Core and Fable.Core shouldn't be on the server.

Thoth.Json.Net depends on Fable.Core in order to support StringEnumattributes see this issue.
So this is normal.

How can I remove this warning?
C:\Users\...\src\function.fsproj : warning NU1603: Thoth.Json.Net 3.4.0 depends on Fable.Core (>= 0.0.0) but Fable.Core 0.0.0 was not found. An approximate best match of Fable.Core 1.0.1 was resolved.

It's really annoying

@MangelMaxime ^

Upgrade to version 3.5.1 this should be solved in this version.
3.4.0 dependencies

3.5.1 dependencies

@MangelMaxime
Thank you very very very mach!


设置时间为0结果天数为1

设置时间为0结果天数为1

我也发现这个问题了

I'll translate the issue in English:
If you show days in the count down label, you'll notice the difference. It should not use the date from 1970.1.1 for counting down.
In my case, the time left is 87225 seconds, and is should take one day to count down to zero. But if you use 1970.1.1 to calculate the result, you'll get a wrong result:

具体应该怎么设置才可以正常显示天数啊 我的总比正确的多一天


Font Spec

Related to #166
Adds the initial font specification document. This outlines the planned implementation for supporting fonts. This will be updated to match the actual implementation.
Microsoft Reviewers: Open in CodeFlow

From a PowerShell point of view are we thinking something like: WinGetFont as the Object type as opposed to a WinGetPackage?

From a PowerShell point of view are we thinking something like: WinGetFont as the Object type as opposed to a WinGetPackage?

I would prefer not to have to create an entire COM and PowerShell interface that is 90% the same with different names.  That does mean a slant mapping of faces to versions.  But version can be used to describe different flavors of things, and faces of fonts are just versions of the primary theme.  So it still fits, somewhat.


build: add additional logging when extracting the MSIs

This adds additional logging for the extraction and explicitly marks the extraction as per-machine.

@swift-ci please test


Alternative to callbacks recipe/example

There are times where you basically want a callback, but that's strongly discouraged. I'd like to have a recipe for this because it comes up a lot for me. I'm going to share my solution which I'm not too happy about, and hopefully get feedback on it, and then I'll create a PR to add the recipe or an example.
The most common situation for this is: you have a form modal. The user enters some stuff and then you send it to the server. You wait until there's a response, and then you close the modal if it succeeds. This is super easy with promises or callbacks, but it's tricky in redux.
My current solution is 'receipts'. The component creates a random id and stores it on this; it sends that as part of the action. The request is sent with middleware. The receipt id is attached to the success action. The receipt reducer 'signs' it, resulting in a state with {[receiptId]: true, ...}. The full receipt state is passed to a react component via connect. In the react component's componentWillReceiveProps checks if the receipt changed from false to true. If so, it does the thing like closing the modal.
Here's some example code: https://gist.github.com/brigand/9186cafa7ea40841121057a2547c523e
Should an exception to the 'no callbacks' rule be made to keep this simpler?
Of course, I'm open to very different ideas about how to solve this problem. Is this a problem you've run into yourself, or maybe you've found ways to avoid this?

I'm not sure what you mean by a "no callback rule", specifically.  However, you may be interested in this question I answered on SO the other day: http://stackoverflow.com/questions/39610970/how-to-pass-a-thunk-or-callback-function-into-a-redux-action-serializing-functi/39622799#39622799 .

Yeah, the redux docs you linked to in that post say "It is highly recommended that you only put plain serializable objects, arrays, and primitives into your store." which I interpret as callbacks in actions/state being bad. Also callbacks are bidirectional which breaks the unidirectional flow of flux/redux. I'm still not sure if that's worth compromising on or not.

Well, per my comments: if you're okay with time travel debugging probably not working as intended, go ahead and put callbacks in your actions and your state.
On the flip side, based on the description of your situation, I'm not sure why you'd actually need to go that route.  If you follow a dialog management approach like the one described by Dan, then there's no need to have the component look for updated props to close itself.  This logic fits very well into thunks or sagas.  For example:
function doStuffOnModalClicked(someParam) {
    return (dispatch, getState) {
        myAjaxLib.post("/saveTheThing", {someParam})
            .then(response => {
                dispatch({type : "THING_UPDATED", response);
                dispatch({type : "CLOSE_MODAL");
            });
    }
}

Ah, I can see why this hasn't come up before. My situation is a combination of edge cases (multiple modal layers, passing data back to the parent on finish) and the goal of not making assumptions, which allows me to handle the edge cases without extra effort.
Hard avoidance of assumptions has served me well so far, but it's not popular enough to make it into official docs, as far as I can tell.

For what it's worth my own app can have stacked modals as well, and they're still centrally managed by a single DialogManager component.  It's basically the same technique that Dan describes, just with an array listing open modals instead of a single variable.
Ultimately, though, per your original question: putting non-serializable stuff like functions into actions and your Redux store is generally discouraged because it breaks stuff like time-travel debugging, and other potential use cases like store persistence.  If you're okay with that, go ahead and do it.  But, it also sounds like it should be very feasible to architect your app so that's not necessary.
If you'd like to chat about this further, feel free to drop by the Reactiflux chat channels on Discord.  The invite link is at http://www.reactiflux.com .  Probably easier to discuss stuff there.

Thanks for the feedback and links! I'll try to adapt my app to use these ideas.


Fix GitHub actions

What type of PR is this?
(REQUIRED)

[ x ] bug
[ ] cleanup
[ ] documentation
[ ] feature

What this PR does / why we need it:
(REQUIRED)
All PR checks currently fail because of this change in workflows:
   https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

The ::set-env and ::add-path commands no longer work and need to be replaced.
Which issue(s) this PR fixes:
(REQUIRED)
NONE
Release Notes
(REQUIRED)
Fix for github workflow checks.

D'oh, wrong master.


Typo in require path #326 Still a problem in 0.0.42

Typo in require path #326 is still a problem for linux users in app-scripts 0.0.42

This will be published today.
Thanks,
Dan

This is resolved. Please install npm install @ionic/app-scripts@latest.
Thanks,
Dan

Tks @danbucholtz

Hi @danbucholtz .
Thanksfor the support but how can I see the sample app in my browser I am using cloud9.
Like before we can choose if local host or the private ip 172.31... but now only it says:
[21:36:23]  dev server running: http://localhost:8100/
Have I missed something. Thanks.
[21:36:10]  ionic-app-scripts 0.0.43
[21:36:10]  watch started ...
[21:36:10]  build dev started ...
[21:36:10]  clean started ...
[21:36:10]  clean finished in 4 ms
[21:36:10]  copy started ...
[21:36:10]  transpile started ...
[21:36:13]  transpile finished in 2.67 s
[21:36:13]  webpack started ...
[21:36:13]  copy finished in 2.80 s
[21:36:19]  webpack finished in 6.42 s
[21:36:19]  sass started ...
[21:36:23]  sass finished in 4.06 s
[21:36:23]  build dev finished in 13.20 s
[21:36:23]  watch ready in 13.27 s
[21:36:23]  dev server running: http://localhost:8100/

@NoelLauigan,
Please create a separate issue for that as it's unrelated.
Thanks,
Dan


move: support dvc.yaml stages (1.0) [qa]

I understand dvc move only works for data added (.dvc files) in 1.0
However dvc remove works for both stages in dvc.yaml as well as .dvc files. Should move also support dvc.yaml stages or their outputs?

Oops wrong repo... Transferring ⏳


What do you want to test?

Following up on our call today regarding testing real vs. fake http requests ...
I wouldn't say there's a one size fits all solution for this, but when you are making http requests to the internet in a package in a test, you aren't just testing your code, you're also testing the remote API. So some folks to better isolate and only test their own code (and not whether a remote API is up or down or changed) they use something (fakes, mocks, etc.) to prevent real http requests from being made.
There's a variety of options in the book. I'm most familiar with vcr and webmockr since I maintain them, and can help with those if you go that way.
Even if you do most of you tests without running real http requests, it's a good idea to have some real http requests in case the remote API changes - one way to do that with vcr by changing one env var VCR_TURN_OFF, which you could do on github actions or locally to ignore the fixtures and do real http requests
One caveat with this route is that you should be sure you're not including secrets in your test fixtures.
You could decide you do want real http requests for all your tests, and no changes needed! happy to chat more about this

Thanks for this helpful explanation @sckott.
@cansavvy I think we should definitely have some real http requests using vcr, and I can work on this down the road.

Yes for sure! I wanna make all the testing more specific and robust! So this is great. Thanks for writing this down, @sckott !

@howardbaek

have some real http requests using vcr

note that vcr helps you avoid real HTTP requests, but using that env var thing mentioned you can turn it off to make real HTTP requests. just clarifying in case there's any confusion.

Thanks @sckott . I'll let you know if I have any further questions once I drive deeper.


CodeSpace running in recovery mode due to configuration error

When I open the VS code for the CS50 class I am taking, this error code pops up:
"This code space is currently running in recovery mode due to a configuration error. Please review the creation logs, update your dev container configuration as needed, and run the "Rebuild Container" command to rectify."
I have no idea what to do. I am super new to this. Would love help. Thanks!

Version: 1.79.2
Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.5 Safari/605.1.15
Embedder: codespaces

cc @jkeech

@manchegouser the simplest thing would be to commit/push any pending changes and create a new codespace. If you want to troubleshoot the error, you can click "View Creation Log" and look for the error about why the codespace went to recovery mode, followed by running the Codespaces: Rebuild Container command once the issue is corrected.
If you are still having issues, please open a discussion at https://github.com/orgs/community/discussions which is the Community Support forum for Codespaces.


TinyMCE

How to use the tinymce for textarea

just take reference from other modules like Pages. You can check the admin.js file where we have initialized the tinyMCE on textarea.


Enable dev mode after disabled (Android)

I want to re-enable dev-mode after i disabling it using:
react-native bundle --platform android --dev false --entry-file index.android.js --bundle-output android/app/src/main/assets/
I tried removing the generated files, in the assets folder index.android.bundle and index.android.bundle.meta and then execute react-native run-android but after that the application will only crash.

@facebook-github-bot no-template


Groups Sidebar Automatic Re-ordering

Helps when you go between multiple groups.

Moved to https://github.com/fastchat/web/issues/29


bug: VNode from HTML string fails on unclosed input tags

Describe the bug
When I generate a VNode from a string using fromStringToDomNode and fromDomNodeToVNode, if it contains an <input> tag, the library fails with:
This page contains the following errors:
error on line 1 at column 96: Opening and ending tag mismatch: input line 1 and div

According to the HTML spec, input tags are represented like this:

  <input type="text">

It appears the the parser or the patch function thinks that the input tag should be closed. If I manually manipulate the string to be the following, it works:

  <input type="text"></input>

That's not correct though, and if you take any input in the dom and get its html (via outerHTML, or innerHTML, for example), it will not be closed in the resulting string.
To Reproduce
window.addEventListener("load", function() {
  let input = "
<input type='text'>
"
  let dom = fromStringToDomNode(input)
  let vnode = fromDomNodeToVNode(dom)
  patch(document.body, vnode)
})

Expected behavior
I would expect this to render without errors
Device (please complete the following information):

MacOS
Chrome
latest

Looking into this 👍

PR in  #277

Thanks so much! I haven't been in JS-land in a while, have these changes been published? How can I pick them up in my project?

@seanhess sorry about the wait! let me publish a new version for you :)


removed regex clean & model requirement for aOrbitals

Summary
Closes #143 . User should now be able to input molecules without active orbitals and run benchmarks with the algorithms using those molecules.
How to test
Check if adding a molecule without active orbitals succeeds and you get some results from running an algorithm with that specific molecule.

Codecov Report

Merging #151 (d0c69b4) into main (20990a9) will increase coverage by 0.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #151      +/-   ##
==========================================
+ Coverage   87.71%   87.98%   +0.27%
==========================================
  Files          33       33
  Lines        1050     1057       +7
==========================================
+ Hits          921      930       +9
+ Misses        129      127       -2

Impacted Files
Coverage Δ

WebCLI/forms.py
98.41% <100.00%> (+0.05%)
:arrow_up:

WebCLI/models.py
87.69% <100.00%> (ø)

WebCLI/tests/testNewMolecule.py
100.00% <100.00%> (ø)

WebCLI/views/algorithm_details_view.py
80.00% <0.00%> (+4.07%)
:arrow_up:

Continue to review full report at Codecov.

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


How to make only the first label of axis bold easily without making changes in chart js methods

I want to make the X axis first label bold by some simple way if possible not by overriding any chart js method like draw and others. If you see the img below

i want to make only the "Average Overall" bold and not the others. How can i make it

Please ask questions on StackOverflow: https://stackoverflow.com/questions/tagged/chart.js


Bluetooth: host: Fix RPA timeout expiration

Fix RPA timeout expiration when BT_EXT_ADV has been enabled.
Always invalidate the device RPA on RPA timeout.
This RPA is used by the scan roles, and the advertiser that was started
using the bt_le_adv_start API.
Call the RPA expire callback only on advertising sets that are enabled
and not being limited by a timeout and not using the identity address.
On RPA timeout always invalidate the RPA of advertising sets that are
disabled.
Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/51208
Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/51228
Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/51247
Add shell support for RPA expire callback.
This prints a message when the callback is called on the specific
advertising set.
Add a command to control the return value of this callback for a
specific advertising set.

@joerchan please update this PR with the conclusion in the comments.

I've updated the PR with the RPA expired callback now always being called when the RPA of the advertising set is no longer valid.
The application can choose to not invalide RPA on inactive advertising sets.
I've included this as a separate commit. I can squash this with the fix commit if that is preferred.

@joerchan, now it seems that the rpa_expired callback works reliably. The BT_ADV_RPA_VALID flag clearing operation seems always to be preceded by execution of the rpa_expired callback (when the BT_EXT_ADV and BT_PRIVACY are enabled). Thanks for improving callback behaviour!
And once again sorry for introducing unwanted side effects for the Bluetooth Central privacy feature with my initial rpa_expired PR

@jhedberg @alwa-nordic please re-review


Dict.fromList & Set.fromList duplicate key merge

implements #326 except for insert
Set.fromList [ a, a ] --> Set.fromList [ a ]
Dict.fromList [ ( a, v0 ), ( a, v1 ) ] --> Dict.fromList [ ( a, v1 ) ]

Honestly I'm surprised we didn't implement this already because it seems pretty useful.
The check for duplicates is n^2 but I guess that shouldn't be too bad because these explicit fromList calls are rare and usually not terribly long in non-codegen code.

This looks great, thanks @lue-bird!


Adventure: Suggest a route of X kilometers around me, via as many Nearby points as possible

Imagine you are at a train station, with 30 minutes of free time before you have to board your train.
You estimate that you can walk 2 kilometers in 30 minutes.
The app could:

Fetch points from Nearby (only points existing and needing a picture) around the user's current position, similar to what the GPX export feature is doing but instead of 10 kilometers use the half of 2 kilometers.
Use some optimization algorithm (maybe a variation of the Travelling salesman problem) to find a route that goes through as many points as possible in less than 2 kilometers, starting from the train station and ending back at the train station. Because we have no geographic data about rivers/streets/obstacles we can only make an approximation, so we should take a margin of 50%, meaning only 1 kilometer as the crow flies from point to point.
Show the route on the Nearby map, apparently OsmDroid can draw polylines.
Show a button to open the route on Google Maps, I hope we can construct a URL like this: https://www.google.com/maps/dir/MUSEUM+OF+ICE+CREAM,+558+Broadway,+New+York,+NY+10012,+United+States/Washington+Square+Park/Ukrainian+Village/@40.7251567,-73.9958407,15.96z/data=!4m20!4m19!1m5!1m1!1s0x89c259e713d43875:0xbfd5eb3c3e1bb328!2m2!1d-73.9979167!2d40.7238545!1m5!1m1!1s0x89c25990e23d7e8d:0xfa615edfd1b67e18!2m2!1d-73.997332!2d40.7308838!1m5!1m1!1s0x89c2599cd5c824f5:0xb1c419b3b8a7b355!2m2!1d-73.9874105!2d40.7273114!3e2?entry=ttu This allows the user to check whether the route actually takes less than 30 minutes or not.

I would suggest "Adventure" as the activity's name to make it fun, but other name suggestions (and other idea improvements) are welcome. :-)
Once this is done, the next phases (as separate issues) could be:

Walk from a point to another.
As an alternative to asking for kilometers, offer a way to input for instance "30 minutes walk", "30 minutes cycling", "30 minutes by car".

It seems really fun to work on this feature. I would like to work on this enhancement :)

@nicolas-raoul this seems to be closely related to #1118 ?

@rohit9625  I unassign for now, but if you are you still working on this, please let us know. If no answer, someone else may be assigned to it. Thanks a lot. :-)


jenkins_plugin task no longer supports params option

Ansible 2.5 made some changes to the jenkins_plugin task which prevents plugins from being installed.
03:22:41 TASK [geerlingguy.jenkins : Install Jenkins plugins using password.] ***********
03:22:42 failed: [test] (item=job-dsl) => {"changed": false, "item": "job-dsl", "msg": "The params option to jenkins_plugin was removed in Ansible 2.5since it circumvents Ansible's option handling"}

There's a link to this discussion in jenkins_plugin.py: https://meetbot.fedoraproject.org/ansible-meeting/2017-09-28/ansible_dev_meeting.2017-09-28-15.00.log.html
Looks like the fix is just to replace params with args.
Somebody also seems to have reported this in #164 .

Thanks for opening this! I'm running into this now as well. Do you know if the change is backwards-compatible with 2.4? I haven't tested yet... but it would be really annoying to have to maybe maintain two versions of the role while people upgrade to 2.5 :/

Thanks for the issue report and PR! This is fixed in 3.3.0.

This change has broken the module. Params was used to circumvent the modules fixed parameter list I suspect.
http://docs.ansible.com/ansible/latest/modules/jenkins_plugin_module.html
Doesn't have a url_token when it's used as a parameter directly and an error is thrown. I /think/ tokens are just used as basic auth passwords a la:-
curl 'http://admin:token@JENKINS_URL:8080/
So possibly the following is just required:-
- name: Install Jenkins plugins using token.   jenkins_plugin:     name: "{{ item }}"     url_username: "{{ jenkins_admin_username }}"     url_password: "{{ jenkins_admin_token }}"     updates_expiration: "{{ jenkins_plugin_updates_expiration }}"     url: "http://{{ jenkins_hostname }}:{{ jenkins_http_port }}{{ jenkins_url_prefix }}"   with_items: "{{ jenkins_plugins }}"   when: jenkins_admin_token != ""
This highlights a gap in the test plan. I think you can setup a 2 stage install test though and grab the token programmatically:-
https://stackoverflow.com/questions/36633725/generate-jenkins-api-token-using-rest-api
then feed that back in.

it still does not work, the same issue - plugins are not installed but script went through

This fixed the issue for me, but I don't use the token method like @gilesw . Is that what you are using, or are you just using the password method?

I'm hitting the following issue:
failed: [jenkins-dev] (item=job-dsl) => {"changed": false, "item": "job-dsl", "msg": "Unsupported parameters for (jenkins_plugin) module: url_token Supported parameters include: attributes, backup, client_cert, client_key, content, delimiter, directory_mode, follow, force, force_basic_auth, group, http_agent, jenkins_home, mode, name, owner, params, regexp, remote_src, selevel, serole, setype, seuser, src, state, timeout, unsafe_writes, updates_expiration, updates_url, url, url_password, url_username, use_proxy, validate_certs, version, with_dependencies"}

I have the following variables set:
jenkins_plugins:
  - job-dsl
  - credentials
  - git
  - workflow-aggregator

jenkins_plugin_timeout: 120
jenkins_admin_user: foo
jenkins_admin_pasword: bar
jenkins_admin_token:<PHONE_NUMBER>1234567890

Please see: https://github.com/geerlingguy/ansible-role-jenkins/issues/205
I don't currently use the token method, so I don't know how much I can help there...


Added required param in BindStyledParameterWithLocation

Thank you.

Thanks for raising this @itpavelkozlov - to confirm, did you take the code that @renom wrote in https://github.com/oapi-codegen/runtime/pull/14 or was your implementation from scratch?


init Callback is not being called on the first run

Hi,
I know this issue is the same as #272 but it is still remain unresolved.
To reproduce, just download the code example and run the project.
When you click the start button for the first time you will not see on the console the lines called with print().
However when you click the stopped button and click on the start button again, you will see on the consoles the lines called with print().
I really hope that this issue will be fixed. Tried downgrading from 1.6.12 to 1.6.6 but issue is the same. The init callback is not being called on the first run, hence the callback method could get the passed init params.

same experience in my app too


Code for graph classification task

Thanks for your amazing work! And I'm also very interested in the graph classification in your work. May I have your code for graph classification task?

Hello,  I'm also looking for code on graph classification. Have your found any implementation of graph classification?

Hi, I am also looking for code on graph classification. Have you found the implementation?


Test fails to run in the collections format if the role name contains '-' (dash)

This is just a testing convenience, but if converted to the collections format, ci-testing/tests/tests_default.yml fails as follows:
ERROR! the role 'fedora.system_roles.ci-testing' was not found in /tmp/tmp4fnee3t7/tests/ci-testing/roles:~/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/tmp/tmp4fnee3t7/tests/ci-testing
The error appears to be in '/tmp/tmp4fnee3t7/tests/ci-testing/tests_default.yml': line 7, column 7, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:
  roles:
    - fedora.system_roles.ci-testing
      ^ here

If it's renamed to ci_testing, it passes.
Note: lsr_role2collection replaces '.' (dot) in a sub-role name, but '-' (dash) is not. Also, the main role names are not touched.

Not only that - if the role is referenced in collection_release.yml to test the release automation, the release_collection tool (correctly) aborts with "ERROR: The import failed for the following reason: role name invalid format: ci-testing".


How to set constraints?

I have a viewcontroller in storyboard, that I am using as a custom presenter. Im setting constraints for the viewcontroller, but when it shows as a custom popup, all my views are all over the place..
I'm not exactly sure how to set proper constraints to make this work, could you please guide me a little,
Thanks!

Hey @jadslim313 !
Well basically with auto-layout the goal is to make a screen work in any size (iphone, plus, ipad, etc.) so Presentr is counting on that so that it does not matter the view controller size it looks "good".
There are some limits to this of course, for example if you present a tiny view controller size where your views do not fit, it will obviously not look good. But generally you should not have this problem.
With auto-layout each view should have a width and height (either by fixed constant contraint, percentage constraint or intrinsic size like in a UILabel) , and each view should have an X and Y position constraint which separates the view from the superview in both axis. This can also be a constant, a percentage, it can be a vertical or horizontal "center" constraint, anything that gives it an x and y position is good.
If you do this for all views in your view controller you should be good. But this applies to any view controller, not just for Presentr.
For example, you can try viewing your view controller in the Interface Builder in different phone sizes and see how your view constraints react to the changes.

Let me know if you can fix this. Maybe with some sample code.
Thanks for using Presentr :)


Improve preload speed

When doing HasOne and BelongsTo Preloads (and maybe others also), GORM is quite slow when the there are many query results and many referenced results.
Example scenario:

Table User and table Profile, where User "belongs to" Profile (i.e. User has a ProfileID FK referencing a column ID in Profile)
Query all users in a table with Preload("Profile")
If there is large number of users (and large number of profiles), GORM is slow

I've looked at the code, and found that handleBelongsToPreload and handleHasOnePreload in callback_query_preload.go are inefficient when assigning results of relation queries.
Currently, ForeignField values from one set of results and AssociatedForeignField values from the other set are compared as strings in a nested for loop. This is quadratic complexity, which is quite bad if there is large number of rows (more than a few thousand).
This can be improved by storing field values of one set of results in a map with a string key, iterating through the field values/objects of the other set and looking for the same values in the map.

The issue and speed improvement I mentioned happens after the SELECT query, when the foreign field values returned by query are actually assigned to objects.
Duplicate keys in queries should also be optimized potentially, although I don't think impact on preload speed is as big as the assignment of field values to objects. I believe sql is smart enough to remove duplicates when executing the query. However, removing duplicates before would help to reduce data transmitted over network and might also help with speed in some cases.

The more reasonable way to tackle a performance problem is not to try to guess where the bottleneck is, but use a profiler and actually see what is causing slowness.
A quadratic-time loop is kind of a low hanging fruit for immediate speed improvements though. How about a PR?

For me I just noticed the redundancy inside my query while debugging. I do not face actual performance issues. I was just wondering if this issue could be related and posted here instead of opening a separate issue.
I did not intended to "guess" where the bottleneck is, if that's what you mean ;)

Is this still an issue ? am closing
cos imho, it a chat about things that not relevant.. atmo
BUT please repoen if missing the point.. thanks all

I believe the issue from my OP still exists.
Since I needed this in my project immediately, I forked the repo and did a quick fix of one case. You can see the change here.
Unfortunately, I don’t have time to make a proper PR here...

I also have a need for this improvement, since the Preloads take several tens of seconds for fairly average resultsets (30000 simple records).
For now I am manually querying the Preload-objects and then looping the main resultset in order to populate the missing artifacts (which only takes a couple of milliseconds for the same resultset). It isn't a complex task, shouldn't gorm be doing it in the Preloads out-of-the-box?

I'm also having this problem, Preloading is very slow, and with @vanjapt 's patch it works much faster.
It would be great if this improvement can be merge.


Filter out "default integrations" from integration list

Great idea / improvement!
Would it be an idea to be able to "filter out default integrations" - integrations that HA always loads - in the list of integration usage? Reason for bringing this up is that some integrations will always hog the first few lines as they are always (or almost always) loaded with HA (i.e. auth, zone, websocket, frontend etc)
Looking at the list now, this would for instance at least skip the first 23 lines potentially even more.
I am thinking of a checkbox style kind of thing, which upon selection removes the "ha default integrations" out of the list in the users browser. All at the user side - I am not suggesting to remove the data from the backend, just filter it in the frontend list when the box is ticked.

Hiding defaults from the portal are already in the works https://github.com/home-assistant/analytics.home-assistant.io/pull/17.
We use Github to track issues, if that PR does not cover your need, feel free to open a Feature Request on the forums :+1:

Totally missed that one 😬
Thanks Joakim!


Feature default command

Hello Louis-Gabriel,
I've another PR as suggestion, because I think we should not run containers when we run only the devilbox-cli command.
Perharps as I did it in this PR, only show the version and help screen. That will allow to make possible config change before running containers.
Tell me what you think about it !
BTW, I've another PR that I would like to suggest. It will add a new compose command to help to handle additional docker services.
Tell me also, if you think it will be useful for community and your project.
Thanks in advance
Laurent

Thanks again @llaville, you are right it seems to be the behaviour users are used to on other cli tools
I just published 0.3.2 with your changes


Explain sequencing

At least a brief introduction about primers, amplification, library generation... It is kind of basic but it is important later for the discussion on microbiota and comparing multiple microbiome datasets (V1 vs V3-V4..., ASV vs OTUs)...
Also it might be interesting to have on writing how the human mRNA is sequenced (selection, primers...)

Already sent without this section... I might need to add it later or maybe it is not so important to include on the introduction. At least a sentence and some references with pointers about how to do that...

It is mentioned on the appendix (at least how the STAR and RSEM part was done)


Hacked-Emails Bad Res + advancedcheckbackground never run

Hi,
So first thing first i have no banner thats start directly on lookup prompt ;)
But the real problem is the Hacked Email who give me this error:
Bad Res: list index out of range                                                 Bad Res: local variable 'trsplit' referenced before assignment
And the AdvanceBackgroundChecks run for ever without output

post the command you're trying to use

also hacked-emails has changed their formatting and moved to ipv6, will look into it for next release

hello sir,
sry for the delay, i use python skiptracer.py -l email
and after that i have directly the prompt, not the banner and for hacked-email ihave
[?] Hacked-Emails Bad Res: list index out of range Bad Res: local variable 'trsplit' referenced before assignment   [X] Attempting to get source, again ... Bad Res: list index out of range Bad Res: local variable 'trsplit' referenced before assignment   [X] Attempting to get source, again ... Bad Res: list index out of range Bad Res: local variable 'trsplit' referenced before assignment   [X] Attempting to get source, again ... Bad Res: list index out of range Bad Res: local variable 'trsplit' referenced before assignment   [X] Attempting to get source, again ... Bad Res: list index out of range Bad Res: local variable 'trsplit' referenced before assignment   [X] Failed at accessing site ... Try again later ...
But yeah i think there is something with hacked-emails because i have an other tools in python which produce error with them too
For the AdvanceBackgroundChecks i have no output at all after 30min running so i stop it.
Thanks for all, have a nice day

Comment out the Hacked emails from the skiptracer.py for skippy to run correctly.
#from plugins.hackedemails import HackedEmailGrabber
If you just run:
python skiptracer.py
you will get the banner you are looking for and than you can enter "phone|name|plate" etc.
If you run it with the "-l" (python skiptracer.py -l)  you will get the banner after it completes the query and returns the results.

hacked-emails changed their format to not offer lookups, it will email the person first. will be removed on version 2. for now as pingywon said comment it out. still checking on the advancedbackground issue.


[8.x] Add optional arguments to value() for the closure

This PR aims to add arguments to value() helper that will be passed to the closure.
Examples
// Single value
$test = fn ($name) => 'Hello ' . $name;

echo value($test, 'John'); // Hello John

// Multiple values
$test = fn ($firstName, $lastName) => sprintf('Hello %s %s', $firstName, $lastName);

echo value($test, 'John', 'Doe'); // Hello John Doe

// Same result but with splat operator if working with an array
echo value($test, ...['John', 'Doe']); // Hello John Doe

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


[SPARK-32577][TEST] Fix wrong config for shuffled hash join in test in-joins.sql

What changes were proposed in this pull request?
Fix Fix wrong config in test in-joins.sql
Why are the changes needed?
Fix wrong config in test in-joins.sql
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #130735 has started for PR 30280 at commit a54a3c6.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35344/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35344/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35344/
Test PASSed.

Test build #130735 has finished for PR 30280 at commit a54a3c6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130735/
Test PASSed.


Wait until node is ready before trying to ssh

https://ocd.appscale.com:8080/job/Daily Build/1386/


fix: reconcile Pooler service and add labels.

This patch fixes two issues that we currently have with the Pooler service.
a) we never reconcile the service once is created
b) we don't add the proper labels to the created service

/test limit=local

LGTM


Include property descriptions in response schema?

Hello. Is there a way to have response schemas display property descriptions, in addition to the name and type?
Currently, running OPENAPI=1 bundle exec rspec  generates a file like:
openapi: 3.0.3
info:
  title: rspec-openapi
paths:
  "/tables":
    get:
      summary: index
      tags:
      - Table
      parameters:
      - name: page
        in: query
        schema:
          type: integer
        example: 1
      - name: per
        in: query
        schema:
          type: integer
        example: 10
      responses:
        '200':
          description: does something cool
          content:
            application/json:
              schema:
                type: array
                items:
                  type: object
                  properties:
                    id:
                      type: integer
                    resource:
                      type: string
                    # ...

I'd like to generate a file including property descriptions:
openapi: 3.0.3
info:
  title: rspec-openapi
paths:
  "/tables":
    get:
      summary: index
      tags:
      - Table
      parameters:
      - name: page
        in: query
        schema:
          type: integer
        example: 1
      - name: per
        in: query
        schema:
          type: integer
        example: 10
      responses:
        '200':
          description: does something cool
          content:
            application/json:
              schema:
                type: array
                items:
                  type: object
                  properties:
                    id:
                      type: integer
                      description: 'Description of ID from the schema'
                    resource:
                      type: string
                      description: 'Description of name from the schema'
                    # ...

Property description is not automatically generated yet.
We need to figure out how to annotate properties in RSpec/Minitest.
As a workaround, one can manually add description to the generated properties.
rspec-openapi preserves unsupported properties (description for this case) to allow manual modification.


Problem when directive is used inside ngIf

I am using your directive to hide modal on click-outside, but I am facing a problem when modal is inside *ngIf, and when I click to open modal, your directive is called and modal close at the same time, I tried to fix this with some async function, but it didn't work. I notice that your directive is not working correctly when it is using with *ngIf.
So this helped me:
onClickedOutside(e: Event) {     if (this.counter < 1) {       this.counter += 1;     } else{       this.closeModalOnClick();       this.counter = 0;     }   }
Now basically I have to add this on 10 places when I am using your directive inside *ngIf.

Did you try using delayClickOutsideInit?

Going to close this as a dupe of #40.