invalid bls_signature

At Vector chain we are trying to connect a public testnet of our chain with Sepolia testnet using IBC with LCP. We have the following setup:

We  are using the version v0.2.9 of the LCP service application as well as the corresponding enclave
no debug mode enclave
with mainnet preset enclave
simulated remote attestation mode
We are using the IBC solidity contracts of the v0.2.9 comos-ethereum-ibc-lcp demo repository.
yui-relayer v0.5.4 (with extensions, because vector chain is a cosmos chain using eth accounts)
lcp-go v0.2.10

We have the following chain and path config. Some of the fields are marked with <> - because they contain confidential data or are bound to the local system I am using:
{
  "global": {
    "timeout": "10s",
    "light-cache-size": 20,
    "logger": {
      "level": "DEBUG",
      "format": "json",
      "output": "stderr"
    }
  },
  "chains": [
    {
      "chain": {
        "@type": "/relayer.chains.tendermint.config.ChainConfig",
        "key": "ibc",
        "chain_id": "vsc_420044-1",
        "rpc_addr": "<vector-rpc>",
        "account_prefix": "vsc",
        "gas_adjustment": 1.0,
        "gas_prices": "1000000000000avsg",
        "average_block_time_msec": 6000,
        "max_retry_for_commit": 5
      },
      "prover": {
        "@type": "/relayer.provers.lcp.config.ProverConfig",
        "origin_prover": {
          "@type": "/relayer.chains.tendermint.config.ProverConfig",
          "trusting_period": "70h",
          "refresh_threshold_rate": {
            "numerator": 2,
            "denominator": 3
          }
        },
        "lcp_service_address": "localhost:50051",
        "mrenclave": "<mr-enclave>",
        "allowed_quote_statuses": [
          "GROUP_OUT_OF_DATE"
        ],
        "allowed_advisory_ids": [
          "INTEL-SA-00219",
          "INTEL-SA-00289",
          "INTEL-SA-00334",
          "INTEL-SA-00477",
          "INTEL-SA-00614",
          "INTEL-SA-00615",
          "INTEL-SA-00617",
          "INTEL-SA-00828"
        ],
        "key_expiration": 604800,
        "elc_client_id": "07-tendermint-1",
        "message_aggregation": true,
        "is_debug_enclave": false,
        "operators": [
          "<operator-address>"
        ],
        "operator_signer": {
          "@type": "/relayer.provers.lcp.signers.raw.SignerConfig",
          "private_key": "<operator-priv-key>"
        },
        "operators_eip712_evm_chain_params": {
          "chain_id": 11155111,
          "verifying_contract_address": "<lcp-contract-address>"
        }
      }
    },
    {
      "chain": {
        "@type": "/relayer.chains.ethereum.config.ChainConfig",
        "chain_id": "eth",
        "eth_chain_id": 11155111,
        "rpc_addr": "https://ethereum-sepolia-rpc.publicnode.com",
        "signer": {
          "@type": "/relayer.chains.ethereum.signers.hd.SignerConfig",
          "mnemonic": "<eth-6-word-signing-seedphrase>",
          "path": "<eth-derivation-path>"
        },
        "ibc_address": "<ibc-handler-contract-address>",
        "initial_send_checkpoint": 0,
        "initial_recv_checkpoint": 0,
        "enable_debug_trace": true,
        "average_block_time_msec": 12000,
        "max_retry_for_inclusion": 5,
        "allow_lc_functions": {
          "lc_address": "<lcp-contract-address>",
          "selectors": [
            "0xa97c61d6",
            "0x6ac73aa0"
          ]
        },
        "gas_estimate_rate": {
          "numerator": 3,
          "denominator": 2
        },
        "max_gas_limit": 10000000,
        "tx_type": "auto",
        "blocks_per_event_query": 1000,
        "abi_paths": [
          "<where-the-abi-resides>"
        ]
      },
      "prover": {
        "@type": "/relayer.provers.lcp.config.ProverConfig",
        "origin_prover": {
          "@type": "/relayer.provers.ethereum_light_client.config.ProverConfig",
          "beacon_endpoint": "<private-sepolia-beacon-endpoint>",
          "network": "mainnet",
          "trusting_period": "168h",
          "max_clock_drift": "0",
          "refresh_threshold_rate": {
            "numerator": 2,
            "denominator": 3
          }
        },
        "lcp_service_address": "localhost:50051",
        "mrenclave": "<mr-enclave>",
        "allowed_quote_statuses": [
          "GROUP_OUT_OF_DATE"
        ],
        "allowed_advisory_ids": [
          "INTEL-SA-00219",
          "INTEL-SA-00289",
          "INTEL-SA-00334",
          "INTEL-SA-00477",
          "INTEL-SA-00614",
          "INTEL-SA-00615",
          "INTEL-SA-00617",
          "INTEL-SA-00828"
        ],
        "key_expiration": 604800,
        "elc_client_id": "ethereum-0",
        "message_aggregation": trie,
        "is_debug_enclave": false,
        "operators": [
          "<operator-address>"
        ],
        "operator_signer": {
          "@type": "/relayer.provers.lcp.signers.raw.SignerConfig",
          "private_key": "<operator-priv-key>"
        },
        "operators_eip712_cosmos_chain_params": {
          "chain_id": "vsc_420044-1",
          "prefix": "ibc"
        }
      }
    }
  ],
  "paths": {
    "redeem": {
      "src": {
        "chain-id": "vsc_420044-1",
        "port-id": "redeem",
        "order": "unordered",
        "version": "redeem-1"
      },
      "dst": {
        "chain-id": "eth",
        "port-id": "redeem",
        "order": "unordered",
        "version": "redeem-1"
      },
      "strategy": {
        "type": "naive",
        "src-noack": false,
        "dst-noack": false
      }
    }
  }
}

There is no problem creating the clients using the handshake method from the handshake script in the demo repository. However, when it comes to activcating the client vie yrly lcp activate-client redeem --src=false the update of the ELC fails with the following series of messages:
{"time":"2024-11-01T21:32:54.331916734Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).loadEKIAndCheckUpdateNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":143},"msg":"no active enclave key in memory","module":"lcp-prover","chain_id":"eth"}
{"time":"2024-11-01T21:32:54.332038094Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).loadEKIAndCheckUpdateNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":150},"msg":"no unfinalized enclave key info found","module":"lcp-prover","chain_id":"eth"}
{"time":"2024-11-01T21:32:54.332161876Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).loadEKIAndCheckUpdateNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":159},"msg":"load last finalized enclave key into memory","module":"lcp-prover","chain_id":"eth"}
{"time":"2024-11-01T21:32:54.332174363Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).loadEKIAndCheckUpdateNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":169},"msg":"active enclave key is finalized","module":"lcp-prover","chain_id":"eth"}
{"time":"2024-11-01T21:32:54.332195092Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).checkEKIUpdateNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":97},"msg":"checkEKIUpdateNeeded","module":"lcp-prover","chain_id":"eth","enclave_key":"e14b6f05c0ff0b381407b37293a14a6971b70066","now":1730496774,"attestation_time":1730495080,"expiration":604800,"update_time":1730797480}
{"time":"2024-11-01T21:32:54.334320781Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).UpdateEKIfNeeded","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":39},"msg":"loadEKIAndCheckUpdateNeeded","module":"lcp-prover","chain_id":"eth","updateNeeded":false}
{"time":"2024-11-01T21:32:54.334369881Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.activateClient","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":647},"msg":"try to activate the LCP client","module":"lcp-prover","chain_id":"eth","elc_client_id":"ethereum-100"}
{"time":"2024-11-01T21:32:54.336322892Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/beacon.Client.get","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/beacon/client.go","line":102},"msg":"Beacon API request","endpoint":"http://<>:5052/eth/v1/beacon/light_client/finality_update"}
{"time":"2024-11-01T21:32:54.422092072Z","level":"INFO","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/relay.(*Prover).buildAccountUpdate","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/relay/execution.go","line":21},"msg":"buildAccountUpdate: get proof","chain_id":"eth","module":"ethereum-light-client","block_number":6992629,"ibc_address":"0x9B74b00037e72427998aCB1F30C4d5234FB49b60","account_proof":"[encoded data, 6700 characters]","storage_hash":"0f42a18032b0c5570f0c58d853f6573c3ab5f87a2c066ee967d44825f373f0ef"}
{"time":"2024-11-01T21:32:54.422246784Z","level":"INFO","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/relay.(*Prover).GetLatestFinalizedHeader","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/relay/prover.go","line":281},"msg":"build latest finalized header","chain_id":"eth","module":"ethereum-light-client","block_number":6992629,"timestamp":1730495712,"state_root":"9bb1518d31f1836bb20ad935138360870246500d83d85ec70aaabc89c37ccc02"}
{"time":"2024-11-01T21:32:54.422356702Z","level":"INFO","source":{"function":"github.com/datachainlab/lcp-go/relay.(*Prover).updateELC","file":"/home/till/go/pkg/mod/github.com/vsc-blockchain/lcp-go@v0.2.10-vsc.3/relay/lcp.go","line":298},"msg":"try to setup headers","module":"lcp-prover","chain_id":"eth","elc_client_id":"ethereum-100","client_state.latest_height":{"revision_height":6992511},"latest":{"revision_height":6992629}}
{"time":"2024-11-01T21:32:54.422427044Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/relay.(*Prover).SetupHeadersForUpdate","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/relay/prover.go","line":115},"msg":"query the latest height of the counterparty chain","chain_id":"eth","module":"ethereum-light-client","latest_height":{}}
{"time":"2024-11-01T21:32:54.457225931Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/beacon.Client.get","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/beacon/client.go","line":102},"msg":"Beacon API request","endpoint":"http://<>/eth/v1/beacon/genesis"}
{"time":"2024-11-01T21:32:54.463180993Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/relay.(*Prover).SetupHeadersForUpdate","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/relay/prover.go","line":139},"msg":"try to setup headers for updating the light-client","chain_id":"eth","module":"ethereum-light-client","lc_latest_height":{"revision_height":6992511},"lc_latest_height_period":760,"latest_period":760}
{"time":"2024-11-01T21:32:54.463238033Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/beacon.Client.get","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/beacon/client.go","line":102},"msg":"Beacon API request","endpoint":"http://<>/eth/v1/beacon/light_client/updates?start_period=760&count=1"}
{"time":"2024-11-01T21:32:54.481950295Z","level":"DEBUG","source":{"function":"github.com/datachainlab/ethereum-ibc-relay-prover/beacon.Client.get","file":"/home/till/go/pkg/mod/github.com/datachainlab/ethereum-ibc-relay-prover@v0.3.1/beacon/client.go","line":102},"msg":"Beacon API request","endpoint":"http://<>/eth/v1/beacon/light_client/bootstrap/0xefa7316ee6ae0a5dc0bb326bbb904ccfe35dab62481b4902860ade302bc7c8e7"}
Error: All attempts fail:
#1: rpc error: code = Aborted desc = Command error: status=SGX_ERROR_UNEXPECTED descr=StringTracer: LightClientCommand error: LightClient error: Light Client specific error: ics02 error: `header verification failed with reason: `client specific error: `verification error: invalid bls signatures```

