Add more zone config extensions examples

Fixes DOC-8558

@dikshant I guess zone config extensions owned by foundations now? is there someone on the eng side you would recommend have a look at this as well?
re: the PR content, the super regions example is a pretty straight easy port to cockroach demo with a multi-region movr DB
the x-region write latency I had to tweak a bit b/c once movr becomes multi-region all tables have REGIONAL BY TABLE already which is nice.  I had to remove the setting of lease_preferences that was given in the Jira issue b/c that is not allowed by zone config extensions, but presumably making the distant regions have non-voting replicas is enough?

the x-region write latency I had to tweak a bit b/c once movr becomes multi-region all tables have REGIONAL BY TABLE already which is nice. I had to remove the setting of lease_preferences that was given in the Jira issue b/c that is not allowed by zone config extensions, but presumably making the distant regions have non-voting replicas is enough?

Good question, I'm not sure for the REGIONAL case, though it's not allowed to set lease_preference with zone config extension, whether we should set it with ALTER DATABASE movr CONFIGURE ZONE USING lease_preference prior to setting the zone config extension. Started a thread in https://cockroachlabs.slack.com/archives/C4ERHM60Z/p1696266134754579

the x-region write latency I had to tweak a bit b/c once movr becomes multi-region all tables have REGIONAL BY TABLE already which is nice. I had to remove the setting of lease_preferences that was given in the Jira issue b/c that is not allowed by zone config extensions, but presumably making the distant regions have non-voting replicas is enough?

Good question, I'm not sure for the REGIONAL case, though it's not allowed to set lease_preference with zone config extension, whether we should set it with ALTER DATABASE movr CONFIGURE ZONE USING lease_preference prior to setting the zone config extension. Started a thread in https://cockroachlabs.slack.com/archives/C4ERHM60Z/p1696266134754579

ok I'll jump on that thread! thanks

@ecwall and @dikshant , removing y'all since Jane has LGTM'd (Thanks Jane!!!)
Gonna send this on for docs review so we can get it merged

Looks good, just style and formatting nits!

Awesome, thanks Ryan!  I'm also gonna port all these changes back to
at least v23.1
(Update: patches failed to apply to v22.2 so skipping for now)

In src/current/v23.2/alter-database.md:
Looks like this is lowercase?

Sure why not?

⬇ Suggested change
-- How you could implement Super regions
+- How you could implement super regions.

Check

⬇ Suggested change
-- How to minimize cross-region write latency
+- How to minimize cross-region write latency.

Fixed

