Can't run xctestrun file with scan when created with gym

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.3

Ruby
2.3.1

Bundler?
true

Git
git version 2.14.3 (Apple Git-98)

Installation Source
~/.rbenv/versions/2.3.1/bin/fastlane

Host
Mac OS X 10.13.3 (17D102)

Ruby Lib Dir
~/.rbenv/versions/2.3.1/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode-9.2.app/Contents/Developer/

Xcode Version
9.2

System Locale

Variable
Value

LANG
de_DE.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
default_platform :ios

platform :ios do
  lane :build_with_unit_tests do
    build_lane(
      scheme: "Test",
      configuration: "Debug"
    )
    run_unit_tests
  end

  lane :build_with_unit_and_ui_tests do
    build_lane(
      scheme: "Test",
      configuration: "Debug"
    )
    run_unit_tests
    run_ui_tests
  end

  private_lane :run_unit_tests do |options|
    scan_lane(scheme: "Test", only_testing: 'TestTests')
  end

  private_lane :run_ui_tests do |options|
    scan_lane(scheme: "Test", only_testing: 'TestUITests')
  end

  private_lane :build_lane do |options|
    scheme = options[:scheme]
    configuration = options[:configuration]
    gym(
      scheme: scheme,
      configuration: configuration,
      project: "Test.xcodeproj",
      clean: true,
      include_bitcode: false,
      derived_data_path: "build",
      export_method: "development",
      xcargs: "build-for-testing"
    )
  end

  private_lane :scan_lane do |options|
    scheme = options[:scheme]
    only_testing = options[:only_testing] || []
    scan(
      xctestrun: "build/Build/Products/Test_iphonesimulator11.2-x86_64.xctestrun",
      only_testing: only_testing,
      derived_data_path: "build"
    )
  end
end

No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.87.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

bundler
1.16.1

io-console
0.4.5

CFPropertyList
3.0.0

public_suffix
2.0.5

addressable
2.5.2

atomos
0.1.2

babosa
1.0.2

claide
1.0.2

colored
1.2

colored2
3.1.2

highline
1.7.10

commander-fastlane
4.4.6

declarative
0.0.10

declarative-option
0.1.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

dotenv
2.2.1

excon
0.62.0

multipart-post
2.0.0

faraday
0.14.0

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.12.2

fastimage
2.1.1

gh_inspector
1.1.3

jwt
2.1.0

little-plugger
1.1.4

multi_json
1.13.1

logging
2.2.2

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.2

httpclient
2.8.3

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.1.0

representable
3.0.4

retriable
3.1.1

google-api-client
0.13.6

json
2.1.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.4.0

rubyzip
1.2.1

security
0.1.3

naturally
2.1.0

simctl
1.6.2

slack-notifier
2.3.2

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

word_wrap
1.0.0

nanaimo
0.2.4

xcodeproj
1.5.7

rouge
2.0.7

xcpretty
0.2.8

xcpretty-travis-formatter
1.0.0

generated on: 2018-03-27

I am referring to #8376. I want to build the application via gym and then reuse the build to run unit/uitests for my application. As stated in this issue: #8376 this should be possible by giving gym the build-for-testing option via xcargs. However when running it like this gym will create an xcarchive in which the .app file is located. When running scan with the created xctestrun after this the .app file can not be found because scan is searching for the app in Build/Products/Debug-(iphoneos/iphonesimulator)/AppName.app.
Is there any way to use gym to build a valid xctestrun file ? I got it working by replacing gym with xcodebuild because xcodebuild action will run xcodebuild build and not xcodebuild archive. Any help would be much appreciated :).
Btw. I could not append the output of --capture_output because it does not print any thing for me. I tried with bundle exec fastlane ios build_with_unit_and_ui_tests --capture_output.

@hjanuschka Do you have time to look at this issue? IIRC you have gotten this working before

@getaaron , i give it a try 👍 , @dehlen have you tried setting derrived_data path to the same value, in gym and scan? -> https://github.com/fastlane/fastlane/issues/8376#issuecomment-282754603

Yes you can see that in the attaches Fastfile. Both paths are set to „build“.
Thanks for helping out :)

You can test it out by creating a new Single View iOS App called Test with a UnitTest and UITest target. Then the provided Fastfile can be used attached to this issue. You can see that the scan lane will fail because when trying to run the xctestrun file it will search for Test.app in Build/Products/Debug-iphone*/Test.app bit the app is created by gym in Library/.../28-03-2018Test.xcarchive/.../Test.app.

@hjanuschka any other ideas for me ?

@dehlen ok i tried the test app and your fastfile.
problem is the PR's for adding "native" support for - build-for-testing to gym have been reverted/not-accepted.   https://github.com/fastlane/fastlane/pull/7131
taken the sample from here:
https://github.com/hjanuschka/fastlane-plugin-aws_device_farm
using xcodebuild itself, you can achieve what you want.
changing your gym call to:
    xcodebuild(
        scheme: scheme,
        destination: 'generic/platform=iOS',
        configuration: configuration,
        derivedDataPath: 'build',
        xcargs: "clean build-for-testing"

      )

produces the correct xctestrun file.
here is your original fastfile, adopted:

Fastfile
default_platform :ios
platform :ios do
lane :build_with_unit_tests do
build_lane(
scheme: "Test",
configuration: "Debug"
)
run_unit_tests
end
lane :build_with_unit_and_ui_tests do
build_lane(
scheme: "Test",
configuration: "Debug"
)
run_unit_tests
run_ui_tests
end
lane :run_unit_tests do |options|
scan_lane(scheme: "Test", only_testing: 'TestTests')
end
lane :run_ui_tests do |options|
scan_lane(scheme: "Test", only_testing: 'TestUITests')
end
private_lane :build_lane do |options|
scheme = options[:scheme]
configuration = options[:configuration]
xcodebuild(
scheme: scheme,
destination: 'generic/platform=iOS',
configuration: configuration,
derivedDataPath: 'build',
xcargs: "ENABLE_BITCODE=NO CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO clean build-for-testing"
)
end
lane :scan_lane do |options|
scheme = options[:scheme]
only_testing = options[:only_testing] || []
scan(
xctestrun: "build/Build/Products/Test_iphoneos11.3-armv7k.xctestrun",
only_testing: only_testing,
derived_data_path: "build"
)
end
end

changes:

private lanes made public (just for testing)
gym replaced with xcodebuild (thats the trick)
path adopted for current ios sdk

Thanks, thats what I meant when I wrote

I got it working by replacing gym with xcodebuild because xcodebuild action will run xcodebuild build and not xcodebuild archive.

However the problem still persists that my current gym command will produce a valid ipa which is signed by match. How can I replicate this behaviour with xcodebuild ? To me it looks like xcodebuild with run xcodebuild build but gym runs xcodebuild archive. The example I posted above uses exportMethode development which was only for testing purposes. My gym command which I'd like to replicate with xcodebuild in order to also build-for-testing looks like this:
gym(       clean: true,       scheme: "MyScheme",       output_directory: output_dir,       configuration: configuration,       derived_data_path: "build",       export_method: "ad-hoc",       export_options: {         provisioningProfiles: {            "de.some.bundle" => "match AdHoc de.some.bundle"         },         compileBitcode: false       }     )


php: use %E in errorformat instead of postprocess

Now that #1006 is merged in and the tests for this refactor are passing, are any other changes required?


Error when enabling 1Password as a Provider

What happened?
I followed the documentation as mentioned here: https://www.pulumi.com/docs/esc/providers/1password-secrets/ to enable the 1Password Provider into Pulumi ESC.
I created a new environment via the UI and called it home as i'm managing my homelab with it.
In this environment I setup the the integration (more in the example below) and I got the following error when trying to open the Environment via the UI
home:4:7 - values["1password"].secrets:
		Error: error creating client

Example
Here's the environment config
values:
  1password:
    secrets:
      fn::open::1password-secrets:
        login:
          serviceAccountToken:
            fn::secret: "op_asdf"
        get:
          personal-public-key:
            ref: "op://Infrastructure/SSH - Personal/public key"

Output of pulumi about
Version      3.113.1
Go Version   go1.22.2
Go Compiler  gc

Plugins
NAME       VERSION
go         unknown
proxmoxve  5.10.0

Host
OS       darwin
Version  14.2.1
Arch     arm64

This project is written in go: executable='/Users/henrique.dalssaso/.asdf/installs/golang/1.21.3/go/bin/go' version='go version go1.21.3 darwin/arm64'

Current Stack: dalssaso/setup-proxmox/home

Found no resources associated with home

Found no pending operations associated with home

Backend
Name           pulumi.com
URL            https://app.pulumi.com/dalssaso
User           dalssaso
Organizations  dalssaso
Token type     personal

Dependencies:
NAME                                         VERSION
github.com/muhlba91/pulumi-proxmoxve/sdk/v5  v5.10.0
github.com/pulumi/pulumi/sdk/v3              v3.76.1
github.com/ryanwholey/go-pihole              v0.0.4

Pulumi locates its logs in /var/folders/8g/b590qbh52gzdc_b1qn39hk340000gp/T/ by default

❯ esc version
v0.8.3

Additional context
No response
Contributing
No response

Generally that error indicates that the provided service account token is invalid. We need to do a better job of surfacing that error. Have you checked to ensure that your service account credentials are valid?

Hi, I had a similar issue related to the use of an SSH “file.” I saved the “file” in 1Password and referenced it. I was unable to load this file into the ECS configuration until I saved the content of the file as a text field in 1Password.

@pavelbinar We rely on the official 1password go sdk, which as of now does not yet support "SSH public keys, fingerprint and key type". Once that gets supported we will make sure that it works in ESC.


Address accessing of non-existing field _maybe_parsed_body within httpx.Response object

It seems that the following code:
response: httpx.Response = await self.http_client.call(
    url=self._url(f'records/{key}'),
    method='GET',
    params=self._params(),
)

returns a response of type httpx.Response. Later the field _maybe_parsed_body is accessed:
return {
    'value': response._maybe_parsed_body,
    '...': '...',
}

There are 2 occurrences of this:

https://github.com/apify/apify-client-python/blob/v1.6.0/src/apify_client/clients/resource_clients/key_value_store.py#L120
https://github.com/apify/apify-client-python/blob/v1.6.0/src/apify_client/clients/resource_clients/key_value_store.py#L323

