[Migration] New database tables required for organization management feature from IS6.0.0

Describe the issue:
It is required to created a set of additional tables, in both Identity and Shared databases, to use the Organization Management feature. The following links direct to the database table creation queries that should be run on the corresponding database type
Identity Database
Tables added:SP_SHARED_APP

DB2
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/db2.sql#L1720-L1743

H2
https://github.com/wso2/carbon-identity-framework/blob/a86261eeb41adab8b90858e2005231997de1aba0/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/h2.sql#L1172-L1183

MsSQL
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/mssql.sql#L1301-L1314
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/mssql.sql#L1412-L1413

MySQL Cluster
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/mysql-cluster.sql#L1339-L1350

MySQL
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/mysql.sql#L1197-L1208

Oracle
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/oracle.sql#L1790-L1813

Oracle RAC
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/oracle_rac.sql#L1778-L1801

PostgreSQL
https://github.com/wso2/carbon-identity-framework/blob/1bbcd706b7e1b97babab6534ae1f0b68fc6a8175/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/postgresql.sql#L1391-L1405

Shared Database
Tables added: UM_ORG , UM_ORG_ATTRIBUTE , UM_ORG_ROLE , UM_ORG_PERMISSION , UM_ORG_ROLE_USER , UM_ORG_ROLE_GROUP , UM_ORG_ROLE_PERMISSION , UM_ORG_HIERARCHY

DB2
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/db2.sql#L964-L1063

H2
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/h2.sql#L501-L592

MsSQL
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/mssql.sql#L688-L784

MySQL Cluster
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/mysql_cluster.sql#L482-L557

MySQL
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/mysql_cluster.sql#L482-L557

Oracle
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/oracle.sql#L975-L1075

Oracle RAC
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/oracle_rac.sql#L963-L1063

PostgreSQL
https://github.com/wso2/carbon-kernel/blob/d596bae109795d3cf3d31dca31741a29c8a5136c/distribution/kernel/carbon-home/dbscripts/postgresql.sql#L638-L727

Changes made in https://github.com/wso2/carbon-kernel/pull/3375 need to be added to the migration client scripts.


GPU selection [CH59445]

@alexcombessie I put the try import mxnet on top to display an understandable error message when mxnet-cuXXX is used on an instance without Cuda instead of having the obscur <class 'OSError'> : [Errno None] None: 'None'

@alexcombessie I put the try import mxnet on top to display an understandable error message when mxnet-cuXXX is used on an instance without Cuda instead of having the obscur <class 'OSError'> : [Errno None] None: 'None'


Add --cache option for faster linting

Feature request
Is your feature request that we implement a new rule?
No
Is your feature request related to a problem? Please describe.
We just moved our project from JS + Eslint to TS + Tslint, and now every time we run tslint --project tsconfig.json in our project's folder It's taking about 40 sec for not so big project (266 .ts files). Before it was taking 2-4 sec with eslint --cache. We run linter on every git push so it's a bit annoying to wait every time.
Describe the solution you'd like
In eslint there is a --cache option that when passed will create a .eslintcachejson file that contains mtime of every file he checked last time. On next run it will not check the file again unless mtime changed.
Additional context
The .eslintcache file looks like the following:
{
  "/Users/Admin/Code/my-project/foo.js": {
    "size": 2283,
    "mtime":<PHONE_NUMBER>740,
    "hashOfConfig": "rlkggo",
    "results": {
      "filePath": "/Users/Admin/Code/my-project/foo.js",
      "messages": [],
      "errorCount": 0,
      "warningCount": 0,
      "fixableErrorCount": 0,
      "fixableWarningCount": 0
    }
  },
  "/Users/Admin/Code/my-project/bar.js": {
    "size": 2449,
    "mtime":<PHONE_NUMBER>404,
    "hashOfConfig": "rlkggo",
    "results": {
      "filePath": "/Users/Admin/Code/my-project/bar.js",
      "messages": [],
      "errorCount": 0,
      "warningCount": 0,
      "fixableErrorCount": 0,
      "fixableWarningCount": 0
    },
  },
  ...

This is a little harder with typed rules, as changes to one file may cause lint complaints in another file. For example, adding a @deprecated tag can cause other files to complain on the deprecation rule.

As @JoshuaKGoldberg  mentions, this is a pretty difficult thing to implement as you would also have to cache a graph of dependencies, such that if you modify one file, you then have to re-compile all files that depend on that file, and then all files which depend on that file, etc. For large codebases with many branches I could still see some benefit here.

Given the lack of activity on this issue and the upcoming TSLint deprecation in #4534, this seems rather out of scope now. 😢


Crash under certain conditions

vim version: 25065ec375a8a55462f6c07c76dc1a72a770ac19
valgrind log: here

Thanks for the report, and sorry for the long hiatus! I'm not sure about the Python runtime complaints, but at least some of those invalid reads are #25.


Improve Store Buttons/src/store-button-google-play-lightmode.html

Need to improve this component on Store Buttons/src/store-button-google-play-lightmode.html
Improvements required :
remove border and make in three different size
Any reference image?

Any external reference?

You can checkout this one for better understanding.

@nik132-eng  go ahead


Accept vec of packages in uv tool uninstall

Summary
Follow up https://github.com/astral-sh/uv/pull/7037#discussion_r1744515370
Test Plan
cargo test

Thanks for following up! I appreciate it.


Adding comments to support v4 and v6 subdomains

I was able to make v4 and v6 subdomains work in my environment using the documentation I modified in this PR.

This works if you have complete control over DNS. However, due to its popularity https://ifconfig.co is behind Cloudflare now which doesn't allow this configuration.


Finish testing wrappers

The coverage is 100% due to the usage of these decorators throughout the code base. It would still be better to have some more tests for simple cases.

Module was renamed when API changed. Changes already present in the Ilastik workflow so did not break anything.

This is complete in master.


dump_obj or equivilant

As suggested by @dmontagu at https://github.com/samuelcolvin/pydantic/pull/958#issuecomment-554670739 we should have a way to get a model's data which accounts for custom root types (e.g. like .json() without the JSON encoding).
This would probably rely on #1001, and might have to wait for v2 anyway.
Options:

dump_obj as suggested by @dmontagu, maybe just dump() would be preferable.
unpack_custom_roots argument to .dict() as suggested by @dmontagu, problem is that in this case dict() wouldn't necessarily return a dict which seems confusing to me, could be overcome with @overload i think.
.dict() respect custom root types same as .json(), same problem as 2 but can @overload work based on a class attribute?

Overall I think I prefer 2.

Is there any existing solution that allows me to convert a Pydantic object to a json serializable python dict? Currently I'm serializing the pydantic object to a json string and deserializing it back into a python dict.

@theFong Does .dict() not work for you?
@samuelcolvin I'm not sure how a standalone dump_obj function in pydantic.tools would factor into these options. (I'm planning to work on that now for the sake of improving FastAPI performance.)

My model has datetime and Enum attributes. Discovered use_enum_values = True which solved some of my use cases but it appears that .dict() does not serialize datetime into strings (probably by design).

@theFong I see. If you are using fastapi you can use fastapi.encoders.jsonable_encoder(model) -- that will create an object that is json encodable. I think you can also use encoder=pydantic.json.pydantic_encoder as a keyword argument to json.dumps.

I believe this will be closed by https://github.com/pydantic/pydantic/pull/5765


feat: A nice animation for the onboarding

Hi everyone,
I have tried what we can do with Rive and the gyroscope tonight.
For example, the onboarding, could be way less static than today.
First, the sun is an animation from Rive.
And second, have a look at hills, where they move as I shake my device (thanks to the gyroscope).
Here is a video: https://github.com/openfoodfacts/smooth-app/assets/246838/e3e20b63-9e92-4ca0-bb8b-4e0685cd075d
I don't know if it's worth a merge, but clearly, that is the kind of small thing we can do to make the app more friendly.

Codecov Report

Merging #4311 (ee7f999) into develop (1cd22a6) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #4311      +/-   ##
===========================================
- Coverage    10.80%   10.77%   -0.03%
===========================================
  Files          287      287
  Lines        14202    14235      +33
===========================================
  Hits          1534     1534
- Misses       12668    12701      +33

Impacted Files
Coverage Δ

...oth_app/lib/pages/onboarding/reinvention_page.dart
0.00% <0.00%> (ø)

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

The hills, how shall I put it ? 🍑

The hills, how shall I put it ? 🍑

I'm shocked, you have one of those sick minds 🙀


Constants - Spring Cleaning

name: Constant Clearing
about: Combining all the seasonal bot chat constants to one chat
Closes #192

This PR removes any season-specific channel IDs and combines them into one seasonalbot_chat constant in Channels.
This also changes any cogs which previously used the aforementioned channel IDs
Hope this is okay :D

Should we add the role_ids in the list of clean ups as well?

For example?
I think we're fine to keep the seasonal roles encapsulated as seasonal constants.


png to svg

I would like to replace the icons in the assets from png to svg (react-native-svg). What do you think about it?

I don't think this would be a good change since this requires an added dependency to the library. If using react-native-svg with .svgs suits your use case, you should use a forked version of react navigation. I'm not a fan of adding react-native-svg as a dependency for a few reasons:

Doesn't improve the navigation experience for a significant amount of use cases

Requires linking or manual installation

Differences in minor updates to react-native-svg only support specific RN versions.

If you do think this would be a good change, you're welcome to open a PR with the change and include a write up about your motivation behind the change.


Missing doc about Kerberos HTTPS setting

Instructions on how to set Kerberos SPNs for HTTP exist, yes. But how do we set SPNs for secure PBIRS on HTTPS. Would it as easy as the following:
Setspn -a HTTPS/contosoreports.contoso.com ContosoRS
Setspn -a HTTPS/contosoreports ContosoRS
Thanks,
Jan

Thanks for your comment, @bishalgoswami.
This looks like a support request or product question rather than documentation feedback.
You might find a solution in the Power BI Community. If not, try our support page.


Rename OptionButton to DropdownButton

Closes https://github.com/godotengine/godot-proposals/issues/1998.
Affects file names & documentation.
I got the hang of this, but I hope I'll never see project_converter_3_to_4.cpp ever again. Unless...

