Improve compatibility with .so files produced by perf inject --jit

Fixes #248.
In addition to the missing unit ranges, there was a bug where we weren't able to find the file name that was added during line rows iteration, and we were creating an absolute path instead of a relative path.

I considered adding a test for this by dropping the 1KB test file into the fixtures directory, but I couldn't find a good place to add the test without creating a bunch of new infrastructure. Most of the tests are rather self-referential. Only the parse.rs tests make use of a frozen file (fixtures/addr2line-release), and those don't check for correctness, only for non-panic.

I could rename correctness.rs to correctness_self.rs and add a new correctness_frozen.rs. How does that sound? Or maybe _snapshot or _fixtures.

The current correctness.rs is basically checking that we can still handle whatever rustc generates.
We do need to add regression tests though (both for this and ideally for all the previous bugs that have been fixed). Perhaps regression.rs would be suitable for that, but I don't have any preference. If we were to regression test properly, we could potentially end up with a largish number of binary files. I'm not sure that we want to put them in this repository. For object, I created the object-testfiles repository (which we could reuse); I'm not sure that's a good solution either, since having to update both of them is cumbersome. Open to any suggestions here.


webview_flutter memory leak

Originally posted by @nuptdzs in https://github.com/flutter/flutter/issues/24714#issuecomment-567408846

with IOS ,I use webview_flutter plugin,I repeatedly enter and exit this page .The memory will grow all the time, and in the leaks tool of Xcode, we can see that a variable is not released after exiting, as shown in the figure below, it seems that some other changes to GL cause this problem.

[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.14.6 18G87, locale zh-Hans-CN)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.38.1)
[!] Proxy Configuration
    ! NO_PROXY is not set
[✓] Connected device (1 available)

Can you try to analyze this problem, and look forward to your answer. This problem will cause our application memory to grow until it crashes, hoping to give us a solution.

@nuptdzs can you please share a minimal code sample that reproduces the issue?
cc @cyanglaz

I also encountered the same problem，See the previous solution https://github.com/flutter/flutter/commit/93f339fb67df2934e618cfca0f9d5bd7982e4bcd , flutter version is v1.12.13+hotfix.5.
But the problem still exists

This has been open for 7 months -- am I correct in assuming platform views in flutter have been causing memory leaks for the last seven months?

Tagging all Webview team: @blasten @TahaTesser @chinmaygarde @jason-simmons Do this issue is fixed ?

Hi @amirh, can you please confirm if you're still experiencing this issue on the latest flutter stable and webview_flutter version?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


wording and exit code

Fixes for #1212:

Changes phrasing of the confirmation prompt when removing a wg config (from "[Y/n]" to "[y/N]")
Prints to stdout that operation is aborted
Returns exit code 1

LGTM, Merging into test.


Member of team owner cannot create a new repository when also member of a non privileged team via LDAP

Description
If a user happens to be a member of both,

the team "owner" of an organization "org" and additionally
a non-privileged  team "non-priv" of the same org that gets populated via LDAP. This team does not have the flag "can-create-repository" checked.

Then the user can choose "org" when trying to create a repo. But they are prevented from actually creating the repository. Instead of a view of the newly created repo they get the word "forbidden" on an otherwise empty page. See screenshot.
Expected behavior: The user can create the repositories in the organization org. They are member of team owner, after all.
Since the bug involves group data pulled from an LDAP data base, I was not able to reproduce the bug on the demo site.
Excerpt of our gitea.log (same as gist)
Gitea Version
1.20.2
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/KaiMartin/92f92b85f224c94ecf2ef2dabbfaac50
Screenshots

Git Version
2.39.2
Operating System
debian 12 -  bookworm/stable
How are you running Gitea?
Got gitea from  https://dl.gitea.io ( https://dl.gitea.io/gitea/1.20.2/gitea-1.20.2-linux-amd64 )
We run gitea from systemd:
systemd --version
systemd 252 (252.12-1~deb12u1)
+PAM +AUDIT +SELINUX +APPARMOR +IMA +SMACK +SECCOMP +GCRYPT -GNUTLS +OPENSSL +ACL +BLKID +CURL +ELFUTILS +FIDO2 +IDN2 -IDN +IPTC +KMOD +LIBCRYPTSETUP +LIBFDISK +PCRE2 -PWQUALITY +P11KIT +QRENCODE +TPM2 +BZIP2 +LZ4 +XZ +ZLIB +ZSTD -BPF_FRAMEWORK -XKBCOMMON +UTMP +SYSVINIT default-hierarchy=unified
Database
PostgreSQL

I haven't looked into my logs, but see the same symptoms. I also run on Bookworm + Pg.


Replaced raw-loader with css-loader to resolve url dependencies

I'm not sure if this is optimal for a production deployment but it resolves the glyphicons 404 issue.

Excellent! Yup that'll do it, nice job! 👍

I noticed those in my fork but I removed them from the csproj before the
pull request. I can't explain why the duplication occurs.
On 9 Apr. 2017 8:14 pm, "Boris Prpic"<EMAIL_ADDRESS>wrote:

@hheexx commented on this pull request.
In Asp2017.csproj
https://github.com/MarkPieszak/aspnetcore-angular2-universal/pull/164#discussion_r110537698
:

@@ -25,6 +25,24 @@

wtf?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MarkPieszak/aspnetcore-angular2-universal/pull/164#pullrequestreview-31720375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8zNcj3qRba9q4b_2ukTXxSJtmni8QDks5ruK-KgaJpZM4M34eA
.


Create maithrivh.json

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

i followed along eddie's YT content and created this pull request.

hey @maithrivh did you mean to close the pull request?


Support iOS 8.x

I made some tests running the lib with iOS 8.x and seems fine.
Evidence:
https://www.dropbox.com/s/5rcypvw74qdpn1f/ios8.mov

Show https://github.com/krimpedance/KRPullLoader/issues/3#issuecomment-334629631


Normalize the names returned from bank d and grand

I did not put the shared method to Common since this is a PoC with ToTitleCase with only one row, otherwise it creates to much over code to instantiate the helper based on Options.

Following up on the discussion in https://github.com/ActiveLogin/ActiveLogin.Authentication/issues/20-
I'm not really sure if we should actually implement this. From your testing you said there are quite a few scenarios not handled right with the implementation we found. I'd say it might actually cause more issues than it solves.
If we were to merge it we'd need to add test to show what we support and what we don't.
But, if not at least 99% of names works, we should maybe skip it?

I agree... would be nice to have a poll or some another way to ask community what would they wish

I’d vote for writing a small doc under FAQ instead. So if someone wants to normalize we can show that code is a good start. Then we populate the claims with raw values but a consumer can normalize later.

We close this one and use the documentation implemented in #37 instead.


Ordering the Board on PCBWay

I am new to ordering PCB's or even assembling them. I want to upgrade my PSVita with the Psvsd Board but sadly it's sold out on Indiegogo :(
I asked you already on Twitter where I can order the Board assembled and you told me to look on PCBWay but sadly I have no clue what to fill in or how to produce the Board. Can you give me a Screennshot of what you filled in ? / share the Project via Link if possible ? (example)

Download both files from https://github.com/yifanlu/psvsd/releases/tag/v2.1.1
Go here https://www.pcbway.com/pcb-assembly.html "Quantity" (I recommend > 100 or it'll be REALLY expensive per unit) "Number of Unique Parts" = 11 "Number of SMT Parts" = 11 "Number of Thru-Holes Parts" = 0. Click calculate.
Now change "Assembly Side(s)" to Double sides. Click "PCB Specifications".
Change Quantity to the same quantity specified earlier. "Size (single)" = 30x33.2mm "Thickness" = 1.0 and leave other options as is.
Press calculate and add to cart. You will be asked to upload gerbers for the PCB (pass psvsd-v21-1.zip as is) and to upload the BOM (extract psvsd-v21-1-assembly.zip and pass in the xlsx).
Wait a day for them to send you the proper quote. Make sure to tell them to ensure that the GL823 part is the 3.3V version! This is important, they almost messed up the first time.

It will cost ~$1400 for 500 pieces. ~$700 for 100 pieces. ~$100 for 10 pieces and so on so it's cheaper to buy more at once.

thanks for the Answer but for a normal Person it's way too expensive (the assemblie) :(


Sql generation improvements

select it.Name from Customer as it
instead of
select it.[Name] as [Name] from Customer as it
Two points:
#1, as [Name] is not necessary in this scenario.
#2, it.[Name] should be it.Name, since Name is not a keyword, I think it's better to maintain a keywords list and only use [] for keywords.
This will decrease the generated sql and much readable.

How do you detemine "Name" is not a keyword in all different supported databases?

Like I said, the keywords list should be maintained, and when new database version comes out, we updated the keywords list with updated provider version.

Sound like a helluva lot of work for very little gain to me.

Is there any value in not escaping the names? I don't think it would make any difference to the plan compiler in SQL Server. I think you could also argue that it's easier to read if everything is enclosed in [].
Avoiding aliasing with the same name sounds like a reasonable simplification.

Thanks, maybe I was wrong about the alias without [], I guess it's not worth to do this.
So my only question is the #1.

@ryanlin1986 can you share a code listing that demonstrates where you are seeing the unneeded aliasing happening? In general we don't do this, but maybe you have found a case where we do.
For example, here is a query we generate from RC1-final
SELECT [b].[BlogId], [b].[Created], [b].[Url]
FROM [Blog] AS [b]
ORDER BY [b].[Url]

@rowanmiller  RC1-final is just fine, maintain the keywords list is not worth it, since RC-1 does this well, I close this now.


Clarify How Endpoint Monitor Requests are Made

It's not clear on this page how the endpoint monitor requests are made, exactly - and we're dealing with a number of different possibilities. Does it use the IP address, the Traffic Manager hostname, the CNAME hostname if aliased via Azure DNS, the Azure resource hostname (*.azurewebsites.net / *.cloudapp.azure.com), or something else?
For resources where IIS is set up within the VM, we need to know which hostname the monitor agent is using for requests because, for example, if the IP address is used and this is not bound in IIS, the endpoint monitor will fail even though the resource is not actually down.
Can this page be updated with some examples to show how exactly the HTTP(S) requests are made, including what hostname(s) are sent in the request?

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

ID: 9fed131c-15d3-3266-3570-08e445eda1b8
Version Independent ID: fe645535-673e-4c59-0163-be55107b0e58
Content: Azure Traffic Manager endpoint monitoring
Content Source: articles/traffic-manager/traffic-manager-monitoring.md
Service: traffic-manager
GitHub Login: @KumudD
Microsoft Alias: kumud

@qJake , Thank you for your feedback. By default, if you haven't configured hostname, TM will send the hostname with its probing IP address. If you add Web App then the probe would fail, and hence you need to configure custom header with host:*.azurewebsites.com .
Hope that answer's your question.

Yes @msrini-MSFT, thank you! Can this be added to the documentation? I think this is really valuable info!

@KumudD , can you make this documented ?

This is already documented at https://docs.microsoft.com/en-us/azure/traffic-manager/traffic-manager-faqs#what-host-header-do-endpoint-health-checks-use

#please-close


Unable to load web image

Found an issue or bug with electron-vue? Tell me all about it!
Questions regarding how to use electron or vue are likely to be closed as they are not direct issues with this boilerplate. Please seek solutions from official documentation or their respective communities.
Describe the issue / bug.
#  Unable to load web image
How can I reproduce this problem?
#  vue-cli New project
If visual, provide a screenshot.
#
Tell me about your development environment.

Node version:  v14.4.0
NPM version:   6.14.5
vue-cli version: (if necessary) 2.9.6
Operating System: window

If you are looking to suggest an enhancement or feature, then feel free to remove everything above.

this project is dead , dont use it


Ma/update runit

Description
Update to newest runit and enterprise-chef-common cookbooks. Necessary to be able to move to Chef 15. Still a bit broken, work in progress.
Built on top of ma/remove_drbd, because the new enterprise_chef_common cookbook removes some drbd/keepalived logic
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]
Check List

