Amplify Auth Subscriptions Clarification

Reading through the documentation found at this link: https://docs.amplify.aws/lib/graphqlapi/subscribe-data/q/platform/ios
I call a subscription like so: subscription = Amplify.API.subscribe(request: .subscription(of: Todo.self, type: .onCreate))
However, having the @auth directive results in a "Not Authorized Exception". So I switched over to using this other function
subscription = Amplify.API.subscribe(request: .subscription(to: Todo.self, subscriptionType: .onCreate, ownerId: "ownerID")) where I can provide the ownerID. But then I get an error saying "Validation error of type FieldUndefined" for fields:  _version, _lastChangedAt, _deleted.
My schema is as so:
type Todo @model
@auth(rules: [
{
allow: owner,
operations: [create, update, delete, read]
}
])
{
id: ID!
content: String
}
Is the second function the correct one to use? If so how do I add the missing fields that are being asked for?
Environment:
Amplify Framework Version: 1.1.1
Dependency Manager: Cocoapods
Swift Version : 5.0
Device Information:
Device: iPhone 11 Pro Simulator (13.3)

Hey @obrian33, the message your shared:

But then I get an error saying "Validation error of type FieldUndefined" for fields: _version, _lastChangedAt, _deleted.

Indicates me you have a DataStore-enabled backend, but you're trying to interact directly with the backend. In that case you will need to manage the metadata fields AppSync use for distributed data sync. Take a look at this and let me know if it's helpful: https://docs.amplify.aws/lib/datastore/how-it-works/q/platform/ios#writing-data-from-the-appsync-console

@drochetti That was helpful but I don't think I'm using the DataStore because the functions I'm using are in Amplify.API vs. Amplify.DataStore. I might have a misunderstanding on my approach but what I was trying to accomplish was that a user has a collection of Todos and if a new Todo is added for that user by someone else the subscription would push it to the user and they would see it. In order to accomplish this would I need the DataStore?

@drochetti That was helpful but I don't think I'm using the DataStore because the functions I'm using are in Amplify.API vs. Amplify.DataStore. I might have a misunderstanding on my approach but what I was trying to accomplish was that a user has a collection of Todos and if a new Todo is added for that user by someone else the subscription would push it to the user and they would see it. In order to accomplish this would I need the DataStore?

The fact that your error message contains _version, _lastChangedAt and _deleted tells me your backend has conflict resolution enabled, which is something DataStore uses to control the distributed data. If that was intentional, then you need to refer to that link and make sure you include those fields in your GraphQL operations, otherwise you can use the CLI to update you backend: amplify update api and then Disable DataStore for entire API.

@drochetti Gotcha! I do remember enabling that. I tried to disable it but it didn't work so then I tried your first approach and added what is needed for the DataStore (_version, _lastChangedAt, _deleted). After a few missteps on my part, I finally managed to get it working! Thank you so so much :)


Caresets Medication scheme Business document RIZIV/INAMU

20241001 Caresets MedicationSchema V1.1 FR.docx

Remarks for 1.4.1 Version:

we suggest to use the identifier.period and remove versionIdentifier

Remarks for 1.4.1 Recorder:

we need to change this to really mean "recorder". Since MedicationStatement does not have such an element, we will create a custom extension (after double-checking with the international community).

Remarks for 1.4.1 Product/Substance:

we don't prescribe terminologies in the profile just yet (we may when they are provided by terminologist). We do however recommend changing the document to add VMP Group to the list of target terminologies.
suggest add to the document a consideration that sometimes the product is not coded (e.g. magistral) or not in the existing valuesets (e.g. foreign products) : ", sauf pour les cas où l’information n’es pas structurée ou codée – p.ex. préparations magistrales."
we will also rename medicinalproduct.identifier to .code (CodeableConcept, to allow also free text)

Remarks for 1.4.1 ReasonReference:

we agree to keep it, but will add rationale (which is: 1. this is so common that we do want to make it central when it exists and 2. if we use this only through CarePlan, then careplan becomes mandatory for adding an indication. which is not ideal).

Remarks for 1.4.1 DosageInstruction:

we agree that will provide some examples. we will investigate the discrepancies between SAMv2 and FHIR. Corilus to propose a new issue for creating dosage examples.

Remarks for 1.4.1 DosageOverrideReason:

if the prescription is for e.g. 2-3 times the recommended dosage, there should be a warning
what if there is no dosage stated in the prescription - this can happen e.g. for creams, ointments, etc.
what if the dosage is not structured?
We agree that the DosageOverrideReason is optional and it can be defined by the physician in different circumstances. It may be driven by SAMv2 but that may not always be the case - in case of unstructured dosages, or some medications. For this reason, we will not add any logic in the profile for determining when the DosageOverrideReason must be filled in. We can provide this rationale in the guidance.

Remarks for 1.4.1 Note:
RIZIV will change the document to read "Texte libre avec informations additionnelles sur la ligne de médication". instead of the "prestataire de soins"
Remarks for PSS Number and PSS Reason:

The PSS Number+Reason are separate from the dosage  Override. We will clarify the separation in the data elements:
PSS Number indicates what was the PSS ID that triggered the creation of the MedicationLine, if any. PSS Reason contains the prescriber's justification for deviating from that PSS ID, if applicable. This is different from the DosageOverrideReason which is only about the dosage. In ohter words, PSS Number and Reason is about whether a medication has been proposed by the PSS system and we need to keep track of that advice- PSS Reason is not only about the dosage; DosageOverride(Reason) is independent of PSS or not, and whether the dosage differs from the SAMv2 recommendations.
All of this will be commented out in the first version of the MedicationLine.

Remarks on Medication/MedicationType - how to use VOS:

VOS is the granularity of the medication. So it will be in the code systems of the Medication element. It is not a medicationType

Visiflag to be removed.


Mods are not being installed on initial boot

Right now if someone configures his server with mods and run the server using dst-server start the mods are never installed. (Use --update=mods to workaround this problem).
And if they run dst-server update --mods before they run a dst-server start the mods are not updated/installed because the file dedicated_server_mods_setup.lua is not generated.

Could we check if MODS is set and if $DST_HOME/mods/dedicated_server_mods_setup.lua is non-existent so we just add --update=mods? Would that work? Does that make sense?

yes, that would work. But this just show why #67 is a better option. We must create images with the mods in it.


Fix link to part 1 in README.

The link for part 1 in the README is currently pointing to part 2 :) This PR fixes the link.

good catch, thanks!


[AIRFLOW-1436, 1475] EmrJobFlowSensor consideres Cancelled step as Successful

Change EmrBaseSensor NON_TERMINAL_STATES and FAILED_STATE to list
EmrJobFlowSensor consideres Cancelled step as Successful
JIRA

[ ] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1436
https://issues.apache.org/jira/browse/AIRFLOW-1475

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

+1 we are having this very same issue and is causing headaches in production.
To add my Use Case

We trigger the Cluster creation
It failed due to a Validation Error
The Cluster start shutting down and the Steps are created
Steps are marked as Cancelled
Cluster shuts down.

Please add a test and provide a commit message following the commit guidelines. Thanks!

@bolkedebruin Okay then I will add another new pull request.


Parametrized destinations

Description
Draft implementation of parametrized destinations.
I added a base DestinationFactory with same interface as the destination modules just to hold the config arguments.
Usage is e.g.
import dlt
from dlt.destinations import snowflake, filesystem

pipeline = dlt.pipeline(
    "callable_dest_staging",
    destination=snowflake(),
    staging=filesystem("s3://my-bucket")
)

The biggest change is that I moved all destinations to dlt.destinations.impl. Just because I think it would be more confusing to have a module and a callable with the same name.
I.e. dlt.destinations.postgres(...) is the factory, dlt.destinations.impl.postgres is the package with all the destination code.

Related Issues

Resolves https://github.com/dlt-hub/dlt/issues/508

@rudolfix would be great to know what you think of the approach before I finalize it.

Not quite done, but some more progress.

