Improving the Cards of our website courses

🛠 Fixes Issue #361

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Please review my PR @arpit456jain @singhmansi25 @KKhushhalR2405 @raghavTinker

this branch is also behind not able to merge

this branch is also behind not able to merge


Istio Performance Dashboard Missing data for release-1.10 and Fortioclient deployment failure

Bug description
Istio Performance Dashboard https://perf.dashboard.istio.io/benchmarks/cpu_vs_conn/ is currently undergoing 2 issues.

Istio Current Release Build does not have dropdown list available for release-1.10

Fortioclient deployment failure since May 22nd for nighthawk and fortio perf test.

nighthawk: https://prow.istio.io/job-history/gs/istio-prow/logs/daily-nighthawk-performance-benchmark
fortio: https://prow.istio.io/job-history/gs/istio-prow/logs/daily-fortio-performance-benchmark

@mandarjog @richardwxn Could you find an owner for the dashboard and help with debug the issue?
Thanks!
[ ] Docs
[ ] Installation
[ ] Networking
[X] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade

this need to be cherrypicked to 1.10 https://github.com/istio/tools/pull/1567
now the prow jobs for 1.10 are failing because of the same reason of master.

FIxed by bump CUR_RELEASE and add the prow job
In the benchmark dashabords, there is release 1.10 data collected. https://perf.dashboard.istio.io/benchmarks/cpu_vs_conn/


4.0.0 beta: changing the Arch/System/GPU causes a crash

Describe the issue
Happens during VM creation, or afterwards when modifying a VM:
Opening the Architecture or System menu works, but selecting a different arch/system causes a crash.
Also, the 'System' field doesn’t show any selection (Img. 1, Img. 2) that may be related.
Configuration (required)

UTM Version: 4.0.0 beta 1
OS Version: 14.8.1
Device Model: iPad Pro 11 2018
Is it jailbroken (name jailbreak used)? No
How did you install UTM? AltStore

Crash log
UTM-2022-08-06-123646.log
Debug log
Not applicable (not related to VM launch)

Actually, this appears to happen for all submenus: even for example the upscaling/downscaling mode causes a crash


get data from api /getInfo and save in user class (add more endpoints if needed)

The puzzle 9-1264d400 from #9 has to be resolved:
https://github.com/ammaratef45/Attendance/blob/6458784661c60ff240bc92347d7306a070af4ac8/attendance/lib/homePage/home_page_viewmodel.dart#L119-L119
The puzzle was created by Ammar Atef on 16-Apr-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@ammaratef45/z please, pay attention to this issue

The job #50 assigned to @ammaratef45/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Job gh:ammaratef45/attendance#50 already assigned to @ammaratef45, can't assign to @ammaratef45

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @ammaratef45/z

@0pdd the puzzle #99 is still not solved; solved: #100, #101.


MiDaS v2.1 small model missing

the link to MiDaS v2.1 small model is missing

You can find it there:

https://github.com/intel-isl/MiDaS/releases/tag/v2_1
https://pytorch.org/hub/intelisl_midas_v2/
https://tfhub.dev/intel/midas/v2_1_small/1


UML-3380 mock data store

Purpose
UML-3380 Mock data store
Approach
Impostor mock similar to IaP mock but based on data-store instead
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] New event_codes have been documented on the wiki page
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] I have notified the Interaction Designer of any content changes so that appropriate screenshots/flow diagram changes can be made
[ ] The product team have tested these changes

Is this missing the mock-config.yaml file?

Yes, I did have one and had failed to push it. 🤦 :)


Add "auth_method" and "auth_source" option to support explicit auth method and auth_source

This PR is based on #49. So, first 6 commits are unrelated ones.
This PR will solve #50
Currently, plugin user have to use uri option when they want to specify auth mechanism and auth source.
in:
  type: mongodb
  uri: mongodb://myuser:mypassword@localhost:27017/my_database?authMechanism=SCRAM-SHA-1&authSource=another_database

They can't specify auth source with separated URI parameters,
To solve this problem, I added 2 new options - auth_method and auth_source.
Plugin user will be able to set auth method and auth source explicitly.
in:
  type: mongodb
  hosts:
  - {host: localhost, port: 27017}
  auth_method: scram-sha-1
  user: myuser
  password: mypassword
  database: my_database
  auth_source: another_database

MongoDB supports various kind of auth methods.
https://docs.mongodb.com/manual/core/authentication/
http://mongodb.github.io/mongo-java-driver/3.0/driver-async/reference/connecting/authenticating/

SCRAM-SHA-1
MONGODB-CR
x.509
Kerberos (GSSAPI)
LDAP (PLAIN)

However, what I supported were only scram-sha-1, mongodb-cr and auto.
When plugin user choose auto,  mongodb-java-client will negotiate the best mechanism based on the version of the server that the client is authenticating to.
If the server version is 3.0 or higher, the driver will authenticate using the SCRAM-SHA-1 mechanism.
Otherwise, the driver will authenticate using the MONGODB_CR mechanism.

@mjalkio Related to #50, I don't have an environment that will use another auth source.
Could you try this PR if possible?
You can build by ./gradlew gem -x test with skipping test, and able to install gem install --local /path/to/embulk-input-mongodb-0.6.1.gem

Hi @sakama, thank you for adding this! I won't have time to test it soon, but it looks good to me.

@mjalkio Got it. Thanks!

@kamatama41 Could you take a look at this PR and https://github.com/hakobera/embulk-input-mongodb/pull/49 when you can?

Sorry for late, LGTM 👍


False positive in plugin-next / no-page-custom-font

Hello,
I am running oxlint v0.3.5 and it's reporting a false positive for no-page-custom-font rule for the following code
<link
  rel="apple-touch-startup-image"
  href="/assets/public/pwa/splash/apple-splash-2048-2732.jpg"
  media="(device-width: 1024px) and (device-height: 1366px) and (-webkit-device-pixel-ratio: 2) and (orientation: portrait)"
/>

As you can see, this is a splash image link element. This is not related to custom fonts at all. Please let me know if there's additional information I can provide here

Thank you for your feedback! You have provided enough information.

Thank you for resolving this so quickly! ⚡


Make mobile nicer, round sizes

Make time a modified ISO format
Make sizes rounded to B, MB, GB, etc.
Add back viewport change for mobile, make filenames (and only filenames) wrap anywhere to prevent horizontal scrolling.

For short filenames this is fine:

When long filenames are addedd, things start to wrap.
However, the time will wrap between the date and time and the size will not wrap at all:

Spaces don't make much difference:

Nor do unicode names:

Looks great, TYSM <3


Explicitly enable yamllint trailing spaces option.

When investigating #30010 I realized that this is something yamllint should also be catching, which would have helped add corroborating signal to offset the noise in the nosetests.
Looking at the yamllint docs, trailing whitespace checks should be enabled by default but aren't behaving that way in 1.26.1. This explicitly enables trailing whitespace checks which might be a good idea while I figure out what's happening in yamllint and share my discovery upstream.

So it's not just the trailing-spaces rule but also several others, however via experimentation I was able to determine that key-duplicates is at least enabled by default, however it's then disabled by my cli config above and so this PR will need to change.

Okay after a lot of time being VERY confused. The problem appears to be the .yamllint configuration file in this repository. It seems to be disabling by virtue of omission quite a few rules that are probably valueable and this PR should be amended to update the .yamllint configuration.

Okay, I did some more fooling around and figured out that the .yamllint config we had was not doing much because it was not explicitly extending the default yamllint configuration. I've put in a new yamllint config which does extend the default and then tuned it to match our conventions. I've also set it to ignore certain rules for contents which do not meet the requirements currently. We can discuss if it's best to ignore those rules long term or update the contents to conform.
For the old releases/ files I think we can leave them be but there are some rosdep definition lines that are ~500 chars long and it may make sense to split those up for readability.


Installation permission issues

I have tried installing SWOT 1.0 on both Windows 10 and Mac OS 10.12 and both appear to be having installation issues when running gem install swot. Other gems seem to install fine and I don't see any specific installation instructions outside of a regular gem install. Any idea on what could be causing this?
On Windows 10/Ruby 2.3.3  I get the following error
ERROR:  While executing gem ... (Gem::Package::PathError)
    installing into parent path //./nul of C:/RailsInstaller/Ruby2.3.0/lib/ruby/gems/2.3.0/gems/swot-0.4.2 is not allowed

On Mac OSX 10.12/Ruby 2.0 I get the following error
ERROR: While executing gem ... (Gem::FilePermissionError)
    You don't have write permissions for the /Library/Ruby/Gems/2.0.0 directory.

Im getting the same issue on windows


warning about growl notifications with guard-rspec #56

I just put in the warning for other users like me that are trying to get guard-rspec, guard-spork, and growl-notify working.

Do you think it's still valid with Guard 0.9.4 ?


Preparing documentation for version 0.2 release

It is not required for this release, but I think we can write few more details about forking/pull-requesting back in Contributing/Get Started! to make it more easy for those who have not done it before.

Also in contirbuting/Pull Request Guidelines I think we can add something like:
4. Follow the core developers' advices as they are given for keeping code consistent regardless of variety approaches used by many contributors.
5. If you don't have time to continue working on pull request, or simple lost interest. Please write a 5 second comment about it, so we know we can apply changes there to get it done.
Or something similar in meaning.

And do we need Contributing/Tips with one tip about nosetests speed up? We run whole suite in 5 seconds and this is well known approach for anyone uses nose and we kinda "tipping" to list folder contents with ls command.

As for Credits I think everybody deserve to be there.
Here is the list of all commiters:
Artur Sadurski
Claudio Salazar
Elias Dorneles
Eugene Amirov
Faisal Anees
Ismael Carnales
Joseph Kahn
Mark Baas
Marko Horvatić
Mateusz Golewski
Opp Lieamsiriwong
Rajat Goyal
Raul Gallegos
Shuai Lin
Sviatoslav Sydorenko
Tom Russell
Umair Ashraf
Waqas Shabbir

I also added  Ismael and Shuai here as they were at the beginning of this module, but moving it to separate repository broke commit history.

+1 on the Credits list!
At the time I extracted the package, it was a pain to get the commiters from git -- the history wasn't reliable because I basically copied the code from the internal project.
Sorry for any injustices! ;)


exploitdb: 2024-03-13 -> 2024-03-15