[ ] New functionality includes tests
[x] All tests pass
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Running in CI as https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/190


Swagger docs validate failed when enums_as_ints=true

Problem statement
Here is the
https://github.com/hnlq715/proto/actions/runs/4052643492/jobs/6972309198#step:5:41
Please remove the sections that don't apply
Swagger specification
https://github.com/hnlq715/proto/blob/main/gen/go/api/helloworld/v1/helloworld.swagger.json
Steps to reproduce
Details can be found in this repo https://github.com/hnlq715/proto
Environment
swagger version:   v0.30.4
go version: v1.19.5
OS:   macos

The swagger spec at "gen/go/api/helloworld/v1/helloworld.swagger.json" is invalid against swagger specification 2.0.
See errors below:
- in operation "GreeterService_SayHello", default value in response 200 does not validate its schema
- 200.data.sex.default in body must be of type integer: "string"
- 200.data.sex.default in body should be one of [1 2]
- in operation "GreeterService_SayHello2", default value in response 200 does not validate its schema
- definitions.v1Sex.default in body must be of type integer: "string"
- definitions.v1Sex.default in body should be one of [1 2]

Everything works fine when generating swagger doc with enums_as_ints=false
https://github.com/hnlq715/proto/actions/runs/4052618200/jobs/6972254556

Here is the PR details: https://github.com/hnlq715/proto/pull/3/files

Emm, seems like this issue should be moved to https://github.com/grpc-ecosystem/grpc-gateway

Closing as unrelated to this project


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2349 ms

Resolved: Samsung performance has improved in 828d824.


How to use ReportingServicesTools's command after installing it?

Hi,
With you advice, I installed ReportingServicesTools via Power Shell, but I don't know how to use it.
Do you have a document?
Sorry, I am new for using PowerShell.
Best regards.

Checkout the list of commands https://github.com/Microsoft/ReportingServicesTools/issues/165 , you can use Get-Help and the name of the command


Flyclient

"I'm submitting a ... "

[ ] feature request

Check out @benediktbuenz’s Tweet: https://twitter.com/benediktbuenz/status/1101241113262776321?s=09
What do you guys think of better version of SPV clients known as flyclient which is super light client for all PoW blockchains. One only needs to download O(log(n)) block headers. Please read the paper I've attached it's already working in two projects.
"In this paper, we introduce Flyclient, a novel transaction verification protocol for light clients that is efficient both asymptotically and practically. Our protocol requires to download only a logarithmic number of block headers to synchronize and verify transactions while storing only a single block header between executions. We formally prove that Flyclient is optimal for this class of protocols. For Ethereum, our protocol achieves a proof size of only less than 500 KB. This is achieved by utilizing a simple design based on Merkle Mountain Range (MMR) commitments and a probabilistic block sampling protocol. Flyclient overcomes the limitations of NIPoPoWs and generates shorter proofs over all measured parameters. We also discuss how Flyclient can be implemented via a soft fork in Bitcoin/Ethereum."

We really appreciate your contribution, but unfortunately those clients are independent from our current codebase (afaik). In the end they would result in a completely new client, which has to be developed and maintained. Thus I think it would make more sense to first establish a client with some developers as we don’t have any open resources and then we can talk about a possible integration into our codebase (if any changes are needed)

Thanks & ya you are right it is independendent of verge codebase but Afaik
it can be soft forked into verge codebase as told by its dev. There are
couple of projects who did so and saw the results which is impressive as
their flyclients are already live. In the meantime if you find some time
please give the paper a read.
On Fri, Mar 1, 2019, 12:12 marpme<EMAIL_ADDRESS>wrote:

We really appreciate your contribution, but unfortunately those clients
are independent from our current codebase (afaik). In the end they would
result in a completely new client, which has to be developed and
maintained. Thus I think it would make more sense to first establish a
client with some developers as we don’t have any open resources and then we
can talk about a possible integration into our codebase (if any changes are
needed)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/issues/884#issuecomment-468561297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjzlvGwh_wAJ_h4YiKnxtt0pXB_QyQv6ks5vSMvAgaJpZM4bYNI6
.

Let's keep a note of the same and don't miss this... Looks a value addition
to us if gets integrated, whilst am continous about our developers time as
well.
On Fri, 1 Mar 2019 17:51 karanpatel863<EMAIL_ADDRESS>wrote:

Thanks & ya you are right it is independendent of verge codebase but Afaik
it can be soft forked into verge codebase as told by its dev. There are
couple of projects who did so and saw the results which is impressive as
their flyclients are already live. In the meantime if you find some time
please give the paper a read.
On Fri, Mar 1, 2019, 12:12 marpme<EMAIL_ADDRESS>wrote:

We really appreciate your contribution, but unfortunately those clients
are independent from our current codebase (afaik). In the end they would
result in a completely new client, which has to be developed and
maintained. Thus I think it would make more sense to first establish a
client with some developers as we don’t have any open resources and then
we
can talk about a possible integration into our codebase (if any changes
are
needed)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/vergecurrency/VERGE/issues/884#issuecomment-468561297>,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AjzlvGwh_wAJ_h4YiKnxtt0pXB_QyQv6ks5vSMvAgaJpZM4bYNI6
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/issues/884#issuecomment-468563181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJWN2obv3jPgp3lODixU1-tasAMURUwVks5vSM4JgaJpZM4bYNI6
.

Seems like interest is gone unless someone wants to actively develop this

also we have the option of adding electrum server to the core, which could be a great value to our existing spv wallets


Add kluster specific user-agent to openstack api calls

Still need to test with older versions if flags are already available:

[ ] 1.20 earliest csi deployment
[ ] 1.13 earliest openstack cloud controller manager

TODO: exclude old cluster versions in helm template.


parse Province/Country fields when aplicable.

Some countries (China,UK) report some provinces or territories in additional rows using province field.
The parser and data extraction functions need support for that.

Multiple rows summation done here https://github.com/dgabi/rona/commit/7b6d77e22088a3ef8a2f7acb53080777e29f5070
Filtering by both country/province not implemented yet


Code inspection

Add code inspection using Qodana, This will fix #33

Should we add a qodana.yml fil for configuration? Like threashold for errors?
https://www.jetbrains.com/help/qodana/qodana-yaml.html#Set+up+a+profile

That sounds like a good idea!, So you mean like different profiles have different thresholds for errors? For example: profile 1 has a error threshold of 2 and profile 2 have a error threshold of 1


Link progressbar, range, label and textinput in demo

I saw you did it in one of your branches @fabrixxm . Would it be possible for you to make a PR ? Let me know if you need help :)

I'll wait for pr #27 to be merged in master.

no problem ;)

PR #29

Done in #30


Native image build fails (JDK 11, windows-latest)

Just noticed that the native image build fails since yesterday. Only JDK 11 / Windows seems to be affected.
Execution failed for task ':nativeImage'.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
> Couldn't find an installation of Windows SDK 7.1 suitable for GraalVM.