Location:
    /home/till/.cargo/registry/src/github.com-1ecc6299db9ec823/flex-error-0.4.4/src/tracer_impl/eyre.rs:10:9

It looks like the aggregated signature of the data from the beacon chain can not verified with the avaiable headers in the ethereum ELC. I was hoping to get some insight on the presented error or how to debug the issue.
With kind regards:
Till

Forgot to mention. That we have a e2e test derived from the cosmos-ethereum demo. Where we substituted the tendermint chain with our vector chain. The e2e test is running smoothly with the minimal preset. Handshake works like a charm.

Hi @fragwuerdig, sorry for the late reply.
This is probably cosmos-ethereum-ibc-lcp issue, not lcp issue.
Anyway, I have checked your configuration and I think the setting "network": "mainnet" need to be modified to "network": "sepolia".

Thank you very much. Very much appreciated. This resolved the issue. The issue can be closed now ❤

Glad to hear that!


Compatibility with Ant Pro Components

Hi there! Thanks for the awesome repo.
I've been trying to make use of this repo to enjoy its auto reload feature when a variable change. However, I find that its compatibility with Ant Design Pro is not as straightforward.
How to re-create
I've made a sample repo based on mkn over here: https://github.com/darrist/mkn-with-pro-components
Current Behavior

pro-* css classes is not being loaded to the app. These classes are provided by .less files within the library, and it should be automatically consumed when importing the component.

Intended Behavior

My Assumptions
I am thinking it is possibly related to lacking of less-loader, so that .less files inside the library cannot be loaded correctly.

Close this for now, I just realize they provide .css as well.
For anyone who wanted to know, it's here: https://procomponents.ant.design/en-US/docs/getting-started
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>


Combined guideline about documenting errors, panics, safety

Currently there are separate guidelines for how to document errors and how to document panics. Let's combine these and extend it to cover documentation of safety in unsafe functions.

Combined in ca737109257231ff6f5d871d5e21063eab51474b and added safety in e72e08293d326bda65e2a15217b307e5e9ecace7.


Corrected variable name layer to sublayer

@edbosne , Could you please make the changes to .md and .ipynb files too. Thanks!

@sachinprasadhs
Well noted. Thanks for letting me do it :D


Lazy loaded styles are never hashed

We are great fans of lazy-css functionality in ng build, it allows us to serve several apps from one code-base, as we add app1.css ... appN.css depending on the request, using angular universal.
Given our high-traffic page we use a CDN and therefore rely on changing hashes for changed CSS, as we otherwise would have to manage CSS cache invalidation manually.
How can we have hashed lazy CSS back?
See also #11235 and #11704
Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [x] build

Versions
$ node --version
v8.11.4

$ npm --version
5.10.0

$ ng --version

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 6.2.4
Node: 8.11.4
OS: darwin x64
Angular: 6.1.2
... animations, common, compiler, compiler-cli, core, http
... platform-browser, platform-browser-dynamic, platform-server
... router, service-worker

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.0.0
@angular-devkit/architect-cli      0.0.0
@angular-devkit/build-angular      0.0.0
@angular-devkit/build-ng-packagr   0.0.0
@angular-devkit/build-optimizer    0.0.0
@angular-devkit/build-webpack      0.0.0
@angular-devkit/core               0.0.0
@angular-devkit/schematics         0.0.0
@angular-devkit/schematics-cli     0.0.0
@angular/cdk                       6.4.5
@angular/cli                       0.0.0
@angular/material                  6.4.5
@angular/pwa                       0.0.0
@ngtools/json-schema               1.1.0
@ngtools/webpack                   0.0.0
@schematics/angular                0.0.0
@schematics/schematics             0.0.0
@schematics/update                 0.0.0
ng-packagr                         4.1.0
rxjs                               6.2.2
typescript                         2.9.2
webpack                            4.16.4

MacOS 10.14 Mojave
Repro steps
my-app/angular.json -> projects/my-apparchitect/build:
{
  "input": "src/extra.css",
  "lazy": true,
  "bundleName": "extra"
}

Then run:
$ ng build --prod

Example in: https://github.com/maosmurf/ng-demo/tree/lazy-styles
The log given by the failure
chunk {4} extra.css (extra) 0 bytes [initial] [rendered]

Desired functionality
extra.123456678.css instead of extra.css, as it was until recently.

Our use-case:
We render the app (kurier.at) and have lazy styles. Given we use angular universal, we have no problem checking /dist for the current css hash file.
After deploying a new version, the new css (with changed hash) is included, yet the old CSS remains valid on CDN (cloudflare) - so there are 2 benefits fron this:

no need for invalidating style.css
old CSS remains valid for old cached pages.

How can we get the old functionality back? Currently, our only option for upgrading to ng6 would be to manually add hashes after ng build --prod

Thank you for the feedback.

This is intentional
I fully understand the motivation for the recent changes, yet the backwards-incompatible changes caused some headache.

it could do so with a post build step.
Done.

I'm going to mark this as a feature request.
Would be great to have old functionality back using already provided tools (without the need of manual post-build step), so thank you for that :)

We are running into similar issue with invalidating lazy loaded stylsheets.  Would be great if there can be a config setting in angular.json to turn this on or off.

You create lazy-loadable hashed styles with Webpack.
My Angular CLI version: 8.3.19

ng new lazy-load-css --style css
Open the angular.json and remove styles.css from styles array.
Create style-loader.js in the src folder and then replace with the following code:

import styles from './styles.css';
const node = document.createElement('style');
node.textContent = styles;
document.querySelector('head').appendChild(node);

Add the ngOnInit method in the AppComponent like below:

  ngOnInit() {
    setTimeout(() => {
      import('src/style-loader');
    }, 1500);
  }

See Webpack import() function. Also you can see Webpack magic comments for chunk naming, etc.

Add a style to styles.css to see changes. I added the background color style.
Run the app

Run the ng build --prod to see the hash

@maosmurf have you found any workarounds?

Our workaround for this was to create a post-build script. Here's some code that anyone is free to use, you may need to modify sourceDir. It outputs a manifest.json to the root directory so that you are able to reference it later by your app at runtime.
We also used hasha for the actual hashing.
const hasha = require('hasha')
const fs = require('fs')

let manifest = {}

const sourceDir = './dist'
fs.readdirSync(sourceDir)
  .filter((filename) => filename.endsWith('.css'))
  .forEach((filename) => {
    const hash = hasha.fromFileSync(`${sourceDir}/${filename}`).substring(0, 10) // unique enough for our cache busting purposes

    const withoutExtension = filename.slice(0, -4)
    const hashedFileName = `${withoutExtension}.${hash}.css`
    fs.renameSync(`${sourceDir}/${filename}`, `${sourceDir}/${hashedFileName}`)
    manifest[filename] = hashedFileName
  })

fs.writeFileSync('./manifest.json', JSON.stringify(manifest, null, 4))

Our workaround for this was to create a post-build script. Here's some code that anyone is free to use, you may need to modify sourceDir. It outputs a manifest.json to the root directory so that you are able to reference it later by your app at runtime.
We also used hasha for the actual hashing.
const hasha = require('hasha')
const fs = require('fs')

let manifest = {}

const sourceDir = './dist'
fs.readdirSync(sourceDir)
  .filter((filename) => filename.endsWith('.css'))
  .forEach((filename) => {
    const hash = hasha.fromFileSync(`${sourceDir}/${filename}`).substring(0, 10) // unique enough for our cache busting purposes

    const withoutExtension = filename.slice(0, -4)
    const hashedFileName = `${withoutExtension}.${hash}.css`
    fs.renameSync(`${sourceDir}/${filename}`, `${sourceDir}/${hashedFileName}`)
    manifest[filename] = hashedFileName
  })

fs.writeFileSync('./manifest.json', JSON.stringify(manifest, null, 4))

@mehmet-erim your method seems interesting. Does webpack also minify / prefix styles added this way or we have to provide a css file minified/prefixed beforehand?

@bogacg Obviously I have not checked this. I have provided minified CSS files.

This feature is quite helpful because I want to refer this lazy loaded files within the application code. So, if this lazy loaded styles are hashed it would not be possible to refer these files within application code.
Can't we have any possible workaround if we want to access hashed file styles file from application code?
Case:
style files which are loaded (lazily and non lazily) both needs to referenced from service.

I would also like to comment in favour of this feature.
On one hand, we all know how important fast load time is for Web Apps, that's the main motivation of lazy loading modules in Angular.
The problem is that if I reference big UI libraries, like Angular Material or even a specific component, it is a pain to lazy load those styles so I have to include +1Mb of content as part of my main module.
Manually hashing the file is not desired as a simple npm update could change the content and no one might remember to update the hash.
If a post-build task is possible then why don't we include the hashing as default?
 is body-enabled so we could reference them inside any component. Why not to specify a component property inside the style object that tells the builder which component will reference the link and then it is added just as it adds  to index.html...

@clydin I think that's an oversimplification of styles management.
First, most complex app won't use pre-built theme, but custom themes, either by creating their own or by customizing the pre-built ones.
In addition you usually also add customizations for components that you don't want to duplicated on every angular component that uses it, like different colors, margins, sizes for tabs, list items, etc.
Where would you put those? If you import the styles in your global stylesheet you are still downloading several Kbs of styles that you don't use to bootstrap the app... if you add customizations to styles.scss to make them global, same thing...
The solution here would be to leverage lazy loading styles just the same way we leverage lazy loading modules...
Currently, my main module bundle is sitting at 1.22Mb after optimizations, and around 30-40% of it are styles imported in the styles.scss file

Styles that affect the entire application such as theming are recommended to be placed in a global stylesheet. With the latest style optimizations enabled it will be loaded in a non-blocking manner in parallel with the initial JavaScript bundles.
Styles that only affect a particular component are recommended to be placed in component specific styles (inline styles or external styleUrls).  If the component is lazy loaded then these styles will also be lazy loaded.  Since the component specific styles are required to be present for the component to successfully render, separately lazy loading the stylesheets for a component will increase the time necessary to initialize and render the component; as well as increase the amount of data that would need to be transferred due to the overhead of the additional file and the code necessary to bootstrap it.
Regarding the size of the main bundle, if the styles are already present in the global styles.scss, they should not need to be imported into any component styles.  There should ideally not be any rule overlap between the global stylesheet and component styles.