Diff: https://gitlab.com/exploit-database/exploitdb/-/compare/refs/tags/2024-03-13...2024-03-15
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


Insufficient Gas Fee Validation in Cross-Chain NFT Bridge

From https://codehawks.cyfrin.io/c/2024-07-ark-project/s/14
The smart contract allows users to deposit NFTs into an escrow and initiate a cross-chain transfer to StarkNet (L2) with an arbitrary gas fee. The lack of a meaningful minimum gas fee requirement can result in NFTs being locked in the escrow contract if the L2 message is never processed due to insufficient gas.

Hello @ptisserand please i will love to take this up by tomorrow

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
GM, I am Sagar Rana, a smart contract developer and full stack engineer. I have 3 years of experience building robust full stack applications and over a year of writing smart contracts. You can see my projects and contributions to some major repos on my GitHub profile. The tech stack I use mainly includes Solidity, Rust, JavaScript and Typescript. I am also contributing to the Starknet and Rust ecosystems and building on Cairo language. I am interested in contributing to projects like this to learn more about these technologies and help make these projects better. Please assign me as I would be really glad to be a contributor in this project! :)
How I plan on tackling this issue
Hi @ptisserand, I went through the codehawks article and I am pretty confident I can resolve this issue.

Implement a higher minimum gas fee

uint256 public constant MIN_GAS_FEE = 1e9; // Example: 1 gwei minimum gas fee

Implement check to ensure minimum gas fee is maintained

require(msg.value < MIN_GAS_FEE, "Insuffiecient gas fee");

Add a Reclaim Mechanism:
To allow users to reclaim their NFTs if the L2 transaction fails or isn't processed after a certain period, we'll need to track deposits and implement a reclaimTokens function. This function will allow users to reclaim their NFTs after a set timeout period if the L2 message wasn't processed.

Tasks:

[ ] Implement minimum gas fee
[ ] Implement Reclaim function

ETA: 1 Day

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
i'm a solidity and cairo smart contract  developer with over 2 years experience and believe i have the skill set for the task and i am also very good with smart contract testing

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Good morning once again ArkProject My name is Deon and I'd like to apply formally for the task presented. I am a Web and blockchain engineer with a passion for building user interfaces and Dapps that deliver meaningful experiences. With a background in Computer Science (BSc) and hands-on experience. If given the chance to contribute this will be my second official contribution via onlydust and I'm confident in my ability to deliver on the feature you're looking for.
How I plan on tackling this issue

Understand the Deposit and Escrow Logic:
I will review the smart contract’s logic, focusing on how gas fees are handled when users deposit NFTs for cross-chain transfer. This includes understanding how gas fees are passed to the L2 network and identifying the conditions where transactions could fail due to insufficient fees.

Implement a Minimum Gas Fee Requirement:
I will introduce a minimum gas fee requirement within the depositTokens function. This will prevent users from initiating transfers with extremely low gas fees that are unlikely to be picked up by the L2 network. The minimum threshold will be set at a level that ensures the L2 message has enough gas to be processed.

Add a Reclaim Mechanism for Stuck NFTs:
I will implement a function that allows users to reclaim their NFTs if the L2 message has not been processed within a certain time period. This mechanism ensures that users can recover their assets if the cross-chain message is not executed due to insufficient gas fees.

Write Unit Tests to Validate Gas Fee Checks and Reclaim Logic:
I will write unit tests to ensure the new minimum gas fee validation works as expected. Tests will cover both successful transfers with adequate fees and rejections due to insufficient fees. Additionally, tests will be written for the reclaim mechanism to verify that users can retrieve their NFTs if the L2 message fails to process.

Finally I will ensure all edge cases are covered, including scenarios where fees are just above or below the threshold, and that the reclaim mechanism is reliable in cases where L2 transactions remain pending.

Closed by #251


Add a .formatter.exs to export the DSL

To prevent mix format from putting parentheses on name/1, option/1 and so on, we currently need to add these macros by hand in our .formatter.exs. This files also allows to export and import the list of macros which should not have parentheses. With this PR, one can do:
[
  import_deps: [:ex_cli]
]

in its .formatter.exs to automatically avoid mix format putting parenthese on the DSL macros.

Thank you very much, I just pushed a new version to hex.


can add TensorFlow version for Mac M1 aarch64 version for install

What's the problem this feature will solve?
install tenforflow for Mac arm arch

Describe the solution you'd like
whl file for pip3 install

Additional context
no

@stride83 This is the repo for the https://pypi.org/ website, not the actual libraries and projects it hosts.
Please report TensorFlow issues to the project at https://github.com/tensorflow/tensorflow, and it looks like https://github.com/tensorflow/tensorflow/pull/45404 might be what you're after.


[Feat] More flexible IndexerServiceMetrics

Problem statement
There's only 3 fields in the metrics tracked. It would be good to have flexibility in this struct so we can add more metrics for individual data services instead of hosting a separate metrics service.
pub struct IndexerServiceMetrics {
    pub requests: IntCounterVec,
    pub successful_requests: IntCounterVec,
    pub failed_requests: IntCounterVec,
}

Expectation proposal
Generic type/field added to IndexerServiceMetrics and function to register more metrics.
Alternative considerations
We can have a separate metrics server on the data service level, but that structure seems a bit convoluted for the data providers.

Now metrics are static and can be created anywhere in the code.


first go at delete test

Would this be the proper way to test a 'success' case for the DELETE route?  If so, I'll add on to the rest of the test cases in a similar format. If not, let me know what is wrong here.
Also, is there a way to group these tests together in a better way? If this is the right approach...it seems like many 'success' cases for different routes would share a lot of the same code (i.e. the 'success' for deleting a user, inserting a user, and getting user by ID all return 200 Ok after making request)...is there a better practice to share code across tests?
Thanks!

@strad234 , please review new changes. If I understand correctly, when specifying a type of IO[Unit], you are implying that nothing will really be returned right? To simulate that, I just have the output print a line to the console. This makes it so that the 'notFound' and 'success' methods both return 200 though, I'm not sure if that's preferable though.
If this is good, I'll use the exact same logic for the inserts and move on to next task. If it's not, let me know what's wrong and I'll fix it up. Thanks!


[v1.5rc] Download Image

Pull Request Test Coverage Report for Build 315

61 of 69 (88.41%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 91.337%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/recipe/model/recipe/Image.java
60
68
88.24%

Totals

Change from base Build 295:
-0.09%

Covered Lines:
2446

Relevant Lines:
2678

💛  - Coveralls

-0.03% coverage due to exceptions impossible to cover.


update kind e2e test version in jobset

What would you like to be added:
Since kubernetes v1.30 has been released, we can update the e2e test version
Why is this needed:
update the e2e test version

[ ] kind version update in jobset makefile
[ ] test-infra yaml update

/cc @danielvegamyhre I will do some testing and submit PR in the next few days. If you are free, please help review it.

/assign


different color scheme per conversation

enhancement - would love to be able to change the appearance of different conversations separately - sub-color scheme change for each conversation to allow some to be different from others. i'm forever typing in 1 thinking that i'm in the another, and sending a message to the incorrect party.
thx.bri.

Interesting and cool idea


typo

Thanks! This was handled in another PR, so I'll now close this one. But thanks for the effort! 🍰


Apply exponential backoff to code-server reconnection

Current behavior
When code-server loses the connection to the server, it attempts to re-connect at 30 second intervals repeatedly until the connection succeeds. If there is a large-scale event causing an outage while clients have code-server open, then this can cause repeated spikes in traffic on the server.
Steps to reproduce

Open VSCode via master.cdr.dev
Rebuild your environment while VSCode is open

While the environment is rebuilding, we just see this message every 30 seconds, counting down to zero and then restarting back at 30 seconds:

Proposed solution
We should consider applying a randomized exponential backoff scheme (e.g. try to reconnect in 10 seconds, then 20 seconds, then 40 seconds) to avoid coordinated events of large-scale deployments (hundreds or thousands of clients) hammering the server by attempting to reconnect at approximately the same time. This also allows us to begin with much smaller intervals to begin with (e.g. 5 seconds) making the reconnection much faster in the common case of a transient network issue.

Seems like a valid point! Though I would imagine this behavior is from VS Code and not specific to code-server - though I hope @code-asher will correct me if that's wrong.

Yup it is from VS Code, maybe it's something we could patch upstream!
But we've patched this connection logic locally before (those changes
have since been removed since they were added upstream) so it wouldn't
be a first haha.


sunbeam broken on windows since v22

v22. Straight up crashed.
v25, doesn't not print anything to console, none of the commands work.

Apologies, this is weird. It seems to work on a second machine. I'm going to close it and check why its broken on my first machine.

It's probably because I'm iterating on the extension system. Sorry about the trouble

Yes. Just realized it prints the json instead of displaying any menu. No worries. Using v21

I'll ping you once I'm done, you might need to reinstall your extensions. V1 is coming soon, I will garanty backward compatibility from there

@pomdtr, asking since I've seen recent changes. are you dropping support for Windows? :(
I understand wsl is an option but would appreciate if windows was also supported. Can assist in testing if that's the issue.

Hey, I plan to add support back soon, but since I did not test it yet I have removed it for now

support for windows is back on master !


[BUG] Cluster Name is limited to 190 characters

When you try to create a new cluster or import an existing one there is an issue when the cluster name as more than 19 characters.
For example, I tried harvester-loic-provo and this lead to the cluster stall in Creating state with this error message: clusters.management.cattle.io "harvester-loic-provo" not found. No issue if I name the cluster harvester-loic-prv.
I tried different names with different characters and it's clearly a limitation with the number of characters.

@ldevulder What version of rancher are you using?

Close but it's not the number of characters directly, but because it contains a substring of the formc-xxxxx.  Shorter only works because you don't have 5 characters after thec- anymore.
This is a duplicate of https://github.com/rancher/rancher/issues/33694


Correct typing for ImageryProvider constructor

Mark the ImageryProvider constructor parameters as optional.  Fixes #11625
Test plan

Open up packages/engine/index.d.ts on main.  Find ImageryProvider and verify that both parameters are required for the constructor.
Open this branch and run npm run build-ts.  Find ImageryProvider and verify that both are now optional.

Found this one again in my notes and realized I never opened up the PR for it

Thanks @angrycat9000!


Rework buildbot_slaves module

Make:

password setable
make module more generic
grab slave name from facter's hostname fact
fix formatting

