collection.createDocument : Javascript SDK misses this function

The page gives an example stored procedure and in this it gets a "collection" from getContext().getCollection(). First of all this is confusing when working with Core "SQL" API because that is a container. Then later it says collection.createDocument(), but I cannot find this function because when I execute this stored procedure it complains that the first parameter is invalid. Now, when I execute this from the portal I cannot see any useful information other than a very complicated log data that says the first argument is not correct. I tried to lookup the SDK documentation and I cannot find any reference to the "Collection" nor of any "createDocument()" function. Microsoft should focus on useful documentation.

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

ID: 14913a18-671a-c55f-d082-e750bec7baab
Version Independent ID: 9cc08ba2-a79f-3ee2-9beb-8919f77d10d3
Content: Write stored procedures, triggers, and UDFs in Azure Cosmos DB
Content Source: articles/cosmos-db/how-to-write-stored-procedures-triggers-udfs.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @timsander1
Microsoft Alias: tisande

@sijucm
Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@sijucm
Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@sijucm The Collection and container are used interchangeable sometimes but we do acknowledge the fact that containers should be used for SQL API.
Additionally here are the two documents for the createdocument function which is applicable to stable Java SDK.

CreateDocument
Authoring SP

Hope these document helps  and please get back to us for any questions.

@sijucm The Collection and container are used interchangeable sometimes but we do acknowledge the fact that containers should be used for SQL API.
Additionally here are the two documents for the createdocument function which is applicable to stable Java SDK.

CreateDocument
Authoring SP

Hope these document helps  and please get back to us for any questions.

@sijucm Please let us know if you need any further help regarding this.

@sijucm Please let us know if you need any further help regarding this.

@sijucm We are going ahead and closing this feedback at this time. Please get back to us if you have any further questions.

@sijucm We are going ahead and closing this feedback at this time. Please get back to us if you have any further questions.

But this is about a javascript code and NOT Java. The code in that page is
a javascript code.
On Thu, Jan 7, 2021, 7:18 PM NavtejSaini-MSFT<EMAIL_ADDRESS>wrote:

@sijucm https://github.com/sijucm We are going ahead and closing this
feedback at this time. Please get back to us if you have any further
questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/68360#issuecomment-756290428,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJBWQLJB7ANYC5SK3DXGTLSYX3IJANCNFSM4VTI2PLQ
.

But this is about a javascript code and NOT Java. The code in that page is
a javascript code.
On Thu, Jan 7, 2021, 7:18 PM NavtejSaini-MSFT<EMAIL_ADDRESS>wrote:

@sijucm https://github.com/sijucm We are going ahead and closing this
feedback at this time. Please get back to us if you have any further
questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/68360#issuecomment-756290428,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJBWQLJB7ANYC5SK3DXGTLSYX3IJANCNFSM4VTI2PLQ
.

@timsander1 Can you please check this and comment regarding this issue.

@timsander1 Can you please check this and comment regarding this issue.

Hi @sijucm, this doc references older SDK versions. I'll create a work item to update this doc. Here are some useful samples for working with stored procedures using the latest .NET SDK (https://docs.microsoft.com/en-us/azure/cosmos-db/sql-api-dotnet-v3sdk-samples#server-side-programming-examples), in case you happened to be using that one.
Let me know if you are using a different SDK version and I can send that link.

Hi @sijucm, this doc references older SDK versions. I'll create a work item to update this doc. Here are some useful samples for working with stored procedures using the latest .NET SDK (https://docs.microsoft.com/en-us/azure/cosmos-db/sql-api-dotnet-v3sdk-samples#server-side-programming-examples), in case you happened to be using that one.
Let me know if you are using a different SDK version and I can send that link.

#please-close


A few devnet related fixes

These fixes are originally from the DIP3 branch but should be included into develop ASAP, as otherwise devnet users will have issues connecting to a devnet. See individual commits.

As discussed on Slack (and more related), non-standard ports seem to cause re-connection issues. If a node starts on non-default ports, it will not connect automatically to a remote peer running on that port unless addnode is used every time the node starts. It will connect to a remote peer that is running on a non-standard port if it is running on the default.
To summarize:

Local Port
Remote Port
Connection

18999
18999
No automatic re-connect (after initial addnode)

19999
18999
Automatic re-connect works (after initial addnode)

Also, a little off-topic for the PR probably, but why does -devnet require assigning -port and -rpcport when there are defaults set in chainparams? :thinking:

Added suggested changes. I however left the multiple ifs in OpenNetworkConnection intact, as I always found the original if extremely hard to read. I added a few comments to make it easier to read.

Tested and it seems to work as expected.
utACK -> ACK


feat(report): report x:scenario and x:expect ID in XML report

XSpec compilers and reporters rely on generate-id() for ID generation. Consequently,

IDs have to be normalized when comparing the report HTMLs
template/function names in the compiled stylesheets/queries are not human-friendly

oXygen plugin uses a patched version of XSpec to generate a report XML like this:
<x:scenario template-id="x019911a9-4764-325f-9c76-788ac5926e8f">

To ease those pain points, this pull request

generates less volatile IDs of x:scenario and x:expect

based on the element position in the gathered and unshared schenarios

writes the generated IDs in the report XML

similar to @template-id above, but under a different attribute name (@id) to better represent its characteristics

reuses IDs when formatting the report HTML

therefore the report HTML files in test/end-to-end/ no longer have to normalize @id.

Merging this, as this doesn't change end-user facing behavior.

@xspec/oxygen-xml-editor-xspec-support
In the next version of XSpec, the scenario ID will be readily available in the result XML, although you will still need to generate the ID by yourself using #734.


refactor: refactor TsJestTransformer

Summary

Use @jest/create-cache-key-function to create cache key
Rename interface ConfigSetItem -> CachedConfigSet
Rename field configSetItems -> cachedConfigSets
Set cachedConfigSets, logger, configsFor as protected methods for users to extend.

Test plan

Rewrote unit tests, green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 6107

10 of 10 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 93.664%

Totals

Change from base Build 6104:
0.5%

Covered Lines:
1102

Relevant Lines:
1132

💛  - Coveralls


Handle file scheme in Article.download

Update the Article download function to handle the file scheme.
Add test cases for using Article.download with a file url

Updated. Sorry for the delay in updating the PR.


Fix secured connection to kafka for both consumer and producer. Suppo…

Use a kafka secure connection only if security protocol is different than PLAINTEXT . Add support for secure connection to kafka AdminClient

Hey, thanks for this.  I'm not sure I understand why the if clause has been inverted.  Isn't this saying, 'if plaintest security is not configured in properties, then go ahead and enable plaintext security'?

Hi,
Basically, the security.protocol can be one those values PLAINTEXT, SASL_PLAINTEXT, SASL_SSL, SSL. So if the security.protocol property is different than PLAINTEXT security should be enabled.


xliff-simple-merge fails on unix host

How to reproduce:
$ yarn add -D xliff-simple-merge
$yarn run xliff-simple-merge --help
Result:
yarn run v1.22.19
$ /home/datav/dv-connect/itron/itron-common/node_modules/.bin/xliff-simple-merge --help
/usr/bin/env: ‘node\r’: No such file or directory
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
This is caused by the bin file having DOS/Windows line endings.

Hi @mpumford thanks for reporting this - would you like to provide a PR for this?


How to set Tab Index for keyboard navigation?

I am doing an accessibility review of our new website.  I have a major flaw at the moment: The main navigation from the SideNav cannot be reached via the keyboard (tab to the menu item, press Enter to fire the link).  What am I doing wrong?
Here is an example NavItem with the tab index trying to be defined:
                 <NavItem tabindex="5" eventKey="myutilities">                     <NavIcon >                       <Industry32 className={"my-custom-class"}/>                     </NavIcon>                     <NavText>                       My Utilities                     </NavText>                   </NavItem>
The resulting page source has tabindex="-1", no matter what index number I put on an item.  I've attached a screen shot of the Inspect Element results.

TabIndex shows better when put on the NavIcon or the NavText.  But those don't actually fire the link when the Enter key is pressed.
I'm on a tight deadline, and this is a major accessibility issue for our app.  I hope there is a simple fix.  Will you enlighten me, please?  :-)
Thank you!

We hard code tabIndex to -1
Need to refine this attribute
https://github.com/trendmicro-frontend/react-sidenav/blob/e21ab4afc63c773b357cc9247f958b534838c43c/src/NavItem.jsx#L150

When might this bug be fixed?  Our new app must go to beta testing next month, and to full public production just by July.  Without being able to tab thru the sidenav (and so requiring mouse clicks for all our major navigation), we don't meet accessibility standards.
Can you suggest a workaround if the bug isn't fixed?

Our launch was delayed due to COVID-19 but is getting started back.  This issue still needs to be corrected.  Has any progress been made on your end?
How are other sites dealing with accessibility while using this sidenav?


Documentation in Sphynx format

Start with examples and document all methods used there.
Generate Sphynx for those first.

Has this been started? I'd be willing to help with documentation.

Along the same lines, is there any interest in adding Python type hints?


refactor: class MonitoringService as close only

Motivation
Lodestar does not support start stop syntax in general. It's unnecessary for downstream classes to expose such behavior

Part of https://github.com/ChainSafe/lodestar/issues/4501

Description
MonitoringService as close only class

:tada: This PR is included in v1.9.0 :tada:


Images Alpine, optimization

Thanks for your help, but I'll prefer if we split change to db and app images.
I think it is ok to use Alpine for our database image, just need to test it. But app image is way more sensitive since Alpine is not officially supported by Mattermost, so we will need to discuss this also with Mattermost staff.
So it is better to have a PR for database, which can be quickly merge if tests worked, and another one for app which need more discussions :)

@pichouk app work, I tested everything on my test server


CI backend build continues after pre-push hook fails

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
No response
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
9.2.1
What operating system are you using?
CI Build
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Amplify Categories
hosting
Amplify Commands
push
Describe the bug
I have a pre-push hook setup to do some pre-building for the lambda functions.
In the CLI, if the pre-push hook fails then the push aborts. In the CI build, it continues and then falls over later because the pre-building failed.
Expected behavior
The CI build should abort if the pre-push hook exits with a non-zero error code.
Reproduction steps
Add a pre-push.sh file to amplify/hooks/ with the code
#!/bin/bash

# **** SETUP ****
# exit when any command fails
set -e
# keep track of the last executed command
trap 'last_command=$current_command; current_command=$BASH_COMMAND' DEBUG
# echo an error message before exiting
trap 'echo "\"${last_command}\" command filed with exit code $?."' EXIT

