feat(query): Add types & Visit Array in AST for ArrayAST type

+ RegExpLiteral
+ RegExpLiteral implements HasSpan
+ RegExpLiteral implements Expression
+ RegExpLiteral.pattern

+ RegExpLiteralAST
+ RegExpLiteralAST implements ASTNode
+ RegExpLiteralAST implements RegExpLiteral
+ RegExpLiteralAST implements HasSpan
+ RegExpLiteralAST implements Expression

Codecov Report
Patch coverage has no change and project coverage change: +0.03% :tada:

Comparison is base (2b1e535) 92.47% compared to head (0ee7186) 92.51%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #777      +/-   ##
==========================================
+ Coverage   92.47%   92.51%   +0.03%
==========================================
  Files         202      203       +1
  Lines       41279    41463     +184
==========================================
+ Hits        38174    38358     +184
  Misses       3105     3105

see 3 files with indirect coverage changes

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


fix(engineimage): check if engine image is deployed correctly

Which issue(s) this PR fixes:

Issue # longhorn/longhorn#9038
What this PR does / why we need it:
After we tainted a node as NoSchedule and delete the EngineImage DaemonSet, nodeDeploymentMap will be unstable for a period and it is not enough only to check if the nodeDeploymentMap has the expected count for deloyed node.
Special notes for your reviewer:
Additional documentation or context

@Mergifyio backport v1.7.x v1.6.x


Removed installation exports

It just feels very weird to have them here! Never seen this on any other docs page.

Fair enough!


Updated to Gymnasium v0.26.2

Description
Changed the dependency on gym to gymnasium at v0.26.2.
Fixes # (issue)
A list of the changes:

Changed the output of step from obs, reward, done, info to obs, reward, termination, truncation, info.
Removed return_info in reset() and returns obs, info all the time.
Added render_mode to the environments.
Changed the tests to use gymnasium and adapt to the new output of reset() and step().
Removed seed()

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Could you make the suggested changes. Is there a test that runs check_env with all of the miniworld envs ? This is for a future pr or release where we change the randomness to use the env.np_random

test_env_checker() runs check_env() on all of the environments
@pytest.mark.parametrize("env_id", gym_miniworld.envs.env_ids)
def test_env_checker(env_id):
    if "RemoteBot" in env_id:
        return

    env = gym.make(env_id).unwrapped
    warnings.simplefilter("always")
    with warnings.catch_warnings(record=True) as w:
        check_env(env)

    for warning in w:
        if warning.message.args[0] not in CHECK_ENV_IGNORE_WARNINGS:
            raise gym.error.Error(f"Unexpected warning: {warning.message}")

    env.close()


Add resolve directories to webpack config

Use https://www.npmjs.com/package/babel-plugin-module-alias instead, so tests are also covered.


Is it possible to make the socket server use all cpu cores?

I've done some high volume tests the other day. I managed to get 1 cpu to 100% usage but the other one stays at 0.7%. Is it possible to make the websocket server use both cores?

This is led by the OS, not by the application.

Thanks for the reply. I run miltiple Laravel instances using Apache. They are all multi thread. But when using "php artisan websockets:serve" the websocket server is single thread (Because php artisan is single thread?). So how could I fire up the websocket server multi threaded?

If you give one process a limit of 1 thread, it will stick with it. Or, in some other instances, you can spawn new children by executing new processes along the way PHP. So there is no way of multi-threading it other than spawning multiple processes (in this case you need to use 2.x).
Apache has nothing to do with the websockets process either, unless you proxy it through nginx/apache.


Added avro record dataset and avro record parse

** Added test cases
** Added examples
** Remove any tf. calls by directly calling the package function
** Integrate into tensorflow build by adding avro 1.7.7 to setup.py and various shell scripts
** Added readme file.
@skye Sorry, this took a bit longer. We got everything integrated into the dataset API. Please review; especially the setup of the dependencies: avro and compilation of avroc.

Please address these comments and then I'll continue reviewing.

@skye Thanks for these comments. I'll plan to have them addressed by the end of the week. BTW: Do you know if there is a way for me to kick off a run on google's CI system, for just the Linux build?

Hm I'm not sure you can trigger it. I think I should be able to trigger it though...
Jenkins test this please.

Looks like you have some build failures... I didn't look super thoroughly, but I'm seeing errors like:
tensorflow/contrib/avro/ops/parse_avro_record.cc:1:10: fatal error: 'avro.h' file not found
#include <avro.h>

I'm guessing you have to use the full path since you included the Avro C library in contrib/avro?

Nagging Assignee @skye: It has been 17 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

You're not blocked on me, right? I'm not sure how to tell via the UI...

@skye I'm not blocked. Thanks for checking in.

We need to get the libavro.so binary blob out of there.
I've seen a couple of ways that tensorflow specifies external dependencies.

TF_USE_MPI and MPI_HOME: The user must specify the location of a pre-built MPI install directory.
Kafka and NCCLv1: Build locally. Probably pretty reasonable if you pin the dependency version.
TF_USE_HDFS: Tensorflow contains a copy-pasted hdfs.h header file, but doesn't link to libhdfs.so until runtime. I consider this pretty ugly since it is possible that hdfs.h could change upstream.
I've also seen a pre-built artifact of a dependency be downloaded as well.

I consider (1) easiest. I expect (2) to be harder since you have to consider cross-platform considerations. I dislike (3). (4) would be nice, but unfortunately, https://github.com/apache/avro does not make binary releases.
If a tensorflow maintainer can comment on a preferred method to specify an external dependency, that would be helpful.

I realize my request in my last comment is a little unusual, but I would greatly appreciate a response so I can work on cleaning that bit up.

@fraudies There is a PR #19461 which adds Apache Parquet support for tf.Dataset. If may have some similarity with the Avro support (like the dependency/etc of apache libraries). Please take a look if interested.

@yongtang Thanks. I took a brief look here: https://github.com/tensorflow/tensorflow/pull/19461/files#diff-af8bcb46ccf2bcb12dcb3d2298137be2. I guess we would have to write a similar bazel build file for avroc. Correct?