DestinationReference is removed entirely. client/capabilities/spec defined on the factory and always use it
Also renamed DestinationFactory to Destination, think that makes sense as now it's "the" base destination
I create capabilities at import time ( https://github.com/dlt-hub/dlt/blob/sthor%2Fparametrized-destination/dlt/destinations/impl/postgres/factory.py#L15 ). I couldn't see a reason to have them behind a callable (except for dummy where they can be configured). Any problem with this?


Allow account address lookup using wallet address

Description
When setting the wallet address with Accounts.sol, there is a WalletAddress role that is set, and the wallet address is set as the default signer for this role. This allows the lookup of the account address from the wallet address by taking advantage of the signerToAccountWithRole function.
Previously, you could set the wallet address to the same thing as the account address. You could also set it to the null address. These are no longer possible.
Tested
The accounts test suite was modified accordingly
Related issues

Fixes #5874

Backwards compatibility
Attempting to set the wallet address to the same thing as the account address or to the the null address will fail silently.
Documentation
None

Trying to set the wallet address to the null address or to the account address itself no longer works, and will result in an error


Downgrade cmake version to 3.20

Description

Downgrade cmake version to 3.20 in code/Examples/c++/DDSHelloWorld/CMakeList.txt and upgrade 3.12 in CMakeList.txt
Reference PR in docs master:

eProsima/Fast-DDS-docs/pull/635

Related fastdds PR:

eProsima/Fast-DDS#4232

Contributor Checklist

[x] Commit messages follow the project guidelines.
N/A Code snippets related to the added documentation have been provided.
N/A Documentation tests pass locally.
N/A Applicable backports have been included in the description.

Reviewer Checklist

[x] The PR has a milestone assigned.
[x] The title and description correctly express the PR's purpose.
[x] Check contributor checklist is correct.
[ ] CI passes without warnings or errors.

@richiprosima please test this

@richiprosima please test this


[resotocore][fix] Improve help

Description

help does not show placeholders and aliases any longer via help
help placeholders shows available placeholders
help <cmd> always shows available aliases or the proper command name
all info messages have a dot.

To-Dos

[x] Add test coverage for new or updated functionality
[x] Lint and test with tox
[ ] Document new or updated functionality (someengineering/resoto.com#XXXX)

Issues Fixed

Fixes #XXXX

Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report

Merging #778 (b86cf37) into main (635a782) will decrease coverage by 0.03%.
The diff coverage is 85.14%.

@@            Coverage Diff             @@
##             main     #778      +/-   ##
==========================================
- Coverage   89.75%   89.71%   -0.04%
==========================================
  Files          69       69
  Lines        9737     9830      +93
==========================================
+ Hits         8739     8819      +80
- Misses        998     1011      +13

Flag
Coverage Δ

resotocore
89.71% <85.14%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

resotocore/resotocore/web/api.py
66.97% <33.33%> (+0.20%)
:arrow_up:

resotocore/resotocore/core_config.py
95.59% <50.00%> (-1.20%)
:arrow_down:

resotocore/resotocore/cli/cli.py
92.97% <71.42%> (-1.19%)
:arrow_down:

resotocore/resotocore/model/model.py
92.56% <84.21%> (-0.95%)
:arrow_down:

resotocore/resotocore/cli/command.py
87.20% <97.50%> (+0.26%)
:arrow_up:

resotocore/resotocore/__init__.py
100.00% <100.00%> (ø)

resotocore/resotocore/__main__.py
87.68% <100.00%> (ø)

resotocore/resotocore/dependencies.py
87.60% <100.00%> (+1.65%)
:arrow_up:

resotocore/resotocore/parse_util.py
100.00% <100.00%> (ø)

resotocore/resotocore/web/certificate_handler.py
94.00% <100.00%> (+0.31%)
:arrow_up:

... and 2 more

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 690c1fd...b86cf37. Read the comment docs.


GlobalInit()] [E] Failed to initialize ORM engine: migrate: do migrate: CreateIndexes RepoUnit: pq: relation "IDX_repo_unit_S" already exists

As of commit 94130da63a31da59207dd7c4f12aa46fde17ab5b I cannot start Gitea against the development database I've been using so far, due to this error:
2017/02/06 17:56:09 [I] Migration: create repo unit table and add units for all repos
2017/02/06 17:56:09 [...itea/routers/init.go:54 GlobalInit()] [E] Failed to initialize ORM engine: migrate: do migrate: CreateIndexes RepoUnit: pq: relation "IDX_repo_unit_created_unix" already exists

I think the call comes from here:
models/migrations/v16.go:       if err := sess.CreateIndexes(&repoUnit); err != nil {

I'm not sure why that migration is even attempted to be performed, given the DB is already at version 16:
=# select * from version;
 id | version
----+---------
  1 |      16
(1 row)

@lunny ideas ?

I've fixed this by dropping the repo_unit table. Could be I tried the repo_unit branch before...

yes, because you tried that PR


How to make a Frontier demo for running Ethereum smart contracts

Description
Inside the Frontier repo/template folder https://github.com/paritytech/frontier/tree/master/template
It was not clear How to make a Frontier demo for running Ethereum smart contracts?
see https://substrate.stackexchange.com/questions/3766/how-to-make-a-frontier-demo-for-running-ethereum-smart-contracts
the template/Readme says: This template is maintained in the Frontier project repository, and can be used to generate a stand-alone template for use in an independent project via the included template generation script.... This is Option 1. That is to manually run that script.
But then further down that Readme, it says To build the chain, execute the following commands from the project root:... this is Option 2. That is to build it via cargo build --release
Also I am aware of @JoshOrndorff's tutorial about how to add Frontier into a Substrate Node Template: https://github.com/PureStake/substrate-node-template/commits/frontier-workshop-2021, but that is a few months old. This is Option 3: that is to add Frontier code to the latest Substrate Node Template...
I am very confused by those options.
After fixing the Frontier with the correct Substrate version I need, how can I build a demo for running Ethereum smart contracts?
Incorrect Option 2 Steps
in the template/readme.md, it says
$ cargo build --release
$ ./target/debug/frontier-template-node --dev

but after building, there is no debug folder under target!
And the argument --manual-seal does not work, either
This works:
./target/release/frontier-template-node --dev

Then I followed
Example 1: ERC20 Contract Deployment using EVM dispatchable
The following steps are also available as a [Typescript script](https://github.com/paritytech/frontier/blob/master/template/examples/contract-erc20) using Polkadot JS SDK

https://github.com/paritytech/frontier/tree/master/template/examples/contract-erc20
it says
Installation and Usage
Use npm i to install dependencies. To create an ERC20 contract, execute node_modules/.bin/ts-node create-erc20.ts while your template node is running in --dev mode.

I did that, but got errors:
contract-erc20 git:(patch-polkadot-v0.9.24) ✗ node_modules/.bin/ts-node create-erc20.ts

/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:434
    return new TSError(diagnosticText, diagnosticCodes)
           ^
TSError: ⨯ Unable to compile TypeScript:
create-erc20.ts:4:28 - error TS2307: Cannot find module 'web3-utils' or its corresponding type declarations.

4 import * as web3Utils from 'web3-utils';
                             ~~~~~~~~~~~~

    at createTSError (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:434:12)
    at reportTSError (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:438:19)
    at getOutput (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:578:36)
    at Object.compile (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:775:32)
    at Module.m._compile (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:858:43)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Object.require.extensions.<computed> [as .ts] (/mnt/sda3/polkadot/frontier-paid/template/examples/contract-erc20/node_modules/ts-node/src/index.ts:861:12)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)

Expected vs. Actual Behavior
Environment

Operating system:
Linux Mint 20.3 Una, based on Ubuntu 20.04 focal

Project version/tag:

Rust version (run rustup show):

installed toolchains
stable-x86_64-unknown-linux-gnu (default)
nightly-x86_64-unknown-linux-gnu
active toolchain
stable-x86_64-unknown-linux-gnu (default)
rustc 1.61.0 (fe5b13d68 2022-05-18)
Logs, Errors or Screenshots
Additional Information

As you noticed, it should be target/release/frontier-template-node. A PR for that is welcome.
I can't help you with the ERC 20 demo. That should be its own issue. You could also use any solidity example you find around the internet.
If you just want to use any chain that has working Frontier, then you should build their template. My tutorial is more useful if you have an already existing codebase and want to add Frontier to it. If you are starting a new project I would recommend building the existing Frontier template.
And speaking of my tutorial, there is a video recording discussing each commit at https://www.youtube.com/watch?v=V9KfvhoqLJ4 that may help you if you choose option 3.

Hi Josh, I figured it out ... here are the steps to take:
cargo build --release
./target/release/frontier-template-node --dev

That runs a chain based on Substrate Node Template.
The hidden port is actually 9933, so here is the correct URL to connect your Ethereum tools to the running chain:
      url: "http://<IP_ADDRESS>:9933",
      chainId: 42,

I have no idea why the chainId was set to 42, which is the same chainId of the Ethereum Kovan network.
So you can change the chainId here: template/runtime/src/lib.rs
To clarify when to use the Frontier's template folder, I would suggest the following:
If you have another repo that uses Frontier as a dependency, then use Option 1 above with Josh's video.
If you want to run Ethereum smart contracts inside a substrate based chain with minimum effort, use Option 2 above and my steps above.

Hi Joshf @JoshOrndorff, may I clarify a few points here?...
quote "If you just want to use any chain that has working Frontier, then you should build their template."
I have a parachain that has Frontier in it as a dependency, but I can run it I compile it via cargo build --release and run it via ./target/release/parachain-template without building the Frontier template.
I don't understand why I should build Frontier's template... could you clarify?
What can I do after building the Frontier template?
quote from Frontier https://github.com/paritytech/frontier/tree/master/template: "used to generate a stand-alone template for use in an independent project via the included template generation script."
Can this template generate a parachain project?
I need to generate a parachain project(based on substrate parachain template) that uses this Frontier repo as a dependency.
How can I do that?
Thank you

If your parachain node is already working, that's great. I didn't know that before. I thought you just wanted any node at all so that you could experiment with Frontier. If you had nothing done already, I was suggesting you could use the template.
Since you already have a node working, that suggestion does not apply to you.

There is a bug in my parachain regarding Frontier, so I wanted any node because I wanted to test if my upgraded Frontier branch with polkadot-v0.9.24 is working or not.
That tested good. But I need to find the bug in my parachain...
So I need a template parachain that uses Frontier branch as a dependency.
is there such a template parachain? Thanks

Okay I have solved the Ethereum error...
the Frontier Node Template repo is really useful: https://github.com/substrate-developer-hub/frontier-node-template/tree/main


add documentation on how to deploy OCP cluster using Assisted Installer

Hi Team
I had to change one thing in mkdocs.yml file to activate tabs.
 markdown_extensions:
 - pymdownx.tabbed:
      alternate_style: true

Other than that:

I have checked rendering - please review
I have checked content - please review
Uploaded terraform files in zip format and changed the URL in the instructions to point to nutanix-cloud-native/opendocs repository

Will submit a new PR when the main is up to date with the recent menu and layout changes. This will make it easy for me. Thank you.


html time element use case

Be good if there was a canonical example how to use it in conjunction with uses of https://html.spec.whatwg.org/multipage/semantics.html#the-time-element
For example is there a non-jquery example that simply presents any time element with dateTime= defined in a web page? Something better than my own: view-source:http://bus.dabase.com/

Sounds like a good topic for a blog article.  Feel free to write one. :)

Yes, I am also curious how to use moment.js to represent only time.

Closing due to age. please re-open if you would still like this.


Autorest Regen Preview to 3.0.0-alpha.20230802.1 by Dapeng Zhang from refs/pull/3483/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3483

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ChangeAnalysis


Tighter likelihood

It gives slightly higher results than the reconstruction loss because the z drawn from q(z/x) has a higher likelihood under the measure q(z/x) than under p(z), which makes sense because of the samll discrepancy between the two distributions. The library size doesn't impact differently this computation of the LL. Closes #73

Is compute_log_likelihood the same as compute_tighter_log_likelihood with n_samples_mc=1? Could you rename compute_tighter_log_likelihood to  compute_log_likelihood, and delete what's currently called compute_log_likelihood?

I think it'd be better not to have two different functions for computing log likelihood, if at all possible, and the new code looks pretty general.

@jeff-regier actually they're different, compute_log_likelihood only computes the reconstruction loss, so it's still different than when n_samples_mc =1. We could replace the compute_log_likelihood function so that per default it uses the new formula, but what I fear is that if we set n_samples_mc=1 it might have high variance and not be as a good indicator as just the reconstruction loss, and if we set n_samples_mc to be higher it will take too long to compute it every time we compute the general metrics for the model (every 5 epochs for instance). What do you think about it?

Ah, I got it. How about renaming compute_tighter_log_likelihood then to compute_marginal_log_likelihood? That should distinguishes it from compute_log_likelihood, which estimates log p(x | z) rather than log p(x). Also maybe add some docstrings to clarify it?

I'm on it

Would you call the new function too from the benchmarks, instead of calling the old .ll method here?
https://github.com/YosefLab/scVI/blob/d2c6c221fe64b8b2c1b2eeef22d2718afa43d51c/scvi/benchmark.py#L24
It's been log p(x) we wanted there all along, not log p(x | z).

Codecov Report

Merging #138 into master will decrease coverage by 0.05%.
The diff coverage is 91.89%.

@@            Coverage Diff             @@
##           master     #138      +/-   ##
==========================================
- Coverage   94.29%   94.23%   -0.06%
==========================================
  Files          35       35
  Lines        1700     1736      +36
==========================================
+ Hits         1603     1636      +33
- Misses         97      100       +3

Impacted Files
Coverage Δ

scvi/benchmark.py
100% <100%> (ø)
:arrow_up:

scvi/inference/variational_inference.py
89.74% <83.33%> (-0.26%)
:arrow_down:

scvi/metrics/log_likelihood.py
96.15% <93.33%> (-3.85%)
:arrow_down:

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 d2c6c22...1f0e39a. Read the comment docs.


Updated UI

Created a new branch so that only the UI is changed.
Changes made to HTML and SCSS files.
Fixed the link (in the previous UI, the chrome web store link did not work)
UI before:

UI after

Added a UI const which would make it easier to change UI in the future

Using * {} is not really recommended because you're literally saying "Select everything".
Now, in this extension this is "OK", BUT if you're working on a larger project where each page might have different styling, this will become an issue.
So it's always good practice to be more specific or target classes/IDs when possible.
I'll ignore it for now, because I'm planning on to clean up the HTML/SCSS a tiny bit, because if you notice in SCSS we target a lot of IDs. This once again is "OK" on this small scale, BUT its better practise to change the IDs in HTML to classes, this will also allow you to have same style buttons on every page with just one style and so on.
For the sake of this PR its ok tho as there is cleaning up job to come

BUT its better practise to change the IDs in HTML to classes, this will also allow you to have same style buttons on every page with just one style and so on.

I was about to use classes but since everything else was in ids, I decided to go with the flow, but I will keep it in mind for the next time.

good job

Thanks 👍


Style the font

I cannot figure out how to style the title. Help is appreciated, thanks!

you can write html and inline css within the caption option, see this demo


PROTECT API

How can I protect my API by using this plugin?

Hello, this is captcha from client side for simple solutions, it don't depend or care about APIs


Incorrect null_count of DictionaryArray

Describe the bug
null_count of DictionaryArray calls the default implementation which gets null count of its data member. So basically it looks at the null count of keys. If a corresponding value of a key is null, null_count cannot get correct null count.
is_null and is_valid also have same issue.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Removing bug as it was determined that this was not a bug

Yea, thanks @alamb


Provide SQL Server service

What we're after
cloud.gov provides a brokered SQL Server service.
Hypothesized benefit(s)/why:

We have had requests from customers interested in migrating to cloud.gov to offer a brokered version of SQL Server
Applications written in .NET are often more tightly coupled with SQL Server, so supporting a brokered version might encourage more system owners to look at migrating .NET applications to the platform.
Enable faster transition for applications included in the MAX.gov migration effort

Potential metrics

New BD leads advanced because of offering this option
Number of new RDS types created

Further context for those unfamiliar with what we're doing
Currently, cloud.gov offers several different flavors of brokered RDS - PostgreSQL, MySQL and Oracle. SQL Server had been offered at one point in time, but was deprecated due to lack of demand.
Security considerations
Would require modifications to the AWS broker. Uncertain if adding a new RDS type would intro any new security related issues beyond what are normally assumed for other RDS services. We will submit an SCR to the FedRAMP JAB reviewers, but they believe this capability can be re-enabled with minimum fuss.
Notes for implementers
See this thread in #cg-support, which includes an archaeological dig into the history of the AWS broker code for SQL Server.
Related issues/sub-projects

[ ] #1772
[ ] Re-enable SQL Server in the AWS broker
[ ] Ensure SQL Server instances meet the CIS standard (or STIG standards if possible)
[ ] Re-add SQL server in the RDS service site documentation
[ ] Announce availability of SQL Server in RDS service and contact previously-interested customers

We spoke to the FedRAMP JAB about re-enabling SQL Server as a plan in the cloud.gov RDS broker this week.
As this change would not impact existing controls, diagrams, etc. and the capability existed previously, they agreed we should be able to re-enable it without full assessment and turn around the approval quickly.
They asked us to do a write-up in the Significant Change Request (SCR) template and mail it to them to kick that off. We will do that early this coming week.

@pburkholder said:

Might help to demonstrate, with examples like https://github.com/mitre/aws-rds-oracle-mysql-ee-5.7-cis-baseline and https://github.com/mitre/aws-rds-infrastructure-cis-baseline, that our broker churns out CIS benchmarked instances.

We're a little restricted there... Note that:

The most recent CIS benchmark available for SQL Server is 2017, but it's oriented toward Windows hosts, not AWS RDS.
There's a separate 2017 benchmark repository for AWS RDS instances, but it's new and empty.
There's another AWS RDS-specific repository, but for 2014.

Do they want SQL Server 2019 or 2016 or ...?

Here's the full AWS RDS SQL Server version list. I'd like to only support the latest versions, but that's a good question for @wesley-dean-gsa and @lanierkyle. Will prompt them to post here.

SQL Server 2012 would be a marketable option for cloud.gov considering it is backwards compatible for earlier versions, and we have two partners specifically using 2012.  Another partner is on 2014, with two additional partners using 2019. For each major version mentioned (2012, 2014, 2019), would kindly suggest the latest minor version and latest RDS API engine version.
Should a partner not be on the latest minor version, it would not be unreasonable on our side to encourage and assist their effort towards healthy database upkeep.

Can anyone comment as to the overall status of this epic, given work in the most recent quarter?

No real progress. I did some exploratory work on determining the CIS benchmarks for this and how to evaluate compliance using Inspec and MITRE's work.


Add support of GOST2012 algorithms for CmsSignedDataGenerator

Add support of GOST2012 algorithms for the CmsSignedDataGenerator class. This helps to generate a digital signature with actual russian algorithms.

Bump. It is still needed

Hi, can you please include this changes. I think it's linked with your MR.
I'm like your MR, but verification of GOST34.10-2001 cms signature doesn't work propper. I'm resolve this by creating this patch:
diff --git a/crypto/src/cms/CMSSignedHelper.cs b/crypto/src/cms/CMSSignedHelper.cs
index 9717c018..22c29608 100644
--- a/crypto/src/cms/CMSSignedHelper.cs
+++ b/crypto/src/cms/CMSSignedHelper.cs
@@ -89,6 +89,10 @@ static CmsSignedHelper()
                        AddEntries(EacObjectIdentifiers.id_TA_RSA_v1_5_SHA_256, "SHA256", "RSA");
                        AddEntries(EacObjectIdentifiers.id_TA_RSA_PSS_SHA_1, "SHA1", "RSAandMGF1");
                        AddEntries(EacObjectIdentifiers.id_TA_RSA_PSS_SHA_256, "SHA256", "RSAandMGF1");
+                       AddEntries(CryptoProObjectIdentifiers.GostR3411x94WithGostR3410x94, "GOST3411", "GOST3410");
+                       AddEntries(CryptoProObjectIdentifiers.GostR3411x94WithGostR3410x2001, "GOST3411", "ECGOST3410");
+                       AddEntries(RosstandartObjectIdentifiers.id_tc26_signwithdigest_gost_3410_12_256, "GOST3411-2012-256", "ECGOST3410");
+                       AddEntries(RosstandartObjectIdentifiers.id_tc26_signwithdigest_gost_3410_12_512, "GOST3411-2012-512", "ECGOST3410");

                        encryptionAlgs.Add(X9ObjectIdentifiers.IdDsa.Id, "DSA");
                        encryptionAlgs.Add(PkcsObjectIdentifiers.RsaEncryption.Id, "RSA");
diff --git a/crypto/src/security/SignerUtilities.cs b/crypto/src/security/SignerUtilities.cs
index 3a3e6fe5..e77a49d7 100644
--- a/crypto/src/security/SignerUtilities.cs
+++ b/crypto/src/security/SignerUtilities.cs
@@ -638,11 +638,11 @@ public static ICollection Algorithms
                 return new SM2Signer(digest);
             }