There is already some custom logic for the nativeImage test build in https://github.com/casid/jte/blob/master/test/jte-runtime-cp-test-gradle-convention/build.gradle#L63
(I'm assuming this is the one that is being hit).
You shouldn't need to remove JDK 11 builds for the whole project.
I should revisit this: GraalVM has dropped JDK 8 support and picked up JDK 17 support.

Thanks for looking into it! I'll reopen this then.

So this is probably relevant https://github.com/actions/virtual-environments/issues/4856
I'll test with windows-2019

windows-2019 passes.
So there's an incompatibility with windows-2022, either with GraalVM itself or the Palantir Graal gradle plugin.

Looks like the Palantir plugin checks for VS version 2019 or 2017
https://github.com/palantir/gradle-graal/blob/develop/src/main/java/com/palantir/gradle/graal/GraalExtension.java
(I basically never use Windows myself so my appetite for trying to work around this is low. I think that Graal native images are usually going to be used in Docker anyway.)

Thanks for looking into it! Totally agree with you, I'm closing this now, as windows-2019 is a good workaround for the CI build.


Upgrade Bootstrap to version 5

Currently the Theme is using Bootstrap 4 and it would be good to upgrade to version 5.
Split from #252 / #250

Bootstrap 5 did remove jquery as a dependency. Not sure how broken that would be to upgrade, or if it would be a mostly drop-in replacement given our minimal bootstrap usage. https://getbootstrap.com/docs/5.2/migration/
Originally posted by @vsalvino in https://github.com/wagtail/sphinx_wagtail_theme/issues/252#issuecomment-1460813826

It may be worth considering a small clean up of unused styles in the process. See https://github.com/wagtail/sphinx_wagtail_theme/issues/64

it used Bootstrap CDN ??

It's a front-end dependency see https://github.com/wagtail/sphinx_wagtail_theme/blob/main/package.json#L23

i can fix it

Just a thought. Is it worth looking at something more lightweight instead of Bootstrap.
Maybe too lightweight - https://open-props.style/ - but super flexible with CSS Custom Properties as the core method of designs.
Not sure on other options, maybe we need to review the actual Bootstrap parts used.
Also need to consider the ability to customise and override things for others.

As part of #299, I have started splitting the Bootstrap import into per-component imports, so we can start to remove the parts of Bootstrap which we don’t use. There are a lot of low-hanging fruits there I assume, though definitely also a case for just rebuilding it all together with another CSS framework.

The theme is pretty simple, I would argue to not use any framework. Bootstrap could probably be replaced with 100 lines of vanilla CSS at this point (i.e. replicate/copy the few classes we use from Bootstrap - containers, rows, collapse, button). Much less maintenance will be required this way. Most of the theme is already using custom CSS and very few Bootstrap classes anyhow.
Using a JS build framework is also probably unnecessary, as it is contained in a single file with 1 import - autocompleter could probably be imported a using normal script tag.
That setup would be mostly "set it and forget it" and also make it easier for juniors/newbies/python devs to help maintain and fix bugs.


Document pages goes blank out after chrome updates

Attach (recommended) or Link to PDF file here:
You can use the official Modern browsers PDF example to reproduce this bug:
https://mozilla.github.io/pdf.js/web/viewer.html
Configuration:
- Web browser and its version:  I was testing in multiple browsers(Brave, Firefox, Chrome) and I only was able to reproduce on Chrome after this version: 124.0.6367.92 that was released at April 26, 2024, currently I am using the most recent version: 125.0.6422.61 and the issue is still there you can replicate.
One point that would be great to add here, I downgraded the chrome version to use this one: 123.0.6312.12 that was released at April 12, 2024 and doing the same steps I was not able to replicate the issue.
- Operating system and its version: macOS, Sonoma 14.4.1
- PDF.js version: In my local project I am using this version: "pdfjs-dist": "3.8.162", but we can replicate it here: https://mozilla.github.io/pdf.js/web/viewer.html as well, don't really know which version are you using
- Is a browser extension: No, it's a NPM package
Steps to reproduce the problem:

Open the official modern browser example: https://mozilla.github.io/pdf.js/web/viewer.html
Open a new tap with whatever content.
Swap between the taps a couple of times and after that keep the page that is not the PDF document selected
After a couple of minutes(Could be more than 3 minutes) try to open again the tap where is the document
If you are lucky you will able to reproduce it

What is the expected behavior? (add screenshot)
You should see how the Document pages goes blank out and after we scroll down and then up the document page is back.
https://github.com/mozilla/pdf.js/assets/146257340/1ba6a0cc-134e-4a3e-b297-74c8c6b84bf5
https://github.com/mozilla/pdf.js/assets/146257340/bec839a6-3102-40bd-86d9-1f5cecc80729
What went wrong? (add screenshot)
Document pages goes blank out

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): Not able to share my internal project, but you can try with this: https://mozilla.github.io/pdf.js/web/viewer.html

Duplicate of #18100


x/mobile/cmd/gomobile: gomobile bind fails a package importing gl

What version of Go are you using (go version)?
go version go1.10 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/hajimehoshi/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/hajimehoshi/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/7t/qw3np69559591s1v0mk5_p1m0000gn/T/go-build050927311=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
$ cat $GOPATH/src/github.com/hajimehoshi/gomobilebindtest/test.go
// +build android

package gomobilebindtest

import (
        _ "golang.org/x/mobile/gl"
)

func Foo() {}

$ gomobile bind -v github.com/hajimehoshi/gomobilebindtest

What did you expect to see?
Building succeeds.
What did you see instead?
Building fails with this error:
type-checking package "github.com/hajimehoshi/gomobilebindtest" failed (/Users/hajimehoshi/go/src/github.com/hajimehoshi/gomobilebindtest/test.go:6:4: could not import golang.org/x/mobile/gl (type-checking package "golang.org/x/mobile/gl" failed (/Users/hajimehoshi/go/src/golang.org/x/mobile/gl/gl.go:20:12: undeclared name: context)))
can't load package: package gobind: cannot find package "gobind" in any of:
        /usr/local/go/src/gobind (from $GOROOT)
        /var/folders/7t/qw3np69559591s1v0mk5_p1m0000gn/T/gomobile-work-299819452/src/gobind (from $GOPATH)
        /Users/hajimehoshi/go/src/gobind
gomobile: go build -v -buildmode=c-shared -o=/var/folders/7t/qw3np69559591s1v0mk5_p1m0000gn/T/gomobile-work-299819452/android/src/main/jniLibs/armeabi-v7a/libgojni.so gobind failed: exit status 1

CC @eliasnaur

Yeah I did go install so that I'm using the latest gomobile binaries.

Can you test CL 108336?

Yeah that worked. Thanks!


The Witcher 1 incorrect mouse input on 4K

When running the Witcher 1 on 4K resolution, the mouse input doesn't work correcty in the main menu. When you hover the mouse over a menu item, it doesn't get highlighted and you cannot click on it. Same issue is not present on 1080p. Same issue also not present when running the game with Gallium Nine.
Software information
The Witcher 1 - latest version as downloaded from GOG
System information

GPU: AMD RX 570
Driver: RADV, mesa 19.0.3
Wine version: wine-4.7-1.fc30.x86_64
D9VK version: https://git.froggi.es/joshua/d9vk/pipelines/381/builds

Can't reproduce. Played for a short while and never got any issues here.
Gonna close this... Feel free to re-open if you have more details you can help me repro with, but it's working fine here.


add methods for message updating

Add:

EditMessageCaption
EditMessageReplyMarkup
EditMessageText

See https://core.telegram.org/bots/api#updating-messages

Current coverage is 76.57% (diff: 24.32%)

Merging #6 into master will increase coverage by 0.01%

@@             master         #6   diff @@
==========================================
  Files            20         20
  Lines           384        397    +13
  Methods         383        395    +12
  Messages          0          0
  Branches          1          2     +1
==========================================
+ Hits            294        304    +10
- Misses           90         93     +3
  Partials          0          0

Powered by Codecov. Last update 2bfc7bc...b6ae662


fix: sort not working

Current coverage is 91.80% (diff: 100%)

Merging #657 into master will not change coverage

@@             master       #657   diff @@
==========================================
  Files           179        179
  Lines          1988       1988
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1825       1825
  Misses          163        163
  Partials          0          0

Powered by Codecov. Last update 20eea5c...731475a


[CC] No documentation on how to disable tf lite

--define=lite=0 doesn't work
build still fails with
ERROR: /home/eights/Projects/minigo/cc/dual_net/BUILD:122:1: C++ compilation of rule '//cc/dual_net:lite_dual_net' failed (Exit 1)
In file included from cc/dual_net/lite_dual_net.cc:15:0:
./cc/dual_net/lite_dual_net.h:25:10: fatal error: tensorflow/contrib/lite/context.h: No such file or directory
 #include "tensorflow/contrib/lite/context.h"
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I was running the test command
bazel test --define=board_size=9 cc/...  &&  bazel test cc/...
for reference

thinking about this at a reasonable hour I guess you need tflite to test the tflite code :(

We can do some stuff with a Bazel select I think, e.g.:
minigo_cc_library(
    name = "lite_dual_net",
    srcs = select({
        ":enable_remote": ["lite_dual_net.cc"],
        "//conditions:default": [],
    }),
    hdrs = select({
        ":enable_remote": ["lite_dual_net.h"],
        "//conditions:default": [],
    }),
    deps = [ ...etc... ],
)

And maybe wrap the select up in a function so it's not so obnoxious.


Single declaration doesn't work without semicolon

It seems to only work when you add an additional space at the end.

@cqliu1 Thanks for opening this. Can you confirm the screenshot is from version 2.1.0? Have added a test case here to reproduce and it is passing.

@buildbreakdo Yep, I'm using version 2.1.0. It might be an issue with our implementation. Let me take a look.

Btw this is the exact string being passed into Style.it: ".canvasPage {\n  color: red\n}"

@cqliu1 Cool, thanks, I'll try that one and see if I can reproduce.

@cqliu1 Yes, that failed, thanks! Very interesting.. should have a fix in today. Thanks so much for opening this!

@buildbreakdo Great! Thanks so much!

@cqliu1 Okay this is fixed in 2.1.1, test to confirm is here: https://github.com/buildbreakdo/style-it/blob/master/tests/Style-24.js
Thanks for the assist with this!

@buildbreakdo you rock!
Also, I just learned this and thought I'd share; |[\n\t\s]+ is redundant, \s includes \n and \t, so all you need is |[\s]+. I changed it locally and the test still passes. No real need to change it, of course.

@w33ble Nice catch! Can drop the character class too after reducing to |\s+. Had |[\n\t ]+ and never was fond of the space character in the middle of a regex (bit too camouflaged) and switched to \s. Redundancy removed in 2.1.2

I just sent you a message using the email in your commit history.


Adding test for OverrideTabSelection and renamed the listener

Added tests
Added remove method for listeners instead of having to pass null
Changed signature of add listener methods to be @NonNull

Coverage increased (+8.7%) to 66.304% when pulling 34311350ba541c7a971232e68a4bbff02ea50bf0 on yombunker/adding_test_for_override_tab_selection into a2bd1f661ab8476acf5648ecc8d41690e8919cc5 on master.

@roughike i applied the changes you requested.

Coverage increased (+8.7%) to 66.436% when pulling d301dba198b795087f9cc6dbf3159c39a7719593 on yombunker/adding_test_for_override_tab_selection into f464d65c40dbb154cd0c6206f81cf077180affc7 on master.

Coverage increased (+8.7%) to 66.436% when pulling d301dba198b795087f9cc6dbf3159c39a7719593 on yombunker/adding_test_for_override_tab_selection into f464d65c40dbb154cd0c6206f81cf077180affc7 on master.

+1 for TabSelectionInterceptor

@mikecole20 this code is in, you can expect it on the next release which would most likely be tomorrow dues @roughike time zone :)