@clydin  global styles are included as part of the initial bundle for optimization/performance criteria.
What about styles for a Grid? Would you duplicate those styles on each component that uses the grid? Or would you add it to the global styles? If you add it to the global styles, why are we downloading KBs of styles that might never be displayed? (the grids might be part of an admin module?)

The Grid could be a third-party component, first-party or even non-Angular.  The encapsulation is still possible and as an application scales in size, this can become increasing beneficial.  For example, if the Grid needs to be changed to a different underlying implementation, the change only needs to happen in one place and the entire application is updated. Having a set of shared components that are used throughout an application is a frequently used pattern in Angular applications.
Angular material themes can be customized to only include specific components.  Please see the theming guide for additional information: https://material.angular.io/guide/theming#theming-only-certain-components
If this alone does not provide a global styles file that meets the project's requirements then the component theme styles can be bifurcated with the less used placed in a relevant lazy route's main component that has view encapsulation turned off.

This issue seems to have gone a bit off-topic, but to refocus on the original request of hashing lazy loaded styles...
We don't really have a feature for "lazy loaded styles" as defined here. You can bundle styles using the CLI and skip injecting them in the index.html page as is typically done. This pattern is more of an "unreferenced stylesheet" which can be manually loaded onto a page by application code. Angular doesn't do anything special with such a stylesheet after bundling it. (Sidebar: We do have critical CSS inlining which defers some styles to be loaded lazily, however this is a different feature than what's discussed here and is done transparently for developers.)
This distinction is important because appending a hash to the filename makes the reference unstable and means it can't be easily referenced by application code. By contrast, we can add hashes to JS files because the CLI typically controls all the references to them (the index.html page and dynamic imports of lazy loaded chunks), so we can make sure everything is consistent. Since the CLI doesn't have this same control over bundled styles, we have no hook to allow applications to use a stable reference to a stylesheet with a hash in the filename.
Leaving the hash out of the bundled CSS is the only way to effectively reference that file, so we think it is the right approach here. Including the hash could be useful if Angular had proper support for lazy loading styles, but that's a separate FR. If that's something that you think would be interesting and have a use case for it, then please file that FR.


Update validation.md

Fixes #30276

@cn-byn please review. @mkArtakMSFT please assign a reviewer.


Added: support for custom economic constraints gE in NonLinMPC

Codecov Report
Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.69%. Comparing base (44dba57) to head (9992daf).
Report is 8 commits behind head on main.

Files with missing lines
Patch %
Lines

src/controller/nonlinmpc.jl
87.50%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
- Coverage   98.96%   98.69%   -0.27%
==========================================
  Files          24       23       -1
  Lines        3574     3450     -124
==========================================
- Hits         3537     3405     -132
- Misses         37       45       +8

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

just a little note, I've never really come across the term "economic constraints" before, is this from some particular book on nonlinear MPC? I use nonlinear MPC with nonlinear constraints very often, but it almost never have an economic interpretation. Would it perhaps be more generic to call them "nonlinear constraints"? I'm thinking that someone who want's to add safety constraints such as collision avoidance will not find what they are looking for, not knowing to look for "economic constraints".

That's a good point. I named them economic constraints merely for consistency with my current notation. The nonlinear constraint function will not receive $\mathbf{U}$, $\mathbf{\hat{Y}}$ and $\mathbf{\hat{D}}$ but the slightly longer version appended with the $E$ subscript:

It is also consistent with the custom objective term $J_E$

Would it perhaps be more generic to call them "nonlinear constraints"?

It would not be precise enough since there is also the plant output and the terminal bounds that are also nonlinear (if the plant model is nonlinear). It's not a replacement of the nonlinear constraints but an addition to the current nonlinear constraints. Maybe just naming them "custom nonlinear constraints" would be sufficient. I would also change my notation to:
J_E(\mathbf{\bar{U}, \bar{Y}, \bar{D}, p}) \quad \text{and} \quad \mathbf{g_C}(\mathbf{\bar{U}, \bar{Y}, \bar{D}, \epsilon, p})

with the new notation $\mathbf{\bar{U}} = \mathbf{U}_E$, $\mathbf{\bar{Y}} = \mathbf{\hat{Y}}_E$ and $\mathbf{\bar{D}} = \mathbf{\hat{D}}_E$.
While we are at it, do you think that it's a bad idea to name the custom objective term the economic term? (but renaming it and changing the notation would be a breaking change since they are keyword arguments.)

While we are at it, do you think that it's a bad idea to name the custom objective term the economic term?

Not necessarily bad, but I'd imagine that many aren't familiar with the term.

but renaming it and changing the notation would be a breaking change since they are keyword arguments.

One way of changing stuff like this in a non-breaking way is to introduce the new keyword, keep the old, and then do something like
function fun(; new = default, old = nothing)
    new = something(old, new)
end

the function something returns the first argument that is not nothing. If the old keyword is removed from the docs, no one will use it in the future, but existing usage will continue to work.

@baggepinnen there is no test at all for now so it's very experimental. I'll implement this soon. 🙂
But for now, do you have any comments about the documentation ? (see NonLinMPC in the dev doc)

One problem I see right now is how it's easy to forget to assign the nc keyword argument in combination with the custom nonlinear constraint function. I'm thinking of calling the JE and the gc function at least once in the NonLinMPC constructor and emit some warnings to guide the user a little bit.
Just calling sim! on NonLinModel is a quick way to troubleshoot bugs in the f and h functions. There is now equivalent functionalities for the JE and gc args right now...


snakemake cluster script ImportError snakemake.utils

Snakemake version
5.9.1 / 5.5.4
Describe the bug
I described the problem in detail in this SO thread, but did not yet get an answer. Could you please take a look and see if you can help?
Thanks!

Turns out this is not related to snakemake - sorry about that.


fix: add dedicated configuration for S3 endpoint override

Description
Add dedicated configuration for S3 endpoint URI override.
Test Steps

Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue
Fixes #447.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thx a ton as always @agebhar1 LGTM!
@ryancuk do you want to test this build locally before we merge just to verify it works as expected? Build URL: https://github.com/awslabs/eventbridge-kafka-connector/actions/runs/12166351514?pr=451 (see Artifacts section)

will work on it.

@agebhar1 @embano1 i have completed the test. It  works expectedly. thank you SO much for the great help!!

Glad to hear! Now "Go build!" as Werner Vogels would say 😀

Glad to help @ryancuk!


Custom attributes

Do we have any insight on how to customize the highlightning for custom attributes? say I want to add something called "user mention" like: [user][username], is this possible? if not out of the box then maybe guide me to the correct classes to implement it myself

Hi @SolorzanoJose, sorry for my delayed response. This is supported! Check out custom regex. Feel free to reopen with any other questions.


fix(material/datepicker): update active date on focusing a cell

When a a date cell on the calendar recieves focus, set the active date
to that cell. This ensures that the active date matches the date with
browser focus.
Previously, we set the active date on keydown and click, but that was
problematic for screenreaders. That's because many screenreaders trigger
a focus event instead of a keydown event when using screenreader
specific navigation (VoiceOver, Chromevox, NVDA).
Note that this is the second attempt at this. First attempt was #24279,
which was reverted in #24380 due to an internal issue.
Fixes #23483

caretaker note: this PR was previously reverted in #24380 due to causing and internal issue.


Update cluster Defaulter to return error

Issue #, if available:
Description of changes:
So that the Defaulter can be used for context aware operations as well.
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Neighborhoods & counties: separate and de-Boundariesize

This is going to affect a lot of things, but I'll try to keep as little in this branch as possible before moving on to census stuff.
Data

[ ] add neighborhoods and counties layers from our own custom tilesets as separate, distinct layers
[ ] rm all instances and refs to MB Boundaries, including lookup tables

Controls

[ ] below the 2D/3D btn, add a ... btn that triggers a popout containing the "show/zoom my location" and (for now at least) the Counties toggle.
[ ] if needed, include neighborhoods toggle in that menu as well.
[ ] either way, add neighbs toggle to /Explore/Neighborhoods and /Explore/Neighborhoods/Something

Behavior
probably won't get ALL the stuff we discussed in this branch, but counties and neighborhoods will both still:

[ ] zoom to selection on click
[ ] make bg color of selected polygon stand out symbology-wise from the others
[ ] popup: just the name for now. bells whistles later.

Can get all fancy with the routing and highlighting in another branch. Big stuff here is just to start using our own layers.

@rperlin-ela i'm just going to rewrite this as neighborhoods-only, then apply a new issue to counties.

below the 2D/3D btn, add a ... btn that triggers a popout containing the "show/zoom my location"

can we scrap this? i think it's fine in Search Locations, don't really need another map btn. was mostly to de-clutter w/neighbs and counties toggles in there, but those will be in their own panels now.

if needed, include neighborhoods toggle in that menu as well.

also scrap, right?

make bg color of selected polygon stand out symbology-wise from the others

should counties be same color as neighborhoods? we don't have a legend for either one (and we're not going to in this scope since we didn't have it before, except for upcoming census gradient)


Camel native Component support for DNP3 power grid device protocol

Wonder if this ticket is not more suitable here in camel-labs as a iot component?
https://issues.apache.org/jira/browse/CAMEL-6736

I will try to take a look at this. Thanks for letting me know.


Do not fail on early timestamps during GET

Do not fail on early timestamp during HEAD/GET calls

Yes, looks like there was an issue with yesterdays master->develop merge which makes me to redo Bluetooth related changes.
Issue that Kevin is mentioning: right now "Invalid timestamp" error is returned if lastTimestamp is smaller that UtcNow - 28 days, while it should be accepted and available data returned.


Also pass clientid in on_connect notify

It's useful to be able to associate the connect notify with a later play or publish notify for the same connection.

Hi, any feedback?

@arut do you have the time to look into this ? It looks useful to me.


NotImplementedError: PMFSNet has not yet been implemented

When I want to try the demo, meet the "NotImplementedError: PMFSNet has not yet been implemented", please help~~

hello @isbrycee , thank you for your interest, we will release the source code of PMFSNet after the formal submission of our paper in a few days.

hello @isbrycee , thank you for your interest, we will release the source code of PMFSNet after the formal submission of our paper in a few days.

Thanks for your reply~~ I'm looking forward to your  source code of PMFSNet. Thank you for your powerful work!


add functionality to mop_setup.py to check and update status of processed files

So far we don't have any  code to check the status of files to produce in the mopper.db. It would be useful to add some simple db operations to mop_setup.py to:

check which files have failed and why
based on the status message give a chance to the user to update the status to unprocessed, so if the job is re-submitted the files can be processed again, without having to run mop_setup again
often files can have different name than expected even if processing was otherwise successful. Give option i9n such a case to change status to processed

Changing this plan as a separate update_db.py script that can be run separately. It will be automatically copied in the outpath so it's at same level of the mopper.db database to change.
To run:
python update_db.py  [dbname]
exp should be the same exp_id in the filelist table
dynamo is optional by default mopper.db
It will give a summary of all valid-table that have 1 or more files with status different from processed.
Then it will give an option to change their status to

"processed" for example if filename mismatch but files are otherwise ok
"unprocessed" if there's an error and cause has been fixed, so job can be re-submitted
If answering yes to unprocessed option then tool will ask if user wants to also update the mapping for that valid-table.
This gives an option to fix potential issue in the maps/Table.json files beforehand and then get the foilist table updated before pre-running the job.


export 'objectProp' (imported as 'objectProp') was not found in 'vue-ts-types' (module has no exports)

This error has appeared after a deployment was made. Looks like something in the last release today may have caused this to fail. Falling back to 1.6.1 fixed the issue for me.

Same here. 1.6.2. also works.

Thanks for letting me know. I've mistakenly switched from CommonJS to ESM in a minor version. I switched back to CommonJS in #707 and released https://github.com/FloEdelmann/vue-ts-types/releases/tag/v1.7.1 containing this fix. Please let me know if it works for you!


Building dashboard is not incremental

Hello,
I would like to be able to build my dashboard incrementally, by defining what the structure of my dashboard in different .js files.
From my test, and what is stated in the documentation, call to  admin.dashboard(nga.dashboard()) reset the dasboard, which stop the behavior I would expect.
For instance, the menu can be build incrementally by writing code like:
admin.menu(require('./core/menu')(nga, admin)); // create the base level menu admin.menu().addChild(require('./moduleA/menu')(nga, admin)); // Add the moduleA submenu admin.menu().addChild(require('./moduleB/menu')(nga, admin));// Add the moduleB submenu
If I try something similar with the dashboard:
admin.dashboard(nga.dashboard().template(require('./html/dashboard.html')));; // create the base dashboard, only set the template admin.dashboard(nga.dashboard().addCollection(nga.collection(admin.getEntity('entityA')))); // Add a collection of entityA to the base dashboard
Then only the last admin.dashboard call is used.
Coudl this be listed as possible enhancement ? If you point me in the right place where this code is done i dont mind looking into it.
Or is there some other way to do what I want?
Thanks!

Application.menu() and Application.dashboard() behave exactly the same: given  parameter, they become setters, and with no parameter, they are getters.
For your example, just use the same syntax with the dashboard as with the menu:
admin.dashboard(nga.dashboard().template(require('./html/dashboard.html')));; // create the base dashboard, only set the template
admin.dashboard().addCollection(nga.collection(admin.getEntity('entityA')))); // Add a collection of entityA to the base dashboard