-            if (mechanism.Equals("GOST3410"))
+            if (mechanism.Equals("GOST3410") || mechanism.Equals("GOST3411WITHGOST3410"))
             {
                 return new Gost3410DigestSigner(new Gost3410Signer(), new Gost3411Digest());
             }
-            if (mechanism.Equals("ECGOST3410"))
+            if (mechanism.Equals("ECGOST3410") || mechanism.Equals("GOST3411WITHECGOST3410"))
             {
                 return new Gost3410DigestSigner(new ECGost3410Signer(), new Gost3411Digest());
             }

What do you think?
P.s. I found this note at the bouncy castle home page

This is the final feature release with support for legacy .NET platforms. From 1.9.0 we will be targeting more modern .NET frameworks (see https://github.com/bcgit/bc-csharp/pull/68) and updating our build and packaging systems. The 1.8.x series will continue to receive bug fixes, but limited new functionality.

So, I think your functianality will be merged after 1.9.0 release :)

Finally merged, thanks.
@lo1ol These were already there in latest code.


Extend session cookie to work around CSRF issue

Some browsers have a behaviour where although they will delete a session
cookie when the app is shutdown, they will still serve a cached version
of the page on relaunch. This causes issues with CSRF protection as it
relies on the token embedded in the page/form matching the token in the
session cookie. So a user that then submits the form will get a 422
error (ActionController::InvalidAuthenticityToken) because their is
nothing in the session. We get around this by giving the session cookie
a life beyond the browser session.
See https://github.com/DFE-Digital/claim-additional-payments-for-teaching/commit/613320f028b075b430315c364a9bc34431dabe35
and rails/rails#21948 for further details on
the issue.
Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-881