ls random

GraphQL schema(s)
No response
Project Identifier
No response
Log output

2022-08-18T15:08:01.009Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2022-08-18T15:08:01.120Z [INFO]: ✅
2022-08-18T15:08:02.420Z [INFO]:
2022-08-18T15:08:02.420Z [INFO]: ----- 🪝 pre-push execution start -----
2022-08-18T15:08:02.596Z [INFO]: ><EMAIL_ADDRESS>build function
                                 > ./build.sh
2022-08-18T15:08:02.602Z [INFO]: "ls random" command filed with exit code 2.
2022-08-18T15:08:02.617Z [INFO]: ----- 🪝 pre-push execution end -----

...

2022-08-18T15:08:10.227Z [WARNING]: ✖ An error occurred when pushing the resources to the cloud
2022-08-18T15:08:10.229Z [WARNING]: ✖ There was an error initializing your environment.
2022-08-18T15:08:10.232Z [INFO]: [0m[Error: ENOENT: no such file or directory, lstat '...'] {
                                 errno: -2,
                                 code: 'ENOENT',
                                 syscall: 'lstat',
                                 path: '...'
2022-08-18T15:08:10.250Z [ERROR]: !!! Build failed

Additional information
Sorry in advance if there is a specific repo for CI build issues.

@aaron-hutton 👋🏽 Are you still running into this issue? If so, please provide you app ID so we can further investigate this behavior.

Hi @hloriana, the project has since been closed, so I can't use it to investigate if the bug still exists. I'm happy for this issue to be closed if you want to wait and see if anyone else ever encounters it?


Bakelet for NodeJS not installing

Error Description
Getting the error:
Error: Error: Cannot find module '/snapshot/Baker/lib/bakelets/tools/nodejs
To Reproduce

baker.yml file:

name: xyz
vm:
  ip: <ip_address>
tools:
  - nodejs9

Run baker bake

Expected behavior
It should have installed nodeJS as a tool on the given VM.
Screenshots

Please complete the following information:

OS: MacOS
Version 10.14

Need to use lang instead of tool.

In looks like you're using tools, when it is expecting lang. Sounds like we can use a better error message here.
name: suites
vm:
  ip: <IP_ADDRESS>
lang:
  - nodejs9

You can also verify all the bakelets names by checking here:
https://github.com/ottomatica/bakelets-source


Don't trigger key bindings when using search forms

Key bindings (Repost, Like, ..) seems to be highlighted when using search or other input forms. The trigger should probably check for this before firing (seems to be purely cosmetic though, the actual repost or like action doesn't fire).

There are measures taken not to trigger those key binding when typing inputs.
Avoiding menu item highlights is s little trickier. The problem is, shortcuts like "L for Like" are implemented on soundcloud.com itself, the shortcut on the menu item is only relevant when the main window does not have keyboard focus (minimized, in background, etc).
The proper solution would disable the menu shortcuts when the main window is focused and re-enable them on blur. I did not investigate whether this is doable with Electron or not, but a pull request to improve is always welcome.


provision-begin.sls: work around podman/runc bug

Works-around: https://github.com/ceph/ceph-salt/issues/130
Signed-off-by: Nathan Cutler<EMAIL_ADDRESS>

I can confirm that this makes the needed modification to the file:
2020-03-20 11:40:49,792 [salt.state       :1821][INFO    ][12947] Running state [/usr/lib/systemd/system/salt-minion.service] at time 11:40:49.792628
2020-03-20 11:40:49,793 [salt.state       :1854][INFO    ][12947] Executing state file.line for [/usr/lib/systemd/system/salt-minion.service]
2020-03-20 11:40:49,805 [salt.state       :321 ][INFO    ][12947] File changed:
---
+++
@@ -3,7 +3,7 @@
 After=network.target salt-master.service

 [Service]
-Type=notify
+#Type=notify
 NotifyAccess=all
 LimitNOFILE=8192

and
2020-03-20 11:40:49,805 [salt.state       :1821][INFO    ][12947] Running state [/usr/lib/systemd/system/salt-minion.service] at time 11:40:49.805920
2020-03-20 11:40:49,806 [salt.state       :1854][INFO    ][12947] Executing state file.line for [/usr/lib/systemd/system/salt-minion.service]
2020-03-20 11:40:49,808 [salt.state       :321 ][INFO    ][12947] File changed:
---
+++
@@ -4,7 +4,7 @@

 [Service]
 #Type=notify
-NotifyAccess=all
+#NotifyAccess=all
 LimitNOFILE=8192
 ExecStart=/usr/bin/salt-minion

However, it doesn't work because the salt-minion.service needs to be restarted after the file is modified, and I don't know how to do that :-(

The file needs to be munged (and salt-minion.service restarted) before the Salt States begin executing.

Apparently, it is possible to restart the salt-minion.service using States:
https://docs.saltstack.com/en/latest/faq.html#restart-using-states

salt '*' cmd.run 'unset NOTIFY_SOCKET; podman run --net=host --rm registry.opensuse.org/opensuse/leap:15.2 bash -c "seq 5080 | awk '"'"'{print<PHONE_NUMBER>1234567890}'"'"' "'
<works>

I think unset NOTIFY_SOCKET; is less intrusive?

Codecov Report

Merging #132 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #132   +/-   ##
=======================================
  Coverage   77.93%   77.93%
=======================================
  Files          10       10
  Lines        2116     2116
=======================================
  Hits         1649     1649
  Misses        467      467

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 71c823d...1bb724c. Read the comment docs.

I think unset NOTIFY_SOCKET; is less intrusive?

Seems work too: https://paste.opensuse.org/view/raw/38547391

I think unset NOTIFY_SOCKET; is less intrusive?

So we put that before the cephadm bootstrap call?

@bk201 Nice! Closing this one, then.

I think unset NOTIFY_SOCKET; is less intrusive?

Seems work too: https://paste.opensuse.org/view/raw/38547391

@bk201 I've created a PR with your patch https://github.com/ceph/ceph-salt/pull/134


Is it possible to use .env files?

I have a project with a .env file in the root and I use import "dotenv/config"; to load the .env to process.env. I didn't realize it at first but this is not working with email/index.ts as the env values I'd expect to be defined (present in my .env) are not set on process.env.

Hm, I'm guessing what's happening here is that you're seeing this behavior when sending from the mailing UI because the import "dotenv/config"; is somewhere else in your app. Could that be the case?
If you add it to the top of email/index.ts does it work?

That said, I think this is probably common enough that mailing should include dotenv/config internally and run it on every CLI command.

I was actually adding the import "dotenv/config"; to the top of emails/index.ts (I kept writing email as the directory above but was tired last night -- I'm using the default locations).
Oddly enough today it is all working. I thought it was due to isolatedModules in my root tsconfig.json as mentioned in https://github.com/successor-software/mailing/issues/90 but I'm flipping that on and off and it doesn't make a difference now.
There isn't anything compiled/cached between npx mailing runs is there? If not, it seems like there was some kind of glitch yesterday that I'm not seeing at all today.

I edited emails/index.ts to change the contents of a console.log and it's updating without restarting (wasn't sure if that would be the case) so I'd chalk this up to some kind of odd edge case.
I'll keep trying to reproduce as I use mailing but I'm fine with closing this out and can reopen if I can get it to happen again. Sorry for what amounts to noise at the moment.
On a side note, really happy to find mailing -- thank you for sharing it. I was already using MJML but had a email template compile step and previewing wasn't as nice as I'd like plus I wasn't using react-mjml yet as it didn't fit well in my workflow. But I'm excited to be able to use all of that and have the excellent preview capabilities of mailing!

There isn't anything compiled/cached between npx mailing runs is there? If not, it seems like there was some kind of glitch yesterday that I'm not seeing at all today.

Nope, at this point everything is in memory.
I think adding dotenv to the CLI so that people don't have to import it themselves is a good simplification, so I appreciate the issue.
Excited mailing is useful for you. If you have a minute, you mind telling us a little bit about how you plan to use it and your email stack? Just trying to learn about users so we can figure out what to iron out + how else we can ship to support devs sending email.

Sure, I'll send an email about my usage and adding dotenv by default sounds great!

Just wanted to add this is a great feature to build in. I had to slightly tweak mine because I use a file with a different name:
import * as dotenv from 'dotenv';
import { buildSendMail } from 'mailing-core';
import nodemailer from 'nodemailer';

dotenv.config({
  path: '.env.local',
  debug: true,
});

Using debug helped me figure out that due to the way I have tsconfig setup, by base path is the root of the repo, and not the location of emails/index.ts.
Great job on this, looking forward to see where it goes!


Adding missing property for FindingPublishing

Add property listed here in AWS documentation. :
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-guardduty-detector.html#cfn-guardduty-detector-findingpublishingfrequency

Thanks!


API Documentation (Public)

Hello, I am unable to find any mention of a public API for source backend that allows interested students to experiment with without setting up their own server, obtaining luminus_api_key etc. Would it be possible to provide a public API? It would be of great help in some of the mini projects I wish to do. Thank You!

Are you working on a project on frontend or backend? For front-end you can develop without an active backend api

Are you working on a project on frontend or backend? For front-end you can develop without an active backend api

I am playing around with a telegram bot that can take in source code and return the output. It's currently done rather ineffectively because the code is manually inserted into the playground for execution. I was hoping there is a public API that I can use instead especially since it was announced that source is likely to revamp its frontend. Thank you for your help!

check out running jsslang on the cli bro

check out running jsslang on the cli bro

But a telegram bot would require server support, right?
The current Source Academy runs all programs (playground and missions) in the client, using js-slang, except the automated grading. For that, we use AWS Lambda.
The problem for your use case: There is no API yet for running Source programs on the server.
You could design such an API and implement it using JS-slang and Node.js.

@tjtanjin You can look up eval bots, since that is what you seem to be doing. You may only require https://github.com/source-academy/js-slang to run Source.

Thank you for all the pointers and suggestions! I'll look them up!


TLTableViewController doesn't configure cells correctly under iOS 8

Demo project showing broken cell here: https://github.com/fatuhoku/Demo-TLIndexPathTools-Tables-Broken-Under-iOS8/tree/develop. Tested on Simulator, iPad Air (iOS 8).
Two symptoms:

text labels for default styles of UITableViewCell (Basic, Left / Right Detail, Subtitle) are misplaced
custom styles cause cells to overlap

The provided demo project covers the first case.
Using UITableViewController and explicitly wiring it up with TLIndexPathController still works.

Workaround published in 0.3.10. Demonstrated in Settings example project. Comment from Settings project:

In previous versions of TLTableViewController, the logic for heightForRowAtIndexPath
would in certain scenarios dequeue a cell from the storyboard and return that cell's
height. This case is problematic with new storyboards that use size classes because
the dequeued cells have zero height. The new version of heightForRowAtIndexPath
detects this case and returns UITableViewAutomaticDimension instead of zero, which
falls back (or perhaps forward) to the new auto sizing cell behavior.

Note that UITableViewAutomaticDimension appears to have some animation issues. In
particular, the cells below the "volume" cell visibly jump when the "volume" cell
appears. I assume this is related to the cell being taller than the other cells
and the UITableViewAutomaticDimension behavior setting up the animations with
starting values based on the default cell height. The workaround for this issue
was to make the "volume" cell and instance of TLDynamicHeightCell, which causes
TLTableViewController's classic cell height calculation to kick in.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2665

Questions ?
Please refer to the Backport tool documentation


Null pointer exception, when there is no certificate under the given alias

Description:
In the JWT authentication scenario, when there is no public certificate under the given alias, the service will return a 500 response with null pointer exception.
https://github.com/ballerina-platform/ballerina-lang/blob/5a42f042aae200e61c91d593c9f54e1955a7eeaf/stdlib/crypto/src/main/java/org/ballerinalang/stdlib/crypto/nativeimpl/Decode.java#L68
Needs to check if this object is null.
Steps to reproduce:
N/A
Affected Versions:
ballerina 1.1.1
OS, DB, other environment details and versions:
MacOS Mojave
Related Issues (optional):
https://github.com/wso2/product-microgateway/issues/1020
Suggested Labels (optional):

Suggested Assignees (optional):

This is already fixed with following PRs and will be available with 1.1.4 and 1.2.0 releases.
https://github.com/ballerina-platform/ballerina-lang/pull/21190
https://github.com/ballerina-platform/ballerina-lang/pull/21189


Use new error animations on the other applications

Currently, the new 404, 500 and 403 error animations are only shown in laboratory, but they should also work on classroom, bibliotheca and office.

Office is now deprecated, and introducing them in the angular pages will be too difficult. Closing


Actor reminder rename can leave reminder deleted in failure scenario

In what area(s)?
/area runtime
What version of Dapr?
1.6
Expected Behavior
If a rename fails, the existing reminder should still be present.
Actual Behavior
If the method fails anywhere after the DeleteReminder call, the reminder will not be put back and is thus lost. This can happen if:

The statestore has any failure while storing the reminder
If the evaluationBusy flag is somehow set after the delete (this would technically be another bug if I'm reading the code correctly)

Steps to Reproduce the Problem
Force a failure after the delete.
Release Note
RELEASE NOTE: FIX Condition in ReminderRename that would leave Reminder deleted.

This feature is going to be deprecated in the v1.8 release.

https://github.com/dapr/dapr/issues/4445


When inspector.reduce(prepared) failed, it should print the pretty-printed string of the prepared object in the StackTrace message

inpector.reduce(prepared) tends to fail due to the given MProductGroup prepared object has unexpected status.
When it fail, we need to look at how the prepared object's contents.
v0.4.7, I have to insert a debug print a statement like
println JsonOutput.prettyPrint(prepared.toString())

It is convenient if it print the string representation of the MProductGroup in the StackTrace.

0.4.8

not enough

At https://github.com/kazurayam/materialstore/issues/259
a digram of MProductGroup object is compiled


Form with non-unique element names and remote validator exception + proposed FIX

Subject of the issue
Exception is thrown when validating input with remote validator when element name is not unique on the form (input is nested in some other elements with the same name).
The problem is related to usage of findByName function.
Your environment

jQuery Validation Plugin v1.15.0
IE 11, Chrome

Steps to reproduce
Look at my example code here:
https://plnkr.co/edit/93QhyqGer417qLPhflrJ?p=preview
Expected behaviour
There are three elements with the same name="Nip" on the form (I am using some custom Angular components in my app), but only one input element to validate.
Remote validator returns false, so error label should appear for input[name=Nip].
The validator should ignore any element on the form, wchich it could not validate.
It should look only for validatable elements, such as inputs and text areas.
Actual behaviour
Exception in line 995: 'Uncaught TypeError: Cannot read property 'replace' of undefined'
escapeCssMeta: function( string ) {
return string.replace( /([!"#$%&'()*+,./:;<=>?@[]^`{|}~])/g, "$1" );
},
because string is undefined.
The function findByName: function( name ) in line 1018 is called during validation, it returns three elements. In the next steps the first element from the collection is considered as input element, wchich is wrong.
When using simple "required" validator instead of remote, it works as expected - function findByName is not called at all.
My solution
Change findByName function to look for elements, wchich are validatable (input, textarea).
    findByName: function( name ) {
        //var result = $( this.currentForm ).find( "[name='" + this.escapeCssMeta( name ) + "']" );
        var result = $( this.currentForm ).find( "textarea[name='" + this.escapeCssMeta( name ) + "'],input[name='" + this.escapeCssMeta( name ) + "']" );
        return result;
    },

The patch above solved my problem, but it could possibly break something else, if this function is used also for finding non-input elements - so verify it please.
I have included patched query.validate.debug.js, the patch in line 1021 is commented.
https://plnkr.co/edit/93QhyqGer417qLPhflrJ?p=preview
greetings
Mike

This still persists in  v1.16.0 .
Why is this still open?


fix: Made canvas responsive on mobile and some ui responsiveness

[x] issue # https://github.com/keep-starknet-strange/art-peace/pull/226
[x] follows contribution guide
[ ] code change includes tests
[ ] breaking change

This PR addresses the following for mobile experience:

Adjusted the canvas to be fully responsive on mobile devices.
Ensured that the canvas scales properly on mobile.
Made several UI tweaks to improve the overall look of Nft tab on mobile.
Improved the usePreventZoom hook

To test, you can run the project on a real mobile device to see the changes in action.
https://github.com/keep-starknet-strange/art-peace/assets/47580934/1253fb78-7bc1-47e1-b0fb-1303ee7ab0b8

Thank you! Cool to be able to explore the canvas on mobile now!


Update Linux-Development-Docs for Beginner

While building ROS2 on Linux according to the documentation, I found a flaw in the documentation. I hope this improvement will be useful to you, as it can mislead first-time students who came in from ROS2.

I just tried the instructions from https://index.ros.org/doc/ros2/Installation/Dashing/Linux-Development-Setup/ on a brand-new container, and without any changes to them, they worked fine for me.  Can you please walk through the instructions again in a fresh container and paste the exact errors you are getting?  Because I don't actually think the instructions need any changes right now.

@clalancette
I just tried to copy and paste the following commands:
sudo apt update && sudo apt install curl gnupg2 lsb-release
curl -s https://raw.githubusercontent.com/ros/rosdistro/master/ros.asc | sudo apt-key add -

However, I didn't execute
curl -s https://raw.githubusercontent.com/ros/rosdistro/master/ros.asc | sudo apt-key add -

So I add the above command properly. Then it works well. I must have been mistaken. I'm really sorry that I'm taking up so much of your time. I'll be careful the next time...

So I add the above command properly. Then it works well. I must have been mistaken. I'm really sorry that I'm taking up so much of your time. I'll be careful the next time...

No problem at all.  We appreciate the time you took to open an issue about it.


KafkaProducer send locks threads

Monix KafkaProducers's send is locking threads of scheduler when tasks are executed in parallel, e. g.
val topic = "example"
val sc = Scheduler.forkJoin(12, 12)
val producer = KafkaProducer[String, Array[Byte]](config, sc)

val tasks = List.fill(12)("test")
Task.wanderUnordered(tasks)(producer.send(topic, _)).runAsync(...)

here's output of async-profiler with -e lock flag:
---<PHONE_NUMBER>77 ns (44.57%), 19443 samples
  [ 0] monix.kafka.KafkaProducer$Implementation
  [ 1] monix.kafka.KafkaProducer$Implementation.$anonfun$send$2
  [ 2] monix.kafka.KafkaProducer$Implementation$$Lambda$7060.1333366705.run
  [ 3] akka.dispatch.TaskInvocation.run
  [ 4] akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec
  [ 5] akka.dispatch.forkjoin.ForkJoinTask.doExec
  [ 6] akka.dispatch.forkjoin.ForkJoinPool$WorkQueue.runTask
  [ 7] akka.dispatch.forkjoin.ForkJoinPool.runWorker
  [ 8] akka.dispatch.forkjoin.ForkJoinWorkerThread.run

This happens because of synchronization on producer and leads to performance degradation due to thread locking and impossibility of parallel sending
Given that apache KafkaProducer is thead-safe and asynchoronous one can benefit from parallel sending if batch size and linger time is high enough, when send is called in parallel it puts messages and callbacks on producer’s  internal buffer and sends them in batches and completes callbacks on own io thread.

fixed in #95


Testing with chrome using scala-js-env-playwright with esmodules and …

scala-js-env-playwright is an independent project that offers a JSEnv that uses Playwright for JavaScript execution.
This PR enables testing with chrome browser.
Laminar is most likely to be used with fastLinkJS and ESModule hence testing with these setting enabled will help identify errors in real browsers.
In the latest edition allows you to verify the actual version of the browser being used. debug = true to the PWEnv
 jsEnv := new jsenv.playwright.PWEnv(
      browserName = "chromium",
      headless = true,
      debug = true
    ),

Hi, thanks for the PR. The playwright js env is a useful project for end-user applications where you could test e.g. page transitions and such, but Laminar tests don't have such needs. It's true that testing in a real browser environment is more faithful than in jsdom, but in 7 years of using jsdom for Laminar tests, I don't recall encountering any such issues with it.
I try to minimize dependencies, so without a pressing need, I would rather avoid any changes. I will keep the PR open for now to serve as a reference for anyone wanting to use playwright in their own project.

@raquo Thanks for the feedback. Would it be helpful if this PR includes both options in the build? I propose having tests based on NodeJS+JSDom as the primary testing and release process, with testing on multiple browsers as a secondary option. The net effect is that the proven process remains unchanged, and whenever there is a need for quick validation, one can run browser-based tests.

@gmkumar2005 I don't know, Laminar's repo seems like the wrong place for this. Laminar shouldn't have dependencies that it doesn't need, even test / optional ones. Some other projects include all kinds of integrations into their repos (e.g. with cats, or various test frameworks / json libs / whatever), but my approach is to have such integration in separate repos, maintained by whoever wants to maintain them, which is usually not me. I link to all such things that I know of on the https://laminar.dev/resources page (I'll add your project there too, it's just too hard to update that page while Laminar 17.x is not final).
There's nothing Laminar-specific about the js env playwright configuration, so you may want to add playwright tests as a submodule to your own repo, for example my own test utils have tests that verify their own setup: https://github.com/raquo/scala-dom-testutils; and my https://github.com/raquo/scala-dom-types generator has tests that use that generator in the canonical way, without relying on the Laminar repo to test it.

Agree


Contact Us Section Of Homepage

The contact us section of the code isn't looking very pleasing , i wish to make it more appealing to the potential consumers and add value to the website.
I wish to work on this issue.

Hey
I would like to work on this issue.

I would like to fix this issue

I am Praful Nigam. Under GSSOC'24, I would like to work on this issue. Pls assign me this!!

Hello @khushi-joshi-05
I'm proposing an update to the Contact Us section design to enhance its aesthetics and usability. This update will include improvements in colour scheme, typography, layout, and responsiveness.

I would like to work on this issue under GSSoC'24.

Hey @khushi-joshi-05,
Hope you're doing well! I'd like to take on the assignment of enhancing the Contact Us section of the homepage. Given my experience as a content writer, having published over 30 articles on Asiana Times' website and completed numerous content writing internships, I believe I can add significant value to this task. Could you please allocate this to me with the GSSOC tag?

Thanks for assigning me this issue,
I have been tinkering for a good layout, I'll be sending a pull request
soon enough.
On Sun, 23 Jun, 2024, 5:36 pm Sukhbir Singh, @.***>
wrote:

Assigned #66
https://github.com/khushi-joshi-05/Food-ordering-website/issues/66 to
@sukritcharredsoul https://github.com/sukritcharredsoul.
—
Reply to this email directly, view it on GitHub
https://github.com/khushi-joshi-05/Food-ordering-website/issues/66#event-13255538662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFYW5KNXWQIZNLBYKLWNPTTZI223LAVCNFSM6AAAAABHQVCX3CVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGI2TKNJTHA3DMMQ
.
You are receiving this because you were assigned.Message ID:
<khushi-joshi-05/Food-ordering-website/issue/66/issue_event/13255538662@
github.com>


ChatBaiduWenxin stream mode doesn't work as expected

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain.js documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain.js rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
const model = new ChatBaiduWenxin({
    model: "ERNIE-Bot", // Available models are shown above
    streaming: true,
    temperature: 1,
    baiduApiKey: BAIDU_API_KEY, // In Node.js defaults to process.env.BAIDU_API_KEY
    baiduSecretKey: BAIDU_SECRET_KEY, // In Node.js defaults to process.env.BAIDU_SECRET_KEY
});

const stream = await model.stream("Hello! Tell me about yourself.");
const chunks = [];
for await (const chunk of stream) {
   chunks.push(chunk);
   console.log(`${chunk.content}|`);
}

Error Message and Stack Trace (if applicable)
Current output is return the entire final results

it should be streaming mode return like this
|
Hello|
!|
 I'm|
 an|
 AI|
 language|
 model|
 created|
 by|
 Open|
...

Description
When using ChatBaiduWenxin, how to return in a streaming fashion instead of returning a whole readable stream.
System Info
langchain 0.2.2
mac

ChatBaiduWenxin does not implement ChatBaiduWenxin.stream; it will default to calling the parent class and return data in non-streaming mode. If you want to use the stream mode, you can try this:
let result = ''
const model = new ChatBaiduWenxin({
    model: "ERNIE-Bot", // Available models are shown above
    streaming: true,
    temperature: 1,
    baiduApiKey: BAIDU_API_KEY, // In Node.js defaults to process.env.BAIDU_API_KEY
    baiduSecretKey: BAIDU_SECRET_KEY, // In Node.js defaults to process.env.BAIDU_SECRET_KEY,
    callbacks: [
      {
        async handleLLMNewToken(token: string) {
          result += token;
          console.log(`${token}|`);
        },
      },
    ],
});

@Robert-ZLF


[Branch View] manual merge tag

(https://github.com/go-gitea/gitea/issues/7976)  Show Merge Tag if it is merged manualy

Imho this should be checked on push and saved to pr

Imho this should be checked on push and saved to pr

I dont understand? What do you mean?

Do you mean this https://github.com/go-gitea/gitea/issues/7976#issuecomment-524673002 ?

Imho this should be checked on push and saved to pr

I dont understand? What do you mean?

Nevermind, it's late I thought it is about PR :)

do it the short way ... https://github.com/go-gitea/gitea/issues/7976#issuecomment-524934617

Nevermind, it's late I thought it is about PRs manual merge displaying :)

@lafriks but what dou you think of this solution to https://github.com/go-gitea/gitea/issues/7976 anyway

WIP notice: combine if statements

@lafriks can you review it?

Just to ask, is this also going to indicate a branch is merged if the branch was just created and pushed from the master branch?

It does. -> https://github.com/go-gitea/gitea/issues/7976#issuecomment-525257557

Just to ask, is this also going to indicate a branch is merged if the branch was just created and pushed from the master branch?

It does. -> #7976 (comment)

IMHO, having those branches that have no commits ahead of the master being called merged seems a bit weird as it could also mean that the branch was just created. I feel like that is going to cause some confusion.

we can check if it is at least one commit behind the default one to avoid souch or if it doesnt use to mouch cpu time lookup the merge commit

We also could detect if the second commit of the branch is not the second one of the default branch.

We also could detect if the second commit of the branch is not the second one of the default branch.

nice idear this would save lookup time and works with a lot of constructs ...
but if something is merged without --no-ff and no commits betwen it will not show up as merged ...
but i personaly have no problem with that :D

We also could detect if the second commit of the branch is not the second one of the default branch.

current_branch.comit.parentcommit == current_branch.comit.parentcommit -> branch merged = false ??

to test if divergence.Behind != 0 should have the same impact?

we could use git branch --merged | grep $branch

I cleanup the 22 commits so it is easy to understand - the actual diff is very smal ...

Actual State:
manual merge tag if:

no commits ahead default branch
not same as default branch

UI:

thx now its ready to merge :rocket:

Close thiss issue because of brainstorm with lunny
...
lunny
So Is it possible to know it's a cloned branch or a merged branch?

6543
as far as i know there is no way cou can be sure

6543
i got an idear but this will add a fourth lable to the branch view
if we dont calle it "manualy merged" but "included"
...


Fix link to "QGIS Application Network Connections" doc

"Testing" docs are only in English.

Sorry if I'm late here. Maybe should we instead use latest, and not testing?

Maybe should we instead use latest, and not testing?

It is an option, but if the testing is the most up-to-date page, then I think it is better to link to the testing page.

This is a topic that is not related to a particular QGIS version or the docs CI/CD, so unless we forget, I do not see a particular reason the occasional changes would not be backported.


Implement GetModuleList over grpc

Implement grpc GetModuleList service call

interesting - it builds now :)

I incorporated the requested changes and fixed session loading, ptal again.


Accept multiple input files

This is a quick solution to #31
In principle it does what it needs to do. What we will need in the future is referencing the file name in case of parsing errors. Currently files are concatenated and the line numbers that ANTLR error messages refer to are probably not consistent with file numbers of actual files.
So I'd say it's a step in the right direction, but we'll need to follow up on the issue.

With d1a5c92 our error messages also have file names associated with them, yay! Also, I decided to get rid of our stream util (that would wrap a stream around a string) because ANTLR has ANTLRInputStream that we can directly use.


Fix the invalid identifier 'FULL-TEST-SUITE' in the environment variable

What do these changes do?
Only ASCII letters (of either case), '_' and digits are supported, and the first character must not be a digit.

Related issue number

Fixes

Codecov Report

Merging #1289 (b299515) into main (26f5214) will decrease coverage by 33.41%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #1289       +/-   ##
===========================================
- Coverage   73.61%   40.20%   -33.42%
===========================================
  Files          88       88
  Lines        9509     9509
===========================================
- Hits         7000     3823     -3177
- Misses       2509     5686     +3177

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-98.44%)
:arrow_down:

python/graphscope/tests/unittest/test_context.py
0.00% <0.00%> (-81.04%)
:arrow_down:

python/graphscope/tests/unittest/test_download.py
0.00% <0.00%> (-20.00%)
:arrow_down:

python/graphscope/tests/unittest/test_session.py
0.00% <0.00%> (-64.22%)
:arrow_down:

python/graphscope/tests/unittest/test_udf_app.py
0.00% <0.00%> (-38.24%)
:arrow_down:

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 41 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 26f5214...b299515. Read the comment docs.


Enhancement 🧑💻:

WebMasterLog Repository (Proposing new issue)
:red_circle: Project Title : Home Page renders nothing
:red_circle: Enhancement Aim : There is literally nothing
:red_circle: Category : Angular, CSS, Front-end, NextJS, NodeJS, React, Vanilla, VueJS, etc.
:red_circle: Changes :
Screenshots 📷

Bro as u can see the home page literally renders nothing but an empty page
Assign under GSSOC

:white_check_mark: To be Mentioned while taking the issue :

Full name : Nalin Dalal
GitHub Profile Link : https://github.com/nalindalal
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.)
GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

there's already an issue for this and it's assigned.

so i can't work that means?

No. #140 is already assigned and someone is working on it. Check it.

@NalinDalal go ahead.

once check issue 344

I have reviewed and left the comment. But still homepage is blank. Just addition of footer is not counted as as a contribution.


Provide better getting started guide.

Hi. Great project, but I had a hard time to compile an elixir file written by myself. I'm running it on linux.
Also compiling the example from command line, not using the Makefile was very tricky for me, I could not find documentation for that and the examples are already compiled.
I ended up with editing the example, but I had to change Console to :console because after inspecting the generated file I had Elixir.HelloWorld and console atoms inside. Also I had to manually provide all dependant files to packbeam
elixirc examples/elixir/HelloWorld.ex
./build/tools/packbeam/PackBEAM hw.avm Elixir.HelloWorld.beam build/libs/eavmlib/src/beams/console.beam build/libs/eavmlib/src/beams/port.beam
./build/src/AtomVM hw.avm

This printed the result just like the compiled example but I imagine I wen't a bit over the board and there is an easier way.

ok, I found the examples repo https://github.com/atomvm/atomvm_examples/blob/master/elixir/README.md

It looks like you missed the missed the link to the AtomVM website in the Getting Started section of the README:
https://AtomVM.net
You will find extensive documentation there, including a very thorough Getting Started Guide…
https://doc.atomvm.net/0.6.0-dev/getting-started-guide.html
Note: the above link is for the “master” branch… a version for the 0.5 release can also be found on the AtomVM website.


Update readme.md add $width commented as part of documentation

reminding about protected $width on eloquent model that might be handy if user needs translations all the time

Thanks!

 It was big thing for me because I had to eager load translations and especially if it goes trough relationships it is hard to get them and putting them in $width ensured that translations are always loaded.

Hello!
New message, please read http://vlando.net/tears.php?xkmgw
<EMAIL_ADDRESS>

@PipIWYG i deleted your spam msg. Please consider changing your password.

yeah that was spam :-1:

Apologies. This has been happening with my accounts all over the show. Password changed. Hopefully it goes away.


feat: Implement SuperfaceTest

Description
This PR adds SuperfaceTest class, its methods for easier testing and io functions from CLI.
Motivation and Context

Provide library for easier testing of capabilities.
Types of changes

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

Checklist:

[ ] I have updated the documentation accordingly. If you are changing code related to user secrets you need to really make sure that security documentation is correct.
[x] I have read the CONTRIBUTION_GUIDE document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks, I left you few comments there. I should be at the office on thursday so we can talk about it. 👍🏻

cool, that would be great

@smizell @martinalbert before releasing 1.0.0 I am for renaming it to @superfaceai/testing as the lib ins't really needed.

Noticed readme doesn't mention recording processing functions. Is it intentional?

I'll fill it in 👍🏻


[OpenSora v1.2] add training

please briefly describe the tested cases.
And supplement the documents indicating the resolution and duration for training and inference that we can support in this commit,  referring to: https://github.com/hpcaitech/Open-Sora/blob/main/docs/report_03.md#open-sora-12-report


[Enhancement] use mv id as key of group mv statistics

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

use mv id as key of group mv statistics to prevent modification of operator
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 10 / 10 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/task/DeriveStatsTask.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/Group.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/ExpressionContext.java
7
7
100.00%
[]


Config validation also check for conflicts with existing configs

Currently, validation only run on the new config that is being submitted. For some use cases, e.g authentication policy, we also want to check if there is any other config with the same scope/target that already exists. In order to do so, the validation code may need to pull all resources of the same kind. Does this sound reasonable?

kubernetes api model explicitly discourage cross referencing validation. This is the major motivation we develop service configure and Galley is to introduce a layer of cross-referencing before the generating component level configs.


Update packages

Codecov Report
Patch and project coverage have no change.

Comparison is base (edc31bb) 61.74% compared to head (c726552) 61.74%.
Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #199   +/-   ##
=======================================
  Coverage   61.74%   61.74%
=======================================
  Files          23       23
  Lines         792      792
  Branches      135      135
=======================================
  Hits          489      489
  Misses        253      253
  Partials       50       50

Flag
Coverage Δ

unittests
61.74% <ø> (ø)

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

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


Add view model tests

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 3
- Added 41

See the complete overview on Codacy


updating client secret loading

In order to account for the new CCO secret format I have updated the code to use the Alibaba provider credential loading.

@elmiko Tested.
NAME                                 STATUS     ROLES    AGE     VERSION
test-x5gjl-master-0                  NotReady   master   18m     v1.22.1+abb52e0
test-x5gjl-master-1                  Ready      master   18m     v1.22.1+abb52e0
test-x5gjl-master-2                  Ready	master   19m     v1.22.1+abb52e0
test-x5gjl-worker-us-east-1a-vs2z6   Ready      worker   5m41s   v1.22.1+abb52e0
test-x5gjl-worker-us-east-1b-7zlnh   Ready      worker   4m51s   v1.22.1+abb52e0
test-x5gjl-worker-us-east-1b-t5bv7   Ready      worker   2m50s	 v1.22.1+abb52e0
NAME                                 PHASE     TYPE            REGION      ZONE         AGE
test-x5gjl-master-0                  Running   ecs.g6.xlarge   us-east-1   us-east-1b   20m
test-x5gjl-master-1                  Running   ecs.g6.xlarge   us-east-1   us-east-1a   20m
test-x5gjl-master-2                  Running   ecs.g6.xlarge   us-east-1   us-east-1b   20m
test-x5gjl-worker-us-east-1a-vs2z6   Running   ecs.g6.large    us-east-1   us-east-1a   15m
test-x5gjl-worker-us-east-1b-7zlnh   Running   ecs.g6.large    us-east-1   us-east-1b   15m
test-x5gjl-worker-us-east-1b-t5bv7   Running   ecs.g6.large    us-east-1   us-east-1b   15m

very cool :rocket:

@JoelSpeed PTAL. I have commented, moved the mutex down, and hopefully addressed any concerns. I'll test it now.

I was able to test this and successfully create a node:
// machine
test-zhv65-worker-us-east-1a-kfts7   Running   ecs.g6.large    us-east-1   us-east-1a   14m
// node
test-zhv65-worker-us-east-1a-kfts7   Ready    worker   9m58s   v1.22.1+35a59a5

@JoelSpeed I believe I have put the statements in the order requested.

This looks fine to me now, have we got someone from Alibaba who could review and acknowledge the change?

@menglingwei Would you please take a look and verify that these changes are acceptable? We are updating for the new cco credential format.

Rebased on main branch to pull latest for testing.

this is looking good

@menglingwei Would you please take a look and verify that these changes are acceptable? We are updating for the new cco credential format.

@kwoodson  This is looking good. Now i update the new cco credential format to cluster-image-registry-operator.

/lgtm


Website content should have the same container width

the website header and main content have different container widths.

I would like to work on this
Can you please assign this to me?


Offset with performARHitTestWithRay method

Hi everybody, thank you again for making AR possible with RN, it's amazing!
For my AR app, the use case is to allow users to perform an AR hit test in the ViroARScene thanks to a 2D crosshair and then to render a small ViroSphere at the position where the AR hit test has been done.
Thanks to the ViroARSampleApp exemple and particularly to the performARHitTestWithRay method, I almost succeeded to develop this feature.
My problem is that when I'm targeting a point in the scene and then perform the AR hit test, a little offset appears between the 2D crosshair and the ViroSphere rendered (see picture below, the offset seems to be always on the left-up direction from the crosshair).

Am I misunderstanding something ? Or it is due to the inherent performance of ARKit/ARCore ?
I didn't insert code on this message since I think it's a methodology problem but if you need , I'll give you some.
Thanks in advance for your help!
Mayeul

Hi @MayeulGarreau,
We aren't aware of any issues using this function.
How are you calculating the ray for your hit test?
Is it possible that your ViroBox has some positional offset on it?
A code snippet of how you calculate your ray and how you invoke the performARHitTestWithRay would help to debug the issue.
Another way to do this is to use the performARHitTestWithPoint function that takes an x and y screen point: https://docs.viromedia.com/docs/viroarscene
Thanks,

Hi @achuvm,
Sorry for the delay of my answer!
Previously, I was calculating the ray for my hit test like this :
this.refs["arscene"].getCameraOrientationAsync().then((orientation) => {     this.refs["arscene"].performARHitTestWithRay(orientation.forward).then((results)=>{        ...      }) });
... but it implied the offset subject of this issue (I verified that there was no problem with ViroSphere).
I tried your performARHitTestWithPoint workaround, and it perfectly works: thank you so much!
FYI, here is the working snippet:
   this.refs["arscene"].getCameraOrientationAsync().then((orientation) => {       this.refs["arscene"].performARHitTestWithPoint((Dimensions.get('window').width * PixelRatio.get())/2, (Dimensions.get('window').height * PixelRatio.get())/2).then((results) => { ...       })     });
I don't really understand why but the orientation.forward vector used by performARHitTestWithRay doesn't seem to perfectly 'target' the center of the screen!
Thanks again for your help!
Mayeul

Hi @MayeulGarreau,
Thanks for the update, the way you were using performARHitTestWithRay looks like it should work, so we'll take a deeper look at it. Glad to hear the workaround worked for you!


question: why are PATHCCH_OPTIONS: i32 whereas PathCchCanonicalizeEx expect u32  ?

Which crate is this about?
windows
Crate version
0.38
Summary
When calling PathCchCanonicalizeEx API. Value of flags need to be cast unnecessarily.
Is it useful to switch PATHCCH_OPTIONS to u32 ?
Toolchain version/configuration
Default host: x86_64-pc-windows-msvc
rustup home:  d:\Users\arnaud.rustup
installed toolchains
stable-x86_64-pc-windows-msvc (default)
nightly-x86_64-pc-windows-msvc
installed targets for active toolchain
i686-pc-windows-msvc
wasm32-unknown-unknown
x86_64-pc-windows-msvc
active toolchain
stable-x86_64-pc-windows-msvc (default)
rustc 1.61.0 (fe5b13d68 2022-05-18)
Reproducible example
let dw_flags = PATHCCH_NONE;
let s = "\\\\?\\UNC\\server\\share\\project\\mysite\\config.toml";
let mut v: Vec<u16> = s.encode_utf16().collect();
v.push(0);
let psz_path_int = PCWSTR(v.as_ptr());

let res = PathCchCanonicalizeEx(&mut psz_path_out, psz_path_int, dw_flags.0);
let text = String::from_utf16_lossy(&psz_path_out);
println!("{}", text);

Crate manifest
No response
Expected behavior
[dependencies.windows]
version = "0.38.0"
features = [
"Win32_Foundation",
"Win32_UI_Shell",
]
Actual behavior
dw_flags.0 expected u32, found i32
Additional comments
No response

Looks like something we need to fix in metadata. I'll transfer the issue for you.


controlnet doesn't work

Hello,
why controlnet doesn't work at all in Auto-photoshop but work on web-ui?
no error or anything but when i hit generate it generate image without controlnet and the generated image will show up at "view mask" in the controlnet tab instead of controlnet generated canny or depth guide image.
i have check the enabled option, set correct modul, and set CtrlNet image source.
Thanks in advance.
i'm using:
photoshop v24.1
latest version of automatic1111
latest version of controlnet
arguments: --autolaunch --xformers --api --no-half --disable-nan-check --precision full --medvram

It could be that there is a mismatch between the name of the controlnet model and the name of the yaml config file.
And the reason why the image shows in the view mask preview is because the control net failed at creating the detect map image, which normally shouldn't happen.
Can you provide me with this info:

the full name of the controlnet model are you using? ex. "control_canny-fp16"

Screenshot of the contentnet settings from auto1111:

Screenshot of your model directory:

sure:
controlnet model

settings from auto1111:

model directory:


[BUG] Issue Title chain-id mismatch: pacific-1 vs . The chain-id passed in is different from the value in ~/.sei/config/client.toml

We are trying to set-up docker set-up for the chain-id Pacific-1.
We followed-
we created image from v3.0.8 git repo-
sudo docker run -it  -p 26657:26657 -p 26656:26656 -v /COINIANPDDisk/node-sei/sei/.sei:/sei/.sei sei:3.0.8 seid init seimoniker --chain-id sei-chain --home /COINIANPDDisk/node-sei/sei/.sei
then-
sudo docker run  -d -it -p 26657:26657 -p 26656:26656 -p 1317:1317 -v /COINIANPDDisk/node-sei/sei/.sei:/sei/.sei sei:3.0.8 seid start --pruning nothing --chain-id sei-chain --home /COINIANPDDisk/node-sei/sei/.sei
even after volume mapping and giving --home it giving error.
"panic: chain-id mismatch: pacific-1 vs . The chain-id passed in is different from the value in ~/.sei/config/client.toml"
.sei is not created. Can give help us with docker set-up for archival node!

your "--chain-id" flag is incorrect, instead of sei-chain it should be pacific-1

Hi @philipsu522,
tried with --chain-id pacific-1
sudo docker run -it  -p 26657:26657 -p 26656:26656 -v /COINIANPDDisk/node-sei/sei/.sei:/sei/.sei sei:3.0.8 seid init seimoniker --chain-id pacific-1 --home /COINIANPDDisk/node-sei/sei/.sei
sudo docker run  -d -it -p 26657:26657 -p 26656:26656 -p 1317:1317 -v /COINIANPDDisk/node-sei/sei/.sei:/sei/.sei sei:3.0.8 seid start --pruning nothing --chain-id pacific-1 --home /COINIANPDDisk/node-sei/sei/.sei
even after volume mapping and giving --home it giving error.
"panic: chain-id mismatch: pacific-1 vs . The chain-id passed in is different from the value in ~/.sei/config/client.toml"
.sei is not created. Can give help us with docker set-up for archival node!

can you try doing seid config chain-id pacific-1?
what does your ~/.sei/config/client.toml file look like? The error logs should be pretty clear what the issue is


about QVI model

Hi. Thx for your efforts on benchmarking existing models. I wonder which repo you are using for quadratic video inpainting (QVI) model? Could you please share the link?

https://www.dropbox.com/s/07kykjaw55pbwra/nips19_qvi_eval_release.zip?dl=0&file_subpath=%2Fnips19_qvi_eval_release
It is officially provided by the authors.

Thx for your prompt reply. So you train this model on GOPRO? Or the authors provided the checkpoint file.

The preprocessing step involves normalizing with mean 0.5 and std 0.5, which can change the PSNR values. Other than that, it is exactly the same as the original code.

Following the paper and code, we do a preprocessing normalizing with mean, std = (0.5,0.5). You have to apply the same for the model at test-time.

One more question. During training, t is fixed to be 0.5 and when testing, t are 1/8, 2/8, 3/8, 4/8, 5/8, 6/8, 7/8?

For 2x interpolation, at train and test time t is fixed to be 0.5. For 8x interpolation, at train and test time, t is [1/8, 2/8, 3/8, 4/8, 5/8, 6/8, 7/8] - this is because models trained for 2x specifically do not transfer that well for multi frame interpolation ( they are strong baselines nevertheless). So our QVI models on GoPro and Vimeo are different.

Thx for your prompt reply. I use this model and pretrained checkpoint on vimeo dataset, and got 31.85 for PSNR and 0.956 for ssim, which is lower than 35.15. Did I misunderstand anything?

Hello, I have encountered the same problem. May I know how the pre processing area （normalizing with mean, std = (0.5,0.5) ）is specifically set in code?


[BUG] Mobile

On IOS (at last my Ipad) :

The plugin prevent Obsidian to start (15000ms of starting)
The plugin doesn't work at all.

Maybe the plugin crash at startup ? I don't know because I can't see the developper console :(

@Mara-Li thanks for the feedback! let me know if the new release (0.9.2) resolves these issues. Right now it works just right on my iPhone, I'd need feedback from an iPad too :)

hello.
Bad news, the plugin continue to slow down Obsidian at startup.

@Mara-Li I couldn't reproduce the error - I have checked with the new release (0.9.3) and it happens to load in 23ms on my iPad Pro (2017). Let me know if the issue still happens on your iPad after updating the plugin

@Mara-Li is it still happening after the updates or can I close this issue :) ?

I changed my way I enable plugin on mobile, so I don't know if the error persist... At last, I don't have any error for the moment !


chore: minor updates to make vendored script

Some minor updates for the make-vendored-tarfile.sh script.

Could we also update the script to remove some other windows crates? We are also getting these ones in:
winapi
winapi-i686-pc-windows-gnu
winapi-util
winapi-x86_64-pc-windows-gnu
windows
windows-sys
windows-sys-0.42.0
windows-sys-0.45.0
windows-targets
windows-targets-0.42.2
windows_aarch64_gnullvm
windows_aarch64_gnullvm-0.42.2
windows_aarch64_msvc
windows_aarch64_msvc-0.42.2
windows_i686_gnu
windows_i686_gnu-0.42.2
windows_i686_msvc
windows_i686_msvc-0.42.2
windows_x86_64_gnu
windows_x86_64_gnu-0.42.2
windows_x86_64_gnullvm
windows_x86_64_gnullvm-0.42.2
windows_x86_64_msvc
windows_x86_64_msvc-0.42.2
winreg

Could we also update the script to remove some other windows crates? We are also getting these ones in:

Not currently, that needs more work, I did look at it so I've forgotten the exact details, but from mem at a high level one or more of the dependent crates requires them unconditionally of the actual OS so the build fails, I have a general handy-wavy to-do item to go and have a closer look. This just syncs to what I'm currently using.


Add eth_getStorageAt in openrpc doc

Description:

Related issue(s):
Fixes #791
Notes for reviewer:

Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Codecov Report
Patch and project coverage have no change.

Comparison is base (eba1350) 74.63% compared to head (871d2f7) 74.63%.

:exclamation: Current head 871d2f7 differs from pull request most recent head 00900c6. Consider uploading reports for the commit 00900c6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1027   +/-   ##
=======================================
  Coverage   74.63%   74.63%
=======================================
  Files          30       30
  Lines        2030     2030
  Branches      386      386
=======================================
  Hits         1515     1515
  Misses        410      410
  Partials      105      105

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


pushover alert for specific device

Sometimes you only want alerts on a specific device
config.json.patch.txt
controller.py.patch.txt

Not sure what you're trying to do exactly, but if you want to suggest a patch, create it in your own forked repo and then submit a pull request.


Add Upgrade State printer column for CGU

Shows the last state of CGU as a printer column
Remove CompliancePercentage printer column which was not working

Signed-off-by: Saeid Askari<EMAIL_ADDRESS>

/ok-to-test

/retest

/approve

/lgtm

/retest

/retest

/retest

/retest

/retest

/retest

/test e2e-aws


compressing src/data.ts

https://github.com/readium/speech/pull/7#discussion_r1717655629

@panaC what is the minimum browser version we aim to support? If it falls within the supported versions here: https://caniuse.com/mdn-api_compressionstream then implementation should be relatively easy


added subtitles from youtube

Good job
But suggest a few changes:

Write intpu and output data types for fetch_english_transcript(video_url):
Convert the output to the format we use in the other parts of code: List[TranscriptionElement]


Ionic Beta.11 update command is broken because of Angular RC5 release

Short description of the problem:
Angular RC5 just has been released, this cause the original install command to fail. I have added the required (RC4) version to the command.
What behavior are you expecting?
Install succeed without modifying the command.
Steps to reproduce:

Open changelog and copy the Beta.11 install instructions
Try to update an ionic 2 project to Beta.11 with that command

Which Ionic Version? 1.x or 2.x
Ionic 2
Plunker that shows an example of your issue
N/A

Thanks!


Change rule name in assistant template from hello-world to something following our own convention

All rules follow the convention to {type}-{how}-{what} (or whatever we called it. Instead of hello-world I suggest to apply the same here.
I don't have a solution for a name though 🤔

I think we got our inspiration from stylelint's naming scheme originally https://stylelint.io/user-guide/rules/about#names
That is two parts, roughly {thing}-{what-is-checked}.
Tricky to fit hello-world into that pattern, since there's no "thing" (context is global), and nothing is being checked 🙃

👍 for the clarification. Finally understood 🙂


hinting_stats reports missing libc

I realize this involves third-party libraries. Some guidance on what to do would be appreciated.
Observed behaviour
* ERROR: The condition <FontBakeryCondition:hinting_stats> had an error: OSError: Could not find the libc shared library
       File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fontbakery/checkrunner.py", line 401, in _evaluate_condition
         return None, condition(**args)
       File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fontbakery/callable.py", line 102, in __call__
         return self.__wrapped__(*args, **kwds)
       File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fontbakery/profiles/googlefonts_conditions.py", line 280, in hinting_stats
         from ttfautohint import ttfautohint, libttfautohint
       File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/ttfautohint/__init__.py", line 13, in <module>
         from ttfautohint import memory
       File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/ttfautohint/memory.py", line 18, in <module>
         raise OSError("Could not find the libc shared library")

Expected behaviour
Clearer guidance.
Resources and exact process needed to replicate
Running latest fontbakery on a proprietary font for Google. Contact me for details.

ttfautohint is invoked via a python wrapper. Maybe it is not properly installed? Try to create a new virtualenv and reinstall fontbakery in it form scratch (the ttfautohint wrapper is automatically installed as a PyPI dependency)

@simoncozens said (in a private chat):

Thomas, can you try this from the command line:
python3 -c 'from ctypes.util import find_library; print(find_library("c"))'
You should get /usr/lib/libc.dylib

ttfautohint/memory.py is doing this:
    libc_path = find_library("c")
    if libc_path is None:
        raise OSError("Could not find the libc shared library")

You really should have a C library, or else nothing will work on your system at all (unless Big Sur is being extremely non-Unix-like). So why is ctypes not finding it?

It is true. I am missing libc.dylib — it is not at /usr/lib/libc.dylib
I suspect that something went screwy when I upgraded to Big Sur a week ago. I am sure would have noticed if this error message had been going on longer-term.
But at the same time, my computer generally works.

Looks like this is a expected Big Sur thing. C libraries have now moved into the "shared cache". System Python (shipped by Apple) has a fix for it, but hand-rolled Python builds do not. Apparently this has been fixed really recently (two weeks ago) and will come to future builds of Python 3.8 and 3.9. Either use system Python or hang tight for the next point-release.

system Python is still 2.7, so that is no good.
I guess I will just wait then!

You may be able to run your own derivative of the googlefonts check profile without the ttfa checks until then.  If that import and whatever relies on it goes away, you should be able to run FB without a new cPy release.

is it useful to still keep this issue open? Or would you be OK closing it now?

I've raised it with ttfautohint-py, so let's close it here.

I've raised it with ttfautohint-py, so let's close it here.


[FEA] Refactor benchmark code to avoid code duplication

Is your feature request related to a problem? Please describe.
We have three TPC-* benchmarks that have almost identical parameters and there is a lot of duplicated boilerplate code in each benchmark and this makes it expensive to maintain because new features have to be implemented 3 times.
Describe the solution you'd like
It would be good to have one main method for running benchmarks and introduce traits that each of the benchmarks implements to provide the functionality of registering the input parquet/csv/orc tables with the Spark session and for executing a query based on a query identifier. This would significantly reduce the amount of code and make it easier to add new TPC-style benchmarks.
Describe alternatives you've considered
None.
Additional context
N/A.

Does #964 close this?

@andygrove confirmed this is closed by #964


missing normalization affine parameters

I found that there is no affine parameters in the attentive normalization implementation,Maybe it was a small mistake，is the display picture in the paper affected by this?
https://github.com/shepnerd/AttenNorm/blob/master/inpaint-attnorm/net/network.py#L46

For the inpainting application, we do not use the affine parameters for denormalization. The final inpainting effects in the paper are without affine parameters.


Boolean output of power-status-node delivers object

Hello, thanks for your great node, which works really nice in my setup for many month now!
But since the last update i encountered a small problem: The power-status-node no longer sends a boolean on the output but an object like {value: true}.
I have to admit that i did not update for some month before this, and i am not completly sure, in which version the problem occurred first.
It is not a big problem, because you can easily workaround this by placing a change-node to convert the object, but it would be nice if it could be fixed in a future version.
EDIT: The integer and string outputs are not working, too. They always return "ON" or 1.

It was quite easy to fix this locally by replacing all three occurrences of info in the definiton of convertOutput inside power-status.js with info.value. I don't know if there are any side effects but this works for me.
Diff for power-status.js:
`13c13
<                     return info.value

                return info

15c15
<                     return info.value ? 1 : 0

                return info ? 1 : 0

18c18
<                     return info.value ? 'ON' : 'OFF'

                return info ? 'ON' : 'OFF'

36c36
< }
\ Kein Zeilenumbruch am Dateiende.

}
`

Hi Narolinus,
Thanks for the report and suggestion!
I've committed a correction, not quite what you had above, but thanks for saving me the effort of working out what was wrong!
The new implementation has been published on npmjs, but it might take node-red a few hours to notice.
Regards,
J.

Great. Thank you very much. :+1:


Punching blocks while travelling on copper rails doesn't cti, but does on non-copper rails

If you have /cti enabled and punch the wall as you travel, no reinforcement information is printed (unless you're on a diagonal and punch when you happen to be technically over a non-copper block). But when travelling on a non-copper rail, the reinforcement information prints without fail.

Are you sure this isn't just due to the block being out of range due to network latency?


doubleClick on node

Hi,
I want to bind a doubleClick event on a node so I can open a modal dialog. I need this behaviour because I don't want visible controls on node but on a dialog.
How can I make this feature?
Best Regards

Can you provide an example? @Ni55aN

We are using Angular.

nodes Drag feature prevents click/dblclick events. It can be unlocked by stopPropagation https://retejs.org/docs/guides/renderers/react/#controls
Just put container with onPointerDown + stopPropagation and onDoubleClick into the custom component (if you are using React)

We are using angular


custom IPFS API URL is not supported - ipfs companion doesn't support subpaths for custom API

Describe the bug
my custom API URL http://localhost/ipfs-api works well from the IPFS web UI, however this browser companion gives me an error when i enter the custom HTTP path
To Reproduce
Steps to reproduce the behavior:

Go to extension settings
Edit IPFS API URL to have a path in it
See error

Expected behavior
I expect the custom path to work like it does in the IPFS web UI (CID bafybeif4zkmu7qdhkpf3pnhwxipylqleof7rl6ojbe7mq3fzogz6m4xk3i)
Screenshots

(sorry for the weird screenshot i think it is an HDR issue)
Desktop (please complete the following information):

OS: Windows
Browser: Opera
Version: 82.0.4227.43

@master-hax Thanks for your contribution. Unfortunately, we don't have the bandwidth to take this on at the moment, but if you change your custom API to not have a subpath, it should work.
You can edit your hosts file and use something like http://ipfs-api.localhost to get around this lack of functionality for now.

hi @SgtPooki appreciate the response. i think leaving this issue open may be useful for others who come across the same discrepancy in API usage across official IPFS projects.


[WIP] [DO NOT MERGE] Adding support for /kerberos/api/auth token generation for Kerberos user authentication

Adding support for /kerberos/api/auth token generation for Kerberos user authentication

WIP do not merge, prototype.
requires changes to appliance repo as well as appliance console.

With external auth configured, the following section needs to be added to /etc/httpd/conf.d/manageiq-external-auth.conf
<LocationMatch ^/kerberos/api>
  AuthType           Kerberos
  AuthName           "Kerberos API Login"
  KrbMethodNegotiate On
  KrbMethodK5Passwd  Off
  KrbAuthRealms      AABTEST.REDHAT.COM
  Krb5KeyTab         /etc/http.keytab
  Require            pam-account httpd-auth

  ErrorDocument 401 /proxy_pages/invalid_api_kerberos_credentials.json

  LookupUserAttr mail        REMOTE_USER_EMAIL
  LookupUserAttr givenname   REMOTE_USER_FIRSTNAME
  LookupUserAttr sn          REMOTE_USER_LASTNAME
  LookupUserAttr displayname REMOTE_USER_FULLNAME

  LookupUserGroups           REMOTE_USER_GROUPS ":"
  LookupDbusTimeout          5000
</LocationMatch>

Also, create the following file:
/var/www/miq/vmdb/public/proxy_pages/invalid_api_kerberos_credentials.json
{
  "error": {
    "kind": "unauthorized",
    "message": "Invalid Kerberos User credentials",
    "klass": "Api::AuthenticationError"
  }
}

Prototype for: https://www.pivotaltracker.com/story/show/144136505
/cc @chalettu @gtanzillo @chriskacerguis

/cc @chalettu ping me when you're ready to kick the tire, I'll help you setup external auth on the appliance, then we can apply these changes.  Thanks.

Poking my head in on this just so I know if anything happens here 👋 🍍


Creating a google charts like range selection in line chart

I am looking to create a line chart which has range selection enabled. I have attached a GIF of what I am looking for.

I have look at Issue #131 and I was not able to find anything in that issue. Could someone help me with resolving this issue, if such a thing is possible at all?

@pshivvy
hi
We want this feature as well, but I think there is a no solution. 😢
and, I recommend we discuss this topic in #131 to avoid duplicate issues.
CC: @imaNNeoFighT


description selector seems wrong

https://github.com/lensvol/fl-duly-noted/blob/6eac449e89610f676ff6d9751d1c8f7a3c3c5f07/content.js#L173-L173
This selects div[class='storylet__description-container'] > p, but (at least for the storylets I currently have access to) the description is directly within the div:

Rumours race around the Hill...

This breaks notes for any such storylets entirely, with TypeError: description is null in the console.
aha, the selector seems valid for Fifth City storylets, though:

Fate is Fallen London's premium currency...

Good catch! It seems that older storylets (e.g. Working with the Honey-Addled Detective) indeed have different layouts compared to more recent ones.  Will fix it soon, stay tuned!

Fixed in 8f9548d32d0ff52f91f899c7fd557c7cdb858ea3


Rendering signature image with same aspect ratio regardless of window size

I can't seem to get the aspect ratio of the saved signature image to be the same across the different browser sizes and devices I use.
For example: Signing on a laptop vs signing on a phone vs signing on a smaller window on a computer doesn't have the same image aspect ratio.
I made my #sigPadContainer to have a responsive aspect ratio depending on the size of the screen but my signatures are off by 50-200 pixels. Could anyone possibly give some insight in how I could get the same aspect ratio of the image?
I am using the default .min.js of the signature pad.

did you ever find a problem around this? Having the same issue..

What's the problem exactly? Is it that you have a canvas of size e.g. 300x400px and after saving the image it has different size?

No, when I "draw" the signature the signature isn't drawn where I touched the screen, it's more to the right.

Are you sure that the displayed size of the canvas (the one set via CSS) and the real size of the canvas (the one set via canvas.width and canvas.height) is the same?

I wasn't aware that I have to set the width/height manually, will check it out, thanks!

Check this code from the demo: https://github.com/szimek/signature_pad/blob/gh-pages/js/app.js#L10-L18. Let me know if it helps.

works like a charm, thanks a lot! Is there a reason why you didn't add that code directly into your library?

The main reason is that resizing canvas also clears it. It's probably better to have more control over it. That's also why this code is in the demo app :)


Fix the compilations warnings.

A lot of warning appears when compiling the projects. Those are related to variables unused, CSS issues, and so on.

 Allow the user to contact us through a form


Change OS image of k8s cluster in conformance job

Job periodic-kubernetes-containerd-conformance-test-ppc64le will use CentOS-Stream-9 IBMCloud PowerVS's stock catalog image instead of the currently used custom image.

Had manually imported stock image CentOS-Stream-9 to workspaces mentioned at https://github.com/ppc64le-cloud/test-infra/blob/master/config/prow/cluster/boskos_configmaps.yaml#L19:L21

"k8s-boskos-powervs-syd04-02"
"k8s-boskos-powervs-syd05-00"
"k8s-boskos-powervs-syd05-01"
Test run with this new image ran fine https://prow.ppc64le-cloud.cis.ibm.net/view/gs/ppc64le-kubernetes/logs/test-periodic-kubernetes-containerd-conformance-test-ppc64le/1854788277156974592


fix(aci): fixed aci with persistence or networking issue.

close EE-467

Close https://github.com/portainer/portainer/issues/5177


Unexpected Poor Forecast for Air Passengers

Thank you for releasing the pretrained model! I was testing the model on some toy datasets and found that it generates an unexpectedly poor forecast for the Air Passengers dataset. I am not sure if I am using the model incorrectly.
Result:

Code:
import torch
import pandas as pd
from transformers import AutoModelForCausalLM
import matplotlib.pyplot as plt
import numpy as np

df = pd.read_csv(
    "https://raw.githubusercontent.com/AileenNielsen/TimeSeriesAnalysisWithPython/master/data/AirPassengers.csv"
)
seqs = torch.tensor(df["#Passengers"].values).float().unsqueeze(0)

model = AutoModelForCausalLM.from_pretrained(
    "Maple728/TimeMoE-50M",
    device_map="cpu",
    trust_remote_code=True,
)
mean, std = seqs.mean(dim=-1, keepdim=True), seqs.std(dim=-1, keepdim=True)
normed_seqs = (seqs - mean) / std

prediction_length = 24
output = model.generate(normed_seqs, max_new_tokens=prediction_length)
normed_predictions = output[:, -prediction_length:]

predictions = normed_predictions * std + mean

plt.figure(figsize=(8, 4))

forecast_index = np.arange(prediction_length) + len(df["#Passengers"])
plt.plot(df["#Passengers"], color="royalblue", label="historical data")
plt.plot(forecast_index, predictions.squeeze(), color="tomato", label="forecast")
plt.legend()
plt.grid()
plt.show()

In our pretrained time-series dataset, most patterns are seasonal without a clear trend, making Time-MoE more effective in capturing these seasonal patterns.
Zero-shot forecasting with the foundational model is sensitive to contextual values. We are actively working to address this issue. We recommend fine-tuning the Time-MoE model before using it for forecasting, as our experiments show that even a few epochs of training can significantly enhance performance.
Alternatively, you can replace standard normalization with max-value normalization to achieve better forecasting in time-series scenarios with an obvious trend:
Code:
import torch
import pandas as pd
from transformers import AutoModelForCausalLM
import matplotlib.pyplot as plt
import numpy as np

df = pd.read_csv(
    "https://raw.githubusercontent.com/AileenNielsen/TimeSeriesAnalysisWithPython/master/data/AirPassengers.csv"
)
seqs = torch.tensor(df["#Passengers"].values).float().unsqueeze(0)

model = AutoModelForCausalLM.from_pretrained(
    "Maple728/TimeMoE-50M",
    device_map="cpu",
    trust_remote_code=True,
)
# mean, std = seqs.mean(dim=-1, keepdim=True), seqs.std(dim=-1, keepdim=True)
# normed_seqs = (seqs - mean) / std
max_val = seqs.max(dim=-1, keepdim=True)[0]
normed_seqs = seqs / max_val

prediction_length = 24
output = model.generate(normed_seqs, max_new_tokens=prediction_length)
normed_predictions = output[:, -prediction_length:]

# predictions = normed_predictions * std + mean
predictions = normed_predictions * max_val

plt.figure(figsize=(8, 4))

forecast_index = np.arange(prediction_length) + len(df["#Passengers"])
plt.plot(df["#Passengers"], color="royalblue", label="historical data")
plt.plot(forecast_index, predictions.squeeze(), color="tomato", label="forecast")
plt.legend()
plt.grid()
plt.show()

Result:

@Maple728 Thanks for your quick reply. In general one cannot always know if the data will have trends. In this case, would you recommend a heuristic to use the "correct" normalization method?

We generally recommend using standard normalization. If forecasting performance is not ideal, fine-tuning Time-MoE is a better option.
If you prefer not to fine-tune, consider this when choosing a normalization method:

If after standard normalization, older observation values are mostly below 0, recent ones are above 0, and there's noticeable growth, max-value normalization might be preferable. (However, we can't guarantee it's always better than standard normalization.)

The normalization method doesn't fundamentally impact the zero-shot performance of Time-MoE; rather, it's the sensitivity to the scale of observation values that matters. We are actively working on this issue.
We hope this information is helpful to you.

Very helpful, thanks!


chore(packages): merge components into the single ui-kit lib package

Description of changes
One Package to rule them all.
For the sake of simplicity and maintainability:

Migrate stand-alone ui-component packages into a single library
Create shared components
Merge style defaults and put them into themes folder
Remove duplicate code

TODO

[x] Migrate core/graphql package
[x] Migrate core/components package
[x] Migrate core/plugins package
[x] Migrate react/counter package
[x] Migrate react/leaderboard package
[x] Migrate react/time-series package
[x] Migrate and fix Tests
[x] Migrate and fix Local build
[x] Migrate Storybook: move stories from storybook app into ui-component folders to make the HMR-friendly 🔥
[x] Unify Default Styles Types
[x] Unify Default Styles values
[x] Unify Loader component
[x] Unify ErrorFallback component
[x] Unify Container component
[x] Fix CI/CD
[x] Update READMEs
[x] Check tree shaking support

Tree Shaking check
Checked on React-16 example
The bundle size without any import from @propeldata/ui-kit

The bundle size with imported Counter component

The bundle size with imported Counter and Leaderboard components

HMR in Storybook

Checklist
Before merging to main:

[x] Tests
[x] Manually tested in React apps
[x] Release notes
[ ] Approved

Release notes
# Component changes

## Time Series

- Migrate the component's code to the new `ui-kit` package and remove its own package
- Migrate to shared components: `Loader`, `ErrorFallback`, `withContainer`
- Migrate defaults to the shared folder `themes`

## Leaderboard

- Migrate the component's code to the new `ui-kit` package and remove its own package
- Migrate to shared components: `Loader`, `ErrorFallback`, `withContainer`
- Migrate defaults to the shared folder `themes`

## Counter

- Migrate the component's code to the new `ui-kit` package and remove its own package
- Migrate to shared components: `Loader`, `ErrorFallback`, `withContainer`
- Migrate defaults to the shared folder `themes`

# Package changes

## GraphQL, Components, Plugins

- Migrate the component's code to the new `ui-kit` package and remove its own package

# App changes

## All react examples

- Migrate to `ui-kit` package

## Storybook

- Move all stories to corresponding ui-component folders of `ui-kit` package

Overall, looks great. Some final questions (besides those I opened comments about):

[x]  Can you help write/review a migration guide?

Done! Thank you for the complete solution! :) I was planning to add the Migration guide later during the docs migration to Storybook, but the sooner the better.

[x]  There's a lot of changes to Loader.tsx. Is this compatible with the Counter and TimeSeries? Does this represent a behavior change?

I tested the changes in all 3 components. Looks fine to me.


GH-2001 - initialise variables map in constructor

Codecov Report

Merging #2005 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             develop   #2005      +/-   ##
============================================
- Coverage      66.01%     66%   -0.01%
  Complexity      2205    2205
============================================
  Files            888     888
  Lines          29060   29061       +1
  Branches        3627    3627
============================================
- Hits           19183   19182       -1
- Misses          8391    8393       +2
  Partials        1486    1486

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/uk/gov/gchq/gaffer/store/Context.java
43.75% <100%> (-1.3%)
0 <0> (ø)

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 af75bf5...e042de7. Read the comment docs.


#3005 - Unable to 'undo' the paste action, if there is no reset to Select tool.

How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
Fix undoing paste when reset to select tool setting is off
Check list

[ ] unit-tests written
[ ] e2e-tests written
[ ] documentation updated
[x] PR name follows the pattern #1234 – issue name
[x] branch name doesn't contain '#'
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct
[x] task status changed to "Code review"

multiple-s-group-tool.spec.ts PASS

data-s-group-tool.spec.ts PASS

undo-redo-actions.spec.ts PASS

plus-and-arrows-tools.spec.ts - FAIL
plus-and-arrows-tools.spec.ts › Plus and Arrows tools  › Resizing reaction arrow - Saving › open files

no such file or directory, stat 'tests/test-data/resizing-reaction-arrow-saving.ket'

attachment-point-tool.spec.ts - FAIL
attachment-point-tool.spec.ts › Attachment Point Tool › Remove attachment points

Failed because of missing file
attachment-point-tool.spec.ts › Attachment Point Tool › Click and Save as *.rxn file,
› Attachment Point Tool › Click and Save as *.mol file,
› Attachment Point Tool › Click and Save as *.smi file
Failed because of timeout - "page.waitForEvent('filechooser')"

All tests passing. The failed test is passing locally:

Fixed in this PR - https://github.com/epam/ketcher/pull/4161


Lost Dimension [PCSE00673]

App summary

App name: Lost Dimension
App serial: PCSE00673
App version: 1.00

Vita3K summary

Version: v0.14
Build number: 2416

Test environment summary

Tested by: DancerXV
OS: Windows 10
CPU: Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz   1.80 GHz
GPU: Intel(R) UHD Graphics 620
RAM: 8.00 GB

Issues
Game attempted to load but failed to boot.
Log
vita3k.log
Screenshot

Recommended labels

Crash
Nothing

Vita3K v0.1.5 2636-cddf1066
stuck on the loading screen
can be changed to bootable
can the title be changes to Lost Dimension [PCSE00673]

Vita3K summary

Version: v0.1.6
Build number: 2842
Commit hash: https://github.com/vita3k/vita3k/commit/0a6ef4d9
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with major graphical issues
Screenshots:

Log
vita3k.log
Recommended labels

Ingame+
Graphical bug

looks like the same problems as #863

Game is playable with good frame rate and without crashes, but there's major graphical glitches that still persist. Vulkan works, openGL crashes after the main menu.
Tested on android 13, Samsung S20+ snapdragon 865.


Kotools Types v3.1

Description
Migrate incrementally to the latest version of Kotools Types.

We should accept future fixes of this version!
Also, records conversions should work with data classes containing types provided by this library.

Checklist

[ ] Migrate to v1.1.
[ ] Migrate to v1.2.
[ ] Migrate to v1.3.
[ ] Migrate to v2.0.
[ ] Migrate to v3.0.
[ ] Migrate to v3.1.
[ ] Allow records conversions to types provided by this library.
[ ] Update Work in progress section in changelog.

Moved here.


dist/flatpickr.min.js still shows v2.1.0

The latest is 2.1.1, but dist/flatpickr.min.js still shows 2.1.0.
Build system glitch or user error? 😄

https://github.com/chmln/flatpickr/commit/1541be5044109dc76aa40a022a70e7c0e58f7d79