Check if http_client.call really returns a httpx.Response object and in such case fix the accessing of non-existing field _maybe_parsed_body.

http_client.call returns a httpx.Response which is in some cases enhanced with a _maybe_parsed_body. It's a bit of a hack which was really useful when we still used requests, and simplified the code a lot, but when we switched to using httpx and did some more changes, it ended up being used only in one method, when fetching a key-value store record body. So now we could put the "maybe" response parsing into that method, instead of having it in the http_client, to have a better separation of concerns, and less hacks.

My IDE was not able to recognize that, instead it raised an accessing of non-existing field error. The self.http_client.call is annotated to return httpx.Response object. So it might be the source of the problem.

Yeah, that's probably it. I wonder how this ever passed lint without a #noqa comment 🤷

Yeah, that's probably it. I wonder how this ever passed lint without a #noqa comment 🤷

I think because accessing fields is checked by Mypy. That's the reason why there is # type: ignore .


First Week of Month Shows on Previous Month

When the date is March 1, the overflow from February shows the current week / day.

@freiksenet When I close this bug we should do a review and then push a new release to npm and gh-pages.

@lolJS What's your npm username? I'll add you to package collaborators.

@freiksenet Thanks, it's anoobis.

Done.
On Fri, Mar 4, 2016 at 7:00 PM Jonathan Brito<EMAIL_ADDRESS>wrote:

@freiksenet https://github.com/freiksenet Thanks, it's anoobis
https://www.npmjs.com/~anoobis.
—
Reply to this email directly or view it on GitHub
https://github.com/freiksenet/react-calendar/issues/30#issuecomment-192357880
.


Sendding two Chat Completions requests at the same time fails

when sending two request to the API at the same time I get the following :
privateGPT-main\PrivateGpt\Lib\site-packages\llama_cpp\llama_cpp.py", line 1622, in llama_decode
return _lib.llama_decode(ctx, batch)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
OSError: exception: access violation reading 0x0000000000002600

I have the same case, when 2 requests gets simultaneous sent it gives me a Segmentation Fault error.

this is only a prototype, and serve one 1 request at a time, you cant use it for production, use vllm intead of this, thats more robust

@imad-ict Thank you for your response.
So if I go with Ollama instead of llamaCPP will I have anything changed ?

@loayghawji  Ollama has in-memory queue to support parallel requests. Please check Ollama docs:
https://github.com/ollama/ollama/blob/main/docs/faq.md#how-does-ollama-handle-concurrent-requests


swagger core does not support parameter fields defined in swagger spec 2.0

To support swagger spec 2.0 the following fields have to be added to SerializableParameter and AbstractSerializableParameter
See spec 2.0 params: https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md#parameterObject
missing fields:
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor17.
     */
    public Number maximum;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor17
     */
    public Boolean exclusiveMaximum;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor21
     */
    public Number minimum;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor21
     */
    public Boolean exclusiveMinimum;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor26
     */
    public Integer maxLength;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor29
     */
    public Integer minLength;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor33
     */
    public String pattern;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor42
     */
    public Integer maxItems;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor45
     */
    public Integer minItems;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor49
     */
    public Boolean uniqueItems;
    /**
     * See http://json-schema.org/latest/json-schema-validation.html#anchor14
     */
    public Number multipleOf;

recommended solution: https://github.com/swagger-api/swagger-core/pull/1524
Please take take a look.

@iushankin - can you please review the linked PR? Make sure it covers everything including tests. If all is well and the build was successful, feel free to merge.

@webron I approved and merged this PR

Cool, thank you.

Hi,
can you please update the SNAPSHOT artifact at https://oss.sonatype.org/content/repositories/snapshots/io/swagger/swagger-core/1.5.5-SNAPSHOT/
I would need the latest build from swagger-core for the swagger-codegen project. Now my tests are failing.

i just pushed 1.5.5-SNAPSHOT

Is there any annotation support for minItems, maxItems, etc?

Not yet, but it's about to be in.


Changing hover effects in navbar

Is your feature request related to a problem? Please describe.
The hover effects on the navbar items don;t look really good. I would want to improve it.
Describe the solution you'd like
Make the transition speed slower and change the support us' dropdown's bg color.
Describe alternatives you've considered
Maybe change the bg of navbar to make it more translucent.
Additional context

I want to work on this issue
Pls assign me

Yo! mschandana16
Can I work on this issue I would really like to improve the features

Can i work on this issue

The hover effects on the navbar items are not visually appealing.Some changes i want to make is
Slow down transition speed: Adjust the CSS transition speed to create smoother hover effects.
Change "Support Us" dropdown's background color: Modify the background color to enhance visibility and aesthetics.
Changing the background of the entire navbar to a more translucent style for improved overall appearance.
So,If you like my idea i want to work in this project.
Thanks and Regards,
Dibyanshu Jha

Please assign me this issue , I am a frontend developer with proficiency in react

Please assign this issue to me. I will fix the issue. I will make the links to pop up which will make them beautiful. I will add transition in the navlinks

Please assign me this issue

i want working of this issue, please assign me

@akshitagupta15june, I've noted the opportunity to improve hover effects in the navbar. With my experience in frontend development and UI design, I'm well-equipped to make these adjustments. You can learn more about my background and projects on my LinkedIn profile https://www.linkedin.com/in/prajna-paul-117086256/ and my GitHub profile https://github.com/prajna321/. Please assign this issue to me. I'm eager to contribute. Thank you!


Bitcoin Lost

I had brought Bitcoin and send it to the address on Copay and by accident i refreshed the address. Do you think the bitcoin will still come ? or i lost it ?

Hello @adot092
To protect your privacy, the Copay/BitPay wallet generates new addresses automatically once you use a bitcoin address once. Wallet addresses do not expire – any payments sent to past addresses will continue to appear in your balance.
Please see this article for more information: Why does my BitPay wallet's bitcoin address keep changing?
Kind regards.

yea but i haven’t received any funds

it has never appeared on my wallet

@adot092
we have developed our own hardware for cracking all versions of bitcoin wallet password with superfast speed
and if you still has problem I can help you run it. you can find my contact details on my profile
or you can send me message in telegram: @viladmir2022 or gmail<EMAIL_ADDRESS>


refactor(v2): remove focus outline from mouse users

Motivation
We currently have outline displayed after clicking on the dropdown item, similar to tabs (which was decided in #3171).
Therefore, in order to avoid duplicate code, we are better off using more general solution for the whole application than applying this fix (remove outline when not tab) to proper components.
This a11y issue is detailed in this blog post https://hackernoon.com/removing-that-ugly-focus-ring-and-keeping-it-too-6c8727fefcd2  that I was inspired in this PR.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

This code is likely to be shared between the 3 themes, and not meant to be swizzled, so I'd rather move it to src/client, until we create a new package with all the design-agnostic theme code. (there's outline none, but I think this still make sense for all themes to adopt this behavior)

This must be done in a separate PR. Moreover, I did not delete anything in the Bootstrap theme, so focus behavior will is the same as before.

Hey @lex111, found this potential new solution that looks more future proof.
Do you think we should adopt it instead?
https://matthiasott.com/notes/focus-visible-is-here

It's cool, but Safari and Opera don't support it. I think it's better to wait until global support is at least 50% https://caniuse.com/?search=focus-visible We also have tablets that are connected to keyboard (for example, iPad with Magic Keyboard). WDYT?


Dead?

today i test some workers and dont get the video from yt, i think youtube dont like grab free live streams :c

It's actually a hit or miss, sometimes it will work, sometimes it doesnt.


SAP Cloud SDK for JavaScript tutorials: minor improvements

This PR introduces some minor corrections and improvements to the tutorials about the SAP Cloud SDK for JavaScript.
Also, the change to the product name (previously "SAP S/4HANA Cloud SDK") is reflected here.
This PR supersedes #3418, i.e. all of the changes of that PR are also in here. Therefore, #3418 can be closed when this PR is merged.

Thanks for the update. When writing notes, usse the MD notes syntax > and avoid writing "Notes".


[FEATURE REQ] Duration for Arm Deployments

Library name
Azure.ResourceManager.Resources
Please describe the feature.
This feature request is for ArmDeploymentProperties. I would like to request the field "duration" which is the duration of the deployment for deploying ARM templates using the ArmClient. Right now it appears to be defaulting at 4 days.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@stevengiannakopoulos I suppose you want to add duration to ArmDeploymentProperties.
SDK is generated from swagger, if we take a look at DeploymentProperties in swagger, duration is missing. This is not something SKD team can do, you need to raise a feature request to the service team. Please let me know if I misunderstand.

Since this issue is related to the service side and not the SDK side, so we are not able to help you with it. Therefore, we are closing this issue for now. If you need more assistance, please open an Azure support request. We apologize for any inconvenience this may cause you.
We value your feedback and we want to make sure that your problem is solved. If you think that we have misunderstood your issue or closed it incorrectly, please feel free to comment on this thread and reopen the issue. We will be happy to assist you further. Thank you for your understanding and cooperation.


Remake the project with C# and Avalonia

Why
While C++ and Qt is really performant, it's not the most easy thing to use and the existing jungle of spaghetti sucks and I don't really want to have to deal with C++ again for every aspect of the project.
The initial "release" was basically only ever meant to be a testing ground for seeing if it's possible to make a Steam GUI ourself, but because of the initial positive feedback I want to keep maintaining this project, which isn't possible currently.
Rewriting this in C# should make this a client that the community can integrate cool things into, like ProtonDB support, theming, game issue workarounds, easy custom Proton version downloading and a whole lot else.

It makes system requirements pretty high (Qt6 is NEW)
Cross-platform isn't as easy (In C#/Avalonia it comes OOTB basically)
Sharing code kind of sucks
UI is dreadful to write
Debugging is a pain, especially for the average Joe sending in a crash report

With C# and Avalonia:

MVVM pattern makes it nice and easy to make UI
Avalonia's default style looks fantastic on all platforms
Code sharing is very easy
C# has all the goodies built in like string formatting
C# is much easier so getting contributors should be a lot easier
Don't have to worry about memory corruption and race conditions, only when dealing directly with natives