Closing because there isn't a really strong consensus. Even last time, in the PR meeting, I recall the maintainers mentioning this PR's name to be superior, but even then, there was no strong opposition against the current one.
Overall it felt like a change for the sake of change. Maybe next time, because a whole class rename at this stage of development would really be too much.


Use recaptcha composer library

https://packagist.org/packages/google/recaptcha

https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/8c1fbd5ccb9cbebec1ff665d49256eb10d9b23de

https://github.com/nenuadrian/Secret-Republic-Hacking-Browser-Game-V3/commit/03723759fd5aefd3c3080bd92ccac059ab2f261b


Update zio, zio-test, zio-test-sbt to 2.0.15

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 1.0.18 to 2.0.15 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #259.


Fix typo in comment of samples_to_complex method

I'm currently trying to understand how to actually use rustfft to do create a spectrum analyzer and your code & comments are veeery helpful and a great insight! So first thanks a lot for this. Second, I think I found a typo? Not sure. I don't know shit about this :P

Hi, thanks a lot :)
Yes you're right, that's a typo!


Cancel a campaign

I want to cancel a selling campaign for my token. I can seem to find it. May I have some help please

@kumbitcoin what dApp do you use to initialize the selling campaign ? It allows maybe to cancel it too

I used Starta Launchpad App, now the app is disabled
Evans Silien
786-238-3466

NOTICE: This e-mail message is confidential, intended only for the named
recipient(s) and may contain information that is privileged. If you are not
the intended recipient(s), you are notified that the dissemination,
distribution or copying of this message is strictly prohibited. If you
receive this message in error, or are not the named recipient(s), please
notify the sender at either the e-mail address or telephone number above
and delete this e-mail from your computer. Thank you.
The sender believes that this E-mail and any attachments were free of any
virus, worm, Trojan horse, and/or malicious code when sent. This message
and its attachments could have been infected during transmission. By
reading the message and opening any attachments, the recipient accepts full
responsibility for taking protective and remedial action about viruses and
other defects. The sender's company is not liable for any loss or damage
arising in any way from this message or its attachments.
Nothing in this email shall be deemed to create a binding contract to
purchase/sell real estate. The sender of this email does not have the
authority to bind a buyer or seller to a contract via written or verbal
communications including, but not limited to, email communication
On Sun, Mar 24, 2024 at 2:02 AM La Loutre @.***> wrote:

@kumbitcoin https://github.com/kumbitcoin what dApp do you use to
initialize the selling campaign ? It allows maybe to cancel it too
—
Reply to this email directly, view it on GitHub
https://github.com/cryptoloutre/solana-tools/issues/10#issuecomment-2016705129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VZM4Y75KVOSVNL5ZHT6PTYZZT6BAVCNFSM6AAAAABFD7PZD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJWG4YDKMJSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@kumbitcoin did you try to fork their code and deploy your own UI ?

Thank you for your answer.
I haven’t because I don’t really know how.
If you have docs or step by step instructions I will learn and to do it.
Thank you, I really appreciate your support
Evans Silien
786-238-3466

NOTICE: This e-mail message is confidential, intended only for the named
recipient(s) and may contain information that is privileged. If you are not
the intended recipient(s), you are notified that the dissemination,
distribution or copying of this message is strictly prohibited. If you
receive this message in error, or are not the named recipient(s), please
notify the sender at either the e-mail address or telephone number above
and delete this e-mail from your computer. Thank you.
The sender believes that this E-mail and any attachments were free of any
virus, worm, Trojan horse, and/or malicious code when sent. This message
and its attachments could have been infected during transmission. By
reading the message and opening any attachments, the recipient accepts full
responsibility for taking protective and remedial action about viruses and
other defects. The sender's company is not liable for any loss or damage
arising in any way from this message or its attachments.
Nothing in this email shall be deemed to create a binding contract to
purchase/sell real estate. The sender of this email does not have the
authority to bind a buyer or seller to a contract via written or verbal
communications including, but not limited to, email communication
On Wed, Mar 27, 2024 at 9:39 AM La Loutre @.***> wrote:

@kumbitcoin https://github.com/kumbitcoin did you try to fork their code
https://github.com/StrataFoundation/strata and deploy your own UI ?
—
Reply to this email directly, view it on GitHub
https://github.com/cryptoloutre/solana-tools/issues/10#issuecomment-2022796828,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VZM47RSP7F5YMW7SB5MBDY2K4XTAVCNFSM6AAAAABFD7PZD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRSG44TMOBSHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@kumbitcoin I don't have docs because Strata didn't explain how to do it. But you can try to run in a terminal the following :
git clone https://github.com/StrataFoundation/strata/tree/master/packages/marketplace-ui
npm install
npm run dev
You will probably need to install git and NodeJS

Thanks, I'll check it out.
On Wed, Mar 27, 2024 at 10:27 AM La Loutre @.***> wrote:

@kumbitcoin https://github.com/kumbitcoin I don't have docs because
Strata didn't explain how to do it. But you can try to run in a terminal
the following :
git clone
https://github.com/StrataFoundation/strata/tree/master/packages/marketplace-ui
npm install
npm run dev
You will probably need to install git https://git-scm.com/downloads and
NodeJS https://nodejs.org/en/download
—
Reply to this email directly, view it on GitHub
https://github.com/cryptoloutre/solana-tools/issues/10#issuecomment-2022908389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VZM44FLU3PHYGZFOUG7ELY2LCLLAVCNFSM6AAAAABFD7PZD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRSHEYDQMZYHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Auto close tmux pane after opening the URL

Currently, after I open a URL in urlview pane, the tmux pane is still shown.
Is there any way to config tmux-urlview to close automatically?

Bump?


Fix peer dep.

The latest release hadn't fixed the peerdep. message since two pipe symbols are required for the logical OR per https://docs.npmjs.com/misc/semver

@all-contributors please add @brettz9 for code


Rename server module from dj to datajunction_server for consistency

Summary
This renames the dj module in datajunction-server to datajunction_server for consistency between the library name and the module name.
Test Plan
Locally + all unit tests

[ ] PR has an associated issue: #
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

@samredai more small steps towards consistency 😄


Fix Online User Count

It's related to https://github.com/emedvedev/slackin-extended/pull/108

2019-12-22T00:56:01.142090+00:00 app[web.1]: Sun Dec 22 2019 00:56:01 GMT+0000 (UTC) – fetching
2019-12-22T00:56:01.989525+00:00 app[web.1]: Sun Dec 22 2019 00:56:01 GMT+0000 (UTC) – online 0, total 1803 (+848ms)


