Make Fake data generator to generate valid phone numbers

Make the Fake data generator to generate valid phone numbers.  Therefore, they will pass the parsePhoneNumber (phone number validation).  For detail of phone validation, see Element.kt:794 - fun parsePhoneNumber(phoneNumber: String): Pair<String?, Phonenumber.PhoneNumber?> {}.

No longer needed.


Generated TypeScript code contains references to an undefined name '_Service'?

package.json
  "dependencies": {
    "fastify": "^4.3.0",
    "mercurius": "^10.1.0",
    "mercurius-codegen": "^4.0.1",
  }

Steps to reproduce:

Clone https://github.com/api-haus/mercurius-codegen-bug
Run npm install && npm run codegen to generate code

TypeScript errors in generated code:
src/generated.ts:148:33 - error TS2304: Cannot find name '_Service'.

148   _Service: ResolverTypeWrapper<_Service>;
                                    ~~~~~~~~

This issue is persistent for many months. I wonder why this is happening. Previously we had a working version and just removed broken code from file so the typings would work correctly.

As a workaround I can create a global definition for _Service.
export {};

declare global {
  type _Service = any;
}

as a workaround, you can define new scalar
scalar _Service


rename AlarmRegistry in pubsublite

As mentioned here https://github.com/googleapis/google-cloud-cpp/pull/8575, rename AlarmRegistry to something like PeriodicTimer.

Punt for now.

We are not actively working on Pub/Sub Lite, will retake this once we start again.


partial aggregation does not work

apend the following to the end of the file examples/benchmark/run_snakemake.sh,
now i want to test the rule
--allowed-rules agg_partial_results
works with
snakemake --cores 1 -s "domainlab/exp_protocol/benchmark.smk" --configfile "examples/yaml/demo_benchmark.yaml" --keep-going --allowed-rules agg_partial_results
I got
Building DAG of jobs...
total_num_params=5
=2 * 2 + 1
MissingInputException in rule agg_results in file /home/sunxd/DomainLab/domainlab/exp_protocol/benchmark.smk, line 96:
Missing input files for rule agg_results:
    output: zoutput/benchmarks/demo_benchmark/results.csv
    affected files:
        zoutput/benchmarks/demo_benchmark/rule_results/2.csv

@smilesun It seems to me, that there is no intuitive way for aggregating partial results with snakemake.
At least when I'm trying this, often reruns of some rules are triggered, you have to clean up with incomplete files etc.
I think it would be best to add a command line argument to Domainlab, which, given the benchmark config, aggregates all available results and generates the graphs. However, I don't know how this is on clusters.


Request textDocument/codeAction failed.

I provided a Testrepo where it is pretty easy to reproduce this error within the Output of Eslint within Visual Studio Code.

npm i
Open foo.json
Go to the Eslint Output within Visual Studio Code

That's it.
[Info  - 1:13:01 AM] ESLint server is starting.
[Info  - 1:13:02 AM] ESLint server running in node v16.17.1
[Info  - 1:13:02 AM] ESLint server is running.
[Info  - 1:13:04 AM] ESLint library loaded from: C:\Home\tmp\eslint\node_modules\eslint\lib\api.js
[Error - 1:13:12 AM] Request textDocument/codeAction failed.
  Message: Request textDocument/codeAction failed with message: Invalid regular expression: //** eslint-disable-next-line/: Nothing to repeat
  Code: -32603

What is the reason for this error?
I cannot find any regular expression that looks like the one within the error message, not even within node_modules.

This was fixed in #1611, but I think that there hasn't been a release since then.

Dups #1611
Will look into creating a new pre-release this week

Thank you very much

Published pre-release 2.4.1

I was also experiencing this issue. The pre-release fixed it. Thanks @dbaeumer!

When can we expect this to be released?

2.4.2 got released today.


Multiple versions bumps: patternfly (6.0.0), eslint-plugin-jsx-a11y (6.10.2), eslint-plugin-react (7.37.2) and http-proxy-middleware (cve)

Description
Multiple versions bumps: patternfly (6.0.0), eslint-plugin-jsx-a11y (6.10.2), eslint-plugin-react (7.37.2) and http-proxy-middleware (cve)
How Has This Been Tested?
Run web app locally.
Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[X] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work.
[X] Code changes follow the kubeflow contribution guidelines.

/lgtm


#15 cleaned up tests

Codecov Report

Merging #18 (a60aaa2) into master (67c07a5) will increase coverage by 1.07%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   26.27%   27.35%   +1.07%
==========================================
  Files          63       63
  Lines        1583     1583
  Branches       51       51
==========================================
+ Hits          416      433      +17
+ Misses       1143     1127      -16
+ Partials       24       23       -1

Files
Coverage Δ

...rc/Contracts/User/CreateUser/CreateUserResponse.cs
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Template needs updating to conform to current standards

When deployed the solution fails most security policy checks, mainly due to the storage account being gen1 and defaulting to TLS 1.0. Please update this to use gen2 with tls 1.2 default.

Yes, we have the same issue. Tenant policy requires that storage accounts be restricted to TLS 1.2 and above, and since we cannot use an existing storage account, trying to deploy Start/Stop VMs v2 fails validation. Is there any plan to adjust this? Being able to use an existing storage account would fix the problem. Thank you.

This has now been addressed. As of early 2024, the latest versions of StartStop V2 use storage accounts that require TLS 1.2 to be used.


add options for placeholder?

How about an option to change the input placeholder from Add new country?
Thank you

Fixed in PR #6


TypeScript: Enum array treated as ObjectArray and produces a wrong URL

I have an enum like this:
export enum UserRole {
    Role1 = "Role1",
    Role2 = "Role2",
    Role3 = "Role3"
}

When generating a TypeScript client, the parameters in the method is generated as includedUserRoles: any[] and therefore the it's being treated as an ObjectArray. See the template on these lines.
This results in a wrong URL:
IncludedUserRoles[0].0=R&IncludedUserRoles[0].1=o&IncludedUserRoles[0].2=l&IncludedUserRoles[0].3=e&IncludedUserRoles[0].4=1
Expected output is simply IncludedUserRoles=Role1.
Is this related to Swagger generation or array handling in NSwag?

Can you also post the produced swagger spec?

Here's a swagger.json.

I'd say the output is as expected... you need to change the parameter type to array of this enum not array of any

The swagger is generated from C#
public List<UserRole> IncludedUserRoles { get; set; }

With what do you generate the spec?
It should be:
            "type": "array",
            "items": {
                "$ref": "#/definitions/UserRole"
            },

Can you do that? Maybe with an operation filter/processor?


NPM package peer dependencies

I had a lot of issues installing this package (required by a third-party component...) because of the strict versions declared as peer dependencies in package.
Example: it requires @angular/router: "16.0.0", but I actually have 16.0.3 installed...
It forced me to use the --legacy-peer-deps npm option to complete the installation.
  "peerDependencies": {
    "@angular/common": "^15.1.0 || ^16.0.0",
    "@angular/core": "^15.1.0 || 16.0.0",
    "three": "^0.148.0 || ^0.149.0 || ^0.150.0 || ^0.151.0 || ^0.152.0",
    "rxjs": "7.8.0",
    "@angular/router": "16.0.0"
  },

IMHO it should read like this (added ^ prefix to @angular/core, rxjs and @angular/router versions):
  "peerDependencies": {
    "@angular/common": "^15.1.0 || ^16.0.0",
    "@angular/core": "^15.1.0 || ^16.0.0",
    "three": "^0.148.0 || ^0.149.0 || ^0.150.0 || ^0.151.0 || ^0.152.0",
    "rxjs": "^7.8.0",
    "@angular/router": "^16.0.0"
  },

I', having the same issue

Is anyone taking care of this?
I could open a PR for it... but I see there's one waiting till 2023-05-19 🤔...

Can't wait for #35 to complete, then...😉
Keep up the good work!


Fix shutdown sequence with cp mgr.

Add blocking cp flush in cp mgr shutdown. Added atomic to stop cp to trigger another cp.
Destroy in btree and index dont need cp context. As cp mgr already shutdown.

Test meta blk used directly calling meta service api's to simulate recovery, instead use homestore restart to simulate a recovery.

Codecov Report
Attention: 996 lines in your changes are missing coverage. Please review.

Comparison is base (9a91944) 68.21% compared to head (5ecde2e) 0.00%.
Report is 17 commits behind head on master.

Files
Patch %
Lines

src/lib/replication/repl_dev/raft_repl_dev.cpp
0.00%
251 Missing :warning:

.../lib/replication/log_store/home_raft_log_store.cpp
0.00%
136 Missing :warning:

src/lib/replication/service/raft_repl_service.cpp
0.00%
130 Missing :warning:

...rc/lib/replication/repl_dev/raft_state_machine.cpp
0.00%
109 Missing :warning:

src/lib/replication/service/generic_repl_svc.cpp
0.00%
75 Missing :warning:

src/include/homestore/superblk_handler.hpp
0.00%
34 Missing :warning:

src/lib/replication/log_store/repl_log_store.cpp
0.00%
28 Missing :warning:

src/lib/replication/repl_dev/common.cpp
0.00%
24 Missing :warning:

src/lib/replication/repl_dev/solo_repl_dev.cpp
0.00%
24 Missing :warning:

src/include/homestore/btree/detail/varlen_node.hpp
0.00%
23 Missing :warning:

... and 29 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #272       +/-   ##
==========================================
- Coverage   68.21%   0.00%   -68.22%
==========================================
  Files          96     111       +15
  Lines        8016    9002      +986
  Branches     1041    1208      +167
==========================================
- Hits         5468       0     -5468
- Misses       2033    9002     +6969
+ Partials      515       0      -515

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


InputBox Caret Position always resets to last character

The arrow keys on the InputBox don't work at all because ValidateEdit() calls PositionCursor(), which always resets the caret to the last character. I'm found the bug while using the NuGet release but I see the error in the current source code.
https://github.com/Thraka/SadConsole/blob/9d3d9ee22edef7e61f841c1164b43e1be4a291a4/src/SadConsole.Shared/Controls/InputBox.cs#L315

