Clone failed: could not read Username for '$url': No such device or address

Describe the bug
Say you have two pipelines defined where the first clones fine but takes about an hour to finish. The default clone operation fails on the second pipeline with the message:
fatal: could not read Username for '$url': No such device or address

when running against Forgejo v1.19 even when WOODPECKER_AUTHENTICATE_PUBLIC_REPOS=true is set.
I believe this issue is related to some timeout value on Forgejo's side since in its logs I get a 404 Unauthorized access message.
System Info
The /version slug is not working for me against my server instance -- I'm getting a 404.
Version: next-3a475ce2
Image: docker.io/woodpeckerci/woodpecker-server:next
Hash: 09e1c0597a92
Compose file example:
version: "3.8"
services:
  app:
    image: docker.io/woodpeckerci/woodpecker-server:next
    ports:
      - "<REDACTED>"
    environment:
      - WOODPECKER_OPEN=true
      - WOODPECKER_ADMIN=<REDACTED>
      - WOODPECKER_HOST=https://<REDACTED>
      - WOODPECKER_AGENT_SECRET=<REDACTED>
      - WOODPECKER_GITEA=true
      - WOODPECKER_GITEA_URL=https://<REDACTED>
      - WOODPECKER_GITEA_CLIENT=<REDACTED>
      - WOODPECKER_GITEA_SECRET=<REDACTED>
      - WOODPECKER_AUTHENTICATE_PUBLIC_REPOS=true
    volumes:
      - woodpecker:/var/lib/woodpecker/
volumes:
    woodpecker:

Additional Context
As a workaround I implemented my own clone step using an access token as the password instead with an alpine container. For example, cloning<EMAIL_ADDRESS>works fine for me.
This is what I'm using specifically:
skip_clone: true

pipeline:
  clone:
    image: docker.io/alpine/git:latest
    secrets:
      - source: access-token-secret
        target: ACCESS_TOKEN
    commands:
      - git init -b $$CI_COMMIT_BRANCH
      # NOTE: Replace GIT_USER on the next line
      - MODIFIED_REPO_URL=$$(printf "%s\n" "$$CI_REPO_REMOTE" | sed -e "s|https://|https://GIT_USER:$$ACCESS_TOKEN@|g")
      - git remote add origin $$MODIFIED_REPO_URL
      - git fetch --no-tags --depth=1 --filter=tree:0 origin "+$$CI_COMMIT_REF"
      - git reset --hard -q $$CI_COMMIT_SHA
      - git submodule update --init --recursive
      - git lfs fetch
      - git lfs checkout

that has something to do with the crafted netrc config ...

... not sure what exactly go wrong through ... based on the info you provide

BUT, if I remove the clone

Now that you mention it, I recall trying to configure the clone plugin like that and I stumbled upon the same issue.
In fact, I was able to reproduce the issue with just this:
clone:
  git:
    image: woodpeckerci/plugin-git:2.0.3

Could be related to the change from #1352

Could be related to the change from #1352

Probably it is. After rollback to docker image before this pr almost everything works again.

As a workaround, you can disable this in the repository settings, it's work for me:

@Sebclem, I have the same issue, but somehow I don't see the option you mention to disable as workaround. Why is that?

@Sebclem, I have the same issue, but somehow I don't see the option you mention to disable as workaround. Why is that?

I had Cloudflare caching an old version on my end, I had to disable it. Could you have a similar issue? After fixing that on docker container version next-9b32d021b3 I now see the option.

I do have the same issue. One thing I noticed is that it seems to work fine if only one pipeline is run at a time (which means that they can start immediately). As soon as jobs are queuing (which means that they start delayed) they run into this problem. I disabled (unchecked) "Only inject netrc credentials into trusted containers but it still doesn't work.
I am also using the next version and my step is configured like that:
clone:
  git:
    image: woodpeckerci/plugin-git
    environment:
      - PLUGIN_LFS=false
      - PLUGIN_SKIP_VERIFY=true

... not sure what exactly go wrong through ... based on the info you provide

what info would be useful to provide?

Unchecking "Only inject netrc credentials into trusted containers" worked for me.

I found this issue today by coincidence: https://github.com/gitkraken/vscode-gitlens/issues/1027. They report to having this problem when using HTTPS instead of SSH. I then saw in the log git remote add origin https://forgejo.***.ch/***/***.git. Maybe switching to SSH would make it more stable?

Maybe switching to SSH would make it more stable?

you can use ssh only with users ssh key and woodpecker does not have it and should not have it either so that's not really an option

Unchecking "Only inject netrc credentials into trusted containers" worked for me.

I wonder if this should be the default given how many people seem to face issues with it. And not all of them will arrive here and read through this issue?

As this is only in development version and 1.0 will be breaking anyway, it's better to use secure by default

Unchecking "Only inject netrc credentials into trusted containers" worked for me.

I wonder if this should be the default given how many people seem to face issues with it. And not all of them will arrive here and read through this issue?

I still have the issue even when I uncheck the option. Doesn't seem to be a "reliable workaround".

@patrickuhlmann Do you still face this error with the latest plugin version and latest WP server? If so, could you post your setup in more detail and also what repo options are enabled/disabled?

I updated all components and still face the same issue.
I run everything in docker containers on a Synology Diskstation. I have the following containers:

codeberg/forgejo/forgejo:1.20
woodpeckerci/woodpecker-server:v1.0.4
woodpecker/woodpecker-agent:v1.0.4 (first runner)
woodpecker/woodpecker-agent:v1.0.4 (second runner)

The error happens when I run renovate. This job is running more than one hours and occupies one runner. It creates many pull requests which in turn trigger builds on other repositories. These builds are running on the second runner. In the beginning everything works fine. After a while (when lots of jobs are queued up) they start to fail.
The job output is always like this:
+ git config --global http.sslCAInfo /opt/MyLAN.crt
+ git init -b master
Initialized empty Git repository in /woodpecker/src/forgejo.me.ch/My/repo.git/
+ git remote add origin https://forgejo.me.ch/My/repo.git
+ git fetch --no-tags --depth=1 --filter=tree:0 origin +refs/pull/108/head:
fatal: could not read Username for 'https://forgejo.me.ch': No such device or address
exit status 128

All pipelines are similar. For example
clone:
  git:
    image: woodpeckerci/plugin-git
    environment:
      - PLUGIN_LFS=false
      - PLUGIN_CUSTOM_SSL_PATH=/opt/MeLAN.crt
    volumes:
      - /volume1/docker/woodpecker/MeLAN.crt:/opt/MeLAN.crt

pipeline:
  verify:
    image: gradle:8.1.0-jdk17-focal
    commands:
    - gradle assemble
    - gradle check
    volumes:
    - /volume1/docker/woodpecker/gradle:/root/.gradle

In the forgejo log I see
..rvices/auth/basic.go:130:Verify() [E] UserSignIn: user's password is invalid [uid: 1, name: patrick]
...s/auth/middleware.go:23:func1() [E] Failed to verify user: user's password is invalid [uid: 1, name: patrick]
 ...eb/routing/logger.go:102:func1() [I] router: completed GET /My/repo.git/info/refs?service=git-upload-pack for <IP_ADDRESS>:0, 401 Unauthorized in 80.4ms @ auth/middleware.go:20(auth.Auth)

In the runner I see
ERR grpc error: wait(): code: Unknown: rpc error: code = Unknown desc = Step finished with exit code 1,  | error=rpc error: code = Unknown desc = Step finished with exit code 1,
WRN cancel signal received | repo=My/repo pipeline=104 id=3156 error=rpc error: code = Unknown desc = Step finished with exit code 1,

In the woodpecker logs, I see
 ip=<IP_ADDRESS> latency=12411.027311 method=POST path=/hook status=500 user-agent=Go-http-client/1.1
ERR failure to save pipeline for My/repo | error=database is locked
ERR error=Error #01: failure to save pipeline for My/repo

One thing I am wondering is this "database is locked". Is this normal? Might the problem be that I am using an sqlite3 database?

Likely, I've seen this error with sqlite3 in the past when there was too much load on the DB. And as you're saying, during the renovate run more runs spin up from the PRs opened by renovate which then likely overload the sqlite3 DB.
sqlite3 is usually only suitable for dev purposes, it's better to use postgres or mysql even for semi-production home use.

Hi all, I'm running into this problem with Gitea latest and woodpecker latest as of today.
Checking/unchecking "Only inject netrc credentials into trusted containers" has no effect on the outcome of the job.
I've also tried a brand new repository (private and public), with and without the checkbox above enabled. All the same.

I switched to postgres but face still the same problem. Btw. in the end it would have surprised me as Sqlite is very much underestimated. It was already able to handle thousands of selects and inserts in a very short time even on multiple gigabytes large databases years ago (you will find plenty of benchmarks info if you search for it).

@thechubbypanda can you verify the issue only exists in WP latest and not in WP 2.0.0? If so, maybe you can track it down further to a specific commit? All main branch commits have associated images.

I am now running WP 2.0.0 and still have this issue

Ok so:
WP Server: next-1ca549190b
WP Agent: next-1ca549190b
plugin-git: latest

Gives:

@thechubbypanda Thanks. Strange though, as many people use 2.0 + meanwhile and we haven't yet heard of more issues like this.
I also administrate multiple instances and haven't come across the issue in months.
Are you running WP in docker or via a host install?
And since you wrote

Hi all, I'm running into this problem with Gitea latest and woodpecker latest as of today.

Did it work before with an older version/different setup?

In terms of "today" I merely meant that I just tested it. Given that the issue is a few months old now.
I'm running dockerized at the moment.
I started spinning up a completely clean installation of both Gitea and woodpecker yesterday. Will report back if that works. Then it's a matter of narrowing down what setting or situation is causing the problem.
I will note that it's extremely tough to debug the docker images given they appear to not even have sh installed.

FOUND IT:
GITEA__service__REQUIRE_SIGNIN_VIEW: true
Set that to true and the error appears, set to false, it works as expected.