NATS Message Connector

http://nats.io
I might have a go at this myself.

+1 for NATS. Given their performance claims over kafka, would be interesting to see how much DeepStream performance improves over other connectors (if any benchmarks exists with, say, kafka with deepstream), especially in the pubsub and rpc side where persistence is not much required.

Awesome!
Theres a message template that should take u 50% of the way there
https://github.com/deepstreamIO/deepstream.io-msg-connector-template

I have implemented a minimal NATS connector for deepstream based on the msg-connector template. All the basic tests that come with the connector are passing.
https://github.com/UltraNemesis/deepstream.io-msg-nats

I'm keeping this open because I want to see if I can find a volunteer that can set it up. Really like what I read about NATs and would be interesting to see how it plays with our performance framework

https://github.com/nxtedition/deepstream.io-msg-nats


New option for XH.reloadApp to reload specific app path.

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[X] Caught up with develop branch as of last change.
[X] Added CHANGELOG entry, or determined not required.
[X] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[X] Updated doc comments / prop-types, or determined not required.
[X] Reviewed and tested on Mobile, or determined not required.
[X] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Looks good to me!
Only thing I can think of is perhaps being a bit more forgiving with a leading slash (e.g. accepting both XH.reloadApp('/mobile') and XH.reloadApp('mobile')), but that might be overkill


Modified the code to fix the hex() output in case of big endian

In case of s390x, the hex() produces the output in the reverse order which fails the following test cases like 02456_aggregate_state_conversion and 00396_uuid.
Changelog category (leave one):

Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Modified the code to fix the hex() output in case of big endian

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

This fix would reverse hex representation for all strings, for example:
create table simple (
    id UInt64,
    a String
) engine=Memory;
insert into simple(id, a) values(1, 'abc');
select hex(a) from from simple;

The result would be '636261', which is wrong.


test_basic/enqueue_map: Initialize all the data

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jason Ekstrand seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: Move the advisor name to low cardinality key values

Adviser name is moved from hight to low cardinality key values.
Updated tests in  to reflect these changes, ensuring the new low cardinality key is correctly captured.

Resolves #1716

merged in 7474852b58fd30829924beb8aea5aeb65cd2d586


Remove the purple parking agents from the unzoomed view. Instead, add…

… them to the parking layer, so supply and demand can be seen in the same place.
Before:

After:

The trade-offs:

The old minimap was inconsistent; most of the checkboxes are normal categories, but there was an extra subcategory represented.
But seeing the purple dots previously was helpful as an "alert" that parking might be scarce somewhere.
The new approach shows supply and demand of parking in one place, which should be more clear.

Yuwen and I also discussed resurrecting an old UI for changing the agent color scheme from the minimap. Once upon a time, there was a way to color all of the unzoomed dots by how long an agent was waiting at a particular spot, or by the total trip distance. If there are more ideas for usefully coloring unzoomed agents, we could bring this back in some form.
@michaelkirk, @muzlee1113

It looks good!

Would some kind of colored switch control be crazy?

Yep, using checkboxes just to take advantage of color. I agree switches would be more consistent; once we decide whether the control itself or the text should be colored, I'll send a followup change


remove @label(NonLiteral.Value) due to it's bad performance

fixes https://github.com/mpollmeier/gremlin-scala/issues/288

@timw @jeremysears

LGTM.

sonatype seems down again, will retrigger the build later
[error] java.io.IOException: Failure:java.util.concurrent.ExecutionException: org.apache.http.NoHttpResponseException: oss.sonatype.org:443 failed to respond

this is released as <IP_ADDRESS>


Policy for checking for arbitrary file existence

Policy for checking for arbitrary file existence
User Story
As a Security Engineer, so that I can enforce organization-wide policies regarding arbitrary files, I would like a new File Check policy that I may configure to meet my organization's needs.
Overview
OSSF Allstar current includes policies for checking the existence of various specific files.  For example, consider SECURITY.md via pkg/policies/security/ ; this policy specifies a constant at security.md line 31 of polName with a value of SECURITY.md.  In #469, @Matias-jt asked if it was possible to verify if a file exists and @ArisBee responded with advice to update the polName variable.  Similarly, @melmos asked in #450 if would be possible for a policy to check for security.json instead of SECURITY.md and it was suggested that the two issues may be duplicates with #469 referencing a general use-case while #450 is more specific.
The suggested change in the value of polName would require recompiling the tool -- that is, the value is set at compile-time, not at configuration-time or run-time.
Current Use-Case
My organization has multiple repositories.  We have an organizational policy (governance, not Allstar policy) of requiring repositories to include a configuration files for Pre-Commit, a tool that supports hooks for running detect-secrets, gitleaks, etc..  That is, we want to iterate across all our repos and make sure they all have .pre-commit-config.yaml files -- basically exactly what Allstar does, but instead of SECURITY.md, we would look for .pre-commit-config.yaml files.  While we could fork Allstar, we're thinking that it may be possible to help others who are in similar situations (e.g., the aforementioned issues) by generalizing a policy in Allstar that could be configured separately.
Alternative Implementation
High-level Goals

Allow changing the name of the file to be check at configuration-time
Allow the policy to check for the existence of absence of a file
Allow the policy to check in multiple locations
Allow the policy to check for multiple files

Without getting overly-prescriptive, it may be worthwhile to consider breaking down a new policy into smaller goals and iterate across them instead of trying to do the whole thing at once.
Iterative Steps
Iteration 1: variable filenames
Allow polName to be set via configuration file.  The configuration file would function just like the other tools' configuration files (e.g., security.md line 30 which specifies configFile = "security.yaml").  The sample quickstart file would be extended to support an additional parameter, such as fileName.  At runtime, instead of referencing a constant, a variable would be extracted from the configuration file.  Everything else would remain the same.  That is, if the file in question exists, policy compliance is accepted; if it's missing, the repository is non-compliant and the usual remedies (block checks, create issues, etc.) are taken.
It is recommended that basic checks are run to only allow tests for files in the repository, not arbitrary files on the filesystem.

read fileName string from the config file
if fileName exists, pass
otherwise fail

Consider the following configuration file:
---
optConfig:
  optOutStrategy: true
  action: issue

  fileName: "SECURITY.md"

Iteration 2: support file absence
Instead of having a policy check receive configuration via a string named fileName, the tool will accept a map with two keys of fileName and state where fileName is the name of the target file (as before) and state is either present or absent (consider Ansible's ansible.builtin.file module which allows absent and a variety of potential file types).

File exists?
State
Result

Yes
Present
Pass

Yes
Absent
Fail

No
Present
Fail

No
Absent
Fail

The most significant aspect of this iteration isn't the supporting the absence of a file, it's changing the configuration approach
from accepting a string to accepting a map.

read fileCheck map from the config file
extract fileName string from fileCheck
extract state string from fileCheck
if fileName exists and state == present, pass
if filename doesn't exist and state == absent, pass
otherwise, fail

Consider the following configuration file:
---
optConfig:
  optOutStrategy: true
  action: issue

  fileCheck:
    fileName: "SECURITY.md"
    state: "present"

Iteration 3: support multiple locations
Instead of having the policy check for a single specific location for a file (e.g., /SECURITY.md) specified as a string, accept an array of locations to check.  In the case of checking for file existence, if any of the locations match, consider the check passed; in checking for file absence, if any of the locations match, conside the check failed.  For example, the SECURITY.md file may be located at:

/SECURITY.md
/.github/SECURITY.md
/docs/SECURITY.md

The most significant change here is having the location be specified as an array rather than a string and iterating across the array instead of just checking one location.

read fileCheck map from the config file
extract fileNames array from fileCheck
extract state string from fileCheck
iterate across fileNames
a. if state == present and any match, pass; otherwise fail
b. if state == absent and any match, fail; otherwise pass

Consider the following configuration file:
---
optConfig:
  optOutStrategy: true
  action: issue

  fileCheck
    fileNames:
      - "/SECURITY.md"
      - "/.github/SECURITY.md"
      - "/docs/SECURITY.md"
    state: "present"

Iteration 4: support multiple checks
Currently, there's a 1:1 correspondence with regards to file checks and policies (e.g., the Security policy can't look for a CODEOWNERS file); this iteration supports multiple checks.
Instead of accepting a single map, fileCheck, accept an array of maps with each item representing the check for
a single file (although it may be in multiple locations).
Consider the following configuration file:
---
optConfig:
  optOutStrategy: true
  action: issue

  files:
    - SECURITY.md:
      fileNames:
        - "/SECURITY.md"
        - "/.github/SECURITY.md"
        - "/docs/SECURITY.md"
      state: present"

    - CODEOWNERS:
      fileNames:
        - "/CODEOWNERS"
        - "/.github/CODEOWNERS"
        - "/docs/CODEOWNERS"
      state: "present"

read files array from the config file
iterate across files array
extract fileNames array from iterant
extract state string from iterant
iterate across fileNames
a. if state == present and any match, pass; otherwise fail
b. if state == absent and any match, fail; otherwise pass

Hi @wesley-dean-flexion, for the fileNames like other GitHub Actions, is there a reason to not support the "Typescripty" way with GitHub Action configurations that support recursive wildcards?
So instead of supporting just the example below from your report.

  files:
    - SECURITY.md:
      fileNames:
        - "/SECURITY.md"
        - "/.github/SECURITY.md"
        - "/docs/SECURITY.md"

It could also support this?

  files:
    - SECURITY.md:
      fileNames:
        - "*/**/SECURITY.md"

You can look for specific files in specific subdirectories you can use the original method.
In either case, this looks like a nice a recommendation and I would support this addition. I have similar needs for such functionality in allstar. :smile:

@xee5ch that's a really good thought.  It would be really nice if it could accept recursive globs (**/SECURITY.md) or, really, any globs.
It appears -- and please correct me if I'm wrong as this is really foundational to the whole idea -- that Allstar does not download / clone / fetch / pull the repositories being scanned; instead, it makes calls using the GitHub client library for Go:
https://github.com/ossf/allstar/blob/964a34c8c454d9d3605fcd482c5cc2752ea4545b/pkg/policies/security/security.go#L98-L101
(sidenote: Allstar is coded to use v59 of the library; v60 was released on February 29, 2024)
I'm looking through the Github's GraphQL documentation to see if it's possible to use GraphQL to use globs to match files.

@wesley-dean-flexion This is great, thanks for digging into this.
The polName const in the existing "SECURITY.md" Allstar policy is not used for anything except for identifying the policy in logs.  The actual check is done by querying GitHub's GraphQL API here: https://github.com/ossf/allstar/blob/main/pkg/policies/security/security.go#L124-L137 Checking the Repository->IsSecurityPolicyEnabled bool. This allows for a SECURITY.md file to be placed in the GitHub Org's ".github" repository as well.
I'd suggest a new policy and to not touch the current "SECURITY.md" Allstar policy at all.
As far as iterations, feel free to go at your own pace. If you want to start with a simple single-file policy, that is ok, or if you want to go all the way to multiple checks with either/or lists of files to find, that works as well.
One thing to consider with the multiple-file checks is what to do about org vs repo level config. For policies that have single-variable options, the repo level config can override what is set at the org level (If overrides are allowed).  When you have a list, then it is hard to remove/override an org level requirement at the repo level. Usually you would just add both lists together.
For getting files, we have a cache-friendly wrapper around Repository.GetContents here: https://github.com/ossf/allstar/blob/main/pkg/config/contents.go#L26 This can be made to be exported from the config pkg, or moved to another package in Allstar.
I agree that getting contents of files directly using the API is the most desired implementation, but that might not work well with globs. I'm not sure if there is a search API or something under the GraphQL side that will work better. In the end, the Scorecard policy downloads the tarball, so we can do that for this as well if we find a way to share the downloaded tarball with the Scorecard policy.

@jeffmendoza this is amazing feedback.  Thank you so much!!

The polName const

I suspected that as I couldn't see where or how a call for that constant was used.  Reading this comment in #469 led me to my questioning myself (i.e., clearly I'm missing something):

Yes, that's doable if you duplicate the security.go check and replace SECURITY.md variable in polName.

I'd suggest a new policy

Yes, absolutely.  Agreed.

As far as iterations, feel free to go at your own pace.

My thinking with the iterations was three-fold:

make it clear what efforts would be involved and describe them as finite chunks of work
allow for regular demonstrations that can be tested cleanly
lower the barrier to entry for developers wishing to gain some experience working on an Open Source project

org vs repo level config

Oh, that's a really good point.  I suppose one could add an override option on a file-by-file basis; however, that may lead to an undesired level of complexity.  My hope was to keep things as simple as possible (I'm prone to over-engineering / bike-shedding), but that's a really good point that individual repos may have additional needs or customizations.  Based on your comment, I'm thinking of 3 models:

org level wins
repo level wins
merge both

So, maybe instead of a boolean (allowOverride, my first thought), perhaps it's a "merge strategy" (org, repo, mergeas options formergeStrategy`).
Something to think about....  :thinking:

For getting files, we have a cache-friendly wrapper

Excellent.  This is extremely helpful.  Thank you!

might not work well with globs

I haven't found evidence to see if globs would be supported.  It would be great if it could, but I don't know how much effort would be involved.  While it would be more resource-intensive, we could fetch the list of files and replicate a glob match (against the list in-memory) instead of using a GraphQL query.  I don't know what effort would be involved in fetching that list.
Some examples showed how to perform a non-recursive query for the list of names of files in a repo and suggested that a recursive option may be available in future revisions of the the API.  However, those conversations were several years old.
That said, running a query on the ossf/allstar repo's / path in the HEAD branch shows that while the query doesn't recurse, the isDirectory response field would allow us to recurse ourselves with multiple API queries.
I just ran a test with a path of "/**/*.md" and the response tree object came back as empty.
So, if we were to build a list of files in a repository, that would likely result in several API calls (one per directory entry); however, if we just queried the files for a given policy check, we're talking about one query per file check.  If we're looking for one..two..three locations for a file, it's highly probable a full repository scan would result in more API calls.
We could codify a check to see if a path query looks like a glob, we could do the full repo scan and do our own matching; however, that's a level of complexity that I don't think would be warranted at this time.
I really, really appreciate your taking the time to review this, offer suggestions and corrections, and support the refinement of this idea.
Tangentially, I noticed the Allstar list of future policies includes Dependabot.  Cool.  A file existence checker could look for the presence of .github/dependabot.yml and address a significant portion of that policy.  It could tell if that file existed but it wouldn't verify if it's valid YAML or if it matches the Dependabot schema or if it's configured properly.

@wesley-dean-flexion All this looks great. I support passing on globs for now and getting something helpful for many/most cases in sooner without it.
The merge strategy you described sounds good. Most policies use the "allow override" from the OptConfig, so we will just need appropriate documentation on this policy that it follows the merge policy setting instead.

@jeffmendoza Terrific.  Thank you very much.  Agreed on passing on globs for now -- seems nice to have, possibly for a future iteration.
Agreed re: documentation.  Perfect.
Again, I really appreciate your taking the time to read, review, and respond to this thread and for being open to the conversation.  Thank you.

I was pointed to todogroup/repolinter earlier today.  It looks like it can do what I wanted with this policy, so I'm having serious questions about whether or not it makes sense to duplicate it in Allstar.
One nice thing that I like about Allstar (among many!) is that it interacts with GitHub's API without cloning the repository being examined.  Repolinter is able to accept URLs to repositories hosted on SCMs with the --git flag; this results in the repo in question being cloned and then examined locally.  For a small repo, this may be just fine; however, for a large repo, this could result in a lot of unnecessary traffic as opposed to, say, a dozen API calls.
Another nice thing I like about Allstar is that it includes the ability to be used on an organization level, not just on a repository level.  I stumbled across a posting on organizationally required GitHub Actions that may be able to accomplish something similar.  Based on that article -- and I haven't tested anything yet -- an opt-out model appears to be more effort than Allstar's model, especially for organizations with many repositories.  It's entirely possible that I misinterpreted the source material or the functionality has changed since the article was posted.
A third nice thing that I like about Allstar is the ability to trigger actions based on findings, such as by creating issues in repositories that are missing, for example, SECURITY.md files.  I believe it may be possible to create an issue if a Repolint run failed (i.e., if: {{ failure{} }} ); however, implementing that would involve publishing an action to call Repolinter and react to Repolinter failures, and making that action required organizationally.  Long story short, more code to write and maintain.
I'm curious, @jeffmendoza , if you have any thoughts.  I have no ego in this so if you believe that it's not helpful to add this functionality to Allstar and would recommend this issue be closed / wont-do, that's totally fine with me.


Release some test dataset to evaluate

Hi, Authors,
The Mega_depth dataaset is too large to download, could you please release some bechmark data for us to do evaluation test?
Thank you.

Hi, mega1500 is the same files as in loftr.

Hi, could you provide a download link to mega 1500? Thank you.

Thank you. I find the loftr dataset and download megadepth dataset. Do you know what kind of data format I should put in directory for evaluation and training?


Azure AD B2C choose Web Applications user has permission to access

I'm doing a project where we want to centralize all the user's registration in our main website using Azure AD B2C.
In our internal web application we would like to choose which Application the user who have just signed up using B2C he/she could access. For example I could give permission to a user to access an Application A but not to access the Application B, so when he/she sign-in in this application A with B2C, he/she would successfully login, but in application B he/she would not be able.
In the previous project I did with Azure AD B2C, the users who have registered in one of my application could access all my other applications.
But in this case I don't want this, I would like to choose which applications a specific user could access.
Do you know if it is possible to do with Azure AD B2C?
Thank you very much in advance.

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

ID: 2c49481e-fe77-d60a-9a1a-7ab32ea7258a
Version Independent ID: a7afa85f-a91c-f80a-6982-900230432fff
Content: Technical and feature overview - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/technical-overview.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@fabio770 Thank you for your query. We will have this investigated and update this thread.

With custom policy, you can store the user's application in a comma delimiter string forma in Azure AD B2C user profile. To set the user's applications, use MS Graph. During the sign-in you can read the value of this custom attribute and call a claims transformation to check whether the attribute contains the app ID. If not, present an authorised error message to the user.  I'll try to build such a sample for you.

@fabio770 Please let us know if you have any further question or shall we proceed with closure of this thread?

@yoelhor Thank you so much! It's clear to me what I have to do now.

@amanmcse You can close this thread. Thank you!


can access the external mysql server when use kubernetes,cant to access when use istio

can access the external mysql server when use kubectl create -f * ,cant to access when use
kubectl crate -f <(istioctl kube-inject -f *)
and i entry the pod .
use the ping mysqlserver can connected .
and ping other url  can connected too.
but wget url   . return the 404 .
why ???

i known.
need to crate the engress rule


How to preprocess the dataset

Hi Shashi,
I am planning to run the code on your own data. How can I preprocess the dataset such that it can be used by the code (Refresh)?
Thanks,
Prateek

Send me an email, I will send you the demo code that I have not made public.

Sure, just sent you the email. My email id is<EMAIL_ADDRESS>

Hi Shashi,
I am planning to train the model on our own dataset. To understand the formatting of dataset required, I downloaded the dataset from the link mentioned on GitHub, I see cnn.training.doc, cnn.training.title, cnn.training.image, cnn.training.label.multipleoracle and cnn.training.label.singleoracle.
It will be great if you could please give some pointer about the multipleoracle training file and how would I go about generating those for my own dataset?
Also, is title assumed as the gold summary? I was confused, since in the paper it was mentioned highlight is the gold summary.

Check out my latest commit, hope this helps. You may have to adapt file names etc for your datasets.

articles are files with input documents (tokenized and sentence segmented, one sentence per line).
abstracts are files with the gold summary (tokenized and sentence segmented, one sentence per line).
Try to adapt this to your dataset. I am not authorized to release those formats for CNN/DM datasets.