⬇ Suggested change
-In this example, we use [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) to provide an alternative implementation of [Super Regions]({% link {{ page.version.version }}/multiregion-overview.md %}#super-regions), which are useful for [data domiciling]({% link {{ page.version.version }}/data-domiciling.md %}).
+In this example, we use [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) to provide an alternative implementation of [super regions]({% link {{ page.version.version }}/multiregion-overview.md %}#super-regions), which are useful for [data domiciling]({% link {{ page.version.version }}/data-domiciling.md %}).

Fixed

⬇ Suggested change
-Next, enter the statements below at the SQL prompt. These statements will:
+Next, enter the following statements at the SQL prompt. These statements will:

Fixed

⬇ Suggested change
-- Make the [movr database]({% link {{ page.version.version }}/movr.md %}) into a [multiregion database]({% link {{ page.version.version }}/multiregion-overview.md %}) by [setting the primary region]({% link {{ page.version.version }}/alter-database.md %}#set-primary-region).
+- Make the [movr database]({% link {{ page.version.version }}/movr.md %}) into a [multi-region database]({% link {{ page.version.version }}/multiregion-overview.md %}) by [setting the primary region]({% link {{ page.version.version }}/alter-database.md %}#set-primary-region).

Done. But I'm never going to stop trying to make "fetch" happen!  (fetch = multiregion)

⬇ Suggested change
-Now that our multiregion cluster is configured, we will use the ALTER LOCALITY statements shown below to [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that:
+Now that the multi-region cluster is configured, use the following ALTER LOCALITY statements to [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that:

Done

⬇ Suggested change
-- Add the remaining regions we defined at cluster start time to the multi-region movr database (for a total of 6).
+- Add the remaining regions defined at cluster start time to the multi-region movr database (for a total of 6).

Done

Per style (feel free to replace my suggestion though)
https://github.com/cockroachdb/docs/blob/main/StyleGuide.md#style-and-tone
⬇ Suggested change
-In this example, we use [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) to provide an alternative implementation of [Super Regions]({% link {{ page.version.version }}/multiregion-overview.md %}#super-regions), which are useful for [data domiciling]({% link {{ page.version.version }}/data-domiciling.md %}).
+In this example, [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) are used to provide an alternative implementation of [Super Regions]({% link {{ page.version.version }}/multiregion-overview.md %}#super-regions), which are useful for [data domiciling]({% link {{ page.version.version }}/data-domiciling.md %}).

Done

⬇ Suggested change
-In this example, we use [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) to minimize cross-region write latency in a cluster with the following characteristics:
+In this example, [Zone Config Extensions]({% link {{ page.version.version }}/zone-config-extensions.md %}) are used to minimize cross-region write latency in a cluster with the following characteristics:

Done

⬇ Suggested change
-- 3 main US regions (us-west1,us-east1,us-central1).
+- Three main US regions (us-west1,us-east1,us-central1).

Done

⬇ Suggested change
-- 2 additional distant regions (europe-west1, asia-northeast1). Only [follower reads]({% link {{ page.version.version }}/follower-reads.md %}) will be made from these regions.
+- Two additional distant regions (europe-west1, asia-northeast1). Only [follower reads]({% link {{ page.version.version }}/follower-reads.md %}) will be made from these regions.

Done

⬇ Suggested change

To minimize the cross-region write latency, we will [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that the 3 US regions have voting replicas (that is, replicas that participate in [Raft quorum]({% link {{ page.version.version }}/architecture/replication-layer.md %}#raft)). This will leave the 2 additional non-US regions with only [non-voting replicas]({% link {{ page.version.version }}/architecture/replication-layer.md %}#non-voting-replicas).
+To minimize the cross-region write latency, [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that the 3 US regions have voting replicas (that is, replicas that participate in [Raft quorum]({% link {{ page.version.version }}/architecture/replication-layer.md %}#raft)). This will leave the 2 additional non-US regions with only [non-voting replicas]({% link {{ page.version.version }}/architecture/replication-layer.md %}#non-voting-replicas).

Done

⬇ Suggested change
-Next, enter the statements below at the SQL prompt. These statements will:
+Next, enter the following statements at the SQL prompt. These statements will:

Done

⬇ Suggested change
-- Make the [movr database]({% link {{ page.version.version }}/movr.md %}) into a [multiregion database]({% link {{ page.version.version }}/multiregion-overview.md %}) by [setting the primary region]({% link {{ page.version.version }}/alter-database.md %}#set-primary-region).
+- Make the [movr database]({% link {{ page.version.version }}/movr.md %}) into a [multi-region database]({% link {{ page.version.version }}/multiregion-overview.md %}) by [setting the primary region]({% link {{ page.version.version }}/alter-database.md %}#set-primary-region).

Done.  But in protest!  e-mail became email eventually!!!!1!!!!

⬇ Suggested change
-- Add the remaining regions we defined at cluster start time to the multi-region movr database (for a total of 5).
+- Add the remaining regions defined at cluster start time to the multi-region movr database (for a total of 5).

Done

Link ALTER LOCALITY.

Done

⬇ Suggested change
-Now that our multiregion cluster is configured, we will use the ALTER LOCALITY statements shown below to [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that:
+Now that the multi-region cluster is configured, use the ALTER LOCALITY statements shown below to [configure replication zones]({% link {{ page.version.version }}/configure-replication-zones.md %}) such that:

Done

⬇ Suggested change
-- This will leave the 2 additional regions (Europe and Japan) with only [non-voting replicas]({% link {{ page.version.version }}/architecture/replication-layer.md %}#non-voting-replicas) that can be used to serve [follower reads]({% link {{ page.version.version }}/follower-reads.md %}).
+- The 2 additional regions (Europe and Japan) are left with only [non-voting replicas]({% link {{ page.version.version }}/architecture/replication-layer.md %}#non-voting-replicas) that can be used to serve [follower reads]({% link {{ page.version.version }}/follower-reads.md %}).

Done

In src/current/v23.2/cockroach-demo.md:

⬇ Suggested change
---demo-locality |  Specify [locality]({% link {{ page.version.version }}/cockroach-start.md %}#locality) information for each demo node. The input is a colon-separated list of key-value pairs, where the ith pair is the locality setting for the ith demo cockroach node.For example, the following option assigns node 1's region to us-east1 and availability zone to 1, node 2's region to us-east2 and availability zone to 2, and node 3's region to us-east3 and availability zone to 3:--demo-locality=region=us-east1,az=1:region=us-east1,az=2:region=us-east1,az=3By default, cockroach demo uses sample region (region) and availability zone (az) replica localities for each node specified with the --nodes flag.
+--demo-locality |  Specify [locality]({% link {{ page.version.version }}/cockroach-start.md %}#locality) information for each demo node. The input is a colon-separated list of key-value pairs, where the ith pair is the locality setting for the ith demo cockroach node.For example, the following option assigns node 1's region to us-east1 and availability zone to 1, node 2's region to us-east2 and availability zone to 2, and node 3's region to us-east3 and availability zone to 3:--demo-locality=region=us-east1,az=1:region=us-east1,az=2:region=us-east1,az=3By default, cockroach demo uses sample region (region) and availability zone (az) replica localities for each node specified with the --nodes flag.

Done, but only because it looks better.  God markdown is a mess


Lots of armor items don't show up in game and Empress Garb appearance not working for some outfits

Mod Version: <IP_ADDRESS>
Game Version: 1.4.31_Final (GOG) with all DLCs installed and working
First the Empress Garb Outfits that don't work, tested on male and female Half Orc, Dwarf and Half Elf

Druid Leather Armor
Scavenger Ranger Outfit, also makes character bold
Bard Armor
Warlock Armor

Haven'tt testedd on all characters but basically all crafted Breastplate, Half-Plate and FullPlate armours that this mod adds don't show up in game when equiped.

@Silvergriffon , I confirm this is happening but no idea why... We're copying over the itemPresentation from original into new items. Can you please take a look?

What is even more strange. If work for Hide and others but not for some. All use same routine to copy from original.

I think it somehow has to do with the armor addressable fields. I will try to look this weekend, but right now I have to go get ready to drive an ice cream truck in a Christmas parade. As one does.

PR for fix submitted

Thanks @Silvergriffon , do you believe we should do the same with the custom armors we add?


Swift nightly has improved alloc counts.

This patch updates the alloc counter to enforce the new numbers.

This is not as good as it looks: this is merely fixing a regression from 5.5.


Use consistent default prom port

Align sample file / CSV with kubebuilder / CRD, using 9102

/approve


Automated Test Suite Integration

Forms-flow-ai comes with a suite of tests that can be integrated into the pipeline. The tests and CI files for GitHub actions are already included in the project for the following components:

forms-flow-web
forms-flow-webapi
forms-flow-bpm
forms-flow-documents
forms-flow-data-analysis-api

AC:

Integrate CI files into the pipeline.
Make sure all the tests are passing after the integration. Our customizations may have some tests to fail. In this case, those tests would need to be compatible with our changes.

Forms-flow-ai comes with a suite of tests that can be integrated into the pipeline. The tests and CI files for GitHub actions are already included in the project for the following components:

forms-flow-web
forms-flow-webapi
forms-flow-bpm
forms-flow-documents
forms-flow-data-analysis-api

AC:

integrate CI files into the pipeline.
Make sure all the tests are passing after the integration. Our customizations may have some tests to fail. In this case, those tests would need to be compatible with our chagnes.

As the first step, Bhumin will connect with Simen to plan for test integration.

On hold: Will be completed once the upgrade is deployed.

This work is complete. Additional work to be done before deployment in a new ticket


rover run plan issue

Hello team,
I started to deploy CAF Azure for my first time. i follow the guide that find here:
https://aztfmod.github.io/documentation/docs/enterprise-scale/landingzones/platform/single reuse/elsz-single-reuse
When i run onther configuration-->level0-->launchpad-->readme.md command:
rover
-lz /tf/caf/landingzones/caf_launchpad
-var-folder /tf/caf/configuration/level0/launchpad
-tfstate_subscription_id xxxx
-target_subscription xxxx
-tfstate caf_launchpad.tfstate
-launchpad
-env contoso
-level level0
-p ${TF_DATA_DIR}/caf_launchpad.tfstate.tfplan
-a plan
I find this error:
@calling terraform_plan -- azurerm
╷
│ Error: reading Management Group "xxxxx": managementgroups.Client#Get: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="NotFound" Message="'/providers/Microsoft.Management/managementGroups/xxxxxxxxx' not found"
│
│   with module.launchpad.data.azurerm_management_group.level["root"],
│   on /home/vscode/.terraform.cache/launchpad/contoso/modules/launchpad/roles.tf line 41, in data "azurerm_management_group" "level":
│   41: data "azurerm_management_group" "level" {
│
╵
Terraform plan return code: 1
Error on or near line 391: Error running terraform plan; exiting with status 1
could you help me to fix this. for your information i use visual studio code for configure this env.
Thanks a lot.

i fixed this quick and dirty, not as intended, but in my tenant, the managemtn groups weren't activated, so logging in to azure, opneing management groups, and activating, or starting or something, after that it ran fine.

@20FS21 I would check 2 things:

be sure you properly fulfill the fields required by the '/tf/caf/landingzones/templates/platform/deploy_platform.sh'
from the Azure Active Directory portal click on the Properties blade and check the option "Access Management fo Azure resources" for the account you're using to deploy is set to YES.

Closed as answered by @simonepegperego

Thanks a lot


unclosed character literal '\u005B'

thank you for your detailed submission which contains everything needed to reproduce the problem in the current version of scalameta, as well as unequivocal evidence that the problem is indeed in 4.8.11 or later.
p.s. since i can't copy-and-paste it, i can't test it either.

you could just type out that unicode literal though... there's nothing much to copy-paste here. and the error is from a vscode extension. you could maybe ask for the version, and i would give some extra effort to find out the version.. anyway, thanks 👍

Looks like it's working for me in the latest Metals. Could you double check @audacioustux ? If not please copy and paste the snippet, it's reach much easier than having a screenshot. Thanks!

@tgodzik The unicode literal was given in the issue title. and yes, the problem was in metals v1.0.1, and fixed in 1.1.0

The unicode literal was given in the issue title

Sometimes it might be an issue caused by a specific combination of syntax, but I guess this wasn't the case here.
Cool to hear it works. Thanks @kitbellew for fixing this!


Use lib-pixelstreamingfrontend, raises player not found error in signalling server

**UE Version: **
E.g. UE 5.1.1
**Frontend Version: **
E.g. 0.3.0
Problem component
E.g. Signalling Server, Frontend
Description
A clear and concise description of what the bug is.
Steps to Reproduce:

Initiate a PixelStreaming object with the following config
initialSettings={{
AutoPlayVideo: false,
AutoConnect: true,
ss: 'ws://<IP_ADDRESS>:91',
StartVideoMuted: false,
HoveringMouse: false
}
<IP_ADDRESS> is the public ip of the signalling server. 91 is the exposed port for Players
The websocket connection error is terminated. From signalling server log, I observed this Dropped message iceCandidate cause no player XXX found.

Ok

Does this still occur when you use UE 5.2 and frontend version https://github.com/EpicGames/PixelStreamingInfrastructure/releases/tag/UE5.2-0.6.2
?


Branch Creation Rights

@CardcaptorRLH85  can you give me right to create branches and make releases please ?
git -c diff.mnemonicprefix=false -c core.quotepath=false --no-optional-locks push -v --tags --set-upstream origin feature/throttle-turns:feature/throttle-turns
remote: Permission to FRC0322/FRC322Java2020.git denied to ashtonbradley45274.
fatal: unable to access 'https://github.com/FRC0322/FRC322Java2020.git/': The requested URL returned error: 403
Pushing to https://github.com/FRC0322/FRC322Java2020.git
Completed with errors, see above.

You should have permission now.

thanks man
On Tue, Feb 4, 2020 at 11:34 PM CardcaptorRLH85<EMAIL_ADDRESS>wrote:

Closed #2 https://github.com/FRC0322/FRC322Java2020/issues/2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FRC0322/FRC322Java2020/issues/2?email_source=notifications&email_token=AE3CGCDIIFLAUHFKHF7JTNLRBI6WPA5CNFSM4KQEQ3V2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWNJYOFY#event-3008595735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3CGCH23LX74EX5Y4UJDJ3RBI6WPANCNFSM4KQEQ3VQ
.

--
Ashton Junior Ceaser Bradley B.S.C.S MS , Information System Business Intel.
IBM Technical Delivery Manager /Baker College Adjuct. Instructor / Medical
Information Sec. Consultant
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>|
<EMAIL_ADDRESS>|<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>


async configureStore

Because offline first apps need to fetch data from async local storage abstractions like redux-storage.

nice pattern, but over Este scope


Fixed-size array property not being referenced properly

I've come across a bug where a fixed-sized array property isn't being accurately represented in UEP.
The property in question is part of UCurveVector (Source/Runtime/Engine/Classes/Curves/CurveVector.h) and is defined as:
UPROPERTY()
FRichCurve FloatCurves[3];

To show what is happening, create a blueprint and add a Timeline to it. Add a single VectorTrack to it, and add a few keyframes. Then run the following:
timeline = bp.Timelines[0]
floatCurves = timeline.VectorTracks[0].CurveVector.FloatCurves

I would, at this point, expect floatCurves to be an array of 3 items. However, it is just the first item, giving no access to the other 2.
I'm interested in having a dig around in UEP to figure out where this might be happening, but I'd imagine it's pretty core code in UEP, so I'm reluctant to risk breaking anything by proposing a change myself!

Aha - awesome - thanks for the pointer (pun not intended) there.


conditional orderbook tab and button

One thing to note is that logged out mobile users still can't see the orderbook. This was an intentional choice as there wasn't a good spot to put it and I don't think a logged out mobile user cares that much.
Signed out desktop users, non-eligible sweeps signed-in desktop users, and non-eligible sweeps signed-in mobile users can now see the orderbook (although this is limited to binary markets rn - which should be fine since those are the only sweeps markets).

Could you post a screeny?


[Simulation, Predictoor bot approach3] Fix "WARNING: diffs not ok: mn=3600000, mx=3600000" #263

Duplicate with #263. Closing


Bump Dependencies #1616100068

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1616100068

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


Add the sageExtGet function to adapt custom version of buildTools, compileSdk and targetSdk

Add the sageExtGet function to adapt custom version of buildTools, compileSdk and targetSdk

can this be merged? addresses PR https://github.com/maxs15/react-native-spinkit/issues/104 and https://github.com/maxs15/react-native-spinkit/issues/109

can this be merged? addresses PR #104 and #109

yes  same problem  they should use API 28

Same issue. Can this be merged?

Another way to get around this issue and have code in source control is to update android\build.gradle under 'allProjects' section to force comipler to use later versions:
    subprojects {
        afterEvaluate {
            project ->
                if (project.hasProperty("android")) {
                    android {
                        compileSdkVersion = rootProject.compileSdkVersion
                        buildToolsVersion = rootProject.buildToolsVersion
                    }
                }
        }
    }

Another way to get around this issue and have code in source control is to update android\build.gradle under 'allProjects' section to force comipler to use later versions:
    subprojects {
        afterEvaluate {
            project ->
                if (project.hasProperty("android")) {
                    android {
                        compileSdkVersion = rootProject.compileSdkVersion
                        buildToolsVersion = rootProject.buildToolsVersion
                    }
                }
        }
    }

well， but in this way  minSDK and targetSDK can not  be replace with custom.

Same issue. Can this be merged?

yes it can be merged

Thanks ! Going to merge it, everything looks good

Thanks ! Going to merge it, everything looks good

still not working in 1.3.0


Syntax Highlighting Completely Broken

Description of the problem or steps to reproduce

Region highlighting doesn't work as expected and to say it "works" at all is a bit of a stretch. Highlighting seems to operate on Russian roulette with this build. Using subgroups no longer works. ( The highlight system no longer matches symbol.tag, it only matches symbol despite symbol.tag being defined in my colorscheme. Has the colorscheme format changed? ) Using includes doesn't seem to work correctly. Micro's yaml linting doesn't complain, so I have no idea what's causing this.
See my highlighting file here: https://somethinghub.com/magicant/html.yaml
Commit hash: 7ef175a
This issue has showed up on multiple terminals on multiple OS'es. Locally and over ssh. I've tried recompiling, I've tried editing the yaml files, I've tried clearing out the go sources and recloning the repo before build. It shows up in every single instance.
Also, as a bit of nitpicking, it seems as though many of the improvements I made to the old syntax files aren't there anymore.

There have been some unforeseen problems with the new syntax highlighter :) -- I guess this is what we have released versions.
I see what's happening here and it has to do with the tinkering I was doing yesterday to try to get string escaping working properly. I have now reverted the things that were causing problems but now I need to go through and add skip statements to all string regions. The highlighting for your file should be working better now (although it would be nicer if the <script> and <style> tags were highlighted -- currently not possible).
About the improvements you made: there was a large mess merging the syntax files because a number of files had been converted to use regions before your syntax improvements were made so the improvements for those files are currently not there. I do have all the files with your improvements and intend to add regions to them at some point.
It seems like the git history got a bit messed up with the merge (I didn't rebase first which made merging easier but I think was a mistake) so it seems like the improvements are gone from the git history (not sure)? I can put them in a repo if you want to be able to access them though.
This view refactor merge has been more of a mess than I thought it would be but it's good to work all these problems.

Html in particular is a hard filetype to get syntax highlighting right for.
Ok I've now added support for highlighting the limits of a region in a certain color by using the limit-group property.
On the latest version, the syntax file looks like this:

@zyedidia I still have them locally and can add them to the new yaml files and set up the region stuff. It's annoying, but not the end of the world. As for the  not being highlighted, that should possibly be fixable, I'll mess around with it until I get it.


Fixing old test harness bug where we could get multiple services created

@pcross616

sweet, looks good, will merge after test build


Contributor guide link in Community page results in a 404

Update the link to https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md


proxy api?

What is the role of the proxy at https://chaction.csie.cool/? It appears to be a data siphon with no purpose.

Hello @tymrtn,
The proxy performs 2 functions:

It converts HTML pages into Markdown, which reduces the token count while maintaining the same semantics, thus preventing the context limit from being reached.
It responds in JSON format, which was necessary when I developed this, as ChatGPT Actions only accepted JSON at the time. (I am not sure if this has changed now.)

The code is available here, and is quite simple, comprising only 60 lines of code.

no, no need it

negative, no need it


GUI playground

It's not quite as good as the Luna park editor but we've got an InfiniteCanvas component in use in our applications that would make for a passable GUI playground to visualize the cst tokens.
I'll look to make some progress in the next week or so.

@balbatross If you're looking to contribute, and especially to built on top, there's a lot of other work to be finished first! I've been using Notion to track work, but maybe it's time to move everything back here. That way if people are interested they can see what work remains to be done!

Since the easiest way to render a tree of tokens is to just print code, I was thinking that my first visual demo would be a simple syntax highlighting playground. So far no languages yet have complete working implementations. It doesn't matter yet because the core is broken, but anything I can do to help people write and test the necessary language definitions will be paid back many times over.


Update aerospike image to <IP_ADDRESS>

Codecov Report

Merging #41 into develop will decrease coverage by 1.96%.
The diff coverage is 16%.

@@             Coverage Diff              @@
##             develop     #41      +/-   ##
============================================
- Coverage      83.16%   81.2%   -1.97%
- Complexity       239     241       +2
============================================
  Files             61      63       +2
  Lines            790     814      +24
  Branches          12      12
============================================
+ Hits             657     661       +4
- Misses           126     146      +20
  Partials           7       7

Impacted Files
Coverage Δ
Complexity Δ

...m/playtika/test/aerospike/AerospikeProperties.java
28.57% <0%> (ø)
1 <0> (ø)
:arrow_down:

...m/playtika/test/common/utils/PackageInstaller.java
0% <0%> (ø)
0 <0> (?)

...tika/test/common/properties/PackageProperties.java
0% <0%> (ø)
0 <0> (?)

...pike/AerospikeTestOperationsAutoConfiguration.java
100% <100%> (ø)
6 <2> (+2)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 965b2a7...0e4a7f4. Read the comment docs.


Allow Trino connections with password

fix https://github.com/erezsh/sqeleton/issues/14

Most of the files are not mandatory. Those are just instructions and files to spun a Trino cluster using password file authentication.
We don't need to bump the Trino version. As I was working those files I though we can bump it.

New options in server config
If using jdk < 22, new versions of Trino will complain about

dd-trino  | ERROR: Trino requires -XX:+UnlockDiagnosticVMOptions -XX:G1NumCollectionsKeepPinned=10000000 on Java versions lower than 22.0.2 due to JDK-8329528
https://bugs.openjdk.org/browse/JDK-8329528

-XX:+UnlockDiagnosticVMOptions
Unlocks additional diagnostic options for the JVM.
Here's more info https://github.com/trinodb/trino/pull/12251
-XX:G1NumCollectionsKeepPinned
More info https://github.com/trinodb/trino/pull/21999

To allow Trino connections with password, I believe two routes would work.
Check for "auth" parameter like we have in presto.py
https://github.com/erezsh/sqeleton/blob/01ef6c7ee742aee754a8ea628fc3b6f0ec2d5655/sqeleton/databases/presto.py#L173C1-L174C95
Check for password
if kw.get("password"):
    kw["auth"] = trino.auth.BasicAuthentication(
        kw.pop("user"), kw.pop("password")
    )

The second method would work for sqlalchemy like conn strings and also when using db_conf dict with user and password.
@erezsh I may be missing something, let me know what you think and I can change it to check for auth instead. Thank you so much let me work on that one.

Thanks for the PR!  It looks pretty good. I especially like the README you added with the extra info.
A few points:

The tests are failing because they can't connect. What URL should we use for trino now?

If there are any other explanations you can add for the added lines (like why you changed jvm.config), it would be welcome.

The code can be a bit simpler if you use cert = kw.pop("cert", None) and then test if cert is None.

@erezsh thank you so much for the feedback.
The tests in test_database could run against the same settings as before. Or the password authentication could be the default settings for Trino cluster, then we test TRINO_URI with localhost:8081 (without auth) and localhost:8443 (with password). There is an option that allow us to access http even with password enabled. (insecure but just for the tests). If we go for the later I need to edit de ci to build the certs before starting the server, or we can let "default" certs in git. Not secure but it is only for the tests.

Thank you for contributing!


vertica-go-sql connector hangs when processing failed exec command.

This is a bit of a complex test but none the less it fails predictably.
Situation is that during the development of a UDX Source plugin we found that simply throwing an error from the UDSouce::processWithMetadata function causes the vertica-go-client to never return from the call and hangs the session.
// The below implementation on the source causes the vertica go client to hang indefinitely.
StreamState TestSource::processWithMetadata(ServerInterface &srvInterface, DataBuffer &output,
LengthBuffer &output_lengths)
{
vt_report_error(2, "FORCE process with Metadata error");
return StreamState::DONE;
}
Now we use the vertica-go-sql sample app thats provided with the appropriate connection string changed. Plus the following additional snippet...
// Query a standard metric table in Vertica.
_, err = connDB.ExecContext(ctx, "COPY public.from_test SOURCE TestSource() PARSER KafkaJSONParser(flatten_arrays=True, flatten_maps=True) DIRECT NO COMMIT;")
if err != nil {
    testLogger.Fatal(err.Error())
    os.Exit(1)
}
testLogger.Info("Test complete")

Running the above snippet from vsql, the proper error is returned every time and no hangs.
And the vertica.log file shows that the TestSource was executed and returns with same logs as when executed from the vsql command.
testudx.tar.gz

NOTE: I reproduced the same results against vertica versions 9.2.0, 9.2.1, and 9.3.0.

When inspecting the go stack trace.. the error appears to be in that the Close() stmt.go:124 is expecting a message response from the server with either a type=msg.BECloseCompleteMsg to exit the loop, bu no message is ever received in this case. But a message is never sent from Vertica.

Possible patch to this issue it to set s.rollback = true. Which allows the close logic to exit gracefully when a error is reported from the Exec command.
// ExecContext docs
func (s *stmt) ExecContext(ctx context.Context, args []driver.NamedValue) (driver.Result, error) {
stmtLogger.Trace("stmt.ExecContext()")
rows, err := s.QueryContext(ctx, args)

if err != nil {
	s.rolledBack = true
	return driver.ResultNoRows, err
}

@klwilson227, I'm worried this issue might manifest in other ways. Since it would be kind of tricky to replicate outside of your environment, could you check to see if there are some other messages getting ignored in the process? For example, inside of the Close() method, can you call defaultMessageHandler() to log any other conditions?

I have replicated this issue already in 3 different vertica environments including the community 9.3 bits that I pulled down about 8 weeks ago. I realize that compiling and deploying the .so from the vertica go test infrastructure; adds a bit to the complexity but it has been very simple to replicate after loading the SO by hand.
I believe this issue would replicate in hundreds of different scenarios in which the vt_report_error api is call either internally by vertica code or externally via a UDX plugin. You can likely reproduce the issue on any of the vertica provided UDX plugins, by giving them what appears to be a valid configuration, but one that has a operational/runtime reported issue. For example, try the same with the KafkaSource, and give it a kafka endpoint that is invalid.  This could simplify the testing scenario but complicate the UDX plugin used.
From what I have seen in the tracing this is not a matter of miss handling of messages but a matter of never receiving a response to the close message that is issued during the Close() method. I see two bind messages come in prior to simply running out of data responses.
By setting the rolledBack, I am avoiding the close message being sent. And thus the code no longer gets stuck here. I am a novice at the wire protocol used by vertica. But something certainly is not in alignment here, and I hope those whom know better can make a decisions on what the right course of action should be to resolve this issue.  The work around/solution I proposed I will have more formally in a  PR once, I can get the test environment running.

Would also ask @sitingren and @tomwall to have a look at this. @sitingren is on the vertica-python driver and @tomwall is master of all things Vertica.

Could you please provide the log file of executing your Go program? I'd like to see the wire protocol communication messages btw the server and vertica-sql-go. Thanks!

Kevin and I discussed offline; this seems like a general problem with how the client responds to certain kinds of errors. You can reproduce with a much simpler query: select throw_error('whatever').
I suspect it may be specific to errors that pass the syntax/semantic checks but fail at execution time. In other words, the query/bind+exec message is accepted by the database, the database does semantic analysis and it sends back the row description for the result, but then at execution time the error response is sent where you'd usually see row data / command complete instead.
If the client does a read expecting a certain message that will never arrive, like a row or command complete, then it will get stuck. If that's whats happening the client should issue a sync and then seek until it gets the RFQ. Before packaging up that error and returning it to the application.

Interesting. I've seen that pattern once before and fixed it with a simple catch and a call to sync(). I think it was in the Parse/Describe phases. It seems it can happen in the Bind/Execute sides as well.
Tonight I'll have a look at fixing this in a consistent manner.

It looked line a one-liner. Please check https://github.com/vertica/vertica-sql-go/tree/huebnerr-lock-on-error

Will get this done as soon as I can.
Get Outlook for iOShttps://aka.ms/o0ukef

From: huebnerr<EMAIL_ADDRESS>Sent: Monday, March 23, 2020 2:50:44 PM
To: vertica/vertica-sql-go<EMAIL_ADDRESS>Cc: Kevin Wilson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [vertica/vertica-sql-go] vertica-go-sql connector hangs when processing failed exec command. (#79)
It looked line a one-liner. Please check https://github.com/vertica/vertica-sql-go/tree/huebnerr-lock-on-error
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/vertica/vertica-sql-go/issues/79#issuecomment-602849034, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA4UNK6YUOT7FH6Y5Q3KB7LRI7DSJANCNFSM4LQ7F27A.

Addressed in https://github.com/vertica/vertica-sql-go/pull/80


Question about the code that predicts the capacity of the entire cycle

In the inferring_voltage code file, the file for charge_data and discharge_data used for the input values of the model uses information that is not from the first 100 cycles, for example print(charge_data[0].shape) is (687, 4, 500). How to predict the whole complete using only the first 100 cycle's, would like to have your answer thank you!
`cell_feature={}
for i in tqdm(range(len(charge_data))):
charge_feature=feature_selector(feature_selector_ch,
charge_data[i],charge_norm)
discharge_feature=feature_selector(feature_selector_dis,
discharge_data[i],discharge_norm)
cell_feature['%d'%(i)]=concat_data(charge_feature,discharge_feature,
summary_data[i])
x_in1,x_in2,y_in1,y_in2,y_in3=process2predict(cell_feature,section)
in_x1=np.vstack(x_in1[cell_number]).reshape(-1,input_data_len,12)
in_x2=np.vstack(x_in2[cell_number]).reshape(-1,1)
predict_voltage,predict_capacity,predict_power=predictor.predict([in_x1,in_x2],batch_size=256)`

Thank you for your interest in our research. This demonstration primarily focuses on efficiently feeding all the data into the pipeline for prediction.
During the presentation, we will use the following settings:
'''
cell_number = 9     # battery ID
phase = 99          # which age of feature to predict
start_cycle = 500   # starting cycle
append = 5          # attach n cycle features
'''
These settings control the prediction conditions, ensuring that only information from the start_cycle and the following 5 appended cycles is used for the target prediction. Performing individual predictions without contamination from data outside the target range is also possible. You only need to refer to the padding method used in process2predict for handling the data, then input it into the model for prediction.


Swipe to reveal the year scroller does not feel good enough

Swiping the month scroller to left will reveal the year scroller. This is a good idea, but the swipe implementation should be revisited.
The current implementation needs a long and determined swipe. A casual quick and short swipe should be enough to trigger the effect. The <app-drawer> element might give some ideas for the implementation.
This is definitely a pet peeve of mine on the <vaadin-date-picker>. 😄

While changing the swipe code, please be careful not to re-introduce #91.


"Link to this answer" does not work if FAQ page is accessed via anchor link or "Link to this answer" was used once before

Where to find the issue

https://www.coronawarn.app/de/faq
https://www.coronawarn.app/en/faq

Describe the issue
If the FAQ page is accessed using an anchor link or the "Link to this answer" feature was used once before & the option "Link to this answer" is clicked on any FAQ entry, the URL in the address bar of the browser is not updated. So "Link to this answer" actually does nothing in this case.
Suggested change
Fix this issue.

@Ein-Tim
Please check if you think this is now fixed.


Bump rancher to v2.6.2

According to the Rancher team, some rancher-related issues are already resolved in the Rancher v2.6.2.
Bump rancher to v2.6.2 to verify those dependency issues.
https://github.com/harvester/harvester/issues?q=is%3Aopen+is%3Aissue+milestone%3Av1.0.0+label%3Aarea%2Francher-related

@guangbochen It would be nice to let this in first: https://github.com/harvester/harvester-installer/pull/103
It can handle the images.

Do you think we can bump to the master version? https://github.com/rancher/rancher/issues/34908 is not available in 2.6.2

@guangbochen It would be nice to let this in first: #103 It can handle the images.

sure, will take a review of this PR, @bk201 IIRC, we don't support fetching images from the master-head version right? so how do we specify the image when using the master-head for validation.

@guangbochen It would be nice to let this in first: #103 It can handle the images.

sure, will take a review of this PR, @bk201 IIRC, we don't support fetching images from the master-head version right? so how do we specify the image when using the master-head rancher for validation.

No, it can't handle master images because the image-list files are not published to GitHub release page.
For master-head-xxx releases, I need to run it in an air-gapped environment to know what are missing. :(

@gitlawr @bk201 I have bumped the rancher to the latest master-head image as per https://github.com/harvester/harvester-installer/pull/159/commits/e8b40cdd100e70a598e2a87811db7ba1fc82a5b3, probably just need to do an air-gap validation, thanks.

Some pods fail to start because the following images are missing:
rancher/shell:v0.1.13
rancher/fleet:v0.3.8-rc1
rancher/gitjob:v0.1.22
rancher/fleet-agent:v0.3.8-rc1
rancher/system-upgrade-controller:v0.8.0
rancher/system-agent:v0.1.1-rc4-suc

@guangbochen  2a66ce5 test OK!


removed explicit reference to Newtonsoft

Codecov Report

Merging #57 (98b2e50) into main (faf2a52) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #57   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          447       447
  Branches        29        29
=========================================
  Hits           447       447

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


reject duplicate fields

In https://github.com/adamgreig/stm32-rs/pull/83#issuecomment-435731760 we found that duplicate fields in a register aren't rejected by svd2rust. (There are probably other duplicate conditions that could be caught, such as duplicate register names?)
I'll make the required changes, but I need to know if you want to handle this in svd-parser already, while reading the SVD, or in svd2rust, while generating the output?

In my view it is better to check this in svd-parser with ValidationLevel::Strict


Update de.UTF-8

Edited a few umlauts to unicode syntax

Will merge it closer to the new release date! Thanks!!

why? lol you don't need this on UTF-8... please don't do unuseful stuff...
;-) Ilia... ...

Not sure what you mean?

I mean that is not required... it's enough to encode the original file to UTF-8 (if it's not already.,.) you don't need to change the text at all ;-)

just checked the /usr/libexec/webmin/authentic-theme/lang here on CentOS 7
the de.UTF-8 file is already encoded in UTF-8 (BOM-less) so you don't need to edit the text...
you can write "Überweisung" without any issue...

I'm sorry for any unnecessary work, but so far German umlauts aren't shown correctly. See attachment.
Am 17.02.2015 10:45 schrieb 7starsone notifications@github.com:just checked the /usr/libexec/webmin/authentic-theme/lang here on CentOS 7
the de.UTF-8 file is already encoded in UTF-8 (BOM-less) so you don't need to edit the text...
you can write "Überweisung" without any issue...
—Reply to this email directly or view it on GitHub.

what browser are you running? to me is impossible... 'cause the browser autolearns what charset you can use... is something on your own

are you sure to have selected de.UTF-8 as language and not just de ?

I am so, so sorry! You are right, I selected "de" not "de.UTF-8".
I just wanted to help ;)
One last question: Why are the umlauts written in different ways in the language files? Sometimes "ü" and sometimes "ü"?
Am 17.02.2015 10:58 schrieb 7starsone notifications@github.com:are you sure to have selected de.UTF-8 as language and not just de ?
—Reply to this email directly or view it on GitHub.

ok, found... you don't have de.UTF-8 now in the current Virtualmin package
so you got confused, thinking that even UTF-8 needs that... BUT UTF-8 doesn't need that ;-)
you can do this if you want to use UTF-8 entirely without changing anything:

go to http://www.motobit.com/util/charset-codepage-conversion.asp
download the /webmin/virtual-server/lang/de file
copy all the text
paste into the field at motobit
source iso-8859-1 | destination utf-8
copy the result text to a new de.UTF-8 file
upload the new file to /webmin/virtual-server/lang/

I don't understand why they don't do this easy thing one time only and then continue to work on that file....

Ilia,
here you can download the Portuguese pt.UTF-8 for your theme /webmin/authentic-theme/lang
https://mega.co.nz/#!lRoDFSSD!9JS6Fofr2KigyW6TUQqSzyoF5K5oHK6MdmytJTXrPjQ
;-)

@7starsone Thank you very much!! Will add it to the next release.


IndexError: list index out of range (HOW TO FIX?)

When I sent this in cmd:
python download_tweets.py viniccius13
Has showed this error:

How to fix this error?

same here

same here

dude, is just edit the download_tweets.py.
in (default)
'username:none,
limit:none, '
I did this changes:
'username:viniccius13,
limit:16000,'
in your case, put the username that you want and the limit of tweets that you wanna download.

same here

dude, is just edit the download_tweets.py.
in (default)
'username:none,
limit:none, '
I did this changes:
'username:viniccius13,
limit:16000,'
in your case, put the username that you want and the limit of tweets that you wanna download.

C:\Users\arceus\Pictures\arceus but ai\download-tweets-ai-text-gen>python download_tweets.py arceusoldandnewaccs.txt
Retrieving tweets for @arceusoldandnewaccs.txt...
Oldest Tweet:   0%|                                                                                | 0/40000000 [00:00<?, ?it/s]CRITICAL:root:twint.get:User:
CRITICAL:root:twint.get:User:
CRITICAL:root:twint.get:User:
CRITICAL:root:twint.get:User:
Oldest Tweet:   0%|                                                                                | 40/40000000 [04:05<68233:11:10,  6.14s/it]Traceback (most recent call last):
File "download_tweets.py", line 161, in
fire.Fire(download_tweets)
File "C:\Users\arceus\AppData\Local\Programs\Python\Python38\lib\site-packages\fire\core.py", line 138, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "C:\Users\arceus\AppData\Local\Programs\Python\Python38\lib\site-packages\fire\core.py", line 463, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "C:\Users\arceus\AppData\Local\Programs\Python\Python38\lib\site-packages\fire\core.py", line 672, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "download_tweets.py", line 152, in download_tweets
tweet_data[-1].datetime / 1000.0
IndexError: list index out of range
Oldest Tweet:   0%|

Thank you, that worked

I'm getting the same error but I think I downloaded the updated script (in the master branch, updated on May 12). Any idea how could I fix this? thanks

If you want to download all tweets you may just set some big limit for downloading
Like:
python3 download_tweets.py user 100000


removing raw pointers and std::map for *Solver m_state

First stab at using some smart pointers in the solver classes. I didn't test the elasticity / solid dynamics examples much because that implementation is about to be significantly reworked by https://github.com/LLNL/serac/pull/5 in the near future.
I'm not really sure what the appropriate smart pointer type is for many of the members, but changing from std::shared_ptr to std::unique_ptr is pretty easy to do.
The new test serac_dtor used to fail for by deleteing things too many times, but passes now.

It seems like m_state should have a Getter based on the "name" of the state. This ways you have an opportunity to check for "typos" and if the key exists. This would help with the spelling errors.

I had already reverted some of the std::map stuff before our meeting. This PR will be just for the shared_ptrs.

One other hiccup: my 2FA for my LLNL github account is tied to my laptop which is still on-site. So I'm stuck with my personal account for the next few weeks. This rules out my ability to merge pull requests or anything that requires write access to the repo.

One other hiccup: the 2FA for my LLNL github account is tied to my work laptop, which is still on-site. So I'm stuck with my personal account for the next few weeks. This rules out my ability to merge pull requests or anything that requires write access to the repo.

OK. Let me know when you think this is ready and I can merge it.

@jonwong12 @white238 any other thoughts on this branch? I'd like to get this merged in since it fixes the ubuntu builds for local development.

So, it looks like the dynamic test is failing to set up boundary condition attributes correct, after merging the new implementation via nonlinear_solid_solver. I spent some time looking into this, but without much success.

So, it looks like the dynamic test is failing to set up boundary condition attributes correct, after merging the new implementation via nonlinear_solid_solver. I spent some time looking into this, but without much success.

This sounds vaguely familiar, so I'll take a look after this is merged.


fix uri fragment slugification #94508

Previously, all links to local files had their fragment parts normalized (e.g. foo.md#Bar baz became foo.md#bar-baz). This allowed links to specific sections of other markdown files (file#Section header), but broke many other links.
This PR limits the normalization to paths without a file extension (as per suggestion).
This may, however, have an unfortunate side effect of breaking some markdown links that do have an extension (e.g. foo.md#Bar baz won't work, but foo#Bar baz will).
This PR fixes #94508

Added an additional check for known markdown file extensions. Now links are slugified if they have a markdown file extensions or do not have an extension at all.

Thanks! This change will be in the next VS code insiders build and is scheduled to go out with VS Code 1.45


add fake rpc vtgateconn

fakeVtgateConn implements VTGateConn interface and provides a fake
implementation that could be used for doing unit tests. fakeVtgateConn
does not need a running vtgate and thus more lightweight and much faster.

The fake may need to enforce calls / returns specific results to be actually useful. You probably plan on adding that later I assume? LGTM for now.

Actually I am thinking about a proper way to inject requests and specific results so that this fake vtgateconn could be reused in different unit test.
I feel if instead of relying on the dialer model (use vtgateconn.RegisterDialer & vtgateconn.GetDialerWithProtocol), we add factory methods and do dependency injection for any place in Vitess that uses a vtgateconn. That will help us to write unit tests more easily. E.g we can have a NewFakeVtGateConn(...) func that returns a fakeVtgateConn, it has methods like addRequests(request, expectedResult) and pass this instance to places that needs a vtgateconn instance.

Dependency injection has the added benefit that you won't be able to accidentally spin up a vitess cluster with a fake vtgateconn if you use the wrong flags.
Alain also has a good example of returning the values that you want from a fake: https://github.com/youtube/vitess/blob/master/go/vt/tabletserver/queryctl.go#L197

For simplicity, you can even poke into the module directly if you need to,
i.e. the test guy is an exported singleton, and has public variables that
describe what it got, and what it should return. Then each test just reads:

set dbcon's expected return values
run the test
check dbcon got what it should have
(it might read better if the fake dbcon has both what it expects and what
it returns).

It should end up read well in the unit test as well...
On Mon, Mar 16, 2015 at 5:16 PM, Ammar Aijazi<EMAIL_ADDRESS>wrote:

Dependency injection has the added benefit that you won't be able to
accidentally spin up a vitess cluster with a fake vtgateconn if you use the
wrong flags.
Alain also has a good example of returning the values that you want from a
fake:
https://github.com/youtube/vitess/blob/master/go/vt/tabletserver/queryctl.go#L197
—
Reply to this email directly or view it on GitHub
https://github.com/youtube/vitess/pull/497#issuecomment-82002114.

ok. I think I would like to take the dependency inject approach as @aaijazi suggested. Will update the code later.

Did some polish and now FakeVTGateConn has several Add* methods which allow a test to add a request along with expected return result.

LGTM


fix(hardsuits): Give the carp hardsuit a suit storage slot

About the PR

Gave carp hardsuit a suit storage slot
Why / Balance

It is a hardsuit. It should have suit storage.

Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

add to carp suit or else validhunters will kill anyone that has an oxygen tank on


ServiceBus: Message lock auto-renew

Working with the c# ServiceBus SDK it would seem that there is an auto-renew of the locks that happens behind the scenes for long running message related workflows. However, in the Java world this functionality doesn't exist. This means when I am working with messages I have to always write boilerplate code that will execute a scheduled thread pool to renew the locks, or my messages get abandoned etc.
Describe the solution you'd like
Having looked at the SDK it would seem that the MessageReceiver could have an additional property added to set auto-renew on or possibly take a configuration object to its constructor. If this is set to true then we could have a new object of type AutoRenewMessage that implements IMessage or even could inherit Message and extend it.
MessageReciever can be extended to have a ScheduledThreadPool that will keep track of these AutoRenewMessage using a new Runnable for example:
public class MessageLockAutoRenewTask implements Runnable {

    private final IMessageReceiver receiver;
    private final IMessage message;

    public MessageLockAutoRenewTask(IMessageReceiver receiver, IMessage message) {
        this.receiver = receiver;
        this.message = message;
    }

    @Override
    public void run() {
        try {
            this.receiver.renewMessageLock(message.getLockToken());
        } catch (InterruptedException ex) {
            // do something else here
        } catch (ServiceBusException ex) {
            // do something here
        }
    }
}

If the MessageReceiver was to be closed/disposed then the ScheduledThreadPool can be abandoned and all executing tasks will naturally close down.
When a message is abandoned, DLQ'd or completed it would just be a matter of checking if the IMessage is of type Message or AutoRenewMessage and then making sure the object stopped its scheduled task.
Describe alternatives you've considered

Extend the existing MessageReciever to have a new AutoRenewMessageReciever and then maintain message states on a concurrent collection.
Implement a new intermediary class similar to the c# library whereby all calls to the CoreMessageReciever go through this intermediary object and it tracks the state of messages in flight and renews the locks etc.

Additional context

I am currently working on a POC of this new code and was just wanting to raise this Issue as a proposal before I get round to raising a PR to get feedback regarding the approach and if the PR would be accepted?

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

@storey247, Thanks for raising this issue. Yes, your PR with the suggested approach is more than welcome. We will looking forward for that PR and we will review it.

Hi @jfggdl here is a PR with a POC implementation: https://github.com/Azure/azure-sdk-for-java/pull/5444
Feedback always welcome 🙂

@storey247
The auto renewal is only supported through Message (or Session) Handlers. Even in C#, if you utilize the MessageReceiver, then you have to explicitly renew the lock using RenewLockAsync(). see this comment
Why not leverage MessageHandler In Java and set MaxAutoRenewDuration in MessageHandlerOptions to manage the lock renewal?
Let us know if you have any further questions.

Hi @axisc,
Thanks for your insight, yes you are correct the MessageHandler does seem to support this behaviour but that Class is designed for the pub-sub paradigm. We recently had a situation where we needed more control over the rate at which messages were being consumed to maintain a throughput rate on upstream services.
We ended up using the MessageReceiver due to this required level of control so we could pull down batches of messages rather than having to react via an handler. This is why we encountered this strange lock renewal issue.
My pull request implements feature parity between the two paradigms to avoid confusion 🙂

@storey247
so you're trying to match the 2 paradigms and not the 2 language SDKs, right?
because even in the C# MessageReceiver, you still have to renew the lock manually.

@axisc
Correct, i had an implementation using the MessageHandler but this didn’t give us the necessary throughput, so we move to using MessageReceiver and batch pulling messages and were surprised to see that the auto-renew functionality was missing and had to be manually built
Regards,
Dave

@storey247  playing devil's advocate here, but it seems like if the throughput on the MessageHandler was improved it would solve your problem?

If you are using Java SDK, my gut tells me there will be no noticeable throughput difference with either MessageHandler or MessageReceiver. Would you mind sharing the throughput numbers you are seeing with either approach?

@yvgopal unfortunately I don't have the metrics to hand anymore as the project has moved on substantially now. Please note that this was an internal MSFT CSE project i am referring to.
@axisc unfortunately in this case I think the issue was that we were trying to retrieve messages in batches in a pull-push paradigm, not the usual pub-sub paradigm. The MessageHandler does not provide you with an easy platform from which to do timed pull events, process and then from the processed messages, push to another platform at a controlled rate of request.
I understand that the MessageHandler is the "preferred" method here, but in some cases it just isn't feasible and the MessageReceiver is actually the mechanism people prefer to use; hence why our MSFT CSE team has provided this PR in an attempt to prevent external users of the SDK falling into the same pitfall we did of having locks expire on them without realising they didn't auto-renew.

@storey247
I'll add the feature request to the backlog.


undefined key is treated as ''

Current Behavior
ioredis will treat a key of undefined the same as the key '', meaning you can call redis.set(undefined, 'some value') and later use redis.get('') to get 'some value' from the redis instance.
Expected Behavior
ioredis either throws an error that undefined was passed when there should be an explicit string or Buffer, or it is documented that ioredis will automatically convert the undefined to ''.
Minimum Reproduction Repository
https://github.com/jmcdo29/ioredis-undefined-key
Reproduction steps
git clone git@github.com:jmcdo29/ioredis-undefined-key.git
cd ioredis-undefined-key
pnpm i
docker compose up -d
node index.js
docker compose down

Observe the assertion failure from notEqual() in the getBlankStringKey method.

same issue
set(key, value, expiryMode, time, setMode)
expiryMode, time, setMode are undefined
ReplyError: ERR syntax error
at parseError (/app/node_modules/redis-parser/lib/parser.js:179:12)
at parseType (/app/node_modules/redis-parser/lib/parser.js:302:14) {
command: {
name: 'set',
args: [
'keyxxxxxxx,
'valuexxxxxxxx',
'',
'',
''
]
}

any help?

hello?

Hey @jmcdo29 @dontbesatisfied 👋
Yeah undefined and null are treated as empty strings so we should document it. We can change that in the next major version if there are good reasons.

IMO it makes sense to differentiate between undefined and null as there is a semantic difference between the two values.
In my case we have a getOrSet method which first checks if the value exists in the cache and if not it tries to fetch the value from a third party and stores it in the cache before returning the value.
Here it makes sense to differentiate between undefined and null. An undefined value will indicate that there is nothing stored in the cache and we want to fetch from the database. A null value will indicate that nothing exists for the key being looked up (either in the cache or the third party).


latest changes

ask.py added

Looks like you've accidentally added temporary build files (CMakeFiles) and build results (dll, so). They need to be cleaned up before I'm able to do review :)

Please also set a more understandeable name for this PR. A summary of the changes is a good option

I see no much value in keeping this PR open. Feel free to open a new PR with more specific changes, if you wish to do so.


vue-horizontal@next not yet support TS (Vue3) ?

vue-horizontal@next not yet support TS (Vue3) ?

src/main.ts:13:27 - error TS7016: Could not find a declaration file for module 'vue-horizontal'. 'C:/Github/self-serve-ordering.reno1/node_modules/vue-horizontal/dist/vue-horizontal.umd.js' implicitly has an 'any' type.
Try npm i --save-dev @types/vue-horizontal if it exists or add a new declaration (.d.ts) file containing declare module 'vue-horizontal';
13 import VueHorizontal from 'vue-horizontal';

@renogies I managed to get around this by creating a declaration file like this. Make sure to have the directory where you put it in your includes in the tsconfig.json.
vue-horizontal.d.ts:
declare module 'vue-horizontal' {
  import { DefineComponent } from 'vue'
  const VueHorizontal: DefineComponent<
    {
      button?: boolean
      buttonBetween?: boolean
      scroll?: boolean
      responsive?: boolean
      displacement?: number
      snap?: 'start' | 'center' | 'end' | 'none'
    },
    {
      prev: () => void
      next: () => void
      scrollToIndex: (index: number) => void
      scrollToLeft: (distance: number, scrollBehavior: 'smooth' | 'auto') => void
      refresh: () => void
    }
  >

  export default VueHorizontal
}


Wildcard for allowed commands and flags

I tried this
"ALLOWED_COMMANDS": "",
"ALLOWED_FLAGS": "",
but it does not seem to work. Is it possible to allow all? I understand the danger of this but I use ALLOWED_DIR for safety.
Thanks

Hello @ichoosetoaccept,
Thank you for your interest in the project and for bringing this to my attention.
I realize that the README was not updated with the latest information. You can indeed allow all commands and flags/parameters by using "all" in the configuration:
  "mcpServers": {
    "cli-mcp-server": {
      "command": "uvx",
      "args": ["cli-mcp-server"],
      "env": {
        "ALLOWED_DIR": "<REDACTED>",
        "ALLOWED_COMMANDS": "all",
        "ALLOWED_FLAGS": "all",
      }
    }

Then you can double-check if it worked by asking the client:
🧑 User: Which commands are you allowed to execute in the CLI?
╰─❯ claude-3-5-sonnet-latest:
I'll check the security rules to see what commands and operations are allowed in this environment.

▌ Triggered: show_security_rules.
Based on the security rules, I can execute:

1 All commands
2 All flags are allowed
3 Commands must be:
   • Less than 1024 characters in length
   • Complete execution within 30 seconds

I will update the README shortly to reflect this information.
Please give it a try and let me know if you encounter any issues
Cheers

Hi @MladenSU
Thanks for the swift response! After using the settings like you described and testing it with the same prompt as you, I can confirm that this does exactly what I wanted.
Thank you!
Cheers 😊

Glad to hear all is well.
Have fun using it! 😊
Cheers!


sending one month minus month to laravel backend

hello there
i am using this package with laravel backend whenever i am trying to catch the month in back end it's giving me one month earlier value such as .. if i send  2020-01 in back end i am getting 2019-12 how can i solve it ?

i am also getting the same problem how can i solve this problem


Fix broken URI fragment and a redirected link.

@mdo: are we OK with that new translation link? It does look very close to the official domain...


[Bug] free(): invalid pointer

🐛 Bug

To Reproduce
python3 build.py --hf-path databricks/dolly-v2-3b --quantization q3f16_0 --max-seq-len 768
Weights exist at dist/models/dolly-v2-3b, skipping download.
Using path "dist/models/dolly-v2-3b" for model "dolly-v2-3b"
Database paths: ['log_db/redpajama-3b-q4f16', 'log_db/redpajama-3b-q4f32', 'log_db/dolly-v2-3b']
Failed to detect local GPU, falling back to CPU as a target
Target configured: llvm -keys=cpu
Load cached module from dist/dolly-v2-3b-q3f16_0/mod_cache_before_build_cpu.pkl and skip tracing. You can use --use-cache=0 to retrace
Dump static shape TIR to dist/dolly-v2-3b-q3f16_0/debug/mod_tir_static.py
Dump dynamic shape TIR to dist/dolly-v2-3b-q3f16_0/debug/mod_tir_dynamic.py
Finish exporting to dist/dolly-v2-3b-q3f16_0/dolly-v2-3b-q3f16_0-cpu.so
Finish exporting chat config to dist/dolly-v2-3b-q3f16_0/params/mlc-chat-config.json
free(): invalid pointer
Aborted
Steps to reproduce the behavior:

Expected behavior

Environment

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA):
Operating system (e.g. Ubuntu/Windows/MacOS/...): ubuntu
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...)  x86
How you installed MLC-LLM (conda, source): source
How you installed TVM-Unity (pip, source): source
Python version (e.g. 3.10):  3.10
GPU driver version (if applicable): no GPU
CUDA/cuDNN version (if applicable):
TVM Unity Hash Tag (python -c "import tvm; print('\n'.join(f'{k}: {v}' for k, v in tvm.support.libinfo().items()))", applicable if you compile models):
Any other relevant information:

Additional context

Actually, this error message will not influence (though annoying) the model building and you have already finished it.
The reason you get this is that you haven't hidden LLVM symbol when compiling TVM Unity and there is some symbol conflict when a python program imports both tvm and other packages such torch that also depends on LLVM.
You cou can install our prebuilt wheels where we already hide symbols.
If you build from source, you should append these two lines in the config.cmake:
set(USE_LLVM "llvm-config --ignore-libllvm --link-static")
set(HIDE_PRIVATE_SYMBOLS ON)

We have fixed our docstring #275 accordingly.


Tentatively restrict pytest-timeout version to <1.3.0

pytest-timeout>=1.3.0 requires pytest>=3.6.

Jenkins CI test succeeded without errors!

LGTM!

Travis's python image contains pytest (3.3.0 as of today).
Travis pip cache was irrelevant.
Refs:

https://docs.travis-ci.com/user/reference/trusty/#Pre-installed-pip-packages

https://travis-ci.org/kmaehashi/sandbox-travis/builds/392105025#L433

https://docs.travis-ci.com/user/caching/#pip-cache


Something is very wrong with inspection warning placement

This is happening everywhere in LaTeX files:

TeXiFy 0.4, PhpStorm.

The warnings say 'The referenced label has not been defined.'

This means that there is no fig:employees label defined in your LaTeX document. Could you check that this is the case?

Oh nvm, they are not placed correctly are they? I have seen it happen multiple times before but I don't know what exactly causes it.

It's actually right above that text:

And it's also with warnings in strange places. The point is not the false positive warning — it's that the warnings are all over the text in random places.

See message above :3

Just installed the extension, I have latest PhpStorm. macOS. Just created a new project (folder) with just one .tex file, it's the same:

This is very odd. I don't know if you have IntelliJ installed. If so, could you try to reproduce it in IJ?
We did change some code with the UnresolvedReference inspection, so b0.5 might solve this issue. Can't say for sure though.

The last example you gave is the NonBreakingSpace inspection, btw. That one works as intended. Try using a wrong label.

Aha. Good inspection. I'm now trying to reproduce it, but it seems that the problem is in using multiple files and /input

It looks like the warnings are global for all files. So if a warning is in one .tex file of the project in a certain location, it will show up in all files at the same locations (char offset and length).

Could be. Not sure though. I have changed something with inspection offset (namely only underlining the label name, not the whole command) so it might already be resolved.
If you are able to produce a valid test case be sure to let us know so we can check if it's still occurring in b0.5. (Or build it yourself :P)

Here is the sample project. Warnings from one file are seen in another file. Looks like something is static :)
Archive.zip

Thank you :D Will look at it once I'm home.

Can confirm. It is still broken b0.5. I will investigate it further, and hopefully provide a fix soon :)

It was as simple as changing commandsInFileSet() to commandsInFile(). Will be fixed in the upcoming release! (See also PR #114)

Np :)


Add YAML

YAML syntax highlighting would be really nice.
Awesome lib, btw! Thanks so much.

isagalev,
Here is some info, let me know if I can do anything else to help. Thank you
http://docs.ansible.com/YAMLSyntax.html
https://github.com/clifford-github/sublime-ansible

@wwi-johnbuhay new languages in highlight.js don't happen this way, here's how: http://highlightjs.readthedocs.org/en/latest/language-requests.html

This issue server as at least an expression of interest in case a
contributor wants to submit a language.
On Sun, May 17, 2015 at 3:49 PM, Ivan Sagalaev<EMAIL_ADDRESS>wrote:

@wwi-johnbuhay https://github.com/wwi-johnbuhay new languages in
highlight.js don't happen this way, here's how:
http://highlightjs.readthedocs.org/en/latest/language-requests.html
—
Reply to this email directly or view it on GitHub
https://github.com/isagalaev/highlight.js/issues/119#issuecomment-102733008
.


Expose Database for external applications

Database should be exposed via nginx-proxy so applications can access using domain

This is not something defined in this repo (Can be done in docker-compose.yml)


Put all json files in one folder

looks messy and unorganized

Ah fair point, but package.json and package-lock.json are files used by NodeJS. I'm not sure if the project would work if I move it to a different directory and I don't wanna experiment with that
But I could move constants.json to somewhere else. Maybe to utility/data/constants.json or something like that

Ah fair point, but package.json and package-lock.json are files used by NodeJS. I'm not sure if the project would work if I move it to a different directory and I don't wanna experiment with that But I could move constants.json to somewhere else. Maybe to utility/data/constants.json or something like that

make a copy of this repo then experiment with it

Ah fair point, but package.json and package-lock.json are files used by NodeJS. I'm not sure if the project would work if I move it to a different directory and I don't wanna experiment with that But I could move constants.json to somewhere else. Maybe to utility/data/constants.json or something like that

make a copy of this repo then experiment with it

like copy this files you have (all of islebot) then experiment with it

Ah fair point, but package.json and package-lock.json are files used by NodeJS. I'm not sure if the project would work if I move it to a different directory and I don't wanna experiment with that But I could move constants.json to somewhere else. Maybe to utility/data/constants.json or something like that

make a copy of this repo then experiment with it

like copy this files you have (all of islebot) then experiment with it

Yeah I'll do it sometime this week

falbbers


Creature for EaselJS

Hi all!
I'm starting with HTML5 integrations. Specifically, I started with EaselJS. I have chosen this engine because it is very easy to import elements from Adobe Flash.
My question is whether you intend to export to this graphics engine. I tried to read the Phaser and Trree exporters, but I understand all the proccess to make my own runtime.
Best regards!

Hello,
I took at look at EasIJS:
http://www.createjs.com/docs/easeljs/modules/EaselJS.html
It does not seem to have WebGL or Mesh support. Creature is a mesh based animation system so I am not sure if this will run.
Cheers


Add like function.

This function will be used to implement the LIKE operator in presto_cpp.
The LIKE operator is used in TPC-H queries 14 and 16.
Note: This version is for early feedback. I'm working on changing the null handling and more detailed tests.
Fixes #208

Have we looked at existing implementations to see if we can reuse one of them, e.g. https://github.com/duckdb/duckdb/blob/master/src/function/scalar/string/like.cpp seems fairly complete?

Have we looked at existing implementations to see if we can reuse one of them, e.g. https://github.com/duckdb/duckdb/blob/master/src/function/scalar/string/like.cpp seems fairly complete?

DuckDB has custom implementation of LIKE operator. They have considered switching RE2, but found that RE2 is 2-4x slower: https://github.com/duckdb/duckdb/issues/157
select count(*) from lineitem where l_shipinstruct LIKE '%DE%I%ER%';
select count(*) from lineitem where l_shipinstruct SIMILAR TO '.*DE.*I.*ER.*';

1 thread
RE2: 0.64s
LIKE: 0.2s

4 threads:
RE2: 0.17s
LIKE: 0.06s

select count(*) from lineitem where l_comment NOT SIMILAR TO '.*str.*';
select count(*) from lineitem where l_comment NOT LIKE '%str%';

1 thread
RE2: 0.75s
LIKE: 0.23s

4 threads:
RE2: 0.23s
LIKE: 0.06s

Hi Masha,
Let's sync on this Friday if you have time. I'll set something up.


Playwright: Multiple Tests via xml Testsuite error to run the suite

Discussed in https://github.com/microsoft/playwright/discussions/18343

Originally posted by yeassine October 26, 2022
Hi,
im facing problem with xml test suite using TestNG and Playwhright, i try for example to start following test suite:
suite name="test suite" parallel="false">
parameter name="ReportValue" value="Rep"/>
test name="test1">
classes>
class name="path.test1"/>
/classes>
/test>
test name="test2">
classes>
class name="path.test2"/>
/classes>
/test>
/suite'
i call in every test script @Aftertest annotation with teardown method:
public static void tearDown() throws IOException {
context.close();
browser.close();
playwright.close();
}
and in the annotation @BeforeTest i lunch the startBrowser() method with context and playwhright browser.
Problem:
the first test1 is running without problem
seconde test is opening the browser but i get the error:
com.microsoft.playwright.PlaywrightException: Error {
message='Target page, context or browser has been closed
please can someone help me to find out what the problem is?

Usually you keep Playwright running for your whole test execution but only re-create the context and by that the page for each test.
We don't have a dedicated section for TestNG but it should be very similar to the JUnit section: https://playwright.dev/java/docs/test-runners#junit

i call in every test script @Aftertest annotation with teardown method:

public static void tearDown() throws IOException {
  context.close();
  browser.close();
  playwright.close();
}

Normally you don't need to close browser and playwright between the tests, closing the context should be sufficient. It might be that you close the context but don't create a new page in the next test. Can you share the entire test so with setup/teardown fixtures so that we can better help you?

@yury-s  and @mxschmitt  thanks for your answer :).
the tests are like this:
public class Test1{
@BeforeTest
public static void lunchBrowser(){
lunch URL with the page and context, i call here another base class.
Baseclass.starturl(URL);
}
@Test
public static void test1{
starting the test scripts
}
@Aftertest{
i call teardown() from the same base class
Baseclass().teardown();
}
}
test2 is also on separate class,
normally the tearDown close the browser and playwright and also the context, but in test2 i open all of them new int "lunchBrowser()" base method
Running the tests separate works without any problems, but if i will run the test suite it works on test1 but crash on the test2, 3 etc...

We need too see Baseclass too, basically all the code that leads to paywright calls, otherwise we can only speculate about what's going on.

here is my baseclass:
public static void initBrowser(String environmentURL) throws IOException, InterruptedException, AWTException {
		PWBaseConfigKredit.environmentUrl = environmentURL;
		playwright = Playwright.create();

	if (PWBaseConfigKredit.browserType.equalsIgnoreCase("chrome")) {
		browser = playwright.chromium().launch(new BrowserType.LaunchOptions()
				.setHeadless(PWBaseConfigKredit.headless)
				.setChannel("chrome"));

	} else if (PWBaseConfigKredit.browserType.equalsIgnoreCase("firefox")) {
		browser = playwright.firefox().launch(new BrowserType.LaunchOptions().setHeadless(PWBaseConfigKredit.headless));
	} else {
		browser = playwright.webkit().launch(new BrowserType.LaunchOptions().setHeadless(PWBaseConfigKredit.headless));
	}
	Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
	int width = (int)screenSize.getWidth();
	int height = (int)screenSize.getHeight();

	context = browser.newContext(new Browser.NewContextOptions().setViewportSize(width,height));
	page = context.newPage();
	page.setDefaultTimeout(PWBaseConfigKredit.MyworkspaceDefaultTime);

		page.navigate(PWBaseConfigKredit.url);

	}
	}

public static void tearDown() throws IOException {

		PWBaseReportKredit.createReportSeperator();
		PWBaseReportKredit.softAssertion.assertAll();

		context.close();
		browser.close();
		playwright.close();

}

It's unclear from the snippet what Baseclass.starturl(URL); does and where the page, browser etc. variables are stored and when they are reset. Can you just share entire class?

this is the entire class, that Baseclass.Start(URL) was only example, normally i call the initbrowser(URL); in the test1 and test2, and into PWBaseConfigKredit.url is only config class wich has different urls what i test and some configurations...

We need a reproducible example that we could run locally to make this report actionable. Please file a new issue with a full repro and link it to this one.


Can't edit files that were included or read by php

I made small system that allows you to chose which page to edit using php. First i used include 'file'; then i saw i cant edit them so i went with other method which reads the whole file and places it in webpage. Now i have yellow box around it which is indicator that is editable. But it's not. So i totally removed editable region from main page and placed it in function where php is "including" desired files. Same result.

I would really appreciate if someone responds to this issue.

Can you Post the HTML that's generated by your PHP application that can't be edited?

`

123

`
This is inside the file I'm including with php.
$content = file_get_contents('../assets/themes/default_theme/extensions/pages/html/'.$file.'.html'); echo '
'.$content.'
';
And this is a php script that loads the file.

Sorry so the HTML has no editable regions so that's why the editor wont show, is this the HTML output by the PHP? If so why isn't it like:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title></title>

1

2

3

The output from php is:

1

2

3

```

First time i did as you in the code you provided and i didn't even had yellow indicator that it is editable.

OK I don't know why the PHP isn't including the
 after the first time, but lets try to figure out why you can't edit the page event when it contains the editable region. I assuming you've sent me a simplified version of the page as there's no JS or CSS includes in the template, could you send through or post some where the full HTML output and the JS you have for initialising the editor.

This is the function that loads chosen page it's on line 305.
This is the page where content tools is.

Cool ty - OK since I can't run this myself to test can I ask you do the following and let me know:

After refreshing the page can you send me:

Console errors
An exact copy of the outputted HTML

Can you add a console.log(document.querySelectorAll('*[data-editable]')) on line 7 of editor.js and send me the output.

If I can't work it out from this maybe we could catch up on discord or something and you can screen share so I can work through it with you?

Thi is the output of HTML

1

2

3

No errors on console.
And there is no output.

OK so that looks like some how the editable region
...
 included the section tag in it and so the editor interpreted that as a static element (one it can't edit) and that's why it has class="ce-element ce-element--type-static" against it. However, class="ce-element ce-element--type-static" would only be applied when you edit the page those classes are only applied when you click to edit and CT parses the page content) so I must be missing a step here.
If you don't use PHP to drag in data dynamically and you replace <?php PageEditorLoad($conn); ?> with

Testing

Are you able to edit the heading in this case?

Yes i'm able to edit after i replaced <?php PageEditorLoad($conn); ?>  with `
Testing
` should i somehow exclude `` from files im trying to load? Also am i able to edit if i have `

Testing

`

ContentTools only handles a subset of tag/element types for editing. You can have multiple editable regions in a page and these can be any block level element pretty much, but they can only contain elements CT supports, unsupported element types will still appear but will be marked as static and you wont be able to edit them or their contents, the following is OK for example:

Testing

I see so there is no way i can edit something that is in this format.

Testing

So not using that structure no. We can typically work around these restrictions (the CT site for example is all done with CT as is) but form more complex sites we use the ContentFlow library - however - this isn't documented so it's hard to get started with I'm afraid.
Sorry I couldn't be more helpful :

Well can i implement it in my project? Do i need to have node.js or something that needs to be installed by git?

Both ContentTools and ContentFlow are client side libraries, neither require node, any backend you want to use is fine - though it's quite a bit of work to set up support for ContentFlow currently, this is something I hope to improve in future releases.

Okay thanks for the info i will keep my eyes on ContentFlow!


[Feature request] Power loader but for text/string

Lora Power Loader node is absolutely fantastic and I instantly imagned that the same node format could also work for loading strings.
Multiple nodes chained together, each for different thing (style, lightning, perspective, people, hair, clothes, background etc.) with multiple toggable text that later can be combined into a cohesive prompt.
Something like this:

Giving force to this idea.


Update latest translation

funny how a lot of languages just add like 200% characters when you translate something

German terms in particular are often much longer than English ones. but i thought swedish terms are pretty long too


Disable clicking outside of last button

It seems the NSClipView visible right of the last button reacts to mouse clicks as well. This visually indicates you can interact with it, which you don't.
(I'm looking into this and try to fix it myself, but just to let you know I work on this.)

Thanks for the bug report! I'm focused on the rewrite of KPCTabsControl in Swift. But I'll include the bugfix anyway!

Swift sounds great :) If you make the to-do list part of an issue here, I'd like to help 👍

Good idea! I'll do it.

You can check out the to-do list for v2.0 here: #9


[0.8.x] GUVNOR-2430: Delete popup: Remove confirmation check

See https://issues.jboss.org/browse/GUVNOR-2430
Corollary of https://github.com/uberfire/uberfire-extensions/pull/215 for 0.8.x
(cherry picked from commit fc5bc6c04b33f11ad6f260113d71246b0f728bc8)

+1

Merged into 0.8.x.


Backport script handles backport-done label

Two backport script fixups

handle the case where "stable/backport-done" is left on the PR. This saves me from opening every PR to verify what happened.
Print a link to the PR in the console. This helps open the relevant PR when I need to verify what happened.

Amusingly, this should be backported
How to test (optional):
/contrib/backporting$ ./check_stable

test-me-please

test-me-please

backport in https://github.com/cilium/cilium/pull/3771
@eloycoto I added new commits. You may wish to re-review (but you approved these commits 3 minutes ago).


Configuration Issues

Hi, I found it difficult to install and configure using the guide in the readme file. Simply posting this if someone else runs into these issues and needs to know how to use;
module.exports = {
  theme: {
    pagination: theme => ({
      color: theme('colors.teal.500'),
      link: 'py-2 px-3 bg-white block border-t border-b',
      wrapper: 'justify-start flex my-10',
      item: 'block',
      linkFirst: 'border-l border-r bg-gray-200 rounded-l-sm',
      linkLast: 'border-r border-l bg-gray-200 rounded-r-sm',
    })
  },
  variants: {},
  plugins: [
    require('tailwindcss-plugins/pagination')
  ]
}

Hi there, thanks for that! 👌
As mentioned in a more recent issue I had forgotten to update the documentation for the pagination plugin when I upgraded them to TailwindCSS 1.x. The documentation should now be up-to-date. Thanks again! 🍺


include the Capital Gain data in actions

For mutual funds, a lot of the times the action is "Captial Gain". Can you please include the "capital gain" column in the history data?
We get the capital gain data by use params["events"] = "capitalGain,div,splits"  instead of "div,splits".  Then the post processing of the captial gain data should be the same as dividends data.

+1 for this feature, it would be very useful. Thank you!

@thirumalairajr this request might be interesting to you

@ValueRaider Sure, I have raised a PR https://github.com/ranaroussi/yfinance/pull/1166 for this, can you please review it ?

@thirumalairajr I have already reviewed. Also this request is different, suggesting to modify Ticker.history(), maybe it returns different (better?) capital gains data.

Yeah I have raised a new PR https://github.com/ranaroussi/yfinance/pull/1166 in which I have added Capital Gains column to Ticker.history() * Ticker.actions dataframes. I have also added a new Ticker.capital_gains property for getting Capital Gains data as a series.
Are these changes fine or am I missing something?


Pre-receive hook that checks for Jira Issue

This pre-receive hook checks that a Jira issue is present in the commit message. The regex pattern I used is /*\[jira-.*\] which would match [jira-1234], for example.
/cc @webdog resident regex guru
/cc @MikeNwin who had the 💡 to add this
/cc @osowskit who helped me on this
/cc @github/solutions-engineering bc ❤
Would love all ya'll's feedback on this 📣🐦

Nice!
I'd suggest using '.+?' compared to '.*' as the latter is a greedy
capture and may give you unintended behavior if it captures a similar
string.
Try using a regex parser to visually see the difference with a really long
string that includes your capture parameters to see the difference 👍👍👍
On Thu, Apr 6, 2017 at 10:30 AM Pavan Ravipati<EMAIL_ADDRESS>wrote:

This pre-receive hook checks that a Jira issue is present in the commit
message. The regex pattern I used is /[jira-.] which would match
[jira-1234], for example.
/cc @webdog https://github.com/webdog resident regex guru
/cc @MikeNwin https://github.com/MikeNwin who had the 💡 to add this
/cc @osowskit https://github.com/osowskit who helped me on this
/cc @github/solutions-engineering
https://github.com/orgs/github/teams/solutions-engineering bc ❤
Would love all ya'll's feedback on this 📣🐦
You can view, comment on, or merge this pull request online at:
https://github.com/github/platform-samples/pull/135
Commit Summary

Create require-jira-issue.sh

File Changes

A pre-receive-hooks/require-jira-issue.sh
https://github.com/github/platform-samples/pull/135/files#diff-0
(29)

Patch Links:

https://github.com/github/platform-samples/pull/135.patch
https://github.com/github/platform-samples/pull/135.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/platform-samples/pull/135, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6m0oK6oQZ8kI7mhuiEYx20tdBwrHBiks5rtQURgaJpZM4M1xQo
.

--
| Christian Weber | Solutions Engineer | GitHub |
<EMAIL_ADDRESS>|<PHONE_NUMBER>

If you do a merge via the WebUI you would need to include a JIRA Ticket in the merge. I had requests from users to exclude those merges from the hook. However, if merges were done on the command line they wanted to check those.
As a result, you might need to add in:
if [[ "${GITHUB_VIA}" == "pull request merge button" ]]
then
  exit 0
fi

I also have a similar hook to pick up JIRA issues. Not sure how efficient it is, but it works:
#!/usr/bin/env bash
#
# Pre-receive hook that will reject all pushes that do not start with the Prefix JIRA
# Set the prefix on line below
PREFIX=JIRA-

if [[ "${GITHUB_VIA}" == "pull request merge button" ]]
then
  exit 0
fi

read oldrev newrev refname

for COMMIT in `git rev-list --no-merges $newrev | git name-rev --stdin | grep -v "("`
do
  REF=`git log --format=%B -n 1 ${COMMIT}`
  if echo "${REF}" | grep "^${PREFIX}[0-9]"
  then
    #echo -e "No problem with commit ${COMMIT}"
    DUMMY=y
  else
    echo -e "[*** ERROR: A valid JIRA Issue number is missing from commit message... format='${PREFIX}123 ...' ***]"
    echo -e "Problem is with commit ${COMMIT}"
    # We don't want to display all of the failures so leave straight away.
    exit 1
  fi
done

exit 0

We include information on the commit, as that is what developers wanted. Some also wanted a list of the commits that did not produce problems, hence the line that is commented out.

I'd suggest using '.+?' compared to '.*' as the latter is a greedy capture and may give you unintended behavior if it captures a similar string.

IIRC, Bash uses regex(3) and not a PCRE, so it doesn't have a non-greedy option.
# Greedy
bash-3.2$ [[ "[jira-124] [jira-1234] [jira-12345]" =~ \[jira-.*\] ]]; echo "$?: ${BASH_REMATCH}"
0: [jira-124] [jira-1234] [jira-12345]    # exit 0 but the greedy capture

# Non-greedy try
bash-3.2$ [[ "[jira-124] [jira-1234] [jira-12345]" =~ \[jira-.+?\] ]]; echo "$?: ${BASH_REMATCH}"
2: [jira-124] [jira-1234] [jira-12345]    # Exit 2 is malformed regex

the following might be a better match for numbers (caveat emptor): \[jira-[0-9]+\]
# \[jira-[0-9]+\]
bash-3.2$ [[ "[jira-124] [jira-1234] [jira-12345]" =~ \[jira-[0-9]+\] ]]; echo "$?: ${BASH_REMATCH}"
0: [jira-124]

Hi folks 👋  -- I wanted to mention this alternative version of a JIRA commit message check. This one has a loop to check through all commits, which may address the issue reported by @mzzmjd above https://github.com/github/platform-samples/pull/135#issuecomment-292318716
#!/bin/bash
#
# check commit messages for JIRA issue numbers formatted as [JIRA-<issue number>]

REGEX="\[JIRA\-[0-9]*\]"

ERROR_MSG="[POLICY] The commit doesn't reference a JIRA issue"

while read OLDREV NEWREV REFNAME ; do
  for COMMIT in `git rev-list $OLDREV..$NEWREV`;
  do
    MESSAGE=`git cat-file commit $COMMIT | sed '1,/^$/d'`
    if ! echo $MESSAGE | grep -iqE "$REGEX"; then
      echo "$ERROR_MSG: $MESSAGE" >&2
      exit 1
    fi
  done
done
exit 0

👏  @djdefi I did a bit of testing from my command line and that script seems to have done the trick. The remote rejected git push when a ref was missing a jira ticket from the commit message 👇

After rebase-ing and adding a jira ticket to the failing commit message, my refs passed the pre-rx hook 👇


Hide fields not allowed in security

I think maybe it makes sense to move all security related stuff to a separate class. WDYT ?
Implemented.


DOC: Adding Mentors in the Mentors Section of readme.MD

What's wrong with the existing documentation
The mentioning of mentors of this project is important in the README as they hold an important position and have quite
a prominent role in this project.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

@MonalikaPatnaik The repository of Umatter which I forked, is no more available in my repositories. I was again trying to search for your repo but it's no more available on GSSoC Project list?? What do I do now??? How am I supposed to send my PR?


Generate @Deprecated annotations for configuration accessors in Kotlin DSL

Follow up to: #8585

Maybe we can also offer a ReplaceWith.

Work suspended as causing unexpected feedback in the IDE - see #10794


ihanhyvareittiopas.fi is down

"Server not found."
This also says it's down: http://www.downforeveryoneorjustme.com/www.ihanhyvareittiopas.fi
nslookup ihanhyvareittiopas.fi
Server:     <IP_ADDRESS>
Address:    <IP_ADDRESS>#53

** server can't find ihanhyvareittiopas.fi: NXDOMAIN

Seems it's been down a long time. Too bad, I absolutely loved using this service!

Same here! Hopefully the author will fix it, but on the other hand who knows... Maybe one of us fans could re-host it. I could try it, especially if the author hasn't got time for it. Lets see.

Closing as stale.


[SPARK-44957][PYTHON][SQL][TESTS] Make PySpark (pyspark-sql module) tests passing without any dependency

What changes were proposed in this pull request?
This PR proposes to fix the tests to properly run or skip when there aren't optional dependencies installed.
Why are the changes needed?
Currently, it fails as below:
Running PySpark tests. Output is in /.../spark/python/unit-tests.log
Will test against the following Python executables: ['python3']
Will test the following Python modules: ['pyspark-sql']
python3 python_implementation is CPython
python3 version is: Python 3.10.12
Starting test(python3): pyspark.sql.tests.pandas.test_pandas_grouped_map_with_state (temp output: /.../spark/python/target/8e530108-4d5e-46e4-88fb-8f0dfb7b47e2/python3__pyspark.sql.tests.pandas.test_pandas_grouped_map_with_state__jggatex7.log)
Starting test(python3): pyspark.sql.tests.pandas.test_pandas_grouped_map (temp output: /.../spark/python/target/3b6e9e5a-c479-408c-9365-8286330e8e7c/python3__pyspark.sql.tests.pandas.test_pandas_grouped_map__1lrovmur.log)
Starting test(python3): pyspark.sql.tests.pandas.test_pandas_cogrouped_map (temp output: /.../spark/python/target/68c7cf56-ed7a-453e-8d6d-3a0eb519d997/python3__pyspark.sql.tests.pandas.test_pandas_cogrouped_map__sw2875dr.log)
Starting test(python3): pyspark.sql.tests.pandas.test_pandas_map (temp output: /.../spark/python/target/90712186-a104-4491-ae0d-2b5ab973991b/python3__pyspark.sql.tests.pandas.test_pandas_map__ysp4911q.log)
Traceback (most recent call last):
  File "/.../miniconda3/envs/vanilla-3.10/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/.../miniconda3/envs/vanilla-3.10/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/.../workspace/forked/spark/python/pyspark/sql/tests/pandas/test_pandas_map.py", line 27, in <module>
    from pyspark.testing.sqlutils import (
  File "/.../workspace/forked/spark/python/pyspark/testing/__init__.py", line 19, in <module>
    from pyspark.testing.pandasutils import assertPandasOnSparkEqual
  File "/.../workspace/forked/spark/python/pyspark/testing/pandasutils.py", line 22, in <module>
    import pandas as pd
ModuleNotFoundError: No module named 'pandas'

PySpark tests should pass without optional dependencies.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Manually ran as described above.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master and branch-3.5 (as it fixes testing framework too)


[code-completion] Fix a crash on invalid operator decls

We cache failures in the operator maps, so handle null pointers when
walking them for code-completion.
rdar://problem/29275272

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test OS X platform

Looks like earlier tests probably encountered sporadic failures?
@swift-ci please smoke test and merge

@jtbandes thanks!