I will note that it's extremely tough to debug the docker images given they appear to not even have sh installed.

Yes, this is (partly) known. Even though my guess would have been that it is very unlikely that it is an issue in the image as otherwise we would have gotten many more reports here.
It is likely that WOODPECKER_AUTHENTICATE_PUBLIC_REPOS helps, it might to partly the same that GITEA__service__REQUIRE_SIGNIN_VIEW does.
@qwerty287 Maybe we should add a warning if Gitea is used as a forge and WOODPECKER_AUTHENTICATE_PUBLIC_REPOS is not true?

Is there a scenario where just having that enabled by default is a bad idea? Or at least inversing it?

It is likely that WOODPECKER_AUTHENTICATE_PUBLIC_REPOS helps

Well regardless, that has fixed the issue for me at least. Thanks

The error message is quote non-descriptive and it's not easy for users to find out how to solve the issue. Even when searching the main repo this is tricky as the issue is being discussed/reported here, and not all users will even arrive here.
I wonder how this can be better communicated - in the end, such details/issues lead to a bad user experience.

Closing here now as the issue seems to be resolved and other users arriving here should find the solution.
Yet I think we should somehow assert GITEA__service__REQUIRE_SIGNIN_VIEW if that's the real underlying cause of this error?

So it was  #25 all along :/

For me I think the problem was weak hardware. As soon as I switched from my diskstation to a dedicated computer and forgejo as well as woodpecker run much faster the problem was gone as well.


Jetson-TX2 contribution

Hi, and thanks for your work.
I updated your code to make it work on a jetson-TX2 which is compatible with:

OpenCV GSTREAMER
C++14

https://github.com/ltetrel/YOLOv8-TensorRT-CPP/tree/feat/jetson-tx2
Let me know how you want to proceed, from there I see 3 options:

Changes in another branch feat/jetson-tx2 (as I did on my fork). I recommend this and I would need you to create this branch on your repo so I can make a PR.
Changes living in main branch. I don't recommend this to avoid polluting your original code but that could be possible to have everything at the same place with a bunch of #ifdef depending on cuda version , c++ etc...
No mention to this work on your repo, even if I think it could benefit the community.

See also: https://github.com/cyrusbehr/tensorrt-cpp-api/issues/21
Let me know,

Hi @ltetrel
Thank you for the good work.
Let's go with your recommended approach. I'll create the feat/jetson-tx2 branch and you can create a PR with your work. Thank you for your contribution.

Thanks,
https://github.com/cyrusbehr/YOLOv8-TensorRT-CPP/pull/23

https://github.com/all-contributors please add @ltetrel for code

@all-contributors please add @ltetrel for code


what's ATen?

i follow solution in #54, but i met this problem:
home/sti/kevin/work/smoke/SMOKE/smoke/csrc/cpu/dcn_v2_psroi_pooling_cpu.cpp:16:10: fatal error: ATen/ceil_div.h: No such file or directory
#include <ATen/ceil_div.h>
^~~~~~~~~~~~~~~~~
compilation terminated.

please read readme first.

Hi, I found the same problem as @shaqing  on the master branch using pytorch1.8.1+cuda111. When I try pytorch1.6 branch, I get the following error:
 error: ‘THFloatBlas_gemm’ was not declared in this scope
         THFloatBlas_gemm('n', 't', n, m, k, 1.0f,

any comments?

Am not going support old pytorch anylonger, but you can switch to older branch for luck.

I understand. Thank you very much!


fix: add json attributes to TokenSearchRequest

Description

Adds JSON attributes to TokenSearchRequest model to properly serialize property names

Testing required outside of automated testing?

[X] Not Applicable

Screenshots (if appropriate):

[X] Not Applicable

Rollback / Rollforward Procedure

[X] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

v1.37.1
GitHub release

Your semantic-release bot :package::rocket:


[Fix] update odnoklasniki docs to new domain ok

Hi, there.
About a year ago odnoklassniki social network switched out their domain and name to ok.
You can chech it by url (http://ok.ru/group/57170323832868/topic/62946583920676).
I've updated documentation and oauth entry points.

Up

Thanks!


#475

Thanks for the doc fix. Hopefully we can improve this restart restriction later on.

Yeah, let’s start a discussion about that :)

On 29 Jan 2016, at 18:09, Joe Nelson<EMAIL_ADDRESS>wrote:
Thanks for the doc fix. Hopefully we can improve this restart restriction later on.
—
Reply to this email directly or view it on GitHub https://github.com/begriffs/postgrest/pull/477#issuecomment-176891519.


Link to the Nuget package

My journey was:

Read the Microsoft documentation, find "sqlite-net-pcl"
Open it in Nuget, finding version 1.8.116
Searching again for the name, finding this Github repo
Now reading here the only link to Nuget as "sqlite-net-pcl" again
Wondering why repo says 2.x and that package has 1.8.x
Reading about "Microsoft.Data.Sqlite" here, that it uses PCLRaw, too
Open it in Nuget, finding version 8.0.1 uses "SQLitePCLRaw.core", which has version 2.1.7

Am I the only one? If someone reads official docs, then finds this Git repo, and still wants to use SQLitePCLRaw without another wrapper around it, then I would have used the wrong Nuget package. Recommending a "raw" package to any user should be a no-go in my eyes. If you don't know this "raw" thing, you ain't gonna need it, I would suppose.
Can you give some guidance to this or am I the only one to ever wonder about versions and links?

I am confused by your story of how you were confused.  :-)
Yes, there are multiple Nuget packages in your story.  SQLitePCLRaw (actually a collection of packages) is a dependency of the other two, sqlite-net-pcl and Microsoft.Data.Sqlite.  They're all distinct, maintained by different people or organizations, which (in part) is why they have different version numbers.  I maintain SQLitePCLRaw, and I don't have any control over the other packages or the Microsoft documentation.
If you are suggesting a change here in this project, please be specific.

Okay. I thought maybe have "SQLitePCLRaw.core" or something as NuGet link, because the result of the repo is pushed somewhere, I guess, if you are this guy https://www.nuget.org/profiles/ericsink


Add MediaStream video support section

@mounirlamouri Does that look good to you?

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2018/10/watch-video-using-picture-in-picture.md#L318 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful

Can we merge this please?

:tada: This has been pushed live to https://developers.google.com/web/


[opentelemetry-kube-stack] Define default versions for instrumentation libraries

         The current PR might be good enough as is for this topic. It might be worth creating a follow up issue for the work to ensure the used instrumentation version values are in sync with the version of the operator used. If a new ticket is created, I would like to volunteer to work on it.

Originally posted by @jvoravong in https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1114#discussion_r1557854017

Please assign this ticket to me. Will populate the ticket information and follow up as needed.


Issue with VSCs being cleaned up

Although we delete the VSCs after a backup/restore, certain VSCs can still persist in the cluster if they include a finalizer. These need to be fully removed.

Will be resolved with Velero 1.9


Different fps for video and preview output

Is it possible to modify the code such that the frame rates for video and preview output are different? I'm using the camera on a resource contraint platform and want to lower the computational burden. As an example I would like to have 30 fps for the preview output and 2 fps for the video output.
Alternatively, could there be a ROS service to capture still images at the full sensor resolution?

We are cleaning up this repository as we intend to use it for some use cases of our ROS nodes - as this is already moved to depthai-ros I think it is safe to close this issue here.


git-tag-project-version-and-build-number 1.0.0

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

@trapacska Done 👍 Ready for round 2

Since there is no output variables generated by step you should also remove envman add --key EXAMPLE_STEP_OUTPUT --value 'the value you want to share' from here https://github.com/ned1988/bitrise-step-git-tag-project-version-and-build-number/blob/master/step.sh#L73 as well as envstore file.

@koral-- Done 👍

Thx @ned1988 @koral-- @BirmacherAkos !
🦄

Hi!
There was an issue with releasing the step.

This could happen when you manually change the git tag after you called the
bitrise create-release workflow.
You should not do that.
The solution is:
You need to create a new release with v1.0.1.

Whoops! 😯 Sorry for that, will try to fix asap.

@BirmacherAkos created new PR


Improve styles / visuals

I'm opening this issue so we can discuss specifically about style and visual improvements.
I'll list the questions I have here as I'm exploring your code.
Questions

What are the absolute no-gos in terms of css, is there anything we should avoid because of perf issues?
Would you accept some (small) svgs for some Icons, or is the overhead something you want to avoid?

