multi directional?

Could the scroll be multi-directional?
if i have an element off viewport and both scrollbars are visible, right now it just scrolls horizontally to the elements left or vertically elements top.
is it possible to scroll diagonally to the element?

You may achieve that by using the PageScrollService and starting two PageScrollInstances with different namespaces. One scrolls horizontally, the other one vertically.
https://plnkr.co/edit/nYS3rVk4nLwXmRkBxqZj?p=preview
If it's not possible to properly scroll to the top/left corner of the target element (e.g. as the overall scroll distance is not large enough), the scroll animations may not reach the target at the same time.

thank you so much, I appreciate it greatly!
On Wed, Mar 15, 2017 at 7:45 AM, Sebastian Fuss<EMAIL_ADDRESS>wrote:

You may achieve that by using the PageScrollService and starting two
PageScrollInstances with different namespaces. One scrolls horizontally,
the other one vertically.
https://plnkr.co/edit/nYS3rVk4nLwXmRkBxqZj?p=preview
If it's not possible to properly scroll to the top/left corner of the
target element (e.g. as the overall scroll distance is not large enough),
the scroll animations may not reach the target at the same time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nolanus/ng2-page-scroll/issues/102#issuecomment-286718414,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFO2vOUYX-4zcsv7MiPgKg28kSMkWaxKks5rl89rgaJpZM4MdwvN
.

--
Hrayr Mirzoyan


Add initial zeroes to snapshot date string suffix

Problem
When changesets generates a date string suffix for a snapshot, leading zeroes are not added to single-digit numbers, which means that it’s possible for newer generated versions to be lower than older versions. Here’s the one that bit me:
0.0.0-canary-20217561928 — old version, published 2021-08-05 06:19:28
0.0.0-canary-2021714575 — new version, published 2021-08-14 05:07:05
202175 is greater than 202171, therefore installing version<PHONE_NUMBER> will give you deps matching<PHONE_NUMBER>8.
Solution
Add leading zeroes to the date numbers in getSnapshotSuffix:
let dateAndTime = [
  now.getUTCFullYear(),
  now.getUTCMonth(),
  now.getUTCDate(),
  now.getUTCHours(),
  now.getUTCMinutes(),
  now.getUTCSeconds()
]
  .map(number => number.toString().padStart(2, "0"))
  .join("");

Potential issue, needs discussion
After this change lands, all new versions will now start with 202107, which happens to be lower than 20217, meaning that all new versions will now be lower than any version published this year. Not good!
Potential solution

Use single digits for numbers less than 36 but use toString(36) to ensure that there will always be one digit. If now.getUTCHours() returns 12, it’ll be stringified to c.
const now = Date('2021-08-14T15:33:42.426Z');

let dateAndTime = [
  now.getUTCFullYear(),
  now.getUTCMonth().toString(36),
  now.getUTCDate().toString(36),
  now.getUTCHours().toString(36),
  now.getUTCMinutes().toString(36).padStart(2, '0'),
  now.getUTCSeconds().toString(36).padStart(2, '0'),
].join("");

output: 20217ef0x16

Shorten the year to two digits. 21 is greater than 20, problem solved.
const now = Date('2021-08-14T15:33:42.426Z');

let dateAndTime = [
  now.getUTCFullYear() - 2000,
  now.getUTCMonth(),
  now.getUTCDate(),
  now.getUTCHours(),
  now.getUTCMinutes(),
  now.getUTCSeconds(),
]
  .map(number => number.toString().padStart(2, "0"))
  .join("");

output:<PHONE_NUMBER>42

Thank you for the PR! Would you mind adding a test case for this?

Sorry that we didn't get to merging this - it turns out though that this has been fixed now in https://github.com/changesets/changesets/pull/692

yay, glad it’s fixed!


Just a question.

I'm away for a long time too so I can't help.


How do you multi search?

I'm working on a task where I need to get a set of result in JSON from 3 different indices.
So, I was thinking of using the Elastic Multi Search API.
How do I use muti search with this client? Guides or suggestions would be very helpful.
Thanks!

Hi @aniskhan001 , the link provided by @geoffmore is indeed what you need. Please re-open if you'd run into issues.

@karmi That link wasn't much help I'm afraid. A simpler answer would be to implement the following:
	var buf bytes.Buffer

	match := []interface{}{
		map[string]interface{}{"match": map[string]string{"podcast_url": podcast_url}},
		map[string]interface{}{"match": map[string]string{"text_value": word}},
	}

	query := map[string]interface{}{
		"query": map[string]interface{}{
			"bool": map[string]interface{}{
		  		"must": match,
				},
			},
		}

	if err := json.NewEncoder(&buf).Encode(query); err != nil {
		log.Fatalf("Error encoding query: %s", err)
  	}

I think it would be a good idea to update your examples to include this multi search logic as it's not very clear at all, and took me a good bit of trial and error to figure out. (There is literally no documentation out there which shows how this works).

In general, the client accepts the search definition or other payload as an io.Reader, which is what you need to pass to the Msearch() function — that should correspond exactly to what you find eg. in the documentation on the elastic.co website.
Note that to facilitate encoding custom structures or map[string]interface{} types, the package provides the esutil.JSONReader helper:
https://github.com/elastic/go-elasticsearch/blob/e9eb76d55d12256a0777ad825bf25cd5e02fa105/_examples/encoding/jsonreader.go#L30-L34

I am able to get Multi search results directly from the Elastic Search API using _msearch for the following request Body
`
{"index":"csv1"}
{"query" : {"match" :  {"_meta._clusterID": "8811547417"} }, "size" : 1}
{"index":"csv1"}
{"query" : {"match" :  {"_meta._clusterID": "88115479144"} }, "size" : 1}
{"index":"csv2"}
{"query" : {"match" :  {"_meta._clusterID": "86443104548"} }, "size" : 1}
{"index":"csv3"}
{"query" : {"match" :  {"_meta._clusterID": "86642503112"} }, "size" : 1}`
However, the above suggestions are very difficult to understand how to use the go-elasticsearch API for Msearch.  What should the MsearchRequest look like , I tried using the JSON option, It always complains an error.
If someone did use this API, any examples would be highly appreciated.


Update dependencies, add Scala Native support for circe integration, add jsoniter-scala integration

Close #6 by adding jsoniter-scala module
@ybasket @satabin Can I use code from this PR in a tutorial for jsoniter-scala?

@plokhotnyuk thank you for contributing! I unfortunately don't have the time right now to dive deep into how the codecs work, but it looks good at first glance and I'll try to find some time soon-ish if @satabin isn't faster.
As for using the code in a tutorial, no objections from my side, looking forward to read it. A link back to geo-scala would of course be appreciated :)

@ybasket Please approve workflow run, I hope the latest commit will fix CI build failures

@ybasket Please approve the workflow run, I hope the latest commit will fix CI build failures

Done. Sorry, we probably should have updated after the last release (still learning the details of sbt-typelevel).

Ups, it failed again... Now looking a way how to generate CI scripts with Java 11 for the jsoniterScala module...

@armanbilge Thank you a lot! It looks like a clear solution (not a workaround) isn't it?

Yes, I would consider it to be a solution. In fact there is a proposal to make a configuration like this the default for the next major release of sbt-typelevel.

https://github.com/typelevel/sbt-typelevel/issues/215

Thank you, @ybasket! I missed to run the githubWorkflowGenerate task again

Btw, I think this can be removed:
https://github.com/gnieh/geo-scala/blob/876e75ca8c5766944449eaa54ad4a90cbba01c78/build.sbt#L30-L32

Note: you should never set mimaPreviousArtifacts := Set.empty. This is wrong. You always fall into one of two scenarios:

You are not publishing that module. So .enablePlugins(NoPublishPlugin).
It is a brand new artifact, not yet published. So set

  tlVersionIntroduced := Map("2.12" -> "1.2.3", "2.13" -> "1.2.3", "3" -> "1.2.3")

Assuming that 1.2.3 is the first version it will be published in.

@armanbilge IMHO just removing of mimaPreviousArtifacts := Set.empty doesn't enable checking of binary compatibility...
Have the sbt-typelevel plugin something to replace the following workaround?
  mimaPreviousArtifacts := {
    import scala.sys.process._

    val oldVersion = "git describe --abbrev=0".!!.trim.replaceAll("^v", "")

    def isCheckingRequired: Boolean = {
      val Array(newMajor, _, _) = version.value.split('.')
      val Array(oldMajor, _, _) = oldVersion.split('.')
      newMajor == oldMajor
    }

    if (isCheckingRequired) Set(organization.value %% moduleName.value % oldVersion)
    else Set()
  }

Yes, sbt-typelevel automatically determines the previous artifacts for MiMa binary-comaptibility checking. You can read more about it in the documentation.
https://typelevel.org/sbt-typelevel/faq.html#what-is-a-base-version-anyway

I rolled back and force pushed updating of docs but the CI build still fails in the static code analysis part...

What is Codacy Static Code Analysis? How to open logs or results of its checks?

@ybasket @satabin Should I fix "issues" reported by Codacy Static Code Analysis?
Some of them like in README.md are not fixable, others will lead to less readable and more error prone code...

Hi @plokhotnyuk You can disregard Codacy, their profile is not tuned yet for our usage.

@ybasket @satabin This PR is ready for final review.
I propose make other improvements for binary compatibility checks and replacement of List by ArraySeq in subsequent PRs

@ybasket @satabin This PR is ready for final review.
I propose make other improvements for binary compatibility checks and replacement of List by ArraySeq in subsequent PRs

@armanbilge I have tried sbt +mimaReportBinaryIssues without  mimaPreviousArtifacts := Set.empty line and it doesn't work.
All modules are still unchecked due to empty mimaPreviousArtifacts.
We should set it explicitly, isn't it?

Is the base version correctly set?

@plokhotnyuk yes, that is the correct, expected behavior. Because you have bumped the base version from 0.2 -> 0.3 you have signified that you are making a breaking change, and therefore sbt-typelevel stops checking against previous versions.
You should not update the base version if you want to continue checking against previous versions.
I highly recommend you read the material that I linked you to before :) it explains this in detail, with examples.
https://typelevel.org/sbt-typelevel/faq.html#how-do-i-introduce-breaking-changes-intended-for-my-next-version
https://typelevel.org/sbt-typelevel/faq.html#what-is-a-base-version-anyway

@armanbilge I don't understand how all the magic should work... Locally when I revert the base version back to 0.2 it doesn't validate any module.

Have you pulled the tags to your local clone? Otherwise it will be unable to detect previous versions locally (but tags are always present in CI).