lgtm, will merge.


Spring AMQP - Publisher Confirms not closing unused channels

Publishing and getting acks/nacks is working as expected. My concern here is, it is not closing extra channels which are not in use. On the other hand with Publisher Confirms set to false the unused channels are getting closed as expected.
ConnectionFactory sample:
cachingConnectionFactory.setCacheMode(CachingConnectionFactory.CacheMode.CHANNEL);
cachingConnectionFactory.setChannelCacheSize(confirmChannelCacheSize);
cachingConnectionFactory.setPublisherConfirms(Boolean.TRUE);
cachingConnectionFactory.setPublisherReturns(Boolean.TRUE);
RabbitTemplate sample:
template.setMandatory(true);
template.setConfirmCallback(deepConfirmCallback);
template.setReturnCallback(returnCallback);
Spring AMQP dependency:

org.springframework.boot
spring-boot-starter-amqp
2.1.1.RELEASE

Don't ask the same question in multiple places; it is as waste of your time and ours.
I will add some comments on the Stack Overflow question; if it turns out to be a bug, we can reopen this.


Upgrade SourceIndexServer to ASP.NET Core 2.0.3

Resolves #87

@KirillOsenkov it didn't require many code changes. Tests pass in VS. Is there some other validation I should check to make sure this works fine?

Nah, let's get this in and fix it up afterwards if something's broken :)
Cc @alexperovich if you could check this out whenever you have time?

@natemcmaster I do have one question. Here are the steps I use:

Set HtmlGenerator as startup and F5. It will generate a website for TestSolution under C:\SourceBrowser\src\HtmlGenerator\bin\Debug\net46\Index
Now I want to set SourceIndexServer as startup and F5. The problem is I don't know how to point it to serve from the root specified in step 1. Basically how do I change the Environment.ContentRootPath from C:\SourceBrowser\src\SourceIndexServer to C:\SourceBrowser\src\HtmlGenerator\bin\Debug\net46\Index?

If all works well, if you serve the website from that page, it should show up, searching should work and clicking on search results should display the source code in the right pane.
Thanks for your help!

It seems to work the other way: if you make sure HtmlGenerator generates its output to SourceIndexServer\Index, then it's all working fine.

Basically how do I change the Environment.ContentRootPath

You can still add a call UseContentRoot.
        public static IWebHost BuildWebHost(string[] args) =>
            WebHost.CreateDefaultBuilder(args)
                .UseContentRoot(@"C:\SourceBrowser\src\HtmlGenerator\bin\Debug\net46\Index")
                .UseStartup<Startup>()
                .Build();

I see, cool. It's interesting that if I set the Working Directory to that path in Project Properties -> Debug, the current directory at runtime doesn't seem to change. How is Environment.ContentRootPath initialized?

Not sure what that UI changes does. Does it set RunWorkingDirectory? By default Microsoft.NET.Sdk.Web always runs your project from the project folder.
https://github.com/aspnet/websdk/blob/5ab771ee5333ffa83fd7521ff6308ef863c04594/src/ProjectSystem/Microsoft.NET.Sdk.Web.ProjectSystem.Targets/netstandard1.0/Microsoft.NET.Sdk.Web.ProjectSystem.targets#L22

Oh, RunWorkingDirectory? Excellent, this is what I want. Thanks!


GitHub Releases No Longer Publishing

Bug Report
We're not seeing changelog diffs when we update Twurple with Renovate:

It appears the last release on GitHub was v5.1.5, so I'm thinking maybe the changelog generation broke? This is a lot easier to read than combing through 100+ commit messages, especially for a minor release. Lerna should be doing this for you automatically when you publish, but you have to set a $GITHUB_TOKEN in the env so it can communicate with GitHub's API. Happy to help if needed, we use these libraries every day and they are really nice!

I never do generation. I do these manually. Will follow up later today.

@d-fischer No worries, I read through the publish script and noticed you were using Lerna, so I figured something got out of sync. Anyway, if you are doing these releases manually, just be aware that automatic dependency update tools may be sending out PRs that look like this, and causing developers to have to hunt through 100+ commits and/or happen to remember that the GItHub release was added later. Not a huge deal for patches, but for minor releases it's a bit more of a burden.


How to save the diagram?

Would you please tell me how to save the diagram to a png or svg file?
Thanks in advance.

You have 2 options (which do the same thing)

Set the save_graph=True when calling draw_graph
or
Use the result graph:

model_graph = draw_graph(...)
model_graph.visual_graph.render(format='png')

Note: There is also directory argument in draw_graph if you want to change the location to save the image.


Optimization Suggestions for Dense SIFT and PatchMatch

Hi,
Thank you for your fantastic work! I've been using Dense SIFT and PatchMatch, and they work remarkably well for my project. However, I’ve noticed that the processing speed is a bit slow.
Do you have any suggestions for optimizing the workflow? Additionally, would it be possible to accelerate the process using GPU?
Thank you for your time and insights!
Brian

Hi Brian,
Thank you for your kind words and interest in this project.
Unfortunately, it has been some time since I worked with this code.
I recommend checking the code from D2PRL (https://github.com/byc33/D2PRL/blob/main/deep_PM.py) for a more recent implementation of PatchMatch.
If you find a more efficient implementation, please share it here :)
Kind Regards,
João

That's great, Matthias!
Thanks for sharing.
I would also recommend checking for some functionalities from the following works:
1 - https://github.com/phillipecardenuto/upm
2 - https://github.com/danielmoreira/sciint
Kind Regards,
João

Am 19.12.2024 um 10:51 schrieb Phillipe Cardenuto @.***>:
I would also recommend checking for some functionalities from the following works:
1 - https://github.com/phillipecardenuto/upm
2 - https://github.com/danielmoreira/sciint

cool - didn’t know upm but distributed the link here already here among my friends…
With the bestes wishes for a Merry Christmas and a Happy New Year,
Matthias


Full path to sources revealed in binary code: suspect FILE macro

It appears when using LOG() commands that the full path to the sources in which the commands are used  can appear in the binary code of the compiled output.
I checked the output of an executable, ruled out all other possible libraries or code, and saw that using LOG() commands resulted in the full path to sources being revealed in strings in the compiled client code.
This may not be ideal for many people and organizations. So, perhaps there is some way to introduce an option to restrict such full path output, e.g. only filename or using a custom "__FILE__" equivalent macro that can be overridden?
Notes.

__FILE__ may not be directly interfered with on your compiler. In particular, in MSVC compilers, it is a reserved word that cannot be treated like an ordinary macro, e.g. by #undef or redefine.
After some intense searching around (which will hopefully save others the trouble), the only workaround I could find that may or may not work on most compilers is to use the preprocessor "'#line" directive to indirectly override __FILE__. This may be sufficient, and even better for some users, but ideally each library should solve this issue their own way.

This is fixed in next version

Actually no, but it should be easy fix.

base::utils::File::buildBaseFilename

use %fbase

@abumusamq I don't see how changing the format will avoid having FILE in the binary.
This is a major issue.

@abumusamq I don't see how changing the format will avoid having FILE in the binary.
This is a major issue.


video+logs

i did a little POC locally to test if showing logs alongside of the video is possible without much change.
my findings is that it would only require a minor change in selenoid for the websocket stream logs
https://github.com/aerokube/selenoid/blob/master/selenoid.go#L664
wsconn.PayloadType = websocket.TextFrame
fc, err := ioutil.ReadFile(path.Clean(logOutputDir + "/" + sid + ".log"))
if err != nil {
  log.Printf("[%d] [LOG_NOT_FOUND] [%s]", requestId, err)
	return
}
websocket.Message.Send(wsconn, string(fc))

and then add the Log component in the video ui. to get an output like this:

keep in mind this is just a proof of concept so is not pretty :P
i wanted to check if this is something the project wants to support moving forward.
Thanks

Maybe with a bit more work it can even scroll along with the video :)

@eacevedo79 don't get the use case. Logs are being streamed via web-sockets when session exists, whereas videos are saved when session finishes.

Yes but after the session is done the log is not streamed anymore. You can just see the video. This feature will allow to see both the video and the log after the session is over.

@eacevedo79 are you showing the log from saved file?

@vania-pooh yes

@eacevedo79 what then prevents you from loading file contents by using the following API: https://aerokube.com/selenoid/latest/#_downloading_log_files_from_selenoid

@vania-pooh the log component in the ui uses the websocket to show the xtern like ui, I'm not sure how much work would be to build a new component that uses the http endpoint, but it should be possible right @lanwen ?

dont see the reason as well - isn't better to just download log and open that in a full terminal, where you can search, edit and expand full screen?
I wouldn't go in this direction, but thx for the PoC :) was interesting


updated gitignore to ignore venv

I think this is usually in .git/info/exclude rather than the shared .gitignore. People can have different names for their virtualenv, or multiple ones for testing different versions.


MTGA icon color

In Windows, the app icon is white/silver instead a black A:

On their page, they use a very similar light colored A:
https://magic.wizards.com/de/mtgarena

Yep that's better d120be40902df47af442184f0a1c3f387a976e53

Way better now, yes 👍


[Warhammer 4e Character Sheet] Coin converter

Changes / Comments

Added coin converters buttons on the inventory page.
Various Qualities text fields now align to the left.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Change the map structure

Change the map structure as it uses array for setting options and depends on index.
Index based setting is a restrictive in nature.
So, change from array to object with keys for setting.

markdown has to be updated


Skip "Choose your username" page for returning users on login

Feature Request
Desired behaviour
Can key off of the {"canChangeUsername":false}  from https://staging.dockstore.org/api/users/user/extended

I have made this change, though I think a user logging in again who has already published a workflow should be linked to another page entirely. Maybe the accounts page.

Agreed, I think that makes sense.

Ah, my mistake.
I think it should be the equivalent of  (extendedUser).canChangeUsername && (user).setupComplete
This still appears for me after changing a username if I'm still allows to change the username

using develop, login for returning users that have already changed their username now goes to the onboarding wizard without the username change step.


path check before log [CPP-851]

check before writing to file

refactors sbp logging struct to manually convert into json bytes to handle the counter changes.
changes from compactformatter to haskellishfloatformatter
simple path check (assuming file doesnt get changed in that interval) to stop byte counter

will merge this back into #862 to finalize the ticket,

Frontend and Release Workflow Started here


Change repo name to avoid import maps confusion?