Thanks for your enthusiasm! My performance decisions are all data-based. So if the App interactive (ms) in the console doesn't change by a mile then everything is fine.
The focus performance-wise is on the return visitor. For those, 50% will have the entire site cached (Android + Windows support service workers) and the other 50% (iOS doesn't) will just have external assets cached. So external assets are preferred (e.g. svg files used as <img>s).
Assets are served over HTTP2 so multiple network requests are preferred (e.g. not sprites).
Some small SVGs would be fine if they serve a purpose.
You can put them in /assets with the other pngs and they should be added to the caching setup. Make sure that once you've done npm run build and npm start and loaded the site, that it works and loads all the assets if you don't have network connection (in Chrome you can turn off the network).

Well it's only normal since you are, in my opinion the one to be thanked for!
I won't be able to iterate super quickly (you know, work...) though I'm excited to dive into this and explore your crazy fine tuned application.
Ok then, I'll try to document as much as I can what works and what doesn't work well in terms of performances.


Updates breaking change documentation to use xrefs

This PR:

Changes all the usages I could from pointing to msdn.microsoft.com to the new docs.microsoft.com.
Changes API references to use cross-links
Edits the wording issues mentioned in PR 1923.
Removes the [More Information] links that led to Application Compatibility page with the exact same info.

Required to generate correct data for https://github.com/dotnet/docs/pull/1923
/cc: @rpetrusha

@rpetrusha The documentation for cross-referencing says that this is possible.  The xrefmap.yml map for possible cross references has an entry like:
- uid: System.Runtime.Serialization.NetDataContractSerializer.#ctor
  name: NetDataContractSerializer()
  href: api/System.Runtime.Serialization.NetDataContractSerializer.-ctor.html#System_Runtime_Serialization_NetDataContractSerializer__ctor
  commentId: M:System.Runtime.Serialization.NetDataContractSerializer.#ctor
  fullName: System.Runtime.Serialization.NetDataContractSerializer.NetDataContractSerializer()
  nameWithType: NetDataContractSerializer.NetDataContractSerializer()

So, if I wanted my text to display "System.Runtime.Serialization.NetDataContractSerializer.NetDataContractSerializer()", I'd use displayProperty=fullName, but if I just wanted to see "NetDataContractSerializer()", I'd use: displayProperty=name

@conniey, yes, that's how it should work. Is that how it's actually working?
The displayProperty=fullName key/value should actually cause the text to be "NetDataContractSerializer.NetDataContractSerializer()" without the namespace name, although the current behavior is to include the fully qualified namespace as well.

@conniey, actually, I apologize -- in your previous comment, I read what I thought was there, rather than what you actually wrote. The correct syntax is displayProperty=fullName.
From

@rpetrusha

"displayProperty: the property of display text when the cross reference is has resolved correctly.
e.g.: will be resolved as System.String."

The displayProperty=name maps correctly to just the name.  The property in your example is "fullname", the property I am trying to show is "name".
MD file: eventlistener-truncates-strings-with-embedded-nulls.md
Preview: 46: EventListener truncates strings with embedded nulls
The ApiPort tool uses the Affected Apis.  We map those docIds to the correct cross reference when creating a PR for the docs site.

Another example is the event System.Web.UI.WebControls.GridView.PageIndexChanging, where I want to see just the name "PageIndexChanging" and use displayProperty=name:
116: GridViews with AllowCustomPaging set to true may fire the PageIndexChanging event when leaving the final page of the view

@conniey, I don't think that displayProperty=name is supported -- it happens to work since displaying the type or member name only is the default behavior. But since it's not breaking anything, it's harmless. I'll approve the changes, and leave it to you to merge.


proposal: cmd/go: make "go mod vendor" also copied the used std libs

For example, the source code like below:
package main

import "fmt"

func main() {
    fmt.Println("hello world")
}

If we use go mod vendor for it, it can't guarantee the code of fmt is consistent.
Pros:

It will help people to track the changes on stdlib, such as unexpected changes on stdlib when they upgrade Go version.
Make debugging stdlib on the fly much easier. Such as people may modify the source code of fmt in their local GOROOT and forget to revert it. If the vendor has a copy of fmt, it will be much easier to modify the stdlib.
Make cross-compile stdlib between different Go runtime much easier. Such as let old stdlib benefit from the improvements from newer GC, etc.

Cons:

May break things if the newer Go compiler tries to compile old stdlib?
Hard to implement? The development cost can't justify the gain.

Maybe we can first add a flag to the CLI to make it optional: go mod vendor -s, s means also include the stdlib.

The standard library — particularly the runtime package — is tightly coupled to the Go compiler. It is infeasible to vendor the standard library separately from the compiler.
Certain parts of the standard library could be made into wrappers around external modules, and in theory we could then allow those modules to be upgraded and vendored separately from the standard library. That proposal is #30241.

Thanks for the mention!


BWS halt

Besu fails to finish syncing failing on the backward sync over and over.
These logs are repeated until Besu is restarted. After restarting Besu finishes the BWS normally.
{"@timestamp":"2024-04-22T00:43:48,910","level":"INFO","thread":"vert.x-worker-thread-0","class":"BackwardSyncContext","message":"Starting a new backward sync session","throwable":""}
{"@timestamp":"2024-04-22T00:44:55,166","level":"INFO","thread":"EthScheduler-Timer-0","class":"BackwardSyncContext","message":"Current backward sync session failed, it will be restarted","throwable":""}
{"@timestamp":"2024-04-22T00:45:00,426","level":"INFO","thread":"vert.x-worker-thread-0","class":"BackwardSyncContext","message":"Starting a new backward sync session","throwable":""}
{"@timestamp":"2024-04-22T00:46:06,945","level":"INFO","thread":"EthScheduler-Timer-0","class":"BackwardSyncContext","message":"Current backward sync session failed, it will be restarted","throwable":""}
{"@timestamp":"2024-04-22T00:46:13,141","level":"INFO","thread":"vert.x-worker-thread-0","class":"BackwardSyncContext","message":"Starting a new backward sync session","throwable":""}

maybe this is fixed by something in main - haven't seen this on any burn in nodes

Haven't seen this recently. Closing, we can re-open if we see this again


Error: A route for the path "/en/pathInEnglish" already exists by the name of "myPage".

I have an app I've been working on for about two months. I had a router.js with about thirty routes in it, all with /en and /it prefixes and it was working perfectly thanks to iron-router-i18n.
About a week ago I did a 'meteor update' and since then meteor just crashes at start up with the above message or similar. Since then I have been working continuously trying to work out what is wrong/correct the problem. I have reached dead end after dead end. The only way to stop the crash is to uninstall your package.
I have constructed a totally minimal router.js to illustrate the problem. I would be grateful if you would simply look at my minimal router.js and tell me what is wrong with it. Clearly there is no other route already existing with the same name.
=> Your application is crashing. Waiting for file change.
=> Modified -- restarting.
W20150517-23:44:13.156(2)? (STDERR)
W20150517-23:44:13.157(2)? (STDERR) /Users/malcolm/.meteor/packages/meteor-tool/.<IP_ADDRESS>wysac9++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150517-23:44:13.157(2)? (STDERR) 						throw(ex);
W20150517-23:44:13.157(2)? (STDERR) 						      ^
W20150517-23:44:13.177(2)? (STDERR) Error: A route for the path "/en/pathInEnglish" already exists by the name of "myPage".
W20150517-23:44:13.177(2)? (STDERR)     at Iron.utils.assert (packages/iron:core/lib/iron_core.js:10:1)
W20150517-23:44:13.178(2)? (STDERR)     at Function.Router.route (packages/iron:router/lib/router.js:137:1)
W20150517-23:44:13.178(2)? (STDERR)     at Function.Router.route (packages/martino:iron-router-i18n/lib/router.js:269:1)
W20150517-23:44:13.178(2)? (STDERR)     at app/lib/router.js:15:8
W20150517-23:44:13.178(2)? (STDERR)     at app/lib/router.js:27:3
W20150517-23:44:13.178(2)? (STDERR)     at /Users/malcolm/git/malcolm-warren/sagrona/.meteor/local/build/programs/server/boot.js:222:10
W20150517-23:44:13.178(2)? (STDERR)     at Array.forEach (native)
W20150517-23:44:13.178(2)? (STDERR)     at Function..each..forEach (/Users/malcolm/.meteor/packages/meteor-tool/.<IP_ADDRESS>wysac9++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150517-23:44:13.178(2)? (STDERR)     at /Users/malcolm/git/malcolm-warren/sagrona/.meteor/local/build/programs/server/boot.js:117:5
=> Exited with code: 8
=> Your application is crashing. Waiting for file change.
Router.js
I18NConf.configure({

    defaultLanguage: 'en',

    languages: ['en', 'it'],

    autoConfLanguage: false

});

Router.configure({
    layoutTemplate: 'layout'
});

Router.route('/en/pathInEnglish', {
    name: 'myPage',
    i18n: {
        languages: {
            it: {
                path: '/it/pathInItalian'
            }
        }
    }

});

It's a strange configuration as usually Iron Router i18n is used to automatically add the lang prefix ('en', 'it') to the route so . Did you just update meteor or also updated Iron Router i18n? Which version of Iron Router i18n were you using and which are you using now?
It should work anyway so I'll have a look asap.

<EMAIL_ADDRESS>was the version before update
<EMAIL_ADDRESS>is current version
iron-router stayed at 1.0.7
Yes I do have the lang prefixes, but they aren't essential to me.
I tried without as well, but Meteor continued crashing, so after a couple of days of trying I'm assuming it wasn't me, and am turning to you in the hopes that you can find something.
Current versions file for what it's worth:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I've tried my little example without prefixes and yes it works, so please don't waste time on this. I will go back to the main app and take a further look at this, and get back if necessary

This is what I was thinking: what confuses Iron Router i18n are the prefix routes, neverthless it should work also with lang prefixed routes or at least give a warning so I'll keep this issue open even if it's a minor issue :-)

Thank you for your help, it's all beginning to work again. I can't explain why I put the prefixes. I also had to change Router.getLanguage() to I18NConf.getLanguage() which I found in another issue.
Thanks again.

Ok, I now recall why I ended up adding the lang prefixes: it was because it was the only way I found to solve an apparently simple task, which is a little exception for the home page.
In english, and only in english, I need to leave off the lang prefix for the home page, thus on my test machine http://localhost will respond in english and my domain home page in production will respond nicely in the same way.
Should be easy: must be a combination of compulsoryLangCode: false with missingLangCodeAction or addLangCode (or maybe exclude). I tried every imaginable combination of these, but nothing wanted to work. FATAL ERROR!
I finally worked it out: I should have left compulsoryLangCode: true . Because missingLangCodeAction isn't even called if compulsoryLangCode is false. I suppose with hindsight that this is logical, but I spent an indecent amount of time, which I won't admit to anyone, solving this problem.
Now I am trying to get missingLangCodeAction to work, again with success. If you search the internet you will see that there is not a single example on the whole of the web, and I am stuck.
The path provided is actually a url string, so I first find the path without the host, then I'd like to see the page of course. But I just get the normal layout, header and footer, and the actually content doesn't show (if I return false). If I return true, I get a blank page.
I would like to just say "Router.render('home')" I suppose, and return true but I don't have a reference to Router here.
Here is my code:
   missingLangCodeAction: function(path){
        if(Meteor.isServer)
            return false;

        var doubleSlashIndex = path.indexOf('://');
        var firstPathSlash = path.indexOf('/', doubleSlashIndex + 3);
        var myPath = path.substring(firstPathSlash);

        if (myPath === '/') {
            console.log('found my path and responding', myPath);
            I18NConf.setLanguage('en');
            //Router.render('home');
            return true;
        }
         return false;
    }

There is an option to avoid lang code for all default language routes which is langCodeForDefaultLanguage (default is true) but I guess you just want to avoid language code for the home page.
The router is available inside any configurable hook as "this", not sure whether rendering the template inside missingLangCodeAction would work.
A cleaner way to solve your problem could be to use exclude: true option for the home page default language route and manually add the other i18n paths/routes needed.
P.S.
As for the fact missingLangCodeAction is called just when compulsoryLangCode is true I will add this on documentation (it's just stated in  compulsoryLangCode docs unfortunately).

Ok, I shall drop my stupid idea to have just the home page without the prefix. Much better to go with the package defaults (I will use langCodeForDefaultLanguage = false) and have less of my own code to worry about.
Thank you so much for your time and help.


Fix generate_map_case

Which issue does this PR close?

Closes #1456.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Thanks @viirya  -- sorry for the delay -- I was traveling last week


[Bug/Regression?] UIM_DEBUG not doing anything

UIM_DEBUG appears to not be valid for use anymore
Python: Python 3.6.8
Library: maltego-trx==1.3.7
Client: Maltego One <IP_ADDRESS>79 Enterprise
Sample Code Snippet from transform
from maltego_trx.maltego import UIM_FATAL,UIM_PARTIAL,UIM_INFORM,UIM_DEBUG
response.addUIMessage("Hello FATAL World", UIM_FATAL)
response.addUIMessage("Hello PARTIAL World", UIM_PARTIAL)
response.addUIMessage("Hello INFORM World", UIM_INFORM)
response.addUIMessage("Hello DEBUG World", UIM_DEBUG)

from maltego_trx.maltego import UIM_TYPES
response.addUIMessage("Hello FATAL World", UIM_TYPES['fatal'])
response.addUIMessage("Hello PARTIAL World", UIM_TYPES['partial'])
response.addUIMessage("Hello INFORM World", UIM_TYPES['inform'])
response.addUIMessage("Hello DEBUG World", UIM_TYPES['debug'])

Output - Transform Output
Running transform [AV] AKAToEmailAddress on 1 entities (from entity "aka007")
Hello FATAL World (from entity "aka007")
Hello PARTIAL World (from entity "aka007")
Hello INFORM World (from entity "aka007")
Hello FATAL World (from entity "aka007")
Hello PARTIAL World (from entity "aka007")
Hello INFORM World (from entity "aka007")
Transform [AV] AKAToEmailAddress returned with 1 entities (from entity "aka007")
Transform [AV] AKAToEmailAddress done (from entity "aka007")

@tbennett6421, @totakura,
Debugging is still supported. In order to see the debug messages in the Maltego Desktop Client Output window, enable debug in the filter settings.


does this work with react-native 0.26.0?

react-native 0.26.0
`
import React, { Component } from 'react';
import {
AppRegistry,
StyleSheet,
Text,
View,
TouchableHighlight
} from 'react-native';
let SortableListView = require('react-native-sortable-listview');
class SortableListViewExample extends Component {
render() {
return (
  <View style={styles.container}>

  </View>
);

}
}
AppRegistry.registerComponent('SortableListViewExample', () => SortableListViewExample);
`

Any News on this? Same thing for me

Hey there, PR was just accepted that should resolve this. RN 0.26 deprecated requiring react from the react-native package.


Got SIGTERM, shutting down Homebridge... when Error: application token is invalid

Thanks to your new discovered plugin I'm trying to also play around Pushover.
I noticed that homebridge is crashing when the token is invalid.
Maybe not the whole homebridge should stop working and crash in such a case?
Thanks in advance. Love that I have found your project.
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]: [2020-4-23 15:34:57] [Messenger] Test-Pushover-Messages : Message sent to xxxxxxxxx
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]: [2020-4-23 15:34:57] Error: application token is invalid
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]:     at Pushover.errors (/usr/local/lib/node_modules/homebridge-messenger/node_modules/pushover-notifications/lib/pushover.js:83:12)
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]:     at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/homebridge-messenger/node_modules/pushover-notifications/lib/pushover.js:167:11)
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]:     at IncomingMessage.emit (events.js:322:22)
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]:     at endReadableNT (_stream_readable.js:1187:12)
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]:     at processTicksAndRejections (internal/process/task_queues.js:84:21)
Apr 23 15:34:57 raspberrypi-3BPlus homebridge[6416]: [2020-4-23 15:34:57] Got SIGTERM, shutting down Homebridge...