Hi Shashi,
Thanks, I was able to have some progress. However, a problem I am facing -
Running the stanford core nlp (jar) for each document is very expensive. So I tried to concatenate all the docs to one large text/string and then pass it to the corenlp, but then I figured there is a limit to the text size you can pass it as an argument to corenlp. So I had to concatenate smaller number of docs as one text blob and then pass that to corenlp. Which kind of made it worked, but it seems hacky/doesn't feel right. As I will end up having multiple training files (.doc) & label.multipleoracle files, whereas your dataset contains just one .doc and .multipleoracle file.
Wanted to ask you if there is something I am missing or alternatively would you be able to share something that can help me.

I ended up using Stanford CoreNLP REST API. That solved my above issue.

I am glad you found a solution :)

Hi Shashi,
I am planning to run the code on your own data. How can I preprocess the dataset such that it can be used by the code (Refresh)?
Thanks,
Prateek

Dear  Shashi and prateekverma1,
Could you please give me some hints about the input file, I have downloaded the preprocessed-input-data from the link. Do training.doc and training.highlight represent raw text and gold abstract which is per sentence per line? However, what does other suffix mean？Such as image, label and title?
Thank you so much!

Hi Liuyue,
Shashi can correct me if I am wrong.
.doc,.image,.title files are used as the input to the summarization model.
.multipleoracle file is used for the label and reward that is given to the
model. It is the output generated from the estimate_multiple_oracle code
using the abstract files.
(As seen from the line - highlights_dir = os.path.join(data_dir, 'abstracts'
))
I am not sure regarding .highlight files as I could not find in the code
where it is being used.
Thanks,
Prateek
On Tue, Jan 1, 2019 at 10:57 AM liuyue<EMAIL_ADDRESS>wrote:

Hi Shashi,
I am planning to run the code on your own data. How can I preprocess the
dataset such that it can be used by the code (Refresh)?
Thanks,
Prateek
Dear Shashi and prateekverma1,
Could you please give me some hints about the input file, I have
downloaded the preprocessed-input-data from the link. Do training.doc and
training.highlight represent raw text and gold abstract which is per
sentence per line? However, what does other suffix mean？Such as image,
label and title?
Thank you so much!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EdinburghNLP/Refresh/issues/14#issuecomment-450739595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEUkofOmdsbKgyfKia1PcD7Iz057-NDoks5u-4VfgaJpZM4ZNrzD
.

Hi Shashi,
Sorry for asking same questions.
I have same question on cnn.training.doc, cnn.training.title, cnn.training.image, cnn.training.label.multipleoracle and cnn.training.label.singleoracle too.
I can't find code in scripts/oracle-estimator to generate .singleoracle or .multipleoracle or .title  ...etc
It will be great if you could please give some information about the cnn.training.doc, cnn.training.title, cnn.training.image, cnn.training.label.multipleoracle and cnn.training.label.singleoracle .

Check this for creating *.training.label.multipleoracle and *.training.label.singleoracle for your own datasets.
Oracle Estimation
Check our "scripts/oracle-estimator" to compute multiple oracles for your own dataset for training.
title and image files can be ignored. They are not used, just reminiscent from Sidenet codes. doc file is preprocessed input document file. Documents are separated by an empty line. For each document, each line represents a tokenized sentence with vocab id replacing each token. Check out the demo code : https://github.com/EdinburghNLP/Refresh/files/2846901/demo.tar.gz

Hi! Sorry. But I find title and image used in the code. please give some information about the title and image. Thank you very much!

Sorry please leave them empty (they are reminiscent of old code). They are not used here.

Hi Shashi,
I would like run your code on my own data. How can I preprocess the dataset that can be used in your code and get the .doc, .title files, just like the data in the path "Refresh-NAACL18-preprocessed-input-data"? @shashiongithub
Thanks for your reply.


TypeError: Cannot read property 'length' of undefined in 'Object.find' during 'getApplicableRefactors'

This issue comes from crash dumps in telemetry. We've tried to de-duplicate issues on a best-effort basis, comparing the sequence of methods called and the command requested while ignoring line numbers.
TypeScript Version Prefix: 2.7.1
VSCode Version 1.20.0-insider
command requested: getApplicableRefactors
hitting sessions: 3195
proportion of all sessions: 0.01192729390420087
stack
TypeError: Cannot read property 'length' of undefined
    at Object.find (tsserver.js:1523:34)
    at recordUsagebySymbol (tsserver.js:81559:41)
    at recordUsage (tsserver.js:81528:36)
    at collectUsages (tsserver.js:81521:25)
    at visitNode (tsserver.js:12595:24)
    at Object.forEachChild (tsserver.js:12958:24)
    at collectUsages (tsserver.js:81524:28)
    at visitNode (tsserver.js:12595:24)
    at Object.forEachChild (tsserver.js:12949:24)
    at collectUsages (tsserver.js:81524:28)
    at visitNode (tsserver.js:12595:24)
    at Object.forEachChild (tsserver.js:12776:24)
    at collectUsages (tsserver.js:81524:28)
    at visitNode (tsserver.js:12595:24)
    at Object.forEachChild (tsserver.js:12807:24)
    at collectUsages (tsserver.js:81524:28)
    at visitNodes (tsserver.js:12604:30)
    at Object.forEachChild (tsserver.js:12796:24)
    at collectUsages (tsserver.js:81524:28)
    at collectReadsAndWrites (tsserver.js:81397:17)
    at getPossibleExtractionsWorker (tsserver.js:80843:38)
    at getPossibleExtractions (tsserver.js:80800:26)
    at Object.getAvailableActions (tsserver.js:80399:35)
    at unknown (tsserver.js:77085:128)
    at getIterator (tsserver.js:1733:23)
    at Object.next (tsserver.js:1728:35)
    at Object.arrayFrom (tsserver.js:2267:32)
    at Object.getApplicableRefactors (tsserver.js:77084:23)
    at Object.getApplicableRefactors (tsserver.js:83299:32)
    at IOSession.Session.getApplicableRefactors (tsserver.js:90156:53)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:89112:61)
    at unknown (tsserver.js:90355:88)
    at IOSession.Session.executeWithRequestId (tsserver.js:90346:28)
    at IOSession.Session.executeCommand (tsserver.js:90355:33)
    at IOSession.Session.onMessage (tsserver.js:90375:35)
    at Interface.<anonymous> (tsserver.js:91576:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

@aozgaa These line numbers don't seem to be lining up with the file from npm install<EMAIL_ADDRESS>

Without line numbers, this is just a guess: I believe symbol.getDeclarations() is returning undefined on this line
const declInFile = find(symbol.getDeclarations(), d => d.getSourceFile() === sourceFile);

Looks like that's expected (getDeclarations(): Declaration[] | undefined) and find doesn't handle undefined.

Prototype properties don't have declarations.

This issue reproduced in the following context:
TypeScript Version Prefix: 2.7.2
VSCode Version 1.20.1
command requested: getApplicableRefactors
hitting sessions: 466
proportion of all sessions: 0.009567020468496583
stack
TypeError: Cannot read property 'length' of undefined
    at Object.find (tsserver.js:1524:34)
    at recordUsagebySymbol (tsserver.js:81757:41)
    at recordUsage (tsserver.js:81726:36)
    at collectUsages (tsserver.js:81719:25)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12770:24)
    at collectUsages (tsserver.js:81707:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12696:21)
    at collectUsages (tsserver.js:81722:28)
    at visitNodes (tsserver.js:12632:30)
    at Object.forEachChild (tsserver.js:12833:24)
    at collectUsages (tsserver.js:81722:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12847:24)
    at collectUsages (tsserver.js:81722:28)
    at visitNodes (tsserver.js:12632:30)
    at Object.forEachChild (tsserver.js:12824:24)
    at collectUsages (tsserver.js:81722:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12731:21)
    at collectUsages (tsserver.js:81722:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12806:21)
    at collectUsages (tsserver.js:81722:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12696:21)
    at collectUsages (tsserver.js:81722:28)
    at visitNodes (tsserver.js:12632:30)
    at Object.forEachChild (tsserver.js:12833:24)
    at collectUsages (tsserver.js:81722:28)
    at visitNode (tsserver.js:12623:24)
    at Object.forEachChild (tsserver.js:12831:21)
    at collectUsages (tsserver.js:81722:28)
    at visitNodes (tsserver.js:12632:30)
    at Object.forEachChild (tsserver.js:12824:24)
    at collectUsages (tsserver.js:81722:28)
    at collectReadsAndWrites (tsserver.js:81595:17)
    at getPossibleExtractionsWorker (tsserver.js:81041:38)
    at getPossibleExtractions (tsserver.js:80998:26)
    at Object.getAvailableActions (tsserver.js:80597:35)
    at unknown (tsserver.js:77283:128)
    at getIterator (tsserver.js:1734:23)
    at Object.next (tsserver.js:1729:35)
    at Object.arrayFrom (tsserver.js:2268:32)
    at Object.getApplicableRefactors (tsserver.js:77282:23)
    at Object.getApplicableRefactors (tsserver.js:83497:32)
    at IOSession.Session.getApplicableRefactors (tsserver.js:90354:53)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:89310:61)
    at unknown (tsserver.js:90553:88)
    at IOSession.Session.executeWithRequestId (tsserver.js:90544:28)
    at IOSession.Session.executeCommand (tsserver.js:90553:33)
    at IOSession.Session.onMessage (tsserver.js:90573:35)
    at Interface.<anonymous> (tsserver.js:91774:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)


Bypass version mismatch problems in generated Makefile

Hello,
Is there a way in connectal, where i don't check files for their versions, and build directly with bsc compiler.(Hopefully a place to decide whether to insert -u flag or not).
Incidentally, I have some previously compiled .bo files(unfortunately their source files are not available.) which are required. Now if i run connectal directly, It does a version check and throws error stating that binary versions doesn't match, need to recompile. Is there a way to solve this problem.
Attached log files.
log.txt

That message is from the bsc compiler. I don't know of any way to bypass it.
I usually clean old .bo versions whenever I change versions of bsc.
On Mon, Feb 24, 2020 at 5:07 AM rajat-agarwal-<EMAIL_ADDRESS>wrote:

Hello,
Is there a way in connectal, where i don't check files for their versions,
and build directly with bsc compiler.(Hopefully a place to decide whether
to insert -u flag or not).
Incidentally, I have some previously compiled .bo files(unfortunately
their source files are not available.) which are required. Now if i run
connectal directly, It does a version check and throws error stating that
binary versions doesn't match, need to recompile. Is there a way to solve
this problem.
Attached log files.
log.txt
https://github.com/cambridgehackers/connectal/files/4244234/log.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cambridgehackers/connectal/issues/158?email_source=notifications&email_token=AASTPMYA6XR4LR26IHQWTJTREOS6NA5CNFSM4K2GCJ6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IPWHHCQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASTPM34UDKROJWECUPFTJDREOS6NANCNFSM4K2GCJ6A
.


[Constraint solver] Minor cleanups for overload filtering & favoring

A grab bag of minor cleanups for overload filtering and favoring in the solver.

@swift-ci please smoke test and merge

@swift-ci please smoke test Linux

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


little thinks, I worry about.....

Hello,
I think this is a good idea making this. But I found some thinks, They don't work.

The rate of turn never gives a negativ Value. But I think the reason is this :
https://github.com/ttlappalainen/NMEA2000/issues/250#issuecomment-1040427529
It is not clear how the MPU9250 is mounted. in the Up case the direction of Rate of turn is the wrong way.
change to :  SetN2kRateOfTurn(N2kMsg, SID, -(gz * 0.017448352875489));

Why is there made a tumbling of the Compass, when the Values are never used ? It is overwriten immediately by "refresh_data()"
"magScale" is never set to "Mag_x_scale" ...
And in this case ... ? Why do you save "Mag_x_scale" in the EEPROM

In the function "calc_pitchRollYaw()" I thing the q[..] Values are a little confused. ... this Part also needs work.
See exactly the code on this Side :  http://en.wikipedia.org/wiki/Conversion_between_quaternions_and_Euler_angles

Rate of turn could very well be the case - I haven't tested it well yet.
MPU9250 was mounted upsidedown
Tumbling was part of the original code I based mine on. It can be removed.

I wasn't happy with the stability of the MPU9250 and ordered a CMPS14 to replace it.
The other plan is to rework it using SensESP (https://github.com/SignalK/SensESP) to make over the air updates possible and have a clean intergration with SignalK.
I'm not sure if you're using SignalK, but if you are it's worth looking into SensESP. We could work on a version together that has support for both MPU9250 (if you''re happy with it) and CMPS14.

Ok, I understand now.
I thought, I was tested out.
I am also not so happy with the MPU, but "rate of turn", roll, an Pitch works fine. ... with the little changes.
The heading was not working correct , but I thought the reason is my "Testequipment" and the house around :-).
the CMPS14 is a realy good Idea. ... but in the future ...
With SignalK ... I don't know for what ?? I don't know any Industrial Equipment woking with it.
a new translation ... ?? Oh help, not now.
And a new hardware connection ... why?

Hello htool. I'm trying to repeat your project, but the ESP32 does not see the MPU9250. How is your MPU9250 connected? By i2c or SPI?
What is the best way to disable BMP280 in code (only heading pointer needed)? Is there a diagram?
Thank you.

@BerndCirotzki Did you manage to get the MPU9250 working well? Maybe you can assist Eduard?

hi, I don't get how the MPU9250 has to be mounted, mine is vertical, I thought the sensors calibration would enable it to figure out its position but after calibration I still don't get correct pitch/roll/yaw behavior..
Would also be interested in CMPS14..either directly through SH-ESP32 which I prefer as much simpler and direct than having to fiddle around with SignalK..

Repeat the project on bno055  - it works quite well


Is there another application/website that does this?

Some, but not completely what we're aiming for, see https://github.com/FCCAmsterdam/tps3000/issues/2


Suggestion: Merging this with the base engine

I might consider merging this with Dave engine, since I'm the (official) maintainer done by Erizur himself because this has plenty of stuff Dave Engine itself lacks currently

Wait there's an official engine now, is there a repository?

That I can see

Wait there's an official engine now, is there a repository?

https://github.com/Erizur/FNF-DaveEngine there's been one for a long while now

It looks cool but there's a lot of bugs and unfinished stuff with this engine rn so it may not be the best time to merge it, but I am going to work on this engine soon when I have the time

It looks cool but there's a lot of bugs and unfinished stuff with this engine rn so it may not be the best time to merge it, but I am going to work on this engine soon when I have the time

yeah that's what I said, I've been working on it for a while now and I'm trying to fix them

I was talking about my engine my bad


Changing terminal name in config.json for better usability

Is your feature request related to a problem? Please describe.
The current terminal name is "xterm-color", it works fine but when i use commands like "htop" the mouse clicks and scrolls are disabled.
Describe the solution you'd like
By changing the terminal name to "xterm-256color" it  support for 256 colors enabled and also solves this mouse problem.

the GET parameter sshterm= can be used on the URL to set the term to whatever you want.
https://localhost:2222/host/<IP_ADDRESS>&sshTerm=xterm-color256

Should do what you want. Of course you can also edit the config.json or if you're using docker you can bind a local config.json to the space in the container where it expects it (just tested this with 0.4.6)
docker run -it --rm --name webssh2 -p 2222:2222 --mount type=bind,source=./config.json,target=/usr/src/config.json -e DEBUG="webssh2*" billchurch/webssh2:latest

I'll consider making this the default in future versions though.


LateInitializationError: Field '_onRouteEvent@1568038182' has not been initialized.

This issue I am seeing in the sentry with 41k issue. I have removed the late in the file. Will this solve the error ?
LateInitializationError: Field '_onRouteEvent@1568038182' has not been initialized.
Late ValueSetter? _onRouteEvent; to ValueSetter? _onRouteEvent

Stale issue message


Checks inside /vendor/bundle

I'm using default rubocop-rails settings:
inherit_gem:
  rubocop-rails:
    - config/rails.yml

Still it some how checks inside /vendor/
❯ bundle exec rubocop
Inspecting 130 files
....................C

Offenses:

vendor/bundle/bundler/gems/nexter-823fc8c247bd/.pryrc:2:22: C: Final newline missing.
require "./spec/book"

This is fixed in v0.4.1.
(Excluding dotfiles specifically was need. ref. https://github.com/bbatsov/rubocop/issues/4554 )


Why are the of methods not declared static?

Description
In the github notes of Lecture 4 on Generics, why is it that the of methods are not declared static in the ImList class in the snippet below? Without the static keyword, how can you call these methods? When I try, my jshell throws me an error (and, at least according me, understandably so).
import java.util.ArrayList;
import java.util.List;

public class ImList<E> {
    private final ArrayList<E> elems;

    private ImList() { // constructor is private
        this.elems = new ArrayList<>();
    }

    private ImList(List<E> list) { // constructor is private
        this.elems = new ArrayList<>(list);
    }

    static <T> ImList<T> of() {
        return new ImList<>();
    }

    static <T> ImList<T> of(List<T> list) {
        return new ImList<>(list);
    }

    // I added the code below
    public String toString(){
        return elems.toString();
    }

}

Topic:
Static Methods
Generic Methods
Screenshots
Here's the one of the errors when I try to run
/open ImList.java
List<Integer> list = List.of(1, 2, 3)
ImList.of(list)

jshell> ImList.of(list)|  Error:
|  non-static method <T>of(java.util.List<T>) cannot be referenced from a static context
|  ImList.of(list) // ImList<Integer> type-inferred implicitly since list is List<Integer>
|  ^-------^
|  Warning:
|  unchecked call to <T>of(java.util.List<T>) as a member of the raw type ImList
|  ImList.of(list) // ImList<Integer> type-inferred implicitly since list is List<Integer>
|  ^-------------^

That might help:
https://docs.oracle.com/javase/tutorial/java/generics/methods.html

In the github notes of Lecture 4 on Generics, why is it that the of methods are not declared static in the ImList class in the snippet below?

It is most likely a typo since it is written immediately after the code block that both of methods are static.

Here's the one of the errors when I try to run

List's of method is generic. Therefore you must specify the type. In your case,
List<Integer> list = List.<Integer>of(1, 2, 3);
ImList.<Integer>of(list);

Yes, it's a typo.  Fixed.


Project Dependency hugo-installer Doesn’t Support Hugo Versions Above v0.101.0

Description
The hugo-development team has merged their hugo arm and intel releases into a single universal binary. Project dependency hugo-installer is broken as a result, and It is unclear if the hugo-installer project is still actively maintained.
Steps to reproduce
Bump Hugo to any version above v0.101.0 and run rpm install.
Expected result
Specified Hugo version should be fetched and installed.
Actual result
An error is thrown detailing that an appropriate Hugo binary could not be found.
Environment

Checking for existing binary
Downloading binary from "https://github.com/gohugoio/hugo/releases/download/v0.102.0/hugo_extended_0.102.0_macOS-ARM64.tar.gz"
Error: An error occured while trying to download the binary from "https://github.com/gohugoio/hugo/releases/download/v0.102.0/hugo_extended_0.102.0_macOS-ARM64.tar.gz". Details: Response code 404 (Not Found)

To be solved upstream
https://github.com/dominique-mueller/hugo-installer/issues/49

Fixed: https://github.com/dominique-mueller/hugo-installer/issues/49#issuecomment-1245654329


feat: unit tests + fixes

:tada: This PR is included in version 0.107.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update Cake to version 0.28.0

This PR updates the Cake dependencies to latest version, avoiding to skip the validation if the user is using the latest version.

Updated the PR to build against latest Cake version containing breaking changes (0.28.0)

Failing build will be fixed once https://github.com/cake-contrib/Cake.Yaml/pull/11 is merged and this branch is rebased.

@joaoasrosa your changes have been merged, thanks for your contribution 👍


Travis builds not working

The last travis build of master was 2 years ago: https://travis-ci.org/mat007/turtle/branches
Is it disabled or so?

Yeah, CI is in a bad state, I started https://github.com/mat007/turtle/pull/45 to try and fix it a few months ago but the combination of compilers and boost versions made it really annoying…

Ah right, no that's not intentional, trying to fix that with #45 as well.
Thanks!


ci: add missing contents permission to dependabot-auto-merge workflow

Followup to #846

:tada: This PR is included in version 8.8.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Bug] Tests for models in installed packages are triggered for models with the same name in the current package

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When you install a package containing models with specified tests as your project dependency, and create a model in the current package with the same name as one of those models - tests for the upstream model will be triggered also for the new one (even if package:this selector is used) and can cause dbt test / dbt build failures.
Expected Behavior
Tests are not propagated to downstream packages, especially when using package:this selector.
Steps To Reproduce