thanks! this used to work and i haven't taken the time to investigate it.

I think I need to rethink the textbox design. It's a little too fragile at the moment with trying to fix this bug.

This was fixed in #183 !! Thanks @darrellp


Feat/share common codebase

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/cw_storage.wasm
313.23 KB (0%)

target/wasm32-unknown-unknown/release/cw_law_stone.wasm
636.51 KB (+2.6% 🔺)


Converted HaxlSharp.Core and HaxlSharp.Fetcher to PCLs

I converted the Core and Fetcher projects to portable class libraries in order to use them in a wider range of projects.

I agree 100%. I guess I should have done that before making the pull request. I did try the PCLCrypto nuget package, but their MD5 hash failed when I ran your tests. I have some time this evening so I can take a look.

Should be good to go now. I swapped out that library with the BouncyCastle nuget package. All tests passed.

Looks good, thanks mate!


Some classes won't download

I purchased 5 classes tonight.  2 of them download fine.  The other 3 don't start.  They get stuck on the message "Building Course Data" and never proceed past that.  They don't show up in the download list and they don't create the folder to put the class data in.  I'm using 1.6.1.  Not sure what else I can tell you.
I think I found a free class that duplicates the issue.  GarageBand 101- Learn How To Write & Record Your First Song!.  https://www.udemy.com/garageband-101-learn-how-to-write-record-your-first-song/.

I have the same issues. I think there is restrict setting from UDEMY instructor setting that disable for download. Any solutions for this fix?
Thanks,

I have the same issues.

Hi @Smitty010 Thanks for reporting the issue. This issue has been reported before as well where some users aren't able to download some courses. The only thing stopping me from fixing this is that I can only reproduce this issue if I have access to those courses.
I see that you have linked me to a free course that has same issue, but it appears to work for me. It downloaded just like other courses. Can you try again on the same course.
And since most of the courses that have this issue are paid courses. I would need access to one of those courses. If anyone here is willing to share their credentials (through email) for the sake of fixing this issue, let me know.

I was having the same problem, now it looks like is working.
Thanks a lot

Looking good on my machine.  I think you fixed it.  Thanks

I don't see this option??

I still face the same issue with version 1.8.2 on both windows and mac for some course. enabling and/ or disabling "skip attachment" and "skip subtitles" didn't work. Has anyone found a solution for this issue?

Same here worked until a month ago, Works on some fails on other
Works: Android 50: Crash Course into Android Development
Fails: HashiCorp Certified: Terraform Associate Practice Exam 2020

Same here worked until a month ago, Works on some fails on other
Works: Android 50: Crash Course into Android Development
Fails: HashiCorp Certified: Terraform Associate Practice Exam 2020

I am still having this issue. Skipping subtitles or attachments doesn't work.

I have issue on these courses:
React - The Complete Guide (incl Hooks, React Router, Redux)
Learn Python Programming Masterclass

I think this happens when you have downloaded a course before and want to download again

React - The Complete Guide (incl Hooks, React Router, Redux)
I am having the same problem for this video.
I would translate the .srt file because my english is not enough.
what should i do now

Having the same issue right now.
Stuck on "Building Course Data"

FaisalUmair
Please solve problem,
Thank you

please solve this problem

Having the same issue right now.
Stuck on "Building Course Data"
The course is : Javascript: The complete guide 2021

I think this instructor is deliberately making it impossible to download his courses. its the same with all his courses

Having the same issue right now.
Stuck on "Building Course Data"
The course is : Javascript: The complete guide 2021

I think this instructor is deliberately making it impossible to download his courses. its the same with all his courses

ah now I get it, thank you !

After Effects CC 2020: Complete Course from Novice to Expert
Also has problem
Udemy has changed its website.
FaisalUmair Please

React - The Complete Guide (incl Hooks, React Router, Redux)
Having the same problem stuck on building data!

facing the same issue, downloaded a course last month and it worked fine. But now I cannot download any paid course only the free courses are downloadable. Please note that I cannot download any paid course I tried downloading all my previous courses but got no luck all of them are stuck with the same message "Building Course Data". This issue is not linked with any particular instructor it's happening with all the paid courses.

I have downloaded paid course
The problem is not this

I have downloaded paid course
The problem is not this

When did you download it ?

It's not as simple as a paid course or not.  I tried to download 4 courses
a few days ago.  2 of them worked.  2 of them didn't.  They were all paid.
On Wed, Apr 21, 2021 at 6:10 AM napthenod @.***> wrote:

I have downloaded paid course
The problem is not this
When did you download it ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FaisalUmair/udemy-downloader-gui/issues/145#issuecomment-824010892,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4V7SYKXOKA6Q5YYDD5KLTTJ2XDPANCNFSM4FNNSY3Q
.

I faced the same issue in recent days. I couldn't download any course. Was anyone able to fix this issue?

facing the same issue, courses can be downloaded but one specific course stuck at building course data, the exact scenario was i started downloading the course then stopped the tried to resume it stuck.
other courses working good.
tried to logout and log back in but nothing changes
also tried the move the downloaded videos to other location, so download can start from the beginning but nothing happened.
kindly fix this issue