I think you got confused between nga.dashboard(), which is a factory function, and admin.dashboard(), which is a smart getter.

Ho dear .. i need an holiday :)
Thanks :)


CAST from string to INTERVAL

To solve the Issue: CAST from string to INTERVAL #324, implemented cases for both interval literal and interval value.
Case1. Interval literal
// src/data/value/literal.rs
(DataType::Interval, Literal::Text(v)) => {
    Ok(Value::Interval(Interval::try_from(v.as_str())?))
}

Case2. Interval value
// src/data/value/mod.rs
(DataType::Interval, value) => value.try_into().map(Value::Interval),

// src/data/value/into.rs
impl TryInto<Interval> for &Value {
    type Error = Error;

    fn try_into(self) -> Result<Interval> {
        Ok(match self {
            Value::Str(value) => Interval::try_from(value.as_str())?,
            _ => return Err(ValueError::ImpossibleCast.into()),
        })
    }
}

Thanks! There's nothing to comment about the behavior.
A few requests here are just code style tweaks.
Looks great, integration tests also look very informative, thanks 👍

Thank you for reviewing and advising nice code style!
I just fixed and re-pushed it!


ERR_SSL_NO_CIPHER_MATCH when use TLS v1.0 cipher in proxy setting

Expected behaviour
I config mountebank with below proxy setting to proxy to real service.
    "proxy": {
        "to": "https://myserver.test.com",
        "mode": "proxyAlways",
        "ciphers": "RC4-SHA"
    }

RC4-SHA should be a valid TLS v1 cipher.
Actual behaviour
I got no cipher match error when proxy to real service:
::ffff:<IP_ADDRESS>:42356 X=> {"library":"SSL routines","function":"SSL_CTX_set_cipher_list","reason":"no cipher match","code":"ERR_SSL_NO_CIPHER_MATCH","message":"error:1410D0B9:SSL routines:SSL_CTX_set_cipher_list:no cipher match","name":"Error","stack":"Error: error:1410D0B9:SSL routines:SSL_CTX_set_cipher_list:no cipher match\n    at Object.createSecureContext (_tls_common.js:177:13)\n    at Object.connect (_tls_wrap.js:1401:48)\n    at Agent.createConnection (https.js:125:22)\n    at Agent.createSocket (_http_agent.js:234:26)\n    at Agent.addRequest (_http_agent.js:193:10)\n    at new ClientRequest (_http_client.js:276:16)\n    at Object.request (https.js:309:10)\n    at Object.<anonymous> (/usr/local/mountebank-v2.1.2-linux-x64/mountebank/node_modules/agent-base/patch-core.js:23:20)\n    at Object.request (/usr/local/mountebank-v2.1.2-linux-x64/mountebank/node_modules/https-proxy-agent/node_modules/agent-base/patch-core.js:25:22)\n    at getProxyRequest (/usr/local/mountebank-v2.1.2-linux-x64/mountebank/src/models/http/httpProxy.js:93:41)"}

Not sure if this is because these ciphers are not available in recent node.js version or not.
Steps to reproduce

use node.js  version 12.13.1 with latest mb version
use "ciphers": "RC4-SHA" in proxy
try create request that would proxy to real service

Software versions used
OS         : MacOS 10.14.6
mountebank : 2.1.2
node.js    : 12.13.1
  (only if installed via npm)
Installation method : npm

Some Resources
RC4 prohibited - https://tools.ietf.org/html/rfc7465 apparently RC4 support is also dropped in nodejs
Try https://www.ssllabs.com/ssltest/index.html to verify if server indeed supports RC4-SHA


Report from httpd image

Work in progress...
Currently container closes immediately after running it with no logs output.

Fixed in #11


Extending Roles beyond ? & @ query on approach

About to try to add some additional roles (following the simpler approach to RB access mentioned in a few posts) So plan to override the already overridden AccessRules class to handle the additional roles  and then update the controller rules for the specified roles. Have you any views on this approach?

Actually need to look at full RBAC as above may not suffice my requirement. Am closing this until I know more

Ok I have installed RBAC & created the required data in the auth tables to run some tests against the controller accessRules. I am not sure I have the rules set up correctly as some are functioning i.e. this is fine
			[
				'allow' => true,
				'actions' => ['protected'],
				'roles' => ['createJob'],
				'scopes' => ['protected'],
			]

will only allow users in who have the protected accessToken & the RBAC auth permissions 'createJob' (note this is a permission not a role) to access the api endpoint.
My controller rules are
return [
			[
				'allow' => false,  //deny anonymous users
				'roles' => ['?'],
			],
			[
				'allow' => true,  // allow users with accessToken & 'viewJob' permission
				'actions' => ['view'],
				'roles' => ['viewJob'],
			],
			[
				'allow' => true, //allow users with protected accessToken & permission
				'actions' => ['protected'],
				'roles' => ['createJob'],
				'scopes' => ['protected'],
			]
	]

However, when I try and access this rule block
			[
				'allow' => true,
				'actions' => ['view'],
				'roles' => ['viewJob'],
			]

I get a 403 with any of the valid permissions I am assigning to the 'role' unless I set the 'role' to general logged in user '@'
Any idea why this is happening on the viewJob as the permission, user & accessToken are valid?

Well it happens that I was missing the action 'index' from the rule which is the end point I was actually calling via view was working already
         [
                'allow' => true,
                'actions' => ['index', 'view'],
                'roles' => ['viewJob'],
            ]

So yep RBAC permissions work fine!


not working with spago@next

Not sure if I do something wrong but the spago run configs still expect dhall. and my Main.purs has errors:

If you manually run Spago build, after removing the output directory, dose that work?

build works, but it just doesn't pick up any of the import libs 🤔

I'm very new to spago/purs, so not sure if these screenshots help

That "P" folder looks suss, is that how it should be or is it a bug in Spago, can't remember that the folder with modules was called "P"


feat(layouts): add basic structure for layouts

Needs to be rebased on dev
Structure is just preliminary, changes are about the layouts, not the structure as much

@Mohammer5 I see you already reviewed parts of this. I'm currently still sketching, I'll let you know when it's ready for review.

@Mohammer5 I see you already reviewed parts of this. I'm currently still sketching, I'll let you know when it's ready for review.

Ok, ready for review now!


RVC Integration

In this PR I will try to integrate an RVC that will transform the result

Closed due to conflicts, easier to create a new PR )


Revert 71718 and fix 71814

This PR:
https://github.com/zephyrproject-rtos/zephyr/pull/71718
introduced a regression breaking CI:
https://github.com/zephyrproject-rtos/zephyr/issues/71814
Let's just revert it, and let that PR be submitted again with a proper fix.
Another quick fix just initializing the variable (to pacify the compiler) was sent in #71813
but there was valid concerns about the code having issues we are just masking. So let's revert the PR instead.
(Note the original PR did not respect the 2 working days review time frame)
Fixes #71814

There was an indentation issue in the original code the revert uncovered triggering checkpatch.
The revert commit now also fixes that indentation issue.

@ycsin the PR you refer to is an exact duplicate of https://github.com/zephyrproject-rtos/zephyr/pull/71813 which has not been merged after 2 developers raised concerns about just pacifying the compiler and masking a likely issue in the code.
Please remove your rejection of this PR.
This failure is breaking main and stopping other merges.
The original PR can be resubmitted and reviewed calmly after.