Btw, I see that v0.3.0 is already released. So now I am confused what is happening in your PR.
https://github.com/gnieh/geo-scala/releases/tag/v0.3.0

For the record: if I leave the tlBaseVersion := "0.3" and I remove the mimaPreviousArtifacts := Set.empty, then it is all configured correctly.
sbt:geo-scala> show mimaPreviousArtifacts
[info] jsoniterScalaJVM / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-jsoniter-scala:0.3.0 (e:info.versionScheme=early-semver))
[info] coreJS / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-core:0.3.0 (e:info.versionScheme=early-semver))
[info] circeNative / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-circe:0.3.0 (e:info.versionScheme=early-semver))
[info] jsoniterScalaJS / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-jsoniter-scala:0.3.0 (e:info.versionScheme=early-semver))
[info] jsoniterScalaNative / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-jsoniter-scala:0.3.0 (e:info.versionScheme=early-semver))
[info] circeJVM / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-circe:0.3.0 (e:info.versionScheme=early-semver))
[info] coreNative / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-core:0.3.0 (e:info.versionScheme=early-semver))
[info] polylineJVM / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-polyline:0.3.0 (e:info.versionScheme=early-semver))
[info] circeJS / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-circe:0.3.0 (e:info.versionScheme=early-semver))
[info] polylineJS / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-polyline:0.3.0 (e:info.versionScheme=early-semver))
[info] coreJVM / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-core:0.3.0 (e:info.versionScheme=early-semver))
[info] polylineNative / mimaPreviousArtifacts
[info]  Set(org.gnieh:geo-scala-polyline:0.3.0 (e:info.versionScheme=early-semver))
[info] mimaPreviousArtifacts
[info]  Set()
sbt:geo-scala>

It doesn't work for me... Have I missed to run some sbt task?
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ sbt
[info] welcome to sbt 1.7.1 (Azul Systems, Inc. Java 17.0.2)
[info] loading global plugins from /home/andriy/.sbt/1.0/plugins
[info] loading settings for project geo-scala-build from plugins.sbt ...
[info] loading project definition from /home/andriy/Projects/com/github/plokhotnyuk/geo-scala/project
[info] loading settings for project root from build.sbt,publish.sbt ...
[info] resolving key references (15836 settings) ...
[info] set scmInfo to https://github.com/plokhotnyuk/geo-scala
[info] set current project to geo-scala (in build file:/home/andriy/Projects/com/github/plokhotnyuk/geo-scala/)
[info] sbt server started at local:///home/andriy/.sbt/1.0/server/0eabc375e6e88395df69/sock
[info] started sbt server
sbt:geo-scala> show mimaPreviousArtifacts
[info] coreJVM / mimaPreviousArtifacts
[info] 	Set()
[info] jsoniterScalaJVM / mimaPreviousArtifacts
[info] 	Set()
[info] circeJS / mimaPreviousArtifacts
[info] 	Set()
[info] coreJS / mimaPreviousArtifacts
[info] 	Set()
[info] polylineNative / mimaPreviousArtifacts
[info] 	Set()
[info] polylineJS / mimaPreviousArtifacts
[info] 	Set()
[info] circeNative / mimaPreviousArtifacts
[info] 	Set()
[info] jsoniterScalaJS / mimaPreviousArtifacts
[info] 	Set()
[info] circeJVM / mimaPreviousArtifacts
[info] 	Set()
[info] jsoniterScalaNative / mimaPreviousArtifacts
[info] 	Set()
[info] polylineJVM / mimaPreviousArtifacts
[info] 	Set()
[info] coreNative / mimaPreviousArtifacts
[info] 	Set()
[info] mimaPreviousArtifacts
[info] 	Set()
sbt:geo-scala>
[info] shutting down sbt server
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git diff
diff --git a/build.sbt b/build.sbt
index 59a5aa7..0432a9c 100644
--- a/build.sbt
+++ b/build.sbt
@@ -26,10 +26,7 @@ lazy val commonSettings = Seq(
     "org.scalatest" %%% "scalatest" % "3.2.13" % Test,
     "org.scalatestplus" %%% "scalacheck-1-16" % "<IP_ADDRESS>" % Test,
     "org.scalacheck" %%% "scalacheck" % "1.16.0" % Test
-  ),
-  // disable MiMa until we have proper version released on gnieh / all platforms
-  // once removed, the tlBaseVersion has to adjusted as well
-  mimaPreviousArtifacts := Set.empty
+  )
 )

 lazy val core = crossProject(JVMPlatform, JSPlatform, NativePlatform)

@plokhotnyuk what is the output of git tag on your system?

@armanbilge Thanks for the tip! There were no tags in the local clone of my fork until I fetched them from the upstream:
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git tag
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git remote add upstream https://github.com/gnieh/geo-scala
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git remote -v
origin	git@github.com:plokhotnyuk/geo-scala.git (fetch)
origin	git@github.com:plokhotnyuk/geo-scala.git (push)
upstream	https://github.com/gnieh/geo-scala (fetch)
upstream	https://github.com/gnieh/geo-scala (push)
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git fetch upstream
remote: Enumerating objects: 51, done.
remote: Counting objects: 100% (51/51), done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 36 (delta 9), reused 35 (delta 9), pack-reused 0
Unpacking objects: 100% (36/36), 5.09 KiB | 473.00 KiB/s, done.
From https://github.com/gnieh/geo-scala
 * [new branch]      feature/refresh-versions             -> upstream/feature/refresh-versions
 * [new branch]      fix-polyline-on-small-negative-delta -> upstream/fix-polyline-on-small-negative-delta
 * [new branch]      geohash                              -> upstream/geohash
 * [new branch]      main                                 -> upstream/main
 * [new branch]      switch-to-sbt-typelevel              -> upstream/switch-to-sbt-typelevel
 * [new branch]      upgrade-to-2.13                      -> upstream/upgrade-to-2.13
 * [new tag]         v0.1.0                               -> v0.1.0
 * [new tag]         v0.1.0-2023c94f                      -> v0.1.0-2023c94f
 * [new tag]         v0.1.0-907d7c6d                      -> v0.1.0-907d7c6d
 * [new tag]         v0.1.0-fb77b3a3                      -> v0.1.0-fb77b3a3
 * [new tag]         v0.1.1                               -> v0.1.1
 * [new tag]         v0.1.2                               -> v0.1.2
 * [new tag]         v0.2.0                               -> v0.2.0
 * [new tag]         v0.3.0                               -> v0.3.0
andriy@notebook:~/Projects/com/github/plokhotnyuk/geo-scala$ git tag
v0.1.0
v0.1.0-2023c94f
v0.1.0-907d7c6d
v0.1.0-fb77b3a3
v0.1.1
v0.1.2
v0.2.0
v0.3.0

I bumped the base version due to changes in the circe module that replaced io.circe.Encoder#AsObject by io.circe.Encoder in some pubic methods which were required for passing added serialization assertions in tests.

https://github.com/scala/scala/pull/9982 produces some warnings while compiling that are then turned into errors 😢
I downgraded to Scala 2.13.8 to avoid this. I didn't see anything in the release notes of 2.13.9 we would profit from…

@plokhotnyuk thank you for contributing this and your patience with the CI 👍. Would be interested to read that blog article, maybe you throw a link here once ready?
And also thank you 👍, @armanbilge!
I'll release as 0.4.0.


How do I develop a debug ID ？？？

Why is the development and debugging mode so slow? How did you develop and debug?
The command I executed: npm run start
release branch

Found under the develop branch can not be developed and debugged? How to do?

For help. . .

@clearJSer Build time is something we recently worked to reduce. The default branch, develop, compiles in under 20 seconds on my laptop. The release branch takes considerably longer.

See also https://github.com/openstreetmap/iD/issues/7505#issuecomment-612126213.


Staging environment jubilee

Is your feature request related to a problem? Please describe.
The staging environment was almost completely down for a couple of days last week due to work on #867, but it didn't seem to cause any major issues. So do we really need all those old secondaries? Or can we reset back to a clean staging environment and start over?
Describe the solution you'd like
Delete all existing staging secondaries, start over.
Describe alternatives you've considered
Identify any special secondaries that need to be kept.

Medium priority after arch call discussion

@cpswan @athandle - Removed all secondaries and data for the staging server.

thanks @kumarnarendra701 I've been watching the numbers come down.


Add acl-private param when uploading files to s3

In order for the backup files not to be publicly available (since they
may containe sensitive data), this commit adds the extra param
--acl-private that ensures that only the user can access the file
being uploaded

thx for this PR. I need to spare some time to look at it and then merge it


Unable to submit using eval ai cli

🐛 Bug

I would like to submit online according to this markdown.
I executed the above command, but I get the following error message and cannot submit.
$ evalai push ovmm_baseline_submission:latest --phase minival
Error: Challenge phase with slug minival does not exist

I looked through the github of evalai cli, but no such error stood in the issue. Also, the results of running evalai challenges were as follows.
$ evalai challenges
404 Client Error: Not Found for url: https://eval.ai/api/challenges/challenge/all

If you know how to solve this problem, please let me know.
Steps to Reproduce
Steps to reproduce the behavior: Run online submission section as is.

Expected behavior

Additional context

Could you make sure you followed these instructions: https://eval.ai/web/challenges/challenge-page/2100/submission

The phase names are a bit unintuitive, and may not be appearing on the EvalAI side.
Check out this PR I just created; it includes examples of the phase commands: https://github.com/facebookresearch/home-robot/pull/326
Hopefully this resolves your issue

Specifically you want to run this:
# For minival submissions
# With a public submission, anyone can see your score
evalai push <image>:<tag> --phase neurips-ovmm-minival-2023-2100
# You can also make a private submission - no one but you and the organizers can see the results
evalai push <image>:<tag> --phase neurips-ovmm-minival-2023-2100 --private

Thank you for your response. I got to submit it.


I haven't got the foggiest

@jimisola Does this means that it worked? I have absolutely no idea what is going on.

Let's try


Enhance ConfigBuilder