And I have problems to identify what my "user" and "token" is?
When I go over to https://pushover.net and login, I did "Create a New Application". I received a API Token/Key like "ab34cd78ef12gh56ij90kl34mn78op"
I assume this is to put in the config under "token": "ab34cd78ef12gh56ij90kl34mn78op".
But what do I have to use for "user": "xxxx",? Obviously it's not my E-Mail I login to Pushover.
Unfortunately I can't anything in the readme instructions here.
Thx.

To be honest, it's as designed.
If Pushover or any other service is not able to send the message, it raises an error.  The goal for this is to tell you that something is wrong. That being said, I will try and find an improvement on this. Homebridge doesn't have any warning mecanism.

Saying that, this does make sense. 😅
"user": = User Key from Dashboard
"token": = Create a New Application, API Token/Key
Thanks you. 🙈 Works now.
If you are willing and it is possible to implement url and attachment Pushover seems to be a great alternative for IFTTT Notification maybe. At lease another option.

URL is possible. So is sending HTML messages.
Attachment would probably require to add a file upload fonction. See pushover API. That doesn't seem like a good idea.

I thought it would maybe possible to grab the image from an URL like IFTTT is doing it.
I use the URL for URL-Scheme to open with the Push an Application like evehome:// or HomeConnect://
May you move me over to your plugin by finding some equal features. :)
By the way, I miss your PayPal or BuyMeACoffee link. ;)
Thanks a lot. I appreciate you work and the friendly contact.

IFTTT is a bit different than Pushover. The APIs are not the same. However, Pushover is faster and more reliable.  You could have an image in HTML format like this in your message. That will work once the HTML property will be added.

That looks perfect to me. Main thing is to insert a custom url to point to the image. And the Image is showing up in the notification like in the example.

An click on "View", I think in Pushover it's "Open URL" would open what is under "url": evehome://
👍

