[BUG] Jxpath not working with @Accessors(chain=true)

Describe the bug
The modified signature of the setXyz methods, not returning void, seems to break the bean-property-lookup methods of Jxpath. The setter method is not found by Jxpath anymore, thus setValue(...) does not work anymore.
The bug could be fixed by providing a withXyz(...)-method instead.
To Reproduce
package bug;

import lombok.Data;
import lombok.experimental.Accessors;
import org.apache.commons.jxpath.JXPathContext;
import org.junit.Test;

import static org.junit.Assert.*;

public class JxpathTest {

  @Data
  @Accessors(chain = true) // <<< comment this in out out to see difference
  public static class Stuff {
    String string;
  }

  @Test
  public void setValuePureJxPath() {
    // arrange
    Stuff stuff = new Stuff();
    stuff.setString("old");
    JXPathContext instance = JXPathContext.newContext(stuff);
    // act
    instance.setValue("string", "value");
    // assert
    assertEquals("value", stuff.getString());
  }
}

With @Accessors(chain = true) the test fails. Commenting it out it passes.
Expected behavior
The test should pass with @Accessors(chain = true).
Version info (please complete the following information):
  <dependency>
    <groupId>org.projectlombok</groupId>
    <artifactId>lombok</artifactId>
    <version>1.18.0</version>
    <scope>provided</scope>
  </dependency>

IntelliJ, unittest run in IDE.
IntelliJ IDEA 2018.3.4 (Ultimate Edition)
Build #IU-183.5429.30, built on January 29, 2019
JRE: 1.8.0_152-release-1343-b26 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.15.0-45-generic

Additional
I don't think this is fixable in Lombok without any changes to how @Accessors work (this could only be fixed in Jxpath, if at all, because that relies on bean property specification, I think).
My suggestion would be: Instead of modifying the signature of the Setter methods it would be better to add a method like Type withXyz(Type newValue);. Luckily @Accessors is marked experimental...

Chaining makes the setter non-compliant to the bean spec, so jxpath is actually doing the right thing.
Furthermore, withXyz() is colliding with @Wither, so that probably won't happen.
Disclaimer: I'm not a Lombok maintainer, just a contributer.

this could only be fixed in Jxpath, if at all, because that relies on bean property specification, I think

I'd bet, it does, but I could imagine a chain option dealing with this.

My suggestion would be: Instead of modifying the signature of the Setter methods it would be better to add a method like Type withXyz(Type newValue);.

Are you proposing to generate two (differently named) setters? I'm afraid, this would be far worse than anything else and that's what the project owners thinks, too (there was such a posting recently).

My suggestion would be: Instead of modifying the signature of the Setter methods it would be better to add a method like Type withXyz(Type newValue);.

Are you proposing to generate two (differently named) setters? I'm afraid, this would be far worse than anything else and that's what the project owners thinks, too (there was such a posting recently).

I guess I did. But agree completely with your/owners assessment. Such an interface would be bad.
Good that you pointed that out, though, because I use it all the time with jaxws. I use fluent interface generation for my @XmlElements to do this:
MyType m = new MyType()
    .withMyStuff(new MyStuff()
        .withMyThing("thing")
        .withMyBar(17))
    .withMyVar(Vars.ONE));

...and now realize that having void setMyStuff(MyStuff) and MyType withMyStuff(MyStuff) is not a good interface.
But fluent interfaces are really convenient. They circumvent Javas lack of named/optional parameters. But the bean spec is in the way. Hrm...
How about an auto generated builder class then? That offers all the with-methods but no setters? Is that convenient and a good interface? I didn't try out the existing @Builder yet, but I will. I have the feeling it may offer what I want.
Note: I wasn't aware of @Wither, but creating clones all the time is not what I mean with "convenient".

How about an auto generated builder class then?

There are @lombok.Builder and .SuperBuilder (where the latter can deal with class hierarchies). I'd say, both have issues, but both are pretty good.

That offers all the with-methods but no setters?

They offer chained fluent setters (no prefix).

Is that convenient and a good interface?

In general, yes, in your case, you'll see.

Note: I wasn't aware of @Wither, but creating clones all the time is not what I mean with "convenient".

Right, @Wither is perfect when you want to modify a single field, it's on par with @Builder for two and loses for more.
Note that all these annotations are primarily meant for constructing immutable instances.

 Unfortunately, have to close it.


WIP: split files under server/

just moving files and functions around for better mental sanity..
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

pls, do not merge yet.

@mrunalp could you ack on the direction of this PR before I go further with image.go and sandbox.go

@runcom Looks good :+1:

Should you create a subdir of server/containers/.go for all of the containers_.go?  Then you could just have list.go, run.go ...

We could, I'd rather keep this in the same directory for without incurring
in any potential circular dependency though (and go will complain).
On Nov 22, 2016 8:20 PM, "Daniel J Walsh"<EMAIL_ADDRESS>wrote:

Should you create a subdir of server/containers/.go for all of the
containers_.go? Then you could just have list.go, run.go ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cri-o/pull/207#issuecomment-262338285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAz3--ALI6lCDTHodby64pAP8gCWopVdks5rA0CWgaJpZM4K5rCa
.

Dan, also, it makes little sense to create new packages (created with each subdirectory) since the CRI is a flat API right now and I'm just separating the API calls here (the rest is well separated under top level ./oci package for instance).

Ok, I was just complaining about the container_foobar namespace, just a thought, but the circular dependency problem is a good reason not to.

Yeah, go gets in the way of doing that :/

@mrunalp PTAL, ready to go

LGTM


Single demo grasp

Single Demo Grasping learner class and data augmentation for generating datasets.

[ ] possible coding style improvements
[ ] simulation of franka emika panda robot and necessary tools in webots will be provided separately??
[ ] ROS nodes for streaming images from webots and running inference will be provided

I convert this PR to draft.
Please change the status when it is ready for review.

Hi, as discussed during the meeting we would like the know what is the status of this PR. Considering it's quite old, it should also be re-synched with master, in case any issue surfaces. Just as a reminder this page has additional info regarding what should be included when adding a tool.