UI is dreadful to write

have you tried qml instead of hand-writing the ui?
I'm all for ditching c++ and qt, but have you considered a binding to qt?  As far as I know, only the rust and the python binding are not abandoned, and I wouldn't like a python program.

I know nothing about Avalonia, and am indiffrent to c++/c# in general.
But if you looking for opinions, i'd say choosing what's simplest for you is a good idea (persumably c#).
A thing that tends to worry me, is having lots of unused deps. (sometimes running and unused).
But considering the initial test release already had build flags, i'm not worried about it.

I'm all for ditching c++ and qt, but have you considered a binding to qt? As far as I know, only the rust and the python binding are not abandoned, and I wouldn't like a python program.

Can second the anti-python sentiment.
But i also don't particularly care about rust and qt even though i do like durams color scheme,
although i assume something simmilar can be done with Avalonia?

Yeah, QML could've been pretty good, however it is just JS, without the added benefits of something like electron.

I expected to see something like steam://open/minigameslist here, something lightweight with offline support to launch some installed singleplayer games. Still looking into sources if I can adapt it to be like this.
I have some rhetoric questions because of this remake:

Why a fancy ui? Or clone steam client ui?
Regular Qt app ui should do the trick (menu, panel, status bar, standard theme).
C# and "getting contributors" on linux?
Most people will sit on release builds, others will need to setup C# dev env.

I'd better chose Python for remake (like a tool, not because I like it or know)

there are already some libs to interact with steam and its files
simple enough for most linux gamers to understand and contribute
available almost everywhere (we need it for Proton)

See (for example):
https://wiki.qt.io/PySide2 / https://wiki.qt.io/PySide6  - Qt gui
https://github.com/ValvePython/steam - should do most of the work (not Valve project, just named like this)
https://github.com/ValvePython/steamctl - cli (w/o launcher)

The old codebase was basically just an initial test hacked together fast. Still the original source code can be used by someone smarter than me to potentially code a native Qt/GTK app. C# is also great since it has SteamKit2 and DepotDownloader, so we could potentially change download and verify code ourself. Also I want to make a client I can daily drive, and that leaves no room for bugs. And I also want the client to look better, maybe even clone the new UI somewhat, just more performantly.

While C# is a better choice to not deal with UB, raw memory and threads, it's still mostly controlled by a single entity (Microsoft) and to me was always a pain to run anything. Development tools for C# on Linux are varying from non-existent to barely functioning.
Sorry, didn't want to sound like a fossbro, but C# is still a concern.

Just for the sake of it, I'll mention such an option:

core in Rust
GUI via

GTK (Rust bindings are in good shape, there is also relm4 built on top of those bindings)
Qt5 or Qt6 via PySide (official Python bindings for Qt)

Yeah, can't say this is likely, but Rust is great with its safety and performance / fearless concurrency and C interop if needed etc. while GTK/Qt are classical GUI choices on Linux.

While C# is a better choice to not deal with UB, raw memory and threads, it's still mostly controlled by a single entity (Microsoft) and to me was always a pain to run anything. Development tools for C# on Linux are varying from non-existent to barely functioning.

If the dev is confortable with avalonia, he should use it, but separating the core would help alternate front-ends a lot.

Something tells it's going to be a heated discussion.

I didn't pick python as I've never really liked it and (sorry to all devs I may offend) they always seem to have issues. I absolutely love how convenient bottles is, but sometimes it just breaks (not an issue with bottles itself, rather some packaging issue?) and it requires a ton of AUR dependencies. As for some of your concerns:

The code is split into OpenSteamworks, Common, ClientUI and ClientConsole. Common aims to implement bootstrapping and most of the client code, where ClientUI and ClientConsole just essentially implement "ViewModels" style things
I develop the client solely on linux, but I will test it on Windows as well, so yes good editors exist (I use VSCode)
Tooling so far has been fantastic to work with

Moving away from C++/Qt is a big mistake and frankly, kills my interest in this project stone dead.

While C# is a better choice to not deal with UB, raw memory and threads, it's still mostly controlled by a single entity (Microsoft) and to me was always a pain to run anything. Development tools for C# on Linux are varying from non-existent to barely functioning.
Sorry, didn't want to sound like a fossbro, but C# is still a concern. All in all I still agree with you, DepotDownloader for example is a pretty cool program and I used it to get game data on platforms that can't run Steam. I found out about this project today and didn't knew you planned migration to dotnet.

@a1batross in this case, everything is true, you said. Why use C# in linux. It's such an amazing choice.

Avalonia is great, but unfortunately doesn't adapt to desktop consistency, would you consider making an adapter for GTK4 with something like Nickvision's Aura or GTK4DotNet? It's an (admittedly borderline unreasonable) tall ask so I'd completely understand if this is out of scope, but having a truly native Steam client for desktop Linux with GTK4 and Libadwaita would be amazing.


DRAFT: Bump registry-creds image version to 0.2.6 in manifest.yaml

Description
Bump the version of ghcr.io/alexellis/registry-creds in manifest.yaml to 0.2.6
Which issue # does this fix? And was it approved before you worked on it?
Fixes #24
Checklist:

[X] Fixes issue: #24
[ ] I waited for approval before creating this PR

Note: if the PR is for a typo, please close it and raise an issue instead.
How Has This Been Tested?

I validated that the behavior experienced in #24 was not happening when running make install && make run after cloning the repository, which told me it was likely due to an old version of the controller
Indeed, the version of the controller specified in manifest.yaml was 0.2.5, when the latest version is 0.2.6.
Updating manifest.yaml to reference v0.2.6 of the controller image, and using that file when running kubectl apply fixed the issue.

How are existing users impacted? What migration steps/scripts do we need?

Users that are following the README get v0.2.5 of the controller, which does not watch for service account creation
No migration steps should be necessary

Checklist:
I have:

[X] updated the documentation and/or roadmap (if required)
[X] read the CONTRIBUTION guide
[X] signed-off my commits with git commit -s
[X] added unit tests (N/A)

Does this version need to be changed as well? https://github.com/RothAndrew/registry-creds/blob/6a6ec4c938d430b487709395a4933aea45074fc0/manifest.yaml#L12

Now I'm thinking manifest.yaml is a generated file, but I don't know how to regenerate it with the new tag


Testing, please ignore. (mrtvze)

Bug bounty test - please ignore.... Please DO NOT APPROVE THIS! (mrtvze)

Hi @mourlijuao, thank you for your interest in vcpkg!
Our CI resources are intended only for contributions to the vcpkg catalog and not for security investigation. If you'd like to


[AddressLowering] Consider opening instructions of all local archetypes.

When determining where the "latest opening instruction" is, consider not just OpenedArchetypeTypes but any LocalArchetypeType which includes PackArchetypeTypes.

@swift-ci please test


Fix USDC position value not loading bug

The usdc position of the user has been traded to usdt by the account manager. In the Binance API, we must trace the userAsset API to acquire the new symbol user-traded.
This is the same issue as issue #24


userCanPay(PayProvider.apple_pay) return always false to me

The app is working fine in simulator with simulated card but when i running same code in physical devices, then userCanPay(PayProvider.apple_pay) always retruning false.

@DineshTaral12 Do you have apple pay configured/enabled on your real device?

Hi @DineshTaral12, take a look at the canMakePayments call on the Apple Pay docs. In short,  your device needs to be able to make payments, and have cards that match the payment methods supported if you set the supportedNetworks property in your configuration.


[full version] Toolbar icon gets count badge even when notifications are muted

I noticed that the toolbar icon had a badge with a count of 1, but these are the only extensions with counts against them in the panel:

They all have notifications muted — all the other extensions have a count of 0 since I last hit Reset.
I'm not sure which of these is happening:

Something with notifications enabled is making a connection that gets shown in the badge counter and not in the individual extension's counter.
Something with notifications disabled is incrementing the badge counter anyway in some circumstances.

The number in the counter is not equal to the numbers against any of the extensions, so it only seems to be happening occasionally.

The count appeared again after a reset, and the only extension to have a non-zero count was Bitwarden, so I guess that's the one 😀


feat(client): add implemenntation of classes related to SearchResult

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 61 (18.03%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 85.874%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tensorbay/client/search.py
11
61
18.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
7848

Relevant Lines:
9139

💛  - Coveralls


Add scheduling algorithms in java.

Please assign this issue to me and I would love to work on this for hacktoberfest.
Happy Hacking.

Closing in favor of #80 .
Thank you for showing interest in contributing. 😃
@jinicode please create a PR having the algorithms implemented, I'll merge them.
Also have a look at: https://github.com/amanmehara/programming/blob/master/CONTRIBUTING.md


How to calculate all 3D points at once using the depth image

Hi,
I would like to know how to calculate 3D point data (X,Y,Z distance) at once using depth image given by the realsense camera.
python_code_
depth_point = pyrealsense2.rs2_deproject_pixel_to_point(depth_intrinsic, pixel coordinates, depth_distance_to pixel)
depth_point shape is (3,1)
This function needs to run in a double for loop through rows and columns in order to get the 3D points for all depth image pixels.
Is there any function or matrix operation, that I can use to get the 3D point matrix for the whole image without using for loops? (lets say the function should output (3,rows,columns) shape matrix which includes X,Y,Z distances for all pixel values)
Thank you

@Uvindu  Did you get it through with dorodnic's reply? Looking forward to your update. Thanks!

@Uvindu Could you please update?Thanks!

@Uvindu Any update from your side? Thanks!

@Uvindu  If no other questions or issues about this, will close this. Thanks!


fix continuous integration

Development version tests will be broken until Terra 0.21 is released; see #6


add google site verification

Purpose
add google site verification for 3.2.0

Related to https://github.com/wso2/docs-apim/issues/4036


Refactor public profile

@vi-fu  Just to make you aware of some refactoring and design changes I did.


Polyfill typings: IDBKeyRange only refers to a type

Hey there! I'm using this module in a bit of a strange way, and am having some trouble with typings.
I'm importing the polyfill in test files. The reason for this is that I am re-exporting this package to a NPM distribution using dnt, in a context where IndexedDB will already be provided for.
Once I've imported the polyfill, typings for IndexedDB more or less work in other files, even without importing anything. Except for one thing:
TS2693 [ERROR]: 'IDBKeyRange' only refers to a type, but is being used as a value here.
      const range = IDBKeyRange.bound(
                    ~~~~~~~~~~~

I can fix this by importing the ponyfill, but then the types clash with the built-in ones from TS when I port this through dnt. Is there something I'm missing?

Sam, which version are you using? Please make sure it's the latest at https://deno.land/x/indexeddb

This is when I’m using v1.3.4.

On 18 Aug 2022, at 22:51, Aaron Huggins @.***> wrote:
Sam, which version are you using? Please make sure it's the latest at https://deno.land/x/indexeddb https://deno.land/x/indexeddb
—
Reply to this email directly, view it on GitHub https://github.com/aaronhuggins/indexeddb/issues/3#issuecomment-1219949220, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAENPI7PFSXII5O2VFHTGYLVZ2O5LANCNFSM566EBAGQ.
You are receiving this because you authored the thread.

Sam, please try 1.3.5 when you have a chance. Should show up in a few minutes.

Thank you, 1.3.5 fixed it!


Added Sphinx Documentation

Added Initial Scripts for Sphinx Documentation. w.r.t (#54)
@oke-aditya

Can you remove the existing CI action for docs and fixup flake8. Rest I feel it is good to go.
In next PR we can add new CI action and deploy the docs


vector: 0.24.0 -> 0.24.1

Description of changes
vector update, the changelog hasn't been published yet, probably a bug fix.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

simple bug fixes for this release https://vector.dev/releases/0.24.1/
I won't wait before merging, but happy to take care of feedback later.


Rename Application

Given the enhanced functionality that is to come, the name of the bot doesn't fully convey its usefulness.

Won't do at this point in time.

Renamed application and updated all functionality to reflect new repository name.


no need in unsafe in topic name - deep copying

I prefer https://github.com/ozontech/file.d/pull/54/files over this one

LGTM


Segfault in align with a debug build

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 18)

Kernel Version (Linux Only)
4.15.0-54

Platform
PC

SDK Version
2.24.0

Language
C++

Segment
AR

Issue Description
When using a debug build with -D_DEBUG, align frames to depth (or colour) results in a segmentation fault whilst performing an atomic operation inside a shared pointer of a stream profile. The relevant file is rs_frame.hpp line 376 (profile = other.profile;). In release mode there are no problems.

@knicos I tried the latest SDK v2.28 and FW <IP_ADDRESS> and didn't get the issue. Could you please try the latest SDK and FW to see if any luck? Thanks!

@knicos Could you please update if you still get issue with the latest SDK and FW? Thanks!

I updated to 2.28.1 and the latest FW update but I do still have the segfault in the debug build. It does not appear to be resolved for me.

@knicos , is this reproducible with the SDK tools/examples, such as rs-align ?
If not -  can you share the reproduction snippet?

The examples do work correctly. We are using it as a part of a much larger application that has a lot of threading with no guarantee that it is the same thread calling align.process for each frame. The application also uses OpenCV and CUDA extensively in case this matters. Unfortunately I don't have time to construct an example to reproduce it and can't provide our full source.
In a class constructor:
rs2::config cfg;
cfg.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
cfg.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_BGRA8, 30);

rs2::pipeline_profile profile = pipe_.start(cfg);
rs2::device dev = profile.get_device();
rs2_intrinsics intrin = profile.get_stream(rs2_stream::RS2_STREAM_DEPTH).as<rs2::video_stream_profile>().get_intrinsics();

rs2::depth_sensor ds = dev.query_sensors().front().as<rs2::depth_sensor>();
scale_ = ds.get_depth_scale();

And in a process frame method:
rs2::frameset frames;
if (!pipe_.poll_for_frames(&frames)) return false;
frames = align_to_depth_.process(frames);

The last line is the problem in debug mode.

Ok, please ignore my previous comment. The examples DO NOT WORK if -D_DEBUG is passed to compiler. rs-align then segfaults.

Did you append a cmake generated makefile to add the -D_DEBUG flag? I want to reproduce the segfault you're seeing

@knicos If you're thinking to build debug version, -D_DEBUG flag is not the right one. I think you should build with **-DCMAKE_BUILD_TYPE=Debug  **
Could you please have a try and update? Thanks!

I do use -DCMAKE_BUILD_TYPE=Debug, and that alone seems ok. However, other code we have, in libraries that are not our own, uses _DEBUG and we need to enable it.

#2567

@knicos Could you please try gdb to get the core dump information about the Segfault? Thanks!

@knicos Did you get chance to try gdb to get the core dump information? Looking forward to your update. Thanks!

@RealSenseCustomerSupport Have you checked my gdb log or try to combine Realsense with OpenVINO on Raspberry Pi 3/4?
I am still stuck with the Segmentation fault problem of the Alignment API. As a result, I cannot use for  Intel device for my project.

@whatisor Your issue seems closed on github. Please create another new issue if it's not resolved. Thanks!

@knicos  Any update from your side? Thanks!

Any idea how to compile rs-align? Don't remember what I did last time but 2.31.1 doesn't compile on my machine, it fails to find libusb (it is installed), attempts to compile an internal version which also fails (missing config.h). I can't spend time on this so if it doesn't work I can't help further.
I can give you this from our own application which still crashes in debug (with _DEBUG) but the error is different (stack smashing detected).
#0  0x00007fffda6f1e97 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:51
#1  0x00007fffda6f3801 in __GI_abort () at abort.c:79
#2  0x00007fffda73c897 in __libc_message (action=action@entry=do_abort, fmt=fmt@entry=0x7fffda869988 "*** %s **: %s terminated\n") at ../sysdeps/posix/libc_fatal.c:181
#3  0x00007fffda7e7cd1 in __GI___fortify_fail_abort (need_backtrace=need_backtrace@entry=false, msg=msg@entry=0x7fffda869966 "stack smashing detected") at fortify_fail.c:33
#4  0x00007fffda7e7c92 in __stack_chk_fail () at stack_chk_fail.c:29
#5  0x00007ffff5ab8608 in librealsense::stream_filter_processing_block::should_process(rs2::frame const&) (this=, frame=...) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/proc/synthetic-stream.cpp:308
#6  0x00007ffff5ab94dc in librealsense::generic_processing_block::<lambda(rs2::frame, const rs2::frame_source&)>::operator() (source=..., f=..., __closure=0x55555712bc78)
at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/proc/synthetic-stream.cpp:69
#7  0x00007ffff5ab94dc in rs2::frame_processor_callback<librealsense::generic_processing_block::generic_processing_block(char const)::<lambda(rs2::frame, const rs2::frame_source&)> >::on_frame(rs2_frame *, rs2_source *) (this=0x55555712bc70, f=, source=)
at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/obj-x86_64-linux-gnu/../include/librealsense2/hpp/rs_processing.hpp:128
#8  0x00007ffff5ab60a7 in librealsense::processing_block::invoke(librealsense::frame_holder) (this=0x555556853f80, f=...) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/proc/synthetic-stream.cpp:43
#9  0x00007ffff5c223f1 in librealsense::synthetic_sensor::<lambda(librealsense::frame_holder)>::operator() (__closure=, f=...) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/sensor.cpp:1473
#10 0x00007ffff5c223f1 in librealsense::internal_frame_callback<librealsense::synthetic_sensor::start(librealsense::frame_callback_ptr)::<lambda(librealsense::frame_holder)> >::on_frame(rs2_frame *) (this=, fref=) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/types.h:935
#11 0x00007ffff5c3331b in librealsense::frame_source::invoke_callback(librealsense::frame_holder) const (this=0x555557095bc0, frame=...) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/source.cpp:125
#12 0x00007ffff5c187bb in librealsense::uvc_sensor::<lambda(librealsense::platform::stream_profile, librealsense::platform::frame_object, std::function<void()>)>::operator()(librealsense::platform::frame_object, std::function<void()>) (__closure=, f=..., continuation=..., p=...)
at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/sensor.cpp:377
#13 0x00007ffff5c18f27 in std::_Function_handler<void(librealsense::platform::stream_profile, librealsense::platform::frame_object, std::function<void()>), librealsense::uvc_sensor::open(const stream_profiles&):---Type  to continue, or q  to quit---
td::_Any_data &, librealsense::platform::stream_profile &&, librealsense::platform::frame_object &&, std::function<void()> &&) (__functor=..., __args#0=..., __args#1=..., __args#2=...)
at /usr/include/c++/7/bits/std_function.h:316
#14 0x00007ffff5b437ee in std::function<void (librealsense::platform::stream_profile, librealsense::platform::frame_object, std::function<void ()>)>::operator()(librealsense::platform::stream_profile, librealsense::platform::frame_object, std::function<void ()>) const (__args#2=..., __args#1=..., __args#0=..., this=0x555557104890) at /usr/include/c++/7/bits/std_function.h:706
#15 0x00007ffff5b437ee in librealsense::platform::v4l_uvc_device::poll() (this=this@entry=0x555557104730) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/linux/backend-v4l2.cpp:954
#16 0x00007ffff5b44058 in librealsense::platform::v4l_uvc_device::capture_loop() (this=0x555557104730) at /home/jenkins/workspace/LRS_SDK_CI_Debian/src/src/linux/backend-v4l2.cpp:1290
#17 0x00007fffdb11766f in  () at /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#18 0x00007ffff6ddb6db in start_thread (arg=0x7fff2cff9700) at pthread_create.c:463
#19 0x00007fffda7d488f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

Just taking another look. Is this not the problem?
Lines 623 in rs_frame.hpp (release 2.31.1)
#ifdef _DEBUG
        stream_profile profile;
        unsigned long long frame_number = 0;
#endif

If your debug build is without _DEBUG but I have _DEBUG then the rs_frame class is of a different size and hence will corrupt the stack as I'm seeing.

@knicos  Did you install "libusb-1.0-0-dev" for libusb? And did you build SDK from source code or use the DKMS binary? What's CPU of your setup?

@RealSenseCustomerSupport Have you checked my gdb log or try to combine Realsense with OpenVINO on Raspberry Pi 3/4?
I am still stuck with the Segmentation fault problem of the Alignment API. As a result, I cannot use for Intel device for my project.

@whatisor have you tried wrapping the C API in your C++ program? I've run into the same segfaults everyone else has but I've found success in wrapping the C API align functions in my C++ application

@ThomasBrandonD thanks for response.
I think Realsense is C++, isn't it?
I used rs2::align

... don't know (can't check atm, probably the wrong one)
No, always the DKMS binary.
Intel Core i7

I was adding _DEBUG to my own cmake for my own purposes, not rebuilding realsense SDK with it. Hence the conflict.

@whatisor
It is, but you can still use C in your C++ app, check out https://stackoverflow.com/questions/1041866/what-is-the-effect-of-extern-c-in-c
I'm assuming you're using some API to modify your the image frames you're getting from the realsense (like OpenCV). You can just use the Realsense C API to get the data into a cv::Mat
If that makes sense give it a shot or I can make a small example

@ThomasBrandonD , Do you mean issue only occurs for C++ API of realsense?

@whatisfor Yep, it doesn't exactly "solve" the issue of this ticket but if you need a solution, using the C API is one work-around you might try which worked for me

Hi,
Will you be needing further help with this?
If we don’t hear from you in 7 days, this issue will be closed.
Thanks


WIP: Param backend replace utarray with HashMap

Utarray uses the struct memory address as a key in a hashtable. This means we have to pass around the object instances to reference things like params and topics. This switches the backend to HashMap that does string based hashing. It will allow us to directly implement a ROS like api for param management.
I plan to keep everything 100% backwards compatible, but change the param handles to be strings internally and for new code to just uses the strings directly.
@stephanbro FYI
TODO

[x] Add HashMap submodule
[x] Move param.c to param.cpp
[ ] Create HashMap for params
[ ] Change backend to use HashMap

Useful to know: gcc does string deduplication (-fmerge-constants): https://gcc.gnu.org/onlinedocs/gcc-4.0.3/gcc/Optimize-Options.html, so 100 equal string literals will not use more Flash.
LLVM does that as well.

@LorenzMeier unclear about your comment here: https://github.com/PX4/Firmware/pull/6283#issuecomment-272396707
Do you think I'm headed in a good direction with this PR?

I just added jinja template generation for the params here. This is so I can easily swap the template now and generate the HashMap data structure.

Oh, I assumed jinja was for sitl_gazebo. Looks like I added it to the wrong image.

This now has 8 bytes/param storage size. I don't need to store the char * for the name since I'm using the dict. So (4 bytes for  float/int32 storate), 1 byte for type (uint8_t), and then 3 bytes for alignment that we could store something else there if we wanted. The existing param struct has (4 bytes for float/int32 storage), (4 bytes for const char *, and 1 bytes for type, so I think it takes up 12 bytes/param, need to check.

@dagar did I not reference docker correctly? https://github.com/PX4/Firmware/pull/6305/commits/9644f7bf9606cefde205df7ba46d94d61bde4fa0
I'm still getting issue where jinja2 is not found.

The existing param struct has (4 bytes for float/int32 storage), (4 bytes for const char *, and 1 bytes for type, so I think it takes up 12 bytes/param, need to check.

@jgoppert rebasing on master will pick up @dagar wonderful Makefile changes and we can see how the sizes change.

@jgoppert unfortunately some of it still needs to be configured on the semaphore interface. Travis-ci will need jinja installed for mac. They should all be fixed shortly.

@jgoppert should be good to go after a rebase.

@davids5 Is there somewhere I can look in the bulid log to see the size delta or do I just need to upload and check with free. Also currently this has both param systems since I haven't changed the guts of param.cpp to use the new param dictionary and am trying to do this incrementally.

@jgoppert Flash can be checked on semaphoreci - in the Build Sizes
make sizes - will also give you flash usage
For free RAM - you need to load and use free on nsh.

@dagar Once the docker image is updated on master it will need the px4tools api change for the CI: https://github.com/PX4/Firmware/pull/6305/commits/28d234af3f6c7fa2c9c8062b5fb037fb3d309f8a

The last bit of this will require some thought. I have everything working but need to decide how to handle the existing API.

Closing PR, but keeping the branch.


Add link for Linux

Hi @dlech, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Thanks for the PR! Merged in c762e19936920bcfce16054b6dc9921ee6d83199


Add computed genotyped likelihood

Here I added the computed genotyped likelihood file for the individual to be imputed for Glimpse modules.
The index is also added.
The detail of the procedure is present in the branch imputation of test-datasets.

Thanks for the fast review !


Can't get status from webserver

By reading the documentation and source code it looks like you can't get the status (as saved in the state json file) from the web server.  Is this by design?  If not do you have a preferred path part of the url you would like to see this implemented under?

Sorry I should indicate my use case:  I want to write a nagios check that confirms the service is up and everything is okay.

Good point, it's indeed not possible to get a status right now, but you could send a request to list your basebackups to make sure the object store interfaces are up and running, assuming your pghoard site is default:
curl http://<IP_ADDRESS>:16000/default/basebackup

To answer the original question I'd be fine with both fetching the status and the method/path looking something like GET /status

I plan to return the json state file in the response; should I open the file, read the contents into a variable and pass that variable to HttpResponse or is there a better way that fits into the existing architecture?


UseIdentityServerBearerTokenAuthentication - Not raising failure events in IdentityServer

Hi,
I have a WebApi that is configured to use the "UseIdentityServerBearerTokenAuthentication"
appBuilder.UseIdentityServerBearerTokenAuthentication(new IdentityServerBearerTokenAuthenticationOptions
{
Authority = configurationManager.SecurityTokenServiceAuthority,
ValidationMode = ValidationMode.ValidationEndpoint,
RequiredScopes =  configurationManager.RequiredScopes
})
and i have a Security Token Service configured to listen out for failed events
EventsOptions = new EventsOptions()
{
RaiseErrorEvents = true,
RaiseFailureEvents = true,
}
When an unauthorised request is made to the the WebApi (i.e access token has invalid scope), is there a way for the Security Token Service to register the unauthorised failed event so that i subsequently raise a security event, using a custom event service.
Or is this not designed for this?

The introspection endpoint might have all the eventing you need:
https://github.com/IdentityServer/IdentityServer3/blob/master/source/Core/Endpoints/Connect/IntrospectionEndpointController.cs
By not setting a client id / secret on the validation MW, you are using a legacy validation endpoint. When you set the id and secret - you start using introspection. Give it a try.

thank you, that worked! :)


feat: direct contract call with dynamic link

Description

Closes #144, https://github.com/line/wasmvm/pull/55, https://github.com/line/lbm-sdk/pull/427
It's a poc implementation.
Performance, Exception handling, Gas consumption, security checks, unit tests, CI, etc. need to be checked.
These checks will be registered as separate PRs targeting dynamic_link branch.
Types of changes

[ ] Bug fix (changes which fixes an issue)
[x] New feature (changes which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[x] I followed the contributing guidelines.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] The PR title and commits are followed conventional commit form.

@shiki-tak
I removed unused wanrings in the sample contracts, and Removed unnecessary content by template.
Please review again, thks!


Fix typo

Motivation:

Make better documentation.

Modifications:

Fix log message, comments and codes.

Result:

No impact

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@hexoul Could you sign the ICLA?

@trustin Sure. I did right now. Thanks for checking.

Thanks, @hexoul !


Update codecov coverage

#879  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alanvardy and the rest of your teammates on  Graphite


Doesn't work for Canada, SSL Error

Get this in the log
error during setup: Cannot connect to host www.mybluelink.ca:443 ssl:default [[SSL: WRONG_SIGNATURE_TYPE] wrong signature type (_ssl.c:997)], waiting 1 minute and trying again

you should find this fixed in version 1.6.6


[ServiceBus] Update

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1053

Deprecated by https://github.com/Azure/azure-sdk-for-python/pull/1053


Making defined routes and auto routing work together

The documentation has a note saying For security reasons, if a controller is used in the defined routes, Auto Routing (Improved) does not route to the controller.
Can anyone please explain why is making both work together a security issue? In my opinion it would be a very user friendly feature.

If you really want to use defined routes and auto routing together, you could use Auto Routing (Legacy).
But, for example, see this tutorial. If you build the tutorial app with Auto Routing (Legacy), an attacker could get list of users without token.

No I can understand the issues in Legacy Auto Routing. That's why I moved to the improved one.
Now let's say I have 10 methods in a controller, and for only one of them, I want to use a defined route. Now is there any way(probably regex route or something) I can make the other methods keep working as they usually do?

I have 10 methods in a controller, and for only one of them, I want to use a defined route.

How do I know which method is used for the defined route when I look at the controller code?
And why do you need to use defined route only for the method?
In order to prohibit such complicated configurations, auto routing improved does not work for the controller if even one method is registered to a defined route. This is because allowing complicated configurations makes it easier for configuration errors to occur. Keep it simple.
If you use defined route, you must define all routes for the controller.
If you use auto routing improved, you cannot use defined route at all for the controller.
This is much simpler and easier to understand than the behavior of stopping auto routing only for the method registered in a defined route.
My recommendation is one (action) method in one controller. If you follow this practice, you never have 10 methods in a controller.

We use GitHub issues to track BUGS and to track approved DEVELOPMENT work packages. We use our forum to provide SUPPORT and to discuss FEATURE REQUESTS.


Algo quick sort in java

Quick Sort Algorithm in Java
The provided Java code implements the Quick Sort algorithm, which is a popular and efficient sorting technique.
Quick Sort Overview
Quick Sort is a divide-and-conquer sorting algorithm that works by selecting a 'pivot' element from the array and partitioning the other elements into two sub-arrays according to whether they are less than or greater than the pivot. The process is then repeated on each sub-array.
Code Explanation
public class QuickSort {
    public static void quickSort(int[] arr, int low, int high) {
        if (low < high) {
            int pi = partition(arr, low, high);

            // Recur for sub-arrays on left and right of the pivot
            quickSort(arr, low, pi - 1);
            quickSort(arr, pi + 1, high);
        }
    }

    // Helper method to partition the array
    private static int partition(int[] arr, int low, int high) {
        // Choose the pivot as the last element in the array
        int pivot = arr[high];
        int i = low - 1;

        for (int j = low; j < high; j++) {
            if (arr[j] < pivot) {
                i++;

                // Swap arr[i] and arr[j]
                int temp = arr[i];
                arr[i] = arr[j];
                arr[j] = temp;
            }
        }

        // Swap arr[i+1] and arr[high] to place the pivot in its correct position
        int temp = arr[i + 1];
        arr[i + 1] = arr[high];
        arr[high] = temp;

        // Return the position of the pivot
        return i + 1;
    }

    public static void main(String[] args) {
        // Example array to be sorted
        int[] arr = {5, 2, 8, 1, 3};

        System.out.println("Original array: " + Arrays.toString(arr));

        // Call the quickSort method to sort the array
        quickSort(arr, 0, arr.length - 1);

        System.out.println("Sorted array: " + Arrays.toString(arr));
    }
}

Explanation

quickSort is the main method that recursively sorts the sub-arrays. It chooses a pivot and calls the partition method.
partition selects the pivot (here, the last element), rearranges the array so that elements smaller than the pivot are on the left, and elements greater than the pivot are on the right. It returns the position of the pivot.
The main method demonstrates how to use the quickSort method by sorting an example array of integers.

This Quick Sort implementation efficiently sorts an array in-place by selecting a pivot and dividing the array into two sub-arrays, ultimately achieving the sorted result.

I have accepted your suggestion and ave followed your valid argument in a new PR


feat: use py limited ABI

fixes #175

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

Quick base for a Limited API build; this creates both a correctly named wheel, with correctly named insides seen with unzip -l, when I run pipx run --spec build pyproject-build. I've reworked the wheel building just a bit to make it easier to at test with a later version of Python (probably you want to run tox but load the wheel each time, perhaps? Or just load on 3.9 and verify it passes PyTest. This part depends on the project specifics).
If you can clean up the setup.py, go ahead, don't want to get in the way of that. :)

Also, if you want to support PyPy, you'd need to disable this based on PyPy vs. CPython, and add pp3* to CIBW_BUILD, I think. It wasn't supported before so didn't try to support it here; PyPy currently just doesn't build the speedups, AFIACT.

Thanks! I already have these changes in my branch, just missing the line in setup.py: https://github.com/pallets/markupsafe/tree/new-cibuildwheel-features, so I'll close this and update my branch. Also got some other new features like MacOS universal2.
Re: PyPy, based on https://github.com/pallets/markupsafe/issues/176#issuecomment-774609877 I determined that the speedups were actually slower on PyPy, so I'm leaving it to install from the sdist for now.

No problem, just meant to be an proof of concept / illustration. :)

You can remove setup-python from the workflow, it's not needed (completely ignored, technically) for the cibuildwheel action.


Support ES username/password change dynamically

By following #4475, the ES username/password could be changed through file, like using Vault.
Provide a properties format and being watched file, which could be changed from time to time by 3rd party tool. Once the file content changed, the ES storage will trigger ES reconnection.

tested user/password changed scenario, and LGTM:
······
	Caused by: org.elasticsearch.client.ResponseException: method [POST], host [http://localhost:9200], URI [/profile_task/type/_search?typed_keys=true&ignore_unavailable=false&expand_wildcards=open&allow_no_indices=true&search_type=query_then_fetch&batched_reduce_size=512], status line [HTTP/1.1 401 Unauthorized]
{"error":{"root_cause":[{"type":"security_exception","reason":"failed to authenticate user [elastic]","header":{"WWW-Authenticate":"Basic realm=\"security\" charset=\"UTF-8\""}}],"type":"security_exception","reason":"failed to authenticate user [elastic]","header":{"WWW-Authenticate":"Basic realm=\"security\" charset=\"UTF-8\""}},"status":401}
······
org.apache.skywalking.oap.server.library.client.elasticsearch.ElasticSearchClient - 176 [pool-2-thread-1] INFO  [] - elasticsearch cluster nodes: localhost:9200

provided.


docs: update readme

Summary
Updated README with new installation steps and feature list
Changes

new feature list remarks and rejection , new version explanation of sign and verfication

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tests fail with latest mypy

As pointed out here, test_black.py has some type errors that only show up on the latest version of mypy.

There's a quick fix in #409 which is basically to specify the version in the pip install commands for the CI config files. A better one would be to get CI to use pipenv. But until this issue is resolved, all builds will break.


Errors trying to download using Swift Package Manager

When trying to do a swift build with Starscream in my Package dependencies I'm seeing this error:
error: the package has an unsupported layout, unexpected source file(s) found: .../Packages/Starscream-2.0.0/Tests/StarscreamTests.swift
fix: move the file(s) inside a module

Hrm, not sure. I'll have to checkout the package manager again, haven't touched it since Swift 2.X.
@nuclearace maybe I am misremember, but aren't you a Mac/Swift Package Manager user? I'll look into it to, just didn't know if you had any ideas off hand.

Yeah I saw this when working on my new Discord library. Seemed to happen on the latest preview release of swift and the pm. Didn't make much sense to me deleting the test folder and rebuilding is a workaround

I think you need to move the file in Tests into its own folder. Like Tests/StarscreamTests/*.swift

#277 should have fixed this (in master now).

Thanks for resolving this. It seems Swift Package Manager still picks tag 2.0.1, so the error continues to exist until a new release.

2.0.2 released.


Bug Report: The game crashes

The game crashes when I run it

I compile this myself

Anyone can help me?

Can you please attach console logs so I can see what's the cause of the problem? If there is no console logs, you may be compiling on the release build, you'll have to instead compile on the debug build.

Called from polymod.backends.LimeBackend.init (polymod/backends/LimeBackend.hx line 127)
Called from polymod.backends.PolymodAssets.init (polymod/backends/PolymodAssets.hx line 133)
Called from polymod.Polymod.init (polymod/Polymod.hx line 252)
Called from TitleState.create (TitleState.hx line 67)
Called from flixel.FlxGame.switchState (flixel/FlxGame.hx line 637)
Called from flixel.FlxGame.create (flixel/FlxGame.hx line 360)
Called from openfl.events.EventDispatcher.__dispatchEvent (openfl/events/EventDispatcher.hx line 402)
Called from openfl.display.DisplayObject.__dispatchEvent (openfl/display/DisplayObject.hx line 1417)
Called from openfl.display.DisplayObject.__dispatchWithCapture (openfl/display/DisplayObject.hx line 1478)
Called from openfl.display.DisplayObjectContainer.addChildAt (openfl/display/DisplayObjectContainer.hx line 266)
Called from openfl.display.DisplayObjectContainer.addChild (openfl/display/DisplayObjectContainer.hx line 155)
Called from Main.setupGame (Main.hx line 72)
Called from Main.init (Main.hx line 51)
Called from openfl.events.EventDispatcher.__dispatchEvent (openfl/events/EventDispatcher.hx line 402)
Called from openfl.display.DisplayObject.__dispatchEvent (openfl/display/DisplayObject.hx line 1417)
Called from openfl.display.DisplayObject.__dispatchWithCapture (openfl/display/DisplayObject.hx line 1478)
Called from openfl.display.DisplayObjectContainer.addChildAt (openfl/display/DisplayObjectContainer.hx line 266)
Called from openfl.display.DisplayObjectContainer.addChild (openfl/display/DisplayObjectContainer.hx line 155)
Called from Main.main (Main.hx line 27)
Null Object Reference
this?

This appears to be a common issue that happens with Polymod installations sometimes. Follow the instructions I give here to solve your problem.

okk

Closing due to inactivity.


LazyVerticalStaggeredGrid two columns with five items with difference horizontalArrangement, the last item sometimes on the left, sometimes on the right

Describe the bug
LazyVerticalStaggeredGrid two columns with five items with difference horizontalArrangement, the last item sometimes on the left, sometimes on the right
i only have android. i only runing on android. did i post to Jetpack Compose?
Affected platforms
android
Versions

Libraries:

Compose Multiplatform version: 1.6.10

Kotlin version:  2
OS version(s) (required for Desktop and iOS issues): api29
OS architecture (x86 or arm64): x86
Device (model or simulator for iOS issues):  simulator

To Reproduce
Steps to reproduce the behavior:

Run this code snippet:

@Composable
@Preview
fun App() {
    MaterialTheme {
        val paddingDp = 1.dp // <----------
        LazyVerticalStaggeredGrid(
            columns = StaggeredGridCells.Fixed(2),
            verticalItemSpacing = 2.dp,
            horizontalArrangement = Arrangement.spacedBy(paddingDp),
            contentPadding = PaddingValues(
                bottom = paddingDp,
            ),
        ) {
            item(
                span = StaggeredGridItemSpan.FullLine
            ) {
                Text(
                    text = paddingDp.toString()
                )
            }
            // SightCards
            (1..5).forEach { _ ->
                item {
                    Box(modifier = Modifier
                        .aspectRatio(1F,)
                        .background(Color.Black)
                    )
                }
            }

        }
    }
}

i only have android. i only runing on android. did i post to Jetpack Compose?

Yes please report about this there. It was even mentioned in issue template: "If the bug is Android-only, report it in the Jetpack Compose tracker"
Compose Multiplatform uses original Google's binaries on Android (no changes, even no re-compilation), so it's not something that caused by changes in JetBrains fork.

@MatkovIvan i repeat it on windows, this issue can reopen?

Our changes here won't affect Android. Please open the issue in Google's tracker. It needs to be fixed in AOSP, not in fork

it's looks right side boxes is smaller.

In your Android screenshots rounding issues persist too

@MatkovIvan thanks.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Secrets as resource

Remove existing secrets class, instead implementing secrets as a resource, except one that is saved into Vault (instead of RNS) for security purposes. Includes subresource implementation for provider secret type. (Cluster and Env secrets are new concepts addressed in a separate PR)
Secrets Workflow

User constructs a Secret resource, generally in one of the following ways. The .values property of the resource conveniently extract the values from the object, file, or env (python or .env file).

# custom secret -- instantiate with secrets values or path
custom_secret = rh.secret('my_secret', values={"key": "val"})
custom_secret = rh.secret('my_secret', path="values.json")

# passing in provider name; will load values from default path
aws_secret = rh.provider_secret(‘aws’)
aws_secret.values   # {'access_key': 'XXXX', 'secret_key': 'xxxxx'}

# passing in provider name and values, in-memory resource
lamdba_secret = rh.provider("lambda", values={"api_key": "xxxxx"})

To save the secret config (to Vault/local). By default, secret values are saved, but can also save just the metadata by setting values=False.

custom_secret.save()
aws_secret.save(values=True/False)

To write down a secret, if it is an in-memory secret. By default, file overriding is not supported, as we don't want to permanently delete/override existing secrets on user's dev environment. Instead, recommend them to manually update the path+secret, or can set overwrite param to True. If a path different from the secret path is given, will return a copy of the original secret, but with the updated path.

lamdba_secret.write()  # writes down to default lambda path ~/.lambda_cloud/lambda_keys
lambda_secret.write("~/.lambda_cloud/lambda_keys_2")

To send the secret to a cluster, including optionally writing it down to a remote file (True by default).

remote_secret = aws_secret.to(cluster, path=aws_secret.path)
# or, remote_secret = aws_secret.to(cluster).write()
remote_secret.path   # remote File resource where the secrets are saved
remote_secret.values  # {'access_key': 'XXXX', 'secret_key': 'xxxxx'}

To login and write down the secrets (requires having saved the values, otherwise can only extract the metadata). This will also store the path the secrets are stored in the runhouse config.

rh.login(download_secrets=True)

To logout. This goes through the local (?) and previously downloaded secrets (with paths stored in runhouse config), and prompts the user through each one to confirm deletion.

rh.logout(delete_loaded_secrets=True)

Separate PR -- Env + Cluster Secrets and Workflows
pipeline_secrets = ["aws", "openai", "env"]
rh.env(reqs=[...], secrets=pipeline_secrets)  # will set up secrets on the servelet code is run on; replace "env_vars" key

cluster_secret = rh.cluster_secret(values=config_dict)
rh.cluster("rh-a10x", secrets=cluster_secret)
<EMAIL_ADDRESS> # also shares the cluster secrets with them to be able to access cluster

TODO for this PR

Some refactoring/cleanup
Solidifying Secret class methods / utils
Fix test_login

Separate/Futures PRs

rns_server repo: Update Vault saving stuff
Extend “path” to paths – ex/ aws may require multiple files containing different info
Env + Cluster Secrets
Secrets as part of rh.cluster or cluster.share()
Adding secrets to rh.env, e.g. env_var secrets
[?] Is_enabled/from_provider support; using provider APIs to check if keys/secrets are valid
Additional providers support – WandB, OpenAI, Databricks, Anthropic

takeaways from discussion Oct 4

secrets param is confusing, change to values (and by default save these values)
don't override any secrets files, throw error
additional_files param, along w/ mapping in .to
cluster + ssh secrets discussion, design tbd
things to consider in the future -- versioning (vault supports 10 latest), sharing retention/invalidation

Closing in favor of #135 and splitting up Secrets work into more iterative components


User-Agent string is still referencing "insomnia" and out of sync with current version number

Expected Behavior
The default User-Agent string should follow the pattern of insomnium/current version.
Actual Behavior
The default User-Agent string on an HTTP request is referring to "insomnia" rather "insomnium", and the version that appears is still "0.2.0".
Reproduction Steps

Create a new HTTP request.
Click the headers tab.

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
I found this comment from insomnia's issue tracker saying that the intention for the default User-Agent string in a HTTP request is to follow the pattern of program/current version.
For this fork, the program name should be updated to Insomnium to reflect that it is distinct from Insomnia.
It also seems like whatever logic Insomnia had to keep the version number that appears in the User-Agent string in alignment with the current application version number has been out of sync in Insomnium since v0.2.0.
Great work maintaining this fork by the way <3
Insomnium Version
0.2.2
What operating system are you using?
Other Linux
Operating System Version
Operating System: Arch Linux, KDE Plasma Version: 5.27.9, KDE Frameworks Version: 5.111.0, Qt Version: 5.15.11, Kernel Version: 6.1.59-1-lts (64-bit), Graphics Platform: X11
Installation method
AUR
Last Known Working Insomnium version
No response

good find; fixing this now
but doesn't look like there is a straightforward way to update the User-Agent of old requests created in older versions.


Settings Sync: No way to specify a platform specific extension

I am synchronizing settings across Windows and Mac.
On my Windows machine, I have the Remote - WSL extension, which does not make sense to install on my Mac.
There doesn't seem to be a way to specify by platform which extensions to sync (or not sync). ignoredExtensions makes sense if I don't want to synchronize across any machine, but if I add a second Windows machine into the mix (or rebuild a Windows machine), then the WSL extension is not installed.

This gets supported automatically when platform specific extensions is supported.


Mouse without borders - set pointer position when you leave a screen.

Description of the new feature / enhancement
2 laptops using mouse without borders.  When I move the mouse pointer from the secondary machine back to the primary machine (left to right), the pointer on the secondary machine settles at the top centre of the screen.  If Chrome is topmost app and maximised, you get a tab preview.  To stop this from happening, ideally, you should have options what to do with the mouse pointer on exit of the screen.
Scenario when this would be used?
when using mouse without borders and the non focused PC shows content that isn't helpful due to where the app leaves the mouse pointer.
Supporting information
No response

I wanted to suggest a similar thing and in order to not create duplicates I will add my comment here:
Proposed Behaviours:

When the mouse exits the screen from the left, right, top, or bottom, it should be hidden on the previous screen/system.

Additional Considerations:

This could be implemented as an optional setting in the software, allowing users to choose the behaviour that best suits their workflow.


Survey: location to show the webview

Today, the extension has three kinds of webviews which are: Preview, Top Voted Solution & Result. Each of them has its own logic to show in some certain places:

Preview: Determined by the setting leetcode.enableSideMode
Solution: Get the location of Preview page. If there's no opened Preview page, it will show in the active column
Result: Always in Column Two

Here are some proposals for improvement:

Just use current implementation without any change
Always show in the active column
Always show in Column Two
Always fetch the setting leetcode.enableSideMode before showing for all the three webviews
Each webview has its own setting to determine the location

Please let us know which one you are preferred. You can directly add the 👍 in the following choices.
Thank you for your patience.

Just use current implementation without any change
Click 👍 if you vote for this

Always show in the active column
Click 👍 if you vote for this

Always show in Column Two
Click 👍 if you vote for this

Always fetch the setting leetcode.enableSideMode before showing for all the three webviews
Click 👍 if you vote for this

Each webview has its own setting to determine the location
Click 👍 if you vote for this

I want this, just like the web leetcode.


fix: Ensure project can build with Opam on Windows

This actually relies on fixes in conf-python-3 and conf-cmake to land in the mingw repository, so draft for now.
This is all the changes to get this project working from opam on windows.

Looks like those fixes were merged into the mingw repository. Running the CI now, but this should be ready.


The specified deps.json [*.deps.json] does not exist after VS update to 16.4

This issue has been moved from a ticket on Developer Community.

While following this instruction:
https://docs.microsoft.com/ru-ru/aspnet/core/tutorials/razor-pages/model?view=aspnetcore-3.0&tabs=visual-studio
during running “Add-Migration InitialCreate” in PMC i suffered error: The specified deps.json [RazorPagesMovie\RazorPagesMovie\RazorPagesMovie.deps.json] does not exist.
RazorPagesMovie.deps.json file places in “RazorPagesMovie\RazorPagesMovie\bin\Debug\netcoreapp3.0”
Visual studio Community 2019 v. 16.4

Original Comments
Visual Studio Feedback System on 12/4/2019, 08:28 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Could you share a minimal project that reproduces this error?

I am running into same issue.   I have a core 3.1.100 web api, and I'm using VS 2019 16.5 Preview 1.0.
I'm trying to add-migration.   I get the following result in package management console:

..I go to check and sure enough the file is there.   I gave full permissions to Everyone so that should be the issue either.  Please help!   Btw, this application is hosted on a TFS server.   I copied the code to my thumb drive and brought it home to my laptop.   I opened it up and added migration without any issue.  It therefore, seems to be some kind of environment issue.   Could it be that it works when not bound to version control?

@kJacobi can you file a separate issue with a minimal project that reproduces the error?


DNN-8573 Suppress X-Frame-Options header by default.

This change is a bit against the standard recommendations that are being made with regards to cross-site scripting recommendations.  By doing this, we will go back to a situation where a user could embed any page of DNN in another site with an IFrame.
I realize that we did allow htis with prior versions, but I'm curious if this is really the change in behavior we desire?

@mitchelsellers Due to the MVC implementation in 8.0 this header was set by default for ALL requests.  This is standard Asp.Net MVC behavior. This was ignoring any X-Frame-Options that you might have configured in web.config.
This PR fixed the problem where system will respect the web.config setting. However, we take your point that by default there will be no option. We are back to pre 8.0.
We are going to add this option on new installs with value SAMEORIGIN. Users can change to whatever they desire.
Thoughts?
Thanks
Ash

@ashishpd My only thought here is that the default behavior could be used to assume proper security by those that have developed new MVC applications.  I'd almost be in favor of leaving this code change in, however, additionally ensuring that on upgrade "SAMEORIGIN" is set for installations that didn't already override the default behavior.
My concern would be in the following example.
I set up a DNN 8.0.0 site, and everything is secured.  I then upgrade to the latest version, now all of my stuff can be viewed without the proper context.
If we set the value for those that haven't already explicitly set it, we would prevent this change in behavior.

@ashishpd I have modified the code to Suppress X-Frame-Options Header only if there is some configuration specified in the Web.Config for it. It might be remove or add header. In this way it will have same behavior as 8.0 for all the websites without any settings for this header in the Web.config but if someone wants to override this behavior intentionally, they would be able to do it from web.config.
Let me know in case of any questions.
/Amarjit

Changes looks good, merged.


Add hack/update-template-opensuse.sh

$ hack/update-template-opensuse.sh
update-template-opensuse.sh: Update the openSUSE Linux image location in the specified templates

Usage:
  update-template-opensuse.sh [--version-major-minor (<major>.<minor>|current|stable|tumbleweed)|--version-major <major> --version-minor <minor>] <template.yaml>...

Description:
  This script updates the openSUSE Linux image location in the specified templates.
  Image location basename format:

    openSUSE-(Leap-<major minor version>|Tumbleweed)-Minimal-VM.<arch>-Cloud.qcow2

  Published openSUSE Linux image information is fetched from the following URLs:

    Leap:
          <major>.<minor>: https://download.opensuse.org/distribution/leap/<major>.<minor>/appliances/?jsontable
          current: https://download.opensuse.org/distribution/openSUSE-current/appliances/?jsontable
          stable: https://download.opensuse.org/distribution/openSUSE-stable/appliances/?jsontable

    Tumbleweed:
      x86_64: https://download.opensuse.org/tumbleweed/appliances/?jsontable
          not x86_64: https://download.opensuse.org/ports/<arch>/tumbleweed/appliances/?jsontable

  The downloaded files will be cached in the Lima cache directory.

Examples:
  Update the openSUSE Linux image location in templates/**.yaml:
  $ update-template-opensuse.sh templates/**.yaml

  Update the openSUSE Linux image location to version 15.6 in ~/.lima/opensuse/lima.yaml:
  $ update-template-opensuse.sh --version-major-minor 15.6 ~/.lima/opensuse/lima.yaml
  $ limactl factory-reset opensuse

  Update the openSUSE Linux image location to tumbleweed in ~/.lima/opensuse/lima.yaml:
  $ update-template-opensuse.sh --version-major-minor tumbleweed ~/.lima/opensuse/lima.yaml
  $ limactl factory-reset opensuse

Flags:
  --version-major-minor (<major>.<minor>|current|stable|tumbleweed) Use the specified <major>.<minor> version or
                                                                    aliases "current", "stable", or "tumbleweed".
                                                                    The <major>.<minor> version must be 15.0 or later.
  --version-major <major> --version-minor <minor>                   Use the specified <major> and <minor> version.
  -h, --help                                                        Print this help message

This PR aims to add openSUSE Linux support as per https://github.com/lima-vm/lima/issues/1347.

Thanks! 🙏🏻


[Wechat] Fix isWeChatTweakInstalled

Description

Fix isWeChatTweakInstalled function, previous version only tested the first path.
From:
fs.existsSync(
      "/usr/local/bin/wechattweak-cli" ||
        "/usr/local/Cellar/wechattweak-cli" ||
        "/usr/local/opt/wechattweak-cli" ||
        "/usr/local/bin/wechattweak-cli" ||
        "/opt/homebrew/bin/wechattweak-cli" ||
        "/usr/local/bin/wechattweak-cli"
    )

to:
[
      "/usr/local/bin/wechattweak-cli",
      "/usr/local/Cellar/wechattweak-cli",
      "/usr/local/opt/wechattweak-cli",
      "/usr/local/bin/wechattweak-cli",
      "/opt/homebrew/bin/wechattweak-cli",
      "/usr/local/bin/wechattweak-cli"
    ].some(fs.existsSync)

Screencast

Checklist

[ *] I read the extension guidelines
[ *] I read the documentation about publishing
[ *] I ran npm run build and tested this distribution build in Raycast
[* ] I checked that files in the assets folder are used by the extension itself
[ *] I checked that assets used by the README are placed outside of the metadata folder

It's ok to merge the pull,THX @lx200916


new(tests): basic EOF execution tests

🗒 Description

Basic EOF execution tests migrated from ethereum/tests:
EIPTests/StateTests/stEOF/stEIP3540/EOF1_Execution.json (filler source)
Extend execution test of legacy invoking EXTCODE* instruction on EOF with the case from ethereum/tests.

✅ Checklist

[x] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[x] All: Considered updating the online docs in the ./docs/ directory.
[x] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[x] Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
[x] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[x] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

need to merge coverage script update so it wont fail


Add Dockerfile and patches for building Chromium with rav1d

AFAIK, the package dir contains files used to package dav1d. I don't love that we're adding stuff to build and package Chromium here. Wouldn't it be cleaner to contribute these patches to Chromium?


[NEW SHEET] EXALTED FIRST EDITION

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < EXALTED (First Edition)  >

The publisher of this game is: < Paradox Interactive  >

The name of this game system/family is:

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.

Changes / Description

This is the Roll20 character sheet for the first edition of the game Exalted

Character Sheet Info Roll20 Internal Use only.

Hi @VJRolmasters!
Thank you for your work on the Exalted First Edition sheet. This is good to merge but I wanted to double check with you about the spelling of a couple words. From lines 268-276, "Awareness" is missing the first "e" and then "Twilight" is missing the second "i" on lines 457 and 459.
If this isn't a concern, we can get this merged for you. :)
Thanks!
Kayla

Hi Kayla
Thank you very much!
I have corrected the errors
I hope now it's ready to be merged.


[Bug]: invalid state of textContentCache after mutation in text-node

Feature description
I would like to start with apologising for using a feature-template, just the bug-template is not suitable for the problem I’m going to describe below. (please change the label of the ticket to "bug")
Bug: if a mutation occurs within a text-node (type: 3) without a mutation of the parent element (type: 1) in its text, then textContentCache remains in an outdated invalid state (not updated) due to incorrect MutationOverserver settings that are missing this mutation.
Next, the problem affects TextQuerySelector (custom p-selector).
I found the root of the problem:
TextQuerySelector has a condition instanceof Element before calling createTextContent (TextQuerySelector.ts#L51)
Similarly: TextContent.ts#L135 and TextContent.ts#L142
And then the cache begins to observe mutations precisely on this Element (TextContent.ts#L146)
Problem is in settings: { childList: true, characterData: true, }

[characterData]
Set to true to monitor the specified target node (and, if subtree is true, its descendants) for changes to the character data contained within the node or nodes. The default value is true if characterDataOldValue is specified, otherwise the default value is false.
see: https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver/observe#characterdata

childList setting not affect on characterData. And the changes on the text-node, that happens to always be a child of the observer target (always Element in case of TextQuerySelector) and an absence of subtree: true - doesn’t  trigger the mutation in the observer.
Fix: use the following settings { childList: true, characterData: true, subtree: true} here TextContent.ts#L146
I’ve checked the proposed changes locally and after adding a subtree, the search works as expected.
Original (incorrect) state was like this:
page.locator('::-p-text(NewText)').wait() - failed
page.locator('::-p-text(OldText)').wait() - found Element with "NewText" in.
Information about the environment used and a product version were deliberately omitted as it seems obvious that it doesn’t cause the problem.

Hello, thank you for issue and PR a colleague of my was assigned to take a look.
In the mean time, can you provide a minimal, reproducible example that this behavior can be observed?


fixed Grid V2 breakpoint only applied with props order

https://github.com/mui/material-ui/issues/34928
fixed Grid V2 breakpoint only applied with props order #34928

[x] I have followed (at least) the PR section of the contributing guide.

@Falkyouall Thanks for the fix! I really like your solution, so I think this can be used in https://github.com/mui/material-ui/pull/34987 (added you as a reviewer).
I am closing this.

Why not merge this directly from this PR, is there something i can do better?

The test is missing and I told the author of another PR to add it before I saw this PR. Sorry for that.

Thanks for the feedback, i will work on some other issues that hopefully will be merged :-)


feat: add --why flag for zarf dev find-images

Description
Add a --why flag for zarf dev find-images
Related Issue
Fixes #2272
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

Partial (relevant) test output
$ build/zarf-mac-apple dev find-images examples/dos-games --why defenseunicorns/zarf-game:multi-tile-dark --tmpdir /var/folders/v7/yqyq_d6x2996hfnnwvgs5f080000gn/T/zarf-1881051164

 NOTE  Using config file /Users/vibhavbobade/go/src/github.com/defenseunicorns/zarf/zarf-config.toml

 NOTE  Saving log file to
       /var/folders/v7/yqyq_d6x2996hfnnwvgs5f080000gn/T/zarf-2024-02-20-20-44-58-2844959790.log

 NOTE  Using build directory examples/dos-games
  ✔  Looking for images in component "baseline" across 3 resources
  ✔  Looking up cosign artifacts for discovered images (0/1)
component: baseline
manifests: multi-games
resources:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: game
spec:
  selector:
    matchLabels:
      app: game
  template:
    metadata:
      labels:
        app: game
    spec:
      containers:
      - image: defenseunicorns/zarf-game:multi-tile-dark
        name: multi-game
        ports:
        - containerPort: 8000
          name: http
          protocol: TCP
        resources:
          limits:
            cpu: 250m
            memory: 128Mi
          requests:
            cpu: 50m
            memory: 32Mi

I am going to make the following changes

don't mention other components for --why based on https://github.com/defenseunicorns/zarf/pull/2309#issuecomment-1954537274 only show the resources with the relevant image
simplify the logic with a helper function https://github.com/defenseunicorns/zarf/pull/2309#discussion_r1496740223
parse manifests and charts https://github.com/defenseunicorns/zarf/pull/2309#discussion_r1496822220

Updated the PR based on the reviews

@waveywaves there are some merge conflicts that will need to be resolved

@lucasrod16 I will do the needful

seems like the k3d test is flaking, can we restart that test ?

the tests failed both times on kiwix data injection test against a minikube cluster, testing the test locally with minikube

the e2e-test-with-cluster test failed twice at a single kiwix data injection test against a minikube cluster first time the test failed because zarf could not connect to the cluster and right now it has failed because the test could not find a pod which was successfully deployed as seen in the logs below
https://github.com/defenseunicorns/zarf/actions/runs/8013856761/job/21891629494?pr=2309#step:8:5575
running the same possibly flakey TestDataInjection passes locally with minikube
--- PASS: TestDataInjection (47.62s)
PASS
ok      github.com/defenseunicorns/zarf/src/test/e2e    48.707s

Unfortunately that is a known flake - the issue actually isn't with the Kiwix package or Zarf data injections but with how the test expects to be able to lookup the pod afterward to check its contents are there - it assumes it can reuse the same pod but Kubernetes could recycle it in the intervening time and if it does the test will try to reach the wrong pod and fail. We just haven't had time yet to invest in fixing it yet.

If this issue documented ?

@lucasrod16 updated the PR


Add documentation for service userns_mode key

Describe the proposed changes

Add documentation for docker/compose#4216
Unreleased project version

This is for unreleased Docker Compose
Related issue or PR in another project

https://github.com/docker/compose/pull/4216

LGTM