@skye I've updated the PR. Summary:
-- I've addressed all but one comment (python code formatting)
-- @galv will help making changes on how we resolve nesting in records, indexing in arrays, key-access in maps. In brief, we'll make the following syntax changes
/ -> .
/[ -> [
[' -> [
'] -> ]
to be consistent with spark avro, databricks avro interfaces.
-- we'll also add namespace support as follows

warn if no namespace is provided and use top-level namespace
fail for wrong namespace

@fraudies Yes you can add a bazel BUILD file for Avro and add the package information in tensorflow/workspace.bzl

Just to confirm, this is ready to review minus formatting?

Jenkins test this please.

@skye Don't bother reviewing this again yet.

Nagging Assignee @skye: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@skye This is ready for review again. Summary of changes

Addressed all review comments (including python code re-formatting using yapf)
Changed syntax to align closer with spark API; / -> ., .[ -> [
Added a switch to configure.py; To compile with avro support the user must point to the avro lib install on the system

We'll leave the full support of namespaces for later.  We'll introduce this feature in a backward compatible way (essentially the use of no namespace as of now will be supported as well)

Awesome!
Quick overview Q: I see the Avro C library is still part of this commit, is this intentional? Like is the idea to point configure to this internal copy of the library? I would prefer not to check it in I think, but you've probably thought about this more than I have.

Nagging Assignee @skye: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi sorry, I was waiting for further comments and then forgot to follow up. I'll review the rest of it now.

Was asked to report a query regarding the PR here - https://github.com/tensorflow/tensorflow/issues/22161#issue-358273520

I suck at using github... if it's not too much trouble, can you resolve any comments that you've addressed? I always have trouble knowing when this is ready to review. I'm also open to other suggestions for how to figure this out :)

Nagging Assignee @skye: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 41 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@fraudies wondering if you still need this PR , if yes can you please resolve conflicts

@rthadur My plan is to merge an updated implementation of this avro dataset into tensorflow.io see PR https://github.com/tensorflow/io/pull/139.  It might be helpful for reviewers to go back to this PR.

@rthadur My plan is to merge an updated implementation of this avro dataset into tensorflow.io see PR tensorflow/io#139. It might be helpful for reviewers to go back to this PR.

@fraudies thank you , i will close this PR.


Web Service Key not accessible

[Enter feedback here]
The instructions are not clear.
You must specify that in order to access the generated key, you must login with that account otherwise you will see only asterix.

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

ID: f0617e3b-c3af-7f9e-ec75-6f7c69866287
Version Independent ID: c6bf5953-2917-9d3d-8dfc-29a2f53b3a1c
Content: Web Services Authentication - Business Central
Content Source: dev-itpro/webservices/Web-Services-Authentication.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

Hello, Thanks for the valuable feedback. We've made changes to the article to make this clear. The update will be published soon.

Hello, Thanks for the valuable feedback. We've made changes to the article to make this clear. The update will be published soon.


Card duplication when loading existing Archipelago save

The Problem
When I start the game and load an existing archipelago save and continue the game ("continue" from the main menu), I notice that I am getting duplicate cards like Caged Wolf and Shunted Wolf. This is every time I load the AP save so if I need to restart the game because of a connection error, I would get a new set of duplicates. I only tested this on Act 1, but it may affect the other acts too.
To Reproduce

Start the game and create a new Archipelago game
From the main menu choose "Continue"
Quit the the game
Restart the game and load the existing Archipelago game. Note the logs of reapplying certain items
From the main menu choose "Continue"
Observe the deck

Screenshot:

Log:
Note: For privacy sakes of the other players I was playing with, I substituted their names with unique numbered names.
[Message:   BepInEx] BepInEx <IP_ADDRESS> - Inscryption (10/27/2023 10:58:50 AM)
[Info   :   BepInEx] Running under Unity v2<IP_ADDRESS>37584
[Info   :   BepInEx] CLR runtime version: 4.0.30319.17020
[Info   :   BepInEx] Supports SRE: False
[Info   :   BepInEx] System platform: Bits64, Windows
[Message:   BepInEx] Preloader started
[Info   :   BepInEx] Loaded 1 patcher method from [BepInEx.Preloader <IP_ADDRESS>]
[Info   :   BepInEx] 1 patcher plugin loaded
[Info   :   BepInEx] Patching [UnityEngine.CoreModule] with [BepInEx.Chainloader]
[Message:   BepInEx] Preloader finished
[Message:   BepInEx] Chainloader ready
[Message:   BepInEx] Chainloader started
[Info   :   BepInEx] 1 plugins to load
[Info   :   BepInEx] Loading [ArchipelagoMod 0.2.0]
[Message:   BepInEx] Chainloader startup complete
[Info   :   Console] Searching for compatible XInput library...
[Info   :   Console] Found Xinput1_4.dll.
[Info   : Unity Log] Received stats and achievements from Steam

[Warning: Unity Log] Font size and style overrides are only supported for dynamic fonts. Font 'Marksman (Crisp)' is not dynamic.
[Info   :ArchipelagoMod] Connecting to archipelago.gg:53880 as JKL-Inscryption...
[Warning:ArchipelagoMod] Item ID 147038 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147019 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147031 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147024 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147021 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147019 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147001 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147012 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147020 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147029 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147003 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147005 didn't apply properly. Retrying...
[Message:ArchipelagoMod] JKL-Inscryption (Team #1) playing Inscryption has joined. Client(0.4.1), [].
[Message:ArchipelagoMod] Now that you are connected, you can use !help to list commands to run via the server. If your client supports it, you may have additional local commands you can list with /help.
[Info   : Unity Log] Test save file:
[Info   : Unity Log] DiskCardGame.StoryEventsData
[Info   :ArchipelagoMod] DeathLink is set to False
[Message:ArchipelagoMod] Received Holo Pelt from [redacted1]
[Message:ArchipelagoMod] Received Epitaph Pieces from [redacted2]
[Message:ArchipelagoMod] Received Nano Armor Generator from [redacted2]
[Message:ArchipelagoMod] Received Mycologists Holo Key from [redacted2]
[Message:ArchipelagoMod] Received Bone Lord Femur from [redacted1]
[Message:ArchipelagoMod] Received Epitaph Pieces from [redacted1]
[Message:ArchipelagoMod] Received Stunted Wolf Card from [redacted1]
[Message:ArchipelagoMod] Received Extra Candle from [redacted1]
[Message:ArchipelagoMod] Received Monocle from [redacted1]
[Message:ArchipelagoMod] Received Dock's Clover Plant from [redacted1]
[Message:ArchipelagoMod] Received Skink Card from [redacted1]
[Message:ArchipelagoMod] Received Caged Wolf Card from [redacted3]
[Info   : Unity Log] Test save file:
[Info   : Unity Log] DiskCardGame.StoryEventsData

My YAML:
Inscryption:
  progression_balancing: 50
  accessibility: items

  # false: Only die when you die (vanilla).
  # true: When other players using deathlink die, you die as well.
  deathlink: 'false'

  # 0: Send a death when sacrificed by Leshy.  Receiving a death will extinguish a candle.
  act1_deathlink_behaviour: 0

  # 0: Complete each act and the epilogue in order. You can return to previously completed acts.
  # 1: Complete each act in any order, then the epilogue. All acts are available from the start.
  # 2: Complete Act 1 by finding the New Game button. Great for a smaller scale randomizer.
  goal: 1

  # false: No code randomization (vanilla)
  # true: randomizes certain codes and passwords so that you can't unlock everything by memory.
  randomize_codes: 'false'

  # 0: No deck randomization (vanilla).
  # 1: Randomize cards within their type on each encounter (Common/Rare, only works on act 1 for now).
  # 2: Randomize cards completely on each encounter.
  randomize_deck: 0

  # 0: No ability randomization (vanilla).
  # 1: Randomize modded abilities only (like those gained from sacrifices).
  randomize_abilities: 0

  # 0: Make a death card on every act 1 death (vanilla).
  # 1: Gain the option to skip death cards on every act 1 death.
  # 2: Gain the option to skip death cards only when dying from death link in act 1.
  optional_death_card: 0

  # false: Play through the tutorial (vanilla).
  # true: Skip the first few tutorial runs. Bones are available from the start.
  skip_tutorial: 'true'

  # 0: Randomizes all nine pieces as their own item.
  # 1: Randomizes pieces in groups of three.
  # 2: Group all nine pieces as a single item.
  epitaph_pieces_randomization: 1

description: 'Template file from https://github.com/DrBibop/Archipelago_Inscryption/releases'
game: Inscryption

# Change this to your own username
name: JKL-Inscryption

APWorld sha256:
b7f3a5e99fcef9d1668818b08e86f34f9d32ef34754264a20e44265e2cf42dd3  inscryption.apworld
If there is anything extra that is needed, feel free to reach out.

Looks like an issue with the save file. Usually, if you receive items, it shouldn't have to reapply again if everything saved correctly. I'll try to reproduce those steps.

Looking back at our code, there is a possibility that the save file is being loaded too late which makes the mod think some items weren't applied properly.
Try downloading this version and see if it fixes the issue: https://we.tl/t-MG8s8k9DH7
In the mod manager, go to "Settings" > "Locations" > "Browse profile folder". In the folder that appears, go to "BepInEx\plugins\Ballin_Inc-ArchipelagoMod" and put the downloaded file there.

It does seem to be fixed. Though, I noticed in the log I am still being reapplied Epitaph Pieces. It does not seem like there a problems with duplication with those though.
Log:
[Message:   BepInEx] BepInEx <IP_ADDRESS> - Inscryption (10/27/2023 10:58:50 AM)
[Info   :   BepInEx] Running under Unity v2<IP_ADDRESS>37584
[Info   :   BepInEx] CLR runtime version: 4.0.30319.17020
[Info   :   BepInEx] Supports SRE: False
[Info   :   BepInEx] System platform: Bits64, Windows
[Message:   BepInEx] Preloader started
[Info   :   BepInEx] Loaded 1 patcher method from [BepInEx.Preloader <IP_ADDRESS>]
[Info   :   BepInEx] 1 patcher plugin loaded
[Info   :   BepInEx] Patching [UnityEngine.CoreModule] with [BepInEx.Chainloader]
[Message:   BepInEx] Preloader finished
[Message:   BepInEx] Chainloader ready
[Message:   BepInEx] Chainloader started
[Info   :   BepInEx] 1 plugins to load
[Info   :   BepInEx] Loading [ArchipelagoMod 0.2.0]
[Message:   BepInEx] Chainloader startup complete
[Info   :   Console] Searching for compatible XInput library...
[Info   :   Console] Found Xinput1_4.dll.
[Info   : Unity Log] Received stats and achievements from Steam

[Warning: Unity Log] Font size and style overrides are only supported for dynamic fonts. Font 'Marksman (Crisp)' is not dynamic.
[Info   :ArchipelagoMod] Connecting to archipelago.gg:53880 as JKL-Inscryption...
[Info   : Unity Log] Test save file:
[Info   : Unity Log] DiskCardGame.StoryEventsData
[Warning:ArchipelagoMod] Item ID 147019 didn't apply properly. Retrying...
[Warning:ArchipelagoMod] Item ID 147019 didn't apply properly. Retrying...
[Message:ArchipelagoMod] JKL-Inscryption (Team #1) playing Inscryption has joined. Client(0.4.1), [].
[Message:ArchipelagoMod] Now that you are connected, you can use !help to list commands to run via the server. If your client supports it, you may have additional local commands you can list with /help.
[Info   :ArchipelagoMod] DeathLink is set to False
[Message:ArchipelagoMod] Received Epitaph Pieces from [redacted2]
[Message:ArchipelagoMod] Received Epitaph Pieces from [redacted1]
[Info   : Unity Log] Test save file:
[Info   : Unity Log] DiskCardGame.StoryEventsData

Does it reapply everytime like the previous times?

They do reapply each time, and there are indeed six pieces each time as intended.

It might be our item verification code that's wrong. I'll take another look.

The issue was found and fixed with this commit.
The item verification step was taking place before options were retrieved so the mod didn't know you chose to group epitaph pieces yet. The good news is this doesn't harm anything in any way apart from getting the notification everytime you connect. We'll post an official update with those fixes sometime soon after we fix something else related to deathlink. Thanks for reporting those issues!


Isolate certain images on certain hosts.

This implements a [hosts] <=> [images] mapping in the simple scheduler
that partitions your host resources into the part that services a
particular image set, and the general cloud.  This is useful, for
example, if you want to specify a set of hosts to run utility VMs
(cloudpipe, bastion, etc) that you don't want consuming resources from
your generally available pool.
When specifying a host with --isolated_hosts flags (comma-separated
list) those hosts will only run the images specified in
--isolated_images, and will not run any other images.  The isolated
images will not run on any other hosts.
You can specify --skip_isolated_core_check to allow overcommitting of
the isolated hosts.  This allows utility vms that are not cpu bound to
avoid the resource cheks the scheduler usually performs (based off of
--max_cores).
Change-Id: Ib2db5a605cb7560a169af9ff2a6dadb649da9c1d

Sorry for everyone who got notified.... oops.


react applications not loading

Describe the bug
previous react project are not working when reopen
Link to the blitz that caused the error
https://stackblitz.com/~/github.com/kushanz/REACTAPP?file=src/components/Examples.jsx:L63
Steps to reproduce
previous react project are not working when reopen
Expected behavior
previous react project are not working when reopen
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Version = 1.84.0
Hash = 499be3932f50ab06349575b1c9042f95bc072a8b
WebContainer = adbcd1bf

Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 127
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 71489373,
  "usedJSHeapSize": 62760333,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Additional context
No response

Hey @kushanz. Can you be more clear on what's not working? I opened the project, refreshed it a couple of times and it works fine. The more information we have, the better we can debug it on our end. If WebContainer fails to boot, screenshots of the Chrome Devtools could also be interesting.

i want my system configation route!
This is the value of theft's that i am going through and i can't find the remote system that is responsible for alot more than thefts to force point to Ga military about logistic's and hoyt as there employee


Add more cycleway values that are used commonly

I ran through all the bay area network and found some additional cycleway values that appear to be valid based on the wiki.  Hence this PR to add them.
Below are the other values I found in the bay area network that I believe are tagging mistakes based on reading the wiki, and a suggested change by me. We can make these changes ourselves potentially; it would be nice to improve the data directly. With some additional work I can make a list of all these incorrectly tagged ways.

cyclestreet ==> cyclestreet=yes
sidewalk ==> track
designated ==> bicycle=designated or highway=cycleway
"shouder" ==> shoulder
none ==> no

In addition, I found and want to handle values like "track;lane", but only found this in one place.

Does just adding these to the enum have an effect, or is this preparatory to further changes?
Should "left" and "right" be here? It looks like "cycleway=left" or "cycleway=right" wouldn't be meaningful, but rather it's used like "cycleway:left=track" "cycleway:right=shared_lane" (in this example, the "left" side has a protected bike lane while the "right" side has sharrows). https://wiki.openstreetmap.org/wiki/Key:cycleway

@graue

Does just adding these to the enum have an effect, or is this preparatory to further changes?

The effect is, when we request the Cycleway detail from graphhopper, it will provide these actual text values properly instead of assigning them "other".

Should "left" and "right" be here? It looks like "cycleway=left" or "cycleway=right" wouldn't be meaningful, but rather it's used like "cycleway:left=track" "cycleway:right=shared_lane" (in this example, the "left" side has a protected bike lane while the "right" side has sharrows). https://wiki.openstreetmap.org/wiki/Key:cycleway

Actually both kinds of tagging are in use in the bay area, and both are documented in the wiki.  Here's where left/right/both are documented for tag values generally. The syntactic meaning is that there is a cycleway in the left or right side of the way. For example, bike lanes are usually on the right side of the direction of travel, but sometimes can be on the left (I can think of one location in the wiggle).


Added the Query for Transactions to be used by Rosetta

PR Category: Feature
Added the Storage, DbStorage, and Search  objects for indexing and querying transactions. Also added the appropriate tests.

This PR is not needed.


Message Rewrite

Added MessageAction
Added handling for other MessageTypes
Added more methods to Message
Made Message and MessageEmbed immutable
Added DataMessage as output of MessageBuilder
Improved some events / added more events

I would like to move forward with these changes.

I will review these changes again.

Just wanna note that @TheMonitorLizard would appreciate it when he gets a heads-up when the merge is ready (might not matter anyway as it seems like he is waiting for shardmanager as well)

@DV8FromTheWorld can you give your view on this PR? Some people have been using this branch and seem to have no issues


[VueUiTreemap] Fontsize too big when  use scaleDraw-animation on parent div and enable responsive

bug source:

      <VueDataUi :component="'VueUiTreemap'" :dataset='[{
        name: "Parent1",
        value: 100,
        children: [
            {name: "P1 C1", value: 30},
            {name: "P1 C2", value: 20},
            {name: "P1 C3", value: 10}
        ]
      }]' :config="{responsive: true,userOptions:{show: false}}"/>

look

i konw that
but its not a use problem

It's not a bug either :)

Hi @colaiven
You can manage fontSize in the config:

const config = ref({

    responsive: true,

    userOptions:{show: false},

    style: {

        chart: {

            layout: {

                labels: {

                    fontSize: 12

                }

            }

        }

    }

})

You can also manage your own content through the #rect slot (see documentation page in the slots tab), if this is not enough.

i konw that
but its not a usage question
the question is when i use css animation, you can run my code

There is no css referring to your inline styles in your code.

There is no css referring to your inline styles in your code.

There is no css referring to your inline styles in your code.

first line

...

Yes, but your animation: scaleDraw 1.2s ease-in-out; is not referring to any keyframe animation, you did not provide the keyframes, no nothing to replicate on my side.

Yes, but your animation: scaleDraw 1.2s ease-in-out; is not referring to any keyframes animation, you did not provide the keyframes, no nothing to replicate on my side.

The bug appears in this line
wait me creat a online link for you

Yes, but your animation: scaleDraw 1.2s ease-in-out; is not referring to any keyframes animation, you did not provide the keyframes, no nothing to replicate on my side.

I'm so sorry, I made a mistake, scaleDraw  is my own definition，It's a scale transform.
@keyframes scaleDraw {
  from{transform: scale(0.2)}
  to{transform: scale(1)}
}

online source:
https://stackblitz.com/edit/vitejs-vite-jufprh?file=src%2FApp.vue

You can solve this using the slots:

<template>

    <VueDataUi
      :component="'VueUiTreemap'"
      :dataset="[
        {
          name: 'Parent1',
          value: 100,
          children: [
            { name: 'P1 C1', value: 30 },
            { name: 'P1 C2', value: 20 },
            { name: 'P1 C3', value: 10 },
          ],
        },
      ]"
      :config="{
        responsive: true,
        userOptions: { show: false },
        style: {
          chart: { layout: { labels: { showDefaultLabels: false } } },
        },
      }"
    >
      <template #rect="{ rect, shouldShow, fontSize, isZoom, textColor }">

          {{ rect.name }}: {{ rect.value }}

      </template>
    </VueDataUi>

</template>

Is this a problem？
@graphieros 🫡

Not really.
I intentionally override outside transitions on the components, to avoid interferences with libraries.
This use case is very specific, you rarely put this kind of animation on a chart, but the #rect slot does provide a solution.

Not really.
I intentionally override outside transitions on the components, to avoid interferences with libraries.
This use case is very specific, you rarely put this kind of animation on a chart, but the #rect slot does provide a solution.

okok thanks verymuch😁


HV-1295 Upgrade validation-api to 2.0.0.Alpha2

https://hibernate.atlassian.net/browse/HV-1295

Jenkins, retest this please

Jenkins, retest this please


GitHub:)

GitHub can help with:

transparency
e.g. making scripts, codes public in a public github repo.
reproducibility
e.g. having the commit history can others to use the exact version of a program in reproducing an analysis.

It addresses some of the issues with transparency and reproducibility, but not all so it's not sufficient.

Agreed! 👍 Paired with zenodo, it makes a powerful and citable versioned resource.
Although I have heard talks that Github is becoming too much of a monopoly and that if it ever went under for an extended period of time or was terminated for whatever reason, all benefits are lost.

wow, zenodo is so cool. thank you Greg.

if it ever went under for an extended period of time or was terminated for whatever reason, all benefits are obviously lost.

with this user base, and underlying git's open nature, I doubt that.
as long as you have your repo you can host it somewhere else. if they go down, I suspect bitbucket, others will fill in the void.
having said that with other tools on top, GitHub is more than its core functionality now. and that will take a little longer to develop for sure.

with this user base, and underlying git's open nature, I doubt that.

Yeah, me too, I hope its here to stay!
I think a much more likely scenario where github fails alone is when a github repo associated with a manuscript is deleted, updated, moved, or altered in any way that separates it from the paper submission. Sure a user can roll back commits to the submission date, but not in all situations...

Fair point. How can we make a persistent snapshot of a repo? :thinking:
I love @greenelab's GitHub enthusiasm! 💯

We use github releases tied to a zenodo DOI. This may have some similar limitations though.
There are probably other solutions too!

First time using GitHub with this assignment. Seems pretty user friendly, but haven't yet tried using/commiting to more extensive scripts. Agree that it supports transparency, reproducibility, and collaboration. As mentioned by Greg and Onur above, having a permanent copy of one's work is important for research and may not be provided by Github.

I have found github useful for both collaborations as well as doing my own personal projects. I like the history feature because I tend to change parts of scripts that I sometimes need to have not changed. Looking through the history, it is easy to fix mistakes using Github. However, I am still trying to learn the difference between branches and forks. Like most of the tools listed, there is a learning curve to getting the commands on the command line correct. For example, the difference between a commit and push is subtle.

Github is an awesome tool for colaberation and managing your own script. It could even generate website. However, I feel like for new user, the learning curve is quite steep. You have to keep practice to get use to it.


Update Tracis CI to Github Apps

Current setup using Services is deprecated and Github Apps is the supported path forward.
Also, need to switch to travis-ci.com from travis-ci.org, since the .org one is also deprecated.

This is done and Travis CI has been migrated to using https://travis-ci.com/macvim-dev/macvim and using Github Apps.


[coverity] Fix 3 Jshint error

JSHINT.W018
JSHINT.W033
JSHINT.W058

The file domtestcase.js isn't referenced by any file.

@haoxli, PTAL.

LGTM


MyFaronics.MyAnti-Virus version 4.31.3102.572

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I don't really know how to read the MSI logs, but late in them, even with /norestart manually added, I see:
Property(S): REBOOT = Force

So I think that the MSI will always reboot.  Given that I have created https://github.com/microsoft/winget-cli/issues/2041 and this PR should probably be blocked on that feature.  I don't really know if we can support automated validation with reboots though, but we should probably not treat it as an internal error if we can help it.


getsaleor.com - update storefront screenshots

Updates main storefront page screenshots.

@dominik-zeglen The screenshots should present the English version of the site.


Create new launch screens

We should prepare new launch screens.

@etolstoy лончскрин, состояние загрузки, ошибка загрузки

error1.mp4.zip

Can you send me the gif of the error state?

@etolstoy try this one. If it's too big, I'll try to reduce the size

Great, thanks


Issues with Actor Import (Script Failure on KeyError)

Hi,
Every few months there seems to be an issue with Actor Importing that causes the script to crash. The first time it was Sphynx which I see was updated. Today the issue is warnings / error DRAGNET/OCTANE Panda and a script crash with a Key Error for SAIGA.
Is there a solid solution to make sure that this doesn't keep breaking or some way to make sure the actor list is dynamically updated?
If I am missing something, please let me know. This is a really great tool and it works well for us most of the time.
On the latest update (0.7.3)
[2024-05-08 15:11:33,733] WARNING  processor/thread_7   Adversary OCTANE PANDA missing field first_activity_date.
[2024-05-08 15:11:42,466] WARNING  processor/thread_8   Adversary DRAGNET PANDA missing field first_activity_date.
[2024-05-08 15:11:42,467] WARNING  processor/thread_8   Adversary DRAGNET PANDA missing field last_activity_date.
[2024-05-08 15:11:44,910] WARNING  processor/thread_8   Could not add or tag event ADV-225759 DRAGNET PANDA (People's Republic of China). Will retry in 0.3 seconds.
day is out of range for month: 0
Traceback (most recent call last):
  File "<redacted>/misp_import.py", line 408, in <module>
    main()
  File "<redacted>/misp_import.py", line 387, in main
    importer.import_from_crowdstrike(int(settings["CrowdStrike"]["init_reports_days_before"]),
  File "<redacted>cs_misp_import/importer.py", line 342, in import_from_crowdstrike
    self.actors_importer.process_actors(actors_days_before, self.event_ids)
  File "<redacted>/cs_misp_import/actors.py", line 223, in process_actors
    if fut.result():
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "<redacted>", line 79, in batch_import_actors
    act_detail = Adversary[actor_name.split(" ")[1].upper()].value
  File "/usr/lib/python3.10/enum.py", line 440, in __getitem__
    return cls._member_map_[name]
KeyError: 'SAIGA'

Same here i am also facing this same issue, tried in both production and test instances.
[2024-05-16 18:54:38,510] WARNING  processor/thread_5   Could not add or tag event ADV-224558 SLY SPIDER (eCrime). Will retry in 0.3 seconds.
Error code 500: An Internal Error Has Occurred.
[2024-05-16 18:54:38,790] WARNING  processor/thread_3   Could not add or tag event ADV-209074 TUNNEL SPIDER (eCrime). Will retry in 0.3 seconds.
Error code 500: An Internal Error Has Occurred.
Traceback (most recent call last):
File "/home/ayadav3/MISP-tools/misp_import.py", line 408, in
main()
File "/home/ayadav3/MISP-tools/misp_import.py", line 387, in main
importer.import_from_crowdstrike(int(settings["CrowdStrike"]["init_reports_days_before"]),
File "/home/ayadav3/MISP-tools/cs_misp_import/importer.py", line 342, in import_from_crowdstrike
self.actors_importer.process_actors(actors_days_before, self.event_ids)
File "/home/ayadav3/MISP-tools/cs_misp_import/actors.py", line 223, in process_actors
if fut.result():
File "/usr/lib/python3.10/concurrent/futures/_base.py", line 451, in result
return self.__get_result()
File "/usr/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
raise self._exception
File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/ayadav3/MISP-tools/cs_misp_import/actors.py", line 79, in batch_import_actors
act_detail = Adversary[actor_name.split(" ")[1].upper()].value
File "/usr/lib/python3.10/enum.py", line 440, in getitem
return cls.member_map[name]
KeyError: 'SAIGA'

I have a temporary solution that can solve the error I can share tomorrow, but the deeper issue is extensive use of static enums for data that is dynamic.
in this case, the error is caused because the script is searching for country attribution for SAIGA, but the adversaries enum (not at my computer so will comment tomorrow with full details / file names) does not have a country for SAIGA and it crashes.
A long term solution could be to implement a mechanism (moving away from static enums) to handle new attribution as new data comes in. Rather than just waiting for a new error and then updating the enum.

Okay so temporary solution is to add SAIGA attribution to this enum

tagging dev @jshcodes for visibility

The adversary branch enumerator has been updated and released in version 0.7.4.

@jshcodes is it possible to work out a way where the script wont break when new adversary info come through crowdstrike?


Add user defined labels to snapshot info

Part of #2172

After install: elemental install --debug --snapshot-labels myLabel=foo,myOtherLabel=bar
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T08:43:38Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
efi:
    label: COS_GRUB
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        labels:
            myLabel: foo
            myOtherLabel: bar
        date: "2024-09-09T08:43:38Z"
        fromAction: install
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            active: true
            labels:
                myLabel: foo
                myOtherLabel: bar
            date: "2024-09-09T08:43:38Z"
            fromAction: install

After upgrade: ELEMENTAL_UPGRADE_SNAPSHOT_LABELS="myLabel=bar,myOtherLabel=foo" elemental upgrade --debug --system oci://<IP_ADDRESS>:30000/elemental-os:dev-next
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T08:45:57Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
efi:
    label: COS_GRUB
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        labels:
            myLabel: foo
            myOtherLabel: bar
        date: "2024-09-09T08:43:38Z"
        fromAction: install
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            labels:
                myLabel: foo
                myOtherLabel: bar
            date: "2024-09-09T08:43:38Z"
            fromAction: install
        2:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:248b22b3dc24b730d1487df31c32fb527c40a95c74ea69c2d233d877a4e81d67
            active: true
            date: "2024-09-09T08:45:57Z"
            fromAction: upgrade

After upgrade+recovery: ELEMENTAL_UPGRADE_SNAPSHOT_LABELS="myLabel=bar,myOtherLabel=foo" elemental upgrade --debug --system oci://<IP_ADDRESS>:30000/elemental-os:dev-next --recovery
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T08:50:48Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
efi:
    label: COS_GRUB
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        date: "2024-09-09T08:50:48Z"
        fromAction: upgrade
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            labels:
                myLabel: foo
                myOtherLabel: bar
            date: "2024-09-09T08:43:38Z"
            fromAction: install
        2:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:248b22b3dc24b730d1487df31c32fb527c40a95c74ea69c2d233d877a4e81d67
            date: "2024-09-09T08:45:57Z"
            fromAction: upgrade
        3:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:248b22b3dc24b730d1487df31c32fb527c40a95c74ea69c2d233d877a4e81d67
            active: true
            date: "2024-09-09T08:50:48Z"
            fromAction: upgrade

After upgrade recovery only: elemental upgrade-recovery --debug --snapshot-labels myLabel=foobar --recovery-system.uri oci://<IP_ADDRESS>:30000/elemental-os:dev-next
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T08:54:49Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
efi:
    label: COS_GRUB
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        labels:
            myLabel: foobar
        date: "2024-09-09T08:54:49Z"
        fromAction: upgrade-recovery
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            labels:
                myLabel: foo
                myOtherLabel: bar
            date: "2024-09-09T08:43:38Z"
            fromAction: install
        2:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:248b22b3dc24b730d1487df31c32fb527c40a95c74ea69c2d233d877a4e81d67
            date: "2024-09-09T08:45:57Z"
            fromAction: upgrade
        3:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:248b22b3dc24b730d1487df31c32fb527c40a95c74ea69c2d233d877a4e81d67
            active: true
            date: "2024-09-09T08:50:48Z"
            fromAction: upgrade

After reset (from recovery partition): elemental reset --debug --snapshot-labels myLabel=foobar,myOtherLabel=barfoo
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T08:56:44Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        labels:
            myLabel: foobar
        date: "2024-09-09T08:54:49Z"
        fromAction: upgrade-recovery
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            active: true
            labels:
                myLabel: foobar
                myOtherLabel: barfoo
            date: "2024-09-09T08:56:44Z"
            fromAction: reset

FYI the documentation regarding labels is missing, since the end-user is not going to be able to conveniently use them.
I'll add full documentation about labels with the elemental state feature.

Noticed a bit late the environment variables were always empty, that needed a custom decoder, it's better now:
ELEMENTAL_UPGRADE_SNAPSHOT_LABELS="myLabel=bar,myOtherLabel=foo" elemental upgrade --debug --system oci://<IP_ADDRESS>:30000/elemental-os:dev-next
# Autogenerated file by elemental client, do not edit

date: "2024-09-09T12:33:25Z"
snapshotter:
    type: btrfs
    max-snaps: 4
    config: {}
efi:
    label: COS_GRUB
oem:
    label: COS_OEM
persistent:
    label: COS_PERSISTENT
recovery:
    label: COS_RECOVERY
    recovery:
        source: dir:///run/rootfsbase
        fs: squashfs
        labels:
            myLabel: foo
            myOtherLabel: bar
        date: "2024-09-09T12:31:50Z"
        fromAction: install
state:
    label: COS_STATE
    snapshots:
        1:
            source: dir:///run/rootfsbase
            labels:
                myLabel: foo
                myOtherLabel: bar
            date: "2024-09-09T12:31:50Z"
            fromAction: install
        2:
            source: oci://<IP_ADDRESS>:30000/elemental-os:dev-next
            digest: sha256:11cb5c6f7b6b9e4daff67ec3ec7fa4a028c24445f2e4834a78e93c75d73eb5c3
            active: true
            labels:
                myLabel: bar
                myOtherLabel: foo
            date: "2024-09-09T12:33:25Z"
            fromAction: upgrade


Debug npm sometimes failing build

Seems to be an issue with Travis CI having a dirty instance. Sometimes, there's a lock even though we're starting a fresh build instance.
Workaround for now is just restarting the build.

Can confirm, this is not our problem.
Would recommend closing.


CI workflow

Description
CI for the repo.
Todos
Notable points that this PR has either accomplished or will accomplish.

[ ] CI tests should pass

Status

[ ] Ready to go

Codecov Report

Merging #6 into main will not change coverage.
The diff coverage is n/a.


Uncaught TypeError: undefined is not a function

SerialPort Version:4.0.7

NodeJS Version:6.9.2

Operating System and Hardware Platform:win10 x64 electron 1.4.15 ia32

Have you checked the right version of the api docs?:yes

Are you having trouble installing and you checked the Installation Special Cases docs?nO

Are you using Electron and have you checked the Electron Docs?:yes

Summary of Problem
When i unplug serialport device when my app runing ,console occur belows error in cycle:
Uncaught TypeError: undefined is not a function

i can't debug it,because it not stop.

I think maybe serialport unlimited try connect the port after i unplug device,i try to on('close')  and on('disconnect') but not tigger.

@reconbot thks reply! i update to 5.0.0-beta3,it fixed!


Create Loras folder

Add the default loras folder in models, otherwise it is not apparent where to put the loras, as they are seperated out now versus the original repo behavior of putting them all in the same folder.

Right didn't realize it wasn't created. I'll make the merge. Thanks!


Consider safer api for BlockHash.to_string

          > Using `.to_string` on `BlockHash` does the right thing independent of underlying order of bytes

This will still be a lurking footgun in the future. Perhaps out of scope for this quick fix, but it's worth considering a safer api so we don't need to know when .to_string is needed (e.g. BlockHash) and when .encode_hex::<String> is appropriate.
Originally posted by @bradleystachurski in https://github.com/fedimint/fedimint/issues/5813#issuecomment-2276982302

Perhaps we should impl encodable/decodable for BlockHash similar to bitcoin::Txid, then use .consensus_encode_to_hex. Would be great if we can also prevent .encode_hex::<String> with semgrep.
8be68a6 (#3994)

From call: we should get rid of this encode_hex since it assumes the internal byte representation is correct. Should remove hex crate dependency and switch to conservative_hex used by rust bitcoin. Forces you to implement ToHex and guarantee correctness.


Error popups upon opening Poedit

After upgrading 2.0beta4 to 2.0 stable and opening Poedit, I received this error:

Poedit seems to operate normally after confirming the error message.
Renaming "Vaclav Slavik" in HKLM\Software\Vaclav Slavik to "Vaclav Slavik1" and reopening the attached POT file fixed it (of course by recreating "Vaclav Slavik" with "fresh" settings).
Also, in the same session where I see this error, changing the Poedit's lang to hebrew and restarting it (by opening the attached POT file [opening Poedit from scratch doesn't trigger this error]) doesn't actually change it to hebrew (and after the same restart, no error message is apparent. After a second restart the error message appears again).
Attached HKLM\Software\Vaclav Slavik1 registry key (don't worry about the "1" at the end of the "Slavik", it was just to backup the old settings, license and stuff for Poedit to recreate "Vaclav Slavik").
I see other error messages such as these, when opening the attached POT file:

Download link for both POT file and reg file containing the reg folder:
http://www9.zippyshare.com/v/HsG0dSZh/file.html

I have the same problem under Linux. I'm maintaining the Fedora package for a while. Today I wanted to update to v2.0, and when opening a po file from the file manager, I get the same error message as described above. Here's the backtrace, as copied from the error window:
ASSERT INFO:
./src/gtk/dataview.cpp(211): assert "gtk_tree_selection_get_select_function(m_selection) == func" failed in CheckCurrentSelectionFunc(): selection function has changed unexpectedly, review this code!
BACKTRACE:
[1] wxDataViewCtrl::DoSetCurrentItem(wxDataViewItem const&)
[2] wxEvtHandler::TryHereOnly(wxEvent&)
[3] wxEvtHandler::ProcessEventLocally(wxEvent&)
[4] wxEvtHandler::ProcessEvent(wxEvent&)
[5] wxEvtHandler::ProcessPendingEvents()
[6] wxAppConsoleBase::ProcessPendingEvents()
[7] wxApp::DoIdle()
[8] g_main_context_dispatch
[9] g_main_loop_run
[10] gtk_main
[11] wxGUIEventLoop::DoRun()
[12] wxEventLoopBase::Run()
[13] wxAppConsoleBase::MainLoop()
[14] wxEntry(int&, wchar_t**)
[15] __libc_start_main
The system is Fedora 25 with wxGTK3-3.0.2. By the way, I can open Poedit from the application menu without problems. But after opening a po file, the same error will be issued.

I have the same problem under Linux

No, you don't, this has no relation whatsoever. As a package maintainer, you ought to understand that "a window is shown" being the only similarity doesn't make it the same issue.
Please file a separate issue. Please also provide more accurate information on how to reproduce — there's at least two ways to "open a PO file" and you didn't provide the file. As I'm sure you noticed, the backtrace is useless because you don't have debug symbols.

Can you please attach the file directly here?  Just drag the ZIP to the comment area.

Never mind, managed to download it with wget (!), here it is:
files.zip
FYI, attempts to download it from the site, using the only download link, which looks like this:

result in downloading apparent malware, Your File Is Ready To Download.dmg, on Mac, and opening adds in Edge on Windows. It's my understanding now that it used to be a legitimate place, but it is painfully obvious it no longer is.

As a workaround until it is fixed:

Restore the settings from your reg file.
Delete everything under license.
Re-enter your key.
This is a multi-threading issue that only affects upgrade from betas to 2.0 final. I tried to upgrade beta's lic settings automagically so that I don't bother you with re-entering the key.
Wow sorry about that, now that I try to download the file from there I receive many popups as well. Too bad :(

Just drag the ZIP to the comment area.
Yeah, I tried that at first, but the text editor here claimed it can't accept the file for some reason ("We don’t support that file type"), even though it was a valid ZIP file.

As a workaround until it is fixed:
Restore the settings from your reg file.
Delete everything under license.
Re-enter your key.

Will I be able to verify this is fixed in the next version if I do this? (I have a license for 1.x, so I'm not sure if upgrading the key to 2.0 will make the issue reproducible then)

Will I be able to verify this is fixed in the next version if I do this?

No, but it doesn't really matter, this issue affected only low tens of people (and for those who upgraded, which because the beta is expiring soon, will be most of them, the damage is already done).

I tried to remove license and add again with serial number.
Nothing changed.
Always popup error.
It seems that appear if I disable the side bar on the right.
Sometimes when I get the error message if I close and re-run Poedit with same .po file it works without error.

I have some news after removing the registry folder (to start Poedit with "fresh" settings) & registering Poedit with v2 license:

The issue persists and it behaves exactly the same as my first comment here.
Quoting @bovirus:

Sometimes when I get the error message if I close and re-run Poedit with same .po file it works without error.

Same here.
3. Quoting @bovirus again:

It seems that appear if I disable the side bar on the right.

Hiding the side bar doesn't make the popups go away, but I've noticed that closing and reopening Poedit after hiding the sidebar doesn't always preserve this setting (that hides the side bar). Same for other settings. I've disabled the "Automatically compile MO file when saving" and again- Poedit intermittently doesn't preserve this setting.
I think this is related to this issue and fixing the error messages would fix that, but @vslavik if you think it's not related tell me and I'll open another issue for that.

This is a race condition issue; the apparent randomness is part of it.

Can you give this build a bit of testing please: https://jumpshare.com/v/9KjthsXbHiJ7KMyiRsns ?
It seems to be fine here (doing the same steps as described here, which I was able to reproduce the bug with), but due to the uncertain nature of the bug, I'd appreciate your confirmation.

Yep, seems to be fixed.
I've verified it by opening the previously mentioned PO file a few dozens of times with the "fresh" registry settings from my previous comment and with the "old" reg settings (the one I had when I've upgraded 2.0beta to 2.0 stable, with the 1.0 license)- in both cases no popup messages are shown.
Any more specific testing you'd like me to try?

Nope, just the same fiddling with it that you and @bovirus did before -- I figured you have a better sense of the bug than I do.
Thanks a lot for your help with this (and of course reporting it in the first place)!


NOTICE ⚠: GSSOC EXtended Program Manager

Hi @mansiruhil13,
Please add me as a collaborator in your repository as soon as possible!!
We have noticed spam practices in your repository involving contributor @4F24L, which is unacceptable. Can you please justify this?
The contributor, who is a collaborator on your repo, is self-assigning tasks, adding high-level labels for minor tasks, and removing lower-level labels. This behavior is considered spam.
As a result, we may need to remove all scores for this contributor from the repository or ban contributor from the leaderboard.
27 Merged PRS with such High level of spamming with unfair self labeling

Self assigning issues with high levels

Level 3 for such minor changes

Removing lower levels and adding hih=gher ones

We will be removing all following PR scores to zero due to such practices. @mansiruhil13, please provide an explanation promptly. Why did you add this contributor to your repository, and why are these practices occurring? please explain

Hello @MastanSayyad ,
I sincerely apologize for any confusion that may have arisen. I want to clarify a few points:
The majority of the labels in my PRs are genuine and not intended to artificially increase levels.
I was added as a collaborator specifically to assist with the deployment of the website, which I successfully helped complete.
I have only self-assigned the issues that I personally raised. In this repository, it's standard practice for issues to be assigned to the person who raised them.
If necessary, I fully understand if you need to adjust the level labels or dismiss me as a collaborator. However, I kindly request that the PRs I submitted remain, as I've put significant effort into them.
Thank you for your understanding, and I’m open to discussing this further if needed.
Best regards,
4F24L

Additionally, all other PRs in this repository are labeled similarly. You can see that the first issues were labeled by the admin, and almost every issue and PR has been given level 2 or 3, with level 1 being quite low in count. Based on this pattern, I labeled mine accordingly.
I fully respect the repository's guidelines and did not intend to bypass any rules. My goal has always been to contribute meaningfully, and I appreciate your time in reviewing this and helping to find a fair resolution.
Furthermore, I had requested the admin to handle labeling, issue assignment, and PR merging. In their absence, I have been taking on these responsibilities to help maintain the flow of the project.

This aint a collaboration , @MastanSayyad get the access to the repo,
if admin not giving access ban the project.
We cant sacrifice quality on program

@mansiruhil13 I haven't heard anything from you yet? kindly reply, are you giving labels without reviewing the code? as I have checked there are so many PRs where minor changes have level 2 or level 3, if you have any confusion please ask us and keep an eye on PAs channel on discord

@4F24L we need some response, this affecting the quality of the program,

@sanjay-kv
I have already explained everything to @MastanSayyad, and as a result, he has removed the labels from all the PRs, following our discussion.
Additionally, I would like to inform you that I am no longer contributing to this repository and have shifted my focus to contributing to other repositories. If there is anything further required from my side regarding this matter, please let me know.


doc: replace tracking issues

In fact, it may be more appropriate to use milestones rather than tracking issues at all.
(Source: https://github.com/danth/stylix/pull/325#issuecomment-2035527982)

The tracking issue format is inspired by Rust's project management. Unfortunately, I was unable to find an explicit clarification on why they have not adopted the relatively new GitHub Milestones or GitHub Projects formats yet.
IMHO, project management tools beyond (tracking) issues and PRs are overkill and a waste of time in nearly every single-developer side project. However, Stylix is slowly attracting more contributors, and the number of feature, bug, refactor, and performance requests increases likewise. Despite the project having "only" 79 open issues (at the time of writing), it would take an enormous amount of time (in FOSS context) to figure out what overall direction the project is taking due to conflicting or very similar requests. This makes it harder to understand and discuss the overall optimal direction of the project.
I suggest adopting GitHub Projects or GitHub Milestones for Stylix, despite having no practical experience with them yet.
A somewhat elaborate comparison of GitHub Milestones and GitHub Projects can be found at: https://stackoverflow.com/questions/39591795.
Considering Stylix's primarily rolling release development model, I cannot think of any good examples for using GitHub Milestones, other than major (breaking) changes that end-users and developers may want to look forward to; although it would in most cases merely duplicate the GitHub Projects and waste additional effort in project management. As pointed out earlier, it might be beneficial to replace every currently existing tracking issue with an individual GitHub Project.

As pointed out earlier, it might be beneficial to replace every currently existing tracking issue with an individual GitHub Project.

Upon reconsideration, I prefer a simpler development environment similar to the Linux Kernel email process. The GitHub idiomatic equivalent would be issues and PRs.

I agree, there is no problem with the way things are organised right now.


Fix __api_url_params method

Use "&" as concatenator in api_url if it already has params

Hi @man-c! Please check this PR, I just tried to use library, found this small issue and decided to fix it.
Would love to continue to use this lib.
Cheers

Hi @gkrasulya and thanks for contibuting!
Could you please provide an example of this issue?
Cheers

Hi!
I was checking for exactly that.
If you add some optional arguments to a request that has required ones, you end up having 2 '?' in the url.
Here is an example with the get_market_chart_by_id function, which requires 3 args, but has 1 optional, the resulting url when adding this optional arg is :
'https://api.coingecko.com/api/v3/coins/bitcoin/market_chart?vs_currency=usd&days=max?interval=daily'
Could you please merge this PR ?
And thank you both for your work by the way ^^

Thanks for the feedback!
I will run some tests and merge the PR.


Fix CTE subqueries not finding parent bindings

Continuation of https://github.com/elixir-ecto/ecto/pull/3642
Both PRs are an attempt at fixing the generation of the query from issue https://github.com/elixir-ecto/ecto/issues/3619
Tested using the provided repo https://github.com/joaopaulobdac/parenting
19:02:13.791 [debug] QUERY OK source="groups" db=0.8ms decode=0.6ms queue=35.3ms idle=0.0ms
SELECT g0."name", s1."names" FROM "groups" AS g0 LEFT OUTER JOIN LATERAL (WITH RECURSIVE "breadcrumbs" AS (SELECT ssg0."id" AS "id", ssg0."name" AS "name", ssg0."supergroup_id" AS "supergroup_id" FROM "groups" AS ssg0 WHERE (ssg0."id" = g0."id") UNION ALL (SELECT g0."id", g0."name", g0."supergroup_id" FROM "groups" AS g0 INNER JOIN "breadcrumbs" AS b1 ON b1."supergroup_id" = g0."id")) SELECT array_agg(sb0."name") AS "names" FROM "breadcrumbs" AS sb0) AS s1 ON TRUE []
[
  %{name: "Supergroup", query_test: ["Supergroup"]},
  %{name: "Subgroup 1", query_test: ["Subgroup 1", "Supergroup"]},
  %{name: "Subgroup 1.1", query_test: ["Subgroup 1.1", "Subgroup 1", "Supergroup"]},
  %{name: "Subgroup 1.1.1", query_test: ["Subgroup 1.1.1", "Subgroup 1.1", "Subgroup 1", "Supergroup"]},
  %{name: "Subgroup 1.1.2", query_test: ["Subgroup 1.1.2", "Subgroup 1.1", "Subgroup 1", "Supergroup"]}
]

Looks great! We just need some tests and this should be good to go! Please look at the test/ecto/adapters folder.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Fix #434: Remove obsolete TODO section in README

Update the link to the auto-generated versions of the P4Runtime specification on the P4.org web site.
Update the section "P4 Language Version Applicability" to version 1.2.4 of the P4_16 language specification, but list 3 known exceptions of features that have not been explicitly addressed yet.

Intended to resolve issue https://github.com/p4lang/p4runtime/issues/434
Also to explicitly document that we are NOT yet resolving the issues mentioned here: https://github.com/p4lang/p4runtime/issues/424

Note to everyone who reviewed and approved commit 1 of this PR: I have just added commit 2, which adds 7 more items to the bullet list in the language version compatibility section for features new in P4_16 language spec version 1.2.4 that might lead to future changes to the P4Runtime API specification to address compatibility.  There are already open issues in the https://github.com/p4lang/p4runtime repo to remind us of each of them, and I would recommend that we try to remember to remove these bullet items from the text in this language compatibility section as we believe these compatibility issues have been resolved.

Thanks @jafingerhut for the follow-up content!


Can you provide pre-trained weights for RefCOCO, RefCOCO+, and RefCOCOg?

It seems that the current released weight is for gRefCOCO.
Can you provide pre-trained weights for RefCOCO, RefCOCO+, and RefCOCOg?
It will be helpful for my research.
Thanks.

Hi,
The current release is for gRefCOCO only. We will release the checkpoint and config for refcoco soon. Please stay tuned!


Ionic 4 Support

Support Ionic 4. Currently it's only for Ionic 3.
Maybe this component will be a native Angular component or native html to support all enviroments.

State-Stepper is now a Web-Component. Read more


Sometimes doubled stream on airplay

What version of Music Assistant has the issue?
2.1.0b6
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Sometimes after adding a track to the queue and playing it, the sound is duplicated with a certain delay. I have attached a recording from a voice recorder. The issue is repeatable, but hard to replicate consistently. Sometimes the queue starts correctly, and sometimes it doesn't. Once it does start, every track plays correctly. The problem persists regardless of the MA version, and has been occurring for at least a few months. DLNA works well, but sometims 'freezes' and doesn't work till amplifier reset, but that's a different issue.
How to reproduce
Just play track through airplay
Music Providers
Tidal, Remote Share, YT Music
Player Providers
Problem only appears on Airplay
Full log output
MA log.txt
https://github.com/music-assistant/hass-music-assistant/assets/94012672/8506e327-f450-495d-a165-47ed4f54f99c
Additional information
No response
What version of Home Assistant Core are your running
2024.6.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

So I take it you are streaming direct to the Onkyo Amp?
P.S. Take a moment to fix your tags as well to avoid other unrelated problems!

You have a very old version running.
Can you start by updating to the latest stable or beta version ? We have had lots of bugfixes last couple of days and I fixed 2 for Airplay that may be related to this issue,

Yes. Directly to onkyo amp.
Tagless tracks have to be tagged manually. This is low priority on my to-do list :)
Version is dated 14.6.2024. 2.1.0b6. Log is short because it was generatad after recent update.Just didn't check after copy-paste. My bad.

OK, We will need to reproduce this issue. Any clues when it happens ?

Do you use it with automations perhaps ?
Does it happen (only) when the receiver is grouped with other players or does that not matter ?

No. I've tried to find any pattern, but no luck.

No, i don't use automations. Just using MA interface.
This is the only player i use. No groups.

It looks like this:

Hit any track to play and it just happens sometimes.
I need to clear queue and play different track. If it's ok, then i can back to the track played before. If not, repeat the step.
If it's ok, then it works well through entire queue. I did not find any issues when jumping track-to-track on started queue, so flow mode is not connected with this case i guess.

There hasn’t been anything dedicated to this but could you check beta 15 and confirm this is still an issue.

On 13 it was still there. I'll check 15 when i get home.

I did not find any issues with doubled stream, BUT i did just a few tests, so the sample is not representative.
Anyway my MASS just updated to 2.1

Ok we will need your thoughts on whether this is still an issue. We can close it and reopen later if it comes back?

Sure

Update:
On 2.1.0rc1 problem still exists.
Steps:

One track playing
Clear queue (music stops)
Play another track

In queue there is only "new" track, but music goes from both.

This one should be fixed in 2.1.0 / 2.2.0b0

@FreyrPL Closing assumed fixed. Please advise if not


Add "Fully Supported" and definition to README

What does "fully supported" mean?
This is the official supported-by-Camunda Nodejs SDK for Camunda Platform 8.
What does this mean?
There are two aspects to this:

Technical support - what Camunda will support you with.
Engineering support - what features the SDK supports.

You can get an Enterprise Support agreement with Camunda and get technical support for using this SDK in development and production. With a support agreement, you can open tickets and get technical support on using the features that the SDK implements.
At the moment, we do not guarantee that the SDK will support all features of Camunda Platform 8 immediately upon their release.
Our goal with the SDK to support the features of a minor release within the minor release window. This means, for example, that all 8.5 features are enabled by this SDK before the 8.6 release.
Prioritisation of implementing features is influenced by customer demand for them.

Thanks for initiating this.
Some notes:

Let's please not use the term fully supported because there is no such thing as fully / partly / 20% supported.

Essentially => we support Camunda 8 as specified, where the specification is what we put into the docs.
Following, it would be sufficient to put the node-js SDK into docs.camunda.io and state "The SDK supports the following features / APIS / ...``

You can get an Enterprise Support agreement with Camunda and get technical support for using this SDK in development and production. With a support agreement, you can open tickets and get technical support on using the features that the SDK implements.

That is true, but I don't see the need to add a specific note about this in the docs (again, we support what is specified).

At the moment, we do not guarantee that the SDK will support all features of Camunda Platform 8 immediately upon their release.
Our goal with the SDK to support the features of a minor release within the minor release window. This means, for example, that all 8.5 features are enabled by this SDK before the 8.6 release.

That makes sense, and I think is worth highlighting.
Thanks!


Flutter for Web - Wrong localhost address url http://[::1]:8080/#/

Steps to Reproduce

flutter channel beta + flutter upgrade
Run any Flutter for Web application

Expected results:
Chrome Web browser URL address is:
http://localhost:8080/#/
Actual results:
Chrome Web browser URL address is:
http://[::1]:8080/#/
Details
With "[::1]" address instead of "localhost" it is not possible to use return Url's for testing application on localhost

Fixed by https://github.com/flutter/flutter/pull/49625


alonetone rails version

alonetone is now using Rails 7. (https://github.com/sudara/alonetone)

@lawyer-omer thanks!


feat: support -e as shortened flag for --exclude-namespaces

What this PR does / why we need it
This PR adds support for -e as shortened flag for --exclude-namespaces.
$ kor all --exclude-namespaces ns1,ns2
$ kor all -e ns1,ns2

In additon, an updated output of kor --help was added to README.md.
PR Checklist

[ ] This PR adds K8s exceptions (false positives)
[x] This PR adds new code
[ ] This PR includes test for any new code

Github Issue
None.
Notes for your reviewers
The disparity was found while working on #258.
The -e flag was referenced in https://github.com/yonahd/kor/blob/main/pkg/filters/options.go#L116, but was missing in root.go.

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

Project coverage is 43.37%. Comparing base (da2b1fe) to head (85c77fb).
Report is 1 commits behind head on main.

Files
Patch %
Lines

cmd/kor/root.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #261   +/-   ##
=======================================
  Coverage   43.37%   43.37%
=======================================
  Files          58       58
  Lines        2808     2808
=======================================
  Hits         1218     1218
  Misses       1400     1400
  Partials      190      190

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


Add support for Manta zkAddress & zkAsset

Is your feature request related to a problem? Please describe.
Please integrate Manta zkAddress & zkAsset. Below are their docs:

https://docs.manta.network/docs/guides/MantaSigner
https://github.com/Manta-Network/sdk => you should switch to signer_extension branch
https://www.npmjs.com/package/manta-extension-sdk

Supported view ZK tokens.


Enhancement: Possibility format rss data

Hello, im trying to have a github releases notification channel in my discord.
Sadly, the data comes with little hiccups that I would like to remove, but as now there's no way to do it.
Eg:

Im asking for a tab in the dashboard to allow simple formatting, like regex replace.

Hi there!
After some research, it seems that GitHub RSS feed entries do not contain information about the summary type, so Axobot cannot remove the HTML tags. On the other hand, their content type is correctly described as text/html, so if you don't mind getting longer texts, you should probably use {full_text} instead of {description} in your template.
If this solution doesn't work for you,  I've also added your suggestion to our public ideas list. Feel free to upvote it there!
If there are no further questions, could you please close this ticket? Thanks!


My app keeps crashing when I initials the model with initLlama

So I'm using an emulator that uses Android 12 (API 31, named : S), and whenever I call initLlama inside my app it just crashes. I'm using developement builds btw.
here is my code :
  import * as DocumentPicker from 'react-native-document-picker'
  import {initLlama, LlamaContext} from 'llama.rn'

  ...

  const result = await DocumentPicker.pickSingle({
      type: [DocumentPicker.types.allFiles],
    })

    console.log('result.copyError',result.copyError)

    if(!result.copyError){

      try{
        const mod = await initLlama({
          model: result.uri,
        })

        console.log('I\'m Ready!')
        setModel(mod)
      }catch(err){
        console.log('Error while initializing the model',err)
      }

      console.log('the model was selected succesfully.\nThe file path :',result.uri)
    }else{
      console.log('an Error Occured while selecting the file')
    }

My code goes into the if(!result.copyError) statement and my app crashes at the initLlama function call (I don't get any Alert or error message my app just shuts down).
At first I thought that maybe that has something to do with the model I'm using because I started out with a 3.62GB model, but at the end I used a 322MB model and it still crashed so the model size really isn't a problem, here are the models that I used in this order :

bloom_3b.gguf, 3.62GB
Phi-3.1-mini-4k-instruct-IQ2_M.gguf, 1.22 GB
qwen2-0_5b-instruct-q2_k.gguf, 322MB

I also ended up using the pro-guard rules even thought I didn't even have them unabled to start with, but I did it while staying on the expo managed workflow using This method, maybe I need to do it using native code instead? but it still is something optional so I don't see this fixing the problem.
I'm pretty lost here and I seem to have tested out pretty much every possible solution, So if anyone has an idea of how to fix this or if this is just a bug maybe tied to something like my android version I would appreciate someone telling me what to do.

From my experience using llama.rn, doing it this way always seems to fail. This has two steps:

Copying the model from storage to app cache.
Loading the model.

It seems that for whatever reason, it tends to crash on the copying part. On my project, ChatterUI: https://github.com/Vali-98/ChatterUI, I prompt the user to first import the GGUF model, then load it after it is contained within the app directory. This works far better.
Also proguard rules shouldn't matter in dev builds (I think) as it doesn't minify any code. Also, proguard rules should be added to your app.json file using expo-build-properties: https://docs.expo.dev/versions/latest/sdk/build-properties/ under expo-build-properties.android.extraProguardRules

For whatever reason, llama.rn crashes on the copying part. To solve this on my project, ChatterUI: https://github.com/Vali-98/ChatterUI, I prompt the user to first 'import' the GGUF model which copies the model to a /models dir in the app directory. Then, load it after it is contained within the app directory.

First, Thanks a Lot man! thanks to you I was finally able to initials the model and get it to respond.
I also left a ⭐ on your ChatterUI project 😉.


Valid UK number flagged as invalid

http://checkwhocalled.co.uk/phone-number/01604704451
2.1.5 :003 > Phony.plausible?('+44<PHONE_NUMBER>1')
 => false

The same applies to +44 07771 5862xx.

@weppos Do you have time for a PR?

Closing this issue – just reopen it if you need to. Thanks! :dancer:


Find references for string union member finds all occurences of the string literal, instead of only the union member references

TypeScript Version:  3.9.4

Search Terms: union find references
Code

checkout https://github.com/AlCalzone/node-zwave-js/tree/dafcab951f25419a9cf92df68b0ee98d9c43e350
open src/lib/commandclass/CommandClass.ts
go to line 814
trigger find all references  on the union member "partial"

Expected behavior:
It finds the following references of the union member "partial":

Driver.ts, line 1291
SendDataMessages.ts, line 232
CommandClass.ts, line 814

Actual behavior:

where each of the other occurences is just the literal "partial":
			log.controller.logNode(node.id, {
				endpoint: this.endpointIndex,
				message: `${this.constructor.name}: doing a ${
					complete ? "complete" : "partial"
				} interview...`,
				direction: "none",
			});

Playground Link: not applicable
Related Issues: none found

This is the intended behavior; "partial" in one union and "partial" in another union are the same type and the request was to find all references to that type.

@RyanCavanaugh

"partial" in one union and "partial" in another union are the same type

This part I understand. However the results above are not part of any union, they are just "random" occurences of the string literal.


ASPSP Simulator - move OB data model module and 3rd party dependencies into common projects

Story
As a member of the ForgeRock Open Banking Team
I want to have all the third party dependencies and shared FR dependencies in one place
so that the dependencies are easy to manage and update between releases
Acceptance criteria

All third party dependencies are in securebanking-parent
securebanking-openbanking-aspsp-model has been replaced by securebanking-common-openbanking-uk-datamodel
securebanking-openbanking-aspsp consumes the secure-banking-parent and securebanking-common-openbanking-uk-datamodel

Tasks

[x] Copy latest changes from securebanking-openbanking-aspsp-model to securebanking-common-openbanking-uk-datamodel
[x] Remove securebanking-openbanking-aspsp-model (but keep multi-module structure with single module for now?).
[x] Move third party libraries from securebanking-openbanking-aspsp to securebanking-parent and extend this parent pom instead of spring-boot-starter-parent.

Release Notes
Affected App: securebanking-openbanking-aspsp

Duplicate of #27


feat: Add another GTFS validator and a GTFS-RT->Siri API

add another validator implemented in Rust and a GTFS-RT to SIRI-Lite API.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Requiring signing a CLA for a link contribution seems a bit overkill. And I'm new to those stuff, but requiring my phone number to this seems weird.

@antoine-de Don't worry about a phone number. We have an organizational CLA for all CUTR at USF projects, and since this repo falls under the organization, the CLA exists.

Thanks @antoine-de! Sorry for the delay, I didn't realize you had signed the CLA.

Thanks @antoine-de! Sorry for the delay, I didn't realize you had signed the CLA.


[FEATURE] support for spark/databricks connect

Been experimenting w/ fugue and databricks. My goal is to be able to programmatically alter the target databricks cluster/workspace as needed based on the various types of queries i'm running through fugue.
I've tried databricks-connect 11.3.x and this works fine w/ fugue spark as long as you have a ~/.databricks-connect file with valid databricks endpoint configs... however this does not work well with the need to configure the connection programatically such as doing something like this (which does not work... the underlying config in ~/.databricks-connect` is still utilized.
  self.spark = SparkSession.builder.config("spark.databricks.service.address", "https://adb-xxx.18.azuredatabricks.net/?o=xx#folder/xx") \
                        .config("spark.databricks.service.token", "xx-x")  \
                        .config("spark.databricks.service.clusterId", "x-x-xx") \
                        .config("spark.databricks.service.orgId", "xx") \
                        .config("spark.databricks.service.port", "15001")  \
                        .config("spark.sql.execution.arrow.pyspark.enabled", "true") \
                    .getOrCreate()

fa.fugue_sql_flow(sql_flow_stmt).run(self.spark,self.engine_config)

Next I see that databricks-connect 13.x permits for programatic configuration of the spark session via ENV vars OR the DatabricksSession builder as described here.  When i've tried something like this I get: Fugue execution engine is not recognized (<pyspark.sql.connect.session.SparkSession object at 0x14c8924a0>, {'fugue.workflow.checkpoint.path': '/tmp/da-egress-service/results'}, {}). You may need to register a parser for it.
from databricks.connect import DatabricksSession

spark = DatabricksSession.builder.remote(
  host       = "https://<workspace-instance-name>",
  token      = "<access-token-value>",
  cluster_id = "<cluster-id>"
).getOrCreate()

fa.fugue_sql_flow(sql_flow_stmt).run(self.spark,self.engine_config)

Next per @goodwanghan i tried registering an exec engine register_execution_engine using databricks-connect 13.0.0
from databricks.connect import DatabricksSession
from fugue.execution.factory import register_execution_engine
from fugue_spark import SparkExecutionEngine
from pyspark.sql.connect.session import SparkSession

register_execution_engine(SparkSession,lambda session, conf: SparkExecutionEngine(session, conf))

self.dbxsession = DatabricksSession.builder.remote(
    host       = "https://xxxx-xxxx.azuredatabricks.net/?o=xx#folder/xx",
    token      = "xxx",
    cluster_id = "xxxxxx"
).getOrCreate()

fa.fugue_sql_flow(sql_flow_stmt).run(self.dbxsession,self.engine_config)

However the pyspark.sql.connect.session.SparkSession doesn't implement sparkContext(): error: sparkContext() is not implemented.
https://github.com/apache/spark/blob/master/python/pyspark/sql/connect/session.py#L479
At this point this is getting beyond my comfort level/knowledge.
Appreciate the help and work on this project!
@goodwanghan @kvnkho for reference: https://fugue-project.slack.com/archives/C015RGNUW77/p1682529474759219

Thanks for writing this up! We need to update Fugue to be able the new version here. We'll do it in the next week or two.

From 0.8.4.dev2, we started support spark connect.
If you are using Databricks 13.0+ you don't even need fugue-cloudprovider, just initialize your DatabricksSparkSession on your local env in the standard way, and that spark session can be recognized by Fugue as the spark execution engine.

You need to understand Spark connect has these limitations by design:
https://docs.databricks.com/dev-tools/databricks-connect.html#pyspark-dataframe-api-limitations
https://docs.databricks.com/dev-tools/databricks-connect.html#databricks-and-databricks-connect-limitations


[Bug] fetch-prompts.mjs` doesn't parse english prompts correctly

Describe the bug
fetch-prompts.mjs doesn't parse  "" correctly.   if the prompt content doesn't contain "",  it has no problem. but as long as there are "" inside the content. you can see the problem as in my screenshot.
This issue only happens in English prompts, while Chinese version of prompts works fine.
To Reproduce
Steps to reproduce the behavior:

in chat input textarea, input '/'
search prompt name "Python Interpreter" (there are two prompts of the same name, both can reproduce this problem).
select it, and look at the end of the prompt content.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Deployment

[ ] Docker
[ ] Vercel
[ ] Server

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional Logs
Add any logs about the problem here.

i am going to create a pull request regarding to this bug.


Add descriptor_type to the descriptors

Problem:
Someone gives you two YAML files, a NSD and a VNFD. But you are not told which of the files is the NSD and which the VNFD. To find out, you can either check the fields in the descriptor and make a qualified guess or we could directly include a type field in the descriptor's header that tells us.
This would look like this:
descriptor_version: "1.0"
descriptor_type: "eu.5gtango.nsd"
...

Possible values for the type could be (but are not limited to):

eu.5gtango.nsd
eu.5gtango.vnfd
eu.5gtango.msd
eu.sonata-nfv.nsd
eu.sonata-nfv.vnfd
eu.sonata-nfv.msd

This will also support to distinguish VNF only vs. NS packages.

I suggest to only allow values based on the layer tags defined here: https://github.com/sonata-nfv/tng-schema/wiki/PkgSpec_LATEST#list-of-pre-defined-tags

I just included descriptor types in all schemas and descriptor examples.
I documented the descriptor types here: https://github.com/sonata-nfv/tng-schema/wiki/Descriptor-Type

I would suggest to change the decriptor_types to use the MIME types for the 5GTANGO descriptors as defined here: https://github.com/sonata-nfv/tng-schema/wiki/PkgSpec_LATEST#5gtango-specific-mime-types
Using layer tags for it seems to be a bit strange. We cannot express schemas for e.g. OSM. What do you think?
(I am also re-opening this issue because I need to add the descriptor_type field to the NAPD and REF schemas, examples and specs)

As we decided to replace descriptor_version and descriptor_type with descriptor_schema (path to the corresponding schema), I will remove the descriptor_type wiki page.

We still have to implement/adjust the validator to check descriptors against the referenced schema in descriptor_schema.

I don't think, we need this anymore


ec_resource: Use deployment template as blueprint

Description
Changes the topology block within each of deployment resources from
required to optional by using the deployment_template template of
the GET deployment templates API call. Not only aligning with how the
Elastic Cloud Web UI works, but also benefiting from defaults, important
metadata and pre-configured settings set in the deployment template now
consumed, as a result a series of side-effects have cascaded down to
the ec_resource schema itself, allowing users to create deployments
specifying much less, and making getting started much easier.
Both on Create and Update, the GET /deployments/templates endpoint
is called, filtering out the specified deployment_template_id field,
and reading the .deployment_template object which contains a blueprint
to create a deployment using this template.
The deployment_template contains all the resource kinds node_types,
curation settings, node attributes (for tiered data topologies), and
instance_configuration_id, all details that the user was burdened to
know before to be able to create a deployment. Since the Elasticsearch
resource is the most complex of resources, it's the one benefiting most
from this change.
Then, the template is used as a blueprint, and any attributes set by the
user will be overridden from the deployment template blueprint.
Also removing the default values for memory_per_node and zone_count.
This caused a little bit more of an impact on the code than expected,
thus all the handling of empty and / or unset topology settings and mem
size.
Since there's no reason to allow users to tweak the Elasticsearch and
Enterprise Search node types, they have been removed as settable, and
are only computed for reference.
As part of this patch, the "region" field has been marked as ForceNew
since changing the region field will cause a deployment to be destroyed
and a new one created.
Additionally, two different bugs are fixed in this patch.
First, removing any resource from the Terraform definition didn't cause
the resource to be removed from the backend due to prune_orphans being
set to false.
Second, any resources which were removed, still appear on the plan with
"stopped" in the status, since no filtering was done on the flattening
functions, an infinite loop was triggered and they were attempted to be
removed again, and persisted in the state. Filtering any resources with
status == "stopped" fixes this.
Last, a series of new unit tests have been added which now test more
scenarios where users are some overrides on the topology to ensure that
the code behaves as it should.
Examples
Minimal
resource "ec_deployment" "example_minimal" {
  # Mandatory fields
  region                 = "us-east-1"
  version                = "7.9.2"
  deployment_template_id = "aws-io-optimized-v2"

  elasticsearch {}

  kibana {}

  apm {}

  enterprise_search {}
}

Omitting the instance_configuration_id, but setting overrides
# Create an Elastic Cloud deployment
resource "ec_deployment" "example_minimal" {
  # Mandatory fields
  region                 = "us-east-1"
  version                = "7.9.2"
  deployment_template_id = "aws-io-optimized-v2"

  elasticsearch {
    topology {
      memory_per_node = "1g"
    }
  }

  kibana {
    topology {
      memory_per_node = "2g"
    }
  }

  apm {
    topology {
      memory_per_node = "1g"
    }
  }

  enterprise_search {
    topology {
      memory_per_node = "4g"
    }
  }
}

Related Issues

Closes #94
Motivation and Context

Try and reduce the Minimal configuration required
How Has This Been Tested?

Thorougly manually and through the acceptance tests.
Types of Changes

[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

@karencfv apologies for the big PR, but there wasn't a great way to split this. I have described to the best of my ability what all the changes are.

hmmm... enterprise_search keeps failing on the acceptance tests for some reason 🤔

awesome stuff @marclop !
few questions though on your writeup -

it does not rely on
the default memory size in the deployment_template, but instead a local
default is used for Elasticsearch (4Gb instead of 8Gb per node).

why do we use this local default?

Since there's no reason to allow users to tweak the Elasticsearch and
Enterprise Search node types, they have been removed as settable, and
are only computed for reference.

could you please elaborate? does this align with all future plans we have for elasticsearch node types?

awesome stuff @marclop !
few questions though on your writeup -

it does not rely on
the default memory size in the deployment_template, but instead a local
default is used for Elasticsearch (4Gb instead of 8Gb per node).

why do we use this local default?

The description was outdated, I have now updated to reflect what the patch is changing, there's no more local defaults for memory_per_node or zone_count.

Since there's no reason to allow users to tweak the Elasticsearch and
Enterprise Search node types, they have been removed as settable, and
are only computed for reference.

could you please elaborate? does this align with all future plans we have for elasticsearch node types?

We're just not allowing users to set those settings anymore since they are set by the deployment template on each of the instance configurations.
Now, we match the instance configuration specified (or not, then the default for that DT), and those instance configurations already have the "roles" defined, when I say node types I'm referring to data, master, ingest, ml. In any case I think that users are not allowed to specify different node_types than what's in the Instance Configuration anyway, so we might have saved users from a configuration which they could set but wasn't allowed anyway.

@karencfv the enterprise_search instance seems to be bootlooping...
Status: [Failed] Applying Actastic Migration: <UpdateUsersAndEngines version=20191229125356>: Error = Swiftype::ES::IndexAlreadyExistsException: index [.ent-search-actastic-users_v2/2ZPAnIE3R5es9poii8kGfA] already exists index '.ent-search-actastic-users_v2'

@karencfv I fixed the integration tests on 060daf8

Jenkins test this please.

Hi @marclop thanks for doing the change, this simplifies a lot of configuration, i was testing the new version, during which i found the below issue, for deployment template
resource "ec_deployment" "example_minimal" {
  # Optional name.
  name = "horizon-hot-warm-test"

  # Mandatory fields
  region                 = "aws-eu-west-2"
  version                = "7.9.2"
  deployment_template_id = "aws-hot-warm-v2"

  elasticsearch {}

  kibana {}
}

is getting created but if we update the configuration like
resource "ec_deployment" "example_minimal" {
  # Optional name.
  name = "horizon-hot-warm-test"

  # Mandatory fields
  region                 = "aws-eu-west-2"
  version                = "7.9.2"
  deployment_template_id = "aws-hot-warm-v2"

  elasticsearch {
    topology {
      instance_configuration_id = "aws.data.highio.i3"
      zone_count = 1
      memory_per_node = "1g"
    }
    topology {
      instance_configuration_id = "aws.data.highstorage.d2"
      zone_count = 1
      memory_per_node = "2g"
    }
  }

  kibana {}
}

the terraform is passing but the configuration is not updating at Elastic cloud and getting the error like
Elasticsearch - The requested cluster configuration change is not permitted. Value of field 'plan fields' cannot be changed.
Can u please also check updating the deployment works if it is provisioned from template.

@ggsood that's strange, I'll try to reproduce locally and see what's going on.

H @marclop https://github.com/elastic/terraform-provider-ec/pull/108 this will help ?

It will only mitigate part of the problem, the real issue is that we're overwriting the whole Elasticsearch object when the config part of the definition is expanded, I've described the issue in a comment: https://github.com/elastic/terraform-provider-ec/issues/111#issuecomment-704360511


Local Host Shop Bug

https://gyazo.com/0d161a8be3c28d7245ec0fd287b5b27f
As you can see, I set 3 items into our shop so far, costin 1 million, 500k, and 200k respectively. (placeholders). I then gave myself 1m currency. Then, i bought slot 1, for 1m.. However only 100k was taken from my total currency.
upon testing, it would seem i only required 1m to have access to buying it, however the actual purchase price is only 10% of the listed value.. Any way to make it 100%?
~Edit~ Testing on the Public (Nadeko Server) Bot, shows it to be working just fine, It seems to possibly be just a Local issue. That, or I did the command wrong? (Typed exactly as the commands list shows) .shopadd role price rolename

The person who created the shop entry receives 90% of the profits.