[ ] Strongly-typed references to the tables to Include / Exclude ( see #2 )
[ ] Inject validation or other logic OnBegin and OnAfter
[ ] Map to a ViewModel?

Fritz,
A suggestion ... don't call them tables. Especially when it is to be mapped on Entity Framework Core.  Call them "entities".

Fritz, A suggestion ... don't call them tables. Especially when it is to be mapped on Entity Framework Core. Call them "entities".

Yep... For the JSON API feature, we called them Entities.  For EF, I think we can revisit that naming

Moved last items on this checklist to their own issues:
#52 and #53

The config should allow for soft-delete entities.


Correct use of "is typing"

Enquiry NOT issues.
What event to I listen for when sending the is typing event to the clients connected, by using:
bot.sendIsTypingMessageTo(update.sender.id);
If have tried all of these with no success:
socket.on('typing', function(raw){
console.log("typing");
});
socket.on('is typing', function(raw){
console.log("is typing");
});
socket.on('istyping', function(raw){
console.log("istyping");
});
Does the flag that is typing is 'off' get sent by it self, and is it also an event? What event?
Regards,
Willem

Hi @willemevenwel,
Support for isTyping is not there for socket.io bots. This can be seen in the code here:
https://github.com/botmasterai/botmaster-socket.io/blob/c3c32d0bce2410c1df483c2c528af9ac57e9ce6b/lib/socket.io_bot.js#L54
I.e. no senderAction are enabled.
Happy to review any pull request adding this capability to the library however.  It's a small file, so shouldn't be too tricky to do. An update to the __sendMessage function in the library should do. Simply looking for the format of a typing indicator message and sending a different (recognizable) kind of event whenever it is detected should do.

Thanks for the feedback, I close this issue.


Support external process credentials in AWS inventory plugin config

Use Case
We would like to to have the ability to use an external credentials process to authenticate with AWS.
The specific use case we encountered was the that of an initially unprivileged user
assuming a privileged role that requires MFA authentication.  By using the AWS CLI's aws configure export-credentials feature, it is possible to delegate the MFA authentication to the CLI, and then reuse the CLI's cached session token.
This feature would mostly be helpful for interactive CLI users of Bolt, but the general purpose nature of the external mechanism might apply to other cases as well.
Describe the Solution You Would Like
A new parameter causes the inventory plugin to run an external process credentials command and use the resulting session token for accessing the AWS API.
Describe Alternatives You've Considered
Attempts to use a static credentials file with the credential_process setting resulted in uninitialized class variable errors. Attempts to generate a static credentials file containing a session token obtained from aws configure export-credentials  also failed.
After these failures it seemed worth seeing if the desired behavior of using aws configure export-credentials (particularly the CLI session cache) could be added directly to the inventory plugin.  Adding an alternative credential_process parameter that uses the underlying AWS SDK function proved to work in our environment. The result is submitted in PR #23.

Nice! This seems great. Will try to get your PR reviewed soon. Thanks so much

Thank you! Please let me know if there is anything I can do to assist with the review process.

I can report that since I submitted this PR, we have successfully used the inventory plugin with AWS SSO based CLI sessions. Again, please let me know if there is anything I can do to assist with the review.


Validation W3C

Expected Behavior
Running validator https://validator.w3.org on basic <picture> elements as per setup config should not result in errors.
Current Behavior
Currently when running validator on:
<picture class="lozad" style="display: block; min-height: 1rem" data-iesrc="images/thumbs/04.jpg" data-alt="">
    <source srcset="images/thumbs/04.jpg" media="(min-width: 1280px)">
    <source srcset="images/thumbs/05.jpg" media="(min-width: 980px)">
    <source srcset="images/thumbs/06.jpg" media="(min-width: 320px)">

</picture>

Validator will give error:
Element img is missing required attribute src.
Possible Solution
Setting up <nosript> tag with  results in
Element noscript not allowed as child of element picture in this context.
When adding  tag without  surrounding it, html parser will simply output 2 images...
Steps to Reproduce (for bugs)
Run validator on any website with lozad setup
Context
Trying to get a clean sheet from validator.

This should be in the docs imo as people will not realise it's w3c valid (obviously should be checking anyway), but:
lozad.js will overwrite an existing src attribute, so, you can preset it with an inline data uri. As an example:

Would be helpful if a library of various inline data images like this existed.

Turns out you can get w3c errors still if you use the data-srcset attribute and an accompanying sizes attribute... sigh

The sizes attribute may be specified only if the srcset attribute is also present.

This is something else that #144 would fix I believe

Hah, just fixed with:
const observer = lozad('.lozad', {
          loaded: (el) => {
            el.setAttribute('sizes', el.dataset.sizes);
            el.removeAttribute('data-sizes');
          },
        });

Should be built in though, if you can do the same with data-srcset

@joepagan This probably fixes the issue with W3C validator. But in a quick test, only Chrome and Firefox downloaded the correct sizes for images. Other browsers like IE and Safari were ignoring the sizes attribute when it’s updated in the loaded callback. I assume that the sizes attributes would have to be updated before the srcset attribute is updated.

Well, what I suggested is just a temp work around, I think lozad should deal with this really.
Maybe a more suitable fix is to leave the sizes attribute on the element

Any news on this? it has been a while and there is still no fix

hey @ApoorvSaxena please assign this issue to me

+1


Source control build.sh output

Hi there,
It'd be really convenient to have the artifact built by build.sh source controlled. Otherwise, I have to build myself after cloning. There were also some weird permissions issues on build.sh. Had to +x in order to run.
Perhaps there's a git hook you can use to run the build task prior to merging to master?

This makes sense since there's no npm-based flow to use the script, will do.

Thanks!

#4 fixed a bunch of issues related to broken env after cloning and there's no need to build when using jscodeshift


webwhatsapi.wapi_js_wrapper.JsException: Error in function sendMessage

Getting below error while trying to send message . Currently running on container with firefox . Appreciate your help to fix the issue.
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/webwhatsapi/wapi_js_wrapper.py", line 135, in call
return self.driver.execute_async_script(command)
File "/usr/local/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 657, in execute_async_script
return self.execute(command, {
File "/usr/local/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 321, in execute
self.error_handler.check_response(response)
File "/usr/local/lib/python3.8/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.JavascriptException: Message: TypeError: chat.sendMessage(...) is undefined
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 10, in
File "/usr/local/lib/python3.8/site-packages/webwhatsapi/objects/whatsapp_object.py", line 18, in wrapped
return func(self, *args)
File "/usr/local/lib/python3.8/site-packages/webwhatsapi/objects/chat.py", line 38, in send_message
return self.driver.chat_send_message(self.id, message)
File "/usr/local/lib/python3.8/site-packages/webwhatsapi/init.py", line 633, in chat_send_message
result = self.wapi_functions.sendMessage(chat_id, message)
File "/usr/local/lib/python3.8/site-packages/webwhatsapi/wapi_js_wrapper.py", line 143, in call
raise JsException(
webwhatsapi.wapi_js_wrapper.JsException: Error in function sendMessage (TypeError: chat.sendMessage(...) is undefined). Command: return WAPI.sendMessage('@c.us','Hhh', arguments[0])

I changed method "chat_send_message" (init.py) to use sendMessage2 instead of sendMessage.  Worked without errors.


Hung debug with larger input and head

Repro is with Zed commit e7623c5.
After seeing the ztest with head as part of #5196, I happened to try this variation with a larger input sequence and found it hangs.
$ seq 101 | zq 'debug f"Ive got {this}" | yield this+1 | head 1' -
"Ive got 1"
"Ive got 2"
...
"Ive got 100"
"Ive got 101"
2
[never exits...]

That 101 seems to be a magic number here. If I drop it to just 100 it exits without complaint.

Verified in Zed commit 2e2c19b.
Now the program exits fine.
$ zq -version
Version: v1.17.0-33-g2e2c19bc

$ seq 101 | zq 'debug f"Ive got {this}" | yield this+1 | head 1' -
"Ive got 1"
"Ive got 2"
...
"Ive got 99"
"Ive got 100"
2

$

FWIW, the way the debug output stops after 100 lines is because debug works in batches of values which are of size 100 by default. Since the head 1 limits the program's main output to just one value, only the first batch of 100 values ever had to be pulled through the flowgraph, so debug only ever ended up producing output based on that first batch of 100 values.
Thanks @mattnibs!


support reading SMILES that start with a branch

Is your feature request related to a problem? Please describe.
rdkit does not want to read the following SMILES:
(c2cc(N3CCOC[C@@H]3C)nc(N4CC5CCC(C4)O5)n2)c0cnc1ccccc1c0
Many program read it correctly (e.g. Chemaxon's mview).
Describe the solution you'd like
Understand this SMILES as the branch being attached to the first
atom right after the branch.
Describe alternatives you've considered
Rewriting by hand the SMILES, but that's just a workaround.
Additional context
Add any other context or screenshots about the feature request here.
I agree such a SMILES is not standard, but even opensmiles.org does not specify
that a SMILES must not start with a branch.

I'm ambivalent about this one since I think it's really confusing.
Still, you're right that neither the Daylight SMILES page nor the OpenSMILES documentation clearly say that this form isn't allowed.

I think it's a bad smiles and will stand behind that.
OpenSmiles looks like it enforces an atom before the branch in the definition:
branched_atom ::= atom ringbond* branch*

I'm more concerned about the aromatic hydrogen
[C@@h]

out of curiosity, what package is generating these smiles?

@bp-kelley : the aromatic SMILES is just because @UnixJunkie didn't quote the SMILES string.
If you forget that then [C@@H] renders as [C@@H]

@bp-kelley just a prototype software under development. However, the fact that chemaxon's mview can read them
hints that some more software in the wilde can produce such SMILES.
I could modify my software to not produce those.
Maybe rdkit could accept those if the right boolean is turned on in the Chem.MolFromSmiles function.
I also think that rdkit could explicitely say that it does not support reading in such SMILES and throw a very explicit error about this.
@bp-kelley is right that the opensmiles specification does not say "a SMILES string cannot start with a branch" however he read their grammar more carefuly than me, and the grammar indeed says "a SMILES string cannot start with a branch".

I’m not convinced.  OpenEye can’t read them either and have multiple employees that worked at Daylight, but I’ll leave this one up to Greg.

maybe allowing leading branch components could require some explicit convention about the implied ordering of substituents around a chiral center, also in case multiple leading branches were allowed? or would the base rules still suffice? if the atoms in the smiles are ordered according to a traversal of the represented molecular graph, a leading branch wouldn't be possible (right?), so it may be not obvious if (C)C@HI would be equivalent to CC@HI or to C@H(F)I, I think the latter feels more consistent with the general rules, although the former looked nicer at first. If multiple leading branches are present then it may be not clear to me if they should be recursively repositioned after the first atom, or as a block, to preserve their order. (sorry in case the comment doesn't make sense, I would have bet the limitation was explicitly stated somewhere on the daylight site)

From the Daylight documentation [1]: "In all cases, the implicit connection to a parenthesized expression (a "branch") is to the left."
This then suggests to me that you can't start a SMILES with a branch, since there is no atom to the left.
Also, a slightly different but related consideration is that if you enclose a SMILES in parentheses, it becomes SMARTS syntax [2], not sure if that would cause other issues.
[1] https://www.daylight.com/dayhtml/doc/theory/theory.smiles.html
[2] https://www.daylight.com/dayhtml/doc/theory/theory.smarts.html

Ok, the consensus seems to be that we should not support this case.
Thanks for the feedback from so many experts.


Strange initialState not object error.

I was trying the sample:
import {combineReducers, createStore, applyMiddleware, compose} from 'redux';
import {localeReducer, cssLazyLoader} from "../../../dist";

const composeEnhancers = window.__REDUX_DEVTOOLS_EXTENSION_COMPOSE__ || compose;

export const store = createStore(combineReducers({
	locale: localeReducer("en", require("../../locales/index").default)
}), composeEnhancers(
	applyMiddleware(
		cssLazyLoader(["LOCALE_CHANGED"], {
			"en": {address: "en.css", direction: "ltr"},
			"fa": {address: "fa.css", direction: "rtl"}
		}),
	)
));

But got this error, any suggestions?
initialState has unexpected type of "Function".

Visit https://fb.me/react-error-boundaries to learn more about error boundaries.
__stack_frame_overlay_proxy_console__ @ index.js:2178
logCapturedError @ react-dom.development.js:9747
captureError @ react-dom.development.js:10540
renderRoot @ react-dom.development.js:10391
performWorkOnRoot @ react-dom.development.js:11014
performWork @ react-dom.development.js:10967
requestWork @ react-dom.development.js:10878
scheduleWorkImpl @ react-dom.development.js:10732
scheduleWork @ react-dom.development.js:10689
scheduleTopLevelUpdate @ react-dom.development.js:11193
updateContainer @ react-dom.development.js:11231
(anonymous) @ react-dom.development.js:15226
unbatchedUpdates @ react-dom.development.js:11102
renderSubtreeIntoContainer @ react-dom.development.js:15225
render @ react-dom.development.js:15290
./src/index.jsx @ index.jsx:10
__webpack_require__ @ bootstrap 85112c131f1fc323db88:678
fn @ bootstrap 85112c131f1fc323db88:88
0 @ app.scss:26
__webpack_require__ @ bootstrap 85112c131f1fc323db88:678
(anonymous) @ bootstrap 85112c131f1fc323db88:724
(anonymous) @ bootstrap 85112c131f1fc323db88:724
translatable.js:130 Uncaught TypeError: _this.store.getState is not a function
    at new Translated (translatable.js:130)
    at constructClassInstance (react-dom.development.js:6355)
    at updateClassComponent (react-dom.development.js:7839)
    at beginWork (react-dom.development.js:8225)
    at performUnitOfWork (react-dom.development.js:10224)
    at workLoop (react-dom.development.js:10288)
    at HTMLUnknownElement.callCallback (react-dom.development.js:542)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:581)
    at invokeGuardedCallback (react-dom.development.js:438)
    at renderRoot (react-dom.development.js:10366)
    at performWorkOnRoot (react-dom.development.js:11014)
    at performWork (react-dom.development.js:10967)
    at requestWork (react-dom.development.js:10878)
    at scheduleWorkImpl (react-dom.development.js:10732)
    at scheduleWork (react-dom.development.js:10689)
    at scheduleTopLevelUpdate (react-dom.development.js:11193)
    at Object.updateContainer (react-dom.development.js:11231)
    at react-dom.development.js:15226
    at Object.unbatchedUpdates (react-dom.development.js:11102)
    at renderSubtreeIntoContainer (react-dom.development.js:15225)
    at render (react-dom.development.js:15290)
    at Object../src/index.jsx (index.jsx:10)
    at __webpack_require__ (bootstrap 85112c131f1fc323db88:678)
    at fn (bootstrap 85112c131f1fc323db88:88)
    at Object.0 (app.scss:26)
    at __webpack_require__ (bootstrap 85112c131f1fc323db88:678)
    at bootstrap 85112c131f1fc323db88:724
    at bootstrap 85112c131f1fc323db88:724

if you changed store into LocalStore then you should change example/app/app.jsx and change store into LocalStore too. while I cant reproduce the first error you have got but place these lines of code in this file example/app/redux/store.js
import {combineReducers, createStore, applyMiddleware, compose} from 'redux';
import {localeReducer, cssLazyLoader} from "../../../dist";

export const store = createStore(combineReducers({
	locale: localeReducer("en", require("../../locales/index").default)
}), compose(
	applyMiddleware(
		cssLazyLoader(["LOCALE_CHANGED"], {
			"en": {address: "en.css", direction: "ltr"},
			"fa": {address: "fa.css", direction: "rtl"}
		}),
	)
));

do you have any other changes ?

I changed into this now:
export const LocaleStore = createStore(combineReducers({
    locale: localeReducer("en_us", locales),
}), applyMiddleware(
    cssLazyLoader(["LOCALE_CHANGED"], {
        //"en_us": {address: "en_us.css", direction: "ltr"},
        //"en_uk": {address: "en_uk.css", direction: "ltr"},
    }),
));

Got the same errors as:
  127 |
  128 | (0, _invariant2.default)(_this.store, "Could not find \"store\" in either the context or " + ("props of \"" + connectDisplayName + "\". ") + "Either wrap the root component in a <Provider>, " + ("or explicitly pass \"store\" as a prop to \"" + connectDisplayName + "\"."));
  129 |
> 130 | var storeState = _this.store.getState();
  131 | _this.state = { storeState: storeState };
  132 | _this.clearCache();
  133 | return _this;

as you change store variable into LocalStore you should change example/app/app.jsx file into this
import {LocaleStore} from "./redux/store";
import {Provider} from "react-redux";
import React from 'react';
import DashboardContainer from "./containers/dashboard/DashboardContainer";

export default class App extends React.Component {
	render() {
		return (
			<Provider store={LocaleStore}>
				<DashboardContainer/>
			</Provider>
		)
	}
}

Ah, okay, thanks, I accidentally wrote this:
<Provider className="app" store="LocaleStore">

But still found this error:
initialState has unexpected type of "Function".

The locales is this import locales from '../locales';, basically the same as require.default. This works for me perfectly:
export const LocaleStore = createStore(combineReducers({
    locale: localeReducer("en_us", locales),
})/*, compose(applyMiddleware(
    cssLazyLoader(["LOCALE_CHANGED"], {
        //"en_us": {address: "en_us.css", direction: "ltr"},
        //"en_uk": {address: "en_uk.css", direction: "ltr"},
    }),
))*/);


Installation of 2 HomeKit Locks

Good morning, I would like to install 2 Homekey readers, one at the front door and one at the apartment door to open each individual door. At the apartment door with Nuki and Home Assistant, the automations work perfectly. If I now install the second one, I have the problem that the "reader" (ESP32 and PN532) in HA are the same MQTT device, the same TAGS (device_id) and there is only one Homekey card in Wallet. I can't manage to separate the automations in HA and only control one device, even with scanned_tags or something like that. I changed the name in config.h to #define MQTT_CLIENTID & #define Name. But both always respond. I also changed the serial number and device ID etc. in the depths of your project. In HA in MQTT they are now also displayed as 2 devices. But the device IDs for the TAGs are the same.
Is it because both devices use the same Homkey card in Wallet? Or what can I do to ensure that both readers are recognized as completely different devices and work as such?
I hope someone can help me, I'm desperate :-)
Best regards,
Stefan

Hello!
In order to deploy multiple devices flashed with this, you only need to do two things

Change MQTT_CLIENTID to something unique or the device won't be able to join MQTT
Modify the topics in the "MQTT Topics" section to something else as that's how you end up with this mangling, it's enough to change the first part, e.g. "topic/custom_state" -> "another_topic/custom_state"

The name is only used for display purposes and the serial number it's derived from the mac address of the device so it is already unique.
Apple designed the HomeKey to be one per Home in order to simplify things as it doesn't make any sense to have multiple "digital keys" for each lock within the same house and makes it easier to identify devices.
Try the steps and let me know how it went, if it doesn't work, you might need to cleanup your MQTT setup.

Wow, you're my hero. It works perfectly. Unfortunately, I'm not that good at MQTT yet.
Thank you for the quick help and for making this possible.

Great, happy to help!


Transaction Error

Transaction 4769ED7DEDBE6A9340494F42BCEB410CC3D2563CB247FC4068E503E6C1DB0D9A is an outgoing tx but it is showing incoming (Green arrow)

PR Reference: https://github.com/proximax-foundry/web-explorer-vuejs/pull/147


fix(tooltip): allow string literal templates

Can now pass a string literal with the templateUrl to tooltip-template and popover-template,
rather than just an expression / scope variable. This is a potential solution to my issue here: https://github.com/angular-ui/bootstrap/issues/3852

This is my first submission to the project.  I made sure to add a test that exposed the problem, and then verified all tests passed after making my changes.  I am totally open to feedback on my solution.  I wasn't sure if adding the $templateCache dependency to the directive was the best way to go about this, but I couldn't think any other good way to make sure the tooltip / popover wouldn't come up "blank" (because of an invalid template).

If you wrap the template string in single quotes it works.  I recall this being the suggested approach when working with string literals for other directive attributes.  We should check with Chris though and see what his intent was.

@RobJacobs Thanks again for sharing that workaround - that makes a lot of sense.
I guess I just found it confusing that this works:
$scope.myUrl = 'myTemplate.html';
...

Selector Text

But this doesn't:

Selector Text

Which is, intuitively, the same thing.

Which is, intuitively, the same thing.

Nope, it is not. Expressions are evaluated against variables on the scope, so doing myTemplate.html would imply: find the myTemplate variable on a scope.
Anyway, I don't think this PR is valid, as it would introduce confusing API. There are already directives in AngularJS (ex.: ngInclude) that behave exactly like this one so I don't think we should deviate from the well-established practices in the community.
This would be -1 from me when it comes to merging this PR.

"Nope, it is not." - Perhaps I should have said that it was intuitive to me, rather than generalizing.  Sorry about that, @pkozlowski-opensource.
I can definitely see how this change would cause confusion, especially after you pointing out the ngInclude example.  I completely understand why this change wouldn't be merged.
At the very least, I can at least now better understand why my code was not working =)  Thanks for taking the time to look into this.

I too used ng-include for reference before responding, since for developer intuitive use we should match how Angular handles situations & agree with the others on this.
Closing as per discussion.


lang0: implement foreign procedure support

Summary

add support for foreign procedures to L0 through L30 ILs. A
foreign procedure is one whose implementation is provided separately
implement VM modules, which are used/needed for a post-code-
generation link step

Details
Foreign procedure are meant to be used for providing functionality or
host interaction (in case of the VM) that cannot be implemented
in-language.
They're defined via a Foreign definition in the procedure list of a
module in the L0 through L30 ILs. A Foreign definition specifies
a signature type and a name, the latter which identifies the foreign
procedure to the code generator / linker / etc. . Foreign procedures
are used / called like any other procedure.
For the VM backend, a foreign procedure is implemented via a VM
callback.
In order to handle the name to callback resolution, and as a
preparation for future multi-module support, the to-VM build process is
extend with a "link" step, now working as follows:

pass0 produces a VmModule for the input IL module (instead of
appending the code directly to the VmEnv)
all VM modules part of the build are linked via vmmodules.link
all callbacks (provided by a name -> callback map) referenced by VM
modules are added to the environment during linking

Since there's no multi-module support at the moment, there's no import/
export support yet. The implementation of link still supports more
than one module already, with in-code references to types, globals,
etc. being patched as needed.
The host_impl module provides the VM callbacks implementing the Phy
host API, which only consists of a single procedure used for testing
foreign procedure support.
Finally, the tree_parser is extended to support standalone string
literals (i.e., strings can appear in S-expression without an
accompanying node kind, but only when supported by the IR).
Two VM bugs are fixed:

ignore stub and callback procedures when validating bodies (they
previously resulted in false-positive errors)
handle stub and callback procedures when disassembling, instead of
crashing

Notes For Reviewers

foreign procedures are an important prerequisite for further work, such as #70 and I/O support

An alternative to the current Foreign procedure mechanism in the L0 I considered was adding an FFICall expression, which behaves much like a Call expression, but with the Proc operand replaced by a type and name.
This would have effectively allowed for ad-hoc imports of foreign procedures, requiring pass0 to extract foreign procedure definitions from these calls. The extra pass0 work, as well as the approach not fitting with the general direction is why I ultimately decided against it.
As for the general direction, my plan is to remove the Foreign definition again once module support (i.e., imports and exports) is implemented in the ILs. Instead, all FFI bits should be provided via special FFI modules, which are target-language-specific modules that are imported like any other module.
From the view of module-local code, a foreign procedure is then no different from any other imported procedure. The benefit of this approach is that most ILs/IRs can stay target-language-independent, with the target-language-dependent parts outsourced to the FFI modules.


Reaper - not loading

Reaper Scans source does not load, the Safari Icon on the top opens the website, and it requires a cloudflare check every time, not sure if this is the reason

Can't replicate

Happened to mine as well. No logs were made when it failed to load favorited series from Asura as a source.
My fix was to uninstall Asura Scans as a source, re-add the source. Then migrate it into itself as the old series were broken.
Broken as in new chapters were marked as unavailable (on the resume reading button). When migrating most did not have series art and authors were “unknown”. The migration generally found all the chapters and fixed it.

It's showing this empty browse for me as well.

Not loading for me as well. Safari doesn't do Cloudflare checks for me tho.


Fix parsing of the model name tag in ollama embeddings provider

Fixes #1188
The model name tag was not being parsed correctly in the ollama embeddings provider. This patch fixes the parsing of the model name tag in the ollama embeddings provider.

Thank you so much for finding and fixing this @minamijoyo!

@mldangelo Thank you for your support. I fixed the lint issue. Could you rerun CI?

Thank you for writing tests! I'll merge and cut a release!

@mldangelo Thanks for your quick response and review!

Of course! Please let us know if you notice any other bugs / need anything else.


Import / Export SIA

@pete-mc Could there also be a chance here to add an import and export feature to SIAs?, Exporting the SIA from terrain as a whole and importing it for another youth member? Thanks for all your great work on this.
Originally posted by @MrNO13 in https://github.com/pete-mc/Summit/issues/31#issuecomment-1912859657

Merging with similar issue #31 to be addressed at the same time.


Show account open orders

Added command to show all open orders for account

Hey @ionutgalita! Thank you for your contribution. :+1:
Regarding the sandbox mode, I think the paper trading implementation that is done already does a better job at it.
The sandbox mode will simulate orders, but it won't give you a response with an order id, nor tell you if the order could have been filled.
Take a look at this issue from node-binance-api
And with the paper trading, you can activate/deactivate it for each strategy.
For the open orders, please do the following changes:

Change getAccountOpenOrders to accountOpenOrders to be consistent with the other functions;
In getAccountOpenOrders, parseFloat(response[i]['origQty']).toFixed(2)__ might cut the quantity important to see for some cases, for example, 0.004 of ETH/BTC will show 0.00. You probably can leave it as response[i]['origQty'];
In getAccountOpenOrders, you're iterating through all orders and you've created the var orders, but then you don't send it in next, you just send the whole response.
In getAccountOpenOrders, instead of using response[i]['symbol'] in the message, use vars.pairs[response[i]['symbol']].chatName(), this way it will auto create the tag in Telegram for the token.
In getAccountOpenOrders, don't use the :barber: emoji for all orders, following the example for the list command, use the emoji at the beginning like:
:book: Orders
...

Thank you a lot for your contribution! Probably for future updates, we can identify if the order belongs to a certain strategy and organize the message per strategy. Not sure though.
Anyway, I always went for the Binance app when I wanted to see the orders, so this will help me save time!

Hi @andresilvasantos
First thank you for the sandbox tip. Exactly now I was trying to understand where is the order id :)
Second, i amended the code with your suggestions.
Let me know if everything is ok.
Thank you

Nice work!


What API should I use to get information of Mod using ProjectID ?

I was wondering how were you able to get the mod description by using the slug or projectID? I can't find these in the curseforge documentation.

Hey!
I use this endpoint to get the mod description: https://docs.curseforge.com/#get-mod-description (https://api.curseforge.com/v1/mods/{modId}/description)
And this one for mod information in general: https://docs.curseforge.com/#get-mod (https://api.curseforge.com/v1/mods/{modId})
So for instance, my own modpack would have this:

https://api.curseforge.com/v1/mods/688687
https://api.curseforge.com/v1/mods/688687/description

@itssimple In order to get the modId I first need to find it right?
For example using only this link https://www.curseforge.com/minecraft/mc-mods/clay-overhaul
How were you able to retrieve the modID?
Because I think that projectID does not work with /v1/mods/{modId} API

The project id found on the mod page, is the modId.
But as for searching with slugs, I have a very manual approach, where I split the URL into segments, and get game, class and slug.
https://github.com/CurseForgeCommunity/CFLookup/blob/master/CFLookup/Pages/Index.cshtml.cs#L164-L172
And then I have code that fetches the game info (all of it) and all categories for all games, and then I use that to search for mods with a matching slug.
https://github.com/CurseForgeCommunity/CFLookup/blob/master/CFLookup/SharedMethods.cs#L199-L236

Thank you for the clarification. That was the confusion all along. modID = projectID


feat(i18n): 14293 - panel config created on the go

https://kontur.fibery.io/Tasks/Task/The-translated-names-of-the-panels-are-switched-back-to-the-English-version-after-page-refreshment-14293
The problem is next (some panel titles aren't translated):

The reason is that all PanelFeatureInterface objects created on app start, but language comes async later from user profile request, that's why lang was outdated.
I've decided to create these objects on react render, as i18n expected scenario.

Codecov Report

Merging #401 (7288b29) into main (08c2947) will not change coverage.
The diff coverage is 0.00%.


Sync published with master

Deploy preview for docsdocker ready!
Built with commit 336dcd66f2c717221669ad71568b612a19247f95
https://deploy-preview-8142--docsdocker.netlify.com


translate blog (date 2022/05/30 )

Does this blog need to translate? Assign to me if needed.
https://kubevela.io/zh/blog/2022/05/30/abstraction-kubevela

/assign


Add missing ``` which was screwing up the rest of REGEX.md

Can one of the admins verify this patch?

@anusharanganathan shall we go ahead and merge this?
@justinccdev thanks for contributing this! Sorry it's taken so long to look at.

Thanks for writing this. We're super excited to get even the smallest contributions. I'm merging it now!


youku

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.02.01. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2017.02.01

Before submitting an issue make sure you have:

[ ] At least skimmed through README and most notably FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2017.02.01
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Carefully read new issue template and provide all requested information.


[QoL Feature Suggestion] Show MoM threshold alongside Life/Mana Stats

In the Stats Summary Box on the left, alongside the Life and Mana Paragraphs, if any % of damage is taken from Mana instead of Life (currently 30% with MoM alone and additional 10% with 3.0's Cloak of Defiance), Show the calculatedeffictive EHP values.

Example:
Total Life: 5509
%inc Life from Tree: 166%
Life Regen: 35.9
Effective Life with damage taken from Mana: 7161.7
Total Mana: 3580
%inc Mana from Tree: 124%
Unreserved Mana: 1861.6
Unreserved Mana: 52%
Mana Regen: 84.0
Damage taken from Mana before Life: 30%
Damage Threshold: 1652.7

If Damage Threshold is greater than Unreserved Mana, add Unreserved Mana to Life for EHP instead of Damage Threshold

This information is already displayed in the Calcs tab, in the Damage Taken section; you'll find the EHP calculations in the breakdown for Mind over Matter. I'd prefer to not add those to the sidebar, as there's already too many stats there.

Thanks!
I missed that.


Language-specific guides request

Problem description
PHP missing
Problem location

I couldn't find the information I wanted. I expected to find it near the following URL: https://docs.docker.com/language/

Suggestions for a fix
add a Language-specific getting started guide for php.
i was told to do this, by your tutorial, but for now and me, the others will work too

Thanks for requesting; let me close this as duplicate of https://github.com/docker/docker.github.io/issues/12719

[image: image.png]
i see nothing... from php
Am Do., 2. Dez. 2021 um 15:36 Uhr schrieb Sebastiaan van Stijn <
@.***>:

Closed #13533 https://github.com/docker/docker.github.io/issues/13533.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/issues/13533#event-5706277285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASEONAXNUGFT6JJTTBZCLLUO577PANCNFSM5D7YORXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Tommy Noël Paulik
Webdeveloper
Email:   @.***
Telefon: +4917647058256

It's not there yet, but I closed this ticket as a duplicate of https://github.com/docker/docker.github.io/issues/12719

Toll...
Sebastiaan van Stijn @.***> schrieb am Do., 2. Dez.
2021, 21:37:

It's not there yet, but I closed this ticket as a duplicate of #12719
https://github.com/docker/docker.github.io/issues/12719
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/issues/13533#issuecomment-984981911,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASEONDBC6COEXETDY6OUUDUO7KHPANCNFSM5D7YORXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


fixed internation page

Fix #287
Test URLs:

Before: https://main--sling--aemsites.aem.live/international
After: https://287-international--sling--aemsites.aem.page/international

Hero area in mobile is much different
Check with customer why Desi Binge Plus package is only available on Mobile?
Polish circle is missing on your page


Add 'women' as a key to the gentation_to_number dict

I got a KeyError on a certain profile with this key.  I'm not sure if the mapping is correct.

Yeah, this kind of thing seems to always be in flux.


Avoid stripping additional characters from GIS SQL params.

If the param lead or ended with characters identified in the strip
calls, those characters would also be removed. Using split will
retain all of the characters in between the given strings.
When we drop support for < 3.9, we can switch to
the removeprefix and removesuffix functions.
Fixes #1543
This should add a test case that confirms the fix for 1543, but I'm unsure of what that would look like. If we think we absolutely need a test case for it, then perhaps we move it to panels/sql/utils.py so it can be in a unit test rather than an integration test.

Ah, this needs to not run when it's not a GIS sql param.

Now that I read the code again I'm really unsure. We're modifying the SQL which is being executed instead of only logging it and maybe fixing it later when trying to analyze/explain it.
Is there a good reason for modifying the params list before calling cursor.execute, cursor.callproc etc. at all?

I'm not sure. Even though the tests indicate we test for postgis, it's not used. I'm working through setting it up locally to see if the tests pass.

@matthiask I'm going to revert #1426. I don't think it ever covered the change and the test is broken.

Replaced by #1547


Full stack projects

Is it possible to create full stack project e.g. node + react?
All starters seem to be purely server side.

Hi @mwisnicki ! Sure, it's definitely possible. If you would open a server side application on port 8000, and start a react app on port 4000, they will be able to communicate. We aim to support everything you can do on local.
Just give it a try and let us know if you run into issues 🙏 !
Hope this answers your question.


Don't skip golden lure

https://github.com/UnownHash/Golbat/blob/391a5457a4b45dc04649e03f375c1a980d45816e/decoder/pokestop.go#L152
Golden lure is id 506

https://github.com/UnownHash/Golbat/commit/264e0eaec644345fc7add4552768a8e43e877ca7
Resolved


protobuf: generate code using a script

Currently protobuf bindings have to be generated manually, the process
is being documented as a code comment in api/v1alpha/api.proto.
This automates protobuf generation by introducing a script and
also a make target "protobuf".
Fixes #2004

@alban @jonboulle PTAL

doh: https://travis-ci.org/coreos/rkt/jobs/120903030#L336

ugh, the failed travis build is reproducible with a clean directory, investigating.

@alban travis seems to be happy now, I also catched up with #2377; PTAL

@alban @jonboulle PTAL

1,812 additions, 0 deletions not shown because the diff is too large. Please use a local Git client to view these changes.

seriously? smdh

@alban @jonboulle PTAL

LGTM

Wait, we have both vendoredApps and different build tags? Isn't that redundant?

er, never mind, I'm blind.

LGTM


MS PowerBI Rest error: Invoke-RestMethod : The 'Content-Type' header must be modified

Hi,
I have PowerBI setup with a gateway and have been able to refresh the datasource using the MS tools.
I have customized  the ps1 program with my company's settings and have installed the PowerShell tools.
When I run the manageRefresh.ps1 command, I get the errors:
Invoke-RestMethod : The 'Content-Type' header must be modified using the appropriate property or method.
Parameter name: name
At C:\temp\manageRefresh.ps1:78 char:1
+ Invoke-RestMethod -Uri $uri -Headers $authHeader -Method POST -Verbose
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Invoke-RestMethod : The 'Content-Type' header must be modified using the appropriate property or method.
Parameter name: name
At C:\temp\manageRefresh.ps1:82 char:1
+ Invoke-RestMethod -Uri $uri -Headers $authHeader -Method GET -Verbose
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Any ideas? Thanks in advance for looking. Nancy.

I am getting a similar error:
Invoke-RestMethod : A positional parameter cannot be found that accepts argument 'â€Headers
System.Collections.Hashtable â€Method'.
At \PowerBI_Powershell_Script\Updated_Powershell_Script.ps1:119 char:1

Invoke-RestMethod -Uri $uri â€“Headers $authHeader â€“Method POST â€“ ...

  + CategoryInfo          : InvalidArgument: (:) [Invoke-RestMethod], ParameterBindingException
  + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

@NancyAb
I was getting the same error and these updates made my code work:
$authHeader = @{ 	Authorization=$token.CreateAuthorizationHeader() 	Content='application/json' }
and I believe the sample code mixes long and short dashes which was giving me an error as well so here is the modified code for that:
Invoke-RestMethod -Uri $uri -Headers $authHeader -Method POST -Verbose

Thanks so  much. I will have a look. Nancy.

Hi,
I had the same issue and got an error with BrightKnight6 code too, but adding single quotes around Authorization and Content seemed to do the trick.  So the key difference between the original and this is Content instead of Content-Type.  I'm a newbie to all the concepts here so can't really explain why.
This is what I ended up with:
$authHeader = @{
'Authorization'=$token.CreateAuthorizationHeader()
'Content'='application/json'
}
Scott

I consistently receive the error below and my dataset does not refresh, any help would be appreciated.
"
Invoke-RestMethod : The remote server returned an error: (400) Bad Request.
At line:85 char:1

Invoke-RestMethod -Uri $uri –Headers $authHeader –Method POST –Verbose

  + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-RestMethod], WebException
  + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

VERBOSE: GET https://api.powerbi.com/v1.0/myorg/datasets/17c40dff-c522-484b-ac34-abec2005c002/refreshes with 0-byte payload
VERBOSE: received -1-byte response of content type application/json; odata.metadata=minimal; odata.streaming=true
"

HI All,
I am also getting :
PS C:\temp> .\sample.ps1
[Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationResult
System.Collections.Hashtable
Invoke-RestMethod : A positional parameter cannot be found that accepts argument 'â?Headers
System.Collections.Hashtable â?Method'.
At C:\temp\sample.ps1:78 char:1

Invoke-RestMethod -Uri $uri â?"Headers $authHeader â?"Method POST â?"Verbose

  + CategoryInfo          : InvalidArgument: (:) [Invoke-RestMethod], ParameterBindingException
  + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.InvokeRestMethodCommand](url)

Please help

I'm also getting similar error:
Invoke-RestMethod : A positional parameter cannot be found that accepts argument 'â€Headers System.Collections.Hashtable â€Method'.
At C:\Users\toneyc\Downloads\copyWorkspace.ps1:76 char:16

... l_groups = (Invoke-RestMethod -Uri $uri â€“Headers $auth_header â€“Me ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Invoke-RestMethod], ParameterBindingException
FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

HI All,
I am also getting :
PS C:\temp> .\sample.ps1
[Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationResult
System.Collections.Hashtable
Invoke-RestMethod : A positional parameter cannot be found that accepts argument 'â?Headers
System.Collections.Hashtable â?Method'.
At C:\temp\sample.ps1:78 char:1

Invoke-RestMethod -Uri $uri â?"Headers $authHeader â?"Method POST â?"Verbose

  + CategoryInfo          : InvalidArgument: (:) [Invoke-RestMethod], ParameterBindingException
  + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.InvokeRestMethodCommand](url)

I tried all changes mentioned above but nothing worked. Please help

You need to save you code to file that is not UTC. More than likely you copied code from a webpage and it include special characters that need to be removed.
You can always delete the problem line and retype it manually.

+1
Tried all suggestions and nothing worked.
@nickodemis99 The file was download via git, not a web browser


Docker high CPU usage

I am continually starting/killing/removing a high volume of containers and running short-lived processes in them.  I am getting very high CPU usage from the docker process, with low usage everywhere else (often 65-80% of total CPU).
docker info
Containers: 22
Images: 25
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 1927
 Dirperm1 Supported: false
Execution Driver: native-0.2
Kernel Version: 3.13.0-49-generic
Operating System: Ubuntu precise (12.04.5 LTS)
CPUs: 2
Total Memory: 3.661 GiB
Name: <server name redacted>
ID: RMGD:WCQM:VYVD:UZAO:PRSI:BCOU:MYBO:GULG:3FQZ:YDXT:ZO3O:POBU
WARNING: No swap limit support

docker version
Client version: 1.6.0
Client API version: 1.18
Go version (client): go1.4.2
Git commit (client): 4749651
OS/Arch (client): linux/amd64
Server version: 1.6.0
Server API version: 1.18
Go version (server): go1.4.2
Git commit (server): 4749651
OS/Arch (server): linux/amd64

uname -a
Linux <server name redacted> 3.13.0-49-generic #81~precise1-Ubuntu SMP Wed Mar 25 16:32:15 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

ulimit
core file size          (blocks, -c) 0
data seg size           (kbytes, -d) unlimited
scheduling priority             (-e) 0
file size               (blocks, -f) unlimited
pending signals                 (-i) 29572
max locked memory       (kbytes, -l) 64
max memory size         (kbytes, -m) unlimited
open files                      (-n) 1024
pipe size            (512 bytes, -p) 8
POSIX message queues     (bytes, -q) 819200
real-time priority              (-r) 0
stack size              (kbytes, -s) 8192
cpu time               (seconds, -t) unlimited
max user processes              (-u) 29572
virtual memory          (kbytes, -v) unlimited
file locks                      (-x) unlimited

top (typical)
2021 root      20   0 2120m 283m 8484 S   75  7.6 196:49.01 docker
    1 root      20   0  116m  95m 1332 R   20  2.5  61:41.39 init
15042 kinvey    20   0  660m  36m 6364 S   20  1.0   0:00.60 node
14988 kinvey    20   0  660m  37m 6364 S   12  1.0   0:00.64 node
11635 kinvey    20   0 1123m 211m 7232 S    7  5.6   5:09.81 node
15027 kinvey    20   0  4440  648  552 S    2  0.0   0:00.05 sh
15018 root      20   0  177m 5632 4268 S    1  0.1   0:00.04 docker
15054 root      20   0  169m 5368 4040 S    1  0.1   0:00.04 docker
15066 root      20   0  121m 5444 4008 S    1  0.1   0:00.04 docker
    7 root      20   0     0    0    0 S    1  0.0   3:16.30 rcu_sched

So the high usage occurs while restarting containers or in between?

I'm constantly starting/killing containers.  But it's when starting and killing containers and the act of killing/starting the containers is taking a long time (several seconds).

I am also having fairly random high cpu usage on a node and apache containers, also 1.6 with aufs.
Running same containers on otther host also 1.6, but with brtfs, do not have high load... wonder if this is aufs issue.

Switching to OverlayFS solved the problem.

I am also getting this. We are using a machine for CI and I am just going to have to restart every day to get around this. No processes will be running but Docker will still be using a ton of CPU and memory.
I'll try to pin it down.
root@localhost:~# docker info
Containers: 11
 Running: 8
 Paused: 0
 Stopped: 3
Images: 145
Server Version: 1.10.1
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge host null
Kernel Version: 4.2.0-27-generic
Operating System: Ubuntu 14.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 3.859 GiB
Name: localhost
ID: 5FP4:5O4B:XMHV:OIFZ:5E44:IRKY:B3OG:AB34:RQVH:UXYU:ZJK4:DOIY
root@localhost:~# docker version
Client:
 Version:      1.10.1
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   9e83765
 Built:        Thu Feb 11 19:27:08 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.1
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   9e83765
 Built:        Thu Feb 11 19:27:08 2016
 OS/Arch:      linux/amd64

I am also getting this too,the only difference is my docker Storage Driver is device mapper. docker version is 1.10.3


Add PowerShell menu entry

This PR creates a PowerShell shortcut which can be used to manage conda environments.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #4

This uses the base powershell available on Windows 10+ not the newer PowerShell Core (pwsh.exe). Note that my testing indicates that mamba/conda work equally well with both pwsh and powershell so if there is interest in accepting another PR, I can add a third entry to the list with the same arguments.
@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Please let me know if there are any issues, this is my first PR to this ecosystem, so still learning what needs to be done.


⚠ Mobitopup has degraded performance

In c54977c, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 15432 ms

Resolved: Mobitopup performance has improved in f88e5f3.


onlatest:true don't add pre or suffix on latest tag

Behaviour
Steps to reproduce this issue
Look to https://github.com/Secure-Compliance-Solutions-LLC/GVM-Docker/runs/3182377485?check_suite_focus=true
Expected behaviour
The latest tag should have the suffix appended
Actual behaviour
The latest tag has no suffix
Configuration

Repository URL (if public): https://github.com/Secure-Compliance-Solutions-LLC/GVM-Docker
Build URL (if public):  https://github.com/Secure-Compliance-Solutions-LLC/GVM-Docker/runs/3182377485?check_suite_focus=true

name: Docker Image Build and Release

on:
  push:
    branches: [master]
  pull_request:
    branches: [master, dev]
  create:
    tags:

concurrency: ci-${{ github.ref }}

env:
  # Use docker.io for Docker Hub if empty
  REGISTRY: ghcr.io
  # github.repository as <account>/<repo>
  IMAGE_NAME: ${{ github.repository }}
  IMAGE_NAME_GHCR: ghcr.io/${{ github.repository }}
  IMAGE_NAME_DOCKER: securecompliance/gvm

permissions:
  contents: read
  packages: write

jobs:
  build_test:
    name: Build and Test
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: Set tag var
        id: vars
        run: echo ::set-output name=docker_tag::$(echo ${GITHUB_REF} | cut -d'/' -f3)-${GITHUB_SHA}

      - name: Download artifact
        uses: dawidd6/action-download-artifact@v2
        with:
          # Optional, GitHub token, a Personal Access Token with `public_repo` scope if needed
          # Required, if artifact is from a different repo
          github_token: ${{secrets.GITHUB_TOKEN}}
          # Required, workflow file name or ID
          workflow: build-apk.yml
          # Optional, will use the branch
          branch: master
          # Optional, uploaded artifact name,
          # will download all artifacts if not specified
          # and extract them in respective subdirectories
          # https://github.com/actions/download-artifact#download-all-artifacts
          name: apk-builds
          # Optional, directory where to extract artifact. Defaults to the artifact name (see `name` input)
          path: ${{ github.workspace }}/apk-build/
          # Optional, defaults to current repo
          repo: Secure-Compliance-Solutions-LLC/GVM-APK-build

      - name: Build the Docker image
        run: docker build . --file Dockerfile --build-arg OPT_PDF=1 --tag myapp:${{ steps.vars.outputs.docker_tag }}

      - name: Scan with Phonito Security
        uses: phonito/phonito-scanner-action@master
        with:
          image: myapp:${{ steps.vars.outputs.docker_tag }}
          phonito-token: ${{ secrets.PHONITO_TOKEN }}

  build_release:
    name: Build and Release
    runs-on: ubuntu-latest

    outputs:
      labels: ${{ steps.meta.outputs.labels }}
      tags: ${{ steps.meta.outputs.tags }}

    steps:
      - name: Checkout repository
        uses: actions/checkout@v2
        with:
          submodules: recursive

      - uses: docker/setup-buildx-action@v1
        id: buildx
        with:
          install: true

      # Login against a Docker registry except on PR
      # https://github.com/docker/login-action
      - name: Login to GitHub Container Registry ${{ env.REGISTRY }}
        uses: docker/login-action@v1
        with:
          registry: ${{ env.REGISTRY }}
          username: ${{ github.repository_owner }}
          password: ${{ secrets.GITHUB_TOKEN }}

      - name: Login to DockerHub
        if: github.event_name != 'pull_request'
        uses: docker/login-action@v1
        with:
          username: ${{ secrets.DOCKERHUB_USERNAME }}
          password: ${{ secrets.DOCKERHUB_TOKEN }}

      - name: Relase Prepare for latest Tag
        id: releasePreareLatestTag
        shell: bash
        run: |
          if [[ "$GITHUB_EVENT_NAME" == "create" ]] && [[ "$GITHUB_REF" =~ ^refs/tags/v.* ]]; then
            echo -n "::set-output name=latest::true"
          else
            echo -n "::set-output name=latest::false"
          fi
      - name: Relase Prepare
        id: releasePreare
        run: |
          echo -n "::set-output name=images::"
          if [ "${GITHUB_EVENT_NAME}" != "pull_request" ]; then
            echo -n "${IMAGE_NAME_DOCKER}"
            echo -n ","
          fi
          echo -n "${IMAGE_NAME_GHCR}"
      - name: Download artifact
        uses: dawidd6/action-download-artifact@v2
        with:
          # Optional, GitHub token, a Personal Access Token with `public_repo` scope if needed
          # Required, if artifact is from a different repo
          github_token: ${{secrets.GITHUB_TOKEN}}
          # Required, workflow file name or ID
          workflow: build-apk.yml
          # Optional, will use the branch
          branch: master
          # Optional, uploaded artifact name,
          # will download all artifacts if not specified
          # and extract them in respective subdirectories
          # https://github.com/actions/download-artifact#download-all-artifacts
          name: apk-builds
          # Optional, directory where to extract artifact. Defaults to the artifact name (see `name` input)
          path: ${{ github.workspace }}/apk-build/
          # Optional, defaults to current repo
          repo: Secure-Compliance-Solutions-LLC/GVM-APK-build

      # Extract metadata (tags, labels) for Docker
      # https://github.com/docker/metadata-action
      - name: Extract Docker metadata
        id: meta2
        uses: docker/metadata-action@v3
        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          images: ${{ steps.releasePreare.outputs.images }}
          flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=
          tags: |
            type=ref,event=branch
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{raw}}
            type=semver,pattern={{major}}.{{minor}}
      # Build and push Docker image with Buildx (don't push on PR)
      # https://github.com/docker/build-push-action
      - name: Build and push Docker image
        uses: docker/build-push-action@v2
        with:
          context: .
          push: true
          tags: ${{ steps.meta2.outputs.tags }}
          labels: ${{ steps.meta2.outputs.labels }}
          build-args: |
            SETUP=0
      # Extract metadata (tags, labels) for Docker
      # https://github.com/docker/metadata-action
      - name: Extract Docker metadata
        id: meta3
        uses: docker/metadata-action@v3
        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          images: ${{ steps.releasePreare.outputs.images }}
          flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            onlatest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=-full
          tags: |
            type=ref,event=branch
            type=ref,event=tag
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{raw}}
            type=semver,pattern={{major}}.{{minor}}
      # Build and push Docker image with Buildx (don't push on PR)
      # https://github.com/docker/build-push-action
      - name: Build and push Docker image -full tag
        uses: docker/build-push-action@v2
        with:
          context: .
          push: true
          tags: ${{ steps.meta3.outputs.tags }}
          labels: ${{ steps.meta3.outputs.labels }}
          build-args: |
            SETUP=0
            OPT_PDF=1
      # Extract metadata (tags, labels) for Docker
      # https://github.com/docker/metadata-action
      - name: Extract Docker metadata
        id: meta4
        uses: docker/metadata-action@v3
        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          images: ${{ steps.releasePreare.outputs.images }}
          flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            onlatest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=-data
          tags: |
            type=ref,event=branch
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{raw}}
            type=semver,pattern={{major}}.{{minor}}
      # Build and push Docker image with Buildx (don't push on PR)
      # https://github.com/docker/build-push-action
      - name: Build and push Docker image with -data tag
        uses: docker/build-push-action@v2
        with:
          context: .
          push: ${{ github.event_name != 'pull_request' }}
          tags: ${{ steps.meta4.outputs.tags }}
          labels: ${{ steps.meta4.outputs.labels }}
          build-args: |
            SETUP=1
      # Extract metadata (tags, labels) for Docker
      # https://github.com/docker/metadata-action
      - name: Extract Docker metadata
        id: meta5
        uses: docker/metadata-action@v3
        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          images: ${{ steps.releasePreare.outputs.images }}
          flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            onlatest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=-data-full
          tags: |
            type=ref,event=branch
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{raw}}
            type=semver,pattern={{major}}.{{minor}}
      # Build and push Docker image with Buildx (don't push on PR)
      # https://github.com/docker/build-push-action
      - name: Build and push Docker image with -data-full tag
        uses: docker/build-push-action@v2
        with:
          context: .
          push: ${{ github.event_name != 'pull_request' }}
          tags: ${{ steps.meta5.outputs.tags }}
          labels: ${{ steps.meta5.outputs.labels }}
          build-args: |
            SETUP=1
            OPT_PDF=1

Logs
logs_61.zip

@Dexus
onlatest is an attribute of prefix or suffix flavor, not a type. So
         flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            onlatest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=-full

should be:
         flavor: |
            latest=${{ steps.releasePreareLatestTag.outputs.latest}}
            prefix=
            suffix=-full,onlatest=${{ steps.releasePreareLatestTag.outputs.latest}}

Thank you, it looks i missed that.

Sorry for reopening this.
For me
flavor: |
            latest=auto
            suffix=-${{matrix.python}}-${{matrix.architecture}},onlatest=true

resulted in tags like: "***/pyinstaller:feature-buildWithGA-py3-amd64,onlatest" as you can see in: https://github.com/white-gecko/docker-pyinstaller/runs/3311866222?check_suite_focus=true#step:6:67
But I don't want the string onlatest being added to the tag. So I had to remove flag all together. How do I provide it correctly?

@white-gecko docker/metadata-action@98669ae865ea3cffbcbaa878cf57c20bbf1c6c38 (v3.3.0) doesn't provide this feature. Upgrade to v3.4.1.


Unrecognized option 'nostdin'

Hello. My installation went smoothly but when I try to play a song I get this message
[Download] Cached: https://www.youtube.com/watch?v=tg00YEETFzg
The ffmpeg program is only provided for script compatibility and will be removed
in a future release. It has been deprecated in the Libav project to allow for
incompatible command line syntax improvements in its replacement called avconv
(see Changelog for details). Please use avconv instead.
Unrecognized option 'nostdin'

and in the discord channel the bot doesn't do anything and no sound is heard.
May you please help? I'm running the bot on Ubuntu 12.04

I had the same problem, but am on debian. It was because apt-get was automatically installing an older version of ffmpeg. I had to add a third party built source here: http://www.deb-multimedia.org/ then select the newest version for ffmpeg and it fixed my problem

That message is from avconv, not ffmpeg.  I have to figure out if I can use avconv with similar options but for now you need to use ffmpeg.

Having the same issue the vps is running CentOS 6 I hope you can figure out if you can use avconv or not

The ffmpeg program is only provided for script compatibility and will be removed
in a future release. It has been deprecated in the Libav project to allow for
incompatible command line syntax improvements in its replacement called avconv
(see Changelog for details). Please use avconv instead.
Unrecognized option 'nostdin'

Good news.  I have figured out if avconv (libav) can be used or not.  The answer is no.    Good to know that nothing needs to be changed.


Updates to the parser and CLI

Fixed array destruction syntax (and added support to checker)
Update CLI things
Added retaining blank lines for formatting
Use BufferedStandardStream
Updated release compiler options
Added missing run checker example
Optional class properties
Removed Eq from parser items
More additional syntax for destructuring with type annotations
Added setter assignment subtype check and more setter tests
Add more excess property tests

🖕

A combination of retaining blanks lines as and the bizarre ; empty statement causes the fuzzing to break the parse -> print -> parse -> print equality. As this is an edge case and doesn't break anything I am going to merge and have another go at fixing it once I have a better idea of how to solve it.


MPF Dependency preventing this extension to be installed with VS2019 Community Edition

As the title suggest, on VS2019, it will pop up a dialog saying "The following reference must be present before installation - Visual Studio MPF 15.0". How do i fix it?

It currently does not support Community Edition 2019. Mainly because I haven't tested it with it and therefore didn't add it as a target. Therefore Visual Studio 2019 Pro or Enterprise is a prerequisite.
I can try to update it when I get the time.


Display and cursor corrupted when replacing a smooth-scrolled line

Problem:    When the topline is replaced while it is smooth-scrolled,
"w_skipcol" is not adjusted and the cursor becomes invalid.
Solution:   Reset "w_skipcol" when the topline is replaced.

This issue is also reproducible when using if_pyth.
With Vim script:
40vsplit
rightbelow 4new
call setline(1, ['aaa', 'bbb', 'c'->repeat(548), 'ddd'])
set smoothscroll scrolloff=0
normal!  G
redraw
python3 vim.current.buffer[2] = 'c' * 188

So maybe a more general solution is needed?

Maybe we should save the line length of topline and reset skipcol in update_topline() if it has changed. Or maybe it is necessary in check_cursor_moved() although that might be too expensive.

Or will adding this to each module that replaces buffer lines be preferable? Are there others?

I guess changed_lines() is common to any modules that change buffer lines. I'll try moving it there.

Should we use win_linetabsize() instead of STRLEN()?

IDK if this the right approach to fixing this problem but it does seem to work properly.
Not sure what's going on with the test failures either, it passed here: https://github.com/luukvbaal/vim/actions/runs/4939090814.

Instead of resetting w_skipcol to zero a "smoother" solution would be to use the largest value that works. But it probably only makes a difference in very few situations.

Hmm right. I guess that means setting w_skipcol to the largest integer multiple of the window width (accounted for col offset) that is smaller than the linetabsize.

Though I think even that would not necessarily be the "smoothest". That would probably be setting w_skipcol to the value that ensures the same amount of lines of the topline are visible as before the change. That is not necessarily the same as setting it to the largest value that works. Not sure that is even feasible without storing the number of visible skipcol lines.


Fix some issues

Fix #229
Fix #173
Fix potential problem with n_jobs. It may appear if some multiprocessing modules cannot work with n_jobs < 0. Now n_jobs is checked and converted to positive value in GraphRequirements
Fix problem with retaining composing time calculation. Initial generation is taked as full generation, but it does not consume any time for preparing. Also fixed problem with generation num increment.

поменяй плиз версию голема в этих файлах на 0.4.0

Codecov Report

Merging #230 (85431a9) into main (29cd362) will decrease coverage by 0.56%.
Report is 1 commits behind head on main.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
- Coverage   72.16%   71.61%   -0.56%
==========================================
  Files         135      136       +1
  Lines        8061     8130      +69
==========================================
+ Hits         5817     5822       +5
- Misses       2244     2308      +64

Files
Coverage Δ

golem/core/adapter/adapt_registry.py
94.59% <100.00%> (ø)

golem/core/dag/graph_utils.py
97.29% <100.00%> (ø)

golem/core/dag/linked_graph.py
98.62% <100.00%> (ø)

golem/core/dag/linked_graph_node.py
97.61% <100.00%> (ø)

golem/core/log.py
89.34% <100.00%> (-1.64%)
:arrow_down:

golem/core/optimisers/advisor.py
88.88% <100.00%> (ø)

golem/core/optimisers/fitness/fitness.py
86.90% <100.00%> (ø)

golem/core/optimisers/genetic/evaluation.py
93.33% <100.00%> (-1.05%)
:arrow_down:

...ore/optimisers/genetic/operators/base_mutations.py
90.81% <100.00%> (ø)

...lem/core/optimisers/genetic/operators/crossover.py
88.14% <100.00%> (-4.64%)
:arrow_down:

... and 27 more

... and 8 files with indirect coverage changes


CPU Bench needs more testing on different CPUs

Needs to get tested on a Core i5 6500, and a Pentium 4, should be finished before Beta 3's release.

Now getting into both machines, so tests should start soon!

Both machines are benchmarking, and the Pentium 4 is painfully slow...

Still benchmarking the Pentium 4, it's about 55% through the Long test (260,000)

About 50 minutes in, and let's see...320,000!
Anyone have a 10 hour loop of Jeopardy think music?

About 60 minutes in, let's see...348000!

Pentium 4 testing is done. It took 2 hours.
Issue closed!


Further the Project to Universal Notify

This project is only under maintenance, no further features will be added.


Don't flag whitespace around
 tags

Ref: https://github.com/yannickcr/eslint-plugin-react/issues/2988
This PR simply removes BR from the set of inline elements, because I don't think there's any case where whitespace around a br tag would result in a different result rendered in the browser.
Adds example tests for code that should not be flagged by this rule:
<App>
  A

  B
</App>

<App>
  A

  B
</App>

<App>
  A
B
</App>

<App>A
B</App>

Codecov Report

Merging #2989 (ad2b39d) into master (9aa539d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2989   +/-   ##
=======================================
  Coverage   97.59%   97.59%
=======================================
  Files         110      110
  Lines        7268     7268
  Branches     2651     2651
=======================================
  Hits         7093     7093
  Misses        175      175

Impacted Files
Coverage Δ

lib/rules/jsx-child-element-spacing.js
100.00% <ø> (ø)

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 9aa539d...ad2b39d. Read the comment docs.


Read account url from config instead of sending a http request to AWS…

Read the MediaConvert account url from config, instead of sending a http request to AWS to get this url for every job.
The describeEndpoints() has quite a low rate limit, hence multiple users uploading files at the same time, or a single user uploading > 5 files will trigger a
429 Too Many Requests
error from AWS.
This endpoint doesn't change for an account, hence AWS recommends requesting it only once and caching it.
https://docs.amazonaws.cn/en_us/mediaconvert/latest/apireference/custom-endpoints.html
It's fairly easy to get this account url

Via the console
https://console.aws.amazon.com/mediaconvert/home?#/account

Via AWS CLi
aws mediaconvert describe-endpoints

Cheers, hope it's helpful, thanks again.

@DennisOng your commits are very much appreciated and this is a great improvement! Thank you 🙏🏼
My apologies for the late response here, been a busy few weeks, but I will make sure to get this merged today.


Logarithmic axes

There are two charts that plot log10 values on the Y-Axis. However, the axis doesn't use the "logarithmic" type that chart.js supports. This is because of this chart.js issue which was marked as a duplicate of another chart.js issue.
The latter was "Closed" and marked for the v2.8.0 release (which currently isn't out yet). Once it is, the log axis type should be usable again and the plugin can be updated.

This problem is really confusing users, maybe with the newest chartjs libs this is fixed?


Ability to specify spider legs width, opacity and colour

A little feature request for one of the best leaflet plugins I've seen.
I have tried setting the stoke properties via css but it is a little glitchy as it seem the stroke is actually animated, and overriding them causes them to be what they are at the end of the animation.
ETA: I've just realised the spider leg animation is actually glitchy overidden or not, it was just more noticeable when I made the legs white and wider.
If it were possible to specify the ending stroke width, stoke opacity and stroke colour in javascript it would be nice, or do you suggest just doing this via css?
Many thanks

We have spiderLegPolylineOptions
https://github.com/Leaflet/Leaflet.markercluster#defaults
Which should do it?

Ah I'm blind - thanks danzel.


Add integration to Blend

https://github.com/madskristensen/DialToolsForVS/issues/20#issuecomment-480872017

How is this related to Avalonia?

You just need to do what mgoertz-msft said in that discussion and then Avalonia VSIX will become installable to Blend.