@ycsin the PR you refer to is an exact duplicate of #71813 which has not been merged after 2 developers raised concerns about just pacifying the compiler and masking a likely issue in the code.

Just to stick up for @ycsin - there is no other issue in the code. This would have fixed it (and yes, it is just pacifying the compiler).
Sorry I did not get to it earlier, but I'm in bringup at the moment and am otherwise trying to deal with a minor injury.
For clarification, the issue was reproducible with
west build -p auto -b qemu_riscv64 -t run tests/posix/common -- -DCONFIG_PTHREAD_LOG_LEVEL_DBG=y

I will resubmit the original PR.


Language  - store String instead of code

Current implementation stores language code, and then added language name to solr ... but is this necessary? do we need the language code in the data at all? couldn't we simply switch what is saved from the dropdown to be the term rather than the id?

There's a rake task in hyrax (rake hyrax:controlled_vocabularies:language) that loads lexvo languages into the database: https://github.com/projecthydra-labs/hyrax/blob/master/lib/tasks/controlled_vocabularies.rake
Could be used instead of our local code for languages. Code already in hyrax for an auto-suggest on the language field.


Risk status not shown anymore

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name: Samsung Galaxy S21
Android version: 12
App version: 2.26.0

Describe the bug
I can't see the status banner anymore (the green/red box that shows me if I had risk contacts in the last days or not.
This morning I saw that the last sync was yesterday afternoon, which is unusual. Therefore I stopped the app and started it again 2 or 3 times to trigger a fresh sync, which usually works. Instead, the status banner just disappeared.
I might be worth mentioning that there currently is a positive PCR result in my app.
Rebooting the device and wiping cache did not work.
Steps to reproduce the issue
I don't know if it's possible to replicate
Can anybody help?

The box will come back
a) 7 days after registering the positive test
b) if you have an increased risk encounter
This is not a bug but the intended behaviour.

I see, thanks for the info!
So it vanished because my risk encounter is now > 10 days old.
Do you know why it is implemented like this? As you can see this is causing confusion.

@error401de

So it vanished because my risk encounter is now > 10 days old.

Yes

Do you know why it is implemented like this?

I can only assume that this was implemented in this way as the risk status is irrelevant if you are positive.

Yeah, I assumed this as well. To me, this is really counterproductive as

it is confusing UX-wise
It hides a very important learning: It is possible to get Covid without captured risk contacts. This should be clear for the majority of the people, but it's always good to get this visualised again.

I will leave this ticket open for a couple of days, maybe there will be an official statement.

@error401de
I suggest you open an enhancement request reg. this in the https://github.com/corona-warn-app/cwa-wishlist repository.
From there, the team will take care of your issue (asses it, etc.).

Good hint, thanks!

@error401de As @Ein-Tim  already mentioned ,that is intended behaviour. if risk card is not high and you have a valid positive test in the app, risk card will be hidden, but the calculation will keep running in the background which means if the latest calculation turned into high in the mean time it will show up again. it does not make sense to show you that you are in low risk while you have a positive test in the app. Also after the test validity period is outdated the risk card will show up again

@mtwalli I do understand it wouldn't make sense to show "low risk" while being positive. Still it's confusing to just remove the box, especially after there is a high risk warning for a couple days. Which also doesn't make any sense to be shown, as it's obviously not just a risk anymore. So you can see: The way it's implemented now is not implied.

@error401de  Feel free to submit a wishlist request?  maybe that can highlighted in a better way

As announced I took care ;)  https://github.com/corona-warn-app/cwa-wishlist/issues/860


The voice assistant feature is not working

Description
The voice assistant feature near the search bar is not working.
Training a machine learning model for voice recognition would help to solve this issue.
I have worked before in this aspect. kindly Assign me this issue.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100
Enabling the functionality of voice assistant.
Kindly assign her.

Pls show with the help of video that what issue are you getting, as I checked that it is working fine @Amishachoudhary22


Remove legacy port argument from WebSocketServer

This shouldn't be an issue since new projects using Fleck shouldn't be using the port argument anyway. Old projects that use the port argument and update Fleck probably also update their .NET version. And if not, including the port in the string shouldn't be too difficult to fix.

Much appreciated!


64 bit compulsion

Please update library as it is 32 bit and google play console requires 64 bit configuration.Library is good but having conflicts now.Please update its as soon as possible

Have you found a way to upgrade this to 64bit ?

I have tried to upgrade it by adding ndk.abi filters as it contains .so
files for 32 bit only.but it didn't work .try to make it fron 64 bit
machine and this problem is mainly because of native code which is used for
polygon crop
On Fri, Aug 9, 2019, 02:17 edwinjeppe1<EMAIL_ADDRESS>wrote:

Have you found a way to upgrade this to 64bit ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jhansireddy/AndroidScannerDemo/issues/102?email_source=notifications&email_token=AM3E4N467LOIGGRXEM7JO5DQDSA5XA5CNFSM4IKGPR2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD343LKA#issuecomment-519681448,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM3E4N56WGPTSSBYL2ANNJTQDSA5XANCNFSM4IKGPR2A
.

I got it working by literally only taking the files from here and pasting it into the folder within my ScanLibrary location where they are located. It added the 64bit version libs for me.

Yes same with me today . I successfully able to use this library again.
Thanks to you also
On Fri, Aug 9, 2019, 12:57 edwinjeppe1<EMAIL_ADDRESS>wrote:

I got it working by literally only taking the files from here
https://github.com/jhansireddy/AndroidScannerDemo/tree/2cd23d3d362d0a6248cf489a79ebc4ba2c425c60/ScanDemoExample/scanlibrary/src/main/libs
and pasting it into the folder within my ScanLibrary location where they
are located. It added the 64bit version libs for me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jhansireddy/AndroidScannerDemo/issues/102?email_source=notifications&email_token=AM3E4N6NST5ZNMQHV5H4CGTQDUL73A5CNFSM4IKGPR2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD353HZI#issuecomment-519812069,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM3E4N2JNKDQYMJVSFA4WP3QDUL73ANCNFSM4IKGPR2A
.

Generated native libraries for all architectures now as part of https://github.com/jhansireddy/AndroidScannerDemo/commit/d29f33bb0aff512f9c90706f3fcf1804f368c75f


100% Coverage for MockSupport_c - Part 1

because crashOnFailure_ is never passed to
MockFailureReporterTestTerminatorForInCOnlyCode.

Ooops. Something went wrong re-opening #580, so this is now its replacement.

#5448F86 - An unrelated test fails on TravicCI because the crash function remains set

Decided to split this PR into smaller ones. This one now only contains one additional test case and additions to another.


bugfix : aync await in process function

Output was not saved to file when serverelss node process finished before writing the file.
This could happen when there was no changes in the stack
Service files not changed. Skipping deployment...

has been fixed in this commit : https://github.com/anttiviljami/serverless-stack-output/commit/bd8bfdec3071ce9790ff5b525f88bff59b12d517


reuse extended event class files too

I added a line to include the extended LAT files with a different name also to the list of files to be checked if they were previously downloaded already.

Codecov Report

Merging #596 (3e75b40) into master (47f691c) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #596      +/-   ##
==========================================
- Coverage   69.64%   69.61%   -0.03%
==========================================
  Files         121      121
  Lines       16058    16059       +1
==========================================
- Hits        11183    11180       -3
- Misses       4875     4879       +4


Pre-optimization for ARIMAX + ARMA models

ARIMAX models in particular need better initialization.

On further investigation, issue may be related to the choice of optimization parameters for Scipy's L-BFGS routine. Now going to try and reduce the tolerance threshold to see the effects.


Format results in a separate call from check

Upgrading from nsp 2.x to 3.x, the ability to format the results in a separate call from the check was lost. For my projects work flow it is important to able to do this. This PR adds that ability back.

Any thoughts on allowing some feature like this in nsp v3?

The Node Security Platform has been acquired by npm, Inc.
On April 10, 2018, the Node Security Platform joined npm, Inc., which operates the npm JavaScript package registry.
Learn more here.
This service remains operational for current users in its current state. No new features or fixes will be implemented.


Add ssh-portal logs capability

With https://github.com/uselagoon/lagoon-ssh-portal/pull/243 merged, it would be nice to get support for this in the charts now

Closing as this has been merged and released already


Added field to light.proto to define if a light is on or off

Signed-off-by: ahcorde<EMAIL_ADDRESS>🎉 New feature
Related with :

https://github.com/ignitionrobotics/ign-gazebo/pull/1343
https://github.com/ignitionrobotics/sdformat/pull/851

Summary
Added field to light.proto to define if a light is on or off
Test it
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[x] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@osrf-jenkins retest this please

Something to keep in mind regarding boolean fields is that they default to false if unset. This means that any message not populating this field will have the light turned off. This may cause issues for users migrating from Fortress to Garden.
I'd suggest flipping the field's name so that the default behavior (light on) happens when the field is unset. i.e.
msgs::Light lightMsg;
assert(!lightMsg.is_light_off());


Populating charts for remote Restic repos

@christianwgd, RepositoryChart() seems a very good place for quickly check (visually) if backups are proceeding smoothly as expected.
For me, this is the very first reason to have a GUI for Restic.
I noticed the log_size management command and was thinking about the possibility to collect data from remote repos with the stats Restic command, which gives the following result:
scanning...
Stats in restore-size mode:
Snapshots processed:   51
   Total File Count:   16253589
         Total Size:   45.928 GiB

Apart from the total size, which you're already collecting for local repos, both files count and snapshots count are interesting, and could be conveniently added to RepoSize model
Or maybe I should rather try rest-server and attach it to Prometheus
What do you think ?

Hi morlandi,
half way through I’ve detected the rest server, but didn’t change at that time. I gave priority to the MVP. Sure, give it a try and let’s see if we can change to that.
As soon I have a little time, I will subclass the json field to „EncryptedJSONField“.
Thanks for your effort,
Christian

Am 17.06.2022 um 19:34 schrieb morlandi @.***>:
@christianwgd https://github.com/christianwgd, RepositoryChart() seems a very good place for quickly check (visually) if backups are proceeding smoothly as expected.
For me, this is the very first reason to have a GUI for Restic.
I noticed the log_size management command and was thinking about the possibility to collect data from remote repos with the stats Restic command, which gives the following result:
scanning...
Stats in restore-size mode:
Snapshots processed:   51
Total File Count:   16253589
Total Size:   45.928 GiB
Apart from the total size, which you're already collecting for local repos, both files count and snapshots count are interesting, and could be conveniently added to RepoSize model
Or maybe I should rather try rest-server and attach it to Prometheus
What do you think ?
—
Reply to this email directly, view it on GitHub https://github.com/christianwgd/django_restic_gui/issues/13, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACG2A36N2NRDFAH3YLYMYH3VPSZMFANCNFSM5ZC6L76A.
You are receiving this because you were mentioned.