Does setting it to two weeks mean that an InvalidAuthenticityToken error occurs if I return to the page in two weeks' time?

Does setting it to two weeks mean that an InvalidAuthenticityToken error occurs if I return to the page in two weeks' time?

The answer is yes but why would you leave a browser tab open for 2 weeks, after performing a search and before subscribing to a job alert?

Does setting it to two weeks mean that an InvalidAuthenticityToken error occurs if I return to the page in two weeks' time?

The answer is yes but why would you leave a browser tab open for 2 weeks, after performing a search and before subscribing to a job alert?

The description mentions 'shutting down' and 'relaunching' the app, so I thought it would be about a case where I close the page and reopen it later. Is this actually about keeping a page open for a long time?


cabana: DBCManager only expose methods/signals through MessageId

Prerequisite for https://github.com/commaai/openpilot/pull/27435
Emits message changed signals separately for each source to prepare for multiple DBC files.

@deanlee any thoughts?

No, everything seems to be okay.

@adeebshihadeh can this be merged? It's blocking some other PRs.

@adeebshihadeh can this be merged? It's blocking some other PRs.

Didn't see this one until after merging a few others. I'll merge as soon as the conflicts are fixed.

Rebased!


⚠ Bofrost.gr has degraded performance

In 8f3855c, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 3072 ms

Resolved: Bofrost.gr performance has improved in 169c509 after 21 minutes.


Rome should support TypeScript enums

Description:
Rome does not handle TypeScript enums properly. Compilation strips enums in the same way types are stripped, causing the resulting code to fail.
Steps To Reproduce
>    echo 'enum Foo { bar }; console.log(Foo.bar);' > bug.ts
>    rome run bug.ts

     ReferenceError: ___R$$priv$project$rome$bug_ts$Foo is not defined

Expected Results
Rome should not crash and should output the underlying enum value for Foo.bar.

It looks like an enum transpilation module is needed at @romejs/compiler/transforms/compile/transpile/enums.ts that modifies the AST as-needed to create an in-scope enum variable.
Working on a fix now.


Examples do not all execute correctly

from @rafaellehmkuhl
I'm trying to run the examples after installing through Pypi, but I cannot make them run.
The hello_world.py gives me NameError: name 'Text' is not defined.
The beam1.py gives me NameError: name 'drawing_tool' is not defined.

Table of progress:

example
status

beam1.py
:ok_hand: fixed

beam2.py
:ok_hand: fixed

colors_test.py
:ok_hand: fixed

comprehensive_rectangles.py
:ok_hand: fixed

finite_differences.py
:ok_hand: fixed

flow_over_gaussian.py
:ok_hand: fixed

hatch_test.py
:ok_hand: fixed

hello_world.py
:ok_hand: fixed

integral_comic_strip.py
:running: need to be able to pass kwargs to backend

layered_medium_2.py
:ok_hand: fixed

layered_medium_3xi.py
:ok_hand: fixed

layered_medium_general.py
:ok_hand: fixed

mesh_function.py
:ok_hand: fixed

oscillator_sketch1.py
:ok_hand: fixed

oscillator_sketch2.py
:adds nothing above osclilator_sketch1.py

pendulum1.py
:persevere: fixed (with a bug - why is the mass not blue in the free body diagram?!?)

pendulum2.py
:ok_hand: fixed

simple_arc.py
:ok_hand: fixed

simple_arrow.py
:ok_hand: fixed

simple_axis.py
:ok_hand: fixed

simple_circle.py
:ok_hand: fixed

simple_curve.py
:ok_hand: fixed

simple_rectangle.py
:ok_hand: fixed

simple_spline.py
:ok_hand: fixed

simple_text.py
:ok_hand: fixed

simple_triangle.py
:ok_hand: fixed

simple_wall.py
:ok_hand: fixed

staggered_mesh_function.py
:ok_hand: fixed

vehicle.py
:ok_hand: fixed

vehicle_dim.py
:ok_hand: fixed

wheel_on_inclined_plane.py
:ok_hand: fixed

This is all working now aside from the comic strip stuff and the weird lack of blue ball on the end of the pendulum. I really don't think drawing comic strips is a priority for 0.1.0 so unless anyone shouts on this issue, I'll resolve the pendulum example and then call this done.

I checked all examples in the repo. Except the 'comic-strip' ones (2 files), all are working: they produce an image, or a movie without throwing exceptions; that is not to say they are correct, I have not checked that.
The list is as follows:
working examples
animation.py     --> produces an .mp4
beam1.py
beam2.py
colors_test.py
comprehensive_rectangles.py
finite_differences.py
flow_over_gaussian.py
hatch_test.py
hello_world.py
layered_medium_2.py
layered_medium_3xi.py
layered_medium_general.py
mesh_function.py
oscillator_sketch1.py
pendulum1.py
pendulum2.py     --> produces an .mp4
simple_axis.py
staggered_mesh_function.py
vehicle.py
vehicle_dim.py
wheel_on_inclined_plane.py

broken examples
integral_comic_strip.py
ForwardEuler_comic_strip.py

I have submitted a PR to isolate the broken examples under a legacy folder
As a side question, not directly related to pysketcher: the images produced by matplorlib are huge (way larger than my 27 in screen) - there is a tight_layout option for pyplot that solves this with retina displays; does someone know what is the equivalent command for matplotlib.Figure?


Include title when exporting a chart

See: http://jsfiddle.net/BdHJM/
http://api.highcharts.com/highcharts/exporting.chartOptions

Calling update method on the chart object sets the title first time, but subsequent calls cause an error when hitting the button.


JsonRpcProvider failed to detect network and cannot start up; retry in 1s (perhaps the URL is wrong or the node is not started)

Hi Team, after running the node deposit.js , i hit the error that read
"JsonRpcProvider failed to detect network and cannot start up; retry in 1s (perhaps the URL is wrong or the node is not started)" , could you help how to solve this?

Oh I found my own solution. As the error says that it's link to RPC PRovier, so I check for another RPC for sepolia and found the correct one that work: https://rpc2.sepolia.org so to solve it, simply change the URL to https://rpc2.sepolia.org or search for another RPC for sepolia that work.
All the best folk


Use precisionErrorTolerance

Description
Prefer using precisionErrorTolerance rather than random hardcoded epsilons when we need an epsilon.  Also prefer moreOrLessEquals (which uses precisionErrorTolerance by default) rather than closeTo.
Tests
I added the following tests:
Updated existing tests.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Some of the updated tests appear to fail on windows/Linux but pass on macOS...


Adding circle ci

Sorry, I didn't notice this in time, but I also see that the request was closed same day.
What was the reason for closing, and what is the better alternative you found? Thanks.

I was experimenting with CircleCI on our fork, and accidentally raised a PR against your repository.
Sorry, should have made that clear when I closed the PR.


Scalajs version upgrades + Char#isLetter workaround

Hi Sébastien,
I've been using it quite extensively and it seems to be working well on RC1. I need the Char#isLetter workaround though we talked about like 3 months ago (as it is not implemented in scala.js)... See Lexical.scala ... Imho it is not usable without it.
I'll push following commits upgrading to 0.6.0 and scala 2.11.5 soon. I'll might try to rebase further commits that happened after you forked it.

That's all.

Done, it works the same as before. There are 4 failing tests but it was inevitable as I remember.

And the two commits are in the wrong order. The patch for isLetter should come first, otherwise the upgrade commit doesn't build.

And CI says the .travis.yml should be updated.
Note that the build passes on the upstream branch scalajs: https://travis-ci.org/sjrd/scala-parser-combinators/builds/35359954
So everything should still pass in the PR.

Done, now it really should comply :-)

Done,  I don't know how to translate the saying that the third attempt will certainly do :-)

LGTM Thanks!


[style]:

What's the style idea?
We need to improve  On hover effect on navbar
Add screenshots
No effect on hover

Code of Conduct

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

Duplicate of https://github.com/priyankarpal/ProjectsHut/issues/373. please try to check the existing issues first


Adding Colorado State University.

@anicharumpa I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host colostate.edu.ng not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


CDMD-2542 Improve displayed message for codemod list

do you think it's needed? if you're looking to improve the codemod list command i would look for a way to display it in multiple rows and make them colored based on if the codemod is verified or not, if we have access to it (later, we ideally should have it, to tell the user what are the codemods that are safe to run)
what i mean by rows can be seen when using lsd CLI:

yes, I absolutely agree with you on this. I will close the PR.


recent code cause runtime error for UKAEA openmc workshop task12

