Skip dropping the db in production env

Coverage remained the same at 96.031% when pulling 1404502fcb590ab517be41f24a807de60b3663cf on skip-dropping-db-if-prod into 6ef0e78d167eaf82d791648d72a21c7ce51abe9c on master.

Coverage remained the same at 96.031% when pulling 1404502fcb590ab517be41f24a807de60b3663cf on skip-dropping-db-if-prod into 6ef0e78d167eaf82d791648d72a21c7ce51abe9c on master.


#4 controllers

Dear Reviewer,

In this PR, I followed the instructions of Blog app - controllers.
By hand, I:

Added the routes seen in the wireframe
Created two controllers, with two actions each
Created a view file for each action

Hi @MattGomb,  Kindly add the key features and future features section of your app to your readme


Should test API responses don't include unexpected values

Right now we usually only test if the API response has some properties of some type. We do not test that there are no extra properties. IMHO we should also test that we don't have any extra properties on the return objects.

@vojtechsimetka as per our actions revision you mentioned that we might be already fulfilling this issue. Do you think we can close it?


Fix multipart parser issue

Wrong arguments passed to the functions setMark/clearMarkSymbol inside the _transform method of the class MultipartParser

No it's not. In line 119 there's a const called markSymbol which uses name.
Does yarn test:jest pass?

@CandyPopee, aaah yes, actually true. Thanks again! :)

Actually there is another issue with the setMark function : when idx = 0 (passed at line 126), it always set the i variable.
Should be replaced by :
const setMark = (name, idx) => { this[${name}Mark] = typeof idx === 'number' ? idx : i; };
Those two patches will solve errors while parsing buffer which results in corrupted upload file.

@CandyPopee thanks. Please add it and I can merge :)


.extend`` not working with namespace

Hey there,
I am not entirely sure if this is an issue for intended behaviour, so maybe someone can just explain?!
But I have the problem, that I have some styled components, which I am also extending and then using in a bigger application. That's why I wanted to use this plugin, to up the specificity of those components.
For example
export const Button = styled.button`
[...]
`;

export const ExtendedButton = Button.extend`
[...]
`;

The problem is that the extended styles don't seem to get the prefix like the main styles do:

Just for reference, this is the config for the plugin I am using: ["@quickbaseoss/babel-plugin-styled-components-css-namespace", { "cssNamespace": "folders" }]
Can anyone help?

My own little workaround was to write the extending components with a double self-reference:
export const ExtendedButton = Button.extend`
  && {
    [...]
   }
`;


Testing, please ignore. (skilnc)

This is a bug bounty test. Please ignore.. Please DO NOT APPROVE THIS! (skilnc)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.sababaegozim seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


chore: re-enable rate_limit module

#107  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Therefore, I am closing this PR.


Bulk Insert using PreparedStatement with an IDENTITY column raises exceptions

Driver version
12.1.0.jre11-preview
SQL Server version
Microsoft SQL Server 2019 (RTM-CU18) (KB5017593) - 15.0.4261.1 (X64)   Sep 12 2022 15:07:06   Copyright (C) 2019 Microsoft Corporation  Developer Edition (64-bit) on Windows 10 Pro 10.0  (Build 19045: )
Client Operating System
WINDOWS 10 Pro, Version 22H2, Build 19045.2364
JAVA/JVM version
JDK 19.0.1
Table schema
MS provided AdventureWorks database (but any database will do as will CREATE a new Table in Java source
Problem description
Referring to
https://learn.microsoft.com/en-us/sql/connect/jdbc/use-bulk-copy-api-batch-insert-operation?view=sql-server-ver16
I created a scratch Java program (attached, suffix renamed from .java to .txt) that
a) Connects to MS-SQL AdventureWorks
b) Creates a simple SELECT statement to check communicating with SQL Server ok (not needed to reproduce issue)
c) Create a new SQL table (Person.Test) with two columns, IDENTITY and VARCHAR
d) Attempt (by various means) to BULK insert a 1000 rows into the table (using MS Web page quoted above as a template)
e) Various exceptions raised depending on approach - listed below
Checked that Table is being Dropped and Created and Selectable by adding Break points to the source and querying the table in SSMS.

Leave the first column undefined (similar to if using a CSV leaving the first field empty - e.g. ",Text"). This is how the attached java source is setup. I would have expected, by not defining column 1 as it is an IDENTITY field that the driver would detect this and so auto-generate new identity integers for each row.

SQL Exception: com.microsoft.sqlserver.jdbc.SQLServerException: The value is not set for the parameter number 1.
Stack Trace
com.microsoft.sqlserver.jdbc.SQLServerException.makeFromDriverError(SQLServerException.java:237) com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.buildParamTypeDefinitions(SQLServerPreparedStatement.java:441)
com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.buildPreparedStrings(SQLServerPreparedStatement.java:394)  com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.doExecutePreparedStatementBatch(SQLServerPreparedStatement.java:2817)  com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement$PrepStmtBatchExecCmd.doExecute(SQLServerPreparedStatement.java:2774)
com.microsoft.sqlserver.jdbc.TDSCommand.execute(IOBuffer.java:7637)
com.microsoft.sqlserver.jdbc.SQLServerConnection.executeCommand(SQLServerConnection.java:4027)
com.microsoft.sqlserver.jdbc.SQLServerStatement.executeCommand(SQLServerStatement.java:270)
com.microsoft.sqlserver.jdbc.SQLServerStatement.executeStatement(SQLServerStatement.java:244)
com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.executeBatch(SQLServerPreparedStatement.java:2171)
Scratch.ConnectToSqlServer.main(ConnectToSqlServer.java:74)

Uncomment the lines marked [A] to provide the insert with specified IDENTITY values gives the following exception

SQL Exception: java.sql.BatchUpdateException: An explicit value for the identity column in table 'Person.Test' can only be specified when a column list is used and IDENTITY_INSERT is ON.
Stack Trace
com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.executeBatch(SQLServerPreparedStatement.java:2180)
Scratch.ConnectToSqlServer.main(ConnectToSqlServer.java:74)

Comment back the [A] and uncomment [B]  ("; useBulkCopyForBatchInsert=true") to request the use of Bulk Insert for performance gives the following exception

SQL Exception: java.sql.BatchUpdateException: Unable to retrieve data from the source.
Stack Trace
com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.executeBatch(SQLServerPreparedStatement.java:2138)
Scratch.ConnectToSqlServer.main(ConnectToSqlServer.java:74)

Finally, uncommenting out the 3 statements marked [A] gives the same exception as in (2) above

Expected behavior
The 1000 rows to be added to table Person.Test
Actual behavior
Exceptions reported as described above in points [1..4]
Error message/stack trace
Detailed above (and reproducible from the source file attached)
Any other details that can be helpful
None that I can think of
ConnectToSqlServer.txt
JDBC trace logs
Provide the JDBC driver trace logs.  Instructions can be found here: https://docs.microsoft.com/sql/connect/jdbc/tracing-driver-operation

Hi, we'll do some investigating on our end and get back to you. At the moment we're performing release tasks so answers may be delayed.