screen shakes [BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
when im playing blox fruits when i punch or anything that would normally make the screen shake is severly over exaggerated when i do this when i have the software open i uninstalled and it wasnt doing it i really want to know what fixs it

the screen shake is severly over exaggerated

If you set the framerate limit to zero, does the problem go away?

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


Range features for latin

This doesn't work for things like `\e or \mite:
\setmathfont[range={"1D70B},math-style=upright]{Cambria Math}

I think this is now fixed.


Fe: navbar social links

The used to be in the footer but the footer is removed now.

duplicate, closing in favor of https://github.com/webaverse/app/issues/1290


Phoenix LiveView doesn't handle server restart gracefully

Hi @mtrudel,
First off, bandit is really looking great, excellent work on the whole thing!
I filed a bug report here with LiveView, but Jose asked me to file an issue with bandit instead.
The basic run-down of what is happening is:
When the server is restarting, bandit closes the websocket connection with status code 1001 (meaning: server going away). LiveView, however, is interpreting this as "browser navigated to another page" and subsequently does not attempt to reconnect, leaving the user with a spinning cursor until they reload the page manually.
Returning a status code 1000 instead would solve the issues with Phoenix. I'm not sure what your take is on that though from the perspective of standard compliance.

Fixed in 0.6.8!

Thanks for the quick turnaround on this, will push that to prod now. Shortest-lived fork I have probably ever had to run in prod.


NavigationBar ignores existing contentInsets

HidingNavigationBar works perfectly apart from this one point.
Before implementing:

After implementing:

The navigation hiding functionality works fine but I am losing my initial contentInset values.
tableView.contentInset = UIEdgeInsets(top: 8, left: 0, bottom: 0, right: 0)
Would be a great improvement.
I've used a hack in the meantime:
	private func updateContentInsets() {
		let navBarBottomY = navBarController.view.frame.origin.y + navBarController.view.frame.size.height + 8

If you have time to make a pull request for this that would be great!


Process is not defined

Describe the bug
I got error as below
Uncaught ReferenceError: process is not defined
    at Object.4043 (<anonymous>:2:13168)
    at r (<anonymous>:2:306599)
    at Object.8048 (<anonymous>:2:9496)
    at r (<anonymous>:2:306599)
    at Object.8641 (<anonymous>:2:1379)
    at r (<anonymous>:2:306599)
    at <anonymous>:2:315627
    at <anonymous>:2:324225
    at <anonymous>:2:324229
    at HTMLIFrameElement.e.onload (index.js:1:1)

I clone the repo from example and already install @ffmpeg/core
Expected behavior
It should work as expected because i have not changed anything from the example except I added setup proxy for CRA
Screenshots

Desktop (please complete the following information):

React version 16.3
OS: Big Sur 11.6
Browser Chrome
Version 97.0.4692.99

Getting the same issue

I am getting the same issue here did you find the solution?


graphql-api-construct addResolver for Query failing

Amplify CLI Version
12.2.5
Question
I can successfully add Appsync JS resolvers for any field in my api schema. However, if I try and add my own resolver for a Query such as getRegistryItem or listRegistryItems I receive the following error on deploy:
Resource handler returned message: "Resource already exists: arn:aws:appsync:eu-west-2::apis/*********/types/Query/resolvers/getRegistryItem"
Here is my resolver code:
    const registryItemTable = Table.fromTableName(
      this,
      "RegistryItemTable",
      amplifyApi.resources.cfnResources.cfnTables["RegistryItem"].ref
    );

    const registryItemDS = amplifyApi.addDynamoDbDataSource(
      "RegistryItemDataSource",
      registryItemTable
    );

    amplifyApi.addResolver("ListRegistryItems", {
      dataSource: registryItemDS,
      typeName: "Query",
      fieldName: "listRegistryItems",
      code: AppsyncCode.fromAsset(
        path.join(__dirname, "resolvers/registryItem/query", "listRegistryItems.js")
      ),
      runtime: FunctionRuntime.JS_1_0_0,
    });

Am I doing something wrong? I've tried disabling the creation of the query in the schema but then I get the following error:
"No field named listRegistryItems found on type Query"

UPDATE:
After looking more at this I have worked out how to do the following:

extend the generated resolvers by inserting a VTL template into specific slots (either "init" | "preAuth" | "auth" | "postAuth" | "preDataLoad" | "postDataLoad" | "finish")  with this:

  const amplifyApi = new AmplifyGraphqlApi(this, "AmplifyCdkGraphQlApi", {
      //...rest of graphql definition
      functionSlots: [queryFunctionSlot]

override an after mapping or before mapping using something like this (which overrides the final response mapping template in the generated pipeline resolver in this case:

    const existingResolverResource = amplifyApi.resources.cfnResources.cfnResolvers['Query.listRegistryItems'];
    existingResolverResource.addPropertyOverride('ResponseMappingTemplate', `$util.toJson($ctx.result)`);

But I don't seem to be able to override or add an appSync resolver for the main data request or response function of a generated query by the api definition?
Using addResolver seems to work on listRegistryItems if I define a custom Query or Mutation in the schema and disable the generated one like this but ideally I'd like to do something to override the generated Query.listRegistryItems.req.vtl  that I would do normally do with Amplify:

type RegistryItem
  @model #(queries: { list: null })
  ) {
  id: ID!
  name: String
  version: Int
}

type Query {
  listRegistryItems(id: ID!): String
 }

Any further information on this would be much appreciated.


Code Quality: Disabled cache in MessageFormat

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

From #15508
Closes #15554

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.
Test with C:/Windows/System32

@XTorLukas it's good progress, but I don't think this entirely fixed the issue. Try opening C:\Windows\System32 in main and then compare the results with stable.


Add fsspec to tuner

What does this PR do?

Fixes #4434
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

cc @ananthsub pls review.

cc @ananthsub pls review.

Hello @rohitgr7! Thanks for updating this PR.

In the file tests/trainer/test_trainer_tricks.py:

Line 233:15: E271 multiple spaces after keyword
Line 233:40: E225 missing whitespace around operator


"Message is undeliverable to connection" Issue

Just referring to this ticket
https://github.com/hyperledger/aries-mobile-agent-react-native/issues/112
It occurred with me when trying to create a connection between 2 AFJ wallets on iOS
If the inviter is ace-py it works fine, but if the invitation is coming from AFJ this error occurs

issue resolved


double-beep: Blacklist fantasy\Wpower\W11

double-beep requests the blacklist of the keyword fantasy\W*power\W*11. See the MS search here and the Stack Exchange search here.

fantasy\W*power\W*11 has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Copied from my comments in chat:
Unless it's specifically your intent to double-detect "fantasypower11.com", then the regex added to the keyword blacklist needs to exclude detecting "fantasypower11.com". If it is your intent to double-detect the domain, then that needs to be justified in some manner. Based on the content of the detected posts, I'm assuming that your intent is to catch this as a product name in the post body or as a username, without also catching the fantasypower11.com domain.
In that case, you can use a regex like fantasy\W*power\W*11(?!\.com(?<=fantasypower11\.com)) (regex101) (MS: in body, in username)
However, that's not really a good detection for usernames, as that username wasn't being used by them in the last 2 months.
In addition, it's not clear to me if it should qualify as a blacklist, because the primary thing you are, probably, trying to detect is this as text in post bodies, which only has 2 detections. Although it does have 3 other detections in usernames, it doesn't appear it's your intent to be catching usernames, given that you are not trying to catch the username that's currently being used for this spam.


Working with SASS dependencies in node_modules

 Issue by laurent-le-graverend
Wednesday Aug 09, 2017 at 02:55 GMT
Originally opened as https://github.com/Shopify/slate-cli/issues/133

Problem
It does not seem to be possible to import a sass file from the node_modules which have some dependencies.
Scope
Issue related to slate-watch.
Replication steps
In theme.scss:
@import url('../../node_modules/lib-a/scss/lib-a.scss');

In lib-a.scss:
@import '../_vendor/lib-b/sass/lib-b';

Error when using slate watch:
[10:47:05] change in src/styles - change theme.scss
[10:47:05] running task - build:css
[10:47:05] Error in plugin 'gulp-cssimport'
Message:
    Cannot find file '../_vendor/lib-b/sass/lib-b' from
    '/[project-path]/node_modules/lib-a/scss' (includePaths: )

More Information
It could be caused by the result of this logic:
https://github.com/Shopify/slate/blob/master/docs/theme-scaffold/index.md#sass-helpers
The tool is uploading local SCSS files, not compiling them. This might cause an issue with files from node_modules?
What would be the workaround?

@laurent-le-graverend the problem is most likely that we're using the "gulp-cssimport" dependency to handle stylesheet importing, which is not the same as using native SCSS imports. Looks like the problem in your case is that the current working directory for URLs is not being set correctly.
We will be dealing with this problem in the next version of Slate when we implement build tools to handle both scss and scss.liquid separately.


Add DisableRootPackagesPage field to Config struct in config.go

🎉 This PR is included in version 2.0.0 🎉
The release is now available here
Thank you for your contribution. ❤📦🚀


⚠ Bofrost.be has degraded performance

In dce89e9, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2192 ms

Resolved: Bofrost.be performance has improved in 558aa84.


[Umbrella] Security Assessment Process for Kubernetes sub-projects

Goal: Define, execute and improve on a community driven security assessment process for kubernetes sub-projects, which is self-service and welcomes beginners without prior security background to engage and contribute
Background
CNCF TAG Security has a well established process for project maintainers to engage them in a security review process. However, typically the scope is limited to CNCF projects not sub-projects of graduated projects like Kubernetes. In case of Kubernetes, SIG Security is taking up this responsibility to work within the project community through self-assessments as a partnership between maintainers and SIG Security Members
Scope

[X] Identify first project as a pilot for this process (Cluster API)
[X] Create a draft template for self-assessment and gather feedback (Self-Assessment Template)
[x] Publish draft template after incorporating feedback, in markdown under kubernetes/sig-security
[ ] Publish a task list that covers the entire security assessment process (channel creation, identifying contributors, presenting and resolving findings, etc.)
[x] Create label area/security-assessment to allow filtering of these requests (https://github.com/kubernetes/test-infra/pull/23070)
[x] Create a Github Issue Template to allow anyone to request a security assessment for a sub-project (https://github.com/kubernetes/community/pull/5922)
[ ] Perform retrospective on the process and identify improvements after every assessment

Related Issues
Cluster API pilot discussion: https://github.com/cncf/tag-security/issues/603
Cluster API tracker: https://github.com/kubernetes/community/issues/5814
Admission controller: https://github.com/cncf/tag-security/issues/731
/assign @PushkarJ
cc @rficcaglia @reylejano @savitharaghunathan @kubernetes/sig-security-leads
/sig security

/area security-assessment
(Trying out the new label that was merged today)

/unassign
/assign aladewberry

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle rotten

@aladewberry I updated the description to cover links to all the tasks in the list and marked them as done. Since we have a new sub-project that is going places and making things secure, the issue has served its purpose.
So, I am closing this issue since we can track future self-assessments through their own issues like you are already doing for vsphere-csi-driver #21 and #40 for cluser-api
Feel free to re-open if you disagree :) (Just doing this as part of backlog clearing on my end)


👁 Explorer focus mode to hide hidden files

I'd like to see a simple 👁 button in Explorer that toggles the visibility of hidden files (dotfiles), to cut visual clutter and improve focus on real project files.

I know this could be built as an extension, but it just makes so much sense to include this as part of Code to encourage developer focus.

This would be most helpful with monorepos, where dotfiles litter every package directory. Generally speaking if I've got two packages expanded, more than half of the entries visible in my Explorer at any time are dotfiles.

This is not a request for a files.exclude toggle in Explorer (#21403, #53369). If I add **/.* to files.exclude, I exclude dotfiles from fulltext search and quick open as well. I don't want that; I just want a more focused file listing for visual browsing.

This could potentially be improved to respect .gitignore files as well, similar to the existing search.useIgnoreFiles and UI toggle, but for Explorer. #38878 is a very popular feature request for this.

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks


dynamically choosing the instrumentation ?

This one looks technically complicated.
We choose the instrumentation as part of the objective function.
Maybe the optimization algorithm might decide to change the instrumentation ?
For example, depending on the arity of a discrete domain, we might prefer Softmax or UnorderedChoice.

I agree, still thinking about it and no solution yet. The thing is that you may also want to enforce the instrumentation as a user (typically for running expriments) so we would need to support both.
#513

Agree... maybe we do not change anything in the structure of the codebase, and simply the optimizer can detect the instrumentation, creates another instrumentation, and converts data in order to adapt...

Agree... maybe we do not change anything in the structure of the codebase, and simply the optimizer can detect the instrumentation, creates another instrumentation, and converts data in order to adapt...

This would be hard to manage and can be confusing for a user :s but that's probably what will happen eventually


Render Default Membership Plan

By default: it should highlight 'FALL 2019', also render a text says: 'selected' under the card
currently (not highlighted by default but it is default in the api):

Closed by #197, where the user must pick a plan before continuing.


Combine all action scripts into one

As discussed in #27.

The filter scripts need to be modified to pass input to the action script in the form of {type}-{value}. Currently, only the value is passed.

Oh yeah I updated the scripts in the .alfredworkflow file, but not in the .applescript files.

Really? Because I discovered the issue when running the workflow file itself.
When I try to play a song, the Alfred debugger returns the following log message:
Unknown type: 1333

I'm fairly confident that 1333 is the ID of the song I'm trying to play. I believe that there is some flaw in the type/value-parsing logic you added.

Yeah but that what because I did the following:

Edited the files in Afred and confirmed they worked
Updated the action file
Ran the python script, which then overwrited the files inside the .afredworkflow

It should be fixed now, however.

Well, would you look at that! It works now. :P
As I said, everything looks good. I'll merge this in and create a new release when I have some time to spare.


update doc refs

updated incorrect doc ref and solved DCO issue.
Sorry about the inconvenience :)

Hi, Daniel
Thanks. The toolbox is a very useful tool to help connecting two agents. Many thanks to your work.
I’m wondering if there is a tutorial to help me understand the workflow about how to use the mediator. If there is no official documentation, could you simply explain the workflow, probably I can help with the documentation 😊
Thank you!
Best Regards,
Wei
From: Daniel Bluhm @.>
Sent: Thursday, March 17, 2022 12:31 PM
To: hyperledger/aries-toolbox @.>
Cc: wy.dev @.>; Author @.>
Subject: Re: [hyperledger/aries-toolbox] update doc refs (PR #280)
Merged #280https://github.com/hyperledger/aries-toolbox/pull/280 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/hyperledger/aries-toolbox/pull/280#event-6260421070, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADFVM5HXXDJXRJKW57PNXWTVANM4RANCNFSM5Q7JI3QA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Is there any maintenance for this project?

A month with no commit...
We have a project depend on this, so if this project deprecated, we need a new plan.

It is not deprecated, feel free to contribute


Added Asus Zephyrus GA502

Description of changes
Added a configuration specific for the Asus Zephyrus GA502. My AMD card is on PCI:6:0:0 and the Nvidia card is on PCI:1:0:0
Things done

[x] Tested the changes in your own NixOS Configuration
[ ] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

@bors merge

@bors merge


please help

@galvesribeiro Hello, i really want to manage this library, please help me handle it

Sure, how can I help?
Get Outlook for iOShttps://aka.ms/o0ukef

From: 阿凌<EMAIL_ADDRESS>Sent: Wednesday, March 4, 2020 10:23:55 PM
To: OrleansContrib/Orleans.MultiClient<EMAIL_ADDRESS>Cc: Gutemberg Ribeiro<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [OrleansContrib/Orleans.MultiClient] please help (#2)
@galvesribeirohttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgalvesribeiro&data=02|01||d6bfdc3c82474d711f0108d7c0a3df83|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189682362926331&sdata=70XY%2B0vthnWUIhdobb%2BG454QuPlFRxkRIJMNuCRLHUY%3D&reserved=0 Hello, i really want to manage this library, please help me handle it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOrleansContrib%2FOrleans.MultiClient%2Fissues%2F2%3Femail_source%3Dnotifications%26email_token%3DABD64OGD7DIU6YZ6QH3USS3RF35KXA5CNFSM4LB7S5LKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IST7RFQ&data=02|01||d6bfdc3c82474d711f0108d7c0a3df83|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189682362936335&sdata=4PB%2BipzdbeN3golh34Qrb%2Fmcukjs3El%2FV4KjMiWZxys%3D&reserved=0, or unsubscribehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABD64OEVVC7NX767PWNAFBTRF35KXANCNFSM4LB7S5LA&data=02|01||d6bfdc3c82474d711f0108d7c0a3df83|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189682362946340&sdata=UckNRHPUIdsR6FO00ilTmrJqpHNmBZURgi18GHYrYiU%3D&reserved=0.

@galvesribeiro Is it possible to set as a member of this library

@galvesribeiro How do I deal with PR? I really don't think I have permission to deal with it.

you was added as Admin on this project and GitHub should have send an email to you with a link

In case you missed it, please check this link and accept the invitation https://github.com/OrleansContrib/Orleans.MultiClient/invitations

@galvesribeiro Sorry, I didn't see the email

@lfzm just sent again

@galvesribeiro thank you


[Backport 2.2] Fixed a bug with doc level trigger creation.

Description
Manually backporting PR #513.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #534 (9da7218) into 2.2 (0be72e8) will increase coverage by 0.06%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              2.2     #534      +/-   ##
==========================================
+ Coverage   52.68%   52.74%   +0.06%
==========================================
  Files         207      209       +2
  Lines        5444     5451       +7
  Branches      764      763       -1
==========================================
+ Hits         2868     2875       +7
  Misses       2574     2574
  Partials        2        2

see 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


PVC correction using GTM

For the PVC correction using GTM, the latter is computed as
  with vj being the total volume of ROIj and RSFi being the regional spread function for ROIi according to [1].
However in https://github.com/UCL/PETPVC/blob/master/lib/petpvcGTMImageFilter.txx#L161 it seems that the contribution of ROIi in ROIj is normalized by the number of voxels in ROIi instead of the number of voxels in ROIj. Is there a trick there that I am misunderstanding?
[1] Rousset OG, Ma Y, Evans AC (1998) Correction for partial volume effects in PET: principle and validation. J Nucl Med 39:904-911

I haven't checked this carefully, but this looks like a choice in normalisation factor. As GTM does return the correct ROI values, I suspect that this is an arbitrary choice, so I'll close the issue. Feel free to reopen if you don't agree.


ws.Account.SubscribeToAccountUpdatesAsync doesn't work in v2.6.7

After upgrading to v2.6.7.  It seems that ws.Account.SubscribeToAccountUpdatesAsync() isn't working either.

I've fixed this issue and will release tonight

New version has been released and issue fixed. Thanks for feedback. Regards.


HLS live streams have no timeline on UI

Have you read the Tutorials?
yes
Have you read the FAQ and checked for duplicate open issues?
No common issues
What version of Shaka Player are you using?
lastest 3,2,1
Please ask your question
HLS live streams have a hidden timeline
we are not able to navigate in the past because of the hidden timeline
may I know why the timeline is kept hidden when we load hls live streams?
Is it possible to display the timeline for HLS live streams?
like we have for the DASH manifest?
Thank you in advance

Hello there,
could you please answer my question?
thank you :)

Sorry for the delay.  HLS live streams are not seekable by default.  This is consistent with Apple's implementation, and with the lack of seek range information in HLS.
You can override this in the player config, however.  Just set 'manifest.availabilityWindowOverride' to some number of seconds.

Thank you it worked! :tada:


build: replace local e2e tests with e2e-monitor-ci

Closes ui#1007
This updates the .circleci/config.yml to invoke the monitor-ci pipeline for UI e2e testing. It won't run on forks since a credential is required to be present as an environment variable, so there is logic in the config to handle running from a fork.
A "candidate" docker image is built for the backend, tagged with the commit SHA and pushed to quay, and then the run-monitor-ci-tests from the ui repo is executed with this SHA so that the monitor-ci pipeline pulls the correct backend image to test the latest UI image against. Videos and screenshots of any failed tests are saved as artifacts that can be viewed from the public influxdb pipeline.
A follow-up PR will be opened to remove all remaining traces of JavaScript from the influxdb repository after this change is in place.

Higher-level question: can we delete ui/ as part of this? We're still receiving contributions there, and it'd be nice to nuke for clarity.

Absolutely, that will be the next order of business!


Invalid Route when creating route via cli

Hello,
This command, which looks correct according to the help output will not work for me.
$ flynn -c my-cluster \
        -a my-app \
        route add http \
        -s my-app-web \
        -p 50443 \
        -c /tmp/star-com/star-com.crt \
        -k /tmp/star-com/star-com.key myurl.mycompany.com
validation_error: Invalid route

I have checked that my cert is valid, and I have added the complete chain in the .crt file
Where can I find out why this has gone awry?

@rlangfordBV it's the -p 50443 that is causing you issues, Flynn currently only supports creating HTTP routes on the default router ports (typically 80 and 443), and so any value for -p currently leads to an invalid route.
What was your intention with that flag? Do you actually want the router to be listening on the externally facing port 50443?

Yes, we do.  We have a public load balancer for making flynn apps open to the public internet.  This ELB expects to be able to talk to the flynn router on 50080 and 50443.
The reasoning for this is not known to me, it's one of those "it was this way when I found it" things.

@rlangfordBV ah ok, so I broke this in https://github.com/flynn/flynn/pull/4569, sorry about that.
So if this is causing you issues in production I suggest pinning to a version from before I made that change (e.g. v20191231.0, let me know if you need help with that), and I'll try to ship a fix in the next week or so.

@lmars,
Cool, I can do that. I'll knock it out tomorrow.
Thanks!

Can I restore a cluster backup from a newer version to that older version?  It seems like the postgres databases could potentially have issues with that

@lmars,
Unfortunately, I don't think using an old version will work in our case as our cluster backups are based on the latest version of flynn, and have the database changes as well.  I've tried to spin up a half a dozen clusters to no avail.

Ok, I migated the app to k8s. ¯_(ツ)_/¯

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


[VS Code] The host rejected your request to join the collaboration session.

Error:
The host rejected your request to join the collaboration session.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.535

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.26.0

platformName
MacOS

platformVersion
17.7.0

@vlueviolet This message indicates that the host removed or rejected you, and you aren't allowed to join the session. Do you know whether the host accidentally removed you? If so, the workaround is to simply have the host stop sharing and then share again, since the rejection is only scoped to a specific session. Let us know if that helps unblock you!
In general, this behavior is "by design", but we're looking into whether there's a bug that might accidentally mark a user as removed. The user in #753 reported that the host didn't actually remove him, so there may be a bug somehwere here that we'll try to track down. Thanks!
// CC @cait1inhennessy

This has been resolved in the latest Live Share release. Let us know if you experience it again 👍


Does not highlight the contents of JSXExpressionContainer

const App = () => {
  console.log('highlight.js')
  return (
    <ul>
      {console.log('highlight.js')}
    </ul>
  )
}

The console.log in the ul tag is not highlighted by the parsing
https://codepen.io/linkontoask/pen/mdXBNax
It should be the same as the last console.log

Closing as a dup of #2324.


Why dependency for Newtonsoft.Json?

Because of the dependency on Newtonsoft.Json this library can't be used from a Universal XAML application built with C++. Wouldn't a solution leveraging the Windows.Data.Json namespace be sufficient?

There's no goal for Template 10 to support, C++, to be honest. After all, the T10 library is a .NET assembly, not a Windows Runtime Component - nothing about the T10 library is ready for C++. A WinMD would considerably dilute the capabilities of the library, too. I am not arguing that a C++/CX app is viable. It's just that Template 10 is really targeting the 80% of developers. C++ is certainly the top 10% of developers. It's going to be a lot of sacrifice and discipline to keep T10 focused on most solutions and not all of them. I am open to discuss this, so I will not close your question just yet. But does what I am saying make sense? At least at this moment, T10 is a library for most XAML+.NET apps.

@JerryNixon I am really happy that Template10 is a C# library, not a WinMD component. We have several projects with UWP; not all of them can use Template10 as some need extensive customization on many areas, but all of them can utilize parts of Template10 C# code. We have also learned a lot from the way Template10 handles things. Making it a WinMD component would alienate lots of developers using the good old Windows 8.1 C#/XAML templates.

Not only that, it would make it harder to build T10. That being said, I could see in the future a secondary library called something like Template10.WinMD but that's only a dream at this point.
Thanks for your comment. I hope it makes sense the direction we are going. And I hope you jump in and contribute/test the library. The more the merrier!


Set timeout on flutter:scrollUntilVisible

When flutter:scrollUntilVisible can't find the item, it goes on an infinite loop of waiting. It retries 3 times with a frequency of 60 and a timeout of 100000 milliseconds and after that, no other action is performed.
Flutter code
void main() {
  enableFlutterDriverExtension();
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: HomeScreen(),
    );
  }
}

class HomeScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: ListView(
        key: const Key("list"),
        children: List.generate(
            18,
            (index) => Container(
                  key: Key(index.toString()),
                  height: 100,
                  color: Colors.primaries[index],
                )),
      ),
    );
  }
}

Appium Code Node.js
  const driver = await wdio.remote(opts);

  const list = find.byValueKey('list');

  index = 0;

  while (true) {
    console.log(index);
    var visible = true;

    const item = find.byValueKey(index + "");

    await driver.execute(
      "flutter:scrollUntilVisible",
      list,
      { "item": item, "dxScroll": 0, "dyScroll": -100}
    ).catch((_) => visible = false);

    if (!visible) {
      break;
    }

    await new Promise(resolve => setTimeout(resolve, 1000));

    index += 1;
  }

  //Scroll back to the first. This is visible in the log, but never executed
  await driver.execute(
    "flutter:scrollUntilVisible",
    list,
    { "item": find.byValueKey("0"), "dxScroll": 0, "dyScroll": 100 }
  );

  await new Promise(resolve => setTimeout(resolve, 10000));

  driver.deleteSession();

Appium version: 2.0.0-beta.46
Flutter driver version<EMAIL_ADDRESS>

What about using flutter_driver directly instead of via Appium?
https://api.flutter.dev/flutter/flutter_driver/FlutterDriver/scrollUntilVisible.html
Appium Flutter driver calls the method, but how it works depends on Flutter/Dart VM, it is out of Appium Flutter driver. You could ensure that it works via flutter_driver directly.

Yes, with flutter_driver it works as expected, but I need Appium for testing a webview of the app.

sorry, i was wrong. it continued scrolling til https://github.com/appium-userland/appium-flutter-driver/blob/868cab2a460dc63445172826cd2bb72f56cd1501/driver/lib/commands/execute/scroll.ts#L72
What about using scrollUntilTapable?

The first time the app crashed
{"command":"scroll","finderType":"ByValueKey","keyValueString":"locals","keyValueType":"String","dx":-250,"dy":0,"duration":100000,"frequency":60}
[FlutterDriver] Connection to ws://<IP_ADDRESS>:33433/gJHvO9IaUkU=/ws closed

The second and third time it behave as flutter:scrollUntilVisible so it retried 3 times to find the element and then it threw an error (as expected)
This exception is correct, it would be useful to set the timeout and not wait for 120000ms * 3
Request failed with status undefined due to TimeoutError: Timeout awaiting 'request' for 120000ms when running "execute/sync" with method "POST" and args "flutter:scrollUntilTapable"

The problem is that after this, appium goes on a timeout loop on the next commands
Client-side:
Request timed out! Consider increasing the "connectionRetryTimeout" option.

Server-side:
{"command":"scroll","finderType":"ByValueKey","keyValueString":"locals","keyValueType":"String","dx":-250,"dy":0,"duration":100000,"frequency":60}
[debug] [FlutterDriver] <<< {"isError":false,"response":{},"type":"_extensionType","method":"ext.flutter.driver"} | previous command scroll

After 3 retries the client closes the connection, but the server keeps on logging the line above
scrollUntilTapable is wrapped inside a try-catch to handle when appium scrolls to the last item.
I tested it on another app, but the error is the same of the given example

I solved the problem with this function.
export const scrollUntilVisible = async (
  self: FlutterDriver,
  elementBase64: string,
  opts: {
    item: string;
    alignment: number;
    dxScroll: number;
    dyScroll: number;
    frequency?: number;
    timeout?: number;
  },
) => {
  const { item, alignment = 0.0, dxScroll = 0, dyScroll = 0, timeout = 1000, frequency = 60 } = opts;

  if (
    typeof alignment !== `number` ||
    typeof dxScroll !== `number` ||
    typeof dyScroll !== `number`
  ) {
    // @todo BaseDriver's errors.InvalidArgumentError();
    throw new Error(`${opts} is not a valid options`);
  }

  if (dxScroll === 0 && dyScroll === 0) {
    // @todo BaseDriver's errors.InvalidArgumentError();
    throw new Error(`${opts} is not a valid options`);
  }

  // Call waitFor function before the loop
  let isVisible = false;
  try {
    await waitFor(self, item, timeout);
    isVisible = true;
  } catch (error) {
    // log.error(`Item ${item} is not visible`);
  }

  // Use a countdown timer for the loop control mechanism
  let countdown = frequency;
  while (!isVisible && countdown > 0) {
    await scroll(self, elementBase64, {
      dx: dxScroll,
      dy: dyScroll,
      durationMilliseconds: 100,
    });
    try {
      await waitFor(self, item, timeout);
      isVisible = true;
    } catch (error) {
      // log.error(`Item ${item} is not visible`);
    }
    countdown--;
  }

  // Return the result of the scrollIntoView function
  if (isVisible) {
    return scrollIntoView(self, item, { alignment, timeout });
  } else {
    throw new Error(`Item ${item} was not made visible within the specified timeout`);
  }
};

@KazuCocoa do you feel it is easy to implement a timeout for this function? I am trying to implement something outside the driver, but the scroll command keep firing forever. If there is any other way to set this timeout, could you please give us an example?

what about https://github.com/appium-userland/appium-flutter-driver/pull/408 ?

Looks suitable! I made a workaround in my code to check if the element exists before calling scrollUntilVisible. Otherwise the driver was flooding Appium with repeatedly scrolling calls.

I haven't tested as no good example I have. It is appreciated to try the pr.
https://github.com/appium-userland/appium-flutter-driver/issues/372#issuecomment-1356792600 is also maybe good(?)

From the top of my head, I don't really know how I can use the driver from a specific branch... maybe I can edit the code inside my local lib?

The easiest is: 1) git clone the branch https://github.com/appium-userland/appium-flutter-driver/pull/408
2) Then build it
cd appium-flutter-driver/driver
npm run install

Then, copy build directly into ~/.appium/node_modules/appium-flutter-driver/build

You can keep the old build somewhere.

Let me test it now, I didn't have time enough to do this in the past few days.

@arctouch-thiagowerner Did you try my version? You can replace the function scrollUntilVisible with mine and run the test.
When the driver will not find the item, you will see three error messages as this Item ${item} was not made visible within the specified timeout.
Remeber that you need to have a try catch when you do  driver.execute("flutter:scrollUntilVisible"...) as this example https://github.com/appium-userland/appium-flutter-driver/issues/372#issue-1464551183

@MatteoAntolini I tried your solution, but for any reason it did not work here.
@KazuCocoa I am not sure if the problem of closing the session after throwing the error is something related to development environment.

@arctouch-thiagowerner What you mean? It doesn't compile, It gives you an error or it crash?

I added a try/catch as you mentioned, and used the script using the timeout argument. But the code is still flooding the script in the command line.

Modified https://github.com/appium-userland/appium-flutter-driver/pull/408 to handle loop. (I misunderstood some this driver's behavior)

Let me test it again.

@KazuCocoa worked as expected. We're good to go.

Thank you. The CI is not happy right now, so once I've fix it, I'll push a new version

1.13.0 has the change. The timeout name changed as waitTimeoutMilliseconds (changed the naming a bit)
https://github.com/appium-userland/appium-flutter-driver/blob/main/driver/lib/commands/execute/scroll.ts


Escrow: use PubKey instead of DataScript in various places

IMO this is nicer, feel free to accept or close if you don't like it.

Closing for now.


Add status field to Data model

@alexanderjordanbaker did you see https://github.com/apple/app-store-server-library-java/issues/32?


Canonicalize nulls in dictionary encoding

We should make sure nulls map to the same dictionary value. This is the case for varbin but likely not the case for primitives.

This happened in #138


feat($parseProvider): add the additionalIsIdent configuration method

Allows extending of the set of character allowed in identifiers used in Angular expressions.
Closes #2174 #3847

Most needed feature. Please someone merge it, dammit !

See #5881


fix: JB-502 Inconsistent text styling between cards - Colour and font size. -

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3777   👈

This comment was auto-generated by Graphite.

@wraeth-eth started a stack merge that includes this pull request via Graphite.

@wraeth-eth merged this pull request with Graphite.


missing a license

This repo misses a license. You could use https://github.com/sindresorhus/awesome#license as inspiration.

@arthurvr Thanks for the suggestion. I've just added a license for this repos. By the way, does every repo need a license?

@marcobiedermann Depends, but if you're in doubt and are able to add one, just add one.


dependencies enable flag not work as doc explain

Output of helm version:
version.BuildInfo{Version:"v3.1.2", GitCommit:"d878d4d45863e42fd5cff6743294a11d28a9abce", GitTreeState:"clean", GoVersion:"go1.13.8"}
Output of kubectl version:Client Version:
version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:30:10Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.7", GitCommit:"8fca2ec50a6133511b771a11559e24191b1aa2b4", GitTreeState:"clean", BuildDate:"2019-09-18T14:39:02Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.): AKS
We need install a chart with some dependencies and som time we need disable some dependendt charts....for example I have a chart definition like this:
apiVersion: v2
name: app-deployment
description: A Helm chart for deploying Technisys' microservices-based CDP solution
type: application
version: 1.0.0
appVersion: 3.0
dependencies:
  - name: appA
    version: 1.0.1
    repository: https://myrepo.azurecr.io/helm/v1/repo
    enabled: false