Hi Christian,
sooner or later I will likely check the rest server, but for now I'm quite happy with my simple setup: I have a few restic repos on a storage box c/o Hetzner, and collect backups from different remote servers; each server has access to the storage box via SSH with it's own private key and uses restic commands via SFTP.
That's what I expect from a backup strategy: to be simple, secure and reliable.
django_restic_gui is nice and simple; I just miss an overall view with a single summary chart for all repos; let me do some experiments and I'll be back with a new PR on this.
The restic "stats" command is rather slow given my actual configuration .. but who cares ? I'll collect new data in the background, once per day, via cron.

Hi Christian,
first of all I wish you an Happy New Year 🍾
I needed to encrypt the JSONFields in another project, and for this reason I created a fork of the original project:
https://github.com/morlandi/django-encrypted-json-fields
since the maintener at that moment was not available to manage my PR on the original, and the copy available on github was out of date.
Maybe in the future we could evaluate whether or not it is worth using it here too ...

 Nice work, I just included your lib here and seems to work find.


Remove init from TERRAFORM_COMMANDS_NEED_LOCKING

terraform init does not support locking
resolves https://github.com/gruntwork-io/terragrunt/issues/1641

Tests passed! Merging now.

https://github.com/gruntwork-io/terragrunt/releases/tag/v0.28.23


Error while parsing bai2 files.

I am trying to parse 2 different bai2 files but it throwing errors.
File 1:
01,021000018,55287,051025,0600,1,80,080,2/
02,55287,021000018,1,051022,0000,,/
03,0101999999,,015,+00000004060801,,,045,+00000003836014,,,100,+00000001589285/
88,00008,,400,+00000001831945,00002,,040,+00000004059283,,/
16,175,000000000346685,Z,00087829876,Miami/
88,DEPOSIT LOCATION, ITEM COUNT 00004/
16,175,000000000336375,Z,00088137654,Fort Lauderdale/
88,DEPOSIT LOCATION, ITEM COUNT 00008/
16,175,000000000262575,Z,00086095789,Boca Raton/
88,DEPOSIT LOCATION, ITEM COUNT 00007/
16,175,000000000186250,Z,00087793469,Palm Beach/
88,DEPOSIT LOCATION, ITEM COUNT 00001/
16,175,000000000167775,Z,00087792345,Atlanta - Downtown/
88,DEPOSIT LOCATION, ITEM COUNT 00002/
16,175,000000000127200,Z,00088101927,Atlanta - Lenox/
88,DEPOSIT LOCATION, ITEM COUNT 00010/
16,175,000000000083875,Z,00088157722,NYC - 5th Ave/
88,DEPOSIT LOCATION, ITEM COUNT 00002/
16,175,000000000078550,Z,00086079576,NYC - Village/
88,DEPOSIT LOCATION, ITEM COUNT 00002/
16,451,000000001821445,Z,ACD0410288888888,01650088/
88,01650088 ABC Retailers - BNY/
16,555,000000000010500,Z,VVN0410298989898,NYC - Trump/
88,DEPOSITED CHK RETURN     00000000/
49,000000018798558,23/
03,6901355555,,015,+00000013561120,,,045,+00000013561120,,,100,+00000000063500/
88,00001,,400,+00000000070000,00001,,040,+00000013561120,,/
16,175,000000000063500,Z,00086081234,LA/
88,DEPOSIT LOCATION, ITEM COUNT 00001/
16,451,000000000070000,Z,ACD0410288553311,01650090/
88,01650090 ABC Retailers - BNY/
49,000000040950360,7/
98,000000059748918,2,32/
99,000000059748918,1,34/

Error is:
File "/home/atul/bai2/bai2env/lib/python3.6/site-packages/bai2/parsers.py", line 143, in _parse_field_from_config
    field_value = parser(raw_value) if raw_value else None
ValueError: invalid literal for int() with base 10: '2/

File 2:
01,122099999,123456789,040621,0200,1,65,,2/
02,031001234,122099999,1,040620,2359,,2/
03,0123456789,,010,+4350000,,,040,2830000,,/
88,072,1020000,,,074,500000,,/
16,,115,450000,S,100000,200000,150000,,,/
49,9150000,4/
03,9876543210,,010,-500000,,,100,1000000,,,400,2000000,,,190/
88,500000,,,110,1000000,,,072,500000,,,074,500000,,,040/
88,-1500000,,/
16,115,500000,S,,200000,300000,,,LOCK BOX NO.68751
49,4000000,5/
98,13150000,2,11/
02,053003456,122099999,1,040620,2359,,2/
03,4589761203,,010,10000000,,,040,5000000,,,074,4000000,,/
88,400,50000000,,,100,60000000,,,110,20000000,,,072,1000000,,/
16,218,20000000,V,040622,,SP4738,YRC065321/
88,PROCEEDS OF LETTER OF CREDIT FROM THE ARAMCO OIL CO
16,195,10000000,1,,,/
49,180000000,6/
98,180000000,1,8/
02,071207890,122099999,1,040620,2359,,2/
03,0975312468,,010,500000,,,190,70000000,4,0,110/
88,70000000,15,D,3,0,20000000,1,30000000,3,20000000/
49,140500000,3/
98,140500000,1,5/
02,071207890,122099999,3,040620,2359,,2/
03,7890654321,,010,800000,,,040,6000000,,,110,5000000/
88,4,/
49,11800000,3/
98,11800000,1,5/
99,345450000,4,31/

Error:
File "/home/atul/bai2/bai2env/lib/python3.6/site-packages/bai2/parsers.py", line 143, in _parse_field_from_config
    field_value = parser(raw_value) if raw_value else None
  File "/usr/lib/python3.6/enum.py", line 293, in __call__
    return cls.__new__(cls, value)
  File "/usr/lib/python3.6/enum.py", line 535, in __new__
    return cls._missing_(value)
  File "/usr/lib/python3.6/enum.py", line 548, in _missing_
    raise ValueError("%r is not a valid %s" % (value, cls.__name__))
ValueError: '450000' is not a valid FundsType

The Second file is corrupted
check the fifth line
16**,,**115,450000,S,100000,200000,150000,,,/


Enable 16550 UART driver on x86_64

This platform has basic console output wired in via printk, but the existing UART driver didn't work out of the box when I tried it.
There's no reason the existing driver shouldn't work.  Make it work.
Note that there are tests that have to be whitelisted because CONFIG_LOG requires a UART and won't work with printk's stream (a different issue, see #11975)

ancient, long since addressed


Changed "on GitHub" to "at GitHub"

Docs Build status updates of commit 2f30e43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2022/03/devtools-100.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Email content is lost in every message

Ubuntu 14.04 64bits, HHVM 3.7.3, Postfix 2.11.0, Nginx 1.4.6
MySQL Ver 14.14 Distrib 5.5.43, for debian-linux-gnu (x86_64) using readline 6.3

First, congratulations for all RainLoop creators and community.
In my Rainloop install with <IP_ADDRESS>3 and <IP_ADDRESS>3 (latest), the body content of e-mail becomes empty.
If I send an email, it is sent but received empty. Or if I save in draft, the saved message also becomes empty. The message content is not saved, it is lost.

First I created a new email, insert a content to it and saved in draft or send it.

Both the received or draft message lost its content.

Opening the original email data, no body content is found:

Mime-Version: 1.0
Date: Fri, 21 Aug 2015 16:31:18 +0000
Content-Type: multipart/alternative;
 boundary="--=_RainLoop_248_549824324.1440174678"
Message-ID<EMAIL_ADDRESS>X-Mailer: RainLoop/<IP_ADDRESS>3
From<EMAIL_ADDRESS>Subject: Just a subject
To: "Paulo Coghi"<EMAIL_ADDRESS>

----=_RainLoop_248_549824324.1440174678
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

----=_RainLoop_248_549824324.1440174678
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

----=_RainLoop_248_549824324.1440174678--

HHVM 3.7.3

Try to use the standard PHP, HHVM does not work well in some places.

The standard PHP works well. Closing.


Describe what is rule

Closes #156
@simoess What's missing? Maybe I am not the best person to describe it.
Preview https://arche.readthedocs.io/en/rules_docs/

Codecov Report

Merging #166 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #166   +/-   ##
=======================================
  Coverage   81.35%   81.35%
=======================================
  Files          22       22
  Lines        1620     1620
  Branches      290      290
=======================================
  Hits         1318     1318
  Misses        249      249
  Partials       53       53

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 ced8822...0c7ee10. Read the comment docs.

Codecov Report

Merging #166 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #166   +/-   ##
=======================================
  Coverage   81.35%   81.35%
=======================================
  Files          22       22
  Lines        1620     1620
  Branches      290      290
=======================================
  Hits         1318     1318
  Misses        249      249
  Partials       53       53

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 ced8822...0c7ee10. Read the comment docs.

@simoess It looks like nbsphinx doesn't render lists properly, so I added some newlines. Thanks for pointing out.


Singletonized Measure and Reset instructions

Summary

Singletonized instructions Measure and Reset
Resolved circular import error during test run
Drafted release notes highlighting the features and upgrades

Details and comments
This PR partially addresses the issue #10953. It involves singletonizing Measure and Reset instructions. However, Barrier is not singletonized after discussing with the maintainer, as the base singleton code needed some additional machinery to handle instructions like Barrier.
Running tox test suite resulted in circular import error, which also has been solved with the guidance of the maintainer.
Finally added the release notes, elaborating on the features and upgrades brought upon by this pull request.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.004%) to 86.903%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
3
91.41%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
74019

Relevant Lines:
85174

💛  - Coveralls

Thank you so much for the review!
Sure sir, will add a commit soon enough covering all these changes.