Hi
I am mostly working on preparing the simulation part of the package. However the learner class itself requires minimal changes. I will soon ask for review latest by the end of this week.
Best Regards,
Amir Mehman Sefat
From: ad-daniel @.>
Sent: Monday, 4 October 2021 16.20
To: tasostefas/opendr_internal @.>
Cc: Amir Mehman Sefat (TAU) @.>; Author @.>
Subject: Re: [tasostefas/opendr_internal] Single demo grasp (#111)
Hi, as discussed during the meeting we would like the know what is the status of this PR. Considering it's quite old, it should also be re-synched with master, in case any issue surfaces. Just as a reminder this pagehttps://github.com/tasostefas/opendr_internal/wiki/Adding-New-Tool has additional info regarding what should be included when adding a tool.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/tasostefas/opendr_internal/pull/111#issuecomment-933478932, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALR3TD3BGVBNP3WN3AWQCI3UFGSZTANCNFSM5AUUIIVA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


(feat) Add given-when-then alternates.

@ihrwein is this what you were thinking?
Fixes #25

:+1: Thanks!


civic homepage not load from China and Hong kong

I am from China, and when I access civic, I got an access error
Error: [$injector:modulerr] http://errors.angularjs.org/1.4.7/$injector/modulerr?p0=civicClient&p1=%5B%24injector%3Anomod%5D%20http%3A%2F%2Ferrors.angularjs.org%2F1.4.7%2F%24injector%2Fnomod%3Fp0%3DcivicClient%0AI%2F%3C%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A6%3A416%0Ade%2F%3C%2F%3C%2F%3C%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A24%3A186%0Aa%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A23%3A252%0Ade%2F%3C%2F%3C%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A23%3A495%0Ah%2F%3C%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A37%3A427%0Am%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A7%3A320%0Ah%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A37%3A275%0Afb%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A41%3A35%0Azc%2Fd%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A19%3A463%0Azc%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A20%3A274%0AZd%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A19%3A83%0A%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A293%3A238%0Aa%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A174%3A399%0AHf%2Fc%40https%3A%2F%2Fajax.googleapis.com%2Fajax%2Flibs%2Fangularjs%2F1.4.7%2Fangular.min.js%3A35%3A212%0A

Seems like googleapis is not accessible here.
Error remained if I VPN to HongKong, but disappeared if I VPN to Dallas, US.

Hi @sfchen, the error msg you're seeing indicates that angular's dependency injector is attempting to inject a dependency that wasn't instantiated during application bootstrap. We load many of these dependencies via a couple of Content Distribution Networks, including Google's. It may be that China's firewall blocks requests for these libraries, which then causes this error when Angular's injector attempts to instantiate and inject them.
Could you attempt to load the site again, and keep an eye on your browser's Network tab in its Developer Tools window? This will show you all of the network requests made during application bootstrap, including any blocked requests. If our assumptions are correct, you'll notice at least one blocked request.
We did find an online tool that purports to show how a site loads from behind the firewall, and it reports that all the CDN libraries were loaded properly:
https://www.dotcom-tools.com/DetailView.aspx?id=A10DF5DB1536454DA17B96AAD1652415:11&visit=first&Tab=TimeWaterChart
However, I understand that firewall behavior changes quite often and may not be consistent everywhere in China. Please let us know what you find! We'd like to ensure that CIViC will load and run properly everywhere.

OK, based on this Wikipedia article, it appears that Google's CDN is indeed blocked. We're looking into switching to an un-blocked CDN.

@sfchen, we've rewritten the CDN configuration to a CDN that looks like it's permitted through the CN firewall. This update will be out in the next deploy sometime next week. Please reopen the issue if you continue to have issues after the deploy.


network-configs/genesis.json: no such file or directory

I'm having this issue when running with the following configuration in a Mac with M1:
Launching participant network with 1 participants and the following network params struct(additional_preloaded_contracts = {}, churn_limit_quotient = 65536, custody_requirement = 4, data_column_sidecar_subnet_count = 128, deneb_fork_epoch = 0, deposit_contract_address = "0x4242424242424242424242424242424242424242", devnet_repo = "ethpandaops", eip7594_fork_epoch = 100000001, eip7594_fork_version = "0x60000038", ejection_balance =<PHONE_NUMBER>0, electra_fork_epoch = 100000000, eof_activation_epoch = "", eth1_follow_distance = 2048, genesis_delay = 20, genesis_gaslimit = 30000000, max_blobs_per_block = 6, max_per_epoch_activation_churn_limit = 8, min_validator_withdrawability_delay = 256, network = "mainnet", network_id = "3151908", network_sync_base_url = "https://snapshots.ethpandaops.io/", num_validator_keys_per_node = 64, prefunded_accounts = {}, preregistered_validator_count = 0, preregistered_validator_keys_mnemonic = "giant issue aisle success illegal bike spike question tent bar rely arctic volcano long crawl hungry vocal artwork sniff fantasy very lucky have athlete", preset = "mainnet", samples_per_slot = 8, seconds_per_slot = 12, shard_committee_period = 256)

There was an error executing Starlark code
An error occurred executing instruction (number 56) at github.com/ethpandaops/ethereum-package/src/full_beaconchain/full_beaconchain_launcher.star[177:14]:
  exec(service_name="beaconchain-schema-initializer", recipe=ExecRecipe(command=["./misc", "-config", "/app/config/beaconchain-config.yml", "-command", "applyDbSchema"]), description="Applying db schema")
  Caused by: Exec returned exit code '1' that is not part of the acceptable status codes '[0]', with output:
    "time=\"2024-09-29T23:21:21Z\" level=info msg=\"compile time check completed\" module=templates\ntime=\"2024-09-29T23:21:21Z\" level=info msg=starting config=/app/config/beaconchain-config.yml version=bdbf2f1\ntime=\"2024-09-29T23:21:21Z\" level=info msg=\"loaded chain config from node with genesis time<PHONE_NUMBER>\" module=oauth\ntime=\"2024-09-29T23:21:21Z\" level=fatal msg=\"error reading config file: error opening EL Chain Config file /network-configs/genesis.json: open /network-configs/genesis.json: no such file or directory\"\n"

Error encountered running Starlark code.

can you paste your config.yaml that you used?

can you paste your config.yaml that you used?

Sure @barnabasbusa, I've had different errors with different el + cl + network configurations.

This is the configuration that caused the previous error:
# Specification of the participants in the network
participants:
  # EL(Execution Layer) Specific flags
    # The type of EL client that should be started
    # Valid values are geth, nethermind, erigon, besu, ethereumjs, reth, nimbus-eth1
  - el_type: geth
    # el_image: "ethereum/client-go:v1.14.8"

    # The log level string that this participant's EL client should log at
    # If this is emptystring then the global `logLevel` parameter's value will be translated into a string appropriate for the client (e.g. if
    # global `logLevel` = `info` then Geth would receive `3`, Besu would receive `INFO`, etc.)
    # If this is not emptystring, then this value will override the global `logLevel` setting to allow for fine-grained control
    # over a specific participant's logging
    el_log_level: ""

    # A list of optional extra env_vars the el container should spin up with
    el_extra_env_vars: {}

    # A list of optional extra params that will be passed to the EL client container for modifying its behaviour
    el_extra_params: []

    # Persistent storage size for the EL client container (in MB)
    # Defaults to 0, which means that the default size for the client will be used
    # Default values can be found in /src/package_io/constants.star VOLUME_SIZE
    el_volume_size: 0

    # Resource management for el containers
    # CPU is milicores
    # RAM is in MB
    # Defaults to 0, which results in no resource limits
    el_min_cpu: 0
    el_max_cpu: 0
    el_min_mem: 0
    el_max_mem: 0

  # CL(Consensus Layer) Specific flags
    # The type of CL client that should be started
    # Valid values are nimbus, lighthouse, lodestar, teku, prysm, and grandine
    cl_type: lighthouse
    # cl_image: "sigp/lighthouse:v5.3.0"

    # The log level string that this participant's CL client should log at
    # If this is emptystring then the global `logLevel` parameter's value will be translated into a string appropriate for the client (e.g. if
    # global `logLevel` = `info` then Teku would receive `INFO`, Prysm would receive `info`, etc.)
    # If this is not emptystring, then this value will override the global `logLevel` setting to allow for fine-grained control
    # over a specific participant's logging
    cl_log_level: ""

    # A list of optional extra env_vars the cl container should spin up with
    cl_extra_env_vars: {}

    # A list of optional extra params that will be passed to the CL client Beacon container for modifying its behaviour
    # If the client combines the Beacon & validator nodes (e.g. Teku, Nimbus), then this list will be passed to the combined Beacon-validator node
    cl_extra_params: []

    # Persistent storage size for the CL client container (in MB)
    # Defaults to 0, which means that the default size for the client will be used
    # Default values can be found in /src/package_io/constants.star VOLUME_SIZE
    cl_volume_size: 0

    # Resource management for cl containers
    # CPU is milicores
    # RAM is in MB
    # Defaults to 0, which results in no resource limits
    cl_min_cpu: 0
    cl_max_cpu: 0
    cl_min_mem: 0
    cl_max_mem: 0

    # The number of validator clients to run for this participant
    # Defaults to 1
    vc_count: 1

  # Participant specific flags
    # Count of nodes to spin up for this participant
    # Default to 1
    count: 1

    # Prometheus additional configuration for a given participant prometheus target.
    # Execution, beacon and validator client targets on prometheus will include this
    # configuration.
    prometheus_config:
      # Scrape interval to be used. Default to 15 seconds
      scrape_interval: 15s
      # Additional labels to be added. Default to empty
      labels: {}

    # A set of parameters the node needs to reach an external block building network
    # If `null` then the builder infrastructure will not be instantiated
    # Example:
    #
    # "relay_endpoints": [
    #<EMAIL_ADDRESS>    #<EMAIL_ADDRESS>    #<EMAIL_ADDRESS>    #<EMAIL_ADDRESS>    # ]
    builder_network_params: null

    # Participant flag for keymanager api
    # This will open up http ports to your validator services!
    # Defaults null and then set to default global keymanager_enabled (false)
    keymanager_enabled: null

# Default configuration parameters for the network
network_params:
  # Network name, used to enable syncing of alternative networks
  # You can sync any public network by setting this to the network name (e.g. "mainnet", "sepolia", "holesky")
  # You can sync any devnet by setting this to the network name (e.g. "dencun-devnet-12", "verkle-gen-devnet-2")
  network: "sepolia"

  # The network ID of the network.
  network_id: "3151908"

  # The address of the staking contract address on the Eth1 chain
  deposit_contract_address: "0x4242424242424242424242424242424242424242"

  # Number of seconds per slot on the Beacon chain
  seconds_per_slot: 12

  # The number of validator keys that each CL validator node should get
  num_validator_keys_per_node: 64

  # This mnemonic will a) be used to create keystores for all the types of validators that we have and b) be used to generate a CL genesis.ssz that has the children
  # validator keys already preregistered as validators
  preregistered_validator_keys_mnemonic: "giant issue aisle success illegal bike spike question tent bar rely arctic volcano long crawl hungry vocal artwork sniff fantasy very lucky have athlete"

  # The number of pre-registered validators for genesis. If 0 or not specified then the value will be calculated from the participants
  preregistered_validator_count: 0

  # How long you want the network to wait before starting up
  genesis_delay: 20

  # The gas limit of the network set at genesis
  genesis_gaslimit: 30000000

  # Preset for the network
  # Default: "mainnet"
  # Options: "mainnet", "minimal"
  # "minimal" preset will spin up a network with minimal preset. This is useful for rapid testing and development.
  # 192 seconds to get to finalized epoch vs 1536 seconds with mainnet defaults
  # Please note that minimal preset requires alternative client images.
  # For an example of minimal preset, please refer to [minimal.yaml](.github/tests/minimal.yaml)
  preset: "mainnet"

  # Preloaded contracts for the chain
  additional_preloaded_contracts: {}
  # Example:
  # additional_preloaded_contracts: '{
  #  "0x123463a4B065722E99115D6c222f267d9cABb524":
  #   {
  #     balance: "1ETH",
  #     code: "0x1234",
  #     storage: {},
  #     nonce: 0,
  #     secretKey: "0x",
  #   }
  # }'

  # Repository override for devnet networks
  # Default: ethpandaops
  devnet_repo: ethpandaops

  # A number of prefunded accounts to be created
  # Defaults to no prefunded accounts
  # Example:
  # prefunded_accounts: '{"0x25941dC771bB64514Fc8abBce970307Fb9d477e9": {"balance": "10ETH"}}'
  # 10ETH to the account 0x25941dC771bB64514Fc8abBce970307Fb9d477e9
  # To prefund multiple accounts, separate them with a comma
  #
  # prefunded_accounts: '{"0x25941dC771bB64514Fc8abBce970307Fb9d477e9": {"balance": "10ETH"}, "0x4107be99052d895e3ee461C685b042Aa975ab5c0": {"balance": "1ETH"}}'
  prefunded_accounts: {}

# # Global parameters for the network

# By default includes
# - A transaction spammer & blob spammer is launched to fake transactions sent to the network
# - Forkmon for EL will be launched
# - A prometheus will be started, coupled with grafana
# - A beacon metrics gazer will be launched
# - A light beacon chain explorer will be launched
# - Default: []
additional_services:
  - assertoor
  - broadcaster
  - tx_spammer
  - blob_spammer
  - custom_flood
  - goomy_blob
  - el_forkmon
  # TODO: https://github.com/blockscout/blockscout-rs/issues/1001
  # - blockscout
  - beacon_metrics_gazer
  - dora
  - full_beaconchain_explorer
  - prometheus_grafana
  - blobscan
  - dugtrio
  - blutgang
  - forky
  - apache
  - tracoor

# Configuration place for dora the explorer - https://github.com/ethpandaops/dora
dora_params:
  # Dora docker image to use
  # Leave blank to use the default image according to your network params
  image: ""

  # A list of optional extra env_vars the dora container should spin up with
  env: {}

# Configuration place for transaction spammer - https://github.com/MariusVanDerWijden/tx-fuzz
tx_spammer_params:
  # A list of optional extra params that will be passed to the TX Spammer container for modifying its behaviour
  tx_spammer_extra_args: []

# Configuration place for goomy the blob spammer - https://github.com/ethpandaops/goomy-blob
goomy_blob_params:
  # A list of optional params that will be passed to the blob-spammer comamnd for modifying its behaviour
  goomy_blob_args: []

# Configuration place for prometheus
prometheus_params:
  storage_tsdb_retention_time: "1d"
  storage_tsdb_retention_size: "512MB"
  # Resource management for prometheus container
  # CPU is milicores
  # RAM is in MB
  min_cpu: 10
  max_cpu: 1000
  min_mem: 128
  max_mem: 2048

# Configuration place for grafana
grafana_params:
  # A list of locators for grafana dashboards to be loaded be the grafana service
  additional_dashboards: []
  # Resource management for grafana container
  # CPU is milicores
  # RAM is in MB
  min_cpu: 10
  max_cpu: 1000
  min_mem: 128
  max_mem: 2048

# Configuration place for the assertoor testing tool - https://github.com/ethpandaops/assertoor
assertoor_params:
  # Check chain stability
  # This check monitors the chain and succeeds if:
  # - all clients are synced
  # - chain is finalizing for min. 2 epochs
  # - >= 98% correct target votes
  # - >= 80% correct head votes
  # - no reorgs with distance > 2 blocks
  # - no more than 2 reorgs per epoch
  run_stability_check: false

  # Run normal transaction test
  # This test generates random EOA transactions and checks inclusion with/from all client pairs
  # This test checks for:
  # - block proposals with transactions from all client pairs
  # - transaction inclusion when submitting via each client pair
  # test is done twice, first with legacy (type 0) transactions, then with dynfee (type 2) transactions
  run_transaction_test: false

  # Run additional tests from external test definitions
  # Entries may be simple strings (link to the test file) or dictionaries with more flexibility
  # eg:
  #   - https://raw.githubusercontent.com/ethpandaops/assertoor/master/example/tests/block-proposal-check.yaml
  #   - file: "https://raw.githubusercontent.com/ethpandaops/assertoor/master/example/tests/block-proposal-check.yaml"
  #     config:
  #       someCustomTestConfig: "some value"
  tests: []

# The global log level that all clients should log at
# Valid values are "error", "warn", "info", "debug", and "trace"
# This value will be overridden by participant-specific values
global_log_level: "info"

# EngineAPI Snooper global flags for all participants
# Default to false
snooper_enabled: false

# Enables Ethereum Metrics Exporter for all participants
# Defaults to false
ethereum_metrics_exporter_enabled: false

# Disable peer scoring to prevent nodes impacted by faults from being permanently ejected from the network
# Default to false
disable_peer_scoring: false

# Whether the environment should be persistent; this is WIP and is slowly being rolled out accross services
# Note this requires Kurtosis greater than 0.85.49 to work
# Note Erigon, Besu, Teku persistence is not currently supported with docker.
# Defaults to false
persistent: true

# Enables Xatu Sentry for all participants
# Defaults to false
xatu_sentry_enabled: true

# Xatu Sentry params
xatu_sentry_params:
  # The image to use for Xatu Sentry
  xatu_sentry_image: ethpandaops/xatu:latest
  # GRPC Endpoint of Xatu Server to send events to
  xatu_server_addr: localhost:8080
  # Enables TLS to Xatu Server
  xatu_server_tls: false
  # Headers to add on to Xatu Server requests
  xatu_server_headers: {}
  # Beacon event stream topics to subscribe to
  beacon_subscriptions:
    - attestation
    - block
    - chain_reorg
    - finalized_checkpoint
    - head
    - voluntary_exit
    - contribution_and_proof
    - blob_sidecar

# Global flag to enable checkpoint sync across the network
checkpoint_sync_enabled: true

# Global flag to set checkpoint sync url
# Tried https://beaconstate-sepolia.chainsafe.io, but had https://github.com/sigp/lighthouse/issues/4559
checkpoint_sync_url: "https://sepolia.beaconstate.info/"

# Global paarameter to set the exit ip address of services and public ports
port_publisher:
  # if you have a service that you want to expose on a specific interfact; set that IP here
  # if you set it to auto it gets the public ip from ident.me and sets it
  # Defaults to constants.PRIVATE_IP_ADDRESS_PLACEHOLDER
  # The default value just means its the IP address of the container in which the service is running
  nat_exit_ip: KURTOSIS_IP_ADDR_PLACEHOLDER
  # Execution Layer public port exposed to your local machine
  # Disabled by default
  # Public port start defaults to 32000
  # You can't run multiple enclaves on the same port settings
  el:
    enabled: false
    public_port_start: 32000
  # Consensus Layer public port exposed to your local machine
  # Disabled by default
  # Public port start defaults to 33000
  # You can't run multiple enclaves on the same port settings
  cl:
    enabled: false
    public_port_start: 33000
  # Validator client public port exposed to your local machine
  # Disabled by default
  # Public port start defaults to 34000
  # You can't run multiple enclaves on the same port settings
  vc:
    enabled: false
    public_port_start: 34000
  # Additional services public port exposed to your local machine
  # Disabled by default
  # Public port start defaults to 35000
  # You can't run multiple enclaves on the same port settings
  additional_services:
    enabled: false
    public_port_start: 35000

The main changes were using network: "sepolia", checkpoint_sync_enabled: true and checkpoint_sync_url: "https://sepolia.beaconstate.info/"

If I use network: "kurtosis" this does not happen, I've tried with mainnet, sepolia and holesky, and all get me to this error.

Can you try to disable full_beaconchain_explorer ?

After disabling full_beaconchain_explorer I do not get the same error, but I still get one:
There was an error executing Starlark code
An error occurred executing instruction (number 11) at github.com/ethpandaops/ethereum-package/src/cl/lighthouse/lighthouse_launcher.star[82:38]:
  add_service(name="cl-1-lighthouse-geth", config=ServiceConfig(image="ethpandaops/lighthouse:stable", ports={"http": PortSpec(number=4000, transport_protocol="TCP", application_protocol="http"), "metrics": PortSpec(number=5054, transport_protocol="TCP", application_protocol="http"), "tcp-discovery": PortSpec(number=9000, transport_protocol="TCP", application_protocol=""), "udp-discovery": PortSpec(number=9000, transport_protocol="UDP", application_protocol="")}, public_ports={}, files={"/data/lighthouse/beacon-data": Directory(persistent_key="data-cl-1-lighthouse-geth", size=500000), "/jwt": "jwt_file", "/network-configs": "el_cl_genesis_data"}, cmd=["lighthouse", "beacon_node", "--debug-level=info", "--datadir=/data/lighthouse/beacon-data", "--disable-enr-auto-update", "--enr-address=KURTOSIS_IP_ADDR_PLACEHOLDER", "--enr-udp-port=9000", "--enr-tcp-port=9000", "--listen-address=<IP_ADDRESS>", "--port=9000", "--http", "--http-address=<IP_ADDRESS>", "--http-port=4000", "--http-allow-sync-stalled", "--slots-per-restore-point=32", "--disable-packet-filter", "--execution-endpoints=http://{{kurtosis:732640fdd4924ee1863325d7fc87257f:ip_address.runtime_value}}:8551", "--jwt-secrets=/jwt/jwtsecret", "--suggested-fee-recipient=0x8943545177806ED17B9F23F0a21ee5948eCaa776", "--metrics", "--metrics-address=<IP_ADDRESS>", "--metrics-allow-origin=*", "--metrics-port=5054", "--enable-private-discovery", "--checkpoint-sync-url=https://beaconstate.ethstaker.cc/", "--network=mainnet"], env_vars={"RUST_BACKTRACE": "full"}, private_ip_address_placeholder="KURTOSIS_IP_ADDR_PLACEHOLDER", ready_conditions=ReadyCondition(recipe=GetHttpRequestRecipe(port_id="http", endpoint="/eth/v1/node/health"), field="code", assertion="IN", target_value=[200, 206], timeout="15m"), labels={"ethereum-package.client": "lighthouse", "ethereum-package.client-image": "ethpandaops-lighthouse_stable", "ethereum-package.client-type": "beacon", "ethereum-package.connected-client": "geth", "ethereum-package.sha256": ""}, tolerations=[], node_selectors={}))
  Caused by: Unexpected error occurred starting service 'cl-1-lighthouse-geth'
  Caused by: An error occurred waiting for all TCP and UDP ports to be open for service 'cl-1-lighthouse-geth' with private IP '<IP_ADDRESS>'; this is usually due to a misconfiguration in the service itself, so here are the logs:
  == SERVICE 'cl-1-lighthouse-geth' LOGS ===================================
  Sep 30 07:53:42.625 INFO Logging to file                         path: "/data/lighthouse/beacon-data/beacon/logs/beacon.log"
  Sep 30 07:53:42.635 INFO Lighthouse started                      version: Lighthouse/v5.3.0-2e0eb6d
  Sep 30 07:53:42.635 INFO Configured for network                  name: mainnet
  Sep 30 07:53:42.635 INFO Data directory initialised              datadir: /data/lighthouse/beacon-data
  Sep 30 07:53:42.635 WARN Discv5 packet filter is disabled
  Sep 30 07:53:42.635 WARN Ignoring --http-allow-sync-stalled      info: this flag is deprecated and will be removed
  Sep 30 07:53:42.640 INFO Deposit contract                        address: 0x00000000219ab540356cbb839cbe05303d7705fa, deploy_block: 11184524
  Sep 30 07:53:42.663 INFO Blob DB initialized                     oldest_blob_slot: Some(Slot(8626176)), path: "/data/lighthouse/beacon-data/beacon/blobs_db", service: freezer_db
  Sep 30 07:53:42.677 INFO Starting checkpoint sync                remote_url: https://beaconstate.ethstaker.cc/, service: beacon
  Sep 30 07:54:18.167 INFO Loaded checkpoint block and state       block_root: 0xeeadbe88ab5ca885ed36e97459adde73562efe7ae10dfb370cae10420831cbe4, state_slot: 10071488, block_slot: 10071488, service: beacon
  Sep 30 07:54:18.177 INFO Loaded deposit tree snapshot            deposits loaded: 1697898, service: beacon
  Sep 30 07:54:24.546 INFO Block production enabled                method: json rpc via http, endpoint: Auth { endpoint: "http://<IP_ADDRESS>:8551/", jwt_path: "/jwt/jwtsecret", jwt_id: None, jwt_version: None }
  Sep 30 07:54:24.561 WARN Execution endpoint is not synced        last_seen_block_unix_timestamp: 0, endpoint: http://<IP_ADDRESS>:8551/, auth=true, service: beacon
  Sep 30 07:54:24.561 ERRO Error updating deposit contract cache   error: Failed to get remote head and new block ranges: EndpointError(FarBehind), retry_millis: 60000, service: beacon
  Sep 30 07:55:24.554 WARN Execution endpoint is not synced        last_seen_block_unix_timestamp: 0, endpoint: http://<IP_ADDRESS>:8551/, auth=true, service: beacon
  Sep 30 07:55:24.555 ERRO Error updating deposit contract cache   error: Failed to get remote head and new block ranges: EndpointError(FarBehind), retry_millis: 60000, service: beacon

looks like you using sepolia network with mainnet checkpoint sync url?

Yes, this was my bad while testing different combinations. I also had to disable dugtrio as it was failing to start (I might open a different issue for that one)
If I'd like to test full_beaconchain_explorer is there another test network that could be used? Or can this only be used with kurtosis network?

Would need to fix the config to support native chains. We haven't been using kurtosis for running public chains, thats why the lack of support for all the tooling.


Fix missing await and add tests

The modal to disable clusters was not waiting for an async operation to complete. Thus, follow up actions (getting the fresh data from backend) were triggered before all changes were submitted. As such,the view was not consistent.
Test for this have been added.
Fixes RHINENG-10897
The test added here require a revisit, as the implementation is not the best one.
I have tried something more elaborated, but did not work as expected. Advice will be appreciated.
describe.only('call order checking', () => {
  it('single host', () => {
    let number_of_calls = 0;
    let calls_so_far = 0;

    cy.mount(
      <MemoryRouter>
        <Intl>
          <Provider store={getStore()}>
            <DisableRule
              isModalOpen={true}
              rule={{ rule_id: 'foo|BAR', disabled: false }}
              host={'7795cbcd-0353-4e59-b920-fc1c39a27014'}
              afterFn={() => {
                // trick, as an expect here does not raise an error in the tests
                calls_so_far = number_of_calls;
                // expect(number_of_calls).to.equal(1);
              }}
            />
          </Provider>
        </Intl>
      </MemoryRouter>
    );

    cy.intercept(
      'PUT',
      '/api/insights-results-aggregator/v1/clusters/**/rules/**/error_key/**/disable',
      (req) => {
        req.on('after:response', () => {
          number_of_calls++;
        });
        req.reply({
          statusCode: 200,
          delay: 2 * 1000, // 2 secs
        });
      }
    ).as('disableRequest');

    cy.ouiaId(SAVE_BUTTON).click();
    cy.wait('@disableRequest').then(() => {
     expect(calls_so_far).to.equal(1);
    });
  });

  it('multiple hosts', () => {
    let number_of_calls = 0;
    let calls_so_far = 0;

    // beforeEach(() => {
    cy.mount(
      <MemoryRouter>
        <Intl>
          <Provider store={getStore()}>
            <DisableRule
              isModalOpen={true}
              rule={{ rule_id: 'foo|BAR', disabled: false }}
              hosts={[
                {
                  id: '084ac7a7-1c7d-49ff-b56e-f94881da242d',
                },
                {
                  id: '7795cbcd-0353-4e59-b920-fc1c39a27014',
                },
              ]}
              afterFn={() => {
                // trick, as an expect here does not raise an error in the tests
                calls_so_far = number_of_calls;
                // expect(number_of_calls).to.equal(2);
              }}
            />
          </Provider>
        </Intl>
      </MemoryRouter>
    );

    cy.intercept(
      'PUT',
      '/api/insights-results-aggregator/v1/clusters/**/rules/**/error_key/**/disable',
      (req) => {
        req.on('after:response', () => {
          number_of_calls++;
        });
        req.reply({
          statusCode: 200,
          delay: 2 * 1000, // 2 secs
        });
      }
    ).as('disableRequest');

    cy.ouiaId(SAVE_BUTTON).click();
    cy.wait(['@disableRequest', '@disableRequest']).then(() => {
      expect(calls_so_far).to.equal(2);
    });
  });

  it('no hosts', () => {
    let number_of_calls = 0;
    let calls_so_far = 0;

    cy.mount(
      <MemoryRouter>
        <Intl>
          <Provider store={getStore()}>
            <DisableRule
              isModalOpen={true}
              rule={{ rule_id: 'abc', disabled: false }}
              afterFn={() => {
                // trick, as an expect here does not raise an error in the tests
                calls_so_far = number_of_calls;
                // expect(number_of_calls).to.equal(1);
              }}
            />
          </Provider>
        </Intl>
      </MemoryRouter>
    );

    cy.intercept('POST', '/api/insights-results-aggregator/v2/ack', (req) => {
      req.on('after:response', () => {
        number_of_calls++;
      });
      req.reply({
        statusCode: 200,
        delay: 2 * 1000, // 2 secs
      });
    }).as('ackRequest');

    cy.ouiaId(SAVE_BUTTON).click();
    cy.wait('@ackRequest').then(() => {
      expect(calls_so_far).to.equal(1);
    });
  });
});```

/retest

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

Project coverage is 7.69%. Comparing base (d6fd289) to head (014f683).

Files
Patch %
Lines

src/Components/Modals/DisableRule.cy.js
0.00%
25 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #757      +/-   ##
=========================================
- Coverage    7.74%   7.69%   -0.05%
=========================================
  Files          68      68
  Lines        3860    3885      +25
  Branches      837     837
=========================================
  Hits          299     299
- Misses       3561    3586      +25

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

/retest

:tada: This PR is included in version 1.43.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot pair uri

![image](https://user-images.githubusercontent.com/43994963/222873922-910016e3-536f-4c1e-865b-445ed35f2c7b.png
Can you explain this error to me?
I try run example

hello @luannv1203 , are you trying to create pair uri from example and use the example to do pair with that uri ?

Hey @luannv1203 , let's use the example first. If the issue is still happen on your side, feel free to re-open the issue. Thank you!


question ??? how i can see the optinal param is notes section

,when i do hubot help
i got

hubot ec2 ami create --instance_id=*** - Create an ami.

but there is also another option to use , see notes section .
how i can show the notes section to the user ????

thanks

# Commands:
#   hubot ec2 ami create --instance_id=*** - Create an ami.
#
# Notes:
#   --instance_id=*** : [required] Target instance id.
#   --name=***        : [optional] Ami name to be created. If omit it, the Name of config is used.
#   --config_path=*** : [optional] Config file path. If omit it, HUBOT_AWS_EC2_CREATE_AMI_CONFIG is referred to.
#   --dry-run         : [optional] Checks whether the api request is right. Recommend to set before applying to real asset.

You probably need to follow this pattern:
Description:
Tell people hubot's new name if they use the old one

Commands:
None


Prepare 2.10.0 release

What does this PR do?
This PR prepares 2.10.0 release.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

[!IMPORTANT]
Can we wait and include this PR which fixes some perf issues in SR?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.15%. Comparing base (daec9a0) to head (407cac9).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2049      +/-   ##
===========================================
+ Coverage    83.10%   83.15%   +0.05%
===========================================
  Files          494      494
  Lines        17720    17711       -9
  Branches      2686     2682       -4
===========================================
+ Hits         14726    14727       +1
+ Misses        2253     2242      -11
- Partials       741      742       +1

see 28 files with indirect coverage changes


exported descriptions

Add a 1st 2nd xth... to the birthday response
Put the date in `{string}`
Put the time in birthday config
Split birthday into 2 fxns
Add a /privacy fxn (maybe combine with /github)
Add a /config fxn to see the servers settings

Along with this we will need to make a way for us to remind ourselves to update /help each time a function is modified/created


invoke-ai: Onnx inference based pipeline for Text to Image conversion

Initial version of ONNX based inference pipeline for Text to image conversion based on Stable-diffusion models.
Performance related information: Tested on Raphael (AMD 7600X) and Raptor Lake (I5-13000K)
Onnx based inference performance - InvokeAI-pipelines.pdf

Created and handled conflicts in #3380.


Grading curve

The math is done. Now find a reasonable design of class GradingCurve or @dataclass GradingMixture or whatever to represent and conveniently sample the radii according to a grading curve.

[ ] Provide defaults for DIN 1045-2


Maximal Square Solution (Leetcode) Added problem number 221

Related Issue
Fixed : #843
Description
I had Added the Solution of problem "Minimum Cost to Connect Two Groups of Points" under Leetcode Solution > 0200 - 0299> 221
Type of PR

[ ] Bug fix
[x] Feature enhancement
[x] Documentation update
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ x] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

Hi @Ajay-Dhangar , i have added solution you can review it.
And tell me the suggestions.

Hi, @Ajay-Dhangar resolved, can you check it and give suggestions


release testnet 39: Praxidike

Testnet Release
Testnet name: Praxidike
Release date: January 9, 2023
Testnet release manager: @redshiftzero
Testnet Release Manager Checklist
Preceding Friday (sprint planning day):

[x] Create GitHub project column, work with team to populate the milestone with tickets targeted for the release.

Tuesday (this will be done on Jan 5 or 6, 2023):

[ ] Construct the genesis data for the release:

[ ] Create new testnet directory with initial genesis allocations for this testnet by running cd testnets && ./new-testnet.sh

This genesis data will be used for testnet-preview with a randomized version of the future testnet's chain ID.

Thursday:

[ ] Check in with team again in a release meeting and update the GitHub milestone to ensure it represents what will make it into the testnet.
[ ] Draft an announcement for peer review to ensure major changes included are comprehensive.

Following Monday (release day):

[ ] Verify that testnet-preview.penumbra.zone is operational; it is redeployed on every push to main, and is an exact preview of what is about to be deployed.
[ ] Create new git tag e.g. 006-orthosie on main (tags created on any other branches will not transfer when merged in) and push to shared remote: git tag -a <tag_name> - must be annotated tag, i.e. git tag -a for Vergen build. This will create a Waiting GitHub Action for deployment.
[ ] You must manually review the Waiting deployment in the GitHub Action UI before the deployment will begin. Monitor the GitHub action to ensure it completes after it is approved.
[ ] Update the User Guide to mention the newly created git tag.
[ ] Delegate to the Penumbra Labs CI validator
[ ] Make GitHub release object and include the announcement
[ ] Make the announcement to Discord! 🎉🎉🎉

We experienced a chain halt on the long-running 038-kalyke testnet, so I'm going to proceed with releasing 039 today, rather than waiting till next Monday.

New testnet is live. The early birds in Discord are reporting trouble connecting to peers, though. It looks like some local IPv4 addresses made it into the config, which shouldn't happen because of https://github.com/penumbra-zone/penumbra/pull/1672. Still, it is happening. Leaving this ticket open to follow up on tomorrow and report results.

For the fix to Praxidike, I'm going to re-deploy on the old infra to keep things moving.

After some discussion in Discord, we decided to keep the testnet offline, rather than reuse the legacy infra. I'll focus on resolving the remaining blockers and post updates regularly.


add attrTween transition to Donut chart

Hi,
How to add attrTween transition to donut chart when initialize event?

Hi there! We try to keep the issue queue clear for specific code-related problems with C3, could you please try asking your question on either the Google Group or Gitter instead? Thank you!


Setup of package.json, install depen, setup folders and index

Setup of package.json, install dependencies , setup sub folders and index.js

init of package and of basic folder structure and installations


[INLONG-2023][Bug]  Agent stream id is not passed to proxy

Title Name: [INLONG-2023][Bug]  Agent stream id is not passed to proxy
Fixes #2023
Motivation
Agent stream id is not passed to proxy
Modifications
Agent stream id passed to proxy
Documentation

Does this pull request introduce a new feature? (no)

Codecov Report

Merging #2025 (671494a) into master (59d9be6) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2025      +/-   ##
============================================
- Coverage     12.27%   12.26%   -0.01%
- Complexity     1157     1160       +3
============================================
  Files           413      413
  Lines         35225    35225
  Branches       5542     5542
============================================
- Hits           4325     4322       -3
- Misses        30129    30136       +7
+ Partials        771      767       -4

Impacted Files
Coverage Δ

.../java/org/apache/flume/sink/tubemq/TubemqSink.java
51.42% <0.00%> (-4.00%)
:arrow_down:

.../producer/qltystats/DefaultBrokerRcvQltyStats.java
45.70% <0.00%> (+0.39%)
:arrow_up:

.../tubemq/corebase/policies/FlowCtrlRuleHandler.java
34.51% <0.00%> (+0.44%)
:arrow_up:

.../inlong/tubemq/corebase/policies/FlowCtrlItem.java
40.00% <0.00%> (+1.11%)
:arrow_up:

Continue to review full report at Codecov.

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


[local_auth] throw error on LockedOut case when useErrorDialogs false

Hi, have found that when useErrorDialogs: false iOS still shows the platform specific dialog on handling errors, instead of throwing an exception into the application.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@arefimenko Are you planning on completing the rest of the items in the PR checklist?

@stuartmorgan hi, later maybe, sorry. Can you make it draft? Thanks!

Since this is marked as a draft and hasn't been touched in a few months I'm going to close it to clean out our review queue. Please don't hesitate to submit a new PR if you decide to revisit this. Thanks!


Surface better error message if creating too many slots

Response comes back with 409 conflict and message in body: "Cannot complete the operation because the site will exceed the number of slots allowed for the 'Dynamic' SKU."
Portal only displays:

THis is the same as Need to prevent creating a second slot in consumption plan  #1314  - we decided to hide the Create capability for multiple slots on Dynamic sku with a proper message.

Closing this since this is same as " Need to prevent creating a second slot in consumption plan #1314"


Implement optionsUrl for registry CustomRegistrationFields

This is required to implement #1080

Removed the implementation of validationUrl from the scope of this ticket as I think we don't have a good sense of the usage of this feature, and we can come back to add this if users want it


(feat) add .Cluster Info at Event Template Runtime

Thank you @chifu1234
Please use dev instead of main. Essentially we use dev for development and before release:

cut a new release branch out of dev
merge dev to main

Thank you!


ToMatrix() does not have backward operation implemented

Hi,
I just read through your paper and code  and really adore this project on lie group in tangent space.
I hope that the ToMatrix() operation could be fully differentiable for researchers to use.
Do you have a plan to implement the ToMatrix(), or may be FromMatrix() in the future? and a planned date in mind?
Cheers
Lixin

Hi Lixin, I agree this would be good to have. I do have equations for the gradients of ToMatrix so I can add this in the next few days. I will need to think about FromMatrix.
In the future, I am working on trying to make conversions from all different representations differentiable (e.g quaternions, matricies, ect).
Best,
Zach

Best, Zach.
Thanks again for the great contribution you have made.
Lixin


Refresh Render Issue Solved

Title and Issue number

Title: Refresh Rendering of Pages not Working
Issue No.:
Close #400

Screenshots/Video (mandatory)

Video only possible after deployment.... So can't make it you can first test it once then go with merging

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
[ ] I have gone through the contributing.md file before contributing.

Additional context (Mandatory):
Are you contributing under any Open-source programme?

[ ] I'm a GSSOC-EXT contributor
[ ] I'm a HACKTOBERFEST contributor

Please give the video of your work.

Here's the video:
https://github.com/user-attachments/assets/612a5aae-4aad-4088-bb71-8f14bd4e2567
Even after multiple refreshes the page renders.

Here's the video:
Screen.Recording.2024-11-07.101729.mp4
Even after multiple refreshes the page renders.

Okk first i have to test then after i can merge this PR.

👍

Well done @NK-Works


AMD MI210 only got 20% performance of Nvidia A100 of MultiRHSProf - 8rhs

I test latest code on MI210, I got 0.36TOPS,
if i removed #program unroll，I got 0.88TOPS.
compares to nvidia-A100 4.3 Tops, it's only 20%.
considering A100 and Mi210 has similar fp32 ops capability, is it reasonable or still have space to improve mi210 performance?
configs:
Lattice = 64 64 64 20
SIMULATeQCD code: main branch, git commit 2022 Aug 2, https://github.com/LatticeQCD/SIMULATeQCD/commit/551ccca01d1a443de928b32181a40d416c1dd6c2
-DARCHITECTURE="gfx90a" //for mi210
mpiexec -np 1 --allow-run-as-root ./MultiRHSProf (8rhs)

Hi Alice, thanks for looking into this, and sorry about the extremely late response.
We have been noticing poor performance on MI250X compared to A100. We are not yet sure what is causing this... We will let you know if we figure out what is causing our performance loss.


Release version 4.0.0

#8 is breaking since we return term::Result in public functions

Note, FYI that lalrpop is the only user of this library at this time (https://crates.io/crates/ascii-canvas/reverse_dependencies), so updating only matters for lalrpop.  Shipping this as breaking, in addition to being accurate is nice so that we can update lalrpop to use ascii-canvas 4.0 and term 1.0 at the same time, rather than carrying two incompatible term versions in the lalrpop dependency tree.

Should we also update the edition while we are at it?

Yeah, good idea on both comments.  Opened #15

@Pat-Lafon are you good with this now that your two suggestions are merged?

Published: https://crates.io/crates/ascii-canvas/4.0.0


Allow ATE to update boolean values of token data

I've tried to use Active Token Effects to toggle the Gradual Illumination of a token using ATL.light.gradual > Override > false and ATL.light.gradual > Add > false, with no success.
Asking about it at Foundry Discord, you answered that "ATE never had to deal with boolean stuff before, so I doubt it will work", and recommended me to open an issue for you to take a look, so here I am.

Fixed in https://github.com/kandashi/Active-Token-Lighting/tree/new-proto-rework


Include but don't flatten vectors from root branches

Is it possible to read a branch with entries with a vectors, but store them as entries which are arrays in the dataframe rather then duplicating rows and anding the __array_index column?

Unfortunately pandas doesn't support this[1]. There is the option to instead implement it as a multi-index which might be cleaner, but it still results in duplication.
[1] Technically there is a Panel class for building a 3D/4D dataframe-like object however this has been deprecated in favour of xarray.

Hi,
As a quick example I was envisioning something like
import numpy as np
import pandas as pd

d = {"col1": [5, 6, 7],
     "col2": [np.array([1, 2 ,3]), np.array([0]), np.array([4, 4, 4, 5])]}

df = pd.DataFrame(data=d)

which gives a dataframe
   col1          col2
0     5     [1, 2, 3]
1     6           [0]
2     7  [4, 4, 4, 5]

When reading in from ROOT, col2 would contain the cohesive arrays from TTrees which would ordinarily be split  into different entries by the flattening process.

@chrisburr pandas does fully support this, even root_numpy does :)
@lunik1 actually, it is as easy as:
ar1 = root_numpy.root2array(filenames="rootfile.root", treename='tree', branches=['branch_with_array'])
df = pd.DataFrame(ar1)

will give you a perfect DataFrame, even the columns are named in the correct way.
(I personally find handling of arrays to be very unfortunate in root_pandas, the main reason I still stick with root_numpy...)
I would be very interested as well if root_pandas would not forbid that as it currently limits itself actively (it tests and does not allow an array for conversion, as far as I remember)
So any plans in this direction?

Seems I'm outdated/incorrect about the functionality of both pandas and root_numpy.
This could be much nicer, I'll look into it.

@lunik1 @mayou36
Thanks for those snippets, root_pandas now supports this as of v0.3.1.
It turns out pandas doesn't support multidimensional data. You can however store arbitrary python objects inside dataframes, such as numpy arrays. This works out of the box for jagged arrays, however it takes a little bit more work if the array is square (which root_pandas now does automatically). For example:
scalar = np.random.random(5)
array = np.random.random((5,3))
jagged_array = np.array([np.random.random(i) for i in range(5)])

# This works
df = pd.DataFrame({'a': scalar, 'b': jagged_array})

# This raises "Exception: Data must be 1-dimensional"
df = pd.DataFrame({'a': scalar, 'b': array})

# This also works
new_array = np.zeros(len(array), dtype='O')
for i, row in enumerate(array):
    new_array[i] = row
df = pd.DataFrame({'a': scalar, 'b': new_array})


[BUG] The reference failed after the udf was added

Describe the bug
The reference failed after the udf was added.
Occurs in DAG, add HiveSql
Which version of Dolphin Scheduler:
1.2.0.release

could you add the exception log ?

This issue can refer to this issue #2435


Fixes 3423: add arm64 repos

Summary
adds the base arm64 repos
Testing steps
configure a cdn cert in the config if you have one, if you don't reach out to me to get one.
load in the repos:
make  repos-import
trigger the sync:
go run cmd/external-repos/main.go nightly-jobs
wait a while
make sure that all repos have snapshots.
Checklist

[x] Tested with snapshotting feature disabled and pulp server URL not configured if appropriate

https://issues.redhat.com/browse/HMS-3423

Hi, looks good:


can't find orgmode_store.bin.gz

I run Linux w/ sublime text v3099
I get this error message every time I launch Sublime Text since I installed orgmode.
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
+ Debug: Invalid language: Plain text. Available: Go, Python3, Ruby, Tcl, RHTML, Python, XBL, Django, XUL, HTML, Perl, SCSS, CSS, Mason, Node.js, Smarty, PHP, HTML5, Twig, Sass, JavaScript, TemplateToolkit, XSLT, XML, Less
Invalid language: Plain text. Available: Go, Python3, Ruby, Tcl, RHTML, Python, XBL, Django, XUL, HTML, Perl, SCSS, CSS, Mason, Node.js, Smarty, PHP, HTML5, Twig, Sass, JavaScript, TemplateToolkit, XSLT, XML, Less
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
unable to auto detect encoding for ~/docs/formelsamling/ss/main.pdf, using fallback encoding Western (Windows 1252)
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 224, in on_pre_close
    callback.on_pre_close(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 47, in on_pre_close
    self.save(view, 'on_pre_close')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 224, in on_pre_close
    callback.on_pre_close(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 47, in on_pre_close
    self.save(view, 'on_pre_close')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
unable to auto detect encoding for ~/docs/formelsamling/ss/main.pdf, using fallback encoding Western (Windows 1252)
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 224, in on_pre_close
    callback.on_pre_close(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 47, in on_pre_close
    self.save(view, 'on_pre_close')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 224, in on_pre_close
    callback.on_pre_close(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 47, in on_pre_close
    self.save(view, 'on_pre_close')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
unable to auto detect encoding for ~/docs/formelsamling/ss/main.pdf, using fallback encoding Western (Windows 1252)
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 224, in on_pre_close
    callback.on_pre_close(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 47, in on_pre_close
    self.save(view, 'on_pre_close')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'
Traceback (most recent call last):
  File "/opt/sublime_text_3/sublime_plugin.py", line 320, in on_deactivated
    callback.on_deactivated(v)
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 41, in on_deactivated
    self.save(view, 'on_deactivated')
  File "/home/user/.config/sublime-text-3/Packages/orgmode/orgmode_store.py", line 90, in save
    with GzipFile(self.store, 'wb') as f:
  File "./gzip.py", line 184, in __init__
FileNotFoundError: [Errno 2] No such file or directory: '/opt/sublime_text_3/Settings/orgmode-store.bin.gz'

The gz it is trying to find I believe is located at:
/home/user/.config/sublime-text-3/Packages/Settings/orgmode_store.bin.gz

To temporarily solve this just copy the file from /home/user/.config/sublime-text-3/Packages/Settings/orgmode_store.bin.gz into /opt/sublime_text_3/Settings/orgmode-store.bin.gz.
Also, give it permissions so that the user you're running Sublime with can access it.
That's all.

This commands as workaround for this bug.
sudo cp ./orgmode-store.bin.gz /opt/sublime_text/Settings/orgmode-store.bin.gz
sudo chown -R user:user_main_group /opt/sublime_text/Settings/orgmode-store.bin.gz

But how about "right" method of this bug correction?
Maybe rewrote this piece of code from orgmode_store.py:
def __init__(self, *args, **kwargs):
        self.debug = False
        self.db = {}
        self.store = join(
            abspath(sublime.packages_path()),
            'Settings',
            'orgmode-store.bin.gz')
        try:
            makedirs(dirname(self.store))
        except:
            pass
        try:
            with GzipFile(self.store, 'rb') as f:
                self.db = load(f)
        except:
            self.db = {}

With kind regards.

Think we got a fix here: fa487b1a493afcaabae4a640671b66170b814fb4


some odd bug that makes basic almost useless

I thought I was losing my mind, it works for other people
I have found a quite cumbersome work-around
LOAD /bbcbasic.bin
RUN  .   /some/path/to/some.bas
-----  this will work, nothing else will
So you have to *bye and do this all over again for some other bas
ie (errors are)
LOAD some.bas
No such variable
LOAD .  some.bas
Type mismatch
If I replace LOAD with RUN the errors are the same
mos 1.04rc1
I don't know how to get basic version

And I did a update to console8 current, still the same
(both mos and vdp)

Stupid me,,   now I see, the filename has to enclosed in quotes..." "
LOAD "some.bas" ... this works OK
Maybe in the FAQ?


Failed to generate audio

I saw the demo on the readme page and sounds very good, so I went ahead I tested the code however I run into an issue where it worked for generating one of the two with error for the 2nd one:
Failed to generate audio for W. L. Oxley: Client error '400 Bad Request'
I tested it with multiple voices and always the error for the 2nd person. even I try switching the voice ID and still the same, error for the 2nd voice.

after restarting the installation it works but stuck on this step:
Crossfade duration: 10 ms
Crossfade duration: 10 ms

I saw the demo on the readme page and sounds very good, so I went ahead I tested the code however I run into an issue where it worked for generating one of the two with error for the 2nd one:
Failed to generate audio for W. L. Oxley: Client error '400 Bad Request'
I tested it with multiple voices and always the error for the 2nd person. even I try switching the voice ID and still the same, error for the 2nd voice.

You need to add and select the voice you want in your ElevenLabs dashboard and provide those voice ids in the prompt. Also I'm not sure if the free trial will allow you to generate one short podcast.

Thanks for the reply. I have a subscription with ElevenLabs, and after entering the Voice ID, it seems to work initially. However, it gets stuck on the final step in the log, and the files never combine.
The API successfully generates the files on ElevenLabs, but I can't get the process to complete as shown in your example.
Am I missing something, or is this a bug?

@Tobe2d not exactly sure why that's happening. If you received two the Crossfade duration: 10 ms twice on the terminal then probably export is taking time. Is there any ffmpeg related exception or any other exception?


Shared IAM role for node_groups.

We have a couple eks_mamanged_node_group's and we would like to share the same IAM roles as we want to have them static due to access to/from another account.
Currently it's seems trying to create IAM role per each node_group. And even more if we set create_iam_role=false and iam_role_arn='arn:xxx:xxx' we need to manage all permissions manually.
So after upgrading from v14 to v19 we need to import and manage IAM role and policies on our own.
Is there any way to do this using  module?

The shared IAM role would need the permissions applied - the module does not attach permissions to a role it did not create as this would cause a conflict


Improved 1_notmnist

Added a .gitignore for files created by ipython
Changed extract method such that extractions are performed only when necessary. This will make
subsequent runs of 1_notmnist faster.
Changed data_folders to have only folders.

Can one of the admins verify this patch?

Closing due to inactivity.


Make Client-Specific Override options more flexible. Implement #12522

add individual server overrides for the optional client settings (local routes/gateway, remote routes, DNS, NTP, Netbios...)

add option to override topology when selecting "Remove All Server Options" (push-reset)

add option to set gateways (useful when removing routes)

add option to override Redirect IPv6 Gateway in addition to existing IPv4 option

move all server override options to the "Override Configuration" group

hide override options with a "Select Server Overrides" toggle

disable un-needed overrides when "Remove All Server Options" (push-reset) is selected

when "Redirect IPv4/6 Gateway" is toggled, the Local Routes fields are hidden/shown correspondingly like for the server edit part

[x] Redmine Issue: https://redmine.pfsense.org/issues/12522

[x] Ready for review

Preview of changes on a playable page:
https://philz-cwm6.github.io/pfsense_preview/vpn_openvpn_csc_v1dot3_gui.html
If the GUI changes and settings are fine for you as a layout, I can proceed to adding actions in openvpn.inc

@PhilZ-cwm6 sidenote: I love how you posted a working version of the patch! Any tricks for doing that or is it just a copy of the branch on github?

@PhilZ-cwm6 sidenote: I love how you posted a working version of the patch! Any tricks for doing that or is it just a copy of the branch on github?

I download the page with full scripts using SingleFile chrome extension. I then add it to Github Pages:
https://pages.github.com/
It takes a few minutes for the page to be online, so it can let you think it doesn't work. Your repo must be init with Pages like in the link I posted

About the last upstream fix for the 0 / empty, here's a gist with test code for all exceptions
https://gist.github.com/PhilZ-cwm6/4676368b77fc01d478f42b5c94f26a66

@jim-p
I pushed the full changes with the actions set in openvpn.inc. That was the easier part as most of them were already there and just needed the settings implementation through GUI
A clickable preview of the page is here:
https://philz-cwm6.github.io/pfsense_preview/vpn_openvpn_csc-v1dot5.html
I fully tested all the functions and client side logs on connection.
The config shown in preview page for example does a push-reset followed by all needed actions for a working connection.
This simplifies the user interaction as it is not longer mandatory to enter custom options.
We can also remove selectively server-defined routes or DNS servers and push them to the custom config without a whole push-reset
I posted a patch in redmine

Let me know if there is interest / time in this for a planned merge, and I will commit the changes to resolve conflicts

Hi, this was ever commited? I have a request, but as it's a small one it could be lumped together with this Openvpn client change, please let me know your thoughts.
Currently I have a lot of Openvpn clients configured, but every time I reboot my pfSense all clients are restarted. There's missing an option to select which clients should be restarted at boot. This could be implemented with a checkbox on the client configuration and checking these flags on openvpn.inc (for what I investigated, not an expert and I could be wrong).
Thanks in advance!

I rebased and limited the changes to the client specific overrides
Hope it can be reviewed
Preview link:
https://philz-cwm6.github.io/pfsense_preview/vpn_openvpn_csc_v1dot3_gui.html

Nits:

Netbios -> NetBIOS
ipv4 / ipv6 -> IPv4 / IPv6
There's a lot of redundancy in the individual overrides descriptions. Perhaps a list selection (like the Server List) would fit better here, then a single description can be used - e.g. 'A "push-remove" option will be sent to the client for the selected options, removing the respective server-defined option. Client-specific options on this page are not affected.'

Notes:

I don't think the "Redirect Gateway" setting should be split into individual IPv4/6 options in the GUI. That should be left to the advanced options.
Local/Remote Routes Settings should be kept in the Tunnel Settings group.
Some changes were made to these pages (related to NetBIOS/WINS/NBDD) - conflicts need to be resolved.
I've not looked at the code yet, but it seems like there are breaking changes for current configurations. These should be resolved in upgrade_config.inc so previously configured CSOs maintain the same behavior.

@marcos-ng
I fixed the code as your suggestions.
Before committing, I have 2 questions about your comments:

There's a lot of redundancy in the individual overrides descriptions. Perhaps a list selection (like the Server List) would fit better here, then a single description can be used - e.g. 'A "push-remove" option will be sent to the client for the selected options, removing the respective server-defined option. Client-specific options on this page are not affected.'

Is it better now if it looks like attached pic, rather than using a list without detailed info on the commands ?

I don't think the "Redirect Gateway" setting should be split into individual IPv4/6 options in the GUI. That should be left to the advanced options.

That's the case already in the server options where it shows in GUI. So, if removed from CSOs, it will be a discrepancy between both interfaces, like it was before, with some options not set in GUI for CSOs while available in server
After your feedback, I will look at the upgrade_config.inc

I fixed the conflicts and changed as you suggested

removed redundancy with shorter descriptions
about your comment: "Redirect Gateway setting should be split into individual IPv4/6 options in the GUI. That should be left to the advanced options."

That's the case already in the server options where it shows in GUI. So, if removed from CSOs, it will be a discrepancy between both interfaces, like it was before, with some options not set in GUI for CSOs while available in server.

upgrade_config.inc
Sorry, I did not find any issues because all old CSO settings were not changed in new code. Importing old config on new update brings the current CSO as they were configured and they shoudl work. I am not used to the code there, but let me know exactly where are the issues if any.

Thank you

Given that the redirect gateway option is split between IPv4/6 in the server settings, I agree with separating the option here as well. Since the previous config behavior is to redirect both IPv4/6, upgrade code is needed to handle gwredir. For example, on upgrade:

Change the id name from gwredir to gwredir4
In upgrade_config.inc, check for gwredir and enable/disable gwredir4/gwredir6 accordingly

There should be an IPv6 equivalent to the new IPv4 Gateway option, and both should be placed above the "Redirect IPv4 Gateway" setting. Additionally, push_reset would also enable server_overrides_enabled in the upgrade code. There may be other instances of this, I have not checked all options.
Glancing at the code itself, I see some commented out code and it's not clear why it's there.
There are some changes to the fontawesome classes which need to be omitted - keep the original classes as is.

Really sorry, my previous commit, for some reason, missed some file changes and commits to fontawesome classes were tocuhed
I fixed the code properly, and I will look at your suggestions later

Given that the redirect gateway option is split between IPv4/6 in the server settings, I agree with separating the option here as well. Since the previous config behavior is to redirect both IPv4/6, upgrade code is needed to handle gwredir. For example:

Change the id name from gwredir to gwredir4
In upgrade_config.inc, check for gwredir and enable/disable gwredir4/gwredir6 accordingly

The previous code did properly implement both gwredir (ipv4) and gwredir6 (ipv6) variables in the server GUI to redirect IPv4 and/or IPv6 traffic:
https://github.com/pfsense/pfsense/blob/master/src/etc/inc/openvpn.inc#L802-L807
The options were missing in CSO GUI and were only available as custom options. My patch just adds the existing gwredir and gwredir6 to the CSO GUI code

There should be an IPv6 equivalent to the new IPv4 Gateway option

The current code doesn't support pushing IPv6 gateway even in server:
OpenVPN doesn't support this yet, clients do not recognize the option fully.
https://github.com/pfsense/pfsense/blob/master/src/etc/inc/openvpn.inc#L1129-L1134
That's why, I added a commented code for IPv6 gateway push "route-ipv6-gateway"
Adding the IPv6 gateway option in CSO GUI will only provide a non working feature
Once available in pfsense, we can just uncomment the code and unviel the GUI option in CSO
When sending a push-reset, having the IPv4 gateway option in GUI is needed to get the config properly working without advanced custom options

and both should be placed above the "Redirect IPv4 Gateway" setting.

I just pushed a commit to follow your recommendation for this

Additionally, push_reset would also enable server_overrides_enabled in the upgrade code. There may be other instances of this, I have not checked all options.

That part I am still not sure what you mean as in my tests, I did not find issues to upgrade from old to new configs

Glancing at the code itself, I see some commented out code and it's not clear why it's there.

I will clean it once a final commit is validated

There are some changes to the fontawesome classes which need to be omitted - keep the original classes as is.

fixed my commit, sorry

Regarding the topology option, it states:

It must match the actual topology specified in server. NOTE: This will perform a "push topology [selection]" without a previous "push-remove topology".

If the pushed topology "must match the actual topology specified in server", then why allow pushing a different one at all? If it must match, then topology_override is enough.

That part I am still not sure what you mean as in my tests, I did not find issues to upgrade from old to new configs

If push_reset is enabled in an older config, then there should be an option in upgrade_config.inc that would automatically enable server_overrides_enabled in the config. Thinking about this some more however, the config doesn't need to track this at all since the option only serves as a display toggle. Regardless, we can instead use a list as mentioned previously - see below for some code to get it started.
//==== Form fields
$section->addInput(new Form_Select(
	'reset_options',
	'Reset Server Options',
	$pconfig['reset_options'] ?? 'keep_all', // if empty, default to not reset options
	[
		'keep_all' => 'Keep server options', // default, no config, unsets all
		'push_reset' => 'Reset all options', // unsets specified
		'push_reset_keep_topology' => 'Only keep topology', // unsets specified
		'remove_specified' => 'Remove specified options' // no config, display toggle
	]
))->setHelp('Prevent this client from receiving server-defined client settings. Other client-specific options on this page will supersede these reset options.');
$group = new Form_Group('Reset Options');
$group->addClass('override_list');
$group->add(new Form_Select(
	'override_list',
	null,
	$pconfig['override_list'],
	[
		'remove_route' => 'Local Routes',
		'remove_iroute' => 'Remote Routes',
		'remove_dnsdomain' => 'DNS Domains',
		'remove_dnsservers' => 'DNS Servers',
		'remove_ntpservers' => 'NTP Options',
		'remove_netbios_ntype' => 'NetBIOS Type',
		'remove_netbios_scope' => 'NetBIOS Scope',
		'remove_wins' => 'WINS Options'
	],
	true
))->setHelp('A "push-remove" option will be sent to the client for the selected options, removing the respective server-defined option.');
$section->add($group);

//=== Javascript
function resetoptions_change() {
	hideClass('override_list', ($('#reset_options').find('option:selected').val() != 'remove_specified'));
}
// change handler
$('#reset_options').on('change', function() {
	resetoptions_change();
});
// initial page load
resetoptions_change();

If the pushed topology "must match the actual topology specified in server", then why allow pushing a different one at all? If it must match, then topology_override is enough.

Ok, you are right, I verified and it is in fact a client setting only. So, it can be set differently here and override the server setting.
I just should fix the hint text
I wanted to make minimal code changes in openvpn.inc and just enhance the current pfsense gui which makes it manadatory to define the topology and gateway manually in custom options to get a working setup
I did not mean to make it a break-proof setup for un-aware users.
That was my initial idea. What are your thoughts about it now that this option can be set by user differently from the server defined one ?

If push_reset is enabled in an older config, then there should be an option in upgrade_config.inc that would automatically enable server_overrides_enabled in the config. Thinking about this some more however, the config doesn't need to track this at all since the option only serves as a display toggle.

Yes, it is not needed. I looked at it and I am quiet sure that the changes won't break old configs. I use same options and I checked that these exceptions are accounted for in the vpn_openvpn_csc.php file
I also just tested, and when push_reset is checked in old config, it is refelected in new one, same when remove_route is checked

Regardless, we can instead use a list as mentioned previously - see below for some code to get it started.
//==== Form fields
$section->addInput(new Form_Select(
	'reset_options',
	'Reset Server Options',
	$pconfig['reset_options'] ?? ($pconfig['override_list'] ? 'remove_specified' : 'keep_all'), // if empty, default to not reset options
	[
		'keep_all' => 'Keep server options', // default, unsets config option, unsets all
		'push_reset' => 'Reset all options', // unsets all specified, sends push-reset
		'push_reset_keep_topology' => 'Only keep topology', // unsets all specified, sends push-reset and server topology
		'remove_specified' => 'Remove specified options' // unsets config option, display toggle
	]
))->setHelp('Prevent this client from receiving server-defined client settings. Other client-specific options on this page will supersede these reset options.');
$group = new Form_Group('Reset Options');
$group->addClass('override_list');
$group->add(new Form_Select(
	'override_list',
	null,
	$pconfig['override_list'], // e.g. ['remove_iroute', 'remove_dnsservers']
	[
		'remove_route' => 'Local Routes',
		'remove_iroute' => 'Remote Routes',
		'remove_dnsdomain' => 'DNS Domains',
		'remove_dnsservers' => 'DNS Servers',
		'remove_ntpservers' => 'NTP Options',
		'remove_netbios_ntype' => 'NetBIOS Type',
		'remove_netbios_scope' => 'NetBIOS Scope',
		'remove_wins' => 'WINS Options'
	],
	true
))->setHelp('A "push-remove" option will be sent to the client for the selected options, removing the respective server-defined option.');
$section->add($group);

//=== Javascript
function resetoptions_change() {

	hideClass('override_list', ($('#reset_options').find('option:selected').val() != 'remove_specified'));
}
// change handler
$('#reset_options').on('change', function() {
	resetoptions_change();
});
// initial page load
resetoptions_change();

Ok, I will look at it if you really prefer a list
But if you agree, I would drop the push_reset_keep_topology option since I clarified this above ?

I agree.

I've reworked this MR to implement the feature in a simpler way. Give there are significant changes, I'm closing this and updating the redmine. Thanks!


siege: build on all Unix

Confirmed to work on Darwin

@GrahamcOfBorg build 44985

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: 44985
Partial log (click to expand)
Cannot nix-instantiate `44985' because:
error: the expression selected by the selection path '44985' should be a list but is a set

@GrahamcOfBorg build siege

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: 44985
Partial log (click to expand)
Cannot nix-instantiate `44985' because:
error: the expression selected by the selection path '44985' should be a list but is a set

Success on x86_64-linux (full log)
Attempted: siege
Partial log (click to expand)
these paths will be fetched (0.09 MiB download, 0.25 MiB unpacked):
  /nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4
copying path '/nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4' from 'https://nix-cache.s3.amazonaws.com'...
/nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: 44985
Partial log (click to expand)
Cannot nix-instantiate `44985' because:
error: the expression selected by the selection path '44985' should be a list but is a set

Success on aarch64-linux (full log)
Attempted: siege
Partial log (click to expand)
these paths will be fetched (0.08 MiB download, 0.24 MiB unpacked):
  /nix/store/g7vwnkgfhqqf5yqza8kk5c2bfyccrcq2-siege-4.0.4
copying path '/nix/store/g7vwnkgfhqqf5yqza8kk5c2bfyccrcq2-siege-4.0.4' from 'https://cache.nixos.org'...
/nix/store/g7vwnkgfhqqf5yqza8kk5c2bfyccrcq2-siege-4.0.4

Success on x86_64-darwin (full log)
Attempted: siege
Partial log (click to expand)
make[2]: Leaving directory '/private/tmp/nix-build-siege-4.0.4.drv-0/siege-4.0.4/html'
make[1]: Leaving directory '/private/tmp/nix-build-siege-4.0.4.drv-0/siege-4.0.4/html'
post-installation fixup
gzipping man pages under /nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4/share/man/
strip is /nix/store/251fvx4mx9q4zarbca5cciasmn66p668-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4/bin
patching script interpreter paths in /nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4
/nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4/bin/bombardment: interpreter directive changed from "/bin/sh" to "/nix/store/hp9yq2m1i066irvasv20zr2lmj54fk9h-bash-4.4-p23/bin/sh"
/nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4/bin/siege.config: interpreter directive changed from "/bin/sh" to "/nix/store/hp9yq2m1i066irvasv20zr2lmj54fk9h-bash-4.4-p23/bin/sh"
/nix/store/fb9f227c97ifbw45iqbn9sqdzzv4wn69-siege-4.0.4

Success on aarch64-linux (full log)
Attempted: siege
Partial log (click to expand)
/nix/store/g7vwnkgfhqqf5yqza8kk5c2bfyccrcq2-siege-4.0.4

Success on x86_64-linux (full log)
Attempted: siege
Partial log (click to expand)
these paths will be fetched (0.09 MiB download, 0.25 MiB unpacked):
  /nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4
copying path '/nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4' from 'https://cache.nixos.org'...
/nix/store/x9s1kscrnqfrnz8wd9ynjz7r8h2dwdg9-siege-4.0.4


Adding subscriptions results in error

When the monitor items method calls subscription.Create, i receive the following error:
Opc.Ua.ServiceResultException: 'BadSubscriptionIdInvalid'
This happens when i try to add a larger number of items to monitor, and it happens completely randomly, have not established a pattern.
Any idea why this is happening and how i could resolve it?
On a related note, would you recommend putting all my monitored items in one subscription or should i have separate subscriptions for all items?

I'm working on a release which will allow you to pass options to the UaClient which might fix your issue.
Take a look here. Setting one of these differently might fix your issue:
https://github.com/hylasoft-usa/h-opc/blob/master/h-opc/Ua/UaClientOptions.cs
The way I use the monitor is usually like so:
foreach (var itemTag in itemTags) {
  client.monitor<bla>(itemTag, (tag, val) => {}); //...
};

Hey, thanks for the fast response!
I use the monitor method in the same way (although i extended it a bit to include item quality, status timestamp, id etc... and pipe it all into one method that processes it depending on the tag). Ive got over 500 tags to monitor, so it is a bit much i guess. The siemens DA wrapper also wasnt happy to cooperate so i turned to UA and found your library as a good place to start over.
I sadly already tried to mess with all those settings, i also pulled the latest base libraries from the OPC foundation but nothing got rid of this problem. It seems like the create method sets an ID that is not valid... Strangely if i just use one subscription and add all my monitored items into that it works just fine.
Do you see any problem in doing that?

Can you share a code example of what you mean by one subscription. Not sure
I'm on the same page
On Fri, Jul 28, 2017 at 11:33 AM ParsedID<EMAIL_ADDRESS>wrote:

Hey, thanks for the fast response!
I use the monitor method in the same way (although i extended it a bit to
include item quality, status timestamp, id etc... and pipe it all into one
method that processes it depending on the tag). Ive got over 500 tags to
monitor, so it is a bit much i guess. The siemens DA wrapper also wasnt
happy to cooperate so i turned to UA and found your library as a good place
to start over.
I sadly already tried to mess with all those settings, i also pulled the
latest base libraries from the OPC foundation but nothing got rid of this
problem. It seems like the create method sets an ID that is not valid...
Strangely if i just use one subscription and add all my monitored items
into that it works just fine.
Do you see any problem in doing that?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hylasoft-usa/h-opc/issues/55#issuecomment-318701321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACjlTm8Ypd5CF30bbn2_JsWB-eB3wdLnks5sSg1MgaJpZM4OmruH
.

Something like this - note i just typed this on my tablet from memory, i didnt build and check it. I can post my functions on monday when im back if you want.
public void Monitor(List tagList, string subscriptionName, Action<MonitoredItem, object, Action> callback)
{
var sub = new Subscription
{
PublishingInterval = _options.DefaultMonitorInterval,
PublishingEnabled = true,
LifetimeCount = _options.SubscriptionLifetimeCount,
KeepAliveCount = _options.SubscriptionKeepAliveCount,
DisplayName = subscriptionName,
Priority = byte.MaxValue
};
  foreach (tag in taglist)
  {
    var node = FindNode(tag);

    var item = new MonitoredItem
    {
      StartNodeId = node.NodeId,
      AttributeId = Attributes.Value,
      DisplayName = tag,
      SamplingInterval = _options.DefaultMonitorInterval
    };

    item.Notification += (monitoredItem, args) =>
    {
      var p = (MonitoredItemNotification)args.NotificationValue;
      var t = p.Value.WrappedValue.Value;
      Action unsubscribe = () =>
      {
        sub.RemoveItems(sub.MonitoredItems);
        sub.Delete(true);
        _session.RemoveSubscription(sub);
        sub.Dispose();
      };
      callback(monitoredItem, t, unsubscribe);
    };

    sub.AddItem(item);
  }

  _session.AddSubscription(sub);
  sub.Create();
  sub.ApplyChanges();

}

I don't know of any reason why that would cause problems. Let me know how it works out for you.

Updated with the actual functions. This works just fine for our case.

Note that we also had to change the options, as subscription would stop working after a while. This also allows us to debug without the callbacks stopping.
        SessionTimeout = 3600000U;
        UseMessageSecurity = false;
        SubscriptionLifetimeCount = 500;
        SubscriptionKeepAliveCount = 5;
        MaxSubscriptionCount = 500;
        MaxMessageQueueSize = 500;
        MaxNotificationQueueSize = 50;
        MaxPublishRequestCount = 50;

Very good. Will review
On Mon, Jul 31, 2017 at 6:25 AM ParsedID<EMAIL_ADDRESS>wrote:

Note that we also had to change the options, as subscription would stop
working after a while. This also allows us to debug without the callbacks
stopping.
    SessionTimeout = 3600000U;
    UseMessageSecurity = false;
    SubscriptionLifetimeCount = 500;
    SubscriptionKeepAliveCount = 5;
    MaxSubscriptionCount = 500;
    MaxMessageQueueSize = 500;
    MaxNotificationQueueSize = 50;
    MaxPublishRequestCount = 50;

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hylasoft-usa/h-opc/issues/55#issuecomment-319040834,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACjlToB35gy6JKXd0Ue6NBy6HDGyNPLiks5sTbmVgaJpZM4OmruH
.


[ARM BugBash] Need instructions to populate LUIS appsettings.json values

Sample information
Any sample that's using LUIS
Describe the bug
We need to specify instuctions on how to populate appsettings.json LUIS section (better yet, ssome automation around retrieving the values)
{
"MicrosoftAppId": "",
"MicrosoftAppPassword": "",
"LuisAppId": "",
"LuisAPIKey": "",
"LuisAPIHostName": ""
}
To Reproduce
Steps to reproduce the behavior:
Download sample or template that's using LUIS. Try to use LUIS in your bot.
Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

Closing issue as we're not using the README-LUIS.md in these samples at the moment, but instead pointing to some online Docs. The bug has been moved to stevengum/arm-templates-for-sample#1 so the bug is still fixed.


Use new Rollup output file prop

opts.dest was deprecated around<EMAIL_ADDRESS>see:
https://github.com/rollup/rollup/commit/30cd1caeaa2e4741c98fc2311e43afbefd89142b

Thanks for the PR, let's make this backwards compatible and then merge it.


Don't use jar files, use dependencies from mavenCentral/jCenter

Both remaining jar files, guava and joda-time, are available in mavenCentral, why not use those?
compile "com.google.guava:guava:17.0"
compile "joda-time:joda-time:2.5"

I cannot see any changes regarding this, should I open a PR?

Sure

https://github.com/pklinken/Earmouse/pull/6

Cheers :)
On 14 April 2016 at 21:35, Boris Kraut<EMAIL_ADDRESS>wrote:

#6 https://github.com/pklinken/Earmouse/pull/6
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/pklinken/Earmouse/issues/1#issuecomment-210114207