URL and Message will be added when issue #3 will be completed. It will used Pushover's features.
It should work like you wish (but it's not related to plugin, but how Pushover works).
I will close this issue.


Settings are not saved

OS version: Windows 10 (1809)
Browser version: Firefox 66.0
Extension version: 1.23.3

🐛 Describe the bug
Settings are reset every time the browser is restarted
Expected behaviour
Settings like idle detection are persistent
Steps to reproduce

login in the extension popup
open the new standalone settings page by clicking the settings icon or via the add-ons page
change something
optionally wait some time, refresh the page, do some stoff
quit Firefox
re-open Firefox
open settings page again
many settings are restored to its default values

Other details or context

happening since the last update, works fine in 1.22.2 (previous version, installed from mozilla.org)

ref -<PHONE_NUMBER>0

Thanks for reporting this. A fix is in the works.

1.23.4 has been published and should arrive in your browser in the next few hours. Thanks for your patience.


Minor correction

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/data-guide/scenarios/data-transfer.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Attention C Mathematical formula？

Can you provide a mathematical formula for the feature calculation of the attention mechanism C?
Note that the mechanism C only has graphics
Thank you very much for your reply, thank you   ：）

The Global Image Feature Vector is passed through a single Dense layer to get the Intermediate Feature Vector.  After this, it is the same as Attention Mechanism B.


⚠ Samsung has degraded performance

In 4c4f04d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2380 ms

Resolved: Samsung performance has improved in 162cb23.


[New game]: Knife Game

🎮 Game Request
In this game the user has to click on the knife and it will go to the board , if two knives collapse the game will end. It has 4 different levels and each level has different strategy. If a player loses a level it will automatically goto the first level. I am a GSSoC'24 contributor.
Point down the features

It has 4 levels and after clearing 1 level only the user can go to another level.
There will only be particular number of knives and it will decrease on each throw.
It will look as follow.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @sharmi4590 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Assign

/assign

Hey @sharmi4590 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Add Lazy(T value) constructor

Corresponds to dotnet/coreclr#6304.

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


GmsCore couldn't be installed on your phone (Lenovo Z90-7)

I'm looking for an alternative to gapps since my phone comes with none. When adding your repository to F-Droid it would not download GmsCore, but the other files. I tried to manually get the GmsCore and I get GmsCore couldn't be installed on your phone... It's a Z90-7 with 6.0.1.

Is your rom supporting signature spoofing? Only with signature spoofing you would be able to make use of microg's features.


How to update query depth after schema creation?

** Which Category is your question related to? **
API/GraphQL
** What AWS Services are you utilizing? **
GraphQL
** Provide additional details e.g. code snippets **
The default query depth when intializing a graphql schema is 2, however as my schema becomes more defined, I found myself needing to nest certain items within the schema, thus making the depth 4.  Without a way to update this, everytime I add a field or update my schema and have codegen run, some fields are ommitted in my queries.js and mutations.js files because they are nested deeper that 2.
Is there a way to update the depth so that updating my schema doesn't regenerate resolvers that require manual additions?

Yea to update this you can run
amplify codegen configure


Bug in the homepage

when you move your cursor on the header links which are home, about, and other it's displaying the background code which is not a good development.

@all-contributors please add @code1511  for bug


Open Graph metadata and og:image generation

Make sure Title and description are corrent
Autogenerated og:image using edge, see example https://github.com/gaboesquivel/gaboesquivel.com/blob/main/app/og/route.tsx
use https://www.opengraph.xyz/

https://nextjs.org/docs/app/building-your-application/optimizing/metadata


normalizeSpaces strips highlight.js formatting

Platform: Any
Mercury Parser Version: 2.1.0
Node Version (if a Node bug): N/A
Browser Version (if a browser bug): N/A

Expected Behavior
Code snippets should retain formatting
Current Behavior
Code snippets such as those created with highlight js often contain nested span tags. The function normalizeSpaces is stripping formatting information from these as it's not currently checking for nested values in pre or code tags.
Steps to Reproduce

Attempt to scrape an html document containing highlightjs code
Verify result does not retain formatting

Detailed Description
Possible Solution
I'm not entirely sure how you'd like to proceed but I have time in the next few days to submit a PR if you let me know you'd like this addressed. Otherwise I will maintain a fork for myself.

Looks like this was addressed in https://github.com/postlight/mercury-parser/commit/a250f403f5? I'm going to close my issue since it's working for me now.

I think this issue should be re-opened because <span> elements in <pre> tags still removed.
Here example page<EMAIL_ADDRESS>If you enable Mercury Parser you will not see any code examples on this page


fetchHistory returns true

Disclaimer: I'm not familiar with coffescript, in order to read the source I had to compile channel.coffee to javascript. Here is the JS file I'm interpeting this from: http://sprunge.us/GRSP
I'm hoping to be able to load a channel's history, however, I don't see any history appearing in the console (I believe this is the current behavior), nor being returned by the function, when I use:
  var h = slackChannel.fetchHistory();

  console.log(h);

From what I understand, underneath the hood, after the parameters of fetchHistory are interpreted the client performs an appropriate _apiCall.  It populates the api call's with the proper command based on where the bot is (channels.history or groups.history or im.history) and optional range params. Then we have the callback which is supposed to return the history call's data to return this._client.logger.debug(data)
Am I'm misinterpreting something or is the function incomplete?

+1
Is this possible?

@brandonhorst I don't remember how exactly I solved this issue, but a quick search on the codebas suggest that I haven't used fetchHistory in the end. I could however see that I must've moved the history fetching aspect to be a request made on the api directly, by hand. Hope this helps.

Yeah, I ended up doing the same. Thanks!


Wrong IDE platform prefix

What happened?
Starting with 2023.3.2 IDE launched by RunIdeBase task has the wrong platform prefix, it's always JetBrainsClient.
It happens because this code finds -Didea.platform.prefix=JetBrainsClient param in bin/jetbrains_client.sh file of IDE artifact and it considers JetBrainsClient as IDE platform prefix.
As a result, any checks based on platform prefix, for example, all checks in com.intellij.util.PlatformUtils, work unexpectedly for such IDE instances.
Relevant log output or stack trace
No response
Steps to reproduce
Add val platformPrefix = PlatformUtils.getPlatformPrefix() anywhere using 2023.3.2 IDE builds and check value
Gradle IntelliJ Plugin version
1.16.1
Gradle version
8.5
Operating System
macOS
Link to build, i.e. failing GitHub Action job
No response

The issue is caused by improper shell script scanning to discover the platform prefix.
This should not happen, as 2022.3 explicitly provides such value through the product-info.json file.
Added proper platform version check.


version CR: add package manage name field

IMPORTANT NOTES (please read, then delete):

Have you followed the guidelines in
Contributing Documentation?

The PR title should start with "Fix #bugnum: " (if applicable), followed by a
clear one-line present-tense summary of the changes introduced in the PR. For
example: "Fix #bugnum: Introduce the first version of the collection editor.".

Please make sure to mention "Fix #bugnum" (if applicable) somewhere in the
description of the PR. This enables GitHub to link the PR to the corresponding
bug.

Meant to approve 😅


The catch-all route breaks some umbraco endpoints

I noticed this with package saving and publishing

Fixed with 33203b8e516c590eb972463ea292835eca52580b


collect nb2workflow metadata before it is even run and also during the run

use internals of nb2rdf

I suppose you refer to this ?
So, briefly, given a notebook, this produces an RDF graph object that contains a number of annotations related to the notebook itself (ie input, outputs, parameters) using its tags.
Ideally, we want to collect those annotations before the actual execution of the noteobok.
Am I correct?

I suppose you refer to this ?
So, briefly, given a notebook, this produces an RDF graph object that contains a number of annotations related to the notebook itself (ie input, outputs, parameters) using its tags.
Ideally, we want to collect those annotations before the actual execution of the noteobok.
Am I correct?

Yes. Maybe there is another renku hook to use, or just run it after generating renku graph, adding this graph to it.

include type for each ParameterBinding
only analyze output notebook
additional command inspect , to inspect the input notebook and attach those info to the file itself, still using nb2rdf

add those info in a different place to  be added to the graph exported initially
insert also the ontology inside this graph


Store

Hi Andre!! I'm sending this pull request for you to review my code. Thanks in advance for your feedback!
Milestones 🎯:
📌 Generate store.js file with configureStore
📌 Add logger
📌 Import store & Provider in index.js file, and render App in this Provider
🎖 General requirements:
Gitflow
No linter errors
Ensure the use of best practices and Ruby style

@AndreaM2429 Many Thanks for your feedback Andre!!


"No coverage for file" for projects with coverage

I can use extension with the example project in this repo.  But I cannot with my typescript projects that I use istanbul-remap.  Here is an example project:
https://github.com/markis/strip-whitespace
Just clone that repo and run npm test then see that coverage gutters reports "No coverage for file" on my source files.

Nevermind, it was related to windows bash.  Windows bash outputs /mnt/c and vscode needs relative paths or C:\

hey @markis relative path resolution for the source files within the lcov is on by default in version 1.x of the extension so you shouldnt have had this issue... I did run the npm test on the project you specified by the info.lcov showed only the test file as having coverage?

and if I did use the display command on the test.js file it did show the lcov coverage correctly

You probably need to adjust istanbul to also provide coverage for the other files in that project. Hopefully this helps you.


GCV to HOCR or PAGE conversion not working

Hi all,
I am new to using this software so please bear with me if this has been asked before or I'm not using the tool correctly.
I have the JSON output of google vision OCR of a PDF (emphasis on PDF and not an image).
I would like to create a searchable version of that PDF using the OCR results. I have tried using gcv2hocr but it doesn't seem to work on PDFs, or it has some other error, because the HOCR output I'm getting from it is basically just the metadata. I tried using ocr-fileformat on the same file, but once again I get only the metadata as a result. Trying to convert it to PAGE fails as well, with the result being some java lines indicating exceptions have occurred. Does ocr-fileformat supports GCV JSON generated from PDF?
The file I'm trying to run it on is the sample file from google:
gs://cloud-samples-data/vision/pdf_tiff/census2010.pdf
And the JSON is generated following this tutorial:
https://cloud.google.com/vision/docs/pdf#vision_text_detection_pdf_gcs-python
If you could assist me or point me in the direction of how to solve it I would be very grateful, as I'm in an urgent need to solve this issue.
Thanks in advance!

Ok @dinosauria123 ! Thanks

Is this issue still live? I'm getting a similar error (org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 1; Content is not allowed in prolog.) when I try to convert GCV to PAGE. I'm attaching a zip file with the JPG and two versions of GCV: gcv-google-api (which was made with a Python script I wrote to interact with the Google API) and gcv-sh (which was derived from the shell script provided by @dinosauria123 at https://github.com/dinosauria123/gcv2hocr). Thank for your consideration.
gcv-sample.zip

@sarepal  I'm still having issues converting GCV to HOCR and, I could be wrong, I think the conversion to PAGE goes via HOCR. Are you using a result from TEXT_DETECTION or DOCUMENT_TEXT_DETECTION?


Download Counter

Is there any download counter in plan?

Now there is 😁

In the same run, it would be nice if the files would not be accessable directly, to be sure that files are downloaded only by clicking the link on the forum.
This would prevent my site being used as file hoster for other sites.
And between clicking on the link and downloading the file, there should be a possibility for a customized message from the website owner. To give hints that the files are scanned for viruses or something else.

Solved in the latest 0.5.* releases that use a php proxy. Downloads are still a todo, although they can be tracked now.

how to show download counter in button?
<div class="Button">
	<xsl:value-of select="@downloads"/>
</div>

not work


Add hint text for text-based components

A hint would be useful to have as a property for text-based components. Maybe something along the lines of this?

For some odd reasons the border isn't aligned properly, at least in the gif.

Because you're using a proportional font.


Draft message doesn't remain when It is empty

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to any chat
Send new message
Edit this message
Replace the message by empty string (don't click save)
Click another report
Back to previous report

Expected Result:
Draft message should be remained
Note that: When replacing draft message by another string, It will be remained
Actual Result:
Draft message doesn't remain
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.55-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/21cba4ea-a809-42bd-af7f-384443df330f
https://github.com/Expensify/App/assets/43996225/95d38886-ce30-4881-b7cb-8e3143f1d906
Expensify/Expensify Issue URL:
Issue reported by: @dukenv0307
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691551143871559
View all open jobs on GitHub

@greg-schroeder  I'm logging off now and will be OoO for the next week so reassigning this to you in the meantime.  Thanks!

Ok, sure!

Hmmmmm. I don't know about this. Why would a user replace a message with an empty string? That doesn't seem like a viable use case to me. I don't know that we should optimize for that.

Started a discussion

Team agreed it's worth moving over to External

@samh-nl I followed through with your proposal, but it could have resulted better.

The current user with a bunch of reportActionsDrafts_ saved in their storage will be affected, so I don't think changing it to undefined will be a good idea.

@samh-nl Could we use the current key reportActionsDrafts_? If possible, we should avoid adding a new key and use the associate key.

Not overdue, it still open for proposals.

Awaiting new proposals

Still awaiting proposals, might send this to CS/SWM soon

Proposal
Please re-state the problem that we are trying to solve in this issue.
Draft message doesn't remain when it's empty
What is the root cause of that problem?
We save draft while it's changing here
https://github.com/Expensify/App/blob/b57d3a3d347aff057edb0bba2a4cd65448f4d670/src/pages/home/report/ReportActionItemMessageEdit.js#L198-L203
As you can see, when draft is empty, we store the original value. So when we remount the ReportActionItemMessageEdit component, original message would appear
This is the root cause
What changes do you think we should make in order to solve the problem?
We can store the space character if draft is empty and trim it when mounting.
In the above line 202
        debouncedSaveDraft(' ');

And change the line
https://github.com/Expensify/App/blob/b57d3a3d347aff057edb0bba2a4cd65448f4d670/src/pages/home/report/ReportActionItemMessageEdit.js#L104
with
        return Str.htmlDecode(props.draftMessage.trim());

I think this change wouldn't have any side effects

Result
https://github.com/Expensify/App/assets/126839717/a94b68d9-29b4-4398-9a2e-b1c9a9c19df0

What alternative solutions did you explore? (Optional)

Bump @mollfpr for proposal review

Proposal
Please re-state the problem that we are trying to solve in this issue.
When we edit a comment, delete all chars change report and comeback the draft input is filled with the former text instead of empty string
What is the root cause of that problem?
We intentionally set the old string to the text input if it's empty because the condition for the Edit input to show up is that the draft message is not empty. We do that here: https://github.com/Expensify/App/blob/b57d3a3d347aff057edb0bba2a4cd65448f4d670/src/pages/home/report/ReportActionItemMessageEdit.js#L198-L203
What changes do you think we should make in order to solve the problem?
We should not replace empty string by the old text.
In the collection ONYXKEYS.COLLECTION.REPORT_ACTIONS_DRAFTS we currently only store strings, we should also store a boolean isEditing. We must change this line https://github.com/Expensify/App/blob/d2bdaece9151c8c65d3b4bc4c9578be663f1950d/src/ONYXKEYS.ts#L384
with this:
[ONYXKEYS.COLLECTION.REPORT_ACTIONS_DRAFTS]: Record<string, boolean>;
Of course right now the messages stored in onyx don't have this boolean, we will set it to false but for compatibility we will have to test both conditions like this:
if (props.isEditing || !props.draftMessage)
    show Edit Input

What alternative solutions did you explore? (Optional)

Will review the proposals in the morning.

Just a note: I agree with this: https://github.com/Expensify/App/issues/25488#issuecomment-1699632670 , my proposal is if a migration is difficult/impossible

@samh-nl I'm unsure if it is worth doing, and I've never seen us doing it. We can make it work by creating a new key and eventually removing the old one.
@s-alves10 That might be work, but it's clearly a workaround.

Just a note: I agree with this: https://github.com/Expensify/App/issues/25488#issuecomment-1699632670 , my proposal is if a migration is difficult/impossible

@ShogunFire Could you elaborate on this? Are your solution require a migration too?

@mollfpr
We're now treating empty string as an action, not as a draft
We're treating spaces as an empty in draft already.
So I suggested to use space as an empty draft. This doesn't require onyx migration and requires minimal changes. Also this doesn't define a new state of messages.
I don't think this is a workaround
Note: We need to distinguish empty draft and non-draft. There are two ways to do this: one suggested by @samh-nl - needs onyx migration, and one described in my proposal. I think we should not introduce new state as possible.

@mollfpr my proposal doesn't require a migration because we would still be testing the current condition: if the draft string is not empty then we show the edit mode. We would just add another condition with a new boolean isEditing, part of  the existing collection. Inspired by what you suggested here: https://github.com/Expensify/App/issues/25488#issuecomment-1688174202

Thoughts on the above @mollfpr?

I don't think this is a workaround

@s-alves10 How it's not a workaround? A whitespace is not an empty string; an empty string should have no character.

@ShogunFire How the isEditing will be set? Will changing the value types cause the error log when the draft value is only a string?

@samh-nl I believe you already have the answer to the Onyx migration.

The alternative I outlined would be to add a separate boolean value to store which are being edited, for example reportActionsEditing_. For backwards compatibility, we could set editing = true if draft is not empty (or isEditing = editing || !empty(draft)). I can work out the implementation details more if you think it may be a good approach.

Could this solution be implemented? I'm okay with adding a new key now since we don't have any clean approach.

The alternative I outlined would be to add a separate boolean value to store which are being edited, for example reportActionsEditing_. For backwards compatibility, we could set editing = true if draft is not empty (or isEditing = editing || !empty(draft)). I can work out the implementation details more if you think it may be a good approach.

@samh-nl  I'm inclined to see this as a proposal, if you will.

Please give the other option of renaming the Onyx key also some consideration, it's a bit more elegant and the trade-off (existing drafts are removed when people refresh) may not be a problem in practice.

If you can explain the clear plan, you can start a discussion in the slack regarding the trade of.

Looks like conversation continues on trying to find the right proposal

Bumped slack discussion

Not overdue. I will revisit the proposals and the slack discussion.

Slack discussion hasn't really gone anywhere. What do you want to do here @mollfpr?

Sorry for the delay 🙏

Please give the other option of renaming the Onyx key also some consideration, it's a bit more elegant and the trade-off (existing drafts are removed when people refresh) may not be a problem in practice.

@greg-schroeder I like the above suggestion better, but the trade-off might need to be conveyed to the internal.

Okay. I bumped the internal team

@samh-nl can you summarize where we ended up here?

Excellent recap, thank you. Let's hold then.

No update yet

holding

@samh-nl Looks like the issue we were holding for was finished up - do you think we're good to proceed here?

Sorry for the delay.
The proposal looks good, but I still don't know when and how the migration will be run. @samh-nl Could you elaborate on the migration step?

Bump @samh-nl!

Does that help answer your question @mollfpr?

Migrations are defined in this directory and subsequently called here

Got it! The proposal looks good to me!
🎀 👀 🎀 C+ reviewed!

This is on staging, awaiting deploy to prod

You can ignore this @sonialiap, it's a bug, I will handle payment

[@mollfpr] The PR that introduced the bug has been identified. Link to the PR:
[@mollfpr] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

No offending PR.

[@mollfpr] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression test should be good.

[@mollfpr] Determine if we should create a regression test for this bug.
[@mollfpr] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Open any report
Send a message
Press the Edit button in the newly added message
Empty the draft message input
Navigate to a different report
Return to the first report
Verify that the draft message input is empty
👍 or 👎

@greg-schroeder No need to send me the offer; I'll make a manual request in NewDot. Thank you!

reporter: $50
c: $500
c+: $500
Offers sent to @dukenv0307 and @samh-nl, and you can do a manual request @mollfpr.

filing regression test

$500 payment approved for @mollfpr (c+) based on summary above.

Sorry all, let me review this one

Hmm. This is tricky. The issue was reported/created in August before our first pricing changes, but no PR was created until late 2023, well after our payscale changed. I think we could probably honor the original reporting bonus given the date, but the work done on the PR took place well after the $500 baseline was established. That seems most fair to me.


Feature request: Adding the ablility to automaticly create collections

this will open lots of possibilities for large apps and will make Pocketbase 1 step closer to being able to be used for production, but this idea might cause a lot of problems for servers, so I suggest to making it rate limited by default and that it requires authentication methods like needing a Pocketbase's admin account password to stop DDos attacks, it also needs to have a way to make api rules changes (prevent api rules changing on admin/manual created collections by defaul) and a way to difinshiate between auto created collections and admin/manual created ones on the admin dashboard
I hope that this idea will get added in feature releases as it makes making large app easier to make

well why cant it be automated with code, that is my question

@H1Mohamed I don't understand what do you mean. The title of the issue doesn't match with the description.
If you want to create collections programmatically, please refer to https://pocketbase.io/docs/go-collections/.
Additionally, note that auto migrations are enabled by default so any change from the Admin UI will result in a migration file. This migraiton file can be safely commited in your repo and they will be executed automatically on serve.


Save Note Popup displays even when none of the note content is modified

When I tap a note and view it, then try to go back it pops up a 'you have unsaved changes' popup.
ref issue: https://github.com/SankethBK/diaryvault/issues/209

This is the code used for showing unsaved changes popup in note_create_page.dart or note_read_only_page.dart,
    return WillPopScope(
      onWillPop: () async {
        bool? result = await showCloseDialog(context);

        if (result == true) {
          notesBloc.add(RefreshNote());
          return true;
        }
        return false;
      },

NotesBloc stores the state of currently opened note. NoteInitialState is the initial state and controller parameter holds the rich text editor's internal state. It will only be transitioned to NoteUpdatedState when title, createdAt, allNoteAssets or tags is updated.  But if user types something into rich text editor without changing any of these, the state will still remain as NoteInitialState as controller takes care of managing rich text editor's state internally. So there's no way to distinguish if an actual edit has happened in such cases.
Solution:
When user clicks on back button in note_create_page.dart or note_read_only_page.dart, retrieve the note from database and compare it with the current note, if its different then only show the popup, otherwise pop the screen.
There are multiple ways for comparing two notes, easiest one can be to compare hash property of notes which will be modified if any of the content is changed. Refer saveNote method to know how hash is updated when a note is changed.
Relevant files:
- lib/features/notes/presentation/pages/note_create_page.dart
- lib/features/notes/presentation/pages/note_read_only_page.dart
- lib/features/notes/data/repositories/notes_repository.dart

Hi @SankethBK ,  I'd like to contribute to this issue. Is that okay?

Hi @malachieborohoul , sure

Great, thanks


Does not work / Breaks liveries with non in-line countries in description.lua

The program automatically comments out the line "countries = {" which works on most lua files, but on some the country names are in another line below that, which ends up breaking the livery so that it cant be chosen in-game at all.
Examples:

Here you can see where the country abbreviation is not inline with the commented out line the unlocker changes.

Example where it works as intended
The way to fix this it seems is to look for the closing curly bracket and either delete or comment out anything between. As long as the last bracket is there it wont work.

Hm...Didn't even imagine that could happen, thank you for the heads up. I'll try fix it this week. Then I'll get back to you. Thanks for the attention!


The formula should be refactored

This formula should have many more states than just build from source and I'm going to make them. Specifically I want to have twemproxy.pkg and twemproxy.config states.
So I'd like to propose my contribution here, but I don't know what to do better:

add new states and leave init.sls unchanged
rename init.sls to build.sls and make formula by standards, but in this case we lose backwards-compatibity.

Personally I prefer second choice because it doesn't look like formula has many users and packages are already available in repositories (at least for Debian), so building from sources is not a popular solution.
What'd you like me to do? Maybe there is another way to make everybody happy?

If someone would be interested I finally got time to open source my personal formula: https://github.com/genuss/twemproxy-formula

@genuss It appears I wasn't watching this repo closely, sorry for not responding back in January. I prefer the second choice as well. If you want to PR into this one that would work for me, though you might have some naming conflicts in the repos since you need to fork.

@gravyboat will do my best to make a PR as soon as I'll get some time!


Added support for OCP 4.14, removed support for OCP 4.11

Description
Added support for OCP 4.14, removed support for OCP 4.11
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Will run pipeline after the weekend, as renovate PRs will mean it will need a rebase anyway

/run pipeline

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

GitHub release
v5.9.0

Your semantic-release bot :package::rocket:


Portable version for Windows?

I'm glad to finally see releases with EXE-files, but I'll be happy with portable (7Z or ZIP) one, because I don't wanna install egram.tel, I just wanna unpack it archive and use.

That seems reasonable. I'll do that.
However you should know that the installer does nothing but extracts files into target directory.


Broken eval

Install missing package for log processing benchmark (tcpdump)
Replace dead links for media conversion benchmark

Signed-off-by: Dimitris Karnikis<EMAIL_ADDRESS>

@tammam1998  https://github.com/binpash/pash/pull/493/commits/b4065ffc5a634c1a845c85da66d97b4d1f714ed2 should fix https://github.com/binpash/pash/issues/496

@tammam1998 Do you think this PR is ready? Can we merge?


Updated pbdirect to 0.0.4

I discovered a bug related with divergin implicit resolution and consequence of this, they just fixed the bug and there is a new version.
https://github.com/btlines/pbdirect/commit/1a60b960448d76b83264198d1fbbbd91d3edf340
Could you update this number version please?
I could change the version and send a PR but I don't know what is the procedure to check that all works fine after change one specific version.

Feel free to open a PR bumping up the version. You could also remove the Snapshot suffix in the version.sbt file, in our to release a new patch version.


WEB: add some interesting stats to other parts of the Court page, like expectedRewardPerPnk, votesPerPnk, disputesPerPnk

WEB: add some interesting stats to other parts of the Court page, expectedRewardPerPnk, votesPerPnk, disputesPerPnk.
related to #1671 in the sense that the logic for calculating these values can be reused from there, since its already done there, but should be treated as a separate issue and PR


test: use T.TempDir to create temporary test directory

A testing cleanup.
This pull request replaces ioutil.TempDir with t.TempDir. We can use the T.TempDir function from the testing package to create temporary directory. The directory created by T.TempDir is automatically removed when the test and all its subtests complete.
Reference: https://pkg.go.dev/testing#T.TempDir
func TestFoo(t *testing.T) {
	// before
	tmpDir, err := ioutil.TempDir("", "")
	if err != nil {
		t.Fatalf("unexpected error %v", err)
	}
	defer os.RemoveAll(tmpDir)

	// now
	tmpDir := t.TempDir()
}

Duplicate https://github.com/distribution/distribution/pull/3767
Would you mind closing this @Juneezee ?


Publishing/Migrating Database Issue

When publishing your file...
"php artisan vendor:publish --provider="vendocrat\Followers\FollowersServiceProvider"
You need to change your FollowersServiceProvider file location to "database/migrations/create_followers_table.stub.php"
Currently it's "database/migrations/create_media_table.php.stub" and your table name is "create_followers_table.stub"

Sorry for the very late reply. I've fixed that!


Plugin directory

Just a quick question to the d3 developers: is there an official repos or listing for plugins? I was trying to add the humble comic.js to this repos but it seems dead.

Did you ever get this issue resolved?  The repo looks dead to me too, but @mbostock will know more.

Currently there is this:
https://github.com/d3/d3-plugins
As part of the 4.0 restructuring, I am probably going to break apart the plugins into lots of repos under the D3 organization here, allowing separate teams to own each plugin as desired:
https://github.com/d3
Another thing we could do is make a page on the Wiki that lists plugins, so that people individuals can host their own repositories if they don’t want to put them under the D3 organization.

I think both of those are good ideas.  Is there a way I can close this issue?

Sure. Here’s the wiki page: https://github.com/mbostock/d3/wiki/Plugins


feat(demo): aggregations, filters and limit for search

Description

Changes how the filters work on the Search page
Add new Aggregations for the count to the Search page
Adds the count from the aggregations to properties or manufacturers in Template
Use filter instead of "global" properties and manufacturer (aggregations are now updated)
If you have properties or manufacturers in the URL the filters are opened automatically
Added some select to set products (limit) per page
Added ProductCardSkeleton on Search and Category Pages

Type of change
New feature (non-breaking change which adds functionality)
ToDo's

[ ] Clean up and remove the functions from the search
[x] Integrate the same logic into category
[x] Create new e2e tests for all the new features

Additional context
Sometimes the count does not match the products the customer sees.
Currently, we have a problem getting the aggregations into CMSPage listing elements. :warning:

@patzick, @mkucmus, and @mdanilowicz I am done with that.
Also, PR is getting a bit too big. So please check and let's merge that.
:rocket:

there are some bugs in preview, I'll try to see what is the problem


Have problem with webpack-isomorphic-tools

for the Universal approach (share assets both client&server side), I wanna using webpack-isomorphic-tools.
It use global variable(e.g. global.webpackIsomorphicTools) on server-side.
first, declare on entry point.
https://github.com/halt-hammerzeit/webapp/blob/master/code/page-server/entry.js#L12
then using that var on server side render,
https://github.com/halt-hammerzeit/webapp/blob/master/code/page-server/webpage rendering.js#L19
https://github.com/halt-hammerzeit/webapp/blob/master/code/page-server/webpage rendering.js#L38
but Reduxible encapsulated it.
https://github.com/Pitzcarraldo/reduxible/blob/master/src/Reduxible.js#L47
so we need a solution for injecting optional var/method on render. lol

I added 'extras' property to Reduxible default constructor arguments. You can add anythings to Reducible by extras fields, and it will be injected with spreaded.
So if you add something to extras like below,
new Reduxible({
...
    extras: {
      assets : 'something'
    }
  });

You can get it in the container like this.
...
    const { component, store, assets } = this.props;
    console.log(asset);//print 'something'
...

And webpackIsomorphicTools.refresh() can run in reloader.
This changes deployed version '0.0.2-alpha1'.
You can test with that version, and if you have problem with it, plz comment here.
Thx :)

nice :+1: I'll test latest version soon.

Still got a problem.
instance webpackIsomorphicTools do not exist on Reduxible's initialize time.
so error occurs:  webpackIsomorphicTools is not defined.

I have tried to apply webpack isomorphic tools to reduxible applicaion.
Here is the sample of it.
https://github.com/Pitzcarraldo/reduxible-example/tree/feature/isomorphic
https://github.com/Pitzcarraldo/reduxible-example/commit/95929dde26e848643c815977efabc46c7155f31b
But this is not finished.
I will make it clear and merge to master branch.
THX


Major update

NOTE
Please verify that the base repository above has the intended destination!
Github by default opens Pull Requests against the parent of a forked repository.
If this is your personal fork and you didn't intend to open a PR for contribution
to the original project then adjust the base repository accordingly.

@geremiasanti please close this pr


added TextNote, JSON representations, Repository interface, and registered GET endpoints for Board and TextNote

added TextNote, JSON representations, Repository interface, and registered GET endpoints for Board and TextNote.
ran locally:
[GIN-debug] Environment variable PORT is undefined. Using port :8080 by default
[GIN-debug] Listening and serving HTTP on :8080
[GIN] 2022/08/14 - 21:26:20 | 200 |     223.086µs |             ::1 | GET      "/board/myShinyNewBoard"
[GIN] 2022/08/14 - 21:26:21 | 200 |      17.199µs |             ::1 | GET      "/board/myShinyNewBoard"
[GIN] 2022/08/14 - 21:26:27 | 200 |      57.802µs |             ::1 | GET      "/text_note/myShinyNewNote"
[GIN] 2022/08/14 - 21:26:28 | 200 |      21.042µs |             ::1 | GET      "/text_note/myShinyNewNote"

tested in the browser:
{"id":"myShinyNewNote","title":"title: myShinyNewNote","content":"myShinyNewNote myShinyNewNote myShinyNewNote myShinyNewNote myShinyNewNote myShinyNewNote myShinyNewNote myShinyNewNote"}

I don't see the option to approve anymore. We must be added as reviewers


Improve reliability of large restores

What does this PR change? What problem does it solve?
Restoring large files can fail in some cases, for example https://forum.restic.net/t/errors-restoring-with-restic-on-windows-server-s3/6943 . This appears to be caused by blobs that are referenced a large number of times. When streaming a pack file, it takes a long time to write all instances of that blob to disk which can cause the network connection to be closed in the meantime. If writing the blob took longer than 15 minutes, then currently no retries are performed, which causes an incomplete restore.
https://github.com/restic/restic/pull/4624 has somewhat improved the situation by ensuring that each blob in a pack file is only processed once if streamPack has to retry the download. This helps if the blob processing delay is short enough that retries still happen, but won't fix other cases.
This PR addresses the problem from a different angle. It modifies the filerestorer such that blobs that are frequently referenced are downloaded individually and thereby avoids the timeout problem.

Was the change previously discussed in an issue or on the forum?
Related to https://github.com/restic/restic/pull/4605
Fixes https://forum.restic.net/t/errors-restoring-with-restic-on-windows-server-s3/6943

Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[x] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

It modifies the filerestorer such that blobs that are frequently referenced are downloaded individually and thereby avoids the timeout problem.

Let's say there is a blob referenced 1000 times and has 1 MB. Does this mean there is now 1000 MB downloaded, whereas before only 1 MB was downloaded?

@JsBergbau No. The amount of downloaded data hardly changes. The only difference is that frequently referenced blobs are now downloaded separately from other blobs, but still only once. For example, if restore needs three blobs blob1,blob2,blob3 from a pack file and blob2 is frequently referenced, then blob2 is downloaded separately from blob1,blob3. In a few cases this can mean that blob2 is downloaded twice, but that's about it.


Repository Initialization

The idea here is to get the Excubiae project up and running with a basic boilerplate to test the configs. As a general rule, we look to the best-maintained and best-configured TS projects in PSE as guidance. Some examples include semaphore and zk-kit.

Note that CI fails because of a symlink (packages/contracts/README.md). This is a prettier bug and we can overlook it for now, as there will be no more symlink file changes to format next.


Debug Failure. Symbol parent was undefined.

TS Template added by @mjbvz
TypeScript Version: 4.1.0-dev.20200921
Search Terms

Completion
suggestions

VSCode Version: 1.49.0 (user setup)
OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:
In any *.ts or *.js file, when I try to use auto-completion for eg console by typing con and then pressing [STRG]+[SPACE] I'm not prompted with the expected console. However: once I completed typing console I get suggestions for console's members.
Looking at the "Log (Extension Host)" I see the following error:
[2020-09-16 04:49:13.048] [exthost] [error] [vscode.typescript-language-features] provider FAILED
[2020-09-16 04:49:13.049] [exthost] [error] Error: <semantic> TypeScript Server Error (4.0.2)
Debug Failure. Symbol parent was undefined. Flags: ExportDoesNotSupportDefaultModifier|Alias|AliasExcludes|ModuleMember|Classifiable|Transient. Declarations: undefined.
Error: Debug Failure. Symbol parent was undefined. Flags: ExportDoesNotSupportDefaultModifier|Alias|AliasExcludes|ModuleMember|Classifiable|Transient. Declarations: undefined.
    at getSymbolParentOrFail (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:113347:25)
    at Object.getNameForExportedSymbol (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:113340:61)
    at pushSymbol (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:116339:40)
    at c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:116323:28
    at Map.forEach (<anonymous>)
    at getSymbolsFromOtherSourceFileExports (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:116321:54)
    at getGlobalCompletions (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:116082:49)
    at tryGetGlobalSymbols (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:115993:25)
    at getCompletionData (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:115787:22)
    at Object.getCompletionsAtPosition (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:115129:34)
    at Proxy.getCompletionsAtPosition (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:143481:35)
    at IOSession.Session.getCompletions (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:154320:64)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:153204:61)
    at c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:154953:88
    at IOSession.Session.executeWithRequestId (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:154944:28)
    at IOSession.Session.executeCommand (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:154953:33)
    at IOSession.Session.onMessage (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:154977:35)
    at Interface.<anonymous> (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:157193:27)
    at Interface.emit (events.js:223:5)
    at Interface._onLine (readline.js:315:10)
    at Interface._normalWrite (readline.js:460:12)
    at Socket.ondata (readline.js:172:10)
    at Socket.emit (events.js:223:5)
    at addChunk (_stream_readable.js:309:12)
    at readableAddChunk (_stream_readable.js:290:11)
    at Socket.Readable.push (_stream_readable.js:224:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:181:23)
	at Function.create (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:1:76465)
	at h.dispatchResponse (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:1:70340)
	at h.dispatchMessage (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:1:69199)
	at c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:1:68709
	at l.fire (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:46:475)
	at g.onLengthData (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:45:60018)
	at Socket.<anonymous> (c:\Users\henning\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:45:59666)
	at Socket.emit (events.js:223:5)
	at addChunk (_stream_readable.js:309:12)
	at readableAddChunk (_stream_readable.js:290:11)
	at Socket.Readable.push (_stream_readable.js:224:10)
	at Pipe.onStreamRead (internal/stream_base_commons.js:181:23)

However: for a split second, this seems to work alright - maybe before the internal initialization of some vscode routine complete?

Does this issue occur when all extensions are disabled?: Yes

Thanks. I confirmed the crash.
Here's the stack from TypeScript 4.1.0-dev.20200921:
<semantic> TypeScript Server Error (4.1.0-dev.20200921)
Debug Failure. Symbol parent was undefined. Flags: ExportDoesNotSupportDefaultModifier|Alias|AliasExcludes|ModuleMember|Classifiable|Transient. Declarations: undefined.
Error: Debug Failure. Symbol parent was undefined. Flags: ExportDoesNotSupportDefaultModifier|Alias|AliasExcludes|ModuleMember|Classifiable|Transient. Declarations: undefined.
    at getSymbolParentOrFail (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:114958:25)
    at Object.getNameForExportedSymbol (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:114951:61)
    at pushSymbol (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:118252:40)
    at /Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:118236:28
    at Map.forEach (<anonymous>)
    at getSymbolsFromOtherSourceFileExports (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:118234:54)
    at getGlobalCompletions (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:117995:49)
    at tryGetGlobalSymbols (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:117906:25)
    at getCompletionData (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:117693:22)
    at Object.getCompletionsAtPosition (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:117013:34)
    at Proxy.getCompletionsAtPosition (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:145573:35)
    at Proxy.<anonymous> (/Users/matb/.vscode-insiders/extensions/jpoissonnier.vscode-styled-components-0.0.31/node_modules/typescript-template-language-service-decorator/lib/template-language-service-decorator.js:84:24)
    at IOSession.Session.getCompletions (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:156424:64)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:155308:61)
    at /Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:157058:88
    at IOSession.Session.executeWithRequestId (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:157049:28)
    at IOSession.Session.executeCommand (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:157058:33)
    at IOSession.Session.onMessage (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:157082:35)
    at Interface.<anonymous> (/Users/matb/projects/san/vscode-autocomplete-error/node_modules/typescript/lib/tsserver.js:159298:27)
    at Interface.emit (events.js:223:5)
    at Interface._onLine (readline.js:315:10)

It seems this issue https://github.com/microsoft/TypeScript/issues/40355 is related. I a little bit investigated it and ran into this crash. I used the following test to reproduce the issue
/// <reference path='fourslash.ts'/>

// @esModuleInterop: true
// @moduleResolution: node
// @target: es2015
// @module: esnext

// @Filename<EMAIL_ADDRESS>//// export = Foo;
//// declare class Foo {}
//// declare namespace Foo {}  // class/namespace declaration causes the issue

// @Filename: /node_modules/foo/index.d.ts
//// import * as Foo from "@bar/foo";
//// export = Foo;

// @Filename: /index.ts
//// import Foo from "foo";
//// /**/

goTo.file("/index.ts");
verify.completions({
    marker: "",
    includes: ["Foo"],
    preferences: {
        includeCompletionsForModuleExports: true
    }
});

At first glance, the crash occurs because of resolvedModuleSymbol returns symbol without a parent, which is required in getSymbolParentOrFail.
https://github.com/microsoft/TypeScript/blob/7c6462aa10329884618c51ff9af76c66b503f47c/src/services/completions.ts#L1658
https://github.com/microsoft/TypeScript/blob/7c6462aa10329884618c51ff9af76c66b503f47c/src/services/completions.ts#L1727
https://github.com/microsoft/TypeScript/blob/7c6462aa10329884618c51ff9af76c66b503f47c/src/services/utilities.ts#L2879
cc @DanielRosenwasser @sheetalkamat

I can confirm I'm having the same problem with TypeScript v.4.0.x in my VSCode. I have to stick to v.3.9.7 for now unless there's a workaround?

Hmm - but that doesn't seem to work for me. I changed in the package.json the typescript to "^3.9.7" and then set the typescript-sdk to the local workspace. But still I get
[2020-09-24 10:01:54.221] [exthost] [error] [vscode.typescript-language-features] provider FAILED
[2020-09-24 10:01:54.222] [exthost] [error] Error: <semantic> TypeScript Server Error (3.9.7)
Debug Failure.
Error: Debug Failure.
    at Object.getNameForExportedSymbol (C:\projects\playground\vscode-ts-error\broken\node_modules\.pnpm\typescript@3.9.7\node_modules\typescript\lib\tsserver.js:110423:70)

Hmm - but that doesn't seem to work for me. I changed in the package.json the typescript to "^3.9.7" and then set the typescript-sdk to the local workspace. But still I get

Hmm... interesting, 3.9.7 works for me, it doesn't throw that error and the suggestions work. Interestingly, when I tried a fresh project (I use Next.js), there was no error at all with 4.0.2, so I'm suspecting one of the dependencies in my project may be causing that issue...

maybe - I did upload a sample-project, which shows this error ...

The problem is in "esModuleInterop": true I try with all TypeScript versions but all with same error.
Disabling esModuleInterop all work fine

I'm having the same problem with a nextJS project. When i disable esModuleInterop it works.


Live is not in talos

Hi!
I am having this problem
from talos import live
is returning an error as well.
The solution you proposed in another issue (from talos.utils import live) or (talos.utils.live()) is not working anymore.
Please, can you tell me how to solve this issue?
Also, this import
from talos.metrics.keras_metrics import fmeasure_acc
is returning an error.
Thanks for your help!

You can just directly use the callback from talos.callbacks.TrainingPlot(). Closing here, feel free to open new issue if anything.