here is error message
task12 code
use a plain point source coming with OpenMC official is fine, as suggested by Jonathon.  see the code tweak at the end.
===============>     FIXED SOURCE TRANSPORT SIMULATION     <===============
Simulating batch 1
WARNING:  WARNING:  WARNING: Could not find the cell containing particle 1
WARNING: Could not find the cell containing particle 501
Could not find the cell containing particle 876
Could not find the cell containing particle 251
WARNING: Could not find the cell containing particle 751
WARNING: Could not find the cell containing particle 626
WARNING: Could not find the cell containing particle 376
WARNING: Could not find the cell containing particle 126
Traceback (most recent call last):
File "example_unstructured_mesh_simulation.py", line 99, in
model.run()
File "/home/qxia/.local/lib/python3.8/site-packages/openmc/model/model.py", line 228, in run
openmc.run(**kwargs)
File "/home/qxia/.local/lib/python3.8/site-packages/openmc/executor.py", line 218, in run
_run(args, output, cwd)
File "/home/qxia/.local/lib/python3.8/site-packages/openmc/executor.py", line 28, in _run
raise subprocess.CalledProcessError(p.returncode, ' '.join(args),
subprocess.CalledProcessError: Command 'openmc' died with <Signals.SIGSEGV: 11>.
# creates a source object
source = openmc.Source()
if False:
    plasma_params = {
        "elongation": 2.9,
        "ion_density_origin": 1.09e20,
        "ion_density_peaking_factor": 1,
        "ion_density_pedestal": 1.09e20,
        "ion_density_separatrix": 3e19,
        "ion_temperature_origin": 45.9,
        "ion_temperature_peaking_factor": 8.06,
        "ion_temperature_pedestal": 6.09,
        "ion_temperature_separatrix": 0.1,
        "major_radius": 1.9*100,
        "minor_radius": 1.118*100,
        "pedestal_radius": 0.8 *  1.118*100,
        "plasma_id": 1,
        "shafranov_shift": 0.44789,
        "triangularity": 0.270,
        "ion_temperature_beta": 6,
    }

    plasma = PlasmaSource(**plasma_params)
    # this creates a neutron distribution with the shape of a tokamak plasma

    #source = openmc.Source()
    source.library = SOURCE_SAMPLING_PATH
    source.parameters = str(plasma)
else:
    # point source from task4
    source.space = openmc.stats.Point((0, 0, 0))
    source.angle = openmc.stats.Isotropic()
    source.energy = openmc.stats.Discrete([14e6], [1])

sett.source = source

Ive no idea on this im afraid, with a volume source its possible that particles are being born on surfaces, which is naughty, but not really a parametric plasma source problem.


How to get data from url in webview

How can I track the change of url on webview. Because I want to get data from the path to my process

Have you found a solution?


feat: BREAKING metadata store

error and warning are currently their individual stores. Probably overkill.
Instead, have a combined metadata store that contains the error, warning and any other user-relevant data for your valueStore.
My use case is to have $metadata.visited, so onValidate won't trigger an error before the user has visited an input.

Closed — will not impact svelte-object for Svelte 5


Fix cherrypick discrepancy in #93 and #144

The code snippet was not removed by cherrypicking from #93 in #144. Remove it to fix multi-repo test failure on v1.13 branch.

/hold

/unhold

/retest


ImportError: cannot import name 'cygrpc' from 'grpc._cython'

What version of gRPC and what language are you using?
Python, grpcio 1.51.1
What operating system (Linux, Windows,...) and version?
Mac
What runtime / compiler are you using (e.g. python version or version of gcc)
3.9
I'm actually using Bazel to run it:
load("@rules_python//python:repositories.bzl", "python_register_toolchains")

python_register_toolchains(
    name = "python3_9",
    ignore_root_user_error = True,
    python_version = "3.9",
)

What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
Running a python file in Docker image which imports grpc module.
Sample code could be seen at https://github.com/wsoesanto/docker_python. To repro, it can be seen by running bazel run :main.
What did you expect to see?
Not throwing exception.
What did you see instead?
It crashes with the following error:
Traceback (most recent call last):
  File "/app//main.binary.runfiles/__main__/main.py", line 1, in <module>
    import grpc
  File "/app/main.binary.runfiles/pypi_grpcio/site-packages/grpc/__init__.py", line 22, in <module>
    from grpc import _compression
  File "/app/main.binary.runfiles/pypi_grpcio/site-packages/grpc/_compression.py", line 20, in <module>
    from grpc._cython import cygrpc
ImportError: cannot import name 'cygrpc' from 'grpc._cython'

Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?
My laptop is Apple M1 Pro.
$ uname -a
Darwin <IP_ADDRESS> 22.1.0 Darwin Kernel Version 22.1.0: Sun Oct  9 20:15:09 PDT 2022; root:xnu-8792.41.9~2/RELEASE_ARM64_T6000 arm64

This is a known incompatibility between the gRPC Python Bazel build rules and Bazel Python toolchains. There is ongoing work to resolve this incompatbility.
The gRPC Python Bazel build requires access to Python source headers (i.e. Python.h) to build the C extension. This is pulled from the host system. What you're seeing here is an instance of the gRPC Python library having been built for a different Python minor version than the version of the interpreter you're running (evidently 3.9 from your message). For the moment, I would recommend that you install the same version of Python on the host system (including the dev package for that version, which includes the build headers) as you configure in your toolchain and make it resolveable as python3 based on your PATH. Alternatively, you can set the environment variables PYTHON3_BIN_PATH and PYTHON3_LIB_PATH to point to the appropriate location. If using pyenv, be aware that only pyenv global will work with Bazel, not pyenv local since it is the Bazel daemon that does the resolution, not the client process spawned from your shell.
Be on the lookout for full Bazel Python toolchain support within the next couple of months.

Is this bug fixed ?

@gnossen i would also like to see if this is fixed. Still fails for me

I'm still having this problem, too. Is it only fixed on certain Python versions, or only for certain GRPC versions?

I get this error when importing google texttospeech module :
Traceback (most recent call last):
  File "/home/vic/aiychatgpt/./aiychatgpt.py", line 14, in <module>
    from google.cloud import texttospeech
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/cloud/texttospeech/__init__.py", line 21, in <module>
    from google.cloud.texttospeech_v1.services.text_to_speech.async_client import (
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/cloud/texttospeech_v1/__init__.py", line 21, in <module>
    from .services.text_to_speech import TextToSpeechAsyncClient, TextToSpeechClient
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/cloud/texttospeech_v1/services/text_to_speech/__init__.py", line 16, in <module>
    from .async_client import TextToSpeechAsyncClient
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/cloud/texttospeech_v1/services/text_to_speech/async_client.py", line 35, in <module>
    from google.api_core import gapic_v1
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/api_core/gapic_v1/__init__.py", line 16, in <module>
    from google.api_core.gapic_v1 import config
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/google/api_core/gapic_v1/config.py", line 23, in <module>
    import grpc
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/grpc/__init__.py", line 22, in <module>
    from grpc import _compression
  File "/home/vic/aiychatgpt/env/lib/python3.11/site-packages/grpc/_compression.py", line 20, in <module>
    from grpc._cython import cygrpc
ImportError: cannot import name 'cygrpc' from 'grpc._cython' (/home/vic/aiychatgpt/env/lib/python3.11/site-packages/grpc/_cython/__init__.py)

While google cloud claims to support python 3.11. Or can it be something else in my setup is wrong?
Would it work when I use another python version than 3.11? If so, which one?


Fix SBT import example

Really minor thing - noticed that quotes are missing on your example for updating libraryDependencies.

Good spot, thanks for the help


bruig: Context Menu issues

Long press not working on chats
Context Menu not closing when left-clicking category/app bar

Adding:

Neither long press nor right click work on the user name itself in chats (only right clicking on the icon currently), while it works on both in the user list
No context menu on long press or right click on the channel list entries (it does bring up the bar on the right, but no context menu like with users)

Another thing that may or may not be intentional is that the the entries in the user context menu do not match the entries in the user profile menu to the right for users.

https://github.com/companyzero/bisonrelay/pull/229 will fix long presses on chats and add gc context menus.
About:

Context Menu not closing when left-clicking category/app bar

This happens with all our modals, context menu is just another kind of modal so that's a broader issue.

Neither long press nor right click work on the user name itself in chats (only right clicking on the icon currently), while it works on both in the user list

This is intentional. It works in the channel list because we want it to work in the entire "list cell".

Another thing that may or may not be intentional is that the the entries in the user context menu do not match the entries in the user profile menu to the right for users.

This is intentional as well. The entries left out only work well in the "chat area". We don't have a good way to list posts or show user content outside of the chat, for example. But the User Context Menu is used in another places like the Feed and Post screens.


Do not use more than three initials in case of authshort key

According to the documentation, the authshort key is supposed to give

The last name if one author is given; the first character of up to three authors’ last names if more than one author is given. A plus character is added, if there are more than three authors.

This is also reflected in the comment in line 326 of the modified file. However, the current implementation gives first characters for all authors, and then adds a plus if there were more than three of them.
To reproduce this bug: My configuration is [authshort:nopunct][shortyear]; a paper by authors Alpha, Beta, Gamma, Delta receives the key ABGD+18, whereas the expected behavior would be ABG+18.
Based on the surrounding code, I expect this pull request to fix the bug. However, I failed at building the plugin, and therefore was unable to check this. Could someone please test it before merging?

Thank you for this fix! The tests passed even if the build showed red; you can ignore the semaphore builds, the Travis builds are authoritative, and Travis only showed red because the deploy step failed. I've excluded deploy from pull requests, so in the future anyone who submits a pull request can look at the Travis state to see if it passes.
I've added a test case for this bug since the existing test suite did not contain a sample that triggered the fault, so the tests running green would only have confirmed that there's no regression but not that the fault has been detected and fixed; adding this test case will prevent regression on this particular bug in the future.
Not that I would have insisted you built & test before hand, I really, really appreciate you thought about this. May I ask what problems you were experiencing in building BBT?

Don't bother with the test release, I've cut a new formal release

Wow, you are fast :) Thank you so much for your help! I will make an issue with my building problems in a minute.
One more question: Are the points between initial expected behavior? E.g., if my configuration was just [authshort], the citation key currently would be A.B.G+, my assumption would have been ABG+.

Fair :) (Especially since my preferred behavior can be recovered with [authshort:nopunct] anyway.)
Thank you very much for the plugin!