.....

But after apply command helm upgrade the helm try to deploy appA chart. How we can include declaration to avoid chart dependencies installation with enabled flag?

Could be functionality lost from Helm 2, switching from requirements.yaml to chart.yaml. May be a bug

There is definitely a bug in here that I have been hunting for this week. There are a few related issues tracking similar problems with enabled.

@technosophos are you still working on this bug? If not I can have a go at it.

I was looking into this issue. Turns out to be quite related to #7887 however this documented usage of enabled never seemed to be supported.
I think this comes from the fact that in v2, the Requirements struct had a field called Enabled. But that was an "internal" property, which was used to first make sure that all subcharts are enabled before iterating over the tags&conditions to determine if a subchart should be included. Looking at the v2 documentation doesn't give any reference to this Enabled field: https://v2.helm.sh/docs/developing_charts/#tags-and-condition-fields-in-requirements-yaml

I think there might still be a way to make this work, and will try to submit a PR shortly. But I'd also be inclined to think that having two other options for disabling a dependency chart might be enough...

This seems to warrant further investigation.


Ambiguous call

Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0121	The call is ambiguous between the following methods or properties: 'Microsoft.Extensions.DependencyInjection.OptionsServiceCollectionExtensions.Configure(Microsoft.Extensions.DependencyInjection.IServiceCollection, System.Action)' and 'Microsoft.Extensions.DependencyInjection.OptionsServiceCollectionExtensions.Configure(Microsoft.Extensions.DependencyInjection.IServiceCollection, System.Action)'	SwiftCourier.Web..NETCoreApp,Version=v1.0	C:..............\Startup.cs	68	Active