create package_a
create model_a
create generic test sample_test
create schema.yml like:

version: 2

models:
  - name: model_a1
    tests:
      - sample_test

create package_b
create model_a
add package_a as a dependency
don't create schema.yml
run dbt ls -s "model_a1":

project_b.model_a1
project_a.model_a1
project_a.sample_test_model_a1_

run dbt ls -s "package:this,model_a1":

project_b.model_a1
project_a.sample_test_model_a1_

run dbt test -s "package:this,model_a1":

15:26:56  Running with dbt=1.8.6
15:26:57  Registered adapter: duckdb=1.8.3
15:26:57  Found 10 models, 1 operation, 3 data tests, 529 macros, 2 groups
15:26:57
15:26:57  Concurrency: 1 threads (target='duckdb')
15:26:57
15:26:57  1 of 1 START test sample_test_model_a1_ ........................................ [RUN]
15:26:57  1 of 1 PASS sample_test_model_a1_ .............................................. [PASS in 0.03s]
15:26:57
15:26:57  Finished running 1 test in 0 hours 0 minutes and 0.14 seconds (0.14s).
15:26:57
15:26:57  Completed successfully
15:26:57
15:26:57  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

I have also run {{ print(model) }} inside the test - and can confirm it's triggered for the model from the current package, not the upstream one.
Environment
- OS: macOS
- Python: 3.10
- dbt: 1.8.5

Which database adapter are you using with dbt?
redshift, duckdb (doesn't matter)

Thanks for reporting this @jaklan !
I was able to reproduce what you reported. But I didn't need to use a custom generic test. Using a standard out-of-the-box test like not_null was sufficient. Details in the "reprex" section below.
More details
I was also able to get this to behave correctly by:

temporarily deleting project_b.model_a1,
using --no-partial-parse when doing a dbt list (or dbt parse, etc) to recreate the partial parsing file
reinstating project_b.model_a1,
proceeding from there

That's not a sustainable workaround! But it did give me a good manifest.json that I could compare with a bad one:
The key differences were in the parent_map and child_map sections.
Good
    "parent_map": {
        "model.package_a.model_a1": [],
        "test.package_a.sample_test_model_a1_id.ff32e12b9f": [
            "model.package_a.model_a1"
        ],
        "model.package_b.model_a1": []
    },
    "child_map": {
        "model.package_a.model_a1": [
            "test.package_a.sample_test_model_a1_id.ff32e12b9f"
        ],
        "test.package_a.sample_test_model_a1_id.ff32e12b9f": [],
        "model.package_b.model_a1": []
    },

Bad
    "parent_map": {
        "model.package_b.model_a1": [],
        "model.package_a.model_a1": [],
        "test.package_a.sample_test_model_a1_id.ff32e12b9f": [
            "model.package_b.model_a1"
        ]
    },
    "child_map": {
        "model.package_b.model_a1": [
            "test.package_a.sample_test_model_a1_id.ff32e12b9f"
        ],
        "model.package_a.model_a1": [],
        "test.package_a.sample_test_model_a1_id.ff32e12b9f": []
    },

Basically, when parsing from scratch, it is favoring the model name from the root package (package_b) and incorrectly linking it to the upstream test definition (from package_a). 💥

Reprex

package_a/models/model_a1.sql
select 1 as id

models/schema.yml
models:
  - name: model_a1
    columns:
      - name: id
        tests:
          - not_null

package_b/models/model_a1.sql
{{ config(alias="package_b__model_a1") }}

select 1 as id

packages.yml
packages:
  - local: ../package_a

Run these commands:
cd package_b
dbt deps
dbt list --no-partial-parse
dbt list
dbt list --select "package:this,model_a1"
dbt test -s "package:this,model_a1"

💥
Here's a workaround to get it to behave as desired (at least for a little while):
mv models/model_a1.sql models/model_a1.sql.x
dbt list --no-partial-parse
mv models/model_a1.sql.x models/model_a1.sql
dbt list
dbt list --select "package:this,model_a1"
dbt test -s "package:this,model_a1"

✅
But this is not a sustainable workaround!

@dbeatty10 thanks for verification!


Public Users can search the registry and select reports to pay for

As a Public User I want to be able to easily lookup my specific site So That *I can purchase available site registry reports related to a specific site *
Additional Context

Should work on desktop and mobile
Mobile view, as a fallback, can have map entirely disabled.

Acceptance Criteria

[ ] Given the user is on the search/home page When the user searches an address, Then results should be shown both in the text section as well as pins on the interactive map.
[ ] Given search results are displayed, When the user clicks on a pin on the interactive map OR clicks on the arrow in search results, Then available reports should be displayed with their price.
[ ] Given available reports are displayed, When the user checks one, Then it should be added to the total at checkout.
[ ] Given available reports are displayed, When the user clicks on a previously selected report, Then it should be unselected (i.e. user can toggle them like a checkbox).

Out of Scope

Payment process integration (eg PayBC)
Report generation. The actual report content (PDF etc) will be handled in another ticket.

Designs
https://www.figma.com/proto/xqs2urLzPur7FUb0pEsrGm/EPD-Design-Files?page-id=47%3A2&node-id=74%3A96&viewport=2151%2C336%2C0.23&scaling=scale-down&starting-point-node-id=74%3A96&show-proto-sidebar=1
Definition of Done

[ ] Ready to Demo in Sprint Review
[ ] Does what I have made have appropriate test coverage?
[ ] Documentation and/or scientific documentation exists and can be found
[ ] Peer Reviewed by 2 people on the team
[ ] Manual testing of all PRs in Dev and Prod
[ ] Merged

What does "Report Generation" look like?  We have example "Report Names" in the designs, but what are their actual content?  How will we get it?

@acoard-aot if we need pdf reports , then we could look at using cdogs(https://github.com/bcgov/common-document-generation-service). I have used it in the past and here is the sample code consuming the cdogs api to generate pdf reports.
https://github.com/bcgov/EDUC-PEN-REPORT-GENERATION-API

Hey team! Please add your planning poker estimate with Zenhub @midhun-aot @nikhila-aot @RyanBirtch-aot

TODO: Split into two stories, one for search, one for 'checking out'.  May be too vague to estimate now without knowledge of backend part.


[MLIR] Erase dead lmhlo.constant ops

An lmhlo.constant op on an memref that is locally allocated and with
no users other than dealloc's can be deleted. Add a canonicalization
pattern for this.

@River707 @sherhut @stellaraccident for visibility.

That's exactly what I had in mind and was also related to the question on the memory effect interface I posted on discord. It'd require a bit more thought to design that.

Can you add a TODO in the pattern?

Can you add a TODO in the pattern?

Done. In fact, the generalization would go beyond dead stores -- to also transparently cover other ops like DMA operations transferring to that memref.

(I used "store" in a liberal way)

Build is back green, feel free to rebase!

Actually I think we will be able to integrate directly.

Actually I think we will be able to integrate directly.

Anyway, rebased and pushed.

It got pushed here: https://github.com/tensorflow/tensorflow/commit/aaed01bdb99945d671ac28bb0d3203cc50028b87
I don't know why it didn't mark the PR as closed?

@joker-eph sometimes auto-merge does not happen ,it is something to do with copybara.


Connects to local and 3/4G but not public wifi?

More than likely nothing wrong with your code, and thanks for a great product.
I can connect locally on my own wifi and via 3/4G but not on a public wifi, thoughts?
Again thanks

Most likely public Wi-Fi is restricted. Try selecting port 53 during setup and hope for the best.


Fix imag for 0-size array

Fixes #2885

If #2870 is merged earlier, its TODO can be addressed.

If #2870 is merged earlier, its TODO can be addressed.

Done. PTAL

Jenkins, test this please.

Jenkins CI test (for commit 228c256381b5a8416d850bcc3203d30891d56d09, target branch master) succeeded!

Jenkins CI test (for commit 228c256381b5a8416d850bcc3203d30891d56d09, target branch master) failed with status FAILURE.


[yaml] remove PubSubLite from API docs

PubSubLite is deprecated as per https://cloud.google.com/pubsub/lite/docs, scheduled to be turned down March 18, 2026, and is no longer available for new customers as of September 24, 2024.
In order to avoid confusion for new Beam YAML users, this transform will be hidden from the API docs, but still usable within the YAML API

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @robertwb

R: @damccorm

Test failures seem unrelated


Create Hide-Anything

The Hide Anything plugin is a RuneLite plugin that allows you to hide specific players and NPCs in the game. It provides an easy way to customize your in-game experience by removing distractions or focusing on what you find important. You can use the right-click menu to select the entities you wish to hide.

Hello? https://github.com/runelite/plugin-hub/pull/7037#issuecomment-2495749553


winget

winget is too slow

gonna offer help to fix it? or just whine

When I type winget search or install
/
It keeps spinning around and there's no response
发送自 Windows 10 版邮件https://go.microsoft.com/fwlink/?LinkId=550986应用
发件人: @.>
发送时间: 2021年6月4日 12:43
收件人: @.>
抄送: @.>; @.>
主题: Re: [microsoft/winget-cli] winget (#1102)
gonna offer help to fix it? or just whine
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/winget-cli/issues/1102#issuecomment-854353672, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQCXTH3BURRT2PBEMICQMKDTRBKV7ANCNFSM46CBMFSA.

Works great on my PC, and i dont have the best internet. What speeds do you get?

Can you post the output of winget --info?


[FEATURE]: karavi-topology Image scan Workflow

Describe the solution you'd like
We need a workflow that  scans all karavi-topology images and alerts maintainers when a bug is found
Describe alternatives you've considered
I have considered using Jenkins Job
Additional context
N/A

This issue will be closed since the feature will be resolved with Jenkins Jobs