I worry the current repo name is a little close to the import-maps proposal, which may cause confusion that they are related.
How about something like pkg-entry-map or esm-entry-map instead?

Yep, good point. Changed to proposal-pkg-exports.


Support also yaml files for the AIPluginTool

Many OpenAI plugin implementations are starting to use yaml for the api doc.
class AIPluginTool(BaseTool):
    api_spec: str

    def from_plugin_url(cls, url: str) -> "AIPluginTool":
        response = requests.get(url)

        if response.headers["Content-Type"] == "application/json":
            response_data = response.json()
        elif response.headers["Content-Type"] == "application/x-yaml":
            response_data = yaml.safe_load(response.text)
        else:
            raise ValueError("Unsupported content type")
...

Looks like it's been fixed:

def marshal_spec(txt: str) -> dict:
    """Convert the yaml or json serialized spec to a dict."""
    try:
        return json.loads(txt)
    except json.JSONDecodeError:
        return yaml.safe_load(txt)


Couldn't deploy python getting started example

Was trying to put together the Getting started example, however I'm constantly getting the same error:
"message": "org.apache.flink.runtime.rest.handler.RestHandlerException: Could not execute application.\n\tat org.apache.flink.runtime.webmonitor.handlers.JarRunOverrideHandler.lambda$handleRequest$4(JarRunOverrideHandler.java:248)\n\tat java.base/java.util.concurrent.CompletableFuture.uniHandle(CompletableFuture.java:930)\n\tat java.base/java.util.concurrent.CompletableFuture$UniHandle.tryFire(CompletableFuture.java:907)\n\tat java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:506)\n\tat java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1705)\n\tat java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)\n\tat java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)\n\tat java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)\n\tat java.base/java.lang.Thread.run(Thread.java:829)\nCaused by: java.util.concurrent.CompletionException: org.apache.flink.client.program.ProgramAbortException: java.lang.RuntimeException: Python process exits with code: 1\n\tat java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:314)\n\tat java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:319)\n\tat java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1702)\n\t... 6 more\nCaused by: org.apache.flink.client.program.ProgramAbortException: java.lang.RuntimeException: Python process exits with code: 1\n\tat org.apache.flink.client.python.PythonDriver.main(PythonDriver.java:140)\n\tat java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)\n\tat java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)\n\tat java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)\n\tat java.base/java.lang.reflect.Method.invoke(Method.java:566)\n\tat org.apache.flink.client.program.PackagedProgram.callMainMethod(PackagedProgram.java:355)\n\tat org.apache.flink.client.program.PackagedProgram.invokeInteractiveModeForExecution(PackagedProgram.java:222)\n\tat org.apache.flink.client.ClientUtils.executeProgram(ClientUtils.java:114)\n\tat org.apache.flink.client.deployment.application.DetachedApplicationRunner.tryExecuteJobs(DetachedApplicationRunner.java:84)\n\tat org.apache.flink.client.deployment.application.DetachedApplicationRunner.run(DetachedApplicationRunner.java:70)\n\tat org.apache.flink.runtime.webmonitor.handlers.JarRunOverrideHandler.lambda$handleRequest$3(JarRunOverrideHandler.java:239)\n\tat java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1700)\n\t... 6 more\nCaused by: java.lang.RuntimeException: Python process exits with code: 1\n\tat org.apache.flink.client.python.PythonDriver.main(PythonDriver.java:130)\n\t... 17 more\n",     "messageType": "ERROR",     "messageSchemaVersion": "1",     "errorCode": "CodeError.InvalidApplicationCode"
I'm using  flink-sql-connector-kinesis-1.15.2.jar
Locally the setup works perfectly, I can see the code running
Could you point me at next steps to check?

@jeremyber-aws I've faced the same error. I was following official guide - https://docs.aws.amazon.com/kinesisanalytics/latest/java/gs-python-createapp.html. I double checked that I did everything as was told in instructions but it is not working. Do you have any ideas what I miss?

If it is working locally, I can only imagine it would be an error with permissions or packaging. Have you walked through the getting started guide found here? Apologies for the documentation overload, but I personally built this documentation and am confident it works when followed. Please join Apache Flink slack and reach out to me to discuss in more detail if this doesn't work.

@jeremyber-aws Thank you! I've followed your documentation and got it working. For anyone who will be stacked here, to be able to see exact error you can: add some print statement at the top of your script, after you got "Could not execute application" error, go to CloudWatch group and search by your log text. It will let you find logs with concrete error thrown by your script (In my case I just had wrong name of config variable provided to KDA 😖)


T2bis pcf

Uploading in the main branch the PCF implementation. Correspond to issue #7

Thanks @fvarray!! I have manually merged your changes with my refactorings in T2-merge-PCF and integrated into main.


Add default azure credential sample for eventhubs

As subject says.
Forked from prior PR so that EH and Blob samples can go in distinctly.

Can one of the admins verify this patch?

Hi @KieranBrantnerMagee , two things worth to note.

we have renamed folder examples to samples.
we removed EventHubClient, for sending it's EventHubProducerClient.

I'm already put an identity sample here.
Maybe you could update that sample to include the DefaultAzureCredential?

@yunhaoling Have adjust the PR as we discussed, largely integrating this sample into the client_identity sample yall wrote.  Let me know if you think this is forcing too much content into once place, I can always split it out into its own file/rewrite/whatever.
(To avoid dealing with reverts/etc for a change that's almost entirely pivoting, I've put this into a new PR, here and am closing this)


treewide: avoid alias usage for intel-vaapi-driver based on nixos version

Description of changes
Fixes configs with nixpkgs.config.allowAliases = false after NixOS/nixpkgs#235202
I couldn't find any prior art in the repo for package aliases based on NixOS version, so let me know if there is a better/more preferred way to handle this
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

bors merge


Update enumeration methods to allow a stop flag to be flipped by caller

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger

🚫 CI failed with log

Thank you for the contribution, this will be super useful!

@maicki I think go for it because we're still in beta territory?


AndroidManifest: '.AnyOrientationCaptureActivity' is not assignable to 'android.app.activity'

I am not sharing my manifest file(for obvious reasons) but I would like to point out this error:
'AndroidManifest: '.AnyOrientationCaptureActivity' is not assignable to 'android.app.activity'
in the AndroidManifest.xml. I think its an error in the src code.

Are you getting this error with the sample app or your own app?
AnyOrientationCaptureActivity is only part of the sample app, not the library itself.

From a quick check it seems like that error is common when referencing something in the manifest which does not extend android.app.Activity (e.g. if it's a Fragment, or something else completely).

ahh okay I get the problem. I still dont understand though... how do I actually use it to scan a picture in my app!!

Check the readme and sample app. I can't help without a more specific question.

@rkistner I followed the README and ran into an error right off the bat... After putting in

import android.content.Intent;
import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.widget.Toast;

import com.google.zxing.integration.android.IntentIntegrator;
import com.google.zxing.integration.android.IntentResult;

public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }

    new IntentIntegrator(this).initiateScan(); // `this` is the current Activity

    // Get the results:
    @Override
    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
        IntentResult result = IntentIntegrator.parseActivityResult(requestCode, resultCode, data);
        if(result != null) {
            if(result.getContents() == null) {
                Toast.makeText(this, "Cancelled", Toast.LENGTH_LONG).show();
            } else {
                Toast.makeText(this, "Scanned: " + result.getContents(), Toast.LENGTH_LONG).show();
            }
        } else {
            super.onActivityResult(requestCode, resultCode, data);
        }
    }

    IntentIntegrator integrator = new IntentIntegrator(this);
    integrator.setDesiredBarcodeFormats(IntentIntegrator.ONE_D_CODE_TYPES);
    integrator.setPrompt("Scan a barcode");
    integrator.setCameraId(0);  // Use a specific camera of the device
    integrator.setBeepEnabled(false);
    integrator.setBarcodeImageEnabled(true);
    integrator.initiateScan();
}

I am getting errors in line new IntentIntegrator(this).initiateScan(); // this is the current Activity and the last (paragraph?) of code. The errors in the line are:
new: unexpected token
IntentIntegrator: Invalid method delaration; return type required
this: Parameter expected
initiateScan(): Invalid method delaration; return type required   Missing method body, or declare abstract
Errors in the last paragraph are :
All the integrator.xyz are- Cannot resolve symbol xyz
Unknown class: IntentIntegrator.ONE_D_CODE_TYPES

The code cannot be in the body of the class - it must be inside a method, e.g. one triggered from a button press.

aaahhh  okay will post updates

@rkistner Hi, I'm back!(haha)
Most of the stuff got solved by putting it inside the button, but a few new errors popped up. All the errors are here:
// Get the results:
    @Override
    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
        IntentResult result = IntentIntegrator.parseActivityResult(requestCode, resultCode, data);
        if(result != null) {
            if(result.getContents() == null) {
                Toast.makeText(this, "Cancelled", Toast.LENGTH_LONG).show();
            } else {
                Toast.makeText(this, "Scanned: " + result.getContents(), Toast.LENGTH_LONG).show();
            }
        } else {
            super.onActivityResult(requestCode, resultCode, data);
        }
    }

Errors are: @Override: Annotations are not allowed here
right before int-';' expected
before the 2n int- Identifier expected
on all references of word 'data'-Cannot resolve symbol data
Full (button)updated code:
import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.view.View;
import android.widget.Toast;

import com.google.zxing.integration.android.IntentIntegrator;
import com.google.zxing.integration.android.IntentResult;

public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }

    public void scanNow(View view) {
        new IntentIntegrator(this).initiateScan(); // `this` is the current Activity

// Get the results:
        @Override
        protected void onActivityResult(int requestCode, int resultCode, Intent data) {
            IntentResult result = IntentIntegrator.parseActivityResult(requestCode, resultCode, data);
            if(result != null) {
                if(result.getContents() == null) {
                    Toast.makeText(this, "Cancelled", Toast.LENGTH_LONG).show();
                } else {
                    Toast.makeText(this, "Scanned: " + result.getContents(), Toast.LENGTH_LONG).show();
                }
            } else {
                super.onActivityResult(requestCode, resultCode, data);
            }
        }

        IntentIntegrator integrator = new IntentIntegrator(this);
        integrator.setDesiredBarcodeFormats(IntentIntegrator.ONE_D_CODE_TYPES);
        integrator.setPrompt("Scan a barcode");
        integrator.setCameraId(0);  // Use a specific camera of the device
        integrator.setBeepEnabled(false);
        integrator.setBarcodeImageEnabled(true);
        integrator.initiateScan();
    }
}