in startup.cs I have
services.Configure<RequestLocalizationOptions>(
                options =>
                {
                    var supportedCultures = new List<CultureInfo>
                        {
                            new CultureInfo("ru-RU"),
                        };

                    options.DefaultRequestCulture = new RequestCulture(culture: "ru-RU", uiCulture: "ru-RU");
                    options.SupportedCultures = supportedCultures;
                    options.SupportedUICultures = supportedCultures;
                });

@dvdobrovolskiy do you perhaps have a mix of RC1 and RC2 packages? Check your project.json and make sure everything is using rc2-final packages. If you can't find it, please share your project.json file and we can take a look.

nope, rc2-*

hm
rc2-3002702 instead of rc2-* helped. Issue resolved

@dvdobrovolskiy ah ok, so we can close this issue?

I think so. Just strange why it appeared any way

I am having the same issue  :'(   there is no documentation about it

Seemed like outdated code example.
this works (in Configure)
var supportedCultures = new[]
{
new CultureInfo("ru-RU"),
};
        app.UseRequestLocalization(new RequestLocalizationOptions
        {
            DefaultRequestCulture = new RequestCulture("ru-RU"),
            // Formatting numbers, dates, etc.
            SupportedCultures = supportedCultures,
            // UI strings that we have localized.
            SupportedUICultures = supportedCultures
        });