(Don't worry about pressing "update branch", by the way - our merging procedures involve using a merge train that will automatically handle bringing a PR up to date.)

Thank you very much!
I have made the following corrections in the previous commit

Thank you so much for the approval! Excited to continue learning and contributing.
Looking forward to more opportunities!


Incompatibility error on Linux 64-bit

After trying to install a package that depends upon chokidar, I got the following error:
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>
I'm running Linux 64-bit.

I think that's probably deliberate. fsevents is a Mac-only module, but because it's optional it won't fail the build when you install chokidar on a different platform.

@rhoggSugarcrm is correct, everything is fine.

Is there any way that we could somehow get rid of that warning when it is expected (e.g. chokidar should be able to dynamically build fsevents only when the user is on MacOS)?
I just got bit by this too (and am trying to clean up my Yarn v2 yarn install output):
➤ YN0000: ┌ Resolution step
➤ YN0000: └ Completed in 0.52s
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed in 0.99s
➤ YN0000: ┌ Link step
➤ YN0062: │ fsevents@patch:fsevents@npm%3A2.1.3#builtin<compat/fsevents>::version=2.1.3&hash=495457 The platform linux is incompatible with this module, building skipped.
➤ YN0062: │ fsevents@patch:fsevents@npm%3A1.2.13#builtin<compat/fsevents>::version=1.2.13&hash=495457 The platform linux is incompatible with this module, building skipped.
➤ YN0000: └ Completed in 1.97s
➤ YN0000: Done with warnings in 3.71s


BrowserStack issues?

Hi @threepointone,
the CI is currently broken and we don't know why. It looks like the project can be build correctly and then just aborts. @otbe downloaded the corresponding Travis image and run the test suite with our own BrowserStack credentials and everything seemed to work fine (besides another inconsistency of css text output in IE). Any idea what's wrong? Can you look into our BrowserStack account, if you can see any error logs or something like that?
Thank you!

I'm thinking of moving to saucelabs, in general my experience with browserstack hasn't been great. Any objections?

No. Only used SauceLabs for OSS before :)

I've whacked out phantomjs, looks like it was causing problems. I'll close this for now, browserstack seems to be working fine again. will revisit the saucelabs decision later,


Volume control is not working on my Dell u2720q display (brightness control does)

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Read through the Monitor-Troubleshooting Wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the issue
Volume control is not working on my Dell u2720q display. Brightness control works just fine.
I use a 2018 macbook pro (intel macbookpro15,2) running Monterey 12.6 and MonitorControl 4.1.0 build 7034.
The monitor is connected via a USB-C cable (DP alt mode).
DDC/CI is turned ON in the settings of the monitor.
The monitor has no built-in speakers and discrete speakers are connected through its audio-out port (3.5mm jack).
When I press the volume keys on the keyboard, MonitorControl displays the macOS OSD "scale" but the volume of the speakers is not affected. I can only change the volume by using the physical knob on the speakers.
Again, brightness control works as expected.
Expected behavior
I wish that volume control keys on the keyboard would work as expected (turn the speakers volume up and down).
Anything else?
No response
Environment Information (please complete the following information)
- macOS version: 12.6
- Mac model: macbookpro15,2 (13'' touch 2018)
- MonitorControl version: 4.1.0 build 7034
- Monitor(s): Dell u2720q
- Monitor Cable(s)/Connection(s): macbook pro -> USB-C (DP alt mode) -> Dell u2720q
- Apple Silicon/M1 (yes or no): no

I tried switching the USB-C to USB-C cable for something else:

macbook pro -> USB-C to DP adapter -> DP cable -> Dell u2720q
However, I get exactly the same outcome.

I'm having the exact same issue with my Dell U4021QW, via USB-C with external speakers. Happy to provide additional context or testing if that's helpful.
My system:
- macOS version: 12.4
- Mac model: 16" 2019 Intel
- MonitorControl version: 4.1.0 build 7034
- Monitor(s): Dell U4021QW
- Monitor Cable(s)/Connection(s): MBP USB-C -> Dell USB-C
- Apple Silicon/M1 (yes or no): No

Same issue here with an HP monitor. Speakers connected to the monitor via 3.5mm jack. Brightness control works, but sound does not. Only way to actually adjust the volume is directly on the speakers.
Setup:
- macOS version: 12.4
- Mac model: 16" 2021 M1
- MonitorControl version: 4.1.0 build 7034
- Monitor(s): HP U28 4K HDR
- Monitor Cable(s)/Connection(s): MBP USB-C -> HP USB-C
- Apple Silicon/M1 (yes or no): Yes

Hi everyone,
Did you select the right volume output in the macOS native "Sound" menu (or in the "System Settings" app) ?
You can also disable ddc volume control directly in the app which should let macOS handle volume completly

I have same issue with a DELL 2419H and MacBook 16inch M1. I can change the external display brightness but the volume control doesn't work.
I'm using USB-C (Mac) to HDMI(Dell).

I can also confirm same behaviour for Dell U2720Q linked via USB-C cable with Macbook 14 M1 Pro. Sound system is connected via 3.5mm jack into monitor. With MonitorControl 4.1.0 I can easily adjust level of brightness however not sound level at all. Does not change behavior if I turn off or on DDC/CI in monitor settings. In MacOS of course default sound source is set to monitor (see attached image).

Confirming same issue with DELL S2722QC.

Actually, disregard.  Issue appears to be resolved after I Enabled Accessibility in Security & Privacy control panel and rebooted my machine.  Now volume control appears too be working.
I did have to disable to "Use hardware DDC control" as it was causing the internal monitor "are you sure" message to pop up on screen repeatedly.

I have checked and did not help for me.
On Dell U2720Q I have no pop up messages.
I have checked Accessibility and it is on.
I have tried to disable DDC hardware control and reboot computer and still same "not function" situation.

Adding one more detail.
Keyboard mute doesn't work.  I can reduce the volume to zero using the volume down... or by adjusting volume in the Monitor Control menu.  But some sound will still play through the speakers.

@lalato You can try to enable "true" mute using "Advanced settings" for your monitor

@the0neyouseek If you mean this setting, then it is already set in MonitorControl advanced settings.

I'm having the same issue with my Dell U2723QE. I the display connected to my M1 macbook pro, with headphones playing through the 3.5mm jack. When I press the volume keys, the animations indicating a volume change display correctly, but no such change occurs on the audio.
- macOS version: 12.6
- Mac model: 16" 2021 Apple
- MonitorControl version: 4.1.0 build 7034
- Monitor(s): Dell U2723QE
- Monitor Cable(s)/Connection(s): MBP USB-C -> Dell USB-C
- Apple Silicon/M1 (yes or no): Yes

It would be awesome if this could get fixed. The brightness control works great though, thanks!!

Dell U2520D same issue on Monterey Intel cpu

Same here with Lenovo T27p-10 and M1 Macbook Pro and Air.

Same issue and behavior with an Acer XR382CQK connected via either HDMI or USB-C

I'm having the same issue where my MacBook pro with M1 chip connected USB-C port to adaptor to HDMI to Dell monitor (new as of this black friday) displays the animation of volume going up and down but the volume stays the same volume in reality. The volume is coming out of the Dell monitor's built-in speakers just fine, just no volume change via the buttons.

I had the same issue with an Acer monitor using a HDMI- or DisplayPort- to USC-C dongle on an M1 MacBook Pro
Using a direct USB-C connection from monitor to MacBook worked fine.

I had the same issue with an Acer monitor using a HDMI- or DisplayPort- to USC-C dongle on an M1 MacBook Pro
Using a direct USB-C connection from monitor to MacBook worked fine.