onActivityResult must be a function on its own.
I don't want to be rude, but these are basic Java / Android questions, and I cannot keep helping with those.

I completely understand


add -json flag for json output

This is a great tool for helping manage developer environments. One thing that would increase the usability for our team is being able to check the binaries from list and ensure that they are installed from a script.
Rather than having to parse the tab output with bash, I think a json output makes this much easier. My proposal is to add a -json flag for output.
Usage
❯ ./bingo list -json
[
        {
                "name": "copyright",
                "binary_name": "copyright-v0.0.0-20210112004814-138d5e5695fe",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "embedmd",
                "binary_name": "embedmd-v1.0.0",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "faillint",
                "binary_name": "faillint-v1.8.0",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "goimports",
                "binary_name": "goimports-v0.0.0-20210112230658-8b4aab62c064",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "golangci-lint",
                "binary_name": "golangci-lint-v1.50.0",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "mdox",
                "binary_name": "mdox-v0.9.0",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "misspell",
                "binary_name": "misspell-v0.3.4",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        },
        {
                "name": "proxy",
                "binary_name": "proxy-v0.10.0",
                "package_version"<EMAIL_ADDRESS>                "build_env_vars": null,
                "build_flags": null
        }
]

For example, this is a small script that checks if we have a binary installed:
for b in $(./bingo list -json | jq -r '.[] | .binary_name'); do
        if ! command -v $b;
                then echo "$b not found";
        else
                echo "$b found"
        fi
done

Output:
copyright-v0.0.0-20210112004814-138d5e5695fe not found
/Users/collinforsyth/bin/embedmd-v1.0.0
embedmd-v1.0.0 found
faillint-v1.8.0 not found
goimports-v0.0.0-20210112230658-8b4aab62c064 not found
golangci-lint-v1.50.0 not found
mdox-v0.9.0 not found
misspell-v0.3.4 not found
proxy-v0.10.0 not found

Hopefully not too big of a PR! Let me know if there's anything else needed for this.

I realized I didn't update the -h.
Will get to that on Monday.


Adding AWS::Transfer resources, per 2019 May 23 update

@axelpavageau I'm not sure we want to include the CloudFormationResourceSpecification.json into the repo since that is maintained externally by AWS. I'd be curious about your thoughts since you included it in this PR and #1408.

you're right, it doesn't make sense. I'll remove the file from both PRs

Thanks!


Select fields with i18n not working in Safari

Current Behavior
When using a select field with internationalisation in a node html file the selected value won't show after opening the nodes dialog. Issue is on Safari, in Chrome it works fine. After clicking the select field, the select values will shown and then when clicking outside the correct value is shown.
Found in Node-RED 3.0.2.
Expected Behavior
The selected value will be shown directly when opening the node dialog.
Steps To Reproduce

Add a Dashboard Switch Node
Open the Switch Node Dialog
Look at the select field where "checking all rules" should be selected.

Example flow
paste your flow here

Environment

Node-RED version: 3.0.2
Node.js version: 18.16.0
npm version: 9.5.1
Platform/OS: MacOS
Browser: Safari

#4136


Crash due to GraphQLEnumType variable types not serialized correctly

Currently String Enum types are not being converted to its rawValue before being used as a parameter. This causes a crash as Swift Enum types are not allowed in JSON serialization and the following error is thrown:
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Invalid type in JSON write (_SwiftValue)'
Here is some relevant code demonstrating the issue:
API.swift
public enum Size: String {
  case small = "small" /// A Small Size
  case large = "large" /// A Large Size
}

public final class UpdateSizeMutation: GraphQLMutation {
   ...
  public var variables: GraphQLMap? {
    return ["size": size]
  }
}

GraphQLNetworkTransport
    func send<Operation: GraphQLOperation>(operation: Operation, completionHandler: @escaping (GraphQLResponse<Operation>?, Error?) -> Void) -> Cancellable {
        let params: [String: Any] = ["query": type(of: operation).queryDocument as Any, "variables": operation.variables as Any]
        let request = manager.request(baseURLString, method: .post, parameters: params, encoding: JSONEncoding.default, headers: headers)
        ...
        return request
    }

The Swift Enum is therefore included inside of the params variable when sending the request and causes an error when encoding to JSON.
A potential fix is to use .rawValue for string enum types inside of variables during code generation.
Potential Fix
  public var variables: GraphQLMap? {
    return ["size": size.rawValue]
  }

This is a problem that can also be addressed within apollo-ios.

I'm not sure where the send implementation you quoted comes from, but it's not the one in the project!
If you look at the corresponding lines in HTTPNetworkTransport, they treat the body as a GraphQLMap and pass it to JSONSerializationFormat.serialize before setting it as the httpBody. (GraphQLMap is JSONEncodable, so that basically just calls jsonValue on it to get a JSON value).

Ah you're totally right, my apologies


Updates contentful handler to use the new USE_CONTENTFUL_SPACE toggle

USE_CONTENTFUL_SPACE toggle allows user to specify whether to use prod or prep space
in test_runtime, adds require_contentful_space method to be referenced in contentful_spec_helper
require_contentful_space method checks if user used USE_CONTENTFUL_SPACE toggle and if the user provided correct values for it
Changes read_contentful_tokens method in contentful_handler to use the user specified contentful space to get qa_key

@mhallwac Please close the issue https://github.com/ndlib/QA_tests/pull/237 after this branch is merged.


Comma added to scope parameter

I'm using Azure Active Directory B2C, which requires me to add scope=openid as a query parameter.
I've set Swagger up like this:
c.OAuth2("oauth2")
 .Description("OAuth Implicit Flow")
 .Flow("implicit")
 .AuthorizationUrl($"https://login.microsoftonline.com/{tenant}/oauth2/authorize")
 .TokenUrl($"https://login.microsoftonline.com/{tenant}/oauth2/token")
 .Scopes(sc =>
 {
     sc.Add("user_impersonation", "Access Swagger");
 });
 c.OperationFilter<AssignOAuth2SecurityRequirements>();

And the UI:
var queryStringParams = new Dictionary<string, string>
{
    { "resource", ConfigurationManager.AppSettings["ida:Audience"]},
    { "p", ConfigurationManager.AppSettings["ida:Policy"]},
    { "scope", "openid" },
    { "nux", "1" },
};
c.EnableOAuth2Support(clientId, clientSecret, redirectUrl, appName, additionalQueryStringParams: queryStringParams);

The response I get back has the error:
AADB2C90012: The scope ',openid' provided in request is not supported.
Am I doing something wrong? How does the comma get there?

I am also experiencing the same issue, if you open fiddler and see what it sends to the auth url you will see the url it contructs will be something like this: &scope=&someotherparam=someothervalue&scope=openid (i.e. two scope params, one being empty). I think swashbuckle should not automatically add the scope param to the url, this will fix the issue as we can then specify the scope params we want. I raised a question on stackoverflow here: https://stackoverflow.com/questions/46952916/web-api-add-openid-scope-to-auth-url-for-swagger-swashbuckle-authorisation-url

Would love to help you guys troubleshoot this...
Anyone has a minimal project reproducing this issue?

It's a bit difficult to provide a project as it would contain sensitive information as the azure ad tenant, app Id's etc, plus you'd also have to have an account to login. From what I can gather, it automatically adds a &scope= param to the auth url. This will stop us from adding our own scope param in the additionalQueryParams dictionary.
One more question, in the auth url it build, it adds a &response_type=token, how do I change this to be &response_type=id_token?
I suppose what you could do is, check the additonalQueryParams dictionary that you send into EnableOAuth2Support and don't automatically add params that exist in this dictionary (i.e. we could then send up out own response_type too).

Without a project I will not even consider troubleshooting this... I've been around that rabbit hole before and wasted too much of my time.
I do have an MSDN subscription and have Azure monthly credits, here is something I host there:
http://swashbuckletest.azurewebsites.net/swagger
Don't you have one too? If not I guess I can help you with AD but you still have to create a sample project reproducing the issue.