Url domain detected wrongly

URL :: http://www.dummy.pb.gov.br/
Domain Detected via weburl.getDomain() :: pb.gov.br
Actual domain :: dummy.pb.gov.br
pb.gov.br is TLD(check here: https://publicsuffix.org/list/effective_tld_names.dat ) but crawler4j weburl object returned this as a domain.

I believe this is really a duplicate of #318, the issue is caused by the same code not determining etld / private subdomain as per user expectation.


GSE130148,EGAS00001001755 - lungCellularCensus

Project short name:
lungCellularCensus
Primary Wrangler:
Ida
Secondary Wrangler:

Associated files

Google Drive: https://drive.google.com/drive/folders/1LXtmfaRLMLGPkc1Zq_7dCd5NGFHFkAWQ

Published study links

Paper: https://doi.org/10.1038/s41591-019-0468-5

Accessioned data:

GSE130148
EGAS00001001755
EGAS00001002649

Ingest
https://contribute.data.humancellatlas.org/projects/detail?uuid=c0518445-3b3b-49c6-b8fc-c41daa4eacba
Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[x] Collect any matrix and cell-type annotation files
[x] Upload sheet to validate metadata
[x] Check linking using ingest graph validator
[x] Transfer raw files to ingest to validate data files
[x] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[x] Submit dataset to Production
[x] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

The only available data are cell by gene count matrix

For managed access data from healthy donors h5da files are available here:

parenchyma: 6 donors as expected
nasal: 2 donors as expected
bronchial: includes biopsy 6 and lung brush 3

Data available

Source
Publication name
Source name
Tissue
#Donors

GEO
Lung resection
Lung resection
Lung lobe
4

covid19cellatlas
Lung trasplant
Parenchyma
Parenchyma
6

covid19cellatlas
Bronchoscopy biopsy
Nasal
Upper airways
2

covid19cellatlas
Bronchoscopy biopsy
Bronchi
Bronchioli
6

covid19cellatlas
Bronchoscopy biopsy
Bronchi
Lung brush
3

The IDs of the Bronchoscopy donors from the supplementary materials don't match those coming from available data. They have been matched up using the information on cell count available in the supplementary material, a brilliant idea from @Wkt8. Otherwise I might have tired to ask the authors but they might have been unable to give me that information due to privacy concerns.
Missing donor metadata
Donor ARMS052 has no metadata available, despite being included in the dataset and cell count information from the supplementary materials. I will contact the authors to determine whether the metadata is unavailable intentionally or whether the ID is incorrect.
Now that the picture of the donors and their specimen is clear I can update the metadata spreadsheet with the new information.
I still have to fill in the analysis tab.

For the nasal and lung brush no dissociation protocol is explicitly stated, I'm assuming that the bronchoscopy dissociation protocol covers these specimens as well.

Waiting for authors' reply

Information on mystery donor ARMS052
covid19cellatlas: does the data published come from smartSeq2, 10x or both?

I fixed a number of errors in the metadata that showed up in ingest and reorganised the analysis protocol and files tabs.
To do:

[ ] Waiting for the authors to reply, I might ping them after the holidays and ask if the available data had corrections applied to it
[ ] fix the grant information

The authors confirmed that the data published in the covid19cellatlas is exclusively 10x with corrections already applied.
I updated the Data avilable table to summarise the information

@wei assigned as sec reviewer.

Looks good! Well done - I especially like how neat and informative the dissociation protocols are.
Just a couple things:
Specimen_from_Organism:
specimen_from_organisms for the transplants have a typo 'translpant' instead of 'transplant'
Similarly the linking from CS to spec_from_organism will also need to be changed!
Cell_suspension:
This isn't necessary but if you wanted the estimated_cell_count to show on the project you'd need to put in the cell counts at the cell suspension level which are technically available by summing the different cell types from the supplementary table S1
Analysis_File
File source in Analysis_File needs to be added in. 'GEO' for the GEO ones.
I would use 'Publication' for the covid19cellatlas ones.
There's also the lingering question (depending on what the contributor says) about if you are going to keep or delete ARMS052!

Thanks for reviewing!
I applied the changes you suggested and I am exporting the project.
donor ARMS052
The authors told me that they are working on a new manuscript that will include part of the data I wrangled for this project and new data (some of it coming from donors from this wrangled publication).
I'm opting to keep donor ARMS052 in the project with the minimal metadata available right now, since from what I understood we are not too concerned with donors being duplicated in the HCA.
I hope I can update with additional metadata when they become available through a the new publication the authors spoke of

Donor ARMS052 (748aab09-0dc1-4dd1-bda5-dbc29c86cafb contains age units but no age information.
From what I understand reading the thread above, this information is not available
AI:

[ ] Delete Age unit for that donor
[ ] Re-export
[ ] Fill out import form

@idazucchi to update the donor.

@ESapenaVentura created the ticket to update the graph validation to check unit and donor age.

the dataset was exported as part of release 13 but during the indexing there were some errors and we found out that there was a metadata error with donor ARMS052 (see Enrique's comment above).
I fixed the mistake, re-exported for release 14, and marked it for release 14 in ingest and zenhub

Enrique spotted that some contributor had n/a as institution. I've fixed the project metadata and re-exported it (metadata only)
Is the spreadsheet also exported now?

verified in the data browser!

vieira19_Alveoli_and_parenchyma_anonymised.processed.h5ad is corrupted, needs to swapped out

I've swapped the corrupted file.
Updating file metadata
I tried deleting the file metadata (checksum, cloudUrl, size) via api - no error message but the metadata was still there
I tried syncing the file from an hca-util area --> this effectively swapped the file and updated the file metadata
Export
First export - new file was exported but the file descriptor still had the old file's metadata and export date was still 2023
I deleted the relevant file descriptor and re-exported
 gsutil ls -l gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/
       578  2023-07-27T06:01:18Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/aaa051e9-6f3a-4461-a8cc-adb3d84e13f2_2022-01-11T15:17:55.983000Z.json
       560  2023-07-27T06:01:22Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/abcff8d9-8624-4dbc-a963-58edc994f336_2022-01-11T15:17:55.919000Z.json
       576  2023-07-27T06:01:22Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/c8669817-1d49-4f1c-850e-f921ac5d6db0_2022-01-11T15:17:56.010000Z.json
       593  2024-03-20T15:26:49Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/ee5b3cdf-26b1-4b9b-8a2b-100e0a33ef08_2022-01-11T15:17:55.965000Z.json
       564  2023-07-27T06:01:15Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/f2bbb21c-9df9-4607-92e0-98ae5caa9927_2022-01-11T15:17:55.885000Z.json
       554  2023-07-27T06:01:17Z  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/c0518445-3b3b-49c6-b8fc-c41daa4eacba/descriptors/analysis_file/ff7d5b00-aec6-47c8-b2c7-e2fa974ca46f_2022-01-11T15:17:55.947000Z.json

the descriptor content is updated, and the export date is correct - but the filename still has the date of the first export 2022-01-11
I think this is a bug

Filled import form

the file was not updated correctly - I'm investigating

File has been fixed following this SOP. This change has been verified in the browser.
Note that in browser, the matrices tab does not show any file although we have analysis files however, we can access files either with the download tab or via filtering the specific project in explore.


Auto-parameterised values

Premise
A lot of values fed to Velocity are numbers, but quite often they have various bits of text surrounding them. The most obvious things are single number values which often simply have a unit such as "px" appended to them.
For animating colours we have more complicated rules in Velocity that split each colour into three (or four) separate attributes, and animate each separately.
For transform values we have multiple attributes that aren't ordered.
Something needs to exist to let us animate these "non-standard" values in such a way as to be logical and extendable.
Solution
How about simply comparing two strings and parsing out the numbers, then animating those back into the original string instead. That way it would be simple to animate from "calc(78% - 4em)" to "calc(100% - 2em)" without having to do it as a custom function.
As an added bonus passing a simple transform value would keep the order, and colours could be directly animated.
Bonus / Gotchas

Pulling the colour replacement code out could then include all the named web colours translated directly to rgba(1,2,3) values (specific attributes would get automatic replacement, anything else can have an exposed function to do it if needed) - currently only a small number of named colours are supported.
rgba(red, 0.7) could be automatically "fixed" to work.
rgb may need to be translated to rgba on the fly (with the alpha set to 1).
Multiple attributes (ie, box-shadow) would work directly.
transform could be used directly, the "legacy" separate transform shortcuts could just be added to the transform string in the order they appear.
First version would need to be force-fed only (transform specifically needs support code to translate the computed css back into the original string).
Whitespace needs to be ignored.
Can add pure text values for attributes like position that don't need changing, but can revert to initial value after finishing.
Possibly add automatic unit conversion when a number+unit doesn't match between the start and end (wouldn't be right for calc however).

Pseudo-code
if the attribute supports colours
   replace any /(rgba\()?colour(,\d\))?/g with the correct rgb / rgba colour value
endif

for each letter in from and to
   if they are digits
      parseFloat to an array each
      skip all digit characters in source
      if the numbers are the same
         put the number into the format string
      else
         add a %s to the format string
      endif
   else if they are different and neither is a space character
      report an error and end immediately
   else
      add the character to the format string
   endif
endfor

Future

Support other colour formats such as hsv() etc
Support non-forcefed values (wants testing first)
^- Support re-ordering of arguments such as box-shadow:inset on IE / everything else

Potentially Fixes / Related Issues
#677
#459
#650 (possibly)
#625 (needs mobileHA to be added to transform)
#562
#561
#543
#506
#471
#466
#403
#388
#263

This could do with some testing before being promoted to the next minor release - it works simply by taking two strings with differing numbers and animating those numbers.
If you want to change units then using calc is the way to go - $().velocity({left:["calc(50% + 0px)", "calc(0% + 50px)"]}); - making things like this automatic is a future possibility.

I am hitting a bug when animating to 0, I think directly related to this.
I have an animation like this (using velocity-react):
<VelocityComponent animation={{ marginLeft: !myFlag ? '0%' : '25%' }} duration={300}>
I've found that the animation breaks when animating from %25 to %0 (%0 to %25 works fine).
I've done some digging, and I see that this is triggering a "pattern", which comes out to calc({0}px + 0%). The start pixel amount is calculated fine, and the end pixel amount is 0.
This ends up creating a tweenContainer with a startValue of [400, 0] (using 400 px for example sake), and endValue of [] (an empty array). The patternReplace code chokes on the empty array endValue. If I change the code below to set an endValue of [0, 0], everything works as expected.
From line 3475 in velocity.js:
// Different units, so put into a "calc(from + to)" and animate each side to/from zero
var nStart = parseFloat(tStart),
    nEnd = parseFloat(tEnd);

pattern += (inCalc < 5 ? "calc" : "") + "("
    + (nStart ? "{" + aStart.length + (inRGB ? "!" : "") + "}" : "0") + uStart
    + " + "
    + (nEnd ? "{" + (aStart.length + 1) + (inRGB ? "!" : "") + "}" : "0") + uEnd
    + ")";
if (nStart) {
  aStart.push(parseFloat(tStart));
  aStart.push(parseFloat(0));
}
if (nEnd) { // This results in an empty array because nEnd is 0. If I change it to if (nEnd !== undefined), things work for my case. Not sure if that is the appropriate solution
  aEnd.push(parseFloat(0));
  aEnd.push(parseFloat(tEnd));
}

I don't know enough about the code structure in this project to propose the correct fix. Maybe this case shouldn't be hitting the "pattern" code in the first place, since the start and end could be kept in consistent units (either % or pixels).
Let me know if I can provide more information.

Specifically - the auto-parameterised values are purely for force-feeding right now, it's not supposed to support "normal" values yet as I've not got the time to go through and fix it up for otherwise (specifically there's differences in how the various browsers report things that aren't a single value).

Thanks for the response, can you explain what you mean by force-feeding?
In my example (<VelocityComponent animation={{ marginLeft: !myFlag ? '0%' : '25%' }} duration={300}>) I wasn't trying to trigger the auto-parameterised functionality, I was just trying to animate a percentage value. This always worked in previous versions, so I thought I would bring it to your attention.

Ah - shouldn't have triggered without it - http://velocityjs.org/#forcefeeding - it's basically saying "this is the start and end values", rather than taking the "current value". The big issue with the "current value" is that it's always in pixels (unless a previous animation within velocity set it otherwise)...

Ah thanks for the clarification on forcefeeding.
So I think one of the following things must be happening:
VelocityComponent (part of velocity-react) is triggering forcefeeding. It is forcefeeding a start of 25% and end of 0%. Velocity is converting the 25% to pixels, and leaving the 0% as a percent. The forcefeeding code is choking on the 0% endpoint due to the code I pointed out above.
VelocityComponent is triggering forcefeeding. It is caching/looking up the pixel value of the margin property. It is then passing velocity a start value in pixels, and an end value of 0%. The forcefeeding code is still choking on the 0%.
However it's happening, it still seems to me like there is a bug in the pattern code, since it will provide an empty array as the start or end point when a value is 0.

Think I got it, it was pushing to start or end, when it should have put them into start and end in pairs. Also realised I was using parseFloat() on the same thing twice, so a slight optimisation there too - let me know if the latest push works correctly so I can promote a release :-)

Yes, exactly! That's what I thought but was running my head in circles a bit on the pattern generation code. I will try it out soon - but maybe won't have time until next week.

That's ok, I'm recovering from the flu, so my brain is a bit slow at things right now ;-)

