concurrency issue: Internal Error occured.

It doesn't do it every time but running the following spek sometimes causes the below NPE.
class ParallelSpek : Spek({
    given("some parallel operations") {
        // uses `parallelStream` from `kotlinx-support-jdk8-0.1`
        listOf(1, 2, 3, 4, 5).parallelStream().forEach { int ->
            on("execution") {
                it("should something") {}
                it("should something else") { fail() }
                it("should yet something else") { check(false) }
            }
        }
    }
})

Internal Error occured.
java.lang.NullPointerException
	at java.util.LinkedList$ListItr.next(LinkedList.java:893)
	at org.jetbrains.spek.junit.JUnitDescriptionCache.create(JUnitDescriptionCache.kt:45)
	at org.jetbrains.spek.junit.JUnitDescriptionCache.get(JUnitDescriptionCache.kt:20)
	at org.jetbrains.spek.junit.JUnitDescriptionCache.create(JUnitDescriptionCache.kt:30)
	at org.jetbrains.spek.junit.JUnitDescriptionCache.get(JUnitDescriptionCache.kt:20)
	at org.jetbrains.spek.junit.JUnitSpekRunner.getDescription(JUnitSpekRunner.kt:23)
	at com.intellij.junit4.JUnit4IdeaTestRunner.getDescription(JUnit4IdeaTestRunner.java:123)
	at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:99)
	at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:42)
	at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:234)
	at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:74)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:144)

In addition, if the Internal Error does not occur sometimes all of the on blocks do not execute.
I'm loving what I see so far with Spek but I do need to be able to run some of the execution for testing in parallel and I would like to avoid synchronizing DescribeBody method calls.
Thanks.

What's the use case for this? Spek going some significant changes before 1.0, this may or may not be applicable when those changes are applied.

Can't you move multithreading to it("something") so it'll live inside of the test?
Also, I'm not sure that JUnit runner will wait for such tests without explicit blocking inside of the tests.

If I move the multi-threading to it("something") then I will have multiple asserts in a single test.
JUnitSpekRunner doesn't have, from what I can tell, any kind of timeout on describing the test. The time for the tests is only the time spent asserting things and not the total execution time as the tests don't actually start until after all of the HTTP requests are finished. This is typical though in time statistics for integration tests.
Basically, I want to do a whole bunch of work in parallel and schedule assertions while I go and then execute all of the assertions at the end and report.

I guess the problem here is Spek not supporting parallel executions of tests (it, on etc.. blocks), it will definitely happen sometime in the near future.


Improve error messages for RowToList type classes

Environment

PureScript 0.12
Halogen 4.0.0
purescript-halogen-formless 0.1.0

Current behavior
When Formless isn't able to solve one of its various RowToList classes, an extraordinarily cryptic error results (noted by @Unisay)
  No type class instance was found for

    Prim.RowList.RowToList t4
                           t5

  The instance head contains unknown type variables. Consider adding a type annotation.

while checking that type forall pq cq cs form out m spec specxs field fieldxs output countxs count inputs inputsxs.
                           Ord cs => Monad m => RowToList spec specxs => RowToList field fieldxs => RowToList count countxs => RowToList inputs inputsxs => EqRecord inputsxs inputs => FormSpecToInputField specxs spec field => InputFieldsToInput fieldxs field inputs => SetInputFieldsTouched fieldxs field field => InputFieldToMaybeOutput fieldxs field output => CountErrors fieldxs field count => AllTouched fieldxs field => SumRecord countxs count (Additive Int) => Newtype (form FormSpec) { | spec } => Newtype (form InputField) { | field } => Newtype (form OutputField) { | output } => Newtype (form Input) { | inputs } => Component HTML (Query pq cq cs form out m)
                                                                                { validator :: form InputField -> m (form InputField)
                                                                                , submitter :: form OutputField -> m out
                                                                                , formSpec :: form FormSpec
                                                                                , render :: { validity :: ValidStatus
                                                                                , dirty :: Boolean
                                                                                , submitting :: Boolean
                                                                                , errors :: Int
                                                                                , submitAttempts :: Int
                                                                                , form :: form InputField
                                                                                , internal :: InternalState form out m
                                                                                }
                                                                                -> HTML (ComponentSlot HTML cq m cs (Query pq cq cs form out m Unit)) (Query pq cq cs form out m Unit)
                                                                                }
                                                                                (Message pq form out)
                                                                                m
  is at least as general as type Component HTML t0 t1 t2 t3
while checking that expression component
  has type Component HTML t0 t1 t2 t3
in value declaration render

where t0 is an unknown type
      t2 is an unknown type
      t1 is an unknown type
      t3 is an unknown type
      t5 is an unknown type
      t4 is an unknown type

See https://github.com/purescript/documentation/blob/master/errors/NoInstanceFound.md for more information,
or to contribute content related to this error.

Users of the library have essentially no information to work from as to how to solve this error. There are some methods which are useful:

Try to eliminate as many polymorphic variables as you can (test out making everything concrete)
Make sure all the individual inputs to Formless compile (submitter, validator, etc.)
Use typed holes on parts of the input to see if you can narrow the error to a particular input

But the error message says nothing about this, nor is it obvious to users that they ought to do this.
Expected behavior
RowToList classes ought to have more informative error messages. @Unisay recommended custom type errors as written about here:
https://github.com/paf31/24-days-of-purescript-2016/blob/master/21.markdown
Why not apply these liberally to the type classes in order to help narrow down what to do about these errors?

Unfortunately this error shows up in the constraints for the component itself, so there may only be that location with which to provide the error.

I have created a tiny project that isolates aforementioned compilation error:
formless-test.zip

@Unisay I've written up a more detailed description of your error in another issue, #16. I'll leave both issues open until I've added better documentation for this error along with a custom error message for the component.

I'd like to make this better using the new Warn / Fail type class work here, but am not sure how to properly identify this particular error and provide a useful error for each type class. Still working on it!
https://github.com/purescript/purescript/issues/3134

Unfortunately, unless someone comes along and corrects me, it looks like even adding a Fail fallback to the type classes used by Formless will not help solve issues when the rows themselves can't be solved; the compiler never makes it past the RowToList constraint. So I'm closing this but will ensure it's discussed in documentation.


fix(console): set options, idp layout

Partial of #5654

### Definition of Ready