I downloaded a few courses last week - but facing this issue since last couple days. Stuck at building course data for all courses - tried with different courses but the same result. :(

I too facing same issues on 1.8.2 release of Mac app

i have same issues with 1.8.2 on mac app this days
what is the problem

The same paid courses that I was able to download few weeks ago are no longer downloadable...
@FaisalUmair  could you kindly look into it? Thanks in advance!

Some courses can be d/led, some cant, just keep saying building course data, why is that?

@FaisalUmair Getting Building Course Data on every course except for a free course that I have. I understand that you mentioned you won't be fixing issues for the app now, but this basically renders the app completely useless. Please take a look into it. That'd be very considerate of you. Thanks


Add Discord webhook integration

What is the purpose of the PR?

Integration kubewatch with Discord channel
Send content and color message same MS Teams
Update docs for Discord config

Testing:

Test wit unit test
Test on my cluster

Please merge this, I want to start using kubewatch with my discord server

I am also interested in this feature.


Examples folder

Hi everyone,
I enjoy using urdf2casadi in general.
However, there is a few things that currently do not work properly in the provided examples.

python3 support: The example in the README.md was changed to python3 in b1a9f7d
dependencies: The example are dependent on other libraries such as rbdl, kdl_parser_py, etc -> this is not in the installation guide
root link with inertia: Once all of the dependencies are installed, kdl fails to load gantry.urdf for example duo to a rootlink with inertia

Maybe these issues can be addressed in the future.
I am also willing to help if required or desired.
Best,
Max.

Hi Max!
Glad to hear you enjoy the library in general.

* python3 support: The example in the README.md was changed to python3 in [b1a9f7d](https://github.com/mahaarbo/urdf2casadi/commit/b1a9f7d653afda980b2a9bd498d889213548f970)

What exactly is the problem there? Are there other places where this library fails with python3? It was intended to work with both python2 and python3, but the numerical and timing examples which were used to generate the results in the article have not been updated to reflect this.

* dependencies: The example are dependent on other libraries such as rbdl, kdl_parser_py, etc -> this is not in the installation guide

The numerical testing examples which were used to generate the results in the article are dependent on rbdl, kdl_parser_py, and pybullet as stated here: numerical/README but I see that it this information is missing from the timing folder. Currently rbdl seems to be having some issue with their python wrapper, so perhaps it is best to move numerical and timing examples to a article_results folder? This might make it clearer that they were intended for the article, and can be used to test for oneself whether things work as expected with your desired URDF. Another alternative is to split the examples into a separate repository so as to not have too much data in the main repo.

* root link with inertia: Once all of the dependencies are installed, kdl fails to load gantry.urdf for example duo to a rootlink with inertia

This is an issue in kdl_parser, and does not affect the algorithm in this library, if I'm not mistaken.
Any contributions that further develop the library are greatly appreciated! Things that haven't been done but could be interesting include:

More examples (e.g. clarifying examples)
Testing on other systems (ROS vs non-ROS, windows)
Support for kinematic trees rather than just kinematic chains

Hi mahaarbo,
I update all the python print statements to be compatible with python3. I will shortly make a pull request.
I didn't see these requirements in the README file, I am sorry for that.
As I am working with the franka emika panda, I might add an example with that in the future.
Best,
Max.

If there's nothing left to add at this moment, I'll close this issue. Thanks for the PR!


Enable OS server console access from ccloudvm host browser

This commit enables console access to OS servers from horizon running on the web browser of the ccloudvm host.  It does this by modifying ccloudvm to pass the HostIP into instance workloads if that IP address is local and by using this IP as the openstack HOST_IP in the devstack workload.  It also opens up the port 6080 in the devstack workload to permit console access from the ccloudvm host.

@ganeshmaharaj could you take a look at this one?

Coverage decreased (-0.09%) to 68.226% when pulling 72640371fc3ed3b1dd5571dd6bdc65a471ecf462 on markdryan:fix-os into b329fc845bfcd1cc92b82de804f5a6112bea994a on intel:master.


Make program parallel

Find a way to analyze chunks parallel on many threads / cores. Program work is almost only count characters, until it comes to adding the counted characters of the different chunks no result is needed from the previous operation. In general the result of  operation "before" (count chunk n-1) is not needed to work on currrent operation (count chunk n), and to work on chunk n+1 results of of chunk n are not needed and so on ...
Assumption: Parallelized program should perform almost linear with core count. At first glance, only input bandwidth from mass storage (HDD, SSD) may limit the degree of parallelization.

Its not a malfunction -> not an issue. Has become a milestone.
See https://github.com/eitieatgithub/wordlistStatisticsPython/milestone/1


Update top level trace information

What does this PR do?
Adds top level type and service name.
Additional Notes

Review checklist
Please check relevant items below:

[ ] This PR includes all newly recorded cassettes for any modified tests.

[ ] This PR does not rely on API client schema changes.

[ ] The CI should be fully passing.

[ ] Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.


Update to GOV.UK Frontend 4.2.0; use govukPagination component

Update govuk-frontend to v4.2.0

Update govuk-prototype-components to 0.4.0

Update module imports for the above packages (issues from #644 appear to have been fixed)

Update govuk-components to v3.1.1, which includes govuk_pagination

Replace bespoke pagination component with that provided by the above and govuk-frontend v4.2.0

Update tests – design system recommendation is not to show previous/next page links on first/last pages:

Do not show the previous page link on the first page — and do not show the next page link on the last page.

(Preview attained by temporarily changing item limit in Page’s config)

(Am rethinking showing total number of results per page below pagination; seems slightly repetitive given we provide a summary of results above the table, too. For another day…)


Axis with scaleLog base 10 yields too many labels

Here's a reproduction of the issue: http://bl.ocks.org/guilhermesimoes/af7ef9fe50c7c71b2165
Just switch the scale to Log10.
You can see that this does not happen in the latest release of D3: http://bl.ocks.org/guilhermesimoes/15ed216d14175d8165e6

By the way, using the same examples, we can see that the latest release of D3 renders the wrong labels in a log2 axis. There, the third red dot (100) is located between 6e+1 (60) and 1e+2 (100) which is obviously wrong.
In the d3-axis and d3-scale example this no longer happens. There, the third red dot (100) is correctly located between 64 and 128.

Thanks for the test case. I will investigate. Can’t tell yet where the problem lies.

Fixed in d3-scale 0.5.2.

My block is already working and everything :smile:
Thanks for the hard work! Can't wait for the 4.0 release!


Digest authentication was improved

Using information from here digest authentication method was expanded to support more subtypes of this auth type.

SabreDAV is throwing a 500 error with this. I'll look into it when I find some time.

@chuckwagoncomputing , can you provide SabreDAV server's response, please?

It's a 500. It appears that the body of the propfind request is not being sent. I tried my last change again and got the same result. Apparently something has changed on my end, but I'm not sure what. If this change is working for you I'd say we should go ahead and merge it and I'll try to find my problem from there.

@chuckwagoncomputing , I create issue #20 to describe this problem. After I tried to find problem, found it, fix it and send this fix as pull-request #21. When you will have a time, please try to use the code from this pull-request to make request to SabreDav server, and write results here. Hope it fix this problem.

@chuckwagoncomputing , any news?

I can confirm that this is working with SabreDAV.

Awesome! Thank you very much.


Add multiapi combiner

Create venv with the multiapi package you want to combine installed as an editable instalation
Pass local path to pkg python multiapi_combiner.py --pkg-path /Users/isabellacai/Desktop/github_repos/azure-sdk-for-python/sdk/network/azure-mgmt-network

See https://github.com/Azure/azure-sdk-for-python/pull/27973 instead


Journey picker is broken

https://discuss.newrelic.com/t/customer-journey-nerdpack/83270/2

We set up two journeys in config file, but only the first one is always displayed. Each one as works when it is the only one in the file, but when there are tow - we can switch to second in the UI but we still see the first one only. Should it work with multiple journeys?

Resolved 2be14ead85efe8e2c2f7af4199061388ee9cea9d


Discussion: MAUI controls

Every platform has basic controls such as buttons and switches, but more complex controls like NavigationView or TabView are either only available in WinUI or don't function in the same way.
Wouldn't it make sense to decouple the non-primitive controls from WinUI and move them over to MAUI? This way you could actually get them on every platform.

Android has a few concepts that would be an equivalent.  Also the NavigationView can be orientated to a top bar.
WinUI could enable bottom placement to work with iOS and Android navigation.
But at some point custom Xaml WinUI controls will need to be created to match common patterns from macOS, iOS, Android etc
In some circumstances, it may be appropriate to create versions of WinUI controls that work on the other platforms too

I think it would be best if MAUI created its own controls based on common primitives found on each platform, like Stackpanel or Grid.
For example in dotnet/maui#542 controls like DataGrid are mentioned, which may not exist natively in any of the platforms. If they're built on WinUI only (#1500), they'll be useless for the other platforms.

I think it would be best if MAUI created its own controls based on common primitives found on each platform, like Stackpanel or Grid.
For example in dotnet/maui#542 controls like DataGrid are mentioned, which may not exist natively in any of the platforms. If they're built on WinUI only (#1500), they'll be useless for the other platforms.

This is where Fluent UI plays it's part.  Creating controls for Web, Android, iOS, etc. And design tokens shared among all of these and WinUI, allow for customisations like corner radius, Shadow, colours, metrics etc.
MAUI can support these as a shared set of controls, whilst still allowing native controls and rendering where developers do not want to use the same pattern for certain platforms.
There is already a Cross Platform effort going on with Fluent UI, to standardise basic primitives like Button and Text

No way, not at the cost of Windows ecosystem which is already almost ruined. This would greatly slow down development, plus, cross-platform controls would have limited feature set, poor quality and design. Current workflow, when one team polishes a library for Windows, and another one uses it in their cross-platform framework, makes more sense. If they didn't create full copy of NavigationView or TabView, perhaps they had reasons, especially keeping in mind that all major platforms are proprietary. As competitors they may use common primitive controls like buttons and text, but have different requirements for concepts of higher order, like navigation, layout, design or input. And if Windows would only have controls implementing common subset of such requirements without advanced features, it would be just incompetitive.


The .msixbundle file cannot be installed

My computer cannot install the msixbundle file, and the system does not have a Microsoft store, so I tried various methods, but still cannot install the notepads . I hope you can produce an .exe file to facilitate the installation of people like me. Thank you very much!
OS: Windows 10 21H1 Build 19043.928

Did you modify Windows installation to remove Windows store? Or what installation methods did you use to install windows?

Sorry, I did not see your reply in time. The Windows system I installed is a lite version, so there is no Microsoft store, nor can I install msixbundle files.（Windows 10 21H1）
OS: Windows 10 21H1 Build 19043.928

I tried various methods of restoring the Microsoft Store and installing msixbundle files on the Internet, but failed, so I came to ask you, but there may be only a small number of people on a system like mine. If possible, I still hope to be able to produce an exe file. , Thank you very much!

Sorry, I did not see your reply in time. The Windows system I installed is a lite version, so there is no Microsoft store, nor can I install msixbundle files.（Windows 10 21H1）
OS: Windows 10 21H1 Build 19043.928

I can't find any details on a "Lite" version of Windows. Can you share more details on where you got this version?

Sorry, I did not see your reply in time. The Windows system I installed is a lite version, so there is no Microsoft store, nor can I install msixbundle files.（Windows 10 21H1）
OS: Windows 10 21H1 Build 19043.928

I can't find any details on a "Lite" version of Windows. Can you share more details on where you got this version?

My sister installed my win10 system for me. Maybe she wanted to make my system cleaner, so some functions were deleted. I don't know much, sorry.

The OS you installed is an unofficial, illegally produced (doesn't matter if you've a legal activation or license) distribution Windows 10, so I would highly recommend you to clean install with the ISO image from Microsoft's website from https://www.microsoft.com/en-us/software-download/windows10.
Alternatively, you can use the "Add-AppxPackage" command to manually install the application onto your system.


generic operator int() etc.

I suspect that char is missing from the list of overloads. But more generally, there should be nothing stopping conversion to/from any integer. But for starters, __int128 is missing from the list of overloads and more generally, generic solutions FTW!

more generally, there should be nothing stopping conversion to/from any integer.

Hi @johnmcfarlane I'm really not sure if there is a standard rule governing this. Consider conversion of uintwide_t to built-in integer. These kinds of conversions are often narrowing casts. My subjective philosophy with UDTs has usually been to make narrowing casts explicit.
This seems like a subjective interpretation on my side, so I'm open either way. I believe Boost.Multiprecision might require explicit narrowing casts. I seem to recall some dialog with motivating reasons for/agains explicit narrowing casts. But I wonder if there is a rule governing this philosophy?

Re. explicit, I assume I'm replacing like for like here. Does the 'explicitness' of these functions change as a result of this PR?

I should add that my original comment did not refer to explicit, simply that conversation did be possible at all - something the master branch currently does not facilitate.

But I wonder if there is a rule governing this philosophy?

For completeness, I recommend the C++ Core Guidelines for guidance on matters like this, i.e. style and best practice. E.g. this rule.

For completeness, I recommend the C++ Core Guidelines for guidance on matters like this, i.e. style and best practice.

Indeed. Those are good guidelines. It's amazong how often I forget to adhere to some of them, or maybe didn't even have complete awareness of some of the newer ones such as those regarding move semantics correctness. Thanks for sharing.

Does the 'explicitness' of these functions change as a result of this PR?

I still need to check this. Sorry about the delay @johnmcfarlane.

Does the 'explicitness' of these functions change as a result of this PR?

I still need to check this. Sorry about the delay

Of course we can integrate this @johnmcfarlane. Sorry I missed the fact that the explicit keyword is still present in your template. I think you are aware that GCC's data type __int128 will not test positive for std::is_integral, but I assume you have a workaround for this if you rely on it in your code. I simply neglected with disregard cast to that particular type.
@johnmcfarlane I did, however, add a few more tests to CI, as I had noticed that I had earlier forgotten to run CI on gcc-clang-9. And I also enabled something in the spot tests.
So if you could merge my current master (after #138 gets cycled and merged to master), we will be good to go on merging or not needing any more the CNL branch --- however you see best.
Then i think we will have at least a draft working model for continuing with CNL integration.

Hi @johnmcfarlane there are actually 8 new checks in CI's YAML script and also set up in GitHub branch protection. Sorry about the late unannounced change. You probably need to merge wide-integer master here to move forward.

Thank you @johnmcfarlane I kind of lost track with all the activities going on.
Is wide-integer moving in a usable direction now?

@ckormanyos very much so, yes. I don't know if we were very close earlier in the year or whether you've made changes that really help but almost none of my tests are failing now after I substitute uintwide_t.
One last thing I'm looking at right now is this error. Which related to your earlier comment...

I think you are aware that GCC's data type __int128 will not test positive for std::is_integral, but I assume you have a workaround for this if you rely on it in your code.

It's not as simple as that. It varies depending on toolchain and on the conformance setting. If you build with -std=c++2a (or 17, 14 etc.) then you get strict conformance and __int128 tests negative. But if you build with -std=gnu++2a it passes. A much more thorough exploration can be found in this blog.
The error I linked to was in the only job where I enable strict conformance (via the CMAKE_CXX_EXTENSIONS CMake variable).
A fix probably involves writing two constructors wrapped in #if defined(__SIZEOF_INT128__). LMK if you'd like me to have a go at writing those.

BTW tests passing isn't necessarily the end of the story. I'm not sure how I want integration to happen. Given the relative scale of the two projects, dropping the one header directly into CNL and maintaining updates by hand might be the easiest thing. Have to think about it. Also there's the previous discussion on things like naming, error handling etc.. Let's revisit that once CI is passing.

don't know if we were very close earlier in the year or whether you've made changes that really help but almost none of my tests are failing now after I substitute uintwide_t

We really got on the right track in spring with your help, @johnmcfarlane. Further improvements throughout summer helped further (some directly motivated through your comments, other clients and my own progression).

Interoperate with built-in floating-point types (also mostly constexpr).
Increase constexpr correctness even further.
Fix several bugs pointed out by clients
Clean up the constructor landscape.
Activate and handle conversion warnings.
Establish C++11,14,17,20 portability on about 20 compilers from 8-bit micros to powerful 64-bit workstations, not only MSVC/GCC/clang, but also microcontroller compilers from 3 commercial suppliers on multiple target architectures.

These led to a much more mature integral type and brought this work pretty close to actually being portable.

A fix probably involves writing two constructors wrapped in #if defined(__SIZEOF_INT128__).

I have an existing preprocessor switch called #define WIDE_INTEGER_HAS_LIMB_TYPE_UINT64. This activates the potential instantiation of the 64-bit limb and thereby (indirectly) assumes the existence of __int128. The problem here is that my existing compiler switch is intended to be set manually and the client is expected to know up front if __int128 is there or not.
In Boost.Math/Multirpecision if believe there is a sequence of compiler queries that attempts to check at compile time for the presence of __int128.
I don#t know if we should expect the client to manually activate the use of __int128 or if we should attempt to detect it via preprocessor queries as you have done. At any rate, i don't want any preprocessor detection to mess up one of my existing preprocessor manually set options so we mith need to mix those logic parts however we end up going.

LMK if you'd like me to have a go at writing those.

Sure @johnmcfarlane. I agree with you and believe that think constructor-from/cast-to signed and unsigned versions of __int128 would go a long way on moving forward and at least reveal any more subtle problems that might lie beneath that first layer of support.
Feel free to make an int28 branch or similar and give it a try. Or would you like me to make a branch?

dropping the one header directly into CNL and maintaining updates by hand might be the easiest thing.

Definitely, @johnmcfarlane. johnmcfarlane/cnl will probably be better if it remains as dependency-free as possible --- certainly you'll want to be independent of a dependency on ckormanyos/wide-integer.
If anything big happens on either your side or mine, we can synchronize up any time.

I'm also really pleased with how it's going. I wouldn't be surprised to find a few more issues because I never got as far as really putting wide types through their paces and will lean more heavily on wide-integer moving forward.

In wide-integer, I do not know if we should expect the client to manually activate the use of __int128 or if we should attempt to detect it via preprocessor queries as you have done.

I support both those things via CNL_USE_INT128. My chosen approach can be viewed here:

If CNL_USE_INT128 is not defined, CNL tries to use 128-bit if available.
If CNL_USE_INT128 is defined as 0, CNL doesn't use 128-bit.
If CNL_USE_INT128 is defined as 1, CNL does use 128-bit. (Hard error if it cannot.)

I should probably make more effort to separate use of 128-bit in wide-integer from 128-bit conversions. And I think wide-integer should probably do the same, hence I recommend wrapping uintwide_t conversions in __SIZEOF_INT128__, and not WIDE_INTEGER_HAS_LIMB_TYPE_UINT64.

Feel free to make an int28 branch or similar and give it a try. Or would you like me to make a branch?

OK. I'll try and get to that this week.

Definitely, @johnmcfarlane. johnmcfarlane/cnl will probably be better if it remains as dependency-free as possible --- certainly you'll want to be independent of a dependency on ckormanyos/wide-integer.

Really, it's just a scale thing. I think a dependency has advantages: fixes from wide-integer can easily be applied to CNL and it is crystal clear to users what the relationship is between the two packages.

Then at that time of uniform compatibility, break off on self-managed copies with manual sync if needed.

I'm assuming manual syncs will be needed from time to time. For this reason, I'd support certain naming and API changes. I don't think they're essential but they have knock-on effects for the quality of CNL.

Agreed on all points @johnmcfarlane. Thank you.
Some of the issues mentioned are tracked at #91 and #139.


Haskell implementation of typeid

I would be glad if you can kindly include this Haskell implementation in the README.
There's no validation yet, but I will include them soon.

Hi @MMZK1526 , I'd be happy to.
Could you send me a PR in our opensource monorepo modifying the README. Once we merge the code there, it'll be automatically published to this repo.

Closing since we just merged the change into the opensource monorepo


Adds autoForce option to start()

This allows for any app to enable the option to route to the current
href. Currently, links to the current page are ignored. This feature has
been implemented in a BC manner. Existing code will still work as usual.
fixes #105

Tests pass, but Travis is failing.
npm ERR! Linux 4.8.12-040812-generic
npm ERR! argv "/home/travis/.nvm/versions/node/v7.4.0/bin/node" "/home/travis/.nvm/versions/node/v7.4.0/bin/npm" "run" "karma"
npm ERR! node v7.4.0
npm ERR! npm  v4.0.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>karma: `karma start test/karma.conf.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>karma script 'karma start test/karma.conf.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the riot-route package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     karma start test/karma.conf.js

thank you for your PR. I have just released the new Route.js version so I will not accept new PRs on the old version unless they are security patches


Official wallet 1.7.0 offers are not supported

Offers created with Chia 1.7.0 do not show up in the frontend. This seems to be caused by the new offer format based on the web console errors:
Error: The data is compressed with version 6 and cannot be parsed. Update software and try again.
    NuxtJS 8
entry.ab974e08.js:4159:89230
    NuxtJS 6

To reproduce, check USDS offers in the Webapp.

Thank you for your report, the new offers are now supported!


Drive home

The agent will now drive home when tagged and ignore any input from any controller. Removed the redundant drive home code in the base agents.

@ptcrowley I added the fix. I figured I would add it in the _generate_agents_start since that's where they manipulate all the player statuses. Lmk what you think!

Looks good! Thanks Patrick!


[$500] Web- Emoji search does not work when searching emoji using letters not from start of word

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

Action Performed:

Open the app
Open any report
Click on emoji to open emoji picker
Search for 'water', search should include 'watermelon' emoji
Erase the search term and search for 'melon', now result does not have 'watermelon' emoji
Similarly search for 'nut' and observe that app does not display 'doughnut'
Try similarly on slack and slack displays the results right

Expected Result:
App should display emojis in result if we search by terms of emoji which are part of emoji text but emoji text does not begin with that term
Actual Result:
App does not display emojis in result if we search by terms of emoji which are part of emoji text but emoji text does not begin with that term
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/07e5511c-c99f-41db-a51a-fb88b6df2cb4
https://github.com/Expensify/App/assets/93399543/3a1446e1-afc2-4180-ac0c-cb1a4188fc91
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695224513815769
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018f1ac8e63f683a64
Upwork Job ID:<PHONE_NUMBER>225764352
Last Price Increase: 2023-09-22

also https://github.com/leeoniya/uFuzzy

This is not a bug but a feature request. Based on https://github.com/Expensify/App/issues/12189 I think we should just close this one. cc @roryabraham @stitesExpensify

This is not a bug but a feature request.

i find this reponse to be pretty comical. this is absolutely a bug unless users are expected to know what every one of 2500 emojis starts with, which is ridiculous on the face of it. for "melon" not to return "watermelon" is the definition of broken design, even if it isnt strictly a bug in the actual code.

I agree with @ZhenjaHorbach and @s77rt  that this doesn't meet the definition of a bug.
There are many ways in which emoji search could be improved, no doubt. I think everyone agrees that some form of fuzzy-search would be better than what we have today. But we live in a world of trade-offs, the biggest trade-off being where to focus our time. We have an effectively infinite amount of work (2200 open issues in the repo) and right now things that are actually broken, performance, and the existing product roadmap are our priority.
There will come a time when this is worth doing, but I don't think now is that time. I'm going to go ahead and close this issue, but feel free to keep the discussion going if you disagree.

But we live in a world of trade-offs, the biggest trade-off being where to focus our time.

that's a totally reasonable justification for not working on it. moreso than saying this is a feature request.


Add method in fields for set closure that prepare value for field

Example:
$field->prepareValue(
    static fn(mixed $value) => return $value . '-suffix',
);

Isn't that a hydrator job?


Karpenter fails to provision any new nodes when a PVC is missing

Version

Karpenter: v0.9.1

Kubernetes: v1.21
Expected Behavior
Karpenter should provision new nodes as expected.
Actual Behavior
If a PVC is missing/deleted from a deployment Karpenter fails to provision any new nodes from any of the
applied provisioners.
Steps to Reproduce the Problem
Delete/Remove a PVC from a deployment.
Resource Specs and Logs
The Karpenter controller logs show the following error message:
Provisioning failed, getting volume topology requirements, getting persistent volume claim my-vol, PersistentVolumeClaim “my-“vol not found	{"commit": "a7e26e6"}

Hey @centr0nix ,
Thanks for reporting this issue.  I'm going to work on reproducing this problem today.  I'll give an update once I have more info.

Hello again.  I believe I was able to reproduce this.
To clarify the issue, if a PVC is missing/deleted for a single deployment or pod, Karpenter will fail to provision nodes for all future pods, even those which aren't dependent on the missing PVC.

Hello again. I believe I was able to reproduce this.
To clarify the issue, if a PVC is missing/deleted for a single deployment or pod, Karpenter will fail to provision nodes for all future pods, even those which aren't dependent on the missing PVC.

Hello! That is correct.

OK great.  This certainly is not intended.  I'll be working towards a fix for this shortly.


NPC - Trader [Question]

I would like to be able to make trader NPCs to only trade with people who have a certain flag, such as the Warrior guild trader will only work with those who are in the warrior guild and has the appropriate flag on their character, i am currently using the Cititrader plugin which i know is not updated in quite some time but it has been serving its purpose for me so far.
So I was just wondering is that possible? Also which trader plugin should I use, is there any that work well with Denizen and offer the same or similar GUI that Cititrader uses?

We generally recommend people build their own shop code or use an existing Denizen-driven one... half because that's what we always do with every plugin question, and half because good trader plugins aren't commonly available.
Anyway, you can get it working like that with any system... just cancel the NPC click event if they don't have the flag, in theory. A well made plugin will accept this and not do the trade.

Thanks for the reply, yes I watched your youtube video with the inventory GUI and saw the potential to make a pretty swell shop using that in a script. I might script something like that eventually, I was just curious what was possible with the other plugins, again thanks


queue in cpp

Queue.cpp
this program simulates queue data structure in c++
Closes https://github.com/fnplus/interview-techdev-guide/issues/192

@sefirakarina : Thanks for Contributing


[New Feature]: Would u like to add favicon image to your website

🎮 feature Request
feature logic and basic description
Wanted to add the logo image to the favicon of the website.
Point down the features
feature points

instead of chrome symbol and image will be added.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Anjaliavv51 Please raise any other issue, because someone is already working on it...


In 580a

my local created a bunch of local configs when i imported the latest development branch. the only commits i care about in this PR are the last 3. IF the localhost configs give you pause, feel free to remove them.


test: upgrade tests for 8488 smartWallet with promiseWatchers

Inserted walletFactory as a:
built submission files in #8488
enhanced vat-status.js to look up vats by name (as substring)

this is included in an upgrade now


Dynamic Ingestion of a Website

https://issues.redhat.com/browse/REDSAIA-107

If you can fix the conflicts Sukanta I will get this guy merged in.

Need write access to this repo to merge the PR.

Gave you member access to the org.  But the changes are on your personal fork. So you need to pull the upstream to fix the conflicts.


Task Cancelled when calling GetUserTokenAsync using Emulator

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
4.2.1
Describe the bug
I expect to retrieve a mock user access token when calling GetUserTokenAsync, using the emulator.
I actually get an exception with message:
{System.IO.IOException: Unable to read data from the transport connection: The I/O operation has been aborted because of either a thread exit or an application request. ---> System.Net.Sockets.SocketException: The I/O operation has been aborted because of either a thread exit or an application request
--- End of inner exception stack trace ---
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error)
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.GetResult(Int16 token)
at System.Net.Http.HttpConnection.SendAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)}
To Reproduce
Steps to reproduce the behavior:

Open Bot Framework Emulator 4.2.1
Call GetUserTokenAsync()
Wait around 30 seconds
See error

Expected behavior
I expected a mock user token to be retrieved
[bug]

For the record, this is the behavior in .NET that is caused by #1212. So you can probably link this issue to that one and confirm it's gone once that bug is fixed.

This was caused by #1212  which was fixed and related to #1268 which was also fixed after a lengthy investigation. Closing this.


The sidebar will be cut into 2 parts when using fixed footer

gentelella version: 1.3.0
Broswer: Chrome 51.0.2704
Resolution: 1920*1080
Reproduce: https://colorlib.com/polygon/gentelella/fixed_footer.html

Here is fix https://github.com/puikinsh/gentelella/pull/234


Ambilight don't work and Theater don't undimm in Android TV (Nvidia Shield)

So when i force the hue_ip to avoid #37 the ambilight mode don't work.
When a playback starts the lights turn red and/or blue and never changes again.
The theater mode starts working, but when the playback pauses/stop don't undimm to previous state.
This is the onlye error i get:
> 11:52:41.204 T:138410161232   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.AttributeError'>
                                            Error Contents: 'MyPlayer' object has no attribute 'playlistpos'
                                            Traceback (most recent call last):
                                              File "/storage/emulated/0/Android/data/org.xbmc.kodi/files/.kodi/addons/script.kodi.hue.ambilight/default.py", line 138, in onPlayBackEnded
                                                if self.playlistpos < self.playlistlen-1:
                                            AttributeError: 'MyPlayer' object has no attribute 'playlistpos'
                                            -->End of Python script error report<--

I try with SPMC (and branch v16), Kody17.0, Kody17.1 RC1 and always the same...

Ok!
After a lot of web searching and tests i found a workaraound for the addon in a Nvidia Shield TV device (and i supposse every android tv device...)
Go to System->Settings->Video
Be sure "Expert" mode is enable
Uncheck MediaCodec(surface) hardware acceleration
Now the addon works again :)

Cool, seems like similar issue to the previous raspberry pi (hw video processing).

That fix worked for me too.

ok. Closed!

I can't seem to find this option in my Kodi. I'm runing on Nvidia switch as well.
Have they moved the option or am I just not seeing it?

Maybe your skin change the location?
Search in the video or playback settings... With the expert mode enable...

Thank you for the reply.
I found it, I was looking in the wrong location
It was under the player section.
Working well now


rofi-calc: 1.6 -> 1.7

Motivation for this change
Update rofi-calc!
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 87290 1

1 package built:
- rofi-calc


Investigate CF Pages

Maybe Cloudflare Pages is a better alternative to GitHub Pages? It'd get rid of the 404 status code.
https://github.com/mikaeldui/masteries-quest/tree/cf-pages

Initializing build environment
2m 30s


[BUG] Missing pkg_resources for Python >= 3.12

Describe the bug
syncall/google/gtasks_side.py imports pkg_resources which is part of setuptools. This pkg_resources deprecated and will not be installed by default in virtual environments for Python >= 3.12 (see https://github.com/python/cpython/issues/95299). A workaround might be to run pip install setuptools in the virtual environment but this is not clear from the given error. A solution might be to configure a dependency to setuptools for the package.
To Reproduce
Upgrade python to 3.12.
Create venv with python -m venv .venv.
Activate venv with source .venv/bin/activate
pip install syncall[google,tw]
tw_gtasks_sync --help
2024-05-04 12:07:53.418 | ERROR    | syncall.app_utils:inform_about_app_extras:242 -  You have to install the google,tw extras for tw_gtasks_sync to work. With pip, you can do it with something like: "pip3 install syncall[google,tw]" Exiting.
Versions
OS Version (cat /etc/*-release; uname -a)
DISTRIB_ID="Arch"
DISTRIB_RELEASE="rolling"
DISTRIB_DESCRIPTION="Arch Linux"
NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo
Linux archlinux 6.8.9-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 02 May 2024 17:49:46 +0000 x86_64 GNU/Linux
Python version (python --version; python3 --version)
Python 3.12.3
Python 3.12.3
--version of your executable
1.8.5

Please have a look at the latest state of master and if it still exists, let's re-open this!
Thanks for the bug report!


docs:clean repetitive words

Purpose or design rationale of this PR
use https://github.com/Abirdcfly/dupword check duplicate word globally
related issue need to discuss further https://github.com/scroll-tech/scroll/issues/1308
PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[ ] build: Changes that affect the build system or external dependencies (example scopes: yarn, eslint, typescript)
[ ] ci: Changes to our CI configuration files and scripts (example scopes: vercel, github, cypress)
[ ] docs: Documentation-only changes
[ ] feat: A new feature
[ ] fix: A bug fix
[ ] perf: A code change that improves performance
[ ] refactor: A code change that doesn't fix a bug, or add a feature, or improves performance
[ ] style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
[ ] test: Adding missing tests or correcting existing tests

Deployment tag versioning
Has tag in common/version.go been updated or have you added bump-version label to this PR?

[ ] No, this PR doesn't involve a new deployment, git tag, docker image tag
[ ] Yes

Breaking change label
Does this PR have the breaking-change label?

[ ] No, this PR is not a breaking change
[ ] Yes

Feel free to remove the duplicated docker in this PR.

Feel free to remove the duplicated docker in this PR.

done


[ISSUE #9021] Correct the error message of acl command

Fixes #9021 .

Codecov Report
Attention: Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.

Project coverage is 47.70%. Comparing base (f4c4984) to head (2b623ea).

Files with missing lines
Patch %
Lines

...ocketmq/broker/processor/AdminBrokerProcessor.java
0.00%
24 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9022      +/-   ##
=============================================
- Coverage      47.77%   47.70%   -0.08%
+ Complexity     11820    11802      -18
=============================================
  Files           1305     1305
  Lines          91385    91388       +3
  Branches       11736    11735       -1
=============================================
- Hits           43661    43596      -65
- Misses         42333    42397      +64
- Partials        5391     5395       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Sheathe function (Name issue, stamina consumption)

Not technical issues, but I think the name itself "sheathe" isn't quite cohesive, considering what it actually does. Sheathe alludes to the use of weapons, and you "sheathe" even items which are non-weapons, which is confusing. I think a more sensible name would be something like "Ready Hand".
I would also like to discuss the fact that the "sheathe" function consumes stamina. It doesn't make sense, even for balancing purposes, I don't think it should consume stamina.
Now, I think maybe I know why it would consume stamina, as it shows a visual cue for sheathing, but I don't think it should be necessarily done through the stamina function, because as I said, it doesn't make sense that readying a weapon/item consumes stamina. It's immersion breaking, if you will. There being some delay when sheathing may be enough for balancing/immersion purposes.
Love the idea of your mods, these are suggestions for gameplay cohesiveness/ quality of life/ immersion !! I think these mods have the potential to become MUSTS for people to download.

I designed the hand slots to only allow specific items (defined by an item tag and supposed to be only weapons) in them, which is why I called it "sheathe". Non weapon items are supposed to be in the hot bar slots, which can be accessed when the main hand is sheathed. But feedback on that is always welcome ^^
The stamina cost for sheathing, swapping and toggling 2-handed is supposed to make those actions matter in stamina draining situations like combat. I like the idea that players have to manage their stamina and restricting sheathing your weapon to get access to eg a health potion is part of that.
But I also realize that not all players like this gameplay, so I made the stamina requirements/costs configurable. Even 'Stamina Attributes' itself is optional :)
The feedback for those actions is important, I agree. Triggering a sound event is on the Todo list. I thought about ways to introduce a delay, but haven't come to a satisfying result yet.
Thank you very much for your feedback! It is very much appreciated!

Understood! The sound event is genius, that would be perfect for that purpose.

Understood! The sound event is genius, that would be perfect for that purpose.

Sound does make everything better :^)
Also, maybe there can be a unsheathing speed bonus/penalty for certain weapons (separate from attack speed), like for katanas having higher unsheathing speed to quickly strike an approaching opponent, or a cumbersome weapon having slower unsheathing speed.

I'll make adding the sound a higher priority^^
I have not found a good implementation idea for the sheathing/swapping delay.
One idea I had was to apply a status effect with a variable duration on sheathing/swapping, which would prevent attacking
Just fyi, I will try to keep compatibility with other mods as good as possible, which is why I want to avoid messing with the basic item use cooldown.
Suggestions about implementation details are always welcome! :)

Something I forgot to mention is, that all in game texts are localized, so "Sheathe" can be changed using a resource pack.


Revoked signing certificate on MSI download

I just downloaded the CLI from the download link here and I'm seeing a message that the signing cert has been revoked.

What's the date on your machine/

2/28/2018

Okay thanks. I have no immediate resolution here.
What area the valid from and valid to dates on the cert.?

All dates are valid. I have a coworker who was able to download and install. I've tried several times and get the same error message. This issue can be closed with the assumption that it's a problem with my machine.


Add tests for sf::CircleShape

Description
Some more Graphics module unit tests :)
I had to increase the precision of the sf::Vector2<T> printer to discern the very small difference between floating point numbers in testing.
Tasks

[x] Tested on Linux
[x] Tested on Windows
[x] Tested on macOS
[x] Tested on iOS
[x] Tested on Android

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

Fixed merge conflicts and rebased.

SFMLBuildMaster: Build this please.

Rebased after #2055 was merged just to be sure that there we no new conflicts.


[Feature] Nested menu support

What about supporting nested menus with a variable crawler ?
This would auto generate up to 3 levels of submenus (rather sensible BootStrap limitation)

Maybe you could write a plugin.

I'm going to close this issue.


ARM64 support consider?

Hi there, your application runs well on my ARM Windows11 PC, anyway, the performance lost caused by the emulation still makes it a little unpleasant. I wonder whether it is possible to release a version for  Windows On Arm64?

I don't have ARM64 machine so I won't know if the code will work or not. However, you can feel free to try to build this and if it works you can let me know and I will try to include it in the next store experimental release.


GitHub Actions caching

Let's see how this works!

Closing this as it was just a test 🙂 The whole point of daily rebuilds is to use the latest gitpod:workspace-full image, so caching would ruin that


Slow lookup

I have only one lonely entity in datastore and the lookup using it's key is very slow: 0.33-0.42 sec.
This is what curl_getinfo shows (I placed it in GuzzleHttp\Handler\CurlHandler):
Array
(
    [url] => https://datastore.googleapis.com/v1/projects/***:lookup
    [content_type] => application/json; charset=UTF-8
    [http_code] => 200
    [header_size] => 393
    [request_size] => 454
    [filetime] => -1
    [ssl_verify_result] => 0
    [redirect_count] => 0
    [total_time] => 0.416758
    [namelookup_time] => 0.060624
    [connect_time] => 0.095299
    [pretransfer_time] => 0.302201
    [size_upload] => 96
    [size_download] => 584
    [speed_download] => 1401
    [speed_upload] => 230
    [download_content_length] => -1
    [upload_content_length] => 96
    [starttransfer_time] => 0.416411
    [redirect_time] => 0
    [redirect_url] =>
    [primary_ip] => <IP_ADDRESS>
    [certinfo] => Array
        (
        )

    [primary_port] => 443
    [local_ip] => <IP_ADDRESS>
    [local_port] => 36792
)

Is it really so slow? I tried different server locations to request from and it doesn't have any visible influence on request speed.
Even using compute engine VM instance the speed is extremely slow:
Array
(
    [url] => https://datastore.googleapis.com/v1/projects/***:lookup
    [content_type] => application/json; charset=UTF-8
    [http_code] => 200
    [header_size] => 345
    [request_size] => 480
    [filetime] => -1
    [ssl_verify_result] => 0
    [redirect_count] => 0
    [total_time] => 0.457879
    [namelookup_time] => 0.00464
    [connect_time] => 0.009902
    [pretransfer_time] => 0.025045
    [size_upload] => 103
    [size_download] => 591
    [speed_download] => 1290
    [speed_upload] => 224
    [download_content_length] => -1
    [upload_content_length] => 103
    [starttransfer_time] => 0.457698
    [redirect_time] => 0
    [redirect_url] =>
    [primary_ip] => <IP_ADDRESS>
    [certinfo] => Array
        (
        )

    [primary_port] => 443
    [local_ip] => 10.*.*.*
    [local_port] => 46776
)

I am seeing the same slowness: from Container Engine (K8s) to Google Cloud Datastore I consistently see simply get operations (by key) taking 400-500ms.

@thijsterlouw Yes, I'm using transient connections but it doesn't matter since I'm measuring response time without pretransfer time. The main problem as I think is that Google uses US IP addresses in DNS for it's API endpoints and when the storage located in Europe region the requests from Russia go to Western Coast of USA, than Google internally requests data from european datacenter and sends response back through US.
It's extremely strange that there are no local endpoints for each region. For instance, europe-west1.datastore.googleapis.com and not just datastore.googleapis.com.

Hi Lisio, that would be extremely inefficient. I was under the impression that GLB (I assume that is used here as well) is smarter: it should resolve to a nearby Google datacenter and from there enter the Google network. Unfortunately I did not see any way to see what my current Cloud Datastore location is (eg if I did not make a mistake and pick US by accident, instead of EU)

This seems like an issue with the datastore product rather than the code in this repo specifiically so I'm closing this. If you have a support contract it might help to open a ticket there.


docs: Update trigger phrases for CI

Document recent changes to trigger phrases for CI.

Coverage decreased (-0.02%) to 45.483% when pulling d64921bf31aa4dada79df4148d8917f669359d5a on pr/pchaigno/update-ci-doc into 6caa55c094346553fdcecfdd3624969cec0ca446 on master.

test-me-please


[Minor] Advanced tokens for resultset table component

New advanced tokens in resultset table component.

Should we add a border-radius property to change the applied to the table header?

If we add border-radius it is added to the Table component and not to the ResultSet Table.


Massive Speed Up

Lookups in just_gtfs are done on std::vectors with std::find. This is slow if you do a lot of looking up things by id which, many workflows could forseeably do.
These same lookups also make copies of the structs they look up, presumably so they can signal that they have not found a certain thing by its id using std::optional (which cannot hold a const reference).
This pr changes that by:

sorting any feed that is loaded by the id used to look them up (and tie breakers after that)
this allows the look ups to be done in O(log n) time instead of O(n) with 0 added space (O(1) look up could be had but comes at the cost of a bunch of memory)
for those functions which return a range we now return an iterator range
for those functions which optionally sorted the output we now give no choice, output is always sorted
sorting does take some time but its is tiny
we no longer make copies of things we look up instead returning a const reference
for things we do not find, we replace the "not found" semantic with a default constructed static struct of the relevant type. this struct can be checked for validity using gtfs::valid which makes use of the equality operator to check for equality with a default constructed object

these changes made huge improvements in performance cutting the time to process even a small feed from 6 minutes down to 80 seconds, about an 80% reduction.
see https://github.com/valhalla/valhalla/pull/4167 for more info

We're about to merge https://github.com/valhalla/valhalla/pull/4167 @mesozoic-drones . We're also fine running on our fork, or even shift development to that fork, or just PR'ing here. Let us know what your preference is.


protobuf: pin to 3.17 for Python 2

Motivation for this change
ZHF: #144627
@jonringer, @tomberek , @nrdxp
Protobuf 3.18 and later are no longer compatible with Python 2, so pin the last working version in the Python 2 specific packages. Fixes build of ddar.
Things done

Built on platform(s)

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

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

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

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 145706 run on x86_64-linux 1

1 package built:

ddar

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 145706 run on x86_64-darwin 1

1 package built:

ddar


invalid syntax / async def Feed(self)

Hi,
Hope you are doing all well !
I have the following error while executing a script:
import twint

tweets = []

c = twint.Config()

c.Username = 'noneprivacy'
c.Limit = 20
c.Store_object = True
c.Store_object_tweets_list = tweets

twint.run.Search(c)

Traceback (most recent call last):
  File "simple.py", line 1, in <module>
    import twint
  File "/Users/xxxxxxxx/src/go/src/github.com/x0rzkov/twint/twint/__init__.py", line 14, in <module>
    from . import run
  File "/Users/xxxxxxxx/src/go/src/github.com/x0rzkov/twint/twint/run.py", line 45
    async def Feed(self):
            ^
SyntaxError: invalid syntax

Any idea what it could be ?
Cheers,
X

Python version?

my bad, I use python2 :-(


Review synchronous logic resets for all DFF / registers inside the RSA unit

Update source code to have a "clean" reset for all registers. Don't add combinatorial logic within design for async reset.
Async resets shall come only from top rst_n that propagate to all registers (without combinatorial logic) Clear (load '0) for registers shall be implemented as synchronous commands when needed.

Clean version of source code. Debug on going.


The request for locale.properties of pdf.js returns 404

Link to PDF file (or attach file here):
Configuration:

Web browser and its version:Firefox 49.0.1
Operating system and its version:win7
PDF.js version:1.5.188
Is an extension:

Steps to reproduce the problem:
I'm trying to set up PDF.JS in an SpringMVC application for displaying PDF-files. Therefore I want also to use the pdf-viewer that is shipped with PDF.JS.
But I'm having problems with the "locale.properties" file. The error returns an 404 for that file.
My web server is tomcat7.0.
I'm trying to add mime-type（application/octet-stream） to web.xml and still return 404.

As mentioned in both the ISSUE_TEMPLATE.md and CONTRIBUTING.md, please provide access to a working example since it's impossible to tell what might be wrong from the information given.
Closing as incomplete for now; please provide a link to a complete example in order to re-open.


[CONTESTING] Fix some issues in contesting

Got some issue reports from a user and try to fix some stuff

The following commit handles a more tricky case.
if a user starts a new session with contest "A"
logs a QSO with contest "A" and switches then the contestname to contest "B"
the whole session will be broken. Reason for this is, that the contest_id in the contest_session get's updated, but not the qso data which is necessary to get the correct QSO list. As we don't want to edit already logged QSO's (the first logged QSO was logged with contest "A") a change of the contest during logging should be handled like a new session. Now it would be really bad to directly and with no further information just reset the session.
The more safe way is to just disable the contestname select during after logging the first QSO. If you need to change the contest you have to start a new session. The wrongly logged QSO in the beginning is in the database but not part of the session anymore.

I think its ready for a first test.


Feature/implement sort options in property mappings list

What was done?
Define structure of sorting spine properties
Implement the different alignment sorting criteria

On the "Synthetic Spine Property Mapping By Category" screen, looking at the issue #71, implement the different criteria for sorting the alignments to a spine term.

Ensure every property has its selectable domains collection

Since different organizations specifies the available domain list for a property differently, we take care of ensuring that, in the end, we have a collection of string values to show the user in the edit term screen.

Complete the spine sort options implementation
Options to sort spine properties available:

"Overall Alignment Score"
"Organization"
"Spine Class/Type"
"Spine Property"
"Total Identical Alignments"

Complete the alignment sort options implementation
Options to sort alignments available:

"Organization"
"Class/Type"
"Alignment Score"
"Property"
"Has Alignment Issues"

Options still unavailable for sorting spine terms (awaiting requirement clarifications)

"Has Alignment Issues"
"Source Data Order"

I thought I had resolved the two open matters: (1) "Has Alignment Issues" on spine; and (2) "Source Data Order" on alignment sort.  The "Source Data Order" is irrelevant here since it relates to the ordering in the old spreadsheets. The "Has Alignment Issues" on spine sort puts those properties with issues at the top of the list.


disable RocksDB indexing for some secondary index operations

Please note that for legal reasons we require you to sign the
Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2151/


Added FF2024

Thanks for this PR. Unfortunately, the formatting of the addition is not ideal and I'm afraid will not lead to a nice result. I think the following changes are needed. Would you be willing to make these changes? Alternatively, we can merge the PR as-is and then I can edit the result – tell me what you prefer.
Changes:

Formatting Markdown in the following way,

Something: blah blah blah
Something else: blah blah blah

doesn't lead to separate lines with headings, as might be expected. Instead, the resulting formatted text is run together:

It looks like the intention was to have separate lines for the items like the date and CFP, and actually, it is possible to get that kind of formatting using a certain Markdown syntax, but I want to suggest in this case it's preferable simply to write the information as prose. So, in other words, make it a single short paragraph, not a database record.

I think it would be better not to make entries like this into subsections, but rather as list items. Here I'm following the example of most other "awesome lists" (e.g., this one or this one). The required change is simply to not use ###, and instead make it a bulleted item.

Update: I added past FF events to show the suggested formatting.

Thanks for your comments! I've simplified the entry - how does it look now?

Thanks for working on it. It's still not quite right, but I'll just take the PR and fix up the details.


Highchart - clear series data

Hi - I have a use case where the user selects one or more items from a dropdown, clicks a button, and data is plotted in highcharts based on their selections. The user can continue to add and change items to plot... so the chart needs to update based on their selections.
The place I'm stuck is clearing out the existing series data when updating the chart. Seems like no matter what I try, existing data is still retained. There's some explanation in a highcharts post here. But couldn't get a "remove" command to work (which was the recommendation in that post).
In the test below, the chart is supposed to update from showing 3 brands to just show 1 brand; however, I haven't found a way to remove the data for the existing 3 brands.
import justpy as jp

my_chart_def = """
{
        chart: {
            type: 'spline'
        },
        title: {
            text: 'Trending'
        },
        xAxis: {
            categories: ['Jan 2020', 'Feb 2020', 'Mar 2020', 'Apr 2020']
        },
        yAxis: {
            title: {
                text: 'Metric'
            }
        },
        series: [{
            name: 'Facebook',
            data: [550, 520, 480, 450]
        }, {
            name: 'Google',
            data: [780, 750, 751, 760]
        }, {
            name: 'Oracle',
            data: [980, 950, 951, 960]
        }
        ],
        credits: {
            enabled: false
        },
}
"""

def chart_test():
    wp = jp.QuasarPage()
    q_btn_trend = jp.QButton(a=wp, color="primary", icon="trending_up", label="Update Chart")
    trend_chart = jp.HighCharts(a=wp, classes='m-2 p-2 border', style='width: 600px')
    trend_chart.options = my_chart_def

    # replace the chart series data with new data
    def update_trend_chart(self, msg):

        # tried several ways to remove the existing series data... but no luck :(
        trend_chart.options.series.pop(0)
        trend_chart.options.series.pop(0)
        trend_chart.options.series.pop(0)

        # add some new series data
        brand = {}
        brand['name'] = "Apple"
        brand['data'] = [480, 450, 741, 740]
        trend_chart.options.series.append(brand)

        jp.run_task(wp.update())

    q_btn_trend.on('click', update_trend_chart)
    return wp

jp.justpy(chart_test)

Please add the line:
trend_chart.update_create = True

as the fifth line in chart_test and it will solve the problem. This tells the component to create a new chart on update instead of just updating the old chart which left the old series in.
I will change this in the upcoming version so this is not required by making the update one to one and deleting series that are not specified.

works great... thanks!


Upgrade test failure

  * service[etcd-service] action restart
    ESC[0m
    ================================================================================ESC[0m
    ESC[31mError executing action `restart` on resource 'service[etcd-service]'ESC[0m
    ================================================================================ESC[0m

ESC[0m    Mixlib::ShellOut::ShellCommandFailedESC[0m
    ------------------------------------ESC[0m
    Expected process to exit with [0], but received '1'
ESC[0m    ---- Begin output of /usr/bin/systemctl --system restart etcd ----
ESC[0m    STDOUT:
ESC[0m    STDERR: Job for etcd.service failed because a timeout was exceeded. See "systemctl status etcd.service" and "journalctl -xe" for details.
ESC[0m    ---- End output of /usr/bin/systemctl --system restart etcd ----
ESC[0m    Ran /usr/bin/systemctl --system restart etcd returned 1ESC[0m

ESC[0m    Resource Declaration:ESC[0m
    ---------------------ESC[0m
    # In /var/chef/cache/cookbooks/cookbook-openshift3/recipes/services.rb
ESC[0m
ESC[0m     64:   service 'etcd-service' do
ESC[0m     65:     service_name 'etcd'
ESC[0m     66:     action :nothing
ESC[0m     67:   end
ESC[0m     68: end
ESC[0m
ESC[0m    Compiled Resource:ESC[0m
    ------------------ESC[0m

on master1, chef on server, separate etcd:
imiell@cage:/var/jenkins/workspace/cookbook-openshift3-upgrades/cookbook-openshift3-test-11/shutit-openshift-cluster/vagrant_run/shutit_openshift_cluster_l3aNxF  ⑂ master  $  vagrant status
Current machine states:

chefserver                running (virtualbox)
chefwkstn                 running (virtualbox)
master1                   running (virtualbox)
master2                   running (virtualbox)
etcd1                     running (virtualbox)
etcd2                     running (virtualbox)
etcd3                     running (virtualbox)
node1                     running (virtualbox)
node2                     running (virtualbox)

This environment represents multiple VMs. The VMs are all listed
above with their current state. For more information about a specific
VM, run `vagrant status NAME`.
imiell@cage:/var/jenkins/workspace/

upgrading from 1.3 => 1.4

role.json:
{
  "chef_type": "role",
  "default_attributes": {},
  "description": "Base role for test",
  "env_run_lists": {},
  "json_class": "Chef::Role",
  "name": "role",
  "override_attributes": {
    "cookbook-openshift3": {

      "deploy_containerized": false,

      "deploy_dnsmasq" : true,
      "openshift_HA": true,
      "openshift_builddefaults_env": [
        {
          "NO_PROXY": "master1.vagrant.test"
        }
      ],
      "openshift_cluster_duty_discovery_id": "testcluster1",
      "openshift_cluster_name": "master1.vagrant.test",
      "openshift_common_sdn_network_plugin_name": "redhat/openshift-ovs-multitenant",
      "openshift_deployment_type": "origin",
      "ose_major_version": "1.3",

      "ose_version": "1.3.3-1.el7",

      "openshift_hosted_managed_router": false,
      "openshift_hosted_managed_registry": false
    }
  },
  "run_list": [
   "recipe[cookbook-openshift3::default]",
   "recipe[cookbook-openshift3::common]",
   "recipe[cookbook-openshift3::master]",
   "recipe[cookbook-openshift3::node]"
  ]
}

Can you also update your run list for the tests please?
Should be:
"run_list": [
"recipe[cookbook-openshift3::default]"
]
The logic of dealing nodes masters or etcd components is in one place now. Therefore do not need to add specific roles such as master or node...

OK. Will update and re-run all tests.


Error: Domain awesomegithubprofile.tech expired on 12/09/2021

Describe the Error
Visiting https://awesomegithubprofile.tech/ resolves to a page that says This domain name expired on 2021-12-09 23:59:59. No content is shown.
Expected behavior
Visiting https://awesomegithubprofile.tech/ should bring users to the intended site with the relevant content.
Additional context
Cool project!

It's still available over https://abhisheknaiidu.github.io/awesome-github-profile-readme/ @abhisheknaiidu, it would be great if you could update the link in the repo info.

@alexdelorenzo @Just-Moh-it thanks for raising the issue, i've added the url now. 😄


ZXingScannerView IsScanning does not behave as expected

The IsScanning property on the ZXingScannerView does not behave as expected.
After experiencing issues where we could not scan more than once after toggling both IsScanning and IsAnalyzing, i took a quick look in the rendering code and discovered that the only difference between setting them to true is that IsScanning also sets the ScanningOptions, and the BarcodeFound handler, then they both call _cameraAnalyzer.ResumeAnalysis();
However when you toggle them both to false, the behaviour is a lot different:
IsAnalyzing --> this simply calls _cameraAnalyzer.PauseAnalysis();
IsScanning --> this calls _cameraAnalyzer.ShutdownCamera();
I would expect IsScanning to only delete the scanningOptions and remove the handler when toggled to false, or to redo the camera setup when toggled to true.

I am experiencing this issue from a slightly different perspective (and it only affects Android apps).
In the Xamarin.Forms sample, IsScanning is set true in the scanning page's OnAppearing method and set false in OnDisappearing, which seems reasonable.
If the sample app is modified such that a nested page is pushed onto the navigation stack, the scanning page's OnDisappearing will execute, causing camera capture to be suspended while the nested page is visible. When the nested page is popped from the stack, the scanning page's OnAppearing method will again execute, which should restart camera capture. However, since IsScanning is broken, camera capture never restarts, which means that it is not possible to navigate away from and return to a barcode scanning page without stalling the scanning operation.

After digging deeper into the code, it seems that the problem is a mismatch between the effects of the StartScanning() and StopScanning() methods of ZXing.Mobile.ZXingSurfaceView.
Since theses methods are triggered by toggling the setting of the ZXingScannerView.IsScanning property, I would expect them to be equal and opposite in their effects, but they are not.
StopScanning() fully stops and cleans up camera capture but StartScanning() simply restarts a previously initialized capture session, making the assumption that all the necessary setup has already been done. This assumption fails when the containing page disappears and then reappears, since all the camera setup is discarded when StopScanning() is triggered by the page's OnDisappearing() method.
I have worked around the problem by creating a custom renderer for ZXingScannerView, which in turn uses a custom implementation of ZXingSurfaceView, based on the original code, but with the following custom implementations of StartScanning() and StopScanning():
private Action<Result> _scanResultCallback;

public void StartScanning(Action<Result> scanResultCallback, MobileBarcodeScanningOptions options = null)
{
	_scanResultCallback = scanResultCallback;
	ScanningOptions = options ?? MobileBarcodeScanningOptions.Default;
	_cameraAnalyzer.BarcodeFound += OnScanResult;
	_cameraAnalyzer.SetupCamera();
	_cameraAnalyzer.ResumeAnalysis();
}

private void OnScanResult(object sender, Result result)
{
	_scanResultCallback?.Invoke(result);
}

public void StopScanning()
{
	_cameraAnalyzer.BarcodeFound -= OnScanResult;
	_cameraAnalyzer.ShutdownCamera();
}

I suspect the above problem is also the cause of Issue #712, although I have not tested this.

@aquisio can you share the full code of the custom renderer please?

This should behave better in the 3.0 series.  Please open a new issue if there are other problems with it on that version, thanks.

I have the same problem with version 3.0.0-beta5. @Redth
Do you plan to push an update for this soon?


[3.x] Optimize anim tree

Removes redundant lookups on HashMap.
Notes

Another small optimization to bottleneck, the hashmap is looked up twice for no need, and doesn't seem to be optimized out by compiler.

Thank you for not forgetting about version 3.x

Thanks!

Is this relevant for master as well?

master looks quite different to me, and it depends whether relevant code appears in profiles on master.
But in general the has() method of hash_map is better avoided in bottleneck code and replaced by get_ptr() with a null check, it doesn't seem a good paradigm to use from c++ as it seems to almost always do needless expensive work (maybe it exists for bound functions where checking NULL might be more difficult).

I second @JekSun97 opinion fully. I also would like to thank personally @lawnjelly @Calinou @akien-mga and other contributors for their due diligence on supporting 3.x and backporting 4.x features back to the 3.x branch.
FWIW I have a bit of a longer rant about this: in my humble but knowledgable opinion, nothing in the 4.x line can match the performance of 3.x with GLES2 + batching in 2D space, and in the mobile space in general, the latter being 50% of the game market today. I am not going to argue about this, but that's a factor any serious project must take into account. I have also noted that @clayjohn, willingly or not, was careful enough not to touch on any rendering performance figures for 4.x while presenting the future of 4.x rendering at GodotCon in Munich which I attended personally. :-) I understand we need to switch over to 4.x at some point in time, but there has to be some serious advance of 4.x about optimizing rendering performance on low-end mobile devices, backed up with numbers, to convince me that the time has come to do the transiton. Today, I am not positive this is the case for projects where 3.x + GLES2 is good enough feature-wise for a 2D game.


add empty _configure to DockerContainer

I've used DbContainer in the past as a parent to a class where I redefined _configure() and did not redefine start(). I'm now using DockerContainer and wanted to follow the same pattern but it doesn't have a _configure() function called in start(). I figure this gives the easy option without being invasive?
tests pass

ok, its in the java version - https://github.com/testcontainers/testcontainers-java/blob/18ea976a640e43e3fa1a1da7285c4630fd9cdbac/core/src/main/java/org/testcontainers/containers/GenericContainer.java#L701 - , i say why not


Don't email importer when importing

(Requested by @AlanRockefeller.)

Could I do this by using the API instead of create?


HIVE-24539: OrcInputFormat schema generation should respect column delimiter

What changes were proposed in this pull request?
OrcInputFormat  getDesiredRowTypeDescr method should use column delimiter to generate column names.
Why are the changes needed?
Current logic can create wrong names when column names contain commas.
Does this PR introduce any user-facing change?
no
How was this patch tested?
q file

Hey @abstractdog @maheshk114  can you please take a look?

Gentle ping @abstractdog

Gentle ping @abstractdog

Shall we push this change?

Shall we push this change?


Linting

checked my code for all occurrences of lines > 80 characters. 4 lines found. Reduced the 4 lines to <=80 characters per line rather than 120.

@fetorres don't open a new pull request to add code to your existing PR, it fragments the discussion. Simply push your changes to the branch you opened #12556 from, which would be master. Also, it's usually better to not use master for PRs.


Useless token consumption

Repeatedly consuming tokens in an error resulted in a large amount of tokens being wasted on the wrong build. This experience is very bad. I am a $100 subscription user and I am very frustrated
Furthermore, as I am constantly trying to build more projects, I am unable to distinguish which one is my most satisfactory project

I canceled my subscription today, I only get error messages. The same command in lovable.dev worked immediately. Bolt still need to work on it a bit until it's really mature.

Thanks @SchneiderSam - I am also just trying out lovable.dev , thanks for your valuable suggestion to save time.


[Blades in the Dark] Dark Mode compatibility adjustment

Submission Checklist
Required

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

[x] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Minor changes to fix dark mode compatibility:

Removed dark purple inline roll highlights
disabled z-index for cohort condition names and hold strength, as they disappeared in dark mode

Character Sheet Info Roll20 Internal Use only.


React Native support

One of the biggest reason to use styled-components is RN support.
It definitely, should be some CSS→RN loader for webpack. We need only show some example in the docs.

what would this look like for RN, they don't have stylesheets right? we'd need to turn the extracted style into something they can use...i'm super unfamiliar with RN :P

Me too :(.
Seems like they need inline styled or something similar.
I don’t suggest to have first-class support for RN. Just any snippet in docs for marketing purposes.

looks like SC already solved this, so we just need to add a RN entry that uses https://github.com/styled-components/css-to-react-native
I wonder if that would be a good idea tho, it would be all at runtime, which is fine for RN but a little "off brand" so to speak for us

right, i don't really see the point...

I mean it is some amount of work, we can convert to an object before hand  reducing the runtime cost of something like this, so it would be an improvement runtime-wise over SC. The main benefit tho i'd see is potential sharing? Or rather not having to switch abstractions/packages if you want to use a SC style api in both. However, most of our value as an abstraction is derived from the underlying preprocessor which you'd lose maybe here.
I have no idea what the RN build step is as well...is it even possible to preprocess the file?


Make sure folders exist before writing files

Handling this common I/O issue would be helpful for a lot of users, especially when we’re writing an image to a new folder.

Sorry about the multiple commits - my linter is wrecked at the moment and doesn't pick up on undefined variables or naming conflicts.

Sorry for the late response. Nice catch :+1:. It is true that something should be done about creating the directories that the files are written into. I'm not sure about the implementation though.
The easiest way I think this could be done is by adding a step in here that creates the directories, so that it isn't necessary to implement this for each stylesheet generator and compositor.
Let me know wether you want to adapt your PR or want me to take a look at this. If you want to create a PR, please add tests for the functionality.

@selaux Heya, I'm not sure how to write unit tests and the code changes are only small, so perhaps it would be more useful for you to adapt the PR. Sorry about that :/

No worries, thanks for the bug-report :+1: . I will close this in favor of https://github.com/selaux/node-sprite-generator/pull/46.