Rycochet - newbie question if you know - what is the best way to test your commit in my project (which uses NPM to install velocity-react, which in turn installs velocity-animate)?
Do I clone this repository, checkout your commit, maybe build source somehow (maybe not needed?), and then reference the local copy somehow in my package.json? Would I also need to clone velocity-react to have it point to the local version of velocity-animate?
Or alternatively, I could just clone this project directly into my node_modules directory (first removing the copy of velocity-animate that npm installed) - does that make more sense?

Temporarily do
npm install julianshapiro/velocity

...then when you've finished testing just do...
npm uninstall julianshapiro/velocity
npm install

...note that it's not saving either time, just loading locally and going back to the "correct" one after finishing - though do wait till the npmjs version has updated with a new release if you're relying on a bugfix version ;-)

Phew so simple, glad I asked!

They improved things a lot in the last year ;-)

Just had a chance to test this in my project - and everything is working great.


Make the addSceneLocationEstimate function fileprivate.

I suppose this has to do with Swift 4, but this way it's compatible with the "legacy" Swift 3.2 projects. (for now)

With this being the only change needed to support Swift 3.2 it makes sense to do it. Thanks for contributing.

Not sure if this PR is responsible, but trying to use the repo on an ObjC project, I'm getting this error when compiling:
SceneLocationView.swift:491:14: 'addSceneLocationEstimate' is inaccessible due to 'private' protection level
Changing it to public made the project compile again


Build error : Cannot find type 'SFSymbol' in scope

Hi, this looked like an interesting package. I tried using it in a sample app but I get the following error when trying to build after adding the package:
UIError.swift:26
"Cannot find type 'SFSymbol' in scope"
Good luck with this and thanks for any info you can provide with this issue.

Yeah, we’re still merging some changes. Check back tomorrow. However, we’re made this public for easier distribution to clients and aren’t planning to write up a readme or make a sample project at this time

Closing this as fixed by #2. I'm not tagging an initial release just yet. We're going to be adding more SwiftData integrations next week and some types are going to get renamed. Once those changes are in, there will be a release.


File tree does not update when a file is created/deleted externally

Lapce Version
nightly-50216c4
System information
Debian Sid
Describe the bug
When you open a workspace, files created and deleted externally (say, terminal or file manager) do not get reflected in the tree view.
Additional information
Other information that can be used to further reproduce or isolate the problem.
This commonly includes:

screenshots
logs
theories about what might be going wrong
workarounds that you used

This issue appears to be resolved in nightly-0c7c95e. Closing this now...


Remove version: RingCentral.RingCentral version 24.2.280.10541

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

/azp run

URL: https://app.ringcentral.com/download/RingCentral-24.2.280-x64.msi
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)

/azp run


Added functionality to Vector2

Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[X] Has this change been discussed on the forum or in an issue before?
[X] Does the code follow the SFML Code Style Guide?
[ ] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
Added mag, dist, dot, and cross
For magnitude of a vector, distance between two vectors, dot product of two vectors, and cross product of two vectors, respectively.
This PR is related to the issue #1810
Tasks

[ ] Tested on Linux
[X] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

https://github.com/SFML/SFML/pull/1987#issuecomment-1025103342