Following on from advice from those responsible for the documentation (see related GitHub issue [(https://github.com/MicrosoftDocs/sql-docs/issues/8478)]) I have managed to do a Bulk Insert using the same table and data using the SQLServerBulkCopy() class, although this is a lot more longwinded than using the useBulkCopyForBatchInsert=true option on the connection string. For comparison, I have also attached the two source files needed for this approach (again, having changed the suffix from .java to .txt).
ConnectToSqlServer.txt
SqlBulkRecord.txt

David,
Thanks for pointing this out. I would never have spotted that as my background thought process reminded me that all the columns must be declared for the Bulk Insert to work (e.g. when importing CSV files) and also when I implemented the ISQLServerBulkRecord interface.
I have since expanded the example to 4 columns in the SQL data table to get a better feel of the requirements when dealing with the MS-SQL server only version.
It took a couple of goes to get your suggestion to work as I first thought that the INSERT INTO only needed to declare enough ? for the columns I was importing (1 in my case). This didn't work as got the 'number of columns doesn't match' and so tried 3 ? (omitting just the identity column) and defining NULL values for the other two columns and this worked. In essence you have to pretend that the IDENTITY column doesn't exist and so all the other columns are then referenced from 1 (rather than 2 as would be the case for my example table attached above).
I have a GitHub issue for expanding the documentation to handle an IDENTITY column which should help others if they explore this route.
I have since implemented the MS-SQL API one and while it means that I have to reference the MS-SQL JDBC API rather than go through java.sql to keep a generic interface and the ISQLServerBulkData implements Serializable which caused me a few issues, I now have both working and so will look forward to comparing the performance of each.
One small query, if I could be indulgent regarding
Set	ISQLServerBulkRecord.getColumnOrdinals()
The documentation says that your implementation of the interface should "Get the ordinals for each of the columns represented in this data record."
I have set this to 1, 2, 3, .. n, where n is the number of columns in the table (including the IDENTITY column).
My question is - When would you not just return set(1, 2, 3, 4, ... n) as otherwise why not just supply the number of columns in the table? I must be missing what the true purpose of this method is.
Thanks so much for your time in resolving my query. Apart from an appreciated answer to the question above, this issue can be closed.

My question is - When would you not just return set(1, 2, 3, 4, ... n) as otherwise why not just supply the number of columns in the table? I must be missing what the true purpose of this method is.

There isn't any specific reason for that method that I'm aware of (this was before my time). To me, the method looks more like a potential for trouble if the implementation doesn't return an ordered set of ordinals if that is what is expected.


⚠ Samsung has degraded performance

In 533bd09, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 5354 ms

Resolved: Samsung performance has improved in 0947fe9.


Add GreedyMockAttribute (#766)

I prepared also abstract MockCustomizeAttribute for any further attributes (like for xUnit, NUnit etc)

I also have proposal to add similar "greedy" functionalities into other mocking projects like Rhino, FakeItEasy etc. Maybe it is not very important functionality, but then most projects will have similar functionalities.
Of course I can do that

@michalkowal Thanks a lot for giving your attention here! 🥇
I gave it a quick look and would like to share a few thoughts FWIW:

The following customization implementation might not work with the ConfigureMembers = true customization mode:return new ConstructorCustomization(parameter.ParameterType, new GreedyMockConstructorQuery());

If you wrap argument with GreedyMock attribute, the generated object will not be auto-configured.
There is the ongoing issue #1013 to refactor the way we activate mocks. That should be an enabler for this PR (as you want to customize the mock activation without touching the further configuration actions). I have that change in my TODO list and hope to take a look in a week or two.
If you are in progress, please ensure to add the [WIP] PR title prefix. This way I know that PR is not ready for review and you are still working on it 😉

I also have proposal to add similar "greedy" functionalities into other mocking projects like Rhino, FakeItEasy etc

I'd say that Rhino is completely dead, but for FakeItEasy and NSubstitute it might make sense to consider the feature. However again, that could happen after the restructure (#1013, #1014) only.

@zvirja thanks for your review.
Right now, as you suggest, I stop activity here and I will monitor status off this two issues


Adjusting line-height

Is there a way to adjust the calculated line-height for fluid font size?
I've tried additionally specifying ~leading-X/Y, but the line-height baked into the ~text-X/Y takes precedence. Adding the bang (~leading-X/Y!) also doesn't seem to be supported as tailwind breaks to compile.
P.S. Thanks for making this btw, it feels much nicer to use than some other fluid type alternatives.

Great catch! I just fixed this in v0.2.1. Also, it shouldn't be necessary any more, but you can mark fluid utilities important by adding the exclamation mark to the front, i.e. !~leading-4/6 (I might want to add this to the docs)


ADAP Aligner (GC) and Saving Project

Basic information

My operating system and version: Windows 10
My MZmine version: 3.2.48

What happened

Please describe what happened and how it differs from what you expected to happen.
After ADAP Aligner (GC) module, I can't save my project anymore. I ran again the workflow from the beginning saving and opening the project at each step and I had no problem before the Aligner.

Please upload screenshots of any errors and your log/mzmine.log file.

Please upload a standalone MZmine project file (or a batch + raw files) that demonstrates the problem you are reporting. You can use Google Drive, Dropbox, or a similar repository to upload the project.
A simple project with 2 analysis: https://mycore.core-cloud.net/index.php/s/6IX7fr9BcuSumBJ/download
Here is the batch xml with the module parameters: https://mycore.core-cloud.net/index.php/s/xctjhzOKTZ7Cpj7/download

If I can help for tests or further information, feel free to ask.
Have a good day!

hi @nic-Oban I think this issue has been addressed. Could you try our newest development release? https://github.com/mzmine/mzmine3/releases/tag/Development-release

Hello @tnguy271, @asmirn1,
Thanks a lot for modifications, Saving works fine now!
Have a good day.


Investigate if we can use e.g. a Node.js script to update mediaqueries.scss with all SCSS mediaquery mixins

It would be really cool if mediaqueries.scss would automatically update when a new file gets added.
We could e.g. have a Node.js script listen to file changes and when a file that has a mediaquery mixin (e.g. @mixin header-small) gets added/changed, mediaqueries.scss gets updated with all the new mediaquery mixins from all files that have them.
What do you think?

Fixed in https://github.com/doberman/dbrmn-scss-framework/pull/12


Ig conversion

Converting the repo to a FHIR IG

This is partially/manually merged into main. We're still deciding if we want to use FSH or JSON for the tx resources.


Relax test condition

I changed test condition to pass.

Why does directional differentiation require these relaxation, theoretically?

I'm not sure but in the previous implementation some gradients especially for array functions have the same direction to random vectors. I mean these functions returns an input vector itself and the previous implementation uses the input vector as a random vector. In the new implementation, they have different directions and it may causes calculation errors.

This PR will be replaced with #3652

OK.


Issue with creating project

Here's the error
TF200019: The following project already exists on the Azure DevOps Server: SmartHotel360. You cannot create a new project with the same name as an existing project. Provide a different name.

Please select different organization or provide different name to project and try again.


generated using svd2rust 0.11.1

Hi yuri91, I just updated your project with japaric's latest version of svd2rust. Regards, Christoph

Hi, thanks for the PR! I merged it and published on crates.io the new crate version


Add missing parenthesis in README example

Dang, that's embarrassing!  Thanks @poacher2k !! :D

You're live (and in the README) as of 2.0.3!  Now... if you REALLY want some credit, help me find ways to shave characters or condense the regex... I have a goal of < 400 bytes :)

Says 382 bytes here 🤷♂

Able to cut another 9 bytes by exporting the router directly as so:
module.exports = { Router: ... }


[WIP] New version with cleaner options

We've seen a lot of potential features for this where we need to handle forking paths gracefully. By default cookiecutter can't do this (see https://github.com/audreyr/cookiecutter/pull/848). It's been years, so we can't reasonably expect this to change upstream...
This implements a monkey-patching workaround to enable this behavior. It introduces a couple of major changes, so here are my recommendations.
Here are the big differences for a user:

we now need to run ccds <path to repo> instead of cookiecutter <path to repo>
the options and their defaults are changing
as a consequence of supporting more environments/dependency managers at setup time, we stop supporting them at execution time. this means that teams will have to pick one of each and stick to it across developers. I think this is pretty widespread already. (i.e. make create_environment will only support one of the options rather than multiple like it does now)

Implementation details are:

Add setup.py to make this package installable and give it a CLI
add monkey_patch.py to patch the cookiecutter functions that we need to handle our use case
support a list of dictionaries in cookiecutter.json that let's a use pick an option and then sub-options
use post_gen_project.py to create the environment file based on a standard list of libraries. (#5)

There is also work for a number of longstanding items in this branch as well:

we now support multiple storage providers (#120)
we allow user to choose package manager (#118)
we allow user to choose a dependency format
src is now {{ cookiecutter.module_name }} (#140)
remove test_environment.py which is just cruft IMO

Recommended path forward:

[x] tag current master as v0.1 so anyone relying on the current flow/structure can continue to use it easily
[ ] add documentation with new installation (pip install cookiecutter-data-science) and new initiation (ccds <path to repo>)
[x] implement the rails for the high priority items below
[x] make comprehensive tests run on CI/CD to de-risk
[ ] add table of options with links to the project documentation in our docs
[ ] update screencast and add screenshots ( #197 )

I think this also unblocks a whole suite of items that may require choice-specific configuration:

choose and configure DAG tool (#156)
choose docs package and integration (#7)
VCS choice and integration (#56)
data version controls (dvc or mlflow) integration
experiment management integration
docker support (#62)

I would suggest changing cookiecutter.module_name to cookiecutter.package_name. While it's technically true that all Python packages are modules, when arranged into a collection of other modules and having a __init__.py, they then also become a package, and this is more salient from the end user perspective. For example, the status as package informs the user that it can be pip installed.

I would suggest changing cookiecutter.module_name to cookiecutter.package_name. While it's technically true that all Python packages are modules, when arranged into a collection of other modules and having a __init__.py, they then also become a package, and this is more salient from the end user perspective. For example, the status as package informs the user that it can be pip installed.

module_name is correct. For example, "sklearn" is the module name for the package named "scikit-learn".

module_name is correct. For example, "sklearn" is the module name for the package named "scikit-learn".

Ah yes. Good point. Just found it a little counter-intuitive at first.

Are there any updates regarding this pull request? The master branch is quite outdated and I think this pull request adds a lot of value and addresses many open issues.

Or perhaps you can expect this to change upstream: New maintainier. Cookiecutter 2.0 coming. https://github.com/cookiecutter/cookiecutter/issues/1256

I've got a decent head of steam on a comprehensive set of tests we needed anyway for this change (and for the project generally). I'll keep going to get all the environment options tested on all platforms and then we can assess what to look to cookiecutter for and what to do ourselves.
The biggest decision is whether this becomes a tool where you have to do ccds .... or cookiecutter .... I guess, once this is all working, we can always integrate upstream cookiecutter functionality and remove the monkey_patch. However, I wouldn't want to go back to cookiecutter ... being the entrypoint since that is annoying churn in the command that you run and the packages you need installed to create environments.

Moving this work on to an official v2 branch!


Merge main into sparsehistogram, now for real

This is only going into the sparsehistogram branch, only interesting for @codesome.

The DCO alert seems spurious. It complains about different ways of writing the name of the author.

The DCO alert seems spurious. It complains about different ways of writing the name of the author.

I manually set it to pass now


Add color support

Wouldn't it be cool if we could print to the terminal with color? Credit
to @mzfr for the idea!
Now we can. This change adds a new property to Figures: color. And a
new initializer to create a figure with color: NewColorFigure.
When the color is valid, it will print in that color. This includes
Blink, Scroll, and Dance, in addition to plain old Print.

@common-nighthawk Is there an ETA on when this feature will be available?

All set!  Sorry--forgot to hit merge earlier.  I'll update docs in a separate PR.


Could a new version be released?

Hi, I was wondering if a new version of cffconvert could be released or maybe it doesn't have to be. Version 2.0.0 is 2.5 years old.

For example, version 2.0.0 incorrectly converts to schema.org the type.  I have type: dataset in my CITATION.cff file, and get "@type": "SoftwareSourceCode" in the output.

The last released version doesn't have a pre-commit-hooks.yaml file in it, too! It would be nice to be able to rely on pre-commit autoupdate & pre-commit.ci's auto update feature, which is best with a tag.


Summary of Key Findings 3

Review a portion of the Key findings for the Github repo

Closing this as your branch has been moved into Master already.


Connection error from keda-operator-metrics-apiserver logs

Report
After updating KEDA to version 2.9.2 (via chart version 2.9.3) from version 2.8.1 the metrics apiserver logs shows connection issue from the keda operator service at port 9666
Expected Behavior
The keda metrics apiserver to be able to use gRPC connection to successfully get metrics from operator
Actual Behavior
When I deployed the latest KEDA version 2.9.2 started seeing the below error in the metrics apiserver logs,
I0201 19:41:19.967541       1 main.go:162] keda_metrics_adapter "msg"="Connecting Metrics Service gRPC client to the server" "address"="keda-operator.mynamespace.svc.cluster.local:9666"
Err: connection error: desc = "transport: Error while dialing failed to do connect handshake, response: "HTTP/1.1 502 Bad Gateway\r\n
Steps to Reproduce the Problem
Update KEDA version from 2.8.1 to 2.9.2
Logs from KEDA operator
I0201 19:41:19.967541       1 main.go:162] keda_metrics_adapter "msg"="Connecting Metrics Service gRPC client to the server" "address"="keda-operator.<MYNAMESPACE>.svc.cluster.local:9666"

Err: connection error: desc = "transport: Error while dialing failed to do connect handshake, response: \"HTTP/1.1 502 Bad Gateway\\r\\n

KEDA Version
2.9.2
Kubernetes Version
1.23
Platform
Amazon Web Services
Scaler Details
No response
Anything else?
The keda service (13-keda-service.yaml) was introduced with the latest version referring to this issue https://github.com/kedacore/keda/issues/3920 as this service wasn't there in the older version
I can try KEDA_USE_METRICS_SERVICE_GRPC=false but as it will be deprecated, wanted to see how to resolve the issue
KEDA runs behind enterprise proxy and the values are set via HTTP_PROXY HTTPS_PROXY and NO_PROXY

Hello,
I can see a typo in the url, the message says keda-operator..svc.cluster.local:9666 but it should be keda-operator.KEDA_NAMESPACE.svc.cluster.local:9666.
The value is set here https://github.com/kedacore/charts/blob/v2.9.3/keda/templates/22-metrics-deployment.yaml#L109
Could you check if that value is present? I think that the message you see is because there are missing configs after the update. If that line isn't provided, KEDA tries to generate it in fallback but KEDA namespace is required, and that variables is also missing

Hello, I can see a typo in the url, the message says keda-operator..svc.cluster.local:9666 but it should be keda-operator.KEDA_NAMESPACE.svc.cluster.local:9666. The value is set here https://github.com/kedacore/charts/blob/v2.9.3/keda/templates/22-metrics-deployment.yaml#L109 Could you check if that value is present? I think that the message you see is because there are missing configs after the update. If that line isn't provided, KEDA tries to generate it in fallback but KEDA namespace is required, and that variables is also missing

@JorTurFer its just a typo while I wrote it, it actually has the namespace name there ""address"="keda-operator.mynamespace.svc.cluster.local:9666""

@zroubalik , Maybe we should exclude this communication from the proxied traffic, as it's internal traffic. WDYT?

After adding cluster.local to NO_PROXY list, the 502 bad gateway error is resolved, however still seeing the below error in the metric server logs,
Err: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:9666: connect: connection refused"

After adding cluster.local to NO_PROXY list, the 502 bad gateway error is resolved, however still seeing the below error in the metric server logs,
grpc: addrConn.createTransport failed to connect to "Addr": "keda-operator.mynamespace.svc.cluster.local:9666"

Sorry, I missed your reply 😢
Is the error persistent? I ask because we print the failure message but we don't print the established connection message, if you see the error only once or twice, it's not a problem and it's normal.

Has this been resolved ?
After upgrading keda from 2.6.1 to 2.10.1 autoscaling is not working and seeing this error in apiserver log:
Err: connection error: desc = "transport: Error while dialing: dial tcp: lookup keda-sa.keda.svc.cluster.local on <IP_ADDRESS>:53: no such host"
1 client.go:93] keda_metrics_adapter/provider "msg"="Waiting for establishing a gRPC connection to KEDA Metrics Server"
1 provider.go:110] keda_metrics_adapter/provider "msg"="timeout" "error"="timeout while waiting to establish gRPC connection to KEDA Metrics Service server" "server"="keda-sa.keda.svc.cluster.local:9666"

Has this been resolved ?
It is, yep.

Seeing that error during the startup could be normal if the operator is down, but it should be transitory.
Is your operator working?

I'm new to Keda.  How can I confirm the operator is working correctly ?

No configuration changes whatsoever.
I think the url configured in metrics apiserver deployment should be the operator internal endpoint:  keda-sa.keda:9666 instead of keda-sa.keda.svc.cluster.local:9666.
keda-sa is the operator service name.

After changing the apiserver deployment metrics-service-address image arg sill getting the same error:
lookup keda-sa.keda.svc on <IP_ADDRESS>:53: no such host
Wondering if the --metrics-service-address argument format is wrong.   It is:  --metrics-service-address=keda-sa.keda.svc:9666'
which translates to:  ..svc:9666

From operator logs and from testing keda by sending SQS messages keda is scaling out successfully.
2023-06-15T17:10:16Z INFO scaleexecutor Successfully updated ScaleTarget {"scaledobject.Name": "aws-sqs-queue-scaledobject-ds", "scaledObject.Namespace": "test-namespace", "scaleTarget.Name": "ds", "Original Replicas Count": 0, "New Replicas Count": 1}
Just trying to understand the root cause of the timeout errors and its potential impact on production deployments.

Imho there should be keda-operator instead of keda-sa - basically a name of K8s Service that fronts the operator deployment.

keda-sa is the operator service name

I missed this note.
So you did some changes because the original name is keda-operator

From operator logs and from testing keda by sending SQS messages keda is scaling out successfully.
2023-06-15T17:10:16Z INFO scaleexecutor Successfully updated ScaleTarget {"scaledobject.Name": "aws-sqs-queue-scaledobject-ds", "scaledObject.Namespace": "test-namespace", "scaleTarget.Name": "ds", "Original Replicas Count": 0, "New Replicas Count": 1}
Just trying to understand the root cause of the timeout errors and its potential impact on production deployments.

It won't scale to more than 1 replica

@zroubalik
Spot on.  I just realized, that's exactly the behavior, keda won't scale to more than 1 replica.
Why is that ?

keda-sa is the operator service name

I missed this note.
So you did some changes because the original name is keda-operator

I tried patching metrics-apiserver deployment by changing metric-service-address to the operator service endpoint which is
keda-sa.keda:9666 but to no avail.

Well, in first place we should ask why is the operator service name changed?

@zroubalik Also, can you expand on why keda "won't scale to more than 1 replica" ?

Does someone have a solution?
I facing the same issue ....


Enhance IcebergDataset to detect when files already at dest then proceed with only delta

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

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

https://issues.apache.org/jira/browse/GOBBLIN-1707

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Enhance IcebergDataset to detect when files already at dest then proceed with only delta
Tests

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

unit tests included
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"

Codecov Report

Merging #3575 (499f47f) into master (71da34b) will decrease coverage by 3.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3575      +/-   ##
============================================
- Coverage     46.90%   43.90%   -3.01%
+ Complexity    10610     2057    -8553
============================================
  Files          2111      406    -1705
  Lines         82533    17545   -64988
  Branches       9178     2143    -7035
============================================
- Hits          38714     7703   -31011
+ Misses        40261     8986   -31275
+ Partials       3558      856    -2702

Impacted Files
Coverage Δ

...lin/util/filesystem/FileSystemInstrumentation.java
92.85% <0.00%> (-7.15%)
:arrow_down:

...n/data/management/copy/iceberg/IcebergDataset.java

...apache/gobblin/metrics/ConsoleReporterFactory.java

...blin/runtime/scheduler/QuartzJobSpecScheduler.java

...source/extractor/filebased/FileBasedExtractor.java

...he/gobblin/runtime/JobExecutionEventSubmitter.java

...ain/java/org/apache/gobblin/hive/HiveRegProps.java

...obblin/util/limiter/RestliServiceBasedLimiter.java

...blin/source/extractor/resultset/RecordSetList.java

...source/extractor/filebased/GZIPFileDownloader.java

... and 1698 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

it seems as long as there is one new snapshot generate on source, we will go through all the data files available on source to do copy even there is only one new file added.

actually we'll always go through the complete metadata on source to list every file reachable from at least one snapshot.  it's true we do that even when there's not any 'new', unreplicated snapshot.  actual copy however only happens for files that are not present on the destination.  further, we need not examine every file, to determine whether it exists on dest.  rather, thanks to the immutability of iceberg files, we may short-circuit evaluation of an entire subtree of the iceberg metadata, when the root (e.g. manifest-list or manifest) is found already to exist at dest.  for details, I've added the comment // ALGO: in IcebergDataset.getFilePathsToFileStatus()

How do we plan to handle the file deletion on source? i.e. expire snapshot operation?

good question!  the answer is: distcp is not responsible.  instead we will expect reachability analysis and orphan file deletion to happen elsewhere.  a good candidate would be the destination catalog we'll eventually register the copied 'metadata.json' file with.  e.g. that catalog would hold the metadata version prior to the registration and could easily determine which snapshots 'expire' from the act of replacing the older metadata file with the newer one (replication has copied from source)


[App Service] BREAKING CHANGE: Redact tokens output on deployment source update-token

Related command
az functionapp deployment source update-token
az webapp deployment source update-token
Description
Currently, these two commands return the tokens as part of the output. In this PR, we are redacting the tokens so prevent accidental security leaks through CI logs.
Testing Guide

Create functionapp (az functionapp create)
Update token (az functionapp deployment source update-token)

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

App Service


Who will be the maintainer?

I am willing to maintain the repo

Can you ask the prof. to assign you as the maintainer

Done


fix: slider thumb inner background issue, fixes #1133

This fixes any issue where the thumb of a slider is transparent when it should be opaque.
Description

Fixes #1133
Changed background for spectrum-Slider-handle class.
How and where has this been tested?

How this was tested:
Tested by building locally and navigating to http://localhost:3000/docs/slider.html#slider and viewing behavior of thumb.
Browser(s) and OS(s) this was tested with:
Chrome 89 - Mac 10.15

Screenshots

https://user-images.githubusercontent.com/25776618/111511276-f4c42600-8724-11eb-9385-f1f942ffed53.mov
https://user-images.githubusercontent.com/25776618/111511290-f857ad00-8724-11eb-8b39-4013bfa0cc78.mov
To-do list

[X] If my change impacts other components, I have tested to make sure they don't break.
[X] If my change impacts documentation, I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

[X] This pull request is ready to merge.

I'm getting this error from Travis that I can't find a solution for.

I restarted the job.

I suspect PR #1547 will end up superseding this so I've highlighted this PR there to ensure changes address this concern.

I suspect PR #1547 will end up superseding this so I've highlighted this PR there to ensure changes address this concern.

I suspect PR #1547 will end up superseding this so I've highlighted this PR there to ensure changes address this concern.

@castastrophe I pinged PJ to weight in as we currently don't have this documented in the Tokens package nor the Tokens Design Documentation. (Ramp and Ticks is not in there yet).

@BaileyMcKelway is correct here, we should ensure in these 2 cases for Ramp and Tick that the Ticks and Slider Ramp is not visible.
I will make the updates to #1547

For Range, I already took care of this in PR #1547

I added this to PR #1547

Closing this PR out so we can address it in our tokens migration work but super appreciate the work done here to help us find a solution.


Add CodeQL security scanning

Hi, I'm a PM on the GitHub security team. This repository is eligible to try the new GitHub Advanced Security code scanning beta.
Code scanning runs a static analysis tool called CodeQL which scans your code at build time to find any potential security issues. We've tuned the set of queries to be only the most severe, most precise issues. We'll show alerts in the security tab, and we'll show alerts for any net new vulnerabilities on pull requests as well. We've tried to make this super developer friendly, but we'd love your feedback as we work through the beta.
If you're interested in trying it out, you can merge this pull request to set up the Actions workflow.

/assign @viglesiasce
The build is working on my fork, so this should be good to go if you're happy to merge it.


Remove version: ImageLine.FLStudio version <IP_ADDRESS>39

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://install.image-line.com/flstudio/flstudio_win64_<IP_ADDRESS>39.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 897)


backport-2.1: paralellize queue uses distsql; add '2.0' setting for distsql

PR: https://github.com/cockroachdb/cockroach/pull/30209
From release notes:

Added the 2.0 value for both the distsql session setting and the sql.defaults.distsql cluster setting, which instructs the database to use the 2.0 auto behavior for determining whether queries are distributed or run through the gateway node. [#30209][#30209] {% comment %}doc{% endcomment %}

Discussed this with @jordanlewis last week and decided that we wouldn't document the 2.0 values since they are mainly for internal use and will be eventually deprecated. Closing the issue!


FreeLightning.com Now Supported

I have added SupercellWx to my firewall as an Allowed Software.  Thank you very much for creating this project.  It's amazing!  From a fellow C coder, please let me know if you need any help or anything at all.  -Derek @ https://www.freelightning.com

Thank you for adding it!!  I tested Free Lightning a little bit when I first added placefiles to Supercell Wx, and it was great.  I look forward to continuing, and I'm sure the community does as well!
I have a link under Placefile Resources to your site: https://supercell-wx.readthedocs.io/en/stable/user-guide/placefile-manager.html#free

Thank you, sir!  I’m really impressed with your coding and the scWX application. It runs great and I’d love to contribute some features to your application via GitHub. BTW, I have the GLM lightning coding complete and it’s updated every 20 seconds and has great coverage. Built in lightning layers to scWX would be great. If you’d like the code just let me know and I’ll send it our way. You can either host it on your website or have it built in, like WSV3.  Thanks again for your hard work & making this open source! Much respect.


Make AMD and commonjs consistant

Coverage decreased (-0.8%) to 91.928% when pulling ef236c6983904359d5617f7df7d1523334e68059 on graingert:patch-1 into 7798910c528d910fc431705c88919355de066300 on urish:master.

Thanks!
I'd go with a different approach, where the angularMoment function itself will return the module name, and then both AMD and CommonJS will just use its return value directly. Thoughts?

Fixed in commit  919882e


Let extractor and profile configuration to be managed from "create/edit pipeline" modal in UI

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2298
Originally created by @DouweM on 2020-08-21 18:08:49

We could allow a configuration profile (https://gitlab.com/groups/meltano/-/epics/51) to be explicitly selected, and/or we could default to one named after the pipeline.

View 1 previous comment from the original issue on GitLab


Typing a keyword as an identifier should be handled as a compile error

I have now taken this out of the parsing.
If you try to define an identifier using a keyword you should get a compile error e.g:
 identifier may not be a reserved keyword
If you use a keyword as if it were an identifier in an expression, you will anyway get a message that the identifier is unknown. But it might be better to intercept that and check that if the unknown identifier is a keyword, in which case the error is a red-error saying e.g.
keyword 'as' may not be used as an identifier

Not yet complete as this needs to be applied to:

var name declaration (DONE)
function name declaration (global and method)
procedure name declaration (global and method)
parameter name declaration
constant name declaration
enum value declaration
property name declaration

? Have I missed anything ?


You did a great job. Give you some praise. I encountered a serious problem during the test

You did a great job. Give you some praise. I encountered a serious problem during the test, that is, using onpanupdate to rotate doesn't work. The initial rotation is always the opposite, which is too rebellious. In addition, the X in the upper right corner, I think the user should close the operation object immediately when clicking it. At present, it needs to be operated twice

Thank you for your valuable feedback it will resolved soon in next update.
On Fri, 10 Dec, 2021, 11:14 AM dbb987, @.***> wrote:

You did a great job. Give you some praise. I encountered a serious problem
during the test, that is, using onpanupdate to rotate doesn't work. The
initial rotation is always the opposite, which is too rebellious. In
addition, the X in the upper right corner, I think the user should close
the operation object immediately when clicking it. At present, it needs to
be operated twice
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Harsh-Radadiya/sticker_editor/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQWI6D3YG7Z4CSQ24KYR6VDUQGHVTANCNFSM5JYJQQ2Q
.
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.


audio: Remove Close functions from decoders and players

Removing Close makes it more explicit who should call the source's Close. Decoding will no longer require io.Closer

This also can remove ByteReadSeekCloser


(#548) Change typedef of bool from int32 to 1-byte 'unsigned char'.

In SplinterDB's public splinterdb_config{} config, we have few fields defined as 'bool' which is typedef'ed to int32 on our side. This creates compatibility problems when linking this library with other s/w which may have defined 'bool' as 1-byte field. (Offsets of fields in the splinterdb_config{} struct following 1st field defined as 'bool' changes across objects. This commit slightly adjusts our definition to default to 1-byte 'unsigned char', which should reduce the risk of such incompatibilities.
Found only one instance where 'bool' return value from routing_filter_is_value_found() had to be adjusted to fit 1-byte return value.

@rtjohnso - Triggered by the fix I discovered as being necessary in src/routing_filter.h, I am doing a code walk thru myself, cross-checking all functions that are defined to return bool, to see if other similar issues are lurking elsewhere.
I am not finding any ... hopefully, this fix in src/routing_filter.h is the ONLY place where code needs to be adjusted.

@rtjohnso - I completed my code-walk through for all instances of bool in the code. I found a few straggling places where code was using hard-coded 0 or 1 for boolean variables.
None of this changes the logic / semantics, but will be good to make sure I get one more pair of eyes over it (lest, I'm making some silly mistakes).

HI, @rtjohnso - I reworked this fix as per what we discussed this past Tue's PR meeting. Now, in public config struct all boolean fields are defined as _Bool.
The rest of the change set stays as it was before, including minor cleanup for (what was previously) one instance of a bug, and other cleanup of hard-coded uses of 0 /1 with FALSE / TRUE.

@rtjohnso - It's not straightforward w/o too much work to address your suggestion of : "Don't forget to move the definition of bool from the public platform to the private platform."
There are some cascading dependencies which cause compilation problems.
For instance, in include/splinterdb/public_platform.h we explicitly document a dependency that:
 14 The external definitions include:
 15 - uint64, which is used for buffer sizes, etc.
 16 - bool, typedef'd to int32 on linux

I had tried to move this typedef to src/platform_linux/platform.h and this immediately results in a compilation failure:
Compiling            src/btree.c                                        [build/release/obj/src/btree.o]
In file included from include/splinterdb/data.h:23,
                 from src/btree_private.h:14,
                 from src/btree.c:4:
include/splinterdb/public_util.h:30:15: error: unknown type name ‘bool’
   30 | static inline bool
      |               ^~~~

This is (partly) why in my original version of the fix, I had retained the location of typedef bool to be in include/splinterdb/platform_linux/public_platform.h but changed it to unsigned char.
Unless I'm missing something, I suspect to implement this new approach (of using _Bool in extern headers, and to use bool as int32 for internal usages) will need some .h file reorganization.
I'm not sure it's a good idea to take that on right now. It should be a separate item.

If we leave our definition of bool in the public header, then won't you
still have your postgres problem?
Just change the static inline bool functions in public headers to use _Bool
instead.
Best,
Rob
On Fri, Mar 24, 2023 at 5:18 PM gapisback @.***> wrote:

@rtjohnso https://github.com/rtjohnso - It's not straightforward w/o
too much work to address your suggestion of : "Don't forget to move the
definition of bool from the public platform to the private platform."
There are some cascading dependencies which cause compilation problems.
For instance, in include/splinterdb/public_platform.h we explicitly
document a dependency that:
14 The external definitions include:
15 - uint64, which is used for buffer sizes, etc.
16 - bool, typedef'd to int32 on linux
I had tried to move this typedef to src/platform_linux/platform.h and
this immediately results in a compilation failure:
Compiling            src/btree.c                                        [build/release/obj/src/btree.o]
In file included from include/splinterdb/data.h:23,
from src/btree_private.h:14,
from src/btree.c:4:
include/splinterdb/public_util.h:30:15: error: unknown type name ‘bool’
30 | static inline bool
|               ^~~~
This is (partly) why in my original version of the fix, I had retained the
location of typedef bool to be in
include/splinterdb/platform_linux/public_platform.h but changed it to unsigned
char.
Unless I'm missing something, I suspect to implement this new approach (of
using _Bool in extern headers, and to use bool as int32 for internal
usages) will need some .h file reorganization.
I'm not sure it's a good idea to take that on right now. It should be a
separate item.
—
Reply to this email directly, view it on GitHub
https://github.com/vmware/splinterdb/pull/549#issuecomment-1483623740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA45FROW2WPUH2SVAF7ZEGTW5Y2T7ANCNFSM6AAAAAAWBW6VDU
.
You are receiving this because you were mentioned.Message ID:
@.***>

If we leave our definition of bool in the public header, we won't have the field-offset mismatch when integrating with Postgres because the boolean fields in the public splinterdb_config{} have been all changed to _Bool.
So irrespective of whether struct is compiled with our library of PG-glue-code, we should see the same field definition.

@rtjohnso - I amended the code to use _Bool for few methods whose protos are in public_util.h . This allows me relocate typedef bool int32 to platform.h and to go through compilation.
(Removed the newly added test-case.)
Let me know if you find this new version of the fix acceptable.


prev-track directive being called twice!!

Recently, I have met a strange thing.
When I click the next button with next-track directive, the step is 1, namely id + 1,
but when I click the button with prev-track directive, the step is 2, namely id - 2.
I have used console to debug. I have sawn the prevTrack() was called twice. It seems that the prevTrack() function was called by angularJS additionally.
I have checked your demo that works fine.
Has anyone met this? I will update this if I find more info about this issue.

After packing through Gulp, this issue seems fixed by itself.


How to iterate EntityType properties?

Is there a way how to iterate over properties of particular EntityType.
I can iterate over EntitySets like this:
var metadata = (EdmModelBase)Client.GetMetadataAsync().Result;
var entitySets = metadata.EntityContainer.EntitySets().ToArray();

However there is no way how to iterate over list of properties. The only way I found is blocked since Microsoft.OData.Edm.Csdl.CsdlSemantics.CsdlSemanticsEntityContainer is internal.
Is there any other way?
Thanks

The EdmStructuredType has DeclaredProperties. Those are available in SchemaElements. I think you need to iterate over SchemaElements which provides items of IEdmStructuredType type. This way you can get declared properties for each IEdmStructuredType type.
var metadata = (await client.GetMetadataAsync()) as EdmModelBase;
var schemaElements = metadata.SchemaElements;

foreach (IEdmSchemaElement schemaElement in schemaElements)
{
    if (schemaElement is IEdmStructuredType)
    {
        IEdmStructuredType element = (IEdmStructuredType)schemaElement;

        foreach (IEdmProperty property in element.DeclaredProperties)
        {
            Console.WriteLine(property.Name);
        }
    }
}


Minor changes to FAQ.md

Minor improvements to the phasingout de2.
Additionally, the .md linter/autoformatter might made some minor
adjustments to formatting.

sorry took me a while but incorporated both suggestions :)


BasicRenderManager: Stop relying on sRGB framebuffers

Right now the BasicRenderManager relies on an sRGB framebuffer, which requires special setup before ShowBase is created. The processor_app can guarantee an sRGB framebuffer, but things get less reliable when the BasicRenderManager is dropped into end-user applications. Instead it would be preferable to handle any necessary setup in BasicRenderManager.__init__().
Switching to a regular color buffer and using render-to-texture with a color correction shader should allow us to switch sRGB on/off at runtime after a window/framebuffer has already been created.

Implemented in 5e44d3771fcd56a2d3a1926a67a1e3f65e6a8ae6


Web component: Tabs extended &  Tabs extended with media - Keyboard tab focus issues. - DDS consulting

Detailed description
Issue:
For keyboard only users
1.Each tabs are receiving the tab focus.
2.We are not able to navigate between the tabs with right/left arrow keys.
Expected:
The only selected tab should receive the tab focus and rest of the tabs should be navigable with right/left arrow keys.
Additional information

@Praveen-sr Is the left and right arrow support required for IBM accessibility standards?  Will this support also be required for Masthead navigation?

@RobertaJHahn Yes its a IBM and WCAG accessibility requirement.
The arrow key behavior only required wherever we have Tabs, Therefore its required for Tabs extended & Tabs extended but not for Masthead.
Masthead is having menu options not tabs.
Here is the link for your reference:- https://www.w3.org/TR/wai-aria-practices-1.1/examples/tabs/tabs-1/tabs.html


Increment quasar version to v53.0.0

Will get us in line with https://github.com/slamdata/slamdata-backend/pull/430

@alissapajer Wasn't there an automated PR which did this?

@djspiewak the automated PRs don't work because of the license/cla.

@alissapajer What do you mean they don't work? They can still be merged.

@alissapajer
I'll check this soon since I'll need to do some manual testing before merging this. Related issue:
https://app.clubhouse.io/data/story/565/verify-quasar-s3-loads-into-slamdata-backend-in-the-travis-build


Does not work on macOS

Today I found this extension and that was inspiring due to exceptions, I already using Swagger Viewer. But unfortunately it works only on linux, which would be fine, but I use macOS mainly. The only thing I see when I open a viewer is black message:
Cannot GET /file//Users/.../asyncapi.yml

I've tried to play around, but did not find quick solution.

Hi @seitbekir-seidametov-codeit
I don't have a macOS to test...
Maybe you can help me track this..
Can you try setting this config asyncapiPreview.previewInBrowser to true, so you can open preview in chrome
Inside chrome devtools console try this javascript window.location.pathname.replace(/\/file\//g, '');
I'm guessing that in macOS is not doing the replacement of /file/ prefix
Thanks and sorry for the delay answering

Using Arch Linux, it's also not doing the replacement of /file/ prefix
Cannot GET /file//home/.../tabata.yaml

Using Arch Linux, it's also not doing the replacement of /file/ prefix
Cannot GET /file//home/.../tabata.yaml

Hi @nealdam and @seitbekir-seidametov-codeit
I have uploaded version 0.0.3 to the marketplace which fixes this issue
Sorry for the delay fixing this

Hi @nealdam and @seitbekir-seidametov-codeit
I have uploaded version 0.0.3 to the marketplace which fixes this issue
Sorry for the delay fixing this


Revert "Enable new ServiceMonitor"

Reverts vespa-engine/vespa#3607
@hmusum @hakonhall please ack
this broke many system tests

ack


Revert #1 and revamp the script.

The portability offers few benefits when the script is hideous, with a messy method for parsing.
Shift back to bash for using arrays and bitwise operations rather than hacky iterations.

There is support for bitwise operations in dash, so perhaps a cleanup attempt would be preferable.


Remove ticks from sliders

Removing ticks until we have designs and behaviors from spectrum
Closes https://github.com/adobe/react-spectrum/issues/1258
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Can you also remove the API for it from the types?


Consistency rules for missing cardinalities are too lenient

The consistency rules resource_prop_cardinality_any and value_prop_cardinality_any will overlook a missing cardinality in the following case:

Value V is of type T, which has a cardinality for datatype property P1, but no cardinality for datatype property P2.
V has P1 with object "foo", and P2 with object "foo".

The cardinality for P1 will match the rule's pattern for verifying P2, so no violation will be found.
The rule is written this way to allow the case where T has a cardinality for a superproperty of P1, rather than a cardinality for P1 itself.
Consider disallowing this case and making the rule more stringent to avoid this problem.

This issue has been inactive for a significant time, so we’re closing it for now. If it’s still relevant, feel free to reopen.


Tentatively fixes #431

Generates a Project_name for control prep files, based on Sample_Plate.
The trailing plate number information is removed from the string before
assignment.

@AmandaBirmingham mentioned this, and I agree, that a fix for #483 might involve changes that affect resolving #431 for the better. I'm going to close out this PR without merging.


Recurring Bills

[x] Crud

IN CREATION SCREEN

[x] All changes in Bills creation screen also applicable in recurring Bills. (landed cost, add items in bulk etc...)
[x] Give default drop down options for 'repeat every' column and give custom option or keep option for 'create new repeat every' options.
[x] Check duplication in profile name.
[x] Remove entity type columns.
[x] Price list in above item table to apply all line of item.
[x] Never expires tick box (set default)
[x] Remove terms and conditions columns and attach files feature.
[x] Remove source of supply, destination of supply and purchase person column.
[x] Discount type selection in transaction type not properly working.
[x] Submit button rename to save.

IN DETAILED SCREEN

[x] Overview, next bill and recent transactions in detailed view.
[x] In active: stop, delete, create bill and edit options.
[x] In stopped: resume, delete, create bills and edit.
[x] In expired: delete, create bills and edit.
[x] Filter options in listing view of detailed screen. In sorting add vendor name and remove recurring purchase invoice, date, name and payment mode. rename total price to amount in sorting.

IN LISTING SCREEN

[x] Filters in listing screen.

Completed


How to set userAgent of a page ?

I want to set mobile useragent to a specific page , so far
await searchPage.setUserAgent(userAgent);
and
searchpage.emulate(device) doesn't seem to work . How can i do this ?

Changing only user agent is very dangerous and can be detected. You need to change fingerprint instead, it changes lots of properties including user agent.

I'm actually okay if website detect any abnormaly by doing this , I want to set or emulate mobile useraAgent to a specific page for some seconds and then continue with the Default userAgent defined in my fingerprint for further tasks.
Both setUseragent and page.emulate() Doesn't seem to work.
Is there any way to archive this ? changing the fingerprint to mobile and launching another instance will not be suitable for my tasks.

Setting user agent through CDP will be overridden with fingerprint data. So it has no effect.
Right now it is not possible to change fingerprint or any of its part at runtime. Maybe we will add this possibility in the future.
You can try to examine source and change fingerprint files. But it won't necessarily work. And it is a hack, so no docs on it.


Anyone having problems with bcrypt?

I am trying to deploy this demo site https://github.com/Zetoff/mantra-accounts-test
Everything works fine in when I run meteor.
I deploy using:
meteor-now -e NODE_ENV=production

And I get a this log: https://zetoff-accounts-material-ui-demo-ivrdrtaxvy.now.sh
I am guessing is a problem with bcrypt, but not sure.

I found this issue related to https://github.com/meteor/meteor/issues/7512
No apparent solution.

I was able to clone and deploy it here: https://mantra-accounts-test-fntitbpsdt.now.sh/

@purplecones which process have you followed? 🤔

Nothing extra, just deployed with meteor-now -d to watch log. Although the first deployment failed with strange error that I haven't seen before: https://mantra-accounts-test-skfrjivapf.now.sh/
Container started
Error response from daemon: network brddepl not found

Without changing anything I redeployed and it worked fine. Must be a now related issue.

Created https://github.com/zeit/now-cli/issues/295 for additional insight

@PolGuixe also this one https://mantra-accounts-test-xjylwlkkye.now.sh/ was deployed with
meteor-now -e NODE_ENV=production

and it also worked fine just in case you were wondering if the NODE_ENV flag was causing any issue

This is the error I am getting when I run meteor-now -e NODE_ENV=production -d:
> ▲ docker build
> Step 10 : RUN cat x* > bundle.tar.gz
>  ---> Running in 702c56bfefc5
>  ---> 07db413641f6
> Removing intermediate container 702c56bfefc5
> Step 11 : RUN tar -xzf bundle.tar.gz
>  ---> Running in bbb55e0cff0d
>
> gzip: stdin: decompression OK, trailing garbage ignored
> tar: Child returned status 2
> tar: Error is not recoverable: exiting now
> The command '/bin/sh -c tar -xzf bundle.tar.gz' returned a non-zero code: 2

can you link the deployment url?

here: https://zetoff-accounts-material-ui-demo-ubhnsopiet.now.sh

Is this the same repo as before? https://github.com/Zetoff/mantra-accounts-test

Yes

I am able to deploy your app with
meteor-now -e NODE_ENV=production -d

https://mantra-accounts-test-gjqhjmouiv.now.sh/
Don't think there is an issue with the app here. Are you able to deploy any meteor app using meteor-now? If so, can you try deploying this app again and see if you still get the error?
It might be another now related issue. Hard to say at this point.

I've deployed these two:

https://mantra-meteor-now-deployment-test-xgpfnfeliv.now.sh
https://meteor-now-test-quczomxioo.now.sh

And both seem fine.
I'll try to delete and clone the back the repo with which I am having the problem.

After deleting and cloning the repo I had the problem with, it worked.
Result:

https://zetoff-accounts-material-ui-demo-dtegtmwdsl.now.sh

I have no idea what was causing the problem.


Master based azure speech

Azure Speech bindings and working sample.

related issue (feature request)
https://github.com/xamarin/XamarinComponents/issues/640

/azp run

/azp run

/azp run


dart: 1.16.1 -> 1.24.3

Also, added support for the "dev" channel.
Motivation for this change
Trying to get flutter packaged, and it needs a particular version of dart (maybe?).
Also, just upgrading dart.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build dart

Failure on aarch64-linux (full log)
Partial log (click to expand)
Package ‘dart-1.24.3’ in /var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/interpreters/dart/default.nix:65 is not supported on ‘aarch64-linux’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/bin/dartdevc: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/bin/pub: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/bin/dart2js: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/bin/dartfmt: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/bin/dartanalyzer: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/lib/analysis_server/tool/spec/generate_files: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/lib/analyzer/tool/generate_files: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3/lib/front_end/tool/fasta: interpreter directive changed from "/bin/bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
checking for references to /tmp/nix-build-dart-1.24.3.drv-0 in /nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3...
/nix/store/c6wzik353cj0n6kcccgvih7qqmvyi33q-dart-1.24.3

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘dart-1.24.3’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/development/interpreters/dart/default.nix:65 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

@izuk Did you end up packaging flutter?

Nope.
On Thu, Jul 4, 2019 at 1:04 PM Aaron Andersen<EMAIL_ADDRESS>wrote:

@izuk https://github.com/izuk Did you end up packaging flutter?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/35104?email_source=notifications&email_token=AAI26JJ4CNHNPC3YMZ4INVLP5YURBA5CNFSM4ERGVVT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZH3MRA#issuecomment-508540484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI26JKT6MIEGILZPQBJGRTP5YURBANCNFSM4ERGVVTQ
.

Thanks.


Default webcard

Summary

Adds in ability to create webcards more easily on the app without the need
for an app update.
Note: This PR is for non authenticated cards that mainly use the openLink
javascript channel. A version that supports auth cards will come in later.
Changelog

[General][Add] - Add dynamic card creation for webcards
Test Plan

Need to log in to all account types (student, staff, visitor), then go to the home page.
General home page testing like we do for the regression testing will suffice.

Closed due to new PR posted which supports authenticated webCards.


Push 0.8.1 release to Github

Currently, it's missing there: https://github.com/bastibe/PySoundFile/releases

It's also missing on readthedocs.org, this would do it:
git push origin 0.8.1

Nothing changed. 0.8.1 only updated the wheel files on PyPi

Thanks, I updated readthedocs.org to show 0.8.1 by default.


add packages to R section

Hi Eduardo. Here are a few packages for spatial analysis that I thought you might wanna add to the R section.

r5r: Rapid realistic routing on multimodal transport networks with R5
gtfs2gps: Converting Transport Data from GTFS Format to GPS-Like Records
geobr: Download Official Spatial Data Sets of Brazil

Hi Rafael. I just added the packages to the list. Thanks a lot for your contribution! :)


getOfflineSignerAuto

I haven't tested this, because I found it quite hard to yarn link all the cosmos-kit packages locally, but I believe it resolves https://github.com/cosmology-tech/cosmos-kit/issues/99.
What happens is that getOfflineSigner in Keplr defaults to returning a wallet that supports Amino and Direct signing. This is obviously false in the case of Ledger devices and is leading to some of the errors we've been seeing.

oh wow thanks @0xArbi !
we'll test this asap!


[T4A5][T15-A2]Yoshiaki Nishimura

Ready for Review (sorry about the spacings)

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.

@yoshi-1224 Some comments added. Please close the PR after reading comments. Sorry to bother you again. You may want to see the comments.


iOS solution if using Expo

If using Expo, where the iOS and Android folders aren't exposed to you, is there a way to implement your iOS fix so this module works? Thanks.

@GitHubGreg is it related?!? https://github.com/react-native-community/react-native-text-input-mask/issues/73

@GitHubGreg did u find alternative solution?


Win7 titlebar forced on PWA windows on Win10

I have #disable-windows-10-custom-titlebar enabled, as I'm using glass8 to get aero glass in my Win10 titlebars.
Supermium main window works perfectly, but all PWA windows have a Win7 titlebar, rather than a Windows 10 or glass8 titlebar:

Left is Supermium with YouTube PWA (displaying incorrect Win7 titlebar), right is Ungoogled Chromium with --disable-windows10-custom-titlebar and AlternativeTo PWA (displaying correct Win10/glass8 titlebar)
Possibly related

Also, after first installing a PWA, I get 2x windows for it popping up.
https://github.com/win32ss/supermium/issues/621
https://github.com/win32ss/supermium/issues/135

This extension which opens pages as popup has almost the correct window borders on Win10 (seems to be general window borders, not glassy)


Counter bug in w2v forced alignment tutorial

🐛 Describe the bug
Hey @mthrok, first off thanks for the Forced Alignment Tutorial, it was really fun to play around with. I believe I found that the backtrack step has a subtle bug that can lead to Point objects being assigned to the wrong token_index. E.G. if the label we're aligning is me and the first timestep that is aligned to e has a larger score than any of the timesteps assigned to m, then that timestep will be added to the path, but with token index 0 (m) instead of 1 (e). This primarily causes problems at word boundaries because the probability scores for the space-separator token '|' are often near 1.
Current code
# Main loop of backtrack step
for t in range(t_start, 0, -1):
        # 1. Figure out if the current position was stay or change
        # Note (again):
        # `emission[J-1]` is the emission at time frame `J` of trellis dimension.
        # Score for token staying the same from time frame J-1 to T.
       stayed = trellis[t - 1, j] + probs[t - 1, blank_idx]
        # Score for token changing from C-1 at T-1 to J at T.
        changed = trellis[t - 1, j - 1] + probs[t - 1, tokens[j - 1]]

        # 2. Store the path with frame-wise probability.
         prob = probs[t - 1, tokens[j - 1] if changed > stayed else 0].exp().item()

        # Return token index and time index in non-trellis coordinate.
        # BUG HERE: path is appended with token_index `j-1` but it should be done
        # after updating the token in step 3. Fixed code below
        path.append(Point(j - 1, t - 1, prob))

        # 3. Update the token
        if changed > stayed:
            j -= 1
            if j == 0:
                break
    else:
        raise ValueError("Failed to align")
    return path[::-1]

I think this will fix it, but I haven't tested much so I may accidentally be introducing another bug.
Fixed code
# Main loop of backtrack step
for t in range(t_start, 0, -1):
        # 1. Figure out if the current position was stay or change
        # Note (again):
        # `emission[J-1]` is the emission at time frame `J` of trellis dimension.
        # Score for token staying the same from time frame J-1 to T.
       stayed = trellis[t - 1, j] + probs[t - 1, blank_idx]
        # Score for token changing from C-1 at T-1 to J at T.
        changed = trellis[t - 1, j - 1] + probs[t - 1, tokens[j - 1]]

        # 2. Store the path with frame-wise probability.
         prob = probs[t - 1, tokens[j - 1] if changed > stayed else 0].exp().item()

        # 3. Update the token
        if changed > stayed:
            j -= 1

        # Fix: Only append to path after token_index j has been updated. Move break to bottom of loop.
        path.append(Point(j - 1, t - 1, prob))

         if j == 0:
             break
    else:
        raise ValueError("Failed to align")
    return path[::-1]

Also, in this line  prob = probs[t - 1, tokens[j - 1] if changed > stayed else 0].exp().item(), shouldn't the else be blank_idx and not 0?
Versions
torchaudio 0.13.1
torch 1.13.1

Hi @rbracco
I think you are right. I also think path array should be initialized with one element for the final position, before this loop starts. Also the visualization right after this seems to be off-by-one for both axes as it uses emission coordinate.
BTW, this week, all sudden I am getting many inquiries about FA tutorial. Do you know why? Was it featured somewhere?

Yeah sorry, I should've waited to open an issue, I thought I'd pinned it down but now I'm having trouble wrapping my head around it as well. In my code I made some changes to the trellis generation as well so I'm worried I might have broken something. I'm stepping through the colab now trying to check. I'll report back if I find anything
In my case a friend linked me in response to a question I'd asked about force alignment.

Yeah sorry, I should've waited to open an issue, I thought I'd pinned it down but now I'm having trouble wrapping my head around it as well. In my code I made some changes to the trellis generation as well so I'm worried I might have broken something. I'm stepping through the colab now trying to check. I'll report back if I find anything

I tried your suggestion in #3440
https://docs-preview.pytorch.org/pytorch/audio/3440/index.html (will be available once build_docs CI job is done.)

The average word-wise probability is lower. Let me think again.

In my case a friend linked me in response to a question I'd asked about force alignment.
Also did the trellis code change? This section of the get_trellis code in colab is different than what I remember
# Trellis has extra diemsions for both time axis and tokens.
    # The extra dim for tokens represents <SoS> (start-of-sentence)
    # The extra dim for time axis is for simplification of the code.
    trellis = torch.empty((num_frame + 1, num_tokens + 1))
    trellis[0, 0] = 0
    trellis[1:, 0] = torch.cumsum(emission[:, blank_id], 0)
    trellis[0, -num_tokens:] = -float("inf")
    trellis[-num_tokens:, 0] = float("inf")

Few changes were made such as #3172 and  #2544.

Thanks, I'll take a look again tomorrow and let you know if I make any progress.

Sorry for the premature issue, I was able to resolve it.
This was the issue:
prob = probs[t - 1, tokens[j - 1] if changed > stayed else 0].exp().item(), should be prob = probs[t - 1, tokens[j - 1] if changed > stayed else blank_id].exp().item()
This only affects implementations where blank_id is not 0 so code in the tutorial is not affected. Thanks for helping me to debug.

@rbracco Thanks for the follow-up. I remembered that the code in tutorial is somewhat simplified version for educational purpose, and it is known that the treatment of blank tokens is slightly different than what it should be.
You can look at more accurate version https://github.com/pytorch/audio/pull/3067/files#diff-de50e62569b0186b8b6cd648b3be81585132cbea26be2b61880876f2194a0e2e, which is the basis of the new C++ implementation https://pytorch.org/audio/main/generated/torchaudio.functional.forced_align.html#torchaudio.functional.forced_align, which works on CPU/GPU and can do a bit more elaborated things
https://pytorch.org/audio/main/tutorials/ctc_forced_alignment_api_tutorial.html#dealing-with-missing-transcripts-using-the-star-token

Thank you @mthrok that clarifies things a lot. I had noticed that the final token could only ever have a duration of 1 timestep in the best path which was suboptimal, but adding a space token to act as  and  fixes that.
Please note in the linked PR the added '|' is mistakenly referred to as a blank instead of a space. Also I think inserting a blank as SOS/EOS won't work, but adding a space does.

Please note in the linked PR the added '|' is mistakenly referred to as a blank instead of a space. Also I think inserting a blank as SOS/EOS won't work, but adding a space does.

Thanks for pointing out. Fixed.


"Cannot read property 'toString' of undefined" when attempting to access collections

Hi,
New to SyncLounge, just installed the docker version.
When I click on a collection (movie library), I got an error saying "Cannot read property 'toString' of undefined".
Thanks for your work.

Thanks for the bug report. We have an outstanding fix for this error while we work on supporting collections: https://github.com/synclounge/synclounge/pull/277

Ok sorry, I didn't check pull requests...

Resolved in https://github.com/synclounge/synclounge/issues/283


Implements DBConnector and adds the required MySQL Connector dependency.

See title.

Since this contains some other stuff it'll wait for a bit.


[BUG] MLCubes are always the same version

Describe the bug
The MLCubes getting generated (model or metrics) are always the same version, as evidenced by the following files (just search for the string 0.0.1):

https://github.com/mlcommons/GaNDLF/blob/master/mlcube/metrics_mlcube/mlcube.yaml
https://github.com/mlcommons/GaNDLF/blob/master/mlcube/metrics_mlcube/mlcube_medperf.yaml
https://github.com/mlcommons/GaNDLF/blob/master/mlcube/model_mlcube/mlcube.yaml
https://github.com/mlcommons/GaNDLF/blob/master/mlcube/model_mlcube/mlcube_medperf.yaml

To Reproduce
N.A.
Expected behavior
For consistency, the version of the mlcube should follow the version set by the version.py file.
Media
N.A.
Environment information

N.A.
Additional context
N.A.

Tagging @hasan7n because I am unsure what the best approach here should be. Do you have any ideas?


make HTTP request headers optional

Extra HTTP request headers are usually not required. This change makes
them optional and makes it so that the HTTP driver only requires the
url field to perform a request. This also adds tests for using default
HTTP requests that are both successful and result in an error.

Codecov Report

Merging #260 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
+ Coverage   93.34%   93.35%   +<.01%
==========================================
  Files          26       26
  Lines         842      843       +1
  Branches      177      178       +1
==========================================
+ Hits          786      787       +1
  Misses         49       49
  Partials        7        7

Impacted Files
Coverage Δ

packages/moon/src/http/driver.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update example version in README

About time to bump the example version used in the README :)

:D


Error "Unexpected token E" when processing numbers with ExponentPart

When faced with a JSON document that is:
{
    "value": 8.4127988E8
}

A SyntaxError is reported
SyntaxError: Unexpected token E
    at Object.parse (native)
    at chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:1410
    at l (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:2355)
    at p (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:3532)
    at new a (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:2589)
    at r (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:1330)
    at chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:1008
    at o (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:2305)
    at chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:2916
    at MutationObserver.r (chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1:3929)(anonymous function) @ chrome-extension://gbmdgpbipfallnflgajpaliibnhdgobh/assets/viewer.js:1

and only a raw rendering of the document is shown. FYI, using JSON.parse(...) directly works fine:
> JSON.parse('{"value":8.4127988E8}')
Object {value: 841279880}

Using:

Version 0.9.2 of the extension
Chrome 44.0.2403.81 beta-m (64-bit) on Windows

Really odd, I'll check this up. Thanks!

Awesome. And my beloved number made it into the test data :)

Many thanks :+1: Version 0.9.3 rolling out!

I just confirmed with 0.9.3 that the full document that contained that number is rendering perfectly.

:tada:


Bump version 0.29.0 of cadl-ranch-specs

Cadl Ranch Contribution Checklist:

[ ] I have written a scenario spec
[ ] I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
[ ] I have written a mock API
[ ] I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

Did it via pnpm changeset version? Seems only 1 package get bumped version?

Did it via pnpm changeset version? Seems only 1 package get bumped version?

Yes. But I misunderstood the prompts, so I only bumped 1 package. I'll create another PR to bump all.

need to fix cspell and bump missing packages in https://github.com/Azure/cadl-ranch/pull/506