Create filters in grafana to support subgroups of apps

@mpryc FYI.. what else came out of that meeting?

is this a duplication of #964?


event view for instructors gets very slow for increasing number of students

I noticed this for the DTC Software Engineering Course, the intructor view for the event is very slow. Perhaps because there is alot of students and alot of problems all being rendered. Need to profile this page and make it scale better with increasing students/problems

perhaps react-virtuoso


Make kind argument in CFTimeIndex._maybe_cast_slice_bound optional

Pandas recently deprecated the kind argument in Index._maybe_cast_slice_bound, and removed its use in several internal calls: https://github.com/pandas-dev/pandas/pull/41378.  This led to some errors in the CFTimeIndex tests in our upstream build.  We never made use of it in CFTimeIndex._maybe_cast_slice_bound so the simplest fix for backwards compatibility seems to be to make it optional for now -- in previous versions of pandas it was required -- and remove it when our minimum version of pandas is at least 1.3.0.

[x] Closes #5356
[x] Passes pre-commit run --all-files
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Hello @spencerkclark! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/coding/cftimeindex.py:

Line 471:1: W293 blank line contains whitespace

thanks again, @spencerkclark.


[util] Add alternate hjson paths for reggen_only modules

Fixes #8207

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

@sha-ron i'm not completely sure if this resolves your issue.
Please give it a try.

@sha-ron , can you try the static generation first to see if you get the right ast reg file.
And maybe we should file a separate issue to track the ral-gen change?
It might make sense for me to rework this in a different way?

Sorry, maybe I didn't explain the problem well.
There was no issue with the static generation.
The issue is that according to top_earlgrey.hjson the ast.hjson file should be located under top_earlgrey/ip/ast/data/ast.hjson,  but we need to use the private ast.hjson that is located under Nuvoton repository.

just to confirm.
The fix i added basically tells topgen to look elsewhere for an hjson file.
Once it finds its, the generated rtl however is still stored in
hw/top_earlgrey/ip/ast/rtl, thus overwriting the original.
Is this not the behavior you're seeing?
On Mon, Nov 15, 2021 at 4:10 PM sha-ron @.***> wrote:

Sorry, maybe I didn't explain the problem well.
There was no issue with the static generation.
The issue is that according to top_earlgrey.hjson the ast.hjson file
should be located under top_earlgrey/ip/ast/data/ast.hjson, but we need to
use the private ast.hjson that is located under Nuvoton repository.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/9184#issuecomment-969324048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSV7M2VPSHJHITCIWMTUMFZLLANCNFSM5H3PVTJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I didn't see this behavior but this is probably because as a bypass I just copied the correct hjson to hw/top_earlgrey/ip/ast/rtl.

ah please try it with the file in the actual location and let me know if it
helps.
I think we still need to resolve the ralgen issue i think, where ralgen is
also aware of such a path and passes it to topgen.
On Mon, Nov 15, 2021 at 4:32 PM sha-ron @.***> wrote:

I didn't see this behavior but this is probably because as a bypass I just
copied the correct hjson to hw/top_earlgrey/ip/ast/rtl.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/9184#issuecomment-969341210,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSQGXXMGZRL7JE4V5L3UMF353ANCNFSM5H3PVTJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

yes need some help from @sriyerg about how to try it without dvsim


conformance-profiles: Add Submodule and util function

 Validate normalised data against profiles; expose results in API; display on status page

closed in favour of new one


Error while running moodbot tutorial

Rasa Core 0.7.3
OS: Ubuntu 16.0.4
Issue description:
Upon running
python -m rasa_core.run -d models/dialogue -u models/model_20171101-161157/
the following error is occurring
INFO:root:Rasa process starting Using TensorFlow backend. 2017-11-01 16:50:25.531870: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations. 2017-11-01 16:50:25.531896: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations. 2017-11-01 16:50:25.531917: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations. 2017-11-01 16:50:25.531922: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations. 2017-11-01 16:50:25.531927: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations. Traceback (most recent call last):   File "/home/chaitanyag/software/anaconda2/lib/python2.7/runpy.py", line 174, in _run_module_as_main     "__main__", fname, loader, pkg_name)   File "/home/chaitanyag/software/anaconda2/lib/python2.7/runpy.py", line 72, in _run_code     exec code in run_globals   File "build/bdist.linux-x86_64/egg/rasa_core/run.py", line 126, in <module>   File "build/bdist.linux-x86_64/egg/rasa_core/run.py", line 105, in main   File "build/bdist.linux-x86_64/egg/rasa_core/agent.py", line 55, in load   File "build/bdist.linux-x86_64/egg/rasa_core/interpreter.py", line 27, in create   File "build/bdist.linux-x86_64/egg/rasa_core/interpreter.py", line 132, in __init__   File "build/bdist.linux-x86_64/egg/rasa_core/interpreter.py", line 151, in _load_interpreter   File "build/bdist.linux-x86_64/egg/rasa_nlu/model.py", line 212, in load   File "/home/chaitanyag/software/anaconda2/lib/python2.7/site-packages/future/types/newstr.py", line 331, in __getattribute__     return super(newstr, self).__getattribute__(name) AttributeError: 'newstr' object has no attribute 'pipeline'

Which version of rasa NLU are you using? You need to install the latest one

The issue is sorted now. Thanks


Sometimes rounding goes wrong

Here is my fiddle with problem.
https://jsfiddle.net/9nd5x348/6/
When I turn off wNumb formatting problem disappears

When you add wNumb to your noUiSlider, there no longer is a default number of decimals. You should add them manually.
Updated fiddle.


Prometheus not showing metrics for terminated Pods

I am using kubelet cadvisor for collecting container/pods metrics.
In my prometheus.yml target, I have mentioned "myclusterip:10255/metrics/cadvisor". and I can see metrics for running containers/Pods but no details available for Pods which got terminated earlier. I have a retention period of 14 days.
I need metrics for old (terminated/deleted) containers for performance analysis.

a part of my prometheus config from compose-file
prometheus:
    image: prom/prometheus:v2.8.1
    networks:
      - monitor
#    ports:
#      - '9090:9090'
    volumes:
      - /promstack/prometheus.yml:/etc/prometheus/prometheus.yml:ro
      - /promstack/prom_data:/prometheus:rw
      - /promstack/alert.rules:/etc/prometheus/alert.rules:ro
    user: "0"
    command:
      - '--config.file=/etc/prometheus/prometheus.yml'
      - '--storage.tsdb.path=/prometheus'
      - '--storage.tsdb.retention=14d'

and my prometheus.yml looks like this-
global:
  scrape_interval:     60s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
  evaluation_interval: 60s # Evaluate rules every 15 seconds. The default is every 1 minute.
  scrape_timeout: 30s #scrape_timeout is set to the global default (10s).

# Alertmanager configuration
alerting:
  alertmanagers:
  - static_configs:
    - targets:
      - serverip:9093
    timeout: 30s
# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  - "/etc/prometheus/alert.rules"

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'Prometheus'
    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.
    static_configs:
    - targets:
      - localhost:9090

Can someone pls tell, what is missing here due to which the old container details (which got terminated an hour back) not available in Prometheus.

Are you querying for the past timestamp or the current timestamp? Because once a series is not available for some time during scraping (5m), the series is marked stale and won't be shown in the query if you query for the current time.

Thanks @codesome for answering. I'm closing it for now since it looks like a usage question rather than a real issue. If you have further questions, please use our user mailing list, which you can also search.


https://github.com/joomdigi/mod_bootstrapnav/issues/5

Added the ability to hide sub items on a parent item.
Added code to helper.php; to recursively build the MENU and its ITEMS (and sub ITEMS). This change will allow sub items; to have sub sub items.
Formatted helper.php and mod_bootstrapnav.php
Fixed hover bug in default.php with .dropdown:hover; because it was confusing with the click event controlled by BootStrap.
https://github.com/joomdigi/mod_bootstrapnav/issues/5

Menus with more than 2 levels; still feels kind of buggy... in my fork

Content can now correctly display more than 2 levels of menu items.

Even though this revision; can display categories with more than 2 levels of menu items; the following menu items that are positioned after a parent menu item, seem to disappear.

The menu seems to finally not buggy; it will not duplicate the same menu item more than once.


Update dependencies and workflows

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

Comparison is base (c15f037) 100.00% compared to head (c9444fe) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #255   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           95        95
  Lines         3123      3123
  Branches       255       255
=========================================
  Hits          3123      3123

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


Group binaries in "bin" dir and install them globally with setup.py

Hi fairseq team!
I would like to make a pull-request aiming to give fairseq a group of globally callable binaries. Just like tensro2tensor's t2t-train or t2t-datagen, I think that fairseq should expose its binaries like: fairseq-datagen, fairseq-train, fairseq-lmeval, fairseq-generate and fairseq-decode. This is also a fundamental step towards installable pip wheel.
So my idea is to:

Create a bin directory at top-level.
Move eval_lm.py, preprocess.py, train.py, generate.py and interactive.py into bin/fairseq-lmeval, bin/fairseq-datagen, bin/fairseq-train, bin/fairseq-generate and bin/fairseq-decode
Configure setup.py to make it install bins in user bin folder (I think usually /usl/local/bin)
For the sake of back-compatibility, replace top-level scripts (eval_lm.py, preprocess.py, ...) with scripts that print a deprecation warning and then run the correct bin script.

The result would be a much more integrated fairseq installation, for example I can run a training no matter from with the command:
fairseq-train  data-bin --save-dir ...