- [ ] I am happy with the code
- [ ] Short description of the feature/issue is added in the pr description
- [ ] PR is linked to the corresponding user story
- [ ] Acceptance criteria are met
- [ ] All open todos and follow ups are defined in a new ticket and justified
- [ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
- [ ] No debug or dead code
- [ ] Critical parts are tested automatically
- [ ] Where possible E2E tests are implemented
- [ ] Documentation/examples are up-to-date
- [ ] All non-functional requirements are met
- [ ] Functionality of the acceptance criteria is checked manually on the dev system.

and if it's not too much work please align the naming of the azure tenant types with their wording. customer should be consumer

@peintnermax and the tenant id field is always shown. It should only be shown as soon as the undefined type is selected


The method resetValidationError is not supported for this object (IE11)

Also when promise.js is included I got this error when doing an AJAX call with a input (Ajax request example at the homepage of sweetalert2).
Also, when I click the OK or cancel button, nothing happends.
Could you, please in the next release, get rid of promise.js and fix this issue?
I am using IE 11.0.96 with the latest swal2.

Can't reproduce. Please make jsfille and reopen.


Duplicate Type attribute when using ElmLibraryWriter to generate JSON from Library

When using clinical_quality_language ElmLibraryWriter.java ( ./Src/java/elm/src/main/java/org/cqframework/cql/elm/serializing/ElmLibraryWriter.java ) to generate JSON from a Library that was created with org.cqframework.cql.cql2elm.CqlTranslator
In version 3.3.2, the resulting JSON was correct (ie., could be parsed by org.json.JSONObject). When using version 3.11.0, the resulting JSON is incorrect and yields the message  "Duplicate key 'type' encountered at line 308 column 20" ( Below, you can see the two "type" definitions under localId 263
"resultTypeSpecifier" : {
               "localId" : "262",
               "type" : "ListTypeSpecifier",
               "elementType" : {
                  "localId" : "263",
                  "type" : "ChoiceTypeSpecifier",
                  "type" : [ ],
                  "choice" : [ {
                     "localId" : "264",
                     "name" : "{urn:healthit-gov:qdm:v5_6}PositiveEncounterPerformed",
                     "type" : "NamedTypeSpecifier"
                  }, {
                     "localId" : "265",
                     "name" : "{urn:healthit-gov:qdm:v5_6}PositiveInterventionPerformed
[3.3.2-LibraryJson.json](https://github.com/user-attachments/files/16144388/3.3.2-LibraryJson.json)
[3.11.0-LibraryJson.json](https://github.com/user-attachments/files/16144389/3.11.0-LibraryJson.json)
[3.3.2-LibraryJson.json](https://github.com/user-attachments/files/16144406/3.3.2-LibraryJson.json)
[3.11.0-LibraryJson.json](https://github.com/user-attachments/files/16144407/3.11.0-LibraryJson.json)
",
                     "type" : "NamedTypeSpecifier"
                  } ]
               }
            },

I haven't determined whether the problem is with the translator, or the Writer.
The error seems to occur when CQL contains UNION
library MAT6463GenerateElmJSON version '2.0.000'

using QDM version '5.6'

parameter "Measurement Period" Interval<DateTime>

context Patient

define "IPP":
  true
define "Undefined Encounter":
 ["Encounter, Performed"]
define "Undefined Intervention":
 ["Intervention, Performed"]
 define "Union of UnDefined":
 ("Undefined Encounter" union "Undefined Intervention")

3.3.2-LibraryJson.json
3.11.0-LibraryJson.json

@hossenlopp I'm not seeing where this was mentioned in https://github.com/cqframework/cql-execution/pull/320
But it's impacting our ability to incorporate anything from cqframework 3.6.0 or greater into MADiE. Can I provide any additional information, or do any additional research to help debug this issue?

I mentioned this issue in https://github.com/cqframework/cql-execution/pull/320 since that PR updated our testing deck to be translated with cql-to-elm 3.12.0 this issue broke a handful of tests. I had to manually remove the duplicate type attributes, specifically around ChoiceTypes, in the translated ELM to get the tests to have valid data for input.
There is no way for us to create a workaround in cql-execution as it would require replacing the built in JSON.parse() function which would be an unreasonable amount of work. This fix needs to be made in the translator.

@hossenlopp Do you know if this has been submitted as a ticket to https://github.com/cqframework/clinical_quality_language? cc @brynrhodes


docs(v2): Added FireCMS to the showcase

Motivation
Added FireCMS to the showcase
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Related PRs

thanks!

Thank you!


Sync eng/common directory with azure-sdk-tools for PR 2913

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/2913  See eng/common workflow

/check-enforcer override


Final Review of IntentMaxHistoryTrackerFeaturizer

Misc. doc string edits in IntentMaxHistoryTrackerFeaturizer

https://github.com/RasaHQ/rasa/pull/8913/files#r661351580
https://github.com/RasaHQ/rasa/pull/8913/files#r662279690
https://github.com/RasaHQ/rasa/pull/8913/files#r661508961
https://github.com/RasaHQ/rasa/pull/8913/files#r662298463
https://github.com/RasaHQ/rasa/pull/8913/files#r662313726
https://github.com/RasaHQ/rasa/pull/8913/files#r661360084
https://github.com/RasaHQ/rasa/pull/8913/files#r662422407

Split IntentMaxHistoryTrackerFeaturizer into a separate module. Split the corresponding test file as well.

https://github.com/RasaHQ/rasa/pull/8913/files#r661356109

Updated UnexpecTEDIntentPolicy and its tests to import IntentMaxHistoryTrackerFeaturizer correctly after module split.

(a) create a tracker_featurizers subdirectory with an init.py that imports the different tracker featurizers or

That seems to be the neatest option. If this is an overkill then I'd maybe skip the whole "moving to separate module" for now as I think there are probably more important things to do now.

(a) create a tracker_featurizers subdirectory with an init.py that imports the different tracker featurizers or

That seems to be the neatest option. If this is an overkill then I'd maybe skip the whole "moving to separate module" for now as I think there are probably more important things to do now.

@wochinge After discussing with @dakshvar22 and @JEM-Mosig, we are going to make splitting tracker_featurizers into separate modules its own issue to be tackled later.


Add cert revocation improvement docs

Purpose
This PR contains the doc changes related to the changes introduced with the following PR,
https://github.com/wso2/wso2-synapse/pull/2152

Fixed the suggestions.


labelText doens't turn red when validating in some Form Builder fields

In my case, they are Nickname labelText and Birthday labelText.
      decoration: InputDecoration(
        labelText: 'Nickname',
        border: InputBorder.none,
        hintText: 'Choose a nickname',
        filled: true,
        floatingLabelBehavior: FloatingLabelBehavior.always,
      ),

This still a issue?

Due to lack of response and an old error, I will close this issue.
If the bug still exists, feel free to open a new issue


Colored arrows

Hi,
Do you think it could be possible to add a color value to the dependance between 2 objects to display red , green or black arrows ?

Firstly, I tried to simply modify the color of the arrows. So, in script.js, I tried to assigned 'red' to the color of each link between objects by adding this line of code :
 link.stroke = 'red;' in
 graph.links = []; for (var name in graph.data) {         var obj = graph.data[name];         for (var depIndex in obj.depends) {             var link = {                 source : graph.data[obj.depends[depIndex]],                 target : obj             };             link.strength = (link.source.linkStrength || 1)                           * (link.target.linkStrength || 1);             link.stroke = 'red';             graph.links.push(link);         }     }
but it doesn't work.
Someone has a better idea ?

I found a solution by :

removing the stroke attribute of .link in svg.css
adding a color_value in [depends]


Feature/177 create a splash screen showing when the game boots

Depends on #162

Codecov Report

Merging #187 (47a13b4) into master (ab5f859) will decrease coverage by 1.50%.
The diff coverage is 0.00%.

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


Galaxy-intro-strands tutorial format

Hi i went through the tutorial for Introduction to Genomics and Galaxy  and i observed that:

some hands on do not follow the right format . Example

incorrect image guide and icon.

i will like to work on this

Should be fixed in referenced PRs.


Cannot infer_schema! with Uuid type

Diesel version:
diesel = { version = "0.12.0", features = ["postgres"] }
diesel_codegen = { version = "0.12.0", features = ["postgres"] }

Rust: rustc 1.17.0-nightly (0aeb9c129 2017-03-15)
I have a User model with an UUID:
-- Your SQL goes here
CREATE TABLE users (
  id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
  email VARCHAR NOT NULL,
  password TEXT NOT NULL,
)

The infer_schema! macro fails everytime however:
infer_schema!("dotenv:DATABASE_URL");

error[E0412]: cannot find type `Uuid` in this scope
 --> dutrack-lib/src/db/schema.rs:3:1
  |
3 | infer_schema!("dotenv:DATABASE_URL");
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ not found in this scope
  |
  = help: possible candidate is found in another module, you can import it into scope:
            `use uuid::Uuid;`
  = note: this error originates in a macro outside of the current crate

I tried importing Uuid into the scope in db/schema.rs but this didn't help either.

Nevermind, just saw that I need to add uuid to my features in the Cargo.toml.

Hate to tag an old thread but, I DO have this:
diesel = { version = "1.3.3", features = ["postgres", "uuid"] } in my Cargo.toml and I'm seeing this problem.


Change Text for "Notification of possible encounter"

Avoid duplicates

[x] This enhancement request has not already been raised before
[x] Enhancement request is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in CWA-Wishlist
[x] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation

Currently, if bluetooth or the location service is enabled on the device, a notification pops up with the main title "notification of possible encounter" (see first Screenshot attached). After disabling bluetooth again, a notification with the same title shows a second time. The intent of the notifications is to suggest to enable the contact tracing by enabling bluetooth and location services. But due to the title and the red alert logo, every time I see this notification, for a moment I am not sure if it is a warning about me having a possible encounter with someone who is infected.

Suggested Enhancement

I propose to use a different wording for the notification, e.g. "To start tracing, turn on bluetooth and location services", like currently shown in the subtitle of the notification.
Expected Benefits

I expect the intent of the notification to be clearer to the user and thus increasing the user experience for using the app.

Thanks for this enhancement proposal. However, we already have a corresponding issue. Please follow-up in #532 We will therefore close this issue.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Unreadable dropdown.

Describe the bug
Dropdown color text is not readable in darkmode. See screenshot
To Reproduce
Steps to reproduce the behavior:

Upload the spoiler log
Click on 'dark theme' if not enabled yet.
Click items per page dropdown.
See error

Expected behavior
A color that is readable.
Screenshots

Desktop (please complete the following information):

Browser: firefox 91.0

Additional context
Maybe it can be fixed with the css:
.css-etlbv9 > option {
    background-color: #171923;
}

Can you copy the sharing link and post it here?

Spoiler log: https://ootrandomizer.com/spoilers/get?id=856540
Found the line that defined the options: https://github.com/amickael/spoilizer/blob/15c64b764a78503b1b052eede8695e270bdb7b94/src/components/ItemList/ItemList.tsx#L126

That select component is using the browser's native select. What OS are you running?

I'm using Windows 10. The CSS trick that I posted fixes it for firefox.

Looks like this is a bug with Chakra UI, the component library I'm using. Will implement fix from here: https://github.com/chakra-ui/chakra-ui/pull/1388


[AML]Split GA version workspace RP swagger to it's own file

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

ARM reviewer: this PR is to split one json file into two, there is no additional change other change move the API and definition to a new file.

/azp run

      "Operation"

Is the operations path expected to be moved also?

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2024-10-01/machineLearningServices.json:40 in 5e6c6ff. [](commit_id = 5e6c6ff2a3d8b7ac561a166d8be6c905dbfeb72b, deletion_comment = True)

      "Operation"

Is the operations path expected to be moved also?
Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2024-10-01/machineLearningServices.json:40 in 5e6c6ff. [](commit_id = 5e6c6ff, deletion_comment = True)

Yes, this you can see we already did this in the preview version, we split the APIs since they belong to two different team now.

/azp run

The required Swagger PrettierCheck fails, to be fixed

/azp run

I can't find the missing breaking items for Python in CI report so add approval label directly.


Authentication with Dashlane Authenticator

Describe the bug
Original discussion here. It seems that for some Dashlane accounts, the user is prompted to choose between Dashlane Authenticator and TOTP. Since Dashlane Authenticator is no longer supported, is this step still required ? Many thanks in advance.
To Reproduce
Steps to reproduce the behavior:
% dcli sync
? Please enter your email address<EMAIL_ADDRESS>? What second factor method would you like to use? (Use arrow keys)
❯ Dashlane Authenticator
  Totp

Expected behavior
% dcli sync
? Please enter your email address<EMAIL_ADDRESS>? Please enter your OTP code:

Environment (please complete the following information):

OS: macOS Sonoma 14.5
Version: 6.2432.0

EDIT : How does the DUO push notification method mentioned in the docs works ?

Hello,
When the user has setup multiple second factor methods the question What second factor method would you like to use? will be prompted. In the code we support up to 4 different methods (see here). One of them is the Dashlane Authenticator that has been sunset, and is unusable as app has been removed from the store. That said some user may still have the configuration option on their profile which led the API to return dashlane_authenticator as one of the potential options.
I'm going to make a PR to filter out this option manually from the results.
Concerning DUO verification, it's very similar to the Dashlane Authenticator behaviour, a call is fired to the API that transmits it to DUO servers, a notification to approve is sent to the user's phone and once validated the response goes back to the initial caller. Which means that the CLI will basically hang until it gets an answer from the API. There are very few companies using DUO setup for now, so I guess you can safely ignore this path.


Add missing OpenStack events from recent features

This PR adds missing OpenStack events that should be specified due to the recent addition of new features.

@miq-bot add_label euwe/yes

@miq-bot add_label bug

looks great :+1:

@miq-bot add_label blocker

Euwe BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1391127

Euwe Backport details:
$ git log -1
commit 38fd52bb3bb3d02bd6c1046e8a0bc5138476d9d6
Author: Greg Blomquist<EMAIL_ADDRESS>Date:   Wed Nov 2 09:55:24 2016 -0400

    Merge pull request #12267 from tzumainn/openstack-event-refresh

    Add missing OpenStack events from recent features
    (cherry picked from commit 3a0dab52827e47b46c49d473b35bfda412ed5bc2)

    https://bugzilla.redhat.com/show_bug.cgi?id=1391127


Add a backport bot.

Fix #1072
@seanpmorgan I'll send you the BOT_TOKEN by email. You don't have to set the GITHUB_TOKEN . It's automatic.
You can look at the folks of https://github.com/qgis/QGIS who are making heavy use of this bot.
You need to add a label to the PR you want to backport: e.g. https://github.com/qgis/QGIS/pull/34436
See https://github.com/marketplace/actions/backport-bot

It seems we'll need to restart the tests after the secret has been added.

@gabrieldemarmiesse  ##[error]Input required and not supplied: bot_token
Are we sure it should be a capitalized token?

Yes, because we get it with:  ${{ secrets.BOT_TOKEN }}

Yes, because we get it with:  ${{ secrets.BOT_TOKEN }}

Gotcha I follow. Not sure what this issue is then. I've added the token:

I'm going to investigate.

From a security point of view, it would seem logical for it to work only when it's merged in master.

Ok I understand what is going on.
A secret can't be used in a pull request made by a fork. As such, we got the error. To fix it, we need to publicly expose the token, this is why they recommend to give it minimal permissions (actually less than a normal github user) to discourage people with bad intentions from using it. It's what I did.
I've put in the bio of my bot to report any issue to me. If I ever get a problem because someone wanted to prank me, I'll just remove the token and we'll drop the idea of automatic backporting. To check that the bot works, we'll need to test it with a pull request (we can close the pull request of the backport if it was just to test it).
This can be merged.

Actually no, github told me that my token has been revoked because it was found in a commit. Let me find how to fix that.

Ok if someone really wants to find my token it's possible, but this token has only comment rights and PR rights (but only in addons), so I suppose that it'd be in the best interest of this person to create another bot. I'm fine with the security implications of having this token out there.
This can be merged. Finally :p


Change cert-manager installation link

Per k8s cer-manager page(https://cert-manager.io/docs/installation/kubernetes/), the installation link for k8s >1.15 should be kubectl apply --validate=false -f https://github.com/jetstack/cert-manager/releases/download/v0.15.0/cert-manager.yaml

Thanks,  the ci failed.

I get the same
manifest_sorter.go:192: info: skipping unknown hook: "crd-install"
manifest_sorter.go:192: info: skipping unknown hook: "crd-install"
With the current version as well.

I get the same
manifest_sorter.go:192: info: skipping unknown hook: "crd-install"
manifest_sorter.go:192: info: skipping unknown hook: "crd-install"
With the current version as well.

This warning is fine, ref: https://github.com/crossplane/crossplane/issues/1366


Question about No module named 'recbole.data.dataset'

Traceback (most recent call last):
File "run_recbole.py", line 48, in
run_recbole(
File "/root/AdaMCT-master/recbole/quick_start/quick_start.py", line 70, in run_recbole
dataset = create_dataset(config)
File "/root/AdaMCT-master/recbole/data/utils.py", line 40, in create_dataset
dataset_module = importlib.import_module("recbole.data.dataset")
File "/root/miniconda3/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'recbole.data.dataset'
I check the path ,there is no dataset document in AdaMCT source code,but in original recbole ,there has this document。Need I to redownload recbole?
Thank you!

"Remove the 'recoble' package from the source code, install the official 'recoble' package using conda, then run the code again, and install any missing packages according to the error messages."


"node_sqlite3.node is not a valid Win32 application" on Windows on ARM64

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Windows 11 [Version 10.0.22631.3085], ARM64
- Continue: v0.8.11 (but the same problem is in v0.9.58)
- IDE: VSCode 1.86.1

Description
After just installing the current version of continue (and pre-release, the same problem) and opening its window, the "clock" icon stays forever and nothing more happens, but I finally managed to find the appropriate VSCode log section where there is an error message:
2024-02-09 23:31:33.438 [error] Error: \\?\c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\build\Release\node_sqlite3.node is not a valid Win32 application.
\\?\c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\build\Release\node_sqlite3.node
    at process.func [as dlopen] (node:electron/js2c/asar_bundle:2:2131)
    at Object.<anonymous> (node:internal/modules/cjs/loader:1403:18)
    at Object.func [as .node] (node:electron/js2c/asar_bundle:2:2131)
    at Module.load (node:internal/modules/cjs/loader:1173:32)
    at node:internal/modules/cjs/loader:1014:12
    at Function.<anonymous> (node:electron/js2c/asar_bundle:2:13327)
    at Function.l._load (c:\Users\ktos\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:173:5635)
    at Function.r._load (c:\Users\ktos\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:170:29791)
    at Function.t._load (c:\Users\ktos\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:135:35292)
    at Module.require (node:internal/modules/cjs/loader:1197:19)
    at h (c:\Users\ktos\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\loader.js:4:647)
    at bindings (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:45868:52)
    at ../../core/node_modules/sqlite3/lib/sqlite3-binding.js (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:45940:41)
    at __require (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:13:51)
    at ../../core/node_modules/sqlite3/lib/sqlite3.js (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:45990:20)
    at __require (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:13:51)
    at ../../core/indexing/docs/db.ts (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:93109:30)
    at __init (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:10:56)
    at ../../core/indexing/docs/index.ts (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:99781:5)
    at __init (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:10:56)
    at src/debugPanel.ts (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:163096:5)
    at __init (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:10:56)
    at src/commands.ts (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:163482:5)
    at __init (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:10:56)
    at src/activation/activate.ts (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:171636:5)
    at __init (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:10:56)
    at c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:171683:89
    at dynamicImportAndActivate (c:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\extension.js:171683:53)

The file "node_sqlite3.node" referenced there is actually a x86-64 DLL, not an ARM64 DLL, so I believe the problem may lay in there:
C:\Users\ktos\.vscode\extensions\continue.continue-0.8.11-win32-arm64\out\build\Release>file node_sqlite3.node
node_sqlite3.node: PE32+ executable (DLL) (GUI) x86-64, for MS Windows, 7 sections

A bit related to the #724 where I asked for Win32-ARM64 support related to the esbuild - the pre-release version v0.9.8 was behaving very similarly but I couldn't pinpoint the actual error message until today.
To reproduce
No response
Log output
No response

How can I solve the same problem I encountered?

the same problem +1

Looks to be an issue when building sqlite3 and onnxruntime-node, Same issue occurs for me on Apple silicon, Solution for me was to run the following ( the same approach should also work on W11 ARM as well )...
# change to plugin directory
cd ~/.vscode/extensions/continue.continue-*

# remove sqlite3 and onnxruntime libraries
npm uninstall sqlite3 onnxruntime-node

# reinstall with targetet architecture flags
npm install sqlite3 onnxruntime-node --build-from-source --target_arch=arm64 --fallback-to-build

# copy onnxruntime binaries to output folders
cp ./node_modules/libonnxruntime.1.17.3.dylib ./bin/napi-v3/darwin/arm64/libonnxruntime.1.17.3.dylib
cp ./node_modules/onnxruntime_binding.node ./bin/napi-v3/darwin/arm64/onnxruntime_binding.node

# copy sqlite binary to build output folder
cp ./node_modules/sqlite3/build/Release/node_sqlite3.node ./out/build/Release/node_sqlite3.node

Relevant stack overflow post: https://stackoverflow.com/questions/72553650/how-to-get-node-sqlite3-working-on-mac-m1

Same error for Win11 x64.
The file C:\Users\me\AppData\Roaming\JetBrains\GoLand2024.1\plugins\continue-intellij-extension\core\build\Release\node_sqlite3.node is the arm32 version. I 'fixed' it (this error, but processing seems to hang) by copying the right binary from C:\Users\me\AppData\Roaming\JetBrains\GoLand2024.1\plugins\continue-intellij-extension\core\win32-x64.
Now it's stuck at "Completed indexing chunks", with no errors in the core.log. Hope this might give anyone a clue or point of reference.
I'm bummed that my first experience with Continue is a bad experience. It seemed promising.

@theNailz sorry to hear about this. Luckily I think it's related to a fix I'll be making today—were you on Continue version 0.0.54?

@theNailz sorry to hear about this. Luckily I think it's related to a fix I'll be making today—were you on Continue version 0.0.54?

Yes 0.0.54

Same error for me after upgrading from W10 Jetbrains 0.0.56 to 0.0.59. Got fixed by manually copying the correct compiled file as already explained from core\win32-x64\node_sqlite3.node to build\Release folder

I get the same BUG in Webstorm windows11 x64,is anyone know how to fix?

I'm also encountering this issue, on a brand new Windows 11 ARM64, using VS Code (ARM). Tried the latest release *0.8.46) as well as the latest pre-release, to no avail.
From the debug console:
Activating extension 'Continue.continue' failed: \\?\c:\Users\x\.vscode\extensions\continue.continue-0.8.46-win32-arm64\out\build\Release\node_sqlite3.node is not a valid Win32 application. \\?\c:\Users\x\.vscode\extensions\continue.continue-0.8.46-win32-arm64\out\build\Release\node_sqlite3.node.
I tried the suggestions above from https://github.com/continuedev/continue/issues/839#issuecomment-2081875150 and https://github.com/continuedev/continue/issues/839#issuecomment-2215973239 but the error message stays the same, even after disabling / enabling the extension as well as restarting VS Code.

Bump, still occurring Windows 11 ARM64 (Snapdragon Elite)

Same problem here with snapdragone Elite. Anyone has a solution?

I was also curios about the ARM64 Windows experience so I went and got a hold of one of the new Snapdragon Elite laptops. It's been working out great so far except for this plugin. I just started using a local ollama setup for code completion and this plugin works great for x64 on windows and the ARM M1 apple, not so much for winARM.
I know its probably a small user base right now, but man: the snapdragon elite laptops are awesome: fast, (relatively) cheap vs Apple and all day battery life, so I bet ya it will have more users soon.
Long story short, +1 request for a winARM64 build/fix please!

+1 here too
Would love to use it on my ARM Windows machine.

I managed to run the VSCode extension on my Snapdragon X Elite laptop after building sqlite3 (as described higher up in this thread) and lancedb locally with some tinkering.
I'd be happy to share binaries or walk you through the build process. Though I guess the only sustainable way forward is for the developers of these upstream packages to provide pre-built binaries for windows arm64?

For sqlite3 one way forward could be switching to better-sqlite3 as proposed here.
For lancedb I don't imagine it's viable to switch it out for something else, but maybe we can get them to provide binaries for windows arm64. The build was straightforward once I managed to straighten out the build dependencies, at least on an Windows arm64 platform, no idea about cross-compiling.

I managed to run the VSCode extension on my Snapdragon X Elite laptop after building sqlite3 (as described higher up in this thread) and lancedb locally with some tinkering.
I'd be happy to share binaries or walk you through the build process. Though I guess the only sustainable way forward is for the developers of these upstream packages to provide pre-built binaries for windows arm64?

For sqlite3 one way forward could be switching to better-sqlite3 as proposed here.
For lancedb I don't imagine it's viable to switch it out for something else, but maybe we can get them to provide binaries for windows arm64. The build was straightforward once I managed to straighten out the build dependencies, at least on an windows arm64 platform, no idea about cross-compiling.

 Have you managed to get Continue working on Windows 11 ARM64? Could you share the necessary files?

Here you go, hope this is permitted.
From the continue folder (e.g. C:\Users\<name>\.vscode\extensions\continue.continue-x.y.z-win32-arm64 ), you need to put node_sqlite3.node under .\out\build\Release\ and the vectordb-win32-arm64-msvc folder under .\out\node_modules\@lancedb\
bundles.zip
Keep in mind that I just built this off lancedb master, so no guarantees that it will work for updated versions of continue (or the current one, for that matter).
I'll see if I can cross-compile lancedb for arm64 from my windows x86 machine. If that works I guess there's a chance they could integrate it in their normal builds and provide binaries.

I managed to run the VSCode extension on my Snapdragon X Elite laptop after building sqlite3 (as described higher up in this thread) and lancedb locally with some tinkering.
I'd be happy to share binaries or walk you through the build process. Though I guess the only sustainable way forward is for the developers of these upstream packages to provide pre-built binaries for windows arm64?

For sqlite3 one way forward could be switching to better-sqlite3 as proposed here.
For lancedb I don't imagine it's viable to switch it out for something else, but maybe we can get them to provide binaries for windows arm64. The build was straightforward once I managed to straighten out the build dependencies, at least on an windows arm64 platform, no idea about cross-compiling.

 Have you managed to get Continue working on Windows 11 ARM64? Could you share the necessary files?

Here you go, hope this is permitted. bundles.zip
From the continue folder (e.g. C:\Users\<name>\.vscode\extensions\continue.continue-x.y.z-win32-arm64 ), you need to put node_sqlite3.node under .\out\build\Release\ and the vectordb-win32-arm64-msvc folder under .\out\node_modules\@lancedb\
Keep in mind that I just built this off lancedb master, so no guarantees that it will work for updated versions of continue (or the current one, for that matter).
I'll see if I can cross-compile lancedb for arm64 from my windows x86 machine. If that works I guess there's a chance they could integrate it in their normal builds and provide binaries.

This worked! You saved me, I was so desperate that I installed the x64 version of VSCode in emulation mode, and of course, that's not ideal. Now I’ve disabled automatic updates and will manually check for new versions from time to time, both to test compatibility with your files and in the hope that the Continue extension developers will provide a fix. Thank you.

@svantesorberg I wanted to try Continue in my ARM64 window laptop and I couldn't. Thank you, it's working fine.

Thanks all for providing some workarounds here. I'm going to reach out to LanceDB and SQLite to see if they might be willing to include windows arm64 builds in their releases, but will also take a stab at doing the build myself soon (now that it looks finally available in GH actions! https://github.blog/news-insights/product-news/arm64-on-github-actions-powering-faster-more-efficient-build-systems/)

I created a script that simplifies the update with the files provided by @svantesorberg .
Just download the bundles.zip and place it into the same directory as this script.
Executing it should update the relevant files of the current continue extension so the extension should work on Windows on ARM64.
import os
import shutil
import zipfile

def get_extension_directory(username):
    base_path = f"C:\\Users\\{username}\\.vscode\\extensions"
    all_dirs = [d for d in os.listdir(base_path) if os.path.isdir(os.path.join(base_path, d))]

    matching_dirs = [d for d in all_dirs if d.startswith("continue.continue-") and "win32-arm64" in d]

    if len(matching_dirs) == 0:
        print(f"No directories found matching the criteria.")
        # List all directories in the base path for manual verification
        print(f"All directories in {base_path}:")
        for dir_name in all_dirs:
            print(dir_name)
        raise Exception("Could not find any matching extension directory.")
    elif len(matching_dirs) > 1:
        print(f"Found multiple directories matching the criteria: {matching_dirs}")
        raise Exception("Found more than one matching extension directory.")

    return os.path.join(base_path, matching_dirs[0])

def unpack_bundle(zip_path, extract_to):
    with zipfile.ZipFile(zip_path, 'r') as zip_ref:
        zip_ref.extractall(extract_to)

def main():
    # Get the current user's username
    try:
        username = os.getlogin()
    except OSError:
        print("Could not determine the current user. Using environment variable USERPROFILE.")
        username = os.path.basename(os.environ['USERPROFILE'])

    print(f"Using username: {username}")

    # Locate the VS Code extension directory
    try:
        extension_dir = get_extension_directory(username)
        print(f"Found extension directory: {extension_dir}")
    except Exception as e:
        print(e)
        return

    # Define paths for unpacking and copying
    script_dir = os.path.dirname(os.path.abspath(__file__))
    bundle_zip_path = os.path.join(script_dir, "bundles.zip")  # Renamed to bundles.zip
    temp_extract_dir = os.path.join(script_dir, "temp_extract")

    if not os.path.exists(bundle_zip_path):
        raise Exception("bundles.zip not found in the script directory.")

    # Unpack bundles.zip
    unpack_bundle(bundle_zip_path, temp_extract_dir)
    print(f"Unpacked {bundle_zip_path} to {temp_extract_dir}")

    # Define source and destination paths for copying files
    node_sqlite3_source = os.path.join(temp_extract_dir, "node_sqlite3.node")
    vectordb_source = os.path.join(temp_extract_dir, "vectordb-win32-arm64-msvc")

    out_build_release_dest = os.path.join(extension_dir, "out", "build", "Release")
    node_modules_lancedb_dest = os.path.join(extension_dir, "out", "node_modules", "@lancedb", "vectordb-win32-arm64-msvc")  # Ensure the directory is copied as a whole

    # Ensure destination directories exist
    os.makedirs(out_build_release_dest, exist_ok=True)
    os.makedirs(os.path.dirname(node_modules_lancedb_dest), exist_ok=True)  # Create parent directories if they don't exist

    # Copy files and directories
    shutil.copy2(node_sqlite3_source, out_build_release_dest)
    print(f"Copied {node_sqlite3_source} to {out_build_release_dest}")

    shutil.rmtree(node_modules_lancedb_dest, ignore_errors=True)  # Remove existing directory if it exists
    shutil.copytree(vectordb_source, node_modules_lancedb_dest)
    print(f"Copied {vectordb_source} to {node_modules_lancedb_dest}")

    # Clean up temporary extraction directory
    shutil.rmtree(temp_extract_dir)
    print(f"Cleaned up temporary extraction directory: {temp_extract_dir}")

if __name__ == "__main__":
    main()

Hi, I'm also experiencing this issue on my Windows and mac system. I'm wondering if there has been any progress or resolution on this yet?
If not, could you confirm if the following workaround is the recommended approach?

Hi, I'm also experiencing this issue on my Windows and mac system. I'm wondering if there has been any progress or resolution on this yet?
If not, could you confirm if the following workaround is the recommended approach or is there some other approach?

Thanks for the above bundle.zip and python script. It works. But sometimes the window used to work Continue Extension start to hang again while opening a new vscode window works. What could be the reason?

I created a script that simplifies the update with the files provided by @svantesorberg . Just download the bundles.zip and place it into the same directory as this script. Running this script should update the relevant files of the current continue extension so the extension should work on Windows on ARM64.
import os
import shutil
import zipfile

def get_extension_directory(username):
    base_path = f"C:\\Users\\{username}\\.vscode\\extensions"
    all_dirs = [d for d in os.listdir(base_path) if os.path.isdir(os.path.join(base_path, d))]

    matching_dirs = [d for d in all_dirs if d.startswith("continue.continue-") and "win32-arm64" in d]

    if len(matching_dirs) == 0:
        print(f"No directories found matching the criteria.")
        raise Exception("Could not find any matching extension directory.")
    elif len(matching_dirs) > 1:
        print(f"Found multiple directories matching the criteria: {matching_dirs}")
        raise Exception("Found more than one matching extension directory.")

    return os.path.join(base_path, matching_dirs[0])

def unpack_bundle(zip_path, extract_to):
    with zipfile.ZipFile(zip_path, 'r') as zip_ref:
        zip_ref.extractall(extract_to)

def main():
    # Get the current user's username
    try:
        username = os.getlogin()
    except OSError:
        print("Could not determine the current user. Using environment variable USERPROFILE.")
        username = os.path.basename(os.environ['USERPROFILE'])

    print(f"Using username: {username}")

    # Locate the VS Code extension directory
    try:
        extension_dir = get_extension_directory(username)
        print(f"Found extension directory: {extension_dir}")
    except Exception as e:
        print(e)
        return

    # Define paths for unpacking and copying
    script_dir = os.path.dirname(os.path.abspath(__file__))
    bundle_zip_path = os.path.join(script_dir, "bundles.zip")  # Renamed to bundles.zip
    temp_extract_dir = os.path.join(script_dir, "temp_extract")

    if not os.path.exists(bundle_zip_path):
        raise Exception("bundles.zip not found in the script directory.")

    # Unpack bundles.zip
    unpack_bundle(bundle_zip_path, temp_extract_dir)
    print(f"Unpacked {bundle_zip_path} to {temp_extract_dir}")

    # Define source and destination paths for copying files
    node_sqlite3_source = os.path.join(temp_extract_dir, "node_sqlite3.node")
    vectordb_source = os.path.join(temp_extract_dir, "vectordb-win32-arm64-msvc")

    out_build_release_dest = os.path.join(extension_dir, "out", "build", "Release")
    node_modules_lancedb_dest = os.path.join(extension_dir, "out", "node_modules", "@lancedb", "vectordb-win32-arm64-msvc")  # Ensure the directory is copied as a whole

    # Ensure destination directories exist
    os.makedirs(out_build_release_dest, exist_ok=True)
    os.makedirs(os.path.dirname(node_modules_lancedb_dest), exist_ok=True)  # Create parent directories if they don't exist

    # Copy files and directories
    shutil.copy2(node_sqlite3_source, out_build_release_dest)
    print(f"Copied {node_sqlite3_source} to {out_build_release_dest}")

    shutil.rmtree(node_modules_lancedb_dest, ignore_errors=True)  # Remove existing directory if it exists
    shutil.copytree(vectordb_source, node_modules_lancedb_dest)
    print(f"Copied {vectordb_source} to {node_modules_lancedb_dest}")

    # Clean up temporary extraction directory
    shutil.rmtree(temp_extract_dir)
    print(f"Cleaned up temporary extraction directory: {temp_extract_dir}")

if __name__ == "__main__":
    main()

Here is the revised version of your PowerShell code, rewritten using a neural network ;), with an added check for the VSCodium fork installation:

# Get current user's username
$Username = $env:USERNAME
Write-Host "Using username: $Username"

# Define paths
$VscodeExtensionsPath = "C:\Users\$Username\.vscode\extensions"
$VscodeOssExtensionsPath = "C:\Users\$Username\.vscode-oss\extensions"

# Check if the directories exist
$VscodeExists = Test-Path $VscodeExtensionsPath
$VscodeOssExists = Test-Path $VscodeOssExtensionsPath

if ($VscodeExists -and -not $VscodeOssExists) {
    $ExtensionBasePath = $VscodeExtensionsPath
    Write-Host "Using extensions path: $ExtensionBasePath"
}
elseif (-not $VscodeExists -and $VscodeOssExists) {
    $ExtensionBasePath = $VscodeOssExtensionsPath
    Write-Host "Using extensions path: $ExtensionBasePath"
}
elseif ($VscodeExists -and $VscodeOssExists) {
    Write-Host "Both .vscode and .vscode-oss extensions directories exist."
    Write-Host "1) $VscodeExtensionsPath"
    Write-Host "2) $VscodeOssExtensionsPath"
    $choice = Read-Host "Enter 1 to use .vscode, 2 to use .vscode-oss"
    if ($choice -eq "1") {
        $ExtensionBasePath = $VscodeExtensionsPath
    } elseif ($choice -eq "2") {
        $ExtensionBasePath = $VscodeOssExtensionsPath
    } else {
        Write-Host "Invalid choice. Exiting."
        exit
    }
    Write-Host "Using extensions path: $ExtensionBasePath"
}
else {
    Write-Host "Neither .vscode nor .vscode-oss extensions directories exist. Exiting."
    exit
}

# Locate the VS Code extension directory
$AllDirs = Get-ChildItem -Path $ExtensionBasePath -Directory

$MatchingDirs = $AllDirs | Where-Object { $_.Name -like "continue.continue-*" -and $_.Name -like "*win32-arm64*" }

if ($MatchingDirs.Count -eq 0) {
    Write-Host "No directories found matching the criteria."
    exit
}
elseif ($MatchingDirs.Count -gt 1) {
    Write-Host "Found multiple directories matching the criteria:"
    $MatchingDirs | ForEach-Object { Write-Host $_.Name }
    Write-Host "Exiting."
    exit
}
else {
    $ExtensionDir = $MatchingDirs[0].FullName
    Write-Host "Found extension directory: $ExtensionDir"
}

# Define paths for unpacking and copying
$ScriptDir = Split-Path -Parent $MyInvocation.MyCommand.Definition
$BundleZipPath = Join-Path $ScriptDir "bundles.zip"
$TempExtractDir = Join-Path $ScriptDir "temp_extract"

if (-not (Test-Path $BundleZipPath)) {
    Write-Host "bundles.zip not found in the script directory."
    exit
}

# Unpack bundles.zip
Expand-Archive -LiteralPath $BundleZipPath -DestinationPath $TempExtractDir -Force
Write-Host "Unpacked $BundleZipPath to $TempExtractDir"

# Define source and destination paths for copying files
$NodeSqlite3Source = Join-Path $TempExtractDir "node_sqlite3.node"
$VectorDbSource = Join-Path $TempExtractDir "vectordb-win32-arm64-msvc"

$OutBuildReleaseDest = Join-Path $ExtensionDir "out\build\Release"
$NodeModulesLanceDbDest = Join-Path $ExtensionDir "out\node_modules\@lancedb\vectordb-win32-arm64-msvc"

# Ensure destination directories exist
if (-not (Test-Path $OutBuildReleaseDest)) {
    New-Item -ItemType Directory -Path $OutBuildReleaseDest -Force | Out-Null
}

$NodeModulesLanceDbDestParent = Split-Path $NodeModulesLanceDbDest
if (-not (Test-Path $NodeModulesLanceDbDestParent)) {
    New-Item -ItemType Directory -Path $NodeModulesLanceDbDestParent -Force | Out-Null
}

# Copy files and directories
Copy-Item -Path $NodeSqlite3Source -Destination $OutBuildReleaseDest -Force
Write-Host "Copied $NodeSqlite3Source to $OutBuildReleaseDest"

# Remove existing directory if it exists
if (Test-Path $NodeModulesLanceDbDest) {
    Remove-Item -Path $NodeModulesLanceDbDest -Recurse -Force
}

Copy-Item -Path $VectorDbSource -Destination $NodeModulesLanceDbDest -Recurse -Force
Write-Host "Copied $VectorDbSource to $NodeModulesLanceDbDest"

# Clean up temporary extraction directory
Remove-Item -Path $TempExtractDir -Recurse -Force
Write-Host "Cleaned up temporary extraction directory: $TempExtractDir"

There is now some work going on in the lancedb project to get builds working for win32-arm64. Maybe you had something to do with this @sestinj :)
I'll try to help out if I can be of any use there, but I'm quite busy over the next few days so maybe else will work it out before I even get started.

Hi, same problem here on my X Elite machine running 26100.2314. Both the VSCode and JetBrains extensions do not work at all. I've configured Ollama with the recommended Qwen 2.5 1.5B model and it works fine. I haven't tried the solution above which is for VSCode only, and I don't really have time to try to compile the JetBrains extension for arm64 if that even works at the time of writing. Really looking forward for ARM64 support 🙏

Thanks a lot for the scripts, they worked !!


Fix javadoc for Configuration Cache build feature

Fixing typo in a method javadoc

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


[TESTS] Remove _i32 suffix for index constants

IIRC, they are indeed i32 in triton

I opened this PR because I was going through the tests and got confused by these _i32 suffixes.
There are 155 matches for scf.for in the *.mlir files, but the following command (regex via ripgrep):
$ rg '_i32 = arith.constant [0-9]* : index' test/

found only 5 matches. For example, this is another test in test/TritonGPU/combine.mlir that does not use _i32 suffixes for the index variables that are used in the loop.
https://github.com/openai/triton/blob/e97b3ac518e91f2a3d742ed87feba1134a8ffce1/test/TritonGPU/combine.mlir#L287-L303

Thanks for the cleanup, but:

Please refrain from submitting low-effort pull requests (PR) -- such as minor formatting/typo fixes

https://github.com/openai/triton/blob/main/CONTRIBUTING.md

The full sentence in CONTRIBUTING.md states:

Please refrain from submitting low-effort pull requests (PR) -- such as minor formatting/typo fixes -- solely for the purpose of appearing in the commit history.

Although I am okay with accepting that this PR isn't a major improvement and closing it. I'm not okay with the insinuations made here. I would like to make it clear that it was not low effort (I spent many hours on the Triton codebase during which I found these typos) nor was it aimed solely to appear in the commit history. Like I said, the _i32 variable name confused me. I hoped to avoid other people from having similar confusion in the future.


[BugFix] Fix bug drop not exists task NPE

Fixes #24741
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/DDLStmtExecutor.java
0
4
00.00%
[877, 878, 879, 881]

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


PLANNER-648 Workbench: Rename *ConstantsWithLookup classes to match zanata plugin include pattern

See https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/blob/master/pom.xml#L819

+1 to merge (when green).


Improve playlist demo

As a developer or user of the library I want to be able to test playlist capabilities extensively in the demo as well.
Acceptance criteria
A playlist demo is available with advanced features, like:

A list of the items in the playlist.
The current item clearly identified.
Addition / deletion / reordering of items with list gestures.

Tasks

[x] Create dedicated layout for the playlist example.
[x] Display items and current item (maybe adding a description to the sample streams is required).
[x] Implement add / delete / reorder.
[x] Implement shuffle.
[x] Implement reload with other items (pool of items from which some are pulled randomly).

Add new item

Display current item

Delete

Reorder

Shuffle

Reload with other items

As expected implementing and using the playlist demo made us learn about our existing implementation. We performed the following changes:

To ensure better overall AirPlay stability (see #134) we implemented a strategy where the first item of the AVQueuePlayer is updated first, before the other items are appended a bit later. With the playlist demo we realized there was an issue in the implementation when moving an item (externally the playlist was correct but the player was restarting playback). To fix this issue we had to implement a better strategy to update the queue player items when either the source list is updated or a source is updated (e.g. transitioning from the loading to the loaded state).
For convenience The advanceToNextItem(), returnToPreviousItem() and skipToLive() methods now return no boolean, making them easy to use as button actions. Methods still exist to know whether these actions are possible and, if they aren't, calling the above corresponding listed methods has no effect.
We introduced a method to skip to a playlist item index directly. Unlike the above methods this method throws when an invalid index is provided.
Knowing the current player item is not really useful, as such items provide no meaningful services (e.g. associated media metadata that an app could use). To make it easier to connect a playlist player item with an external data source for the playlist, managed by the app, it therefore appeared that the current index is far more interesting, as it allows to loosely couple medias loaded into the player with metadata managed by the app and used to load the player in the first place. The playlist demo shows how both can be synchronized in a real-life example.
We removed @MainActor globally, as it was introduced unnecessary friction in code using Player. We might restore it when SDKs better annotate their code for main actor support (e.g. SwiftUI button actions) but currently this just adds frustration in API usage with no real significant benefit (we namely mostly use Combine for the implementation anyway).
We improved RawPlayer.replaceItems(with:) implementation which was incorrectly crashing when updating the queue with an item list containing the current item. We also improved the implementation to avoid updates when not necessary. The implementation was clearly lacking tests but is now covered well.


Feature request - Access get_slack_client() in filters

Feature request to access get_slack_client() in filters... So we can interact with the Slack API in filters.
Thanks.

Big +1 on this.

:+1:

By filters do you mean plugins?

I totally forget. Closing this since I moved to a Go library. Someone can feel free to open a new Ticket and reference this.

Cool, thanks for the response. Sorry we've taken so long to get around to this.


Quickstart send email with SMTP from Azure doesn't mention SMTP is blocked by default

Hello! I would like to point out why this article surprises me. This article is distinctly located under "Azure". In Azure, it's mostly well-known that SMTP outbound, port 25 is blocked by default.
Wouldn't someone coming across this article, located within "Azure" docs, attempting to send an email via SMTP, immediately run into a problem where doing this is blocked?
Yes, not all subscription types have this blocked, in some it's possible to remove the block, but I think the majority would run into a problem, but please correct me if I'm wrong.

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

ID: 9e702d3e-fb9f-5a08-215b-94d3489ce6a2
Version Independent ID: 64caa261-8a10-da47-7df9-a769c644249b
Content: How to use SMTP to send an email with Azure Communication Services. - An Azure Communication Services quick start guide.
Content Source: articles/communication-services/quickstarts/email/send-email-smtp/send-email-smtp.md
Service: azure-communication-services
GitHub Login: @ddouglas
Microsoft Alias: ddouglas

@samrueby
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for the feedback. This doc provides a quickstart guide for sending email via SMTP using Azure Communication Services (ACS) with SMTP authentication.
In this case, the doc aims to include step-by-step instructions for configuring the SMTP client with authentication, creating an email message, and sending the message via SMTP.
Port 587 is a commonly used port for email submission. It is used for sending email messages from an email client to an email server or from one email server to another. Port 587 is often used as an alternative to port 25, which is the default port for SMTP. Port 587 is preferred over port 25 because it supports encryption and authentication, which can help prevent spam and other malicious activity.


Address gaps in Process Activity and Memory Activity wrt process injection.

Related Issue:
#1041: Process Activity and Memory Activity cannot adequately describe common injection-related behaviours
Description of changes:

Added entry to injection_type_id enum in dictionary.json to cover queueing of an asynchronous procedure call (APC).
Added entry to activity_id enum in events/system/memory.json to cover mapping of shared memory object.

It says that I should put labels on this but the ability to do that seems to be disabled for me. The labels I would use if I could would be process_activity, memory_activity, non_breaking, and v1.2.0_and_later.

This is a great catch, @davemcatcisco. One minor thing, could you change the branch to be merged to to main from ocsf:v1.2.0

For process_activity we are adding an additional injection type, is this the more specific way people think of process injection via memory activity? In other words, is the same situation representable as both process activity or memory activity?

Thanks, Paul. I'm not sure I fully understand your question but I'll just say this in case it helps. Process injection broadly speaking requires two distinct things to happen:

Memory pages containing the code that the attacker process wishes to execute and/or the data upon which that code will operate must be present in the virtual memory space of the victim process with the appropriate page protections. There are various ways to achieve this, represented in OCSF by a Memory Activity event.

The attacker process must initiate execution of that code by a thread running within the context of the victim process. This can be achieved either by re-purposing an existing thread (temporarily or permanently) or by creating an entirely new thread for this purpose. In OCSF, this type of activity is represented by a Process Activity event.

It is important, I think, that these continue to be treated as two distinct activities because each can be used in isolation for other purposes, some benign and some not.

Closing as this PR has now been superseded by #1060.


fix #217 and add shifting to the tree

Thanks @antgonza! Looks like there's conflicts, can you resolve those? Also, @kwcantrell if this looks good to you can you go ahead and merge this PR?

@antgonza, there's conflicts again due to a merged PR. Can you re-solve conflicts?

@antgonza, let's wait for @kwcantrell to submit his review and then this should be good to merge and solve conflicts.

Thanks @antgonza @kwcantrell!


Compilation error

The type or namespace name 'SVGExport' could not be found (are you missing a using directive or an assembly reference?)
where could I found SVGExport assembly reference ?

Have you install the required plugin SVG Export (http://community.sparxsystems.com/community-resources/706-svg-diagram-export-add-in)  please locate this SVGExport.dll in the install.
Cheers, Phil

I added reference to "C:\Program Files (x86)\Siemens\EA SVG Export\SVGExport.dll"
But 'EAPlugin' does not contain a definition for 'SaveDiagramAsSvg'	in DiagramManager.cs.

I think you have the wrong version of this plugin.  I think you have installed version 1.04 rather than 1.06.
You can also just comment out that reference as that is simply a convenience menu option to export diagrams as SVG.

Ok thanks that works !

Excellent. Good luck


how to run in production?

I run multiple tornado workers for production, curious, what's the best way of sharing the connection pool? Should each worker open it's own redis connection?

You can use a single Client object by tornado worker (global var for example) or you can use a  ClientPool object by tornado worker (useful only if you execute blocking commands).
But no way to share these objects between different tornado workers. And probably not useful.

Hi, I would like to ask you a few questions. My server now serves for around 300 concurrent requests (timeout after 3 seconds), and I noticed that the server get query results (from a remote Redis server, ping ~ 0.5ms ) under high latency (long queue waiting for response). I have tried following ways to try to decrease the latency:

create several Client instance, and choose a random client to send command for each query.
limit the write buffer size
after experiments, it seems like that both ways do not work, so I get really confused now : (
Could you pls give some suggestions or point out if I am using this library in a wrong way? Thanks!


TCP closed even before accept() or reject()

Scenario:

The server receives a WS request and gives the app the request instance.
The app must do some async stuff (DB queries or whatever) so it takes some time before it calls accept() or reject().
In the meanwhile the browser closes the TCP connection (browser closed).
Nothing happens in the server. If the app calls request.accept() then the Connection will immediately emit "close" (not sure what would happen if the app calls request.reject().

I've a usecase (a WebSocket bridge server which just calls accept() once two peers start the WS handshake) in which I do need to react if the first peer's pending request is closed due to TCP closure. Currenlty I do it in a not very nice way:
request.httpRequest.socket.on('end', onPeerDisconnectedSoon);

Of course I remove such a listener if the connection is established (but it does not work and it's called anyhow, still don't know why but that is another subject).
Would it make sense to have a new WebSocketRequest event called "requestClosed" which is just fired in case the client closes the TCP connection before accept() or reject() is called?

I think we could add a requestClosed event.  We'll have to think about the details and make sure that it doesn't affect backward compatibility with existing code.

The only difference is that existing code may just react on requestAccepted or requestRejected, but I assume that if requestClosed is emitted the others would not.

Solution: Just behave differently if the user has set as least one listener for requestClosed, otherwise do the same as now.

Second issue: The TCP connection is closed by the client but the app still calls accept() or reject() after that.

Solution: Same as above. If there is at least one requestClosed listener then throw if accept() or reject() are called after it. Otherwise nothing would change.


enable snapshot flatten for non-persistent disks

Author Name: Anton Todorov (Anton Todorov)
Original Redmine Issue: 4272, https://dev.opennebula.org/issues/4272
Original Date: 2015-12-30

Hello developers,
I would like to have an option to flatten non-persistent image snapshots as the persistent ones.
Something almost like the procedure used for the persistent images:

suspend VM
snapshot_flatten
restore VM

(or fsfreeze - flatten - fsthaw ...)
In this way I can make a backup snapshot before VM upgrade, do the upgrade and if everything is ok, flatten("delete") the created snapshot as it is not needed anymore.
Kind Regards,
Anton Todorov

Original Redmine Comment
Author Name: Anton Todorov (Anton Todorov)
Original Date: 2015-12-30T15:19:16Z

Please update the subject...
dnspdhoy = snapshot :)
Kind Regards,
Anton


[PROTON] Automatic backend selection using triton.runtime.driver.active

@mwootton

Hi @zhanglx13 and @antiagainst , could you please take a look? We'll need an approval from you guys to merge it into main; this PR is used to unblock Michael when cupti is not available on AMD GPUs.

Just one question, what if the user specified backend does not match the real backend?

It will error out in the backend if the profiler library is not available.
Or if the profiler library is available but the device doesn't match, the user probably just doesn't get any metrics.
We assume the user knows what he wants to do if he specified the backend.
I reserved this argument here because in the future we may have CPU kernels but running on a node that has GPUs. In that case, we could either profile CPUs or GPUs.


False positive for unnecessary parentheses

Code:
static bool isString(object o) {
	if (!(o is string s)) return false;
	return true;
}

Expected: No error.
Actual: Program.cs(5,8): warning SA1119: Statement must not use unnecessary parenthesis

This appears to be a duplicate of #2372, which was fixed for the 1.1.0-beta004 release. If you are using this release (or newer) and still encounter the behavior, please reopen this issue.


OCPBUGS-20266: update bad url endpoint termination handler test

This change moves the bad url test to its own Context block so that it does not get caught in the mock termination service counter. In some cases it is possible for the bad url test to be run first, when this happens the counter will never advance because the url is not valid. This produces a dead lock condition in the test. Migrating the test to its own block alleviates this issue.

/lgtm
/approve

looks like we might have hit another issue with this one, i'm gonna dig in a little more

/hold found another issue

/hold cancel
found another test that was affected by the ordering, i've added logic to reset the counter and properly record its increase.

/lgtm


Restrict looping over some map types

It doesn't make sense to loop over these types:

avg
stats
hist
lhist

This is because these maps have complex values and keys, which don't work for iteration. Example:
$ sudo ./src/bpftrace -e 'BEGIN { @x[1] = avg(5); for ($kv : @x) { print(($kv.0, $kv.1)); } exit(); }'
Attaching 1 probe...
(1, )
(1, )

@x[1]: 5

Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

I'm wondering if we could come up with a way to allow iteration for these.

I was working on avg but it's a little tricky because it keeps two keys (count and total). I'm sure it's possible for all of these, it just requires some thought on the desired behavior.


HTTP Auth credentials don't show up after stack install

After installing EE, run:
ee stack install --mysql --php7 --wpcli --redis --phpmyadmin --phpredisadmin

Expected output
HTTP Auth User Name: easyengine
HTTP Auth Password : XXXXXXX

But, it does not show up. Instead, I have to use ee site create and after the first site is created they do show up, which is very strange, since the http auth credentials for port 22222 should show up after you start adding things like phpMyAdmin.

[x] lsb_release -a

No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.4 LTS
Release:        14.04
Codename:       trusty

[x] ee -v

EasyEngine v3.6.2
Copyright (c) 2016 rtCamp Solutions Pvt. Ltd.

[x] ee info

NGINX (1.10.0):

user                             www-data
worker_processes                 auto
worker_connections               4096
keepalive_timeout                30
fastcgi_read_timeout             300
client_max_body_size             100m
allow                            <IP_ADDRESS>

PHP (5.6.22-2):

user
expose_php                       Off
memory_limit                     128M
post_max_size                    100M
upload_max_filesize              100M
max_execution_time               300

Information about www.conf
ping.path                        /ping
pm.status_path                   /status
process_manager                  ondemand
pm.max_requests                  500
pm.max_children                  100
pm.start_servers                 20
pm.min_spare_servers             10
pm.max_spare_servers             30
request_terminate_timeout        300
xdebug.profiler_enable_trigger   off
listen                           <IP_ADDRESS>:9000

Information about debug.conf
ping.path                        /ping
pm.status_path                   /status
process_manager                  ondemand
pm.max_requests                  500
pm.max_children                  100
pm.start_servers                 20
pm.min_spare_servers             10
pm.max_spare_servers             30
request_terminate_timeout        300
xdebug.profiler_enable_trigger   on
listen                           <IP_ADDRESS>:9001

MySQL (10.1.14-MariaDB) on localhost:

port                             3306
wait_timeout                     600
interactive_timeout              28800
max_used_connections             11
datadir                          /var/lib/mysql/
socket                           /var/run/mysqld/mysqld.sock
my.cnf [PATH]                    /etc/mysql/conf.d/my.cnf

[x] wp --allow-root --info

PHP binary:     /usr/bin/php7.0
PHP version:    7.0.7-2+donate.sury.org~trusty+1
php.ini used:   /etc/php/7.0/cli/php.ini
WP-CLI root dir:        phar://wp-cli.phar
WP-CLI packages dir:    /root/.wp-cli/packages/
WP-CLI global config:
WP-CLI project config:
WP-CLI version: 0.23.0

Hi @khromov
EasyEngine configures HTTP Auth only when Nginx is installed with stack.

@iam404 Isn't phpMyAdmin and phpRedisAdmin running on Nginx?

@khromov you can use ee secure --auth to change HTTP auth. https://easyengine.io/docs/commands/secure/
Sorry for a delayed reply. I hope your issue is fixed by now.
If you are still facing the issue, please use http://community.rtcamp.com/c/easyengine for further support.
We are cleaning up issue tracker and closing this issue because we won't be able to contribute to this discussion further.


Issue/255

#255
Ties in the sorting logic of the Notifications REST controller with the existing sorting logic.
New parameters available ( values are case insensitive )
sort = [ PRIORITY | DUE_DATE ] < default is no sorting
order = [ ASCENDING | ASC | DECENDING | DESCENDING | DESC ]  < default is ASCENDING

note - I inadvertently had the sorting param listed as 'strategy'.  It is in fact 'sort'.  The PR description is updated.

Please don't merge -- I want to make a few adjustments.

Closing in favor of #257


[Feature request] Not to index specific package

Feature request
Feature description
Not to index specific package
How the feature is useful
As an author of the package, I'd like to tell cdnjs that a certain package is obsolete, replaced, or just the author doesn't want it to be indexed by the search engines. This might be useful for the dev. experience so certain (obsolete, unmaintained, replaced...) packages won't be included in the search engine results. Therefore, there is a higher chance developer will use the best and up to date package for his/her needs.
Discussion: https://github.com/cdnjs/cdnjs/discussions/14062

Hello! Is there any chance you will address this issue in the foreseeable future? If you don't have resources, could I implement it? Thank you.
Cheers!


Secondary Progress

How to add a secondary progress for showing buffering stream media?

I don't think this functionality is supported atm. I'd consider writing a different component to handle that case.


Buy and Sell stocks with CoolDown

Please merge it.

Not accepting CPP/C dsa codes
Alternative ->
1)Create a page using HTML & CSS Add dsa problem statement and solution in that
2)You can contribute in python and java OR
Ask for an issue assign


[fix] Increase timeout on ManagedLedgerCompressionTest flaky test

Fixes #16497
Motivation
In reviewing the errors associated with ManagedLedgerCompressionTest#testRestartBrokerEnableManagedLedgerInfoCompression, it seems like the problem is the number of broker restarts that occur. When I run this test on my laptop, it takes 20 seconds. I profiled it, and the most expensive part is starting up the broker's WebService, which is unfortunate because it is not tested here.
I propose as a temporary "fix" that we increase the timeout on this test to 60 seconds. The failure reports are all related to timeouts, and there is nothing fancy about this test that makes me think the test itself should be flaky. If we see further problems, we can investigate later.
Modifications

Increase test timeout from 30 to 60 seconds

Documentation

[x] doc-not-needed

Codecov Report

Merging #20352 (924ad04) into master (e4f1f09) will increase coverage by 36.07%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #20352       +/-   ##
=============================================
+ Coverage     36.80%   72.88%   +36.07%
- Complexity    12046    31849    +19803
=============================================
  Files          1687     1864      +177
  Lines        128755   138318     +9563
  Branches      14002    15173     +1171
=============================================
+ Hits          47389   100812    +53423
+ Misses        75106    29504    -45602
- Partials       6260     8002     +1742

Flag
Coverage Δ

inttests
24.15% <ø> (-0.03%)
:arrow_down:

systests
24.97% <ø> (+0.09%)
:arrow_up:

unittests
72.16% <ø> (+40.17%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1433 files with indirect coverage changes


'Observe' sends duplicate messages to the same target

The add_target function should handle duplicate targets to avoid flooding a particular target.

As far as I am aware @rjrodger fixed the duplicated target few days ago (now the ID is checked before adding it to the authoritative list). Do you have any example where this is happening? can you pull the latest version and confirm that the problem is gone?
This commit: https://github.com/rjrodger/seneca-balance-client/commit/3b91622723827028d9e108a8b5c5a2eaee92df56
Fixes the duplicated target problem.

@dgonzalez I'd say this is NodeZoo. I'm due to do a test run over the next couple of days so I can check this.

@mcdonnelldean @dgonzalez This was found in NodeZoo.

@eduardobs79 Do you have any logs or anything to show the multiple messages? If not, how did you figure it out? I'm looking for steps to reproduce the problem. This could be a potential performance hog so I'm keen to see if it's still broken :D

Fixed in release.


Add Reset() method to Iterator

I found Iterator creation was expensive in some benchmarks. Adding a Reset() method to the interface to allow for iterator pooling.

@mschoch would you mind taking a look

This looks good to me.  One remaining item is that we ask that contributors sign the Couchbase CLA, see https://github.com/couchbaselabs/vellum/blob/master/CONTRIBUTING.md

Cheers, I signed/submitted the agreement.

Thanks


Add a Jupyter adapter

Jupyter is a notebook application that allows to interactively write and execute code snippets in a web browser. It is particularly applied for data analytics, where it allows for flexible queries and data analytics that can be immediately visualized. In consequence, Jupyter is also a good fit for Metacrate.
Concretely, Metacrate should be integrated with Jupyter via the Jupyter Scala kernel. This JVM-based kernel allows to host Metacrate and its script language Scala is a fit for the code in the mdms-analytics module.

Completed via #58.


Add cache busting to publishing step

Changes to site.json, css, js, and similar assets aren't immediately changed for users.  We could have a some time of build steps that either updates the e-tag of index.html or adds cache busting strings to file imports to ensure the changes are immediately reflected to the end users.

Maybe we can peg this to the pwa. Instead of the service worker sending a message about new content hey reload, it could just set a timestamp variable which would force a fresh copy of files. This is how it works when NOT published so we already have all the logic in there for it

#428 is exposed under SEO and now that we've got .php driven pages and they can be upgraded dynamically based on saving the manifest we can support this in what currently exists and without the need for the publishing side of things. You can now disable the service worker if it's a site where you need instant updating / none of the advanced offline capabilities we've got available.
The CDN can now also be cached for offline viewing though it's not the default


DBZ-923 docs - gtid.new.channel.position property

Thanks, @pimpelsang. Added one more commit for some minor adjustments. Will merge after the next release.


Add StateIO paper badge in README

Hi @WesIngwersen , I added the badge for the paper. I also noticed v0.2.1 release (congrats!!) but the badge was not updated probably because you haven't put it on Zenodo?

I believe that needs to be set up in Zenodo itself? to automatically grab the latest release?

Correct, Catherine knows all about how to do it.