This fix for this would be relatively simple I assume, check the additonalQueryParams dictionary that gets sent into EnableOAuth2Support and if the query params exist in this dictionary, only add them once (e.g.. if the dictionary contains a query param called scope, use the dictionary value and don't add an additional scope query param).

How long do you assume it will take you to fix it?

Thanks for the help.

Sure anytime, if you want to create that minimal project I could help coding the fix

This might fix it, in https://github.com/swagger-api/swagger-ui/blob/master/src/core/oauth2-authorize.js
change
  if (Array.isArray(scopes) && 0 < scopes.length) {
    let scopeSeparator = authConfigs.scopeSeparator || " "

    query.push("scope=" + encodeURIComponent(scopes.join(scopeSeparator)))
  }

to
  if (Array.isArray(scopes) && scopes.length) {
    let scopeSeparator = authConfigs.scopeSeparator || " "

    query.push("scope=" + encodeURIComponent(scopes.join(scopeSeparator)))
  }

This would check the scopes length before adding it as a query param (obviously 0 < 0 is false)

In that case, you are reporting to the wrong project, please post that on the Swagger-UI:
https://github.com/swagger-api/swagger-ui/issues

I will do that, p.s. smile, it's nearly Friday.

Think I was barking up the wrong tree, so, looking at the source, I can see a file in my libs called swagger-oauth.js how does this get there and where do i find its source?
Thanks

ahh the old version...
This is the begging of a lot of other problems (the UI team is not doing any changes to 2.x)
https://github.com/swagger-api/swagger-ui/blob/2.x/dist/lib/swagger-oauth.js
This thread is going way to long, just reach me directly on gmail I'm heldersepu there too.


GCM click_action is not sending

Actually I'm using GCM to send all the push notifications to android and iOS devices, but when I add the click_action property it doesn't stick to the push, but the other properties appears.
Notification received:
 {
   aps =     {
       alert =         {
           body = “Lorem Ipsum…..";
           title = “New Message";
       };
       badge = 1;
       content-available = 1;
       sound = default;
   };
}

this is one example of how I build the payload
$payLoad['notification'] = [
                'title' => $push->getTitle(),
                'text'  => $push->getAlert(),
                'badge' => 1,
                'sound' => 'default',
                'click_action' => $push->getClickAction()
];

What's the value you're setting for click_action. It accepts a "string". Does the same push for android also drop click_action or does it have it?

Yes it is a string. I also hardcoded the value but that even doesn't work

Is it only dropped for iOS or also for Android?

both of them :/

@usckuro I checked and "click_action" is working correctly on my test app on Android.
Note that I sent the message using a simple curl command line, I didn't use the PHP code in your example.
I suggest to check your client implementation, or the sending library that you are using.


API Product import with flag --upsert outputs error

Hi,
When importing a new API Product using the import command with the --upsert flag, the product gets created in Apigee but I still get the following error:
{
  "error": {
    "code": 404,
    "message": "ApiProduct with name TestImport does not exist",
    "status": "NOT_FOUND",
    "details": [
      {
        "@type": "type.googleapis.com/google.rpc.PreconditionFailure",
        "violations": [
          {
            "type": "keymanagement.service.apiproduct_doesnot_exist",
            "subject": "[2002:a17:531:292b::]:4169:wmfz23:9825:260695:117263226",
            "description": "ApiProduct with name TestImport does not exist"
          }
        ]
      },
      {
        "@type": "type.googleapis.com/google.rpc.RequestInfo",
        "requestId": "6579398348678207337"
      }
    ]
  }
}

Is there a way of hiding this error while still getting the API Product data from the response?
Thank you in advance for your time!

This error isn't really an error (or perhaps it is better to say is, the 404 is expected). The upsert command does a GET first to decide between a create vs. update. Since you are importing a new API Product, the GET returns a 404.
You can hide errors (and usage message) with two environment variables
export APIGEECLI_NO_USAGE=true and APIGEECLI_NO_ERRORS=true. This preserves the exit code in bash.

I was just trying those env settings and various flags, but I was not able to block the output. Nor was I able to see appropriate exit codes using echo $?.
export APIGEECLI_NO_USAGE=true
export APIGEECLI_NO_ERRORS=true
apigeecli products import --file products.json --upsert=true

The above shows the 404 errors regardless of the env settings. It returns exit status of 0 when products exists or not.
apigeecli products import --file products.json --upsert=false

The above returns 409 and exit status of 0 in both cases.
The env settings do affect the display of errors (e.g. Error: unknown flag: --foo) and usage (e.g. after a bad param).
The use of --print-output=false doesn't seem to do anything. The use of --no-outout blocks all output but exit status code is still 0.
Attached is a simple products.json file that I used to test.
products_json.txt

@kurtkanaskie and I chatted, #204 will address the exit status when importing API products.
@ana-r-goncalves-alb - What is the reason for getting the API Product data from the response? It is not possible to suppress 404 and get the API response.

The scenario I was testing was importing an API product that doesn't exist in Apigee yet, while using the --upsert flag (because I don't know beforehand if it exists or not and want to update if it doesn't), APIGEECLI_NO_USAGE=true and APIGEECLI_NO_ERRORS=true. I want to retreive the API response because I need the updated createdAt and lastModifiedAt values


Realtime transcription

The model will be used in real time app, so let's see how in performs in this case.

Switch to https://github.com/luweigen/whisper_streaming


docs: 846 docs include google analytics

Closes #846
I added GA for MkDocs https://squidfunk.github.io/mkdocs-material/setup/setting-up-site-analytics/#configuration

Thanks @davidberenstein1957!


Doesnt work and breaks zsh-syntax-highlighting

Have not been able to get this plugin to work, config as follows. Also, enabling the plugin will stop zsh-syntax-highlighting from working, so theres some kind of interference there?
zstyle ':znap:*' repos-dir ~/.cache/znap
source ~/.config/znap/znap.zsh

znap prompt romkatv/powerlevel10k

# Completions
znap source zsh-users/zsh-completions
znap source zsh-users/zsh-autosuggestions
znap source MichaelAquilina/zsh-you-should-use
zstyle ':completion:*' matcher-list 'm:{a-z}={A-Za-z}'
setopt globdots

# History
znap source zsh-users/zsh-history-substring-search
bindkey -M viins '^[[A' history-substring-search-up
bindkey -M viins '^[[B' history-substring-search-down
bindkey -M vicmd 'k' history-substring-search-up
bindkey -M vicmd 'j' history-substring-search-down

# autoenv
znap source Tarrasch/zsh-autoenv
  export AUTOENV_FILE_ENTER=.in
  export AUTOENV_FILE_LEAVE=.out

# Colours
znap source zpm-zsh/colorize
[[ ! -f $XDG_CACHE_HOME/lscolor ]] && vivid generate molokai > $XDG_CACHE_HOME/lscolor # because vivid output is non deterministic
export LS_COLORS="$(cat ${XDG_CACHE_HOME}/lscolor)"
export CLICOLOR=1
znap source marlonrichert/zcolors
znap eval zcolors "zcolors ${(q)LS_COLORS}"
zstyle ':completion:*:default' list-colors ${(s.:.)LS_COLORS}

znap source zsh-users/zsh-syntax-highlighting

bindkey -v
export KEYTIMEOUT=1

That’s because vivid generates 24-bit themes, but Z Colors supports 4-bit colors only. I will update the Readme to mention this. 🙂
I might add support for 24-bit colors at some point, but it’s not high on my todo list. Patches welcome!

ooohhhh I would never have spotted that haha. Thankyou for the response!


Use the VMAlertManager definition to define VMAlert notify.url

Hello,
today we were hit by a strange problem.
We use the operator to host a local installation, of vmsingle,vmalert, vmalertmanager and etc...
In our fist version we put the vmalertmanager endpoint in our vmalert notify.url field.
that broke glorious as we were using the alertmanger with replica 2, and the we need to define both endpoints as comma-separated url.
But as the operator has both information, it should be possible for him to set the notify.url for local alertmanager always to all local instances.
This way it will remove this error class.
Any opinions on that? Did i miss some obvious, why it is not implemented that way?

Hello, for now there are few ways to achieve that:

use notifiers discovery by operator:

apiVersion: operator.victoriametrics.com/v1beta1
kind: VMAlert
metadata:
  name: example-vmalert
spec:
  datasource:
    url: "http://vmagent-example-vmsingle.default.svc:8429"
  notifiers:
    - selector:
        namespaceSelector:
          any: false
          matchNames:
            - default
        labelSelector:
          matchLabels:
            usage: dev

use vmalert notifier discovery with configuration file:

apiVersion: operator.victoriametrics.com/v1beta1
kind: VMAlert
metadata:
  name: example-vmalert
spec:
  replicaCount: 1
  datasource:
    url: "http://vmsingle-example-vmsingle-pvc.default.svc:8429"
  notifierConfigRef:
    key: notifier.yaml
    name: vmalert-notifier
  selectAllByDefault: false
---
apiVersion: v1
kind: Secret
metadata:
  name: vmalert-notifier
stringData:
  notifier.yaml: |
        dns_sd_configs:
        - names:
           - vmalertmanager-example.default.svc
                type: 'A'
                port: 9093

more info for discovery at docs: https://docs.victoriametrics.com/vmalert.html#notifier-configuration-file

Thanks Nikolay, that worked.


release loftr model ?

congraduation, thanks for your work. could you please release the loftr model?

when will release gim+LoFTR model weight?

@sunshineywz123 @ttaohe
We will release it, so please be patient for a little while longer and thank you for your ATTENTION.

@sunshineywz123 @ttaohe
Thank you for your patience. We have just released the network weights and code for gim_loftr.


new resume added

They were same, nonetheless I changed it to my actual name if it was the issue, however I do prefer my alias if possible


Add running example to documentation

Paper's running example needs to be added into example/

Other good example<EMAIL_ADDRESS>


False Warning

Microsoft PowerToys version
0.69.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
I need to remap the menu key with play/pause the media but it is giving me an false error
✔ Expected Behavior
To remap the key
❌ Actual Behavior
It instead giving me an false warning

Other Software
No response

The error which i am facing is

This is correct bahaviour. It states that no key is now Play/Pause


Dot net variable with Business Central OnPrem issue

Hi All,
I'm using dot net variable in Business central onprem
This code works perfectly well in NAV 2018 but in D365 BC below error is thrown.
The code line is
Window.UPDATE(3, FORMAT(ProdDetails.sku()));
//here ProdDetails is a Dotnet variable and sku is a Dot net System.String variable.

Does this repro when using the latest docker image? I have fixed something similar a long time ago. Have you applied the latest CU? If this still repros, please provide code to reproduce.

Closing due to no repro and lack of additional input.


Wildcard route is conflicting when creating a route to cover generic path

Hi,
I just realised that if you try to create a generic route to cover a entire path the app will break if you have a second and more specific route. One exemple:
const app = new Application()

app.get("/hello/:param", (req: Context) => {
    console.log(req.params)
    return `Hello, ${req.params}`
})

app.get("/:param", (req: Context) => {
    console.log(req.params)
    return `Hello2, ${req.params}`
})

app.start({port: 8080})

This will give you an error like:
error: Uncaught Error: wildcard route ':param' conflicts with existing children in path '/:param'
        throw new Error(
              ^
    at Node.insertChild (node.ts:269:15)
    at Node.addRoute (node.ts:183:16)
    at Router.add (router.ts:20:10)
    at Application.add (app.ts:143:17)
    at Application.get (app.ts:89:17)
    at Testing.ts:11:5

I believe that' s not intended

In fact, it was designed :P, but I also realized it was very inconvenient, and I needed a little time to correct this.

Need to be informed: before discussing params conflicts, I need to solve the static routing priority issue (see https://github.com/zhmushan/router/issues/1)

should we close this issue and open it on the new route repo?

also I've been looking into this and got a I'm trying to figure out why the moment you solve the conflict between this two routes
/route and /:generic this one becomes a problem
/:generic and /route/:param.
Was this designed as well?

In the new router design, we will always match the static route first.
For your situation:
/route/foo     match
/foo               match
/foo/bar        no match
/router/foo    no match

@shuminyang @Jipfr Fixed in https://github.com/zhmushan/abc/commit/ad63d65ec0f73bfeabc3bc80fb1ae6ae5e25904f


Add ClientUsername feature to ForwardAuth middleware

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Both BasicAuth and DigestAuth fill the accesslog ClientUsername field and Traefik expose it in logs. The ForwardAuth does not have this functionality.
I propose to add the option to indicate in the configuration the header name which are providing this information. Ex.:
labels:
  - traefik.http.middlewares.test-auth.forwardauth.authResponseClientUsernameHeader=X-Auth-User

which will be injected into accesslog:

if len(forwardResponse.Header[authResponseClientUsernameHeader]) > 0 {
  logData := accesslog.GetLogData(req)
  if logData != nil {
    logData.Core[accesslog.ClientUsername] = req.Header[authResponseClientUsernameHeader]
  }
}

I want to work on this @rtribotte

Hello @MallikharjunaTeja,
Thanks for your interest in Traefik!
All contributions are welcome, and if you want to so, feel free to open a PR following the Pull Request Guidelines.


Workaround for Cygwin/MSYS progress bar

On some Cygwin/MSYS terminals (MSYS rxvt exhibits this problem every time), process.stdout.columns returns undefined, because why not, what did you think this is, software engineering or engineer softening?

Anywho... barsize here will fail and throw the dreaded RangeError: Invalid array length (ref https://github.com/snobu/destreamer/issues/83)
new cliProgress.SingleBar({
           barCompleteChar: '\u2588',
           barIncompleteChar: '\u2591',
           format: 'progress [{bar}] {percentage}% {speed} {eta_formatted}',
           barsize: Math.floor(process.stdout.columns / 3),
           stopOnComplete: true,

Temporary fix is to handle the undefined case and fall back to an integer. The progress bar won't render at all but the download works.
Current behavior -

New behavior after this PR gets merged -

i'm sure this is not a good idea, issues will change from "unknown error" to "my progress bar does not work"
we should handle this.
we can just switch to a text based progress when colums is undefined
something like
on top: isBadTerm = colums == undefined
and we wrap bar creation and calls with ifs
inside update we just else console.log(ffmpeg data)

Yes, i agree we should fall back to ffmpeg raw (or json?) output. Is there anything cli-progress can do for us in this case?

maybe thumbnails should be hidden too on these terms
do you really want to support these terms?
i mean, if they have more issues, we can print a message and exit
after all, when bins will be available, noone will probably use mingw to run destreamer

I don't, but they seem to be awfully popular out there, i had no idea.
This workaround is enough work for now, next time there's an issue we'll just throw in the towel and exit with a condescending error message.
My last commit makes things look bearable -


MH-13177, Further Simplify MediaPackageElementFlavor

This extends the work pull request #493 and pull request #498 started,
adding a few more simplifications of the media package elements flavor
class.

I we continue with the simplifications, we end up have the class not at all anymore ;-)
Thanks for the follow-up patch.

To be honest, I did shortly look at parseFlavor and the operations that were using it. But I stopped when I realized that you could overhaul that code quite a bit as well ;-D

Note that you could just re-use MH-13174: JIRA will list all pull request that relate to a ticket, so no problem having multiple pull requests for one issue as long as going into the same release. Would also be better for the change log as mentioning "Simplify MediaPackageElementFlavor" once is enough.


Remove the use of add_guard

@gilbo Can you take a brief look at the changes, before I change all the use cite of add_guard ?

Looks great!

@alexreinking Thanks, I still need to remove more use cite of add_guard for this PR

I think it's fine to merge this first, but if you want to wait, that's fine, too.

@gilbo @alexreinking I'll merge this PR if it looks alright~


cannot use --net with -h

Docs in the README reference combining --net along with -h.  However recent versions of Docker disallow this.
$ APP='plex'
$ HOST=`uname -n`
$ CONFIG='/mnt/plex/config'
$ DATA='/mnt/plex/data'
$ docker run -d --net="host" -h ${HOST} -v ${CONFIG}/${APP}:/config -v ${DATA}:/data -p 32400:32400 plex
docker: Conflicting options: -h and the network mode (--net). See 'docker run --help'.
$ docker --version
Docker version 1.5.0, build a8a31ef/1.5.0

Cool don't use it in that form so thanks for picking it up.
Will check what they've changed and sort it out quickly.
Timing was good as was just about to about to add supervisord for someone else's request.

Is there any reason to use --net=host? This should only be required in exceptional cases and is actually discouraged; https://docs.docker.com/articles/networking/#how-docker-networks-a-container

It also allows the container to access local network services like D-bus. This can lead to processes in the container being able to do unexpected things like restart your computer. You should use this option with caution.

I agree should not normally be used.
Part of the problem in this case is the Avahi advertisement of the server that makes it so the players or other servers on the same network are able to pick the server up.
With out --net=host the Plex server is effectively on a different network.
I personally just add the server IP to the clients. Though as an example there isn't an option to do this on the new beta app on roku (I assume they will be adding it at some point).
I agree that this is not normally best practice. Though in this example there is no other way around this that I can think of. (I haven't really played with Avahi so may be a way of tweaking it to do multiple networks).
Do you know of away around this?

Ok quick scan through its using broad cast so with out the --net=host or using something that relays it you can't do it.
Getting someone to set up a replay defeats some of the advantages of docker. Also require some technical know how.

Hm, no I don't have a direct answer to this, but will let you know if I've found something.
Thanks for looking into it though; it's good to know it's there with a reason (I've encountered many cases where people "just" used some settings out of convenience). But I agree, it would be nice if there's a way to make it work without --net=host

For now I've removed the -h from the example.
I'm going to close it for now. Can re-open if anyone comes up with a better way of letting Avahi work without using --net=host

@thaJeztah  Agree with not just using a setting :). Especially one like that or the privileged one.
The only other place i've had to use it is with a proxy one to speed up docker builds when I'm testing.
To be able to do a transparent iptables intercept and redirect. Due to it needing it.

I've not tried it, but what if you open all the ports plex uses?
https://support.plex.tv/hc/en-us/articles/201543147-What-network-ports-do-I-need-to-allow-through-my-firewall-
-p 32400:32400 -p 32443:32443 -p 1900:1900/udp -p 32463:32463 -p 5353:5353/udp -p 32410:32410/udp -p 32412:32412/udp -p 32413:32413/udp -p 32414:32414/udp


Peer discussion page design

Issue Number
fixes #136

Describe the changes you've made
I added a design for peer discussion page
Checklist

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

@poojadas07 I reviewed your code and there are some things I want to suggest:

There are some indentations and code style errors because of which these CI tests failed, you can easily resolve those errors just by following console warnings.

for ex: follow these warnings

As you can see in the above snip that in the console there are some warnings with line number and folder name, you just need to add and remove some spaces in order to solve this.

Every time user opens the doubt form(peer section) on the profile page then the whole website reloads itself which made it look like a static site which led some problems in the android version, I think it is a routing issue. you can easily resolve it by doing some minor changes

I think that this page reload thing is happening because of this line:
 onPress: () => { window.location.href='/peer' },
as you used the window object it will reload itself because the window object in javascript reopersents the browser window.

UI looks a bit inconsistent as compared to the whole site (for ex: color scheme and buttons in mobile view in peer page ) but our main focus is not UI right now we can improve that over time. you can make a separate PR in order to improve UI or you can raise a separate issue to improve UI if you are not interested in designing work, I will take care of that.

okay ....I will check this one

@Abhishek-kumar09 Please check this PR.


FATA[0000] could not get registry endpoint: no registry with prefix 'XXXXXX.dkr.ecr.us-east-1.amazonaws.com' configured

I can't figure it out the reason of this error .
Steps I followed:
1.- export AWS_ECR_PULLSECRET="AWS:eyJwYXlsb2FkIjoiKzkzVWR0blloSW5zdWFrS0I3UXFXcFV6UDU2eDJJem9HaVh0eWN0UWl1YzZwMnQ2OWJvaWJSM3dFMasdpU"
2.- argocd-image-updater test XXXXXXXX.dkr.ecr.us-east-1.amazonaws.com/k8s/ingress-nginx --update-strategy latest --credentials env:AWS_ECR_PULLSECRET
Result:
INFO[0000] getting image                                 image_name=k8s/ingress-nginx registry=XXXXX.dkr.ecr.us-east-1.amazonaws.com
FATA[0000] could not get registry endpoint: no registry with prefix 'XXXXXXXXX.dkr.ecr.us-east-1.amazonaws.com' configured
any ideas about how to fix it ?
thanks

I needed to include a prefix in the configmap  registries

I needed to include a prefix in the configmap  registries


fix(MultiSelect): remove item autosort behavior

Closes IBM/carbon-components-react#1897
This PR removes the automatic sorting after selecting <MultiSelect> options for v10 onwards

@IBM/carbon-developers Can anybody do another review...? Thanks!


Issue building on Ubuntu 2204 jammy

I've been bumping a docker container from Ubuntu2004 to Ubuntu2204 and encountered the following error:
# github.com/google/go-tpm-tools/simulator/internal
In file included from /root/go/pkg/mod/github.com/google/go-tpm-tools@v0.2.1/simulator/ms-tpm-20-ref/TPMCmd/tpm/include/BnValues.h:317,
                 from /root/go/pkg/mod/github.com/google/go-tpm-tools@v0.2.1/simulator/ms-tpm-20-ref/TPMCmd/tpm/include/Global.h:82,
                 from /root/go/pkg/mod/github.com/google/go-tpm-tools@v0.2.1/simulator/ms-tpm-20-ref/TPMCmd/tpm/include/Tpm.h:52,
                 from /root/go/pkg/mod/github.com/google/go-tpm-tools@v0.2.1/simulator/internal/internal.go:34:
/root/go/pkg/mod/github.com/google/go-tpm-tools@v0.2.1/simulator/ms-tpm-20-ref/TPMCmd/tpm/include/Ossl/TpmToOsslMath.h:52:5: error: #error Untested OpenSSL version
   52 | #   error Untested OpenSSL version
      |     ^~~~~

I've only found the following related issue: https://github.com/ibm-openbmc/dev/issues/3452
Anyways, my workaround was to manually install libssl-dev_1.1.1:
wget http://security.ubuntu.com/ubuntu/pool/main/o/openssl/libssl1.1_1.1.1f-1ubuntu2.16_amd64.deb
sudo dpkg -i libssl1.1_1.1.1f-1ubuntu2.16_amd64.deb
wget http://security.ubuntu.com/ubuntu/pool/main/o/openssl/libssl-dev_1.1.1f-1ubuntu2.16_amd64.deb
sudo dpkg -i libssl-dev_1.1.1f-1ubuntu2.16_amd64.deb

Are there any plans to address this issue?

Right now the version of our simulator only works with OpenSSL v1, so it's required to build the code.
We will need to update the simulator to a newer (potentially patched) version to make this work.

@m-kostrzewa this particular issue was fixed in https://github.com/google/go-tpm-tools/commit/51429b623805be4a2db6f6ff3763b27d6bb36e6b so if you use go-tpm-tools v0.3.0 or later you shouldn't have this issue.
It seems like you are currently using v0.2.1

Just a mention, since it may be relevant to someone. I just checked it and you're right. I've been using v0.2.1 as it's been a dependency of google/go-attestation. They have bumped go-tpm-tools dependency from v0.2.1 to v0.3.1 on Dec 9, 2021 (literally days after I first installed it :roll_eyes: ).


fix readme

The README seems to reference a from_str function that doesn't exist any more.  The following code doesn't work for me:
parse_datetime::parse_datetime::from_str(tstr)

returning
failed to resolve: expected type, found function `parse_datetime` in `parse_datetime`

If instead I change it to
parse_datetime::parse_datetime(tstr)

it works fine.

Thanks!


[NFC][asm igemm]Refactor asm igemm nhwc perf config

Refactor PerformanceConfigAsmImplicitGemmGTC as #1268 mentioned.
It still has compile error.
@atamazov could you please check if the changes are in the right way?

Of course. I'll look as soon as time permits, hopefully today.

@atamazov Will this PR affect the perf-db?

This PR has gone stale and has not been updated in more than six months.

This PR has gone stale and has not been updated in more than six months.

@JehandadKhan This is my fault as I missed reviewing this. @shaojiewang can you please re-open PR and update from develop? I will review soon after that.


Refactor profile validation rules in ProfileRequest.php

Issue Reference
Refactor profile validation rules in ProfileRequest.php #9726

Checked and Found that the validation has been removed.
Video
https://github.com/bagisto/bagisto/assets/92721837/6c2a5fd1-42ba-46cf-a660-ddca8648656d


core_schema.literal_schema behavior on nan values

Related to https://github.com/pydantic/pydantic/issues/10037
from pydantic_core import SchemaSerializer, core_schema
s = SchemaSerializer(core_schema.literal_schema([0.1]))
s.to_python(float("nan")) # outputs: nan
s.to_json(float("nan")) # outputs: b'null'

New to pydantic-core but the issue here points out that the behavior of model_dump_json() and model_dump(mode="json") should be the same -- if so, then should s.to_json(float("nan")) output b'NaN' instead of b'null'?

I tried and checked here:
>>> json.loads('{"a": NaN, "b": "12345"}')
{'a': nan, 'b': '12345'}

so json can handle NaN so outputting NaN seems more in-line with the RFC

This seems like the most obvious place to make the change -- I haven't figured out how to work with Rust + Python together yet. But I believe this could help?

Looks like the default value for inf_nan_mode=InfNanMode::Null in this line

okay, more exploration and I think the simple way to get the NaN behaviour is by passing in a config for ser_json_inf_nan:
from pydantic_core import SchemaSerializer, core_schema, CoreConfig
s = SchemaSerializer(core_schema.float_schema(), config=CoreConfig(ser_json_inf_nan="constants"))
print(s.to_json(float("nan"))) # outputs: b'NaN'

Is this only available in pydantic-core or can it be passed in from pydantic as well?

turns out ConfigDict does exactly this


CI: Report for tagged tests

What this PR changes/adds

Create a composite action for a generic task to run tests + upload test report XMLs as artifacts
Merge the two workflows for UT and IT into one (required for the item below)
Organize the workflow to run UT and ITs in several parallel jobs, and then a final job that combines the result XMLs into one report

Why it does that
GitHub PR test report previously contains only unit tests. With this PR it also includes integration tests.
Further notes
Also merged the two jobs running AWS integration tests into one.
See sample PR with a failing test, showing the report related actions are also executed in that case: https://github.com/agera-edc/DataSpaceConnector/pull/212
Linked Issue(s)
Closes #1046
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

removing myself as reviewer since @ndr-brt already reviewed it.

@juliapampus @jimmarino this PR is OK to merge, thanks


Home App under iOS 11.2.1 does not find device

Hello,
I am currently using the Homebridge on a Raspberry Pi 3 in combination with the following plugins:
homebridge-config-ui
homebridge-homematic
homebridge-broadlink-rm
homebridge-http-simple-switch
homebridge-fritz
Everything is up to date and works fine so far.
Now, I planned to move the Homebridge into a Docker container on my DS718+. The installation worked flawlessly and the Homebridge started without any error message (except from the well known warnings). See here and here for more details. However, my problem is that in contrast to the Raspi installation now the Home App under iOS 11.2.1 can not find the respective device anymore. I have installed no firewall under DSM 6.1. Do you have any idea what to do?
Cheers,
Thorsten

I changed the IPv6 settings

The container doesn't work well with IPv6. Is it disabled for you local lan?

I will check that.

The container doesn't work well with IPv6. Is it disabled for you local lan?

I tried both disabling and enabling IPv6. Unfortunately, there has been no success in either case.
Someone wrote that the issue might result from the fact that the DS718+ has two Ethernet interfaces. However, I only use one of them. What about the other guys having the same issues? Does your DiskStation has two LAN ports as well?

My 415+ also has two lan ports. I use them with a bond and had no problems with it.

Maybe it help others:
My iOS devices also could not find Homebridge (running in Docker on Synology).
After I stopped the second docker image that was running ioBroker, all of them could find Homebridge.
Digging a bit into it, a network scan while running both docker images showed that my Synology was not found under its normal name "Diskstation" but under the ioBroker docker image name.
I was able to start ioBroker docker image afterwards without any problems to the Homebridge connection.


rc_dynamics_api: 0.10.0-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository rc_dynamics_api to 0.10.0-1:

upstream repository: https://github.com/roboception/rc_dynamics_api.git
release repository: https://github.com/roboception-gbp/rc_dynamics_api-release.git
distro file: noetic/distribution.yaml
bloom version: 0.9.3
previous version for package: null

rc_dynamics_api
* improve exception handling using new NotAvailable

@flixr Looks like there's a merge conflict, mind rebasing this PR?

sure, rebased...


Can't build .NET 6 from source

Greetings, I'm following all the steps to build .NET 6 from source in my Ubuntu 22.04.1 LTS and they succeed until I get to the step where I've to run this command...

...and I get the following error...

This looks like building from the source-build tarball.  It looks like you might have a corrupt set of package archives - could you delete packages/archive/*.tar.gz and the .dotnet directory at the root and try ./prep.sh and ./build.sh again?  If that doesn't work, could I get the contents of packages/archive/archiveArtifacts.txt?


Unable to get Coral PCIE Driver to load

Description
I am unable to get the Coral PCIE driver to load on a fresh Ubuntu 22.04.1 LTS VM. This VM is running on ESXi. The Coral PCIE device has passthrough enabled passing the device directly to the Ubuntu VM.
I followed the instructions provided at https://coral.ai/docs/m2/get-started/#2-install-the-pcie-driver-and-edge-tpu-runtime
I am able to see the device but the driver never loads.
user@coraldev2-srv:~$ lspci -nn | grep 089a
03:00.0 System peripheral [0880]: Global Unichip Corp. Coral Edge TPU [1ac1:089a]
user@coraldev2-srv:~$ ls /dev/apex_0
ls: cannot access '/dev/apex_0': No such file or directory
Kernel Version-
user@coraldev2-srv:~$ uname -r
5.15.0-69-generic
Click to expand!
Issue Type
Build/Install
Operating System
Ubuntu
Coral Device
Mini PCIe
Other Devices
No response
Programming Language
No response
Relevant Log Output
No response

Hi, could you please uninstall the apex drive and share the installation logs.
sudo apt-get install gasket-dkms

Thanks.. Can you please check the output of below command. You might be having same issue as others in this thread: https://github.com/google-coral/edgetpu/issues/343
sudo dmesg |grep apex

user@coraldev2-srv:~$ sudo dmesg |grep apex
[    8.835089] apex 0000:03:00.0: Page table init timed out
[    8.838448] apex 0000:03:00.0: MSI-X table init timed out
[    8.848389] apex: probe of 0000:03:00.0 failed with error -110

Yes, same as the other issue: https://github.com/google-coral/edgetpu/issues/343
Closing the issue here, If you would like, please add a comment their if  anyone has as any suggestions or workaround for this. Thanks!

Thanks for the confirmation...


Route timeout conformance tests

What type of PR is this?

/kind test
/area conformance
What this PR does / why we need it:
Conformance tests for HTTPRoute request and backend request timeouts.
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

NONE

Too small of numbers / differences and it may be flaky, too high and its slow..

@howardjohn I tried to find a sweet spot by making the timeout half a second and then calling a service delayed 1 sec and no delay, so we have a half second buffer in either direction and only a couple of slow tests with half second waits.

/ok-to-test
/cc @arkodg @mlavacca @sunjayBhatia

thanks @frankbu, hoping this has been manually tested in at least 1 implementation

Yep, it's tested and passing in Istio.

@robscott @shaneutt Is there anything else needed to finish this off? If not, can we merge it now?

@robscott I believe I've addressed your review comments and confirmed that the new tests are passing in Istio. Please let me know if there's anything else needed before merging.
Regarding the two echo images used for conformance testing, I think we're stuck with them both, at least for now. See my comment here: https://github.com/kubernetes-sigs/gateway-api/pull/2254/files#r1340651689

Thanks @frankbu!
/lgtm


Report errors from the workflow phase

This produces a much more comprehensible error than the one the refers to JS phase.
With that in hand, it was possible to debug and enable a more comprehensive date_part regression test that had been pending.

Current coverage is 64.58% (diff: 71.42%)

No coverage report found for master at 973340d.

Powered by Codecov. Last update 973340d...536adeb

Finally, A legitimate conflict!


#4614: gitmodules: Use https URLs for submodules

This means that if you clone with https://github.com/tenstorrent-metal/tt-metal, the submodules will also use https. Slight removal of friction for those not using SSH for github (cloning).
The alternative would be to use relative submodule paths, but that means that anyone forking the tt-metal repo also needs to fork the submodules for them to be possible to clone, which is awkward.

@olofj @TT-billteng going to merge this as soon as it passes.


Please provide java/android code.

Please provide java/android code. @SwatiModi

The Android skeleton code is written in MediaPipe itself and is present in this repo