Are you interested in this feature?

Fixed by https://github.com/pytorch/fairseq/commit/829bd8ce5fc336991c828f178ac5f231250ef77d


feat: change ci workflow

Signed-off-by: disaster37<EMAIL_ADDRESS>

Codecov Report

Merging #7 (d93517d) into main (5e79dae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   58.04%   58.04%
=======================================
  Files          18       18
  Lines        1647     1647
=======================================
  Hits          956      956
  Misses        586      586
  Partials      105      105

Flag
Coverage Δ

unittests
58.04% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 5e79dae...d93517d. Read the comment docs.


Argument type says Sequence but Tuple is not allowed

What happened?
Documentation says that required type is Sequences but in reality pulumi complains if I provide a tuple.
Example
I have confirmed that tuple is a Sequence
from collections.abc import Sequence
isinstance(tuple(), Sequence)

True

But
gcp.compute.Firewall(
    "security-control-egress-allow-approved",
    network=network.name,
    direction="EGRESS",
    destination_ranges=(
        "<IP_ADDRESS>/0"
    ),
    allows=(
        gcp.compute.FirewallAllowArgs(
            protocol="tcp",
            ports=(
                "80",
                "443",
            ),
        ),
    ),
    priority=1000,
)

Returns:

error: gcp:compute/firewall:Firewall resource 'security-control-egress-allow-approved' has a problem: Attribute must be a list. Examine values at 'security-control-egress-allow-approved.destinationRanges'.

Output of pulumi about
Enter your passphrase to unlock config/secrets
(set PULUMI_CONFIG_PASSPHRASE or PULUMI_CONFIG_PASSPHRASE_FILE to remember):
Enter your passphrase to unlock config/secrets
CLI
Version      3.94.2
Go Version   go1.21.4
Go Compiler  gc
Plugins
NAME    VERSION
gcp     7.2.1
python  unknown
Host
OS       ubuntu
Version  23.04
Arch     x86_64
This project is written in python: executable='/home/me/.cache/pypoetry/virtualenvs/[POETRY_ENV_NAME]/bin/python3' version='3.11.4'
Current Stack: organization/Infrastructure/dev
TYPE                                     URN
pulumi:providers:gcp                     urn:pulumi:dev::Infrastructure::pulumi:providers:gcp::default
pulumi:pulumi:Stack                      urn:pulumi:dev::Infrastructure::pulumi:pulumi:Stack::Infrastructure-dev
pulumi:providers:gcp                     urn:pulumi:dev::Infrastructure::pulumi:providers:gcp::default_7_2_1
gcp:compute/network:Network              urn:pulumi:dev::Infrastructure::gcp:compute/network:Network::default
gcp:compute/subnetwork:Subnetwork        urn:pulumi:dev::Infrastructure::gcp:compute/subnetwork:Subnetwork::default
pulumi:providers:gcp                     urn:pulumi:dev::Infrastructure::pulumi:providers:gcp::default_7_0_0
Found no pending operations associated with dev
Backend
Name           XXXXXX
URL            file://~
User           me
Organizations
Token type     personal
Dependencies:
NAME                        VERSION
backcall                    0.2.0
black                       23.11.0
flake8                      6.1.0
ipython                     8.18.1
mypy                        1.7.1
pickleshare                 0.7.5
pip                         23.3.1
pre-commit                  3.5.0
pytest                      7.4.3
wheel                       0.41.2
Pulumi locates its logs in /tmp by default
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @ar-qun. I'm sorry you hit that. I think this is an issue in our codegen or python SDK, so I'm going to transfer the issue to pulumi/pulumi.

Note:
We translate lists as sequences here when generating python SDKs:
https://github.com/pulumi/pulumi/blob/273951bb17ef45efeeebfe25be174c6df10c1e08/pkg/codegen/python/gen.go#L2320-L2322


qTip2 v2.2.1 - Tooltip does not disappear when the pointer has moved away from url

Mouseleave does not destroy tooltip. I have to mouse over the tool tip again for it to disappear.

Thanks

Can you provide a test case for this please?


Violate java coding standard (naming convention)

We are currently using a Person class to represent Supplier. All methods and instances of Person needs to be refactor to Supplier


Add svelte/sort-attributes rule

This PR adds svelte/sort-attributes rule that enforce order of attributes.
close #150

Looked at the docs, love where this is headed and very impressed how fast it came together!
Quick question about the config format, I assume this uses standard regex flags so the /u usage in the examples isn't always necessary unless you need to match extended Unicode characters in attributes?

Thank you for checking! I will proceed with the implementation as it is 😉.
The u flag is not required. I like to use the u flag as similar to strict mode. The u flag will find the wrong use of the regex and let me know as an error. For example, new RegExp ('\\w{1,5', 'u')


TypeError: generate_feature_vectors() missing 1 required positional argument: 'featIdx'

TypeError                                 Traceback (most recent call last)
 in ()
1 drive_time_thresholds = [60, 120, 180, 240] # minutes
----> 2 drive_vectors = ma.generate_feature_vectors(inR, mcp, inD, drive_time_thresholds)
3 drive_vectors.to_file(os.path.join('/content/drive/Shareddrives/Applied Data Science/Engagements/Stanford Community Health Resilience Project/Travel Time/drive_vectors.shp'))
TypeError: generate_feature_vectors() missing 1 required positional argument: 'featIdx'
I am getting this error when trying to generate drive time vectors

In the function documentation they say they are using tempID for default but they don't set featIdx as a kwarg.
https://github.com/worldbank/GOSTNets_Raster/blob/ec96520b3d4e7d7f4b2c8d0a17016d696af3d100/src/GOSTNetsRaster/market_access.py#L168
so you'll need to add the name of the column like this:
drive_vectors = ma.generate_feature_vectors(inR, mcp, inD, drive_time_thresholds, 'tempID')


some doubt about the mat file in submission

I'm confused, the cine_sax.mat and cine_lax.mat which I need to submit, should be in the image domain or k-space domain? If in image domain, should the data range be 0~255 or 0~1?

You should submit the images in spatial domain.
Please refer to the reconstruction demo code we provided: https://github.com/CmrxRecon/CMRxRecon/tree/main/CMRxReconDemo.

2023年8月31日 09:08，hannah-zhangzz @.***> 写道：
I'm confused, the cine_sax.mat and cine_lax.mat which I need to submit, should be in the image domain or k-space domain? If in image domain, should the data range be 0255 or 01?
—
Reply to this email directly, view it on GitHub https://github.com/CmrxRecon/CMRxRecon/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3LGS646TJMGKTCFAC3SIOLXX7PYXANCNFSM6AAAAAA4FJZTZU.
You are receiving this because you are subscribed to this thread.


Adding the blue wordmark version of the footer/navbar logo and renaming the footer/navbar section of HTML files to match

Hello, VulcanSphere is here.
When I browsed this repository, I noticed that this repository lacking the blue wordmark variant of the footer logo (the blue wordmark exists for the horizontal version).
So, I decided to modify the white wordmark version of the footer logo to create the blue wordmark variant and rename the footer logo in HTML files to include the white designation.
Kindly check the changes and adjust them if needed, thank you.

Is the blue version actually used anywhere?
I think an updated version of the logo is planned with a bit of spacing between the JPEG and XL, maybe it's better to wait a bit...

Ah, I see. I'll wait for the updated version but how about changing the file name for consistency (with the horizontal version) but keeping the current footer/navbar logo as it is (not uploading the blue version)?
If you don't mind, it is up to you.

I did something similar to this when uploading the updated versions of the logos, so I think this one can be closed now.


Remove Loki from stf-run-ci

This PR removes everything Loki related from stf-run-ci and hardcodes the logs backend as disabled when deploying with stf-run-ci.

I'll run the tests manually


Issue 7

pylinting

The example.py change should have been on a different branch.
One branch per issue is a cleaner workflow.


Add IPLD CIDs as a multicodec type