Closing, vector extensions will be handled with #1979.


Add options struct for util DoGet function

What does this PR do?

Add a DoGetWithOptions function which takes an option struct with options for DoGet.
Motivation

Being able to provide an auth token as option and stop using the global variable.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Covered by unit tests.

/merge


Output outside frame

This code:
\begin[papersize=a5]{document}
\script[src=packages/frametricks]
\nofolios
\set[parameter=document.parindent,value=0pt]
\font[family=Kohinoor Devanagari,style=Regular,language=hi,size=18pt]
गुलों में रंग भरे, बाद-ए-नौबहार चले
\showframe[id=content]
\end{document}

Produces this output. Note the rendering of text outside the frame.

Any clues? FWIW, I am seeing this only with the Devanagari fonts on OS X, not any others.

This does replicate for me on Linux copying the Kohinoor font files from a Mac.

OK, I understand this. Harfbuzz's opentype implementation doesn't know how to measure a glyph's descender. But Behdad has convinced me that I don't need to and that there are better ways of handling linespacing than TeX's height/depth algorithms. This is happening within the linespacing package, which implements nearly everything apart from the positioning of the first baseline on the page.

No, this is a bit more complicated than I thought. y_bearing and height are both coming out of hb_font_get_glyph_extents as zero for Indic, but not for Latin. It will be fixed when we move to the new linespacing package, but I am not sure why it's broken now. @behdad?

Oh, I don't understand. hb-shape with OT funcs is not showing any extents for Latin or Indic, but SILE is correctly getting heights and depths for "abc" from somewhere anyway.

> makenodes("abc गुलों", { family="Kohinoor Devanagari"})             ..""
N<14.0380859375pt>^7.568359375-0.146484375v(abc)
G<2.642578125pt plus 1.10107421875pt minus 0.73404947916667pt>
N<6.6064453125pt>^0-0v(गुलों)}"

"abc" has height and depth, गुलों doesn't. Same algorithm used to get both.

"abc" has height and depth, गुलों doesn't. Same algorithm used to get both.

CFF font versus TrueType?  Is this the algorithm using glyph bounds?

Kohinoor Devanagri is listed as an ‘OpenType Postscript’ font. So that makes it a CFF font.
There is a code-path in justenoughharfbuzz.c here at the latest commit on master:
    /* Harfbuzz's support for OT fonts is great, but
       there's currently no support for CFF fonts, so
       downgrade to Freetype for those. */
    if (strncmp(font_s, "OTTO", 4) == 0) {
      hb_ft_font_set_funcs(hbFont);
    } else {
      hb_ot_font_set_funcs(hbFont);
    }

This is before hb_font_get_glyph_extents is called. So is that the problem then?

That's basically it. The final piece of the puzzle is that Kohinoor is a TrueType Collection. So it's a CFF font - so we should use the Freetype functions - but because it's a TTC, it doesn't start with the "OTTO" tag at the beginning of the file. So the wrong functions are being picked up here.
I've extended the comparison to also use Freetype when dealing with a TTC. It wouldn't be too hard to sniff around in the data and extract the sfnt header for the appropriate font within the collection, but I can't immediately do that with get_table and it doesn't actually matter if we use Freetype - it's just that the OT functions are faster. So it just avoids a possible optimization.
Anyway, again the proper fix is to use typoAscender to set the position of the first line, and I'm hoping to get that done soon.

Passing HB_TAG_NONE to hb_face_reference_table should give you the whole font (or you can use hb_face_reference_blob which does the same).


replace the email with play store link

I think it is better then mail, since you need a link to the app in this page.

Good idea, next release will do this


[NEW ALGORITHM] Number of Recent Calls

Name:
[NEW ALGORITHM]  You are given a list of recent calls made. Each call is represented as a string of length 14 containing the phone
number that was called and the timestamp in the format "hh:mm:ss". You need to count the number of calls made
in the last 5 minutes. USe Queue data structure to solve this problem
About:
Implementation of queue data structure.
Input:
Enter the number of calls along with number and time stamp.
Output:
The number of calls made in last 5 mins.

Labels:
new algorithm, gssoc-ext, hacktoberfest, level1

Assignees:

[x] Contributor in GSSoC-ext
[x] Want to work on it

Additional Information:
I want to work on this please assign it to me.

@shuvojitss kindly assign me the issue.

@jashwanthbavandlapalli See I am not the author that's why I can't assign you.
You kindly raise a new unique issue. If it's gets accepted the author will assign it to you.

Do not create unnecessary PR until you complete the previous one.


Logout reducers

There are some issues with your Logout reducers:
case LOGOUT_SUCCESS:
      return Object.assign({}, state, {
        isFetching: true,
        isAuthenticated: false
      })

In case of success, isFetching shall be set to false.
Moreover, LOGOUT_REQUEST is missing.

@sylvainlap In the actions file there is this function logoutUser which first dispatches requestLogout ( where isFetching=true) and then afterwards receiveLogout ( where isFetching=false).
// Logs the user out
export function logoutUser() {
  return dispatch => {
    dispatch(requestLogout())
    localStorage.removeItem('id_token')
    dispatch(receiveLogout())
  }
}

However, LOGOUT_REQUEST is missing from the auth reducer. I just added it myself to my project, I'm rewriting this in typescript.

also, just to add on to this,  in LOGOUT_REQUEST user creds are added but never cleared.


create a new filter for raritan SX2

Noticed SC4S supports raritan vendor sourcetype:dsx.
I have a raritan SX II and all it logs fall into "sourcetype:fallback"
I think I it could be great and simple to somekind of edit the current supported "sourcetype:dsx" and create a new one as "sourcetype:sx2" using the same vendor filter as a template.
I was trying to edit and create the current Know Vendor Raritan "sourcetype:dsx" but I don't have enough experience.
Thank you,

Hi, @amalitol please provide .pcap file or raw message.

Hello, @amalitol ! Can you please to provide us .pcap file? We need to know real log format that producing your device.

Hi @ikheifets-splunk The .pcap file is on its way.
Thank you,

Hi @ikheifets-splunk && @mateuszpierzchala-splunk
.pcap attached in a .zip file as well as the raw messages in a note above.
myrarirancap.zip
I'm waiting for your comments.
Thank you,

Hello, @amalitol !
In your attachment I found 2 different formats of logs:
<14>[Port Disconnected]: Port 'ccserveraggsw1 - N9K- Top' disconnected by user 'admin' from host '<IP_ADDRESS>'.

<189>drnetappsw1: 2023 Jul  8 20:31:32.389 EDT: %VSHD-5-VSHD_SYSLOG_CONFIG_I: Configured from vty by securescan on<EMAIL_ADDRESS>
Which of them I need to support?
P.S. More details on the bottom:

I would say the first format.<14>…
On Tue, Jul 11, 2023 at 9:23 AM Ilya @.***> wrote:

Hello, @amalitol https://github.com/amalitol !
In your attachment I found 2 different formats of logs:
<14>[Port Disconnected]: Port 'ccserveraggsw1 - N9K- Top' disconnected by user 'admin' from host '<IP_ADDRESS>'.
<189>drnetappsw1: 2023 Jul  8 20:31:32.389 EDT: %VSHD-5-VSHD_SYSLOG_CONFIG_I: Configured from vty by securescan on @.***
Which of them I need to support?
P.S. More details on the bottom:
[image: Screenshot 2023-07-11 at 12 55 28]
https://user-images.githubusercontent.com/138466237/252672909-90d5e11c-26ec-4934-a7a5-85f90adc5fe5.png
—
Reply to this email directly, view it on GitHub
https://github.com/splunk/splunk-connect-for-syslog/issues/2085#issuecomment-1630827345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQEL4CLFOXZFNJNMW3DLPTXPVHWVANCNFSM6AAAAAAZRY3DXQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@amalitol Kindly dont post pcap to open sourced space, from next time kindly use engineers email or support case, i have deleted the pcap from the comments.

Hello, @amalitol !
It's seems your device generating very generic message by this reason we're asking to use non-default port (custom) when sending logs from this device. Because for such generic messsaes there is only one way to identify the source (by device source port or device source hostname).
P.S. The same situation with Raritan DSX (that you mentioned), it's sending logs using 9001 port by TCP (instead of default port). because we can't indentify this device by another way. I can prepare Pull Request that will identify your device by some fixed port.

Hi @ikheifets-splunk,  It makes sense to me. What to do next?

Hello, @amalitol !
You just need to send logs to 9001 port under TCP (same as Raritan DSX) and that already will work (it's didn't go to fallback) and your device would be identified as "Raritan DSX". If you need to separate DSX and DSX2 devices we can make pull request.


ISPN-10724 Ignored caches per server causes a performance regression

https://issues.jboss.org/browse/ISPN-10724

run performance tests please
cs-hotrod dist writes

run performance tests please
cs-hotrod dist writes

run performance tests please
cs-hotrod dist writes

run performance tests please
cs

Perf tests look good to me, some benchmarks show better perf than JDG 7.3.1 or slightly worse, but hard to tell if this is noise or not

run performance tests please
cs

run performance tests please
cs

run performance tests please
cs

run performance tests please
cs

run performance tests please
cs

I collected the results for all perf tests executed above, to figure out a supposedly perf drop on HotRod DIST writes compared to master before the 1st Igored Cache PRs:

Test
Throughput 10 threads
Mean 10 Threads
Throughput 100 Threads
Mean 100 Threads

Lookup
16500
4709966
18701
42464694

Lookup
19768
3927914
22975
34585711

Boolean
21604
3597446
26397
30100640

Boolean
17033
4556401
18954
41935692

Boolean
20094
3841325
23153
34295585

ShortCircuit
21765
3552460
26512
29915599

ShortCircuit
20228
3846750
23368
33999102

Where Lookup is when reading from a ConcurrentSet (always empty), Boolean is bypassing the Set lookup if it's empty, and ShortCircuit is always returning  a hardcoded false for the isCacheIgnored method. This PR settled in the Boolean method of checking for ignored cache on remote requests.

Integrated, thanks Gustavo!
We'll have to look further into PerfAck and see why the numbers are so different when the code didn't change.