@bwiernik when you say "I had the same issue" are you referring to the choppy sound issue (#1238) that has been resolved, or this issue here (#1187) of volume control not actually adjusting the volume? Thanks for the clarification!

The volume control not working. The volume control UI shows up and changes, but the volume doesn't actually adjust.

can confirm same volume control not actually changing volume or mute on Gigabyte M27Q connected via USB-C on M1

same issues with Lenovo T34W-20 connected via usb-c on MacBookPro M1Max

Just a reminder - monitor support for volume control is all over the place, some displays do support it, some don't. Even big name manufacturers and expensive displays have issues with this (my super expensive mini-LED Lenovo Creator Extreme does not support it for example).


fix(DIST-426): Use latest Jarvis version

[ ] Explain the motivation for making this change.
[ ] Provide a solid test plan for your changes.
[ ]  Link to the issue - if any issue number here

[BOT] Preview available: https://embed.typeform.com/edba464617033c4997b299527c3ee00d9898c1ab/embed.js

:tada: This PR is included in version 0.25.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Switch DayNight theme (Light <---> Dark Theme)

Second part of #83
Adds an option to switch the Theme
Any suggestions for better colors of the theme are welcomed.
Preview:

RecyclerView

People SideBar

NOTE: I have not fixed the current bugs of the branch #81 which is used to build this branch!
And this have to be merged after #81 gets merged in!

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

@niftynei updated this branch..

this is beautiful. lgtm. thanks @kunall17 .


Add options to scope the secret

Can we add an option to scope the secret to either a particular namespace or have it cluster scoped.

Hi @diptripa
I'm not sure what you mean. Can you please explain your request more. E.g. add some examples of the expected result?

@diptripa
Thank you for the feature request.
I'm currently don't have the capacity to work on this.
If you want to check it yourself, feel free to make a PR.

I would give it a try. But let's talk about implementation details first so I don't implement stuff in the wrong way.
My suggestion to implement the cluster-scope: when there is no "namespace" set in the Secret field it automatically seals for the whole cluster. What do you think of that?

I would give it a try. But let's talk about implementation details first so I don't implement stuff in the wrong way.
My suggestion to implement the cluster-scope: when there is no "namespace" set in the Secret field it automatically seals for the whole cluster. What do you think of that?

Yes I think this would be a good option.
I was also thinking of removing the encode/decode option on the ui, it has the potential to confuse newbies.

Yes I think this would be a good option.
What do you think about the suggestion @bakito?

I was also thinking of removing the encode/decode option on the ui, it has the potential to confuse newbies.
What would you suggest to use instead? I think we definitely need a button for encoding / decoding. You only want to seperate these two?

@christianknell adding an option for xluster scoped, seems ok for me. E.g. via a checkbox in the UI.
I agree in keeping the buttons unless there would be a better option. But this schould be handled separately

@bakito: alright. I will try to find time between christmas and new year.

@bakito @christianknell you can now manage the scope by having the annotation on the secret
  annotations:
    sealedsecrets.bitnami.com/cluster-wide: "true"


Fix appeveyor build

Appveyor build started breaking recently and we need to figure out the cause and fix it

Found a discussion on AppVeyor that might help. Attaching it here

Found a discussion on AppVeyor that might help. Attaching it here

Thanks @Sarthaknegigit for the ref!


Typo : missing quote in toString method's result

No bug here, jus a typo : a missing quote for the id value in the toString method.

Thanks for catching that.  Say no to typos!


custom Error handling PHP 7+ not working

I build my own ErrorException error handling to control my application. the problem is it works perfect with PHP 5.6 but not working with PHP 7.1.3!!
I get 500 status but no messages to be send.
Should i ignore Slim errorHandler ,phpErrorHandler functions and build my own Error handler according to what i understand here (may be i understand wrong)
PHP Error Handler
or there is mistake in my code?
BTW this is my Code
set_error_handler(function ($errNum, $message, $file, $line) {
    throw new ErrorException($message, $errNum, 0, $file, $line);
});

$c = $app->getContainer();

$c['errorHandler']      = function ($c) {
    require "../mc/models/errorHundling/Error.php";
    return new Error();
};
$c['phpErrorHandler']   = function ($c) {
    require "../mc/models/errorHundling/Error.php";
    return new Error();
};

This is what i found during searching but still i miss something!!
$app->get('/exception', function ($req, $res, $args) {
    // errorHandler will trap this Exception
    // throw new ErrorException("An error happened here");
    require "../mc/models/errorHundling/Error.php";
    return new Error();
});

$app->get('/php7', function ($req, $res, $args) {
    $x = function (int $x) {
        return $x;
    };

    // phpErrorHandler wil trap this Error
    $x('test');
});

$app->get('/warning', function ($req, $res, $args) {
    $x = UNDEFINED_CONSTANT;
});

Error Class code:
<?php

class Error {
   	private $error = null;

   	public function __invoke($request, $response, $e) {  // pre App
        $this->provideError($e);
        return $response
            ->withStatus($this->error->status)
            ->withHeader('Access-Control-Allow-Origin', '*')
            ->withHeader('Content-Type', 'application/json')
            ->write(json_encode($this->error));
   	}

   	private function provideError($e=null){
   		$msg 						= $e->getMessage();
   		$this->error 				= new stdClass();
   		switch ($msg){
	   		case 'Expired token' || 'Invalid token! Please contact admins':
	   			$this->errorStatus(511, $e);
	   			break;
	   		default:
	   			$this->errorStatus(500, $e);
	   	}
   	}
   	private function errorStatus($status=500, $e=null){
   		return $this->error 		= (object) array( 'status' 	=> $status,
                  	   								   'title'	   => 'Error',
                  											'message'   => $e->getMessage(),
                  											'detail'	   => (object) array('file'		=>$e->getFile(),	// This is for advance Error debuging
                  							 				'code'	  	=> $e->getCode(),
                  						 					'line'	   => $e->getLine(),
                  						 					'trace'    	=> $e->getTrace()) );
   	}
}

Can you create a one-file Slim application that shows the problem?

Hi Akrabat,

Please check this link it is a single file application to demonstrate error handling
Repository

I solve the issue by wrapping my app with try...catch and use my error class called inside the catch using throwable with php 7 and exception with php less than 7 (5.6)
it works perfectly with all but i was wondering what is wrong if i use the built in error handler of Slim framework!!
Thanks in advance.
Thanks in advance.

Hi @akrabat

Did you test the single page project ?? waiting your answer. Also please advice me if i should go with try and catch approach or there is something wrong with my code that prevent PHP7 from using slim built in error handler.
Thanks

I'm on holiday and my laptop has failed, so I will have to wait until I get home next week.

Awww, really sorry sir
Don't worry i will be patient.
Wishing you all the best from my heart.
Sorry again.
Have a nice holiday.

@AhmedBHameed seems like this one has stalled, were you able to figure it out?

@geggleto sorry i did not get you point do you mean about solution?

+1

Hi @AhmedBHameed.
The problem is pretty simple, since PHP 7 classname Error is predefined and used internally.  You could change the Error name to ErrorHandler.  Checkout the documentation for this: http://php.net/manual/en/class.error.php

I believe this has been resolved.


  I've cleaned them and greased the worm and sliders, but they are still stiff (probably junky grease hidden inside of the bearings).  If I move them manually--hard to do, messy, and possibly risky--I can get them to start seeking properly, but it's quite time consuming.
Would it be possible to add a head stepper exercise mode where it just steps the head to --ecyl and back to track 0?  Optionally a 'count' option for the # of times to do it (0 being forever).  Or I could just wrap it in a script if that doesn't fit into your vision of gw.py.
Thank you.

I second here - some tools for us refurbishing drives would be great indeed !

Yes can do. I could also do a calibration mode? Read sectors off a known good Dos disk at certain tracks repeatedly and dynamically update #good sectors on screen in a loop?

I wouldn't complain!  The more the better.  If you are going that far how
about seeking around and making sure the sectors are for the correct track?
On Sat, May 16, 2020, 12:57 PM Keir Fraser<EMAIL_ADDRESS>wrote:

Yes can do. I could also do a calibration mode? Read sectors off a known
good Dos disk at certain tracks repeatedly and dynamically update #good
sectors on screen in a loop?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keirf/Greaseweazle/issues/32#issuecomment-629675725,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPEX7BYDQB2IN2K5QFEUS3RR3AX3ANCNFSM4NC6ZQIA
.

 See how simple greaseweazle/tools/erase.py is for example, and a simple head step could be a few lines of code after importing greaseweazle.usb and maybe greaseweazle.tools.util

Yes I would calibrate on the track you specify on the command line, reading both heads and reporting nr sectors are how many are track too high, low, or just right. In a loop til you ctrl-c. It's basically what I implemented in my amiga test kit.


Add missing CiliumNetworkPolicies

for kyverno-admission-controller, kyverno-plugin, kyverno-policy-reporter, kyverno-background-controller and kyverno-reports-controller
Without them, the Kyverno doesn't work well in the locked environments. Example: https://tekton.ci.giantswarm.io/#/namespaces/mc-bootstrap/pipelineruns/pr-mc-bootstrap-729-generate-mc4zfmt?pipelineTask=generate-mc&step=generate-mc
I tested this here: https://github.com/giantswarm/mc-bootstrap/pull/729 I was creating them in the mc-bootstrap to workaround this issue, but they logically belong to the app itself.
Checklist

[x] Update changelog in CHANGELOG.md.
[x] Make sure values.yaml and values.schema.json are valid.

Will hold this to check if need or not ingress

I think that's it 🚀


Some questions about LibriPhrase evaluation.

Hi !
I'd like to construct the test datasets mentioned in your paper.  In the paper, I found "For the evaluation, we generate 4,391 / 2,605 / 467 / 56 episodes of each length consisting of positive and negative audio phrases  for enrolled keywords (anchor) using the train-others-500 subset. ".
How can I generate the same phrases as you. Just only running "bash run.sh" is ok?  I have not found the clear process of generating LP_{E} and LP_{H} in the repo readme.
Thanks for your help!

Hi. @YuXI-Chn !
I just updated the repository and released the official evaluation or test data used in our paper.
Please refer to the README of this repository.
If you have any problem, please let us know!

Thanks!  So helpful!  I am going to download the new 4 four .csv fiies.
Let me make a list of the things I have to do and point out any mistakes if have.
First, just execute bash run.sh, use the mode='diffspk_all', which is uncommented in the shell script.
Second, download the official evaluation data csv file and  extract the corresponding wavs from the step1.
or
As I found there is a LibriPhrase_evalset.zip in google drive, so maybe I don't need to execute 'bash run.sh' If only need the evaluation dataset but not training dataset?  just downloading the zip file is enough? (I am downloading it,  not finish yet, so I am not sure which type files are in the zip file.)
Thanks so much for help!

I have finished downloading. the zip contains the split wavs! So helpful!

Hello @gusrud1103 !
I found there is no more introduction about how to construct the "4391/ 2,605 / 467 / 56" episodes. What should I do to construct episodes?  For example, for length = 4, I need to construct 56 episodes.  I randomly select 56 words in the libriphrase_diffspk_all_4word.csv file to construct the 56 episodes? Or I miss some important information to do this, but you have pointed somewhere.
Look forward to reply, thx!

Sorry for the late response! @YuXI-Chn
We used all of the LibriPhrase evaluation set we uploaded.
As you can see in the CSV files, each episode contains 27 rows(9 rows positive, 9 rows easy negative, 9 rows hard negative), and 3 anchor samples from the same speaker are in each episode for a fair comparison with QbyE methods which need 3 anchor samples for enrollment.
Each anchor sample is paired with 3 positive and 3 (easy or hard) negative samples.

@gusrud1103 , Can you please share scripts to generate for train-100, train-360 hours subsets?


SCM Changes-List: Introduce an option which controls whether the diff-viewer will be shown with a single or double-click on each file that changed

Essentially what the subject says.

Depending on what you are looking for, please see if the following would help.
 // Controls whether the diff editor should be opened when clicking a change. Otherwise the regular editor will be opened.
"git.openDiffOnClick": true,

  // Controls how to open items in trees and lists using the mouse (if supported). For parents with children in trees, this setting will control if a single click expands the parent or a double click. Note that some trees and lists might choose to ignore this setting if it is not applicable.
"workbench.list.openMode": "singleClick",

Granted the latter setting is for all tree views.
Does this address your request?

Depending on what you are looking for, please see if the following would help.
 // Controls whether the diff editor should be opened when clicking a change. Otherwise the regular editor will be opened.
"git.openDiffOnClick": true,

  // Controls how to open items in trees and lists using the mouse (if supported). For parents with children in trees, this setting will control if a single click expands the parent or a double click. Note that some trees and lists might choose to ignore this setting if it is not applicable.
"workbench.list.openMode": "singleClick",

Granted the latter setting is for all tree views.
Does this address your request?


Appdaemon not install on Home Assistant

Appdaemon not install

(Why the issue was filed)

Appdaemon to start

(What you expected to happen)

Installation stop with error

ERROR: unable to select packages: chromium, chromium-chromedriver (no such package):
Failed installing package chromium-chromedriver

Install Appdaemon  through Add-on on Home Assistant

(How can someone else make/see it happen)

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

Same here
ERROR: unable to select packages:
chromium-chromedriver (no such package):
required by: world[chromium-chromedriver]
[13:13:46] FATAL: Failed installing package chromium-chromedriver

Configuration
system_packages:

chromium-chromedriver
chromium
python_packages:
selenium
pillow
init_commands:

echo "http://dl-cdn.alpinelinux.org/alpine/edge/community" >
/etc/apk/repositories

echo "http://dl-cdn.alpinelinux.org/alpine/edge/main" >>
/etc/apk/repositories
apk update
log_level: info

This isn't an addon issue, it's  the fact that the aarch64 chromium-chromedriver appears to be missing on the edge repo.
Nothing that can be done here.

Sorry but Appsdaemon 14.0 is still the best release you've done.  15.0 and 16.0 both do not work on my home assistant "It says it moves the file from the appsdaemon folder to another location that does not exist on my HA server" glad a backup everything daily.