Add IPLD CIDs as a multicodec type (ref https://github.com/ensdomains/multicodec/issues/1)

Also see: https://github.com/ipld/cid/issues/25

Seems to be a duplicate of #49.

Kind of. That one is about re-defining 0x1 to be the CID multicodec. This is about defining an entirely new codec.


Add webhook proposal

This is my current view of how webhooks should be added into OLM. Feedback/discussion encouraged.

/retest

/test e2e-aws-console-olm

/retest

/test e2e-aws

Regarding certificate management, have you thought about relying on cert-manager or OpenShift's service-ca?


[Event Request] Report selections: allow multiple prints with same Report ID

Describe the request
Hi, we need to be able to print the same report multiple times via Report Selections. By default the same Usage-ReportID pair is excluded if it exists more than once. We would need to add custom logic that allows multiple printing of the same report.
The logic is contained in the object Table 77 Report Selections, procedure SelectTempReportSelectionsToPrint.
procedure SelectTempReportSelectionsToPrint(var TempReportSelections: Record "Report Selections" temporary; var TempNameValueBuffer: Record "Name/Value Buffer" temporary; WithCheck: Boolean; ReportUsage: Enum "Report Selection Usage"; TableNo: Integer)
    var
        TempReportSelectionsAccount: Record "Report Selections" temporary;
        AccountNo: Code[20];
        LastSequence: Code[10];
    begin
        if TempNameValueBuffer.FindSet() then
            repeat
                AccountNo := CopyStr(TempNameValueBuffer.Name, 1, MaxStrLen(AccountNo));
                TempReportSelectionsAccount.Reset();
                TempReportSelectionsAccount.DeleteAll();
                SelectTempReportSelections(TempReportSelectionsAccount, AccountNo, WithCheck, ReportUsage, TableNo);
                if TempReportSelectionsAccount.FindSet() then
                    repeat
                        LastSequence := GetLastSequenceNo(TempReportSelections, ReportUsage);
                        // ---- Start old version ---->
                        if not HasReportWithUsage(TempReportSelections, ReportUsage, TempReportSelectionsAccount."Report ID") then begin
                        // <---- End old version ----
                            TempReportSelections := TempReportSelectionsAccount;
                            if LastSequence = '' then
                                TempReportSelections.Sequence := '1'
                            else
                                TempReportSelections.Sequence := IncStr(LastSequence);
                            TempReportSelections.Insert();
                        end;
                    until TempReportSelectionsAccount.Next() = 0;
            until TempNameValueBuffer.Next() = 0;
    end;

To get the expected result you could save the current logic in a variable, check the value of that variable for insertion, with the addition of an event just before the check, where we can change the value of the variable.
procedure SelectTempReportSelectionsToPrint(var TempReportSelections: Record "Report Selections" temporary; var TempNameValueBuffer: Record "Name/Value Buffer" temporary; WithCheck: Boolean; ReportUsage: Enum "Report Selection Usage"; TableNo: Integer)
    var
        TempReportSelectionsAccount: Record "Report Selections" temporary;
        AccountNo: Code[20];
        LastSequence: Code[10];
        // ---- New variables ---->
        InsertReportSelection: Boolean;
        // <--- End new variables ----
    begin
        if TempNameValueBuffer.FindSet() then
            repeat
                AccountNo := CopyStr(TempNameValueBuffer.Name, 1, MaxStrLen(AccountNo));
                TempReportSelectionsAccount.Reset();
                TempReportSelectionsAccount.DeleteAll();
                SelectTempReportSelections(TempReportSelectionsAccount, AccountNo, WithCheck, ReportUsage, TableNo);
                if TempReportSelectionsAccount.FindSet() then
                    repeat
                        LastSequence := GetLastSequenceNo(TempReportSelections, ReportUsage);
                        // ---- Start new version ---->
                        InsertReportSelection := not HasReportWithUsage(TempReportSelections, ReportUsage, TempReportSelectionsAccount."Report ID");
                        OnBeforeCheckInsertReportSelectionToPrint(TempReportSelections, ReportUsage, TempReportSelectionsAccount, InsertReportSelection);
                        if InsertReportSelection then begin
                        // <---- End new version ----
                            TempReportSelections := TempReportSelectionsAccount;
                            if LastSequence = '' then
                                TempReportSelections.Sequence := '1'
                            else
                                TempReportSelections.Sequence := IncStr(LastSequence);
                            TempReportSelections.Insert();
                        end;
                    until TempReportSelectionsAccount.Next() = 0;
            until TempNameValueBuffer.Next() = 0;
    end;

...

[IntegrationEvent(false,false)]
local procedure OnBeforeCheckInsertReportSelectionToPrint(var TempReportSelections: Record "Report Selections" temporary; ReportUsage: Enum "Report Selection Usage"; TempReportSelectionsAccount: Record "Report Selections" temporary; var InsertReportSelection: Boolean)
begin
end;

Alternatively, you could directly modify the HasReportWithUsage method to handle the entire Report Selection record as a parameter (and not just the ID), adding an event before FindFirst(), allowing us to add our custom filters to the source record.
local procedure HasReportWithUsage(var TempReportSelectionsSource: Record "Report Selections" temporary; ReportUsage: Enum "Report Selection Usage"; TempReportSelectionsAccount: Record "Report Selections" temporary): Boolean
    var
        TempReportSelections: Record "Report Selections" temporary;
    begin
        TempReportSelections.Copy(TempReportSelectionsSource, true);
        TempReportSelections.SetRange(Usage, ReportUsage);
        TempReportSelections.SetRange("Report ID", ReportID);
        OnBeforeHasReportWithUsageFindReportSelections(TempReportSelectionsSource, ReportUsage, TempReportSelectionsAccount);
        exit(TempReportSelections.FindFirst());
    end;

...

[IntegrationEvent(false,false)]
local procedure OnBeforeHasReportWithUsageFindReportSelections(var TempReportSelectionsSource: Record "Report Selections" temporary; ReportUsage: Enum "Report Selection Usage"; TempReportSelectionsAccount: Record "Report Selections" temporary)
begin
end;

Additional context
A custom extension adds a report capable of printing documents in different ways, by modifying parameters in the "Report Selections" record linked to the document. At the moment, if the customer wants to print the document through the custom report in two different ways with our same custom report, the aforementioned standard logic prevents it from working.
Internal work item: AB#534602

Can you write some custom logic on top that calls this twice? If you need two prints?

Can you write some custom logic on top that calls this twice? If you need two prints?

Unfortunately, no... The call to this procedure is a consequence of the standard "Print" button present in all documents connected to the report selection.
Assuming it is possible to replicate the call to the print procedure, the standard report selection method would still return the first report both times, and not both of those set, with the consequence of printing the first allowed report twice.
Replicating the entire standard report selection block and custom adding a button for every single existing document (and non-existing ones, since our extension also allows the printing of custom documents with a setup), makes no sense.
The addition of an event in the report selection, which allows you to add custom logic to the decision on whether or not to use a report selection record in printing, would allow us to "apply an extra filter" to the record, and manage all the cases.

I try to add further context on our extension: we have added a Code field to the Report Selection record, the need would be to modify the grouping that the standard logic uses in printing (Report ID) and manage the pair (Report ID, custom Code field), so as to allow the printing of the same report in different report selection records, as long as our custom field is different from one of those already inserted.

You have multiple report selection records for the same report?
And the standard print always pick the first one, is that correctly understood?
Seems to me that we should look into supporting multiple report selections when printing, like your scenario.

You have multiple report selection records for the same report? And the standard print always pick the first one, is that correctly understood?
Seems to me that we should look into supporting multiple report selections when printing, like your scenario.

That's right, our documents might have multiple Report Selection records with the same Report ID, and we want to print both of them. At the moment, since you have implemented a logic whereby only one record per Report ID is added to the list of "records to print", this is not possible.
Since the report selection allows the insertion of multiple records for the same document, even if the Report ID is the same, yes: I was hoping that this type of functionality would be supported.

Super. I created an internal work item for us to track this effort. AB#551054
I want it to be build in the design rather than doing it as an event.
Will close this request as non-planned.


Indent when guard clauses on the same level as the definition

https://github.com/lexmag/elixir-style-guide#guard-clauses

I'll start working on this one today.

Great, thanks!


How to include Zip Code in the address label

Short description
I need to show zip code in the address label but it is not there by default and I also tried getSuggestLabel but that did not help either. How can I achieve the following?
Expected results
I would like to have label like this: 123 Main Street, San Francisco, CA 94001, United States

My suggestion would be to make sure that this is based on a prop called showZipInLabel

We won’t introduce a new prop, but there was renderSuggestItem added recently. Does this help?

My suggestion would be to make sure that this is based on a prop called showZipInLabel

I dont see showZipInLabel nor can ctrl + f find it....

@ro-ka i dont really even need it in my label, I just need to access the ipCode associated to the label... great tool by the way.


Properly hash PSO based on content, not based on its resource slots

          This does not point to the content but to the slot id of each resource within the PSO, each slot id is unique so if the same value comes out that means it will point to the same object if all of the slot ids are the same the PSO will also be the same. Hashes are already stored within a hash table within the dx renderer class.

Originally posted by @Dyronix in https://github.com/New-Creta/Rex/pull/233#discussion_r1437567472

fixed in development/gfx c2998eb3c70f513f47d1f8ef44dc9e20c49e2f62


Channel data error issue

I found that setting the Type and Sync of the channel to ChannelType.Master and ChannelSyncType.Time
The data of this channel will be incorrect.

Is there any way to solve this problem?

Thank you.


AUR

https://aur.archlinux.org/
Please, consider publishing to Arch User Repositories!

@omnigenous
Thank you for your comment! It may take some time, but I will consider it!


Ctrl+Shift+L keybinding conflicts

Hi there, there is a default for Ctrl+Shift+L to select all highlighted instances.
May you please change it to Ctrl+Alt+L. Otherwise I have to put two extra keybindings in custom. One to Restore Ctrl+Shift+L to { "key": "ctrl+shift+l", "command": "editor.action.selectHighlights", "when": "editorFocus" } and another to make this extension work: { "key": "ctrl+alt+l c", "command": "extension.gitHubUrlPerma", "when": "editorTextFocus" }

@Noitidart just keep in mind that typically it's a tough call to find a reasonable unused hotkey. And taking into account how easy it is to customize keys in vscode I would say it's not a big deal, especially given that selectHighlights is not sth as common as copy, change tab etc.

@Noitidart what do you suggest for a new default keybinding?
I'm probably going to leave the default keybindings as is for the reasons @mlewand suggested, and I don't want to break existing users of this extension.

I actually switched to mac. And am absolutely loving the default keybindings. The only issue is Ctrl + Left/Right (I am trying to figure out how to disable the desktop global hotkey so that vscode can use it)

Ah, excellent!
Not sure how to modify global hotkeys on mac, sorry :)


Grafana not showing data beyond 7 days

Hi Guys
I have this docker-grafana-graphite setup which has been working pretty well , however i have a bigger problem. My metric data can't live beyond 7 days. I can only see data of last 7 days.

I am aware of retention policy in graphite and i have tuned that as per my needs, but still graphite data is not getting stored for long term.
[root@monitoring-server ~]# docker exec -it 11e1290bd77c /bin/bash
root@11e1290bd77c:/# find . -name storage-schemas.conf
./opt/graphite/conf/storage-schemas.conf
root@11e1290bd77c:/# cat /opt/graphite/conf/storage-schemas.conf
[default]
pattern = .*
retentions = 10s:120d,60s:120d,120s:1095d
root@11e1290bd77c:/#

As you can see retention is properly set in the container graphite configuration file but still it cannot preserve data beyond 7 days.

@dpsoft @ivantopo any advice here ?  Thanks

Managed to get this fixed after changing retention to retentions = 10s:7d,1m:21d,10m:105d,60m:315d,6h:945d